💄 Mobile | Update quiz results page (requires designer review)

Pain:
The existing Quiz Results page leaves a lot to be desired. While we had some great designs from Joe we had to quickly adapt and change the existing design to fit with the AI Quiz structure.
Solution:

[x] Solicit a new design for the Results page
[x] Update this PBI with an @adamcogan approved design screenshot
[x] Implement the design

### Tasks
- [ ] https://github.com/SSWConsulting/SSW.Rewards.Mobile/issues/844
- [x] Implement Design

Done - this was completed here https://github.com/SSWConsulting/SSW.Rewards.Mobile/issues/844


Add an attributes parameter to toJSON on model, which returns whitelisted properties.

Add an attributes parameter to toJSON method which, when passed, runs a whitelist filter on returned attributes
Example Use:
model.toJSON(['one', 'two']) == {one: 1, two: 2}
Use Case:
When getting JSON representations of the model we want to send certain attributes to certain endpoints, and other attributes to other endpoints

What do you think about the case, when there are some dirty attributes that have been changed, but should not be sent to the server after the save method invoke. Right now I've just created method that selects only these attributes that need to be sent right now.
saveAttributes: (attributes, options={}) ->
  data = {}
  _(attributes).each (attribute) =>
    data[attribute] = @get(attribute)

  params =
    data: $.param(data)

  _.extend(params, options)

  Backbone.sync('update', @, params)

But the same method I need to use in other models too. So actually easy to extend other models, but it feels a little dirty, and I guess there are must have something similar like that.
Please let me know any thoughts on that.
Thanks!


feat: add graph_stores, impl Simple KG & Nebula KG

draft for RFC #1318

add graph_stores
implement Simple kg and NebulaGraph kg

WIP:

[x] get comments from jerry
[x] continue to rebase for changes in 0.6.0(introduce graph_store to storage context)
[x] implement load_from_disk and save_to_disk not needed after 0.6.x

@wey-gu this is cool, thanks for the PR - will try to take some time today to review and offer suggestions on how to rebase

Hey @wey-gu this is great! As @jerryjliu noted, we made some fairly significant changes to how we handle storage (see https://gpt-index.readthedocs.io/en/latest/how_to/storage.html)

Happy to help and answer any questions you might have about 0.6.0

awesome yeah @wey-gu adding on to what @Disiok said, it's possible a new graph store abstraction would be mostly used for specific indices, like our knowledge graph index. can be an optional part of our StorageContext that's none by default

Dear @Disiok

Some specific notes:

for storage based on external connection (e.g. NebulaGraph) here, we no longer try to save the configuration, instead we ask user to reconstruct the connection

wey: Got it, now I don't have to implement that :)

we created a storage context that bundles docstore, index store, and vector store. The question here is whether if we should add another graph store object into it. Would be great if you take a look at 0.6.0 and let us know your thoughts.

wey: This new abstraction is awesome, is it possible to enable chain-able storage context? The knowledge_graph index(now I added graph_store) comes with embedding support, which is memory based, is it possible to enable this embedding storage inside the knowledge_graph index consuming storage context in the future?
Dear @jerryjliu

awesome yeah @wey-gu adding on to what @Disiok said, it's possible a new graph store abstraction would be mostly used for specific indices, like our knowledge graph index. can be an optional part of our StorageContext that's none by default

wey: now the graph_store was introduced towards StorageContext and knowledge_graph_index was adapted to be based on simpleGraphStore or nebulaGraphStore :)

What do you think please of this change?
I will be working on more typical stories/demos for documents to help users understand how it works and how it helps

better consuming global/cross-node context
consuming existing knowledge graph in a custom retriever
composable index co-existing with other indexes
build a knowledge graph by simply dragging/putting docs in

in separate doc PR + blogs after this is merged.
Thanks again! I am super excited about this change :D
BR//Wey

Sorry, will fix the lint and UT issue.

linting and UT passed now.
cc @Disiok @jerryjliu

pushed another version to address the conflicts related to changes from GPTKnowledgeGraphIndex to KnowledgeGraphIndex by @Disiok

Thanks @logan-markewich for helping with the review!
Also, another rebase to resolve the lint error..

This looks super good, and thanks for tackling such a complicated PR!
A few minor nits, but my main worry is backwards compatibility (both with previously saved knowledge graph indexes, as well as previously saved indexes in general)
The storage context code likely needs a bit of TLC to be more robust
Lastly, and maybe not this PR, but some docs on using Nebula would be cool. (I actually still need to try setting it up lol)

Dear @logan-markewich ,

For backward compatibility, I am trying to add previously saved knowledge graph indexes support(or provide a converting tool) today.
I will revisit and improve the storage context part now
I'll add docs of NebulaGraph later in upcoming PRs! For now, in case you are about/interested in setting it up, on your own machine, you could go with docker extension, and on a server with docker, you could go with nebula-up, both of which are one-liner or one-click, and then we could create needed space and schema from NebulaGraph console or Studio following existing notebook's "Prepare for NebulaGraph" chapter.

I think this is good to ship! Looking forward to your future work @wey-gu ! Knowledge graphs can be very powerful, and hoping llama-index can continue to be a great tool to leverage them

Addressed a majority of your comments, I think it's good to land for now :)

@wey-gu this is an amazing change, thanks for the contribution. thanks to @logan-markewich @Disiok for the reviews too. just a heads up, planning to publish this friday morning pacific time - let me know your twitter handle!

@wey-gu this is an amazing change, thanks for the contribution. thanks to @logan-markewich @Disiok for the reviews too. just a heads up, planning to publish this friday morning pacific time - let me know your twitter handle!

Dear @jerryjliu ,
Thanks so much, I am honored to have the chance to bring something the Llama Index project and it's been an awesome experience working within the great Llama community with you, @logan-markewich and @Disiok , I am working on upcoming PR and demo-project/video on top of this change.
My handle is wey_gu :)
Thanks!

I think this is good to ship! Looking forward to your future work @wey-gu ! Knowledge graphs can be very powerful, and hoping llama-index can continue to be a great tool to leverage them

Dear @logan-markewich ,
Many thanks for the great help and guide(and big thanks to @Disiok !!)!! I am preparing for upcoming PRs/DEMOs, let's make LLMs understand more knowledge with graphs!
BR//Wey

Need to load the existing data indexes from nebula graph into the retriever for knowledge graph query_engine. Is that functionality available in llama-index ?

Need to load the existing data indexes from nebula graph into the retriever for knowledge graph query_engine. Is that functionality available in llama-index ?

For text2cypher, yes, it's already implemented! Follow https://gpt-index.readthedocs.io/en/latest/examples/query_engine/knowledge_graph_query_engine.html is all needed.
For Graph RAG(find major entities with keyword or embedding from the task, get subgraph as context), not yet fully supported. I am thinking of adding this soon.

Need to load the existing data indexes from nebula graph into the retriever for knowledge graph query_engine. Is that functionality available in llama-index ?

@pachgadehardik
For text2cypher, yes, it's already implemented! Follow https://gpt-index.readthedocs.io/en/latest/examples/query_engine/knowledge_graph_query_engine.html is all needed.
For Graph RAG(find major entities with keyword or embedding from the task, get subgraph as context), not yet fully supported. I am thinking of adding this soon.

Thanks a lot @wey-gu  for the update. However  I am facing an issue while running KnowledgeGraphQueryEngine. NebulaGraphStore is being loaded but when executing the KGQueryEngine, was facing a query syntax -
ValueError: Query failed. Query:
MATCH ()-[e:relationship]->()
WITH e limit 1
MATCH (m)-[:relationship]->(n) WHERE id(m) == src(e) AND id(n) == dst(e)
RETURN "(:" + tags(m)[0] + ")-[:relationship]->(:" + tags(n)[0] + ")" AS rels
, Param: {}Error message: Scan vertices or edges need to specify a limit number, or limit number cannot push down.

Need to load the existing data indexes from nebula graph into the retriever for knowledge graph query_engine. Is that functionality available in llama-index ?

@pachgadehardik
For text2cypher, yes, it's already implemented! Follow https://gpt-index.readthedocs.io/en/latest/examples/query_engine/knowledge_graph_query_engine.html is all needed.
For Graph RAG(find major entities with keyword or embedding from the task, get subgraph as context), not yet fully supported. I am thinking of adding this soon.

Thanks a lot @wey-gu for the update. However I am facing an issue while running KnowledgeGraphQueryEngine. NebulaGraphStore is being loaded but when executing the KGQueryEngine, was facing a query syntax - ValueError: Query failed. Query: MATCH ()-[e:relationship]->() WITH e limit 1 MATCH (m)-[:relationship]->(n) WHERE id(m) == src(e) AND id(n) == dst(e) RETURN "(:" + tags(m)[0] + ")-[:relationship]->(:" + tags(n)[0] + ")" AS rels , Param: {}Error message: Scan vertices or edges need to specify a limit number, or limit number cannot push down.

Dear @pachgadehardik
Could you share the NebulaGraph version? If it's an older version of NebulaGraph like 3.1.0, it's highly recommended to be upgraded to NebulaGraph 3.5.0, this is basically just a binary replacement(offline)

Need to load the existing data indexes from nebula graph into the retriever for knowledge graph query_engine. Is that functionality available in llama-index ?

@pachgadehardik
For text2cypher, yes, it's already implemented! Follow https://gpt-index.readthedocs.io/en/latest/examples/query_engine/knowledge_graph_query_engine.html is all needed.
For Graph RAG(find major entities with keyword or embedding from the task, get subgraph as context), not yet fully supported. I am thinking of adding this soon.

Thanks a lot @wey-gu for the update. However I am facing an issue while running KnowledgeGraphQueryEngine. NebulaGraphStore is being loaded but when executing the KGQueryEngine, was facing a query syntax - ValueError: Query failed. Query: MATCH ()-[e:relationship]->() WITH e limit 1 MATCH (m)-[:relationship]->(n) WHERE id(m) == src(e) AND id(n) == dst(e) RETURN "(:" + tags(m)[0] + ")-[:relationship]->(:" + tags(n)[0] + ")" AS rels , Param: {}Error message: Scan vertices or edges need to specify a limit number, or limit number cannot push down.

