"npm install" fails due to unmet webpack dependency

The package.json file calls for webback ^3.0.0, but this doesn't meet the peer dependency requirements of<EMAIL_ADDRESS>which is loaded for the karma-webpack ^2.0.0 dependency:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^2 || ^2.1.0-beta.0 || ^2.2.0-rc.0 but none was installed.
The same goes for the<EMAIL_ADDRESS>dependency.
The package file needs compatible versions of webpack, karma-webpack, and html-webpack-plugin.

Can you still run npm run dev? It works for me.
We have long used formally incompatible packages. E.g. using angular 1.6 while angular-material still required angular 1.5. There wasn't any problem in development or production, other than those npm warnings. Would be nice to selectively supress them.

I did a small update of the new package-lock.json file and ran npm install && npm run dev in a freshly cloned repo, it works fine here. Can you try with newest master?

Depending on the machine I run it on, npm run dev also fails... no idea. What do I need to change?
The last system complained about missing babel-generator.
(venv-3.5.3) ubuntu@box606:~/foodsaving-e2e/foodsaving-frontend$ npm install
removed 3 packages and updated 447 packages in 20.075s
(venv-3.5.3) ubuntu@box606:~/foodsaving-e2e/foodsaving-frontend$ npm run dev

><EMAIL_ADDRESS>dev /home/ubuntu/foodsaving-e2e/foodsaving-frontend
> test -f .env && env $(cat .env) gulp || gulp

[21:06:00] Requiring external module babel-register
module.js:487
    throw err;
    ^

Error: Cannot find module 'babel-generator'
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.Module._load (module.js:437:25)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/home/ubuntu/foodsaving-e2e/foodsaving-frontend/node_modules/istanbul-lib-instrument/dist/instrumenter.js:25:23)
    at Module._compile (module.js:569:30)
    at Module._extensions..js (module.js:580:10)
    at Object.require.extensions.(anonymous function) [as .js] (/home/ubuntu/foodsaving-e2e/foodsaving-frontend/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
[21:06:00] Requiring external module babel-register
module.js:487
    throw err;
    ^

Error: Cannot find module 'babel-generator'
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.Module._load (module.js:437:25)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/home/ubuntu/foodsaving-e2e/foodsaving-frontend/node_modules/istanbul-lib-instrument/dist/instrumenter.js:25:23)
    at Module._compile (module.js:569:30)
    at Module._extensions..js (module.js:580:10)
    at Object.require.extensions.(anonymous function) [as .js] (/home/ubuntu/foodsaving-e2e/foodsaving-frontend/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)

I deleted node_modules and package-lock.json, ran npm install and commited the resulting lock file to master: https://github.com/yunity/foodsaving-frontend/commit/ccf18ed34d25e5024252534eb934a6453168d895
Now you should get exactly the same package versions as me.

This should be resolved by now, but please let us know if the problem persists.


Document support for packaging projects

When creating non-trivial nugets with multiple projects included, the simple instructions in the current docs typically fall short and you need a proper packaging project instead.
This is already supported by nugetizer, but not documented at all. We need to document how a simple packaging project like:
<Project Sdk="Microsoft.Build.NoTargets/2.0.1" DefaultTargets="Pack">
  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
    <PackageId>MyPackage</PackageId>
    <Description>...</Description>
    <PackageTags>dotnet proxy runtime dynamic</PackageTags>
  </PropertyGroup>
  <ItemGroup>
     <!--  PackageReference/ProjectReference/Content/None/etc. items just work here too -->
  </ItemGroup>
</Project>

can be used to get very granular control of what's packed and where. The project needs to have a .msbuildproj extension.
May need multi-targeting feature from https://github.com/devlooped/nugetizer/issues/89 to be a comprehensive solution.

Done now


fix(deps): update dependency query-string to v6.11.1

This PR contains the following updates:

Package
Type
Update
Change

query-string
dependencies
patch
6.11.0 -> 6.11.1

Release Notes

sindresorhus/query-string
v6.11.1
Compare Source

Fix arrayFormatSeparator TypeScript type (#245)  338072a

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v3.0.52
GitHub release

Your semantic-release bot :package::rocket:


Blog : Customise theming in your Node-RED Dashboard 2.0

Related Issue
#2221
Description
Considerations:

Blog tile : Customise theming in your Node-RED Dashboard 2.0

Content refernce:
https://github.com/FlowFuse/website/pull/2353

I'm closing this one as it seems to be a duplicate from #2411


Convert CI from Travis CI to GitHub Actions

Rationale: https://www.jeffgeerling.com/blog/2020/travis-cis-new-pricing-plan-threw-wrench-my-open-source-works

Hmm... the deployments-rolling.yml test playbook is failing on some Node.js tests:
> examplenodejsapi@ test /opt/demo-nodejs-api
> mocha test --exit

restify listening at http://[::]:8080
  Hello
    /hello/:name endpoint
      ✓ returns a welcome message based on the name

  Ping
    /ping/:server endpoint
      1) tells us when a server is reachable

  Ping
    /ping/:server endpoint
      2) tells us when a server is unreachable

  1 passing (2s)
  2 failing

  1) Ping
       /ping/:server endpoint
         tells us when a server is reachable:
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure \"done()\" is called; if returning a Promise, ensure it resolves. (/opt/demo-nodejs-api/test/ping.js)
      at listOnTimeout (internal/timers.js:554:17)
      at processTimers (internal/timers.js:497:7)

  2) Ping
       /ping/:server endpoint
         tells us when a server is unreachable:
     Uncaught AssertionError: Body

    Actual:   '\"google.com is unreachable.\"'
    Expected: /\"google.com is reachable/

      at assert (node_modules/hippie/lib/hippie/assert.js:36:10)
      at value (node_modules/hippie/lib/hippie/expect.js:92:10)
      at verify (node_modules/hippie/lib/hippie/client.js:476:5)
      at statusCode (node_modules/hippie/lib/hippie/expect.js:23:5)
      at verify (node_modules/hippie/lib/hippie/client.js:476:5)
      at Client.verify (node_modules/hippie/lib/hippie/client.js:477:5)
      at /opt/demo-nodejs-api/node_modules/hippie/lib/hippie/client.js:437:12
      at Client.exports.raw [as parse] (node_modules/hippie/lib/hippie/parsers.js:33:3)
      at Request._callback (node_modules/hippie/lib/hippie/client.js:435:10)
      at Request.self.callback (node_modules/request/request.js:185:22)
      at Request.EventEmitter.emit (domain.js:483:12)
      at Request.<anonymous> (node_modules/request/request.js:1157:10)
      at Request.EventEmitter.emit (domain.js:483:12)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1079:12)
      at IncomingMessage.EventEmitter.emit (domain.js:483:12)
      at endReadableNT (_stream_readable.js:1241:12)
      at processTicksAndRejections (internal/process/task_queues.js:84:21)

Running those tests locally works:
$ npm test

> test
> mocha test --exit

restify listening at http://[::]:8080
  Hello
    /hello/:name endpoint
      ✓ returns a welcome message based on the name

  Ping
    /ping/:server endpoint
      ✓ tells us when a server is reachable (41ms)

  Ping
    /ping/:server endpoint
      ✓ tells us when a server is unreachable

  3 passing (105ms)

Building and running in the Vagrant environment seems to break though:
Uncaught Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

A different error in each case, maybe an environment issue or a Node.js version issue :-/

Inside Vagrant:
$ node --version
v12.20.0

On localhost:
$ node --version
v15.3.0

Disabled that test for now :/

And the jenkins.yml playbook is hitting:
TASK [geerlingguy.jenkins : Install Jenkins plugins using password.] ***********
changed: [localhost] => (item=ansicolor)

