changed JanetTArrayView to use JanetBuffer

Definitely breaking backwards compatibility but IMO worth it. The only issue with the native buffer is a 2GB limit, which may be an issue

i see, i didnt pay attention about how large the two buffertypes could get... perhaps capa and count could be moved into a struct in a union with a size_t size? (but that would break even more ._.) like so:
struct SizeInfo {
    int32_t count;
    int32_t capacity;
};

struct JanetBuffer {
    JanetGCObject gc;

    union {
        SizeInfo s;
        size_t size;
    } sizing;

    uint8_t *data;
};

disregarding the bad var names, this way the JanetBuffer type could stay the same size. when using it as a string buffer it could behave as before, except accessing count and capa via sizing, and in a typed array it could use the 64bit size. (of course this complicates things i had in mind where you could just write into a buffer and then plug it into a ta-view -.-)
as it is now when calling ta/properties, the buffer is always shown as empty since count stays 0. but i would rather only print the first and last couple of bytes (if at all). similar to how large arrays are printed.
another thing with potential for bugs (at least in heteroendian (is that even a word? XD) systems/networks): i moved the endianess-holding-flag from buffer to view; perhaps endianess could be resolved on the fly by the view upon access, just like the datatype, this way you could have one buffer and interpret it in many different ways. in this case i would add a function to write a copy of the buffer with the other endianess baked in for a permanent speedup

have yet to figure this out. the test i added iterates over a table, in this table are 4 identical tarrays, for iterates over the range of the arrays, inside the forloop is eachp eachk or each (tried them all but its all the same template function anyways..) iterating over the table. and after some amount of iteration (have observed anywhere between 2600 and 9600 but the higher number was achieved by calling getline before the innerloop @_@ ), the key from eachp/eachk is either garbage (usualy a very small double like 1.02255e-312) and the loops continue for a bit, or it just crashes... im certain it is my code that is doing silly things, but by now i have run out of ideas where or how to look... i hope monday will be a better day to debug this ._.

A number of issues looking at the code, don't use janet_buffer_init to get a garbage collected buffer. Also things like returning buffer->capacity instead of buffer->count for the buffer length seems wrong


HoloToolKit.Unity.XdeGuestLocator - Failed to locate internal switch adapter

Overview
I'm working on a Hololens project with two other developers. We're all on the same Unity version, and the toolkit files are in source control, so we are all using the same version of that as well. In the past, on an old version of HoloToolKit (before it was Mixed Reality Toolkit), I've been able to use "Build SLN, Build APPX, then Install" option on the Build Window to deploy to our Hololens.
Recently I reinstalled windows on a new SSD, so I had a brand new installation of Visual Studio (VS2017 4.4.X or around there), Unity, and then made a fresh pull from our repository. When I try to build the Hololens app, I get the "Failed to locate internal switch adapter" error. This has happened on two completely different computers, but does not happen for my coworkers.
What I've Tried
So far, I've tried a number of different options. I've updated Windows, Reinstalled Unity, checked my Visual Studio packages, tried at least two different versions of Visual Studio 2017, tried repairing Visual Studio, installing the Hololens Emulator, deleting CSPROJ/SLN files and regenerating them, and so on.
I've looked at this old issue, but nothing from there seemed to help. I haven't tried upgrading/downgrading Unity yet because my coworkers aren't having any problems with the same version.
At this point, I'm wondering if there's just some small or more obvious thing that I can't think of that might explain the problem.
Unity Editor Version
Unity 2017.2.0f3
Mixed Reality Toolkit Release Version
2017.2.0
Thanks!

#1471 fixes this error from showing up in the editor.


Norwegian code (no) translations are missing

Internal: b/150470184
There are no _no files here https://github.com/flutter/flutter/tree/master/packages/flutter_localizations/lib/src/l10n. It seems like we are not generating strings for Norwegian. 'no' is the Google-canonical language code for Norwegian.

We support Bokmål as the standard code nb. We don't support Nynorsk (nn).
Are you asking for Nynorsk translations? Or?
There's not really a language code no.

It would be Bokmål. I think the keyword here was Google-canonical. When Google says no, they mean nb.
Would you be comfortable having Flutter localization delegates to use this alternative code as well and map it to nb or do you prefer this to be google3 specific?

The deprecated Android/Java locales are modernized test in packages/flutter_localizations/test/override_test.dart verifies that we map some obsolete locale names to their modern equivalents, but it doesn't cover no (which we supported a few years ago - https://github.com/flutter/flutter/pull/17788).  It looks like no isn't "Google-canonical" is a Java issue - http://www.localeplanet.com/java/no-NO/index.html. So perhaps we should automatically convert no to nb.

CC @shihaohong

It looks like we've been incorrectly leaving out the 'no' (Norsk) locale, see https://en.wikipedia.org/wiki/List_of_ISO_639-1_codes
Even though Google's translations for 'nb' and 'no' are typically identical, it should be a supported locale per https://api.flutter.dev/flutter/flutter_localizations/GlobalMaterialLocalizations-class.html

Based on my current understanding, nb, no, and nn are distinct locales according to ISO 639. It seems like the original transition from no to nb was necessary because Android devices use recognize nb and not no.
However, as @HansMuller has pointed out, Google's translations for nb and no are essentially identical.
We could solve this problem by adding logic to generate nb localization delegates off of _no.arb when _nb.arb is not present. This would allow us to handle nb for Android devices, as well as correctly pulling down the _no.arb files without having to do the manual change from _no.arb to _nb.arb.

I don't have a strong opinion here. If we want to support no and nb as synonyms that works for me.

I'm going to share some relevant context from discussing with the i18n experts:
For Android `no` is never used.

Before Android N there was no way for a user to select `nn`, only `nb`
Starting with Android N one can select `nb` and / or `nn`.

There are no fallbacks or aliases, they behave as different languages
(but no can never be selected in user settings)
...
In general no is not recommended, as it is ambiguous.
IANA lists it as a macrolanguage, with nb and nn as "members"

If one has to do a "naive" mapping then it might map no == nb
A better mapping might interpret no as "some kind of Norwegian", so it might
match both (giving priority to nb if both exist)
LanguageMatcher is does "the better mapping", but it is not used in Android.

So in theory no could be used to indicate Norwegian in any form, eg. spoken or
written as nb or written as nn.

In practice Bokmål is the preferred written form for 90% of the population of
Norway, so for a lot of people and for a lot of software the language codes no
and nb are simply taken to mean exactly the same thing.

My intention is to have no and nb be synonyms for now, since Android only recognizes nb vs nn.


Movesearch: Add ability to search for moves in more than one Pokemon's learnset

I saw some use for this for some OMs but didn't think about how this would affect the param hunts in Scavengers, so this change is awaiting approval by room staff there. Please don't merge until then. :)

This pull request introduces 3 alerts when merging 81b7df0a16656a6c6cf2dce226cf32379cef0782 into 233ffd2cba79ac0accc04f33089526b405389cb0 - view on LGTM.com
new alerts:

3 for Useless conditional

Comment posted by LGTM.com

The opinion was pretty split. @Zarel you can merge if you want. Really up to you on whether or not a split opinion is good enough to push this through.

I mean, this doesn't look like bad code... I really don't want to make calls on movesearch, though.
@bumbadadabum or @HoeenCoder, opinions?

I'll probably get to this on tuesday (or sooner if i get some time).


Update helm charts

@MujkicA - thanks for the PR

this file is empty- helm/fuel-block-committer/templates/config.yaml

how does the config.yaml get places in that mount path -  "/mnt/config/config.yaml"- it needs to be there within the container - like get copied in Dockerfile

@MujkicA - sent some comments over slack but refer to https://dev.to/joshduffney/kubernetes-using-configmap-subpaths-to-mount-files-3a1i- your config.yaml is empty will need to see that to assess how this works- but the article I will sent will show you to use a subpath- everything looks good overall- just some changes

hi @MujkicA -

your configmap has one extra data field in it - I don't think line 6 should be there

For line 71 - given you reference "committer-config" key- in helm/fuel-block-committer/templates/deployment.yaml it should be your configmap otherwise I think deployment will fail


nixos-rebuild --install-bootloader does not reinstall the bootloader

Describe the bug
Contrary to https://nixos.wiki/wiki/Bootloader#Re-installing_the_bootloader adding --install-bootloader to nixos-rebuild... does not reinstall the boot loader.
Steps To Reproduce
Steps to reproduce the behavior:

sudo nixos-rebuild --install-bootloader switch

Expected behavior
nixos-rebuild should print roughly the following
Copied "/nix/store/3s1ygbkri8rc2z39cv53z55z4cqldpwq-systemd-247.6/lib/systemd/boot/efi/systemd-bootx64.efi" to "/boot/EFI/systemd/systemd-bootx64.efi".
Copied "/nix/store/3s1ygbkri8rc2z39cv53z55z4cqldpwq-systemd-247.6/lib/systemd/boot/efi/systemd-bootx64.efi" to "/boot/EFI/BOOT/BOOTX64.EFI".
Random seed file /boot/loader/random-seed successfully written (512 bytes).
Successfully initialized system token in EFI variable with 512 bytes.
Created EFI boot entry "Linux Boot Manager".

Additional context
A change in ~2019 added sudo -- as a prefix to various calls including switch-to-configuration whenever $SUDO_USER is set. This happens for both sudo nixos-rebuild ... and when run as root. (during the build? process sudo_user is set to root)
--install-bootloader relies on  exporting NIX_INSTALL_BOOTLOADER to communicate through? switch-to-configuration.pl to systemd-boot-builder.py. sudo -- does not preserve NIX_INSTALL_BOOTLOADER effectively disabling this option.
Notify maintainers
@zimbatm ?
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.66, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.15`
 - channels(<user>): `"home-manager"`
 - channels(root): `"nixos-21.11pre325810.2deb07f3ac4"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

diff --git a/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh b/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh
index 198a6eae600..6928a4734bb 100755
--- a/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh
+++ b/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh
@@ -127,7 +127,7 @@ while [ "$#" -gt 0 ]; do
 done

 if [ -n "$SUDO_USER" ]; then
