Refactored OCMF powermeter transaction handling

Describe your changes
Refactored OCMF powermeter transaction handling:

Powermeter interface now contains proper user identification arguments (IS, IT, IF, IT, ID) to start the transaction
Refactored EvseManager and LemDCBM400600 modules to address the changed types
Added module config params for LemDCBM400600 : cable_id, tariff_id
Fixed test cases in LemDCBM400600 due to changes made

Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation
[x] I read the contribution documentation and made sure that my changes meet its requirements

Missing some documentation about why for ex. cable_id and tariff_id have been removed from the powermeter type and are now only local config options for the LEM powermeter

The cable_id is no OCMF parameter but it is specific to the LEM powermeter and shall therefore not be an argument within the TransactionReq type. A tariff_id is also no OCMF parameter (but Tariff Text (TT) is) and in the LEM communication protocol it is unclear how this parameter shall be set. This might require further changes in the LEM module, but it doesnt belong to the TransactionReq type.

Missing some documentation about why for ex. cable_id and tariff_id have been removed from the powermeter type and are now only local config options for the LEM powermeter

The cable_id is no OCMF parameter but it is specific to the LEM powermeter and shall therefore not be an argument within the TransactionReq type. A tariff_id is also no OCMF parameter (but Tariff Text (TT) is) and in the LEM communication protocol it is unclear how this parameter shall be set. This might require further changes in the LEM module, but it doesnt belong to the TransactionReq type.

That is exactly what I would like to see in the commit or PR message, so that we're able to reason about why those changes happened.

@Pietfried The MR description explains the "what" happened, but not the "why", please extend, i.e. I have a similar question than @a-w50.

Hi @SirVer , this response to @a-w50 request explains "why" the changes are required: https://github.com/EVerest/everest-core/pull/640#issuecomment-2067176131 .
Please have a look at the Rust build, which is currently failing. It looks like minItems and maxItems keywords are not supported in the rust part.

Hi @SirVer , this response to @a-w50 request explains "why" the changes are required: https://github.com/EVerest/everest-core/pull/640#issuecomment-2066376028 .

Yes, I saw that @Pietfried. My statement was more that this should go into the MR description, because it is important context for posterity and should not be buried in the discussion.

@Pietfried The MR description explains the "what" happened, but not the "why", please extend, i.e. I have a similar question than @a-w50.

Exactly, that's what I was trying to say.


Reports are not updated after last policy removal

I have a CA policy that produces audit-report on the namespace.
When I delete this policy audit report on namespace is not removed / retriggered, leaving behind clusterpolicyreport.
To reproduce:

install Kubewarden with recommended policies
create safe-labels policy with following settings

apiVersion: policies.kubewarden.io.v1
kind: ClusterAdmissionPolicy
metadata:
  name: safelabels
spec:
  backgroundAudit: true
  policyServer: default
  module: ghcr.io/kubewarden/policies/safe-labels:v0.1.14
  rules:
    - apiGroups:
        - ''
      apiVersions:
        - ''
      resources:
        - 'namespaces'
      operations:
        - CREATE
        - UPDATE
  settings:
    constrained_labels: {}
    denied_labels:
      - qqq

add denied label to default namespace kubectl label ns default qqq=1
trigger audit scan (at this point clusterpolicyreport for default namespace with 1 FAIL is created as expected)
delete safelabels policy and trigger audit scan

Current state:
Clusterpolicyreport for default namespace with 1 FAIL is not removed, UI points to non-existent policy
Expexted state:
Clusterpolicyreport results from non-existent policies are cleaned

To whoever takes that, there must be something wrong with owner references and cluster admission policies.


Annotate warnings

Supported only in pytest >= 6.0.0 (docs)
Adds an --exclude-warnings option to exclude warning annotations (perhaps --exclude-warning-annotations is a better name?)

Yeah, I think --exclude-warning-annotations or something else specific to this plugin only would be better.

Yeah, I think --exclude-warning-annotations or something else specific to this plugin only would be better.

Done https://github.com/pytest-dev/pytest-github-actions-annotate-failures/pull/68/commits/494dca7bc1dc895ee300f9613350806fd73f2738!

FYI, we need to hide our warnings to get though CI. Mostly on pytest 6.

Rebase post #86?

Rebased.
In a separate PR, I'd like to add test coverage or other means of detecting dead code after we dropped support for Pytest < 6.

That sounds good, though IMO top priority should be fixing the line calculation for pytest 7.4+. :)

Thanks!

That sounds good, though IMO top priority should be fixing the line calculation for pytest 7.4+. :)

I'll take a stab.

When I attempted it last, I had this:
diff --git a/pytest_github_actions_annotate_failures/plugin.py b/pytest_github_actions_annotate_failures/plugin.py
index 8cf1e62..c635c0c 100644
--- a/pytest_github_actions_annotate_failures/plugin.py
+++ b/pytest_github_actions_annotate_failures/plugin.py
@@ -7,7 +7,7 @@ from collections import OrderedDict
 from typing import TYPE_CHECKING

 import pytest
-from _pytest._code.code import ExceptionRepr
+from _pytest._code.code import ExceptionRepr, ExceptionChainRepr

 if TYPE_CHECKING:
     from _pytest.nodes import Item
@@ -23,6 +23,28 @@ if TYPE_CHECKING:
 # https://github.com/pytest-dev/pytest/blob/master/src/_pytest/terminal.py

+def compute_path(filesystempath):
+    runpath = os.environ.get("PYTEST_RUN_PATH")
+    if runpath:
+        filesystempath = os.path.join(runpath, filesystempath)
+
+    # try to convert to absolute path in GitHub Actions
+    workspace = os.environ.get("GITHUB_WORKSPACE")
+    if workspace:
+        full_path = os.path.abspath(filesystempath)
+        try:
+            rel_path = os.path.relpath(full_path, workspace)
+        except ValueError:
+            # os.path.relpath() will raise ValueError on Windows
+            # when full_path and workspace have different mount points.
+            # https://github.com/utgwkk/pytest-github-actions-annotate-failures/issues/20
+            rel_path = filesystempath
+        if not rel_path.startswith(".."):
+            filesystempath = rel_path
+
+    return filesystempath
+
+
 @pytest.hookimpl(tryfirst=True, hookwrapper=True)
 def pytest_runtest_makereport(item: Item, call):  # noqa: ARG001
     # execute all other hooks to obtain the report object
@@ -38,24 +60,6 @@ def pytest_runtest_makereport(item: Item, call):  # noqa: ARG001
         # collect information to be annotated
         filesystempath, lineno, _ = report.location

-        runpath = os.environ.get("PYTEST_RUN_PATH")
-        if runpath:
-            filesystempath = os.path.join(runpath, filesystempath)
-
-        # try to convert to absolute path in GitHub Actions
-        workspace = os.environ.get("GITHUB_WORKSPACE")
-        if workspace:
-            full_path = os.path.abspath(filesystempath)
-            try:
-                rel_path = os.path.relpath(full_path, workspace)
-            except ValueError:
-                # os.path.relpath() will raise ValueError on Windows
-                # when full_path and workspace have different mount points.
-                # https://github.com/utgwkk/pytest-github-actions-annotate-failures/issues/20
-                rel_path = filesystempath
-            if not rel_path.startswith(".."):
-                filesystempath = rel_path
-
         if lineno is not None:
             # 0-index to 1-index
             lineno += 1
@@ -65,22 +69,24 @@ def pytest_runtest_makereport(item: Item, call):  # noqa: ARG001

         # get the error message and line number from the actual error
         if isinstance(report.longrepr, ExceptionRepr):
+            tb_entries = report.longrepr.reprtraceback.reprentries
             if report.longrepr.reprcrash is not None:
                 longrepr += "\n\n" + report.longrepr.reprcrash.message
-            tb_entries = report.longrepr.reprtraceback.reprentries
-            if len(tb_entries) > 1 and tb_entries[0].reprfileloc is not None:
+            if tb_entries and tb_entries[0].reprfileloc is not None:
                 # Handle third-party exceptions
                 lineno = tb_entries[0].reprfileloc.lineno
+                filesystempath = tb_entries[0].reprfileloc.path
             elif report.longrepr.reprcrash is not None:
                 lineno = report.longrepr.reprcrash.lineno
+                filesystempath = report.longrepr.reprcrash.path
         elif isinstance(report.longrepr, tuple):
-            _, lineno, message = report.longrepr
+            filesystempath, lineno, message = report.longrepr
             longrepr += "\n\n" + message
         elif isinstance(report.longrepr, str):
             longrepr += "\n\n" + report.longrepr

         print(
-            _error_workflow_command(filesystempath, lineno, longrepr), file=sys.stderr
+            _error_workflow_command(compute_path(filesystempath), lineno, longrepr), file=sys.stderr
         )

But I still had a couple of failures. Maybe that's helpful though.


Bert qa

Adding example for BERT SQUAD with onnx export.

Done with code changes. Working on Readme changes @bfineran  can you give the code another look through for comments/thoughts? README.md will have to wait until all the experiments finish.

Fixed issue with transformers. Ready for merge.


info: exchange_blotter: order has not reached the trigger at current price

Dear Catalyst Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: ubuntu 14.04
Python Version: 2.7
Python Bitness: ``
How did you install Catalyst:pip
Python packages: $ pip freeze

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue
During backtest I noticed a lot of my buy orders are not executing and catalyst logs  ...order has not reached the trigger at current price. I suspect this is to do with my limit orders but I can't seem to get rid of it.  Can you explain this behavior and how to set my orders up so they execute?

What did you expect to happen?
Orders execute
What happened instead?

catalyst issues above warning and never executes the orders
Here is how you can reproduce this issue on your machine:
I have not idea what's the cause so can't tell you how to accurately reproduce other than:
Reproduction Steps

select a pair on bitterex
run a backtest over a period of time
use  order_target_percent(asset=asset,                                    target=1,                                   limit_price=price * 1.05)
...

What steps have you taken to resolve this already?
increased limit_price
...
Anything else?
...
Sincerely,
demontamer

I believe that this is a duplicate of #183. We fixed in the develop branch but did not release yet. We'll release in the next day or two.

We're releasing this tomorrow.

In release 0.5.2


If div in html - html to quill won't render anything

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
flutter_quill: ^9.5.12 quill_html_converter: ^9.5.12 flutter_quill_extensions: ^9.5.12
Steps to reproduce

