Fix SQL Lab tables' preview

~Based on #244.~
Closes #245.
This fixes getting the table preview in SQL Lab for SparkSQL engine.

Could you help me to understand why preview works for non-gitbase tables in spark?

can this change break it?

also you need to re-format the code to follow flake8 rules.

mhmmm interesting, what tables are there? Is the screen coming from demo instance? Does the demo instance already include the SparkSQL engine spec?

It has some tables from postgresql, some tables created in spark itself and some from gitbase. Looks like only gitbase tables preview doesn't work. I'm not sure where this particular table is coming from, postgresql or spark.
The screen is from demo instance.
It does not include SparkSQL engine spec.

Do you have an easy way to create a table in Spark? I locally rested this and previews for tables of gitbase db (that uses sparksql) and previews for pg tables work. But I didn't test it for tables of gitbase db different from the default ones.
Most probably in the case in this specif case those tables have a schema that fully qualifies them correctly. For the other tables superset uses the default schema which is incorrect.

Never tried but something like CREATE TABLE should work most probably.
By locally tested pg tables do you mean pg tables registered in gsc? It's a bit unclear from your comment.

By locally tested pg tables do you mean pg tables registered in gsc? It's a bit unclear from your comment.

Nope, sorry. Only pg tables in main database. I didn't tested on any table except from those available by default (commits, refs, etc.).

let's test with manually registered tables from pg and manually created tables in spark and make sure this change doesn't break them. (or maybe discover another solution for the issue during the process)

@smacker I tested this again with:

gitbase view,
pg table registered on spark,
spark table created from SQL Lab.

The preview works for all of them 🎉.

gitbase view ([view] commits)

pg table registered on spark (I just registered the slices db)

spark table created from SQL Lab (I created a simple table with 3 int cols where I added 3 rows)

In case you missed it, flake8 test is failing on travis

@carlosms Thanks, I forgot about that. BTW, travis doesn't seem to notice the changes, I'm gonna close and reopen this.


Windows tokio local socket will never complete write unless the message contains \n

