Add pruvious to existing nuxt project with pages

Hi again,
I have an existing Nuxt website which uses the 'pages' directory i.e file routing. I'm trying to implement blog/news feature using pruvious. I've setup the collection, but when i get to the dashboard for add blogs/news, I get my error page saying no such file or directory for the blog/news page on both the preview and on opening on the new tab too.
ENOENT: no such file or directory, open '/pages/news/[...slug].vue'

I figured it's because I'm using file routing.
So is there a way to do this or am I missing something?

Hi! You can try changing the pruvious.api.routes['pages.get'] option in in your nuxt.config.ts. See here: https://github.com/pruvious/pruvious/blob/b58d71b5512d0e4f4c28de09acbc06645b84db7b/src/runtime/module-types.ts#L117-L125

Hi! Could you provide more information about the files in the pages directory? If this is an existing Nuxt project, you need to include a catch-all page with the <PruviousPage /> component, like this: https://github.com/pruvious/pruvious/blob/main/src/runtime/pages/[...slug].vue

Hi! You can try changing the pruvious.api.routes['pages.get'] option in in your nuxt.config.ts.

What should I change it to? I tried setting it to false but nothing changed.

Hi! Could you provide more information about the files in the pages directory? If this is an existing Nuxt project, you need to include a catch-all page with the <PruviousPage /> component, like this: main/src/runtime/pages/%5B...slug%5D.vue

What specific information do you need?
pages
├── about-us
│   ├── foundation.vue
│   ├── index.vue
│   ├── labworks.vue
├── index.vue
├── privacy-policy.vue
├── production.vue
├── products.vue
├── [...slug].vue
└── trainings.vue

This is how my pages directory looks like.
I added the [...slug].vue catch all route but i still get the same nuxt error page for route not found.
This is my news collection:
import { defineCollection } from "#pruvious";
import { pageLikeCollection } from "#pruvious/standard";

// @see https://pruvious.com/docs/collections
export default defineCollection(
  pageLikeCollection({
    name: "news",
    pathPrefix: "news",
    icon: "Pin",
    allowedLayouts: ["news"],
    additionalPublicPagesFields: ["headline", "author", "tags"],
    additionalFields: {
      headline: {
        type: "text",
        options: {
          label: "Headline",
          description: "If left empty, the page title will be used",
        },
      },
      tags: {
        type: "chips",
        options: {
          choices: {
            ktl: "KTL",
            hts: "HTS",
            charity: "Charity Works",
            httc: "HTTC",
            donation: "Donations",
          },
          required: false,
          label: "Tags",
          sortable: true,
          description: "Category tags for the news",
        },
      },
      author: {
        type: "record",
        options: {
          collection: "users",
          fields: ["firstName"],
          populate: true,
        },
      },
    },
  })
);

What should I change it to? I tried setting it to false but nothing changed.

I misread your question initially, so I quickly deleted my first answer, hoping you wouldn't see it. Please ignore this, as it relates to /api/pages.

What specific information do you need?

Actually, the error ENOENT: no such file or directory, open '/pages/news/[...slug].vue' indicates that something is trying to access the pages folder from your top-level directory. Does the same thing happen if you disable Pruvious and open http://localhost:3000/news/foo?

So, that error doesn't seem to always occur. The preview pane shows 500 error but when i manually navigate to something like news/foo i get a 404 even without removing pruvious from nuxt.config.

This is hard to guess without a reproduction. If you’d like, you can invite me to your repository and I can take a look.

Well, its inside my company repo, but I will try to see if I can give you access.
Funny thing, when I create a page using the pages tab in the dashboard with example news/pages, it works fine, so is the news prefix the problem?
I would use this method, since I'm on a tight schedule, but the drawback is that I don't think I'll be able to access the news index page with the listing of all the news articles.

I was finally able to fix the error. Apparently it was a vite error caused by a typo in a component I was using 🤦♂
Apologies for this and thank you for the help.


beatsaber server crash and restart batch script

sorry im new to this Linux os for the server automatic restart on crash i see the script on the site and i understand to place it in the serverhub dir but what do i save the files as whats the extension type how do i make the files. Is there somewhere a can download them and just edit my details into them. thanks for your help

I don't really use Linux either, so I don't know, but you can ask in Beat Saber Modding Group Discord to help you

Ok thanks


Implement Tree Traversal Algorithms

Missing: Tree Traversals (Inorder, Preorder and Postorder) implementation
more on tree traversal can be found here:https://en.wikipedia.org/wiki/Tree_traversal#Types

hi, may I take up this issue?

@MansiAyer yes,Implement all the Tree Traversals & send pull request

👍


rostopic echo problem & how to visualize my own rosbag in DreamView?

System information:
OS Platform and Distribution (Linux Ubuntu 14.04):
Apollo installed from (source):
Apollo version (3.0):
Steps to reproduce the issue:

get into dev_container
start roscore
rosbag play 2017-06-15-13-03-58_filterd.bag --loop (which is a rosbag containing the gps-data from ublox with the info below):

rostopic echo [topic showed in the info picture above]

Problem:
only the two topics /Raw_Curvature_sensor [std_msgs/Float32MultiArray], /ublox_gps/fix [sensor_msgs/NavSatFix] can be echoed with the massages below:
header:
  seq: 25
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 594009119
  frame_id: gps
status:
  status: 0
  service: 1
latitude: 51.0356254
longitude: 13.7393869
altitude: 183.797
position_covariance: [7.584516, 0.0, 0.0, 0.0, 7.584516, 0.0, 0.0, 0.0, 12.299049]
position_covariance_type: 2

layout:
  dim:
    -
      label: yaw__lat__steer__spdms
      size: 4
      stride: 1
  data_offset: 0
data: [0.02500000037252903, 0.06666681170463562, -1.5010906457901, 0.0]

the other three topics /ublox_gps/navposllh [ublox_msgs/NavPOSLLH], /ublox_gps/navvelned [ublox_msgs/NavVELNED], /ublox_gps/rxmraw [ublox_msgs/RxmRAW] all reported error below:
/use_sim_time is not set, will not subscribe to simulated time [/clock] topic
ERROR: Cannot load message class for [ublox_msgs/NavVELNED]. Are your messages built?

Questions:

why do topics from the same rosbag work differently with "rostopic echo" im Apollo? it appears to have something to do with the massage type. But what is the reason exactly?
how can I deal with this problem? actually i'm trying to play this rosbag in dreamview, is it even possible?
any comments are appreciated. Thanks. :)

The type of played topics (std_msgs/Float32MultiArray and   sensor_msgs/NavSatFix) are statard message type from ros.
So,if you want to use these message type, you need to compile them firstly.

@mickeyouyou Thank you for your reply. Can you please give me some hints about how to compile the message type in Apollo? or is there any tutorial I can refer to ?

@mickeyouyou  Thank you for your reply. Can you please give me some hints about how to compile the message types in Apollo? Or maybe is there any tutorials to refer to?

@mickeyouyou or more specifically in dev_container :)

ok, you may need refer driver like https://github.com/KumarRobotics/ublox
that contains the types of all you want to play :

NavVELNED
NavPOSLLH
RxmRAW

In apollo you can find drivers like velodyne lslidar rslidar have some in common.
How to compile ublox driver into apollo ?
we just need to compile some message to ROS, so generally use catkin tool , you can find a function in apollo.sh in top directory :
function build_velodyne() {
  CURRENT_PATH=$(pwd)
  if [ -d "${ROS_ROOT}" ]; then
    ROS_PATH="${ROS_ROOT}/../.."
  else
    warning "ROS not found. Run apolllo.sh build first."
    exit 1
  fi

  source "${ROS_PATH}/setup.bash"

  cd modules
  catkin_make_isolated --install --source drivers/velodyne \
    --install-space "${ROS_PATH}" -DCMAKE_BUILD_TYPE=Release \
    --cmake-args --no-warn-unused-cli
  find "${ROS_PATH}" -name "*.pyc" -print0 | xargs -0 rm -rf
  cd -

  rm -rf modules/.catkin_workspace
  rm -rf modules/build_isolated/
  rm -rf modules/devel_isolated/
}

@mickeyouyou Thank you so much. I'll give it a try. ；）

So, I followed the advice from @mickeyouyou and succeeded in echoing the rostopics. Here are steps I did  in case someone else also has this problem:

download the ublox driver from https://github.com/KumarRobotics/ublox to the directory apollo/modules/drivers
create one function called build_ublox() similar to function build_velodyne() in apollo.sh
edit apollo/modules/drivers/ublox/ublox_msgs/CMakeLists.txt by following:

add_message_files(
  DIRECTORY msg
  FILES
  NavPOSLLH.msg
  NavVELNED.msg
  RxmRAW.msg
)

bash apollo.sh build_ublox
Then problem solved.

Further Questions:
Like I mentioned before, my goal is to visualize this rosbag in DreamView. I'm aware that the massage type is very different from Apollo, but I still want to know a possible way to do it:

How can I transfer these rosbag massages to the Apollo usable massages?
How can I make DreamView to subscribe theses messages?

Cool! thank you so much  @lygincn .

if you use --loop when rosbag play   you shall check that if your module handle the event time increasing   and you shall check apollo version at the rosbag record machine.
if you want the info show in dreamview   you shall publish the message dreamview already supported   otherwise you shall expand dreamview.
——Reply reference comes from the Apollo developer community issue master contest https://apollo.auto/developer/questiondata_cn.html?target=619


[cli] Fix, import datasources exported by UI

CATEGORY
Choose one

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Databases/Datasources export on the UI can't be imported by superset import_datasource cli.
The fix was made on import_from_dict to add the key, since YamlExportMixin is used by several ModelView's
ADDITIONAL INFORMATION

[X] Has associated issue: #8453
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@villebro

Codecov Report

Merging #8477 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #8477      +/-   ##
==========================================
- Coverage   66.52%   66.51%   -0.01%
==========================================
  Files         449      449
  Lines       22595    22598       +3
  Branches     2367     2367
==========================================
  Hits        15032    15032
- Misses       7425     7428       +3
  Partials      138      138

Impacted Files
Coverage Δ

superset/utils/dict_import_export.py
19.51% <0%> (-1.55%)
:arrow_down:

Continue to review full report at Codecov.

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

I think this should be fixed in the UI export, as currently the UI and CLI are generating different yamls.

Right, but YamlExportMixin is used in a bunch of ModelView's. So some options:

We go with this solution
Implement a new mixin something like YamlExportDatabaseMixin
implement the export right on the DatabaseModelView


Use RENDERMODE_WHEN_DIRTY

Fixes #579

Codecov Report

Merging #580 (bb683be) into main (ce4b25e) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

