Update code examples in SDKs

Following these issues #99 and #102, README.md should be updated like this
1. Remove $ in front of command lines
closes: #99
example: meilisearch/meilisearch-js/pull/827
As stated in de markdownlint recommendetation, $ should not be displayed if no output of the commands are showed.
Example:
Wrong ❌: no output, no $ signs
$ ls
$ cat foo
$ less bar

Correct: ✅
ls
cat foo
less bar

Correct: ✅. As there are output, there should be a $ signed to
$ ls
foo bar
$ cat foo
Hello world
$ cat bar
baz

These dollar signs should be removed from the following files:

README.md
CONTRIBUTING.md
.code-samples.meilisearch.yaml
any other markdown file in a sub directory

Change book examples with movies examples
closes: #102
Example: meilisearch/meilisearch-js/pull/828
To keep the getting started simple to try, but to also make documentation examples compatible with the getting started, we are replacing book examples with movies examples.
The Movie dataset is used throughout the documentation and is used in the javascript code-samples. The only place where book is used is in the tests (which can stay as it is) and in our getting started.
Changes
Index name should be changed in README.md
  const index = client.index('movies')

Provided dataset should be changed with the following:
 const documents = [
      { id: 1, title: 'Carol', genres: ['Romance', 'Drama'] },
      { id: 2, title: 'Wonder Woman', genres: ['Action', 'Adventure'] },
      { id: 3, title: 'Life of Pi', genres: ['Adventure', 'Drama'] },
      { id: 4, title: 'Mad Max: Fury Road', genres: ['Adventure', 'Science Fiction'] },
      { id: 5, title: 'Moana', genres: ['Fantasy', 'Action']},
      { id: 6, title: 'Philadelphia', genres: ['Drama'] },
  ]

comment should be changed
// If the index 'movies' does not exist, MeiliSearch creates it when you first add the documents.

All displayed examples should be updated accordingly
TODO
Books to movies Examples

[ ]  meilisearch-dotnet
[ ]  meilisearch-go
[ ]  meilisearch-java
[x]  meilisearch-js
[ ]  meilisearch-php
[ ]  meilisearch-python
[ ]  meilisearch-ruby
[ ]  meilisearch-rust
[ ]  meilisearch-swift
[ ] Laravel Scout
[ ] .Net
[ ] Dart

Remove $

[ ]  meilisearch-dotnet
[ ]  meilisearch-go
[ ]  meilisearch-java
[x]  meilisearch-js
[ ]  meilisearch-php
[x]  meilisearch-python
[ ]  meilisearch-ruby
[x]  meilisearch-rust
[ ]  meilisearch-swift
[ ] Laravel Scout
[ ] .Net
[ ] Dart
[ ] Symfony
[ ] Angular
[ ] React
[ ] Vue
[ ] VuePress plugin
[ ] instant-meilisearch
[ ] docs-searchbar.js
[ ] meilisearch-digitalocean
[ ] meilisearch-aws
[x] meilisearch-kubernetes
[ ] Cloud Scripts
[ ] docs-scraper
[x] meilisearch-wordpress

