add_compute_plan - which batch size to use

Summary
When we add a compute plan with N tasks, we can set autobatching to True and set the batch size.
This submits the tasks to the backend by batches of size batch_size. The fastest option is to increase the batch size as much as possible without getting backend errors.
The default batch size is 500, the question here is: how to find the maximal batch size we can use?
What happens when the batch size is too big
When the batch size is too big (451 tasks * 400 data samples per task), we get the following error
Requests error status 429: {"message":"grpc: received message larger than max (6228668 vs. 4194304)"}
Traceback (most recent call last):
  File "HIDDEN/substra/sdk/backends/remote/rest_client.py", line 114, in __request
    r.raise_for_status()
  File "HIDDEN/requests/models.py", line 960, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 429 Client Error: Too Many Requests for url: HIDDEN/task/bulk_create/

When the batch size is too big, we get the following error
Requests error status 429: {"message":"grpc: received message larger than max (6228668 vs. 4194304)"}
Traceback (most recent call last):
  File "HIDDEN/substra/sdk/backends/remote/rest_client.py", line 114, in __request
    r.raise_for_status()
  File "HIDDEN/requests/models.py", line 960, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 429 Client Error: Too Many Requests for url: HIDDEN/task/bulk_create/

@AurelienGasser you said that size_of_grpc_packet = const_value * number_of_samples_per_task * number_of_tasks_per_batch
and size_of_grpc_packet is capped, to 4194304 in the error shown in the description.

Is the max value of size_of_grpc_packet always the same, or does it depend on a deployment configuration?
Can we define a rule of thumb to get the value of the batch size to use?

For the rule of thumb, in the error shown in the description, we can calculate
constant = 6228668 / (451*400) ≈ 34,
and max_size_of_grpc_packet = 4194304
so the max value of number_of_samples_per_task * number_of_tasks_per_batch should be 4194304/34 = 123361
so is it correct to say that batch_size = math.floor(number_of_samples_per_task * number_of_tasks_per_batch / 130000)

There is no "one size fits all" batch size. It depends on both the number of tasks and the number of inputs.
Would it be feasible to catch this error at the SDK level and lower batch size before retry?

There is no "one size fits all" batch size. It depends on both the number of tasks and the number of inputs.

But if we can have a rule of thumb on which batch size works, we can indicate to the user what value to use.
The default batch size used to be 20 and it was very slow, so I think it's good to give an idea of how high the batch size can be for a particulare use case

Would it be feasible to catch this error at the SDK level and lower batch size before retry?

Sure, we can try that, I would do that on top of documenting the "optimal batch size"
If we retry automatically, is there a risk that the backend/orchestrator are busy because of the previous call and fail?

Do we have data on how much slower it is to have a small batch size vs a big batch size?

@tanguy-marchand from what you said, 15 rounds, 136 tuples, with 1257 data samples took 5min to submit?
(1257 samples per task or in total? The number that we are interested in is the number of samples per task)

Is the max value of size_of_grpc_packet always the same, or does it depend on a deployment configuration?

It's always the same. We could change it but have chosen no to so far. The limit serves the purpose of limiting the load on the server and avoid resource starvation.

@tanguy-marchand from what you said, 15 rounds, 136 tuples, with 1257 data samples took 5min to submit?

A CP using 2 centers (with respectively 1257 and 999 samples) and 30 rounds (overall 512 tuples) takes 17 minutes to submit

OK, so as a first fix what we can do is:

expose the batch size in substrafl
provide a way to estimate the best batch size (batch_size = math.floor(120000 / number_of_samples_per_task))

then discuss a better solution:

calculate the batch size in the SDK so that the user does not need to set it
provide a helper function to help calculate it

I think the best would be if we are able to calculate it but I am worried it would slow down the execution and we'll want to keep being able to override this if for any reason the calculation is wrong.


Fix multi-protocol commitments merge-reveal and conceal procedures

This is downstream compilation non-breaking part of https://github.com/LNP-BP/client_side_validation/pull/127
NB: While it doesn't break compilation downstream, please keep in mind that it still breaks consensus rules, changes APIs (which were not yet used downstream) and invalidates existing stashes and inventories (i.e. stocks).

The trickiest part were the problems happening when 3+ assets are involved. So I will also be working on increasing test coverage to scenarious above 100 assets. However, right now we can't have 100+ assets in a stable way b/c of #128 (we can't guarantee that all 100+ assets have a unique modulo divider within a scope of powers of two from 1 to 16).


search index not updating properly when a versioned reference is updated to a different versioned reference

Created a Patient, deleted it, then created it again (current version is 3)
Created a Condition with subject set to Patient/<patient.resource.id>/_history/1
Did a search for this Condition with _include of the subject:
.../Condition?_id=<condition.resource.id>&_include=Condition:subject
Received the expected response of the Condition along with Patient/<patient.resource.id>/_history/1
Updated the Condition subject to Patient/<patient.resource.id>/_history/2
Repeated the search above
Expected to receive only the Condition resource since the subject versioned reference is a deleted Patient.
Instead, received the previous search response of the Condition along with Patient/<patient.resource.id>/1
Working on the latest version of main.

Per discussion with Robin ... added label, estimate, and release

It appears as though the reference version field is not included in the fingerprint and so the parameter update is being skipped because the ingestion code thinks that nothing has changed.

I verified that the behavior described in this issue is now working as expected.
In this case, while testing, after updating the Condition to point to (deleted) version 2 of the Patient, the Patient is no longer returned as "included", which matches the expected behavior.
Marking as closed.


Consistent Python Binding implementation along C++

[ ] 1. The type of Param Value along C++
[ ] 2. Allow Vector/List to bind

1. The type of Param Value along C++
ion-kit 1.0.0 takes int/boolean/double value for Params of BB, but Python binding takes only string
e.g.
C++
...
      .set_param(
        Param("num_devices", num_device),
        Param("frame_sync", true),
        Param("realtime_diaplay_mode", false)
      );

Python
# set params
num_devices = Param('num_devices', str(num_device))
pixel_format_ptr = Param('realtime_diaplay_mode', 'true')
frame_sync = Param('frame_sync', 'false')

We would like to make Python API look like:
Python
# set params
num_devices = Param('num_devices', num_device)
pixel_format_ptr = Param('realtime_diaplay_mode', True)
frame_sync = Param('frame_sync', False)

2. Allow Vector/List to bind
ion-kit 1.0.0 allows users to bind vector of buffer as follows:
C++
std::vector< int > buf_size = std::vector < int >{ width, height };
    if (pixel_format == "RGB8"){
        buf_size.push_back(3);
    }
    std::vector<Halide::Buffer<T>> output;
    for (int i = 0; i < num_device; ++i){
      output.push_back(Halide::Buffer<T>(buf_size));
    }
    n["output"].bind(output);

However, python version does not allow to bind (port-map) from multiple output in a single port to List of Buffers, so we have to do one by one as follows by accessing the index:
Python
    # create halide buffer for output port
    outputs = []
    output_size = (width, height, )
    if pixelformat == "RGB8":
        output_size += (3,)
    outputs.append(Buffer(Type(TypeCode.Uint, depth_of_buffer, 1), output_size))

    # set I/O ports
    output_p[0].bind(outputs[0])

We want to allow user to bind from multiple output to List of buffer.

https://github.com/fixstars/ion-kit/pull/207


Parallel computing does not work fine for experiment.

Parallel computing does not work fine for experiment.
Different values of max_workers (e.g., max_workers=1 and max_workers=10) consume the same computation time.
class Experiment:
def init(self, output_dir: str, jobs: List[Job], m_workers: int = 6):
"""Run an experiment to execute a list of jobs.
    :param output_dir: Base directory where each job will save its results.
    :param jobs: List of Jobs (from :py:mod:`jmetal.util.laboratory)`) to be executed.
    :param m_workers: Maximum number of workers to execute the Jobs in parallel.
    """
    self.jobs = jobs
    self.m_workers = m_workers
    self.output_dir = output_dir
    self.job_data = []

def run(self) -> None:
    with ProcessPoolExecutor(max_workers=10) as executor:
        for job in self.jobs:
            output_path = os.path.join(self.output_dir, job.algorithm_tag, job.problem_tag)
            executor.submit(job.execute(output_path))
            self.job_data.append(job.get_algorithm_data())

Hi. Sorry for respoding late
What problem are you optimizing?


[ServiceBus] Fix receiver iterator stops after recovery from error

/azp run python - servicebus - tests


tests/test.lua: Use io.exit() instead of return

Apparently, Lua ignores the return value of the main chunk. To make this
fail explicitly, this now calls io.exit() instead.
Note that this exposes a bug in the meson build, which now fails.
Signed-off-by: Uli Schlachter<EMAIL_ADDRESS>

Sigh. It's os.exit, not io.exit. How did I test this?!?


PEP 8 Inspection over-generalizes blank lines around functions

PEP 8 only requires blank lines around "top-level function and class definitions" and "method definitions inside a class". However, pep8 marks any function without a line above it as failing the rule.
This is especially troublesome for functions that are inside other functions or methods, because they are often short and have no reason for a line above:
def test():
   def success_callback():  # no need for blank line above
       print('Success! A wonderful thing has completed!')
   def error_callback():  # no need for blank line above
       print('Error! The entire program is crashing around us!')
   run_async_function(success_callback, error_callback)

This is especially troublesome for functions that are inside other functions or methods, because they are often [in my experience] short and have no reason for a line above:

I added a clarifying phrase to your statement. If you ever venture into the OpenStack code bases there are nested functions which are anything but short. This rule helps that code base become more legible.
That said, the literal phrasing of the PEP (not its spirit which is not something I'm going to debate with anyone) means that this rule over steps its responsibilities. I'd be in favor of retaining the check but in a separate error code specifically for nested functions (if we can even easily make that distinction) such that you could ignore that code for your entire code base. Would that be a reasonable accommodation to you @yoleg?

@sigmavirus24 Yes, that should work. Your clarifying phrase is correct. I myself write long nested functions and would appreciate the ability to turn this inspection on and off. It is only when using callback-based libraries that I find myself using dozens of very short closure functions.
Side note 1: Perhaps this error is more closely associated with the PEP 8 rule "Use blank lines in functions, sparingly, to indicate logical sections." After all, (long) nested functions could be considered "logical sections".
Side note 2: The inspection only seems to notice blank lines missing above the nested function. It seems to me more logical to check for a blank line both above and below, as this looks much cleaner when separating long nested functions.
Side note 3: It is actually my IDE (IntelliJ with Python/ PyCharm plugin) that is using this library in their inspections. They do allow me to disable specific PEP8 errors.

Re side note 2: See #400 as a related issue.


Optimize Accumulo setting for Fluo

Need to optimize the Accumulo settings for fluo.

Also noticed that cache settings are duplicated in accumulo-site.xml


No longer "A solution recommended by React Team"?

🐛 Bug Report
Not really a bug per say, but it looks like with React 18, this is no longer "A solution recommended by React Team"
I'll still have to use this for a while though, I don't think we can drop IE11 support yet where I work.

https://github.com/reactwg/react-18/discussions/37
So React.lazy and Suspense will just work for SSR and bundle splitting now?

Haven't tried it yet myself, but according to the docs, seems like.

I just looked at the SSR example given and it's not any more "just works" than loadable-components. Still need to wire up the initial chunks (via the new renderToPipeableStream), wire up the asset manifest from webpack, and inject static CSS if you use that. The wiring is also a bit different.
However the demo does indeed render the React.lazy +  component children on the server, but if you load fetch data remotely inside a Suspense boundary it still gets punted to the client to load.

I guess my main point is mention of loadable-components has been stripped from the React docs. (Regardless of whether what we get in v18 really does the same thing as loadable-components or not)

Sounds like it. React 18 can handle code splitting out of the box, keeping Suspense boundaries not hydrated while styles are in flight. To some intent - https://github.com/facebook/react/issues/22692
To support static CSS and optimise loading times one still need library like this.


fix: allow hostname to start with number when binding

Tested with acorn version - v0.10.1-rc1-25-g92d24ac7
Able to deploy apps successfully with publish urls  having hostname start with number
acorn run  -p 111.<example-domain>-n mytestpublish ghcr.io/acorn-io/library/hello-world
acorn apps
NAME               IMAGE                                  COMMIT         CREATED    ENDPOINTS                                                                                MESSAGE
mytestpublish      ghcr.io/acorn-io/library/hello-world   eab5f8c77bd6   97s ago    https://111.<example-domain>/, https://mytestpublish-xxx.<domain>/          OK

Able to access publish urls successfully.


Set fixed uid and gid for mule user

This change fixes the uid and gid for the mule user. The default value is 4000.

LGTM


Bump version from 3.11.11 to 3.11.12 in README.md

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


fix: pr_evaluation wokrflow has conflict

The pr_evaluation workflow doesn't work.
Error:
The workflow is not valid. .github/workflows/pr_validation.yaml (Line: 13, Col: 12): Job 'detect-unresolved-conflicts' depends on unknown job 'semantic_pr'.

Change the dependency of the workflow, can be fixed by moving them in topological order. Also set specific event for each workflow, current they all run whenever the PR is edited, repopend etc etc, but semantic PR  check should only be run again if the PR title is edited, similarly map other checks to their conditions as well, this will allow us to stop not run workflow uselessly.

Thanks for reporting. But if you prepare a fix, please only focus on the fix - don't include the other stuff you mention on workflow triggering, that's a separate issue (possibly more).


Added link to migration tool

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add number of apps behind the platform header

Sometimes we look up for a lot of apps.
Therefore, it would be great to have a total number of apps behind the platform header.
In case there are more than 1.

Fixed via https://github.com/StefMa/AppVersions/commit/d2d7794a5a5746d214fc602f2038dd089a942289


Simplify helpers in eval.ml

Like part of #42, this PR streamlines some of the logic in eval.ml

rather than using UnboundVariable, add a separate exception for undeclared classes
remove eval_assn: alst is already a perfectly good association list
remove eval_helper: unpacking the classes, assignments, and return from a program can be done by eval
implement eval_plst and eval_weighted_plst  with List.map
fix the patterns in eval_pol the same way this change does for string_of_policy: no need to split lists into head and tail

The other parts of the diff are either from dune fmt or cascading effects of these changes.

Ah, this is a somewhat strange habit I've picked up.
You likely saw it a lot in PR #45; there, I kept making minor changes to our github actions. Using this --amend and the force push trick seemed to let me re-trigger CI without polluting the commit log with lots of "Minor tweak" and "Re-trigger CI" commits.
But now that you bring it up, I'm wondering:

do think there's a better to achieve this kind of behavior?
are there any adverse affects of lots of commit --amends and push --force?

 So, when they git pull, git will yell at them.


table gets distorted if I reduce column with to an extent.

when I have 3 or 4 column in my datatable, and I try to reduce width, the headers get arranged as per the width but the table body gets distorted.
Any solution for this?

just add this code after --> var newColNeighbourWidth = Math.max(minColumnNeighbourWidth, prevNeighbourWidth - dx);
if (newColWidth === minColumnNeighbourWidth || minColumnNeighbourWidth === newColNeighbourWidth ) {
return false;
}


os.File type of option for output

I have the following in option in my file:
	Fileo *os.File `goptions:"-o, --output, description='The dump output', wronly"`

And my main begins with:
func main() {
goptions.ParseAndFail(&options)

Howerver, when I invoke my program with
my_program -o /tmp/out

It errors out, with the following message:
Error: open /tmp/out: no such file or directory
Usage: ....

I think for wronly option, that is meant solely for output, the package should create the file instead of error out.

As stated in the docs:
Any combination of create, append, rdonly, wronly, rdwr,
excl, sync, trunc and perm can be specified and correspond directly with
the combination of the homonymous flags in the os package.

So you need to add the create flag :)

ops, silly me. thanks.

Hmm, didn't find it in https://godoc.org/github.com/voxelbrain/goptions,
So, should I or them together like in C:
Fileo *os.File `goptions:"-o, --output, description='The dump output', wronly|create"`

or use extra commas?

Admittedly, the indentation is off. And no, flags are separated by comma. So
Fileo *os.File `goptions:"-o, --output, description='The dump output', wronly, create"`

should work.

Oh, I meant to look for/ask whether or them together like C or use commas. Thanks for the answer.


FIXinsert into doris table with array/map type by local tvf

Proposed changes
Issue Number: close #xxx
fix read map type data when insert into doris table by local tvf

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.36 seconds
stream load tsv:          510 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>9 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.03 seconds
stream load tsv:          513 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>1 Bytes


Add query parameter to "List of All Saved Examples of the Subject" endpoint

https://github.com/exadel-inc/CompreFace/blob/master/docs/Rest-API-description.md#list-of-all-saved-examples-of-the-subject
Right now this endpoint returns all subject examples. We need to add the query parameter "subject" to return examples of this subject.
e.g. we have such subject examples:
[
    {
      "image_id": 1,
      "subject": "John"
    },{
      "image_id": 2,
      "subject": "John"
    },{
      "image_id": 3,
      "subject": "Jane"
    }
]

With this request:
http://localhost:8000/api/v1/recognition/faces?subject=John
It should return such response:
{
  "faces": [
    { "image_id": 1, "subject": "John" },
    { "image_id": 2, "subject": "John" }
  ],
  "page_number": 0,
  "page_size": 10,
  "total_pages": 1,
  "total_elements": 2
}

@pospielov  I'd like to work on this issue, can you assign this to me? Also if possible, can you point me to the controller where the API logic is defined?

Hi, thanks for the interest in our repository, I assigned the issue to you
https://github.com/exadel-inc/CompreFace/blob/master/java/api/src/main/java/com/exadel/frs/core/trainservice/controller/EmbeddingController.java#L102
Here is the line where you should start the investigation.


rake spec results in "uninitialized constant Gem::RDoc"

I followed doc/development/SETUP.md to get a working development environment:
$ git clone https://github.com/bundler/bundler.git
$ cd bundler
$ bin/rake spec:deps
$ bin/rake spec

However, bin/rake spec resulted in this error:
/usr/bin/ruby -S gem install 'rack:< 2.0' 'rack-test:~> 1.1' 'artifice:~> 0.6.0' 'compact_index:~> 0.11.0' 'sinatra:~> 1.4.7' 'rake:12.3.2' 'builder:~> 3.2' ruby-graphviz --no-document --conservative
ERROR:  While executing gem ... (NameError)
    uninitialized constant Gem::RDoc

Full logs: https://gist.github.com/duckinator/e3d11c6b0168e4dc804473558a3695e8
Weirdly, manually running the command that results in the error works fine... but still doesn't fix subsequent runs of bin/rake spec.

OS: Fedora 29
Ruby version: 2.5.3p105
Gem version: 3.0.3
gem env: https://gist.github.com/duckinator/d69f6f4f5a129f80d9eceb460dd33e1c
Not really sure how to start debugging this, honestly.

This gets farther, but still results in multiple thousands of test failures on my system:
GEM_HOME=~/.gem/ruby/2.5.0/ RUBYOPT='-rrake' bin/rake spec

Seems like this is an issue with Ruby on Arch? It's fixed on master, though: https://github.com/rubygems/rubygems/issues/2483.

Looks like it may be that, or at least related to it. (Although this is Fedora, not Arch.)
How would I work around this, prior to the next rubygems release?

@duckinator my impression is that you could edit the rubygems file directly on disk to allow you to install rdoc? or maybe check out the latest rubygems from master and install it.

Yeah, the patch is very simple, so probably the easiest is to apply it manually.

(Also this makes me think that shipping a bugfix release so people can upgrade with gem update --system or whatever would be a great thing to do soon if someone has time. 😄)

This looks answered.


write a users guide

There's enough complexity to the system that having more extensive documentation and in particular more of a cookbook for "how to do X with skicka", various usage models, etc, is probably warranted.

One topic would be using multiple accounts as per issue #73

Should also discuss the risks of using the new -download-google-apps-files option with download: if you download with that on and then upload back to the same place, you'll clobber the Google Apps files, which is likely not what you wanted.


stock_buy_and_sale file wrongly present in C++/datastructure

the file location should be changed from c++/datastaructure to c++/algorithms/greedy

I have some algorithms to add in the c++/algorithms section, kindly provide your permission


installation ?

Hello,
I would like to run this but there is no installation instructions ?
Can you please help me?
Any help is appreciated.

Sure, I can help.. Can we connect on Skype? NilamDoc

Hi,
yes we can. What is your skype?

NilamDoc - Skype
Sent from Windows Mail
From<EMAIL_ADDRESS>Sent: Thursday, June 15, 2017 12:28
To<EMAIL_ADDRESS>Cc: ND A<EMAIL_ADDRESS><EMAIL_ADDRESS>Hi,
yes we can. What is your skype?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/nilamdoc/msx/issues/1#issuecomment-308646338, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAOC-jdfNBiUaMD3ePG6D-w1IScgQWgkks5sENYJgaJpZM4NXTDs.

Hello,
I would prefer the installation procedure here.
Skype doesnt work for me.
regards


[Fix-#6605][Common] Fix the situation where the obtained string property is empty.

Purpose of the pull request

this pr close #6605
When the value is not configured, the tool class gets an empty string.
resource.manager.httpaddress.port=

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #6661 (12152df) into dev (f204447) will decrease coverage by 0.02%.
The diff coverage is 11.94%.

@@             Coverage Diff              @@
##                dev    #6661      +/-   ##
============================================
- Coverage     38.48%   38.45%   -0.03%
- Complexity     3225     3227       +2
============================================
  Files           646      646
  Lines         25886    25899      +13
  Branches       2808     2808
============================================
- Hits           9961     9960       -1
- Misses        15013    15025      +12
- Partials        912      914       +2

Impacted Files
Coverage Δ

...ache/dolphinscheduler/spi/utils/PropertyUtils.java
0.00% <0.00%> (ø)

...e/dolphinscheduler/common/utils/PropertyUtils.java
28.91% <15.62%> (-5.30%)
:arrow_down:

...che/dolphinscheduler/common/utils/HadoopUtils.java
28.42% <75.00%> (+0.14%)
:arrow_up:

...ache/dolphinscheduler/server/log/LoggerServer.java
78.26% <0.00%> (-8.70%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
53.52% <0.00%> (+1.40%)
:arrow_up:

...org/apache/dolphinscheduler/remote/utils/Host.java
40.00% <0.00%> (+2.22%)
:arrow_up:

Continue to review full report at Codecov.

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


Consolidate resolver package with containerd.

Today, to support registry mirror in cri, we copied containerd docker resolver package https://github.com/containerd/containerd/blob/master/remotes/docker/resolver.go, and maintained it at https://github.com/containerd/cri/blob/master/pkg/containerd/resolver/resolver.go.
This introduced a lot of code duplication. And we have to keep porting fix to our own resolver package, e.g. https://github.com/containerd/cri/issues/722.
We should:

Either move the registry mirror logic into containerd (may need a more comprehensive proposal).
Or at least make most code in docker/resolver a library that we can reuse in cri.

@abhi

I don't think we can finish this issue this release. Move to next.


OTK SMS spike working

Summary | Résumé
This work is related to sending the one-time-key (OTK) to patients via SMS. When a new key has been generated, portal users will have the option to navigate to a new form that allows them to enter a patient's mobile phone number and send them the generated key via SMS.
To enable this feature, it was necessary to cache the generated key within the server-side session. Currently, when the user navigates back to the start page, the cache is deleted.
Test instructions | Instructions pour tester la modification

Enable SMS for the province of your user (go to admin panel and edit the provinces to enable 'sms_enabled'). Note that you must be a super user to do this
Login
Generate a key and note the new button "Text key to patient"
Fill out your mobile phone number and send the text
Ensure that you receive a text message
You should be taken to a success page with options to navigate to other pages, ensure that the navigation works

@pcraig3 I figured out why I need the phone number param in the otk_sms_sent url. It's because on that page we display the text "did {{ phone_number }} receive the text?"

@pcraig3 I figured out why I need the phone number param in the otk_sms_sent url. It's because on that page we display the text "did {{ phone_number }} receive the text?"

Okay, slapped a commit on the end updating some of the words (and adding one more check to the form because it would throw an error if there was no value).
I think we could still look at formatting the number so that it looks better ((416) 667 5327 instead of +14166675327), but let's just ship this ⛵

Okay, slapped a commit on the end updating some of the words (and adding one more check to the form because it would throw an error if there was no value).
I think we could still look at formatting the number so that it looks better ((416) 667 5327 instead of +14166675327), but let's just ship this ⛵


coverage outputting some html files in src directory

Reproducible on two win 10 machines (desktop + laptop)

Windows 10
git bash or command prompt (same result)
node v6.0.0
jest-cli v12.0.2

scott@LeanMachine MINGW64 ~/dev/sotdl-chargen (1_test_suite)
$ gs
On branch 1_test_suite
Your branch is up-to-date with 'origin/1_test_suite'.
nothing to commit, working directory clean

scott@LeanMachine MINGW64 ~/dev/sotdl-chargen (1_test_suite)
$ ./node_modules/.bin/jest --coverage --verbose true
Using Jest CLI v12.0.2, jasmine2, babel-jest
 PASS  __tests__\utils\capitalize-test.js (0.422s)
capitalize
  ✓ it has loaded the module
  ✓ it capitalizes a basic string
  ✓ it does nothing to TestString

 PASS  __tests__\components\ActionBar-test.js (0.781s)
ActionBar Component
  for view mode
    ✓ it renders the container element
    ✓ it renders three buttons
    ✓ it triggers no event functions yet
    for viewButton
      ✓ it is not hidden from the User
      when onClick is triggered
        ✓ it triggers onEdit
        ✓ it triggers no other event functions
    when onClick is called on editButton
      ✓ it is hidden from the User
      ✓ it triggers onView
      ✓ it triggers no other event functions
    when onClick is called on createButton
      ✓ it triggers createButton
      ✓ it triggers no other event functions
  for edit mode
    ✓ it hides the view button
    ✓ it shows the edit button

16 tests passed (16 total in 2 test suites, run time 2.297s)
-----------------------------------------------------|----------|----------|----------|----------|----------------|
File                                                 |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
-----------------------------------------------------|----------|----------|----------|----------|----------------|
 C:\Users\scott\dev\sotdl-chargen\src\js\components\ |      100 |     87.5 |      100 |      100 |                |
  ActionBar.js                                       |      100 |     87.5 |      100 |      100 |                |
 C:\Users\scott\dev\sotdl-chargen\src\js\utils\      |      100 |      100 |      100 |      100 |                |
  capitalize.js                                      |      100 |      100 |      100 |      100 |                |
 src/js\                                             |      100 |      100 |      100 |      100 |                |
  EmptyModule.js                                     |      100 |      100 |      100 |      100 |                |
-----------------------------------------------------|----------|----------|----------|----------|----------------|
All files                                            |      100 |    91.67 |      100 |      100 |                |
-----------------------------------------------------|----------|----------|----------|----------|----------------|

scott@LeanMachine MINGW64 ~/dev/sotdl-chargen (1_test_suite)
$ gs
On branch 1_test_suite
Your branch is up-to-date with 'origin/1_test_suite'.
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        src/js/components/ActionBar.js.html
        src/js/components/index.html
        src/js/utils/capitalize.js.html
        src/js/utils/index.html

nothing added to commit but untracked files present (use "git add" to track)

scott@LeanMachine MINGW64 ~/dev/sotdl-chargen (1_test_suite)
$ ls coverage/lcov-report/src/js/
EmptyModule.js.html  index.html

Branch is here:
https://github.com/ScottMaclure/sotdl-chargen/tree/1_test_suite
Will add more details here as I find them.

thanks for bringing this up. Unfortunately Windows support for Jest is mostly self-service – we don't have the resources to support this at FB. If you'd like to dig into this, I'm pretty sure this must be an istanbul bug and it isn't a Jest bug because we don't write these files. We call istanbul's writer here: https://github.com/facebook/jest/blob/master/packages/jest-cli/src/reporters/IstanbulTestReporter.js#L38
I'm going to close this out because it doesn't seem like a bug in Jest. Thank you so much for providing a repository with a repro, I hope that someone working on istanbul can take a look at this. If this does turn out to be a bug in Jest, I'm happy to receive any PRs that make this situation better for you.


clarification around "= version" conversions

It looks like currently, GHSA entries with affected versions using the "= X" operator without a patched version (e.g. https://github.com/advisories/GHSA-wxhq-pm8v-cw75), get converted to:
      "ranges": [
        {
          "type": "ECOSYSTEM",
          "events": [
            {
              "introduced": "X"
            }
          ]
        }
      ],
      "versions": [
        "X"
      ]

(example)
According to the OSV spec, this actually implies all versions after and including X are affected, because there is no corresponding "fixed" to end the affected range.
Would it be possible to encode such cases as just:
      "versions": [
        "X"
      ]

Without the erroneous "range"?

@oliverchang Ah, yes, good catch. I can open an issue to address that going forward.

@oliverchang We have a fix for this in progress now

@oliverchang Fix has been deployed and I've pushed updated OSV files for all affected advisories. The original you linked to doesn't qualify anymore because all of the unfixed vulnerabilities were withdrawn, but you can see an example of the change in this diff.
Will you please verify that this has resolved the issue? If so I'll close this out.
Affected GHSA IDs

Affected GHSA IDs

GHSA-2j5v-fc74-j9q2

GHSA-2q6w-rxf3-4wc9

GHSA-333g-rpr4-7hxq

GHSA-3x83-p476-vv95

GHSA-4g4c-8gqh-m4vm

GHSA-4v2c-g2xc-47fv

GHSA-5h5r-ffc4-c455

GHSA-5q54-8p9j-x74j

GHSA-5rqg-jm4f-cqx7

GHSA-5w9c-rv96-fr7g

GHSA-6fjr-m7v6-fpg9

GHSA-6qh5-wx38-q92g

GHSA-73qr-pfmq-6rp8

GHSA-7jfh-2xc9-ccv7

GHSA-8gr3-2gjw-jj7g

GHSA-8qm2-24qc-c4qg

GHSA-8rxg-9g6f-vq9p

GHSA-92px-q4w8-hrr5

GHSA-9hx9-w2j6-rw76

GHSA-9pv8-q5rx-c8gq

GHSA-9x64-5r7x-2q53

GHSA-c82c-8pjw-6829

GHSA-c8h6-89q2-mgv8

GHSA-c9rj-pgxv-84jc

GHSA-cpp2-q66x-fq44

GHSA-fwx5-5fqj-jv98

GHSA-g2q5-5433-rhrf

GHSA-g982-9r8g-6qxw

GHSA-gm68-g349-gxgg

GHSA-gp82-xr77-88f4

GHSA-hfc6-79wv-5hpw

GHSA-j49g-mp79-5vm5

GHSA-m25q-fwg4-9v2p

GHSA-m6wh-m8m8-6xx5

GHSA-mc9x-v9xg-25pm

GHSA-p59g-6cqr-m73w

GHSA-pcm6-g2qp-9gw8

GHSA-qmxf-fxq7-w59f

GHSA-rhgq-vv9x-j4p5

GHSA-rqp5-pg7w-832p

GHSA-vjcj-5g2r-vxqc

GHSA-vp8g-53fw-r9f2

GHSA-vqcm-7f7f-r539

GHSA-w7q9-xr2x-wh7x

GHSA-w8hg-mxvh-9h57

GHSA-wg6j-r28m-7293

GHSA-ww4j-c2rq-47q8

GHSA-x3g3-334f-q6h4

GHSA-x65c-4fgj-5fc3

GHSA-x9gm-qxhh-rf75

GHSA-xp5m-4c9f-498q

Looks great! Thanks a lot @chrisbloom7 for fixing this!!


Will VueJS 2.0 be compatible with IE9?

Hello,
VueJS is, for now, compatible with IE9+. Will it remain the same for VueJS 2?
Thanks!
Jerome

Yes it will.

Ok thanks, I'll use that :)


doc: Add Contributors section

Also fixes minor problems

@all-contributors please add @rvosshall

@all-contributors please add @rvosshall for code doc data ideas

@all-contributors please add @rvosshall for design code doc data ideas

@all-contributors please add @canbooo for design code ideas doc


Remove partition over by to fix memory limit issue

Refer to this bug - https://github.com/fivetran/dbt_fivetran_log/issues/56
Pull Request
Are you a current Fivetran customer?

Julien Goulley
Fivetran Employee
Solution Architect
What change(s) does this PR introduce?

Remove memory issue which is due to large partitions
Did you update the CHANGELOG?

[x] Yes

Does this PR introduce a breaking change?

[ ] Yes (please provide breaking change details below.)
[x] No  (please provide an explanation as to how the change is non-breaking below.)
New script will scan the existing table and add/update accordingly

Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

[x] Yes

Is this PR in response to a previously created Bug or Feature Request

[x] Yes, Issue/Feature link bug/feature number here
[ ] No

How did you test the PR changes?

[ ] CircleCi
[x] Local (please provide additional testing details below)

Select which warehouse(s) were used to test the PR

[x] BigQuery
[ ] Redshift
[ ] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

:chocolate_bar:
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

thanks for opening julien! our team will look into this a bit more and fold it in in an upcoming sprint


Is there any way to add css just for specific devices when we using css plugin

Like the way 'css!style@printer' or some other way?

Hello @yaoazhen!
People have done this in a couple of ways:

modernizer
has.js
has! plugin

If you use modernizer or has.js to test the browser environment before loading modules, you can use the results of those test to create module lists.  Some other curl.js users have done something like this:
<!--[if IE 6]><script>window.isIE6 = true;</script><![endif]-->
<script>
curl(['link!base.css', has('css-position-fixed') ? 'link!header-layout.css' : 'link!sidebar-layout.css', isIE6 && 'link!ie6-fixes.css']);
</script>

In this example, has.js is used to detect the existence of position:fixed and load an appropriate style sheet.  It also loads an additional style sheet if IE6 is detected.  I used has.js here since I am more familiar with it, but I hear modernizer works great, too.
The has! plugin is something exclusive to dojo at the moment.  We want to write one for curl.js as soon as we find the time. :)  It works like this:
curl(['link!base.css', 'has!css-position-fixed?link!header-layout.css:link!sidebar-layout.css']);

As you can see, it's not much different from using the has() syntax of has.js except for one major point: you don't have to run the has.js tests beforehand.  The tests are run as needed.  Pretty cool.  :)
If you prefer modernizer and feel like writing a modernizer! plugin, that'd be great too!  :)
I'm closing the issue, but please feel free to keep the discussion going!
Regards,
-- John

I realize I misunderstood.  You were wondering if there were a way to create a print stylesheet!  :)

Currently, there is no way to do this.  If we added this feature, it would probably look like this:
curl(['css!print.css!media=print'])

You can, however, use a @media {} block within your style sheet.  Is this aceptable?
-- John

yes! modernizer is great~ I also using this library.
And I like the way curl(['css!print.css!media=print']) as you said. But I'm a little worried  about '！' will make print.css to be a plugin when we want to use multiple plugin.

Yes, it does look like curl(['css!print.css!media=print']) will attempt to use two plugins, now that you mention it. :)
Do not worry. AMD does not allow cascading plugins (imho), and curl.js doesn't either.
Is it possible for you to use a @media {} block within your style sheet?

