Refactor SolveExercise page to use Skeleton Screens when loading data

Can be implemented after https://github.com/FrontEnd-ro/frontend.ro/issues/337

Done!


Default code highlighting in Markdown

When there is no specified language for a code block, the colors default to white. This should be changed to a bright block of a different color. Screenshot included.

Thanks for the request!
This is part of update v2.9.0.

Cool, thank you!


Problem streaming with docker.

Im running a poc that consists to stream IP cameras to KVS.
So in my first tests I was streaming using the docker image inside /docker_native_scripts that I build myself via Dockerfile.
As I ran against a few problems I was instructed to use the latest docker image<PHONE_NUMBER>75.dkr.ecr.us-west-2.amazonaws.com/kinesis-video-producer-sdk-cpp-amazon-linux:latest).
So now Im streaming using this latest image but I cant see anything in the KVS media playback at AWS console. All I got is the message "No media found. Verify that you are streaming to us-east-1 and the fragment timecodes are correct, or download the Kinesis Video Streams Producer SDK to set up your device."
Earlier when I was using the docker container that I build from the scripts I used to start the container (and the streaming) with the following command.
docker run -it <IMAGE_ID> <AWS_ACCESS_KEY_ID> <AWS_SECRET_ACCESS_KEY> <RTSP_URL> <STREAM_NAME>
Now with the latest docker image Im using the followings commands, since Im not able to run it like I was running before.
sudo docker run -it --network="host" --device=/dev/video0<PHONE_NUMBER>75.dkr.ecr.us-west-2.amazonaws.com/kinesis-video-producer-sdk-cpp-amazon-linux:latest /bin/bash

gst-launch-1.0 -v rtspsrc location=<rtsp-url> short-header=TRUE ! rtph264depay ! video/x-h264, format=avc,alignment=au ! h264parse ! kvssink stream-name=Channel4 storage-size=128 access-key=<access-key> secret-key=<secret-key> aws-region="us-east-1"

The logs appears to be all ok and the region is being printed correct in the logs as well.
Am I doing something wrong?

Hello @pdalmaso1 ,
Can you share the logs?

[logsKVS.txt]
Is there any way to run the latest version of the docker and start streaming?? Should I edit the docker to do so?

logsKVS.txt
Is there any way I can run the latest container and already start streaming? Maybe should I edit the container to do so?

Hi @pdalmaso1 ,
Your log looks correct and everything is working as expected. Can you refresh the AWS Console page and check the media player again while you are streaming? You might need to click the skip button on the right bottom. Please note the player would be freeze while you are not staying on the page and it will not automatically restart if it already get into status of "No media found. Verify that you are streaming to us-east-1 and the fragment timecodes are correct, or download the Kinesis Video Streams Producer SDK to set up your device.".
You can also check the playback using HLS https://aws-samples.github.io/amazon-kinesis-video-streams-media-viewer/. Thanks.


[Automation] Generate Fluent Lite from cognitiveservices#package-2021-04

[Automation] Generate Fluent Lite from cognitiveservices#package-2021-04

/check-enforcer override

ci.data fails, but not touched that.


Hipchat adaptor for Hubot is out of date with the installed version of node.js

Issue from support email

Done in #116.


[API breaking] Use polymorphic value type holder for manifolds and constraint sets

This PR:

removes use of shared_ptr for manifold and constraint set types, switching to jbcoe/value_types's polymorphic template class everywhere as a holder with value semantics
enhances the library's ergonomics by avoiding having to call std::make_shared<T>(...) all the time, see changed C++ examples
updates the examples and tests accordingly
adds a util header proxsuite-nlp/python/polymorphic.hpp for dealing with values/references of polymorphic<T>, register conversions, etc
updates Python bindings for manifolds and constraint sets accordingly
adds a PolymorphicVisitor to the Python bindings which registers a type Y as convertible to the template argument polymorphic<X>
splits up the big constraint instantiation header constraints.txx: closes #87

Known issues

returning a polymorphic<X> by value does not check whether the underlying object is a boost::python::wrapper<U> for some type U which inherits from X
this PR breaks the C++ API of proxsuite-nlp

Background
This is the first in a series of PR which will cut down on our overuse of shared_ptr across both proxsuite-nlp and aligator and replace it with a holder for polymorphic objects with value semantics.
The drawbacks to the current approach with shared_ptr was:

no real copy constructor for shared_ptr<T>, except if we add virtual clone member functions implemented in all derived classes
calls to members did not satisfy const-correctness (not a problem for manifolds as they so far have no non-const member functions, but a problem for namely BoxConstraint)
shared mutable state

The polymorphic value type was described by Jonathan Coe: https://www.youtube.com/watch?v=sjLRX4WMvlU. It allows composite classes (with polymorphic data members) to behave well using value-semantics all the way down (with const-correctness).
Why not unique_ptr ?
unique_ptr completely removes any ability to copy the underlying object (except if a virtual clone() was implemented) and disables default copy ctor/operators for all composite classes using it.
Many thanks to @edantec for his contribution.

The last thing to do before merging, which is very relevant for downstream integration into aligator, is being able to extend interfaces from Python. Right now, inheriting from bp::wrapper<PythonClass> and using the current macros doesn't seem to work anymore. We need to:

[ ] add this case to the test
[ ] find a way to fix this. AFAIK, for shared_ptr<T>, Boost.Python handles calling virtual methods implemented in Python by fetching a pointer to the Python object that is hidden in the deleter

@jorisv @edantec I think Windows is finally fixed (my god...)

While discussing with @jorisv, we found that:

passing the wrapper class to a function/ctor taking polymorphic<T> does not properly copy the object
by properly copying, we mean that the reference to the initial object is not properly handled, there's a missing incref somewhere
in Boost.Python, the converter class shared_ptr_from_python, which is registered in the class_metadata.hpp header, takes the source PyObject*, borrows it with bp::borrowed (which casts it to a pointer to the special type bp::detail::borrowed<PyObject> (see boost/python/detail/borrowed_ptr.hpp)), and shoves it into a bp::handle<> object, the constructor of which increfs the pointer to borrowed and upcasts it back to PyObject to store it.

Are we ready to merge this yet?

Okay, we will be merging this today. This is not the perfect solution imo, I'd be willing to look at other types of type erasure in the future for the definitive API (and rip out the OOP class hierarchies here and in aligator, downstream).
Having value semantics is already good enough though and we can make progress on other fronts.


Improve workspace documentation

Changes
Some rewording and clarifications.
The user does not have to create a "subPath" before declaring it on e.g. a new workspace volume,
as can be seen in the subPath-example: https://github.com/tektoncd/pipeline/blob/master/examples/v1beta1/pipelineruns/pipelinerun-using-different-subpaths-of-workspace.yaml
/kind documentation

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
NONE

/lgtm


Linux x64 riscv-none-elf-gdb is built without ELF support

Linux x64 riscv-none-elf-gdb fails to load symbols from an ELF file.
Release 13.2.0-1 Linux x64:
➜  test-gdb  % riscv-none-elf-gdb --version
GNU gdb (xPack GNU RISC-V Embedded GCC x86_64) 13.2
Copyright (C) 2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
➜  test-gdb  % riscv-none-elf-gdb -q bme_sensor_oled.elf
Reading symbols from bme_sensor_oled.elf...
I'm sorry, Dave, I can't do that.  Symbol format `elf32-littleriscv' unknown.
(gdb) q
➜  test-gdb  %

Windows version works fine
X:\>xpack-riscv-none-elf-gcc-13.2.0-1\bin\riscv-none-elf-gdb.exe --version
GNU gdb (xPack GNU RISC-V Embedded GCC x86_64) 13.2
Copyright (C) 2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

X:\>xpack-riscv-none-elf-gcc-13.2.0-1\bin\riscv-none-elf-gdb.exe -q bme_sensor_oled.elf
Reading symbols from bme_sensor_oled.elf...
(gdb) q

X:\>

Older 12.3.0-1 release doesn't work (GDB version there is also 13.2)
➜  test-gdb  % xpack-riscv-none-elf-gcc-12.3.0-1/bin/riscv-none-elf-gdb --version
GNU gdb (xPack GNU RISC-V Embedded GCC x86_64) 13.2
Copyright (C) 2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
➜  test-gdb  % xpack-riscv-none-elf-gcc-12.3.0-1/bin/riscv-none-elf-gdb -q bme_sensor_oled.elf
Reading symbols from bme_sensor_oled.elf...
I'm sorry, Dave, I can't do that.  Symbol format `elf32-littleriscv' unknown.
(gdb) q
➜  test-gdb  %

