esp_bignum aborts randomly when in task with no CPU affinity / Timed out waiting for RSA operation (op_reg 0x3ff02810 int_reg 0x1)

Hi,
A heavy duty program (every 10ms UART IO, continious messaging with available speed thru MQTT over WebSockets over TLS), using mongoose over the esp-idf net/ssl stack.
After several minutes of operation under load, the device aborts resulting in error messages as below, so far noticed only during re-establishing connection to broker. Any guidance towards where to look for resolution welcomed.
The surrounding code in esp_bignum.c says /* indicates a fundamental problem with driver */ ;-)
Resulting abort:
E (235656) bignum: Timed out waiting for RSA operation (op_reg 0x3ff02810 int_reg 0x1)
Stack trace:
0x40081884 in esp_log_write (level=<optimized out>, tag=<optimized out>,
    format=0x3f40ffec "\033[0;31mE (%d) %s: Timed out waiting for RSA operation (op_reg 0x%x int_reg 0x%x)\033[0m\n")
    at /home/nathan/xnode/esp-idf/components/log/./log.c:188
188	    (*s_log_print_func)(format, list);
(gdb) bt
#0  0x40081884 in esp_log_write (level=<optimized out>, tag=<optimized out>,
    format=0x3f40ffec "\033[0;31mE (%d) %s: Timed out waiting for RSA operation (op_reg 0x%x int_reg 0x%x)\033[0m\n")
    at /home/nathan/xnode/esp-idf/components/log/./log.c:188
#1  0x4012f7b8 in gettimeofday (ptimeval=0x1, ptimezone=0x3ffd0b08) at ../../../.././newlib/libc/syscalls/sysgettod.c:12
#2  0x40113095 in ecp_double_jac (grp=0x3ffde854, R=0x3ffed17c, P=0x3ffed17c)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecp.c:924
#3  0x40113f2b in ecp_precompute_comb (grp=0x3ffde854, T=0x3ffed158, P=<optimized out>, w=<optimized out>, d=64)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecp.c:1212
#4  0x40114142 in ecp_mul_comb (grp=0x3ffde854, R=0x3ffd0d80, m=0x3ffde8d0, P=0x3ffde900,
    f_rng=0x401396c8 <mbedtls_ctr_drbg_random>, p_rng=0x3ffb9b38 <ctr_drbg>)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecp.c:1377
#5  0x40114614 in mbedtls_ecp_mul (grp=0x3ffde854, R=0x3ffd0d80, m=0x3ffde8d0, P=0x3ffde900,
    f_rng=0x401396c8 <mbedtls_ctr_drbg_random>, p_rng=0x3ffb9b38 <ctr_drbg>)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecp.c:1631
#6  0x401425e6 in mbedtls_ecdh_compute_shared (grp=0x3ffde854, z=0x3ffde924, Q=0x3ffde900, d=0x3ffde8d0,
    f_rng=0x401396c8 <mbedtls_ctr_drbg_random>, p_rng=0x3ffb9b38 <ctr_drbg>)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecdh.c:69
#7  0x401427f4 in mbedtls_ecdh_calc_secret (ctx=0x3ffde854, olen=0x3ffdebf0, buf=0x3ffdec34 "", blen=1024,
    f_rng=0x401396c8 <mbedtls_ctr_drbg_random>, p_rng=<optimized out>)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecdh.c:251
#8  0x40144cb7 in ssl_write_client_key_exchange (ssl=0x3ffd7750)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ssl_cli.c:2779
#9  0x4014528b in mbedtls_ssl_handshake_client_step (ssl=0x3ffd7750)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ssl_cli.c:3330
#10 0x4013b8d0 in mbedtls_ssl_handshake_step (ssl=0x3ffd7750)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ssl_tls.c:6286
#11 0x4013b926 in mbedtls_ssl_handshake (ssl=0x3ffd7750)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ssl_tls.c:6310
#12 0x4012a112 in mg_ssl_if_handshake (nc=0x3ffd6908) at /home/nathan/xnode/SNESP32/components/mongoose/./mongoose.c:4535
#13 0x4012acc0 in mg_ssl_begin (nc=0x3ffd6908) at /home/nathan/xnode/SNESP32/components/mongoose/./mongoose.c:3466
#14 0x4012af41 in mg_write_to_socket (nc=0x3ffd6908) at /home/nathan/xnode/SNESP32/components/mongoose/./mongoose.c:3358
#15 0x4012b080 in mg_mgr_handle_conn (nc=0x3ffd6908, fd_flags=<optimized out>, now=1484451057.7952161)
    at /home/nathan/xnode/SNESP32/components/mongoose/./mongoose.c:3556
#16 0x4012b2c4 in mg_socket_if_poll (iface=<optimized out>, timeout_ms=50)
    at /home/nathan/xnode/SNESP32/components/mongoose/./mongoose.c:3742
#17 0x401299f1 in mg_mgr_poll (m=0x3ffb6db4 <mgr>, timeout_ms=50)
    at /home/nathan/xnode/SNESP32/components/mongoose/./mongoose.c:2287
---Type <return> to continue, or q <return> to quit---
#18 0x40109734 in NetworkTask (pvParameters=<optimized out>) at /home/nathan/xnode/SNESP32/main/./network.c:632

I start to believe the fundamental problem there is, actually lies elsewhere, noticing now the surprising call to gettimeofday in the details of the backtrace. Indeed my software quite frequently (every few hundred ms or so) will call gettimeofday in one of the threads. Will explore this, anyhow any guidance welcomed.

Another backtrace, this time without gettimeofday
Remote debugging using /dev/ttyUSB1
0x40081890 in esp_log_write (level=<optimized out>, tag=<optimized out>,
    format=0x3f40ffec "\033[0;31mE (%d) %s: Timed out waiting for RSA operation (op_reg 0x%x int_reg 0x%x)\033[0m\n")
    at /home/nathan/xnode/esp-idf/components/log/./log.c:188
188	    (*s_log_print_func)(format, list);
(gdb) bt
#0  0x40081890 in esp_log_write (level=<optimized out>, tag=<optimized out>,
    format=0x3f40ffec "\033[0;31mE (%d) %s: Timed out waiting for RSA operation (op_reg 0x%x int_reg 0x%x)\033[0m\n")
    at /home/nathan/xnode/esp-idf/components/log/./log.c:188
#1  0x40084e14 in vTaskExitCritical (mux=0x1) at /home/nathan/xnode/esp-idf/components/freertos/./tasks.c:4127
#2  0x401130a1 in ecp_double_jac (grp=0x3ffddc30, R=0x3ffdf8f0, P=0x3ffdf8f0)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecp.c:924
#3  0x40113f37 in ecp_precompute_comb (grp=0x3ffddc30, T=0x3ffdf860, P=<optimized out>, w=<optimized out>, d=52)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecp.c:1212
#4  0x4011414e in ecp_mul_comb (grp=0x3ffddc30, R=0x3ffddcb8, m=0x3ffddcac, P=0x3ffddc58,
    f_rng=0x401396d4 <mbedtls_ctr_drbg_random>, p_rng=0x3ffb9b50 <ctr_drbg>)
    at /home/nathan/xnode/esp-idf/components/mbedtls/library/ecp.c:1377
.....

Resolved this problem by locking the Task core affinity to single core. My best bet is that the BIGNUM "co-procesor" perhipal is suffering the same issue as the generic FPU co-procesor, which is it can't start and stop at a different core.
So there is indeed a fundamental problem in the driver. Need to add similar somewhere in the lines of execute_op in esp_bignum similar code to as in components/freertos/xtensa_vectors.S which blocks the task to single core, starting from line 935.
    /* TODO: Remove this as soon as coprocessor state moving works across cores - JD */
    /* FPU operations are incompatible with non-pinned tasks. If we have a FPU operation
       here, to keep the entire thing from crashing, it's better to pin the task to whatever
       core we're running on now. */
    movi    a2, pxCurrentTCB
    getcoreid a3
    addx4     a2,  a3, a2
    l32i    a2, a2, 0                       /* a2 = start of pxCurrentTCB[cpuid] */
    addi    a2, a2, TASKTCB_XCOREID_OFFSET  /* offset to xCoreID in tcb struct */
    s32i    a3, a2, 0                       /* store current cpuid */

The above resolution method does not remove the occurence of this problem. What seems to help is #256

Sorry I didn't reply to this earlier, I just commented on the PR linked above.

This did not re-occur after introduction of recent SPI flash fixes.


Support bulk rewrap requests

Essentially, the SDK should look for multiple KAOs with the same KAS urls, and send the requests in bulk, instead of individually.

Requires https://github.com/opentdf/platform/issues/777 to be completed first.


Changing PowerApps CLI installation directory

Hi,
I'm trying change the installation directory of Microsoft PowerApps CLI from %APPDATA%\Microsoft\PowerAppsCLI to C:\ProgramFiles or C:\ProgramsFiles(x86).
I'm using  /INSTALLDIR command-line to change the directory but the application is still installing in %APPDATA%.
Can you please tell me how can I change the installation directory via command-line?
Thanks,
Anique

@anique9999
Thanks for your feedback.
The documentation about installing PAC CLI is here: Install Microsoft Power Platform CLI.
I don't see that INSTALLDIR is mentioned anywhere in that topic, or any content that describes how the location of the installation can be modified.
So, this sounds like a feature request. If we have this capability but it isn't documented, I'll ask @kkanakas to update the doc.

Information was forwarded to @kkanakas. As mentioned before, this sounds like a feature request. Closing this.


feat: lookup view

feat: lookup view
This PR:

Adds a new view in the explorer called Lookup View
This view:

only appears when note lookup is active
displays current lookup controller's state (modifiers)
has 2-way binding between the lookup modifier buttons

Pull Request Checklist
You can go to dendron pull requests to see full details for items in this checklist.
General
Quality Assurance

