SVG Icons: Unable to Supply  Element

Although the SVG font icons support a children prop, I am unable to pass in a defs element to define a custom linear gradient. I.e.:
         import Stars from 'material-ui/svg-icons/action/stars';

          <Stars
            children={
              <defs>
                <linearGradient id="MyGradient">
                  <stop offset="5%" stopColor="#F60" />
                  <stop offset="95%" stopColor="#FF6" />
                </linearGradient>
              </defs>
            }
            className="star--half"
          />

The icon renders without the defs element.

@joncursi The pre-built icons don't currently support children, as the path for the icon itself is the SvgIcon children.
import React from 'react';
import pure from 'recompose/pure';
import SvgIcon from '../../SvgIcon';

let ActionStars = (props) => (
  <SvgIcon {...props}>
    <path d="M11.99 2C6.47 2 2 6.48 2 12s4.47 10 9.99 10C17.52 22 22 17.52 22 12S17.52 2 11.99 2zm4.24 16L12 15.45 7.77 18l1.12-4.81-3.73-3.23 4.92-.42L12 5l1.92 4.53 4.92.42-3.73 3.23L16.23 18z"/>
  </SvgIcon>
);
ActionStars = pure(ActionStars);
ActionStars.displayName = 'ActionStars';
ActionStars.muiName = 'SvgIcon';

export default ActionStars;

@joncursi If the prebuilt SvgIcons supported defs as children, how would you apply them to the built-in SVG paths?

@mbrookes I was looking to apply a linear gradient over the SVG path rather than a solid color. CSS can't do this alone, it requires DOM elements be defined like this: http://www.w3schools.com/svg/svg_grad_linear.asp
The fill name can then either be passed in as an additional prop, or specified to be used in CSS.

@joncursi The problem is that some icons have multiple paths (or other elements circle etc.), so which to apply what id to?

Requiring that SvgIcons embed the raw source of an SVG instead of some way to link or load from an external source is rather limiting. Creating these in TypeScript, you can't embed an icon-specific <style> tag if there's a class to be reused by some/all of the contained paths. it fails to compile because it interprets the css. I know the styles could potentially be applied by the caller creating the icon, or at a global level, but this seems really awkward if the goal is to encapsulate an 'icon' with this pattern.

We have 3 paths going forward:

We tell users to copy and paste the icon path. So we don't change anything.
We make all our icons exporting the path.
We add an extra property to the SvgIcon component. extraChildren is something like this.

Option 1 and 3 sounds like the best ones to me.

I just encountered this problem today... I like option 3. There is a project called Grommet that does something like this http://grommet.io/docs/icon
Each path is wrapped in <g ...attributes><path /></g> flag and you can specify path attributes
<SVGIcon viewBox='0 0 130 108'
  version='1.1'
  type='logo'
  a11yTitle='Locations Finder'>
  <g stroke='#865CD6'
    strokeWidth='4'
    fill='none'
    strokeLinejoin='round'>
    <path d='M40,65 L40,96 L16,107 L16,49 L16,49 L28.4679195,43.2855369 M75.6892892,43.6424091 L88,38 L88,96 L64,107 L64,64.5 L64,64.5 M64,64 L64,107 L40,96 L40,65 M89,38 L113,49 L113,107 L89,96 L89,38 Z M52,49 C56.971,49 61,44.971 61,40 C61,35.029 56.971,31 52,31 C47.029,31 43,35.029 43,40 C43,44.971 47.029,49 52,49 L52,49 Z M52,76 C52,76 28,58 28,40 C28,25 40,16 52,16 C64,16 76,25 76,40 C76,58 52,76 52,76 Z' />
  </g>
</SVGIcon>

@dustingraves People can already do that with Material-UI. I don't follow.

I didn't read the original problem fully... I will remove my comment to keep the issue clean

Actually, I think that we have an even better solution to the problem. We can make the SvgIcon supports a component property. It's a pattern we use everywhere.


DiscordMail is down?

Where did it go? :(

It didn't move?

@lepon01 I had noticed the bot was down, it's been back up now.


Addition of a data mining section

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
Addition of data mining section which will contain all the relevant resources related to it.
Explain your solution
No response
Any alternative approaches/features
No response
Additional Context
No response

@jfmartinz  kindly assign me this


open dropdown on page load

How can i make dropdown open on page load.

Unfortunately there's no built-in function to do that.
You'll need to modify the directive code, in order to do that.


{containerapp} | Fixed logs-workspace-id description

Fixes https://github.com/Azure/azure-cli/issues/22914

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

containerapp


Create troubleshooting-cni-plugin-related-errors.md

Signed-off-by: bin.hui<EMAIL_ADDRESS>

? what happened?

I am sorry, I am try  commint changes.

Hey @ihuibin, we have moved Mandarin contain from content/zh/docs to content/zh-cn/docs, please rebase and move you file accordingly.
You can /unhold by yourself once it's done.
/hold

/unhold

/close    close this pr ,create a  new  in zh-cn.

@ihuibin Are you proposing a new PR for this?
We are waiting for this one to close #34221.

Yes， I proposing a new one in zh-cn.

@ihuibin where is it?

/remove-language en ja ko pt


Error parsing From address from Feedbackloop

I was testing Sisimai with a set of feedback loop messages and noticed that Multi line From addresses with a @  would break parsing.
To reproduce I downloaded  arf-22.eml file from repo and changed the from Address

perl -MSisimai -e 'print Sisimai->dump("arf-22.eml.");'  | jq '.[].addresser'

<EMAIL_ADDRESS>Then changed the From address
--- arf-22.eml.1	2020-03-31 19:41:41.056614734 +0000
+++ arf-22.eml	2020-03-31 19:41:54.292589247 +0000
@@ -36,7 +36,8 @@
     Thu, 29 Apr 2016 23:34:45 +0000
 Date: Thu, 29 Apr 2016 23:34:45 +0200
 To<EMAIL_ADDRESS>-From: Neko<EMAIL_ADDRESS>+From: ass@as
+<EMAIL_ADDRESS> Subject: Nyaan
 MIME-Version: 1.0
 Content-Transfer-Encoding: 8bit

perl -MSisimai -e 'print Sisimai->dump("arf-22.eml.");'  | jq '.[].addresser'

"ass@as"

@jcbf Hello, Thanks for the bug report. We have created the same email and tested it. Then, the DDP module shows the following results:
            addresser        Sisimai::Address  {
                internals: {
                    address   "ass@as",
                    alias     "",
                    comment   "",
                    host      "as",
                    name      "ass@as",
                    user      "ass",
                    verp      ""
                }
            },

Perhaps, this is a bug in Sisimai::Address->find method...

@jcbf We have fixed and tagged with v4.25.5p11 :-)

Good, I'll be parsing some DSN regularly and give feedback.
Is there any simple way just to parse the response from the remote server?
example:

550 5.1.10 RESOLVER.ADR.RecipientNotFound; Recipient not found by SMTP address lookup (in reply to RCPT TO command))

There is no simple way to parse the response because of many formats of error messages, modules in Sisimai::Lhost::* is needed to get the response. However, when the response exists in any scalar value, the following sample may help you:
#!/usr/bin/env perl
use strict;
use warnings;
use Sisimai;
use Sisimai::SMTP::Reply;
use Sisimai::SMTP::Status;

my $e = '550 5.1.10 RESOLVER.ADR.RecipientNotFound; Recipient not found by SMTP address lookup (in reply to RCPT TO command))';
printf("%s\n", Sisimai::SMTP::Status->find($e));    # 5.1.10
printf("%s\n", Sisimai::SMTP::Reply->find($e));     # 550
printf("%s\n", Sisimai->match($e));                 # userunkonwn