:exclamation: Current head bb683be differs from pull request most recent head 6ec4ebb. Consider uploading reports for the commit 6ec4ebb to get more accurate results

@@            Coverage Diff             @@
##             main     #580      +/-   ##
==========================================
- Coverage   70.79%   70.75%   -0.04%
==========================================
  Files         108      108
  Lines        5441     5444       +3
  Branches     1083     1084       +1
==========================================
  Hits         3852     3852
- Misses        849      852       +3
  Partials      740      740

Impacted Files
Coverage Δ

...composelife/ui/cells/OpenGLNonInteractableCells.kt
0.00% <0.00%> (ø)

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


Added chloenlpvlemmmmd.onion and swehackmzys2gpmb.onion

I'm away from home until later tonight, if you resubmit the pull request with the latest commit on there I can accept it from my phone.
Otherwise I'll fix the merge myself later.
Thanks for the additions

Merged.
Thanks for the additions!


Feature Line Roll - Projections & Extension methods

The geodesic default creates odd results on small segments that are not intuitive. Perhaps if the coordinate system is projected, it uses planar methods.

This is resolved and more methods to do rolling windows can be done with the tool soon.


vulkan tools: Fix image usage error reported by validation layers

VUID-VkImageCreateInfo-usage-00963 says it's invalid to use VK_IMAGE_USAGE_TRANSIENT_ATTACHMENT_BIT with VK_IMAGE_USAGE_TRANSFER_SRC_BIT.
Ran into this on linux.

CI Vulkan-Tools build queued with queue ID 5860.

CI Vulkan-Tools build # 627 running.

CI Vulkan-Tools build # 627 passed.


How can I build for WinUI3 from ui-xaml source?

I can confirm that there is a memory leak related to ScrollViewer, so I would like to investigate.

