Upgrade to kotlin 1.8

Also upgrade kotlinx libraries. NB: kotlinx-coroutines is a beta version, probably better to wait for an official release.
Build time is divided by two! 🎉

TODO: fix Linux build:

Task :compileKotlinLinux FAILED
Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.
e: warnings found and -Werror specified

Rebased on master.


What is the max_seq_len in Mistral?

What is the max_seq_len (or max_position_embeddings) of Mistral-7B-v0.1 when training?
The official code says it is 128_000. (https://github.com/mistralai/mistral-src/blob/147c4e68279b90eb61b19bdea44e16f5539d5a5d/mistral/model.py#L201C69-L201C69)
The config file in huggingface says it is 32768. (https://huggingface.co/mistralai/Mistral-7B-v0.1/blob/main/config.json).
And the official blog mentions 16k.

Also, why is the precompute_freqs_cis function being called with a hardcoded end=128_000?


Use GitHub for marketplace issue tracking

It would be great if issue tracking for marketplace issues was in public GitHub repository instead of sending an email.

We do use GitHub, but it's a private repository since the Marketplace is not open-source and serves more products other than VS Code. I will create an issue on that repository asking the team to consider having a public facade for their issues. But for now this will be closed since it's not a VS Code issue per se.
cc @prashantvc


[Feat]: Add Code Template for the user

Is your feature request related to a problem? Please describe.
We always want to do fast coding!!
Describe the solution you'd like
Code Snippets can be saved for a particular user(This can be done only after the login/signup feature is added which is in the issue bucket list)
Motivation
When the user login and creates a new room(not for joining room as some other code may get affected) this code snippet for only one particular language (ex: c++) will be displayed in the editor.
Are you willing to contribute to this issue? [Yes/No]
Yes
Describe alternatives you've considered
A button to add a snippet for a particular user in the login room(assuming after the user logins he will get a chance to create or join the room) which will allow the user to add a particular code snippet for anyone(latest remains) particular language and it will automatically be added when a user creates a room.

Hi @jainaayush01, First of all, thank you so much for taking out the time to create this issue.
Yes, it's a cool idea 💡 , but for now, let's wait for issue #18 to get resolve so that we can look over this part, and we can discuss the user flow, that after login where a user should be redirected to either room or home or user profile(if created).
so even we will think about it, and you can also think on it, what can we do 🙌

Yes

I would like to work on this, will you assign me?


Support Draw.io's XML based library file structure

The Draw.io app supports importing and exporting from an XML file format. This should be supported.
Also, if the base libraries that ship with the Draw.io app are saved in this format then this could supplant the TOML defs for the built in libraries.

Hi, just started using drawpyo and really like what I've seen so far. Great work!
The feature you suggest here would be really helpful since I've already got several custom libraries from before that I would like to continue using with drawpyo. Is there any manual workaround that you are aware to achieve the same?

Hey @tmjo, thanks! I won't get a chance to really look into this for a few weeks but for an interim solution you can either recreate your libraries as a TOML (docs at the bottom of this page) or style your objects by copying the style string in Draw.io (halfway down this page under 'Generating from a style string'). To make this easier you can also pass in the template_object arg to the init function when you create a new object and it will copy the styling.
Neither might be ideal right now but I'm traveling so I won't be able to do any meaningful development on parsing the XML libraries. But I don't see any blockers to implementing that feature, it'll just take a bit of time.


Update vector-search.md

Fixed the problem where all the links to the How to pages for each language were to the Python page.

@jcodella

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


IsAuthenticated always false

I am using keycloak for user management, after configuring the oidcSetting, i tried logging in but the auth.isAuthenticated is always false even after i was successfully redirected by keycloak and the keycloak session is active when i check it in keycloak dashboard.
here is my configuration:
 const oidcOptions = {
    redirect_uri: "localhost:5173",
    authority: `https://auth.mycompany.com/auth/realms/dev`,
    client_id: "test-client",
  };

and this is how i am logging in to it
       auth.signinSilent({
            redirect_uri: "http://localhost:5173/",
          });

the localhost redirect is properly configured in keycloak so it doesnt give any errors. Also auth.error is undefined so its not giving any errors as well

I'm facing the same issue but only when i run my application at AWS CloudFront with context like "https://mydomain.com.br/mycontext". I did not find out why, but what i found is when i access the application in CloudFront, after authenticate on keycloak my aplication does not save token and other informations at sessionStorage and i tried with basepath "/" and it worked fine.

You will need to debug/log you application. You can enable logging like described here: https://authts.github.io/oidc-client-ts/#md:logging

After some debuging i found out what's happening in my case. For some reason when i try to access and endpoint like this /context?session=value cloudfront redirect to /context/ without the auth params so the lib can't confirm the authentication. So i just added an "/" at the final redirect url like https://domain.com/context/ and it worked


Initial callback/later callback discrepancy

The setting of start and end dates made things haywire. This block needs modification:
https://github.com/AMP-SCZ/eeg-qc-dash/blob/254c4e5127155f997456e08c251f6ed27bdb1819/app.py#L198-L206

Plan B: have the user click Filter to render the initial set of EEGs.

Fixed by https://github.com/AMP-SCZ/eeg-qc-dash/commit/b942d4a98925e6bb35fbdcfb6409349522079c8a


Formatting fixes

Some whitespace crept into a couple of files, reformatted those files with black

this branch is borked, ignore


remove API Placement PlacementDecision ClusterSet ClusterSetBinding API version v1alpha1

Ref: https://github.com/open-cluster-management-io/community/issues/145
Signed-off-by: haoqing0110<EMAIL_ADDRESS>

/assign @qiujian16 @elgnay

@qiujian16 @elgnay I've fixed all the comments, plz help review again when you are available, thank you.

/approve

/lgtm


Undesirable serialization of conditionless queries when using object initializer syntax

When using the object initializer syntax, any property that accepts IEnumerable<QueryContainer> (for example, BoolQuery.Must) is serialized with a null value per conditionless query.
Excerpt of an example query:
var request = new SearchRequest
{
    Query = new BoolQuery
    {
        Must = new List<QueryContainer>
        {
            new QueryStringQuery
            {
                Query = someNotNullNotEmptyString
            },
            new QueryStringQuery
            {
                Query = someNullString
            },
            new QueryStringQuery
            {
                Query = someEmptyString
            },
        }
    }
};

Excerpt of the example response:
{"must":[{"query_string":{"query":"some search terms"}},null,null]}
Expected:
{"must":[{"query_string":{"query":"some search terms"}}]}

See: https://github.com/elastic/elasticsearch-net/blob/2.0.1/src/Nest/QueryDsl/Abstractions/Query/QueryBase.cs#L72

Thanks for reporting @tuespetre :+1:

Just pushed a commit for this so that when serializing a collection of queries, a query will only be serialized if it's not null, strict or not conditionless.


build: adopt spec-0

Adopt

@adrn, good for v1? It is a very new Astropy, but that's what Spec-0 suggests.


ProcessOrder method must be called from Handle methods

Missing part in exercise 03. ProcessOrder method must be called from Handle methods.

@ramonsmits I merged it into exercise-2-new-design that is where I'll branch off for to apply changes for exercise 3


Landing page improvement - rendering text above related cards

What is it and who's it for?
Editors should be able to add regular body text to a landing page so that it appears above the related cards
Currently, only the featured text appears at top, with the regular body text block appearing below the cards. In Prismic the order is how it should appear on page.
https://preview.wellcomecollection.org/pages/Wuw2MSIAACtd3Ssg

My bad, this is a non-problem.
When setting a Landing page style, the correct implementation is to relate each 'child' page to the Landing page using the Content relationships tab on each individual content page.
This will enable:

additional body text to appear between the Featured text and cards
Content pages become 'part of' the parent Landing page.


Can't connect to Exchange Online O365 via Powershell on MacOS

Hi guys,
I've already followed the procedures in all the other threads regarding the same question but none worked for me. I hope somebody can help with my issue:
PS /Users/user> $Session = New-PSSession -ConfigurationName Microsoft.Exchange -ConnectionUri https://outlook.office365.com/powershell-liveid/ -Credential $UserCredential -Authentication Basic -AllowRedirection
New-PSSession: This parameter set requires WSMan, and no supported WSMan client library was found. WSMan is either not installed or unavailable for this system.
PS /Users/user> $PSVersionTable
Name                           Value

PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Darwin 17.7.0 Darwin Kernel Version 17.7.0: Tue…
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
MacOS High Sierra 10.13.6
Brew Installed, OpenSSL Installed. Please help, I'm in severe troubles!

WSMan is not supported on MacOs.
Your feedback should be directed to Exchange server team.


Error checking needed

Hi,
Thanks for creating this script. Keep up the good work.
On first pass, it looks like there needs to be some error checks. See this example:
$ doas ./edgewalker.sh
What is the FQDN hostname to use? opti3020.in.example.net
Secret: 3rd2U4kPM5U8YubK
setting up sysctl.conf
Setting up PF
Primary net interface re0 <IP_ADDRESS>
fe80::7a45:c4ff:fe07:6621%re0
Restarting PF
/etc/pf.conf:9: syntax error
/etc/pf.conf:11: syntax error
/etc/pf.conf:15: syntax error
/etc/pf.conf:20: syntax error
/etc/pf.conf:22: syntax error
/etc/pf.conf:24: syntax error
pfctl: Syntax error in config file: pf rules not loaded
rm: lets-encrypt-x3-cross-signed.pem*: No such file or directory
./edgewalker.sh[309]: wget: not found
cp: lets-encrypt-x3-cross-signed.pem: No such file or directory
Setting up httpd
httpd(ok)
httpd(ok)
Setting up OpenIKEd
Getting LE certificates
Generating private keys
vpn

Renewing certificates
Renewing certificates
vpn
Parsing account key...
Traceback (most recent call last):
  File "acme_tiny.py", line 197, in <module>
    main(sys.argv[1:])
  File "acme_tiny.py", line 193, in main
    signed_crt = get_crt(args.account_key, args.csr, args.acme_dir, log=LOGGER, CA=args.ca, disable_check=args
.disable_check, directory_url=args.directory_url, contact=args.contact)
  File "acme_tiny.py", line 75, in get_crt
    out = _cmd(["openssl", "rsa", "-in", account_key, "-noout", "-text"], err_msg="OpenSSL Error")
  File "acme_tiny.py", line 28, in _cmd
    raise IOError("{0}\n{1}".format(err_msg, err))
OSError: OpenSSL Error
b'unable to load Private Key\n9310926458816:error:09FFF06C:PEM routines:CRYPTO_internal:no start line:/usr/src
/lib/libcrypto/pem/pem_lib.c:694:Expecting: ANY PRIVATE KEY\n'
Applying sysctl settings
net.inet.ip.forwarding: 1 -> 1
net.inet.ip.redirect: 0 -> 0
net.inet.ipcomp.enable: 1 -> 1
net.inet.ah.enable: 1 -> 1
net.inet.esp.enable: 1 -> 1
net.inet.esp.udpencap: 1 -> 1
Starting OpenIKEd
iked(failed)
Enabling SSL on HTTPd
httpd(ok)
httpd(ok)
quirks-3.494 signed on 2020-12-01T18:07:31Z
iOS/iPadOS/macOS VPN config QR code

So there's a few things:

pf rejects the rules
wget isn't installed
There's some python issue with acme_tiny.py
iked failed to start

Note: I didn't edit X509 or USERNAME. Would this have caused the failures above, though?

Is there some reason you're not using rcctl to enable, start, stop services?

You're using pkg_add already:
pkg_add libqrencode
You could move this at the top (or near) and also include wget

Thank you for reporting this. I have been using Vultr VMs for the last testing, mostly because I don't have to worry about NAT traversal for the Let's Encrypt negotiation, and it didn't occur to me that their images would have extra stuff a vanilla OpenBSD install wouldn't have, like wget or python3.
Regarding the PF error, it's because you have both IPv4 and IPv6 addresses and I didn't test the AWK script that gets the main IP on a system that has both.
The OpenSSL error suggests you did not use a valid Let's Encrypt account key, or more likely you used one in the JWK format used by Certbot rather than the PEM format used by Acme-Tiny and OpenSSL. Can you confirm? A JWK key would start with a {, a PEM key would start with -----BEGIN RSA PRIVATE KEY-----.
In any case, I pushed some fixes, can you see if they help?

Hi,
Sorry for the late reply.
I like the changes you did. Perhaps one small critique I could make would be to print out a message of what packages are being installed. Maybe something like "checking to see if wget is installed" and/or "installing wget, python3"

The OpenSSL error suggests you did not use a valid Let's Encrypt account key

That's correct. I think I just typed some random characters in and pressed ^D. So that looks like a failure on acme_tiny.py part, but it's expected, since the key isn't legitimate.
Thanks again for the script!


Julia Registrator

Feature description
To register a package we can install the julia registor on github.
I requested this access. Maybe you have to install this @danielinteractive
Here is the link: https://github.com/JuliaComputing/Registrator.jl?tab=readme-ov-file#install-registrator
Code of Conduct

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

Contribution Guidelines

[X] I agree to follow this project's Contribution Guidelines.

Security Policy

[X] I agree to follow this project's Security Policy.

OK let me try this just now

So step 1 of the instructions is now done, i.e. the JuliaRegistrator is added to this repo:

So @ya0 you can continue from step 2 in https://github.com/JuliaComputing/Registrator.jl#via-the-github-app


"Code for Philly Check-in System","\"Make sure to check in! Seriously, guys!\" *-- Chris Alfano, every meetup*","","","",""
Which shows up in the CfAPI as http://codeforamerica.org/api/projects/36729

They've fixed this on their end. Their CSV was causing them problems as well, so I don't think we need to support it.


Supervisor supports the ENV.

Supervisor supports the ENV.
Generate a table to show the ENV list.

http://supervisord.org/subprocess.html#subprocess-environment


Usage properties in Network SDK should be converted to double

currentValue and Limits are declared as int64. they should be declared as double (on the service they are declared as double)

@EvgenyAgafonchikov  please do this is on the sdk and PS after 9/25

Still breaking the api. Any news on this topic?

based on https://github.com/Azure/azure-rest-api-specs/issues/2366#issuecomment-438414616, issue is fixed.


Dependencies missing: dask, distributed, colorlog

dask
{
	"name": "ModuleNotFoundError",
	"message": "No module named 'dask'",
	"stack": "---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[4], line 1
----> 1 from concordia import RegionMapping

File c:\\Users\\kikstra\\Documents\\GitHub\\emissions_harmonization_historical\\.venv\\Lib\\site-packages\\concordia\\__init__.py:8
      5 from importlib.metadata import version as _version
      7 from .report import add_hypothesis, add_plotly_header, add_sticky_toc, embed_image
----> 8 from .utils import (
      9     CondordiaMagics,
     10     RegionMapping,
     11     VariableDefinitions,
     12 )
     15 try:
     16     __version__ = _version(\"concordia\")

File c:\\Users\\kikstra\\Documents\\GitHub\\emissions_harmonization_historical\\.venv\\Lib\\site-packages\\concordia\\utils.py:10
      7 from pathlib import Path
      8 from typing import Self, TypeAlias
---> 10 import dask.distributed as dd
     11 import numpy as np
     12 import pandas as pd

ModuleNotFoundError: No module named 'dask'"
}

distributed
ModuleNotFoundError                       Traceback (most recent call last)
File c:\Users\kikstra\Documents\GitHub\emissions_harmonization_historical\.venv\Lib\site-packages\dask\distributed.py:13
     [12](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/dask/distributed.py:12) try:
---> [13](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/dask/distributed.py:13)     from distributed import *
     [14](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/dask/distributed.py:14) except ImportError as e:

ModuleNotFoundError: No module named 'distributed'

The above exception was the direct cause of the following exception:

ImportError                               Traceback (most recent call last)
Cell In[4], [line 1](vscode-notebook-cell:?execution_count=4&line=1)
----> [1](vscode-notebook-cell:?execution_count=4&line=1) from concordia import RegionMapping

File c:\Users\kikstra\Documents\GitHub\emissions_harmonization_historical\.venv\Lib\site-packages\concordia\__init__.py:8
      [5](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:5) from importlib.metadata import version as _version
      [7](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:7) from .report import add_hypothesis, add_plotly_header, add_sticky_toc, embed_image
----> [8](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:8) from .utils import (
      [9](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:9)     CondordiaMagics,
     [10](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:10)     RegionMapping,
     [11](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:11)     VariableDefinitions,
     [12](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:12) )
     [15](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:15) try:
     [16](file:///C:/Users/kikstra/Documents/GitHub/emissions_harmonization_historical/.venv/Lib/site-packages/concordia/__init__.py:16)     __version__ = _version("concordia")
...

colorlog
{
	"name": "ModuleNotFoundError",
	"message": "No module named 'colorlog'",
	"stack": "---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[4], line 1
----> 1 from concordia import RegionMapping

File c:\\Users\\kikstra\\Documents\\GitHub\\emissions_harmonization_historical\\.venv\\Lib\\site-packages\\concordia\\__init__.py:8
      5 from importlib.metadata import version as _version
      7 from .report import add_hypothesis, add_plotly_header, add_sticky_toc, embed_image
----> 8 from .utils import (
      9     CondordiaMagics,
     10     RegionMapping,
     11     VariableDefinitions,
     12 )
     15 try:
     16     __version__ = _version(\"concordia\")

File c:\\Users\\kikstra\\Documents\\GitHub\\emissions_harmonization_historical\\.venv\\Lib\\site-packages\\concordia\\utils.py:15
     13 import pycountry
     14 from attrs import define
---> 15 from colorlog import ColoredFormatter
     16 from IPython.core.magic import Magics, cell_magic, magics_class
     17 from pandas import DataFrame, isna

ModuleNotFoundError: No module named 'colorlog'"
}

That sounds like you did not install the conda environment from environment.yml

You're right. And you're also right that that's is the way that it is currently supported - according to the readme.
I was doing it using poetry, which I suppose looks at the pyproject.toml.

It does look at that, but it's more by design than mistake. pyproject and pip/poetry and consorts in general have a very bad track record of working with anything that involves geospatial analysis around the gdal and osgeo libraries. It is in my experience safer to always manage environments with conda/mamba for those analyses. If zeb or others say that is no longer the case, i would be open to including them in pyproject.toml, but a few years back this has caused hard to debug headaches.

Nah all good @coroa thanks. We'll switch to pixi

Okidokes, so then I suppose this issue can be closed.

@znichollscr Thanks. pixi looks very helpful.

Glad it helps. Our default environment managers are now basically:

anything needing conda dependencies, use pixi (an example repo doing the whole set is here if you're interested: https://github.com/climate-resource/input4mips_validation)
everything else, use uv (e.g. here: https://github.com/climate-resource/bookshelf)

In both cases, we use pdm as the build backend because it has some features we like, like locked builds (this repo basically demonstrates it https://github.com/openscm/openscm-zenodo). I reckon that uv will have their own version of this pretty soon, at which point we might stop using pdm.


Escape jQuery identifiers

As discussed on the forum.
See: https://groups.google.com/forum/#!topic/liftweb/jorUqMmCLDQ

If we're going to fix this, should we fix it for all of the meta characters where this is a problem?
This SO entry has some good info on the problem.
I think using this is probably the best way to go:
jQuery(document.getElementById(elementOrId))

I don't hate that solution.

Good call.  I'll update my local env and test a little bit.  If it looks good there, I'll update the pr.

quick question, why is this marked for 3.1 and not for the next RC?

IIRC, it's not a regression, so there was no reason to hold up a 3.0 final release for it (this was opened after th RC cycle began).

ok, and I imagine we don't want to add it now to the next RC in case of a regression caused by this fix, right? (because there will be another RC to handle that one js bug we still have)

Soo I think we can move forward on this. @eltimn it sounds like you had a look here? We good to go?

Yes. Looks good.

All in then!


Add VLC OS X web plugin 2.2.1

Thank you for the contribution. It needed some fixes, so they were made in 0fc9ce4a8689953f27487d9a13b05d1956766799. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


tf.layers.conv3d_transpose missing

In tf.layers, there are conv2d and conv2d_transpose. There is conv3d, but no conv3d_transpose.
I'm wondering if it is possible to have conv3d_transpose any time soon.
Thanks,

Hi, I'm taking this issue up. Please expect a PR soon. :smile:


How to suppress warnings

Since you probably run a "private" instance of sbcl, how can I suppress warnings like this one:
Destructive function SORT called on constant data: "elbow"
See also:
The ANSI Standard, Special Operator QUOTE
The ANSI Standard, Section 3.7.1
sbcl isn't strictly speaking a functional language.  It's ok to have mutable data unless you practice the immutable religion--but that should be a choice each person gets to make.

That warning comes from sbcl, so you could try aiming your rant at them and see if anything comes of it.
A quick google search for "sbcl suppress warnings" shows that putting
(declaim (sb-ext:muffle-conditions cl:warning))

at the top of the file, or executing it in the repl, will do it.

Well, sorry and that should do it.   Making it source file specific makes sense.
From: nobody-famous @.>
Reply-To: nobody-famous/alive @.>
Date: Thursday, July 27, 2023 at 3:27 PM
To: nobody-famous/alive @.>
Cc: Lewis Levin @.>, Author @.***>
Subject: Re: [nobody-famous/alive] How to suppress warnings (Issue #165)
That warning comes from sbcl, so you could try aiming your rant at them and see if anything comes of it.
A quick google search for "sbcl suppress warnings" shows that putting
(declaim (sb-ext:muffle-conditions cl:warning))
at the top of the file, or executing it in the repl, will do it.
—
Reply to this email directly, view it on GitHubhttps://github.com/nobody-famous/alive/issues/165#issuecomment-1654403911, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIYWLNXWEU3KJKVJCHYOCTXSK6IFANCNFSM6AAAAAA22OOEGQ.
You are receiving this because you authored the thread.Message ID: @.***>


feat: created custom help message

Description
Add the reverted changes from #124

when I was merging that old branch to any other branch it was overwriting the commits that were ahead. So I created another branch from the upstream/main and then manually merged the changes from that merge. I think it should work now, After this, I will re fork again.

@boyney123 can you help me with this, I am getting this error in the action, it is running fine in my local setup. The error is originating from scripts.

><EMAIL_ADDRESS>build /home/runner/work/cli/cli
> node scripts/fetch-asyncapi-example.js && tsc

internal/modules/cjs/loader.js:905
  throw err;
  ^

Error: Cannot find module 'request'
Require stack:
- /home/runner/work/cli/cli/scripts/fetch-asyncapi-example.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)
    at Function.Module._load (internal/modules/cjs/loader.js:746:27)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at require (internal/modules/cjs/helpers.js:93:18)
    at Object.<anonymous> (/home/runner/work/cli/cli/scripts/fetch-asyncapi-example.js:3:17)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:76:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/home/runner/work/cli/cli/scripts/fetch-asyncapi-example.js' ]
}

Hey @Souvikns ,
Yeah, that's very odd! Just checked on my machine and working fine, 🤔 also the request module is in the package.json and lock file.....
Interesting warning we get on that log stack
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

Looks like it might be failing/missing to install the node_modules (run npm install?)?

npm info it worked if it ends with ok
npm verb cli [
npm verb cli   '/opt/hostedtoolcache/node/14.18.1/x64/bin/node',
npm verb cli   '/opt/hostedtoolcache/node/14.18.1/x64/bin/npm',
npm verb cli   'install',
npm verb cli   '--loglevel',
npm verb cli   'verbose'
npm verb cli ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm verb npm-session 88392f2cfdf4e5e3
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm timing stage:loadCurrentTree Completed in 18ms
npm timing stage:loadIdealTree:cloneCurrentTree Completed in 1ms
npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but package-lock.json was generated for lockfileVersion@2. I'll try to do my best with it!
npm timing stage:loadIdealTree:loadShrinkwrap Completed in 1331ms
npm http fetch GET 200 registry.npmjs.org/fsevents 137ms
npm http fetch GET 200 registry.npmjs.org/fsevents/-/fsevents-2.3.2.tgz 47ms
npm timing stage:rollbackFailedOptional Completed in 0ms
npm timing stage:runTopLevelLifecycles Completed in 2040ms
npm verb stack Error: Unsupported platform for<EMAIL_ADDRESS>wanted {"name":"fsevents","version":"2.3.2","engines":{"node":"^8.16.0 || ^10.6.0 || >=11.0.0"},"os":["darwin"],"dependencies":{},"optionalDependencies":{},"peerDependenciesMeta":{},"devDependencies":{"node-gyp":"^6.1.0"},"bundleDependencies":false,"peerDependencies":{},"deprecated":false,"_resolved":"registry.npmjs.org/fsevents/-/fsevents-2.3.2.tgz","_integrity":"sha512-xiqMQR4xAeHTuB9uWm+fFRcIOgKBMiOBP+eXiyT7jsgVCq1bkVygt00oASowB7EdtpOHaaPgKt812P9ab+DDKA==","_shasum":"8a526f78b8fdf4623b709e0b975c52c24c02fd1a","_shrinkwrap":null,"_id":"fsevents@2.3.2","_from":"fsevents@~2.3.2","_requested":{"type":"range","registry":true,"raw":"fsevents@~2.3.2","name":"fsevents","escapedName":"fsevents","rawSpec":"~2.3.2","saveSpec":null,"fetchSpec":"~2.3.2"},"_spec":"fsevents@~2.3.2","_where":"/home/runner/work/cli/cli/node_modules/chokidar"} (current: {"os":"linux","cpu":"x64"})
npm verb stack     at checkPlatform (/opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/node_modules/npm-install-checks/index.js:46:14)
npm verb stack     at thenWarnEngineIssues (/opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/lib/install/validate-args.js:63:5)
npm verb stack     at /opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/node_modules/iferr/iferr.js:3:64
npm verb stack     at checkEngine (/opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/node_modules/npm-install-checks/index.js:25:10)
npm verb stack     at module.exports.isInstallable (/opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/lib/install/validate-args.js:60:3)
npm verb stack     at resolveWithNewModule (/opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/lib/install/deps.js:677:10)
npm verb stack     at /opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/lib/install/deps.js:600:7
npm verb stack     at /opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/node_modules/iferr/iferr.js:3:64
npm verb stack     at /opt/hostedtoolcache/node/14.18.1/x64/lib/node_modules/npm/node_modules/call-limit/call-limit.js:28:43
npm verb stack     at processTicksAndRejections (internal/process/task_queues.js:77:11)
npm verb pkgid<EMAIL_ADDRESS>npm verb cwd /home/runner/work/cli/cli
npm verb Linux 5.11.0-1021-azure
npm verb argv "/opt/hostedtoolcache/node/14.18.1/x64/bin/node" "/opt/hostedtoolcache/node/14.18.1/x64/bin/npm" "install" "--loglevel" "verbose"
npm verb node v14.18.1
npm verb npm  v6.14.15
npm ERR! code EBADPLATFORM
npm ERR! notsup Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm ERR! notsup Valid OS:    darwin
npm ERR! notsup Valid Arch:  any
npm ERR! notsup Actual OS:   linux
npm ERR! notsup Actual Arch: x64
npm verb exit [ 1, true ]
npm timing npm Completed in 4447ms

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/runner/.npm/_logs/2021-11-26T11_21_43_435Z-debug.log
Error: Process completed with exit code 1.

Interesting looks like npm install is not working correctly

@Souvikns not sure if it's related to your PR, but looks like problems with fsevents which is only supported on mac OSX.
This windows test environment is failing. (More details here https://stackoverflow.com/questions/48898312/npm-error-unsupported-platform-for-fsevents-when-deploying-to-azure).
There is some suggestions in this solution that we could tell it to make fsevents an optional dep, which might get skipped on windows... but not sure, do you want to give it a try?

It is also running in my local environment, and I am using ubuntu. I am giving it a try with what you suggested.

@boyney123 thanks it is working now 👍🏻

@derberg @fmvilas @magicmatatjahu waiting for your review.

@Souvikns just curious why we need to ignore so many files in the case of Sonar Cloud scans?

It was showing some function is bigger than it should be, but in this case, it is needed, and in the command directory there were some warnings for the function naming conventions, which were needed by oclif to work correctly,

@Souvikns but we are ignoring not specific file but all src/help/** with wildcard.

so for the cognitive complexity instead of ignoring in config I suggest you just put /* eslint-disable sonarjs/cognitive-complexity */ before the complex function that can't be simpler
in case of naming conflict, just ignore specific files and add comments there why there are ignored so in the future it is clear

@Souvikns Also, if you need to disable sonar (not eslint, but exactly sonar cloud) only for certain function, you can add comment // NOSONAR next to function declaration/type etc:
function fn() { // NOSONAR

I will be removing the sonar.exclusion property and see what errors I am getting and then I will solve them one by one

@derberg @magicmatatjahu I have removed all .sonarcloud.properties file as there was only exclusion, now sonar cloud is scanning the complete codebase, and I am resolved the code smells.

I have removed the warnings but for some reason, I am getting new code smells, Will be fixing them soon.

I guess I need @magicmatatjahu and @jotamusik approval to merge

@Souvikns at least @magicmatatjahu as he "requested changes" initially and we can't merge before he accepts

Yeah, I made the changes @magicmatatjahu requested, I have removed the help command and now the only --help flag works, so now you won't see an extra help command in the root help.
Let me show how the help message is looking right now.
Root Help

Command Help

@jotamusik waiting from your review

@Souvikns just fyi you do not need a review of every CODEOWNER, this is needed only when we want to add someone to CODEOWNER file. Imho if you have the approval of 2 maintainers here, you are good to go, unless you synced with Jorge and he needs to have a look.
btw tests are failing, please have a look

ok @derberg I will merge this after I fix the test, They were working before, but when I updated the branch they are failing.


Add getter in Registry for (target, chipfamily)

The idea is to use getters to obtain a list of controllers that belong to the same family, and to obtain families from the controller name. This will help in building a convenient API for a GUI.

Welcome 👋, thanks for contributing!
Could you explain why you want this function?
@Tiwalun
The idea is to use getters to obtain a list of controllers that belong to the same family, and  to obtain families from the controller name. This will help in building a convenient API for a GUI. As shown in my comment above with a screenshot, it is already working for me.

@Tiwalun mind acking & merging or re-reviewing? :)


V3 usage metrics no longer support "include" filtering

In the v2 usage metrics, you could specify which usage types to include, like the following:
  utilizationLogger:
    subscriptions:
     - type: fabric.usage
       include:
        - usage.ingress.tx
        - usage.egress.tx
        - usage.ingress.rx
        - usage.egress.rx

It would be nice to have this capability in the V3 metric format as well since this data can emit at a large scale, and being able to fine tune is helpful.

I don't believe fabric.usage has ever supported include configuration, but it shouldn't be hard to add.


IDE / gui system that’s like Postman.

If anyone is interested there is an open effort going on to build a golang gui for api modelling and testing. It’s all golang too .
https://github.com/chapar-rest/chapar
Just go run . , and you’re flying …
it already supports rest , grpc, and I selfishly added an issue to support AsyncAPI.
https://github.com/chapar-rest/chapar/issues/47
The validation of asyncapi is already a golang code base so the effort to build the gui is realistically doable.

Thanks for pointing that out ! I'll add a message on the issue you made on Chapar to support your proposal :)


Changed Color Palette

Having gotten Polymer up and running locally, I followed the Google Developers tutorial in order to see how to change the theme color options.

Submitted 644d5af for change.


Should all raw source data be preserved?

When a source data file is downloaded for the first time, as part of loadGeometry or loadVariable, should it be preserved as-is in the repository file system?
A good location would be inst/data//.ext
loadgeometry and loadVariable currently look in the tmpdir for downloaded files so that they are only downloaded per user session. Saving them here would allow for them to only ever be downloaded once per instance of gaiadb.
Maybe a preserve data flag to allow users to opt out of saving the file locally. Or a clean storage function to allow them to clear the files.

Question: How would this apply to an API data_source/variable_source?


Redesign heroModifiers to not be a string API

I think the API would be far clearer and easy to use if heroModifers wasn't a string. The benefits of a non "stringly-typed" API are pretty clear, but to outline a few:

Improved safety
Compiler checking
Easier discoverability and documentation

I would love to take on this task, but I wanted to ask which of two directions I should go, and if you might merge a PR like this before I start working on it.
Option 1
Introduce a new Enum with associated values for each Modifier - something like
enum Modifier {
    case scale(CGFloat)
    case fade
    case ...
}

Views would retain the heroModifiers property, but it would become an array of Modifier, used like this: foo.heroModifiers = [.fade, .scale(0.5)].
This design is nice because an enum encapsulates the possible modifiers well, especially with associated values. One drawback is that it's unclear at usage that you can only use one of each Modifier. However, the current String API also has this problem.
Option 2
Introduce a new Modifiers struct with optional properties for each Modifier, something like:
struct Modifiers {
    var scale: CGFloat?
    var fade = false
    var transform: (CGFloat, CGFloat)?
    ...
}

This design is nice because it ensures one of each Modifier is used at the most. Views would also have a heroModifiers property, but would be the type of the struct. This design might be used like this:
foo.heroModifiers.scale = 0.5
foo.heroModifiers.fade = true

I'd love your opinion on this, or would love to know if you're already working on something similar. I'd love to help implement it!

+1 I had this same thought while reviewing this repository and was glad I was not alone when I went to file an issue! Personally I prefer the syntax/feel of using the enum but it seems to me that the struct is the way to go to enforce one of each type of modifier.
One issue I see with both; however, is the inability for users of this library to add their own modifiers, especially since structs and enums in swift are final types. Perhaps have an extra enum case .custom(CustomModifier) where CustomModifier is some protocol which describes how to perform the transition?

Yeah, very tough to decide between enum or struct. Does the current API allow custom modifiers? Or does that deserve a new/separate issue? Haven't had too much time to go in depth.

That is a good point. Actually, I previously have a branch that uses swift enum for HeroModifiers. But I dropped that because I wanted storyboard and plugin support.
One thing about what you are saying is incorrect. When applying transforms to views we might need to use multiple of the same modifiers and the orders are important.
For example: translate(50,50) rotate(1.57) behave differently than rotate(1.57) translate(50,50)
One rotates first, one translates first.
Something like translate(50,50) rotate(1.57) translate(50,50) is also supported currently.
I do feel that what you are proposing is far cleaner than the string API. It is more easy to discover and better to document them and probably faster too. But we need some more discussions about this. There are some immediate drawbacks too:

Breaks storyboard support, IBInspectable does not support enum type at all.
Plugins won't be able to introduce their own modifiers
Same modifier would not be able to support multiple parameters

If we are going to implement it. I would prefer the 1st option. The second one, regards to what I have said about multiple modifiers, wouldn't work.

In that case, an array seems perfect - it'll retain order and allow multiple of the same. RE: storyboard support - could there be an IBInspectable property that allows you to use a string in IB? But isn't exposed programmatically? And when set, would convert to the array? Haven't played with IBInspectable much.

@kylebshr I feel like that conversion would need to be done at runtime

Yes, that's what I meant :)

The current behaviour of HeroModifiers will recognize the order of modifiers that needs them (all the transform modifiers). For other modifiers like fade, arc... where only one is needed, the last one will override the options of the previous ones.
e.g cascade(0.02), cascade(0.05) will be equivalent to just cascade(0.05)
We can still do this with an array of enums. So I think it is the better way to go.
For storyboard support, It could be done at runtime using a setter.
But we will have two different attribute, one is of string type, the other one is a [Modifier] type.
If we change the array, do we expect the string attribute to return the modified version? And how do we name those attributes? one heroModifiersString? the other one heroModifiers? It might get ugly.

I agree it could get ugly... ideally there would be a storyboard only string property. And perhaps it could be set only?

That would be ideal but in swift you can't do something along the lines of public private(get) heroModifierString: String

I have also tried this fun thing with swift. I am not saying that we should do this.
extension String: ExpressibleByArrayLiteral{
    typealias Element = Modifier
    init(arrayLiteral: Element...) {
        // returns a string from [Modifers]
    }
}

That makes this work! which is what we want. heroModifers is still a string type
view.heroModifers = [.fade, .scale(0.5)]

But I couldn't find a way to make this extension private with swift 3. So that is a huge pollution to the String type.

@ikzhao You may be able to make an intermediate type thats both ExpressibleByArrayLiteral and ExpressibleByStringLiteral

@alexzielenski You bet. But storyboard's IBInspectable doesn't support ExpressibleByStringLiteral. They won't show up if it is not a String type. sad! 😭😭

May be we can use conditionally compile preprocessor macro.
#if TARGET_INTERFACE_BUILDER
    public var heroModiferString:String
#else
    internal var heroModiferString:String{
       set{
          // change heroModifers
       }
    }
    public var heroModifers:[Modifer]
#endif

I think we're kind of discovering why a lot of UIKit APIs aren't usable in storyboards.. many issues with a well designed programmatic API compatibility.

On Jan 3, 2017, at 9:03 PM, Luke Zhao<EMAIL_ADDRESS>wrote:
May be we can use conditionally compile preprocessor macro.
#if TARGET_INTERFACE_BUILDER
public var heroModiferString:String
#else
internal var heroModiferString:String{
set{
// change heroModifers
}
}
public var heroModifers:[Modifer]
#endif
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@kylebshr The conditionally compile macro may just work. If you are interested, can you take a look a it?
Another option that maybe better is something like this.
struct Modifier{
  var duration:TimeInterval?
  var delay:TimeInterval?
  var timingFunction:CAMediaTimingFunction?
  required init(fromString:String){}
}

struct Fade:Modifier{}
struct Scale:Modifier{

}

This have some benefits over the enum one:

support custom modifier
individual delay, duration, timingFunction is supported

What do you guys think?

I assume you mean a protocol or class, since you can't inherit from a struct? If a protocol works, that's a probably pretty good idea, but I wouldn't want to use subclassing. It doesn't really help with the storyboard issue either as far as I can tell.
I'll take a look at the macro!
On Jan 3, 2017, 10:20 PM -0500, Luke Zhao<EMAIL_ADDRESS>wrote:

@kylebshr The conditionally compile macro may just work. If you are interested, can you take a look a it?
Another option that maybe better is something like this.
struct Modifier{
var duration:TimeInterval?
var delay:TimeInterval?
var timingFunction:CAMediaTimingFunction?
required init(fromString:String){}
}
struct Fade:Modifier{}
struct Scale:Modifier{
}
This have some benefits over the enum one:
• support custom modifier
• individual delay, duration, timingFunction is supported
What do you guys think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Also — this framework seems to be picking up a lot of steam! Could we get some labels for issues and PRs? Maybe "discussion" "WIP" etc.? Thanks! 😀

Looks like the macro is a no-go — it compiles the var away when built for device/sim. The TARGET_INTERFACE_BUILDER macro is literally only for when it's compiled for IB within Xcode.

Yea, but we can have the same var in the else clause right? We just want to hide it from being accessed in code. i.e. change it to internal
#else
    internal var heroModifierString:String{
       set{
          // change heroModifiers
       }
    }
    public var heroModifiers:[Modifier]
#endif

But IB properties are set at run time, and it can't set it because it has become internal at that point — you get this:
2017-01-03 22:37:21.789 Locator[34050:1110487] Failed to set (heroModifierString) user defined inspected property on (UILabel): [<UILabel 0x7fb1f6d03b20> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key heroModifierString.

Really? I thought setValue:forKey: works even when the variable is internal in swift.

I can double check with a more real test, but I don't think that'll work across a framework. Sure, it can set an internal var within the same framework/project, but then we can't use cocoapods.

@kylebshr  LOL. I got this working even through cocoapods:
#if TARGET_INTERFACE_BUILDER
  @IBInspectable public var heroModifiersString: String? {
    get { return nil }
    set {}
  }
#else
  internal var heroModifiersString: String? {
    get { fatalError() }
    set {
      heroModifiers = newValue
    }
  }
#endif

I uploaded it on the swift-internal-test branch.
You can use to install it.
pod "Hero", :git => 'https://github.com/lkzhao/Hero.git', :branch=>"swift-internal-test"

And you will see that storyboard can access the variable even though it is marked internal.

That's very interesting... can't help but wonder if it's a bug. I guess now we just have to decide the rules around its use? What happens if a string is set and programmatic options are set? I'm thinking raise an exception, but we could just pick a priority.

On Jan 3, 2017, at 11:21 PM, Luke Zhao<EMAIL_ADDRESS>wrote:
@kylebshr LOL. I got this working even through cocoapods:
#if TARGET_INTERFACE_BUILDER
@IBInspectable public var heroModifiersString: String? {
get { return nil }
set {}
}
#else
internal var heroModifiersString: String? {
get { fatalError() }
set {
heroModifiers = newValue
}
}
#endif
I uploaded it on the swift-internal-test branch.
You can use to install it.
pod "Hero", :git => 'https://github.com/lkzhao/Hero.git', :branch=>"swift-internal-test"
And you will see that storyboard can access the variable even though it is marked internal.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

The string one should just modify the programmatic options when set (only springboard can do this). Since developers cannot access the string one (because it is marked as internal), it is not a problem. The programmatic options should always be the source of truth.

Maybe you could adapt the pattern I've used for Traits, e.g. leverage hero identifier in both code and IB, and then have a central repository that contains modifier configurations for it?
Also take a look at scripts, I've a script there that generates typed interface for Storyboard identifiers, so once someone sets them in Storyboard, it would generate enumeration containing all identifiers, that way you can avoid strings in code

Enums would be the best option here imho

@evermeer The 'Mirror' class is pretty cool. Never knew that they existed.
One thing that concerns me with this implementation is the pollution to the default String type. Is your solution able to extended String again to support another array literal type?
I still feel like we should convert it to a pure swift enum API. With the ability to set it in the storyboard with a string. That will give us the excellent syntax when matching these internally.
'''swift
switch incomingModifier {
case .scale(let x, let y, let z):
//...
}
'''

I agree Luke, there's not much reason to go from enum -> String, just the other way around. I'll take a stab at implementing today, using the macro trick.
Sent from my iPhone

On Jan 4, 2017, at 10:47 AM, Luke Zhao<EMAIL_ADDRESS>wrote:
@evermeer The 'Mirror' class is pretty cool. Never knew that they existed.
One thing that concerns me with this implementation is the pollution to the default String type. Is your solution able to extended String again to support another array literal type?
I still feel like we should convert it to a pure swift enum API. With the ability to set it in the storyboard with a string. That will give us the excellent syntax when matching these internally.
'''swift
switch incomingModifier {
case .scale(let x, let y, let z):
//...
}
'''
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Well, you could also leave out the String extension and just use the .toModifierString() on the array. But I agree that It would be better to have an enum api.
Using 'Mirror' is kind of my thing. It's about the only reflection option available in Swift. It's the base for my https://github.com/evermeer/EVReflection library.

Sorry for joining the conversation late. I started tinkering with ideas to add a composable API yesterday, before I realized this conversation was taking place. By the time I had a PR ready it looks like the conversation already made a lot of progress.
I posted a PR here for a modifier string composition API using swift method chaining. I didn't mean to suggest that the PR should be taken as-is without discussion. Rather, it was just easier to convey my thoughts in code, so I decided to write it and put it up for review.
https://github.com/lkzhao/Hero/pull/15
I tried a few approaches before settling on the method chaining interface. The inspiration for method chaining actually came from the Hero usage docs. The examples for modifiers take the form of "fade() position(0,100) arc(1)" etc., and I thought a method chaining seemed most similar to the documentation.
I looked at String extensions too and had similar concerns around API pollution.
I looked at Swift enums too. Enum cases with associated values seem to make a ton of sense to encapsulate a modifier. But the trouble I ran into here was constructing a simple and intuitive API to actually create and add these modifiers to build the animation. The array example above seems like the closest, but ultimately still leaves something to be desired to me.
I settled on method chaining because of the readability, simplicity, and excellent discoverability aspect. Autocomplete on the HeroComposition makes it dead simple to see what options are available to add additional attributes to the modifier composition. Using optional method parameters, you can also construct very complex modifiers that still have simple implementations and public interfaces.
Here's an example of what a modifier string composition looks like with method chaining:
HeroComposition().fade().translate(0,150).rotate(-1,0,0).modifier

It also supports modifiers like cascade using swift enums for those cases:
HeroComposition().fade().translate(0,150).rotate(-1,0,0).cascade(1,.bottomToTop).modifier

I'm going to go ahead and close the PR to let the conversation continue further without any pressure. If ya'll like this idea feel free to incorporate it in the evolution of the API, or let me know and I'll be happy to modify based on feedback.

This has been a great discussion, but other stuff has come up and I likely won't be able to help implement this any time soon. Feel free to carry on discussion and implementation without me!

I can take this on. Will have a PR up in a day or two. Lets discuss before I merge.  😀

Sounds awesome. Let's form a rock solid solution here

Finished my initial implementation. Lets discuss over at #20.

Is this already done or should I create a pull request @lkzhao ? :)

looks like its already done

Yep this is done. In master already. Thank you all!


Bug: Nav icon (hamburger icon) is invisible in light mode

Screenshot:

Could you please assign this issue to me?

@N-Shar-ma Already made an issue about problem with theme in navbar(#211), and currently working on it.

@N-Shar-ma Already made an issue about problem with theme of navbar in mobile view(#211), and currently working on it.

Oh alright then, I'll close this issue


feat(core): [LW-9405] create SetupSharedWallet component

Checklist

[x] JIRA tickets: https://input-output.atlassian.net/browse/LW-9405
[x] Screenshots added.

Proposed solution
This PR implements the foundation for the shared wallets flow.
Screenshots
Shared wallet setup step components in Storybook in the core package:

@renanvalentin thanks for building the Storybook base 👍 I can see you have the same merge conflict as I do, hopefully you won't stumble into the same build issue too 🤞

@renanvalentin i could resolve my issue with the staking package by adding "@types/react": "17.0.18", to the peerDependencies

coverage: 52.908% (-0.07%) from 52.976%
when pulling f2a0d4e4ce2e7a744d1c4f03ccf5b3f9db764e2c on lw-9405-shared-wallet
into 2ff61f57829d97e1510dd3b017a93c81c41ecf03 on main.

This PR is ready to be merged but blocked due to missing approvals from @input-output-hk/lace-staking  @input-output-hk/lace-tech-leads


Amplify update causes useless error

Just updated the Amplify CLI. Ran amplify push after changing the redirect URLS and was given this error:

Syntax Error: Expected Name, found String "version"
An error occured during the push operation: Syntax Error: Expected Name, found String "version"

Any ideas?

@oliverandersencox
Please provide more information so we can investigate;

Operating system
Amplify CLI version
Nodejs Version
Which categories are added to your backend (e.g. api, auth, etc.), which category gave you the error?

@UnleashedMind

Mac OSX Catalina 10.15.1
Amplify Cli - 4.9.0
Node JS - 13.5.0
I have Auth, Api and Storage all working absolutly fine.

This morning I ran the command to install/update the amplify client
I then ran 'amplify update auth' so as ti change the redirect URL's to work with my IOS app as I have done many times before.
After I then ran amplify push and was immediately presented with the error.
I was able to add to my api just after the update, in which amplify push api worked just fine, so I would presume the auth part is causing the issue.

@UnleashedMind I have tried everything I can think of and nothing is resolving this issue. It happens everytime i run 'amplify push' or amplify push auth/api etc.
I recently upgraded to node 10 in which amplify asked me if I would like it to update everything to node 10, in which I said yes.
I cannot do anything until a solution is found. The error message indicates its a graphQL parsing error but it doesn't give a reason.

Syntax Error: Expected Name, found String "version"
GraphQLError: Syntax Error: Expected Name, found String "version"
at syntaxError (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/error/syntaxError.js:24:10)
at expect (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:1299:32)
at parseName (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:92:15)
at parseField (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:295:21)
at parseSelection (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:284:72)
at many (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:1348:16)
at parseSelectionSet (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:272:17)
at parseOperationDefinition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:194:21)
at parseExecutableDefinition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:172:12)
at parseDefinition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/graphql/language/parser.js:146:12)

@oliverandersencox This doesn't seem to be related to the Node version update but seems ties to your GraphQL schema. Could you please share your schema?

@oliverandersencox Were you able to resolve this issue?

Don't you dare!


remove uses of github.com/runc/libcontainer/cgroups

relates to https://github.com/containerd/containerd/pull/8722

remove uses of github.com/runc/libcontainer/cgroups
runc considers libcontainer to be "unstable" (not for external use),
so we try not to use it. Commit ed47d6ba7652259e9a458581ef8f34fb44d0f561 (https://github.com/containerd/containerd/pull/8722)
brought back the dependency on other parts of libcontainer, but looks to
be only depending on a single utility, which in itself was borrowed from
github.com/coreos/go-systemd to not introduce CGO code in the same package.
This patch copies the version from github.com/coreos/go-systemd (adding
proper attribution, although the function is pretty trivial).
runc is in process of moving the libcontainer/user package to an external
module, which means we can remove the dependency on libcontainer entirely
in the near future. There is one more use of libcontainer in our vendor
tree; it looks like CDI is depending on one utility (devices.DeviceFromPath);
https://github.com/containerd/containerd/blob/a943033a8b6d8dd94a171c6e9528ef413b5b22f3/vendor/github.com/container-orchestrated-devices/container-device-interface/pkg/cdi/container-edits_unix.go#L38
We should remove the dependency on that utility, and add a CI check to
prevent bringing it back.
pkg/systemd: use sync.Once for systemd detection
This brings over the enhancement from https://github.com/moby/moby/commit/a506630e570735d13c587a9d755f3d5729658f28 (https://github.com/moby/moby/pull/41656).
We don't expect the systemd state to change while containerd is running,
so we can use a sync.Once for this, to prevent stat'ing each time.

also opened https://github.com/cncf-tags/container-device-interface/pull/158 to remove the remaining part


1.3.0

Release 1.3.0

Should we update the readme #62 before bumping the version?

what's the conflict? i think mickael already published when he makes these

Good point, there isn't a conflict just wanted to see if we can weasel in updates to the readme in the new version

I run npm publish only after merging this type of PRs. Please let me know if this works!


Remove need/use of password.txt

Removes redundant generation/use/deletion of password.txt by placing the password as a variable and running echo with the variable directly.
Previously, the text 'password' was echoed into password.txt, then read from it again, before the file would finally get deleted.

Thanks for this!


Contribute to this repo 🔔

Contribute to this respository to greet the world and getting started with Open Source/Github.

Please be active to accept the pull requests!

@Nazchanel, do you like to contribute?

Yes

Go ahead @Nazchanel


Laravel mix is not recognizing Tailwind classes in components

When I run "npm run watch" it doesn't save the classes in the components files.
composer.json
"laravel/framework": "^9.11",
"mkocansey/bladewind": "^0.1.7",
Package.json
"postcss": "^8.4.6",
"postcss-import": "^14.0.2",
"tailwindcss": "^3.0.18"
"laravel-mix": "^6.0.6",
tailwind.config.js
content: [
    './vendor/laravel/framework/src/Illuminate/Pagination/resources/views/*.blade.php',
    './storage/framework/views/*.php',
    './resources/views/**/*.blade.php',
],

webpack.mix.js
mix.js('resources/js/app.js', 'public/js').postCss('resources/css/app.css', 'public/css', [
require('postcss-import'),
require('tailwindcss'),
require('autoprefixer'),
]);

@gsbarbo I tried replicating your problem with the exact package versions above but got the desired results.
With npm run watch running,
I edited resources > views > components > bladewind > button.blade.php and added blur-lg to line:53
{{ $attributes->merge(['class' => "bw-button blur-lg $size $type $name $primary_color $is_disabled"]) }}
My public > css > app.css file got updated with the css for .blur-lg.
It will help to know (so I can replicate ditto)

Which component(s) you tried adding/updating css for
What css did you add/change.

@gsbarbo any luck getting this to work?

I was maybe premature with opening a ticket or at least I don't remember the exact thing I was looking at. I was looking at the modal component and the widths. (w-1/2 w-1/3 w-1/4 w-1/5 w-1/6) They are in app.css but all the models are the same size (Full width) except for big. Then its small.

Oh right. The modal one was actually a bug that has been fixed in the current version.


Register package

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/44853
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 4224f9769dad25b555ffa39c5ec2f216e988129c
git push origin v0.1.0

Merged 🎉


The 'params' keyword is left out for array type parameters

For OutputTypeAttribute, the constructor is public OutputTypeAttribute(params Type[] type).
However, in PowerShellStandard.Library, the params keyword is left out, and thus I got compilation errors when compiling some existing code.
It's possible that the params keyword in other APIs is also left out in the ref assembly.
The params keyword is represented in the reflection metadata by a 'CustomAttribute':
PS:13> $parameterInfo.CustomAttributes

AttributeType              Constructor  ConstructorArguments NamedArguments
-------------              -----------  -------------------- --------------
System.ParamArrayAttribute Void .ctor() {}                   {}

Other instances:
ScriptBlock.Invoke(object[] args)

yes - https://github.com/PowerShell/PowerShellStandard/issues/3 is a specific instance of this

fixed in https://github.com/PowerShell/PowerShellStandard/pull/9


faster LIST operations

This should be a faster LPOP, -= first run indexOf then remove, just remove is enough.
Redis list is like a linked list, so use ListBuffer instead of ArrayBuffer could be a improvement.
Before:
PING_INLINE: 80906.16 requests per second
PING_BULK: 92936.80 requests per second
SET: 72463.77 requests per second
GET: 74349.44 requests per second
INCR: 76452.60 requests per second
LPUSH: 3743.22 requests per second
LPOP: 3724.39 requests per second
SADD: 64143.68 requests per second
SPOP: 74682.60 requests per second
LPUSH (needed to benchmark LRANGE): 4025.60 requests per second
LRANGE_100 (first 100 elements): 23501.76 requests per second
LRANGE_300 (first 300 elements): 12115.34 requests per second
LRANGE_500 (first 450 elements): 10577.53 requests per second
LRANGE_600 (first 600 elements): 6588.48 requests per second
MSET (10 keys): 16160.31 requests per second

After:
PING_INLINE: 39093.04 requests per second
PING_BULK: 63775.51 requests per second
SET: 47961.63 requests per second
GET: 69783.67 requests per second
INCR: 66844.91 requests per second
LPUSH: 61919.50 requests per second
LPOP: 66755.67 requests per second
SADD: 69979.01 requests per second
SPOP: 75244.55 requests per second
LPUSH (needed to benchmark LRANGE): 63091.48 requests per second
LRANGE_100 (first 100 elements): 32851.51 requests per second
LRANGE_300 (first 300 elements): 17214.67 requests per second
LRANGE_500 (first 450 elements): 12843.57 requests per second
LRANGE_600 (first 600 elements): 10126.58 requests per second
MSET (10 keys): 19876.76 requests per second

This is amazing - thank you!

Maybe you could update benchmark in README

Yep I'll do that, thanks for the reminder!
On 17/07/2015 3:12 PM, "tramchamploo"<EMAIL_ADDRESS>wrote:

Maybe you could update benchmark in README
—
Reply to this email directly or view it on GitHub
https://github.com/stephenmcd/curiodb/pull/12#issuecomment-122187020.

Updated benchmarks here: 6a6a700f07c34098d22f2ccf9add652c506fe869
Also added a script for doing it automatically: https://github.com/stephenmcd/curiodb/blob/master/benchmark.py

That's cool


Add activation code for Judges upgrade

Make all necessary preparations for the Judges upgrade June 20 2024. Used previous Joshua upgrade commits as template for this commit.
~Still need to add the Python test.~ f6ba1a7

@schancel I removed you from the reviewers to try and just push through the merge myself. But I'll wait for you to handle it.

@schancel please review, I'm not sure if I screwed up the python test or if that's expected to fail at this point.

@schancel bump

Joy. Teamcity is not working. I will have to investigate later.


'rotations' parameter is redundant if Qrack converts to observable basis

expval() has a parameter for the observable type to calculate, but this turns out to be redundant with rotations parameter passed into apply_operations(): I realize now, one either uses rotations parameter or transforms basis for expval() parameters, not both.

For everyone else's sake: @cosenal noted that the wrapper method, without parsing rotations, was redundant if rotations wasn't used. I removed the level of indirection, but I kept the (public API method) signature, as it seems to be what PennyLane expects (since it does pass in rotations as a kwarg).


Hotfixes for TestProxy LiveTesting

@rakshith91

@rakshith91 There is a problem with indentation in file eng/pipelines/templates/stages/archetype-sdk-tests.yml, there should be a space in front. Maybe it should be changed as shown below.
The same problem also exists in file eng/pipelines/templates/jobs/live.tests.yml.  Please take a look.

I found this error when I ran the pipeline.

@v-xuto this was not merged by me, apologies for the issues. Thanks @rakshith91 for reverting the original merge.


Please invite me to the Community Organization

Name
followDev
Discord Username (if applicable) E.g John#412
No response
Questions & suggestions?
I would like to be part of this awesome community.

Thanks.


ERROR: http: server gave HTTP response to HTTPS client

Describe the bug
running 'adguardhome-sync run' gives me the error "http: server gave HTTP response to HTTPS client"
To Reproduce
Steps to reproduce the behavior:

What version of AdGuardHome sync used? adguardhome-sync version devel
What version of AdGuardHome us used? Version:
v0.107.22
How does the configuration look? out of the box (https is not turned on)
What is the error message?
ERROR   sync    sync/sync.go:104        Error getting origin status     {"from": "<IP_ADDRESS>:3000", "error": ": Get "https://<IP_ADDRESS>:3000/control/status\": http: server gave HTTP response to HTTPS client"}

Expected behavior
that the sync would work?
Log Files
$ adguardhome-sync run
INFO    sync    sync/sync.go:29 AdGuardHome sync        {"version": "devel", "build": "N/A"}
INFO    sync    sync/http.go:52 Starting API server     {"port": 8080}
INFO    sync    sync/sync.go:65 Running sync on startup
DEBUG   client  client/client.go:139    do get  {"host": "<IP_ADDRESS>:3000", "method": "GET", "path": "status", "username": "admin"}
2023-01-29T02:26:56.756-0600
DEBUG   client  client/client.go:148    error in do get {"host": "<IP_ADDRESS>:3000", "method": "GET", "path": "status", "username": "admin", "status": 0, "body": "", "error": "Get "https://<IP_ADDRESS>:3000/control/status\": http: server gave HTTP response to HTTPS client"}
ERROR   sync    sync/sync.go:104        Error getting origin status     {"from": "<IP_ADDRESS>:3000", "error": ": Get "https://<IP_ADDRESS>:3000/control/status\": http: server gave HTTP response to HTTPS client"}
github.com/bakito/adguardhome-sync/pkg/sync.(*worker).sync
/root/go/pkg/mod/github.com/bakito/adguardhome-sync@v0.4.11/pkg/sync/sync.go:104
github.com/bakito/adguardhome-sync/pkg/sync.Sync.func3
/root/go/pkg/mod/github.com/bakito/adguardhome-sync@v0.4.11/pkg/sync/sync.go:66
Additional context
Add any other context about the problem here.

Please provide the adguardhome-sync configuration you applied

@geoffrothman I can not provide you any support if you do not provide your configuration values.


Updates to Service Explorer, show child services

This PR changes the service explore fields being displayed and adds showing the child services for services that have them.
PT: https://www.pivotaltracker.com/story/show/137174911
Jira: https://patternfly.atlassian.net/browse/CFUX-294
Finishes 137174911

@beanh66 @serenamarie125 @chriskacerguis
@miq-bot add_label euwe/no
@miq-bot add_label ux

Thanks @jeff-phillips-18 the changes look great!

@jeff-phillips-18 some of those screens still show the icon near Retires ... but other show Never.  Can you confirm that the icon was replaced with Never ?

@serenamarie125 sorry, those were older screenshots. I've updated them.

Updated per comments.
And yes, this is to follow the latest design from https://github.com/ManageIQ/manageiq-design/blob/master/UX/ui-service/MyServices/design.md.
I can add the link to Jira and PT.

@AllenBW can you check the update?

@chriskacerguis THIS THING IS 🌮

@AllenBW can you change your approval status?


[DB Schema] Create Module Model

Notion ticket link
API Routes Module
Implementation description

Builds API routes for the Module table

Steps to test

Investigate the module.ts file and try writing some mongoose tests in the dbTest file if you would like

What should reviewers focus on?

Ensuring that the module.ts file follows best practice (https://mongoosejs.com/docs/schematypes.html) and works as intended

Checklist

[x] My PR name is descriptive and in imperative tense
[x] My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
[x] I have run the appropriate linter(s)
[x] I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

Can you test this model by creating an instance of it inside the dbTest endpoint

Yessir:

Also got rid of the explicit default empty array


Fixed #26118 - Added 'is' comparison operator to if tag.

merged in c00ae7f58c34962ed6dbec4eb8aaf747da59ed2f, thanks!


telegraf: 1.29.1 -> 1.29.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/telegraf/versions.
meta.description for telegraf is: The plugin-driven server agent for collecting & reporting metrics
meta.homepage for telegraf is: https://www.influxdata.com/time-series-platform/telegraf/
meta.changelog for telegraf is: https://github.com/influxdata/telegraf/blob/v1.29.2/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.29.2 with grep in /nix/store/68if9hw5zglslmf4kk55mrnv7ij1gv95-telegraf-1.29.2
found 1.29.2 in filename of file in /nix/store/68if9hw5zglslmf4kk55mrnv7ij1gv95-telegraf-1.29.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
telegraf

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/68if9hw5zglslmf4kk55mrnv7ij1gv95-telegraf-1.29.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A telegraf https://github.com/r-ryantm/nixpkgs/archive/b6ef6a50160eae4dfac1697708611f18f142d827.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b6ef6a50160eae4dfac1697708611f18f142d827#telegraf

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/68if9hw5zglslmf4kk55mrnv7ij1gv95-telegraf-1.29.2
ls -la /nix/store/68if9hw5zglslmf4kk55mrnv7ij1gv95-telegraf-1.29.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

telegraf

Maintainer pings
cc @Mic92 @roblabla @timstott @zowoq for testing.

@ofborg eval

@ofborg eval


fn: error image is now deprecated

Please use fn-test-utils instead for testing.

Partially fixes #656


use exec instead of matchAll while keeping structure

Babylon native's js engine doesn't support matchAll (an ES2020 feature).
Instead, I am recreating the same structure using .exec.
Performance-wise it is on-par and even faster (at least on my benchmarks. See here - https://jsbench.me/wblj4eghw5/1)

Snapshot stored with reference name:
refs/pull/13986/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13986/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13986/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13986/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13986/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13986/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13986/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13986/merge#BCU1XR#0


Makes TextBoundary and its subclasses public

Make it public so it can be reused in SelectableRegion
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

blocked on https://github.com/flutter/engine/pull/36176/files


Provisioner type - powershell-azure not working

Is Provisioner type - powershell-azure even supported? I can see it in examples, one of the example is https://github.com/MSOpenTech/packer-azure/blob/master/config_examples/packer-azure_Win2012r2_inet.json but when i execute it, it simply doesn't work.
As per code, powershell-azure is not even supported. When i check the following file, it doesn't even import powershell/communicator.go which indicates that only provisioner type azure-custom-script-extension is supported.
https://github.com/MSOpenTech/packer-azure/blob/master/packer/builder/azure/driver_restapi/targets/win/step_set_provision_infrastructure.go
Am i missing something? I am using the current release version.

I'll take a look. A lot has changed in the code in the past few weeks. We removed the powershell builder in favor of the REST API builder. You may very well be right that the powershell provisioner is no longer functional. Do you have a use case that requires this provisioner over the custom-script provisioner?

No, We don't have a use case but the key will be #54 to work.
Poweshell provisioner supports folder upload but custom-script-extension doesn't

should we be using packer-community/packer-windows-plugins for windows images? That may fix a lot of things at the same time?
cc: @ahmetalpbalkan @Snesha

The ARM builder has full support for WinRM communicator and the PowerShell provisioner.


[Intro to iOS] Project 1 - Review my app

my app is complete with the features mentioned in the video. please review @codepathreview.

Hi Sree,
Your submission is missing a walkthrough gif. Can you upload the GIF, link in the README, and then let us know via this issue?
Thanks,
-sumeet
p.s. - don't worry about xcode8 for now. it's still in beta and we'll be teaching the class using xcode 7.3

Hi Sumeet,
I created video using licecap. committed to this repo. updated video link in the README.
Sure, let us stick to Xcode v7.3.1.
Thanks
Sree


Add transfer operator

Initial implementation of https://github.com/gaioguys/GAIO.jl/pull/32#issuecomment-663098767.
This PR is in a very early stage. (I'll add a TODO list later here.)
The code
using StaticArrays

function logistic_map_test()
    g = boxmap(x -> SVector(4*x[1]*(1-x[1])), LinRange(-0.99, 0.99, 15))
    boxset = RegularPartition(Box((0.5,), (0.5,)), 20)[:]

    gstar = TransferOperator(g, boxset)
    boxfun = eigs(gstar)[2][1]

    plot(abs ∘ boxfun)
end

produces the following output:

Awesome. I am going to put in some detailed comments.

Is this one ready for merging?

This PR depends on #34, so we should merge that one first. (We can merge #34 once someone takes quick look at it and approves it.)
Other than that, the main thing that is missing here is some tests.
I'd also be ok with quick comparison with the output of MATLAB's GAIO. Just to check that nothing major is wrong with this implementation :)
On another note: I'm sorry for being so inactive here (master's thesis has been keeping me busy). In about a month, I'll hopefully have the time to work on this project again.

I tested this against MATLAB's GAIO with the logistic map and the Lorenz system - it gives the same eigenvalues and vectors up to 1e-12 accuracy. I guess this can be merged.
No problem - thanks for still having an eye on this.


Health Bar

So, I finally got into my main world to try the new update and it seems that you're messing with health in a way that it removes all bonus health gained by other mods, and I cant seem to get it back.  My android powers are trying to regain the lost health and I keep getting the "Damaged" screen shake every so often.
Ah well. Going to have to revert.

I seem to found the reason for my health-bar problem. I also experienced that my maximum hearts are reduced after I used the creative gamemode for a second.
Anyway I updated to NuclearCraft-1.9d and it happened. In the mean time the update was changed to a BETA release (probably because of the upcoming 1.10 update). I got this health problem.
After an "update" or exchange was showed me on Curse, I updated the the "NuclearCraft-1.9d" to NuclearCraft-1.9d files, because I presumed a bugfix...
Sadly with this update I wasn't able to load my world anymore because a (probably config related) crash. I noticed the change in FILE-CHANGELOGS, the arrangement with the files and my mistake and I downgraded back to NuclearCraft-1.9c. But unfortunately didn't fixed it the health problem.
2 Hearts (4 half-hearts) are gone. I had to fix it with the NBT-Explorer (set (level.dat/Data/Player/HealF) from 16 to 20 again.).
I hope this helps a bit.

Well, what you experienced was having a ton of radiation. as far as I know.  Or just maybe my mods counteract it. Anyway, I didnt have to do anything special other than downgrade and get all my hearts back. (like 89 hearts, tinker's and a heart mod)

Hopefully this helps - here's a stable build without the radiation mechanics: https://minecraft.curseforge.com/projects/nuclearcraft-mod/files/2330045
This is certainly recommended for those who simply want to update the mod for their long-term worlds, but remember to turn off all Fission Reactors before updating!


Information request regarding assembler

Hi,
I would like to ask if STRONG can be used with the high-resolution graph (HRG) obtained specifically from metaSPAdes, or if it is possible to use also other assemblers such as megahit.
Thanks in advance for the information.
Sincerely
Stefano

Hi Stefano
I'm currently unaware of other sources of high resolution assembly graph. Since STRONG is basically just a snakemake, if you generate yourself a high resolution graph and the corresponding normal assembly graph, by naming them and placing them in the right folder you could have STRONG use them.
So at the moment only SPAdes is supported, I think we are open to implementing alternate assembly options if they can propose similar high resolution graph. Do you have source for megahit based HRG?
Best,
Seb


point templates at canary tag

Update templates to point at the canary tag to allow for installing templates for llm.

Won't this make all installs of the templates point to canary? (even if you want to target main?)

Only Canary Spin installs templates from main

@karthik2804 i am not sure i see where that happens -- do you mind pointing it out? Are we checking the semver of Spin in the templates command to see if it is beyond a release somewhere?

@kate-goldenring . When installing templates, spin looks for a tag spin/templates/<major>.<minor> first, if available - it installs that else it picks up templates from main.

Thanks @karthik2804. So sounds like we want to add to our release checklist to point templates to canary tags. We'd want to do the same for the go template too. Is there a way to do this toggling smoothly?

The canary/main distinction confuses the heck out of me here.  Okay, so I grab canary Spin.  Then I try to install templates, which finds no corresponding version tag, so gets the templates from main.  But the templates on main point to canary, which is who knows what - maybe the same commit as the templates, maybe not - and is in no way guaranteed to have any relation to the canary of Spin that I have because time has elapsed.
I'd prefer to just refer to main (with "canary" being the latest binary built from main, not a separate tag).  Or if we want to have a way to stamp a consistent set, then tag that set, or fetch templates via a commit SHA instead of main (Spin knows the commit SHA it was built from), and have the the templates refer to SDKs at the corresponding SHA.

Also note if we do this then we need to fix the template makefile before the next minor release as per https://github.com/fermyon/spin/issues/1733#issuecomment-1716754811

@karthik2804 I think we'll want to nail down a general approach in https://github.com/fermyon/spin/issues/1733 as opposed to the changes in this PR.  Shall we close?

@vdice I agree - we can close this


Duplicate acl_ before update

This is necessary in chef 14 since default values are frozen
Change-Id: I352a3449abb0135b65529dc68f5211213b60b7e6

Thanks @kamaradclimber! Does this have any negative impact on Chef 13?
I realize that’ll be EOL in a month, but it’s nice to preserve functionality if we can, and call it out if we cannot.

I have not tested with chef 13 but i dont see any potential breakage since
it is simply a duplication of an object.
On Mon, Mar 9, 2020, 19:46 Jeff Byrnes<EMAIL_ADDRESS>wrote:

Thanks @kamaradclimber https://github.com/kamaradclimber! Does this
have any negative impact on Chef 13?
I realize that’ll be EOL in a month, but it’s nice to preserve
functionality if we can, and call it out if we cannot.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/evertrue/zookeeper-cookbook/pull/222?email_source=notifications&email_token=AAD254NG7KGNKUEXYLCE4XLRGVBTDA5CNFSM4LEN72B2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOISXYI#issuecomment-596716513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD254OI7RY4AMO3IXOLEALRGVBTDANCNFSM4LEN72BQ
.

I don't think there is incompatibility with old chef versions


Add note about nuspec elements being case sensitive

Address https://github.com/NuGet/docs.microsoft.com-nuget/issues/2039

Docs Build status updates of commit 1f5e677:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

Line 0, Column 0: [Error-committish-not-found] Can't find branch, tag, or commit 'master' for repo https://github.com/Microsoft/templates.docs.msft.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 8a5a156:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/nuspec.md
:bulb:Suggestion
View
Details

docs/reference/nuspec.md

Line 490, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 501, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 512, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 525, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 538, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 570, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 583, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 598, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 611, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 624, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 637, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 650, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 661, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 699, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 707, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 718, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Can't download small version of dataset...

Hello. Thanks for your work. I have applied for the dataset through online form and got a readme file to download the dataset. But i can't figure out how to download the small version (5gb) of the dataset.
Looking forward for the response. Thank you!

Hi, you can simply run this command to download the small version, as written in the README.
rsync -rhz --info=progress2 --ignore-existing rsync://vandaldata.polito.it/sf_xl/small .

Thanks for your help. I am successfully done with the training by changing some parameters as per my requirement. But my final goal in a little bit different.
I am mainly working with the 3D lidar. I convert the lidar point cloud to lidar intensity image or lidar range image (one of these). So my questions is: can i modify and train this network model for lidar images?

Hi, you can simply use convert those images to RGB and the code will work, as long as the images/crops have the required metadata in the filename (check this issue).
I'm not sure if that will achieve good results though


[io.mechanics] allow disabling output in hdf5 during simulation

This is a python patch which allows to disable the hdf5 records during simulation if output_frequency==0. In my case I use the io.mechanics run() function but I retrieve all data during simulation with a controller.
PATCH-output_frequency-disable.txt

The patch has been applied in ff5a1770ca981bf16f681c083417d871e704f4f7
Thanks @batzkass

I close the issue. Feel free to reopen if needed.


ask about automatic services startup after setup

let's improve setup process and ask about automatic start after successful setup.
also in scope of this ticket let's remove start.sh and stop.sh at all

done: https://github.com/zebrunner/zebrunner/commit/6c4e9faa2d5a0535618ab3d4234680db136adccd
testing scenario:
after cloning from develop branch run setup
at the end it will ask "Do you want to start service y/n [y]: "
if you answer "y" services should be started

In the case if you answer "y"
Actual result: services is not started

Verified.
Opened new ticket https://github.com/zebrunner/zebrunner/issues/310


Catch errors too

I suggest you do
} catch (Exception | NoClassDefFoundError e) {
instead, so we don't catch things where it would be better to die.

Moved catching to make sure that we log something where thread that gets config will die anyway.


Keeping the [disabled] attribute

If is on the option tag, should go also on the li

resolved using classes


Why do I get stable back on the list when I manually disabled it

Details
I disabled "stable" however when I build my project I get the "stable" version back.
The base url moderngl.readthedocs.io gets redirected to stable however I set it to redirect to the "latest". My settings only work for a couple of minutes.

tried to make "stable" private
tried to disable "stable" (multiple times)

Project URL: https://moderngl.readthedocs.io/en/latest/
Build URL (if applicable): https://readthedocs.org/projects/moderngl/builds/5464362/
Read the Docs username (if applicable): cprogrammer1994

Expected Result
I don't want the "stable" version at all.
The moderngl.readthedocs.io should redirect to the latest.
Actual Result
sometimes (probably after I push changes to master then I build the docs) a version stable appears and moderngl.readthedocs.io gets redirected to stable.

I think that there is no way to disable stable version (there is a PR that will probably enable this, but it's a work in progress and needs review #2908)
On the other hand, when I go to moderngl.readthedocs.io it redirects me to https://moderngl.readthedocs.io/en/latest/, which I understand that it's what you want.

I'm closing this one because #2908


sample dockercompose

please provide sample Dockercompose file

+1

+1

Sorry I've missed this ticket.
We've added a Dockerfile in #5


Broken Umlauts and other non-ascii chars in output

Describe the bug
running eledo.exe [programname] shows UAC and runs program, but shows "broken" line-drawing characters, umlauts, etc
Environment
OS: Microsoft Windows [Version 10.0.19041.264]
Term: wezterm 20200517-122836-92c201c6-6-g7ddff705
To Reproduce
Run whoami /groups on German Windows
Expected behavior
Captured program output is displayed correctly

I pushed a commit that may help, but I'm not totally convinced--it feels too easy :-p
I think https://github.com/microsoft/terminal/issues/4551 may be related.
Is your system set to use a German code page rather than UTF-8?  (I'm not asking you to change anything, just want to understand the environment)

Unfortunately the fix does not solve the problem.
chcp shows
Aktive Codepage: 850.

I think I may have it resolved in the latest commit; it seems to work locally when I try changing my code page to 850 and do some simple input/output experiments.

I can confirm my umlauts work again :-) The last commit fixes the problem. Thank your very much!