Describe the bug
When using the tokio local socket on Windows you cannot complete a write to the stream unless the message you're writing contains \n
To Reproduce
Apply this diff to examples/tokio_local_socket
diff --git a/examples/tokio_local_socket/client.rs b/examples/tokio_local_socket/client.rs
index c2ce888..0ca076d 100644
--- a/examples/tokio_local_socket/client.rs
+++ b/examples/tokio_local_socket/client.rs
@@ -31,7 +31,7 @@ pub async fn main() -> anyhow::Result<()> {
     let mut buffer = String::with_capacity(128);

     // Describe the write operation as writing our whole string.
-    let write = writer.write_all(b"Hello from client!\n");
+    let write = writer.write_all(b"1");
     // Describe the read operation as reading until a newline into our buffer.
     let read = reader.read_line(&mut buffer);

diff --git a/examples/tokio_local_socket/server.rs b/examples/tokio_local_socket/server.rs
index 0031236..7d2412e 100644
--- a/examples/tokio_local_socket/server.rs
+++ b/examples/tokio_local_socket/server.rs
@@ -22,7 +22,7 @@ pub async fn main(notify: Sender<()>) -> anyhow::Result<()> {
         let mut buffer = String::with_capacity(128);

         // Describe the write operation as writing our whole message.
-        let write = writer.write_all(b"Hello from server!\n");
+        let write = writer.write_all(b"1");
         // Describe the read operation as reading into our big buffer.
         let read = reader.read_line(&mut buffer);

Now run the example with cargo run --example tokio_local_socket --features tokio_support
Expected behavior
I expect that I don't have to include \n in the contents I want to write.
Screenshots
The program never progresses past this point

Additional context
I don't want to write text to this socket, I want to write my own binary protocol.

This is probably not correctly diagnosed. Closing for now.


Fix NULL in table-datanodes when node is down

Fix possible null-warning 'Requested unknown parameter' in WEB UI when one of datanodes is down

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
40
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ trunk Compile Tests _

+1
mvninstall
1078
trunk passed

+1
shadedclient
861
branch has no errors when building and testing our client artifacts.

-0
patch
888
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1
mvninstall
65
the patch passed

-1
jshint
217
The patch generated 2585 new + 0 unchanged - 0 fixed = 2585 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
733
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
asflicense
25
The patch does not generate ASF License warnings.

2954

Subsystem
Report/Notes

Docker
Client=18.09.8 Server=18.09.8 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-342/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/342

Optional Tests
dupname asflicense shadedclient jshint

uname
Linux 522bde3e541a 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 7f1b76c

jshint
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/1/artifact/out/diff-patch-jshint.txt

Max. process+thread count
412 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/1/console

versions
git=2.7.4 maven=3.3.9 jshint=2.10.2

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
38
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ trunk Compile Tests _

+1
mvninstall
1025
trunk passed

+1
shadedclient
780
branch has no errors when building and testing our client artifacts.

-0
patch
801
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1
mvninstall
59
the patch passed

-1
jshint
213
The patch generated 2585 new + 0 unchanged - 0 fixed = 2585 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
702
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
asflicense
25
The patch does not generate ASF License warnings.

2773

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-342/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/342

Optional Tests
dupname asflicense shadedclient jshint

uname
Linux a6baf61346d4 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c01e137

jshint
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/2/artifact/out/diff-patch-jshint.txt

Max. process+thread count
446 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/2/console

versions
git=2.7.4 maven=3.3.9 jshint=2.10.2

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
42
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ trunk Compile Tests _

+1
mvninstall
1041
trunk passed

+1
shadedclient
862
branch has no errors when building and testing our client artifacts.

-0
patch
883
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1
mvninstall
58
the patch passed

-1
jshint
213
The patch generated 2585 new + 0 unchanged - 0 fixed = 2585 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
676
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
asflicense
24
The patch does not generate ASF License warnings.

2858

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-342/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/342

Optional Tests
dupname asflicense shadedclient jshint

uname
Linux b1d6c34bf89e 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / e872ceb

jshint
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/3/artifact/out/diff-patch-jshint.txt

Max. process+thread count
446 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/3/console

versions
git=2.7.4 maven=3.3.9 jshint=2.10.2

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
81
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ trunk Compile Tests _

+1
mvninstall
1257
trunk passed

+1
shadedclient
978
branch has no errors when building and testing our client artifacts.

-0
patch
1004
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1
mvninstall
70
the patch passed

-1
jshint
255
The patch generated 2585 new + 0 unchanged - 0 fixed = 2585 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
808
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
asflicense
31
The patch does not generate ASF License warnings.

3406

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-342/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/342

Optional Tests
dupname asflicense shadedclient jshint

uname
Linux 9cd24e2dde22 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 00b5a27

jshint
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/4/artifact/out/diff-patch-jshint.txt

Max. process+thread count
306 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/4/console

versions
git=2.7.4 maven=3.3.9 jshint=2.10.2

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
35
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ trunk Compile Tests _

+1
mvninstall
1173
trunk passed

+1
shadedclient
952
branch has no errors when building and testing our client artifacts.

-0
patch
975
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1
mvninstall
67
the patch passed

-1
jshint
235
The patch generated 2585 new + 0 unchanged - 0 fixed = 2585 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
815
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
asflicense
27
The patch does not generate ASF License warnings.

3234

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-342/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/342

Optional Tests
dupname asflicense shadedclient jshint

uname
Linux da55d409b37c 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / e356e4f

jshint
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/5/artifact/out/diff-patch-jshint.txt

Max. process+thread count
341 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/5/console

versions
git=2.7.4 maven=3.3.9 jshint=2.10.2

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
42
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ trunk Compile Tests _

+1
mvninstall
1085
trunk passed

+1
shadedclient
857
branch has no errors when building and testing our client artifacts.

-0
patch
885
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1
mvninstall
61
the patch passed

-1
jshint
208
The patch generated 2585 new + 0 unchanged - 0 fixed = 2585 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
735
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
asflicense
32
The patch does not generate ASF License warnings.

2976

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-342/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/342

Optional Tests
dupname asflicense shadedclient jshint

uname
Linux cdc56ffb8a62 4.4.0-157-generic #185-Ubuntu SMP Tue Jul 23 09:17:01 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 094d736

jshint
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/6/artifact/out/diff-patch-jshint.txt

Max. process+thread count
412 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-342/6/console

versions
git=2.7.4 maven=3.3.9 jshint=2.10.2

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.


Problem downloading MapTile

kz.informenergo.manat W/org.osmdroid.tileprovider.modules.MapTileDownloader: Problem downloading MapTile: /16/46770/24026 HTTP response: HTTP/1.1 403 Forbidden
Version of osmdroid 4.3

Probable duplicate with #1288
cf. FAQ
Feel free to reopen the issue if necessary.


operator: remove lastPodRestart in unmanaged pod controller

operator: remove lastPodRestart in unmanaged pod controller
The restart-unmanaged-pod controller logic was:

Find unmanaged pod to delete.
Lookup the last restart time by the pod's ID (namespace + name).
If last restart time is less than 5 minutes ago, skip.
Otherwise, delete the pod.
If the deletion was successful, set lastPodRestart[podID] = time.Now()

However, step 2 would never find pod ID in the lastRestartTime map, because podIDs are only added to the map after a successful deletion (step 5). Any new pods created after the deletion would be assigned a unique, different, pod ID. Therefore, step 3 would never be executed.
Since the controller behaves the same without checking lastPodRestart, and I don't believe there is a way to reliably identify which pod replaced the deleted pod, remove the lastPodRestart map.

Why would the podID be different after restart? It's the namespace/name after all and not e.g. the types.ID?

There is one case in which pod names are constant, even across deletion: StatefulSets. I agree, though, the logic is convoluted to say the least.

Ok that makes sense. Has the naming behavior changed at some point in Kubernetes? Looking at the original issue (https://github.com/cilium/cilium/issues/6839) it shows that it was added to fix multiple restarts of kube-dns pods and there the name didn't change. I would be hesitant to removing this code until we're sure we don't just reintroduce #6839 back again.

Ok that makes sense. Has the naming behavior changed at some point in Kubernetes? Looking at the original issue (#6839) it shows that it was added to fix multiple restarts of kube-dns pods and there the name didn't change. I would be hesitant to removing this code until we're sure we don't just reintroduce #6839 back again.

hmm, looking at that issue I'm now less confident in this change
Reading through https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#pod-termination I suspect cilium-operator might end up sending multiple delete requests between when apiserver sets DeletionTimestamp on the pod and when kubelet finishes cleaning it up.
Given that the current code isn't causing any problems and I'm not 100% sure removing it is safe, I'll close this PR. Thanks for the review and comments!


PR Request with complete app - catalog of my things

In this project we:
Create a main.rb file that will serve as your console app entry-point.
Present the user with a list of options to perform.
Let users choose an option.
Ask for parameters for the option.
Have a way to quit the app.
The following options are available:
List all books
List all music albums
List all movies
List of games
List all genres (e.g 'Comedy', 'Thriller')
List all labels (e.g. 'Gift', 'New')
List all authors (e.g. 'Stephen King')
List all sources (e.g. 'From a friend', 'Online shop')
Add a book
Add a music album
Add a movie
Add a game
All data is preserved by saving collections in .json files.
Created a schema.sql file with tables that is analogical to the structure of the classes in our app.
Tests created for all of the methods in the classes
Our video:
https://drive.google.com/file/d/1BoHUWu7Stw_zyWdkht_UENjxB_c3E3NR/view?usp=drive_link
Some of our commits:

@TadesseDev  I forgot to push changes, already done.

@TadesseDev  would you review the PR or other person would do it? We have not being reviewed our code after you


Added doc on docker.run

Updated README.md with documentation on docker.run function

Remote API documentation should always be checked at it's source, which is the Docker official documentation.
We don't want to copy it to here, which may create all type of problems.
And there's already information about this in the readme under the "Usage" section.
I'm ok with the small change detailing docker.run parameters types.
All this "indentation" commits https://github.com/apocas/dockerode/pull/210/commits should be compacted into one.


Add potpie to autonomous agents section

Potpie parses your codebase and builds a graph tracking relationships between functions, files, classes, etc. We generate inferences for each node and embed and store it in the graph. This can be used to curate the correct context by performing a similarity search based on users query. The graph can also be queried to understand the code flow, it can be queried to understand the project structure etc.
This allows Potpie's agents to understand the codebase and reason about the code.
Potpie provides pre built agents for Q&A, Debugging, Testing, Understanding Code Changes, Low level design, but userss are not limited to these agents, you can build your own agents leveraging the tools provided to interface with the knowledge graph.
https://github.com/potpie-ai/potpie

Hey @aprxi could you take a look?


Fix typo: sckit-learn -> scikit-learn

Found a typo in the Official AzureML Autotrain configuration page.
It mentions that we need to install a specific version of scikit-learn in specific circumstances, but a simple copy-and-paste didn't work, and it took me a while to realize that the package name had a typo in it.

@besirkurtulmus : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The typo fix looks good; thanks @besirkurtulmus.


[Test] Add a regression test for SR-13258

We regressed in Swift 5.3 (Xcode 12 beta 1 & 2) and accepted the following ill-formed code, but we don't do that anymore on master. So, add a regression test to make sure it doesn't happen again.
let a = 1
let b = Int?.none
if let c = b ?? a { _ = c } // no error in 5.3

@swift-ci please smoke test and merge

@swift-ci please smoke test Linux

@swift-ci please clean smoke test Linux


Pins definition examples discussed in the workgroup.

5 files with the different pins definition options discussed in the workgroup.

Obsolete


Add MAINTAINERS.md file

Please duplicate the file in https://raw.githubusercontent.com/open-horizon-services/web-helloworld-cpp/ac403917069c896f685a317f4ca6b503694f5187/MAINTAINERS.md in this repo.

Can I work on this?


version numbers missing on macOS

i've downloaded Version '0.4.0' but when selecting it in the Finder and using 'Get Info' it doesn't show any Version-number at all. i guess the CFBundleVersion and CFShortVersionString entries in your Info.plist file are missing

Thanks. I will check it and fix.

awesome, thanks ;)


Odd rendering bug for checkboxes in widgets

Picture is pretty clear, the "False" case is wrapping:

Snippet for convenient copy/paste testing:
from ipywidgets import interact
interact(lambda x:x, x=False);

For reference, this is what it looks like in the classic nb:

I see you closed this. Just checking, right now we have the output rendering correctly, but the checkbox is to the right of the label, instead of the left. This is to be consistent with other controls that have the label on the left, control on the right. Is that all right with you?

CC @fperez...

I think that's ok, and it makes sense to be consistent left/right across widgets. TBH I'd never really paid attention to this inconsistency before, but now that I see it, I agree it's better to clean it up. So no objection from me here.


Update Set-HostedContentFilterPolicy.md

'FalsePositiveAdditionalRecipients' is no longer documented nor available for O365 admins.

Docs Build status updates of commit 0d8b75d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/antispam-antimalware/Set-HostedContentFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 1a70624:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/antispam-antimalware/Set-HostedContentFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Does Piranha support scala projects

Does Piranha have support for scala with current code or can it be supported with minimalistic changes in java?

You can potentially add rules for Scala and make Piranha applicable for Scala. See here

@eankit you can use https://github.com/tree-sitter/tree-sitter-scala for the grammar. I think the rules will be very similar to the ones implemented for Java/Kotlin.

can some help with how to and where to make changes in .rs files in the polygot pirhana. I understand the .toml file syntaxes but it seems a lot goes into .rs files before it could be used in the toml file.
@ketkarameya @mkr-plse

@eankit adding new language support does requires very minimal changes .rs files . I am streamlining the implementation here - https://github.com/uber/piranha/pull/283/.
After this merges, it should be easy (though slightly tedious) to add scala support. I will update the README. I will keep you posted.

@ketkarameya  thanks for updating it, I am working on your branch, till it gets merged to the master. I hope this will not have any changes as it looks reviewed.
Will keep this thread open for any more queries on it.

I assume you get this error on adding the tree-sitter-scala right?

@ketkarameya I have added the it in Cargo.toml, I have added the one mentioned above  -
My cargo.toml looks like this -
tree-sitter-kotlin = { git = "https://github.com/ketkarameya/tree-sitter-kotlin.git" }
tree-sitter-java = { git = "https://github.com/tree-sitter/tree-sitter-java.git" }
tree-sitter-swift = { git = "https://github.com/ketkarameya/tree-sitter-swift.git", branch = "add_parser" }
tree-sitter-strings = { git = "https://github.com/ketkarameya/tree-sitter-strings.git" }
tree-sitter-python = { git = "https://github.com/tree-sitter/tree-sitter-python.git" }
tree-sitter-go = { git = "https://github.com/tree-sitter/tree-sitter-go.git" }
tree-sitter-typescript = { git = "https://github.com/tree-sitter/tree-sitter-typescript.git" }
tree-sitter-scala = { git = "https://github.com/tree-sitter/tree-sitter-scala.git" }

@ketkarameya  upon fetching the latest pull from master, I am getting this error without adding scala. This error is coming for typescript also up doing a pip install .

I am on my local mac

So you are able to build the application locally (with no changes)?

yes if I remove typescript from your code/master it is building it correctly

@ketkarameya  after making the below change I am able to run the pip install . successfully
diff --git a/polyglot/piranha/src/models/language.rs b/polyglot/piranha/src/models/language.rs
index d5a90fc..639e371 100644
--- a/polyglot/piranha/src/models/language.rs
+++ b/polyglot/piranha/src/models/language.rs
@@ -148,24 +148,6 @@ impl From<&str> for PiranhaLanguage {
scopes: vec![],
comment_nodes: vec![],
},

 "ts" => PiranhaLanguage {

   name: language.to_string(),

   supported_language: SupportedLanguage::Ts,

   language: tree_sitter_typescript::language_typescript(),

   rules: None,

   edges: None,

   scopes: vec![],

   comment_nodes: vec![],

 },

 "tsx" => PiranhaLanguage {

   name: language.to_string(),

   supported_language: SupportedLanguage::Tsx,

   language: tree_sitter_typescript::language_tsx(),

   rules: None,

   edges: None,

   scopes: vec![],

   comment_nodes: vec![],

 },
 _ => panic!("Language not supported"),

}
}

I am using a Mac too.  I did the following

git clone ...
cd piranha/polyglot/piranha directory.
Then ran the command cargo build --no-default-features (which ran successfully)

Are you doing the same?

I am following these steps
git clone https://github.com/uber/piranha.git
cd polyglot/piranha
Create a virtual environment:
python3 -m venv .env
source .env/bin/activate
Install Polyglot Piranha
pip install --upgrade pip
pip install . to run demo against current source code (please install Rust, it takes less than a minute)
Or, pip install polyglot-piranha to run demos against the latest release.

Can you please try  cargo build --no-default-features after cd piranha/polyglot/piranha step ?

@ketkarameya I wish I could share my screen
anbhanda@Ankits-MacBook-Pro-6 piranha % git status
On branch master
Your branch is up to date with 'origin/master'.
nothing to commit, working tree clean
anbhanda@Ankits-MacBook-Pro-6 piranha % cargo build --no-default-features
Compiling piranha v0.2.1 (/Users/anbhanda/Documents/Pirahnah_Polygot/piranha/polyglot/piranha)
error[E0308]: mismatched types
--> src/models/language.rs:154:19
|
154 |         language: tree_sitter_typescript::language_typescript(),
|                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct tree_sitter::Language, found a different struct tree_sitter::Language
|
= note: perhaps two different versions of crate tree_sitter are being used?
error[E0308]: mismatched types
--> src/models/language.rs:163:19
|
163 |         language: tree_sitter_typescript::language_tsx(),
|                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct tree_sitter::Language, found a different struct tree_sitter::Language
|
= note: perhaps two different versions of crate tree_sitter are being used?
For more information about this error, try rustc --explain E0308.
error: could not compile piranha due to 2 previous errors

@ketkarameya
Started in a fresh directory works fine for existing code!!!!
Thanks for your help, will start with making changes for Scala now.
Any suggestions that you want me to follow before starting up with the changes in language.rs and cargo.toml
Or do you feel it will be more changes required before I start to write the rules for scala

Hey! I think once u add that struct for scala you will need to write rules (no more rust changes)

It compiles will with adding scala dependency in cargo.toml file but compilation fails with following error
It looks to me a rust issue
https://github.com/rust-lang/cargo/issues/8536
anbhanda@-MacBook-Pro-6 piranha % cargo build --release --no-default-features
Compiling piranha v0.2.1 (/Users/anbhanda/Documents/Pirahnah_Polygot_2/piranha/polyglot/piranha)
error[E0308]: mismatched types
--> src/models/language.rs:172:19
|
172 |         language: tree_sitter_scala::language(),
|                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct tree_sitter::Language, found a different struct tree_sitter::Language
|
= note: perhaps two different versions of crate tree_sitter are being used?
For more information about this error, try rustc --explain E0308.
error: could not compile piranha due to previous error
PS: I compiles just after doing git clone and making the required changes

Yes, I is a tree-sitter version mismatch. So we (Piranha) is using 0.20.6, while tree-sitter scala is using 0.19.
I would recommend upstreaming the version update change for tree-sitter scala to something like this.
While the upstream accepts your changes, you can make a fork of tree-sitter-scala, update the tree-sitter version in a particular branch and point to this fork & branch of tree-sitter-scala (this will allow you to continue to experiment with piranha)

This latest PR adds built-in cleanup rules for Go.
https://github.com/uber/piranha/pull/265
You can use it as reference :)