[ ] Add a test for the new feature
[ ] Make sure all the existing tests pass
[ ] Do a spot check by running your feature with our test workspace
[ ] If your tests involve running [[manual Testing|dendron://dendron.dendron-site/dendron.dev.qa.test#manual-testing]], make sure to update [[Test Workspace|dendron://dendron.dendron-site/dendron.dev.ref.test-workspace]] with any necessary notes/additions to perform manual test ^9jtWc6ov340p
[ ] After you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: If you running MacOS or Linux, pay special attention to the Windows output and vice versa if you are developing on Windows

Special Cases

[ ] If your tests changes an existing snapshot, make sure that snapshots are updated
[ ] If you are adding a new language feature (graphically visible in VS Code/preview/publishing), make sure that it is included in test-workspace. We use this to manually inspect new changes and for auto regression testing

Docs

[x] Make sure that the PR title follows our commit style
[x] Please summarize the feature or impact in 1-2 lines in the PR description
[ ] If your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Special Cases
Git

[x] Make sure your branch names adhere to our commit style [[#^6zdCscSXs1MM]]

All PRs should start with [feat|fix|enhance|]/[{description-of-pr-in-kebab-case}]

eg. feat/add-thisthing

Analytics

[ ] If you are adding analytics related changes, make sure the Telemetry docs are updated

Github Issue

[ ] If this pull request is addressing an existing issue, make sure to link this PR to the issue that it is resolving.
[ ] If the resolution comes with a document update, link the docs PR to the issue as well.

I'll do some clean-up and style the view a bit and change the draft status of the PR. Meanwhile, this is ready for review on the functional side. Would love some feedback on the UX and how the code is structured as-is
@kevinslin merry christmas from the future :)

also, there's been a bunch of changes in master. can you rebase?

Lookup view now resides in dendron-plugin-views instead of dendron-next-server. I've also cleaned up the component code a bit.
Will get the docs ready soon with a quick loom recording to showcase it.

tooltips added:

Select options come in dendron green (the big green button color)

Form tooltips come in gray (default color)

note type modifier form removed
pending confirmation on enablement clause and IDendronExtension interface questions.

probably a good idea to push this in first and rebase this branch before merging.

probably a good idea to push this in first and rebase this branch before merging.

are you saying that https://github.com/dendronhq/dendron/pull/2004 should be pushed ahead of this branch?

probably a good idea to push this in first and rebase this branch before merging.

are you saying that #2004 should be pushed ahead of this branch?

yes. I figured since it has some changes to lookup it would be simpler to rebase

probably a good idea to push this in first and rebase this branch before merging.

are you saying that #2004 should be pushed ahead of this branch?

yes. I figured since it has some changes to lookup it would be simpler to rebase

#2004 is now pushed :)

UPDATE:

pre-emptive refactors that are not strictly related to the lookup view are all removed.
switched back to using getExtension for the time being until we decide where the treeview related properties in the concrete DendronExtension is going to end up (currently IDendronExtension does not have these properties).
waiting on the lookup button regression fix in #2037. Once this gets pushed to master I will rebase this once again and this and the 2-way binding of lookup view / controller will be correctly functional.

rebased!

excited to merge this. good work!


Generate ttl files contain wrong syntax " >"

The ttl files have in some situations a  space before the closing > of some triples that leads to errors on loading process

issue is fixed in most recent RDF generation


Layout/SpaceBeforeBrackets: a space is incorrectly added to strip_whitespace % w[name email]

Expected behavior
Does noting to before_validation { to_downcase %w[email] }
Actual behavior
Layout/SpaceBeforeBrackets: a space is incorrectly added to before_validation { to_downcase %w[email] } => before_validation { to_downcase % w[email] }
Steps to reproduce the problem
Add before_validation { to_downcase %w[email] }to model file
RuboCop version
$ [bundle exec] rubocop -V
1.7.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.3.0, running on ruby 2.7.2 x86_64-linux)
  - rubocop-performance 1.9.1
  - rubocop-rails 2.9.1
  - rubocop-rspec 2.1.0

Another example that should not generate a warning:
link_to [@site, person] do
  content_tag(:span, person.email)
end

Actually, this fails with any method that accepts an array as params, if not using parenthesis (which is quite common in DSLs)


Install ghostscript to enable optional PDF support.

Currently docker image has no /usr/bin/gs, so "ImageMagick PDF support available (optional)" is unavailable.
This patch just installs ghostscript to enable the option.

Great thanks

Checked 4.0-stable branch but it doesn't have the PDF support item.  So doesn't need this commit.

PDF feature has been merged on 4.1.
https://www.redmine.org/issues/22481


[BLOCKED] records: update metadata for files async

closes #205

Blocked by: https://github.com/inveniosoftware/invenio-files-rest/pull/226


No Image Download in FireFox 38

I am using the picture element and srcset in an unusual way, but should work.
The HTML
<picture>
	<!--[if IE 9]><video style="display: none;"><![endif]-->
	<source srcset="images/imgWelcome-hoopHouseRasperries.jpg, images/imgWelcome-hoopHouseRasperries_x2.jpg x2" media="(min-width: 825px) and (max-width: 63.5em)" />
	<!--[if IE 9]></video><![endif]-->
	<img alt="" />
</picture>

Basically, I have an image place holder, it's empty, so nothing downloads by default.  This particular image is simply visual filler. (replaces a logo that moves to the header for medium screens)
In Chrome 43, Opera 23, Safari 5.1 everything works as expected, there is no image downloaded unless in the specified screen size, and the image downloads/displays when re-sizing the browser window. (in Chrome Canary 45 all the source images download regardless)
The problem is with FireFox 38, when resizing the browser window, (from a size that the image does not show) the image will not download.  If the page is loaded within the size requirement, it will  download. (and thus show/hide as necessary)

Duplicate of #501
This is a bug with the native picture implementation that landed in FF38, and should be solved by 39:
https://bugzilla.mozilla.org/show_bug.cgi?id=1135812
We do have a stand-alone fix for this issue, though: https://github.com/scottjehl/picturefill/blob/afarkas-pf3-proposal/src/plugins/gecko-picture/pf.gecko-picture.js
Hopefully that does the trick; let us know if not.


[Select] Add SelectDisplayProps prop

Closes #10401

Is this error something I can resolve? It's not entirely clear what the error is exactly

@noah-potter run yarn docs:api or npm run docs:api

@oliviertassinari Should that scoping happen whenever SelectInput is split? I'm not sure how to make this change anticipate the two SelectInputs

I think that the simplest path to solve #10335 is to introduce a NativeSelect component, sharing some code with Select.


Preserve GPX metadata and trkpt extensions in converted geojson

In addition to the heart rate extension that is already converted, this commits stores metadata and all GPX trkpt extensions into the generated geojson, so that this data is not lost and can be processed by the application. The corresponding xmlns are also stored.

@opoto you might want to checkout https://github.com/tmcw/togeojson as a more maintained fork

@opoto you might want to checkout https://github.com/tmcw/togeojson as a more maintained fork

Thanks for the notice. I will consider migrating, but I made significant updates in current version  which I might want to preserve.


[BugFix] Prohibit the use of CTEs with the same name in the same level of name scope

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #13047
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[FE PR Coverage Check]
:heart_eyes: pass : 9 / 11 (81.82%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/QueryAnalyzer.java
8
10
80.00%
[123, 130]

:large_blue_circle:
com/starrocks/sql/analyzer/Scope.java
1
1
100.00%
[]

@Mergifyio rebase

run starrocks_admit_test

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 9 / 11 (81.82%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/QueryAnalyzer.java
8
10
80.00%
[123, 130]

:large_blue_circle:
com/starrocks/sql/analyzer/Scope.java
1
1
100.00%
[]

run starrocks_admit_test


Getting MockClient memory leak log

After running a bunch of unit-tests through jest, I get this log in the console.
(node:66873) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 start listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)
(node:66873) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 query listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit
(node:66873) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 query-error listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit
(node:66873) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 query-response listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit

I dont know if this is of any concern or it is just due to large number of parallel running async test functions.
Thanks.

I have  total 13 trackers.on in directory of unit-tests.

By trackers.on do you mean the tracker that the lib comes with or something else?

This is how I use it. Since we dont have step in this lib like knex-mock, I try to make it work this way.
test('Name ', async () => {

        const response = [
                response of 1st query, response of 2nd query
        ];
        const trx = await knexObj.transaction();

        const a = new A();
        let step = 0;

        tracker.on
            .any(({ method, sql, bindings }: RawQuery) => {
                if (step === 0) {
                    expect(method).toEqual('select');
                    expect(sql).toMatchInlineSnapshot(....);
                    expect(bindings).toEqual([...]);
                }
                step++;

                return true;
            })
            .response(() => {
                return response[step - 1];
            });
        await a.b()
});

By trackers.on do you mean the tracker that the lib comes with or something else?

Tracker of this lib.

Tracker.on is not using listeners at all

Your tests are not written as it meant to be used :]
You should no expect inside query matcher (the on part).
The lib provides an history object for each type of queries (select / insert ...)
test('Name ', async () => {
  const trx = await knexObj.transaction();

  const a = new A();
  tracker.on.select(({ sql }) => sql.includes('table_name')).responseOnce({ id: 1, name: 'bla' });
  await a.b(trx);
  await trx.commit();

  expect(tracker.history.select[0].sql).toContain('table_name');
});

I cannot just rely on sql.includes('table_name')) as all queries might be on same table.
I want to match entire sql which I am doing using matchInlineSnapshot to see if new commit has not unintentionally added/removed query select/returning fields or where clauses or anything else. If it was an intentional change, I can update the jest snapshot of the sql.
I want to test the order of query in code. No one should unintentionally alter the sequence of queries. This I agree that I can it with tracker.history.any but since I have send response also to queries sequentially, I have to use step for response. (For first query hitting the client, this response should be send. For second, this should be send & so on). Since step is being used in response, I have to maintain (increment) it inside query matcher. So added expect there only, so it can fail early if a particular query is not as expected. (Should-be second query is unintentionally changed to first query)

Tracker.on is not using listeners at all

Okay! The logs in the first comment regarding listeners in MockClient do not hint any serious concerns right ? In this case, we can close the issue.

I ran node --trace-warnings node_modules/jest/bin/jest.js to trace the log, I got this.
(node:71047) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 start listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:595:17)
    at MockClient.addListener (node:events:617:10)
    at makeTxClient (abc-user/node_modules/knex/lib/execution/transaction.js:335:13)
    at abc-user/node_modules/knex/lib/execution/transaction.js:196:43
    at Transaction_PG.acquireConnection (abc-user/node_modules/knex/lib/execution/transaction.js:254:20)