Init the controller : final QuillController _controller = QuillController.basic();
Init the document controller :

_controller.document = Document.fromDelta(
        Document.fromHtml('
Name : Test
')
            .toDelta());

log(_controller.document.isEmpty().toString());
Result: true

If we remove the div from the HTML it works fine. (but I need the divs)
Expected results

Actual results

Code sample
Code sample
final QuillController _controller = QuillController.basic(
    configurations: QuillControllerConfigurations(),
  );

  @override
  void initState() {
    _controller.document = Document.fromDelta(
      Document.fromHtml('
Name : Test
').toDelta(),
    );
    _controller.readOnly = false;

    log(_controller.document.toPlainText().toString());

    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return QuillEditor.basic(
      configurations: QuillEditorConfigurations(
        controller: _controller,
        enableScribble: false,
        enableInteractiveSelection: false,
        sharedConfigurations: QuillSharedConfigurations(
          locale: Locale(Localizations.localeOf(context).languageCode),
        ),
        embedBuilders: kIsWeb
            ? FlutterQuillEmbeds.editorWebBuilders()
            : FlutterQuillEmbeds.editorBuilders(),
      ),
    );
  }

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

@HeropolisDa2ny this isn't directly related with the package. The issue comes from flutter_quill_delta_from_html that not support
 tags. At this moment this is a opened issue related with  and html wrapped within it issue

I already make a PR to resolve this


Sync develop with support/3.0.x

Last merge from support/3.0.x to develop.

@ojii @mkoistinen @evildmp are you ok to merge this?

Wow, great job on getting all this to merge. I'm sure it was a pain. +1 from me!

nice

Way less painful than anticipated

gonna be honest, this is too big to properly review for me but I want this merged! we still have some time to fix develop should this break stuff.

@ojii more a request for an ack on the way to merge this, not a code review which is kind of impossible :D


Private Repository access

I am trying to setup test containers to pull from my private repository.  As per the documentation I have
In the file testcontainers.properties the following
    ryuk.container.image = privaterepo/testcontainers/ryuk:0.3.0
    tinyimage.container.image = privaterepo/alpine:3.12

I can see it attempting to pull these images.  The error I get on a pull is 404... I suspect this is due to security...  error is:
   com.github.dockerjava.api.exception.NotFoundException: Status 404: {"message":"No such image:
   privaterepo/testcontainers/ryuk:0.3.0"}`

On that note, I have set the following in docker-java.properties
registry.url=*****
registry.username=*****
registry.password=*****

this is a spring boot application running spock... so I put the these configuration properties in the path at src/test/resources/...
It appears that the docker-java.properties are not picked up... and/or i have something wrong...thank you!!

you can ignore... issue was with version of testcontainers... upgraded to 1.15.1


progress indicator

Add the progress indicator to keep users informed about the current process status.
[new]

[old]

The "new" version is nice for the eyeballs, but appears too not be the reality of what is happening. Rather prints a UI that is orthogonal to the progress and expected output.
The "old" version could likely use some improved verbiage, but represents reality a little better.
This is a good step, but we should iterate further.


Add an option for deleting fields that contain empty string values in jsonData

When jsonData.endpoint is an empty string, we were experiencing bugs outlined in this Twinmaker issue.  The bug could also be fixed in Twinmaker itself but I wonder if there are other places in which having an empty string endpoint would cause unexpected behaviors.
To reproduce, follow the steps here and you should no longer see the error.

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

I see some places in Twinmaker that we check for the endpoint string, but can you point me to the piece code this bug is caused by? I wonder if we're just not checking for truthy values properly there, since I would normally expect if something is an empty string, that it doesn't get evaluated at all.

I see some places in Twinmaker that we check for the endpoint string, but can you point me to the piece code this bug is caused by? I wonder if we're just not checking for truthy values properly there, since I would normally expect if something is an empty string, that it doesn't get evaluated at all.

It actual looks like the error is somewhere deep in a dependency of the scene viewer panel, where new URL('') is being called and resulting in Uncaught TypeError: Failed to construct 'URL': Invalid URL.
To clarify, when I mentioned that we could fix this in Twinmaker itself, I was thinking more along the lines of adding a useEffect in ConfigEditor.tsx to check if the value was an empty string, but didn't actually see a place in which we're not checking truthy values in the Twinmaker code, but that's a good point, maybe this can be fixed in SceneViewer.tsx, I'll take a look :). Regardless, I do wish we weren't passing along empty strings as values in the jsonData when the field is empty

will likely close this pr in favor of https://github.com/grafana/grafana-iot-twinmaker-app/pull/208


can i remove data from this github?

https://github.blog/2016-02-01-working-with-submodules/

maybe also do this with evaluation folder


Add update and delete views to the tournaments webpages

We have implemented the update and delete views for the tournaments (issues https://github.com/uchicago-cs/chigame/issues/151 , https://github.com/uchicago-cs/chigame/issues/87 ). Currently, only staff members are allowed to edit most of the information about a tournament and delete a tournament. This would be changed later to allow non-admin users to access the functionalities with restrictions (issue https://github.com/uchicago-cs/chigame/issues/172 ).

The html file has been updated. Thanks!

The pull request has been closed.
Code: Mike, Sam, Joshua.
Review: Rafael.

Issue Score: Excellent
Comments:
Good work on this PR! Like my approval, these pages are simple and very helpful for interacting with the tournaments feature. Mentioned it in our meeting, but just wanted to reiterate that PRs should only be opened once your code is fully ready. Further commits should mostly just be readdressing review changes and any testing errors.


Modify unit test to make it configurable

add 3 variables (docNumber, parentNumber, childNumber) which indicates the documents to create during unit test, and set a relatively small number by default instead of the previous 1000 cause in some cases it takes too long to create a lot of indexes.

To add 3 variables looks good to me.
Please remove elasticsearch-reindexing.iml
To remove (Settings) does not work on Eclipse. Could you keep existing test codes other than adding 3 variables?

Thanks for your response and I've modified it as you mentioned, please re-test it.


Heap corruption when calling _storage.resize inside CellCache::reset

Hi, i'm having my script crash due to error 0xC0000374 (Heap Corruption) returned when executing _storage.resize() inside CellCache::reset.
This happens both with the current release and latest git (compiled with vs2019).
In debug mode i was not able to trigger the exception, i don't know if it's because in debug mode doesn't happens or if it was to slow to test more than a couple of frame.
I've attached the last crash dump
vsedit.exe.4152.dmp.zip

Thanks for your report. I’m going to check this. Also it would be great and very helpful if you could post a minimal script causing the crash.

Thanks. It should be fixed in commit 3509f2fed6cbf732344cbc5d2cf1752b54f682c4 and in the r2 release.

Thank you for the fix, sadly i can't try it right now as i'm in the middle of an encode and can't update the plugin to try the new version, i'll let you know ASAP

Ok, it doesn't crash anymore, thanks :)


Feat: Replace file-sharer dependency for file-sharer-es in order to reduce bundle size

Describe the Feature
Using capacitor-filesharer in an Angular application produces the following output during the building process:
<EMAIL_ADDRESS>depends on 'file-saver'. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

This is because there is a dependency on file-sharer which uses the CommonJS syntax.
Platform(s) Support Requested

Android
iOS
Web

Describe Preferred Solution
Replace the dependency of file-sharer to file-sharer-es in order to use ES modules syntax.
Describe Alternatives
None
Related Code
None
Additional Context
None

I tried it with file-saver-es but it broke the build. I'm happy to accept a PR but please test it as I have no time to do it.

As no PR was made I'm closing this. Feel free to provide a PR.


Add ability to use own font for term2 tabs

I'm using custom symbols in my terminal statusline, and I'd like to tell this package what font to use.
Or atleast use the same font which the editor uses.

+1

Yes, please!

:+1:  Yep..  same as editor would be just fine to start.

:+1: Same font as editor default would be ideal.

:+1:


Error Convert_Civitai_OpenVINO

I tried Convert_Civitai_OpenVINO.py and got many warnings and an error.
D:\Program Files\Python310\lib\site-packages\diffusers\models\unet_2d_condition.py:915: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if dim % default_overall_up_factor != 0:
D:\Program Files\Python310\lib\site-packages\diffusers\models\downsampling.py:135: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
assert hidden_states.shape[1] == self.channels
D:\Program Files\Python310\lib\site-packages\diffusers\models\downsampling.py:144: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
assert hidden_states.shape[1] == self.channels
D:\Program Files\Python310\lib\site-packages\diffusers\models\upsampling.py:149: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
assert hidden_states.shape[1] == self.channels
D:\Program Files\Python310\lib\site-packages\diffusers\models\upsampling.py:165: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if hidden_states.shape[0] >= 64:
D:\Program Files\Python310\lib\site-packages\diffusers\models\unet_2d_condition.py:1206: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
if not return_dict:
Unet successfully converted to ONNX
Traceback (most recent call last):
File "D:\astable-diffusion\openvino_notebooks\notebooks\225-stable-diffusion-text-to-image\t5\civitai.py", line 87, in
model = mo.convert_model(UNET_ONNX_PATH, compress_to_fp16=True)
File "D:\Program Files\Python310\lib\site-packages\openvino\tools\mo\convert.py", line 364, in convert_model
ov_model, _ = _convert(cli_parser, framework, params, True)
File "D:\Program Files\Python310\lib\site-packages\openvino\tools\mo\convert_impl.py", line 957, in _convert
raise e.with_traceback(None)
RuntimeError: Check 'false' failed at src\frontends\common\src\frontend.cpp:54:
Converting input model
bad allocation
torch: 2.1.2+cpu
onnx: 1.15.0
onnxruntime: 1.16.3
tensorflow-intel: 2.14.0
Intel Core i5-6500 CPU
I found https://github.com/openvinotoolkit/openvino/issues/12747 that tensorflow-intel >=2.8.0 will cause unexpected error.
I tried to downgrade tensorflow-intel to a version tensorflow-intel==2.8.0, but it asked me to downgrade other packages like
Other possible workarounds are:

Downgrade the protobuf package to 3.20.x or lower.
Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

Since you made it working, could you create virtualenv with needed packages?

It would be good to have the next improvements:
1)virtualenv
2)negative prompt
3)possibility to run in Intel GPU (iGPU)
Let me know if you want to make any tests or get some donations. I'm very interested in making this program working.