-    maybeSudo=(sudo --)
+    maybeSudo=(sudo --preserve-env=NIXOS_INSTALL_BOOTLOADER --)
 fi

 if [ -z "$buildHost" -a -n "$targetHost" ]; then

That will fix the issue for local builds. I'm not sure if maybeSudo also needs to be changed when --use-remote-sudo is set.

@anund in 73369f8d0864854d1acfa7f1e6217f7d6b6e3fa1, I see:
https://github.com/NixOS/nixpkgs/blob/73369f8d0864854d1acfa7f1e6217f7d6b6e3fa1/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh#L131-L133
And at the top of the file:
https://github.com/NixOS/nixpkgs/blob/73369f8d0864854d1acfa7f1e6217f7d6b6e3fa1/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh#L33-L34
So to what version of the file did you mean to apply that patch?

@doronbehar https://github.com/NixOS/nixpkgs/pull/144429/files is the associated pr. Both lines you're referencing are part of the change. it used to be ~maybeSudo=(sudo --) which cleared the env "unsetting" NIXOS_INSTALL_BOOTLOADER

Bah, Closed via https://github.com/NixOS/nixpkgs/pull/144429


PlacementGroup and AdditionalPackages Unknown field error

Hello,
I updated to pcluster version 3 and I'm getting this error bellow. Any help would be appreciated it.
Required Info:

AWS ParallelCluster version [3.0.0]:

Error:

Bug description and how to reproduce:
run:
pcluster create-cluster --cluster-name smahane-clusters --cluster-configuration config.yaml
Config file:
pcluster-v3.txt

Hi Smahae,
The PlacementGroup should be under Networking. The AdditionalPackage should be on the root level. This example configuration file is useful. We will work on improving our documentation
Thank you!
Hanwen

It is supported. Just add the dot in the middle of the compute instance type (e.g. m6i.32xlarge). The instance on the head node is written correctly.

yes it was a typo. thanks for catching it.
Now I'm getting this error that i don't understand :

I have the region set in the config file

Any tips of how to debug/validate the config file?

I couldn't understand why it is complaining about the region. Can you attach your new configuration file (without sensitive information) and the command used?
In terms of debugging/validating the config file, there are generally two types of errors. "ConfigSchemaValidator" error mean the basic format of the config file is wrong (like your first error message). Other errors are more related to the cluster functionality (like your second error message).

Please see attached:
pcluster-v3 - Copy.txt
This is the command:
pcluster create-cluster --cluster-name smahane-clusters --cluster-configuration config.yaml
Thank you

Try pcluster create-cluster --cluster-name smahane-clusters --cluster-configuration config.yaml  --region us-west-1
The region arg shouldn't be necessary. I will look into it tomorrow

Hi, were you able to run pcluster create-cluster --cluster-name smahane-clusters --cluster-configuration config.yaml --region us-west-1? I couldn't reproduce the region error.

Hello,
Yes the --region arg helped me move forward. However, I have many other issues;

I can't find an AMI centos7 based that support m6i instances. centos7 is needed for IntelHpcPlatform: true
The Placement Group 'Cluster' is unknown . What should it be? That's what i could find in the docs.
Many errors related to EFA

Please see the error file. I would appreciate any insights
pcluster-errors.txt

Hi Smahane,
ParallelCluster can create a placement group or use an existing placement group
To let ParallelCluster create a new placement group:
Networking:
  PlacementGroup:
    Enabled: true

To let ParallelCluster use an existing placement group:
Networking:
  PlacementGroup:
    Enabled: true
    Id: your-existing-placement-group-name

I suppose the misunderstanding was the Id parameter was to specify the name of the new placement group.
But the current behavior is that the Id is used to provide the name of an existing placement group. If you would like ParallelCluster to create a new placement group, you should only use the Enabled: true and cannot specify the name/id of the new Placement group.

That helped a lot. Thank you.

How to solve these errors please? Do I have to create an IAM with pcluster? if so, how?
How about the EFA error?

If the EFA is enabled for a compute resource, the placement group of the networking of the queue needs to be explicitly specified.
For example:
...
SlurmQueues:
...
    - Name: queue2
      Networking:
        ...
        PlacementGroup: # This section is a must because there is a EFA setting in the ComputeResources
          Enabled: true # You can set it to either true or false. But true is recommended for better performance
      ComputeResources:
        - Name: compute-resource-1
          ...
          Efa:
            Enabled: true
            GdrSupport: false
         ...

For the AMIs, I believe you are using a custom AMI?

No It's an AWS linux2 AMI from the market place

If you have no specific requirement of the AMI. You could write the image section without providing the custom AMI. e.g.:
Image:
  Os: alinux2

Therefore, ParallelCluster will use an AMI with the corresponding OS.
If you are interested in using another AMI from the market place as a base AMI, you have to use build-image command first to create another AMI from the base AMI and then use it for create-cluster command

Here is the minimal configuration file to create a cluster.

Now the cluster creation fails and rolls back with no errors clear errors in the pcluster-cli.log

Can you share the latest configuration file (without sensitive information) and what warnings were printed when creating the cluster? What is the failure message on CloudFormation?
Because the new commands do not wait for the execution. So the CLI log cannot capture the roll back failure. However, we implemented other commands (e.g. get-cluster-stack-events) to get information about the stack events


No pdf download on mobile?

