asked to submit error to github by integration

The problem
unknown error occuring with the error asking the user to post the log as an issue on github
What version of this integration (ha-nest-protect) has the issue?
0.4.0b2
What version of Home Assistant Core has the issue?
Frontend 20230705.1 - latest
Device / Model
Topaz-2.7
Diagnostics information
No response
Home Assistant log
Logs
```
`This error originated from a custom integration.
Logger: custom_components.Nest_protect
Source: custom_components/Nest_protect/pynest/client.py:224
Integration: Nest_protect
First occurred: August 18, 2023 at 9:55:29 PM (41 occurrences)
Last logged: 4:57:59 PM
Unknown pynest exception. Please create an issue on GitHub with your logfile. Updates paused for 1 minute.
Traceback (most recent call last):
File "/config/custom_components/Nest_protect/pynest/client.py", line 217, in authenticate
self.nest_session = NestResponse(**nest_response)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: NestResponse.init() got an unexpected keyword argument 'error_description'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/Nest_protect/init.py", line 163, in _async_subscribe_for_data
entry_data.client.nest_session = await entry_data.client.authenticate(auth)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/Nest_protect/pynest/client.py", line 224, in authenticate
raise PynestException(
custom_components.Nest_protect.pynest.exceptions.PynestException: 400 error while authenticating - {"error":"invalid_request","error_description":"missing user credentials"}. Please create an issue on GitHub.
`

</details>

### Additional information

_No response_

I get the same errors on a daily basis, reloading the integration gets it back to running state.
This error originated from a custom integration.
Logger: custom_components.nest_protect
Source: custom_components/nest_protect/pynest/client.py:224
Integration: Nest Protect (documentation, issues)
First occurred: August 24, 2023 at 13:26:44 (52 occurrences)
Last logged: 07:26:55
Unknown pynest exception. Please create an issue on GitHub with your logfile. Updates paused for 1 minute.
Traceback (most recent call last):
File "/config/custom_components/nest_protect/pynest/client.py", line 217, in authenticate
self.nest_session = NestResponse(**nest_response)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: NestResponse.init() got an unexpected keyword argument 'error_description'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/nest_protect/init.py", line 163, in _async_subscribe_for_data
entry_data.client.nest_session = await entry_data.client.authenticate(auth)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nest_protect/pynest/client.py", line 224, in authenticate
raise PynestException(
custom_components.nest_protect.pynest.exceptions.PynestException: 400 error while authenticating - {"error":"invalid_request","error_description":"missing user credentials"}. Please create an issue on GitHub.
This error originated from a custom integration.
Logger: custom_components.nest_protect.pynest
Source: custom_components/nest_protect/pynest/client.py:222
Integration: Nest Protect (documentation, issues)
First occurred: August 24, 2023 at 13:26:44 (52 occurrences)
Last logged: 07:26:55
Could not interpret Nest response
This error originated from a custom integration.
Logger: custom_components.nest_protect.pynest
Source: custom_components/nest_protect/pynest/client.py:214
Integration: Nest Protect (documentation, issues)
First occurred: August 24, 2023 at 13:26:44 (52 occurrences)
Last logged: 07:26:55
Authentication error: invalid_request

Which version do you use of the component?

v0.4.0b2

same here "v0.4.0b2"
Communication to the device seems to work even if I get the errors

same here "v0.4.0b2" Communication to the device seems to work even if I get the errors

@Diddz7 If it does not reconnect when you reload the integration you may have to generate new issueToken and cookie and configure the integration again.


builtins.AttributeError: 'DialogueStateTracker' object has no attribute 'trigger_followup_action'

Why am I getting this error if I have a class :
class customRestarted(Event):
    """Conversation should start over & history wiped.
    As a side effect the ``Tracker`` will be reinitialised."""

    type_name = "restart"

    def __hash__(self):
        return hash(32143124312)

    def __eq__(self, other):
        return isinstance(other, Restarted)

    def __str__(self):
        return "customRestarted()"

    def as_story_string(self):
        return self.type_name

    def apply_to(self, tracker):
        #from rasa_core.actions.action import ACTION_LISTEN_NAME
        for slot in tracker.slots:
            if slot == 'username':
                username = tracker.get_slot(slot)
            elif slot == 'useremail':
                useremail = tracker.get_slot(slot)

        tracker._reset()
        tracker._set_slot('username', username)
        tracker._set_slot('useremail', useremail)
        tracker.trigger_followup_action("action_listen")

(i want to restart the tracker but keep with this two slots, the event is just a copy of Restarted(event) in events)
EDIT
apparently in my Rasa core version the function is defined as trigger_follow_up_action instead of trigger_followup_action. Sorry, i'll close this issue.

Hi, is your code working?

Hi Can this be implemented in rasa-sdk?


When refresh,server logs "The local repository is dirty"

version:   spring-cloud-config-server  :1.3.0.Release
when is refresh the config,the server use git to merge local and remote git commits,but i encourt some warn logs,such as:
1.The local repository is dirty. Resetting it to origin
How does it occured;
Following,use git reset command to discard local changes,but there also has some warn logs,such as:
1.Could not reset to remote for [label] (current ref=[ref])
As a result:  The refresh change keys is empty!!!
can some help me solve these problem;

Can you please try using Edgware or Finchley?

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


[CI] Recommendation: add pytest-vcr into tests

Description
Current tests will consume your api quota every time when a PR is merged. A test with vcr will intercept the new http request and replay a recorded one. This may safe your api quota.
Pros:

Safe api quota and speed up tests.
Still can run when openai server is down.

Cons:

May not be aware of api change. (seldom happen)

Reference
https://pytest-vcr.readthedocs.io/en/latest/

we do not have Unit testing so We should keep real world  test maybe better.
About the cost  I think its OK for just main branch merge


visualize the res image

Hi, firstly thanks for your wonderful job.
And I wanna know whether you try to visualize the polar res image?
Cuz I wanna know why polar res image could be better as motion features ?
I have tried to visualize the polar res image like motionseg3d, however I failed, I couldn't find the var that I need or I don't know how to process it .
So if it is possible, could you provide me some guidance..
Thanks in advance!

Hi, firstly thanks for your wonderful job. And I wanna know whether you try to visualize the polar res image? Cuz I wanna know why polar res image could be better as motion features ? I have tried to visualize the polar res image like motionseg3d, however I failed, I couldn't find the var that I need or I don't know how to process it . So if it is possible, could you provide me some guidance.. Thanks in advance!

I think that MotionBEV gets promising performance since its data preprocessing has obtained a good initial result.
Meanwhile, BEV perspective may indeed be easier to capture motion than range view.

这是来自QQ邮箱的假期自动回复邮件。您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

I'm curious if you're in the same research group as the author of MF-MOS. That's a great job.
Sincerely hope that we have the opportunity for in-depth communication. Thank you very much.
My Email<EMAIL_ADDRESS>

Hi, firstly thanks for your wonderful job. And I wanna know whether you try to visualize the polar res image? Cuz I wanna know why polar res image could be better as motion features ? I have tried to visualize the polar res image like motionseg3d, however I failed, I couldn't find the var that I need or I don't know how to process it . So if it is possible, could you provide me some guidance.. Thanks in advance!

I think that MotionBEV gets promising performance since its data preprocessing has obtained a good initial result. Meanwhile, BEV perspective may indeed be easier to capture motion than range view.

Thanks for your reply. You are right, BEV in some conditions performs better than RV. Both of them have their own pros, so I try to visualize the polar image to find out the reason and finally I think that the motion information is in the 8 channels since the result of visual image about single channel is not good.

I'm curious if you're in the same research group as the author of MF-MOS. That's a great job.
Sincerely hope that we have the opportunity for in-depth communication. Thank you very much.
My Email<EMAIL_ADDRESS>


Better error handling for database type mismatches

Describe the bug
If I change something in the database schema, often the first indication I get of this is the following type of exception:

How is this even possible? It would seem the intent is for _isNullableType to only return true if null is a valid value for that type.
Can some more useful information be displayed in this case?

Actually t in this case is bool, so _isNullableType seems to be broken.

I agree that the error message is a bit vague. We should improve it so that it is clear that the data deserialized doesn't match the target data type.
That is the issue encountered in the first image. I would guess that you have changed a column (or field). from bool? to bool but you are still fetching null values from the database. This could happen if you have re-generated the project but not migrated the database.
The isNullableType method is actually working as expected, although the name should be improved. It's purpose is to inform us if the value that is being deserialized is either a nullable or a non nullable version the passed in type.
On the row in the second image it recognizes that t(that is passed from generated code) is either bool or bool?.

No, I think there's still a problem here.
I think
  bool _isType<T1, T2>(Type t) => t == T1 || t == T2;

  bool _isNullableType<T>(Type t) => _isType<T, T?>(t);

should be
bool _isType<T>(Type t) => t == T;

bool _isNullableType<T>(Type t) => _isType<T?>(t);

because the test t == T1 || t == T2 will always return true for _isType<T, T?>(t).
The way to fix the larger issue though is something like the following, in generated protocol files, in fromJson methods, for fields that are not nullable:
Replace
      blocked:
          serializationManager.deserialize<bool>(jsonSerialization['blocked']),

with
      blocked:
          serializationManager.deserialize<bool>(jsonSerialization['blocked']
                  ?? throw 'Non-nullable field `blocked` is missing from JSON for `UserInfo`'),

So, to recap, I think there are two separate issues that need fixing here.

Addressed by #1527 and #1528.


Allow user to specify a specific rust toolchain

It would be useful if the script needs dependencies that are attached to a specific rust version for one to be able to instruct which toolchain should be used.

Does not the -t, --toolchain <toolchain> argument do what you want?


docs: add libraries and frameworks section

Made a slight error with the naming of this PR. We normally follow conventional commits so this should have been docs: add libraries and frameworks section (#24)


[backend] added leveldb auth backend, derived from gob backend;

Hello, I just added an leveldb auth backend by copying the gob-file-backend and adjusted it accordingly. I hope this could be upmerged!

So... travis is failing because of the unresolved leveldb dependency.

Whoa, sorry about missing this for so long. I'm going to take a look today.

See PR #24. I'm going to add my standard tests before merging though.


Update p9s.js

add script for Ja

Hi, same issue here as in #5600.  You need to do what @xiashtra did in #5601 for #5600 for this PR.

I think this was covered by #5613.  It's hard to tell because the diff is weird.
I am going to close this as I cannot merge something into the triggers branch directly.  Sorry, as I mentioned before, you need to edit the main branch and not the triggers branch.  Let me know if I can help with any confusion.


🐞 Rendering Invisible Text

Right now, CodeEditTextView attempts to render every line of text at once. This leads to a large reduction in performance for files that extend beyond the bounds of the visible text as that text is always rendered. This isn't a problem for syntax highlighting, as the highlighter only uses the text that is visible, but any text that is being rendered that isn't visible is wasted CPU resources.
The root cause is this line in STTextView, where TextKit asks a NSTextViewportLayoutControllerDelegate for the bounds to render. In the current implementation, STTextView returns the entire bounds of the text view, effectively telling TextKit to render the entire document.
https://github.com/krzyzanowskim/STTextView/blob/deab1ae18d45a0c865aefc897645ebca882d0190/Sources/STTextView/STTextView%2BNSTextViewportLayoutControllerDelegate.swift#L8
    public func viewportBounds(for textViewportLayoutController: NSTextViewportLayoutController) -> CGRect {
        // Return bounds until resolve bounds problem
        return bounds

The comments in the implementation say that the view lays out the entire document because layout fragments outside the viewport are broken, which if I'm not mistaken is the expected result. One of the benefits of TextKit2 is that it only lays out text in the viewport, so any layout fragments outside the viewport will of course be broken. This also, of course, breaks the ruler view.
The solution for the ruler view is to calculate line heights starting at the first rendered line, instead of the actual first line. If we align the first rendered lines in both the text view and the ruler view, it will appear that all drawn lines are lined up with their corresponding text lines. This will also make the ruler view more performant.
I've already been able to fix the viewport calculation problem using a custom delegate class, and @ben-p-commits mentioned on Discord they'd be willing to take on the ruler view. Both changes will have to be merged at the same time so this issue is to track this.

I've already been able to fix the viewport calculation problem using a custom delegate class

as far as I'm concerned, simply re-enabling viewportBounds is not enough. The frame has to update, and it doesn't work properly. Anyway I enable the viewport on main branch https://github.com/krzyzanowskim/STTextView
Related discussion: https://github.com/krzyzanowskim/STTextView/discussions/12


Fixed wallet creation

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Nox complains about not finding pip

Hello,
I'm on Chapter 3, and I've added most of the stuff in that article (without comitting atomic - sorry!) and I can't seem to commit and push the changes so far. When running nox -rs lint, it fails;
$ nox -rs lint
nox > Running session lint-3.8
nox > Re-using existing virtual environment at .nox/lint-3-8.
nox > poetry export --dev --format=requirements.txt --output=/tmp/tmpl2c9lseu
nox > pip install -c /tmp/tmpl2c9lseu flake8 flake8-annotations flake8-bandit flake8-black flake8-bugbear flake8-import-order
nox > Error: pip is not installed into the virtualenv, it is located at /home/per/.pyenv/shims/pip. Pass external=True into run() to explicitly allow this.
nox > Session lint-3.8 failed.

 Yet lint doesn't play ball. This is the code for it, think it's identical to yours.
def install_with_constraints(session, *args, **kwargs):
    with tempfile.NamedTemporaryFile() as requirements:
        session.run(
            "poetry",
            "export",
            "--dev",
            "--format=requirements.txt",
            f"--output={requirements.name}",
            external=True,
        )
        session.install(f"--constraint={requirements.name}", *args, **kwargs)

@nox.session(python=["3.8"])
def lint(session):
    args = session.posargs or locations
    install_with_constraints(
        session,
        "flake8",
        "flake8-annotations",
        "flake8-bandit",
        "flake8-black",
        "flake8-bugbear",
        "flake8-import-order",
    )
    session.run("flake8", *args)

Any ideas? Having a hard time debugging on my own. Thanks for an awesome guide.

Looks to have been an issue with cached stuff. Works now. Thanks again!


Antiquities and the Machines: AI for ancient world data

I was thinking of a slightly different format, like a half day, kinda informal workshop. Ancient world data are a peculiar kind of humanities data (that are already pretty complicated ;-) ), because they are even more fragmentary and incomplete than "regular" historical data. That means that even a new nugget of information can literally change the way we understand our ancient past. There is a great interest in the classics and archaeology community for AI methods. And these disciplines are actually quite advanced in the applications of DH. I was thinking that we could invite Thea Sommershield (the AI and epigraphy project based at Venice that was on the news recently), my colleague Charlotte Tupman at Exeter, who has also done some work on AI and stonemasonry in Roman epigraphy, and Perhaps Peter Campbell (Oxford, I believe) who was also experimenting with AI and archaeological data. I could involve the ArcheoFoss community (archaeology and open software) as well as the large and well established CAA (computer application in archaeology) community. And the Digital Classicists. I'd be very happy to organise this, maybe in collaboration with one of the persons above, in addition or in alternative to the other idea I suggested for the Fall.


Force controls to hide

Windows controls (Close, Min and Max) are not considering the border in WndProc so we must hide them in CreateParams

Duplicate of #31


[#2229] feat(server,client): Introduce OutputUtils class to reduce log size

What changes were proposed in this pull request?
Introduce OutputUtils class to reduce log size
Why are the changes needed?
Fix: #2229
Does this PR introduce any user-facing change?
Log become useful but tiny enough.
How was this patch tested?
new UT

@jerqi Thanks for your review, merge it

If having multi modules associated change, the style commit log could be unified as the feat(server)(client). I have seen a lot of different styles like feat(server,client) and feat(server/client) . @maobaolong

@maobaolong Sorry for this mistake, will check it before merge.

If having multi modules associated change, the style commit log could be unified as the feat(server)(client). I have seen a lot of illegal different styles like feat(server,client) and feat(server/client) . @maobaolong

A lot of projects adopt the style feat(server,client), too.


fix to javadoc commenting Item Class #15 error

fix to javadoc which was causing javadoc generation to fail

still no button?


Image chooser modal slow to open

If there are a number of image thumbs to be generated when opening an image chooser, there can be a delay of a few seconds before the modal appears. While waiting you can click any button that opens the image chooser modal again and open multiple instances which breaks all of them.

Closing as duplicate of #2364


[packages][tool] Should make-deps-path-based sort pubspecs after modifying them?

The camera/camera/example app had the following dependency override:
 dependency_overrides:
   camera_web:
     path: ../../camera_web

When running dart flutter_plugin_tools.dart make-deps-path-based --current-package --target-dependencies=camera_android_camerax on the branch of this PR, the script produced the following changes:
diff --git a/packages/camera/camera/example/pubspec.yaml b/packages/camera/camera/example/pubspec.yaml
index 9f3749514..66087e5b4 100644
--- a/packages/camera/camera/example/pubspec.yaml
+++ b/packages/camera/camera/example/pubspec.yaml
@@ -28,9 +28,15 @@ dev_dependencies:
   integration_test:
     sdk: flutter

+
+# FOR TESTING AND INITIAL REVIEW ONLY. DO NOT MERGE.
+# See https://github.com/flutter/flutter/blob/master/docs/ecosystem/contributing/README.md#changing-federated-plugins
 dependency_overrides:
+
   camera_web:
     path: ../../camera_web
+  camera_android_camerax:
+    path: ../../../../packages/camera/camera_android_camerax

 flutter:
   uses-material-design: true

That change is invalid, because it breaks our sort_pub_dependencies lint:
Finding changed packages relative to "cebedf87bf381c605850b7c5e04c580396e3666e"...
Rewriting references to: camera_android_camerax...
  Modified packages/camera/camera/pubspec.yaml
Running for all packages that have uncommitted changes

Changed packages: camera/camera
[0:00] Running for packages/camera/camera...
Running command: "flutter pub get" in /b/s/w/ir/x/w/packages/packages/camera/camera
Resolving dependencies...
Downloading packages...
+ ... list of deps
Changed 56 dependencies!
12 packages have newer versions incompatible with dependency constraints.
Try `flutter pub outdated` for more information.
Resolving dependencies in `./example`...
Downloading packages...
Got dependencies in `./example`.
Running command: "dart analyze --fatal-infos" in /b/s/w/ir/x/w/packages/packages/camera/camera
Analyzing camera...

   info - example/pubspec.yaml:38:3 - Dependencies not sorted alphabetically. Try sorting the dependencies alphabetically (A to Z). - sort_pub_dependencies

1 issue found.

The following packages had errors:
  packages/camera/camera

Should the tool sort the pubspec after modifying it?

In the meantime, I am removing the override in the camera example, which probably shouldn't be there. See:

https://github.com/flutter/packages/pull/8331

It'd also be nice if the script listed all the pubspecs it touched in the output, currently it only says it Modified packages/camera/camera/pubspec.yaml, but packages/camera/camera/example/pubspec.yaml was also modified.


[bug ?] Not actualising Active Effects automatically

System : DND5e
How to reproduce : Add an effect on an actor.
How to make it work : You need to clic on the window's module to make it actualize.

Fixed in v1.1.0


Can revert of "Cluster manager task throttling feature" be forward ported to 2.x and main?

We are seeing failure for k-NN of backwards compatibility tests (2.1-2.5): https://github.com/opensearch-project/k-NN/actions/runs/3475055465/jobs/5824919899.
> Task :qa:rolling-upgrade:testAgainstTwoThirdsUpgradedCluster

org.opensearch.knn.bwc.IndexingIT > testKNNDefaultIndexSettings FAILED
    org.opensearch.client.ResponseException: method [PUT], host [http://[::1]:33417], URI [_cluster/settings], status line [HTTP/1.1 400 Bad Request]
    {"error":{"root_cause":[{"type":"illegal_argument_exception","reason":"illegal value can't update [cluster_manager.throttling.thresholds.] from [{}] to [{}]"}],"type":"illegal_argument_exception","reason":"illegal value can't update [cluster_manager.throttling.thresholds.] from [{}] to [{}]","caused_by":{"type":"illegal_argument_exception","reason":"All the nodes in cluster should be on version later than or equal to 2.4.0"}},"status":400}
        at __randomizedtesting.SeedInfo.seed([D1CA7E7E74CEB5B:E996EB0D8D75EE2B]:0)
        at app//org.opensearch.client.RestClient.convertResponse(RestClient.java:375)
        at app//org.opensearch.client.RestClient.performRequest(RestClient.java:345)
        at app//org.opensearch.client.RestClient.performRequest(RestClient.java:320)
        at app//org.opensearch.knn.KNNRestTestCase.updateClusterSettings(KNNRestTestCase.java:510)
        at app//org.opensearch.knn.KNNRestTestCase.clearCache(KNNRestTestCase.java:678)
        at app//org.opensearch.knn.KNNRestTestCase.cleanUpCache(KNNRestTestCase.java:134)
        at java.base@11.0.17/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base@11.0.17/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base@11.0.17/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base@11.0.17/java.lang.reflect.Method.invoke(Method.java:566)
        at app//com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1750)

I believe changes causing this were reverted for 2.4: https://github.com/opensearch-project/OpenSearch/pull/5203. Can this revert be forward ported to 2.x and main?

cc: @dhwanilpatel

@Bukhtawar @dhwanilpatel Any update on this? This is blocking our CI for k-NN. Instead of fixing in #5305, can we revert the change in 2.x and main so that we are unblocked faster? We have several PRs currently blocked on this.
cc: @vamshin

@jmazanec15 we have merged fix in both main and 2.x. Let us know if you still face any issue in CI.

Thanks!


Update footer.ejs

Adding the footer of the page
Description
Describe the changes you have made.
Issue Resolved
Link to the issue that is resolved by this pull request goes here e.g. Fixes #10.
Screenshots
Provide some screenshots of the changes made (if applicable).
Checklist
Please make sure to review the following before submitting your PR:

[x] I have read the contribution guidelines.
[x] I have read the code of conduct.
[x] I have reviewed my submission in detail.

Also, @mishank2002, comment on Issue #9 so I can assign this issue to you.


Add issue template

Add an issue template directing users to the docker/docker repository
for bug reports related to engine's swarm mode.
cc @aluzzardi @mgoelzer @thaJeztah

LGTM

Current coverage is 53.58% (diff: 100%)

Merging #1525 into master will decrease coverage by 0.08%

@@             master      #1525   diff @@
==========================================
  Files            82         82
  Lines         13410      13410
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           7197       7186    -11
- Misses         5228       5236     +8
- Partials        985        988     +3

Powered by Codecov. Last update 22c765c...2522471

LGTM (but no permissions to merge) :smile:

LGTM

LGTM


[Ember 2.3]: DEPRECATION: Using the injected container is deprecated.

Hi!
I upgraded to Ember 2.3 and I get the following deprecation:
DEPRECATION: Using the injected `container` is deprecated. Please use the `getOwner` helper instead to access the owner of this object. [deprecation id: ember-application.injected-container] See http://emberjs.com/deprecations/v2.x#toc_injected-container-access for more details.
        at HANDLERS.(anonymous function) (http://localhost:4200/app/assets/vendor.js:15523:7)
        at raiseOnDeprecation (http://localhost:4200/app/assets/vendor.js:15433:12)
        at HANDLERS.(anonymous function) (http://localhost:4200/app/assets/vendor.js:15523:7)
        at invoke (http://localhost:4200/app/assets/vendor.js:15539:7)
        at deprecate (http://localhost:4200/app/assets/vendor.js:15492:32)
        at Object.deprecate (http://localhost:4200/app/assets/vendor.js:25501:37)
        at Object.defineProperty.get (http://localhost:4200/app/assets/vendor.js:10953:26)
        at _bindToAuthenticatorEvents (http://localhost:4200/app/assets/vendor.js:173516:31)

When I step through the stacktrace it points to the following places:
(anonymous function)         application-route-mixin.js:89
(anonymous function)         application-route-mixin.js:83
setupSessionRestoration      setup-session-restoration.js:7
initialize                   ember-simple-auth.js:5

restore                    internal-session.js:53
beforeModel                setup-session-restoration.js:17
setupSessionRestoration    setup-session-restoration.js:7

_bindToAuthenticatorEvents  internal-session.js:150
_setup                      internal-session.js:89
(anonymous function)        internal-session.js:64

authorize        session.js:209
exports.default._ember.default.Mixin.create.ajaxOptions.hash.beforeSend        data-adapter-mixin:76

Hope this helps. Thanks

PR would be great - should be easily doable with https://github.com/rwjblue/ember-getowner-polyfill

@marcoow okay, I'll give it a try :smiley: but I'm not sure if I can find all the appearances since I'm not too familiar with the ember-simple-auth internals

are you running against simple auth master? a lot of them have been fixed but there is still one outstanding https://github.com/simplabs/ember-simple-auth/pull/822
in you package.json
 "ember-simple-auth": "git://github.com/simplabs/ember-simple-auth.git"
rm -rf node_modules/ember-simple-auth && npm install

@jcope2013 no I'm not on master. I just realized when I updated this morning to Ember 2.3. Does it make sense to create a PR or is is already almost solved?

I'm using master, and the simple auth test helpers are also giving me deprecation messages since they're using app.__container__.lookup. So whoever digs in to squash the deprecations, check there too (I couldn't figure out how access the app instance in the helpers)

im using master and still getting this deprecation on ember 2.3
DEPRECATION: Using the injected `container` is deprecated. Please use the `getOwner` helper instead to access the owner of this object. [deprecation id: ember-application.injected-container] See http://emberjs.com/deprecations/v2.x#toc_injected-container-access for more details.
            at HANDLERS.(anonymous function) (http://<IP_ADDRESS>:4200/assets/vendor.js:16693:7)
        at raiseOnDeprecation (http://<IP_ADDRESS>:4200/assets/vendor.js:16603:12)
        at HANDLERS.(anonymous function) (http://<IP_ADDRESS>:4200/assets/vendor.js:16693:7)
        at invoke (http://<IP_ADDRESS>:4200/assets/vendor.js:16709:7)
        at deprecate (http://<IP_ADDRESS>:4200/assets/vendor.js:16662:32)
        at Object.deprecate (http://<IP_ADDRESS>:4200/assets/vendor.js:26671:37)
        at Object.defineProperty.get (http://<IP_ADDRESS>:4200/assets/vendor.js:12123:26)
        at Function.ClassMixinProps._lazyInjections (http://<IP_ADDRESS>:4200/assets/vendor.js:47114:19)
        at instantiate (http://<IP_ADDRESS>:4200/assets/vendor.js:12078:34)

It is getting pretty annoying having to install specific commits as versions because releases aren't being tagged. Not to mention working with latest versions of ember should happen fast and never do.
When I have time will have to roll my own code instead of use this.

@robclancy I don't think there is a release. I think they are still working on the deprecations.
I could contribute/help a little bit but I don't know how to start or if it is still necessary to help.

there is 2 failing tests for the PR to fix the last deprecation, I could use some help trying to fix them
https://github.com/simplabs/ember-simple-auth/pull/822
https://travis-ci.org/simplabs/ember-simple-auth/builds/104554171

I believe this will solve the problem, it worked for me
https://github.com/yurifrl/ember-simple-auth/commit/bb121825bba872aa4f86c7100782cde9bca03b85

Still getting same deprecation warning. Would you please let me know that how i can get latest build.

Yup.. Same here.. Still getting the deprecation warnings. Hate to see the yellow warnings screaming in the console.

Until this is fixed, add this file to hide the deprecations (of course this only works until Ember 3.0, whenever that is):
// app/initializers/deprecations.js
import Em from 'ember';

export function initialize(application) {
  Em.Debug.registerDeprecationHandler((message, options, next) => {
    if (options.id === 'ember-application.injected-container') {
      return;
    }
    next(message, options);
  });
}

export default {
  name: 'deprecations',
  initialize
};

@2468ben Thanks, that worked =) I thought you could turn it off in a feature flag ember-debug-handlers https://github.com/emberjs/rfcs/pull/65 but I guess not.

Try using https://github.com/mixonic/ember-cli-deprecation-workflow to manage your deprecations

@kylenathan yeah the only reason it didn't work for me is because I manually set EmberApp.tests = false for my dev environment. If you're not disabling tests in dev, then that plugin should work great.

This was marked as closed, but I'm still getting deprecation warning... ?

Are you actually using ESA 1.1.0-beta.1?

@marcoow would it be possible to backport the fix to v1.0 and do a patch release?

In theory yes but I don't actually think it's necessary as it's only a deprecation and we're not even close to Ember 3.0 which would be the first Ember release where this would actually turn into a real problem.

that's true, but the deprecations are quite annoying :wink:
I've just installed the addon mentioned by @kylenathan and hope that will bring me some silence on the console, but it's certainly best for the average user to just fix such deprecations.

You can always install 1.1.-beta.1 of course

We've been running with 1.1 beta since it was released, no changes were required and it works perfectly.

As @marcoow pointed out, they are only deprecation warnings. Of course they are annoying but I rather have deprecation warnings than installing a beta. So I'll wait until 1.1 becomes stable :+1:

This is actually fixed in 1.1.0-beta.1


Increase Version-Strategy is ignored with Grouped Beta for NPM

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
npm
Package manager version
npm 9.8.0
Language version
node 18.16.1
Manifest location and content before the Dependabot update
https://github.com/broksonic21/repro-increase-dependabot-grouped/blob/main/package.json
dependabot.yml content
https://github.com/broksonic21/repro-increase-dependabot-grouped/blob/main/.github/dependabot.yml
Updated dependency
"@next/env": "^13.4.10"
should go to
"@next/env": "^13.4.11",
What you expected to see, versus what you actually saw
"@next/env": "^13.4.10" is in my package.json
It's also transitively loaded via other next dependencies. It is in fact updated in the package-lock.json, but not package.json.
Given we have "increase" version strategy turned on, I would have expected to get an update notification from Dependabot.
~/testrepo (main ✔) npm ls @next/env
<EMAIL_ADDRESS>/Users/me/testrepo
├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>deduped

Dependabot logs thing it's up to date:
updater | 2023/07/21 14:53:03 INFO <job_698761792> Checking if @next/env 13.4.11 needs updating
  proxy | 2023/07/21 14:53:03 [018] GET https://registry.npmjs.org:443/@next%2Fenv
  proxy | 2023/07/21 14:53:03 [018] 200 https://registry.npmjs.org:443/@next%2Fenv
  proxy | 2023/07/21 14:53:03 [020] GET<EMAIL_ADDRESS>  proxy | 2023/07/21 14:53:03 [020] 200<EMAIL_ADDRESS>updater | 2023/07/21 14:53:03 INFO <job_698761792> Latest version is 13.4.11
updater | 2023/07/21 14:53:03 INFO <job_698761792> No update needed for @next/env 13.4.11

which, again - it is in the lock , but not in the package.json itself.
Native package manager behavior
npm-check-updates shows an update to do:
~/testrepo (main ✔) npx npm-check-updates @next/env
Checking /Users/me/testrepo/package.json
[====================] 1/1 100%

 @next/env  ^13.4.10  →  ^13.4.11

Run npx npm-check-updates @next/env -u to upgrade package.json

Images of the diff or a link to the PR, issue, or logs
https://github.com/broksonic21/repro-increase-dependabot-grouped/network/updates/698761792
Smallest manifest that reproduces the issue
https://github.com/broksonic21/repro-increase-dependabot-grouped shows this
seems only to happen if:

package is both a direct and transitive dependency
up to date in the lock , but not in the package.json itself
use increase strategy

Note: if the packages are all versioned as "13.4.10", it works fine.
issue is when they are "^13.4.10" - it doesn't bump all of them.

I think this was fixed by https://github.com/dependabot/dependabot-core/pull/7561
I forked the repro and rolled back to the first commit, this PR looks correct to me https://github.com/jakecoffman/repro-increase-dependabot-grouped/pull/1
@broksonic21 do you agree?

That looks great - thanks!


Flash button will appear after switch back/front first

[x] I have verified there are no duplicate active or recent bugs, questions, or requests
[x] I have verified that I am using the latest version of Material Camera.
[x] I have given my issue a non-generic title.

Include the following:

Material Camera version: 0.3.2
Device OS version: 6.0
Device Manufacturer: LG
Device Name: Nexus 5X

My method to use material-camera:
   public static void doUseCamera(Activity activity) {                 new MaterialCamera(activity)                         /** all the previous methods can be called, but video ones would be ignored */                         .stillShot() // launches the Camera in stillshot mode,                         .primaryColorAttr(R.attr.colorPrimary)                         .defaultToFrontFacing(false)                         .saveDir(Config.CAMERA)                         .start(SELECTED_CAMERA);     }
Reproduction Steps

start a camera with doUseCamera()

Expected Result

camera starts
Icon to Switch front / back appears bottom left
Icon to set flashmode apperas bottom left

Actual Result

Icon to set flashmode don't appear
after Switch to front and to back, a flashmode appears!

A patch is being written that fixes this issue. Thanks for the info. FYI,
the flash button is actually still there. And clicking the button will
toggle the states, triggering logic to choose a drawable for the button.
This is a much quicker way to get the button to show up than  switching
between front-facing and rear camera. Again, this is just FYI to help make
your experience a little easier until the patch is done.
On Sep 16, 2016 8:35 AM, "fegl84"<EMAIL_ADDRESS>wrote:

I have verified there are no duplicate active or recent bugs,
questions, or requests
I have verified that I am using the latest version of Material
Camera.
I have given my issue a non-generic title.

Include the following:

Material Camera version: 0.3.2
Device OS version: 6.0
Device Manufacturer: LG
Device Name: Nexus 5X

My method to use material-camera:
` public static void doUseCamera(Activity activity) {
new MaterialCamera(activity)
/** all the previous methods can be called, but video ones would be
ignored */
.stillShot() // launches the Camera in stillshot mode,
.primaryColorAttr(R.attr.colorPrimary)
.defaultToFrontFacing(false)
.saveDir(Config.CAMERA)
.start(SELECTED_CAMERA);
}`
Reproduction Steps

start a camera with doUseCamera()

Expected Result

camera starts
Icon to Switch front / back appears bottom left
Icon to set flashmode apperas bottom left

Actual Result

Icon to set flashmode don't appear
after Switch to front and to back, a flashmode appears!

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/afollestad/material-camera/issues/102, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AI064_zTS9xPhST9AyFqN-U_27A_eDaPks5qqo0SgaJpZM4J-5GY
.

https://github.com/afollestad/material-camera/releases/tag/0.4.0

https://github.com/afollestad/material-camera/releases/tag/0.4.1


Definitive Guide: Links to Widgets incorrect

In the "Definitive Guide" the links to the Bootstrap and jQuery UI widget documentations are incorrect.
They link to the Guide main page rather than the widget pages.
Here's a screenshot:

Duplicate of #9922


bmh: add HostFirmwareSettings resource and unit tests

Adds the HostFirmwareSettings resource to the bmh package and unit tests for the functions it exports.

cc: @dgonyier


⚠ Bofrost.com has degraded performance

In 1d05293, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 538 ms

Resolved: Bofrost.com performance has improved in 0ab623d after 14 minutes.


[BSO] DG shop quality of life

i looked around but could find anythings about this anywhere sorry if its actually here.
add how many items you want to buy instead of having to copy paste the msg 20+ times
example
=dg buy 10 Dungeoneering dye
=dg buy 400 Offhand Chaotic crossbow
this could save alot of time that is really a waste with this small fix/add if its possible ofc :)

This has now been added.


Horizontal/Vertical Image Mirroring

Is there a trick that I'm missing to mirror the contents of an NVGpaint horizontally or vertically when drawing it? I'm using nvgImagePattern to draw tiles out of one large texture atlas, but am not having much luck being able to mirror it. Scaling both through NVGScale(nvg, -1) and trying to scale the paint's x transform matrix by -1 have not seemed to work, but it might just be me being bad at matrix math. I see NVG_IMAGE_FLIPY but that's at image load, and I'd like to be able to toggle it on a per-NVGpaint basis.

Negative scaling should do the trick. Can you post a snippet of code which you're trying to use for that?

Loading the image:
    hnd = nvgCreateImageMem(info->nvg, 0, (unsigned char *) buffer, sz);

Rendering:
	nvgScale(nvg, -1, 1);
	auto paint = nvgImagePattern(nvg, 400, 400, 320, 32, 0, hnd, 1.0f);
	nvgBeginPath(nvg);
	nvgRect(nvg, 400, 400, 320, 32);
	nvgFillPaint(nvg, paint);
	nvgFill(nvg);

No visible output when nvgScale is used that way. No change if I move nvgScale to after the paint and before the path. No change if I use 400+320 to shift it by the width.
If I try nvgTransformScale(paint.xform, -1, 1); I get what looks like one column of fragments stretched out. Thinking it might be a tex coord thing, I tried adding NVG_IMAGE_REPEATX when loading the image, which gives me something different but nothing that resembles horizontal mirroring.
In this case there's no nvgTransform here, but in the real thing I'm using nvgTransform as a 2D camera control, so I'm keen on trying to adjust the paint instead of trying to change the global transform, which is why I started digging around in the NVGpaint struct and tried changing xform there.
I'm running the GL3 implementation with the HEAD revision of NanoVG

If you put scale there, then your rectangle will be drawn at -400,400, which is probably not what you want. Something like this should work:
float x = 400, y = 400, w = 320, h = 20;
nvgSave(nvg);
nvgTranslate(nvg, x+w/2, y+h/2);  // center of the image
nvgScale(nvg, -1, 1); // flip
nvgBeginPath(nvg);
nvgRect(nvg, -w/2, -h/2, 320, 32); // center of the image at (0,0)
nvgFillPaint(nvg, paint);
nvgFill(nvg);
nvgRestore(nvg);

Yup, bitten by my not remembering how transforms work again :) Thanks!

If you run into problems with transforms, a good way is to google something like:
html canvas how to flip image and you should usually be able to derive the solution from there.


About outputting PDF with Chinese information

I want to use this to output a PDF in Chinese. Is this the correct setting?
pdf-engine = "xelatex"
[output.pandoc.profile.pdf.variables]
CJKmainfont = "SimSun"

yep，i just try and it was work

This is useful to me too, Thank you.


Namespaces implementation

There was problem in case of several cache instances in the app.
All cache instances was forced to use same formats because tables, that not presented in formats list, was deleted by setFormats.
I decided to solve this problem with namespaces. Now instances with different namespace will not affect on each other. Changes are backward compatible with current FastImageCache.

+1 to this. We're using FIC is multiple places (libraries, etc) and are having all our caches cleared on startup.

+1


Conform to semantic versioning

Please considering changing your versioning scheme to stick to semantic versioning. The current version number increase does not conform to this convention as far as I can jugde. I refer to adding new features such as setting the default color which should cause an increase of the minor version number.

Noted for the future. I did not consider that feature to be something worth a minor release, so I chose to keep it in the 1.0 release.

The new feature would be a version change from v.1.0.z to v.1.1.0.


[BUG] Ground all and automat all buttons are disabled despite giving user the correct permission

Describe the bug
Ground all and automat all buttons are disabled despite giving user the correct permission. This is caused by having a missmatch between permission strings in roles and in OperatorView. See below:

/cib


panic: cant find keyval map

Describe the bug
Running otel-go-instrumentation on a simple binary produces the following failure:
{"level":"info","ts":1668622016.422307,"caller":"cli/main.go:37","msg":"starting Go OpenTelemetry Agent ..."}
{"level":"info","ts":1668622016.422393,"caller":"opentelemetry/controller.go:107","msg":"Establishing connection to OpenTelemetry collector ..."}
{"level":"info","ts":1668622018.4382257,"caller":"process/discover.go:57","msg":"found process","pid":157218}
panic: cant find keyval map

goroutine 1 [running]:
github.com/open-telemetry/opentelemetry-go-instrumentation/pkg/process.(*processAnalyzer).findKeyvalMmap(0xb48c60?, 0xc0001328c0?)
	/usr/local/google/home/mikedame/go/src/github.com/open-telemetry/opentelemetry-go-instrumentation/pkg/process/analyze.go:92 +0x14f
github.com/open-telemetry/opentelemetry-go-instrumentation/pkg/process.(*processAnalyzer).Analyze(0xc000482120?, 0x26622, 0x2?)
	/usr/local/google/home/mikedame/go/src/github.com/open-telemetry/opentelemetry-go-instrumentation/pkg/process/analyze.go:118 +0x1a9
main.main()
	/usr/local/google/home/mikedame/go/src/github.com/open-telemetry/opentelemetry-go-instrumentation/cli/main.go:74 +0x37f

Environment

sample app built with go 1.18 (using golang:1.18 docker container)

To Reproduce
I am building the following code (from here:
package main

import (
	"fmt"
	"log"
	"net/http"

	"github.com/gorilla/mux"
)

func main() {
	r := mux.NewRouter()

	r.HandleFunc("/books/{title}/page/{page}", func(w http.ResponseWriter, r *http.Request) {
		vars := mux.Vars(r)
		title := vars["title"]
		page := vars["page"]

		fmt.Fprintf(w, "You've requested the book: %s on page %s\n", title, page)
	})

	log.Fatal(http.ListenAndServe(":8080", r))
}

Save, run go mod init and go tidy then build with go build -o gm main.go.
Run with ./gm &
Start a local OTEL collector at localhost:4317
Build the auto-instrumentation tool with make build
Run auto-instrumentation with the following command:
OTEL_SERVICE_NAME=otel-collector \
OTEL_EXPORTER_OTLP_ENDPOINT=localhost:4317 \
OTEL_TARGET_EXE=/path/to/gm ./otel-go-instrumentation

Expected behavior
Auto-instrumentation to work, or provide clearer information about what went wrong and how I can fix it.
Additional context
I'm still figuring out ebpf, but from the code the error comes from https://github.com/open-telemetry/opentelemetry-go-instrumentation/blob/e42d188a44e6d5fc26607ac92689285a0802fb51/pkg/process/analyze.go#L92. I got enough from there to check /proc/<id>/maps and it looks like none of the maps have the full read/write/execute permissions that auto-instrumentation needs:
$ cat /proc/157218/maps
00400000-00623000 r-xp 00000000 fe:00 8129336                            /path/to/gm
00623000-00823000 r--p 00223000 fe:00 8129336                            /path/to/gm
00823000-00861000 rw-p 00423000 fe:00 8129336                            /path/to/gm
00861000-0089a000 rw-p 00000000 00:00 0
01b66000-01b87000 rw-p 00000000 00:00 0                                  [heap]
c000000000-c000400000 rw-p 00000000 00:00 0
c000400000-c004000000 ---p 00000000 00:00 0
7fe4ac000000-7fe4ac021000 rw-p 00000000 00:00 0
7fe4ac021000-7fe4b0000000 ---p 00000000 00:00 0
7fe4b0000000-7fe4b0021000 rw-p 00000000 00:00 0
7fe4b0021000-7fe4b4000000 ---p 00000000 00:00 0
7fe4b4000000-7fe4b4021000 rw-p 00000000 00:00 0
7fe4b4021000-7fe4b8000000 ---p 00000000 00:00 0
7fe4b8000000-7fe4b8021000 rw-p 00000000 00:00 0
7fe4b8021000-7fe4bc000000 ---p 00000000 00:00 0
7fe4bc000000-7fe4bc021000 rw-p 00000000 00:00 0
7fe4bc021000-7fe4c0000000 ---p 00000000 00:00 0
7fe4c02ea000-7fe4c02eb000 ---p 00000000 00:00 0
7fe4c02eb000-7fe4c0aeb000 rw-p 00000000 00:00 0
7fe4c0aeb000-7fe4c0aec000 ---p 00000000 00:00 0
7fe4c0aec000-7fe4c12ec000 rw-p 00000000 00:00 0
7fe4c12ec000-7fe4c12ed000 ---p 00000000 00:00 0
7fe4c12ed000-7fe4c1aed000 rw-p 00000000 00:00 0
7fe4c1aed000-7fe4c1aee000 ---p 00000000 00:00 0
7fe4c1aee000-7fe4c22ee000 rw-p 00000000 00:00 0
7fe4c22ee000-7fe4c22ef000 ---p 00000000 00:00 0
7fe4c22ef000-7fe4c4e00000 rw-p 00000000 00:00 0
7fe4c4e00000-7fe4d4f80000 ---p 00000000 00:00 0
7fe4d4f80000-7fe4d4f81000 rw-p 00000000 00:00 0
7fe4d4f81000-7fe4e6e30000 ---p 00000000 00:00 0
7fe4e6e30000-7fe4e6e31000 rw-p 00000000 00:00 0
7fe4e6e31000-7fe4e9206000 ---p 00000000 00:00 0
7fe4e9206000-7fe4e9207000 rw-p 00000000 00:00 0
7fe4e9207000-7fe4e9600000 ---p 00000000 00:00 0
7fe4e9600000-7fe4e9628000 r--p 00000000 fe:00 2359372                    /usr/lib/x86_64-linux-gnu/libc.so.6
7fe4e9628000-7fe4e9798000 r-xp 00028000 fe:00 2359372                    /usr/lib/x86_64-linux-gnu/libc.so.6
7fe4e9798000-7fe4e97f0000 r--p 00198000 fe:00 2359372                    /usr/lib/x86_64-linux-gnu/libc.so.6
7fe4e97f0000-7fe4e97f4000 r--p 001f0000 fe:00 2359372                    /usr/lib/x86_64-linux-gnu/libc.so.6
7fe4e97f4000-7fe4e97f6000 rw-p 001f4000 fe:00 2359372                    /usr/lib/x86_64-linux-gnu/libc.so.6
7fe4e97f6000-7fe4e9803000 rw-p 00000000 00:00 0
7fe4e9810000-7fe4e9860000 rw-p 00000000 00:00 0
7fe4e9860000-7fe4e98e0000 ---p 00000000 00:00 0
7fe4e98e0000-7fe4e98e1000 rw-p 00000000 00:00 0
7fe4e98e1000-7fe4e9960000 ---p 00000000 00:00 0
7fe4e9960000-7fe4e99c3000 rw-p 00000000 00:00 0
7fe4e99c3000-7fe4e99c4000 r--p 00000000 fe:00 2366791                    /usr/lib/x86_64-linux-gnu/libpthread.so.0
7fe4e99c4000-7fe4e99c5000 r-xp 00001000 fe:00 2366791                    /usr/lib/x86_64-linux-gnu/libpthread.so.0
7fe4e99c5000-7fe4e99c6000 r--p 00002000 fe:00 2366791                    /usr/lib/x86_64-linux-gnu/libpthread.so.0
7fe4e99c6000-7fe4e99c7000 r--p 00002000 fe:00 2366791                    /usr/lib/x86_64-linux-gnu/libpthread.so.0
7fe4e99c7000-7fe4e99c8000 rw-p 00003000 fe:00 2366791                    /usr/lib/x86_64-linux-gnu/libpthread.so.0
7fe4e99cf000-7fe4e99e1000 rw-p 00000000 00:00 0
7fe4e99e1000-7fe4e99e3000 r--p 00000000 fe:00 2359341                    /usr/lib/x86_64-linux-gnu/ld-linux-x86-64.so.2
7fe4e99e3000-7fe4e9a09000 r-xp 00002000 fe:00 2359341                    /usr/lib/x86_64-linux-gnu/ld-linux-x86-64.so.2
7fe4e9a09000-7fe4e9a14000 r--p 00028000 fe:00 2359341                    /usr/lib/x86_64-linux-gnu/ld-linux-x86-64.so.2
7fe4e9a14000-7fe4e9a16000 r--p 00033000 fe:00 2359341                    /usr/lib/x86_64-linux-gnu/ld-linux-x86-64.so.2
7fe4e9a16000-7fe4e9a18000 rw-p 00035000 fe:00 2359341                    /usr/lib/x86_64-linux-gnu/ld-linux-x86-64.so.2
7fff80864000-7fff80886000 rw-p 00000000 00:00 0                          [stack]
7fff809be000-7fff809c2000 r--p 00000000 00:00 0                          [vvar]
7fff809c2000-7fff809c4000 r-xp 00000000 00:00 0                          [vdso]

Not sure what to do with this or how to fix it

ping @edeNFed I realize this is probably not a bug, but something I'm doing wrong. do you know what this issue could be?

I'll try to explain why we need this map and how it is created.

Trace ids and span ids are generated in eBPF (if not propagated from the incoming request). In order to add the generated values to the request headers we need to be able to read them from within the Go application, which obviously can't access memory in kernel space. The way we solve this is by allocating off-heap memory on the Go application (1Mb if I am not mistaken) which the eBPF program writes into. This allocation happens only once - when the application starts.

You can create this memory area in two ways: A. via code - invoke the mmap syscall and the instrumentation agent will find it. B. via launcher - a small program that I wrote that allocates this map area and then starts the target application. Unfortunately, I didn't have time to release it as open source yet (hopefully will happen next week). For now, you can use the docker image keyval/launcher:v0.1 and use it like here

Hope that makes sense, please let me know if it's still not clear.

Thanks @edeNFed, that's very helpful! So is it correct that right now, this auto-instrumentation requires either code changes (invoking mmap) or to use your launcher to work?
It would be great if you could contribute the code/dockerfile for the launcher to this repo, so we can package something users can run out of the box.

Sure, will open a PR shortly

@edeNFed as suggested on slack, i started to use https://github.com/keyval-dev/launcher , to launch the binary . Am now getting a new error
{"level":"info","ts":1670773134.3289542,"caller":"cli/main.go:37","msg":"starting Go OpenTelemetry Agent ..."}
{"level":"info","ts":1670773134.3289938,"caller":"opentelemetry/controller.go:107","msg":"Establishing connection to OpenTelemetry collector ..."}
{"level":"info","ts":1670773136.3361652,"caller":"process/discover.go:57","msg":"found process","pid":112920}
{"level":"info","ts":1670773136.3364372,"caller":"process/analyze.go:88","msg":"found addr of keyval map","addr":139924897234944}
{"level":"info","ts":1670773136.3405275,"caller":"process/analyze.go:151","msg":"found relevant function for instrumentation","function":"google.golang.org/grpc/internal/transport.(*http2Client).createHeaderFields","returns":3}
{"level":"info","ts":1670773136.340637,"caller":"process/analyze.go:151","msg":"found relevant function for instrumentation","function":"google.golang.org/grpc.(*ClientConn).Invoke","returns":2}
{"level":"info","ts":1670773136.3408015,"caller":"process/analyze.go:151","msg":"found relevant function for instrumentation","function":"google.golang.org/grpc.(*Server).handleStream","returns":5}
{"level":"info","ts":1670773136.340816,"caller":"cli/main.go:79","msg":"target process analysis completed","pid":112920,"go_version":"1.19.1","dependencies":{"github.com/golang/protobuf":"v1.5.2","github/dineshg13/goplay/libs/logging":"v0.0.0-00010101000000-000000000000","github/dineshg13/goplay/protos":"v0.0.0-00010101000000-000000000000","go.uber.org/atomic":"v1.7.0","go.uber.org/multierr":"v1.6.0","go.uber.org/zap":"v1.24.0","golang.org/x/net":"v0.0.0-20220722155237-a158d28d115b","golang.org/x/sys":"v0.0.0-20220722155257-8c9f86f7a55f","golang.org/x/text":"v0.4.0","google.golang.org/genproto":"v0.0.0-20200526211855-cb27e3aa2013","google.golang.org/grpc":"v1.49.0","google.golang.org/protobuf":"v1.28.1"},"total_functions_found":3}
{"level":"info","ts":1670773136.341011,"caller":"cli/main.go:85","msg":"invoking instrumentors"}
{"level":"info","ts":1670773136.3461616,"logger":"allocator","caller":"allocator/allocator_linux.go:34","msg":"Loading allocator","start_addr":139924897234944,"end_addr":139924897247232}
{"level":"info","ts":1670773136.3470068,"caller":"instrumentors/runner.go:83","msg":"loading instrumentor","name":"google.golang.org/grpc"}
{"level":"info","ts":1670773136.3480148,"caller":"inject/injector.go:82","msg":"Injecting variables","vars":{"clientconn_target_ptr_pos":24,"end_addr":139924897247232,"is_registers_abi":true,"start_addr":139924897234944,"total_cpus":32}}
{"level":"error","ts":1670773136.3515298,"caller":"instrumentors/runner.go:86","msg":"error while loading instrumentors, cleaning up","name":"google.golang.org/grpc","error":"field UprobeHttp2ClientCreateHeaderFields: program uprobe_Http2Client_CreateHeaderFields: load program: permission denied: R1 type=ctx expected=fp\n0: R1=ctx(off=0,imm=0) R10=fp0\n; int uprobe_Http2Client_CreateHeaderFields(struct pt_regs *ctx)\n0: (bf) r6 = r1                       ; R1=ctx(off=0,imm=0) R6_w=ctx(off=0,imm=0)\n1: (b7) r1 = 0                        ; R1_w=0\n; struct go_slice slice = {};\n2: (7b) *(u64 *)(r10 -96) = r1        ; R1_w=P0 R10=fp0 fp-96_w=00000000\n3: (7b) *(u64 *)(r10 -104) = r1       ; R1_w=P0 R10=fp0 fp-104_w=00000000\n; if (is_registers_abi)\n4: (18) r1 = 0xffff9c5890049510       ; R1_w=map_value(off=0,ks=4,vs=48,imm=0)\n6: (71) r2 = *(u8 *)(r1 +0)           ; R1_w=map_value(off=0,ks=4,vs=48,imm=0) R2_w=scalar(umax=255,var_off=(0x0; 0xff))\n; if (is_registers_abi)\n7: (7b) *(u64 *)(r10 -448) = r6       ; R6_w=ctx(off=0,imm=0) R10=fp0 fp-448_w=ctx\n8: (15) if r2 == 0x0 goto pc+15       ; R2_w=scalar(umax=255,var_off=(0x0; 0xff))\n; slice.array = (void *)ctx->rax;\n9: (79) r1 = *(u64 *)(r6 +80)         ; R1_w=scalar() R6_w=ctx(off=0,imm=0)\n; slice.array = (void *)ctx->rax;\n10: (7b) *(u64 *)(r10 -104) = r1      ; R1_w=scalar() R10=fp0 fp-104_w=mmmmmmmm\n; slice.len = (s32)ctx->rbx;\n11: (7```

@dineshg13 Can you please give more details about the kernel version that you are using? it will help me to try to reproduce it (output of running uname -r will be great)

@edeNFed just checking, were you planning to donate the launcher code as well? I think that's the final piece of this

@damemi I found a way to allocate memory without the launcher, will open a PR that includes the new mechanism soon.

This PR should help if you are not instrumenting GRPC services: https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/32


1.1.7 nvm-noinstall.zip?

If this is a question about how to use NVM4W, please use stackoverflow instead.
If this is an issue regarding antivirus, make sure you search the existing issues first.
I'm using NVM4W version:

[x] 1.1.7
[ ] 1.1.6
[ ] 1.1.5
[ ] 1.1.4
[ ] 1.1.3
[ ] 1.1.2
[ ] 1.1.1
[ ] Older
[ ] OTHER (Please Specify)

I have already...

[x] read the README to be aware of npm gotchas & antivirus issues.
[x] reviewed the wiki to make sure my issue hasn't already been resolved.
[x] verified I'm using an account with administrative privileges.
[x] searched the issues (open and closed) to make sure this isn't a duplicate.
[x] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[ ] settings.txt
[ ] proxy support (Have you tried version 1.1.0+?)
[ ] 32 or 64 bit support (Have you tried version 1.1.3+?)
[ ] Character escaping (Have you tried version 1.1.6+?)
[ ] A standard shell environment (terminal/powershell)
[ ] A non-standard shell environment (Cmder, Hyper, Cygwin, git)

Expected Behavior
The Wiki indicates that there is a nvm-noinstall.zip for releases.
Actual Behavior
However, for 1.1.7 this asset does not seem to be published.
Steps to reproduce the problem:
See: https://github.com/coreybutler/nvm-windows/releases

Good catch. Apparently I mistakenly removed it. I'll publish the no-install version as soon as I can.

Here's some code that I use from PowerShell to update the system environment variable:
$syspath = [System.Environment]::GetEnvironmentVariable("path", "machine")
$syspath = $syspath + ";C:\Program Files (x86)\WinMerge\;C:\Program Files\KDiff3\;C:\ProgramData\chocolatey\tools\;C:\Program Files\Git\bin;C:\Program Files\Notepad++"
[System.Environment]::SetEnvironmentVariable("path", $syspath, "machine")


Earnings report

i appreciate the idea of not having to look at my lightning node thx to automated managment, but how do i track earnings? i have RTL running but that doesn't give enough information. manually tracking the nodes balance i see it going up but also down over time. is there a command to export all data regarding fees earned and paid into CSV or some format that i can use to do my accounting?

lightning-cli clboss-status has the basic information
https://gist.github.com/johnongit/97f3b03599b80e02ef37d154ece7323e sums it up nicely, how much did you spend and how much did you earn.....

Could somebody of you fork this script to make it executable even without having Docker installed?

Could somebody of you fork this script to make it executable even without having Docker installed?

I didn't try the script, but just looks like to me you'd change line 8 from:
JSON=`docker exec cln lightning-cli clboss-status | jq .offchain_earnings_tracker`

to
JSON=`lightning-cli clboss-status | jq .offchain_earnings_tracker`

and it should work

Thanks a lot! That actually helped.
Unfortunately, I still get  an error. Under my Raspiblitz installation within the admin directory it says:
lightning-cli: Moving into '/home/admin/.lightning/bitcoin': No such file or directory

Oh well you'll need to set the BASEDIR directory variable properly. Perhaps @openoms can help with what it should be for the blitz

@BlueBallroom is wolking in admin, but CLN and CLBOSS is running with the bitcoin user.
Switch to it and the default directory will work.

@openoms @willcl-ark thanks again for your support!
After some twisting it now works great. I created a repository for everyone else to contribute:
https://github.com/BlueBallroom/CLN-Earning-Script-Raspiblitz

closing


Make 2.0.0 trigger checks more often

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Make 2.0.0 trigger checks more often
Issues Resolved
Part of #1624
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1832 (71f780e) into main (9fd0770) will increase coverage by 5.46%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #1832      +/-   ##
=============================================
+ Coverage     94.53%   100.00%   +5.46%
=============================================
  Files           176         6     -170
  Lines          3622       105    -3517
  Branches         27        19       -8
=============================================
- Hits           3424       105    -3319
+ Misses          194         0     -194
+ Partials          4         0       -4

Impacted Files
Coverage Δ

src/manifests/bundle_manifest.py

tests/jenkins/jobs/BuildManifest_Jenkinsfile

src/ci_workflow/ci_check_package.py

...teg_test/integ_test_suite_opensearch_dashboards.py

src/assemble_workflow/bundle_file_location.py

...nIntegTestScript_OpenSearch_Dashboards_Jenkinsfile

tests/jenkins/jobs/PublishNotification_Jenkinsfile

...rtifacts_actions_OpenSearch_Dashboards_Jenkinsfile

src/git/git_repository.py

...ensearch_dashboards/build_artifact_check_plugin.py

... and 160 more

Continue to review full report at Codecov.

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


Refactoring admin code and do up developer documentation

Pull Request Test Coverage Report for Build 367

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.825%

Totals

Change from base Build 366:
0.0%

Covered Lines:
3158

Relevant Lines:
3477

💛  - Coveralls


Drop tslib from packageconfig.

yocto zeus has no tslib support anymore, so the build fails with:
ERROR: Nothing PROVIDES 'tslib' (but meta-qt5/recipes-qt/qt5/qtbase_git.bb DEPENDS on or otherwise requires it). Close matches:
taglib
ERROR: Required build target 'qtbase' has no buildable providers.
Missing or unbuildable dependency chain was: ['qtbase', 'tslib']

Please also consider backporting this to zeus branch

Can one of the admins verify this patch?

commit e389a96cd4d7ad77154f0d3569e09ec125707803
Author: Maxin B. John<EMAIL_ADDRESS>Date:   Wed Aug 10 11:32:07 2016 +0300
tslib: remove recipe

Modern systems generally use the kernel driver or libinput instead
of tslib. Move tslib from oe-core along with xtscal.

(From OE-Core rev: d37f6b595fd9ce53c79ff9281f2e20df7fa0503d)

Signed-off-by: Maxin B. John<EMAIL_ADDRESS>Signed-off-by: Richard Purdie<EMAIL_ADDRESS>
:100644 000000 1b0da937e8<PHONE_NUMBER> D  meta/recipes-graphics/tslib/tslib/ts.conf
:100644<PHONE_NUMBER>e8d922<PHONE_NUMBER> D  meta/recipes-graphics/tslib/tslib/tslib.sh
:100644 000000 b0264c3b38<PHONE_NUMBER> D  meta/recipes-graphics/tslib/tslib_1.1.bb

This is broken since warrior too, so probably we should backport to it also...

@LocutusOfBorg I would expect qt5 layer to drop that package config. Did they change it?

@agherzan they have it not enabled by default, but I plan to open a pull request also for them

but I'm quite confused about disabling it, because an user might have it on a custom recipe

https://github.com/meta-qt5/meta-qt5/pull/252

@kraj, you added tslib to the package config in this file in 2018, long after the commit in 2016 that @LocutusOfBorg reference removing it from oe-core. Any comments on this? Is it still a good default for qtbase on Raspberry Pi?

tslib is still in meta-oe see [1] and I could not get touch shields working without tslib
[1] https://layers.openembedded.org/layerindex/recipe/53080/

@LocutusOfBorg could you ensure meta-oe is included in your BBLAYERS?

closing then...


Add ability to use pybids db

This PR adds in functionality to save the pybids db.
It adds a pybids_db to the config, where it stores the db directory, and a flag as to whether the db should be overwritten. Wanted this to be separate from the pybids_inputs. This also updates the generate_input.py to make use of these config options. If the directory is left blank (e.g. ''), the database will be set to None (if left as '', it saves the db in the current working directory). Also added in a few small test cases to make sure the db is working as expected.
~@pvandyken I didn't end up using the pytest.fixtures and tmpdir (mentioned last week) to create the new temporary data during testing - looked as if these temp files would be created at the beginning of the test which would have been picked up by pybids and wanted to test for changes.~

Things to add / change:

For relative pathing, relative to bids_dir
sqlite file sould be added to a .bidsignore(user handled) for validation purposes -> print out warning message
separate out pybids_db in the function

Most of the things above are now fixed.
I didn't end up including a warning message for the relative path case - need to think about this one a bit more as it would depend on where that db is stored (e.g. in the input directory itself vs a derivatives folder down the line).
Also ended up just adding a comment about the relative pathing in the boilerplate for now. Functionally, this should be working. I haven't been able to test it on a real use case, and only been testing on the test cases that have been implemented.

Some good discussion on the interface issue at the dev lunch today, here's an attempt to summarize the main questions we're left with:

How do we activate the DB feature?
How do we specify the path to the DB file?

Are relative paths allowed? If so, how do we resolve them (i.e. what are they relative to)?

How do we specify if the database is to be reset?

The answer to each of the questions basically boils down to:

Via the config file.
Via the call to generate_inputs in the Snakefile.
Via the command line.

With the current implementation, everything is handled by the config file, and relative paths are resolved relative to the input directory. My preference would be to instead resolve relative paths relative to the output directory, but then generate_inputs would need another argument with the output directory, which might end up redundant.
Potential Solutions
Config file only, paths relative to input directory
# pybids_db:
#   database_dir: '/path/to/db_dir' # Leave blank if you do not wish to use this
#   reset_database: False # Change this to true to update the database

config.update(
    snakebids.generate_inputs(
        bids_dir=config["bids_dir"],
        pybids_inputs=config["pybids_inputs"],
        pybids_db=config.get("pybids_db"),
        pybids_reset_database=False,
        derivatives=config["derivatives"],
        participant_label=config["participant_label"],
        exclude_participant_label=config["exclude_participant_label"]
    )
)

Pros: Already implemented, conceptually simple, minimal changes to generate_inputs interface, backwards compatible.
Cons: Might prefer not to default to putting pybids db in the input directory
Variant: Absolute paths only (less flexible but simpler implementation and no questionable default).
Everything in generate_inputs call
config.update(
    snakebids.generate_inputs(
        bids_dir=config["bids_dir"],
        pybids_inputs=config["pybids_inputs"],
        derivatives=config["derivatives"],
        participant_label=config["participant_label"],
        exclude_participant_label=config["exclude_participant_label"]
        pybids_db=config["output_dir"],
    )
)

Pros: Easily customizable with full use of the config dict
Cons: Need to go in and edit the workflow any time you want to change anything with the DB.
To be continued on Monday...

Potential Solutions Continued
CLI Only
run.py {bids_dir} {out_dir} {analysis_level} --pybids_db {db_path} --reset_pybids_db
Could implement this in cli.py as part of the standard parser or add them to the standard parse_args mapping in the template config.
Pros: No path ambiguity (assuming db_path gets resolved relative to the working directory where run.py is called), easy to change DB use from run to run.
Cons: Adds a lot of length to CLI invocations every run if a user always wants to use the pybids DB.
A combination of the above
For example, could allow the pybids DB location to be defined in the config file (maybe only as an absolute path), but have a CLI argument that overrides it if present. All the pros and cons of whatever solutions are present, plus:
Pros: Flexibility for the user to use whatever works best for their use case.
Cons: Need to make clear how we handle the case where the DB location is defined differently in different places.

Idea from today's meeting: Combine config (absolute paths only) and CLI (resolve path before adding it to the config dict), with CLI taking precedence if a location is defined on both.

I think this is mostly ready for review (thanks @tkkuehn for the help with the CLI).
Only thing missing is testing to see if config is updated from the CLI. Still trying to figure out the best way to do this (probably making use of the mock object).

test_update_config is intended to test the function called update_config() in app.py. The best current model for testing such changes is the existing tests in test_app.

It does check what args were passed to the write_config_file function:
io_mocks["write_config"].assert_called_once_with(
    config_file=Path("/tmp/output/mock/config.yaml"),
    data=expected_config,
    force_overwrite=True,
)

Admittedly, write_config_files has no test of its own...

Yeah, that's right. More precisely, its write_config_file that needs a test

Yeah, that's right. More precisely, its write_config_file that needs a test

Yeah, I mean it probably is worth adding in a test to make sure the args are passed, but I was thinking more of the write_config_file test, and that should probably be a separate PR that considers more than just the DB update.

Think I figured it out - added the two new values pybids_db_reset and pybids_db_dir for testing to at least make sure it is properly passed along to the write_config_file in both bidsapp and workflow mode.

Brought up during the dev lunch today:
Uncomment out the config values, setting pybids_db_dir to None and adding in a separate flag/value to opt in (with a default suggested path).

@kaitj This needs a merge from main, then I'll take a last look and merge.

Also, could you squash to 1 or two commits before merging?

Also, could you squash to 1 or two commits before merging?

@pvandyken Is there an easy way to do this (a lot of commits happened since the beginning)? Actually just pushed what I thought was squashed history that ended up messing up the git history and had to revert to f9068a5.

I usually just rebase... Something like git rebase -i HEAD~40 in this case. You can ctrl+v block select in vim to change all the picks to squash at the same time. And make the first one reword.

Also, could you squash to 1 or two commits before merging?

@pvandyken Is there an easy way to do this (a lot of commits happened since the beginning)?

More info on this, for reference: https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History#_squashing

Yeah, the problem is coming from the merge from main - that history is in the logs too (probably should have rebased there)

Another option that's particularly useful if you want to just squash everything into one commit and don't mind losing the commit messages: https://git-scm.com/book/en/v2/Git-Tools-Reset-Demystified#_squashing_2

Just managed to squash to a single commit.
Issue was the merge from main rather than a rebase. Got that resolved by just rebasing from main instead.

In the spirit of rebase tips, reflog is a vital tool for recovering from rebase errors. (https://www.atlassian.com/git/tutorials/rewriting-history/git-reflog). It's saved me a few times

Alright, after a lot of back and forth between commits, I think finally caught and fixed most of the issues that I introduced from rebasing / squashing. Definitely my least favourite part of git :sweat_smile:

Looks like the test reverted back to a single test for all three cases, rather than 3 separate tests?

Looks like the test reverted back to a single test for all three cases, rather than 3 separate tests?

Should be fixed in latest commit.

@kaitj, finally got a chance to use this feature in the wild... it's soooo much faster! Definitely worth all the trouble it took.


fix linting errors in js files

Issuer: Kevin Kang
Assign to: Back-end team

Issue Done Criteria:

[ ] fixed linting errors described above

Resolved with PR #64


Added Hawaii HACC

Created a directory for August and added the Hawaii Annual Coding Challenge

@Nitharios Thanks for the PR.


Adding log_resolved param to detect_active_interpreter

Good evening,
This is a super quick fix for #372

One thing that could be useful is to display the resolved environment when --python auto is used (or maybe even every time regardless?)

What I did was create a new param, log_resolved. If True, print the resolved Python path.
I have only activated this to the --python auto option.

@kemzeb Good evening,
Please review this MR when you have time.
Thanks


Add consistency check for linear-prediction in SpectralSimulation

This puts an upper bound on the relative time one can predict with linear-prediction in SpectralSimulation (see #314)

@p-thoma
Wanted to include this before the upcoming release, but I couldnt push to your fork.
I ended up redoing this, with fixes in the review, in #384.
Can you work with branches on the tenpy/tenpy repo in the future?


Feature: Scan & Spider Coverage Control (instead of hardcoded "false" or "" to the API call)

Hi,
Feature: Add scan coverage control section on the jenkins plugin UI.  Where the user can control the depth of the spider / the scan, instead sending hardcoded "false" or ""  to the API call.
Regards

@mabdelmoez Resolved in pull request #41


go/build: copy/delete instead of rename existing binaries on windows

A surprising behavior on Windows allows executable on disk associated with running processes to be renamed without error. This behavior only occurs for the rename or 'move' operation, and not 'delete' or 'write'.
Currently, go build will move an old binary like main.exe out of the way by renaming it to main.exe~. However, main.exe may still be associated with a running process. Windows thinks this is OK, so it allows the rename operation to happen without error.
This raises two issues:
1.) It is confusing when I terminate the original process (long after a build is finished), run the associated program again on disk, only to see the program is different in some way, because it has been replaced between executions.
2.) The call to os.Executable is wrong after a rename or move operation on Windows. This might be a separate issue, but its worth mentioning, because it may be unfixable. The userspace process environment block is not updated to reflect the change of the rename operation.
My request is that go build instead copies main.exe -> main.exe~ and then attempts to delete the original main.exe. If that operation fails it can then delete main.exe~. To my knowledge this is compatible with the current behavior of go build on Windows systems.
What version of Go are you using (go version)?
go version go1.9 windows/amd64
Does this issue reproduce with the latest release?
Yes
What did you do?
One line reproduction for Windows
mkdir goissuexxxxx & cd goissuexxxxx && echo package main;import "time";import "os";import "fmt"; func main(){println("1");for { time.Sleep(100e6); fmt.Println(os.Executable());};};|gofmt > main.go && go build main.go && start main.exe && echo package main;import "time";import "os";import "fmt"; func main(){println("2");for { time.Sleep(100e6); fmt.Println(os.Executable());};};|gofmt > main.go && go build main.go && start main.exe && dir
What did you expect to see?
Copy main.exe -> ~main.exe
Delete main.exe
Resume build
What did you see instead?
Move main.exe -> ~main.exe
Resume build
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=C:\gotools
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\g
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config

I sent an e-mail to an MS employee about this.
Considering backup files, I note that running Emacs' eshell to overwrite a text file (on Windows) creates a backup.

I don't actually understand what the problem is.  In this scenario, the go tool has been asked to create an executable.  It is possible that that executable is running which the go tool is also running.  This typically happens when using go install to update a binary that happens to be running already.  On Unix, this is straightforward: we simply remove the executable and then open the pathname for writing.  This does not work on Windows, because Windows does not permit removing a running executable.  So on Windows, as a workaround, we rename the executable, because Windows does permit that.  Then we proceed as normal.  The code in question can be found in cmd/go/internal/work/build.go:
	mayberemovefile(dst)
	df, err := os.OpenFile(dst, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, perm)
	if err != nil && base.ToolIsWindows {
		// Windows does not allow deletion of a binary file
		// while it is executing. Try to move it out of the way.
		// If the move fails, which is likely, we'll try again the
		// next time we do an install of this binary.
		if err := os.Rename(dst, dst+"~"); err == nil {
			os.Remove(dst + "~")
		}
		df, err = os.OpenFile(dst, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, perm)
	}

Your proposal for what we should do instead will not work.  We have already tried and failed to remove the executable.  It won't help to copy the file first.
We could say that using go install to update a running binary simply fails on Windows, but that seems like a regression from past releases.

I primarily use UNIX based systems, but a quick peer survey returned the conclusion that this behavior was confusing on Windows.
From the perspective of a UNIX user, there is no problem because this approximates the behavior of a UNIX system. But Windows is not a UNIX system and doesn't normally allows files associated with a running process to be altered.

If I try to delete a running main.exe on Windows, it will fail. This is the expected behavior on a Windows box.
If I do a go build, that executable is successfully replaced. This is unexpected. You can't uninstall a program, delete it, or write to it, but you can go build  it.

It should be the user's responsibility to satisfy the conditions of altering a file on their system. Windows users expect files associated with running processes to be immutable, therefore go build should fail if that precondition is not met.
The community can decide whether this is a compelling issue or not. The goal was to get the data on the issue tracker for those who search for it later.

OK.  But the suggestion of copy/delete does not make sense.  It sounds like you are suggesting that if go install is used to replace a binary that is currently running, we should simply let the build fail.
This would essentially reverse  https://golang.org/cl/5502054 .  CC @alexbrainman .
It's worth noting that this change would almost certainly break the special case of go install cmd/go.

we should simply let the build fail.

Yes, that would be the idealistic behavior on a Windows system. Reading the cl gave me some perspective on how long this behavior has been in place, and it does indeed suggest that a change would break things.
Perhaps there is a way to inform the user that a file associated with a running process is being moved, at least then the element of surprise is gone.
Another odd behavior occurs when the backup file main.exe~ already exists. In this case go build main.go fails, saying: go install command-line-arguments: open main.exe: The process cannot access the file because it is being used by another process.
mkdir goissue21997 & cd goissue21997 && echo package main;import "time";import "os";import "fmt"; func main(){println("1");for { time.Sleep(100e6); fmt.Println(os.Executable());};};|gofmt > main.go && go build main.go && start main.exe && echo package main;import "time";import "os";import "fmt"; func main(){println("2");for { time.Sleep(100e6); fmt.Println(os.Executable());};};|gofmt > main.go && go build main.go && start main.exe && go build main.go

we should simply let the build fail.

Yes, that would be the idealistic behavior on a Windows system.

I am not sure I agree that this is ideal. At this moment "go build" and "go install" just silently replaces executable file whether it is running or not - the user does not need to worry if executable is running or not. What you are proposing will make "go build" and "go install" fail occasionally with "executable file is used by another process". People will come here and complain about "go build" is broken sometimes, and we would have to explain the details.
I can see that it confuses knowledgeable users like yourself. But I think this is small price to pay - you could always look at the code or ask here.

Perhaps there is a way to inform the user that a file associated with a running process is being moved, at least then the element of surprise is gone.

How do you propose to do that? We don't want warnings that user should not act upon. And these will only happens occasionally - because there is no reliable way to determine if process is exited or not. I also wouldn't be surprised if we would get many triggers from anti-virus software opening executable file during or after its run.

Another odd behavior occurs when the backup file main.exe~ already exists. In this case go build main.go fails, saying: go install command-line-arguments: open main.exe: The process cannot access the file because it is being used by another process.

Yes that happens when you are trying to actually run new copy of main.exe while previous main.exe process is still running, and we tried our trick of renaming main.exe into main.exe~ and that failed (so we are out of all options). I think the error message is clear about what has happened. So the user has to terminate previous process before trying to build new executable.
I think what we have now is as good as it can be. But if you think you can improve the situation, go ahead and propose you changes. Do not forget, you need to address Ian's comment of:

It's worth noting that this change would almost certainly break the special case of go install cmd/go.

I suggest you find solution to that problem first.
Alex

I suggest you find solution to that problem first.

Given that the behavior existed for 5+ years and this seems to be the only issue logged against it, I don't think the improvement is worth pursuing. Nobody else seems to have an opinion on it except @forskning either, so it's probably best we just leave things as they are in a working state.

@as I will close this as "working as expected". But feel free to reopen if you have some new suggestions.
Alex


Headers is not assignable to ClientOptions

Description
It seems like ClientOptions interface did not include any fetch-related options.

Reproduction
I'm using version 0.0.8
Expected result
ClientOptions should extends fetch options.
Checklist

[ ] I’m willing to open a PR (see CONTRIBUTING.md)

Just published a new version that fixes a number of type issues. Can you see if the issue persists on the new version? Thanks!


Add ruby validator (mri)

Hi there!
Thanks for building validator.vim. :) I've just added it to my setup and so far it seems like a great replacement to syntastic.
As the title says, this adds support for ruby validations.
Feel free to comment on anything I can do to improve this PR.

