Updating based on work this past year

Updated progs_dump and added it to the project folder. The start map has been updated and tested in this new folder setup.

I am updating the project. This is to get everything inline with where I currently am with the project. I need to make a few changes to the start map to align with the files, but I removed the map end until I know exactly what the next map naming convention is.

I am updating the project. This is to get everything inline with where I currently am with the project. I need to make a few changes to the start map to align with the files, but I removed the map end until I know exactly what the next map naming convention is.


leaderboard sorting error

leaderboard is not in ascending order, there must be some error in script.Current error can be seen with vishal pani's score, he is with 1370 points but is ranked 18.

i would like to do it

please do the previous assigned issue first

please give this issue to someone else :p

it wasn't assigned to you :stuck_out_tongue:

@Anupam-dagar I would really love to work on this issue.

@aayushjoglekar you gotta work fast
assigned


FR: Allow use of local whisper instance

I am using the local version of whisper: https://github.com/ahmetoner/whisper-asr-webservice/
Can or is there be a setting to use the local endpoint? I might have missed it!

Or perhaps using this port of OpenAI's Whisper model in C/C++: https://github.com/ggerganov/whisper.cpp

@brimwats how is the performance of the local whisper model. Which model do you use? and whats the hardware specs?

@tmfelwu you it's pretty easy to test.
I was going to use https://github.com/matusstas/openai-whisper-microservice but the one brimwats posted might be a better choice. As long as these services support the OpenAI whisper API then switching the URL out should be no problem, otherwise adapters will be needed.

It doesn't seem like too big of a lift, I'm forking and tracking here: https://github.com/dahifi/whisper-asr-webservice-obsidian-plugin/issues/1

Not many people know how to start the Whisper service on their local. So, I think it's better to keep it connected to the API for now until we get more traction from the community.

Not many people know how to start the Whisper service on their local. So, I think it's better to keep it connected to the API for now until we get more traction from the community.

Conversely, you might get more traction, if you could run Whisper locally.

@nikdanilov I think anyone that's using Obsidian over Notion or any of the other PKM systems does so out of choice. Notion may be easier to use but it's closed source, they lock in your data and make it hard to export & the service is proprietary. Using an API to do STT is the sort of thing Notion would advocate, not Obsidian
IMO Obsidian users:

They probably don't want to share their data with a corporate
They're tech savy - Setting up and enabling something as simple as a Template using Templater is pretty complex when you compare it with a couple of clicks needed in Notion

@MSBack has hit the nail on the head. I think the missing link in most of the community AI plugins for obsidian is that they're all using openai API calls to do all the heavy lifting, there are no local options and it makes no sense given the etho/principles behind Obsidian.md

why not have both the options, openai api's are alreaady implemented, lets have a switch to change from remote to local instance

Currently, Open AI say that they don't train there models on API calls and have a 30-day data retention policy source. But that could change. My obsidian notes are about the last thing I want going in the training data. A self-host option, even if it's hard to set up, would be deeply appreciated.

To run local whisper model and connect with plugin:
https://gist.github.com/gavrilov/4537a569b7fa8e20e64a199e924d458a

Conversely, you might get more traction, if you could run Whisper locally.
Exactly. the only reason i am Not using this plugin is because it does not function with a local Whisper installation.
looking into gavrilovs solution, but really think it would make more sense to have it already built in, for all the reasons listed above, and then some.
I thank the author for the effort and share, regardless

gavrilovs above solution worked beautifully. It also simplified the install process for me. I used a Conda environment, instead of venv, but was still able to easily follow the instructions. Thank you both, gavrilov and nikdanilov.  Also, Whisper on a 12GB 3060 is almost instant (not real-time, i mean in transcribing the recording) and the accuracy is amazing. I am quite impressed.

I am using the local version of whisper: https://github.com/ahmetoner/whisper-asr-webservice/
Can or is there be a setting to use the local endpoint? I might have missed it!

I built a plugin around this a bit ago - instructions for setting up locally with Docker are linked
https://github.com/djmango/obsidian-transcription

I am using the local version of whisper: https://github.com/ahmetoner/whisper-asr-webservice/
Can or is there be a setting to use the local endpoint? I might have missed it!

I built a plugin around this a bit ago - instructions for setting up locally with Docker are linked
https://github.com/djmango/obsidian-transcription

Dude! @djmango
This worked flawlessly. Running the Docker container on my Macbook and it worked no problem. Will have to play around with the models -- output isn't up to par with whatever AI Commander's transcription command prompt is, but thank you so much for this.

I am using the local version of whisper: https://github.com/ahmetoner/whisper-asr-webservice/
Can or is there be a setting to use the local endpoint? I might have missed it!

I built a plugin around this a bit ago - instructions for setting up locally with Docker are linked
https://github.com/djmango/obsidian-transcription

Dude! @djmango
This worked flawlessly. Running the Docker container on my Macbook and it worked no problem. Will have to play around with the models -- output isn't up to par with whatever AI Commander's transcription command prompt is, but thank you so much for this.

That's great to hear! I poked around the source of AI Commander (I was pleasantly surprised to see my stackoverflow solution linked as a source 😆), it seems to be using the OpenAI Whisper API, which I believe uses the openai/whisper-large-v2 model.
Given sufficient memory, 16gb if I recall correctly, you should be able to run that same model on Whisper ASR service - at the cost of speed, but I found quality of output to be worth it. I ended up rolling my own webservice so that I can run it on my mobile devices too, but I know use cases differ significantly in the notetaking/knowledge work space.
What do you suggest as to other tools/apps I should build transcription plugins for? I'm looking to expand the functionality of Obsidian Transcription  significantly over the next two weeks, and from there build a wider library of plugins so more people can use the tool.

@djmango I'm building pipelines for Discord using Steamship right now. If you're interested in touching base I can send you a link to my Discord server.

@djmango I'm building pipelines for Discord using Steamship right now. If you're interested in touching base I can send you a link to my Discord server.

Please do!

@djmango I sent a forwarded GH email to your swiftlink account with the invite

I never got this invite
From: BCM @.>
Date: Tuesday, October 10, 2023 at 9:27 PM
To: nikdanilov/whisper-obsidian-plugin @.>
Cc: Sulaiman Ghori @.>, Mention @.>
Subject: Re: [nikdanilov/whisper-obsidian-plugin] FR: Allow use of local whisper instance (Issue #2)
@djmangohttps://github.com/djmango I sent a forwarded GH email to your swiftlink account with the invite
—
Reply to this email directly, view it on GitHubhttps://github.com/nikdanilov/whisper-obsidian-plugin/issues/2#issuecomment-1756594760, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGZNIXF55L4HFGHYOGP3FOLX6XYXPAVCNFSM6AAAAAAWXQ3T5CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJWGU4TINZWGA.
You are receiving this because you were mentioned.Message ID: @.***>

Here's the Steamship Discord. You'll see me in there and we can connect: https://discord.gg/GaEeyGG9

I've made a docker image to run it as a local server (based on the project that @gavrilov is referring to in their gist)
Install Docker and run:
docker run -d -p <IP_ADDRESS>:8000:8000 -v ./.cache:/root/.cache:rw didmar/whisper-api-server:latest

Then change the URL to http://localhost:8000/v1/audio/transcriptions in the plugin settings, and that's it!

+1 for the docker image @didmar. working great for me!

+1 for the docker image @didmar. working great for me!

I think a better solution would be using huggingface transformer in the browser directly. No installation, No dependency, totally local
https://huggingface.co/spaces/Xenova/whisper-web

I think a better solution would be using huggingface transformer in the browser directly. No installation, No dependency, totally local
https://huggingface.co/spaces/Xenova/whisper-web

Folks 👋
I appreciate your feedback and understand the importance of using the local Whisper model. I'm currently exploring options to add this into the plugin. Thank you for your patience and support! 😊

I've managed to get this working using https://github.com/ggerganov/whisper.cpp .
I couldn't find if anyone else had got it working so I thought I'd leave a short guide for how I did it.
I'm running the server in an incus container on a linux (pop-os) host. I used a container because i have an AMD 6800 xt and Rocm does not play well on pop-os. I used the following tutorial to set up my container (https://discuss.linuxcontainers.org/t/ai-tutorial-llama-cpp-and-ollama-servers-plugins-for-vs-code-vs-codium-and-intellij/19744) you just need to swap out the llama.cpp stuff for whisper.cpp (extra make commands are needed to compile for a GPU).
whisper.cpp has a server funtion which provides the endpoint that we can point to with the API URL.
I start the whisper.cpp server with ./server -m /home//ubuntu/whisper.cpp/models/ggml-medium.en.bin --host <IP_ADDRESS> --convert
As for the plugin settings in Obsidian:

API key: <Does not matter as long as it's something>
API URL: for me it is http://<container IP>:8080/inference
Model: blank
Prompt: blank
Language: blank

I hope this helps others!

I couldn't find if anyone else had got it working

https://github.com/nikdanilov/whisper-obsidian-plugin/issues/2#issuecomment-1644647644
Well, all you had to do was check the messages above yours, as a working solution has been shared and verified quite some time back. a guide, if accurate, is always welcome of course.

Ahh, I assumed because the issue was left open it wasn't a fully solved solution.
I did lean on this thread for guidance, including gavrilov's excellent example, however his solution uses the full-fat whisper models and (from my skimming of his code) generates his own server to handle requests. I think for less experienced users, a pre-established project like whisper.cpp offers a lot of extra troubleshooting support, and having something prepackaged that also runs realatively quickly on a CPU opens up usability to a lot more people.
Cheers!

The api url I had to be  http://<IP_ADDRESS>:8000/inference instead of just <IP_ADDRESS>:8000/inference for me to get this running.


Add ARM dependencies

/azp run

I know this is a draft PR. What I have found so far:

missing images for rook
allow_mismatch flag not specified for registry:2.8.0 image (this is what we have in develop)
calico pods not working properly:

kube-system            calico-node-5vqcp                                                       0/1     Running            0               65m
kube-system            calico-node-l72vd                                                       0/1     Running            0               53m
kube-system            calico-node-pvpzm                                                       0/1     Running            0               53m

Events:
  Type     Reason     Age   From               Message
  ----     ------     ----  ----               -------
  Normal   Scheduled  43m   default-scheduler  Successfully assigned kube-system/calico-node-l72vd to ci-devarmawsalmacalico-kubernetes-node-vm-0
  Normal   Pulling    43m   kubelet            Pulling image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/cni:v3.20.3"
  Normal   Pulled     43m   kubelet            Successfully pulled image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/cni:v3.20.3" in 1.370066396s
  Normal   Created    43m   kubelet            Created container upgrade-ipam
  Normal   Started    43m   kubelet            Started container upgrade-ipam
  Normal   Started    43m   kubelet            Started container install-cni
  Normal   Pulling    43m   kubelet            Pulling image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/cni:v3.20.3"
  Normal   Pulled     43m   kubelet            Successfully pulled image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/cni:v3.20.3" in 59.674753ms
  Normal   Created    43m   kubelet            Created container install-cni
  Normal   Pulling    43m   kubelet            Pulling image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/pod2daemon-flexvol:v3.20.3"
  Normal   Pulled     43m   kubelet            Successfully pulled image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/pod2daemon-flexvol:v3.20.3" in 444.5771ms
  Normal   Created    43m   kubelet            Created container flexvol-driver
  Normal   Started    43m   kubelet            Started container flexvol-driver
  Normal   Pulling    43m   kubelet            Pulling image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/node:v3.20.3"
  Warning  Unhealthy  43m   kubelet            Readiness probe failed: calico/node is not ready: BIRD is not ready: Failed to stat() nodename file: stat /var/lib/calico/nodename: no such file or directory
  Normal   Created    43m   kubelet            Created container calico-node
  Normal   Started    43m   kubelet            Started container calico-node
  Normal   Pulled     43m   kubelet            Successfully pulled image "ci-devarmawsalmacalico-repository-vm-0:5000/calico/node:v3.20.3" in 1.749567977s
  Warning  Unhealthy  43m   kubelet            Readiness probe failed: 2022-06-06 10:02:23.357 [INFO][808] confd/health.go 180: Number of node(s) with BGP peering established = 1
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  43m  kubelet  Readiness probe failed: 2022-06-06 10:02:23.558 [INFO][818] confd/health.go 180: Number of node(s) with BGP peering established = 1
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  43m  kubelet  Readiness probe failed: 2022-06-06 10:02:33.419 [INFO][1862] confd/health.go 180: Number of node(s) with BGP peering established = 2
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  43m  kubelet  Readiness probe failed: 2022-06-06 10:02:43.424 [INFO][2812] confd/health.go 180: Number of node(s) with BGP peering established = 2
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  43m  kubelet  Liveness probe failed: calico/node is not ready: Felix is not live: Get "http://localhost:9099/liveness": dial tcp [::1]:9099: connect: connection refused
  Warning  Unhealthy  42m  kubelet  Readiness probe failed: 2022-06-06 10:02:53.379 [INFO][3838] confd/health.go 180: Number of node(s) with BGP peering established = 2
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  42m  kubelet  Readiness probe failed: 2022-06-06 10:03:03.413 [INFO][4889] confd/health.go 180: Number of node(s) with BGP peering established = 2
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  42m  kubelet  Readiness probe failed: 2022-06-06 10:03:13.394 [INFO][5905] confd/health.go 180: Number of node(s) with BGP peering established = 2
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  42m  kubelet  Readiness probe failed: 2022-06-06 10:03:23.447 [INFO][6878] confd/health.go 180: Number of node(s) with BGP peering established = 2
calico/node is not ready: felix is not ready: readiness probe reporting 503
  Warning  Unhealthy  3m28s (x285 over 42m)  kubelet  (combined from similar events): Readiness probe failed: 2022-06-06 10:42:23.377 [INFO][225930] confd/health.go 180: Number of node(s) with BGP peering established = 2
calico/node is not ready: felix is not ready: readiness probe reporting 503

canal deployment failed on TASK [kubernetes_master : Wait for CNI plugin to become ready]

kube-system   canal-xt2h4                                                            1/2     Running   0          75m

Events:
  Type     Reason     Age                  From     Message
  ----     ------     ----                 ----     -------
  Warning  Unhealthy  72s (x493 over 76m)  kubelet  Readiness probe failed: HTTP probe failed with statuscode: 503


Possibility to see the names of the author

Feature Request
Is your feature request related to a problem? Please describe.
Some people that have used ASReview would like to have an option in the settings so they can see the names of the authors of the article. They think the current amount of information is very small.
Describe the solution you'd like
A setting to show the names of the authors during the screening proces under the title. Respondents didn't say anything about if the names should also be taken into account when training the model, it's good to think about if this should be taken into account when training the model.

Thanks for reporting this feature. Many people ask us about showing authors or other information. We believe this will induce bias in the decision making. @bmkramer would you mind explaining why? :-)

