Trino Queries not considering UI selected schema in query  and failing with syntax error

Is the issue already present in https://github.com/cloudera/hue/issues or discussed in the forum https://discourse.gethue.com?
Nope
Describe the bug:
Below error when not specifying schema name even that is selected from UI
TrinoUserError(type=USER_ERROR, name=MISSING_SCHEMA_NAME, message="line 1:15: Schema must be specified when session schema is not set", query_id=20221215_173151_00229_mmkqb)
Steps to reproduce it?
select the schema from UI and then run any query on any table without specifying it in table. It should take default as UI selected but not working as expected
Select * from
Hue version or source? (e.g. open source 4.5, CDH 5.16, CDP 1.0...). System info (e.g. OS, Browser...).
4.10

Anyone facing similar issue ? do we have any workaround for this ?

Hi @sbbagal13, thanks for reporting this. Will take a look on this issue.
For the time being, adding this issue in the roadmap pipeline.

@bjornalm could you keep this issue open

Hello @sbbagal13,
We've made some change to the Hue code to better support Trino. Could you please try the latest version of Hue and let us know if you encounter the same issue? Your feedback is valuable. Thank you for reporting the issue.

Hello @sbbagal13, We've made some change to the Hue code to better support Trino. Could you please try the latest version of Hue and let us know if you encounter the same issue? Your feedback is valuable. Thank you for reporting the issue.

I found that there are several issues with Hue's support for trino connectors:

Users in trino must authorize all catalogs, otherwise an error message will appear: Query 20240319:031005_22463_1sm8u failed: Access Denied: Cannot select from columns [schemaName] in table or view schema (I have observed that in trino, execute: SHOW SCHEMAS From catalog; list all databases.)
Configure account passwords and LDAP trino. Only one user can be specified in the hue configuration file, and cannot impersonate a user or configure "auth_username": "${User}", "auth_password": "${PASSWORD}" to fill in the username and password. This will result in an error message when logging in with different hue users: io. trino. spi. security AccessiDeniedException: Access Denied: User usr_data_collect cannot impersonate user hive

@2416210017 Thanks for pointing this out.
Can you share what Hue config are you setting into Hue for enabling Trino editor? Also share the config the default_user config value under [desktop].
Also any specific Trino side configurations you've added or is it a vanilla setup?

@2416210017 Thanks for pointing this out. Can you share what Hue config are you setting into Hue for enabling Trino editor? Also share the config the default_user config value under [desktop].
Also any specific Trino side configurations you've added or is it a vanilla setup?

Thank you for your reply!
Of course, the following are the configuration files for [desktop] and [trino] in hue
[desktop]
secret_key=9XOhLUeIAvipV!&o
http_host=<IP_ADDRESS>
http_port=8887
time_zone=Asia/Shanghai
app_blacklist=pig,zookeeper,hbase,oozie,indexer,jobbrowser,rdbms,jobsub,sqoop,metastore
django_debug_mode=false
http_500_debug_mode=false
cherrypy_server_threads=50
default_site_encoding=utf
collect_usage=false
enable_prometheus=true
[[django_admins]]
[[custom]]
[[auth]]
backend=desktop.auth.backend.LdapBackend
idle_session_timeout=28800
[[[jwt]]]
[[ldap]]
ldap_url=ldap://<IP_ADDRESS>:389
ldap_username_pattern="uid=<username>,ou=People,dc=cc,dc=com"
use_start_tls=false
search_bind_authentication=false
create_users_on_login=true
base_dn="ou=People,dc=cc,dc=com"
bind_dn="cn=Manager,dc=cc,dc=com"
<EMAIL_ADDRESS>
[[interpreters]]
[[[trino]]]
name=Trino
interface=trino
options='{"url": "https://trino.cc.com:443", "auth_username": "hive", "auth_password": "xxxxxxxxxxxx"}'

In addition, our trino is enabled for LDAP authentication and HTTPS. The configuration file is as follows:
cat /data1/trino-server/etc/password-authenticator.properties
password-authenticator.name=ldap
ldap.url=ldap://<IP_ADDRESS>:389
ldap.user-bind-pattern=uid=${USER},ou=People,dc=belle,dc=com
ldap.user-base-dn=dc=cc,dc=com
ldap.allow-insecure=true

Hi @2416210017 ,
Could you also provide the config.properties file so that we can replicate the same behaviour in our environment?

Hi @2416210017 , Could you also provide the config.properties file so that we can replicate the same behaviour in our environment?