(node:71047) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 query listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:595:17)
    at MockClient.addListener (node:events:617:10)
    at makeTxClient (abc-user/node_modules/knex/lib/execution/transaction.js:340:13)
    at abc-user/node_modules/knex/lib/execution/transaction.js:196:43
    at Transaction_PG.acquireConnection (abc-user/node_modules/knex/lib/execution/transaction.js:254:20)
(node:71047) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 query-error listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:595:17)
    at MockClient.addListener (node:events:617:10)
    at makeTxClient (abc-user/node_modules/knex/lib/execution/transaction.js:345:13)
    at abc-user/node_modules/knex/lib/execution/transaction.js:196:43
    at Transaction_PG.acquireConnection (abc-user/node_modules/knex/lib/execution/transaction.js:254:20)
(node:71047) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 query-response listeners added to [MockClient]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:595:17)
    at MockClient.addListener (node:events:617:10)
    at makeTxClient (abc-user/node_modules/knex/lib/execution/transaction.js:350:13)
    at abc-user/node_modules/knex/lib/execution/transaction.js:196:43
    at Transaction_PG.acquireConnection (abc-user/node_modules/knex/lib/execution/transaction.js:254:20)

The warnings are created from knex lib, not this lib.


fix bug in the random agent example script

When I ran the original script I got
Logging to /tmp/openai-2018-12-14-17-36-08-425220
make: Entering directory '/home/act65/repos/coinrun/coinrun'
make: Nothing to be done for 'all'.
make: Leaving directory '/home/act65/repos/coinrun/coinrun'
Traceback (most recent call last):
  File "main.py", line 6, in <module>
    env = make('standard', num_envs=16)
  File "/home/act65/repos/coinrun/coinrun/coinrunenv.py", line 216, in make
    return CoinRunVecEnv(env_id, num_envs, **kwargs)
  File "/home/act65/repos/coinrun/coinrun/coinrunenv.py", line 156, in __init__
    obs_space = gym.spaces.Box(0, 255, shape=[self.RES_H, self.RES_W, num_channels], dtype=np.uint8)
AttributeError: module 'gym' has no attribute 'spaces'

Adding import gym.spaces fixes the bug (see issue here). Although maybe it should be imported elsewhere!?

The random agent example script doesn't seem to reference gym.spaces at all, I think you meant to put that fix into coinrunenv.py which is where the error is in your stack trace.

yea, that makes more sense (still not super familiar with git/contributing).
this should be ok?

Yeah looks good, thanks for the PR!


RuntimeError: Given groups=1, weight of size [128, 1, 3, 3], expected input[2, 2, 512, 512] to have 1 channels, but got 2 channels instead

My input data has the shape [2, 2, 512, 512] ([B, C, H, W]). Due to the nature of the data, I cannot change it to a single-channel input. Which parameters in the model should I adjust accordingly?

change the blue line in the bottom as “in_channels = 2"


Set debug logging level

This PR adds | fixes:

Print debug messages to console

How to test:

Try the API by uploading Variants and CaseData files both using main branch and this branch

Expected outcome:

[ ] This branch should display the content of the parsed files

Review:

[ ] Code approved by
[ ] Tests executed by

This version is a:

[ ] MAJOR - when you make incompatible API changes
[x] MINOR - when you add functionality in a backwards compatible manner
[ ] PATCH - when you make backwards compatible bug fixes or documentation/instructions

When submitting files using main branch:

Using this branch:


Feature Request: Get a report with all dependencies versions and licenses (for a legal purpose)

Hi,
I have to check with the legal department to check if there is not contaminating licenses in dependencies, e.g. a dependency with a GPL license which make automatically my project under the same GPL obligations.
So I'm looking for a tool/plugin to do it automatically, but I think it will be usefull if it can be done natively from the bower cli.

This is something 3rd party module could do. Like https://www.npmjs.com/package/bower-license


Fix a crash when hovering over "Favorites" in the FileSystem dock

Fixes https://github.com/godotengine/godot/issues/77425. Shouldn't conflict with https://github.com/godotengine/godot/pull/77208.
As mentioned in a comment, the logic based on strings seems flimsy to me, but we already rely on it, so... If someone wants to take a stab at improving its robustness, feel free to open a follow-up PR.

Thanks!


Run Conformance checks for v1.27/rke

Run Conformance checks for k8s v1.27 provisioned through standalone RKE

PR Raised - https://github.com/cncf/k8s-conformance/pull/2840


Fixes #2420 REPL analyzer processes do not exit

Fixes #2420 REPL analyzer processes do not exit
Disposes the evaluator when the associated window closes.

👍


Offer Create tx displays wrong buy/sell pair

In this screenshot, it says xrp was bought / phi was sold

in the detail view it says phi was bought / xrp sold

the 2nd screenshot is correct, the first one displays the currencies flipped the wrong way.
https://testnet.xrpl.org/accounts/rfc9vxWtSFeWtpD9N3qsn4YHqZ7Ph2pZvN

Thanks for pointing this out! (Sorry I forgot to reply when I read this earlier)


text-transform is prefixed with -webkit

The text-transform is prefixed by stylis.js because the regexp is checking on the transform keyword.

Thanks, patched in 3.1.8


Automate app versioning

use https://github.com/conventional-changelog/standard-version
https://medium.com/@jsilvax/automate-semantic-versioning-with-conventional-commits-d76a9f45f2fa

Done. but might expend to passing real version into docker image instead of having version hash

So far so good. Closing as it covers all the requirements.


az storage blob list polls continuously when > 5000 records are retrieved using nextMarker continuation tokens

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
$blobs = az storage blob list --account-name "<redacted>" --container-name <redacted> --only-show-errors --show-next-marker --num-results 1000 --include t | ConvertFrom-Json;

$nextMarker = $blobs.nextMarker;
$nextMarker;

while ($null -ne $nextMarker)
{
    $newBlobs = az storage blob list --account-name "<redacted>" --container-name <redacted> --show-next-marker --num-results 1000 --include t --marker $nextMarker --debug | ConvertFrom-Json;
    $nextMarker = $newblobs.nextMarker
    if ($null -eq $nextMarker)
    {
        Write-Host nextMarker is null!!;
        Write-Host $(Get-Date);
    }
    else
    {
        Write-Host nextMarker is not null!!;
        Write-Host $(Get-Date);
    }

    $nextMarker;
}

Describe the bug
For the storage account in question, which contains tens of millions of blobs (which might not be relevant), the script above executes 5 times, returning batches of 1000 records and a continuation token.
This is as expected.
Upon the 6th invocation, the CLI seems to poll forever, eventually giving up with a memory exception. I have attached the debug log below.
az cli debug trace.txt
To Reproduce
Run the script above.
Expected behavior
Batches of 1000 blobs should be returned until the list is exhausted.
Environment summary
Windows-10-10.0.19045-SP0
Python 3.10.8
Installer: MSI
azure-cli 2.45.0 *
Extensions:
azure-devops 0.25.0
Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

Previous issue which has likely been superseded by this one: https://github.com/Azure/azure-cli/issues/25595

@xgithubtriage for visibility.

route to service team


[GSSOC'23] Adding Back to top link at the end of README

Proposed Method: Add "Back to Top" Link at the End of README
Description
This issue proposes adding a "Back to Top" link at the end of the README file. The link will provide an easy way for readers to navigate back to the top of the README without scrolling manually. This small addition will enhance the user experience and improve accessibility.
To what class it belongs?: README File
Example Usage
## Table of Contents
- [Introduction](#introduction)
- [Installation](#installation)
- [Usage](#usage)
- ...

## Contributing
...
...

## License
...

---

*[⬆ Back to Top](#table-of-contents)*

Hi , I am a contributor in GSSOC'23, Please assign this issue to me. I have prior knowledge in markdown files and have done a lots of efficient and effective documentation work during my projects and internships.

Add it @mkswagger :))

Add it @mkswagger :))

Yes On it :)


Auto-response functionality

Thank you for the auto-response functionality, it is working very well, however I would like to kindly request passing the form field values to the auto-response as well.
I would like to send a copy of the form data in the auto-response message, I've tried using the template that you have set in the notification but it does not capture the data.
Thank you for your kind assistance.

The data should be already there.
Just put the name of your field (on the template) like:
{{ name }}
{{ email }}
{{ message }}

Perfect! Thanks you :)


Use itertools to chain packages for efficient looping

Problem Statement
In order to find the packages in an image layer that are not in previous layers, you need to maintain a 'master list' of packages and for each of the layers, compare with this master list and update it as you go along. This makes inefficient use of memory and forces you to maintain a global variable.
Describe the Proposal
Instead of maintaining a 'master list' you could use the itertools's chain method to loop through each of the previous layer's list of packages in order, thus maintaining a 'master list' without allocating memory or having a global variable. Since you are already mounting layers from 0 to x, you can loop through packages in layers from 0 to x-1 to find packages that are unique to x.
Steps to Implement Proposal
import itertools

for pkg in itertools.chain(x for x in image.layers[:curr]) <-- may not work, need to experiment