As @Rensvandeschoot indicated, we believe decisions to include/exclude papers should be made based solely on the content of the paper (relevance and methodological quality), and not be influenced by proxies like journal reputation or author names.
If a publication would otherwise not be included based on content, should it be included because it's from a particular research group, or because it's published in a certain journal? And vice versa, should name recognition or journal brand influence inclusion of a paper that is in itself seems relevant and methodologically sound?
Of course, when clicking through to the paper from ASReview, users will still be able to check the names of the authors and the publication venue. We do not prevent that, but since we want to avoid bias in screening as much as possible, we do not show this information in ASReview itself.


Bug fix for maven in Artifactory Uploader

Bug fix for maven.Mavan name is created in Artifactory uploader cli while running python packages.

Waiting to confirm by looking at a manual test case, which fixes this.

Waiting to confirm by looking at a manual test case, which fixes this.

confirmed and works.


Cannot build

Hello,
after following your instructions for Gradle dependencies, I get this compilation error:

Dependency com.mindscapehq.android:raygun4android:1.2.0 on project Application resolves to an APK archive which is not supported as a compilation dependency. File: C:\Users\Igor.gradle\caches\modules-2\files-2.1\com.mindscapehq.android\raygun4android\1.2.0\762ffb14c64e8dffedf6ac6f0431d83755caefc3\raygun4android-1.2.0.apk

Please advise,
Igor