Thanks for merging! :)


feat: support 'xhrTimeout' option

You'd better use a more general name, like requestTimeout or networkTimeout, since we are using fetch + ReadableStream on most platforms rather than XHR-based for now.
BTW, what time is your timeout pointing to?
The elapsed time from the request sent until the response header is received?
Or the elapsed time of downloading the full file?

I have renamed the xhrTimeout option to requestTimeout, it stands for The elapsed time from the request sent until the response header is received.
Why we need such option?
Because browsers like Chrome require 20 seconds to throw an error when accessing an unreachable address, sometimes we prefer to a faster response.

Have you confirmed that a slow download (where the response header is received) won't cause a timeout in XHR?

Thanks for pointing out my mistake. According to the XHR Standard https://xhr.spec.whatwg.org/#the-timeout-attribute, even if a response header is received, a timeout error will be dispatched as long as the request has not been completed, and I have already tested such situation.
So I implemented requestTimeout option using setTimeout instead.


Post Variant Calling Metrics ID

Because all Post Variant Calling Metrics are with filter PASS, we probably want to simpify the metrics ID
for example here
https://github.com/ga4gh/quality-control-wgs/blob/9ad37fc8dae317d670e299a005b0ef8ad06cedf9/metrics_definitions/metrics_definitions.md?plain=1#L99
change count_snvs_pass to count_snvs

