[BUG] Exception in RestClient when Content-Encoding header is missing

Describe the bug
Using OpenSearch sdk version 1.3.5 with an OpenSearch domain on v1.3, I am trying to use the RestClient to GET /_ingest/pipeline/[id] via the API ingest().getPipeline(). But an exception is thrown on this line https://github.com/opensearch-project/OpenSearch/blame/main/client/rest/src/main/java/org/opensearch/client/RestClient.java#L361 .map(HttpEntity::getContentEncoding), due to Content-Encoding being null (although the response itself has 200 status). This occurs both with and without:compressionEnabled(true) and custom RequestOptions that add an Accept-Encoding header input to ingest().getPipeline(). This code was/is working with Elasticsearch Java High Level REST client v7.1.1.
Can a fallback be added in the code in case the Content-Encoding header is not present on a response?
To Reproduce
Steps to reproduce the behavior:

Create an OpenSearch domain v1.3
Set up OpenSearch RestClient v1.3.5
Create an ingest pipeline
Try to get the ingest pipeline config using the RestClient's ingest().getPipeline() with RequestOptions.DEFAULT
Observe exception

Expected behavior
Exception should not be thrown
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
I am seeing a similar stack trace as reported in: https://forum.opensearch.org/t/we-are-facing-call-site-initialization-exception/10857.
https://opensearch.org/docs/1.3/opensearch/rest-api/ingest-apis/get-ingest/

Is this issue exists in 2.x branch?

The answer to https://github.com/aws/aws-sdk-java-v2/issues/652 resolved this for me. I had to upgrade org.apache.httpcomponents:httpcore to version 4.4.9.


✨ feat (content/team.md, static/css/style.css): added and refactored social icon and links to each member, testing w/ KemingHe's first

I refactored and renamed iconTextLink shortcode to easily add outlinks with icons. This is useful for the team page, I included an example of that with @KemingHe 's github

Keming, I wasn't able to recreate the issue on the screenshot in chromium or firefox, looks like the CSS didn't properly load when you started local server. I did replace all logo files with a pre-downscaled png, which should resolve the issue. I also slightly adjusted the CSS to impose a 16px height limit, which should bound anyone who didn't predownscale a png. The margin is important to have a gap between an icon and text. The text is by default baseline aligned and should not be changed, so the icon will be aligned properly.

[ ] Maybe a darker background will work? I'll add that as a suggestion and let you decide. (Def not background-color: black; 🤣 )

@KemingHe I changed the style to icons only here, I think it looks clean:


If they all point to the same Redis, and claim a lock on the same key, then this will block them all right?
Trying to look for a blocking solution that will work across multiple VM's.

Hey NekoNova,
That's exactly right!

okay, cheers for the confirmation.
Concurrency issues are starting to get on my nerves, and trying to push forward the Redis Semaphore implementation to handle them across Sidekiq queues.


Pausing before passing conversation to next agent[Feature Request]:

Is your feature request related to a problem? Please describe.
The feature request is related to the issue of managing rate limits.  As all are aware OpenAI limits the rate of tokens per second and it is easy to hit that limit without warning.  A way to insert a pause (similar in function to the Unix "sleep" command) to allow a set pause to occur before the next agent in a conversation starts processing the output of the previous agent in the conversation would allow for a simple way of avoiding hitting the rate limits for a given account.
Describe the solution you'd like
For instance, in the "Auto Generated Agent Chat: Performs Research with Multi-Agent Group Chat" example it is easy to hit the rate limit as the Coder agent, Critic agent, and Scientist agent discuss how to implement the task of finding papers on LLM applications from arxiv and creating a markdown table of different domains.
The solution I'd like to see is a method to set a pause time, similar in functionality to the Unix "sleep" command, before the output of one agent is sent onto the next agent.  This should be easier to implement than a more dynamic 'check to see current number of tokens sent during a window of.....' although ultimately this would be the best solution of all.
Additional context
No response

@sonichi Do you think this would have value? If so, where would be the sleep call live in the code, and how should the option be exposed?

This behaviour could be achieved with existing code by registering a hook that just calls sleep.
@rickyloynd-microsoft am I right?

Good point. For instance, the new AgentCapability class could be called CanSleep. It would define a hook and call register_hook to attach it to the existing hookable method, process_last_message, which gets called by generate_reply on every invocation.

@LopezNuance Can we close this?

Has there been any changes?  I only see a discussion on how this could be implemented.

@LopezNuance this could be implemented on the application level, there is no need to change anything in the framework. If you wish, we could create an example for you.

Ok, I misunderstood the posts, sorry about that.  Sure, an example would be great.  Then go ahead and close the feature request.  Thanks!

Just to link #1240 . It would be a piece of cake with middleware.

I see Davor Runje's comment especially applies:
https://github.com/microsoft/autogen/pull/1240#issuecomment-1892502988


support for a validation pre-commit hooks

Pre-commit hooks for dasel validate.
Notes

