arithmetic on a nil value

Hi. I'm using dashboard only for the header, without any navigation. This is my config:
dashboard.custom_footer = { [[]] }
dashboard.custom_center = { {} }

The first time dashboard opens without an error, but after switching a buffer and running :Dashboard i get the following error:
Error executing Lua callback: ...share/nvim/plugged/dashboard-nvim/lua/dashboard/init.lua:342: attempt to perform arithmetic on a nil value
stack traceback:
        ...share/nvim/plugged/dashboard-nvim/lua/dashboard/init.lua:342: in function 'load_from_cache'
        ...share/nvim/plugged/dashboard-nvim/lua/dashboard/init.lua:390: in function 'instance'
        ...l/share/nvim/plugged/dashboard-nvim/plugin/dashboard.lua:63: in function <...l/share/nvim/plugged/dashboard-nvim/plugin/dashboard.lua:62>

Separate issue, but related to this: since I don't have any need for navigation with hjkl I want to map l to edit my lua init file, but it doesn't look possible:
au FileType dashboard nnoremap <buffer> <silent>l :e ~/.config/nvim/lua/init.lua<CR> <-- this doesn't work in the new version, but it used to work fine in the past.

use
db.custom_center = { {desc = ' ' }}
db.custom_footer = { ' '}


TLS ERROR with Vault using self-signed certificate

