S3 resource does not exist causing module not found

Context

I do not understand why I am getting the following error. It seems the lambda cannot be loaded from S3 and therefore is not able to find the module to execute the function.
[1565266143895] [DEBUG] 2019-08-08T12:09:03.895Z ef9eb08e-f4f9-43a5-9072-c11f15be1b80 Changing event name from docs.*.cloudsearchdomain.Search.complete-section to docs.*.cloudsearch-domain.Search.complete-section
[1565266143897] The 's3' resource does not exist.
The available resources are:
 -
: ResourceNotExistsError
Traceback (most recent call last):
  File "/var/task/handler.py", line 602, in lambda_handler
  return LambdaHandler.lambda_handler(event, context)
  File "/var/task/handler.py", line 245, in lambda_handler
  handler = cls()
  File "/var/task/handler.py", line 102, in __init__
  self.load_remote_project_archive(project_archive_path)
  File "/var/task/handler.py", line 170, in load_remote_project_archive
  s3 = boto_session.resource('s3')
  File "/var/task/boto3/session.py", line 347, in resource
  has_low_level_client)
boto3.exceptions.ResourceNotExistsError: The 's3' resource does not exist.
The available resources are:
 -
[1565266201109] [DEBUG] 2019-08-08T12:10:01.109Z ef9eb08e-f4f9-43a5-9072-c11f15be1b80 Zappa Event: {'Records': [{'eventVersion': '2.1', 'eventSource': 'aws:s3', 'awsRegion': 'eu-central-1', 'eventTime': '2019-08-08T12:09:00.157Z', 'eventName': 'ObjectCreated:Put', 'userIdentity': {'principalId': 'AWS:AIDAY5TJ3FA44X4WWYBA7'}, 'requestParameters': {'sourceIPAddress': '<IP_ADDRESS>'}, 'responseElements': {'x-amz-request-id': 'B35B73FF4F75D59E', 'x-amz-id-2': 'ELkqeB94Gb17TPF12ffhVtASmEhtR7NlQO4DevDruHvA5I5DrFlln/oYPSJkDx9RO/D7MMxERKE='}, 's3': {'s3SchemaVersion': '1.0', 'configurationId': 'function-dev:sample.lambda_handler', 'bucket': {'name': 'test-function', 'ownerIdentity': {'principalId': 'A3V2HVB9FLXO6B'}, 'arn': 'arn:aws:s3:::test-function'}, 'object': {'key': 'test_5.pts.gz', 'size': 7924, 'eTag': '3b39cf26af0e2aa49b917a2771bcc068', 'sequencer': '005D4C10DC203B4EB2'}}}]}
[1565266201111] No module named 'sample': ModuleNotFoundError
Traceback (most recent call last):
  File "/var/task/handler.py", line 602, in lambda_handler
  return LambdaHandler.lambda_handler(event, context)
  File "/var/task/handler.py", line 248, in lambda_handler
  return handler.handler(event, context)
  File "/var/task/handler.py", line 423, in handler
  app_function = self.import_module_and_get_function(whole_function)
  File "/var/task/handler.py", line 239, in import_module_and_get_function
  app_module = importlib.import_module(module)
  File "/var/lang/lib/python3.6/importlib/__init__.py", line 126, in import_module
  return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'sample'

I verified the tar.gz is in the correct bucket-zappa bucket on S3
The Zappa LambdaRole is correctly created and has access to S3

Expected Behavior

I expected the application to be found on S3 and the specific module.function to be ran upon the event.
Actual Behavior

It seems the package cannot be found on S3 and when the S3 event comes in Lambda is not able to find the correct module.function to run
Possible Fix

Steps to Reproduce

Its hard to specify how to reproduce because I have no idea what is causing this, from one day to the other my Lambda applications start raising this issue.
Your Environment

Zappa version used: 0.48.2
Operating System and Python version: Xenial 16.04 python3.6
The output of pip freeze:
boto3==1.9.204
Link to your project (optional):
Your zappa_settings.py:

{
    "dev": {
        "project_name": "test",
        "runtime": "python3.6",
        "exclude": ["env","tests","data","output","notebooks", "launch.sh"],
        "s3_bucket": "bucket-zappa",
        "slim_handler": true,
        "aws_region": "eu-central-1",
        "events": [{
            "function": "sample.lambda_handler",
            "event_source": {
                "arn":  "arn:aws:s3:::test-function",
                "events": ["s3:ObjectCreated:*"]
            }
        }],
        "keep_warm": false,
        "apigateway_enabled": false
    }
}

Hmm.. I believe my issue might be related to this https://github.com/Miserlou/Zappa/issues/1271#issuecomment-484359007

I believe I found the issue which is similar to this https://github.com/Miserlou/Zappa/issues/1579
The items listed in "exclude": ["env","tests","data","output","notebooks", "launch.sh"], are not only excluded for my own code, but apparently for everything that should end up in the final zip package.


multiple push notifications have no way to distinguish which one was clicked on

attn. @cbaggers
Reported on community. Start reading at the highlighted post to the end of the thread.
In particular: How to detect on which notification I have clicked ? When I click on one notification, all pending notification are delievered in order as they arrived (first the oldest then others) and they all have value for fromNotificationBar "true" and I dont have none information which notification I have clicked.
Suggestion for a solution:
it is perfectly OK that all of the pending notifications are all delivered to the app when it enters foreground - this is the behaviour we have always had.
it is NOT OK that all of the delivered notification have fromNotificationBar set to true, and supposedly only the one clicked on should have that.

The forum thread Push Notifications Body JSON Structure posted by Levi Olson was linked to this issue.

Hi there, just reposting my feedback from forum: https://forums.fusetools.com/t/android-push-notifications-payload-event-not-fired-if-there-are-more-than-1-notifications-in-the-list/7125
Fuse 1.8.1
Marshmallow
If I send the device a single push notification at a time and open it each time before sending the next one, the payload is 100%, tested via lock screen and via tray.
If I send the device several push notifications, the following happens:

after opening any notification (could open the latest received) via lock screen or via tray, the payload of the 1st notification received is returned.

after opening subsequent notifications via the lock screen or via tray, the payload event is not fired.


Fix github actions outputs

While the ::set-output workflow command (which will be deprecated on June 1 2023) works by just sending it to stdout, the new method requires actually opening and appending to the file that can be found in the $GITHUB_OUTPUT environment variable.
This PR fixes the broken github actions output.

Nevermind! This was done via https://github.com/hellofresh/action-changed-files/blob/c5338e4409bc8a710c4b6b339aef615589787bda/action.yml#L42


System cannot find the file specified when running "sam package" command

Traceback (most recent call last):
  File "C:\Python27\Scripts\sam-script.py", line 11, in <module>
    load_entry_point('aws-sam-cli==0.4.0', 'console_scripts', 'sam')()
  File "c:\python27\lib\site-packages\click\core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "c:\python27\lib\site-packages\click\core.py", line 697, in main
    rv = self.invoke(ctx)
  File "c:\python27\lib\site-packages\click\core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "c:\python27\lib\site-packages\click\core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "c:\python27\lib\site-packages\click\core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "c:\python27\lib\site-packages\click\decorators.py", line 64, in new_func
    return ctx.invoke(f, obj, *args[1:], **kwargs)
  File "c:\python27\lib\site-packages\click\core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "c:\python27\lib\site-packages\samcli\commands\package\__init__.py", line 22, in cli
    do_cli(args)  # pragma: no cover
  File "c:\python27\lib\site-packages\samcli\commands\package\__init__.py", line 26, in do_cli
    execute_command("package", args)
  File "c:\python27\lib\site-packages\samcli\lib\samlib\cloudformation_command.py", line 18, in execute_command
    subprocess.check_call([aws_cmd, 'cloudformation', command] + list(args))
  File "c:\python27\lib\subprocess.py", line 181, in check_call
    retcode = call(*popenargs, **kwargs)
  File "c:\python27\lib\subprocess.py", line 168, in call
    return Popen(*popenargs, **kwargs).wait()
  File "c:\python27\lib\subprocess.py", line 390, in __init__
    errread, errwrite)
  File "c:\python27\lib\subprocess.py", line 640, in _execute_child
    startupinfo)
WindowsError: [Error 2] The system cannot find the file specified

I get this error when running sam package with the specified flags in the README.md for the project that is created using sam init --runtime go1.x
I found that if I change the following line to set the aws_cmd to aws always instead of aws.cmd on windows I can run sam package without getting an error.
https://github.com/awslabs/aws-sam-cli/blob/48f8b1f0b6b22ccb25d2ee594a6a42d87c40584b/samcli/lib/samlib/cloudformation_command.py#L16
Used PowerShell on Windows 10.
Hope this helps.

I'm seeing the same thing regarding aws vs aws.cmd
May be a duplicate of #415

@mjiderhamn Yes this looks to be a duplicate of #415.
Closing this in favor of the older one.


Change of projection back to perspective breaks the aspect ratio

Objects appear more oblong after the change back to then perspective projection.
However, a soon as the viewer panel is resized, the aspect ratio is fixed.

Looks like we'd have to also emit resize event on camera change.


can we add a link / preview of the survivor chat channel to the client - to add social layer for new users

Could you provide more details, would you prefer if we include the survivor chat channel to the discord links on the UI? Or would you like another link somewhere in the UI that does this specifically?


Commit Company Pulse App - Creator Kit

We're so eager to add your sample to this repository as soon as possible, but we need all samples follow the same structure.
To make sure we process it as soon as possible, make sure to follow these instructions.

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
no

Solution Details

Q
A

Solution requires premium license
no

Solution uses experimental features (see licensing guidelines)
no

Which connectors are used?
RSS Feed

Solution uses on-premises connectors?
no

Solution uses custom connectors?
no

Layout
Tablet

What's in this Pull Request?
Created new sample, similar to Company Pulse App by MS, but using together Creator Kit controls and standard controls.

Hey @angelogulisano -
Looks great! Only one comment:

Could you please remove the unused screen from the Company Pulse app? I'm seeing a screen with the name screen1 but it only has a couple of containers... no actual functionality.

Thanks!

Sure, job done! Thanks for support

Thanks @angelogulisano! Merged!

Thanks!


Change docker service variable

Docker Registry installation is failing with error:
TASK [docker-install : Save Docker Registry status] ************************************************************************************************************************************************************************************
fatal: [cnx65-cp.stoeps.home]: FAILED! =>
  msg: |-
    The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'ansible_facts'

    The error appears to be in '/home/stoeps/work/iac/connections-automation/roles/third_party/docker-install/tasks/setup_docker_registry.yml': line 119, column 3, but may
    be elsewhere in the file depending on the exact syntax problem.

    The offending line appears to be:

    - name:                             Save Docker Registry status
      ^ here