The version specified in the documentation (v1.25.1) refers to a new release that includes .pre-commit-hooks.yaml
~The version specified in .pre-commit-config.yaml (master) will work only after merging this change. For testing (see below) the commit hash for this pull-request should be used (e.g. https://github.com/TomWright/dasel/pull/232/commits/1217e03c614c14d0c98b0a7cdc5c65e93132b378). This file is optional, and needed only if pre-commit is not in used for maintaining dasel project~
~I couldn't find the source code for the documentation files to update them accordingly, and that is the reason that the link in the Features section is left empty. If you will point me to them, I would love to make a contribution~

Tests
Execute pre-commit within the repository (added for sanity tests)
$ pre-commit run -a dasel-validate --verbose
Validate JSON, YAML, XML, TOML...........................................Failed
- hook id: dasel-validate
- duration: 0.04s
- exit code: 1

fail tests/assets/broken.json could not load input: could not unmarshal data: invalid character '}' after array element
pass benchmark/data.json
pass benchmark/data.yaml
pass .github/workflows/codeql-analysis.yml
Error: 1 files failed validation
pass tests/assets/example.yaml
pass .github/FUNDING.yml
pass .github/workflows/build-test.yaml
pass benchmark/data/nested_property.json
pass benchmark/data/root_object.json
pass benchmark/data/delete_property.json
pass .github/workflows/build.yaml
pass tests/assets/deployment.yaml
pass tests/assets/example.json
pass .github/workflows/build-dev.yaml
pass benchmark/data/array_index.json
pass benchmark/data/list_array_keys.json
pass .github/workflows/bump-homebrew.yaml
pass benchmark/data/top_level_property.json
pass .pre-commit-hooks.yaml
pass tests/assets/example.xml
fail tests/assets/broken.xml could not load input: could not unmarshal data: xml.Decoder.Token() - XML syntax error on line 1: element <a> closed by </b>
pass benchmark/data/append_array_of_strings.json
pass .github/workflows/test.yaml
pass codecov.yaml
Error: 1 files failed validation
pass benchmark/data/overwrite_object.json
pass benchmark/data/update_string.json

cc @TomWright

Hey @foolioo, the repo containing the docs are here: https://github.com/TomWright/dasel-docs

@foolioo Yes please that would be great!

Could you update the CHANGELOG.md file please? You'll need to add the version at the top of the file aswell as links towards the bottom.
Once that's done I'll merge and release

Hello there! I'm the author of JReleaser 😄
Though I'd love to see more people using the tool I could not fail to notice this particular project is built with Go, in which case I strongly recommend you to use GoReleaser instead, as it provides deeper integration with Go's build toolchain https://goreleaser.com/

Thanks for the suggestion @aalmiray, I'll take a look 😄


Consume master of fauxhai

Yup.
/cc @chef/client-dev-tools @chef/engineering-services

Seems reasonable? Fauxhai should be stable on master to my knowledge...


bugfix: 修复重复收集相同的自动触发任务的问题 (fixed #1)

Codecov Report

:exclamation: No coverage uploaded for pull request base (V2.1.345rc@223f808). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             V2.1.345rc      #89   +/-   ##
=============================================
  Coverage              ?   58.90%
=============================================
  Files                 ?      394
  Lines                 ?    25912
  Branches              ?        0
=============================================
  Hits                  ?    15263
  Misses                ?    10649
  Partials              ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 223f808...18f79a3. Read the comment docs.


[ZEPPELIN-1721] Let each visualization define 'setting'.

What is this PR for?
ZEPPELIN-1003 made an abstraction layer for visualization and now visualizations are moved to [zeppelin-web/src/app/visualization/builtins] https://github.com/apache/zeppelin/tree/master/zeppelin-web/src/app/visualization/builtins from single monolithic place paragraph.controller.js.
However, 'setting' of each visualizations (i.e. pivot keys, groups, values selection, column selection in scatter chart, etc) are still in paragraph.controller.js and its html template is hardcoded.
This PR lets each visualization implement their own 'setting' and provide facilities to store/load configs for it.
https://github.com/apache/zeppelin/blob/d80a810184457b51b0d9229c663badca01199c26/zeppelin-web/src/app/visualization/builtins/visualization-linechart.js#L86  can be an simple example how each visualization can have its own setting and save configs.
What type of PR is it?
Improvement
Todos

[x] - Make each visualization provides setting template
[x] - load / save configs for the setting

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1721
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

I think test failures are irrelevant to this change. Ready to review.
AngularElem
- should able to be created from implicit conversion
Run completed in 1 second, 632 milliseconds.
Total number of tests run: 18
Suites: completed 5, aborted 0
Tests: succeeded 17, failed 1, canceled 0, ignored 0, pending 0
*** 1 TEST FAILED ***

Results :

Failed tests:
  PySparkInterpreterTest.testBasicIntp:128 expected:<SUCCESS> but was:<ERROR>

Tests run: 7, Failures: 1, Errors: 0, Skipped: 0

Merge to master if there're no further discussions.


schedule to restart instance

Hi,
is there a way to tweak this solution to restart an EC2 instance?

We only support start/stop scenario's as a way for customers to save  on their EC2 and RDS costs. As we constantly monitor the state of each instance at every schedule cycle to act on desired schedule changes, the logic is optimised for "period based" scheduling. Rebooting an instance, which is more a "moment" based scenario, fits better in a different schedule logic that is bases in time or events as we use in our Ops Automator Solution. We're working on a next version of this solution and we might add it as an action you can execute to reboot tagged instances at specific moments in time or events, like adding a specific combination of tags.

+1


Errors in the CUDOS dashboard preventing it from working with asset bundle exports

I have created a dashboard following the manual instructions using the CUDOS template with the following ARN - arn:aws:quicksight:us-east-1:223485597511:template/cudos_dashboard_v3
When using the quicksight asset bundle exports APIs that have been recently added, using the following command
aws quicksight start-asset-bundle-export-job  --aws-account-id ************  --asset-bundle-export-job-id my-job-1  --resource-arns arn:aws:quicksight:us-east-1:************:dashboard/my-dashboard  --include-all-dependencies  --export-format QUICKSIGHT_JSON
The asset bundle export job fails, saying "The dashboard has associated errors and cannot be exported. Call DescribeDashboardDefinition to retrieve the errors.".
The describe-dashboard-definition for the dashboard in question says the errors are
"Errors": [         {             "Type": "FILTER_SOURCE_PARAMETER_NOT_FOUND",             "Message": "The filter is using a parameter that has been deleted. Choose a new parameter and save changes.",             "ViolatedEntities": [                 {                     "Path": "filter-group/bd3276ea-9c57-4eeb-aa77-e35c3a1da239/filter/7d4a6fe9-3027-4d65-8693-89d664fc6e73"                 }             ]         },         {             "Type": "DATA_SET_NOT_FOUND",             "Message": "The DataSet that used in the calculated field can not be found in the analysis",             "ViolatedEntities": [                 {                     "Path": "calculated-field/Savings %"                 }             ]         },         {             "Type": "DATA_SET_NOT_FOUND",             "Message": "The DataSet that used in the calculated field can not be found in the analysis",             "ViolatedEntities": [                 {                     "Path": "calculated-field/Cost"                 }             ]         },         {             "Type": "DATA_SET_NOT_FOUND",             "Message": "The DataSet that used in the calculated field can not be found in the analysis",             "ViolatedEntities": [                 {                     "Path": "calculated-field/RI_SP Savings"                 }             ]         },         {             "Type": "DATA_SET_NOT_FOUND",             "Message": "The DataSet that used in the calculated field can not be found in the analysis",             "ViolatedEntities": [                 {                     "Path": "calculated-field/RI_SP Wasted Cost"                 }             ]         },         {             "Type": "DATA_SET_NOT_FOUND",             "Message": "The DataSet that used in the calculated field can not be found in the analysis",             "ViolatedEntities": [                 {                     "Path": "calculated-field/RI_SP Total Savings"                 }             ]         },         {             "Type": "DATA_SET_NOT_FOUND",             "Message": "The DataSet that used in the calculated field can not be found in the analysis",             "ViolatedEntities": [                 {                     "Path": "calculated-field/RI_SP Usage Cost"                 }             ]         },         {             "Type": "DATA_SET_NOT_FOUND",             "Message": "The DataSet that used in the calculated field can not be found in the analysis",             "ViolatedEntities": [                 {                     "Path": "calculated-field/RI_SP Type"                 }             ]         }     ]
Could these errors be resolved so that it is possible to then use the asset bundle export APIs with CUDOS?

Found the duplicate issue which is already closed


Ability to add port details as a label to a link

The plugin is fantastic and i've used it widely over the last few weeks, however it would be hugely beneficial to add a label to the A and Z side of each link to denote the physical port.

Would you mind being more specific by what you mean by this? Do you mean a label visible at all times, or one that just shows up when you hover over the link?

Either would work to be fair. The problem I face with the plugin at the moment is when modelling a large environment it becomes quite cumbersome to quickly work out what physical interfaces a specific link is using.

My preference would be one that is there all the time, but rotated on the link. Here's an example of how I used to do this in the old PHP Weathermap.


fix(zip): handle when unsubscribe is called from within next

Description:

This PR makes sure the error reported here (#6716) doesn't get thrown anymore.
BREAKING CHANGE:  There are no breaking changes.
Related issue (if exists): #6716
I've added a test case which covers the bug but I wasn't sure if there might be a better way to test this. Please let me know if I need to change anything.

Sorry for tagging you @benlesh.  But given that this PR already had it's first anniversary and I rebased it a couple of times I hope it's okay. I would really appreciate if you could take a look. The PR only adds a single character and a test.
Many thanks in advance.

I just faced this issue in my code too and it would be great to have the PR merged 😉


Update README.md

Added steps for compilation after issues arose

Thank you for your contribution. It's a good idea to add instructions directly in this repo. However the ones you provided are only for linux based systems. I think this should be as platform agnostic as possible. Also generating project files directly in the root directory isn't very clean. I would suggest using a procedure like this:
mkdir build
cd build
cmake ..
cmake --build .

and for MSVC users it is necessary to specify the configuration to build because by default it's debug
cmake --build . --config Release

how to run this ım new


Update Neuroimaging sig

Add new member @GuillaumeTh
@netlify /special-interest-groups/neuroimaging

@mashehu I requested a review from you, but was also wondering : is it ok to handle reviews internally in the SIG ? For minor modifications affecting only the neuroimaging SIG pages that is. Just want to get a heads up.

yes, no need to wait for my 👍🏻 on this kind of small changes, but thanks for asking


fix: providers config for redteam when not using OpenAI

I am using Azure OpenAI  and when testing the redteam features I noticed that when passing a different provider for the redteam configuration similar to:
redteam:
  numTests: 1

  provider:
    id: azureopenai:chat:gpt-4o-2024-05-13
    config:
      apiHost:  myendpoint.openai.azure.com

and running redteam generate  I would get an undefined host.
ason: getaddrinfo ENOTFOUND undefined░░░░ 0% | ETA: 0s | 0/3Error in extraction: API call error: Error: Request failed after 4 retries: request to https://undefined/openai/deployments/gpt-4o-2024-05-13/chat/completions?api-version=2023-12-01-preview failed, re
/Users/alexrrr/projects/promptfoo/src/redteam/extraction/util.ts:14
    throw new Error(`Failed to perform extraction: ${error}`);

This is simply of a bug on how loadApiProvider is invoked.
BTW: the option of passing a different provider for redteam is not documented and I only found out when looking at the code. I'll try to add a PR for the documentation when I get a chance later.

not working for actual evaluation, just generation. Closing

Hey @AlexRRR, I think this PR should solve the problem on the eval side: https://github.com/promptfoo/promptfoo/pull/1516


An additional question about the code of this nice work.

Hi, I am very interested in this excellent work but I observe that the code is not available now.
May I ask another question: Did you implement your code based on someone else's code? If so, can you tell me where can I get this base code？
Thank you very much, my best wish.

Sorry for the delay.


Migrate a reasonably-sized application to a Modulith

In order to get some real-world experience, we should migrate a Spring Boot app with more than a couple hundred lines of code to a Modulith. For this to work, the library must have matured a bit and at least a snapshot should be released to Maven Central or JCenter.

I think we can cross this off as we're 10 GA now and there are a couple of projects already using the library, most notably Quarano on the open-source side.


Brand new issue so new!

This is the copy of the new issue.
I've edited it!

hi

this is a comment
This is a new line

A comment I've just written.


JOIN should require ON condition

Describe the bug
SELECT ... FROM l JOIN r is not a valid SQL query, the ON condition should not be optional
Additional context
https://github.com/apache/datafusion/issues/13486

take

Proposed to wait for use case.
https://github.com/apache/datafusion-sqlparser-rs/pull/1552#discussion_r1866317958


[DRAFT] Make HedgedRpcChannel real async

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 35s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 35s
master passed

+1 :green_heart:
checkstyle
1m 37s
master passed

+1 :green_heart:
spotbugs
2m 56s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 21s
the patch passed

-0 :warning:
checkstyle
1m  5s
hbase-server: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
11m 15s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

-1 :x:
spotbugs
1m  9s
hbase-client generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

_ Other Tests _

+1 :green_heart:
asflicense
0m 26s
The patch does not generate ASF License warnings.

36m 46s

Reason
Tests

FindBugs
module:hbase-client

Unread field:HedgedRpcChannel.java:[line 177]

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1601

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux d4a60e9f470f 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 8fceec8cb1

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

spotbugs
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-general-check/output/new-spotbugs-hbase-client.html

Max. process+thread count
94 (vs. ulimit of 12500)

modules
C: hbase-client hbase-server U: .

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  1s
the patch passed

+1 :green_heart:
compile
1m 34s
the patch passed

+1 :green_heart:
javac
1m 34s
the patch passed

+1 :green_heart:
shadedjars
5m 16s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 28s
hbase-client in the patch failed.

-0 :warning:
javadoc
0m 42s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m  8s
hbase-client in the patch passed.

+1 :green_heart:
unit
139m 36s
hbase-server in the patch passed.

168m 15s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1601

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 7556071f1af6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 8fceec8cb1

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/testReport/

Max. process+thread count
3680 (vs. ulimit of 12500)

modules
C: hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 56s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 59s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
6m 30s
master passed

+1 :green_heart:
compile
2m 33s
master passed

+1 :green_heart:
shadedjars
8m 15s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 13s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 54s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
6m 13s
the patch passed

+1 :green_heart:
compile
1m 24s
the patch passed

+1 :green_heart:
javac
1m 24s
the patch passed

+1 :green_heart:
shadedjars
5m 26s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 59s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m  4s
hbase-client in the patch passed.

+1 :green_heart:
unit
186m  5s
hbase-server in the patch passed.

227m 22s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1601

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 8465e2840997 4.15.0-91-generic #92-Ubuntu SMP Fri Feb 28 11:09:48 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 8fceec8cb1

Default Java
1.8.0_232

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/testReport/

Max. process+thread count
3207 (vs. ulimit of 12500)

modules
C: hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1601/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

In general, for the implementation for HedgedRpcChannel, I think a better solution is to wrap several RpcChannels as the callMethod method is asynchronous.

// ** WARN ** This is a blocking call if the underlying connection for the rpc client is
// a blocking implementation (ex: BlockingRpcConnection). That essentially serializes all
// the write calls. Handling blocking connection means that this should be run in a separate
// thread and hence more code complexity. Is it ok to handle only non-blocking connections?

The comment in code is incorrect, even for BlockingRpcConnection, the callMethod is still asynchronous...

BTW, implementing hedged read in rpc framework, it to use a special channel(HedgedRpcChannel) here to wrap multiple RpcChannels, as the RpcChannel provide a callMethod method to do asynchronous call.
But for the protobuf stub, it also has the ability to do asynchronous call(BlockingInterface vs. Interface, newBlockingStub vs. newStub), so wrapping on the stub can also solve the hedged read problem, and this is exactly what we have done for implementing the read replicas feature.
So to be honest, I do not see the necessity to change the wrapping layer from stub to rpc channel...
Thanks.

The comment in code is incorrect, even for BlockingRpcConnection, the callMethod is still asynchronous...

But for the protobuf stub, it also has the ability to do asynchronous call(BlockingInterface vs. Interface, newBlockingStub vs. newStub), so wrapping on the stub can also solve the hedged read problem, and this is exactly what we have done for implementing the read replicas feature.

Ah that makes sense. I missed this part, thanks for pointing it out. I was under the impression that the BlockingRpcClient was always defaulted to a blockingRpcChannel instead of async RpcChannel(). In that case, I think it is much easier to wrap multiple RpcChannels and support both connection types.

It was decided that we will move the logic to MasterRegistry itself. #1593 does that, it has a very similar logic around how it does the group calls. Closing this for now. See the other PR for more context.


Revert "[cinder-csi-plugin] Enable resize e2e tests"

Reverts kubernetes/cloud-provider-openstack#931
As citynetwork cloud doesn’t support cinder online resize feature, disabling the same.

/release-note-none

/assign @lingxiankong

/lgtm
/approve


Assorted ‘except Exception’ related fixes

Merged as the series ending with b7a94be152dc2c1e5aea2babf12e588cbba84fe7, after resolving merge conflicts.  Thanks @andersk!


Update to Go 1.5.1

Update to minor release 1.5.1

Thank you for the contribution. It was merged directly as 194abbf4634343064ee6d6fb908a124957cf813f to keep commit history cleaner. Your contribution is still credited to you.


Example displays base map but not points

mapboxgl 0.10.2 in a Python 3.8.6 kernel in Jupyter:
Code
with open('mapbox_token.txt','r') as f:
    # Must be a public token, starting with `pk`
    token = f.read().strip()

import os

import pandas as pd

from mapboxgl.utils import create_color_stops, df_to_geojson
from mapboxgl.viz import CircleViz

# Load data from sample csv
data_url = 'https://raw.githubusercontent.com/mapbox/mapboxgl-jupyter/master/examples/data/points.csv'
df = pd.read_csv(data_url)

# Create a geojson file export from a Pandas dataframe
df_to_geojson(df, filename='points.geojson',
              properties=['Avg Medicare Payments', 'Avg Covered Charges', 'date'],
              lat='lat', lon='lon', precision=3)

# Generate data breaks and color stops from colorBrewer
color_breaks = [0,10,100,1000,10000]
color_stops = create_color_stops(color_breaks, colors='YlGnBu')

# Create the viz from the dataframe
viz = CircleViz('points.geojson',
                access_token=token,
                height='400px',
                color_property = "Avg Medicare Payments",
                color_stops = color_stops,
                center = (-95, 40),
                zoom = 3,
                below_layer = 'waterway-label'
              )
viz.show()

I expected to see points on the map, like in the screenshot in the docs:

df looks vaguely reasonable / non-empty:

Also note the UserWarning mentioned in https://github.com/mapbox/mapboxgl-jupyter/issues/150#issuecomment-475405241, not sure if it's relevant:
/usr/local/lib/python3.8/site-packages/IPython/core/display.py:717: UserWarning: Consider using IPython.display.IFrame instead
  warnings.warn("Consider using IPython.display.IFrame instead")

What am I missing? Thanks.

examples/notebooks/circle-vector.ipynb is basically the same viz, but constructed differently (it doesn't use df_to_geojson), and it works for me… not sure what the issue is with the df_to_geojson example


support sorting decimal data type

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
part of #122
After some pull request from https://github.com/apache/arrow-datafusion/issues/122, the type of decimal can be used in datafusion.
We need to sort the result by value with the decimal data type. Like that
SELECT col  FROM table_with_decimal_col order by col

The col type is decimal.
Describe the solution you'd like
It's is should be implemented in the kernel of arrow-rs.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

It's resolved in my branch, and will be cherry-picked later

Sorting decimal has been merged, this issue need to be closed.


stage_build_artifacts does not correctly query extraBuildArtifiacts

additional artifacts added during the staging process are stored in spacefx-config.json extraBuildArtifacts.
However, stage_build_artifact.sh does not correctly query the extra build artifacts: https://github.com/microsoft/azure-orbital-space-sdk-setup/blob/30ff9c062a5dd58a1355f9f800bc34e5c2dac088/scripts/stage/stage_build_artifact.sh#L170

@alexlianides tasked with resolving


No attribute `node['to_str']' exists on the current node.

#75 / #76 broke some very simple use cases for me with chef-sugar 2.5.
For example, I had to make this change:
- Chef::Log.debug("redis_discovery found nodes #{redis_nodes.join(', ')}")
+ Chef::Log.debug("redis_discovery found nodes #{redis_nodes.map(&:name).join(', ')}")

To get rid of this exception:
 Failure/Error: ChefSpec::ServerRunner.new(platform: platform, version: version) do |node, server|
     Chef::Node::AttributeDoesNotExistError:
       No attribute `node['to_str']' exists on
       the current node. Please make sure you have spelled everything correctly.

Perhaps this is also a difference between Ruby 2.1.5 and 1.9.3-x branches, but it seems like the exception is misleading at best. I'm not sure what a better solution is, but I thought I'd bring it to your attention.

@martinb3 I don't even see where you are using #deep_fetch! in there... that is just mapping over node objects and relying on their underlying #to_s implementation to home it does the right thing. Your new solution is the proper thing to do. The #to_s implementation might have changed in Ruby 2.1.5 or it's not being implicitly called.
If this continues to be an issue once an official version of Chef that supports 2.1.5 is released, we can revisit it.

@sethvargo OK, thank you for taking a look. I was under the impression 0.0.1 wasn't even a published version of chefspec (the tag does not exist on github). Also just to clarify, I consulted these RFC sections first:

At the time of writing, Ruby 2.1 is the latest Ruby version and 2.0 is the "N-1" version.

https://github.com/opscode/chef-rfc/blob/master/rfc034-ruby-193-eol.md

Chef Client omnibus packages ship with the Latest version of Ruby at the time of a major version bump. Latest & Latest - 1 versions of Ruby are supported. Ruby version bumps only happen at the time of Chef major version bumps.

https://github.com/opscode/chef-rfc/blob/master/rfc015-chef-12.md
I understood those to imply that chef 12 was released while ruby 2.1.5 was already out, so the official version of Chef supports Ruby 2.1 (with point releases).
FWIW, I do expect you'll see more of these. My colleagues and I discovered we had all been pinning chefspec forward without each other knowing it.
Thanks for your time, again; it's very much appreciated!


kstars: 3.6.0 -> 3.6.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/kstars/versions.
meta.description for kstars is: "Virtual planetarium astronomy software"
meta.homepage for kstars is: "https://kde.org/applications/education/org.kde.kstars"
meta.changelog for kstars is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.6.1 with grep in /nix/store/b7j1j0kjwch0wz02kz97wmiw61fz15hk-kstars-3.6.1
found 3.6.1 in filename of file in /nix/store/b7j1j0kjwch0wz02kz97wmiw61fz15hk-kstars-3.6.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
kstars

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/b7j1j0kjwch0wz02kz97wmiw61fz15hk-kstars-3.6.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kstars https://github.com/r-ryantm/nixpkgs/archive/e92749db6ad43eb848b67228a3c2e519f4204698.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/b7j1j0kjwch0wz02kz97wmiw61fz15hk-kstars-3.6.1
ls -la /nix/store/b7j1j0kjwch0wz02kz97wmiw61fz15hk-kstars-3.6.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

kstars

Maintainer pings
cc @TimPut @hjones2199 for testing.

Result of nixpkgs-review pr 199074 run on x86_64-linux 1

1 package built:

kstars


Variants not referencing initial theme

What is the point of the theme if the variants prop doesn't pull from it.
so I have an array of primary colors, and if you do
variant({
   scale: "typography",
   variants: { red: {} }
})

// theme file
{
    colors: {
        primary: ["#FFFFFF", "#480000"]
    }
    typography: {
        red: {
            color: "primary.1"
        }
    }
}

that should set color to #480000 yet it sets it to the string of primary.1

Close was me being an idiot and using a value outside the array.


Yolact++ to onnx？

Has anyone successfully converted a Yolact++ model to onnx?
    preds = net(batch)     dummy_input = Variable(torch.randn(1, 3, 550, 550))     torch.onnx.export(net, dummy_input, "yolact_0421_20000.onnx", verbose=True)
but got the errors, any suggestions would be appreciated.
RuntimeError: Tried to trace <__torch__.yolact.FPN object at 0x9c26df60> but it is not part of the active trace. Modules that are called during a trace must be registered as submodules of the thing being traced.

you need change the
class FPN(ScriptModuleWrapper):

to
class FPN(nn.Module):

this doesn't work for me...did it actually work for u?

您的邮件已收到 谢谢

Any update on this? @zhoujinhai

Any update on this? @zhoujinhai

Yes, It worked for me!

how to do this?

@MiaoRain make use_jit = false in Yolact.py. and  'no_jit': True in data/config.py
Then you will have to make effort to export DCN_v2. For that you can see my comment here

you need to set use_jit=False in yolact.py:
# As of March 10, 2019, Pytorch DataParallel still doesn't support JIT Script Modules use_jit = torch.cuda.device_count() <= 1 use_jit = False  # add this line if u get RuntimeError if not use_jit:     print('Multiple GPUs detected! Turning off JIT.')

you need to set use_jit=False in yolact.py:
use_jit = torch.cuda.device_count() <= 1
use_jit = False  # add this line if u get RuntimeError
if not use_jit:
print('Multiple GPUs detected! Turning off JIT.')

you need to set use_jit=False in yolact.py: use_jit = torch.cuda.device_count() <= 1 use_jit = False # add this line if u get RuntimeError if not use_jit: print('Multiple GPUs detected! Turning off JIT.')

you need to set use_jit=False in yolact.py:
'''use_jit = torch.cuda.device_count() <= 1
use_jit = False # add this line if u get RuntimeError
if not use_jit:
print('Multiple GPUs detected! Turning off JIT.')'''

you need to set use_jit=False in yolact.py:
‘’‘use_jit = torch.cuda.device_count() <= 1
use_jit = False # add this line if u get RuntimeError
if not use_jit:
print('Multiple GPUs detected! Turning off JIT.')’‘’

you need to set use_jit = False in yolact.py:
use_jit = torch.cuda.device_count() <= 1
use_jit = False  # add this line if u get RuntimeError
if not use_jit:
    print('Multiple GPUs detected! Turning off JIT.')


Why the version 1.2.1 is not published yet?

I've tried to follow the README using the version 1.2.1 but sbt can't find it, I've tried to look into sonatype and also in mavenrepository but can't find it either. Why it hasn't been published yet?

1.2.1 is published (http://mvnrepository.com/artifact/org.scorexfoundation/scrypto_2.12/1.2.1) while latest stable version is 1.3.3.

Nice


Direction code breaks if the user selects geometry type as GeoJSON

In the direction & map matching response the Geometry field will be of the type lineString object or Encoded string based on the GeometryType in the request parameter.
This has also been mentioned in the mapbox document
But in the response struct we specify that the geometry field is of type string go-mapbox/lib/directions/types.go
This breaks the code when the GeometryType is lineString .
Steps to reproduce the error

Change line no 37 in go-mapbox/lib/map_matching/map_matching_test.go to opts.SetGeometries(GeometryGeojson) and run the test

How can we overcome this issue? Expecting some suggestions from you @ryankurte

Had a bash at it using interface{} -> map[string]interface{} -> [] interface{} -> float64. It's a bit mad / would love to know of a better way, but seems to work?


allow multiple spaces between alias and flag

Hi Sindre,
Here's a little patch to parse alias / flags with help output like:
-h,   --help          # Print the generator's options and usage

I was trying to get options / arguments from yeoman generator's help output.
Thanks a lot for this little lib, simple but very useful.

Oh, hey Mickael. Haven't seen you in like forever.

I think this is my first pull request since like, well, forever :p

Thanks :)

from
Usage:
  yo trails:model [options] <model-name>

Scaffolds a new model

Options:
  -h,   --help          # Print the generator's options and usage
        --skip-cache    # Do not remember prompt answers             Default: false
        --skip-install  # Do not automatically install dependencies  Default: false

Arguments:
  model-name    Type: String  Required: true

parse-help very helpful :=)
Would you consider the ability to parse commands as well ? In most case, when cli tools define commands, the help output is something like:
Options:
  -h, --help              Lorem Ipsum

Commands:
  install                 Blah Blah

This way, I could automate the completion for pretty much any program that follows this help output.

Yes. That would be useful. PR welcome :)


fix: prevent cng permission pollution

ensurePermission checks if another plugin adds the permission (or if the permission was added in a previous prebuild invocation). Prevents excess churn in AndroidManifest.xml when using an Expo prebuild workflow.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Validator] Impossible one-ways

We should implement impossible one-ways validator in osmlint
Previous http://osmlab.github.io/to-fix/#/admin/deadendoneway
source: keepright
cc @maning @planemad @aaronlidman

Some progress here: https://gist.github.com/anonymous/ab53615d9eb520b7a9a7d6153a2cfee1.
https://github.com/osmlab/osmlint/tree/imposibleoneways/validators/imposibleOneWays
Still there are many false positive.  I will figure out more on this validator, is someone has more ideas , welcome!

Some progress here: https://gist.github.com/anonymous/ab53615d9eb520b7a9a7d6153a2cfee1.
https://github.com/osmlab/osmlint/tree/imposibleoneways/validators/imposibleOneWays
Still there are many false positive.  I will figure out more on this validator, is someone has more ideas , welcome!

Some progress here: https://gist.github.com/anonymous/ab53615d9eb520b7a9a7d6153a2cfee1.
https://github.com/osmlab/osmlint/tree/imposibleoneways/validators/imposibleOneWays
Still there are many false positive.  I will figure out more on this validator, is someone has more ideas , welcome!

done!


Dashboards would be more useful if they could aggregate across instances

First off, kudos on the fantastic dashboards! They're very insightful.
We've observed that the dashboards only presents data per "Application Instance". This setup prevents us from having an overview of metrics across all instances. Is this a design decision or an unavoidable constraint?
Describe the solution you would like to see
It would be tremendously helpful to see aggregated metrics spanning all instances.
How would you expect this feature to work
Maybe the "Application Instance" filter could accept a wildcard ("*") to display metrics from all instances? Or could a toggle switch between individual and aggregated views?
We were expecting the dashboards to display a comprehensive view by default, and then allowing us to drill down further when/if needed.
Additional context
We deploy to Kubernetes pods so the "Application Instance" dropdown fill up with dozens of entries, and each individual instance lives for very short time as new instances appear every time we deploy.

I believe it's as simple as setting this to true in all the dashboard templates
https://github.com/akoutmos/prom_ex/blob/master/priv/phoenix_live_view.json.eex#L1320


Tooltips can get rather wide with a lot of content - perhaps would work better with an (optional?) max width

I'm using tooltips to provide hints for form fields. In some cases, the content can be several sentences or more.
The result is a very wide tooltip. I think it would look better if the width was fixed and text could wrap into several lines.

I think that just adding some w-max-md or something to this line:
https://github.com/themesberg/flowbite-react/blob/c9c257e75f702961739f13fcc926eb58af5e3670/src/lib/theme/default.ts#L888
Should solve the issue. Can someone test it and create a PR?

I tested it, and I didn't find the bug. This was before we enable access to className to the components. If necessary now it is possible to limit the width of the tooltip just by using it.


Block order dependency in brightness block

There appears to be a dependency on block order for the neopixel package set brightness block.
Used on its own the block does not change the brightness of the LED.
If it is followed by a show colour block then the brightness changes.
Additionally it only appears to change the brightness of the pixel that has been coloured.
This script:
https://makecode.microbit.org/_Ra5f7VR9xVXf
shows button A does not change brightness, button B does.
Initially reported to micro:bit support as issue 2678,  Jonny suggested posting this directly here.

@jaustin Tagging you for reference.


Update README.md

added href to text

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@SarthakDandotiya Thank you for submitting a pull request (PR). I have closed the PR given that the link you have added is incorrect and leads to a 404, and there is also a merge conflict. Please kindly resubmit a PR with a tested link if you wish to improve the article again.

Note for collaborators: this PR was not reviewed in order because it was not labelled by the bot.


How Do I get an instance object of codemirror from AwaitableInstance

Hi there, great project you've built here.
I created an ink instance using the following method：
const editor = ink(document.getElementById('editor')!, options)

But how can I obtain the object of the codemirror editor?
For example, I need to call the cm.getDoc().getCursor() method of codemirror to obtain the current cursor position, etc. to use the https://github.com/susisu/mte-kernel provided Table tools
It needs to implement the following interfaces：

So can you tell me how to obtain the instance of codemirror from AwaitableInstance, or is there any other way，look forward your prompt reply.♥

This issue has been resolved through ViewPlugin and Codemirror 6 Migration ,so I closed this issue 🎉


(PUP-7283) Disable CRL checking when fetching the CRL

When we're fetching the CRL for the first time and a CRL hasn't been provided out of band, we need to disable CRL checking until that request has completed.
I've also included a graphical representation of the indirector for anyone interested.

CLA signed by all contributors.

@adrienthebo are you able to do a local acceptance run against this?

@MosesMendoza I've run the tests locally and I don't get the infinite recursion that we did last time. However I was getting a failure on the autosign command tests but that was also failing against the 4.9.4 tag, so I don't know if it's related. I'm going to investigate more and see if the failure is related or due to some precondition that another test was satisfying.

@branan @MosesMendoza I've been getting a lot of failures when running beaker locally, and they generally seem unrelated to the changes I've made. The autosigning failure that I've seen is perplexing but it happens on master and my branch, so I'm baffled. May we try merging this and see if it works? If nothing else the infinite recursion looks fixed.

@adrienthebo sounds good. puppet/master tests are currently red but good to merge when green.

Thank you!

@MosesMendoza it looks like the puppet master component tests are passing, can this get merged today?

Since this has a thumb from @MosesMendoza pending a green master, master is green, and I am also :+1: : I'm hitting the big green button. Nobody can stop me. NOBODY! MUAHAHAHAHAHAHAHA!


Add payment gateway

Please add to XMR payment gateway services : https://www.cdpay.eu/

+merchant
+in progress


Reverse proxy best practice

It seems like a common practice is to place rails behind a reverse proxy. Nginx is smart enough to rewrite "Location" headers during redirects but does not rewrite the "turbolinks-location" header, which breaks turbolinks. I can't seem to find nginx document to rewrite arbitrary headers in a reverse proxy...
What is the recommended course of action?

don't use redirect_to?
don't use a reverse proxy?

I solved this on my own in the following way:
The current hostname under which I host my application is already in another configuration section Rails.application.config.action_mailer.default_url_options[:host]
I added a global after_action :rewrite_turbolinks_header in application_controller.rb with the following contents:
def rewrite_turbolinks_header
  return unless response.headers.key? "Turbolinks-Location"
  uri = Addressable::URI.parse response.headers['Turbolinks-Location']
  uri.scheme = nil
  uri.port = nil
  uri.host = Rails.application.config.action_mailer.default_url_options[:host]
  response.headers['Turbolinks-Location'] = uri.to_s
  return
end

How exactly does this break turbolinks?


Type-annotate files in the cli directory and add them to mypy

This PR adds type-annotations to files in the cli directory and adds them to mypy

Codecov Report

Merging #1256 (7964130) into master (78fee3b) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 7964130 differs from pull request most recent head 1581d6a. Consider uploading reports for the commit 1581d6a to get more accurate results

Additional details and impacted files

Flag
Coverage Δ

packit-e2e
71.78% <100.00%> (-0.01%)
:arrow_down:

testsuite
55.74% <77.27%> (+0.04%)
:arrow_up:

unittests
13.08% <0.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

keylime/cli/options.py
52.94% <100.00%> (ø)

keylime/cli/policies.py
90.36% <100.00%> (+1.47%)
:arrow_up:

keylime/tornado_requests.py
65.00% <0.00%> (-10.00%)
:arrow_down:


non-blocking sampling - need report progress and enable canceling

@WardBrian
Would be great if we could expose a lower level entry point into tinystan so that we can perform sampling in a non-blocking manner and provide real-time updates to the user. Right now the full sampling is done in a single blocking call to _tinystan_sample
Alternatively could we get a callback from _tinystan_sample with the progress? And perhaps if the callback returns false, then it cancels the sampler?
Would be great to be able to hook into mcmc monitor.

I will have to think about this.
I'm not immediately clear on what you mean by a lower-level entry point. Ultimately Stan implements its inference algorithms as fairly monolithic blocking calls, so my first reading is what you're asking for can't be done internally to tinystan.
Re: callbacks -
Stan does natively support a "interrupt" callback. It's what allows Ctrl+C to work inside the algorithms (see tinystan/src/interrupts.hpp). I could have the option to supply one from the outside in environments like this were the default interrupt handler does not make sense.
This callback is called regularly, so it would need to be fast, and it does not receive the current iteration number.
Honestly, so far as I can tell, most progress bars in the Stan ecosystem are based around observing the output text from the sampler, e.g. the stuff currently ending up in the console:  Chain [4] Iteration: 1000 / 2000 [ 50%]  (Warmup)
Here's where cmdstanpy matches that: https://github.com/stan-dev/cmdstanpy/blob/cbea79f3e2f03dfe0f056c30cb613bf30d527750/cmdstanpy/model.py#L2140
The second argument to the StanModel.load interface (currently null in this codebase) is a callback which redirects the output currently going to console.log. For stan-web-demo I implemented this in a way where it soaks up all the output for a run and then calls it, but this was probably over-specialization and real time streaming is a better default.

Okay I need to understand the StanModel.ts better. If you have a chance could you stop by so I can ask some questions about what's happening there?

@WardBrian I did some more testing, and it seems that in order to cancel and update progress on the page, it's going to be necessary to get this into a worker thread, and I think that means we're going to need to change what gets output as .js and .wasm from the compiler. It needs to be suitable for a worker thread. This is too bad since the current .js and .wasm worked so seamlessly. We'll need to discuss.

An alternative would be something like this
https://emscripten.org/docs/porting/asyncify.html

My reading of the asyncify page is it requires inserting things like emscripten_sleep() into the core loops to actually get it to yield, which is not really an option.
em++ does support a --proxy-to-worker flag, but I don't fully understand it's usage.

I think --proxy-to-worker is not ideal.
I've seen several suggestions along the lines of "load your wasm module on the main thread and then ship it over". There are several linked here: https://producement.com/development/using-emscripten-webassembly-in-web-workers-and-interacting-with-native-c-c-code-concurrently/
There are flags --pre-js and --post-js which will insert javascript code before or after the module definition in the generated .js from emscripten. I would be very hesitant to do anything more custom than that, since it would almost certainly break from version to version

I tried a couple things, and I think we'll be able to achieve what we need by popping up a new window on the same domain that shows progress and allows to cancel. I'll work on that next week.

I have a branch at https://github.com/flatironinstitute/stan-playground/tree/worker-thread which moved the sampling off the main thread. It still needs some work to receive the messages back, and it was being done at the same time as your changes to try to report progress so those will need to be given a fresh look after it’s ready

@WardBrian I think what you are trying here is the right approach, if it can work. I gave this a shot but got stuck with the worker thread not supporting "import" and other features of the code generated by the compilation server. Were you able to confirm that the worker actually is able to run without error?

yes, it samples and the UI in the main thread stays responsive. Right now the only way to see the result is in the console but that’s “easy” to fix, I think. I will pick up where I left on on probably Tuesday

This is amazing!
I think it would be good to encapsulate the worker interaction into an interface class. Something like this
import StanWorker from '../tinystan/worker?worker';

type Progress = {
    current: number
    total: number
    // etc
}

class Sampler {
    #isSampling: boolean;
    #onProgress: ((progress: Progress) => void) | undefined;
    constructor(private d: {worker: StanWorker}) {
        this.#isSampling = false;
        // receive messages from worker
        this.d.worker.onMessage((msg: any) => {
            if (msg.type === 'progress') {
                this.#onProgress && this.#onProgress(msg.progress);
            }
        })
    }
    static async create(o: {data: any, compiledModelUrl: string}) {
        const worker = new StanWorker();
        await this.sendRequest({
            type: 'init',
            compiledModelUrl: o.compiledModelUrl,
            data: o.data
        })
        const sampler = new Sampler({worker});
        return sampler;
    }
    async startSampling(onProgress: (progress: Progress) => void) {
        if (this.#isSampling) {
            throw new Error('Sampling already in progress');
        }
        this.#onProgress = onProgress;
        this.#isSampling = true;
        await this.#worker.sendRequest({type: 'startSampling'});
    }
}

I started working on that, but then thought I should let you finish up what you started.


Update templates and write canonical examples.

This change is

Changed the format of the tasks, tutorials, and concepts landing pages according to feedback from jaredbhatti.

@steveperry-53 , is this ready to go live? If so, I'll merge. Or you can. I think you have privileges.

New pull request for this: #1309


(internal) add preview support to tests

Adds preview-image support to Terraform testing pipeline.

TESTS:

controller/node preview
controller/node preview with SSH key

Verified controller and node version in this run. Preview images indeed being used.

regression (public latest)


Replace discover with redfish basic login.

import redfish
from redfish.discovery import discovery
from redfish.rest.v1 import (
    RetriesExhaustedError,
    SessionCreationError,
    InvalidCredentialsError,
)

if __name__ == "__main__":
    obj = None
    try:
        obj = redfish.redfish_client(base_url="{endpoint from ipmitool}", username="{username}", password="{password}", timeout=1, max_retry=3)
        obj.login(auth="basic")
    except (SessionCreationError, InvalidCredentialsError) as e:
        # Can find endpoint but Credential is wrong
        print(e)
    except RetriesExhaustedError as e:
        # Can't find endpoint
        print(e)
    finally:
        if obj:
            # Login success
            obj.logout()

The current discover is not only just finding the local redfish endpoint but also using SSDP to find available redfish services in the same network. Instead of using redfish.discover, we can define a discover function which only try to login the local's BMC with the address found by ipmitool. we can use code above to verify if the redfish is available

This would really help us simplify and avoid the issues with the discover_ssdp() method. Looks like we're essentially merging the functionality for redfish_call_success and redfish_service_available.
Questions:

Is there any specific reason we're passing auth="basic" rather than auth="session" to the login method?
Do you suggest that we also change the hw-verifier detection for redfish so that if RetriesExhausted exception is raised while logging in, then redfish doesn't exist on the machine and we won't enable the collector? But there might be a case where redfish is present but we're not able to login due to some network issue for example and this raises the RetriesExhausted exception.
If that's the case, then would redfish_call_success metric fail only when there is invalid credentials or the HTTP session was not created?

The only sad part is that we spent quite a bit of time optimising and refactoring the discover code (cache, staticmethod etc.)


fishbc data not up to date

Issue:
While building the fiss_species_table.csv in extract-fiss-species-table.R I noticed that the results from fishbc::cdc are not up to date with https://species-registry.canada.ca/index-en.html#/species?ranges=British Columbia&sortBy=commonNameSort&sortDirection=asc&pageSize=10&keywords=sockeye.
Example:
For example the Sockeye Salmon - Francois-Fraser-S Population is listed as a "Special concern" with COSEWIC but that does not show in the results form fishbc::cdc.

Looks like fishbc pulls from BC Conservation Data Centre (CDC) and which is also up to date with Sockeye Salmon - Francois-Fraser-S Population listed as a "Special concern" .

This is also true for the following populations in this projects scope (use the Nechako river):

Sockeye Salmon - Takla-Trembleur-EStu Population https://species-registry.canada.ca/index-en.html#/species/1392-989
Sockeye Salmon - Takla-Trembleur-Stuart-S Population https://species-registry.canada.ca/index-en.html#/species/1393-1004

What I found:
Looks like the the cdc.csv provides the data (code here) has not been updated for 4 years... https://github.com/poissonconsulting/fishbc/blob/main/data-raw/cdc/cdc.csv
I could be totally wrong but it seems odd that this csv hasn't been updated in 4 years...

Put a bunch of time into the issue and I've got it all working except for one step.

https://github.com/lucy-schick/fishbc/blob/main/R/cdc.Rmd contains all the current instructions to update the cdc.csv. I say current because hopefully in the future we may be able to pull directly from the API or find a better way to do this. There was still a significant amount of data wrangling to do.

all the updated csv's and rdata are also in my fork, https://github.com/lucy-schick/fishbc/tree/main/data-raw

STILL TO DO: actually run fishbc::cdc and get the updated data. I've run:

> usethis::use_data(cdc, overwrite = TRUE)
✔ Setting active project to '/Users/lucyschick/Projects/repo/fishbc'
✔ Saving 'cdc' to 'data/cdc.rda'
• Document your data (see 'https://r-pkgs.org/data.html')

Then
fishbc::cdc

but it still returns the old csv. I've tried restarting R and Rstudio, same issue. I've also tried running it in a different repo and still the same issue. There's a good chance I may be just doing something wrong so any ideas are welcome!

Thought I found my issue but now not so much
When using fishbc::  it uses the old data. This makes sense I guess since I am explicitly calling the package? I think. but if I just call cdc it returns the new data, very obviously since that is an object.
But, Is there a way to call  fishbc::cdc and it return the updated data? Yes I think so but I need to build the package locally to do so I think(using devtools like we did for local fpr functions). I'm thought that running usethis::use_data(cdc, overwrite = TRUE) would use the updated data when calling the package but I guess not...
Now the next question: how do we use the updated data in other projects? Options:

Build package locally so we can call  fishbc::cdc and it uses the updated data OR

read in the updated cdc file in each project. scripts/02_reporting/extract-fiss-species-table.R creates the fish species table and burns it to 'data/inputs_extracted/fiss_species_table.csv' so not the end of the world to read in the cdc file because we only do this step once.

Did you try "Build / Install / Clean and Install" in the Rstudio IDE by chance?

Build / Test first can be a good move too
Once it's loaded library(that_pkg) will load that installed package.
You can add issues to your fork of a repo using settings in github.  Prob not a bad idea to do that and point to this one.

another way to do it is to push branch to github and load with pak similiar to what we do with rbbt
pak::pkg_install("NewGraphEnvironment/rbbt@bbt_bib_result")  - bbt_bib_result is the name of the branch

Build problem fixed by using "Build / Install / Clean and Install" in the Rstudio IDE, thanks!
I realized the updated data was missing the dates in the COSEWIC and SARA columns so I went to go fix that and realzed that the Summary Data download form the cdc has all the info we need in one place to update cdc.csv (instead of having to join other downloaded results together) which is awesome but also needed a bit of work.
Everything is good to go now (finallyyy). I would be happy to make a PR to fishbc with my branch but maybe we could review what I've done so that it's in the best format for them. I'm not sure if they will want an Rmd file but I did my best to explain the whole updating process so that it will be quick in the future but not sure if thats what they will want... we can deal will this later once we have more time. For now I will just update the fish species tables in the reports.
Link to fishbc issue https://github.com/poissonconsulting/fishbc/issues/13


A change to libvirt_ignition should result in libvirt_domain being recreated

System Information
Arch Linux toolbox
Linux distribution
cat /etc/os-release
NAME=Fedora
VERSION="34.20210516.0 (Silverblue)"
ID=fedora
VERSION_ID=34
VERSION_CODENAME=""
PLATFORM_ID="platform:f34"
PRETTY_NAME="Fedora 34.20210516.0 (Silverblue)"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:34"
HOME_URL="https://fedoraproject.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora-silverblue/"
SUPPORT_URL="https://fedoraproject.org/wiki/Communicating_and_getting_help"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=34
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=34
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="Silverblue"
VARIANT_ID=silverblue
OSTREE_VERSION='34.20210516.0'

Terraform version
Terraform v0.15.4
on linux_amd64
+ provider registry.terraform.io/carlpett/sops v0.6.2
+ provider registry.terraform.io/cloudflare/cloudflare v2.21.0
+ provider registry.terraform.io/dmacvicar/libvirt v0.6.3
+ provider registry.terraform.io/poseidon/ct v0.8.0

Provider and libvirt versions
v0.6.3

Checklist

[ ] Is your issue/contribution related with enabling some setting/option exposed by libvirt that the plugin does not yet support, or requires changing/extending the provider terraform schema?

[ ] Make sure you explain why this option is important to you, why it should be important to everyone. Describe your use-case with detail and provide examples where possible.
[ ] If it is a very special case, consider using the XSLT support in the provider to tweak the definition instead of opening an issue
[ ] Maintainers do not have expertise in every libvirt setting, so please, describe the feature and how it is used. Link to the appropriate documentation

[X] Is it a bug or something that does not work as expected? Please make sure you fill the version information below:

Description of Issue/Question
When performing CRUD on a ignition file the libvirt domain needs to be recycled. Currently all we do is update the file in place.
Setup
https://github.com/anthr76/kubic-terraform-kvm/blob/02c22a816cca1e9815ef97b4842d797d4b7f3a65/masters.tf#L66-L70
Steps to Reproduce Issue
https://github.com/anthr76/kubic-terraform-kvm/blob/02c22a816cca1e9815ef97b4842d797d4b7f3a65/masters.tf#L66-L70
Changes here result in the file being updated but the domain not changing
I have tested this against kubic, and Fedora CoreOS
Per https://coreos.github.io/ignition/rationale/#ignition-runs-only-on-the-first-boot we should be recycling the resources of the updated ignition

I think this is related to #931 even though that issue is about cloud init


Gracefully handle search index exceptions if possible

Q
A

Type
bug

BC Break
no

Fixed issues
https://github.com/doctrine/mongodb-odm/issues/2634#issuecomment-2239054724

Summary
#2630 introduces support for managing search indexes in the schema manager. However, the logic does not correctly handle cases when the server does not support search indexes. This PR adds more graceful handling in cases where no search index changes are necessary. Here's a list of the changes:

creating search indexes: no changes; if search indexes are declared on a class and the command to create them fails, we expect an exception to be thrown
updating search indexes: if no search indexes are declared on a class, failure of the listSearchIndexes command will no longer cause an exception. If a class has search indexes, the old behaviour still applies
deleting search indexes: failure of the listSearchIndexes command no longer throws an exception, since there are no search indexes to be removed.

We specifically check for error code 115 (CommandNotSupported) and check the exception message to make sure that it's actually an issue with the search index command, as there are no good ways to check for search index support.

hi @alcaeus, I would be happy to help, but it seems I cannot test your PR since my app requires doctrine/mongodb-odm-bundle which depends on this lib. So when i add your repo as a new vcs repository in composer.json, composer does not accept to update to dev-fix-search-indexes:
  Problem 1
    - Root composer.json requires doctrine/mongodb-odm dev-fix-search-indexes, found doctrine/mongodb-odm[dev-fix-search-indexes] but these were not loaded, likely because it conflicts with another require.
  Problem 2
    - doctrine/mongodb-odm-bundle is locked to version 5.0.1 and an update of this package was not requested.
    - doctrine/mongodb-odm-bundle 5.0.1 requires doctrine/mongodb-odm ^2.6 -> satisfiable by doctrine/mongodb-odm[2.6.0, ..., 2.9.x-dev] from composer repo (https://repo.packagist.com/arte) but doctrine/mongodb-odm[dev-fix-xml-mapping-empty-array, dev-fix-search-indexes, dev-drop-bc-issue-type, 1.0.0-BETA4, ..., 1.3.x-dev, 2.0.0-alpha1, ..., 2.0.x-dev] from vcs repo (github https://github.com/alcaeus/mongodb-odm) has higher repository priority. The packages from the higher priority repository do not match your constraint and are therefore not installable. That repository is canonical so the lower priority repo's packages are not installable. See https://getcomposer.org/repoprio for details and assistance.

So when i add your repo as a new vcs repository in composer.json, composer does not accept to update to dev-fix-search-indexes:

Ah, it's been a while. You can use an inline alias to require the package:
doctrine/mongodb-odm@dev-fix-search-indexes as 2.8.1-dev
That should fix the issue you encountered.

hey thanks, I just learnt something! :blush:
Now I have required your PR, I still have the same error:
➜ bin/console doctrine:mongodb:schema:update -n
Updated indexes for all classes
Updated validation for all classes
Unrecognized pipeline stage name: '$listSearchIndexes'

with exit code 255

BTW, we're using mongo 4.4

Ah, that's probably the reason why you're getting a different message. Thanks for taking a look, I'll update this PR after some more testing.

@nikophil I've updated the code to also handle server versions where $listSearchIndexes doesn't exist (tested with 4.4.29 on my machine). The error message is consistent, so this should hopefully handle all kinds of server versions.

With 2.8.1 this is still causing us a problem in TheBigBrainsCompany/TbbcMoneyBundle.
I think the circumstances are an index exists (not a search index), then the schema is dropped.
It tries to run a $listSearchIndexes pipeline, which errors (255).
We are dropping the schema with doctrine:mongodb:schema:drop (which I believe comes from mongodb-odm-bundle).

@mustanggb I just ran the tests for TbbcMoneyBundle locally. With MongoDB 7.0.9, I was able to reproduce the failures using ODM 2.8.0, but using 2.8.1 fixes the tests. Removing the constraint in #180 works on my machine - could you please give it another look and share the exception if one happens? I noticed that the command invocation uses a NullOutput, which makes debugging much more difficult than it should be.

Looks like it's saying $listSearchIndexes stage is only allowed on MongoDB Atlas.
https://github.com/mustanggb/TbbcMoneyBundle/actions/runs/10579960710/job/29313659438#step:8:1123

For me also the error is $listSearchIndexes stage is only allowed on MongoDB Atlas

I've released 2.8.2 with a fix for this. Sorry for the delay here.

Confirming this is working for us now.


Provide drop index commands in loc_scan_missing_dirs.md for indexes that show up.

Can you provide DROP INDEX commands instead of DROP TABLE in loc_scan_missing_dirs.md for indexes that show up.

Since index is not available in CDP, customers want to clean up indices.


Please enable Snippets package for Docblockr to function properly.

Please enable Snippets package for Docblockr to function properly.

This means you have to activate the snippet package. The snippet package is used to insert the documentation see: #217.


Remove unused dev gems: aruba & cucumber

I can't see any reference to those testing gems. Tests are still passing. Maybe they have been used in the past.

Nice! Since the tests still pass, this looks good to me.

Merged in https://github.com/thoughtbot/suspenders/commit/61f193d78d0e4b2c86bbe9c050a9d252ed5d85ad. Thanks!


Avoid UB due to uninitialized memory in serde_at

mem::MaybeUninit::uninit().assume_init() is almost always UB.
(Caught by Miri)

Awesome! Nice catch!
Thank you!


Change RegisteredClient to an interface

Hey guys!
It would be nice if the RegisteredClient class were an interface. Something like Spring Security's UserDetails interface.
It would be easy to implement JPA based clients, or use another persistence frameworks or databases like DynamoDB, Redis, etc.
Thanks!

Closing as duplicate of gh-80


move api config structs from daemon to api/types

ContainerLogsConfig
ContainerStatsConfig
ContainersConfig
ContainerAttachWithLogsConfig
ContainerWsAttachWithLogsConfig

Signed-off-by: Morgan Bauer<EMAIL_ADDRESS>No more dependency on daemon in a couple of places.
Might confict a little bit with #18943 on the merge.
@calavera @duglin

I can see another config for Exec as well.

I'm gonna let @cr7pt0gr4ph7 rebase his #19133 and close this.


[T4A2][T2] Kevin Lam

Ready for review

@kevinLamKB Some comments added. Please close the PR after reading them.


Cleanup DAPS references

WHAT
Currently the repository is full of documentation/modules/tests related to DAPS, but it is not in use since version 0.4.x.
Let's :axe: them.
WHY
cleanup, avoid confusion
HOW
// if possible, outlines a solution proposal
FURTHER NOTES
// anything else you want to outline
Please be sure to take a look at
our contribution guidelines and
our PR etiquette.

:axe:


Centralized interactive input() function that checks if STDIN is a TTY

Promote _prompting to prompting
Use prompt() to check TTY before getting user input

Closes https://github.com/Azure/azure-cli/issues/1349

Can we have a wrapper on getpass that confirms the typed password? I'd rather do that than roll my own in VM/VMSS create.

User should always type the password twice?

I would argue that yes, the user should always enter the password twice, but you could parameterize it with a default of False. The idea is that 1) the portal and like 99% of regular applications do this and 2) it would be very annoying to mistype your password and find you have no ability to access the VM you just waited to create because of a password typo. This at least ensures you got your password right or made the same mistake twice (in which case, no one can help you!)

