[Video Player] Doesn't work auto repeat

Currently, auto-repeat (looping video) doesn't work correctly.
Related issues

https://github.com/sony/flutter-elinux-plugins/pull/20

The video player example it crash

Hello.
I solved it this patch:
diff --git a/packages/video_player/elinux/gst_video_player.cc b/packages/video_player/elinux/gst_video_player.cc
index 2a252a9..d0e1e09 100644
--- a/packages/video_player/elinux/gst_video_player.cc
+++ b/packages/video_player/elinux/gst_video_player.cc
@@ -160,9 +160,10 @@ int64_t GstVideoPlayer::GetCurrentPosition() {
       is_completed_ = false;
       lock.unlock();

-      stream_handler_->OnNotifyCompleted();
       if (auto_repeat_) {
         SetSeek(0);
+      } else {
+        stream_handler_->OnNotifyCompleted();
       }
     }
   }

I see. Can you please make a change? thanks.


Duplicate text on the venue page.

See this:
http://www.pyconuk.org/venue/
The following paragraph appears twice in two different locations:
"If you come by car, there are car parks to both left and right of the entrance (Use the right hand one please, to leave the left hand one for other building users). They are barrier-controlled; take a ticket from the machine, don't lose it, and if you hand it in at Reception they will be able to validate it so that you don't have to pay when you leave :-)"
Is there a good reason for this..?  ;-)

I reckon we should remove this, most likely a copy/pasta error anyway ;)


Platform-aware extraction of container image digest

The utility method inspect_remote_img introduced in #208  extracts the digest from the first manifest in a manifest list. This will produce an undesired outcome if the manifest list is multi-arch, e.g. contains both x86 and arm64 images. A fix would be to give the method an argument platform, that can take values from this list such as
var SupportedArches = map[string]OCIPlatform{
	"amd64":    {OS: "linux", Architecture: "amd64"},
	"arm32v5":  {OS: "linux", Architecture: "arm", Variant: "v5"},
	"arm32v6":  {OS: "linux", Architecture: "arm", Variant: "v6"},
	"arm32v7":  {OS: "linux", Architecture: "arm", Variant: "v7"},
	"arm64v8":  {OS: "linux", Architecture: "arm64", Variant: "v8"},
	"i386":     {OS: "linux", Architecture: "386"},
	"mips64le": {OS: "linux", Architecture: "mips64le"},
	"ppc64le":  {OS: "linux", Architecture: "ppc64le"},
	"riscv64":  {OS: "linux", Architecture: "riscv64"},
	"s390x":    {OS: "linux", Architecture: "s390x"},

	"windows-amd64": {OS: "windows", Architecture: "amd64"},
}

and then return the digest of the image of the corresponding platform.

To select the manifest given a platform (OS + Arch), we can do something similar to this:
jq -r '.manifests[] | select(.platform.os == "linux") | .digest'


Nested footnotes gives incorrect number order

Description
When creating nested footnote, they are created with the wrong order, all the no-nested first, then nested what give a weird display.
Hey, it this a bug#footnote[it's 1] ?
That the footnotes#footnote[it's 2#footnote[should be 3]] are not in the correct order#footnote[should be 4]
Reproduction URL
No response
Operating system
Web app
Typst version

[X] I am using the latest version of Typst

That's unfortunately a known limitation of how things work currently. I can't do much about it right now, but I'm sure we can fix it down the road as the layout system improves.


Link to  BitcartCC?

This project is not sponsored by BitcartCC.
Why there's a link on README?

Hi! Well, it is sponsored by my time spent on maintaining the project and fixing the bugs, replying the issues in time etc. I mean, it just tells the users that if they don't want to re-implement from scratch, they can rely on a ready implementation written here:
https://github.com/bitcartcc/bitcart/blob/master/daemons/trx.py
But it doesn't stop them from using tronpy in whatever way they wish, which now uses native cryptography for example which was quite a pain to code really. Sponsoring by development time is a thing (:

Closing this, but if you have any concerns feel free to tell them (:


auto-generating the dockerfile

USE_DOCKER=true variable allows to build images and rootfs using docker. To do that sometimes is required to use a Dockerfile with the packages needed to create a job.
Because there are different Dockerfiles and they use a specific  dockerhub distro version, add a way to define all those versions in a file (versions.txt) to easily update when required.

Fixed by #40.


french train data notebook

Brief Description

I'd like to write a notebook that leverage French train data to illustrate some basic pyjanitor functions and the difference between adding new column statistics in the call chain verses one at a time.
(optional but encouraged) This notebook would likely cover the following pyjanitor functions:

clean_names
remove_empty
rename_column
drop_columns
join_apply, add_columns

Dataset
https://raw.githubusercontent.com/rfordatascience/tidytuesday/master/data/2019/2019-02-26/small_trains.csv

French train data source - National Society of French Railways

I would like to work on this issue.


Adds conda install instructions

We're now up on conda-forge, so this updates the installation instructions to reflect that.
closes #104

That makes a lot of sense, I was just being lazy. I pushed a reorganization of the docs that hopefully makes things clearer for the user with a heavy use of tabs. This makes the raw rst file a bit harder to read, but I think it's worth it! Let me know what you think


Refactor HandlerOutput and HandlerResult

HandlerOutput and HandlerResult should be modeled roughly as:
struct HandlerResult<T> {
    // We want to return logs even if the processing fails
    logs: Vec<String>,
    result: Result<HandlerOutput<T>, Error>
}

struct HandlerOutput<T> {
    // previously `result`, which "collides" with Rust's `Result`
    value: T,
    events: Vec<IbcEvent>,
}

HandlerOutputBuilder would also be renamed to HandlerResultBuilder, and build a HandlerResult. with_result() could be split into with_value(T), and without_value() (to avoid the ugly with_result(()) when there's no result). Hopefully we could get rid of merge() and merge_output() as well.
@hu55a1n1 and @romac contributed to these ideas.

Related: #23

This API is no longer used since #279. It will be completely removed after #442.


⚠ Bofrost.at has degraded performance

In 76bcb15, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1191 ms

Resolved: Bofrost.at performance has improved in 28ae634 after 304 days, 19 hours, 11 minutes.


how to add struct to bind js class?

I want to add struct,like this:
struct User {
    username: String,
    email: String,
    sign_in_count: u64,
    active: bool,
}

fn build_user(email: String, username: String) -> User {
    User {
        email,
        username,
        active: true,
        sign_in_count: 1,
    }
}

impl User{

    fn introduce(self){
        println!("{} {}",self.username,self.email)
    }
}

and   I  want to use User in JS , like this
let user=new User("Jason","xxxxx@gmail.com",false,10);
user.introduce();

how to do it?  thanks!

maybe need support JS_NewClass？ Consider implementing this data type?

maybe need support JS_NewClass？ Consider implementing this data type?

Correct. This is not implemented yet.


Setup Issues

I found that after installing via BRAT, I was able to turn on an configure the plugin options.  When I went to a note to test it, typing at opened a menu with the options "Today, Yesterday, Later". I found that the popular "Natural Language Dates" pluggin uses @ by default to trigger a menu. I turned off that feature of it's plugin, but then found that People Link still wasn't working, @ didn't do anything.  I tried to go back to the configuration page, but People Link now doesn't show a configuration page. It's in the list of plugins, shows it's turned on, has a heart and an X option, but no configuration icon.  Note that I've exited Obsidian a number of times hoping to get it to open fresh and work, but no dice.

Thank you for your feedback. I will update the readme to address the conflict between Natural Language Dates and People Link.
I attempted to replicate the issue using the same method but was unsuccessful. Are you using the Dataview plugin? It is necessary for People Link to function correctly.

but People Link now doesn't show a configuration page.

This looks like a fatal error happened while loading the plugin or the settings page. Can you give me a screenshot of the console tab in the developer tools opened by CTRL-SHIFT-I (option-cmd-i for mac)?

1.0.1 has been released. Check if the new instructions and improved error handling help with your problem.

I updated to 1.0.1, but the behavior has stayed the same.  I do have Dataview installed.  Here's the error I'm seeing in the console, it appears maybe you don't like paths to begin with underscores, or you require quotes around paths (I think I probably didn't use quotes)?
Uncaught (in promise) Error: Failed to parse query in 'pagePaths': Error:
-- PARSING FAILED --------------------------------------------------

> 1 | _Resources/People
    | ^

Expected one of the following:

'!', '(', '-', 'csv(', 'outgoing(', file link, string, tag ('#hello/stuff')

    at DataviewApi.pagePaths (plugin:dataview:18656:19)
    at DataviewApi.pages (plugin:dataview:18678:21)
    at PeopleSuggest.updateSuggestionsCache (plugin:people-link:9060:13)
    at PeopleSuggest.getDataviewAPI (plugin:people-link:9044:14)
    at new PeopleSuggest (plugin:people-link:8986:10)
    at PeopleLinkPlugin.onload (plugin:people-link:9175:26)

I deleted the plugin folder and started from scratch.  My 'people' notes are located at "_Resources/People", so I set that as the Dataview Source (including quotes this time), and now the Config page stays viewable.

Yes, the folder requires double quotes, which is the syntax of Dataview
source. I’ll fix this problem to ensure setting tab is always shown, and
add more hints to help users find out what is going on.
On Mon, Apr 10, 2023 at 20:30 jpfieber @.***> wrote:

I updated to 1.0.1, but the behavior has stayed the same. I do have
Dataview installed. Here's the error I'm seeing in the console, it appears
maybe you don't like paths to begin with underscores, or you require quotes
around paths (I think I probably didn't use quotes)?
Uncaught (in promise) Error: Failed to parse query in 'pagePaths': Error:
-- PARSING FAILED --------------------------------------------------

1 | _Resources/People
| ^

Expected one of the following:
'!', '(', '-', 'csv(', 'outgoing(', file link, string, tag ('#hello/stuff')
at DataviewApi.pagePaths (plugin:dataview:18656:19)
at DataviewApi.pages (plugin:dataview:18678:21)
at PeopleSuggest.updateSuggestionsCache (plugin:people-link:9060:13)
at PeopleSuggest.getDataviewAPI (plugin:people-link:9044:14)
at new PeopleSuggest (plugin:people-link:8986:10)
at PeopleLinkPlugin.onload (plugin:people-link:9175:26)

I deleted the plugin folder and started from scratch. My 'people' notes
are located at "_Resources/People", so I set that as the Dataview Source
(including quotes this time), and now the Config page stays viewable.
—
Reply to this email directly, view it on GitHub
https://github.com/reorx/obsidian-people-link/issues/1#issuecomment-1501762877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADDDVELNDAJ2CLZIIBTVPLXAP4MZANCNFSM6AAAAAAWYPOL2Y
.
You are receiving this because you commented.Message ID:
@.***>

--
Sincerely,
Xiao Meng
reorx.com


fix(agents): append assistant msg in answer endpoint

I think this is likely better fixed by #558 .

Closing in favor of #558


Update README.md

Minor changes.

 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.

I think just "Rebranded Corona SDK" is far better. IMO, repeating the same name is not entirely necessary.
(By the way, it could be a good idea to sign CLA to submit your changes.)


[scatter-plot] remove contribution mode from time-series scatter chart

Request to remove contribution mode from time-series scatter chart from @junlincc

@stephenLYZ sorry for overwhelming you with all the echarts requests 😅


Adding additional outputPaths throws merge-trees error

outputPaths: {
      app: {
        css: {
          'activate': '/assets/activate.css'
        }
      }
    },

Returns the following error:

Merge error: file assets/activate.css.map exists in /Users/iradchenko/workspace/dashboard/tmp/broccoli_merge_trees-input_base_path-Li6gifv7.tmp/0 and /Users/iradchenko/workspace/mapgeo2/dashboard/tmp/broccoli_merge_trees-input_base_path-Li6gifv7.tmp/1
Pass option { overwrite: true } to mergeTrees in order to have the latter file win.

And adding the overwrite to the merge trees in my ember-cli-build.js file doesn't do anything. Note, I do have an app.less.

@stefanpenner do you think this is a regression somewhere in the build pipeline?


Unit test

Submit Junit test

@Hexinyiyi note that your PR is supposed to be targeted against the module organisation's repository and not under se-edu.
Do kindly submit your work against the right upstream repository.
We will be closing this PR.


Compass Features Without Touch Map

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
In Map When i tap Custom Compass button after that Compass Feature start.The map will now rotate based on the direction you're facing, with the compass needle pointing to the north.
Proposal
Tracking Order By Delivery Driver

@dineshtechnoproject
Can you take a look at below issues and see if they resemble your case ?
https://github.com/flutter/flutter/issues/26117
https://github.com/flutter/flutter/issues/28191


tab is duplicated when using ng-if and binded value for condition is refreshed

I have a tab with ng-if condition
<tab id="..." heading="..." ng-if='orphanInstances' >

And a function in the controller
$scope.refresh = function () {
...
     $scope.orphanInstances = data;
...

If I replace ng-if='orphanInstances' by ng-hide='!orphanInstances' I have only one tab, as expected.
AngularJS v1.2.0

Is this still an issue for you in the current master branch? Can you create a plunkr reproducing the problem?

This does not appear to be an issue still, as can be seen here.
Closing as resolved.


Fixed undefined emoji

fixed the flag emoji by changing name of country to correct one https://github.com/llaske/sugarizer/issues/1603
https://github.com/llaske/sugarizer/assets/116213566/45d41d77-f12c-415e-9683-14d66410a0f6

Nice. But should be cool to ensure there are no other issues on flags.
In short: ensure that all countries names in geojson files match a country in flag.js file.
Could be automatized.
What do you think?

That sounds nice..let me work on it .

hi @llaske ..so i was getting familiar with the code base and i noticed that in map.js,target name and tool_display was being matched and taken care of .i still didnt get why we got this error after this.However, I am still trying to figure out a way so that countries in geojson and flag.js match.if you have something you can guide me.                                                                                other thing was the emoji for French Southern Territories was missing so i took care of that.Rest of the flags are alright .

My idea is to extract the list of existing countries in the app and comparing it to flag list.
For example, one way to do that could be to:

Write a script to extract all NAMES in world.geojson, europe.geojson, ... and put it in a CSV file order by name (and deduplicated).
Extract country names in flag.js and put it in a CSV order by name (and with _ replaced by white space).
Compare both CSV files to see where there is missing or misspelled flags.

@piyushmishraa make sense?

Hmm makes sense👍, getting back to work ✌🏼

@piyushmishraa do you still work on it?

Fixed in https://github.com/llaske/sugarizer/commit/461795ebeb5171f80e707f6b559ae0a42f86be3f


Extending siddhi-doc-gen module

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Codecov Report

Merging #1175 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1175      +/-   ##
============================================
+ Coverage     61.33%   61.35%   +0.01%
  Complexity       35       35
============================================
  Files           617      617
  Lines         24715    24715
  Branches       3841     3841
============================================
+ Hits          15160    15163       +3
+ Misses         8271     8267       -4
- Partials       1284     1285       +1

Impacted Files
Coverage Δ
Complexity Δ

...ore/util/persistence/InMemoryPersistenceStore.java
53.84% <0%> (-5.13%)
0% <0%> (ø)

.../io/siddhi/core/util/snapshot/SnapshotService.java
38.3% <0%> (-0.21%)
0% <0%> (ø)

...ut/stream/state/AbsentStreamPreStateProcessor.java
86.61% <0%> (+1.4%)
0% <0%> (ø)
:arrow_down:

...rocessor/stream/window/SessionWindowProcessor.java
75.37% <0%> (+1.49%)
0% <0%> (ø)
:arrow_down:

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 076616a...4dfd2c9. Read the comment docs.


Docker cannot run in DO VM

There are issues running docker with the current provisioning scripts:
root@netobs-chadell:~# docker
Traceback (most recent call last):
  File "/usr/lib/command-not-found", line 28, in <module>
    from CommandNotFound import CommandNotFound
  File "/usr/lib/python3/dist-packages/CommandNotFound/CommandNotFound.py", line 19, in <module>
    from CommandNotFound.db.db import SqliteDatabase
  File "/usr/lib/python3/dist-packages/CommandNotFound/db/db.py", line 5, in <module>
    import apt_pkg
ModuleNotFoundError: No module named 'apt_pkg'

@chadell
I had to do a remove and install python3-apt
But the docker installation is not working well
    sudo apt remove python3-apt
    sudo apt install python3-apt
    sudo apt-get update
    sudo apt-get install docker-ce docker-ce-cli containerd.io docker-buildx-plugin docker-compose-plugin

This has been fixed with the docker installation script


feat: add volumes / fix issue with port

Pull Request Template for Helm Chart
Description
Add optional volumes & fix issue with exposed port from deployment.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Tested locally on a deployed K3s instance
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

Additional Information

Does this change introduce any new Kubernetes resource types? (No)
If yes, please list them:

Does this change modify any existing Kubernetes resource types? (Yes)
If yes, please describe the modifications:
Added volumeMounts for uploads

Are there any changes to the values.yaml file? (No)
If yes, please describe the changes and their purpose:

Does this change require any specific Kubernetes permissions or RBAC changes? (No)
If yes, please describe the required permissions:

Are there any changes to chart dependencies? (No)
If yes, please list the changes:

Have you updated the CHANGELOG.md file? (No)

Additional context
Add any other context or screenshots about the pull request here.

This LGTM, I've sent it to CI :-) Thanks very much @Mattchewone .

Hey @Mattchewone , looks like the job failed, needing a version bump.
It's a bit weird how that works - I didn't set that bit up, and I'll try and fix the job later. Would you mind just bumping to 1.0.4 in the chart.yaml to see if that fixes the job?

Hey @Mattchewone , looks like the job failed, needing a version bump.
It's a bit weird how that works - I didn't set that bit up, and I'll try and fix the job later. Would you mind just bumping to 1.0.4 in the chart.yaml to see if that fixes the job?

Ah ok, sure let me try that.

Great PR! I tried it, and it fixed the issue with the app loading on the wrong port (3000 instead of 5000).
@Mattchewone Can you please, add the following values into the values.yaml file?
extraVolumes: []
extraVolumeMounts: []

Without, we get two issues with Helm-Intellisense:
Value not defined
postiz-deployment.yaml(42, 24): .Values.extraVolumeMounts

Value not defined
postiz-deployment.yaml(68, 20): .Values.extraVolumes

I'd be happy to merge this now, but @Mattchewone , could you consider @timoa 's comments please? I'm indifferent (don't mind) about adding them, I think.

@timoa / @jamesread sorry I was away so couldn't respond.
I've updated. Hopefully this resolves any issues with the linter.


Multiple select has no selected class in search results

In detail,
Affect in multi-select mode with a searchable list

Search items
Select an item from filtered list
Close the drop
Open the drop and you will see that, in the drop the selected item is not highlighted


fix sample to use correct listIncomingRelationships() overload

Description
Update sample to use correct overload of listIncomingRelationships.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

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

@johngallardo Any progress about this PR?

/azp run

/azp run java - digitaltwins - tests

@v-xuto can you help merge this PR? We are seeing "Required status check "license/cla" was not set by the expected GitHub app.".

@johngallardo Any progress about this PR?

It looks blocked by this... how do we resolve @v-xuto ?

@johngallardo Thanks for your prompt feedback. But I don't have permission to merge this PR either.


"amplify function push" times out with "Inaccessible host" error

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
10
Amplify CLI Version
4.45.2
What operating system are you using?
mac
Amplify Categories
function
Amplify Commands
push
Describe the bug
amplify push times out with error logged below
Expected behavior
no error, push succeeds
Reproduction steps
amplify function push
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Inaccessible host: `amplify-wesponsored-web-hdev-170732-deployment.s3.us-east-2.amazonaws.com'. This service may not be available in the `us-east-2' region.
UnknownEndpoint: Inaccessible host: `amplify-wesponsored-web-hdev-170732-deployment.s3.us-east-2.amazonaws.com'. This service may not be available in the `us-east-2' region.
    at Request.ENOTFOUND_ERROR (/Users/tomhirschfeld/.nvm/versions/node/v13.12.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/event_listeners.js:507:46)
    at Request.callListeners (/Users/tomhirschfeld/.nvm/versions/node/v13.12.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/Users/tomhirschfeld/.nvm/versions/node/v13.12.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/Users/tomhirschfeld/.nvm/versions/node/v13.12.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at ClientRequest.error (/Users/tomhirschfeld/.nvm/versions/node/v13.12.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/event_listeners.js:339:22)
    at ClientRequest.<anonymous> (/Users/tomhirschfeld/.nvm/versions/node/v13.12.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/http/node.js:96:19)
    at ClientRequest.emit (events.js:327:22)
    at ClientRequest.EventEmitter.emit (domain.js:485:12)
    at TLSSocket.socketErrorListener (_http_client.js:432:9)
    at TLSSocket.emit (events.js:315:20)
    at TLSSocket.EventEmitter.emit (domain.js:485:12)
    at errorOrDestroy (internal/streams/destroy.js:128:12)
    at onwriteError (_stream_writable.js:406:3)
    at onwrite (_stream_writable.js:427:7)
    at internal/streams/destroy.js:60:7
    at TLSSocket.Socket._destroy (net.js:679:5)
    at TLSSocket.destroy (internal/streams/destroy.js:55:8)
    at WriteWrap.onWriteComplete [as oncomplete] (internal/stream_base_commons.js:93:12)
An error occurred when pushing the function resource

@tomhirschfeld This seems like a network issue. I've seen this happen when I'm behind a proxy disallowing access to certain endpoints.

@kaustavghosh06
I'm not on a proxy to be clear

@tomhirschfeld Few questions

Are you able to continuously repro this or is this intermittent?
Can you try running amplify push instead of amplify function push as a workaround?

From this issue out here - https://github.com/aws/aws-sdk-js/issues/2618 looks like it's an intermittent S3 DNS issue.

@tomhirschfeld Were you able to get past this issue?

yea, it would be nice to have a better error message when this happens,
maybe a bullet point list of like:
"things to check to resolve this error:

confirm you have no proxy between you and amazon
confirm no internet connectivity issues in your network
check for outages on aws
this might be a transient issue related to aws internal networks, try
again in an hour
"

On Mon, Mar 22, 2021 at 10:28 AM Kaustav Ghosh @.***>
wrote:

@tomhirschfeld https://github.com/tomhirschfeld Were you able to get
past this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/6930#issuecomment-804252712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA233UMNTBUXD2UA2AU25TLTE543JANCNFSM4ZSAP3SA
.


Better error reporting

Currently, only generic HTTP error codes are reported. These should be annotated with some known reasons:

302 (redirect to authelia) Forgot to login to NDS?
404 Not found - wrong URL of OpenAPI JSON, wrong reference in JSON file?
Check other error codes and add contextual information on-demand, e.g. from OpenAPI error specifications.

We agreed that it would be better to forward HTTP errors as they are embedded into the response body of called endpoints.

Fixed with #50


Build error

ERROR in ./src/js/ThreadView.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
 @ ./src/js/ThreadView.js 29:16-29

ERROR in ./src/js/App.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
 @ ./src/js/App.js 42:16-29

ERROR in ./src/js/MessageStore.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
 @ ./src/js/MessageStore.js 25:15-28

ERROR in ./src/js/ThreadStore.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
 @ ./src/js/ThreadStore.js 22:15-28

ERROR in ./src/js/BaseStore.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
 @ ./src/js/BaseStore.js 16:15-28
webpack: bundle is now VALID.

I got around this by changing the require( 'rx' ) statements to require( 'rx-lite' )

Oh good catch. I must have installed both.
On Sat, May 9, 2015, 12:14 PM Eric Lewis<EMAIL_ADDRESS>wrote:

ERROR in ./src/js/ThreadView.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
@ ./src/js/ThreadView.js 29:16-29
ERROR in ./src/js/App.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
@ ./src/js/App.js 42:16-29
ERROR in ./src/js/MessageStore.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
@ ./src/js/MessageStore.js 25:15-28
ERROR in ./src/js/ThreadStore.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
@ ./src/js/ThreadStore.js 22:15-28
ERROR in ./src/js/BaseStore.js
Module not found: Error: Cannot resolve module 'rx' in /Users/eric/Projects/nuclearmail/src/js
@ ./src/js/BaseStore.js 16:15-28
webpack: bundle is now VALID.
I got around this by changing the require( 'rx' ) statements to require(
'rx-lite' )
—
Reply to this email directly or view it on GitHub
https://github.com/ianobermiller/nuclearmail/issues/4.


Quiz App

Describe the project you want to add with tech stack

I would like to create a Quiz App using Html , Css and Javascript. It will ask few questions to user and if its correct ,the user score will increase .Please assign it to me.
Expected behavior

Screenshots (optional)

Additional context (optional )

Already present in the repo


[v1.36.0] Advanced examples no longer working

Describe the bug
I'm trying out Maestro Cloud, and was surprised that my flow failed immediately because it couldn't find one of my subflows. I downloaded the examples to see what I might be missing, and I discovered that those flows also didn't know what to do with subflows.
To Reproduce
Simply follow your own walkthrough:

Install/update the CLI with curl -Ls "https://get.maestro.mobile.dev" | bash
Log in with maestro login
Download the samples with maestro download-samples
Try to run one of the advanced flows from the 'samples' folder: maestro cloud sample.apk android-advanced-flow.yaml
See an error message:

This also applies to the advanced iOS tests run with maestro cloud sample.zip ios-advanced-flow.yaml.

Expected behavior
The example test cases should run. More broadly, you should be able to refer to and run other Maestro flows within the same directory and within child directories.

@kimthompson I understand why this is confusing for you - the walkthrough shows you how to run the simple Flows, whereas the advanced Flows require you to upload the full samples folder since just uploading ios-advanced-flow.yaml means that the subflows that that Flow is referring to is not part of the upload. I recognize that the error message is not the best here and I updated the docs to hopefully make this more clear.
So, to summarize - running maestro cloud samples/sample.zip samples should work for you.


solargraph: Document the original bundle, not the bundlerApp's:

bundlerApp creates a bundle, but when running solargraph as a
bundlerApp, you would ~always want to interact with the original bundle,
not the one containing solargraph and its dependencies.
Motivation for this change
My users want to run solargraph on ruby projects without adding it to their Gemfiles.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Do you know if solargraph requires bundler explicitly, ie: require "bundler"?
One option that hasn't really been explored yet is that since we have all the gems, those could be composed together solely using GEM_PATH and skip bundler entirely at runtime. And if no gem is requiring gem directly, LOAD_PATH could also be used to compose the ruby libraries together.

So, yes, it does explicitly require bundler; however, it doesn't explicitly run Bundle.setup or require bundler/setup.
I've considered this too for my own bundlerDevEnv tooling: having the two paths by which ruby code can be read (symlinked and canonical) can often confuse ruby into double-loading files, and on top of that, creating the big bag of symlinks is a little expensive for large gemsets. It seems like a slam-dunk for bundlerApp—if not categorically, at least as an option.

I've wondered, too, whether we could get away with providing a completely-fake shim implementation of bundler/setup that would just pass, because we previously put everything on the GEM_PATH.
If there's ever going to be an app that some strategy like this wouldn't work on, it's going to be Shopify's core platform. Maybe I'll give this a shot there when I get a chance.


feat: migrate remix v2

Since Remix just released v2, I have updated this template to use v2. I also made some changes to drizzle code as well.

Hey @kishanhitk, sorry I missed this!
I'll give it a spin/fix it up and merge it in soon


Portal Glitch

How to reproduce, make a portal. Break portalblock (Not obsidian), place 1 ore two obisidian in portal, relight portal. Remove two obsidian blocks. Remove fire. Light it.
It doesn't work then!
Tested it multiply times!

The bug here is Remove fire.
What is that ? Is it possible to remove fire in vanilla ?
Do we handle that ? I bet we don't.

ah what we don't support is making fire. So the server doesn't know about the fire which was placed. Not sure what's the consequence of that exactly.

@melerpe fixed ;)


Configuration documentation missing Google Plus Comments details

The configuration documentation provides details about about setting up comment providers.
It starts with a list of comment providers and then has a section for each that details it's configuration information.
Google Plus is listed as one of the providers but there is not details section for it.

That's because there is no configuration for Google Plus comments. You either enable it or not. Unlike the other options there are no configs needed.
provider: google-plus

Ok! Thank you for the clarification. Would you like me to close the issue?


AMS-sketch

What's it ?
related to #563
This is a monoid implementation for AMS sketch. Not all the optimization from Count Min Sketch algorithm are implemented, but it's a beginning and for some operation like innerProduct estimation, join, ..
What does it  bring ?
I made some tests like join between two AMS / CMS :
[info] Running (fork) org.openjdk.jmh.Main -t1 -f1 -wi 4 -i 4 .*AMSJoin.*
[info] Benchmark                                (bucket)  (depth)  (size)   Mode  Cnt      Score      Error  Units
[info] AMSJoinBenchmark.amsJoinBenchmarkString        27       16    1000  thrpt    4  74803,226 ± 13458,935  ops/s
[info] AMSJoinBenchmark.amsJoinBenchmarkString       543       16    1000  thrpt    4   3355,353 ±  298,165  ops/s
[info] AMSJoinBenchmark.amsJoinBenchmarkString      5438       16    1000  thrpt    4    243,427 ±   92,126  ops/s

and for CMS :
[info] Running (fork) org.openjdk.jmh.Main -t1 -f1 -wi 4 -i 4 .*CMSJoin.*
[info] Benchmark                                  (delta)   (eps)  (size)   Mode  Cnt      Score      Error  Units
[info] CMSJoinBenchmark.amsJoinBenchmarkString  0.0000001     0.1    1000  thrpt    4  45450,447 ± 5838,600  ops/s
[info] CMSJoinBenchmark.amsJoinBenchmarkString  0.0000001   0.005    1000  thrpt    4   3502,317 ±   83,948  ops/s
[info] CMSJoinBenchmark.amsJoinBenchmarkString  0.0000001  0.0005    1000  thrpt    4    320,697 ±   23,353  ops/s

the delta / eps are translate to the proper buckets / depth for AMS / CMS.
I think with more optimisation we can have better score for AMS.
Better error estimation on inner-product
The estimation for innerProduct or F2 between AMS aren't properly  implemented yet. I'll dig more in research paper to find probability, bounds, ...
BUT
I benchmark on something like :
class AMSSketchInnerJoinCMS extends WordSpec with Matchers {
  val numEntries = 10000

  def innerProduct(arr1: Vector[Int], arr2: Vector[Int]): Long =
    arr1.zip(arr2).map(p => p._1 * p._2) sum

  val amsMonoid = new AMSMonoid[Int](16, 543)
  val cmsMonoid = CMS.monoid[Int](16, 543, 1)

  " an AMSSketch " should {
    " have a better approximation of it inner product than CMS " in {
      val entries = (0 until 100).map(_ => Random.nextInt(100))
      val entriesCompare = (0 until 100).map(_ => Random.nextInt(100).abs)

      val trueInner = innerProduct(entries.toVector, entriesCompare.toVector)
      val ams = amsMonoid.create(entries)
      val cms = cmsMonoid.create(entries)

      val amsCompare = amsMonoid.create(entriesCompare)
      val cmsCompare = cmsMonoid.create(entriesCompare)

      val innerAMS = ams.innerProduct(amsCompare)
      val innerCMS = cms.innerProduct(cmsCompare)

      val errorAMS = trueInner - innerAMS.estimate
      val errorCMS = trueInner - innerCMS.estimate

      assert(errorAMS.abs <= errorCMS.abs)

    }

  }

}

I found this test always pass ans AMS create better estimation for the innerProduct.
How to use
  "An AMSSketchMonoid " should {
    "be used like an algebird monoid " in {
      val aMSMonoid = new AMSMonoid[String](100, 100)
      val sketch = aMSMonoid.create(Seq("aline", "aline", "aline"))
      assert(sketch.f2 ~ 9 )
    }
  }

 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.


fix(package): fix prod and aot build error

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the current behavior? (You can also link to an open issue here)
Build prod returns error, see #1213

What is the new behavior (if this is a feature change)?
Build prod and aot without error

Hey @apeter10, thanks for the PR, could you please pin the exact version of the CLI?

Hi @nnixaa, I have changed CLI to 1.2.4

Ref #1213


Security #4: JWT

Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I think it should go after #1626 since it changes nsm.go, mostly refactored by me and moved into chains :-)


Placing water on a water source block crashes the game

Only a spoonful of water they said, that was until I pulled out my Comically large spoon -

crash-2021-03-10_17.57.26-client.txt
almost forgot this

Fixed.


Zoom Bug - Lines are turning solid when set to Dashed (or anything that isn't solid)

Issue: I encountered an issue where zooming in/out will cause a line dashed line to turn into a solid line past a certain zoom level.
This is also possible to encounter when using the middle-click range zoom. Zooming out turns the line back into a dash.
ScottPlot Version:
<PackageReference Include="ScottPlot" Version="5.0.37" />
<PackageReference Include="ScottPlot.WPF" Version="5.0.37" />

https://github.com/user-attachments/assets/9b2e3509-fc44-49d6-aa0b-06414b5ba9f1
Github appears not to like any OBS recording I put up, but windows plays it just fine:
ScreenRecording.zip
Code Sample: This was zooming in and out using native functionality in the wpf app.

The lines are generated like so:
public DataPointSource? Render(PlotHelper plotHelper)
{
    if (!IsVisible) return null;

    // setup the Y axis
    var plot = plotHelper.Plot;
    var color = (LineColor ??= ColorViewModel.GetNextColor(plotHelper)).ScottPlotColor;

    IYAxis yAxis;
    if (Slot.SignalType is SignalType.Input or SignalType.Output)
    {
        yAxis = plotHelper.BooleanAxis;
    }
    else if (Slot.SignalType is SignalType.VacuumLevel)
    {
        yAxis = plotHelper.VacuumAxis!;
    }
    else
    {
        if (plotHelper.IsFirstLeftAxisUsed)
        {
            yAxis = new ScottPlot.AxisPanels.LeftAxis();
            plot.Axes.AddLeftAxis(yAxis);
        }
        else
        {
            yAxis = plot.Axes.Left;
            plotHelper.IsFirstLeftAxisUsed = true;
        }

        yAxis.Label.Text = ChartDisplayName;
        yAxis.Label.ForeColor = color;
        yAxis.IsVisible = yAxis == plot.Axes.Left || yAxis == plot.Axes.Right || ShowAxisLabel;

        switch (Slot.SignalType)
        {
            case SignalType.Counter:
                yAxis.TickGenerator = new ScottPlot.TickGenerators.NumericAutomatic() { IntegerTicksOnly = true };
                break;

            case SignalType.VacuumLevel:

                break;

            case SignalType.AxisCurrent:
            case SignalType.AxisData:
                yAxis.Label.Text = $"{ChartDisplayName} {(Slot.Unit == RecordUnit.UNITMM ? "(mm)" : "")}";
                break;

            default:
                break;
        }
    }

    // setup the plot
    switch (Slot.RecordMode)
    {
        case RecordMode.Poll:
            var signal = plot.Add.SignalXY(Slot.TimeStamps, Slot.Values, color);
            signal.Axes.YAxis = yAxis;
            signal.LegendText = DisplayName;
            signal.LinePattern = LinePattern;
            return _dataPointSource = new DataPointSource(this, signal);

        case RecordMode.Edge:
            Coordinates? prevCoordinate = null;
            List<double> x = new List<double>();
            List<double> y = new List<double>();
            for (int i = 0; i < Slot.TimeStamps.Length; i++)
            {
                Coordinates prev = prevCoordinate ?? default;
                var cord = new Coordinates(Slot.TimeStamps[i], Slot.Values[i]);

                // check to create a vertical line as the edge
                if (i > 0 && cord.Y != prev.Y)
                {
                    x.Add(cord.X);
                    y.Add(prev.Y);
                }
                // current coordinate
                x.Add(cord.X);
                y.Add(cord.Y);

                prevCoordinate = cord;
            }

            // horizontal line until end of graph
            if (prevCoordinate is Coordinates last)
            {
                x.Add(plotHelper.MaxXY.X);
                y.Add(last.Y);
            }

            var scatter = plot.Add.ScatterLine(x, y, color);
            scatter.Axes.YAxis = yAxis;
            scatter.LinePattern = LinePattern;
            scatter.LegendText = DisplayName;
            return _dataPointSource = new DataPointSource(this, scatter);

        default:
            throw new NotImplementedException($"Unknown Recording Mode : {Slot.RecordMode}");
    }
}

I was just reviewing a similar problem I'm having. I'm not sure exactly, but I think it's a result of Signal.RenderHighDensity? The dashes are the exact same size regardless of the zoom level until I hit multiple data points per pixel, which triggers RenderHighDensity, and the dash setting is immediately lost and it starts to draw as a solid line. My suspicion is that it's caused by the path.LineTo -> path.MoveTo loop at the bottom of the function, but I'm not sure a good solution exactly for it.

So in my scenario, the issue was definitely not to do with multiple data points per pixel.
My square wave shown has 1 data point per corner, and thats it.
for example:

start
Corner1
Vertical (UP)
Horizontal
vertical (DOWN)
Continue to edge

(0,0)
(100,0)
(100,1)
(200,1)
(200,0)
(300,0)

Having a bit of a closer look, it runs RenderHighDensity when the visible x range span, divided by the total data width, divided by the space between data points, is at least 1, and RenderHighDensity draws lines to connect the top and bottom of any "pixel column", where any two Y points share the same X pixel (not data). Which looks like it's still the cause of both of our problems.
It's still unclear to me why the draw call doesn't utilize the dashed line paint, though, so I'm not positive the solution.

Looks like using the stroke cap Square or Round could cause these problems, but definitely needs a bit more investigation. I'll test using a different stroke cap when I have the chance.

I'll close this issue for now, but if a solution is reached we can open it back up and discuss how to correct it!
If someone creates a PR to fix this issue, I will be happy to review it an merge it in quickly.
I probably won't start working on this myself soon because I can tell this is the type of issue that you can get sucked into for a very long time 😅
Thanks for reporting this issue, and for your comments along the way!

Re: a problem that you can get sucked into for a very long time. Yes, indeed!
I've dug a bit more into this. It's interesting, because it's two separate problems, as Signal (what I saw this bug in) and SignalXY use completely different render paths. But they may be the same idea.
Reproducing this issue with both Signal and SignalXY, I'm still not sure exactly what is happening or where the breakdown is occurring, but I can see that there are many offscreen pixels being passed to the Skia renderer for both, when these artifacts begin showing up. Specifically, there are points being drawn billions of pixels off the screen, usually in the Y direction.
At the moment between drawing a solid line and a dashed line, I also noticed that the maximum range between two drawn-offscreen pixels crossed the 8billion threshold. Before zooming in, it's dashed and good, but after zooming in, it's solid and has two pixels with a line drawn between them (offscreen) that is > 8 billion pixels long.
I haven't yet worked through if clamping those pixels out will fix it, but that feels an oversight regardless. Both SignalXY and Signal, in their own clamping patterns, only clamp pixels in the X range, and never seem to clamp pixels in the Y range. SignalXY has interpolation performed in SignalXYSourceDoubleArray.GetPixelsToDraw() (or similar ISignalXYSource classes), but it's only done in the X direction (or in the Y direction when rotated, which has the same effect of interpolation only happening in one range). Signal.GetVisibleXRange() does the same thing, without any interpolation at all for off-screen artifacts, and only testing if a point's X value is within the drawn rectangle and never the Y value.
While I'm a little suspicious this may be causing the issue at hand, it's also certainly a large performance drain. For a regular Signal with some noise, after zooming in so 0 points are on screen and only one intersecting line is visible, it was still running calculations and render calls on hundreds of points. I think the current process (ignoring the Y values) is used to ensure the lines between offscreen points are visible if they intersect the drawn canvas.
If I come up with a solution in the coming weeks when I have time to pick at this, I'll set up a PR and let you know!

//pulled from Signal

   private CoordinateRange GetVisibleXRange(PixelRect dataRect)
    {
        // TODO: put GetRange in axis translator
        double xViewLeft = Axes.GetCoordinateX(dataRect.Left);
        double xViewRight = Axes.GetCoordinateX(dataRect.Right);
        return (xViewLeft <= xViewRight)
            ? new CoordinateRange(xViewLeft, xViewRight)
            : new CoordinateRange(xViewRight, xViewLeft);
    }

My gut says this might have to do with what I was running into for IGetNearest. The data rectangle parameter might not be correct for this plottable's assigned axes, which results in the calculation being incorrect. You may want to backtrack where the parameter comes from as a first line of inquiry if this method is the culprit is your line of thinking


New createVertices option for Control.ModifyFeature

Option to disable virtual vertices in the ModifyFeature control
see: http://lists.osgeo.org/pipermail/openlayers-dev/2012-February/008365.html

Looks good, please merge.


Remove stray backtick from doc

Just a minor typo.

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

Thanks!
@bors r+

Thanks!
@bors r+

:pushpin: Commit 206ded3d497082ff799e83a7a5eed5c17be925f5 has been approved by ehuss

:pushpin: Commit 206ded3d497082ff799e83a7a5eed5c17be925f5 has been approved by ehuss

:hourglass: Testing commit 206ded3d497082ff799e83a7a5eed5c17be925f5 with merge 49a32afe5f6e97ef3aaf24780ebca77557c88d54...

:hourglass: Testing commit 206ded3d497082ff799e83a7a5eed5c17be925f5 with merge 49a32afe5f6e97ef3aaf24780ebca77557c88d54...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing 49a32afe5f6e97ef3aaf24780ebca77557c88d54 to master...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing 49a32afe5f6e97ef3aaf24780ebca77557c88d54 to master...


Export and Open buttons not working with SAP HANA driver

Buttons for exporting or opening the results of a query ran through the SAP HANA connection are nonresponsive and produce no output. I don't have this problem for other connections (MySQL).

SQLTools Version 0.27.1
VS Code Version: 1.78.0
OS: Windows
Driver: SAP HANA

I went there first but saw that this issue was brought up a couple years ago and resolved by referring people to this repo

That issue was likely connected to #832 which has since been resolved.
The fact that you're not getting the problem with MySQL connections make me think it's either driver-specific or else related to the data you're trying to export with the SAP HANA driver (e.g. volume, exact contents, or some other factor).

Ok I'll bring up with the driver author. Wondering if it could even be a setting in hanaOptions I'm not aware of.


Use headers to allow anchor links to rules

This allows linking directly to them within the page at the cost of the emphasis of the rule name.

(The force push was to fix the "italicizing" -> "emphasis" in the commit message.)


unresolved import kube::api::AttachParams no AttachParams in api

Current and expected behavior
Current behavior:
When I run the example pod_shell.rs https://github.com/kube-rs/kube/blob/main/examples/pod_shell.rs, compile error: unresolved import kube::api::AttachParams no AttachParams in api
The Cargo.toml:
[package]
name = "example-ssh"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
anyhow = "1.0.69"
async-trait = "0.1.64"
dotenv = "0.15.0"
env_logger = "0.10.0"
futures = "0.3.26"
ldap3 = "0.11.1"
russh = "0.36.1"
russh-keys = "0.24.1"
tokio = { version = "1.26.0", features = ["full"]}
kube = { version = "0.80.0", features = ["client", "runtime", "derive"] }
k8s-openapi = { version = "0.17.0", features = ["v1_26"] }
tracing = "0.1.37"
serde_json = "1.0.93"
tracing-subscriber = "0.3.16"
serde = { version = "1.0.152", features = ["derive"] }

Possible solution
No response
Additional context
No response
Environment
% kubectl version --short
Flag --short has been deprecated, and will be removed in the future. The --short output will become the default.
Client Version: v1.25.4
Kustomize Version: v4.5.7
Server Version: v1.25.4
Configuration and features
No response
Affected crates
No response
Would you like to work on fixing this bug?
None

I stumbled on the same error. The cause for this is that the API docs list it as requiring client not ws.

Thanks this is a documentation issue on our end. Have raised this separately in #1242


am i doing something wrong or

i got the project alpha bot to check my stats and net worth overtime
some issue i faced was the bot wouldn’t take any minecraft ign saying “it never logged onto hypixel”
i tried with other names and it gave me the same issue
is there something i’m doing wrong or is there an issue

Same for me i think the bot is in a very early alpha and it cant do anything

Same for me i think the bot is in a very early alpha and it cant do anything

Issue was resolved, this was because of the api key being invalid.
I also abandoned this project long ago so that is why it took this long


BZ Keyword

Describe the bug
The documentation is wrong about the keyword "BZ". It is not a valid keyword. Instead of "BZ", "BZR" works
To Reproduce
Any example file with BZ as keyword is rejected, whereas with BZR it works.
Expected behavior
Besides the wrong keyword, with BZR mopac works as expected.
Operating system
macOS 13 arm, but this should not matter.
Additional context
I found the correct keyword by searching the keywords in the source code.
P.S. Is it possible to attach a text file, like an mopac input file? I could not find such a possibility.

I don't know whether that is actual bug.
See https://openmopac.github.io/keywords/BZ.html

Sorry, my fault. The keyword needs to be "BRZ" and not "BZR". I'll correct the bug report.
But it definitely is a bug. Whereas the program is called BZ, the keyword for mopac needs to be "BRZ". Obvious source lines are in bz/bz.F90, src/input/wrtkey.F90 and output/writmo.F90:
bz/bz.F90,
line 81:      is_brz = (keywrd(:3) == "BRZ")
output/writmo.F90,
line 735:        if (mers(1) /= 0 .and. .not. mozyme .and. index(keywrd, " BRZ") /= 0) then
src/input/wrtkey.F90,
line 880:  if (myword(allkey, " BRZ"))    write (iw, '(" *  BRZ         - Write file .brz for use by program BZ")')
It took me ages to find it out and is at least a severe bug in the documentation.


Change Cognito UserPool SchemaAttribute "Required" value to boolean from basestring

SchemaAttribute documentation mentions that the "Required" field is a boolean, similar to the "Mutable" field. Wasn't sure if it was just an oversight but to avoid confusion I thought it should also accept plain boolean values.
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-cognito-userpool-schemaattribute.html

Looks correct. Thanks!


SVG not Found on the Server

Hey,
I've the problem that he did not found the svg on the server but it's there
GET http://xxx/wp-content/themes/XXX/assets/images/svg/icons/icon_26_26.svg 404 (Not Found)
Local it runs all fine but remote I've no chance that it works.
Any Idea?

Sorry that nobody responded until the end of September. I imagine you already solved this issue?
You need to debug at the server level how your request is being handled. The process to do this is different for every platform / host / level of access combination. If you share some details about your environment - we may be able to suggest a few ways to troubleshoot this.
If this is fixed, before closing the issue could you maybe explain how you fixed the issue? This would help other people who come across the same problem to potentially fix it more quickly. :)
Thanks
Mark


feat: add environment variable substition to source queries

Description

Adds interpolation for variables in source queries using ${} syntax
"Advanced" Connectors have the option to disable this functionality when reading files (it is done automatically for "simple" connectors)
All variables must be in the environment as EVIDENCE_VAR__[var name] to ensure that they were deliberately set.
e.g.
SELECT * FROM MY_TABLE
WHERE client_id = ${client_id}

EVIDENCE_VAR__client_id=5 pnpm run sources

-- actually runs
SELECT * FROM MY_TABLE
WHERE client_id = 5

Checklist
- [ ] For UI or styling changes, I have added a screenshot or gif showing before & after

[x] I have added a changeset

We may consider scoping this syntax?
For example:
SELECT * FROM MY_TABLE
WHERE client_id = ${env.client_id}

or
SELECT * FROM MY_TABLE
WHERE client_id = ${var.client_id}

This might make it easier to distinguish variables from other queries if we were to add query chaining in the future?
SELECT * FROM ${my_other_query}
WHERE client_id = ${env.client_id}

This might make it easier to distinguish variables from other queries if we were to add query chaining in the future?

I think this may merit a larger discussion; query chaining will have to be (or rather, really should be I think) implemented at the connector level, rather than by Evidence itself, because the rules for doing so will vary based on the query syntax (e.g. how would query chaining work for yaml-based queries). Simple raw variable interpolation is safer and should be pretty portable.

Sounds good!

Added a manual escape for users as well; ${variable} gets replaced with the variable value, $${variable} gets replaced with ${variable}


xdebug not working with PHP 7.4

Describe the bug
Xdebug is not working when creating an env with PHP 7.4.
To Reproduce
Steps to reproduce the behavior:

I followed these steps 1 to 3.

1 .Create a directory (mkdir my-new-project)
2. Change to the new directory (cd my-new-project)
3. Run ddev config

After that, I edited the PHP version in the config.yaml to 7.4, and I updated the xdebug_enabled to true.
ddev start
ddev ssh
In the SSH, I ran php -v, and I got the following result (without the xdebug):

PHP 7.4.0beta4 (cli) (built: Aug 27 2019 13:24:38) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0-dev, Copyright (c) Zend Technologies
with Zend OPcache v7.4.0beta4, Copyright (c), by Zend Technologies

If I update my config.yaml to use PHP 7.2 or PHP 7.3, it works properly. Result from php -v:

PHP 7.3.9-1+0~20190902.44+debian9~1.gbpf8534c (cli) (built: Sep  2 2019 13:31:30) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.9, Copyright (c) 1998-2018 Zend Technologies
with Zend OPcache v7.3.9-1+0~20190902.44+debian9~1.gbpf8534c, Copyright (c) 1999-2018, by Zend Technologies
with Xdebug v2.7.2, Copyright (c) 2002-2019, by Derick Rethans
with blackfire v1.27.0~linux-x64-non_zts73, https://blackfire.io, by Blackfire

Expected behavior
xdebug working when creating an env with PHP 7.4.
Version and configuration information (please complete the following information):

Host computer OS and Version: macOS Catalina 10.15.7
Docker Desktop version if on macOS or Windows (from "About Docker Desktop"): 3.0.0
ddev version information (use ddev version):

DDEV-Local version      v1.11.2
commit                  v1.11.2
db                      drud/ddev-dbserver:v1.11.0-10.2
dba                     drud/phpmyadmin:v1.11.0
ddev-ssh-agent          drud/ddev-ssh-agent:v1.11.0
docker                  20.10.0
docker-compose          1.27.4
os                      darwin
router                  drud/ddev-router:v1.11.0
web                     drud/ddev-webserver:v1.11.0

config.yaml contents for the misbehaving project:

APIVersion: v1.11.2
name: site-temp
type: php
docroot: ""
php_version: "7.3"
webserver_type: nginx-fpm
router_http_port: "80"
router_https_port: "443"
xdebug_enabled: true
additional_hostnames: []
additional_fqdns: []
mariadb_version: "10.2"
nfs_mount_enabled: false
provider: default
use_dns_when_possible: true
timezone: ""

Do you have any custom configuration (nginx, php, mysql) in the .ddev folder? If so, have you tried without them? No. My last test was in a new folder with no configurations.

My ddev was in an old version. I updated and it worked well!


Adding deeplabv3-mobilenetv2

Hi.
Currently, the TFLite variant of the DeepLabV3 model that is available on TFHub is probably coming from this checkpoint- mobilenetv2_dm05_coco_voc_trainval. The TFLite model proposed in this PR is a different one and enhances performance without any unnecessary size increase or drop in segmentation quality.

I thought it is also possible to submit TFLite models. Let me know if that isn't the case.

Hi @sayakpaul, the PR looks good now.
Since this is your first time contributing to the TFhub.dev repository, we just need you to read & agree to the TensorFlow Hub terms of service at https://tfhub.dev/terms. If you feel comfortable moving forward, please add a comment "I have read and agree to the TensorFlow Hub Terms of Service" to this thread.

I have read and agree to the TensorFlow Hub Terms of Service.
@maringeo


Running the remote debugger

Is there a way to run the remote debugger from mocha-phantomjs?

phantomjs --remote-debugger-port=9000 lib/mocha-phantomjs.coffee <your tests> <reporter> <config as json>

Tracking the long term fix for this in #186


split load from flattening

Description
Separate incremental load from and flattening in the msgs and msg_attributes models.
Tests

[x] Please provide evidence of your successful dbt run / dbt test here
[x] Any comparison between prod and dev for any schema change

Checklist

[x] Follow dbt style guide
[x] Run git merge main to pull any changes from remote into your branch prior to merge.
[x] Tag the person(s) responsible for reviewing proposed changes
[x] IMPORTANT Note for deployment if a full-refresh is needed for any model this PR impacts.

If any schema is changing, whether it be a table or a view, we will need to run a full-refresh in prod to avoid errors with the scheduled refresh jobs.
The PR can be reviewed, but should not be merged unless by someone with prod access.

Tag forgxyz if a full-refresh will be required on prod.

Incrementa run throws this error


issue

---InitializeWizard---
---DONE---
---PrepareToInstall---
Font name.....: Hack Fonts
Font version..: 3.003
Setup version.: 1.6.0
Script version: 2.11
Local time....: 2023-10-04 21:12
Fonts folder..: C:\WINDOWS\Fonts
Dest folder...: C:\Program Files\Hack Fonts
---DONE---
---BeforeInstallAction START---
-CHECK UNWANTED FONT FILES-
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-Bold_.ttf
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-Regular_.ttf
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-BoldItalic_.ttf
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-Italic_.ttf
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-BoldOblique*.*
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-RegularOblique*.*
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-Regular-linegap*.*
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-Bold-linegap*.*
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-Italic-linegap*.*
Unwanted font file entry: C:\WINDOWS\Fonts\Hack-BoldItalic-linegap*.*
-CALCULATE HASH FOR EXISTING FONTS-
Calculating hash for Hack-Bold.ttf
File from setup: 28b45407cfdb0e9c5ea5797b5963e01c18a2c269
File in \fonts : -NOT FOUND-
Calculating hash for Hack-Regular.ttf
File from setup: b1cd50ba36380d6d6ada37facfc954a8f20c15ba
File in \fonts : -NOT FOUND-
Calculating hash for Hack-BoldItalic.ttf
File from setup: 12edfb7fc8c0a7e5b2bdb50007dd2024b02d530d
File in \fonts : -NOT FOUND-
Calculating hash for Hack-Italic.ttf
File from setup: a49b01c9fe79f45aa6e951db193c8f39c6c5e2df
File in \fonts : -NOT FOUND-
Checking for differences between fonts in setup and this system
IsSetupFontSameAsInstalledFont(): Hack-Bold.ttf
Checking for font name in registry: Hack Bold (TrueType)
Font not found in registry!
Found difference for file Hack-Bold.ttf
Installation required.
Font deletion required: False
Font installation required: True
Pending changes: True
-DELETE UNWANTED FONT FILES-
---BeforeInstallAction END---
IsSetupFontSameAsInstalledFont(): Hack-Bold.ttf
Checking for font name in registry: Hack Bold (TrueType)
Font not found in registry!
IsSetupFontSameAsInstalledFont(): Hack-Regular.ttf
Checking for font name in registry: Hack (TrueType)
Font not found in registry!
IsSetupFontSameAsInstalledFont(): Hack-BoldItalic.ttf
Checking for font name in registry: Hack Bold Italic (TrueType)
Font not found in registry!
IsSetupFontSameAsInstalledFont(): Hack-Italic.ttf
Checking for font name in registry: Hack Italic (TrueType)
Font not found in registry!
IsSetupFontSameAsInstalledFont(): Hack-Bold.ttf
Checking for font name in registry: Hack Bold (TrueType)
Font not found in registry!
IsSetupFontSameAsInstalledFont(): Hack-Regular.ttf
Checking for font name in registry: Hack (TrueType)
Font not found in registry!
IsSetupFontSameAsInstalledFont(): Hack-BoldItalic.ttf
Checking for font name in registry: Hack Bold Italic (TrueType)
Font not found in registry!
IsSetupFontSameAsInstalledFont(): Hack-Italic.ttf
Checking for font name in registry: Hack Italic (TrueType)
Font not found in registry!
Changes detected, require reboot

And the issue is?

@MeganOveLee and @texhex, I recommend closing this issue.


callback to dynamically load CA certificate by "name"

I'm thinking about adding a callback called during certificate chain validation thats called when the CA certificate is not found on the internal list.  This would allow all, or most, common CA certs to be stored say in SPIFFS and then only the one needed to validate a specific connections can be loaded on the fly without have to specify it ahead of time.  Would that be an acceptable change?  Any other ideas on how this could be done?
Update: After some more looking, if I don't treat SSL_CTX as opaque then this can be handled in the Arduino layer....
Update: strike the above - really should stay opaque.  Now thinking to add a function to return the name CA) on the last certificate in the chain.  That can be called before validateChain() to make sure the correct CA cert is loaded and added to the context.

N/A with bearssl integration.


Add tail hedge strategy [6h]

97% of portfolio in spx
3% in 60-70 dte 30% otm puts
sell puts 30 days before expiration

This is taking longer than planned. Had to refactor Portfolio to accomodate two different data sources.
Also, some issues surfaced:

Does the portfolio need rebalancing every n days?
Strategy & allocation seem related
Options data handler gives only latest day's data


How do I properly adjust rectangle's width after container resize?

I'm trying to deal with responsiveness, and I'm creating a complex Gantt with pixi.
Whenever the container is resized and the stage's width changes, I'm trying to resize all my rectangles in the Gantt accordingly, so they'll still be in that correct place timeline-wise.
When I tried to simply change their scale, it wasn't enough, because their position had to be changed too.
Changing their width seem to change their scale as well, so is there another way to do this?

extra containers. Or you can rewrite transforms and how width/height is defined, PIXI tree is based on Flash stage, its not suited for many tasks. Its ok to rewrite it once you know how it works :)