I think this line is the reason:
https://github.com/HCL-TECH-SOFTWARE/connections-automation/blame/main/roles/third_party/docker-install/tasks/setup_docker_registry.yml#L117
When I print the variable docker_registry_status I get:
  docker_registry_status:
    changed: true
    container:
      AppArmorProfile: ''
      Args:
      - /etc/docker/registry/config.yml
      ...
      RestartCount: 0
      State:
        ...
        Running: true
        StartedAt: '2021-09-20T12:16:51.785239363Z'
        Status: running

So the variable to check is docker_registry_status.container.State.Running and not the used docker_registry_status.ansible_facts.docker_container.State.Running
Regards
Christoph

Could you please confirm the OS where you are installing docker. This works fine on CentOS 7.
When I print the variable docker_registry_status I get:
docker_registry_status :{    "changed":true,    "ansible_facts":{       "docker_container":{          "Id":"abc",          "Created":"2021-09-20T11:44:10.397381946Z",          "Path":"/entrypoint.sh",          "Args":.......          "State":{             "Status":"running",             "Running":true,             "Paused":false,             "Restarting":false,             ...             "Error":"",             "StartedAt":"2021-09-20T11:44:11.027854512Z",             "FinishedAt":"0001-01-01T00:00:00Z"          }          ............

I install on CentOS 7.9 and I checked the registered variable. So I'm really interested why the behave different.
Oh one thing, I use the 6.5 playbook, because I want to test the newest CFix. So maybe it is dependent to the docker/containerd version?

Thanks @stoeps13
I am using below versions-
ansible 2.9.15
Docker version 19.03.15, build 99e3ed8919
containerd containerd.io 1.4.6 d71fcd7d8303cbf684402823e425e9dd2e99285d

As variable register is an Ansible funcition, I believe it is Ansible version related. Maybe we need to add a conditional statement for the different naming.

solved, thanks @nitinjagjivan


doesn't work with mac os sierra

doesn't show list of resolutions.

Same

As I type it shows:

Then once I get to the normal position it then shows as if it's a search:

Debugging info:

I fix this problem, and create pull request about this.
If you want temporary fixed version, you can download here.
https://github.com/luinhon/alfred-set-resolution-workflow/raw/master/alfred-set-resolution.alfredworkflow

I had this too, and with a few other workflows as well. Turns out it's related to macOS's default ruby, which has changed.
For me it worked to open the workflow, double click on the first step to open the settings, click on the button (1) to open the workflow's folder in Finder and change the ruby version in the bundle/ruby subfolder (2) from 2.0.0 to 2.3.0.

[08:59:58.156] ERROR: Set Resolution[Script Filter] Code 1: /Users/abd/Dropbox/Alfred.alfredpreferences/workflows/user.workflow.CAC029B5-ADD5-45E9-89AD-01299B53E93C/bundle/ruby/2.6.0/gems/builder-3.2.2/lib/builder/xchar.rb:111: warning: constant ::Fixnum is deprecated /Users/abd/Dropbox/Alfred.alfredpreferences/workflows/user.workflow.CAC029B5-ADD5-45E9-89AD-01299B53E93C/workflow_config.rb:66:in rebuild_resolutions': undefined method each_with_index' for nil:NilClass (NoMethodError) 	from /Users/abd/Dropbox/Alfred.alfredpreferences/workflows/user.workflow.CAC029B5-ADD5-45E9-89AD-01299B53E93C/workflow_config.rb:57:in load_display_resolutions'
from /Users/abd/Dropbox/Alfred.alfredpreferences/workflows/user.workflow.CAC029B5-ADD5-45E9-89AD-01299B53E93C/workflow_config.rb:13:in initialize' 	from ./feedback.rb:11:in new'
from ./feedback.rb:11:in <main>'
Any ideas?


Editor filter want to show only "Open" status

from editor

Where do you need to enter the conditions?

maybe getItems() ?

Or create a custom lookup... See the docs for Handling Lookup Scripts
Your criteria would be something like
.Where(new Criteria(fld.InvoiceStatus) = "Open");


Warning to You. Ignore the first part

[02:46:37 ERROR]: [SimpleMsgPlugin] [STDERR] net.kyori.adventure.text.minimessage.internal.parser.ParsingExceptionImpl: Legacy formatting codes have been detected in a MiniMessage string - this is unsupported behaviour. Please refer to the Adventure documentation (https://docs.advntr.dev) for more information.
&bme &#fb0f00->§r &bGo_ing§b: &7 hi
^^
[02:46:37 WARN]: Nag author(s): '[MouseJava]' of 'SimpleMsgPlugin v1.5.0' about their usage of System.out/err.print. Please use your plugin's logger instead (JavaPlugin#getLogger).

Hi, I guess this error is related to the fact that I switched to using MiniMessages.
In the configs of old versions of the plugin, old message formats are installed in some lines. You need to change some lines in the config for everything to work correctly. Here is a link to the original plugin config for version 1.5.0 (https://github.com/MusiJVR/SimpleMsgPlugin/blob/main/src/main/resources/config.yml) and here is also a link to this MiniMessage format (https://docs.advntr.dev/minimessage/format.html). It indicates what tags there are for formatting and how to do it correctly.


windows

Can I use it on Windows

absolutely yes. it's possible

Have you successed?

yes of course, tomorrow I will fork this repository and will change for using in windows, you can clone my version and use it.
please remember me, tomorrow if you can.
tnx
@TheLostIn

@masoudpz hai, have you successed? where to clone your version?

https://github.com/masoudpz/RFBNet
check this link, i've changed test.py file and utils/nms_wrapper for running on windows.you can check these two files, load trained model and run the code on windows
I can help you if needed.
@zhuixunforever

I'll try it, thank you very much!

how did you compile ./utils/build.py on windows??  just run ./make.sh???
@masoudpz

./make.sh is needed for coco dataset, you can use rfbnet by other dataset without ./make.sh

that's all right! thanks

./make.sh is needed for coco dataset, you can use rfbnet by other dataset without ./make.sh

Nice man,could u share me a py about RFB demo.py on windows or linux?I really need it ,thanks for your friendly help.Thank you

./make.sh is needed for coco dataset, you can use rfbnet by other dataset without ./make.sh

Nice man,could u share me a py about RFB demo.py on windows or linux?I really need it ,thanks for your friendly help.Thank you

you can use below link, I have changed it to use in windows
https://github.com/masoudpz/RFBNet


Issue 741 | Docs: remove deprecated algorithms module

Summary
Remove algorithms from:

[x] VQE tutorial
[x] QAOA tutorial

Details and comments
Since application module is deprecated we needed to rewrite part of tutorials using scipy optimization package.
Example: https://qiskit.org/ecosystem/ibm-runtime/tutorials/vqe_with_estimator.html
Closes #741

Good catch!


Document features of dotfizzles

Like tabbing thorugh directories, case insensitivity... Type a char and tab thorugh options. etc.

https://github.com/mhulse/dotfizzles/wiki/Highlights


C++ assembly lsp not working in neovim

Hi,
thanks for this tool.
I generate C++ assembly using CMake using GCC . The file has an extension of  "*.a" and I added that extension to filetypes in the config

However when I open the file , the filetype is not recognized as asm so I set the filetype both in the config and also manually using :set filetype=asm the file type is recognized but lsp still does not work

here is the lsp info

Also when I generate Rust asm then it generates assembly with a *.s" extension . I open the file both filetype is recognized and lsp / hoover works
I am using Arch and using GCC 13.2.1 and neovim 0.9.0

Could you upload a sample .A assembly so I can have a look ?

@bergercookie
I had to change the extension .a.txt since github did not except .a extension. Attached is the assembly file and also I used this in Cmake file to generate the assembly file
add_compile_options( -O1 -g -S )
add_link_options(-g -S )

libalgoFunLib.a.txt

It works fine on my end. I'm using coc to interact with asm-lsp so I'm not exactly repro-ing your LSP client/server setup. When I open either libalgoFunlib.a.txt or I rename and use  the .a version and then I :setf asm hovering works :/

The first screenshot says LSP inactive on the bottom left. Is this correct? Why?
In the second screenshot that you posted above it says 3 active clients not attached to this buffer. Could this be related ?
Are you sure that asm-lsp has loaded correctly and listens to events for this opened buffer/file of yours ?

@bergercookie
somehow on that file lsp does not work correctly but if manually create a file named test.a then asm-lsp loads. I am using nvim-lspconfig

I dont have to set file type it directly detects as asm file and asm_lsp  is active. Same thing happens when I build the assembly with Rust but somehow for C++ I could not make it work.
I even tried downloading file I have submitted here . changed the extension and set filetype=asm , the highlighting works but  the lsp gives an error Here is the error from the log ; which I posted in my previous comments

[START][2023-09-27 13:07:43] LSP logging initiated
[ERROR][2023-09-27 13:07:43] .../vim/lsp/rpc.lua:734	"rpc"	"asm-lsp"	"stderr"	"INFO [asm_lsp] Populating instruction set -> x86...\n"
[ERROR][2023-09-27 13:07:44] .../vim/lsp/rpc.lua:734	"rpc"	"asm-lsp"	"stderr"	"INFO [asm_lsp] Populating instruction set -> x86_64...\n"
[ERROR][2023-09-27 13:07:44] .../vim/lsp/rpc.lua:734	"rpc"	"asm-lsp"	"stderr"	"INFO [asm_lsp] Starting lsp server...\n"
[ERROR][2023-09-27 13:07:44] .../vim/lsp/rpc.lua:734	"rpc"	"asm-lsp"	"stderr"	"INFO [asm_lsp] Starting LSP loop...\n"

I have both lvim (a preconfigure neovim with a different runtime)
I also my neovim config and it works there so it must be somethng related to lvim config


Add an ITraceEventOperand interpreter

The first step to make the tool working E2E is to add an interpreter that will call into helper methods using a interpreter stack.

Obsolete.


Change cms label from reset-password to the pw/change

Environment Details
Spartacus: Release 1.0
Browser: Safari
Device: mac
Happens on: https://dev-com-19.accdemo.b2c.ydev.hybris.com:4200/electronics-spa/en/USD/
Steps to Reproduce
Click on Forgot password
Enter your email add and click 'Send'
Go to the email and click on the link to reset password
Actual Result
The link brings you to the Login Page instead of the Reset password page.
However the link in the email is ok, it's
http://dev-com-19.accdemo.b2c.ydev.hybris.com:4200/login/pw/change?token=Tgv4JbOoP4Wzqd6ov%2BtIRL%2BOaTVnCh8P%2Bwi5LIWpmVBBr2ngB19WZaAh7kFtO4PQlIcSAenKpPZ9lLhuDBV%2FDKMMAtaS6us6OxrS5dN5R66YIIbTfsYj34LZfWdUAkzA9BVreLIWiHFwGgUaJOyB99zjxQdMGu5qL54%3D
But when I click on it, it redirects me to this link which then displays the login page: (Notice the extra "electronics-spa/en/USD" part in the link.
http://dev-com-19.accdemo.b2c.ydev.hybris.com:4200/electronics-spa/en/USD/login/pw/change?token=Tgv4JbOoP4Wzqd6ov%2BtIRL%2BOaTVnCh8P%2Bwi5LIWpmVBBr2ngB19WZaAh7kFtO4PQlIcSAenKpPZ9lLhuDBV%2FDKMMAtaS6us6OxrS5dN5R66YIIbTfsYj34LZfWdUAkzA9BVreLIWiHFwGgUaJOyB99zjxQdMGu5qL54%3D
Note
The reason this happens is because the url is misconfigured in the backend. Kris says: "The url sent in the email is wrong - /login/pw/change. The correct one is /login/reset-password."
More info in ticket #3459
So we will need to update this to make sure we have the right configuration going forward.
But Wei says there is no way to configure what the url pattern is for reset password, her solution is to change the cms label from reset-password to the pw/change.
So this ticket is to do just that.

I tested this theory on dev19, wei is correct
she said she'd update the sample data

PR is here:
https://github.tools.sap/cx-commerce/c3po-legacy-spartacussampledataaddon/pull/5

Can you also update the reset-password.e2e-spec.ts test to reflect the new label please?
Line 5: cy.visit('/login/pw/change');
Line 15: cy.url().should('match', //login/pw/change$/);
Line 27: cy.url().should('match', //login/pw/change$/);

Tested on my local by running the e2e tests.


feat: aura theme

Why
Expands the collection of themes available.
How
Adds a new aura dark theme.
Test Plan

Checklist

Thanks @shubh73, looks good 👍


no background contrasts

loading the plugin I am a bit disappointed in how there are no difference between visual layers. Opening menus on top of a text file and you can't visually tell what is what. I wish colors could be tweaked a little to make distinctions clearer

Hi @Draic, highlight groups can indeed be nuanced, as different users often have varying preferences when it comes to highlighting styles. To address this, many colorscheme plugins offer configuration options that allow users to customize highlights to their liking. For instance, while I personally find the contrast between certain layers to be a bit unusual, I understand that others may appreciate it.
In the default configuration of the colorscheme, transparency is used, which is why Float elements are also transparent. However, if you'd like to give Floats a distinct background color, you can easily adjust this through the plugin's configuration. For example, you could set a different background color for NormalFloat like this:
-- Add specific hightlight groups
on_highlights = function(highlights, colors)
    highlights.NormalFloat.bg = colors.bg1
end,

This approach allows you to tailor the appearance according to your preferences.
Happy Coding ❤


[suggestion] Optional More Decimal Precision for values

It would be nice if we could see decimal precision down to the hundredths place for what the horse's speed and jump are.

Sorry for replying so late to this. Is this still relevant for you? Can you describe me, why you want a higher precision?

Do I can tell which horse is better in breeding. Even if very minor


Support for enhancd

Would be great to have a plugin/completion support for Enhancd.

This looks promising. Feel free to create a PR with support for this...
Pull Requests are always welcome.


Don't wrap cStringIO in try/except

Has to go in after #6934

Well that obviously didn't work. Let me investigate.

try:
    from cStringIO import cStringIO as StringIO
except ImportError:
    from StringIO import StringIO

always fell back in the first place since
from cStringIO import StringIO

is the correct way.

test this please

Ok, good to merge after the JSON wrapper stuff. Remaining red X is for the pylint stuff.
Thanks @amaltaro

test this please

Looks good as well. I'll merge this one and cut a new tag & create private RPMs


fetch env files from s3

@masarakki Thank you for your PR! I will check this code, please give a little time.
Perhaps, I'll modify error handling.


⚠ AGEG website (prod) has degraded performance

In ea1d55b, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2381 ms

Resolved: AGEG website (prod) performance has improved in 41d6bde.


chore: make /assets a named volume for dev container

This PR adds a named volume "app-assets" mapped to /assets inside the dev container.
Configs and scripts were modified to point to this new location.
fixes #4012

Can we remove the data directory entirely?

app-create-dirs.sh fixed. Search didn't return any other files still using data/developers.


Extract cookbook to a single file for each section

@anthony-khong
I extracted each cookbook-code to self-contain code for each section.
Please review and let me know if you like to change anything.
Best,
Burin

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@1d891c5). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             develop       #241   +/-   ##
============================================
  Coverage           ?   100.000%
============================================
  Files              ?         34
  Lines              ?       2630
  Branches           ?          0
============================================
  Hits               ?       2630
  Misses             ?          0
  Partials           ?          0


tooltip not works when button is disabled

It's a feature or bug?
If I'd like to display the tooltip on a disabled button, how to do?
Thank you.
<button class="btn tooltip" data-tooltip="Lorem ipsum dolor sit amet" disabled>top tooltip</button>

@shiny I will fix it. Thank you.

fixed in https://github.com/picturepan2/spectre/commit/15746d90fba75b8845a5f607130ec16ff86f4eb0


plex: <IP_ADDRESS>68-28fc46b27 -> <IP_ADDRESS>18-6f393eda1

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

This is again the latest version, rebased on the updated one. Please merge before someone else open again a PR and merges it. @r-ryantm


[ENHANCEMENT] Add a short account menu with a way to logout

This PR is adding an account menu on the navbar. For the moment you just have the theme and the way to logout.
https://github.com/perses/perses/assets/4548045/6685d3b6-90c4-4553-b58b-d961d07f6813

In addition, here a new display proposition for the theme:

Like that we have a menu a bit align.


Game mode

is it posible to make a service to turn on and off "game mode"?

if possible I'd still love to have this. Can't seem to make this work reliable with remote commands


use full delayed mission for example file

Working on suggestions in #4

TODO: Add optics etc. from #4

Sample dataset is now a full mission with optics and adcp pressure

@MOchiara check if the new example file has all the variables you need

@callumrollo looks good! Thanks


webapp: initialize webapp

closes #40
This creates a basic emberjs app.

We have decided to not use ember for now. So closing this PR.


system/sched_note/note_main.c:  Add interrupt/syscall call decode.

Summary
Add support for decoding interrupt and system call instrumentation
Impact
None other than to system/sched_note/note_main.c.
Testing
Tested with sim:nsh

Merge notes:

There are three nxstyle complaints that need to be ignored:
system/sched_note/note_main.c:609:64: error: Mixed case identifier found
system/sched_note/note_main.c:653:0: error: No indentation line
system/sched_note/note_main.c:705:0: error: No indentation line

The first is the use of PRIdPTR from inttypes.h.  The second two make no sense are are, most likely, errors in nxstyle.

The PR checks will fail until incubator-nuttx:#1259 is merged.  It depends on a definition added by that PR.

@ouss4 please add commit messages !

@ouss4 please add commit messages !

@jerpelea I'm not quite sure what you mean by that.

This is a best parctice that helps us better document the commits in any project.

This is another example, from another well known project, how commits look with commit mesages
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/log/?h=v5.7.2


Add a script and lint check for maximum path lengths.

Windows has difficulty building the project when directory and file paths get too long. This adds a lint check for maximum directory path length as a nudge to keep us from adding any paths longer than our current longest.

Sample failure
https://github.com/iree-org/iree/actions/runs/3040322825/jobs/4896242190

Sample success
https://github.com/iree-org/iree/actions/runs/3040651662/jobs/4896945763

skip-ci

Thanks for the comments. I adapted your suggestions a bit. PTAL


Initial change to see if VPN amendment works

This is the initial test to see if adding new addresses will clear an issue outstanding on NOMS. They have lost access for some time due to losing both VPNs.
The change includes a  new VPN (which may not work, unknown - noms_dev_vpn_static_routes) and adding another address to the current noms_live_vpn_static_routes based on a value provided for the azure system.
The former is "<IP_ADDRESS>" (dev/test) and the latter "<IP_ADDRESS>" (prod). Have been told the dev/test azure location is not an issue if that breaks.
A test pull request will be undertaken to make sure there are no obvious issues.
A pull gave me issues which I sorted out but only created the production line I added. I need to put some additional code in a json so I will do this once I decide what we need. Various other changes in here to correct initial plan issues.

See this PR - this one won't be needed.


Feature Request: Keybind for 'toggle terrain layer always displayed'

That's the whole ask. I can just use a macro to hit the toggle, but a first class hotkey would be nice.

Should be added to the latest version.


Proposal: Add tuning swappiness of container

The memory cgroup has swappiness feture (takes 0-100) which can be used to tune swappiness behavior of container. In real world, it helps in improving the performance of highly active container
by reducing the major fault and thus reducing swapping behavior (when we tune it to lower value).
corresponding libcontainer changes taken in https://github.com/docker/libcontainer/pull/622

@ktraghavendra Please don't close this issue to open a new one on the same topic. https://github.com/docker/docker/issues/13881 didn't need to be closed and this issue didn't need to be opened.

@unclejack sorry, I had the same doubt and confusion. Thanks for clarification.
waiting for vendor libcontainer update.

will wait for #13893 to create a updated vendor patch for libcontainer (if necessary)

PR was merged.

Thanks @cpuguy83 Opened https://github.com/docker/docker/pull/14004

Closing since #14004 was merged.


ah - made the UI to create a list

Made the UI for when a user wants to create a new list.

[x] Create a user list page/component
[x] Able to set the Name
[x] Able to set Rating System

Values are not saved for now, but once it is connected to the backend, values should be saved

Closes #124

The modal might not be needed, but I was thinking that maybe we can add info about the rating system selected on the modal

I think the modal could be a good idea long term to select not only their rating systems but to also search for other rating systems as well and to display for information about a rating system before selecting.
it could also be a good idea to reduce the scope of your issue and keep it simple by only having a dropdown menu of possible rating system ids moving the modal UI in another issue. We will need only the id of the rating system when submitting the form.

Replaced the button to access the modal with a dropdown menu and left the modal for later. I'm not sure how to make the dropdown menu with a variable number of options, so I just left a single temporary option.


sonarqube complete installation steps

could you please provide the latest installation steps you followed to install sonarqube with mysql.
as temporarly i hosted on docker due to issue with manual installation steps provided.
sudo apt-get update
sudo apt-get install openjdk-8-jdk
sudo apt-get install sonar=6.7.4
sudo service sonar start
installing sonarqube didnt work using apt-get on ubuntu.

hi venkat, call to this number<PHONE_NUMBER>.

https://directdevops.blog/2019/01/05/sonarqube/


checkGrammarTypeInfer validation ignores declared types in other .langium files

Definition of a rule X and a type XType in different files, gives you a validation error The type 'XType' is not explicitly declared and must be inferred.
interface XType {
    a: string
}

X returns XType:
    a=ID;

@pluralia
I can't  reproduce this issue using the example you provided.
Could you add a real world example that produces the error mentioned above?

@dhuebner, I also can't reproduce this issue. I close it.

I can reproduce it by removing the import statement.
The error message is misleading. Instead, there should be an error on the return type of X because it cannot be resolved.
We should really consider filtering the global scope to include only imported grammars as proposed in https://github.com/langium/langium/issues/746#issuecomment-1303673167.

I disabled the error in #763 with an additional check in the validator:
https://github.com/langium/langium/pull/763/files#diff-3321518d3ed64fdf6ef10fd76c0c81fd6b44242981303125b983f429b36caa1a
But this is only a workaround. I think the analysis made by the validator should filter out unresolved references earlier.

@spoenemann
The  unresolved return type check you added works as expected and there is no other much better place where to put it.
So I would propose to close this issue. We can create a new one if the problem described here occurs also after the type system refactoring.


CI is crashing with the new Flutter version

We're having issues with our Pipeline in Azure DevOps (I don't know if it happens with other CI platforms)
It's crashing when it tries to download the .zip of latest version (1.17.0). And it seems to be the URL pattern for each release that has changed with the new release:
v1.17.0: ..../flutter_macos_**v**1.17.0-stable.zip
v1.12.13+hotfix.9: ..../flutter_macos_v1.12.13+hotfix.9-stable.zip
v1.12.13+hotfix.8: ..../flutter_macos_v1.12.13+hotfix.8-stable.zip
....
Has someone the same issue?
Thanks

Having same issue after upgrading 1.17.
Here's the details, need hotfix:

looks like it's an issue with the DevOps extension and they are working on fix:
https://github.com/aloisdeniel/vsts-flutter-tasks/issues/46#issuecomment-624761829

Hi @DavidFD005
Looks like this isn't issue with Flutter but  DevOps extension
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

Thanks!


Removes double-maintenance in UltraDNS when adding new record types

UltraDNS had its own copy of Util.toMap. This removes that to prevent
drift when folks add support for new record types.

denominator-pull-requests #411 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » denominator » denominator-pull-requests #36 FAILURE
Looks like there's a problem with this pull request

weird.. passes locally..

denominator-pull-requests #412 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » denominator » denominator-pull-requests #37 FAILURE
Looks like there's a problem with this pull request

denominator-pull-requests #415 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » denominator » denominator-pull-requests #40 FAILURE
Looks like there's a problem with this pull request

ahah.. fails w/JDK 7, but not 8

found it. Collections.singletonList is sortable in JRE 8, but not 7

denominator-pull-requests #416 SUCCESS
This pull request looks good

NetflixOSS » denominator » denominator-pull-requests #41 SUCCESS
This pull request looks good


Check before delete

Description
This pr is to check all downstream service before deleting ml model. Two downstream tasks are checking here:

Agent
Pipelines

For agent, we enforce this tool factory to implement a method to return the key field name of each tool. Then we create a should DSL query like
{
  "query": {
  "should": [
  "terms": {
            "tool.parameters.model_id": ["delete_model_id"]
        },
    "terms": {
            "tool.parameters.embedding_model_id": ["delete_model_id"]
        }
  },
  "terms": {
            "tool.parameters.inference_model_id": ["delete_model_id"]
        }
  }]
}

For pipelines, we fetch all ingestion pipelines and search pipelines and check for each pipeline whether they contain the candidate model id.
Related Issues
#3191
#3087
#3088
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] API changes companion pull request created.
[ ] Commits are signed per the DCO using --signoff.
[ ] Public documentation issue/PR created.

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

@xinyual You can raise another PR in the skills repo. It's better we can review both PRs togeter.

@xinyual You can raise another PR in the skills repo. It's better we can review both PRs togeter.

Sure.

Please update DeleteConnectorTransportAction with connector_id in ConnectorTool.

It's not related to this PR. I will check it and open another PR if it's need.


getmail: fix port option type mismatch

Fixed type mismatch in commit 410f5732267d48d78ff0a1e6f38f512c4aea4808.
Added test case to ensure it works well.
Sorry for inconvenience caused :cry:

Thanks! Rebased to master in a177d0282fbb6af0668b63fba3dfb74cb25f3178.


Support for Avro name aliases

Please consider adding support for aliases in the generated schema.
This could be added in the same way as default value with a custom annotation, e.g.
[Aliases("OldCourse")]
public class Course
    {
        [DefaultValue("200")]
        [Aliases("Level")]
        public string NewLevel { get; set; }
    }

Ok but no ETA yet.
Thanks

Guess this issue has been fixed!


Trying out in VirtualBox

I'm trying to install the OS as a VM in VirtualBox but it reboots after entering the disk name. I'm using the EFI boot option. After rebooting, if I boot into Arch Linux from ISO, I repeat the same thing all over again. The other option is an EFI Shell in which booting from bootx64.efi takes me back to the Arch Linux selection menu.

Fixed in latest commit


Adds csv names to the spec of installplan after unpacking bundles.

Description of the change:
Adds the CSV name to InstallPlanSpec.ClusterServiceVersionNames after unpacking a bundle.
Motivation for the change:
Addressing https://github.com/operator-framework/operator-lifecycle-manager/issues/1347, especially the case I mentioned in the comment, that packages from a CatalogSource defined by an index image cannot be upgraded and reports error like pre-existing CRD owners found for owned CRD(s) of dependent CSV. The ClusterServiceVersionNames field in the Spec of InstallPlan is empty, although steps of kind CustomResourceDefinition exists in the Status.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

/ok-to-test

Thanks for the fix, I will close this PR. @ecordell


Fix broken link on SDK downloads

There was a 404 to the SDK downloads. I fixed it with the correct link.

Hi @seanfreiburg, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Sean - looks like the broken link fix is simple enough. But your pull request has merge conflicts. Can you resolve them? Thanks.

Sent mail

Sean, could you please resolve the merge conflicts and remove the /en-us/ part of the path you added to a link? That section breaks the localized experience by forcing users from a translated article back to the EN version of the article.

Closing since merge conflict has not been resolved.


[SPIKE] Align on responsibility for dependabots, deployments, etc

Problem statement
Victor and Chris to sync with Patrick on who is responsible for dependabots, deployments, etc
What you need to know
This item came from retro
Acceptance criteria

[ ] Sync with Patrick
[ ] Share with team in public channel(s)

Gotcha @LaFreda1 closing this one


routing from DB #138 (first part)

This is the first part to solve this issue as I understand it. I will add other missing parts when we are OK for that part. To test it you have to enter the data manually to the database like shown in the image below containing the path and the name of the component inside templates folder. (read this link to understand why we need a separate folder for dynamic templates ). then enter the dynamic urls after the hash sign /#/dynamic-url . I switched temporary to vue-router hash mode just for testing and I will switch it back when everything is Ok.

Hello Abdelaziz,
Can I call template 1 with routex and routey, where each can display the sa,e tempalte but with different dataset?
Thanks

Ok so if I understand your question well. Yes we will be able to call the same template with different routes. for example (supposing we have the routes above in the image) we can do it like this:

/dynamic1?content_id=1 to display the content 1 in the Template1.vue
/dynamic2?content_id=1 to display the content 1 in the Template2.vue
/dynamic2?content_id=15 to display the content 15 in the Template2.vue

The process of fetching the needed data and displaying it will be handled in the template. for example when the route "/dynamic1/content_id=5" is reached. The content 5 is fetched and displayed inside Template1.

Hello abdelaziz. I mean can we do this with two different routes not the same route with two different parameters after the question mark

Hi Mahmoud, yes we can do this by adding an other route to "templates" that point to the same template. for example:
{
  path: '/route1',
  component: 'Template1'
},
{
  path: '/route2',
  component: 'Template1'
}

now both /route1 and route2 point to Template1. Concerning data to render in the template we have to pass it's identifer for example /route1/content1 or /route1?content_id=1 we will get this identifier in the template then get the corresponding data and display it.

Do the data identifier needs to be part of the url? Cant the router pass it from the database to the tenplate? What i want to store in db ia a route using a template to display specific content. So the content identifier that we want to display is in the db.

Hello, I am merging then testing, since we need this in the code for the rest of upcoming pull requests.


FlowDocument in ItemTemplate

Hi,
I have a special case where I need to display flowdocuments in an itemscontrol and to support ordering of these flowdocuments by drag&drop. It looks that this special case is not currenlty supported.
Is there a way to make this work with the gong drag&drop?
Here is an example of ItemTemplate:
<DataTemplate>
    <FlowDocumentScrollViewer IsSelectionEnabled="False">
        <FlowDocument>
            <Paragraph>Test</Paragraph>
        </FlowDocument>
    </FlowDocumentScrollViewer>
</DataTemplate>

@hawelo try setting IsHitTestVisible="False" (i tested it shortly and it seems to work, i added it to the default example)
<DataTemplate>
  <FlowDocumentScrollViewer IsSelectionEnabled="False" IsHitTestVisible="False">
    <FlowDocument>
      <Paragraph>Test</Paragraph>
    </FlowDocument>
  </FlowDocumentScrollViewer>
</DataTemplate>


Test: Validate DNS before trying to connect on curl

This commit intruduced a new Kubectl method that make sure that the DNS
exits before pods try to connect to other pod using DNS.


improve query to use index

Pull Request Test Coverage Report for Build 596

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

Totals

Change from base Build 593:
0.0%

Covered Lines:
685

Relevant Lines:
1790

💛  - Coveralls


When using "go test -v", I get  "unknown shorthand flag: 't' in -test.v=true"

I have a project that imports and uses pflag.
Whenever I run tests with the "-v" flag , I get "unknown shorthand flag: 't' in -test.v=true" and also a print of the help text for the flags defined with pflag.
Without "-v" tests succeed.
Searching I have found the following PR in Cobra to seem to be related : https://github.com/spf13/cobra/pull/155
Since Cobra also imports pflag, is there either a way to fix pflag or alternatively, some work around I can add to my own program?

I have fixed this and created a pull request: https://github.com/spf13/pflag/pull/66 .
Note that this fix was done without really understanding the mechanism. I just check for the "test.v=true" case, that causes the problem and ignore it.


Fabric Release Automation

Set up automated releases using semantic release and github actions as per  https://gist.github.com/benja/211fbddf3883ffed95a55edcfd1e7e9f and https://github.com/fabric-ds/issues/blob/master/rfcs/0004-releases.md.
Also moved over to NPM to slowly move over to using the same package manager for all repos.

Also, did you manage to get commit messages checked/validated/linted as part of build or commit?

I've not done this yet, no. Will look into this asap.


Pluralize table name

Laravel Version: 5.3.8
PHP Version: 5.6.11
Database Driver & Version: mysql  Ver 14.14

Description:
Its looks like a problem with plural word metadata. Eloquent tries to pluralize this word as: metadatas (my table is named a metadata). Models name: Metadata
I search something here: http://english.stackexchange.com/questions/180801/is-metadata-considered-plural
And its seems to be bug in the pluralize engine inside Laravel.
Example:
$plural = str_plural('metadata');
var_dump($plural); // metadatas

Steps To Reproduce:

create table metadata
create model Metadata
try to call something Table 'metadatas' doesn't exist (SQL: select * from metadatas)

This was fixed in 5.3.26, update your laravel version and try again, check:
https://github.com/laravel/framework/blob/5.3/src/Illuminate/Support/Pluralizer.php#L32

Its helps with metadata name, but in the engine_metadata problem still occur.

Its looking for engine_metadatas ? This is a relationship table between engine and metadata or is it just a common table? If it's a relationship table we could do something about it, if it's a common table then you'll have to specify the table name, we can't cover these cases....

Yeep, its a common table name.

You can always set the table name in your model, so that Laravel doesn't try to work it out automatically.
    /**
     * The table associated with the model.
     *
     * @var string
     */
    protected $table = 'metadata';

There's probably a way to teach Laravel certain pluralisations that it doesn't know, but I'm not sure of how.

Of course, now I have set this property. But, all the models <=> tables relations should be generated automatically.

So, I close this issue.
Conclusion:
I'm using protected $table property to set a table name.


Add last chance check to scheduled pass

Closes #116

@lee-dohm Do you plan to merge this anytime soon?


docs: update metrics.md

Include instructions on https://github.com/libp2p/js-libp2p/blob/master/doc/METRICS.md and update to include a JSON of the base dashboard for tcp metrics https://github.com/libp2p/js-libp2p/issues/2077
discussed in https://pl-strflt.notion.site/js-libp2p-metrics-monitoring-Cayman-e9191e5d022d476db979f91da1e66f3d?pvs=4

closing in favor of https://github.com/libp2p/js-libp2p/issues/2077


Save location by GPS

Tap Edit Location, tap "Use My Location".

prompt to use location w/ CLLocationManager
on callback, use lat/lng and reverse geocode to get address
save these values to UserDefaults, preferrably under a single key w/ a dictionary value
clear any saved zip code
trigger a refresh (via notification or callback) when upon change
IssuesViewController reads the saved location / address and passes to the fetch operation when loading issues.
location button title should be changed to match the city from the saved location

One thing to keep in mind is that we'll have to clear out the saved zip when GPS is used, and clear out the saved GPS if zip is used.  The issues view controller will have to prefer using GPS over zip, but check both.
Alternatively we could use another key that specifies which location type was used:
let locationType = UserDefaults.standard.string(for: UserDefaultKeys.locationType.rawValue)
if locationType == "gps" {
  // use saved address
} else if locationType == "zip" {
  // use zip
} else {
  // nil location
}

Up to you.

For now, I'm proceeding with "when in use" permissions only.

Yes, in fact I wish there was a “single use only” option :)

On Feb 2, 2017, at 10:17 AM, Chris Brandow<EMAIL_ADDRESS>wrote:
For now, I'm proceeding with "when in use" permissions only.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/5calls/ios/issues/6#issuecomment-277003494, or mute the thread https://github.com/notifications/unsubscribe-auth/AADnBL2hhHiGSdkAsA-i1B2dNBum2FRDks5rYgGYgaJpZM4L0pX7.


BSD support

An rclone user requested that rclone mount support openbsd.
OpenBSD apparently supports fuse and libfuse now so maybe cgofuse could too?

OpenBSD apparently supports fuse and libfuse now so maybe cgofuse could too?

Likely, but it would mean setting up and testing against a system that I have never used.
Does rclone currently supports OpenBSD (outside the mount command)?

Does rclone currently supports OpenBSD (outside the mount command)?

It does yes.  However I don't test against OpenBSD, and I think building a native binary will be difficult.
Unfortunately it seems like xgo doesn't support OpenBSD though which would make creating automated builds very difficult.

It might be nice to build BSD support, but I agree that without xgo builds would be difficult. We would probably have to also update xgo to support BSD.
I might look into all this in the future, especially if you get more than one person asking for it. I am rather short on time right now though.

I'm interested in NetBSD port.

@krytarowski does NetBSD support FUSE? Can you point me to good documentation?
A search finds this: https://www.netbsd.org/docs/puffs/

NetBSD supports 3 implementations for fuse: fuse, perfuse, refuse.
They vary in the scope of implementation. I recall that one of them is using just high-level interfaces, the other one attempted to support both, and the third one aimed to reuse upstream libfuse library with local patches.

I would get one that is completed, fully-featured and fully-compatible with the world. I hope that one of the can work with cgofuse.

Reading a bit through the NetBSD docs it seems that librefuse is the way to go. Only problem is that it only support the FUSE high-level API and I did have ideas of one day adding FUSE low-level support to cgofuse (once I get it in WinFsp that is).
So we have so far:

NetBSD: librefuse
OpenBSD: ???
FreeBSD: ???

https://wiki.freebsd.org/WhatsNew/FreeBSD10 notes libfuse work.. but there is an option for wait for requests.

FreeBSD information is somewhat contradicting. It does appear that there is good FUSE support in FreeBSD 10 and 11, but good documentation on how to load and use it is lacking. See, for example:
https://forums.freebsd.org/threads/54672/

I've never installed FreeBSD, no insight from me.

My first attempt at porting cgofuse to FreeBSD is in commit 0acc17d3001df68aa676d4ac0607d69b9af55e8b (currently in the freebsd branch).
Most things seem to work:

General file system use, although no rigorous testing has been done.
Cleanly unmounting the file system.

A notable exception is that signal handling seems to not be working properly. It is unclear currently why this is the case.

Additional information: I traced the system calls during memfs startup using truss and only kept traces related to sigaction and SIGTERM:
sigaction(SIGTERM,0x0,{ SIG_DFL 0x0 ss_t })      = 0 (0x0)                             <1>
sigaction(SIGTERM,{ 0x4556c0 SA_ONSTACK|SA_RESTART|SA_SIGINFO ss_t },0x0) = 0 (0x0)    <1>
sigaction(SIGTERM,0x0,{ 0x4556c0 SA_ONSTACK|SA_RESTART|SA_SIGINFO ss_t }) = 0 (0x0)    <2>
sa.handler=0x0                                                                         <2>
sigaction(SIGTERM,0x0,{ 0x4556c0 SA_ONSTACK|SA_RESTART|SA_SIGINFO ss_t }) = 0 (0x0)    <3>
sigaction(SIGTERM,{ 0x800bc0d90 SA_SIGINFO ss_t },{ 0x4556c0 SA_ONSTACK|SA_RESTART|SA_SIGINFO ss_t }) = 0 (0x0)    <3>

My analysis:
<1>: the Go runtime tests and sets a SIGTERM handler. It uses the SA_ONSTACK|SA_RESTART flags.
<2>: The modified hostMount executes sigaction and fprintf(stderr). It finds no SIGTERM handler.
<3>: Libfuse executes fuse_set_signal_handlers. It also finds no SIGTERM handler and for this reason it sets its own.
I cannot explain why after SIGTERM is set by the Go runtime, FreeBSD then reports that it is not set.

I have merged into master, although I cannot cut a release as signal handling remains broken in FreeBSD.

I believe I have found the reason for the "mysterious" sigaction behavior. I also believe that it is a Golang problem and I will report it as such.
The following simple cgo program:
package main

/*
#include <stdio.h>
#include <signal.h>

static void sigprint(int sig) {
	struct sigaction sa;
	int ret = sigaction(sig, 0, &sa);
	fprintf(stderr, "ret=%d handler=%p\n", ret, sa.sa_handler);
}
*/
import "C"
import "syscall"

func main() {
	C.sigprint(C.int(syscall.SIGTERM))
}

Succeeds (prints non-0) on Darwin and Linux:
$ go run sigbug.go
ret=0 handler=0x4034150

But fails (prints 0) on FreeBSD:
$ go run sigbug.go
ret=0 handler=0x0

The reason that this happens is that the Go runtime invokes sigaction using SYSCALL. However FreeBSD expects that C (and cgo) programs should use the libc sigaction, which is "interposed" by libthr to __thr_sigaction. Libthr maintains its own copy of sigaction's and overrides what the kernel returns.
A likely fix for this is to do what Linux does for sigaction. Invoke it using SYSCALL if "!cgo" or use the C library sigaction if "cgo". See rt_sigaction.

Turns out this is fixed on the current go repo tip. So the FreeBSD port should work without problems once Go 1.11 is out.

NetBSD wants to obsolete syscall(2) as this isn't that much portable across CPUs. Ideally we would call C symbols in C.

I ended up adding support for OpenBSD in commit c3f5fa1f80fbe228fc9f44ffe531c519ecff0c17. NetBSD support will have to wait for the next rainy Saturday :)
This support is experimental. It has known issues that stem from the differences in the OpenBSD libfuse implementation from the reference libfuse implementation.
- Signal handling is broken due to a bug in the OpenBSD implementation of fuse_set_signal_handlers.
- Option parsing may fail because the fuse_opt_parse function is not fully compatible with the one in libfuse.
An additional caveat is that in OpenBSD only root can mount and unmount FUSE file systems. There used to be a kern.usermount option, which allowed non-root users to mount file systems, but it was removed in OpenBSD 6 [link].
Finally note that because of the lack of a CI solution for BSD, it will be hard for me to maintain all these ports. I expect that bitrot will eventually set in.

I am on vacation and slightly jetlagged so I am working on NetBSD support. Will see where I get :)
In other news I have created Poor Man's CI for testing the BSD ports. It runs on the Google Cloud and currently supports FreeBSD. Will try to add NetBSD and OpenBSD support for it.

good progress!

@krytarowski good to hear from you.
BTW, how do I get Go 1.10 for NetBSD? Do I have to build it myself? (Even if that is the case I still need a bootstrap Go for NetBSD from somewhere.)

The typical way is to get pkgsrc. lang/go is 1.10.3 http://pkgsrc.se/lang/go
pkgsrc.org has some instructions.
Please note that there might be need to install mozilla-rootcerts for https.

Hmmm... I was misled by the list linked below, which does not appear to list a Go(?):
http://ftp.netbsd.org/pub/pkgsrc/current/pkgsrc/devel/README.html
I just tried the following and it worked.
export PKG_PATH="ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/x86_64/7.1.2/All"
pkg_add -v go

I indeed needed mozilla-rootcerts to be able to git clone. Thanks for the pointer.

go is in lang, not devel.

Makes sense. Just learning my way around NetBSD, hence the n00b mistake.
BTW, I got this to the point of passing most basic smoke tests, but have to troubleshoot a SIGSEGV, while calling fuse_opt_free_args. Getting very late now and the jetlag "boost" is gone, I will continue tomorrow.

NetBSD support was added in commit 3bc5ead903ef9354af159a4a0651a7925922477a.

This looks great! Thank you.

@krytarowski I am attempting to add NetBSD support to Poor Man's CI so that I can regularly test the NetBSD port using the Google Compute Engine.
I am currently using NetBSD 8.0_RC2, but I am not clear on how I should setup my PKG_PATH. I currently have it set as ftp://ftp.netbsd.org/pub/pkgsrc/packages/NetBSD/x86_64/8.0_current/All, but I get warnings such as:
pkg_add: Warning: package `curl-7.55.1nb1' was built for a platform:
pkg_add: NetBSD/x86_64 8.99.1 (pkg) vs. NetBSD/x86_64 8.0_RC2 (this host)

Any pointers on what PKG_PATH I should use for 8.0_RC2? I did try replacing 8.0_current with 8.0_RC2`, but that did not work.

Please try: http://ftp.netbsd.org/pub/pkgsrc/packages/NetBSD/x86_64/8.0_2018Q1/All/
8.99.1 is NetBSD-current and 8.0 was branched a year ago.

Thank you. This works without warnings.

Thank you for the port! I wonder whether perfuse would work better in NetBSD... we tend to have too many features for the same thing. 3 fuse implementations.

There are indeed 3 choices in NetBSD.

The canonical one is libpuffs, which best represents the kernel capabilities. However it does not have a FUSE compatible interface and therefore can not easily be used by cgofuse.

The librefuse implementation is a FUSE compatibility layer on top of PUFFS. It appears to be the natural choice for cgofuse.

The perfuse implementation is an emulation of the /dev/fuse linux kernel interface using an external daemon perfused. This is useful to FUSE "low-level" file systems, but introduces an unnecessary extra layer for cgofuse thus adding complexity and inefficiency.

I think that despite some incompatibilities librefuse is the right choice.

I see. Thank you for making it clear.
From a NetBSD developer point of view perhaps it looks like it would be the best to sync the kernel capabilities with Linux and get feature parity in libpuffs.. and keep an option for RUMP/PUFFS users with librefuse and export there low-level parts.
The libperfuse option can be kept around as an intermediate form to get the job done on NetBSD.
Your work is highly appreciated and thank you for the NetBSD port!

@krytarowski sorry to bother you again. I am facing a NetBSD failure while doing Continuous Integration using Poor Man's CI.
I note that cgofuse compiles and runs fine under NetBSD 7.1.2 (on my local VM) but fails when run under NetBSD 8.0_RC2 in the Google Cloud.
Here is the build log:
+ export GOPATH=/tmp/go
+ mkdir -p /tmp/go/src/github.com/billziss-gh
+ cp -R /tmp/repo/cgofuse /tmp/go/src/github.com/billziss-gh
+ cd /tmp/go/src/github.com/billziss-gh/cgofuse
+ go build -v ./...
github.com/billziss-gh/cgofuse/fuse
# github.com/billziss-gh/cgofuse/fuse
In file included from /usr/include/puffs.h:44:0,
                 from /usr/include/fuse.h:41,
                 from fuse/host_cgo.go:51:
/usr/include/fs/puffs/puffs_msgif.h:595:2: error: unknown type name 'register_t'
  register_t  pvnr_retval;  /* IN */
  ^
In file included from /usr/include/fuse.h:41:0,
                 from fuse/host_cgo.go:51:
/usr/include/puffs.h:225:27: error: unknown type name 'register_t'
      puffs_cookie_t, int, register_t *);
                           ^