@tjprescott Added prompt_pass with confirmation option.

Thanks @derekbekoe !


[QUESTION] Is it possible to search for only a part of a word/affix

Is it possible to use c.Search (or Twint in any other way) to search for only a part of a word or an affix? For example I would use c.Search = 'foo', and Twint would return tweets containing 'foo', but also tweets containing 'foobar', 'foot', fool', 'food', etc. Or I would search for 'cat' and Twint would return tweets containing 'cat', 'catlady', 'autocat', etc.
If I'm not mistaking, you can't use regex in Twint?
Thanks in advance for the answer

Hi @ElizabethCappon
you can't use regex in Twint because it uses the search field of the Twitter Advanced Search and Twitter does not allow regex search


Tests & cleanup

add tests for errors
remove dead code

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+9.5%) to 93.939%

Totals

Change from base Build<PHONE_NUMBER>:
9.5%

Covered Lines:
217

Relevant Lines:
231

💛  - Coveralls


Captum Insights for Image Classsification with IG and Deconvolution

Captum Insights for IG and Deconvolution

Thanks, looks good to me.


Report on Lion Optimizer

First run looks very promising although I implemented it myself but it basically near identical to what you did. I just divided the learn rate by 3 though.
But so far it seems to converge nicely and probably faster too than with AdamW. With AdamW I actually had an issue recently when I changed the linear attention to full attention on the two coarsest layers in down and up sample branch of the UNet. There I had to restart training twice because it got stuck with NaNs