Hi Igor,
It appears there has been an update or some breaking change which has made the previous method not work. We are investigating this, but would you mind trying this in your build.gradle:
dependencies {
    compile 'com.google.code.gson:gson:2.1'
    compile 'com.mindscapehq.android:raygun4android:1.3.0:sources'

Regards,
Callum Gavin
Mindscape Limited

Yes, it works.  Thank you.


Bug in interpreting correctly parsed SQL with aliases

Describe the bug
Some statements that involve subqueries with an alias, are not evaluated correctly when the subquery contains both group by and order by.
To Reproduce
With datafusion-cli:
❯ select a from (select a from (select 1 as a) as foo group by 1 order by a) as c;
SchemaError(FieldNotFound { field: Column { relation: Some("foo"), name: "a" }, valid_fields: Some([Column { relation: Some("c"), name: "a" }]) })

Expected behavior
❯ select a from (select a from (select 1 as a) as foo group by 1 order by a) as c;
+---+
| a |
+---+
| 1 |
+---+
1 row in set. Query took 0.005 seconds.

(it works on postgres and sqlite)
Additional context
Same happens if there is an actual table called foo:
❯ create table foo as select 1 as a;
0 rows in set. Query took 0.032 seconds.
❯ select a from (select a from foo group by 1 order by a) as c;
SchemaError(FieldNotFound { field: Column { relation: Some("foo"), name: "a" }, valid_fields: Some([Column { relation: Some("c"), name: "a" }]) })

Interestingly if I remove as c in the outer expression it works:
❯ select a from (select a from (select 1 as a) as foo group by 1 order by a);
+---+
| a |
+---+
| 1 |
+---+
1 row in set. Query took 0.005 seconds.

It's also interesting that using positional column references in the order by works:
❯ select a from (select a from (select 1 as a) as foo group by 1 order by 1) as c;
+---+
| a |
+---+
| 1 |
+---+
1 row in set. Query took 0.005 seconds.

Furthermore, the bug reproduces only if group by a and order by a are both present:
❯ select a from (select a from (select 1 as a) as foo group by a) as c;
+---+
| a |
+---+
| 1 |
+---+
1 row in set. Query took 0.005 seconds.
❯ select a from (select a from (select 1 as a) as foo order by a) as c;
+---+
| a |
+---+
| 1 |
+---+
1 row in set. Query took 0.004 seconds.

It appears that the alias applied to a subquery somehow affects the field references inside the subquery. For example this this should return an error because c is not defined inside the select a from foo group by 1 order by c.a query:
❯ select a from (select a from foo group by 1 order by c.a) as c;
+---+
| a |
+---+
| 1 |
+---+
1 row in set. Query took 0.024 seconds.

I tried to look at the parsed SQL AST to see if the bug was in the parser or in the way the AST is interpreted (in order to know where to file the bug):
I ran:
use sqlparser::dialect::PostgreSqlDialect;
use sqlparser::parser::Parser;

fn main() {
    let dialect = PostgreSqlDialect {}; // or AnsiDialect, or your own dialect ...

    let sql = "select a from (select a from (select 1 as a) as foo group by a order by 1) as c";
    let ast = Parser::parse_sql(&dialect, sql).unwrap();
    println!("NUM: {:?}", ast[0]);

    let sql = "select a from (select a from (select 1 as a) as foo group by a order by a) as c";
    let ast = Parser::parse_sql(&dialect, sql).unwrap();
    println!("SYM: {:?}", ast[0]);
}

and got the following output:
NUM: Query(Query { with: None, body: Select(Select { distinct: false, top: None, projection: [UnnamedExpr(Identifier(Ident { value: "a", quote_style: None }))], into: None, from: [TableWithJoins { relation: Derived { lateral: false, subquery: Query { with: None, body: Select(Select { distinct: false, top: None, projection: [UnnamedExpr(Identifier(Ident { value: "a", quote_style: None }))], into: None, from: [TableWithJoins { relation: Derived { lateral: false, subquery: Query { with: None, body: Select(Select { distinct: false, top: None, projection: [ExprWithAlias { expr: Value(Number("1", false)), alias: Ident { value: "a", quote_style: None } }], into: None, from: [], lateral_views: [], selection: None, group_by: [], cluster_by: [], distribute_by: [], sort_by: [], having: None, qualify: None }), order_by: [], limit: None, offset: None, fetch: None, lock: None }, alias: Some(TableAlias { name: Ident { value: "foo", quote_style: None }, columns: [] }) }, joins: [] }], lateral_views: [], selection: None, group_by: [Identifier(Ident { value: "a", quote_style: None })], cluster_by: [], distribute_by: [], sort_by: [], having: None, qualify: None }), order_by: [OrderByExpr { expr: Value(Number("1", false)), asc: None, nulls_first: None }], limit: None, offset: None, fetch: None, lock: None }, alias: None }, joins: [] }], lateral_views: [], selection: None, group_by: [], cluster_by: [], distribute_by: [], sort_by: [], having: None, qualify: None }), order_by: [], limit: None, offset: None, fetch: None, lock: None })
SYM: Query(Query { with: None, body: Select(Select { distinct: false, top: None, projection: [UnnamedExpr(Identifier(Ident { value: "a", quote_style: None }))], into: None, from: [TableWithJoins { relation: Derived { lateral: false, subquery: Query { with: None, body: Select(Select { distinct: false, top: None, projection: [UnnamedExpr(Identifier(Ident { value: "a", quote_style: None }))], into: None, from: [TableWithJoins { relation: Derived { lateral: false, subquery: Query { with: None, body: Select(Select { distinct: false, top: None, projection: [ExprWithAlias { expr: Value(Number("1", false)), alias: Ident { value: "a", quote_style: None } }], into: None, from: [], lateral_views: [], selection: None, group_by: [], cluster_by: [], distribute_by: [], sort_by: [], having: None, qualify: None }), order_by: [], limit: None, offset: None, fetch: None, lock: None }, alias: Some(TableAlias { name: Ident { value: "foo", quote_style: None }, columns: [] }) }, joins: [] }], lateral_views: [], selection: None, group_by: [Identifier(Ident { value: "a", quote_style: None })], cluster_by: [], distribute_by: [], sort_by: [], having: None, qualify: None }), order_by: [OrderByExpr { expr: Identifier(Ident { value: "a", quote_style: None }), asc: None, nulls_first: None }], limit: None, offset: None, fetch: None, lock: None }, alias: None }, joins: [] }], lateral_views: [], selection: None, group_by: [], cluster_by: [], distribute_by: [], sort_by: [], having: None, qualify: None }), order_by: [], limit: None, offset: None, fetch: None, lock: None })

wdiff shows only a difference in the OrderByExpr:
OrderByExpr { expr: [-Value(Number("1", false)),-] {+Identifier(Ident { value: "a", quote_style: None }),+} asc: None, nulls_first: None }

I think this shows that sqlparser is parsing the statement correctly and that the bug is in datafusion.

I will try to fix it.

look like planner sort build wrong.
create table foo as select 1 as a;
select a from (select a from foo group by 1 order by a) as c;

Projection: c.a
  Sort: foo.a ASC NULLS LAST
    Projection: foo.a, alias=c
      Aggregate: groupBy=[[foo.a]], aggr=[[]]
        TableScan: foo
Projection: c.a
  Projection: c.a, alias=c
    Projection: c.a
      Sort: foo.a ASC NULLS LAST
        Projection: foo.a, alias=c
          Aggregate: groupBy=[[foo.a]], aggr=[[]]
            TableScan: foo

we should Sort: c.a instead of Sort: foo.a


Use v1 branch of GOA

There's been a UUID issue that @aslakknutsen has fixed in his PR in GOA: https://github.com/goadesign/goa/pull/841
This change makes sure we benefit of this change by using the v1 branch of GOA.

Current coverage is 49.61% (diff: 100%)

Merging #413 into master will not change coverage

@@             master       #413   diff @@
==========================================
  Files            39         39
  Lines          1685       1685
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            836        836
  Misses          786        786
  Partials         63         63

Powered by Codecov. Last update fd204a0...9d76cf4


The localized 'If disabled' is different on Services page

Storage Explorer Version: 1.10.1
Build: 20191011.12
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Language: Chinese (zh-TW)
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open Settings panel -> Preference (Locale) -> Select Chinese (traditional) as application language -> Restart Storage Explorer.
Open Settings panel -> Open Services page.
Check the localized 'If disabled'.

Expect Experience:
The localized 'If disabled' is consistent.
Actual Experience:
The localized 'If disabled' is different.

More Info:

This issue doesn't reproduce in Chinese(zh-CN).
Only verified this issue in Chinese(zh-CN) and Chinese(zh-TW).

It's on linguistic review

We don't consider this to be causing confusion. Won't fix it.


Add map.has()

Adding map.has() to test if a key exists in a map. Previously it was not possible to distinguish between a key containing nil and an absent key.
> m = {'a':1, 'b':2}
> m.has('a')
true
> m.has('z')
false

Sure. I will change it if you prefer.


dynamic voxelization : RuntimeError: CUDA error: invalid configuration argument

environment： cuda11.0  pytorch:1.7.0  mmdetection3D:tag v0.10.0
I run "python ./tools/train.py ./configs/dynamic_voxelization/dv_pointpillars_secfpn_6x8_160e_kitti-3d-car.py ", then get error "RuntimeError: CUDA error: invalid configuration argument".Can you give some suggestions on this issue？Thanks
Traceback (most recent call last):
File "./tools/train.py", line 166, in
main()
File "./tools/train.py", line 155, in main
train_detector(
File "/opt/conda/lib/python3.8/site-packages/mmdet/apis/train.py", line 170, in train_detector
runner.run(data_loaders, cfg.workflow)
File "/opt/conda/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 125, in run
epoch_runner(data_loaders[i], **kwargs)
File "/opt/conda/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 50, in train
self.run_iter(data_batch, train_mode=True)
File "/opt/conda/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 29, in run_iter
outputs = self.model.train_step(data_batch, self.optimizer,
File "/opt/conda/lib/python3.8/site-packages/mmcv/parallel/data_parallel.py", line 67, in train_step
return self.module.train_step(*inputs[0], **kwargs[0])
File "/opt/conda/lib/python3.8/site-packages/mmdet/models/detectors/base.py", line 247, in train_step
losses = self(**data)
File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/opt/conda/lib/python3.8/site-packages/mmcv/runner/fp16_utils.py", line 84, in new_func
return old_func(*args, **kwargs)
File "/home/mmdetection3d/mmdet3d/models/detectors/base.py", line 59, in forward
return self.forward_train(**kwargs)
File "/home/mmdetection3d/mmdet3d/models/detectors/voxelnet.py", line 89, in forward_train
x = self.extract_feat(points, img_metas)
File "/home/mmdetection3d/mmdet3d/models/detectors/dynamic_voxelnet.py", line 39, in extract_feat
voxel_features, feature_coors = self.voxel_encoder(voxels, coors)
File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/opt/conda/lib/python3.8/site-packages/mmcv/runner/fp16_utils.py", line 164, in new_func
return old_func(*args, **kwargs)
File "/home/mmdetection3d/mmdet3d/models/voxel_encoders/pillar_encoder.py", line 301, in forward
voxel_feats, voxel_coors = self.pfn_scatter(point_feats, coors)
File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/mmdetection3d/mmdet3d/ops/voxel/scatter_points.py", line 112, in forward
voxel, voxel_coor = self.forward_single(
File "/home/mmdetection3d/mmdet3d/ops/voxel/scatter_points.py", line 90, in forward_single
voxels, voxel_coors, num_points = dynamic_scatter(
File "/home/mmdetection3d/mmdet3d/ops/voxel/scatter_points.py", line 37, in forward
results = dynamic_point_to_voxel_forward(points, coors, voxel_size,
RuntimeError: CUDA error: invalid configuration argument

Please refer to #306 .

Close due to duplication.


uk-select error attribute does nothing

Im using:
  <link rel="stylesheet"<EMAIL_ADDRESS>/>
  <script<EMAIL_ADDRESS>type="module"></script>
  <script<EMAIL_ADDRESS>type="module"></script>

and applying error to a uk-select seems to have no effect.  I expected a red danger state on the select?
<uk-select error uk-cloak>
      <option value="option1">Option 1</option>
      <option value="option2">Option 2</option>
      <option value="option3">Option 3</option>
      <option value="option4">Option 4</option>
      <option value="option5">Option 5</option>
    </uk-select>

I would love an answer to this also @sveltecult

Please eloborate, why you expect that applying error to a uk-select, results in red color?
Have you read that in the docs?

Yes, the docs say:

To indicate an error state in the form, simply add the error attribute. This will apply a “danger” state to the component, providing visual feedback to the user.

And unless I'm missing something no visual feedback is applied.  The example in the docs has some manually (as far as I can tell) added text above and below.

I have this error only when I try to render ul-select option dynanimcally

The original shadcn design does not have red color unless you focus on that element.


Optional inputs and outputs visualization.

Please add visualization of node's optional inputs/outputs which have empty name. Reference doc: here.

Can you please provide a sample file, repro steps, expected and actual.

Great thanks for quick solution!


Jimp contructor from JPEG Buffer.

Hi
I use img-scramble that use Jimp to manage file.
It works on little png files but as soon as I test with a jpeg about 700KB to 2MB (750x1300 px) It crashes.
I tracked the error and seems like it comes from this line
this.bitmap = JPEG.decode(data);
The image is valid, but can't be handled by Jimp.
Here is the complete error I got
/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:132
      throw err;
      ^

Error [ERR_UNHANDLED_ERROR]: Unhandled error. (marker was not found)
    at GridReadStream.emit (events.js:169:17)
    at GridReadStream._error (/var/www/html/rapp/node_modules/gridfs-stream/lib/readstream.js:231:8)
    at /var/www/html/rapp/node_modules/gridfs-stream/lib/readstream.js:184:28
    at err (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:414:14)
    at executeCallback (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:403:25)
    at executeOperation (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:421:7)
    at GridStore.read (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/gridfs/grid_store.js:766:10)
    at GridReadStream._push (/var/www/html/rapp/node_modules/gridfs-stream/lib/readstream.js:181:17)
    at /var/www/html/rapp/node_modules/gridfs-stream/lib/readstream.js:131:42
    at result (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:413:17)
    at executeCallback (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:405:9)
    at /var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/gridfs/grid_store.js:926:7
    at /var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/gridfs/grid_store.js:1230:7
    at result (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:413:17)
    at executeCallback (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:405:9)
    at handleCallback (/var/www/html/rapp/node_modules/mongoose/node_modules/mongodb/lib/utils.js:128:55)

I'm quite new in the node world so I maybe made a mistake somewhere, but as it works fine with PNG but not with JPEG, I don't know what I could do wrong.

I still have the problem and can't get my code to work just becase of that.
My code works well on small images though.

JPEG on that line refers to the jpeg-js package. Might be worth submitting an issue on that.

hmm sorry I don't remember which image I used and as I don't have to use jImp now I can't really test again.
I'll try to find the page and I'll send it if I find it.


Issue 73 github read commands

In an effort build up commands to complete issue #73 I've added various github repository listing commands.
Instead of waiting for a large all encompassing branch, opting to break this out into smaller functionality sets.
Adds...
and-cli github
and-cli github -u wintondeshong

Thanks for making those updates! Looks good to me.


Differentiate between user and request errors for auth calls

Differentiate between endpoint connect 401 and generic errors

Currently show Could not connect, please try again for both
Need to show some kind of unknown username or password combination style message for 401 errors.

Also need to do the same on the log in screen

Could possibly use same component/visual look of endpoint connect log in (?)

This issue was moved to cloudfoundry-incubator/stratos#1514


env HCP_CLIENT_ID & HCP_CLIENT_SECRET

with latest docker image hashicorp/packer:latest

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

Tracking this in https://github.com/hashicorp/packer-github-actions/pull/34


trigger 3.1.0 release

Trying this again after format changes #1375 and #1374.
@getquill/maintainers

Release failed due to nexus artifacts already being in place. Need to undo and then perform a piecemeal deployment.


Unable to reach TestRail when using API key instead of password

I'm using the below mentioned way to reach testrail,
var TestRail = require('testrail-promise')
var testrail = new TestRail("https://testrail.EmployerName.com/", "UserName", "T6U4L52JpZlKOqIzOgGl-CCmWhxNr5Ldblg2dNhu1");
But I'm unable to log my errors by using the way.
when I'm using password instead of API key, it is working as expected

Very old issue but since I also wasted an hour figuring this out, if you created a new API key, make sure you have saved it in the TestRail settings page before using it..
https://github.com/billylam/newman-reporter-testrail/issues/6


Support Recorder panel in Edge, which can record & replay a user flow

Chrome devtools has supported the Recorder panel from Chrome 97: Recorder: Record, replay and measure user flows - Chrome Developer, which can record & replay a user flow, also can export the user flow as Puppeteer script to customize the script further.
I wonder whether there's any plan for Edge devtools to include this Recorder panel. And is it possible to export the user flow as Playwright script?
AB#39424931

Thanks for your question! @zoherghadyali will be interested in this, so let me ping him here.
As you might know, Edge is built on Chromium, which Chrome uses too, and the DevTools are part of the project.
However, my understanding is that the Recorder panel itself, is not part of Chromium but instead only exists in Chrome. That means that other Chromium-based browsers (such as Edge) don't automatically have it too.
It's great knowing that you are interested in it. We certainly have a lot of interest for test automation tools and workflows, and will consider this as part of our priorities.

Any news on this?

No we don't have any news to share on this at the moment unfortunately. This request is definitely on our backlog, but not currently in our prioritized list.

Following...

Following too . . .

It seems to be good in v120

Correct, the Recorder tool now works in Edge DevTools. I'll update this issue when I know more details about this.


InputInfo update v1 complete

InputInfo now working with new course and session class. Commented.

test is wrong! code is good


Netmiko connection task

Here are some tests I did on it:
### One way to call it
result = b.run(task=netmiko_connection)
#result = b.run(task=netmiko_connection, global_delay_factor=4)

### Second way to call it
#net_connect = b.inventory.hosts['pynet-rtr1'].get_connection("netmiko", global_delay_factor=4)
#print(net_connect.find_prompt())
#print(net_connect.global_delay_factor)

#net_connect = b.inventory.hosts['pynet-rtr1'].get_connection("netmiko", netmiko_host='arista2.twb-tech.com')
net_connect = b.inventory.hosts['pynet-rtr1'].get_connection("netmiko", host='arista2.twb-tech.com')
print(net_connect.find_prompt())

Merge was messed up...easier to just resubmit.


Allow saving sprites bigger than 1MB

AppEngine default object store limits the objects to 1MB.
For now, a psikel is stored as a single object containing a base64 representation of the spritesheet, which can go easily over 1MB when working with a big resolution sprite.

investigate alternate solutions to store blobs > 1MB
whatever the new size limit is, display an accurate error message when saving - it would actually be nice to check the size before uploading and to display a warning to the user, even if it's not 100% accurate.

For client-side or server storage? If it's client-side you could use localStorage. Wikipedia says it takes at least 5MB.

Sorry I should have been more accurate ! This issue is mainly about server side storage, which has a cap of 1MB per object.
For client-side saving we already use localStorage. However the localStorage cap is shared for all objects of a single domain. So if you save a lot of big sprites you will still run into storage issues. And this is problematic since we also use localStorage to manage user preferences.
Ultimately we should move the client-side save to indexedDB [1].
[1] https://developer.mozilla.org/en/docs/Web/API/IndexedDB_API


Parse task queue stats from /proc/schedstat

These are useful as an indication of CPU contention and scheduler
latency.

@discordianfish @pgier ready for review.
I'll update https://github.com/prometheus/node_exporter/pull/1389 to use this after review.

@pgier Everything addressed I think, please re-review

@discordianfish @pgier bump, review please

@pgier ok, sorry to change things just after asking for review. It occured to me reading the diff with fresh eyes and a cup of coffee that there were untested corner cases. I've added some more tests and rearranged some things. Should be good now, for reals.

Merged, thanks!

What's the process for getting a released tagged so I can complete https://github.com/prometheus/node_exporter/pull/1389 ?

Actually now that I've learned a bit more about go modules I've figured out how to update it without a release. Seems I'm not the first one to pin node_exporter to a non-released version of procfs.


Please invite me to the GitHub Community Organization

Name
Vishwa.R
Discord Username (if applicable)
No response
Additional Context
No response

Welcome to this community @code-reaper08 ✨✨🎉

Welcome to this community @code-reaper08 ✨✨🎉

Thank you for welcoming me @Sam-Varghese
Happy to be a part of the community 😃


Product question

What

Changed the Color of the Text to make it compatible with both dark and light theme for the app.

Screenshot
before:

After:

Fixes bug(s)

#1270

Part of

https://github.com/openfoodfacts/smooth-app/issues/525
(please be as granular as possible)

Heyy @PrabeshPP this is already handled in #1249, but with changing the background color of the question card instead

@M123-dev  okay no problem!


Add a license to the repository

I recently came across your project qgss_2021_labs and I noticed that the repository github.com/aaronj0/qgss_2021_labs currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thanks!


Visualization service has no /tmp/data volume mount

We only see the execution from the cell in the notebook. We cannot re-run the cell in the visualization service

Solved by


Adding CVEs for kube-proxy-1.28

Adding Advisory GHSA-8pgv-569h-w5rw for kube-proxy-1.28
Adding Advisory GHSA-rcjv-mgp8-qvmr for kube-proxy-1.28

added in https://github.com/wolfi-dev/advisories/pull/776


update travis script

@paulb777 this fixed the code 65 issue for travis.

Great! Please update CocoaPods to 1.3.1 and reconfirm - and add a comment about how to update it in the future.

Just updated to latest one, and put a comment on gemfile about how to update.


Refactored helm-chart-annotations.md

@iamrajiv Sorry, I am a bit lost here. Can you please update the title and description with what is the intent of this pull request?
As of now, looks like some relevant content has been removed.


FR: check CI job result

I was about to suggest that if the PR is labeled V8 Engine we should check for a V8 CI job, but I just saw that we already do 😄 :
i  Last V8 CI on 2018-11-10T22:17:13Z: https://ci.nodejs.org/job/node-test-commit-v8-linux/1838/
i  Last Lite PR Pipeline CI on 2018-11-14T17:57:34Z: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/1615/

So how about we query Jenkins for it's status?

So..the code for querying Jenkins and parsing the results are already part of ncu-ci, I guess we just need some glue to invoke them from git-node..

I'll try to draft something this weekend.

Implemented on https://github.com/nodejs/node-core-utils/pull/422


Version 3.1.0 not created on github but is on rubygems

Current version of cancancan on github is 3.0.2 while on rubygem it is 3.1.0.
It seems like there is a potential that someone forgot to create the release on github.

Also, the changelog does not reflect the 3.1.0 release. This makes me wary of upgrading - it appears as if the 3.1.0 is not an official release.

Forgot to push the version bump. sorry for the confusion.


Domain whitelisting

Description:

Add in the frontend a config page to 1) edit whitelisted domains and 2) show all whitelisted domains
Add to the api service the CRUD APIs to Insert/Delete/Get whitelisted domains in the postgres DB

Estimation: 5

Unwanted


Problem decoding PostgreSQL timestamps

Trying to list the records in a table with timestamp columns triggers this error:
Error:  702 : typeMismatch(Swift.Double, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "starting", intValue: nil)], debugDescription: "Could not cast Optional(2015-08-02 16:30:00 +0000)", underlyingError: nil))

CREATE TABLE public.availability
(
    id bigint NOT NULL DEFAULT nextval('availability_id_seq'::regclass),
    team_member_fk bigint NOT NULL,
    starting timestamp with time zone,
    ending timestamp with time zone,
    overtime boolean,
    on_shift boolean,
    CONSTRAINT pk_availability PRIMARY KEY (id),
    CONSTRAINT fk_availability_team_member_fk FOREIGN KEY (team_member_fk)
        REFERENCES public.team_member (id) MATCH SIMPLE
        ON UPDATE RESTRICT
        ON DELETE RESTRICT
)
...

struct Availability : Codable, Model {

    var id: Int
    var starting: Date
    var ending: Date
    var onShift: Bool
    var overtime: Bool
    var teamMember: Int
}

extension Availability {
    static var tableName: String = "public.availability"
}

extension Availability {
    enum CodingKeys: String, CodingKey {
      case id, overtime, starting, ending
      case onShift = "on_shift"
      case teamMember = "team_member_fk"
   }
}

Hey @fwgreen, what version are you using?

@EnriqueL8 The Package.resolved says version 0.2.0.

After replicating your issue - I have seen that we extract the TimeInterval from Date and store it in the database. Since public typealias TimeInterval = Double, we just make the table column of type double precision. In you case since you are constructing the Create Table manually , you have set a different type for that column that is why we fail to decode the value because we are expecting a double instead of a timestamp.
Maybe it makes more sense to save the Date as a timestamp in the Database? A decision to make is to default to timestamp without time zone for the column type when creating the table at runtime and then if that type needs changing then the Create Table statement can be written manually.
I believe the process of decoding from timestamp without time zone and timestamp with time zone should be the same so it should be fairly easy to construct the Date Type. We will implement this change and release a new version as soon as possible.
Thanks for raising this issue @fwgreen!

@EnriqueL8 Thanks for taking the time to investigate this issue.  My database is part of a preexisting Java application and it seems the PostgreSQL JDBC driver defaults java.time.LocalDateTime to timestamp, a fact I took for granted.  These are the risks when running in polyglot environments 😄

I'm having the exacly issue trying to retrieve a datetime field from a preexisting MySQL database..

Having dates stored in the database as a number has been a  problem for me.  It would be great to be able to use native database format (preferrable with time zone support to).  I have a system with lots of calendar computations and pre existing schema and reporting that will not work with a double.

I have merged changes to address alternative date encoding formats under PR #110 .
Hopefully they address the majority of use cases, should anything further be required please raise a new enhancement request.


Set up apply audit events

Changes in this PR
This PR sets up audit events for Apply, borrowing heavily from CAS1. Note this PR errs on the side of following CAS3 audit name conventions (VERB_CONTEXT_DETAIL[_SUCCESS || _FAILURE]), rather than copying CAS1's naming
Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

Keeping audit keys consistent per service feels valuable. Whether it's our team wanting to get audit data to use in metrics or the HMPPS Audit team wanting to review behaviour, they are only likely going to want to do this at a service level.


Benchmarking / Performance Notes

See https://github.com/bytecodealliance/wasmtime-dotnet/issues/9#issuecomment-610221118 for .12 to .15 differences (from roughly 1.5ms to now > 30ms)

#8 supersedes this issue, and with the update to 0.18.1-preview1, this 0.15 wasmtime issue is resolved.


Updated with Azure Active directory audit logs

@ndolker : Thanks for your contribution to the Azure documentation! The author, @TomShinder, has been notified to review your proposed change.

#sign-off


AIX multipackage with source

I have a requirement to install multiple packages from a single source directory to support a particular client.  However, when I provide the package_name as an array, and also the source (as required for AIX), I get an error
    Chef::Exceptions::InvalidResourceSpecification
    ----------------------------------------------
    You may not specify both multipackage and source

Which comes from chef/provider/package:
      def check_resource_semantics!
        if new_resource.package_name.is_a?(Array) && new_resource.source != nil
          raise Chef::Exceptions::InvalidResourceSpecification, "You may not specify both multipackage and source"
        end
      end

I would rather not create multiple stanzas (either through a loop or the hard way).  I can work around the problem by using an execute block, but that seems non chef-like to me.  Is this expected behavior that I need to work around?  Thanks!

The only providers that support arrays are yum and deb package.

Ok. I didn't notice that. I thought I saw on the bff_package page that it
could be an array. I guess I was mistaken. I'll have to check it out
tomorrow. Thanks.
On Thu, Sep 3, 2015 at 1:35 PM Lamont Granquist<EMAIL_ADDRESS>wrote:

The only providers that support arrays are yum and deb package.
—
Reply to this email directly or view it on GitHub
https://github.com/chef/chef/issues/3863#issuecomment-137522132.


JSON frontend support

Hi, i love wttr.in and have been using it for a lot of personal projects lately.  I believe wego supports JSON as an output (specified as a "frontend", though I haven't tested this personally).  It would be awesome if wttr.in could support JSON output (or specifying any arbitrary frontend I suppose)... maybe something like:

GET /Buffalo - buffalo ny weather
GET /Buffalo?frontend=json - buffalo ny weather with -f json passed to the wego command

There might need to be some extra processing by wttr.in to specify the appropriate headers to the requesting client to let it know the response is JSON.
If there is interest in this I can take a look at the code and see what I can do

I believe GET /Buffalo.json is of less surprise.

Hi, I took the liberty of creating a REST API with JSON support had based on the wttr.in information.
Take a look:
https://goweather.herokuapp.com/weather/Buffalo
Or try execute:
curl -X GET https://goweather.herokuapp.com/weather/Buffalo
If you want to check or contribute with the code (in Go), feel free to visit:
https://github.com/robertoduessmann/weather-api
I hope to help somebody  :smiley:
Also, feel free to comment or request new features ;)

I like the idea of JSON output too. @robertoduessmann 's goweather app seems a good idea in theory, but so far it's too bare-bones to be very useful, since it only includes (current) temperature and wind conditions. I'd want to see a description ("Rainy" or "Sunny"), wind and rain conditions, for both the current time as well as a forecast. Y'know, like a regular weather API.

Hi @curtisgibby , I understood your point of view, thanks for your feedback.
Anyway, if you still have interest in goweather, in the last weeks, we improved the API.
We've added weather description and forecast to 2 days. Take a look and fell free to suggest more things you consider important to add in API.

Thanks. I am still interested in consuming this API, and the improvements that you've added do make the system much more useful. I created a couple of new issues for further possible refinements.

Nice, I really thank for your comments @curtisgibby . I will work and add the features as soon as possible :smile:

Was looking for additional text format options as did not see JSON as a current option.  Ideally JSON or XML where I can format out preferred.
In mean time existing text output, how do I add my own text for formatted output, add  at end and maybe  or  tagging, or even something simple like "Wind:%wHumidity:%h" to current output.

@mikwei I hope that both JSON and XML APIs will be published soon. Currently you can use the single output API; it possible to have using the output you are describing:
$ curl wttr.in/London?format="%l:+%c+%t"
London: ☀ +15°C

w is for Wind; h is for humidity etc. See the full list in README.md.

There appears to be a solution built in: ?format=j1

@xewl Yes, that's true; thank you for reminding me of this issue. It is fixed now


remote desktop and keylogger

in remote desktop cant control his pc.  and also old keylog not come

It's not an offline keylogger and it's a bug with DPI scaling or something. Man I wish I could close issues. Are there roles that can be assigned to users on GitHub where you can close issues?

@Archikle, yep there are. Also I feel the exact same way


Set default color for only one level contour

Description
contour doesn't work when only one level is available in levels.
Steps to reproduce
import proplot as plot
import pandas as pd
import numpy as np
fig, axs = plot.subplots()
state = np.random.RandomState(51423)
data = state.rand(6, 6)
data = pd.DataFrame(data, index=pd.Index(['a', 'b', 'c', 'd', 'e', 'f']))

# Line contours with labels
axs.contour(
    data.cumsum(axis=1) - 2,
    # color='gray8',
    levels=[0]
)
axs.format(title='Line contour plot with labels')

Expected behavior:
Just like the color is specified:
color='gray8'

Actual behavior:
Traceback (most recent call last):
  File "/home/xin/Desktop/test.py", line 13, in <module>
    levels=[0]
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/ui.py", line 682, in _iterator
    result.append(func(*args, **kwargs))
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/axes/plot.py", line 3867, in wrapper
    return driver(self, func, *args, **kwargs)
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/axes/plot.py", line 897, in standardize_2d
    return func(self, x, y, *Zs, **kwargs)
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/axes/plot.py", line 3867, in wrapper
    return driver(self, func, *args, **kwargs)
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/internals/warnings.py", line 97, in wrapper
    return func_orig(*args, **kwargs)
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/axes/plot.py", line 2867, in cmap_changer
    minlength=(1 if name in ('contour', 'tricontour') else 2),
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/axes/plot.py", line 2552, in _build_discrete_norm
    levels, descending = pcolors._check_levels(levels)
  File "/home/xin/Software/miniconda3/envs/s5p/lib/python3.7/site-packages/proplot/colors.py", line 1760, in _check_levels
    raise ValueError(f'Levels {levels} must be 1d vector with size >= 2.')
ValueError: Levels [0] must be 1d vector with size >= 2.

Proplot version
0.6.3

This was due to the fact that DiscreteNorm (used for all colormaps in proplot) requires at least 2 levels. I added an edge condition to fix it -- the result of your example is now a single contour whose color is the central color of the specified (or default) colormap, which matches the behavior in matplotlib.
Example:
import proplot as plot
import pandas as pd
import numpy as np
fig, axs = plot.subplots()
state = np.random.RandomState(51423)
data = state.rand(6, 6)
data = pd.DataFrame(data, index=pd.Index(['a', 'b', 'c', 'd', 'e', 'f']))

# Line contours with labels
axs.contour(
    data.cumsum(axis=1) - 2,
    cmap='grays',
    levels=[0],
)
axs.format(title='Line contour plot with labels')

Thanks Luke! Nice fix ;)