kernel_provisioner ignored with custom kernel

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
When using a custom kernel via the extension, the metadata.kernel_provisioner details appear to be ignored.
For example, install some custom provisions using pip install gateway-provisioners (https://github.com/jupyter-server/gateway_provisioners).  You can validate they installed correctly using jupyter kernelspec provisioners.  Now create a kernel.spec that uses a custom provisioner (e.g. jupyter docker-spec install).
When switching to the custom kernel, I see an error and log output like shown below, indicating that the custom provisioner is being ignored.  If instead I run from a terminal (jupyter console --kernel docker_python), I can see that these variables are defined, and therefore the custom provisioner is being used.
VS Code Version
1.81.1
Jupyter Extension Version
v2023.7.1002162226
Jupyter logs
20:19:51.499 [debug] KernelProvider switched kernel to id = .docker_python./python./.python#~/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py#--kernel-id#{kernel_id}#--port-range#{port_range}#--response-address#{response_address}#--public-key#{public_key}#--kernel-class-name#ipykernel.ipkernel.ipythonkernel#--connection-file#{connection_file}

Coding Language and Runtime Version
Python v3.9
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Remote

When switching to the custom kernel, I see an error and log output like shown below, indicating that the custom provisioner is being ignored.

Please can you share the logs, you forgot to share that information

I actually included the one log line that shows the launch parameters are not being resolved.  Let me know if you need more detail.

Thats not the full argument list, that is just an ID generated (used for internal use).

I see an error and log o

I do not see any errors in the logs you have provided.
Please could you share that.

Here is a more detailed log.  In it, you can see parameters like {port_range} aren't being resolved when switching to the new kernel, which seems to indicate the custom provisioner is being ignored.
13:41:40.620 [debug] KernelProvider switched kernel to id = .docker_python./python./.python#~/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py#--kernel-id#{kernel_id}#--port-range#{port_range}#--response-address#{response_address}#--public-key#{public_key}#--kernel-class-name#ipykernel.ipkernel.ipythonkernel
13:41:40.621 [debug] Initialize notebook communications for editor /mnt/efsdev/<username>/test.ipynb
13:41:40.622 [debug] Resolving notebook UI Comms (resolve) for /mnt/efsdev/<username>/test.ipynb
13:41:40.623 [debug] initialize CommonMessageCoordinator
13:41:40.624 [debug] Registering commtarget jupyter.widget
13:41:40.624 [debug] IPyWidgetMessageDispatcher.initialize
13:41:40.625 [debug] Attempting to determine version of IPyWidgets
13:41:40.626 [debug] Controller selection change completed
13:41:40.626 [debug] Waiting for IPyWidgets version promise
13:41:40.626 [debug] Version of IPyWidgets 7 determined after 0.001s
13:41:41.319 [debug] IPyWidgetMessageDispatcher.initialize
13:41:41.319 [debug] Web view is ready to receive widget messages
13:41:51.670 [info] Handle Execution of Cells 1 for /mnt/efsdev/<username>/test.ipynb
13:41:51.672 [debug] start the kernel, options.disableUI=false for /mnt/efsdev/<username>/test.ipynb
13:41:51.674 [info] Starting Kernel startUsingLocalKernelSpec, .docker_python./python./.python#~/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py#--kernel-id#{kernel_id}#--port-range#{port_range}#--response-address#{response_address}#--public-key#{public_key}#--kernel-class-name#ipykernel.ipkernel.ipythonkernel  for '/mnt/efsdev/<username>/test.ipynb' (disableUI=false)
13:41:51.676 [debug] Creating raw notebook for resource '/mnt/efsdev/<username>/test.ipynb'
13:41:51.679 [debug] Kernel Launcher. launch, Class name = Ky (started execution), Arg 1: "~/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py#--kernel-id#{kernel_id}#--port-range#{port_range}#--response-address#{response_address}#--public-key#{public_key}#--kernel-class-name#ipykernel.ipkernel.ipythonkernel", Arg 2: 60000, Arg 3: "/mnt/efsdev/<username>/test.ipynb", Arg 4: "/mnt/efsdev/<username>/nlp/sagemaker/test", Arg 5: ""
13:41:51.699 [info] Dispose Kernel process undefined.
13:41:51.700 [error] Kernel Launcher. launch, Class name = Ky, completed in 21ms, has a falsy return value, Arg 1: "~/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py#--kernel-id#{kernel_id}#--port-range#{port_range}#--response-address#{response_address}#--public-key#{public_key}#--kernel-class-name#ipykernel.ipkernel.ipythonkernel", Arg 2: 60000, Arg 3: "/mnt/efsdev/<username>/test.ipynb", Arg 4: "/mnt/efsdev/<username>/nlp/sagemaker/test", Arg 5: "" [Error: Connection file not found in kernelspec json args, python ~/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py --kernel-id {kernel_id} --port-range {port_range} --response-address {response_address} --public-key {public_key} --kernel-class-name ipykernel.ipkernel.IPythonKernel
	at Ly.updateConnectionArgs (~/.vscode-server/extensions/ms-toolsai.jupyter-2023.7.1002162226/out/extension.node.js:24:94424)
	at Ly.launch (~/.vscode-server/extensions/ms-toolsai.jupyter-2023.7.1002162226/out/extension.node.js:24:90548)
	at Ly.i.value (~/.vscode-server/extensions/ms-toolsai.jupyter-2023.7.1002162226/out/extension.node.js:2:1695691)
	at Ky.launchProcess (~/.vscode-server/extensions/ms-toolsai.jupyter-2023.7.1002162226/out/extension.node.js:24:101920)
	at processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async ~/.vscode-server/extensions/ms-toolsai.jupyter-2023.7.1002162226/out/extension.node.js:24:100416]
13:41:51.702 [debug] Disposed Kernel process undefined.
13:41:51.702 [error] Failed to connect raw kernel session: Error: Connection file not found in kernelspec json args, python ~/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py --kernel-id {kernel_id} --port-range {port_range} --response-address {response_address} --public-key {public_key} --kernel-class-name ipykernel.ipkernel.IPythonKernel
13:41:51.702 [debug] Shutdown session -- complete

In this case, the kernel.json file is the following
{
  "argv": [
    "python",
    "/home/<username>/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py",
    "--kernel-id",
    "{kernel_id}",
    "--port-range",
    "{port_range}",
    "--response-address",
    "{response_address}",
    "--public-key",
    "{public_key}",
    "--kernel-class-name",
    "ipykernel.ipkernel.IPythonKernel"
  ],
  "display_name": "Docker Python",
  "language": "python3",
  "interrupt_mode": "signal",
  "metadata": {
    "debugger": true,
    "kernel_provisioner": {
      "provisioner_name": "docker-provisioner",
      "config": {
        "image_name": "da4cbecb-6ec7-4e3e-93c1-853d0cf6ca3c",
        "launch_timeout": 30,
        "port_range": "40000..60000"
      }
    }
  }
}

If I switch the kernel via the command line, I see these parameters being resolved.
$ jupyter console --kernel docker_python
[ZMQTerminalIPythonApp] ResponseManager is bound to port 8877 for remote kernel connection information.
[ZMQTerminalIPythonApp] DockerProvisioner: kernel launched. Kernel image: da4cbecb-6ec7-4e3e-93c1-853d0cf6ca3c, KernelID: 830bb406-83e9-4201-ae11-7dd08b93cc26, cmd: '['python', '/home/<username>/.local/share/jupyter/kernels/docker_python/scripts/launch_docker.py', '--kernel-id', '830bb406-83e9-4201-ae11-7dd08b93cc26', '--port-range', '40000..60000', '--response-address', '<IP_ADDRESS>:8877', '--public-key', 'MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQCdSOltNdoeSJD2G9VofQO3d2nxIdgXHWzsW13s5ZAj3uWqFuiTxHIiiWRWOLNqUO2/LHOqdVoAWsnTfBRxbTYoeaa7kwsXpYnILioRo3NqKyMVxL/PgB8ka0cD0IB4nQdaDZfgXZOrUbyzUpxyZzV3LzTffj6a+UefMYqJ3WL1ewIDAQAB', '--kernel-class-name', 'ipykernel.ipkernel.IPythonKernel']'

Thanks, I think i understand whats going on here
Will look into this

I've had a look at this, and this is a custom Jupyter Server plugin that provides its own way to launch kernels.
The Jupyter extension in VS Code currently only supports one way of launching Jupyter kernels, and thats the standard model of launching local jupyter kernels.
Updating this as a feature request.

Its been a while and there haven't been enough upvotes on this issue.
Hence closing this


test: run integration tests against emulator

Based on #305 , fixes some problems in some of the integration tests.

Thanks so much for this change @olavloite.


Analyzer model + backend

Datastore model + a bit of backend operations (to store and get the analysis model object).

PTAL, almost everything changed :)
Important note: I've switched to int id, but I couldn't figure out how to get the appengine-assigned id, I did a fallback to use millisecondsSinceEpoch instead. I think collision here is minimal, and even if it happens, we won't end up in a bad state.

@isoos It is super easy:
final model = new Analysis();
assert(mode.id == null);
await dbService.commit(inserts: [model]);
assert(mode.id is int);

Will take another look at the PR later on.

PTAL. Now package points only to version, and version to analysis. Also moved the object creation and updates into model.

Now package points only to version,

I don't think it does, the current PR is still setting it to analysis.key:
   PackageAnalysis _createPackage(Analysis analysis) {
     final package = new PackageAnalysis()
       ..parentKey = db.emptyKey
       ..id = analysis.packageName
       ..latestVersionAnalysisKey = analysis.key;
     return package;
   }

The github reviewing makes this quite difficult, since I cannot see the changes you uploaded since the last review. (there would be an option to use rietveld)
Have you uploaded your latest changes or is there something missing?

Have you uploaded your latest changes or is there something missing?

My bad, typed in git push ..., hit enter, but didn't check if it was running all right. Turns out it sometimes fails, fixed it.


Docs does not work properly

I have no idea how http://fms-cat.com/automaton/docs/modules/automaton.html does not work

https://github.com/TypeStrong/typedoc/issues/149
ah,

#27 has resolved the it

How did that resolve it?


Refactored the code and tests, added proper error handling and error …

 Let me know what you think about it and if you would like the code style to go a different route.

@Plommonsorbet Great work. Will give it a good review as soon as possible :)