@Mut1nyJD nice! so in the paper, in the limitations section, they said it didn't really help improve anything for Imagen, something noteworthy
in my mind, if this technique is not worse than adam, it is worth using as it is simpler, slightly more memory efficient. i think this technique should also require a learning rate cooldown too.

But so far it seems to converge nicely and probably faster too than with AdamW. With AdamW I actually had an issue recently when I changed the linear attention to full attention on the two coarsest layers in down and up sample branch of the UNet. There I had to restart training twice because it got stuck with NaNs

are you saying that you are seeing anecdotally more stability during training?

@lucidrains

are you saying that you are seeing anecdotally more stability during training?

Yes but I need a slightly longer run and I've got only one sample point so not enough for a definitive conclusion

@Mut1nyJD thanks for sharing!

Emm, for me, lion works very bad when I directly replace adamw with lion, the performance degrades significantly.

@Zhaoyi-Yan how high is your batch size and what was your learning rate?

It is the standard lr divided by 3 which is probably 1:3e-05 and batchsize is 56 I definitely see a faster convergence will post samples when I am back from holiday

@Mut1nyJD some people are reporting that although it has faster convergence, it does not generalize as well. would be curious about what your final FID scores are

@Mut1nyJD some people are reporting that although it has faster convergence, it does not generalize as well. would be curious about what your final FID scores are