:(

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

ID: 590ef2fd-0b1a-a2e0-d6a5-109f997f1db7
Version Independent ID: 913df696-e028-f6b4-2a41-f203798badb0
Content: Microsoft Writing Style Guide Released
Content Source: teamblog/style-guide.md
Product: team-blog
GitHub Login: @dend
Microsoft Alias: danielfe

PDF downloadable version could be useful, indeed.


navigation correction

The navigation should show:
SharePoint admin center > Policies > Sharing

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

ID: 030743fd-0c22-025b-f190-f637a2772506
Version Independent ID: fbded472-e3ea-4297-c527-810e117ea7ea
Content: Microsoft 365 guest sharing settings reference
Content Source: microsoft-365/solutions/microsoft-365-guest-settings.md
Product: microsoft-365-enterprise
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@msbemba Thank you very much for the contribution and updating the documentation with PR. @fixx-er Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[FEATURE request] Command-line interface and language-agnostic configuration language

For use in other languages, it would be great to expose a C FFI and (more easily done first) a CLI interface with a language-agnostic configuration language, something like:
# Global to all service-manager-rs
constants: &global_const
   ${if_win}:
      exe: ".exe"
   ${if_not_win}:
      exe: ""

Then you'd roughly have:
name: service_name
constants: &constants
    prefix:
        ${if_win}:
            "%TMP%\foo"
        ${if_not_win}:
            "/tmp/foo"
lifecycle:
   start:  my_init*global_const.exe --prefix *constants.prefix start
   reload: my_init*global_const.exe --prefix *constants.prefix reload
   stop:   my_init*global_const.exe --prefix *constants.prefix stop

Which could be used like so:
$ server-manager install -c "my_yaml_file.yml"
$ server-manager remove "service_name"

And alternatively be specified all at the CLI:
$ server-manager install "my_init" --start "my_init start" --reload "my_init reload" --stop "my_init stop"
$ server-manager remove  "my_init"

@SamuelMarks I use this library in some of my own CLI tools. Do you see there being much demand for a standalone CLI that offers the functionality you described? Is yaml the popular configuration language to use?
This wouldn't take very long to implement, but thinking of the value here.

@chipsenkbeil YAML is pretty popular, as is TOML, JSON and what-have-you. YAML seemed like the simplest syntax to get this example working for you.
I think a CLI would create a whole of new use-cases. For example, I'm working on a large number of new package-managers; originally written in C; then Go; and now Rust (also ² & ³).
(but for outside contributors I wouldn't want to limit their implementation language)
If you think about what is required to build a package manager, it is:

Download
Extract
Dependency resolution (libsolv I guess works here - is open-source and cross-platform [even to Windows])
"Install"
Verification (for download, extract, dependency-resolution, install)

Now the install step might also involve installing services. I haven't seen any other project like yours which does it in such a cross-platform manner.
Kudos!
So what do you think; have I made a sufficient case for being language agnostic and/or creating C bindings? =)
Thanks for your consideration


Session detail page design

should

include all session details expanded
have a speaker(s) summary module teasing speaker full page
link to slides
be the stream
feature this ux https://samdutton.github.io/transcripter/lK3OiJvwgSc.html
nice to have: next/prev session buttons
CTA to buy tickets/register

Always:

include all session details expanded
have a speaker(s) detail module

Before the event:

Module to buy tickets/register

Before and during the event:

A way to 'star' this session and get notifications
nice to have: next/prev session buttons (in schedule order, if there's a schedule)
nice to have: Also on at this time (if there's a schedule available, and it's a multi-track conference)

During the session:

"Live now!" link to livestream

After the event:

Video if available
Links to assets if available

@argyleink any chance of doing this in the next few days? We only need the 'always' and 'before the event' states for the 20th launch.

good timing, v1 is ready to review, so we can squeeze an iteration in if we want. i struggled with this page, it has the most turnover and iteration than any other. curious whatcha think.

Desktop v0.0.1

Mobile v0.0.1

FYI - I'll be looking at limiting the titles to 75 characters max.  This is a good thing on all levels, but it is a requirement for YouTube.

@argyleink what do you think about moving the date/time/cal bit to the space in the right-hand side of the title? No need to comp it if you're pushed for time. We can just do it.
We should also have a way to toggle time between event time and user's time.

good call, details to the right of the title look great 👍
working on timezone toggle in #261 💀 🤘

Desktop v0.0.2

Mobile v0.0.2

@argyleink What is the difference between the left lorem ipsum and the right lorem ipsum in the desktop view? Is the right one the transcript? Or the long abstract? (I don’t think we will have short and long abstract. Just one paragraph per talk or something)

idea for the left lorem area was:

spot for any session meta embellishments about the team/duo/talk/demo/short/tldr abstract/etc
follows the design patterns from the rest of the pages, so was a consistency thing and an overflow space in case content authors needed it

notice how it's not in the mobile comp (the left lorem area), and that's because it's superfluous/meta content that's not critical. great candidate for an if/else content block, if we have it great, if we dont, no biggie.
the right lorem area was for the long abstract / session description
that help? i made it sound like we dont need it lol, which maybe we dont. i'll chat with content folks to see if it's a useful space or it's unneeded.


Add Lab doc

Add Lab doc and fix location and some markdown and syntax errors.

/lgtm
/approve

/retest


Override environment file changes

Clear and concise description of the problem
Hi! I tried this plugin with the hope that the restart of the dev server could also catch changes in .env files that vite uses (https://vitejs.dev/guide/env-and-mode.html#env-files), to avoid restarting the dev server manually each time there is a change in the dev setup with environment variables.
Suggested solution
I managed to have the desired result by calling dotenv within my vite.config.js:
export default defineConfig(({ mode }) => {
  require("dotenv").config({ path: ".env", override: true });

  return {
    ...
    plugins: [
      ...
      ViteRestart({ reload: [".env"] }),
    ],
  };
});

Is there a cleaner alternative to doing this? Thanks!
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

This works by default with the latest version of vite


Fix for checking duplications in einsum

For now, turn off considering the number of duplications since it becomes slow...

Codecov Report

Merging #21 (2bbb8b6) into main (b9467f2) will decrease coverage by 3.03%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   89.48%   86.44%   -3.04%
==========================================
  Files          11       10       -1
  Lines         637      627      -10
==========================================
- Hits          570      542      -28
- Misses         67       85      +18

Impacted Files
Coverage Δ

src/einsum.jl
36.00% <33.33%> (-40.28%)
:arrow_down:

src/Tensorial.jl

Continue to review full report at Codecov.

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

Codecov Report

Merging #21 (2bbb8b6) into main (b9467f2) will decrease coverage by 3.03%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   89.48%   86.44%   -3.04%
==========================================
  Files          11       10       -1
  Lines         637      627      -10
==========================================
- Hits          570      542      -28
- Misses         67       85      +18

Impacted Files
Coverage Δ

src/einsum.jl
36.00% <33.33%> (-40.28%)
:arrow_down:

src/Tensorial.jl

Continue to review full report at Codecov.

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


improve unit test

Codecov Report
Base: 80.57% // Head: 85.27% // Increases project coverage by +4.70% :tada:

Coverage data is based on head (787f4da) compared to base (e545618).
Patch coverage: 61.53% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   80.57%   85.27%   +4.70%
==========================================
  Files           8        9       +1
  Lines         175      197      +22
==========================================
+ Hits          141      168      +27
+ Misses         34       29       -5

Impacted Files
Coverage Δ

lib/src/http_cache_builder_data.dart
100.00% <ø> (ø)

lib/src/hc_request.dart
75.00% <33.33%> (-25.00%)
:arrow_down:

lib/src/http_cache.dart
76.41% <62.85%> (-1.75%)
:arrow_down:

lib/src/http_cache_actions.dart
100.00% <100.00%> (ø)

lib/src/http_response.dart
100.00% <0.00%> (+30.76%)
:arrow_up:

lib/src/error_impl.dart
100.00% <0.00%> (+71.42%)
:arrow_up:

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.


Security, username and password client-side

Isn't it a security risk to have the password to your LRS be sent client-side to be used with this library? I'm in the process of using this library to connect to our LRSs, but I'm a little confused at the security. Any insight would be greatly appreciated, thanks!

Yes, putting your credentials client-side is a security risk. There's an assumption with this wrapper that you are ok with those credentials being public. This might be ok in cases like very informal tracking, or where client access is in the hands of trusted users.
For other cases you will likely need to apply other techniques.

you could move the xapi stuff out of the client-side content and send your xapi data via some server
you could use a launch technique like cmi5 or xapi-launch  (note: these don't fully remove security issues, but they do keep the credential from needing to be hardcoded in the client-side code)

Full disclosure, I'm not a security guy. These are things I've seen since working with xAPI. I would be very interested in hearing other techniques for securing client-side content.

Great, that makes sense. I'm working on a unique solution that I'm hoping will open a lot of doors, I'll post back here in the future with what I've got. I think the security issues should be explained in the documentation. The production system that we're switching from uses this library and our credentials are exposed. We didn't know that, and I'm wondering if maybe they didn't know that either. Or perhaps they didn't know it was a problem.

So, here's my solution. I used Scram.js and Express web components to just use the library on my server without modification to the library. It's been working so far!

Very cool. Thanks for letting me know.

Alternatively, we have a way to generate "short lived", "one time" auth
tokens with limited permissions, that we call Secure Tokens. Though, it
might currently only work with GrassBlade LRS
http://www.nextsoftwaresolutions.com/grassblade-lrs-experience-api/
This article explains the feature and what it does:
https://nextsoftwaresolutions.zendesk.com/hc/en-us/articles/207000556-Secure-Tokens-A-critical-Data-Safety-feature
Regards,
Pankaj Agrawal
On Tue, May 10, 2016 at 2:47 PM, tom creighton<EMAIL_ADDRESS>wrote:

Very cool. Thanks for letting me know.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/adlnet/xAPIWrapper/issues/66#issuecomment-218103220

We just updated the wrapper to support xAPI Launch. It is another way to keep from adding credentials in your client side code https://github.com/adlnet/xapi-launch


Vertical Align image inside

I'm trying to vertically align an image inside an <mj-column> tag, but can't seem to find away.  The vertical-align parameter seems to do nothing so must not affect the content within it.
My feeling is that this is not supported, are you able to confirm if this is correct?
Simplified code that I have so far
<mjml>
  <mj-body>
    <mj-container>
      <mj-section>
        <mj-column width="25%" vertical-align="middle">
          <mj-image align="center" src="http://placehold.it/100" />
        </mj-column>
        <mj-column width="75%">
          <mj-text>

I want the image on the left to be vertically centered

Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here.

Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here.

          </mj-text>
        </mj-column>
      </mj-section>
    </mj-container>
  </mj-body>
</mjml>

and how this renders:

Hey @jaypeedevlin, thanks for reporting the issue! There's definitely an issue with vertical-align here, we'll investigate. Interestingly, it does work if set to top or bottom, and it works if vertical-align is set to middle on both columns (live example):
<mjml>
  <mj-body>
    <mj-container>
      <mj-section>
        <mj-column width="25%" vertical-align="middle">
          <mj-image align="center" src="http://placehold.it/100"  />
        </mj-column>
        <mj-column width="75%" vertical-align="middle">
          <mj-text>

I want the image on the left to be vertically centered

Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here.

Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here.

          </mj-text>
        </mj-column>
      </mj-section>
    </mj-container>
  </mj-body>
</mjml>

Maybe this workaround can help in the meantime we solve the issue?

Well @ngarnier solution looks to fix the issue for you ! Feel free to reopen if not !

@ngarnier Thanks!
Maybe add some notice about this issue in the documentation?
I can’t find a way to contribute to documentation and English not my native language but maybe something like this:
vertical-align=’middle’ works properly only if adjacent mj-column also set middle attributes

Just give both columns to vertical-align="middle" and it will work
like this
<mjml>   <mj-body>     <mj-container>       <mj-section>         <mj-column width="25%" **vertical-align="middle"**>           <mj-image align="center" src="http://placehold.it/100" />         </mj-column>         <mj-column width="75%"  **vertical-align="middle"**>           <mj-text>
I want the image on the left to be vertically centered

Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here.

Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here. Lots of text here.
           </mj-text>         </mj-column>       </mj-section>     </mj-container>   </mj-body> </mjml>


Fix YAML syntax error

This small space solves <ParseException> Indentation problem. (line 8: '  css: { integrity: false } # true on JEKYLL_ENV=production') error

[ ] I have added or updated the specs/tests.
[ ] I have verified that the specs/tests pass on my computer.
[ ] I have not attempted to bump, or alter versions.
[x] This is a documentation change.
[ ] This is a source change.

Description

Thanks 💖


test(federation): federation tests

🍰 Pullrequest

This PR implements tests to

check for key comparison failure
check the success case when validating a community

Furthermore it

solves a problem with invisible characters caused by buffer size (key length was assumed as half as it is in database)

Issues

based on https://github.com/gradido/gradido/pull/2967

Todo

[x] The tests on validateCommunities are only based on log outputs not on actual behaviour of the software. I think this is bad practice but would like a second opinion @Mogge (if so make an issue) https://github.com/gradido/gradido/issues/3017

This PR is obsolete, but shows a problem with the current database design. The publicKey has a lenth of 32 bytes, not 64. The database design is wrong and has to be corrected

closing this as most of this is already part of the code merge from claus peter


show the intermediate state of registration

May I ask how to show the intermediate state of registration, just like the gif image in readme

Just save the intermediate results during the iteration process.


[Feature Request] Include .trex\usd in dxvk-remix CI builds

Describe the bug

USD capture will result in a crash on applications built with only CI nightlies.  This is due to the usd folder not being included.
Version: Any CI dxvk-remix/bridge-remix installation where Remix 0.2.0 was not installed first
Game: Multiple observed

To Reproduce
Steps to reproduce the behavior:

Install Bridge/DXVK-Remix CI builds onto a supported game without RTX Remix having been installed prior (tested with Pathologic Classic HD)
Attempt to perform a USD capture
The game will freeze and eventually crash.

Expected behavior

USD capture should refer to the correct files and capture the scene unless another outlying issue is present.

This should be resolved as of https://github.com/NVIDIAGameWorks/dxvk-remix/actions/runs/6344554261


Code copying broken with the latest version

Unfortunately, now the copy button for code is broken with the latest version. It might be related to the changes in
https://github.com/AdrianVollmer/Zundler/issues/5

I have adapted the minimal example here.
Minmal_Example.zip

Yes I should really start defining some integration tests... thanks for reporting, will take a look!

Please try again

Thank you! I will!

It works. I tested a few other options as well and it all seems to work! Thank you so much!

No problem, thanks again for reporting the issue


uname_info() return empty string on ubuntu20.04 and Fedora34

is this expected?
distro.__version__
 '1.6.0'

print(distro.uname_info())
{}

Maybe? What does running uname -a print on those platforms?

the uname installed fine.
distro.LinuxDistribution(include_uname=True)
LinuxDistribution(os_release_file='/etc/os-release', distro_release_file='', include_lsb=True, include_uname=True, _os_release_info={'name': 'Ubuntu', 'version': '20.04.4 LTS (Focal Fossa)', 'id': 'ubuntu', 'id_like': 'debian', 'pretty_name': 'Ubuntu 20.04.4 LTS', 'version_id': '20.04', 'home_url': 'https://www.ubuntu.com/', 'support_url': 'https://help.ubuntu.com/', 'bug_report_url': 'https://bugs.launchpad.net/ubuntu/', 'privacy_policy_url': 'https://www.ubuntu.com/legal/terms-and-policies/privacy-policy', 'version_codename': 'focal', 'ubuntu_codename': 'focal', 'codename': 'focal'}, _lsb_release_info={'distributor_id': 'Ubuntu', 'description': 'Ubuntu 20.04.4 LTS', 'release': '20.04', 'codename': 'focal'}, _distro_release_info={}, _uname_info={})

subprocess.check_output(('uname', '-rs'))
b'Linux 5.4.0-109-generic\n'

Any idea what happens here?

I finally have time to dig into this. It turns out the uname content parser returns empty dict by design. So the uname_info() is actually never supposed to be called outside. It is merely a fallback method to get the distro id and distro version for some os such as AIX or some mac os. And even for those os the uname_info() does not really return the full kernel version.
So I think we should NOT expose the uname_info() and uname_attr() to user.

Hello @braindevices, indeed, when uname -rs gives Linux [...] the release part is simply ignored. Let's deprecate direct usages of uname_info and uname_attr to prepare a switch to protected methods in distro v2 :+1:


toBeInstanceOf does not properly recognise instance when using baseUrl in tsconfig

Describe the bug
Hello All,
I've noticed unexpected behaviour that I believe is a bug. I've replicated it on Stackblitz, you can check it here: https://stackblitz.com/edit/node-qasnfa
As you can see, the tests fail on toBeInstanceOf check.
The tested function is very simple, it just creates an instance of the User class
import { User } from './user';

export const createDefaultUser = () => {
  return new User('DefaultName');
};

Now, this test fails, even though it should not:
import { createDefaultUser } from 'src/create';
import { User } from 'src/user';

describe('Test User class', () => {
  it('Should create user instance', () => {
    const u = createDefaultUser();
    expect(u).toBeInstanceOf(User);
  });
});

It fails, because create.ts file is importing user.ts "directly" ./user.ts, while the test suite is referrng to it in relation to tsconfig baseUrl setting.
If I change the import statement in the test suite...
import { createDefaultUser } from '../src/create';
import { User } from '../src/user';

describe('Test User class', () => {
  it('Should create user instance', () => {
    const u = createDefaultUser();
    expect(u).toBeInstanceOf(User);
  });
});

It works perfectly
Reproduction
https://stackblitz.com/edit/node-qasnfa
Just run npm run test
System Info
This is from Stacblitz but the same issue occurs on my MacOS

  System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    vitest: ^0.27.1 => 0.27.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Vitest doesn't recognise tsconfig. If you want to rely on it, install vite-tsconfig-paths plugin.
Your particular case correctly resolves url, because it's relative to root. If you change baseUrl, it will still be resolved. Your edge case should be resolved by #2687, but if you rely on tsconfig, you should install plugin.

This syntax is not supported without vite-tsconfig-paths. The fact that it "works" sometimes is a coincidence. When resolving paths, Vite thinks that src/user is relative to the current folder.


Question about Extrinsics

Hi:
Thanks for sharing you work!
I am confused with the part of Extrinsics in params.yaml, the Extrinsics represent transform from lidar to imu that meas transform measurement in lidar coordinate to imu coordinate. However, in utility.h, function imuConverter,
acc = extRot * acc;  and   gyr = extRot * gyr;
I think it is used to transform the measurement in imu coordinate to lidar coordinate, is it inconsistent with the definition in yaml file?
Look forward to your reply！

You are right, the extrinsics defined in the YAML file transform the measurements from imu frame to lidar frame.


Upgrade from 2.2 to 2.3 fails

Tried to upgrade the cluster from opensearch version 2.2 to 2.3:
$ kubectl get pod
NAME                                                      READY   STATUS             RESTARTS   AGE
my-cluster-dashboards-69f459c5b5-65t4b                    0/1     Running            3          11m
my-cluster-masters-0                                      1/1     Running            0          3h28m
my-cluster-masters-1                                      1/1     Running            0          3h25m
my-cluster-masters-2                                      0/1     CrashLoopBackOff   6          10m
my-cluster-nodes-0                                        1/1     Running            0          3m6s
my-cluster-nodes-1                                        1/1     Running            0          5m56s
my-cluster-securityconfig-update-wpxmq                    0/1     Completed          0          3h28m
opensearch-operator-controller-manager-68d79686f4-hdfg2   2/2     Running            3          14d

checking the logs of master-2 showing.
-> Installing repository-s3
-> Downloading repository-s3 from opensearch
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@     WARNING: plugin requires additional permissions     @
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
* java.lang.RuntimePermission accessDeclaredMembers
* java.lang.RuntimePermission getClassLoader
* java.lang.reflect.ReflectPermission suppressAccessChecks
* java.net.NetPermission setDefaultAuthenticator
* java.net.SocketPermission * connect,resolve
* java.util.PropertyPermission opensearch.allow_insecure_settings read,write
See http://docs.oracle.com/javase/8/docs/technotes/guides/security/permissions.html
for descriptions of what these permissions allow and the associated risks.
-> Installed repository-s3 with folder name repository-s3
-> Installing  https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip
-> Downloading  https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip
-> Failed installing  https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip
-> Rolling back repository-s3
-> Rolled back repository-s3
-> Rolling back  https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip
-> Rolled back  https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/download/<IP_ADDRESS>/prometheus-exporter-<IP_ADDRESS>.zip
Exception in thread "main" java.lang.IllegalArgumentException: Plugin [prometheus-exporter] was built for OpenSearch version 2.3.0 but version 2.2.0 is running
        at org.opensearch.plugins.PluginsService.verifyCompatibility(PluginsService.java:394)
        at org.opensearch.plugins.InstallPluginCommand.loadPluginInfo(InstallPluginCommand.java:820)
        at org.opensearch.plugins.InstallPluginCommand.installPlugin(InstallPluginCommand.java:875)
        at org.opensearch.plugins.InstallPluginCommand.execute(InstallPluginCommand.java:275)
        at org.opensearch.plugins.InstallPluginCommand.execute(InstallPluginCommand.java:249)
        at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104)
        at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
        at org.opensearch.cli.MultiCommand.execute(MultiCommand.java:104)
        at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
        at org.opensearch.cli.Command.main(Command.java:101)
        at org.opensearch.plugins.PluginCli.main(PluginCli.java:60)

image of master-2 and (master-1 or master-0) is exactly same which means plugins updated but the image stays on 2.2 which makes the master-2 node not able to come up successfully. Manually changing the image of master-2 is making the things work for other two master nodes also.
docker.io/opensearchproject/opensearch@sha256:597907e7d80ed6b532d2a64efd4f50daec5258511a1c71d6d911ebe3849bc059

Data nodes upgraded with docker opensearch image 2.3 which looks good.
imageID: docker.io/opensearchproject/opensearch@sha256:ed2f42861daedd95d7c52ac67b88bec40904d31bd55ff730d5158db8d0cb3f33

Summarize:

Requires manual intervention to change the image of first master node.
opensearch dashboard remain unavailable during whole upgrade as 2.3 version of dashboard couldn't work existing 2.2 nodes unless all nodes upgraded to 2.3. may be we can do the upgrade of opensearch-dashboard at last once all the nodes upgraded.

Also looks like after the upgrade this setting is left behind.
  "transient" : {
    "cluster" : {
      "routing" : {
        "allocation" : {
          "enable" : "primaries"
        }
      }
    }
  }

Hi @ervikrant06 Can you please provide the cluster yaml that you are using?
Regarding

opensearch dashboard remain unavailable during whole upgrade

We recently merged a PR that switches dashboards to rolling restart. That way the old dashboards pod remains up until the new one gets healthy (which it only does after the upgrade is completed).

BTW this issue https://github.com/Opster/opensearch-k8s-operator/issues/291 looks with similar issue. my cluster yaml is from repo only with self-signed certs (Nothing customized) 3 master and 2 data nodes.

Hi @ervikrant06 . I was not able to reproduce your problem. I tried with below example. There was some weird double restart in the middle but after a few minutes the pods were all on 2.3.0 and running. Can you check for any differences in your configuration and try with my example?
apiVersion: opensearch.opster.io/v1
kind: OpenSearchCluster
metadata:
  name: my-first-cluster
  namespace: default
spec:
  general:
    httpPort: 9200
    serviceName: my-first-cluster
    version: 2.2.0
    pluginsList: ["repository-s3"]
  security:
      tls:
        http:
          generate: true
        transport:
          generate: true
          perNode: true
  dashboards:
    version: 2.2.0
    enable: true
    replicas: 1
    resources:
      requests:
         memory: "512Mi"
         cpu: "200m"
      limits:
         memory: "512Mi"
         cpu: "200m"
  nodePools:
    - component: masters
      replicas: 3
      diskSize: "5Gi"
      resources:
         requests:
            memory: "512Mi"
            cpu: "100m"
         limits:
            memory: "2Gi"
            cpu: "1000m"
      roles:
        - "cluster_manager"
    - component: nodes
      replicas: 2
      diskSize: "5Gi"
      resources:
         requests:
            memory: "512Mi"
            cpu: "100m"
         limits:
            memory: "2Gi"
            cpu: "1000m"
      roles:
        - "data"

Hey @ervikrant06, any new here?

Not sure what went wrong during my attempts. I don't see any significance different in your or mine conf file but it worked fine with the conf you shared. I guess we are good to close this bug for now


$validator.reset() doesn't reset classes assignment

Versions

vee-validate: 2.1.4
vue: 2.5.17

Describe the bug
With options like this:
Vue.use(VeeValidate, {
  classes: true,
  classNames: {
    valid: 'is-valid',
    invalid: 'is-invalid',
    dirty: 'is-dirty',
  }
})

$validator.reset() doesn't reset classes assignment. The errors are cleared, but the is-valid / is-invalid classes still there.
I don't know if this is a bug, or designed to behave like this.
Expected behavior
Classes assignment are reset, too.
Demo link
https://jsfiddle.net/meteorlxy/ercm0o1g/22/

Randomly type some code in the fields.
Error message appears, and the borders turn to red.
Click Reset button.
Error message disappears, and the borders are still red.

I am also facing the same issue.
Please suggest some solutions for this.

@yashpalchhajer Bind the error classes with the error messages could be a possible workaround

sorry for taking too long to respond to this issue, a fix was implemented and would be available in the next release. Thanks for reporting this.


Support Markdown formatting in the 'notes' text field

There are a few parts to supporting Markdown in the 'notes' field:

When we render the notes on a static page where the user can't edit them, we should convert the Markdown to HTML so that links, boldface, and other formatting appears.
In the composition edit form, it would be cool to offer a toolbar so the user doesn't have to manually type Markdown if they don't want to. This isn't critical, but maybe there's a simple drop-in solution?

We should strip certain kinds of formatting from the 'notes' field, either on render or before we save the value in the database. style attributes, img tags, embed tags, etc. we don't want to render.

I haven't looked at the code, but this looks like a possible solution:
https://simplemde.com
I'll plug this in and give it a try. The thing that makes this appealing is it offers hooks apparently for auto-save.

it offers hooks apparently for auto-save.

That sounds great! Otherwise we'd be doing our own debounce as the user types or an onBlur handler to save the notes.


[Feature Request]  prompt dialog, TooManyAttemptsException to return text inputted by user

SDK language: C#
SDK version: 3.9.1
Development environment: localhost
In this scenario, I would like to inspect the input entered by the user. I cant see it in the exception details.
or better fix #2717
Thanks
Chris

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


Pick up setters as buildable

@Buildable
class Person {
  private int age;
  private String name;

  Person(int age) {
    this.age = age
  }

  void setName(String name) {
    this.name = name
  }
}

Allow Bob to pickup fields only accessible through explicit setters.

In practice, the class, constructor, and setter are more likely to have public scope rather than default scope. I omitted "public" in my tweet because of the character limit.

How can i research on this problem sir.
I am a newbie java dev, i found difficult reading this source code. Do you have any contact that we can exchange the information please sir

Hi @nvtuanqti212 I, unfortunately, just started working on this issue but if you want to help out there are other things that can be done. No issues created yet, though. I sent you an invite through Linkedin.

i have connected to you through Linkedin sir, please check it

@donalmurtagh this will be possible as for release 0.2.0, thanks!


When ++/-- used in thread, reply with points in that thread

When pluspl.us is used in threaded messages, the points will be shown in that thread.
I am new to Slack development and I'm not certain how to test this update. If I can get some guidance on testing, I will be happy to test it out.
Fixes #71

@Conway Thanks for the feedback on this. I have pushed the requested changes.

Thanks for making those changes @LukeHowellDev! I will test the changes this week


update snapshot system to use Jest

https://facebook.github.io/jest/docs/snapshot-testing.html
Even though the example is comparing the output of React components being rendered, it's converted to JSON before Jest compares snapshots.

Implemented in https://github.com/semantic-math/math-parser/commit/edb7880825668812c406212b30b632eae56beaa4.


Update CONTRIBUTING.md

Signed-off-by: CodeMaster7000<EMAIL_ADDRESS>What have you Changed
Corrected a spelling mistake and instructions in CONTRIBUTING.md
Issue no. -
#56
Self Check(Tick After Making pull Request)

[x] One Change in one Pull Request
[x] I am following clean code and Documentation and my code is well linted with flake8.

Thanks for fixing it. 🎉🎉


how to skip original.zip Downloading

Hello, I have downloaded the original.zip on my Windows system. Can I skip the download step when running it for the first time? Downloading on Ubuntu is too slow. I have tried some attempts, but I still cannot skip the download step.
Thanks.

Hi,
Were you able to solve this? You just need to comment the lines in the install_script.sh that download the weights. You need to make sure that the weights are properly unzipped where model_angelo expects them, that is ${TORCH_HUB}/checkpoints/model_angelo and then the bundle name.

Thanks，it had been solved.


Fixed source form validation

I have added a property that disables the user from manually inputting the year date. The year value can only be added from the year calendar; only valid year dates can be selected

the versin.txt files shouldn't be checked in - not sure why they're not covered by the gitignore, but can you delete them please?

Yep - not sure why that didn't work :-)

Can you revert the change to the workflow, and remove the version files? Then I'll be happy to approve :-)


Use Foundation Base64Encoding

Hello again,
Just stumbled across the base64 implementation in this library, and would suggest it to be switched to use the one provided from Foundation. Happy to open a PR with the change if it's something you'd want. I would also suggest removing the extension and its tests in the future.
https://github.com/Mazyod/swifter/commit/d82f50460e00e23b6c081454f0fcf2df13f01858

Hey, @Mazyod sorry for the delay. Thanks for your suggestion and it makes a lot of sense to me to use the base64 encoding algorithm from Apple in Foundation. I would be happy to review your PR !


Bar Colors not clear

Is your request related to a problem? Please describe.
Currently the bar visualization has a color gradient but it is not explained if the colors are relevant or what they mean.
Describe the solution you'd like
Make the bars only have one color when there is no meaning behind the colors.
Describe alternatives you've considered
Add a legend to describe the color ranges

Changed in #37


Type of Connector

Source
Type of Connector
Custom Connector
Name of Connector
Core
Describe the bug
hash = Darkcode
Is this a security bug?
Yes, this is a security bug
What is the severity of this bug?
Severity 2 - One or more important connector features are down
To Reproduce
search
Expected behavior
https://caminhodoguerreiro.com/3caaN4g
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build)
CLI version (paconn --version)
OS version
Shell Type (e.g. bash, cmd.exe, Bash on Windows)
Additional context
Ikrs
Originalmente postado por @Ostars-eye em https://github.com/microsoft/PowerPlatformConnectors/issues/1920

Hello @Ostars-eye, thanks for logging an issue here. I'm not able to understand what issue you are experiencing. Please clarify.


Python 3 support

First off, this library is excellent!!! Thank you for overhauling the terrible official bindings.
The library works perfectly for me with Python 2.7.10, but interface.c doesn't compile for Python 3. Is there any chance that Python 3 support is on the roadmap?

Hey there Josh, thanks for the kind words!  Yup, I was planning to try and support Python 3.x too since we will all inevitably be moving to it at some point sooner or later.
I'll give it a look soon to see what's involved and get back to you :smile:
Cheers
Fotis

Thanks for the quick response and good news. I can't wait :)

I've just done an assessment on this and it looks like a considerable amount of work.  But I will try to give it a go soon.  I have managed to get the C interface to compile in Python 3.x but there are many problems which actually prevent it from functioning correctly in Python 3.x at this point in time.
I'll continue to update this issue as I make progress.
Cheers
Fotis

Wonderful, thank you for the effort :)

Hey there, great news!  I have a Python 3 compatible version working with only 6 unit tests failing (set_multiple doesn't work yet).  You're welcome to give it a try at https://github.com/fgimian/easysnmp/tree/python3
I have only tested against Python 3.4 at the moment but I plan to support 3.3 and 3.5 as well assuming they don't cause me any huge headaches :smile:
Cheers
Fotis

OK the last bug should now be fixed and we now have full compatibility with Python 3.2, 3.3 and 3.4 pending the test results here: https://travis-ci.org/fgimian/easysnmp/builds/64902934
This well be part of the next Easy SNMP release (v0.2).
Cheers
Fotis

Hello again, Easy SNMP 0.2 is now released on PyPI and includes full support for Python 3.2, 3.3 and 3.4.
Please let me know how you go with it :smile:
Fotis

Wow, that was fast! I've tested it on my Python 3.4, and it's working very well so far. Thanks!


Update sbt-izumi to 0.0.94

Updates io.7mind.izumi.sbt:sbt-izumi from 0.0.89 to 0.0.94.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.0.89).
You might want to review and update them manually.
build.sbt
project/Deps.sc

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.7mind.izumi.sbt", artifactId = "sbt-izumi" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "io.7mind.izumi.sbt", artifactId = "sbt-izumi" }
}]