@ketkarameya thanks a lot, forking scala tree-sitter worked.
Currently struggling with comprehending query language for scala, any help in this regard will be appreciated.

Hmmm. yea.
Did you read the README throughly (specially this)?  Where are you stuck at ? Can you make a PR for any work you have in progress

@eankit. Can you create a PR for just adding Scala to Piranha ?
We can take it from there, one step at a time.


Have you seen AceJump?

Just some feedback: Personally I think it takes too much concentration and it becomes a distraction away from work the fact that you have to press "Enter" to move on to the next batch of results. I use my monitor vertically and there is always a lot of code visible, if I choose to jump to letter "A" which is quite common, I might have to press Enter up to 3 times to get to it. This has made me stopped using EasyMotion as much as I would like too.
What I'd like to suggest is an approach similar to AceJump (which I am using with IntelliJ) where if there are more results than letters it switches into a mode where you can press 2 letters instead of one to jump further up. This way you could allow much higher coverage. If I were more proficient in Python I would add this myself but I am not.
I think this is worth considering and I would personally love this plugin if you would add this feature. I'm attaching an example of AceJump from IntelliJ below. It is uglier than EasyMotion but just to make a point.
Thanks!

An AceJump plugin for ST3 has been released, I'd check that out: https://github.com/ice9js/ace-jump-sublime

Pretty confusing how to set up, I'm not sure if like it said copying the contents actually works. Also I have to set the syntax to acejump??


Gather diagnostic info

Is your feature request related to a problem? Please describe.
Currently it is difficult if not impossible to diagnose some issues, especially sync, short of debugging jointly with a customer. Even then, some info is not readily available
Describe the solution you'd like
Implement a simple DiagosticService and simple DiagnosticObserver
Enhance AtClient to share diagnostics via the DiagnosticsService. Initially support streaming of data mutation events (out or delete in AtClient) and detailed sync processing events.
Create a simple implementation of a client of the DiagnosticsService which developer can use to gather and share diagnostic info  with minimum effort
Describe alternatives you've considered
No response
Additional context
No response