So I am back first numbers also show better FID scores than AdamW in my generated samples but I have to do a more excessive test on later checkpoints with AdamW I guess as Lion converges quicker. But after 100k iterations Lion optimizer wins with a 20 point margin.


"An unexpected error occurred"

Not a recurring problem for me; just reporting the bug to give you data.
I was opening looking at prompts.  I'd just evaluated LLMPrompt["KnowAboutMe"], and after the resource was fetched, the resulting TemplateObject appeared in the Out[2] field.  I clicked on the + icon to display the content, and that's when the error came up.  There was very little time between these events; I suppose I may have misclicked something, but can't imagine what.
The error said it happened during evaluation of In[3], which I hadn't entered yet.  (I'd only entered LLMPrompt["HaikuStyled"] and LLMPrompt["KnowAboutMe"] so far.)

Debug Data

Property
Value

Name
"Wolfram/Chatbook"

Version
"1.0.0"

EvaluationEnvironment
"Session"

FrontEndVersion
"Desktop: 13.3 for Linux x86 (64-bit) (June 3, 2023)"

KernelVersion
"13.3.0 for Linux x86 (64-bit) (June 3, 2023)"

SystemID
"Linux-x86-64"

Notebooks
True

DynamicEvaluation
False

SynchronousEvaluation
False