@Plommonsorbet I've given it a review and run the tests. It is all looking great!
Just a bit of clarification before we merge, in the following:
                             - "pier.toml"
                             - "$PIER_CONFIG_PATH"
                             - "$XDG_CONFIG_HOME/pier/config.toml"
                             - "$XDG_CONFIG_HOME/pier/config"
                             - "$XDG_CONFIG_HOME/pier.toml"
                             - "$HOME/.pier.toml"
                             - "$HOME/.pier"
                            [env:PIER_CONFIG_PATH: ]```

It's not clear what the first option or last line mean.

I looked it over and fixed some mistakes with the ordering and clarified the meaning. I was not able to remove the last line however, it's autogenerated by clap. Tried the build hide_env_values argument flag from the clap docs but it didn't seem to work for me. Maybe my usage was wrong, didn't spend so much time on it. Maybe we can fix it in some later patch?


Fix test_indexes to suppress warnings.

We should use assertRaisesRegex instead of assertRaisesRegexp.
DeprecationWarning: Please use assertRaisesRegex instead.

This is a simple fix. I'd merge this now. cc @HyukjinKwon

Nice!


Run callback of setDestructor immediately when GC happens

@mitchhentges Can you try if this fixes #4733 for you? I can't reproduce the original issue so I'm not able to try it myself.

Sure, just need to set up the reproduce scenario again

Looks good, I tested two things:

electron-connect which is failing on v0.37.2
Having two inputs: one of them uses a script's root-level-scope version of window, and console.logs it. The other one is a closure as follows:

(function() {
const window = remote.getCurrentWindow();
console.log(window)
}());

Ideally, if there was something wrong with the destructor, the closure would cause a GC every time that it finished executing.
Both cases succeeded. :+1:

@zcbenz just to confirm, this is good to be merged, imho.
I'm not sure if you were still waiting on a response, or leaving this PR on purpose :)

@mitchhentges I wanted to leave it open for a while to collect possible feedbacks, since it may have other side effects. I'm merging now.


Combined template 39

See issue #39
"We have 3 OpenShift templates for each of the 3 components in the Obsidian project but we want a single combined one that will create all the necessary applications in one step."

Does it work with oc apply -f ?

No idea @gastaldi . I'm not very much at home with the OpenShift command line.


Added Die Hard 4 or Live Free or Die Hard

Checklist for submitting a pull request to Movies For Hackers:

[x] I have carefully read and comply with the Contributing Guidelines of this repo.
[x] I have searched the PRs of this repo and believe that this is not a duplicate.

Title:
Link:
Genre:
Rating:

Description:

It's a duplicate; already in the list as "Live Free or Die Hard".

already included.

@k4m4 please close this issue

Hey @Raavtube. Thank you for your contribution. I am closing this because it seems that the list already includes "Live Free or Die Hard".


Add markdown lint workflow

This PR introduces markdown linting.
The idea is to have a uniform style for all the markdown documentation files and ensure this by automation.
Automation comes in the form of a pre-commit hook (husky) that runs the linting step and will prevent the commit, in case there is a violation to a rule.
Additionally, a GitHub workflow is added, that runs on PRs to main. This workflow will also run the lint step.
Linting is done via markdownlint-cli2

This PR also contains the changes from PR #3. This is because #3 is not reviewed yet, and both PRs are created from the same fork

Checked MD linting and pre-commit-hook in own fork together with @hazierer

checked Docusaurus build + deploy on own fork together with @carslen.


1.10.2 Furnace Explodes

Furnace explodes when cooking even a single piece of food. Is this intended behavior? If so, is there a way to turn it off or make it much less likely?
I am trying this mod for the first time for a modpack and this is a show stopper in early game.

Hello, @scottosx!
This is surely not intended. I've added a furnace explosion just to test the furnace pollution behavior and unfortunately forgot to remove it.
The bug will be fixed at the nearest time!

Fixed

Thanks for the quick fix! I didn't think it was as intended. Still, the mechanic is interesting and I can see that it could be a consequence of very high pollution levels. Are you looking at something like that? It would need to be configurable of course.


[BUG]: Nested sql.raw statements don't work

What version of drizzle-orm are you using?
0.34.1
What version of drizzle-kit are you using?
0.25.0
Describe the Bug
I have a sql`` query that needs to interpolate a sql.raw statement and that sql.raw statement interpolates another sql.raw inside but then the innermost one isn't correctly parsed and gives a sql error.
Something like:
const a = `...`
const b = `...${sql.raw(a)}...`
sql`...${sql.raw(b)}...`

Expected behavior
No response
Environment & setup
No response

I was tired. Actually you can just nest the template literals and then call sql.raw at top level


Some numbers (benchmark)

Hi! i was just looking for some numbers about the "overhead" that this library could introduce in some use cases, i made my own tests, since i haven't seen benchmarks my plan is to put here some of the ones i'm creating. I hope it can help to other people too.
Benchmark library: https://github.com/dotnet/BenchmarkDotNet
Version used: 0.12.1
My Machine information:
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.19041 N/A Build 19041
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free

System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
                           [01]: Intel64 Family 6 Model 158 Stepping 11 GenuineIntel ~4008 Mhz

System Locale:             en-us;English (United States)
Input Locale:              en-gb;English (United Kingdom)

Available Physical Memory: 7.862 MB
Virtual Memory: Max Size:  22.466 MB
Virtual Memory: Available: 8.338 MB
Virtual Memory: In Use:    14.128 MB

Benchmark: File.IsFile comparison
public static class FileSupportStatic
{
        public static bool IsFile(string path)
        {
            return File.Exists(path);
        }
 }

public class FileSupport
{
        #region Members
        private IFileSystem _fileSystem;
        #endregion

        #region CTOR's
        public FileSupport(IFileSystem fileSystem)
        {
            _fileSystem = fileSystem;
        }

        public FileSupport() : this (new FileSystem())
        {
            // Default implementation for FileSystem
        }
        #endregion

        public bool IsFile(string path)
        {
            return _fileSystem.File.Exists(path);
        }
  }

Wow, thanks a lot for your work!
What do think about adding your benchmarks and results to this repo?

Sure, i have more test now, i will PR them this weekend.

https://github.com/System-IO-Abstractions/System.IO.Abstractions/pull/608

I'll add more these days

I'll close this issue. Looking forward to more PRs 🥇


Discord_Shutdown does not unset Discord user's Playing status

For both the send-presence sample and our own integration, a call to Discord_Shutdown does not call the disconnect handler, and the Playing <game> status in Discord stays there for the user even after the process has been shut down. Calling Discord_UpdateConnection() and Discord_RunCallbacks() after Discord_Shutdown() (to really cover all our bases) didn't work either. The only fix for this is restarting Discord client-side; the RP status disappears then. Time doesn't seem to play a role either, as the RP just stays there.
The send-presence sample that was tested was taken from the 2.0.1 release of this repo. For our own integration, discord-rpc was compiled as a 32-bit DLL from the master source as of commit 5d5bc82 on a Win 7 64-bit machine:
cmake .. -DCMAKE_INSTALL_PREFIX=..\out -DBUILD_SHARED_LIBS=ON -DUSE_STATIC_CRT=ON -DENABLE_IO_THREAD=OFF -G "Visual Studio 14 2015"

But the process doesn't exist in my scenario. For example, this was taken 5 minutes after starting send-presence.exe, typing f into the prompt, and closing it as soon as Discord: ready was shown. As you can see, no send-presence process exists, but the RP info is still displayed with the time left having counted down to 00:00.

Looking at the Send-Presence example 'Discord_Shutdown' is called twice when you 'quit' using the q input option, I actually did this in my own code too accidentally and had the same issue where the rich presence status was stuck displayed even though the process had closed after the 'Discord_Shutdown' function was called (all be it was twice).
After removing the second un-needed execution of the 'Discord_Shutdown' function and testing it seemed to have fixed it in my code/senario... Could this be an issue then with executing 'Discord_Shutdown' more than once? If that's the case even though you shouldn't there probably should be something stopping this issue from even happening in the first place.

Nah, Discord_Shutdown should be safe to call a second time. That sounds like a bug on Discord's side.

Fixed on canary and ptb, should get to stable soon.

Good that this is fixed. I thought I was going crazy.

Should be fixed in stable now.


command to add - roborock S6 MaxV

hi,
In version 0.4.4, i have commande ok
but in version 0.6.2 two or three command have disappeared
Log :
'

roborock.0
2024-04-21 13:34:28.755
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.mop_forbidden_enable" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:34:28.755
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_carriage_status" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:34:28.755
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_mode" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:34:28.621
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_status" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:33:28.664
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.mop_forbidden_enable" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:33:28.664
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_carriage_status" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:33:28.664
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_mode" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:33:28.616
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_status" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-21 13:32:28.695
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.mop_forbidden_enable" has no existing object, this might lead to an error in future versions

`

Hi, thanks for the log.
Should be fixed in the latest dev: https://github.com/copystring/ioBroker.roborock/tree/dev would be great if you could confirm the fix.

it's the same log with 0.6.4
`

roborock.0
2024-04-28 11:00:42.844
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.mop_forbidden_enable" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-28 11:00:42.844
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_carriage_status" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-28 11:00:42.843
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_mode" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-28 11:00:42.759
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_status" has no existing object, this might lead to an error in future versions

`

With this version maps,mqtt is down (KO)

How did you install the dev?

change beta to iobroker

Ok. That's not how this works.
https://github.com/copystring/ioBroker.roborock/tree/dev is not the same as the beta.
So you need to install via the link provided above. You can do that by clicking the GitHub cat in iobroker and then pasting the dev link.

sorry i don't understand to install the  dev branch
i have :

i don't have roborock to dev branch

Try this but it's in German: https://www.smarthome-tricks.de/software-iobroker/adapter-von-github-installieren/

ok thanks you
for Dev version, it's the same log
`

roborock.0
2024-04-29 18:49:11.438
warn
Failed to execute get_room_mapping on robot 7aTL79f4Fx1iyTSXB82Yk6 Error: Local request with id 5 with method get_status timed out after 10 seconds

roborock.0
2024-04-29 18:49:01.428
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.mop_forbidden_enable" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-29 18:49:01.428
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_carriage_status" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-29 18:49:01.428
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_mode" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-29 18:49:01.418
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.deviceStatus.water_box_status" has no existing object, this might lead to an error in future versions

`

Please download the new dev to try again

Maps is already KO.
the log is différent :
`

roborock.0
2024-04-29 19:20:09.002
warn
Failed to map rooms. You need to name your rooms via the mobile app on your phone.

roborock.0
2024-04-29 19:19:08.979
warn
Failed to map rooms. You need to name your rooms via the mobile app on your phone.

roborock.0
2024-04-29 19:18:18.915
warn
Failed to map rooms. You need to name your rooms via the mobile app on your phone.

roborock.0
2024-04-29 19:18:18.790
warn
Failed to execute get_status on robot 7aTL79f4Fx1iyTSXB82Yk6 Error: Local request with id 2 with method get_status timed out after 10 seconds

roborock.0
2024-04-29 19:18:08.826
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.cleaningInfo.2" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-29 19:18:08.825
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.cleaningInfo.1" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-29 19:18:08.824
warn
State "roborock.0.Devices.7aTL79f4Fx1iyTSXB82Yk6.cleaningInfo.0" has no existing object, this might lead to an error in future versions

roborock.0
2024-04-29 19:18:08.724
info
Starting adapter finished. Lets go!!!!!!!

`

maps is ok

the zone is not in the maps.

Please record a video to see the problem.

Video

again error in the log
`

roborock.0
2024-05-08 12:26:47.050
warn
Failed to map rooms. You need to name your rooms via the mobile app on your phone.

`

Video

I don't see a problem. Everything is working as intended. But your map is weird. That is not something I can change. Did you to clean a zone? Does it work as intended?

again error in the log
`
roborock.0	2024-05-08 12:26:47.050	warn	Failed to map rooms. You need to name your rooms via the mobile app on your phone.`

Did you read the message? You have some rooms not named. If you want the adapter to work correctly, you need to name all your rooms.


Add ability to force HTTPs for all services

Does this include the collector?

No this should be everything BUT the collector.


fix(ci): Remove git submodule from shell lint

The shell linter did updates on the submodules, but we're updating the submodules regularly so it just slowing the check down imo

it also should be moved to submodules, e.g. each submodule should have its own shellcheck (probably I did it already)

actually there's no shell at all in the subs, so it was a good removal

In the docker folders there is bootstrap.sh right? Is that not taken into account?

only sometimes, and it is neglectable


Delete or edit notes in tasks

There is, as far as I can see, currently no way of editing or deleting notes from a task.
This would be useful when mistyping notes or accidentally adding notes to the wrong task.

This issue is fixed after including the PR.
Please try it and let me know if something is missing.


[DOCS] review: generated singularity run commands from example YAML files

In this issue, I will provide singularity run commands generated based on example YAML files that we will release. I will ask informatics team to review the commands, to see if it makes sense as examples.

fMRIPrep, --anat-only - please check if this is good for both 23.1.3 (recent release) and 20.2.3 (old version):

