NEST Thermostat "blip" caused removal of thermostats from HomeKit

Expected Behavior

Current Behavior

I was actively working with my Indigo setup when I recieved warnings about my two NEST thermostats being removed from HomeKit Bridge.  I looked at the devices and they were still present in Indigo, but I noticed that they had blanked out their values for most thermostat relavent devices like the current temp, set points, modes, etc.  There were no errors in the logs pertaining to the NEST plugin.
Steps to Reproduce (for bugs)
Have not been able to reproduce yet.
Versions

Plugin: 1.0.0 rc 1
NEST: 2.0.50

Configuration
Let me know if you need this.
Homebridge Log / Command Output

HomeKit Bridge Warning          Indigo device Downstairs Thermostat was removed and is linked to HomeKit, removing from any impacted servers
   HomeKit Bridge                  Caching all HomeKit Bridge devices...
   HomeKit Bridge Error            Exception in homekit.special_nestHvacMode line 1805: 'key id<PHONE_NUMBER> not found in database'
							 CODE: obj = indigo.devices[self.objId]

For object id<PHONE_NUMBER> alias 'Upstairs Thermostat'
   HomeKit Bridge Error            Exception in homekit.special_thermTemperatureSetPoint line 1921: 'key id<PHONE_NUMBER> not found in database'
							 CODE: obj = indigo.devices[self.objId]

For object id<PHONE_NUMBER> alias 'Upstairs Thermostat'
   HomeKit Bridge Error            Exception in homekit.detCharacteristicValues line 1003: 'key id<PHONE_NUMBER> not found in database'
							 CODE: obj = indigo.devices[self.objId]

For object id<PHONE_NUMBER> alias 'Upstairs Thermostat'
   HomeKit Bridge Error            Exception in homekit.detCharacteristicValues line 1000: 'key id<PHONE_NUMBER> not found in database'
							 CODE: if getter.replace("attr_", "") in dir(indigo.devices[self.objId]): hasvalue = True

For object id<PHONE_NUMBER> alias 'Upstairs Thermostat'
   HomeKit Bridge Warning          Indigo device Upstairs Thermostat was removed and is linked to HomeKit, removing from any impacted servers
   HomeKit Bridge                  Caching all HomeKit Bridge devices...
   HomeKit Bridge                  Caching all HomeKit Bridge devices...
   HomeKit Bridge                  Attempting to stop 'HomeKit Bridge'
   Sonos                           SoCo Reactor Ignition...
   HomeKit Bridge                  HomeKit server 'HomeKit Bridge' has been stopped
   HomeKit Bridge Warning          Device ID<PHONE_NUMBER> (Upstairs Thermostat) is linked to server 'HomeKit Bridge' but that device ID no longer exists in Indigo and will be removed from the configuration
   HomeKit Bridge                  Caching all HomeKit Bridge devices...
   HomeKit Bridge                  Attempting to start 'HomeKit Bridge'
   Grafana Home Dashboard          a value (-32768.0) for device Front Porch Luminance was not in the configured range (-10000.0, 10000.0) and will not be sent to InfluxDB
   HomeKit Bridge                  Server 'HomeKit Bridge' has successfully started, you can use your HomeKit apps or Siri for this accessory
   HomeKit Bridge                  Caching all HomeKit Bridge devices...
   HomeKit Bridge                  Attempting to stop 'HomeKit Bridge'
   HomeKit Bridge                  HomeKit server 'HomeKit Bridge' has been stopped
   HomeKit Bridge                  Attempting to start 'HomeKit Bridge'
   HomeKit Bridge                  Server 'HomeKit Bridge' has successfully started, you can use your HomeKit apps or Siri for this accessory
   Z-Wave                          received "Foyer Motion Sensor" status update is off

May 11, 2018 at 11:39:20 AM
   Error                           device "Downstairs Thermostat" state key humidityInput1 not defined (ignoring update request)
   Error                           device "Downstairs Thermostat" state key humidityInputsAll not defined (ignoring update request)
   Error                           device "Downstairs Thermostat" state key humidityInput1 not defined (ignoring update request)

Device Simulation Report
Let me know if you need this.

You are right about the device ID for the thermostats.  I just confirmed that they changed after this ordeal.  I did not update or actively do anything with the NEST plugin.  Yesterday I linked my NEST account with Haiku fans, but that was it.
The NEST plugin auto creates the devices from your NEST account.  There is ZERO useful information in the logs folder for the Nest plugin.  If I need to provide this bug over to Chameleon, I have very little to give him.
Luckily I dont have any triggers or actions for those devices, otherwise, I'd have a fun time ahead making the cascading changes.  I only set home/away status via the Home Master, which didn't change.
So, this is not a bug with HomeKit Bridge it would seem?

No, I don’t believe it is for a number of reasons - for one it doesn’t have the ability to delete another plugin’s device and re-create it nor would I ever have a reason to write that into the plugin.  I also run Nest and haven’t had anything like that happen to me yet, but when I upgraded it late last year it wiped out my Nest devices and recreated them and it did reach havoc on my system until I manually fixed the bunch of triggers and actions that referred to the Nest.

FYI - https://forums.indigodomo.com/viewtopic.php?f=170&t=20810&p=161582#p161582


Add Experimental Plugin Support for the Techdocs backend plugin

Signed-off-by: Oleg S<EMAIL_ADDRESS>Hey, I just made a Pull Request!

This pull request exposes the techdocs-backend plugin using the new plugin system.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

This pull request is a part of #13084

@jhaals I've changed the plugin from techdocs-backend to techdocs to be consistent with the rest of the project. Was there anything else that you needed from this PR?

Someone else got in before you to rename the todo-backend. You can revert that, so that the bot doesn't complain that you don't have a changeset for it :)

@freben Fixed


[WIP] Add UID and GID to the user object from user package

Currently, the UID and GID for users need to be read from the local system which requires local users to be present. This change retrieves that information from the user and auth packages and adds methods to retrieve it.
Closes #964

@labkode the PR is ready for review. Some initial testing is done, working on deploying it for further tests.
One thing which I'm not very sure about is this part. Whenever we stat for resources, EOS returns the UID, which we encode as "uid:xyz" since we have no way of resolving it at the fs level. This is done at the gateway layer to get the user ID back from the UID. I modified the GetUser method to return the user based on UID passed as opaque.
Please let me know if this is fine or should we switch to some other approach.

@labkode the PR is ready for review. Some initial testing is done, working on deploying it for further tests.
One thing which I'm not very sure about is this part. Whenever we stat for resources, EOS returns the UID, which we encode as "uid:xyz" since we have no way of resolving it at the fs level. This is done at the gateway layer to get the user ID back from the UID using this method. I modified the GetUser method to return the user based on UID passed as opaque.
Please let me know if this is fine or if we should switch to some other approach.

What about having the EOS driver calling the userprovider grpc endpoints to resolve the uid to username and then pass it to the upper layers? This approach will avoid to have the uid: hack but will add a dependency to another grpc service. What do you think? Also @butonic ?

@labkode yes that could be an approach as well. Would save a lot of the changes I had to do at the upper levels.

@ishank011 let's try then that approach, the drivers that need extra user information for implementing their logic they can connect to the gateway and request for specific user metadata.
Can you adapt that part to contact the Gateway API instead?

@labkode Cool, I'll make the change. Thanks!

Alternate approach #995 merged.


CI/CD: add inclavare containers development centos8.2 and ubuntu 18.04 docker images

Signed-off-by: Yilin Li<EMAIL_ADDRESS>

build test: https://github.com/hustliyilin/inclavare-containers/actions/runs/610511798
the build of rune、shim、 epm、sgx-tools、skeleton is ok.
the docker is installed in the docker images.
the rpm and deb build don't need to be installed any packages.

* build test: https://github.com/hustliyilin/inclavare-containers/actions/runs/610511798

* the build of `rune`、`shim`、 `epm`、`sgx-tools`、`skeleton` is ok.

* the docker is installed in the docker images.

* the rpm and deb build don't need to be installed any packages.

shelter and ra-tls are not tested?

This docker image is created for inclavare development, so it is necessary to make shelter and ra-tls buildable. In addition, the workflow names should be renamed as 'Dockerfile-inclavare-dev-foo'.

* build test: https://github.com/hustliyilin/inclavare-containers/actions/runs/610511798

* the build of `rune`、`shim`、 `epm`、`sgx-tools`、`skeleton` is ok.

* the docker is installed in the docker images.

* the rpm and deb build don't need to be installed any packages.

shelter and ra-tls are not tested?

* build test: https://github.com/hustliyilin/inclavare-containers/actions/runs/610511798

* the build of `rune`、`shim`、 `epm`、`sgx-tools`、`skeleton` is ok.

* the docker is installed in the docker images.

* the rpm and deb build don't need to be installed any packages.

shelter and ra-tls are not tested?

Non-core components will be added later but not now.


Use XDocument.Save or XDocument.SaveAsync to write to response body when possible

With the new TwiML.ToXDocument() method (PR) from the Twilio C# helper library, we can now avoid serializing the XML to a big string before writing it to the response body.
ASP.NET Core does require writes to the response to be async, and in older .NET Core versions XDocument.SaveAsync is not available. In that case, the XML is serialized to a string and written to the response.
For ASP.NET, there's only XDocument.Save available but there's no need for asynchronous writes.

Additionally, I replaced null checks with string.IsNullOrEmpty for BaseUrlOverride because sometimes the setting is an empty string.
Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

Assuming PR twilio-csharp#669 is merged into Twilio helper library for .NET, the new ToString, Save, and SaveAsync would be more performant. Putting this on hold.


"Unknown provider: $compileProvider" when the angular creates an injector of my module with directives

Hi!
I have two code snippet:

first works fine

angular
    .module('app', [])
    .factory('foo', function() {
        return 'test';
    });

try {
    alert(angular.injector(['app']).get('foo'));
} catch(e) {
    alert(e.message);
}

second throws exception

angular
    .module('app', [])
    .factory('foo', function() {
        return 'test';
    })
    .directive('bar', function() {
        return {};
    });

try {
    alert(angular.injector(['app']).get('foo'));
} catch(e) {
    alert(e.message);
}

Question: Why second code sample throws exception?
Thanks =)

This is because you had not included the ng module (which contains the $compile service) in your list of modules to load into the injector.

@petebacondarwin thank you so much :smiley_cat:


MAX_BLOCK_SIZE / documentation outdated?