Closing as you have an answer! Please create a new issue if there is anything else you need help with. Thanks!


Bower support

Is it possible to add bower support for this library, it is very nice and works very well.

Hi @HarelM. Thanks, glad you've found it useful! I'm afraid I don't use Bower. If you have a chance to create a pull request I would be happy to take a look though?

Where do you keep the generated css file? The demo page is referencing raw github location but I could not find it in the repository. Also I don't see any releases here at Github... If you create a release with a css file inside I think I can manage to create a bower package from it.

Thanks @HarelM. I'll try to get round to this on Friday and give you a ping.

@HarelM I've added the compiled CSS and done a v1 release :)
https://github.com/samrayner/bootstrap-side-navbar/blob/source/source/assets/stylesheets/navbar-fixed-side.css
https://github.com/samrayner/bootstrap-side-navbar/releases/tag/v1.0.0

Sorry for nagging, can you make another release called v1.1 or v1.0.1 that has the bower.json file in it?
This repository is now registered but when using bower to get it the bower.json file is missing so bower does not know which file to serve the developer.

https://github.com/samrayner/bootstrap-side-navbar/releases/tag/v1.0.1
If that doesn't work it might be because it needs to be on the master branch? The default branch for this repo is source.

It worked like a charm! Thanks!


HTTP status code from Docker containers