Metrics ID amended to the following..

count_snvs
count_insertions
count_deletions
ratio_insertion_deletion
ratio_heterozygous_homzygous_snv
ratio_heterozygous_homzygous_indel
ratio_transitions_transversions


Test: Getting Started framework

Refs: https://github.com/microsoft/vscode/issues/106715

[ ] anyOS @kieferrm
[ ] anyOS @sana-ajani

Complexity: 3
Authors: @misolori
Create Issue

This iteration we worked on creating a framework for displaying a "getting started" experience. The content is still very much a work in progress, but the framework aspect is ready for initial testing.

"workbench.experimental.gettingStarted": true (also try "workbench.startupEditor": "gettingStarted")
Locate editor via Help -> Getting Started or opening a new empty window, depending on how you've enabled it.
Ensure layout is reasonable.
Test that UI state persists across reloads, moving tabs, etc.
Test that progress bars work as expected (can be reset with Getting Started: Reset Progress)
Generally test that things work as expected.

This looks awesome Jackson! I'm really excited to see this experience come together.


build: use terser 5.5.1

This PR updates Terser from version 4.7.0 to 5.5.1 and changes few lines in the build scripts (adding await and etc according to Terser changes). As a result: swiper-bundle.min.js went from ~140kb to ~137kb

Thanks!