Here: https://docs.microsoft.com/en-gb/python/api/azure-storage-blob/azure.storage.blob.blockblobservice.BlockBlobService?view=azure-python it's specified that Each block can be a different size, up to a maximum of 4 MB. In the sdk (blockblobservice.py) it says: Each block can be a different size, up to a maximum of 100 MB. The variable that (I guess) corresponds to this specifies 4MB (MAX_BLOCK_SIZE = 4 * 1024 * 1024).
Dunno which is correct as I just started to play with the sdk, but I guess something needs to be updated.

@AljoSt This is a Storage issue, wich is handled in its own repo for now.
@zezha-msft now you can transfer issue to your repo with Github, look at the "Transfer this issue [Beta]" button on the bottom right. I would encourage you to do it here :)

Hi @AljoSt, the max block size limit is 100MB, the docs need to be regenerated. Thanks for bringing up this issue!
@lmazuel I do not see the issue transfer button, could you please transfer it to the Storage repo?

@zezha-msft I need admin permissions on your repo to do so. I won't use that power to do any harm, I promise ;)

Hi @lmazuel, sure! Sorry for the late reply, I've been OOF.
@pemari-msft Could you please add @lmazuel to our repo?


Improve error messages for Accuracy interfaces

What does this PR do?

adds a MisConfigurationError to utilities
uses MisConfigurationError to provide users a configuration error message in the functional and module interface of accuracy if the user passes in an incorrect task.
removed asserts in each interface and replaced with MisConfigurationError

The PR concept was discussed in Slack/new-contributors and acknowledged by Jirka.
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

fyi, pre-commit will be fixed in #1464

just the typing :)

@Borda I plan to work on the typing next week.

@Borda @justusschock I decided to update the return type to Optional[Tensor] because the final case returns None. The pre-commit-ci bot added the return value 4 weeks ago, and that caused mypy to fail for the return value.
Is it better to have updated the expected return type, or to update the CI bot to not automate this type of update?


ValidationError: Validation error for type any | null

Hello,
I tried to use the getLivestream(channel: string) function on my channel instance but i'm receiving this error and i don't know how to fix (sorry to bother you)
It seems that it cannot correctly parse the response but i can be wrong..
errors: [     ValidationErrorItem {       path: '',       code: 'type',       message: 'No valid union member found. Valid: any | null',       value: {         id: 22166273,         slug: '46347792-watch-blind-dating-free-therapy-for-viewer-calls-s-my-links',         session_title: 'WATCH BLIND DATING + FREE THERAPY FOR VIEWER CALLS 🍑  !s --> my links',         created_at: '2024-01-21T17:12:57.000000Z',         language: 'English',         is_mature: false,         viewers: 4981,         category: {           id: 15,           name: 'Just Chatting',           slug: 'just-chatting',           tags: [ 'IRL' ],           parent_category: { id: 2, slug: 'irl' }         },         playback_url: 'https://fa723fc1b171.us-west-2.playback.live-video.net/api/video/v1/us-west-2.196233775518.channel.phlvZbvP2gwW.m3u8?token=eyJ0eXAiOiJKV1QiLCJhbGciOiJFUzM4NCJ9.eyJhd3M6Y2hhbm5lbC1hcm4iOiJhcm46YXdzOml2czp1cy13ZXN0LTI6MTk2MjMzNzc1NTE4OmNoYW5uZWwvcGhsdlpidlAyZ3dXIiwiYXdzOmFjY2Vzcy1jb250cm9sLWFsbG93LW9yaWdpbiI6Imh0dHBzOi8va2ljay5jb20saHR0cHM6Ly9wbGF5ZXIua2ljay5jb20saHR0cHM6Ly9hZG1pbi5raWNrLmNvbSxodHRwczovL3d3dy5nc3RhdGljLmNvbSIsImF3czpzdHJpY3Qtb3JpZ2luLWVuZm9yY2VtZW50IjpmYWxzZSwiZXhwIjoxNzA1OTAzODc1fQ.7UQg7eC6l8lFS3ScsQ44CYUpMdkFCgui70omNXvX9ljv68e2GnQW-Cu--5Q3KS1fM9q6Oj6JAy3NOsMRdMegr2gfB-NjEcZExPbWFH2QtQDoQ7Pwa9YuzkxLQj8aFklh',         thumbnail: {           src: 'https://images.kick.com/video_thumbnails/phlvZbvP2gwW/rZTzXpK0yoLQ/720.webp',           srcset: 'https://images.kick.com/video_thumbnails/phlvZbvP2gwW/rZTzXpK0yoLQ/1080.webp 1920w, https://images.kick.com/video_thumbnails/phlvZbvP2gwW/rZTzXpK0yoLQ/720.webp 1280w, https://images.kick.com/video_thumbnails/phlvZbvP2gwW/rZTzXpK0yoLQ/360.webp 480w, https://images.kick.com/video_thumbnails/phlvZbvP2gwW/rZTzXpK0yoLQ/160.webp 284w, https://images.kick.com/video_thumbnails/phlvZbvP2gwW/rZTzXpK0yoLQ/480.webp 640w'         }       }     }   ] }

Hi @0xHal,
Thanks for the report. Not sure what could be happening, but it looks like @deepkit/type isn't getting any type information. I've had a few refactors internally and I don't have any issues with my current codebase.
I've published version to 1.0.1-9, could you let me know how it goes?
I've somewhat stopped working on Kient until the full API releases, hopefully very soon.

It works fine now!
Thanks man.
This problem seems to happen elsewhere too:
errors: [
ValidationErrorItem {
path: 'data.livestream.thumbnail',
code: 'type',
message: 'Not an object',
value: undefined
}
]
}
This problem only happen on the following kick channel: roshtein
When i try with other channel it works fine.
I can't wait for their API to be released but i'm very happy to work with your custom API.
Thanks a lot for your work!

If you could teach me how you deal with these validation maybe i could help you with PR instead of asking and maybe bother you.

It works fine now!
Thanks man.
This problem seems to happen elsewhere too:
errors: [
      ValidationErrorItem {
        path: 'data.livestream.thumbnail',
        code: 'type',
        message: 'Not an object',
        value: undefined
      }
    ]
  }

This problem only happen on the following kick channel: roshtein
When i try with other channel it works fine.
I can't wait for their API to be released but i'm very happy to work with your custom API.
Thanks a lot for your work!

So essentially the issue is that in get-livestream.response.ts we are expecting that the thumbnail be in an object that contains src and srcset, in this circumstance there is no thumbnail object and it is instead returning undefined.
So, to fix this, we would need to mark the thumbnail property as optional and deepkit/type will figure out the rest.

It works fine now!
Thanks man.
This problem seems to happen elsewhere too:
errors: [
      ValidationErrorItem {
        path: 'data.livestream.thumbnail',
        code: 'type',
        message: 'Not an object',
        value: undefined
      }
    ]
  }

This problem only happen on the following kick channel: roshtein
When i try with other channel it works fine.
I can't wait for their API to be released but i'm very happy to work with your custom API.
Thanks a lot for your work!

So essentially the issue is that in get-livestream.response.ts we are expecting that the thumbnail be in an object that contains src and srcset, in this circumstance there is no thumbnail object and it is instead returning undefined.
So, to fix this, we would need to mark the thumbnail property as optional and deepkit/type will figure out the rest.

Thank again for your time and your explanations.
I took a look on the index.js but it seems that i will not be able to fix this problem following this path.
I made a change is it correct?

Maybe thumbnail {...} | undefined was enought

Published version 1.0.1-10 to npm to fix above issue.
Any further validation issues, let me know. Due to how deepkit/type works, the types are compiled into the build, unfortunately just editing the type declarations isn't enough.

Nice, yes i had another bug too
[KientSomethingWentWrong: Unknown error has occured] {
[cause]: ValidationError: Validation error for type BannedUserInstance:
data.expires_at(type): Not a Date
    errors: [       ValidationErrorItem {         path: 'data.expires_at',         code: 'type',         message: 'Not a Date',         value: undefined       }     ]   } }
Thanks for all the updates you'r making!

I don't know which type is returned but maybe it's an number so here my change i'll test it

Pushed another fix for the validation issues, @0xHal. Kient version 1.0.1-11 has been published.
I didn't realise that Kick sent over different responses depending on permanent bans and timeouts. I've also added a helper function on the instance, isPermanent, to return a boolean depending on the ban type.


Stop GSM8k generation at double new line

This loosens the stopping criteria for GSM8k generation, to allow for a single newline which is sometimes generated at the start of an output, but stops at a double newline (as compatible with the prompt).
It's a bit hacky, but should work for tokenizers where "\n\n" is either one or two tokens. Often "\n\n" is indeed a single token, so this should actually be more effective than the previous "\n" stopping condition if the model actual does generate a double new line.  FYI @dwadden

I'm a little hesitant to merge this without checking if it changes our old evaluations. I wonder if a good solution would be to make the stop sequence configurable to avoid this issue. @yizhongw thoughts?

Yeah, I agree the backwards compatibility is potentially awkward. We could put it behind a flag which would preserve earlier behavior. I'm not sure how strict you feel about not changing earlier evals. E.g., I noticed a typo in line 56 ("Quesion: " + example["question"]), is that something you also wouldn't want to change now?

Yeah, I think in general this is an awkward thing. I don't want to make changes when papers are in motion to avoid having to re-compute old evals, but also its worth tracking and updating these things, while also trying to avoid making some large number of flags. Really, coming up with some sort of 'release process'/versioning setup for this would be nice...

I need this PR for evals I'm running now. Instead of changing the existing gsm task, could we just add another one that allows newlines?

I added an option now so that this shouldn't change existing behavior unless the option --stop_at_double_newline is added to the run_eval call. E.g., in submit_eval_jobs.py would do this to get the new behavior:
    elif experiment_group == "gsm_cot":
        task_spec['arguments'][0] = '''
            python -m eval.gsm.run_eval \
            --data_dir /data/gsm/ \
            --max_num_examples 200 \
            --save_dir /output/ \
            --use_vllm \
            --model_name_or_path /model \
            --tokenizer_name_or_path /model \
            --n_shot 8 \
            --use_chat_format \
            --chat_formatting_function eval.templates.create_prompt_with_tulu_chat_format \
            --stop_at_double_newline
        '''

Thanks, thats fine! Probably not an ideal solution for all scenarios, but seems like a decent stopgap for now.

Perfect, thanks! I'll rerun evals with this.