Of course not a problem, the following are the corresponding<config. properties>and nginx.conf (the frontend uses nginx to enable SSL proxy for the backend's trino server) in the corresponding trino.
# cat config.properties

discovery.uri=https://trino.cc.com
http-server.http.port=8060
coordinator=true
node-scheduler.include-coordinator=true
internal-communication.shared-secret=lhoOHWfehuKgkXnu4E6Gq5L5K4i8iMPlHFw
query.max-memory=100GB
query.max-memory-per-node=50GB
http-server.authentication.type=PASSWORD
http-server.process-forwarded=true
query.max-length=1000000000
query.max-history=5000
exchange.max-buffer-size=64MB
query.max-concurrent-queries=30
task.client.timeout=60s
query.client.timeout=20m

# cat nginx.conf

    server {
        listen      443 ssl;
        server_name trino.cc.com;

    ssl_certificate  /etc/nginx/conf.d/ssl/xxx.crt;
    ssl_certificate_key /etc/nginx/conf.d/ssl/xxx.key;
    ssl_session_timeout 5m;
    ssl_ciphers ECDHE-RSA-AES128-GCM-SHA256:ECDHE:ECDH:AES:HIGH:!NULL:!aNULL:!MD5:!ADH:!RC4;
    ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
    ssl_prefer_server_ciphers on;

    access_log  /var/log/nginx/access_443.log  proxy;
    proxy_read_timeout 3600s;

        location / {
            proxy_set_header Host $host;
            proxy_set_header X-Real-IP $remote_addr;
            proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
            proxy_redirect    off;
            proxy_set_header X-Forwarded-Proto $scheme;

            proxy_pass   http://<IP_ADDRESS>:8060;
        }
    }


Img conflict name

fixes #3156

the hashes are different from what I got in my local machine.
I haven't found a method in the File class that supports fuzzy search.
I will start by getting the hashes of files generated on the Github actions environement and stick them in the tests.
if this is not good enough, I will add a helped function that uses fs-extra to do something like this:
/**
 * Finds the files in a folder that have a given file name and extension.
 * This helps when looking for files generated by webpack with hashes
 *
 * @param {string} path
 * @param {string} fileName
 * @param {string} extension
 * @returns
 */
export async function filesFoundInFolder(path, fileName, extension) {
	const filesInFolder = await fs.readdir(path);
	const filteredFiles = filesInFolder.filter(
		f => f.startsWith(fileName) && f.endsWith(extension)
	);

	return filteredFiles;
}

So webpack hashes aren't generated based on the absolute paths, but from the where the root dir of the project. Which means that the hashes are consistent if the test are run in local and github actions. I set the exact has I had in local and it works now in Github. It didn't work earlier to me because I used 6 chars instead of 8


ctr: fix the incorrect image unmount error hint

Fix the following incorrect unmount error hint:
[root@localhost ~]# ctr images unmount
ctr: please provide a target path to mount to
Signed-off-by: Li Ning<EMAIL_ADDRESS>

@fuweid Hello, please help review my PR. It's a very simple patch.

@lining2020 Could you mind to remove the part of commit message?
Fix the following incorrect unmount error hint:
[root@localhost ~]# ctr images unmount
ctr: please provide a target path to mount to

The change is LGTM

@lining2020 Could you mind to remove the part of commit message?
Fix the following incorrect unmount error hint:
[root@localhost ~]# ctr images unmount
ctr: please provide a target path to mount to

The change is LGTM

OK, I will push a patch v2.


Fix #807, Add ES Application Behavior Functional Tests

Describe the contribution
Fixes #807
Add ES Application Behavior Functional Tests
Testing performed
Build and run unit test
Expected behavior changes
No impact to behavior
System(s) tested on
Ubuntu 20.04
Additional context
CFE_ES_ExitApp was not directly tested. https://github.com/nasa/cFE/issues/1761 will be needed to test that.
Contributor Info - All information REQUIRED for consideration of pull request
Alex Campbell GSFC

CCB:2021-08-11 APPROVED


[AMQP] Ensure the EndpointState terminate upon ReceiverClient closure

The ReceiveLinkProcessor relies on the current ReceiverClient's EndpointStates termination to obtain a new ReceiverClient to continue streaming events downstream or terminate the downstream. I.e., we want to ensure EndpointStates termination happens in all scenarios; else, it can cause a hanging downstream.
Unfortunately, in two scenarios, the EndpointStates termination will not happen -

If the client fails to schedule the local-close.
If there is a missing remote-close ack for a client-initiated local-close, this can occur if,

a). the remote-close never arrives.
b). the parent session/connection hosting link was already closed or in a faulted state.
c). the ProtonJ event-loop thread never picked the local-close work (because it's been shutting down).

We want to slightly update the ReceiverClient's closure route to ensure the termination of EndpointStates. Ideally, after the update, the flow looks like this -

\cc @conniey @ki1729

Attached is a section extracted from the customer log, where the case "2.c)" happened, leading to the stoppage event-receive from 3 out of 4 partitions.
Demystifying the incident flow from logs:

Application consuming from 4 partitions (P0, P1, P2, P3).
A single connection CON0 is hosting 4 ReactorReceivers (one per partition).
Sevice sent remote-close for P2 (seems idle timeout, a 10 min quiet time between the last received event and remote-close).
The P2 ReactorReceiver reacted to the remote-close and marked itself as disposed.
The P2 AmqpReceiveLinkProcessor reacted to this disposal by requesting a new  ReactorReceiver for P2.
The application used "azure-messaging-eventhubs:5.11.1", which has the code path prone to the bug_27716.
The P2 ReactorReceiver hit this code path.
As explained in bug_27716, the "no current delivery" exception from P2 ReactorReceiver bypassed and faulted the shared parent connection CON0, leading to the shutdown preparation of CON0.
The CON0 notified shutdown to P0, P3, and P4. The P2 was already disposed (step4); hence, it won't be notified about the shutdown.
In response to the shutdown signal, P0, P3, and P4 ReactorReceivers enqueued (scheduled) local-close work to ProtonJ EventLoop.
The CON0 continued the shutdown process and requested ProtonJ EventLoop to run one last iteration.
The ProtonJ EventLoop ran, but it didn't pick the local-close(s) scheduled in step 10; it indicated that it is shutting down though there are some tasks to process.
Not much to do here; we can't wait forever for the shutdown step complete (the library wants to finish CON0 and recover).
The CON0 completed the shutdown process.
Meantime, the request in step5 was honored, resulting in a new ReactorReceiver for P2 on a new connection CON1.
Since the local-close work from ReactorReceiver P0, P1, and P3 was never executed, the corresponding AmqpReceiveLinkProcessors never detected the termination of those 3 ReactorReceivers.
The missing signal resulted in those AmqpReceiveLinkProcessors not requesting new ReactorReceivers, causing downstream application receivers for P0, P1, and P3 to hang.

Notes:
Had the customer used the next version, i.e., azure-messaging-eventhubs:5.11.2, the application wouldn't have hit the bug_27716 code path and wouldn't have faced this incident.
But the underlying issue of AmqpReceiveLinkProcessor missing the terminal signal was still a bug; it's that combining bug_27716 with the edge case that ProtonJ EventLoop ignoring some work while shutting down revealed this.
Additional thought was - as observed, the P2 idle timed out (step2), but other partitions were also inactive for 10 mins; it's possible that the service sent remote-close for those as well, but those closures may not bubble up as the client started the shutdown  (no low-level logs to prove it though).

This is addressed in the pr https://github.com/Azure/azure-sdk-for-java/pull/29201


 To see the full traceback, use -vvv. The error was: ValueError: Invalid type provided for "string": {'template': 'templates/single_nic_vlans/single_nic_vlans.j2'}
2023-09-08T08:35:32.169891346+00:00 stdout F fatal: [edpm-compute-0]: FAILED! => {"msg": "Unexpected failure during module execution: Invalid type provided for "string": {'template': 'templates/single_nic_vlans/single_nic_vlans.j2'}", "stdout": ""}

recheck

right, old job will need some attention as well -.-...

right, old job will need some attention as well -.-...

https://github.com/openstack-k8s-operators/install_yamls/pull/524

Also trying with https://github.com/openstack-k8s-operators/ci-framework/pull/533

recheck

it seems to be a real failure:
2023-09-08T10:34:10.667558475+00:00 stdout F TASK [osp.edpm.edpm_network_config : Render network_config from template] ******
2023-09-08T10:34:10.668816944+00:00 stdout F Friday 08 September 2023  10:34:10 +0000 (0:00:00.193)       0:04:42.315 ******
2023-09-08T10:34:11.127426071+00:00 stdout F An exception occurred during task execution. To see the full traceback, use -vvv. The error was: ValueError: Invalid type provided for "string": {'template': 'templates/single_nic_vlans/single_nic_vlans.j2'}
2023-09-08T10:34:11.127426071+00:00 stdout F fatal: [edpm-compute-0]: FAILED! => {"msg": "Unexpected failure during module execution: Invalid type provided for \"string\": {'template': 'templates/single_nic_vlans/single_nic_vlans.j2'}", "stdout": ""}

https://logserver.rdoproject.org/58/458/7cd507f8d3ffaa26d1624cb46fb3aad48a919d61/github-check/openstack-operator-crc-podified-edpm-deployment/a28bb72/ci-framework-data/logs/crc/pods/openstack_dataplane-deployment-configure-network-openstack-edpm-r7knr_9b898dc0-a60a-4a3a-8dab-107f67a954e4/openstackansibleee/0.log

it seems to be a real failure:
2023-09-08T10:34:10.667558475+00:00 stdout F TASK [osp.edpm.edpm_network_config : Render network_config from template] ******
2023-09-08T10:34:10.668816944+00:00 stdout F Friday 08 September 2023  10:34:10 +0000 (0:00:00.193)       0:04:42.315 ******
2023-09-08T10:34:11.127426071+00:00 stdout F �[0;31mAn exception occurred during task execution. To see the full traceback, use -vvv. The error was: ValueError: Invalid type provided for "string": {'template': 'templates/single_nic_vlans/single_nic_vlans.j2'}�[0m
2023-09-08T10:34:11.127426071+00:00 stdout F �[0;31mfatal: [edpm-compute-0]: FAILED! => {"msg": "Unexpected failure during module execution: Invalid type provided for \"string\": {'template': 'templates/single_nic_vlans/single_nic_vlans.j2'}", "stdout": ""}�[0m

https://logserver.rdoproject.org/58/458/7cd507f8d3ffaa26d1624cb46fb3aad48a919d61/github-check/openstack-operator-crc-podified-edpm-deployment/a28bb72/ci-framework-data/logs/crc/pods/openstack_dataplane-deployment-configure-network-openstack-edpm-r7knr_9b898dc0-a60a-4a3a-8dab-107f67a954e4/openstackansibleee/0.log
maybe the depends-on was added after the run?

Maybe, the current run has already passed that step successfully. I rechecked cause I saw a timeout error from etcd that made the CR apply step to fail.


Change default "/" route to use "HomeController@showWelcome"

hello!
I spent about 2 hours trying to figure out why my default installation of Laravel wouldn't allow me to do render variables in to my view. I tried View::make("hello")->with("test", "some test value") and I tried View::make("hello", array("test" => "some test value")) with no luck. I constantly received "variable undefined" when trying to use the $test variable in my views.
It turns out the default route for "/" was calling View::make("hello") directly in a callback function, while I was modifying the showWelcome method in HomeController.  Once I changed the "/" route to use the controller method, everything worked fine. But it took me a long time to figure this out. It would be good to have the default "/" route configured like this:
Route::get('/', "HomeController@showWelcome");
so that others don't run in to the same frustration.

Route::get('/', 'HomeController@index')->name('home');


delete .travis.yml

fixes https://github.com/kubernetes/heapster/issues/1771
migrated to prow by
https://github.com/kubernetes/test-infra/pull/7463 and
https://github.com/kubernetes/test-infra/pull/7465
green run:
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/pr-logs/pull/heapster/1995/pull-heapster-unit/1/
/assign @DirectXMan12
cc @BenTheElder

/approve
/lgtm

/hold
until the other PR merges

/hold cancel
prow PR is merged


[WIP] Rlp block bodies bug

What was wrong?
Seems like we have a discrepancy between how we encode our typed transactions & receipts in the portal wire spec, and then how they're encoded in eth1 devp2p messages.
How was it fixed?
To-Do

[ ] Clean up commit history and use conventional commits.

closing in favor of #1234


DRILL-5007: Dynamic UDF lazy-init does not work correctly in multi-no…

…de cluster

+1. Good catch guys!


script_tag: cosmetic for value-less attributes

Description
For <script /> elements, some attributes are usually written without values, for example :
<script defer async src="..." />
This is purely cosmetic as boolean attributes should also be accepted with an empty
value or a value identical to the attribute's name :
https://dev.w3.org/html5/pf-summary/infrastructure.html#boolean-attribute
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Thank you for sending a PR!
It seems you forgot GPG signing.
https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#signing

@xlii-chl There are some coding style violations. Please fix them.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#php-style
   1) CodeIgniter4/system/Helpers/html_helper.php (is_null, unary_operator_spaces, not_operator_with_successor_space)
      ---------- begin diff ----------
--- /home/runner/work/CodeIgniter4/CodeIgniter4/system/Helpers/html_helper.php
+++ /home/runner/work/CodeIgniter4/CodeIgniter4/system/Helpers/html_helper.php
@@ -208,7 +208,7 @@
                 }
             } else {
                 // for attributes without values, like async or defer, use NULL.
-                $script .= $k . (is_null($v) ? ' ' : '="' . $v . '" ');
+                $script .= $k . (null === $v ? ' ' : '="' . $v . '" ');
             }
         }

      ----------- end diff -----------

   2) CodeIgniter4/tests/system/Helpers/HTMLHelperTest.php (trim_array_spaces)
      ---------- begin diff ----------
--- /home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Helpers/HTMLHelperTest.php
+++ /home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Helpers/HTMLHelperTest.php
@@ -250,7 +250,7 @@

     public function testScriptTagWithSrc()
     {
-        $target   = [ 'src' => 'http://site.com/js/mystyles.js' ];
+        $target   = ['src' => 'http://site.com/js/mystyles.js'];
         $expected = '<script src="http://site.com/js/mystyles.js" type="text/javascript"></script>';
         $this->assertSame($expected, script_tag($target));
     }
@@ -257,7 +257,7 @@

     public function testScriptTagWithSrcWithoutProtocol()
     {
-        $target   = [ 'src' => 'js/mystyles.js' ];
+        $target   = ['src' => 'js/mystyles.js'];
         $expected = '<script src="http://example.com/js/mystyles.js" type="text/javascript"></script>';
         $this->assertSame($expected, script_tag($target));
     }
@@ -264,7 +264,7 @@

     public function testScriptTagWithSrcAndAttributes()
     {
-        $target   = [ 'src' => 'js/mystyles.js', 'charset' => 'UTF-8', 'defer' => '', 'async' => null ];
+        $target   = ['src' => 'js/mystyles.js', 'charset' => 'UTF-8', 'defer' => '', 'async' => null];
         $expected = '<script src="http://example.com/js/mystyles.js" charset="UTF-8" defer="" async type="text/javascript"></script>';
         $this->assertSame($expected, script_tag($target));
     }

      ----------- end diff -----------

https://github.com/codeigniter4/CodeIgniter4/runs/5892697479?check_suite_focus=true

Hello,
thanks for reviewing.
Sorry about the misses, I thought the gpg-signing etc. were for the core members or reviewers. Since signing modify the commit, I'll force-push in a bit if it's okay with you.
I'll put the is_null($v) ? ... → null === $v ? ... change in a separate commit (it seemed kinda less readable to me : needs to recall the operator precedence) so you can omit it if you feel the same way.

Fore push is okay.
git rebase is recommended way to sync the PR branch.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#pushing-your-branch

I'll put the is_null($v) ? ... → null === $v ? ... change in a separate commit (it seemed kinda less readable to me : needs to recall the operator precedence) so you can omit it if you feel the same way.

The coding style change is a must. If you don't fix it, GitHub Action check fails, and we can't merge this PR.

@xlii-chl Thank you for updating!
One last thing, we need the User Guide update.
https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#documentation

Hello,
For the doc, I've added a mention in the example:
--- a/user_guide_src/source/helpers/html_helper/011.php
+++ b/user_guide_src/source/helpers/html_helper/011.php
@@ -1,6 +1,6 @@
 <?php