Dear @pachgadehardik
Could you share the NebulaGraph version? If it's an older version of NebulaGraph like 3.1.0, it's highly recommended to be upgraded to NebulaGraph 3.5.0, this is basically just a binary replacement(offline)

@wey-gu, the current version deployed in kubernetes cluster is 3.4.0

Need to load the existing data indexes from nebula graph into the retriever for knowledge graph query_engine. Is that functionality available in llama-index ?

@pachgadehardik
For text2cypher, yes, it's already implemented! Follow https://gpt-index.readthedocs.io/en/latest/examples/query_engine/knowledge_graph_query_engine.html is all needed.
For Graph RAG(find major entities with keyword or embedding from the task, get subgraph as context), not yet fully supported. I am thinking of adding this soon.

Thanks a lot @wey-gu for the update. However I am facing an issue while running KnowledgeGraphQueryEngine. NebulaGraphStore is being loaded but when executing the KGQueryEngine, was facing a query syntax - ValueError: Query failed. Query: MATCH ()-[e:relationship]->() WITH e limit 1 MATCH (m)-[:relationship]->(n) WHERE id(m) == src(e) AND id(n) == dst(e) RETURN "(:" + tags(m)[0] + ")-[:relationship]->(:" + tags(n)[0] + ")" AS rels , Param: {}Error message: Scan vertices or edges need to specify a limit number, or limit number cannot push down.

Dear @pachgadehardik
Could you share the NebulaGraph version? If it's an older version of NebulaGraph like 3.1.0, it's highly recommended to be upgraded to NebulaGraph 3.5.0, this is basically just a binary replacement(offline)

@wey-gu, the current version deployed in kubernetes cluster is 3.4.0

I see, the current schema fetching way is not compatible with the cluster that's older than 3.5.0, I'll pr to fix this today!
Thanks for letting me know this and sorry for it!

pachgadehardik

It should be fixed in #7204


Add docs for Ruff language server

Summary
This PR adds documentation for the Ruff language server.
It mainly does the following:

Combines various READMEs containing instructions for different editor setup in their respective section on the online docs
Provide an enumerated list of server settings. Additionally, it also provides a section for VS Code specific options.
Adds a "Features" section which enumerates all the current capabilities of the native server

For (2), the settings documentation is done manually but a future improvement (easier after ruff-lsp is deprecated) is to move the docs in to Rust struct and generate the documentation from the code itself. And, the VS Code extension specific options can be generated by diffing against the package.json in ruff-vscode repository.
Structure

Setup: This section contains the configuration for setting up the language server for different editors
Features: This section contains a list of capabilities provided by the server along with short GIF to showcase it
Settings: This section contains an enumerated list of settings in a similar format to the one for the linter / formatter
Migrating from ruff-lsp

[!NOTE]
The settings page is manually written but could possibly be auto-generated via a macro similar to OptionsMetadata on the ClientSettings struct

resolves: #11217
Test Plan
Generate and open the documentation locally using:

python scripts/generate_mkdocs.py
mkdocs serve -f mkdocs.insiders.yml

I haven't read through the original PR, so it's likely that some of what I bring up here has been discussed on that PR.
My first impression is that we shouldn't set ruff server in the spotlight. The server is an implementation detail. What people are looking for is how they can use Ruff in their editor. The beauty of the server is that most VS code users will unlikely ever know that they use ruff server. It just works for them.
I don't think the documentation has to change significantly. It's just the structure that needs to change a little. I suggest renaming the top-level Ruff Server navigation entry to Editors, similar to Prettier, and change Integrations only to cover what's Other integrations today.
I think this will also help to reduce some overlap between Integrations and Ruff Server today where both cover the server and Ruff LSP.
This raises a few questions.

What should we do with python-lsp-server and python-lsp-ruff. I'm leaning toward not mentioning them anymore. Or are there any workflows that these LSPs support that we don't?
The main unclear thing is how we document the unofficially supported editors. But maybe that becomes a sub-section at the very end in editors?

My first impression is that we shouldn't set ruff server in the spotlight. The server is an implementation detail. What people are looking for is how they can use Ruff in their editor. The beauty of the server is that most VS code users will unlikely ever know that they use ruff server. It just works for them.

I see what you mean. I like that way of framing Ruff's editor integration.

I don't think the documentation has to change significantly. It's just the structure that needs to change a little. I suggest renaming the top-level Ruff Server navigation entry to Editors, similar to Prettier, and change Integrations only to cover what's Other integrations today.
I think this will also help to reduce some overlap between Integrations and Ruff Server today where both cover the server and Ruff LSP.

I'm assuming that integrations using the ruff CLI would be moved in the specific editor setup section? For example, there are various plugins which can also be used for Neo(Vim) editor via ruff CLI as seen at the end.

What should we do with python-lsp-server and python-lsp-ruff. I'm leaning toward not mentioning them anymore. Or are there any workflows that these LSPs support that we don't?

I think it might be ok to remove python-lsp-ruff. The only difference between workflows I see based on the README is custom severities and providing more configuration options directly in the server settings instead of pyproject.toml.

The main unclear thing is how we document the unofficially supported editors. But maybe that becomes a sub-section at the very end in editors?

By this, I'm assuming you mean editors like PyCharm and Textmate. If so, then I think it should just be included in the "Editors" section given the rename. It wasn't included alongside other editors because the "The Ruff Language Server > Setup" section was specific to setting up the language server in those editors.

A small side note. Why is the ruff server section expandable and the other sections aren't?

It's because each section is in it's own file, otherwise the entire page becomes too cluttered with a lot of information. This is inspired by the uv docs. Internal message thread: https://discord.com/channels/1039017663004942429/1080996377611489350/1262939217437261907

Thinking about it a bit more...
Overall, there are two ways one could setup Ruff in their editor (except for VS Code where we provide an official extension):

Setting up the language server via some mechanism
Using the executable directly via some builtin mechanism or an external plugin

If there's just an "Editors" section, then we could include all different ways to setup Ruff in individual editors. But then I'm not sure if other sections (features, settings, etc.) goes under "Editors" because those sections are specific to the language server.

I tried to find other projects that document their editors along with the tool itself and couldn't find any good examples.

If there's just an "Editors" section, then we could include all different ways to setup Ruff in individual editors. But then I'm not sure if other sections (features, settings, etc.) goes under "Editors" because those sections are specific to the language server.

I think the features section fits well because it's the features you (might) get when using Ruff in your editor. Assuming the editor supports the required LSP features.
I agree, that the settings are a bit more LSP specific. But I think we have a VS code setting for every LSP setting? We could even discuss the Common settings, followed by the editor specific settings.

@MichaReiser I've updated the structure and changed some wording as well. I might need to read through it once to update the "Ruff Language Server" references. What do you think of the new structure?

Thank you for this PR, this is amazing, perfect for me.


⚠ SceneTime has degraded performance

In d31d8d0, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1472 ms

Resolved: SceneTime performance has improved in cf38070 after 6 minutes.


Handle RSA and Eliptic curve signing for JSON Web Tokens

I propose the following implementation:

a JwtSignature interface with the getSigner() and getVerifier() methods
a HMACSignature for HMAC signature
a ECDSASignature  for Eliptic Curve signature
a RSASignature for RSA signature

*Signature classes being used to build the JwtGenerator and JwtAuthenticator.

Oh no.. I just ran into this error: Unsupported JWS algorithm ES256, must be HS256, HS384 or HS512
How soon before it is implemented?

Planned for 1.9.1 so after the 1.9 release for which I must first finish: https://github.com/pac4j/pac4j/issues/558. That said, it could perfectly go into 1.9.
There are some other issues related to JWT support as well: https://github.com/pac4j/pac4j/issues/567, https://github.com/pac4j/pac4j/issues/568, https://github.com/pac4j/pac4j/issues/569 and https://github.com/pac4j/pac4j/issues/570.
The signers are different for the different family of encryption methods (ES, HS). The nimbusds jose library already supports all, but the pac4j JWT support must be changed to take that into account, which is feasible with various configurations (which I called Signature in my proposal above).
This is something you could perfectly work on and contribute to pac4j, if you find some free time.

PR: https://github.com/pac4j/pac4j/pull/621
Signature and encryption configurations are now available: https://github.com/leleuj/pac4j/tree/pac4j-619/pac4j-jwt/src/main/java/org/pac4j/jwt/config


Add School SMKN 1 DOMPU

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


Updates owner aliases.

Description of changes:
Adds more people in owner aliases.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest


bug: Can't copy text from the chat

Version
1.16.1
Describe the bug
When I am in the chat window and I want to copy text, neither the normal cmd+c nor the right click then picking "Copy" work for me. In fact it works intermittently and I don't know why honestly
Expected behavior
Selected text to be copyable from the chat window
Additional context
No response

I can reproduce this intermittently.

This is still present in version 1.18.0

Related bug, but not as bad as this one: you can't select code as it's streaming

For everyone who's not able to copy to clipboard from Cody chat, which version of VS Code are you on? They fixed a bug in VS Code in 1.89.0: https://github.com/sourcegraph/sourcegraph/issues/62361#issuecomment-2091490763. Could you please try upgrading to that version and reporting back?

I am on 1.89.1 still same issue.

For everyone who's not able to copy to clipboard from Cody chat, which version of VS Code are you on? They fixed a bug in VS Code in 1.89.0: sourcegraph/sourcegraph#62361 (comment). Could you please try upgrading to that version and reporting back?

Version: 1.89.1
Commit: dc96b837cf6bb4af9cd736aa3af08cf8279f7685
Date: 2024-05-07T05:14:32.757Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Darwin arm64 23.5.0
The issue is intermittent, that's why i didn't have a video when i opened the bug. for example i just tried now and i could copy, but last night i couldn't

Same issue here on VSCode 1.89.1, Win 11, I find alt tabbing will bring back copy functionality if it was bugged

@sqs The bug is really irritating as users can't utilize the tool fully. Even though it was reported multiple times, the request is being closed or not being paid attention (refer ). I am using VS Code 1.90.2 on Linux. Details attached.