I think this will do the job.
def update_current_layer_pacakges(img_obj,layer_index):
    '''Given an Image and layer index it will update the packages found in that
    particular layer by removing the packages which are already found in previous
    layers.
   '''
    layer_obj = img_obj.layers[layer_index]
    package_list = itertools.chain(layer.packages for layer in img_obj.layers[:layer_index])
    unique = []
    for item in layer_obj.packages:
        if item in package_list :
            layer_obj.packages.remove(item)

Hi @nishakm i think this function can replace update_master_list.
def update_layer_package_list(img_obj,layer_index):
    '''Given an Image and layer index it will update the packages found in that
    particular layer by removing the packages which are already found in previous
    layers.
    '''
    layer_obj = img_obj.layers[layer_index]
    layers_list = itertools.chain(x for x in img_obj.layers[:layer_index])
    package_list = itertools.chain(pkg.packages for pkg in layers_list)
    for item in layer_obj.packages:
        if item in package_list :
            layer_obj.packages.remove(item)

We can call this wherever update_master_list was called and get rid of the master_list

@abhaykatheria Sorry for getting back to you on this so late. It turns out itertools.chain doesn't actually make a list of unique items. So this implementation will not work. I will close both the issue and the PR.


Request: Scrub Playhead with Scroll Wheel (instead of Zooming)

It would be useful to be able to scrub the playhead with scroll wheel, for those like me, who would prefer that.
Also worth pointing out, that I think the normal playback should stop, once scrubbing's been initiated. I've seen this scrubbing behavior interfere with playback rather than stop it, in other apps. Thus causing a stuttering effect, which is highly undesirable. So if you think this function is worth implementing, please let scrubbing take over and normal playback stop.

Hi, sorry it's not documented but if you press the "Ctrl" key while using the mouse wheel it will scrub frames instead of zooming. Having said that it would be good to add a setting so you could remap it to work without the "Ctrl" key.
I can also add a setting for whether playback is stopped when scrubbing, since some people like the playback to continue.

As mentioned here: Custom shortcuts to the rescue!


GROOVY-8690 bump picocli to v3.2.0 from 3.1.0

Details in https://issues.apache.org/jira/browse/GROOVY-8690

+1 for master and GROOVY_2_5_X


Screenshot utils

How is Hyprshot different from our own Grimblast?

How is Hyprshot different from our own Grimblast?

Is this question for me?
If so, I don't know. I didn't try Grimblast.


fixed sass-loader errors:

Closes #
After creating a jhipster angular6 application, including carbon-components fails because of scss errors from sass-loader / webpack
This PR fixes it
Changelog
Removed

linebreak in styles.scss
seemingly invalid scss code in _select.scss

Testing / Reviewing
Steps to reproduce: create new jhipster application, https://www.jhipster.tech/installation/
in main folder:

npm i --save-dev carbon-components-angular carbon-components

add to src/main/webapp/content/scss/global.scss:
@import "~carbon-components/scss/globals/scss/styles.scss";
start application (./gradlew in main folder)
sass-loader errors that are fixed
ERROR in ./src/main/webapp/content/scss/global.scss (./node_modules/css-loader!./node_modules/postcss-loader/lib!./node_modules/sass-loader/lib/loader.js??ref--11-3!./src/main/webapp/content/scss/global.scss)
Module build failed:
$deprecations--message: '#{$deprecations--message}
^
Expected '.fixed sass-loader error:
ERROR in ./src/main/webapp/content/scss/global.scss (./node_modules/css-loader!./node_modules/postcss-loader/lib!./node_modules/sass-loader/lib/loader.js??ref--11-3!./src/main/webapp/content/scss/global.scss)
Module build failed:
$deprecations--message: '#{$deprecations--message}
^
Expected '.
ERROR in ./src/main/webapp/content/scss/global.scss (./node_modules/css-loader!./node_modules/postcss-loader/lib!./node_modules/sass-loader/lib/loader.js??ref--11-3!./src/main/webapp/content/scss/global.scss)
Module build failed:
&[data-invalid] ~ .#{$prefix}--select__arrow {
^
Top-level selectors may not contain the parent selector "&".

Hi @Johann-Foerster! 👋 Thanks so much for taking the time to fix these problems that you encountered during your setup. It would be really helpful if, before making a Pull request, you make an issue detailing the bug that you're encountering. This allows us to verify that it is a bug before any work starts on it 😄
With respect to the issues you noted, I would imagine that something is different with the node-sass version itself. All of these files are checked for scss compilation in this test harness: https://github.com/IBM/carbon-components/blob/master/tests/styles-test.js
Moving forward, it might be helpful to go and create that issue with the steps to reproduce the problem so we can better diagnose what the underlying issue is!


in release 0.1, file riscv-test-suite/rv64i/Makefrag, variable rv64i_sc_tests contains SRAIW twice

in release 0.1, file riscv-test-suite/rv64i/Makefrag, variable rv64i_sc_tests contains "SRAIW" twice; this is still the case in commit 13e1b57b52f1c0708bc6442a9668c5c01b4eb79e

Makefile:49: target 'SRAIW.elf' given more than once in the same rule

I think I fixed this and another similar bug in this pull request:
https://github.com/riscv/riscv-compliance/pull/98
however, see the note in that pull request that this change is tested in release 0.1 but not in HEAD as running the same tests in HEAD that I do in release 0.1 is not working

Thanks very much for making the pull request; every little bit helps.
On Wed, Mar 11, 2020 at 4:07 PM Daniel S. Wilkerson <
<EMAIL_ADDRESS>wrote:

I think I fixed this and another similar bug in this pull request:
#98 https://github.com/riscv/riscv-compliance/pull/98
however, see the note in that pull request that this change is tested in
release 0.1 but not in HEAD as running the same tests in HEAD that I do in
release 0.1 is not working
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-compliance/issues/97#issuecomment-597923689,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJRIBC4ODM5HXHABM4TRHAKS3ANCNFSM4LCWXTRQ
.

merged and fixed. I have tested riscvOVPsim and spike and they seem to pass.


Show help for API commands

Add two new commands openxpkicli --list and openxpkicli --help COMMAND.

Implemented in b2a52ce5a and 3a99315fe


Issue with waveform.js in live server

When I run the program using the live server in vs code it runs properly but when I open the file from the file explorer the javascript file in script tag is not loading

no feedback, closing.


Certain torrents return "-1" for seeders and leechers.

More info to come.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25616087-certain-torrents-return-1-for-seeders-and-leechers?utm_campaign=plugin&utm_content=tracker%2F10163092&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F10163092&utm_medium=issues&utm_source=github).

This can be from last commit sql changes, or from database update.
Waiting for more info.

This has been happening for a while, but is a little difficult too debug. I'm going to test some torrents over the next few days.

Sometimes the tracker answer with 'Could not set last maintenance time.'
While cleaning idle peers the tracker access an unexistent column in 'tasks' table, (maybe it exists in one of other projects).
Isn't this related to the problem?

The database:
	$result = mysqli_query($connection,
		'CREATE TABLE IF NOT EXISTS `'.$settings['db_prefix'].'tasks` (' .
			'`name` varchar(5) NOT NULL,' .
			'`value` int(10) unsigned NOT NULL' .
		') ENGINE=MyISAM DEFAULT CHARSET=latin1'
	);

The task:
$task = mysqli_query('REPLACE INTO `'.$settings['db_prefix'].'tasks` (`prune`) VALUES (\''.$time.'\')');

I think that task is supposed to be:
$task = mysqli_query('REPLACE INTO `'.$settings['db_prefix'].'tasks` (`name`, `value`) VALUES (\'prune\', \''.$time.'\')');

That could well be the problem. Once I am home I will check if that has ever run successfully.


Add support for teams

Added support for creating / deleting / updating teams
Added support for getting team members

When I started testing I called Get-Team. After that worked. I ran Get-Team | Get-TeamMembers and I got a lot of errors.  I would expect this to work.

Oh by the way. I like your update! :)

I kind of forgot to properly implement pipeline support... Should work now, so you can do things like
Get-Project | Get-Team | Get-TeamMember


I don't understand what the --crit and --warn flags are supposed to do

I don't know how to interpret "Critical/Warning level if pattern has a group."
It seems like --crit N would only trigger a critical event if the check found N or more matching lines.
It seems like --warn-only --warn N would trigger a warning event if the check found N or more matching lines.
These don't work for me as expected, and don't seem relevant to the code at all.  Was a breaking change in Ruby that's affecting this?  I'm using Ruby 2.3.1, and it seems like it's impossible to get in to the if m[1] clause at https://github.com/sensu-plugins/sensu-plugins-logs/blob/master/bin/check-log.rb#L192.
If I'm misunderstanding the intended use of --crit and --warn, what are they intended to do?
If my interpretation is correct, I'm happy to submit a PR to update them (it's pretty much ready to go).  But I worry that it could be a breaking change for existing implementations.

I have not used this plugin but what you describe sounds rationale given the common pattern and name of options. I would be happy to look at a PR to evaluate if it breaks intended use cases.

following pull request addresses this:
https://github.com/sensu-plugins/sensu-plugins-logs/pull/24#issuecomment-302952064

released: https://rubygems.org/gems/sensu-plugins-logs/versions/1.1.1

@majormoses
#24 ended up being backwards incompatible in two ways:

The original use case of crit and warn applying only to a numeric value in a capture group was removed.
there was an implicit critical severity that would be returned if any matches occured at least once.

That said, IMHO the current behavior is actually more reasonable than the previous. I have create #33 to correct the documentation

@cgeers ouch and thanks for the assist. What I think I will do:

create a patch 1.3.2 which basically reverts the only functionality, this prevents people from pulling in broken code without a version bump.
create a new release for 2.0.0 including the content of 1.3.1 as well as your fix.

Its a bit more work but I really firmly believe in using semver properly and all breaking changes should go in on a major release.

Sorry I lost track of this.  Thank you for following up on it.

@majormoses Sounds good. I'll push the requested changes ASAP.


