switch-multi

I have the following issue,
on a switch-multi the homekit app does not update the state to 0 and I have to turn it off manually, so the last line comes up.
Aug 05 18:28:03 rpi3 homebridge[771]: [8/5/2017, 6:28:03 PM] [Broadlink RM Mini3] Volume Up setSwitchState: 1
Aug 05 18:28:03 rpi3 homebridge[771]: [8/5/2017, 6:28:03 PM] [Broadlink RM Mini3] Volume Up sendHex (<IP_ADDRESS>; 0d:43:b4:f9:1a:5d)
Aug 05 18:28:03 rpi3 homebridge[771]: [8/5/2017, 6:28:03 PM] [Broadlink RM Mini3] Volume Up setSwitchState: 0
Aug 05 18:28:05 rpi3 homebridge[771]: [8/5/2017, 6:28:05 PM] [Broadlink RM Mini3] Volume Up setSwitchState: 0
Aug 05 18:28:05 rpi3 homebridge[771]: [8/5/2017, 6:28:05 PM] [Broadlink RM Mini3] Volume Up setSwitchState: already 0

fixed after homebridge restart


Enable Vagrant support

These set of changes facilitates the creation of development environments with Vagrant. They don't pretend to replace the current provisioning process.

/label ok-to-test

/ok-to-test

/approve

/lgtm


JAVA-28953: Upgrade spring-boot-modules to Spring Boot 3

See https://github.com/eugenp/tutorials/pull/15834


19 support of new input system

Additional stuff

It fixes movie updates skipping 1 update
Adds way to choose when does frame advancing advance towards

Closes #176


feat: add 404.sql handler

Fixes #529 .
@lovasoa Is this an acceptable business logic to you? I'm sure the code can be cleaned up, and there might be an argument for using PathBuf directly instead of doing String manipulation, but besides that?

Hi @wucke13 ! The logic seems to match exactly what we discussed; everything good for me!
If you can clean up the code, and maybe put it in a separate function, I'll merge and this will be in the next version.
Thank you for your contribution!

I'm not sure what the best way to teach this new feature to users is. I think we should have 404.sql files in at least some of the examples on this repository.
We could also mention its existence in the documentation of the json component and the path function. We could also mention it in configuration.md.

I definitely plan on adding an example. I'm quite indifferent on one thing: Should we do the path traversal stuff using a String or a PathBuf. What I currently like about using a String, is that we can discern a/b/c/ and /a/b/c. In the former case we want to check /a/b/c/404.sql, but in the latter case we don't. I think this nuance gets lost when converting the String to a PathBuf. @lovasoa Would you agree on using a String?

Yes, we should use a string and not a PathBuf. If there are other places where I used a PathBuf to represent a URL path, it is a mistake. PathBufs represent a platform-specific local file path; we want URL path handling to work exactly the same on all platforms.

Ok, I now have a cleaner solution. Please let me know if you are happy with it as well. If yes, I'll also push an example, squash the first two commits and hopefully then we are ready for merge :smile: .

@lovasoa I adjusted again. Most notably, now the serve_fallback call is only made when the error is 404:
// On 404/NOT_FOUND error, fall back to `404.sql` handler if it exists
let response = match maybe_response {
    // File could not be served due to a 404 error. Try to find a user provide 404 handler in
    // the form of a `404.sql` in the current directory. If there is none, look in the parent
    // directeory, and its parent directory, ...
    Err(e) if e.as_response_error().status_code() == StatusCode::NOT_FOUND => {
        serve_fallback(&mut service_request, e).await?
    }

    // Either a valid response, or an unrelated error that shall be bubbled up.
    e => e?,
};
```

I'm not sure how to test this elegantly, though at least at a superficial glance this seems to be a pretty robust implementation of `only ever try to do something when there is an `404 Not Found` error.

I also added an example, that both illustrates the use and explains the fallback mechanism in prose. While I did mention custom error handling and implementing REST APIs as potential use-cases, I intentionally did not pitch the use of this feature as gateway to dynamic routing in general, I feel that goes against the overall spirit of SQLpage. Creative users anyway will do what they want :smile: , but we don't have to encourage it in the examples.

Great! Thank you @wucke13 !
You can add a test in tests/index.rs that makes a request to tests/xyz.sql that doesn't exist, and add a tests/404.sql.
Maybe we can still mention using it for APIs in the documentation of the json component.

@lovasoa  Done. Also squashed a little for a clean history.

I'd be glad if we can get this done rather sooner or later, my motivation is somewhat depleted.

Thank you @wucke13 ! This PR is going to be really useful! I'll try to reference the new feature more in the docs to make it more discoverable


OXE rlds data format to hg lerobot format

What this does
A simple and generic implementation to convert existing OXE dataset (RTX)  which is in standard RLDS to Lerobot dataformat.
Example
To upload a sample dataset austin_buds_dataset_converted_externally_to_rlds, run:
python lerobot/scripts/push_dataset_to_hub.py \
--raw-dir /hdd/tensorflow_datasets/austin_buds_dataset_converted_externally_to_rlds/0.1.0/ \
--repo-id youliangtan/austin_buds_dataset_converted_externally_to_rlds_to_hg  \
--raw-format oxe_rlds # new type

The uploaded data is located here in: https://huggingface.co/datasets/youliangtan/austin_buds_dataset_converted_externally_to_rlds_to_hg
TODO:

test with more datasets
dynamically include keys (step_metadata, and episode_metadata). that are custom to each rlds dataset (maybe require a transform config file?)
clean up impl

Also, is there any further plan to support the on-browser visualization of the lerobot dataset's (parquet file/videos), without needing the user to download the dataset? :thinking:

Also, I would like to check with the hugging face team. Is there any further plan to support the on-browser visualization of the lerobot dataset's (parquet file/videos), without needing the user to download the dataset? 🤔

@youliangtan @tlpss Yes! We are currently working on this. In particular, we noticed that our video encoding format is only supported by  google-chrome, and not in other common browsers like firefox and safari. We are currently benchmarking other options. Then it will be easy to visualize the datasets on the hub directly ;)

@youliangtan Out of curiosity, do you have higher resolution videos? Also the fps looks a bit off in the video (frames are switching slowly). Is it expected? Thanks!
https://github.com/huggingface/lerobot/assets/4681518/ef7bee3d-d64a-4d34-81c3-2c5d1a67f979
https://github.com/huggingface/lerobot/assets/4681518/da3c518d-fc8e-41fa-bb2f-06f772875811

I have updated the austin_buds dataset to 20fps via the --fps arg. Control Frequency for all OXE datasets is specified here.
https://github.com/huggingface/lerobot/assets/32189404/3583cab8-a702-4ce5-be3b-19ddca7326c2
Also, I added the language_instruction field, and uses T5-small as the default lang tokenizer (similar to octo). i also ignore it in the compute_stats process. The implementation might look abit sketchy, feedback is appreciated. =)

@youliangtan Sorry for the wait. We are going to review this PR soon with @michel-aractingi .
Best

Thanks for reviewing it. I made some changes such that each officially supported open-x dataset should have a "config" to help standardize the RLDS format (used in Octo and OpenVLA) before converting to the LeRobot format.
Currently, this work still requires cleanup and more tests across all OXE datasets. Some high-level comments would be appreciated before I start converting all existing OXE datasets. :pray:
Additionally, I noticed that the Compute mean, min, max: ... process is taking a considerable amount of time (90% of the conversion time). I can open another GitHub issue to address this.

Closing this PR and the work here has been merged in #354 . Thank you @youliangtan!


Add ability to take a photo of the receipt upon delivery

The photo should be saved to the device for the time being. A subsequent enhancement would make it possible to upload this .

This has been done

This has been done


Highlight google search results

I've noticed that this extension also removes highlight on google search results.
Is it possible to change highlights to bold or italic font with color? Otherwise you will need to ctrl+f every time to achieve the same purpose.
Not quite famalir with css, but please let me know if there's anything I can help, thanks!

Sorry, but do you have any example to reproduce this? The original google highlights are in bold, I didn't see the extension removed them. This extension is developed for using in e-ink screen which doesn't care colors. I see you mentioned color,  what scenario do you use this extension?

Sorry for my late reply.

Um I didn't notice that the original google highlights should be in bold since it's not on my side. No clue why it's not, and seems like there's also no related chrome settings.
I'm using e-ink (Mira pro) + LCD dual screen setup, so adding colors is beneficial when switching to the LCD side.

ok, thanks for your feedback. I did some tests, it should be related to the google account settings. When I logout my account, I can see the red color highlights. I will make some research to see if I can keep the colors as more as possible.


Full support for Gitpod

Description
Similar to the issue open for supporting GitHub CodeSpaces. When you try to use Gitpod for sir-lancebot, you have to install lots of things manually.
Reasoning
Quicker development times online as there is no need to manually install dependencies.
Proposed Implementation
Creating a .gitpod.yml file, described here
Would you like to implement this yourself?

[x] I'd like to implement this feature myself
[ ] Anyone can implement this feature

Gonna leave my two cents here:
I don't have anything against gitpod, and wouldn't mind us adding this. What I do have a problem with, however, is if this will "open the floodgates" so to speak. Will this lead to more editor specific changes? If so, which editors do we accommodate?
I'm not going to shoot this down immediately, because it seems cool and helpful, but I also won't approve it until we get more opinions.

I don't think so, there aren't many browser dev environments used besides these two @HassanAbouelela

I think this falls under the same reasoning of if https://github.com/python-discord/sir-lancebot/issues/451 should be kept open as well. Codespaces are now paid for, so not many users are going to have this as an advantage.
My point is: should we implement codespaces at all? @HassanAbouelela @Kronifer please give me your opinions.

I think either way works. It would be nice to have dependencies installed right off the bat, but at the same time it isn't hard to install them

I don't mind adding support for gitpod, I just ask we maintain a high standard for what we add.
Gitpod is a large service, with many users, so it's okay to add. Smaller services are a dime a dozen, and don't make as much sense to support.

@Xithrius, is this approved and could I be assigned?

Yes & yes

@Kronifer should we document the gitpot support in the README, so people know where to quickly start working on the bot?

@jchristgit I'll do that once I get home

Thank you 😌


README.md needs to be updated to reflect AGDC v2 description

README.md currently has the agdc-research-trial documentation and this needs to be replaced with the AGDC v2 description and logo

Fixed


Move the output directory from config.yaml to a flag

Now the output directory is defined in the config.yaml file.
It would be interesting to remove it from there and add it as a flag from the command line.

I can give it a chance tomorrow morning :)


Add nodejs and golang as dependencies.

Hi, how are you?
Can you please add both node js and Go language as dependencies to the boilerplate? I tried to install it on a new ubuntu 23.10 computer without this packeges and it didnt work properly.

Thanks for the project anyway, im rebuilding my config starting from this one!

Fixed, thx for your attention)


