Modify Exists function

Modifies the Exists function so that it's applicationName, componentName similar to other functions within the code-base.

Closes: https://github.com/redhat-developer/odo/issues/416

all function in the component package has component name before application. check my comment in #416


Date.fromString failed when tried to convert string that was obtained using toString on Date.

Error: Unable to parse '<Sat Jan 14 2017 17:06:24 GMT-0500 (Eastern Standard Time)>' as a date. Dates must be in the ISO 8601 format.
The same code worked well on Chrome but failed on Firefox (50.1.0 on Windows 8)

IE 11 had the same issue

Here is a snippet of code that reproduce the issue:
addHistoryEntry : DestParty -> Date -> UserData -> UserData
addHistoryEntry destParty callDate userData =
let
dateStr = (toString callDate)
dateForVal = Date.fromString dateStr
dateStr2 = case dateForVal of
Ok _ -> dateStr
Err err -> "Got:"++ err
............................................................................................

Can you try to properly format your example code in the last comment using a Markdown code block?

Sure, I have recreate the issue with very small program.  Here it is:
import Html exposing (text)
import Date exposing (..)
import Time exposing (..)
main =
text (case
(
Date.fromTime<PHONE_NUMBER>252 |>
toString |>
Date.fromString
) of
Ok date -> toString date
Err err -> err)

Look at how your comment (also the new one) looks on GitHub. No formatting, unreadable.

Can you please look at attached file?

Please open a new issue, with all relevant information in the first comment, in readable form. Our current conversation here will just make it more complicated to see what the problem is and to act on it.

Done

@Wildsoft, please do take a look at https://github.com/elm-lang/core/issues/810#issue-200876297. I edited your comment there to put the code right there into it as a Markdown block (instead of a link only). This is how you should present examples like this in an issue if you want to make it easy for the maintainer to work with your issue (which you should want).

@jvoigtlaender , sure, thanks for your help and patience


Revert "update backup script (#912)"

This reverts commit aa6e42a56b48cd0cebab4f38801193900b6a4849.

Summary
Related issue(s)
Fixes #

/cc @clyang82


[scratchpad] fix clowntown

Motivation

replace IDE generated use self::xx with use crate::
remove unnecessary ref in a match clause.
remove unnecessary as_ref() on a ArcSwap load guard.

Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
existing coverage
Related PRs
introduced by #7957

/land

:broken_heart: Test Failed - ci-test-success

/land


[Bug]: shadcn-nuxt doesnt install any dependency

Reproduction
https://stackblitz.com/edit/nuxt-shadcn-errors
Describe the bug
I have been creating new nuxt projects and installing the shadcn-nuxt module right away, using docs steps, of course, and after adding a component and training to the project, I started getting issues that some dependencies were not installed; the first complaint is a tailwind-animate, then all the ones for clx, etc
System Info
WARN  Failed to load config ./tailwind.config.js due to the error below. Skipping..       nuxt:tailwindcss 2:12:24 PM
 Cannot find module 'tailwindcss-animate'