Multiple "Active" class submenu using something_underline

When we have used a submenu like these hrefs:

Parts

Combs

Vehicle

There are added the "active" class to vehicle 'li a' and to Parts 'li a' or Combs (depends on click).
How can I solve this?
Thanks.

I have the same error.
How can I solve this?
Thanks.

Hei All.
I solve it, using ugly way,changing the link value.
Att.
..
Omar Fernando Pessôa
http://opessoa.info
Desenvolvedor de Sistemas / Systems Software Developer
C++ / PHP / C# / Javascript
2017-06-19 5:40 GMT-03:00 DanielGołąbek<EMAIL_ADDRESS>
I have the same error.
How can I solve this?
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/onokumus/metismenu/issues/79#issuecomment-309375937,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEVF38G1zNBmU3U-sB0o-v5KQvhwbvKpks5sFjQIgaJpZM4HG_8o
.


Add feedback link

It's possible that some of the rules will have bugs/false positives/false negatives.
If the plugin reports any issues we should include a link to an issue template to submit feedback.
Using Danger's markdown function to attach a feedback link below the table seems like the way to go here.

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms


Highlighting the mention text whilst typing

Could anyone help with the possibility of highlighting the mention text as it is being typed (before selection) and then the highlight disappears as soon as a selection is made from the suggestions?
Basically, this is what I am looking to achieve for example:

User types “#” on their keyboard
Text directly after “#” is highlighted in a soft gray
Two spaces cancels highlighting and tag is not made
If one of the suggestions is selected, it is then colored blue

Hello, thanks for your feedback. Is your issue still actual?
As a suggestion – you can add another part type with pattern for highlighting plain hashtags (/#(.)+/gi) and it should highlight your hashtag before you will select some from list.

Duplicates #58.
Please, continue the discussion in that issue.


Run SonarCloud only on pull requests to master

Run SonarCloud less often to reduce build times for normal CI builds.

Approved


issue while running the sample file for PBBO

The small example of PBBO I ran is this: emukit/examples/preferential_batch_bayesian_optimization/minimum_working_example.py
When I directly run this py file, sometimes I encounter the issue of covariance is not positive-semidefinite, but sometimes this problem does not occur and the program can be executed completely. What is causing this? How do you think I could avoid this problem?
The error message for the problem is as follows:
xxx/gitwork/PBBO/emukit-main/emukit/examples/preferential_batch_bayesian_optimization/pbbo/inferences/ep_batch_comparison.py:556: RuntimeWarning:covariance is not positive-semidefinite.

Hi! What you are observing is numerical instability of a GP training process, and that can happen to a variety of reasons. It's hard to say why exactly it affects the PBBO example without deep investigation. But I see a few priors set there for noise and lengthscales. It they are too far off the actual objective, it could lead to these warning. So perhaps a simple first try would be to update them and see if that changes anything?
Also, this end-to-end example is only there to demonstrate how various bits of PBBO fit together, so it's not quite surprising it might occasionally fail in that way.

Hopefully this answers the question, closing


[CI] Establish Meshery Edge Release Channel (ci.yml)

Current Behavior
See #284 as the Epic for this issue, which specifically addresses adding an Edge release channel.
Edge Channel
Nearly the same label structure that is used for the Stable release channel can use for the Edge release channel. With slight modification the edge channel docker images tags will look like this:

Image 1 (newer)

layer5/meshery:edge-latest
layer5/meshery:edge-v0.4.1 (strip off other suffix "-edge" tag is present in release tag)
layer5/meshery:edge-289d02

Image 2 (older)

layer5/meshery:edge-324vdgb

The part that we’ll need to give some thought to is the logic (flag) by which we determine whether a PR is to tag docker images with “stable” or with “edge” tags. Simple logic could be:
Every docker image built gets Edge tags whether a release tag is present or not. We would leave the Stable channel logic the same as it is now in that docker images get the “stable” tags only in the presence of a Release Tag (e.g. v0.4.1).
In this way, the Stable release channel remains controlled through GitHub Release Tags, while the Edge release channel has fresh images pushed to and available in Docker Hub with every single merge to the master branch.
In this approach, we wouldn’t have an edge- docker image tag, but rather we would only have edge-latest and edge-<commit-sha> . This is probably less than ideal. We will ideally have an edge release number, too.
In order to facilitate this, we will establish a naming convention in our GitHub Release tags. e.g., release tags for Stable will follow the same format that they do today, while release tags for Edge will always have a “-edge” suffix, like so:  v0.4.1-edge
Implementation
Assuming this design is befitting, we will then need to:

[x] Update ci.yml to account for the presence of release tags with a -edge suffix.
[x] Document how users can manually switch between channels by updating the docker image tags in their meshery.yaml / manifest files. (can be addressed in another issue)
[x] Create a convenient sub-command - mesheryctl system config --channel [stable|edge] (can be addressed in another issue)
[x] Create a convenient new setting in the Preferences area of the Meshery UI, so that people can alternatively use the UI to switch between channels if they like. (can be addressed in another issue.

Resources
See the Meshery Build & Release Strategy document.

Adding the willfix label. @kushthedude a good thing to see finished up. Having this channel means that we will have access to artifacts from every merge to master

@leecalcote @kushthedude is the issue still open ? Can i work in this issue?

Sure go ahead!

Heads-up .... this is a fairly involved issue.

Let's make this issue an epic since it has four major milestones.

@kushthedude FYI from the description: "See #284 as the Epic for this issue, which specifically addresses adding an Edge release channel."

I just did it because most of the details are populated in the build & release doc and not every contributor until he is a member can see the doc I guess.

@kushthedude totally. I should have kept my mouth shut. Epic-to-epic-to-epic-to-epic-...-to-epic-to-child is just fine. Keep breaking down until the child is a bite-sized morsel.
Wait... that came out wrong.

@crustyy still banging away here?

@kushthedude will you confirm that this complete?

@kushthedude will you confirm that this complete?

We need the mesheryctl command to switch

3 checkboxes down! Very nice.

Completed.


Panzer Paladin - not starting on Game Console R36S

OS: ArkOS latest version v2.0 (02/23/2024)
device: Game Console R36S, RG351P, RG351V, RG351MP
issue:
At the frist boot a part of the script is running, then mono_gdb_render_backtraces not supported on this platform error

after that, game is not starting and exit to the ports menu.
no log file created

ArkOS regression, this keeps happening time and time again for some reason.
I have already reported this, it was already fixed, and then it showed up again, it was fixed, and now I am getting a new wave of reports that this is happening. I'm sorry this is outside of my wheelhouse.

Thanks, for your fast reply.
I can confirm that Panzer Paladin was running at the beginning of 2023, ArkOS update in summer 2023 stopped that port, later on it was running again, after the december 2023 update it stopped again.
In this case it is only related to the ArkOS environnement and path changes which stopped the .sh script at one point.


Implemented support for gateway service name in meshNetworks

This supports specifying a network's gateway service name (registryServiceName) instead of explicit address.
If the service is a LoadBalancer, Pilot will extract the external LB address(es) from the matching k8s service resource on that network.
Benefits:

Users don't need to modify the primary's cluster ConfigMap with the remote gateway IP after setting up the remote cluster
Dynamically update changes to the LB addresses instead of manually modifying the ConfigMap

Closes #11837

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ymesika
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [ymesika]
pilot/OWNERS [ymesika]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ymesika: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test istio-integ-k8s-tests

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

@ymesika: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test istio-integ-k8s-tests

prow/e2e-simpleTests-minProfile.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test e2e-simpleTestsMinProfile

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

@ymesika: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test istio-integ-k8s-tests

prow/e2e-simpleTests-minProfile.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test e2e-mixer-no_auth

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

@ymesika: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test istio-integ-k8s-tests

prow/e2e-simpleTests-minProfile.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
552e629b9f5bc4f967bbfc42be836daa06f8330d
link
/test istio-pilot-multicluster-e2e

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

/test e2e-mixer-no_auth

/test e2e-simpleTestsMinProfile

@ymesika: PR needs rebase.

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

Replaced by #12728 (master)


redirectToSignIn ignores arguments

redirectToSignIn claims to take several arguments, but promptly ignores them, resulting in broken behavior in backwards compatibility.
export function redirectToSignIn(redirectURI: string = `${window.location.origin}/`,
                                 manifestURI: string = `${window.location.origin}/manifest.json`,
                                 scopes: Array<string> = DEFAULT_SCOPE) {
  console.warn('DEPRECATION WARNING: The static redirectToSignIn() function will be deprecated in the '
    + 'next major release of blockstack.js. Create an instance of UserSession and call the '
    + 'instance method redirectToSignIn().')
  const userSession = new this.UserSession()
  userSession.redirectToSignIn()
}

https://github.com/blockstack/blockstack.js/blob/master/src/auth/authApp.js#L64-L72
To honor these arguments, I believe we'll need to create an AppConfig with the passed arguments and pass it to the UserSession constructor.

Thanks for the report @freethejazz, please keep filing issues for any backwards compatibility bugs you run into. We don't yet have tests for all public function signature permutations.

Closing since this has been merged and should be available next release.


TCC.xsl:48-50: directly-related-pairs currently don't...

The puzzle 9-57e7e31a from #9 has to be resolved:
https://github.com/yegor256/jpeek/blob/b625770426cbfa670c987c135c0b3c98b7a24c8d/src/main/resources/org/jpeek/metrics/TCC.xsl#L48-L50
The puzzle was created by Kapralov Sergey on 26-Jan-18.
Estimate: 30 minutes, role: IMP.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The job #120 assigned to @llorllale/z, here is why. The budget is 30 minutes, see §4. Please, read §8 and §9. If the task is not clear, read this and this.

My own notes:

They keep using the term "instance variable" but as far as I can see they do not actually differentiate it from class variables
They deliberately exclude ctors from the analysis because "It will generally access all instance variables in the class, and thus, share instance variables with virtually all other methods. Constructors create connections between methods even if the methods do not have any other relationships."
They deliberately exclude "non-visible" methods from outside the class because "Class cohesion refers the relatedness of visible components of the class which represent its functionality. Class cohesion is the degree that those components are related. In our model of class cohesion, invisible components of a class are included only indirectly through the visible components."
They leave open the possibility for including inherited methods and instance variables. We are currently including the FQN of methods used (will do the same for fields in the near future), but we also need to know if the owning class is a parent of the one under analysis. I opened #187 for this.
Formula is TCC(C) = NDC(C)/NP(C), where NP is the max number of possible connections (N*(N-1)/2), and NDC is the number of directly connected methods
The direct connectivity between methods is determined from the class abstraction. If there exists one or more common instance variables between two method abstractions then the two corresponding methods are directly connected.
They note the following when they applied their cohesion measure: "Overloaded methods within the same class are treated as one method."
Class cohesion measures for visible methods defined only within the class are also useful, because the measures are not affected by the cohesion of a superclass. This should be configurable?
Quote: "Our results show that the classes that are heavily reused via inheritance exhibit lower cohesion".

Based on my notes above I've decided the following:

TCC does not care about relations between methods via method calls, so the request to do so in the puzzle will be discarded
Considering the consistent use of the term "instance variable" and their consideration of inheritance, etc., I am going to only consider class attributes that are not static.
I am going to exclude ctors
I am going to exclude private methods
I am going to leave a puzzle to come back and fix TCC after #187 is closed. Inclusion of inherited attrs and methods should be configurable.

The puzzle 9-57e7e31a has disappeared from the source code, that's why I closed this issue.

@ypshenychka/z please review this job, as in §30

@yegor256 missed points+payment on this one

@0crat status

@0crat status (here)

@llorllale This is what I know about this job, as in §32:

The job #120 is in scope for 21days
The role is DEV
The job is assigned to @llorllale/z for 4days
There is a monetary reward attached
@llorllale/z will get 50% of their hourly rate for this job
The job doesn't have any impediments
The budget is 30 minutes/points
These users are banned and won't be assigned:

@0pdd/z: This user reported the ticket

Job footprint (restricted area)

@llorllale/z this job was assigned to you 5 days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat quality good

@ypshenychka seems like you must also review the associated PR? (#182). See this points in the policy:

Pull request title explains the problem it is fixing.
Pull request description explains the solution proposed and contains a link to the original ticket it is related to.
The code reviewer found at least three problems in the code.
Comments were mostly about design problems, not cosmetic issues.
All problems found by the code reviewer were addressed by the pull request author.

QA review completed: +15 points just awarded to @ypshenychka/z, total is +180

@ypshenychka nevermind

Order was finished, quality was "good": +35 points just awarded to @llorllale/z, total is +655

@0pdd these puzzles are still not solved: 193, 194, 195

@0pdd these puzzles are still not solved: 193, 194, 195

@0pdd these puzzles are still not solved: 193, 194, 195

@0pdd 2 puzzles #193, #194 are still not solved; solved: #195.

@0pdd 2 puzzles #495, #496 are still not solved.

@0crat status (here)

@llorllale This is what I know about this job in C7JGJ00DP, as in §32:

The job #120 is not in scope
Job footprint (restricted area)

@0crat quality good (here)

@ypshenychka You @ypshenychka/z need to have one of these roles in order to do "Complete QA review": ARC, QA; I'm sorry to say this, but at the moment you've got no roles in this project


Wrong connectivity in Brunel10K sample

In your Brunel10K sample code you're creating the connections P->E and P->I twice by calling connect_with_sparsity() one-too-many times in each case. This results in an observed synapse count of
#syn ~= 0.075 * #neurons^2
, when the expected count is
#syn ~= 0.05 * #neurons^2
Please confirm, thank you!

Hi Dennis,
Good catch, you are quite right the P->N connections are being created twice. This has slipped through from the iterating of the Brunel10K network for the SNNSimulatorComparison repository. I'll get it corrected.

36f79cfa7e3cf515578f888358c8fc3efb6ccd04


Error does not show in select field  required

Please give more detail and provide a reproducible example here:
http://jsbin.com/conekusupo/1/edit?html,js,output

For anyone else that might stumble on this, make sure your label and select, input fields are enclosed in <div class="form-group"> or else you won't see the validation error.


Fix service imports

Problem
Services do not contain the correct import statements. They import any structs that are within the same thrift file as the service, but are missing imports for external structs.
Solution
Scan the service node in the AST for any struct-like types and import them.
Result
Services will contain imports for the struct-like types they require.

#4


Allow ambassador to work with clusters running their own certificate authority

It looks like Ambassador runs into issues with kubernetes clusters that run their own certificate authority.
When installing ambassador with helm, I see the kube-api logs throwing errors given here
http: TLS handshake error from <IP_ADDRESS>:42916: remote error: tls: unknown certificate authority
And the ambassador pod logs errors given here
2018-09-06 19:50:09,024 WARNING Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /api/v1/namespaces/default/configmaps
2018-09-06 19:50:09 kubewatch 0.39.0 WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /api/v1/namespaces/default/configmaps
2018-09-06 19:50:09,030 WARNING Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /api/v1/namespaces/default/configmaps
2018-09-06 19:50:09 kubewatch 0.39.0 WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /api/v1/namespaces/default/configmaps
2018-09-06 19:50:09,035 WARNING Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /api/v1/namespaces/default/configmaps
2018-09-06 19:50:09 kubewatch 0.39.0 WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),)': /api/v1/namespaces/default/configmaps
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 601, in urlopen
    chunked=chunked)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 346, in _make_request
    self._validate_conn(conn)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 850, in _validate_conn
    conn.connect()
  File "/usr/lib/python3.6/site-packages/urllib3/connection.py", line 326, in connect
    ssl_context=context)
  File "/usr/lib/python3.6/site-packages/urllib3/util/ssl_.py", line 329, in ssl_wrap_socket
    return context.wrap_socket(sock, server_hostname=server_hostname)
  File "/usr/lib/python3.6/ssl.py", line 407, in wrap_socket
    _context=self, _session=session)
  File "/usr/lib/python3.6/ssl.py", line 814, in __init__
    self.do_handshake()
  File "/usr/lib/python3.6/ssl.py", line 1068, in do_handshake
    self._sslobj.do_handshake()
  File "/usr/lib/python3.6/ssl.py", line 689, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/ambassador/kubewatch.py", line 458, in <module>
    main()
  File "/usr/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/ambassador/kubewatch.py", line 438, in main
    sync(restarter)
  File "/ambassador/kubewatch.py", line 261, in sync
    for x in v1.list_namespaced_config_map(restarter.namespace).items ]
  File "/usr/lib/python3.6/site-packages/kubernetes/client/apis/core_v1_api.py", line 11650, in list_namespaced_config_map
    (data) = self.list_namespaced_config_map_with_http_info(namespace, **kwargs)
  File "/usr/lib/python3.6/site-packages/kubernetes/client/apis/core_v1_api.py", line 11753, in list_namespaced_config_map_with_http_info
    collection_formats=collection_formats)
  File "/usr/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 321, in call_api
    _return_http_data_only, collection_formats, _preload_content, _request_timeout)
  File "/usr/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 155, in __call_api
    _request_timeout=_request_timeout)
  File "/usr/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 342, in request
    headers=headers)
  File "/usr/lib/python3.6/site-packages/kubernetes/client/rest.py", line 231, in GET
    query_params=query_params)
  File "/usr/lib/python3.6/site-packages/kubernetes/client/rest.py", line 205, in request
    headers=headers)
  File "/usr/lib/python3.6/site-packages/urllib3/request.py", line 66, in request
    **urlopen_kw)
  File "/usr/lib/python3.6/site-packages/urllib3/request.py", line 87, in request_encode_url
    return self.urlopen(method, url, **extra_kw)
  File "/usr/lib/python3.6/site-packages/urllib3/poolmanager.py", line 321, in urlopen
    response = conn.urlopen(method, u.request_uri, **kw)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 668, in urlopen
    **response_kw)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 668, in urlopen
    **response_kw)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 668, in urlopen
    **response_kw)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 639, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/usr/lib/python3.6/site-packages/urllib3/util/retry.py", line 388, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /api/v1/namespaces/default/configmaps (Caused by SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)'),))