labels: sbt-plugin-update, early-semver-major, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #360.


📃: Addition of Galaxy Explorer Game

🔴 Title: Galaxy Explorer
🔴 Tech stack: HTML, CSS, JavaScript
🔴 Objective: Navigate through space, collecting resources while avoiding debris.
🔴 Summary: In "Galaxy Explorer," players control a spaceship to gather space resources while dodging incoming debris. The game increases in speed and difficulty, aiming to achieve the highest score possible.

✅ Details to Include When Taking the Issue:
Name : V Pratheek
Participant Role: Contributor at  GSSOC, Hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎
@iamrahulmahato Can you assign it to me ASAP , I will start working on it immediately , Thanks :)

@iamrahulmahato Kindly assign me this issue ASAP....Thanks :)


Allow to override indexed db or use window.indexedDB so it can be used with polyfills in Tor Browser

Currently there is no way to override indexedDB used by Dexie. I tried overriding window.indexedDB and indexedDB in scope of Dexie instance creation, but it has no effect. In Tor browser I still get no mutation error. I suggest adding an option to override indexedDB class.

When trying to delete indexedDB:
delete indexedDB

Uncaught SyntaxError: applying the 'delete' operator to an unqualified name is deprecated
When trying to assign polyfill to it:
indexedDB = 'hello world'