Fix #68 : a marker is printed

This PR avoid to get a defined center in the 'print marker loop' after a right or left-clic on the map (which set a center to the overlay)

lgtm thx

nice


PollEvent() fails outside package main on macOS Sierra

Operating system: macOS Sierra 10.12.6
I ran go test -v in the go-sdl2/sdl directory and an uncaught exception was thrown. The culprit is the PollEvent() call in go-sdl2/sdl/events_test.go. Here is a minimal example that replicates the failure (run go test foo_test.go on this):
package foo
// package main

import (
	"testing"
	"github.com/veandco/go-sdl2/sdl"
)

func TestFoo(t *testing.T) {
	sdl.Init(sdl.INIT_EVERYTHING)
	defer sdl.Quit()

	sdl.PollEvent() // BAD!!
}

// func main()  {
// 	sdl.Init(sdl.INIT_EVERYTHING)
// 	defer sdl.Quit()
//
// 	sdl.PollEvent()
// }

and here is the stack trace of the failure:
2017-09-07 12:31:52.809 foo.test[8900:29022] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'nextEventMatchingMask should only be called from the Main Thread!'
*** First throw call stack:
(
	0   CoreFoundation                      0x00007fffd466a2cb __exceptionPreprocess + 171
	1   libobjc.A.dylib                     0x00007fffe947a48d objc_exception_throw + 48
	2   AppKit                              0x00007fffd2855e82 -[NSApplication(NSEvent) _nextEventMatchingEventMask:untilDate:inMode:dequeue:] + 4480
	3   libSDL2-2.0.0.dylib                 0x00000000042f0c9b Cocoa_PumpEvents + 247
	4   libSDL2-2.0.0.dylib                 0x0000000004290568 SDL_PumpEvents_REAL + 23
	5   libSDL2-2.0.0.dylib                 0x00000000042905d0 SDL_WaitEventTimeout_REAL + 57
	6   foo.test                            0x00000000040f59fd _cgo_f7c1af2e6e77_Cfunc_SDL_PollEvent + 29
	7   foo.test                            0x0000000004058530 runtime.asmcgocall + 112
)
libc++abi.dylib: terminating with uncaught exception of type NSException
SIGABRT: abort
PC=0x7fffe9e8ed42 m=5 sigcode=0
signal arrived during cgo execution