vso-convertx-to-dvd: Add version <IP_ADDRESS>

I am not really fan of all these trials beign added to official bucket.
@r15ch13, @rasa Any idea how this could be handled. Easiest solution is just trial boolean property.

I'm having my own complete license of thoses softwares, I'm just trying to have my full set of software available from scoop. I you say that thoses are not fitting the official repo, what I completely understand, I should create my own repo then.
I was mistaken by the LGPL file left in the folder after installation.

is there a variable to manage the "7" in the autoupdate url ?

See fixed manifest.

sorry I did not saw because it was collapse.

@Ash258 Do you want me to stop sending pr for softwares that requires a license to unlock full capabilities ?

Hello, is there still any issue preventing the merge of this pull request?

Closing as branch has been deleted.


README.md: ~Makes it easier to edit~ Regenerates contributors

This doesn't change how it looks like, but makes it easier for ourselves to edit.
Before

After

!? I didn't merge yet...

Thanks for your interest in this feature!
If using all-contributors makes maintaining this repository complex, we can drop it! 🙏


Windows build fails with 2.7.1

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Windows build segfaults with 2.8.6, see https://github.com/conda-forge/nauty-feedstock/pull/22


[Feat] Message verification and signing

Hey,
I think adding message verification and signing would be a nice addition to this library.
As I need to sign some content using the users Wif or private key for a DApp v2.
Wouldn't mind implementing this.
An API along the lines
const signature = Neon.create.signMessage('PrivateKeyOrWiff', 'Hello World')
const verified = Neon.create.verifyMessage('NeoAddress', 'Hello World', signature)