Yep ~ @media it could be work well~


Update Article “epic-level”

Automatically generated by Netlify CMS

👷 Deploy Preview for 4shame processing.

Name
Link

🔨 Latest commit
1b559923e5be6214ef270cac1b3477e29c4d04d1

🔍 Latest deploy log
https://app.netlify.com/sites/4shame/deploys/646e40f451547b0009a8f586


Regression in track/album artist templating

Describe the bug
It is possible for %{trackartist} to be None, this is caused by #212
To Reproduce
Command to reproduce the behavior:
bandcamp-dl --template="%{trackartist} - %{title}" https://shirobon.bandcamp.com/album/ps1-reverb-impulse-responses
https://shirobon.bandcamp.com/album/ps1-reverb-impulse-responses

Expected behavior
trackartist should be Shirobon, artist should also be Shirobon

Tentatively fixed with f2d24825bb0f72a51d8af17eaf257e40b6b0e223


Extension of generic type 'Array' from a different module cannot provide public declarations

Hello, here is my problem when using your library. This is my first Swift (and iOS) application so I could have made errors in importing or whatever.
CONTEXT
importing your library into my application by drag and dropping the ExSwift folder (not the xcodeproj) into the root of my iOS application.
ACTION
Build the application. No error in my personal code.
RESULT
errors in all methods of the Array class of ExSwift : "Extension of generic type 'Array' from a different module cannot provide public declarations"
EXPECTED
The application should build.
Thanks in advance for your help. Great library by the way !