According to what was mentioned in the call yesterday, Source only for 2 namespaces *
The source will only be released for 2 namespaces microsoft.ui.xaml* and microsoft.ui.xaml.control (if I'm not mistaken).
And the release time of the source is after the release of version 1.4.2.
And more importantly, you will not be able to compile, and currently you can only read the codes

I got it.
thank you.

I got it. thank you.

also they will publish entire symbols so i think you can do something...

Currently, the source code in the repository is for WinUI 2, not WinUI 3.

also they will publish entire symbols so i think you can do something...
I would like to wait until the symbols are released to the public.

Currently, the source code in the repository is for WinUI 2, not WinUI 3.

I will wait until the WinUI3 source is released.


ACR login fails with latest version of Azure CLI

Orb version
0.1.3

What happened

The latest version of the azure-cli introduced the following change:
az acr login: Throw a CLIError if there are errors returned by docker command
This breaks the step Log into ACR with the following output:
#!/bin/bash -eo pipefail
az acr login --name $AZURE_REGISTRY_NAME_STAGING
Login Succeeded
docker: WARNING! Your password will be stored unencrypted in /home/circleci/.docker/config.json.
Configure a credential helper to remove this warning. See
https://docs.docker.com/engine/reference/commandline/login/#credentials-store

Exited with code exit status 1

Expected behavior

The docker login command should exit successfully

@poloagustin, I'm facing this problem as well. Do you have any workaround?

@AustinFelipe
Yeah, for the time being I created a custom job that installs a previous version of the cli
Here's a quick example:
executors:
  default: azure-acr/default
commands:
  my-build-and-push-image: azure-acr/build-and-push-image
  push-image:
    executor: default-executor
    steps:
      - run:
          command: |
            # Verify the CLI isn't already installed
            # Use which instead of command -v for wider coverage of envs

            if which az > /dev/null; then
              echo "Azure CLI installed already."
              exit 0
            fi

            # Set sudo to work whether logged in as root user or non-root user

            if [[ $EUID == 0 ]]; then export SUDO=""; else export SUDO="sudo"; fi

            $SUDO apt-get update && $SUDO apt-get -qqy install apt-transport-https

            if [[ $(command -v lsb_release) == "" ]]; then
              echo "Installing lsb_release"
              $SUDO apt-get -qqy install lsb-release
            fi

            # Create an environment variable for the correct distribution
            export AZ_REPO=$(lsb_release -cs)

            # Modify your sources list

            echo "deb [arch=amd64] https://packages.microsoft.com/repos/azure-cli/ $AZ_REPO main" | \
              $SUDO tee /etc/apt/sources.list.d/azure-cli.list

            if [[ $(command -v curl) == "" ]]; then
              echo "Installing curl"
              $SUDO apt-get -qqy install curl
            fi

            # Get the Microsoft signing key

            curl -L https://packages.microsoft.com/keys/microsoft.asc | $SUDO apt-key add -

            # Update and install the Azure CLI

            $SUDO apt-get update
            $SUDO apt-get -qqy install \
              ca-certificates \
              azure-cli=2.0.81-1~xenial
            echo "Azure CLI is now installed."
          name: Install Azure CLI Version 2.0.81, if not available
      - my-build-and-push-image:
          attach-workspace: <<parameters.attach-workspace>>
          checkout: <<parameters.checkout>>
          login-server-name: <<parameters.login-server-name>>
          registry-name: <<parameters.registry-name>>
          repo: <<parameters.repo>>
          tag: <<parameters.tag>>
          path: <<parameters.path>>
    parameters:
      alternate-tenant:
        default: false
        description: Use an alternate tenant. Only applicable for user logins.
        type: boolean
      attach-workspace:
        default: false
        description: |
          Boolean for whether or not to attach to an existing workspace. Default is false.
        type: boolean
      azure-password:
        default: AZURE_PASSWORD
        description: |
          Environment variable storing your Azure password. Only applicable for
          user logins.
        type: env_var_name
      azure-sp:
        default: AZURE_SP
        description: |
          Name of environment variable storing the full name of the Service
          Principal, in the form http://app-url. Only applicable for Service
          Principal logins.
        type: env_var_name
      azure-sp-password:
        default: AZURE_SP_PASSWORD
        description: |
          Name of environment variable storing the password for the Service
          Principal. Only applicable for Service Principal logins.
        type: env_var_name
      azure-sp-tenant:
        default: AZURE_SP_TENANT
        description: |
          Name of environment variable storing the tenant ID for the Service Principal. Only applicable for Service Principal logins.
        type: env_var_name
      azure-tenant:
        default: AZURE_TENANT
        description: |
          Environment variable storing your Azure tenant, necessary if
          `alternate-tenant` is set to true. Only applicable for user logins.
        type: env_var_name
      azure-username:
        default: AZURE_USERNAME
        description: |
          Environment variable storing your Azure username. Only applicable for
          user logins.
        type: env_var_name
      checkout:
        default: true
        description: |
          Boolean for whether or not to checkout as a first step. Default is true.
        type: boolean
      dockerfile:
        default: Dockerfile
        description: Name of dockerfile to use. Defaults to Dockerfile.
        type: string
      executor:
        default: default
        description: executor to use for this job
        type: executor
      extra-build-args:
        default: ''
        description: |
          Extra flags to pass to docker build. For examples, see https://docs.docker.com/engine/reference/commandline/build
        type: string
      login-server-name:
        description: |
          The Azure login server for your ACR instance, e.g. {yourregistryname}.azure.io
        type: string
      path:
        default: .
        description: |
          Path to the directory containing your Dockerfile and build context. Defaults to . (working directory).
        type: string
      registry-name:
        description: The name of your ACR registry
        type: string
      repo:
        description: A URI to an ACR login-server-name/repository
        type: string
      tag:
        default: latest
        description:
          Comma-delimited string containing docker image tags (default
          = latest)
        type: string
      workspace-root:
        default: .
        description: |
          Workspace root path that is either an absolute path or a path relative to the working directory. Defaults to '.' (the working directory)
        type: string

It's basically the same step that runs the installation in the azure-cli orb but with the version hardcoded in the apt-get :)

Awesome! Thanks, man

@poloagustin Thank you for reporting the issue! We'll look into how to resolve this and post updates.

@poloagustin @AustinFelipe An update on this issue - we are able to resolve the ACR login error by using a Docker credentials store, and should be releasing a fix to the circleci/azure-acr orb likely this or next week.

This issue should now be resolved in<EMAIL_ADDRESS>


Improve prevent to enter restricted region.

So the blocked region feature works well.
But when i enderpearl into spawn and don't move, i will just stay in there until i get out of combat.
And since the worldguard enderpearl flag will also affect non combat players, that isn't a solution for me.
I would really like if this got fixed, since this a problem.

@Superhyro This is fixed in the latest version.


Comment google_plus oauth config

It will not work unless configured and we've got several questions/issues about this, such as https://github.com/CartoDB/cartodb/issues/9696.

Retest this please


[dep] quote update w/o upgradeing workspace hack

Motivation
Upgrade quote without touching workspace hack cargo toml.
Have you read the [Contributing Guidelines on pull requests]
Y
Test Plan
CI
Related PRs
Dependabot update mistakenly upgrading workspace hack.
https://github.com/diem/diem/pull/7616

LSR dependency change summary:
host packages:
  M quote 1.0.9 (direct third-party, crates.io)
    * version upgraded from 1.0.8
    * (unchanged features: default, proc-macro)

LEC dependency change summary:
host packages:
  M quote 1.0.9 (direct third-party, crates.io)
    * version upgraded from 1.0.8
    * (unchanged features: default, proc-macro)

release binaries dependency change summary:
host packages:
  M quote 1.0.9 (direct third-party, crates.io)
    * version upgraded from 1.0.8
    * (unchanged features: default, proc-macro)

ughhhh I didn't anticipate dependabot would be so bad :(

/land


Populate db with area data (for use in toilet queries)

Instead of querying mapit for area data upon the creation of each new report we want to simply load area data into the mongo database and use geo-queries in order to determine in which area a toilet exists.
Plan

[ ] removal of existing mapit query code in report generation
[ ] sourcing area data (specifically boundary polygons, likely in geojson format) from a service such as ONS or MapIt
[ ] importing area boundary data into a mongodb collection with accompanying schema
[ ] method of updating area boundaries reliably
[ ] thought put into how queries will be structured to make use of area boundary info
[ ] thought put into how existing area info tied to loo objects will be safely removed from the db (perhaps we run a query once on live or use the existing work on migrations)
[ ] queries translated into a UI or API which allows end user to execute queries for toilets based on boundaries

Acceptance Criteria

[ ] Area boundaries are loaded into the mongo database
[ ] Area boundaries can be freshened reliably from a known source
[ ] Database can be queried to find toilets within area boundaries
[ ] Admin UI allows for users to construct queries of their own (potential to shard into another ticket? @marxian)

Useful links

MapIt

https://mapit.mysociety.org/area/2514.html, https://mapit.mysociety.org/area/2514.geojson
https://mapit.mysociety.org/legal/#licensing

ONS Digital Boundaries

https://www.ons.gov.uk/methodology/geography/geographicalproducts/digitalboundaries
https://www.ons.gov.uk/methodology/geography/ukgeographies/censusgeography

@jthistle This is an early version of the rework of areas we were talking about...
The plan needs refinement.

possible shapefile source: ONS Local Authority Boundaries, provides shapefiles for download. Available at different resolutions (full res, 20m, 500m).

how to change area system: instead of assigning loos an area by querying an external api on a report submit, query the loo area from mongo separately when querying loo details

this requires importing ONS shapefiles into mongo, via an automated script if possible

to think about:

what about an interactive explorer chloropleth map with areas on it?
How can we serve shapefiles at different resolutions for this map?
How does mongo perform with different resolutions when querying for the area that a point falls in?
What about areas of the UK that don't fall in a given ONS shapefile? Do we need a fallback system?

Links

MongoDB GeoJSON docs
All ONS local authority files
React Simple Maps

https://github.com/jthistle/areaToDatabase

https://github.com/neontribe/gbptm/tree/404-areas-from-db


Should we clearly indicate that this is only for AKS with Service Principal as this will not work with AKS with Managed Identity

[Enter feedback here]
Should we clearly indicate that this is only for AKS with Service Principal as this will not work with AKS with Managed Identity
See https://github.com/Azure/AKS/issues/4190 for further details

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

ID: 2bb3308c-4f06-1de3-3ea5-c0f445d23c8e
Version Independent ID: 440ce702-5525-b9ee-b189-f0b7181901c0
Content: Authenticate from AKS cluster to Azure container registry in different AD tenant - Azure Container Registry
Content Source: articles/container-registry/authenticate-aks-cross-tenant.md
Service: container-registry
GitHub Login: @tejaswikolli-web
Microsoft Alias: tejaswikolli

@AdelRefaat
Thanks for your feedback! We will investigate and update as appropriate.

@AdelRefaat
I'm going to assign this to the document author so they can take a look at it accordingly.
@tejaswikolli-web
please review it.

@tejaswikolli-web - feel free to close this as we already have it documented under the header, This article walks through the steps to enable cross-tenant authentication using the AKS service principal credential to pull from the container registry.

#please-close


perf: performance improvements

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

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

Your semantic-release bot :package::rocket:


Swipe on transaction list

Main screen https://zpl.io/2vOGKKr
Tap on Cancel - https://zpl.io/bPxz1kk
Tap on Repeat:
Send TX - https://zpl.io/amdEkAv.
Receive TX - https://zpl.io/bA70DAn
As we repeat transaction so amount, comment, address and fee should be already filled but user should be able to edit them if he wants.
Tap on Delete - https://zpl.io/VYrdo5p
UNDO for actions:
Cancel - https://zpl.io/bWNpXOp
Delete - https://zpl.io/aRxgGRp

Already implemented


Additional copy_services fields

https://trello.com/c/OZT4aPjn/367-copying-services-some-missing-fields
We reinstated some of the fields during the initial clone of the G12 content (https://github.com/alphagov/digitalmarketplace-frameworks/pull/599), but some got missed. This caused a bug, because we reintroduced energyEfficientDatacentres, but not energyEfficientDatacentresDescription (a dependent question).
There are also a few fields from G9 that never made it into the G11 copy list! See ticket for details.

I think the Travis build stalled during the github issue the other day (the builds all passed, but the results never got back to this PR). I've restarted it.


VS2017 linker internal error only when run under cargo

Software versions:
Windows 10
Visual Studio 2017 Professional (Microsoft (R) C/C++ Optimizing Compiler Version 19.15.26726 for x64)
cargo 1.28.0 (96a2c7d16 2018-07-13)
I started a Developer Command Prompt, cloned https://github.com/alexcrichton/git2-rs, and ran cargo build.  After a fairly lengthy build, I get a build failure:
Lib:
  C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Tools\MSVC\14.15.26726\bin\HostX86\x64\Lib.exe /OUT:"C:\Users\richard.altherr\Projects\third_party\git2-rs\target\debug\build\libssh2-sys-1e36fb8d5d3524d0\out\build\src\Debug\libssh2.lib" /NOLOGO /MACHINE:X64  /machine:x64 libssh2.dir\Debug\libssh2.res
  libssh2.dir\Debug\wincng.obj
  libssh2.dir\Debug\agent.obj
  libssh2.dir\Debug\bcrypt_pbkdf.obj
  libssh2.dir\Debug\blowfish.obj
  libssh2.dir\Debug\channel.obj
  libssh2.dir\Debug\comp.obj
  libssh2.dir\Debug\crypt.obj
  libssh2.dir\Debug\global.obj
  libssh2.dir\Debug\hostkey.obj
  libssh2.dir\Debug\keepalive.obj
  libssh2.dir\Debug\kex.obj
  libssh2.dir\Debug\knownhost.obj
  libssh2.dir\Debug\mac.obj
  libssh2.dir\Debug\misc.obj
  libssh2.dir\Debug\packet.obj
  libssh2.dir\Debug\pem.obj
  libssh2.dir\Debug\publickey.obj
  libssh2.dir\Debug\scp.obj
  libssh2.dir\Debug\session.obj
  libssh2.dir\Debug\sftp.obj
  libssh2.dir\Debug\transport.obj
  libssh2.dir\Debug\userauth.obj
  libssh2.dir\Debug\version.obj
  Microsoft (R) Library Manager Version 14.15.26726.0
  Copyright (C) Microsoft Corporation.  All rights reserved.

  "/OUT:C:\Users\richard.altherr\Projects\third_party\git2-rs\target\debug\build\libssh2-sys-1e36fb8d5d3524d0\out\build\src\Debug\libssh2.lib" /NOLOGO /MACHINE:X64 /machine:x64 libssh2.dir\Debug\libssh2.res
  LINK : the 32-bit linker (C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Tools\MSVC\14.15.26726\bin\HostX86\x64\link.exe) failed to do memory mapped file I/O on `libssh2.dir\Debug\libssh2.res' and is going to restart linking with a 64-bit linker for better throughput
  LINK : restarting link with 64-bit linker `C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Tools\MSVC\14.15.26726\bin\HostX64\x64\link.exe'

LINK : fatal error LNK1000: Internal error during ParseCommandLine [C:\Users\richard.altherr\Projects\third_party\git2-rs\target\debug\build\libssh2-sys-1e36fb8d5d3524d0\out\build\src\libssh2.vcxproj]

    Version 14.15.26726.0

    ExceptionCode            = C0000005
    ExceptionFlags           = 00000000
    ExceptionAddress         = 000CED12 (000A0000) "C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Tools\MSVC\14.15.26726\bin\HostX86\x64\link.exe"
    NumberParameters         = 00000002
    ExceptionInformation[ 0] = 00000000
    ExceptionInformation[ 1] = 00000000

  CONTEXT:
    Eax    = 00000000  Esp    = 00B9EAB0
    Ebx    = 772B5AB0  Ebp    = 00B9EAD8
    Ecx    = 03BA15A8  Esi    = 03BA15A8
    Edx    = 00000000  Edi    = 00000000
    Eip    = 000CED12  EFlags = 00010246
    SegCs  = 00000023  SegDs  = 0000002B
    SegSs  = 0000002B  SegEs  = 0000002B
    SegFs  = 00000053  SegGs  = 0000002B
    Dr0    = 00000000  Dr3    = 00000000
    Dr1    = 00000000  Dr6    = 00000000
    Dr2    = 00000000  Dr7    = 00000000
Done Building Project "C:\Users\richard.altherr\Projects\third_party\git2-rs\target\debug\build\libssh2-sys-1e36fb8d5d3524d0\out\build\src\libssh2.vcxproj" (default targets) -- FAILED.

Trying to build the libssh2-sys crate directly results in the same failure.  Now, if I run cargo build, wait for the failure, and then go into the CMake build directory (target/debug/build/libssh2-sys-/out/build) and run the exact same CMake command that cargo ran ("cmake" "--build" "." "--target" "install" "--config" "Debug" "--" as obtained via cargo -v build), the build finishes without issue.
Googling for "vs2017 failed to do memory mapped file I/O" turns up https://developercommunity.visualstudio.com/content/problem/160714/memory-error-for-linker-in-vs-155-x64.html which is a somewhat different issue (missing a 64-bit linker entirely) but the solution suggested (setting the PreferredToolArchitecture environment variable to "x64") allows cargo to build libssh2-sys.
I don't know if the bug is in cargo or vs2017's link.exe.  Something about the environment cargo runs link.exe under is causing the problem though.

The VS2017 developer command prompt now defaults to 32-bit architecture (which I find to be an odd choice).  You can run VsDevCmd.bat -arch=amd64 to switch to 64-bit mode (or set that up with a shortcut).  Presumably you are using a 64-bit installation of rust.  Or, if you really want to compile in 32-bit, the stable-i686-pc-windows-msvc rust toolchain worked for me.

I'm explicitly starting a x64 console

How is that set up?  By default (AFAIK), VS2017 does not ship with that anymore.  It looks odd that it is trying to use a 32-bit linker.  Does it compile if you start a console without any of the vs env vars?

The shortcuts we're in my start menu after installing. If I start a plain
console, the build can't find cmake.exe. keep in mind that the cmake build
works perfectly fine in the same console if I run the cmake invocations
that cargo does. It only fails when run under cargo.
On Wed, Aug 15, 2018, 6:36 PM ehuss<EMAIL_ADDRESS>wrote:

I'm explicitly starting a x64 console
How is that set up? By default (AFAIK), VS2017 does not ship with that
anymore. It looks odd that it is trying to use a 32-bit linker. Does it
compile if you start a console without any of the vs env vars?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/cargo/issues/5898#issuecomment-413394230,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJBDlxg7NC2-6FHIbslP6n9MHy2yFLreks5uRMyAgaJpZM4V-_3D
.

I typically use cmake installed from https://cmake.org/ instead of the one bundled with visual studio (just add it to your PATH).  I typically never use the visual studio env setup, it's usually not required.  Otherwise, I'm not sure, your setup seems to be different from mine.  I'm no expert on visual studio, so I'm not really sure.

Installed CMake via chocolatey and added it to PATH.  Running 'cargo build' for libssh2-sys in a regular Command Prompt and PowerShell results in the same linker error.  Setting PreferredToolArchitecture=x64 still works.


add support for hardware key (2fa) login

I use Yubikey's on my iCloud account, so on icloud.com when I sign in I'm prompted the username and password as well as a subsequent prompt from my operating system to authenticate using my Yubikey. When I try logging into the extension, it instead just prompts me for a 2-factor six-digit code, which does not exist, and I can't log in.

Duplicate of #26

Thanks for submitting this issue @henkisdabro! Just a heads up that I've been working on a change that will enable security keys. I plan to publish it by EOW.

Amazing! Can't wait!

+1

Chrome one also got the approval!

Been using it the past week, works fine.

Great, closing this.


Custom filenames

Basically, provide support for something like #54.
While this should be considered for v4, it's not necessarily going to be part of the initial v4 release.

Added in c96626b7cb28e923148ccf5b4e16e79b029b9c40.


Simplifies the service dialog

Fixes #2916
Also Fixes #5878
Changes proposed in this pull request:

The Service dialog is now divided into two dialogs in accordance to the proposal, which can be travelled back and forth easily by the user.
There separate files that deal with each of the dialogs.
The first dialog also displays the service-url  along with the service name.

@Lydiaofficial Concerning the alignment of radio buttons, maybe my proposal to have them vertically centered is not the best one: you could instead imitate what is done in the type selection part of the second step of the dialog. There, the radio button is vertically aligned with the name of the type, which also looks good to me. You might even be able to reuse the DOM structure and the CSS classes from there, to ensure those are displayed consistently.

Here are modified mock ups for the corresponding issue. The major changes are the addition of radio buttons and preselction of service history.:

Default service UI without service selection history

Selected service / preselection history.

Hyperlink

Disabled Next >> button and message

@Lydiaofficial Concerning the alignment of radio buttons, maybe my proposal to have them vertically centered is not the best one: you could instead imitate what is done in the type selection part of the second step of the dialog. There, the radio button is vertically aligned with the name of the type, which also looks good to me. You might even be able to reuse the DOM structure and the CSS classes from there, to ensure those are displayed consistently.

Your actual suggestion is the best for the placement of the radio buttons and I beforehand already did the same in the second step of the dialog .

Hey everyone,
I have worked upon most of the reviews in my recent commit, I still need to -
1)Add a message along with the disabled next button
2)add the feature of preselecting the service according to the previous history.
3)I also noticed that there is no way right now to unselect a service, we can only select a new service to unselect the previous service, should I add a way to do this?