-$script = ['src' => 'js/printer.js'];
+$script = ['src' => 'js/printer.js', 'defer' => null];

 echo script_tag($script);
-// <script src="http://site.com/js/printer.js" type="text/javascript"></script>
+// <script src="http://site.com/js/printer.js" defer type="text/javascript"></script>

For such a simple thing, it seemed enough to me but do you feel we should add more ? Since I am not a native english speaker, I may need help to write something good.
Or do you mean the Changelog ?
This change may be a little trivial for that, but what about, in the Enhancements section :

script_tag() in the html helpers now able to concisely write boolean attributes : <script src="..." defer />.

I thought we need explanation for the parameter $src, but as you say the sample code is enough.
Please update:

Parameters $src type
https://codeigniter4.github.io/CodeIgniter4/helpers/html_helper.html#script_tag

mixed is vague and useless.
    :param  array|string|null  $src: The source name of a JavaScript file

Change Log
https://codeigniter4.github.io/CodeIgniter4/changelogs/v4.2.0.html#enhancements

I think all enhancements should be included the changelog, otherwise users don't know the enhancements.
How about like this?
- The boolean attribute can now be written concisely by passing ``null`` to the ``script_tag()`` parameter ``$src`` in the HTML helper. See :php:func:`script_tag`'s sample code.

And could you also update the param type in PHPDoc? It is not mixed but array|string|null.
https://github.com/codeigniter4/CodeIgniter4/blob/a8d5b894c9ae17f1a8c8721eae44c0da22bec87c/system/Helpers/html_helper.php#L192
Don't forget to run composer cs-fix.


Bitbucket repo Webhook connection URL with repeated "https"

Bug description 🐞
Hey guys, i would like to report a bug (dont know if its really a bug, but looks like it)
I'm doing some tests here to finally get rid of terrraform cloud and use terrakube on its place.
When i was setting up my new workspace and attaching the VCS (Bitbucket) to it, i was asking myself why the new commits werent appearing, when i saw this at the webhook section:
When i edited it and removed the repeated https it started working.
I checked some other repos i work with (that are linked to tf cloud, and it creates it normally)

I was using latest version at this time (2.23.2), did a full database wipe and then downgraded to 2.22.0, to check if it was a specific problem of the latest version, but looks like it wasn't.
I'm using the docker compose version of terrakube on an EC2 machine. (I can't use them on kubernetes because i want terrakube to manage/edit/remove my clusters)
Steps to reproduce
Create a new workspace and put bitbucket as its VCS
Expected behavior
I expected the correct URL, without repeated https://
Example repository
No response
Anything else?
No response

Hello @isbig5I saw something similar some time ago because the docker compose environment variable in the example was wrong. Check this PR
https://github.com/AzBuilder/terrakube/pull/796/files

@alfespa17 Thanks! I changed what you told me and it worked fine.


bug: path_exists() does not respect set working-directory

path_exists() function computes relative paths relative to the justfile location, not the working directory.
This one is probably tricky because the order of commands might be important:
with_foo1 := if path_exists("foo") != "true" { 'yes' } else { 'no' }
set working-directory := 'bar'
with_foo2 := if path_exists("foo") != "true" { 'yes' } else { 'no' }

Great find! The order of statements doesn't matter, everything in a justfile works in an order-independent fashion. This was a problem in five functions. Fixed in #2294.

:rocket: :rocket: :rocket:  Insanely fast turnaround!!! Thank you :)


Invalid status 6a80 when trying to generate a transaction

Solution(s)
When you are connected via a Ledger you will sometimes encounter this error when attempting to generate a transaction. It may be because....

You are not using a secure (SSL) connection

You are not using Chrome/Opera