Hi @msoap
Thanks for this tool.
I have the same use case described in #51. I followed your solution but I only get 200 response when running it in a Docker container.
docker run --name shell-server --rm -p 8080:8080 shell2http-with-scripts -show-errors -cgi -include-stderr /health './error.sh > /dev/null || echo "Status: 503\n\nError\n"' /ok './ok.sh || echo "Status: 503"'

The response for the above:
curl -v localhost:8080/health
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8080 (#0)
> GET /health HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.54.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Server: shell2http 1.13
< X-Shell2http-Exit-Code: 0
< Date: Tue, 31 Dec 2019 11:55:39 GMT
< Content-Length: 23
< Content-Type: text/plain; charset=utf-8
<
Status: 503\n\nError\n
* Connection #0 to host localhost left intact

Do you know why?

I figured it out. Pasting here if anyone ends up with the same issue.
echo may behave differently depending on the OS. So it's not reliable. We should use printf instead.
Docker shortened sample equivalent to @msoap solution in #51
docker run --name shell-server --rm -p 8080:8080 shell2http-with-scripts -show-errors -cgi /health './error.sh > /dev/null || printf "%s\n\n%s\n" "Status: 503" "Error"'


Mark resin.logs.clear() & resin.logs.historySinceClear() as deprecated

Change-Type: patch
Connects-To: #527
In preparation for #525 for v9
---- Autogenerated Waffleboard Connection: Connects to #527
Question: Should I regenerate the docs myself or will it be done automatically on release?

@flesler Can you rebuild the docs please?

@LucianBuzzo I had asked above if I should or not. Now that I was able to fix the build locally I just did. All good now?

@thgreasi deleted the test in another commit


Blazor Getting Started... SSL Connection could not be established

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/create
Operating System: macos
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
I am using OSX Sonoma - 14.3.1 and I recently performed a clean up of old applications, deleting Visual Studio and old versions of dotnet, leaving only dotnet 8.0.101.
Attempting to create BlazorApp per instructions results in errors...
prefix: "/usr/local/share/dotnet/sdk/8.0.101/NuGet.targets(156,5): error :"
----Unable to load the service index for source https://api.nuget.org/v3/index.json.
----The SSL connection could not be established, see inner exception.
----Unable to write data to the transport connection: Broken pipe.
----Broken pipe
I performed dev-certs clean and trust several times to refresh the certificates and each time did a check to ensure the certs were cleared and renewed.
I considered refreshing my dotnet 8 installation.  I tried to uninstall dotnet 8 with the uninstaller, but the uninstaller does not support uninstalling dotnet 8 or above
I tried creating a simple console app in the cli to see if this problem was isolated to blazor.  I received the same errors as above.
I wanted to try deleting everything in the /usr/local/share/dotnet directory, but I am concerned that may cause my problem to get worse.
Any suggestions or help will be greatly appreciated.
Thank you

@danroth27 can you help with this one?

It looks like connecting to nuget.org to restore the packages failed. Adding @JonDouglas to help out on the NuGet side.
@sealamb Are you able to browse to https://api.nuget.org/v3/index.json without issue?

@danroth27 Yes, I am able to open the link in a browser without issue.
After seeing this may be a nuget issue, I attempted  "dotnet netget  trust" with the following results, which are similar to what I had reported before:
$dotnet nuget trust list
There are no trusted signers.
$dotnet nuget trust source NuGetTrust --source-url https://api.nuget.org/v3/index.json --owners "Nuget;Microsoft" -v d
X.509 certificate chain validation will use the fallback certificate bundle at '/usr/local/share/dotnet/sdk/8.0.101/trustedroots/codesignctl.pem'.
X.509 certificate chain validation will use the fallback certificate bundle at '/usr/local/share/dotnet/sdk/8.0.101/trustedroots/timestampctl.pem'.
error: Unable to load the service index for source https://api.nuget.org/v3/index.json.
error:   The SSL connection could not be established, see inner exception.
error:   Unable to write data to the transport connection: Broken pipe.
error:   Broken pipe

SOLVED ...
I had the guts this morning to delete the /usr/local/share/dotnet directory and files.  I installed a new the dotnet sdk from the Blazor Tutorial page and verified thet 8.0.101 was installed.  I attempted to create a blazor app and came up with the same errors as before.  Therefore, I see the problem is with my system not dotnet.
 It Worked!   I am able to build dotnet applications now on this machine.
Thank you for your help and quick response.

@JonDouglas should we move this to the nuget repo?

Seems resolved, but yes for future NuGet source problems our repo is a great place.

TDLR - in my bash terminal from windows using Rider I ran
dotnet nuget trust source NuGetTrust --source-url https://api.nuget.org/v3/index.json --owners "Nuget;Microsoft" -v d
and suddenly my issues are fixed.


Fix waifu selector dropdown titles

Not much to say about this.


odd behavior of ndarray

Is this expected or by design?

import numpy as np
import mxnet as mx
x = np.array([0, 1, 2, 3, 4, 5, 6, 7, 8, 9])
y = mx.nd.array([0, 1, 2, 3, 4, 5, 6, 7, 8, 9])

x[0:0].shape
(0,)
y[0:0].shape
(10,)

@sxjscience

Fixed now.


Auto filter Row

Hi,
I'm new in FLAUI.
i'm creating an automated UI project and I have a problem with a grid and the Auto filter Row.
If I run the application and search the Grid definition, I saw the Next:

My Grid have 3 rows, (header, Row0 and Row1)
But if I see my grid in the application I have 4 rows

(Header, FILTER ROW, Row0, Row1)
I can't see the Filter row in the inspect tool until i clicked on it in the applciation:

So... How can I access to this filter row (I need to filter for an specific text) if the object do not exist so i can not click on it...

Depending on your grid-control it might be that the filter row is not visible (no control at all) until it is activated (clicked). If that is the case you could try to  just click 10 pixels under the header row in order to activate the filter control, then get the filter control as edit (probably with a small Retry or at least delay) and edit the value in it.


Release Google.Cloud.Asset.V1 version 3.10.0

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit cba7759)