AMBASSADOR: kubewatch sync exited with status 1
Here's the envoy.json we were trying to run with:
ls: /ambassador/envoy*.json: No such file or directory
No config generated.
AMBASSADOR: shutting down (1)

Since many clusters run their own CA or integrate with tools that provide their own CA such as Vault, it would be nice if Ambassador had a way to work with these clusters.

I encounter the same issue
how to set Insecure flag to kubernetes client ?

Same problem as #807, and resolved in #808.


Fix tests for visualization jupyter

If I turn on tests for the visualization jupyter they fail in CI, but work locally. They can be turned on in the Makefile: https://github.com/OML-Team/open-metric-learning/blob/main/Makefile#L49.
I assume that the problem may be related to a pytorch version.


HDDS-2305. Update Ozone to latest ratis snapshot(0.5.0-3f446aa-SNAPSHOT). Contributed by  Mukul Kumar Singh.

NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute

@swagle is updating the ratis version with https://github.com/apache/hadoop-ozone/pull/23.

Thanks @mukul1987.  OM HA acceptance test works fine with https://github.com/apache/hadoop-ozone/pull/23/commits/4b03f42a28323a712f9ef6e19d203f8638651830


Release 1.0

I've added this meta issue where we can track items that we want to complete before a 1.0 release.
TODO