Require stack:
- /home/projects/nuxt-starter-agyncz/tailwind.config.js

  Require stack:
  - tailwind.config.js
  at Module._resolveFilename (https://nuxtstarteragyncz-vgwq.w-corp-staticblitz.com/builtins.ddb8d84d.js:144:12334)
  at Function.resolve (https://nuxtstarteragyncz-vgwq.w-corp-staticblitz.com/builtins.ddb8d84d.js:163:2873)
  at _resolve (node_modules/jiti/dist/jiti.js#cjs:1:241889)
  at jiti (node_modules/jiti/dist/jiti.js#cjs:1:244606)
  at eval (tailwind.config.js#cjs:1:92)
  at evalModule (node_modules/jiti/dist/jiti.js#cjs:1:247388)
  at jiti (node_modules/jiti/dist/jiti.js#cjs:1:245316)
  at eval (node_modules/tailwindcss/lib/lib/load-config.js#cjs:56:30)
  at Module.loadConfig (node_modules/tailwindcss/lib/lib/load-config.js#cjs:58:6)
  at eval (node_modules/@nuxtjs/tailwindcss/dist/module.mjs:242:75)

ℹ Tailwind Viewer: http://<IP_ADDRESS>:3000/_tailwind/                                         nuxt:tailwindcss 2:12:25 PM
ℹ Compiled plugins/server.mjs in 754.66ms                                                                  2:12:26 PM
ℹ Compiled plugins/client.mjs in 798.51ms                                                                  2:12:26 PM
ℹ Compiled types/plugins.d.ts in 816.38ms                                                                  2:12:26 PM
✔ Vite client built in 541ms                                                                               2:12:29 PM
✔ Vite server built in 1428ms                                                                              2:12:30 PM

[2:12:30 PM]  ERROR  Pre-transform error: [postcss] /home/projects/nuxt-starter-agyncz/assets/css/tailwind.css:1:1: The border-border class does not exist. If border-border is a custom class, make sure it is defined within a @layer directive.

[2:12:31 PM]  ERROR  Pre-transform error: Failed to resolve import "class-variance-authority" from "components/ui/button/index.ts". Does the file exist?

 ERROR  Pre-transform error: [@vue/compiler-sfc] Failed to resolve extends base type.                       2:12:31 PM
If this previously worked in 3.2, you can instruct the compiler to ignore this extend by adding /* @vue-ignore */ before it, for example:

interface Props extends /* @vue-ignore */ Base {}

Note: both in 3.2 or with the ignore, the properties in the base type are treated as fallthrough attrs at runtime.

/home/projects/nuxt-starter-agyncz/components/ui/button/Button.vue
5  |  import { type ButtonVariants, buttonVariants } from '.'
6  |
7  |  interface Props extends PrimitiveProps {
   |                          ^^^^^^^^^^^^^^
8  |    variant?: ButtonVariants['variant']
9  |    size?: ButtonVariants['size']

✔ Nuxt Nitro server built in 3377 ms                                                                 nitro 2:12:35 PM
ℹ Vite client warmed up in 1ms                                                                             2:12:35 PM
ℹ Vite server warmed up in 1108ms                                                                          2:12:36 PM

Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

What I found interesting was, when running npx nuxi@latest module add shadcn-nuxt, checking the package.json shows it is installing v0.10.4. Instead of v0.11.0 which is the latest as of writing. Running npx<EMAIL_ADDRESS>init will setup the project perfectly fine, however running npx shadcn-vue@latest init causes this error.

@SmokeyStack, you are right, shadcn-vue is on 0.11.0, but shadcn-nuxt is 0.10.4, at least on npm registry. even when 0.11.0 is already out here on github, I notice on Readme that even it has the nuxt color mode module installed too

Now the issue that I have is this one
❯ pnpm run dev

> nuxt-app@ dev /home/rromanve/projects/my-app
> nuxt dev

Nuxt 3.13.2 with Nitro 2.9.7                                                                                5:15:19 PM
                                                                                5:15:19 PM
  ➜ Local:    http://localhost:3000/
  ➜ Network:  use --host to expose

ℹ Using Tailwind CSS from ~/assets/css/tailwind.css                                                                                nuxt:tailwindcss 5:15:20 PM
  ➜ DevTools: press Shift + Alt + D in the browser (v1.6.0)                                                                                5:15:20 PM

ℹ Tailwind Viewer: http://localhost:3000/_tailwind/                                                                                nuxt:tailwindcss 5:15:20 PM
✔ Vite client built in 25ms                                                                                5:15:21 PM
✔ Vite server built in 401ms                                                                                5:15:21 PM

 ERROR  Pre-transform error: [@vue/compiler-sfc] Failed to resolve extends base type.                                                                                5:15:21 PM
If this previously worked in 3.2, you can instruct the compiler to ignore this extend by adding /* @vue-ignore */ before it, for example:

interface Props extends /* @vue-ignore */ Base {}

Note: both in 3.2 or with the ignore, the properties in the base type are treated as fallthrough attrs at runtime.

/home/rromanve/projects/my-app/components/ui/button/Button.vue
5  |  import { type ButtonVariants, buttonVariants } from '.'
6  |
7  |  interface Props extends PrimitiveProps {
   |                          ^^^^^^^^^^^^^^
8  |    variant?: ButtonVariants['variant']
9  |    size?: ButtonVariants['size']

✔ Nuxt Nitro server built in 435 ms                                                                                nitro 5:15:22 PM
ℹ Vite client warmed up in 0ms                                                                                5:15:22 PM
ℹ Vite server warmed up in 385ms                                                                                5:15:22 PM

Nevermind, that Primitive error is because typescript, I needed to add it as dev dependency too

This issue is annoying really. please help!!

This issue is annoying really. please help!!

I was helped by installing TypeScript into the project npm install -D typescript

I fixed my integration by adding the following dependencies to my Nuxt 3 project:

radix-vue
clsx
tailwind-merge

i can verify that issue happened to me as well at fresh install. had to install tailwindcss-animate to fix it :/

Apparently the error is that cli init does not install the dependencies. In my case I solved it by manually adding the following packages (yarn add ...):

@nuxtjs/tailwindcss
@radix-icons/vue
class-variance-authority
clsx
radix-vue
shadcn-nuxt
tailwind-merge
tailwindcss
tailwindcss-animate

and as a dependency of dev

typescript

Make sure that the tailwind.config file.
//tailwind.config.js
const animate = require("tailwindcss-animate")

/** @type {import('tailwindcss').Config} */
module.exports = {
  darkMode: ["class"],
  safelist: ["dark"],
  prefix: "",
  modules: ['radix-vue/nuxt'],
  theme: {
    container: {
      center: true,
      padding: "2rem",
      screens: {
        "2xl": "1400px",
      },
    },
    extend: {
      colors: {
        border: "hsl(var(--border))",
        input: "hsl(var(--input))",
        ring: "hsl(var(--ring))",
        background: "hsl(var(--background))",
        foreground: "hsl(var(--foreground))",
        primary: {
          DEFAULT: "hsl(var(--primary))",
          foreground: "hsl(var(--primary-foreground))",
        },
        secondary: {
          DEFAULT: "hsl(var(--secondary))",
          foreground: "hsl(var(--secondary-foreground))",
        },
        destructive: {
          DEFAULT: "hsl(var(--destructive))",
          foreground: "hsl(var(--destructive-foreground))",
        },
        muted: {
          DEFAULT: "hsl(var(--muted))",
          foreground: "hsl(var(--muted-foreground))",
        },
        accent: {
          DEFAULT: "hsl(var(--accent))",
          foreground: "hsl(var(--accent-foreground))",
        },
        popover: {
          DEFAULT: "hsl(var(--popover))",
          foreground: "hsl(var(--popover-foreground))",
        },
        card: {
          DEFAULT: "hsl(var(--card))",
          foreground: "hsl(var(--card-foreground))",
        },
      },
      borderRadius: {
        xl: "calc(var(--radius) + 4px)",
        lg: "var(--radius)",
        md: "calc(var(--radius) - 2px)",
        sm: "calc(var(--radius) - 4px)",
      },
      keyframes: {
        "accordion-down": {
          from: { height: 0 },
          to: { height: "var(--radix-accordion-content-height)" },
        },
        "accordion-up": {
          from: { height: "var(--radix-accordion-content-height)" },
          to: { height: 0 },
        },
        "collapsible-down": {
          from: { height: 0 },
          to: { height: 'var(--radix-collapsible-content-height)' },
        },
        "collapsible-up": {
          from: { height: 'var(--radix-collapsible-content-height)' },
          to: { height: 0 },
        },
      },
      animation: {
        "accordion-down": "accordion-down 0.2s ease-out",
        "accordion-up": "accordion-up 0.2s ease-out",
        "collapsible-down": "collapsible-down 0.2s ease-in-out",
        "collapsible-up": "collapsible-up 0.2s ease-in-out",
      },
    },
  },
  plugins: [animate],
}

//components.json
{
  "$schema": "https://shadcn-vue.com/schema.json",
  "style": "new-york",
  "typescript": true,
  "tsConfigPath": "./.nuxt/tsconfig.json",
  "tailwind": {
    "config": "./.nuxt/tailwind.config.cjs",
    "css": "assets/css/tailwind.css",
    "baseColor": "slate",
    "cssVariables": true,
    "prefix": ""
  },
  "framework": "nuxt",
  "aliases": {
    "components": "@/components",
    "utils": "@/lib/utils"
  }
}

For anyone coming here facing this issue, run these
yarn add class-variance-authority clsx radix-vue tailwind-merge tailwindcss-animate

yarn add -D typescript

and will fix things up


[WIP] Country Code

Add ZZ
Country explanation

Pull Request Test Coverage Report for Build 154

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

Totals

Change from base Build 152:
0.0%

Covered Lines:
79

Relevant Lines:
81

💛  - Coveralls


  Future promotions and status checks then utilize this metadata to make decisions about what to do.   This causes a huge problem because of the way this metadata is used.  People who want to use a CI/CD server to promote their akamai changes ( instead of allowing developers to run promotions on their local box which is just bad practice ) will have this metadata updates in the currently directory in which the build is happening but it will be lost and cleaned up on the next run.  The next run then has no idea that is need to create a new version..etc.. and things blow up.
Is this being actively worked on or talked about?  Could we get some assistance?  This seems like a huge blocker.
@sidph @dshafik

Yep i agree.  We also attempted to get our jenkins server to commit the metadata after the promote but it's not really ideal because now we.

We also just had trouble getting jenkins to commit the metadata since he clone's the repo in a detached head state.  Also looking online it looks like noone even does commits from the ci/cd server.

We are currently working towards an example of how you can use / set up jenkins.  You'll definitely be the first to be notified.

Hey @jnchin-james ,
Thanks soo much.  I thought about doing this but long live branches are kind of a bad practice.  Though i think this is sufficient for now if the idea is that developers don't touch the long lived branches and the jenkins pipeline is the one to handle all the maintenance.

Closing but thanks for the blog.  Will give it a read when i have time.


Update v7-feature-tracker.md

Reflects the decisions made on the 2022-11-30 TC49/TG2 telcon.

@BillWagner Please merge this.


Cannot delete the first workspace while deletion icon is active

I don't know whether this is a bug or intended behaviour, but either way something goes wrong.
The first workspace cannot be deleted like the other workspaces while it can be renamed.

If it cannot be deleted, the trash icon should not be shown in my opinion.

Ah ofcourse! Did not even notice that haha. I could also submit a PR to update the error message if you are busy with other stuff
Really like bruno so far, keep the progress going!

@bramhoven That'd be awesome.
Here is the guide to get started and bring bruno on your local
To save you some time, here is where the generic error message is being thrown and here the error gets thrown
I have some code already baked in to implement generic error handling. Its here
The way I would approach this it throw a BrunoError in workspaces actions, and then have the DeleteWorkspace use the toastError
I also, think it'd be good to update toastError to support a default error
export const toastError = (error, defaultErrorMsg = 'An error occurred') => {
  if (error instanceof BrunoError) {
    if (error.level === 'warning') {
      return toast(error.message, {
        icon: '⚠',
        duration: 3000
      });
    }
    return toast.error(error.message, {
      duration: 3000
    });
  }

  return toast.error(error.message || defaultError);

Trying to run bruno, but getting this error.
bruno on  feature/improve-error-workspace-deletion via  v14.17.0
❯ npm run dev:web

> usebruno@ dev:web C:\workspaces\bruno
> npm run dev --workspace=packages/bruno-app

npm ERR! missing script: dev

npm ERR!      C:\Users\bram.hoven\AppData\Roaming\npm-cache\_logs\2022-10-21T15_01_03_235Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! usebruno@ dev:web: `npm run dev --workspace=packages/bruno-app`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the usebruno@ dev:web script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\bram.hoven\AppData\Roaming\npm-cache\_logs\2022-10-21T15_01_03_278Z-debug.log

Currently the scripts in package.json are:
"scripts": {
  "dev:web": "npm run dev --workspace=packages/bruno-app",
  "build:web": "npm run build --workspace=packages/bruno-app",
  "dev:electron": "npm run dev --workspace=packages/bruno-electron",
  "build:chrome-extension": "./scripts/build-chrome-extension.sh",
  "build:electron": "./scripts/build-electron.sh"
}

So it is missing the dev script required to run npm run dev --workspace=packages/bruno-app

Hmm.. strange..
Can you check if the npm version is 8.x
This error is common in older npm versions that do not support workspaces
If the issue still persists, then for now, you can
cd packages/bruno-app
npm run dev

I was running version 6.14. I am upgrading now. Thanks for the quick response!

@helloanoop I created PR #39. If you have the time let me know what you think about it!
I also added that the error messages is gotten through the parseError method since that seemed specifically made for getting the correct message from an error.

Merged !
Thanks for this @bramhoven !
The fix should be deployed in the web version https://play.usebruno.com/ in the next couple of min.
Will be cutting a desktop release with these fixes tomorrow.

Great! I did see the Vercel build had failed... I was not able to see why it had failed though

Yup. Some deployment config got updated in the vercel settings.
This is fixed now. The latest build is now deployed to https://play.usebruno.com

Cool! Tested it and it seems to work 😁

@bramhoven Release v0.2.0 with your fixes is out. Thanks again for the work on this.
You can download the latest versions at https://www.usebruno.com/downloads
In case you love dark more, its included in the v0.2.0 release
Currently its based on vscode dark theme, but plan to support custom themes in the near future.


Contributors Page UI Update

Can I change contributors card to neumorphism cards?

@Abhijit2505 Can I start working on this issue?

@Abhijit2505 Can I start working on this issue?

@deepti23-web please do this.

@deepti23-web please do this.


Part 1.1: Explore New Feeds

I will choose to go with the RSS feeds of both Analyticsvidhya as well as TowardsDataScience Websites RSS feeds. I will be Using these Websites to refer for my future reading of blogs related to Data Science and more.


[BUG]

Describe the bug
Build aar1 and aar2 with Prefab feature, and aar1 depend on ffmpeg.so, aar2 depend on aar1 and use ffmpeg interface, run on android devices which os api is 5.1 and below will System.loadLibrary("aar2") failed.
java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "av_freep" referenced by "/data/app/com.example.xplayer-2/lib/arm64/libXPlayer.so"...
Devices above 5.1 will load success, if I config ffmpeg.so in CMakeLists.txt from aar2 then 5.1 and below success too.
I found aar1Config.cmake file INTERFACE_LINK_LIBRARIES is empty, if I config ffmpeg.so path here will work too.
Environment:
Prefab version: 1.1.3
Device: Oppo A59s

I don't know what I can do without a repro case. We don't have an AAR for ffmpeg so idk if the bug is with your AAR or with prefab.
https://github.com/KeepSafe/ReLinker is usually the answer to "works on newer versions of Android" though.
Closing but will reopen if we get the info we need to proceed.

thx, but ReLinker not working, not only ffmpeg, it's all the symbol in transitive libraries won't be found by aar2, include glesv3、jni-graphics ... in ndk, but if I compile by aar2.cmake depend on aa1.cmake instead of Prefab feature，Oppo A59s will loadLibrary success.
I curious about is there something different with 5.1 below and above, and how can I make ffmpeg included to exported_libraries in module.json, I can't found more info about config in gradle or other way.


Upgrades to cats 1.0.0-RC1

It also releases a new patch version.

I think this is ready for review.
I have:

Upgraded catbird-util and finch-core
Updated the travis and sbt files with the latest Scala versions
Changed the Monix documentation. The instance Task.nondeterminism has been removed from Monix. I've added a snippet about how to do parallel processing, but I'm not sure if we should remove the entire section.

@juanpedromoreno @peterneyens @raulraja

@peterneyens do you think we can move it forward?

We have to look a bit closer if we can leverage Parallel somehow (like mentioned in #463), but I think we can move forward for now.

Sounds good, this PR won't close #463


prisma init should not ask for existing schema when choosing "no existing data"

(https://github.com/graphcool/prisma/issues/new?template=bug.md)

When choosing existing database, but no existing data, the CLI should not ask for existing schema

#2407 linked

This is now available at 1.13-beta.

Fixed in 1.13.


Apk file built by buildozer is large in comparision to other apks

(Note: I am week in English so if any mistakes are found just ignore them.)
I have built some Android apps using Kivy Framework and I observed that the apk files are very large in size as compared to the apks built using Android Studio.
I know that Python is Interpreter language and requires Python Virtual machine to execute on any device.
For this problem I think I have found a solution i.e.,

An app store would be created and all the apps written in Kivy Framework could be released in this App store. This app store creates a folder in "/sdcard" as "/sdcard/.kivy" and in that folder all the standard libraries and python interpreter will be compiled and stored . As all libraries and python is already present. there is no need pack apks with Python libraries.
If the app is installed in a device where the kivy app store not installed. Then the app should download the libraries and place them in the folder "/sdcard/.kivy" so that other Kivy apps could use the same.

I understand why you would want that, but i don't think it's a realistic idea.
Making kivy app non standards and installed in a different way would limit the ability of users to install them
Sharing the python interpreter and libraries between applications would mean they all need to use the same version of all the libraries, as an app updating the libraries would potentially break other apps. This would contrain apps to old version of libraries forever.
Being less integrated in the system, kivy apps would be easily broken by system updates when android decide to restrict access to sdcard for security reasons, i'm not even sure you can load a library from the sdcard currently, as this would be a bad practice for security reasons.

I have not thought that there would be these much limitations.
Thanks for your such a early reply.
Do you have any other idea to reduce the size of my app...


Error handling examples...

...would help the README. Thanks!

Yeah I agree, will do that soon.
Though it's quite simple, fetch('/api').catch(function(err)).

updated

How would we go about handling server errors (4xx- 5xx errors) since the catch function doesn't "catch" those?

@DDChiang you don't, because 4xx-5xx are not errors in Fetch Spec.
use res.ok to easily distinguish 2xx and other responses.


runtime: lock ordering problems

#!watchflakes
post <- builder ~ `.*-staticlockranking` && log ~ `lock ordering problem`

(Pulled out from #55167, because these issues tend to be easier to diagnose; CC @golang/runtime)

cc @mknyszek these look related to pinning

They're fixed at HEAD. :) Forgot to reference the issue because the change landed before this issue was updated.

I don't understand this last failure because traceStrings < MALLOC < fin explicitly in the lock rank order...

Okay, this last one is my bad. Should've seen that coming.

I don't think we have any new ones. Moving this to Backlog.

gscan -> profInsert split to #64706.


Show warning when using incompatible node version

Description
Ticket: https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07EE00000fOo0rYAC/view
This PR makes sure to show a warning when a developer uses an incompatible node version while doing some actions (npm ci/npm install, npm start, generating a new project). The PR does 2 main things:

Making sure that all packages have defined and use consistent engines field in their package.json
Within our generator script, perform a check for the developers' node version and show a warning if it's incompatible

This PR builds upon what we already have.. there's already an existing mechanism that does this kind of check. For example, before this PR, run npm ci at the root of the monorepo and you'll encounter our script that asks whether you want to proceed or not with your incompatible node. But we found that the check is not done in some areas like the generator.
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

How to Test-Drive This PR
First, try testing the generator:

Switch to the root folder
Use an incompatible Node version: nvm use 13
Generate a project with node packages/pwa-kit-create-app/scripts/create-mobify-app-dev.js --outputDir /tmp/project
You should see a warning like this

Then with your generated project:

cd /tmp/project
Use an incompatible node: nvm use 16 ← I found version 13 to not show a warning message
npm ci
You should see a warning like this. It's the default message that comes from npm.

Also, try running the dev server with your project:

npm start
You should see a warning like this. FYI I didn't have to implement this, as it already exists.

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

@olibrook yes, good point.. better to have the root package as the source of truth for the engines and version. Ok I've made the change.


Consistency layout ( UI / UX )

Margin ( space )

Padding ( space )

Contrast ( colors )

Commit: https://github.com/buttercomponents/butter-component-settings/commit/cd7a0f0f8a76155aad903bb3199504855c349d04

Commit: https://github.com/buttercomponents/butter-component-settings/commit/45d5825ebebf6e39aed8212c46f1beb3c50eb070


MONDO:0024361 & MONDO:0003406 sleep-wake/circadian sleep disorder[Merge]

Mondo term (ID and Label)
MONDO:0003406	Sleep-wake disorder
MONDO:0024361	Circadian rhythm sleep disorder
Reason for deprecation
Not sure if this is a merge, but the definitions and synonyms on these 2 terms seem to need review; as is, they seem to be equivalent concepts.
(Also the sleep-wake disorder MONDO:0003406 term has syns of ‘sleep disorder’ which is the parent concept, so synonyms should probably be reviewed with respect to other terms in this branch.)
Term to be merged with
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Hi @kanems - We decided to keep both of these terms, but updated the definitions and synonyms to make it more clear the difference between these two terms. Also, for MONDO:0003406 sleep-wake disorder the xref was changed from: UMLS:C0877792 to: UMLS:C4042891 since this Mondo term highly references MESH:D012893 and this MESH term is mapped to UMLS:C4042891.

Thanks for following up. I agree with the CUI mapping update as well. I'll make sure our Mondo ID-CUI relationships is updated as such.


Add shape to BatchedTensor

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (07533eb) 100.00% compared to head (a65d80d) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #237   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          921       924    +3
  Branches        78        78
=========================================
+ Hits           921       924    +3

Impacted Files
Coverage Δ

src/redcat/tensor.py
100.00% <100.00%> (ø)

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


clarification on the ChIP-seq bigwig needed for training

Hi,
would it be possible to clarify what type of bigwig (coverage, fold-enrichment, -log10(pval)) is required for the ChIP-seq for training? We couldn't find that info anywhere...
Thanks in advance!

Hello @plger!
Thank you for your comment. We generate coverage tracks for ChIP-seq data. We have a snakemake pipeline that can help you generate ChIP-seq file bigwigs (SRR, SRX number of experiment to final bw), (https://github.com/MiraldiLab/snakeCHIP).
Once you have generated your bigwig files please use maxatac average to average across bioreps if you have more than one (https://github.com/MiraldiLab/maxATAC/blob/main/docs/readme/average.md).
Then you will need to normalize your data using maxatac normalize, (https://github.com/MiraldiLab/maxATAC/blob/main/docs/readme/normalize.md). They should be normalized as minmax 01 percentile 99. You should be able to find the parameters in the paper as well.

We use binary TFBS peak calls, as available from ENCODE (using IDR
analyses) or using MACS2 for non-ENCODE samples. More details available in
the Methods section of Cazares et al., 2023, PLoS Comp. Bio.
On Thu, Sep 5, 2024 at 2:04 PM FaizRizvi @.***> wrote:

Hello @plger https://github.com/plger!
Thank you for your comment. We generate coverage tracks for ChIP-seq data.
We have a snakemake pipeline that can help you generate ChIP-seq file
bigwigs (SRR, SRX number of experiment to final bw), (
https://github.com/MiraldiLab/snakeCHIP).
Once you have generated your bigwig files please use maxatac average to
average across bioreps if you have more than one (
https://github.com/MiraldiLab/maxATAC/blob/main/docs/readme/average.md).
Then you will need to normalize your data using maxatac normalize, (
https://github.com/MiraldiLab/maxATAC/blob/main/docs/readme/normalize.md).
They should be normalized as minmax 01 percentile 99. You should be able to
find the parameters in the paper as well.
—
Reply to this email directly, view it on GitHub
https://github.com/MiraldiLab/maxATAC/issues/132#issuecomment-2332343180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUJXVKMJPHDHEFRGMT4H7DZVCMLLAVCNFSM6AAAAABNV6RF3OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZSGM2DGMJYGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Emily Miraldi, Ph.D.
Associate Professor
Divisions of Immunobiology and Biomedical Informatics
Cincinnati Children's Hospital

Hello @plger!
Thank you for your comment. We generate coverage tracks for ChIP-seq data. We have a snakemake pipeline that can help you generate ChIP-seq file bigwigs (SRR, SRX number of experiment to final bw), (https://github.com/MiraldiLab/snakeCHIP).
Once you have generated your bigwig files please use maxatac average to average across bioreps if you have more than one (https://github.com/MiraldiLab/maxATAC/blob/main/docs/readme/average.md).
Then you will need to normalize your data using maxatac normalize, (https://github.com/MiraldiLab/maxATAC/blob/main/docs/readme/normalize.md). They should be normalized as minmax 01 percentile 99. You should be able to find the parameters in the paper as well.

The track values in the generated ChIP-seq training data's .bw files are all 1. In model training, is it actually only the ChIP-seq peak positions that are needed?


Fixed some lint warnings

Fixes part of #608, covered the lint warnings for the following rules:

no-unused-vars
prefer-destructuring
unicorn/no-for-loop
unicorn/prefer-includes
unicorn/prefer-string-slice
unicorn/prefer-query-selector

Signed-off-by: Vitika9<EMAIL_ADDRESS>

@psiinon

For no-negated-condition do we need to change !someBool to someBool == false?
Do we really need to use unicorn/filename-case and change almost all the file names (just confirming before I make changes)?
Please let me know if I should squash all the commits into one.

Thanks!

All seems sane to me.

Please help me with my questions.

Sorry, missed this question :(

Re no-negated-condition no, I dont think so - see https://eslint.org/docs/latest/rules/no-negated-condition
I'm fine not addressing unicorn/filename-case at this stage, its one of the less important ones imo
Please squash the commits into one

Re the test failure - thats unfortunately expected so we can ignore that.

Have you tried testing the HUD manually with these changes?
It loads for me but then none of the controls work :/
Click on the HUD buttons on ther left and right - no dialogs are shown.


Safari fullscreen border showing on link hover

This is related to #334
When Safari is fullscreen the border will show on link hover, basically when the status bar is showing.

Fixed @ master.


iOS export improvements

Just a few suggestions for improving the iOS export now that I'm using it for the first time in months :)
Biggest issue, when you don't fill in mandatory information instead of a helpful error you get a nasty "export template is corrupt" error. You can see the actual error in the log however when you start Godot as an application bundle that is hidden from view.
We need better checking before the export is being run so we can give a better message.
Other improvement would be to ensure we have default icons and splash screens. You can't do a quick export just to test if something works until you have selected icons and splash screens. It would be good that if you leave these empty, it would just use the defaults (might be a bug)

Honestly I wish the icons and splash screen were not required in the exporter at all. Would be nice to have the option to control this solely in Images.xcassets and storyboards in xcode since Apple's requirements seem to change so quickly.

This has been improved a lot over the past years.


gradlew not found

I tried to run this today and got this:
$ docker run mpl7/go4droid
docker: Error response from daemon: OCI runtime create failed: container_linux.go:296: starting container process caused "exec: \"./gradlew\": stat ./gradlew: no such file or directory": unknown.
ERRO[0000] error waiting for container: context canceled

I logged into the container with docker run --rm -i -t mpl7/go4droid bash and there is no gradlew to be found anywhere. :(

thanks, i'll have a look.

@jeffallen ./gradlew is not provided in the docker image, as it is specific to each gradle project. if the project you want to work on does not already have a ./gradlew, you're supposed to generate it with the 'gradle wrapper' command.
This is exactly what is described in https://github.com/mpl/go4droid#usage-example

Thanks, sorry I misunderstood how to use it.
I followed the instructions more carefully and got to here instead:
Unzipping /home/gopher/.gradle/wrapper/dists/gradle-2.10-bin/baigpnfu14tdk6ztbfwcl8275/gradle-2.10-bin.zip to /home/gopher/.gradle/wrapper/dists/gradle-2.10-bin/baigpnfu14tdk6ztbfwcl8275
Set executable permissions for: /home/gopher/.gradle/wrapper/dists/gradle-2.10-bin/baigpnfu14tdk6ztbfwcl8275/gradle-2.10/bin/gradle
Incremental java compilation is an incubating feature.
:app:preBuild UP-TO-DATE
:app:preDebugBuild UP-TO-DATE
:app:checkDebugManifest
:app:preReleaseBuild UP-TO-DATE
:hello:gobind FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':hello:gobind'.
> Cannot get property 'classpath' on null object

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 7.548 secs

This is probably due to golang/go#21594.

Yes, I've been seeing that as well since yesterday (when i was rechecking the steps before answering you about gradlew), and I don't know what's wrong yet.
Thanks for the tip about the golang issue!


Change the generation of the typings

A followup to #104, I converted index.ts into a TypeScript file as the generated typings lacked an entry point, and also changed the export mode for the typings to use a directory instead of a single file (necessary for the type definitions to be properly defined).

I removed the typegen code from package.json, it was still trying to overwrite everything. I published a patch: 2.2.4


Request support for QLIE 3.0

The end of the.pack file is FilePackVer3.0
Chinese characters that are not in cp932 cannot now be displayed

Vntextpatch only supports cp932. You have to custom vntextpatch for your own need, change these lines
To:
_writeEncoding = Encoding.GetEncoding(936);
return Encoding.GetEncoding(936);
Or you can just use this repos: link

Please provide a sample file (or game).

vntextpatch 只支持 cp932。您必须根据自己的需要自定义 vntextpatch，将这些行更改 为：
                _writeEncoding = Encoding.GetEncoding(936);
                return Encoding.GetEncoding(936);

或者你可以只使用这个回购协议：链接

Can I make the translated patch file export gb2312 code instead of cp932 code? Because I have many problems in using SJIS tunnel.

Can I make the translated patch file export gb2312 code instead of cp932 code?
Yes - simply change the _writeEncoding field.

However, this approach will probably result in 文字化け in the game, or even a crash. Visual novel engines often use custom code for scanning over the characters in a SJIS string. Such code won't work on GB2312 strings.
Please let me know what problems you're facing with SJIS tunneling so I can fix them.

我可以让翻译后的补丁文件导出gb2312码而不是cp932码吗？

是的 - 只需更改 _writeEncoding 字段。
但是，这种做法很可能会导致游戏中出现文字化け，甚至崩溃。视觉小说引擎通常使用自定义代码来扫描 SJIS 字符串中的字符。此类代码不适用于 GB2312 字符串。
请让我知道您在使用 SJIS 隧道时遇到的问题，以便我解决。
First of all, thank you for your reply.
Yes, using gb2312 encoding causes garbled code and other problems, but some engines can make it display gb2312 encoded text normally by modifying exe.  Because I have no C # foundation, even if I downloaded Visual Studio, it did not compile successfully.  If it is not troublesome, I hope you can share a version that can implement this.  Besides, this is a game https://vndb.org/v24211 When the game and SJIS are running together, the game will crash.
12.zip

The game is Artemis Engine

Not QLIE 3.0...? :D

vntextpatch

Sorry, not QLIE3.0.
Using vntextpatch is good for exporting and inserting text to make the game work, but it always crashes when running with SJIS. I also tried the Softpal engine. I saw that Cosetto's reply above seemed to be able to make vntextpatch export other codes, so I asked the question. Sorry for asking other questions under the question about QLIE

Crash should be fixed now.

For reference, you can change this line  to GB2312 for Chinese charset

For reference, you can change this line to GB2312 for Chinese charset

That will only affect font selection. It won't help with text encoding problems.

现在应该修复崩溃。
I downloaded the latest version of 0.33, which won't crash, but it doesn't seem to work. These characters that are not supported by cp932 are not displayed when the game is running

Turns out Artemis uses a library called "Babel" for doing the SJIS->UTF16 conversion, rather than just using the builtin Windows functionality. VNTextProxy now supports this too (v0.0.34).
To make SJIS tunneling work for this particular game, you'll also need to make a change in its scripts: edit system\msg.iet and change the function get_font_face() so it only returns .ttf/.otf files. By default, most of the entries have no extension (e.g. "image/font/FOT-AnticCezannePro-DB") - these refer to .rft files, which are Artemis "Rendered FonTs". SJIS tunneling won't work for these.

事实证明，Artemis 使用名为“Babel”的库进入 SJIS->UTF16 转换，而不只是使用内部设置的 Windows 功能。VNTextProxy现在也支持此功能 (v0.0.34)。
sjis隐道适用于游戏游戏，您需要更改其：编辑systems \ msg.iet并更改函数函数get_font_face()“ image/font/FOT-AnticCezannePro-DB”）——这些是指rft Artemis 的“Rendered FonTs”。SJIS 隐道不适合用于这些。

The game is running perfectly. Thank you for your help

请提供示例文件（或游戏）。
美少女万華鏡 -忘れな草と永遠の少女-


Dropdown is not scrollable on mobile

Hi folk.
I'm using<EMAIL_ADDRESS>It seems that dropdown menus are not scrollable on mobile : my dropdown menu is closed as soon as my finger start to swipe to scroll the dropdown, which has a fixed max-height. The problem appears on every mobile device I've tried.
All the needed code to reproduce the problem is here : https://jsfiddle.net/3ywogd9u/
That's very annoying, as it makes things like notification panels unusable. I'haven't figured out what caused this strange behavior in the lib.
Any idea / solution about it ?

Yeah, I can confirm this  - http://codepen.io/kmmbvnr/details/GNKdaP/
Dropdowns are no scrollable on iPhone Safari iOS 10.1 and Android Chrome 54

Yeah, I hate that bug :D

Fixed in 1d94110

I had the same issue after a day of head scratching I found out I was using the older version of the library.
Please check CDN version from
http://materializecss.com/getting-started.html
I am sharing code snippet on as new as 27/03/2017. Hope it helps somebody.

<meta name="viewport" content="width=device-width, initial-scale=1.0"/>

<!--Import Google Icon Font-->
<link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet">

<!--Import materialize.css-->
<!-- Compiled and minified CSS -->
<link rel="stylesheet"
      href="https://cdnjs.cloudflare.com/ajax/libs/materialize/0.98.1/css/materialize.min.css">
<link rel="stylesheet" href="https://cdn.datatables.net/1.10.13/css/jquery.dataTables.min.css">

<!--Import jQuery before materialize.js-->
<script type="text/javascript" src="https://code.jquery.com/jquery-2.1.1.min.js"></script>

<!-- Compiled and minified JavaScript -->
<script type="text/javascript"
        src="https://cdnjs.cloudflare.com/ajax/libs/materialize/0.98.1/js/materialize.min.js"></script>

Had this issue with the hover:true option. Here is my solution :
if ($(window).width() < 992) { 	$(".dropdown-button").dropdown({ 		hover:false, 		constrainWidth:false, 		belowOrigin:true 	}); } else { 		$(".dropdown-button").dropdown({ 		hover:true, 		constrainWidth:false, 		belowOrigin:true 	}); }

Had this issue with the hover:true option. Here is my solution :
if ($(window).width() < 992) { $(".dropdown-button").dropdown({ hover:false, constrainWidth:false, belowOrigin:true }); } else { $(".dropdown-button").dropdown({ hover:true, constrainWidth:false, belowOrigin:true }); }

Many thanks @lewiss444 was able to use your work-around to resolve similar challenge.


[1868] show all contact errors on provider editing

Context
Display all contact address errors.
Changes proposed in this pull request

"field" to "fields"
fields is now an array rather than a single string

 Provider editing: Contact address should get highlighted when it has an error


Issue while following instructions to install an instance locally

Expected Behavior
bundle install to run smoothly
Actual Behavior
error because installing nokogiri failed
Steps to Reproduce
After installing all prerequisites necessary on my raspberry pi 3 with "ubuntu-18.04-preinstalled-server-armhf+raspi3.img.xz" (from https://wiki.ubuntu.com/ARM/RaspberryPi), I followed the instructions and got the error at "bundle install"
Results of git log -1
I am not a coder and don't understand that, sorry.
Run git log -1 and paste the results here.
Neither that, but here is a helpful screenshot:

@derletztename Please try running the server on a traditional x86 based computer (eg: a desktop).
You will hit numerous issues trying to run a server on a Pi. Common issues reported when running a web server on a Pi:

Problems compiling C code for ARM that was intended to run on x86 devices (I suspect this is the error you are facing)
Insufficient RAM
Problems building / running Docker images (required for MQTT server)

If you want to use the farmbot off grid, you need either a really big internet flat rate (I recently learned there is a lot of data tranfer necessary just by using the farmbot) or an "offline" Web-App server.
And I can't easily put a regular PC on a field. Do you have any recommendation for a tiny x86 PC that could be used on a field as a server with reasonable pricing? One could use the UDOO (https://www.udoo.org/udoo-x86/), but it's not cheap as a raspi and I don't know if it fits your server requirements (are they listed somewhere?)

@derletztename Any x86 based processor with at least 4gb of memory at a few gigabytes of storage will suffice. You might be able to go as low as 2gb of RAM, but 4 is ideal (I've seen servers hit issues with only 2gb).
I have never worked with a Udoo board, but it looks like the "advanced" model just barely meets the specs. Another option (if you are on a very tight budget) would be to search locally for used desktop computers. There are often many available locally that will easily meet the specs and are often cheap (less than the cost of the Udoo) or free.

The issue is more to power the server and protect it from the weather on the field. It would be helpful if the server runs on 12V and fits under the solarpanel housing.
I did some research, what do you think of this pc:
https://www.zotac.com/ae/product/mini_pcs/zbox-ci325-nano
It got wifi, you can fit enough ram in and with a tiny transformer it can be used with 12V and consumes only small amounts of energy. Also it's passive cooled, so less vulnerable to the dust, etc on the field

So I bought that Zotac x86 PC and installed Ubuntu Server, the amd64 version, on it.
And I ran into the same exact problem than before. I guess it's not related to the ARM architecture then?
Could you please help me getting your software to run @RickCarlino, the plant are growing and I can't use the farmbot without the server as it is off grid and farmbot software needs more data than I can spend money on mobile data contracts. Also I wanted to modify the server to take pictures regularly and save them on that server...

@derletztename
For some reason, your system can't find ruby.h, which I don't see often on rvm based installations.
Please run the following and send me a screenshot:
lsb_release -a

rvm list

Also, how did you set up the pre-requisite packages? Did you use ubuntu_example.sh?

I wanted to modify the server to take pictures regularly and save them on that server

You won't need to modify the server for this. A regimen or farm event should work fine for such a task. Please direct any questions about this to the forum as Github is intended for software development related questions.

Thx for answering so fast. I think I made a mistake installing ruby/rvm?
I did not use ubuntu_example.sh, I don't even know what this is. I just installed package after package with apt-get install.
I did what you asked for and added the screenshot, but I guess I have to install rvm and not ruby?

After installing rvm (following ) I was able to install ruby with rvm in a different way. I guess this will fix the issue. I'll comment again when it's finished

No, after installing rvm and with that ruby 2.5.1 and doing a reboot and apt-get update, I still was not able to install with bundler. It's the exact same error :(

@derletztename Try
sudo bundle install

instead of bundle install.
If that doesn't work, try re-flashing Ubuntu and using with these instructions. It is difficult to pinpoint the exact differences between systems, so starting with a fresh installation of Ubuntu 18.04 and following the directions in that file will help eliminate some of the uncertainty.

I tried sudo bundle install, without it hadn't even started before.
So then I completely reinstalled and tried following your instructions, but failed at the very first bit, because it won't install docker this way.
I am using Ubuntu Server 18.04 LTS... do I have to switch back to Ubuntu 17 to get it running?

There is no Ubuntu 17 on the Ubuntu download site: https://www.ubuntu.com/download/alternative-downloads

@derletztename

it won't install docker this way.

Please show me the errors.

There is no Ubuntu 17 on the Ubuntu download site

Ubuntu 18 should work. I just installed on Ubuntu 18.04 this week without issues (it was a Desktop edition machine, but server edition has worked fine in the past).
I am going to take a closer look at the configuration issues today. I will let you know if I find anything.

Hey sorry I forgot to write again. So the issue with docker was I mistyped apt-key with apt-get.  But when I tried to start the server with "bundle exec rails api:start" it told me there is no bundle installed on my machine and I lost it. I gave in for the rest of the weekend and will ask an software expert tomorrow

@derletztename
Usually, when things like bundle go missing, it is because rvm is not using the correct ruby version. Make sure that RVM is on the correct Ruby version (rvm use 2.5.1) and try again.

Today a friend of mine helped me out and now we finally got it running. The issue was that line 17 has to be fitted slightly (source /usr/local/rvm/scripts/rvm) for my system. It gave a message after typing it in but I hadn't recognised. So now the webserver is finally running :)
Just one more question, how do I stop the mgtt server when it is running? We haven't found a solution other than shutting the whole pc down. And also is there a source for questions like this in your wikis? So I don't have to ask

@derletztename I'm glad you were able to fix the issue :tada:.

how do I stop the mgtt server when it is running?

To stop the MQTT process you can run:
sudo docker kill farmbot-mqtt

is there a source for questions like this in your wikis

There are a few places:

Non-technical questions / end user questions: Search the Farmbot Forum
Server / software development Raise an issue (like you did just now). Ideally, we ask that you raise one issue per question because it makes it easier for others to find answers.

We are also building an official software developer portal, but it is not ready yet :hourglass_flowing_sand: Please subscribe to the newsletter to be notified when it is published.

@derletztename do you have the web server running on your pi?


vi mode commands

Adds new functionality to vi mode using vi parser

@sholderbach I added your suggestions. Thanks, that helped a lot with characters larger than 1 char.
There is one issue that I haven't figured. When you are in a multi line edit, something like this
let a = {
sentence 1
sentence 2
sentence 3
}

if I delete the word sentence with dw the word doesnt disappear, but it isnt there any more
I checked the emacs bindings and it as well uses the same edit command (DeleteWord)
Is there something I am missing?


Fix: add 'yarn upgrade'

Summary
To be sure to found webpack, otherwise you can have an error who can do crashing portus webpack service.

Without that everytime when you start your services with docker-compose up with the docker-compose.yml at the root of the directory, an error occur on the portus_webpack service. And this fix seem to correct this error and all starting properly.
I really found this fix by hazard.

I will screenshot you the error, you will maybe more understand.
(I'm not native English speaker :/)

(I'm not native English speaker :/)

Me neither. Don't worry.
I had an issue with webpack not being found a few hours ago:
webpack_1     | Checking dependencies...
webpack_1     | yarn run v1.5.1
webpack_1     | $ webpack --watch --config config/webpack.js
webpack_1     | sh: 1: webpack: not found
webpack_1     | error An unexpected error occurred: "Command failed.
webpack_1     | Exit code: 127
webpack_1     | Command: sh
webpack_1     | Arguments: -c webpack --watch --config config/webpack.js
webpack_1     | Directory: /srv/Portus
webpack_1     | Output:
webpack_1     | ".
webpack_1     | info If you think this is a bug, please open a bug report with the information provided in "/srv/Portus/yarn-error.log".
webpack_1     | info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

but I solved it by removing the node_modules.
Your suggestion has side effects because it will indeed upgrade the packages based on the package.json constraints causing a change on yarn.lock, which is not ideal.
My guess is that it was just a coincidence but not the real fix.

It seem it need to be run just one time. I removed it from my bootstrap-webpack and it's working correctly.
I try to clean-up all containers and restart all.

I will try to update my version, but I had an issue with the new version. I will open an issue.

Ok, I'll close this one then.


Sometimes layout broken via RDP connection

After reconnected to my working desktop, the vs code window layout does not fit with my screen size and leaving a dark area. See the pic below.

And cursor focus miss-match...for example, to open the "file" menu , I have to click the dark area.

Reload window command does not solve this problem. but close the window by "X" button and open again works.

I'm having the same issue, but I'm thinking it is a problem with Electron apps in general. I have the same problem happen with Azure Data Studio (also Electron based) when working with these apps remotely via remote desktop connection and reconnecting. Restarting the app will resolve the issue but it is annoying to restart these apps after losing a remote desktop connection.

The problem exists


Allow missing values in logical fields when validating tabular data

This is regarding the R package.
Currently, when I use airr::read_rearrangement() I will get warnings like "Warning: vj_in_frame is not logical for row(s): ..." when the only values in that column are TRUE, FALSE, and NA. This stems from the fact that checks on the logical fields currently only allow TRUE and FALSE, not NA. I think that NA should be a valid value in a logical field.

Thanks for submitting this PR @kelliemac. I'm sorry this one slipped through the cracks. This looks good to me.

Thank you @javh ! I'm happy that this package exists and glad to be able to contribute in a small way.


Add pyproject.toml and replace Makefile with tox environments

Please provide description of the purpose of this pull request, as well as its
motivation and context.
Is it a new feature? A bug fix? Does it address an existing issue?

Add pyproject.toml file to configure coverage, black, and ruff. Update .gitignore to include editor folders and extra ignores provided by template-operator, and replace the top-level Makefile with environments in tox. Replacing the Makefile with environments in tox.ini makes it easier to pinpoint potential issues, and means folks have less dependencies they need to install on their system to work on the slurm charms.

How was the code tested?
Please describe the conditions under which the code has been tested.

Locally on my machine. Some work needs to be done to the CI/CD pipeline since there are errors with Juju.

Final checklist

[x] I am the author of these changes, or I have the rights to submit them.
[x] I added the relevant changed to the README and/or documentation.
[x] I self reviewed my own code.
[ ] I updated the CHANGELOG according to the contributing
guide <- Not updated since this a part of a series of updates to improve the overall testing process.

Rescinding this pull request. This repository is now archived and therefore should not be worked on. The slurm charms are now in their own individual repositories.


Build error in codellama model's word embedding layer (32016 not divisible by 64)

When I use build.py to build a code llama model, I got an error when processing word-embedding layer.
The cause is that the word-embedding size of code llama is 32016 which can not be divisible by 64. The script padded its newly initialized word-embedding matrix to be divisible by 64, but it did not change the original word-embedding matrix. Therefore the script tried to fit a matrix of shape 32064,hidden_size into 32016,hidden_size.
I believe there exists a bug when building lm_head for code llama in build.py as well.
I fixed the bug by expand the code llama's word-embedding to 32064 and save it in advance, then process it in build.py.
Everything works perfectly after that.

@CaesarWWK, thank you for the report. Can you create a PR to introduce how do you fix this issue? It is welcome to contribute to TensorRT-LLM.

@CaesarWWK, thank you for the report. Can you create a PR to introduce how do you fix this issue? It is welcome to contribute to TensorRT-LLM.

Hi,
Sorry for mistaken, I actually took a "workaround" but not "fix" the bug. I tried to fix the bug but it's more complicated than I thought.
For the workaround, I load the code llama in advance then call model.resize_token_embeddings(32064) to resize the word-embedding related layers. Then I save the model weights and use the new weights to build engines. Everything works well.
I will try to fix the bug again, any update will be reported to this thread.
Thank you

@CaesarWWK, thank you for the report. Can you create a PR to introduce how do you fix this issue? It is welcome to contribute to TensorRT-LLM.

Hi, Sorry for mistaken, I actually took a "workaround" but not "fix" the bug. I tried to fix the bug but it's more complicated than I thought.
For the workaround, I load the code llama in advance then call model.resize_token_embeddings(32064) to resize the word-embedding related layers. Then I save the model weights and use the new weights to build engines. Everything works well.
I will try to fix the bug again, any update will be reported to this thread. Thank you

Thank you, Can you tell your specific steps?
Do you take a torch load model and resize it, then save the weights, then quantize it with AWQ, and then build it as tensorrt?

You can try below:

Use the latest main branch.
Just comment out the assert https://github.com/NVIDIA/TensorRT-LLM/blob/main/tensorrt_llm/parameter.py#L80-L82
Change this line https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/llama/weight.py#L1163 to if v.shape[0] % 64 != 0:
Change this line https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/llama/weight.py#L1173 to torch.nn.functional.pad(v[1], [0, 0, 0, 64 * 32 - v[1].shape[0] % (64 * 32)],

I pushed the MR and may be merge in few days, you can try using instruction above, it works from my side.

@CaesarWWK, thank you for the report. Can you create a PR to introduce how do you fix this issue? It is welcome to contribute to TensorRT-LLM.

Hi, Sorry for mistaken, I actually took a "workaround" but not "fix" the bug. I tried to fix the bug but it's more complicated than I thought.
For the workaround, I load the code llama in advance then call model.resize_token_embeddings(32064) to resize the word-embedding related layers. Then I save the model weights and use the new weights to build engines. Everything works well.
I will try to fix the bug again, any update will be reported to this thread. Thank you

Thank you, Can you tell your specific steps? Do you take a torch load model and resize it, then save the weights, then quantize it with AWQ, and then build it as tensorrt?

Yes, exactly as you said.

@CaesarWWK @shatealaboxiaowang Hi, You can try below:

Use the latest main branch.
Just comment out the assert https://github.com/NVIDIA/TensorRT-LLM/blob/main/tensorrt_llm/parameter.py#L80-L82
Change this line https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/llama/weight.py#L1163 to if v.shape[0] % 64 != 0:
Change this line https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/llama/weight.py#L1173 to torch.nn.functional.pad(v[1], [0, 0, 0, 64 * 32 - v[1].shape[0] % (64 * 32)],

I pushed the MR and may be merge in few days, you can try using instruction above, it works from my side.

@Tracin  Tried this on main branch with codellama-13b-instruct, it doesn't work. On main branch, the original error message is RuntimeError: Padding length too large. With your modify, the error message just the same like release branch AssertionError: The value updated is not the same shape as the original. Updated: (32064, 5120), original: (32016, 5120), which means I need to modify the vocab_size of model.

@gesanqiu Hi, The file parameter.py in step two is probably installed in the environment, We actually comment out the assert, so you will not get this AssertionError anymore.

@gesanqiu Hi, The file parameter.py in step two is probably installed in the environment, We actually comment out the assert, so you will not get this AssertionError anymore.

Sorry for misinformation, I know that I need to comment the assertion lines, what I mean is that I didn't meet vocab_size issue on main branch, so I don't see the necessity of your modify. And the issue on main branch is RuntimeError: Padding length too large.

@CaesarWWK @shatealaboxiaowang Hi, You can try below:

Use the latest main branch.
Just comment out the assert https://github.com/NVIDIA/TensorRT-LLM/blob/main/tensorrt_llm/parameter.py#L80-L82
Change this line https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/llama/weight.py#L1163 to if v.shape[0] % 64 != 0:
Change this line https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/llama/weight.py#L1173 to torch.nn.functional.pad(v[1], [0, 0, 0, 64 * (v[0].shape[1] // group_size) - v[1].shape[0] % (64 * (v[0].shape[1] // group_size))],

I pushed the MR and may be merge in few days, you can try using instruction above, it works from my side.

main branch seem has been modify, the code line is mismatch, can you show you pr link


[SoC2020] Performance Optimization

性能优化Performance optimization

What is SoC2020: #1599

描述 Description
AntV G6 是关于开源图可视化引擎。性能一直是前端图可视化瓶颈。
G6 的渲染性能在优秀的底层渲染引擎的支持下，已经有了长足的进步，但在 G6 层面仍有很大的成长空间。当渲染超过 2w+ 节点时，渲染和交互上会出现一定的性能问题，随着数据量的增加，G6 的性能成为瓶颈。参考资料：

G6 文档及教程；
G6 性能测评；
G6 GitHub。

AntV G6 is an open source graph visualization engine. Performance has always been a problem for graph visualization with browsers.
Supported by a excellent rendering engine, the rendering performance of G6 has been improved a lot. But there is still a lot of room. When rendering exceeds 20,000 nodes, certain performance problems will occur in rendering and interaction. As the amount of data increases, G6 performance becomes a bottleneck.

The documents of G6;
Performance of G6;
GitHub of G6.

目标 Goal
实现一套完整的性能优化方案，使得 G6 能够支持 10w+ 节点数量级的图的合理展示及实时交互，帧率达到 30FPS 以上。

设计和实现一套渲染、交互的优化方案；
给出介绍优化方案的文档；
提供验证方案可行性与性能提升程度的实验结果；
提供可运行的 Demo ，兼容 Chrome、Firefox、Safari 及 IE11 等主流浏览器。

提示：

可以有合理的过滤筛选、聚合、裁剪等操作；
实验数据可以使用 G6 性能测评 中的公开数据集。

Achieve a complete set of performance optimization solutions, so that G6 can support large data with 100 thousand nodes rendering and real-time interaction, the frame rate reaches  30FPS.

Design and implement the performance optimization solution in G6;
Provide the documents about your solution;
Provide the evaluation to show the feasibility of the solution and the degree of performance improvement;
Provide a working Demo, compatible with mainstream browsers such as Chrome, Firefox, Safari and IE11.

Tips:

There can be reasonable filtering, aggregation, cropping and other operations;
You can use the public datasets in Performance of G6.

需要技能 Prerequisite Skills

JavaScript、Canvas 等前端基础技能；
图可视化基础理论知识。

JavaScript, Canvas and front-end engineering skills;
Basic theory knowledge of graph visualization.

Unfortunately, no one's proposal passed the committee's selection. But this issue is still of great concern to us, and we still need help and co-construction in this direction. Interested friends can continue to contact us, although not as an ASoC project, but still a good open source cooperation task.

这个题目做的怎么样了

这个题目还需要做吗，想学习下g6的

目前 SoC 已经结束了。但是 G6 作为开源产品，我们还是非常欢迎社区共建 G6 的！

这个题目还需要做吗，想学习下g6的

性能问题还是G6长期来看要持续优化的问题，这个课题仍然是需要做的，欢迎提供想法和建议！


Remove "unofficial" from homebrew

If homebrew way is unofficial, then downloading binary from forum/github is unofficial as well

Why is it not showing up? http://braumeister.org/search/minetest

Because it is in alternative (but still official) repository: http://braumeister.org/repos/Homebrew/homebrew-games/formula/minetest

It can't really be said that its official, as homebrew is a third party app repo to osx. Also, its not part of the "official" binaries on github. But its not really unofficial either, at least its not more unofficial than the forum link above, so :+1: .
Probably it would be great if above link were included as well.

btw @neoascetic can I list you as maintainer here: http://dev.minetest.net/Releasing_Minetest#Notify_other_known_package_maintainers
How can you best be reached? freenode? email?

Yeah, no problem. Personally, I prefer email.

And I don't think it would be helpful to have link to braumeister since it is even less official than everything else :D

So

21a50be7d6aedcb136a4bea56e4474a5c5703262


Placeholder component wave animation not working in Edge

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Hi,
I'm using Bootstrap 5.2.3 in my Vite project. Installed based on Vite Getting started guide from the docs approximately a year ago.
Everything has been working great so far, except for the placeholder-wave animation.
I noticed that the mask attributes located in the placeholder-wave class have -webkit prefixed variants on the docs site where the animation works for me in the same browser, but are missing on my site.
As soon as I add the prefixed variants the animation starts working.
Any idea what might be wrong? Why are these prefixes not added to the bootstrap final css on my site?
It's like I'm not getting Autoprefixer sanitized Bootstrap css on my website. But I thought Bootstrap.scss I'm referencing from the Bootstrap package is already supposed to be sanitized?
Reduced test cases
v5 template:

Bootstrap v5 Bug Report

Here it works so I'm not sure what I'm doing wrong on my site. Followed the Vite Getting started guide to bit.
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Microsoft Edge
What version of Bootstrap are you using?
v5.2.3

1
2
3

I think you need Autoprefixer. We use it under the hood to keep vendor prefixes up-to-date based on supported browsers (defined in .browserlistrc).

Oooh... I thought it's somehow being used by Bootstrap npm package to postprocess bootstrap code so I don't have to integrate it (autoprefixer) at all.
So it's not like there is a bug in Bootstrap and it's just problem on my end by not using Autoprefixer? Hence all Bootstrap code is without any vendor prefixes on my end?

With Vite I just added the following into the vite.config.ts and it works now.
  css: {
    postcss: {
      plugins: [autoprefixer()],
    },
  },

I haven't seen too many issues on the missing Autoprefixer, let's leave things as they are unless there's a recommendation. Happy to accept new PRs.


Build "Starter" JSON Template to Display Resource Strings Pulled from YAML File

This is a sub-task to #4, and is still in the "exploratory"/WIP phase.  If we're going to make our YAML work in a reasonable fashion (or a DB for that matter), we need to think through what we are going to template, how we are going to template it, and where we put the templates and the templating logic.   And this may or may not be the right strategy.  Please see the discussion under #4 for additional considerations - and feel free to chime in!
Current proof of concept is to use template strings, and pull various pieces of info from a YAML file of resources.  The resource file would be filtered by Topic/level, and the template(s) would pull in the needed info and make the JSON necessary to send out the message.
Attached is a (not yet working) copy of a template.  Suggestions welcome.
let message = {
      token: process.env.SLACK_TOKEN,
      as_user: true,
      link_names: true,
      mrkdwn_in: ['text', 'pretext'],
      attachments: JSON.stringify([
        {
          pretext: '${language-icon}  ${language-desc}\n\n\n'
          title: '\n${level} ${language}\n\n',
          text:  '${video-link}  ${video-desc}\n\n${tutorial-link}  ${tutorial-desc}\n\n${book-link}  ${book-desc}\n\n${class-link}  ${class-desc}\n\n${help_link}\n\n${more-questions}\n\n\n${maintainer}',
          color: '${sidebar-color}',
        }]),
    };

Fixed by @stain88 in his /resources pull request.


EDGE SCRIPT1028: SCRIPT1028: Expected identifier, string or number

Environment
Knex version: 0.20.4
Database + version:
OS: windows 10
Bug

Errors on Edge trying to run my Nextjs application

2.SCRIPT1028: SCRIPT1028: Expected identifier, string or number (client.js)

initializePool(config = this.config) {
if (this.pool) {
this.logger.warn('The pool has already been initialized');
return;
}
const tarnPoolConfig = {
...this.getPoolSettings(config.pool), <-- on this line
};
// afterCreate is an internal knex param, tarn.js does not support it
if (tarnPoolConfig.afterCreate) {
delete tarnPoolConfig.afterCreate;
}
this.pool = new Pool(tarnPoolConfig);
},

Please add steps to reproduce the error. Doesn't sound like a knex problem. Can be re-opened if there are any reason to believe that this is a bug in knex.


BabyBear shift machine

Basic tests passed. Still working on the rest :)
Operations:

shl<0> A1, A2, B -> C1, C2
shr<1> A1, A2, B -> C1, C2
A1 A2 are 16 bit limbs of 32 bit A in little-endian order. Likewise for C1 and C2

Implementation:

We adopted a similar implementation to our prior shift machine, which decomposes A to 4 bytes and looks up each byte to a lookup table of [A_byte, B (shift amount), block row, operation id], so the size of the lookup table is [256, 32, 4, 2] = 65536. Each row looks up to the resulting byte after the bit shifting, and the results are added together to obtain C.
In our design, instead of looking up to 32-bit C column, we are looking up to two 16-bit C1 and C2 columns. Overall, there are more witness columns due to decomposing to 16-bit limbs the in the main shift machine and one more fixed column in the lookup table, but the same number of lookups performed.

Future optimization:

There's ample ground for "reshaping" the main machine to have more columns but fewer rows, and do more lookups in each row so that we are not just processing one byte in each row. In the most aggressive case, we can even process everything in the same row.
For example, processing two bytes in one row (instead of one byte) should have half the number of rows but less than twice the number of columns, and therefore fewer cells in total. This should be good for provers with time linear to the number of cells.

@chriseth All comments fixed. Ready for a final review. :)


docs: correct the descriptions about the key of filters in Basic Search

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7984

Relevant Lines:
9268

💛  - Coveralls


Client should track current broker leader

@benbjohnson
This may not be a complete implementation, but should be an improvement.

Looks good, can you add a test?

Cool, let me look into a test now.

@benbjohnson -- can you take another look? I added tests, and also fixed a bug -- the entire redirect URL was getting set in the client, when it should only have been the host (and port if present).

One comment on the URL copy but otherwise lgtm.

Thanks for the feedback @benbjohnson -- made that one final change. I will merge on green.

Green build, merging now.


DiceBear works without this fork in the browser

DiceBear works without this fork in the browser. If you have problems with the original package, let's find a solution together.

Add me on discord +_player#5016 so we can discuss further.

Sorry, but I only provide support in the official repository so that the community can benefit from discussed problems and solutions.
Feel free to use the GitHub discussions to describe your problems:
https://github.com/dicebear/dicebear/discussions
And please correct the license. Calling you the author is pretty cheeky.
https://github.com/hustLer2k/dicebear-browser/blob/main/LICENSE.MD

Thanks for the quick fix on the license.
My guess is that you ran into a problem with DiceBear in the context of your "pimp" project. (Interesting project, by the way.) I'm looking into using DiceBear in the context of Next.js. I could imagine a problem in SSR mode.

That's right. The project fails to compile with the following errors:
./node_modules/@dicebear/converter/lib/node/core.js:36:26
Module not found: Can't resolve '@resvg/resvg-js'

https://nextjs.org/docs/messages/module-not-found

Import trace for requested module:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./src/app/account/setup/Previewer.tsx
./src/app/account/setup/page.tsx

./node_modules/@dicebear/converter/lib/node/core.js:57:0
Module not found: Can't resolve 'exiftool-vendored'

https://nextjs.org/docs/messages/module-not-found

Import trace for requested module:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./src/app/account/setup/Previewer.tsx
./src/app/account/setup/page.tsx

The component where the error is thrown if dicebear is used - https://github.com/hustLer2k/pimp/blob/main/src/app/account/setup/Previewer.tsx.
If I disable ssr it works though

Should I create an issue in the dicebear repo?

Okay, I'm going to try again directly with your project. There has to be a difference somewhere. :)

You use the experimental "app" directory in your Next.js application.
https://beta.nextjs.org/docs/app-directory-roadmap
This doesn't seem to work with dependencies that provide a version for the browser and a version for Node.js.
https://github.com/dicebear/dicebear/blob/main/packages/%40dicebear/converter/package.json#L19-L24
NextJS tries to package the frontend build with the server dependencies here as well, unlike the old "pages" directory. This does not work.
Unfortunately, I haven't found a bug in Next about this yet. It would probably have to be created with an MWE in the Next.js project.
Workaround:
// next.config.js
const path = require('path');

/** @type {import('next').NextConfig} */
const nextConfig = {
  experimental: {
    appDir: true,
  },
  webpack: (config, { isServer }) => {
    config.resolve.alias['@dicebear/converter'] = path.resolve(__dirname, 'node_modules/@dicebear/converter/lib/index.js');

    return config;
  }
}

module.exports = nextConfig

Thanks, will test it out.

It works, thanks. This package is now deprecated and will be removed soon.

This workaround doesnt seem to work for me.
My packages
dependencies": {
"@dicebear/collection": "^7.0.1",
"@dicebear/converter": "^7.0.1",
"@dicebear/core": "^7.0.1",
"@spotify/web-api-ts-sdk": "^1.1.2",
"lucide-react": "^0.288.0",
"next": "^13.5.5",
"pocketbase": "^0.18.3",
"react": "^18",
"react-dom": "^18",
"react-toastify": "^9.1.3",
"zod": "^3.22.4"
},

next.config.mjs and "next": "14.1.4":
import path from 'path';
import { fileURLToPath } from 'url';

/** @type {import('next').NextConfig} */

// Convert the module URL to a directory path
const __dirname = path.dirname(fileURLToPath(import.meta.url));

const nextConfig = {
  webpack: (config, { isServer }) => {
    config.resolve.alias["@dicebear/converter"] = path.resolve(
      __dirname,
<EMAIL_ADDRESS>    );

    return config;
  },
};

With next.config.mjs and "next": "14.1.4" (you have to npm i @dicebear/converter):
import path from 'path';
import { fileURLToPath } from 'url';

/** @type {import('next').NextConfig} */

// Convert the module URL to a directory path
const __dirname = path.dirname(fileURLToPath(import.meta.url));

const nextConfig = {
  webpack: (config, { isServer }) => {
    config.resolve.alias["@dicebear/converter"] = path.resolve(
      __dirname,
<EMAIL_ADDRESS>    );

    return config;
  },
};

Thanks it works for me with     "next": "14.1.4",


you need to provide the link

There is a link .. somthing like aka.etcetc for setting up the MFA  it needs to be posted here for admins to quickly find so they can send to end users.

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

ID: 3ed95865-eddb-4ad1-d2e2-5bd618151c1b
Version Independent ID: ef32bb26-e78a-caad-5b41-d7e49f6ef9b4
Content: Set up multi-factor authentication for Office 365 users
Content Source: Office365-Admin/security-and-compliance/set-up-multi-factor-authentication.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

Direct users to follow the instructions in THIS Microsoft document - https://answers.microsoft.com/en-us/msoffice/forum/all/regenerate-qr-code-for-my-microsoft-authenticator/c9c975da-f8f5-48f5-b403-193fa3d66554

@Darrell-P Thank you for your feedback. This link is about regenerating a code when you have lost access to the existing app. We appreciate your time to help us.
Thanks
Sri

You are looking for a link for end-users as to how to download, install, and then scan the QR code?

@Darrell-P  Thank you for the feedback. The addition will reflect steps that a user can take to install the Authenticator App. Once the PR is approved, You will see the changes in doc.
Thanks
Sri

@jansenbro  - Thank you for submitting feedback.
From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/OfficeDocs-O365ITPro/commit/a14c019423b0a35a2dbd6e8983dae31f86596515.
Thank you for your contribution to make the docs better! Much appreciated!


Fix/addressbook

@polymorpher

Added auto remove wallet address in knownAddress
Added Edit Address Label page for other known address

closes #196

thanks, I will try after wrapping up things with a deadline, later today

Overall looks good. I will merge it and do the minor fixes. I will also add you as collaborator in the repository so it would be easier to edit and fix in the future

Another thing: let's make sure the code is formatted by eslint


Move to decred org

Perhaps this lib could join the party at https://github.com/decred now that it is a key dependency of dcrtimegui and it was approved via the Timestamply proposal.

I guess it is now https://github.com/decred/dcrtimejs
It looks like a new repo was created as opposed to transferring ownership of this one. The difference is existing PRs are not transferred. I guess it's not a big deal?
In any case, feel free to close this if it's done.


Include generated files in repo

I understand your reasons for not including them, however, lots of editors configure tree-sitters by git url, which of course isn't possible if it's not in the repo. Examples being Helix (issue) or Nvim, both supporting local files too, but that of course is an extra step to take and keep up to date and can't be included in default configurations.

Nvim seems to do ok with things as they are, and this is pretty clearly the direction that the tree-sitter folks want to take things. I’d rather invest the effort in getting Helix to a state where it can handle this. Is there a good explanation somewhere or what’s still needed for that?

As a side note, nvim-treesitter’s ability to regenerate parsers from source also lets them take advantage of query optimizations that require an ABI bump, which I implemented in tree-sitter specifically to benefit Swift. If we pregenerated the parser files we wouldn’t be able to take advantage of those until every consumer was on ABI 14.

For Helix to not depend on generated parser files checked into source: https://github.com/helix-editor/helix/issues/1970
Generating the parser files with tree-sitter-cli is not a good fit for Helix. I think a runtime or build-time dependency on tree-sitter-cli flies in something like nvim-treesitter because tree-sitter is the main feature, but for tools where tree-sitter is an implementation detail (semgrep and difftastic for example), it's an undue hassle for users and packagers.
On the one hand I detest checking in generated src and I agree tree-sitter consumers should not assume they're checked in. OTOH it's a pragmatic and low-cost way to make it easy for the majority of tree-sitter consumers to package the grammar. It also doesn't stop anyone from using a non-default ABI: eager consumers would just need to re-generate the parser.

That’s a fair point.
To me the biggest concern with the generated files is that they make pull requests more likely to have merge conflicts. Could we maybe compromise by having a separate branch with generated source? GitHub Actions could trigger an automatic checkin every time we publish a new release. That way the diffs would stay clean and there wouldn’t be too many new large deltas in the repo, but the system would still look like a git-based tree-sitter parser.

All right, this has now been done! The with-generated-files branch of tree-sitter-swift gets automatically updated every time I publish a new release (actually, any new tag). I also published 0.2.0 to seed it with an initial set of static files. @d12bb @the-mikedavis FYI


WSL not properly resolving sh.rustup.rs or static.rust-lang.org

Your Windows build number:
Microsoft Windows [Version 10.0.18362.535]
What you're doing and what's happening:
I'm trying to install Rust and rustup with this command:
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh

I get the following response:
info: downloading installer
curl: (6) Could not resolve host: static.rust-lang.org
rustup: command failed: downloader https://static.rust-lang.org/rustup/dist/x86_64-unknown-linux-gnu/rustup-init /tmp/tmp.P0N5YPdQT2/rustup-init

But sometimes I just get:
curl: (6) Could not resolve host: sh.rustup.rs

What's wrong / what should be happening instead:
It seems that WSL is trying to use IPv6 to look up the address, but I don't have IPv6 with my ISP provider and I have it disabled in Windows.
Strace of the failing command, if applicable:
I don't know if it's applicable or not, but I did a strace like this:
strace -o curl-rustup.strace -f curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh

but that probably just gives an strace of everything before the |, right? So I did a second strace like this to trace the second half:
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | strace -o curl-rustup.strace -f sh

Gists: first half of strace | second half of strace

Also, WSL is able to resolve the address if I use the -4 flag with curl or wget to have explicitly use IPv4 but the install script will still fail because the script doesn't explicitly use IPv4 to download the rest of the install files.

ALSO.
I cannot rustup update or re-install rust because this problem.

I had a same issue, but for me - adding a -4  and -v for verbose output, help:
build31@Ordi-PC:~/projects$ curl --proto '=https' --tlsv1.2 https://sh.rustup.rs -sSf -v -4 | sh
*   Trying <IP_ADDRESS>:443...
* TCP_NODELAY set
* Connected to sh.rustup.rs (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1

It's been a long time since I made this, but I decided to try this again in WSL2 just for kicks, and it downloaded fine.

on windows, using wsl.
worked for me
echo "nameserver <IP_ADDRESS>" | sudo tee /etc/resolv.conf > /dev/null


libressl: remove weird and old man compressing workaround

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg eval

Result of nixpkgs-review pr 172102 run on aarch64-darwin 1

3 packages marked as broken and skipped:

acme-sh
testssl
twa

16 packages failed to build:

cvsq
gitolite
hyprspace
iodine
libressl_3_5
nettools
openconnect
openconnect_openssl
openconnect_unstable
swtpm
unixtools.arp
unixtools.ifconfig
unixtools.netstat
unixtools.ping
unixtools.route
vpnc-scripts

13 packages built:

catgirl
disnix
dydisnix
dysnomia
libressl
netcat
notemap
openntpd (openntpd_nixos)
perseus-cli
restream
sacc
telescope
wasm-pack

Result of nixpkgs-review pr 172102 run on aarch64-darwin 1

3 packages marked as broken and skipped:

acme-sh
testssl
twa

16 packages failed to build:

cvsq
gitolite
hyprspace
iodine
libressl_3_5
nettools
openconnect
openconnect_openssl
openconnect_unstable
swtpm
unixtools.arp
unixtools.ifconfig
unixtools.netstat
unixtools.ping
unixtools.route
vpnc-scripts

13 packages built:

catgirl
disnix
dydisnix
dysnomia
libressl
netcat
notemap
openntpd (openntpd_nixos)
perseus-cli
restream
sacc
telescope
wasm-pack

libressl_3_5 fails. did it also fail before?

Never build https://hydra.nixos.org/job/nixpkgs/trunk/libressl_3_5.aarch64-darwin


Integrate model builder with MMM

Necessary changes of MMM base and delayed_saturated_mmm along with some new functions to allow of full integration of MMM with Model Builder.
Done so far:
created data formatters that prepare necessary configs and dataset for Model Builder, need discussion on whether this will be final design.

@twiecki , @tomicapretto , @ricardoV94 I'll gladly take any feedback about the initial idea of breaching the gap between params in MMM and ModelBuilder, also I've marked 2 places where I'm not sure how to implement the interactions between predictors without extremely overcomplicating it. Any suggestions highly welcome!

@michaelraczycki Something seems to have gone wrong with a rebase?
In general I think this looks good. Would be nice to add a test to show the different functionality in action. Or just a quick NB.

@michaelraczycki I'm not sure which files I should look at. There are 49 modified files in this PR. Could you give me some directions to know what to look at?


Psr 1510 4

@narendrapaduchuri @westwater do we still need this PR?

Hi @kberec, we have done  all then changes on this branch PSR-1510-4  instead of PSR-1542, it might be worth to keep it open for some time, please. FYI @westwater


Switching profiles causes cast error

Start DCSFP and open a profile for airplane A
Open another profile B
Manipulate the radio

There is an casting error when it tries to cast the RadioPanelKnobA to RadioPanelKnobB.
Somehow there is a reference to the original RadioPanelKnobA when in fact it should have been destroyed when switching profile.
Error|System.InvalidCastException: Unable to cast object of type 'NonVisuals.Radios.Knobs.RadioPanelKnobA10C' to type 'NonVisuals.Radios.Knobs.RadioPanelKnobSpitfireLFMkIX'.|Unable to cast object of type 'NonVisuals.Radios.Knobs.RadioPanelKnobA10C' to type 'NonVisuals.Radios.Knobs.RadioPanelKnobSpitfireLFMkIX'.    at DCSFlightpanels.Radios.PreProgrammed.RadioPanelPZ69UserControlSpitfireLFMkIX.SetGraphicsState(HashSet`1 knobs) in C:\dev\GitHub\DCSFlightpanels\Source\DCSFlightpanels\Radios\PreProgrammed\RadioPanelPZ69UserControlSpitfireLFMkIX.xaml.cs:line 106

Could be the radio base class causing this.

The fix was to clear the list of radio knobs in the base class SaitekPanel's Dispose().


changed sidenav, notification icons and added settings to topbar notifications

Closes ___ (e.g. #230)
What does this PR do?
A work in progress... Probably a lot of details on the same request that should be tackled separately. Here are the things that have been changed:

Some options on sidenav were hidden or changed place
Icons on notification toggle were changed (trying to bring more coherence)
Added a shortcut to notification settings when clicking at message notifications on the topbar (still not very pretty and need to make it not selected by default after you click on it)

Links to related issues
Rework Group Sidenav - https://github.com/yunity/karrot-frontend/issues/1535
Checklist

[ ] added a test, or explain why one is not needed/possible...
[ ] no unrelated changes
[ ] asked someone for a code review
[ ] joined #karrot-dev channel at https://slackin.yunity.org
[ ] added an entry to CHANGELOG.md (description, pull request link, username(s))

I finally had a look today, sorry for being late! Great work you did there! Some comments:
Latest messages menu: the grey bell icon now looks like it indicates a special kind of notification, whereas it actually says that you don't get emails for it. I think it's a bit out of context here, maybe we just get rid of it altogether?

Sidenav: I think it would be nice if the "Show more" button would hide when clicking it, then it doesn't look like it's yet another sidenav button. It wouldn't be possible to hide the buttons without reloading the website, but I think that might be ok.

It would also avoid this badge duplication (by the way, great that you added this):

In case you like these changes but feel it would be too much hassle for you, I could do them.

@tiltec I think the suggestions make sense!
1 - Grey bell notifications on latest messages: yes we could get rid of it altogether. We could also consider changing the icon to a slashed envelope, but I could not find something like this at FA's icons. Maybe if we stack it. Another detail is that hovering on it says "Notifications are disabled", but the right thing would be "E-mail notifications are disabled". I wonder if this string is used elsewhere in other contexts...
2 - Sounds good to me. I'm not sure the Expansion Item component has this, I could not find it. How would you do it?
3 - Do you mean that the badge on Show More should disappear when clicked?
I will probably not work on this anytime soon (except for #1 which looks the easiest). If you want to, feel free to do them! :)

I wonder if this string is used elsewhere in other contexts...

I checked, it's only used there. So we are free to change or remove it.

I'm not sure the Expansion Item component has this, I could not find it. How would you do it?

That requires a bit of trickery, since it's technically not just expansion anymore. Maybe to not use the ExpansionItem... needs to experimentation.
@nicksellen as you already tried out some variations of this, did you also consider the option to hide the "show more" button after expansion?

Do you mean that the badge on Show More should disappear when clicked?

Exactly, or rather if the whole "Show more" entry disappears it doesn't matter anymore ;)

@nicksellen as you already tried out some variations of this, did you also consider the option to hide the "show more" button after expansion?

nope, I can't quite imagine how that would work, as there would be no way to close it back up again after...?
relatedly, I recently was enjoying the protonmail settings menu with expanding sections:

so that's another possible direction, somehow categorizing the menu items rather than hiding the less used ones behind the "show more", which, with a quick think could be like:

wall
pickups

pickups
feedback

offers
community

members
applications
issues

(where clicking on the category expands the menu and takes you to the first sub-item)
then remove "Show public group information" (who wants to see that?) and "Map" (is also available from the expand icon on the map itself).
also, I guess we shouldn't be discussing this on a merged PR ;)

I like the idea above. Kind of solves the problem of excess of buttons in the future when more features are developed.
Btw the "Map" button was not supposed to show on desktop, only on mobile. There's a conditional there on the code that doesn't work. Tilmann said one if I remembe correctly that it probably had to do with the Quasar update.

Another implementation of show more/less, from slack:

That seems a good option! Also, it looks a bit different from the other options, so it's less confusing (e.g. no hover background)


chore: Disable prop-types rule in ESLint configuration, cleanup code

Issue Title: Cleanup code
resolve issue: #230
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

How Has This Been Tested? ⚙
Describe how have you verified the changes made.
I have made minor changes in the code such as formatting, commenting out code, applying Prettier, ESLint
I have run the application locally using npm run dev and it seems fine to me.
Let me know if I need to make any additional changes. Thanks:)

Hey @singodiyashubham87, I'm not sure how to resolve this merge conflict. Kindly let me know.

@singodiyashubham87 Could you please review this PR and let me know if there are any required changes?
Thanks

@singodiyashubham87 Hey. This PR was part of SSOC, right?
I'm unable to see this contribution on their site...
Could you please help?


How to build Skia Sharp for only specific purposes? (e.g. for Font rendering only)

I'd like to build a skia sharp library file (dynamic/shared) that will only be used for Font rendering, thus reducing the size of my program's dependencies. However, I am unable to find a way to configure Skia Sharp to only include certain parts of it and would like to know if there is any guide I can follow for this.

SkiaSharp is already quite small, usually between 4-5 MB per platform architecture. The last few releases was actually a bug, see #573 and #584
To further reduce the size, you could have a look at building your own. We typically include everything because we don't know what you are going to be using. You can remove quite a bit.
It is relatively simple to build your own: https://github.com/mono/SkiaSharp/blob/v1.60.2/cake/BuildExternals.cake#L532-L559
In most cases, it is just a case of running gn to generate the build files - here you pass along all the bits that you want. and then either run ninja or ndk-build. In fact, this is how most people build for Linux: https://github.com/mono/SkiaSharp/wiki/Building-on-Linux#building


Inline filter for toSeq causes deadloop

Desc
import pylib
import std/sequtils
discard filter(None, [1,0,3]).toSeq  # <-- deadloop here

Version
Nim: any
pylib: 0.9.0
Possible Solution
There is a workaround, see below.
Additional Info
If using closure, then everthing is fine:
let filt = filter(None, [1, 0, 3])
discard filt.toSeq

closed by https://github.com/nimpylib/pylib/commit/dab45239b4c05b91e12b5c8fd43adf1eacdc26c2

details
XXX: NIM-BUG:

that's because currently Nim will generate bad C code
for inline zip/filter like: for i in zip(...)
(in detail, the zip object initialization code
is placed within loop body!
So that iterator initialization is repeated)
    So just extract `zip` outside of the for loop, like:

    ```Python
    z = zip(...)
    for i in z: ...
    ```