TaskEvaluation
False

Settings
Inherited

Stack Data
throwFailure
trimURL

Failure Data
<|
  "Evaluation" :> Wolfram`Chatbook`Actions`Private`writeChunk[Dynamic[Wolfram`Chatbook`Actions`Private`container$14877], CellObject["29ae40d0-14a9-482d-ae1a-ce25f8e4ae73", "068ff4a8-f6ca-4437-bcbe-b0a4edb6937f"], ",\"finish_reason\":null}]}\n\n"]
  "Arguments" -> {}
|>

Duplicate of #210


Fix message when GEI_SKIP_VERSION_CHECK is turned on

Whilst testing the new feature introduced in #1077 to skip the "latest version" check, I spotted that the log message we emit when the check is skipped is incorrect. This fixes it.

I just built releases and tested this end to end locally, and this is all looking good ❤ Once the tests pass, I'll merge and cut a new release, v1.6.0.


Move from DuckDB to Snowflake

@edsioufi - did you mean to PR this? I do have some thoughts on how to land this project on snowflake and it would be to do that as part of the branch related to #102

Oops sorry @matsonj this was opened by mistake! It was work meant for my forked repo only. Happy to help with SF migration at a later stage though.


Chain Abstraction Template

Vision doc: https://www.notion.so/connext/Chain-Abstraction-761e0445ebf74565a3a8aaaeb6a949b0
Goal is to create reusable contracts+SDK+docs for simple and easy chain abstraction integrations