@homevk15
Let's remove TensorFlow dependencies since the model was successfully converted to ONNX format. We can disregard the warnings for now. The issue seems to be with converting the ONNX model to OpenVINO format. As a workaround, you can comment out the conversion line in the code and instead use the Model Optimizer tool to do the conversion:
mo --input_model unet/unet.onnx --compress_to_fp16
About the negative prompt, it can be easily added. Something like that:
        bs_embed, seq_len, _ = uncond_embeddings.shape
        uncond_embeddings = uncond_embeddings.view(bs_embed, seq_len, -1)
        text_embeddings = torch.cat([uncond_embeddings, text_embeddings])

I don't have iGPU, so can't test the code on it.

After updating tensorflow-intel to tensorflow-intel-2.14.1 I got success.
I got  unet, text_encoder, vae_decoder, vae_encoder (.xml and .bin files).
The program didn't generate an image
I see only
Loading pipeline components...: 100%
without errors.
Do you know what to change to get an image in an output?
Also, please, update Convert_Civitai_OpenVINO with the code of negative prompt you posted. I'm new to this stuff. Thank you.

Yes, this code is intended only to convert a model. It doesn't generate images. For that you should use https://github.com/bes-dev/stable_diffusion.openvino

The last question how opset calculated?
It's opset = 16 for 512x512 images?


Added check for "name" channelMetadata attribute