11.3.0-1 release works fine (GDB version 12.1):
➜  test-gdb  % xpack-riscv-none-elf-gcc-11.3.0-1/bin/riscv-none-elf-gdb --version
GNU gdb (xPack GNU RISC-V Embedded GCC x86_64) 12.1
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
➜  test-gdb  % xpack-riscv-none-elf-gcc-11.3.0-1/bin/riscv-none-elf-gdb -q bme_sensor_oled.elf
Reading symbols from bme_sensor_oled.elf...
(gdb) q
➜  test-gdb  %

Linux x64 riscv-none-embed-gdb (release 10.2.0-1.2) also works fine.
I checked build logs in GH Actions here and I believe that this failing check is responsible for the ELF format support based on info from this old bug report:
2023-08-24T16:44:57.6812604Z mkdir -p -- ./gdb
2023-08-24T16:44:57.6831647Z Configuring in ./gdb
...
2023-08-24T16:45:05.2050580Z checking for ELF support in BFD... no
...

From gdb/configure script in GDB source tree - it tries to compile a file with "bfd.h" and "elf-bfd.h" headers, so I would suspect missing Linux-specific dependencies included from those headers
# Add ELF support to GDB, but only if BFD includes ELF support.

  OLD_CFLAGS=$CFLAGS
  OLD_LDFLAGS=$LDFLAGS
  OLD_LIBS=$LIBS
  OLD_CC=$CC
  # Put the old CFLAGS/LDFLAGS last, in case the user's (C|LD)FLAGS
  # points somewhere with bfd, with -I/foo/lib and -L/foo/lib.  We
  # always want our bfd.
  CFLAGS="-I${srcdir}/../include -I../bfd -I${srcdir}/../bfd $CFLAGS"
  LDFLAGS="-L../bfd -L../libiberty"
  intl=`echo $LIBINTL | sed 's,${top_builddir}/,,g'`
  LIBS="-lbfd -liberty $intl $LIBS"
  CC="./libtool --quiet --mode=link $CC"
  { $as_echo "$as_me:${as_lineno-$LINENO}: checking for ELF support in BFD" >&5
$as_echo_n "checking for ELF support in BFD... " >&6; }
if ${gdb_cv_var_elf+:} false; then :
  $as_echo_n "(cached) " >&6
else
  cat confdefs.h - <<_ACEOF >conftest.$ac_ext
/* end confdefs.h.  */
#include <stdlib.h>
	   #include <string.h>
	   #include "bfd.h"
	   #include "elf-bfd.h"
int
main ()
{
return bfd_get_elf_phdr_upper_bound (NULL);

  ;
  return 0;
}
_ACEOF
if ac_fn_c_try_link "$LINENO"; then :
  gdb_cv_var_elf=yes
else
  gdb_cv_var_elf=no

fi
rm -f core conftest.err conftest.$ac_objext \
    conftest$ac_exeext conftest.$ac_ext

fi
{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $gdb_cv_var_elf" >&5
$as_echo "$gdb_cv_var_elf" >&6; }
  CC=$OLD_CC
  CFLAGS=$OLD_CFLAGS
  LDFLAGS=$OLD_LDFLAGS
  LIBS=$OLD_LIBS
if test "$gdb_cv_var_elf" = yes; then
  CONFIG_OBS="$CONFIG_OBS elfread.o stap-probe.o dtrace-probe.o \
		gcore-elf.o elf-none-tdep.o"

$as_echo "#define HAVE_ELF 1" >>confdefs.h

Oops!
Thank you for your detailed bug report, I'll take a look during the next few days.

Oh good, I was about to report something similar.
As reported by @zador-blood-stained , 12.3.0-1 was the first version that broke. 12.2.0-3 works just fine.

Both 12.3 and 13.2 use the same build scripts, so will be fixed together.
I added a notice in the release notes to mention this issue.