Created release for Google.Cloud.Asset.V1-3.10.0


Add protocol version feature

Provides a backward compatibility with version 3 protocol

It is cool that it is done as a feature :+1:

@ernestas-poskus yeah, so far so good

Fantastic!


[DPE-5387] Grants priviledges to non-public schemas

Issue
When the relation's user is created, we grant them privileges to objects inside the database/schemas, but not to the schemas themselves. For example, when we unrelate/re-relate to the charm, all non-public schemas ownership doesn't update accordingly. Moreover, new users weren't being specifically granted USAGE and CREATE access to the non-public schemas.
Solution
Grant all necessary privileges explicitly + update unit tests accordingly.

LGTM, but Q: will this work for two users accessing the same database?
juju deploy postgresql...
juju deploy data-integrator diA -config database-name=test123
juju deploy data-integrator diB -config database-name=test123
juju relate postgresql diA:database
juju relate postgresql diB:database

Yes, it will. I tested it locally yesterday with two postgresql-test-app and their run-sql action.

LGTM, but Q: will this work for two users accessing the same database?

Yep! Ownership and USAGE/CREATE permissions are handled accordingly depending on the relate order, and everything works as expected.


DSEGOG-8 user can reorder columns

Description
Note: this contains resizing columns functionality so probably best to review that PR first - https://github.com/ral-facilities/operationsgateway/pull/4
Adding functionality to allow the user to reorder the table columns using drag and drop functionality provided by react-beautiful-dnd. The result is a nice looking, simplistic method by which the user can reorder the displayed columns without having to add or remove columns to achieve the same thing. To drag and drop, the user can click anywhere on the column header that isn't the close button or the divider. I had to be a bit clever to make sure their functionality didn't overlap.
It's not really feasible to test this functionality outside of a browser so I used this opportunity to add Cypress tests to the project. I've only done tests for this functionality so far, not made any for the previous functionality I've done yet. Currently using Cypress 9.6.0 to match DataGateway because this was the simplest to get set up with. I considered upgrading to Cypress v10 but this contains many breaking changes so I made a separate issue for this to tackle later.
Testing instructions
Experiment with adding columns to the display and then reordering them by dragging the headers. Experiment with both large and small tables to make sure the functionality works in all cases. Also make sure that the drag and drop functionality does not interfere with column resizing and closing.

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage

Agile board tracking
Closes DSEGOG-8

@louise-davies apologies for the length this PR has turned out as but it is now ready for review again. It includes a lot of optimisations that improve the maintainability of things going forward.
The gist of what I've done since I marked this as draft is fixing a bug which meant, when removing a column and adding it back, it went back into the same place it was removed from instead of being added to the end of the table as expected. I also added some code that more efficiently manages the column order and the table state overall. Please try and test as much of the functionality as possible to make sure everything plays nicely with each other.
Although this is a lengthy PR, remember that much of it is also taken up by yarn package additions and a couple of big snapshots!


Could not broadcast to retrieve funds. What to do?

having the same issue.

Any luck?

Same thing happened to me but thanks to Blackhathacker111 who helped me recover all my funds,message him on telegram his telegram username is https://t.me/Blackhathacker111

Any luck?
Same thing happened to me but thanks to Blackhathacker111 who helped me message him on telegram his telegram username is https://t.me/Blackhathacker111

Same thing happened to me but thanks to Blackhathacker111 who helped me get recovered my lost funds,message him on telegram his telegram username is https://t.me/Blackhathacker110

Figured it out finally for Ethereum. Download mycrypto.com Desktop application. Then put in your Bitpay Seed when creating a new wallet. After that, just go through the pages until you find the wallet with the ETH on it, load it, and send the ETH to a different receiving address.
If your funds are stuck and you're using BTC, then I'm pretty sure you can import your Bitpay Seed into Electrum and use the console to increase the gap limit so that Electrum can pull in the funds from the addresses.
Do that if the mycrypto wallet doesn't support BTC the same way it does ETH.


