Update to spec 0.31.2

The spec update changes these things:

It simplifies the HTML regex so that <!-- a -- b --> is an HTML comment. HTML5 reports this as an error, but still parses it.
It changes the set of known HTML block elements to match HTML5, adding search and removing source.
It adds Unicode Symbols to the set of punctuation characters that are used to evaluate flankingness. This is blocked on https://github.com/markdown-it/uc.micro/pull/15

This commit also changes the declaration HTML regex to match lowercase, even though that change was technically made in spec version 0.30.

This is blocked on https://github.com/markdown-it/uc.micro/pull/15

Blocker should be resolved.
Can you please rebase and update pull request?

Also, can you move actual code changes and spec update to separate commits, if that's too not much of a trouble? (i.e. one commit with test/fixtures/commonmark/* and another with everything else)


Missing documentation: minecraft:givable

Missing documentation on the trigger object structure.

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

ID: c8bbde5f-4ced-ab9a-950f-875848113f05
Version Independent ID: 48a7ced5-056f-3fb1-304f-aabcbc9a3bd1
Content: minecraft:giveable
Content Source: creator/Reference/Content/EntityReference/Examples/EntityComponents/minecraftComponent_giveable.md
Product: gaming
GitHub Login: @v-jeffreykim
Microsoft Alias: v-jeffreykim

Thank you for your feedback! This issue seems to no longer be applicable, and will be closed.

Thank you for your feedback! This issue seems to no longer be applicable, and will be closed.

@JimSeaman42 Has it been fixed?
Im still seeing vanilla examples that use the trigger object, but no documentation


feat(challenges): Added more assertion for euler problem twenty two

more assertion and updated question

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[ ] Small bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

Updated Project Euler seeds for problem 22
This challenge requires fs, or rather I use module.exports, so I can access the text file as an array of strings in javascript, any other idea how to make this challenge work better?
Here's the solution, I can't add it to the json file, due to it's always failing on the test.
function sumNames(n) {
    const names = require('./p022_names');
    names.sort();

    var sum = 0;

    for (let i = 0; i < n; i++) {
        let atoi = names[i]
            .split('')
            .map((_, j) => names[i].charCodeAt(j) - 64)
            .reduce((c, n) => c + n, 0);

        sum += (i + 1) * atoi;
    }

    return sum;
}

I added to the json file like this
[
	"function sumNames(n) {",
	"    const names = require('./p022_names');",
	"    names.sort();",
	"",
	"    var sum = 0;",
	"",
	"    for (let i = 0; i < n; i++) {",
	"        let atoi = names[i]",
	"            .split('')",
	"            .map((_, j) => names[i].charCodeAt(j) - 64)",
	"            .reduce((c, n) => c + n, 0);",
	"",
	"        sum += (i + 1) * atoi;",
	"    }",
	"",
	"    return sum;",
	"}"
]

Here's the error message
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ Error: Cannot find module './p022_names'
      ✖ plan != count
      ✖ Error: Cannot find module './p022_names'
      ✖ Error: Cannot find module './p022_names'
      ✖ Error: Cannot find module './p022_names'
      ✖ Error: Cannot find module './p022_names'
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: sumNames is not defined
      ✖ ReferenceError: sumNames is not defined
      ✖ ReferenceError: sumNames is not defined
      ✖ ReferenceError: sumNames is not defined
      ✖ ReferenceError: sumNames is not defined
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ SyntaxError: Unexpected end of input
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ ReferenceError: names is not defined
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ SyntaxError: Unexpected token .
      ✖ ReferenceError: sum is not defined
      ✖ ReferenceError: sum is not defined
      ✖ ReferenceError: sum is not defined
      ✖ ReferenceError: sum is not defined
      ✖ ReferenceError: sum is not defined
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Illegal return statement
      ✖ SyntaxError: Illegal return statement
      ✖ SyntaxError: Illegal return statement
      ✖ SyntaxError: Illegal return statement
      ✖ SyntaxError: Illegal return statement
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }
      ✖ SyntaxError: Unexpected token }

CODE LGTM

This problem does not work with the freeCodeCamp platform since the description says to use an external file. This is impossible with the platform. We are going to remove this problem completely in another pull request.


Create a draft definition of done document

Create a draft definition of done document that is agnostic to the technology and tools we use.

A draft DoD document has been created.


Texture for deformable objects

Just wanted to know if the png files of wood in the pick and place task could be used for deformable objects. I have tried using it for rigid objects and it worked, but faced issues with deformable objects.

Hi @saketh2002 ,
yes, deformable objects and textures work.
As an example, see the tissue retraction scene, with texture objects set to True.
https://github.com/ScheiklP/sofa_env/blob/main/sofa_env/scenes/tissue_retraction/tissue_retraction_env.py#L689
However, you probably have to do some uv unwrapping for your surface mesh.
https://en.wikipedia.org/wiki/UV_mapping#:~:text=map with coordinates-,UV unwrapping,-[edit]
For example with blender.
https://docs.blender.org/manual/en/2.79/editors/uv_image/uv/editing/unwrapping/mapping_types.html#unwrap
Cheers,
Paul


Implement LPOP

https://redis.io/commands/lpop

lpop isn't used in the sidekiq lib code, but it is used in sidekiq tests


Incompatible architectures with dss_capi.dylib [MacOS Apple Silicon]

Hello! I am trying to run the 13Bus example on MacOS (arm64), and I am getting an error.  I have the folder containing +DSS_MATLAB added to MATLAB path, and then when I run examples/13Bus/run.m, I get the following error:
Error using loadlibrary
There was an error loading the library
"/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib"
dlopen(/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib, 0x0006): tried:
'/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib' (mach-o file, but is an
incompatible architecture (have 'arm64', need 'x86_64')),
'/System/Volumes/Preboot/Cryptexes/OS/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib'
(no such file), '/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib' (mach-o file,
but is an incompatible architecture (have 'arm64', need 'x86_64'))

Error in DSS_MATLAB.APIUtil (line 49)
                loadlibrary(DLLfilePath, @DSS_MATLAB.dss_capi_no_thunk);

Error in DSS_MATLAB.IDSS (line 76)
            apiutil = DSS_MATLAB.APIUtil(varargin{:});

Error in run (line 36)
DSS = DSS_MATLAB.IDSS;

Caused by:
    Error using loaddefinedlibrary
    dlopen(/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib, 0x0006): tried:
    '/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib' (mach-o file, but is an
    incompatible architecture (have 'arm64', need 'x86_64')),
    '/System/Volumes/Preboot/Cryptexes/OS/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib'
    (no such file), '/Users/rsparks3/Documents/dss_matlab/+DSS_MATLAB/dss_capi.dylib' (mach-o
    file, but is an incompatible architecture (have 'arm64', need 'x86_64'))

I tried downloading dss_capi v0.14.5 and using the .dylib files from that, but I get the exact same error.  I think I have installed opendss properly as I can run opendsscmd in terminal and it will allow me to run DSS commands.
Has anyone seen this error or can point me in the right direction? Thanks a bunch.

From the error message:

(have 'arm64', need 'x86_64'))

Looks like you're running an Intel x64 installation of MATLAB. In that case, you need to download/use our x64 package instead.
To check, try running this on MATLAB:
archstr = computer('arch')

Per MATLAB's help, it should return one of these:

'maci64' - 64-bit macOS platform
'maca64' - 64-bit macOS platform, Apple silicon

If it returns maci64, you have a x64 MATLAB install, running through Rosetta. As a reminder:

Native Apple silicon MATLAB is available for MATLAB R2023b and up.

If your MATLAB is indeed R2023b and up, try reinstalling MATLAB, ensuring the ARM/Apple Silicon version installation is used.

I think I have installed opendss properly as I can run opendsscmd in terminal and it will allow me to run DSS commands.

We don't use or require opendsscmd are all. Currently, I personally don't recommend using opendsscmd outside of Windows since EPRI doesn't support it and there are many known issues on Linux/macOS.

Installing the silicon version of matlab worked great! Thank you so much for your help.  I'll also get rid of opendsscmd, I had previously installed it thinking it was required for this.


Create new footer

The end result is visually great, no real complaints there (besides maybe the full stops in the "funded/supported by" links being part of the link now!). Do say if you spot anything @sjd210.
On a more general note though, I think we should aim to be a bit more thorough in cleaning out old code; a lot of the things you haven't explicitly styled here fall back on to the old Nomensa designs, most of which has changed so far as to do nothing but bloat the CSS at this point.
The same can be said for the DOM structure itself; in a few places we really don't need as many elements as we have (take the ListGroupFooter component for example; there are three wrappers (and, unrelated, but the children are <li/>s with no surrounding list!)).
I have a PR over this one here to show which bits I'd change.