5SP effort during PR48.
Will close out in PR49 (move some code to at_commons, PRs for at_commons and at_client_sdk, publish packages

All done, just need to publish now ( #822 )


Possible incorrect regex for header test

The line:
https://github.com/VeliovGroup/spiderable-middleware/blob/05078ab58400307f2acdbfeec3f56f651e5e76ac/lib/index.js#L102
should probably be:
this.headersRE = new RegExp('^' + this.ignoredHeaders.join(strs.pipe) + '$', 'i');

otherwise you will match 'user-agent' for example since it has 'age' in it.

@arabidopsis then it also should be wrapped with parentheses like
new RegExp('^(' + this.ignoredHeaders.join(strs.pipe) + ')$', 'i');

Or I'd prefer ES6:
new RegExp(`^(${this.ignoredHeaders.join(strs.pipe))$`, 'i');

Hello @arabidopsis ,
Thank you a lot for reporting this one, please update to v1.6.2
Feel free to close it in case if the issue is solved on your end.

Closed due to silence at issue owner end.
Feel free to reopen it in case if the issue still persists on your end.


Use example objects to show response examples

Now ReDoc tries to generate examples based on schema field of responce object. But we have examples in respnse to and if example field is present, it'll be better to use it instead of schema.

@elcamlost could you share a part of spec where you have examples in response?

I'm sorry, my bad.
Swagger spec says about schema field for response object "If this field does not exist, it means no content is returned as part of the response." So I was wrong, when providing example objects without schema object and awaiting examples in ReDoc.


When I try to add Com.Airbnb.Xamarin.Lottie to my PCL

I have Xamarin.Forms added already.  Not sure why its complaining?
Installing Com.Airbnb.Xamarin.Forms.Lottie <IP_ADDRESS>.
Install failed. Rolling back...
Package 'Com.Airbnb.Xamarin.Forms.Lottie.<IP_ADDRESS> : Xamarin.Forms [<IP_ADDRESS>, )' does not exist in project 'oneiota'
Package 'Com.Airbnb.Xamarin.Forms.Lottie.<IP_ADDRESS> : Xamarin.Forms [<IP_ADDRESS>, )' does not exist in folder 'C:\Users\ryan\Dropbox\Programming\oneiota\packages'
Could not install package 'Com.Airbnb.Xamarin.Forms.Lottie <IP_ADDRESS>'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile259', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

And do you have Xamarin Forms <IP_ADDRESS> installed?

I sure do!

👍 same issue here as well...

Yes, I did the same, I installed the iOS and Android version then copied the Renderer's over and it works.

I just tried creating a new project and adding it, same failed install.

It appeared that the PCL profile in the nuspec was not compatible with Profile111 which is standard used in Forms. I used Profile7 which worked perfectly fine. This is updated now, and it is possible to use in the latest release.


[OPER_Mike-26] Unnecessary dependencies

This mission is requiring ASR and R_Unfold to be loaded. These mods tends to create unwanted dependencies, and should not be included in edition preset.

Update : similar dependencies also appear on other Operations.

Theses are part of the core modpack, is there really a problem with their presence ?

These mods are not required for the mission to work properly, so they should not be considered as dependencies. Changing the modpack will be difficult if we have such uncontrolled dependencies here and there.


[flutter_tools] Revamp observatory port discovery logic on Fuchsia

Fix dart observatory discovery protocol on Fuchsia.

Further work on https://github.com/flutter/flutter/issues/83609
This PR is now ready for review.
This should be merged after https://github.com/flutter/flutter/pull/85752.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

I have got some progress here, you can see https://youtu.be/XCaNaxVI1hI to see my working code.

A noteworthy commit message here, it documents why did I changed the implementation of FuchsiaIsolateDiscoveryProtocol
fix: prevent FlutterIsolateDiscoveryProtocol from using creating DDS instance
Before this change, FlutterIsolateDiscoveryProtocol identifies the correct vm service port for the flutter app we are debugging by
connecting to the vm service, creating a Flutter DDS instance, and then query whether the the vm is serving our required isolate. This method has indeed caused some problems, one of them is that the FlutterIsolateDiscoveryProtocol passes the port to the DDS instance back to startApp, in which ResidentRunner tries to create another DDS instance on top of the existing DDS instance. This causes the flutter_tools to crash since the underlying vm service has already been instructed to enter Single Client Mode by the first DDS, hence the second DDS will not be able to connect.
The current implementation instead queries the flutter apps running under each runner by querying the /hub. Although implementation does use the currently hub v1 protocol which is currently under deprecation, it does not involves creating extra VM services hence less error prone. I have tried to use the /hub-v2 protocol instead but have met several bugs that made it not possible to be used to complete this task. A TODO has been added to migrate to the hub-v2 protocol when possible.
This version is verified to have flutter run working with the qemu-x64 and workstation.qemu-x64-release build on emulator.

@chaselatta The coding has been done with relavant tests written and unneccessary changes rolled back so I will mark it ready for review now. Please review it. This is a rather larger PR so you may want to find other reviewers. Thanks for help in advance.

I can see some tool_tests failing, but I am not sure if that was my problem. I don't have code I changed are related to those failing tests.

I can see some tool_tests failing, but I am not sure if that was my problem. I don't have code I changed are related to those failing tests.

The failing tests that I see do look related to this change. For example there is a failure in:
test/general.shard/fuchsia/fuchsia_device_start_test.dart: Fuchsia app start and stop:  fail when cant get host address

@zanderso @chaselatta I have refactored as discussed and all tests is passing now. Please review.

How about this one? Are there any problems blocking this one from being merged other than #85752 ? I will merge all changes in #85752 into this PR.

@michaellee8  , @chaselatta is OOO for several weeks so we might not be able to make a decision on whether to go with this approach (versus switching to more of the SDK tooling as it is made ready).

@jonahwilliams Nevermind, so I guess we will stall this PR until Chase is back

What is the status of this PR?

I'm going to close this PR since it seems to have been abandoned.


Add administrative scripts to locate maps and/or layers that satisfy some criteria

Arose from looking for any layer that referenced a particular URL (one that
included 'lance-modis'; I wrote the routine below, but it would be nice to
generalize to a function that would accept a callable as an argument.  The
function would search the maps and/or layers for those where the callable
returned true, and then return the results as a list of (mapID, layerID) pairs.
Then a handful of utility functions that could be used to e.g. look for a
particular string could be added.

all_maps = Map.GetAll()
all_maproots = []
for m in all_maps:
  all_maproots.append(json.loads(m.current_version.maproot_json))

for m in all_maproots:
  print m.get('id', '<no map id>')
  layers = maproot.get('layers', [])
  print '%d layers' % len(layers)
  for l in layers:
    if 'lance-modis' in str(l):
      print ' -- layer %s (title %s) matches' % (l['id'], l['title'])
    else:
      print ' -- no match'

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 30 Sep 2013 at 11:16

Closing - This repository is being archived. It is no longer maintained.


update SPI

updates SPI to latest images and deployment reference. This brings couple of important changes like:

vault token storage
removing webhooks
new internal trigger CRD spiaccesstokendataupdates.appstudio.redhat.com

/retest


Retina issue

Only get bottom left quadrant with retina turned on.

It works on a non retina macbook, but I'll try it out on retina display soon.

Yes, when I switch off retina, it works as normal.

Should be fixed now according GLFW FAQ.

Yep. That fixed it.


Increase recursion limit in ruff_linter

See https://github.com/astral-sh/ruff/pull/10325#issuecomment-1992596557 for motivation. I'm not sure of the implications of the change.

Huh?!?

Something to do with the proc macros... this is beyond me. @snowsignal maybe you could take a look?

The problem is that the map_codes macro creates a very long type signature in
https://github.com/astral-sh/ruff/blob/e0bc08a7582d64f6e69a69835f748bff59922d6d/crates/ruff_macros/src/map_codes.rs#L390-L397
which is probably also very slow at runtime because we chain a very long sequence of iterators together, meaning that next need to check if any iterator still returns an element.
I think we can rewrite the implementation to use a Vec instead. This has the downside that calling iter allocates but the method isn't called frequently (to my knowledge).  We potentially could do even better by unrolling EnumIter but that's a more involved change

@zanieb I will close this PR because #10452 resolved the underlying issue without increasing the recursion limit.


How can i config collector.backend_service to use HTTP protocol?

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
I want to use HTTP protocol (12800) to communicate with OAP Service in AGENT, but it seems only support GRPC protocol? like this:

SW_AGENT_COLLECTOR_BACKEND_SERVICES=my-oap-service:12800

Java agent doesn't support HTTP protocol, and it is not recommended.


[Bug] The stream of LZMA is not set up correctly

I used LoadBundleFile to load an AssetsBundle that was compressed using LAMZ
The code looks like this

When I try to get Assets using LoadAssetsFileFromBundle, it tells me to read more than the stream size.
I found that the underlying stream used by Reader is the original FileStream rather than the decompressed MemoryStream.

I thought I fixed this already since I had a convo about it in the discord. The issue is that the code to check LZMA does it by checking the header compression format instead of the block compression format. When I get back home, I'll check if it's just something I committed but didn't push.

Check latest commit.


Issues with external links

Links at bottom of building report redirect to a password protected page (https://energysolutions.seattle.gov/contact-us/)

https://www.seattle.gov/light/free-assessment
https://www.seattle.gov/light/incentives

Download link at bottom of main app goes to a 404

┆Issue is synchronized with this Asana task

Thank you for flagging that the existing links to SCL no longer work. It appears that all commercial programs are now nested under the link lis ted below. I need to do a bit more investigation to figure out how to highlight relevant programs. There is a different link for Multifamily incentives.

https://www.seattle.gov/city-light/business-solutions/large-commercial-and-industrial-business-solutions

The link sections are more complicated to correct than I have time to navigate right now. I will need to work with Seattle City Light to update the text and add appropriate links. Is it possible to hide the call to action section? This work will need to be handled in 2023, hopefully when I have a new staff person to lead this work and navigate the back and forth. The older building reports from 2020, 2019 on back now have broken links too. These should be corrected too so we will need to create a plan of action.

Links have been updated

Download Data link: https://www.seattle.gov/environment/climate-change/buildings-and-energy/energy-benchmarking/data-and-reports
Weather Normalized Source EUI (kBtu/ft2), reference link on Glossary:  https://www.energystar.gov/buildings/tools-and-resources/portfolio-manager-technical-reference-climate-and-weather
Links on building performance reports in Call to Action - hiding this section for now


scite: 3.7.5 -> 4.0.5

Motivation for this change
Things done

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

[x] NixOS
[ ] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg  build scite

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: scite
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: scite
Partial log (click to expand)
install -m 644 ./Sci48M.png /nix/store/zpnjkrp1d1yz4bpl20s8k5ycy9hy8bkm-scite-4.0.5//share/pixmaps/Sci48M.png
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/zpnjkrp1d1yz4bpl20s8k5ycy9hy8bkm-scite-4.0.5
shrinking /nix/store/zpnjkrp1d1yz4bpl20s8k5ycy9hy8bkm-scite-4.0.5/bin/SciTE
strip is /nix/store/92d2ifxcni4n3zx9s8wnkcjlvnx5ajlc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/zpnjkrp1d1yz4bpl20s8k5ycy9hy8bkm-scite-4.0.5/bin
patching script interpreter paths in /nix/store/zpnjkrp1d1yz4bpl20s8k5ycy9hy8bkm-scite-4.0.5
checking for references to /build in /nix/store/zpnjkrp1d1yz4bpl20s8k5ycy9hy8bkm-scite-4.0.5...
/nix/store/zpnjkrp1d1yz4bpl20s8k5ycy9hy8bkm-scite-4.0.5

Thanks!


[Release Radar Request] NativeBase v3.2.2

Open Source Project name
NativeBase
What is your project?
NativeBase is an accessible, utility-first component library that helps you build consistent UI across Android, iOS, and Web.
Version
3.2.2
Date
November 11, 2021
Description of changes
Fix FormControlErrorMessage rendering
Fix shadow/opacity not working with Input
Fix dependency issues when installing with npm 7+
Fix Text variant typing
Fix textAlign property
Fix nested text styling
Fix TextArea alignment on Android
Fix SSRProvider warning
Avatar group flex-direction error
GitHub Repo
https://github.com/GeekyAnts/NativeBase
Website
https://nativebase.io
Link to changelog
https://github.com/GeekyAnts/NativeBase/releases/tag/v3.2.2
Social media
https://twitter.com/nativebase
Anything else to add?
No response

Sorry we couldn't include this in the November Release Radar as that had already shipped. Please definitely submit again for v4.0 and we'd love to consider your project 😄


The error class and tooltip can not be removed on valid when it's empty

successList does not include those empty inputs.
Example:
"email-edit" : {
email : true
}
This input is optional.
It shows error when I input a wrong email address,but when I delete my input,the error info is still there.
So I think the "defaultShowErrors" method should be like this:
$.each(this.successList, function(index, value) {
...
});
change to:
$.each(this.validElements(), function(index, value) {
...
});

I'm going to have to agree with you.  After taking a look at the jQuery validation source and open issues, you may have solved a number of open issues with vague descriptions that sound just like this.
When I have free time later tonight, I will to do more investigating and testing to ensure this fix does not break anything else.

Hmmm.  This is going to be more work than I originally though.  Sorry, I'll try to have a fix in the next day or two.

I was making it more complicated than it actually was.  You were right.  I pushed the fix.


pause_builds(): should it wait for pending builds?

When waiting for the resource quota to be satisfied, what happens to a pending build? Has it been admitted, and will inevitably become 'running'?

it should at some point, right? if not, there's a bug in scheduling

Currently pause_builds() only checks for builds that are 'running', before returning.
But to be sure all the admitted builds have drained, should it also check for builds that are 'pending'?
i.e. can/does this happen?

build submitted (new)
new -> pending
pods hard quota limit set to 0
pending -> running

oh, I misread the question; I guess this would be an excellent question for openshift devs :)

https://github.com/openshift/origin/blob/master/pkg/build/doc.go seems to have the answer:

For pending builds, the BuildController will attempt to create a pod to perform
the build.

So we should not wait for pending builds.

Oh, and now this seems to contradict it:
https://github.com/openshift/origin/blob/master/pkg/build/controller/controller.go#L79

// HandleBuild deletes pods for cancelled builds and takes new builds and puts
// them in the pending state after creating a corresponding pod

Pinging @bparees or @mfojtik; I think this question falls under the DevExp realm.

pending builds will sit in pending until sufficient resources are available at which point they will run.

@bparees so when a build is pending, it hasn't created a pod yet? So after setting pod limit to 0, pending builds will stay pending?

@twaugh at that point the pod is created, but not scheduled.  so yes, pod limit of zero should leave the build in pending, unless it gets rejected from being created at all based on that limit.

Thanks!


Add bluetooth examples

I couldn't find any examples that interacted with Bluetooth, especially BLE, using Rust.  There seems to be some promise in https://github.com/sdobz/hanging-plotter/blob/dbf661f78add7eea9aba52934a8ddae480fc5da9/esp32/esp32-bluetooth/src/main.rs, but I'm still working through that.
It would be incredibly helpful if there were examples provided in this repo.  I imagine building such an example would prompt the creation of Rust wrappers so that we don't have to call all the C symbols unsafely.

I’m currently working on a wrapper (see the develop branch) for the GATT Server API in bluedroid. It still has a lot of work to do, and it relies on many constants that are specific for my project, but as soon as I can test basic characteristic reading/writing operations, I’ll hopefully make it into a crate.

@persello:  Thanks for this update, this is great to hear.  After working with the example that I linked above, we're starting to put together some ideas about how to wrap the nimBLE stack in a crate, starting with our needs first of course.  But I think our needs are pretty common:

set up advertising data and start advertising
set up services with characteristics
support characteristic read, write, subscribe, and indications
support configuring encrypted connections (we don't plan to use pairing PINs)

Based on my reading about how bluedroid and nimBLE work on esp32, I gather that nimBLE is expected to use fewer resources for a BLE-only and GATT-focused solution.  Can you share why you selected bluedroid for your GATT server implementation?

I've used NimBLE once in the past (in a C project, more than three years ago) and found its documentation quite lacking for somebody who didn't have previous experience with Bluetooth LE. Since I'm developing this crate for an academic project and my time is quite limited, I chose Bluedroid because of the major availability of documentation and examples.

@persello: Thanks, that's a good answer.  I'll keep that in mind.

Mentioning @rgcottrell in case you're interested.

BLE is currently more of an esp-idf-svc thing. Fore more info have a look here: https://github.com/esp-rs/esp-idf-svc/issues/55

Closing in favor of https://github.com/esp-rs/esp-idf-svc/issues/55


Move nasa-veda hub URL

We keep redirects in place from the old URL, in case there are external links already present.
The OAuth callback URL needs to be changed at
https://github.com/organizations/2i2c-org/settings/applications/2096065 just before merging.
Ref https://github.com/2i2c-org/infrastructure/issues/3029

Let's wait for explicit approval in https://github.com/2i2c-org/infrastructure/issues/3029#issuecomment-1693843216 before going through with this.

@freitagb is going to use this as an opportunity to use a brand new domain. So let's hold off for a bit more.

Ping @freitagb - what do we do with this one?

Ping @freitagb - should we move forward on this?

I've re-pinged @freitagb on slack now.

yes, can we move forward with migrating to openveda.cloud?

@yuvipanda can we use hub.openveda.cloud

@yuvipanda done!

@slesaad hmm, I'm not sure. I guess I'll just wait a little more and see?

@slesaad they still don't resolve for me - are they resolving for you?

@yuvipanda they are not 🤔 i wonder why. let me delete and recreate them and see if that helps. I don't know what else to try 😅

@slesaad yeah i don't see it still. I'm not sure what the problem is. I still see it return an NXDOMAIN, which means the authoritative DNS server doesn't think it exists still. Maybe there's an extra 'rollout' step or something? I'm not sure.
I also see that openveda.cloud itself doesn't really resolve to anything, so maybe that's also part of the issue? Whoever is managing that entire domain may be able to help.

Thanks @slesaad!


Custom editors: how to find out that text document is readonly?

Refs: https://github.com/microsoft/vscode/issues/91325
A text document can be readonly if it's underlying file system provider is readonly. I am not sure how a custom editor implementor can figure this out and provide the correct experience.

Could we add a isReadOnly property to TextDocument?
Does VS Code already know if a given textDocument is isReadOnly?

We basically ask the file system provider for the resource of the document for being readonly or not. That should be doable.


MS Defender TVM updates

Problem
TVM was not bringing in ip address and tags. Customers also reported seeing inactive assets
Solution
This pr brings in only Active assets and their ip addresses, tags

Going to close this one. It was kind of implemented with pipeline and would very quickly hit the microsoft limit
Rate limitations for this API are 100 calls per minute and 1500 calls per hour.


I/O Exceptions in MatomoAnalytics ctor on Android & iOS

I tried to vaguely follow the initial-setup procedure from https://github.com/bnotech/Matomo.Xamarin.Forms/blob/main/README.md#initial-setup (which is unfortunately missing for Matomo.Maui). However, whenever I try to use Matomo.Maui v7.1.1 in a net8.0 app, e.g. via MatomoAnalytics.TrackPage, I get I/O exceptions inside the MatomoAnalytics constructor, both on Android and iOS.
Exception on Android 14:
System.IO.IOException: Read-only file system : '/matomodata.json'
  at at Interop.ThrowExceptionForIoErrno(ErrorInfo errorInfo, String path, Boolean isDirError)
  at at Interop.CheckIo(Error error, String path, Boolean isDirError)
  at at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String path, OpenFlags flags, Int32 mode, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException)
  at at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, UnixFileMode openPermissions, Int64& fileLength, UnixFileMode& filePermissions, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException)
  at at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode, Func`4 createOpenException)
  at at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
  at at System.IO.Strategies.UnixFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
  at at System.IO.Strategies.FileStreamHelpers.ChooseStrategyCore(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
  at at System.IO.Strategies.FileStreamHelpers.ChooseStrategy(FileStream fileStream, String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
  at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize)
  at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
  at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, Boolean useAsync)
  at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize)
  at at System.IO.StreamWriter.ValidateArgsAndOpenPath(String path, Boolean append, Encoding encoding, Int32 bufferSize)
  at at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding, Int32 bufferSize)
  at at System.IO.StreamWriter..ctor(String path, Boolean append)
  at at System.IO.File.CreateText(String path)
  at at Matomo.Maui.SimpleStorage.WriteToDisk()
  at at Matomo.Maui.SimpleStorage.Put[String](String key, String value)
  at at Matomo.Maui.MatomoAnalytics..ctor(String apiUrl, Int32 siteId)

Exception on iOS 18:
Exception has occurred: System.UnauthorizedAccessException
Access to the path '/private/var/containers/Bundle/Application/CCC5627A-32C5-4C5D-B0A5-ED02E15AF4B3/MyApp.app/matomodata.json' is denied.
    at at Interop.ThrowExceptionForIoErrno(ErrorInfo errorInfo, String path, Boolean isDirError)
    at at Interop.CheckIo(Error error, String path, Boolean isDirError)
    at at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String path, OpenFlags flags, Int32 mode, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException)
    at at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, UnixFileMode openPermissions, Int64& fileLength, UnixFileMode& filePermissions, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException)
    at at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode, Func`4 createOpenException)
    at at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
    at at System.IO.Strategies.UnixFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
    at at System.IO.Strategies.FileStreamHelpers.ChooseStrategyCore(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
    at at System.IO.Strategies.FileStreamHelpers.ChooseStrategy(FileStream fileStream, String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
    at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize)
    at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
    at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, Boolean useAsync)
    at at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize)
    at at System.IO.StreamWriter.ValidateArgsAndOpenPath(String path, Boolean append, Encoding encoding, Int32 bufferSize)
    at at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding, Int32 bufferSize)
    at at System.IO.StreamWriter..ctor(String path, Boolean append)
    at at System.IO.File.CreateText(String path)
    at at Matomo.Maui.SimpleStorage.WriteToDisk()
    at at Matomo.Maui.SimpleStorage.Put[String](String key, String value)
    at at Matomo.Maui.MatomoAnalytics..ctor(String apiUrl, Int32 siteId)