2)I also noticed that there is no way right now to unselect a service, we can only select a new service to unselect the previous service, should I add a way to do this?

I don't think it is needed. It does not make sense to go to the next stage without selecting a service. If the user changes their mind and no longer wants to reconcile after all, they can just close the dialog.

I almost forgot to generalise the pop up message, doing that asap.

To be a little bold; do we need the radio buttons and the "next" button at all? It's not like the selection action enables any other action?

Yes it would indeed save a click if selecting the service would directly take you to the next dialog. I would be happy with that too!

I tried implementing this and I am a little confused about what would happen if the radio button is already checked with the service the user used the last time to reconcile the data with as then user has to click the same already selected radio button again.

Almost there! But I noticed a curious bug: when deleting a service, the selected service is changed too:

fixed this! have a look

I tried implementing this and I am a little confused about what would happen if the radio button is already checked with the service the user used the last time to reconcile the data with as then user has to click the same already selected radio button again.

I would imagine it without  radio buttons as each service would act as a button in itself.

@Abbe98 @lozanaross so we have two suggestions of changes from you both:

going without radio buttons and turning each service into a button, sparing us the next button
keeping radio buttons but adding a background color to the currently selected service

Because the suggestions seem to be mutually exclusive to me, is it okay for you if we treat those as follow-up improvements to be decided on and merge the PR as is?

Because the suggestions seem to be mutually exclusive to me, is it okay for you if we treat those as follow-up improvements to be decided on and merge the PR as is?

I don't consider this a blocker at all! It might, however, be nice to reach consensus in this thread. I think there should be a background color in either scenario either to indicate selection or focus/hover.

Ah, a background color on focus/hover would make a lot more sense to me, as it would also be of great help to line up the "delete" button with the name and URL of the service.

Almost there! But I noticed a curious bug: when deleting a service, the selected service is changed too:

fixed this! have a look

I tried this again and I still have the same behaviour.
The hover effect is nice, but I would tweak it a little perhaps. Instead of this current appearance:

I would prefer something like this, which would match @Lydiaofficial's design better (note the expansion of the background color to fill the entire list view):

This can be achieved by removing the padding: 10px from .recon-dialog-service-list-container and replacing it by a padding: 5px 10px on .recon-dialog-service-list-element.
Also, to my taste the color used for the hover background is too dark: it makes the endpoint poorly readable. I would expect something lighter, for instance the same color as the dialog header:

Almost there! But I noticed a curious bug: when deleting a service, the selected service is changed too:

fixed this! have a look

I tried this again and I still have the same behaviour.
The hover effect is nice, but I would tweak it a little perhaps. Instead of this current appearance:
I would prefer something like this, which would match @Lydiaofficial's design better (note the expansion of the background color to fill the entire list view):
This can be achieved by removing the padding: 10px from .recon-dialog-service-list-container and replacing it by a padding: 5px 10px on .recon-dialog-service-list-element.
Also, to my taste the color used for the hover background is too dark: it makes the endpoint poorly readable. I would expect something lighter, for instance the same color as the dialog header:

Oh yes! this is not working indeed, following up on this.

Small detail: when you reply to post on GitHub, I think it's good to avoid quoting the message if it contains images, because that duplicates the images in your new post, which makes the discussion quite lengthy and hard to read. You could edit your last message and remove the quotation, so that only your reply remains.
In our call this week, we thought that it would be better not to introduce a self._record variable to hold the current selection in Javascript, but rather to only read it when going to the next dialog. This simplifies the state of this class and should help avoid the remaining bug mentioned above, with the selection being reset whenever a service is deleted.

Great, let's merge this!


ADD: tailwind prefix to prevent override

Without a prefix set for tailwind, the configuration for this package would clash with projects importing it.  The prefix chosen is ws- standing for "wallet selector"
@mitchbne thank you for reporting this bug.

@mitchbne it looks like in this case, the existing yarn.lock was generated with yarn v1, and I have v3 locally.

Published


Add links to playground and scanner

This PR adds links to the Playground and Scanner from the Details view

I think it should always be shown as it is useful information that doesn't include debug info

@palas I suggested debug for non approved features when we implemented them rushing to the Summit. If we agree that this version can be included I don't think we should hide it behind any flag.
I will merge this today I think. Sorry for the delay

Thanks @hrajchert!


Rename upper_camel to pascal

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (a662765) compared to base (52eebb0).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #201   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            9         9
=========================================
  Hits             9         9

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

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


update generator used for sparse matrix vector product

While the default matrix class is represented as a dictionary value, when computing a matrix product, the current code runs over an entire inner product of a row of the first matrix and a column of the second matrix. If some of the entries do not exist in the dictionary they are treated as zeros, and so many unnecessary terms are computed in this inner product.
We can check that the keys (i,k) and (k,j) respectively exist in the left and right matrices of the product in the generator that gets fed to fdot. Checking if a key exists is very fast (O(1) average case) so this is a decent way to avoid iterating unnecessary elements.
I'm nots sure what types of applications people usually use this library for, so perhaps there are some cases where the overhead of checking if the keys exists will slow down some people's code. That said, I feel that the overhead will be small so it would only be noticeable small dense matrix multiplications in low precision. As soon as any sparsity is added to the matrix, the method in the pull request becomes a lot faster because the number of products of mpmath numbers is reduced.
This can easily be improved further (reduction from O(nmk) to O(nnz(self)*k)) by only looping over the nonzero entries in self, however that would change the order of some operations, and so the outputs will be different (up to machine precision). I have this implemented on my fork, but didn't want to make a pull request for a more significant change until I am more familiar with the workflow of this project.

When I test this on my local machine all the same tests fail with my changes as without them, and I'm not quite sure how to interpret the Travis CI results, so any feedback would be appreciated.