or
const account = new wallet.Account(privateKey)
const signature = account.signMessage('Hello World')
const verified = Neon.get.verifyMessage('NeoAddress', 'Hello World', signature)

Thoughts or suggestions?

probably Neon.sign.message and Neon.verify.message would be good. I think you probably can reuse some of the code for signing but verification im not so sure about that. But looks like something useful for a library!

@Said007 any progress? Do check in so I can know if you are working on it!

@snowypowers Been quite busy, I have working prototype which signs and verifies using elliptic.js which is pretty much very similar to generateSignature function.
I need to write tests and do some clean ups on my code. Will make a pull request latest by end of next week.

no worries! Just checking in so we know whats being worked on.
In the meantime, would be good to have a sketch of what to expect:

Method signature
Method name
location within the package

Within the wallet module.
message.js
/**
 * Generates a signature of the message based on given private key.
 * @param {string} message
 * @param {string} privateKey
 * @return {string} Signature
 */
export const signMessage = (message, privateKey) => {
  /// Sign and generate message signature using elliptic p256
}

/**
 * Verifies signature matches message and is valid for a given public key.
 * @param {string} message
 * @param {string} publicKey
 * @param {string} signature
 * @return {boolean}

 */
export const verifyMessage = (message, publicKey, signature) => {
  // verify publicKey, message, signature match using elliptic p256
}