1- UI Example
We are evaluating building a barebones example for devs, since integrations have struggled on how to design cross-chain UX, there are gaps to fix, and we should overall be leading this vision of improving UX
Spec https://www.notion.so/connext/Spec-Chain-Abstraction-UI-Template-aff9ad1877c54e12a2a4edca407c1686
Epic  https://app.zenhub.com/workspaces/connext-6272a34e0368a200141cd949/issues/gh/connext/monorepo/4249
Wireframe  https://lucid.app/lucidchart/f9542edb-092d-498c-a8c0-140bf3312beb/edit?viewport_loc=-120%2C-226%2C4608%2C2340%2C0_0&invitationId=inv_06307e91-3cdc-4511-b590-e3bce34906c4

Not deploying Mean.
Will pick this up later.


RPC: Add merkle tree and coinbase transaction to the protx diff rpc command

Added merkle tree and coinbase transaction to the protx diff rpc command by addding cbTxMerkleTree and cbTx fields to CSimplifiedMNListDiff#ToJson

Hmm, win32 unit tests are crashing. Not sure if related to this PR (I assume not)


[Bug]: Failed to apply for permissions on a logical table

ODC version
4.3.2
OB version
no related
What happened?

In the ticket, select a schema to apply for a table in the schema, click Submit, and an error message is reported
What did you expect to happen?
The application logic table ticket is successfully created
How can we reproduce it (as minimally and precisely as possible)?
In the ticket, select a schema to apply for a table in the schema, click Submit, and an error message is reported
Anything else we need to know?
No response

Fixed In Commit fcd5430ebbd9cf2f6445dee12800f2f3d181ff22
------
畅晚
Fri Sep 20 15:57:39 2024 +0800

pass


download skip all adult mags

The download feature automatically skip all adult mags.
Perhaps it didn't got list of those title because all zinio platform had default settings to hide adult content.

Unfortunately, it's not possible to download adult magazines using the Zinio mobile API (see this for more details). Desktop and web application allow them, but using the different API, which my tool doesn't support.


self signed certificate in certificate chain

When i call
YD.on("error", function(error) {
console.log("erro aqui",error);
});
returning this error: self signed certificate in certificate chain

Seems to be a local issue. What exactly are you trying to achieve?

When i try, is returning this error:  UnhandledPromiseRejectionWarning: Error: self signed certificate in certificate chain

@mouragilvan I assume you are behind a company proxy that transparently decrypts and re-encrypts all web traffic. You can tell by looking at the certificate chain when you navigate to pages like https://google.com or https://cnn.com. Chances are, the root certificate at the top of the chain is a CA cert self-signed by your company, or by one of the web monitoring providers like goskope.
Take a look at this SO posting for a possible solution to your problem: https://stackoverflow.com/questions/29283040/how-to-add-custom-certificate-authority-ca-to-nodejs


Introduce "SendType" for Message and Signal.

While parsing a DBC file at my work using this parser library I realized that this library doesn't support parsing of "SendType" for both Signal and Message. For our particular project we are using this information from the DBC file. So, I modified this library a bit and added a "SendType Parser" that gives the assigned "SendType" information for each Message and Signal otherwise it will return default value as "Cyclic".
The available send types are defined in the DBC at:
BA_DEF_ BO_  "GenMsgSendType" ENUM  "Cyclic","Event","NotUsed","NotUsed","NotUsed","NotUsed","NotUsed","IfActive","NoMsgSendType","NotUsed","NotUsed","NotUsed","vector_leerstring";
These can change from DBC to DBC for example the following is also possible:
"cyclic","triggered","cyclicIfActive","cyclicAndTriggered","cyclicIfActiveAndTriggered","none"
The Message "Send Type" can be found here:
BA_ "GenMsgSendType" BO_ 1424 7;
Where "BO_ 1424" is the id of the message and "7" is the Enum value
If a message has no definition of BA_ "GenMsgSendType" BO_ XXXX X  we should use the default value which is defined here:
BA_DEF_DEF_  "GenMsgSendType" "Cyclic";
Also each signal can also have a "SendType" Information that can be found here:
BA_DEF_ SG_  "GenSigSendType" ENUM  "Cyclic","OnWrite","OnWriteWithRepetition","OnChange","OnChangeWithRepetition","IfActive","IfActiveWithRepetition","NoSigSendType","OnChangeAndIfActive","OnChangeAndIfActiveWithRepetition","NotUsed","NotUsed","NotUsed";
BA_ "GenSigSendType" SG_ 1424 SeatControlCommand1 8;
SeatControlCommand1 is the Signal Name and "8" is the ENUM value.
If this entry is not present the default value shall be used:
BA_DEF_DEF_  "GenSigSendType" "Cyclic";

Hi,
did you have a look into the CustomProperites map available on both Signal and Message class?
Can you confirm that you can find it there?
Thanks
A

@Usamaahmad751 any news on this? Did CustomProperties answered your questions?
Please let us know, or give some more feedback, otherwise I will close this issue,
Cheers
A