Which is breaking idempotence :(

Now the includes.yml playbook is hitting:
fatal: could not create leading directories of '/var/www/.cache/composer/vcs/https---git.drupalcode.org-project-coder.git

In the task Install Drupal dependencies with Composer..
le sigh

Created follow-up issue now that tests are green: https://github.com/geerlingguy/ansible-for-devops/issues/361


Add Github template files for PRs

We don't need these here, as the templates are coming from the organization repo.
The only exception would be the yaml for stale boot. But then again, we don't have issues here and the PRs usually internal.

We don't need these here, as the templates are coming from the organization repo.
The only exception would be the yaml for stale boot. But then again, we don't have issues here and the PRs usually internal.

Are you sure? I didn't see a template when creating the MQTT PR?!

Are you sure? I didn't see a template when creating the MQTT PR?!

You are correct about the PR template! 😁  This was the only one left on the individual repos to accommodate differences in the context. It's OK to have it here of course. Feel free to adapt the default one to make it more suitable to this sample context (more relaxed, no testing, docs, contribution, fixes this, etc).


Different colors for all bottom panel components.

This PR adds feature to set different colors for all UI components in bottom panel. It may be handy to set for example different color for active page indicator or for finish button. For example: If you use white background for your onboarding screens it looks better to use grey color for divider, unactive page dots and NEXT/SKIP button and some accent color for active page dot.
PR also removes duplicated code in ActivityHomePage which IMHO hasn't any purpose.

@JosefHruska could please take https://github.com/edsilfer/android-user-onboarding/issues/3


Fix duplicate ImageProcessing defines in NME particles

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15146/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15146/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15146/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15146/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15146/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15146/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15146/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15146/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15146/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15146/merge/testResults/webgpuplaywright/index.html


not working after 1epoch

It worked well in the old github code.
However, in the current github code, the following error occurs after 1epoch as follows.
[TypeError: unsupported operand type(s) for +: 'PerReplica' and 'PerReplica']
thanks...
#===================================================================================
Evaluating lfw:   0%|          | 0/63 [00:00<?, ?it/s]WARNING:tensorflow:Using MirroredStrategy eagerly has significant overhead currently. We will be working on improving this in the future, but for now please wrap call_for_each_replica or experimental_run or run inside a tf.function to get the best performance.
WARNING:tensorflow:Using MirroredStrategy eagerly has significant overhead currently. We will be working on improving this in the future, but for now please wrap call_for_each_replica or experimental_run or run inside a tf.function to get the best performance.
Evaluating lfw:   0%|          | 0/63 [00:01<?, ?it/s]
Traceback (most recent call last):
File "C:/Users/facetag/Face_project/my_insight2/real_train.py", line 44, in
tt.train(sch, 0)
File "C:\Users\facetag\Face_project\my_insight2\train.py", line 460, in train
self.train_single_scheduler(**sch, initial_epoch=initial_epoch)
File "C:\Users\facetag\Face_project\my_insight2\train.py", line 443, in train_single_scheduler
self.basic_train(initial_epoch + epoch, initial_epoch=initial_epoch)
File "C:\Users\facetag\Face_project\my_insight2\train.py", line 349, in basic_train
workers=4,
File "C:\Users\facetag\Face_project\venv\lib\site-packages\keras\engine\training.py", line 1230, in fit
callbacks.on_epoch_end(epoch, epoch_logs)
File "C:\Users\facetag\Face_project\venv\lib\site-packages\keras\callbacks.py", line 413, in on_epoch_end
callback.on_epoch_end(epoch, logs)
File "C:\Users\facetag\Face_project\my_insight2\evals.py", line 37, in
self.on_epoch_end = lambda epoch=0, logs=None: self.eval_func(epoch, logs, eval_freq=1)
File "C:\Users\facetag\Face_project\my_insight2\evals.py", line 87, in eval_func
embs = self.do_predict_distribute()
File "C:\Users\facetag\Face_project\my_insight2\evals.py", line 68, in do_predict_distribute
emb = emb + emb_f
TypeError: unsupported operand type(s) for +: 'PerReplica' and 'PerReplica'

Uh, It's caused by a change in evals.py, that my trying to fit TF 2.8.0, reversed. Currently will not work for TF 2.8.0-rc0. I may take more tests, and change that part again.

It should work for both TF 2.8.0 and earlier version now.

Fix again... Seems I need more tests on this...


Frontend admin support for Peace

This adds admin support for peace events and fixes an Usaneko "bug" where you are wished a happy new year every time you coin in or use paseli.

Thanks for the PR, going to check out dark mode now!


WebsiteRedirectLocation system-defined S3 object's metadata value not copied using multipart copy

When calling https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/s3/object/copy.html with https://boto3.amazonaws.com/v1/documentation/api/latest/reference/customizations/s3.html#boto3.s3.transfer.TransferConfig passed to a call so that it triggers multipart upload for any copy operation (Config=TransferConfig(multipart_threshold=1)) and when passing ExtraArgs with WebsiteRedirectLocation key set to a specific value moto ignores this value and copies object with WebsiteRedirectLocation set to None. When running this code against AWS WebsiteRedirectLocation value is copied correctly.
Versions:
moto==4.2.11
boto3==1.26.90
s3transfer==0.6.2

Thanks for raising this @donsokolone. It looks like there are a few problems in this area, as we do not respect the ChecksumAlgorithm-parameter either.
Will raise a PR with a fix shortly.


Can't init addon

Problem/Motivation
Can't init the addon
Expected behavior
Initialice
Actual behavior
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 00-banner.sh: executing...
Add-on: SteVe
OCPP server for EV charging stations
Add-on version: 1.2.0
You are running the latest version of this add-on.
System: Home Assistant OS 5.12  (aarch64 / raspberrypi4-64)
Home Assistant Core: 2021.3.3
Home Assistant Supervisor: 2021.03.4
Please, share the above information when looking for help
or support in, e.g., GitHub, forums or the Discord chat.
[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] 10-create-config.sh: executing...
[19:52:39] ERROR: Got unexpected response from the API: Service not enabled
[cont-init.d] 10-create-config.sh: exited 139.
[cont-finish.d] executing container finish scripts...
[cont-finish.d] 99-message.sh: executing...
            Oops! Something went wrong.

We are so sorry, but something went terribly wrong when
starting or running this add-on.
Be sure to check the log above, line by line, for hints.
[cont-finish.d] 99-message.sh: exited 0.
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.

Hi.
You need to have the Official MariaDB add-on installed and running.
SteVe will create a database on startup to store its data.

10 mars 2021 kl. 19:56 skrev angelsanchezz<EMAIL_ADDRESS>
Problem/Motivation
Can't init the addon
Expected behavior
Initialice
Actual behavior
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 00-banner.sh: executing...
Add-on: SteVe
OCPP server for EV charging stations
Add-on version: 1.2.0
You are running the latest version of this add-on.
System: Home Assistant OS 5.12 (aarch64 / raspberrypi4-64)
Home Assistant Core: 2021.3.3
Home Assistant Supervisor: 2021.03.4
Please, share the above information when looking for help
or support in, e.g., GitHub, forums or the Discord chat.
[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] 10-create-config.sh: executing...
[19:52:39] ERROR: Got unexpected response from the API: Service not enabled
[cont-init.d] 10-create-config.sh: exited 139.
[cont-finish.d] executing container finish scripts...
[cont-finish.d] 99-message.sh: executing...
        Oops! Something went wrong.

We are so sorry, but something went terribly wrong when
starting or running this add-on.
Be sure to check the log above, line by line, for hints.
[cont-finish.d] 99-message.sh: exited 0.
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Update version of iced and iced_aw

Required in order to solve the dependency issue on web-sys that prevents us from updating iroh-net.
That blocks the discovery and web-build.

Due to dependency issues, the update to these is included in the same PR that's updating iroh-net
https://github.com/andrewdavidmackenzie/pigg/pull/343

Duplicate of https://github.com/andrewdavidmackenzie/pigg/issues/466


[ko][zh-cn] How to get chinese/korean fonts to work in matplotlib + Colab?

The default Matplotlib setup in Colab doesn't include Chinese or Korean fonts, so these characters don't render.
I believe this is one of the reasons we have not been translating figure text.
I can get the browser to render this text by outputting svg-text:
from IPython.display import set_matplotlib_formats
set_matplotlib_formats('svg')
matplotlib.rcParams['svg.fonttype'] = 'none'

But that messes up a lot of the formatting.
Some searching shows that it might just be a matter of installing the right fonts and adding them to the matplotlib.rc configuration, but I haven't found a end-to-end setup that works yet.
Does anyone have experience setting this up?

Status: still broken.
The reference from TFHub is that they have one multi-lingual tutorial that breaks if they push it to the site because the fonts are not available.
If this is high enough priority, we could upstream Haesun's font setup (+fonts for Chinese) into Colab and the site generator. Worth it? or should we just close this?

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


[P0] Intervention scheduling for generation

Description
Basic functionality for scheduling interventions to happen on positions not present in the prompt (i.e. generated tokens). Ideally should follow the same procedure for GRU.
Changelog:

timestep_selector, a list of length num_intv of boolean callbacks with signature Callable[[int, torch.Tensor], bool] can be passed to generate() calls. Each intervention calls its callback function with the current position to determine whether the intervention should operate on that position or not.
New handling of Nonevalues in unit locations: If Nones are specified at the batch dimension then interventions are not applied to those examples in the batch.
weird logic where _intervention_getter(), _intervention_setter() functions were being called with single interventions even though they were written to handle an array of intervention keys and return a list of handlers, has been removed
Efficiency and readability improvements in gather_neurons() and scatter_neurons()

Testing Done

Tests added: test_nulling_intervention, test_generation_with_source_intervened_prompt, test_dynamic_static_generation_intervention_parity, test_generation_noops
Tests fixed: test_with_subspace_negative, test_scatter_neurons_gpt2_attn_with_head_positive

Checklist:

[x] My PR title strictly follows the format: [Your Priority] Your Title
[ ] I have attached the testing log above
[x] I provide enough comments to my code
[x] I have changed documentations
[x] I have added tests for my changes

Is this PR done? I would really like to use this functionality for my ongoing project.


[8.x] Skip date casting for custom casts

Hey!
I just noticed, when you have custom casts on attributes which are also in the $dates property, they will first be cast to an date string and after that, the custom cast would be applied.
Considering this, custom casts could not be shared between dates that are defined in the $dates property and not. But why not just using the $dateFormat property? This property will be applied to all date attributes. So, mixed formats are not supported.
Example:
class Task extends Model
{
    protected $casts = [
        'created_at'  => DateTimeWithMicrosecondsCaster::class,
        'finished_at' => DateTimeWithMicrosecondsCaster::class,
        'updated_at'  => 'datetime:Y-m-d H:i.s',
    ];
}

class DateTimeWithMicrosecondsCaster implements CastsAttributes
{
    public function get($model, string $key, $value, array $attributes)
    {
        return is_null($value)
            ? null
            : Carbon::createFromFormat('Y-m-d H:i:s.u', $value);
    }

    public function set($model, string $key, $value, array $attributes)
    {
        return is_null($value)
            ? null
            : $value->format('Y-m-d H:i:s.u');
    }
}

Note: This will be a breaking change, since users will eventually have to change their custom cast classes.

An attribute should never appear in the cast AND the date properties. Use one or the other.


Upgrade rxjs dependency to 6.x?

Hey guys, when are you planning to upgrade your rxjs dependency to latest (rxjs6 is official now)?
Currently kinvey-nativescript-sdk references<EMAIL_ADDRESS>-- that will cause a problem with the imminent release of Angular6 support in NativeScript (ng6 implies<EMAIL_ADDRESS>and it comes with its fair share of breaking changes).
So the end result is that if you create a {N} app with Angular6 and kinvey-nativescript-sdk it cannot even compile unless one installs the compatibility layer provided by rxjs-compat and we would eventually like to drop it to reduce app size.

@thomasconner Can you check this?

Yes I can

Since kinvey-nativescript-sdk/package.json still lists<EMAIL_ADDRESS>as a dependency, a quite workaround is to change lines 1-3 in kinvey.d.ts from:
import {
  Observable
} from 'rxjs/Observable';

to:
import {
  Observable
} from 'rxjs';

@thomasconner it has been 2.5 months now that this issue is open. Any updates?

I have just submitted a PR that updates rxjs to version 6. Our next release is scheduled for August 8. It will be part of that release.

@thomasconner any updated estimates for the next release?

The release will be made by the end of the day next Wednesday (August 22).

@thomasconner any updates on this one? it has been almost a month after August 22 but no release?

@manoldonev We are in the process of making a release within the next week that will contain this fix.

This has been fixed with 3.11.7


User profile

Creating pull request to main (user-profile nearly completed - just doesn't include the users posts on their account yet)

Closed because idk how to merge it to main even though i resolved the conflicts


Add Http Client Adapter to allow usage of different http clients

Hey @jcarvalho-ptech Thanks for this PR. Feature definitely sounds interesting.
I'll test it with couple of different clients and I'll update the PR.

Thanks @Imgkl !
This would allow us to use any http client. An example using Chopper:
class EventFluxClientAdapter implements HttpClientAdapter {   @override   Future<StreamedResponse> send(BaseRequest request) async {     chopper.ChopperClient client = GetIt.I.get(instanceName: "chopperClient");     StreamedResponse response = (await client.send<Stream<List<int>>, String>(             chopper.Request("GET", request.url, Uri.parse(""),                 headers: request.headers)))         .base as StreamedResponse;     return response;    ...
And feel free to improve the PR!

@jcarvalho-ptech Thanks for the example.


doc: in README.md examples use vim.api.nvim_set_hl instead of vim.cmd([[highlight ...]])

no need to use vimscript for this :)

This plugin is backwards compatible until Neovim 0.5 (for the most part)
nvim_set_hl is newer than that.
I want to deprecate older Neovim versions in v3. Please leave the PR open, I'll merge it then.

Code in v3 changed too much, I'll close this


Correct metadata for Array parameters and setpoint arrays

Fixes #499
@QCoDeS/core

This should probably have some tests

This is not quite as simple as setting these arrays to setpoint changes their names by appending a _set. I'm not sure we want that?

Decided to simplify this. Now only fixes the missing unit propagation from Multiparameter to the arrays


Persistent Ore cluster processing

You get and ore cluster item when using the persistent drill on an persistent ore.
It needs to be processed in some way before you can actually get the metal from it.

for now clusters are crushed into ores https://github.com/Aiz0/create-grand-expanse/pull/212


Migrate from PyPDf2 to pypdf.

PyPDF2 has been deprecated and renamed to pypdf. We have also found that there are documents that pypdf can process that PyPDF2 cannot.

IT failed to import pypdf

IT failed to import pypdf

Oops. I updated all the poetry.lock files, but I missed the one at the root, which is of course the one that matters for the integ tests. Fixed now.


PHPCast: Move all the API interaction into Wrappers.

Move all the API interaction into a wrapper will allow for the PHP application to call methods outside of the API meaning the worker system when implemented will be able to make use of the wrapper.

This issue has now been fixed and now has been rolled out.


Allow disabling default elasticsearch output for parser job

For those cases where you want to specify all your own outputs via logstash_parser.outputs.
This issue was moved from https://github.com/logsearch/logsearch-boshrelease/issues/191 original issue.

Merged in https://github.com/cloudfoundry-community/logsearch-boshrelease/pull/15


Missing path in AndroidSdkWindows

I am attempting to build e4k on a fresh installation of Windows and running into a missing Android SDK exception.
In AndroidSdkWindows.cs under external\Xamarin.Android.Tools, there are a number of locations listed that are used to search for the Android SDK. The problem is that a clean install of Visual Studio does not create any of those registry entries listed. The closest I can find is
var android_default = Path.Combine (OS.ProgramFilesX86, "Android", "android-sdk-windows")
but the path VS uses is "android-sdk" not "android-sdk-windows".
Since this is under external, I am not sure what the rules are to fix this. Removing the windows part of the string does let the build complete. I don't see any Environment Variables in that class that I can set to point to the right SDK, and I don't really want to create a registry entry following one of the paths in the file to find the SDK, but I will do that if its the recommended fix!

Looks like we are already on the latest version of https://github.com/xamarin/xamarin-android-tools/blob/master/src/Xamarin.Android.Tools.AndroidSdk/Sdks/AndroidSdkWindows.cs so this seems like a bug in Xamarin.Android.Tools.
@Zomb I think the way forward might be to send a PR to Xamarin.Android.Tools repo and add a check for "android-sdk" in addition to "android-sdk-windows".
cc'ing @dellis1972 from Android team. Does that approach sound ok?

There are a number of persons who also depend on the nugent package to run embeddinator. Would this fix also go in for the nugent package (Pardon me I am asking this in the wrong area :)


Auto-Pause Monitor

Would it be possible to implement an option in order to pause a Monitor for a certain timespan during the day?
For example: My Router does a Daily-Reconnect at 5am, which causes "false-alerts" on a daily basis. I would like to define something like "don't run this monitor during 05:00-05:15 am".
Thanks!
Is it a duplicate question?
No
Info
Uptime Kuma Version: latest
Using Docker?: Yes
OS: Ubuntu 20.04 LTS
Browser: Firefox

Duplicate of #191 and #204.


chore: Adds a tooltip for the alert's SQL input

SUMMARY
It's not very clear that the SQL used to create an alert should return a numeric value. When non-numeric values are returned, the error Alert query returned a non-number value is thrown when firing the alert. This PR adds a tooltip for the alert's SQL input to instruct users in advance.
Closes https://github.com/apache/superset/issues/26298
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS
1 - Open the alert creation modal
2 - Check the displayed tooltip
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@michael-s-molina technically it doesn't need to be numeric, i.e., it could be a string, as we use the Python float() method. Maybe something like:

... a numeric-esque value

While we're at it, should we also simply remove tooltip for the threshold value,

given that it technically doesn't need to be a double precision number, e.g., an integer will work. It seems like the input box rejects non-numerical values (excluding the period) and thus the input in sanitized.

@john-bodley Applied your suggestions.


Postgres migrations: incomplete transaction handling

Abstract
For postgres, the migration script and the _sqlx_migrations state change run in two different transactions, which requires that the migration scripts are idempotent. This is quite hard to archive for certain scripts. It would be nice if both parts would run within the same transaction.
Technical Details
See this code here:
https://github.com/launchbadge/sqlx/blob/d9fd21c94e57db3e02bf2ef6b3a9cc94296ce64a/sqlx-core/src/postgres/migrate.rs#L217-L247
Now imagine that the process dies here:
https://github.com/launchbadge/sqlx/blob/d9fd21c94e57db3e02bf2ef6b3a9cc94296ce64a/sqlx-core/src/postgres/migrate.rs#L229
Now the migration script was executed and committed, but SQLx forgot about that fact. A subsequent migrator run will execute the script again.
References
I'm quite certain that I've observed that issue with InfluxDB IOx (commit 500ece7c138e6b4f04b6761c68bd447ad69a6845) which uses sqlx = 0.6.0 and postgres 13.6.

I do agree, it seems strange for the insert into _sqlx_migrations to happen outside of the transaction. I believe it's so that execution_time includes the transaction commit (which is where most of the work is done), but it also seems strange to store that in the database in the first place.

To keep execution_time while fixing this issue, I would do the insert into _sqlx_migrations inside the transaction with execution_time set to 0, then commit the transaction and update execution_time afterward.

To keep execution_time while fixing this issue, I would do the insert into _sqlx_migrations inside the transaction with execution_time set to 0, then commit the transaction and update execution_time afterward.

That sounds like a good way to solve it. Would a nested transaction construct also work?

I don't see why migration need that execution time in first place. I just played with migration yesterday and felt awkward with that column. I always think that the execution time for a migration isn't that important, and it'll be difference with each computer run it too so not much point for store that value.


Openshift Provider

New Provider Request
Details

Name: Openshift
URL: https://www.redhat.com/de/technologies/cloud-computing/openshift
Modes: read
GitHub projects: https://github.com/openshift
API Documentation: https://docs.openshift.com/container-platform/4.12/rest_api/index.html

Hey @bnova-stefan
thanks for the idea, If you're interested, would you consider opening a pull request with your suggested solution?
We here to help you if you needs


Use ordered dict for values

Hello!  The choices attribute is a list of tuples that’s convenient for a django model field declaration, but recently I had a need to convert that to an OrderedDict to send it in a JSON response.  I noticed that a dict can be found as the values attribute, but it isn’t ordered.

Fixed with https://github.com/bigjason/django-choices/pull/45, thanks!


feat : Merging FHIR SDK and jwe helper to integration SDK

@maheshkumargangula  -  Fixed all the feedback comments , Please review .


'rpi_rf' has no attribute 'RFDevice'

I've just upgraded ha to the latest version and needed to use your integration but upon adding via hacs I get the following error.
I've tried all sorts to fix it. Any idea?
Logger: homeassistant.components.switch
Source: custom_components/rpi_rf/switch.py:69
Integration: Switch (documentation, issues)
First occurred: 13:56:13 (3 occurrences)
Last logged: 13:56:13
Error while setting up rpi_rf platform for switch
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
await asyncio.shield(task)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/rpi_rf/switch.py", line 69, in setup_platform
rfdevice = rpi_rf.RFDevice(gpio)
AttributeError: module 'rpi_rf' has no attribute 'RFDevice'

It appears that your system might not have installed the dependency library "rpi_rf" as that's where the rpi_rf.RFDevice should be called from.
I'm presuming you restarted Home Assistant after install and before you add the lines to the configuaration file?  Could you comment out any lines that you added for this integration and uninstall and reinstall and restart HA.

I did all of that. Same error or a variation of it. And yes, restarted without any config (which doesn't create an error) but as soon as I add the config and restart I get the error.
Is there a way to manually install the library?
I tried getting the code from a previous release and had the same error. I've

i reinstalled my backup again and its now working


Fix missing parameter needed at least on android

As mentioned in #11 no matter of how the jni is integrated for android it fails as local referenced is deleted or something.
After debugging it on my android device i found out that for innerclasses the superclass must be passed as well.
https://stackoverflow.com/questions/25363027/jni-getmethodid-not-working-for-constructor-of-inner-class
I dont have a clue why it is working on common jvms for linux or windows but it crashes on android.
I tested the fix locally on my android device and with ci in github for x86_64 and arm64:
https://github.com/Hatzen/LibOQSTestApp/runs/2249616782
I didnt test it yet for previous working common jvms. But i hope circle ci can do this at this point?

@Hatzen I tested it and it works on Linux. There is a similar instance on Signature.c in line 49 that maybe needs the obj as well.

Good point, thanks for advice.
I just ported the SigTest.java to android and tried running it (additionally i found out my tests didnt worked correctly yet..) i had to fix an other issue with not finding java.lang.Long class for some reason. java.lang.object a few lines above seems to work for some reason..
Changes made regarding to: https://github.com/Hatzen/LibOQSTestApp/commit/0fe8541ffd457aeb2f7bc9ace1f3fdd88930ddf7


Updated hive version to 2.3.4 and added changelog file

Added initial release date to the changelog according to Github but can add more detail on other changes if appropriate?

@teabot, @nahguam, @patduin this is ready for another review when you have the time, thanks!


Generated video with third-party watermarks

System Info / 系統信息
As tile, The generated videos appears as a "pond5" watermark.
Is this normal?
Information / 问题信息

[ ] The official example scripts / 官方的示例脚本
[ ] My own modified scripts / 我自己修改的脚本和任务

Reproduction / 复现过程

Deployment of CogVideo 2B models on the cloud
The generated videos by text appears as a "pond5" watermark

Expected behavior / 期待表现
I hope the officials can explain and provide solutions as much as possible.

I believe this should be the 2B model, not the 5B model. The 2B model has a slight issue during the SFT stage, which causes it to have a pond5 watermark under certain prompts. We haven't maintained this version of the model much. Our focus of research is on the 5B model, which does not have this issue.
Currently, the issue with the 2B model cannot be resolved.

I can confirm 5b does do it rarely. I'm cataloging out every gs and steps combination possible on my hardware to see if there's any pattern or reason to it.
Hopefully we can just avoid certain maths.


Smart pointer constructors for Secure Session

Using legacy C++03 interface of Secure Session may be a bit hard with modern practices introduced by C++11. Let's provide a more idiomatic interface where Secure Session assumes ownership over the provided instance of secure_session_callback_interface_t via smart pointers.
We do so using the shared_ptr instance. This allows both shared and unique ownership over the provided callback interface. For example, if the users need only get_public_key_for_id interface then they can share an instance of secure_session_callback_interface_t between all Secure Sessions. However, if they need to use the transport interface (send_data/receive_data) then they are likely to provide a unique instance for each Secure Session.
We keep the old non-owning constructor for backwards compatibility. However, we mark is deprecated to incite the users to move on to a more safe and idiomatic interface with smart pointers.
We still need to test the old interface so silence the relevant warning in this file.

Let's wait for #379 to be merged and rerun the CI build with that.


journal: omap implementation for volumegroup

Implement the required function to store/retrieve the details from the omap for the snapshotgroup.
This adds a new omap object that contains the mapping of the RequestName and all the volumeID and its corresponding snapshotID belonging to a group.
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@mergifyio requeue

@Mergifyio queue


Normalize session errors to align with regristration

There was inconsistency between error messages from session vs. registration validation.
I've updated the session controller to follow the same pattern as the one used in registration.
Invalid password:
{
    "errors": [
        "Invalid login credentials. Please try again."
    ]
}

{
    "errors": {
        "full_messages": [
            "Invalid login credentials. Please try again."
        ]
    },
    "success": false
}

(the added success key must be a change from the released gem to the master-branch checkout - but it does not match up with registration use of error)
An error in registration:
{
    "data": {
        "created_at": null,
        "email"<EMAIL_ADDRESS>        "id": null,
        "provider": "email",
        "uid": "",
        "updated_at": null
    },
    "errors": {
        "full_messages": [
            "Password can't be blank"
        ],
        "password": [
            "can't be blank"
        ]
    },
    "status": "error"
}

I don't agree with this PR, it's weird that you reply with full_messages when it isn't an active record's error. In fact, we should change to:
{
    "error": "Invalid login credentials. Please try again."
}

That's something we can do for the next version or use a config like Zach said.

Closing this, we should make a breaking change returning error: "string message" instead


Deprecate X64 flag

This is the external tracking bug for deprecation of the X64 flag.
The final desired outcome here is the following:

X64 flag no longer exists
jax.numpy functions (e.g. jnp.arange, jnp.zeros, etc.) default to 32-bit, unless an explicit dtype is requested.
Python scalars default to weak-typed 64-bit

The plan is roughly the following:

Change jax.numpy function defaults to 32-bit. This should only affect users of X64 mode, and will require adding explicit 64-bit dtypes to some code. At this point, enabling X64 will produce the desired long-term behavior.
Change the default X64 flag state to True. Users can still recover X32 behavior by explicitly setting it to False
Deprecate setting the X64 flag
Remove the X64 flag

The main rationale is that the modalism in the API complicates JAX both from a user perspective and a developer perspective.
For the user, there's no way to reason about how a particular block of code will behave without global knowledge of the environment in which it is run (e.g. the value of the JAX_ENABLE_X64 environment variable at the time that JAX was imported).
Additionally, due to the way JAX tracing works, attempts to allow local overrides of the global state face fundamental challenges (e.g. https://github.com/google/jax/issues/5982).
Confusion about the x64 flag is one of the most common drivers of user confusion in our feedback forums.
Other similar frameworks (e.g. pytorch and tensorflow) do not have any such global modalism.
On the developer side, the global flag doubles the amount of CI testing we need to cover the package, and means that every line of code in the package must be written with this modalism in mind.
These are some of the reasons we've decided to explore the possibility of deprecating the X64 flag. I'm curious to hear your thoughts on those reasons! Thanks

May I ask in terms of specific examples?
How would jax.numpy.linalg.svd behave?  Would it always do the computation in 64-bit or would it depend on whether the type of the argument a is 64-bit float or 32-bit float? Or would the implementation internally increase precision to give accurate results even though it finally converts the output singular values to the data-type of a?
Several functions use an internal helper function _promote_arg_dtypes  to promote to an inexact type if the input is an integer type.  Would these functions then promote to 32-bit or 64-bit?
CPUs usually have 64-bit floating-point units by default. So 32-bit floating-point computation doesn't lead to computational savings? NumPy usually goes for 64-bit computation by default. Would JAX become more aligned with it or would it have different precision management rules?

The general idea is that JAX would do computations in the precision of the input type. In the presence of mixed inputs, JAX would decide the output type based on its existing Promotion rules.

Hi @jakevdp, I can see that this is a work in progress, so potentially annoying of me to ask this, but is it currently possible to emulate the behavior of
JAX_ENABLE_X64=True JAX_DEFAULT_DTYPE_BITS=32

without using environment variables? I am working on a small extension to a larger code-base, where I only need 64-bit dtypes in the extension and want to (as far as possible) preserve the behaviour of the rest, which was written with x64 disabled.
When I run
from jax.config import config
config.update("jax_enable_x64", True)
config.update("jax_default_dtype_bits", 32)

assert jnp.array([1.], dtype='float64').dtype == jnp.float64
assert jnp.array([1.]).dtype                  == jnp.float32

I currently get an assertion error from the final line, i.e. the config.update("jax_default_dtype_bits", 32) doesn't seem to have had any effect. Running the same snippet with the above environment variables set does work, but I'd like to avoid having to use environment variables if possible.
I also don't want to rely on API that is about to be removed, so if there is a better overall approach to this then please let me know.

Unfortunately the effect of the flag happens at import time, so changing the value after import will have no effect.
If you want to avoid setting system variables outside your Python process, you could do something like this:
import os
os.system['JAX_DEFAULT_DTYPE_BITS'] = '32'

import jax

So long as jax is not imported until after the system variable is set, it should work.

Thanks, that should do the trick for now.

@jakevdp do you still plan to deprecate the flag, or you changed plan and this likely won't happen anymore?

This has proven difficult to deprecate, because so many users rely on the current behavior. I'm going to leave this open, but de-prioritize it.

I think we've come to the conclusion that we can't remove the X64 flag, given the extent that users depend on it. I'm going to close this.


v0.10.4 - Markdown help improvements

0.10.4

Enhancements

Fixed new markdown hover help for QB64 PE keywords up to version 3.13.1

Now works with config.helpPath

For how the markdown is generated look here: https://github.com/grymmjack/qb64pe-wiki-to-markdown

Damnit. I did it again. Sorry.


Delete by version name

Would be nice to have an option to delete version by version name (not version id which is hard to fetch based on name). In my workflow I publish packages upon new release based on tag name. And upon release deletion I remove tag and associated with it deployed version.
Currently I has to resort to regex negative look-ahead to achieve this feature:
- name: Delete published package
  uses: actions/delete-package-versions@v2
  with:
    package-name: "${{ github.event.repository.name }}"
    ignore-versions: ^(?!${{ github.event.release.tag_name }}).*$
    token: ${{ secrets.GITHUB_TOKEN }}

Since you already checking version names via regex, wouldn't it be easy to implement
package-version-names which would work same way as package-version-ids except would match version names ?
Thanks in advance

We wrote an action https://github.com/castlabs/get-package-version-id-action to search for the package ids based on the version name. The main reason was to combine it with delete-package-versions to remove those versions. Its limited to packages in the repository that runs the workflow currently, but maybe it helps non the less.


[BUG] fix default versions not overrided in prepare-ldh-image.sh with mvn

What is the purpose of the change
fix default versions not overrided in prepare-ldh-image.sh with mvn
Related issues/PRs
None
Brief change log

fix default versions not overrided in prepare-ldh-image.sh with mvn

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[x ] I have explained the need for this PR and the problem it solves
[ x] I have explained the changes or the new features added to this PR
[ x] I have added tests corresponding to this change
[ x] I have updated the documentation to reflect this change
[ x] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[ x] If this is a code change: I have written unit tests to fully verify the new behavior.

mv to #2887


Wait for closed resources to actually close before detecting open handles

Summary
Note: this contains two major changes (waiting before detecting open handles, detecting indirectly opened handles), but they affect the same part of the code and one revealed the other as an issue, so it seemed simpler to combine them. I’ve made separate commits for each so it’s a little easier to see here, but am also happy to break them into separate PRs.
Some types of async resources in Node.js are not destroyed until after their close or end or similar callbacks/events run, leading to a situation where the --detectOpenHandles option can falsely flag resources that have been cleaned up in user code and are already scheduled for destruction. For example, if a test ends from the callback to TCPServer.close() and no other tests or lifecycle functions impose additional delays, --detectOpenHandles will flag the server as open even though it has been closed. This is also the main cause of issues people encounter with Supertest (see #8554).
Example:
const {createServer} = require('http');

test('--detectOpenHandles flags this even though the server was properly closed', done => {
  const server = createServer((_, response) => response.end('ok'));
  server.listen(0, () => {
    // Close server and return immediately on callback. During the "close"
    // callback, async hooks usually have not yet been called, leading to Jest
    // flagging the server as an open handle.
    server.close(done);
  });
});

This addresses the issue by adding a short delay before collecting open handles. It’s similar to some work @SimenB implemented (and then unimplemented) here: https://github.com/facebook/jest/pull/11278/commits/74ed376a9149a3e9930eff38f841af2f76317858

Unfortunately, this got a little bit more complex because the delay caused a test to start failing, which revealed a new issue: It turns out that some arguments to TCPServer.listen() cause the TCPSERVERWRAP async resource for the server to be created indirectly, where its stack trace has no reference to the user code that was ultimately responsible for it, which causes Jest not to track the resource.
I’ve added tracking for those indirectly created resources here. When we track an indirectly created resource, we use the stack trace from the original call that triggered the indirect created it so the user can still see the call in their code that’s causing the problem.
For example, code like:
import {createServer} from 'http';
(function userCode() {
  server.listen(0, () => {
    console.log('ok');
  });
})();

Leads to async resource creation/destruction like:
init ( asyncId: 2, type: "TCPSERVERWRAP", triggerAsyncId: 1 )
  Stack: Error: TCPSERVERWRAP
    at TCP.emitInitNative (node:internal/async_hooks:204:43)
    at createServerHandle (node:net:1230:14)
    at Server.setupListenHandle [as _listen2] (node:net:1273:14)
    at listenInCluster (node:net:1354:12)
    at Server.listen (node:net:1441:7)
    at userCode (/Users/rbrackett/Dev/async-hooks-test/index.js:52:10)

But
(function userCode () {
  const server = http.createServer((_request, response) => response.end('ok'));
  server.listen({host: 'localhost', port: 0}, () => {
    console.log('ok');
  });
})();

Leads to async resource creation/destruction like:
init ( asyncId: 2, type: "DNSCHANNEL", triggerAsyncId: 1 )
  Stack: Error: DNSCHANNEL
    at ChannelWrap.emitInitNative (node:internal/async_hooks:204:43)
    at new Resolver (node:internal/dns/utils:48:20)
    at node:internal/dns/utils:133:23
    at NativeModule.compileForInternalLoader (node:internal/bootstrap/loaders:312:7)
    at nativeModuleRequire (node:internal/bootstrap/loaders:341:14)
    at node:dns:44:5
    at NativeModule.compileForInternalLoader (node:internal/bootstrap/loaders:312:7)
    at nativeModuleRequire (node:internal/bootstrap/loaders:341:14)
    at lookupAndListen (node:net:1486:32)
    at Server.listen (node:net:1437:7)
    at userCode (/Users/rbrackett/Dev/async-hooks-test/index.js:64:10)
init ( asyncId: 3, type: "GETADDRINFOREQWRAP", triggerAsyncId: 1 )
  Stack: Error: GETADDRINFOREQWRAP
    at GetAddrInfoReqWrap.emitInitNative (node:internal/async_hooks:204:43)
    at Object.lookup (node:dns:153:21)
    at lookupAndListen (node:net:1487:7)
    at Server.listen (node:net:1437:7)
    at userCode (/Users/rbrackett/Dev/async-hooks-test/index.js:64:10)
init ( asyncId: 4, type: "TCPSERVERWRAP", triggerAsyncId: 3 )
  Stack: Error: TCPSERVERWRAP
    at TCP.emitInitNative (node:internal/async_hooks:204:43)
    at createServerHandle (node:net:1230:14)
    at Server.setupListenHandle [as _listen2] (node:net:1286:14)
    at listenInCluster (node:net:1354:12)
    at GetAddrInfoReqWrap.doListen [as callback] (node:net:1492:7)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:71:8)
destroy ( asyncId: 3 )

Note that the TCPSERVERWRAP stack does not have userCode in it, and that the GETADDRINFOREQWRAP resource (which does have userCode in the stack) was destroyed.
The changes here allow us to detect that TCPSERVERWRAP  resource and associate it with the same line of user code as the GETADDRINFOREQWRAP resource.
I also increased the size of the stack trace that we check for entrypoints to user code. The V8 default is 10, which caused us not to track resources like the DNSCHANNEL above because it has more than 10 stack frames within node internals, causing the wrapper functions that mark this as user code to get cut off the end of the stack trace.
Fixes #8554
Closes #9532
Test plan
This PR includes several unit and end-to-end tests that should cover the issues this is meant to solve.

I don’t think I have time for this today, but will make the changes you suggested and try to add the relevant tests from #9532 and #11278 sometime in the next few days. 🙌

Awesome @Mr0grog, thanks! I can probably try myself if you will be busy for the next couple of days 🙂

The test added here is better than the stuff I was doing in #9532. I pulled in the test from #11278.
This is awesome 😀

Whoa, you cleaned this up and merged it in quick! Thanks. 😄 🙌

Thanks for the awesome PRs for this feature @Mr0grog! It's a really cool one that has had a couple of blemishes for a long time you've smoothed out 👍
This PR is included in 27.0.0-next.10 (stable coming next week)

Woohoo! Glad I could help improve things. 😄


Add Previous Link

Description
Adds support for previous links in search bundles. This is optional behind a header "x-ms-add-previous-link". This works in both Cosmos and SQL services, but due to Cosmos continuation tokens already being rather long I wouldn't recommend using it with a Cosmos service.
Related issues
Addresses [issue #].
Testing
Currently just manual test, will add unit and E2E if team decides to take on the new feature.
FHIR Team Checklist

Update the title of the PR to be succinct and less than 65 characters
Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
[ ] CI is green before merge
Review squash-merge requirements

Semver Change (docs)
Patch

/azp run


swift-api-digester: teach the tool to directly compare two module interfaces

We used to use a Json format to capture the ABI/API detail of Swift modules to walk-around module compatibility issues across different compiler versions. Since Swift module now is in stable format, we can compare two swiftinterface files directly without dumping the content into Json format.
Still Work in progress.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci Please smoke test OS X platform


Passing arguments to snippets

As mentioned in #2042, it would really be great to be able to pass arguments to snippets.
For example, in the following example, passing arguments like path, and port number would allow to simplify a lot the caddyfile, having snippets like the following:
(forwardToVmName) {
    proxy <variablePath> vmName:<variablePort> {
        transparent
        insecure_skip_verify
        ...
    }
}

I would recommend pre-processing, yes, like with a template engine.


Added shouldRotateToLandscape mode

As it currently stands I don't believe this would look good at all. The design/layout was pretty explicitly portrait.

Maybe I gave a wrong description. The problem was that I didn't want rotation on OnboardVC's, so I added this property in order to disable it (since rotation is enable in all masks by default).


Selecting vertical/horizontal/grid containers within a vertical/horizontal/grid container should draw a box when selected

In the above scenario I'd expect a blue box around both space views in the vertical container since it was selected (and a grey one on hover, in accordance with out highlighting colors).

(this screenshot is lacking selection here, this should ofc only show when still selected)
This would help localizing where's what in the viewport.

First step:

https://github.com/rerun-io/rerun/pull/6596


replacing primer scheme submodule with a git subtree

This removes the primer scheme submodule, which (expectedly) caused broken links, and replaces it with a subtree.
The advantages:

existing links will work
don't need to use submodule cloning
we still have the upstream DOI'd repo of primer schemes

The disadvantages:

essentially a copy of the primer scheme
requires manual push/pull to sync with the remote (where primer scheme updates should be published)

To pull primer scheme repo updates into this repo:
git subtree pull --prefix primer_schemes/ https://github.com/artic-network/primer-schemes.git master --squash

This will close https://github.com/artic-network/artic-ncov2019/issues/43


Add extra label to metric so similar hostnames don't collide

This PR adds node_id to every serfLanMembersStatus metric generated, which is a unique identifier and will avoid the collapse of this metric when there's two nodes (different id) with similar node name. It also solves https://github.com/pcarranza/nomad-exporter/issues/5
cc @pcarranza @omame

ohai!


VSTest Coded UI Tests cannot download dependent files

we have an existing MTM coded UI testing process that we are in the process of migrating to VSTS Build/Release.
During test execution the test is designed to communicate with VSTS and download a relevant database backup file for use.  We have had this working, until the last couple of weeks.  Since then we consistently see "TF30063: You are not authorized to access https://********.visualstudio.com/." when using our older connector (which uses the SOAP API).
Updating the connector to use the REST API (via the Microsoft.TeamFoundation.Client package) now produces the following error:

Unit Test Adapter threw exception:
Type is not resolved for member 'Microsoft.VisualStudio.Services.Common.VssUnauthorizedException,Microsoft.VisualStudio.Services.Common, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.

The VSTTAgent (qtagent32_40.exe) is from VS2015 (i.e. version 14).  So I think that explains the type not resolved error.
Is it possible to upgrade the VSTTAgent to VS2017? If so, then how (we never explicitly installed Visual Studio on any of the test VMs, so I'm unsure at what point these files were installed)?
Why would we be seeing errors when attempting to communicate with VSTS?  We are using a PAT to authenticate to VSTS.  This works in a local environment, in our old MTM lab, and has previously worked in this environment. I have verified the PAT has not expired, and is scoped to all scopes.

Hi @acm74, Thanks for reporting. Please attach your run logs for us to understand your scenario better. You can reach out to devops_tools[at]microsoft[dot]com

@prawalagarwal the release logs have not been showing much of anything.  The error messages are appearing in stacktraces for test execution (either in .trx files, or sometimes in standard console output).
This is now becoming a more serious problem for us.  In other internal systems that we rely on, when we change the personal access token, we are seeing the same type of authorization exceptions (TF30063).

@prawalagarwal we are seeing authorization errors in more of our system - this has also been reported separately at: https://developercommunity.visualstudio.com/content/problem/258204/login-using-new-pat-fails.html

Hi @acm74 Firstly, please dont take dependency on Microsoft.TeamFoundation.Client packaged with Visual studio. There is a nuget package specifically made to communicate to VSTS link. Please use this nuget package.
If this doesnt solve your issue please attach a sample source code which is failing to connect to VSTS. You can reach out at devops_tools[at]microsoft[dot]com

@prawalagarwal I have tried to use both the Microsoft.TeamFoundation.Client NuGet package and the ExtendedClient package.  Same problem with either package.

Please attach your logs with system.debug set as true in release variables and a sample code which is failing to communicate. You can send them to devops_tools[at]microsoft[dot]com

@prawalagarwal release logs are 32Mb - this is too big to upload to GitHub.  Where can I send it?

@acm74 We did not get the mail from you. It might have been filtered as spam due to the attachment size.  Is it possible to share both logs and the sample code on a share and provide us the link on<EMAIL_ADDRESS>

@prawalagarwal I will try again later today.  Please note that the related issue with authorization has been fixed, and I was able to get a test run started.  I have had to roll back our code changes to remove the use of the VSTS REST APIs (via the Microsoft.TeamFoundation.Client NuGet package).

Help!!!

@Nekobravo As I mentioned on the other thread, please create a separate issue or reach out to us at<EMAIL_ADDRESS>

@acm74 Please let us know if you are unblocked. Also it will help if you could mention the steps which helped you unblock.

@prawalagarwal we are currently still experiencing issues with unavailability of VSTS.  We are seeing HTTP requests timing out, service unavailable errors, and service unavailable for anonymous authentication errors.

@prawalagarwal we have a work-around for the issue with the test adaptor throwing an exception.  We can get tests running now (when VSTS authentication and authorization is working).
In the longer term we would like to start using the VSTS REST APIs, so any insight or help you can provide would be appreciated.  I am still trying to find out just how the dependencies to VS2015 were installed on our test VMs.

@prawalagarwal I will see what I can do.  Our codebase is not so easy to work with, and unfortunately I don't have the time to try and put together a proof of concept.
In the meantime, I am still seeing issues with VSTS itself.  All my attempts today to configure new interactive test agents are failing with "TF400813: resource not available for anonymous access".

@acm74 ensure that the PAT token you are using is still valid. you will generally see this error when PAT token has expired

@acesiddhu sorry about the delay in replying - I've had to work on other issues in the past couple of days.  The PAT is still valid. I updated the agent version being used (from 2.129.0 to 2.133.3), and this seems to have fixed the problem.

Thanks for updating us. Closing this issue.


dataframer:  include condition in Observation dataframe

Looking to add cancer type / biopsy location to the FHIR labkey metadata. Looking through the labkey data it seems like this information can be gained by joining via patient id the enrollment table to the sequencing table. How would you approach this?

Assumptions:

a Condition MAY exist - creating the condition is the responsibility of the data submitter, the condition may be created in any fashion; by hand, by a g3t_etl transformer or other mechanism.

Approach:

The subject of the observation is called here if the subject is a Patient
Note that the condition is not mapped directly from the observation.  There can be several paths from Observation to Condition via   Observation.focus:

Procedure
Patient->Condition

Effectively the "join" is observation.subject == condition.subject :

Items from the patient should be mapped to the observation similar to how fields from a linked Procedure is mapped
This may be overly simplistic as the patient can have multiple conditions, however this will get the basic observation dataframe populated

See https://github.com/ACED-IDP/gen3_util/pull/80

merged PR #95 and commit 8733923 might be relevant here


feat(protocol-designer): add modal to pause after set temp

overview
Closes #5117
changelog

Add modal
Refactor add/select step selectors for mocking and add tests

review requests
Code

This modal is controlled by the step-form/ButtonRow component state. I think that's the best place to put it b/c there it should be extensible to different steps if we want "auto-add a subsequent step" to be a regular thing. Seems OK?
Tests OK? Thunks calling thunks requires me to mock more than I'm used to

Behavior

[ ] Saving a Temperature step with "set temp" should trigger the modal. Modal should match design in #5117. Temperature number should be correct in the modal.
[ ] Clicking "I will build a pause later" should save the current form and do nothing else. The Temperature step should still be selected
[ ] Clicking "Pause protocol now" should create a new Pause step which will be selected in the sidebar but the form should be closed (as if you just clicked "save" on that form). The Pause step should "pause until temperature reached" with the same temperature
[ ] Making a Temperature step with "deactivate" should not trigger the new modal
[ ] Try to break it by deleting the module at different points. It should be robust and not whitescreen anywhere

@jen-fong

For the action creator tests using redux-thunk, you could use the mock store so you don't have to mock out the thunks. There is an example in the [https://redux.js.org/recipes/writing-tests/#async-action-creators](redux docs).

This could be a good approach. I tried it out just now with the addAndSelectStepWithHints test and it didn't work out there. Since addAndSelectStepWithHints dispatches addStep which in turn dispatches selectStep (which uses a handful of selectors), we would need to mock uuid for addStep, and also mock the new selectors that selectStep uses. It would also end up re-testing things that are already tested eg addStep.test.js should cover addStep so we don't need the same tests for addAndSelectStepWithHints -- I think??
I'm now going to try using the mock store like in the example to add tests for selectStep. So then we should have full coverage for this cascade

addAndSelectStepWithHints should just be dispatching addStep which is mocked
addStep should dispatch ADD_STEP with a uuid and also dispatch selectStep, both are mocked
selectStep should dispatch SELECT_STEP then compute the step contents and dispatch them in a POPULATE_STEP


Error Occurs When Sharing an Article from the Bundle App

Hello, when I try to share articles in the "Bundle" app, I encounter the error "FormatException: Illegal scheme character (at character 10)." Can you help about it? Thank you.

Hi @Medullitus,
What do you mean by "bundle app"? Is it an actual android application? I can't find it anywhere...
Does it occur for every url or for specific domains?

Bundle is a news RSS application. The Android Play Store link is below:
https://play.google.com/store/apps/details?id=com.dwarfplanet.bundle&hl=tr&gl=US
Unfortunately, the same error is occurring for every content. If you test it yourself, you can see it more clearly. Thank you very much for your response.

Thank you for the clarification, I  just installed it and I can reproduce the issue.
Unfortunately, the share button doesn't send the url link of the article webpage, it just send the title as text, so it is impossible for markdownr to download the page and convert it.
You have to tap the three dots in the upper right corner, tap the copy link and then paste it in markdownr.
I can't do anything to change this behavior, reach out the bundle team and ask them to fix it.

I thank you for the quick responses and your attention as a user. I use the application for Joplin. But I'm glad to have known a dedicated developer like you. I would have loved to see you on the Joplin team. I hope you continue to design beautiful and functional applications!


Add evaluateOnNewDocument method to API

I need some data to be synchronously available in app from its start. E.g., I would like to fetch settings from my own storage and inject them to the app by evaluating on new document something like:
evaluateOnNewDocument(`Settings = { a: 1, b: 2, c: 3}`);

I can create file with settings somewhere in temporary directory and then serve it but it looks much less convenient.

You should be able to serve this via network interception now.


feat(consumer): support multiple balance strategies

As shown below, Kafka supports specifying multiple rebalance protocol from the very begining. (source)
v0 supported in <IP_ADDRESS> and greater
JoinGroupRequest => GroupId SessionTimeout MemberId ProtocolType GroupProtocols
  GroupId => string
  SessionTimeout => int32
  MemberId => string
  ProtocolType => string
  GroupProtocols => [ProtocolName ProtocolMetadata]
    ProtocolName => string
    ProtocolMetadata => bytes

v1 supported in <IP_ADDRESS> and greater
JoinGroupRequest => GroupId SessionTimeout MemberId ProtocolType GroupProtocols
  GroupId => string
  SessionTimeout => int32
  RebalanceTimeout => int32
  MemberId => string
  ProtocolType => string
  GroupProtocols => [ProtocolName ProtocolMetadata]
    ProtocolName => string
    ProtocolMetadata => bytes

However, Sarama only supports one group protocol right now, by specifying Consumer.Group.Rebalance.Strategy. This prevents users from enjoying some benefits in Kafka. For example, if users wanna change the default one range to a more advanced one like sticky, when the new consumers with sticky protocol try to join in the consumer group, they will be rejected by the coordinator with error ErrInconsistentGroupProtocol(kafka server: The provider group protocol type is incompatible with the other members) and thus cannot consume any messages. Only when all the instances are upgraded with sticky protocol can they resume consume message again.
In contrast, the client supporting multiple group protocols enables rolling upgrades without downtime -- The upgraded member includes both the new protocol and the protocol, and the coordinator will choose a single protocol which all members support. Once all members have upgraded, the coordinator will choose whichever protocol is listed first in the GroupProtocols array.
In this PR, I introduce a new config Consumer.Group.Rebalance.GroupStrategies and mark the old one Consumer.Group.Rebalance.Strategy deprecated. To make it backward compatible:

The dafault value for Consumer.Group.Rebalance.GroupStrategies is []{BalanceStrategyRange}
Consumer.Group.Rebalance.Strategy takes precedence of Consumer.Group.Rebalance.GroupStrategies

Thus,

if Consumer.Group.Rebalance.Strategy isn't explicitly configured previously, then the new default value range in Consumer.Group.Rebalance.GroupStrategies will be used.
if Consumer.Group.Rebalance.Strategy is configured by users previously, Consumer.Group.Rebalance.Strategy is also chosen as the group protocol.

I believe it's a desirable feature in sarama, since other Go Kafka Libraries, like segmentio/kafka-go, already support it.

@Jacob-bzx  thanks for your great contribution!


Allow keyboard to be dismissed by swiping it down

I felt like it would be useful to be able to hide the keyboard while editing a file. This PR includes literally just this change:

Awesome, thank you!


User Feedback Widget Part 1

Description
Milestone 2 of #18548 .
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:

Changes

[x] Added user feedback widget to MXNet main website and Python tutorials&API pages.

Comments

Preview

Main website: http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/api/scala/docs/tutorials/io
Python microsite: http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/api/python/docs/tutorials/

@mxnet-label-bot add [Website]

@mxnet-bot run ci [centos-cpu, unix-cpu]

@aaronmarkham can you please help review?

@ys2843 can you please send an email to the mailing list requesting guidance on how to disclose the use of google analytics? @aaronmarkham made the point that the website is not compliant with GDPR.
We should ask ASF to consider including whatever guidance they give, into their policies.

Will do. I will share the info with team later.


MWPW-138696 | Icon upload

Icon upload
Resolves: MWPW-138696
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/
After: https://MWPW-138696--express--adobecom.hlx.page/express/

Codecov Report

Merging #408 (4555312) into stage (9b1a542) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            stage     #408      +/-   ##
==========================================
- Coverage   31.23%   31.21%   -0.02%
==========================================
  Files         124      124
  Lines       25126    25126
==========================================
- Hits         7847     7844       -3
- Misses      17279    17282       +3

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add Projection support for Kafka

A typical use-case in the presence of Kafka is to have one topic for Aggregate type, and many partitions as the id of each Aggregate instance.
The events to be published in these partitions are the latest values of the Aggregate state.
This can be implemented as a general Projection that can be easily imported in the code.
Kafka library to use: https://github.com/fede1024/rust-rdkafka

Closing this issue for now due to inactivity and change of focus to other things :v:


API docs refer to Capybara::Element not Capybara::Node::Element

Very minor thing but the API docs refer to Capybara::Element which does not exist e.g. https://github.com/jnicklas/capybara/blob/master/lib/capybara/node/finders.rb#L25 - I presume they should refer to Capybara::Node::Element?

Technically in the location referenced it could be Capybara::Node::Element or Capybara::Node::Simple I believe, but yes it is incorrect -  PR would be welcome


Writing Schedule data to Maccor?

Hello,
Do you have any plans to add routines for writing schedule data to a Maccor Tester?
If you have any examples of the data format, I would be happy to help collaborate.
Thanks!

Hi there,
When you refer to schedule, do you mean to write in a test procedure over MacNet?
Honestly that wasn't a need when I set this up but I'd be happy to work with you to get that going.
Charith


Intelisense not available in Java projects.. Taking forever "Starting Java Language Server"

1.9.0
Windows 10

Steps to Reproduce:
I'm using the extension "Language Support for Java by RedHat".

Open a folder housing several reasonably large java projects.
c:\git-repos
\repo1
\repo2
\repo3
....
Not all repositories are java but the the majority are.
Open a java source file from one of the projects (c:\git-repos\repo1\src\main\java\com...\SomeJavaFile.Java) and look at the intelisense for anything.

"Loading..." will appear in the tooltip and the status bar you will see "Starting Java Language Server..." I've waited over 30 minutes and still see this with no intelisense available. The projects are not insanely huge. This has worked on a tiny test project outside the folder containing lots of java project repositories,

Reading your description it seems that the issue originates from an extension and not VS Code itself. Thereby I will close this issue and kindly ask you to report this issue against the extension repository.
Most extensions are on GitHub as well so it should be easy to file it against the extension. If you open an extension on the market place, click on the Support link to find out more.

Thanks again for your feedback!


listSecrets and generic list* functions not recognized

https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-group-template-functions-resource#list

Adding listSecrets.  Others we'll add as people try to use them.

Is there telemetry to record the usage of the other commands?

#84


Get-CMObject

I would like to be able to confirm the object moved, can we get a 'Get-CMObject' command, or something that reports the Folder location?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 015c02b6-5b69-fb85-7cb8-0df13542d71c
Version Independent ID: fe4333a1-cf0c-c0c0-4960-eed571a06fd3
Content: Move-CMObject - Configuration Manager
Content Source: sccm-ps/ConfigurationManager/Move-CMObject.md
Product: configuration-manager
Technology: configmgr-other
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Yes it will make the search easier !!! from the current tables and views how do we get the ObjectID?

I get the InputObject this way:
Move-CMObject -FolderPath "AAA:\Application\Tools" -InputObject (Get-CMApplication -Name "ApplicationName")

Thanks  for taking the time to provide feedback on Configuration Manager! This is an interesting feature idea, and I want to make sure it gets directly into the hands of the engineering team. Fortunately, we've got dedicated channels for that very purpose. :) See the Product feedback section in our docs. I think it’s important that you submit it directly, because that allows you to receive notifications and more closely monitor the progress. I'm closing this issue as there's nothing actionable for docs at this time.


Cannot run default implementation of GAIL

Hi all,
Thanks so much for putting this together, I'm very excited about it!
I am trying to run the default implementation of GAIL, but it is throwing an error about the input policy being the wrong type as shown below. Note that I already downloaded the default expert data for the deterministic Hopper expert, and that it is logging out the return stats on the data.
AssertionError: Error: the input policy for the TRPO model must be an instance of common.policies.ActorCriticPolicy.```

How to reproduce:
In command line: `python -m stable_baselines.gail.run_mujoco`

Please let me know if I'm missing something, thanks again!

Hi,
Thanks for reporting the bug. @hill-a, it seems that we forgot to refactor GAIL policies.
It's hard for us to debug GAIL for now, as we don't have a mujoco licence...
Feel free to submit a PR if you solve the issue ;)
To guide you a bit:
take a look at common.policies.py, you will need to tweak gail.mlp_policy.py a bit so it follows the structure of the base class.
Don't hesitate to ask if you have any additional questions ;)

Hi! I'm currently taking a look into this issue, will let you know if I can figure out the cause! I think it might actually be related the trpo_mpi.setup_model method, (as the saved models are loaded from a .npz file), and I reckon that these saved models are not consistent with the current class definition

Hi @krishpop ,
Thank you for working on that!

Is the general design pattern for these scripts is not using the policy_fn?

I'm not sure to understand what you mean...

It seems in much of the original baselines code, and in some of the code in stable-baselines, there is a usage of a lambda argument called policy_fn that generates a policy of a particular type when called. However, it seems the usage of policy_fn is being phased out in stable-baselines, after looking into the trpo_mpi implementation, as well as the BaseRLModel, it seems now that there is a policy registry used to organize the different policies.

Ok, now I understand what you mean ;)
So, yes this behavior was removed in favor of passing either string (if the policy is registered) or class.

Yes, thanks for clarifying. I will try fixing that, test, and send a PR soon

@flaurida @krishpop I started working on updating GAIL (branch gail-test. I'm also adapting it to make it work with other environments (not only mujoco ones) and check at the same time the performance.


Winter Solstice 2024 Upgrade (Ruth epoch)

There are no changes that will activate with this upgrade.

All required parameters for Ruth are correct. Merging.


Add timers at the end of some of the import progress bars to indicate duration of each step

These two phases are not showing run times/duration: Post-import Processing and PerfStatsAnalysis

Make them consistent with the other steps

After the change


Track mission progress

As a user I want to be able to track the mission progress so that I can estimate the time to complete
Acceptance Criteria:
Scenario: The user has details about the mission progress shown on the frontend application and is able to evaluate the results in real time.
Given that the user has undertaken a mission
When the user starts the mission
Then the Rover will report back mission status updates for mission completion continuously in real time.

More relevant to Mission control section. Re-assigning to Milestone 4.

Deferring to Sprint 4 pending mission drive feature completion.


res.redirect is not a function

I have a sandbox app with linkedin however, I keep getting the issue res.redirect is not a function not sure where the error lies.
import { Context } from 'vm';
const Linkedin = require('node-linkedin')('client_id', 'client_secret', 'callback_url');
const linkedin = Linkedin.init('client_id');

export const auth = (ctx: Context): any => {
  const scope = 'r_liteprofile';
  const info = Linkedin.auth.authorize(scope);
  console.log('res', info);
};

I got the same error.
I looked into the code which caused the error, which is in node_modules/node-linkedin/lib/auth.js
then at line 32, I see the below code.
if (res && res.constructor === Array) {
    redirectURI = state;
    state = scope;
    scope = res;
    res = null;
}

and I thought this code is trying to make the first parameter set to scope when the first value corresponds to the type of scope(i.e. string). (by default the 'authorize' method takes the first parameter as 'res')
So I feel weird that the part res.constructor === Array should be res.constructor === String not Array then I changed the part.
Also, there was a weird code on line 55,
if (scope && scope.length > 0) {
   url += '&scope=' + scope.join('%20');
}

the variable 'scope' should be string, not array. so .join('%20') is weird as well. so I commented out the part.
if (scope && scope.length > 0) {
   url += '&scope=' + scope/*.join('%20')*/;
}

then the error is gone.
I am new to Linkedin API and I don't know much about this library.
I am not sure if this is what I'm supposed to do for the error but I hope it helped.
Sorry for my bad English if it is hard to read.


chore(deps): update prettier to ^2.3.0

"needs" https://github.com/synapse-wireless-labs/jasmine-marbles/pull/69 to be able to use prettier^2.3.0 due tu node version
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=10.13.0". Got "10.12.0"

Will you rebase this branch on master to CI is happy?


KYLIN-4297 Build cube throw NPE when partition column is not set in J…

…DBC Data Source

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@31cb208). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #993   +/-   ##
=========================================
  Coverage          ?   25.44%
  Complexity        ?     6164
=========================================
  Files             ?     1416
  Lines             ?    85899
  Branches          ?    12052
=========================================
  Hits              ?    21860
  Misses            ?    61908
  Partials          ?     2131

Impacted Files
Coverage Δ
Complexity Δ

...rg/apache/kylin/source/jdbc/JdbcHiveInputBase.java
21.11% <0%> (ø)
8 <0> (?)

Continue to review full report at Codecov.

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


create a show instance for a move and a coordinate

I will take a look.

Hello!
This is going to sound weird, but I'm trying to start making contributions in open source and I resolved this issue to understand Github a bit better (as a test).


Project is broken under Linux UE 5.0.2

I get this error
ERROR: Expecting to find a type to be declared in a module rules named 'OnnxRuntime' in UE5Rules, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null.  This type must derive from the 'ModuleRules' type defined by Unreal Build Tool.

Missed the fact that you need to be on the ue5.0 branch


A trailing comma is not permitted after the rest element

When I import the module appers this error:
/app/node_modules/react-native-scrollable-tab-view/SceneComponent.js: A trailing comma is not permitted after the rest element (9:32)

   7 |
   8 | const SceneComponent = (Props) => {
>  9 |   const {shouldUpdated, ...props, } = Props;
     |                                 ^
  10 |   return <View {...props}>
  11 |       <StaticContainer shouldUpdate={shouldUpdated}>
  12 |         {props.children}

any update on this?

@UzairAhmedSiddiqui  you can just have a work around on this by removing , in
const {shouldUpdated, ...props, } = Props;
in the node_modules/react-native-scrollable-tab-view/SceneComponent.js

Honestly, that is not a workaround. node modules are never committed. This will only work if you are working on a local environment. Please fix this

When I import the module appers this error:
/app/node_modules/react-native-scrollable-tab-view/SceneComponent.js: A trailing comma is not permitted after the rest element (9:32)

   7 |
   8 | const SceneComponent = (Props) => {
>  9 |   const {shouldUpdated, ...props, } = Props;
     |                                 ^
  10 |   return <View {...props}>
  11 |       <StaticContainer shouldUpdate={shouldUpdated}>
  12 |         {props.children}

Also experiencing similar issue as well. Any resolution yet on this?


Sweet Module example is not working

http://www.sweetjs.org/doc/1.0/tutorial.html#_sweet_modules
I tried the example given at the above link and it is giving me below error
/usr/lib/node_modules/@sweet-js/cli/node_modules/resolve/lib/sync.js:42
throw err;
^
Error: Cannot find module './es2015-macros'
Updated the path in import statement to 'import { class } from './es2015-macros.js'; ' i.e. added .js in the module name. Re-run the example but getting below error this time
/usr/lib/node_modules/@sweet-js/cli/node_modules/sweet-core/dist/syntax.js:360
throw new Error('Scopeset ' + debugBase + ' has ambiguous subsets ' + debugAmbigousScopesets);
^
Error: Scopeset {outsideEdge_5} has ambiguous subsets {outsideEdge_5}, {outsideEdge_5}
at Syntax.resolve (/usr/lib/node_modules/@sweet-js/cli/node_modules/sweet-core/dist/syntax.js:360:17)

I worked a bit with the example above. Found out that the macro works if used in the same file, and the error gets thrown when the macro gets imported. Maybe this info's gonna help a bit.

Same here....

I can't reproduce this, but I also can't get the class macro to expand. @manisuec can we get a look at your code please?

I can repro it. The extension problem must have been recently introduced. I'll fix and add regression tests for that.
The ambiguous scopesets error is surprising because I thought we had tests for that. I'll investigate.

@disnet any idea why this cropped up after the project was split?

I think the extension issue was introduced in https://github.com/sweet-js/sweet-core/commit/1087af322cb7589953e69dc40e2b636a059a89f5#diff-828804bf7ffcce021806a0fc90ea20f9
Defaulting to the empty array was the problem.
The scopeset issue has been there since introducing recursive modules I think. I'll write up what happened when I get a fix ready (should be soon).


This should fix MySQL 5.7 issues caused by ONLY_FULL_GROUP_BY

First of all, I have to say that this is my first pull request ever created
This should fix MySQL 5.7 issues caused by ONLY_FULL_GROUP_BY
The proposed code works for me, maybe others could share opinions on a fix for this, it is very annoying when working with new versions of MySQL

#5622

CI currently fails because of tests expect string equality. PR will probably need to contain the updated tests in order to get merged.

I am aware.
However, I lack the time to dig into it and change tests accordingly. I also have no clue if and if so, how this would affect other DBMS, so I'll simply leave my code here and hope it would help you guys get a decent solution out of it :)

@wucdbm Any chance you get to implement the comments any time soon? It would be great to see this bug fixed.

I ran into the same problem. The problem is still not fixed? @Ocramius

@Shandur doesn't look like it. Maybe you can finish that PR? it doesn't look like much is missing from it.

@theofidry I'll try to do it in few days.

hello guys, any progress with this?

@lukasneubauer Hey. May was really busy for me, I didn't have any time to work with it. Gonna try to finish it the next week, I'll have some free time. I'll keep you posted.

Hello,
We are all waiting for your PR in our company and I guess we are not the only ones :)

@bourdeau Hey. In order not to stop your developing, you can use this crap as temporary solution before executing a query.
private function disableMysqlMode()     {         if ($this->_em->getConnection()->getDriver()->getName() === 'pdo_mysql') {             $sql = " SET sql_mode=(SELECT REPLACE(@@sql_mode, 'ONLY_FULL_GROUP_BY', '')) ";             $this->_em->getConnection()->prepare($sql)->execute();         }     }

@bourdeau then maybe on waiting on it someone could finish this up ;)

@Ocramius I think we can close this one and leave it to merge #6143 (seems more complete), agreed?

@lcobucci yeap.

Handled in https://github.com/doctrine/doctrine2/pull/6143


Add approvers & maintainers corner to CONTRIBUTING.md

While chatting with @theletterf, we figured out that it would be good to write down some practices we apply while doing code reviews and PR management. I call them practices because none of this is a must do and there are always exceptions.
@open-telemetry/docs-approvers, all: please take a look!

Please take another look
I will create a follow up PR for open questions:

https://github.com/open-telemetry/opentelemetry.io/pull/3632#discussion_r1412343877
https://github.com/open-telemetry/opentelemetry.io/pull/3632#discussion_r1412344685

/fix:format


Repartition Optimisation Pass Breaks Sorting (wrong answer with limit)

Describe the bug
The Repartition optimisation pass fails to account for order sensitive operations, which causes it to repartition a sorted partition. As there isn't (yet) an order preserving merge operator (#362) this results in an incorrect final physical plan.
To Reproduce
For example,
LogicalPlanBuilder::scan_csv(&path, options, None)?
            .sort(vec![col("c4").sort(true, true)])?
            .filter(col("c2").eq(lit(1)))?
            .limit(10)?
            .build()?;

Results in
GlobalLimitExec {
    input: MergeExec {
        input: CoalesceBatchesExec {
            input: FilterExec {
                predicate: BinaryExpr {
                    left: Column {
                        name: "c2",
                    },
                    op: Eq,
                    right: TryCastExpr {
                        expr: Literal {
                            value: Int32(1),
                        },
                        cast_type: Int64,
                    },
                },
                input: RepartitionExec {
                    input: SortExec {
                        input: CsvExec {
                            source: PartitionedFiles {
                                path: "/home/raphael/repos/external/arrow-datafusion/testing/data/csv/aggregate_test_100.csv",
                                filenames: [
                                    "/home/raphael/repos/external/arrow-datafusion/testing/data/csv/aggregate_test_100.csv",
                                ],
                            },
                            ...
                        },
                        expr: [
                            PhysicalSortExpr {
                                expr: Column {
                                    name: "c4",
                                },
                                options: SortOptions {
                                    descending: false,
                                    nulls_first: true,
                                },
                            },
                        ],
                        ...
                    },
                    ...
                },
            },
            target_batch_size: 4096,
        },
    },
    limit: 10,
}

This is incorrect, as the MergeExec does not preserve ordering and therefore the limit will return an arbitrary set of rows
Expected behavior
I would expect the Repartition pass to not introduce partitioning on paths with operators that are order sensitive. Potentially once there is an order preserving merge, this restriction could be relaxed provided the inserted merge is order preserving, but until then this optimisation is incorrect.
I will shortly be creating a ticket with some proposals on how we could make DataFusion sort order aware
Additional context
This issue was spawned out of investigation on https://github.com/apache/arrow-datafusion/pull/378#issuecomment-847158217

This is a good find 🕵  👍  @tustvold

@tustvold thanks for the detailed write up!

I just hit a variation of this in IOx -- see https://github.com/influxdata/influxdb_iox/pull/3633#issuecomment-1030126757
Conveniently @tustvold added some infrastructure to support this in https://github.com/apache/arrow-datafusion/pull/1732


Generator changed type of subscriptionId from string to Guid

The latest generator updates have changed the type of subscriptionId from string to Guid.

.Net uses strings to represent Guid types so I believe this is a bug.
This problem can be resolved with autorest transforms but we should update the generator to output the correct type.

Generator never changes types from nowhere. Please double confirm if something has changed in your swagger.
If possible, please share a PR link of your SDK changes so that we could take a look to see where the change comes from.


[Documentation] is incorrect on how variable assignment works

Description
https://dbeaver.com/docs/dbeaver/SQL-Execution/#variables-binding states that a variable can be assigned encapsulated in quotes and with a semi-colon ending a line. This is incorrect. The quotes and the semi-colon become part of the value.

The correct assignment is: @set actor_name=Mark

DBeaver Version
23
Operating System
macos 13.6

Hello @JTMosaic, thank you for your comment, we fixed the documentation.


Bug: Unit Capacity section not loading (Test Automation)

Issue:
When a user tries wishes to review (Global-view) or Add/Edit (Workspace) data for Unit Capacity, the Unit Capacity section doesn't load. The spinner keeps going

Bug resolved. Cann view unit capacites in global and the workspace