singularity run --cleanenv \
        -B ${PWD} \
        -B /test/templateflow_home:/SGLR/TEMPLATEFLOW_HOME \
        -B /home/faird/zhaoc/software/FreeSurfer/license.txt:/SGLR/FREESURFER_HOME/license.txt \
        --env TEMPLATEFLOW_HOME=/SGLR/TEMPLATEFLOW_HOME \
        containers/.datalad/environments/fmriprep-23-1-3/image \
        inputs/data/BIDS \
        outputs/fmriprep_anat \
        participant \
        -w ${PWD}/.git/tmp/wkdir \
        --n_cpus 1 \
        --stop-on-first-crash \
        --fs-license-file /SGLR/FREESURFER_HOME/license.txt \
        --skip-bids-validation \
        --output-spaces MNI152NLin6Asym:res-2 \
        --force-bbr \
        -v -v \
        --anat-only \
        --bids-filter-file "${filterfile}" \
        --participant-label "${subid}"

Path to the example YAML files I'm working on: fMRIPrep 23.1.3, --anat-only, MSI Slurm

Detailed review is probably not needed. I close this issue for now.


If there is a wordpress plug-ins on the better.

What do you think?

I don't understand you. What you mean?


Optimize _encode_multipart_message

See commit messages for more details. The changes were initially implemented on top of v0.53 (due to https://github.com/herrmannlab/dicomweb-client/issues/70#issuecomment-1312807928) but I cherry-picked them on top of master to open a PR.

Thanks!


Let plugin authors define screenshot filename

Corresponding to https://github.com/equinor/webviz-core-components/issues/63

Closed by #269.


Discussion of implementation of copy/paste support for marquee selections

@riknoll @darzu @shakao @jwunderl
Awhile back, we decided to close https://github.com/microsoft/pxt/pull/6990 and implement a full copy/paste-like model for marquee selections in the image editor.
I've started working on an implementation and there are a couple of decisions I've had to make. I'm to the point where I think I want to get some input on the planned approach before proceeding further.
Also I'm really sorry there are so many words here. If there's a potentially better communication medium, I'd be willing to try alternatives.
Focus issues
In the current design of the image editor, there are a couple of places the user could legitimately focus and select text. I want to ensure the marquee copy implementation doesn't interfere with that. Furthermore, focus isn't consistently moved off of these locations when interacting with the image surface.
An example: 1) if you drag out a marquee selection, but then decide you want to change image dimensions, you could 2) double-click in either the width or height input and copy... we wouldn't want to interfere with the text copying here (despite the fact the user has an active marquee selection). Similarly, after interacting with a height/width input, if you then click back into the marquee selection, focus isn't removed from the input. ... and this is only in the current design of the page. I could certainly see more text inputs getting added someday, and I certainly don't want to hardcode handling for today's known inputs.
My plan—that in practice seems to work, but I don't love—is use Selection.type=== 'Range' as a heuristic to indicate that the normal browser copy should be used, even if there is a Marquee tool selection on the image. It seems fine to copy selected marquee data if the document's selection type is either 'Caret' or 'None'.
Existing bugs with copy/paste while image editor is open
It seems like there are some existing issues with the image editor being open and copy + paste making their way down to the underlying blocks surface.
If you select a block before opening the sprite editor, and then Ctrl+C, Ctrl+V with the sprite editor open, you'll find that block duplicated in the blocks editor when you close the sprite editor.
I do not plan on specifically addressing this problem.
Custom clipboard data
I plan on setting custom clipboard data in copy (with setData) and reading it in paste (with getData).
Browser support seems good, and if we actually use the clipboard (rather than setting some local member state to track the image data), then we should get correct behavior for cases like someone switching applications and putting something else on the clipboard.
What data should go on the clipboard? The most natural thing seems like it might be to use the bitmapToImageLiteral style format for the image data. Is that a bad idea? Should that go in the text/plain format of the clipboard too? (That would enable a use case where you could copy a marquee selection; close the sprite editor, and paste into JavaScript.)
Potentially complicated offset logic
I could see a lot of time getting sunk into designing the ideal offset placement for pasted data that handles all of the (literal) "edge" cases and pleasingly cascades down as multiple pastes are made...
I might start with the very simplest possible solution of pasting to 0,0 (or maybe the center of the canvas, or something), to sidestep some of those issues. Please let me know if that sounds too lame to consider merging, and I can invest more here.
Summary (feedback requested)

Focus issues: I plan to use the Selection.type to avoid preventing default on a potentially legitimate copy selection. In addition to requiring marquee tool with an active selection, the browser's document.getSelection().selectionType must not equal Range.
Existing copy/paste focus bugs: There are some existing issues with copy / paste with the sprite editor open. I do not plan on addressing these in the upcoming PR (beyond ensuring I don't introduce new issues).
Custom clipboard data: I plan to use a custom clipboard format with the img`…` formatted data attached. I also plan to include this same data as the text/plain format when copying a marquee selection.
Offset logic: is naive offset logic (like pasting to 0,0) a non-starter? This seems like it could easily be enhanced separately.

Focus issues - could you file a separate bug on that? I think that just needs to get fixed - we shouldn't keep focus on an element after the user has interacted with other things. E.g. /beta has number keys as shortcuts for colors, so this could end up changing the dimensions improperly if you started drawing again, right?
Other copy and paste issue from shortcut leaking to editor - another issue for this as well would be nice, I fixed the same thing happening with delete and backspace a while ago, probably need to be a bit more diligent and prevent propagation of most keyboard events (besides e.g. tab and escape)
The copy/paste behavior using bitmapToImageLiteral seems perfect to me, very nice to keep consistent between blocks / js.
A static offset feels perfect to me - I'd imagine you'd want to position explicitly after each paste anyway, as they get merged / don't stay as separate layers like in photoshop. Maybe (1, 1) so there's a little bit of space around the floating layer when it starts? (Either way, I wouldn't ever worry about putting up something with out all the bells and whistles, it's nicer to add things piecemeal and have time to play around with things instead of getting blocked by minutiae - plus we always really appreciate your PRs / suggestions / participation ~).

I agree with Joey that I think focus issues seem out of scope. Thanks for pointing these out, but I wouldn't hold up this feature for that. Using a workaround like you mentioned sounds good, especially if there's a comment that references the focus issues issue
Yes, out of scope, the team should fix this before the next Arcade release.
img... format sounds perfect because this would allow copy-pasting between programs and between sprites, I really like this idea.
Fixed offset like Joey said sounds great to me. A simple extension of this would be that if copied version ends up mostly off an edge, just flip the offset (e.g. 2,2 offset becomes -2,-2 at the bottom corner)

Like Joey said, thanks for all the effort and thought you've put into this!


Correct the color when true-color

Awesome. Thanks so much. I'm pretty sure someone mentioned this to me in chat once and I didn't understand what they meant.


Fix #81 - allow object in sealed class

also removed code from previous PR that are not used anymore

Thx - do you think a new version could be tagged soon?
Le dim. 8 nov. 2020 à 20:09, Thorsten Hake<EMAIL_ADDRESS>a
écrit :

Merged #82 https://github.com/avro-kotlin/avro4k/pull/82 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avro-kotlin/avro4k/pull/82#event-3971082942, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABA4DJ2U3G56TCT7CHOLZTSO3UE7ANCNFSM4TM4RRRQ
.

--
Gilles Barbier
CEO - Zenaton
+33678094540
Paris
zenaton.com
https://s3.sendassets.io/s2/cyp1zmru
https://s3.sendassets.io/s2/pvub0sen
https://s3.sendassets.io/s2/6nb35otd
https://s3.sendassets.io/s2/ptv1o4tn
Join us in supporting the World Health Organization. Install the banner now!
https://s3.sendassets.io/s2/l3i5qum3


BEM doesn't recommend nested childs

BEM doesn't recommend nested childs

Yep, I know.
And I don't encourage this. That's why there is no appropriate example in readme.
But I want to leave this ability for developers who use nested elements.


It does not upload the solution when there are too many submitted

When there are more than 13 submitted in Leetcode, it would not upload the solution.
I try to reload Leetcode and resubmit, but it does not work.
And in the new problem, it works fine.

hmm that's weird. it works for me.
fixed similar issue. see #127


merge from konsum-frontend-svelte

package.json

chore(deps): add ^8.3.11 remove ^8.3.9 (devDependencies.npm-pkgbuild)

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

v1.31.14
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


breaks foxbox

the foxes no longer sit in the boxes when this is installed alongside foxbox

I'll try to add compatibility for it soon


NIFI-7917 Upgrade avro to 1.10 in nifi-nar-bundles/nifi-standard-serv…

…ices/nifi-record-serialization-services-bundle/nifi-record-serialization-services

Updated avro version to 1.10
Replaced org.codehaus.jackson.* with com.fasterxml.jackson classes as previously the former was used and provided transitively through avro 1.8
Added org.tukaani:xz:1.5 and org.xerial.snappy:snappy-java:<IP_ADDRESS> dependencies explicitly as those were also coming through avro

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[x] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

I noticed there are other modules that use Avro. Can these be upgraded as well?

nifi-avro-record-utils
nifi-hadoop-libraries-nar
nifi-database-utils
nifi-cassandra-processors
nifi-hwx-schema-registry-service
nifi-parquet-processors

The build fails due to some remaining references to org.codehaus.jackson.map.ObjectMapper which should be replaced with com.fasterxml.jackson.databind.ObjectMapper:
AtlasAPIV2ServerEmulator.java
ITReportLineageToAtlas.java
QueryCassandraTest.java
GetHDFSEvents.java
TestAvroRecordConverter.java
ConvertJSONToSQL.java
TestConvertJSONToSQL.java

Also two files refer to org.codehaus.jackson.JsonNode and should be changed to com.fasterxml.jackson.databind.JsonNode:
ConvertJSONToSQL
TestConvertJSONToSQL

AtlasAPIV2ServerEmulator refers to DeserializationConfig.Feature which needs to be replaced with DeserializationFeature

I took the liberty of making the changes and putting up a commit, feel free to cherry-pick if you wish.


error

Heelo my friend , nice code .
It is giving me an error . can you have a look .
Thanks

hi there, the ui in your picture does looks like Vueditor, maybe you should open an issue  here https://github.com/PeakTai/vue-html5-editor


Update README.md

Adding Hakiri badge to highlight some potential dependency security warnings.

Coverage decreased (-0.04%) to 96.129% when pulling 7f86eaf529ce756ebe0930cc7b0374accf5f9ecb on vasinov:patch-1 into fb502c755ed8801aa5d4575f18b86a9b7ecf9f5e on intridea:master.


3553 overlay protocol

Removes the protocol of the placeholder image to avoid problems with the https/http domains.
Original issue: #3553

@xavijam: very simple fix, do you think is ok to do it this way?

Yep, it should fix the problem you mention :)