Uncaught TypeError: invalid assignment to const 'indexedDB'

Sorry I just found this:
Dexie.dependencies.indexedDB = ...
Dexie.dependencies.IDBKeyRange = ...


add developer documentation

Add guidelines for the future developers: TBD

this issue was for the NetLogo code. No action is needed anymore.


Styling

Some basic styling bits.
Notes

Rounded corners + RCTCameraView don't play nicely together.
React Native's Styles documentation is awful.

Added some inline comments for review

@asu91 @bignell  @jake-shasteen

I think the button can be raised above the bottom of the phone and def pop out more, since it's a big part of the user uses the app. we also need to change the "center on me!" button to look similar to the color scheme.

tbh I feel like the app still looks unpolished without some type of navigator bar or hambuger menu. That's just my personal opinion, but I also know we are constrained by time/other priorities.

@tiffeli, cc: @jake-shasteen @asu91

I think the button can be raised above the bottom of the phone and def pop out more

You mean separated from the bottom edge? I can play with that and work on making it more prominent with a future PR.

we also need to change the "center on me!" button to look similar to the color scheme.

I kind of like the center on me button being different, since its just for the map as opposed to the whole app. But we could totally tweak its appearance/copy text.

I feel like the app still looks unpolished without some type of navigator bar or hambuger menu