My vault is using self-signed cert and I have followed below:
https://github.com/external-secrets/kubernetes-external-secrets/blob/458f39a3d67c4e2ff02ce1c20b9a1fbb3c3dbce7/README.md?plain=1#L658
Error:
level:50 message_time:2021-12-01T04:53:21.148Z pid:19 hostname:external-secrets-kubernetes-external-secrets-6b9c77644f-mw9vv payload:{err:type:RequestError} payload:{err:message:Error: SSL Error: UNABLE_TO_GET_ISSUER_CERT} payload:{err:stack:RequestError: Error: SSL Error: UNABLE_TO_GET_ISSUER_CERT
    at new RequestError (/app/node_modules/request-promise-core/lib/errors.js:14:15)
    at Request.plumbing.callback (/app/node_modules/request-promise-core/lib/plumbing.js:87:29)
    at Request.RP [as _callback] (/app/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at self.callback (/app/node_modules/request/request.js:185:22)
    at Request.emit (events.js:400:28)
    at Request.emit (domain.js:475:12)
    at Request.onRequestResponse (/app/node_modules/request/request.js:948:10)
    at ClientRequest.emit (events.js:400:28)
    at ClientRequest.emit (domain.js:475:12)
    at HTTPParser.parserOnIncomingClient (_http_client.js:647:27)} payload:{err:name:RequestError} payload:{err:cause:type:Error} payload:{err:cause:message:SSL Error: UNABLE_TO_GET_ISSUER_CERT} payload:{err:cause:stack:Error: SSL Error: UNABLE_TO_GET_ISSUER_CERT
    at Request.onRequestResponse (/app/node_modules/request/request.js:948:24)
    at ClientRequest.emit (events.js:400:28)
    at ClientRequest.emit (domain.js:475:12)
    at HTTPParser.parserOnIncomingClient (_http_client.js:647:27)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:127:17)
    at TLSSocket.socketOnData (_http_client.js:515:22)
    at TLSSocket.emit (events.js:400:28)
    at TLSSocket.emit (domain.js:475:12)
    at addChunk (internal/streams/readable.js:293:12)
    at readableAddChunk (internal/streams/readable.js:267:9)} payload:{err:error:type:Error} payload:{err:error:message:SSL Error: UNABLE_TO_GET_ISSUER_CERT} payload:{err:error:stack:Error: SSL Error: UNABLE_TO_GET_ISSUER_CERT
    at Request.onRequestResponse (/app/node_modules/request/request.js:948:24)
    at ClientRequest.emit (events.js:400:28)
    at ClientRequest.emit (domain.js:475:12)
    at HTTPParser.parserOnIncomingClient (_http_client.js:647:27)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:127:17)
    at TLSSocket.socketOnData (_http_client.js:515:22)
    at TLSSocket.emit (events.js:400:28)
    at TLSSocket.emit (domain.js:475:12)
    at addChunk (internal/streams/readable.js:293:12)
    at readableAddChunk (internal/streams/readable.js:267:9)} payload:{err:options:json:role:argocd} payload:{err:options:json:jwt:[Redacted]} payload:{err:options:resolveWithFullResponse:true} payload:{err:options:simple:false} payload:{err:options:strictSSL:true} payload:{err:options:followAllRedirects:true} payload:{err:options:method:POST} payload:{err:options:path:/auth/kubernetes/login} payload:{err:options:headers:[Redacted]} payload:{err:options:uri:https://vault.vault.svc.cluster.local:8200/v1/auth/kubernetes/login} payload:{err:options:transform2xxOnly:false} msg:failure while polling the secret argocd/exsecret1

Inside pod, I verified the correct ca are at the right place
/app $ env | grep NODE
NODE_EXTRA_CA_CERTS=/usr/local/share/ca-certificates/ca.pem
NODE_VERSION=14.18.1
NODE_TLS_REJECT_UNAUTHORIZED=0
NODE_ENV=production
/app $ cat /usr/local/share/ca-certificates/ca.pem
<removed>

Can you please advice?

This issue can be closed. my vault was not selfsigned, it was signed by intermediate CA. I was able to get rid of this problem with intermediate CA.

My bad, my vault was not self-signed. It was signed by intermediate CA. I was able to get rid of SSL error with intermediate CA.


test: improve test coverage (#83)

This pull request improves the test coverage of the main function test pattern and releases a new version. However, it still fails to address on important case: test failures happening when updating the compiler without re-compiling the full test code.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 99.836%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

test/pattern.go
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>0:
0.9%

Covered Lines:
1830

Relevant Lines:
1833

💛  - Coveralls


Typescript: Missing suggestions in field initializers

I'm finding that TypeScript fails to provide suggestions in very easy places. Check this out:

Thanks. Opened https://github.com/microsoft/vscode/issues/82989 to track this upstream

@mjbvz You might've missed a copy, that points to this issue.

Fixed https://github.com/microsoft/TypeScript/issues/34621


policyAssignmentManagementGroup.bicep doesnt seem to add a policy to additional management groups

What happened? Provide a clear and concise description of the bug, including deployment details.
I a using the ALZ modules and particularly this one - ALZ-Bicep/infra-as-code/bicep/modules/policy/assignments/policyAssignmentManagementGroup.bicep
When I run this and populate the parameter called parPolicyAssignmentIdentityRoleAssignmentsAdditionalMgs like so ['mg-Test', 'mg-PreProduction'] neither of these 2 are being populated only the one I reference like so:-
az deployment mg create --name $deploymentID
--location $primaryLocation --management-group-id 'mg-Production'
--template-file ./policyAssignmentManagementGroup.bicep --parameters ./policyAssignmentManagementGroup.bicepparam
--confirm-with-what-if `
--output none
so mg-Production is assigned a policy but neither of the 2 additional ones are (['mg-Test', 'mg-PreProduction'] )
Please provide the correlation id associated with your error or bug.
n/a
What was the expected outcome?
I would expect all 3 Management Groups to have the Policy assigned
Relevant log output
No response
Check previous GitHub issues

[X] I have searched the issues for this item and found no duplicate

Code of Conduct

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

Hey @gsuttie,
Can you share the module declaration code you are using and what version of the module?
Maybe via a gist or here as a comment so we can try and repro?

Module code im using is from policyAssignmentManagementGroup.bicep
And I am calling from PowerShell like so:-
az deployment mg create --name $deploymentID
--location $primaryLocation --management-group-id 'mg-Production'
--template-file ./policyAssignmentManagementGroup.bicep --parameters ./policyAssignmentManagementGroup.bicepparam
--confirm-with-what-if `
--output none
Here is my bicepparameters file
using 'policyAssignmentManagementGroup.bicep'
param parPolicyAssignmentName = 'Iso27001'
param parPolicyAssignmentDisplayName = 'ISO 27001-2013'
param parPolicyAssignmentDescription = 'This policy assignment is for ISO 27001-2013'
param parPolicyAssignmentDefinitionId = '/providers/Microsoft.Authorization/policySetDefinitions/89c6cddc-1c73-4ac1-b19c-54d1a15a42f2'
param parPolicyAssignmentParameters = {}
param parPolicyAssignmentParameterOverrides = {}
//param parPolicyAssignmentNonComplianceMessages nonComplianceMessageType = []
param parPolicyAssignmentNotScopes = []
param parPolicyAssignmentEnforcementMode = 'Default'
param parPolicyAssignmentOverrides = []
param parPolicyAssignmentResourceSelectors = []
param parPolicyAssignmentIdentityType = 'SystemAssigned'
param parPolicyAssignmentIdentityRoleAssignmentsAdditionalMgs = ['mg-Test', 'mg-PreProduction']
param parPolicyAssignmentIdentityRoleAssignmentsSubs = []
param parPolicyAssignmentIdentityRoleAssignmentsResourceGroups = []
param parPolicyAssignmentIdentityRoleDefinitionIds = []

Ah i see what this is now.
This module assigns a policy only to a single management group but it will allow you to assign a managed identity associated to the policy assignment for remediation to many scopes as this is a common requirement in ALZ.
So if you want the policy assigned to many management groups you need to call the module once for each assignment scope.
Hope that clears it up.


something is fishy with the assignments NUMBERING in Round 15 - part A

please, check the numbering of the assignments in Round 15 - part A

shall we try
 "html_url": "https://plus.cs.aalto.fi/studio_2/k2021dev/k14/osa03/k14_osa03_feedback/",

as a String to sort the exercises/assignments

shall we try
 "html_url": "https://plus.cs.aalto.fi/studio_2/k2021dev/k14/osa03/k14_osa03_feedback/",

as a String to sort the exercises/assignments

Possibly more robust solution: split using / as the separator, then sort by parts k14, osa03, k14_osa03_feedback


Unquote id

Some versions of gerrit return id quoted, when we quote it again we get
unexpected results.

Coverage remained the same at 100.0% when pulling 90c56bae39100f46eff0bee3ed16826dac9582bf on marhag87:unquote into f2879e7bfaad8cde3b12546097bbf70f3e6bd806 on propyless:master.


Test failure: test_stack_trace_decoding

Hello,
I am building an RPM for the shared library of aws-c-common on COPR. One test is failing.
        Start 235: test_stack_trace_decoding
234/380 Test #235: test_stack_trace_decoding ............................................***Failed    0.00 sec
/builddir/build/BUILD/aws-c-common-0.6.2/x86_64-redhat-linux-gnu/libaws-c-common.so.1(aws_backtrace+0x19) [0x7fddbc589e09]
/builddir/build/BUILD/aws-c-common-0.6.2/x86_64-redhat-linux-gnu/libaws-c-common.so.1(aws_backtrace_log+0x3e) [0x7fddbc58e06e]
/builddir/build/BUILD/aws-c-common-0.6.2/x86_64-redhat-linux-gnu/tests/aws-c-common-tests(+0x87f6b) [0x559d8e782f6b]
/builddir/build/BUILD/aws-c-common-0.6.2/x86_64-redhat-linux-gnu/tests/aws-c-common-tests(+0x854c2) [0x559d8e7804c2]
/builddir/build/BUILD/aws-c-common-0.6.2/x86_64-redhat-linux-gnu/tests/aws-c-common-tests(+0x13b93) [0x559d8e70eb93]
/lib64/libc.so.6(__libc_start_main+0xd5) [0x7fddbc398b75]
/builddir/build/BUILD/aws-c-common-0.6.2/x86_64-redhat-linux-gnu/tests/aws-c-common-tests(_start+0x2e) [0x559d8e70ed9e]
***FAILURE*** Expected non-null but got null:
***FAILURE*** strstr((const char *)buffer->buffer, func) [s_test_stack_trace_decoding():/builddir/build/BUILD/aws-c-common-0.6.2/tests/system_info_tests.c@#67]
***FAILURE*** test_stack_trace_decoding [ FAILED ] [s_aws_run_test_case():/builddir/build/BUILD/aws-c-common-0.6.2/include/aws/testing/aws_test_harness.h@#454]

The complete build log is currently available on COPR, but will vanish at some point in the future.
This test failed in the past in #356 and #456. Both issues have been fixed. In the latter one LTO was enabled, just as in this build.
I can provide more details, if needed.

Both issues have been fixed. In the latter one LTO was enabled, just as in this build.
I can provide more details, if needed.

Please make sure that you are using the latest version of all of repo's. If you still run into any tests failing please open a new issue


Icon within the button

It would look much better, if a set icon would be displayed within the clipboard button.

This is a library that we do not develop with the intention of creating a visual component, you can use it in any application you want. If you refer to the example, it is for demonstrative purposes only, feel free to modify it according to your needs.
Thanks for the suggestion anyway.


The RPC server is unavailable.

Hello,
I like this project very much. I don't know how to solve this problem. I hope you can help me.
look at it

As you can see, I opened the RPC service, but I still couldn't perform this operation.

Another problem is that my server uses redhunter, which prompts when I load pstools that the permissions aren't enough, and I've tried to modify the permissions, but it doesn't work.

If there is a systemic language problem, I think I was wrong from the beginning.

Look forward to your reply

If this issue still exists after upgrading to the most recent version of Caldera, feel free to re-open this issue or create a new one. Until then, this issue is being closed due to lack of activity.


[BUG] Flaky E2E tests

Describe the bug
E2E tests randomly failing most of the time.
Steps To Reproduce
See recent E2E builds.
Expected behavior
Green tests.

Trying to reproduce the flakiness on my environment, what I did was to insert a .only in the first test of the createWallet spec and then on the terminal run:
yarn run playwright test tests/e2e/001-createWallets.spec.ts

And sometimes the test would pass and some other times it would fail with a Timeout error:
yarn run v1.22.19
$ /home/fcz/github/lightning-browser-extension/node_modules/.bin/playwright test tests/e2e/001-createWallets.spec.ts

Running 1 test using 1 worker

  ✘  1 [chromium] › 001-createWallets.spec.ts:96:8 › Create or connect wallets › successfully creates an Alby wallet (30.0s)

  1) [chromium] › 001-createWallets.spec.ts:96:8 › Create or connect wallets › successfully creates an Alby wallet

    Test timeout of 30000ms exceeded.

  Slow test file: [chromium] › 001-createWallets.spec.ts (30.0s)
  Consider splitting slow test files to speed up parallel execution
  1 failed
    [chromium] › 001-createWallets.spec.ts:96:8 › Create or connect wallets › successfully creates an Alby wallet
error Command failed with exit code 1.

On one instance of this error, I noticed that there were a toaster/popup error message in the browser screen and I was able to screenshot it:


Replace deprecated arange with int_range

polars.arange was deprecated in Polars v0.18.5. This switches to polars.int_range

Codecov Report

Merging #31 (82835ea) into master (7f8f060) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   86.69%   86.69%
=======================================
  Files          15       15
  Lines         887      887
  Branches      150      150
=======================================
  Hits          769      769
  Misses        105      105
  Partials       13       13

Impacted Files
Coverage Δ

src/tabeline/_data_frame.py
87.74% <ø> (ø)

src/tabeline/_expression/_functions.py
62.96% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fixed socket disconnecting about every 2 minutes

When calling blynk.run(), if no data is received from the server for a while, we would never process the ping messages (heartbeat) which would cause the socket to drop after about 2 minutes.
Calling process() on socket.timeout exceptions will send the heartbeat at the configured frequency and keep the socket connected.

@vshymanskyy please merge this, it's needed, otherwise the current library is pretty useless. Also, please publish a new version so it can be installed via pip.


flounder rebuild: if props are passed, not only the passed are changed

Current
now the flounder rebuild api method also resets props that are not passed as arguments:
E.g.
var flounder = new Flounder(<yourselect>, {data: [1,2,3], selectDataOverride: true} );
flounder.rebuild([1,2], {onSelect: function(){console.log('selected')}});

yields:
flounder.selectDataOverride is reset to false.
Wanted
only the passed props should be altered. at least that's my request/suggestion/what i expected when using the api.

given that the api method is called .rebuild, my feeling is that the data object you pass is the new data object.  I can leave this open for a couple days for discussion.

@mousemke hmm.. discussable. I was just hoping ONLY the props you pass as 2nd argument are changed, not the ones not specified. But one could also have your pov and treat the passed props as the new props (analog to on instantiation), meaning all missing props are set to default. Hmmm..  discussable

i think rebuild should stay how it is.  we could add another api function that took the passed props and extended the original props before passing it to rebuild

given that the api method is called .rebuild, my feeling is that the data object you pass is the new data object. I can leave this open for a couple days for discussion.

@kevin-krug-sociomantic I agree with @mousemke, I cannot see a reason for rebuild to be "incremental", otherwise it would have been called reconfigure. It makes sense that the developer is fully responsible for holding the complete set of options for later use, and not use Flounder as surrogate for that.

fair enough. thanks for contributing to the discussion @tar-viturawong-sociomantic


Protection against current code-block removing

Hi, this should go after previous PRs. We don't need getIndentation.js anymore.
When we are at the beginning of line at our code-block we can remove it by making attempt to go back to previous indent. Here we are checking the current indent and if it zero and this is the first line of the code-block, we do nothing.

Coverage decreased (-12.2%) to 87.075% when pulling 6d75d13d8da61f47eb7fdeb6b414ac724140f00d on newsiberian:protect-against-code-block-removing into c219dc50d42f7db9c69b4b8c9be4b437116e8e6a on SamyPesse:master.

If I understand this correctly, when you're in an empty code block with the cursor at the first line, i.e. like so: (where | is the cursor position)
|

Rather than trying to remove the (non-existant) indentation we just let DraftJS handle it instead? What's not working about this right now?

You can try it here with backspace
If you try to put cursor into position you mentioned (even if text presents in code-block) and press backspace it will remove code-block.

Ohhh I see, that's annoying, great catch! Let's get some tests in here to make sure this doesn't happen again accidentally, and then let's ship it :rocket:

I'll try to add tests tomorrow

Awesome, thank you! There's a bunch of tests already, you can just model them after that 👌

Coverage decreased (-0.6%) to 98.701% when pulling d4796f778e4bb22f07e758ef18bac06d01f2ca04 on newsiberian:protect-against-code-block-removing into c219dc50d42f7db9c69b4b8c9be4b437116e8e6a on SamyPesse:master.


Uncaught SyntaxError: The requested module /service-worker.js does not provide an export named 'default'

After setting up step 3 of the PWA training and fill up service-worker.js we have this error in the console.
Uncaught SyntaxError: The requested module /service-worker.js does not provide an export named 'default'

Any idea how to fix this ?

Any updates after two years? I have got the same error.

In my case I was following the codelab in spanish and it had this line in the main.js file:
import swURL from 'sw:../service-worker.js';
and below it says this:
The import syntax for the service worker URL is required so that our build tool, WMR, can build the service worker correctly. If you don't use WMR, set swURL as the path to your service worker.
So I changed the import line to this and is working now 👍 :
const swURL = '../service-worker.js';


Difficult to get a working config

I'm sharing mine that finally work for ingestion.
The ui was working out of the box, but It was quite difficult to understand how to setup the ingestion url/port, given that the nginx controller listens only on ports 80/443.
Serilog config
        .WriteTo.Seq("https://in.seq.yourdomain.eu", apiKey: "xxxxxxx")

To apply:
helm repo add datalust https://helm.datalust.co
helm repo update
helm upgrade seq datalust/seq -n yournamespace -f seq.yourdomain.helm.yaml --install --dry-run

values.yaml
baseURI: https://seq.yourdomain.eu/

ingress:
  className: nginx
  annotations:
    cert-manager.io/cluster-issuer: letsencrypt-prod
  tls:
  - hosts:
    - seq.yourdomain.eu
    - in.seq.yourdomain.eu
    secretName: seq

persistence:
  size: 2Gi
  existingClaim: seq-yourdomain

ui:
  ingress:
    enabled: true
    path: /
    hosts:
      - seq.yourdomain.eu
    annotations:
      cert-manager.io/cluster-issuer: letsencrypt-prod

ingestion:
  ingress:
    #nginx controller only supports ports 80/443
    enabled: true
    path: /
    hosts:
      - in.seq.yourdomain.eu
    annotations:
      nginx.ingress.kubernetes.io/backend-protocol: "TCP"
      nginx.ingress.kubernetes.io/proxy-buffering: "on"
      nginx.ingress.kubernetes.io/proxy-buffer-size: "128k"
      nginx.ingress.kubernetes.io/proxy-buffers-number: "4"

Thanks for sharing this @softlion 🙏


LIbfranka and Server Incompability

Hi,
I have recently updated FCI to version 2.1.4, updated my OS to Ubuntu 18.04, installed ROS Melodic and libfranka (master branch). When I launch franka_control node from franka_ros it works, fine but when I try to launch franka_gripper node, I get the following error:
terminate called after throwing an instance of 'franka::IncompatibleVersionException'
  what():  libfranka: Incompatible library version (server version: 2, library version: 3). Please check https://frankaemika.github.io for Panda system updates or use a different version of libfranka.

As you can see, the exception states that libfranka and server versions are not compatible but as I said before both are up to date.
What might be the problem?
Kind regards,

For system updates compatible with libfranka >= 0.6.0 or the Schmalz Cobot Pump, please contact support. You already did that, so I'll close this issue here.

I've tried to install from source the 0.8.0 version - ./examples/echo_robot_state works fine indicating that libfranka is good. However the source build of franka_ros fails when trying to use franka_control.launch with
*** Error in `/home/mark.robson/catkin_ws/devel/lib/franka_control/franka_control_node': malloc(): memory corruption: 0x00000000010f8450 ***
Ideally i would like to revert back my hardware to a working state (server version 3) until binaries are available through apt-get repository. is this possible?

I assume you use franka_ros develop branch? Can you try using franka_ros commit '406e118c143cbefb58c2db8b6a5da87345dd1949' (git checkout 406e118c143cbefb58c2db8b6a5da87345dd1949) or franka_ros 0.6.0 and retest your example? Regarding downgrades, please contact<EMAIL_ADDRESS>for more information.

I'm getting the same memory allocation error with the " git checkout 406e118c143cbefb58c2db8b6a5da87345dd1949 " commit also, franka_visualization works but not franka_control.

Hello
I have installed libfranka 0.8.0 from the repository but still getting following error.
libfranka: Incompatible library version (server version: 3, library version: 4). Please check https://frankaemika.github.io for Panda system updates or use a different version of libfranka.
Please help and guide

In this case, the libfranka version is newer than the robot version. You can either downgrade libfranka to 0.7.1 by building from source or upgrade your robot's system version to version 4 (see https://frankaemika.github.io/).

Initially I had the error of
libfranka: Incompatible library version (server version: 3, library version: 4). Please check https://frankaemika.github.io for Panda system updates or use a different version of libfranka.
Then I tried to downgrade the libfranka version by removing libfranka(sudo apt remove) and then started building from source the version 0.7.1 (by got checkout 0.7.1). After cmake, now when I try to run rosrun libfranka echo_robot_state, the error is: package 'libfranka' not found.
Can anyone help me?

libfranka is not a ros package, but a normal library. if you installed it with sudo make install inside the build folder, you can run echo_robot_state <robot-ip>

@marcbone  is correct you need franka_ros package to run with Ros. Libfranka is c++ library. Find out more about franka_ros here https://frankaemika.github.io/docs/franka_ros.html


SQL not working?

See section "Now you can start setting up the Foreign Data Wrapper" here https://github.com/jjb/holycorn/tree/patch-1
And I'll try it right here as well...
DROP EXTENSION holycorn CASCADE;
CREATE EXTENSION holycorn;
CREATE SERVER holycorn_server FOREIGN DATA WRAPPER holycorn;
CREATE FOREIGN TABLE holytable (some_date timestampz) \
  SERVER holycorn_server
  OPTIONS (wrapper_path '/tmp/source.rb');

It looks like this is because of the EXTENSION keyword.
However, Linguist only selects the appropriate grammar. It uses textmate/sql.tmbundle for SQL code. You should open an issue there ;)

You should open an issue there ;)

:+1:


Troubleshoot DTR overlay network

Teach users how to test if the overlay network is working and other helpful troubleshooting tips and tricks.

We have one thing about overlay networks right now: how to do the ping test. We can definitely do better, but we would need help from core to build better debug tools.

Talking about the ping test would be an improvement because we don't really have anything and some of the issues we're seeing are related to overlay networks.

Yep. That's already in troubleshoot-with-logs.md

Solved.


Fixed #7164 -- Irrelevant layers are not saved.

This fixes #7164 by ignoring (in get_config) input and output layers that are not used by the model. This is consistent with the behavior we use for intermediary layers.
I also tried improving the readability of this part of the code (second commit) that makes no semantic change.

That's reasonable, but your code should respect the codebase style, in particular:

docstring format
reasonably sized lines

Made the additions here: https://github.com/joeyearsley/keras/tree/issue7164
Should I add a new PR?

@fchollet, I modified the last commit to include those changes and rebased the whole PR against master. Should be ready to merge.
Thanks @joeyearsley for pointing out what is meant with "reasonably sized lines". :)

@jorgecarleitao No problem, usually I have my IDE use a line length of 80 before wrapping, I think this is in PEP8 somewhere, I always forget were exactly.

All comments fixed, squashed commits and rebased against master. It should be ready to go.


Logical error in movement of certain pieces

This problem is faced by pieces which can move diagonally mainly bishop and queen.
You have used src-dest %7 or %9 to check if move is possible but it doesn't work on every case.
Assume you have a bishop which can move only on white squares on index 61. There is a black square on tile 40. 61-40 %7 is 0 so according to this the bishop jumps there randomly.
Same for queen and pawns at the end of the file
Another problem I noticed is that a 2 step moving pawn cuts anything in its destination. You should have added isDestEnemyOccupied for moving 2 paces. i guess this is the chess rule.
To make it error free you must have an array of size 120 and assign first 2 rows and the last 2 rows and the side rows as offboard and check if the path to the destination passes through the offboard if so return false.
I've added images to help you understand the problem and solution

@Vikram710 thanks for bringing this up. I'll try to take a look soon. But if you could add these improvements and send a PR, that would be great.

Hello,
I want to point one more thing. The king can't move from a position of check to another where there is still a check.

Lol i don't think he has implemented check detection at all

@Vikram710 true.
@AyoubEd you could move a piece such that the king is exposed to check. This of course shouldn't happen. I mentioned that in readme under to dos for improvement.
The code is three years old, so I'd like to bring the React and other packages to the newest versions first. Then fix these logical errors. Hopefully soon!

@Vikram710 I have fixed the logical error you pointed out.  It was happening nearly for all pieces. Used row, col, and diagonal dictionaries to ensure no illogical jumping. Thanks for bringing it up.


Refactor ResumableOperation and ResumableOperationQueue

Builds off of PR #262, which should be merged first.

Removes transfer(_:didUpdateWithState:) since this is covered by the existing transfer(_:didUpdateWithState:andProgress) call.
Adds a transfersDidUpdate(_) method for TransferDelegate which is analogous to the operations(_:didUpdateWithState) method of the now removed ResumableOperationQueueDelegate. Useful when a bunch of transfers change at once and the UI should respond.
Renamed ResumableOperation to TransferOperation and shrunk the implementation.
Renamed ResumableOperationQueue to TransferOperationQueue and greatly reduced the implementation, retaining only a couple helper methods.

The goal here is to reduce the layers of indirection in trying to maintain a pure, standalone ResumableOperationQueue to make diagnosing the CoreData issues easier.

@bsiegel comments addressed


Implements SequenceType, can be used for for-in loop

Interesting; I’m not sure I understand the value of this relative to [ box.value ], however. Could you describe the intended use case?

Oh, I didn't think about [ box.value ]. This is basically from my interest, Box seems a collection which have just one element.
And sometimes I want to use the unboxed value multiple times, but store it to a temporary constant is a bit tiresome. Using for-in loop is preferable for me for that use case.
let value = box.value
if predicate(value) {
    someMethodWithValue(value)
}

// or

for value in box {
    if predicate(value) {
        someMethodWithValue(value)
    }
}

Ah, ok. I think this is an interesting use case, but it’s not something I want to support going forward. That said, thank you for the PR!

Thanks for your consideration. :sweat_smile:


Add CaseNode#branches

I was surprised that IfNode#branches returns a array with a single item for if foo; else; end. I was expecting [nil, nil], not just [nil].
I mimicked this behavior for CaseNode as it seems natural to have this symmetry, but I wonder which is best. Anyone has an idea? cc/ @fatkodima

Symmetry is good, but it seems to me also that it should return [nil, nil] instead of just [nil] for provided example.

There are different possibilities for IfNode#branches

always return the last else branch, even if it is only implicit:

if foo; end # => [nil, nil]

return the last else branch only if there's an explicit keyword

if foo; end # => [nil]
if foo; else; end # => [nil, nil]

return the last else branch only if there keyword and some code in there [current]

if foo; else; end # => [nil]
if foo; else; 42; end # => [nil, s(:int, 42)]

I like #3 the least.
The same question exist for case and I'd like the answer to be the same... @fatkodima you prefer #2 or #1

No. 2 seems the most logical to me.

Thank you :-)
I made the tweak, and not a single spec fails, in rubocop-ast or rubocop... I put this as "potentially breaking" in the Changelog.


[Suggestion/Question]   is there a way to make Light Overlay turn on by default when game starts ?

Light Overlay
Default on = true/false

There is currently no way to do that.
I also don't really see why someone would need that since it's normally only enabled as long as you are building something and not constantly.
I will try to add that, but it will probably end up in the Mod Options / Config file.

I would like this option too. I care a lot about my skyblock safety, and having to check whether it's turned on or not to be able to distinguish it being disabled from actually no spawnable places available, is tedious and can be forgotten. I've lost items due to this. The player should be able to decide whether they want it turned on by default really.


Create the validate schemas command

User Story
As a report stream admin, I would like a CLI command that can be used to validate all the schemas
Description/Use Case
Details on how this command should work can be found here, but at a high level this will be a tool used locally to validate changes and then consumed by a new CI step
Risks/Impacts/Considerations
Dev Notes
This will build on top of the new service method to validate schemas
Acceptance Criteria

[ ] A CLI command validateSchemas exists that can be run against local, staging or production

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @brick-green @jack-h-wang @jalbinson @jimmyfagan @mkalish @thetaurean


Make a simple grid/flex Box UI With buttons

Grid Layout: The game board is displayed as a grid of cells, making it easy to navigate and uncover.
Dynamic Sizing: The grid adjusts to fit the screen size, ensuring an optimal playing experience on any device.

The game board is displayed as a grid of cells, providing a structured and organized layout for the player. This grid-based design makes it easy to navigate through the game board and uncover cells, enhancing the overall gameplay experience.
The grid adapts its size and layout to fit the screen size of the device being used. Whether you're playing on a desktop computer, a tablet, or a smartphone, the game board adjusts dynamically to ensure an optimal playing experience. This responsive design allows players to enjoy Minesweeper comfortably on any device, without sacrificing usability or visual appeal.


3.7+ compatible code

Fixes #111

Codecov Report

Merging #114 (620ec45) into main (9cff00f) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
- Coverage   88.93%   88.83%   -0.10%
==========================================
  Files          11       11
  Lines        2069     2069
==========================================
- Hits         1840     1838       -2
- Misses        229      231       +2

Impacted Files
Coverage Δ

src/picologging/__init__.py
92.69% <100.00%> (ø)

src/picologging/handlers.py
94.02% <0.00%> (-0.46%)
:arrow_down:

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


Add ephemeral onion service functionality to controller API

here we also:

add a TorDialer and OnionListener
complete working example of an onion echo server

How about something like: https://github.com/Yawning/bulb/compare/add_onion
I know I said to jam the convenience stuff under "utils", but the way it's done in my branch is probably ok...


Apply clipping in save_image

Without clipping, any values larger than 255 will be replaced with
int(v mod 256) by byte(), which results in high freq noise in image.

thanks!


About the config: --scene_path /your_root/hdf5/ADE.h5

Hi !
I want to test on the pre-trained model, but now I'm confused about the config:--scene_path /your_root/hdf5/ADE.h5, could you please tell me what the ADE.h5 is and how can I gain this file?
Thank you very much!

Hi,
The ADE.h5 file just stores the paths to the scene images in the ADE dataset. A sample file that I used is now shared in this folder:  https://drive.google.com/drive/folders/1K9DdSu7fK0rVmVbdvTgulOwDQHzxbBsK?usp=sharing
Best,
Ruohan

Hi,
The ADE.h5 file just stores the paths to the scene images in the ADE dataset. A sample file that I used is now shared in this folder: https://drive.google.com/drive/folders/1K9DdSu7fK0rVmVbdvTgulOwDQHzxbBsK?usp=sharing
Best,
Ruohan

Hi !
Thanks for your kind reply and excellent work!!
Now I'm trying to train your model, but I get the error:
OSError: Unable to open file (unable to open file: name = 'hdf5/soloduet/train.h5', errno = 2, error message = 'No such file or directory', flags = 0, o_flags = 0)
Could you please tell me what the file train.h5 is? If possible, could you share this file.
Thank you so much again!!!

Hi,
The hdf5 file only contains the relative path to the object detection results. Please see audioVisual_dataset.py L88-90  for how I load the data. You need to prepare your own hdf5 file based on your data to train the model.
I have just shared the sample hdf5 files for your reference in: https://drive.google.com/drive/folders/1K9DdSu7fK0rVmVbdvTgulOwDQHzxbBsK?usp=sharing
Hope that helps.

Hi,
The hdf5 file only contains the relative path to the object detection results. Please see audioVisual_dataset.py L88-90 for how I load the data. You need to prepare your own hdf5 file based on your data to train the model.
I have just shared the sample hdf5 files for your reference in: https://drive.google.com/drive/folders/1K9DdSu7fK0rVmVbdvTgulOwDQHzxbBsK?usp=sharing
Hope that helps.

It really helps a lot, thank you for your patient guidance ^^


LAB_11-Implement_Monitoring.md

English:
Review
Current Translation:
Tinjau
Suggested Translation:
Tinjauan

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run the bugfix tool and waiting for result. ICMS QUERY

This issue has been fixed, so I close the PR.


Add Eyeglass support

I'm pretty excited about Eyeglass, as I'm hoping it can streamline the management of Sass libraries, by basically making @import as seamless as Node's require().
Eyeglass 1.0 isn't out yet, but I'd like to make Foundation for Sites and Motion UI available as Eyeglass modules.

Implemented in the 6.2 branch: 25e917732e92ea226c61ba4642e982649edea6cb


Custom, dynamic themes

Hello. Love your library and am integrating it into my new project. Thank you for your hard work.
I am wondering what you think would be the bst way to go about creating themes dynamically, for custom dice applications?
For instance, given an array of 6 emoji, i'd like to somehow configure the dicebox such that the dice will show one of each emoji in the dice faces.
The theming system seems a bit imposing, hopefully ther'es a way to do it?

For my purposes, a theme that exclusively supports d6 where I can dynamically provide images or text for each of the faces is what I am lookign to build. I am a seasoned engineer, so any ideas or a few pointers would be appreciated and I might be able to implement and contribute back.

Hi @wtfcarlos. Unfortunately, the theme system is not set up for dynamic face values. I use blender to create the 3d models and bake the textures. I know BabylonJS has support for custom fragment shaders, which I have implemented to dynamically alter the die colors, but altering the die faces was much more complex. These are written in Graphics Library Shader Language (GLSL) which I do not know well enough.
My other dice roller dice-box-threejs should support this feature though. It's a fork of Major's 3D Dice which was forked from Teal Dice (no longer online but mirrored here Darkside Teal Dice). You can see in the dice presets file how the labels and values are configurable.
I know other libraries do support this feature, such as dddice.com and Dice so Nice. Dice So Nice was originally forked from Major's 3D Dice as well but has been heavily altered to integrate with FoundryVTT.
Overall, this seems like a popular feature, so I'm going to add it to the list for the future.

Digging deeper into dice-box-three, the ability to make a custom set or assign the face labels is not exposed, but I imagine it wouldn't take much work to expose them as a config option. Color, texture and material have already be exposed as a custom color set.

Hello, I was able to implement this for d4 only on the mainstream 3D-dice box.
The way I did it was by supporting a new theme type that I call dynamic-canvas.
I set it up in such a way that I can provide an array of base64 strings called canvasProvider that would represent the face of each die.
I then use Babylon's CubeTexture to create a compossite custom cube texture that I can apply to a custom material.
Below is the quick and dirty version I came up with, which works in my project with some accessory modifications in other parts of the code:
class ThemeLoader {
  loadedThemes = {};
  themeData = {};
  constructor(options) {
    console.log("constructor options");
    console.log(options);
    this.scene = options.scene;
    this.canvasProvider = options.canvasProvider;
  }

  async createCubeTextureFromBase64(base64Array) {
    if (base64Array.length !== 6) {
      throw new Error(
        "CubeTexture requires exactly 6 images (one for each face)",
      );
    }

    return new CubeTexture(
      "", // Root URL (we don't need this as we are setting the images manually)
      this.scene,
      null,
      false,
      this.canvasProvider,
    );
  }

  async dataURLToBlob(dataURL) {
    const byteString = atob(dataURL.split(",")[1]);
    const mimeString = dataURL.split(",")[0].split(":")[1].split(";")[0];

    const ab = new ArrayBuffer(byteString.length);
    const ia = new Uint8Array(ab);
    for (let i = 0; i < byteString.length; i++) {
      ia[i] = byteString.charCodeAt(i);
    }

    return new Blob([ab], { type: mimeString });
  }

  async loadDynamicCanvasMaterial(options) {
    const { theme } = options;

    const cubeTexture = await this.createCubeTextureFromBase64(
      this.canvasProvider,
    );

    const material = new StandardMaterial(theme, this.scene);
    material.diffuseTexture = cubeTexture;
    material.diffuseTexture.level = 1;
    material.diffuseTexture.coordinatesMode = Texture.SKYBOX_MODE;
    material.diffuseColor = new Color3(0, 0, 0);

    material.specularPower = 1;
    material.specularColor = new Color3(0, 0, 0); // Use a lower value to reduce brightness

    material.reflectionTexture = cubeTexture; // Or set this as diffuseTexture based on your needs
    material.reflectionTexture.coordinatesMode = Texture.SKYBOX_MODE;
    material.reflectionTexture.level = 1;

    material.backFaceCulling = false;
    material.sideOrientation = StandardMaterial.ClockWiseSideOrientation;

    return material;
  }

  }```

This video shows how it works for my project.
https://github.com/user-attachments/assets/3c5fc124-4fc5-426e-8edf-deb389722bee
This is obviously a hack that only supports d6, but this is  just exactly what I need for my project.
My project will continue using this library heavily, and I can foresee needing more custom adjustments, but I think I'm confident I can modify the library to do anythign I need for my basic d6 purposes

The theme file config I'm using declares a simple cubic mesh d6 and a simple cubic d6_collider, with simple UV mappings:
{
  "producer": {
    "name": "Carlos Rey",
    "version": "0.0.1",
  },
  "gravity": [
    0,
    -9.81,
    0
  ],
  "colliderFaceMap": {
        "d6": {
            "0": 1,
            "1": 5,
            "2": 4,
            "3": 2,
            "4": 6,
            "5": 3,
            "6": 1,
            "7": 5,
            "8": 4,
            "9": 2,
            "10": 6,
            "11": 3
        }
  },
  "meshes": [
    {
      "name": "d6",
      "id": "d6",
      "billboardMode": 0,
      "position": [-0.6, 0, 0],
      "rotationQuaternion": [0, 0, 0, -1],
      "scaling": [1, 1, 1],
      "isVisible": true,
      "isEnabled": true,
      "pickable": false,
      "positions": [
        -1, -1, -1,
         1, -1, -1,
         1,  1, -1,
        -1,  1, -1,
        -1, -1,  1,
         1, -1,  1,
         1,  1,  1,
        -1,  1,  1
      ],
      "normals": [
        0, 0, -1,
        0, 0, -1,
        0, 0,  1,
        0, 0,  1,
        0, -1, 0,
        0, -1, 0,
        0,  1, 0,
        0,  1, 0,
        -1, 0, 0,
        -1, 0, 0,
        1, 0, 0,
        1, 0, 0
      ],
        "uvs": [
          0, 0, 0.16666666666666666, 1,
          0.16666666666666666, 0, 0.3333333333333333, 1,
          0.3333333333333333, 0, 0.5, 1,
          0.5, 0, 0.6666666666666666, 1,
          0.6666666666666666, 0, 0.8333333333333334, 1,
          0.8333333333333334, 0, 1, 1
        ],
      "indices": [
        0, 1, 2, 0, 2, 3,
        4, 5, 6, 4, 6, 7,
        0, 1, 5, 0, 5, 4,
        2, 3, 7, 2, 7, 6,
        0, 3, 7, 0, 7, 4,
        1, 2, 6, 1, 6, 5
      ],
      "metadata": {
        "formatVersion": 3
      },
      "subMeshes": [
        {
          "materialIndex": 0,
          "verticesStart": 0,
          "verticesCount": 8,
          "indexStart": 0,
          "indexCount": 6
        },
        {
          "materialIndex": 1,
          "verticesStart": 0,
          "verticesCount": 8,
          "indexStart": 6,
          "indexCount": 6
        },
        {
          "materialIndex": 2,
          "verticesStart": 0,
          "verticesCount": 8,
          "indexStart": 12,
          "indexCount": 6
        },
        {
          "materialIndex": 3,
          "verticesStart": 0,
          "verticesCount": 8,
          "indexStart": 18,
          "indexCount": 6
        },
        {
          "materialIndex": 4,
          "verticesStart": 0,
          "verticesCount": 8,
          "indexStart": 24,
          "indexCount": 6
        },
        {
          "materialIndex": 5,
          "verticesStart": 0,
          "verticesCount": 8,
          "indexStart": 30,
          "indexCount": 6
        }
      ]
    },
    {
      "name": "d6_collider",
      "id": "d6_collider",
      "billboardMode": 0,
      "position": [-0.6, 0, 0],
      "rotationQuaternion": [0, 0, 0, -1],
      "scaling": [1, 1, 1],
      "isVisible": true,
      "isEnabled": true,
      "pickable": false,
      "positions": [
        -1, -1, -1,
         1, -1, -1,
         1,  1, -1,
        -1,  1, -1,
        -1, -1,  1,
         1, -1,  1,
         1,  1,  1,
        -1,  1,  1
      ],
      "normals": [
        0, 0, -1,
        0, 0, -1,
        0, 0,  1,
        0, 0,  1,
        0, -1, 0,
        0, -1, 0,
        0,  1, 0,
        0,  1, 0,
        -1, 0, 0,
        -1, 0, 0,
        1, 0, 0,
        1, 0, 0
      ],
        "tangents": [
            1, 0, 0,
            1, 0, 0,
            1, 0, 0,
            1, 0, 0,

            -1, 0, 0,
            -1, 0, 0,
            -1, 0, 0,
            -1, 0, 0,

            0, 0, -1,
            0, 0, -1,
            0, 0, -1,
            0, 0, -1,

            0, 0, 1,
            0, 0, 1,
            0, 0, 1,
            0, 0, 1,

            1, 0, 0,
            1, 0, 0,
            1, 0, 0,
            1, 0, 0,

            1, 0, 0,
            1, 0, 0,
            1, 0, 0,
            1, 0, 0
        ],
      "uvs": [
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1,
        0, 0, 1, 0, 1, 1, 0, 1
      ],
      "indices": [
        0, 1, 2, 0, 2, 3,
        4, 5, 6, 4, 6, 7,
        0, 1, 5, 0, 5, 4,
        2, 3, 7, 2, 7, 6,
        0, 3, 7, 0, 7, 4,
        1, 2, 6, 1, 6, 5
      ],
      "metadata": {
        "formatVersion": 3
      }
    }
  ]
}


Add retry dialog in onGitpodRemoteConnection

@jeanp413 👍
I'd like to remove last trigger event for sendHeartbeat in this PR, since the different event name will trigger different ErrorReporting.
Thank you

catch (e) {
	if (e instanceof NoRunningInstanceError) {
		this.logger.error('No Running instance:', e);
		return;
	}
}

This one should included right? but with different dialog: workspace is not running and redirect to browser like Restart Workspace dis

Test restart dialog with change

Results

Retry test code

Result

Nice.


Could not find me in prediction for weekly contest

Hi, my account name is zhzho.
I could not find my account in prediction for weekly contest,  but I could find me in bi-weekly prediction.
Could you please investigate ?
My global rank is 389  in weekly 319 contest. Chinese Nickname: 最佳歌手, account: zhzho
But I am not able to find me in https://lcpredictor.herokuapp.com/contest/weekly-contest-319/ranking/1

Related:
https://github.com/SysSn13/leetcode-rating-predictor/issues/61#issuecomment-1345444008

@zhzho1995 This has been fixed now. Leetcode.com doesn't show CN region users on rankings pages. We were relying on leetcode.com only for fetching the contest ratings.


Updating a pipeline in Data Factory on a different branch than main

I need to update pipeline settings in Data Factory on a dev branch. At the moment, my code can read from main but how do you change branches in the code before you attempt make an update. Any attempt to update the code fails. I am guessing because it doesn't know which branch to update.

@azure-walker - Thanks for your question! To confirm: Are you running into this issue when using the azure-mgmt-datafactory SDK?

Yes. the latest version 2.4.0

Hi @azure-walker, thanks for your feedback. After my investigation, it seems that the rest API does not support submitting branch modifications to the unexpected branch of the collaborationbranch. The definition of collaborationbranch is determined when creating datafactory, and modification is not supported.
I got the same tips on Azure Portal.

An alternative method is to update the branch through Git SDK. Or you could open a new issue here, describe the feature, let the rest API support the modification, and then we will generate and release a new version of SDK. Thanks!


Set up login flow/user auth (WIP)

Description:
Tests Performed: tested on android
Trello Card Link: https://trello.com/c/1grF8dyU/14-setup-login-flow-user-auth

this is a draft (working on rewrite using riverpod), but in the meantime, any comments/reviews r still appreciated


Custom error text is no longer displayed in Veasy 1.7.1

Using Veasy 1.7.1, I updated your example formSchema in app.js to:
title: { minLength: [3, 'Min length is 3'] },
When I click in the field the first time I see my error text. But after I start typing I see the default error text provided by Veasy:

title's length should be equal or greater than 3. Current: 1

Are you able to reproduce this bug?


Compatibility with cstruct v4

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.10%. Comparing base (b28a604) to head (a8337a9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   79.10%   79.10%
=======================================
  Files          15       15
  Lines        1273     1273
=======================================
  Hits         1007     1007
  Misses        266      266

Flag
Coverage Δ

unittests
79.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

No cleanup of the loading of c_esedb in c_esedb.py?

Oversight, fixed it.


singularity file binds leaves target files/paths inside the container

Thanks a lot for implementing file binds in S1 mode in devel!
I can see that target files (and directories) are not being deleted after the binding process in the container is being terminated. I think this should be possible, am I right?
Udocker just needs to keep track on what it had created initially.

That's intentional. checking and removing the mountpoints is not an atomic a operation.
If a container is being executed in another cluster host we can get trouble if one instance is in the middle of removing while other is in the middle of mounting.
The mountpoints are removed when you convert to another execution mode or with:
udocker setup --purge

Thanks for the explanation! Closing therefore...


Check for errors in regex within string literals at compilation

This is the first part for #1511.

+1

+1, I like the error, less bugs at runtime seems good...


Warning when compilng

I have this warning, several times when compiling. The code however works  (I use it embedded in Cloak).
warning: unknown warning option '-Wno-format-truncation' [-Wunknown-warning-option

Apologies for not responding earlier - difficult to find time for open-source work. The warning you are seeing is harmless - it is just saying that the compiler does not know about that option. It does not affect the compilation.


fixing creating relations twice on each model

https://github.com/Kononnable/typeorm-model-generator/issues/89

Yes, i will try to provide it.

Would the schema I provided in #101 work for this? In addition to the foreign key issue which you spotted, it did duplicate relations in lots of models.
Tomorrow I'll test this PR once I've removed the self-referential foreign keys from our schema.

Closing as no schema was provided.
Original error might be connected to #130


Maintain argument order in the schema.

Fields maintain their original schema order in both the docs and the autocomplete suggestions (#884). But arguments are sorted alphabetically.
As with fields, although the order doesn't matter functionally it still indicates importance, grouping, etc. Also it looks inconsistent since input fields are in schema order.

Hi,
Can I have a try on this issue?

@ritikBhandari sure! getAutocompleteSuggestions is where you would start in the language service

thank you @acao, I'll start it right away!

Looks like this is already done in recent versions.


Add ability to disable native automation at the fixture level

What is your Scenario?
There are still several bugs that prevent me from using native automation in every test (iframe support is poor at the moment). I would like a way to disable native automation at the fixture level
What are you suggesting?
https://github.com/DevExpress/testcafe/pull/8021/files
If we can follow the same pattern as the above PR to disable native automation that would be perfect.
What alternatives have you considered?
I currently have my own custom testcafe test runner where I can parse the meta tag and dynamically turn off native automation. This works great when I use the runner but does not work if I just use the CLI (testcafe test runner in vscode). It would be great if I could simply do something like .fixture.disableNativeAutomation. Otherwise people on my team need to manually change the disable flag depending on what test file they're running.
Additional context
No response

Hello @Makavelic ,
Thank you for the feedback.

I currently have my own custom testcafe test runner where I can parse the meta tag and dynamically turn off native automation

Please share your TestCafe test runner code. We will examine it. This will help us better understand your usage scenario.
The Native Automation mode support is our priority. So, it would be very helpful if you share a project sample that demonstrates iframe support issues you are facing with NA mode.

Hello @Makavelic ,
Thank you for the explanation.
We researched the possibility of disabling the Native Automation mode at the level of a test or a fixture and it's turned out that it's a very complex task that requires changing TestCafe's runner architecture. We will take your feedback into account. But for now we have other priorities e.g. Native Automation mode improvements.
As a workaround, we suggest that you split your tests into two different tasks: one where the NA mode is enabled and the other where the NA mode is disabled.


Deprecation of assertPass function

Description
We would like to deprecate the assertPass function with a warning level for using directly the given block instead.
The deprecation notice should also indicate that it will be an error in the next minor release (see issue #52).
Dependencies
This issue is blocked by the following ones:

[x] #35

Checklist

[ ] Deprecate the assertPass function with a warning level and refactor its usages.
[ ] Update the Unreleased section in changelog.

The Kotools Assert project will be archived.
We recommend to use the testing library provided by Kotlin instead.


Uncaught TypeError: Cannot read property 'split' of undefined

[Enter steps to reproduce:]

have un-typed alias at the top of a module

Atom: 1.32.2 x64
Electron: 2.0.9
OS: Mac OS X 10.14.1
Thrown From: elmjutsu package 9.9.1
Stack Trace
Uncaught TypeError: Cannot read property 'split' of undefined
At /Users/drunkengranite/.atom/packages/elmjutsu/lib/elm-make-runner.js:439

TypeError: Cannot read property 'split' of undefined
    at /packages/elmjutsu/lib/elm-make-runner.js:439:31
    at Array.map (<anonymous>)
    at /packages/elmjutsu/lib/elm-make-runner.js:437:65
    at /packages/elmjutsu/lib/eval-helper.js:49:7)
    at emitTwo (events.js:126:13)
    at ChildProcess.emit (events.js:214:7)
    at maybeClose (internal/child_process.js:925:16)
    at Socket.stream.socket.on (internal/child_process.js:346:11)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at Pipe._handle.close [as _onclose] (net.js:554:12)

Commands
     -0:32.7.0 core:save (input.hidden-input)
     -0:30.7.0 core:move-down (input.hidden-input)
     -0:30.5.0 intentions:highlight (input.hidden-input)
  3x -0:30.4.0 editor:move-to-end-of-word (input.hidden-input)
     -0:29.7.0 editor:move-to-beginning-of-word (input.hidden-input)
     -0:29.3.0 editor:delete-to-beginning-of-word (input.hidden-input)
     -0:28.9.0 datatip:toggle (input.hidden-input)
     -0:28.4.0 core:backspace (input.hidden-input)
     -0:27.9.0 intentions:highlight (input.hidden-input)
  2x -0:27.8.0 editor:move-to-beginning-of-word (input.hidden-input)
     -0:27.4.0 datatip:toggle (input.hidden-input)
  2x -0:27 core:move-up (input.hidden-input)
     -0:23 core:paste (input.hidden-input)
  2x -0:22.5.0 core:save (input.hidden-input)
     -0:07.2.0 autocomplete-plus:activate (atom-text-editor.editor.is-focused)
     -0:03.4.0 core:save (input.hidden-input)

Non-Core Packages
atom-ide-ui 0.13.0
autocomplete 0.47.0
busy-signal 1.4.3
elm-format 3.2.0
elmjutsu 9.9.1
file-icons 2.1.26
go-plus 6.0.1
intentions 1.1.5
javascript-snippets 1.2.1
jenkins 0.3.5
language-elm 1.5.0
language-graphql 0.11.0
language-hcl 0.4.0
language-jenkinsfile 0.1.1
linter 2.2.0
linter-ui-default 1.7.1
markdown-preview-plus 3.7.1
prettier-atom 0.56.0
Stylus 3.2.0

I've been getting this when there is an untyped alias or function at the top of a file. Feel free to close.

Hi, can you give me sample code?  Thanks!

module Main exposing (Model, Msg(..), aThing, init, main, update, view)
import Browser
import Html exposing (Html, div, h1, img, text)
import Html.Attributes exposing (src)
aThing =
{ aField = "" }
---- MODEL ----
type alias Model =
{}
init : ( Model, Cmd Msg )
init =
( {}, Cmd.none )
---- UPDATE ----
type Msg
= NoOp
update : Msg -> Model -> ( Model, Cmd Msg )
update msg model =
( model, Cmd.none )
---- VIEW ----
view : Model -> Html Msg
view model =
div []
[ img [ src "/logo.svg" ] []
, h1 [] [ text "Your Elm App is working!" ]
]
---- PROGRAM ----
main : Program () Model Msg
main =
Browser.element
{ view = view
, init = _ -> init
, update = update
, subscriptions = always Sub.none
}

ripped straight from create-elm-app

if it helps at all, sometimes when it gets thrown it emits the DummyElm modules into the project

Hi @aaahrens, I couldn't not reproduce the issue :(
Can you log the value of outString here:
https://github.com/halohalospecial/atom-elmjutsu/blob/master/lib/elm-make-runner.js#L428
Thanks!

Atom: 1.33.0
Mac: Mojave Version 10.14.1 (18875)
Just doing a simple main = text "Hello world!" gives the following stack trace on save
Uncaught TypeError: Cannot read property 'split' of undefined
/Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js:439
TypeError: Cannot read property 'split' of undefined
at /Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js:439:31
at Array.map ()
at /Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js:437:65
at ChildProcess. (/Users/johnmullany/.atom/packages/elmjutsu/lib/eval-helper.js:49:7)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at maybeClose (internal/child_process.js:925:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)

Hi @johnmullany,
Can you log the value of outString and errString here?
https://github.com/halohalospecial/atom-elmjutsu/blob/master/lib/elm-make-runner.js#L428
Thanks!

Also, are you using Elm 0.18 or 0.19?

0.19  and sure.

Not sure how to get the outString and errString...sorry.

@johnmullany

Open /Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js.
Insert the following in line 428:

console.log('errString', errString);
console.log('outString', outString);

Restart Atom.

I'm not getting any console errors but I do get the same error but now on line 441.

Step by step, I put main = "Hello world!". The module Main exposing (main) is then added for me. It show the red underline on main which I can hover over and get the error description. I also get the same error in the diagnostics panel. When I add just text in front of the string I get a new error description. Once I add the import Html exposing (..) I get the error. Sorry I can't be of more help. I haven't used Atom until I started with Elm.

where do we catch stdout of the console log?

@aaahrens, it's in /Users/drunkengranite/.atom/packages/elmjutsu/lib/elm-make-runner.js in your case.

Elm is great. I love it. I don't want to hijack this thread but I am anxious to hear what @aaahrens gets back from the addition of the console logs.

So I didn't get anything from the errString or the outString as far as the log goes so I put another console log for result on line 441 of elm-make-runner.js and get undefined for result. Split then tries to read undefined.
 /Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js:407 result line 441 undefined /Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js:408 Uncaught TypeError: Cannot read property 'split' of undefined     at /Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js:442:31     at Array.map (<anonymous>)     at /Users/johnmullany/.atom/packages/elmjutsu/lib/elm-make-runner.js:439:65     at ChildProcess.<anonymous> (/Users/johnmullany/.atom/packages/elmjutsu/lib/eval-helper.js:49:7)     at emitTwo (events.js:126:13)     at ChildProcess.emit (events.js:214:7)     at maybeClose (internal/child_process.js:925:16)     at Socket.stream.socket.on (internal/child_process.js:346:11)     at emitOne (events.js:116:13)     at Socket.emit (events.js:211:7)     at Pipe._handle.close [as _onclose] (net.js:554:12)

By the way, Elmjutsu is awesome. I used it before I went to Visual Studio Code and returned to Atom only because of Elmjutsu.

Perfect!! v9.9.2 is working. Thanks!!!


RFX: fix frame acks handling

This should make RemoteFX usable with Parallels Client. they always want zero unacked frames on the wire.

                                                  enc_done->enc->flags,
                                                  enc_done->enc->frame_id);
                     }
-                    else
-                    {
-#if 1
-                        ex = self->wm->client_info->max_unacknowledged_frame_count;
-                        if (self->encoder->frame_id_client + ex > self->encoder->frame_id_server)
-                        {
-                            if (self->encoder->frame_id_server > self->encoder->frame_id_server_sent)
-                            {
-                                LLOGLN(10, ("xrdp_mm_check_wait_objs: 1 -- %d", self->encoder->frame_id_server));
-                                self->encoder->frame_id_server_sent = self->encoder->frame_id_server;
-                                self->mod->mod_frame_ack(self->mod, 0, self->encoder->frame_id_server);
-                            }
-                        }
-#endif
-                    }
                     g_free(enc_done->enc->drects);
                     g_free(enc_done->enc->crects);
                     g_free(enc_done->enc);

why remote this else, it's needed in case client does not support frame acks.

OK, I just wanted to make sure it's not needed.


[Feature Request] Login using token

In some you bot altertives  use auth-token login  methot and unbound from Chrome. Login from  bot console or manual provided in config file.

This bot already supports using an auth token to login. Just add "auth_token": "myAuthToken" to your config.

Ty for answer. How to about browserless token generation ?

What do you mean by browserless token generation? I don't know how the auth tokens are generated.

Ok. i not expressed accurately: login from apps console like in pictures:

That actions create file with auth token for further logins.
It from alternative of you project and it uses nodejs too.


SKREAMS-4484 Complete transition to GHA

Description
Need to complete transition to GHA
Tests need to be uploaded
What to test

[ ] CI
[ ] Artifacts contain our test artifacts

https://workiva.slack.com/archives/CQJHUCLM9/p1711027638074329

I'm not sure where all the artifacts are going to be honest

https://wiki.atl.workiva.net/display/CICD/CI-CD+for+Workiva's+Public+(OSS)+Repositories

[ ] RMConsole requirements are not satisfied


Tweaked participating URL so it wont 404 on click

Fixes issue #58

Closing as this isn't needed :+1:


When a queue finishes playing, state is paused instead of idle.

It seems to me that when you play a fixed-length track and it ends with an empty queue, the state emitted on completion should be idle. Currently it emits paused but you are not able to press play to get audio playing again.

Also facing this issue.

Please provide the functions that you are using to add tracks and toggle play /pause. I managed to find a solution for this


MST-6 setGenesisCeremony Issue

MST-6 setGenesisCeremony Issue
Auditor: Movebit
Code: MCR Settlement Contract
Severity: Discussion
Status: Pending
Code Location:
protocol-units/settlement/mcr/contracts/src/staking/MovementStaking.sol#93
Descriptions:
Since the refundAmount paid by setGenesisCeremony to attesters is derived from the MovementStaking contract, it is possible for users to register new domains in order to withdraw tokens.
Suggestion:
It is recommended to ensure that this is as designed.

Duplicate of #452


Choosing Java compiler version?

Would choosing the Java compiler version be possible?
I'm trying to add a public static long to class A and use it in class B, but need it to run on Java 6 (due to incompatibilities with plugins as this is for an old Minecraft server)
with a much newer Java version, it runs fine, but with the likes of Java 6 and even 7, it just throws a NoSuchFieldError
turns out that class A was being overridden by something else lol

The compiler tries to use the current class version as a target as seen here: https://github.com/Col-E/Recaf/blob/68096acb9267ae360de887714caf102318fbff76/src/main/java/me/coley/recaf/ui/controls/text/JavaEditorPane.java#L121
The version is then fit into the range of whatever the current compiler (from your installed version of Java) supports.
Looks like you got this resolved though, so I'll close the issue.

it's possible to manually go in and set the target lower, but that is a tad bit annoying to do


When you select an item, onChange is called twice

Whenever an item is selected from the dropdown, the onChange action is called twice. In my case the value is printed to the console twice.
The template looks like this:
{{#ui-dropdown class="inline search selection" allowAdditions=true onChange=(action 'parentChanged') selected=parent}}
The controller code:
parentChanged(component, id, value) {
	console.log(value);
}

oh  crap i haven't seen thank you for mentioning it .


fix(version): update fabric8-runtime-console to 0.11.2

There were the following issues with your Pull Request

Commit: 1c91cd8d06e1054b90fd7e1560a95ffe063f2aa6
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


Fix pathing on OS X

Apparently OS X uses another env variable. /cc @d3m3vilurr.

Before:
 sdepold@barfooz-20  ~  rustc -v verbose
dyld: Library not loaded: x86_64-apple-darwin/stage1/lib/rustlib/x86_64-apple-darwin/lib/librustc-4e7c5e5c.dylib
  Referenced from: /Users/sdepold/.rsvm/current/dist/bin/rustc
  Reason: image not found
[1]    33569 trace trap  rustc -v verbose

After:
 sdepold@barfooz-20  ~  rustc -v verbose
rustc 0.12.0 (ba4081a5a 2014-10-07 13:44:41 -0700)
binary: rustc
commit-hash: ba4081a5a8573875fed17545846f6f6902c8ba8d
commit-date: 2014-10-07 13:44:41 -0700
host: x86_64-apple-darwin


fix(learn): render note on /responsive-web-design/basic-css/

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #54974

We don't need the show class either, do we?


Support change-dir before calling a command

Closes #13
Changes the bash script generated by command to look up the target executable in the runfiles.
Adds a test-case that executes a command passed as an argument after changing directory.

Thanks!


Powershell remoting fails on linux to linux remoting with banner

If i try powershell remoting on windows, it is fine. If i try to use powershell remoting from a linux machine to a linux machine, and the sshd has a banner defined in '/etc/ssh/sshd_config'(at the destination side) it fails to connect to the other linux machine.
Maybe it is the same issue as #5142, but it does not mention that this fails only on linux to linux remote powershell connections.
Steps to reproduce
PS /home/user> Enter-PSSession -HostName 'linuxmachine' -UserName 'user'

Expected behavior
PS /home/user> Enter-PSSession -HostName 'linuxmachine' -UserName 'user'
user@linuxmachine's password:
[user@linuxmachine]: PS /home/user>

Actual behavior
PS /home/user> Enter-PSSession -HostName 'linuxmachine' -UserName 'user'
user@linuxmachine's password: Enter-PSSession: The background process reported an error with the following message: The SSH client session has ended with error message: ###
# banner
###.

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Linux 4.19.97-v7+ #1294 SMP Thu Jan 30 13:15:58 GMT 2020
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

It is dup #12411 5142


Empty object in Response gets transformed to empty array

When updating to the latest version of Inertia-laravel, I encountered a problem where in our Resources empty objects get transformed to empty arrays.
We used to return an empty object this way, which resulted in an empty object in the Inertia response:
class ArticleResource extends JsonResource
{
    public function toArray($request): array
    {
        return [
            'tag_sets' => new \stdClass(),
        ];
    }
}

And in the controller:
return inertia('ArticlePage', [
    'article' => ArticleResource::make($article),
]);

Which resulted in a prop:
{
    article: {
        tag_sets: {}
    }
}

Since updating to the latest inertia-laravel, this is changed in an empty array:
{
    article: {
        tag_sets: []
    }
}

I come from a rather old version, so I'm not sure when it exactly is changed, but I've tracked down the change to this pull request: https://github.com/inertiajs/inertia-laravel/pull/265. In Reponse.php the data is requested "associative", which converts objects into arrays. (see https://www.php.net/manual/en/function.json-decode.php)
I am curious if this is intentional, and if it would be a good idea to make it more configurable. I don't see a way now to change the behavior.
I can change the frontend to also work with empty arrays, but this would need changes in many places where this is used. Therefore it would be better I think to be able to configure it on the backend.

Hey! That sucks that you bumped into this, but I think at this point we better leave this how it is, as it's been like this for over six months and I worry that changing it again could actually break things for more people.
We could make it an option, but I hate adding unnecessary options to the library, especially if this associative array format is generally the better choice. Of course if I get a lot of feedback about this, I will of course reconsider.
Hope that makes sense 🙏

Just ran into the same issue. Took me quite a while to even figure out that I'm not just using Laravel Resources incorrectly.
I'm sure there's good reasons for this behaviour, but I find it very unintuitive that my Laravel JsonResponse contains an empty object and my Vue component receives an empty array. Is there a suggested way to handle this?


New Url encoders difference in behavior from Url.EscapeDataString

@GrabYourPitchforks
Uri.EscapeDataString("https://www.googleapis.com/auth/plus.login") = "https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fplus.login"

With new encoder:
https:%2F%2Fwww.googleapis.com%2Fauth%2Fplus.login

Per RFC 3986 (Appendix A) / RFC 3987 (Sec. 2.2), the segment, query, and fragment portions of a URI are all allowed to contain the colon character. I originally suspected that the Uri.EscapeDataString behavior was for compliance with RFC 2396, but even that older RFC allows colons pretty much everywhere (including in the userinfo). RFC 3986 was extended to allow colons in the host name to support IPv6 addresses. Will follow up with that team to find out the history behind this behavior.

https://github.com/aspnet/HttpAbstractions/commit/1e9d57f80ca883881804292448fff4de8b112733


babel-eslint last version pipelineOperator resolved

Running with the last babel 7 (rc)
There is on error with "pipelineOperator"
I found on google to resolve we need to pass proposal at "minimal"
So changing in ./lib/parse.js
"pipelineOperator" line 46
to
['@babel/plugin-proposal-pipeline-operator', { 'proposal': 'minimal' }],
resolved my problem.

Thank you for the issue. Now that @babel/eslint-parser has been released, we are making this repository read-only. If this is a change you would still like to advocate for, please reopen this in the babel/babel monorepo.


fix mermaid version

Eliminate possible random errors caused by the most recent version of mermaid when it changes some output.

@paul-sachs  I appreciate your work and I verifed that this PR does fix the issue #91 .  This PR should be merged ASAP but the design of pulling mermaid from CDN is quite a bad idea that has to be fixed.
For everyone who suffers from this issue, I suggest use patch-package to made corresponding code change and add "postinstall": "patch-package" to let CICD works well.


EMERGENCY! EUREKA MAY BE INCORRECTLY CLAIMING INSTANCES ARE UP WHEN THEY'RE NOT. RENEWALS ARE LESSER THAN THRESHOLD AND HENCE THE INSTANCES ARE NOT BEING EXPIRED JUST TO BE SAFE.

EMERGENCY! EUREKA MAY BE INCORRECTLY CLAIMING INSTANCES ARE UP WHEN THEY'RE NOT. RENEWALS ARE LESSER THAN THRESHOLD AND HENCE THE INSTANCES ARE NOT BEING EXPIRED JUST TO BE SAFE.

Reopen if you have details or anything more than what you pasted.

Why get this warning ? when restart eureka server this disappear，but  restart service it reappear。

If there are no instances present it is normal for that message to appear.  It is saying that the expected number of service instances are lesser than expected.  When you restart a service the condition is created.

Is the number of services listed less than normal?

now ，two peer of eureka instance，you mean add eureka instance？

I got the same problem.

Looks like it appears when there's no instance registered for quite some time, if you start one instance and keep refreshing, the message goes away.

Take a look at the conversation in #1322 for more details about why this message appears.

restart the server


HUDI-3836 Improve the way of fetching metadata partitions from table

…config
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Improve the way of fetching metadata partitions from table
Brief change log

Modify getMetadataPartitions in hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java
Delete getCompletedMetadataPartitions  in hudi-common/src/main/java/org/apache/hudi/metadata/HoodieTableMetadataUtil.java

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Thanks for review, if there is a conflict with other submits when you reveiw, ping me, I will fix.

It seems that ITTestHoodieDataSource.testWriteAndReadDebeziumJson is flaky, Should I fix it here or open another issue?

@xicm Thanks for improving the code.  Could you rebase your PR on the latest master?

@hudi-bot run azure

shall we increate the maximum time of pipeline, I get this message twice.
"The job running on agent Azure Pipelines 10 ran longer than the maximum time of 120 minutes."

hi @yihua , Do I not have the permission to rerun CI? I found that I only received the email, but it was not actually executed. I have to push an empty commit to trigger the CI.

You should have permission to rerun CI.  Sometimes CI may not run due to throttling.  The timeout can be due to a temporary load.

hi @yihua， the checks failure is caused by #4459, can u review again, it may need revert.

@xicm could you rebase the PR on latest master to resolve the Java CI failure?


New contributors

Requirements

Filling out the template is required.

All new code requires tests to ensure against regressions.

However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.

[ ] Have you ran tests against this code?

[X] This PR contains zero code changes.

Description of the Change
This PR adds both @Spiker985 @kyjus25 as contributors to the project.

@confused-Techie Hey, thank you!!! 😁


Problem formatting material with "plib" for MCNP input

Describe the bug
#484 introduced a bug where MontePy can no longer write out a material input with "plib"
To Reproduce
Successfully load the deck, and then try to write it out again.
Error Message (if any)
Traceback (most recent call last):
  File "testit.py", line 3, in <module>
    deck.write_problem("out.imcnp", overwrite=True)
  File "montepy/mcnp_problem.py", line 413, in write_problem
    self._write_to_stream(fh)
  File "montepy/mcnp_problem.py", line 455, in _write_to_stream
    lines = obj.format_for_mcnp_input(self.mcnp_version)
  File "montepy/data_inputs/material.py", line 145, in format_for_mcnp_input
    lines = super().format_for_mcnp_input(mcnp_version)
  File "montepy/mcnp_object.py", line 131, in format_for_mcnp_input
    self._tree.check_for_graveyard_comments()
  File "montepy/input_parser/syntax_node.py", line 145, in check_for_graveyard_comments
    flatpack = self.flatten()
  File "montepy/input_parser/syntax_node.py", line 275, in flatten
    ret += node.flatten()
  File "montepy/input_parser/syntax_node.py", line 2279, in flatten
    ret += node.flatten()
  File "montepy/input_parser/syntax_node.py", line 275, in flatten
    ret += node.flatten()
AttributeError: 'NoneType' object has no attribute 'flatten'

MCNP input file snippet
If applicable, please include a small section of the input file you were working on. If it includes any specific values please change or remove them. For example:
plib example
C Cells
1   1 -1.0    -1
2   1 -0.5    -2 +1
3   0            +2

C Surfaces
1    so 0
2    so 2

c Data
m1      8016.71c  2-02
        8017.71c  9-09
        plib=84p
imp:p   2 1 0
mode p
nps 1

Version

Version 0.4.0


Material UI component name change

Deprecation needs to be handled.

NOTE: this is used by SectionTranslatable in the markdown-translatable package.

Per deprecation notice at:
https://material-ui.com/api/expansion-panel/

used by markdown sections.

@cckozie @elsylambert
FYI, for testing:
You can see the OLD name in PROD:

and the NEW name in DEVELOP:

v1.2.0-rc.3 build 28-c60492b
I'm not seeing any problems caused by this update.

Not seeing any issues by this change in translationCore Create - v1.2.0-rc.3 build 28-c60492b.


Is it possible to run applications without modification in inclavare-containers?

Looking for SGX LibOS docker solutions.
Is it possible to run applications without modification in inclavare-containers?

@qiyuangong Graphene-SGX libos is adapting to inclavare-containers, which will be available in next release of inclavare-containers.

OK. Thanks :)


Internal PR test

What this PR does / why we need it: Nothing useful really, just checking how actions can access secrets
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #
Public facing documentation PR (if applicable)

Special notes for reviewer:
Checklist:

[ ] Added relevant notes to WIP-CHANGELOG.md
[ ] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[ ] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Pipeline config PR (if applicable)

If you change some config options (e.g. rename variable or change the default value) we may need to update the pipeline.

This was just for testing. Closing...


Validation: Fix multiple issues

This PR addresses the following issues:
description-missing
image-alt-text-duplicated
#28 fixes across #26 files

Global effort to fix validation errors
@adegeo
The Content & Learning team is fixing build validation errors on docs.microsoft.com. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. I’d very much appreciate your timely review and feedback on these changes I’ve suggested. Thanks!


fix(blueprint): ParameterValueCollection from ParameterValue

The collection was of the ParameterValueBase type rather than ParameterValue which includes the base

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

/cc @filizt since I think you've been working on this and might be able to tell if it makes sense

/cc @filizt since I think you've been working on this and might be able to tell if it makes sense

@nschonni this change will break the polymorphic behavior we'd like for ParamaterValueBase's child types so, please hold off on the changes. We'll likely use discriminated hierarchy as a solution here, still discussing with the team. Please let me know if there are any questions.

@nschonni - The fix in the PR is incorrect. The service team is aware about this issue and will fix it by making ParameterValueBase a polymorphic model with a discriminator property. This requires a service side change. Hence closing this PR.


flush events (rebased)

Current dependencies on/for this PR:

main

PR #103

PR #102

PR #111   👈
PR #109

This comment was auto-generated by Graphite.

I did some more testing and fixed a few issues that popped up when using 1 worker (just needed to handle the nil pool in a few places).
I'm reasonably confident the new BenchmarkDVCClient_VariableParallel benchmark is actually running at a particular concurrency level, with a configurable number of workers, and optionally with events and/or config updates happening.
Single threaded
$ go test -run=^$ -bench ^BenchmarkDVCClient_VariableSerial$ -benchDisableLogs -cpu=1 ./
goos: darwin
goarch: amd64
pkg: github.com/devcyclehq/go-server-sdk/v2
cpu: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
BenchmarkDVCClient_VariableSerial 	   30393	     38837 ns/op	    1824 B/op	      72 allocs/op
PASS
ok  	github.com/devcyclehq/go-server-sdk/v2	2.572s

Parallel with 12 CPUs and 12 workers (default on my machine)
go test -run=^$ -bench ^BenchmarkDVCClient_VariableParallel$ -benchDisableLogs -cpu=12 ./
goos: darwin
goarch: amd64
pkg: github.com/devcyclehq/go-server-sdk/v2
cpu: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
BenchmarkDVCClient_VariableParallel-12    	  180336	      7239 ns/op	         0 reconfigs	        12.00 workers	    1843 B/op	      73 allocs/op
PASS
ok  	github.com/devcyclehq/go-server-sdk/v2	2.371s

That's an average of 86868 ns/op when you factor in the 12X parallelization.
Parallel with 12 CPUs and 12 workers and config updates enabled:
go test -run=^$ -bench ^BenchmarkDVCClient_VariableParallel$ -benchDisableLogs=true -cpu=12 -benchEnableEvents=false -benchEnableConfigUpdates ./
goos: darwin
goarch: amd64
pkg: github.com/devcyclehq/go-server-sdk/v2
cpu: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
BenchmarkDVCClient_VariableParallel-12    	  169506	     28158 ns/op	       141.0 reconfigs	        12.00 workers	    1957 B/op	      73 allocs/op
PASS
ok  	github.com/devcyclehq/go-server-sdk/v2	6.086s

Parallel with 12 CPUs and 12 workers and event logging enabled:
go test -run=^$ -bench ^BenchmarkDVCClient_VariableParallel$ -benchDisableLogs=false -cpu=12 -benchEnableEvents ./
2023/03/09 16:30:49 INFO: Enabling event logging
2023/03/09 16:30:49 INFO: Config set. ETag: TESTING
2023/03/09 16:30:49 INFO: DevCycle SDK Initialized.
goos: darwin
goarch: amd64
pkg: github.com/devcyclehq/go-server-sdk/v2
cpu: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
BenchmarkDVCClient_VariableParallel-12    	2023/03/09 16:30:49 INFO: Enabling event logging
2023/03/09 16:30:49 INFO: Config set. ETag: TESTING
2023/03/09 16:30:49 INFO: DevCycle SDK Initialized.
2023/03/09 16:30:49 INFO: Enabling event logging
2023/03/09 16:30:49 DEBUG: Ticker for event flush triggered
2023/03/09 16:30:49 DEBUG: Flushing events from all workers
2023/03/09 16:30:49 DEBUG: Flushing events from worker 1
2023/03/09 16:30:49 DEBUG: Flushed 1 events from worker 1
2023/03/09 16:30:49 INFO: Config set. ETag: TESTING
2023/03/09 16:30:49 INFO: DevCycle SDK Initialized.
2023/03/09 16:30:49 INFO: Enabling event logging
2023/03/09 16:30:50 DEBUG: Ticker for event flush triggered
2023/03/09 16:30:50 DEBUG: Flushing events from all workers
2023/03/09 16:30:50 DEBUG: Flushing events from worker 13
2023/03/09 16:30:50 DEBUG: Flushed 1 events from worker 13
2023/03/09 16:30:50 INFO: Config set. ETag: TESTING
2023/03/09 16:30:50 INFO: DevCycle SDK Initialized.
2023/03/09 16:30:50 DEBUG: Ticker for event flush triggered
2023/03/09 16:30:50 DEBUG: Flushing events from all workers
2023/03/09 16:30:50 DEBUG: Flushing events from worker 25
2023/03/09 16:30:50 DEBUG: Flushed 1 events from worker 25
2023/03/09 16:30:50 DEBUG: Ticker for event flush triggered
2023/03/09 16:30:50 DEBUG: Flushing events from all workers
2023/03/09 16:30:50 DEBUG: Flushing events from worker 37
2023/03/09 16:30:50 DEBUG: Flushed 1 events from worker 37
  162259	      7527 ns/op	         0 reconfigs	        12.00 workers	    1843 B/op	      73 allocs/op
PASS
ok  	github.com/devcyclehq/go-server-sdk/v2	2.529s```

**This is where I hit an issue. ** When I added more logging here, it looks like the events from the workers are not actually being processed by the main event queue. I think there's a deadlock happening between the event queue calling into the workers, which are pushing onto the queue, which isn't being read by the event queue because it's calling the workers.


[Bug] Clear all the futures when sync is called.

Futures may contain exception when retry happens. Futures.waitAll() returns when it encounters an exception. This will cause execute() call to never succeed.
So sync() call should clear future List so that retry can proceed.

/ok-to-test

/approved
/lgtm


write metadata to header when it's valid

TODO: add a test

Coverage increased (+0.3%) to 75.392% when pulling cbfa870d44122f0914a92e9ba2e1506bb395e435 on keflavich:metadata into bf9b5e1b9a09855cb4c683357edece6d9720acbd on radio-astro-tools:master.

Tests done, ready for final review.


soft delete (on attributes)

no idea how this should be set up-- probably would need to look at best practices, but just tracking it

@Thomasdezeeuw @snyx1 @mphasize @polastre @aclave1 you can now soft-delete records with .archive() - Docs

@rachaelshaw The idea of SoftDelete is cool, but we need to create more implementations.
As a BLUEPRINT to be able to perform this operation directly via ENDPOINT


cro3.bash: Fix completion to work

It seems mapfile does not work on some environment. So partially revert the changes using mapfile.

Thanks for the quick fix!!!! verified, cr+2


feat(spark-lineage): support bigquery, s3, gcs

This PR:

Adds support for bigquery tables.
Recognizes and creates datasets for s3 and gcs with explicit platforms. Previously, platform was getting populated as hdfs
Adds support to configure dataset urn similar to python sources.

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Watching this PR.  this feature would be a big add!

Any update on this PR?

We are actively looking for volunteers to help revive and drive this PR to the finish line!
Cheers
John


[TACHYON-639] Check buffer returned from reading a dummy stream is null.

JIRA: TACHYON-639
This PR fixes bug reported by Pengfei Xuan on JIRA TACHYON-639. The bug causes basic THROUGH test to fail as no null-check is performed for the buffer returned from reading a dummy stream, before copying the data to another buffer.

@ooq Thanks for fixing!

Thanks for looking into this @ooq ! I left a question in the code.

@gpang Such carelessness! Thanks for catching this. Updated.

+1 LGTM

LGTM


Test Mac OS Mojave

Someone who has a Mac, can he test the new Mac OS Mojave that came out today? It may be only our software, but there is a possibility that Avalonia as a whole is broken on it, too.

After updating to latest version of Mac OS Mojave v10.14.2 (18C54), the wallet screen never loads

https://github.com/zkSNACKs/WalletWasabi/issues/935

Should be fixed by https://github.com/AvaloniaUI/Avalonia/commit/700752c4c26a55c1c68843becbbf62d1129a3d01


Doesn't work with React

The project has angular speific functions and calls.
Cannot use this via React, very annoiyng.

Are you sure?
I use this plugin very extensively in my react based projects so I'm pretty certain this library does not contain any angular specific functions (or any framework for that matter). Please do point for us where exactly the angular function calls are being made from within this plugin, it would help us a lot.

@breautek 100% certain just spent the last few hours installing it into a blank project and trying to get it too run. I can also see what when it's installed the angular calls are there.
Thanks for closing the issue instantly.

@breautek if you look at the index.js you can the calls to angular.

@breautek if you look at the index.js you can the calls to angular.

This plugin doesn't have an index.js file.

@breautek mate all node_modules have index.js files.
I'm using the instructions here https://ionicframework.com/docs/v3/native/file/

And the instrutions here: https://ionicframework.com/docs/native/file

npm install cordova-plugin-file
npm install @ionic-native/file
ionic cap sync

Installs a version with Angular calls you can see from the JS on the index.

@breautek mate all node_modules have index.js files.
I'm using the instructions here https://ionicframework.com/docs/v3/native/file/

Cordova plugins are not traditional node_modules. Plugins don't always have single entry point, and furthermore NPM doesn't run on devices. The only way NPM is involved in cordova plugins is that NPM is used as a distribution method, but the package.json main field does not apply.
Also Ionic is a framework that uses but is independent from Cordova. So what it sounds like is something in the Ionic's code base that is using Angular, which makes a little more sense since Ionic was originally built as an Angular framework for Cordova, if I recall correctly. In which case, you'll have to communicate with the Ionic team.
Again, i'm pretty confident there are no angular calls being made from this code base. If I'm wrong, please do point me to the file and line numbers in https://github.com/apache/cordova-plugin-file/tree/master/www and I'll reopen this issue.