For the $ issue, I will use an automated tool 🙂 I know it might not work everywhere like in the Rust SDK repo, so thanks a lot @sanders41 for this PR (https://github.com/meilisearch/meilisearch-rust/pull/117)


Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-zSOHx1/uvloop/

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-zSOHx1/uvloop/

可能是你用了Python2吧？用Python3运行试试
或者提供更详细的错误信息，这样子太模糊了

pip3 install -r requirements.txt


Corrected style and formatting to meet HQRM guidelines - Fixes #281

Pull Request (PR) description
Corrected style and formatting to meet HQRM guidelines for xNetAdapterbinding.
This Pull Request (PR) fixes the following issues:

Fixes #281

Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[x] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[x] Examples appropriately updated?
[x] New/changed code adheres to Style Guidelines?
[x] Unit and (optional) Integration tests created/updated where possible?

This change is

Codecov Report

Merging #282 into dev will decrease coverage by <1%.
The diff coverage is 100%.

@@         Coverage Diff         @@
##            dev   #282   +/-   ##
===================================
- Coverage    97%    97%   -1%
===================================
  Files        21     21
  Lines      1682   1678    -4
===================================
- Hits       1646   1642    -4
  Misses       36     36

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Tests/Unit/MSFT_xNetAdapterBinding.Tests.ps1, line 59 at r1 (raw file):

        }

        $MockBindingMixed = @(

Lower-case here as well?

Tests/Unit/MSFT_xNetAdapterBinding.Tests.ps1, line 77 at r1 (raw file):

$Result

Lower-case here as well. Throughout.

Comments from Reviewable

All changes done! Thanks for reviewing @Johlju

Review status: 6 of 7 files reviewed at latest revision, 2 unresolved discussions.

Tests/Unit/MSFT_xNetAdapterBinding.Tests.ps1, line 59 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
Lower-case here as well?

Done.

Tests/Unit/MSFT_xNetAdapterBinding.Tests.ps1, line 77 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…

$Result

Lower-case here as well. Throughout.

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 6 of 7 files at r1, 1 of 1 files at r2.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Multiple Errors On WebglBuild

Hello, im just getting started with agora and i'm trying to implement the Webgl SDK for Unity, i previously install agora from the package manager and then install this SDK and i have this errors:

i dont know if i miss a step on the installation of the sdk or i have to configure something else of the project, i downloaded the last version of the sdk:

I hope you can help me with my problem, thank you very much in advance!

If you are on Unity 2022.2 you will get these errors.  We just have a change in dev/refactor4.  Try that branch out to see if that works for you.

Hello, thanks for the reply, im on Unity 2021.3.4f1, i downloaded the branch that you told me and i could make a build, there is going to be a new version of the SDK whit those changes so i can build my actual proyect?

https://github.com/AgoraIO-Community/Agora_Unity_WebGL/issues/83
Same bug. Worked for me


When sqlite3 file picker clicked, vscode will be closed

Extension Version
v1.3.1
OS and Version
Arch 6.7.4-arch1-1
VS Code Version
v1.86.1
Database name and Version
Sqlite3
Steps to reproduce
When clicked, vscode will be closed
Expected behavior
.
Screenshots

Additional context
No response

This is weird because I believe the extension runs in a separate Electron process.
Can you provide a short clear screen recording of this please?
This will help me triage this better.

I updated to v1.3.2 Problem solved.

Thanks for the update.
Will close this now :+1:


pull in the updated bank-vaults client with proper logging

Is your feature request related to a problem? Please describe.

Pull this version of the client: https://github.com/banzaicloud/bank-vaults/pull/1087
Describe the solution you'd like to see

Describe alternatives you've considered

Additional context

Fixed by https://github.com/banzaicloud/pipeline/pull/3148


Initially refactor frontend

Purpose of this inital refactoring is clean up the project on a general level before refactoring components.

Remove unnecessary frontend dependencies
Remove tslint
Add and run eslint
Remove vue-decorators

As this PR is quite large and does multiple things, i'll split this into smaller PRs


invalid magic

I'm running gearmand on windows.
When using example/py/worker.py, it works fine. But using example/worker/worker, it seems that gearman-go send invalid magic to server. Below is a log output of gearmand.
2015/03/05 19:19:42 util.go:218: [debug] magic not match 0x32353533
2015/03/05 19:19:42 session.go:59: [debug] invalid magic sessionId 2

Since py/worker.py is working, I'm thinking this is problem of gearman-go.

Cause I don't have a windows environment to run a test-case, could you please show me your environment's details? And also it would be great if you can run test-cases in your env and send the result to me.
By the way, how about util.go & session.go, could you show me codes in those files? Because, no other people tell me the same problem occurred before, I've no idea about what reasons could trigger this issue.

This problem occured from 2014/1 on my environment (Windows7 32bit).
https://github.com/ngaut/gearmand/issues/7
I don't have any test code. I run client/worker.py in your repository
https://github.com/mikespook/gearman-go/tree/master/example/client
https://github.com/mikespook/gearman-go/tree/master/example/py

Does the string "2553" have any special meaning in your application?
I read the source code of gearmand. It may a byte alignment problem. Have you tried the gearmand(C).

Unfortunately, I don't have 32bit linux OS, and gearmand(C) doesn't work on windows.

Does the string "2553" have any special meaning in your application?

Sorry, I don't write any code. I just run example.

I'm guessing this will occur on 32bit OS.

I built a 32bit testing environment in VBox:
Linux i386 3.2.0-4-486 #1 Debian 3.2.65-1+deb7u2 i686 GNU/Linux
However, both python client and golang client are worked fine.


Possibility to disable the leader election

What this PR does / why we need it:
The --leader-elect flag is replaced by --no-leader-election. The leader election is enabled by default and can be disabled by using such a flag.
Which issue this PR fixes
Fixes #55
Special notes for your reviewer:
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect significant changes

Quick note: this technically is a breaking change: it changes the end user interface

Quick note: this technically is a breaking change: it changes the end user interface

Since we are in pre-alpha I don't see it as a problem at all. Do you?

Since we are in pre-alpha I don't see it as a problem at all. Do you?

Right - :smart: - forgot about it. 👍

Seem like this PR broke the main.
$ k logs gateway-operator-controller-manager-7d45bc54fd-fkf8l -n kong-system
flag provided but not defined: -leader-elect
Usage of /manager:
  -controller-name string
    	a controller name to use if other than the default, only needed for multi-tenancy
  -health-probe-bind-address string
    	The address the probe endpoint binds to. (default ":8081")
  -kubeconfig string
    	Paths to a kubeconfig. Only required if out-of-cluster.
  -metrics-bind-address string
    	The address the metric endpoint binds to. (default ":8080")
  -no-leader-election
    	Disable leader election for controller manager. Disabling this will not ensure there is only one active controller manager.
  -zap-devel
    	Development Mode defaults(encoder=consoleEncoder,logLevel=Debug,stackTraceLevel=Warn). Production Mode defaults(encoder=jsonEncoder,logLevel=Info,stackTraceLevel=Error)
  -zap-encoder value
    	Zap log encoding (one of 'json' or 'console')
  -zap-log-level value
    	Zap Level to configure the verbosity of logging. Can be one of 'debug', 'info', 'error', or any integer value > 0 which corresponds to custom debug levels of increasing verbosity
  -zap-stacktrace-level value
    	Zap Level at and above which stacktraces are captured (one of 'info', 'error', 'panic').
  -zap-time-encoding value
    	Zap time encoding (one of 'epoch', 'millis', 'nano', 'iso8601', 'rfc3339' or 'rfc3339nano'). Defaults to 'epoch'.


Greenkeeper/@types/jest 24.0.4

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Icestrom installation issue "Eigen3"

with : cmake -DARCH=ice40 -DCMAKE_INSTALL_PREFIX=/usr/local .
CMake Error at CMakeLists.txt:189 (find_package):
Could not find a package configuration file provided by "Eigen3" with any
of the following names:
Eigen3Config.cmake
eigen3-config.cmake

Add the installation prefix of "Eigen3" to CMAKE_PREFIX_PATH or set
"Eigen3_DIR" to a directory containing one of the above files.  If "Eigen3"
provides a separate development package or SDK, be sure it has been
installed.

My solution:
https://answers.ros.org/question/253181/howto-eigen-installation/

You should be able to do apt-get install libeigen3-dev and then it will just work.  This is what we do to build the Fomu toolchain for Raspberry Pi.


Problem viewing 11-plot-routes-folium-web-map

When I look at the folium example notebook, none of the IFrame outputs displays anything.  Am I missing something?  Is there a problem with the html files?

GitHub itself doesn't render IFrames, so you won't see anything online. But the example works fine on my computer. If it doesn't on yours when you run it locally, please open an issue directly in the OSMnx repo: https://github.com/gboeing/osmnx/issues

Oh, I didn't realize it wasn't expected to show here.  My IPython installation is messed up at the moment, so I was only looking online (GitHub and Jupyter nbviewer).  Thanks!


solanum: remove obsolete BANDB settings/patches

Motivation for this change
well for whatever reason, the BANDB stuff doesn't have any effect anymore. so I guess we can just remove it
Things done

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

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

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 124263 at 8eb5701a run on x86_64-linux 1

1 package built successfully:

solanum

1 suggestion:

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/servers/irc/solanum/default.nix:25:5:
   |
25 |     ./dont-create-logdir.patch
   |     ^

@4z3 @mweinelt

Can we test this holds true? Like oper up, resv a nick, check if it flushes the reservation to the right place on disk.

I removed the ban.db and restarted solanum and the file reappeared in /var/lib/solanum

Fair enough, whats the matter with the aarch64 build though?


[BUG] Disclosure carets don't seem to align nicely with bullets

Was asked to create a new issue for the alignment of the disclosure carets - I've linked the same images below to demonstrate the issue.
Screenshots

Desktop (please complete the following information):

OS: macOS 11.2.3
Obsidian Version: 0.12.0

Originally posted by @macintacos in https://github.com/vslinko/obsidian-outliner/issues/93

@vslinko
and this problem here, maybe it's the same problem ;)

I have the same problem, i record a gif bellow

@macintacos Thank you for your feedback! Should be fixed after 2.0 release. Could you please check that everything is ok?

Closed due to inactivity. If you are still experiencing the problem, please create a new issue.


Process or phenotype? 'synaptic multiplicity (development)'

I have come across the term 'synaptic multiplicity', while annotating PMID:24487234, which shows that mice lacking Cx3cr1 contain significantly fewer multisynapse boutons (MSBs) than wild-type mice, which manifests itself as a deficit in 'synaptic multiplicity' in adulthood.
But should synaptic multiplicity formation be considered a developmental process, or is synaptic multiplicity (or here: its deficit) a phenotype?
A new BP term 'synaptic multiplicity development' could potentially by defined as: 'A postnatal developmental process, modulated by microglia-mediated synaptic pruning essential for proper neural circuit maturation, during which two neurons initially connect through a single synapse and add more connections over time to form multiple spine synapses'.
Could someone please share their thoughts on this? @Pimmelorus, perhaps you'd be able to comment on this?
Thanks,
Barbara
cc:
@RLovering
@paolaroncaglia
@rachhuntley

@BarbaraCzub
To be honest, I have never heard of MSBs or synapse multiplicity before. As I read the paper, synaptic multiplicity (SM) represents the extent to which a single presynaptic terminal forms synapses with different dendritic spines. High SM is observed at an immature developmental stage of the neuronal network:

These findings show that a transient reduction in microglia results in a long-term deficit in synaptic multiplicity despite normal excitatory synapse number and suggest that there is a critical window during which microglia-mediated synaptic pruning is essential for proper circuit maturation.

I would not consider 'SM development' a BP term since SM is a proxy for the development of the neuronal circuitry driven by synapse pruning.

Hi @Pimmelorus, thank you for your feedback. Could you also please comment on 'functional connectivity'? I suppose that similarly to your conclusion regarding 'synaptic multiplicity', you would not consider 'functional connectivity' a process either, because it is also a reflection of more specific underlying processes?
Thanks,
Barbara

I suppose that similarly to your conclusion regarding 'synaptic multiplicity', you would not consider 'functional connectivity' a process either

No, indeed. It is a trait with different phenotypes, not a process.

Thank you!


Should Be Visible passes when element is covered by another window.

When one window is covering the element on another window, and I try to click on that element, it will give me FlaUiError: Element position could not be found because it is hidden.  Even though FlaUILibrary.Element Should Be Visible returns True/Pass
Click on GUI element
    [Documentation]
    ...     Keyword to mouse click on a GUI element with a xPath
    ...
    ...     Args:
    ...     - element: Dictionary obj with a key-value pair of 'xPath': 'path/to/element'
    [Arguments]  ${element}
    Log Many  ${element}
    GUI Element Should Exist And Be Visible  ${element}
    FlaUILibrary.Click  ${element}[xPath]

GUI Element Should Exist And Be Visible
    [Arguments]     ${element}
    Log Many  ${element}
    FlaUILibrary.Element Should Exist  ${element}[xPath]
    FlaUILibrary.Element Should Be Visible  ${element}[xPath]

Modified Post by @Mavnus04 in https://github.com/GDATASoftwareAG/robotframework-flaui/issues/92#issuecomment-1472642351_

Use:
FlaUILibrary.Element Should Exist  ${element}[xPath]
FlaUILibrary.Element Should Be Visible  ${element}[xPath]
FlaUILibrary.Focus  ${element}[xPath]

focus keyword brings it to front so it can be clickable and not hidden under other window.

Then what is the point of checking for Visible then?  If the element is hidden under another window, would it be considered not visible?

I thought after checking is visible maybe something poping in front.
If it is not the case then i cannot tell what is happennig unless you provide more info or log.

Nothing is popping in front.  It just happens to be that exists and visible both return true, even if another window is covering that element.
Yes, using Focus brings the element to the top/front layer.  But I feel that visible should return false if is not clickable.
Or if just going to use Focus, why check for visible at all?

We give an access to the IsOffscreen property from https://github.com/FlaUI/FlaUI/blob/master/src/FlaUI.Core/AutomationElements/AutomationElement.cs#L145
and they use in turn the Windows isofscreen property https://learn.microsoft.com/en-us/dotnet/api/system.windows.automation.automationelement.automationelementinformation.isoffscreen
As you can see in the middle of windows automation link:
"The value of the property is not affected by occlusion by other windows, or by whether the element is visible on a specific monitor."

Another suggestion, rename "Visible" to "Offscreen" to avoid confusion.  Or update the documentation to explain the actual behavior this functionality is performing.

We will change in future this keyword by naming and release this as 2.1.0 version.
Which include these changes to adjust by developers to increase understanding about keywords.

Latest changes are:
Added

#96 New Keywords

Is Element Offscreen
Wait Until Element Is Offscreen

Changed

#96 Multiple Keywords renamed because of misleading keyword understanding

'Element Should Be Visible' to 'Is Element Offscreen'
'Wait Until Element Is Visible' to 'Wait Until Element Exist'
'Wait Until Element Is Hidden' to 'Wait Until Element Does Not Exist'


Inconsistent URL Slash Display on Documentation Link

Which Caliper version are you using?
v0.5.0
Which Node.JS version are you using?
v18.19.1
Which operating system are you using?
Ubuntu 22.04.3 LTS(through WSL)
Please provide some context for your error. For example, when did the error occur? What were you trying to achieve, and how?

While browsing the Hyperledger Caliper documentation (version 0.5.0), I noticed a discrepancy in the URL path shown in the browser's status bar preview when hovering over the 'Documentation' link.
Objective: I was browsing the documentation.
What was the observed incorrect behavior?
The status bar URL preview displays an extra slash (i.e., //) in the URL path, which is not consistent with the actual URL path in the address bar, which has a single slash (/). This was observed on the 'Getting Started' page.
Please provide the error logs and their surroundings.
N/A(Typo in the HTML code that generates the documentation pages.)

Please provide your benchmark configuration file content, if possible.
No response
Please provide your network configuration file content, if possible.
No response
Please provide your workload module content, if possible.
No response
Please provide any additional information you deem relevant to the error.
It may seem a little trivial, but it should ideally be corrected for consistency and to adhere to URL standards.

I've submitted a pull request that addresses this issue: PR link
This PR implements the necessary fix to ensure correct URL formatting in the documentation link, as discussed in this issue. Merging this PR should resolve the issue.
Thank you for your time and consideration.

@PrathamSikka24  your PR #1532 fixed this and has been merged so will close.

@davidkel Yes, that sounds good. Looking forward to contributing more.


S3 - Bugfix - Remove possibilty for concurrent access to file content

Fixes #2789
Bug happened due to concurrent access to the SpooledTemporaryFile that holds the bucket content.
I chose not to add a test in this PR, as it's only reproducible with large files (>100 MB).
The test that I used to reproduce and verify this can still be found here: https://github.com/bblommers/moto/commit/410d9ee90186d5e83c81f97dc93b6a24faf62b39

Coverage increased (+0.001%) to 94.28% when pulling 410d9ee90186d5e83c81f97dc93b6a24faf62b39 on bblommers:bugfix/2789 into 4400238c971cb5f11dffb40feed3ece3a1e08a62 on spulec:master.

Coverage increased (+0.001%) to 94.28% when pulling 410d9ee90186d5e83c81f97dc93b6a24faf62b39 on bblommers:bugfix/2789 into 4400238c971cb5f11dffb40feed3ece3a1e08a62 on spulec:master.

Looks good; thanks!


Allow to define the response_format for OpenAI

Is your feature request related to a problem? Please describe.
Currently it's not possible to define the response_format even though the open ai ruby library supporting it https://github.com/alexrudall/ruby-openai/tree/main?tab=readme-ov-file#json-mode
Describe the solution you'd like
Allow the possibility to define the response format when initializing the Langchain::LLM::OpenAI

@pedroresende You can pass the response_format to the chat() method directly but not yet to the initializer.
llm = Langchain::LLM::OpenAI.new(api_key: ENV['OPENAI_API_KEY'])

llm.chat(messages: [{role:"user", content:"hello"}], response_format: {type:"json_object"})

You will get the following error:
"error": {
  "message": "'messages' must contain the word 'json' in some form, to use 'response_format' of type 'json_object'."
}

This works:
llm.chat(messages: [{role:"system", content:"json"}, {role:"user", content:"hello"}], response_format: {type:"json_object"})

#<Langchain::LLM::OpenAIResponse:0x000000012ae2dfa8
 @model=nil,
 @raw_response=
  {"id"=>"chatcmpl-A4fnhlAvHOXrZfyamZvKh49e32lKF",
   "object"=>"chat.completion",
   "created"=>1725677357,
   "model"=>"gpt-4o-mini-2024-07-18",
   "choices"=>
    [{"index"=>0,
      "message"=>{"role"=>"assistant", "content"=>"\n{\"response\": \"Hello! How can I assist you today?\"}", "refusal"=>nil},
  ...

@pedroresende With this small change you should be able to set it in the initializer:
llm = Langchain::LLM::OpenAI.new(api_key: ENV['OPENAI_API_KEY'], default_options: {response_format: {type: "json_object"}})

Please let me know if that works for you!

@pedroresende With this small change you should be able to set it in the initializer:
llm = Langchain::LLM::OpenAI.new(api_key: ENV['OPENAI_API_KEY'], default_options: {response_format: {type: "json_object"}})

Please let me know if that works for you!

Hi @andreibondarev,
thanks once again for the excelente work and support, however it seems the answer is still content instead of json.
With this change I'm getting the following response
[{\"role\":\"system\",\"content\":\"You are a chat bot that answers questions in a pragmatic, confident, inspiring, yet empathetic tone about Strategyzer that you have stored in your vectorsearch database. If you don't know the answer, just say i don't know.\"},{\"role\":\"user\",\"content\":\"consider a business that provides a platform for people to fina and connect in their field. The platform should be easy to use, and provide a seamless experience for both the user and the experts. The platform should also be customizable, allowing users to tailor the experience to their specific needs and preferentes. suggest me 10 ideas for this project\"},{\"role\":\"assistant\",\"content\":\"\",\"tool_calls\":[{\"id\":\"call_XPHoahLCOeH9638hf3yaLBzV\",\"type\":\"function\",\"function\":{\"name\":\"langchain_tool_vectorsearch__similarity_search\",\"arguments\":\"{\\\"query\\\":\\\"ideas for a customizable platform for connecting people in their field\\\",\\\"k\\\":10}\"}}]},{\"role\":\"tool\",\"content\":\"#\\u003c#\\u003cClass:0x000000012489c090\\u003e:0x0000000167d9f6a8\\u003e\",\"tool_call_id\":\"call_XPHoahLCOeH9638hf3yaLBzV\"},{\"role\":\"assistant\",\"content\":\"I couldn't find specific ideas for a customizable platform for connecting people in their field in the database. However, I can provide you with some suggestions based on common features of such platforms:\\n\\n1.  **Matching Algorithm**: Implement an algorithm that suggests relevant connections based on user profiles and preferences.\\n\\n3. **Customizable Filters**: Enable users to filter and search for connections based on specific criteria such as industry, location, or skills.\\n\\n4. **Messaging System**: Provide a secure messaging system for users to communicate and network with each other.\\n\\n5. **Virtual Events**: Host virtual events like webinars, workshops, or networking sessions to facilitate connections and knowledge sharing.\\n\\n6. **Resource Library**: Offer a repository of resources such as articles, guides, and tools to help users in their field.\\n\\n7. **Feedback and Ratings**: Allow users to provide feedback and ratings on interactions with others to maintain quality connections.\\n\\n8. **Groups and Communities**: Create specialized groups or communities within the platform for users with similar interests to connect and collaborate.\\n\\n9. **Integration with Social Media**: Enable users to link their social media profiles to the platform for seamless networking across platforms.\\n\\n10. **Analytics Dashboard**: Provide users with insights into their networking activities, connections made, and engagement metrics to track their progress.\\n\\nThese ideas can help you create a user-friendly and customizable platform for connecting people in their field.\"}]

It only work if I request after with return the previous answer in json format

@pedroresende With this small change you should be able to set it in the initializer:
llm = Langchain::LLM::OpenAI.new(api_key: ENV['OPENAI_API_KEY'], default_options: {response_format: {type: "json_object"}})

Please let me know if that works for you!

Hi @andreibondarev,
thanks once again for the excelente work and support, however it seems the answer is still content instead of json.
With this change I'm getting the following response
[{\"role\":\"system\",\"content\":\"You are a chat bot that answers questions in a pragmatic, confident, inspiring, yet empathetic tone about the information that you have stored in your vectorsearch database. If you don't know the answer, just say i don't know.\"},{\"role\":\"user\",\"content\":\"consider a business that provides a platform for people to fina and connect in their field. The platform should be easy to use, and provide a seamless experience for both the user and the experts. The platform should also be customizable, allowing users to tailor the experience to their specific needs and preferentes. suggest me 10 ideas for this project\"},{\"role\":\"assistant\",\"content\":\"\",\"tool_calls\":[{\"id\":\"call_XPHoahLCOeH9638hf3yaLBzV\",\"type\":\"function\",\"function\":{\"name\":\"langchain_tool_vectorsearch__similarity_search\",\"arguments\":\"{\\\"query\\\":\\\"ideas for a customizable platform for connecting people in their field\\\",\\\"k\\\":10}\"}}]},{\"role\":\"tool\",\"content\":\"#\\u003c#\\u003cClass:0x000000012489c090\\u003e:0x0000000167d9f6a8\\u003e\",\"tool_call_id\":\"call_XPHoahLCOeH9638hf3yaLBzV\"},{\"role\":\"assistant\",\"content\":\"I couldn't find specific ideas for a customizable platform for connecting people in their field in the database. However, I can provide you with some suggestions based on common features of such platforms:\\n\\n1.  **Matching Algorithm**: Implement an algorithm that suggests relevant connections based on user profiles and preferences.\\n\\n3. **Customizable Filters**: Enable users to filter and search for connections based on specific criteria such as industry, location, or skills.\\n\\n4. **Messaging System**: Provide a secure messaging system for users to communicate and network with each other.\\n\\n5. **Virtual Events**: Host virtual events like webinars, workshops, or networking sessions to facilitate connections and knowledge sharing.\\n\\n6. **Resource Library**: Offer a repository of resources such as articles, guides, and tools to help users in their field.\\n\\n7. **Feedback and Ratings**: Allow users to provide feedback and ratings on interactions with others to maintain quality connections.\\n\\n8. **Groups and Communities**: Create specialized groups or communities within the platform for users with similar interests to connect and collaborate.\\n\\n9. **Integration with Social Media**: Enable users to link their social media profiles to the platform for seamless networking across platforms.\\n\\n10. **Analytics Dashboard**: Provide users with insights into their networking activities, connections made, and engagement metrics to track their progress.\\n\\nThese ideas can help you create a user-friendly and customizable platform for connecting people in their field.\"}]

It only work if I request after with return the previous answer in json format

This should work now! If you're using OpenAI, however, one of the message contents needs to mention the string "json" somewhere.


Getting permission error when pulling from device

//String path = "/data/user/0/com.package/files/";
    String path = "/data/user/0/com.package/files/";
String file = "TEST2.txt";

JadbConnection jadb = new JadbConnection();
JadbDevice device = jadb.getDevices().get(0);

device.executeShell("run-as", "com.package");
try {
	device.pull(new RemoteFile(path), new File(file));
} catch (Exception e) {
                //Keep getting a Permission denied exception
	System.err.println("Error while pulling: " + e.getMessage());
}

I keep getting Permission denied error when trying to pull a file from the device.
Do you have any ideas on why this is happening and how to solve it?
Thank you

That's because you are not allowed to copy a directory. Try something like this instead.
device.pull(new RemoteFile("/data/user/0/com.package/files/TEST2.txt", new File("TEST2.txt"));

Hope this helps, let me know otherwise.

Did you have a chance to try this out?

I'll assume this got resolved, feel free to reopen otherwise @yaron-e


Get-PortalToken causes SSL validation issues for subsequent Invoke-RestMethod and Invoke-WebRequest

I have a PowerShell script that use Invoke-RestMethod to check the state of a service.  After invoking Get-PortalToken the next time I use Invoke-RestMethod with https endpoints I see errors:
Invoke-RestMethod : The underlying connection was closed: An unexpected error occurred on a send.

PSInvalidOperationException: There is no Runspace available to run scripts in this thread. You can provide one in the DefaultRunspace property of the System.Management.Automation.Runspaces.Runspace type. The script block you attempted to invoke was: $true

For example, this works on a fresh machine:
Invoke-RestMethod https://myportal.domain.com/portal/portaladmin/health
then I do something else that calls Get-PortalToken:
$password = ConvertTo-SecureString 'password' -AsPlainText -Force
$credential = New-Object System.Management.Automation.PSCredential ('admin', $password)
$Referer = 'https://localhost'
$token = Get-PortalToken -PortalHostName 'myportal.domain.com' -SiteName 'portal' -Port 443 -Credential $credential -Referer $Referer

after running the above scripts, the first call fails
Invoke-RestMethod https://myportal.domain.com/portal/portaladmin/health

@bweisberg We set the following when we do a get portal token
[System.Net.ServicePointManager]::ServerCertificateValidationCallback = {$true}
This also affects any call you make after making Get-PortalToken
As suggested in the blog post try setting the following before you make a rest call again.
[System.Net.ServicePointManager]::ServerCertificateValidationCallback = $null

@bweisberg
Since u are already using the ArcGIS PowerShell Module, you can also try
Invoke-ArcGISRequest commandlet and see if that works for you.
As @shailesh91 mentions, the above code will ignore certificate errors, mismatches

@shailesh91 setting the callback to $null does indeed work.  @nshampur Invoke-ArcGISRequest also works and I use it whenever I make ArcGIS REST calls.  It sounds like when I need to call other REST methods with Invoke-RestMethod I need to reset the SSL validation with [System.Net.ServicePointManager]::ServerCertificateValidationCallback = $null.
Can the ArcGISUtility functions reset the validation for us?  This solution allows temporarily ignoring certificate errors for individual calls


[FLINK-9361] [sql-client] Fix refresh interval in changelog mode

What is the purpose of the change
This PR removes a InterruptedIOException when changing the refresh interval of the SQL Client CLI in changelog mode.
Brief change log

Do not use interrupts but a wait/notify pattern.
Deselected row on refresh in changelog mode.

Verifying this change
Manually verified.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Thank you @fhueske.


Alerts - color contrast of icons

The icons used for Alerts should be double-checked for color contrast. A quick check shows that the danger color combination is not accessible. These icons should have contrast given that for users who are color blind, this is the means by which they would understand the type of alert.

@kybaker I'm not sure if this issue would be addressed with the color updates you're working on or not, but wanted to go ahead and capture it.

@kybaker I'm not sure if this issue would be addressed with the color updates you're working on or not, but wanted to go ahead and capture it.

@jgiardino do you mind checking this again since we've made the updates? https://pf4.patternfly.org/components/Alert/examples/

These are the results of the color contrast for icons in Alerts:
Info icon on background - pass
Success icon on background - pass
Warning icon on background - pass
Danger icon on background - fail 👎

Thanks! @kybaker or @mceledonia should we make an update here?

@kybaker I heard the alert design might be changing to align with the digital design team design, do we have any updates there?
If not then yes we should change the danger icon on background to achieve a pass result for contrast.

@mceledonia We should just change the icon color for now.

@jgiardino Are we able to subvert the a11y requirement by noting that the icon itself is more decorative and doesn't convey new information that the alert wouldn't have without it?

@mceledonia I was seeing the icon as the alternate means of communication for color blind users who can't perceive the colors we use to indicate alert type. We do have a screen reader alternative for the icon and color, but for users who are color blind, I would expect there to be some accessible identifier for the type of alert without needing to read the contents to discern if the alert is an error or some other type. I was thinking the icon would serve this purpose, given that there's sufficient color contrast.

Here is a solution for this one @jgiardino @kybaker @mcoker
We switch our danger-100 and -200 (which really means switching our red-100 and red-200) to the appropriate brand red colors.
Then we switch danger-300 to use red-500, and change red-500 to #120100 which gives us a perfect 4.5 on top of our new danger-100.

I was chatting with @mceledonia offline about this and want to capture it here for everyone. I'm good with whatever decisions you make for colors, assuming they meet color contrast.
I'm inclined to say that for Alerts (not inline alerts), we can use the 3:1 color ratio, based on this note included in the description for the WCAG 1.4.3 color contrast criteria:

When evaluating this success criterion, the font size in points should be obtained from the user agent or calculated on font metrics in the way that user agents do. Point sizes are based on the CSS pt size as defined in CSS3 Values. The ratio between sizes in points and CSS pixels is 1pt = 1.333px, therefore 14pt and 18pt are equivalent to approximately 18.5px and 24px.

I'm assuming that the two pixel sizes stated above can be applied to our icons. This means that 18.5px plus bold or 24px not bold can use 3:1. It's debatable whether the icons qualify as "bold" or not, and I'd defer to the visual designers on that. But if we were to be conservative and say that the icons would not qualify as bold, then this would mean that the icons in Alerts that are 24px can be 3:1, but other icons that are smaller, like in the Inline Alerts, would still need to be 4.5:1.
I think all of the smaller icons are always on white? So hopefully these sizes and ratios can allow the visual designers to get the color palette they want?


TS2345 [ERROR] in start() at main.ts

When run
deno check main.ts

i have following error
Argument of type '{ routes: { './routes/_404.tsx': typeof $0; './routes/_500.tsx': typeof $1; './routes/_app.tsx': typeof $2; './routes/_middleware.ts': typeof $3; './routes/auth/index.ts': typeof $4; './routes/auth/login.ts': typeof $5; ... 5 more ...; './routes/index.tsx': typeof $11; }; islands: { ...; }; baseUrl: string; }' is not assignable to parameter of type 'Manifest'.
  Types of property 'routes' are incompatible.
    Type '{ './routes/_404.tsx': typeof $0; './routes/_500.tsx': typeof $1; './routes/_app.tsx': typeof $2; './routes/_middleware.ts': typeof $3; './routes/auth/index.ts': typeof $4; './routes/auth/login.ts': typeof $5; './routes/auth/logout.ts': typeof $6; ... 4 more ...; './routes/index.tsx': typeof $11; }' is not assignable to type 'Record<string, RouteModule | MiddlewareModule<any> | AppModule | ErrorPageModule | UnknownPageModule>'.
      Property ''./routes/dashboard/[id].tsx'' is incompatible with index signature.
        Type 'typeof import("file:///D:/Documents/dev/coding/projects/finish/notes-app/routes/dashboard/[id]")' is not assignable to type 'RouteModule | MiddlewareModule<any> | AppModule |

FIx by add as unknown as Manifest
await start(manifest as unknown as Manifest , {
  plugins: [twindPlugin(twindConfig)]})

Can you please provide details on your project? This is difficult to reproduce otherwise.

Ah idk what is the problem, because not impact into my app when run.
Just  error lint, here this app that have issue

@marvinhagemeister, I think we can close this. The manifest looks like this:
const manifest = {
  routes: {
    './routes/_404.tsx': $0,
    './routes/_500.tsx': $1,
    './routes/_app.tsx': $2,
    './routes/_middleware.ts': $3,
    './routes/auth/index.ts': $4,
    './routes/auth/login.ts': $5,
    './routes/auth/logout.ts': $6,
    './routes/dashboard/[id].tsx': $7,
    './routes/dashboard/_middleware.ts': $8,
    './routes/dashboard/index.tsx': $9,
    './routes/dashboard/profile.tsx': $10,
    './routes/index.tsx': $11,
  },
  islands: {
    './islands/Utils.tsx': $$0,
    './islands/note/Edit.tsx': $$1,
    './islands/note/View.tsx': $$2,
    './islands/user/Profile.tsx': $$3,
  },
  baseUrl: import.meta.url,
}

The following three routes use async server components:
    './routes/dashboard/[id].tsx': $7,
    './routes/dashboard/index.tsx': $9,
    './routes/dashboard/profile.tsx': $10,

The error can get fixed by changing RouteContext to RouteContext<PageProps> or by using the canary version.
This is resolved by #1536, so waiting for 1.4 is also a viable option.
Additionally there's a problem with your islands/Utils.tsx file. You're trying to export a function useEventSource which isn't an island, and therefore doesn't belong in this folder.

Thanks for the investigation!
Closing because it's resolved with #1536 .


Have Tyne columns optionally persist to the URL

Some columns (e.g. ints, strings) could easily be persisted to the URL as a query parameter. This would make saving table states a lot nicer for users.
Not every data type may be easily supportable, so it's something that needs consideration for how it's implemented.

Initial work supports TyneStringColumn, other columns can be added later.


[Lib] Implement semver functions

Implement OPA's semver functions

[ ] semver.compare
[ ] semver.is_valid

To implement:

Use ak-semver to start off.  It contains placeholder implementations for the two library functions that only do parameter validation. It also contains tests and the logic needed to register these library functions.
Replace bail! statements in src/builtins/semver.rs with actual implementations.
To test, run cargo test semver. This will run tests that have been added in compare.yaml and is_valid.yaml.

fixed by #49


BaseUrl vs BaseURL, DocumentId vs DocumentID etc.

The most important thing is being consequent, and that looks good - but Golint suggests using all caps for these abbreviations:
http://goreportcard.com/report/spf13/hugo
I guess the only reason to to lowercase these is in languages with automated getter/setter generation (read Java).
Fix it or let it be?

I think it would be fun and great to fix it.  :smile:
But let's hear from others too.  Perhaps @spf13 could chime in?  :wink:

I would follow golint's suggestion on those. I think that the all caps as show reads best. How much would this impact users?

I would say "very little impact" on users, but I'm not totally sure.
But it's better to do it now than kick ourselves in the legs later on.
If @anthonyfok can do the work (suggest using an IDE with a rename function, like liteide) and do it in a pull request, then we can do some testing and see if we have to put in a temporary "alias or two".

Hi all,
This is the first commit that I came up with.  This should be rather safe, as I purposely avoided the command-line flags (cobra and pflag?) as well as struct fields like BaseUrl and Url that are used in Go HTML templates, to avoid breaking compatibility with existing scripts.
Creating aliases for template functions are as simple as adding an extra entry to the FuncMap, e.g. GetJson and GetJSON both work.  But what about template struct fields?
And, in addition to  --baseUrl=http://example.com/', do we want to support --baseURLand--baseurl` too?  Or should we make these long flag names case-insensitive?  (Or not?)
Many thanks!

I think all flag names should be case insensitive personally.

Great minds think alike!  :wink:  <grin, duck, run>
Does that mean we need to patch https://github.com/spf13/pflag?
(I took a quick grep ("upper", "lower", "sensi") and didn't find anything in the upstream https://github.com/ogier/pflag either.)

Correct.
Steve Francia
spf13.com
@spf13

On Mar 11, 2015, at 2:02 PM, Anthony Fok<EMAIL_ADDRESS>wrote:
Great minds think alike!
Does that mean we need to patch https://github.com/spf13/pflag?
(I took a quick grep ("upper", "lower", "sensi") and didn't find anything in the upstream https://github.com/ogier/pflag either.)
—
Reply to this email directly or view it on GitHub.

I have boldly merged the PR. Looked and tested OK by me.
As to the template funcs (getCsv, getCSV) I suggest add a "deprecated decorator func" that logs an error (so it shows up) that says: Fix this, this will be removed in the next version.

Just a quick status update:
Thanks to our group effort, the transition (e.g. BaseUrl → BaseURL) in the Hugo code is essentially complete, and the docs updated.
What remains to be done is to add an option to allow pflag to accept case-insensitive long flag names.  A feature request has just been filed at ogier/pflag/issues/18.  Hope someone from our community can submit a Pull Request!  :-)
(Keeping this issue open until case-insensitive long flag names are implemented.)

We use a fork of pflag. The right place for this is at spf13/pflag
--
Steve Francia
http://stevefrancia.com
http://spf13.com
http://twitter.com/spf13
On Mon, Mar 30, 2015 at 12:46 PM, Anthony Fok<EMAIL_ADDRESS>wrote:

Just a quick status update:
Thanks to our group effort, the transition (e.g. BaseUrl → BaseURL) in
the Hugo code is essentially complete, and the docs updated.
What remains to be done is to add an option to allow pflag to accept
case-insensitive long flag names. A feature request has just been filed at
ogier/pflag#18 https://github.com/ogier/pflag/issues/18. Hope someone
from our community can submit a Pull Request! :-)
(Keeping this issue open until case-insensitive long flag names are
implemented.)
—
Reply to this email directly or view it on GitHub
https://github.com/spf13/hugo/issues/959#issuecomment-87746272.

Sorry about that!  Due to a somewhat peculiar design of GitHub, I was unable to file an issue at https://github.com/spf13/pflag, though filing pull requests are perfectly fine.  So being a bit lazy (as I won't be able to produce a patch in the near future), I went ahead and filed the issue at https://github.com/ogier/pflag/issues instead.  Sorry about that!

I pay attention in spf13/pflag, but rarely in ogier, since that repo is typically dead.  But we already have support for this! (Well you can already do it)
Since you are using hugo (which I believe in turn uses spf13/cobra and spf13/pflag) you can check out the "normalized name" support. It allows you to write your own custom function which in your case could translate all flags to lowercase.
https://github.com/spf13/cobra/blob/master/command.go#L158
https://github.com/spf13/pflag/blob/master/flag.go#L184
It is being used in kubernetes to turn flags_like_this into flags-like-this so we have a consistent -

https://github.com/kubernetes/kubernetes/blob/master/pkg/util/flags.go#L25 Is the function we use in kube for _ to -
With https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/cmd/cmd.go#L125 as an example of applying the function.

@anthonyfok ^

Thanks for the heads up, @bep.
And special thanks to @eparis for writing the "normalized name" feature in Cobra, and for showing us how it is used.  :-)
I have made a preliminary Pull Request #1415, adding helpers.NormalizeHugoFlagsFunc() to handle flag name changes.  I initially thought of making all long flags case-insensitive with the help of strings.ToLower(), but then it would turn all long flags into lowercase, e.g. --buildDrafts, --pluralizeListTitles etc. becoming --builddrafts, --pluralizelisttitles, making them harder to read and less pretty.
So, I have decided to be more conservative with this pull request, i.e. only allowing --baseUrl in addition to the new canonical --baseURL, and disallowing other variations like --baseurl and --BASEURL.  If we relax the rules too much now, it would be hard to tighten the rules again without breaking end users' scripts.
If PR#1415 needs improvement, please do not hesitate to let me know.  For example, I wondered if helpers/general.go were the right place to add the helpers.NormalizeHugoFlagsFunc(), etc. etc.
Once PR #1415 or its iteration is committed, there are two more patches that I would like to commit:

Complete the Url-to-URL transition in tests: anthonyfok@d7f2ec3
Complete the Url-to-URL transition in docs: anthonyfok@d793ad0

What do you think?

I think we have accomplished the goals originally set out for this issue.  Let's close it!  :-)


operator doesn't update image version while upgrading a cluster

I deployed an Opensearch v2.4.1 with opensearch-operator-2.1.1
Then I tried to upgrade the cluster to v.2.5.0 by changing the spec.general.version(and plugin version as well)
Changed from
apiVersion: opensearch.opster.io/v1
kind: OpenSearchCluster
metadata:
  name: my-system
  namespace: my-system
spec:
  general:
    version: "2.4.1"
    httpPort: 9200
    vendor: opensearch
    serviceName: my-system
    pluginsList: ["repository-s3", "https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip"]
<SKIP THE REST>

To
apiVersion: opensearch.opster.io/v1
kind: OpenSearchCluster
metadata:
  name: my-system
  namespace: my-system
spec:
  general:
    version: "2.5.0"
    httpPort: 9200
    vendor: opensearch
    serviceName: my-system
    pluginsList: ["repository-s3", "https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip"]
<SKIP THE REST>

Data nodes got upgraded to v2.5.0 after apply the yaml file. But new master nodes and client nodes failed to boot up with errors below
Exception in thread "main" java.lang.IllegalArgumentException: Plugin [prometheus-exporter] was built for OpenSearch version 2.5.0 but version 2.4.1 is running
	at org.opensearch.plugins.PluginsService.verifyCompatibility(PluginsService.java:394)
	at org.opensearch.plugins.InstallPluginCommand.loadPluginInfo(InstallPluginCommand.java:820)
	at org.opensearch.plugins.InstallPluginCommand.installPlugin(InstallPluginCommand.java:875)
	at org.opensearch.plugins.InstallPluginCommand.execute(InstallPluginCommand.java:275)
	at org.opensearch.plugins.InstallPluginCommand.execute(InstallPluginCommand.java:249)
	at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104)
	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
	at org.opensearch.cli.MultiCommand.execute(MultiCommand.java:104)
	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
	at org.opensearch.cli.Command.main(Command.java:101)
	at org.opensearch.plugins.PluginCli.main(PluginCli.java:60)

Pod events show it's still pulling the v2.4.1 image
Events:
  Type     Reason     Age                     From               Message
  ----     ------     ----                    ----               -------
  Normal   Scheduled  3m57s                   default-scheduler  Successfully assigned my-system/my-system-masters-2 to host3
  Normal   Pulled     3m56s                   kubelet            Container image "public.ecr.aws/opsterio/busybox:1.27.2-buildx" already present on machine
  Normal   Created    3m56s                   kubelet            Created container init
  Normal   Started    3m56s                   kubelet            Started container init
  Normal   Pulled     3m2s (x4 over 3m56s)    kubelet            Container image "docker.io/opensearchproject/opensearch:2.4.1" already present on machine
  Normal   Created    3m2s (x4 over 3m56s)    kubelet            Created container opensearch
  Normal   Started    3m2s (x4 over 3m55s)    kubelet            Started container opensearch
  Warning  BackOff    2m27s (x10 over 3m48s)  kubelet            Back-off restarting failed container

Pod description shows it uses v2.4.1 as well
Containers:
  opensearch:
    Container ID:  containerd://743950cfdef64a76f394416a6a51e9277a610cd5f58cb45b44d281b40e3a8e8a
    Image:         docker.io/opensearchproject/opensearch:2.4.1

Related: https://github.com/Opster/opensearch-k8s-operator/issues/404

I was able to reproduce this. Looks like the operator starts with the upgrade of the data nodes but somehow at the same time starts a rolling restart of the master nodes due to the changed plugin which then fails due to old image version and new plugin version. Seems the operator is not correctly checking for upgrade-in-progress before starting the rolling restart. Not yet sure where in the code exactly and how to fix it.
As a workaround: For me it worked to wait until the data nodes were finished upgrading and then manually delete (kubectl delete pod) the crashing pods. After that the operator continued with the upgrade as expected.

Thank you @swoehrl-mw but the workaround doesn't work for me. I upgraded the operator to 2.2.0, and then all data node to 2.5.0, tried deleting master and client pods that are in a CrashLoopBackOff state. But they still use the OS v2.4.1 image and try to install a v2.5.0 exporter plugin(I need this plugin), so they stuck in CrashLoopBackOff

Did some more testing and found the problem: Non-data nodepools are not upgraded by the operator bust it instead delegates that to the statefulset controller of Kubernetes. This lead to the pods being recreated prematurely with a half-old-half-new config (old docker image, new plugin version) and subsequently to the crashes. That also got the operator into an unexpected state, so the upgrade stalled.
I've created a PR (#431) that should fix the problem.


Mac M2 装好环境后报以下错误 librocketmq.dylib是rocketmqV2.0.0版本下载的

Mac M2 装好环境后报以下错误 librocketmq.dylib是rocketmqV2.0.0版本下载的
OSError: dlopen(/usr/local/lib/librocketmq.dylib, 0x0006): tried: '/usr/local/lib/librocketmq.dylib' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64')), '/System/Volumes/Preboot/Cryptexes/OS/usr/local/lib/librocketmq.dylib' (no such file), '/usr/local/lib/librocketmq.dylib' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64'))

I am also MAC M2. Have you solved the problem

@xzkzdx @xiang1030 Are you install from pypi?

pip install rocketmq-client-python, and then download rocketmq-client-cpp-2.0.0-bin

Yes,All according to Readme.

I think the pre-compiled librocketmq binary in readme and pypi do not support M2 now, you can compile it from source and replace it.

build bad, not support gcc4.8.2，use gcc12 also build error like this '*** Configuration arm-apple-darwin22.2.0 not supported'.

using master branch of https://github.com/apache/rocketmq-client-cpp , and download openssl-3.0.8.tar.gz , and download  boost_1_75_0 file replace boost_1_58_0, and node line in build.sh  '#tar -zxvf ${fname_boost} &> unzipboost.txt' , so build success!


Convert string to sentencecase

Description
Write a program to convert string to sentencecase
Sentencecase is a style of writing in which the first letter of the first word is capitalized.
Input  : "hello world"
Output : "Hello world"

Tracking Issues

[ ] #5056
[ ] #5057
[ ] #5058
[ ] #5059
[ ] #5060
[ ] #5061
[ ] #5062
[ ] #5063
[ ] #5064
[ ] #5065
[ ] #5066
[ ] #5067
[ ] #5068
[ ] #5069
[ ] #5070
[ ] #5071
[ ] #5072
[ ] #5073
[ ] #5074
[ ] #5075

Hey @Pinklemonade33, !assign command is only for good first issue !!!
Please choose another issue.
Thanks for your interest in contributing to this project.


BlockActionPayload is missing app_unfurl information

The BlockActionPayload is missing unfurling related details when the action is triggered from the preview message associated within a shared link.
Reproducible in:
The Slack SDK version
We are using SDK version 1.23.1, but I'm pretty sure it's reproducible in the latest version of the SDK.
Java Runtime version
openjdk version "11.0.15" 2022-04-19 LTS
OpenJDK Runtime Environment Zulu11.56+20-SA (build 11.0.15+10-LTS)
OpenJDK 64-Bit Server VM Zulu11.56+20-SA (build 11.0.15+10-LTS, mixed mode)
OS info
ProductName:	macOS
ProductVersion:	12.5
BuildVersion:	21G72
Darwin Kernel Version 21.6.0: Sat Jun 18 17:07:25 PDT 2022; root:xnu-8020.140.41~1/RELEASE_X86_64
Steps to reproduce:
(Share the commands to run, source code, and project settings (e.g., pom.xml/build.gradle))

Create an app that handles the link_shared event.
In the ChatUnfurlRequest include a button in the UnfurlDetail.
In the Apps manifest, register the app to unfurl some test domain (e.g. example.com)
Share a link to that domain
Set a breakpoint within the app and example the BlockActionPayload

Expected result:
The BlockActionPayload should contain an appUnfurl field with the details from the app_unfurl property within the request body.
This relates to the TODO mentioned here:
https://github.com/slackapi/java-slack-sdk/blob/main/slack-app-backend/src/main/java/com/slack/api/app_backend/interactive_components/payload/BlockActionPayload.java#L44
Additionally the Container object within the BlockActionPayload should contain a string for the app_unfurl_url.
Both of these publicly documented for the alternative interactive payloads for unfurling, see:
https://api.slack.com/reference/messaging/link-unfurling#interactive_payloads
TBH... All we need right now is for the BlockActionPayload.Container to include the app_unfurl_url field. We are less concerned with the full details in the app_unfurl field.
Actual result:
There is no unfurling specific details within the block action payload.
Requirements
Please make sure if this topic is specific to this SDK. For general questions/issues about Slack API platform or its server-side, could you submit questions at https://my.slack.com/help/requests/new instead. :bow:
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

Hi @kfmartin, thanks for writing in! I just submitted a pull request resolving this issue: #1039 If everything goes well, the fix will be released within a few business days.

Hi @kfmartin, thanks for writing in! I just submitted a pull request resolving this issue: #1039 If everything goes well, the fix will be released within a few business days.

AWESOME. Thx U.

@seratch please clarify, will this be released as part of 1.24.1 or 1.25.0?
Also, when will the new version be published to maven's central repository?
Thx!

@kfmartin The version will be 1.25.0 and we are planning to ship the version next week.


Remove main from ros_typedb_interface

Create a new file ros_typedb_node or ros_typedb and move main from ros_typedb_interface to the new file. The class and node creation should not be mixed.

Done at 0fe1dec


comb-prop: Inlining reads from wires

When faced with an invoke of this form:
component p() -> (out: 32) {
  group do_x {
    out = foo.out;
  }
}
// In parent component
invoke p()(out = y.in);

The inline pass will generate assignments like this in the parent component:
group do_x {
  p_out.in = foo.out;
}
y.in = p_out.out;

We should be able to inline the read from the wire p_x.out with a write into y.in:
group do_x {
  y.in = foo.out;
}
y.in = p_out.out;

Stating somewhat formally–unconditional writes into ports from wires can be inlined into the context where the write to the wire occurs. I think this works even if p_x.in is written to from multiple groups but need a sanity check from @sampsyo and @EclecticGriffin on this.
Edits: Replace x.in with y.in for less overloading

Yo yo, I'm a tad bit confused about the first example here (probably just low-level stuff, but bear with me):

It looks like x is being used as an "out" port to p, right? Because it's assigned to in the group. (But the signature has it in the "in" position.)
But… it seems like the invoke uses x as an in port for p, so maybe the above is wrong??
Is the x that's the parameter to p the same as the x used in x.in? Probably not, right—that's just some other cell with the same name as the port?

Taken together, this makes me somewhat confused about what the transformed (last) listing is supposed to do… it naively looks like p_x.out is read without ever being written? Sorry to be so confused, but I don't quite get the insight yet…

Ah, sorry for the sloppy example, I've tried to fix it. Let me know if it makes more sense. At a high-level, here is the use case:

There is a subcomponent p with an output port called out.
The parent component invokes p and connects the input of another component, y to the output from p. This intuitively means that whatever output is generated on out during the execution of p, it should flow into y.in.

Now, when we inline the component p, we need some way to represent this output-input connection. First, we generate the wire p_out to represent p.out. Next, we define the connection y.in = p_out.out to state that whatever output is generated from p.out should flow into y.in (of course, this is not quite correct because the assignment is scoped and shouldn't always be active. This is a bug and I'll open a different issue for this).
Now, onto the actual pass. If we have a connection like this (forgetting for a moment that inlining is doing the wrong thing by not scoping it):
y.in = p_out.out; // p_out is a wire

It means that any use of p.in in the program can be replaced with y.in because the continuous assignment would've immediately forwarded to y.in anyways.
Hence, the optimization: if there is an unconditional continuous assignment from the output port of a wire to a subcomponent's input port, replace all writes to the input port of the wire with the input port of the subcomponent.
Does that make more sense?

When we have multiple reads to outputs:
group do_x {
  p_out.in = foo.out;
}
y.in = p_out.out;
x.in = p_out.out;

turns into:
group do_x {
  y.in = foo.out;
  x.in = foo.out;
}

So if I'm understanding correctly this replaces the continuous assignment with a direct one in the group? This won't always be possible right, only when there's a single consumer for a given sub-component?

This won't always be possible right

See the example in the comment above–you need to inline all consumers of the output port.

Ah fair, and the reason we don't do this by default is because it's hard to manage during the inlining process?

No, it's because its not the job of the inlining pass to cleanup the generated pass. Or rather, this logic is separate from inlining.

fair enough

Thanks for clarifying a bit! This makes sense to me. The "multiple reads" things is a tiny bit tricky, but I think the idea to treat "continuous assignments to wire components" as a propagation opportunity seems pretty clear. Nice!


More sphinx builders

This is a continuation of #2483 to discuss and track the other sphinx builders to add

[x] html: Already fully functional. #2483 tracks the warnings as fatal errors
[ ] linkcheck: In #2483
[ ] man: Needs some consolidations on the hatch-scripts/makefile
[ ] spellcheck: This is a difficult one, because PyEnchant does not seem to support full-word dictionaries, e.g. omni-present needs to be added as omni in the dictionary file. This builder needs more research if there are alternatives (original contributor switched to skyspell)
[ ] doctest + shtest: Requires converting the rst files to the corresponding directives. No usage for doctest found so far

For the spellcheck it seems @happz finds it usable in the current state, so I will deffer the implementation to them. I thought it didn't handle capitalization that well, e.g. CMake, but maybe there is a way of writing appropriate PyEnchant dictionary files for those. I would love to see that.


Replace cocoon behavior

Issue #3556
I am working to replace cocoon with existing Stimulus behavior. This PR replaces cocoon behavior with stimulus. It does not remove cocoon yet though.
Type of change

Refactor

How Has This Been Tested?
Helper Specs
Comments
Buttons
This PR results in some buttons looking a bit different, I don't know if that is a big deal or not. Before there was some inconsistency in the buttons, now more of them will look the same.
Before:

After:

These buttons match the ones in requests.
Disabled Buttons
There is a bit of inconsistency as to whether some Add Item buttons are disabled with a storage un-selected.
Ex: go to /diaper_bank/donations/new as an admin. The Add item button is usable. Go to diaper_bank/transfers/new, you cannot click Add Item until a storage location is selected.
I kept this this same. I am guessing it has to do with some operations needing to make checks against current inventory before even being possible. Just making note of the quirk.
Flaky Test
donation_system_spec:377 fails sometimes. I have no clue why, when I run the page with a system test the JS sometimes doesn't work correctly. I have not been able to replicate outside selenium in systems tests.

The code looks OK to me, but I really don't like the way that button looks. :frowning_face: If we can redo that style I'd be happier.

@cielf Do you have any input on how the buttons should look? Basically, the change I am making will unify (unless specified on a per button basis) the look of the add and remove buttons between requests and barcode related stuff.
How should they look? Image 1 in the PR is "barcode" style, image 2 is the "request" style

I like the idea of unifying on between the banks and the partners.  I wish we had a designer on board, but that being said,  I'm coming down solidly in favour of the style in the "Before"  image, above.
Reasons include:
1/  Our primary clients are the banks, and that's what they are used to
2/  It carries the flavour of the buttons in our indices well.
3/  Gosh darn it, it just looks better!

Apologies for the force push but I had to given I rebased on top of the other PR. Code isn't changing besides renaming some stuff and styles.
Styles are now unified:

Also I think picture 2 has a bit too much red and blue. I took it upon myself to make the submit button green, I think that makes it more in line with other save buttons as well.

Makes sense. I think both this PR and https://github.com/rubyforgood/human-essentials/pull/4184 are worth doing that on. If you do checkout 4184 locally make sure you precompile assets again, I had issues with cached assets trying to call the non-existent dependency.

It'll be at least tomorrow before I get to that - just setting expectations.

@elasticspoon since I merged the other PR, this one has conflicts now. Can you resolve?

I'll try to get to it by Monday.

The same thing is happening with the counties, FWIW.

Good catch should be fixed.

Passes my light functional testing now.  @dorner -- do you want to take a look at the final changes?

All good on my end!


Renamed launcher-backend to launcher-application

Removed test job as they are executed in Circle CI

Can someone please merge this?

[test]

[test]


merge-deep vulnerability

HI, I tried to use the library and I get a vulnerability error in the package that it has as a dependency merge-deep 3.0.3

I don't see any vulnerabilities in 3.0.3 version:
https://security.snyk.io/package/npm/merge-deep
Can you provide more information about this error message?


Create a contact info block

Add links to social networks (LinkedIn, Facebook, Instagram), phone number, email, Telegram username, etc.

Contact info is added


[bug] New-DbaDbUser support for input via pipeline

Example #4 for New-DbaDbUser is incorrect.  It states:
Get-DbaDbUser -SqlInstance sqlserver1 -Database DB1 | New-DbaDbUser -SqlInstance sqlserver2 -Database DB1
When I attempt to execute similar code, I get "The input object cannot be bound to any parameters for the command either because the command does not take pipeline input or the input and its properties do not match any of the parameters that take pipeline input."
Which makes sense because none of the params for New-DbaDbUser accept pipeline input.
It would be nice if it did in fact accept pipeline input so we could copy one or more users from one DB to another.  In the short term, removing the example from the docs would  be sufficient.
https://github.com/sqlcollaborative/dbatools/blob/master/functions/New-DbaDbUser.ps1

Command should accept piping from the following at a mininum:

Get-DbaDatabase
Get-DbaDbUser
Get-DbaLogin

I disagree here. Allowing multiple object types piped in makes the commands complex and may introduce a lot of bugs.
A New-Dba* command should at maximum allow pipeline input from the object one level above in the SMO hierarchy, so New-DbaDbUser should only allow pipeline input from Get-DbaDatabase.
I will correct the documentation and remove the wrong example, then closing this issue.
If someone wants the functionality of the mentioned example, I see this as a feature request and the solution should be a Copy-DbaDbUser command.
We may want to add pipeline input support to this command, but I would see this as well as a (low priority) feature request.


Faulty Ember+ Provider Matrix implementation on Lawo Nova17/Sapphire

I submitted the following bug report to Lawo directly, but their support department has requested that I submit it here because it's an Ember+ issue. I think they think it's a client/consumer issue, when it appears to be a server/provider issue.
I'm having issues with Ember+ control of a Saphire Mk2 (being used as a Nova17).
When a Connection request is sent to re-connect a route that is already made, the matrix returns a Connected message indicating that the original source has been de-routed and Source 0 is now routed, and leaves nothing routed to the target.
I've also noticed that even when switching a target between 2 different sources, the router delivers a Connected message indicating Source 0 is connected, before then delivering a second Connected message indicating that the requested source is routed to the target.

Hello,
since this is an issue with a specific provider implementation and not a general problem with one of the libraries this is not the appropriate forum. Please refer back to the service department asking them to put you in touch with the developers of the product in question.


Error when running matrixPost  with metrics: ['distance']

Hello.
I'm getting the following error when doing a matrixPost with metrics different from the default ['duration']
TimeDistanceMatrix distanceMatrix = await client.matrixPost(locations: cities, metrics: ['distance'], units: 'm');
Do you have any idea why this happens? The request works OK in Postman, why is this happening in Flutter Dart? How can I fix it?
Error: OpenRouteServiceException: Expected a value of type 'List<dynamic>', but got one of type 'Null'; Matrix value can't be determined for given inputs, as per https://openrouteservice.org/dev/#/api-docs/v2/matrix/{profile}/post, at url https://api.openrouteservice.org/v2/matrix/driving-car     at Object.throw_ [as throw] (http://localhost:3277/dart_sdk.js:5080:11)     at ORSMatrix$124matrixPost (http://localhost:3277/packages/open_route_service/src/open_route_service_base.dart.lib.js:1087:21)     at ORSMatrix$124matrixPost.next (<anonymous>)     at http://localhost:3277/dart_sdk.js:40641:33     at _RootZone.runUnary (http://localhost:3277/dart_sdk.js:40511:59)     at _FutureListener.thenAwait.handleValue (http://localhost:3277/dart_sdk.js:35438:29)     at handleValueCallback (http://localhost:3277/dart_sdk.js:35999:49)     at _Future._propagateToListeners (http://localhost:3277/dart_sdk.js:36037:17)     at [_completeWithValue] (http://localhost:3277/dart_sdk.js:35872:23)     at async._AsyncCallbackEntry.new.callback (http://localhost:3277/dart_sdk.js:35906:35)     at Object._microtaskLoop (http://localhost:3277/dart_sdk.js:40778:13)     at _startMicrotaskLoop (http://localhost:3277/dart_sdk.js:40784:13)     at http://localhost:3277/dart_sdk.js:36261:9

Hello.
I'm getting the following error when doing a matrixPost with metrics different from the default ['duration']
TimeDistanceMatrix distanceMatrix = await client.matrixPost(locations: cities, metrics: ['distance'], units: 'm');
Do you have any idea why this happens? The request works OK in Postman, why is this happening in Flutter Dart? How can I fix it?
Error: OpenRouteServiceException: Expected a value of type 'List<dynamic>', but got one of type 'Null'; Matrix value can't be determined for given inputs, as per https://openrouteservice.org/dev/#/api-docs/v2/matrix/{profile}/post, at url https://api.openrouteservice.org/v2/matrix/driving-car     at Object.throw_ [as throw] (http://localhost:3277/dart_sdk.js:5080:11)     at ORSMatrix$124matrixPost (http://localhost:3277/packages/open_route_service/src/open_route_service_base.dart.lib.js:1087:21)     at ORSMatrix$124matrixPost.next (<anonymous>)     at http://localhost:3277/dart_sdk.js:40641:33     at _RootZone.runUnary (http://localhost:3277/dart_sdk.js:40511:59)     at _FutureListener.thenAwait.handleValue (http://localhost:3277/dart_sdk.js:35438:29)     at handleValueCallback (http://localhost:3277/dart_sdk.js:35999:49)     at _Future._propagateToListeners (http://localhost:3277/dart_sdk.js:36037:17)     at [_completeWithValue] (http://localhost:3277/dart_sdk.js:35872:23)     at async._AsyncCallbackEntry.new.callback (http://localhost:3277/dart_sdk.js:35906:35)     at Object._microtaskLoop (http://localhost:3277/dart_sdk.js:40778:13)     at _startMicrotaskLoop (http://localhost:3277/dart_sdk.js:40784:13)     at http://localhost:3277/dart_sdk.js:36261:9

Hmmm just seems like a issue of making the list nullable. Will diagnose this in a few hours.

@Dhi13man I tried finding out what's wrong but I keep getting the same error. It doesn't make sense, the list of locations I pass isn't null, it's in correct type and everything. Tried everything. metris ['distance'] causes the problem but can't find out why.
I run it again and this time the same error is thrown at a different line:
'Error: OpenRouteServiceException: Expected a value of type 'List', but got one of type 'Null'; Matrix value can't be determined for given inputs, as per
https://openrouteservice.org/dev/#/api-docs/v2/matrix/{profile}/post, at url https://api.openrouteservice.org/v2/matrix/driving-car
at Object.throw_ [as throw] (http://localhost:3277/dart_sdk.js:5080:11)
at ORSMatrix$124matrixPost (http://localhost:3277/packages/open_route_service/src/open_route_service_base.dart.lib.js:1087:21)
at ORSMatrix$124matrixPost.next ()
at http://localhost:3277/dart_sdk.js:40641:33
at _RootZone.runUnary (http://localhost:3277/dart_sdk.js:40511:59)
at _FutureListener.thenAwait.handleValue (http://localhost:3277/dart_sdk.js:35438:29)
at handleValueCallback (http://localhost:3277/dart_sdk.js:35999:49)
at _Future._propagateToListeners (http://localhost:3277/dart_sdk.js:36037:17)
at [_completeWithValue] (http://localhost:3277/dart_sdk.js:35872:23)
at async._AsyncCallbackEntry.new.callback (http://localhost:3277/dart_sdk.js:35906:35)
at Object._microtaskLoop (http://localhost:3277/dart_sdk.js:40778:13)
at _startMicrotaskLoop (http://localhost:3277/dart_sdk.js:40784:13)
at http://localhost:3277/dart_sdk.js:36261:9'
Please let me know if there is anything I could do to help you troubleshoot it, I need it fixed for my project. Thanks.

Checking now.

Please fork and test the branch 12-error-when-running-matrixpost-with-metrics-distance. The Issue should be resolved there. If everything works fine, will deploy new version with fix.

Please fork and test the branch 12-error-when-running-matrixpost-with-metrics-distance. The Issue should be resolved there. If everything works fine, will deploy new version with fix.

Yes, it works!!! No error and I get the response I wanted. Everything seems okay. Thank you so much for the quick fix!

Package v1.2.2 with the fix is now up on pub.dev. Thank you for your cooperation.


add tools to prereq section

I've arranged for abstracts in the the hands-on lab session details for the TechEd On Tour event web pages to include a link to the prerequisites. To this end, I think it would be a good idea to surface the tools requirement as well, for those wanting to go for the recommended option. Thanks.

@lechnerc77 just wondering about the Terraform extension for VS Code too? The Local installation mentions it, but the Recommended approach does not ...


Error when re-running the workflow

After signing the comment "I have read the CLA Document and I hereby sign the CLA". The workflow reruns but I see this error
"Error: Error occurred when re-running the workflow: HttpError: Bad credentials". I have the personal access token added to the repo.
To Reproduce

Create a pull request . The action runs and adds a comment saying "sign the cla"
Once the cla is signed it get added to the signatures
When the workflow reruns and see the error in Annotations ""Error: Error occurred when re-running the workflow: HttpError: Bad credentials".

Expected behavior
Workflow re-run and succeed
Screenshots

Hi @swathimr, As far as I know, it is not possible.

yes, there was an issue with the github auto deleting token. I do have another question. Is there a way to run the issue_comment github action only for a specific directory ?

I have exactly the same problem. The PERSONAL_ACCESS_TOKEN was not deleted in my case - it is still there (as expected).
How did you solve the problem?


chore: use go 1.22

related to https://github.com/jenkins-x/jx/pull/8663

/ok-to-test

@JordanGoasdoue Well, that unfortunately failed. You can see the result of CodeQL / Analyze (go) (pull_request)  yourself.

indeed @msvticket I was looking about it
seems to be fixed with this commit
Now let see if everything is :white_check_mark: :crossed_fingers:


Use fleet api instead of cli

https://github.com/CenturyLinkLabs/fleet-api

Using the fleet api is certainly preferred but there were some issues with that.
The fleet api isn't enabled by default, so you have to manually enable it https://coreos.com/docs/launching-containers/config/fleet-deployment-and-configuration/
This is fine, but it looks like things like Deis ship with there own custom version of fleet instead of the standard version on CoreOS since they use it as  part of their back-end. As far as I could see there was no obvious way to enable the fleet API on Deis.

fleet-api version 0.9.0 writes directly to ETCD so it will work without the fleet api turned on.
I'm fairly certain Deis does not ship with a custom version of fleet.  The documentation you linked to works correctly on Deis 1.7.3.

Tested working using fleet-api v0.9.0


No error if the owner address is entered as a safe address during the load safe flow

Title/Description
Check for the
Environment

Browser: Chrome
Safe:
Wallet: MM
App Version: 3.6.0
Environment:

production (mainnet)

Steps to reproduce

Go to the rinkeby
start load existed safe flow
add owner address or use liltestowner.eth as a safe address
add name and press submit

Current result: The address is marked in green as the supported, correct safe address but the user can not go to the next safe.
Expected result
The error should be shown if the wrong/not supported address is entered as safe address
Screenshots

This was solved in #2324 and deployed in v3.6.5


Update cadence

Hi, plotly.py maintainer here. I ended up in this repo after learning that the VSCode Python plugin relies on @nteract/transform-plotly for rendering plotly figures, and that @nteract/transform-plotly gets it's plotly.js version from the project.
I see that there is a bot generated PR to update this repo with the most recent version of plotly.js at https://github.com/nteract/minimal-plotly/pull/12, but that this update hasn't been applied since December. I was wondering how frequently you aim to merge/publish these plotly.js updates to NPM, and whether this is something that could happen alongside the plotly.js releases themselves.  Thanks!

Friendly ping @rgbkrk @lgeiger

Hello @jonmmease! Thank you for being proactive and surfacing this in the Slack team.
I just did a one-off release and bumped this up to 1.47.1. Moving forward, I'll merge the renovate PRs and publish new releases on Monday evenings alongside the release cycle from the nteract monorepo.

Thank you so much!  I'm very excited for nteract and VSCode to be first-class interfaces for plotly.py users.


Expose embeddedSandbox in npm package

Hi,
embeddedSandbox seems to not be exposed in the npm package.
Can I ask why?
thanks in advance for your hard work!

Hi! embedded Sandbox is now exposed via npm package: https://www.npmjs.com/package/@apollo/sandbox
Thanks for the message!


Extraterm not opening on Fedora KDE 32 !!!

Hey there, I hope you are doing well in those hard times on all planet.
I am running at home a minecraft server and well, i don't really like "Konsole" default console. I just discovered Extraterm, which is verry beautifull :)
I looked arround, no tutorial on Google. I downloaded as well the Linux package on official Github, but then if I read the documentation, it say to run "extraterm" exacutable file, which I am doing, I also tried to run extraterm.desktop file, it's loading, but nothing is happening.
please help !

Also here is a screenshot if I run the executable in the console:

Up !

The zip version works here, so I'm at a bit of a loss as to the real problem. Did you just use unzip from a terminal to unzip it?

I did right click and extract.

I'm setting up Fedora is a virtual machine. Did you use the KDE archive app? (ark, if I remember correctly)
The .desktop file might not work from the zip.

I did right click and extract... I don't know the software

And I have extracted the .Desktop file and tried ALL other files one by one ... 20 times.. downloaded/redownloaded...

When you click on .Desktop it ask do you want to OPEN on RUN I say RUN

Nothing... And it's the only thing that doesn't work on my KDE

I just reproduced the problem in my Fedora virtual machine using KDE and its Ark app, via the "Extract here" menu item in Dolphin.
Now I just have to figure out why it fails.

I just reproduced the problem in my Fedora virtual machine using KDE and its Ark app, via the "Extract here" menu item in Dolphin.
Now I just have to figure out why it fails.

I'm so happy you put energy in the problem ^^

By the way, the work around for you is to download the linux.zip file, then in the terminal unzip it with unzip extraterm-.....zip instead of using "Extract here" from Dolphin.
The root problem is that some unzip apps correctly support symlinks, while others don't. I'm now trying to remove any symlinks from the zip file entirely.

Fixed in 0.54.0. It should work now regardless of whether your unzip tool restores the symlinks or not.


Add arm64 RIDs to Razor LanguageServer project.

Summary of the changes
Adds arm64 RIDs for each of the supported platforms.

Fixes: https://github.com/dotnet/razor-tooling/issues/6406

@allisonchou with the new VSCode update if it's easy to absorb these changes / produce arm based language servers that'd be dope. If it turns out to be heavy / extra to absorb these changes we can re-triage

any ETA for this?


fixed autocomplete for subnets

The "right" thing to do would be to change parameter in config.json from "diskSize" to "diskSizeGb", but that will break existing pipelines that use the plugin, e.g. existing pipelines that have Actions using "Launch VM" might refuse to open at all, error on import/export, fail to execute. By accomodating the misnamed parameter in app.js instead we avoid this problem.


Example with picture in picture functionality

https://jira.ops.aol.com/browse/OMSDK-86

Drop deployment target to iOS 8.0.

Please, add 2 screenshots that will demonstrate PiP mode (active and non-active) and README.md that will briefly describe this example.

Done


(#1918) Do not mark wallet address as watched + fix earlier txn errors

These changes satisfy concerns in #1918 and rescues/corrects old order transactions

Can one of the admins verify this patch?


Give Cross-Certified User Admin Access to Server

I can access the demo/DEMO server with my local Notes client after cross-certifying my Notes ID, but I don't have the admin rights I need to create/import agents and change the server settings.
We discussed this with @JustinProminic in the meeting today, and we would like to give the cross-certified user full access to the Domino server.  We can do this in in a couple stages.
Update setup.json to create a special group for cross-certified users and give it access:

Group Name:  AutomaticallyCrossCertifiedUsers
Update the server document to give AutomaticallyCrossCertifiedUsers the same access as LocalDomainAdmins.  We can tweak these settings later if needed.

When running CrossCertifyNotesID (Java), add the corresponding user to the AutomaticallyCrossCertifiedUsers group.  To get the user ID, find the newly created cross-certificate document in the ($CrossCertByName) view in names.nsf.  In the long term, we may want to support multiple cross-certified IDs, so we may need a better solution for this.

@dpastov updated setup.json to create the AutomaticallyCrossCertifiedUsers group and update the server document.
I updated CrossCertifiyNotesID to get the user name for the ID with the logic described above and add it to the AutomaticallyCrossCertifiedUsers groups.
However, I'm still seeing that my cross-certified Notes ID doesn't have sufficient access to import the agents.  It seems like it is not recognized as part of the group.  I'll continue to debug this.

After revisiting the deploy_database.sh script, I found that -Default- did not have permissions to create agents, which explains the DXL Importer error (this would not be a problem if the group were working properly).
I also tried updating the permissions for Anonymous to Author to resolve the runtime permission errors - I have seen this before for DXL Importer.   This worked in my initial test, but Author access seems too high for this database - I'll experiment with lowering this.
These updates resolved the agent errors I have been fighting recently, but I don't think it explains all of the strange behavior I have been seeing.   I'll do more testing with this.

After some testing, I was able to reduce the permissions for Anonymous to "depositor" with isPublicReader and isPublicWriter.
I tried removing the server document update for CrossCertifyNotesID, but I found that this was still necessary.
The permissions are working reliably in my tests now.


Brightness & contrast modifiers

I was having issues with what getpoint was returning, looks like some formats have RGB values higher than 255
Added test to make sure pixel analyzer works for float

Thank you, Tom.


Release numbers are confusing

From @shpros on November 8, 2016 23:39
As of this writing, NPM says 0.0.4 is the latest of 4 releases, but the releases tab of this GitHub project says that v0.8.0 is the most recent of 14 releases going back to v0.3.0.

Why do the release numbers change so drastically between GitHub and NPM?
We are currently using NPM release 0.0.3 but ought to be moving to 0.0.4. So that we can determine what has changed, to which GitHub releases do the NPM releases 0.0.3 and 0.0.4 correspond?

Copied from original issue: Azure/azure-event-hubs#292

True - The reason is that multiple SDKs were shipped from the same Github Repo (the generic https://github.com/Azure/azure-event-hubs).
Now that each SDK has its own repository, things should get better.
Right now the convention is to tag a new release with the date (2017-3-29 for example) and include the NPM package version in the title. This is aligned with what's done for the Azure IoT Hub SDK and takes into account the fact that multiple NPM packages with different versions may be shipped from the same repository. Hopefully we can make our release notes can provide the necessary explanations to not make this confusing.
As of now there is no plan to split further and have a 1-to-1 match between repos and packages.


Objects should be upcastable to supertraits

trait T {
    fn foo(@mut self);
}

struct S {
    unused: int
}

impl T for S {
    fn foo(@mut self) {
    }
}

fn main() {
    let s = @S { unused: 0 };
    let s2 = s as @T;
    let s3 = s2 as @T;
}

error: failed to find an implementation of trait @T for T

I think the https://internals.rust-lang.org/t/casting-families-for-fast-dynamic-trait-object-casts-and-multi-trait-objects/12195 should cover this case too.

@rust-lang/lang talked about this issue today. Allowing for evolution of Rust syntax since pre-1.0, to the best we can tell, this is something that you can now do. Closing; please comment if there's still an issue here.

Allowing for evolution of Rust syntax since pre-1.0,  this is something that you can now do

do you mean "that you cannot do" 😕. also this issue doesn't seem to have any relationship with syntax.

Because of the ancient syntax, it's unclear what's being asked for.
The 4 people in the meeting who looked at this all looked at the example code and understood the request to be about casting a reference to a struct type into a reference to a trait type (a trait that the struct type implements).
If the request is about turning a sub-trait reference into a super-trait reference that is indeed not currently possible.
However, since the example code only has one trait and one struct, we did not think that the sub-trait/super-trait matter was involved here.

As one of those 4 people, my first thought was the same as yours, @kennytm -- that from the issue header this was about casting one dyn Trait to another one, which certainly is not currently possible.
I guess the later examples (https://github.com/rust-lang/rfcs/issues/2765#issuecomment-31582946) are that, but the one in the OP currently isn't.
I'm not sure what the general intent of issues in this repo is these days.  So it doesn't really bother me whether it's open or closed...

@Lokathor the subtrait/supertrait discussion starts from https://github.com/rust-lang/rfcs/issues/2765#issuecomment-31582946. In any case the wording above is not clear enough to tell what is going on — because "this is something you can now do" so the issue (which does not mention changing syntax at all) is closed??? can't comprehend.

The original question was about casting a trait reference to the same trait or another trait that should be part of the typeat that point.

I'm so confused. I'm coming here from https://github.com/rust-lang/rust/issues/18600#issuecomment-317142342 which closes itself as a duplicate of this one. But the discussion here makes it sound like it isn't a duplicate?
This issue is also referenced by https://github.com/rust-lang/rust/issues/18469

@scottmcm wrote:

I guess the later examples (#2765 (comment)) are that, but the one in the OP currently isn't.

I think that was exactly the issue: the example in the OP looked like something that's now possible, and it wasn't obvious that this issue was asking for something else.

Given the amount of time that has passed, and the amount of syntax evolution, could someone involved in the original issue please either point to a current issue that's requesting the same thing using current syntax, or file one?

Kvark's earlier example still doesn't compile:
trait U {}
trait V : U {}
fn foo<'a>(a : &'a dyn V)-> &'a dyn U {
    a as &dyn U
}

error[E0605]: non-primitive cast: `&'a (dyn V + 'a)` as `&dyn U`
 --> src/main.rs:4:5
  |
4 |     a as &dyn U
  |     ^^^^^^^^^^^ invalid cast
  |
help: borrow the value for the cast to be valid
  |
4 |     &a as &dyn U
  |     ^

error[E0277]: the trait bound `&'a (dyn V + 'a): U` is not satisfied
 --> src/main.rs:4:5
  |
4 |     a as &dyn U
  |     ^ the trait `U` is not implemented for `&'a (dyn V + 'a)`
  |
  = note: required for the cast to the object type `dyn U`

error: aborting due to 2 previous errors

I actually made a post about this on reddit confused why this wasn't working a few days ago here. If you're looking to collect more examples of confusion, here's another one.

As I noted above, one could develop this outside rustc+core using https://internals.rust-lang.org/t/casting-families-for-fast-dynamic-trait-object-casts-and-multi-trait-objects/12195  You'll need to manually insert compilation units that unify your casting families however.

Side note that I was thinking that I would like to mentor someone through the implementation steps required here. I think that's the primary blocker. Perhaps I will file a lang team proposal around it first though.

Current tracking issue for this is https://github.com/rust-lang/rust/issues/65991


Support for non-decimal numbers in device string, issue in return value of usb/get_attribute

Maybe I'm missing something but currently non-decimal device strings don't seem to be supported. This should be easily possible. Feel free to point out any issues with this.
the get_attribute() function in usb.py returns only one value if the attribute is unknown. This raises an error because it's obviously not iterable.
try:
return self._get_attribute(attribute), constants.StatusCode.success
except UnknownAttribute as e:
logger.exception(str(e))
return constants.StatusCode.error_nonsupported_attribute

You are completely right about the missing return value after UnknownAttribute. Please look in the whole code as this error repeats a few times.
Regarding the non-decimal device strings. I full agree that we should handle this situations gracefully. Your proposal sounds reasonable and would work if we keep the output of the resource name parser as strings (which is the case)

Regarding the other possible suspects, I found:
highlevel.py:
line 332/334
line 351/353
Currently not sure what the correct fix for those is, append ", constants.sucess" to 334 and 353?
tcpip.py:
line 163/165
fix for that: "return NONE, StatusCode.error_timeout" in line 165 ?
Did you find any others?

I think there are more in usb.py but fix those and I will revisit all of them.

Ok, added them. Not sure if the fixes are correct though - could you please check?

I think this is fixed. Feel free to reopen if needed


MNT: Re-rendered with conda-smithy 2.0.1

This will give us python 3.6 packages.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


implement PyCall-based orm

Codecov Report

Merging #7 (ce8a052) into master (535701b) will decrease coverage by 1.51%.
The diff coverage is 80.70%.

:exclamation: Current head ce8a052 differs from pull request most recent head 3f93656. Consider uploading reports for the commit 3f93656 to get more accurate results

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   94.46%   92.94%   -1.52%
==========================================
  Files           3        3
  Lines         289      326      +37
==========================================
+ Hits          273      303      +30
- Misses         16       23       +7

Impacted Files
Coverage Δ

tyjuliasetup/__init__.py
89.89% <80.70%> (-2.03%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Elec_Loss v2: bug correction and code improvement

[BC] Solve issue on slice model calculation for SCIM
[CC] minor improvements for LUTdq methods

Work in progress, not ready to be merged


HikariCP does not respect minimumIdle

Hello,
We are using v2.3.5 with PGSimpleDataSource; The pool is configured programmatically.
I have noticed that around each 9 minutes the HouseKeeper closes most of the connections (the minimum idle number of connections) due to IdleTimeout. This doesn't make sense - why would the pool close connections if the minimumIdle is reached ? Note that the maxLifetime is set to 0 (zero).
		config.setRegisterMbeans(true); // Default false
		config.setConnectionTimeout(60000); // default 30000 ms = 30s
		config.setIdleTimeout(60000); // Default: 600000 (10 minutes)
		config.setMaxLifetime(0); // Default: 1800000 (30 minutes)
		config.setMinimumIdle(10); // Default: same as maximumPoolSize (10)
		config.setMaximumPoolSize(100); // Default: 10
		config.setValidationTimeout(50000); // Default: 5000 ms = 5s

Reading the HikariCP code I just don't see why in this check below, the minIdle number is not respected (not checked anywhere) - this makes using a pool meaningless ? :
           if (bagEntry.evicted || (idleTimeout > 0L && now > bagEntry.lastAccess + idleTimeout)) {
              closeConnection(bagEntry);
           }

@yosstef From the minimumIdle documentation:
"This property controls the minimum number of idle connections that HikariCP tries to maintain in the pool. If the idle connections dip below this value, HikariCP will make a best effort to add additional connections quickly and efficiently."
Generally speaking, in an application that is even just a little active, connections very quickly obtain lastAccess timestamps that are different than each other.  When this occurs, idle connections will start expiring at different times, rather than all at once.
Expiring connections that reach timeout is considered a higher priority than maintaining a constant number of minimum connections.  Having said that, what you omitted from you code fragment above, is this:
fillPool(); // Try to maintain minimum connections

After expiring connections, HikariCP will immediately (within tens of milliseconds) replace them in the pool with new connections until minimumIdle is reached.  If you find that this is not occurring, please let us know.

I meet one question in 2.4.1 .
Do I need to open one new issue?