WIP: Fix some issues in Travis and add more python env

This PR does some tasks:

[x] : Add requirements dependency in requirements.txt for algorithms.

[x] : Update cache in Travis

[x] : Switch to use tox for running the unittest, coveralls. Add more python environment for testing. It does not only test for python 3.6. It could probably test python 3.4, 3.5, 3.6

It makes sure that Algorithms can successfully run under all version in python3. That is reason why I want to create more environment to test. However, there are some minor issue in it. You can check the travis to see what the issues are. In particular, it pass all tests for python3.5.
But we got fail 1 case in python3.4:
 self.assertEqual(unique_array_sum_combinations(A, B, C, target), answer)
and test_PriorityQueue file.
And similarly for python3.6.
What do you @keon @goswami-rahul think about this idea? Could you help me fix the bugs?

Pull Request Test Coverage Report for Build 479

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+8.7%) to 98.629%

Totals

Change from base Build 477:
8.7%

Covered Lines:
1007

Relevant Lines:
1021

💛  - Coveralls

Here's the code for that function
def unique_array_sum_combinations(A, B, C, target):
    """
    1. Sort all the arrays - a,b,c. - This improves average time complexity.
    2. If c[i] < Sum, then look for Sum - c[i] in array a and b.
       When pair found, insert c[i], a[j] & b[k] into the result list.
       This can be done in O(n).
    3. Keep on doing the above procedure while going through complete c array.
    Complexity: O(n(m+p))
    """
    def check_sum(n, *nums):
        if sum(x for x in nums) == n:
            return (True, nums)
        else:
            return (False, nums)

    pro = itertools.product(A, B, C)
    func = partial(check_sum, target)
    sums = list(itertools.starmap(func, pro))

    res = set()
    for s in sums:
        if s[0] is True and s[1] not in res:
            res.add(s[1])

    return list(res)

It uses set internally, which doesn't have any particular order. We should test with sorted list or make set of the list while testing equality.
As for the PriorityQueue error, it uses collections.deque.insert which is new to Python 3.5.
I think we should change that code to Python python 3.4 compatible, or is it fine to support Python 3.5+ ?? @danghai @keon

In my opinion, I think the title of Algorithms from README: Minimal and clean example implementations of data structures and algorithms in Python 3. It should work fine all version of Python 3. If someone is using Python 3.4, they cannot run test command.

But the collections.deque.insert failed in Python 3.6 test too. Its weird!

It should work fine all version of Python 3.
Yeah, we would need some refactor in the PriorityQueue.

@goswami-rahul Yeah it is weird!
Traceback (most recent call last):
  File "/home/travis/build/danghai/algorithms/tests/test_queues.py", line 99, in test_PriorityQueue
    queue.push(1)
  File "/home/travis/build/danghai/algorithms/algorithms/queues/priority_queue.py", line 32, in push
    self.priority_queue_list.insert(index, node)
AttributeError: 'collections.deque' object has no attribute 'insert'

It is fail in both Python 3.4 and 3.6. But It passes Python 3.5 :)

I have Python3.6 and it (obviously) works in my PC!
In [6]: q
Out[6]: deque([1, 2, 3, 4])

In [7]: q.insert(1, 6)

In [8]: q
Out[8]: deque([1, 6, 2, 3, 4])

Does it work with command: python3 -m unittest tests.test_queues ?

@goswami-rahul I put comment in PriorityQueue, and then creating this issue #330
One of the possible solution is we need to implement the insert method by using function in collection.dequeue.
Travis should work fine now.


flag pkg can't read rune from command line like it is in default

What version of Go are you using (go version)?

$ go version
go version go1.13.6 windows/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\owner\AppData\Local\go-build
set GOENV=C:\Users\owner\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\owner\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
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
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\owner\AppData\Local\Temp\go-build985832958=/tmp/go-build -gno-record-gcc-switches

What did you do?

https://play.golang.org/p/JkumD8mS8ib
What did you expect to see?
I expect the three chars A 7 \u00FC displayed as they are read from the flaginput from the command line, just as they are from the default of that flag, or if they were hardcoded in a string in the src.
I've had this posted for a month on stack overflow with over 35 views and the 1 proposed solution was incorrect.
BTW I am running bash under windows
What did you see instead?

I don't see anything wrong?
» go run . -input A7ü

Using flag default: OK:
DEBUG: Testing rune: A 65
DEBUG: Testing rune: 7 55
DEBUG: Testing rune: ü 252

Using hard coded string: OK:
DEBUG: Testing rune: A 65
DEBUG: Testing rune: 7 55
DEBUG: Testing rune: ü 252

How did you type the input since you don’t have u with umlauts on the keyboard
Bill

On May 11, 2020, at 12:26 PM, Sean Liao<EMAIL_ADDRESS>wrote:

I don't see anything wrong?
» go run . -input A7ü
Using flag default: OK:
DEBUG: Testing rune: A 65
DEBUG: Testing rune: 7 55
DEBUG: Testing rune: ü 252
Using hard coded string: OK:
DEBUG: Testing rune: A 65
DEBUG: Testing rune: 7 55
DEBUG: Testing rune: ü 252
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

copy and paste

I don't think this is a question about Go.  I think it's a question about how your shell takes non-ASCII input and passes it to the program.

In newer versions of bash you can use $'\u00fcother text' to allow unicode escapes.
If you need to handle it inside your program take a look at the strconv.Unquote(Char) functions.

Tnx I was hoping it was not an issue/shell issue. Please close the ticket or do I have to do that?
Bill

On May 11, 2020, at 3:48 PM, D1CED<EMAIL_ADDRESS>wrote:

In newer versions of bash you can use $'\u00fcother text' to allow unicode escapes.
If you need to handle it inside your program take a look at the strconv.Unquote(Char) functions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks for following up.


New CL items at Revenants don't appear as purple

🐛 Bug Report
When you get a new CL slot, the item is outlined in purple to indicate so. At Revenants, this appears to have bugged out and it shows up as any other item. It definitely worked before though.
Image of bug below (Can verify its not a dupe)

This is fixed by PR #4676


Do we want to use a more stringent LD R-squared threshold?

(e.g. R2 > 0.8)

Please confirm if addressed via pipeline or in FE.

We've decided against this. We can re-visit after analysing the coloc results to determine suitable threshold.


Add support for memory viewer and disassembly code

Add support for memory viewer and disassembly code (including instruction stepping) for C/C++ language debugging.

Please file this feature request against the C/C++ debugger.

This feature is now covered by https://github.com/microsoft/vscode/issues/124163.


Use Blur Effect for Dialog Overlays

Is your feature request related to a problem? Please describe.
Avalonia itself now implements the Blur Effect. This can be used to approximate the overlay blur used in WinUI when a ContentDialog is shown.
However, this is a little tricky, in order for the blur to really work it can't be done on the overlay itself -- it must be applied to everything under it (as in WPF but WinUI was a lot easier here). This is a severe limitation; hopefully upstream gets AcrylicBrushes and the like that can blur in the future. That said, it should be possible to render the application window to a Bitmap, apply a blur to the bitmap, then show that blurred bitmap as the overlay backdrop underneath ContentDialog and the like.
It may even be possible to extract this as a "BlurOverlayControl" which will render what's behind it (simplest as the entire app window) to a bitmap and then blur it.
Describe the solution you'd like
As described above.
Describe alternatives you've considered
Wait for upstream to work more like WinUI.
Additional context

As far as I know ContentDialog doesn't use the acrylic brushes. I have experimented with and AcrylicPanel/Border type control using a ICustomDrawOperation in the past but its really tricky to get right, and I want to say had a measurable perf hit (though its been a long while since I did this so I don't really remember).

As far as I know ContentDialog doesn't use the acrylic brushes.

Huh, I was remembering incorrectly! I just checked UWP and you are correct: it doesn't blur the background like I thought it did. FluentAvalonia functions the same as UWP/WinUI.
I'm going to close this issue since I was mistaken on upstream functionality. If blur is desired it will be done on the app side.


label dual elements

label dual elements
It could be useful to have a function to show the form and the force diagram together with the dual elements labelled so that it is easy to see read them

This has been resolved in RV2, by the "show.forces" option in settings.
The corresponding edges can be accessed by looking up the dual edge of the form diagram, or the primal edge of the force diagram.


Add E5 to SDK

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.RedisEnterprise

@ArthurMa1978  could you please help with the  review here.


UI - Unable to create VM from ISO with custom disk offering

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION
KVM, Advanced zone with no security groups
Primary Storage - RBD
Secondary Storage - NFS

OS / ENVIRONMENT
Management - Ubuntu Server 20.04; Hypervisor nodes - Ubuntu Server 20.04 LTS
Hypervisor - KVM

SUMMARY

UI throws the below error when trying to create a VM from the ISO, and only when selecting the "Custom" disk offering.

STEPS TO REPRODUCE

Download any ISO into the secondary storage.

Create a Virtual Machine from the ISO and choose  disk offering with "Custom Disk Size".

Select any other details like Networks, SSH Key pairs etc., and click "Launch Instance".

The error appears in the notification section.

The Disk Offering specifications is the below.

EXPECTED RESULTS

The Virtual Machine to get created with user-specified disk size.

ACTUAL RESULTS

Threw
VM Creation failed. Volume size: -1GB is out of allowed range. Max: 1024 Min:1