@vmdamn I'm sorry you are still seeing this issue. I am not able to repro on Linux or macOS on the same Electron version. Could you please see if the same issue occurs in another webview, such as the Markdown preview?

Open any .md file
Run the Markdown: Open Preview to the Side command
Try copying in the rendered Markdown webview

I want to see if this is a Cody-specific problem, or if it is a webview problem.
Also, could you please try a few different ways of copying:

Selecting text and pressing Ctrl+C or Cmd+C
Selecting text, right-clicking, and clicking Copy

Hi @sqs,
I took the steps as you mentioned:
For a text based Cody answer, no code:

Right clicking and clicking copy is not working.
Manually moving the mouse over the selected text and then pressing Ctrl+C and Ctrl+V is working but destroying the formatting of the text. So not very useful if I want to save say certain bullet points in my notes.

For the code based answer, copy code button(first of the three icon buttons) is working. Right clicking and clicking copy is not working. Manually moving the mouse over the selected code and then pressing Ctrl+C and Ctrl+V is working.

@vmdamn OK, so to summarize:

right-click copy is not working at all
Ctrl+C copy works but loses formatting when you paste

Can you please try right-click copy in the Markdown preview (built into VS Code) as I suggested above? I am curious if that works for you in that Markdown preview webview.
As for losing formatting, I am not able to repro that. When I paste into Google Docs, it retains the bullet list, background color, foreground color, and font. Where are you trying to paste it into, so I can try to repro?

@sqs Yeah summary is right.
Copying (Ctrl+C) from markdown preview tab to my notes tab, formatting is working (if I deliberately insert formatting into .md file)
Copying (Ctrl+C) from a Cody answer tab to markdown(.md file) tab/my notes tab, formatting is not working.
Are you reproducing the whole environment to reproduce the bug? I don't see any reason why the bug cannot be reproduced if the whole environment is same.

@vmdamn

When you say "my notes tab", which application are you pasting it into? I want to repro the whole environment you are using so I can try to repro the bug.
Can you please try right-click copy from the Markdown preview (built into VS Code)? I want to see if the problem is with all VS Code webviews, or just Cody's webview.

I'm also having the same issue with Cody 1.28 and vscode 1.91.1 on Macos


Visual Studio 2015 building?

Hello. How to build bgfx for Visual Studio 2015?

https://bkaradzic.github.io/bgfx/build.html#quick-start


layout .m-recommended-slider is wrong if there are no images

hi everyone, I like the liebling theme for my blog, there are only a few css errors in the .m-recommended-slider class if the post is without an image, like the following, the post title is truncated

I already fixed it, please download the latest version (0.6.2).


Expected array

Laravel Version: 5.7.12
PHP Version: 7.2.9
Database Driver & Version: MySQL 8.0.13
monolog/monolog: 1.23.0 and 1.24.0 too

Description:
When the code throws out the event, in the console I see an error processing it:
ErrorException  : Expected array for frame 0

  at /home/vagrant/code/example.com/vendor/monolog/monolog/src/Monolog/Formatter/LineFormatter.php:144
    140|         }
    141|
    142|         $str = '[object] ('.get_class($e).'(code: '.$e->getCode().'): '.$e->getMessage().' at '.$e->getFile().':'.$e->getLine().$previousText.')';
    143|         if ($this->includeStacktraces) {
  > 144|             $str .= "\n[stacktrace]\n".$e->getTraceAsString()."\n";
    145|         }
    146|
    147|         return $str;
    148|     }

  Exception trace:

  1   Exception::getTraceAsString()
      /home/vagrant/code/example.com/vendor/monolog/monolog/src/Monolog/Formatter/LineFormatter.php:146

  2   Monolog\Formatter\LineFormatter::normalizeException(Object(ErrorException))
      /home/vagrant/code/example.com/vendor/monolog/monolog/src/Monolog/Formatter/NormalizerFormatter.php:95

  Please use the argument -v to see more details.

Test code:
Artisan::command('foo:bar', function () {
    $foo = $bar;
});

Error handling of exceptions in the console occurs whenever there is no catching of exceptions.

If you comment out the 146 line, then the monolog returns a real error, not its own.
Is this an error generated by the framework or an error in the package processed by it?
By the way, calling method getTrace() does not cause errors and returns an array. But calling the conversion method on a string will certainly cause an error.
Command dd($e->getTraceAsString()); also causes an error.