Expression to calculate the angle between two vectors

Hi,
In your answer to my tweet you asked me to open an issue. So here it is.
Original message:
Hi WindowsUI, do you by chance have a ready to use example to calculate the angle between two Vectors using an ExpressionAnimation?

Hey @mgutekunst, thanks for reaching out with your question! We do not have a sample that explicitly does what your asking for, but let me try provide you all the stuff you need. Out of curiosity, what are you planning on using this operation for?
The general formula for calculating the angle between two vectors A and B is:
Cos(θ)=( (A ∙B) ) / ( ||A|| * ||B|| ), where A ∙B is the dot product and ||A|| and ||B|| is the length of each vector (Not sure if you're already aware of the formula but in case you are not :) ). One thing to note is that we do not have an inbox Dot Product function for Expressions (it's on our To-Do list for the future), but we do have length and ArcCosine. So you will need to calculate the Dot Product manually where Dot(A,B) = (A.X * B.X) + (A.Y*B.Y) + (A.Z * B.Z).
Thus when translating this into an Expression animation, simply take the ArcCos of the dot product divided by the product of the lengths - this is what you will input into Expression string portion:
ACos( ( (A.X * B.X) + (A.Y*B.Y) + (A.Z * B.Z) ) / ( Length(A) * Length(B) ) ). The output of this expression will be the angle between the vectors A and B in radians.
Hope this helps!

Hi @patsiu,
thanks for answering. In fact I already tried the formula you described but it's not working in any scenario.
I've got several randomly moving ellipses (they're moving using an Scalar3Animation on their Offset) which I need to link for a specific period of time while they're moving.
I was able to create a line sprite whose length is calculated using an expression. Also the line is sticked to one of the two connected ellipses using another expression animation. To actually connect two ellipses I need to change the angle of the created line and that's where I need your help.
My current solution uses a DispatcherTimer where I calculate the ange using Atan2 which works somehow. Unfortunately there's no expression function for Atan2.
I probably need to take the relative positions of the ellipses to each other into account but I'm a bit lost here. I've added a screenshot to show my current problem.

Hey @mgutekunst, I'm trying to piece together what you're trying to do, so I have a few questions:

Overall, are you simply trying to assign the angle between the two Ellipses Offset values to the rotation angle of the line sprite?
You mentioned you tried the formula - I assume you attached it to the Rotation angle property of the line sprite with an expression? When you say it didn't work, what was the outputted behavior of the Expression Animation?
Visually, how is the line connected with the Ellipses? Is it connected at a specific point? Sit tangentially between each Sprite?

Hi,
thanks for your help so far.

yes, thats exactly what I'm trying to do.
Yes I used an Expression animation to set the angle of the line. Here's the code that's creating the animation

var rotationAnimation = _compositor.CreateExpressionAnimation();
rotationAnimation.SetReferenceParameter("A", visual1);
rotationAnimation.SetReferenceParameter("B", visual2);

rotationAnimation.Expression = "ACos( ( (A.Offset.X * B.Offset.X) + (A.Offset.Y*B.Offset.Y) + (A.Offset.Z * B.Offset.Z) ) / ( Length(A.Offset) * Length(B.Offset) ) )";

_line.StartAnimation(nameof(_line.RotationAngle), rotationAnimation);

The problem here is that the line as always drawn downwards and does not take the relative positions of the ellipses to each other into account. For example visual1 could be above/below and left/right of visual 2
3. the line should connect the ellipses centers but that should be possible by simply adding the radius of the ellipses to the Offset in the above formula. Or am I mistaken?

Gotcha - thanks for clarifying.
A few things that I'm seeing:

The equation above will rotate the line visual when the offset of A or B changes - so I haven't been able to repro what you've been seeing where the line doesn't move at all? On the side, how are the visuals (A/B) being moved?
Also as the ellipses move around, how do you ensure the length of the line remains long enough to connect the two centers?
The equation also won't rotate the line Visual correctly when the offsets of A and B change -  although visually the ellipses may be separated by a particular angle, their vector forms tells a different story. In one example, say A and B are (2, 2) and (2, -2) respectively, basically separated by a height of 4. If you plot out the vectors, they are separated by 90 degrees, so in this case, the line would rotate correctly. However, if A and B are (2, 2) and (2, 6) respectively, it looks the same as the first example, just translated up. However, the angle between these two vectors are now very different (~30 degrees).

Let me take some time and dig around to see if there is a way for you to do this type of scenario with Expressions. Stay tuned.

Thanks for putting so much effort into this.

My line moves as well if i apply the expression to the Rotate of the visual but it's not connecting the ellipses. The visuals (A/B) are moved using a Vector3Animation that is applied to the Offset. Here's the code:

var anim = compositor.CreateVector3KeyFrameAnimation();
anim.InsertKeyFrame(1f, new V(_rnd.Next(0, (int)this.ActualWidth-50), _rnd.Next(0, (int)this.ActualHeight-50), 0));
anim.Duration = TimeSpan.FromSeconds(_rnd.Next(5,10));

ElementCompositionPreview.GetElementVisual(comp).StartAnimation("Offset", anim);

This is repeated when the animation finished. To achieve this, i used a compositor.CreateScopedBatch()

To ensure the correct length of the line I use an expression animation and apply it to the size.

var sizeAnim = _compositor.CreateExpressionAnimation();
sizeAnim.SetReferenceParameter("P1", visual1);
sizeAnim.SetReferenceParameter("P2", visual2);
sizeAnim.SetVector3Parameter("Size", new V((e1.Width / 2), (e1.Height / 2), 1));

sizeAnim.Expression = "Vector2(2,Distance(P1.Offset+Size,P2.Offset+Size))";
_line.StartAnimation("Size", sizeAnim);

Don't get irritated by the class V. I created a wrapper class because I always got the wrong Vector3 from IntelliSense. Also I hat to manually cast all double values to float. This is now wrapped inside this class.

I'm closing this thread due to inactivity. Please reopen if this is still a pressing concern or a blocking issue for your app.


Flip the Switch to Install the Catalog with the Go CLI

Use the Go CLI to install the default catalog instead of the Python CLI

IPG 306.

IPG 320.

IPG 341.

IPG 341

Don't merge yet.

We need to test that params and annotations have the expected type. :)

Parameters are fine. The annotations will be the issue.

Opened https://github.com/openwhisk/openwhisk/issues/952 for issue with annotations

Depends on https://github.com/openwhisk/openwhisk/pull/953.

@dubeejw is this wip?

This was blocked and now pending new pg.

@rabbah, @csantanapr, PG1 128. Should be ready for merge now.

PG1 133.

PG1-133 approved. Feeling lucky?

Merged as https://github.com/openwhisk/openwhisk/commit/b7f8c330b63cfd795e03de328a6082acfad9cc93.


name 'ct' is not defined when trying to run CheckmateTactic

Hello,
I am trying to run the policy train and testing with Checkmate Tactics. Unfortunately I get:
name "ct" is not defined at line 189 in pipeline.py

Fixed by adding in env_settings.py  import as ct

Thanks very much! Merged: https://github.com/tomsilver/policies_logic_programs/commit/232ba1ca66d38cf211f7e3b33ce47798841c2bcc


Atoms processing

@rultor merge

@rultor merge

@olesiasub Done! FYI, the full log is here (took me 6min)


Das_Geek: Watch sharktankpedia.net

Das_Geek requests the watch of the watch_keyword sharktankpedia\.net. See the MS search here and the Stack Exchange search here.

sharktankpedia\.net has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Machavity in Charcoal HQ


Introduce type alias for Sql[ZResultSet]

https://github.com/zio/zio-jdbc/issues/19

Thank you! Will merge if the tests pass. 🤞


doc: revise doc of registry.Reference

Closes #332

@nima I've updated PR title and description for you. You may follow the same pattern for future PRs.

Thank @nima for contributing. The doc will be auto updated soon at https://pkg.go.dev/oras.land/oras-go/v2@main/registry#Reference


Improve maintainability and robustness

Added typing information to dbcfeeder to make code easier to read
and to allow more meaningful linting.
Also refactored some very large functions into multiple smaller
functions to improve readability and separation of concern.

Looks good to me, any preferences to whether this should be merged before or after the other PR @sophokles73? I have not checked if merge conflicts are expected after the first one has been merged.

this should be merged right away FMPOV


Object Dot notation doesn't transpile when bundling for production

When bundling for production I'm getting an error when using dot notation over bracket notation whilst mutating an object:
Property left of AssignmentExpression expected node to be of a type ["LVal"] but instead got "StringLiteral"
Example:
    Platform['OS'] = platform.OS; // works
    Platform.OS = platform.OS; // doesn't work

This can be reproduced with the following steps:
> react-native init my-project
# ...
> cd my-project
> npm install react-primitives --save

# modify index.ios.js to: import { Text } from react-primitives;

node ./node_modules/react-native/local-cli/cli.js bundle --entry-file index.ios.js --platform ios --dev false --reset-cache --bundle-output /tmp/main.jsbundle --assets-dest /tmp/

I am tracking this issue on react-primitives here: https://github.com/lelandrichardson/react-primitives/issues/79

This seems like it would make for a good PR -- it's well-defined and fairly clear what the unit tests should look like. This is the relevant bit of code: https://github.com/facebook/metro-bundler/blob/3cbc2f3ec4ed4c92d2c7da5dc2a4e7a9de2ee3d5/packages/metro-bundler/src/JSTransformer/worker/inline.js#L118

I opened a PR for this. It's a pretty minimal change, but as far as I can tell it should cover the edge cases to solve this and the worked-around https://github.com/facebook/react-native/issues/7607#issuecomment-221425153
https://github.com/facebook/metro-bundler/pull/45

Thanks for the PR.


The project now compiles with Xcode 6.3 (6D570)

Thank you okla for this awesome project

Thanks, I've added your edits to the new commit.


Sidenav buttons style is overriden by button

Steps to reproduce
Seems to be depending on the order of css files imports, no easy reproduction steps.
Expected behavior
Borders on hover are not this much dark @FlyersPh9
Additional information
Button style overrride sidenav button styles:

Released in itwinui-css v0.18.1 and already available in itwinui-react v1.7.0


Update documentation

Adds documentation to migrate to moved ThumbnailOption

@stefanfoulis have a look at this, to see whether it's clear enough

LGTM