Remove hard-coded AWS::Diagram::Canvas

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Found that the hard-coded deletion of AWS::Diagram::Canvas is a change that affects all users because the definition file is cached forever. AWS::Diagram::Canvas will remain hardcoded until #33 is resolved. Close this request and request to change only the definition file again.


Caching for Swift 3 's data type

This is a great library however I cannot make the Cache works with Swift 3's Data type, because Data doesn't conform to NSCoding. Is there a plan to support this use case?

I definitely plan to update the framework for the Codable protocol that will arrive with Swift 4.
In the meantime, the only thing that comes to mind is actually casting Data to NSData.


hotkey with mod+w closes the browser tab

It closes the browser tab even with e.preventDefault() and/or e.stopPropagation()
Is there a way to override this behavior?
hotkeys.add({
  combo: 'mod+w',
  description: 'Close tab',
  allowIn: ['INPUT', 'SELECT', 'TEXTAREA'],
  callback: (e, hotkey) => {
    e.preventDefault()
    e.stopPropagation()
    // code here
  }
})

No. Browsers don't let you prevent users from closing windows or the browser itself using keyboard shortcuts for security and usability reasons.


Scrambled credentials in /me route

Description
When clicking on credentials accepted from the NGDIL demo in the /me route, scrambled credentials are displayed. Scrambled means that the data prop, which contains credentialSubject does not match with the logo and display_name.
It seems that when the backend hands over the state to the frontend, it is already scrambled and the frontend just correctly displays scrambled credentials.
In some cases, the credentials route is blank and the console suggest that this is related to https://github.com/impierce/identity-wallet/pull/351. But this might also be a separate bug.
I also observe inconsistencies with rendered credentials and blank routes in UniMe 0.6.12.
Hardware Specification
n/a
Steps to Reproduce the Bug

On dev, load the credentials from the first journey of the NGDIL demo.
Accept the credentials. You will be redirected to /me.
Tap on different NGDIL issued credentials and compare the credential logo with the text. They do not always match.
In some cases you will see a blank credential page.

Actual Behaviour
Animated gif posted to Slack due to size restriction here.
Expected Behaviour
It should render the credentials exactly as NGDIL issued them.
Errors
If a credential route is rendered as a blank page, this error shows up in the console:
Unhandled Promise Rejection: TypeError: ctx[0].data.credentialSubject[/*field*/ ctx[3]].startsWith is not a function. (In 'ctx[0].data.credentialSubject[/*field*/ ctx[3]].startsWith('data:image/')', 'ctx[0].data.credentialSubject...

This may be a separate bug.
Findings
All of the following findings are frontend only.
Credential offers looks fine:

The /me list also looks fine, i.e. logos match display names:

But the application state read by the CredentialsList already contains scrambled data. Here, e.g., for the National ID:

The credentials route just renders the scrambled credential it found in the state:

This begs the question: do we get scrambled data from the backend? Yes we do. This screenshot shows scrambled data from the payload from the backend that is stored as application store:

Indeed the scrambled credentials bug is different from the blank page bug.
Apparently the credentials get 'scrambled' because the back end assumes that when it does a Batch Credentials Request to the NGDIL issuer that the credentials in the Response are returned in the same order as they have been requested.
The OID4VCI spec indeed states:
Every entry of the array corresponds to the Credential Request object at the same array index in the credential_requests parameter of the Batch Credential Request.
When we receive the credentials, we 'bind' some credential display data to the credentials that we fetch from: https://api.demo.ngdil.com/.well-known/openid-credential-issuer
However, we do this assuming the original order has been respected by the issuer. In this case, that order has not been respected so we incorrectly bound the wrong display data to the wrong credentials.
This means we have a quite crucial reliance on the Issuer in this regard since we need to trust that they will respect original order of the credential IDs in the Batch Credential Request (according to OID4VCI draf13).
The only way of properly preventing this problem is by completely deprecating Batch Credentials, and instead Send multiple 'single' Credential Requests so we have complete control over the order of Credentials that will be issued so that we can always bind the right display data to the right credential. We can safely do this since Batch Credentials are already deprecated in OID4VCI draft 14.
If we do choose for this option we first need to check whether the NGDIL demo supports a flow with multiple single Credential Requests

 Scrambled credentials in /me route


Fix AST error on contains key

This fixes issue #1

@hunter-read Thank you for your contribution.


Does not work with Django 5.1 because get_storage_class is deprecated and removed.

Current Situation
Django DBBackup relies on function get_storage_class which is deprecated since version Django>=4.2 and has been removed in Django 5.1.
python manage.py dbbackup
Traceback (most recent call last):
  File "C:\Users\developer\PycharmProjects\myrcd\myrcd\myrcd\manage.py", line 22, in <module>
    main()
  File "C:\Users\developer\PycharmProjects\myrcd\myrcd\myrcd\manage.py", line 18, in main
    execute_from_command_line(sys.argv)
  File "C:\Users\developer\.conda\envs\myrcd\Lib\site-packages\django\core\management\__init__.py", line 442, in execute_from_command_line
    utility.execute()
  File "C:\Users\developer\.conda\envs\myrcd\Lib\site-packages\django\core\management\__init__.py", line 436, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\developer\.conda\envs\myrcd\Lib\site-packages\django\core\management\__init__.py", line 275, in fetch_command
    klass = load_command_class(app_name, subcommand)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\developer\.conda\envs\myrcd\Lib\site-packages\django\core\management\__init__.py", line 48, in load_command_class
    module = import_module("%s.management.commands.%s" % (app_name, name))
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\developer\.conda\envs\myrcd\Lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "C:\Users\developer\.conda\envs\myrcd\Lib\site-packages\dbbackup\management\commands\dbbackup.py", line 9, in <module>
    from ...storage import StorageError, get_storage
  File "C:\Users\developer\.conda\envs\myrcd\Lib\site-packages\dbbackup\storage.py", line 7, in <module>
    from django.core.files.storage import get_storage_class
ImportError: cannot import name 'get_storage_class' from 'django.core.files.storage' (C:\Users\developer\.conda\envs\myrcd\Lib\site-packages\django\core\files\storage\__init__.py)

Proposed Actions
Make use of Storages instead as described in documentation
System Information
Django==5.1
django-dbbackup==4.1.0

Same error

I've been drafting a fix for this, django.utils.module_loading.import_string covers the deprecated functionality, then dbbackup's management commands need to read storages with some handling for backwards compatibility & non-default storage names.

Same error

Thank you for solving this issue, it works as expected with version 4.2.1. Cheers


Option to display only the window icon (like in original Gnome 3 Alt-Tab)

I find it easier to distinguish windows from one another by looking at their icons rather than a small preview of the window. An option to display the icon instead of a preview would be a great feature for this extension.

It would be great to also give an option to make it work like in Metacity with compositing activated: show the window preview, and a small icon of the window next to the preview. I'll upload a screenshot if you want to see how it look.
Here is an old example that I used for a bug report for Firefox :
http://i12.photobucket.com/albums/a243/OxayotlTheGreat/Capture-1-16.png


Can't work in Blazor WebAssembly App

There is a Blazor Server sample, which is successful. Then I created a Blazor WebAssembly App and build the project. It shows:
Weikio.PluginFramework.Samples.Shared.csproj targets netcoreapp3.1. It can't be targeted to .NETStandard,Version=v2.1
Weikio.PluginFramework.AspNetCore.csproj targets netcoreapp3.1. It can't be targeted to .NETStandard,Version=v2.1

I noticed the TargetFramework in the Blazor WebAssembly App project is:
<PropertyGroup>
    <TargetFramework>netstardard2.1</TargetFramework>
</PropertyGroup>

How to solve this?

I solved this problem by using conditional compilation.
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFrameworks>netstandard2.1;netcoreapp3.1</TargetFrameworks>
    <RazorLangVersion>3.0</RazorLangVersion>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly" Version="3.2.1" />
    <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.Build" Version="3.2.1" PrivateAssets="all" />
    <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.DevServer" Version="3.2.1" PrivateAssets="all" />
    <PackageReference Include="System.Net.Http.Json" Version="3.2.0" />
  </ItemGroup>

  <ItemGroup Condition="'$(TargetFramework)' == 'netcoreapp3.1'">
    <ProjectReference Include="..\..\SourceCodes\PluginFramework\samples\Shared\Weikio.PluginFramework.Samples.Shared\Weikio.PluginFramework.Samples.Shared.csproj" />
    <ProjectReference Include="..\..\SourceCodes\PluginFramework\src\Weikio.PluginFramework.AspNetCore\Weikio.PluginFramework.AspNetCore.csproj" />
  </ItemGroup>

</Project>


Can we run dependency scan on docker images using CLI

Please elaborate your question

ODC is not intended to scan docker images and find the same types of vulnerabilities something like Prisma would find. While ODC would be able to detect some things (like a JAR/WAR deployed in the container that had vulnerable dependencies) but it would not identify vulnerable thing at the OS level or software installed via apk, etc.


@v0.17.1 report this error with go run :# github.com/jmattheis/goverter/xtype ../../../../pkg/mod/github.com/jmattheis/goverter@v0.17.1/xtype/type.go:279:11: t.TypeArgs undefined (type *types.Named has no field or method TypeArgs) make: *** [model_convert] Error 2

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Include the input file that goverter has problems with, and describe the steps
you took to trigger the bug
Expected behavior
A clear and concise description of what you expected to happen.

Could you provide the files to reproduce this bug? Also describe how you've executed goverter.

It v0.17.1 goverter added support for generics, this increased the required go version for building goverter to go1.18. With goverter v0.17.2 the minimum go version for using goverter is go1.16.


[question] Where is download-package.sh ?

I haven't been able to find the download-package.sh script mentioned in README.md

maybe it's related to python download_package method in python/aswfdocker/utils.py ?

Hi! You're correct, I forgot to update the readme, the new way to download packages is to use the aswfdocker download commands line.
Happy New Year 2021!

Bonne annee 2021 - Happy New Year 2021

Bonne annee 2021 - Happy New Year 2021


Fix incorrect variable name on README

Typo in README.md

:+1:

Thank you @likeavirgil :) landed in https://github.com/oauthjs/node-oauth2-server/commit/27fcc9aa2a61388741a99adb587f59d6b4e5f1fe.


fix: add jwt secret env to studio instead of pgmeta

What kind of change does this PR introduce?
Bug fix https://github.com/supabase/cli/pull/2339#issuecomment-2201580014
What is the new behavior?
Add jwt secret to the correct container.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.383%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7021

Relevant Lines:
11438

💛  - Coveralls


update infrastructure to 0.0.23

UpdateBot pushed helm dependency: infrastructure to: 0.0.23

UpdateBot commands:
updatebot push-version --kind helm infrastructure 0.0.23


"Could not locate a valid AppDelegate at root" when running eas build:configure on managed project

Summary
I'm on windows using VS Code with a React Native mobile app. I have an EAS Priority Plan for building and submitting the iOS app to Apple. Steps to reproduce:

eas buld:configure
select "iOS" as the platform to configure for the EAS build
At this point I get a check mark for "Validated eas.json" and then the error message:
"UnexpectedError: Could not locate a valid AppDelegate at root:
"C:\Development\TownsendTech\VesselHealth\Source\VesselHealth"
Please report this as an issue on https://github.com/expo/expo-cli/issues"

There is an App.js file inside the folder listed above as seen in the screenshot below:

In the App.js file an AppDelegate is defined as :
export default function App(){
...
}

Environment
Expo CLI 4.12.0 environment info:
System:
OS: Windows 10 10.0.19042
Binaries:
Node: 14.17.5 - C:\PROGRA~1\nodejs\node.EXE
Yarn: 1.22.11 - C:\Users\Michael Townsend\AppData\Roaming\npm\yarn.CMD
npm: 6.14.14 - C:\PROGRA~1\nodejs\npm.CMD
npmPackages:
expo: ~42.0.1 => 42.0.3
react: 16.13.1 => 16.13.1
react-dom: 16.13.1 => 16.13.1
react-native: ~0.63.4 => 0.63.4
react-native-web: ~0.13.12 => 0.13.18
Expo Workflow: managed
Please specify your device/emulator/simulator platform, model and version
Not sure what is required here. I'm running on Dell XPS and Windows 10
Error output
PS C:\Development\TownsendTech\VesselHealth\Source\VesselHealth> set EXPO_DEBUG=true
PS C:\Development\TownsendTech\VesselHealth\Source\VesselHealth> eas build:configure
💡 The following process will configure your iOS and/or Android project to be compatible with EAS Build. These changes only apply to your local project files and you can safely revert them at any time.
√ Which platforms would you like to configure for EAS Build? » iOS
Warning! Your repository working tree is dirty.
It's recommended to commit all your changes before proceeding, so you can revert the changes made by this command if necessary.
√ Would you like to proceed? ... yes
√ Validated eas.json
UnexpectedError: Could not locate a valid AppDelegate at root:
"C:\Development\TownsendTech\VesselHealth\Source\VesselHealth"
Please report this as an issue on https://github.com/expo/expo-cli/issues
Reproducible demo or steps to reproduce from a blank project
The steps to create a reproduceable demo are huge. I'd have to list the steps to build the entire app. If there is other info I could give, I'll get it though.

Sorry for the slow response. I originally responded to you on my phone but for some reason it didn't go through.
I'm using the eas version: eas-cli/0.29.1 win32-x64 node-v14.17.5
I'd be happy to share the project. The source is housed on github under a private repo. Could I add you as a collaborator so you have access instead of making it public for anyone to download? Or is there another option you would prefer?

hi @michaelto20! you could add me as a collaborator, sure. otherwise if you can try to re-create it in a minimal reproducible example and share it publicly, that also works and is slightly preferable.

@brentvatne sorry I couldn't find a way to re-create it so I've added you as a collaborator. The repo is called VesselHealth.
Also this is still michaelto20, just logged into my other Github account.

@townsendtechllc - sorry this slipped off my radar. is there any chance you could try reproducing this issue in a minimal app? i haven't seen anyone encounter it yet so it seems like a quirk in your app setup, you can try to narrow down the issue with this approach: https://expo.fyi/manual-debugging

Closing because of the inactivity.


expose missing args in BishopBruteForceresult

Parsing or executing a file which had BishopBruteForce as calculation method would raise an error because of the missing exposed attributes for the class BishopBruteForceResults

seems like I've been working on that issue as well, my code is a little more complicated but if this works then it's fine, see https://github.com/Deltares/GEOLib/issues/170

If I use this option I get the following error when parsing a stix file;
Error: Project could not be opened.
Error converting value {null} to type 'System.Double'. Path 'ResultThreshold', line 2, position 27.
I have solved this using ResultThreshold: Optional[float] = 0.0 although this does differ from the other options that are set to None

Fixed in GEOLib v2.3.0 using a similar approach as proposed here


Add support for using callable objects as event handlers.

Hi!
I've started using Busted as test framework. One of it great features is the ability to mock/spy on functions and tables to ensure correct call chains and values. To accomplish that magic, Busted replaces the contents of the original table with wrappers that are all tables. So, when passing these to onEvent added by LuaC, it receives a (callable) table instead of a function, which it in its current version does not support.
This PR changes the onEvent handler to support callable tables. You can view an example of the usage in this test: https://github.com/PerMalmberg/du-unit-testing/blob/main/src/setup_mocks_spec.lua#L23 (this repo isn't ready for use yet as it requires four PRs to NQ's lua-example repo to fix compile errors)
Talking of test, I don't see any tests for LuaC, let me know if there are any and I'll add test cases for this addition.

Support for callable objects and coroutines has now been added as of 5c887be5afec55da54ca73d943c2a4e42b2f1f93. It now matches NQ's support for their own Event implementation.


convertAttributes: set S_ISUD bit from os.FileMode’s os.ModeSetuid

Before this change, the package effectively silently stripped the suid bit.

The macOS travis failure seems unrelated to me.


Trial Fail

Hi, I'm a beginner of optuna, and here I encounter the error

the code is like below
    optuna.storages.RDBStorage("sqlite:///optuna1.db", skip_table_creation=False)
    storage_name = "sqlite:///optuna1.db"
    study = optuna.create_study(
        pruner=optuna.pruners.MedianPruner(n_warmup_steps=3), direction="maximize",
        study_name="fashion_mnist_torch", storage=storage_name,load_if_exists=True
    )
    study.optimize(objective, n_trials=20, timeout=1200)
    pruned_trials = study.get_trials(deepcopy=False, states=[TrialState.PRUNED])
    complete_trials = study.get_trials(deepcopy=False, states=[TrialState.COMPLETE])

Could you tell me why the trial is failed? Thank you very much!

Could you use https://github.com/optuna/optuna/discussions to ask a question? In addition, when you ask a question please kindly share the minimal reproducible code with us/


chore(main): release 0.3.1

:robot: I have created a release beep boop
0.3.1 (2023-11-17)
Continuous Integration

re-enabling python versions 3.8, 3.9 and 3.10 in the CI tests (c30c2a0)

Documentation

small fix to the link of add-sample-hits output in README.md (ef945bf)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/leoisl/plasnet/releases/tag/v0.3.1 :sunflower:


Correct StartLocation

Running CrossPlatformLocation Demo , StartLocation is not Running because it's excluded from{$IF Defined(ANDROID)}

StartLocation is called on Android if RequestPermissions succeeds


Fix lint errors

Changes

Fix lint errors which prevent passing CI like here

/kind cleanup
Supported kinds are: api-change, bug, cleanup, deprecation, removal, documentation, enhancement, performance
/assign @skonto

/lgtm
/approve


Implement workaround for EAttributes with high multiplicity on the M1 level

In LionWeb properties can be 0..1 or 1..1, but they can not have higher multiplicity (so 0..* or 1..* are not supported). In EMF ecore EAttributes can have high multiplicity. To support import of ecore metamodels with EAttributes of higher multiplicity, I updated EMFMetamodelImporter as following: we introduce an intermediate Concept that holds the property corresponding to the EAttribute, and this concept is added to the parent concept as a containment with high multiplicity.
Now we need to update EMFModelImporter and EMFModelExporter so that they are in synch with this meta-model transformation, so that we have a bi-diractional mapping between the original EAtribute and the introduced auxiliary LionCore containment on the M1 level.

Could it make sense to add an annotation on these "synthetic" concepts, so that we know they need special treatment when importing or exporting models?

I had to solve a similar problem with MPS references (0..1) vs. LW references (0..*), with the same approach. I also have annotations like @ftomassetti proposes, they are very helpful (especially for multiple import/export scenarios).