There is a test failure essentially because the matrix multiplication code would previously add zeros of type mpc to mpf terms, creating mpc's, but now that the terms are skipped, it preserves the mpf's.
It's not really a bug; to what extent the types of entries in matrices should be preserved isn't even well-defined (I wouldn't mind having separate real and complex matrix types to be honest). I'd be happy with just updating the doctest to reflect the new result.
By the way, the inner loop should probably be optimized by reading self.__data[] and other.__data[] directly instead of using self[] and other[].
(You could also store xrange(other.__rows) , self.__data and other.__data in local variables, but that probably doesn't save many cycles.)

I change this to directly access self.__data and other.__data.

Checking up on the state of this and whether the doctests will be updated.

The cost is at most 2x more expensive, since the only extra work is checking whether the key exists (which will now happen twice if it does exist). In practice, I suspect the mpf arithmetic will be the dominant cost though.
The runtime for sparse matrices is almost arbitrarily better depending on the sparsity pattern and cost of dot product of mpfloats. In the extreme case that each inner product is zero just by the sparsity pattern, then the new code will not do any mpmath evaluation. For $n\times n$ sparse matrices near diagonal, the cost of the new algorithm can be expected to be roughly $n$ vs the old algorithm which is $n^3$ (assuming the cost of mpf arithmetic is dominating lookup cost).
Here is the summary of a quick benchmark:
import mpmath as mp
mp.dps = 100

n = 200

A = mp.randmatrix(n)
B = mp.randmatrix(n)

DA = mp.diag(A[:,0])
DB = mp.diag(B[:,0])

Dense:

C = A.__mul__(B): 5.62 s ± 63 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
C = A.newmul(B): 6.73 s ± 34.3 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

Sparse:

DC = DA.__mul__(DB): 3.83 s ± 24.3 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
-DC = DA.newmul(DB): 411 ms ± 3.77 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

If mp.dps=20 and n=500:
Dense:

C = A.__mul__(B): : 1min 52s
C = A.newmul(B): 2min 12s

Sparse:

DC = DA.__mul__(DB): 1min
-DC = DA.newmul(DB): 6.33s


Refactor getDirectionFromTree

This PR refactors the planner function getDirectionFromTree.
The old version used a complicated method to determine which tree segment we are currently on and then interpolated between the nodes to find a setpoint. This logic is not adequate for some corner cases, when the planned tree yielded an endpoint that is actually closer to the drone than the start node, it would switch to goStraight mode and fly directly into an obstacle.
The interpolation also doesn't make much sense, since the output is anyway taken as a unit vector which is later scaled depending on FOV and desired velocity.
The new logic simply takes the current velocity estimate and the tree generation time to determine which tree segment we are currently on, and then returns the next node in the tree as the setpoint.
The waypoint generator creates a unit vector toward this location to put into the goto_position to comply with the rest of the logic in the waypoint_generator.

Does this PR include planning all the time or does the go straight mode still exist?  If go straight still exists how do you decide whether the current tree is valid or not?
E.g. If we have followed a nice path to its end, we do not want to stop at the last node but continue to go straight. Whereas if we get such a zigg zagg path and reach the last node (because it is basically equal to the first node) we do not want to go straight but stay at the last node.

You're right, that logic is almost dead now. The only way it will still switch to 'goStraight' is if the tree is invalid. (Too short). This is implicitly done in the local_planner by not computing a tree at all if the histogram is empty.
I've been trying to remove the 'goStraight' altogether, but my problem right now is that this removes the option to return to the mission line as well. I'd have to move that part to the planner

I've found a clever way around the issue @baumanta pointed out:
By returning i > 0 in getSetpointFromTree we achieve the desired result: a tree will be invalid if we've flown past it, and it will switch to goStraight nicely


[BUG] Is calculating Vehicle Weight

Module Version: v0.6.8
Describe the bug
Vehicle weight is being calculated when it should not be.
To Reproduce
Steps to reproduce the behavior:

Create a Vehicle Actor
Set the vehicle's Cargo Capacity to 2 Short Tons
Observe the calculated weight now reads N/4 lbs.

Expected behavior
Should not be performing weight calculations for Vehicles.
Screenshots
Vanilla FoundryVTT

With Variant Encumbrance

Browser:

Brave

Foundry Version: 0.8.9
Game System: DnD5e 1.5.3

With 0.6.9 I should have solved the problem on the vehicle character sheet, but if you didn't want to implement it I added a setting to specify the type of actor for which you want to activate the module in this case you have to change the value "character, vehicle "in" character ".
P.S. I tried to add an integration of your "Vehicle Cargo Capacity Unit" feature if you wanted to try it.

Thanks! Yeah it didn't make sense to me for vehicles since they don't have "Strength" to calculate a carry-weight, instead they just have a plain field for it. I don't mind the idea of encumbering a vehicle though, so long as it doesn't calculate it from Strength, but just just use the data field data.attributes.capacity.cargo.
Either way, I'll check it out and see how it looks :+1:

Sorry, I forgot to get back to this ticket. It all looks to be working well :+1:


3D_sphbubcollapse produces NaNs in the first timestep

Running the example case 3D_sphbubcollapse on master, setting t_step_save to 1, I get:
  Running simulation:

    $ mpirun -np 1 /home/henryleberre/CPG/MFC/build/install/bin/simulation

 Simulating a 99x99x99 case on 1 rank(s)
 [  0%]  Time step        1 of 133301 @ t_step = 0
Note: The following floating-point exceptions are signalling: IEEE_INVALID_FLAG IEEE_DIVIDE_BY_ZERO
ERROR STOP NaN(s) in timestep output.

Error termination. Backtrace:
 NaN(s) in timestep output.          46           0           0           0           1          99          99          99

Without t_step_save=1, the ICFL becomes NaN after the 8th timestep.
Environment: GNU 11.3.0 on Ubuntu (via WSL2).

I get
 [  0%]  Time step        1 of 133301 @ t_step = 0
 [  1%]  Time step        2 of 133301 @ t_step = 1
 ICFL is NaN. Exiting ...

running out of the box case.py with 8 ranks.

Changing the time step to be 1000x smaller still results in ICFL=55.9 > 1 at time step 1. There is a problem with the CFL check or the initial condition, or a bug somewhere else.

With -ffpe-trap=zero:
Program received signal SIGFPE: Floating-point exception - erroneous arithmetic operation.

Backtrace for this error:
#0  0x7f6eb991cad0 in ???
#1  0x7f6eb991bc35 in ???
#2  0x7f6eb960c51f in ???
        at ./signal/../sysdeps/unix/sysv/linux/x86_64/libc_sigaction.c:0
#3  0x56111a70cb7a in __m_rhs_MOD_s_pressure_relaxation_procedure
        at /home/henryleberre/CPG/MFC-official/src/simulation/m_rhs.fpp:1792
#4  0x56111a8da6cf in __m_time_steppers_MOD_s_3rd_order_tvd_rk
        at /home/henryleberre/CPG/MFC-official/src/simulation/m_time_steppers.fpp:390
#5  0x56111a9ca7f6 in p_main
        at /home/henryleberre/CPG/MFC-official/src/simulation/p_main.fpp:253
#6  0x56111a9cc41b in main
        at /home/henryleberre/CPG/MFC-official/src/simulation/p_main.fpp:18

It first produces a NaN on:
https://github.com/MFlowCode/MFC/blob/dae656313147fc1d34f44a6d4888eda733790a72/src/simulation/m_rhs.fpp#L1792
This implies that df_pres must be 0. I also verified this by printing it if it was zero.

I suspect this is a problem with the initial condition, which should be visualized before further code debugging. Doing a couple of code-release bisections to see where this failure started arising would probably also help. Only after that would I dig into the code that's actually running.

I visualized the initial condition, and it looks fine (though one of the patches should have reversed alpha(1) and alpha(2) I believe, though changes this doesn't fix anything).

This appears to be caused by a bug introduced here: https://github.com/MFlowCode/MFC/commit/e87333875ab3217c6ba4bd3fc0db5b9599ac17e2 @wilfonba

I believe this is because alpha_rho(x) is set to zero for one of the fluids in each patch.

The case.py file hasn't changed in a few years (according to github), so while this may (or may not) seem to fix things we should still find the breaking change.

I believe this is because alpha_rho(x) is set to zero for one of the fluids in each patch.

this did not fix the issue, with the following settings also breaking on first time step (even with 100x smaller dt than was initially in the case.py file)
    # Patch 1: High pressured water ============================================
    # Specify the cubic water background grid geometry
    'patch_icpp(1)%geometry'       : 9,
    'patch_icpp(1)%x_centroid'     : 80.E-03/1.E-03,
    'patch_icpp(1)%y_centroid'     : 80.E-03/1.E-03,
    'patch_icpp(1)%z_centroid'     : 80.E-03/1.E-03,
    'patch_icpp(1)%length_x'       : 160.E-03/1.E-03,
    'patch_icpp(1)%length_y'       : 160.E-03/1.E-03,
    'patch_icpp(1)%length_z'       : 160.E-03/1.E-03,
    # Specify the patch primitive variables
    'patch_icpp(1)%vel(1)'         : 0.E+00,
    'patch_icpp(1)%vel(2)'         : 0.E+00,
    'patch_icpp(1)%vel(3)'         : 0.E+00,
    'patch_icpp(1)%pres'           : 1.E+05,
    'patch_icpp(1)%alpha_rho(1)'   : 1000.E+00,
    'patch_icpp(1)%alpha_rho(2)'   : 1.E-10,
    'patch_icpp(1)%alpha(1)'       : 0.9E+00,
    'patch_icpp(1)%alpha(2)'       : 0.1E+00,
    # ==========================================================================

    # Patch 3: Air bubble ======================================================
    # Specify the spherical gas bubble grid geometry
    'patch_icpp(2)%geometry'       : 8,
    'patch_icpp(2)%smoothen'       : 'T',
    'patch_icpp(2)%smooth_patch_id' : 1,
    'patch_icpp(2)%smooth_coeff'   : 0.5E+00,
    'patch_icpp(2)%x_centroid'     : 0.E+00,
    'patch_icpp(2)%y_centroid'     : 0.E+00,
    'patch_icpp(2)%z_centroid'     : 0.E+00,
    'patch_icpp(2)%radius'         : 1.E-03/1.E-03,
    'patch_icpp(2)%alter_patch(1)' : 'T',
    # Specify the patch primitive variables
    'patch_icpp(2)%vel(1)'         : 0.E+00,
    'patch_icpp(2)%vel(2)'         : 0.E+00,
    'patch_icpp(2)%vel(3)'         : 0.E+00,
    'patch_icpp(2)%pres'           : 1.E+03,
    'patch_icpp(2)%alpha_rho(1)'   : 1.E-10,
    'patch_icpp(2)%alpha_rho(2)'   : 0.19E+00,
    'patch_icpp(2)%alpha(1)'       : 0.1E+00,
    'patch_icpp(2)%alpha(2)'       : 0.9E+00,
    # ==========================================================================

Using an epsilon of 1e-9 for the fluid with alpha = 0 and using the resulting alphas to define the alpha_rho's for each fluid fixes the problem. I'll see if I can get the original case to work though.

Update that this issue is "fixed" by PR #145, but that PR won't be merged until we test to see what the true breaking change was (@wilfonba).. if there was one.. maybe this case never worked.


Auto Import in C# for a ASPNET Core Project is not working

Issue Type: Bug
Auto fix and Ctrl+. and Ctrl + space is not getting me any suggestion for the import statment for example I typed ICollection to declare a variable and as previously behaviour we type CTRL + . this Auto imports the name space of the collevtion generics , but the actual behaviour was nothing Suggested
VS Code version: Code 1.27.2 (f46c4c469d6e6d8c46f268d1553c5dc4b475840f, 2018-09-12T16:17:45.060Z)
OS version: Windows_NT x64 6.3.9600

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4710HQ CPU @ 2.50GHz (8 x 2494)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.91GB (4.13GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
67%

Extensions (21)

Extension
Author (truncated)
Version

path-intellisense
chr
1.4.2

vscode-postgres
cko
1.1.3

angular2-snippets
dan
1.0.20

tslint
eg2
1.0.39

auto-close-tag
for
0.5.6

auto-rename-tag
for
0.0.15

beautify
Hoo
1.4.4

rest-client
hum
0.19.1

csharpextensions
jch
1.3.0

diff-tool
jin
0.0.1

Angular2
joh
6.2.0

postgresql
JPT
0.0.2

vscode-extension-auto-import
mop
1.3.3

mssql
ms-
1.4.0

csharp
ms-
1.16.1

team
ms-
1.136.0

debugger-for-chrome
msj
4.10.2

aspnet-helper
sch
0.6.4

vscode-auto-install
spe
1.0.1

autoimport
ste
1.5.3

save-and-run
wk-
0.0.21

Hey @Ammat-ALLAH, since the C# extension is responsible for suggestions for C#, please re-open it on the extension: https://github.com/OmniSharp/omnisharp-vscode

Hey @Ammat-ALLAH, since the C# extension is responsible for suggestions for C#, please re-open it on the extension: https://github.com/OmniSharp/omnisharp-vscode
thanks @RMacfarlane but I am already having it
I used this pack for installing
https://marketplace.visualstudio.com/items?itemName=temilaj.asp-net-core-vs-code-extension-pack
and it`s including everything needed, but I am still having the same problem, No code IntelliSense ,documentation nor auto import

Hey @Ammat-ALLAH, sorry for the confusion.
This is a problem with the C# extension, not VS Code itself. So please report the issue on the C# extension, by creating a new issue here: https://github.com/OmniSharp/omnisharp-vscode
This repository is for problems with VS Code.

Hey @Ammat-ALLAH, sorry for the confusion.
This is a problem with the C# extension, not VS Code itself. So please report the issue on the C# extension, by creating a new issue here: https://github.com/OmniSharp/omnisharp-vscode
This repository is for problems with VS Code.

Thanks, I`ll report the issue there


[T4A6][W09-B4]Vu Hai Long

Ready for Review

can use asserTrue() and assertFalse(). Please close the issue


Fix losing X11 window normal size hint properties

This was caused by XSetWMNormalHints being called multiple times, each time with different values. Calling the method replaces the old data completely, resulting in some of the settings being lost.
Since the method was called 3 times before the window was mapped, this resulted in the position hint being lost and the window always getting opened at a position determined by the WM.
I removed the id != MAIN_WINDOW_ID condition from _create_window, as it seemed unnecessary and didn't break anything. https://github.com/godotengine/godot/blob/5d880bf7db7f46b1509bbe3ed704ea8acbf8e151/platform/linuxbsd/display_server_x11.cpp#L3371
This PR also happens to remove calls to XSetNormalHints replacing them with a call to XSetWMNormalHints, as the former has been superseded by the latter.
Tested on KDE/KWin. Not tested on tiling WMs. Xtrace was a useful tool (combined with a grep) to inspect the protocol traffic.

Pinging @reduz as the code author.

Do you know if this fixes any of the reported bugs in #37734?
CC @groud and @Riteo to test on tiling WMs.

It did however reveal some issues in the X11 display server implementation that makes all the context menus and popups (probably others too) appear in the top left corner of the screen or around the main window, regardless of the requested position. I believe I have found the cause of this tonight. Will try to implement that fix and PR tomorrow.

I can't reproduce this issue myself on KDE/KWin, at least when testing with a single monitor. But in any case I tried the PR and everything seems OK, at least the X11 DisplayServer doesn't seem more buggy than it was already :P And it's a nice cleanup at any rate.

I'm on a dual screen setup (with differing resolutions) and Plasma 5.17. Not sure why it's not happening on your system, but as you mentioned it's an improvement regardless.
I don't think this fixes any bugs from that tracker directly, but for what it's worth I tested #37506 rebased onto this PR and that finally fixed the positioning issues I had with popups.

Thanks!


Hurrican Blank Screen on Retropie 4.4

Retropie 4.4 - Raspberry pi3 b +
Game starts, intro music starts, but i see only a blank screen.
runcommand.log:
Parameters:
Executing: pushd /opt/retropie/ports/hurrican; /opt/retropie/ports/hurrican/hurr
ican; popd
/opt/retropie/ports/hurrican ~
libEGL warning: DRI3: xcb_connect failed
libEGL warning: DRI2: xcb_connect failed
libEGL warning: DRI2: xcb_connect failed

The issue persists after update to RetroPie 4.6
(Raspberry Pi 3 B+)

Hurrican is considered non working with buster

I found a version of hurrican that is working fine all you have to do is extract the archive in a folder and run the executable,can you correct the hurrican port?
http://uploaded.net/file/d7wrn035
Run with ./hurrican

@ symbios24 can you describe step by step how to run it? Unfortunately, it does not start for me


Add infrastructure for writing unit tests on state machine components

Is your feature request related to a problem? Please describe.
We want to be able to write unit tests that check parts of the state machine in isolation. For instance, we'd like to be able to check individual consensus rules (does this consensus rule reject this input?), or to be able to check pre/post conditions for specific parts of the state machine (do we maintain the desired invariants after processing this input? do we write the data we expect?).
Currently, we don't have any meaningful scaffolding for writing these kinds of tests, which means that we rely solely on end-to-end integration tests.  But while end-to-end integration tests are useful to have, they're more complicated and expensive to write and run, and shouldn't be the only option for writing tests.  We should aim that the majority of the test suite is runnable by cargo test, so that most assurance can be accessed through existing Rust tooling.  Not only is this a simpler and easier development workflow -- it only requires cargo, and not other tooling -- it can also run many tests in parallel, and faster.
As a prototypical example, consider #1631, which added a test to check the consensus rule that validator definitions with duplicate consensus keys are rejected. This is a good test to have, but we don't get any assurance benefit out of having it be a full, end-to-end network integration test that:

runs pcli in a subprocess
makes network requests to a running pd instance
parses the JSON output of pcli
edits a new validator definition
assembles it into a full, valid transaction
sends that transaction over the network to a running tendermint instance
has that transaction sent over ABCI to pd
has the mempool check every other consensus rule on the transaction
has the mempool check the desired consensus rule (if it didn't already reject it)
checks that the pcli subprocess that sent the transaction errored out

instead of just checking the specific consensus rule we care about directly.  Not only is this process more expensive, if what we care about is checking that duplicate consensus keys are rejected, it's actually worse for assurance than a unit test, because we don't know that the reason the transaction was rejected was the one we're trying to test, rather than some other reason.
This isn't meant as a criticism of #1631, to be clear, and doing it as an integration test was my suggestion in #1210; the point is that at the moment, we don't have any other alternative structure to use to write this kind of test, but now that we've completed #1454, we can add one. This also isn't meant as a claim that we shouldn't have integration tests, either. Doing the full, end-to-end sequence of steps is important to ensure that none of the plumbing connecting the different parts of the system is broken.  But the goal of the network integration tests should be to test the plumbing, not all of the nooks and crannies of the state machine.
So, how should we test the state machine?
First, let's consider how it's actually structured. The Penumbra application itself is driven by Tendermint via ABCI, and (should) have its state fully and completely determined only by the ABCI messages it receives. This has important implications for testing, because we can deterministically drive the state of the application in Rust test code just by sending a sequence of ABCI messages, without having to run Tendermint at all.
ABCI messages are sent to the penumbra_component::App, which handles InitChain, BeginBlock, and EndBlock messages by calling the methods on each component, and handles DeliverTx messages by calling the ActionHandler implementation for Transaction, which performs transaction-wide checks before dispatching to the ActionHandler implementation for each action in the transaction.
At this point, it's useful to distinguish between two types of state machine tests:

state machine integration tests, testing top-level behavior of all component logic working together;
state machine unit tests, testing individual parts of the state machine.

By passing a sequence of ABCI messages to the application, we can perform state machine integration tests (and we can potentially do so much more efficiently inside Rust test code than by having Tendermint generate and send them), but this doesn't help with the goal of being able to write state machine unit tests, as in the prototypical example in #1631.  The reason is that the top-down processing will check all consensus rules together, not specific ones in isolation.
Because our state machine is written as a series of traits that compose additional behavior onto StateRead/StateWrite to ensure that all execution can be done transactionally, while we can exercise different parts of the state machine in isolation, we'll always be doing so on a State instance, as that's the only sense in which any part of the state machine can be instantiated.
One example of a state machine unit test in the existing codebase is in the IBC code's test_create_and_update_light_client. This test exercises the ICS2 client handshake implementation, by:

creates a tempdir to use as a backing store;
initializes a new Storage instance using it;
manually writes two pieces of data into the Storage that would be written as part of a full InitChain execution;
calls Ics2Client::init_chain to do the ICS2-specific state initialization;
constructs an invalid dummy transaction with IBC actions with handshake messages pulled from the cosmos hub;
pulls the dummy transaction apart to check and execute just those actions (to avoid other transaction checks);
checks that the execution succeeds.

Ideally, we'd be able to simplify this process by factoring out the setup code into common test code, with a helper method that:

creates a tempdir to use as a backing store;
initializes a new Storage instance using it;
performs full InitChain execution using default (or provided) genesis state;
hands the correctly-initialized test Storage to the caller.

We want to perform full initialization on the test storage, because our implementation assumes that the state was always correctly initialized, and we don't want our test code to violate that assumption.
Once we have a correctly-initialized test storage, we can use it for either state machine unit tests or state machine integration tests.
Describe the solution you'd like
To start, we should try to factor out the setup code from the ICS2 client unit test, and use that example as a way to check the ergonomics of the API.

A further thought: why do we need to create a full Storage instance in a tempdir? Since all our code is written in terms of StateRead+StateWrite, we could have a TestState + TestStateTransaction backed by a BTreeMap for purely in-memory testing.

This was done and two new tests were added in #1675, closing


ported most of the serpent contracts from test_contracts.py to solidity

figured since serpent is kinda being deprecated it would be nice to have all the tests in solidity (as well).
also added a few new ones for things that weren't covered yet.
test_subcall_suicider fails because https://github.com/ethereum/pyethereum/pull/360

added a bunch more;

refunds
fallback calls (contract testme { function() {} })
and some variations (including recursive entry)

Oh would love to merge once conflict fixed :)


Sensible warnings for bad bbox

Calls to tilestache-seed.py with a bad bbox return math domain error. It seems like a simple validation block could save the user some trouble.
tilestache-seed.py -b -125.0011 24.9493 -66.9326 49.5904 -c tilestache.cfg -l polygons

Traceback (most recent call last):
  File "/usr/local/bin/tilestache-seed.py", line 287, in <module>
    lr = layer.projection.locationCoordinate(southeast)
  File "/usr/local/lib/python2.7/site-packages/ModestMaps/Geo.py", line 132, in locationCoordinate
    point = self.project(point)
  File "/usr/local/lib/python2.7/site-packages/ModestMaps/Geo.py", line 119, in project
    point = self.rawProject(point)
  File "/usr/local/lib/python2.7/site-packages/ModestMaps/Geo.py", line 151, in rawProject
    math.log(math.tan(0.25 * math.pi + 0.5 * point.y)))
ValueError: math domain error

Awesome idea! Fixed in 83d5541.


Upgrade grpc to v1.19.1

Changes our grpc dep to v1.19.1, which fixes our builds using recent versions of bazel.

This change is

Do you want to update the builds to use a newer Bazel at the same time or should we leave that to a future PR?
The Kokoro Ubuntu failure is unrelated, and should be fixed with #2213.

I'll update the builds to use a newer bazel separately. Thanks.


Emulate iterat reas removal #84

Pull request for issue #84.
This implementation sets QA correctness to INCORRECT for any KB change. The correctness can be further refined (ex: when there were no rules in the initial KB and only fact addition has been performed, when the same statements have been added and removed, etc)

Coverage increased (+0.2%) to 81.549% when pulling 7ac96735c687401566a0a3c61ab4ecd920f57dac on emulate-iterat-reas-removal-#84 into 37846b01384d07c0f2ce9b03a67867edb2bbcd80 on master.

I would be more concerned about the javadoc, which says that the statement is added, not removed :-)
Best,	Ceriel

On 14 Oct 2019, at 11:17, Larry González<EMAIL_ADDRESS>wrote:
@larryjgonzalez commented on this pull request.
In vlog4j-core/src/main/java/org/semanticweb/vlog4j/core/reasoner/KnowledgeBase.java:

}

/**

Adds a single statement to the knowledge base.

@param statement the statement to be added

@return true, if the knowledge base has changed.

*/
boolean doRemoveStatement(Statement statement) {

Validate.notNull(statement, "Statement cannot be Null!");

if (this.statements.contains(statement) && statement.accept(this.removeStatementVisitor)) {

	this.statements.remove(statement);

	return true;

}

return false;

Should we use else { return false; } ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I would be more concerned about the javadoc, which says that the statement is added, not removed :-) Best, Ceriel

Thank you for catching this,
done.

I've used the auto-formatter from eclipse in several files.
I don't have suggestions for changes in classes under src/main directory
I have several suggestions for the classes under src/test directory. But they should be addressed in #96


Set max content width

Set a max width for the content to make it more readable.
Before
Lines go on really long, hard to read

After
Even on wide screen, lines are reasonable

Don't merge this yet, may not work amazing on small screens.


insertion sort in python

i have added insertion sort in python .please merge this under hacktoberfest 2022

i have added insertion sort in python .please merge this under hacktoberfest 2022

Check the sequence of the folders as I have updated it. Add Algorithm folder in Python then add Insertion Sort algo with proper description.

I have added insertion sort in algorithm folder . please merge this under hacktoberfest 2022.


cities_auto() too slow.

cities_auto() takes a long time.
I should look for ways to reduce the time it takes to load the whole dataset.

fixed, use cities_auto_lazy


Update identity.md

Updated page title to reflect content

Overview/Summary
This PR fixes/adds/changes/removes

Breaking Changes
none
Testing Evidence
N/A
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[-] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.

@tredell I see you approved and merged without the changes I noted. Please open a separate issue to track these missing parts.


Does Sorl Thumbnail Support GIFs?

Default image format
supported formats are: 'JPEG', 'PNG'. This also implicitly sets the filename extension. This can be overridden by individual options.
I have THUMBNAIL_PRESERVE_FORMAT = True and noticed sorl.thumbnail.base.EXTENSIONS contains a 'GIF' mapping... and I know my GIF is getting uploaded as a GIF, but the cache is not caching a GIF, it's caching a non-animated gif (probably the first frame).

Answer is NO! Nobody will ever answer this so if this helps anyone, I made a special case for GIFs by creating a subclass of Pil Engine.
import logging
import os.path
from io import BytesIO

from PIL import Image, ImageFile
from sorl.thumbnail import default
from sorl.thumbnail.base import ThumbnailBackend, EXTENSIONS
from sorl.thumbnail.conf import settings
from sorl.thumbnail.engines.pil_engine import Engine
from sorl.thumbnail.helpers import tokey, serialize
from sorl.thumbnail.parsers import parse_geometry

logger = logging.getLogger(__name__)

class CustomPilEngine(Engine):
    """Allows the caching of GIF files"""

    def _get_raw_data(self, image, format_, quality, image_info=None, progressive=False):
        # Increase (but never decrease) PIL buffer size
        ImageFile.MAXBLOCK = max(ImageFile.MAXBLOCK, image.size[0] * image.size[1])
        buffer = BytesIO()

        if format_ == "GIF":

            params = {
                'format': format_,
                'save_all': True,
                'loop': 0
            }

            raw_data = None

            try:
                image.save(buffer, **params)
            except OSError:
                pass
            else:
                raw_data = buffer.getvalue()
            finally:
                buffer.close()

            return raw_data

        else:

            params = {
                'format': format_,
                'quality': quality,
                'optimize': 1,
            }

            # keeps icc_profile
            if 'icc_profile' in image_info:
                params['icc_profile'] = image_info['icc_profile']

            raw_data = None

            if format_ == 'JPEG' and progressive:
                params['progressive'] = True
            try:
                # Do not save unnecessary exif data for smaller thumbnail size
                params.pop('exif', {})
                image.save(buffer, **params)
            except OSError:
                # Try without optimization.
                params.pop('optimize')
                image.save(buffer, **params)
            else:
                raw_data = buffer.getvalue()
            finally:
                buffer.close()

            return raw_data

Above solution https://github.com/jazzband/sorl-thumbnail/issues/687#issuecomment-1031836145 will only return the actual GIF to you, without doing any crop


prefer-const should be enabled?

It seems like it isn't enabled and eslint:recommended doesn't enable it either.
However, in the style guide here:

Declare all local variables with either const or let. Use const by default, unless a variable needs to be reassigned. The var keyword must not be used.

So surely this means it should be an error to use let for a variable which is not reassigned?
edit:
seems this is fixed by #49  if it gets merged.

Yeah, I've pinged the author of that PR to see if he'll sign the CLA. If not (or if he doesn't respond) I can add this myself.


Add search bar in create group screen.

Currently it's quite difficult to scroll till end and select user.

@vishwesh3 there is another PR that is WIP that combines the conversations and all users list and it has a search included.

Okay 👍

Fixed via #2123


Unable to Install Storybook Following Default Steps

This is a follow up issue to #9, where I offered to provide steps to reproduce the issue.
Because I am attempting to use this add-on along with DDEV to learn Storybook, there is a good chance it is my inexperience in configuring Storybook that is causing the issue.
Basic steps are:

Set up minimal, default DDEV php project
Install this add-on as instructed
Run the command to install Storybook as instructed
Observe spawn xdg-open ENOENT errors

Detailed steps:
Developer skills:
PHP: 8/10
DDEV: 8/10
Storybook: 1/10
Operating System: Pop!_OS 22.04 LTS
Kernel: Linux 6.9.3-76060903-generic
Architecture: x86-64
Start DDEV project

ddev config, accept all defaults
ddev start

Install addon and restart ddev

ddev get tyler36/ddev-storybook
ddev restart

Install storybook if you haven't already...

ddev exec npx storybook@latest init

Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y

   You can specify a project type explicitly via `storybook init --type <type>`, see our docs on how to configure Storybook for your framework: https://storybook.js.org/docs/react/get-started/install
? Do you want to manually choose a Storybook project type to install? › (y/N)
y

Select

React
Webpack 5

Observe error message:
/var/www/html/node_modules/storybook/bin/index.cjs:23
  throw error;
  ^
Error: spawn xdg-open ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:286:19)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess._handle.onexit (node:internal/child_process:292:12)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn xdg-open',
  path: 'xdg-open',
  spawnargs: [ 'http://localhost:6006/' ]
}