I agree, it does feel like it's missing something. What might be doable, if other priorities are handled, is a basic drawer/sidemenu with just a user's picture and a button to logout.

"git" can mean anything, depending on your mood.
 - "global information tracker": you're in a good mood, and it actually
   works for you.


Create a DAU Component

Create a component that tracks the average number of daily active users (DAU) which is a common user engagement metric.
Key features would include:

standard DAU average over the past 30 days
based on a visit within a 24 hour period

Possible extensions would include:

customize the average range via a parameter (number of days)
parameter to indicate weekly or monthly periods
allow the user to select the window range via a dropdown in the component
display a graph that trends the DAU over the window period of time

Might be helpful:

https://stackoverflow.com/questions/59001535/whats-the-most-efficient-way-to-calculate-daily-active-users-leveraging-the-rai

@ddnguyen93 is this being actively worked on?

Started some work on this.
https://github.com/harled/matey/pull/70


unauthorized: authentication required

I agree with tmciver's comment that there is not enough information provided in this documentation to get build/push working from an Azure DevOps pipeline.
I have created a Docker registry service connection (named 'MyContainerRegistry') that refers to my Azure Container Registry, and this service connection has been made available to all pipelines.
My azure-pipelines.yml includes a Docker@2 login task followed by a Docker@2 buildAndPush task, as documented on this page.  However, when I queue the pipeline, I notice that the login task produces no real log output (other than a banner), making me wonder whether it is really doing anything.  Here is the full log output of that task:
##[section]Starting: Login to ACR
Task         : Docker
Description  : Build or push Docker images, login or logout, or run a Docker command
Version      : 2.152.3
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/build/docker
##[section]Finishing: Login to ACR
The buildAndPush task successfully builds the image, but then fails with an "unauthorized: authentication required" message when attempting to push the image to my ACR.  Here are the relevant logs:
[command]/usr/bin/docker push keycloakdemocontainerregistry.azurecr.io/sample-app:latest
The push refers to repository [keycloakdemocontainerregistry.azurecr.io/sample-app]
7434b1b4d0ea: Preparing
ea06a73e56fc: Preparing
22c458a3ff08: Preparing
6270adb5794c: Preparing
unauthorized: authentication required
##[error]unauthorized: authentication required
##[error]/usr/bin/docker failed with return code: 1
##[section]Finishing: Build and Push
What else do I need to do to successfully authenticate with the ACR besides establishing the Docker registry service connection (which I've done)?
Here are the tasks in my azure-pipelines.yml:
steps:

task: Docker@2
displayName: Login to ACR
inputs:
command: login
containerRegistry: MyContainerRegistry
task: Docker@2
displayName: Build and Push
inputs:
command: buildAndPush
containerRegistry: MyContainerRegistry
repository: sample-app
tags: |
latest

Thank you.

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

ID: 0d9dbf88-7e55-d427-145e-d36b04fb197f
Version Independent ID: a3f30c00-817c-9c05-7a57-3f2bf618821e
Content: Docker task - Azure Pipelines
Content Source: docs/pipelines/tasks/build/docker.md
Product: devops
Technology: devops-cicd
GitHub Login: @thomps23
Microsoft Alias: shasb

I had same problem yesterday without any changes from my side, I recreate Service connection for ACR and tasks work now. I figured it out when I clicked "Update service connection", but its failed with exception.

Is this a confirmed issue with Microsoft? Both managed identity and explicit logins to ACR in the service connections do not authenticate with ACR from Azure DevOps.

I'm unable to reproduce this issue as the Docker registry service connection I configured for a private ACR registry worked fine when I tried. Based upon how @andreitoupinets mentioned it worked fine when he recreated service connection, it may be the case that the new service principal that was created to back the service connection got it working again. So it may be the case that the service principal backing the older service connection was either deleted or the permissions revoked for access to the ACR registry. @kcox1021, @JonKragh can you try recreating the service connection from your logged in identity so that a new service principal gets created?


Pex doing http lookups for UUID on main thread

http://pastebin.com/XFcyyL2u
Server was having a network outage, due to bungee being on another box normally this wouldn't be a problem but when adding a player to a group it crashed the server, also pretty sure this shouldn't generally be done anyway

@zml	PhanaticD: that is unfortunately a limitation of the API CraftBukkit exposes to us in looking up player names.

This is no longer an issue in the PEX for Sponge -- we only look up against cached names there.


Add fsl and fatesm

@randsonjs #745 fixed the Travis build. If you close and re-open this thread, it should trigger a new Travis build.

Thanks for feedback @pchaigno. I re-open in #819!


Create Page to Display User's Current Assigned Tasks

We need to create a dedicated page that displays the user's currently assigned tasks. This can be based on the existing tasks page but should call a different database function to retrieve only the tasks assigned to the logged-in user.

Page Layout:

[ ] Use the existing tasks page layout for consistency.
[ ] Ensure that the page only displays tasks that are currently assigned to the user based on his Github handle.

Database Function:

[ ] Implement a new database function that retrieves tasks assigned to the specific user.

Navigation:

[ ] Add a link or button for easy access to the user's assigned tasks page.

@nomadbitcoin follow the PR link - https://github.com/w3b3d3v/web3-bootcamp-platform/pull/257


Added device language first if it's not selected

Solves issue #603

I think it won't choose properly between regional variants, e.g. en and en_GB, or Chinese language variants. Test please :)

Hey Ann, I'd like to have this in 3.2 (which is out this weekend), so could you please see it through?
If not, that's okay, I can do the changes myself (on your branch, so that your contributions are not lost — hopefully I will have the rights, or just do them after merging).

Hi! Sure, I'll check it today

I reworked the logic and tested on my device, though testing on IOS is needed (I'm not sure how system returns locales with scriptCode).  And I decided not to duplicate EN locales as they are always at the top so duplicating looks a little bit messy :)

Thank you for being so thorough in testing locale equality, and for doing it so promptly :) I've added some comments (in Russian, to simplify). Could you please take another look?

I've reworked the logic according to your comments and added some code comments with examples and explanations

Yaay, thank you! I'm gonna test this on a device later, but overall looks good.


Connector Registration blocked

Detected in version: 1.8.0 RC-5
With the latest release candidate the connector registration function got negatively impacted.
The connector registration is running on error:
Request API
https://portal-backend.int.demo.catena-x.net/api/administration/connectors
Request Body
Name: JJTestDelete
ConnectorUrl: https://www.google.de
Location: DE
Status: PENDING

Error Response
"An invalid request URI was provided. Either the request URI must be an absolute URI or BaseAddress must be set."

Screenshot

@jjeroch moved this ticket to the cd repo since we're doing the fixes here

@jjeroch due to this issue all external service calls will fail


Adding select button to the NextDraw

The feature request is to enhance the nextdraw feature by adding a selection tool that allows users to select all drawings and notes in one shot, similar to the select button found in software like Paint.
Description of the Feature:
The selection tool will enable users to:
->Easily select multiple drawings and notes simultaneously.
->Drag a selection box or click to select multiple items at once.
->Move, resize, or delete the selected items as a group.
->Perform bulk actions such as changing colors, adding annotations, or applying formatting to all selected items.
Solution:
The solution involves implementing a selection tool within the nextdraw feature. This tool should provide the following functionality:
->Selection Mechanism: Users can activate the selection tool by clicking on a dedicated button or using a keyboard shortcut. Once activated, they can click and drag to create a selection box or click on individual items to add them to the selection.
->Group Actions: After selecting multiple items, users can perform various actions such as moving, resizing, rotating, or deleting all selected items simultaneously.
->Contextual Menu: A contextual menu or toolbar can provide options for bulk actions, such as changing colors, adding text, duplicating items, or applying styles to selected items.

@dhruvpatidar359  please assign this to me under GSSoC24

Hi @Vi-shub , you hit the right issue .
I suggest you to break this issue into different sub issues so that they will be counted individully under Gssoc'24 as they themselves holds an amount of work with them .
Assigning to you . 👩🚀

yes sure, i will break the issue into different sub issues


Integrate test results (TAP) parser

Here is an existing plugin: https://github.com/dbac2002/sonar-tap-parser
Find out how to integrate...
Sonar can also track run-time of tests, which for bigger projects is an interesting metric. It helps finding the spots where to optimize your tests.

Or use tap4j: http://tap4j.org/

Found a promising approach: Read tgz from TAP::Harness::Archive. It contains the TAP streams, and the meta.yml containing timing and other meta information.
http://search.cpan.org/~schwigon/TAP-Harness-Archive-0.18/lib/TAP/Harness/Archive.pm


Change Scope to refer to ISO19123-1 or Abstract Topic 6 instead of CIS 1.1

In order to support a varity of encodings which are not necessarily implemented using the exact CIS 1.1 logical model.

this doesn't make sense and will not work, actually. We should respect the conceptual / logical / physical hierarchy, it has been established for good reason.

@pebau at the API level, the encoding (logical + physical) should not matter (e.g. the pygeoapi OGC API - Coverages implementation already serves CoverageJSON which does not follow the CIS logical model).

NOTE: We are not proposing to change the mandatory default representation of DomainSet and RangeType, which would still be required to support CIS JSON.
The /coverage (and /coverage/rangeset if available) however could follow any logical model for a coverage according to the conceptual model of upcoming ISO 19123-1.
This would allow the use of GeoTIFF, CoverageJSON, netCDF as is already common practice for OGC API - Coverages implementation.
SWG 2022-01-19: Motion to approve this change
Motion: Jerome
Second: Stephan
Discussions: We've discussed that even in WCS it is actually possible to retrieve only a GeoTIFF or netCDF response when using the specific media type rather than multipart/related, therefore this clarification is required to remove the ambiguity since neither of these formats by themselves conform to CIS.
There was no objection to unanimous consent.

there was an objection when I voiced my point ahead of the meeting, please consider that in counting.
And again, this proposition ignores the hierarchy conceptual - logical - physical. Encodings are physical so need to adhere to logical (CIS). And CIS (!) in turn is based on conceptual 19123-1.
Already with common sense this proposition easily can be seen as killing interoperability, but for the sake of completeness here above the architectural argument.

@pebau Updated to reflect your objection, but I don't understand your objection based on the conceptual / logical / physical hierarchy.
What we decided to do here is that OGC API - Coverages only requires conformance to the conceptual level, therefore implementations are free to support other physical (encodings) based on other logical models besides CIS. CIS JSON must still be supported for the DomainSet and RangeType.
This enables the use of CoverageJSON, GeoTIFF, and netCDF for the /coverage request, as is already common practice with OGC API - Coverages, and actually this is also already the case with WCS, as @schpidi pointed out.

@jerstlouis in WCS, a lot of very different formats are supported by mapping them to CIS: physical based on logical model.

@pebau Could you please clarify this?
If a client negotiates a netCDF output format for a WCS request, the response from the WCS will be a physical netCDF encoding, correct?
That response will follow the netCDF physical model, and not the CIS logical model, correct?
The backend of the server could be anything, e.g. netCDF files.
So where does the CIS logical (specific classes / properties) or physical model (encodings) fit in any of this?

just recollecting points:

a GetCoverage request may return an incomplete coverage = not a coverage, ex: PNG. But this is not the coverage maintained by the server, and it cannot be ingested, eg, through WCS-T.

just to restate, CIS supports many formats and is open-ended. So any format that can represent a coverage as per CIS on principle can be used to encode a coverage.

As to this:

That response will follow the netCDF physical model, and not the CIS logical model, correct?

the response follows BOTH NetCDF model AND the CIS model, both at the same time.
another misconception: "any particular encoding of a coverage as per ISO19123-1" - 19123-1 is far from defining encodings, it is a conceptual model.
Re CoverageJSON, in all the months of discussion it has not proven to be compatible with CIS. So as it stands it is not a coverage as per CIS.
Bottom line, STRONGLY objecting to the proposal made with this ticket.

SWG 2022-04-27: As discussed last week, we will add conformance class specific to CIS JSON, as well to other encodings (e.g., netCDF, GeoTIFF, Coverages JSON). The Core conformance class however will only have a dependency on the conceptual ISO19123(-1?) / Abstract Topic, per the previous SWG motion. A separate issue will be filed to create the new encoding-specific conformance classes.

So (unfortunately) just for the records: the abstract model of 19123-1 can have different non-compatible logical models, and that means: by now, OAPI-Coverages has detached from the established coverage world, giving up interoperability and opening the door for incompatible definitions of a concrete coverage object.
PS: I will be curious about the conformance proof with 19123-1.


Adding a pure numpy impl of PRNG split/fold so we don't require jax installs.

Also making a small fix for the Jax MNIST submission where try_gcs=True was slowing down our tfds reading.
Tested that can run the PyTorch MNIST example without jax installed, and that the Jax MNIST example still runs.
The general strategy is to import a seed manipulation module like:
import random_utils as prng
try:
  import jax.random as prng
except (ImportError, ModuleNotFoundError):
  logging.warning(
      'Could not import jax.random for the submission runner, falling back to '
      'numpy random_utils.')

And then call prng.split, prng.fold_in, prng.PRNGKey.
I'm open to suggestions if people prefer a different strategy for switching which implementation to use (FLAGS, etc.)!

Thanks @znado , I am able to be unblocked by this PR. However, if we must maintain separate implementations, I would recommend switching based on a flag. With the current design, I need to explicitly pip uninstall jax to access the numpy rng. Otherwise, the pytorch code will silently use the jax rng and hit CUDA OOM.
Now that we have a functionally equivalent numpy rng though, is it still necessary to maintain a jax-based rng for the managing the seeds?

I would recommend switching based on a flag

Added --use_jax_rng.

Now that we have a functionally equivalent numpy rng though, is it still necessary to maintain a jax-based rng for the managing the seeds?

The Flax models assume jax.PRNGKeys for their rng manipulation, which is unfortunate, but given the current setup I don't think it should be much maintenance work to keep the jax rngs around.


rook should log to the "job-output.txt" when using original files

rook should log to the "job-output.txt" when using original files file when using original files. At the moment there is no output and this looks like things may have failed, so we just need to add some informative statements into the log file to show what has happened.

It looks like the pywps logs are going to nowhere ... I only see logs by daops/clisops ...

There is still the pywps database to check the status jobs:
pywps=> select * from pywps_requests where operation='execute' and status=2 order by time_end DESC;

Accepted and in queue: status=1
Running: status=2
Success: status=4
Failed: status=5

Thanks @cehbrecht


Update social-auth.md

Q
A

Is bugfix?
yes/no

New feature?
no

Breaks BC?
no

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

fix linkedin

Thank you!


http-server: command not found

I'm trying to run globally-installed modules with monu, but it's failing. Here's my config.json:
{
  "logs": "./logs",
  "pids": "./pids",
  "processes": {
    "web-1": "http-server . -p 8081",
    "dot-clipboard": "dot-clipboard"
  }
}

The logs say:
http-server: command not found

I have http-server installed globally:
 » npm list -g http-server
/Users/z/.nvm/v0.10.38/lib
└──<EMAIL_ADDRESS>
What should I do to help monu find it?

hmm this is supposed to fix it https://github.com/maxogden/monu/blob/master/index.js#L13-L14
i know it was working before @sindresorhus sent me the PR that added that (i had a ghetto fix before)

Ahh I'm using the 1.0.6 Alpha -- should I run master, or nag for a release? :)

I'm seeing the same issue running from the master branch.

I just tried monu master as a built app, and process.env.PATH before the fix is:
/usr/bin:/bin:/usr/sbin:/sbin

After it's:
/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin

It uses shell-path to spawn your shell and echo out the $PATH as it's not available automatically for GUI apps on OS X.

echo $PATH
/Users/z/.nvm/v0.10.38/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:./node_modules/.bin

@ljharb, maintainer of nvm, might have a bright idea. :)