Ignore files during pyb install

Hi,
I typically use vim, which creates backup files of the form .<filename>.swp. In general, I find this a useful feature and I wouldn't necessarily want to disable it. Yet with pybuilder projects, it tends to become a little annoying from time to time, because I can't install the script I currently have open in vim into my current virtual environment. Instead, I have to first close vim and then install the script. If I don't do that, pybuilder complains about missing or invalid encoding declarations for the swp files. Here is an example:
BUILD FAILED - Error while executing setup command ['bdist_wheel']. See /home/ingo/code/eksam/target/reports/distutils/bdist_wheel for full details:
	    self.run_command('build')
	  File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
	    self.distribution.run_command(command)
	  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
	    cmd_obj.run()
	  File "/usr/lib/python3.6/distutils/command/build.py", line 135, in run
	    self.run_command(cmd_name)
	  File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
	    self.distribution.run_command(command)
	  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
	    cmd_obj.run()
	  File "/usr/lib/python3.6/distutils/command/build_scripts.py", line 50, in run
	    self.copy_scripts()
	  File "/usr/lib/python3.6/distutils/command/build_scripts.py", line 82, in copy_scripts
	    encoding, lines = tokenize.detect_encoding(f.readline)
	  File "/usr/lib/python3.6/tokenize.py", line 431, in detect_encoding
	    encoding = find_cookie(first)
	  File "/usr/lib/python3.6/tokenize.py", line 395, in find_cookie
	    raise SyntaxError(msg)
	SyntaxError: invalid or missing encoding declaration for 'scripts/.eksam-cli.swp'

Is this the intended behaviour? Is there a way to set files/glob patterns to be excluded from the installation? If so, I might need a little pointer here. Thanks for the support and for pybuilder in general.

What does distutils say about this? Than we can look at how to configure PyBuilder?

I'm not quite sure if I understand your question? Do you mean that this might be an issue in distutils?

