Mixin-with-prefix capability?

Discussed in https://github.com/remkop/picocli/discussions/1808

Originally posted by rcauble September 11, 2022
Hi,
I have a use case where I have 2 commands with each their own set of settings:
class Command1Settings {
    @CommandLine.Option(names = "--foo")
    private String foo;
    @CommandLine.Option(names = "--bar")
    private String bar;
}

@CommandLine.Command(name = "command1")
public static final class Command1  {
    @CommandLine.Mixin
    private Command1Settings settings = new Command1Settings();

    public void run() {
        String results1 = Command1Runner.run(settings);
        System.out.println(results1);
    }
}

class Command2Settings {
    @CommandLine.Option(names = "--foo")
    private String foo;
    @CommandLine.Option(names = "--bar")
    private String bar;
}

@CommandLine.Command(name = "command2")
 public static final class Command2  {
    @CommandLine.Mixin
    private Command2Settings settings = new Command2Settings();
    public void run() {
        String results2 = Command2Runner.run(settings);
         System.out.println(results2);
    }
}

And I am trying to create a 3rd command that computes a diff of the other 2. As you can see above, though there is overlap in the options of the 2 commands and so I'm looking for a MixinWithPrefix capability so that I can do something like this:
@CommandLine.Command(name = "diff")
public static final class Diff  {
    @CommandLine.MixinWithPrefix("first-")
    private Command1Settings settings1 = new Command1Settings();
    @CommandLine.MixinWithPrefix("second-")
    private Command1Settings settings2 = new Command2Settings();
    public void run() {
        String results1 = Command1Runner.run(settings1);
        String results2 = Command2Runner.run(settings2);
         System.out.println(diff(results1, results2));
    }
}

And then this will define a new command where the user can do:
diff --first-foo thing1 --first-bar thing2 --second-foo thing3 --second-bar thing4

To add an additional constraint for context, Command1 and Command2 live in different libraries from the differ and so I can't simply arrange for the names to be distinct as they are not necessarily owned by me.
Any suggestions?