goroutine 19 [syscall, locked to thread]:
runtime.cgocall(0x40f59e0, 0xc420028f18, 0x42118e0)
	/usr/local/go/src/runtime/cgocall.go:132 +0xe4 fp=0xc420028ee8 sp=0xc420028ea8 pc=0x4004b24
github.com/veandco/go-sdl2/sdl._Cfunc_SDL_PollEvent(0x42118e0, 0x0)
	github.com/veandco/go-sdl2/sdl/_obj/_cgo_gotypes.go:5552 +0x49 fp=0xc420028f18 sp=0xc420028ee8 pc=0x40f2959
github.com/veandco/go-sdl2/sdl.PollEvent.func1(0x42118e0, 0xc420028f88)
	/Users/Veritas/Desktop/go/src/github.com/veandco/go-sdl2/sdl/events.go:468 +0xa4 fp=0xc420028f58 sp=0xc420028f18 pc=0x40f3604
github.com/veandco/go-sdl2/sdl.PollEvent(0x0, 0x414c938)
	/Users/Veritas/Desktop/go/src/github.com/veandco/go-sdl2/sdl/events.go:468 +0x2d fp=0xc420028f80 sp=0xc420028f58 pc=0x40f2d2d
command-line-arguments.TestFoo(0xc4200aa0f0)
	/Users/Veritas/Desktop/foo_test.go:13 +0x4a fp=0xc420028fa8 sp=0xc420028f80 pc=0x40f4a4a
testing.tRunner(0xc4200aa0f0, 0x414c880)
	/usr/local/go/src/testing/testing.go:746 +0xd0 fp=0xc420028fd0 sp=0xc420028fa8 pc=0x40b7a60
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2337 +0x1 fp=0xc420028fd8 sp=0xc420028fd0 pc=0x40598c1
created by testing.(*T).Run
	/usr/local/go/src/testing/testing.go:789 +0x2de