Breaking Change:
None
Description:
Add  a check for "name" channelMetadata attribute, This attribute will be used as the default channel name to send to the media_player when no channel number is found. This attribute gets filled when the user says Alexa, tune to "<channelname>"
Related issue (if applicable): fixes #29274
`
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[ ] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Thanks for this. I assume we also need a test case in: https://github.com/home-assistant/home-assistant/blob/dev/tests/components/alexa/test_smart_home.py

I did not add a test case for such a small change but will if the powers
that be suggest I do :)
On Sat, Nov 30, 2019 at 10:21 PM Bruno Filipe<EMAIL_ADDRESS>wrote:

Thanks for this. I assume we also need a test case in:
https://github.com/home-assistant/home-assistant/blob/dev/tests/components/alexa/test_smart_home.py
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/29279?email_source=notifications&email_token=ABW4E7PHU34A3GLE5U34ANLQWMUU3A5CNFSM4JTJPLSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFQ2JEI#issuecomment-560047249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABW4E7OD7HB27L3A4OMHMXLQWMUU3ANCNFSM4JTJPLSA
.

I will need to add the test.  It looks for it. Ok, I will add it and re-pull
On Sat, Nov 30, 2019 at 10:29 PM Alain Turbide<EMAIL_ADDRESS>wrote:

I did not add a test case for such a small change but will if the powers
that be suggest I do :)
On Sat, Nov 30, 2019 at 10:21 PM Bruno Filipe<EMAIL_ADDRESS>wrote:

Thanks for this. I assume we also need a test case in:
https://github.com/home-assistant/home-assistant/blob/dev/tests/components/alexa/test_smart_home.py
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/29279?email_source=notifications&email_token=ABW4E7PHU34A3GLE5U34ANLQWMUU3A5CNFSM4JTJPLSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFQ2JEI#issuecomment-560047249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABW4E7OD7HB27L3A4OMHMXLQWMUU3ANCNFSM4JTJPLSA
.

Looks good 👍


setup frontend (vue and bootstrap)

vue frontend installed via jhipster, but will be changed and will be installed via vuex in near future


fix: Fix XRootD storage plugin at least for file access

This PR updates the storage plugin to allow read/write file access with XRootD, borrowing heavily from the XRootD remote provider in Snakemake<8 to allow remote input/output files.
It should also be possible to support StorageObjectGlob but I would have to think about it a bit more (or of course someone else would also be more than welcome to have a go!).
I tested this with the latest Snakemake (commit e8735c14 from a local editable install with Python 3.12.4) and could successfully:

Use a remote file as an input with retrieve=False
Use a remote file as an input with retrieve=True
Use a remote file as an output
Use remote files as inputs/outputs as above including wildcards in the file paths/URLs

Summary by CodeRabbit

New Features

Introduced a Snakemake storage plugin for reading and writing via the XRootD protocol, enhancing data handling in computational workflows.
Added improved error handling and retry logic for XRootD operations, increasing robustness during file interactions.
Implemented new URL management features, allowing for more flexible URL construction and validation.
Expanded documentation to clarify authentication options and the use of URL decorators.

Bug Fixes

Enhanced error management for file operations, ensuring issues are logged and handled appropriately.

Documentation

Updated documentation to reflect new plugin functionalities and usage guidelines, including details on input and output specifications.

@johanneskoester in the end I put back the options for specifying the host/port/username/password as optional arguments so that it can be used with "fully-formed" URLs or simpler ones which specify some of the information via the storage provider as they could still be useful (and not that much extra effort to include them).
I did have one question however but I was unsure if this is just because I missed something or if it is the expected behaviour:
When I specify a host with the provider, if I then use retrieve=False the input is passed "as-is" rather than the parsed URL with the host in. For example:
storage:
    provider="xrootd",
    retrieve=False,
    host="eosuser.cern.ch"

rule some_rule:
    input: storage.xrootd("root://eos/user/m/mmonk/some_file.root")
    output: "test.root"
    shell: "python some_script.py {input} {output}"

I can see internally it correctly renders the URL to "root://eosuser.cern.ch//eos/user/m/mmonk/some_file.root" to check that it exists but then passes "root://eos/user/m/mmonk/some_file.root" as the input to the script instead of the rendered one.
Using retrieve=True it picks up the right URL and is able to download it so it's only a problem with retrieve=False but I was unsure if that is what it is snakemake is supposed to do or I've just missed something somewhere.
Sorry if that's a bit of an unclear question, hopefully that makes sense!

Good point. I did not foresee that there might be the requirement to return a modified URL when retrieve is set to false. I now think about how to best implement that.

Btw. do you want to become a maintainer of this plugin? You are certainly much better suited than I am.

@MattMonk I have added a skeleton for modifying the query. This uses the new method postprocess_query from StorageProviderBase. You can simply modify the query URL there as needed and then return it. That should solve exactly the use case that you have here.

Aha fantastic! Many thanks for putting the postprocess_query together on the snakemake side --  I'll get the skeleton filled out.
Yeah I'd be more than happy to be added as a maintainer for this, thanks!

One more thing I've realised is that if a password is supplied this will of course end up in the root url like root://my_username@my_password:eosuser.cern.ch//eos/some_file which is then printed in the snakemake output in plaintext...
I guess then if we want to keep this option some change would have to be made so that when snakemake prints out the input/output files when it runs the rule to use the censored version instead.
I'm not sure how much work that would be, possibly for now we could drop support for using a password with this plugin and add it back later or possibly not at all if it's a bit too much of a risk having something that puts your password in plaintext in the URL it uses to access the file? I think using the password field in the URL is not the usual way of authenticating remote files with it anyway

Yes, good point. On the other hand, this will mostly be used in trusted environments. My suggestion is to keep that functionality, but issue a warning with logger.warning() and also put a warning into the help text of the password setting.

Okay yeah that makes sense! I'll do that and then I think otherwise this is ready to go from my side


lynis: 2.5.7 -> 2.6.2

Semi-automatic update. These checks were performed:

built on NixOS
ran /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2/bin/lynis -V and found version 2.6.2
ran /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2/bin/lynis --version and found version 2.6.2
ran /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2/bin/.lynis-wrapped -V and found version 2.6.2
ran /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2/bin/.lynis-wrapped --version and found version 2.6.2
found 2.6.2 with grep in /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2
found 2.6.2 in filename of file in /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2

cc "@ryneeverett"

Success on x86_64-darwin (full log)
Partial log (click to expand)
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
strip is /nix/store/4sdh09gmvl15cy0zb6i7mbvxh5syz206-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/qxbxxjs2kah9d9kkz78igcj4wjiax0i6-lynis-2.6.2/bin
patching script interpreter paths in /nix/store/qxbxxjs2kah9d9kkz78igcj4wjiax0i6-lynis-2.6.2
/nix/store/qxbxxjs2kah9d9kkz78igcj4wjiax0i6-lynis-2.6.2/bin/.lynis-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/sh"
/nix/store/qxbxxjs2kah9d9kkz78igcj4wjiax0i6-lynis-2.6.2

Success on x86_64-linux (full log)
Partial log (click to expand)
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2/bin
patching script interpreter paths in /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2
/nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2/bin/.lynis-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/sh"
checking for references to /tmp/nix-build-lynis-2.6.2.drv-0 in /nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2...
/nix/store/xspm1nc8dznagjc1rnpg2rjxlf88ivkc-lynis-2.6.2

Success on aarch64-linux (full log)
Partial log (click to expand)
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/3p7sgkdkgh481fsd6k9qm8ms4gb0r70b-lynis-2.6.2
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/3p7sgkdkgh481fsd6k9qm8ms4gb0r70b-lynis-2.6.2/bin
patching script interpreter paths in /nix/store/3p7sgkdkgh481fsd6k9qm8ms4gb0r70b-lynis-2.6.2
/nix/store/3p7sgkdkgh481fsd6k9qm8ms4gb0r70b-lynis-2.6.2/bin/.lynis-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/sh"
checking for references to /build in /nix/store/3p7sgkdkgh481fsd6k9qm8ms4gb0r70b-lynis-2.6.2...
/nix/store/3p7sgkdkgh481fsd6k9qm8ms4gb0r70b-lynis-2.6.2


DD_HTTP_PROXY for Optimized Datadog Lambda Extension

Hello,
It looks like that currently the DD_HTTP_PROXY and DD_HTTPS_PROXY variables are not implemented and fallback to the current version of the extension. Is it plan to add it ?

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hey @jozolr - thanks for reaching out!
It's on our list of priorities as we examine which features we need to support to move to GA. I'll leave this ticket open until we add support.
Thanks again!
AJ

Hi @jozolr - in reading/experimenting it seems like DD_HTTPS_PROXY and DD_HTTP_PROXY are only used for requests made from the agent for things like our HTTP checks/healthchecks
In Lambda (as far as I can tell with our main agent) only support HTTPS_PROXY. Is that in line with your expectation? Are you relying on DD_HTTPS_PROXY for something other than proxying data to datadog from a function within a private subnet in a VPC?
Thanks!

Hello @astuyve, we are only relying on DD_HTTPS_PROXY to proxy data to datadog currently, we also use HTTPS_PROXY but for other usages

Hi @jozolr - I see, thanks for that. I think in Lambda we generally prefer HTTPS_PROXY which is used globally (docs), because the libraries we're using already support that.
Unless there's a use case where you'd require a separate proxy? Could you share your config and general use case?
Thanks!
AJ

Also - is there a chance we're conflating/confusing something? (Probably on my end!) From the docs, I don't believe DD_HTTP_PROXY is used by us at all, instead it's rather DD_PROXY_HTTPS:

I quickly tested it with our main agent (not next-gen) in Lambda and wasn't able to make it work. If you could share the names of these environment variables in your setup that would be most helpful. It may make sense to communicate via email<EMAIL_ADDRESS>is fine for me if you'd link.
Thanks!
AJ

You right it's a typo from my part, i'm actually talking about DD_PROXY_HTTP and DD_PROXY_HTTPS

Right gotchya, but are they set to different proxies? What happens if you remove DD_PROXY_HTTPS? I'm just trying to understand your specific use case and dimensionality of the config levels here, and hoping to maybe simplify things for Lambda.
Thanks!

If I remove it the agent is not able to connect and send the traces to Datadog. I dont want to use HTTPS_PROXY because there are internal calls that shouldnt use the proxy.

Thanks @jozolr that's helpful, I was looking for a shortcut which didn't exist. We'll be sure to honor your config in the next release!

Hey @jozolr,
AJ is in OOO, so I'm giving you an update in his place. This should be available in the latest v66 release. Feel free to try it out and let me know if you have any issues!
I'll close this for now, but feel free to re-open!


Cannot set any parameters in config.json

Dear Sir,
Thank you for this great software ! Very useful... I try to change some options, but it seem my config.json is never read, when I launch npm run start:dev, I've got my good config.json in "dist" directory, but the option don't seem to be used
Any help ? Thank you a lot !
Here is an example for my config.json file :
{
"download": {
"duration": 30
}
}

Hello @Shadebzh,
You are right. I just figured the documentation is way out of date with latest set of configurable options.
I'll try to address that as soon as possible.

Hello @Shadebzh,
You are right. I just figured the documentation is way out of date with latest set of configurable options.
I'll try to address that as soon as possible.

Thank you a lot !

Thank you a lot !

You can find all configurable options in this file:
https://github.com/e7d/speedtest/blob/master/packages/web/src/js/worker/config.js
Be aware that the final configuration is mage by combining the result of the different methods. You would need to be able to understand a bit of JavaScript.
I also reckon that there is a problem with the current state of the configuration handling.
Despite being able to configure a lot of download/upload options, some of them are overridden by the front end, through the presets. You can find them by clicking the cog on the top right corner.
As long as you don't change the preset you are fine. But as soon as you set one, even the "normal" one, a combination of parameters are overridden, including the download duration you are interested on. That's probably why you would not see your change on config.json being reflected.
That is definitely an issue i would need to address.

You can find all configurable options in this file:
https://github.com/e7d/speedtest/blob/master/packages/web/src/js/worker/config.js
Be aware that the final configuration is mage by combining the result of the different methods. You would need to be able to understand a bit of JavaScript.
I also reckon that there is a problem with the current state of the configuration handling.
Despite being able to configure a lot of download/upload options, some of them are overridden by the front end, through the presets. You can find them by clicking the cog on the top right corner.
As long as you don't change the preset you are fine. But as soon as you set one, even the "normal" one, a combination of parameters are overridden, including the download duration you are interested on. That's probably why you would not see your change on config.json being reflected.
That is definitely an issue i would need to address.

Thank you again a lot for your answer.
In fact, I try to make (force) download and upload request via websocket rather than XHR (due to XHR number concurrent limit in browser).
I have noticed the parameter endpoint.websocket with ws://, get parameters in config.js and also noticed you make a GET request on config.json in xhr, but when I try to make this :+1:
{
"endpoints": [
{
"label": "Websocket1",
"uri": "ws://<IP_ADDRESS>:5080"
}
]
}
Is there any wat to do this ? Last question I promise ;-) Thank you.

Thank you again a lot for your answer.
In fact, I try to make (force) download and upload request via websocket rather than XHR (due to XHR number concurrent limit in browser).
I have noticed the parameter endpoint.websocket with ws://, get parameters in config.js and also noticed you make a GET request on config.json in xhr, but when I try to make this :+1:
{
"endpoints": [
{
"label": "Websocket1",
"uri": "ws://<IP_ADDRESS>:5080"
}
]
}
Is there any wat to do this ? Last question I promise ;-) Thank you.

Hi again @Shadebzh
The endpoints key only goal is to setup the list of external endpoints usable from a defined speedtest instance.
In this section, you would only reference other speedtest websites, through their respective HTTP-based URIs.
What you seem to want to achieve is to switch from XHR-based bandwidth tests to WebSocket-based ones.
It was possible in past versions, but not anymore. It was kind of working, but the behavior was really inconsistent from an environment to another. Long story short, Chrome was kind of OK on any OS, other browsers/OS combinations lead regularly to strange or unpredictable behaviors.
My project having in mind to be as much all-terrain as possible, i decided to lock out this option, as long as I can't find a satisfying enough solution.
Regards.


Failed to execute 'send' on 'WebSocket': Still in CONNECTIN state

i have encounter an error "Failed to execute 'send' on 'WebSocket': Still in CONNECTIN state". when using session mode from nkn-sdk-js.
it looks like websocket connection is down but send() function is still pushing data to the network.

It's more like info/warning as those data will be resent with other websocket connections. Did you get those errors constantly or just when session is created?

Closing due to inactivity. Feel free to continue the discussion or reopen it.


Please update prometheus chart to use prometheus operator

This is a feature request
https://github.com/coreos/prometheus-operator

cc: @mgoodness

linking this issue with my explanation on how I'll work on it https://github.com/kubernetes/charts/pull/3490#issuecomment-363899621

/remove-lifecycle stale.

/remove-lifecycle stale


Manager dashboard does not show CQL health issue

While Prometheus reporting one of the nodes as status -1

The dashboard shows 0 nodes as "No CQL Connection"

Look like the problem is using cql_status instead of scylla_manager_healthcheck_cql_status

@mmatczuk FYI


Add Jdbc Select

Coverage decreased (-5.2%) to 80.222% when pulling 9cf4e83412d1677451619812825c3c6e30dcf167 on JuHyun419:feature/64_jdbc_interface into 51068998bb36aba7b07ebe298785f6cfcb3f5afa on ztkmkoo:develop.

@JuHyun419
Hi there,
It looks some codes was conflicted. Could you resolve it?

@JuHyun419
Hi there,
It looks some codes was conflicted. Could you resolve it?

@JuHyun419
Hi there,
It looks some codes was conflicted. Could you resolve it?

Okay~

@JuHyun419
Hi there,
It looks some codes was conflicted. Could you resolve it?

Okay~


Add logs correlation to guice-servlet-grpc example

This gives users a simple example for how to wire logs and traces together so that log entries can display inline with trace spans in the Stackdriver Trace Viewer.

@bogdandrutu could you take a look at this when you get a chance?


RedisHashAsyncCommands hmget should return Map, not List

I'm using 4.0Beta2
The hmget return RedisFuture<List>, better to be RedisFuture<Map<K, V>>

Thanks for the feedback. The reason for the current design is the redis response structure:
<IP_ADDRESS>:6379> hmset key f1 v1 f2 v2
OK
<IP_ADDRESS>:6379> hgetall key
1) "f1"
2) "v1"
3) "f2"
4) "v2"
<IP_ADDRESS>:6379> hmget key f1 f2
1) "v1"
2) "v2"

HMGET does not return the keys within the answer and currently this is the only way (except streaming) to retrieve hash values in the order they were inserted. All Map results use a HashMap to transport the data.
What is your motivation for the change request?

If redis is designed that way, I can deal with the result myself, thanks.

Thanks for your fast response. So nothing to do here.


Make PR checks for Python 3.12 required

Actual Situation
Currently, all PR checks for Python 3.12 are optional. See any open PR and notice the missing Required badges for those checks.
Desired Situation
Since Python 3.12 has been released and is officially supported by Copier, those checks should be required as well.
Proposed solution
AFAIK, this is a GitHub project setting that only @yajo can change.

Thanks! Done

Thanks! :+1:


feat: Add urls for Arcjet fly.io deployments

@davidmytton Is this what you were thinking?

Exactly 😎
Let's hold merging this whilst we finish the rollout of the API to Fly: https://github.com/arcjet/arcjet/issues/1741


Ep #2 When making an Invalid Syntax returns None

So lets say that when I type "5 +" for example on episode to at the 17:04 timestamp instead of giving the error it will print
"(INT:5, PLUS, None)" my code is here: https://github.com/penguinpolarDOTnet/Penguin-Bullet---Programming-Language

Ok so I fixed it on line 186 of my code (Line 214 of his code) i wrote just "return" I was supposed to write "return self" so yea


How to install mish-cuda when cuda is 11.1?

my GPU is RTX 3090,so I have to use cuda 11, I already checked my cuda11.1 and it is useful,
but when I use  : pip install git+https://github.com/thomasbrandon/mish-cuda/
to insall mish-cuda, there is a error:
unable to execute ':/usr/local/cuda/bin/nvcc': No such file or directory
error: command ':/usr/local/cuda/bin/nvcc' failed with exit status 1
ERROR: Failed building wheel for mish-cuda
what should I do?

That colon shouldn't be there. Looks like it's part of the CUDA path detected by torch.utils.cpp_extension (which does the compilation). Check if CUDA_HOME environment variable is set and verify value. Otherwise check which nvcc results. Those are the methods used for detection. Otherwise you'll have to look at the detection/compilation logic to see what's going wrong. See torch/utils/cpp_extension.py#L27.

I have been running on cuda11.1 using pytorch1.6 but the compiled file fails with pytorch1.7 which officially supports cuda11.
I am getting a function import failure with an unrecognized character from the compiled library so I downgraded back to pytorch 1.6...

@rafale77 Did you re-install the extension after upgrading PyTorch. I wouldn't expect binary compatibility across versions so you need to re-install to re-compile.

Yes, I did, I tested it both ways: without recompiling, and with recompiling on pytorch 1.7, Same failure.
Compiled by pytorch 1.6 and 1.7, both work fine on pytorch 1.6.

I change my cuda to 11.0, then set environment variable  export CUDA_HOME=/usr/local/cuda
new error appeared：
nvcc fatal : Unsupported gpu architecture 'compute_86'
error: command '/usr/local/cuda/bin/nvcc' failed with exit status 1
so I add  environment variable：export TORCH_CUDA_ARCH_LIST="7.5" downgrade version
it works!

I change my cuda to 11.0, then set environment variable  export CUDA_HOME=/usr/local/cuda
new error appeared：
nvcc fatal : Unsupported gpu architecture 'compute_86'
error: command '/usr/local/cuda/bin/nvcc' failed with exit status 1
so I add  environment variable：export TORCH_CUDA_ARCH_LIST="7.5" downgrade version
it works!

hi, I tried: -export CUDA_HOME=$CUDA_HOME:/usr/local/cuda +export CUDA_HOME=/usr/local/cuda it works
It really works bro！


How do you ignore formatting attributes?

I'm trying to figure out how to stop csharpier from formatting a bunch of attributes. I tried what's shown below, but I understand that doesn't work because cshaprier-ignore-start only applies when "A range is valid around statements and members".
How should I do it?
Also, it would be great if chsarpier supported #pragma warning disable format. I seem to need that anyway to stop VS from removing the whitespace
Input:
    [Theory]
    // csharpier-ignore-start
#pragma warning disable format
    //          start,       end,         yAxisRef       expectedRef
    [InlineData(0, 0,  0,    1, 0,  0,     0,  0,  1,    "Z")]
    [InlineData(0, 0,  0,    1, 0,  0,     0,  1,  0,    "Y")]
    [InlineData(0, 0,  0,    0, 1,  0,     1,  0,  0,    "X")]
    [InlineData(0, 0,  0,    1, 0,  0,     0,  0, -1,    "-Z")]
    [InlineData(0, 0,  0,    1, 0,  0,     0, -1,  0,    "-Y")]
    [InlineData(0, 0,  0,    0, 1,  0,    -1,  0,  0,    "-X")]
    [InlineData(0, 0,  0,    1, 0,  0,   0.5,  0,  1,    "Z")]
    [InlineData(0, 0,  0,    1, 0,  0,  -0.5,  0,  1,    "Z")]
    [InlineData(0, 0,  0,    1, 0, 10,     0,  0,  1,    "Z")]
    [InlineData(0, 0,  0,    1, 0, 10,    -1,  0,  0,    "Z")]
    [InlineData(0, 0,  0,    1, 0,  0,     0,  1,  1,    "0, 0.707, 0.707")]
    [InlineData(0, 0, 10,    1, 0, 10,     0,  1,  1,    "0, 0.707, 10.707")]
#pragma warning restore format
    // csharpier-ignore-end
    public void ToArc_ReturnsArcMember(

Output:
Extra whitespace is removed.
Expected behavior:
unchanged

CSharpier doesn't have a good way to ignore just a set of attributes. #992 is a duplicate.


Deadlock

I have no idea how and why this is happening but it's happening. Quite frequently now. It happens in different places, in different situations. I don't know an exact sequence of steps to reproduce. Here's a full backtrace: https://gist.github.com/xissburg/a65549d78abbe0a8572f
I am just accessing a PFObject property and the deadlock happens.

Hey @xissburg, thanks for the report. Stack trace here is definetly helpful.
From symbol names in the stack trace - I can see that you are using a quite old version of the SDK.
Any chance you can update to the latest one and retest? (at the time of writing this is 1.9.0).
We had few fixes around deadlocks in few recent releases.

It looks like I will have to upgrade my project to Swift 2 and Xcode 7 first.

I am almost sure everything is still going to work on Swift 1.2 and Xcode 6.4 with the current latest version, but let me know if it doesn't.

I get the same problem described in this issue https://github.com/ParsePlatform/Parse-SDK-iOS-OSX/issues/298.

I'm getting lots of Deadlocking as well. Next time I replicate, I'll post a stacktrace.

Here's the stack trace - https://gist.github.com/kevflynn/3deb9da4628a48e49ec0

Thanks for the trace. Any chance you can post the output from all threads (thread bt all) in the lldb console.

Here you go @richardjrossiii https://gist.github.com/kevflynn/00d04bc8df93c1f9ba79

I'm also seeing similar deadlock issues.  Here is the BT: https://gist.github.com/rudywen/31ec2f54574be20784ab
This issue is blocking our app release.  Is there anything we can do to help debug the issue?

@rudywen That deadlock is actually not directly Parse's fault. What's happening here is that every thread in the default thread pool (usually there's 16 threads in the thread pool for iOS/OSX) is waiting on a task to be run in the same thread pool (in this case, it's attempting to create the PFFIle cache directory if necessary), but because all thread pools are going attempting to wait on the same thing at the same time, there's no dispatch queue available from the system to attempt to create the directory.
You should probably rethink your architecture if you're trying to concurrently download so many parse files, or use your own network/caching stack using file.url, as the architecture in Parse is not designed for so many concurrent downloads.

@richardjrossiii is right on this one, it's purely too many files being downloaded at the same time.
We can optimize for it, but in all sanity - please simply add the file downloads to an operation queue, as downloading ~30 files concurrently is probably not what you want.

@richardjrossiii @nlutsenko Thanks for the suggestions. We'll correct it on our end.

After upgrading to Parse 1.9.0 I haven't experienced this issue again. Thanks for the great job guys!

Glad it helped!


Jobby locks task

#Jobby locks current task from repeating while executing
it seems that Jobby lock the current task while it's still being executed, so if the task is supposed to run every minute and it's running at the moment, the next time it should be executed just won't happen until the one before. There is any workaround for this?

No, this is by design. We don't want multiple instances of the same jobs to overlap, since they usually assume they're running alone. For example, a job that queries a database to get a list of tasks to process. If another job started, and it wasn't written correctly, it might start processing the same task twice.


[Feature] skywalking-ui just a front-end program, use the spring boot package, relying on too many jar increase in tar.gz' size

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
The entire skywalking-webapp.jar size is 34.5m, but public folder after the package, only  963kb, skywalking Ui now just need an HTTP server, don't even need https, but spent 30m provides HTTP services
use light http server, so that almost 30m jar size can be reduced.If it is necessary
I can submit a PR, put executable JAR to the reduced to 3.3m, and then Public package, so that you can get the current 170m as a whole, down to 140m
Use case
reduce skywalking-webapp.jar to 3.3m
Related issues
none
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

To use Spring Gateway to host the website, is in order to make people easier to understand that. What container do you to propose? I think this kind of codebase change needs evaluation and PMC decision.

Spring Gateway is too heavy,act as just a http server,so may be a light http server is a better choice. and it still can make people easier to use it and experience it.

I can get your point. Please provide a complete proposal, such as what new stack you want to use.

I can get your point. Please provide a complete proposal, such as what new stack you want to use.

undertow-core can do,also simple,act as a http server,total jar size is only 3.3m

We set up the standard Gateway stack to provide users opportunity to choose more features if they like.
Also, according to Apache 2.0 License, anyone to set up their own webapp binary and image for more people in the community.
I just say this from the official release perspective, and follow the tradition of the community. If you insist to change this, this would be longer term to make decision. I would like to recommend you try to set up a project to implement this first.

ok,thanks for your great work.
I will fork a branch to implement this first.

I am moving this to discussion. Once we have more progress, such as you have those re-distribution and images, and after more people become to choose webapp-slim package, PMC could begin to discuss the replacement.


TensorFlow: attempt to repair the Windows build

The type generic math needs to be imported through the use of
MSVCRT as that is the equivalent to the Darwin or Glibc modules.
ucrt is the pure C library without any additions.

CC: @saeta @vballoli

Windows Run: https://dev.azure.com/compnerd/swift-build/_build/results?buildId=30744&view=results

The Windows run was clean.

Thanks for the speedy fix @compnerd!  I think I'm the medium term, it's going to be important to ensure we don't need conditional imports for Windows or Android for these sorts of common operations. Do you think you can help put together a list of these issues as they come up so we can get a decent survey of them over time?

@saeta that’s going to be required.  Anytime you use the underlying system’s libc there’s going to be conditional imports.  Furthermore, sqrt is definitely in different modules per platform in the swift implementation (AFIACT intentionally to prevent circular dependencies).  This isn’t a special thing about Windows, it’s how swift chose to implement the core libraries (Darwin, MSVCRT, Glibc depend on swiftCore, which cannot depend on them).
One option would be to implement a new module System that exposes the entire C library interface (import C) and add a second module that exposes the POSIX interfaces (import POSIX).  A third module that should be implemented would implement the support for thread local storage and threading (import Threading).  That’s been something which has been brought up a few times, but no one has spent the time to actually present what such a surface would look like.


Refactor filter query on map page

It should be possible to filter this in the query rather than looping through the dojos and checking stage !== 4
https://github.com/CoderDojo/cp-zen-platform/blob/master/web/public/js/controllers/dojos-map-controller.js#L29

postgres version 0.8.3 now supports:
ne$: .list$({ f1: {ne$: v1} }) for not-equal.
eq$: .list$({ f1: {eq$: v1} }) for equal.
lte$: .list$({ f1: {lte$: 5} }) for less than or equal.
lt$: .list$({ f1: {lt$: 5} }) for less than.
gte$: .list$({ f1: {gte$: 5} }) for greater than or equal.
gt$: .list$({ f1: {gt$: 5} }) for greater than.
in$: .list$({ f1: {in$: [10, 20]} }) for in. in$ operator accepts only values of type array.
nin$: .list$({ f1: {nin$: ['v1', 'v2']} }) for not-in. nin$ operator accepts only values of type array.
Note: you can use any operators described above together.
Another change was that sort$ was fixed. To be consistent with the other seneca store the sort order was reversed.

Version bump to 1.0.0


Error code 1021 Timestamp

I get the following error when running most modules, where can I change the recvWindow property to get this working?
Exception: Api Error Code: -1021 Message: Timestamp for this request was 1000ms ahead of the server's time.

@mdobson8 Please let us know which server you changed it to to aid others having the same issue.

Here you go;
time.nist.gov


feat(dataframe): Column variable support

Pull Request
🤨 Rationale
https://ni.visualstudio.com/DevCentral/_workitems/edit/2566754
👩💻 Implementation

Added variable support to query editor columns field.
Added variable query editor to data frame data source that displays a table id field. This field also supports variables. Implemented metricFindQuery to return the selected tables columns.
Since the data frame query editor and variable query editor shared the majority of their logic, this was split out to a helper class that is accessed by both.

🧪 Testing

Verified functionality locally
Added new tests and existing tests continue to pass

✅ Checklist

[X] This PR has a title that follows the commit message format.

@mure - This should be good to go again. The merge was pretty nasty so make sure you give it another look.

@mure - Submitting this since teams is showing you as OOO and so Josh can start playing with it. If you have responses to those two comments I can address them in a separate PR.

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

v3.9.0
GitHub release

Your semantic-release bot :package::rocket:


coming soon image fixed

Description
There was an issue where the "coming soon" image in the cloud native catalog page, it was occupying  the entire available space, causing the main image to be hidden from view on the mobile device with width under 600px
This PR fixes #1384

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

hey @charu1603
Let's discuss it on the websites call.
Please add this as an agenda item in the meeting minutes, if you would :)
https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.lohhtewfwima

@Ghat0tkach please review my pr

Hey @charu1603, was this PR discussed in any of the website meetings? If not so let's discuss this on website call today at 5:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes

Hey @charu1603  Let's discuss this on website call tomorrow at 5:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes.


npm not completely set up

sorry to keep bugging you about this library, but the package.json is not complete.
if i install npm install svg.filter.js --save and then use require('svg.filter.js') it fails because there is not "main" field in the package.json
thanks :D

Is it just "main":"./dist/svg.filter.js"?

yes that should be it, the svg.js project has it in its package.json and it works fine

ok - that should do it

oh sorry i did not mean set it to dist/svg.js
it has to point to the dist/svg.filter.js or dist/svg.filter.min.js
you dont have to fix it now, im sure your busy so when ever you get a chance.

lol - the copycat failed there -.-


Update types

This PR will update some TypeScript types.

I would like to get your feedback about just converting this project to TypeScript. We can do it progressively like we did with node-saml. Trying to figure out the types and keep them separate from the JavaScript is quite a mental load, and I'm not sure of the benefit of doing so.

No issues with that, I actually prefer TypeScript. If it helps you be more productive on this repo then go for it!


chore: wrapping every call to the server using me throught a isAuthed()

It would probably be ideal if we could have the isAuthed variable as part of the state of the client or added to some kind of middleware so that this behavior is automatic. My concern would be that it would be easy to forget when adding a new query to not add the skip argument.

It would probably be ideal if we could have the isAuthed variable as part of the state of the client or added to some kind of middleware so that this behavior is automatic. My concern would be that it would be easy to forget when adding a new query to not add the skip argument.

agree, I also thought about it. not straightforward because we would need to list extensively (and have a check on the extensivity) on every queries and whether they're in one bucket or another one
currently the main issue is server side. it's returning a 401 error. so it's not a big deal if we forget one: it will appear server wise and we would then fix it.

It would probably be ideal if we could have the isAuthed variable as part of the state of the client or added to some kind of middleware so that this behavior is automatic. My concern would be that it would be easy to forget when adding a new query to not add the skip argument.

agree, I also thought about it. not straightforward because we would need to list extensively (and have a check on the extensivity) on every queries and whether they're in one bucket or another one
currently the main issue is server side. it's returning a 401 error. so it's not a big deal if we forget one: it will appear server wise and we would then fix it.

That makes sense to me.


Feat/accept initial country code and phone number

Allow client code to pass initial data for input fields
Initial Country Code to to detect initial default country
Initial Country phone code that can also be used to detect initial default country
Initial phone number to be passed to the phone number input field.

** This is useful in Edit forms where input field should be pre-populated with values
Thanks for this awesome useful library and I hope to find my contribution useful

Hello @hbmartin, 👋
Conflicts were resolved successfully and things will work well
Thank you for yout time 😇☘


Serializer new optimization

*For v0 serializer optimized calculating embedded field length, by trying to read position of next field and then to subtract that position from position of current field.
*In v1 serializer removed unnecessary calls to deserializeValue function. No need for them because in in v1 header contains info about length of the field.
*Covered more types in deserializeValue when is used just to achieve length of the field

Hi,
this is fine for me, if all test pass is go for me.
Regards


fix(relay): wait for channel binding in smoke test script

webrtc-rs has a race condition where send_to does not actually await the channel binding, thus attempting to send something through the channel from the other end my fail because we receive the bytes from the relay before the library registers that there is an active channel.
This should hopefully fix the flakiness of the smoke test script.

Glad the CI managed to catch this!

It is only in the smoke test script though, not production code :)


"Lync Basic 2013" incorrectly named as "Lync Basic 2013 Trial"

In the "Lync Basic 2013 Trial Software" section, there are 3 instances of incorrect "Package Name".
It is named here as "Trial". But the software is free. There is no evaluation/trial version of it.
The correct name is on Download Center (https://www.microsoft.com/en-us/download/details.aspx?id=35451) - "Microsoft Lync Basic 2013 (32 Bit)" (without word "Trial")

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

ID: cc1999a1-66c4-1236-54e5-6491c8b70c1c
Version Independent ID: 8b423b6d-793b-52ee-0031-1f80ea453a3c
Content: Latest updates for versions of Skype for Business that use Windows Installer (MSI)
Content Source: Skype/SfbServer/sfb-client-updates.md
Product: skype-for-business-itpro
GitHub Login: @jambirk
Microsoft Alias: jambirk

This correction is on its way, thanks for bringing it to us.


Redesign the way multiple items are displayed on a search result card

With the current test data, the way we display multiple items on a single search result card is quite convoluted, see attached picture.
If the current test data is similar to the real data we will have in the end, this should be redesigned.

Would the shortening from #139 help with this already?

Do you need more feedback for this @frwms ?

It should look like as in the screenshot.

https://www.figma.com/file/2P0PFrxgSIXC7L1Hj1IHb1/Platform-Design?node-id=1012%3A4199


refactor(perception_rviz_plugin): only delete markers with unused MarkersIDs during updates.

Description
We have observed issues related to the processing speed of RVIZ (perception plugin) under scenarios with large amounts of detectable objects.
After profiling, we note that PredictedObjectsDisplay clears / re-adds all markers during updates with new messages, and this behavior is related to the increasing processing time.
By only deleting old marker ids not used in the newest frame, and allowing RVIZ to directly modify the old markers instead of re-adding them, this PR should improve the processing efficiency of PredictedObjectsDisplay at scenes with lots of objects.
Related links
Private Profiling Results
Tests performed
The tests are performed on the private ROSbag
Profiling Results:

Selective Delete improved significantly on the time (1e-6 s) consumed in the update function.
Notes for reviewers

Interface changes

This PR will add a public "deleteMarker" method in object_polygon_display_base.hpp.
Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@taikitanaka3  @1222-takeshi Could you approve this?


ion-tab: being able to pass parameter through ui-sref

I try to display some tabs
<ion-tabs>
    <ion-tab ui-sref="home.state1({param1: 'current'})">
        <ion-nav-view name="view1"></ion-nav-view>
    </ion-tab>
    <ion-tab ui-sref="home.state2">
        <ion-nav-view name="view2"></ion-nav-view>
    </ion-tab>
</ion-tabs>

with this router
$stateProvider
.state 'home',
    url: ''
    abstract: true
    templateUrl: 'templates/home.html'
.state 'home.state1',
    url: '/state1/{param1:\\w+}'
    views:
        view1:
            templateUrl: 'templates/view1.html'
            controller: 'View1Ctrl'

But it seems that ui-sref is not evaluated with its parameters
It pass home.state1({param1: 'current'})  to $state.href (from ui-router), instead of home.state1, and thus does not find a matching state.
Am I missing something ?

@drewrygh Can we discuss how to fix this ?
ui-sref is not parsed when passed to ion-tab.

Greetings @QuentinFchx!
My sensors indicate that you have created an issue on our tracker requesting usage help, rather than an actual error with Ionic framework.
This GitHub issue tracker is used to track framework features, problems, and development.
If you wish to get help using the framework itself, the recommended place is the forum. There are many humans there who would be happy to offer sage advice.
Thank you for allowing me to assist you.


Reduce more duplication in the CSS

This patch combines a couple of different CSS rules, thus reducing the overall size of the file.

/botio-linux preview

Thank you!


Add MS MARCO (V2) regressions for docTTTTTquery expansions

Building on #1721 #1730 and https://github.com/castorini/pyserini/pull/939
We are now ready to build regressions for MS MARCO (V2) {doc, passage} with docTTTTTquery expansions.

For doc, @ronakice has prepared the two expanded corpora on orca at:
/store/scratch/rpradeep/msmarco-v2/collections/msmarco_v2_doc_segmented_d2q-t5_10
/store/scratch/rpradeep/msmarco-v2/collections/msmarco_v2_doc_d2q-t5

Since these corpora have already been broken into smaller files and individually gzipped, I've simply copied into their final locations on orca:
$ du -h msmarco_v2_doc_d2q-t5/ msmarco_v2_doc_segmented_d2q-t5/
46G	msmarco_v2_doc_d2q-t5/
73G	msmarco_v2_doc_segmented_d2q-t5/

I've also created tarball at /store/collections/msmarco/tarballs:
$ md5sum msmarco_v2_doc_d2q-t5.tar msmarco_v2_doc_segmented_d2q-t5.tar
cdd8e4823b237d9d4d6e05f7c02c4f26  msmarco_v2_doc_d2q-t5.tar
3eb16c3efc19e834b7ca8c62b9c0ddcc  msmarco_v2_doc_segmented_d2q-t5.tar

For passage, @ronakice has prepared two expanded corpora on orca at:
/store/scratch/rpradeep/msmarco-v2/collections/msmarco_v2_passage_augmented_d2q-t5_20
/store/scratch/rpradeep/msmarco-v2/collections/msmarco_v2_passage_d2q-t5_20

Since these corpora have already been broken into smaller files and individually gzipped, I've simply copied into their final locations on orca under /store/collections/msmarco:
$ du -h msmarco_v2_passage_d2q-t5 msmarco_v2_passage_augmented_d2q-t5
47G	msmarco_v2_passage_d2q-t5
66G	msmarco_v2_passage_augmented_d2q-t5

I've also created tarballs at /store/collections/msmarco/tarballs:
$ md5sum msmarco_v2_passage_d2q-t5.tar msmarco_v2_passage_augmented_d2q-t5.tar
61632bdb3313dc5631c563d650acf6d2  msmarco_v2_passage_d2q-t5.tar
7ce979309caeeb0a28dd1d79f24851b6  msmarco_v2_passage_augmented_d2q-t5.tar


Clarification on moving VNet gateways with Standard PIPs across resource groups

Good afternoon,
I would like to get some clarification regarding moving VNet gateways attached to a Standard Public IP to a new Resource Group in the same subscription.
This article lists that moving a VNet gateway and PIP to a new resource group, same subscription and region is supported, however, when you attempt to do so you get the following error from the PIP resource.
(Code: ResourceMoveProviderValidationFailed) Cannot move one or more resources in the request. Please check details for information about each resource. (Code: CannotMoveResource, Target: Microsoft.Network/virtualNetworkGateways) Move for resource type publicIPAddresses is not supported. Move request contains resource /subscriptions/34c97c61-xxxx-42fb-afef-xxxxxxxxxx/resourceGroups/xxxxxx/providers/Microsoft.Network/publicIPAddresses/xxxxxxxx-vgw-pip of that type. (Code: MoveNotSupported)",
"code": "ResourceMoveProviderValidationFailed",
"status": 409
The gateway in question is a VPNGW1 SKU with a Standard Public IP address attached. All the networking objects (VNet, gateway, NSGs, connections, local gateways) are being moved in the same move operation.
Neither this nor the linked Networking document list any limitations for moving public IP addresses or VNet gateways across resource groups in the same subscription / region. The articles specifically list cross subscription / region limitations, but not limitations within the same subscription and region. Our CSP provider reproduced the error in a lab, so it seems to be a general problem.
Please can you clarify whether this should be a supported move operation?

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

ID: 23df5354-6833-a5da-0eb3-d860150c8b78
Version Independent ID: 682ec206-0fda-3e0b-5841-543860a87e34
Content: Move operation support by resource type - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/move-support-resources.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@ssjaronx4
Thanks for your feedback! We will investigate and update as appropriate.

@ssjaronx4, thank you for the question.
@tfitzmac, could you please review this question and share your thoughts. Thank you

The article about moving networking resources states:
When moving a resource, you must also move its dependent networking resources. However, any resource that is associated with a Standard SKU public IP address can't be moved across subscriptions. For example, you can't move a VPN Gateway that is associated with a Standard SKU public IP address to a new subscription.
It sounds like that is similar to your scenario. I'll close this issue for now, but if you think it needs further addressing, reopen it.
#please-close

The article about moving networking resources states:
When moving a resource, you must also move its dependent networking resources. However, any resource that is associated with a Standard SKU public IP address can't be moved across subscriptions. For example, you can't move a VPN Gateway that is associated with a Standard SKU public IP address to a new subscription.
It sounds like that is similar to your scenario. I'll close this issue for now, but if you think it needs further addressing, reopen it. #please-close

Yes, but the article states this limitation only when moving between subscriptions, it doesn't address moving it between resource groups in the same subscription and region. This makes the reader think there is no limitation when moving inside the same subscription, but when trying to move it you get the error described by @ssjaronx4.
So there is a limitation when moving inside the same subscription, but there is nothing in the docs that implies this behavior


node bot.js

I've followed all steps exactly as explained but when I run node bot.js this is the error message I get. Help![]

Is auth.json in the same location as bot.js? if not, you'll need to move them to the same location.

They are in the same folder


Time range limitation?

It's possible to limitate a range only for time slection? For example only from 8:00 AM to 6:00 PM.
Thanks !

+1 for this feature

+1

+1 please

Any update on this?


Zombie tests that can't be deleted

We've run into an issue where deleted tests keep reappearing after some time, even when there are no participation/conversion calls being made with those test names.
cc @sbeal @paulabeth

Is it possible someone is running an old branch that includes the participation code?

Possible, but pretty unlikely since the code was removed a while ago. The tests also fail to load, which makes me suspicious that there's more to it. There are also numerous erroring tests at the bottom with seemingly randomly generated names related to the original test name.


Multiple h1 tags on page

This isn't as much a bear cub thing as much as it is HTML/CSS thing that bear cub uses: Currently the page title and the website/blog title is formatted with h1.
I believe it's considered a bad practice to use multiple h1 tags outside of separate section tags. Bear Blog makes the same mistake.
I was only able to give it little thought how to improve the accessibility situation further; wouldn't it be better to preserve the h1 tag for the actual content and format the page title with CSS? Or, perhaps even better, should we use a section tag for the header and another section tag for the content?

This is a good point. However, since this theme is trying to replicate Bear Blog, I'm not sure it's worth it to fix this specific problem. Changing the HTML in this way might hinder backwards compatibility and that sort of stuff...
Thanks for the suggestion, though!


Post release automated changes for network releases

Post release automated changes for azure-arm-network

@qiaozha Please push changes to this PR to resolve the merge conflicts

close this one in favor of #17515


Editing Starting Lineup from Admin Screen

Im wouldn't say this is a serious issue, just want to raise awareness on it for debugging. When using Microsoft Edge, I cannot get the roster to popup in the edit lineup screen to add players to positions.

When using Firefox, I can get some success in getting it to populate, but even still, sometimes it takes a little smacking upside the head to load.

Not sure if this is a me issue or a site/code issue, just wanted to bring it up.

I’ve noticed that also - usually changing the week will get it to populate.
On Mon, Sep 17, 2018 at 1:58 PM bryarwagner<EMAIL_ADDRESS>wrote:

Im wouldn't say this is a serious issue, just want to raise awareness on
it for debugging. When using Microsoft Edge, I cannot get the roster to
popup in the edit lineup screen to add players to positions.
[image: edge]
https://user-images.githubusercontent.com/33761663/45640805-9ea59780-ba81-11e8-8673-84875b45146b.png
When using Firefox, I can get some success in getting it to populate, but
even still, sometimes it takes a little smacking upside the head to load.
[image: firefox]
https://user-images.githubusercontent.com/33761663/45640803-9cdbd400-ba81-11e8-9700-a294ace07862.png
Not sure if this is a me issue or a site/code issue, just wanted to bring
it up.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nickfrerichs/fflproject/issues/53, or mute the thread
https://github.com/notifications/unsubscribe-auth/ARwi2y2WgI_KMV0HpPZ-_EI0U3_nzzSZks5ub-LDgaJpZM4Wsav4
.


load dtdls together

What
Why
Tested

@joebeernink Do you know why if I load multiple DTDL files (Willow base and Airport), the model parser fails? I was trying to locate the detailed error file, but coudn't...

It's likely that it's because airport is a superset that includes all of the other Willow classes, so we're probably getting duplicates. I think we should just be able to load Airport for now... assuming the Airport submodule is up to date.
I suspect this is going to fall apart pretty quickly, and we're going to have to make some changes to the way the nugets are built.


Integrated pressurecooker

Closing for now, as we won't do on-the-fly video compression on CC server until BBQ is integrated.
Other pieces are already in #141.


New languages Integration

Integrated new languages sanskrit, Bhojpuri and urdu
Finetuned the other 12 languages which are present

conflict problem


Add periods to diagnostic messages

Inspired by a catch of missing periods in https://github.com/Microsoft/TypeScript/pull/14133, I went ahead and added periods to diagnositc messages where it seemed appropriate (ie: those that are sentences).

👍


DOT minor changes

Made some changes to the comments in dot.cpp file

Not that it's a big deal for this repo, but I added the noCI label since this is only comment changes.

Not that it's a big deal for this repo, but I added the noCI label since this is only comment changes.

Daine I would think we shouldn't use noCI for doc changes inside the code otherwise formatting and accidental code changes won't get caught.  no?

Hm, maybe. Since the CI group added the static analysis job, I wonder if we're still planning on running format checks for noCI PRs? As for accidental code changes, that's fair, but if this were a rocBLAS job I would still be tempted to use noCI since I think. I suppose it can be up to reviewer's/developers discretion :)

Hey Daine, would it be okay to remove the noci label?  In my initial commit, i did not do a clang formatting to the .cpp file and did a pull request. Then, i figured out i had to do the clang formatting to the .cpp file and tried to add the file  again and i got a reply that it's up to date. Therefore, I think it is okay to merge this .cpp file with noci label. But, Just to be on the safer side i want to run the CI tests too. Is that okay for you?

Thanks Daine


className hash causing problem with jest enzyme snapshot test

I'm using the latest version of styled-components, and the className hash mechanism certainly works beautifully.
However, it is causing some problem for my CI pipeline. I'm using jest, enzyme, enzyme-to-json to do snapshot test to make sure any change made to react components, will not impact the behaviour of the components. New className hashes generated during the test from time to time is breaking the test. Is there a helper library or any solution to strip out classNames in the snapshots?

There is already an issue at https://github.com/styled-components/jest-styled-components/issues/29


Allow the semantic data provider to return semantizer objects

Instead of returning "anonymous" objects, we could add an option to the semantic data provider to return objects defined according to the https://github.com/assemblee-virtuelle/semantizer-typescript library developed by @lecoqlibre.
This would give us greater control over the data we receive.

Alternative option: https://github.com/assemblee-virtuelle/semapps/issues/1191


pip2 install MemoryError

face_recognition version:  b10c119
Python version: 2.7
Operating System: windows 7

I tried the cmd command:
pip2 install face_recognition
before downloading the files here in github and moving to the python2.7 path,
and got a MemoryError once the process reached 99%.
Tried again after doing all of the above and got the exact same results:

I think the issue is the file is too big for your available pip cache memory.
Can you try pip2 --no-cache-dir install face_recognition instead and see if that fixes the issue?

Thanks I tried it and it continued onward!
Though now I am experiencing a different error, despite already downloading cmake.

This issue is not about cmake itself. It's that dlib is failing to compile because it can't find a c++ compiler on your system.
I've never compiled dlib myself on Windows, so I can't really help you unfortunately. You might check the dlib docs at http://dlib.net. It probably involves installing Visual Studio, installing boost-python, and then compiling dlib. Sorry I don't have an easier answer.

I have been having quite alot of trouble trying to fix it from various computers, with help from what you said...Thanks for the help though!
Also could I possibly get dlib to work it's configuration problems and all that via https://github.com/Microsoft/vcpkg
?
I also am having quite alot of trouble getting vcpkg to work too...

Like I said in the project README, I haven't tested this library on Windows at all. I've only tested it on macOS and Linux. I don't have a machine with Windows, so unfortunately I can't help. Sorry about that.

Closing this issue since the original issue in the title is fixed. But I'm sorry I don't have better advice yet for installing on Windows.

@lambaaada Could you open an issue on https://github.com/microsoft/vcpkg with your Dlib issue? We'd be happy to fix it!

@lambaaada Just a thought here: it seems that you already had dlib installed on your system and that it was uninstalled while using pip... As a workaround, do you think it would help (after having reinstalled dlib from where you installed it in the first place) to clone the face_recognition repo, to edit the setup.py file removing dlib from the requirements list, then run python setup.py install in a terminal?


Added execution time, pushed "action output" header to the actual action output.

Merged into #198.


Mute button on audio player flash fallback.

There is a bug with the muted button (not working correctly), within the flash fallback for the audio player.
This change should fix it. [Taken from the VideoElement.as]

Set "_isMuted" before the event is fired, because the javascript event handler, checks "media.muted" to toggle slider position and muted css class.

Cannot provide a new .swf build, so needs to be done.

Diff view messed up.
Check Line 256:  _isMuted = (_volume == 0);


fix header name

When Accept is capitalized it gives me an error.
Header name must be a valid HTTP token ["Аccept"]

This sounds like a bug in some of the layers (the HTTP client library or the Keymanager server). "Accept" is a pretty canonical capitalization of this header name.

It's not a bug, the A was written in Cyrillic by mistake.


https cert fails when "www." is in url

Describe the bug
When going to, https://www.edhtop16.com/ cert fails
but when going to https://edhtop16.com/ cert passes
Consistently reproducible on Mozilla Firefox 117.0.1
Not sure if this server is using a proxy, but it's likely a proxy rule is missing to handle www. urls
Screenshots

fixed


⚠ Bofrost.gr has degraded performance

In f5134f4, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 1912 ms

Resolved: Bofrost.gr performance has improved in c31cb00 after 8 minutes.


Create Micronaut OpenAPI Generator Client Guide

Generate the HTML without running the tests:
./gradlew micronautOpenapiGeneratorClientBuild -x micronautOpenapiGeneratorClientRunTestScript
Open the HTML with open build/dist/index.html
To generate the guides and execute the tests:
./gradlew micronautOpenapiGeneratorClientBuild

CAan you merge master into the PR @andriy-dmytruk ?

@timyates ./gradlew micronautOpenapiGeneratorClientBuild fails for me locally. However, CI pass. Can you investigate? It seems we are running the tests.

@sdelamo Do you have a link to it passing on CI?  I only see the CLA action 🤔

@sdelamo All the tests are failing as they require ${TWITTER_AUTH_CLIENT_ID} to be set in the environment
I can only find tests in the TwitterFrequentWordsControllerTest class, I'm not sure what is generating all the other tests 🤔
Investigating

@sdelamo All the tests are failing as they require ${TWITTER_AUTH_CLIENT_ID} to be set in the environment
I can only find tests in the TwitterFrequentWordsControllerTest class, I'm not sure what is generating all the other tests 🤔
Investigating

I told @andriy-dmytruk in the PR review how to make the tests pass.

@sdelamo Do you have a link to it passing on CI? I only see the CLA action 🤔

That it is true. I did not hover over the button. But it seems only CLA got executed.

@sdelamo I made changes according to the review. Tests should pass now.
Left 2 of the requested changes unresolved, so you can check the callouts that were created (callout-micronaut-test-client and query-value-server).

Seems to be failing with the following exception:
io.micronaut.http.server.exceptions.ServerStartupException: Unable to start Micronaut server on port: 8080 Caused by: java.net.BindException: Address already in use (Possibly because there are too many test). The tests pass for me locally.
@sdelamo I can change the port to port: -1. Maybe it will prevent the problem next time.

I ran this locally with lsof -r 1 -iTCP:8080 to monitor anything opening on port 8080, and I didn't see anything crop up
As this is a PR from a forked repo, could it be that Github is not applying the secrets to the build, and this is causing issues?
It would explain why we're not getting a build scan either?
@sdelamo this is only a guess...  Do you know?
I've restarted the builds btw to see if it passes this time...  if not, I might create a PR based on this one that's not from a fork and see if that works

Closing this in favor of https://github.com/micronaut-projects/micronaut-guides/pull/874


serial: Simplify the echo process

Summary
Impact
code refactor
Testing
sim:nsh

@xiaoxiang781216 the change seems 100%, but please include some basic testing :-D

@xiaoxiang781216 the change seems 100% correct, but please include some basic testing :-D

Test with sim:nsh. thank for suggestion which actually find one bug in my code.


Issue generating schemas via hatch run scripts:generate-schemas

When running hatch run scripts:generate-schemas, I got the following error:

Generating` models for schema: pcr/BENCHLING/2023/09/qpcr.json...
Traceback (most recent call last):
File "/allotropy/scripts/generate_schemas.py", line 10, in
generate_schemas(ROOT_DIR)
File "/allotropy/src/allotropy/allotrope/schema_parser/generate_schemas.py", line 115, in generate_schemas
lint_file(model_path)
File "/allotropy/src/allotropy/allotrope/schema_parser/generate_schemas.py", line 52, in lint_file
subprocess.check_call(
File "/.pyenv/versions/3.11.0/lib/python3.11/subprocess.py", line 413, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'ruff /allotropy/src/allotropy/allotrope/models/pcr_benchling_2023_09_qpcr.py --fix' returned non-zero exit status 1.

Are there any additional steps to follow in order for this to work, besides the ones described in the readme?

Hi Keith.
This error points to a problem when generating the schemas, or a difference in ruff (which shouldn't really happen with hatch)
Do you have any modifications to the json schemas?  If not, I would be very curious to see a branch that runs into this issue.


Implement inline CIDR block egress peer

This PR adds support for implementing inline CIDR
peer blocks.

/test pull-network-policy-api-verify

@tssurya You just need to run gofmt here
Verifying gofmt
diff ./apis/v1alpha1/shared_types.go.orig ./apis/v1alpha1/shared_types.go
--- ./apis/v1alpha1/shared_types.go.orig
+++ ./apis/v1alpha1/shared_types.go
@@ -176,11 +176,11 @@
 	// +optional
 	Nodes *metav1.LabelSelector `json:"nodes,omitempty"`
 	// Networks defines a way to select IPs and CIDR blocks that represent
-    // entities that live outside the cluster as a peer.
+	// entities that live outside the cluster as a peer.
 	// It is the list of NetworkCIDR (both v4 & v6) that can be used to define
 	// external destinations.
-    // You could use this to select internal cluster networks like podCIDR blocks, but it is
-    // recommended to use namespaces and pods peers instead in those cases.
+	// You could use this to select internal cluster networks like podCIDR blocks, but it is
+	// recommended to use namespaces and pods peers instead in those cases.
 	//
 	// Support: Extended
 	//
Please run make fmt to fix the issue(s)
Test FAILED: hack/../hack/verify-gofmt.sh

/retest

/hold
@astoycos : yeah this is actually only WIP PR, adding hold here, its a PoC for the NPEP PR: https://github.com/kubernetes-sigs/network-policy-api/pull/144 which needs to get merged first.
Need your help in getting PR 144 merged asap.

/hold cancel

ack thanks @JoelSpeed : I will double check if we want to do v4 in v6 support and then a change things accordingly. What about isCIDR() implementation, does that support v4 in v6 or will it consider that as invalid?

I will double check but I'm pretty sure when implementing it we excluded 4-in-6 support

I will double check if we want to do v4 in v6 support

Oh, yeah, we don't. (isCIDR is exactly correct for what we want.)

FYI for reviews, this PR's scope is to get the CIDR peer in and I will add a TODO linking to https://github.com/kubernetes-sigs/network-policy-api/issues/203 to clarify the issues around load balancers...


Game will auto-save but won't quick save

Hi, there.
I don't know if this is a UB issue or not but I've suddenly gotten situations where the game won't allow me to quick save at all. The game will still autosave with a new map or combat. Don't know if one of my UB characters or races is causing the problem.
I tried dragging in my .sav file into this window but it said the file type is not supported.
Thanks,
Brooks

Please always read change log before an update. RESPEC was required before update on some subs.

It was happening before the update and I don't have any subs that I was
told to RESPEC in the change log.
On Wed, Sep 6, 2023 at 3:16 AM Paulo Monteiro @.***>
wrote:

Closed #3977
https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3977 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3977#event-10293997588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCKM4YE3AHBSGIS3VR6WKWDXZBEPTANCNFSM6AAAAAA4NCIHFI
.
You are receiving this because you authored the thread.Message ID:
<SolastaMods/SolastaUnfinishedBusiness/issue/3977/issue_event/10293997588@
github.com>


bump pipeline for swaggerapi url fix

Fixes https://github.com/openshift/origin/issues/15823
@openshift/devex

[test]

[test]

mirror flake https://github.com/openshift/jenkins/issues/348

[test]

flake https://github.com/openshift/jenkins/issues/348

[test]

flake https://github.com/openshift/jenkins/issues/348 again
i'll wait until later today, then try again

[test]

flake https://github.com/openshift/jenkins/issues/348 again

going to just try
[merge]
the new plugin is there

[test]

@stevekuznetsov @dobbymoodge  - hey guys, jenkins repo / rosie builds have failing every time on yum repo / mirror issues for the last 2 days.
Is it possible some sort of set up change was made for the origin repo that needs to get made for this one?

@gabemontero I'm not sure we run with onlineint on by default:
failure: repodata/repomd.xml from rhel-7-server-ose-onlineint-rpms: [Errno 256] No more mirrors to try.

@smunilla @jupierce @bparees would know better what is going on here

ah ... thanks for the highlighting @stevekuznetsov
yeah, @bparees reminded me on the way to snack time that the onlineint stuff is used as part of the new method @jupierce put in for building the "super" RPM that contains all the plugins we install in the jenkins image.
Hence, this repo will always need onlineint as part of building the rhel image.  I certainly see refs to it in the xml files for this repo in aos-cd-jobs/sjb/generated.
@jupierce @smunilla @stevekuznetsov - how can we insure  / is it OK to insure that we run with onlineint on by default ?

[merge]