You are attempting to send a transaction that has data without first turning "contract data" ON in your ledger settings (it's right above 'browser support')

???

I am still seeing this error occasionally despite satisfying all the conditions listed above.

You are absolutely positive your contract data is switched to the ON position?

Hmm, I can't reproduce this anymore. Unplugged/plugged back in the Ledger and everything seems to be working fine now. Ignore this for now - I will post back if I encounter this again. Thanks!

Looks like I am running into the same error... trying to transfer Vslice tokens to Liqui

Found it it was a setting about the contract data set to off...  as the post two up..
face palm

Apologies in advance for not being relevant to the ticket :
I have 'contract data' turned on, created an ethereum txn object that I am trying to hash using rlp.encode similar to MEW.
This code returns Invalid status 6a80. Any insight would be greatly appreciated. Thanks!

I experience the same problem with PPT tokens.
I am using ledger nano S, sending PPT token, it says "invalid status 6a80" when I click generate transaction.

Thanks! You're awesome - quick google search and I resolved my error.


feat: add beta tag to explorer with link to beta announcement (#3938)

Ticket
Closes #3938.
Reviewers
@NoopDog.
Changes

Added Beta Announcement banner.
Added Beta Announcement to Help & Documentation menu.
Added "General Announcement" page.

Note: Beta Announcement banner is persistent and currently will stack underneath Session Timeout banner and / or System Status banners.

Let's go ahead and do the work so these don't stack


Scala 2.11 builds

Hi,
Your README says:

As of Morpheus 0.3.0, the project has migrated to Scala 2.12 and Spark 2.4 series. As of Spark 2.4.1 Scala 2.12 is the default Scala version for Spark, and Morpheus is attempting to mimic this.

However after a few bad releases Spark at version 2.4.3 has switched back to Scala 2.11 for primary releases.
Is it possible to publish 2.11 builds too as changing the Scala version in build.params.gradle alone does not compile.
Thanks

Hello @seddonm1
You're right that Spark switched back to using 2.11 by default. I will modify our README to reflect this fact.
However, that was not the main reason for us to switch Morpheus over to 2.12. We did that in preparation for Spark 3.0, which is going to leave 2.11 behind altogether (link), and also to reduce the effort we would have to cross-build and cross-test all of our modules.
Unfortunately, we are unable to reprioritise on this issue, but we will stick with 2.12 only, even though Spark has changed back to offer 2.12 as non-default, non-prebuilt support. It is still possible to build your own Spark using 2.12 and use Morpheus with it, or to use the 2.4.2 prebuilt binaries.
Since a significant portion of this project will be made official with Spark 3.0, this is where we are focusing most of our efforts currently.
Apologies for the inconvenience
Regards
Mats


[main] Upgrade to latest dependencies

hack was fixed -dprotaso
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/pkg 20aabd5...5f66ecf:
  > 5f66ecf Allow configuration of threads-per-controller (# 2567)
  > 717747b Register webhook GVKs for validating callbacks (# 2590)
  > 501fbd5 upgrade to latest dependencies (# 2592)
  > 3d24b84 fix lint warnings (# 2589)
  > cc6e435 upgrade to latest dependencies (# 2587)
bumping knative.dev/control-protocol 9d1dfb4...64e2528:
  > 64e2528 upgrade to latest dependencies (# 213)
bumping knative.dev/hack 36b2b3c...d1317b0:
  > d1317b0 errexit and pipefail should remain off until downstream is fixed (# 228)
  > df80419 :bug: Fix for test failures of knative/serving# 13292 (# 227)
  > 7556452 :gift: Use modscope tool to get info about Go modules (# 223)
  > 103cdb3 Default `SIGN_IMAGES` variable to `false` (# 225)
bumping knative.dev/networking aded181...cefed20:
  > cefed20 upgrade to latest dependencies (# 710)

/retest

/lgtm
/approve


Drill to Details charts

Hello All, I am new to Superset, I am looking for feature similar to Drill to Details.
I have list of Group Names and other group metadata in base chart and when I click on Group Name, I want to see another dashboard/chart with two tabs, first tab will contain the Group Members and Second Tab will contain Group activity. How can i achieve this ? Any documentation on how to build this would be helpful.
Thanks in Advance.

There's a feature on the roadmap called Jump to Dashboard that sounds a bit like this. Effectively you'd click a part of a chart in Dashboard A, and that part (say a pie chart wedge) would apply a filter as it takes you to Dashboard B with the filter applied. This is still early in development, and we'll be talking about this at a Meetup this Tuesday if you'd like to learn more and see designs and potentially a demo.
Closing this since it's more feature request than bug report, but we appreciate validation of the idea! Feel free to open an Ideas thread on Github Discussions if you want to get more into the design/planning of things.


better handling of variing amount of fields in /proc/stat

Turns out we can't rely on all fields always being available, so we
need to be a little more defensive when parsing /proc/stat
closes #406

jenkins test this

jenkins test this


Reading Environment variables and passing in RunContext

Passing Pipeline Variables to createRun flow

@kapilratra02 please validate this in old pipeline/new pipeline.
Flaky test caused build failure, please requeue

@kapilratra02 please validate this in old pipeline/new pipeline.
Flaky test caused build failure, please requeue

Validated


Synchronous data loading with multiple asyncConnected components.

Hi, I am working on a project in which we use multiple components with asyncConnect decorators under a parent component. We do this by adding each of these components to the react-router component under "components" and naming them.
<Route component={Home}>
<IndexRoute components={{ componentA: ComponentA, componentB: ComponentB}}/>
</Route>
And then inside the Home component we place each of the asyncConnected components using {this.props.componentA} etc.
Resolving multiple promises in each individual component is fine, however resolving the promises on the list of components is done synchronously. i.e all promises are resolved on ComponentA asynchronously first, then all promises are resolved on ComponentB asynchronously.
Is this a feature or a bug? If its a bug I'll open a pull request for a fix.

Since they are on the same level they should resolve asynchronously, making this a bug

On 12 Oct 2016, at 15:39, thomasdrakebjss<EMAIL_ADDRESS>wrote:
Hi, I am working on a project in which we use multiple components with asyncConnect decorators under a parent component. We do this by adding each of these components to the react-router component under "components" and naming them.

And then inside the Home component we place each of the asyncConnected components using {this.props.componentA} etc.
Resolving multiple promises in each individual component is fine, however resolving the promises on the list of components is done synchronously. i.e all promises are resolved on ComponentA asynchronously first, then all promises are resolved on ComponentB asynchronously.
Is this a feature or a bug? If its a bug I'll open a pull request for a fix.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok @AVVS, I have opened a pull request for a potential fix.

@thomasdrakebjss Thanks a lot for the contribution. Merged in v4.0.0


Enhanced key management

At the moment the function
$ near account list-keys <account> network-config mainnet now
displays the keys of the account in an order, presumably, from the newest to the oldest. It would be good to be able to specify listing of full access keys only or highlight somehow these keys. It is important for keeping the clean setup of all keys within an account. Similar issue is with function call keys with infinite allowance.
Besides that it would be nice to be able to manage the keys straight away from the table that is an output of the call. Ideally, a user can navigate the keys using arrows and able to delete them with one key press. The flow should allow for multiple key deletions.
At the moment the cleanup of the keys associated with the account requires periodic display of the key list and manual copying of the keys and deleting them one by one.

Thanks for the suggestion!

At the moment the cleanup of the keys associated with the account requires periodic display of the key list and manual copying of the keys and deleting them one by one.

@alexauroradev Would it help if delete-keys had a multi-select of all the keys listed there with their type in parenthesis?


Canonicalizer Unittests, and Break Canonicalization.

Addresses CR for #311
Adds Canonicalizer unit tests for UnaryOp, Continue and Break.
Corrected Break syntax test. Added a new test.
Implemented Break Canonicalizer that moves all the breaks in the loop to the last line within an if condition.

It looks like this PR is ready to be merged? I'll merge this. If sawada-san has any comments or we find an issue in the current PR, we can handle them in a separate PR. Thanks!


Add documentation for genesis CLI tool

Signed-off-by: Peter Schwarz<EMAIL_ADDRESS>

Taking a look.


[PRFC] Improvements to the scaffolder/next buttons UX

Hey, I just made a Pull Request!
This PRFC is attempting to provide some improvements to the scaffolder/next buttons UX by doing the following:

Added padding around the "Create" button and set the color property to primary in the Stepper component
Added a button bar that includes the "Cancel" and "Start Over" buttons to the OngoingTask component. The state of these buttons match their existing counter parts in the Context Menu. The button bar is visible when the steps are ongoing. When the steps complete with a happy result the button bar is hidden, if this fails then the button bar is visible much like the logs.
Added a "Show Button Bar"/"Hide Button Bar" item to the ContextMenu component

Stepper component:

Before
After

OngoingTask component:

Happy Path
Unhappy Path

Happy Path video:
https://user-images.githubusercontent.com/67169551/232115783-ccc0ed7c-3414-4abd-8cfe-5146467945cc.mp4
Unhappy Path video:
https://user-images.githubusercontent.com/67169551/232116230-75287d87-c47e-403d-9ab3-defa78574394.mp4
ContextMenu component

Show
Hide

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@adamdmharvey @sennyeya @regicsolutions @efenner-cambia @webark just a heads up that I moved PR #1642 to my work account and moved it to ready so that I can more easily get it merged in. Thanks a ton for your patience, just a bit longer as I work though getting this reviewed now 🚀

@freben do I need to do anything special with the DCO? @ahhhndre is me so not sure if this needs anything special.

As for button padding: I think they really shouldn't have padding. They were already nicely aligned with all other content. What was rather missing, was some marginTop, right, to give them some distance to the content above? I think that'd look better than a full padding!

( I really like this PR.  Note as a bit of an outsider who's not used any of the scaffolder/next stuff, but I will say seeing two styles of check mark iconography does throw me off.  Blue checks and green checks? over time I think as a user I'd expect them to start to align from a design stand point! )


152 basic implementation mapper

This Pull Request includes:

Abstract Base Class for the mapping part of the Mapper pass
An IdentityMapper. Mapping the first virtual qubit to the first physical qubit, etc.
An HardcodedMapper. Map using a predefined (hardcoded) mapping.
Standardized tests for the Mapper class. Checks barebones of what we require of a Mapper.
Use the output of the Mapper class in a mapper pass on the Circuit object.

This also includes the following bugfix:

__eq__ of Measure no longer returns an array of bools.

Closes #152

How do I activate the github actions. When I made this pull-request the github actions started and some of them failed. Then (I think) I fixed the problem, but now the github actions are not running. What do I need to do to let them run?

A new commit to this branch should initiate the GitHub actions. Otherwise something is not configured correctly....


Make back button go back to "Objectives" page when viewing an SLO

When I'm playing around with an SLO (e.g. changing error budget time window, scrolling down etc...) and then press the back button on my browser, I expect to be taken back to the "Objectives" page to see all SLOs. What is happening right now is that all scroll movements and time window changes are cycled through in reverse order before going back to the "Objectives" page. I wonder if that's intentional.

This isn't ideal for sure.
We should probably move back to setting the timestamps relatively. As in now-1h and now rather than updating the timestamps e.g. every 10s...
The underlying idea was to have timestamps in the URL to make sharing a SLO with the specific time ranges possible. However, the specific timestamps should probably only be set if one selects the specific range in the graphs...


feat(#532): Remove Outdated Puzzle

Remove the outdated puzzle related to BytecodeAnnotation refactoring.
Closes: #532.

PR-Codex overview
This PR refactors the annotation handling in the eolang project by renaming and replacing instances of XmlAnnotationProperty with XmlAnnotationValue. This change improves clarity in the code regarding the distinction between annotations and their values.
Detailed summary

Renamed XmlAnnotationProperty to XmlAnnotationValue.
Updated constructors and method calls to use XmlAnnotationValue.
Changed method names from props() to values() in relevant classes.
Adjusted test cases to reflect the new class name and functionality.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 31min).


Handle function type in useState

This is not the most straightforward thing, but it does catch an edge case where the types allow incorrect behaviour the state value type is itself a function.
Context: https://flow.org/try/#0JYWwDg9gTgLgBAbwK4GcCmBlGBDGaC+cAZlBCHAORRrYDGMFA3AFDO0QB2K8A2t7mgA0cdDCwCAunAC8cVJhx4APAH4AFNyjAOAcwCU0gHwckAG1OG1J83pbNR4vGrUB9GAC4RMLboPGzprbMQA
This uses the overload type syntax. Hopefully that works correctly?

This change doesn't seem right. Aside from implementation details (using the Function type), useState is overloaded to call (lazily) a provided function. So you can't actually have a function-valued state in React, since React will instead try to call that function.


Adds VoiceChannels and Related Chats to Config

Closes #1304
Adds the following channels to default config:

Code/Help 1 (voice)
Code/Help 2 (voice)
General/Offtopic (voice)
code-help-voice-1 (text)
code-help-voice-2 (text)
staff-voice (text)
admin-voice (text)
Note: admin-voice (text) is still missing an ID.

I tried sepearting out the two groups (text chat and voice chat), as it is confusing in the current form:
L200-L201: voice
L202-L204: text

Coverage increased (+0.02%) to 56.519% when pulling 6204906c734e0f0d44f7c1c544fbf4eb2443acc8 on HassanAbouelela:seperate-voice-chat-channels into ed8160f3e73e2060630b10e1fb0b762e0a51294a on python-discord:master.

It's necessary to merge this PR before #1297

Not sure does this collide with https://github.com/python-discord/bot/pull/1016 .


bug:related to version up

Description
when I upgrade to github.com/containerd/containerd v1.7.11 then registry.PingV2Registry(endpoint, authTransport) this method return parameter reduced to 2 , how to solve this , I am getting issue in wire injection
Steps to reproduce the issue

registry.PingV2Registry(endpoint, authTransport)
Describe the results you received and expected
registry.PingV2Registry(endpoint, authTransport)
What version of containerd are you using?
registry.PingV2Registry(endpoint, authTransport)
Any other relevant information
registry.PingV2Registry(endpoint, authTransport)
Show configuration if it is related to CRI plugin.
registry.PingV2Registry(endpoint, authTransport)

need urgent solution

Can you provide more information about the issue you are facing. Are you seeing the issue when using docker ?The PingV2Registry method is part of moby codebase.

ya earlier , I was getting issue from moby codebase , not it is resolved
Thanyou

solution :
if using https://github.com/oras-project/oras-go
then upgrade this to  v1.2.4


Testing Dynamic DOM content on ASP.NET Pages

I've noticed that PhantomJs does not read updated the DOM for ASP.NET pages.
Expected behavior

Load https://m.novocinemas.com/mobile/MovieListView.aspx using phantomjs
Expect DOM content of div.select_cinema to be updated when retrieved using page.property('content') (the DOM content of div.select_cinema is updated due to a script on the webpage)

Actual behavior

Load https://m.novocinemas.com/mobile/MovieListView.aspx using phantomjs
The DOM Content of div.select_cinema is empty as retrieved from the original page load.

Steps to reproduce the behavior

Execute the following code

var phantom = require('phantom');

phantom.create().then(function (ph) {
    ph.createPage().then(function (page) {
        page.open('https://m.novocinemas.com/mobile/CinemaList.aspx').then(function (status) {
            console.log(status);
            setTimeout(() => { //just added for enough delay for the scripts to render
                page.property('content').then(function (content) {
                    console.log(content);
                    page.close();
                    ph.exit();
                });
            }, 5000)
        });
    });
});

Am I doing something wrong because the DOM does not seem to be updated after page scripts modify the page
Environment

OS: MacOS Sierra (10.12.3)
Node version (node -v): 6.9.5
package version: 4.0.0

I think there is something going on with your page that is unrelated. I would first see if you can actually get this to work with plain phantomjs, without Node. If that works, paste the code so I can debug. That would be a bug.
If that doesn't work then something is up with your dom, because I haven't been able to reproduce it.

hmm let me try normal Phantomjs first and i"ll revert back here. Did you get a chance to try the script I posted above? Are you also getting the same behavior?

I haven't had a chance to run it yet. But I remember seeing that this worked. Look at this code https://github.com/amir20/phantomjs-node/issues/431#issuecomment-207865947. Instead of timeout, you could do wait for object to be there. This was a while ago so I hope it works.
I am at work, but maybe later I can run your script.

Also, I don't think this is related to ASP. I went to https://m.novocinemas.com/mobile/CinemaList.aspx and nothing happened. Perhaps try a different page?

Any reponse?

Hi, I didn't get a chance to explore it further. Will do it this weekend and report back
Sent from my iPhone
On Mar 1, 2017, at 7:45 PM, Amir Raminfar <notifications<EMAIL_ADDRESS>wrote:
Any reponse?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/amir20/phantomjs-node/issues/615#issuecomment-283377155, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AC0qzhJdHlOYBwXxateQ0l4wu432Lia5ks5rhZKtgaJpZM4MI5xZ.

I had the same issues ... but when I updated to phantomjs 2.5-beta (which supports ES6) it all worked fine and the content was updates. I tested by copy/pasting the upper code and just changing the url to my React app.

@SvenAlHamad that's awesome. How did you upgrade to beta? It has been on my to do list for a while but haven't had a chance to play with it.

Good thing I noted everything in my evernote :)
so, I use ubuntu 16.04 and this are the steps:
download it from here:
https://bitbucket.org/ariya/phantomjs/downloads/
tar -zxvf {archive name here}

sudo apt-get install libstdc-dev
sudo apt-get install libwebp-dev
sudo apt-get install libhyphen-dev
sudo add-apt-repository ppa:ubuntu-toolchain-r/test
sudo apt-get update
sudo apt-get install gcc-4.9 g++-4.9
sudo mv phantomjs-2.5.0-beta-ubuntu-trusty /usr/local/share/
sudo ln -sf /usr/local/share/phantomjs-2.5.0-beta-ubuntu-trusty/bin/phantomjs /usr/local/bin
phantomjs --version

oh nice!

Marking as an upstream dependency since phantom seems to have fixed it.

OK I think it's safe to close this since we know Beta fixes it.


Add a %NativeFunctionPrototype% for user created Duktape/C functions and lightfuncs

Inheritance chain for user created Duktape/C functions and lightfuncs is changed to:
value -> %NativeFunctionPrototype% -> Function.prototype

The %NativeFunctionPrototype% object is internal and isn't explicitly registered to a global object. You can still access it e.g. as myFunc.__proto__, similarly to %TypedArrayPrototype%. The previously virtual .length and .name are now getters in %NativeFunctionPrototype%, and handle both normal functions and lightfuncs. Exotic behavior for Duktape/C functions and lightfuncs is removed from property, enumeration, and stack API code.
The change is not applied to built-in native functions (like Math.cos) because their inheritance chain is defined in the Ecmascript specifications and they have concrete .name and .length in any case.
Tasks:

[x] Fix genbuiltins.py to allow a .length accessor
[ ] Core changes throughout
[x] Remove DUK_HOBJECT_FLAG_DUKFUNC
[ ] %NativeFunctionPrototype% code comment consistency
[ ] Duktape/C function automatic .name change?
[ ] FIXME cleanups
[ ] Testcase updates
[ ] Check #1493
[ ] Check #1494
[ ] Release notes migration entry
[ ] Releases entry

At this point this is rough prototyping of what the change would mean and if there are any non-obvious pros/cons.

One interesting opportunity here would be to give Duktape/C functions which don't have an explicit .name property some automatically generated name, like for lightfuncs:
duk> Math.sin.name
= "light_0805ec91_0a11"

So maybe it could be something like:
duk> print.name
= "native_0805cafe_0000"

The _0000 prefix would be the 16-bit magic value which is in some sense part of the function identity (as for lightfuncs).
The automatic name wouldn't prevent establishing an actual name, it would just default to something more useful than an empty string.
Not yet sure if this would be a good change. Common expectation is that anonymous functions have an empty string as their name and so far Duktape/C functions are anonymous by default.

Concrete example, print() doesn't have a name, and in 2.1.0:
duk> print.name
= ""
duk> print({ toString: function mythrower() { throw new Error('aiee'); } })
Error: aiee
    at mythrower (input:1) preventsyield
    at [anon] () native strict preventsyield
    at global (input:1) preventsyield

With an automatic .name:
duk> print.name
"native_00000000004d4164_ffff_0000"/1
= "native_00000000004d4164_ffff_0000"
duk> print({ toString: function mythrower() { throw new Error('aiee'); } })
Error: aiee
    at mythrower (input:1) preventsyield
    at native_00000000004d4164_ffff_0000 () native strict preventsyield
    at global (input:1) preventsyield

Naturally one would prefer to see "print" in the traceback, but if no name exists (as for print), maybe an automatic name would be better. Then again, it looks quite technical and unless you have a symbol file, it's not very useful. It does identify particular functions even without a symbol file, i.e. you can see whether two functions in two different tracebacks are the same or not.

Can't we just add and then encourage to use such a little helper function, which sets the name when it's still known from context? Like:
int duk_register_function(duk_context *ctx, char *name, int (*func)(duk_context *ctx)) {
	duk_push_c_function(ctx, func, DUK_VARARGS);
	duk_push_string(ctx, name);
	duk_put_prop_string(ctx, -2, "name");
	duk_put_global_string(ctx, name);
	return 1;
}

So every function has the name it's "supposed" to have. And the other ones could have such a technical name, so they are distinctive in tracebacks.

@kungfooman Adding such an API would be a useful improvement and there are issues proposing that. But the question remains what to do with functions that don't have a name which will still remain for a few reasons: existing code using duk_push_c_function() will create such functions, and sometimes that's desirable to minimize memory usage on memory constrained devices. Lightfuncs which don't have any property slots are also used in such environments and they have a virtual .name for similar reasons.

I'm inclined to say that, by the principle of least surprise, non-lightfunc Duktape/C functions without a .name should remain anonymous (= empty string), as with anonymous JS functions.  I see the automatic naming for lightfuncs as a necessary evil because they physically can't have a name, but no such restriction exists for regular funcs.
On the other hand, I can see the value for low-memory targets where RAM is tight.  So maybe it could be a compile-time option?

@fatcerberus It could be an option, sure.
I'll leave it out of this pull (some of the code will be #if 0'd because I already wrote it :) so it can be discussed separately.

@fatcerberus And to be clear, I see your point, and I agree some code might expect functions to genuinely be anonymous. I'm not sure what the use case is in practice, but it may still be surprising if there's an automatic name.

duk> Object.defineProperty(print.__proto__, 'name', { get: function() { var i = Duktape.info(this); return 'native_' + i.hptr; } })
= {}
duk> print.name
= "native_0x6cd790"

Hm, using the hfunc pointer as a default name might not be too useful for debugging as it will probably change from run to run.  You wouldn't be able to create a meaningful symbol file.

I agree, just copy-pasted an example of how to override the %NativeFunctionPrototype%'s name for a later test case. It wasn't intended to be an actual use case.

@fatcerberus Here's a more realistic dynamic .name example, based on a hypothetical runtime C function pointer -> name lookup: https://github.com/svaarala/duktape/blob/0db47637c238f42148ba4d14e2bed6d774208735/tests/api/test-dev-override-duktapec-func-getters.c.

This change is somewhat borderline with respect to versioning guarantees. In my view it's OK because it's similar to ES2015 inheritance chain changes which are also made in minor version bumps. The change is mentioned separately in 2.2 migration notes just in case.


Broken AndroidManifest.xml

The MainActivity is mentioned as MainActivity_ in the manifest file.

MainActivity_ is generated dynamtically by AndroidAnnotations during the compiling

Even my Android Studio says the same thing for every alternate build, anyway closing the issue.


Fix unable to reset FieldACL with empty Field ACL

Without this change, SetRecordFieldAccess will result in panic because
the function is trying to INSERT without any data.

@cheungpat, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ben181231 to be a potential reviewer.

Picked as dcff59a


Packagist versions seem incorrect

If you access the Packagist page (https://packagist.org/packages/twbs/bootstrap) you'll see that many of the versions are no longer present.
When running composer with:
        "twbs/bootstrap": "3.0.0",

This error happens:
  Problem 1
    - The requested package twbs/bootstrap could not be found in any version, there may be a typo in the package name.

Which prevents composer from running successfully.
It seems someone (with access) may need to manually update the Packagist information using the update button on their page.

+1 there is only v4 dev versions
for v3 : you can temporarily specify the repository in your composer.json
"repositories": [
{
"type": "git",
"url": "https://github.com/twbs/bootstrap.git"
}

Same here. "twbs/bootstrap" : "3.3.6" was available just a few days ago, now it isn't...

+1 same as @johnmadrak

+1 same here

Strange, we haven't changed anything on our end that would account for this.

Mashed the Update button on Packagist; all the v3 versions are on the Packagist page now.

Perhaps we were a victim of https://github.com/composer/packagist/issues/610

@cvrebert Thanks for resolving this issue. Yes, it seems like that may have been the cause of this. Hopefully they'll come up with a solution to prevent this kind of issue in the future.


fix(i18n): allow for values to be overriden

Adds the notion of an "Overridable" to I18N. From the comments:

Largely an internal usecase. There are situations where we want an Observable that can emit events from a centralized source OR an Observable that will emit events from a component local source. The key example being on/off text in a Toggle - In some cases we want the Toggle to use I18ns global translations, but in others we'd prefer to use a local override. We don't ever need to return to an non-overriden state, but we do need the ability to switch to an overridden sate.

Thanks for the spell check 🙈


Alternative source dirs of dependencies are not picked up by remappings

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[ ] Foundryup

What version of Foundry are you on?
forge 0.2.0 (a152664 2022-05-04T17:45:52.953502+00:00)
What command(s) is the bug in?
No response
Operating System
macOS (M1)
Describe the bug
With the following structure:
.
├── foundry.toml
├── lib
│   ├── dep
│   │   ├── foundry.toml
│   │   ├── src
│   │   │   └── dep.sol
├── src
│   └── try2.sol

Running forge config gives:
# (...)
remappings = [
    'dep/=lib/dep/src/',
    'ds-test/=lib/forge-std/lib/ds-test/src/',
    'forge-std/=lib/forge-std/src/',
]

However, if in ./lib/dep/foundry.toml I replace src = 'src' with src = 'src2', running forge config gives:
# (...)
remappings = [
    'dep/=lib/dep/src/',
    'ds-test/=lib/forge-std/lib/ds-test/src/',
    'forge-std/=lib/forge-std/src/',
    'src2/=lib/dep/src2/',
]

Which means that I cannot use import "dep/dep.sol";

I see, thanks for flagging


integration into official OpenPCDet repository

Hi Weichen,
congratulations on your awesome work! 🥳
I really enjoyed reading your paper 📄 and was glad to see that you used OpenPCDet for your implementations.
In my opinion, it would be awesome if your work could be merged into the official OpenPCDet repository.
@zhangweichen2006: What do you think?
Greets, Martin

Thanks for your suggestion! Unfortunately, we are not planing to integrate our code at the moment as the version compatibility will be a bit messy. Sorry but thanks again for your interests!


Parser File chnages

Hello,
I believe there are three lines in the parser file causes the data connector to fail.  This is the file that is added by the setup script
There is a blank line at line 1
Both project-rename lines don't seem to be able to be resolved.
ThreatName=threat_s
EventSubType=origin_s
Thanks,

@IH2by2 : working by phone, so might have missed something,  but i cant see which file you refer to.
Thanks for raising the issue!
~ Ofer

Hi Ofer no worries,
the main file is https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/SophosEP/Parsers/SophosEPEvent.txt.  I'm guessing there is some code that copies and saves it as a function and this is where a blank line is added to the top.
Martin

Hi @IH2by2 ,
I am in charge of ASIM, which does include many parsers, but not this one, so reassigned. Sorry for the added delay.
@sarah-yo : FYI

Hi @IH2by2,
Sorry for the delay. we have started working on the issues recently.
I have verified the parser and didn't see any issue; seems the issue is resolved by now.
Can you please confirm if you still see issue and need any help from us.

Hi @IH2by2,
Closing the issue as its fixed. Please feel free to reopen if you still facing the issue.
Thanks


How to cast from Widget to a specific Widget type?

Hi,
I am trying to access a specific Widget from an event.
The Widget is dynamic. That is, it appears and disappears.
fn update(vpack: &mut Pack) {
   // I am checking the number of elements in the pack because
   // fltk-rs does not allow me to identify the element by name and/or type.
   if pack.children() == 5  {
      if let Some(mut widget) = vpack.child(4) {
         // widget is a Pack
         widget.add();   // No method named `add` found...
      }
   } else {
      ...
   }
}

You can try widget.as_group().unwrap().add();
casting widgets using from_widget() is currently unsafe because the vtable can’t be verified.

Thank you. It works:
...
   if let Some(c) = vpack.child(4) {
      c.as_group().and_then(|mut g| Some(g.add(&status)));
   }
...


Decode exif from url image

Is it possible?

I'm not sure what you're asking here.  Could you describe in more detail what you are trying to do?


Adding AbstractBlocklyFragment and demo SimpleFragment

Also clarifying comments about the save path overrides.

This change is

Review status: 0 of 14 files reviewed at latest revision, 3 unresolved discussions.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 1 at r1 (raw file):
Previously, RoboErikG wrote…
copyright

Done.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 32 at r1 (raw file):
Previously, RoboErikG wrote…
Missing class doc

Done.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 83 at r1 (raw file):
Previously, RoboErikG wrote…
add "using {@link #getWorkspaceSavePath()}" to match comment above.

Done. Here and in AbstractBlocklyActivity.

Comments from Reviewable

Review status: 0 of 14 files reviewed at latest revision, 6 unresolved discussions.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 66 at r2 (raw file):

 * bundle.  If no workspace is loaded, it defers to {@link #onLoadInitialWorkspace}.
 * <p/>
 * Configure the workspace by providing definitions for {@link #getBlockDefinitionsJsonPaths()},

nit: replace comma with ' and'

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 250 at r2 (raw file):

    }

    public boolean onOptionsItemSelected(MenuItem item) {

missing method javadoc

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 256 at r2 (raw file):

        int id = item.getItemId();

        if (id == R.id.action_save) {

Should this be a switch instead of an if/else?

Comments from Reviewable

Review status: 0 of 14 files reviewed at latest revision, 6 unresolved discussions.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 66 at r2 (raw file):
Previously, RoboErikG wrote…
nit: replace comma with ' and'

Fixed here and in AbstractBlocklyActivity

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 250 at r2 (raw file):
Previously, RoboErikG wrote…
missing method javadoc

Done.

blocklylib-core/src/main/java/com/google/blockly/android/AbstractBlocklyFragment.java, line 256 at r2 (raw file):
Previously, RoboErikG wrote…
Should this be a switch instead of an if/else?

"Resource IDs cannot be used in switch statement in Android library modules."

Comments from Reviewable


Error with<EMAIL_ADDRESS>

Issue summary
When i try to parser this domains :+1:
https://validator.w3.org/#validate_by_input
<EMAIL_ADDRESS><?php
$publicSuffixList = Rules::fromPath('geo/public_suffix_list.dat');
$domain = Domain::fromIDNA2008($_REQUEST['domain']);
$domainResolve = $publicSuffixList->resolve($domain);

echo $domainResolve->registrableDomain()->toString();

Fatal error: Uncaught Pdp\SyntaxError:

The host<EMAIL_ADDRESS>is invalid: it contains invalid characters.

in /home/blok/composer/vendor/jeremykendall/php-domain-parser/src/SyntaxError.php:21

Stack trace: #0 /home/blok/composer/vendor/jeremykendall/php-domain-parser/src/Domain.php(149):Pdp\SyntaxError::dueToInvalidCharacters('login:td3dt1...')

#1 /home/blok/composer/vendor/jeremykendall/php-domain-parser/src/Domain.php(103): Pdp\Domain->parseValue('name:td3dt1...')
#2 /home/blok/composer/vendor/jeremykendall/php-domain-parser/src/Domain.php(62): Pdp\Domain->parseDomain('name:td3dt1...')
#3 /home/blok/composer/vendor/jeremykendall/php-domain-parser/src/Domain.php(87): Pdp\Domain->__construct('IDNA_2008', 'name:td3dt1...')
#4 /home/blok/www/projets/dns.php(19): Pdp\Domain::fromIDNA2008('name:td3dt1...')
#5 {main} thrown in /home/blok/composer/vendor/jeremykendall/php-domain-parser/src/SyntaxError.php on line 21

@Machou thanks for using the library.
I believe the error is self explanatory<EMAIL_ADDRESS>is not a valid host. The parser except to receive a Host not an authoriyt component like the one reported by the exception message.
so instead of submitting<EMAIL_ADDRESS>you should have submitted login.domain.io without the user-info component of your URI.
You should use PHP's built-in function parse_url or a dedicated URI parsing package found on packagist for instance if you want to correctly extract the host part of you URI.
Hopes this answer your question.

I thought the bookstore could clean itself, I misunderstood =)


Check Signature Algorithms are available before offering or accepting them

Different providers may support different signature and hash schemes, and we cannot guarantee that the default provider is always loaded. Therefore we should check our signature algorithms to ensure that the providers we have loaded do actually support them before we offer or accept them.
For now we "cheat" a little bit. We check that the hash is available, and we check that the signature scheme is available. However we don't check that the combination is available (e.g. imagine a provider that only supports RSA with SHA256, even though other digests are available from other providers). That is quite a bit harder to solve.
This is quite tricky to test because it requires the ability to switch on and off the availability of some algorithms. In order to do this I have implemented a "filter" provider within the test suite that provides this functionality. It essentially sits in front of the default provider and enables you to configure which algorithms are made available. This required the addition of a couple of new OSSL_PROVIDER_* functions.
This is marked as WIP because it includes the commits  from #11826.

Rebased now that #11826 was merged. I've also taken this out of WIP.
This is ready for review. Please take a look.

Ping

All feedback addressed. Please take another look.

This pull request is ready to merge

Pushed. Thanks.


👯 Add Concurrency slides and samples

Changes proposed in this PR

Add Concurrency slides and samples
Add licence note to README.md
Update pingserver to work with Seconds rather than Millis

I've deliberately left out the lab. I will add it in at the very end (if there is time) when CRUD-puppy has evolved to a proper restful API running as a server.
View changes at gotraining.xyz

PTAL, thanks for still hanging in there with me @camh-anz !!!!1!

I think I pushed a few more cosmetic changes along with this update - not sure now. (renaming single digit lab names to double digits etc.) sorry, poor form.

updated the 2 sec to 2000 millis in in next pr. sorry.


evaluator: error with disjunction containing default value is needlessly verbose

What version of CUE are you using (cue version)?

$ cue version
cue version v0.0.0-20220901114913-d2250b9bc187

       -compiler gc
     CGO_ENABLED 1
          GOARCH amd64
            GOOS linux
         GOAMD64 v1
             vcs git
    vcs.revision d2250b9bc1870a2488ee3f33b4e8e4fd0561a5f3
        vcs.time 2022-09-01T11:49:13Z
    vcs.modified false

What did you do?
! exec cue vet x.cue
cmp stderr stderr.golden

-- x.cue --
// This would usually be in a schema
x: >50

// This might be in a template or somewhere that defines defaults.
x: *1 | _

// This represents the user data
x: 4

-- stderr.golden --
x: 2 errors in empty disjunction:
x: conflicting values 1 and 4:
    ./x.cue:5:5
    ./x.cue:8:4
x: invalid value 4 (out of bound >50):
    ./x.cue:2:4
    ./x.cue:8:4

The output in that test is more verbose than it could be. Although technically correct, the fact that the default value conflicts with another value and is thus eliminated from the disjunction is not useful information and could probably be omitted.
It would be perhaps less confusing to see a single error like this instead:
x: invalid value 4 (out of bound >50):
    ./x.cue:2:4
    ./x.cue:8:4

https://docs.google.com/document/d/1IlXNW32-tUET56mjm5htF7LLfDIsbiC4zk78cWGvfrA/edit


Cannot use Tabnine Chat

Engine Version: 4.5.28-45
Plugin Version: 1.0.48

I've already been selected to join the Tabnine Chat's beta, but I still cannot use the Tabnine Chat.
The "chat" popup stil display "Tabnine Chat is currently in Beta"
Here is an error I've found in the idea.log, it seems that the binary the "chat" required cannot be downlaod. What should I do now?
2023-08-25 11:22:14,200 [  48726]   WARN - #com.tabnineCommon.binary.BinaryProcessRequesterImpl - Binary returned illegal response: null
com.tabnineCommon.binary.exceptions.TabNineInvalidResponseException: Binary returned illegal response: null
        at com.tabnineCommon.binary.ParsedBinaryIO.readResponse(ParsedBinaryIO.java:33)
        at com.tabnineCommon.binary.BinaryProcessRequesterImpl.readResult(BinaryProcessRequesterImpl.java:63)
        at com.tabnineCommon.binary.BinaryProcessRequesterImpl.request(BinaryProcessRequesterImpl.java:42)
        at com.tabnineCommon.binary.BinaryRequestFacade.lambda$executeRequest$0(BinaryRequestFacade.java:36)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
        at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:702)
        at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:699)
        at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
        at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1.run(Executors.java:699)
        at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: com.tabnineCommon.binary.exceptions.TabNineInvalidResponseException: Binary returned null as a response
        at com.tabnineCommon.binary.ParsedBinaryIO.lambda$readResponse$0(ParsedBinaryIO.java:30)
        at java.base/java.util.Optional.orElseThrow(Optional.java:403)
        at com.tabnineCommon.binary.ParsedBinaryIO.readResponse(ParsedBinaryIO.java:29)
        ... 11 more

gz#26039(related to Zendesk ticket #26039)

@Stark-X
Did you try to sign in (with the same email that you received access to the beta)? See more info about Tabnine Hub here:
https://support.tabnine.com/hc/en-us/articles/4413876147985-Tabnine-Hub-Where-to-find-it-and-how-to-sign-in.
What exactly doesn't work? Can you send us a screenshot/viedo of the issue?
Do you know if code completions work?

I have already logged in.
The code suggestions are functioning properly, but the "chat" displayed the information as shown in the provided screenshot.
The only distinction between my account and the Tabnine account (shown on the Tabnine Hub) is that the "Tabnine Hub" displays the account with the first letter of each word in uppercase.

Did you try to sign in (with the same email that you received access to the beta)? If so, what is the email address?

yes, i've already used my subscribed account to signed in.
Here's my account<EMAIL_ADDRESS>and the mail address displayed on Tabnine hub is<EMAIL_ADDRESS>

Please restart your IDE and check again.

It is currently functioning, thank you very much for your assistance.
By the way, is this an issue with the software or might it be due to an incorrect configuration?

Glad to hear that.


Travis CI stuck?

I noticed that my last pull request was still pending for Travis CI so I went to see what it was doing.  It looks like a build from a couple weeks ago never finished all of the configurations and is holding up any other builds from completing.  Some of the configurations have completed, but the next configuration isn't starting for some reason.
From what I can tell, builds 2099 and 2100 are both waiting to start a gcc/cmake build with coverage, but for some reason that build isn't starting.  2099 seems to be trying to build a commit that doesn't exist, so maybe 2100 is just waiting for that particular configuration to complete before it starts it's build with that configuration?  Maybe someone just needs to cancel the hung build and everything will catch back up to itself again.

Cancelled. Let me know if it is ok now?

The next build started, but looks like it's blocked again.  I think you need to cancel build 2099.

That got another build to try and run, but it still isn't running the last 3 configurations.  Maybe it's related to the conflicts between the excludes and includes in .travis.yml?  Kind of seems like it doesn't like being told to exclude a group of builds and then include a specific instance from that group later on.  Maybe this is a bug in travis-ci?

I set up travis-ci on my fork and ran a branch that's changed to be excludes only without any includes and all of the builds appear to run now.  Looks like mixing includes and excludes causes trouble.  Odd, because it doesn't appear to me at a quick glance that the travis.yml file was changed recently, so why did some older builds run successfully with that configuration?
If you want, I'll submit my change as a pull request, but it seems like this should be fixed on the travis-ci side instead.

I submitted this issue https://github.com/travis-ci/travis-ci/issues/5409.
I think it might be related to the change in infrastructure, but was unsuccessful so far in getting to run the CppUTest build on the old infrastructure ("Blue Box"), so I can't be sure. Travis CI documentation (https://gist.github.com/meatballhat/d0c8aab9e3bd8a8bcacd#file-01-matrix-include-travis-yml-L3-L6) is unclear on how to force the old infrastructure.

@Andne - did you manage to get the same run by using includes only? If so, we should use your setup. The reason we are using both include and exclude is that it seemed the only / easiest way to get the exact combination of builds we want from the matrix.
And yes, the current setup did work for several months, presumably until they changed the underlying infrastructure.

Much of the current setup was arrived at by trial and error. I find the documentation on setting up a (sparse) build matrix highly unsatisfactory.
Which is strange, because NOT setting up a build matrix with as few entries as possible is directly related to more system load, which is directly related to cost, so I am a bit suprised that the people at Travis CI don't seem to pay attention to this.

I did it by modifying the exclude block and removing the include block.  However, it'd probably be easier to read just the include block, so I think I'll set it up that way instead.

That would be great. I must have missed something there, because I never got it to only include those combinations that we need.

Well, I got the matrix to only generate those configurations on include that we wanted - had to remove the compiler, env, rvm and os sections from earlier in the file.  However, it doesn't work.  The list of builds to run is correct, but for whatever reason, they aren't starting.  Looks like travis-ci may have introduced a bug where anything defined through matrix.include can't start running for some reason.

How strange - but probably consistent with the problems we encountered lately....
So, since your "exclude" version seems to work, maybe we just just go with that for now?

Oops, includes do work, I just had a build stuck in the queue that I couldn't see.

I've seen some builds working, but right now, Travis CI appears to be "stuck" again...

...well, let's call it extremely sluggish (queued for hours) :-(

Agreed, my last build took 40 min (11 min of it are actual build time).

There was a related issue with all jobs erroring when for some reason Dosbox could not be installed.
Dosbox shouldn't be installed for any but the make_dos job.

My fork starts normal now.

Issue can be closed, I assume.


incorrect channel/speakers mapping

This bug report was migrated from our old Bugzilla tracker.
Reported in version: unspecified
Reported for operating system, platform: Windows 7, x86_64
Comments on the original bug report:
On 2019-05-31 15:26:25 +0000, AltairPL wrote:

Some time ago I tried using Mix_SetPosition() to test the idea of using rudimentary audio positioning in my engine. Simply put, wrong channels were used. Tried using both mono and stereo audio files and Mix_SetPanning() with pretty much the same result. After that I decided to make a separate test to see how playback of 5.1 ogg is handled and I noticed the same thing. Same file was played correctly in various media players and other applications. Results look like this (with test app restarted between tests):
Speakers set to 2.0/2.1:
src    SDL     should be
FL     FL      FL
FC     FR      FL/FR
FR     FL/FR   FR
BL     ?       FL
BR     FL      FR
LF     LF      LF
Speakers set to 4.0/4.1:
src    SDL     should be
FL     FL      FL
FC     FR      FL/FR
FR     FL/FR   FR
BL     ?       BL
BR     BL      BR
LF     LF      LF
Speakers set to 5.1:
src    SDL    OK
FL     FL     FL
FC     FR     FC
FR     FC     FR
BL     ?      BL
BR     BL     BR
LF     LF     LF
This test was performed using:

SDL 2.0.8 (x86)
pre-release SDL_mixer 2.0.3 (x86) "released" on discourse
(https://discourse.libsdl.org/t/sdl-mixer-2-0-3-prerelease/23959)
Windows 7 64-bit
Auzentech X-Fi Prelude 7.1 sound card
5.1 speaker set
5.1 ogg file from
https://people.xiph.org/~xiphmont/demo/surround/demo.html - for some reason SDL_mixer refused to play 5.1 wav file

Had no chance to test it in SDL_mixer 2.0.4, but I haven't seen any mention of it in changelog, so I assume it's still the case.

@icculus, possibly related to your other planned multi-channel work.


Create a plain text file that lists all drivers/helpers for guide-embedding

rather than try to keep a learn guide updated, we can create a file that we then embed, which will simply list all the drivers, maybe with a single line of text describing it (from the driver repo itself)

I think this was solved with having this file updated as new libraries are added to the bundle. Doesn't include a description, though.

@sommersoft ooh an oldie but goodie - yes we can now embed .md files in learn!  is that file generated by adabot?


Inappropriate ioctl at device at container startup

I'm seeing this message under certain startup conditions.
bash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shell

Here is the Dockerfile and our entrypoint script.
The issue may very well be ours, but I'd like to get a better idea of what is going on at least and how I might approach resolving this issue (if it merits being resolved). Any help would be greatly appreciated.

Hey @jakirkham,
I think this only happens if you're running a container without allocating a TTY (i.e. without the -t flag to Docker run).
That being said, I'm not 100% sure this issue is related to Tini. I cloned your repo, and built your image, except I removed Tini from equation, and I'm still seeing the error:
homas@mocha4 /private/tmp/docker-images/linux-anvil % docker build -t foo . && docker run --rm -i foo /opt/docker/bin/entrypoint bash                         ✘ master
Sending build context to Docker daemon 27.14 kB
Step 1 : FROM centos:6
 ---> a3c09d36ab4a
Step 2 : MAINTAINER conda-forge<EMAIL_ADDRESS> ---> Using cache
 ---> 5506c4179797
Step 3 : ENV LANG en_US.UTF-8
 ---> Using cache
 ---> d02b4f55befc
Step 4 : RUN rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-6
 ---> Using cache
 ---> 80d029555040
Step 5 : RUN yum update -y &&     yum install -y                    bzip2                    gcc-c++                    make                    patch                    tar                    which                    libXext-devel                    libXrender-devel                    libSM-devel                    libX11-devel                    mesa-libGL-devel &&     yum clean all
 ---> Using cache
 ---> 67719a5fcebf
Step 6 : RUN yum update -y &&     yum install -y                    centos-release-scl                    yum-utils &&     yum-config-manager --add-repo http://people.centos.org/tru/devtools-2/devtools-2.repo &&     yum update -y &&     yum install -y                    devtoolset-2-binutils                    devtoolset-2-gcc                    devtoolset-2-gcc-c++ &&     yum clean all
 ---> Using cache
 ---> c9e1548f69d5
Step 7 : RUN curl -s -L -O https://github.com/krallin/tini/releases/download/v0.10.0/tini &&     openssl md5 tini | grep 07b74be7c14bae738afff855eb24e0d9 &&     chmod +x tini &&     mv tini /usr/local/bin
 ---> Using cache
 ---> 9d42727eb3bb
Step 8 : RUN curl -s -L -O https://repo.continuum.io/miniconda/Miniconda3-latest-Linux-x86_64.sh &&     bash Miniconda3-latest-Linux-x86_64.sh -b -p /opt/conda && rm Miniconda*.sh &&     export PATH=/opt/conda/bin:$PATH &&     conda config --set show_channel_urls True &&     conda config --add channels conda-forge &&     conda update --all --yes &&     conda clean -tipy
 ---> Using cache
 ---> ffd3aaa8df0d
Step 9 : RUN export PATH="/opt/conda/bin:${PATH}" &&     conda install --yes obvious-ci &&     obvci_install_conda_build_tools.py &&     conda clean -tipsy
 ---> Using cache
 ---> 27bf33077d92
Step 10 : RUN export PATH="/opt/conda/bin:${PATH}" &&     conda install --yes git &&     conda clean -tipsy
 ---> Using cache
 ---> 83e0f9b5edaa
Step 11 : COPY entrypoint_source /opt/docker/bin/entrypoint_source
 ---> Using cache
 ---> b75d77a704af
Step 12 : COPY entrypoint /opt/docker/bin/entrypoint
 ---> Using cache
 ---> d36527882950
Step 13 : COPY tini /usr/local/bin/tini
 ---> Using cache
 ---> 8c1bec749176
Step 14 : CMD /bin/bash
 ---> Using cache
 ---> aa2646dc324e
Successfully built aa2646dc324e
bash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shell
ps auxwww
USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.3  0.2  12896  2776 ?        Ss   18:40   0:00 bash
root        52  0.0  0.2  14912  2192 ?        R    18:40   0:00 ps auxwww

Tini does do a few things with the TTY (namely creating a process group for the child and attempting to allocate the TTY to the child), but it looks like they're unrelated here?
Let me know if you have reason to believe Tini is part of the problem though. I'm happy to take another look.
(do note that this is just a warning from bash; things — with the exception of e.g. sending CTRL + Z, which would obviously require a tty — should still work)
Cheers,

Hey @jakirkham; did you still need any help here?

Thanks for the info/experimentation. Sorry haven't had a chance to take a closer look of late. Trying to do an upgrade of our build tools. Should be able to cycle back to this in a week or two.

Ya, this is not a tini issue.  Docker build does not allocate a TTY for the build containers.  If you are not seeing this issue during a build, just add -t or --tty to your docker run and it should remove the error message regardless of using tini or not.


Upgrade pannel 1.0.3 to 1.1.1

Execute the script, select upgrade from 1.0 to 1.1.1
Script executes: Your panel has successfully been updated to version 1.1.1
Panel shows: Your panel is not up-to-date! The latest version is 1.1.1 and you are currently running version 1.0.3.
(tried restarting the server as well - same result).
No errors are thrown (aside from the usual question to not run composer as root).
I also had a quick look at the script and it looks like it downloads the correct file.
Any idea of why it does not work is appreciated.
Thanks

Done :+1:


Docker Updates

the new builds for MakeMKV and MKVToolnix did not push to dockerhub

Builds are not pushed automatically for each commit.  I've created a release for both of them.  Docker images are currently building.

i also noticed MakeMKV is using a older Base Image them MKVToolnix

Yes I need to do another update on the baseimage and then I will update all my docker images.


Cannot specify simulation parameters with ASE-OMM CLI

The ase-omm CLI command takes (requires?) as an input a NanoVer-OpenMM simulation xml, however, this runner does not actually read the Integrator field from this xml. This means that you can't specify dynamics parameters (e.g. temperature, timestep) from the command line and these parameters are actually set to some default values. It seems misleading to provide this CLI functionality if these parameters are ignored, and can't actually be changed from the default parameters using this interface (this can only be done from a python script).
It seems like the only use case would be if (a) we provide a way of writing a NanoVer-ASE-OMM xml that specifies parameters that are read the by the ase-omm runner AND (b) these parameters differ somehow from the available OpenMM parameters, otherwise you could just use the standard OpenMM runner.
Maybe we should consider deprecating this CLI for the moment and think about if it's worth doing the above.
If any of this information is incorrect or anything vital is missing, please do say!

In a recent merge both nanover-ase-omm and nanover-omm-server were replaced to just tell you to use nanover-omni --omm and nanover-omni --ase-omm. There's no CLI arguments to do anything with the integrator here either though. I don't think the runners even offer any explicit options for this other than letting you access the integrator yourself to change values.
I agree that it probably doesn't make sense to support ase by CLI since it doesn't fully follow the input XMLs and the main motivation for ASE is having deeper access in python to dynamics (which you obviously don't have on the CLI). Only other thing I can think of is that ASE supports the boundary walls and regular OMM doesn't? Is that right?
The generic CLI even for nanover-omni will always be limited and maybe it's only really fit for demos vs. say launching it from a bespoke python script or jupyter notebook where you can set up exactly what you want. Maybe it would be good for us to figure out exactly what use cases for CLI we want to support vs what ought to be done in custom notebooks or setup scripts?


[W03] Added Artemis example for Dynamic Array

Checklist
General

[x] I locally pass all tests or calculated manually the correct solutions on paper.
[x] I chose a title conforming to the naming conventions for pull requests.
[x] The newly added tests do not reveal the solutions dynamically calculating the solutions as expected in the exercise.

Description

Review Progress

Review

[ ] Review 1
[ ] Optional Review 2

Sorry for the mess. Wanted to add the ExceptionTest but Github was down and produced some bugs in my commit history

There is at least no Test for the IllegalArgumentException

@mbmueller Thank you very much. I missed that, now it's implemented in my Tests.


Update ipxedust version:

Description

This resolves a regression in the previous version that could cause come hardware to panic or fail to DHCP properly when loading the iPXE binary.
Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @nshalman , i had to fix an issue and your approval was auto removed. thank you for responding so quickly! Mind checking this out again, please?

@mergify queue


Migrate Catch2 from v2 to v3

Right now, compiling a single unit test on my laptop takes 6 seconds or more.  Switching to v3, with a statically compiled library, should speed this up.

This is resolved by 7951b04


Diff between version 1 & 2 of Irving & Holden

@bgcarlisle, any chance you made a diff for v1-2 like v2-3.pdf? These diffs are super useful for untangling the evolution of Irving & Holden's incompetence / misconduct.

Yeah, pull request sent!


Initial version of Microsoft.WindowsAzure.Mobile.Service.DynamicsCrm

Merging into repo. I'll review the code later and provide my comments. Thanks!


[dv/unr] Blackbox common security modules from UNR flow

This PR blackbox common security prim modules: prim_count,
prim_double_lfsr, prim_sparse_fsm_flop from the UNR flow.
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

@sriyerg just wanted to check if you have any comments on this PR?


🧹 [CHORE]: FE - check HTML for any leftover tailwind classes

Summary
There may be some tailwind classes left behind in .svelte and .html files and components. Check SVG as well.
Tasks

[x] check for class=" instances and review classnames
[x] check for conditional use of classnames and instances of tailwind visibility helpers, etc being used
[x] vis reg
[x] update as needed
[x] vis reg

Closed with #945


Trouble with node exporters in rasperrypies with public ip

I am setting up a cluster with public ip. The k3s master is a cloud vm and I have two rpis as workers all of them they have public ip, so they are not in the same LAN. The node exporters in the rpis have a down status and they show a context deadline exceeded error. I cannot find what's going wrong. The weird fact is that the cadvisors of rpis have up statuses, and all the required ports are open.

Prometheus scrapes node_exporter from all nodes using internal IPs from the node_exporter pods/services.
When you say "down status" you mean in Prometheus Targets?

@carlosedp yes exactly

First of all, I'd advise against having your nodes exposed to the internet with valid IPs, but it shouldn't affect the usage.
We'd need more information:

Is there any firewall between these LANs? Do they allow the traffic between Prometheus and the node_exporter port (9100)?
Do you have any log from node_exporter? Is it running correctly (both containers, node_exporter and kube-rbac-proxy)?
Can you try to curl -k one of the IPs in port 9100? It will fail since there is no authentication but that's fine.

When I am doing curl from the host I am getting unauthorized message, if I perform it from a other device I am getting timed out message. I will try to find answers to the other two questions and I will post them here, thank you @carlosedp

Apparently there is something blocking the access between the hosts on the needed port since curl didn't work from outside.

In the ufw of the devices I have allowed the required ports. These are the logs for node-exporter and kube-rbac-proxy:

I changed the port of the node-exporters and now it works !!! @carlosedp thank you for your time :)


CBAPI-2200: guide page for device control, inside new "guides and resources"

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Tests have been added that prove the fix is effective or that the feature works.
[ ] New and existing tests pass locally with the changes.
[x] Code follows the style guidelines of this project (PEP8, clean code, linter).
[x] A self-review of the code has been done.

What is the ticket or issue number?

https://jira.carbonblack.local/browse/CBAPI-2200
Pull Request Description

Added the initial guide page for device control, within the framework of "Guides and Resources" as sketched out by Kylie and Lisa.
Does this introduce a breaking change?

[ ] Yes
[x] No

How Has This Been Tested?

Documentation only, no new code added.

This is great! I'll mimic this model to create my example for reputation overrides

Yeah, just link it in the "Guides" section of guides-and-resources.rst. Kylie marked a place for it in her Miro doc.

This is great! I'll mimic this model to create my example for reputation overrides

Yeah, just link it in the "Guides" section of guides-and-resources.rst. Kylie marked a place for it in her Miro doc.


not adapt for iPad

I am experiencing the same issue on the iPad.  Its particularly bad on an 12" iPad Pro.  I've spend the afternoon poking around the code trying to figure out why the cell widths are not properly expanding.
If someone can give me a clue where to look, I'll have another go.

Any news about this issue?

No, nothing.  In the end I chose to move to another form library.

@alldritt ah okay. which one do you use now?

Eureka

create ViewController (UIViewController NOT FormerViewController) on Storyboard
add UITableVIew as @IBOutlet weak var tableView: UITableView!
 lazy var former: Former = Former(tableView: self.tableView)
Profit!

Closing due to inactivity and solution posted by @ADevelopStudio


Make builtin types lazy

So it is a bit strange that I can't create a type called Node when using garph because the library defines a type with the name.
I understand that this goes against relay conventions, and I can use a different name, but ideally being a generic library, garph should not prevent me from doing something that graphql itself allows.
I propose that we make the builtin types like nodeType, pageInfoType lazy so that they get defined if the consumer actually uses them.
  get nodeType() {
    this._nodeType ??= this.interface('Node', {
      id: this.id()
    })
    return this._nodeType
  }

With 9de43dd I have made it possible to overwrite built-in types

Ok, that's a simpler solution 👍🏾


xpdf: 4.02 → 4.03

Motivation for this change

Changelog:

$ cat ANNOUNCE | sed -n '13,24p'
4.03 is primarily a bug fix release.  There are some new features:

* XpdfReader improvements:
  - Implemented selection extension via shift-click, and word/line
    selection via double/triple click.
  - Added default bindings for ctrl-mousewheel-up/down to zoom in/out.
  - Added a help menu item that shows all of the key bindings.

* Various new command line options for pdftotext, pdftohtml, pdftoppm,
  pdftopng, and xpdf.

See the `CHANGES' file for a complete list of changes.

replace download links from xpdfreader-dl.s3.amazonaws.com and ftp.foolabs.com to dl.xpdfreader.com
fix wrapping on darwin
fix licenses
update libxpdf

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112587 run on x86_64-darwin 1

2 packages marked as broken and skipped:

invoice2data
xpdf

2 packages built:

libxpdf
pdf2xml

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112587 run on x86_64-linux 1

3 packages marked as broken and skipped:

invoice2data
pdfdiff
xpdf

2 packages built:

libxpdf
pdf2xml

The meta for xpdf in 4.02 lists a bunch of known security vulnerabilities.  Are these still relevant with 4.03?

The meta for xpdf in 4.02 lists a bunch of known security vulnerabilities. Are these still relevant with 4.03?

Yes, at least there is no official update in Xpdf Security Fixes page.