Additional Observations
Observe command dev storybook opens page with message "Bad Gateway"
Observe error with command: ddev storybook -s
><EMAIL_ADDRESS>storybook
> storybook dev -p 6006
@storybook/core v8.3.6
info => Starting manager..
info => Starting preview..
/var/www/html/node_modules/storybook/bin/index.cjs:23
  throw error;
  ^
Error: spawn xdg-open ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:286:19)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess._handle.onexit (node:internal/child_process:292:12)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn xdg-open',
  path: 'xdg-open',
  spawnargs: [ 'http://localhost:6006/' ]
}
Node.js v20.16.0
Failed to run npm run storybook: exit status 7

Clarification: the pull request I sent with #9 was my likely misguided workaround to get things working on my system.

Thank you for the steps.
I was able to confirm and get the same error.
I did some basic testing and it appears something breaks with the new versions.
I have an old project that has Storybook 8.0.8 installed.
This works fine, but if I run ddev npm update, ddev storybook -s breaks. Reverting to the old package.lock and installing older versions restores functionality.
Test
$ ddev storybook -s
><EMAIL_ADDRESS>storybook
> storybook dev -p 6006

@storybook/cli v8.0.8

info => Starting manager..
info => Starting preview..
│                                                                                │
│   Storybook 8.0.8 for html-vite started                                                  │
│   873 ms for manager and 1.03 s for preview                                              │