I confirm the issue, it affects the arm toolchains too.
I added new tests when building gdb (#23), to catch this issue.
Builds with new releases for all toolchains are planned for the next few days.

I just published a new 12.3 release. Please check if it functional.
13.2 will be updated soon.

Works fine for me
➜  test-gdb  % xpack-riscv-none-elf-gcc-12.3.0-2/bin/riscv-none-elf-gdb -q bme_sensor_oled.elf
Reading symbols from bme_sensor_oled.elf...
(gdb) disassemble main
Dump of assembler code for function main:
   0x000011c2 <+0>:     add     sp,sp,-16
   0x000011c4 <+2>:     sw      ra,12(sp)
   0x000011c6 <+4>:     sw      s0,8(sp)
   0x000011c8 <+6>:     li      a0,72
   0x000011cc <+10>:    auipc   ra,0x1ffff
...

Great, thank you for confirming.
I'll proceed with 13.2.

13.2 was published.


Enable Autopush and Staging tests for E2E test apps.

Add staging and autopush runs of the test apps every 4 hours.
#no-changelog

LGTM - I would double check it actually updates the dev environment. We might want to add a log here so we know when we've set a value for the env: https://github.com/firebase/firebase-ios-sdk/blob/master/FirebaseSessions/Sources/ApplicationInfo.swift#L103

Environment is a part of the event and we list the environment value in the DevEventConsoleLogger. Do you think it makes sense to add another log statement in ApplicationInfo?


Solved typo in inferring types documentation

Description
Documentation error on Client/Inferring types:
https://the-guild.dev/openapi/fets/client/inferring-schema-types
Example:
import type { NormalizeOAS, OASRequestParameters } from 'fets'
import type oas from './openapi'

type AddUserInput = OASRequestParams<NormalizeOAS<typeof oas>, '/user', 'POST'>

Should be:
import type { NormalizeOAS, OASRequestParams} from 'fets'
import type oas from './openapi'

type AddUserInput = OASRequestParams<NormalizeOAS<typeof oas>, '/user', 'POST'>

Related #1784

Type of change
Please delete options that are not relevant.

[x] This change requires a documentation update

Checklist:

[x] I have followed the
CONTRIBUTING doc and the
style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests and linter rules pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Thanks 🙏


refactor(sync): sync from orchestrator

migrate to go modules

issue: openark/orchestrator#1355
cc @shlomi-noach


Error from server (NotFound): pods "utils" not found (Blockchain network install fails)

When trying to install the blockchain network I run into the following error (on macOS):
Using this tutorial in step 2:
./create/create_blockchain.sh
Creating Services for blockchain network
Running: kubectl create -f /ibm-container-service/cs-offerings/scripts/create/../../kube-configs/blockchain-services-free.yaml
error: Unexpected args: [- ibm-container-service/cs-offerings/scripts/create/../../kube-configs/blockchain-services-free.yaml]
See 'kubectl create -h' for help and examples.
Creating new Deployment
Running: kubectl create -f /ibm-container-service/cs-offerings/scripts/create/../../kube-configs/blockchain.yaml
error: Unexpected args: [- ibm-container-service/cs-offerings/scripts/create/../../kube-configs/blockchain.yaml]
See 'kubectl create -h' for help and examples.
Checking if all deployments are ready
No resources found.
Error from server (NotFound): pods "utils" not found

I ran the export command before following this step. But I think the config is not used?

@georg90 can you provide us with following details?

Operating System where the kubectl client is running. I suspect this is a windows system and is not interpreting the kubectl create -f command correctly.
The output of kubectl version

As i wrote above, I am using macOS.
This is the output:
kubectl version
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.3", GitCommit:"f0efb3cb883751c5ffdbe6d515f3cb4fbe7b7acd", GitTreeState:"clean", BuildDate:"2017-11-09T07:27:47Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"7+", GitVersion:"v1.7.4-1+1540c973d4ff9d", GitCommit:"1540c973d4ff9da2d6a204a7709084488a459ed4", GitTreeState:"clean", BuildDate:"2017-10-06T15:08:43Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

@georg90 can you look into ./create/create_blockchain.sh and run the commands manually?
Let's try the first command kubectl create -f /ibm-container-service/cs-offerings/scripts/create/../../kube-configs/blockchain-services-free.yaml.. Does it work or errors out?

This works:
kubectl create -f ../../kube-configs/blockchain-services-free.yaml
service "blockchain-ca" created
service "blockchain-orderer" created
service "blockchain-org1peer1" created
service "blockchain-org2peer1" created

It will not work because of the errors in creating services and other components..
Can you try to run . create_blockchain.sh or source create_blockchain.sh or sh create_blockchain.sh?

same result. I think it has to do with the path settings, they might be off for macOS vs. linux structure? Since the script fails but the actual command works

Looks like it is not interpreting https://github.com/IBM-Blockchain/ibm-container-service/blob/master/cs-offerings/scripts/create/create_blockchain.sh#L42 line correctly.
Can you add echo ${KUBECONFIG_FOLDER} right above that line? for some reason it is replacing -f /ibm-container-service in the command with - ibm-container-service

Output is as expected:
ibm-container-service/cs-offerings/scripts/create/../../kube-configs

@georg90 I don't think the output is as expected.
It is missing the / before ibm-container-service

Sorry, that's an error on my side, the complete path hast a / in front of the path

Found the problem! My path included a -  - so the complete path was /somwhere/some - thing/ibm.. This leads to the error.
Thank you @mrshah-at-ibm for your help! This can be closed


Build monorepo PRs in manyrepos

We want contributors to send pull requests agains the monorepo. The monorepo doesn't build the sub projects - that always happens in the manyrepos.
We therefore need a mechanism to "copy" monorepo pull requests into manyrepo pull requests. This would have to be done with a separate "agent" machine GitHub user. It could be a microservice somewhere that:

Has a webhook that listens for new PRs in the monorepo
Has a GitHub machine user that pulls PRs into a new branch in its own clone of the monorepo
Pushes the PR branch in the monorepo clone to its own manyrepo forks (need to script how to push new branches to manyrepos)
Uses GitHub's pull request API to create a pull request in each manyrepo.
Reports the status back to the original PR using this API

There is now a Makefile in the root that Travis runs. This builds most of the sub projects, which should be good enough.


MVC6 Setup

I'm trying to get this working with MVC6, however, it's serving blank pages. I don't think the route handlers is being used. Please advise.

I haven't tried it in MVC6 yet but there have been some fundamental changes to the way routing works compared with how it works in MVC5.
There is some useful information at http://www.ryadel.com/en/custom-routing-method-names-in-asp-net-5-mvc-6/ which may provide a clue to how ControlleLess can be used with MVC6 (in particular the section titled Convention-Based Approach).
I'll need to set up a sample MVC6 project and give it a try to see if it works.

Development note: Another useful overview of MVC6 routing is http://stephenwalther.com/archive/2015/02/07/asp-net-5-deep-dive-routing

Thanks Brent, I've read this. I've come up with a solution but there is a problem with static files until they implement the ability to ignore routes again.

Hi Chris. Would it be possible to take a look at your solution? Do you have the app.UseStaticFiles(); statement in your Configure(...) method? Thanks.

I do.
`public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
{
loggerFactory.AddConsole(Configuration.GetSection("Logging"));
loggerFactory.AddDebug();
        if (env.IsDevelopment())
        {
            app.UseBrowserLink();
            app.UseDeveloperExceptionPage();
        }
        else
        {
            app.UseExceptionHandler("/Home/Error");
        }

        app.UseIISPlatformHandler();

        app.UseStaticFiles();
        app.UseDefaultFiles();
        //app.UseMiddleware<ServerFileMiddleware>();

        app.UseMvc(routes =>
        {
            //routes.MapRoute(
            //    name: "default",
            //    template: "{controller=Home}/{action=Index}/{id?}");

            routes.MapRoute("Wildcard Routing",
                    "{*anything}",
                    new { controller = "Wildcard", action = "Get" },
                    new { anything = @"^(.*)?$" });
        });
    }`

Thanks for the information. Let me take a look and see what I can do. My gut feeling is that we might need a new version of the ControllerLess code for ASP.NET 5 because of the changes in the way routing and controllers work.
The current ControllerLess code inherits from the System.Web.Mvc.Controller class which probably isn't correct for ASP.NET 5- I think it should probably be inheriting from Microsoft.AspNet.Mvc.Controller instead.

Just following up on this, I've finally had some time to take a closer look at ASP.NET 5/MVC6 and it looks like the routing mechanism has been overhauled. Additionally, as it's still at RC stage there isn't a great deal of documentation about how to hook into the routing of requests.
I'll leave this for now until ASP.NET 5 is released and more information is made available. This will be a new project to port this across to MVC6.


Convert AggregateReference by converters instead of custom code paths.

Closes #992

That's merged and polished now.


Restrict package restore to primary host only

Packages should only be restored by the primary host. Secondary hosts should reload the function using the restore artifacts created by the primary host once the restore operation is completed.

@fabiocav Is this already done?

No. We took care of some of the foundational pieces with the primary host identification and logging from primary, but this has not been done yet. This would likely need to be bundled with the work to cache compilation results and perform compilation on the primary host only.


PHP open tag suggestion in html script section

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

gitlens
eamodio
4.1.3

php-debug
felixfbecker
1.10.0

php-intellisense
felixfbecker
1.4.1

material-icon-theme
PKief
2.0.9

When I add to a script outside of php tags, I get suggestions to create one. In the example below, I go to the next line by pressing enter but instead of going to the next line it puts in an opening tag. This only happens inside .php files.

It looks like this comes from the php-intellisense extension, so you should ask on their repo. But I think it's doing basically the right thing by suggesting completions as you type. It's just that there aren't embedded JS completions available.


FW-4125 Recalculate Custom Sort Task

Description of Changes
Adds a recalculation task for actual recalculation.
Checklist

[ ] README / documentation has been updated if needed
[ ] PR title / commit messages contain Jira ticket number if applicable
[ ] Tests have been updated / created if applicable
[ ] Admin Panel has been updated if models have been added or modified
[ ] Migrations have been updated and committed if applicable

Reviewers Should Do The Following:

[ ] Review the code changes here
[ ] Pull the branch and test locally

Additional Notes
Note that the changes from this PR https://github.com/First-Peoples-Cultural-Council/fv-be/pull/104 are included

Closing this branch to start a new one now that the recalculate preview changes have been merged.


Error generating code. Check the Developer Console AND the backend logs for details. Feel free to open a Github issue.

in this pic is the back-end logs , what is happening

Please see README faq section for utf8 solution.

I meet this promblem too？ how can I sove it？


[Request] Some things with EnderZooItems

Why isn't there any integration with EnderZoo? The items and mob souls could be used as alternative items in the EnderIO Crafting Recipes.

you can add recipes by yourself if you wish so it's pretty easy


Avatar in UserAutoComplete

Proposed Changes

Fixes #8869
Added conditional check to ensure avatar is only rendered when image option is available.

@ohcnetwork/care-fe-code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[x] Prep screenshot or demo video for changelog entry, and attach it to issue.
[x] Request for Peer Reviews
[ ] Completion of QA

Summary by CodeRabbit

New Features

Enhanced user autocomplete functionality with profile picture display in dropdown options.
Improved layout of options in the autocomplete component to include images alongside labels.

Bug Fixes

Refined error handling for no results in the linked facility users component, improving user feedback.

Style

Updated layout and styling for better alignment and presentation of autocomplete options.

@Jacobjeevan the avatar is not available in the users dropdown

Opened a PR on Backend 👍  Right now, it only works onQuery (provided user has an avatar).

@Jacobjeevan PR looks good to me, can you clear the conflicts

@Jacobjeevan Do we need the changes in /cypress/ folder?

@bodhish It looks like pre-commit is sorting imports from cypress/* too, after the recent change in adding support for sorting imports. Merging this should solve it for other PRs too having the same issue.


Issues running Docking example, Front page Document mismatching Custom version

I was trying to run the layout with docking sample on
https://pthom.github.io/imgui_bundle/quickstart.html#_complex_layouts_with_docking_windows,
I couldn't get it running either via C++ or Python by following the instruction provided by the link.
In the Python attempt, the editor (VS Code) was prompting that hello_imgui could not be resolved from source, together with icons_fontawesome_6. Upon a bit of digging, imgui_bundle exports icons_fontawesome instead and fixing it is just a matter of find and replace.
However hello_imgui's issue go a bit deeper as it seems some of its content were not available. This is the error message.
Traceback (most recent call last):
  File "C:\Users\Whenson\src\project\dock-sample.py", line 28, in <module>
    class MyAppSettings:
  File "C:\Users\Whenson\src\project\dock-sample.py", line 29, in MyAppSettings
    motto: hello_imgui.InputTextData
AttributeError: module 'imgui_bundle._imgui_bundle.hello_imgui' has no attribute 'InputTextData'

The sample code seems to rely on quite a few occurrence of content within hello_imgui, so I couldn't just delete or replace it. I was not able to get it running due to all the seemingly missing content.
It does however run fine in the test engine example, so hello_imgui does have some working content.
As for the C++ attempt, in line 21 #include "demo_utils/api_demos.h" is completely missing and I couldn't build it.
I follow build instruction in the previous hello world example section so I might be following the wrong instruction. If that's the case, please point me to the correct instruction.
I am using Windows 10, Python 3.10, the imgui_bundle version is 1.3.0 installed via pip.
I am suspecting that hello_imgui submodule was not generated correctly during the pip install process.

Great! I was about to close the issue as I dig out a working example and was studying the sample now.
Digging through your extensive documents have me realized that the document the first link doesn't match what I was trying to run.
Though VS code is still throwing quite a bit of complaint regarding hello_imgui and some other modules such as imgui_md.
Thanks for illuminating my frustration and giving me a new link for me to start working with. I can't state how much I appreciate your meticulous documentation and effort to maintain this bundle, couldn't asked for more.


Broken stuff?

I'm lately trying to stay away from Pods. That's why I've wanted to download your project manually. You just need to drop SwiftLinkPreview folder into Xcode project (make sure to enable "Copy items if needed" and "Create groups"). There is no folder SwiftLinkPreview.
Also the Example Project is kinda empty. No SwiftLinkPreview files and doesn't build at all. I thought about missing pod install, but a podfile is missing also.

Hi @davidseek!
You are right about the  SwiftLinkPreview folder. Thanks for letting me know. This part was written before I port SLP to SPM. I'll fix that.
And have you read the Example's README.md? Since my project doesn't support only CocoaPods, but Carthage too, for instance, I preferred to leave the decision for the user (him/her)self to install it as wish. The issue on Example is that if you added SLP via drag and drop, the *.plists files would conflict. I'll also fix the Example issue too.
If you wanna try with pod, please create this Podfile:
# Uncomment the next line to define a global platform for your project
# platform :ios, '9.0'

target 'SwiftLinkPreviewExample' do
  # Comment the next line if you're not using Swift and don't want to use dynamic frameworks
  use_frameworks!

  # Pods for SwiftLinkPreviewExample

  pod 'Alamofire'
  pod 'SwiftyDrop'
  pod 'ImageSlideshow', '~> 1.0.0-beta.1'
  pod 'ImageSlideshow/Alamofire'

end


Why do I have a running Agent on WSL2, but the browser shows don't work?

Describe the bug
Why do I have a running Agent on WSL2, but the browser shows don't work?
 ◎ LOGS
   initializeClients
   []
   for
   trump

 ◎ LOGS
   client keys
   []

 ["◎ Visit the following URL to chat with your agents:"]

 ["◎ http://localhost:5173"]

 ["✓ REST API bound to <IP_ADDRESS>:3000. If running locally, access it at http://localhost:3000."]

To Reproduce
pnpm start --character="characters/trump.character.json"
Expected behavior
have normal website
Screenshots

Additional context

I believe you'll have to run the specific pnpm start:client to start the chat UI. pnpm start --character="characters/trump.character.json" won't start it automatically from what I can tell.

I can confirm that this is a BUG facing the same issue from a long time.

pnpm start:client

Yes, need run pnpm start:client in another terminal.

But @cxp-13 is the UI responding ? To me its not responding.

any clues on the above @cxp-13  @oxSaturn

any clues on the above @cxp-13 @oxSaturn @odilitime

May be the network connect time out, I alse meet the error. but it not be involving the agent self.


Send/Receive: Confetti not animated on some phones

Some users are not seeing the confetti animated when sending (Result screen) or receiving (Payment Received modal) a payment.

Likely fixed, closing for now.


Fix distance field doesn't show Pending when removing waypoint while offline

Details

When we remove the distance waypoint while offline, the distance field still shows the old amount instead of Pending. This PR fix it.
Fixed Issues

$ https://github.com/Expensify/App/issues/46952
$ https://github.com/Expensify/App/issues/43569
PROPOSAL:
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Same as QA Steps
QA Steps

Open a workspace chat.
Create a distance request with at least three waypoints.
Open the transaction thread.
Go offline
Press the Distance field
Remove one waypoint and save it
Verify the Distance field shows Pending...

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/1d8461c4-63e5-4e03-9481-2ea833bd169f

Android: mWeb Chrome

https://github.com/user-attachments/assets/2756305d-daff-4f16-b997-4aa338f707b8

iOS: Native

https://github.com/user-attachments/assets/c71075b9-1ebe-4fd6-afe9-d1009bf64db5

iOS: mWeb Safari

https://github.com/user-attachments/assets/d368c3b6-10f3-4ae5-a80d-21272e25f2bb

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/c20f6879-cd55-4e19-9608-657855fda214

MacOS: Desktop

https://github.com/user-attachments/assets/1ec05725-a743-425b-8609-2a3428fbd0c9

@parasharrajat A follow-up to fix https://github.com/Expensify/App/issues/46952

My bad. Added the QA Steps.

Woops, almost missed this. Taking a look later today.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/grgia in version: 9.0.20-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

FYI I believe this was deployed to prod yesterday, from this checklist - #47356


[SPARK-43172] [CONNECT] Expose host and token from spark connect client

What changes were proposed in this pull request?
Expose the host and bearer token as properties of the
SparkConnectClient.
Why are the changes needed?
This allows for querying the connecting host and bearer token
given an instance of spark connect client.
Does this PR introduce any user-facing change?
Introduces two new properties host and token to the
SparkConnectClient class
How was this patch tested?
Unit tests

Merged to master.


support utf-8 languages for hashTag and userTag

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.552%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
28

Relevant Lines:
29

💛  - Coveralls

Thanks for handling it 👍


Adds ability to cluster nearby shows

Allows us to update the drawer position which is needed to collapse the drawer when the user taps a pin
Converts shows data in GeoJSON format for consumption by the clustering engine

As of now, react-native-mapbox-gl doesn't expose an API to get points inside a cluster. I've attempted using supercluster, the underlying clustering library, to get the data but I still haven't gotten it to return the proper data.
What we do have is the cluster coordinates and it's size, so another potential solution is sort coordinates by distance and select the N first points which will correspond to the points in the cluster. The downside of this solution is it would become more inefficient as the data size grows.
TODO:

Handle tapping on a cluster which should show the following UI: https://zpl.io/29oy5Bw

Screengrabs
Clustering Demo

Collapsing drawer

#skip_new_tests

Fails

:no_entry_sign:

Missing Test Files:

src/lib/Scenes/Map/Components/__tests__/ShowCard-tests.tsx
src/lib/utils/__tests__/convertCityToGeoJSON-tests.ts

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS

Alright thanks @ashfurrow, I addressed your comments. Gonna merge this one


[Feature] Add a Global Progess Bar

The number of times it's really useful to have a global progress bar, similar to
https://hilongjw.github.io/vue-progressbar/index.html
I am recommending it because I trust Oruga, more than any other library.
It will be a great addition

@jtommy You can assign me to this Feature!


Chats are showing wrong current time of the user chatting with

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.81-5
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @RachCHopkins
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1717991929286819
Action Performed:
Precondition : user A and user B are in different timezones

User A sent a message to user B
Observe the timestamp of the chat as user B

Expected Result:
Should show the time of user A
Actual Result:
Showing the user B time in PST, after switching to another chat and coming back shows correct time
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

https://github.com/Expensify/App/assets/38435837/704383d5-7675-4437-8f99-d4d4291d346d

View all open jobs on GitHub

I'm not able to reliably reproduce, nor is @luacmartins from this thread.

@NikkiWines I'm finding for me the Social keeps switching to PST. I wonder if changing your timezone for a week might surface it?

@RachCHopkins is this happening on staging only or staging and production?

Not a deploy blocker - reproducible on prod

The pusher always sends the LA timezone data to the user when receiving the message.

I see it coming from the response to AddComment, so I think this is a backend issue.

I'm not sure exactly what it takes to reproduce it, but I've had it happen twice today. If I go into my settings, the timezone is correct for me, and then when I return to the chat, the timestamps are correct again.
After a little bit of chatting, the timestamps then revert to PST.

Fairly certain this issue is a dupe of https://github.com/Expensify/App/issues/43032, which shows almost identical behavior. I think we should close this one out in favor of that one.
cc: @MonilBhavsar since you've been able to reproduce this bug.

Feel free to. There are three issues now, I'll go ahead and remove "Needs reproduction" label on the other issue 😆


Address types do not seem consistent with API

The baseAddress as type does not seem consistent with the API. Please check if I'm wrong or not: https://docs.commercetools.com/api/releases/2021-01-29-custom-fields-for-addresses
I would expect to have 2 types as like documented on the API docs: Address and AddressDraft, also considering mapping between those each other should be easy as both almost same except custom field type.
For instance I can not map/build a valid address type.
CustomerDraftBuilder.of()
.addresses(BaseAddressBuilder.of().custom(?));   // not a method

We had to create this BaseAddress class in order to be able to differentiate between Address and AddressDraft in a backwards compatible way.
The fix here is to not use BaseAddressBuilder but the AddressDraftBuilder. Sorry for the confusion.

Oh yes, I got it now!
I think you could consider this issue as a feedback to add some javadoc for BaseAddress and it's purpose.

Yeah adding some javadoc to generated classes is somehow, let's say complicated :D

Hmm, I have a wrong mindset, I was not thinking about those are generated code, valid point 😄 👍🏽

It says to in the class and as well the folder structure 😉
https://github.com/commercetools/commercetools-sdk-java-v2/blob/5ce6328e1cbf9c21895636f39e75efbc364e46c8/commercetools/commercetools-sdk-java-api/src/main/java-generated/com/commercetools/api/models/common/BaseAddressBuilder.java#L11-L12


The given key does not identify an element in this collection value: the collection has no elements.

Hi,

Created terraform.tfvars in the examples/ubuntu2204 folder with the required credentials.
Ran terraform init
Ran terraform plan that only goes as far as saying that:

Plan: 29 to add, 0 to change, 0 to destroy.
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/centosstream8.tf line 34, in data "oci_marketplace_listing" "centosstream_8":
│   34:   listing_id     = data.oci_marketplace_listings.centosstream_8.listings[0].id
│     ├────────────────
│     │ data.oci_marketplace_listings.centosstream_8.listings is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 16, in output "oci_aarch64_images_map":
│   16:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.display_name,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 33, in output "oci_aarch64_images_map":
│   33:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.id,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 55, in locals:
│   55:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.display_name,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 72, in locals:
│   72:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.id,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 111, in locals:
│  111:         os_image_id  = data.oci_core_images.ubuntu-24_04-aarch64.images.0.id
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/ubuntu2404.tf line 20, in output "Ubuntu-24_04-aarch64-latest_name":
│   20:   value     = data.oci_core_images.ubuntu-24_04-aarch64.images.0.display_name
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/ubuntu2404.tf line 26, in output "Ubuntu-24_04-aarch64-latest_ocid":
│   26:   value     = data.oci_core_images.ubuntu-24_04-aarch64.images.0.id
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵

I think that ubuntu images' names have changed since the last check and centos has been deprecated.
Is there something I can do about this to make it work?
Thank you.

Strangely enough I fixed it yesterday.  Start from a clean directory or
update the module it should work now.
On Fri, Sep 27, 2024, 12:44 PM Sparkxxx @.***> wrote:

Hi,

Created terraform.tfvars in the examples/ubuntu2204 folder with
the required credentials.
Ran terraform init
Ran terraform plan that only goes as far as saying that:

Plan: 29 to add, 0 to change, 0 to destroy.
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/centosstream8.tf line 34, in data "oci_marketplace_listing" "centosstream_8":
│   34:   listing_id     = data.oci_marketplace_listings.centosstream_8.listings[0].id
│     ├────────────────
│     │ data.oci_marketplace_listings.centosstream_8.listings is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 16, in output "oci_aarch64_images_map":
│   16:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.display_name,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 33, in output "oci_aarch64_images_map":
│   33:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.id,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 55, in locals:
│   55:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.display_name,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 72, in locals:
│   72:       data.oci_core_images.ubuntu-24_04-aarch64.images.0.id,
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/images.tf line 111, in locals:
│  111:         os_image_id  = data.oci_core_images.ubuntu-24_04-aarch64.images.0.id
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/ubuntu2404.tf line 20, in output "Ubuntu-24_04-aarch64-latest_name":
│   20:   value     = data.oci_core_images.ubuntu-24_04-aarch64.images.0.display_name
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/ubuntu2404.tf line 26, in output "Ubuntu-24_04-aarch64-latest_ocid":
│   26:   value     = data.oci_core_images.ubuntu-24_04-aarch64.images.0.id
│     ├────────────────
│     │ data.oci_core_images.ubuntu-24_04-aarch64.images is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
I think that ubuntu images' names have changed since the last check and
centos has been deprecated.
Is there something I can do about this to make it work?
Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/AmpereComputing/terraform-oci-ampere-a1/issues/6, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AALXJ4OAPKSAXJDELN5ONCLZYWDNDAVCNFSM6AAAAABO7QQVY2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGU2TGMZUGA2TQOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I think now it's a new problem but still errors.
Plan: 30 to add, 0 to change, 0 to destroy.
Changes to Outputs:

oci_ampere_a1_private_ips = [

[

(known after apply),
],
]

oci_ampere_a1_public_ips  = [

[

(known after apply),
],
]
╷
│ Error: Invalid index
│
│   on .terraform/modules/oci-ampere-a1/centosstream8.tf line 34, in data "oci_marketplace_listing" "centosstream_8":
│   34:   listing_id     = data.oci_marketplace_listings.centosstream_8.listings[0].id
│     ├────────────────
│     │ data.oci_marketplace_listings.centosstream_8.listings is empty list of object
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵

Thank you.

Looks like I had some code to push.  Try again please.  The Centos stuff
was a shot in the dark to see if they had arm64 bits.
pp
On Tue, Oct 8, 2024 at 11:12 AM Sparkxxx @.***> wrote:

I think now it's a new problem but still errors.
Plan: 30 to add, 0 to change, 0 to destroy.
Changes to Outputs:

oci_ampere_a1_private_ips = [

[

(known after apply),
],
]

oci_ampere_a1_public_ips = [
[

(known after apply),
],
]
╷
│ Error: Invalid index
│
│ on .terraform/modules/oci-ampere-a1/centosstream8.tf line 34,
in data "oci_marketplace_listing" "centosstream_8":
│ 34: listing_id =
data.oci_marketplace_listings.centosstream_8.listings[0].id
│ ├────────────────
│ │ data.oci_marketplace_listings.centosstream_8.listings is
empty list of object
│
│ The given key does not identify an element in this collection
value: the collection has no elements.
╵

Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/AmpereComputing/terraform-oci-ampere-a1/issues/6#issuecomment-2400129660,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALXJ4LXU77SO252WQR7LV3Z2PY5JAVCNFSM6AAAAABO7QQVY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGEZDSNRWGA
.
You are receiving this because you commented.Message ID:
@.***>


Add global namespace OP2Utility

Closes #237

I didn't think about all the extra tabs causing so much noise in the PR...

I didn't think about all the extra tabs causing so much noise in the PR...

Ahh, as soon as I read the title, I realized this was going to have merge conflicts with the other two branches that were recently merged.

Ahh, as soon as I read the title, I realized this was going to have merge conflicts with the other two branches that were recently merged.

If you change nothing but the tabs, the diff for that commit is usually pretty easy to read. Though if you change so much as one other character, the diff tends to be extremely messy and hard to read.
In the interest of keeping diffs easy to read and review, sometimes I'll make an intermediate commit with a code change, but no corresponding indentation change. For instance, adding a namespace wrapper block, but not adding corresponding indentation. That means there will be a commit with bad indentation, though I think this is acceptable, as it's temporary in one commit, generally fixed in the following commit, and can make review much easier due to the simpler diffs.

If you change nothing but the tabs, the diff for that commit is usually pretty easy to read. Though if you change so much as one other character, the diff tends to be extremely messy and hard to read.
In the interest of keeping diffs easy to read and review, sometimes I'll make an intermediate commit with a code change, but no corresponding indentation change. For instance, adding a namespace wrapper block, but not adding corresponding indentation. That means there will be a commit with bad indentation, though I think this is acceptable, as it's temporary in one commit, generally fixed in the following commit, and can make review much easier due to the simpler diffs.

@DanRStevens, If you would prefer, I am ok redoing this branch trying to separate the indentation change from the actual insertion of code change. If you do not prefer the rework, I will just fix the merge conflict. It is probably about 30 minutes for me to redo in a new branch which isn't a big deal assuming I can get Visual Studio's auto complete functions to assist me instead of hindering me...
Regarding the comment about not mixing template and non-template functions, I had not heard of this guideline but I am admittedly weak with templates. What are your thoughts about changing the name of the template function? I'm not experienced or picky about it either way. Could just name it DirInternal or DirTemplate.
I wanted to complete the namespace addition and a bit of work with the organization of the source code files before switching towards work in OP2BitmapConverter.

@DanRStevens, If you would prefer, I am ok redoing this branch trying to separate the indentation change from the actual insertion of code change. If you do not prefer the rework, I will just fix the merge conflict. It is probably about 30 minutes for me to redo in a new branch which isn't a big deal assuming I can get Visual Studio's auto complete functions to assist me instead of hindering me...
Regarding the comment about not mixing template and non-template functions, I had not heard of this guideline but I am admittedly weak with templates. What are your thoughts about changing the name of the template function? I'm not experienced or picky about it either way. Could just name it DirInternal or DirTemplate.
I wanted to complete the namespace addition and a bit of work with the organization of the source code files before switching towards work in OP2BitmapConverter.

I was kind of thinking renaming the template function might be a good idea.
Not really a strong preference for rework versus patch up. Maybe a slight preference towards rework if other changes are going to be made. I suppose it would be easier to verify nothing goes badly trying to fix merge conflicts if there was some rework instead.

Downstream projects will need to be updated after this change is merged. Actually, as it stands, downstream projects might already need to be updated due to the StringHelper rename.

I was kind of thinking renaming the template function might be a good idea.
Not really a strong preference for rework versus patch up. Maybe a slight preference towards rework if other changes are going to be made. I suppose it would be easier to verify nothing goes badly trying to fix merge conflicts if there was some rework instead.

Downstream projects will need to be updated after this change is merged. Actually, as it stands, downstream projects might already need to be updated due to the StringHelper rename.


Cleanup naming in module types.

[ ] ModuleInit -> ModuleConstructor (or ModuleGen?)
[ ] Find&replace all ConfigGen and config_gen to match.
Consider I prefix in all module traits that we use in Arc<dyn Trait> and alikes. Just for consistency.
Probably some other stuff. Go around and think about it.

I find the name ServerModulePlugin kind of weird. Plugin and module in my mind mean roughly the same thing. So seeing both next to each other is odd.

Yeap. This was a temporary meassure as we have:
struct Foo; // the dyn newtype wrapper

trait Foo; // type erased module trait

trait FooPlugin; // version of `Foo` with associated types that auto-impls to `Foo`

Better name suggestions appreciated.

@elsirion Maybe ModuleInit should be named ModuleKind or ModuleType. That's kind of a design that is build in my head.
The binary app has to just list all the module "types" it supports (ModuleKindRegistry). Then the federation members build the actual list of module instances with its configuration, where each instance is of type from the supported kinds.

Not sure if it directly refers to this, but it's about naming, too:
I think as someone who wants to implement a module, it would "sound" and "feel right" to just write
impl ServerModule for MyGreatNewServerModule

naming convention may look like:
trait IServerModule;  // interface
struct DynIServerModule<Arc<dyn IServerModule>>;  // dyn new type
trait ServerModule; //  trait with associated types implemented by modules

I find the name ServerModulePlugin kind of weird. Plugin and module in my mind mean roughly the same thing.

I agree. With less work one could just rename ServerModulePlugin to ServerModuleTyped, though, that would carry the "unnecessary" information Typed for an implementor.

@dpc Instead of a ModuleKindRegistry we could have a FedimintdBuilder and a DistributedgenBuilder to which these ModuleKind types could be mounted to respectively.

@elsirion Ack on the builder, though I'm not sure about the composition of the .run and the UI and other misc code in the main() function now. We'll see what we can do.
@mxz42
I think I prefix for dyn-trait is easy to agree on. It's barely visible across the code anyway.
The dyn newtype is used all across the generic code, so it would be great if it had a neat short name. But if we put 3rd party developer DX first-most, maybe typed traits deserve the shortest name.
So then ... how to name the dyn-newtype (currently struct ServerModule(Arc<IServerModule>);)?
Either DynServerModule, or we could have both of them named ServerModule and just rely on module qualificator when there's a conflict. Not exactly unusual in the Rust ecosystem (how many Results there are). Plugin writers will only care about the typed-trait, while the generic code usually only cares about the dyn-newtype. Client code still sometimes touches both, but that should be minimized in the future, and they can prefix with fedimint_wallet::ServerModule to avoid conflicts and job done.
@elsirion Thoughts?

I like the Dyn* naming convention, it makes it easier to see that the type is a type-erased version of something else (I did initially call the Box<dyn …> types Any*, which seemed to be confusing, Dyn* is the more correct prefix it seems).
Letting the base trait have the shortest name seems like a nice benefit too.

@elsirion All right. I think I tried for too long to avoid the Dyn on a newtype, and it might be just a mistake.
@mxz42 Do you think you could do all the renames and other cleanups? It seems like you're on to the modularization already.

@mxz42 Do you think you could do all the renames and other cleanups? It seems like you're on to the modularization already.

Yes, trying to understand it. :+1:  Ok, I'll start renaming the symbols in my comment.
I haven't had a close look at ModuleInit or ConfigGen, so I'll likely need some hints there

I haven't had a close look at ModuleInit or ConfigGen, so I'll likely need some hints there

They are like all other types, methinks. Hit me on Discord with any questions.

while creating #1292 I removed the Module in Lightning* symbols because I think it's redundant. The only place it's left now it the LightningModuleDecoder because currently it is consistent with the other modules such as MintModuleDecoder and WalletModuleDecoder.
might be worth to change it for all modules to MintDecoder etc.
thoughts?

@mxz42 :+1: from me.
BTW. I think I still see Plugin in associated types. Might want to go for these next.

BTW. I think I still see Plugin in associated types. Might want to go for these next.

yes, good point. will do

Another thought occurred while doing the renaming: all the Plugin* prefixes were a nice sign that this was the typed interface. That's now more or less lost.  I guess another option would be to name them something like
trait IModInput; // interface module input
struct DynModInput; // new type
trait ModInput;  // typed trait. -> these would start with `Mod*`

Another thought occurred while doing the renaming: all the Plugin* prefixes were a nice sign that this was the typed interface.

Yes, but they were also confusing the user with their meaning.
I is erased-trait, Dyn is dyn newtype wrapping I, lack of prefix is just what it is ... what the user (module developer) has to interact with.

@elsirion Ack on the builder, though I'm not sure about the composition of the .run and the UI and other misc code in the main() function now. We'll see what we can do.

Eventually the admin UI will probably be a single page app that communicates via API vs server-side templated app it is today. That might work something like:
#[tokio::main]
fn main() {
    FedimintServer::builder()
        .datadir("/var/lib/fedimintd")
        .with_ui("/path/to/ui/build/folder")
        ...
}

If path is provided, fedimintd will serve those files. Otherwise, static UI files will need to be served elsewhere (e.g. by nginx). Calling with_ui could turn on all admin-related HTTP endpoints in the "core" (e.g. have we run dkg yet? which of my peers are offline?) and in the modules.

We could also statically bundle HTML+JS and allow modules to inject their own JS. But your idea is probably more practical in the near term.
One nit: .with_ui(Some("/path/to/ui/build/folder")) should be .with_ui("/path/to/ui/build/folder"), simply not calling it would be equal to None.

One nit: .with_ui(Some("/path/to/ui/build/folder")) should be .with_ui("/path/to/ui/build/folder"), simply not calling it would be equal to None.

I was thinking that calling it with None would enable the admin API endpoints (which paranoid users might want to disable) but not attempt to serve any static files.

Bike shedding: just make it two separate functions, where calling one without the other panics on build. Or you could be fancy and use type-level boolean algebra to determine which combinations are legal.

Closing this due to inactivity


query! macros should allow for a trailing comma

I'd like to take a shot at this if it isn't already taken.

@timmythetiny be our guest! I've got instructions for the intended fix here: https://github.com/launchbadge/sqlx/pull/90#issuecomment-578972226


Issue with k8s.io/docs/concepts/overview/working-with-objects/namespaces/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
$ kubectl api-resources --namespaced=true
Error: unknown command "api-resources" for "kubectl"
Proposed Solution:
No idea
Page to Update:
https://kubernetes.io/...

Kubernetes Version:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.7", GitCommit:"0c38c362511b20a098d7cd855f1314dad92c2780", GitTreeState:"clean", BuildDate:"2018-08-20T10:09:03Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.7-gke.11", GitCommit:"fa90543563c9cfafca69128ce8cd9ecd5941940f", GitTreeState:"clean", BuildDate:"2018-11-08T20:22:21Z", GoVersion:"go1.9.3b4", Compiler:"gc", Platform:"linux/amd64"}

Client Version: version.Info{Major:"1", Minor:"10",

This feature was added in 1.11 in https://github.com/kubernetes/kubernetes/pull/42873. You'll need to be on a v1.11+ cluster for this to work.
Fwiw I can verify that it works on a 1.11+ cluster.
/close

Thanks! Didn't notice.


fix #41 test-08-failOnClassesExcludedFromJarStar

Resolves https://github.com/sbt/sbt-osgi/issues/41
The replacing was too aggressive leading to wrong things happening; we just want to drop the trailing .* if present

FYI @xuwei-k

I'm confused about what's going on here actually now... will look into it more later

I'm confused about what's going on here actually now... will look into it more later

@ktoso pretty sure that the -> in the failing test needs to be replaced by >.

I missed your reply here, sorry about that @larsrh ! Will try then!
And release a new version today I think

I missed your reply here, sorry about that @larsrh ! Will try then!

No worries, I have no recollection of even posting that comment 😆

Yeah I'd also rather not remember any work on anything osgi related ;)

I think it SHOULD fail then... leaving this open for now


Change cancellation semantics in the iomgr DNSResolver

DNS requests were previously cancellable, but it was assumed
that the resolution callback would be called in all cases. Now, requests
provide a bool Cancel(TaskHandle handle) method that lets callers know
whether cancellation is successful, and if so, the callback will not be
run.
This is in accordance with EventEngine's cancellation semantics, and it
is a step towards migration from iomgr to EventEngine.

This change is

Thanks @apolcyn. I missed Mark's bit where he requested your review as well, I will follow up shortly.


seek support when the server does not support Ranges header

h5py and a lot of other libraries (including webdav4) try to detect the size of the file obj by using seek. But, in these cases, they go back to seek(0), so no read is done from the iterator. So, in theory, we could still support this usecase by somehow making the iterator lazy (and, might get better performance too).
See https://github.com/iterative/dvc/issues/5990 for more information.

Closing as resolved.


Setup GitHub actions

Issue #, if available:
Description of changes:

run tests on windows and macOS
setup reviewdog/action-golangci-lint

some tests fail, but it's not due to this pull request.
I'll send other pull requests to fix tests.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Do you know why Travis CI tests are failing ?

Do you mean the tip test? https://travis-ci.org/aws/aws-xray-sdk-go/jobs/637127842
I don't know why, but tip version is removed from the document https://github.com/travis-ci/gimme/pull/102

Could be the reason.


translate docu to German

will do that for the "Hanser Verlag arc42 Booklet" (as printed arc42-Example)

will do that for the "Hanser Verlag arc42 Booklet" (as printed arc42-Example)

will NOT do that in AsciiDoc or on Github - to avoid "multiple languages hassle"

will NOT do that in AsciiDoc or on Github - to avoid "multiple languages hassle"


[ITensors] Fix some MPO ChainRules bugs

Fixes #1313.
This also provides a partial fix for #1314, where we throw an error when trying to take derivatives through gate application of non-Hermitian MPOs with apply_dag=true, which is broken right now but was silently returning incorrect results.

Codecov Report
Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (2fd3e0a) 84.32% compared to head (c26d237) 53.68%.
 Consider uploading reports for the commit df7d2ba to get more accurate results

Files
Patch %
Lines

src/ITensorChainRules/mps/abstractmps.jl
0.00%
7 Missing :warning:

src/ITensorChainRules/mps/mpo.jl
0.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1335       +/-   ##
===========================================
- Coverage   84.32%   53.68%   -30.64%
===========================================
  Files         100       99        -1
  Lines        8568     8525       -43
===========================================
- Hits         7225     4577     -2648
- Misses       1343     3948     +2605

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

[test ITensors mps]


Create Menu components in Kodiak UI

To utilize the useMenu hook, Kodiak UI should provide the corresponding components to style the HTML elements.

MenuList
MenuListItem

Each components should accept the sx and variant props and a ref with the register function passed in.

Main Story: ch52093
This PR will address changes to the following:

[x] Components
[ ] Hooks

Acceptance Crtieria

[x] Developers should be able to use the components above to render the dropdown button and menu and style it with the sx and variant props.

API Changes

[ ] [Breaking] This PR introduces breaking changes
[ ] [Documentation] Documentation has been written about the API change

Deployment
Before merge to master
Note: Code merged to master should be safe to automatically deploy to production as-is.

[x] [QA] User-testing/quality assurance done
[ ] [Test] All components and hooks should have a corresponding unit test
[ ] [Storybook] All components and hooks should have a corresponding Storybook story
[ ] [Documentation] Documentation has been written or updated
[ ] [Version] Version number has been updated

Thinking about tabIndexes, we might want to make the menu items have one or use an element that includes a tabIndex just so tabbing works.

Absolutely, thank you for catching that.
I'll look into scrolling. This is the worst part about portals.

@alexanderchan the scrolling is an issue with the Docs only. Everything works separately.


Add remindT command to developer guide

The developer guide is incomplete.
The developer guide should be up to date and comprehensible.
Let's add the remindT command to the developer guide to improve the quality of documentation.

Codecov Report

Merging #256 (b4b9622) into master (6eccbc4) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #256   +/-   ##
=========================================
  Coverage     53.62%   53.62%
  Complexity      685      685
=========================================
  Files           138      138
  Lines          2816     2816
  Branches        365      365
=========================================
  Hits           1510     1510
  Misses         1200     1200
  Partials        106      106

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


Add pbp job gen program which includes orchestration script

This builds on linux and windows. The pbp-job-agent is now part of the pbp program suite. It accepts a configuration YAML file that contains the job agent settings.
Usage:
    $ pbp-job-agent --config configuration_1.yaml configuration_2.yaml configuration_3.yaml

will spawn three pbp-job-agents as three separate processes. This agent will provision all metadata, nc files and plots.
Important Notes:
The configuration schema itself can be found through examples by entering
    $ pbp-job-agent --help

  >
  usage: pbp-job-agent [-h] --config path [path ...]

  Deploys a job agent to process audio files. Each yaml file configuration argument spawns a new process.

  optional arguments:
    -h, --help            show this help message and exit
    --config path [path ...]
                          path to the YAML file(s) that contain the configurations for the pbp-job-agent

      Examples:
      pbp-job-agent --config /path/to/deployment/global_config_1.yaml /path/to/deployment/global_config_2.yaml

      Runs two job-job-agent process concurrently with the configurations provided in the yaml files.

      An Example YAML Content (For SoundTrap) ~ global_config_1.yaml:

      pbp_job_agent:
        output_prefix: Georges_Bank_2021
        recorder: "SOUNDTRAP"
        audio_base_dir: "/home/user/SOUNDTRAP_DEPLOYMENT/"
        json_base_dir: "/home/user/SOUNDTRAP_DEPLOYMENT/JSON"
        xml_dir: "/home/user/SOUNDTRAP_DEPLOYMENT"
        start: "20220521"
        end: "20220521"
        prefix: "6550"
        nc_output_dir: "/home/user/SOUNDTRAP_DEPLOYMENT/NC"
        global_attrs: "/path/to/deployment/global_config_1.yaml"
        variable_attrs: "/path/to/deployment/variable_config_1.yaml"
        sensitivity_flat_value: "176.6"
        latlon: "-31.711 115.583"
        title: "Georges_Bank_2021"
        cmlim: "36 107"
        ylim: "10 24000"
        meta_output_dir: "/home/user/SOUNDTRAP_DEPLOYMENT/META"
        voltage_multiplier: ""
        sensitivity_uri: ""
        log_dir: "/home/user/SOUNDTRAP_DEPLOYMENT/AGENT"

The --global-attrs file is distinct from the --config file. While they are not the same, the --global-attrs file can serve a similar purpose if it includes the necessary global attributes. The design decision allows users the flexibility to not place all job agent configurations in the global_attrs file, though doing so is entirely acceptable.
If you opt to use the global_attrs file, you must reference its path within the configuration, even if it points to the file itself. This is required because the arguments in the global_attrs file are also used by other pbp program calls.
Additionally, stability on Windows has been improved through better utilization of path libraries.
The tests arent in yet but I am working on them.

Thanks Danelle, Ill try to get them to you asap.

Hi Danelle,
I just realized I’m running into an issue with writing these tests. My initial plan was to assert the existence of paths and directories after they’re created by the agent. However, I don’t have any Soundtrap WAV files to include in the repo for testing, as the ones I use locally are too large. I’m unsure how to handle this without compromising the integrity of the tests.
My next thought was to reimplement the tests you already created using the files you had, but I didn’t see an obvious test for pbp-hmb-gen, only for pbp-meta-gen.
Another option I considered was testing that the job agent’s terminal command profile aligns correctly with the pbp command suite proper. (Feels cheap I admit)
Beyond that, the only idea I have left is to test whether the YAML files themselves are being parsed correctly.
Please advise and thankyou.

Hi @spacetimeengineer
How about you give us a quick demo of this early next week so we can all get up to speed?

Ofcourse!

Echopype has a great technique for handling large test files that I just discovered. They set up a Docker image containing all the necessary files, and in the documentation, developers are instructed to pull the image and map its contents to a specific location in the local repository. This approach allows test files to be very large while still having a local path for easy access and allowing the pulling of the pbp repo still very quick


Minified js go-timepicker

GigaOM/gigaom#6493
Wasnt too sure if i added ' . $js_min . ' in the right spots

:sparkles:


Googlesheets Quickstart example unreachable in localhost

Expected Behavior
From the example provided in - https://developers.google.com/sheets/api/quickstart/go#run_the_sample , the expected behaviour upon running the app first time is:
In your browser, go to http://localhost:8000.

The first time you run the sample, it prompts you to authorize access:

If you're not already signed in to your Google Account, you're prompted to sign in. If you're signed in to multiple accounts, select one account to use for authorization.
Click Accept.
Copy the code from the browser, paste it into the command-line prompt, and press Enter.
Authorization information is stored in the file system, so the next time you run the sample code, you aren't prompted for authorization.

Actual Behavior
nothing running in localhost
curl http://localhost:8000                                                                                7 ↵ ryemane@robel-cs
curl: (7) Failed to connect to localhost port 8000 after 4 ms: Connection refused

and when following the link that the app suggest to visit i.e.
go run quickstart2.go                                                                                1 ↵ ryemane@robel-cs
Go to the following link in your browser then type the authorization code:
https://accounts.google.com/o/oauth2/auth?access_type=offline&client_id=372148273502-3kj1acfo674on8q2hodqdbbnv67vvn2r.apps.googleusercontent.com&redirect_uri=http%3A%2F%2Flocalhost&response_type=code&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fspreadsheets.readonly&state=state-token

I get asked that the app wants to access my google account but when I allow it, it fails to continue and site is unrechable

Steps to Reproduce the Problem

Strictly followed the example here: https://developers.google.com/sheets/api/quickstart/go

Specifications

Go version (go version): go version go1.19.5 darwin/arm64
OS (Mac/Linux/Windows) Mac

I have the same problem

This workaround did work for me. But, clearly, it's a bug that I'd appreciate a lot if it were fixed.
Thanks again.

It seems like there is some code missing from the example. The instructions want you to connect to a local web server but the example code doesn't start one.


Maintenance frequency code list

This code list has not yet been added to the INSPIRE Registry.
To be decided if any action should be taken.

As that code list wasn't published in the INSPIRE Registry, we published it in the Italian Registry. We also published the cross-map register with the mapping between the ISO code list and the EU Vocabularies Frequency Named Authority List based on the alignment defined in GeoDCAT-AP.
We could propose to the INSPIRE Registry control body to publish the code list and then update the related section in GeoDCAT-AP with the URI coming from that Registry.
An alternative option would be to map the two values from ISO code list not covered by the EU Vocabularies Frequency Named Authority List (i.e. 'asNeeded' and 'notPlanned') to the EUV-FREQ value 'OTHER'.

@AntoRot said:

As that code list wasn't published in the INSPIRE Registry, we published it in the Italian Registry. We also published the cross-map register with the mapping between the ISO code list and the EU Vocabularies Frequency Named Authority List based on the alignment defined in GeoDCAT-AP, following the requirements of ISO Standard 19146 (Geographic information — Cross-domain vocabularies).
We could propose to the INSPIRE Registry control body to publish the code list and then update the related section in GeoDCAT-AP with the URI coming from that Registry.

Request submitted: https://github.com/INSPIRE-MIF/helpdesk-registry/issues/9

The maintenance frequency code list is now available from the INSPIRE Registry - see https://github.com/INSPIRE-MIF/helpdesk-registry/issues/9#issuecomment-815489521

GeoDCAT-AP XSLT revised accordingly - see https://github.com/SEMICeu/iso-19139-to-dcat-ap/issues/28

GeoDCAT-AP specification updated accordingly.
A possible additional action is to investigate if asNeeded and notPlanned might be included in the EU Vocabularies Frequency NAL.

I have sent a proposal to add the two terms to the EU Vocabularies Frequency NAL to support more precise alignment with the Maintenance frequency code list.

Response from the Publications Office is that they will investigate the suggestion in the process of the revision for the next release (no timeline provided).

The word from the Publications Office is that the two conepts will be included in the NAL(probably in June 2024). The mapping will be adjusted accordingly.

The items were added as:
http://publications.europa.eu/resource/authority/frequency/AS_NEEDED
http://publications.europa.eu/resource/authority/frequency/NOT_PLANNED


Mention lints and benchmarks next to code coverage in summary

Triggered by: https://github.com/bitcoin/bitcoin/issues/31178#issuecomment-2449435667

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews
See the guideline for information on the review process.
A summary of reviews will appear here.


i18n(ru): update some guides

Update:

ru/guides/i18n.mdx
ru/guides/components.mdx
ru/guides/sidebar.mdx
ru/guides/site-search.mdx

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ru
guides/components.mdx
Localization changed, will be marked as complete.

ru
guides/i18n.mdx
Localization changed, will be marked as complete.

ru
guides/sidebar.mdx
Localization changed, will be marked as complete.

ru
guides/site-search.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Create 1669-merge-in-between-linked-lists.go

File(s) Modified: 1669-merge-in-between-linked-lists.go
Language(s) Used: Go
Submission URL:  _https://leetcode.com/problems/merge-in-between-linked-lists/submissions/1209133978/_

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

Hey @a93a , can you take a look at this?

Thank you!


Remove "cursor: default"

Describe the issue
Current Behaviour
cursor: default makes user cursor look the same even if user hovers over text. This is not like all of the websites work.

Expected Behaviour
User cursor should switch to "text cursor" when user hovers over text

Removing cursor: default will fix this.

I agree.
That's come from sanitize.css
But I probably shouldn't have kept it
Do you want to open the PR?

Do you want to open the PR?

Nope


Create Index using context

Description
In 2.16, we enabled functionality to use context to create index templates. This goes a step ahead, and allows to create index using the context directly specified in index creation.
Related Issues
Resolves  https://github.com/opensearch-project/OpenSearch/issues/12683

Check List

[x] Functionality includes testing.
[x] API changes companion pull request created, if applicable.
[x] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Please don't forget to add context to https://github.com/opensearch-project/opensearch-api-specification.

Please don't forget to add context to https://github.com/opensearch-project/opensearch-api-specification.

Yes, I'm making the changes for the same :)


codacy code cleanup

[Current coverage][cc-pull] is 70.20%

Merging [#44][cc-pull] into [master][cc-base-branch] will increase coverage by 17.18%

@@             master        #44   diff @@
==========================================
  Files            31         22     -9
  Lines           760        574   -186
  Methods           0          0
  Messages          0          0
  Branches        104         85    -19
==========================================
  Hits            403        403
+ Misses          325        139   -186
  Partials         32         32

Powered by Codecov. Last updated by [00e6302...c1da9e0][cc-compare]
[cc-base-branch]: https://codecov.io/gh/web-innovate/bootstraped-multi-test-results-report/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/web-innovate/bootstraped-multi-test-results-report/compare/00e630234a801e9c4ed8a269eb47b2fda3b0e91a...c1da9e08f5181bb0aab4a2cdd8bea730db91d47c
[cc-pull]: https://codecov.io/gh/web-innovate/bootstraped-multi-test-results-report/pull/44?src=pr


Final fix in multi cluster

Signed-off-by: ashish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Do we have multi-cluster tests, yet? // @Abhishek-kumar09

No, I don't think we have it at this point.


Can't integrate pytorch-mobile with react-native android app

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
I want to integrate pytorch mobile in react-native app. though I can build successfully but get error after I invoke method
error messages:
couldn't find DSO to load: libpytorch_jni.so
SoSource 0: com.facebook.soloader.ApkSoSource[root=/data/data/com.example.pytorchmobile/lib-main flags = 1] ...
React Native version:
System:
OS: macOS 11.4
CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
Memory: 96.56 MB / 8.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 12.22.6 - ~/.nvm/versions/node/v12.22.6/bin/node
Yarn: 1.22.15 - /usr/local/bin/yarn
npm: 6.14.15 - ~/.nvm/versions/node/v12.22.6/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.1 - /Users/vergil/.rvm/rubies/ruby-2.6.6/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 15.0, DriverKit 20.4, macOS 11.3, tvOS 15.0, watchOS 8.0
Android SDK:
API Levels: 23, 25, 26, 27, 28, 29, 30
Build Tools: 27.0.3, 28.0.3, 29.0.2, 30.0.2
System Images: android-26 | Google APIs Intel x86 Atom_64, android-28 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7784292
Xcode: 13.0/13A233 - /usr/bin/xcodebuild
Languages:
Java: 14.0.1 - /usr/bin/javac
Python: 3.7.4 - /Users/vergil/opt/anaconda3/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.66.0 => 0.66.0
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

use reate-react-native-library to create project

$ npx reate-react-native-library testproject

follow instruction to add Gradle dependancies
try this demo app example
build android app & run

Expected Results
should load model and run successfully
Snack, code example, screenshot, or link to a repository:
I also add these to testproject > example > android > app > build.gradle to fix "More than one file was found with OS independent path 'lib/x86/libfbjni.so'" issue and prevent pytorch model from compression in the APK
...

android {
     packagingOptions {
        pickFirst 'lib/arm64-v8a/libfbjni.so'
        pickFirst 'lib/armeabi-v7a/libfbjni.so'
        pickFirst 'lib/x86/libfbjni.so'
        pickFirst 'lib/x86_64/libfbjni.so'

        pickFirst 'lib/arm64-v8a/libc++_shared.so'
        pickFirst 'lib/armeabi-v7a/libc++_shared.so'
        pickFirst 'lib/x86/libc++_shared.so'
        pickFirst 'lib/x86_64/libc++_shared.so'
    }

    aaptOptions {
        // pytorch model
        noCompress 'pt'
        noCompress 'ptl'
    }
}

Seems like you also opened this other issue on PyTorch
https://github.com/pytorch/pytorch/issues/66685
Have you applied the workaround suggested here: https://discuss.pytorch.org/t/pytorch-not-compatible-with-react-native-library/73841/10
Is it working for you?