goroutine 1 [chan receive, locked to thread]:
testing.(*T).Run(0xc4200aa000, 0x41447ae, 0x7, 0x414c880, 0x406be46)
	/usr/local/go/src/testing/testing.go:790 +0x2fc
testing.runTests.func1(0xc4200aa000)
	/usr/local/go/src/testing/testing.go:1004 +0x64
testing.tRunner(0xc4200aa000, 0xc42004fde0)
	/usr/local/go/src/testing/testing.go:746 +0xd0
testing.runTests(0xc420094020, 0x41ed670, 0x1, 0x1, 0x4012a87)
	/usr/local/go/src/testing/testing.go:1002 +0x2d8
testing.(*M).Run(0xc42004ff18, 0xc42004ff70)
	/usr/local/go/src/testing/testing.go:921 +0x111
main.main()
	command-line-arguments/_test/_testmain.go:44 +0xdb

rax    0x0
rbx    0x6
rcx    0x700000d85488
rdx    0x0
rdi    0x1003
rsi    0x6
rbp    0x700000d854b0
rsp    0x700000d85488
r8     0x7fffe9e1d650
r9     0x7fffe9ddfa50
r10    0x8000000
r11    0x206
r12    0x700000d85610
r13    0x30
r14    0x700000d87000
r15    0x8
rip    0x7fffe9e8ed42
rflags 0x206
cs     0x7
fs     0x0
gs     0x0
FAIL	command-line-arguments	0.355s

When I uncomment the blocked out code in foo_test.go and run the file as main, the problem disappears. It appears that I cannot call PollEvent() outside the main thread.

Hi @ZeroXLR,
It doesn't seem to happen on Linux. I'll try later on my Macbook Air =)

@veeableful Indeed, it appears to be an Apple-exclusive problem: the exception thrown is an Objective C exception after all.

Weird, it seems removing runtime.LockOSThread() from the library fixes it. I'll look into this more.

@veeableful Can you tell me what you did? I'd like to try out that fix on my fork too

I removed runtime.LockOSThread() from the init() in sdl.go. Then removed the runtime import in the same file. I'm not sure if that counts as fix though :P

Hmm. I will try that out. By the way, here is an even shorter example that causes the failure:
package main

import (
	"runtime"
	"github.com/veandco/go-sdl2/sdl"
)

func main() {
	done := make(chan bool, 1)

	sdl.Init(sdl.INIT_EVERYTHING)
	defer sdl.Quit()

	go func() {
		sdl.PollEvent()
		done <- true
	}()

	<- done
}

Basically, PollEvent() crashes whenever it is in some thread different from main. I don't know whether that helps you understand what is going on.

An obvious but adhoc solution is possible by doing an unlockOSThread() inside TestMain(m *testing.M) before the other tests begin. I will submit a pull request with the patch. However, reading through related issues like golang/go/issues/7905 make me think that there should be a more
comprehensive way to resolve this...

I agree. I've merged the pull request for now and hopefully will find solution to this.

I have confirmed that the tests still fail by running go test -v -race. However, it only fails sometimes now. Depending on how you look at it, this may be worse as it is a hidden bug.

It is known that SDL_PollEvent should be run in the main thread according to SDL_PumpEvents. Perhaps we just need to find a way to do that in the test.

There is certainly a way of doing that by wrapping each PollEvent()/WaitEvent() calls that I could find with sdl.Do(..)s and calling sdl.Main(..) from TestMain(..). This was the way I was originally trying to solve the problem. But I always end up getting a message: 2017-09-12 10:41:14.346 sdl.test[8522:53069] !!! BUG: The current event queue and the main event queue are not the same. Events will not be handled correctly. This is probably because _TSGetMainThread was called for the first time off the main thread..
This message DOES NOT break the tests though (which continue after that message) as far as I know. So if there are not better solutions, we could go with that...

With this issue seemingly fixed by 4fc54e5 I will close it.


Click on '....'
Scroll down to '....'
See error

Self-hosted instance?
No
Error file
Message: Error: connect ECONNREFUSED ::1:5432
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1237:16) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '::1',

Additional context
No response

See this comment


Uncalled function has 100% statement coverage of inner named functions

function namedFn (){
  function a(){} //counted
  function b(){} //counted
  function c(){} //counted
  this.d = function(){} //not counted
}

=============================== Coverage summary ===============================
Statements   : 80% ( 4/5 )
Branches     : 100% ( 0/0 )
Functions    : 0% ( 0/5 )
Lines        : 80% ( 4/5 )
================================================================================

What am I missing?

Looks like all function declarations are initialized to 1 and not instrumented.
https://github.com/gotwarlost/istanbul/blob/master/lib/instrumenter.js#L795
Couldn't all function declarations inside of a function scope be tracked and the first line of parent function be instrumented to increment all the function declarations at that time?

Related to:
#417
Looks like this was to cover the case where the 'declaration' may be in an unreachable place and hoisted.  The fix to instrument the increments to the top seems like it would still work for declarations.

This has a long and convoluted history. Basically function declarations and variable declarations w/o initializers should not be treated as statements by the coverage code and be excluded from coverage analysis. This needs to be fixed with a major version upgrade since it will change coverage reports for everyone.
I'm not making any patches on an already convoluted set of patches.

Is there some open issue I can watch for changes?


Provide MPI environment

provision machine and install required or working MPI implementation. provide mpiexec/mpirun out-of-the-box but wrap invocation in shell scripts.

run mpi ping_pong test or test_mpi.exe packaged with windows build to test Intel MPI environment.


Add option to create Seeder Files

Just an idea, why not let builder also create Database Seeder Files?
~would save us all time having to create them out manually.

@ayumihamsaki sure, feel free to add support in a PR ;)

I be honest and say I'm not sure if I got enough time to create this one. So if anyone else wants to create a PR for it please feel free. If not I will try in a few weeks time...

I be honest and say I'm not sure if I got enough time to create this one. Got so much work going on right now! If anyone wants to create a PR for it, please go ahead.

@ayumihamsaki I'm sure I don't have enough time :) I'll leave this open for now but if there isn't any other activity on it in a month I'll have to close it as abandoned.

Closing as it has been over a month since any activity on this occurred and we are trying to figure out what issues are still relevant. If this is still something that you would like to see through to fruition please respond and we can get the ball rolling.


PMID:39110593 missed

decreased plasma membrane ER tethering

actually we have
06330 | decreased endoplasmic reticulum-plasma membrane tethering
should it be part of

FYPO:0007571
abnormal cortical endoplasmic reticulum organization

add synonyms


Some step by step instructions.

After a little trial and error, I decided to write a howto for those having trouble.
I hope this helps.
1: Install Anaconda (for all users)

https://www.anaconda.com/

=============================================
2: Download Stable Diffusion

https://github.com/CompVis/stable-diffusion
(click green button, download zip)

Unpack to a directory of your choosing
eg D:\StableDiffusion\

=============================================
3 : Download the stable diffusion weights

You can find these at
https://huggingface.co/CompVis/stable-diffusion-v-1-4-original/tree/main
You will need to make an account to access these files

The file you want is sd-v1-4-full-ema.ckpt most likely, although if you have
less than 8gb of ram, you can go for sd-v1-4.ckpt

=============================================
4: Open Anaconda Powershell prompt from the start menu

Navigate to where you unpacked stable diffusion in the console
eg D:\StableDiffusion\

Type the following into console :
conda env create -f environment.yaml

Wait for it to finish downloading everything.
It may pause for a while on some of the downloads, this is fine.
Just keep waiting.

After a little while the command prompt will be ready to accept input again
it should look something like
done
To activate this environment, use
conda activate ldm
To deactivate an active environment, use
conda deactivate
(base) PS D:\StableDiffusion>

Type the following into the command prompt
conda activate ldm

=============================================
5: Copy your downloaded model to the Stable Diffusion directory.
D:\StableDiffusion\models\ldm\stable-diffusion-v1
Rename the model file to model.ckpt and place it in the directory above,
if it does not exist, create it.
=============================================
6: Extract the StableDiffusion GUI to it's own folder.

Click on the settings button in the gui
I set the following options :
Anaconda path : C:\ProgramData\Anaconda3
Environment name : Idm
Stable Diffusion Path : D:\StableDiffusion
txt2txt path : D:\StableDiffusion\scripts\txt2img.py
img2img path : D:\StableDiffusion\scripts\img2img.py
I could not find an imgs2imgs or img2morph script so I left those blank.

=============================================
7 : Stable DiffusionGUI should now work as expected.

Sidenote : Some of the options you enter tend not to get saved.
so you might need to enter them again when you start it up.

@stukkat - How do you run the GUI?

@stukkat I dont see an exe file in this reposatory. https://github.com/Fictiverse/StableDiffusion-Windows-GUI -  I dont know how to get this running StableDiffusion.sln

@stukkat I dont see an exe file in this reposatory. https://github.com/Fictiverse/StableDiffusion-Windows-GUI - I dont know how to get this running StableDiffusion.sln

You don't need to clone the repo.
Simply go to releases and get the newest one (the zip-file).
You can find it here: https://github.com/Fictiverse/StableDiffusion-Windows-GUI/releases

@stukkat -- These instructions are quite helpful.  Thanks!
@stromal - I am running into a problem setting up the environment.  The pip dependencies are not getting found from the .yaml file.  Am using Anaconda3-2022.05-Windows-x86_64.exe version on Win 11.  See attached text file.
Would appreciate any ideas on why this is failing.
Thanks,
environment_error.txt

@stromal - I am running into a problem setting up the environment. The pip dependencies are not getting found from the .yaml file. Am using Anaconda3-2022.05-Windows-x86_64.exe version on Win 11. See attached text file.

Looks like you don't have git installed.

@Haladesta -- Thanks!  That got me moving again.


pretty-quick uses a different version of prettier than the installed in the repo

I found a mismatch between the fixes pretty-quick was doing on commit (the husky setup) and when running prettier directly.
I couldn't figure out what was causing the mismatch. Turns out the prettier version I have installed is quite old and is not the same version pretty-quick is using.