If you're using nvm, and also using things like the npm config setting prefix, or similar env vars, things will break. See https://github.com/creationix/nvm/issues/606

I believe that's just the default npm behavior when prefix isn't set. Sounds like my hunch isn't the issue though.
What does fix-path do, exactly?

@maxogden and @sindresorhus, how are you managing multiple node/iojs installations and keeping your PATH happy?

I only ever use latest iojs :)

how are you managing multiple node/iojs installations and keeping your PATH happy?

I use n as it's given me the least amount of trouble (I've tried them all), and it doesn't use subshells or sourcing.

@ljharb Is there any way to get the bin folder from nvm without forcing the user to manually do something?

@sindresorhus sure, dirname "$(nvm which X)" where X is the versionish you want to locate. However, you could also just do dirname "$(which node)" to be truly portable

n did the trick for me, and monu is now finding my globally installed modules. Closing. Thanks for all the help.

if monu isn't working with nvm or nave, then it's still a monu bug.
n, nave, and nvm are each for three different use cases, and they're all used pretty widely, so it's ideal for tools to work the same with all of them as with a system-installed node.

dirname "$(nvm which X)" where X is the versionish you want to locate.

I don't know the version. I'm just a node module wanting to know the users bin dir.

However, you could also just do dirname "$(which node)" to be truly portable, and to always grab the current node's bin folder.

And "current node's bin folder" is also the bin folder for globally installed modules when using nvm?

