Xaxis annotations not rendered when the label does not show.

Hello,
By design Apexcharts hides xaxis labels to show a more pleasing xaxis.
But if I add an annotation to a specific label that is not displayed, the annotation does not show.
Here in the code pen you have two annotations:

19 Nov 2017 (this does not show)
24 Nov 2017 (this one show)

https://codepen.io/ines-pereira/pen/PojdoPO
Thanks for your help.

In a category chart, it is not possible to determine the annotation position as the x-axis is not linear. The string can appear anywhere on the x-axis and hence ApexCharts uses actual label.
If you use a timeline or a numeric x-axis, the annotation will show even if the actual label is not present on the x-axis.


Add a flag to disable default launch plan generation

TL;DR
Adds a flag to control whether a default launchplan should be generated for the workflow at registration time. The default behavior is to do so.
E.g.
@workflow_class(disable_default_launch_plan=True)
def MyWorkflow():
   ...

my_lp = MyWorkflow.create_launch_plan(...)

#my_lp will be the only launchplan created for this workflow

Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[x] Code documentation added
[x] Any pending items have an associated Issue

Tracking Issue
https://github.com/lyft/flyte/issues/462

PTAL @wild-endeavor


Update instruction, install peerDeps, and remove dependencies

Update install instruction to match the Airbnb's readme
Automatically install peerDependencies from install script
Remove dependencies from package.json (force to install the peerDenpendcies instead)
Add Yarn instruction

Also fixes #7

Should we use caret versioning (^) for both peer deps and dev deps?
Pinned version for peerDependencies seems too strict for me.


stable/minio persistance.size