$ ddev npm update
added 17 packages, removed 580 packages, changed 112 packages, and audited 279 packages in 43s

$ ddev storybook -s
><EMAIL_ADDRESS>storybook
> storybook dev -p 6006

@storybook/core v8.3.6

info => Starting manager..
info => Starting preview..
/var/www/html/node_modules/storybook/bin/index.cjs:23
  throw error;
  ^

Error: spawn xdg-open ENOENT

OK ... just looking at their GitHub and found the following related issue
https://github.com/storybookjs/storybook/issues/28772

Added #13 that should fix the problem.

Will test this out later today to verify / close this issue.
Storybook bug makes perfect sense; since my Storybook skill is 1/10, I was unable to find my own solution.

TESTED: Works with an initial hiccup.
Tested with a forked release branch on my other (work) account.
The initial install still results in the spawn xdg-open ENOENT error, however the Storybook server starts and runs fine afterward.
Is there another command somewhere that would benefit from the --no-open flag?
Test steps:
ddev config

use all defaults

ddev add-on get aberry12085/ddev-storybook (forked test release of command modifications using -- --no-open)
ddev restart
ddev exec npx storybook@latest init
manually select Storybook project type to install

html
Webpack 5

Observe initial error on ddev exec npx storybook@latest init command:
/var/www/html/node_modules/storybook/bin/index.cjs:23
  throw error;
  ^