What do you think? any remarks?

Looks good. Try to accept both formats for private key (WIF and HEX). Similarly, you should check and accept both forms of public key too (encoded and unencoded)

Thanks for your contribution!


PKGBUILD for Archlinux

First of all thank you for this tool.
I just by mere luck bumped into it after a long search for similar tools. Tools that work in the  Linux Console (not only X11). Examples like, Triggerhappy, hkd (alemani.eu), kmonad , see my odissey here, https://aur.archlinux.org/packages/hkd-git.
As soon as I saw "suckless", "config.h", and a clean understandble README (for end users, not linux input/evdev or C coders experts) I knew it was my lucky day.

... See [linux/input-event-codes.h]
(https://github.com/torvalds/linux/blob/master/include/uapi/linux/input-event-codes.h) for a list of available key codes ...

This is the kind of simple sentence, that doesnt cost much to write in a README, and makes life so easier for end users. Instead of wasting time on the Internet, reading things meant for developers.
I'm on Archlinux. I'm not a developer, just a competent command line end user. Framebuffer and console fanatic. Tested immediately and works like a charm.

Question 1: Do you, yourself, use it daily ? Plan on maintaining it long term ?

This kind of tool is also interesting in small  devices, with a small keyboard or just  a simple numeric keypad attached per usb.

Question 2: Since it's C coded it will probably compile everywhere right ?

I'm gonna test it in ARM (Raspberry Pi Archlinux, armv7h etc).  Should work also, and I'll report back.
A big dream would be if it would compile in MIPS architecture, for a small router (with an USB port).  If it could some day make it into  Openwrt, as an official repo package. That wouldextremely usefull for running custom actions assigned to keyboard events, https://openwrt.org/docs/guide-user/hardware/hardware.button .
All these are information that you might want to hint to in the README.
I noticed you seem to be also and Archlinux user. But there isnt' a PKGBUILD for this tool.

Question 3: Could you please provide one, and publish it on the right place, the  AUR system.

The PKGBUILD would  probably also need to:

add a systemd wide service ?
include the README, example "config.h" and eventually other docs to install in /usr/share/doc/hkd-aaron
make sure to use "arch=('x86_64' 'armv7h')" or even better "arch=('any')"

Thanks in advance.
PS: I have, as an end user, other questions, but I'll writed detailed about it in another post.  Maybe as suggestions for the README or some OVERVIEW docs, ex:,
This whole "bazaar" "rocket science" linux input system, evdev, console v. X11, "hotkeys" is not the friendliest terrain  for us end users to get a clean overview.
I've avoided diving into "interception tools" till know, because although it seems well maintained and actively developed, it lacks a clean straight forward README or OVERVIEW. For end users, not the ones programming it. I have difficulties "fitting" it into my head.

I am so sorry I missed this until now! I just happened to check back on the repo and saw this wonderful feedback.

I just by mere luck bumped into it after a long search for similar tools.
As soon as I saw "suckless", "config.h", and a clean understandble README (for end users, not linux input/evdev or C coders experts) I knew it was my lucky day.

... See [linux/input-event-codes.h]
(https://github.com/torvalds/linux/blob/master/include/uapi/linux/input-event-codes.h) for a list of available key codes ...

This is the kind of simple sentence, that doesnt cost much to write in a README, and makes life so easier for end users. Instead of wasting time on the Internet, reading things meant for developers.
I'm on Archlinux. I'm not a developer, just a competent command line end user. Framebuffer and console fanatic. Tested immediately and works like a charm.

This was amazing to read. Someone found my tool independently, used it, and it worked.... What a feeling.

* Question 1: Do you, yourself, use it daily ? Plan on maintaining it long term ?

I still do use this on my main PC. I haven't had to maintain it at all for a while, but if I run into a bug and fix it, I will merge the changes here.

* Question 2: Since it's C coded it will probably compile everywhere right ?

I hope so, let me know if it doesn't.

A big dream would be if it would compile in MIPS architecture, for a small router (with an USB port). If it could some day make it into Openwrt, as an official repo package. That wouldextremely usefull for running custom actions assigned to keyboard events, https://openwrt.org/docs/guide-user/hardware/hardware.button .
All these are information that you might want to hint to in the README.
I noticed you seem to be also and Archlinux user. But there isnt' a PKGBUILD for this tool.
* Question 3: Could you please provide one, and publish it on the right place, the  AUR system.

I am an Arch user, but I have no experience creating pkgbuilds or adding things to the AUR. Given that this is a suckless-style program which requires edits to the source code for configuration, I hadn't planned on adding something like this. I think my first priority if I were to put significant further work into this project would be to add useful features, but I am not against the AUR idea if someone else (you seem to be knowledgeable?) wanted to do this.

PS: I have, as an end user, other questions, but I'll writed detailed about it in another post. Maybe as suggestions for the README or some OVERVIEW docs, ex:,

Hit me

This whole "bazaar" "rocket science" linux input system, evdev, console v. X11, "hotkeys" is not the friendliest terrain for us end users to get a clean overview.

Yes.

I've avoided diving into "interception tools" till know, because although it seems well maintained and actively developed, it lacks a clean straight forward README or OVERVIEW. For end users, not the ones programming it. I have difficulties "fitting" it into my head.

I'm with you on the user friendliness of interception tools. That's why I put in some effort removing hkd's dependency on that library. Let me know if you have questions about interception tools though and I might have an answer.


function are duplicated when name contains underscores.

When there is IAB21_1 and IAB2_11 ScalaPB generate function without underscore that make problem
method isIab211 is defined twice
The proto exemple can be found here

Are you on a recent 0.5.x release? This seems like a duplicate of #94

I'm using addSbtPlugin("com.trueaccord.scalapb" % "sbt-scalapb" % "0.4.20")as seen in README.md

I have follow the information found in the exemple project tu set the build.sbt and the scalapb.sbt of the 0.5.29 tag
And indeed the bug was already fixed !
Thanks