why not use assets inside cartodb?
It's a pain to have those in a different urls, specially for onpremise

Frontend tests were OK :+1: (details)

cdb.config.get('assets_url') ?

I don't remember the reason behind using that URL. Probably was 1) ease of use and 2) shortness… but I'll change it to avoid problems with onpremise.

@xavijam: check again, please please.

thanks \o/

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Fixed and tested in staging.

bump

LGTM :es:


Fixing statusCode checks for sockRequest

Updated status checking for sockRequest and sockRequestRaw, along with necessary test logic changes.
The statusCode is now checked in the sockRequest func, and we no longer assume a statusCode of 200 (http.StatusOK) for all return values. The caller specifies what statusCode is expected, by passing it in as an additional parameter to sockRequest. Some of the tests call sockRequestRaw directly, so I added status checking from within the test itself.
All integration-cli tests pass currently.
Fixes #12449
Signed-off-by: Megan Kostick<EMAIL_ADDRESS>

gofmt :D

@jfrazelle Oh man, thought I did! Let me fix that :)

@kostickm so now you can pass a statusCode to sockRequest to have it checked or check it yourself as a return value?

@runcom For tests calling sockRequest you pass the status code and have it checked, but there are a handful of tests that call sockRequestRaw directly (which is called by sockRequest), so those I checked the return value in the test themselves. I am not sure the reason for using sockRequestRaw instead of sockRequest.

I was trying my best to interpret everyone's discussion in #12449 :) But definitely want input on what everyone thinks the 'standard' should be.

you are right but I haven't understood myself what the final implementation should have been lol plus I don't have vote on what to do here

@runcom Thank you for your input though! And I agree this flow/func-use is a bit concerning. I figured it best to get something out here to keep the discussion going, and hopefully get a decision nailed down.

Dunno, seems inconsistent. I vote for just comparing status outside function and remove this inside function.

+1 @LK4D4

@LK4D4 Should have kept a separate branch since I did that first, but isn't tough to switch back :) Do we still want to keep the always checking for status code 200 removed from sockRequestRaw though?

@kostickm Yeah, I want :) If we'll merge gocheck it will be only one line to check it.

@LK4D4 Okay cool! Maybe I should wait for that to be merged first and then finish updating these tests.

Haha, I said if :) I'm not sure that it will be merged.

@LK4D4 :) Good point, I'll get the initial changes done then, and can always spruce it up 'if' gocheck gets merged.

@LK4D4 That's great gocheck got merged! I'll take a look and see what can be done about the status check with your updates.

@LK4D4 I think this is more what you're looking for, using c.Assert( ) is pretty handy.
I'd like to update other places in the tests to use the gocheck funcs, but figured I'd keep that for a separate PR.
Let me know what you think :)

@kostickm Yeah, you can just refactor sockRequest here.

@kostickm the code looks great, unfortunately it needs to be rebased to be merged.

@calavera Thanks! This has been one for endless rebasing. Will get it rebased though :)

I really feel your pain, let's try to merge it as soon as it's good to merge.

code LGTM, btw

Rebased.

loving c.Assert here :heart:

awesome LGTM


Fixed password display when logging in with some Windows terminal clients

Closes #443

:+1: (assuming CI passes)


consume-a-rest-service

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: consume-a-rest-service
Document link: consume-a-rest-service.md
My Issue/Suggestion

The content of the link in step 1 has changed and is different then the information displayed in the tutorial. This is confusing when trying to interpret the assignment.
Step 3 calls for selecting a 'Raw Data tab'. It is very confusing what is meant by this. Where do i find a 'Raw Data tab'? Is there a tab somewhere with a caption 'Raw Data'?

Thanks for logging this issue, Ries! Step 3 is indeed confusing; you're just supposed to copy the JSON data you see in your browser. "Raw Data" might be available in some browsers but isn't necessary here. I'll make sure to update the guide.
Regarding the data you see when you follow that URL: that is expected to be changing (as it's from Wikipedia); but I can see how this is adding more confusion than being helpful. I'll discuss if we should change or remove this instead.


Updated list of sponsoring establishments

Updated the list to include new entries, and change existing entries from online to print and send applications.
Zendesk: https://govuk.zendesk.com/agent/tickets/1213457
Pivotal: https://www.pivotaltracker.com/story/show/111515402

The diff isn't showing..

@erikse Does the diff not show when there's a large number of changes? It was quicker to paste in the full list - otherwise John would have had to have gone through them all and pick out the changes.

The current file has 3007 lines, and this PR will remove 3001 of the existing lines, and add 3032 new ones - did we really change that many?

It was a complete swap. They sent us a spreadsheet that didn't indicate which ones lines were new sponsors were etc.

Ok, if we can get a CVS dump from that spreadsheet, we can update the existing values with a script in a new PR.

This PR has been superseded by #2243


Cannot add new sections to new installed package where there is NO sample data

OVERVIEW
When installing Grants Kit Package on the new org that has not yet Grants section data, user needs to create new sections or import some data.
While Adding a new section the success message display incorrect notification (New Grant application created) while this is just to set up new section only.
ISSUE
This error is casing the problem
FATAL_ERROR Class.GGW_ApplicationCtrl.createNewSection: line 157, column 1 System.NullPointerException: Attempt to de-reference a null object
The code is relying on existing sections data to get sort order count for new section, but in new org there is no section data so this is null - causes this error
Workaround
Add some section sample data
BACKGROUND/REFERENCE
Add defensive code when no sections available, initialize new section sort order as default 1
Fix notification message in web component

Fixed in commit of #65


Add configuration options for SwiftMailer to config.json

Allow for SMTP server configuration inside config.json or as parameters from Zabbix.
Local config.json will superseed Zabbix configuration (log will contain warning).

v1.12 has config.json and code to configure smtp_server and smtp_port.
Next step: allow for transport encryption via none, starttls, ssl setting (smtp_secure=none|ssl|starttls)

v1.13 has config.json and code to configure smtp_transport. Possible options are 'tls' and 'ssl'. Any other content will switch to 'none'. Tested on Postfix - no issues found. Note that user has to specify the correct port associated (80,465,687).

v1.14 has config.json and code to configure smtp_strict. Possible options are 'yes' or 'no'. Any other content will switch to 'yes'. Unable to test myself although the option 'no' does work with my strict configuration.


Add a license to the repository

I recently came across your project hello_qiskit and I noticed that the repository github.com/barrackHa/hello_qiskit currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thank you!


[FEATURE REQUEST] Homepage Text Alignment Enhancement

Description
Is your feature request related to a problem? Please describe.
No. It's an enhancement in the existing design and alignment.
Describe the solution you'd like
Moving the content more in the center(equally centered from all sides) and well aligned.
Describe alternatives you've considered
Alignment and styling of homepage content
Additional Information
Screenshots or Mockups (optional)
Current Alignment

Expected Updated Alignment

Priority (optional)
Medium
Additional Notes (optional)
N/A

@akshitagupta15june


Refactor CreateCustomerTokenService

CreateCustomerTokenService requires an order as the first parameter, but the API doesn't need it but just user and address info.
We can refactor it by asking for an email and billing address.

Fixed by https://github.com/solidusio-contrib/solidus_klarna_payments/pull/34


Dockerfile: remove superfluous ARG line

We spotted this during the workshop. Thanks for the PR (and the implicit reminder 😆)!

We spotted this during the workshop. Thanks for the PR (and the implicit reminder 😆)!

Exactly! I was in the workshop, and that was when I made the PR. We had the brief chat about python being tested but Perl not :-)
thanks for the workshop, I enjoyed it, even though of course you can never get much done in a workshop setting it was insightful

Yeah, the "We spotted this during the workshop" mention was exactly because of that! Hopefully, someone will see your comment and will start allocating some time for fuzzing the Perl interpreter 👀
I'm soooo happy that you liked it! You surely have implicit limitation during a workshop (different operating systems, different architectures, random errors, limited time, etc.), but I hope all the people left the room with the belief that the people working of fuzzing infrastructure (CC: @DonggeLiu and the OSS-Fuzz crew ;) did an awesome job on lowering the barrier to entry for continuous fuzz testing for OSS projects.


How to setup bigben multinode cluster on different machines

Hi Abhishek
For non docker environment.
make use of overrides.yaml file or alternatively You can modify bigben.yaml
for ex:
hz:
network.members: <IP_ADDRESS>:5701,<IP_ADDRESS>:5701,<IP_ADDRESS>:5701
modify app_run.sh as per your requirement as it contain the startup command.


added support for exclude a final directory

Added support for https://github.com/Chia-Network/chia-blockchain-gui/issues/204

Hi @seeden,
Thanks!!


Color converters / extensions

Description of Change
Added Color extensions and Converters.
Also, to facilitate the writing of the converters I included my "BaseConverters". Since almost all Converters are written the same way (Type testing, NotImplementedException for OneWay ...).
Bugs Fixed

Answer to issue #138

API Changes
Added:

string Color.ToRgbString() (With converter)
string Color.ToRgbaString() (With converter)
string Color.ToHexRgbString() (With converter)
string Color.ToHexRgbaString() (With converter)
string Color.ToCmykString() (With converter)
string Color.ToCmykaString() (With converter)
string Color.ToHslString() (With converter)
string Color.ToHslaString() (With converter)
Color Color.WithRed(double newR)
Color Color.WithGreen(double newG)
Color Color.WithBlue(double newB)
Color Color.WithAlpha(double newA)
Color Color.WithRed(byte newR)
Color Color.WithGreen(byte newG)
Color Color.WithBlue(byte newB)
Color Color.WithAlpha(byte newA)
Color Color.WithCyan(double newC)
Color Color.WithMagenta(double newM)
Color Color.WithYellow(double newY)
Color Color.WithBlackKey(double newK)
byte Color.GetByteRed() (With converter)
byte Color.GetByteGreen() (With converter)
byte Color.GetByteBlue() (With converter)
byte Color.GetByteAlpha() (With converter)
double Color.GetDegreeHue() (With converter)
double Color.GetPercentBlackKey() (With converter)
double Color.GetPercentCyan() (With converter)
double Color.GetPercentMagenta() (With converter)
double Color.GetPercentYellow() (With converter)
Color Color.ToInverseColor() (With converter)
Color Color.ToBlackOrWhite() (With converter)
Color Color.ToBlackOrWhiteForText() (With converter)
Color Color.ToGrayScale() (With converter)
bool Color.IsDarkForTheEye()
bool Color.IsDark()

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