[x] Remove ruby 1.8 compatibility (https://github.com/ipaddress-gem/ipaddress/pull/140)
[x] Remove mongoid stuff (https://github.com/ipaddress-gem/ipaddress/pull/139)
[x] Update Readme to Markdown (https://github.com/ipaddress-gem/ipaddress/pull/138)
[ ] Update Changelog to Markdown
[ ] Switch to Github Actions for automated tests (remove Travis)
[ ] Remove .rock.yml
[ ] Remove testing for all unsupported versions of Ruby (< 3.0.0)

Planned deprecations

[ ] Deprecated a?, b? and c? in favor of class_a_network?, class_b_network? and class_c_network?
[ ] Deprecate IPAddress(str) in favor of IPAddress.parse(str) (invoking a class like a method isn't very common in ruby, better with an explicit parse method).

@bluemonk Feel free to add/remove anything from this list, as you see fit.

@bluemonk I've added three PRs above, for a few items.
Let me know if you have comments.
Also, I'd like to setup Github Actions for automatic tests (instead of Travis), assuming you aren't opposed?
If so, I'll need more repo access, so I can setup branch protection checks requiring passed tests before merge.

@gucki I'm still planning to get around to it!
It's just been very busy at work the last few months.

Hi @sandstrom! What is the status of this project almost a year later?

@javierav A few ideas:

Open a PR for moving to Github Actions
Go through the codebase and add deprecations where reasonable (see above).
Remove testing for all unsupported versions of Ruby.

Happy to discuss thoughts/details in advance, if you have questions.
For all of the above, unless you feel very confident about doing it, a good start is probably to write a small plan, a bullet list of 5-10 items, so we can discuss details before you get started.

In either case, I haven't forgotten about this, and still have plans to get this over the finish line. It's just been very busy the past 3 months with moving and other things. Should get better in 1-2 months hopefully.

Bumping, in particular interested in getting https://github.com/ipaddress-gem/ipaddress/pull/86

Any chance we can get an interim 0.9 release with the code base as it stands now?


Add Map into ScriptBuilder

When using the buildScriptBuilder on NeonInvoker, you will get an error if you use a Map as parameter of an invoke.
This is because when running emitContractCall, you'll eventually trigger this error Unaccounted ContractParamType! : 34

This is a bug introduced on https://github.com/CityOfZion/neon-js/pull/898

I assume you will PR to fix?

@ixje We have an issue on Clickup to fix this. Probably Raul or Lucas are going to work on it. But on the next Sprint, because they are fully allocated on this one.


Easy way to create a new project using the last commited version

A great question from @neoxeo:
Can you tell me, please, what is the easy way to create a new project using the last commited version of each "components" from a template ?

We decided to respond here. There will be at least two solutions for that. One is quick which require copy and paste, and one is correct using console command. The task is the following: to have the easy way to create custom project using latest components version from master like we have with nuget.  There we have a command  for that:
dotnet new alternet-ui
We are working on the solution and will post here any updates on this issue.

I have uploaded first version of the template to
https://github.com/alternetsoft/AlternetUI/tree/master/Install.Scripts/MinMasterTemplate
This is minimal project template to use with Alternet.UI latest master version from github.
Limitations:

You need to edit MinMaster.csproj file and change "AlternetUISourcePath" property to the path where Alternet.UI Source folder is located.
Uixml forms are not supported by this template. You need to create forms and controls from code. See MainWindow.cs for an example.

I will update this issue when we have an improved template.

Thanks a lot !
I will test this very quickly and give you result.

@generalloki
I have tested using MinMasterTemplate with this project (code is not clean but this is not the goal for this moment) :
ASCII_Art3.zip
You need to edit MinMaster.csproj file and change "AlternetUISourcePath" property to the path where Alternet.UI Source folder is located.
You need to open project with VS2022, save it and run "run.bat" and open again. Else it doesn't compile into VS2022.
But this is a good starting point !!!

@neoxeo , Thanks for the feedback on this issue. I have updated MinMaster template and instructions for it.
So, now they are:
How to use:

STEP 1. Edit MinMaster.csproj file in Notepad and change "AlternetUIPath" property
to the path where Alternet.UI folder is located.
STEP 2. Run build.bat.
STEP 4. Open MinMaster.csproj is Visual Studio and enjoy.

And I have seen your Mario ascii art :)

@generalloki
I work on a script to create a new project (UI or Console) from scratch using latest version of AlternetUI development.
I hope to finish it before this end of this week.
I will publish it hete as soon it is available.

@generalloki
Here is the first release of script to create a new  AlternetUI project from scratch using latest version of AlternetUI development :
MinMasterTemplateNew.zip
This zip need to be uncompress into Install.Scripts folder.
Launch it, enter informations, wait and open your new project !

I doubt I will be using this project, I prefer "Copy and paste" approach. For now I added a link to this issue in the  readme.md
of MinMaster folder. Like this:

You can comment or add bug report for MinMaster template here:
https://github.com/alternetsoft/AlternetUI/issues/68
There is an improved template project created by @neoxeo.
It has script file that asks for project name, path. etc.
https://github.com/alternetsoft/AlternetUI/issues/68#issuecomment-1864016068

@generalloki
OK, thanks.
The most important part of the script is how you can create the 2 Nuget packets for AlternetUI/AlternetUI.PAL, on the fly, with the latest comit posted on github and use these packages locally with the NuGet.config file.

@neoxeo , Now when you pointed on this I understand that this is useful - to create nuget on the fly. I will think on how this could be integrated. Maybe even in the main solution, we could have such project or script, so it will be possible to click it to run and to build nuget with the latest source.

For now I am closing this issue, as there are many other more important things to do. The official way is to use nuget. Also there is MinMaster project and solution from neoxeo which is mentioned above.

@neoxeo,
I have used your idea with local nugets in \Source\Tests\UseLocalPackages\  project. If copy UI nugets into LocalPackages subfolder, project will use them. This was useful for debug nugets. Thanks!

@generalloki
Yes I have seen, good news ! Thanks
I have created a powershell script to update an existing project using LocalPackages folder to use the last version of of AlternetUI available (I rebuild nuget packages on the fly and replace current packages present in project with the lastest). I continue tests and post it when I will finish it.

@neoxeo
I have uploaded new version of Install.Scripts\MinMasterTemplate.
Now it supports uixml forms.


Electron Remote creates endless devTool windows with a CSP warning in development mode

I need to use 'unsafe-eval' in my main renderer window in development, but doing so, electron-remote creates devTool windows to warn:

Is there any way to prevent this?

Managed to disable the CSP warnings, but the devTool windows persist.

Thanks for your reply @tayiorbeii. The issue is that it creates new devtools windows for every message when it's coming from a callstack involving electron-remote. As a result disabling warnings, and disabling the CSP warnings, fails to have an effect.

This remains an issue. Is there any fix for this?
Video: http://recordit.co/1sFsWdwoOo


Migrate the network controller functionality to a dedicated component

We need replication of Brokers while sharing Network Controllers for node state. The NC may still run in-proc but the interface should be generic (gRPC?)

References #90

The networkserver component has been implemented in the refactor branch. See #199 for more info.


fix plugins restart on docker restart

- What I did
Installed a plugin, restarted docker a check if the plugin was running
- How I did it
docker plugin install vieux/sshfs
sudo service docker restart
sudo docker-runc list

p.PluginObj.Active represent the desired state, not the actual state. so when docker is starting and the desired state is Active we need to force enable it.
- Description for the changelog
fix plugins restart on docker restart
- A picture of a cute animal (not mandatory but encouraged)

ping @tiborvass @anusha-ragunathan

flaky test? I think I've seen it fail on other PR's
21:59:24 ----------------------------------------------------------------------
21:59:24 FAIL: docker_cli_plugins_test.go:60: DockerSuite.TestPluginForceRemove
21:59:24
21:59:24 docker_cli_plugins_test.go:69:
21:59:24     c.Assert(err, checker.IsNil)
21:59:24 ... value *errors.errorString = &errors.errorString{s:"exit status 1: Error response from daemon: remove /var/lib/docker/plugins/c13470a2352bc4343d1714f1414884299200a09edef11836f1c7c75232ef523a/rootfs/data: device or resource busy\n"} ("exit status 1: Error response from daemon: remove /var/lib/docker/plugins/c13470a2352bc4343d1714f1414884299200a09edef11836f1c7c75232ef523a/rootfs/data: device or resource busy\n")
21:59:24

LGTM

LGTM :cat:

LGTM


MSW Failed to Init

Hello, Ive migrated from cypress 9.7.0 to 10.8.0 and MSW no longer seems to work. I am seeing the following error.
[MSW] Failed to register a Service Worker for scope ('http://localhost:35607/') with script ('http://localhost:35607/mockServiceWorker.js'): Service Worker script does not exist at the given path.

Did you forget to run "npx msw init <PUBLIC_DIR>"?

Learn more about creating the Service Worker script: https://mswjs.io/docs/cli/init

Because this error occurred during a before all hook we are skipping all of the remaining tests.

Although you have test retries enabled, we do not retry tests when before all or after all hooks fail

What does visiting http://localhost:35607/mockServiceWorker.js in your app yield? It could be the URL hasn't been configured correctly in the worker.start command. I'm gonna make a PR to fix this by allowing us to pass an options object to it.

@Joelasaur can you have a look at #18 as this should allow you to do so.


Error when using sample data

I am using COMTSC to process my single-cell data, but when I use COMTSC in windows, I use the examples you provide in normal situations or in virtual environments.All suggest an error. .I have been troubled by this issue for several days and hope to get your help.
XLMHG error
('get_xlmhg_stat() takes from 3 to 4 positional arguments but 6 were given', 'occurred at index AAMP')
q-val error
local variable 'xlmhg' referenced before assignment
error in sliding values
local variable 'xlmhg' referenced before assignment
Creating discrete expression matrix...
discrete matrix construction failed
local variable 'cutoff_value' referenced before assignment

Hi rrrrui11, can you let me know a little more about your set-up and installation? For example, are you using a virtual environment? And what version of Python did you use to install the virtual environment?


Ollama Mixtral uses only 7% of the Nvidia RTX A4000 GPU.

Hello,
When I execute Ollama Mixtral with the Nvidia A4000 (16GB), I observe that only 7% of the GPU is utilized. Do you know why this might be happening? Additionally, the process seems somewhat slow. It appears that Ollama Mixtral is using 40% of the CPU but only 7% of the GPU.

Do you have any suggestions on how to increase GPU utilization instead of  %?

Same here on MacBook  M1 Pro 32Go :
GPU usage with mixtral is 0. Really slow.
Same prompt with mistral gpu usage between 70-90% Really fast.

Hi @nejib1, it seems that your system is bottlenecked on the CPU since the entire model won't fit into memory (only some does, as you can see in nvidia-smi (thanks for sharing this 😊 ) it's 14.8/16.3GiB which is almost all of your VRAM
@MatMatMatMatMatMat thanks for comment – GPU offloading isn't supported in macOS (yet!) so Mixtral will run on CPU on a 32GB Macbook Pro

@jmorganca does this nvidia-smi also mean CPU bottlenecked?  What would solve this, installing more RAM for instance? Thanks in advance!

@jmorganca Mistral is also running on my system by using maximum GPU usage but its like sometimes the GPU usage is lesser and sometimes it's higher but I seem to get a timeout error using mistral on MAC M2 Pro 16GB RAM.

Hi @nejib1, it seems that your system is bottlenecked on the CPU since the entire model won't fit into memory (only some does, as you can see in nvidia-smi (thanks for sharing this 😊 ) it's 14.8/16.3GiB which is almost all of your VRAM
@MatMatMatMatMatMat thanks for comment – GPU offloading isn't supported in macOS (yet!) so Mixtral will run on CPU on a 32GB Macbook Pro

Thank you for your help

It looks like we can close this issue as resolved.  If you're still having problems, please let us know.


Initial proposal for integration with Citadel

 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.

Thank you for your proposal @maxkondr.
Could you please also share more context for the authentication?
i.e. requirements, role of components, how we envision to utilize the components in the long-term (e.g. SSO, custom auth, ...), etc.
This will help us evaluate the choices made.
Thank you


pretrained NVC model

Any plans to release NVC model?

Sorry, but we don't plan to release our pre-trained model.


Fix Manage Shipments count and date bug

@aerinsol this fixes both the count and the recent bug for Manage Shipments.

 2.0 FE: In "Manage Shipments" create separate tabs for SENDING and RECEIVING shipments and add a default sort of "Last Updated" to the table


Exception when retrieving user avatar

I'm trying to build a Matrix app and I want to use the rooms members their avatars in the app, the SDK fails to retrieve them.

What was supposed to happen?
When you have a user object you can call get_avatar_url() to retrieve the avatar url.

What really happens:
As soon as you call get_avatar_url() the following exception is raised:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.4/site-packages/matrix_client/user.py", line 41, in get_avatar_url
    url = self.api.get_download_url(mxcurl)
  File "/usr/lib/python3.4/site-packages/matrix_client/api.py", line 715, in get_download_url
    if mxcurl.startswith('mxc://'):
AttributeError: 'NoneType' object has no attribute 'startswith'

How to reproduce

Get an user object from a room object, user profile, ...
call the get_avatar_url() method
Exception is raised

Environment

Python 3.4.6
OpenSUSE LEAP 42.3 x64
matrix_client version 0.2.0 (downloaded with PIP)

This can happen if the user hasn't got an avatar, as it will default to None. I suggest doing a None check in user.get_avatar_url before trying to fetch it.

I didn't know that! Thanks for the information!

Added PR #202 to fix this :)


Liskov substitution principle

It would be a great benefit to provide public model interfaces, and keep concrete implementations marked as internal. This would allow for better internal object handling and stub testing (perhaps using NSubstitute).
Alternatively...
In order to maintain the public API and not cause a breaking change:
Replace the currently implemented models with an abstracted class, keeping implementations internal.
This change would be handy both for developing the project, and a kindness as a vendor to clients who want to integrate into the API better.

I have something planned which involves a huge refactoring and hopefully results in a better project setup, including your mentioned point regarding models.
Some changes I will implement:

The models will be pure models. Right now, they have to be a child of BaseModel, which includes HTTP properties like Headers and error states. In the future, they should just be JSON models of the response. I don't think I will convert them to interfaces, but at least simple classes.

Separate SpotifyWebAPI in multiple clients, which are implemented based on interfaces. Similar to how octokit.net does it. Makes testing way easier and splits up the huge file we currently have
var spotify = new SpotifyWebAPI();
spotify.Playlist.Get("playlist-id"); // IPlaylistClient
spotify.User.Me(""); // IUserClient

Remove all non-async methods. They're just duplicated Code and are obsolete IMO.

Move error handling to Exceptions. While I'm not a fan of exceptions in general, I think this is more suitable for C#. Also allows us to return pure models from the client methods.

Correctly abstract HTTP Requests and Responses into their own interfaces. Also, to make testing easier. Also, make it easier for users to provide their own HTTP client if needed.

Add more unit tests. I have to think about integration tests (When to run them? What about endpoints which need user authentication? What about changing live-values?)

I will open another issue where I will track the progress and will close this one once I did. Thanks for your input!

Progress will be tracked in #451 and here is a sneak peak how testing could look like with the v6 release:
public async Task<string> GetAlbumName(IAlbumsClient albumsClient, string albumId)
{
  var album = await albumsClient.Get(albumId);
  return album.Name;
}

[Test]
public async Task TestGetAlbumName()
{
  var albums = new Mock<IAlbumsClient>();
  albums
    .Setup(album => album.Get("my-album-id"))
    .Returns(Task.FromResult(new FullAlbum { Name = "My Album" }));

  Assert.AreEqual("My Album", await GetAlbumName(albums.Object, "my-album-id"));
}

Would this be something you would expect? Is there anything we can do to make it even more easier?
If you are adventurous, you can also have a look at the v6 branch :)

Hi @o-litnon
With the beta release of v6, a lot of abstractions are now implemented and testing has been made easier for the user. Endpoint methods  have been split up into clients which provide interfaces, so testing is easier than before. Also, all models are now pure data objects and are not related to any HTTP context.
The Testing Guide describes an exemplary test.
If you have more feedback/problems/suggestions, feel free to reply here or open another issue!


Java: API - What about having the ability to pass different types of variable in the SKContext

Today SKContext only allows String (which makes sense):
public interface SKContext extends Buildable {
    SKContext setVariable(String key, String content);
}

So when we pass other type than String, we need to convert them:
myContext.setVariable("myVar", String.valueOf(myInt));

What about having helper methods that do the conversion? Something like this for int:
public interface SKContext extends Buildable {
    SKContext setVariable(String key, String content);
    SKContext setVariable(String key, int content);
}

And just do the conversion in the implementation:
public abstract class AbstractSKContext implements SKContext {
    @Override
    public SKContext setVariable(@NonNull String key, @NonNull String content) {
        variables.setVariable(key, content);
        return getThis();
    }

    @Override
    public SKContext setVariable(@NonNull String key, @NonNull int content) {
        variables.setVariable(key, String.valueOf(content));
        return getThis();
    }
}

Nothing major but a few methods would help with the String casting code all over the place.

To enable the ability to pass different types of variables in the SKContext in a Java API, you can utilize generics to make the context more flexible and accommodate various data types. Below, I'll provide a code structure and a diagram to illustrate this solution.
Issue Explanation:
The goal is to enhance the SKContext in a Java API so that it can accept and manage different types of variables.
Solution:
Define a Generic SKContext:
Modify the SKContext class to be generic, allowing it to accept a variety of data types. You can use Java generics to achieve this flexibility.
java
public class SKContext {
private T data;
public SKContext(T data) {
    this.data = data;
}

public T getData() {
    return data;
}

public void setData(T data) {
    this.data = data;
}

}
Usage of SKContext:
Use the generic SKContext class in your API wherever you need to pass different types of variables. The context can now be instantiated with various data types.
java
// Example usage of SKContext with different data types
// String data
SKContext stringContext = new SKContext<>("Hello, World!");
// Integer data
SKContext integerContext = new SKContext<>(42);
// Custom data type
SKContext customContext = new SKContext<>(new MyCustomType());
Benefits:
This modification allows the SKContext to be more versatile and adaptable to different use cases. It can hold various data types without the need for separate context classes.
Documentation:
Update the documentation to inform users of the enhanced SKContext and its ability to accept generic data types.

The "Usage Examples" section demonstrates how different types of data can be used with SKContext.
The "Documentation" section highlights the enhanced SKContext and its generic capabilities.

Will be done under #3626

Completed with the V1.0 work to date.


Make sure napalm-salt doesn't break because of python3 support in new napalm-base

@mirceaulinic Can you verify this.
I will verify it for Ansible.

Yes, I will make sure is not impacted. Thanks for the notice!

Confirmed everything is alright.


Tag / list releases properly

Describe the Bug
I like to stay up to date with new package version. But when I don't see what the changes to a previous version are it's super difficult to know if the minor version is gonna introduce / deprecate a prop / component that I'm using and therefore break my app.
The npm pages links to this repo, but here are no tags or releases, so I'm digging through last commits based on timestamps.
https://www.npmjs.com/package/react-email
Thanks
Marc
Which package is affected (leave empty if unsure)
react-email
Link to the code that reproduces this issue

To Reproduce

Expected Behavior

What's your node version? (if relevant)
No response

Not so sure if this is an issue, can you maybe elaborate a bit more?
Also, you can find a change-log that we are trying to keep up to date on our docs here.

It's not a bug. You can convert it into a discussion.
Usually when a new minor/major version is released I check versions or tags on GitHub to see what changed in the code.
Many oss packages release like that.

https://github.com/vercel/ai/releases
https://github.com/4lejandrito/next-plausible/tags

I'm not asking for beautiful release notes, just tag software when you are shipping a release :)

This is super relevant. We should definitely do that.