When I enable persitant storage and setup a size different than default value it keep the old disk(storage capacity) from the previous creation even if I purge it
It doesn't happen if I change the name
Client: &version.Version{SemVer:"v2.12.2", GitCommit:"7d2b0c73d734f6586ed222a567c5d103fed435be", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.12.2", GitCommit:"7d2b0c73d734f6586ed222a567c5d103fed435be", GitTreeState:"clean"}
How to reproduce
helm install --namespace dev --name miniotest --set mode=distributed,accessKey=toto,secretKey=tata,persistence.enabled=true,persistence.size=1Gi stable/minio
helm delete --purge miniotest
helm install --namespace dev --name miniotest --set mode=distributed,accessKey=toto,secretKey=tata,persistence.enabled=true,persistence.size=50Gi stable/minio
I am checking size using df -h
the /dev/sdi mount on /export is about the good size on the first creation but keep it's value on 2nd helm creation, if I change the name it will be ok.

Try deleting the persistent volume created by the persistent volume claim from this chart.
With Automatic Volume provisioning, a pv will be created when you create a pvc. If kubernetes already has a pv with the right name/tags which is not in use, it will use the existing pv.
Just run kubectl get pv to get an Overview.

Yeah it was that, I also deleted associated pvc
Thanks


Update to Swift 4.2

Updated code & project settings.
Bumped version to 0.2.0 and updated README, so 0.1.3 can be used for Swift 4.0

@AgapovOne Thank you!


Host filter doesn't exactly match

Greedy host filter
Steps to reproduce
eg. cap stage --hosts=system01
using a stage with defined hosts:
system011
system012
system020
Expected behaviour
empty host list or ERROR Host does not exists or sth. like this
Actual behaviour
matching any host starting with the hosts parameter
in this case
system011 and system012
Fix
from: https://github.com/capistrano/capistrano/blob/master/lib/capistrano/configuration/host_filter.rb#L17-L27
def regex_matcher(values)
        values.map! do |v|
          case v
          when Regexp then v
          else
            vs = v.to_s
            vs =~ /^[-A-Za-z0-9.]+$/ ? vs : Regexp.new(vs)
          end
        end
        Regexp.union values
      end

Line 23 needs to be changed from
vs =~ /^[-A-Za-z0-9.]+$/ ? vs : Regexp.new(vs)

to
vs =~ /^[-A-Za-z0-9.]+$/ ? %r{^#{vs}$} : Regexp.new(vs)

Addition
Wildcard matching should be used while directly using a regex in capistrano
eg. cap stage --hosts='system01\d'
using only a string it should match exactly

Yes, I believe this is a bug. The documentation implies that the strings should be matched exactly, unless certain regular expression syntax is used.
@leehambley @will-in-wi @cshaffer Do you agree this is a bug?

Also, I don't think %r{^#{vs}$} is exactly correct, since a . in the filter string will be interpreted as a matching any single character. Probably what we want is %r{^#{Regexp.quote(vs)}$}?

I agree that this appears to be a bug. Unless @cseeger-epages is interested in doing so, I can probably whip up a PR.

@will-in-wi Thanks. Can you take the lead on seeing this issue is ultimately resolved? I'll assign it to you if that's OK.

Sounds good to me. @cseeger-epages: Are you interested in making a pull request for this? You already found the offending line. The only additional thing I would suggest would be to make an rspec test that replicates this issue, and another one which checks for the case when what is passed contains a dot.

We tested the better solution from @mattbrictson
[1] pry(main)> vs = "wl"
=> "wl"
[2] pry(main)> %r{^#{Regexp.quote(vs)}$}
=> /^wl$/
[3] pry(main)> vs = "wl.example.com"
=> "wl.example.com"
[4] pry(main)> %r{^#{Regexp.quote(vs)}$}
=> /^wl\.example\.com$/

i will make a pull request during the day for this. Thanks for your fast replys.
Additionally:
The same is code is used in the roles_filter:
https://github.com/capistrano/capistrano/blob/master/lib/capistrano/configuration/role_filter.rb#L17-L27
def regex_matcher(values)
        values.map! do |v|
          case v
          when Regexp then v
          else
            vs = v.to_s
            vs =~ %r{^/(.+)/$} ? Regexp.new($1) : /^#{vs}$/
          end
        end
        Regexp.union values
      end

should this also be fixed to %r{^#{Regexp.quote(vs)}$} ? if someone wants to define some roles using a .  maybe "somerole.somesubrole"

@mattbrictson might weigh in here, but I'd say that the roles_filter fix makes sense too. If you want to make that PR, could you do it as an additional one?
Thanks!

Ok if @mattbricson gives his ok i will make an additional one.

Yes, I'll accept a PR for fixing the roles filter as well. Thanks!

should i open a new issue for this or is it ok to to reference it here ?

I'd just push the PR and reference it here. If @mattbrictson wants a new issue, then it can be created at that point.

Ok thx. If both PR's get merged can someone fix the changelog, theres no need for 2 lines for the same issue.

This can be closed after #1771 is merged (once merge conflicts are fixed).

Fixed via #1771


Accounting Budget:

I am getting many charts, and may be bringing other taxpayers charts into view. Limit to only bring charts related to my TaxPayer and CycleID

Same issue detected in Closing and Opening Balance. We need to check query and database data to see where the error is coming from.


[8.x] Add contains_all validation rule

I have recently been finding it more common to have to validate that certain items are available in an array. However, the items that are required are usually dynamically driven and then stored as JSON in a DB column.
A really simple example of this might be shifts for a team. Each team has working days which are pre-defined and stored in a database. There is then a dynamically generated form which asks for the shift times for these days.
The required data for each shift would be the same, a start time and an end time and would be required. Typically, to validate this dynamic data, you might do something such as:
public function rules() {

$dayRules = BusinessHourDay::sameCompany()->get()->map(function ($businessHourDay) {
  return [
  'days.' . $businessHourDay->day . '.start_time' => 'required',
  'days.' . $businessHourDay->day . '.end_time' => 'required',
  // more rules
  ];
});

return [
  'name' => 'required',
  'description' => 'required'
  'days' => 'array',
  ...$dayRules,
];

This allows us to validate the days that the user has configured in the database are present in the dynamically generated form.
This proposed change would allow this instead:
public function rules() {

$dayRules = BusinessHourDay::sameCompany()->get()->pluck('day');
return [
  'name' => 'required',
  'description' => 'required',
  'days' => [
  'bail',
  'array',
  'contains_all,' . $dayRules->implode(','),
  ],
  'days.*.start_time' => 'required',
  'days.*.end_time' => 'required',
  // more rules
];

This would simplify the validation in the cases where mapping a list of validation rules, which would have the same rules for each entry, but we still want to assert that all the required data was present.
The rule could also be applied to a string field to make sure a part or parts of it given match a particular structure, for example, a form where you might enter an invoice "number" something like "INV:12345-WEB"
public function rules() {

return [
	'invoice_id' = [
		'required',
		'contains_all:INV,WEB',
	],
];

Would be grateful for any feedback or thought you have on this :)
Thanks guys.

I'm curious if we need any additional functionality in the framework for this. It seems you could do:
return [
  'name' => 'required',
  'description' => 'required',
  'days' => 'bail|array',
  ...BusinessHourDay::sameCompany()->get()->flatMap(function ($day) {
      return [
          'days.'.$day->day.'.start_time' => 'required',
          'days.'.$day->day.'.end_time' => 'required',
      ];
  }),
];

I think I'm a having a bit of a hard time following the rule. Can you give another example of using it - you also use contains_all and contains in your examples. I'm not sure if those are for the same rule?

My intended approach here is to handle the times when you might have a some simple validation requirements, like just making sure it has some value using the required validation rule, without having to add any "complexity" to the code. Has the array got all of these keys? Yes, great, if not fail.

Can you give an example of this? Ideally without any loops - just hard-coded strings to make it very clear.

Apologies, it should be contains_all in my examples - I originally named the rule contains and then realized that this was not clear, as it is checking to see if all keys are present, so changed it to contains_all and forgot to update my write-up.
I am just going to cook my kids dinner, and then I will write up a clearer example for you to review :)

If you had an array of questions which you wanted to be asked in a form, you might have something like this:
$questions = [
	'favourite_color' => 'What is your favorite color?',
	'favourite_animal' => 'What is your favorite animal?',
	'maiden_name' => 'What is your Mother\'s maiden name?',
	'favorite_php_framework' => 'What is your favorite PHP framework?',
];

(This would likely be database driven, or a static array on a model, but the method of which we get it is not really relevant to this.)
You could use these to generate a form in a blade file using the key as the input name in an array.
@foreach($questions as $question => $key)
<input type="text" name="questions[{{ $key }}] value="{{ old('questions.' . {{ $key}}) }} ">
@endforeach

When it comes to validation, with these, I don't have to care much about the content, only that the user has answered something for them.
I would build a rule set in a request validator by looping through the array which would end up looking something like this:
public function rules()
{
	return [
		'questions' => 'array',
		'questions.favorite_color' => 'required',
		'questions.favourite_animal' => 'required',
		'questions.maiden_name' => 'required',
		'questions.favorite_php_framework' => 'required',
	];
}

The rules are the same, and I would have had to flatMap or some other form of loop to build up this rule set. Whereas my approach with the contains_all would be to have a rule set looking like this:
public function rules()
{
	return [
		'questions' => [
			'array',
			'contains_all:favorite_color,favorite_animal,maiden_name,favorite_php_framework',
		],
		'questions.*' => 'required',
	];
}

The contents of the contains_all would be imploded from the array.
My belief here is this approach is easier when you are trying to keep validation really simple.

Gotcha. I wonder if something like contains_keys would be a clearer name? Or maybe requires_keys so it's clear that all of them are required?

I feel like array_keys would make sense. I had it like this so the behaviour could be applied to a string as well, as most other rules seem to work across data types, but I am not sold on my string implementation. I feel pulling this back to just work with arrays seems like a sensible move.

I have pushed some new commits to rename the rule to array_keys and also makes the validation fail if the subject is not an array, also removing the validation rules for a string.

@Drewdan @taylorotwell just to make sure you have not forgotten that the “array” rule can actually already take in parameters and if I am remembering correctly already check for precedence of keys in the array?
Could it be that the array rule could be helpful in this case as well, sense you already know the specific keys?

@morloderex - as I understand, the array validation method accepts a list of valid keys, but this does not validate that they are present. This rule differs in that it validates all keys are present in the array. Ideally, you would use these in tandem with each other.

Renamed to required_array_keys.

@morloderex ... @Drewdan is correct that array rule only accepts a list of valid keys but does not validation that they are present.

@taylorotwell renaming to required_array_keys makes sense, it is a much clearer rule name :)


Rate limiter should clear edge counters that have a zero RPS

Otherwise, the list is going to keep growing and create an unnecessary overhead.
cc: @Raynos @kriskowal @jcorbin

@ShanniLi great idea :)

@ShanniLi EdgeCounter is a prototype right ? ;)

@Raynos live and learn :)


runtime: intermittent TestCgoPprof failures since 2021-09-28

step 1
step 8
step 9
step 10
step 11
step 12
step 13
step 14
--- FAIL: TestCgoPprof (12.23s)
    crash_cgo_test.go:344: [/workdir/go/bin/go tool pprof -traces /workdir/tmp/go-build4045495910/testprogcgo_.exe /workdir/tmp/prof1465180680]:
        File: testprogcgo_.exe
        Build ID: 9a7aae8c544e237851f2d51d3d04179d2be73832
        Type: cpu
        Time: Nov 5, 2021 at 12:05am (UTC)
        Duration: 203.77ms, Total samples = 10ms ( 4.91%)
        -----------+-------------------------------------------------------
              10ms   runtime.cgocall
                     main._Cfunc_cpuHog
                     main.CgoPprof
                     main.main
                     runtime.main
        -----------+-------------------------------------------------------
    crash_cgo_test.go:352: cpuHog traceback missing.
    crash_cgo_test.go:344: [/workdir/go/bin/go tool pprof -traces /workdir/tmp/prof1465180680]:
        File: testprogcgo_.exe
        Build ID: 9a7aae8c544e237851f2d51d3d04179d2be73832
        Type: cpu
        Time: Nov 5, 2021 at 12:05am (UTC)
        Duration: 203.77ms, Total samples = 10ms ( 4.91%)
        -----------+-------------------------------------------------------
              10ms   runtime.cgocall
                     main._Cfunc_cpuHog
                     main.CgoPprof
                     main.main
                     runtime.main
        -----------+-------------------------------------------------------
    crash_cgo_test.go:352: cpuHog traceback missing.
FAIL
FAIL	runtime	51.987s

greplogs --dashboard -md -l -e '(?ms)FAIL: TestCgoPprof.*cpuHog traceback missing' --since=2020-06-16

2021-11-04T23:56:29-0e5f287/linux-amd64-buster
2021-11-02T21:18:44-80065cf/linux-amd64-staticlockranking
2021-11-02T16:12:23-599de4b/linux-amd64-sid
2021-10-14T21:08:35-440b63b/linux-amd64-staticlockranking
2021-10-06T19:45:35-e38ec96/linux-ppc64le-buildlet
2021-10-04T17:17:11-cc5e3de/linux-amd64-longtest
2021-09-28T20:54:26-b8a6017/linux-amd64-jessie

The symptoms of these failures match #37201, but that appeared to have been fixed in May 2020.

This issue was forked from https://github.com/golang/go/issues/49065#issuecomment-961970906. In https://github.com/golang/go/issues/49065#issuecomment-962149940, @rhysh replied:

@bcmills this particular kernel bug wouldn't explain the failure on linux-amd64-longtest on October 4 (well before October 18).
Yes, those failures look like what's described in #37201. That issue closed when a particular source of signals was eliminated; there may be another (less frequent) source of signals.
The October 4 failure shows Duration: 200.88ms, Total samples = 10ms ( 4.98%). Stopping the CPU profiler seems to involve 200ms worth of sleeps, so I take that to mean that it got stopped almost immediately, and due to the "traceback function called 2 times"  condition rather than "test ran for one second" (https://github.com/golang/go/blob/cc5e3de593afca73cf1b4d732ddceffb2837b390/src/runtime/testdata/testprogcgo/pprof.go#L89). Samples covering 10ms, at 100Hz, means 1 sample. That's consistent with the process receiving signals in addition to SIGPROF, causing the test to exit early.

Marking as release-blocker, since this appears to be a regression during the Go 1.18 release cycle.
It could be a regression in the runtime's signal behavior, a new source of signals added to the test, or a test that was (still) already inherently flaky tickled by some subtle scheduling or preemption change (although it appears to predate @mknyszek's GC pacer changes 😅).

CCing @cherrymui as our resident pprof owner.

I'll take a look at this one as well.

I have change the following line to an infinite loop, then use gdb conditional breakpoint to observe the received non-27 signal
https://github.com/golang/go/blob/02d7eab52796574b44717d45d5def42c9068b56a/src/runtime/testdata/testprogcgo/pprof.go#L89

(gdb) b x_cgo_callers if sig != 27
Breakpoint 1 at 0x4fe320: file gcc_traceback.c, line 23.
(gdb) r
Starting program: /mnt/d/gopath/src/issues/issue49401/testprogcgo CgoPprof

It has been running two days for new, still nothing happen...

Trybot failure: https://storage.googleapis.com/go-build-log/4b7308f5/linux-amd64-race_38da783b.log

The problem is clear to me now.
CL 324129 introduces the per-thread timers feature. Tt is judged in the validSIGPROF function whether the source of the SIGPROF signal meets expectations, and if it is unexpected, will be ignored. However, this judgment is called later than x_cgo_callers. This makes the condition C.getCpuHogCount() <2 exit early.
It affects almost all linux systems, but currently only amd64 and ppc64le have the CgoProf testing, so I will try to fix it on linux_amd64 first only for now.

@hanchaoqun I agree with your analysis, that seems like the cause of this issue. However, I don't think it is worth fixing by moving validSIGPROF into assembly.
There is nothing fundamentally wrong with calling cgo_callers in a signal that is ultimately dropped, the test simply makes the fragile assumption that most of the calls will be used for the pprof sample. Note that this test already has to disable async preemption because that also breaks this assumption.
IMO, we should drop this assumption from the test. I'll send a CL.

There is nothing fundamentally wrong with calling cgo_callers in a signal that is ultimately dropped,

Will that add bias to the profile, since the program will spend more time than necessary in cgo_callers itself? (I would think it would cause whatever is running on that thread to be slightly over-represented, since the time spent in cgo_callers will have to be made up for by keeping the thread runnable for slightly longer than it otherwise might be.)

Yes, continuing to call cgo_callers unconditionally for all signals will cause additional CPU usage vs calling it only when we are certain it is needed.
In a profile, this would make threads using lots of cgo to receive slightly more samples than others, and since (I'm pretty sure) SIGPROF is masked during signal handling, those extra samples will simply appear somewhere in user code rather than cgo_callers itself.
This is a form of skew, but note that even with something like https://golang.org/cl/363443 we can't eliminate it entirely because valid SIGPROF signals still need to call cgo_callers, making them more expensive than Go-only signals. In a similar vane, Go code with deep stacetraces are also "skewed" because it takes the signal handler longer to generate their tracebacks.
IMO, these small sources of skew are arguments for using a PMU-based profiler if you are really worried about them, rather than trying to make "constant-time" signal handlers.
On the other hand, the extra useless CPU usage, if statistically significant, is a good argument for avoiding calling cgo_callers in as many cases as we can determine it is not needed, which would include both invalid SIGPROF and async-preemption signals.

On the other hand, the extra useless CPU usage, if statistically significant, is a good argument for avoiding calling cgo_callers in as many cases as we can determine it is not needed, which would include both invalid SIGPROF and async-preemption signals.

I should clarify: it is a good argument for potential future work, not for addressing now.

On the other hand, the extra useless CPU usage, if statistically significant, is a good argument for avoiding calling cgo_callers in as many cases as we can determine it is not needed, which would include both invalid SIGPROF and async-preemption signals.

I should clarify: it is a good argument for potential future work, not for addressing now.

Yes, i agree drop this assumption from the test.


RBF Manager 2016

Makes call to QT in PySide 2 that do not exist in pyside 1
"setToolTipsVisible"
Brad Clark pointed out.

This issue was moved to mgear-dev/rigbits#1


TypeError: Cannot read properties of undefined (reading 'mergeRequestParams')

this may be an issue with next.js is anyone has set something like this up for it
I am recieving the error TypeError: Cannot read properties of undefined (reading 'mergeRequestParams')
On initial generation with the --axios flag though I get the same error without it
Am I missing something in the setup.
When I console.log this in the method it is undefined but the api instance is defined and being used
  public request = async <T = any, _E = any>({
    secure,
    path,
    type,
    query,
    format,
    body,
    ...params
  }: FullRequestParams): Promise<AxiosResponse<T>> => {
    const secureParams =
      ((typeof secure === "boolean" ? secure : this.secure) &&
        this.securityWorker &&
        (await this.securityWorker(this.securityData))) ||
      {};
    console.log(this) // undefined
    const requestParams = this.mergeRequestParams(params, secureParams);
    const responseFormat = (format && this.format) || void 0;

    if (type === ContentType.FormData && body && body !== null && typeof body === "object") {
      requestParams.headers.common = { Accept: "*/*" };
      requestParams.headers.post = {};
      requestParams.headers.put = {};

      body = this.createFormData(body as Record<string, unknown>);
    }

    return this.instance.request({
      ...requestParams,
      headers: {
        ...(type && type !== ContentType.FormData ? { "Content-Type": type } : {}),
        ...(requestParams.headers || {}),
      },
      params: query,
      responseType: responseFormat,
      data: body,
      url: path,
    });
  };
}

    const cool = async () => {
        const api2 = await new Api()
        console.log('req', api2.request)
        const res = await api2.api.tokensCreate({ email: 'someemail', password: 'pass' }, { secure: false })
        console.log(api)
        console.log(res)
    }
    useEffect(() => cool(), [])

Upon further inspection it seems that in the compiled typescript file for httpClient the this keyword is undefined for some reason

I have the same problem (

Updated next.js to latest and problem dissappear


Prompted for username/password using 0.3.0

I have changed my config file over to use the new exec style from the haawrapper. When I attempt to run kubectl for anything I am prompted for a username and password. If I switch back to the haawrapper I don't have the issue. I am running Ubuntu Linux 18.04. Any ideas? Thanks.

What kubectl version do you have @christensen143 ? You must be running at least the 1.10 version.

/close


inital setup of simple dutch auction

WIP dutch auction for https://github.com/Geo-Web-Project/core-contracts/issues/26

closing this as its from a forked repo


Converting dependencies targeting GitHub repos to npm libraries.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Don't merge! Regression pull request perf test for no metavalidation

Docs Build status updates of commit f060536:
:clock10: Pending: waiting for processors (19 builds ahead of you)
:warning: Docs Build is busy, currently there are 19 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit f060536:
:clock10: Full build: the average full build time is 18 min(s) 45 sec(s), this is based on the last 199 build(s) for this repository.

Docs Build status updates of commit f060536:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

TestingPullRequest.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix(globalLeftNav): a11y bug for v6

Overview
We forgot about items in the nav that get hidden. When using arrow keys, the hidden items would cause hiccups with the focus state.
Added
javascript that filters out hidden nav items from the focus-able ones
remove aria-expanded from button on esc press
Removed
nothin
Testing / Reviewing
I had to copy-paste these files into Common node_modules to test locally

pinging for a review :) We managed to fix the remaining bugs

👍 fine with the changes, but you're failing a couple linting problems. Can you resolve?
 71:44  error  There should be no spaces inside this paren        space-in-parens
  71:56  error  Unexpected block statement surrounding arrow body  arrow-body-style
  72:66  error  Expected '!==' and instead saw '!='                eqeqeq

I always forget the lint step :\ ...should be ok now!


docker-compose failing

Environment

Covalent version: 0.79.1
Python version: 3.10.0
Operating system: macOS 12.3.1

What is happening?
docker-compose build is failing
~/covalent (develop|…) [255]$ docker-compose build
Traceback (most recent call last):
  File "urllib3/connectionpool.py", line 670, in urlopen
  File "urllib3/connectionpool.py", line 392, in _make_request
  File "http/client.py", line 1255, in request
  File "http/client.py", line 1301, in _send_request
  File "http/client.py", line 1250, in endheaders
  File "http/client.py", line 1010, in _send_output
  File "http/client.py", line 950, in send
  File "docker/transport/unixconn.py", line 43, in connect
FileNotFoundError: [Errno 2] No such file or directory

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "requests/adapters.py", line 439, in send
  File "urllib3/connectionpool.py", line 726, in urlopen
  File "urllib3/util/retry.py", line 410, in increment
  File "urllib3/packages/six.py", line 734, in reraise
  File "urllib3/connectionpool.py", line 670, in urlopen
  File "urllib3/connectionpool.py", line 392, in _make_request
  File "http/client.py", line 1255, in request
  File "http/client.py", line 1301, in _send_request
  File "http/client.py", line 1250, in endheaders
  File "http/client.py", line 1010, in _send_output
  File "http/client.py", line 950, in send
  File "docker/transport/unixconn.py", line 43, in connect
urllib3.exceptions.ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker/api/client.py", line 214, in _retrieve_server_version
  File "docker/api/daemon.py", line 181, in version
  File "docker/utils/decorators.py", line 46, in inner
  File "docker/api/client.py", line 237, in _get
  File "requests/sessions.py", line 543, in get
  File "requests/sessions.py", line 530, in request
  File "requests/sessions.py", line 643, in send
  File "requests/adapters.py", line 498, in send
requests.exceptions.ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker-compose", line 3, in <module>
  File "compose/cli/main.py", line 81, in main
  File "compose/cli/main.py", line 200, in perform_command
  File "compose/cli/command.py", line 60, in project_from_options
  File "compose/cli/command.py", line 152, in get_project
  File "compose/cli/docker_client.py", line 41, in get_client
  File "compose/cli/docker_client.py", line 170, in docker_client
  File "docker/api/client.py", line 197, in __init__
  File "docker/api/client.py", line 221, in _retrieve_server_version
docker.errors.DockerException: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
[3728] Failed to execute script docker-compose

How can we reproduce the issue?
run docker-compose build
What should happen?
the service containers should get built
Any suggestions?
No response

this was user error on my part sorry

The original error was just because I foolishly didn't have Docker running, but after starting Docker I have a different error
#8 23.80   Running setup.py develop for cova
#8 24.98     error: subprocess-exited-with-error
#8 24.98
#8 24.98     × python setup.py develop did not run successfully.
#8 24.98     │ exit code: 1
#8 24.98     ╰─> [6 lines of output]
#8 24.98         running develop
#8 24.98         /tmp/pip-build-env-ywijo3xu/overlay/lib/python3.8/site-packages/setuptools/command/easy_install.py:144: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.
#8 24.98           warnings.warn(
#8 24.98         /tmp/pip-build-env-ywijo3xu/overlay/lib/python3.8/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
#8 24.98           warnings.warn(
#8 24.98         error: [Errno 2] No such file or directory: 'unzip'
#8 24.98         [end of output]
#8 24.98
#8 24.98     note: This error originates from a subprocess, and is likely not a problem with pip.
#8 24.98 error: subprocess-exited-with-error
#8 24.98
#8 24.98 × python setup.py develop did not run successfully.
#8 24.98 │ exit code: 1
#8 24.98 ╰─> [6 lines of output]
#8 24.98     running develop
#8 24.98     /tmp/pip-build-env-ywijo3xu/overlay/lib/python3.8/site-packages/setuptools/command/easy_install.py:144: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.
#8 24.98       warnings.warn(
#8 24.98     /tmp/pip-build-env-ywijo3xu/overlay/lib/python3.8/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
#8 24.98       warnings.warn(
#8 24.98     error: [Errno 2] No such file or directory: 'unzip'
#8 24.98     [end of output]
#8 24.98
#8 24.98 note: This error originates from a subprocess, and is likely not a problem with pip.
------
executor failed running [/bin/sh -c cd /app &&     python -m venv --copies /app/.venv &&     . /app/.venv/bin/activate &&     pip install -e .]: exit code: 1
ERROR: Service 'covalent_data' failed to build : Build failed


[WIP] Deployment Tool - New input validation logic

Closes #1709

@nazlikaya Could you have a look at this?

Design review

Authentication is not needed per default which means if the user opens the dialog for the first time or the endpoint does not need authentication, this view will be shown https://zpl.io/blEkJXP

“Remember credentials” only will be seen when Http basic or bearer token selected

When I type credentials no matter what if i click the remember checkbox, then the credentials should be there when I open the dialog again even if I closed the dialog without deploy.

When I do uncheck the remember credentials and clicked deploy, and opened the modal again. Username is preserved. Expected behaviour, if i unclick remember credentials, none of the credentials are preserved.

When authentication is needed HTTP basic will be the default selected authentication type.

The height of the modal window should be always big enough so when user sees error messages, the error message tooltip fits inside the context area of the modal window. https://zpl.io/blEkJXP, https://zpl.io/bepO83q

When the user types in username field some input, Password field is turning immediately an error status, which should not be happened. We validate the credentials fields when user clicks on Deploy button.

When user types a valid rest endpoint and then the internet connection is lost, the error message appears but when the connection comes back and user hits the Deploy button, the error case still stays there.

Update error messages that shown to user.

You can find how all the error messages look like in the slide deck.
When user types “http://asdf” or “http://google.com” as rest endpoint, the error message is wrong currently. It should be “Should point …”  - slide 7
When there is no internet connection, the message should be “Could not establish a network connection” - slide 9,10
When credentials are missing, the message slide 14
When credentials are wrong, the message slide 14

The height of the modal window should be always big enough so when user sees error messages, the error message tooltip fits inside the context area of the modal window. https://zpl.io/blEkJXP, https://zpl.io/bepO83q

This does not go well with the current designs considering the distance between the form-bottom and modal-bottom. My suggestion is that we should always render tooltip on top of anything else to prevent this:

@nazlikaya Thank you for your feedback. I fixed the issues you mentioned. Could you please verify that it works as expected?

@nikku @barmac It is ready for a code review now, however I'll still keep this in [WIP] until Nazli finishes her 2nd round of review.

I have a few remarks regarding the UX:

When I have Remember credentials checked, erase one of the credentials, and then cancel the deployment, the credentials are gone. If I try now to deploy again, I have to type the credentials again.
Previously, the cancel button only closed the window and did not save any data. Is this change intentional?

Gif for context

Feature implemented via https://github.com/camunda/camunda-modeler/pull/1694 is gone. We display Spring-related endpoint default even if Camunda BPM Tomcat distribution is running on http://localhost:8080.

We show two tooltips with the same content in case of missing credentials:

@barmac These are all part of the new designs. If you have a look at Nazli's first design review, in the 3rd review it covers the topic how we handle the credential saving (pressing on deploy is not a requirement now, we only care about checkbox status)
The authentication tooltips are also part of the design.
About the Spring configuration, I'll investigate it and talk to Nazli. It's possible that it is not intended. I'll let you know.

@barmac Regarding the cancel behavior, which is also not intuitive for me "no surprises", you could explicitly clarify with Nazli.
Our notion was that, if the user cancels no changes are persisted. That is the behavior I'd expect as a user: CANCEL :arrow_right: no harm done.

This is from the design review.

@oguzeroglu But why is that? How does that contribute to a intuitive design?
I understand the behavior if the user closes the dialog by pressing the X. I do not understand the behavior if the user presses CANCEL.

@nikku I'm not exactly sure as I'm not the part of the design process, I'll clarify with Nazli and will let you know if that behavior should stay or go.

I have checked and they are fine.

Validate input fields after Modeler application gains focus
Validate input fields when the user opens the dialog

As discussed in the daily,
The only thing left is when user clicks cancel or close button, these buttons will not save any state on the modal window.
If user clicks deploy button, current state of the content will be saved.

Everything is fine 👍

@barmac https://github.com/camunda/camunda-modeler/pull/1721/commits/dc0611657d7d6c4ac97a013407f916d9412246b8 fixes the Tomcat/Spring URL regression.

https://github.com/camunda/camunda-modeler/pull/1721/commits/01b5562ee49a86582aa6cb72057604d91f9370df fixes this bug (validator trying to render hint after request is timed out, however modal is already unmounted):


How do I download a yanked crate (through a library API)?

Describe the problem you are trying to solve
For cargo-crev, I need to download a source a yanked crate (sometimes arbitrary, and not in Cargo.lock) to calculate the digest and some other things. I'm trying to figure out how to do that exactly. The reason is - I'm trying to allow the user to report a vulnerability in it (which might be the reason why it was yanked).
Describe the solution you'd like
Some allow_yanked: bool field on cargo::util::config::Config maybe?
Notes
I've seen some PRs that were rejected to enable behavior like this.

I'm not sure I understand the question, it is very broad.  There are a variety of ways to download packages directly. For example, either the PackageSet or Source APIs offer ways to download individual packages.
    let src_id = cargo::core::SourceId::crates_io(&config)?;
    let pkg_id = cargo::core::PackageId::new("bitflags", "1.0.5", src_id)?;
    let mut source = src_id.load(&config, &HashSet::new())?;
    let pkg = Box::new(&mut source).download_now(pkg_id, &config)?;

Or you can construct direct URLs to the registry using the url template and download using any http library.

I might be wrong but that's what I'm doing:
https://github.com/dpc/crev/blob/3312163f0d7d77c1a979d8ff516c05ef31a1e6b0/cargo-crev/src/main.rs#L173
the problem is - I think cargo wants me to have this version in Cargo.lock to allow me to download it (as it's yanked). But I don't and can't. And I'd like to avoid having to bring new machinery for something that is just a corner case.

It looks like you are using a PackagetSet created by the resolver. It is pre-populated with unyanked crates, so it won't have the yanked PackageIds.  The example above I created the PackageId manually.
I don't think it is possible to forcefully resolve the graph with yanked crates if you don't ahead of time know which crates are yanked.  I don't have enough context of what you are trying to do to offer any suggestions.

That might do it. I'll try. Thank you!


Translate Datenschutzerklärung and Impressum to english

and probably move them to their own pages

Also add section about application insights in the spirit of https://crossadvise.com/datenschutz/

The translation of the privacy policy should have high priority, since according to GDPR they have to be in a "easily understandable" format - which is clearly not the case when they are in German.
In fact, in some instances English is not even sufficient (does not apply to us, but makes the point): https://www.autoriteitpersoonsgegevens.nl/en/news/tiktok-fined-violating-children’s-privacy


Added Parsing to and from CSV

Added operation to parse CSV string to a string. This string is an array of arrays.

Also created an operation that allowed an array of arrays string to be parse to a CSV string.

This is in response to the issue #277.

Hey @Vimal-Raghubir thanks for your PR.
I think we may have an opportunity not to use an external library for this piece of functionality (see our principles).
There is already a CSV parsing function in use in src/core/Utils, so that could be the core functionality for one method. For the other half, you could look through the source for the npm library you pulled in, see how they implement it and take inspiration from that?


Latest ui issues and feature requests

Transactions list needs to show date and time of transaction.  Currently only shows time.
Addresses are STILL hard to read/use/identify as they all begin w the same long string of characters.
Password should not have to be reentered every single time you do a send.   Currently it is quite a pain to send multiple transactions as each time you copy in the send address and amount you then have to enter the password.

Perhaps I am wrong or don't appreciate the development environment you are in, but these seem like simple changes that could be done in an hour or less.   Is there no room in the project planning to do simple fixes?

Thanks for the feedback @greggdourgarian!

Transactions list needs to show date and time of transaction. Currently only shows time.
Transaction list shows transactions grouped by date which means you can find the exact date of the transaction above the each block of transactions.

Addresses are STILL hard to read/use/identify as they all begin w the same long string of characters.
Addresses format is not easy to change. The addresses are long and I'm afraid you will have to focus on their last part to distinguish them. There are plans to make the addresses shorter in the future.

Password should not have to be reentered every single time you do a send. Currently it is quite a pain to send multiple transactions as each time you copy in the send address and amount you then have to enter the password.
This is a security feature. We can not and should not store spending password in any way. Therefore the user is required to enter it each time he/she want's to create a new transaction.


Eq and Show instances

Original issue by @andrewthad. There is a bit of discussion there.

I've assigned myself for now, but anyone else is free to work on this. As I stated on the old ticket, I think flexible contexts should probably be preferred to the ...1 classes, so that probably puts me in opposition to @andrewthad. Surely we agree that == should call unexposed. There are a number of missing instances of classes in Data.Functor.Classes, both for Of and for Stream. These are a bit of a pain to write by hand, but deriving them requires TH and another dependency. Ugh.

Let's stay with the flexible contexts approach for now. we should definitely fix them so that == (And compare for that matter) calls unexposed. I have a branch I was working on where I have Eq1 and friends implemented. The worst part is, since we provide compatibility back to transformers 0.4, I had to use CPP to make it build, since the type classes were defined differently back then.
If anyone (probably Ryan Scott) ever writes a GHC extension for deriving the higher kinded type classes, I think at that point it would be worthwhile to switch over to using them in the contexts of the regular ones.
Sent from my iPhone

On Sep 14, 2017, at 5:38 PM, David Feuer<EMAIL_ADDRESS>wrote:
I've assigned myself for now, but anyone else is free to work on this. As I stated on the old ticket, I think flexible contexts should probably be preferred to the ...1 classes, so that probably puts me in opposition to @andrewthad. Surely we agree that == should call unexposed. There are a number of missing instances of classes in Data.Functor.Classes, both for Of and for Stream. These are a bit of a pain to write by hand, but deriving them requires TH and another dependency. Ugh.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm in agreement with that, although I don't think I communicated it well. I was just trying to say that I mostly already have them written (with backwards compatibility for transformers)
Sent from my iPhone

On Sep 14, 2017, at 6:32 PM, David Feuer<EMAIL_ADDRESS>wrote:
I think we want the instances anyway, even though they're a pain to write.
That's orthogonal.
On Sep 14, 2017 6:28 PM, "Andrew Martin"<EMAIL_ADDRESS>wrote:
Let's stay with the flexible contexts approach for now. we should
definitely fix them so that == (And compare for that matter) calls
unexposed. I have a branch I was working on where I have Eq1 and friends
implemented. The worst part is, since we provide compatibility back to
transformers 0.4, I had to use CPP to make it build, since the type classes
were defined differently back then.
If anyone (probably Ryan Scott) ever writes a GHC extension for deriving
the higher kinded type classes, I think at that point it would be
worthwhile to switch over to using them in the contexts of the regular ones.
Sent from my iPhone

On Sep 14, 2017, at 5:38 PM, David Feuer<EMAIL_ADDRESS>wrote:
I've assigned myself for now, but anyone else is free to work on this. As
I stated on the old ticket, I think flexible contexts should probably be
preferred to the ...1 classes, so that probably puts me in opposition to
@andrewthad. Surely we agree that == should call unexposed. There are a
number of missing instances of classes in Data.Functor.Classes, both for Of
and for Stream. These are a bit of a pain to write by hand, but deriving
them requires TH and another dependency. Ugh.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/haskell-streaming/streaming/issues/3#issuecomment-329626272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABzi_S06zmo1x-Bb8D3QTtuWAcWbIIE1ks5siaiLgaJpZM4PWHIs
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I don't think there's much point providing compatibility with ancient transformers. Does anyone really care? My current PR just drops that. If you think people really rely on being able to use old transformers versions with new streaming versions, I guess I can put in CPP.

Now that I think about it, I don't actually care about supporting transformers < 0.5. I had only done that originally because I saw that streaming's lower bound for transformers was 0.4, but I kind of forgot that we can just change that.


updates to tailwind 3.2 + adds missing dark mode colors

This updates tailwind to version 3.2 and adds some missing dark mode colors that work well with the dark backend plugin.
Note:
had to rename the class collapse to collapsible  in the side menu as collapse is already used by tailwindcss https://tailwindcss.com/docs/visibility#collapsing-elements

I'm a big fan of dark interfaces (I use them wherever possible), but this PR leaves me confused on two points.
First : I don't see the point of adding an extra dependency through a plugin.
What this plugin offers should be present in this plugin.
Second thing : the dark backend plugin plugin does not take into account the user's preference and only offers two modes, light or dark, it lacks a third state which is based on the user's preference:
window .matchMedia('(preferences-color-scheme: dark)')
or
document.querySelector('meta[name="color-scheme"]')  (which is not my preference).
What is your feeling about it?

To argue my point number one:
The dark backend plugin seeks to add through a script an item in the user menu using
document.querySelector('.mainmenu-accountmenu li.divider').before(toggleTheme);

See : https://github.com/WebVPF/wn-darkbackend-plugin/blob/52bdb5c44490a43431988c5486a39decb2b3ce47/assets/js/script.js#L26
But the .mainmenu-accountmenu selector does not exist in the Tailwind backend theme.

At the moment the dark backend plugin is the only one I know that allows to use at least a dark mode in winter backend. If this plugin should provide it, then I can make a PR for that. But I would recommend to use the media darkMode of tailwind instead of the class dark mode which I basically just made work in my PR even without the dark backend plugin. Tailwind's class based dark mode requires a .dark class to be up in the DOM tree for the dark:... selectors to work.

A fully working dark mode based on user preferences and not a .dark class would also mean alot of additional css to make all existing UI dark. The dark backend plugin already contains a lot of this that could be taken/borrowed and build upon.

Closing in favour of #23


Changed all implicit imports * with explicit classes imports

While integrating oauthlib with a Jython application we've found that Jython 2.7.0 has issues with import *.
This way of importing is in general strongly discouraged due to namespace pollution and unattended conflicts which could happen.
We've found only 3 occurrences of import * and we fixed all of them:
$ grep -R "from .* import \*" *
oauth1/__init__.py:from .rfc5849.errors import *
oauth2/__init__.py:from .rfc6749.errors import *
oauth2/rfc6749/clients/__init__.py:from .base import *

Please consider merging this PR and include in the next release.
Thank you.

@idan Any chances to get it merged and released?  This is blocker currently, when you try to use library with Azure under Jython

I'll have to review this more closely.

LGTM :+1:


[US] As a developer, I want to start using Verax without being allow-listed

As a developer,
I want to start using Verax without being allow-listed,
So that I can discover Verax more easily

For testnets only. Cf: https://snapshot.org/#/verax-snapshot-critical.eth/proposal/0x100e8dbc0ee7493e06f130eaf8c9fa4b70b016baeb1147592efaa5ae3bc1ed73


Few feature requests

Few feature requests:
1. Possibility to keep the same highlight in the displayed results. At the moment folders names are displayed with an italic highlight which make my eyes tired.
EDIT: found:
  highlight default link ScopeMenuMatch Normal
  highlight default link ScopeMenuSubtle Normal

:Scope File takes into account of current folder, whereas :Scope MRU does not. For example if pwd = ~/foo, then ~/foo/bar.txt is displayed as bar.txt in :Scope File and as ~/foo/bar.txt in :Scope MRU. It is a matter of personal preferences, but I prefer to always get filenames displayed in terms of relative path. Perhaps user should be able to select that.
3. The update rate is too fast sometimes and that leads to a sloppy experience when typing text. If I am not wrong it is about 100 ms. Would that be possible to either increase it or to leave the user to choose a custom value?
When typing text in the search box the whole popup blinks at every keystroke. That is a bit disturbing. This is very evident in :Scope Grep Would that be possible to fix it?
EDIT: I tried to play around with grep_poll_interval and timer_delay (I increased both their value) but the problem is still there.

#2
It is possible to add the feature you are requesting, but it may confuse people used to associating MRU list with the output of v:oldfiles, which lists the full path. Maybe I can add an option to toggle it.
#3
You can set grep_skip_len to 2. Every time you type a character, it performs a grep and redraws the popup. It also has to redraw as more results appear (so increasing grep_poll_interval and  decreasing grep_throttle_len helps). Bram indicated somewhere that there is no way to avoid the blinking when it redraws, if I remember correctly.

2 )
I tested with fnamemodify() with :. and :p:. modifiers, but it does not work as you'd expect. The relative path is provided only 'if possible', meaning it will not insert '../' if necessary. You can suggest any other solution. Currently that feature is not possible.

Pl. reopen this if there is something more that needs to be done.

You can set grep_skip_len to 2. Every time you type a character, it performs a grep and redraws the popup.

Do you need to redraw it necessarily? Isn't it possible to just use a popup_settext() to clean & rewrite new text?

:redraw is not called by the plugin though. I just checked carefully. When I do fuzzy.Grep() I do not see any blinking at every keystroke the way you described. Only the new text gets set in the popup window. I do not see anything weird. What is the problem exactly? Can you capture this in a video?

Here is a video: https://jmp.sh/s/a3VU1VzrQ6Wp9CZz4MM6
If the search time is smaller, then the blinking is even more annoying because the blinking frequency goes higher (blinking period is kinda of equal to the keystrokes time interval plus the search time).

Thanks for the video! I've never seen this behavior on mac or linux. Looks like findstr is not returning results fast enough. This is why you see blank screen so often. I'll look intothis.

Here is a video where I run the same command from powershell:
https://jmp.sh/s/BqATSVMlicOXTzbhbGMK

I may have made some local changes in the plugin, but I have no laptop right now. I will check later and let you know.

I re-installed the plugin and updated vim.
Now it works fine. Thanks!

Tested with Grep. Here is the outcome: https://jmp.sh/s/1G8rb2ISLrlEeIE9P9Wj

You want it to pause searching until you finish typing the pattern 'validation'? You could use :Scope grep <pattern> (or bind Scope grep<space> to a key. After you search you can type <c-k> to enter fuzzy search mode where you can narrow the results of the search.

You want it to pause searching until you finish typing the pattern 'validation'? You could use :Scope grep  (or bind >Scope grep to a key. After you search you can type  to enter fuzzy search mode where you can narrow the results >of the search.

Neat solution thanks. Unrelated: no plans at all for adding preview windows in the plugin? At least with the grep, it would be handy.

I never thought of adding preview windows since I never use them. I am not even sure how much work is involved. I'm not sure I'd want to support something I myself don't use. Maybe you can fork this and try adding it.


[Feature] Please state the difference between fetch-depth and fetch_depth

Is this feature missing in the latest version?

[X] I'm using the latest release

Is your feature request related to a problem? Please describe.
What is the difference between fetch-depth and fetch_depth?  fetch-depth is mentioned here: IMPORTANT: For push events when configuring actions/checkout the fetch-depth should be set to either fetch-depth: 0 OR fetch-depth: 2 depending on your use case; and is in the examples.  However when used, an unrecognized error is given and the you are informed to use fetch_depth.  fetch_depth is in the Input Table and is defaulted to 50.
So which one are you supposed to use?
Describe the solution you'd like?
Either an explanation of the difference or just use one them.  If the fetch-depth is related to --depth in cloning, maybe a pointer to it would be helpful.
Describe alternatives you've considered?
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @rick-kenna Please see the answer in this Discussion, I'll also suggest starting a new discussion or responding to the discussion if you require further assistance in the future.
Thanks


[BUG] Debugger not working?

🔵 Describe the bug:
When I try debugging, I can't seem to see any details. Cannot find any options that help.
🔵 To reproduce: (Steps to reproduce the behavior)
Open debugger and Enable.
Begin debugging macro by repeatedly activating macro ke.
🔵 The error:
No errors, but content only appears to show 1st item in the KeyPress variable.
🔵 Screenshots:

🔵 Expected behavior:
Expect to see listing of spells
🔵 GSE.lua file:
GSE copy.lua.txt
🔵 Desktop (please complete the following information):

OS: macOS 12.0
Game Version : retail 9.1

🔵 GSE Version:

Version: 3.0.45
Downloaded From: https://www.curseforge.com

Hi Gary,
This is a result of the way you have written your macro.  The first ability it sees in each actions stack (See Compiled template is Dash.  You will need to reorder your pet abilities to be after your characters action for the debugger to be able to see into your macro.
This is similar to https://github.com/TimothyLuke/GSE-Advanced-Macro-Compiler/discussions/1039  this isn’t a bug but working as expected.  Moving this to a discussion.
TimothyLuke

On 4 Sep 2021, at 5:01 am, GaryREM @.***> wrote:

🔵 Describe the bug:When I try debugging, I can't seem to see any details. Cannot find any options that help.
🔵 To reproduce: (Steps to reproduce the behavior)
Open debugger and Enable.
Begin debugging macro by repeatedly activating macro ke.
🔵 The error:
No errors, but content only appears to show 1st item in the KeyPress variable.
🔵 Screenshots:
🔵 Expected behavior:
Expect to see listing of spells
🔵 GSE.lua file:
GSE copy.lua.txt
🔵 Desktop (please complete the following information):
OS: macOS 12.0
Game Version : retail 9.1
🔵 GSE Version:
Version: 3.0.45
Downloaded From: https://www.curseforge.com
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


Cannot access $parameters in custom Rule class

Laravel Version: 5.6.22
PHP Version: 7.2

Description:
The Rule class generated by php artisan make:rule does not allow access to $parameters which makes it very limited.
Steps To Reproduce:
Create a custom validation rule such as this:
        // Usage Example: ['required|decimal:4,2']
        Validator::extend('decimal', function ($attribute, $value, $parameters, $validator) {
            $number = (int)$parameters[0];
            $decimals = (int)$parameters[1];

            // E.g: /^[0-9]{1,3}.[0-9]{1,2}$/
            // If the decimal format is 3,2
            return preg_match("/^[0-9]{1,$number}.[0-9]{1,$decimals}$/", $value);
        });

If you want to refactor this into its own class you can't because the $parameters are not sent to the passes() method on the custom rule.
     /**
     * Determine if the validation rule passes.
     *
     * @param  string  $attribute
     * @param  mixed  $value
     * @return bool
     */
    public function passes($attribute, $value)
    {
        //
    }

Simple Solution:
It appears trivial to add this in:
4th Parameter here: https://github.com/laravel/framework/blob/4b736f75f533e53f677dff52efd80fa5088a39b6/src/Illuminate/Validation/Validator.php#L359
        if ($rule instanceof RuleContract) {
            return $validatable
//                  ? $this->validateUsingCustomRule($attribute, $value, $rule)
                    ? $this->validateUsingCustomRule($attribute, $value, $rule, $parameters)
                    : null;
        }

And here:
https://github.com/laravel/framework/blob/4b736f75f533e53f677dff52efd80fa5088a39b6/src/Illuminate/Validation/Validator.php#L537
//      protected function validateUsingCustomRule($attribute, $value, $rule)
        protected function validateUsingCustomRule($attribute, $value, $rule, $parameters)
    {
//      if (! $rule->passes($attribute, $value)) {
        if (! $rule->passes($attribute, $value, $parameters)) {
            $this->failedRules[$attribute][get_class($rule)] = [];

            $this->messages->add($attribute, $this->makeReplacements(
                $rule->message(), $attribute, get_class($rule), []
            ));
        }
    }

You could also make it the same as the extension based approach by sending the validator as the 4th parameter.
//      if (! $rule->passes($attribute, $value)) {
        if (! $rule->passes($attribute, $value, $parameters, $this)) {

Happy to try and submit a PR is this is agreed?

It's been brought up a few times (similar to #23331), but you can pass anything you need in the Rule, through the constructor.


Merge/contribute project efforts

Hi, firstly this is fantastic work and thank you for making this. :)
I would migrate over from my node-red instance (using node-red-contrib-energenie-ener314rt) to mqtt-energenie-ener314rt but I notice I added some extra functionality into my MQTT discovery on node-red, which I would like to suggest we add into this.
My repo: https://github.com/genestealer/node-red-energenie-ener314rt-home-assistant-mqtt
Happy to make all the changes and then submit a pull request.

Smart Plug Plus (MIHO005) and Smart Monitor Plug (MIHO004) don't seem to be reporting all their data on your code:
Mine:

Yours:

Removed 1 and 2 and I see this was already completed, my mistake :(

Work complete. PR raised. Please let me know if your unhappy with anything :)
https://github.com/Achronite/mqtt-energenie-ener314rt/pull/48

New PR raised: https://github.com/Achronite/mqtt-energenie-ener314rt/pull/51

Reopened as will be closed when released to master

@genestealer Please take a look at develop, I think I've finished my updates.
NOTE: You didn't fix the VALVE_STATE number, you had HA heat = eTRV open (0) instead of HA heat = eTRV auto (2)

@genestealer Please take a look at develop, I think I've finished my updates.
NOTE: You didn't fix the VALVE_STATE number, you had HA heat = eTRV open (0) instead of HA heat = eTRV auto (2)

Thank you, will look now.


Implementing Future, Stream, Sink yourself

When building a library that uses Tokio, it seems very common to implement custom Futures, Streams, and Sink types (rather than just using built-in combinators). Coming from other languages I think people might expect to just chain functions, but that doesn't always seems to work with Tokio (though I can't quite put my finger on why). It may be worth having a guide specifically on implementing custom types

I added "how to implement" in the outlines for each trait here.
I do agree that calling this out is important and trying to explain why / when to reach for implementing Future yourself is needed.
The outline currently has "TODO: Good rule of thumb for when to reach for this", I think the questions you raised apply.
I'm not quite sure what the rule of thumb would be yet.

Ah right, missed that. I'll comment on that issue


Potential case-insensitive import collision

Due to GitHub handle change (to lowercase) for long term purpose, go get may fail fetching github.com/Unknwon/com.
Please consider take some time to update it to github.com/unknwon/com in the go.mod file.
I truly apology for the inconvenience and unintended troubles caused.

Thank you for your reminder.


[Docker Desktop for Mac] - Run without admin privileges

Tell us about your request
Docker Desktop currently requires that users grant root access on first run, in order for a privileged helper service to be installed.
We'd like to change this flow so that admins who are concerned about developers running code as root on the host or interacting with code running as root (triggering exploits etc) can perform all the necessary steps in advance, via an install flag. Following this change, no root credentials will be required from the end user at first run and Docker Desktop will run completely rootless (on the host).
Which service(s) is this request for?
Docker Desktop for Mac
Are you currently working around the issue?
It is currently possible to avoid running a privileged service in the background by using com.docker.vmnetd for setup during installation and disabling it at runtime.

I think this can be closed now that Docker Desktop 4.11.0 has been released?
According to the release notes a new --user <username> option was added to the install command.

It seems you still need sudo to install via command line. I was hoping for a sudo-less install.

Thanks @thomasjo.
@firien, thanks also for your feedback. As a first pass we thought it best to remove the need for running the privileged helper process (com.docker.vmnetd) in the background when Desktop is running. Could you help me understand why you'd specifically like a sudo-less install ? Any feedback is much appreciated !

No real reason other than paranoia. I just don't like running commands with sudo. (I use rootless on linux)
And if the privileged helper is only for accessing ports <1024 - then I would be using sudo for no reason - as I don't develop on those ports.

Thanks @firien. we're now writing the code for that and it's being tracked via this roadmap item - https://github.com/docker/roadmap/issues/413.


feat: Enable CI for validating resources and building data container

This PR enables CI on this repository. It will validate resources and will package it in a Docker container.
Signed-off-by: Néstor Salceda<EMAIL_ADDRESS>

@nestorsalceda: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign nestorsalceda
You can assign the PR to them by writing /assign @nestorsalceda in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nestorsalceda: There is not a label identifying the kind of this PR.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/kind feature
/release-note-none

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tembleking
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [tembleking]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 5f3c2c9d227390888d8c4520c02b38d68bc313b1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tembleking
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [tembleking]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Integrate official plugins into the main repo

Currently the GL core library and others are in separate repositories, and I want to bring them into this one. We can still publish them as a separate modules for people who want to use them standalone, but management and development of these plugins would be easier if done here.

Totally agree. Also, please look if pixi-display and pixi-compressed-textures can be integrated.

Is this issue still open, I'd like to contribute.

Thanks for wanting to contribute @thamaranth! THis issue is still open since we will be doing it in v5, and it is currently happening on the next branch.


SignalR ChatHub Sample works in IIS Express but Not IIS - failing with Negotiate 404 error.

The SignalR ChatHub sample works in IIS Express but Not IIS where the Negotiate fails with a 404 error.
Note - every Chathub sample that I have found suffers from this same error.
Failed to load resource: the server responded with a status of 404 (Not Found)
chatHub/negotiate:1
I tried 100 dead ends,
Including the URL into the Hub Connection, it then satisfies both versions of IIS. Else both use the root path, which is incorrect when the URL is actually http://localhost/SignalRChat.
chat.js
const url = this.document.URL;  // allow both IIS & IIS Express.
const connection = new signalR.HubConnectionBuilder()
.withUrl(url + "/chatHub")
.configureLogging(signalR.LogLevel.Information)
.build();
I am brand new to Web Dev, so I am sire that there must be better ways to ensure that the correct path is passed down the line, More experienced Web Devs can address that.
But this at least gets the SignalR ChatHub sample running 'out of the box'.

Please re-open this issue using the This page feedback button and form at the bottom of the English-US topic. Use of the This page feedback form adds metadata to cross-link the topic to your issue and automatically pings the author.


Simplify opening new view from browsercontroller

Currently the projbrowserview does the following when an item is double clicked:
Deinitialize
if (deinitialize was success)
{
   select new view in browserview
   open new view
}

Instead, the flow can, without any loss of functionality be changed to:
OpenNewView
if(openNewView was success)
{
   select new view in browserview
}

This requires:

openView in init.ts to return a boolean
each menuhandler.open* to return the boolean from openView
dblClickHandler in ProjectBrowserItem to return a boolean
this.tree.on dblclick in browsercontroller must change flow. as shown in code above.

This simplifies the interface between projbrowsercontroller and init.ts without any loss of functionality.
All of the view opening functions must use the menuhandler for this to work. Currently all except: RTTester.openFileInGUI(path); uses it. @FlorianLapschies will fix RTTester.openFileInGUI(path); in BrowserController.

@CThuleHansen  anyone working on this?

RTTester.openFileInGUI(path) is called to open files in the external RT-Tester GUI program and not for opening files in the into-cps app. Therefore, the currently displayed view is not to be replaced upon double click.
I am not sure whether that this double click action fits well into the proposed scheme.

You can change it to a right-click menu option if you prefer. Personally, I don't mind keeping it as double-click. Hopefully, we will soon have a user manual to explain all this stuff

Now that we have a user manual, I'm closing this.


Send mail error in a test environment dnn 9

Hello, just trying out this awesome app, in dnn 8 it seems working well.
In a test environment using dnn 9 it gives error sending email.
Don't know if it is dnn 9 version the cause, but it seems it is looking for an api in the wrong place
The form example is here http://www.msgweb.it/chisiamo

jquery.js?cdv=103:8526 POST http://www.msgweb.it/API/2sxc/app-api/Form/ProcessForm? 404 (Not Found)
Thanks you
Kinds regards

Fiddler says:
{"Message":"Impossibile individuare un controller per http://www.msgweb.it/API/2sxc/app-api/Form/ProcessForm. Ricerca nei namespace: ToSic.SexyContent.Apps."}
I don't have an explanation yet, but I'll look into it in a few days.

And thanks for the live demo, much faster to see what's going on :)

Found the reason, it's the changed url of API calls in dnn9 - will fix soon:
https://github.com/2sic/2sxc/issues/1099

Nice one! Thanks you :)

Note: fixed it - will be in 2sxc 8.9.1


Replace SSE With C in Haswell dcopy

Should stop crashes #1137
Tested on one single broadwell.

As noted in #1137, the problem is not specific to Haswell and the C fallback kernel appears to perform poorly compared to other options.


CORS on root resource

I'm trying to add CORS support to the root "/" resource.  However, the ApiResource requires a pathPart but doesn't accept a null "" or root "/" string.  Is there a way to create CORS options without specifying a pathPart or is there a magic "null" pathPart that will create things at the root resource?

Hi, the Root resource is already created by API gateway, I'm honestly not
sure it it's possible to manipulate but it would require a separate
resource type to do so.
What you can do is create a regular http method of type OPTIONS on the root
resource but then you have to make sure you add the allow-origin header on
the other methods on the root resource. Tedious....
I'll give this some thought and see what I come up with, enabling Cora on
the root resource makes perfect sense so it sh be supported :)
On mån 18 apr. 2016 at 15:44, Richard Hurt<EMAIL_ADDRESS>wrote:

I'm trying to add CORS support to the root "/" resource. However, the
ApiResource requires a pathPart but doesn't accept a null "" or root "/"
string. Is there a way to create CORS options without specifying a pathPart
or is there a magic "null" pathPart that will create things at the root
resource?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/issues/40

Yes, that makes sense.  I think creating the OPTIONS method on root might work.  I'll give it a shot.  :)

From version 1.6.0 you can specify CORS config on the rest api, this config will be applied to the parent resource of the API. It works exactly the same way as CORS configuration on Custom::APIResource.
https://apigatewaycloudformation.bynordenfelt.com/#custom-RestApi


Error when specifying python version

I tried doing this in a few different locations on my machine with the same result.

python.exe

Wow. Duh. Stuck in linux land. Thank you, Kenneth.


secondary exposures for HTTP API

Merge activity

Aug 12, 5:09 PM EDT: @kenny-statsig started a stack merge that includes this pull request via Graphite.


Test suite

I will add more tests where it is necessary
This issue is there to improve the relevancy of the tests and to try to be as exhaustive as possible (even if it's impossible 😇).
I'm aware that some of the tests on this list are already present 🙂
⚠ These tests list is really basic:

they do not take into account all the specific behavior we noticed for the settings and other specific behaviors of MS
they're only about indexes, documents, update status, search and settings as a first start 🙂
they only test the HTTP part. I mean, they check that the settings have been well updated, not there are applied during the search. These would deserve another set of tests.

🤖 If you disagree with one of the current behavior of MeiliSearch you notice here, feel free to open another issue.
Indexes
GET /indexes

[x] Get an empty array when there is no index
[x]  Get all the indexes: check we get an array with the correct length and the correct UIDs. Check that all the keys/values of one of the elements are present.

GET /indexes/:uid

[x]  Get an error when the index does not exist
[x]  Get an object when the index exists, and check the JSON response

POST /indexes

[x] Get an error when the index already exists
[x] Get an error when creating an index with a bad-formatted uid
[ ] Get an error when creating an index with an unknown field
[x] Create an index with no primary key, and check the JSON response
[x] Create an index with a primary key and check the JSON response
[x] Create multiple indexes and check they all exist
[ ] Get an error when updating an index with a bad-formatted primaryKey

PUT /indexes/:uid

[x]  Get an error when the index does not exist
[ ] Get an error when updating an index with a bad-formatted primaryKey
[x] Get an error when updating an index where the primary-key is already set
[x] Update the primaryKey with no error and check the JSON response
[x] Check empty update is no-op
[x] Check update index updates updatedAt

DELETE /indexes/:uid

[x] Get an error when the index does not exist
[x] Delete the index, check there is no JSON response
[x] Check that index is effectively deleted

Documents
GET /indexes/:uid/documents/

[x] Get an error when the index does not exist
[x] Get an empty array when there is no document
[x] vGet documents with no parameters: check we get an array with the correct length and the correct UIDs. Check that all the keys/values of one of the elements are present.
[x] Get documents with parameters (limit, offset and attributesToRetrieve): check we get an array with the correct length and the correct UIDs. Check that all the keys/values of one of the elements are present.

GET /indexes/:uid/documents/:id

[x] Get an error when the index does not exist
[x]  Get an error when the document does not exist
[x] Get an object when the document exists, and check the JSON response

POST /indexes/:uid/documents

[x] Add many documents, check the JSON response (update status response), get the documents, and check that all the documents have been added
[x] Check that adding a document with an existing id in the DB completely overwrites the previous document (no partial-update)
[x]  Add/Replace documents with a primary key: it should take the primaryKey into account. You need to test with a document without any "inferrable" attributes as a primary key (no id, objectId...)
[x] Add/Replace documents with the primary key in parameter but the primary has been already set before: it should ignore the primary key in parameter
[x] Add/Replace documents with a bad formatted primary key value (e.g.: "id": "coco et titi"): it should return an error in the update status.
[x] Check that an index is automatically created
[ ] It is not an error not to be able to infer a primary key, and the primary key is set to id.
[ ] Check that the index udpatedAt is updated after document addition
[ ] in case of failure check that the index updatedAt is not updated
[ ] Check error when sending invalid json.

PUT /indexes/:uid/documents

[x] Add many documents, check the JSON response (update status response), get the documents, and check that all the documents have been added
[x] Check that adding a document with an existing id in the DB completely does not overwrite the previous document (= checking the partial-update)
[x] Add/Update documents with a primary key: it should take the primaryKey into account. You need to test that with a document without any "inferrable" attributes as a primary key (no id, objectId...)
[x] Add/Update documents with the primary key in parameter but the primary has been already set before: it should ignore the primary key in parameter
[x] Add/Update documents with a bad formatted primary key value (e.g.: "id": "coco et titi"): it should return an error in the update status.
[x] Check that an index is automatically created
[ ] It is not an error not to be able to infer a primary key, and the primary key is set to id.
[ ] Check that the index udpatedAt is updated after document addition
[ ] in case of failure check that the index updatedAt is not updated
[ ] Check error when sending invalid json.

DELETE /indexes/:uid/documents

[x] Get an error when the index does not exist
[x] If there are documents: call the route, check the JSON response (update status response), check that all the documents have been removed.
[x] If there is no document: same than above, no error has been thrown in the update status

DELETE /indexes/:uid/documents/:id

[x] Get an error when the index does not exist
[x] if the document exists: call the route, check the JSON response (update status response), check that the document has been removed.
[x] if the document does not exist: same than above, no error has been thrown in the update status

DELETE /indexes/:uid/documents/delete-batch

[x] Get an error when the index does not exist
[x] if the documents exist: call the route, check the JSON response (update status response), check that the document has been removed.
[x] if the documents do not exist: same than above, no error has been thrown in the update status

Update Status
GET /indexes/:uid/updates:

[x] Get an error when the index does not exist
[x] Get an empty array when there is no document
[x] Get all the update status: check we get an array with the correct length. Check that all the expected keys/values of one of the elements are present.

GET /indexes/:uid/updates/:id:

[x] Get an error when the index does not exist
[x] Get an error when the update does not exist
[x] Get an object when the update exists, and check the JSON response

Settings
GET /indexes/:uid/settings

[x] Get an error when the index does not exist
[x] Get all the settings with the default value: check the JSON response

POST /indexes/:uid/settings

[x]  Index is created if it doesn't already exist
[x]  Get an error when updating the settings with an unknown field
[ ]  Get an error when updating the ranking rules with an unavailable field (e.g.: "lol" as a ranking rule)
[x] Update the settings without overwriting the ones that are not in the request body (= partial update): call the route, check the JSON response (update status response), check the settings have been partially changed.

DELETE /indexes/:uid/settings

[x]  Get an error when the index does not exist
[x]  Resets all the settings with the expected default values: update settings, reset the settings, check the JSON answer (update status response), check the settings have been reset to their default value.

GET /indexes/:uid/settings/XXX (all sub-routes for settings)

[x]  Get an error when the index does not exist
[ ]  Get the settings with the default value: check the JSON response

POST /indexes/:uid/settings/XXX (all sub-routes for settings)

[x]  Index is created if not existing
[ ]  Update the settings: call the route, check the JSON response (update status response), check the settings have been completely changed.

DELETE /indexes/:uid/settings/XXX (all sub-routes for settings)

[x]  Get an error when the index does not exist
[ ]  Resets all the settings with the expected default values: update settings, reset the settings, check the JSON answer (update status response), check the settings have been reset to their default value.

Search

[x]  Get an error if the index does not exist
[x]  Get an error if a parameter is not expected (e.g.: "lol")
[x]  Check a basic search with no parameter
[x]  Check a basic search with only a q parameter
[x] Check a custom search with multiple parameters. At the same time or not, but not to do it in the same test is better. Try to create a search context, e.g. "I want to crop" or "I want to search with my facets": all the parameters have to be checked.
[x] Test the correct behaviour of displayedAttributes (see meilisearch/transplant#69).
[x] Ensure every search return the "ExhaustiveNbHits" field set to false (see meilisearch/transplant#115)
[x] Check all the parameters handling * (attributesToRetrieve, attributesToHighlight, attributesToCrop)

NB: all of these tests should be done for the GET and POST routes.
Non-categorized
Add these tests:

[ ]  a bad formatted JSON (for example the coma in a request body)
[ ]  trying to send a string instead of an object in the request body when creating an index

HTTP Error code
The HTTP code error (202, 201, 400, 404...) should be tested for every test.
How to check an error?

There is exactly the keys message, errorCode, errorType and errorLink
The expected values of at least errorCode and errorType

How to check a response?
What I mean by "check the JSON response" is checking the object contains exactly the expected keys and with the expected values (or at least the expected type).
For example, for the route /indexes/:uid you expect to receive:
{
    "name": "indexUID",
    "uid": "indexUID",
    "createdAt": "2020-06-29T01:01:50.473521Z",
    "updatedAt": "2020-06-30T09:37:34.891675900Z",
    "primaryKey": "id"
}

You have to check that:

there is ONLY the keys uid, name, primaryKey, updatedAt and createdAt
the exact value of uid, name and primaryKey according to your test
the type of the value of updatedAt and createdAt

💡 A function to automatically check the JSON response when receiving an update id (e.g. when updating settings or adding documents) could be great because this response might improve in the future.

@MarinPostma confirmed the search and authentication tests were added!
Closing!


Make languages pluggable

In https://github.com/perliedman/leaflet-routing-machine/issues/446, it was reported that the size of Leaflet Routing Machine ends up very large, and examining this it turns out that the languages from osrm-text-instructions are a major part of that (at least over 80%).
Since a lot of applications will never use more than one language, I can see that this is not ideal.
A lot of other libraries use an opt-in mechanism for languages you need to support, rather than always bundling all of them, which could be one idea to approach this.
Is this something you have thought about? I tried to look for some info on how to exclude languages but didn't find anything, sorry if I missed something obvious.

👋 @perliedman
We looked at this before in https://github.com/Project-OSRM/osrm-text-instructions/pull/107#issuecomment-302864645.
Some options to fix on your end:

Use a bundler that can exclude files
Manually delete language files pre-bundling from node_modules

Loading language files on-demand in the client might be another solution. This breaks osrm-text-instructions being self-contained though, since the language files need to be served out-of-bounds though.

Hi 😃
Ok, good to know. I'll think about this for a while.
My main concern is that most users of Leaflet Routing Machine (that I hear from) use the pre-built plugin, where it's already too late to exclude the files. Also, I'd prefer to not have to instruct people on how to use their bundlers 😛, but I see your point about the module being self-contained.


[#117522857] Install bash in the git-ssh container

What
We use fair complex scripts with this containers, on which we want
to set options like pipefail [1]. Alpine sh does support that option
but not the GNU sh, which causes the scripts fail if they are
executed or tested outside of this container (e.g. travis).
To solve this, we install bash in the container so that we can use
it in the scripts and make them portable.
How to test
just run rake build:git-ssh spec:git-ssh. Travis should succed
Who?
Anyone but @keymon or @henrytk
[1] https://bclary.com/blog/2006/07/20/pipefail-testing-pipeline-exit-codes/

Works as designed.


Please make metrics RBAC configuration and related services optional

What would you like to be added?
I understand this feature is super valuable in multi-tenant k8s clusters. However, for single-tenant k8s clusters that are also private, forcing RBAC on a metrics endpoint and an additional process using resources kube-rbac-proxy is probably a bit too much.
Why is this needed?
Helm charts should be flexible so they can be used in multiple and different use cases.

@c4milo when would you need this completed by?

Bump

I'm not sure about impact yet. It seems OK for now 🤷🏻♂

Okay, I'm going to move this into the backlog for now. Let me know if we need to reprioritize it.


Setup alias to workflow version in Zenodo DOI

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7
┆Issue Number: DOCK-868
┆Epic: Create DOIs

➤ Walt Shands commented:
An alias to a workflow is being added to Zenodo metadata in https://ucsc-cgl.atlassian.net/browse/DOCK-910 ( https://ucsc-cgl.atlassian.net/browse/DOCK-910|smart-link ) ; however currently aliases do not exist for Workflow versions; there are only aliases for the whole Workflow.

➤ Walt Shands commented:
For now we will add three ‘relevant identifiers’ to Zenodo metadata; these will be a Top level workflow alias, workflow version url, and TRS url

To further clarify, they should look somewhat like the following examples (not consistent with each other)

https://dockstore.org/workflows/aliases/10.5281/zenodo.2630727 (links to the workflow as a whole in ui2)
https://dockstore.org/workflows/github.com/DataBiosphere/topmed-workflows/UM_variant_caller_wdl:1.32.0 (links to the correct version of the workflow in ui2)
https://dockstore.org/api/api/ga4gh/v2/tools/%23workflow%2Fgithub.com%2FDataBiosphere%2Ftopmed-workflows%2FUM_variant_caller_wdl/versions/1.32.0/PLAIN-WDL/descriptor/topmed_freeze3_calling.wdl (a link to the "primary" descriptor" in plain text)

3. https://dockstore.org/api/api/ga4gh/v2/tools/%23workflow%2Fgithub.com%2FDataBiosphere%2Ftopmed-workflows%2FUM_variant_caller_wdl/versions/1.32.0/PLAIN-WDL/descriptor/topmed_freeze3_calling.wdl (a link to the "primary" descriptor" in plain text)

Instead of pointing to the primary descriptor, how about pointing to the version?  https://dockstore.org/api/api/ga4gh/v2/tools/%23workflow%2Fgithub.com%2FDataBiosphere%2Ftopmed-workflows%2FUM_variant_caller_wdl/versions/1.32.0
Or both?

Undecided,
My thought is that the former can be used by cwltool (i.e. you can point cwltool at and run workflows from an arbitrary URL) and maybe cromwell down the road (currently?). The latter is only useful for people familiar with GA4GH cloud workstream and likely is only useful for developers/programmatically.

Had a discussion with @denis-yuen, and following up on his first example; we should be generating an alias for Zenodo. Depending on how the API works, the flow should be something like:

Create the DOI
Create an alias for the just created DOI, using the DOI as (part of) the alias, something like doi:10.5281/zenodo.2630727
Update the DOI with Dockstore alias.

@wshands  can substitute a different character for the : or / if you run into encoding issues
We've previously reserved the doi prefix https://github.com/dockstore/dockstore/blob/develop/dockstore-webservice/src/main/java/io/dockstore/webservice/resources/AliasableResourceInterface.java#L60


Change ur icon any time.

Describe your idea.
When you click on the arrow it shows multiple icons. "Spider" "Ship" "Wave" Etc. When you click on one your icon changes to the thing u clicked.
Examples.
"Woah! how did he shapeshift to a spider if theres no portal???"

There's already a mod for this called 'Gamemode Switcher' or something


unable to get the introduction of GitHub

I am still not clear about the GitHub and how to use this issue thing

I am unable to move forward from create branch option


 games cannot use the XUnity.AutoTranslator translation plugin, other games can use the XUnity.AutoTranslator translation plugin

 games cannot use the XUnity.AutoTranslator translation plugin, other games can use the XUnity.AutoTranslator translation plugin

that's not even a unity game


Enable Python projects

A common request for secondary school students is to be able to migrate from projects in Scratch to writing their own projects in Python.
This can be enabled by providing a simple Python SDK that can call the endpoints used by Scratch (so as not to require any new server-side endpoints/APIs).
The UI can display a customized snippet for how to use the SDK for a particular project, as well as helper code to explain how to send images.

This is the start of a new page that demonstrates how to drive the Scratch Keys API from Python code.
The idea is to make this an alternative option for older students who might find Scratch too simplistic or limiting.
This PR enables Python projects for text and images projects only.
I'm using Angular binding to allow students to customize the data that the Python code should submit to the classifier, with the sample code updating live.
Sample code for images comes in two flavours - classifying a local image file or classifying an image from the Internet.
I haven't done numbers projects because I haven't found a way to cover multi-choice and numeric values in a way that is simple and easy to follow.
I also need at least one worksheet to show how this could be used before this issue can be closed.


onPaginationChange when set to undefined doesn't fallback to using default pagination

TanStack Table version
v8.19.3
Framework/Library version
React 18
Describe the bug and the steps to reproduce it
https://stackblitz.com/edit/tanstack-table-ton6ab?file=src%2Fmain.tsx
Table with onPaginationChange explicitly set to undefined will stop being able to paginate.
Your Minimal, Reproducible Example - (Sandbox Highly Recommended)
https://stackblitz.com/edit/tanstack-table-ton6ab?file=src%2Fmain.tsx
Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
No, because I do not have time to dig into it
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

I think this should be the functionality


I cannot run the seed app on web, desktop. on iOs and android it gives a heap of errors

Most of the time I am getting this error
npm ERR! Darwin 16.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "start.desktop"
npm ERR! node v7.10.0
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start.desktop: gulp desktop && NODE_ENV=development electron ./dist/dev
npm ERR! Exit status 1
I have recreated the project several times. Even to install I need to run the install npm command with "sudo"
However Ios installation ran without errors but output was buggy and did not run
Android ran on the emulator but gave a log of errors on missing scripts
I am attaching the desktop build error log here with.
Not sure where the problem is
my system is Mac with macOS Siera 10.12.4
node v 10.7.0
npm v 4.2.0
NativeScript v3.0
Angular v2.4.10/4.1.3 (platform server v 4.1.3 only)
I am new to all this so if there are any shortcomings pl let me know.
Thanks in advance
2017-05-28T23_49_30_825Z-debug copy.txt

I've decided to deprecate this seed in support of a new, better one: https://github.com/TeamMaestro/angular-native-seed

I figured that it was due to a problem with the instalation of Android SDK. The Android CLI has change significantly since most of the comments I found on these forums. Once I manualy set the $ANDROID_HOME and $PATH by using "sudo nano/etc/paths" and added the path to android home and tools/bin it worked. However the CLI sdkmanager did not work properly may be the way I specified the package was incorrect. However I could use the Adroide studio's SDKMANAGER UI to get API 22 installed. @jlooper even your suggestion to work it needed the above fixes. But to make things verse it lacked the documentation in a big way. It does not say you need to install the packages in both at root and in nativescript after installing the seed.


Missing/wrong Octopart break list

Kicost gets Euro prices from Octoparts - great ;-).
There are a few improvements that could be done related to pricing:
I have some prices expressed in USD that I specified in a kicost.supplier.pricing field .  Example: 500:USD2.12;1000:USD1.90 .  I noted that a currency module for python was installed during the installation process and I was hoping that the USD prices would be converted to EUR somehow.
The comment "popup" in excel shows the "$" sign before the prices while they are in EUR.  It is possibly on purpose, but only the first two price breaks are listed - it would be interesting to have the full list.
Here is a screenshot:

Unfortunately Octopart prices are not correct and have fewer prices breaks than the distributer (I attach screenshots of the Octopart prices and the Distributor prices taken at the same moment).   So a part looks much more expensive than it actually is.
Kicost is reusing the right values from Octopart though - so no action for the Kicost tool
Screenshots of the pricing according to Octopart and Mouser:

@mdeweerd, confirm me, there is 2 issues:

The symbol should be correlated with the converted currency and the values are fine (converted correctly);
It is not shown the complete price break list as in the web sites.

About 1, yet, it uses a currency converter, so could exist some minimum difference between the converted value present by KiCost and by Octopart (Octopart also convert the prices but not provide the converted values in the API).

The symbols issue was fixed. MIssing fix the break price issue.

The currency symbol in the comment popup should be the same as the currency displayed.
This is/was not the case (you say you fixed it);
Possibly, if the currency was converted it would be a nice to have to show the value in the original currency and in the converted currency.
Example:  "10: 2.00 EUR (2.51USD)"
The price break list in the popup should be complete.  Currently only 2 lines are shown.
The prices shown were already the same as the ones that are used in the spreadsheet, so that is ok.
When prices are entered in the specific Kicost field in USD, but the spreadsheet is in EUR, then the prices should be converted from EUR to USD.  This is not currently the case.
I specified "500:USD2.12;1000:USD1.90" in the "kicost.supplier.pricing" field of a component and these prices were interpreted as EUR.   Ideally in the comment I would have something like "500: 1.87 EUR (2.12USD)" and of course also the correct pricing for the BOM line and total.

@mdeweerd, I checked I could get all the price breaks. Could you re-check, and re-open if still? The others improvements are in the list.

I am first modifying KiCost to respect the API terms of use. #331

Hi
I do not know what means exactly "The others improvements are in the list.", here are my observations:
The currency symbol is now € when EUR is specified.
However, there are still these two behaviors:

kicost.supplier.pricing="500:USD2.12;1000:USD1.90" has two price breaks, the sheet shows two prices, but the interpretation is not ok.  Minimum order is 500, 100 price break is not shown:

"500:USD2.12;1000:USD1.90" is expressed in USD, but the prices shown in EUR are the same.  I thought there would be a currency conversion.

I had to disable octopart to make this test as my API has been disabled.

More than one issue reported in one topic usually create some misunderstood. I fix the symbols in the title.
I will let you up-to-date.