(I'm aware the spacing between the link ListGroups is now wrong, working on it!)

Both sorted with better solutions than I suggested, but either way I'm happy since the problems are fixed! 👍


Update documentation to include section on using on workstations or HPC systems where it is not installed

How does a scientist build the UFS on her own linux workstation using spack-stack? (Or does she just use spack directly?)

How does a scientist build the UFS on her own linux workstation using spack-stack? (Or does she just use spack directly?)

This documentation is definitely needed, but should not be in the spack-stack repo. It should be in the ufs-weather-model documentation. We did the same for JEDI. spack-stack just builds the prerequisite modules for each of the apps, it's up to the app documentation in my opinion to document how to use it.

There is currently some information in the spack-stack wiki (where it does not belong). Until the wiki is cleaned up, I will keep this issue open. That kind of documentation, as discussed before (https://github.com/NOAA-EMC/spack-stack/issues/253#issuecomment-1182173052), belongs to the individual applications/models that use spack-stack.

Closed via https://github.com/NOAA-EMC/spack-stack/pull/419


spec: add "dns" field to the result JSON

This adds an ability for networks to communicate
about the DNS nameservers they know about to the
container runtime.
Fixes #67

Any particular reason why DNS servers should not be differentiated between IP versions?

@steveeJ is that common to differentiate by protocol? Can resolv.conf do that?

I can think of three appropriate ways of providing nameservers:
* entirely out-of-band with respect to CNI (e.g., if Kubernetes runs its own DNS server);
* in the configuration of a network;
* supplied by an IPAM plugin.

Allowing that I don't always have the right end of the stick, I would be interested to know what suggests this change, since it doesn't (quite) cover off the above situations.

@steveeJ That is a great point and I spent a while thinking about whether to add dns at top level or inside the ip4, ip6 sections. The reason I decided to go with top level is to allow the plugin to specify priority/preference between IPv4 and IPv6 nameservers in dual stack scenarios. However, it's a small win so definitely open to doing it the other way if there's a reason.

@squaremo I'm not sure I understand your concern. There is no right way to do DNS resolution. This change just allows the CNI plugin to communicate about the nameservers it happened to know about (e.g. b/c it learned it via DHCP). It would still be up to the runtime to choose the implementation.

@squaremo I see what you're saying. The IPAM section in the spec is missing the dns field. It will ultimately be coming from there and propagated up. I will fix.

@philips resolv.conf doesn't do it but I wouldn't only solve the resolv.conf use-case in CNI. We provide routes/gateway per protocol, applying that on DNS would be consistent and nice to have for applications that could make use with the extra information.

Few minor syntactical issues, but other than that this approach lgtm.

The only scenario I can think of that this wouldn't work for is split DNS, where queries for certain domains are directed to specific nameservers and the rest go somewhere else.  eg, VPN-like functionality.  That would require that the 'dns' option be a map of string::list-of-string though.  And maybe too complicated for now, since if this actually becomes a need we could add a 'split-dns' key or something. Just food for thought.
But generally LGTM too.  There is value in having the list priority ordered, which implies keeping it separate and not in the IP-version-specific blocks.

@rajatchopra look good to you too?

Useful and good to go I think.
One question which should not block this PR is handling of /etc/hosts, if at all. Its mention is removed from Open Questions, but this PR does not address it.

@rajatchopra This PR takes a stance that /etc/hosts, like /etc/resolve.conf, is outside the scope of the spec (at least for now). The only thing this spec covers in terms of name resolution is the dns key as specified in this PR.

@dcbw I think with split-dns, it's the DNS server that serves up alternate IPs for a some DNS zone. It's usually not done by the local resolver (e.g. /etc/resolv.conf) so I think it's reasonable not to support it here as well. But like you say, we can extend the spec if the need arises.

I think with split-dns, it's the DNS server that serves up alternate IPs for a some DNS zone.

@eyakubovich Not really.  It comes into play when you have multiple sources of network information.  For example, suppose you want your container to connect to multiple networks; one for the local cluster and a VPN-like one.  Each network is handled by a different CNI plugin (or the same plugin, doesn't matter really), and each plugin can return whatever DNS information it wants.  Which do you choose to be "primary"?
Even though priority and how DNS resolution is handled is out of scope, the thing that calls the CNI plugin needs some way to figure that out.  And passing the search domains back from the CNI plugin is pretty crucial for split DNS, since that's the only way you know what domains to send to that network's DNS server.
I'm not saying we need to add it now...  Just that we should think about the use-case as more and more stuff becomes multi-network capable.


Add a config file

A config file could be added which stores some additional information for the ARC
This could e.g. be

the standard editor used for querying arguments
a default study to which all assays and computations are added per default
github settings for syncing the arc.

This file could be created when initiating the ARC receiving its info through command line arguments

@muehlhaus which filePath you think fits better?
.arc/globalParams.yml

everything in one place

for many other tools, this folder(.paket,.fake, ..) does contain only temporary information

 .config/globalParams.yml

this might be more intuitive as it follows the standard

makes the arc base folder more crowded

Chiming in: I would follow the Git model and use .arc/ for all per-ARC config. Maybe one could even consider a single .arc file?
.config is only standardized (in XDG via XDG_CONFIG_HOME) for per-user configuration, and has discoverability issues.

I agree with @cgarth. We definitely should stick to the .acr/ folder decision. It should also be a folder as more files will arrive there...

Alright, that was my first pick anyways 👍

A more sophisticated configuration implementation is available with #20.
I split the configuration in global (arcCommanderBinaries/config) and local (specificArc/.arc/config). Both in ini format.
A more detailed expanation is given in the PR.


Ivo/swich camera

In this PR:

Added button <span class="material-icons-outlined"> cameraswitch </span> to [Android] and NodeJS controllers to switch bot camera remotely.
Minor fixes in nodejs code.
Updated documentation.

Interesting...
I looked into this a little bit deeper. It appears that we can implement our own CameraSwitchHandler by passing in into:
((CameraVideoCapturer) videoCapturer).switchCamera(null);
instead of passing null. Then we can select a camera by name. But I'm not sure we can do better than the default behaviour, since the camera names could change or vary across phones. What do you think?

Interesting... I looked into this a little bit deeper. It appears that we can implement our own CameraSwitchHandler by passing in into: ((CameraVideoCapturer) videoCapturer).switchCamera(null); instead of passing null. Then we can select a camera by name. But I'm not sure we can do better than the default behaviour, since the camera names could change or vary across phones. What do you think?

Actually, after running through the debugger, camera names are just "0", "1" ... So we do not know if they are front, back. macro, etc. They just select them in sequence. So I don't think we can chose specific camera, since we don't know what it is.

Should be possible to find out if it is front or rear camera.
https://developer.android.com/reference/android/hardware/camera2/CameraCharacteristics#LENS_FACING
But I think it is also a nice feature to be able to toggle between the rear cameras. For driving it is good to use the ultra-wide camera. I guess the coolest would be if phone reports the available cameras and user can choose one. But maybe that is overkill. We could also merge now and look into this in a separate PR if you are interested to look into it.

Sounds good.
Ivo
On Wed, Jan 12, 2022, 12:07 PM thias15 @.***> wrote:

Should be possible to find out if it is front or rear camera.
https://developer.android.com/reference/android/hardware/camera2/CameraCharacteristics#LENS_FACING
But I think it is also a nice feature to be able to toggle between the
rear cameras. For driving it is good to use the ultra-wide camera. I guess
the coolest would be if phone reports the available cameras and user can
choose one. But maybe that is overkill. We could also merge now and look
into this in a separate PR if you are interested to look into it.
—
Reply to this email directly, view it on GitHub
https://github.com/isl-org/OpenBot/pull/249#issuecomment-1011265774, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37XPGGGSCESUA6RN4TTUVWYODANCNFSM5LQZKTVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


better line erase using ANSI escape code

I think I fixed it, checkout https://en.wikipedia.org/wiki/ANSI_escape_code.
Can you test it before merging? Also on window's powershell if possible.
Thank you!

Current coverage is 97.51% (diff: 100%)

Merging #18 into master will decrease coverage by 0.03%

@@             master        #18   diff @@
==========================================
  Files            15         15
  Lines           489        483     -6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            477        471     -6
  Misses           12         12
  Partials          0          0

Powered by Codecov. Last update 612235c...d6d99a6

@AlexK-IL Yes, I have a windows instalation on my machine. I'll test before merge it. Thanks for all efforts.

@AlexK-IL However is really common windows shell don't support ANSI escape codes...

I see, that's a shame. I'll boot into my windows and see if I can think of something else.
It did fix the output on urxvt terminal though, could you merge it anyway? Since it didn't work on windows before anyways.

This branch is out-of-date with the base branch. Rebase your branch.
About merge it anyway I'll just test this in my machine, on linux and windows, and if it's ok I'll merge into master. Hold on a little. Would be better if we got compatibility on both terminal systems...

On linux works fine here. I don't care about Windows for today, as you said before, did not works before so this doesn't matters for now.
Thanks for the good job!

I'll release a new version in soon on PyPI and update mal too. Keep calm, I'm a little busy for now unfortunately.


feat(init): create install-man-pages subcommand

Will the existing init subcommand still try to install man pages locally? I would suggest it not do that by default.

No, it currently doesn't install man pages at all (and it still won't).

Will the man page install subcommand be hidden from --help? I typically suggest it my a hidden subcommand, but document it in the project readme or install instructions so distro folks in particular catch it but it doesn't clutter up usage info for end users that may not have access to the root system anyway and shouldn't be stuffing stuff there behind their package manager's back unless they know what they are doing.

The majority of git-branchless users install from source and won't have any way to install man pages (which breaks git <command> -h, see https://github.com/arxanas/git-branchless/issues/88#issuecomment-906861160), so I think I'll leave it as visible in case someone does want to install man pages on their own (at their own risk).


Create PDF version of STFI case study

Description
Create a PDF document of the case study so that it can be available to download via a form
Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

Hi @Hasmin-AC here it is. I'm gonna use the same layout for the other ones, and add this as a template in the FlowFuse Google template gallery.

This is the one I added as a template, and as requested, I just modified a copy of file to be two columns instead of one. Please, check it out before I update the template, it's here. Let me know your thoughts.

yes the two columns makes it much easier to read. thanks!


cc 1.0.70 fails to build ring for target aarch64-apple-ios-sim on x86_64 host.

The problem is that cc ignores "-arch arm64" argument when it is in quotes. If I run this command with -arch arm64 without quotes, it compiles.
  --- stderr
  running "/usr/bin/cc" "-O3" "-fPIC" "-g" "-fno-omit-frame-pointer" "-arch arm64" "-mios-simulator-version-min=11.2" "-isysroot" "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator14.4.sdk" "-fembed-bitcode" "-I" "include" "-Wall" "-Wextra" "-pedantic" "-pedantic-errors" "-Wall" "-Wextra" "-Wcast-align" "-Wcast-qual" "-Wconversion" "-Wenum-compare" "-Wfloat-equal" "-Wformat=2" "-Winline" "-Winvalid-pch" "-Wmissing-field-initializers" "-Wmissing-include-dirs" "-Wredundant-decls" "-Wshadow" "-Wsign-compare" "-Wsign-conversion" "-Wundef" "-Wuninitialized" "-Wwrite-strings" "-fno-strict-aliasing" "-fvisibility=hidden" "-fstack-protector" "-g3" "-DNDEBUG" "-c" "-o/Users/buildagent/.conan/data/quiche/0.9.0/AdguardTeam/NativeLibsCommon/build/6d052288a44309b7369d4a26703162f0cb5a9d94/source_subfolder/target/aarch64-apple-ios-sim/release/build/ring-d34770fe4c482e36/out/aesv8-armx-ios64.o" "/Users/buildagent/.cargo/registry/src/github.com-1ecc6299db9ec823/ring-0.16.20/pregenerated/aesv8-armx-ios64.S"
  clang: warning: argument unused during compilation: '-arch arm64' [-Wunused-command-line-argument]
  In file included from /Users/buildagent/.cargo/registry/src/github.com-1ecc6299db9ec823/ring-0.16.20/pregenerated/aesv8-armx-ios64.S:12:
  include/GFp/arm_arch.h:96:5: error: "unsupported ARM architecture"
  #   error "unsupported ARM architecture"

Related to #614

This has been fixed in https://github.com/rust-lang/cc-rs/pull/1256 by removing the -arch flag altogether on Apple targets, and instead specifying --target


Display placeholder GitHub stats when the API token is missing

GitHub stats require a personal access token. If it doesn't exist, the site should work just fine.

@deammer -- I have a github token in my local env file, so I've been able to get data locally. Will I no longer be able to now?

@jleven if you have the GH token in your env file, you will still be able to fetch data from GitHub. My thinking is that most contributors wouldn't provide a GH token though, hence this disclaimer.


idea: extends the plugin to support other similar insert mode mappings

For example, if we use the following mapping:
inoremap ;; <Esc>A;

We also experience lagging when we press ; in insert mode. So we can also do the same thing. We check if the time interval between pressing the two chars is below the time threshold. If that is the case, we execute its original intended mapping, otherwise, we just insert the character literally.
Here is a crude implementation:
let g:my_map = {";;": "\<Esc>mmA;\<Esc>`ma"}

let g:char_press_time = {}

inoremap <expr> ; My_map()

augroup log_press_time
  autocmd!
  autocmd InsertCharPre * call Log_char()
augroup END

function! Log_char() abort
  if v:char == ';'
    let g:char_press_time[';'] = reltime()
  endif
endfunction

function! My_map() abort
  let idx = col('.')
  let pre_idx = idx-1
  let pre_char = getline('.')[pre_idx-1]

  if pre_char == ';'
    let interval = reltimefloat(reltime(g:char_press_time[";"])) * 1000
    echomsg "interval:" interval

    if interval < 200
      return "\b" . g:my_map[";;"]
    else
      return ";"
    endif
  else
    return ";"
  endif

endfunction

In order to avoid the step left after exiting insert mode with ESC, I now do
 inoremap <silent> kj <C-O>:stopinsert<CR>

What's the best way to use your plugin to map kj this way rather than to ESC?  Thanks


How to pass replace All to command

When running a script on a CI machine, there's no way to tell carthage_cache to automatically overwrite the current Carthage/Build folder if things are already there.

Yes you are right. I was thinking of adding this feature. If you want to submit a PR I would appreciate it but I would try to add this on the weekend. I wasn't experience this issue in my CI because I use travis and I get a fresh VM every time I run a build.


Offer Home Assistant Discovery (/config)

Is your feature request related to a problem? Please describe.
I'm always frustrated when i enable the MQTT feature, and Home Assistant not found the device automaticly.
Describe the solution you'd like
Enabled the mqtt broker, but device was not found with discovery enabled.
Describe alternatives you've considered
Tried different mqtt topics. homeassistant/wallpanel/mywallpanel etc.
Additional context
https://www.home-assistant.io/docs/mqtt/discovery/

I don't think its difficult to just enter your broker address, username and password.  It's not going to discover the wallpanel app and do everything for you.   The app needs to credential from HA first.   Commands are not discoverable, the only thing that would be discoverable is maybe sensor data.

I don't think its difficult to just enter your broker address, username and password.  It's not going to discover the wallpanel app and do everything for you.   The app needs to credential from HA first.   Commands are not discoverable, the only thing that would be discoverable is maybe sensor data.

I also meant that with the sensors. I can see the messages from the broker and the wallpanel in the mqtt, they are both connected.
I added the motion sensor manually, by the way, it works great.

I also meant that with the sensors. I can see the messages from the broker and the wallpanel in the mqtt, they are both connected.
I added the motion sensor manually, by the way, it works great.

We have a PR to add discovery for sensors but I have not reviewed it yet.
https://github.com/thanksmister/wallpanel-android/pull/233

We have a PR to add discovery for sensors but I have not reviewed it yet.
https://github.com/thanksmister/wallpanel-android/pull/233


Unable to set up python environment.

when I select an interpreter through the python environment window, there is a crash on my laptop. I mean I was forced to reboot my machine. after restart my laptop, the "+custom" button in the python environment window cannot be clicked. I have uploaded a pic of my issue.

We have an update in testing to fix this - you can try the RC release by installing the MSI from our 2.2.6 RC release page. You'll get a notification in VS when we release the final fix.
Please let us know if you still see issues after installing this update.

@zooba thanks! it works!


Make compatible with MEDS v0.3.2

Changes: https://github.com/Medical-Event-Data-Standard/meds/compare/0.3.0...0.3.2
Big change: patient_id has become subject_id

Done


Use new @gravitee/ui-particles-angular lib

Issue
https://github.com/gravitee-io/issues/issues/6444
Description
Configure console ui with ivy
Use new lib @gravitee/ui-particles-angular and use it
Additional context

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.


[r] Update DESCRIPTION to 1.5.1

This is a small bugfix release

Should we sweep up more PRs (and #1894 is a candidate though not a must have) and/or wait for an updated-at-CRAN SeuratObject, or is your preference to get this out ASAP?

@eddelbuettel main and release-1.5 are in sync except for https://github.com/single-cell-data/TileDB-SOMA/issues/1733 which is shelved for reasons beyond our control. #1894 is still open.
The discussion last week was to tag 1.5.1 today, while @mojaveazure says he should have the Seurat issue solved later this week. (See also https://github.com/TileDB-Inc/TileDB-Cloud-JupyterHub/pull/562/commits/b01ce792788538cb3fd083d7624c1bbf9ac9ce97.)

@eddelbuettel I don't mind getting #1894 in but it  needs to be accepted and merged to main first.

@eddelbuettel I wasn't a reviewer on #1894 but I'll self-add, given your suggestion. Thanks!

I see.  Thanks for unblocking.


Design agent class

Description
This PR creates the Agent base class and implements the data_store.py agent. Note, currently the implemented agent is not wired up to the neo4j database or the supervisor. A test endpoint was added during development to test the logic with a default answer, £6.99 always being returned from the agent's tool. Multiple tools were tested within the agent too, but just one has been left as it aligns with the epic
Changelog

Created Agent abstract class
Created implementation of agent, data_store.py
Created tool adapter for mistral

Manual testing
To wire up and use the data_store.py agent I was making the following changes to the two files:
Bottom of data_store.py :
def run():
    db_agent = DatastoreAgent()
    answer = db_agent.invoke("How much did I spend on Spotify last month?")
    print(f"ANSWER: {answer}")

Add new endpoint to api.py
from agents.data_store import run

...

@app.get("/test")
async def test():
    run()
    return JSONResponse(status_code=200, content=healthy_response)

then just use postman (or a browser!) to hit http://<IP_ADDRESS>:8250/test

Manual testing
To wire up and use the data_store.py agent I was making the following changes to the two files:
Bottom of data_store.py :
def run():
db_agent = DatastoreAgent()
answer = db_agent.invoke("How much did I spend on Spotify last month?")
print(f"ANSWER: {answer}")
Add new endpoint to api.py
from agents.data_store import run
...
@app.get("/test")
async def test():
run()
return JSONResponse(status_code=200, content=healthy_response)
then just use postman (or a browser!) to hit http://<IP_ADDRESS>:8250/test

Guessing this has been removed?

Manual testing
To wire up and use the data_store.py agent I was making the following changes to the two files:
Bottom of data_store.py :
def run():
db_agent = DatastoreAgent()
answer = db_agent.invoke("How much did I spend on Spotify last month?")
print(f"ANSWER: {answer}")
Add new endpoint to api.py
from agents.data_store import run
...
@app.get("/test")
async def test():
run()
return JSONResponse(status_code=200, content=healthy_response)
then just use postman (or a browser!) to hit http://<IP_ADDRESS>:8250/test

Guessing this has been removed?

yeah exactly, not needed in the end feature, but just how I tested it whilst developing


excessively long subjectKeyId in org.bouncycastle.cms.KeyTransRecipientId ?

The attached mail causes problems on the SMime Decryption.
It seems that the subjectKeyId of the second recipient is excessively long (294 bytes instead of normally approx 20 bytes).
Is there an input check missing here ?
ticket-2670287-37675.eml.zip

(t7028)

Some more insights

Parsing error ?

I have just tested this on latest code and it parses fine, finding a 20 byte subjectKeyId and without errors.
    public void testSubKeyIdGithub1380()
    	throws Exception
    {
    	String filename = "Path/To/ticket-2670287-37675.eml";

        Properties props = System.getProperties();

        Session session = Session.getDefaultInstance(props, null);

        MimeMessage msg = new MimeMessage(session, new SharedFileInputStream(filename));

        SMIMEEnvelopedParser m = new SMIMEEnvelopedParser(msg);

        RecipientInformationStore   recipients = m.getRecipientInfos();
    }

Can you confirm the specific version of BC you are using and also could you test with SMIMEEnveloped instead of the parser please.

Hi Peter,
When I extend your code with the following lines;
               `Iterator<RecipientInformation>` rIt = recipients.iterator();
                int ki = 0;
                while (rIt.hasNext()) {ki++;
					RecipientInformation recipientInformation = (RecipientInformation) rIt
							.next();
					if (recipientInformation instanceof KeyTransRecipientInformation) {
						KeyTransRecipientInformation ktrin = (KeyTransRecipientInformation) recipientInformation;
						RecipientId rid = ktrin.getRID();
						KeyTransRecipientId ktri = (KeyTransRecipientId) rid;
						byte[] ski = ktri.getSubjectKeyIdentifier();
						log.debug(ki+": "+ski.length);
					}
				}`

it turns out that there is more than one subjectKeyId - the output is
1: 20
2: 294

OK, so the 294 bytes is an encoded SEQUENCE:
Sequence
    Sequence
        ObjectIdentifier(1.2.840.1135<IP_ADDRESS>)
        NULL
    DER Bit String[270, 0]

and that 270 byte BIT STRING is also an encoded SEQUENCE:
Sequence
 Integer(19160391302497067434626714016228421851997655162226263015887318605468368628891665938937770336113446563207601554539513769837008303784202016239813021221641142058898772367617743351985020538555361297249922748412184969118105260148481319852670447685101254574556270314721833079665498477532922732652761941888605972463342600810442350987399419570279385283967931308135440687878309318428347753038337520207328081941168699803022177962294162204763397505108168996079460198668510222866515009429159064480929317096544774048992622646806474754128311130152496898281852255006550213801450218917873073167280761134023546341817529331067694724453)
    Integer(65537)

So that's presumably an RSA public key and we note that the outer sequence fits an OriginatorPublicKey.
An OriginatorPublicKey should only appear in a KeyAgreeRecipientInfo/OriginatorIdentifierOrKey, and would also need a [CONTEXT 1] tag instead of 0 as it is here. If the encoder was attempting to encode the OriginatorPublicKey instead of wrapping it in a SubjectKeyIdentifier, then they also missed the CONSTRUCTED bit I think.
So the data seems to be a valid ASN.1 encoding, though almost certainly not what the sender intended. It's not obvious that there's a way to restrict the SubjectKeyIdentifier length unless it be a user-specified restriction.

Thanks, so as the first ski is presumably referencing the sender's own public key, the second one should reference the recipient's public key.
As you seem to say, that appears to misconstructed, so you would agree that it is not a valid smime encrypted message ?
At least with bouncycastle standard classes, me as the recipient will not be able to decrypt it?
Or is there something missing in the BC smime decryption classes ?

There are two KeyTransRecipientInfo recipients. These don't reference sender keys, only receiver keys. Any given receiver only needs one of these to be able to decrypt the message and it is normal to send one message to multiple recipients so recipient infos that you can't use should just be gracefully ignored.
Presumably the second KTRI with the long SKI is meaningless to you (or anybody), but if the first one correctly identifies a receiver cert then you should be able to decrypt the session fine.

Thanks.
But if the first (20-bytes) SKI doesn't match my key pair as a recipient and the 2nd (meaningless) SKI must be ignored, I cannot decrypt - agreed ?
Or should one try to match the presumable public key with one's own public key ?

The second one is not technically meaningless, but will not match your certificate. No, you should not try to reinterpret it as an OriginatorPublicKey (that is invalid in a KeyTransRecipientInfo and in any case it would be saying who sent the message not the intended receiver).
Yes, if neither match then you won't be able to decrypt it. If the message was definitely intended for you, then you would have to discuss with the sender as to why they failed to encrypt to you and instead produced this weird construction.

If I read "6.2.1.  KeyTransRecipientInfo Type" of https://www.rfc-editor.org/rfc/rfc5652#page-22  the attached example does not follow this RFC

Well the ASN.1 format is consistent with it (which is why it parses successfully), but the actual contents of the SubjectKeyIdentifier looks like it is an encoded OriginatorPublicKey (note the encoded object is just an opaque OCTET STRING as far as the outer ASN.1 parser is concerned), which is at least weird and almost certainly a mistake.
Note this text though:

When other certificate formats are referenced, the documents that specify the certificate format and their use with the CMS must include details on matching the key identifier to the appropriate certificate field.

...which makes the contents of SubjectKeyIdentifier something of an "open format" so it can only be validated in the context of some certificate format we are trying to match it to (usually X.509 of course), and failure to validate just means no match in that context.

ticketPrivaSphere.cer_till231201.crt.zip
Your statement is still confusing to me - if you have the attached certificate - would BC attempt to match it in that specific case (ticket-2670287-37675.eml) or if not what is the justification for not doing it ?

As far as BC is concerned there are two valid KeyTransRecipientInfo objects in ticket-2670287-37675.eml, both using subjectKeyIdentifier for their RecipientIdentifier. So yes, BC will attempt to match against both of them.

As far as BC is concerned there are two valid KeyTransRecipientInfo objects in ticket-2670287-37675.eml, both using subjectKeyIdentifier for their RecipientIdentifier. So yes, BC will attempt to match against both of them.

But you would agree that it is highly unlikely that the 2nd attempt will ever succeed ?

Yes I agree. The SubjectKeyIdentifier for an X.509 certificate is a hash of the SubjectPublicKeyInfo, and I doubt anyone is using a hash algorithm with output size 294 bytes.

openssl cms -cmsout -noout -print -in ticket-2670287-37675.eml | less
shows details

Looking at the details again, it turns out that
    JcaX509CertSelectorConverter converter = new JcaX509CertSelectorConverter();
    KeyTransRecipientId kRid = null;
    if (rid instanceof KeyTransRecipientId) {
        kRid = (KeyTransRecipientId) rid;
    } else {
        log.warn("rid not a known type " + rid);
    }
    X509CertSelector conv = converter.getCertSelector(kRid);

might be the problem.
before the conversion, the SKI is 20 bytes and
[-17, -118, 92, 74, -3, 41, 19, -111, -38, 55, 68, -109, 89, 53, -85, -73, 36, -85, 49, -85]
after the conversion with

it is 22 bytes and
[4, 20, -17, -118, 92, 74, -3, 41, 19, -111, -38, 55, 68, -109, 89, 53, -85, -73, 36, -85, 49, -85]

So why are the [4,20] prefixed ?
Is
ConverterTest.testRecipientIdConversion()
with the empty
new byte[20]
a realistic test ?

Once I did the following patch, my decryption started working
1380.patch

Thanks for the patch, it's now applied.

Patch released with 1.74/1.75.


Legion go controller issue

Describe the bug
When I detach only one side of the controller, both of them will be treated as indicidual controllers,like when you split Nintendo switch controller and play them horizontally, which means joystick and button layout will be rotated. And again, this happens only when I detach one controller and leave the other one on the body.
What did you expect to happen?
I would expect them still work together as a full controller. Or maybe even can be setup by users.
Output of rpm-ostree status
No response
Hardware
No response
Extra information or context
No response

@antheas

Yes I do not like it either.
It is a firmware "feature" by Lenovo. It cannot be disabled or worked around.
I would direct you to the legion go forums.


Accept 0x80-0xFF for setcellwidth()

Currently, setcellwidth() accepts only characters with value 0x100 or higher. However, there are many ambiguous width characters between 0xA1 to 0xFE.
See: https://www.unicode.org/Public/UCD/latest/ucd/EastAsianWidth.txt
This makes setcellwidth() to accept characters between 0x80 to 0xFF.

Currently, setcellwidth() accepts only characters with value 0x100 or
higher. However, there are many ambiguous width characters between
0xA1 to 0xFE.
See: https://www.unicode.org/Public/UCD/latest/ucd/EastAsianWidth.txt
This makes setcellwidth() to accept characters between 0x80 to 0xFF.

How about unprintable characters between 0x80 and 0x100?  I think four
should be returned then.  Something like:
if (c >= 0x80)
{
  if (!vim_isprintc(c))
	return 4;		// unprintable, displays <xx>
  int n = cw_value(c);
  if (n != 0)
    return n;

--
How To Keep A Healthy Level Of Insanity:
3. Every time someone asks you to do something, ask if they want fries
with that.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I think they are already covered by this condition:
https://github.com/vim/vim/blob/35a12d2a17119dabeda7d234b7103157dea874c5/src/mbyte.c#L1626-L1628
Or, should we check if a character is unprintable before calling cw_value()?
For c >= 0x100, we call cw_value() before checking if it is unprintable.

I've updated the tests, and I've confirmed that strwidth("\u0081") returns 4 and strwidth("\uFEFF") returns 6 as expected.

Might need a screendump test to verify each character shows up as expected.

It might be better to use a screendump, but I use strwidth() for now.

I had to change 'isprint' on Windows to get the expected result.

I've updated the tests, and I've confirmed that strwidth("\u0081")
returns 4 and strwidth("\uFEFF") returns 6 as expected.

Might need a screendump test to verify each character shows up as expected.

It might be better to use a screendump, but I use strwidth() for now.

I just checked, and it appears it's already wrong without your patch.
This works OK:
call assert_equal(6, strwidth("\uFEFF"))

But then:
call setcellwidths([[0xFEFF, 0xFEFF, 1]])
call assert_equal(1, strwidth("\uFEFF"))

However, entering 0xFEFF in the text shows it's displayed as ,
thus it does take 6 characters.  Nothing in the terminal can change
that.
The width that strwidth() returns should match what happens on the
display.  With your change, please also test with a character between
0x80 and 0x100.  E.g.:
call setcellwidths([[0x81, 0x81, 1]])
call assert_equal(4, strwidth("\u0081"))

--
The Guide says there is an art to flying,' said Ford, or at least a
knack. The knack lies in learning how to throw yourself at the ground
and miss.' He smiled weakly.
-- Douglas Adams, "The Hitchhiker's Guide to the Galaxy"
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Updated. Could you check it?

Updated. Could you check it?

Thanks.  Looks like vim_isprintc() also needs to be used when
USE_WCHAR_FUNCTIONS is defined, it should return 4 then.
I'll have the changes to the .po files for the next runtime files
update.
--
The only backup you need is the one that you didn't have time for.
(Murphy)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Unable to debug

Hey, thank you very much for developing this plugin. I have encountered some problems in the process of using it and I hope to get your help. Here is my Linux system information and I have followed the tutorial to install dotnet、openssh-server and curl
Linux VM-0-16-ubuntu 4.15.0-159-generic #167-Ubuntu SMP Tue Sep 21 08:55:05 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

And this is the error message

The console output

The code is compiled locally.

From the looks of it, you have a solid connection and it uploaded/decompressed your project which is great.
This is a known issue w/ remote debugging and unfortunately Microsoft's VSDbg doesn't provide much documentation or feedback other than failing to launch.  I have seen firewall issues in the past blocking port access (see 'Additional Resources' below). Also, I've had success debugging console apps on Ubuntu 20.04, but issues on 22.04.
To try and help you out:

Which version of Ubuntu are you using?
Are you able to manually launch and Attach to Process via VS > Debug > Attach to Process...?

Additional Resources:

Remote Debugger Port Assignments
Remote Debugging

Sorry for reopening this issue, but I'm having this exact issue but on the latest version of raspbian instead...
My console output looks exactly like the one provided by the original author and the error popup I get is also the same.
I have allowed the VS2022 port (4026) trough the firewall on the pi. I have also made sure the app is configured for remote debugging.
I can attach to the process via VS > Debug > Attach To Process, works just fine. I have also tried adding:
var _ = Console.ReadLine();
As the first line in my Main();
None of these options have been able to resolve my problem. Any more suggestions?

After some more digging I found this in the debug adapter host log:
ERROR: Debug adapter error output: bash: line 1: /home/retina/.vs-debugger/vs2022: Is a directory
However, I do not understand how to fix this... :)

If I change the launch json adapterArgs -T argument from:
~/.vsdbg/vs2022
to
~/.vsdbg/vs2022
and then launch the debug manually with the command window and command:
DebugAdapterHost.Launch /LaunchJson:"path-to-launch.json" /EngineGuid:541B8A8A-6081-4506-9F0A-1CE771DEBC04
it seems to get further... Seems to me, but Im not sure, that the exported path to vsdbg is incorrect in my launch.json.

thank you @TGlev, reopening this issue

thank you @TGlev, reopening this issue. If you have time, please feel free to jump in to address this issue. Unfortunately, been known to be temperamental between environment-to-environment.

Thanks. Im afraid my knowledge doesnt include visual studio extensions. Havent been able to figure much out regarding this issue on my own.


UPGRADE: Sphinx 6.x.

There should not be any fundamental problem with supporting Sphinx 6.x.
To do

[ ] New myst-parser is released, which will let us test on Sphinx 6 here.
[ ] See if tests are happy or anything broken in the output
[ ] If not, we can merge

Closes #118

Seeing as Sphinx 6 isn't finalized or released yet, it seems prudent to wait for a release before changing this. Major Sphinx releases have a tendency to upgrade docutils, which has a tendency to break things.

Would be great to add support for this as this is blocking PyVista in https://github.com/pyvista/pyvista/pull/3762.

Seeing that docutils 0.18 and 0.19 have already been released for a while, that should not be a source of issues.
Sphinx 6 removed jQuery but I can't find anywhere that sphinx-design uses this?
I don't know anything else that could raise concerns here. The release notes are available here: https://www.sphinx-doc.org/en/master/changes.html

I'm +1 on this, but we'll need to bump the versions to latest of all of the other dependencies in tests that in EBP. They tend to cap the upper version of Sphinx in each one, so I suspect that the version of Sphinx we test here is going to be 5 even if we raise the version of this package to 6.

@choldgraf you're right!
Collecting sphinx<7,>=4
  Downloading sphinx-5.3.0-py3-none-any.whl (3.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 3.2/3.2 MB 28.3 MB/s eta 0:00:00

Seems we need a new release of myst-parser to come out, Sphinx 6 was added yesterday: https://github.com/executablebooks/MyST-Parser/pull/664

Codecov Report
Base: 90.02% // Head: 90.03% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (c4a2fb4) compared to base (6df4751).
 Consider uploading reports for the commit a073d5c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   90.02%   90.03%   +0.01%
==========================================
  Files          11       10       -1
  Lines         942      933       -9
==========================================
- Hits          848      840       -8
+ Misses         94       93       -1

Flag
Coverage Δ

pytests
90.03% <ø> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

sphinx_design/cards.py
95.41% <0.00%> (-0.04%)
:arrow_down:

sphinx_design/__init__.py

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

There is a new MyST-Parser release available:
https://github.com/executablebooks/MyST-Parser/issues/710#issuecomment-1449446108

Hey, thanks for the work on this! Linking #118 for reference here, as it is closed by this PR.

@choldgraf Thank you very much :tada:. Seems that pre-commit CI needs some upgrading though?

Any remaining fixes until version bump?

Here's a PR to cut a new release:

https://github.com/executablebooks/sphinx-design/pull/127

If the tests are happy then I'll merge and release on github 👍


add full screen button for particle vortex

wont'd do as it is available here => https://github.com/ArtemAlagizov/particalizor-3000-vortex


Remove the toplevel modules/ folder?

Is there a reason for the toplevel modules/ folder? Perhaps it would be better to remove it, this way you can download the code directly into your local project's modules/ folder it will work

I think that's a good idea, I went ahead and moved it


swarm.key decryption key is different when passing password via argv or config.yaml

Context
1.6.0
Summary
Download 1.6.0 Darwin ARM64 Bee release and make it executable
Wrong
Create config.yaml with password: Test
Run ./bee init --config=config.yaml and try to decrypt swarm.key afterwards
It fails with "Key derivation failed - possibly wrong passphrase"
Correct
Do the same as before but omit password from config.yaml
Run ./bee init --config=config.yaml --password=Test
The resulting swarm.key can be correctly decrypted with the same password
Steps to reproduce
https://github.com/Cafe137/bee-password

So I ran this on a linux machine by adding this const response = await fetch('https://github.com/ethersphere/bee/releases/download/v1.6.0/bee-linux-amd64')
and the script worked fine with no errors. I wonder if the issue is present for for mac.

I tried it on Ubuntu 20.04 VPS and it is indeed correct
On my M1 MacBook the issue is reproducible with both bee-darwin-amd64 and bee-darwin-arm64

I could not reproduce with bee-darwin-amd64


Too much padding in weather card using sections in HA

Is it possible to remove some padding for example or optimize graphics so that the "bezels" of the card don't look so big when using sections in HA?
Example using old layout:

Example using section layout:

I'm not entirely sure if this component has anything to do with the graphics at all or is "just" gathering the data from DWD.

Hi, this integration only provides the data to the card. So if you want to change this, you can either do this by custom CSS or you ask at HA directly.
Best regards

Ok, thanks for clearing that up.


Fedora 36 Image

Add a Fedora 36 image.
ETA: 19.04.2022

:+1:

Fedora 36 is in beta at the moment, final (end user) release date is 26.04.2022.

I'll keep this open until final release arrives.

Once more delayed, new target date is 10.5.2022.

Fedora 36 is available finally :tada:

PR merged, image is available in the registry :+1:.


add button to table row in order to open a dialog

Hello,
I want to add a button into each table row in order to open a dialog on click. But if I add the button inside/outside the last  column, it can not be clicked. Instead, the row is selected. Is there any solution to implement the described behavior?
Thank you, best, Marcel

You'll need to stop the event from propagating when the cell is clicked.

It works - you made my day! Thanks

It works - you made my day! Thanks


docusaurus-plugin-typedoc - Ability to do something in between Markdown conversion and final build

Hello, and thanks for the excellent plugin.
I don't like some of the things that the plugin is doing, such as naming the root page to "Readme". Specifically, I would prefer it to be named "Introduction".
"No problem", I thought to myself. So I opened up the "docs/api/index.md" file, and changed sidebar_label: "Readme" to sidebar_label: "Introduction".
However, doing this has no effect - the final HTML website is already generated by Docusaurus, so none of the changes that I make to the Markdown files will matter. And if I run npm run build again, it will blow away my changes, because it recreates the Markdown files from scratch from the TypeScript code.
So, I would like this plugin to expose a way to do changes to the Markdown files after they are generated, but before Docusaurus proceeds to convert them into HTML.
Brainstorming:
Perhaps the plugin should have an additional option alongside out, tsconfig, readme, and so on. It could be called "afterMarkdownGeneration", or something along those lines. It would take a reference to a function, and then that function would be synchronously called after all of the Markdown files are generated.

Thank you for this.
On the specific point about the readme sidebar label, although not documented can actually already be achieved by adding sidebar.readmeLabel to the plugin options ie:
 sidebar: {
          readmeLabel: 'Introduction',
        },

I take your point about further customisation to the output, TypeDoc already exposes events to achieve the same thing that can be used with custom plugins.  I have been also been working on refactoring the code and making it easier to customise and override the Docusaurus theme but not quite there yet.

Tom, thanks for the quick reply.

On the specific point about the readme sidebar label, although not documented can actually already be achieved by adding sidebar.readmeLabel to the plugin options

Thanks. I'll try that. Can we document that please? I can submit a PR if you want.

I take your point about further customisation to the output, TypeDoc already exposes events to achieve the same thing that can be used with custom plugins. I have been also been working on refactoring the code and making it easier to customise and override the Docusaurus theme but not quite there yet.

Aye, there are actually many modifications that I want to make to the output, not just the name of the readme. =(
Related: Docusaurus exposes an option called sidebarItemsGenerator that is useful here. (I wanted to customize the sidebar.) But frustratingly, if you specify a specific sidebar label in the sidebarItemsGenerator callback, it won't work, because this plugin also generates a sidebar label in the frontmatter, and the YAML frontmatter "wins".
So, I'm stuck and not sure how to proceed here, I think I might just need to fork the library. I'm also considering just using typedoc-plugin-markdown directly, and then writing a script to modify the Markdown files. Can you give a list of all the things that docusaurus-plugin-typedoc actually does? I see that it is:

generating frontmatter (which affects the sidebar)
arbitrarily removing the first line, which is the "path" to the Markdown file
fixing the title to be more plain
arbitrarily removing the table of contents (which presumably isn't needed with Docusaurus because its on the right hand side)

Besides this, I'm curious if there's any magic going on that it is doing?

Related: Docusaurus exposes an option called sidebarItemsGenerator that is useful here. (I wanted to customize the sidebar.) But frustratingly, if you specify a specific sidebar label in the sidebarItemsGenerator callback, it won't work, because this plugin also generates a sidebar label in the frontmatter, and the YAML frontmatter "wins".

So another thing you can do ( sorry undocumented again :( ) is to pass sidebar.autoConfiguration=false to the options. This will omit all sidebar related yaml items and also the category.yml files.
 sidebar: {
          autoConfiguration: false,
        },

I will update the readme.

Besides this, I'm curious if there's any other magic that it is doing?

Not really - it is a pretty simple wrapper, the main thing as you say is adding yaml to the pages and setting up default options for docusaurus (removing breadcrumbs, page title etc). As I say the aim is to do make it very easy to customise the theme in the same way as the html theme works.

As a small update on this, I ended up deciding to use typedoc-plugin-markdown directly, and then writing a script to modify the Markdown files in an arbitrary way.
In case anyone wants to do something similar, here is my horrible, terrible code: https://github.com/IsaacScript/isaacscript/blob/main/packages/docs/scripts/fixIsaacScriptCommon.ts
Some day, it would be fantastic for the plugin to support some of this stuff directly via configuration options.

https://www.typedoc-plugin-markdown.org/docs/customizing-output


Error installing latest fatal: --local can only be used inside a git repository

On Raspberry Pi4

This updates the permission of all the files in the solarthing repository. Modifications will be made to the directory printed below:
/opt/solarthing
continuing
Updating...
fatal: --local can only be used inside a git repository

Can you run cat /opt/solarthing/other/linux/update_perms.sh and show me the output? Have you pulled/cloned SolarThing recently? I too had this error and I believe I fixed it recently, but I may not have

Also worth noting that if we can't figure out how to fix this error, you can continue on configuring SolarThing (assuming you have an up to date solarthing repo). That error indicates that it was unable to set a flag on the repo, which really isn't that important.
In other words, you could ignore it, but I wouldn't recommend it since I would like to figure out what the problem is.

Might be this:Is there a way for git to identify the git directory? It says- fatal: --local can only be used inside a git repositorystackoverflow.comSent from my iPhoneOn Mar 24, 2023, at 2:19 PM, Lavender Shannon @.***> wrote:
Also worth noting that if we can't figure out how to fix this error, you can continue on configuring SolarThing (assuming you have an up to date solarthing repo). That error indicates that it was unable to set a flag on the repo, which really isn't that important.
In other words, you could ignore it, but I wouldn't recommend it since I would like to figure out what the problem is.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Might be this:Is there a way for git to identify the git directory? It says- fatal: --local can only be used inside a git

So I know that the error message sounds like something is being done incorrectly and that it's not a git repository, but when I fixed the problem I found it was actually because of a permission error.
I asked you to run  cat /opt/solarthing/other/linux/update_perms.sh because I had recently updated that file and wanted to confirm that you were using the new version. You have to give me a little more context of what commands you ran before that one and when you cloned the solarthing repo for me to help.
I'd like to replicate your issue but I'm uncertain if this occurred right after cloning or if you ran the update_perms.sh script directly. Unless you messed up the solarthing repo somehow (unlikely), my guess is that this error is being caused by a permission error.

Hello again. Sorry for not understanding your question earlier. I was trying to understand how you got that error, but I recently tried running the clone install instructions and was able to easily reproduce the error you showed above.
 The second command makes sure that the root user has /opt/solarthing configured as a safe directory. The third command makes sure that your user has /opt/solarthing configured as a safe directory.
I have modified the installation instructions and I have also modified the update_perms.sh script, which is run when running the clone_install.sh script. This should mean that users that are installing Solarthing for the first time shouldn't have this problem.
I imagine other users have had this problem for a while now. The main problem is that I am trying to allow anyone part of the solarthing group access /opt/solarthing, but configuring it correctly has proven difficult. I plan on creating a better way to deal with this in the future as the current setup is something I'm trying to migrate away from.
If you need additional help with permission errors or weird git errors, please respond to this issue. I will close this issue after a little while, but feel free to reopen it or create another issue.


server returned HTTP status 404 Not Found How can I solve it?

server returned HTTP status 404 Not Found How can I solve it?

@SuperYogurt
You can install it from Github. Because seems like in PyPi old version of this package. Author merged pull request with a fix for Python3 and it should work correctly for now.
Problem in this row:
 root.putChild(self.path, MetricsResource())
In PutChild first 'path' parametr shoud be ib bytes type:
root.putChild(self.path.encode('utf-8'), MetricsResource())
After that fix all working fine.


a bug from vs2017

When I compile cpp-netlib(version: 0.12.0)，encountered some error:
1.
thread_pool().post(this, self { handler(request_, self); }); in the file async_connection.hpp，the error msg is:
error C2440: “”: can't convert “initializer list” to “handler”
looks like the "handler" is treated as a TYPE
So I think that the handler maybe conflicted with other "handler", so the right way is:
thread_pool().post(this, self{ this->handler(request_, self); });

callback(make_iterator_range(begin, end), ec); in the file async_normal.hpp, there is no way to convert begin to pointer type, I don't know why, so I changed the code to:
callback(make_iterator_range(begin._Ptr, end._Ptr), ec); and the ERROR is gone, is this change right???
my environment is：
visual 2017
cpp-netlib 0.12.0
openssl-1.0.2
i'am from china and sorry for my engish :)

cpp-netlib 0.12.0

Have you tried branch 0.13-release?

@anonimal not yet, i will try it today, thanks for your reply:)

@anonimal i tried 0.13.0-release with boost 1.64 & vs 2017, all is fine:)


Reactify Class Link editing [#174374264]

@scytacki the issue you found (not being able to sort when starting with 0 links) is fixed.  The Sortable HOC didn't like that I was exiting early and returning undefined when there were no bookmarks.  The fixed commit is here: https://github.com/concord-consortium/rigse/pull/776/commits/f31d63ed85c623a2b2369169567573fa1bc1d8fa


autopilot: feature configuration

In this PR we add support for configuration of features within autopilot. Feature configuration can signal preferences to  autopilot which are not enforceable by the litd firewall.

Default configurations for each feature are available via the ListFeatures call
Support for feature configuration is added to litcli
Pubkeys in configuration strings are obfuscated

Replaced by https://github.com/lightninglabs/lightning-terminal/pull/629 (PR from upstream branch)


Downloading files in new teams from a custom teams app (teams toolkit)

Hello!
Using classic teams i can download files from a custom teams app without issues - when the app first loads, and after already multiple other downloads.
However, when switching over to new teams, i can only do one download after the app loads, and after the initial download it doesn't do any more downloads.
Creating a new local custom app and adding 1 button with an onClick function:
    var a = document.createElement("a"); a.href = window.URL.createObjectURL(new Blob(["CONTENT"], {type: "text/plain"})); a.download = "demo.txt"; a.click();
Allows me to repro this issue.

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

ID: 19ddf42e-0a47-7717-52d4-e549155480a2
Version Independent ID: 4bbe9beb-233f-cfd5-097b-f280aab5fde8
Content: Microsoft Teams developer community support and feedback - Teams
Content Source: msteams-platform/feedback.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

Hello @KristapsD - Thanks for raising your query.
Could you please share the complete code snippets so that we can try to repro it from our end.
Also, let us know whether you have created a Tab or a bot in Teams?
Also, let us know on which Teams client(Desktop/Web/Mobile) you are trying it?

@sayali-MSFT - here is my Teams version:

I've just created a completely new teams app (through teams toolkit) and ran it, the only code i changed was in src > components > tab.tsx
This is the tab.tsx file
import { useContext } from "react";
import { Welcome } from "./sample/Welcome";
import { TeamsFxContext } from "./Context";
import config from "./sample/lib/config";

const showFunction = Boolean(config.apiName);

export default function Tab() {
  const { themeString } = useContext(TeamsFxContext);

  const testDownload = () => {
    var a = document.createElement("a");
    a.href = window.URL.createObjectURL(new Blob(["CONTENT"], {type: "text/plain"}));
    a.download = "demo.txt";
    a.click();
  }

  return (
    <div
      className={themeString === "default" ? "light" : themeString === "dark" ? "dark" : "contrast"}
    >
      <button onClick={testDownload}>Test Download</button>
    </div>
  );
}

These were the setup options:

New project: Tab
App features Using a Tab: React with Fluent UI
Programming Language: TypeScript
Workspace folder: default
Application name: minReproDownload

If you wish i can this to a repository and share it with you.

@KristapsD -Yes sure, please share the repository so that we can try and raise the bug accordingly.

@sayali-MSFT Here's the repo: https://github.com/KristapsD/minReproDownload

@KristapsD  -  We have tried to replicate the scenario from our side. We were successful in downloading the file multiple times in both new teams and classic teams. I have attached a video for reference. Are you trying the same?
https://github.com/MicrosoftDocs/msteams-docs/assets/146196379/ce18d465-9e46-40ac-8aa8-c447ead73aec

@SaiPratap-MSFT That's through the website version is it not? Through a website it also downloads for me properly, but whenever i sideload the local app to the desktop client and try to download multiple times there - it fails. Attached a video:

And heres my current teams version:

@SaiPratap-MSFT How do i get the console error from the desktop application? Is it just collecting the support files?

@KristapsD  -   you can refer this doc

@SaiPratap-MSFT Sorry for asking for help again - but the developer preview was enabled following the doc you provided, and i can see it in the old teams. But i can't seem to find it new teams?

@KristapsD  - We have raised a bug for your issue. We will inform you once we get any update.

@KristapsD  - We are checking your query with engineering team and let you know once we get any updates from them.
Thanks

Hi @KristapsD, where you able to figure out any solution for this issue? We have our teams tab app project running into the exact same issue. As you have mentioned, when running the Tab App inside the browser version of Teams, the download works fine i.e., able to download multiple files, etc. But when running the Teams Tab app inside the newer version of Teams Desktop client, the download seems to fail after the first one! Downloading multiple files together do not work.
Note: If we switch back to the older Teams Desktop client then the download is working fine from our Teams Tab app (doesn't matter if downloading single/multiple files together).
Hi @sayali-MSFT @SaiPratap-MSFT, please let us know if there is any update on this! Thanks!

Hello Everyone- We are actively checking the bug status with engineering team and let you know the updates, if any.

아직도 버그를 추적중입니까? 저 또한 동일한 증상이 있습니다!!

안녕하세요,이 문제에 대한 해결책을 어디에서 찾을 수 있습니까? 팀 탭 앱 프로젝트에서 똑같은 문제가 발생했습니다. 언급했듯이 브라우저 버전의 Teams 내에서 탭 앱을 실행할 때 다운로드가 제대로 작동합니다(예: 여러 파일을 다운로드할 수 있음). 그러나 최신 버전의 Teams 데스크톱 클라이언트 내에서 Teams 탭 앱을 실행할 때 첫 번째 다운로드 이후에 다운로드가 실패하는 것 같습니다! 여러 파일을 함께 다운로드할 수 없습니다.
참고: 이전 Teams 데스크톱 클라이언트로 다시 전환하면 Teams 탭 앱에서 다운로드가 제대로 작동합니다(단일/여러 파일을 함께 다운로드하는 경우 중요하지 않음).
안녕하세요, 이에 대한 업데이트가 있으면 알려주십시오! 감사!

이 현상을 해결하셨습니까? 저도 다운로드가 안됩니다

In the New Teams client, enter the Test Teams app you created above and click the Download Files button.
You can then verify that the file is initially downloaded, and that other files that remain thereafter are not downloaded.
Even if you check the download folder, you can see that only a few of the three docx files have been downloaded.
You can then click the Download Files button again to confirm that there is no response.
Once this happens, the New Teams app continues to download only once, even if you completely shut down and run it again.
And what's unusual is that in that situation, you can download it for the first time if you open a new window with the Open Conversation in a new window function in a specific message on the team channel and use the Custom Teams app in it.
A temporary way to break this situation is to initialize the Teams app in the Settings app, but as mentioned above, if you try to download another file when the existing file is not complete, the same problem will recur again.
It seems to be a problem with NewTeams' built-in web browser WebView 2.
When will it be patched so that it can be processed normally?
Or is there a solution?

Hello everyone!
Today when opening teams i'm greeted with this message:

Obviously meaning that on the first of july me and my other colleagues will be forced on the new teams, where we will encounter the discussed issue.
Is there still no ETA on this being fixed? As i can see im not the only person with this issue aswell.

I have the same problem, I can only download a file once, I cannot download the second file. This happens only with new teams. Waiting for the solution.

Hello Everyone,
Engineering team has fixed the bug internally and we will let you know once it is released publicly.

Hi @ChetanSharma-msft, @Prasad-MSFT, @Meghana-MSFT,
We are experiencing the same issue with multiple PDF downloads in the new Teams desktop client. The downloads only work when the app is loaded for the first time. Could you please let us know when we can expect this fix to be released publicly?
Thank you.

Hello!
July 1st has come and our tenant has been forcefully pushed over to the new teams.
The issue is still not resolved.
What now?

We are also facing similar issue, does changing Teams SDK will solve this issue?


feat: adding simple models and solvers

This PR integrates the simple models (GRF, MVG, joint, convolved_likelihood, etc.) into score_models. It also integrates dedicated SDE and ODE solvers. For now the solvers are only integrated for the score_model.sample function. This is a draft PR to discuss API details and further integration.

Codecov Report
Attention: Patch coverage is 88.13131% with 94 lines in your changes missing coverage. Please review.

Project coverage is 78.64%. Comparing base (d3e1240) to head (d305770).
Report is 11 commits behind head on dev.

Files with missing lines
Patch %
Lines

score_models/architectures/ncsnpp.py
58.18%
23 Missing :warning:

score_models/solver/ode.py
85.71%
12 Missing :warning:

score_models/solver/sde.py
82.60%
12 Missing :warning:

score_models/sde/tsvesde.py
30.76%
9 Missing :warning:

score_models/sbm/mvg.py
95.62%
6 Missing :warning:

score_models/sbm/score_model.py
84.61%
4 Missing :warning:

score_models/sde/sde.py
82.60%
4 Missing :warning:

score_models/sde/vpsde.py
75.00%
4 Missing :warning:

score_models/solver/solver.py
93.54%
4 Missing :warning:

score_models/sbm/base.py
75.00%
3 Missing :warning:

... and 9 more

Additional details and impacted files
@@            Coverage Diff             @@
##              dev       #8      +/-   ##
==========================================
+ Coverage   74.84%   78.64%   +3.79%
==========================================
  Files          46       71      +25
  Lines        2155     3371    +1216
==========================================
+ Hits         1613     2651    +1038
- Misses        542      720     +178

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


bpmn:documentation string for BpmnElement

At the moment I am pulling the whole process XML to get element Documentation.
How difficult would be to add bpmn:documentation into BpmnElement?
Thanks

How difficult would be to add bpmn:documentation into BpmnElement?

It would be easy to add the documentation for the element.
Currently, some information is exposed for a BPMN element but the documentation is not part of it. See the GraphQL schema.
The data of the BPMN elements are collected in the backend by the class ProcessService.kt.
Feel free to contribute a PR for it. :rocket:

Hello,
I did the same thing here: https://github.com/HMubaireek/zeeqs/pull/2
I'll send a PR to the main repo soon


carlaviz stops/crashes when multiple frontend react server request for connection

we built our own docker image having only carlaviz in it and running the react client seperately and providing the serverURL accordingly. We found that this architecture works perfectly as well but the issue is when we have the carla and carlaviz running in remote server and provide the public serverURL to the two react server in the 2 different local system ,the carlaviz crashes i,e. there is no handshake taking place with the new request and old connection also breaks. does carlaviz is built to handle single connection or it can handle multiple request??. If it handles single connection how can we achieve to make it accept multiple connection.

CarlaViz is built for multiple connections. Could you paste the error messages from the backend when it crashed?

Yeah I think so but not yet implemented. Please re-open this issue if you still need this feature.


Linux Daedalus Installation error

Hey there
trying to install daedalus on Ubuntu Distribution... But getting these error messages:
`npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
npm ERR! Linux 4.13.0-36-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>preinstall: node node-scrypt-preinstall.js
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script 'node node-scrypt-preinstall.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the scrypt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node node-scrypt-preinstall.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs scrypt
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls scrypt
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/flo/Programme/daedalus/npm-debug.log
`
npm and nodejs are uptodate
Someone any idea?

you have to look for help in proper repository: https://github.com/input-output-hk/daedalus


[SPARK-23197][STREAMING][TESTS][2.4] Fix ReceiverSuite."receiver_life_cycle" to not rely on timing

What changes were proposed in this pull request?
This patch changes ReceiverSuite."receiver_life_cycle" to record actual calls with timestamp in FakeReceiver/FakeReceiverSupervisor, which doesn't rely on timing of stopping and starting receiver in restarting receiver. It enables us to give enough huge timeout on verification of restart as we can verify both stopping and starting together.
Why are the changes needed?
The test is flaky without this patch. We increased timeout to fix flakyness of this test (15adcc8) but even with longer timeout it has been still failing intermittently.
Does this PR introduce any user-facing change?
No
How was this patch tested?
I've reproduced test failure artificially via below diff:
diff --git a/streaming/src/main/scala/org/apache/spark/streaming/receiver/ReceiverSupervisor.scala b/streaming/src/main/scala/org/apache/spark/streaming/receiver/ReceiverSupervisor.scala
index faf6db82d5..d8977543c0 100644
--- a/streaming/src/main/scala/org/apache/spark/streaming/receiver/ReceiverSupervisor.scala
+++ b/streaming/src/main/scala/org/apache/spark/streaming/receiver/ReceiverSupervisor.scala
 -191,9 +191,11  private[streaming] abstract class ReceiverSupervisor(
       // thread pool.
       logWarning("Restarting receiver with delay " + delay + " ms: " + message,
         error.getOrElse(null))
+      Thread.sleep(1000)
       stopReceiver("Restarting receiver with delay " + delay + "ms: " + message, error)
       logDebug("Sleeping for " + delay)
       Thread.sleep(delay)
+      Thread.sleep(1000)
       logInfo("Starting receiver again")
       startReceiver()
       logInfo("Receiver started again")

and confirmed this patch doesn't fail with the change.

cc. @dongjoon-hyun

Test build #111367 has started for PR 25930 at commit 8772f70.

Can one of the admins verify this patch?

Test build #111367 has finished for PR 25930 at commit 8772f70.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class MethodsCallRecorder

Merged build finished. Test PASSed.

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

Merged to branch-2.4.


EarlGrey 2 folder structure in setup guide is incorrect

After running the following two commands as instructed by the setup guide...
git clone -b earlgrey2 https://github.com/google/EarlGrey.git
sh Scripts/download_deps.sh

... the eDistantObject and fishhook folders are not top-level as described here but nested within a Submodules folder.
I wonder whether this is linked to some of the issues related to eDistantObject which I've seen being reported? I haven't got past this step yet so it might not be a problem but thought I'd call it out before I forget.

Thanks for pointing this out. I'll update it. So, the key is the step mentioned where you have to run this from the EarlGrey/ directory. After this, make sure both EarlGreyV2/** and eDistantObject/** are added to your search paths.


Upgrade ipfs deps; use test-util package

tests are failing due to the orbit-db devDep used in tests

dont merge yet, still using github deps

Note this repo does NOT have the @next auto-publish capabilities


Feature Request: Support named instances of services so that multiple credentials can be configured

I've only used the CLI to interface with Artifactory, so I don't know if this would make sense for Bintray, MX, or XRay.
Currently in order to use stored credentials I have to rerun the config every time I switch between production and staging servers. While I can use command line arguments such as --url --user --api-key I'd rather have all my instances url/user/key in the config file, and a single --instance option that selects from them.

Thank you @jgeorgeson. This is indeed something we plan to add in the future.

This would be a nice feature.  Currently, switching between servers is cumbersome.  Thanks.

In the meantime I have a shell function I use to switch
set-jfrog() {
    [ -f ~/.jfrog/jfrog-cli-$1.conf ] || { echo "No config file for \"$1\"."; return; }
    cp ~/.jfrog/jfrog-cli-$1.conf ~/.jfrog/jfrog-cli.conf
}

export -f set-jfrog

The enhancement of this feature was commited to our dev branch, you can see the commited code, build our dev branch and use it.
As part of this enhancement the configuration file has changed, we added a serverName per Artifactory instance configuration and one more field we need in order to get the default server.
To add a new server run:
jfrog rt c serverName
In order to set a server to work with use :
jfrog rt c use serverName
To list all the servers use :
jfrog rt c show
In addition, each of the Artifactory commands now has an additional flag --server-name.
please notice, this is not our final version and things can be change later on.
Once you use this version the configuration file will be changed so older versions of the cli won't be able to use it.

@jgeorgeson,
Version 1.8.0 has been recently released, and supports configuring multiple Artifactory instances using the jfrog rt config command. You can then switch between your configured Artifactory servers using the jfrog rt use command, or alternatively pass the --server-id option to the CLI Artifactory commands.

Nice! I'm having a little trouble with it though. I ran jfrog rt c for each of my servers, and the resulting ~/.jfrog/jfrog-cli.conf only contains the last server I ran the config for. If I manually set each instance in ~/.jfrog/jfrog-cli.conf then I can select the instance with jfrog rt use. Is there a way to print the current default server ID without looking at the conf file?

Hi @jgeorgeson,
The jfrog rt c command can now receive an optional argument with the server ID value.
For example, running jfrog rt c rt-1 configures a new server with rt-1 as its server ID.
If you'd like rt-1 to become the default server, you'll then need to run jfrog rt use rt-1.
Running jfrog rt c without specifying a server ID overrides the configurrd default server.
Please let us know if this helps and resolves the issue.

Ok. I was expecting the config command's documentation to be at the top of the page so I didn't see it at first. Everything appears to be working as documented now. Thanks.


[#3769] fix(client-python): fix invalid-field-call issue

What changes were proposed in this pull request?
Fix  invalid-field-call Pylint rule
Why are the changes needed?
Fix: #3769
Does this PR introduce any user-facing change?
No
How was this patch tested?
./gradlew clients:client-python:test

I think this PR should be reviewed after  #3798, since some changes are based on that PR.


Flask crashes when static files are served (ProgrammingError: Recursive use of cursors not allowed)

When I enable the flask-profiler, the app crashes when files from the static folder are served:
Traceback (most recent call last):
  File "flask\app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "flask\app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "flask\app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "flask\app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "flask\app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "flask\app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "flask\app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "flask\app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "flask_profiler\flask_profiler.py", line 91, in wrapper
    return wrapped(*args, **kwargs)
  File "flask_profiler\flask_profiler.py", line 72, in wrapper
    collection.insert(measurement.__json__())
  File "flask_profiler\storage\sqlite.py", line 128, in insert
    name))
ProgrammingError: Recursive use of cursors not allowed.
Traceback (most recent call last):
  File "flask\app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "flask\app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "flask\app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "flask\app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "flask\app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "flask\app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "flask\app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "flask\app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "flask_profiler\flask_profiler.py", line 91, in wrapper
    return wrapped(*args, **kwargs)
  File "flask_profiler\flask_profiler.py", line 72, in wrapper
    collection.insert(measurement.__json__())
  File "flask_profiler\storage\sqlite.py", line 128, in insert
    name))
ProgrammingError: Recursive use of cursors not allowed.

I first though that this is, because requests to /static are not mentioned in the app routes, but also with a route with return app.send_static_file('index.html') I get the same error. All other routes (none are serving files from static) run fine.
I am using:

Python 2.7.5
Flask 0.10.1
Flask-Profiler 0.5

I am happy to provide more information if needed.

Hi nlohmann,
I am facing the same issue. Did you get to resolve the issue?
Thanks
Subhash

We could not resolve the issue and decided to stop using Flask Profiler.

Oh I am putting a fix and let you know if it works. However, did you get any other open source project to do the same.

No, we implemented some profiling ourselves. Nothing fancy, and a plugin like this would be appreciated.

Was a solution for this problem found? I'm getting something very similar and cant figure it out.
Traceback (most recent call last):
    File "\lib\site-packages\flask\app.py", line 1997, in __call__
        return self.wsgi_app(environ, start_response)
    File "\lib\site-packages\flask\app.py", line 1985, in wsgi_app
        response = self.handle_exception(e)
    File "\lib\site-packages\flask\app.py", line 1540, in handle_exception
        reraise(exc_type, exc_value, tb)
    File "\lib\site-packages\flask\app.py", line 1982, in wsgi_app
        response = self.full_dispatch_request()
    File "\lib\site-packages\flask\app.py", line 1614, in full_dispatch_request
        rv = self.handle_user_exception(e)
    File "\lib\site-packages\flask\app.py", line 1517, in handle_user_exception
        reraise(exc_type, exc_value, tb)
    File "\lib\site-packages\flask\app.py", line 1612, in full_dispatch_request
        rv = self.dispatch_request()
    File "\lib\site-packages\flask\app.py", line 1598, in dispatch_request
        return self.view_functions[rule.endpoint](**req.view_args)
    File "\lib\site-packages\flask_httpauth.py", line 93, in decorated
        return f(*args, **kwargs)
    File "\lib\site-packages\flask_profiler\flask_profiler.py", line 212, in getRequestsTimeseries
        return jsonify({"series": collection.getTimeseries(args)})
    File "\lib\site-packages\flask_profiler\storage\sqlite.py", line 158, in getTimeseries
        self.cursor.execute(sql)
ProgrammingError: Recursive use of cursors not allowed.

The problem here is with a shared sqlite3 cursor.
https://github.com/muatik/flask-profiler/blob/b9d4b7ab6341d60f857ec48d259210fd7f25a7f7/flask_profiler/storage/sqlite.py#L33
https://stackoverflow.com/questions/26629080/python-and-sqlite3-programmingerror-recursive-use-of-cursors-not-allowed

FWIW, I was using flask-profiler 1.7 all this while and didnt face an issue for a year. I just moved to 1.8.1 and am seeing it multiple times.

same to me. error on 1.8.1, but works fine on 1.7


WIP: Lookup refactor

Lookups have been refactored.
For each lookup type (SubjectLookup, SubjectPredicateLookup, PredicateLookup, ObjectLookup) there is now a 'Layer' version (LayerSubjectLookup, LayerSubjectPredciateLookup, LayerPredicateLookup, LayerObjectLookup) which represents a lookup in either the layer's additions or the layer's removals, without taking parent layers into account.
The original implementations for SubjectLookup, SubjectPredicateLookup, PredicateLookup and ObjectLookup have been removed, and have been replaced by single implementations, GenericSubjectLookup, GenericSubjectPredicateLookup, GenericPredicateLookup and GenericObjectLookup. These are implemented in terms of the new Layer Lookup traits.
Unlike the old Lookups these are not implemented recursively, so stack exhaustion should no longer be an issue.

Hi! Do you mind if I have a go at working on your PR today? I have already done some work on the lookups and iterators, and I think I can improve things a bit here.

I can leave a PR to your lookup_refactor branch when I'm done.

Hi! It's great that you want to work on this!
However, I'll likely be merging this today, as we are actually running into stack exhaustion issues which we need to resolve quickly. I'm just awaiting some test feedback on this.
Feel free to base your work off this though, and create a new merge request when you got something ready (against master, assuming this is merged by then).
What are you going to improve?

Thanks for your response.

Feel free to base your work off this though, and create a new merge request when you got something ready (against master, assuming this is merged by then).

Sure.

What are you going to improve?

I believe I can remove a lot of the Boxes, dyns, and clone()s.
While working on this, I've found that some lower-level changes would help significantly. I'll introduce PRs for those first.
If you have time, I would appreciate feedback on my currently open PRs. Thanks!


typeerror

When I accidentely entered rp(0,0)rpunix instead of hp(0,0)unix I got the following error on the console:
Traceback (most recent call last):
  File "/home/folkert/t/python-pdp11-emulator/boot.py", line 394, in <module>
    boot_unix(p, runoptions=runoptions, **unixboot_options)
  File "/home/folkert/t/python-pdp11-emulator/boot.py", line 332, in boot_unix
    p.run(pc=0, **runoptions)
  File "/home/folkert/t/python-pdp11-emulator/machine.py", line 590, in run
    self.go_trap(self.get_synchronous_trap(abort_trap))
  File "/home/folkert/t/python-pdp11-emulator/machine.py", line 685, in go_trap
    self.logger.debug(f"TRAP: {trap}:\n{self.machinestate(fmt=oct)}")
                                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/folkert/t/python-pdp11-emulator/machine.py", line 1046, in machinestate
    pfx = '0' if v > 7 else ''
                 ^^^^^
TypeError: '>' not supported between instances of 'AddressError' and 'int'

Bug in machinestate when using octal formatting for logging, because the result of reading what MMR2 points to can be an AddressError(). Fixed - but this also makes me wonder if doing that "invisible read" for logging is even a good idea at all.
Why it crashes: The boot.c code (from unix7) has a table of devices in it, and "rp" is one of those devices. Same thing would happen if you type any other known (to the bootstrap) device that isn't present. What happens now with this fix is that the boot code hits a halt instruction and exits the emulator (cleanly). In SIMH if you do this, the boot code hangs -- you never get a new prompt but at first glance I'm not sure where the boot code is hanging/looping (but doesn't seem to hit a halt instruction). This is pointing at some underlying difference that is probably another bug somewhere - maybe AddressError is the wrong trap in this situation? Not sure.


staging-next 2022-05-12

workflow docs: nixos.org/manual/nixpkgs/unstable/#submitting-changes-commit-policy
constitutents: hydra.nixos.org/job/nixpkgs/staging-next/unstable#tabs-constituents
jobset: hydra.nixos.org/jobset/nixpkgs/staging-next
nix-review reports: https://malob.github.io/nix-review-tools-reports/
previous staging-next: https://github.com/NixOS/nixpkgs/pull/171497

The 22.05 release schedule plans merge (of this PR) on May 15th; that's perhaps even realistic.

[ ] gnat regressed, probably after updating maintenance (!) patches in glibc.  Say, is @sternenseemann interested in looking at it?

Over longer term it might be difficult to maintain the bootstrap from such old version (gcc/gnat 6).

Today is the day of the planned merge, but to me it doesn't seem suitable yet.

[ ] libpcap split caused several build regressions apparently:
https://github.com/NixOS/nixpkgs/pull/170415#issuecomment-1127875181

[ ]  libpcap split caused several build regressions apparently:
libpcap: move dev things to extra output #170415 (comment)

173297

Current regression list:

https://hydra.nixos.org/eval/1762178?compare=1762151
https://hydra.nixos.org/eval/1762178?compare=1762176  (as aarch64-darwin is separate from trunk jobset)

I consider that merge-able, except that 20k builds are still queued.


[1.x] Add new expectations

This PR adds expectations for collections that contains models, these expectations are assertions on whether a collection contains one or several models.

No plans for those. Thanks!


Question about Performance

Do you have experience about handling big data sets?
I found this blog post: http://josh.zeigler.us/technology/web-development/how-big-is-too-big-for-json/
Do you have numbers like that with Amygdala ?

We haven't done any specific performance testing. The main use of this library for us has been to remove some boilerplate from front-end code that needs to interact with a REST-based API. Our main use case has been for infrequent AJAX'ed CRUD operations.