I just noticed that there is a fork of the the repo, which should probably fix the problem via this commit (by @Alex-Witkowski: https://github.com/DevelappersGmbH/Matomo.Maui/commit/214ba5a4eaea628006a6c6ff09c60af561ffb615

So, the question is how to move forward here. @bnoffer Are you going to keep on maintaining this repo, or have you abandoned it?
If the former, we should create a PR for the changes from https://github.com/DevelappersGmbH/Matomo.Maui/commits/Refactoring/.
If the latter, we have to think of something else. Either @Alex-Witkowski could take over maintainership, or we move the repo into a shared org on GitHub, where we can maintain it together. What do you think, guys?


'Organization over seats' email

Related to #5001, should we warn the organization owner if organization runs out of seats?
cc @andrewxhill @saleiva

+1
On Tue, Sep 15, 2015 at 5:14 AM, Juan Ignacio Sánchez Lara <
<EMAIL_ADDRESS>wrote:

Related to #5001 https://github.com/CartoDB/cartodb/issues/5001, should
we warn the organization owner if organization runs out of seats?
cc @andrewxhill https://github.com/andrewxhill @saleiva
https://github.com/saleiva
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb/issues/5518.

--
andrewxhill http://twitter.com/andrewxhill
http://cartodb.com


I want to add implementation of Array List  in Java.

I can show how Array List works by defining it in another class and use as a Class Like we use other classes and methods in java.

Please assign this to me.


refactor validate_suspend to supports :suspend

What?
Refactor all method definitions called validate_suspend to DSL like definition supports :suspend
Background
The validate_suspend methods are used mainly by the UI to query an instance if it supports a specific feature. The return value is a bit obscure. It returns a Hash with two keys :available and :message. If :available is false, then the feature is generally unsupported by all instances of that class. If true, then it's generally supported, but you would have to look at the :message. If this is present, it's not supported for that particular instance, because of the reason named by :message.
The SupportsFeatureMixin seeks to remedy this by introducing a nice DSL like approach to define the conditions when a feature is supported or not. Please read the documentation. Things worth noting:

You will have to register the feature here
By default all features are unsupported. So mostly you'll only want to define supports :suspend on subclasses that actually support it - a whitelist approach. Not the other way round.

Why?
Other than making the definition more appealing it also lets us produce awesome spreadsheets like this - everybody likes tabellen, right?
How?

Have a look at https://github.com/ManageIQ/manageiq/pull/9683 as an example PR
git grep 'validate_suspend' to find what to refactor - peek
git grep 'is_available.*:suspend to find if the UI still uses the old methods and refactor those too

Bonus Points
Also have a look at the amazon provider if something lurks in there.
please leave a comment to grab this issue.
@miq-bot add_label hacktoberfest

Hi, I want to grab this issue)