@alexturc I see the PR for this feature (#2311) but to be honest I don't like the idea...
Note that there is already a generic mechanism to transform the model:
https://picocli.info/#_model_transformations
If that mechanism has limitations so that it cannot be used for this use case I would rather improve the existing generic mechanism rather than introduce another more narrow new mechanism.

Hi @remkop,
Mixins are a great way of grouping related options such that complex commands can be built while still being able to manage complexity. But a feature which I'm facing quite often and became a burden is inability of reuse mixins in the same command, as option names would clash.
A typical example is for database connections (but I encounter multiple such situations), where a mixin would be
public class DbMixin {
  @Option(name = "--url")
  String url;
  @Option(name = "--user")
  String user;
  @Option(name = "--pass")
  String pass;
}

An application working with multiple DB connections would reuse the above class like this:
@Command
public class DbMixin {
  @Mixin
  DbMixin firstDbMixin = new DbMixin();
  @Mixin
  DbMixin secondDbMixin = new DbMixin();
  @Mixin
  DbMixin thirdDbMixin = new DbMixin();
}

As of now this would not work as option names coming from different DbMixin instances would class an building the command line would fail. And it would fail before having the chance to apply model transformations as you suggested - because the are applied last, after an initial version of the model was build.
The goal here would be to add a prefix to option names originating from the three mixins such that we can successfully build a command line while reusing DbMxin class. --url option from firstDbMixin could become --firstDb.url or --firstDbUrl etc.
The solution int he PR keeps the mapping logic at mixin declaration, so I think it has good developer ergonomics - one can see immediately how option names are transformed.
But I can work with IModelTransformer as well if you would be ok to have @Mixin annotation take a model transformer and apply it immediately after the mixin is built.
In practice I intend the have a transformer which would take the name of the mixin and and use it as a prefix for option names.
Would this be acceptable?
If yes, would you prefer for @Mixin to take an array of transformers or just one, like @Command does?

I sent another PR which uses IModelTransformer with mixins. This alows me to make more changes actually, including prefixing the environment variables from where the default values would be taken by default. Please let me know if this acceptable.

Hi! Thanks for the PR!I’m currently traveling with limited access to internet. I’ll take a look when I get back. On Aug 17, 2024, at 1:08, alexturc @.***> wrote:
I sent another PR which uses IModelTransformer with mixins. This alows me to make more changes actually, including prefixing the environment variables from where the default values would be taken by default. Please let me know if this acceptable.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


Change absolute docs links to learn links

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/index.yml
94
1
link
:white_check_mark:

msteams-platform/resources/teams-contributor-reference.md
93
4
link
:white_check_mark:

msteams-platform/task-modules-and-cards/cards/cards-actions.md
91
10
link
:white_check_mark:

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
94
4
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 602d3ca:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/feed.atom
:white_check_mark:Succeeded
View

msteams-platform/index.yml
:white_check_mark:Succeeded
View

msteams-platform/resources/teams-contributor-reference.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/cards/cards-actions.md
:white_check_mark:Succeeded
View

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
:white_check_mark:Succeeded
View

README.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/index.yml
94
1
link
:white_check_mark:

msteams-platform/resources/teams-contributor-reference.md
93
4
link
:white_check_mark:

msteams-platform/sbs-gs-commandbot.yml
90
27
link
:white_check_mark:

msteams-platform/sbs-gs-msgext.yml
91
45
link
:white_check_mark:

msteams-platform/sbs-meetings-stage-view.yml
85
10
link
:white_check_mark:

msteams-platform/sbs-tab-with-adaptive-cards.yml
89
22
link
:white_check_mark:

msteams-platform/sbs-tabs-and-messaging-extensions-with-sso.yml
83
36
link
:white_check_mark:

msteams-platform/task-modules-and-cards/cards/cards-actions.md
91
10
link
:white_check_mark:

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
94
4
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 2562a16:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/feed.atom
:white_check_mark:Succeeded
View

msteams-platform/index.yml
:white_check_mark:Succeeded
View

msteams-platform/resources/teams-contributor-reference.md
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-commandbot.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-msgext.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-meetings-stage-view.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-tabs-and-messaging-extensions-with-sso.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-tab-with-adaptive-cards.yml
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/cards/cards-actions.md
:white_check_mark:Succeeded
View

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
:white_check_mark:Succeeded
View

README.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hi @Wajeed-msft and @surbhigupta12 , need your review for this PR. These changes should go live (immediately) after rebranding when learn links are active.
CC @v-habibaumme as FYI.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/index.yml
94
1
link
:white_check_mark:

msteams-platform/resources/teams-contributor-reference.md
93
4
link
:white_check_mark:

msteams-platform/sbs-gs-commandbot.yml
90
27
link
:white_check_mark:

msteams-platform/sbs-gs-msgext.yml
91
45
link
:white_check_mark:

msteams-platform/sbs-meetings-stage-view.yml
85
10
link
:white_check_mark:

msteams-platform/sbs-tab-with-adaptive-cards.yml
89
22
link
:white_check_mark:

msteams-platform/sbs-tabs-and-messaging-extensions-with-sso.yml
83
36
link
:white_check_mark:

msteams-platform/task-modules-and-cards/cards/cards-actions.md
91
10
link
:white_check_mark:

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
94
4
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

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

File
Status
Preview URL
Details

msteams-platform/feed.atom
:white_check_mark:Succeeded
View

msteams-platform/index.yml
:white_check_mark:Succeeded
View

msteams-platform/resources/teams-contributor-reference.md
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-commandbot.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-msgext.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-meetings-stage-view.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-tabs-and-messaging-extensions-with-sso.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-tab-with-adaptive-cards.yml
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/cards/cards-actions.md
:white_check_mark:Succeeded
View

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
:white_check_mark:Succeeded
View

README.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Victorian station with no infomation

Not sure if this is the correct method to report issues, but I have been experimenting and in my list of victoria stations I am getting a station with the following information.
Station ID:466
Station name: ENOC MSAP
It is not listed on opennem.org.au and it has no fuel type information or energy output

Thanks for reporting this - i'll take a look at it

VENUS1 is the DUID and it's a registered load - so it doesn't have a fueltech


Is there any plan to add RLTF to the current state-of-the-art code model? Like starcoder or other LLM?

Is there any plan to add RLTF to the current state-of-the-art code model? Like starcoder or other LLM?

Sorry, we have no plans to do these currently. You need to implement it by yourself. Take starcoder as an example, you need to rewrite the forward function of the GPTBigCodeForCausalLM class to adapt to the loss required for RL training. You can refer to our modified CodeGenForCausalLMclass and the original CodeGenForCausalLMclass. When you have finished these, you need to replace the model in train_codegen.py and train_codegen_online_v1.py to perform RLTF training.

Do I need to change the data format? The chat version of starcoder is in instruction/output format and your thing QUESTION/ANSWER format

I think both are fine. Just use same format for training and generating.

Thanks for your reply!


Webpack configuration and first layout implementation for the documen…

Changes Unknown when pulling 78077adc0a409af9bc7cd6b1b434f0f6787f6759 on documentation into ** on master**.


Make ranger window full screen

100% width and height.

Implemented in 1a3dffc, but frame size set to 95% of window instead. Closing.


Covered BenchmarkPlot with tests

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #139 into develop will increase coverage by 3.27%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #139      +/-   ##
===========================================
+ Coverage    84.29%   87.57%   +3.27%
===========================================
  Files           90       90
  Lines         2363     2359       -4
  Branches       231      229       -2
===========================================
+ Hits          1992     2066      +74
+ Misses         327      246      -81
- Partials        44       47       +3

Impacted Files
Coverage Δ

...t/benchmarking/loop_benchmarking/benchmark_plot.py
82.35% <100%> (+82.35%)
:arrow_up:

...benchmarking/loop_benchmarking/benchmark_result.py
100% <0%> (+100%)
:arrow_up:

Continue to review full report at Codecov.

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


Missing texture for MeshEditor light1 and light2 buttons

Godot version
4.0.dev (9142904c24)
System information
Arch Linux
Issue description
Two missing texture symbols over yonder:

Steps to reproduce

Create a new project.
New scene: select 3D node
Add child node MeshInstance
Select Mesh 'New Sphere Mesh'
Open the Mesh editor
See missing textures

Minimal reproduction project
test2.zip

I think the file relevant for this issue is this one. And some additional info from the command line:
tempo:godot$ ./bin/godot.linuxbsd.tools.x86_64
Godot Engine v4.0.alpha.custom_build.9142904c2 - https://godotengine.org
MESA-INTEL: warning: Performance support disabled, consider sysctl dev.i915.perf_stream_paranoid=0
Vulkan API 1.2.0 - Using Vulkan Device #0: Intel - Intel(R) UHD Graphics 620 (KBL GT2)

Editing project: /home/tempo/Other Projects/Godot 4 Projects/test2
Godot Engine v4.0.alpha.custom_build.9142904c2 - https://godotengine.org
MESA-INTEL: warning: Performance support disabled, consider sysctl dev.i915.perf_stream_paranoid=0
Vulkan API 1.2.0 - Using Vulkan Device #0: Intel - Intel(R) UHD Graphics 620 (KBL GT2)
tempo:godot$
WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
     at: _editor_init (modules/gltf/register_types.cpp:75)
WARNING: FBX file import is enabled in the project settings, but no FBX2glTF path is configured in the editor settings. FBX files will not be imported.
     at: _editor_init (modules/gltf/register_types.cpp:99)

Duplicate of https://github.com/godotengine/godot/issues/65090 (same cause).


make sure all apt-get packages are up to date

Among other things, this should make sure that rigs get updated to protect against the Krack attack on WPA2 (wifi encryption).

Ready to merge this if no objections.


CSV import gets TypeError (scope.callback is not a function)

Steps to reproduce:
import a csv file (e.g. agenda, motion or user)
JS console shows error:
angular.js:13920 TypeError: scope.callback is not a function
at Array. (angular-csv-import.js:68)
at Scope.$digest (angular.js:17575)
at Scope.$apply (angular.js:17790)
at FileReader.reader.onload (angular-csv-import.js

Looks like a problem of angular-csv-import...

Note: import works successfully. But we should try to fix this error.

@emanuelschuetze Closed.


Unable to set path with chinese characters

Unable to set path with chinese characters,warning "Invalid folder format. Please use '{title}' or 'Folder Name/{title}' as the folder path."

I changed the folderPattern from:
const folderPattern = /^([\w\s().]+\/)*\{title\}$/;

to
const folderPattern = /^(([\p{L}\p{Script=Hani}\p{Emoji}\s()\-_!@#$%^&()+={}[\];',~][\p{L}\p{Script=Hani}\p{Emoji}\s().\-_!@#$%^&()+={}[\];',~]*\/)*\{title\})$/u;

I'm waiting for Chrome to publish the fixed version of the extension, then you'll be able to add Chinese characters.


Update examples

Fixes #162

Update all examples to use 0.59.1. Make sure they are updated to the latest configuration.
Add make tasks to update examples versions and run all examples to test the collector can at least start.

I can't approve this, but lgtm.


Amanguah32 patch 1

Thanks, @Amanguah32! Your code works with the Counter, but it isn't wrapped in a function, so you didn't quite get to the whole solution.
Also, I moved your code into a notebook file. The file you included in the PR had the .ipynb extension but was coded like a text file rather than a notebook. So I could still open it, but not run it in CoLab. In future weeks, if you can insert the code into a copy of the exercise notebook and submit the whole notebook that would be preferred. It just makes it easier to review because I can run it all in one place. Thanks!


feat: Support non crates.io registries

General
This was to start the conversation around working with publishing crates to an alternate registries. This is useful for teams that make use of semantic-versioning but publish to a private, or non-crates.io registry.
Other Changes

Added .devcontainers/ to the .gitignore file.
Corrected a few clippy lints.

redundant borrow
loop indexing warning

Various documentation fixes.

TODO

[x] Handle alternate registry in the verify stage.
[x] Handle alternate registry in the prepare stage.
[x] Fix Release to not be tied to crates.io

Potential TODO

[x] Try setting up crates.io as an alternate registry for testing.
[ ] Fix failures in napi feature.
[ ] Generate new napi bindings.

Thank you for this contribution, @ncatelli! This code looks great
If you rebase on latest master, you'll pick up a fix for the "Test CLI .*" CI steps.
Then if we can get the "Build Node-API" tests passing I think we're in good shape to merge

Thanks @EricCrosson this has been updated. Would there be a way I could trigger workflows just to help iterate faster on your second point about the napi fixes?

I just tweaked the repository settings, so hopefully workflow runs will trigger automatically now when your branch is updated.
For even faster iterations, it should be possible to build the native bindings locally. On my x86_64-linux machine, I ran
npm ci

and then this line
https://github.com/semantic-release-cargo/semantic-release-cargo/blob/67f68b1fb20288f62177031b7a3682cd02c24b64/.github/workflows/ci.yml#L122
and I see the same errors that CI is reporting

Alright @EricCrosson that should have us back to green. Please note I included updates to the package-lock.json file. Happy to pull that out if need be. Please allso let me know if you'd like those commits squashed down as well.

Looks great to me -- thank you @ncatelli!


Network request failed after unlocking the phone

I am seeing an interesting behavior on iOS using the following simple snippet and performing these steps:

Open the App the first time
Press Home
Re-open the app, as a result of the app becoming active a first fetch request is issued (and it succeeds)
Press Home again
Lock your phone
Unlock your phone
Open the App, as a result of the app becoming active a second fetch request is issued, it fails with the error "Network request failed"!

I can reproduce it every time on my iphone 6 running ios 9.2.1 and I am using react native 0.21.0
I think it is an issue with react native, because I tried to reproduce with a native app but I could not, although I could be mistaken.
Here is the snippet:
import React, {
  AppRegistry,
  AppStateIOS,
  Component,
  View
} from 'react-native';

class TestConnectionFailureRN extends Component {
  componentDidMount() {
    AppStateIOS.addEventListener(
      'change', this._didBecomeActive);
  }
  _didBecomeActive(appState) {
      if (appState != 'active') {
        return;
     }
     return fetch('https://www.google.de', {}).then(
      (response) => console.log(response));
  }
  render() {
    return (
      <View></View>
    );
  }
}

AppRegistry.registerComponent('TestConnectionFailureRN', () => TestConnectionFailureRN);

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/network-request-failed-after-unlocking-the-phone
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.

@mcaoun How did you solve this issue? I'm getting the same behavior on a project I'm currently working on.
For now, I'm implementing a "request retry" whenever a network request error occurs.

I did the same. I retry immediately but only once when catching an error "Network request failed'"

Ok, thanks!


'MjModel' object has no attribute 'eq_active' error caused by mujuco version

Running the example code after installing the dependencies may raise the corresponding error as follows:
Traceback (most recent call last):
  File "/home/pc/PlanInDream/dreamerv3-torch/dreamer.py", line 392, in <module>
    main(parser.parse_args(remaining))
  File "/home/pc/PlanInDream/dreamerv3-torch/dreamer.py", line 266, in main
    train_envs = [make("train") for _ in range(config.envs)]
  File "/home/pc/PlanInDream/dreamerv3-torch/dreamer.py", line 266, in <listcomp>
    train_envs = [make("train") for _ in range(config.envs)]
  File "/home/pc/PlanInDream/dreamerv3-torch/dreamer.py", line 265, in <lambda>
    make = lambda mode: make_env(config, mode)
  File "/home/pc/PlanInDream/dreamerv3-torch/dreamer.py", line 179, in make_env
    env = dmc.DeepMindControl(
  File "/home/pc/PlanInDream/dreamerv3-torch/envs/dmc.py", line 15, in __init__
    self._env = suite.load(
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/suite/__init__.py", line 113, in load
    return build_environment(domain_name, task_name, task_kwargs,
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/suite/__init__.py", line 148, in build_environment
    env = domain.SUITE[task_name](**task_kwargs)
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/suite/walker.py", line 62, in walk
    physics = Physics.from_xml_string(*get_model_and_assets())
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/mujoco/engine.py", line 436, in from_xml_string
    return cls.from_model(model)
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/mujoco/engine.py", line 419, in from_model
    return cls(data)
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/mujoco/engine.py", line 123, in __init__
    self._reload_from_data(data)
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/mujoco/engine.py", line 400, in _reload_from_data
    model=index.struct_indexer(self.model, 'mjmodel', axis_indexers),
  File "/home/pc/anaconda3/envs/PID/lib/python3.9/site-packages/dm_control/mujoco/index.py", line 628, in struct_indexer
    attr = getattr(struct, field_name)
AttributeError: 'MjModel' object has no attribute 'eq_active'

It is an error raised by mujoco version, which could be solved by degrading the version of the mujoco (like 2.3.4)

Hello,
Thank you for reaching out and sharing the issue you're encountering with Docker while trying to run DreamerV3. The problem you're experiencing is indeed related to the recent major update of Mujoco to versions beyond 3.0.0.
To resolve this, I have updated our project to fix the Mujoco version. This fix is included in the latest commit.
If you continue to face any challenges or have further questions, please don't hesitate to reach out again.


Add Underscore.m in Utility

Created a new section called Utility, close #75

can you fix the conflicts?

Fixed

:smile:

:smile:


运行时触发WS 异常：

2022-05-04 21:57:52,571  INFO: 触发异常：[2022-05-04T21:57:52.571520]: Unhandled WebSocket Error:<class 'RuntimeError'>
LastSentText:
{"jsonrpc": "2.0", "id": 7909, "method": "private/get_account_summary", "params": {"currency": "USDC"}}
LastReceivedText:
{"jsonrpc":"2.0","method":"subscription","params":{"channel":"book.ETH-24JUN22-2200-P.none.10.100ms","data":{"timestamp":1651726652457,"instrument_name":"ETH-24JUN22-2200-P","change_id":23054729382,"bids":[[0.019,1965.0],[0.0185,722.0],[0.018,377.0],[0.0175,62.0],[0.017,62.0],[0.0165,54.0],[0.016,64.0],[0.015,20.0],[0.013,45.0],[0.0125,1.0]],"asks":[[0.02,181.0],[0.0205,944.0],[0.021,304.0],[0.022,411.0],[0.0225,41.0],[0.023,49.0],[0.0235,45.0],[0.024,45.0],[0.0245,37.0],[0.0385,75.0]]}}}
Exception trace:
Traceback (most recent call last):
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/vnpy_websocket/websocket_client.py", line 171, in _run
self._ws = await self._session.ws_connect(
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/aiohttp/client.py", line 754, in _ws_connect
resp = await self.request(
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/aiohttp/client.py", line 520, in _request
conn = await self._connector.connect(
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/aiohttp/connector.py", line 535, in connect
proto = await self._create_connection(req, traces, timeout)
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/aiohttp/connector.py", line 892, in _create_connection
_, proto = await self._create_direct_connection(req, traces, timeout)
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/aiohttp/connector.py", line 999, in _create_direct_connection
hosts = await asyncio.shield(host_resolved)
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/aiohttp/connector.py", line 865, in _resolve_host
addrs = await self._resolver.resolve(host, port, family=self._family)
File "/home/yuting/src/vnpy-core/venv/lib/python3.10/site-packages/aiohttp/resolver.py", line 31, in resolve
infos = await self._loop.getaddrinfo(
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 860, in getaddrinfo
return await self.run_in_executor(
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 818, in run_in_executor
executor.submit(func, *args), loop=self)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 167, in submit
raise RuntimeError('cannot schedule new futures after shutdown')
RuntimeError: cannot schedule new futures after shutdown
然后这条报错就不断的在终端上滚动,无法退出，直到ctrl-c (没有使用GUI)

找到问题了，线程其他部分的代码抛出异常了污染了 websocket 线程的 asyncio.


[BoringSSL] Update third_party/boringssl-with-bazel

Change was created by the release automation script. See go/grpc-release.

Closed in favor of https://github.com/grpc/grpc/pull/38274


[BUG] multiMatch lead to unexpected match

Describe the bug
SecRule REQUEST_BODY "@rx abc" "id:111, msg:'test', t:base64DecodeExt,multiMatch, phase:2, block, capture, severity:'CRITICAL', tag:'attack-rce', tag:'paranoia-level/1', setvar:'tx.matched=%{MATCHED_VAR}', chain"
    SecRule REQUEST_BODY "@rx 123"

The above rule match request which body is 'abc'
curl <IP_ADDRESS>:86 -d 'abc'

The match is not expected, because the rule is "chained" rule, the second rule require the body contains "123" string.
If i remove multiMatch ,then everything is expected.
SecRule REQUEST_BODY "@rx abc" "id:222, msg:'test', t:none, phase:2, block, capture, severity:'CRITICAL', tag:'attack-rce', tag:'paranoia-level/1', setvar:'tx.matched=%{MATCHED_VAR}', chain"
    SecRule REQUEST_BODY "@rx 123"

curl <IP_ADDRESS>:86 -d 'abc'  // not match
curl <IP_ADDRESS>:86 -d 'abc123'  // match

So i think the multiMatch flag has bug when it is used with chain rule

Hi @leveryd,
thanks for reporting.
I can confirm this is a bug. I tested this with Apache + mod_security2 and it worked as you expect.
Just one note: the rule above is wrong especially for libmodsecurity3. As I know, every rule (even it's a chained rule) must have some action. So I could have tested with this rule:
SecRule REQUEST_BODY "@rx abc" \
    "id:111,\
    msg:'test',\
    t:base64DecodeExt,\
    multiMatch,\
    phase:2,\
    block,\
    capture,\
    severity:'CRITICAL',\
    tag:'attack-rce',\
    tag:'paranoia-level/1',\
    setvar:'tx.matched=%{MATCHED_VAR}',\
    chain"
    SecRule REQUEST_BODY "@rx 123"
        "setvar:'tx.foo=1'"


JOGL 3D Rendering broken on macOS >= 10.14

Feature extractors and exporters using the JOGLOffscreenRenderer crash on macOS >= 10.14 with the error NSWindow drag regions should only be invalidated on the Main Thread! as described in this forum thread.
The issue is fixed in the 2.4.0 version of JOGL, but this version is still in development and not available as release version.

As there appears no clear timeline for the release of this next library version, is there a workaround which could be used in the meantime?

Maybe jGL could be investigated as an alternative.

What about Three.kt?
From what I understand it is built for real time rendering, like Three.js, but it should probably also work for static rendering.

Interesting. I just checked, it depends on lwjgl for OpenGL which is better maintained than JOGL, offers access to all kinds of things but is heavier. If anything, we could just depend on lwjgl directly.

You are right, using lwjgl makes a lot more sense. It even has macOS aarch64 build artifacts.


Installation advice

Hey Guys,
Been playing with jaeles for a few days now and this is what I did to get it to work:

download one of the pre-compiled binaries, they worked straight away for me. Add it to your path and you good to go. If you need to install it use the GO111MODULE=on go install github.com/jaeles-project/jaeles@latest, the binary is inside the go folder. I have not tested this so unsure how it works.
When using your own signatures, use the full path
if for whatever reason when you do the jaeles config init  you get an error, delete the folder .jaeles and try again, you should se the signatures being downloaded
when sending request from burp, make sure you edit it on responder before sending it. I would also advice to test the signatures on your test server before doing any scans to an operational server.

Otherwise, the tool seems to work like charm. I hope this helps.

Can u explain the 4th point clearly
because, it is stopping at here only and not testing it further.


Consistent Absorbent System behavior

Assumptions
This whole PR operates under the assumption that the below is the expected behavior of how Absorbent system should work:
Upon interacting with an Absorbent entity on an Refillable entity:

as much as possible non-water is to be transferred from absorbent into refillable
as much as possible water is to be transferred from refillable into absorbent

This behavior was inferred from how absorbents currently work on non-empty containers. Notably they currently do not work this way on empty containers.
IMPORTANT: Whether this is how the Absorbent system should work or not is outside the scope of this PR. This PR does not aim to change, it only aims to fix. This means taking the existing behavior and making it consistent, as opposed to making the existing behavior into something else.
About the PR

Made Absorbent component behave consistently:

expected behavior (in assumption) now applies to empty containers as well
fixed one edge case where a popup stating "target is full" would show up even if the transfer completed successfully
added the ability to transfer water from absorbent to container if the only absorbent solution is evaporating and Refillable is empty
added test cases for Absorbent <-> Refillable interaction (feel free to run them on master to see what was changed in this PR)
minor refactoring (updated obsolete calls, removed redundant system dependency, etc.)

Why / Balance

It is possible to transfer dirt from a dirty mop to a dirty bucket, but attempting the same thing with an empty bucket fails unexpectedly. It follows that if a mop can be "wrung" over a bucket filled with a dirty solution, it can just as well be "wrung" over an empty bucket. Similarly, if "wringing" an arbitrary solution is allowed, then wringing clean solution out of the mop should also be possible.
Now it is.
I personally started the game as a janitor. Figuring out game controls for the first time was hard enough on its own, and this inconsistency made it even harder for me to learn how things work. So at the very least this benefits new player learning experience. Although making things consistent is generally good by itself.
Technical details

PREVIOUSLY:
Mop + Non-empty container -> transfer contaminants, absorb water
Mop w/ contaminants + Empty container -> "container has no water" popup, no effect
Mop w/o contaminants + Empty container -> "mop is full of water" or "container has no water", no effect
NOW:
Mop + Non-empty container -> transfer contaminants, absorb water
Mop w/ contaminants + Empty container -> transfer contaminants absorb water*
Mop w/o contaminants + Empty container -> transfer non-contaminants
* - since container is empty, it absorbs 0 water, but this is still consistent
Media

mop.webm
(All visible buckets initially empty - this would not have been possible on current master branch)

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Extra
All comments - especially about the tests welcome. I had a hard time figuring out the expected tests format. Hopefully this is OK.

this is exceptionally solid work especially with the test, i'll take a proper look after i wake up


Is it possible to decouple compile-time checked queries from the use of prepared statements?

I know the answer is "no" currently, so I guess this is potentially a feature request.
Prepared statements in Postgres cannot be used in all cases. For instance, it is incompatible with pgbouncer (unless this has changed in the last year or so). In some deployments there is no option other than to connect to a pgbouncer sidecar for DB connectivity.
I wonder if there is potential to make the use of prepared statements at runtime a separate choice to having compile-time checked queries. Currently, the API has conflated these two decisions, to the best of my understanding.

Support for PgBouncer is being tracked separately but this is for sure a decision we can make separately. The runtime and compile-time layers are mostly separate.
From what I understand PgBouncer supports prepared statements but does not support caching of them. We could add a general configuration for 'prepared statement cache ttl' which a PgBouncer user could set to zero or something.

From what I understand PgBouncer supports prepared statements but does not support caching

Yeah that sounds familiar. I may be mistaking the fact that in other frameworks I had to disable them altogether (perhaps because they could not just disable caching). Either way, in evaluating this project, this came up to me as a potential gotcha.
There may be other scenarios in which prepared statements can't be used, but I am not specifically aware of them.
De-conflating these seems useful in the case where you do want prepared statements but can't, for some reason, pre-compile them. In a case I am contending with, I am include_str!()ing a large query, and the query! macro must take include_str! literally as a token instead of evaluating it and using the query it returns.
This example could possibly be resolved with improvements to query! but my point is more illustrative that there may be additional reasons why prepared statements and compile-time checked queries should be orthogonal concerns.
Thanks for your fast response!

In a case I am contending with, I am include_str!()ing a large query, and the query! macro must take include_str! literally as a token instead of evaluating it and using the query it returns.

We have query_file!() for this situation, although the path is unfortunately not relative to the source file's directory as there's no stable way we can determine that currently; instead it's relative to CARGO_MANIFEST_DIR (the directory containing Cargo.toml for the current crate).
We use prepared statements because it's the only way the databases give us any insight into the query as far as inputs and outputs go. Replacing that would involve writing our own SQL parser and analyzer, basically reimplementing a good chunk of the server-side frontend, for each database flavor we intend to support. We may end up doing that anyway for a number of reasons but that's still a significant undertaking.
For queries that cannot be prepared and are expected to have no inputs or outputs we could instead, potentially, execute them in a transaction that is never committed; however, MySQL doesn't support schema-changing operations in transactions which excludes all but a few edge cases. We also cannot infer when a query cannot be prepared without at least naively parsing the input SQL so this would have to be a different macro entirely anyway.

Yup, I expected that there would be some pretty gnarly stuff to consider here. Thanks for the additional context @abonander!

We have query_file!() for this situation, although the path is unfortunately not relative to the source file's directory as there's no stable way we can determine that currently; instead it's relative to CARGO_MANIFEST_DIR (the directory containing Cargo.toml for the current crate).

Good to know! I'll take advantage of that from now on :)

De-conflating these seems useful in the case where you do want prepared statements but can't, for some reason, pre-compile them. In a case I am contending with, I am include_str!()ing a large query, and the query! macro must take include_str! literally as a token instead of evaluating it and using the query it returns.

Ah I think I misunderstood how SQLx worked when I mentioned this. In the case above, I see now that conn.execute("RAW QUERY") would work in that case.

However, my original prompt of validating queries at compile time but executing them in text mode is still relevant. I think that in terms of behaviour, it would be reasonable to require a DB connection at compile time which uses prepared statements, but the code that it outputs could be the raw query string executed in text mode but decoded to the types whose metadata was collected at compile time. Does that even make sense?

The solution to #121 is going to be extensible to theoretically any kind of configuration options we want to pass into the macros; we could consider adding this as a flag to change the generated code. It doesn't feel right to change it in the general case, however.
I think instead I agree with @mehcode about a flag on the connection itself to avoid using prepared statements; it'll require some experimentation as that may not be enough and the flag may have to turn off prepared statements altogether and instead submit the query string raw.
If you want to add bind parameters we would have to manually interpolate them into the query string, however solving #300 should basically take care of that.

Closing as this discussion seems resolved. Please re-open if not.


Cannot run minimalist example

Hi. Tried the "minimalist example" and all I get is failed setup. i'm running I2c mode and it is wired up via a arduino uno and a quic shield. NOTE: I get the same thing if wired directly to SDA/SCL
If I run the arduino playground iIC scanner I can see a device at 0x1D and 0x1E which should be correct. After I changed the debug value in the header file and this is what I get:
begin::
MAMU_HW_ERROR
Failed setup.
::Reading address 0x20
I2C_ByteRead::Reading from I2C address: 0x1D, register 0x20
I2C_ByteRead::IMU_HW_ERROR
Failed setup.
::Reading address 0x20
I2C_ByteRead::Reading from I2C address: 0x1D, register 0x20
I2C_ByteRead::IMU_HW_ERROR
Failed setup.
begin::
MAG_SetODR::MAG_ReadReg::Reading register 0x20
I2C_ByteRead::Reading from I2C address: 0x1E, register 0x20
I2C_ByteRead::Error: I2C buffer didn't get sent!
I2C_ByteRead::Slave address: 0x1E
I2C_ByteRead::Register: 0x20
Failed Read from MAG_CTRL_REG1
MAG_SetFullScale::MAG_ReadReg::Reading register 0x21
I2C_ByteRead::Reading from I2C address: 0x1E, register 0x21
I2C_ByteRead::Error: I2C buffer didn't get sent!
I2C_ByteRead::Slave address: 0x1E
I2C_ByteRead::Register: 0x21
MAG_BlockDataUpdate::MAG_ReadReg::Reading register 0x24
I2C_ByteRead::Reading from I2C address: 0x1E, register 0x24
I2C_ByteRead::Error: I2C buffer didn't get sent!
I2C_ByteRead::Slave address: 0x1E
I2C_ByteRead::Register: 0x24
MAG_XY_AxOperativeMode::MAG_ReadReg::Reading register 0x20
I2C_ByteRead::Reading from I2C address: 0x1E, register 0x20
I2C_ByteRead::Error: I2C buffer didn't get sent!
I2C_ByteRead::Slave address: 0x1E
I2C_ByteRead::Register: 0x20
MAG_Z_AxOperativeMode::MAG_ReadReg::Reading register 0x23
I2C_ByteRead::Reading from I2C address: 0x1E, register 0x23
I2C_ByteRead::Error: I2C buffer didn't get sent!
I2C_ByteRead::Slave address: 0x1E
I2C_ByteRead::Register: 0x23
MAG_SetMode::MAG_ReadReg::Reading register 0x22
I2C_ByteRead::Reading from I2C address: 0x1E, register 0x22
I2C_ByteRead::Error: I2C buffer didn't get sent!
I2C_ByteRead::Slave address: 0x1E
I2C_ByteRead::Register: 0x22
MAG_SetMode::Failed to read MAG_CTRL_REG3. 'Read': 0x0
ACC_SetFullScale::ACC_ReadReg::Reading address 0x23
I2C_ByteRead::Reading from I2C address: 0x1D, register 0x23
I2C_ByteRead::IMU_HW_ERROR
Failed ACC read
ACC_BlockDataUpdate::ACC_ReadReg::Reading address 0x20
I2C_ByteRead::Reading from I2C address: 0x1D, register 0x20
I2C_ByteRead::IMU_HW_ERROR
ACC_EnableAxis::ACC_ReadReg::Reading address 0x20
I2C_ByteRead::Reading from I2C address: 0x1D, register 0x20
I2C_ByteRead::IMU_HW_ERROR
ACC_EnableAxis::
Accel Error
ACC_SetODR::ACC_ReadReg::Reading address 0x20
I2C_ByteRead::Reading from I2C address: 0x1D, register 0x20
I2C_ByteRead::IMU_HW_ERROR
Failed setup.

ok i found the issue.... the version on the sparkfun website is WAY out of date and doesn't work. just reinstalled with the github master and it works... you need to update the sparkfun library download page

Hi,
Sorry about that. I updated the zip to point to the GitHub repo's ZIP.
It looks like the libraries.properties was not updated with the code. That is what is needed to update the version number in the Arduino's library manager too.  I'll take a look at that too.

Fixed.  I have updated the tutorial links and references  to reflect the changes. You should be able to download the latest version from the library manager. As a quick note, I moved the Wire.begin() and clock stretching into the example code's setup(). It is bad practice to include it in the .cpp file as this resets the bus.  Also, I adjusted the baud rate to 57600. If you are using a 3.3/8MHz, the maximum reliable baud rate is 57600 from my experience.


potential final draft for issue #24

I found a way to count the number of duplicates for larger datasets (line 110 of data_reports/128_WorldwideSCND.Rmd) that worked for the dataset_profile
However, for the dataset_layer it shows that three rows were not removed and I'm not sure of the error.
This last tweak will leave the data report complete and ready for merging.

Ready for pull request!


[SetProperty] Property copy became shallow copy instead of deep copy

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
What package version of the SDK are you using.
Describe the bug
When using SetProperty to copy property A to property B then modify property B, you can find property A had been modified too.
To Reproduce
Steps to reproduce the behavior:

Add a set property:

dialog.CreateCalendarEntry_FromTime is a datetime:
{
"value": "23:00:00",
"timex": "T23:00"
}

Modify dialog.CreateCalendarEntry_ToTime to addSeconds(formatDateTime(dialog.CreateCalendarEntry_FromTime[0].value), int(dialog.CreateCalendarEntry_Duration[0].value)) with set property

Print dialog.CreateCalendarEntry_FromTime

See dialog.CreateCalendarEntry_FromTime changed also

Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

https://github.com/microsoft/BotFramework-Composer/issues/1206

A simple way to repro this:
var rootDialog = new AdaptiveDialog(nameof(AdaptiveDialog))
{
Triggers = new List()
{
new OnConversationUpdateActivity()
{
//Actions = WelcomeUserAction()
},
new OnBeginDialog()
{
Actions = new List()
{
new DateTimeInput()
{
Prompt = new ActivityTemplate("what is your date?"),
Property = "dialog.date",
MaxTurnCount = 3,
                            //AllowInterruptions = "this.value != null"
                        },
                        new SetProperty()
                        {
                            Property = "dialog.date1",
                            Value = "dialog.date"
                        },
                        new SendActivity()
                        {
                            Activity = new ActivityTemplate("{dialog.date}, {dialog.date1}")
                        },
                        new SetProperty()
                        {
                            Property = "dialog.date[0].value",
                            Value = "'updated value'"
                        },
                        new SendActivity()
                        {
                            Activity = new ActivityTemplate("{dialog.date}, {dialog.date1}")
                        }
                    }
                }
            }
        };

Expected behavior:
bot: what is your date?
user: tomorrow
bot: [
{
"value": "2019-10-24",
"timex": "2019-10-24"
}
], [
{
"value": "2019-10-24",
"timex": "2019-10-24"
}
]
bot:[
{
"value": "updated value",
"timex": "2019-10-24"
}
], [
{
"value": "2019-10-24",
"timex": "2019-10-24"
}
]
Actual behavior:
bot: what is your date?
user: tomorrow
bot: [
{
"value": "2019-10-24",
"timex": "2019-10-24"
}
], [
{
"value": "2019-10-24",
"timex": "2019-10-24"
}
]
bot:[
{
"value": "updated value",
"timex": "2019-10-24"
}
], [
{
"value": "updated value",
"timex": "2019-10-24"
}
]


InventoryPages

https://www.spigotmc.org/resources/inventorypages.32432/
Would you support InventoryPages?

Hi, afraid not as that would require a lot of extra work to get the additional pages compatible with this plugin. Also since the plugin is premium I would have no way of implementing it without a copy of the plugin to test with.


Extract topic tests.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

Great, merged, thanks @showell!


Second toilet appearing

Describe the bug
After add ing a toilet to a new place on the map.
The user edits the details, and a second toilet appears in the same location.
To Reproduce
Steps to reproduce the behavior:

Go to 'Add a toilet'
Add detail and save changes
Edit the loo several times and save the changes each time.
A second toilet appears in the same place

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Browser - chrome

Related PR: #400

Testing Details
Toilet added to A147 St.Crispins Roundabout

@ob6160
Boo*


SqlAGReplica: Test may calculate EndpointHostName incorrectly

This is a very low priority issue as it's not blocking me on anything.
Details of the scenario you tried and the problem that is occurring
I was testing a DSC which modifies the ReadOnlyRoutingConnectionUrl.
$configurationData = @{
    AllNodes = @(
        @{
            NodeName = '*'
            PsDscAllowPlainTextPassword = $true
            PsDscAllowDomainUser = $true
        }
        @{
         NodeName = 'localhost'
        }
    )
}

configuration Blah {
    param(
        $Credential
    )
    Import-DscResource -Module SqlServerDsc

    Node $AllNodes.NodeName {
        SqlAGReplica Blah {
            ServerName = 'DAC1N1'
            InstanceName = 'MSSQLSERVER'

            AvailabilityGroupName = 'AG1'
            Name = 'DAC1N1'
            ReadOnlyRoutingConnectionUrl = 'tcp://DAC1N1.lab.com:1433'
            PsDscRunAsCredential = $Credential
        }
    }
}

if (!$credential) { $credential = Get-Credential LAB\LocalAdministrator }
Blah -Credential $credential -ConfigurationData $configurationData
Start-DscConfiguration -Force -Wait -Verbose -Path Blah

In this case DAC1N1 is a secondary, the code finds SEC1N1 is the primary, connects to that to do its update and everything is fine.
But if you try to be smart and connect to the primary first go (ServerName = 'SEC1N1') and expect it just to modify the AG for DAC1N1 things go awry.
VERBOSE: Perform operation 'Invoke CimMethod' with following parameters, ''methodName' = SendConfigurationApply,'className' = MSFT_DSCLocalConfigurationManager,'namespaceName' = root/Microsoft/Windows/DesiredStateConfiguration'.
VERBOSE: An LCM method call arrived from computer CHWK01 with user sid S-1-5-21-4179733049-4140502011-3911003846-1106.
VERBOSE: [CHWK01]: LCM:  [ Start  Set      ]
VERBOSE: [CHWK01]:                            [DSCEngine] Importing the module C:\Program Files\WindowsPowerShell\Modules\SqlServerDsc\DscResources\MSFT_SqlAGReplica\MSFT_SqlAGReplica.psm1 in force mode.
VERBOSE: [CHWK01]: LCM:  [ Start  Resource ]  [[SqlAGReplica]Blah]
VERBOSE: [CHWK01]: LCM:  [ Start  Test     ]  [[SqlAGReplica]Blah]
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Importing the module MSFT_SqlAGReplica in force mode.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Importing the module MSFT_SqlAGReplica in force mode.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Preferred module SqlServer found.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Importing PowerShell module 'SqlServer' with version '21.0.17279' from path 'C:\Program Files\WindowsPowerShell\Modules\SqlServer\SqlServer.psm1'.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Connected to SQL instance 'SEC1N1'.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] 2019-03-13_23-47-10: 'EndpointHostName' should be 'SEC1N1' but is 'DAC1N1'
VERBOSE: [CHWK01]: LCM:  [ End    Test     ]  [[SqlAGReplica]Blah]  in 1.5470 seconds.
VERBOSE: [CHWK01]: LCM:  [ Start  Set      ]  [[SqlAGReplica]Blah]
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Importing the module MSFT_SqlAGReplica in force mode.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Importing the module MSFT_SqlAGReplica in force mode.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Preferred module SqlServer found.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Importing PowerShell module 'SqlServer' with version '21.0.17279' from path 'C:\Program Files\WindowsPowerShell\Modules\SqlServer\SqlServer.psm1'.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Found PowerShell module SqlServer already imported in the session.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Connected to SQL instance 'SEC1N1'.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] The recommended account 'NT SERVICE\ClusSvc' is missing one or more of the following permissions: Trying with 'NT AUTHORITY\SYSTEM'.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Getting the effective permissions for the login 'NT AUTHORITY\SYSTEM' on 'MSSQLSERVER'.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Found PowerShell module SqlServer already imported in the session.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] Connected to SQL instance 'SEC1N1'.
VERBOSE: [CHWK01]:                            [[SqlAGReplica]Blah] The cluster login 'NT AUTHORITY\SYSTEM' has the required permissions.
VERBOSE: [CHWK01]: LCM:  [ End    Set      ]  [[SqlAGReplica]Blah]  in 0.8290 seconds.
VERBOSE: [CHWK01]: LCM:  [ End    Resource ]  [[SqlAGReplica]Blah]
VERBOSE: [CHWK01]: LCM:  [ End    Set      ]
VERBOSE: [CHWK01]: LCM:  [ End    Set      ]    in  2.5190 seconds.
VERBOSE: Operation 'Invoke CimMethod' complete.
VERBOSE: Time taken for configuration job to complete is 2.541 seconds

It seems that the code in SqlAGReplica connects to whatever the ServerName\InstanceName is and uses that to get the EndpointPort and EndpointHostName information. So if you specify a different replica, these are populated incorrectly and so show as "not in desired state".
When the Set code runs, I didn't specify an EndpointPort or EndpointHostName explicitly so they won't be changed. This is a good thing but means desired state is never achieved.
Suggested solution to the issue
I'm not sure whether this should just be a documentation change to make clear that ServerName and InstanceName should match the Name.
Otherwise we could make a third connection to Name and use that to populate Endpoint variable details instead of $serverObject.
Version of the DSC module that was used ('dev' if using current dev branch)
dev

Maybe it should verify that it is on the correct replica (name) before it verifies properties? If on the wrong replica, it connects to the correct replica. But just thinking out loud here. 🙂


[ENH] Ingress HCP preprocessed data

HCP preprocessed data has comparable quality to QSIPrep-preprocessed data. Since so much HCP preprocessed data is available, we should be able to load it directly into the reconstruction workflows.

Hi, just wondering are there any updates on this? Thanks!


Monitor LDAP server(s) availability when using keystone-ldap

When using LDAP via keystone-ldap, if the servers configured in "ldap_server" become unresponsive or cannot be consistently reached the keystone/apache workers remain blocked until they timeout (logs below).
This means that the API/CLI becomes unresponsive. This situation can be hard to troubleshoot depending on the cloud and whether the LDAP server(s) are fully unavailable or just unresponsive.
It would help to add monitoring to have a separate alert that detects this condition.
2021-07-13 21:52:01.508342     raise exc_value
2021-07-13 21:52:01.508346   File "/usr/lib/python3/dist-packages/ldap/ldapobject.py", line 313, in _ldap_call
2021-07-13 21:52:01.508348     result = func(*args,**kwargs)
2021-07-13 21:52:01.508372 ldap.TIMEOUT
2021-07-13 21:59:45.513100 Timeout when reading response headers from daemon process 'keystone-public': /usr/bin/keystone-wsgi-public
2021-07-13 22:02:06.019309 Timeout when reading response headers from daemon process 'keystone-public': /usr/bin/keystone-wsgi-public
2021-07-13 22:02:12.364508 Timeout when reading response headers from daemon process 'keystone-admin': /usr/bin/keystone-wsgi-admin
(...)
2021-07-13 23:19:13.846645 mod_wsgi (pid=1525295): Unable to connect to WSGI daemon process 'keystone-admin' on '/var/run/apache2/wsgi.1299452.6.1.sock' after multiple attempts as listener backlog limit was exceeded.
2021-07-13 23:19:15.374640 mod_wsgi (pid=1349158): Unable to connect to WSGI daemon process 'keystone-admin' on '/var/run/apache2/wsgi.1299452.6.1.sock' after multiple attempts as listener backlog limit was exceeded.

Imported from Launchpad using lp2gh.

date created: 2021-07-14T08:18:11Z

owner: jfguedez

assignee: None

the launchpad url

Hello!
I had the same issue last week , and in my case wasn't about the middleware or ldap. I discovered a network issue that implies in the same sympton in my infrastructure.
I strongly recommed check 2 things:

Persistence configuration, e. g. Apache keepalive.
Network timeouts.

Have a nice day!


feat(ingestion/airflow): Added support of datajob entity as input lineage

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@shubhamjagtap639 can you move this PR to open-source?


Latest releases are not tagged on GitHub

The package is at 0.2.6 and GitHub only shows 0.1.1

@ChrisRackauckas it might be because the https://github.com/SciML/ADTypes.jl/blob/main/.github/workflows/TagBot.yml#L14 is not defined? cc:@thazhemadam

I would be very surprised if there wasn't a SciML-wide GitHub token that can be used here instead of creating a repo-specific one.
The other thing is that the token must be given the correct access, ideally on a job-wise basis with the permissions setting:
https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/enabling-features-for-your-repository/managing-github-actions-settings-for-a-repository#setting-the-permissions-of-the-github_token-for-your-repository


Add "Update" Functionality

I have the functionality ready to push, but waiting till other bugs are fixed before adding this as it can have the potential to cause issues (it affects the entire sesion data) in another area and it'd be best to determine that this was the cause after fixing the other bugs first.

Is this pushed? Can I test?

Yes!

From: ikb6 @.>
Sent: Thursday, September 21, 2023 1:41 PM
To: CDCgov/MicrobeTrace @.>
Cc: Moscoso, Evan (CDC/DDID/NCHHSTP/OD) (CTR) @.>; Assign @.>
Subject: Re: [CDCgov/MicrobeTrace] Add "Update" Functionality (Issue #705)
Is this pushed? Can I test?
—
Reply to this email directly, view it on GitHubhttps://github.com/CDCgov/MicrobeTrace/issues/705#issuecomment-1730027422, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATFBT2QPGDPAWPFYVPWRE7TX3R34PANCNFSM6AAAAAA3EK7KIY.
You are receiving this because you were assigned.Message ID: @.***>

For dev

From: Moscoso, Evan (CDC/DDID/NCHHSTP/OD) (CTR) @.>
Sent: Thursday, September 21, 2023 1:41 PM
To: CDCgov/MicrobeTrace @.>; CDCgov/MicrobeTrace @.>
Cc: Assign @.>
Subject: Re: [CDCgov/MicrobeTrace] Add "Update" Functionality (Issue #705)
Yes!

From: ikb6 @.>
Sent: Thursday, September 21, 2023 1:41 PM
To: CDCgov/MicrobeTrace @.>
Cc: Moscoso, Evan (CDC/DDID/NCHHSTP/OD) (CTR) @.>; Assign @.>
Subject: Re: [CDCgov/MicrobeTrace] Add "Update" Functionality (Issue #705)
Is this pushed? Can I test?
—
Reply to this email directly, view it on GitHubhttps://github.com/CDCgov/MicrobeTrace/issues/705#issuecomment-1730027422, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATFBT2QPGDPAWPFYVPWRE7TX3R34PANCNFSM6AAAAAA3EK7KIY.
You are receiving this because you were assigned.Message ID: @.***>

Just deployed to angular.  Should be available in 15.


Rename Array.prototype.flatten to flat

In lib.esnext.array.d.ts, Array and ReadOnlyArray's flatten needs to be renamed to flat the consensus name as of the May TC39 meeting.

Duplicate and already solved: #24526.

@j-oliveras aha thank you! I thought I remembered seeing something but search turned up dry.


On very first log in , user is not able to see the clusters that user should have access to because of group association unless refreshed.

Rancher server version - Build from master
Steps to reproduce the problem:
FreeIPA Auth enabled rancher server
"user1" is part of "group1" and "group2"
Create 3 clusters (c1,c2,c3)
Add "group1" as cluster owner to c1
Add "user1" as cluster owner to c2
Add "group2" as cluster owner to c3
Log in as "user1".
Notice that UI presents access to only cluster c2.
on refresh , all clusters (c1,c2 and c3)  are made available.

For (only) this situation I would say it should block auth until their info is available.
Alternately if there were a websocket event telling me something about the user changed I could refresh the displayed projects/clusters.


Added review post for CV review

This PR creates a Review post component for CV review
Please provide a short summary explaining what this PR is about.
Fixes # https://github.com/HackYourFuture-CPH/rate-my-cv/issues/89
How to test?
It can be tested in storybook

Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

hey @pdsorensen ,


Container Service SDK doesn't return an error message for 400 errors

👋
We've noticed that the Container Service SDK doesn't return a complete error message when an err object is returned (here's the relevant issue on our side, but we've seen this for other issues too](https://github.com/terraform-providers/terraform-provider-azurerm/issues/1416)):
containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="" Message=""

The full error message is available in the HTTP Response - such that this just isn't being parsed out correctly; as such would it be possible to fix the SDK so that this error is raised?
Thanks!

@tombuildsstuff I don't suppose you have the response body handy, or even better a simple repro?  I've been having a hard time putting a repro together.  :(

@jhendrixMSFT not to hand, in this case the error came from using an incorrect machine size - however I’ll try and put one together in the morning :)

@jhendrixMSFT I've added a repro in this repo: https://github.com/tombuildsstuff/azure-sdk-for-go-bug-2085

Thanks a lot, will dig into this today.

The good news is this is fixed in recent builds.  I'll figure out which commit fixed it.

This was fixed in go-autorest v10.9.0 in the future state machine rewrite.  If possible I would recommend moving to v17 of the SDK in conjunction with v10.9 of go-autorest as the futures rewrite included some changes to our code generator which debuted in v17 of the SDK (this didn't introduce breaking changes for consumers, just the internals of some methods).


Use datastore other than firebase

Hi, Thank you for making this great project.
I would like to ask, is it possible to abstract datastore so one can switch from firebase to mysql easily? or could you please point to the place where the changes can be made to use mysql instead of firebase?  Thank you.

I agree
"No advertisements, no data sharing, no marketing emails" yet, you rely on a Google service, even sqlite would be fine

sqlite would also be useful to resolve these firebase API issues in the docker version ala #430

Yes, this would be great change. I don't want it to be dependent on a Cloud service. Firebase is great voor large scale resume websites. I just want to use it for a couple of personal ones.

+1 this.
I want to self-host and avoid putting my data in a Google datacenter.
MariaDB, Postgresql, sqlite; all good options.

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :)
Thank you!


Raw MIDI output

In #104 I added output ports for CLAP_EVENT_NOTE_BLAH
but I just made midi events fprintf stderr for now
We need to fix that.

removed the solved in next from this; the CLAP_NOTE ones work in next but the MIDI ones dont yet

Note to self: we also want to send CLAP_NOTE_EXPRESSIONS out.

Since initial version is complete, for CLAP_NOTE_EXPRESSIONS we can open a new issue.
Notes, Program Changes and simple controllers are being sent.


feat: Replace ther BottomNavigationBar with a custom NavigationBar

Hi everyone!
We continue to bring some parts of the POC
Here, we're replacing the BottomNavigationBar with a NavigationBar (from Material 3).
Unfortunately, the Flutter Widget doesn't bring all the features we want (in this case, that a non-selected item has an indicator background color). That's why there's a custom implementation.
Also, you may notice that the animation starts from where you clicked 👀
Video:
https://github.com/user-attachments/assets/d0a0f834-ddf6-41a2-9dee-7486cbfe343f

Codecov Report
Attention: Patch coverage is 0% with 150 lines in your changes missing coverage. Please review.

Project coverage is 6.62%. Comparing base (4d9c7fc) to head (ab88ceb).
Report is 496 commits behind head on develop.

Files with missing lines
Patch %
Lines

.../smooth_app/lib/widgets/smooth_navigation_bar.dart
0.00%
139 Missing :warning:

packages/smooth_app/lib/pages/page_manager.dart
0.00%
11 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5900      +/-   ##
==========================================
- Coverage     9.54%   6.62%   -2.93%
==========================================
  Files          325     437     +112
  Lines        16411   24350    +7939
==========================================
+ Hits          1567    1613      +46
- Misses       14844   22737    +7893

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

Oops, I first need #5895 to be merged.
The new icons are in this PR.

@g123k it is merged

Now with custom icons:


massive performance regression between 1.63 and 1.67

Hi,
I haven't been able to bisect yet but I have observed a massive performance degradation between 1.63 and 1.67 when using a BOOST_DATA_TEST_CASE with an xrange of 100,000 values.
It takes 10 minutes just for the test system to initialize itself. It was instant before.

Thanks for the report. I believe this is a duplicate of #163. Solved already but not merged.


Save rllib algorithm and accessing param in visualizer_rllib

Closes #173
Key changes:

visualizer_rllib.py now attempts to look for flow_params["run"] (parsed from params.json in result of an rllib experiment) which stores the algorithm used to run the experiment (e.g. "PPO")
if flow_params["run"] is not found, passing in --run ALG when running visualizer_rllib is still an option. This ensures that results from running previous rllib experiments that didn't store the "run" param can still be visualized.
Proposed usage: examples/rllib/stabilizing_the_ring.py now has run as a param (storing algorithm name "PPO"), and changes were made so that one did not need to import the ppo agent and also type the name of the agent "PPO", but only needed to define the agent name "PPO" once in flow_params and then the rest of the example imports the appropriate agent programmatically and stores it.

LGTM minus the comment about moving run out of flow_params. Nice work!

Also, this is failing pep8 tests. You can see where it failed by examining the travis build.

Currently suggests new usage of storing algorithm name in rllib/stabilizing_the_ring.py example.
If it looks good, I can modify the other examples and tutorials

Yeah, I think this looks good! We could add this to everything in the rllib folder and benchmarks/rllib!

@kev-chien it's not passing Pep8 requirements. You can see the error by clicking on details on the travis


Implement UPath.joinuri

Close #88
This seems to be the best compromise regarding urljoin behavior for UPath classes.
UPath.joinpath behaves as close as possible to pathlib.Path
and
UPath.joinuri behaves as urllib.parse.urljoin

@joouha this might be interesting for euporie.
I've tried getting the join behavior for HTTPPath to be consistent, but I think this is the best compromise.
If you want urljoin behavior for any UPath, you can use UPath().joinuri(...)
Instead UPath() / "..." or UPath().joinpath(...) will provide pathlib join behavior.

Thanks so much for this.
I tried implementing using urljoin behaviour with the / interface, but it felt confusing and wrong when I actually used it, so I think this is a brilliant compromise.


Move nag scripts in subdirectories

This will make it easier to find all scripts related to a given topic.
E.g. all scripts related to the bug type could be in a bug_type directory.

yeah, we could have directories related to the action (e.g. change bug_type, needinfo foobar, ...).


cant import in swift

I can't import like import MRProgress in swift.
No such module 'MRProgress'

I installed it from pod. (pod 'MRProgress')
What is problem here?

I don't know if this is happening only to me?

Which version of CocoaPods do you use? Do you have use_frameworks! in your Podfile for the target you integrate?

I don't use use_frameworks! becouse GoogleCloudMessages doesn't support it.
That is problem?

Otherwise you will need to import it by #import <MRPRogress/MRProgress.h> in your bridging header.

How can i import it in bridging header, any steps?

https://bohemianpolymorph.wordpress.com/2014/07/11/manually-adding-a-swift-bridging-header/

I did this, but still same error: No such module 'MRProgress'.
What i did:

installed from pod without use_framework!
created file MyApp-Bridging-Header.h with content:

//
//  MyApp-Bridging-Header.h
//  MyApp
//
//  Created by Mirza Delic on 7/1/15.
//  Copyright (c) 2015 Mirza Delic. All rights reserved.
//

#ifndef MyApp_MyApp_Bridging_Header_h
#define MyApp_MyApp_Bridging_Header_h

#import <MRPRogress/MRProgress.h>

#endif

in Build Settings Objective-C Bridging Header location 'MyApp/MyApp-Bridging-Header.h`.
then i build and still same error.

What am i doing wrong?

Did you also removed import MRProgress from your Swift code?

Ohh, i forgot that, thanks.


[Bug] Import error about 'mmdet.utils.typing' in mmdet/models/detectors/condinst.py

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (master) or latest version (3.x).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
3.x branch https://github.com/open-mmlab/mmdetection/tree/3.x
Environment
1.9
Reproduces the problem - code sample
https://github.com/open-mmlab/mmdetection/blob/dev-3.x/mmdet/models/detectors/condinst.py#L4
Reproduces the problem - command or script
no
Reproduces the problem - error message
no
Additional information
No response

Thank for your report, we will fix ASAP

fixed in https://github.com/open-mmlab/mmdetection/pull/9376


How to pass parameters with post method?

Just use the standard Retrofit annotations as described in https://square.github.io/retrofit/. See the "Form Encoded and Multipart" section. You can also see added test: 9681af59b79dd6fe75d834b47ec92b4e39507787.


Added a check for the value of Omega_k.

Values of Omega_k < -1.0135 (approximately), causes the kernel to crash, so I added an error message.

Pull Request Test Coverage Report for Build 4944

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 96.995%

Totals

Change from base Build 4932:
0.002%

Covered Lines:
3260

Relevant Lines:
3361

💛  - Coveralls

Just checking, is there anything particular about -1.0135?

If the value is approximate, it might be best to leave a small safety buffer and cut at -1 instead.


sample/sharepoint-list-update-created-by

By submitting this pull request, you agree to the contribution guidelines

If you aren't familiar with how to contribute to open-source repositories using GitHub, or if you find the instructions on this page confusing, sign up for one of our Sharing is Caring events. It's completely free, and we'll guide you through the process.

To submit a pull request with multiple authors, make sure that at least one commit is a co-authored commit by adding a Co-authored-by: trailer to the commit's message. E.g.: Co-authored-by: name<EMAIL_ADDRESS>

Put an x in all the items that apply ([x], without spaces), make notes next to any that haven't been addressed.

[ ] Bug fix?
[x] New sample?
[ ] Related issues: fixes #X, partially #Y, mentioned in #Z
[ ] Needs API permissions?
[ ] Has other prerequisites? (E.g. requires a list, document library, etc.)

What's in this Pull Request?
This is a new sample for a solution called Update "Created by" in a Microsoft List, which is a Power Automate flow updates the “Created By” field of a Microsoft List item, replacing the account with a specified user based on their email address, using an HTTP request to SharePoint.
Checklist

[x] My pull request affects only ONE sample.
[x] I have updated the README file.
[x] My README has at least one static high-resolution screenshot (i.e. not a GIF)
[x] My README contains complete setup instructions, including pre-requisites and permissions required

Hi @Glynnryan ,
Thank you for your submission! We value your time and contribution.
To move forward with the review and merging of your pull request, we kindly request that you update your PR in accordance with the Contribution Guidelines. Please ensure that the source code is prepared following these instructions.
Thank you!
Regards,
Katerina Ch.

Hi @Katerina-Chernevskaya,
Apologies for the incorrect format. I'm swamped with work at the moment and will look into this next week.

Hi @Katerina-Chernevskaya, I found a quick gap to update the PR.
I've submitted the sample using these instructions.
I also forgot to add a description for the changes in commit 79fe3be. Apologies... This was to update the tracking image src.
I hope all is now in order.


Redesign https://ipfs.io/blog/

Instead of listing the links to the blogs, can we feature the most recent blog post? This page/portal could use a lot of love. What do you think @RichardLitt?

That sounds smart to me.

@RichardLitt Glad you agree! What do the next steps for that look like? Who owns the design of this page? Can we move to a Wordpress model? If not, what are our options?

Next steps: Feedback from @jbenet. I don't like making design decisions without his feedback.
Design owner: @jbenet.
Wordpress: negative. We should keep this using IPFS, so that it is permanent.
Options: AFAIK, keep the current system, just redesign it. The website was recently redone, but the blog wasn't included.

However, an overhaul in general shouldn't be talked about here: it's a duplicate of this issue posted by @jbenet a bit ago.

Wordpress: negative. We should keep this using IPFS, so that it is permanent.

Agreed, it's also important that we eat our own dogfeed as much as possible, just to demonstrate that it works, the different use cases, etc.

In the previews for the website there was already a new design for the blog done by &yet, what happened to that?

@em-ly I believe what you mean by a moving to a Wordpress model is more in the lines of getting a nicer experience to publish and track the activity on each blog post, than using Wordpress itself. The reason why we are reluctant to move to Wordpress is because of how it is set up, Wordpress is a full Web Service, with a DB, a Web Server and currently, there is no way to serve a blog out of Wordpress through IPFS (yet! :)).
We can always level up our blog to have a nicer editing experience and a monitoring one (Google Analytics, twitter cards, Open Graph, whatever, you name it :))

Per #101 we will not change blog design until higher volume of posts or activity warrants it.


getNodeAutoInstrumentations type error when initializing tracing in TypeScript

Consider the following express app with tracing initialized from docs/readme:
index.ts:
import express from 'express';

const app: express.Application = express();

const port: number = 3000;

app.get('/', (_req, _res) => {
    _res.send("TypeScript With Express");
});

app.listen(port, () => {
    console.log(`TypeScript with Express
         http://localhost:${port}/`);
});

tracing.ts:
'use strict';

import { NodeSDK } from '@opentelemetry/sdk-node';
import { getNodeAutoInstrumentations } from '@opentelemetry/auto-instrumentations-node';
import { ConsoleSpanExporter } from '@opentelemetry/sdk-trace-base';

const traceExporter = new ConsoleSpanExporter();

const sdk = new NodeSDK({
  traceExporter,
  instrumentations: [getNodeAutoInstrumentations()]
});

sdk.start()

And requiring tracing.ts in the npm manifest: "start": "nodemon -r ./tracing.ts index.ts",
a type error is present:
Type 'Instrumentation[]' is not assignable to type 'InstrumentationOption'.
  Type 'Instrumentation[]' is not assignable to type '(typeof InstrumentationBase)[]'.
    Property 'prototype' is missing in type 'Instrumentation' but required in type 'typeof InstrumentationBase'

At face value, type types appear as though they would be compatible. If I change the getNodeAutoInstrumentations() line to cast to any, then everything works as expected.
If I remove the array brackets around and change it to instrumentations: getNodeAutoAutoInstrumentations(), I get a much gnarlier type error:

gnarlier type error
Type 'Instrumentation[]' is not assignable to type 'InstrumentationOption[]'.
  Type 'Instrumentation' is not assignable to type 'InstrumentationOption'.
    Type 'import("/Users/pcarter/scratch/otel-ts/node_modules/@opentelemetry/instrumentation/build/src/types").Instrumentation' is not assignable to type 'import("/Users/pcarter/scratch/otel-ts/node_modules/@opentelemetry/sdk-node/node_modules/@opentelemetry/instrumentation/build/src/types").Instrumentation'.
      Types of property 'setMeterProvider' are incompatible.
        Type '(meterProvider: import("/Users/pcarter/scratch/otel-ts/node_modules/@opentelemetry/api-metrics/build/src/types/MeterProvider").MeterProvider) => void' is not assignable to type '(meterProvider: import("/Users/pcarter/scratch/otel-ts/node_modules/@opentelemetry/sdk-metrics-base/node_modules/@opentelemetry/api-metrics/build/src/types/MeterProvider").MeterProvider) => void'.
          Types of parameters 'meterProvider' and 'meterProvider' are incompatible.
            Type 'import("/Users/pcarter/scratch/otel-ts/node_modules/@opentelemetry/sdk-metrics-base/node_modules/@opentelemetry/api-metrics/build/src/types/MeterProvider").MeterProvider' is not assignable to type 'import("/Users/pcarter/scratch/otel-ts/node_modules/@opentelemetry/api-metrics/build/src/types/MeterProvider").MeterProvider'.
              The types of 'getMeter(...).createObservableGauge' are incompatible between these types.
                Type '(name: string, callback: ObservableCallback, options?: MetricOptions | undefined) => void' is not assignable to type '(name: string, options?: MetricOptions | undefined, callback?: ((observableResult: ObservableResult) => void) | undefined) => ObservableBase'.
                  Types of parameters 'callback' and 'options' are incompatible.
                    Type 'MetricOptions | undefined' is not assignable to type 'ObservableCallback'.
                      Type 'undefined' is not assignable to type 'ObservableCallback'.

I'm not familiar with the otel-js codebase and its recent changes, so I wouldn't know where to look from here.
What version of OpenTelemetry are you using?
"@opentelemetry/auto-instrumentations-node": "^0.28.0"
"@opentelemetry/exporter-trace-otlp-proto": "^0.28.0"
"@opentelemetry/sdk-node": "^0.28.0"

What version of Node are you using?
v16.13.0

I'm quite sure this is because no release from @opentelemetry/auto-instrumentations-node was made after the 1.2.0/0.28.0 release from core SDK components.
As a result you get a mix of newer and older components.
If you want to use instrumentations form contrib it's most likely needed to stick on SDK 1.0.0/0.27.0 until a new release is done here.

Thanks @Flarna, that would make sense. I have an older app where this doesn't reproduce.
Are there circumstances where it's not a good idea to do instrumentation package releases after a core package? I'm not familiar with the release strategy here.
(also, would this issue be better in the contrib repo? Apologies if this is in the wrong place)

The work to get contrib modules released is ongoing. There are quite some changes needed to update dependencies and keep all the various npm modules happy. An the number of people working on this is not that high.

(also, would this issue be better in the contrib repo? Apologies if this is in the wrong place)

Yes and contrib repo would be better in this case but as the people involved are mostly the same it's not that cirical.

Thanks for the explanation @Flarna, I appreciate it! I'll let some folks who ran into this know for now, and keep my eyes peeled for the next release.

The getting started guide as-written should work with the older package versions.

OTel contrib pacakges were released yesterday but something went wrong and upload to NPM is missing, see https://github.com/open-telemetry/opentelemetry-js-contrib/pull/947

This is now happening again with the latest 0.29.0 release:
src/opentelemetry.ts:59:9 - error TS2322: Type 'Instrumentation[]' is not assignable to type 'InstrumentationOption'.

  Type 'Instrumentation[]' is not assignable to type '(typeof InstrumentationBase)[]'.

    Property 'prototype' is missing in type 'Instrumentation' but required in type 'typeof InstrumentationBase'.

 59         getNodeAutoInstrumentations({

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

 60           '@opentelemetry/instrumentation-http': {

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

...

 62           },

    ~~~~~~~~~~~~

 63         }),

    ~~~~~~~~~~

Found 1 error(s).

Yes, unfortunately.
Work to update contrib components is ongoing, see https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1042
I fear such compatibility problems will persist as long metrics and instrumentations are in experimental phase.

I believe it should be working with:
"@opentelemetry/auto-instrumentations-node": "^0.31.0"
"@opentelemetry/exporter-trace-otlp-proto": "^0.29.2"
"@opentelemetry/sdk-node": "^0.29.2"

@cartermp can you confirm the above versions work for you?

Works for me.


fix(model-server): make specifying a key ID optional

If no static key ID is configured, then the key ID specified in the JWT is used to verify the token. The key is looked up at the JWK URL with the key ID from the token.
This is a fix because it restores functionality that accidentally broke in version 8.14.0.
To be verified by reviewers

[ ] Relevant public API members have been documented
[ ] Documentation related to this PR is complete

[ ] Boundary conditions are documented
[ ] Exceptions are documented
[ ] Nullability is documented if used

[ ] Touched existing code has been extended with documentation if missing
[ ] Code is readable
[ ] New features and fixed bugs are covered by tests

The build is broken because of new bug in Redocly.
See https://github.com/Redocly/redocly-cli/pull/1717

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


TypeError: fromisoformat: argument must be str

Describe the bug
I'm testing this out for use. I'm getting the below error running on ubuntu-latest GitHub runner.
Traceback (most recent call last):
File "/action/workspace/evergreen.py", line 361, in
main()  # pragma: no cover
^^^^^^
File "/action/workspace/evergreen.py", line 83, in main
if is_repo_created_date_before(repo.created_at, created_after_date):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/action/workspace/evergreen.py", line 164, in is_repo_created_date_before
repo_created_at_date = datetime.fromisoformat(repo_created_at).replace(tzinfo=None)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: fromisoformat: argument must be str
To Reproduce

Copied the basic example workflow from readme
Updated 'Run evergreen action' step to the below

      - name: Run evergreen action
        uses: github/evergreen@v1
        env:
          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          REPOSITORY: "myorg/myrepo"
          FILTER_VISIBILITY: "private,internal"

Run workflow manually through GitHub Actions page

Expected behavior
As state in readme, it open issue/PR in the specified repo.
Screenshots
No response
Additional context
I did adding env CREATED_AFTER_DATE and hardcoded "2020-01-01" as value. I got the same error.
I did see the closed issue https://github.com/github/evergreen/issues/82, and the error appears to committed code from the pr https://github.com/github/evergreen/pull/83 that closed that issue.

@stevexiong-hennepin thank you for filing this. I can take a look shortly unless you want to take a try at a PR?

Fix is to add check for created_after_date to L83 of evergreen.py.
if created_after_date and is_repo_created_date_before(repo.created_at, created_after_date):
    continue

I'm afk currently but will get a PR up with a test for it and I'll do a release after it merges.

@stevexiong-hennepin I was unable to recreate the issue with the CREATED_AFTER_DATE env variable set to "2020-01-01".  I've added explicit tests in the PR above.  Can you please let me know if there was something else set?  Thank you in advance.

@stevexiong-hennepin https://github.com/github/evergreen/releases/tag/v1.9.2

Testing some more, it seems like this error only appear when you specify REPOSITORY rather than ORGANIZATION.
I have not tried this with setting ORANIZATION and DRY_RUN: true as I'd like to test it out on a single repository before I create PR for a large amount of repositories within the organization.


feat: add health endpoint

Fixes #13

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


Improve bevy_ecs::system module docs

This includes a lot of single line comments where either saying more wasn't helpful or due to me not knowing enough about things yet to be able to go more indepth. Proofreading is very much welcome.

Very solid documentation work! Accurate, concise, and with helpful examples. I have no comments, lets merge this!

bors r+


Revoke namespace change

This PR revokes the namespace changes done before QEDbase.jl 0.2 was released.
Currently based on https://github.com/QEDjl-project/QEDcore.jl/pull/25, I will rebase once that is merged.

I've also deleted the ProcessSetup tests since fixing them started taking too long and they are deprecated anyways.

General question since we will now often implement interfaces defined somewhere else (i.e. in QEDbase): I think for clarity it would be a good idea to always use function QEDbase.interface_function() ... end directly, because it makes it clear right there that it's implementing an interface. The other option is having an import QEDbase: interface_function at the start of the file. This, in my opinion, kind of hides the fact that when we have function interface_function() ... end later, it is implementing an interface from QEDbase.
I've done it using QEDbase.interface_function mostly, but not everywhere. If we want to make that a guideline I will go through this pr again and do it in the remaining places.
Thoughts?

General question since we will now often implement interfaces defined somewhere else (i.e. in QEDbase): I think for clarity it would be a good idea to always use function QEDbase.interface_function() ... end directly, because it makes it clear right there that it's implementing an interface. The other option is having an import QEDbase: interface_function at the start of the file. This, in my opinion, kind of hides the fact that when we have function interface_function() ... end later, it is implementing an interface from QEDbase.
I've done it using QEDbase.interface_function mostly, but not everywhere. If we want to make that a guideline I will go through this pr again and do it in the remaining places.
Thoughts?

As I mentioned somewhere else, I think QEDbase.interface_function is favorable, because of the reasons you already mentioned. Therefore, I agree with making this the guideline.

@AntonReinhard is there a reason, that this is still a draft?

@AntonReinhard is there a reason, that this is still a draft?

Yes, as I mentioned I will go through again and remove the import QEDbase: ... occurrences. Then I'll mark it ready.


FakeJohannesburg is not 53-qubits

The PR #3226 introduced FakeJohannesburg. However, the 53-qubits promise in the docstring is not fulfilled in the coupling map.

This is just a docstring typo (likely a copy and paste error). Johannesburg is a 20q device.


UnrecoverableUnpackingException is hidden by DefaultBuildCacheController

We have a special exception for when we fail to clean up the outputs after a failed unpack operation. This exception is supposed to signal a build failure to the CacheStep. However, it is caught and wrapped in a GradleException by the DefaultBuildCacheController, so it is never actually handled correctly.

It appears that the recovery from a failed unpack is botched in multiple places.
Here in ResolveIncrementalChangesTaskExecuter we call context.isOutputRemovedBeforeExecution() But that field in the context is only set much later, in CacheStep, when we are actually trying and failing to load something from cache.
This essentially means that we don't disable incremental execution after a failed unpack.

#8783 removes UnrecoverableUnpackingException—problems with cleaning up after a failed load from cache are not handled in a special way anymore. Instead, they simply bubble up to top and make the build fail as any other unexpected error would.

I didn't look deeper, but doesn't that mean that any local build cache issue fails the build? Wouldn't it be nicer if we continued until a certain number of errors when we can?


[PIE-1415] Bump to 1.4.2

Add jasmine and mocha reporters to collector and bump version to 1.4.2
This relates to PIE-1406, with work from https://github.com/buildkite/test-collector-javascript/pull/45

Diff: https://github.com/buildkite/test-collector-javascript/compare/v1.4.1...v1.4.2
Linear Ticket

Thanks @nprizal, added now


Unexpected behaviour mixing options "null" and "from" in modify/3 migration

Environment

Elixir version: Elixir 1.13.1 (compiled with Erlang/OTP 24)
Database and version: PostgreSQL 13.1
Ecto version: 3.7.1
Database adapter and version: postgrex 0.16.1
Operating system: 5.10.3-arch1-1

Current behavior
Modifying whether or not the column accepts null values using the :from option in migrations is not reversible. Although it works and you can actually use it inside change/0 and call mix ecto.rollback without crashing, the field doesn't rollback the nullable option at the db.
def change do
  alter table("account") do
    modify :email, :citext, null: false, from: :text
  end
end

At first I thought this was working as the docs only mentions it "cannot be used to modify primary keys", but turns out it also doesn't work with the option :null.
I did not test with other options (default, size, precision, scale) though.
Expected behavior
Hopefully I'd expect this to rollback nullable to true, but if this is too tricky maybe these two options shouldn't be allowed to be used together at all since it's not possible to rollback: forcing who's writing the migration to write ups and downs manually if they are modifying the option :null with :from.
I'm pretty sure one of these could be possible but if it's not, maybe the docs text should be updated to something like:

Note :from cannot be used to modify primary keys and neither be used with other options, as those are generally trickier to make reversible.

Hopefully I'd expect this to rollback nullable to true, but if this is too tricky maybe these two options shouldn't be allowed to be used together at all since it's not possible to rollback:

I think the issue is that we can't really know if the table was nullable before. null: false would not allow nils but it doesn't mean they were allowed before!
I think maybe we should support something like from: {:text, null: true} and then we use both the type and options to revert. WDYT?


Does anyone have an older version of the SDK (Intel RealSense SDK 2016 R2)?

I tried the official link "http://registrationcenter-download.intel.com/akdlm/irc_nas/vcp/9078/intel_rs_sdk_offline_package_<IP_ADDRESS>6.exe" but it doesn't work anymore and always fails during the download process
I want to use it to drive SR300 camera in Windows 10

Hi @cchengtien  If the 2016 R2 link is not working for you then you can use the much more modern RealSense SDK 2.50.0 which dates from November 2021.  2.50.0 is the final version of this modern SDK that fully supports SR300.
https://github.com/IntelRealSense/librealsense/releases/tag/v2.50.0
You can obtain the full SDK for Windows 10 by downloading and running the file Intel.RealSense.SDK-WIN10 from the 'Assets' file list, or simply download and run the RealSense Viewer tool (Intel.RealSense.Viewer.exe) if you just want to access the camera's streams without doing any programming.

In fact, I wanted to use its old internal 3D scanning feature and see how it worked.

At my location at least, it is possible to launch the 2016 R2 download if you click the link to begin the download and then refresh the web-page that the download is taking place on.  The download should then begin.

Whilst the original download page for the 2016 SDK is no longer online, an archived version of it is available on the Wayback Machine website.  In the 'Downloads' section at the bottom of the page there are download links for R2 and also the R3 version.
https://web.archive.org/web/20180719232831/https://software.intel.com/en-us/realsense-sdk-windows-eol
With the R3 version, instead of a 1.8 gb download it is a set of smaller downloads, with a core Essentials module that you install first and then optional modules, such as 3D Scan, so you can install only the features you need.
The download links require the filling in of a form first before the download begins and I do not know if all the download links still work.

Great! it works
I really appreciate your help

You are very welcome.  It's great to hear that it worked for you.  :)


Ethan: Watch protonmail.com

Ethan requests the watch of the watch_keyword protonmail\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

protonmail\.com has been seen in 108 true positives, 11 false positives, and 6 NAAs.

Rejected by Ollie in Charcoal HQ. Reason: 'Proton Mail is a legit service used for non-spamming purposes (including by some CHQ members).  Use (?:@protonmail.com)?.'