Hi,
this is a known problem with Xcode < 6.3 where Swift frameworks cannot declare public extensions.
As of the latest Xcode 6.3 beta this seems solved, so you have two options:

Use Xcode 6.3 to compile the project (but in that case you won't be able to publish anything on the App Store until 6.3 is released to the public)
Follow the installation steps for Xcode 6.1/6.2: https://github.com/pNre/ExSwift#installation

I thought I followed the steps for Xcode 6.1.
Where do I should write this ?
components.takeFirst() { $0.completed }

It's just an example, you don't have to write it anywhere.

Ok, then I made the point 2. Should I make the point 1 two ? i do not see why it should be needed.

Do I have to import ExSwift in the embedded libraries in the project manager ?

Nope, you have to add the source files from ExSwift to your project (the second point in the list from the readme).

OK so do you know why do I have the error ?

You're probably still linking the framework.
Remove any reference to ExSwift from your project and just add the source files.

Still the same project...

"1. Use Xcode 6.3 to compile the project (but in that case you won't be able to publish anything on the App Store until 6.3 is released to the public)"
@pNre Does this work for you with 6.3b? I'm using Version 6.3 (6D554n) (beta4) and I can't manage to avoid this error. I must be holding it wrong - is there a way to make these declarations public so it can be built und used as a framework?

Can you please link an example project?

What dark magic do you speak of @pNre? It is currently impossible to provide public extensions to generic classes from foreign modules. To till this, no workaround has been found. I hope you've got a "... yet" stored in there for me.

Just add 'public' to an Array extension method like 'each' and you'll get the error, no sample project required :)

You're both right, this cannot be done (yet), I must have been drunk when I tried using it as module in beta 3.

Swift 2.0 now allows extensions to generic types. Hooray!


[Cache] Custom scalars in arguments are not correctly serialized to cache keys

Follow up on https://github.com/apollographql/apollo-android/pull/2566#discussion_r487585561
If a field argument is a custom scalar, the object is used directly without being properly serialized and the cache key will use the default toString() implementation: CustomClass@123456.

Fixed in 3.0.0-alpha01


Fix check condition

Signed-off-by: zawazawa0316<EMAIL_ADDRESS>

Thanks @zawazawa0316, nice catch.


import importlib.util

import importlib.util
Line 444 was failing without this.
https://docs.python.org/3/library/importlib.html

not sure how to delete it. And did not mean to do it!


VSCode locks node_modules/

I have to quit my text editor every time I want to clear node_modules.
What's funny is I get a prompt for admin privs on delete, then the deletion window dissappears and starts all over again with a new prompt, forever.

/needsMoreInfo including version and platform. Screenshots may be useful too.


#2 remove non related files

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #2
Test URLs:
Before: https://develop--diva-showcase--ifahrentholz.hlx.live/
After: https://2-remove-non-related-files--diva-showcase--ifahrentholz.hlx.live/

@LoomingEcho bitte die checks und die conflicts lösen

[ ] Remove also all non related style files


spark.databricks.queryWatchdog.maxQueryTasks exceeded

Hi,
I have a requirement to read thousands of small xml files from azure adls. Currently we have 1,041,620 files and when tried to read them using below script, it fails.
xml_header_tag_df = spark.read
.format("xml")
.option("rowTag","tag_hdr")
.option("inferSchema", "false")
.option("rowValidationXSDPath",xsd_file_name)
.load(input_files_path + "*.xml")
Error:
org.apache.spark.SparkException: The query is not executed because it tries to launch 1041620 tasks in a single stage, while the maximum allowed tasks one query can launch is 100000; this limit can be modified with configuration parameter "spark.databricks.queryWatchdog.maxQueryTasks".
How can we resolve this? Is it advisable to increase maxQueryTasks value?
Thanks,
Sri

Yikes, that's a lot of files! In an extreme case like this, it may be better to ask Spark to read the contents of the files as individual rows with wholeTextFiles. That gives a Dataset of Strings. You can read this at a lower level with the XmlReader class. However that kind of requires that every file's contents is a single row, and nothing more. Is that true of your data? then it's an efficient solution. Otherwise... yeah this is hard. You'd have to try concatenating the files first.

What I tried was to disable the query watchdog and ran the above command and it took  21.76 minutes.
spark.conf.set("spark.databricks.queryWatchdog.enabled", "false")
https://docs.databricks.com/spark/latest/spark-sql/query-watchdog.html#when-should-you-disable-query-watchdog
As per the above article it recommends to disable if it's part of ETL workload. But this ran 1,041,620 tasks :D. Any suggestions? I'm guessing looping through the files will run for even more time.
Env ran in:
Databricks runtime ver: 7.3 LTS
worker: Standard_E8_v3 (64gb mem, 8 cores, 2DBU) (min 1, max 10)
driver type: Standard_E8_v3 (64gb mem, 8 cores, 2DBU)
Details for Stage 13550 (Attempt 0)
Total Time Across All Tasks: 8.2 h
Locality Level Summary: Process local: 1041620
Input Size / Records: 2.9 GiB / 1041520
Associated Job Ids: 7977

A million tasks is pretty extreme. It may work, but not ideal. See the comment above.
The only other thing I can think of is to tell the Hadoop InputFormat to try to put many files into one partition. You could try setting mapred.min.split.size (via spark.hadoop.mapred.min.split.size) to some large value, the size of 100s of your small files, and see if that induces it to put many files into a partition.

Thanks a lot, @srowen for your responses.
I tried setting the below configuration at the notebook level and ran the command again. However, it did not reduce the tasks.
spark.conf.set("mapred.min.split.size", "256000000");
spark.conf.set("mapred.max.split.size", "256000000");
Did i miss anything ?
Thanks again for introducing this setting to me. After you mentioned it, i did see a similar article on Databricks knowledge base (which is trying to do the opposite of what i want).
https://kb.databricks.com/execution/increase-tasks-per-stage.html

Best,
Sri

As I say, in Spark it's probably spark.hadoop.mapred.min.split.size. You do not want to set a max split size in any event.


Corrected paths, watch options, bower depencies, .gitignore, .bowerrc.

Added option atBegin to watch-task and fixed correct partials-path, added bootstrap to bower depencies, fixed .gitignore to ignore app/vendor/bower, added .bowerrc.

These are all project specific changes and the nature of this project is to be a template for any kind of Angular app. That said if want to follow common conventions and best practices all project specific logic needs to be added to each project separately.


the formatOnType does not work when I used it in my vue component

I have three problems:

I use JSON.stringify(data) to format data,but it failed.
I try to use  editor.getAction('editor.action.formatDocument').run() to format , it is successfully,but when page init,it failed。
when I set the readonly true  , the format does not work 。

monaco-editor version: 0.X.Y
Browser:
OS:
Playground code that reproduces the issue:

/needs-more-info


Error on failed mandatory publish

When using the mandatory flag to catch unroutable messages, I'm getting a undefined local variable or method error:
amqp = AMQP::Client.new('amqp://guest:guest@localhost').start
amqp.publish('something', 'amq.topic', 'not.routed', mandatory: true, persistent: true)

#<Thread:0x0000000104cf7ce0 /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:54 run> terminated with exception (report_on_exception is true):
/Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:525:in `next_message_finished!': undefined local variable or method `msg' for an instance of AMQP::Client::Connection::Channel (NameError)

              warn "AMQP-Client message returned: #{msg.inspect}"
                                                    ^^^
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:508:in `body_delivered'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/connection.rb:396:in `parse_frame'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/connection.rb:198:in `block in read_loop'
        from <internal:kernel>:187:in `loop'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/connection.rb:185:in `read_loop'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:70:in `block (2 levels) in start'
        from <internal:kernel>:187:in `loop'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:56:in `block in start'
/Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:525:in `next_message_finished!': undefined local variable or method `msg' for an instance of AMQP::Client::Connection::Channel (NameError)

              warn "AMQP-Client message returned: #{msg.inspect}"
                                                    ^^^
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:508:in `body_delivered'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/connection.rb:396:in `parse_frame'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/connection.rb:198:in `block in read_loop'
        from <internal:kernel>:187:in `loop'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/connection.rb:185:in `read_loop'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:70:in `block (2 levels) in start'
        from <internal:kernel>:187:in `loop'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:56:in `block in start'

Also your Docker example switched the host and container mapping around on:

https://hub.docker.com/r/cloudamqp/lavinmq
https://github.com/cloudamqp/lavinmq

Should be:
docker run --rm -it -p 5672:5672 -p 15672:15672 -v /tmp/amqp:/var/lib/lavinmq cloudamqp/lavinmq
instead of
docker run --rm -it -p 5672:5672 -p 15672:15672 -v /var/lib/lavinmq:/tmp/amqp cloudamqp/lavinmq

Also, I'm not able to use direct reply-to RPC:
Consumer:
amqp = AMQP::Client.new('amqp://guest:guest@localhost').start
amqp.subscribe('amq.direct.reply-to', no_ack: true, prefetch: 1) do |msg|
  msg.body = Zlib.gunzip(msg.body) if msg.properties.content_encoding == 'gzip'
  msg.body = JSON.parse(msg.body) if msg.properties.content_type == 'application/json'

  pp msg
rescue StandardError => e
  puts e.full_message
  msg.reject(requeue: false)
end

at_exit do
  amqp.close
  puts 'WORKER - Connection closed'
end

sleep

Publisher:
amqp = AMQP::Client.new('amqp://guest:guest@localhost').start
amqp.publish('something', '', '', reply_to: 'amq.direct.reply-to')

/Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:421:in `block in wait_for_confirms': Channel[1] closed (406) PRECONDITION_FAILED - Direct reply consumer does not exist (0/0) (AMQP::Client::Error::ChannelClosed)
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:418:in `synchronize'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:418:in `wait_for_confirms'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client/channel.rb:303:in `basic_publish_confirm'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:145:in `block in publish'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:290:in `with_connection'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/amqp-client-1.1.6/lib/amqp/client.rb:143:in `publish'
        from pub.rb:11:in `<main>'

Thank you for your reports! Will look into as soon as i can

Also your Docker example switched the host and container mapping for the volume around, here:

thank you, fixed

Regarding your RPC example I think your code is wrong. The "consumer" or "server" should subscribe to a real queue, that the publisher/client should publish to. Only the client (publisher in your case) should subscribe to "amq.direct.reply-to".
But there might still be a problem, the RPC client have to publish on the same channel as it consumes, and the "high level client" do all publishes on a seperate channel from it's subscritptions. You can access the normal amqp-client with:
conn = AMQP::Client.new("amqp://guest:guest@localhost").connect

instead of .start, where you have direct control over channels.

You can use the main branch locally by putting
gem "amqp-client", github: "cloudamqp/amqp-client.rb"

in your Gemfile and bundle update

Thank you for your quick response and fixes!
I tried getting this to work with your suggestions, but am currently stuck again:
Server:
worker_queue = 'test'
conn = AMQP::Client.new('amqp://guest:guest@localhost').connect
conn.with_channel do |ch|
  ch.basic_consume(worker_queue) do |msg|
    pp msg
    msg.ack
    ch.basic_publish_confirm('Pong!', '', msg.properties.reply_to, mandatory: false)
  end

  sleep
end

at_exit do
  conn.close
  puts 'WORKER - Connection closed'
end

Client:
worker_queue = 'test'
conn = AMQP::Client.new('amqp://guest:guest@localhost').connect
conn.with_channel do |ch|
  ch.basic_consume('amq.direct.reply-to', no_ack: true) do |msg|
    pp msg
  end

  ch.basic_publish_confirm('Ping!', '', worker_queue, reply_to: 'amq.direct.reply-to', mandatory: true)

  sleep
end

at_exit do
  conn.close
  puts 'PUBLISHER - Connection closed'
end

#<Thread:0x0000000104277428 /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/channel.rb:327 run> terminated with exception (report_on_exception is true):
/Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/channel.rb:421:in `block in wait_for_confirms': Channel[1] closed (406) PRECONDITION_FAILED - unknown delivery tag 1 (60/80) (AMQP::Client::Error::ChannelClosed)
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/channel.rb:418:in `synchronize'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/channel.rb:418:in `wait_for_confirms'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/channel.rb:303:in `basic_publish_confirm'
        from worker.rb:15:in `block (2 levels) in <main>'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/channel.rb:555:in `consume_loop'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/channel.rb:327:in `block (2 levels) in basic_consume'
^Cworker.rb:18:in `sleep': Interrupt
        from worker.rb:18:in `block in <main>'
        from /Users/erik/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/bundler/gems/amqp-client.rb-4626b254dcee/lib/amqp/client/connection.rb:111:in `with_channel'
        from worker.rb:11:in `<main>'

@carlhoerberg Can you point me in the right direction, or give a working example?

I found the mistake:
Instead of:
ch.basic_ack(msg.delivery_tag)
ch.basic_publish_confirm('Pong!', '', msg.properties.reply_to, mandatory: false)

I need to do this:
ch.basic_publish_confirm('Pong!', '', msg.properties.reply_to, mandatory: false)
ch.basic_ack(msg.delivery_tag)


ERROR 1105 (HY000): unknown error: unable to cast nil to bool

Error executing example query:
SELECT repository_id,
       file_path,
       uast_extract(uast(blob_content, LANGUAGE(file_path), '//uast:Import/Path'), "Value") AS imports
FROM commit_files
NATURAL JOIN refs
NATURAL JOIN blobs
WHERE ref_name = 'HEAD'
  AND LANGUAGE(file_path) = 'Go'
  AND ARRAY_LENGTH(imports) > 0;

The problem appears to be when LANGUAGE is used without blob_content second param.

Needs this PR https://github.com/src-d/go-mysql-server/pull/768 merged before applying the fix here.


Chrome launch does not work.

Issue Type: Bug
Steps to reproduce
• Open an html file.
• Add a Chrome launch configuration and launch.
Expected result
• Launch Chrome debugging environment
Actual result
• Does not launch chrome. Debugging toolbar appears for a split second and disappears.
• Same result when extensions are disabled.
Edge launch works fine.
VS Code version: Code 1.62.3 (ccbaa2d27e38e5afa3e5c21c1c7bef4657064247, 2021-11-17T08:11:14.551Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No
Extensions (21)

Extension
Author (truncated)
Version

prototxt
dmi
0.0.6

verilogformat
eri
1.0.1

prettier-vscode
esb
9.0.0

gc-excelviewer
Gra
3.0.44

verilog-formatter
Isa
1.0.0

vscode-autohotkey-plus-plus
mar
2.8.1

fluent-icons
mig
0.0.14

vscode-edge-devtools
ms-
1.4.1

python
ms-
2021.11.1422169775

vscode-pylance
ms-
2021.11.2

jupyter
ms-
2021.10.1101450599

jupyter-renderers
ms-
1.0.3

remote-ssh
ms-
0.66.1

remote-ssh-edit
ms-
0.66.1

cpptools
ms-
1.7.1

veriloghdl
msh
1.5.1

material-icon-theme
PKi
4.11.0

LiveServer
rit
5.6.1

sort-lines
Tyr
1.9.0

vhdl-formatter
Vin
1.0.5

vscode-arduino
vsc
0.4.8

/jsDebugLogs

Here is the trace log.
vscode-debugadapter-89cbaad0.json.gz

Can you please try the following:

In the launch config, set the userDataDir in your launch config to somewhere you can find like ${workspaceFolder}/profile
Add "runtimeArgs": ["--enable-logging", "--v=1"]
Reproduce this again
Share the chrome_debug.log file in your configured userDataDir

I have attached the log below.
chrome_debug.log
I found a prompt briefly open during the launch and display the following message.
[11016:17500:1231/202822.488:ERROR:broker_win.cc(56)] Error reading broker pipe: The pipe has been ended. (0x6D)


dependency injection causes table doesn't exist?

Hello,
Can you help me with following issue? Did i something wrong or is it a bug?
I have a command
namespace App\Console\Commands;

use App\Features\PullProductsFromDbFeature;
use Illuminate\Console\Command;

class PullProductsFromDbCommand extends Command
{
    protected $pullProductsFromDbFeature;

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'pim:products:pull-db {feedID=null}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Pull products from feed(s) tables to import_products table';

    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct(PullProductsFromDbFeature $pullProductsFromDbFeature)
    {
        $this->pullProductsFromDbFeature = $pullProductsFromDbFeature;
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $feedID = $this->argument('feedID');
        $this->pullProductsFromDbFeature->execute($feedID);
    }
}

class PullProductsFromDbFeature
    public function __construct(PullProductsFromDbJob $pullProductsFromDbJob)
    {
        $this->pullProductsFromDbJob = $pullProductsFromDbJob;
    }

    /**
     * @param string $feedID
     *
     * @return void
     */
    public function execute(string $feedID): void
    {
        $this->pullProductsFromDbJob->execute($feedID);
    }

class pullProductsFromDbJob
    public function __construct()
    {
        $this->categories = Category::all();
   }

When i execute command with:
php artisan tenants:run pim:products:pull-db --tenants=<site>

i get message central.categories table doesn't exist.
So on this moment the code is not Tenant aware. How can i fix this?
regards,
Toby.

Show me more code & the output of the command.

hereby the output of the command. Which code you want to see? I think i gave you the most important code.
   Illuminate\Database\QueryException  : SQLSTATE[42S02]: Base table or view not found: 1146 Table 'central.categories' doesn't exist (SQL: select * from `categories`)

  at /var/www/pim/frontend/vendor/laravel/framework/src/Illuminate/Database/Connection.php:669
    665|         // If an exception occurs when attempting to run a query, we'll format the error
    666|         // message to include the bindings with SQL, which will make this exception a
    667|         // lot more helpful to the developer instead of just the database's errors.
    668|         catch (Exception $e) {
  > 669|             throw new QueryException(
    670|                 $query, $this->prepareBindings($bindings), $e
    671|             );
    672|         }
    673|

  Exception trace:

  1   App\Domains\Feed\Jobs\PullProductsFromDbJob::__construct()
      [internal]:0

  2   Doctrine\DBAL\Driver\PDOException::("SQLSTATE[42S02]: Base table or view not found: 1146 Table 'central.categories' doesn't exist")
      /var/www/pim/frontend/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOConnection.php:63

Is that the full output? If you used the tenants:run command, you should see the tenant ID printed. https://github.com/stancl/tenancy/blob/2.x/src/Commands/Run.php#L37

Yes this is the full output, no tenant id

Well, seems like the issue happens before my command's handle() is executed then. I wanted to see the full source code because you may be overriding execute in the command or forgetting to call some parent constructor.

I had placed the full code of the command in the first post. Is this
enough?
Op wo 8 apr. 2020 14:22 schreef Samuel Štancl<EMAIL_ADDRESS>
Well, seems like the issue happens before my command's handle() is
executed then. I wanted to see the full source code because you may be
overriding execute in the command or forgetting to call some parent
constructor.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stancl/tenancy/issues/345#issuecomment-610926729, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBC6VU2YJOED3C6OJOV33DRLRUBNANCNFSM4MD2D5QA
.

I have no idea what's the issue then.

What do you mean? With this code i get the error
Op wo 8 apr. 2020 14:58 schreef Samuel Štancl<EMAIL_ADDRESS>
I have no idea what's the issue then.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stancl/tenancy/issues/345#issuecomment-610943803, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBC6VSKMMH4BLT7UEY6UTTRLRYF7ANCNFSM4MD2D5QA
.

I have no idea what is causing the issue. The tests for tenants:run pass so the issue is probably somewhere in your code. Also it would be better to make that command a tenant-aware command rather than running it with tenants:run, since there is no way it can work when run against the central database. See the docs for tenant-aware commands.

That's a pitty. I made the command Tenant-aware but no difference.

With this code all my php artisan commands are failing :-(

That's a problem with your code, not the package. Sorry, I can't help with that. I think tenant aware commands are pretty well documented.


Swarm-that-walks tweaks do not show up in party editor

Describe the bug
In the party editor, the row for changing swarm-that-walks swarm size as shown in #845 does not show up between size and gender. Also, the button for removing all swarm clones is nowhere to be found.
To Reproduce
Steps to reproduce the behavior:

Load a save with swarm-that-walks mythic.
Open ToyBox.
Click on the party editor.
Expand on the stat tab.
Scroll down to the size row.
See error.

Expected behavior
The row for changing swarm size should appear. Probably together with the button to remove all swarm clones?
Screenshots

Settings
settings.zip
Version Info:

ToyBox Version: 1.5.1
OS: Windows
Game Version: 2.1.3j

Additional context
I suspected if it is because I respecced at some point after I got into swarm. However, I tested on an old save before respec, but the problem still persisted.

Also just to confirm; you're not gestalting mythics right?

No I am not. Just swarm.

Can you provide logs + maybe a save file?

Sure. I wanted to zip both of the files but the zipped one is still too big. The following two files are just renamed .zks saves.
I was using this save from this nexus mod page. It is a save right before mythic 8.
Here is another save for the same character after picking swarm.
logs.zip


prevent input box from crashing when empty

As stated. Can't subtract 1 from that index without checking first :  )

Thank you very much :)


Add distribution budget on edit range strategies

fix #1458
Test cases :

Create a disposable range strategy
Buy some liquidity from this strategy
Go into Edit Price
Update the budget only
Distribution option is available: set it to maintain
Validate
See that marginal rate ratio is maintained

Other tests:

Do 1, 3, 4 -> distribution is not available
Do 1, 2, 3, 4 -> change the price -> distribution disappear
Do 1, 2, 3 -> update budget beyond balance -> distribution is not available
Do 1, 2, 3, 4, 5 -> set distribution to reset -> marginal price is reset
Do 1, 2, 3 -> change to Limit -> Update budget -> distribution is not available
Do the same with recurring strategy

[ ] edit prices - strategy type should be in a single row without icons

[ ] Do 1, 2, 3, 4 -> change the price -> distribution remains visible


RUMM-789 Provide data scrubbing callbacks for RUM

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #441 (619bce9) into master (0b22cd7) will increase coverage by 0.22%.
The diff coverage is 96.30%.


Base address is disabled under advanced build property page

Create or open a project
Open the project properties
Go to the build page
Click on the advanced button

Expected: Base address is enabled/disabled based on the output type
Actual: It's always grayed out

The problem is that we're not correctly implementing OutputType in the way that the old project system did. The fix is to do the same thing we're doing for the OutputTypeEx provider

Should be fixed once #1709 is implemented.

Fixed by #1842


JVM performance tuning

Done in https://github.com/turms-im/turms/commit/adb1ba0af4e95e18d4d96280047ee646abefe31c


"in" operator may not be working on non-string filterchecks

I suspect that the "in" operator isn't actually working on non-string filterchecks e.g. "fd.sport in (443, 53, 80)". I need to check but I think the values from the expression are being treated as strings which won't compare properly against the (numeric) fd.sport.

To add more context, I think this problem only appears in falco due to the different filtercheck parser implementations. Falco uses one driven primarily by lua and lpeg, while sysdig has a separate internal parser implementation.


Project Reorganization, Changes to SpriterDotNet Behaviour

Ok, issuing a pull request again with the changes made that you suggested.

Didn't realize that I had removed the DLL, but that should be back in place.
The null play error may have been an issue with the DLL, I wasn't able to reproduce any issues on Play. However, I was getting a null error the first time awake was called. From what I've seen online, Awake on an ExecuteInEditor script gets called multiple times, and from what I was seeing, the first time, SpriterData was not deserializing, but then it was the next passthrough. So I made the script handle deserialization callbacks so it won't even try to load the Spriter data unless it has been deserialized properly.
For some reason, the panel looked fine at the size I was using yesterday, but as soon as I resized my view window it displayed the bad behavior you described. I have expanded the panel to accommodate.
Switched back to the naming conventions. My main hesitance with that is that the public data in Monogame and the Library are all using properties (Microsoft & Unity styleguides say Pascal case for these), but Unity forces us to use fields (Microsft & Unity styleguides say camel case for these). But since Unity doesn't give us Properties that are visible within the inspector, I can see why you want to use the Pascal case anyway.

One last new thing is that I switched unity to use text for scenes and prefabs (this is recommended for source control so that there is a chance the scenes can be merged, or at the very least you may be able to glean what changed). The option is in Edit>Project Settings>Editor>Asset Serialization, which means it won't be committed here since we don't have the ProjectSettings folder. I guess it will just maybe need to be a habit to make sure that setting is changed properly, and it may be worth noting in a future CONTRIBUTE file.
EDIT: Squashed squishy situation with git squash! Learned something today!
I've pulled a fresh copy and everything seems to build fine and run fine on my end.

Good work!
About the conventions - public and protected fields should never be used but unfortunately Unity doesn't allow Properties. I use Pascal case for these to basically treat them as properties because when (if) Unity gives support for properties then we can just change them to properties without modifying any code that accesses them.
I'll definitely add a CONTRIBUTE file in the next couple of days.


sw_engine: gradient shapes with opacity < 255 rastered properly

The cases with gradient shapes with composition are handled
in the same function as gradint shapes with opacity < 255.
Parts of the code from _rasterOpaque... grad functions moved to
_rasterTranslucent... grad functions.
before:

after:

This pr contains #384
Please note, that the only change is that I moved some part of the code from one function into another.

@mgrudzinska could you please rebase your patch ?


Highlight rust macros

Macros in the Rust Enhanced syntax plugin are not highlighted.
before:

after:

I used the colors from https://github.com/catppuccin/ksyntaxhighlighting/blob/3f7bc6ddaa9c77608fd85e734c86d220c6a785ea/catppuccin-latte.theme#L843
The default Rust syntax does not define a name/scope for macros so this does not affect it

Hi,
Sorry for the late merge, I had not noticed this, but thanks for opening the PR!


Local image is not working

Hi, I added an local image but the component does not render it.
  <GifImage           source={localImage}           style={{             width: 100,             height: 100,           }}         />

Device: iPhone 12
OS: 14.5
React Native 0.64.2
React: 17.0.1

Hi, there indeed was a bug with local GIFs. Now it should be fixed in v2.2.1 with #6 . Please let me know if the issue is still there!


0.16.0

Upgrades deno_ast.

@kitsonk is this requirement related to https://github.com/denoland/deno_doc/issues/170 ?

@kitsonk is this requirement related to #170 ?

It simply would have failed CI, but yes.  When you update dependencies, the wasm build will be out of date.


empty commit

Describe your changes
Issue ticket number and link
Closes #x
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have added thorough tests (when necessary).
[ ] I have added the right documentation in the docstring and changelog (when needed)

:books: Documentation preview :books:: https://sklearn-evaluation--300.org.readthedocs.build/en/300/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.09%) to 94.016%

Totals

Change from base Build<PHONE_NUMBER>:
-1.09%

Covered Lines:
3048

Relevant Lines:
3242

💛  - Coveralls


Please add cuda 11.8 support with pip, instead of just working with cuda 11.7 and conda

The current environment is based on CUDA11.7, however, I think it will also works well on higher CUDA version.

thanks, i installed them successfully via cuda 11.8 or cuda 12.2, Error due to anaconda (i removed them and just played pip)


closes #11261 -  Inject document separators when rendering CRDs

What this PR does / why we need it:
This PR address issue #11261 and accounts for charts with CRDs that don't contain file separators at the beginning of the file, as mentioned in the issue.
Special notes for your reviewer:
closes #11261
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Signed-off-by: Kyle L Frisbie<EMAIL_ADDRESS>

@KyleFrisbie merging in main messed with this PR which now shows 174 files being changed. Can you please rebase.


fix: correct message for first peer

Details(context, link the issue, how was the bug fixed, what does the new feature do)

Implementation note, gotchas, related work and Future TODOs (optional)

If multiple non hosts join, they will still see the "You're the first to join" message @amar-1995?


Dockerfile: Remove unnecessary python-pbs dependency.

This hasn't been a dependency of Zulip's provision for over a year.
https://github.com/zulip/zulip/commit/eeb5302ba1e1a8db786c695774060cf878f4421c

Thank you for the PR!


Use of coerce in logrotate_app breaks specs

With the 3.0.0 changes introdoced in https://github.com/sous-chefs/logrotate/commit/dc20786df1b23734379aaaf7cde03dfbcb4c96e7#diff-b7e94f5fcd66051355907f23b26ff6ac96685ca36c95837931a588074999fbf9R26
it's a lot harder to write chefspecs when a logrotate_app resource is called with an paths array
e.g.
logrotate_app 'nginx' do
  cookbook 'logrotate'
  path [
    "#{node['web']['root']}/logs/ssl-access.log",
    "#{node['web']['root']}/logs/ssl-error.log"
  ]
...
end

In older versions when coerce was not used, one could simply write the following chef spec:
  it 'add logrotation' do
    expect(chef_run).to enable_logrotate_app('nginx').with(
      path: [
        "/path/to/www/logs/ssl-access.log",
        "/path/to/www/logs/ssl-error.log",
      ],
...
    )
  end

As coerce merges the array on assignment, this does not work anymore.
The workaround is to "deal with the internals" and format the paths as a coerced string like
  it 'add logrotation' do
    expect(chef_run).to enable_logrotate_app('nginx').with(
      path: '"/path/to/www/logs/ssl-access.log" "/path/to/www/logs/ssl-error.log"',
    ...
    )
  end

which is obviously ugly and does not represent the way, the resource is called in the recipe. Any idea to make this usable again?
Thank you.

Yeah unfortunately that's what you need to do when using ChefSpec.


fix(fallbackFilters): rename fallbackFilters to fallbackParameters

Summary
fallbackFilters will be named fallbackParameters  after the GA release, its structure also changed since it will accept searchParameters, but we only handle facetFilters for now.

Closing this PR since most changes will be deprecated after the release of the new Recommend API client: https://github.com/algolia/algoliasearch-client-javascript/pull/1280
Update will come in a future PR.


[TASK] Add Relayer Info to Fees Component

Description
To improve the visibility of relayer information for each transaction, proposing to incorporate the relayer info within the existing Fees Component (expanded state) of the Hubble Bridge interface. User should be able to view details of relayer via opening the hyperlink icon.
Note: this update will be reflected in the bridge card & confirmation card to display relayer used below relayer fees.
Screenshot

Closed in https://github.com/webb-tools/webb-dapp/pull/1771.


Creating per session cloud table on read requests

Question
I was looking into some timeouts during reads and noticed we are creating a cloud table over and over again for each read transaction even though table has not changed at all. This causes 3-10 seconds delay for each query and sometimes timeouts. I was wondering why read replicas dont keep a copy of the queried tables once its loaded until an modify event occurs in which case, they can reload?

Do you mean the StorageCloudMergeTree object in the worker? Different sub-query from server may search for different parts in the same table.(addDataParts, loadDataParts). So it should not be used outside the query session.

Yes I mean StorageCloudMergeTree object in the worker. I understand loading different parts, but recreating table every time creates an interesting error on retry or fallback when a timeout or rpc socket connection issue occurs:
2024.02.29 07:15:03.325072 [ 1494590 ] {3ba693d9-d6d2-11ee-892c-ba3d820a74a5}<PHONE_NUMBER>76211845)  TCPHandler: Code: 57, e.displayText() = DB::Exception: DB::Exception: Table ed_mt_v1.ed_metric_2d6be233-f7bb-4fe1-90a5-28a95c86ec9c_448054137376211845 already exists. SQLSTATE: 42P07.: While executing Remote SQLSTATE: 42P07, Stack trace:
0. Poco::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, int) @ 0x223c0252 in /opt/byconity/bin/clickhouse

DB::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, int, bool) @ 0xebc1580 in /opt/byconity/bin/clickhouse
DB::readException(DB::ReadBuffer&, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, bool) @ 0xec30dcf in /opt/byconity/bin/clickhouse
DB::RPCHelpers::checkException(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&) @ 0x1b706f8b in /opt/byconity/bin/clickhouse
void DB::RPCHelpers::onAsyncCallDoneWithFailedInfoDB::Protos::SendResourcesResp(DB::Protos::SendResourcesResp*, brpc::Controller*, std::__1::shared_ptrDB::ExceptionHandlerWithFailedInfo, DB::WorkerId) @ 0x1b51522a in /opt/byconity/bin/clickhouse
brpc::internal::FunctionClosure4<DB::Protos::SendResourcesResp*, brpc::Controller*, std::__1::shared_ptrDB::ExceptionHandlerWithFailedInfo, DB::WorkerId>::Run() @ 0x1b516fa1 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/controller.h:704: brpc::Controller::EndRPC(brpc::Controller::CompletionInfo const&) @ 0x1e1ae9a9 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/controller.cpp:0: brpc::Controller::OnVersionedRPCReturned(brpc::Controller::CompletionInfo const&, bool, int) @ 0x1e1ad506 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/details/controller_private_accessor.h:0: brpc::policy::ProcessRpcResponse(brpc::InputMessageBase*) @ 0x1e1d6a30 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/input_messenger.cpp:386: brpc::InputMessenger::OnNewMessages(brpc::Socket*) @ 0x1e1cd2ce in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/libcxx/include/memory:1655: brpc::Socket::ProcessEvent(void*) @ 0x1e2fd6cd in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/bthread/task_group.cpp:304: bthread::TaskGroup::task_runner(long) @ 0x1e19af2d in /opt/byconity/bin/clickhouse

@smmsmm1988 Could you check about the exception and confirm whether it is a knowable issue?

@smmsmm1988 Could you check about the exception and confirm whether it is a knowable issue?

No, it seems to be a new issue. @luffyhwl Could you help check this exception?

@smmsmm1988 Could you check about the exception and confirm whether it is a knowable issue?

No, it seems to be a new issue. @luffyhwl Could you help check this exception?

Yes I mean StorageCloudMergeTree object in the worker. I understand loading different parts, but recreating table every time creates an interesting error on retry or fallback when a timeout or rpc socket connection issue occurs:
2024.02.29 07:15:03.325072 [ 1494590 ] {3ba693d9-d6d2-11ee-892c-ba3d820a74a5}<PHONE_NUMBER>76211845) TCPHandler: Code: 57, e.displayText() = DB::Exception: DB::Exception: Table ed_mt_v1.ed_metric_2d6be233-f7bb-4fe1-90a5-28a95c86ec9c_448054137376211845 already exists. SQLSTATE: 42P07.: While executing Remote SQLSTATE: 42P07, Stack trace: 0. Poco::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, int) @ 0x223c0252 in /opt/byconity/bin/clickhouse

DB::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, int, bool) @ 0xebc1580 in /opt/byconity/bin/clickhouse
DB::readException(DB::ReadBuffer&, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, bool) @ 0xec30dcf in /opt/byconity/bin/clickhouse
DB::RPCHelpers::checkException(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&) @ 0x1b706f8b in /opt/byconity/bin/clickhouse
void DB::RPCHelpers::onAsyncCallDoneWithFailedInfoDB::Protos::SendResourcesResp(DB::Protos::SendResourcesResp*, brpc::Controller*, std::__1::shared_ptrDB::ExceptionHandlerWithFailedInfo, DB::WorkerId) @ 0x1b51522a in /opt/byconity/bin/clickhouse
brpc::internal::FunctionClosure4<DB::Protos::SendResourcesResp*, brpc::Controller*, std::__1::shared_ptrDB::ExceptionHandlerWithFailedInfo, DB::WorkerId>::Run() @ 0x1b516fa1 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/controller.h:704: brpc::Controller::EndRPC(brpc::Controller::CompletionInfo const&) @ 0x1e1ae9a9 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/controller.cpp:0: brpc::Controller::OnVersionedRPCReturned(brpc::Controller::CompletionInfo const&, bool, int) @ 0x1e1ad506 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/details/controller_private_accessor.h:0: brpc::policy::ProcessRpcResponse(brpc::InputMessageBase*) @ 0x1e1d6a30 in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/brpc/input_messenger.cpp:386: brpc::InputMessenger::OnNewMessages(brpc::Socket*) @ 0x1e1cd2ce in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/libcxx/include/memory:1655: brpc::Socket::ProcessEvent(void*) @ 0x1e2fd6cd in /opt/byconity/bin/clickhouse
/home/ByConity/build_docker/../contrib/incubator-brpc/src/bthread/task_group.cpp:304: bthread::TaskGroup::task_runner(long) @ 0x1e19af2d in /opt/byconity/bin/clickhouse

@ozcelgozde
This exception may occur during fallback. The commit below fixes this issue. When sql fails to execute in optimizer mode,  worker resource will be synchronized delete.
commit ac18e6a238905b761af723f6d56d89bc9ec83fc0
Author: Dao<EMAIL_ADDRESS>Date:   Thu Jan 18 10:32:52 2024 +0800
Merge branch 'zema/cnch-2.0-fixRemoveWorkerResource' into 'cnch-ce-merge'

fix(clickhousech@m-3000501254): remove SUBMIT_THREADPOOL for removeWorkerResource

Thanks for the quick fix! Do we have an idea why optimizer fails noticeably often?

Any number of possible errors return table already exists :)


[Product Partnerships] Added AWS Transcribe

Related Task
Asana Task - https://app.asana.com/0/1202974982230192/1202975137301837
Background
Partnership Engineering Team -

Added AWS-Transcribe Entity
Amazon Transcribe is an automatic speech recognition service that uses machine learning models to convert audio to text. You can use Amazon Transcribe as a standalone transcription service or to add speech-to-text capabilities to any application.

Checklist

[X] I've read the guidelines and understand the acceptance criteria.
[X] The value of the attribute marked as identifier will be unique and valid.
[X] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: rahul188:x: manuelrodriguez-nrYou have signed the CLA already but the status is still pending? Let us recheck it.


OpaqueArchetypeSpecializer: inlinable functions are now in this module

...not the one they came from. This is important when specializing an inlinable function that takes an opaque type as a generic argument: that opaque type should still be considered opaque even if it came from the same module as the inlinable function.
rdar://problem/56410009. Fixes a regression introduced in #27666.

@swift-ci Please test

@swift-ci Please test source compatibility

This fixes the issue but I need you reviewers to tell me whether it is the right fix.

Are we likely to have further issues where we use the DC of a SIL function? Should we consider not serializing that information?
…no, wait, I guess we use it to identify things that came from Clang…

The source compat failure is a single project failing to clone rather than something this commit changed, so I'm going to merge anyway.


Destination Points behave strangely if controllers are connected mid-session

Environment

Source of VRTK: Github
Version of VRTK: fdb24039c3f0b2e4b75f8c59bb4911e810045498
Version of Unity3D: 5.5.1f1

Steps to reproduce

Open VRTK/Examples/044_CameraRig_RestrictedTeleportZones.unity
Turn off both of your Vive Controllers
Enter play mode
Turn on one of your Vive Controllers
Attempt to teleport to a destination point

Expected behavior
Each destination point should disable the cursor crosshair, and show / hide the relevant gameobjects as they would normally.
Current behavior
The pointer is never hidden, and the destination points do not respond to player input. Teleporting to them works, but all destination options are ignored.

The version you've listed has borked the toggle-ability of the pointers. @bddckr  is working on a fix right now!

This is still an issue after the fix so I'll raise it as a bug

Fixed with https://github.com/thestonefox/VRTK/commit/09be398dafb42ad714f266b50bc3283bca7512cc


Add Support for Version 2023.3.*

Currently the Plugin is limited to IntelliJ Version 231—232.*
This should be extended to current version 233.*

I've updated the plugin compatibility so it will work with 233.* in 1958fd0. The new version should be available on the marketplace in a couple days once JetBrains validates it

That was fast!
Thank you a lot!


marp: 0.0.11 -> 0.0.12

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/marp/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 0.0.12 with grep in /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12
directory tree listing: https://gist.github.com/691bc668acd4e57a2cf187b6e1062f34

cc @puffnfresh for review

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: marp
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: marp
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12
shrinking /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12/lib/marp/libffmpeg.so
shrinking /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12/lib/marp/libnode.so
shrinking /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12/lib/marp/Marp
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12/lib  /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12/bin
patching script interpreter paths in /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12
checking for references to /tmp/nix-build-marp-0.0.12.drv-0 in /nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12...
/nix/store/q8kib6qhxndj4hzscdzhgvnjkaxm4mbw-marp-0.0.12

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: marp
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.


[Error] Downloading stash for Breach

DebugInfo.docx

Procurement 1.11.0 has been released, please retest and open a new ticket if you have any further issues.
https://github.com/Stickymaddness/Procurement/releases/tag/1.11.0


NRE if user has not roles

If user has not roles, exception throw here https://github.com/Nordes/IdentityServer4.LdapExtension/blob/7447e522ab34b37feb518635587b153a42dba59a/IdentityServer.LdapExtension/UserModel/OpenLdapAppUser.cs#L63
How can I get user roles from AD?

The role from AD are not the same between OpenLDAP and AD (Microsoft schema). An example of how to get the role is in the ActiveDirectory class. The file you're showing is related to OpenLDAP.
https://github.com/Nordes/IdentityServer4.LdapExtension/blob/7447e522ab34b37feb518635587b153a42dba59a/IdentityServer.LdapExtension/UserModel/ActiveDirectoryAppUser.cs#L67
Does it answer your question?

This is not yet fixed.


Save DataHash and LastResultsHash in ConsensusState

Description
This PR adds support of storing DataHash and LastResultsHash merkle roots in client's ConsensusState to verify inclusion of transaction on remote chain and their status of execution

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #1429 (ebc3e62) into main (b7b4400) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1429   +/-   ##
=======================================
  Coverage   80.34%   80.35%
=======================================
  Files         166      166
  Lines       12081    12087    +6
=======================================
+ Hits         9707     9713    +6
  Misses       1918     1918
  Partials      456      456

Impacted Files
Coverage Δ

modules/apps/29-fee/keeper/escrow.go
93.16% <100.00%> (-0.51%)
:arrow_down:

modules/apps/29-fee/keeper/events.go
100.00% <100.00%> (ø)

modules/core/02-client/keeper/keeper.go
85.41% <100.00%> (+0.12%)
:arrow_up:

...odules/light-clients/07-tendermint/types/header.go
100.00% <100.00%> (ø)

...odules/light-clients/07-tendermint/types/update.go
82.09% <100.00%> (+0.22%)
:arrow_up:

...dules/apps/27-interchain-accounts/types/account.go
78.26% <0.00%> (-0.62%)
:arrow_down:

modules/apps/transfer/keeper/grpc_query.go
78.26% <0.00%> (-0.32%)
:arrow_down:

...ps/27-interchain-accounts/host/keeper/handshake.go
84.61% <0.00%> (-0.30%)
:arrow_down:

...lients/06-solomachine/types/misbehaviour_handle.go
93.33% <0.00%> (-0.29%)
:arrow_down:

... and 23 more

Thanks for the PR, Mike. Could you open an issue to describe why this is necessary?
We only include AppHash, NextValidatorsHash, and Timestamp in consensus states because we want to store the minimal amount of information necessary to process IBC application state.
Core IBC and IBC applications would not be able to take advantage of DataHash and LastResultsHash because they are concepts specific to Tendermint.
In your issue, could you please detail the usecase(s) you have in mind and why including this information would be necessary.
Thank you!

@pr0n00gler, @oopcode I was wondering if you still require these changes, since there hasn't any discussion or issue been started in the spec repo. Since there's also not much that can be done in ibc-go at the moment, I was also thinking of closing this PR, if that's ok.

Closing this for now.


Better logging for commands in general

For some reason this command isn't working for me
Paste html as markdown
And I went to read the source code and debug it
However, it's really hard to pin point where the issue is happening
https://github.com/svsool/memo/blob/525e777487aa7321c7cf2ff2d5548676371338e2/src/commands/pasteHtmlAsMarkdown.ts#L11-L35
When I issue the command nothing happens. There could be multiple reasons why thats not happening

Is vscode bugged and not issuing the comamnd?
Is editor a falsy value?
Are selections empty?

I'm just going to manually use turndown for now but I hope you add more logging so its easier to troubleshoot

I mean I could clone it and go through the whole debugging myself but at the moment its just not worth the time

Maybe you could paste what you are trying to copy to reproduce the problem.
Also what version you are using and in which env.
Thanks

Will you add logging options?


Failing test with config override regression

Since 2.7.2, looks like brunch is not correctly merging some values when using the env option. I created a failing test case to illustrate this issue.
In the second test: overrides the config using the specified env the expected value for config.paths.watched is ['app', 'test'] but instead it returns ['app', 'test', 'vendor']. In 2.7.1 this test pass.

travis check failed

@paulmillr It is intended to fail. To demonstrate the regression that needs to be fixed (i.e. the PR did not come with a fix, only the test)

I've changed skemata to perform deep undefaulting which does fix this test. (https://github.com/brunch/skemata/commit/964582f9306fb4459815ad845602cf946273aaff)
@paulmillr can you release https://github.com/brunch/skemata and merge this PR?

I tested with the latest version and worked fine, thanks!
Would you guys accept some PR's improving the brunch test coverage?

Sure. We also are planning on improving our testing strategy, primarily with acceptance tests