@cynepco3hahue great, if you need any help, ping me on the corresponding PR, also link your PR to this by just mentioning it in the PR description.

@durandom I believe we can close it because #11752


Suggested change to docs regarding asynchronous calls in intent handlers

I used these docs to develop my intent handler that requires an asynchronous call to an external server:
https://www.npmjs.com/package/alexa-app#asynchronous-handlers-example
The docs give this code sample as an example of an async call inside an alexa-app intent handler:
app.intent("checkStatus", function(request, response) {
  // `getAsync` returns a Promise in this example. When
  // returning a Promise, the response is sent after it
  // resolves. If rejected, it is treated as an error.
  return http.getAsync("http://server.com/status.html").then(function (rc) {
    response.say(rc.statusText);
  });
});

I think the docs need a fully fleshed out Promise example.  In order to get the body of my Promise working I had to make an explicit call to resolve(response)  when I was done.  Without that call, alexa-app returned an ERROR to Alexa with an empty payload and the end session flag set to true.
Perhaps something like this?:
/**
 * Sample promise that makes an async call that can be called inside an alexa-app intent handler.
 *
 * @param {Object} request - An alexa-app request object.
 * @param {Object} reponse - An alexa-app response object.
 * @return {Promise}
 */
function asyncCallInsideIntent_promise(request, response)
{
	// Build a new promise object that calls the dialog server and process the result before
	//  returning our response to Alexa
	return new Promise(
		function(resolve, reject)
		{
			try
			{
				// Validate input parameters.
				if (request == null)
					throw new Error('The request object is unassigned.');

				if (response == null)
					throw new Error('The response object is unassigned.');

				// Make the async call.  Do not make any explicit return calls.  See the
				//  comment above the resolve() call in the "then" block below.  The
				//  fake async call below simply returns some text to say to Alexa.
				doAsyncCall()
				.then(function(newText)
				{
					// Resolve with the response or the intent will
					//  end up returning an empty JSON payload to Alexa with the
					//  end-session flag set to TRUE.
					resolve(response);
				})
				.catch(function(err)
				{
					// Convert the error into a promise rejection
					reject(err);
				});
			}
			catch (err)
			{
				// Convert the error into a promise rejection
				reject(err);
			}
		});
}

If you want me to fork the repo and make a pull request I'm happy to do that, but I'd like to get your comments first before doing that.

Please make a PR, we can look at the actual change.

@dblock Just did:
https://github.com/roschler/alexa-app/pull/1

+1 on this doc change. I spent several hours struggling with the promise resolve until I read the above. Thx @roschler !

Please make a PR into the root project @roschler.


Why doesn't the new-relic-admin support nonced CSP?

https://github.com/newrelic/newrelic-python-agent/blob/242c51a869b506d4235c8fedf024002251ac502c/newrelic/api/asgi_application.py#L172
Seems weird that the default would be "'unsafe-inline'"?

https://github.com/newrelic/newrelic-python-agent/blob/242c51a869b506d4235c8fedf024002251ac502c/newrelic/api/html_insertion.py#L32

https://github.com/newrelic/newrelic-python-agent/blob/242c51a869b506d4235c8fedf024002251ac502c/newrelic/api/html_insertion.py#L49

https://github.com/newrelic/newrelic-python-agent/blob/ad65494033a6aef95fa2cd10b49ae73cb4c612ed/newrelic/api/web_transaction.py#L42-L44

Any suggestions?

https://docs.newrelic.com/docs/apm/agents/python-agent/python-agent-api/disablebrowserautorum-python-agent-api/

https://discuss.newrelic.com/t/content-security-policy-and-browser-injection/2629

https://github.com/newrelic/newrelic-python-agent/blob/ad65494033a6aef95fa2cd10b49ae73cb4c612ed/newrelic/api/web_transaction.py#L402-L403

Similar:

https://github.com/newrelic/newrelic-ruby-agent/issues/332
https://github.com/newrelic/newrelic-ruby-agent/pull/673/files

As linked above, this has been implemented in the Ruby agent so would appear to be fairly trivial to implement for the Python agent as well. As far as I've seen, the reasoning for not doing this so far hinges on an assumption that it would involve breaking compatibility with outdated browsers, although that's not necessarily true or even important to many people, as expressed by many over 6 years in this thread.

It's not stale?

This should not be marked as stale as it is a security focused feature request

+1

+1

This should still be open as it is a security issue. I am seriously considering dropping newrelic due to the lack of movement on this

Hello @joshuata, we will get this effort scoped out on our teams end and determine when we can add this to our upcoming releases.
cc: @ak-war

@Ak-x Thank you!

Our team using NewRelic would really like support for this. We cannot use NewRelic on the front-end at the moment due to its lack of configuration options to align with our CSP and our requirement to use a nonce. I feel let down as other agent implementations support it except the Python one.
Hope you folks can include this one in! 🤞

@TimPansino Thank you! Happy Holidays. :fireworks:


Auto generated platform import doesn't compile due to file not found error

Description
Auto generating platform imports yields a file not found error during compilation. I've tested this with the basic template.
Steps To Reproduce