To me it looks natural to implement such model transformations (also the one from [#166]) in the ecore world, using for example Xtend. That's what I also do in Rascal: all model transformations between LionCore and Rascal, on the M1 and M2, are implemented in Rascal. Does it make sense to use Xtend here too?

I wouldn't use Xtend for anything but M2T transformations. For M2M, the only real benefit it brings over modern Java is memoization. But we'd introduce a dependency on a kind of "fringe" technology. Doesn't sound like it's worth the risk to me.

And memoization is easy enough to implement in a GPL (even in a transparent way using HOFs) as well.


Create FAQ

Description
Title
Proposal
Title

https://openlane2.readthedocs.io/en/latest/faq.html


[GOBBLIN-1805] Check watermark for the most recent hour for quiet topics

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1805

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Currently, Audit check is done for [currentWatermark, currentWatermark +1hour] window for quiet topics. However when the audit check for this window fails (due to an upstream bug in audit), the watermark never progress as the currentWatermark is never updated. To avoid this we keep progressing the watermark by checking the window [currentHour -1, currentHour] window

Tests

Updated unit test to reflect this change

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3666 (e24261a) into master (7675dc4) will increase coverage by 4.16%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #3666      +/-   ##
============================================
+ Coverage     46.97%   51.14%   +4.16%
+ Complexity    10780     7210    -3570
============================================
  Files          2138     1345     -793
  Lines         83992    50652   -33340
  Branches       9332     5630    -3702
============================================
- Hits          39455    25904   -13551
+ Misses        40959    22487   -18472
+ Partials       3578     2261    -1317

Impacted Files
Coverage Δ

...obblin/service/FlowConfigResourceLocalHandler.java
12.40% <0.00%> (-3.29%)
:arrow_down:

...blin/service/FlowConfigV2ResourceLocalHandler.java
0.00% <ø> (ø)

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


[SPARK-25717][SQL] Insert overwrite a recreated external and partitioned table may result in incorrect query results

What changes were proposed in this pull request?
Consider the following scenario:
spark.range(100).createTempView("temp")
(0 until 3).foreach { _ =>
  spark.sql("drop table if exists tableA")
  spark.sql("create table if not exists tableA(a int) partitioned by (p int) location 'file:/e:/study/warehouse/tableA'")
  spark.sql("insert overwrite table tableA partition(p=1) select * from temp")
  spark.sql("select count(1) from tableA where p=1").show
}

We expect the count always be 100, but the actual results are as follows:
+--------+
|count(1)|
+--------+
|     100|
+--------+

+--------+
|count(1)|
+--------+
|   200|
+--------+

+--------+
|count(1)|
+--------+
|   300|
+--------+

when spark executes an insert overwrite command,  it gets the historical partition first, and then delete it from fileSystem.
But for recreated external and partitioned table, the partitions were all deleted by the drop  table command with data unremoved. So the historical data is preserved which lead to the query results incorrect.
How was this patch tested?
Add test.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Could you add test case for this change?

Is there any more suggestions? @wangyum @viirya

cc @wangyum

ok to test

Test build #99860 has started for PR 22707 at commit 89f7223.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5878/
Test PASSed.

Test build #99860 has finished for PR 22707 at commit 89f7223.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99860/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5918/
Test PASSed.

Test build #99905 has started for PR 22707 at commit 327ecd9.

Test build #99905 has finished for PR 22707 at commit 327ecd9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

@maropu  I had updated the comment, did you notice that？

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Weird issue when piping a chunked data from HTTP to a file in OS X

From http://stackoverflow.com/questions/30085943/nodejs-unexpected-behaviour-when-piping-chunked-http-data-to-a-file-in-os-x?noredirect=1#comment48289848_30085943.
I'm getting a weird issue when piping a chunked data from HTTP to a file in OS X.
When triggering the GET HTTP request that initiates the download, my server responds with the following headers:
< content-disposition: attachment; filename="resin-Rpi2-0.1.0-0.0.12.img"
< content-encoding: gzip
< content-type: application/octet-stream
< date: Wed, 06 May 2015 19:03:33 GMT
< X-Frame-Options: DENY
< x-powered-by: Express
< transfer-encoding: chunked
< Connection: keep-alive

My code goes like this:
fs = require('fs')
request = require('request')

# If you want to try it by yourself, register to https://dashboard.resin.io
# create a Raspberry Pi application (note the id on the url)
# and get the token from the preferences page.
APPID = '...'
TOKEN = '...'
DEST = 'imagetest.img'

request
  url: "https://dashboard.resin.io/download?network=ethernet&wifiSsid=&wifiKey=&appId=#{APPID}"
  method: 'GET'
  gzip: true
  headers:
      Authorization: "Bearer #{TOKEN}"

.on 'data', (data) ->
  console.log(data)

.pipe(fs.createWriteStream(DEST))

If I run that script, I correctly get the initial chunks (corresponding to about 10MB of data) but the connection then suddenly ends, and no other event (close, end, etc) is triggered from any of the streams.
What's even more weird is that commenting out gzip: true leads to the desired behaviour (I of course get a *.img.gz file in that case), and it also works as expected if keeping the gzip: true and piping to process.stdout instead of to a file.
Also notice the issue is not reproducible on Ubuntu. It only seems to happen in OS X and Windows.
cURL also works as expected:
$ curl -H "Authorization: Bearer [token]" "https://dashboard.resin.io/download?network=ethernet&wifiSsid=&wifiKey=&appId=[appid]" -o image.img

I've proceeded to analyse the packet communication (you can get the pcap here):
I can see my IP is suddenly sending a ACK/FIN while the server is still sending me chunks:
Id = 2984
Source = <IP_ADDRESS>
Destination = <IP_ADDRESS>
Captured Length = 66
Packet Length = 66
Protocol = TCP
Date Received = 2015-05-06 18:11:36 +0000
Time Delta = 10.27246999740601
Information = 52264 -> 443 ([ACK, FIN], Seq=2137874087, Ack=3255859962, Win=10407)

Followed by a long loop of RST and ACK:
...
<IP_ADDRESS> -> <IP_ADDRESS>, 52264 -> 443 RST
<IP_ADDRESS> -> <IP_ADDRESS>, 443 -> 52264 ACK
...

And finally an agreed ACK, FIN from both sides.
After a lot of experimentation, the issue is gone if I write the chunks to the destination stream manually instead of piping:
destination = fs.createWriteStream(DEST)

request
  url: "https://dashboard.resin.io/download?network=ethernet&wifiSsid=&wifiKey=&appId=#{APPID}"
  method: 'GET'
  gzip: true
  headers:
      Authorization: "Bearer #{TOKEN}"

.on 'data', (data) ->
  destination.write(data)

It's worth noting that using the builtin https module works fine, which makes me think it's an issue with request.

I've also bisected the issue until https://github.com/request/request/commit/10246c84819db14b32fccca040029b06449242a3, which is the commit that implements gzip content decoding, which didn't help much.

Thanks for the thorough bug report @jviotti pinging @kevinoid

Thanks for the ping @simov, and thank you for the very detailed bug report @jviotti this looks like an interesting issue.  I'll dig in and see what I can figure out.

@simov @kevinoid You're welcome. Let me know if I can help in any way!

In case it also helps, avoiding the zlib.createGunzip() stream (https://github.com/request/request/blob/master/request.js#L1129) also prevents the issue, which at first made me think it could be a bug in zlib, however using https + zlib.createGunzip() works as expected with the same resource.
Maybe there is a subtle bug in the way request overrides classic stream functions (pipe, write, etc) (https://github.com/request/request/blob/master/request.js#L1501)?

Thanks @jviotti, I think I've figured it out.  It looks like the response stream can get into a stalled state due to how backpressure is handled on the streams.  Request was propagating pause and resume calls directly to the response stream rather than to the gzip stream, which could cause the stream to get stuck in the paused state and cause Node to exit prematurely.
The issue appears to affect all platforms (or, at least, I was able to reproduce it on Linux) but only affects Node 0.11 and 0.12, not 0.10, due to the changes discussed in joyent/node#8351 for how stream.Readable now handles backpressure by pairing pause() with flowing = true instead of resume(), which is different than Request which always uses pause() and resume().  But the issue is intermittent due to the sequencing of calls from each stream, so it can be difficult to test/reproduce reliably.
If you could give the changes in #1568 a try and let me know whether it fixes the issue for you, I'd appreciate it!

@kevinoid I can confirm your fix works for me! Thanks a lot for the support! Can you let me know when this change is published to NPM please?


Merge monolith/aiodocker into aiodocker

I am just opening a work in progress pull request. I have been doing some major remodeling and wanted you to have an ability to look at all the changes I making so that you will be on board with them. This is definitely shifting quiet a lot from the way you originally designed it and I am taking in more inspiration from docker-py but with all asyncio capabilities. Currently I partially implemented the Image API stuff and moving forward towards the Container API. Once the container api is complete I'll make sure that all your original unittests pass so that the library will be backwards compatible but I also really want to have a discussion around how you foresee the interface. This way we could coordinate between what I want and what you want. Also note that you will only see one commit. This is because I am developing the entire thing within a monolithic repository and currently have issues exporting commit history without mingling it with history of other projects we are developing on our end. Just keep me posted with any remarks you have and we can talk them out.
One more remark. I removed your pytest stuff because I am just not very familiar with pytest but I am pretty good with python's builtin unittest module. Also I removed some of the other files like travis-ci since I am also fully familiar with how to use those. At a slightly later stage I'll make sure to setup a different CI tool or read up on travis ci and make sure that it will work well with the unittest module an the unittests I am writing.

The BUILD file is due to my internal monolithic build system. We use bazel to run builds with a pex bundler and that's why that file is there. We can remove it from the public version of this library.

Added a working version of the container functionalities but did not yet finish all the unittests for both image and container functions. Once I am done with the unittests, the next step will be to actually make it backward compatible with the functions you previously had like "image.run()" and make sure the new design passes your previous unittests

:+1: thank you!

Replaces your jsonstream with semi-kinda version of StreamReader() to allow functionalities like containers.logs() to work when the user wants to return the stream.

@paultag Sorry for the delays. I got really busy with some of my own work and what I had built here was enough for me to keep moving on with my own system. I'll circle back to this really soon.

Okay I might have been quiet ambitious. I ended up basically building a different library. I have added network api support and have been trying to backward campat to your and hmmm... this is taking tons of time.


mleap-xgboost-runtime doesn't store tree_limit

We're not serializing the tree_limit param here:
https://github.com/combust/mleap/blob/master/mleap-xgboost-runtime/src/main/scala/ml/combust/mleap/xgboost/runtime/bundle/ops/XGBoostClassificationOp.scala#L27
Fixed in #637

Fixed in #637


An existing anaconda 64bit installation(python 3.6.1) could not be automatically found

I have installed latest Visual Studio 2017 15.2 because finally the python workload is released in this version.
However, an existing anaconda 64 bit installation (python 3.6.1) cannot be found automatically either during VS installation or later launch into the IDE , this performs really differently when I used previous PTVS in VS 2015. The same anaconda installation could be automatically found and integrated into VS 2015 without any manually configuration.
The Diagnostic Info could be found as
Diagnostic Info 2017-05-11T194241.txt

We changed how our auto-detection worked in VS 2017 (to be consistent with PEP 514, which is the official way to register Python installs on Windows), but Anaconda also updated their installer to write the correct keys.
If you have installed a recent version of Anaconda (which I think you must have, since you've got 3.6.1), they should have set the keys. You may want to try repairing your Anaconda install to see if that fixes it, and be aware that if you select the "set as my default Python" option when installing Anaconda you should see it twice (because it will register itself twice - one will just look like "Python 3.6" though).

I have checked the registry key referred in PEP 514 and found only the keys created by Visual Studio PTVS. I guess it is caused by this Anaconda is installed through conda command "conda install anaconda", not installed by an anaconda installer. However, I have remembered when I installed conda with miniconda3 installer, it also reminded me to "set as my default Python", although it seems not works as expected according to PEP 514.

That's strange, but perhaps miniconda does not set the registry entries at all. Certainly I would expect that conda install anaconda would not set any registry keys.
Managing conda environments properly is on our roadmap, but for now you will need to manually configure them.

Got it, thanks a lot for your advice.
On Sat, May 13, 2017 at 1:21 AM, Steve Dower<EMAIL_ADDRESS>wrote:

That's strange, but perhaps miniconda does not set the registry entries at
all. Certainly I would expect that conda install anaconda would not set
any registry keys.
Managing conda environments properly is on our roadmap, but for now you
will need to manually configure them.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/PTVS/issues/2530#issuecomment-301135851,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGaaYBA4e0she2tuh-KtpBQ9mVSJk1BDks5r5JULgaJpZM4NX6VT
.


[stable/kubernetes-dashboard] permissions secret when rbac.clusterReadOnlyRole=True

Describe the bug
when using --set rbac.clusterReadOnlyRole=True, the pod logs crash with error:
panic: secrets is forbidden: User "system:serviceaccount:kube-system:dashboard-kubernetes-dashboard" cannot create resource "secrets" in API group "" in the namespace "kube-system"
goroutine 1 [running]:
github.com/kubernetes/dashboard/src/app/backend/auth/jwe.(*rsaKeyHolder).init(0xc4204e30c0)
        /home/travis/build/kubernetes/dashboard/.tmp/backend/src/github.com/kubernetes/dashboard/src/app/backend/auth/jwe/keyholder.go:131 +0x35e
github.com/kubernetes/dashboard/src/app/backend/auth/jwe.NewRSAKeyHolder(0x1367500, 0xc420324c60, 0xc420324c60, 0x1213a6e)
        /home/travis/build/kubernetes/dashboard/.tmp/backend/src/github.com/kubernetes/dashboard/src/app/backend/auth/jwe/keyholder.go:170 +0x64
main.initAuthManager(0x13663e0, 0xc4203259e0, 0xc4204c7cd8, 0x1)
        /home/travis/build/kubernetes/dashboard/.tmp/backend/src/github.com/kubernetes/dashboard/src/app/backend/dashboard.go:185 +0x12c
main.main()
        /home/travis/build/kubernetes/dashboard/.tmp/backend/src/github.com/kubernetes/dashboard/src/app/backend/dashboard.go:103 +0x26b

Version of Helm and Kubernetes:
k8s 1.15
helm
Which chart:
kubernetes-dashboard-1.5.3      1.10.1
What happened:
It works with full cluster admin but not with the ReadOnly one.
What you expected to happen:
work
How to reproduce it (as minimally and precisely as possible):
helm delete --purge dashboard
kubectl -n kube-system delete secret kubernetes-dashboard-key-holder

helm install --name dashboard stable/kubernetes-dashboard --namespace kube-system --set rbac.create=True --set rbac.clusterReadOnlyRole=True
Anything else we need to know:
IMPORTANT NOTE:
If the chart was installed previously with ClusterAdminRole, clusterReadOnlyRole seems to be ok.
But if it's the first time the chart is getting installed, there will be above errors.
Reason: kubernetes-dashboard-key-holder secret is not getting deleted by helm delete --purge <>
There were previous issues, most were closed. It might be it was fixed before but now comes again.
#3104
#4102
#5493
#14289
https://github.com/helm/charts/pull/12496/files#diff-814573c2b6c423b43e4c90eb469a41c8

still valid...

The problem is caused by missing permissions. There are two possible solutions:
Solution 1:
Add the rolebing for the role.
Needed changes for this are in https://github.com/helm/charts/blob/master/stable/kubernetes-dashboard/templates/rolebinding.yaml:
Line 3+4:
{{- if or .Values.rbac.clusterAdminRole .Values.rbac.clusterReadOnlyRole }}
# Cluster role binding for clusterAdminRole == true or clusterReadOnlyRole=true

==>
{{- if .Values.rbac.clusterAdminRole }}
# Cluster role binding for clusterAdminRole == true

and Line 27:
# Role binding for clusterAdminRole == false and clusterReadOnlyRole=false

==>
# Role binding for clusterAdminRole == false

Solution 2:
Add Verb create to two resources in clusterrole-readonly https://github.com/helm/charts/blob/master/stable/kubernetes-dashboard/templates/clusterrole-readonly.yaml:
Line 21 - 24:
   verbs:
     - get
     - update
     - delete

==>
   verbs:
     - get
     - create
     - update
     - delete

and Line 33 - 35:
   verbs:
     - get
     - update

==>
   verbs:
     - get
     - create
     - update

Quickfix, when deployed in Namespace kube-system:
kubectl create rolebinding kubernetes-dashboard --role=kubernetes-dashboard --serviceaccount=kube-system:kubernetes-dashboard --namespace=kube-system

You don't need to add permissions if you create the secret, similarly to how one secret is created already: https://github.com/helm/charts/blob/de8f4384226b70ea1c479acc7ff24562648c3adc/stable/kubernetes-dashboard/templates/secret.yaml
apiVersion: v1
kind: Secret
metadata:
  labels:
    app: kubernetes-dashboard
    chart: kubernetes-dashboard-1.8.0
    heritage: Tiller
    release: backbone
  name: kubernetes-dashboard-key-holder
  namespace: kube-system
type: Opaque

I think this should make its way to secret.yaml.

still valid...

issue is still thre.
On Fri, Nov 15, 2019 at 11:26 AM stale[bot]<EMAIL_ADDRESS>wrote:

This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Any
further update will cause the issue/pull request to no longer be considered
stale. Thank you for your contributions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/helm/charts/issues/15118?email_source=notifications&email_token=ABXDOMYD7TP6OWBDGAT64VTQTZTNVA5CNFSM4H4EWOS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEE2ZNQ#issuecomment-554282166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXDOMY6ICB5HOHDIN23VVLQTZTNVANCNFSM4H4EWOSQ
.


⚠ ClamAV has degraded performance

In e945324, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 93 ms

Resolved: ClamAV performance has improved in cf6726b.


error using particlesets read in from version 2 format

the following script:
from amuse.io import write_set_to_file,read_set_from_file
from amuse.ic.plummer import new_plummer_model
p=new_plummer_model(100)
write_set_to_file(p,"test","amuse",append_to_file=False, version="2.0")
p=read_set_from_file("test","amuse")
p.new_attribute=2*p.mass

generates an error:
Traceback (most recent call last):
  File "test.py", line 10, in <module>
    p.new_attribute=2*p.mass
  File "/home/inti/code/amuse/amuse/src/amuse/datamodel/particles.py", line 1106, in __setattr__
    self.set_values_in_store(None, [name_of_the_attribute], [self._convert_from_entities_or_quantities(value)])
  File "/home/inti/code/amuse/amuse/src/amuse/datamodel/particles.py", line 1358, in set_values_in_store
    self._private.attribute_storage.set_values_in_store(indices, attributes, values)
  File "/home/inti/code/amuse/amuse/src/amuse/io/store_v2.py", line 483, in set_values_in_store
    self.attributesgroup
  File "/home/inti/code/amuse/amuse/src/amuse/io/store_v2.py", line 70, in new_attribute
    dataset = group.create_dataset(name, shape=shape, dtype=input.number.dtype)
  File "/home/inti/code/amuse/prerequisites/lib/python2.7/site-packages/h5py/_hl/group.py", line 69, in create_dataset
    dsid = dataset.make_new_dset(self, shape, dtype, data, **kwds)
  File "/home/inti/code/amuse/prerequisites/lib/python2.7/site-packages/h5py/_hl/dataset.py", line 49, in make_new_dset
    shape = tuple(shape)
TypeError: 'int' object is not iterable
Exit 1

it does not if the version=2.0 is omitted

I've updated the code so that it will work, but you'll need an extra parameter 'allow_writing' to be set to True.
The real cause is in a non-backward compatible change in version 2, for version 1 we always make a "working copy" (an in memory version of the set), for version 2 this depends on an option (by default no working copy).
This is the option:
return_working_copy
If set to True, return a working copy in memory you can manipulate,
savepoint etc. Only available for version 2.0 in version 1.0
a working copy (i.e. a particles set or grid in memory)
is always returned. (default: False)
So another fix is to do:
read_set_from_file("test","amuse", return_working_copy=True)

so what does allow_writing do? (will this change the file?)

and maybe return_working_copy=True would be what naieve users would expect?

closing, but there will be a new one for this


SMT deletion

@wborgeaud -- looks like this can be merged?

@wborgeaud -- looks like this can be merged?

I'm getting some failing tests in the test suite due to SMT deletion, so will try to fix those before merging this PR.

@wborgeaud Is that on a public branch? I could give it a look as well

@wborgeaud Is that on a public branch? I could give it a look as well

Thanks! It is now ;)
I pushed the modified test runner to https://github.com/0xPolygonZero/evm-tests/tree/type2 and the Plonky2 branch I'm testing against is https://github.com/0xPolygonZero/plonky2/tree/type2/testing which is the combination of this PR and #1333.
I haven't run the full test suite, but some failing tests I'm seeing in stCallCodes are
callcodeInInitcodeToExistingContractWithValueTransfer_d0g0v0_Shanghai, callcodeDynamicCode2SelfCall_d0g0v0_Shanghai, callcodeInInitcodeToExisContractWithVTransferNEMoney_d1g0v0_Shanghai, callcodeInInitcodeToExisContractWithVTransferNEMoney_d0g0v0_Shanghai, callcallcodecallcode_011_OOGMAfter_d0g0v0_Shanghai, callcodecallcodecall_110_OOGMAfter_d0g0v0_Shanghai, callcodeInInitcodeToExistingContract_d1g0v0_Shanghai, callcodeInInitcodeToExistingContract_d0g0v0_Shanghai, callcodeInInitcodeToEmptyContract_d1g0v0_Shanghai, callcodeInInitcodeToEmptyContract_d0g0v0_Shanghai, callcodeDynamicCode_d1g0v0_Shanghai, callcodeDynamicCode_d2g0v0_Shanghai, callcodeDynamicCode_d3g0v0_Shanghai, callcodeDynamicCode_d0g0v0_Shanghai, callcodecallcodecallcode_111_OOGMAfter_d0g0v0_Shanghai, callcallcodecall_010_OOGMAfter_d0g0v0_Shanghai, callcall_00_OOGE_d0g0v0_Shanghai

Some failing tests with the latest type2/testing in stCreate:
CreateResults_d23g0v0_Shanghai, CreateResults_d19g0v0_Shanghai, CreateResults_d25g0v0_Shanghai, CreateResults_d24g0v0_Shanghai, CreateResults_d21g0v0_Shanghai, CreateResults_d18g0v0_Shanghai, CreateResults_d20g0v0_Shanghai, CreateResults_d22g0v0_Shanghai, RevertInCreateInInitCreate2_d0g0v0_Shanghai, create2collisionStorage_d0g0v0_Shanghai, create2collisionStorage_d1g0v0_Shanghai, create2collisionStorage_d2g0v0_Shanghai

Merging this and opening another PR with several bug fixes.


Copy-dbalogin - How could we make this to use multiple streams/paths for coping faster

Summary of new feature
To speed up the coping of Logins from one server to another, buy using multiple streams/paths
Proposed technical details (if applicable)
My client has over 1000 Logins, we are migrating from one server to another this  Copy-dbalogin take 2 hrs to complete.  If there was a way to create multiple Steams/paths to run the copy over at the same time.
Something like an option to split into 3 (# can be change) stream/paths running at the same time, buy sort order, divide the number of user by 3 the start those 3 stream/paths at the same , spin up 3 sub PowerShell commands to run .  So that it can copy the logins over faster.
Latest version of dbatools as of writing
1.0.124

get the logins, split into arrays of your choosing and pass them to the -Login parameter

Then to run them at the same time?( with out manually opening new PowerShell Windows to run the commands) and then send the info back so that the info is merged back together.

this just depends on how you want to do things, either via -asjob or runspaces (in which case, either PoshRSJobs or Invoke-Parallel will do the job)

Ok if I understand this ,
I have for example:
Import-Module dbatools #for Con $MyInst= "SQLOldInst" $Mlist=Get-DbaLogin -SqlInstance $MyInst #Copy-DbaLogin -Source $localsourceInstance -Destination $localDestinationInstance -Verbose $loginsCollection0 = @(); $loginsCollection1 = @(); $loginsCollection2 = @(); $loginsCollection3 = @(); foreach($v1 in $Mlist) { if ($v1.name -like "0*") { $loginsCollection0 += $v1; } elseif ($v1.name -like "A*") { $loginsCollection1 += $v1; } elseif ($v1.name -like "B*") { $loginsCollection2 += $v1; } else { $loginsCollection3 += $v1; } $loginsCollection0 | Copy-DbaLogin -Destination "NEWSqlInst" $loginsCollection1 | Copy-DbaLogin -Destination "NEWSqlInst" $loginsCollection3 | Copy-DbaLogin -Destination "NEWSqlInst" $loginsCollection3 | Copy-DbaLogin -Destination "NEWSqlInst"
The part that I don't understand is how to use the Invoke-Parallel to run those 4 commands at the same time with this command.

That didn't format very well

Ok So this is what I got, but Not sure if I am using it correctly:
Import-Module 'C:\Program Files\WindowsPowerShell\Modules\DBATools\1.0.124\internal\functions\flowcontrol\Invoke-Parallel.ps1'     $Mlist=Get-DbaLogin -SqlInstance $localsourceInstance;     #https://gallery.technet.microsoft.com/scriptcenter/Split-an-array-into-parts-4357dcc1     $loginsCollectionArray= Split-array -inArray $Mlist -parts 6;     1..6 | Invoke-Parallel -RunspaceTimeout 60 -ScriptBlock {         "Starting $_"         #Start-Sleep -Seconds $_         "If you see this, we didn't timeout $_"         if ($_ -eq 1) {         Write-Host "Copy-logins part 0"         $loginsCollectionArray[0] | Copy-DbaLogin -Destination $localDestinationInstance         }         if($_ -eq 2) {         Write-Host "Copy-logins part 1"         $loginsCollectionArray[1] | Copy-DbaLogin -Destination $localDestinationInstance         }         if($_ -eq 3){         Write-Host "Copy-logins part 2"         $loginsCollectionArray[2] | Copy-DbaLogin -Destination $localDestinationInstance         }         if($_ -eq 4){         Write-Host "Copy-logins part 3"         $loginsCollectionArray[3] | Copy-DbaLogin -Destination $localDestinationInstance         }         if($_ -eq 5){         Write-Host "Copy-logins part 4"         $loginsCollectionArray[4] | Copy-DbaLogin -Destination $localDestinationInstance         }         if($_ -eq 6){         Write-Host "Copy-logins part 5"         $loginsCollectionArray[5] | Copy-DbaLogin -Destination $localDestinationInstance         }

Get this error for the above code:
Get-RunspaceData : Cannot index into a null array.
At C:\Program Files\WindowsPowerShell\Modules\DBATools\1.0.124\internal\functions\flowcontrol\Invoke-Parallel.ps1:528 char:17

            Get-RunspaceData

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

CategoryInfo          : InvalidOperation: (:) [Write-Error], RuntimeException
FullyQualifiedErrorId : NullArray,Get-RunspaceData

Error fixed, was missing -ImportVariables in  Invoke-Parallel -RunspaceTimeout 60 -ScriptBlock
fixed : Invoke-Parallel -RunspaceTimeout 60 -ImportVariables -ScriptBlock {},
now I just need to see what other errors are comming up.

I will close this for now, but you can reopen it if you have further questions.


VSCode is CPU usage is very high

temp.zip
Product and Version [VSCode]:1.28.2
OS Version: Linux (Ubuntu, Arch Linux)
Live Share Extension Version: 0.3.896
Steps to Reproduce / Scenario:
Connect two computers and scroll around

Thanks for reporting this issue. We will try to reproduce it. Do you have any specific reproducing steps we need to try?

Possibly related issue #1076

@nilsocket A few things:
Looking at your logs, there is a Service not available: workspaceTask error which could indicate you or the person you are working with has a corrupted install. Can you try uninstalling, restarting VS Code (not just reloading), and installing again to see if that resolves the issue?
If not, we had a report of this as #1076 on the "guest" side (the person joining). At the time we suspected it was an extension interaction since we could not repro and the problem went away for the user so we never got to root cause.  Your logs look like they are from the guest side and I am unable to repro this on Ubuntu, so I'll double check on Manjaro, but this could be extension related.
To double check - are you seeing the high CPU on the guest, the host, or both?
For #1076, there was a "rg" process from VS Code that slowly consumed memory and caused high CPU. If you look at processes, which process seems to be consuming memory and CPU?

I will look into it and send the report within 10 hours or so.
On Thu, Nov 8, 2018 at 2:25 AM Chuck Lantz<EMAIL_ADDRESS>wrote:

@nilsocket https://github.com/nilsocket A few things:
Looking at your logs, there is a Service not available: workspaceTask
error which could indicate you or the person you are working with has a
corrupted install. Can you try uninstalling, restarting VS Code (not just
reloading), and installing again to see if that resolves the issue?
If not, we had a report of this as #1076
https://github.com/MicrosoftDocs/live-share/issues/1076 on the "guest"
side (the person joining). At the time we suspected it was an extension
interaction since we could not repro and the problem went away for the user
so we never got to root cause. Your logs look like they are from the guest
side and I am unable to repro this on Ubuntu, so I'll double check on
Manjaro, but this could be extension related.
To double check - are you seeing the high CPU on the guest, the host, or
both?
For #1076 https://github.com/MicrosoftDocs/live-share/issues/1076,
there was a "rg" process from VS Code that slowly consumed memory and
caused high CPU. If you look at processes, which process seems to be
consuming memory and CPU?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/live-share/issues/1251#issuecomment-436773790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcbznK7IRDee6nb-GSIzGgDMfZgn4wZuks5us0iugaJpZM4YNyT4
.

--
from:- nilsocket

Sorry to inform, I lost access to the other laptop.
Sorry.
On Fri 9 Nov, 2018, 10:42 AM Nilsocket<EMAIL_ADDRESS>wrote:

I will look into it and send the report within 10 hours or so.
On Thu, Nov 8, 2018 at 2:25 AM Chuck Lantz<EMAIL_ADDRESS>wrote:

@nilsocket https://github.com/nilsocket A few things:
Looking at your logs, there is a Service not available: workspaceTask
error which could indicate you or the person you are working with has a
corrupted install. Can you try uninstalling, restarting VS Code (not just
reloading), and installing again to see if that resolves the issue?
If not, we had a report of this as #1076
https://github.com/MicrosoftDocs/live-share/issues/1076 on the "guest"
side (the person joining). At the time we suspected it was an extension
interaction since we could not repro and the problem went away for the user
so we never got to root cause. Your logs look like they are from the guest
side and I am unable to repro this on Ubuntu, so I'll double check on
Manjaro, but this could be extension related.
To double check - are you seeing the high CPU on the guest, the host, or
both?
For #1076 https://github.com/MicrosoftDocs/live-share/issues/1076,
there was a "rg" process from VS Code that slowly consumed memory and
caused high CPU. If you look at processes, which process seems to be
consuming memory and CPU?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/live-share/issues/1251#issuecomment-436773790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcbznK7IRDee6nb-GSIzGgDMfZgn4wZuks5us0iugaJpZM4YNyT4
.

--
from:- nilsocket


seg fault in legalRecompute

Private repo seg fault in 1.10 is an assertion fail in 1.9. Output below is from 1.10
out.txt

Can you share (perhaps privately) the code that causes this?
Ideally a minimal reproducer


make sure the msg is bytes

The user may pass the non-bytes variable to the AuthManager, thus the msg=salt+server_key+self.client_key may not be bytes, which causes an error on python2.6.

Looks good, thank you! Sorry for the late reply.
Further improves Python 2.6 support (Issue #1)


Device: button - Samjin

Manufacturer
button
Model
Samjin
Battery Type
CR2450
Battery Quantity
1

Actual model of device from manufacturer:
Samsung SmartThings Button [GP-U999SJVLEAA]


bug: kcl mod add k8s panic

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)

2. What did you expect to see? (Required)
kcl mod add k8s should  add the dependency k8s:1.29 successfully.
3. What did you see instead (Required)
It panic.
4. What is your KCL components version? (Required)
The main branch of KCL CLI installed by go install kcl-lang.io/cli/cmd/kcl@main.

Closed by https://github.com/kcl-lang/kpm/pull/239


Where conditions as associative array

$builder
  ->select()
  ->from('table')
  ->where(['id' => 5])

Closed by: 55a8450


Whitelist - stop creating duplicate notification issues

I've been checking the code, but have no idea how this can happen.
The issue is created only if the account is not in whitelist yet.
All the logs from when this happened the last time are gone, so I need to wait for another occasion.

Looking into the logs again and it really seems like some race condition - that the installation event caused github to actually sent two events to the service. Each was taken and performed by one worker at "exactly" the same time.
Service
[Sun Nov 10 17:35:55.048455 2019] [wsgi:error] [pid 16:tid<PHONE_NUMBER>33984] [client <IP_ADDRESS>:56396] 17:35:55.048 webhooks.py       DEBUG  payload signature OK.
<IP_ADDRESS> - - [10/Nov/2019:17:35:55 +0000] "POST /api/webhooks/github HTTP/1.1" 202 42
[Sun Nov 10 17:35:55.133971 2019] [wsgi:error] [pid 237:tid<PHONE_NUMBER>16928] [client <IP_ADDRESS>:56760] 17:35:55.133 webhooks.py       DEBUG  payload signature OK.
<IP_ADDRESS> - - [10/Nov/2019:17:35:55 +0000] "POST /api/webhooks/github HTTP/1.1" 202 42

Worker-0
[2019-11-10 17:35:55,135: INFO/MainProcess] Received task: task.steve_jobs.process_message[837fcfd2-0a1f-4205-ad78-d4780b77b7ea]
[2019-11-10 17:35:55,137: INFO/ForkPoolWorker-1] Github App installation event. Action: created, id: 5098436
[2019-11-10 17:35:55,137: WARNING/ForkPoolWorker-1] Cannot obtain project from this event!
[2019-11-10 17:35:55,137: WARNING/ForkPoolWorker-1] Skipping private repository check!
[2019-11-10 17:35:55,968: ERROR/ForkPoolWorker-1] Failed to verify that user is Fedora packager. This could be caused by different github username than FAS username or that user is not a packager.
[2019-11-10 17:35:55,972: INFO/ForkPoolWorker-1] Account storaged-project inserted to whitelist with status: waiting for approval
[2019-11-10 17:35:57,642: INFO/ForkPoolWorker-1] Account: storaged-project needs to be approved manually!
[2019-11-10 17:35:57,643: INFO/ForkPoolWorker-1] cleaning up the mess
[2019-11-10 17:35:57,646: INFO/ForkPoolWorker-1] Task task.steve_jobs.process_message[837fcfd2-0a1f-4205-ad78-d4780b77b7ea] succeeded in 2.509502837434411s: {'jobs': {'add_to_whitelist': {'success': True, 'details': {...}}}, 'event': {'trigger': 'installation', 'created_at':<PHONE_NUMBER>, 'installation_id': 5098436, 'account_login': 'storaged-project', 'account_id': 11505466, 'account_url': 'https://api.github.com/users/storaged-project', 'account_type': 'Organization', 'repositories': ['storaged-project/blivet-gui'], 'sender_id': 5063197, 'sender_login': 'vojtechtrefny', 'status': 'waiting'}}

Worker-1
[2019-11-10 17:35:55,050: INFO/MainProcess] Received task: task.steve_jobs.process_message[d8bf6659-9005-4f01-8512-28307606fa8d]
[2019-11-10 17:35:55,051: INFO/ForkPoolWorker-1] Github App installation event. Action: created, id: 5098436
[2019-11-10 17:35:55,052: WARNING/ForkPoolWorker-1] Cannot obtain project from this event!
[2019-11-10 17:35:55,052: WARNING/ForkPoolWorker-1] Skipping private repository check!
[2019-11-10 17:35:57,434: ERROR/ForkPoolWorker-1] Failed to verify that user is Fedora packager. This could be caused by different github username than FAS username or that user is not a packager.
[2019-11-10 17:35:57,438: INFO/ForkPoolWorker-1] Account storaged-project inserted to whitelist with status: waiting for approval
[2019-11-10 17:35:59,133: INFO/ForkPoolWorker-1] Account: storaged-project needs to be approved manually!
[2019-11-10 17:35:59,133: INFO/ForkPoolWorker-1] cleaning up the mess
[2019-11-10 17:35:59,137: INFO/ForkPoolWorker-1] Task task.steve_jobs.process_message[d8bf6659-9005-4f01-8512-28307606fa8d] succeeded in 4.085183219984174s: {'jobs': {'add_to_whitelist': {'success': True, 'details': {...}}}, 'event': {'trigger': 'installation', 'created_at':<PHONE_NUMBER>, 'installation_id': 5098436, 'account_login': 'storaged-project', 'account_id': 11505466, 'account_url': 'https://api.github.com/users/storaged-project', 'account_type': 'Organization', 'repositories': ['storaged-project/blivet-gui'], 'sender_id': 5063197, 'sender_login': 'vojtechtrefny', 'status': 'waiting'}}

Suggested solution: #219

installation event caused github to actually sent two events to the service.

The only difference is that one of them has X-GitHub-Event integration_installation while the other has X-GitHub-Event installation. We can ignore the first one - see #219


[Question] Is it possible to create a default user that automatically logs in?

Hi!
So ive been searching for a link shortener that me and a couple of my colleagues can use for work, and was just wondering if the titles question is possible? I would be using Cloudflare Zero Trust to expose this instance to the public, adding MFA for /dash and /api , so i dont think having accounts is necessary for my use case?
It would be easier if i could just add users to Zero Trust and they all get access to the same Lynx "account".
Thank you beforehand for any response <3

It's not currently implemented but I'll have a look into adding it as I definitely see a use for it

Added in 1.8.0.
Requires SOLE_USER to be set to the user's ID, and for the user to set the below setting. Therefore both the server and the user need to agree they want this to be enabled.
If any endpoints that require authentication are accessed and no cookie is provided, it'll create the cookie and apply it to the current request.


tvl endpoint is missing in docs (README)

Hi,
Seems like the tvl endpoint is working, but there is no mention of it in the docs.
Furthermore, what is the output format ?
I see the following for example :
{
  "25": {
     ...
  },
  "56": {
     ...
   },
  "137": {
     ...
  }
}

What do these numbers mean ?
Thanks

Hi. The "numbers" you reference are the "Chain ID"s of the blockchains on which the Beefy vault contracts have been deployed.
Btw, the most active place to discuss aspects of Beefy's protocol is in the Beefy discord, the #development channel in this case.

Thanks, thought about discord but didn't know if this was too specific...
Shall I close this then, or leave it until info is in the README ?


Please invite me to the Recode-Hive GitHub Community Organization

Name
Divya Bhutani
Discord Username (if applicable)
divya09346
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


(PDB-4177) Fix bash in place

Prior to these commits the producer-timestamp wasn't sent in the correct format from the terminus or pulled out of the query parameters. This caused all cmdrefs to have nil values for producer-ts.

Test PASSed

Test PASSed

Test PASSed

Test PASSed

Test PASSed

Test PASSed

Test PASSed

will need rebase once logging commit goes in

Test PASSed

Test PASSed

Test PASSed

Test PASSed

Looks good.


Broken Demo

Hi, thanks for the lib!
The demo website is broken for me and doesn't show anything in production (both Chrome & FE, and in incognito mode).
Also no special error in the console.
I've cloned the repo and run docs:dev, and the docs worked.
Guess it's something with the deploy script.
LMK if I can help!

Yeah - I was trying to fix it (https://github.com/Andarist/react-textarea-autosize/pull/295), but couldn't configure it correctly with Parcel 2 (I thought it would actually be easier to migrate to Parcel 2 rather than fixing Parcel 1 config, but maybe that was a mistake).
If you could take a look at this and fix it - I would highly appreciate it as I couldn't get time to do it for the past few months.

I've managed to finally fix it. The example page is up and running.


replaced msdn links

Replaced [Events](http://msdn.microsoft.com/library/b6f65241-e0ad-4590-a99f-200ce741bb1f) with [Handling and Raising Events](~/docs/standard/events/index.md)
Related to #2037

I think that’s good @BillWagner. It’s a global search and replace, so in theory, if build report looks ok and it works for some, it should work for all.


Supporting not public secret stores

This is follow up to #2993.
We would like to propose adding gRPC secret store, provider and client. The client should be capable of using a remote server and use gRPC for communication. Remote server would be responsible for communication with secret store. This approach could be used to integrate secret stores that are not:

Free Open Source Software.
Publicly available (e.g. as CSP offering).

An obvious alternative would be to use webhook provider, but it supports only simple use cases, e.g. does not support complex authorization logic that Cerberus uses.
Proposed gRPC interface would mimic behaviour of SecretsClient interface.

@moolen, is this something that community would be open to?

hey, we've been investigating this in this issue here: https://github.com/external-secrets/external-secrets/issues/696#issuecomment-1605931532
We've discontinued working on it due to time constraints. I'm happy to support if you want to put time and effort into it ;)


Unable to run an audit starting with the 07/14/2023 portable release

Hello,
I've been able to use PurpleHats with any release after 07/04.
I execute "Node Index" and it just stops. I found this in "errors.txt":
{"timestamp":"2023-08-22 11:50:11","level":"error","message":"uncaughtException: Unexpected end of JSON input\nSyntaxError: Unexpected end of JSON input\n    at JSON.parse ()\n    at getUserDataTxt (file:///C:/Users/Jackc/Desktop/Purple0822/purple-hats/utils.js:68:27)\n    at file:///C:/Users/Jackc/Desktop/Purple0822/purple-hats/constants/questions.js:12:18\n    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)\n    at async Promise.all (index 0)\n    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)\n    at async loadESM (node:internal/process/esm_loader:91:5)\n    at async handleMainPromise (node:internal/modules/run_main:65:12)"}
Can you help? I'd like to take advantage of the latest updates. I'm on Windows10

Thank you!!! I'm downloading the latest release, cool to see we'll be able to specify the path for results. I do miss the old excel export, but I'm making due. :-)

@younglim Different issue with the latest release. I'll debug on my end to see if I can figure anything out, but I'm not a coder. {"timestamp":"2023-08-24 09:23:35","level":"error","message":"uncaughtException: Unexpected end of JSON input\nSyntaxError: Unexpected end of JSON input\n    at JSON.parse ()\n    at getUserDataTxt (file:///C:/Users/Jackc/Desktop/Purple_0824/purple-hats/utils.js:74:27)\n    at file:///C:/Users/Jackc/Desktop/Purple_0824/purple-hats/constants/questions.js:12:18\n    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)\n    at async Promise.all (index 0)\n    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)\n    at async loadESM (node:internal/process/esm_loader:91:5)\n    at async handleMainPromise (node:internal/modules/run_main:65:12)"}

Hi @jcrain23,
Would you try to remove userData.txt by running the following command?
rm "$env:APPDATA\Purple HATS\userData.txt"
Then retry node index again.

@younglim Figured it out.userData.txt is being created empty. After putting in {} and saving userData.txt it runs fine.

@younglim I spoke to soon. While I was able to get up to the point if it asking me the URL of the website to test, it would execute.

@younglim Ah! Missing folders in the download zip

@younglim Figured it out.userData.txt is being created empty. After putting in {} and saving userData.txt it runs fine.

You would minimally the following within userData.txt
<EMAIL_ADDRESS>

Thanks @younglim, installing Chromium did the trick. I'm up and running! Have a great day!


Naming convention for supermarket

We've adopted the mongodb cookbook that uses "mongodb" as namespace in chef supermarket. Our repo name conflicts with the old one. We need to set a new name for the supermarket.
I think we should define a naming convention for this purpose.
I would go with brigade-mongodb and follow the pattern brigade-{name} Using brigade as namespace for other cookbooks
Eg:

Github: nginx-cookbook  Supermarket: brigade-nginx
Github: varnish-cookbook Supermarket: brigade-varnish

Similar as grunt-contrib, This will prevent any conflict with other cookbooks
Any thoughts?

:+1:

honestly, I'd rather our mongodb replaced the existing mongodb cookbook. I'll bug @jamesonjlee to see if he can get us added to the existing one.
For situations where we can't get the un-namespaced spot, :+1: to brigade-foo

Ok @josephholsten  let's give it some time to see if we can get the original name :+1:

@josephholsten Do we have any news ?

@luishdez sorry, I heard back from @jamesonjlee and he doesn't have access. I'll bug @thekorn and @edelight. Until then, let's proceed with brigade-mongodb.

I think it would be better to use an underscore in the prefix: brigade_. For example brigade_mongodb.
I propose this because using dashes gives problems when making ChefSpec tests: https://github.com/sethvargo/chefspec#testing-lwrps

WARNING Cookbooks with dashes (hyphens) are difficult to test with ChefSpec because of how Chef classifies objects. We recommend naming cookbooks with underscores (_) instead of dashes (-).

@zuazo and that's why I'm glad to have a community of smart folks about.

Indeed. @zuazo great catch. I prefer to have hyphens instead of underscores in repository names, but for consistency's sake underscores are perfectly fine IMO. I also like the brigade_<package> convention. Simple, sexy.

Could someone start a recommendations doc? I've started a template in https://github.com/chef-brigade/cookbook-skeleton, but that's not really where we want guidance for this sort of thing.

Mmmm… I do prefer dashes as well, actually I kinda hate underscores :smile:
I haven't use chefSpec that much so I can't tell, but are we using chef spec in all the repos? If this causes a problem that can not be avoided with good coding style practices I'm fine then.
It's not the most important thing but I think that naming is not trivial
For example with underscores the name is treated as a one word for SEO purposes. In the title h1 and URLs that gives good hints for SEO… If someone searches for mongo cookbook those keywords won't match, This names H1, texts … are shown like that in github, supermarket among others.
Though there is no much competition so it's not a big problem… but I think the "{resource}-cookbook" pattern is the best approach for this.

ChefSpec is an essential tool IMO. You can name the cookbook with an underscore and still name the repo with a dash, use alternative characters in the README, etc. to get around SEO issues, right?

Dashes are a bad option for any ruby code:
% pry
[1] pry(main)> def foo-bar(thing)
SyntaxError: unexpected '-', expecting ';' or '\n'
def foo-bar(thing)
        ^
[1] pry(main)> def foo_bar(thing)
[1] pry(main)* end
=> :foo_bar

We've generally stamped out all those bugs that we know of, but its still considered best practice to use underscores instead of dashes in cookbooks to avoid any future issues.
The issue is not confined to chefspec.  Its just a best practice recommended by nearly everyone who has touched chef-client core.

also if you're really adopting the mongodb cookbook it should still be named mongodb so users can still find it and upgrade and don't have to update all their dependencies.
you only need to do this when you're forking an existing cookbook, and can't adopt the cookbook from the original author.

@lamont-granquist AFAIK we have no permissions over the original mongodb cookbook in supermarket. See https://github.com/sous-chefs/mongodb/pull/106 for more information.

Do we think this is covered by https://github.com/sous-chefs/meta/blob/master/forking.md ?

Closing as per slack discussion on #sous-chefs


Got a token after device code login, now what?

I don't know if I managed to overlook something here, but I don't see any part in the tutorial or the code it shows that actually handles the token after the device code login.
There's code to show it, but that's it.
And I'm also not sure if I'm misinterpreting the tutorial but after the "Add user authentication" it acts as if we can just restart the program and it will NOT ask for a new device code login the next time. Which is what I'm running into.
I'd really like to know what I am supposed to do with the token and how I am supposed to reuse it, IF I am even supposed to be able to do so. The tutorial isn't even clear about this part.

Showing the token is basically to prove that the authentication worked! In a real app, there's no need to display the token.
With our Graph SDK, you don't need to manage tokens at all. The DeviceCodeCredential is wired to the Graph client object, and it handles requesting tokens as you make API calls.
When you restart the program, it will re-prompt you. That's because by default, the Azure Identity library uses an in-memory cache for tokens. Depending on which language you're working with, they provide mechanisms for you to allow it to cache to some persistent storage, which would eliminate the repeated prompts.
As for the "now what?" - if you've gotten a token, you should be good to proceed with the next steps of the tutorial.

I've looked into the whole thing a bit more and by now have a slightly better understanding how it works.
But as someone who literally just started with TypeScript and Graph I felt a little lost when I followed the tutorial and did manage to get the token but had to re-do the whole process each time I changed anything in the code.
I also managed to miss that this tutorial is available in more versions than TypeScript, that one's totally on me...
By now I know that the tokens don't even have a long lifetime in the first place and that they can't be reused anyways.
But since the tutorial just moved on without pointing out that it's normal to have re-doing the device code process each update, nor explaining why we're not storing the token in some form to get around that, I felt like I was missing something fundamental which threw me off.


Update TeX.gitignore

Reasons for making this change:
Add support for easy-todo package (that produces .lod file during the compilation)
Links to documentation supporting these rule changes:
https://www.ctan.org/tex-archive/macros/latex/contrib/easy-todo

@thilaire thanks!


Bug 2154164: disable webhook for rook untill we support

disable webhook untill we say we support it
Signed-off-by: subhamkrai<EMAIL_ADDRESS>

duplicate pr


Automatic installation of plugin fails on Fedora due to incorrect Linux cli (apt-get vs dnf)

Issue Type: Bug
Install Visual Studio Code to Fedora.
Start editing python source
after a while, a popup "Install CTAGs support" is shown
Accept installation => Terminal opens with
"sudo apt-get install exuberant-ctags"
bash: sudo apt-get install exuberant-ctags: command not found...
Problem: Fedora doesn't use apt-get. Fedora uses DNF. The installation method(s) should be distro specific or at least identify which installation method is currently in use.
VS Code version: Code 1.33.1 (51b0b28134d51361cf996d2f0a1c698247aeabd8, 2019-04-11T08:20:22.771Z)
OS version: Linux x64 5.0.9-200.fc29.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 2584)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwaresurface_synchronization: enabled_onvideo_decode: unavailable_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
15.39GB (2.82GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

xml
Dot
2.4.0

beautify
Hoo
1.5.0

language-julia
jul
0.11.6

restructuredtext
lex
109.0.0

python
ms-
2019.5.17517

vscode-docker
Pet
0.6.2

bash-beautify
sha
0.1.1

shellcheck
tim
0.6.0

This comes from an extension. It's not part of VS Code out-of-the-box. Given you explanation, I guess the python extension generates the popup and implements the installation. Please move the issue over there. Thanks a ton!


Added state to FastAPI and up starlette to 0.12.9

and get access to application state for :heavy_dollar_sign: :moneybag:

Thanks @euri10 !
It seems the Git history of this branch got into a weird state and is showing some extra changes that are already there and create conflicts. I also had to update the range, as Starlette's State is not available in 0.11.1.
So I just merged #593 that includes these changes. But thanks for your contribution! :cake:


Naming Conventions

I think we need to decide what our naming conventions are going to be within this repo and get everything currently in the repo to conform to them. Things that will be affected:

Class names
Tag names
File names
File structure (as a side effect)

The convention that seems to be loosely in place already is to prefix everything with "lis" and to suffix with what type something is. The case of the name depends on the convention used for the thing being named, i.e. upper camel case for class names, kebab case for tag names, and kebab case for file names. Take the pagination element, for example:

Class name: LisPaginationElement
Tag name: <lis-pagination-element>
File names: lis-pagination-element.ts and lis-pagination-element.html

However, not all components in the project currently conform to this. For instance, the gene search element has LisGeneSearchElement and <lis-gene-search-element> as its class and tag names, respectively, but its file names are missing the -element suffix.
It's also worth considering what's standard/convention elsewhere:
Standard: None of the standard HTML tags (<table>, <form>, <p>, etc) include a -element suffix because a tag is always an element, so the -element suffix would be gratuitous. However, every JavaScript class for these elements in the HTML DOM API has the prefix HTML and the suffix Element (HTMLTableElement, HTMLFormElement, HTMLParagraphElement, etc). For this reason, it might be worth dropping the -element suffix from our tags but keeping it everywhere else. Additionally, I think we should keep the lis- prefix in a tag names and keep the Lis prefix in place of the HTML prefix in our element classes. Both will serve as a sort of namespace for our elements. I'm not sure whether or not it makes sense to keep the lis- prefix in file names. I'm leaning towards no since we'll be bundling the generated .js files. And even if someone uses the files as a typescript library, the import paths will always start with legumeinfo/web-components/, so having subpaths that start with lis- would be redundant.
Convention: Angular has implemented its own interpretation of Web Components since long before they were supported by the web standard. Angular is an opinionated framework and has conventions for pretty much everything. When it comes to naming, they basically use the convention I just described but with the word "component" in place of "element". Their filenames, however, use a . instead of a - for the type part. For example:

A component: something-awesome.component.ts
A component's template: something-awesome.component.html
A component's SCSS: something-awesome.component.scss
A directive: something-useful.directive.ts
A module: the-umbrella.module.ts
A service: get-data.service.ts

Popular Angular libraries follow this convention as well for their own types, such as NgRX/Store:

For actions: something-awesome.actions.ts
A reducer: something-awesome.reducer.ts
For effects: something-awesome.effects.ts
For states: app.states.ts

Since GCV uses Angular, I've used this convention for my own types of things as well:

A model: organism.model.ts
A utility: internet.util.ts
A mixin: extra-functionality.mixin.ts

The number of examples I've just given may seem excessive but I think it highlights another benefit of this convention - it suggests how things should be organized in directories. Anything that matches *.component.* should be in a components/ directory; anything that matches an NgRX/Store type should be in a store/ directory, and possibly a type-specific subdirectory if there's many files; and my own types get stored in their own type-specific directories. Not only does this help organize things, it also allows a barrel to be made for each directory, allowing import statements to be as vague or specific as the developer would like.
So, considering all of this, I'll give my preference by revisiting the pagination example:

Class name: LisPaginationElement
Tag name: <lis-pagination>
File names: pagination.element.ts and pagination-element.html

Please, someone (anyone! ), share your opinion. It would be nice if we settled on conventions that everyone will enjoy working with rather than me imposing my opinion on everyone.
P.S. Is it worth adding Steven, Sudhansu, and/or Rex to the conversation since they'll be using the tags and elements?

Adding this issue to the documentation Epic since it would be nice to have this in place before turning the key on the docs.

Commit 957e10c5f9db17eaf9b3d4575e892a132de1abe7 updated the necessary existing files and code to make the repo conform to the agreed upon (thanks @adf-ncgr!) naming conventions. Closing.


Entity populations in the civ overview are sometimes negative

Entity populations in the civ overview are sometimes negative.

The 4 chip components

current site
lost sites
entity population
historical figures

could also use some tooltips.

The negative numbers are really in the legends_plus.xml export.
I created an issue in the DFHack repo.


Update broken link

-Updated 404 link to an updated link
-Made a few implicit HTTP calls explicit

@eglaysher I think this PR has been left open for long enough.

@ejona86 can you merge this PR?


Use wide-word Word128 to implement IPv6 addresses.

There are probably a few opportunities to optimize this code further, or to simplify a few places with complicated masking, but I think this is a good start.
Performance is more-or-less the same as the original. In the few cases where using Word128 was substantially slower, I adapted the existing Word64-based code.
 Also, good job paying careful attention to avoid performance degradation. I’m on a phone right now, so I’ll review this a little more thoroughly later.
As a reminder to myself, this will require a major version bump since the data constructor for IPv6 has changed.

Actually, just go ahead and bump the version in the PR. And add a note to the changelog.

OK, done.

Released as ip-1.5.0.


Passing --size to lvcreate twice results in error

While testing a new mount, this error failed a deploy:
Fatal error, `/sbin/lvm lvcreate --extents 100%VG --extents 100%VG
--name cluster-object-storage-brick0 gluster-packages-b0-vg ` returned 3
with 'Option -l/--extents may not be repeated.

Passing size to lvcreate twice results in an error. Size is passed in common_arguments as well.
Fixes #ENGT-9252
@vitroth @lilirui @shr3kst3r

Unblocks https://github.com/apcera/continuum-chef/pull/641

LGTM

I'm confused...  I see the error you're fixing, but if that error is fatal how has this code worked at all for the last 9 months?

confusion cleared from discussion w/ Ben...
The only LVM volumes we create via this recipe right now are the gluster thin provisioned volumes.  The other LVM volume is ephemeral and thus created at run-time via the the ephemeral filesystem mechanisms (that I wrote, doh...)
LGTM


Travis: test_agent_proxy_remote_exceptions

https://travis-ci.org/opensistemas-hub/osbrain/jobs/233143995

Again: https://travis-ci.org/opensistemas-hub/osbrain/jobs/242905748

This might just be a consequence of #137. I mean, the second error might just be a consequence of the first one (i.e.: the first error leaves the tester process in a bad state and results in a new error on the next test execution).
Maybe we should just concentrate on #137 and then see if we still get errors on Travis. If you agree we could just close this issue for now.

Probably fixed by #170.


⚠ ClamAV has degraded performance

In d2af152, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 47 ms

Resolved: ClamAV performance has improved in 8d5cbde after 6 minutes.