/usr/include/puffs.h:436:1: error: unknown type name 'register_t'
 PUFFSOP_PROTOS(puffs_null) /* XXX */
 ^
/usr/include/puffs.h:505:20: error: unknown type name 'vsize_t'
        enum vtype, vsize_t, dev_t);
                    ^
github.com/billziss-gh/cgofuse/examples/shared

BUILD FAILURE

Do you have any idea what the missing register_t and vsize_t C symbols are and why this problem happens on 8.0_RC2 but not 7.1.2?

BTW, this is currently in the netbsd branch.

If this is a C code, please check -D_KERNTYPES or some equivalent like #define _KERNTYPES before including headers.

Thanks for the hint. Adding -D _KERNTYPES does indeed allow cgofuse to be built on NetBSD 8.0_RC2.
(As a user of NetBSD I would argue that this should be considered a problem. The need for _KERNTYPES is not documented in the refuse(3) man page. Furthermore users of FUSE on other systems would likely not know about _KERNTYPES.)
In any case I hit an even more serious problem when trying to run some of the cgofuse smoke tests. It does not look that Go 1.10 is ready for use on NetBSD 8.0_RC2 yet :(
It looks like I will have to shelve doing CI for NetBSD, until 8.0 and its associated tooling has matured a bit more. @krytarowski thank you for your help on this.
+ export GOPATH=/tmp/go
+ mkdir -p /tmp/go/src/github.com/billziss-gh
+ cp -R /tmp/repo/cgofuse /tmp/go/src/github.com/billziss-gh
+ cd /tmp/go/src/github.com/billziss-gh/cgofuse
+ go build -v ./...
github.com/billziss-gh/cgofuse/fuse
github.com/billziss-gh/cgofuse/examples/hellofs
github.com/billziss-gh/cgofuse/examples/shared
github.com/billziss-gh/cgofuse/examples/memfs
github.com/billziss-gh/cgofuse/examples/passthrough
+ go test -v ./fuse -run 'TestUnmount|TestSignal'
=== RUN   TestUnmount
reqid: 0, opclass 1, optype: PUFFS_VFS_UNMOUNT, cookie: 0x0,
    aux: 0x702e1132c038, auxlen: 64, pid: 730, lwpid: 6
    since previous call:<PHONE_NUMBER>.176428
fatal: morestack on g0
SIGTRAP: trace trap
PC=0x456282 m=0 sigcode=1
signal arrived during cgo execution

goroutine 8 [running, locked to thread]:
runtime.morestack()
	/usr/pkg/go/src/runtime/asm_amd64.s:451 +0x22 fp=0x702e1173fd00 sp=0x702e1173fcf8 pc=0x456282
created by github.com/billziss-gh/cgofuse/fuse.testHost
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host_test.go:76 +0x278

goroutine 1 [chan receive]:
testing.(*T).Run(0xc4200a40f0, 0x54d4d0, 0xb, 0x555148, 0x46b1b6)
	/usr/pkg/go/src/testing/testing.go:825 +0x301
testing.runTests.func1(0xc4200a4000)
	/usr/pkg/go/src/testing/testing.go:1063 +0x64
testing.tRunner(0xc4200a4000, 0xc420051df8)
	/usr/pkg/go/src/testing/testing.go:777 +0xd0
testing.runTests(0xc42000c060, 0x7fad20, 0x3, 0x3, 0x411779)
	/usr/pkg/go/src/testing/testing.go:1061 +0x2c4
testing.(*M).Run(0xc42008e000, 0x0)
	/usr/pkg/go/src/testing/testing.go:978 +0x171
main.main()
	_testmain.go:46 +0x151

goroutine 5 [syscall]:
os/signal.signal_recv(0x0)
	/usr/pkg/go/src/runtime/sigqueue.go:139 +0xa6
os/signal.loop()
	/usr/pkg/go/src/os/signal/signal_unix.go:22 +0x22
created by os/signal.init.0
	/usr/pkg/go/src/os/signal/signal_unix.go:28 +0x41

goroutine 6 [syscall]:
github.com/billziss-gh/cgofuse/fuse._Cfunc_hostUnmount(0x702e11310200, 0x702e1130b080, 0x0)
	_cgo_gotypes.go:554 +0x4d
github.com/billziss-gh/cgofuse/fuse.c_hostUnmount.func1(0x702e11310200, 0x702e1130b080, 0x702e1130b080)
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host_cgo.go:704 +0x60
github.com/billziss-gh/cgofuse/fuse.c_hostUnmount(0x702e11310200, 0x702e1130b080, 0x702e1130b080)
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host_cgo.go:704 +0x35
github.com/billziss-gh/cgofuse/fuse.(*FileSystemHost).Unmount(0xc420010840, 0x0)
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host.go:686 +0x4c
github.com/billziss-gh/cgofuse/fuse.testHost(0xc4200a40f0, 0x457c01)
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host_test.go:82 +0x2ad
github.com/billziss-gh/cgofuse/fuse.TestUnmount(0xc4200a40f0)
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host_test.go:102 +0x30
testing.tRunner(0xc4200a40f0, 0x555148)
	/usr/pkg/go/src/testing/testing.go:777 +0xd0
created by testing.(*T).Run
	/usr/pkg/go/src/testing/testing.go:824 +0x2e0

goroutine 9 [chan receive]:
github.com/billziss-gh/cgofuse/fuse.(*FileSystemHost).Mount.func3(0xc420010840, 0xc420060300)
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host.go:659 +0x41
created by github.com/billziss-gh/cgofuse/fuse.(*FileSystemHost).Mount
	/tmp/go/src/github.com/billziss-gh/cgofuse/fuse/host.go:658 +0x3eb

goroutine 10 [select, locked to thread]:
runtime.gopark(0x555690, 0x0, 0x54c6f3, 0x6, 0x18, 0x1)
	/usr/pkg/go/src/runtime/proc.go:291 +0x11a
runtime.selectgo(0xc42002ef50, 0xc4200603c0)
	/usr/pkg/go/src/runtime/select.go:392 +0xe50
runtime.ensureSigM.func1()
	/usr/pkg/go/src/runtime/signal_unix.go:549 +0x200
runtime.goexit()
	/usr/pkg/go/src/runtime/asm_amd64.s:2361 +0x1

rax    0x17
rbx    0x80ea40
rcx    0x459d95
rdx    0x0
rdi    0x2
rsi    0x54fce6
rbp    0xc420040a28
rsp    0x702e1173fcf8
r8     0xc420001980
r9     0xc420001980
r10    0xc42001a070
r11    0x212
r12    0x0
r13    0x10000000
r14    0x0
r15    0x702e1132c000
rip    0x456282
rflags 0x216
cs     0x47
fs     0x0
gs     0x0
FAIL	github.com/billziss-gh/cgofuse/fuse	3.419s

BUILD FAILURE

We were fixing Go in their HEAD branch.. is it possible to check Go from the HEAD?

I propose to file the _KERNTYPES issue to gnats. netbsd.org -> support -> report a bug
If Go is still broke in their HEAD or 1.11, please file a bug for it too.

We were fixing Go in their HEAD branch.. is it possible to check Go from the HEAD?

Yes. I will look into it.

Ok. I compiled my own Go from golang/go tip. Unfortunately not all tests passed, but I will try to use it for building cgofuse on NetBSD 8.0_RC2 nevertheless.

@krytarowski unfortunately the problem persists with latest go :(
builder0# uname -a
NetBSD builder0.c.poor-mans-ci.internal 8.0_RC2 NetBSD 8.0_RC2 (GENERIC) #0: Mon Jun 25 13:01:26 UTC 2018  mkrepro@mkrepro.NetBSD.org:/
usr/src/sys/arch/amd64/compile/GENERIC amd64
builder0# go version
go version devel +0dc814cd7f Sat Jun 30 01:04:30 2018 +0000 netbsd/amd64
builder0# git checkout netbsd
Branch 'netbsd' set up to track remote branch 'netbsd' from 'origin'.
Switched to a new branch 'netbsd'
builder0# go build ./...
builder0# go test -v ./fuse
=== RUN   TestUnmount
reqid: 0, opclass 1, optype: PUFFS_VFS_UNMOUNT, cookie: 0x0,
    aux: 0x700f8132c038, auxlen: 64, pid: 1311, lwpid: 6
    since previous call:<PHONE_NUMBER>.602584
fatal: morestack on g0
SIGTRAP: trace trap
PC=0x45a182 m=0 sigcode=1
signal arrived during cgo execution

goroutine 8 [running, locked to thread]:
runtime.abort()
        /usr/pkg/go-custom/src/runtime/asm_amd64.s:840 +0x2 fp=0x700f812ffcf8 sp=0x700f812ffcf0 pc=0x45a182
runtime.morestack()
        /usr/pkg/go-custom/src/runtime/asm_amd64.s:397 +0x25 fp=0x700f812ffd00 sp=0x700f812ffcf8 pc=0x458835
created by github.com/billziss-gh/cgofuse/fuse.testHost
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host_test.go:76 +0x26f

goroutine 1 [chan receive]:
testing.(*T).Run(0xc000092100, 0x5530aa, 0xb, 0x55af40, 0x46d486)
        /usr/pkg/go-custom/src/testing/testing.go:879 +0x37a
testing.runTests.func1(0xc000092000)
        /usr/pkg/go-custom/src/testing/testing.go:1119 +0x78
testing.tRunner(0xc000092000, 0xc000083e08)
        /usr/pkg/go-custom/src/testing/testing.go:827 +0xbf
testing.runTests(0xc00000c060, 0x842b00, 0x3, 0x3, 0x40dc8f)
        /usr/pkg/go-custom/src/testing/testing.go:1117 +0x2a2
testing.(*M).Run(0xc00008e000, 0x0)
        /usr/pkg/go-custom/src/testing/testing.go:1034 +0x165
main.main()
        _testmain.go:46 +0x13d

goroutine 5 [syscall]:
os/signal.signal_recv(0x0)
        /usr/pkg/go-custom/src/runtime/sigqueue.go:139 +0x9c
os/signal.loop()
        /usr/pkg/go-custom/src/os/signal/signal_unix.go:23 +0x22
created by os/signal.init.0
        /usr/pkg/go-custom/src/os/signal/signal_unix.go:29 +0x41
goroutine 6 [syscall]:
github.com/billziss-gh/cgofuse/fuse._Cfunc_hostUnmount(0x700f81310200, 0x700f8130b080, 0x0)
        _cgo_gotypes.go:554 +0x4d
github.com/billziss-gh/cgofuse/fuse.c_hostUnmount.func1(0x700f81310200, 0x700f8130b080, 0x700f8130b080)
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host_cgo.go:704 +0x60
github.com/billziss-gh/cgofuse/fuse.c_hostUnmount(0x700f81310200, 0x700f8130b080, 0x700f8130b080)
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host_cgo.go:704 +0x35
github.com/billziss-gh/cgofuse/fuse.(*FileSystemHost).Unmount(0xc0000507c0, 0x0)
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host.go:686 +0x4a
github.com/billziss-gh/cgofuse/fuse.testHost(0xc000092100, 0x344801)
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host_test.go:82 +0x2a4
github.com/billziss-gh/cgofuse/fuse.TestUnmount(0xc000092100)
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host_test.go:102 +0x30
testing.tRunner(0xc000092100, 0x55af40)
        /usr/pkg/go-custom/src/testing/testing.go:827 +0xbf
created by testing.(*T).Run
        /usr/pkg/go-custom/src/testing/testing.go:878 +0x353
goroutine 9 [chan receive]:
github.com/billziss-gh/cgofuse/fuse.(*FileSystemHost).Mount.func3(0xc0000507c0, 0xc00005e300)
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host.go:659 +0x41
created by github.com/billziss-gh/cgofuse/fuse.(*FileSystemHost).Mount
        /root/go/src/github.com/billziss-gh/cgofuse/fuse/host.go:658 +0x3d2
rax    0x17
rbx    0x856a00
rcx    0x45c145
rdx    0x0
rdi    0x2
rsi    0x5558f4
rbp    0xc000044a50
rsp    0x700f812ffcf0
r8     0x32
r9     0x32
r10    0x2
r11    0x212
r12    0x0
r13    0x10000000
r14    0x0
r15    0x700f8132c000
rip    0x45a182
rflags 0x216
cs     0x47
fs     0x0
gs     0x0
FAIL    github.com/billziss-gh/cgofuse/fuse     3.389s

I see, thank you for the test. Please report it on gnats!


kgx: Update repo URL

Motivation for this change
KGX has been rebranded to GNOME Console with the repo moved without a redirect. We aren't changing the name since we are still using an old version before the rebrand, with the update blocked by vte (https://github.com/NixOS/nixpkgs/pull/142731#issuecomment-961294830).
This doesn't cause any rebuild.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Superseded by #160343.


fix spring boot project

Hey @rockoder @eric-martin
This commit is failing the build. Can you please look into it, below is the error:
[ERROR] Could not find goal 'update-theme' in plugin com.vaadin:vaadin-maven-plugin:10.0.11 among available goals copy-production-files, package-for-production -> [Help 1] [ERROR]  [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch. [ERROR] Re-run Maven using the -X switch to enable full debug logging. [ERROR]  [ERROR] For more information about the errors and possible solutions, please read the following articles: [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoNotFoundException
CC : @eugenp

This was not happening consistently for me. Raised a followup PR: https://github.com/eugenp/tutorials/pull/6789/files


Need documentation

I cannot use your library, there is a lack of documentation and examples of implementation.
I have to study all the source code to understand how to implement a particular function.
Will there be normal documentation, and if so when ?

All I have right now is here: https://ibm-dev-incubator.github.io/demoparser/. It's a list of all of the events with a brief explanation. It also has the auto-generated API docs. I don't have a lot of time to work on this but I could add some more example code. Are there particular examples that you think would be helpful to have?


Remove residual mentions of --tsv

This PR removes residual mentions of the command line argument --tsv.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install

Hmmm, looks like there is supporting code for this that can be stripped out as well. Look for entry2tsvHelp and associated code

@denis-yuen should https://github.com/dockstore/dockstore-cli/blob/b5eab7c349c0a99cdc289ccb60c70be71e326d26/dockstore-client/src/main/java/io/dockstore/client/cli/nested/WorkflowClient.java#L1214
This method also be removed? Not sure if it's needed anymore.


Vertical

Hi can you add a vertical prop to make a vertical slider?

Ye, @maartenvandillen   I need a moment, wait for me.

I think you can use style to vertical. @maartenvandillen
Example code:
 <Slider style={[
      styles.container,
      {
        transform: [
          {
            rotate: '90deg',
          },
        ],
      },
    ]}
    progress={progress5}
    minimumValue={min}
    maximumValue={max}
    minimumTrackTintColor="#FFAB76"
    maximumTrackTintColor="#FFEEAD"
  />

transforms are only visual. Does that work ok with the gesture handler?

It's ok, I tried, but you may need to adjust the position.

Thanks for looking into this. I just realised I need to upgrade to reanimated v2 and use the Hermes engine on Android. Are those two ready for production? My last experience with Hermes was a SIGFAULT error on app startup so I stayed clear from it for a while.

Yes, I‘ve been using Hermes since react-native 0.64+, hermes engine is awesome. If you encounter any problems using Hermes there are solutions.

Hi, @maartenvandillen If there is no problem, I will close this Issue, thank you!


Add From methods that can create sequences from functions

This builds on top of #331 so that should be reviewed and merged first.

There is a fundamental inconsistency between the overloads of From. The ones taking individual functions as arguments don't allow for null references. The one taking the variable array of functions, however, does!

Good catch! Indeed, this needs to be resolved. Given that we can reasonably expect callers to be creating relatively short sequences, I think we should check for null in the params overload too. What do you think?
I pushed commits fixing your comments. I also picked up the change you suggested for Evaluate docstring.

This switch reads very long (nitpicking on readability)

Maybe it would be better to split the testcase into separate tests...

I'm not much of a fan of nested ? : operators.

Sometimes, there's no choice, e.g., when using anonymous types:
[TestCase(1)]
[TestCase(2)]
[TestCase(3)]
[TestCase(4)]
public void TestFromInvokesMethods(int numArgs)
{
    int F1() => -2;
    int F2() => 4;
    int F3() => int.MaxValue;
    int F4() => int.MinValue;

    var r
        = numArgs == 1 ? new { Actuals = MoreEnumerable.From(F1)            , Expectations = new[] { F1()                   } }
        : numArgs == 2 ? new { Actuals = MoreEnumerable.From(F1, F2)        , Expectations = new[] { F1(), F2()             } }
        : numArgs == 3 ? new { Actuals = MoreEnumerable.From(F1, F2, F3)    , Expectations = new[] { F1(), F2(), F3()       } }
        : numArgs == 4 ? new { Actuals = MoreEnumerable.From(F1, F2, F3, F4), Expectations = new[] { F1(), F2(), F3(), F4() } }
        : throw new ArgumentOutOfRangeException(nameof(numArgs));

    r.Actuals.AssertSequenceEqual(r.Expectations);
}

Now if you do the assertions directly in the switch cases, then it makes more sense and perhaps reads equally simpler:
[TestCase(1)]
[TestCase(2)]
[TestCase(3)]
[TestCase(4)]
public void TestFromInvokesMethods(int numArgs)
{
    int F1() => -2;
    int F2() => 4;
    int F3() => int.MaxValue;
    int F4() => int.MinValue;

    switch (numArgs)
    {
        case 1: MoreEnumerable.From(F1).AssertSequenceEqual(F1()); break;
        case 2: MoreEnumerable.From(F1, F2).AssertSequenceEqual(F1(), F2()); break;
        case 3: MoreEnumerable.From(F1, F2, F3).AssertSequenceEqual(F1(), F2(), F3()); break;
        case 4: MoreEnumerable.From(F1, F2, F3, F4).AssertSequenceEqual(F1(), F2(), F3(), F4()); break;
        default: throw new ArgumentOutOfRangeException(nameof(numArgs));
    }
}

Perhaps the second version is a good compromise?

PS Note also use of local functions to simplify things further since.

Generally, LINQ operators don't validate the elements of a sequence and pass-through any exceptions. However, if we want to make a special case here then it can get tricky. What's the error you would throw? Also, it could be surprising to see Evaluate not behaving in the same manner and if we want to render it consistent with From then it would end up scanning it source sequence before use. In light of all this, I'm inclined to say at the moment that null functions should be accepted and break when invoked.

OK, your argument makes sense to me. I have therefore removed those checks. However, now the null checks fail.
However now the NullArgumentTest fails because it breaks the convention... Should we add an explicit waiver to From somewhere in NullArgumentTest?

However now the NullArgumentTest fails because it breaks the convention... Should we add an explicit waiver to From somewhere in NullArgumentTest?

Aye!

Done.


LinearClassifier feature_columns overwritten in LinearClassifier.fit

tensorflow.contrib.learn.python.learn.estimators.linear.LinearClassifier.fit
effectively returns
tensorflow.contrib.learn.python.learn.estimators.estimator.Estimator.fit
Estimator.fit calls:
tensorflow.contrib.learn.python.learn.estimators.estimator.Estimator._train_model
which has these lines:
      features, targets = input_fn()
      self._check_inputs(features, targets)
      train_op, loss_op = self._get_train_ops(features, targets)

tensorflow.contrib.learn.python.learn.estimators.estimator.Estimator._check_inputs
    if self._features_info is not None:
      ...
    else:
      self._features_info = tensor_signature.create_signatures(features)

So we get to a point where features, as derived from the input_fn, is treated as our feature columns set.
In pseudo code:
def input_function()
    return [foo, bar, baz], quux

lc=LinearClassifier(feature_columns=[foo]) # We pass in what we want our feature column to be
lc.fit(input_function) # run fit on out input function
lc._feature_columns # repr _feature_columns on the instantiated classifier
    [foo]
lc.estimator._features_info # _features_info on the instantiated classifiers instantiated estimator
    [foo, bar, baz]

The issue is:
Although this line appears to indicate that we will be making an estimation based on the feature columns supplied:
lc=LinearClassifier(feature_columns=[foo]) # We pass in what we want our feature column to be
What happens is that the passed in feature_columns is unused and instead the return from the input_function supplied to fit are used.
Am I correct in thinking that if the feature_columns arg is supplied that only those columns should be used by the classifiers estimator?
That when we instantiate the classifier we are setting the feature_columns we expect to be used?
The work around for this is simply to only return the columns you need from your input function however I found this misleading.
Point in the tutorial:
Either the code or the tutorial need to be changed.
https://www.tensorflow.org/versions/r0.10/tutorials/wide/index.html#defining-the-logistic-regression-model
Error raised:
WARNING:tensorflow:Setting feature info to
as per tensorflow/contrib/learn/python/learn/estimators/estimator.py:613

@hengtze: is this something that needs to be changed in the wide tutorial?  If not, is this something the tf.learn side should look into?

Automatically closing due to lack of recent activity. Since this issue is old at this point, please reopen the issue if it still occurs when tried with the latest version of Tensorflow. Thank you.


Add technology mapboxgljs

Website:
https://docs.mapbox.com/mapbox-gl-js/api/
https://github.com/mapbox/mapbox-gl-js
Example websites:
https://www.speedtest.net/ookla-5g-map
https://www.alltrails.com/explore
https://mapcherry.io/services/maps/
https://www.remax.ro/agentii/agentii-imobiliare-cluj-napoca
https://locations.kfc.com/al/albertville/7285-us-highway-431

Thanks!


Sejong is never going to build timestamps

Sejong's block 1 has a timestamp of 0:
{
    "jsonrpc": "2.0",
    "result": {
        "block_hash": "60f2b7d39f7e07ddbec7f8a5966d992d6db53b5f8a2fc2a21dc498b872c06bcf",
        "confirmed_transaction_list": [],
        "height": 1,
        "merkle_tree_root_hash": "",
        "peer_id": "hxd317634efef2c9a08638ce1702335020af82619a",
        "prev_block_hash": "61497566ec3a1459685dc96ea763ffae76edd150b3f1cb537418f8fe12aa0acc",
        "signature": "",
        "time_stamp": 0,
        "version": "2.0"
    },
    "id": 1234
}

If we're checking that no block has a timestamp of 0 (https://github.com/geometry-labs/icon-blocks/blob/main/src/worker/builders/block_time_builder.go#L38), we're never going to be able to process any timestamps. Should we be putting in a network flag or something goofy to override this behaviour? Or maybe just a special case for block 1? We could also probably just fudge it and set timestamp for block 1 to 1.

Could this be solved by having the ETL manually setting block #1's timestamp to block #2's timestamp, or just 1?

I mean, I'll always advocate against having special custom logic that changes data in ETL. The purpose of ETL is to dump the data as faithfully as it can. We could always just go into the DB and adjust it ourselves, but there's also no guarantee the next testnet is going to have a 0 timestamp for block 1.


Fix #922, Update cfe_error.h to use the CFE_Status_t type

Describe the contribution
Fixing issue #922
Contributor Info - All information REQUIRED for consideration of pull request
Full name and company/organization/center of all contributors ("Personal" if individual work)

If NASA Civil Servant Employee or GSFC Contractor on SES II

Address/email/phone and contract/task information (if applicable) must be on file

Else if Company

HAND SIGNED Company CLA must be on file (once per release): Company CLA

Else if Individual

HAND SIGNED Individual CLA must be on file (once per release): Individual CLA

Hi @pducolin thank you for your contribution! Could you sign and email the appropriate Contributor License agreement?

Corporate: https://github.com/nasa/cFE/blob/main/docs/GSC_18128_Corp_CLA_form_1219.pdf
Individual: https://github.com/nasa/cFE/blob/main/docs/GSC_18128_Ind_CLA_form_1219.pdf

email it to<EMAIL_ADDRESS>and copy<EMAIL_ADDRESS>Thanks!

Hi! Contributor agreement sent @astrogeco, thank you for adding the link.
@zanzaben conflicts fixed - I rebased on nasa:main.
Sorry for the delay!

Hi! Contributor agreement sent @astrogeco, thank you for adding the link.
@zanzaben conflicts fixed - I rebased on nasa:main.
Sorry for the delay!

Received! Thank you for the contribution!

@pducolin one more request, can you amend the commit to match the title for this Pull Request?

CCB:2021-02-24 APPROVED


[SPARK-41197] Upgrade Kafka version to 3.3 release

What changes were proposed in this pull request?
This PR upgrades Kafka to 3.3.1 release.
Why are the changes needed?
Kafka 3.3.1 release has new features along with bug fixes: https://www.confluent.io/blog/apache-kafka-3-3-0-new-features-and-updates/
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing test suite

@HeartSaVioR
Can you take a look ?

Hi, 3.3.0 A significant bug was found in the 3.3.0 release after artifacts were pushed to Apache and Maven central but prior to the release announcement. As a result, the decision was made to not announce 3.3.0 and instead release 3.3.1 with the fix. It is recommended that 3.3.0 not be used.
Can you try with 3.3.1

@bjornjorgensen
I have updated the PR.

I noticed the test failures.
Let me analyze the test output.

Can it have something to do with https://github.com/apache/kafka/pull/12794 ?

Possible.
When would the next Kafka be released ?

cc @dongjoon-hyun and @HeartSaVioR FYI

@HyukjinKwon @dongjoon-hyun @HeartSaVioR
Please take a look - tests pass.

This is merged via https://github.com/apache/spark/commit/0ff201cc219884d3cbb6844732c681546e53f4d4 .


dsp/window: add Values.TransformTo window functions and example

Please take a look.

Codecov Report

Merging #1547 (40ef18c) into master (1c2011e) will decrease coverage by 0.03%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##           master    #1547      +/-   ##
==========================================
- Coverage   71.49%   71.46%   -0.04%
==========================================
  Files         474      474
  Lines       54052    54067      +15
==========================================
- Hits        38647    38640       -7
- Misses      13021    13033      +12
- Partials     2384     2394      +10

Impacted Files
Coverage Δ

dsp/window/window_parametric.go
69.23% <20.00%> (-14.77%)
:arrow_down:

graph/flow/control_flow_slt.go
84.72% <0.00%> (-12.50%)
:arrow_down:

optimize/listsearch.go
72.22% <0.00%> (-1.86%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1547 (40ef18c) into master (1c2011e) will decrease coverage by 0.03%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##           master    #1547      +/-   ##
==========================================
- Coverage   71.49%   71.46%   -0.04%
==========================================
  Files         474      474
  Lines       54052    54067      +15
==========================================
- Hits        38647    38640       -7
- Misses      13021    13033      +12
- Partials     2384     2394      +10

Impacted Files
Coverage Δ

dsp/window/window_parametric.go
69.23% <20.00%> (-14.77%)
:arrow_down:

graph/flow/control_flow_slt.go
84.72% <0.00%> (-12.50%)
:arrow_down:

optimize/listsearch.go
72.22% <0.00%> (-1.86%)
:arrow_down:

Continue to review full report at Codecov.

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


Build error with github.com/pelletier/

While building Hugo from todays git clone version, I have the following error:

$ go version
go version go1.18beta2 linux/amd64

$ git log -1
commit<PHONE_NUMBER>d6ec8e034c7a097575defe8611dadb (HEAD -> master, origin/master, origin/HEAD)

$ go build -x -v
...
...
...
0.084s # cd /home/e/go/pkg/mod/cache/vcs/5fc8a6833fc262d7a365ce89f874516d37470daefa52149126edb8776dd883fa; git -c core.autocrlf=input -c core.eol=lf archive --format=zip --prefix=prefix/ c3ba3ef97abbc154a9acc1b0164a022f48e5fd6e
verifying<EMAIL_ADDRESS>checksum mismatch
        downloaded: h1:TA3qh4Cq1VOjh4AJnMHrUFbAsIbD7OnOvpaIRYsU0sI=
        go.sum:     h1:JFNqj2afbbhCqTiyN16D7Tudc/aaDzE2FBDk+VlBQnE=

SECURITY ERROR
This download does NOT match an earlier download recorded in go.sum.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

Delete go.sum. Retry.

Hi! Thanks for the tip @davidsneighbour.
Managed to build by deleting these entries from "mod.sum", and running again "go mod tidy".
github.com/niklasfasching/go-org v1.6.0
github.com/pelletier/go-toml v1.9.3
github.com/pelletier/go-toml v1.9.4
github.com/pelletier/go-toml/v2 v2.0.0-beta.3.0.20210727221244-fa0796069526 github.com/pelletier/go-toml/v2 v2.0.0-beta.6

@EduRam editing go.sum does not sound like a ... great idea. This sounds like a corrupted download. I suggest you try to clean out your mod cache and try again. Since this works fine for me, GitHub CI and many others, I don't see how that checksum can be wrong.