Error: spawn xdg-open ENOENT
    at __node_internal_captureLargerStackTrace (node:internal/errors:496:5)
    at __node_internal_errnoException (node:internal/errors:623:12)
    at ChildProcess._handle.onexit (node:internal/child_process:284:19)
    at onErrorNT (node:internal/child_process:477:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess._handle.onexit (node:internal/child_process:290:12)
    at onErrorNT (node:internal/child_process:477:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn xdg-open',
  path: 'xdg-open',
  spawnargs: [ 'http://localhost:6006/' ]
}
Node.js v18.20.4

Observe ddev storybook -s command successfully starts Storybook server!

Ran through the installation again and it looks like Storybook is pwning itself.
ddev exec npx storybook@latest init installs Storybook as it should
Storybook installer runs something like npm run storybook dev -p 6006 automatically, causing the "Error: spawn xdg-open ENOENT" -- with the context of the related Storybook issue  -- this may be affecting just my Linux systems.
I will try again on a Mac system I happen to have access to today.
I can also try installing xdg-utils on my Linux systems to see if that helps.

My testing show install xdg-utils inside the web contain worked. I did it manually via ddev ssh.
It's unforunate that Storybook starts its server and tries to open that page during installation.
We could add xdg-utils by customizing the docker image. This feels a little hack-y to me but perhaps its the cleaner solution.

Storybook allows install without starting the dev server: https://storybook.js.org/docs/api/cli-options#init
Adjusting the add-on documentation to ddev exec npx storybook@latest init might allow the user to sidestep the initialization fault. Then the --no-open flags can handle the DDEV command issues.
Less work and less hacky way?

Wow. Nice find. That feels much nicer.

#13 and #14 should fix this issue for me.

Thank you for the updates and your work on these!
I try and merge #13 today or tomorrow.


added ci and coverage badges

Coverage remained the same at 91.768% when pulling 64bd032ee50007fc8971e15a66bdebad90518518 on added-badges into 32ebb1bf92dabc5d6f7906e9a8cf445c619a9006 on master.


Add Downloads by Client Report to Statistics Page

This should show a graph.
The data is from on a json blob created by the worker.

There's the "NuGet Client Usage" chart on the stats page.


Export a pprintln

Hello,
I've recently started using your package and it's proving very useful when displaying the output of julia code cells on a website with JuDoc, however I noticed it doesn't skip a line after printing so this:
function foo()
  nt = (a=5,b=7,c=10)
  nt |> pprint
  nt |> pprint
end

gives
(a = 5, b = 7, c = 10)(a = 5, b = 7, c = 10)

(of course it's a silly example but basically the second time it's called, things appear on the same line).
So could their either be the introduction of a lineskip by default OR if that's not desirable, could there be a secondary function pprintln defined as:
pprintln(io, s) = (pprint(io, s); println(io))
pprintln(s) = pprintln(stdout,s)

thanks! (I'm happy to open a PR for this too)

Thanks for the suggestion. I was hesitating between making pprint() emit a trailing linebreak and adding pprintln(), but at the end decided to do the latter.


Add toggleIds on select network dropdowns for QE tests

@seanforyou23 @gildub this is a quick one if one of you wouldn't mind merging


fix: add primary to button and starUpClass and starDownClass to rating

Closes #744
📑 Description

add primary to button
starUpClass and starDownClass to rating

Status

[ ] Not Completed
[x] Completed

✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] I have checked the page with https://validator.unl.edu/
[x] All the tests have passed
[x] My pull request is based on the latest commit (not the npm version).

ℹ Additional Information

It may be a breaking changes when you did not used theme.color overrides in tailwind.config.cjs

@BCsabaEngine Please read https://flowbite-svelte.com/docs/pages/quickstart#Configuration


add local time zone for convert local date time to unix time

Debezium convert local date time to the UTC unix time. It is not right when databse use local timezone, like mysql use chinese time.
Add an environment variable called LOCAL_TIMEZONE for convet the local date time to local unix time.
You can set LOCAL_TIMEZONE="+08:00", then the long value of date time is the right chinese time.
If you not set anything, it use UTC.

@xie-shujian Thanks for the PR. Unfrotunately I don't think this should be done this way. Could you please

create a Jira issue for design discussion
change approach to use connector configuration option instead of env variable
provide a test case
make sure that works for all connectors

Now there is an open question if time datatype should be also affected etc.

@xie-shujian when you configure Debezium, are you setting database.connectionTimeZone (or driver.connectionTimeZone when using Debezium 2.x+)?


World Anchor Issues

Overview
Hello,
I cloned the mixed reality toolkit for unity and I am attempting to use world anchors for an environment in Hololens.
Expected Behavior
Ideally the app would create and remove world anchors using the airtap functionality in Hololens.
Actual Behavior
For some reason I can not find the reference for "WorldAnchorManager" so I can not build my code.
Steps to reproduce
Here is a picture of my code output.

Unity Editor Version
Mixed Reality Toolkit Release Version
Latest Master

master's WorldAnchorManager is currently part of the HoloToolkit.Unity namespace.

What version of Unity are you using?

Sounds like you're missing part of the library.  Also make sure that your build settings is targeting UWP.

I am running this on a mac using Unity 2017.3.1f1
I also have the HoloToolkit downloaded (git clone). But for some reason when I do the
using HoloToolkit.Unity.(fill in here) WorldAnchorManager does not show up.

using HoloToolkit.Unity; worked fine for me, just cloned it too.

Pretty sure the WorldAnchorManager is only availible on Windows/UWP anyway.  Make sure you switch your build target to one of those.

Closing issues older than 180 days. If this is still an issue, please reactivate with recent information.


Highlighten SQL by highlight.js

added highlight.js in order to highlight SQL preview.

Oh, so huge CSS/JS !!
I'm better at Ruby than JS, so I try to rouge instead to maintain long time. How do you think about this?

I think that Ruby way is more gem friendly than JS way, but JS way is easier than Ruby way for me.
I only want to highlight SQL syntax, and so the method does not matter.
You may close this PR.

If you are busy to do it, I will do rouge version.

Pure ruby rouge version was merged by @joker1007 himself at #65.
Thanks !!


Signal Hound closedevice command is broken by recent change

If you encounter a bug use the following template.
Steps to reproduce

Create a signal hound instrument
close the instrument

Expected behaviour
The instrument should close it's connection
Actual behaviour
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-179-10b60e3ec59a> in <module>()
----> 1 SH.closeDevice()

d:\githubrepos\qcodes\qcodes\instrument_drivers\signal_hound\USB_SA124B.py in closeDevice(self)
    168
    169     def closeDevice(self):
--> 170         self.log.info('Closing Device with handle num: ',
    171                       self.deviceHandle.value)
    172

d:\githubrepos\qcodes\qcodes\utils\helpers.py in __getattr__(self, key)
    304         raise AttributeError(
    305             "'{}' object and its delegates have no attribute '{}'".format(
--> 306                 self.__class__.__name__, key))
    307
    308     def __dir__(self):

AttributeError: 'SignalHound_USB_SA124B' object and its delegates have no attribute 'log'

Bug is most likely related to a recent change where all instrument print statements were replaced with the self.log command. However self.log was not added to all instruments.
If this has since been fixed on the master I apologize for reporting.
System
Windows 7
ATS 9870-dev-branch
**** 48cf48

This is fixed by #1154


Fix the structure of the configuration file generated by config:generate

This pull request fixes the config generated by the logchimp config:generate command

Fix the location of the created server configuration variables:

changes config.port to config.server.port
changes config.secretkey to config.server.secretkey
changes config.local to config.server.local

Add the database name to the generated configuration file
Fix the location of local when interactive flag is used

LGTM


Fixing dev

Fixed the issues that showed up when Kaksh modified the users model , should work now

Merge after your review

ok


Tabs

After discussing with @lidavidm and @TrevorEdwards,
After the current efforts are settled, we want to have two tabs:
Cafes, and Dining Halls.
Within each one, currently-open places will be at the top in alphabetical order.
Closed / almost open places will be after them in alphabetical order.

This sounds like something I could do (re: the tabs). Could I try working on this one? Would you want people to be able to swipe between the two tabs (i.e. use Swipe Views http://developer.android.com/design/patterns/swipe-views.html)?

Ah yes I would love that!! (Both the offer and the ability to swipe between the two tabs).
Thank you so much :+1:
Looking forward to it, no rush though :)

Ah by the way heads up, OTOH I'm not sure if the API returns the information of whether the dining location is a cafe or dining hall.
I think one of them might.

Quick questions: would you prefer cafes to be on the left/first or dining halls to be on the left/first?
Also would it make sense to remember which tab the user was last on and default to that, or just default to the first tab?

I would say dining halls on the left first and remember the last tab.

Yeah, I think dining halls are the slightly more common case of checking (because of changing menus), but remembering the last tab makes that choice less meaningful which is nice. So agreed with @TrevorEdwards .
Open to thoughts on it of course though :+1:

One more quick question: What is the min SDK version and target SDK version? I don't see it in the manifest.

Ah yeah it's weird, so there are two build.gradle files, one in the top level of the repo, and one here: https://github.com/genkimarshall/bigredapp-android/blob/master/app/build.gradle, which has the info you're looking for.
I'm not totally sure why there are two. Something with java-specific and android-specific, or something.
So anyway, yeah, min and target is 19 and 23 respectively.

Hm. As far as I can tell, the API doesn't actually tell you whether it's a cafe or a dining hall. How should I proceed?

Hmm, that's a bummer :/ Sorry for sending ya on a wild-goose-chase.
Okay, I guess one not-ideal solution would be to hard-code the ids for the dining halls, because it is quite uncommon for new dining places to spring up, and if one does, it's most likely for it to be a cafe and not a dining hall (The dining halls are even less likely to change). But that's definitely not a good long-term solution, so I'll make a note to discuss this with Kevin, maybe tomorrow @ the Open Source Cornell meeting.

I would have just hard-coded some kind of association data for that.  But definitely would be nicer if the API handled it for us.

New API has that info, I thought? Hopefully that gets pushed up soon.

Yeah, sorry I forgot, @lidavidm is right.
New API does have it:
https://github.com/mrkev/iroh/blob/master/open_status.coffee#L36
It is still being tested and all that but should be in production soon :+1:

@louisel Looks like we are transitioning to using the different server that has
the information now!  http://tangerine-tious.rhcloud.com/dining/
So, first step would be to change the variable BASE_URL in
DiningListFragment.java to be http://tangerine-tious.rhcloud.com/dining/.
Then, if you build, it will fail explaining there is a JSONException.
This is expected because our current code assumes the JSON at that endpoint is
a single array. So, from there, it would just be adding the tabs like you described
to put each sub-array in.
Think you still have time to do this?

Except, the number of cafes seems like it's less than it should be.
E.g. what about Libe Cafe?
Not sure what's going on there... But for now we can just move on with it.

@louisel Heads up, I pushed a fix so that the app correctly handles the new
JSONObject response (instead of the JSONArray) from the new API. This is so I
can publish this fix so the version on the Play Store is not broken xD. Feel
free to work from there or we can just handle merging later.

Since we've switched over to using now.dining.cornell.edu in https://github.com/TrevorEdwards/bigredapp-android/commit/0c257165ef02dfd49bf0a12718b8a6dbd9246540 , this isn't really an option any more for dining unfortunately.  Always open for discussion on this!


wgsl PBR fixes

Made a few small tweaks:

Removed unused clip_position from fragment input
Added a note about the omni_lights field position
Removed the reordering of OmniLight fields: this isn't needed and id actually prefer to keep "weird alignment positions" there to ensure userspace doesn't need to rely on field placement.


Deploy stage success must depend on container_status

Remove dependency on successful puppet runs. Establish dependency on successful container boot events + initializations.

fixed with #97