@nvazquez The results through the API are below and is successful.
(cs-<IP_ADDRESS>) 🐱 > deploy virtualmachine networkids=bb0ab32f-b6a3-4f6a-a613-ad3e9bb1ef48 templateid=efd8c38f-36f4-48e1-9f81-84ae8e5801df serviceofferingid=232f42ef-3844-4b10-ab27-9656a56bc35a diskofferingid=c13cde39-351d-4e41-bd6e-75d432689666 zoneid=d5601a3e-ad22-4c60-af71-a5d6d03106ec hypervisor=KVM size=6 rootdisksize=6
{
  "virtualmachine": {
    "account": "admin",
    "affinitygroup": [],
    "cpunumber": 1,
    "cpuspeed": 1000,
    "created": "2022-06-08T18:21:19+0000",
    "details": {
      "rootdisksize": "6"
    },
    "diskofferingid": "c13cde39-351d-4e41-bd6e-75d432689666",
    "diskofferingname": "Custom",
    "displayname": "VM-39ba2f9e-2687-4d4c-bb2c-f0b2f52c6102",
    "displayvm": true,
    "domain": "ROOT",
    "domainid": "7b078fde-e698-11ec-b1ed-aa0844d5f780",
    "guestosid": "7b12be68-e698-11ec-b1ed-aa0844d5f780",
    "haenable": false,
    "hasannotations": false,
    "hostid": "8904c92b-f4c9-4800-b74e-08119a184468",
    "hostname": "cs-2",
    "hypervisor": "KVM",
    "id": "39ba2f9e-2687-4d4c-bb2c-f0b2f52c6102",
    "instancename": "i-2-50-VM",
    "isdynamicallyscalable": false,
    "isodisplaytext": "Endeavour OS ISO",
    "isoid": "efd8c38f-36f4-48e1-9f81-84ae8e5801df",
    "isoname": "Endeavour OS ISO",
    "jobid": "6fa5d761-2432-462d-aec4-46f97a2a0fd3",
    "jobstatus": 0,
    "lastupdated": "2022-06-08T18:21:25+0000",
    "memory": 1024,
    "name": "VM-39ba2f9e-2687-4d4c-bb2c-f0b2f52c6102",
    "nic": [
      {
        "broadcasturi": "vxlan://700005",
        "deviceid": "0",
        "extradhcpoption": [],
        "gateway": "<IP_ADDRESS>",
        "id": "a2ac399e-de54-4473-9a73-64fc3cfb30e9",
        "ipaddress": "<IP_ADDRESS>",
        "isdefault": true,
        "isolationuri": "vxlan://700005",
        "macaddress": "02:00:01:bc:00:06",
        "netmask": "<IP_ADDRESS>",
        "networkid": "bb0ab32f-b6a3-4f6a-a613-ad3e9bb1ef48",
        "networkname": "net",
        "secondaryip": [],
        "traffictype": "Guest",
        "type": "Isolated"
      }
    ],
    "osdisplayname": "Other Linux (64-bit)",
    "ostypeid": "7b12be68-e698-11ec-b1ed-aa0844d5f780",
    "passwordenabled": false,
    "pooltype": "RBD",
    "receivedbytes": 0,
    "rootdeviceid": 0,
    "rootdevicetype": "ROOT",
    "securitygroup": [],
    "sentbytes": 0,
    "serviceofferingid": "232f42ef-3844-4b10-ab27-9656a56bc35a",
    "serviceofferingname": "Medium Instance",
    "state": "Running",
    "tags": [],
    "templatedisplaytext": "Endeavour OS ISO",
    "templateid": "efd8c38f-36f4-48e1-9f81-84ae8e5801df",
    "templatename": "Endeavour OS ISO",
    "userid": "8ddf2175-e698-11ec-b1ed-aa0844d5f780",
    "username": "admin",
    "zoneid": "d5601a3e-ad22-4c60-af71-a5d6d03106ec",
    "zonename": "hpe-ec3"
  }
}

@zap51 thanks for confirming it, that is a workaround from the API. Another workaround from the UI is creating a fixed disk offering matching the expected disk size. This UI issue for custom disk offerings will be addressed on 4.17.1

@nvazquez Thanks!


Strings of 26 bytes or more get packed as str8. Strings less than 32 bytes should be fixstr.

https://github.com/hirotakan/MessagePacker/blob/2bd864d286e4761fe5c456dba568888cd9ced111/Sources/MessagePackTypes/MessagePackType%2BStringType.swift#L22
A string of e.g. 26 bytes will be packed in fixstr, though the spec says fixstr stores a byte array whose length is up to 31 bytes. Seem that the limit is wrong.
https://github.com/msgpack/msgpack/blob/master/spec.md#str-format-family
In our use case, our iOS and Android apps end up serializing the data differently and signatures don't match. We are using the Java Msgpack library on Android.

@jkjuopperi
I fixed it with this PR and released v0.4.2.
Please try a new version.
Thank you!!


Fix record nested value de/serialization

While working on the Llama-3 implementation I stumbled upon a memory issue when importing pytorch weights with PyTorchFileRecorder.
When I profiled the memory usage for ResNet-152 (checkpoint is 252MB on disk), I saw a huge peak memory usage for what is supposed to be a relatively small model. Up to ~5GB as pointed out by the heaptrack trace below.
Before

After

Checklist

[x] Confirmed that run-checks all script has been executed.

Changes
Added U16s and F32s variants for NestedValue so weights can be parsed as a vector of primitive types instead of Vec<NestedValue>. For example, a vec of f32s is now represented as Vec[v, v, v, ...] instead of Vec[NestedValue::F32(v), NestedValue::F32(v), ...]. The NestedValue enum has a size of 56 bytes so it can grow very rapidly (just imagine for a very large number of parameters like in LLama 8B 🤯 ).

Handle different vec types in Serializer based on the input element type
Make VecSeqAccess's iter generic and add concrete implementations for vec of NestedValue, u16 and f32

Testing
All unit tests pass, including half precision record tests in burn-import/pytorch-tests.

Was running the checks locally and test record::serde::ser::tests::test_param_serde just failed.. will fix.

Before merging, please file uncompleted refactor or fixes.

If we do #1773 alone, then we can deprecate serialization. So you don't need to do other accumulating item types.

If we do #1773 alone, then we can deprecate serialization. So you don't need to do other accumulating item types.

Sure, we can limit the current PR to Vec<u16> (for f16 and bf16) and Vec<f32> (for pretty much all other parameter weights).
The linked issue should capture all element types when we tackle it.


A few strings not included in translation template

I made a collection of all strings not included in the translation file. This may be helpful but is not very important. After all, most strings are translated.
Note that I didn't add any trailing spaces. The string may need to be adjusted afterwards.

CB Panel(Double-click on a CB)

Edit Command Block
Command:
Custom Name:
Track Output

Chest panel

Slot
Slot:
ID/ID Name
ID/ID Name:
Unknown Item
DMG
DMG:
Count:
New Item (1.7+)
Delete This Item
Delete All Instances Of This Item From World

Suggest: Delete ": ", only use word without colon.
(Currently using colon before input areas, w/o colon in chart header)
"Count"(w/o colon) is already included in trn

CB COMMANDS in selection tool

No command blocks found

Analyse in selection tool

Analysis

spawn

Click to set the spawn position.

Below are strings included in trn but not implemented

World Info

Randomseed:

"Randomseed"(w/o colon) shouldn't be translated. It's an NBT tag.
Below are some other minor suggestions related to strings

Currently using "[RECOMMENDED]" in REGENERATE UUID, but "(Not Recommended)" in SESSION LOCK. unify the brackets and cases may look better.
Currently using "Keep data intact" in Fill/Replace, but it only works in Replace mode. Someone can add "replace only" to the hovertext.

Thank you for the report.
I was also tracking the missing translations, and will continue before pushing an update.
You'l be informed when it'll be done :)
Concerning "Randomseed" (without colon), it is not present in the teranslations, AFAIK. And I'v remarked that the correct syntax for NBT data is "RandomSeed".
By the wat, the NBT tags (and more generaly everything in a tree widget) aren't sent to the translation functions ;)
Concerning "Keep data intact" not working in fill mode, it was a bug...
Finaly, we'll have a look to harmonize some strings.

"Click to set the spawn position." Is not included. I just moved it out of "not implemented" subtitle.

The missing strings has been added int hte latest translation update (https://github.com/Khroki/MCEdit-Unified/commit/5ec4cdd2f3facd2b6c277a2b13fdd10aee085fec).
If you find more missing strings, just reopen this issue and report!

Ok. Thank you!  This cannot be undone. Really delete all copies of this item from all land, chests, furnaces, dispensers, dropped items, item-containing tiles, and player inventories in the world?

Concerning string harmonization, string632 "Damage" & string864 "DMG" can be unified.
Besides, I want to know...
what is a Blocks-Only/Fast Increment modifier,
what is button 3,
what does "No key means right click to fast nudge." mean.

reopen

open

open #445

reopen #445

Thank you for the report :)
Button 3 is the mouse middle button.
"No key means right click for fast nudge" means the if no key is defined for this action, holding the right mouse button will permit to move a selection point or face or the whole selection (or an imported/copied object) to be moved faster.
The "Fast increment modifier" is used in the numerical fields to increment the value faster.

Blocks-Only Modifier:
E = Rotate schematic
Alt+E = Rotate all the blocks in the schematic (without rotating the schematic)

Got it! Thank you guys, it helps me a lot to understand the software!

@LaChal Please consider add "(hold)" to "Fast increment modifier".
There are 2 Mega Spruce Taiga(o461 & 462. o462 has a trailing whitespace). The latter one should be Mega Spruce Taiga Hills.
BTW, I can't figure out how exactly "Snap clone to axis" works.

@fhfuih: with "" do you mean an empty string?
Since a modifier can only be Ctrl, Alt or Shift, it seem obious to hold these keys to modify something. But your remark is good since the other modifier keys contain this "(hold)".
Concerning the string 462, it will be corrected.
For now, feel free to add the correct translations to the erroneous strings. They will be corrected later.
"Snap clone to axis" is used when clicking a face of a cloned object and dragging it. It makes the clone to move only on the axis you first moved it, until you release the "Snap clone to axis" key.
"Swap axes looking down", like the tooltip says (string 158), inverts the forward and backward keys whe the camera looks down.

@LaChal the empty"" was meant to be''. I made a typo.

"Toolbar" with a pair of <


Add support form vim style key bindings

It would be really great to have support for vim-style key bindings!

I assume you mean in the request body editor? 😄
Let me know which keys in particular you would like.

That is correct!
I grabbed this bit from https://www.freecodecamp.org/news/vim-key-bindings-reference/
I think most of these could be added without too much fuss. I imagine things like insert vs visual mode might be more difficult.
Moving to Top and Bottom
You can use these keys to move to the top or bottom of the page:
gg  --> Move to the first line of the page
G   --> Move to the last line of the page
How to Edit Text in Normal Mode
Deletion and Copying
You can delete or copy characters using these keys:
x   --> Delete the character under the cursor
dd  --> Delete the entire line
yy  --> Copy (yank) the entire line
p   --> Paste the previously deleted or copied text after the cursor
Undo and Redo
u          --> Undo the last action
Ctrl + r   --> Redo the undone action
Thanks!

These binds would require the addition of an insert mode though. Quite a few Vim bindings already work in the response area since that's read only. However, I would need to add a mode system in order to support the ones you've listed in the request area.
Possibly in the future I'll look at this, but for now you can easily edit the request in Vim by pressing f4 (assuming you have your EDITOR env var set).