Got it. Wasn't aware it were the ./scripts files.
I just checked the code for you (core_plugin.py#65) and pybuilder simply lists all files in the scripts folder. There is no support for a glob pattern or something.
You'll need a PR for this.

Ok, thanks. And thanks for pointing out where. I'll take a look and see if when I have time to submit a fix.

Hi, @igordertigor.
If You will have time for fix have a look into #476. Maybe You just need to rebase and modify it.
Thanks.

That referenced PR would fix your issue as well, no?
I'm not sure if you can simply default to ignore hidden files. Shouldn't this be configurable like an exclude_from_scripts_glob property?

So, I read through most of the stuff regarding this issue. My take is:

we don't need to recurse into subdirectories, we assume there are no subdirectories in scripts or whatever it might be called.
there is no need to even look at directories in scripts

The pull-request in #613 would probably work fine, but I think the statement proposed by @AlexeySanko in #476:
 if os.path.isfile(os.path.join(scripts_dir, script)) and not HIDDEN_FILE_NAME_PATTERN.match(script):
                yield script

is the correct and most pythonic one. Hence I will close #613 and open a new pull-request with those changes.

I did some more research and it turns out, that the problem isn't the fact that this is a hidden file, but that the file has an invalid encoding for distutils. For example, if I have a file .eggs in src/main/scripts which simply contains a print('Hello World!') statement, it will work. So, I am not even sure if skipping hidden files is the correct solution, because the problem isn't that the file is hidden, but that it is binary.

Fix and test now in: #624


Transitions for 3/4 and 3/2 track speeds.

Will likely need to rethink how track transitions works.
We currently calculate next transition in 16th step resolution.
This won't work for 3/2 and 3/4 tracks because they run at 0.75 and 1.5 speed.
For example:
Pattern length is 16 steps. At 3/4 speed, the next transition will occur on (1/0.75) * 16 == 21.333 steps.
Sequencer will go out of sync...

Proposed solution. Each track to have a transition_offset, measured in mod12 ticks. Transition start can be shifted by 0 -> 12 ticks

transition_offsets added.
Still going out of sync... with MD sequencer on transitions.

Transitions for
3/4x 3/2x now working via track offset method described above.
Question: Should the offset be reset to 0 if transitioning back to a standard 1x, 2x scale.


External script sourcemap

Thank you for adding support for external files. I'm probably in the minority, but it's also the structure I prefer and something I really like about vue-loader. It's a lot easier for IDEs to recognize individual files.
However, I'm hitting a roadblock with source maps. I've looked at the source that resolves external files. Here's my setup: I'm using pug and scss (which doesn't matter in this case). My component files have a structure like this,
<template src='./template.pug' lang='pug'></template>
<style src='./style.scss' lang='scss'></style>
<script src='./script.js'></script>

When I try to minify my code and put a breakpoint in script.js, I just end up back at this file instead of the script file.
I tried looking into how vue-loader does this, since they do something similar. However, it looks like they just convert these references to require calls and allow webpack to resolve and add the source map.
I then tried to find inspiration in rollup-plugin-commonjs, since it allows for resolving require statements (see transform.js). It uses the utility magic-string, which seems like it might help in this case.

Looks like preprocess maps aren't currently being used. From svelte's readme,

Note that sourcemaps are currently discarded, but will be used in future versions of Svelte.

So there's some upstream work to be done here. Nevertheless, I'm still finding it very hard to find a simple recipe or example of generating a source map for importing a file. I've poked around a number of places. Looking at the source-maps repo, it seems like this should be possible with SourceNode. I was also looking at test-source-map-generator.js or test-source-node.js for inspiration.

I still think this could be useful, but there's no use for sourcemaps in svelte for the time being. This issue will be kept closed until svelte actually does something with sourcemaps.


To do before release 0.6.0 / suggestions

[ ] Bump LV2 micro version
[ ] Bump libsfizz.so shared library version
[ ] Review the API doc comments ; there are cases of mismatched @since tags between h/hpp, check which is correct and edit accordingly
[ ] Suggestion to change 7bit char arguments (velocity, aftertouch ...) to int.
[ ] Suggestion to have a tempo API based on BPM, the unit that SFZ uses, rather than "samples per beat". (+deprecate the latter?)
[ ] Have LV2 restore parameter to defaults, when not present in the save blob, for compliance to the spec

* [ ]  Suggestion to change 7bit `char` arguments (velocity, aftertouch ...) to `int`.

In the API?

* [ ]  Suggestion to have a tempo API based on BPM, the unit that SFZ uses, rather than "samples per beat". (+deprecate the latter?)

Sure, go for it. If we go for 1.0 now's the time to deprecate stuff.

There was the JSON documentation API files Python script generator (#164), though I think is not a priority now, the files can still be tweaked manually.

https://github.com/sfztools/sfizz/wiki/To-deprecate-on-next-API-ABI-break also !


Automation APIs

General issue to track the implementation of the Automation API https://qlik.dev/apis/rest/automations

closed with 10b16192205780658b31ac285af68a609239ddfe


fix: missed profile avatar on web

Why
I just found that we missed the profile avatar on the desktop because this PR missed passing the user prop to the header-right.tsx file.
cc @intergalacticspacehighway @hirbod
How
changed the type and passed the user prop to the HeaderDropdown component in the header-right.tsx file
Test Plan
Before

After

because already affect the product so have to deploy it first!

Good catch.


Use another buttons style: selective properties

Is this a bug in companion itself or a module?

[X] I believe this to be a bug in companion

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
in beta build (companion-2.2.0-ce31f8c9-3694-win64) when using feedback  to "use another buttons style" to set many buttons color it would take the color and text color. In the newer versions of the the beta build it takes all the button attributes including text value. So all buttons using the "use another buttons style" now all have they same button display name as the original button.
Steps To Reproduce

Set a button color and text color
add a new button
set new button feedback to "use another button style"
new button now uses original button name as well as color

Expected Behavior
use original button color and text color but not name, keep new button name.
Environment (please complete the following information)
- OS: Win10 64Bit
- Browser: Chrome
- Companion Version: companion-2.2.0-ce31f8c9-3694-win64

Additional context
No response

Could we also add feedback states that are conditional on a custom variable?
Use case is: I want to create a single button that will assess a custom variable, and then show the text from a range of specific buttons.
For instance, if custom variable = 1, use the (text, color, background etc) from bank 1 button 1.
If custom variable = 2, us the  (text, color, background etc) from bank 1 button 2.


VSCode plugin does not provide a fully working replacement

Describe the bug
When using the VSCode plugin and triggering a rule, the offered replacement isn't parsed properly.
When DS144886 is triggered for instance, I get a replacement suggestion of +
So: $_REQUEST would need to be replaced with $_GET but the suggestion is $_REQUESTGET (screenshot attached).
I don't think it's the rule itself, but rather a plugin issue?
To Reproduce
Steps to reproduce the behavior:

Create a new PHP file in VSCode
Write <?php  $_REQUEST['id'] = 1;
Place cursor at beginning
See that the wrong replacement is suggested ($_REQUESTGET, $__REQUESTPOST) instead of $_GET and $_POST

Expected behavior
$_GET and $_POST being suggested.
Screenshots

Versions(please complete the following information):

OS: Windows 11 (10.0.22631)
Devskim Version 1.0.33
VSCode 1.89.0 (2024-05-01)

Thanks for the report. Definitely a bug. I suspect with the replacement pattern in the rule.

I was able to dig into this a bit today and was able to replicate the same behavior using the analyze/fix commands in the DevSkim CLI.
For context, the extension acts as a pretty thin wrapper around the language server which uses the same APIs as the CLI to perform analysis and generate fixes. Fixes are generated by this method from a code snippet and rule - even for fixes consumed by the extension: https://github.com/microsoft/DevSkim/blob/ba39d7aba721156f4b8fdeb11b93a36b3ef963b4/DevSkim-DotNet/Microsoft.DevSkim/DevSkimRuleProcessor.cs#L85-L102.
The issue appears to be with the $_ sequence in the replacements specified in the fixes for this rule.  https://github.com/microsoft/DevSkim/blob/ba39d7aba721156f4b8fdeb11b93a36b3ef963b4/rules/default/security/frameworks/php.json#L25-L38
Since the RuleProcessor generates fixes using the Regex.Replace API the replacement text is interpreted by the Regex engine and substitutions are applied. By coincidence, $_ means 'insert the entire matched sequence' from the Regex.
The fix was to update the rule to add an extra $ so the $_ sequence is interpreted literally and not as a substitution.  See #616.

Thanks @martijn-coolminds for confirming!


Auth

Auth is low priority and should be implemented only when the core functionality is done. However, thinking about it in the current stage is not a bad idea.
For the scope of the app, it requires a very minimal amount of Authentication and almost no Authorization. User data is just another way to say cloud-saved reader configs and maybe bookmarks or notes in the future.
 Yes I remember the old adage, but for this app I think it's simple enough for this to be justified.
Use an identity manager like Zitadel (pretty overkill)
Throw turso always, and use Pocketbase with something like Litestream for backup

I guess a better direction to take is to start with Pocketbase first and think about implementing auth later.
It shouldn't be hard to migrate, our entire database is basically just 2 files anyways.


Resolve formulae in brew cleanup.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Resolve formulae in brew cleanup.
Fixes https://github.com/Homebrew/brew/issues/4856.

@xu-cheng, can you try this?

@xu-cheng, can you try this?

Yes, it fixes the problem for me. Thanks for the fix.


Find all not working

const client = new Client();
await client.open('redis://localhost:6379');

let tokenAPiRepository = client.fetchRepository(tokenSchema);
return await tokenAPiRepository.search().return.all();

i get this error:
ERR unknown command `FT.SEARCH`, with args beginning with: `TokenApi:index`, `*`, `LIMIT`, `0`, `10`,
any idea to fixe that ?

This is most likely the result of not having a version of Redis with RediSearch installed. What version of Redis are you running? Where is it running?

@guyroyse
"redis": "^4.0.6",
"redis-om": "^0.2.1"

What version of Redis Server are you running? Where is it running?

@guyroyse it s from current https://hub.docker.com/_/redis

That's the vanilla version of Redis. You need to run Redis Stack or another flavor of Redis that has RediSearch installed. Details on how to get it are here: https://redis.io/docs/stack/. Docker is an option.

@guyroyse for docker + redisstack i found this https://redis.io/docs/stack/get-started/install/docker/
and thank u for answers

Hrm, I'm definitely using RedisStack:
services:
  redisdocker:
    image: redis/redis-stack-server:latest
    restart: "unless-stopped"
    ports:
      - 6379:6379
    volumes:
      - "./apps/backend/storage-redis/data:/data"

  redisinsights:
    image: redislabs/redisinsight:latest
    restart: "unless-stopped"
    ports:
      - '8001:8001'
    volumes:
      - "./apps/backend/storage-redis/redisinsight:/db"

. . . but I'm having the same issue.  When I scan "myRepository:*" via RedisInsight, I have a few hundred items, but when I query via redis-om, less than 100 show up.  I did dramatically adjust the "schema" in the last few days, but I only added to.   Could re-indexing cause this?  🤔

I think this might be in the wrong issue, @humblecoder. My guess, however, would be indexing failures. Try running FT.INFO myRepository:index and look for the indexing failures.


Add About Us webpage

Might want to support individual pages like on the airlab. Could have lab/contributors drop down.


Adding -iree-llvm-linker-path= and enabling LLD in our LLVM configuration.

This doesn't yet depend on lld anywhere in cmake land/have it in an install lists (I still need to find that) - just makes it available as a cmake target. So hopefully no breakage :)

This doesn't yet depend on lld anywhere in cmake land/have it in an install lists (I still need to find that) - just makes it available as a cmake target. So hopefully no breakage :)

Yeah - more interested in whether this pushes us over quota on CI systems :)


Upgrading spotbugs soon?

Are you going to upgrade the spotbugs version soon? Spotbugs 4.1.3 fixed https://github.com/spotbugs/spotbugs/issues/756, which is pretty annoying.

You can always manually specify which version of spotbugs you would like to use:
spotbugs {
  toolVersion = '4.1.3'
}

Sorry, my bad, it even says so right in the README. Thanks!

No problem. I think it is still worth doing, just was mentioning that in the mean time you can use a more recent version to fix the issue.