Activate jaspr through dart pub global activate jaspr_cli
Create a new project with jaspr create -t basic my_app
Add a component with a platform specific import (@Import.onWeb('dart:html', show: [#window]))
See the following error(s) in console

[BUILDER] [ERROR] Response:--dart-sdk-summary=file:///home/f44/dev/flutter/bin/cache/flutter_web_sdk/kernel/ddc_outline_sound.dill --output=/tmp/scratch_spacePILRGT/packages/platform_import_test/app.ddc.dill --packages-file=org-dartlang-app:///.dart_tool/package_config.json --multi-root-scheme=org-dartlang-app --exclude-non-sources --summary-only --target=ddc --libraries-file=file:///home/f44/dev/flutter/bin/cache/flutter_web_sdk/libraries.json --reuse-compiler-result --use-incremental-compiler --used-inputs=/tmp/kernel_builder_OXMFYH/used_inputs.txt --input-summary=org-dartlang-app:///packages/jaspr/jaspr.ddc.dill --input-summary=org-dartlang-app:///packages/jaspr/src/browser/js_data.ddc.dill --input-summary=org-dartlang-app:///packages/meta/meta.ddc.dill --input-summary=org-dartlang-app:///packages/binary_codec/binary_codec.ddc.dill --input-summary=org-dartlang-app:///packages/js/js.ddc.dill --input-summary=org-dartlang-app:///packages/meta/meta_meta.ddc.dill --input-summary=org-dartlang-app:///packages/jaspr/html.ddc.dill --source=package:platform_import_test/app.dart --source=package:platform_import_test/app.g.dart
[BUILDER] [ERROR] org-dartlang-app:///packages/platform_import_test/components/settings_content.imports.dart:4:1: Error: Error when reading 'org-dartlang-app:///packages/platform_import_test/generated/imports/_web.dart': File not found
[BUILDER] [ERROR] export '../generated/imports/_web.dart'
[BUILDER] [ERROR] ^
[BUILDER] [ERROR] AssetNotFoundException: platform_import_test|lib/components/settings_content.imports.ddc.dill
[BUILDER] [ERROR] Error compiling dartdevc module:platform_import_test|lib/components/settings_content.imports.ddc.js
[BUILDER] [ERROR]
[BUILDER] [ERROR] packages/platform_import_test/components/settings_content.imports.dart:4:1: Error: Error when reading 'packages/platform_import_test/generated/imports/_web.dart': File not found
[BUILDER] [ERROR] export '../generated/imports/_web.dart'
[BUILDER] [ERROR] ^
[BUILDER] [ERROR] Failed after 424ms

Expected Behavior
No such error.
Additional Context
Minimal example to reproduce the error: https://github.com/fusion44/jaspr_platform_import_test
The Dart LSP does find all files without problem.

In your repo it appears all files are generated correctly. Strange.
I will try to reproduce this.

Seems to be an issue with jaspr_web_compilers

Should work now after running pub upgrade


Toggle between OCS Provider Server Service Type

Update the storageCluster CR to set the type of service as nodePort or LoadBalancer
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2212773
Different cloud providers needs different annotations to be added in order to bring up the ocs provider server service in private network, hence we might need to create a load balancer externally, hence added a field in storageCluster CR to toggle between NodePort and LoadBalancer Service type
Ref: https://kubernetes.io/docs/concepts/services-networking/service/#loadbalancer

Can you also look at the failures and try to resolve them

/cherry-pick release-4.13

/cherry-pick release-4.12

/cherry-pick release-4.11


Create info.yaml as per Linux Foundation requirements

As per #1252
@schannamallu :
I have ran against yamllint so should be valid yaml
I don't yet have answers to all the questions for permitted value
some sections are empty where it was unclear what to add - others I've taken a guess
please review/comment on the PR
@mandy-chessell
Please review as above. Also I have sourced maintainers from MAINTAINERS.md - I don't have access to review the github lists. please suggest any corrections needed

Bad PR - pulling in incorrect local test changes


Modernize Python 2 code to get ready for Python 3

Use print() function in both Python 2 and Python 3

print() is a function in Python 3.

Old style exceptions --> new style for Python 3

Python 3 treats old style exceptions as syntax errors but new style exceptions work as expected in both Python 2 and Python 3.

Superseded by https://github.com/refack/GYP/commit/82c2f6e4df98d4826659616d90997a0bc06aacb1

I backported https://github.com/nodejs/node-gyp/pull/1335, so eh got all those fixes "for free".
I also added Pipfile with a lint script so pipenv run lint passes. Now we need to do some more delicate work.


Subnet creation permissioning

Context
For risk mitigation and upgradeability reasons, we are proposing early adopters to run fully segregated/independent gateways/registries. (It is also Raúl's personal opinion that the gateway/subnet division of labor is profoundly flawed today).
Unfortunately, it is still possible for anons to turn up to any gateway/registry and just create a subnet. If there's a vulnerability at the gateway level, this pathway could be exploited to attack and loot the actual production subnet.
Ask
A subnet creation permission mode in the gateway and subnet actor to plug this hole. We should have two choices:

unpermissioned (default, today's behaviour)
owner (only the owner is able to create subnets)

This PR updates the SubnetRegistry and a follow up will add the equivalent to Gateway.
Closes ENG-782.

Users can now pass env variable REGISTRY_CREATION_PRIVILEGES=owner to the actors deployment scripts to make it so only the owner of the registry can deploy new subnets.
Tests
Owner is 0x6b505cdccca34ae8eea5d382abad40d2afea74ad.
root@localhost:~/ipc# target/release/ipc-cli subnet create --from 0x06d51f735775c0ca404c337fb7b8b1973cfb509e --parent /r314159 --min-validators 1 --min-validator-stake 1 --bottomup-check-period 30 --supply-source-kind erc20 --supply-source-address 0x9bc7763d2808A3135EeF0A64CD0eDB2f468c421b --permission-mode federated
[2024-03-19T23:10:21Z INFO  ipc_provider::manager::evm::manager] creating subnet on evm with params: ConstructorParams { min_activation_collateral:<PHONE_NUMBER>000000000, min_validators: 1, bottom_up_check_period: 30, ipc_gateway_addr: 0x71052b43a9d41125d0e84059e83666c8ec21a285, active_validators_limit: 100, majority_percentage: 67, consensus: 0, power_scale: 3, permission_mode: 1, supply_source: SupplySource { kind: 1, token_address: 0x9bc7763d2808a3135eef0a64cd0edb2f468c421b }, parent_id: SubnetID { root: 314159, route: [] } }
[2024-03-19T23:10:23Z ERROR ipc_cli] main process failed: error processing command Some(Subnet(SubnetCommandsArgs { command: Create(CreateSubnetArgs { from: Some("0x06d51f735775c0ca404c337fb7b8b1973cfb509e"), parent: "/r314159", min_validator_stake: 1.0, min_validators: 1, bottomup_check_period: 30, active_validators_limit: None, min_cross_msg_fee: 1e-6, permission_mode: Federated, supply_source_kind: ERC20, supply_source_address: Some("0x9bc7763d2808A3135EeF0A64CD0eDB2f468c421b") }) })): Contract call reverted with data: 0x

root@localhost:~/ipc# target/release/ipc-cli subnet create --from 0x6b505cdccca34ae8eea5d382abad40d2afea74ad --parent /r314159 --min-validators 1 --min-validator-stake 1 --bottomup-check-period 30 --supply-source-kind erc20 --supply-source-address 0x9bc7763d2808A3135EeF0A64CD0eDB2f468c421b --permission-mode federated
[2024-03-19T23:11:04Z INFO  ipc_provider::manager::evm::manager] creating subnet on evm with params: ConstructorParams { min_activation_collateral:<PHONE_NUMBER>000000000, min_validators: 1, bottom_up_check_period: 30, ipc_gateway_addr: 0x71052b43a9d41125d0e84059e83666c8ec21a285, active_validators_limit: 100, majority_percentage: 67, consensus: 0, power_scale: 3, permission_mode: 1, supply_source: SupplySource { kind: 1, token_address: 0x9bc7763d2808a3135eef0a64cd0edb2f468c421b }, parent_id: SubnetID { root: 314159, route: [] } }
[2024-03-19T23:12:29Z INFO  ipc_cli::commands::subnet::create] created subnet actor with id: /r314159/t410fgiiwuchbuaycp4wfz7ejqosxbk4k3jc3lajhbei


Fix for #20 (support data provider interfaces instead of TreeDataProvider)

https://github.com/vaadin-component-factory/selection-grid-flow/issues/20

I reverted the PR. At least it requires a demo with a dataprovider that is not a TreeDataProvider.
I think it's not working since internally the component is using TreeData:
https://github.com/vaadin-component-factory/selection-grid-flow/blob/main/selection-grid-flow/src/main/java/com/vaadin/componentfactory/selectiongrid/SelectionTreeGrid.java#L122
It's probably not needed for the selection but it's required for the focus method.

I reverted the PR. At least it requires a demo with a dataprovider that is not a TreeDataProvider.
I think it's not working since internally the component is using TreeData:
https://github.com/vaadin-component-factory/selection-grid-flow/blob/main/selection-grid-flow/src/main/java/com/vaadin/componentfactory/selectiongrid/SelectionTreeGrid.java#L122
It's probably not needed for the selection but it's required for the focus method.

Hm, did not see that. How about an additional interface HasTreeData, that provides the TreeData? If the data provider is a TreeDataProvider, it uses directly its TreeData. If not, then the data provider needs to implement HasTreeData. If that is also not the case, then it throws an UnsupportedOperationException.

Or maybe even more generic a "HasParentProvider" or similar, which takes an item and returns the parent for it. This would allow to use a hierarchical data provider, that not needs to use a TreeData instance.

You don't need to check this, it's already checked in the TreeGrid. It has to be a HierarchicalDataProvider.
What is the use case for not using a TreeDataProvider but that requires a dataprovider that requires a TreeData?


Error on Safari and Firefox: "ReferenceError: Can't find variable: TouchEvent"

Error in Safari and Firefox:
"ReferenceError: Can't find variable: TouchEvent"

Trying to pull a quick fix, but I have no access to Pull Request.

Thanks for reporting this issue. Could you provide more details of when it happens / what exactly isn't working?
I checked the Plunker example page on Firefox 60.0 and from what I can see, it seems to be working fine. I don't have access to any Mac machine at the moment, so I can't check Safari.

Also, you should be able to create a pull request by first forking the repository, making the fix in your fork and then creating a pull request back to the main repository.

The bug happens when I try slider the component.
I am using:
MacOS 10.12.6
Firefox 59.0.2 (64-bit)
Safari Version 11.1
I will try again the pull request.

I managed to reproduce this issue by using an older version of Firefox (36.0). I created my own fix for this which seems to work. Please let me know if this also fixes it on your end @angelomotta.

I tested my fix also on Safari 11 on Mac OSX 10.13 using http://crossbrowsertesting.com/ and it seems to work fine.
I think this is fixed now, so I'll close this issue. If there are any more problems, it can be reopened.


Update Set-DtcClusterTMMapping.md

Issue - "fill in the blank" assignments - Description has been updated now.

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

File
Status
Preview URL
Details

docset/winserver2012r2-ps/msdtc/Set-DtcClusterTMMapping.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dariomws306  @tseward
Peer review needed for this PR. Thanks.


Simplify VisitOperator for VisitConstOperator

Don't generate a format!-per-opcode but instead delegate to a single helper method.

I just ran cargo bloat on the most recent Wasmi v0.40 version that uses the new wasmparser v0.221 and const-expr validator still is marked as bloated:
    Analyzing target/release/wasmi_cli

 File  .text    Size        Crate Name
 1.2%   2.1% 49.0KiB         wast <wast::core::expr::Instruction as wast::parser::Parse>::parse::{{closure}}
 1.0%   1.7% 38.8KiB   wasmparser wasmparser::validator::core::ModuleState::check_const_expr
 0.9%   1.5% 35.6KiB wasm_encoder <wasm_encoder::core::code::Instruction as wasm_encoder::Encode>::encode
 0.8%   1.4% 31.6KiB    wasmi_ir? <wasmi_ir::enum::Instruction as core::fmt::Debug>::fmt
 0.7%   1.3% 29.3KiB        wasmi wasmi::engine::translator::driver::FuncTranslationDriver<T>::translate_operators
 0.6%   1.1% 26.3KiB        wasmi wasmi::engine::executor::instrs::execute_instrs
 0.5%   1.0% 22.3KiB         wast wast::core::binary::encode
 0.5%   1.0% 22.2KiB    wasmi_cli wasmi_cli::context::Context::new
 0.5%   0.9% 21.5KiB        wasmi wasmi::module::parser::buffered::<impl wasmi::module::parser::ModuleParser>::parse_buffered_impl
 0.4%   0.7% 16.0KiB        wasmi wasmi::engine::translator::driver::FuncTranslationDriver<T>::translate_operators
 0.4%   0.7% 16.0KiB   wasmi_cli? <wasmi_cli::args::Args as clap_builder::derive::Args>::augment_args
 0.4%   0.7% 15.4KiB clap_builder clap_builder::derive::Parser::parse
 0.4%   0.7% 15.4KiB clap_builder clap_builder::parser::parser::Parser::get_matches_with
 0.4%   0.7% 15.3KiB          wat wat::Parser::_parse_str
 0.3%   0.6% 13.8KiB clap_builder clap_builder::parser::validator::Validator::validate
 0.3%   0.6% 13.2KiB         wast <wast::core::custom::Custom as wast::parser::Parse>::parse
 0.3%   0.5% 12.5KiB        wasmi wasmi::engine::translator::driver::FuncTranslationDriver<T>::translate_operators
 0.3%   0.5% 12.2KiB  wasmparser? <wasmparser::readers::core::operators::Operator as core::fmt::Debug>::fmt
 0.3%   0.5% 11.3KiB         wast wast::core::module::ModuleField::parse_remaining
 0.2%   0.4% 10.2KiB        wasmi wasmi::engine::translator::driver::FuncTranslationDriver<T>::translate_operators
46.7%  81.8%  1.9MiB              And 5298 smaller methods. Use -n N to show more.
57.1% 100.0%  2.3MiB              .text section size, the file size is 4.0MiB

Ah yeah I'd probably try to get the old wasmparser pruned first to confirm it's not coming from there. If it's still here though happy to help dig in


Implementing CouplingMap.eq

Summary
Based on discussion with @mtreinish , implements CouplingMap.__eq__ to check whether the underlying graphs are isomorphic.
Details and comments
The method simply returns rustworkx.is_isomorphic(self.graph, other.graph). The PR also adds a simple test case to verify that this overriding the standard object equality implementation of __eq__.

I've added some comments to the test case, and added the release note.

the test failure is unrelated to this PR - it's because Aer 0.12 has just released - but fortuitously it's still aimed at you, Dan ;)
test.python.pulse.test_block.TestBlockOperation.test_execute_block
------------------------------------------------------------------

Captured traceback:
~~~~~~~~~~~~~~~~~~~
    Traceback (most recent call last):

      File "/tmp/terra-tests/test/python/pulse/test_block.py", line 382, in test_execute_block
    test_result = backend.run(sched_block).result()

      File "/home/vsts/work/1/s/qiskit/providers/fake_provider/fake_backend.py", line 552, in run
    sim = aer.Aer.get_backend("pulse_simulator")

      File "/home/vsts/work/1/s/test-job/lib/python3.7/site-packages/qiskit_aer/aerprovider.py", line 70, in get_backend
    stacklevel=2,

    DeprecationWarning: The Pulse simulator backend in Qiskit Aer is deprecated and will be removed in a future release. Instead the qiskit-dynamics library should be used instead for simulating at the pulse level.

https://github.com/Qiskit/qiskit-terra/pull/9767 will fix that test failure

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 85.361%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
68056

Relevant Lines:
79727

💛  - Coveralls


Cursor not moving

Offset0 : 48c
Offset1 : 27c
Offset2 : f8
Offset3 : 7b0
Offset4 : 70
ThreadOffset : -32C
Tried with this offset from another solution.
Also could you explain what exactly I have to look for to find the timerpointer?

I have the same problem. All that happens is my mouse pointer gets limited to the osu! window, if I try to move it out, it teleports back. If I have selected the osu!bot window, the mouse cursor gets stuck and I have to use my tablet to get it on the osu! window. The bot dosen't actually do any 'auto-ing', and I can't find any source of help. The closest things I found are demos and people showing their own osu!bots.

Osu!Bot V2 is NOT being supported any more! Please go to Osu!Bot V3 for a continued version.


Docs and tidyup

This is a restructuring of the project - it's still ongoing, so please do not delete this branch once it's done.
The initial HTML docs are in docs/build/html/index.html; open that up in a browser to view them. I'll do more in-depth docs for the each library soon, but the main 'this is what this one does' docstring is in there. We can also host the documentation on Github Pages.

I've merged updates from your branch to the main branch.


add run to right click menu

Although I have speed keys for running with dmenu shortcuts don't always work and need a fall back mechanism


Add seed Command

Description
Requires #287

Testing

Checklist
Things need to be done before this Pull Request can be merged.

[x] Code is well-formatted and adheres to project style guidelines: cargo fmt
[x] Code does not produce any clippy warnings cargo clippy

Looks good now.
Thanks @Commandcracker


[Bug]: Cyclic dependency error is not thrown for cyclic dependency of js object with widget

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When JS object is edited to create a cyclic dependency with a widget, no toast message is thrown, but when the js object is edited again to remove the cyclic dependency, then we get error.
https://github.com/user-attachments/assets/c63ca5ca-0a8c-4e20-927d-06e597674931
Ideally, we should get error only when we have cyclic dependency and not on the removal of it.
Steps To Reproduce

Add a text widget on canvas
Add text widget's textData as {{JSObject1.myFun1.data}}
Go to JsObject1
Edit myFun1 to return Text1.text

Public Sample App
No response
Environment
Production
Severity
Medium (Frustrating UX)
Issue video log
No response
Version
Self hosted - v1.45

hey @sneha122 , I am debugging the issue and from what I understand, the current behavior is that when there's a circular dependency, we aren't seeing an error immediately. Instead, the error toast message only appears when we comment it out. Ideally, we should trigger the toast error message as soon as the circular dependency is detected, not just after commenting it out.
Is that correct, or am I missing something?

hey @sneha122 , I would like to work on this issue, could you please assign it to me

Thanks @ALOK9442 for your interest in this issue! Can you please describe the approach you have in mind in order to resolve this issue?


Add Kafka headers as column

This PR adds the internal _headers field to the kafka connector.
As type a map with key String and value array of byte[] is used.
For better review the changes are splited into two commits:

Extract the type definitions from the enum that describes the internal fields to a map that is initialized in the constructor where the TypeManager can be used.
Add the Header column and adapt the KafkaRecordCursor

I would appreciate if someone could review the code.

Thanks for the review @losipiuk,
I agree that it feels odd to have the static enum and a representation as class with type at runtime. Therefore I'd like to get rid of the enum.
@findepi WDYT?

Thanks! I did not notice previously that you already made a change to the test :)


Make available Mongo version to 4.2

[Enter feedback here]
Added support for Mongo version to 4.2 and update the documentation. A couple of places shows only 3.2 and 3.6
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dc075caf-933b-cda0-703b-6b700f1436f8
Version Independent ID: 102753f5-d177-627e-c14c-e88df91c8000
Content: Resource Manager templates for Azure Cosmos DB API for MongoDB
Content Source: articles/cosmos-db/mongodb/resource-manager-template-samples.md
Service: cosmos-db
Sub-service: cosmosdb-mongo
GitHub Login: @markjbrown
Microsoft Alias: mjbrown

@jjpinto Thank you for your feedback. We are reviewing this and will get back shortly.

@markjbrown is the documentation only for Mongo version 3.2 or 3.6

Quick update.
The ARM and Bicep samples within our docs are actually located in the Azure Quickstart GitHub Repository here.
The samples on GitHub have both regular Azure Cloud and Gov Cloud included parameter files. During PR these are built and deployed to both clouds and must pass to be merged. The PR's to update these to 4.2 are failing because we do not yet have support for Mongo 4.2 in our Gov Cloud. End result is we cannot update our samples until that support gets deployed.
I have an ask into my team for an ETA on when this will happen. Once the support is deployed we will merge these changes which will update the docs. I don't have an ETA on this unfortunately so cannot provide that here.
I am however going to close this docs issue for now as we have PR's in place to do this update.
Thanks for your feedback. We very much appreciate it.
#please-close


Cannot import 'Literal'

When loading the last sample code listed in your README.md I'm getting an error that usually comes up with python 3.8 and above but I'm not as familiar with its use in calendar-view.
The setup.py indicates this is for python 3.6 and above but I didn't see a tox.ini file so there was no version checking when I loaded this.
"ImportError: cannot import name 'Literal' from 'typing' (/Users/user_name/opt/anaconda3/envs/Task-scheduler-problem/lib/python3.7/typing.py)"
Sample Code I used
from calendar_view.calendar import Calendar
from calendar_view.config import style
from calendar_view.core import data
from calendar_view.core.event import Event

style.hour_height = 80
style.event_notes_color = '#7F7F7F'

config = data.CalendarConfig(
    lang='en',
    title='Massage. Antonio',
    dates='2022-06-20 - 2022-06-24',
    show_year=True,
    mode='working_hours',
    title_vertical_align='top'
)
events = [
    Event(day='2022-06-20', start='11:00', end='12:00', title='Jesse Tyson'),
    Event(day='2022-06-20', start='12:30', end='14:00', title='Karry', notes='No music'),
    Event(day='2022-06-20', start='15:00', end='17:00', title='Taylor Davis',
          notes='Ask about the shin that hurts last time.'),
    Event(day='2022-06-20', start='17:30', end='18:30', title='Jose Hope'),

    Event(day='2022-06-22', start='10:00', end='12:00', title='Annabell Moore',
          notes='A therapist for her mother:\n+4487498375 Nick Adams'),
    Event(day='2022-06-22', start='12:30', end='14:00', title='Carlos Cassidy'),
    Event(day='2022-06-22', start='15:00', end='17:00', title='Joe'),
    Event(day='2022-06-22', start='17:30', end='18:30', title='Jose Hope'),

    Event(day='2022-06-23', start='10:00', end='11:00', title='Elena Miller'),
    Event(day='2022-06-23', start='11:30', end='13:30', title='Karry', notes='No music'),
    Event(day='2022-06-23', start='15:00', end='16:30', title='Mia Williams'),
    Event(day='2022-06-23', start='17:00', end='18:00', title='Xander'),
]

calendar = Calendar.build(config)
calendar.add_events(events)
calendar.save("massage.png")

Perhaps I'm implementing it wrong?
If you let me know how I can fix it I'd like to submit a PR if you are open to adding a collaborator.

Hi @datatalking . Thank you for opening the issue.
I cannot reproduce the bug on Python 3.8.17. Please see the attached screenshot.

Can you please provide more details or try starting the project from scratch?
The "Literal" is not part of the calendar-view package. The bug is most probably not related to this package.

As far as I know, tox.ini is a virtual environment file. I didn't add this to the project, because it is not part of the application. And I used to use venv for virtual environments.
python -m venv .venv

Yes, I am open to collaborating. Please, create the pull request with the fix and I will be happy to see it in the next release.


executive_smach_visualization: 2.1.0-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository executive_smach_visualization to 2.1.0-0:

upstream repository: https://github.com/tork-a/executive_smach_visualization.git
release repository: https://github.com/tork-a/executive_smach_visualization-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.26
previous version for package: null

executive_smach_visualization

No changes

smach_viewer
* support for Qt5 (Kinetic)
* update CMakeLists.txt, package.xml, setup.py, smach_viewer.py for new xdot structure
* add necessary lines in xdot/__init__.py https://github.com/jbohren/xdot/pull/14
* copy xdot from https://github.com/jbohren/xdot, since system xdot is released in rosdep key https://github.com/ros/rosdistro/pull/4976
* wx viewer: checking to make sure item urls are strings to prevent crash
* Contributors: Jonathan Bohren, Kei Okada

See https://discourse.ros.org/t/releasing-repositories-form-other-people/1797/10