With the above test code I get the following exception:
$ ~/Sites/test-app  a foo:bar

   ErrorException  : Undefined variable: bar

  at /Users/driesvints/Sites/test-app/app/Providers/AppServiceProvider.php:17
    13|      */
    14|     public function boot()
    15|     {
    16|         \Artisan::command('foo:bar', function () {
  > 17|             $foo = $bar;
    18|         });
    19|     }
    20|
    21|     /**

  Exception trace:

  1   Illuminate\Foundation\Bootstrap\HandleExceptions::handleError("Undefined variable: bar", "/Users/driesvints/Sites/test-app/app/Providers/AppServiceProvider.php", [])
      /Users/driesvints/Sites/test-app/app/Providers/AppServiceProvider.php:17

  2   Illuminate\Foundation\Console\ClosureCommand::App\Providers\{closure}()
      /Users/driesvints/Sites/test-app/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php:29

  Please use the argument -v to see more details.

I'll need more info and/or code to debug this further. Please post relevant code like jobs, commands, notifications, events, listeners, controller methods, routes, etc. You may use https://paste.laravel.io to post larger snippets or just reply with shorter code snippets. Thanks!

This is both funny and not. I found a problem.
My project uses a package that is responsible for saving the exceptions to the database, then creating an application in jira and notifying in slack. This error was generated by the method of serialization of exceptions for writing. This is necessary in order to work with it as with the class of the executor during deserialization.
Why did the error occur in freshly installed applications? I first installed the entire package set ...


s3 stream filling available memory

As a basic example:
$s3Client = new S3Client([
    'credentials' => ['key' => 'S3_KEY', 'secret' => 'S3_SECRET'],
    'region' => 'eu-west-1',
    'version' => 'latest',
]);
$s3Client->registerStreamWrapper(); // do we still need this?
$s3 = new Filesystem(new AwsS3Adapter($s3Client, 'S3_BUCKET', 'S3_PATH'));

 (new Filesystem(new Local($path)))->writeStream($source, $s3->readStream($source));

and whilst monitoring free -m I see the memory usage going from the free column into the buff/cache column, it's a big file so I was hoping it wouldn't be using memory. I might be misreading the code but it seems as if it reads the entire remote file before streaming. readStream calls readObject which calls getObject on the s3Client, so doesn't seem to stream.
Also if the file already exists it doesn't report a file already exists error until it seems to have read the whole thing, which would seem to support my thoughts that it isn't being streamed properly.

@gabriel403 the adapter gets the stream from the aws sdk, we don't have controle over how this stream gets filled or when it's handed back to us. From our end this is the most memory usage friendly as we can get.

@frankdejonge so we call $s3Client->registerStreamWrapper(); and flysystem calls getObject (actually $this->s3Client->getCommand('getObject') on the s3Client, which actually returns a stream?

@gabriel403 you don't have to register the stream wrapper. The getObject command returns a object containing the stream. We detach the stream and return that.

OK, thank you @frankdejonge :)


[WIP] Unify and extend rpc methods to bid and sequence

A bunch of updated rpc keys can lead to breaking changes in users, note it uses camelCase name notation, it can be changed as well. Notable changes:
nomalized_name -> normalizedName
txid -> txId
nHeight -> height
nValidAtHeight (valid at height) -> validAtHeight
nAmount -> amount
nEffectiveAmount (effective amount) -> effectiveAmount
total names -> totalNames
total claims -> totalClaims
total value -> totalValue
Most values in getclaimsfortx

Changes are breaking since i unify outputs which makes changes to the fields.

The cli help (that shows all the available methods) isn't showing the parameters for the methods in rpc/claimtrie.cpp. It's like the argNames are ignored in that file.


Extension installed, don´t see anywhere to sign in.

I really would want to use this feature as it´s very useful so I´d appreciate some help.
I have the extension in vscode installed but I can´t use it at all because I can´t sign in. I´ve already tried Shift+Ctrl+B and type "Github" hoping that I could somehow log in.
 Can someone help?

Hey @Yachont, the extension should show a notification asking to sign in when it detects that the folder you have open has a GitHub remote. What's the result of running git remote -v in your folder? Can you also check what is in the Output Channel when selecting GitHub Pull Request from the dropdown?

Hello @RMacfarlane, thanks for the quick reply.
When I type git remote -v into a terminal I get this:
origin  https://github.com/slavarosin/jakob_praktikum.git (fetch) origin  https://github.com/slavarosin/jakob_praktikum.git (push)
This is the output from the Github Pull Request:
[Info] C:\Users\JakobsPC\.ssh\config: ENOENT: no such file or directory, open 'C:\Users\JakobsPC\.ssh\config' [Info] Looking for git repository [Info] Git repository found, initializing review manager and pr tree view. [Info] No GitHub remotes found [Info] Review> no matching pull request metadata found for current branch master
What does that mean?

Thanks for the info!
Here's a breakdown of what the output means:
The first line with ENOENT is ok, we check to see if there is a config file that has any aliases, in this case we didn't find any.
The extension finds that the current repository uses git and starts to initialize some parts.
The critical line is No GitHub remotes found, this means the extension wasn't able to resolve https://github.com/slavarosin/jakob_praktikum.git as pointing to GitHub for some reason.
Can you check what version of the extension you have? There was a bug with parsing remotes a while ago. The newest version of the extension requires VSCode version 1.31.0

My current version of VSCode is 1.31.1
Maybe I messed something up because it didn´t work and I tried to fix it so I went and did the steps here: https://code.visualstudio.com/docs/editor/versioncontrol -> Common questions
In a terminal I put these lines in:
git remote add origin https://github.com/slavarosin/jakob_praktikum git push -u origin master
Maybe that screwed something up?

Hmm. Those lines are fine, that should just be setting up the remote. You're on version 0.4.0 of the extension?

Yeah, I am. I even tried previous versions of the extension to see if that would do something. Still no luck.

The expected behavior when you install the extension is that this notification should show up:

and that the status bar should have a sign in button:

and also a command to sign in:

From the output log, it looks like the extension is activating. Can you open the command palette and run Developer: Toggle Developer Tools to see if there are any errors in the console?

@RMacfarlane thanks for helping me. I finally found a solution!! All I had to do was open the folder where the extension was, open the README.md file and then it asked me to sign in.
I wasn´t able to do that before. I am so happy. Thanks for helping me. It´s very appreciated.


UIF-300: Upgrade and mark httpclient as test scope

Before: this transitive dependency version of httpclient has a CVE
After: add test scope and it will not be transitive. It is used only in test, so the version upgrade should have no impact.

@confluentinc It looks like @xjin-Confluent just signed our Contributor License Agreement. :+1:
Always at your service,
clabot

Anyone?


old_div fixer misses some expressions

I'm confused by the behavior of the futurize script's old_div fixer.  Given the file below, I'm wondering:

Why does the fixer skip the second case?
Why doesn't the fixer skip the third case?

(Sorry if I'm missing something -- I'm experienced with Python 2, but I'm a Python 3 novice.)
# This line gets fixed
a1 = b / c

# But this line gets skipped?
a2 = 1*b / c

# Shouldn't this be skipped?
a3 = 1.0 / c

$ futurize foo.py
RefactoringTool: Skipping optional fixer: idioms
RefactoringTool: Skipping optional fixer: ws_comma
RefactoringTool: Refactored foo.py
--- foo.py	(original)
+++ foo.py	(refactored)
@@ -1,8 +1,10 @@
+from __future__ import division
 # This line gets fixed
-a1 = b / c
+from past.utils import old_div
+a1 = old_div(b, c)

 # But this line gets skipped?
 a2 = 1*b / c

 # Shouldn't this be skipped?
-a3 = 1.0 / c
+a3 = old_div(1.0, c)
RefactoringTool: Files that need to be modified:
RefactoringTool: foo.py

Works in current version

These were limitations in the fixer that were addressed in 0.17.0 and the upcoming 0.18.0


ECONNRESET trying azure deploy webapp VS Code

Using VS Code, on my webapp's publish folder I select "Deploy to web app", after running for a while it reports ECONNRESET in the lower right popup.  I've logged out/in, rebooted, you name it.  After searching it appears v1.66 is the cause that many others are running into here.  I was literally deploying fine yesterday then it just broke in an instance.  Is it VS Code, an Azure extension, ??
Very annoying, but I ended up using az webapp up to get my updated code deployed.  Which also seems to have the problem that the runtime cannot be discovered when I run from my publish folder.  If I run from the project folder it zips all my files and pushes it, which of course fails on the server ( also puts my source publicly viewable...).  To get it to work I copied the csproj file into the publish folder and that satisfied az webapp enough to zip and push the right files.  So, az webapp up needs to honor --runtime as well ( that seems to be a bug currently?)
Similar?
https://github.com/microsoft/vscode/issues/146577

Same here,

I am also facing the same issue.

It's most likely related to this issue: https://github.com/microsoft/vscode-azurefunctions/issues/2844
VS Code 1.66 upgraded to Node 16 which has been causing deployment issues for both Functions and App Service.  Unfortunately, the only current known workaround is to downgrade VS Code to 1.65.x.  We're currently investigating a proper fix.

Is this error related to the issue?
22:56:44: Error: request to https://blckt.scm.azurewebsites.net/api/zipdeploy?isAsync=true&author=VS Code failed, reason: socket hang up

Until a fix is released in VS Code stable (hopefully the next major release), here are some options:

Wait for the fix to come in VS Code Insiders
Deploy via Azure CLI
Downgrade your Azure App Service extension version to v0.21.1 (Mar 04, 2021), we used a different zip deploy method which seems to be working still. Note: it might not work in environments like WSL or Codespaces, which is why we changed our deploy in v0.21.2 (Mar 09, 2021).
Downgrade to the VS Code February 2022 release, where zip deploy is working as discussed and confirmed above.

Here's some information on the bug we suspect is causing the issue with zip deploy.
We suspect that it was a regression in the libuv library caused by libuv/libuv@ce15b84, which was included in Node.JS 16.13.0.
See this issue, and specifically this comment for more information.

Closing as a duplicate of #2191. Follow along there, or the corresponding Azure Functions issue for further updates and discussion.

It's most likely related to this issue: microsoft/vscode-azurefunctions#2844
VS Code 1.66 upgraded to Node 16 which has been causing deployment issues for both Functions and App Service. Unfortunately, the only current known workaround is to downgrade VS Code to 1.65.x. We're currently investigating a proper fix.

Thanks, at the moment using az webapp up is handling things, although in a less than ideal manner.

Fix has been released in version https://github.com/microsoft/vscode-azureappservice/releases/tag/v0.23.2 of the extension.


URL can't contain control characters

Klarer ikke å sende sms for Canary-alert
[2020-06-30 12:27:33 +0000] [11] [ERROR] Exception in ASGI application
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 385, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/usr/local/lib/python3.7/site-packages/uvicorn/middleware/proxy_headers.py", line 45, in __call__
    return await self.app(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/fastapi/applications.py", line 149, in __call__
    await super().__call__(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/applications.py", line 102, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc from None
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc from None
  File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 550, in __call__
    await route.handle(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 227, in handle
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 41, in app
    response = await func(request)
  File "/usr/local/lib/python3.7/site-packages/fastapi/routing.py", line 197, in app
    dependant=dependant, values=values, is_coroutine=is_coroutine
  File "/usr/local/lib/python3.7/site-packages/fastapi/routing.py", line 148, in run_endpoint_function
    return await dependant.call(**values)
  File "/app/main.py", line 43, in sms
    sendSMS(transformText(message), alert['annotations']['recipients'])
  File "/app/main.py", line 27, in sendSMS
    response = urllib.request.urlopen(url).read()
  File "/usr/local/lib/python3.7/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/local/lib/python3.7/urllib/request.py", line 525, in open
    response = self._open(req, data)
  File "/usr/local/lib/python3.7/urllib/request.py", line 543, in _open
    '_open', req)
  File "/usr/local/lib/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/usr/local/lib/python3.7/urllib/request.py", line 1347, in http_open
    return self.do_open(http.client.HTTPConnection, req)
  File "/usr/local/lib/python3.7/urllib/request.py", line 1319, in do_open
    encode_chunked=req.has_header('Transfer-encoding'))
  File "/usr/local/lib/python3.7/http/client.py", line 1252, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/local/lib/python3.7/http/client.py", line 1263, in _send_request
    self.putrequest(method, url, **skips)
  File "/usr/local/lib/python3.7/http/client.py", line 1114, in putrequest
    self._validate_path(url)
  File "/usr/local/lib/python3.7/http/client.py", line 1205, in _validate_path
    raise InvalidURL(f"URL can't contain control characters. {url!r} "
http.client.InvalidURL: URL can't contain control characters. '/sms?recipients=***&message=[FIRING]%20Canary%20does%20not%20sing%20in%20prod-gcp%0aProblem%20with%20NAIS%20deploy\n\nCanary%20application%20should%20deploy%20every%205%20minutes,%20but%20has%20not%20been%20deployed%20successfully%20in%20over%2020%20minutes.\n\nConsequence:%20NAIS%20deploy%20is%20probably%20unavailable.%20Some%20or%20all%20users%20might%20not%20be%20able%20to%20deploy.\n&messageType=TEXT&LogDelivery=NO&username=***&password=***' (found at least '\n')

Slack-melding
Canary does not sing in prod-gcp
Problem with NAIS deploy
Canary application should deploy every 5 minutes, but has not been deployed successfully in over 20 minutes.
Consequence: NAIS deploy is probably unavailable. Some or all users might not be able to deploy.

Alerten som definert i nais/nais-yaml

Problem with NAIS deploy\n\nCanary application should deploy every 5 minutes, but has not been deployed successfully in over 20 minutes.\n\nConsequence: NAIS deploy is probably unavailable. Some or all users might not be able to deploy.\n

De forhåndsdefinerte \n feiler, så de må enten fjernes eller bare byttes til sms-linjeskift. Tenker sistnevnte er enklest.

Fixed https://github.com/nais/smsmanager/commit/1a2e6cd7eb1487646cb011616ea804856c22a2c6


TreeDataGridRow  Binding Error

The property IsVisible binding of TreeDataGridRow does not work,
Here is the code
    <Style Selector="TreeDataGridRow">       <Setter Property="IsVisible" Value="{Binding (interfaces:IDataNode).IsDisplay,DataType={x:Type interfaces:IDataNode}}"></Setter>     </Style>
When I modify the value of IsDisplay, TreeDataGridRow is not hidden, perhaps

Also seeing this binding fail. Any pointers here?


POC: Site: Add possibility for blocks to register a loader function that allows loading additional data

This is triggered by getStaticProps by recursively analzying the block data (using block meta data)
(+ some changes needed in preparation for this, see individual commits)
This provides another alternatives for blocks to load data. Other alternatives:

client side fetch (with obvious drawbacks)
add more data in transformToPlain (needs api change for frontend, difficult to inject dependencies there, also called in admin)
use react server side components and fetch during runtime (requires next 13 and app router (which we want to switch to anyway), does un-solveable waterfall loading, un-solveable(?) for block preview)

TODO:

[ ] move reusable parts into @comet/cms-site
[ ] avoid waterfall loading (call all loders in pralell)
[ ] block preview is not yet working
[ ] page preview is not yet working

@nsams first we should merge https://github.com/vivid-planet/comet/pull/1338, right?

yes, this was split out as individual PR (that's why it's not dependent) - once that is merged I will continue working here

Also, can this PR target main? This should be a non-breaking addition

I hope so

@chernylu FYI 🙂

What about the dependency on graphql-request? Do you intend to keep it, or try using module augmentation?

first we should decide on that:
#1681 / #1682

What about the dependency on graphql-request? Do you intend to keep it, or try using module augmentation?

first we should decide on that: #1681 / #1682

See https://github.com/vivid-planet/comet/pull/1681#pullrequestreview-1900684210

Now that we are using Next 14, couldn't we also have the blocks themselves retrieve data with streaming + suspense?


File abstractions for project config and env

Adds interfaces for reading/writing agentstack.json and .env files; ConfigFile and EnvFile, respectively.
Ensures we're being responsible with user data and makes testing easy, too!

Relies on the work I did yesterday, so once that gets merged this will be a much smaller changeset.

this is the most beautiful thing i didnt even know i wanted


shardspec visibility fix

Fixes #shardspec visibility.

shardspec visibility

shardspec visibility

This PR has:

[x] been self-reviewed.

[x] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Thanks for the review and the approvals. :)


Delayed bullet selected index

Is it possible to update the bullets when the swipe starts, rather than ends?
When you scroll through the slides fast, the select bullet stays, and then jumps multiple bullets when i'm done swiping.
gif:

i'm exaggerating the speed in the example, but i've felt it when swiping on a real device too
its not a huge deal, but it adds that little extra feeling of snappiness :)

Should be easy with code from #181


prevent Copy and Drop impls for same type

According to https://doc.rust-lang.org/reference/special-types-and-traits.html#copy one also has to check that Drop is not implemented for a type that implements Copy. As far as I can tell this can't be stated currently as there is no logical negation nor a predicate that asserts that a trait is not implemented.
Originally posted by @ltentrup in https://github.com/nikomatsakis/a-mir-formality/issues/10#issuecomment-1107448775

We probably want to add a not-provable { ... } goal into the solver so that we can express this -- but I wouldn't want that exposed as a where-clause.


Reduce page loading time

Can someone please figure out why our page is often loading 7+ seconds when initially loaded?

When does this happen? I analyzed the loading performance in Chrome and the page was complete after about 700ms, even with 5x CPU throttling.

@bjrne It has to render the less, css and js files on initial load. I think in an dev env we have to live with that - aggressive caching is our enemy. In production we have several options to work with caching, depending on the env itself.

It's pretty fast. ~600ms + ~500ms loading of pictures with emptied cache

Hat sich erledigt, oder?

Looks like it. I experienced different times a week ago. Seems fine now.


[Communication][Email] Adding PPE live tests

Adding PPE SDK live tests for the email communication package.
Follow up to https://github.com/Azure/azure-sdk-for-js/pull/24175. That PR added email resource creation to a common resources template shared by all communication tests. However, only communication-email package needs the email resource for its test pipeline. So this PR adds an email-specific resource template, while reverting changes to the common template. This also includes changes to the common file used for the live testing pipeline.

/azp run prepare-pipelines

API change check
API changes are not detected in this pull request.

/azp run prepare-pipelines

/azp run python - azure-communication-identity - tests


Update wdlconfig.qmd: smoorthi edits continued

Here are some additional changes I thought of.

@sitapriyamoorthi -- I totally missed this pull request earlier, really sorry! I brought it up to speed with the main branch and resolved the merge conflicts to incorporate the changes. This look okay?


Added step-function-name as a parameter to the create command

this pull request is to give the user the ability to define step function name on AWS to be different from the flow name when creating it
this could be helpful when updating the step function using the pipeline then you can define which step function to update based on the current branch

So we already accomplish something similar by setting METAFLOW_SFN_STATE_MACHINE_PREFIX env var when deploying/triggering.  Perhaps it makes more sense to expose this option at the CLI-level, so that the name still has some bearing to the actual flow name?
@savingoyal, re: your question of how such a prefix/name option would work when triggering, I think the implication is that users are responsible for setting the same prefix/name at trigger time if they wish to target the same SFN.

PRs #540 , #532 and #541 have implemented this feature.


Add support for nested directories and modules

Describe Request:
Add support for nested directories with modules, so we can target subdirectories/paths like
terraform/development/
terraform/production/

Examples:
N/A
Supporting Documentation Links:
N/A

Hi @bjd2385 - fwiw, I'd appreciate this feature. It would be helpful to be able to use dynamic-continuation-orb with a monorepo directory structure like:

apps/
--- app1
--- app2
packages/
--- pkg1
--- pkg2

This approach is documented nicely by Robin Wieruch at https://www.robinwieruch.de/javascript-monorepos/
Thanks!

@gnaryak I just published an update that I've tested pretty thoroughly since last night. Let me know if you have any issues, happy to help.

Hey, that's great. Thanks, @bjd2385 ! It'll be really nice to be able to use an optimal hierarchical directory structure in our monorepo. We are incorporating this into our documented best practices.
It may be a week or three before we are at a place to be able to give it a thorough test but we are pointed in that direction. I'll reach out if I have any feedback.
Thanks!

@jalgraves this fix adds a feature I think you requested many months ago.


Don't set overrides on imported protocol requirements [5.0]

Fixes rdar://problem/42635132.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

Build failed
Swift Test OS X Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

@swift-ci clean test linux

Build failed
Swift Test OS X Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

Build failed
Swift Test Linux Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

Build failed
Swift Test Linux Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

Build failed
Swift Test OS X Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

@shahmishal It looks like we're still checking out the wrong lldb branch?
/home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/lldb/source/Plugins/ExpressionParser/Swift/SwiftASTManipulator.cpp:598:34: error: no matching function for call to 'create'
02:53:26   result_loc_info.binding_decl = swift::PatternBindingDecl::create(
02:53:26                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
02:53:26 /home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/swift/include/swift/AST/Decl.h:1946:30: note: candidate function not viable: requires 6 arguments, but 7 were provided
02:53:26   static PatternBindingDecl *create(ASTContext &Ctx, SourceLoc StaticLoc,
02:53:26                              ^
02:53:26 /home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/swift/include/swift/AST/Decl.h:1952:30: note: candidate function not viable: requires 8 arguments, but 7 were provided
02:53:26   static PatternBindingDecl *create(ASTContext &Ctx, SourceLoc StaticLoc,
02:53:26                              ^
02:53:26 /home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/lldb/source/Plugins/ExpressionParser/Swift/SwiftASTManipulator.cpp:1239:7: error: no matching function for call to 'create'
02:53:26       swift::PatternBindingDecl::create(
02:53:26       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
02:53:26 /home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/swift/include/swift/AST/Decl.h:1946:30: note: candidate function not viable: requires 6 arguments, but 7 were provided
02:53:26   static PatternBindingDecl *create(ASTContext &Ctx, SourceLoc StaticLoc,
02:53:26                              ^
02:53:26 /home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/swift/include/swift/AST/Decl.h:1952:30: note: candidate function not viable: requires 8 arguments, but 7 were provided
02:53:26   static PatternBindingDecl *create(ASTContext &Ctx, SourceLoc StaticLoc,
02:53:26                              ^
02:53:26 /home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/lldb/source/Plugins/ExpressionParser/Swift/SwiftASTManipulator.cpp:1502:26: error: no member named 'getStorageKind' in 'swift::VarDecl'
02:53:26     if (variable.m_decl->getStorageKind() !=
02:53:26         ~~~~~~~~~~~~~~~  ^
02:53:26 /home/buildnode/jenkins/workspace/swift-PR-Linux/branch-swift-5.0-branch/lldb/source/Plugins/ExpressionParser/Swift/SwiftASTManipulator.cpp:1503:37: error: no member named 'Computed' in 'swift::AbstractStorageDecl'
02:53:26         swift::AbstractStorageDecl::Computed)
02:53:26         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^

Maybe @rjmccall needs to cherry-pick some patches on the lldb side?

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

@fredriss Have you seen this before?
/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-swift-5.0-branch/lldb/source/Plugins/ExpressionParser/Swift/SwiftASTManipulator.cpp:598:34: error: no matching function for call to 'create'
  result_loc_info.binding_decl = swift::PatternBindingDecl::create(
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@shahmishal never seen this

There have been no modification involving PatternBindingDecl::create in LLDB since 2015...

Build failed
Swift Test OS X Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

No, see my comment above.  A patch to that code was committed on July 6th as  https://github.com/apple/swift-lldb/commit/906c4bf9d7262fdd307521732f824656820d3cd1, and it should be in swift-5.0-branch.

Are you talking about 906c4bf9d7262fdd307521732f824656820d3cd1? because according to 'git branch --contains' it is on swift-5.0-branch. But this patch doesn't touch the line where the error is reported

Doh, it touches exactly this piece of code, sorry. The commit is on swift-5.0-branch

Sorry, I'm reacting to Mishal's message, but I should have read the history of the PR. John analysis is exactly correct, this build has checked out the swift-4.2-branch in LLDB.

@swift-ci Please test macOS

Build failed
Swift Test OS X Platform
Git Sha - e440e1dd6c495ac7ca92b9f745ff318540191422

@swift-ci Please test macOS


add Netty sanity check tests

Type of change

Enhancement / new feature

Description

verify io_uring transport
verify epoll transport

Additional Context
Validate presence of Netty transports.
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Review performance test results. Ensure that any degradations to performance numbers are understood and justified.
[ ] Make sure all Sonarcloud warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

Do you have a particular need for these tests? To me they are just validating that we build with the dependencies we declare. It might be much more interesting if we can run this as part of image building or something to assert that the container had the desired libraries but I'm not sure what value they add at the junit stage.

@sullis can you shed some light on your motivation for this PR?

@sullis Thanks for your interest in Kroxylicious and we are always interested in improving the value of our tests suite however we aren't convinced that these tests do anything more than validate that maven built the requested test classpath so see little value in them.
Please reopen the PR and show us what is wrong with our understanding if you disagree.


Implement consistent behaviour on 409 (Conflict) to force deployments

In https://github.com/mesosphere/marathon-ui/pull/202/files we implement the force action upon a 409 response from the REST API. We should do similarly whenever the UI is returning an error suggesting to use the force=true, as is the case when editing an app's definition which is currently being deployed:

Definitely, now having a look at some marathon internals for fun and profit, but I could to the same I already did in https://github.com/mesosphere/marathon-ui/pull/202/files

Hi @Raffo – thanks for jumping in! That would be amazing.
In terms of UI, it might make sense to simply swap the Change and deploy configuration button text with Force change and deploy configuration to avoid stacking up modal dialogs – what do you think?

@pierlo-upitup I would also avoid stacking up modal dialogs. Are you proposing of forcing changes by default? I would not like to do that as it would be inconsistent with the implementation in case of scaling. We could keep the message as it is, but add something like "press again to force change" or something similar. What do you think?

Exactly – I suggest to replace the button text after the error is shown, to indicate the "press again" to force a new deployment.

I would only change the text of the message message (the one in red in the picture above) and add "press again to force change" to it. I would not modify the text of the button between user interactions (I think it could be confusing for the users) though.

Yeah, I see how that might end up being confusing.
On the other hand that error message comes from the API as-is, so we will probably have to just use a custom message as you suggest.

In the end it's not different from what we are already doing in https://github.com/mesosphere/marathon-ui/pull/202/files so I see no problems with that.

Oh yes, I see what you mean. Sure thing!


Can't see country dropdown in address selection of Buy Now

I'm not sure what the solution is other than maybe scroll down the window a bit when they focus on the dropdown.

that UI life... it never ends.


fix: fix link all

Summary:
Fix linkAll to link root assets.
@sijad can you test link with those changes?

I've applied these changes manually, and it works, thanks for fast response.

@sijad Thank you C:


Fixed #27700 -- Optimized inspectdb tests by specifying database tables.

Ticket 27700.

Buildbot, test on Oracle.

Wow, great!


Sticky scroll for screen reader users

Sticky scroll is a feature that allows sighted users to understand the nested context that they are in.
@kieferrm suggested it would be cool if we had this for screen reader users.
Imagine you jump to a line where there's a problem reported. You invoke a command which provides context - what class you are in, the function signature, the conditional, etc.
cc @rperez030 and @jooyoungseo

I would love to understand the visual experience better. I often use commands such as "focus on outline view" for context, but maybe there is a better way. Understanding context certainly can be a challenge.

Yes, sticky scroll allows users to see their current context in the current view - even when the class or function is defined far above the current line.
Example:
line 1: class Telephone
line 2...36
line 37 function _createCall()
line 38....64
line 65 if _cellNumber === 10
line 65...80
line 81 (current line) x += 8.
Before sticky scroll, only lines 65 through 81 would be in the view, so the context of those things was lost.
With sticky scroll for sighted users, here's the view
class Telephone
function _createCall()
if _cellNumber === 10
(current line) x += 8.
so, for SR users, we could have a command which allows you to hear the current context like "in class telephone, function createCall, condition _cellNumber === 10"

@meganrogge what is the current issue? I have been using Sticky Scroll with JAWS and NVDA without challenges.

FYI, I have assigned the following keybindings:
[
  {
    "key": "ctrl+k ctrl+;",
    "command": "editor.action.focusStickyScroll"
  }
]

Oh that is very cool 😄 I had not realized one could focus the components of sticky scroll. It works well. Thanks for pointing this out @jooyoungseo

For those using screen reader and visiting this thread later, I have additionally configured the following settings:
settings.json:
{
  "editor.stickyScroll.enabled": true,
  "editor.stickyScroll.defaultModel": "foldingProviderModel"
}

Hey @meganrogge, my apologies, the focusing behavior was there for a couple of iterations already. I forgot to inform this thread of its existence.

@jooyoungseo , thanks a million! I had no idea sticky scroll even existed. These are the things I wish were better documented. Thanks @meganrogge  for explaining it. I will definitely play with this when I'm back from vacation. Thank you all.

@jooyoungseo  Thank you very much, I was just looking for this.

@aiday-mar this is a very useful feature - are there plans to enable it by default? If so, we can then add info about how to use it to the editor's help menu

Hi @meganrogge, focusing can be done directly through the command palette, without the need for a setting configuration.
Do you mean perhaps, adding a setting to focus directly on sticky scroll when it appears?

@aiday-mar for me, sticky scroll is off by default.

Ah I see, I misunderstood. I can make a PR to enable sticky scroll when screen reader optimization is on, by default.


jira: use the issue Key instead of ID

The issue ID is used only on the backend, while the UI uses the Key to identify an Issue (e.g. OCPBUGS-12).
At the current state, the ID is used to identify the issues on the UI of ci-search:

Since the ID is not indicative for users, this PR changes that to the Key value:

The same is changed for the REST API.

/cc @bradmwilliams

/lgtm


typo for string2 in dice_coefficient in metrics.py

why cannot this branch be merged into master?
I have only corrected a typo in documentation of a function.
Please help me. I am a beginner and new to open source.


command code-run.runner

Issue Type: Feature Request
i am getting an issues like 'command' code-run.runner can u please fix this issue man i am tied of to fix this
VS Code version: Code 1.47.0 (d5e9aa0227e057a60c82568bf31c04730dc15dcd, 2020-07-09T08:02:06.629Z)
OS version: Windows_NT x64 10.0.20161

Can you show me a screenshot of the error?

see at the corner of the photo please
On Mon, Jul 13, 2020 at 2:15 PM João Moreno<EMAIL_ADDRESS>wrote:

Can you show me a screenshot of the error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/102301#issuecomment-657414639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APEXJBQWLS3E36GK7HE26XLR3LCRJANCNFSM4OX3NQDA
.

Please come to Github and upload the screenshot, since e-mail attachments don't work.

this is the one i am getting

once see this please help me with this

i uploaded the screen shot can u check that out in github
On Mon, Jul 13, 2020 at 2:51 PM João Moreno<EMAIL_ADDRESS>wrote:

Please come to Github and upload the screenshot, since e-mail attachments
don't work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/102301#issuecomment-657436141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APEXJBUBU3G3FIJ5C22XE2DR3LGYLANCNFSM4OX3NQDA
.

Please file an issue to the Code Runner extension: https://marketplace.visualstudio.com/items?itemName=formulahendry.code-runner


feat: allow auto play on mixcloud player

Allow auto playing on mixcloud player, bringing it in line with soundcloud

Published in 2.11.0. Sorry for the delay.


PR: Display prompt after errors

This solves issue #4504  but will sometimes display two in[]

To clarify:
There is always an extra In[] when an exception occur with this workaround. But at least the last input line is clean.

@dhirschfeld, what do you think about @impact27's solution?

It seems a bit of a quirky solution - always insert an extra prompt after an exception and sometimes the exception will be printed in that prompt instead of associated with the prompt which caused it.
That said, as a work-around I think it does lead to a better end-user experience as you don't have to escape out of a traceback anymore.
If the underlying issue is with ipykernel perhaps it should be fixed there?

It is definitely a work-around. If this is an issue with ipykernel wouldn't it show in every iPython consoles? Do you think we should open a bug report with ipykernel even though this might be related to spyder only?

Our tests are failing in PyQt4, but I'm not exactly sure how to fix them...

If the error is "AttributeError: 'super' object has no attribute '_handle_error'" I could add a test to check if super has _handle_error? Worst case the msg is not handled, which means the error is not printed. I am not sure how this is happening?

I could add a test to check if super has _handle_error? Worst case the msg is not handled, which means the error is not printed

I agree with this option, but please do it only for PyQt4. There's a constant in qtpy called PYQT4 that let's you check if Spyder is running with those bindings.

I am not sure how this is happening?

No idea either, but it's not so serious because very few people is using PyQt4 now.

Hello @impact27! Thanks for updating the PR.

In the file spyder/widgets/ipythonconsole/shell.py, following are the PEP8 issues :

Line 411:77: W291 trailing whitespace
Line 413:11: E111 indentation is not a multiple of four
Line 417:1: W293 blank line contains whitespace


Document how to run a local development environment

Right now there's no documentation and I'll probably forget how to run this in a matter of days

This is done now!


Windows config.toml set to /usr/bin/mpv, causing error

The config is causing this error: "Failed to run downloader "mpv": The system cannot find the file specified. (os error 2)"
All is well after editing these lines to:
bin = "mpv"
player = "mpv"

Not sure if this is relevant, but I have mpv installed via chocolatey

Please see
README - Windows - Manual installation and README - Configuration - Custom configuration

Perhaps for the Windows download, you could include a different config.toml with just "mpv" instead of "/usr/bin/mpv"? It would cut out the extra step of having to edit the file.

Perhaps for the Windows download, you could include a different config.toml with just "mpv" instead of "/usr/bin/mpv"? It would cut out the extra step of having to edit the file.

Okay, I will do it at next release.
But it isn't useful for most Windows users (because they're not add mpv to the PATH environment variables).
Currently, you can manually modify their values to your want.

Thanks, it will be great for those of us who do have mpv added to the path. I install mpv via chocolatey, which does add it: https://community.chocolatey.org/packages/mpv
Speaking of which, do you have any interest in making mpv-handler available as a chocolatey package?

Speaking of which, do you have any interest in making mpv-handler available as a chocolatey package?

Maybe... After the mpv-handler config file stabilization
(Because the config file will be changed in the next release. I want to keep it simple and stupid)


old armor scale cannot be disabled?

forge 1.18.2
old armor scale with durability cannot be disabled (after having it enabled?). I have set it to false on both client and server, reloaded, restarted the world, and even my game, yet armor still decreases with durability. At first I thought maybe it had permanently altered old armor so we just have to get new armor, yet that also lost protection with durability.

tested it and i couldn't get this issue.
if you have limit_runtime_configs on, changing anything will require a restart.
otherwise old armor scale, should update on next re-equp or damage taken.
The only thing i can think of that might be happening is you used world specific settings and forgot?
try running both of these on the server and restarting it.
/forgery config setWorld *.old_armor_scale false
/forgery config set *.old_armor_scale false

Ah i didnt do setworld

setWorld is a bit of a neiche feature if you've never used it you can just ignore it, i just thought it coulda been the cause.
just to clear things up are you testing this on a server or in a single player and are you using any other mods? (if you're testing on a server please try in singleplayer and if you're using other mods please try without them)

single player and yes I have other mods. Are there any known to cause issues like this?

Nope nothing known, I'm very confused as to what's happening so i'm really hoping that either something obvious was missed, or that it's somehow a mod conflict. because otherwise the only thing i can think of atm is that you upload the logs after running forgery standalone and i try to replicate the install, maybe try a new world? maybe datapack conflict?? but i'm really running out of ideas

It happened in a new world:

ah good news, it was indeed another mod. I had armor curve installed, which does a similar thing so thats why I got it confused with yours. Thank you though!


docs: add sequelize v7 examples

Change Description
Adds examples of using cloud-sql-nodejs-connector library with sequelize 7 for pg, mysql2, and tedious drivers.
Checklist

[x] Make sure to open an issue as a
bug/issue
before writing your code! That way we can discuss the change, evaluate
designs, and agree on the general idea.
[x] Ensure the tests and linter pass
[x] Appropriate documentation is updated (if necessary)

Relevant issues:

Fixes #110

Shall we add some tests for these examples? I worry about keeping these working in the long term otherwise.

@edosrecki after syncing with @enocom I believe tests are something we would want to add for the long term benefit of maintaining these samples. That said, we can work on it after these initial examples are landed so it should not be a blocker to landing your PR 😊

Hi @edosrecki
I have tried to combine Sequelize (Postgres) and the Nodejs Connector using the examples in this PR but it simply doesn't work.
I have also looked at the tests and they use pg and not Sequelize. Can anyone please advise whether Sequelize and Postgres can at this point be used with the Nodejs Connector or not?

@bot-nftmagicians
Sequelize does work with Google Cloud Node.js Connector.
I created a whole repository where i show how to use Connector with many libraries (including sequelize). You can check it out here: https://github.com/edosrecki/google-cloud-sql-nodejs-connector-example
When you say "it simply doesn't work", what doesn't work exactly?

@edosrecki thank you for the quick reply sir!
Apologies for not including the error. This is the one I am getting:
Could not load the default credentials. Browse to https://cloud.google.com/docs/authentication/getting-started for more information.
It is great to hear is should work and I am just doing something wrong 😅
Any help appreciated.

Could not load the default credentials. Browse to https://cloud.google.com/docs/authentication/getting-started for more information.
Any help greatly appreciated 🙏

@bot-nftmagicians How are you trying to run the code? Locally or as part of a Google Cloud Application? (Cloud Run, App Engine etc.) If it is locally then you need to set application default credentials properly.
Please read this section of our README for details: https://github.com/GoogleCloudPlatform/cloud-sql-nodejs-connector#credentials

Thanks @jackwotherspoon
I am running everything locally for now. I have the code below running and connecting so I think my default credentials are OK:
require('dotenv').config()
process.env.GOOGLE_APPLICATION_CREDENTIALS = './xxxxxxxxxxxxxxxxx.json'

const main = async () => {
    const { Pool } = (await import('pg')).default
    const { Connector } = await import('@google-cloud/cloud-sql-connector')
    const { GoogleAuth } = await import('google-auth-library')

    const connector = new Connector()
    const clientOpts = await connector.getOptions({
        auth: new GoogleAuth(),
        instanceConnectionName: 'aaaaaaaaaaaaaa:us-east1:bbbbbbbbbbbbb',
        ipType: 'PUBLIC'
    })
    const pool = new Pool({
        ...clientOpts,
        user: process.env.DB_USERNAME_GCP,
        password: process.env.DB_PASSWORD_GCP,
        database: process.env.DB_NAME_GCP,
        max: 5
    })
    const { rows } = await pool.query('SELECT NOW()')
    console.log( rows )

    await pool.end()
    connector.close()
}

main()

That's with pg. However the attempts with Sequelize keep failing however I try to tweak the code.

@bot-nftmagicians
it seems to me that user name is wrong.
Instead of<EMAIL_ADDRESS>it should be<EMAIL_ADDRESS>

@bot-nftmagicians If you're running into problems, let's open up an issue and we can discuss. I'd like to keep the comments here focused on the PR.

Thanks guys. I have now managed to make it work. In my case the problem was I was using<EMAIL_ADDRESS>which is not the version of Sequelize that works with the connector.
After comparing my code with @edosrecki 's examples line by line I found that he's using<EMAIL_ADDRESS>When I made that change it did the job. I can confirm @edosrecki 's examples work, I can now connect to my GCP database using both methods; password and IAM :)
For other people's sake, if they run into this problem, I just want to emphasize: at this point in time you need to connect with a very specific alpha version of Sequelize.

@bot-nftmagicians, correct, it was added in v7.0.0-alpha.27:
https://github.com/sequelize/sequelize/commit/abeacd5ea2e32f6e6fbf91b6a0c94d9139fafb67


Question about License

Hi, thanks for your great work.
When will the license be added to this repository?

Thank you for your reminder. We have added the LICENSE here.

Thank you very much. I appreciate that.


[docs] Document enabling ping with host firewall

Cilium Feature Proposal
Currently, the docs show how to allow a port, but once that is applied, ICMP packets are also dropped. This may not always be desirable, and locating how to enable ICMP packets is is quite the adventure.
Describe the feature you'd like

Update the docs to point out that applying the policy will drop all other traffic, including ICMP. While this may seem somewhat obvious, the example makes it appear as though we're only applying TCP/UDP rules (and kubernetes usually doesn't concern itself with ICMP).
Include an example of properly enabling ICMP packets to the host.

Good suggestion! We forgot to update the host firewall docs after we added support for ICMP policies. Could you send a pull request to update the documentation as you suggested?

locating how to enable ICMP packets is is quite the adventure.

Do you mean it was hard to find in the documentation? What could we do to improve that?

Could you send a pull request to update the documentation as you suggested?

@pchaigno, if you could point me in the direction on how to do so, I'd be more than happy to help.

Do you mean it was hard to find in the documentation? What could we do to improve that?

Yeah, it was quite difficult to find because you'd have to know what to search for. I initially used "ping" as part of my query, which gave no results. It was only through digging through the CRD spec that I was able to discover it was called ICMP (at which point I face-palmed). Then I had to guess how to implement it in a host firewall correctly, look up what "type 128" and "type 32" meant, and hope I didn't mess something up along the way.
I venture the spec on ICMP types hasn't nor will change very much, but it would be nice to put a table of the types you support and their human meaning; in case they diverge at some distant point in the future. Also, it saves everyone a bit of googling -- especially when using ipv6 and requiring ICMP6 router announcements, and other important messages to get through the firewall.

Could you send a pull request to update the documentation as you suggested?

@pchaigno, if you could point me in the direction on how to do so, I'd be more than happy to help.

Arf, sorry, I started testing an updated policy when I saw the stalebot message. I would appreciate a review on the updated policy though: https://github.com/cilium/cilium/pull/22999.

I venture the spec on ICMP types hasn't nor will change very much, but it would be nice to put a table of the types you support and their human meaning; in case they diverge at some distant point in the future.

ICMP and ICMPv6 types are assigned by the IANA so I don't think they will change any time soon :slightly_smiling_face:

Yeah, it was quite difficult to find because you'd have to know what to search for. I initially used "ping" as part of my query, which gave no results. It was only through digging through the CRD spec that I was able to discover it was called ICMP (at which point I face-palmed). Then I had to guess how to implement it in a host firewall correctly, look up what "type 128" and "type 8" meant, and hope I didn't mess something up along the way.

Thanks! Lots of good actionable comments here I think. Let's keep this issue to track the documentation changes. I've created a new issue for the longer-term feature change (3rd item).

[ ] Mention "ping" in the documentation to ease search.
[ ] Link to the IANA pages for ICMP and ICMPv6 from the ICMP policy docs.
[ ] Support names for the ICMP and ICMPv6 types: https://github.com/cilium/cilium/issues/23000.

Could you send a pull request to update the documentation as you suggested?

@pchaigno, if you could point me in the direction on how to do so, I'd be more than happy to help.

Arf, sorry, I started testing an updated policy when I saw the stalebot message. I would appreciate a review on the updated policy though: #22999.

I venture the spec on ICMP types hasn't nor will change very much, but it would be nice to put a table of the types you support and their human meaning; in case they diverge at some distant point in the future.

ICMP and ICMPv6 types are assigned by the IANA so I don't think they will change any time soon slightly_smiling_face

Yeah, it was quite difficult to find because you'd have to know what to search for. I initially used "ping" as part of my query, which gave no results. It was only through digging through the CRD spec that I was able to discover it was called ICMP (at which point I face-palmed). Then I had to guess how to implement it in a host firewall correctly, look up what "type 128" and "type 8" meant, and hope I didn't mess something up along the way.

Thanks! Lots of good actionable comments here I think. Let's keep this issue to track the documentation changes. I've created a new issue for the longer-term feature change (3rd item).

[ ]  Mention "ping" in the documentation to ease search.
[ ]  Link to the IANA pages for ICMP and ICMPv6 from the ICMP policy docs.
[ ]  Support names for the ICMP and ICMPv6 types: Support names for the ICMP and ICMPv6 types #23000.

I think this Issue is completed


Can't setup the msgFlo

Hi,
I'm trying to setup msgFlo
I'm just following this https://msgflo.org/docs/usage/index.html.
I can setup the participants.
repeater(Repeat) OUT -> IN repeaterdup(Repeat)
In this graph, If I send the message repeater participant receives the message. But repeaterdup is not receiving the message.
msgflo-send-message --queue repeater.IN --json '{ "foo": "bar" }'  -- this command sends message to repeater participant.
The second participant is not getting message at all.
please help me to solve this.
Thanks in advance :)

What broker are you using? What operating system?
What is the output of the second participant if running with envvar DEBUG=msgflo*
?

I'm using ubuntu 18.04. Trying to setup the same demo from here https://msgflo.org/docs/usage/index.html.
while running with debug I'm getting like this
  msgflo:mqtt published fbp undefined undefined +0ms   msgflo:participant registered null +1ms repeater connected to mqtt://localhost   msgflo:mqtt message 1 true +9s   msgflo:participant got msg from queue repeater.IN +9s +++++ { foo: 'bar' }   msgflo:mqtt publishing repeater.OUT {"foo":"bar"} +3ms   msgflo:mqtt published repeater.OUT undefined undefined +0ms   msgflo:participant register +18s   msgflo:mqtt publishing fbp {"protocol":"discovery","command":"participant","payload":{"component":"Repeat","icon":"file-word-o","label":"Repeats in data without changes","inports":[{"id":"in","type":"any","hidden":false,"queue":"repeater.IN"}],"outports":[{"id":"out","type":"any","hidden":false,"queue":"repeater.OUT"}],"role":"repeater","id":"repeater-c3bd010c-f31e-4980-9306-0b8a216cef36"}} +18s   msgflo:mqtt published fbp undefined undefined +1ms   msgflo:participant registered null +1ms   msgflo:participant register +27s

msgflo-setup needs to be run with forever.


Frontend Development for "Contact Us" Page – Pathsphere Scholarship Portal Contribution

I would like to contribute to the Pathsphere Scholarship Portal project by designing and developing the frontend for the "Contact Us" page. This page will enable students, parents, or educational institutions to easily reach out with inquiries related to scholarships, application processes, or general support.
Key Features:

User-Friendly Form: Fields such as Name, Email, Subject, and Message, with intuitive design and clear instructions for users.
Responsive Design: Ensuring the page is fully responsive and works smoothly on all devices, from desktops to smartphones, using React.js, Tailwind CSS, and Material-UI.
Form Validation: Implementing form validation for better user experience and error handling to ensure proper data submission.
Clean and Professional UI: A modern design with Material-UI components to align with the educational and professional tone of the scholarship portal.
I am enthusiastic about contributing to this meaningful project and request that this task be assigned to me under Hacktoberfest and GSSoC.

Looking forward to working on this!

already exists


JSDoc fixes

This pull request contains various fixes to JSDoc comments to not only improve the API docs of function types, but also to make the ol3 source more amenable to be able to generate a TypeScript definition from (which I am currently working on through my JSDoc plugin: https://github.com/jumpinjackie/jsdoc-typescript-plugin)

Use @callback with @param and @return (where appropriate) to document function types instead of @typedef
olx.control.OverviewMapOptions#layers should have ol.Collection<ol.layer.Layer> instead of ol.Collection
Remove an unused goog.require from src/ol/tilequeue.js as a result of the wholesale change to @callback

The change to @callback results in documentation that currently looks like this (v3.13.1)

To this

Closing this. What I want to do (using @callback) is incompatible with the closure compiler

That's a pretty big improvement in the docs.  The major drawback of @typedef is that we cannot document the individual properties.  It would be nice to find a way to achieve this while still providing annotations that the compiler can make sense of.

olx.control.OverviewMapOptions#layers should have ol.Collection<ol.layer.Layer> instead of ol.Collection

This one can be fixed, see #4839


read_ny_name problem

Hi
I hope to get some help with this problem.
I have a working connection to PLC and can read info.
When i do this i get correct answer:
pprint.pprint(device.read_by_name('METERS._EM01_I_MAX_PV', REAL)) ##This works
when i do this i get fault below:
pprint.pprint(device.read_by_name(tag_names[4], tag_types[4]))
if i replace tag_types[4] with REAL i get no error...
[2020-07-24 16:29:38,285][3988:MainThread][adsclient.py/exit][WARNING] AdsClient exiting with exception: <type 'exceptions.AssertionError'>, . <traceback object at 0x76640918>
Traceback (most recent call last):
File "twincat_plc_info_m4.py", line 67, in
main()
File "twincat_plc_info_m4.py", line 54, in main
pprint.pprint(device.read_by_name(tag_names[0], tag_types[0]))
File "/usr/local/lib/python2.7/dist-packages/counsyl_pyads-1.0.0-py2.7.egg/counsyl_pyads/adsclient.py", line 273, in read_by_name
assert(isinstance(ads_data_type, AdsDatatype))
AssertionError

Hi @twoway, it looks like the value of your variable tag_types[4] is not equal to the value of your variable REAL. The error message suggests that tag_types[4] has a different data type than expected.
A good next step for debugging this might be to find out the exact content of tag_types[4], maybe using a debugger or using print:
print type(tag_types[4])
print tag_types[4]


Anonymous user remains anonymous after upgrade using Apple Login

Step 2: Describe your environment

Objective C or Swift: Swift
iOS version: 13.3
Firebase SDK version: 6.6.2
FirebaseUI version: 8.4.1
CocoaPods Version: 1.8.4

Step 3: Describe the problem:
When migrating an anonymous user to a non-anonymous one by choosing the Login with Apple ID, the Auth.auth().currentUser.isAnonymous isn't updated and the attribute remains false.
Steps to reproduce:

(Programatically) log in with an anonymous user
Open the Firebase UI login screen with authUI.shouldAutoUpgradeAnonymousUsers = true
Log in with an Apple ID
Listen for the callback in func authUI(_ authUI: FUIAuth, didSignInWith authDataResult: AuthDataResult?, error: Error?)

Observed Results:

Auth.auth().currentUser.isAnonymous is still true
authDataResult?.user.isAnonymous is false

Expected Results:

Auth.auth().currentUser.isAnonymous has the same value as the user provided in the callback parameter, i.e. it's false

Further notes

When using other login methods, the two values are correctly synced.
There used to be a similar issue (#651) for the email login.
Not sure, whether this is a Firebase UI issue or a Firebase (Auth) issue in general.

I haven't tested this myself, but I'm planning to support anonymous users and apple sign-in, so this would be a high priority request for myself as well. I'll comment again if I see the same issue.

I've filed a bug internally as b/160627975.

@joeljfischer This bug is about Auth.auth().currentUser.isAnonymous not being updated. However, linking an anon user with Apple Sign-in still works fine (minus this bug).
As OP mentioned, you just need to use authDataResult?.user.isAnonymous instead of Auth.auth().currentUser.isAnonymous for now to detect if a user has been upgraded or not for now. This workaround should be temporary and will not need be needed once we fix the bug.

@yuchenshi Thank you for the clarification.


(maint) add patch package for CentOS install class

Minimal CentOS 7 installs do not include the patch binary which is required for nokogiri installation

👍


Add --stdio switch

This was discussed on the forum here.  I understand your focus is on VS code so I'm not taking for granted that this will be accepted -- obviously I think the patch should be applied though :-)
Not sure if it helps, but I don't expect to be coming back with any requests to change behaviour beyond this in order to support emacs.  Everything already seems workable as-is, the emacs lsp-mode.el people target feature parity with vscode, and also provide general-purpose message processing hooks to allow ad-hoc server-dependent treatment.
Should I add an explicit comment next to the command line switch code along the lines below?
// The fact that rescript-vscode uses LSP under the hood is an implementation
// detail we don’t really guarantee.  This --stdio switch is therefore not
// guaranteed to remain useable.

Thanks. I'll cut a release before merging this. Then I'll ping willing folks to test master

Alright, I've cleaned up the code and pushed to your branch. I've briefly tested on Sublime Text and it seems this is good to go now. Feel free to test master.
Thanks!


Revert "[Refactor] refactor parquet file metadata memory management"

Reverts StarRocks/starrocks#41027
to fix https://github.com/StarRocks/StarRocksTest/issues/6204

@mergify rebase

@mergify rebase


fzf has no version option

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[ ] fish

--no-clear wad added 24 days ago but I couldn't verify fzf local version
fzf -v
unknown option: -v

fzf -h 2>&1 | grep ver
    --tac                 Reverse the order of the input
    --reverse             Reverse orientation
    FZF_DEFAULT_OPTS      Default options (e.g. '--reverse --inline-info')

Please mark builds with commit sha1 value at least or something more verbose.

We have fzf --version which is used in install script.

Please mark builds with commit sha1 value at least or something more verbose.

Thanks for the suggestion. I'll think about this. We'll have to be careful though since fzf has been around for more than 3 years and changing the format might break some user scripts.

fzf --help 2>&1 | grep version
    --tac                 Reverse the order of the input
    --reverse             Reverse orientation
    FZF_DEFAULT_OPTS      Default options (e.g. '--reverse --inline-info')

There is no version key, but it works

fzf --version
0.16.6
--version is option for advanced users? It is very distracting for me :disappointed:

--version is an option for advanced users?

No, it's not. I just haven't felt that I needed to list it on --help output as it seemed too obvious to me. bash --version, zsh --version, grep --version, awk --version, ruby --version, python --version, perl --version, the list goes on. Not saying that I won't add it to --help in the next version.

I work under macos and fzf installed as vim plugin.

Me too.

I suspect that there are more that one man fzf

You can easily fix that. Do you use Homebrew?

If you feel that it is ok then it is ok. I just report. Thank you.
Actually I use huge amount of OSes but my workplace is under mac os. That is why I near always read builtin help. It is a point of my distraction. There are too many dialects of same tools.

Yeah, I see, thanks. By the way, I don't know how you installed fzf, but if you installed fzf using Homebrew, you can just add set rtp+=/usr/local/opt/fzf to your vimrc, instead of installing it again using a plugin manager.


Xeditable adds empty class/tag and uncheckes  selected checkbox in the checklist, when values are changed the child model.

xeditable adds empty class/tag and uncheckes  selected checkbox in the checklist, when values are changed the child model.
I have created fiddle and added steps to recreate the issue.
http://jsfiddle.net/etsigankov/0twrL66q/
Steps:

click add shape it will create the model
click "no appliances" and add 1 or two items(it adds items to the model and displays them)
once you change the measurement W or H( it records changes in the model correctly) BUT

Issue: if you go to "add appliance" ( it will uncheck everything that was checked before, thus creating multiple entries).
P.S. Everything works with Select/multi select

Hi, I took a look and found a problem in the app.  I will have to make a pull request to fix it.  Once my pull request is in, you will need to add the following to your editable-checklist :
e-checklist-comparator="compareFn"
Where comparFn is:
  $scope.compareFn = function(obj1, obj2) {
    return obj1.v === obj2.v;
  };

@ckosloski . Thank you for fixing the issue.  Should i wait till your changes will be pushed to the new release of the library to see the fix on my end?

Yes, you won't see the changes until a new build is released, or you pull down the code and build it yourself.