First format the file with prettier, it generates some changes. Then run pretty-quick, changes aren't there anymore (since the commit was done with pretty-quick so the fix basically returns repo to initial state).
So then I check the versions of prettier and turns out they are different.
I looked a bit at the code of pretty-quick and cannot figure out why it is using a different version of prettier than the installed one (thought it could be a global version, but I don't have one).

Interesting. pretty-quick only has a peerDependency on prettier and your version 1.10.2 matches it. pretty-quick does have a devDependency of 1.18.2 on prettier here which does match what you're seeing.
Could you change the console log you inserted to show require.resolve('prettier') instead, to confirm the path that prettier is being loaded from?

It gets resolved to this: node_modules/prettier/index.js, which is indeed version 1.18.2.
$ cat node_modules/prettier/index.js | grep version | head -n 1
var version$1 = "1.18.2";

And then I checked what version the scripts are using:
$ cat $(yarn bin)/prettier | grep version | head -n 1
var version$1 = "1.10.2";

It seems more of yarn/npm problem, in the way dependencies are resolved.
The only solution that occurs to me is to use execa to run prettier on pretty-quick side . Instead of using it as a library. But personally I don't like the idea, the library is handier to use.
~/p/cw-buyer master*
$ cat with-execa.js
const execa = require('execa')

console.log(execa.sync('prettier', ['--version']).stdout)

~/p/cw-buyer master*
$ node with-execa.js
1.10.2

Where does ls -l $(yarn bin)/prettier link to then?
Don't want to switch to sub-processes in the interest of being quick 😉, so I will close this issue, but still interested to see where your prettier is actually being resolved from!

Makes sense.
It didn't occur to me to check were prettier was linked to.
We have it installed as a dependency of a toolbox ("cw-sdk-js") so it comes from there.
~/projects/cw-buyer/node_modules/.bin/prettier -> ../cw-sdk-js/node_modules/prettier/bin-prettier.js


What license?

The README says

I am providing code in this repository to you under an open source license.

but not what the license actually is (at least that I see).

https://github.com/xixixao/Shem/blob/eabac002261d8670763a1e6943841f7038295047/package.json#L22

@vyp that's not good enough.
The license has to be included with the code.

Added LICENSE file. Let me know if that's not enough.


Cleanup  AnimationNode and AnimationNodeTransition APIs

Remove redundant (duplicated) APIs from AnimationNodeTransition
Move find_input() to NodeAnimation from AnimationNodeTransition
Sort lines

I support this, but unable to fully test this.

Superseded by #72326.


Disabling internal metrics

Hi,
I would like to use otel collector to only send metrics from specific namespaces. How can I disable sending of the internal metrics, specifically otelcol?
I have setup exclude filter for containers, but these doesn't seem to have effect for internal metrics.

You are talking about our helm chart - please advise if incorrect.
This is the code we use to manage the internal metrics pipeline:
https://github.com/signalfx/splunk-otel-collector-chart/blob/main/helm-charts/splunk-otel-collector/templates/config/_otel-collector.tpl#L248
We don't support shutting off internal collector metrics as they are useful to measure the health of the ingest pipeline. If you'd like to turn those off, please change the helm chart to reflect your needs, and please feel free to contribute the changes back.


Unable to load "gunnerus-waypoint-following" demo case

Hello,
me and my colleagues are trying to simulate the Gunnerus ship for research purposes and wanted to simulate the waypoint-following demo case. We already simulated other demo cases succesfully, for example "gunnerus-dp" or "construction-vessel". But when loading the waypoint-following demo case we all get the same error:
Cosim Demo App:
Could not create execution from OspSystemStructure.xml file: No resolvers available to handle URI: fmu-proxy://localhost:9090?guid=b71f76f7-b51e-40d5-affb-f63905622e2b
Cosim App:
error: No resolvers available to handle URI: fmu-proxy://localhost:9090?guid=b71f76f7-b51e-40d5-affb-f63905622e2b
As can be seen we tried to simulate it with the demo app and also with the cosim command line interface. We also neither changed the demo case after downloading it nor the cosim demo app or the cosim app.
Maybe someone had the same issue or can direct us towards where the cause of this issue may be? We would really appreciate a response as we are really grasping at straws with this error code.

I'm not able to reproduce this error locally with cosim-demo-app v0.9.0.
The error message indicates use of fmu-proxy. All configurations in this repository have been updated to use proxyfmu instead, so I suspect you might not be using the latest master version. Can you download the latest version of this repository and try again?


[BUG] Pull pre Image failed on K8S cluster

Describe the bug
Pull pre Image failed on K8S cluster
To Reproduce
Steps to reproduce the behavior:

install graphscope pre

pip3 install -U graphscope-client --pre   -i http://mirrors.aliyun.com/pypi/simple/ --trusted-host mirrors.aliyun.com

launch k8s
import graphscope
graphscope.set_option(log_level='DEBUG')
graphscope.set_option(show_log=True)
# Create GraphScope client session, the 'cluster_type' is k8s by default.
session = graphscope.session(
                             k8s_coordinator_cpu=1,
                             k8s_coordinator_mem="1Gi",
                             k8s_vineyard_cpu=0.2,
                             k8s_vineyard_mem="1Gi",
                             vineyard_shared_mem="1Gi",
                             k8s_engine_cpu=0.2,
                             k8s_namespace='gs-new-orc-jacky',
                             k8s_engine_mem="1Gi",
                             num_workers=1,
                             enabled_engines="analytical,interactive",
                             k8s_client_config='~/.kube/config')
print('========= Session created. ==========')

See error
2023-11-20 16:12:26,603 [INFO][utils:187]: coordinator-jzwbkf-65fb74466f-nb967: Failed to pull image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231119": Error response from daemon: manifest for registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231119 not found: manifest unknown: manifest unknown
2023-11-20 16:12:26,604 [INFO][utils:187]: coordinator-jzwbkf-65fb74466f-nb967: Error: ErrImagePull
2023-11-20 16:12:26,605 [INFO][utils:187]: coordinator-jzwbkf-65fb74466f-nb967: Back-off pulling image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231119"
coordinator-jzwbkf-65fb74466f-nb967: Successfully assigned gs-new-orc-jacky/coordinator-jzwbkf-65fb74466f-nb967 to minikube
coordinator-jzwbkf-65fb74466f-nb967: Pulling image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231119"
coordinator-jzwbkf-65fb74466f-nb967: Failed to pull image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231119": Error response from daemon: manifest for registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231119 not found: manifest unknown: manifest unknown
coordinator-jzwbkf-65fb74466f-nb967: Error: ErrImagePull
coordinator-jzwbkf-65fb74466f-nb967: Back-off pulling image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231119"

Environment (please complete the following information):

GraphScope version: 0.26.0a20231119
OS: MacOS
Kubernetes Version 1.23

Specify a tag manually may help. k8s_image_tag=0.25.0

Recent 3 image building failed. The latest usable image are 0.26.0a20231116, please specify it manually by k8s_image_tag='0.26.0a20231116'.
We will fix the building process ASAP.

Recent 3 image building failed. The latest usable image are 0.26.0a20231116, please specify it manually by k8s_image_tag='0.26.0a20231116'. We will fix the building process ASAP.
For reference: alibaba/GraphScope/actions/workflows/build-graphscope-images-linux.yml

Thanks for your reply.
Motivation:
For this bug fix #3363, I want to use this PR on the PRD env ASAP.
I think of two ways

Pull the official pre version, because it is built and released every day
Compile locally and publish to internal docker image repo

Can you guide me how to publish the image from the local development environment to the internal repo?

I'll first try to rerun it, seems it stuck during trying to apt update.
You could reference to this workflow file and I think you only need to

Rerun make analytical in your case.

If your cluster is not a single machine, you could docker save and docker load the image to the nodes machine.
And make sure the image pull policy is IfNotPresent.

Fixed by #3365 , The images would be available by tomorrow.

coordinator-mvekpb-5f9ffddcdd-4nck8: Failed to pull image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231121": rpc error: code = Unknown desc = Error response from daemon: manifest for registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231121 not found: manifest unknown: manifest unknown
2023-11-22 16:03:20,061 [INFO][utils:187]: coordinator-mvekpb-5f9ffddcdd-4nck8: Error: ErrImagePull
coordinator-mvekpb-5f9ffddcdd-4nck8: Successfully assigned gs-new-orc-jacky2/coordinator-mvekpb-5f9ffddcdd-4nck8 to hwy-hn1-riskmgt-online-service-prd-02
coordinator-mvekpb-5f9ffddcdd-4nck8: Pulling image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231121"
coordinator-mvekpb-5f9ffddcdd-4nck8: Failed to pull image "registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231121": rpc error: code = Unknown desc = Error response from daemon: manifest for registry.cn-hongkong.aliyuncs.com/graphscope/coordinator:0.26.0a20231121 not found: manifest unknown: manifest unknown

Sorry, the disk space of runner is exhausted. Rerunning.

Thanks for your reply, I will try it tomorrow

This tag is available now 0.26.0a20231122


Send/Transactions Tab Font Inconsistencies on macOS

Master built on macOS 10.15.7 and Qt 5.15.1 shows font inconsistencies as highlighted by the below screenshots:

Under the Send tab the Pay To field has a noticeably smaller font than the Label field. Additionally under the Transactions Tab, the word "All" looks bad as the A and L are very close together.
These inconsistencies are not showing up with master built on Arch Linux (Kernel 5.18.5) and Qt 5.15.1:

Under the Send tab the Pay To field has a noticeably smaller font than the Label field.

This is caused by the .AppleSystemUIFontMonospaced font, which is chosen by Qt.

Additionally under the Transactions Tab, the word "All" looks bad as the A and L are very close together.

It is very similar to #235. It does not looks like we could solve it effectively.

@hebasto The overview page had the same issue as on the Send Pay To fields and it was solved. by https://github.com/bitcoin-core/gui/pull/79 introducing a Bold monospaced font. For the Send Pay To field, the issue can be fixed by introducing a normal face Roboto Mono font. This can be opt-out like the initiative in https://github.com/bitcoin-core/gui/pull/216


code . not working after upgrade to 1.82.0

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:

1.82.0

OS Version:

windows 10.0.22621.2134
wsl <IP_ADDRESS>
Steps to Reproduce:
in wsl, open a folder using code . , nothing happened.

duplicat of https://github.com/microsoft/vscode/issues/192835

dup of https://github.com/microsoft/vscode/issues/192506


maintainers: remove inactive maintainers

Description of changes

Following https://github.com/NixOS/nixpkgs/issues/290642
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@MP2E

Drafting because I want to adopt some!

No need for a tree-wide right now.


Allow customizing log message format for Logging{Client,Service}?

Related conversation: https://line-armeria.slack.com/archives/C1NGPBUH2/p1598877787005400

Hello everyone!
We are using com.linecorp.armeria.server.logging.LoggingService but we have a little problem. We are trying to change the order of ctx logging and our custom request/response message logging. But now it's seems to be impossible because of LoggingDecorators and its three default ways of logging:
private static final String REQUEST_FORMAT = "{} Request: {}";
private static final String RESPONSE_FORMAT = "{} Response: {}";
private static final String RESPONSE_FORMAT2 = "{} Response: {}, cause: {}";

Is it possible to log request/response first, and then log ctx information? Just for visual usability.
Our log message looks like:
requestId=..., traceId=..., 1 million more headers, [sreqId=..., chanId=..., raddr=<IP_ADDRESS>:port, laddr=<IP_ADDRESS>:8080]h2c://url/service/method#POST, Request: {...}...

And our admins wants to see service and method first. So I need something like ctx sanitizer to remove
[sreqId=..., chanId=..., raddr=<IP_ADDRESS>:port, laddr=<IP_ADDRESS>:8080] and only stay h2c://url/service/method#POST, Request: {...}...

IIRC we used to have dedicated structural logging but removed it since logback / log4j can generally restructure the log well enough based on the information in the context. Is that right?
https://github.com/line/armeria/blob/master/site/src/pages/docs/advanced-structured-logging.mdx

IIRC we used to have dedicated structural logging but removed it since logback / log4j can generally restructure the log well enough based on the information in the context. Is that right?

It was removed because AccessLogWriter or just a custom callback on RequestLog.whenComplete() will do the same job.
I guess what the user is asking is more about the ways to customize the traditional log messages. For the case shown in the conversion, the user could achieve the goal by:

not logging context at all and using RequestContextExportingAppender; or
not logging the entire context but only the request URI and method.

It's doable by writing one's own decorator instead of using Logging{Client,Service}, but the user will have to reimplement/fork some useful functionalities like sampling and sanitization.
Would it make sense to add one more extension point like this?
interface RequestLogMessageFormatter { // name tbd
    String formatRequest(RequestContext ctx, Object sanitizedRequest);
    String formatResponse(RequestContext ctx, Object sanitizedResponse);
    String formatResponse(RequestContext ctx, Object sanitizedResponse, Throwable cause);
}

I'm not sure if it's a good idea or not yet, though. Just exploring.

I see - it feels unfortunate to have to add a formatter interface since the logging libraries provide their own and is sort of duplicate functionality. But I guess since slf4j doesn't provide a way to do structured logging until 2.0, it might be the only way to allow reformatting of our log messages.


Gogs v0.5.11 Linux binary broken

Hi, it seems that version <IP_ADDRESS>3 is included in the linux_amd64.zip for the 0.5.11 release.
2015/01/05 12:43:05 [T] Custom path: /home/git/gogs/custom
2015/01/05 12:43:05 [T] Log path: /home/git/gogs/log
2015/01/05 12:43:05 [I] Gogs: Go Git Service <IP_ADDRESS>3 Beta
2015/01/05 12:43:05 [I] Log Mode: File(Trace)
2015/01/05 12:43:05 [I] Redis Enabled
2015/01/05 12:43:05 [I] Memcache Enabled
2015/01/05 12:43:05 [I] Cache Service Enabled
2015/01/05 12:43:05 [I] Session Service Enabled
2015/01/05 12:43:05 [I] Mail Service Enabled
2015/01/05 12:43:05 [I] Register Mail Service Enabled
2015/01/05 12:43:05 [I] Notify Mail Service Enabled
2015/01/05 12:43:05 [I] SQLite3 Enabled
2015/01/05 12:43:05 [I] Run Mode: Production
2015/01/05 12:43:05 [web.go:66 checkVersion()] [E] Binary and template file version does not match, did you forget to recompile?
2015/01/05 12:43:05 [web.go:66 checkVersion()] [E] Binary and template file version does not match, did you forget to recompile?

Thanks your report!
I've re-uploaded linux binary, please help test again!

Working now... thanks!

Same? issue again with  <IP_ADDRESS>5 Beta
2015/06/23 21:22:06 [T] Custom path: /data/gogs
2015/06/23 21:22:06 [T] Log path: /gopath/src/github.com/gogits/gogs/log
2015/06/23 21:22:06 [I] Gogs: Go Git Service <IP_ADDRESS>5 Beta
2015/06/23 21:22:06 [I] Log Mode: File(Trace)
2015/06/23 21:22:06 [I] Redis Enabled
2015/06/23 21:22:06 [I] Cache Service Enabled
2015/06/23 21:22:06 [I] Session Service Enabled
2015/06/23 21:22:07 [I] SQLite3 Enabled
2015/06/23 21:22:07 [I] Run Mode: Production
2015/06/23 21:22:07 [web.go:77 checkVersion()] [E] Binary and template file version does not match, did you forget to recompile?


Magento Widgets/Custom blocks in CMS data

Hi!
Would be really great if the Magento Widgets and Custom Blocks are processed correctly. Could you please add this option?

@jochhop
Ok, if we come up with idea we will provide the PR

@jochhop will you merge this PR to master ?

Hi @shkodasv, the PR needs to be checked by another reviewer too (we're on it), but I guess it'll be accepted soon

Thanks!

Closing this as it's already in develop


Dropdown fix

This PR removes an out of place preventDefault that was stopping clicks on dropdown items from registering. I am honestly not 100% sure why I felt the need to add this in the first place but I tested out several different scenarios with different dropdowns on the page being open/etc and this should be safe to remove 👍
Closes: #491 , thanks for the heads up @nicholaai!

@colebemis good catch! I was wondering that too when I was playing around with this today, but thought it was taken care of because that's how the dropdowns in blueprints function, BUT turns out we're creating a totally custom dropdown instead of using this one 😂 I need to spend some time thinking about the right API here

@colebemis refreshed a bit on why it's built with a list there - we're expecting users to add links or buttons as children of the <li> elements. The lists help with keyboard navigation and is the standard pattern for a11y dropdowns!

I think this is ready for a final review now!

Ok, so after some rabbit hole adventures, I think this will need to introduce a breaking change. In order to fix the original issue of dropdown menu item clicks being swallowed by the prevent default in the open/close functionality, I needed to introduce a onToggle click handler to the wrapper element. Doing this breaks the render prop method because we then would have two toggle tracking functions listening at the same time.
I'm okay with this change because it seems a little silly to have 3 ways of using this component, and if the user really needs some special functionality to happen onClick or onToggle for the details element, they could add that behavior in separately.


Hyper Link Broken

The HyperLink for known issues and limitations : https://learn.microsoft.com/en-us/azure/virtual-desktop/troubleshoot-insights?tabs=analytics#known-issues-and-limitations does not work.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: b8947b6f-9d4b-b174-8a89-11187d5bee3f
Version Independent ID: 0c0f7abd-a801-9f48-c64c-36353713158b
Content: Troubleshoot Monitor Azure Virtual Desktop - Azure
Content Source: articles/virtual-desktop/troubleshoot-insights.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@KalpeshShetty
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@Heidilohr
Can you please check and add your comments on this doc update request as applicable.

@KalpeshShetty
I've addressed the issue and the changes should go live sometime today. #please-close


Adding a background image can enhance the user interface

the opening page looks like a basic one, adding a good background image or colour can enhance the user interface

you can assign this task to me..

Please assign this to me, I want to work on this particular project.


When subscribing to a topic with wildcards, callback is not called

When an application subscribes to a topic using wildcards '+' or '#', it receives Publish messages from the server, but the function ClientState::GetSubscription cannot find the proper subscription and therefore the application handler callback is not called.
The following patch seems to resolve the issue for me:
--- src/mqtt/ClientState.cpp    22 May 2017 15:59:16 -0000      <IP_ADDRESS>
+++ src/mqtt/ClientState.cpp    26 May 2017 16:49:56 -0000
@@ -54,11 +54,43 @@
            std::shared_ptr<Subscription> ClientState::GetSubscription(util::String p_topic_name) {
                    std::shared_ptr<Subscription> p_sub = nullptr;

                  util::Map<util::String, std::shared_ptr<Subscription>>::const_iterator itr = subscription_map_.find(p_topic_name);
                  if(itr != subscription_map_.end()) {
                          p_sub = itr->second;

           return p_sub;
                  }

       //  Handle '+' or '#'

       for (const auto &pair : subscription_map_) {

           size_t index1, index2;

           for (index1 = index2 = 0; index1 < p_topic_name.size() && index2 < pair.first.size(); ) {

               if (pair.first[index2] == '#' && index2 + 1 >= pair.first.size()) {

                   //  Terminating '#'

                   p_sub = pair.second;

                   return p_sub;

               }

               if (pair.first[index2] == '+' && (index2 == 0 || pair.first[index2 - 1] == '/') && (index2 + 1 >= pair.first.size() || pair.first[index2 + 1] == '/')) {

                   //  A '+' that replaces a whole topic

                   index2++;

                   while (index1 < p_topic_name.size() && p_topic_name[index1] != '/') {

                       index1++;

                   }

               } else if (p_topic_name[index1] == pair.first[index2]) {

                   index1++;

                   index2++;

               } else {

                   //  Mismatch found

                   break;

               }

           }

           if (index1 >= p_topic_name.size() && index2 >= pair.first.size()) {

               //  Match found

               p_sub = pair.second;

               return p_sub;

           }

       }
                  return p_sub;
          }

Hi @serjiozh,
Thank you for the patch. This is a known issue that we have fixed internally already and the fix will be included in the next release of the SDK. I will keep this issue open until the release is out. Please let me know if you have any further questions/suggestions.
Rahul

The SDK has been updated to support wildcard based subscriptions with the v1.1 release.
Please reopen if you have any further questions.
Rahul


How do I install the mods?

Thanks

first you need to install Melon loader: https://melonwiki.xyz/#/?id=automated-installation
Then just drop the DLLs into the new mods folder that should appear in your game files after installing melon.


check in Addy's "Offline Storage for PWAs" article

@addyosmani @petele review comments fixed, PTAL

I think having it in instant and offline makes sense, although I am not sure about the naming of the folder offline-storage-pwa, just call it storage or local-storage or client-storage (or something to that effect). offline is already implied in the parent folder, it's not really pwa only (this works for everything).
One thought for the future of this storage section is that you could use this as the high-level intro and then have new articles and content that dive's into a lot more depth in this space based on this narrative.
Note: I think offline-cookbook should be changed too, but we can deal with that seperately.

Holding for today's launch

Looking now.

@PaulKinlan @petele  My plan was to have a storage landing page at web-storage (or web-storage-overview). That page already exists but doesn't yet have links to related content. This was to be the first related item (you can see the list on the staged web-storage page). Does that make sense? I'll proactively rename this page to client-storage and link it from web-storage but let me know if you disagree.
Btw, all comments so far are fixed. I'm waiting for feedback from @jpmedley and then I'll submit another commit.

@jpmedley @petele
Sorry for the extra commits here, some changes were directly committed to my branch by someone else so I had to do some manual merging. Should be all good now. PTAL

@petele  Re: outstanding question to PK and you above about URL and relationship with storage overview page, let's handle that structural question in a sep PR. Please merge if/when happy with content changes on this page.

@marcacohen - back to you for a few changes.

This PR is abandoned and replaced by https://github.com/google/WebFundamentals/pull/3713.


Refines project card styles

Fixes #145
Refined the spacing and sizing overall in the project card component.
I added a scroll affordance gradient on the card label component (it appears slightly over labels that do not scroll if last label's edge is right at the edge of the container—would require some JS to show gradient only if elements extended past right edge).
I rearranged the elements on the bottom of the cards to accommodate situations where the location or the updated info was wrapping.
Project Cards on Homepage - Before and After

Project Card - Before and After

@adrianthedev can you review this PR please?

GG @colin-johnston!