Excellent, thanks for that tip!

It would be great to move 10 lines up/down by pressing ctr+d/ctrl-u like vim.

I've added ctrl+d and ctrl+u in 1.2.0.

IMHO, due request body is read-only, I think other keybindings for edit shouldn't be included, for editing you can just open the editor. Said that, and the fact that basic movements are already implemented, this issue could be mark as done

Would love at least basic vim support for the URL bar,

t would be great to move 10 lines up/down by pressing ctr+u/ctrl

I like the F4 philosophy, vim emulation for editing is always going to be limited and/or would require significant efforts so why not just using (neo)vim ?
But F4 doesn't seem to work for the URL bar, right ? Would it be possible to enable it there ? It's rather frustrating to have to reach out for the mouse in the terminal.

Would love at least basic vim support for the URL bar,

t would be great to move 10 lines up/down by pressing ctr+u/ctrl

I like the F4 philosophy, vim emulation for editing is always going to be limited and/or would require significant efforts so why not just using (neo)vim ?
But F4 doesn't seem to work for the URL bar, right ? Would it be possible to enable it there ? It's rather frustrating to have to reach out for the mouse in the terminal.

ITOR env var set).

I like the F4 philosophy, vim emulation for editing is always going to be limited and/or would require significant efforts so why not just using (neo)vim ?
But F4 doesn't seem to work for the URL bar, right ? Would it be possible to enable it there ? It's rather frustrating to have to reach out for the mouse in the terminal.

These binds would require the addition of an insert mode though. Quite a few Vim bindings already work in the response area since that's read only. However, I would need to add a mode system in order to support the ones you've listed in the request area.
Possibly in the future I'll look at this, but for now you can easily edit the request in Vim by pressing f4 (assuming you have your EDITOR env var set).

I like the F4 philosophy, vim emulation for editing is always going to be limited and/or would require significant efforts so why not just using (neo)vim ?
But F4 doesn't seem to work for the URL bar, right ? Would it be possible to enable it there ? It's rather frustrating to have to reach out for the mouse in the terminal.

Yep, I can add editor support to the URL bar. You're the second person to ask this week and it should be fairly straightforward, so I'll aim to get it in soon.
Out of curiosity though, why do you feel the need to reach for the mouse when using the URL bar? Is there some key binding you would find useful?

Also, I'm still deciding how best to approach directional navigation. There will likely be something similar to what you suggested, but I'm not 100% sure how it will work yet.
I'm thinking: Ctrl+w followed by lowercase hjkl to move between widgets directionally, or uppercase HJKL to move between "sections" directionally (I.e tabs/collection tree/request area/response area).
I'm also thinking of potentially having a shortcut for this like alt+hjkl and alt+HJKL to do this navigation in a single chord.

Thanks for considering it !
Without vim motions I in fact feel the need to use the mouse (or alternatively spam direction keys) to do simple things, if I had to give a restricted number of vim motions which would highly reduce that need I'd say :

^ and $ (or "g_"), to seamlessly go to each side of the URL to edit the domain name or a querystring for example
the y operator and specifically yy to copy the URL to the browser or another client
the d or c operator, specifically dw/cw to change a query parameter
u/ctrl+r to undo/redo changes

I think those would do the trick for most use cases given the limited scope of a URL bar.

For the Ctrl+W concern I think Ctrl+W, even if a bit long to perform, is fine for vim user, the muscle memory and the convenience of not having to memorize new keybindings specific to an application outweigh the inconvenience of its length imo. Unless you plan to add it as a general shortcut for everyone to have a 4 directions and widget/sections aware alternative tab/shift+tab, in that case yeah ctrl+w might be confusing to others I guess.


Refine tstamp.real

The description of @tstamp.real (and the referenced data type) seem to point to an absolute value of time, at least to me. I would expect it to be a relative value, but then it is not clear from the available information if this refers to the time in the measure, the section, the mdiv, or another element.
Also I'm wondering if ISO time is a useful datatype here, if you always have to enter hours for a piece that takes only some seconds to perform?

@tstamp.real is intended to be a relative time, but the starting point isn't specified.  All values of @tstamp.real have to be measured from the same point.  The starting point can be the start of the piece (most likely) or some other arbitrary point, such as yesterday at 12:01 pm (much less likely).  Values of less than an hour can be filled with zeros, e.g., "00:00:03" indicates a point 3 seconds since the start time.


[Suggestion] Automatically update plugin when new version is out

This is definitely a good issue to have in the backlog, and it's been something I've thought about/wanted for a while. It has two main portions, as I see it:

We need to handle actual updating of the plugin. This includes checking for and downloading updates.
We also need to elegantly handle upgrading the config from one version to the next. This would most likely involve storing a version tag for the config, itself, and building in scripts to convert from one version of the config schema to the next.

I ask, then, how should upgrading work? It should not occur without user consent. There are reasons somebody may stick to an older version (be it due to bugs, removed features, or differently supported server versions).
When we support upgrading, would it make sense to implement it as part of the /gateway command (i.e. "/gateway upgrade" with a "/gateway upgrade confirm" to actually download the new version)? Should it be done in some other way?


Created empty sequences are invalid on parsing

I just noticed that Camel creates empty sequences that it then can't parse successfully.
Steps to reproduce:

Create a YamlMapping and add a YamlSequence with no elements and any key.
Add at least one other key-value pair to the YamlMapping
Call toString() and write the String to a file.
Read the file via YamlMapping m = Yaml.createYamlInput(InputStream).readYamlMapping()
Extract the empty YamlSequence via m.yamlSequence(key)

Expected behaviour:
The last call should return null as there is no yamlSequence there.
Actual behaviour:
An IllegalStateException occurs:
Exception in thread "main" java.lang.IllegalStateException: Indentation of line 1 isn't ok.  It should be greater than the previous line's by 2
	at com.amihaiemil.camel.WellIndentedLine.indentation(WellIndentedLine.java:77)
	at com.amihaiemil.camel.CachedYamlLine.indentation(CachedYamlLine.java:89)
	at com.amihaiemil.camel.RtYamlLines.iterator(RtYamlLines.java:44)
	at com.amihaiemil.camel.ReadYamlMapping.nodeValue(ReadYamlMapping.java:161)
	at com.amihaiemil.camel.ReadYamlMapping.yamlSequence(ReadYamlMapping.java:101)

Example:
Camel produces this YAML file, which triggers the above error
emptyseq:

key1: val1
key2: val2

A simple fix is to write the empty sequence in JSON notation
emptyseq: []

key1: val1
key2: val2

This parses correctly, although yamlSequence("emptyseq") does now not return null but an empty YamlSequence.

@RenWal I see, definetely a bug. I will fix it as soon as I have some time (tomorrrow evening, I think), but the fix should be returning a (working) empty YamlSequence, not null, since the key exists after all.

@RenWal I looked into this, but unfortunately, it is a little harder to fix. I think the simple "solution" for now is to simply ignore empty elements when building the YAML.
This is because the current architecture (layers of abstraction etc) was designed for Block YAML only... it is currently very hard to take into account JSON-like structures, streams and so on. For that kind of stuff, we'll have to change our underlying abstractions, most likely.


Dependency - Update minio-go to v7.0.70

fix issue: #7157
Bumping the Minio-go dependency version to the v7.0.70 as it contains support for EKS Pod Identity

https://github.com/minio/minio-go/releases/tag/v7.0.70

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Bump the minio-go version to v7.0.70

I think it is still unclear to me how to use the EKS Pod Identity feature.

IMO it's more on AWS side; the docs are available on their side and well explained how it works or how to use Pod Identity feature instead of IAM Roles for service accounts (IRSA) in EKS.
in case of Thanos IIUC based on the s3 pkg and your comment we are relying on Minio-go, hence they had to push the update first to support it;
Which they did and here in the update for minio-go, we can see that now beside the other env variables it also checks the AWS_CONTAINER_AUTHORIZATION_TOKEN_FILE  that's a new env var provided by Pod Identity feature

https://github.com/minio/minio-go/pull/1944

Do we need to update docs somewhere for instructions?

I did a quick search in the docs, and maybe we can add it here, wdyt?

Cool, thanks for the comment. I think I am ok to proceed with this pr without updating docs. We don't have doc for AWS_CONTAINER_AUTHORIZATION_TOKEN_FILE as well

There is a conflict in changelog

There is a conflict in changelog

I just rebased my branch so it should be fine now

Just noticed that the changelog entry seems wrong. It shouldn't be under Fixed. Should be Added instead as it is a feature/enhancement.

I've opened a new PR to fix the changelog position;

https://github.com/thanos-io/thanos/pull/7423


lint: add typescript lint parser

cc @czy88840616 @guangwong

Codecov Report

Merging #159 into master will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
- Coverage      81%   80.93%   -0.07%
==========================================
  Files         184      184
  Lines        5953     5953
  Branches      776      776
==========================================
- Hits         4822     4818       -4
- Misses        787      791       +4
  Partials      344      344

Impacted Files
Coverage Δ

packages/metrics/src/collect/MetricsCollector.ts
73.52% <0%> (-11.77%)
:arrow_down:

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 ce6da2b...781b026. Read the comment docs.

Ignore, just use tslint please.


feat(secrets): Implement into_iter for SecretStore

Description of change

Title~
Came up in a discussion about secrets https://discord.com/channels/803236282088161321/1107140940759568384/1118625700824485928
How has this been tested? (if applicable)

Added test

For context for others who don't want to wade through the entire Discord thread: basically I'm wanting to use Rocket's outstanding figment feature that allows you to extract config values directly into structs without futzing around. For example, rocket.figment().extract::<OpenAiConfig>() gives me all I need to configure OpenAI without working with each field individually.
Rocket does this automatically based on environment variables. Shuttle Secrets (set via Secrets.toml) do not get injected into applications as environment variables, and so figment and others were sad.
The idea was to work around this limitation by iterating over all the secrets within SecretStore and call std::env::set_var for each one. SecretStore doesn't provide a convenient way to do this, hence Shuttle Hero @jonaro00 jumped into action with this MR.


[FormsyText] Replace deprecated onEnterKeyDown with onKeyDown

Closes #60

Thanks for the PR 👍 , but I"m going to merge #67 as a more complete solution.


fix custom operator python3 bug

#2130