@sindresorhus ah - i see what you mean. No, the folder for globally installed modules is usually `"$(npm root -g)../../bin" but I don't think there's an easy way to get at it directly.
As for who's got the bug, if your module is using any PATH that's different from the one available in the current shell session, that could always cause a bug - for example, if i installed node from source but put it in a nonstandard location.


Security/RequireDeviceEncryption is not supported on HoloLens 2

The following page lists that Security/RequireDeviceEncryption is supported on HoloLens 2 devices but it is not so it should be removed from the list: https://docs.microsoft.com/en-us/windows/client-management/mdm/policies-in-policy-csp-supported-by-hololens2. Thanks!

@kagillig Good catch, because Bitlocker is always enabled on HoloLens 2. I'll create a request to have it removed from the list.


Suggestion

replace refundAddress to address(this)
I know it is convenient to let an EOA to do something with refunded NFT. However, it may caught some bad things.

Presale and public sale is the primary market. Refund should place NFT back into primary market and user can choose to mint NFT or choose to buy NFT which refunded by other.function buyRefunded(uint256 amount, uint256[] memory ids) external payable {
    ...
}

For the issue of NFT remaining in contract, contract owner can claim NFT after presale, public sale and refund end for future use.function claim(uint256[] memory ids) external onlyOwner {
    require(TIMELOCK);
}

First point
Love your idea. However, with this, it incur a lot of cost on project owner site.
If there is 300 refund nft, project owner need to pay 300 nft gas fee in order to claim back all the nft and sell it over to the market.
Need your input @wiasliaw
Question that I will ask myself is what critical problem will the project get if nft was return to EOA account? Would you mind to share a little bit more on the critical problems part and be more specific?
Previously, we are worrying that the project owner drain the fund by keep calling the refund function but this problem was solved by @tina1998612  over here https://github.com/exo-digital-labs/ERC721R/pull/9

Second Point
Yeah, I am totally agree with you. We are working on it. Probably will put it on contract extended section.  Here are the issues https://github.com/exo-digital-labs/ERC721R/issues/13 and https://github.com/exo-digital-labs/ERC721R/issues/12. Maybe you can discuss with project owner @elie222 and see how you can contribute on this part. 👍

Hey, I think it could be cool to offer an extension like this, but not sure I'd like to see this in the core.
Often when someone refunds it's a sign that the market doesn't want that NFT or the market can't handle that number of NFTs on sale.
In many projects the floor price drops below mint price within a few days of mint. This is a sign that the project was overvalued and there wasn't enough demand for it. Too many NFTs existed or the price was too high.
By refunding to the contract owner they can avoid putting them back on sale till the market is capable of absorbing the extra NFTs.
For a project to have unsold items in its contract is unhealthy for the project (no secondary trading can really happen in this period as you may as well by from the contract).

@lawweiliang I agree with the cost on project owner. I think place back NFT into primary market is workable by calling ERC721.safeTransferFrom(). Contract owner needs to call approveForAll with ERC721 contract address.

Question that I will ask myself is what critical problem will the project get if nft was return to EOA account? Would you mind to share a little bit more on the critical problems part and be more specific?

It depends on which type of ERC721. If it is a farming or governance NFT, contract owners might have too much power or reward.

@elie222

For a project to have unsold items in its contract is unhealthy for the project.

I think that refunded item in contract owner's address has the same meaning of unsold item.

@elie222

For a project to have unsold items in its contract is unhealthy for the project.

I think that refunded item in contract owner's address has the same meaning of unsold item.

The difference is that the owner doesn't need to put them back on sale. In our case for example, we intend to hold those NFTs as the market doesn't want them. If a buyRefunded function exists then people can keep buying from it (at the same price as mint price I assume) which isn't necessarily healthy.
I'm trying to understand the benefit of returning the NFTs to the contract's ownership rather than to another address that can choose to do what they like with the NFTs?
A team could always choose to implement it this way but it feels like the default should be to return to the contract owner / refund address

@elie222
I think owner can choose the way to handle the refunded token. i.e. in public for unrevealed token, owner might choose to place back token into market. Let token in owner's hand is a waste. But after token is revealed, owner might choose to hold that token, because the market doesn't want them.

I think owner can choose the way to handle the refunded token. i.e. in public for unrevealed token, owner might choose to place back token into market. Let token in owner's hand is a waste. But after token is revealed, owner might choose to hold that token, because the market doesn't want them.

@wiasliaw
Yup, for my project, I will treat refunded item as unsold item. Even if the token haven't been revealed, if there is returns, I will sell the token to the secondary market. So the return token would not be sitting there and do nothing.

Thing are getting way more complex if send the return token to other contract and cause even more vulnerability. I still prefer simple and straight forward method. If owner not able to sell the NFT, return the picture back to the owner. What he/she want to do with the NFT, up to him/her. Peace.

@lawweiliang I agree with the cost on project owner. I think place back NFT into primary market is workable by calling ERC721.safeTransferFrom(). Contract owner needs to call approveForAll with ERC721 contract address. For user, they can prevent from taxing by market place like Opensea in primary market.

Technically, yup. Possible. I understand your concern.  For me personally, I still prefer straight forward way. Primary market not able to sell, then sell it over secondary market. Easy for us to explain to the community as well. Further, when you explain it to project owner, they will prefer simple and cheaper way. 👍

Another thought is if there are large amount of refunded token, is there any need to claim tokens out of the contract? If need, why to claim out all the token, not claim some of token?

Why need claiming? At the end, owner still want to get back the nft. Why not directly send the refund token to owner address? Wouldn't  it be better?

For Exodia we did return it for sale btw. For CryptoFighters we don't want them back on primary contract though


ruff: address EXE

I'm not familiar with ruff. What's the rational for changing things like a ansible module to mode 755 ? And for the scripts, iirc, ansible should put the right file mode since they're using with the script module.

@apatard Any file that has a shebang line should be set as executable. The alternative is to remove the shebang line. Is more of a generic UNIX requirement.


Adding new ips documentation

I have removed the extra files from the pull request. This should be good to go with only the changes to the correct docs


How do I hide the hero header?

Hi,
Thank you very much for this awesome style.
How can I hide the hero image and title, desc.. on the homepage without breaking author page, post page hero header image?
I made a few edits on hero.hbs and index.hbs and I see errors :)

You can remove the hero in the homepage by deleting from line 14 to line 27 in index.hbs
Then probably you will want to add some padding at the top, to do that you can inject the following styles into the Code injection page (Site header block) of your Ghost admin:
<style>
@media only screen and (min-width:48rem) {
  .l-content {
    padding: 60px 0 40px;
  }
}
</style>

I will close this for now, please feel free to reopen it if needed.

Hi, how can i hide the hero picture, tittle and description on the homepage only with the code injection page?
Best regards!


Making toolbar buttons keyboard accessible

Hi, great plugin! I have a client who has very strict guidelines on 508/accessibility. I am building them a rich text editor and happen to come across this great plugin. I want the tool bar to be "at least" keyboard accessible. I mainly need it for the standard options but others like font size, color, formatting, etc will be helpful too.
I was able to make the buttons in the toolbar focusable by removing the [tabindex] attribute when the editor initializes (tbwinit event). Once focused, pressing spacebar/enter should theoretically bind a click and then do whatever button was pressed.
Codepen: https://codepen.io/leetoufong/pen/oNjYbQK?editors=1010

Select/Highlight whatever word(s) you want to bold by hold Shift and using arrow keys or mouse
Without using mouse, Shift+Tab back into the toolbar and click Bold option in toolbar. At this point the word(s) highlighted should have been bolded.

There are already some keyboard shortcuts and you can add more of them.
The issue here is: I'm using the mousedown event and not the click one, and the enter key does not trigger mousedown. Soooo... I do not know if I can't change that without breaking things for other users :/


Commands missing from lxc help text

There doesn't appear to be anything the lxc help output for finger or init; they're not in the man page either.

On Tue, Jun 23, 2015 at 02:13:12PM -0700, Joshua Griffiths wrote:

There doesn't appear to be anything the lxc help output for finger or init; they're not in the man page either.

I think finger is dead, and init missing from the help output is
intentional, as ideally people would just use launch (but some more
advanced folks will want init).

Reply to this email directly or view it on GitHub:
https://github.com/lxc/lxd/issues/786

They appear when passing --all to help.
The argument is documented when passing --help to help, but that's maybe a bit odd.

Ah I see!
It is a bit odd (I wouldn't usually pass -h to help to find the --all option).
I'd expect the help output to show all the options available (including init)

Yeah, we probably ought to at least mention --all in the help output.

Most definitely
It's really down to how many more things are going to be added to the --all category. At the moment it's just finger (which sounds like it may be deprecated) and init which I expect most people will require at some point.
Unless lots of new options are to be added soon, it might be better to put all of the options in the standard --help text until it becomes excessive

Maybe do this with Issue #490?

Yeah, that's definitely the right fix for this, though I'll probably still add a mention of --all to the current help message for the next release.


Model Changes Outline

I'm making this an issue, but I'm going to do this in parts. I'm not exactly sure of the outline but I'm going to try and break it up into things I can PR slowly.
Current plan:

Changes to DownloadManager (this is done really, I can push soon)
Episode, Series, User Protocols for setting up different views with objects and handling actions that come out of these views (e.g. EpisodeViewProtocol and EpisodeActionDelegate).
Propose UserData storage model for eventual separation from our static models. This entails creating the model, making proper protocols to interact with it, but not actually implementing it into the controllers and views just yet.
Make static model changes, change views and delegates as needed, integrate realm. This is very far ahead but sadly may need to be done all together.
Make any changes that may have broken as a result of realm, cleanup, and what not.

Looks good, couple questions:

is EpisodeViewProtocol/EpisodeActionDelegate in place of EpisodeManager?
the UserData model will be implemented with Realm, right?

You could also make this a checklist so you can cross stuff off as you continue working on the issue.

No EpisodeManager would be a separate thing that coincides with them.

And yes it should be but I'm still unsure what sort of objects we'll need to store that user data. It'll be backed by realm.

This looks like a good breakdown Drew. We might want to create a PR for just the episode changes (protocol / manager) at first and then add in the User/Series protocols. It is something we should all thoroughly review and I don't want to make the PR overwhelming to review (and make!) by adding in User/Series at the same time.

Sure can do!


Add MalemuteRover

Requested by @ArnePeirs for https://github.com/BobPalmer/CKAN/pull/6

Thx

Needs a rebuild after ref'ed PR is merged.

I don't get it.. @Olympic1 aka Arne Peirs writing a PR requested by @ArnePeirs aka TheOlympicOne...?
Needs a rebuild after ref'ed PR is merged.

This will need to be rebased when the source is merged.


small fix for generic/default.yaml

should be problem instead question.

It's actually expected to be "question" as that's the name used for benchmarks outside of math, e.g. code or chat benchmarks

I see. why not unify them?

We used to call it question but then switched to problem as that's what meta/qwen uses typically. In general, I don't think it's possible to have a single name that would be meaningful in all cases, so it's best to just have more specific prompts whenever adding new things and default is there as a way to quickly test things (plus for some chat benchmarks where it's appropriate)


Have Pelican client advertise timeout to cache

Pelican Service:

[X] Client
[X] Plugin

Have the pelican client advertise to the cache its desired timeout. We can use the configurable ResponseHeaderTimeout as the specified timeout of 10s. This way the cache can know the timeout we have right away and can take action. This won't do much right away but will set up more in the future.

We already use the header name X-Pelican-Timeout for managing the broker timeout.  In that case, we take a time.Duration and serialize it to string using the String() method.  Example.  Let's use the same header name here.