This will be great to have in toolkit! Thanks, @framinosona!

If you have some answers for me @framinosona I think we're pretty close to merging this :)

@framinosona any update on this ?


Plugin should ignore .eslintrc.json when configuration argument is present

If a user passes a configuration into pipe(eslint()) and a .eslintrc.json file is present, then the plugin should ignore the external configuration.
Otherwise, the internal configuration is getting ignored silently.

No. Use useEslintrc option.

@shinnn Thanks, but shouldn't you at least output a warning message if the user provides a configuration argument and useEslintrc is undefined?

@cowwoc gulp-eslint is just a wrapper of ESLint API. So if you'd like to suggest changes in the default bahavior, for example warning message, open a new issue on the ESLint issue tracker instead. If the ESLint team accept your proposal, of course gulp-eslint happily follows it.

@shinnn Makes sense. Thanks for the clarification.


Variant Product Tabbing switch issue

PHP Version: 8.2 or higher   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Expected behaviour:
Hello Support,
After that, I went back to the variant product list page unable to see the variants list & showed there are no variants in the product.
Actual behaviour:
Hello Support,
I had an issue with the Shopware backend while checking the product in the shop. When I switch to a variant product and open the variant product then go to the SEO or any other tabs of their variant product then save the changes or what we can change.
After that, I went back to the variant product list page unable to see the variants list & showed there are no variants in the product.
shopwareDefaultIssue.webm
How to reproduce:
via new release of in the same release

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-38264
[created from NEXT-38264, comment 494455]

Hello Support I have raised the issue as unable to went back to the main product & unable to see the proper variant list while we are in variant product tabs....
shopwareDefaultIssue.webm

Hello Support I have raised the issue as unable to went back to the main product & unable to see the proper variant list while we are in variant product tabs....


Suggestion list doesn't show elements before first search.

In 5.0.0-alpha3 my suggestion list is empty, then when I search some name, the next time the suggestion list show all items.
How can I change the list to show only items that start with the inserted text and not that containt the text.
Is it possible to change the fade effect of the suggestion list items?

I am still unable to get my suggestion list in the first search, I provocated this bug in my code or anyone more have it?

This bug is part of new bugs in SearchAdapter ... I want to fix it in this week.

Okay! Can you add this:
"@lapism ah, should be great that those check boxes can have on click listener, so when the user click them automatically it filters, without the necessary of the user insert text and touch search button"
To a feature request list?

Yes, it is already there +1:


Travis

Updates to the travis.yml file to be able to run functional tests against a live Riak instance and include code coverage.

:+1: af71624


StringPool loses strings when growing

This can be reproduced on 0.23.1 and master.
require "string_pool"

pool = StringPool.new
pool.get("foo")

(0..500).each do |i|
  # add random stuff to grow the pool
  random_str = (1..8).map{ ((rand(2)==1?65:97) + rand(25)).chr }.join
  pool.get(random_str)

  # id for "foo" is not always the same, lost when resizing/rehashing?
  puts "%3d %d" % {pool.size, pool.get("foo").object_id}
end

On my machine the id for "foo" always changes after the pool size reaches 55, and again at 335.

I can't reproduce this in master

Example: https://carc.in/#/r/31k4

This was true before new StringPool implementation. Old StringPool recreated strings on resize.
New StringPool preserves strings, so it is not reproducible on master.

I must have compiled an older version than I previously thought. Recompiled master now and I can't reproduce it. Thanks.

Use 0.24 and you'll get expectable behavior.

For further reference, there's no need to recompile the compiler for this. Just use bin/crystal or set CRYSTAL_PATH to the src directory in this repo


docs(examples): add flash tutorials to spi related examples

add flash tutorials to spi-demo, sdcard-demo, sdcard-gpt-demo
add blri command, use blri by cargo blri <file_path>

Please adhere the test results, need the picture of two adapter board plug sequence transfer and screenshot of serial port output to prove that the test is no problem, thx.

Tested on M1sdock, all examples work well.
Adapter boards:

Pinout:

Results:

spi-demo

sdcard-demo

sdcard-gpt-demo


How to tail a remote log and have goaccess parse it locally in real-time

I have read all the issues about loading the dashboard from a remote server's access.log file but I have not been able to get it to work. I can get a static report.html page but it will not load live time.
This is what I've used to set-up the pipe, which does successfully pull the access.log from the remote server. In my console, I do see the line number increases in live time as I reload the target server webpage but that change is not reflected in the report.html dashboard.
ssh -t user@remote-server 'tail -f -n +1 /var/log/apache2/access.log' | goaccess -o /var/www/html/report.html --log-format=COMBINED --real-time-html -
I've tried a bunch of stuff but have not been able to get this to work. Any ideas/updates/tips?

Please take a look at the examples in the man page (server section). I think I posted this not long ago in twitter.
# ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess -

Note: SSH requires -n so GoAccess can read from stdin. Also, make sure to use SSH keys for authentication as it won't work if a passphrase is required.

Please take a look at the examples in the man page (server section). I think I posted this not long ago in twitter.
# ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess -

Note: SSH requires -n so GoAccess can read from stdin. Also, make sure to use SSH keys for authentication as it won't work if a passphrase is required.

tried your command above but now the report.html is not created in the /var/www/html/ directory.
I am using an ssh alias from config, for example:
ssh -n alias 'tail -f /var/log/apache2/access.log' | goaccess - -o /var/www/html/report.html --log-format=COMBINED --real-time-html
report.html isn't being created

tried your command above but now the report.html is not created in the /var/www/html/ directory.
I am using an ssh alias from config, for example:
ssh -n alias 'tail -f /var/log/apache2/access.log' | goaccess - -o /var/www/html/report.html --log-format=COMBINED --real-time-html
report.html isn't being created

I played around with this some more I can get Server A dashboard to display Server A's access.log in real time. I had to first generate a static report.html, then re-run the command with the --real-time-html operand.
But I still cannot get Server A dashboard to display Server B's access.log. I confirm the following:

I can ssh from Server A to Server B using only keys
I can output Server B's access.log file on Server A's standard out

I cannot:

create a static report on Server A using pipe from Server B (i.e., I cannot ssh -n ServerB_alias  'tail -f /var/log/apache2/access.log' | goaccess - -o /var/www/html/report.html --log-format=COMBINED
When I try to do this, I do not get an error, but nothing is created in Server A's /var/www/html directory
I also cannot ssh -n ServerB_alias 'tail -f /var/log/apache2/access.log' | goaccess - -o /var/www/html/report.html --log-format=COMBINED --real-time-html

I played around with this some more I can get Server A dashboard to display Server A's access.log in real time. I had to first generate a static report.html, then re-run the command with the --real-time-html operand.
But I still cannot get Server A dashboard to display Server B's access.log. I confirm the following:

I can ssh from Server A to Server B using only keys
I can output Server B's access.log file on Server A's standard out

I cannot:

create a static report on Server A using pipe from Server B (i.e., I cannot ssh -n ServerB_alias  'tail -f /var/log/apache2/access.log' | goaccess - -o /var/www/html/report.html --log-format=COMBINED
When I try to do this, I do not get an error, but nothing is created in Server A's /var/www/html directory
I also cannot ssh -n ServerB_alias 'tail -f /var/log/apache2/access.log' | goaccess - -o /var/www/html/report.html --log-format=COMBINED --real-time-html

Sounds like a permissions issue maybe? I'd start with the basics:
Try to see if the data comes as it should and with the right log format. e.g., should output a stream of COMBINED log-format data without requiring credentials
 # ssh -n root@server 'tail -f /var/log/apache2/access.log'

Try outputting to the terminal to see if that works fine:
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED

I'd try to output to your home dir
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED -o /home/user/report.html --real-time-html

Let me know what how it goes.

Sounds like a permissions issue maybe? I'd start with the basics:
Try to see if the data comes as it should and with the right log format. e.g., should output a stream of COMBINED log-format data without requiring credentials
 # ssh -n root@server 'tail -f /var/log/apache2/access.log'

Try outputting to the terminal to see if that works fine:
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED

I'd try to output to your home dir
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED -o /home/user/report.html --real-time-html

Let me know what how it goes.

Sounds like a permissions issue maybe? I'd start with the basics:
Try to see if the data comes as it should and with the right log format. e.g., should output a stream of COMBINED log-format data without requiring credentials
 # ssh -n root@server 'tail -f /var/log/apache2/access.log'

Confirmed. I am able to successfully output remote server's access.log file.

Try outputting to the terminal to see if that works fine:
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED

This works but with a weird behavior. I am able to get output to terminal but it remains static until I resize by terminal window (I'm on OSX). If I resize the terminal window in OSX, I get an updated dashboard. If I leave as is, it is static, despite repeated refreshes/requests made to the remote (i.e., target) web server.

I'd try to output to your home dir
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED -o /home/user/report.html --real-time-html

Ok, this worked but I had to run as sudo. I am running as user$, who is sudo. /var/www/html is owned by $user, so it weird that it wouldn't create. I had to

ssh -n root@server 'tail -f /var/log/apache2/access.log' | **sudo** goaccess . . . .

Let me know what how it goes.

Sounds like a permissions issue maybe? I'd start with the basics:
Try to see if the data comes as it should and with the right log format. e.g., should output a stream of COMBINED log-format data without requiring credentials
 # ssh -n root@server 'tail -f /var/log/apache2/access.log'

Confirmed. I am able to successfully output remote server's access.log file.

Try outputting to the terminal to see if that works fine:
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED

This works but with a weird behavior. I am able to get output to terminal but it remains static until I resize by terminal window (I'm on OSX). If I resize the terminal window in OSX, I get an updated dashboard. If I leave as is, it is static, despite repeated refreshes/requests made to the remote (i.e., target) web server.

I'd try to output to your home dir
 # ssh -n root@server 'tail -f /var/log/apache2/access.log' | goaccess - --log-format=COMBINED -o /home/user/report.html --real-time-html

Ok, this worked but I had to run as sudo. I am running as user$, who is sudo. /var/www/html is owned by $user, so it weird that it wouldn't create. I had to

ssh -n root@server 'tail -f /var/log/apache2/access.log' | **sudo** goaccess . . . .

Let me know what how it goes.

Quick question, are you in v1.4.5? Also, are you able to test this out on a linux machine, I'm curious if this is isolated to OSX... you mentioned sudo, I wonder if #2009 could help in your case.

Quick question, are you in v1.4.5? Also, are you able to test this out on a linux machine, I'm curious if this is isolated to OSX... you mentioned sudo, I wonder if #2009 could help in your case.

Quick question, are you in v1.4.5? Also, are you able to test this out on a linux machine, I'm curious if this is isolated to OSX... you mentioned sudo, I wonder if #2009 could help in your case.

Im running it on Ubuntu -- i just ssh into the box from my osx laptop.
Im running v1.2

Quick question, are you in v1.4.5? Also, are you able to test this out on a linux machine, I'm curious if this is isolated to OSX... you mentioned sudo, I wonder if #2009 could help in your case.

Im running it on Ubuntu -- i just ssh into the box from my osx laptop.
Im running v1.2

Would you be able to build from development? There have been several changes since v1.2 - I just pushed #2009 which may address the sudo issue. If you are on OSX, you probably can run:
brew install --HEAD goaccess

Would you be able to build from development? There have been several changes since v1.2 - I just pushed #2009 which may address the sudo issue. If you are on OSX, you probably can run:
brew install --HEAD goaccess

let me see if I can. My envir is on Ubuntu, I'm only using OSX's terminal to ssh into my box.

let me see if I can. My envir is on Ubuntu, I'm only using OSX's terminal to ssh into my box.

I updated to 1.4

I updated to 1.4

Glad to hear it got resolved.
Closing this. Feel free to reopen it if needed.

Glad to hear it got resolved.
Closing this. Feel free to reopen it if needed.

Glad to hear it got resolved.
Closing this. Feel free to reopen it if needed.

Thank you! And thanks for your help!

Glad to hear it got resolved.
Closing this. Feel free to reopen it if needed.

Thank you! And thanks for your help!


log when retrying actions

retry.retry has an option to add a log every time it retries a failing operation.

/test all


Refactor json service

This properly resolves the issue addressed in #213 by throwing an ErrorResponse instance when the response is parseable as JSON and contains a .error value.
The unit test code checked into #215 will need some minor updates if this merges first.

The JsonService commits look good, can you cherry pick them onto a separate merge request. The other two commits seem to be part of two other merge requests,....

nice work, simplifies code quite a bit


HF Spaces demo not working.

Huggingface demo not working

Hi, thanks for your interest in the SeViLA project. We have re-launched the demo page, and you may try it again.


Fix to enable to be used from ObjC

To use SlideMenuOption class or options property from ObjC source code, this modification is needed.

Thank you pull request !
The swift1.2 to survive and get again pull request from latest from corresponding.


Using Bulk Copy to insert uniqueidentifier to Azure Data Warehouse results in invalid column type

Driver version
<IP_ADDRESS>
SQL Server version
Microsoft Azure SQL Data Warehouse - 10.0.18045.0 Oct 11 2022 03:10:58 Copyright (c) Microsoft Corporation
Client Operating System
Microsoft Windows 11 Pro
JAVA/JVM version
openjdk version "17.0.5" 2022-10-18
OpenJDK Runtime Environment Temurin-17.0.5+8 (build 17.0.5+8)
OpenJDK 64-Bit Server VM Temurin-17.0.5+8 (build 17.0.5+8, mixed mode, sharing)
Table schema

TABLE_CATALOG
TABLE_SCHEMA
TABLE_NAME
COLUMN_NAME
ORDINAL_POSITION
COLUMN_DEFAULT
IS_NULLABLE
DATA_TYPE
CHARACTER_MAXIMUM_LENGTH
CHARACTER_OCTET_LENGTH
NUMERIC_PRECISION
NUMERIC_PRECISION_RADIX
NUMERIC_SCALE
DATETIME_PRECISION
CHARACTER_SET_CATALOG
CHARACTER_SET_SCHEMA
CHARACTER_SET_NAME
COLLATION_CATALOG
COLLATION_SCHEMA
COLLATION_NAME
DOMAIN_CATALOG
DOMAIN_SCHEMA
DOMAIN_NAME

SQLPool
dbo
hc_test_uniqueidentifier
_uniqueidentifier
1

YES
uniqueidentifier

Problem description
When using Bulk Copy to insert a row with a column containing the uniqueidentifier data type to Azure Data Warehouse, I will receive this error: Invalid column type from bcp client for colid 1.
The issue occurs only when using Bulk Copy.
Expected behavior
The row is inserted without any errors.
Actual behavior
I receive the error below:
Error message/stack trace
com.microsoft.sqlserver.jdbc.SQLServerException: 110802;An internal DMS error occurred that caused this operation to fail. Details: Please use this Error ID when contacting your Administrator for assistance. EID:(d65450e6dc884758a9be76dbc1d88e25) SqlNativeBufferBufferBulkCopy.WriteTdsDataToServer, error in OdbcDone: SqlState: 42000, NativeError: 4816, 'Error calling: bcp_done(this->GetHdbc()) | SQL Error Info: SrvrMsgState: 1, SrvrSeverity: 16,  Error <1>: ErrorMsg: [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Invalid column type from bcp client for colid 1. | Error calling: pConn->Done() | state: FFFF, number: 418, active connections: 3', Connection String: Driver={pdwodbc17e};app=TypeD00-DmsNativeWriter:DB40\mpdwsvc (24828)-ODBC;autotranslate=no;trusted_connection=yes;server=\\.\pipe\DB.40-feeb931069d5-0\sql\query;database=Distribution_58

Hi, we'll do some investigating and get back to you. We're in the middle of performing hotfix releases so replies might be delayed.

Hi @hchenatsafe, I'm getting the same error returned as you in my own reproduction. But, I was able to get a workaround. In the meantime, could you try and see if this works for you as well? I found that if I do a CAST to UNIQUEIDENTIFIER I was able to perform the bulk insert.
So, my prepared statement with the cast before insert looked like the following:
DECLARE @val UNIQUEIDENTIFIER = CAST(? AS UNIQUEIDENTIFIER); INSERT INTO uidTable VALUES(@val);
I'm still investigating this issue to narrow down the cause (although to me this seems like an Azure data warehouse problem). I'll keep you posted.

As suspected, this is an Azure data warehousing issue. I discovered a similar issue here. It seems that there was a fix made, but for UNIQUEIDENTIFIER it was missed. I recommend reaching out to support through the Azure portal as this is a server side issue and not specifically a driver issue.

Hello @tkyc ,
Thank you for investigating and your suggestion of using CAST as a workaround.
Our application relies on using the raw Bulk Copy method rather than batch insert. It seems this means we don't have easy access to the CAST workaround.
Is the CAST workaround still doable using our Bulk Copy method? Or could there be other workarounds we can try?

@hchenatsafe Unfortunately, the workaround won't work with the raw bulk copy. The only recommendation I can make is to use the bulk copy API for batch insert which should allow you to do a cast.


Why is init.asm not linked into libhermit.a?

Currently the x86 boot code is compiled into the Linux kernel and copied to hermit_trampoline during the start of a Hermit app.
I don't see a reason, why we can not directly link it into the ELF binary of a Hermit app or libhermit.a
Irrespective of this, I moved init.asm to the Linux repo, renamed it boot.asm and integrated into the kbuild system. If it's only used by the kernel, we also should build it there.
See: stv0g/linux@21e290d9079999ea586c45ebd339f94c380c95c7

It is only a backup of my trampoline code, which I used to jump from 16bit over 32bit into 64bit mode. It is indirectly integrated in the Linux code, where I use the code as string to boot HermitCore on a specific processor.

Ah okay. Is there a reason that the trampoline code is not part of libhermit.a?
Do you want me to submit a PR to integrate the code into the kbuild system?
Adding it to Linux repo makes more sense to me.

A x86 processor starts in 16bit mode. Consequently, we have no paging, no virtual address space and the (physical) start address must be below 1MB. Bad idea to start more than one kernel.
Yes, the integration of the trampoline code has to be in the build system. The wrong location has more historical reasons. I want to migrate the code.

Close issue because I merged stv0g/linux@21e290d


Change the default value for the simulation period to 50 years

Hi @wouterpeere ,
I have heard from people in Germany that we normally run simulations for 50 years. We need this as well for applications. What do you think about this? Do you need 20 years in Belgium?
With best regards
@tblanke

In Belgium, this depends. Practitioners often use 20/25 years for investment exercises, but from a sustainability point of view, we often follow the 40 years as suggested by BREEAM. Is it fine if we meet at 40 years, instead of the current 20?

Yes


Running code file.ext from the command line, then saving, prompts that this file already exists

Issue Type: Bug

Change directory to your favorite/arbitrary directory, cd /some/dir.
Open this directory in VS Code via code .
Change directory to somewhere else completely.
Run code some_file.txt from the command line. (Assumption: File does not already exist.)
Save the file.

We get a prompt that says that this file already exists.
VS Code version: Code 1.51.1 (e5a624b788d92b8d34d1392e4c4d9789406efe8f, 2020-11-11T01:11:34.018Z)
OS version: Darwin x64 20.1.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5250U CPU @ 1.60GHz (4 x 1600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
8.00GB (0.02GB free)

Process Argv
. --crash-reporter-id 1710ea2b-7fcf-4d53-9484-6535956bb330

Screen Reader
no

VM
0%

I cannot reproduce this given the steps provided. Can you reproduce this with extensions disabled? By running code --user-data-dir(then you run with fresh settings)? By using code-insiders?

@egamma I've tried several times to invoke the same behavior but cannot reproduce it myself. Sorry!


docs: Simplified names of Github Actions pipelines and updated README

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Validate .NET 4.6

This is still in progress. I ported the logic from the PS script to a reusable method bool Test(Version minVersion) in a new class NetFrameworkUtil. The class caches the discovered versions.
I moved the .Net 4.5 validation into program.cs (and removed previous hack logic that interpreted based on capability scan).
I'm planning to plugin the 4.6 validation in two ways:

Warn during config with a message indicating that it's required for TFVC only and not required if using Git (similar to the TEE EULA verbiage).
Warn during TFVC get sources. One thing that concerns me here is that folks often only look at the end of the log. So I'm still trying to figure out a good approach here. I need to think through it more, but am thinking about something like, warn at the beginning and then error at the end if the step failed.

looks good.


test: write more unit tests to improve coverage

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.25% :tada:

Comparison is base (355ee0e) 98.78% compared to head (23a0026) 99.03%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
+ Coverage   98.78%   99.03%   +0.25%
==========================================
  Files          47       47
  Lines        4860     4894      +34
==========================================
+ Hits         4801     4847      +46
+ Misses         59       47      -12

Files Changed
Coverage Δ

include/emio/detail/format/formatter.hpp
97.64% <100.00%> (+0.90%)
:arrow_up:

include/emio/detail/utf.hpp
100.00% <100.00%> (ø)

include/emio/writer.hpp
100.00% <100.00%> (ø)

test/unit_test/detail/test_ct_vector.cpp
100.00% <100.00%> (ø)

test/unit_test/test_dynamic_format_spec.cpp
100.00% <100.00%> (ø)

test/unit_test/test_format.cpp
99.61% <100.00%> (+0.01%)
:arrow_up:

... and 3 files with indirect coverage changes

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


v70, batch build script is broken

Describe the bug
build script is broken
To Reproduce
run
./build-ubuntu-20.04.sh
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS] Ubuntu 20.04

How did you get the source code?

https://github.com/oliverw/miningcore/releases/tag/v70
downloaded tarball

https://github.com/oliverw/miningcore/releases/tag/v70
downloaded tarball

That's the problem. Clone the repo please.
I know it should be documented but I'm waiting for package GitVersion.MsBuild to fix that.


WIP: Create OVN-K deploy script

What this PR does
Before this PR:
After this PR:

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

/test pull-kubevirt-e2e-k8s-1.30-sig-network

will reopen if needed, it did work locally, but might worth to patch according the newest findings

replaced by https://github.com/kubevirt/kubevirtci/pull/1193


Add gfx941/942 NN fp32 yamls for large m size

JIRA: SWDEV-444316

CI: http://math-ci.amd.com/blue/organizations/jenkins/gfx94x%2Fprecheckin%2FhipBLASLt/detail/PR-637/1/pipeline

problem's N size is 32 which is larger enough to use MI16x16 or MI32x32. I am curious that have you tried both MI16x16 and MI4x4? And MI4x4 is the winner?

problem's N size is 32 which is larger enough to use MI16x16 or MI32x32. I am curious that have you tried both MI16x16 and MI4x4? And MI4x4 is the winner?

I also have tried MI16x16 and MI32x32 with minimum macro tiles, but encounter BufferStoreOffsetLimitCheck errors like below.
BufferStoreOffsetLimitCheck(MT1:16): (67108864 * 4 * 16 <<PHONE_NUMBER>)
BufferStoreOffsetLimitCheck(MT1:32): (67108864 * 4 * 32 <<PHONE_NUMBER>)
It shows that MT1 should be less than 16, which implies MI4x4 is the one and only choice. Therefore, I decide to use MT1=8.
After tuning, the final solution is MT256x8x8 with MI4x4.

BufferStoreOffsetLimitCheck

got it

problem's N size is 32 which is larger enough to use MI16x16 or MI32x32. I am curious that have you tried both MI16x16 and MI4x4? And MI4x4 is the winner?

I also have tried MI16x16 and MI32x32 with minimum macro tiles, but encounter BufferStoreOffsetLimitCheck errors like below. BufferStoreOffsetLimitCheck(MT1:16): (67108864 * 4 * 16 <<PHONE_NUMBER>) BufferStoreOffsetLimitCheck(MT1:32): (67108864 * 4 * 32 <<PHONE_NUMBER>) It shows that MT1 should be less than 16, which implies MI4x4 is the one and only choice. Therefore, I decide to use MT1=8. After tuning, the final solution is MT256x8x8 with MI4x4.

BufferStoreOffsetLimitCheck simpy use MT1 as the max offset size as the upper limitation, which was wrong. Different kernel parameters would make the max offset size different. For example, you can use MI32x32 kernel with sourceswap with MT1=32, which can solve this problem size and produce correct results, if you remove the checking of BufferLoadOffsetLimitCheck_Beta() and BufferStoreOffsetLimitCheck().

problem's N size is 32 which is larger enough to use MI16x16 or MI32x32. I am curious that have you tried both MI16x16 and MI4x4? And MI4x4 is the winner?

I also have tried MI16x16 and MI32x32 with minimum macro tiles, but encounter BufferStoreOffsetLimitCheck errors like below. BufferStoreOffsetLimitCheck(MT1:16): (67108864 * 4 * 16 <<PHONE_NUMBER>) BufferStoreOffsetLimitCheck(MT1:32): (67108864 * 4 * 32 <<PHONE_NUMBER>) It shows that MT1 should be less than 16, which implies MI4x4 is the one and only choice. Therefore, I decide to use MT1=8. After tuning, the final solution is MT256x8x8 with MI4x4.

BufferStoreOffsetLimitCheck simpy use MT1 as the max offset size as the upper limitation, which was wrong. Different kernel parameters would make the max offset size different. For example, you can use MI32x32 kernel with sourceswap with MT1=32, which can solve this problem size and produce correct results, if you remove the checking of BufferLoadOffsetLimitCheck_Beta() and BufferStoreOffsetLimitCheck().

Thanks @jichangjichang, the formulas in BufferLoadOffsetLimitCheck_Beta and BufferStoreOffsetLimitCheck should be updated, I will investigate and make another PR to fix.

BufferStoreOffsetLimitCheck checking can be revised by another PR. This PR can be a shorterm solution for your ticket.
Another thing is that you should squash your commit into one. There are already 7 commits in your PR, and most of them are empty

Will squash these.


Drag and Drop Activity Bar's item

Drag and Drop Activity Bar's item just like following video show
https://www.youtube.com/watch?v=1xbYbYBchuw

I'm not sure what do you mean. You can already drag and drop them to change their order.

Dropping activity bar item into places other than in the activity bar is not supported


fix(apigateway): allow for @app.not_found() decorator and housekeeping

Issue #, if available:

https://github.com/awslabs/aws-lambda-powertools-roadmap/issues/65

Description of changes:
Changes:

Allow for @app.not_found() decorator
Add docstrings and typing to strtobool
Add code coverage for  strtobool
Minor docstring typos
Add test for DynamoDBPersistenceLayer with a sort_key_attr

Checklist

[X] Meet tenets criteria
[X] Update tests
[X] Update docs
[X] PR title follows conventional commit semantics

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #926 (a3adb5f) into develop (321f493) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #926      +/-   ##
===========================================
+ Coverage    99.88%   99.92%   +0.03%
===========================================
  Files          118      118
  Lines         5272     5274       +2
  Branches       605      606       +1
===========================================
+ Hits          5266     5270       +4
+ Misses           2        1       -1
+ Partials         4        3       -1

Impacted Files
Coverage Δ

aws_lambda_powertools/event_handler/api_gateway.py
100.00% <100.00%> (ø)

aws_lambda_powertools/shared/functions.py
100.00% <100.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks a lot Mike for the quick fix & the housekeeping


Noob Question: Reference CSS files

Expected behavior and actual behavior
Reference CSS File
Steps to reproduce the problem (from a clean middleman installation)
<%= stylesheet_link_tag "stylesheets/css/bundle" %>
File Name: bundles.css.scss
Additional information
I created my own CSS file and cannot load it in the DOM

Ruby version: ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-darwin17]
Middleman version: Middleman 4.2.1
OS version: macOS High Sierra Version 10.13.4

Is bundles.css.scss in src/stylesheets/css/?

yes, checkout my repo: https://github.com/pcabrera83/patcabrera.com

Try:
<%= stylesheet_link_tag "css/bundle" %>

Middleman is already aware of where CSS lives.


Use MediaStream from webaudio, rather than mic input.

Hi!
I'm trying to record my canvas as well as a mediastream that comes from webaudio (tonejs, to be more precise).
I have extracted the audiotrack from the audiocontext, and fed it to the MediaStreamTrackProcessor as you do in your example. However, the AudioEncoder doesn't work, it throws:
DOMException: Input audio buffer is incompatible with codec parameters
This is full relevant code:
const audioStream = Tone.getContext().createMediaStreamDestination();
const audioTrack = audioStream.stream.getAudioTracks()[0];
const audioSampleRate = 48000;

audioEncoder = new AudioEncoder({
      output: (chunk, meta) => muxer.addAudioChunk(chunk, meta),
      error: (e) => console.error(e),
    });
    audioEncoder.configure({
      codec: "mp4a.40.2",
      numberOfChannels: 1,
      sampleRate: audioSampleRate,
      bitrate: 128000,
    });

let trackProcessor = new MediaStreamTrackProcessor({ track: audioTrack });
    let consumer = new WritableStream({
      write(audioData) {
        if (!recording) return;
        audioEncoder.encode(audioData);
        audioData.close();
      },
    });

trackProcessor.readable.pipeTo(consumer);

Thanks in advance!

Are you dealing with real-time data? Or do you, at the start of recording, already know how everything will play out? Because in that case, you should not use a MediaStreamTrackProcessor. Wrong tool for the job! You can use an offline context to render the entire out as fast as possible, then create an AudioData from the resulting AudioBuffer, then pipe that into the encoder all at once.
If you do happen to be working with real-time data, you can consider using a ScriptProcessorNode to extract audio data from your context. If this doesn't work, the MediaStreamTrackProcessor should still be okay. Regarding the error, I think it's very clear. Check the sampleRate and the channel count on your audio context, and try to match that with the encoder.
Anyway, I'm closing this issue since it's not related to this libray directly. But good luck!

Thx for the tips!


Aggregate tables for OTZ outcomes and cohort retention

Created initial aggregate reporting table for cohort retention

@cbrianbet nice! Just checking we had documented to combine the two tables. Was it intentional to split them?

@cbrianbet nice! Just checking we had documented to combine the two tables. Was it intentional to split them?

@nobert-mumo we actually documented OTZOutcome separate from Cohort retention
This PR has changes for both tables ie. AggregateOTZOutcomes and AggregateCohortRetention

@cbrianbet nice! Just checking we had documented to combine the two tables. Was it intentional to split them?

@nobert-mumo we actually documented OTZOutcome separate from Cohort retention This PR has changes for both tables ie. AggregateOTZOutcomes and AggregateCohortRetention

@cbrianbet  my bad, this makes sense.


Add ServiceMonitor support to CVO

This commit adds ServiceMonitor support to the CVO. The label config
follows OCP.

/approved (From the IBM Side for the cert change)


Updated README.ms  regarding hasChanged and property setters

I realised I was using hasChanged in the wrong way and thought that a more explicit documentation on both setter and the property option would be useful

It's README.md not .ms

Will make a new PR when I got a better text to suggest. :)