Thanks for the PR, please also update https://github.com/dmlc/mxnet/blob/master/CONTRIBUTORS.md to add yourself to the list of contributors.
Thanks for making mxnet better for the community!


Ender IO Conduit Performance is Slow

Issue Description:
Under Enigmatica 2 Expert. I spoke with Niller about this (bug here: https://github.com/NillerMedDild/Enigmatica2Expert/issues/1190) and we found it is unlikely to be related to E2E or any mod pack. It appears that Ender Conduits are individually slow.
Version: EnderIO 5.0.44
What happens:
Ticks per second linearly degrades as more conduits are added. You can see this with the profiling tools available in Minecraft:
https://pastebin.com/ejRagWAi
What you expected to happen:
Performance requirements for a conduit should not be this high, per conduit.
Steps to reproduce:

Create conduits.
Measure "debug start" profiling every 100 conduits.
Map TPS over time.

Affected Versions (Do not use "latest"):

EnderIO: 5.0.44
EnderCore: 5.57
Minecraft: 1.12.2
Forge: <IP_ADDRESS>31

Your most recent log file where the issue was present:
https://pastebin.com/ejRagWAi
Example shots of usage: https://filedn.com/lD47MkPDxpzLREeay928LHm/Debugging/
Example counts of conduits when problem becomes untenable (70% cpu time spent on conduits):
enderio:block_conduit_bundle, meta 0, 637
enderio:block_conduit_bundle, meta 1, 405

Please run the vanilla profiler for a couple of minutes and attach the profiling log.

That's the pastebin, above. Or are you asking me to create a new world with only EnderIO and graph the conduit to TPS?

Here's the profiler after 6.7 minutes:
https://pastebin.com/q4R9RfTh

I expected a logfile there, so I didn't even click the link. You're the first person to even know the vanilla profiler...
Ok, that time is spent in updateNetwork/updateConnections. Those run together whenever a conduit is added or removed to a network of conduits. (updateConnections also has other triggers, but seeing that it has about the same runtime as updateNetwork, I don't think those come into play here).
As I don't think you have a block breaker and block placer combo to do this, the most likely reason would be a chunk that is being loaded and unloaded each tick. While Ender IO knows how to handle a conduit network that is partially in loaded and partially in unloaded chunks, there is no way of protecting against a "flickering" chunk in there---the conduits in that chunk just don't know that they are in one.
Please check if that is the cause of your slowdowns. I think there are mods out there to diagnose chunk flickering.

Thanks Henry, that's useful analysis. I'll look into flinkering chunks and get back to you. Might take a day or two, if you don't mind keeping this issue open.

I don't know what's a normal rate for chunk loading. I'm at work so can't spawn into the world to look at specific chunks, but here is chunk load activity over 2000 ticks:
https://pastebin.com/mnS2zeyc

I think we're onto something here. Chunk [    0,    21] is definitely flickering.
17.05 12:28:43 [Server] Server thread/INFO [tellme]:    Chunk Load ; DIM:    -1, Tick: 195698178, Time:<PHONE_NUMBER>986, ChunkPos: [    0,    21]
17.05 12:28:44 [Server] Server thread/INFO [tellme]:  Chunk Unload ; DIM:    -1, Tick: 195698179, Time:<PHONE_NUMBER>036, ChunkPos: [    0,    21]
17.05 12:28:47 [Server] Server thread/INFO [tellme]:    Chunk Load ; DIM:    -1, Tick: 195698254, Time:<PHONE_NUMBER>779, ChunkPos: [    0,    21]
17.05 12:28:47 [Server] Server thread/INFO [tellme]:  Chunk Unload ; DIM:    -1, Tick: 195698255, Time:<PHONE_NUMBER>834, ChunkPos: [    0,    21]
17.05 12:29:59 [Server] Server thread/INFO [tellme]:    Chunk Load ; DIM:    -1, Tick: 195699680, Time:<PHONE_NUMBER>117, ChunkPos: [    0,    21]
17.05 12:29:59 [Server] Server thread/INFO [tellme]:  Chunk Unload ; DIM:    -1, Tick: 195699681, Time:<PHONE_NUMBER>155, ChunkPos: [    0,    21]
17.05 12:30:11 [Server] Server thread/INFO [tellme]:    Chunk Load ; DIM:    -1, Tick: 195699914, Time:<PHONE_NUMBER>007, ChunkPos: [    0,    21]
17.05 12:30:11 [Server] Server thread/INFO [tellme]:  Chunk Unload ; DIM:    -1, Tick: 195699915, Time:<PHONE_NUMBER>036, ChunkPos: [    0,    21]
17.05 12:30:17 [Server] Server thread/INFO [tellme]:    Chunk Load ; DIM:    -1, Tick: 195700055, Time:<PHONE_NUMBER>841, ChunkPos: [    0,    21]
17.05 12:30:17 [Server] Server thread/INFO [tellme]:  Chunk Unload ; DIM:    -1, Tick: 195700056, Time:<PHONE_NUMBER>878, ChunkPos: [    0,    21]

You see how it is loaded and directly unloaded the very next tick?
I didn't compare the timestamps, but it looks to be the same for those chunks, too: [   -1,    19] [   -1,    21] [   -2,    19] [   -2,    20] [   -3,    19] [   -3,    20] [   -4,    20] [   -5,    20] [   -7,    28] [  -12,    19] [  -12,    20] [  -12,    21] [  -12,    22] [  -13,     3] [  -13,     4] [  -13,     5] [  -13,     6] [  -13,     7] [  -13,     8] [  -13,     9] [  -13,    10] [  -13,    18] [  -13,    19] [  -13,    20] [  -13,    21] [  -13,    22] [  -14,     3] [  -14,     4] [  -14,     5] [  -14,     6] [  -14,     7] [  -14,     8] [  -14,     9] [  -14,    10] [  -15,     3] [  -15,     4] [  -15,     5] [  -15,     6] [  -15,     7] [  -15,     8] [  -15,     9] [  -15,    10] [  -15,    11] [  -15,    12] [  -15,    13] [  -15,    14] [  -15,    15] [  -15,    16] [  -16,    13] [  -16,    14] [  -16,    15] [  -16,    16]
My recommendation would be to put down a chunk loader. That's usually much easier than trying to keep chunks unloaded.
The target is to get the the occurrence of "chunk loaded in tick x and unloaded in tick x+1" to a minimum---there's no way of preventing it completely---and the the number of chunks that do that all the time to near zero.

I don't have anything intentionally pushing things into a nether portal, but this gives me a bunch to investigate. I'm at work now, so I can't travel to those chunks right now and look around. I'll definitely track this down tonight. I do think this is looking like the cause of the performance issues and it shouldn't be hard to figure out why these chunks are malfunctioning.

No need to hurry for my benefit. I'll be here for quite a while to come ;)
However, I'll be away for the weekend, so please don't feel ignored.

I couldn't figure out the problem. I migrated the server to an i9. Solve it with compute!


Toward a consistent namespace, and scalacheck augmentation.

Current coverage is 63.41% (diff: 51.38%)

Merging #1535 into master will increase coverage by 0.77%

@@             master      #1535   diff @@
==========================================
  Files           324        327     +3
  Lines         15239      15090   -149
  Methods       14758      14806    +48
  Messages          0          0
  Branches        150        145     -5
==========================================
+ Hits           9545       9569    +24
+ Misses         5694       5521   -173
  Partials          0          0

Powered by Codecov. Last update 4c8bc6e...8cdc16e

Can one of the admins verify this patch?

@sellout @wemrysi It looks like github denies me the green check mark when there's a "skip ci" commit, but this PR does have a nice green check mark on the previous commit, and the skipped one added a changelog.

LGTM, @sellout feel free to merge if you agree.


Bootstrap this repository

Make it look similar to https://github.com/paketo-buildpacks/builder-jammy-buildpackless-tiny.

This is complete: https://github.com/paketo-buildpacks/builder-jammy-buildpackless-static/releases/v0.0.1


Translations keys are normalized but msgid only trimmed

Hello, thank you for your hard work on this project.
We recently ran into an issue caused by the string to be translated containing characters like \n. When translations are loaded in the createGettext function, there seems to be a step that normalizes each key, here
This will get rid of \n and s+ and will trim the keys. The problem is when we try to actually get the translated string, msgid is only trimmed, here meaning that the msgid and the corresponding key in the translations object will not match due to one being normalized and the other not.
We are planning to apply a patch but would also love to contribute to the project if helpful. Would you recommend applying the normalization to the msgid as well or remove the normalization of the keys?

@mussinbenarbia We recently created a new 4.0 pre-release version that made some changes to that, but it hasn't been thoroughly tested:

Breaking Changes

Message ids are not trimmed anymore. this was likely initially implemented to deal with ids in HTML (using <translate>/v-translate)
Line breaks are now normalized to LF across the board (\r\n in messages will always become \n)

v4.0.0-alpha.4
I believe there are some issues with the bin files that I will try to address in the next week, but if you are patient you don't need to creat a patch.

Thank you for the quick feedback!

@lzurbriggen

Line breaks are now normalized to LF across the board (\r\n in messages will always become \n)

Does this mean that the same will be applied to msgid? For example:
gettext(`This is a string.
When extracted it will contain newline characters.`)

When normalized with the new step, if I understand correctly, this will be This is a string.\r\nWhen extracted it will contain newline characters.
But the msgid, at runtime will still be This is a string.\nWhen extracted it will contain newline characters..
Will this not cause the lookup for the translated string to fail?

@mussinbenarbia

When normalized with the new step, if I understand correctly, this will be This is a string.\r\nWhen extracted it will contain newline characters.

The idea is that all \r\n become \n in all messageids: when extracting, when reading in the translations.json and when looking up msgids, so the lookup should succeed. It's possible I missed something, did you run into an issue already?

No issue as I haven't tried v4 (Our project still uses the  component and v-translate directives so we'll need to remove those before updating)
I looked at the code in the v4 branch and I understand the changes now. The code will now call normalizeMsgId() both in getTranslation() and also in normalizeTranslations() so the lookup should match 👍
Kind of an unrelated question but I noticed that normalizeMsgId is implemented as:
export function normalizeMsgId(key: string) {
  return key.replace(/\r?\n|\r/, "").trim();
}

Why is there a need to even remove these characters? I'm only asking because the Vue 2 version did not have this normalization step and I'm curious why it was introduced. Also is it intended that it only replaces the first occurrence?
Since it's not replaceAll, and it's also not using the /g flag, this will only match the first character.

I definitely don't want to trim msgids anymore. I'll have another look at this.


change [DATA] option TRAIN_DIR to JSON_FILENAME

Leaving this so the code finds the .json file for the user for now


Collect exercise failures rather than exiting early.

I generally feel that the build script should accumulate all the errors and report them before exiting. This is nice when testing changes to many exercises. However, we may want to fail fast on CI given the length of the build...thoughts?

I think you raise a good point. Looking at the Travis build time for this PR, I'd say the cache seems to be working. Pre-cache builds were taking 7-8 minutes or more, but now they seem to be holding steady just above 4 min.
I support this PR.


error "SSE4.1 instruction set not enabled" in dependent cryptokit package

I am trying to build google-drive-ocamlfuse and it uses the cryptokit package as a library but fails. See output below
The problem is it pulls cryptokit v1.17 tar file each time and then tries to compile it but it fails because of "error "SSE4.1 instruction set not enabled". According to issue 32 on this project there is an #include <smmintrin.h> in pclmul.c file which is not required but the tar file still contains it and there is no official update.
I have not been able to edit the local copy of pclmul.c as it is overwritten each time. Can you please assist me with a work around?
Is there are way to introduce a delay or halt in the build between retreiving the file from the repo and the compile so that I can quickly make the local edit? Sorry I am a beginner in opam, dune and OCaml and have not been able to see where I can introduce a pause/request to continue by the user....
> opam install google-drive-ocamlfuse
> The following actions will be performed:
>   ∗ install cryptokit              1.17   [required by google-drive-ocamlfuse]
>   ∗ install gapi-ocaml             0.4.3  [required by google-drive-ocamlfuse]
>   ∗ install google-drive-ocamlfuse 0.7.30
> ===== ∗ 3 =====
> Do you want to continue? [Y/n] y
>
> <><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><>
> Processing  3/9: [cryptokit.1.17: http] [gapi-ocaml.0.4.3: http] [google-drive-ocamlfuse.0.7.30: http]
> ⬇ retrieved google-drive-ocamlfuse.0.7.30  (https://github.com/astrada/google-drive-ocamlfuse/archive/v0.7.30.tar.gz)
> ⬇ retrieved cryptokit.1.17  (https://github.com/xavierleroy/cryptokit/archive/release117.tar.gz)
> ⬇ retrieved gapi-ocaml.0.4.3  (https://github.com/astrada/gapi-ocaml/archive/v0.4.3.tar.gz)
> [ERROR] The compilation of cryptokit.1.17 failed at "dune build -p cryptokit -j 1 @install".
>
> #=== ERROR while compiling cryptokit.1.17 =====================================#
> # context     2.1.3 | linux/x86_64 | ocaml-base-compiler.4.09.0 | git+https://github.com/ocaml/opam-repository.git
> # path        ~/.opam/4.09.0/.opam-switch/build/cryptokit.1.17
> # command     ~/.opam/opam-init/hooks/sandbox.sh build dune build -p cryptokit -j 1 @install
> # exit-code   1
> # env-file    ~/.opam/log/cryptokit-23605-09d063.env
> # output-file ~/.opam/log/cryptokit-23605-09d063.out
> ### output ###
> # [...]
> # (cd _build/default/src && config/flags.exe)
> # ZLib: ............................... enabled
> # Hardware support for AES and GCM: ... enabled
> # getentropy():........................ disabled
> # File "src/dune", line 36, characters 9-15:
> # 36 |          pclmul
> #               ^^^^^^
> # (cd _build/default/src && /usr/bin/gcc -std=gnu99 -O2 -fno-strict-aliasing -fwrapv -fno-builtin-memcmp -fPIC -D_FILE_OFFSET_BITS=64 -D_REENTRANT -DCAML09.0/lib/ocaml -I /home/admin/.opam/4.09.0/lib/zarith -o pclmul.o -c pclmul.c)
> # In file included from pclmul.c:24:0:
> # /usr/lib/gcc/x86_64-linux-gnu/4.8/include/smmintrin.h:31:3: error: #error "SSE4.1 instruction set not enabled"
> #  # error "SSE4.1 instruction set not enabled"
> #    ^
>
>
>
> <><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
> ┌─ The following actions failed
> │ λ build cryptokit 1.17
> └─
> ╶─ No changes have been performed

Update: the workaround was to pin the master as below
opam pin add cryptokit git+https://github.com/xavierleroy/cryptokit#master


Docs/side bar accordion

Issue: # 870
What I did
Updated docs sidebar to use our Accordion
Screenshots (if appropriate):
Checklist

[x] changeset has been added
[x] I have made corresponding changes to the documentation
[n/a] New and existing unit tests pass locally with my changes
[n/a] I have added tests that prove my fix is effective or that my feature works

Codecov Report

Merging #899 (83d5d42) into dev (6e05822) will decrease coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #899      +/-   ##
==========================================
- Coverage   86.96%   86.77%   -0.20%
==========================================
  Files         175      175
  Lines        4551     4575      +24
  Branches     1554     1562       +8
==========================================
+ Hits         3958     3970      +12
- Misses        292      301       +9
- Partials      301      304       +3

Impacted Files
Coverage Δ

...react-magma-dom/src/components/InputBase/index.tsx
75.43% <0.00%> (-7.71%)
:arrow_down:

...-renderer/src/components/FieldArray/FieldArray.tsx
76.92% <0.00%> (+5.49%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Removing the animation looks fine when changing between pages, but looks odd when just opening and closing the Accordions 😕  I wonder if we can tweak it some more for the collapsing state?

Agreed! I'll keep tweaking.

Since we are removing the only usage of the react-accessible-accordion package, we should also delete it from the package.json


[WIP] ctrlr: init at 6.0

Motivation for this change
Software is pretty unstable atm
Version 2020-01-28 (stable branch) works both as standalone and vst plugin, but panel doesn't load due errors
Version 6.0 works only as standalone and only in debug mode, vst plugin crashes
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

6.0.35 testing 8127356e910cd29cc09e19a3a423f2d25dc382f1
app - ok
vst - crash
6.0.28 stable 2f9ce2ea74e49cb324442eae530b74aff59c35f2
app - ok
vst - crash
6.0.26 bb8f3e0a92e4976097c53346068ca5e11432cc1a
fatal error: pluginterfaces/vst2.x/aeffect.h: No such file or directory
6.0.25 d21733a835c79f7f8da85b6b17ed9527d5886050
fatal error: pluginterfaces/vst2.x/aeffect.h: No such file or directory


It would be useful to have an example using namespaces

It would be useful to have an example using namespaces.

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

ID: ea4474bd-b6d2-241c-244a-695579a1e7af
Version Independent ID: 458e32a8-93b3-5da1-b7e1-4718721be9dc
Content: XmlPeek Task - Visual Studio
Content Source: docs/msbuild/xmlpeek-task.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@paulomorgado -- Paulo, thank you for your suggestion.  Please consider these resources:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

@ghogen -- Gordon, please look into this issue.


Investigate pre-trained feature models

Use Caffe/Barista.

Use the tutorial CNN of keras.io

Made the tutorial CNN here and looked at the results here.


Elixir + Phoenix + Node.js + PostgreSQL Dev Container revival

Hey, this PR is a revival of PR #432. I noticed that it had gone stale after September and would really love to have this Elixir + Phoenix + Node + Postgres dev container finished and merged.
I have addressed the comments in the PR (#432) and added one or two more nice to have in the configurations. If you have any more feedback just let me know. I love using Dev Containers (and Github Codespaces) and excited to contribute to something awesome.

@Chuxel Thanks for the review and testing out the code. Found the issue with trying to run things as the non-root user. The user doesn't have access to anything installed by the root user with mix so you have to switch to that user before then in the Dockerfile.
I am not 100% great with the Docker-foo so let me know if this is a valid solution.

@talk2MeGooseman Yep, that works, but to avoid the extra layer and leave the image running as root by default, you can use the su command to temporarily run as the other user.  e.g.
RUN su ${USERNAME} -c "mix local.hex --force \
  && mix local.rebar --force \
  && mix archive.install --force hex phx_new ${PHOENIX_VERSION}"

@Chuxel Thanks for the feedback and awesome tip

@PavelSosin-320 The image capable of running both on a local machine and within Codespaces. Curious to know if you see any glaring red flags? So I can keep a lookout for those.

LGTM!  Thanks!

@talk2MeGooseman The Docker's "Checker flag" is the correct configuration and implementation of the HEALTHCHECK Docker healthcheck.
If I can configure docker run --healthcheck options in such a way that ensures that no Docker default timeout, no network timeout no ctrl/c from the user will happen because of the slow container instantiation of the Image it is OK for me regardless of how many things it contains.  Otherwise, I suspect that the user will have trouble with it. According to the Murphy law, it will certainly happen. Since docker ps will not contain any useful information about container status it will be "it is always OK on my computer" issue.


Add --workspace feature

We have a need for a "workspace" in tests/evals, but I find that it could be a generally useful feature to let each log potentially have a workspace subfolder where project files can reside.
This subfolder would be created by providing --workspace to gptme, or detected on log load.

I implemented not quite this in #100, perhaps I should reconsider the implementation.

Redid the implementation so that one can now pass a magic @log workspace and it will create a workspace in ./<logdir>/workspace, as well as default to using any such workspace folder if it exists in the log folder upon load.


raw sync output error

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Related to Issue #223 , I"m attempting to use the --raw  switch with  recyclarr sync, but am receiving the following error:
recyclarr@mm-utils:/opt/recyclarr$ recyclarr sync --raw
CommandParseException: Unknown option 'raw'.
  at void ParseOption(CommandTreeParserContext context, CommandTreeTokenStream stream, CommandTreeToken token, CommandTree node, bool
     isLongOption) in CommandTreeParser.cs:282
  at CommandTree ParseCommandParameters(CommandTreeParserContext context, CommandInfo command, CommandTree parent,
     CommandTreeTokenStream stream) in CommandTreeParser.cs:149
  at CommandTree ParseCommand(CommandTreeParserContext context, ICommandContainer current, CommandTree parent, CommandTreeTokenStream
     stream) in CommandTreeParser.cs:123
  at CommandTreeParserResult Parse(CommandTreeParserContext context, CommandTreeTokenizerResult tokenizerResult) in CommandTreeParser.cs
     :89
  at CommandTreeParserResult ParseCommandLineArguments(CommandModel model, CommandAppSettings settings, IReadOnlyList<string> args) in
     CommandExecutor.cs:110
  at async Task<int> Execute(IConfiguration configuration, IEnumerable<string> args) in CommandExecutor.cs:53
  at async Task<int> RunAsync(IEnumerable<string> args) in CommandApp.cs:84

I have the latest recyclarr downloaded.
Expected Behavior
Raw output
Steps To Reproduce
recylarr sync --raw
Environment & Versions
- Operating System: Debain 12
- Architecture: x86/64
- Recyclarr Version: v7.2.3 (Branch.master.Sha.04b416f73c2aa7909beee1fed0683116276950d5)

Recyclarr Log File(s)
no log generated
Configuration YAML
using standard hd-bluray-web.yml

Sonarr / Radarr Logs
No response

If you refer to the docs you will see that --raw is not a valid or accepted argument for the sync command. I do understand why you're confused, though, since I resolved the linked issue by implementing the requested functionality differently. You can find more detail in the linked commit, but basically the desired output was provided via the NO_COLOR env var.


Allow switching coordinate system according to metadata in SD. Fix consistency bug.

What changes do you make in this PR?

Please describe why you create this PR

Checklist

[ ] I have merged the latest main branch into current branch.
[ ] I have run bash scripts/format.sh before merging.
Please use "squash and merge" mode.

Merge this PR when all tests pass


Build Error on Microsoft Windows 7 -  64Bit

Hi All,
I am having a build error on windows. Please see below, I need help
error: invalid use of non-static data member 'address_range::type'

error: from this location

Please help me!
Many thanks!
God blesses!!!
Best regards,
Sanyaade

Many thanks for your feedback.
I  have gcc version 8.10, do I need to upgrade this version?

Looking at alternatives like cross-compile, tryout Windows 8 or 10
God blesses!!!
Best regards,
Sanyaade

@lurch :
Once again many thanks for your feedback. I upgraded my MinGW compiler to 10.3.0 and I was able to compile pico-sdk and picotool on Windows-7 (64Bit)

I am closing the issue now!
Many thanks!
God blesses!!!
Best regards,
Sanyaade


fix: imports

Summary
What is the background of this pull request?

Resolve import errors

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


installing package using npm doesn't include buildPlan method

Hi,
The bug described in the title occurred on version 0.0.5.
Thanks in advance,
Reuven

PR merged

That's great, thanks!
Still, the problem occurs, the last method in /lib/bamboo is getAllPlans.
Maybe increasing the version might help?

@sebarmeli Please let me know if it's possible to fix this soon.
Thanks in adavance,
Reuven

Hey, @Reuvenc , I just reviewing the code and buildPlan method will even not work, there an error in the code and also no tests.
I will fix it and create PR, then @sebarmeli can review it.

@Reuvenc created PR https://github.com/sebarmeli/bamboo-api/pull/16 , fixed buildPlan method.

@pvoznenko Thanks for your fix.

@Reuvenc , try to get 0.0.7 version , buildPlan should be there. @sebarmeli published update.


Proposal: Please start using Semantic Versioning

I found that this project already supports Go modules. But sadly, the tags doesn't follow Semantic Versioning, which means that all tags of this project will be ignored by Go modules and replaced by pseudo-versions, go get acts weirdly when tags are not in that form. It would be great to have the tagged release be named in the format vX.X.X format so that go mod can read it.
	github.com/jasonlvhit/gocron v0.0.0-20200423141508-ab84337f7963

Else the mod file shows something like github.com/jasonlvhit/gocron v0.0.0-20200423141508-ab84337f7963 which is not very readable and difficult to upgrade. It’s hard to verify which version is in use. This is not conducive to version control
So, I propose this project to follow Semantic Versioning in future versions. For example, v1.0.1, v2.0.0, v3.1.0-alpha, v3.1.0-beta.2etc.

@Streppel  @Arjunmahishi  Could you help me review this issue? Thx :p

Hi @KateGo520! Absolutely, we can fix this here. Be aware, though, that we migrated to a new repo linked in the readme and we are working collectively over there, be sure to check it out!

Updated semver :-)

@Streppel  OK, I see the new link. Thank you!


Empty lines are hidden in source editor

Description of your problem
I freshly installed a python 3.5.4 from python.org (msi-install) under windows 10 and pip-installed spyder, which resulted in spyder 3.2.6 .
When I type my source code and insert empty lines for improved readability, the empty lines do not show in the editor window. If I enter 3 new lines I need 3 back-spaces to get back to the original position with the cursor. Hence, the empty lines seem to exist, but are not shown.
I checked all preferences for an option to hide/un-hide empty lines, but didn't find anything.
Whats wrong???
An additional issue is that line numbers are not shown at the left border, although the option is marked in the preferences. But I can live with that.
What steps will reproduce the problem?

start spyder
type some code
enter newline several times
enter additional code

What is the expected output? What do you see instead?
The expected output is "code -- space (empty lines) -- code"
What I see is "code code"
Please provide any additional information below
Versions and main components

Spyder Version: 3.2.6
Python Version: 3.5.4
Qt Version: 5.10.0
PyQt Version: 5.10
Operating system: windows 10 64-Bit

Dependencies
all dependencies listed are "ok" except for
cython
sympy

I freshly installed a python 3.5.4 from python.org (msi-install) under windows 10 and pip-installed spyder, which resulted in spyder 3.2.6

Please read our
https://github.com/spyder-ide/spyder/wiki/Troubleshooting-Guide-and-FAQ
If you are unable to fix the problem after that, please install Anaconda instead. Unfortunately, we don't have the resources nor the time to help people with custom installations, so it's up to you to figure this one out.
This is because custom installations are for experts or very knowledgeable people who really know what they are doing and are able to easily solve problems like this.

Whats wrong???

Probably the use of PyQt 5.10. My only advice: try to downgrade to 5.9.2 and try again. Else use Anaconda, as I said above.

Although the issue is long closed, I want to document what happened to me today for people finding this issue by web searches in the future:
I used Spyder while on a wireless network (somestimes also with VPN client's virtual network adapter) yesterday on a Windows 10 64-bit laptop, then quit Spyder and hibernated the computer, then started it today with a network cable connected. When starting Python, on the same Python source code files as yesterday several issues were noticed:

The gray area for line numbers was visible but no numbers were printed (not even line 1)
Blank lines were missing or hidden! It was really confusing with the cursor that if I moved it down from a line with code, or right past the end of the line, the text cursor became invisible. Moving it further could make it show up on the next line (which had code). If I typed while invisible, a new line would appear, but with Enter-presses I could never achieve the pair of line breaks that would normally produce a blank line.
This was the case both for some small files and for the big file (with 2300 lines) where I noticed it first.

After not finding any similar issues online, I thought maybe it was time to upgrade Anaconda & Spyder, but after doing this the issue remained. Tried disabling and re-enabling line numbers and other Spyder-editor settings, with no success.
Finally, I rebooted the computer. And now the editor works normally again. My guess is that it was something related to a service running in the background (e.g. something else for code analysis although I have disabled Kite) and getting confused by the way I was on another network (wired instead of wireless) after exiting hibernation. (Spyder 4.1.2, 32-bit Python 3.7.7, IPython 7.13.0, Qt 5.9.7).


Cookie Monster causes errors in Cookie Clicker

Describe the bug
After loading CookieMonster add-on as described on the cookie clicker wiki, Cookie Clicker might no long play some sounds or load some graphics.
To Reproduce
The steps needed to reproduce the behavior:

Load the http:// version of cookie clicker in incognito mode
Load the CookieMonster add-on.
Wait for a while, like 15-minutes.
Keep your dev console open
Start playing actively. It seems to work fine initially, but there'll be errors in the console
It might help to use a save file to speed things up.

Screenshots

Save file
[BenjiBakery.txt](https://github.com/Aktanusa/CookieMonster/files/5814223/BenjiBakery.txt)

I cannot seem to reproduce this error on my own.
Do you get the same issue when you play the game in normal mode (so not in incognito)? Looking at the errors this might be the only thing I can currently think of that can cause these problems.

I cannot seem to reproduce this error on my own.
Do you get the same issue when you play the game in normal mode (so not in incognito)? Looking at the errors this might be the only thing I can currently think of that can cause these problems.

Neither can I on a different machine. I guess it's device related.
I'll try your suggestion tomorrow.

Neither can I on a different machine. I guess it's device related.
I'll try your suggestion tomorrow.

It is probably related to the CookieClicker cookie issue mentioned on Reddit: https://www.reddit.com/r/CookieClicker/comments/kdvgzn/orteil_update_do_not_delete_your_cookies/
After a page refresh I ran into the same problem as described here.

It is probably related to the CookieClicker cookie issue mentioned on Reddit: https://www.reddit.com/r/CookieClicker/comments/kdvgzn/orteil_update_do_not_delete_your_cookies/
After a page refresh I ran into the same problem as described here.

Ah that could be it!
You closed the issue so I assume there are no further problems now?

Ah that could be it!
You closed the issue so I assume there are no further problems now?

Not related to CookieMonster, no ;)

Not related to CookieMonster, no ;)


spiram: Fix corrupting 1 kB of noinit data in spiram in 2T mode

The function psram_2t_mode_check() in esp_psram_impl_quad.c overwrites the first kilobyte of every megabyte with 0xff upon startup which may overwrite noinit data which should be preserved over reboots.
This patch fixes this by saving/restoring the tested memory.
The test size has been reduced from 41024 bytes to 4512 bytes just to keep the stack usage of the fuction within resonable limits (was 2048 bytes, now 1536 bytes).
Fixes https://github.com/espressif/esp-idf/issues/12395
(IDFGH-11231)

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


Liveness check - low level documentation and implementation

[ ] bug report -> please search issues before submitting
[x] feature request
[x] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

We would like to implement the liveness check into our multiplatform application which is written using C++ code that runs on Android, iOS and WebAssembly inside browser.
Is there some lower level documentation or implementation for the liveness?
We assume the Kotlin and Swift implementation just calls some native libraries and performs REST requests which we could re-implement in C++ if needed.
Could we get access to this lower level implementation or their documentation?

Hi, apologies for the delayed reply. We currently have no plan to support direct usage of the lower level implementation. Please use Android and iOS libraries SDK for now. If you would like to use JavaScript, our JS SDK is in private preview and you can reach out to<EMAIL_ADDRESS>for details on how to onboard for access.