Docs on CPU limits could use some elaboration w/regards to time slicing

I have been messing around with CPU limiting and doing a number of tests, and without knowing how this is implemented in LXD started with some assumptions from reading the docs and Stéphane's (now, slightly outdated) blog posts.
The docs are very unclear on how hard slices of CPU time works. That is, when you do the following in a profile:
limits.cpu.allowance: 50ms/200ms
This would logically mean that you give the container one quarter - 25% - of CPU time right? Well, yes and no. It turns out that you are then giving the container 25% of real time on a single core, not total CPU time.
I ran my tests on a Intel® Xeon® E5-1650 v3 CPU which has 12 logical processors. I measured my CPU use in seconds every 30 minutes. I ran a bitcoin address vanity generator, just to run something that maxed out the CPU for long periods of time.  From my tests I saw quickly a pattern emerge:
50ms/200ms - About 451 CPU seconds / 30 min, or about ~5% use across all cores
200ms/200ms - About 1802 CPU seconds / 30 min, or about ~10% use across all cores

So giving a container a slice of CPU time in effect means that you are giving it x seconds off a single processor. Meaning, if I had only had a single logical core this would indeed have equaled 100% CPU time of 1800 seconds pr. 30 min.
So, let's say I have 15 containers on this systems, and I want to give everybody an equal share of CPU time up to the actual maximum. This would then be something like:
12 (cores) / 15 (containers) = 0.8 (cores pr. container)
0.8 (core share) x 200ms (total time pr. core) = 160ms (container cpu time)

Which then gets you to the "correct" slice / setting of:
160ms/200ms
This should then, in theory give my container the correct limit of ~6.6% max. CPU use across all cores.
Questions:

Have I completely misunderstood some fundamentals here, and you are currently snickering at my idiocy? (I would not be surprised) In which case, please enlighten me!

Is this intended, or unavoidable behavior? I was expecting 100% use across all logical processors when specifying 200ms/200ms from reading the docs, as many others surely would. If this is just how it is, it should really be pointed out in the docs.

Furthermore, as this setting is really just an expression of a percentage of real-time, maybe it would make sense to change how this setting is described in the docs.

This makes it a bit weird/difficult to try and figure out what slice to give a container, on a multicore system, with x containers. The maths are, well, not apparent until you do these real-life tests and give it a good think.

It seems it is possible to combine with CPU pinning, i.e. doing something like:

  limits.cpu: "1"
  limits.cpu.allowance: 200ms/200ms

Which then gets you ... Some CPU use. Again, logically here I would assume this should result in 100% use on this single CPU - but due to this funky way CPU time is applied on a multicore system, you actually get less than that.
I'm not complaining actually, this is actually perfect for my current use-case, but it is really not apparent what this config actually does until you spend quite a bit of time experimenting with it.
Oh, I was doing all this on Ubuntu Xenial and LXD 2.20

Followup question
This got me thinking about the percentage setting. You write in the docs that you can specify a percentage which then gets applied if the system is under load. If the system isn't under load, the container can use as much CPU as it wants.
That's all fine and dandy, however, does this percentage setting mean percentage CPU across all cores? To be exact, I'm talking about this:
config:
  limits.cpu.allowance: 50%

What would then happen if I had this high setting across many containers? How would the load distribute? This is a bit difficult to test, but here's a hypothetical:
15 containers on my 12 logical core system. All have allowance set to 50%. One (or two) of these containers start doing something intensive and want 100% CPU time - let's say, they got hacked and some numbnut placed a bitcoin miner on them.
What would happen then?
I am guessing that the standard answer is probably: Well that really depends on what the scheduler decides to do, and that can be hard to predict.
Hmm - well, for me as a user, it's a bit cumbersome that I have to do lots of trial and error here. It would be great to get some more clarity here :) Could you at least enlighten me as to what should happen in theory?
PS: I am assuming that I should, in real life, set the allowance to an equal percentage share, so in this hypothetical they should really have allowance set to ~7% - which should guarantee that no container uses more than their fair share if the system is under load. But ... That's assuming that percentage is actually total CPU time, and not just for a single core, as is with time slicing. I could worry that this limit actually using time slicing under the hood... Maybe
In real life it is rare that all containers on the system really want a lot of CPU time simultaneously - which makes this setting so attractive as it promises only to set limits if the system is under load - but I must admit it is hard for me to quite grasp how this will work in real life, and what a sane setting here should look like for my example given.


"Casing of {file path} does not match the underlying filesystem" Mac

The file path and casing does match.
This issue started happening a couple days ago. I've reloaded the VSCode window, I've restarted VSCode, tried uninstalling and reinstalling the extension, and have restarted my machine several times.

Did you rename those lately. There is an issue in VS Code that keeps the old casing of a file when listed in the editor group. Removing them all there should get you into a clean state again.
I fixed this in ESLint, but haven't released a new version yet. Will come mid Dezemeber.
Dups https://github.com/microsoft/vscode-eslint/issues/1116

No, they were brand new files, only one name was given to them.

Does the validation happen correctly when you use eslint in the terminal. If this is the case can you please provide me with a GitHub repository I can clone that demos what you are experiencing.

Fair enough. I'll work on getting the additional information today and tomorrow between thanksgiving events. Thank you.

Great. Ping when available.

I tried duplicating it in a fresh project and I can't get it to happen... For now I'm going to close this and investigate if it's something to do with our own configuration or if it's a side effect of our configuration and the duplicate you linked to.


is there a way to search by postal code as allowed in google api

https://maps.googleapis.com/maps/api/geocode/json?address=&components=postal_code:06877&sensor=false&key=
Actual behavior
I don't see any option to search by postal_code though actual google api allows it

Lookup (if applicable):
Google

#Update
I tried something like this
results = Geocoder.search('K1G 9L1', components: [:postal_code])
its returning results but not sure if its the actual way

Yes, that's the right way to do it! The lookup will do the best it can with whatever address (or portion of address) you give it.


Analytics beta

Includes error, places, and route v1 and v2 logging

@zeladada Can you highlight where the client-facing portions are? Like what parameters you're expecting, where you're sending the unique Route ID, etc.

+1 on that. Perhaps this might be better to review in person b/c im not sure what to look for


YoutubeLoader does not work on production environment.

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain.js documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain.js rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
"use server";
import "server-only";
import { db } from "@/server/db";
import { z } from "zod";
import { auth } from "@/server/auth";
import { YoutubeLoader } from "@langchain/community/document_loaders/web/youtube";
import { SitemapLoader } from "@langchain/community/document_loaders/web/sitemap";
import { env } from "@/env";
import { RecursiveCharacterTextSplitter } from "@langchain/textsplitters";
import { PineconeStore } from "@langchain/pinecone";
import { Pinecone } from "@pinecone-database/pinecone";
import { OpenAIEmbeddings } from "@langchain/openai";
import { CheerioWebBaseLoader } from "@langchain/community/document_loaders/web/cheerio";

const YoutubeSchema = z.object({
  url: z.string().url(),
  type: z.literal("youtube"),
});

const webPageSchema = z.object({
  url: z.string().url(),
  type: z.literal("webpage"),
});

const siteMapSchema = z.object({
  url: z.string().url(),
  type: z.literal("sitemap"),
});

const schema = z.union([YoutubeSchema, webPageSchema, siteMapSchema]);

export async function addKnowledgeFormExternalSource(
  dbiId: string,
  data: z.infer<typeof schema>,
) {

  const parsedData = await schema.safeParseAsync(data);

  if (!parsedData.success) {
    throw new Error("Invalid data");
  }

  const { url, type } = parsedData.data;

  let loader: YoutubeLoader | CheerioWebBaseLoader | SitemapLoader;
  switch (type) {
    case "youtube":
      loader = YoutubeLoader.createFromUrl(url, {
        language: "en",
        addVideoInfo: true,
      });
      break;
    case "webpage":
      // loader = new FireCrawlLoader({
      //   url: url,
      //   apiKey: env.FIRECRAWL_API_KEY,
      //   mode: "scrape",
      // });
      loader = new CheerioWebBaseLoader(url);
      break;
    case "sitemap":
      loader = new SitemapLoader(url);
      break;
  }
  const docs = await loader.load();

  console.log(docs.length, docs);

  if (docs.length < 1) {
    throw new Error("No docs found");
  }

  const embedding = new OpenAIEmbeddings();

  const fileName =
    type === "youtube"
      ? (docs[0]!.metadata?.title as string)
      : type === "webpage"
        ? (docs[0]!.metadata?.source as string)
        : (docs[0]?.metadata?.title as string);

  const fileId = await createFileRecord(fileName, dbiId);
  const pinecone = new Pinecone({
    apiKey: env.PINECONE_API_KEY,
  });
  const pineconeIndex = pinecone.Index("allweone");
  const splitter = new RecursiveCharacterTextSplitter({
    chunkSize: 1000,
    chunkOverlap: 200,
  });

  const splitDocs = await splitter.splitDocuments(docs);
  const splitDocsWithMetadata = splitDocs.map((doc) => ({
    ...doc,
    metadata: {
      ...doc.metadata,
      fileId,
    },
  }));

  const store = await PineconeStore.fromDocuments([], embedding, {
    pineconeIndex,
    namespace: `${userId}-${dbiId}`,
  });

  await store.addDocuments(splitDocsWithMetadata, {
    ids: splitDocsWithMetadata.map((_, index) => `${fileId}#${index}`),
  });
}

Error Message and Stack Trace (if applicable)
Error: Failed to get YouTube video transcription: [YoutubeTranscript] 🚨 Transcript is disabled on this video (ejVMXu-u1hs)
at IW.load (/var/task/.next/server/chunks/6531.js:365:290354)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async y (/var/task/.next/server/chunks/1201.js:97:6103)
at async /var/task/.next/server/chunks/6531.js:368:24443
at async eg (/var/task/.next/server/chunks/6531.js:368:24270)
at async /var/task/node_modules/.pnpm/next@14.2.5_@babel+core@7.24.5_@opentelemetry+api@1.9.0_react-dom@18.2.0_react@18.2.0__react@18.2.0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:16:418
at async rR (/var/task/node_modules/.pnpm/next@14.2.5_@babel+core@7.24.5_@opentelemetry+api@1.9.0_react-dom@18.2.0_react@18.2.0__react@18.2.0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:15:7978)
at async r7 (/var/task/node_modules/.pnpm/next@14.2.5_@babel+core@7.24.5_@opentelemetry+api@1.9.0_react-dom@18.2.0_react@18.2.0__react@18.2.0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:18:1144)
at async es (/var/task/node_modules/.pnpm/next@14.2.5_@babel+core@7.24.5_@opentelemetry+api@1.9.0_react-dom@18.2.0_react@18.2.0__react@18.2.0/node_modules/next/dist/compiled/next-server/server.runtime.prod.js:16:26324)
at async en.responseCache.get.routeKind (/var/task/node_modules/.pnpm/next@14.2.5_@babel+core@7.24.5_@opentelemetry+api@1.9.0_react-dom@18.2.0_react@18.2.0__react@18.2.0/node_modules/next/dist/compiled/next-server/server.runtime.prod.js:17:1026) {
digest: '2995320257'
}
Description
I am using Next JS and Langchain JS. And hosting my app on Vercel.
I am trying to build a rag app where I can also upload data form youtube videos. But I am facing one issue right now.
When ever I run this code on my local server it works and I can successfully get the transcript data from youtube. But whenever I try do the same things in the production server than I get the above error.
The same video works in development and not on the production.
There is no any environment variable that is required for this so I don't think there is any difference between production and development.
So what is the issue here.
Can anyone help me ?
System Info
Next Js 14.2.5
Vercel
Local Server : Windows 11
Node 20.10.0

@dosu No, that is not the issue because in the development environment I can get the transcript using the same video. So transcript is not disabled.

Scraping is generally a bit janky unfortunately - perhaps Vercel's IP is getting blocked?

Thanks @jacoblee93.
I figured that was the case but now I am clear.


unable to download the installer

sorry, i'm a Chinese developer. Your know, in our country we have the Great Fire Wall(GFW). Then i can't download your releases, especially recent days, even goagent works poorly. So, i want to ask for a help, can you provide a mirror download source that is easy for Chinese to download the releases.
I find something matters, that we Chinese are difficult to access this host "s3.amazonaws.com".

I'm unable to download bracket from its official site for windows.
I've downloaded it two times every time it gets downloaded but does not finish so that I can install it.
Please check this.


rm roamjs-scripts, update files

FYI, I do not have access to this repo to check for repo secrets @dvargas92495
also, there was an dependency issue trying to update roamjs-components to latest
npm i roamjs-components@latest
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/roamjs-components
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>|| 16 || 17" from<EMAIL_ADDRESS>npm ERR!   node_modules/@blueprintjs/core
npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/roamjs-components
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!       peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!       node_modules/roamjs-components
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     1 more (@blueprintjs/select)
npm ERR!   3 more (react-dom, @blueprintjs/datetime, @blueprintjs/select)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@testing-library/react
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@samepage/testing
npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@samepage/scripts
npm ERR!       peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!       node_modules/roamjs-components
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

yea Ive been seeing the dep thing too. I think that library made a peer dep requirement to no longer play nice with react 17. I'll have to address this upstream within samepage


touching up presentation

A few added backticks, reason for Oct 2023 date was Bioc release, not R, blog.png missing in repo. Otherwise fine. If time pressing maybe less spatial background - participants will have the link to the full slide deck.

Great, thanks!


Paranoia Perfect Edition Official v1.01.00 - Updated Image URLs, Updated RollTemplate Default CSS

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < Paranoia  >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <  Mongoose Publishing > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <  Paranoia > (i.e. Dungeons & Dragons, FATE)

[X] I have followed the Character Sheets Standards when building this sheet.

[X] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Changed the image URLs to look into this repository
Overridden the Default RollTemplate CSS

Character Sheet Info Roll20 Internal Use only.


Allows to use .pyi file annotations supplement to .py files in analysis core

PythonAnalyzer.AddModuleAnnotations allows to add a special kind of module, that is external type annotation. In this change, internally those are tracked as ordinary modules with a special name suffix __pyi__ (not .__pyi__! That is required for relative imports to work).
These modules are analyzed just like any other module.
For every normal module DDG creates a reference to corresponding .pyi, if one exists, to create a dependency.
Now that both are analyzed, FunctionAnalysisUnit needs to find its counterpart from supplemental annotations. For this purpose, AnalysisUnit implements GetExternalAnnotationAnalysisUnit. For module, it looks for the annotations module in ProjectState. For class and function, unit simply descends into the scope by class/function name.
Once FunctionAnalysisUnit has it annotations counterpart, it can read analyzed annotations from it, if the local annotations are missing.
This change only applies to the core analysis engine. Language server will need to be updated separately to take advantage of it.

Should resolve #296.

This change only works for scenarios, where the annotations module does not have any internal references.
From the analysis standpoint, class A defined in .py file, and then annotated in .pyi files are two different classes, so if some function foo in A is annotated to return type A in .pyi, then analysis will not equate it to returning A from .py.

@lostmsu , can you create a minimal repro project (or provide content of the files) for the issue that you want to fix with this PR? I'm fixing *.pyi handling inside #361, will be useful for me.

@AlexanderSher
the scenario this change addresses is: suppose you open a directory in VS Code with nothing but the following two files in it:
alongside_annotated.py
class Annotated:
    def foo(self, bar):
        bar

and alongside_annotated.pyi
class Annotated:
    def foo(self, bar: int): ...

The goal is to have VS Code tell you, that bar in alongside_annotated.py is int when you hover over it, and get appropriate completions.
This change only affects analysis part: it enables one to add .pyi files to PythonAnalyzer by calling PythonAnalyzer.AddModuleAnnotations. These are then used for function parameters and return types.

@AlexanderSher did that work for you? Does your work cover this use case?

We are doing major rework of the analysis engine and https://github.com/Microsoft/python-language-server/pull/546 supports them. DDG analysis is no longer actively supported.


Add support for extra secrets

Overview

What this PR does / why we need it

It gives the ability to define extra credentials.
Use case: I want to pass sensitive credentials trough kubernetes secrets and pass them as environment variable.
Special notes for your reviewer
Checklist

[ ] Change log updated in Chart.yaml (see the contributing guide for details)
[ ] Chart version bumped in Chart.yaml (see the contributing guide for details)
[ ] Documentation regenerated by running make docs

@jkroepke I believe you could already achieve the same using envVars or envFrom. Can you please check if those fit your use case? Thanks!

Hi @sagikazarmark
it does not fit our case, since it required an external secret. And if there are changes on that external secret, no new deployment is triggerd. That would be the use case.
On Top, if I want to deployment dex with ArgoCD, I have to create a dedicated ArgoCD App only for the additional secret. With this PR, everything can be bundled in one deployment.

I see. Let me think about this. I'm not particularly fond of this solution TBH.
As for ArgoCD: what I've done in the past is wrapping charts to add additional secrets. That pattern worked quite well as a means to extend charts.

That pattern worked quite well as a means to extend charts.

I agree.
But I general, I much appreciate it, if the upstream helm charts directly supports passing sensitive environment variables.
See other examples:

https://github.com/grafana/helm-charts/blob/3228d102b5ebff3da462579efe9beca84bb15270/charts/grafana/values.yaml#L413-L415

Passing sensitive environment variables is quite common for dex, since the dex config supports environment variables for client secrets, client ids...

@sagikazarmark can we proceed here?


Symbol not found: _cblas_sgemm$NEWLAPACK while running mlx mistral example

System details :
Sonoma 14.2.1
xcode-select version 2405
I am trying to run the default example provided by mlx-examples directory.
python mistral.py --model_path Mistral-7B-Instruct-v0.2/  --prompt "My name is"
I am getting the following error :
ImportError: dlopen(/opt/homebrew/anaconda3/envs/language_models_env/lib/python3.10/site-packages/mlx/core.cpython-310-darwin.so, 0x0002): Symbol not found: _cblas_sgemm$NEWLAPACK
  Referenced from: <F6C66D71-1323-3E71-9C97-70DC2D75278D> /opt/homebrew/anaconda3/envs/language_models_env/lib/python3.10/site-packages/mlx/lib/libmlx.dylib
  Expected in:     <70C4EEC6-ACCF-3E75-804C-659DD8DA930F> /System/Library/Frameworks/Accelerate.framework/Versions/A/Accelerate

It is looking for Accelerate library at a wrong location.
Output of the command :
ls -al /System/Library/Frameworks/Accelerate.framework/Versions/A/
is below
total 0
drwxr-xr-x  5 root  wheel  160 Dec 15 20:13 ./
drwxr-xr-x  4 root  wheel  128 Dec 15 20:13 ../
drwxr-xr-x  4 root  wheel  128 Dec 15 20:13 Frameworks/
drwxr-xr-x  7 root  wheel  224 Dec 15 20:13 Resources/
drwxr-xr-x  3 root  wheel   96 Dec 15 20:13 _CodeSignature/

While output of command :
ls -al /System/Library/Frameworks/Accelerate.framework/
is as below :
total 0
drwxr-xr-x    6 root  wheel   192 Dec 15 20:13 ./
drwxr-xr-x  255 root  wheel  8160 Dec 15 20:13 ../
lrwxr-xr-x    1 root  wheel    27 Dec 15 20:13 Accelerate@ -> Versions/Current/Accelerate
lrwxr-xr-x    1 root  wheel    27 Dec 15 20:13 Frameworks@ -> Versions/Current/Frameworks
lrwxr-xr-x    1 root  wheel    26 Dec 15 20:13 Resources@ -> Versions/Current/Resources
drwxr-xr-x    4 root  wheel   128 Dec 15 20:13 Versions/

I found something relevant :
https://stackoverflow.com/a/70550565/3238085

The reason is explained here: https://developer.apple.com/forums/thread/655588 Since Big Sur, it somehow all became virtual. I still don't understand how it works though, but it works...

That release note says: As part of this change, copies of dynamic libraries are no longer present on the filesystem. Code that attempts to check for dynamic library presence by looking for a file at a path or enumerating a directory will fail. Instead, check for library presence by attempting to dlopen() the path, which will correctly check for the library in the cache

I don't care how Apple internally handles these things. But as it stands currently it is not working for me.
Any debugging steps that will help resolve this for me are welcome.

Did you install with pip?

@awni : Yes with pip.

I will try installing on 14.2.1 to see if I can reproduce this..

I am on M3 Max if it is of any help.  I created a similar issue here. https://github.com/ggerganov/llama.cpp/issues/4440

What's the output of:
xcrun -sdk macosx --show-sdk-version

PS I'm looking at this issue which seems related: https://github.com/numpy/numpy/issues/25026

command : xcrun -sdk macosx --show-sdk-version
output : 14.2

A bit of a shot in the dark:

Maybe try reinstalling mlx: pip install --U mlx
Try this: sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer

If neither of those helps we'll need to investigate further...

Tried both of these steps. Still getting the same error.

I encountered a similar problem. My Apple system version is 13.3 and the chip is M1 max.

@kk  try upgrading your OS (MLX support >=13.5 )

I had a similar issue, after uninstalling mlx and updating to the latest version (0.3) it was fixed.


java.lang.SecurityException: uid 10159 does not have android.permission.ACCESS_COARSE_LOCATION or android.permission.ACCESS_FINE_LOCATION.

Hello, i have this error ocurring:
java.lang.SecurityException: uid 10159 does not have android.permission.ACCESS_COARSE_LOCATION or android.permission.ACCESS_FINE_LOCATION.
I allready added this to the androidManifest in the same place where you ask por internet permissions.
    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION"/>     <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>
I also allready did flutter clean, several times, and still have the same issue.
I have Flutter sdk 1.20.4
I add a partial loging:
I/RootBeer(24900): LOOKING FOR BINARY: /data/local/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /data/local/bin/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /data/local/xbin/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /sbin/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /su/bin/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /system/bin/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /system/bin/.ext/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /system/bin/failsafe/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /system/sd/xbin/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /system/usr/we-need-root/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /system/xbin/su Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /cachesu Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /datasu Absent :( I/RootBeer(24900): LOOKING FOR BINARY: /devsu Absent :( E/MethodChannel#trust_fall(24900): Failed to handle method call E/MethodChannel#trust_fall(24900): java.lang.SecurityException: uid 10160 does not have android.permission.ACCESS_COARSE_LOCATION or android.permission.ACCESS_FINE_LOCATION.
Please Help, thnx !

Same issue, Please @anish-adm fix this issue. Thanks..

https://pub.dev/packages/safe_device


Mid Level Challenge Completed

Code: https://github.com/thoragio/rejection2
Demo: https://rejection2-uozqvyieal.now.sh/
Working with Next.js and learning React along the way was already quite a challenge, so I did not create any unit tests for this one. Will do that in the next project.
Thanks for the suggestion to try out Next.js. I enjoyed working with it, and with React as well.

Nice work!
I like the UI.
On the topic of unit tests, it can be very hard to unit test apps that are too tightly coupled. In particular, unit tests tend to work best with pure functions:

Given the same input, always return the same output
No side-effects

Pure functions aren't great just to make unit testing easier though, they also help you decouple your business logic from side-effects like network access and drawing the UI. For that reason, I like to isolate my app's business logic from network, storage, and drawing concerns.
Take this function for example:
  recalculateScoreTotal () {
    const userAsks = this.state.userAsks
    let scoreTotal = 0
    Object
      .keys(userAsks)
      .map(key => {
        if (userAsks[key].status === 'Accepted') {
          scoreTotal += 1
        } else if (userAsks[key].status === 'Rejected') {
          scoreTotal += 10
        }
        this.setState({
          scoreTotal
        })
        db.updateScoreTotal(this.state.uid, scoreTotal)
      })
  }

If you split the score calculation out, it becomes trivially easy to unit test:
const getScore = asks => asks.reduce((
  score, { status }
) => {
  switch (status) {
    case 'Accepted': return score + 1;
    case 'Rejected': return score + 10;
    default: return score;
  }
}, 0);

Consider implementing your next project using the Redux architecture to isolate business logic from view updates and side-effects.

@ericelliott Awesome! Thanks for the example. I see what you mean now, and I will definitely try to this going forward.
I am already planning to implement my next project using Redux.
Thanks again for all of your input. It's greatly appreciated!


Multiple character location tracking

I'm switching from siggy where every associated character is tracked as the same time in the same tab.
I just realized pathfinder needs a tab per character. This is a terrible step back for us.
Please track every associated character when user is logged on !

Silvio, that´s not terrible step back, that´s technology , which is possible. Go back to Siggy, if you aren´t happy with this tool.

I'm not happy with Siggy, and I'm not saying this tool is bad.
I'm willing to help and contribute to make it better.
This feature is really something missing in my opinion !

Whilst I agree that some way to track multiple characters within a single tab would be preferable, I disagree that pathfinder should track all characters associated with the account as soon as you log in one. Doing so could greatly increase pathfinder's load, often unnecessarily: I for instance have about 8 chars added in pathfinder, many of which I only use once every few months. Tracking all of them when I only log into my main would create a lot of unneeded API calls.
Instead, I propose a similar toggle like the map connection tracking one to be added to the character selection screen, allowing you to (manually) enable multiple characters to be tracked. I'm not entirely sure what backend changes that would require though.

Maybe only tracking online characters could reduce the load ?
You toggle idea sounds fine to me, however this must be persistent over session so I only have to configure it once.

Figuring out if a character is online requires constantly calls to the API as well unfortunately, actually the same location endpoint, so that wouldn't save much.

PF evolved from not so friendly to multiple character tracking to current situation and I doubt, that it will change. IMHO nobody is happy to open multiple tabs, but if there were some possibility to get the tracking done i other way it would be already there.

I guess it never hurts to check or talk to exodus on Slack about it to see if there is any other way.
If there really is none, I can deal with that as well since I just detach and minimize the tabs on a secondary screen anyways.

@MorpheusXAUT if you have some ideas about changing way of character tracking, little brainstorming  will definitely help.

Bump, I'm getting more issue with multiples tabs/char lately and still think loading all the page multiple time instead of tracking multiple chars on the same isn't a good thing.
This would be a major ergonomic improvement to me, and I think I'm not alone.

Man this is quite the dead thread. Would you guys consider a PR for this? If so if someone could point me to the section of code that currently syncs character location that would be great.

This is the only thing holding my corp back from switching over fully to pathfinder, its hard to get everyone to open a tab for every character they have. This would be a huge improvement.

Seems @exodus4d is fixing bugs with multiple tabs open. Hopefully some clean up before making it all work on one tab?

I think it would be great to have this feature. Perhaps you can add a switch for tracking a character to the character selection popup, instead of on the topright. This would relieve some load from the server to track by-default (since it'll be opt-in) but would still allow users to opt-in to tracking their active toons. Also it would allow folks to turn off tracking for toons they don't want to track any more.

Whilst I agree that some way to track multiple characters within a single tab would be preferable, I disagree that pathfinder should track all characters associated with the account as soon as you log in one. Doing so could greatly increase pathfinder's load, often unnecessarily: I for instance have about 8 chars added in pathfinder, many of which I only use once every few months. Tracking all of them when I only log into my main would create a lot of unneeded API calls.
Instead, I propose a similar toggle like the map connection tracking one to be added to the character selection screen, allowing you to (manually) enable multiple characters to be tracked. I'm not entirely sure what backend changes that would require though.

This is probably the best “compromise” that would certainly limit load.  This or some other way to toggle which characters are being tracked.


fix: Teradata large timestamp handling

Closes #1053
Handles large timestamps and dates i.e. TIMESTAMP '9999-12-31 23:59:59+00:00'.
The Pandas backend uses Numpy datetime64[ns] type which supports a maximum value of 2262-04-11. Due to this, we catch the pandas._libs.tslibs.np_datetime.OutOfBoundsDatetime exception and leave the data type as "object" for large timestamps.
This also cleans up some unused imports in ibis/ibis_addon/operations.py.

/gcbrun

Hello @nehanene15 , we are still having issues with then new 5.0 version. Could we arrange a meeting to fix this? We have timestamps with future years that are getting the same error.


GCS bazel cache should use a separate service account

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

ci/kokoro/docker/build-in-docker-bazel.sh, line 64 at r1 (raw file):

  io::log "Using bazel remote cache: ${BAZEL_CACHE}"
  bazel_args+=("--remote_cache=${BAZEL_CACHE}")
  bazel_args+=("--google_credentials=${GOOGLE_APPLICATION_CREDENTIALS}")

FYI: I think we should consider a different service account (and more limited privileges) for Bazel, we do not want Bazel accidentally breaking our tests by changing files in the wrong bucket.
Likewise, we probably don't want one of the tests or benchmarks accidentally corrupting the cache.
Maybe open a bug to clean this up?

https://github.com/googleapis/google-cloud-cpp/pull/4869#pullrequestreview-466696142

Punting for now.

Good idea, but not a major problem right now. Closing.


How to set the animation duration

I'm using snap carousel beta 6 in vertical mode. I have noticed that the onSnapToItem is just called when the scroll animation has ended. Animated documentation says that default duration is 500ms. Is there any way to change scroll animation to have shorter duration so onSnapToItem could be triggered faster? I say that because I have a video in every snap then I need to play the audio when the snap is active. Currently the audio is taking 500+ms to play.
Thanks in advance! Kind Regards!

@viniciusdba27 Why was this closed? Do you figure out how to change the duration?

No, I did not find a way to do it.


When to upgrade dependencies?

There really isn't a clear answer as to when dependencies should be upgraded, and to what extent. For example, should they be upgraded for every major/minor/patch version release, and what kind of changes would be acceptable (i.e. 2.3.4 => 2.3.5 vs 2.3.4 => 2.4.0 vs 2.3.4 => 3.0.0). This should be made clear in CONTRIBUTING.md.
Note that this issue is not limited to all the other alexa-js repositories.

The alexa-app package.json defines the compatible versions of dependencies, usually the minimal dependencies. So it should never be updated unless you have to use a newer version of a dependency because it introduced some new feature or fix that is required for this library.
This is pretty standard, not sure if it's needed in CONTRIBUTING, but feel free.

I think we should close this.


Slowness to generate proofs with recent zerokit versions

As described in these comments:

https://github.com/waku-org/js-rln/pull/58#issuecomment-1530121954
https://github.com/waku-org/js-rln/pull/58#issuecomment-1532050914
https://github.com/vacp2p/zerokit/pull/161#issuecomment-1584985557
The time to generate a proof increased a lot going from ~3s to ~23s, with this commit https://github.com/vacp2p/zerokit/commit/5eb98d4b330e1107ed18c22856707110302263e4 introducing the slowness

My understanding is that in this issue we should:

[ ] investigate why it happens
[ ] mitigate on js side if possible

@richard-ramos is it correct?

Indeed, but i'm not entirely sure we'll be able to mitigate the issue on JS side as it seems to have been introduced in zerokit

@tyshko-rostyslav any idea why this is happening?

If indeed this commit https://github.com/vacp2p/zerokit/commit/5eb98d4b330e1107ed18c22856707110302263e4 is the reason, it's quite odd, nothing special is happening there

There was a significant slowdown, when we changed versions, specifically a lot of operations became compressed. As you've seen in this thread, I tried uncompressed versions in ark-circom 0.4, which somewhat helped, but not to the desired effect

right, I think it'd be more performant if js-rln used a similar approach to https://github.com/Rate-Limiting-Nullifier/rlnjs, maybe even use the package to avoid de-duped work

I need to look into how we can leverage https://github.com/Rate-Limiting-Nullifier/rlnjs

rlnjs was not selected to be used due to fact that we needed to maintain proxy in JS code for our RLN contract and be able to optimize code in our own way.
Ice boxing this task as in future we might have a light weight way of generating / verifying proofs. Potentially we can use parts of rlnjs for this to avoid using wasm version of zerokit.
@adklempner @waku-org/js-waku-developers


JS async issue

Case 4 of #4085 still doesn't parse correctly.
let obj = { async foo(args) { return true; } }; // #4

What's wrong with it? If you want async to show up as a keyword there, that'd require look-ahead, which I think is more trouble than this small adjustment is worth.

Yes, it should be highlighted as a keyword.

I'm not really interested in implementing that. The same behavior exists for get and set.


Missing files? Maybe?

In the repository, the requirements.txt file is missing.
Also, when I installed the necessary libraries, I am getting this error 'Error opening '/Users/anishrao/SciQu/bg.png.webp''
Is it possible that some of the files are missing from the repository?


The role of IORING_ACCEPT_DONTWAIT

What does the wiki mean by “allowing applications to issue a non-retryable accept attempt”

By default, if there's nothing to accept, it'll arm poll and wait for a connection request. Setting DONTWAIT will instead return -EAGAIN and complete the request.

By default, if there's nothing to accept, it'll arm poll and wait for a connection request. Setting DONTWAIT will instead return -EAGAIN and complete the request.

Thank you!


Option for disabling auto styling

There are some some closed issues regarding custom styling/theming, e.g. #143 or #171
The problem is, if you add classes via className those will be added to the toast but the default styles usually overwrite those styles since the are more specific. So you have to always use important for every property that's included in the default styles.
The default styling is only disabled if you provide a "jsx" via custom()method. But with that you aren't able to use Promises since they exclude each other.
I propose to introduce a new config variable like unstyled?: boolean. If set to true the data-styled on the <li> value won't be set to true.
https://github.com/emilkowalski/sonner/blob/main/src/index.tsx#L214C7-L214C40

Added in #187


Make ClusterGoneMissing_FirewallIssue more generic

The pre-existing template would have been perfect for a situation where the cluster's cluster-wide-proxy was denying connections to DMS, except that it was firewall-centric in its wording.
This just lets the SRE put in a reason as a parameter rather than it being strictly a firewall problem.
Also the HTML link to the firewall prerequisites was fixed because the anchor seems to have been renamed lately. I took the opportunity to fix it in other locations where it was similarly busted.

/lgtm /approve

/lgtm

/approve

/lgtm


Adding "clangd.inlayHints.enable" in coc-settings.json didn't work

I've been trying to disable the parameter hinting popup(inlay hints)  or toggling it. I've tried to add
"clangd.inlayHints.enable" : false in coc-settings and add
InlayHints:
  Enabled: No

in .config/clangd/config.yaml, neither worked. I've also tried using clangd.inlayHints.toggle command by typing :CocCommand clangd.inlayHints.toggle but responded Command: not found. This may be a dumb question, I'm new to vim and really don't know how to use the configuration json file.
Edit:
After some tries, I found that adding "clangd.disableCompletion": true in coc-settings.json did work as expected. However, "clangd.inlayHints.enable": false still doesn't work.

So I read some source code and found:
get inlayHints() {
    const virtualText = workspace.isNvim && workspace.nvim.hasFunction('nvim_buf_set_virtual_text');
    return {
      enable: virtualText && (this.cfg.get('inlayHints.enable') as boolean),
      sep: this.cfg.get('inlayHints.sep', '‣'),
    };

I know nothing about javascript and typescript, but I assume this feature is only for neovim?  Is it possible for Vim user to disable this?

After digging into the source code and some research, I came up with this solution:
function! ClosePopUp()
    if coc#float#has_float()
        call coc#float#close_all()
        return ''
    endif
    return ''
endfunction

inoremap <silent><expr> <C-d> ClosePopUp()

Don't know if there are other ways to disable function hinting popups, but this solves my problem in the short run.

It's signatureHelp, you can set "signature.target": "echo" to move the signature to messages.

It's signatureHelp, you can set "signature.target": "echo" to move the signature to messages.

Thank you so much!


feat(product-import): set product state when not specified

Summary
Set state to New for REWE Digital Marketplace partner products if state is not specified. Fixes #125/PIN-1887.*
Description
This work emerged out of the addition of states* to the REWE Marketplace product model. When new products are imported, the state is now set to New if it is not specified.
∗ Sorry, open-source contributors, this link will not be accessible from outside of commercetools and REWE Digital.

Tests

[ ] Unit
[X] Integration
[ ] Acceptance

[ ] Documentation

Coverage increased (+0.05%) to 84.164% when pulling 19099086c7e26f22118a1a8c2c60cc442d4d2114 on 223-set-product-state-when-not-specified into e6cc2ac05ec78539d09fa5589ad93a94facc221f on master.


Add AU20010: Monitored by OSIsoft NOC

Edit to PISYSAUDITCORE.psm1: updated Get-PISysAudit_InstalledComponents to return both 32 & 64-bit programs. Previously would only return 64-bit programs on 64-bit machine.
AU20010 - Performs the following checks in order:

Check if PI Agent and PI Diagnostics are both installed
Check if PI Agent and PI Diagnostics services are both running
Check if PI Agent is registered with the NOC via contents of user.config

Added two commits to do the following:
-Check PowershellHostVersion when loading the module and fail if earlier than 3.0
-Print the audit item (AUxxxxx - Synopsis) in the progress bar
Note: audit items are run in seemingly random order, we may wish to sort them by audit code.


feat(typescript-rtk-query): add importAlternateApiName to conifg options

Description
Allows a user to change the imported api name that comes from baseApi.ts.
Example:
importBaseApiAlternateName: 'alternateApiName'

results in
import  { alternateApiName } from './baseApi.ts'

Related (issue)
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I've added an additional unit test following the previous tests' pattern.

@saihaj on it!

@saihaj I've added a changeset. Not sure if this is what is expected, if not could you please point me to an example :)?

Great work @DominicGBauer will wait for CI to be green and the merge it!


Enhancement: set debug=True in create_test_client by default

We should set debug True in the test helpers.

Funding

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified

100% - I've had this thought many times myself.


Recommended Project Directory Structure

I really appreciate the Skeleton application.
Would it be a good idea to follow the recommended project directory structure?
http://framework.zend.com/manual/2.0/en/ref/project.structure.html
(of is the structure on that page the ZF1 structure?)

That is ZF1 structure. Docs need updating, not skeleton app.

rboen linked the docs for version 2.0. In other versions this page don't exists anymore.

Tnx for clearing it up guys!


Minor version 3.4.0 introduces breaking changes

The new minor version 3.4.0 references bootstrap 4 which is a breaking change for those still using bootstrap 3 with bootstrap-switch v3.3.4. These unexpected breaking changes cause projects malfunction after automated dependencies update.

I think you should deprecate version 3.4.0 and release the same functionality as new major version 4.0.0 to help your dependents.

I do agree. The switch buttons are now malformed.

New style ...

Yup - this killed me after a yarn upgrade :(


python310Packages.mongoquery: init at 1.4.2

Description of changes
https://github.com/kapouille/mongoquery

Things done

Built on platform(s)

[X] 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/)
23.05 Release Notes (or backporting 22.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

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 207444 run on x86_64-linux 1

1 package marked as broken and skipped:

darwin.builder

2 packages built:

python310Packages.mongoquery
python39Packages.mongoquery

Thank you!


Add keycloak support in AuthorizationCodeProvider

Keycloak has a concept called realm for multi-tenancy.
Realms are part of the uri, so a baseUri looks something like this: https://<domain>/auth/realms/<realm-name>. The issue is that baseUri.withPath ignores this prefix, so I've changed that to baseUri.addPath which should still work fine with other providers.
Keycloak also supports looking up endpoints for login, token, logout etc. through a well-known uri:
https://<domain>/auth/realms/<realm-name>/.well-known/openid-configuration but that would a bit require more work so I've just added a static config like for the other providers.

Perhaps my assumption that we can just append to the baseUri was wrong, given the failing test:
https://github.com/ocadotechnology/sttp-oauth2/blob/b834b6f791310d388a9efe138bf6c97839e2e7e7/oauth2/src/test/scala/com/ocadotechnology/sttp/oauth2/AuthorizationCodeSpec.scala#L39-L53
Do other endpoints expect this behaviour?

Do other endpoints expect this behaviour?

I think this is just a legacy assumption from when this library was an internal one. I would say this change is fine, just that we'll have to make a breaking release to let the users know they have to strip the path on their own in case someone relied on that behavior.

We would just need the tests updated to reflect the change.

I've just updated the test code to do that. The tests should pass now.


question: why is katon still in development?

with the new proxy.pac feature, hotel has now reached feature parity with katon.

There's still a few things missing that I plan to implement:

WebSocket support for .dev domains
https support

I've finally updated katon's  ̀READMEwith a note to switch to hotel. I've addedWebSocketsupport and made a few improvements.https` will be for later :)


Example Changes to Publish to Maven Central

Issue #, if available: #1
Description of changes: These changes show how to publish to Maven Central. Note that the groupId has been changed to com.sigpwned so the author could test the changes. This groupId should reverted before the first official release.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The package is now published. Thanks for the prompting and the tips on the gradle.build content. https://repo1.maven.org/maven2/software/amazon/nio/s3/aws-java-nio-spi-for-s3/


Smart Editor - Test webmap with additional layer types

Widget
Smart Editor
Version of widget
Alpha
Bug or Enhancement
Bug
Repo Steps or Enhancement details
We need to test to ensure other layer types do not break the widget
Layer Types
Feature Collections with one layer
FC with more than one layer
Map Service
Tile Maps
Vector Maps
Tables
Field Types for presets
GUID
GlobalID
Range Domains
Date
DateTime(set in pop up config of layer)

Vector tiles
As a layer - Passed
As a basemap - Passed

##Map Notes
As a layer created from the standard viewer, the layer is not present in either the editor or the smart editor - Passed

here's a nice webmap to test to make sure it doesn't blow up smart editor.


How to  upload a folder that contains many files

How to  upload a folder that contains many files， I want to upload a folder  that contains many files

I agree with @tudeptrai102 the zip file would be easiest way.

Just rerunning that code won't resume an incomplete file.
According to the tus spec, to resume the file you need to use the same file URL. A new one is created because you always use tc.Create

my demo code
`static void Main(string[] args)
{
string ServerURL = "http://localhost:5000/files";
var tc = new TusClient.TusClient();
var mataDic = new Dictionary<string, string>();
int count = 0;
tc.Uploading += (bytesTransferred,  bytesTotal) =>
{
count++;
decimal perc = (decimal)(bytesTransferred / bytesTotal * 100.0);
Console.WriteLine("Up {0} of {1}",  bytesTransferred, bytesTotal);
Thread.Sleep(2000);
if(count>5)
tc.Cancel();
};
var fileInfo = new FileInfo("test.zip");
var fileURL = "";
fileURL = tc.Create(ServerURL, fileInfo);
//fileURL = @"http://localhost:5000/files/6e4e6bb327444d5481b9608150f4fcc7";
try
{
tc.Upload(fileURL, fileInfo);
        }
        catch(Exception ex)
        {

        }
        Console.WriteLine("Upload Resumed");

        var tc1 = new TusClient.TusClient();
        tc1.Uploading += (bytesTransferred, bytesTotal) =>
        {
            decimal perc = (decimal)(bytesTransferred / bytesTotal * 100.0);
            Console.WriteLine("Up {0} of {1}",  bytesTransferred, bytesTotal);
            Thread.Sleep(2000);
            //tc1.Cancel();
        };
        tc1.Upload(fileURL, fileInfo);

    }`

Problem: I have changed my code ,but it not work. it can not resume.

you cant run my test site C# upload server;
you should comment some code as follows when you run the test site.
`private DefaultTusConfiguration CreateTusConfiguration(IServiceProvider serviceProvider)
{
var logger = serviceProvider.GetService().CreateLogger();
        return new DefaultTusConfiguration
        {
            UrlPath = "/files",
            Store = new TusDiskStore(@"C:\tusfiles\"),
            Events = new Events
            {
                OnBeforeCreateAsync = ctx =>
                {
                    // Partial files are not complete so we do not need to validate
                    // the metadata in our example.
                    if (ctx.FileConcatenation is FileConcatPartial)
                    {
                        return Task.CompletedTask;
                    }

                   /* if (!ctx.Metadata.ContainsKey("name"))
                    {
                        ctx.FailRequest("name metadata must be specified. ");
                    }

                    if (!ctx.Metadata.ContainsKey("contentType"))
                    {
                        ctx.FailRequest("contentType metadata must be specified. ");
                    }*/

                    return Task.CompletedTask;
                },
                OnCreateCompleteAsync = ctx =>
                {
                    logger.LogInformation($"Created file {ctx.FileId} using {ctx.Store.GetType().FullName}");
                    return Task.CompletedTask;
                },
                OnBeforeDeleteAsync = ctx =>
                {
                    // Can the file be deleted? If not call ctx.FailRequest(<message>);
                    return Task.CompletedTask;
                },
                OnDeleteCompleteAsync = ctx =>
                {
                    logger.LogInformation($"Deleted file {ctx.FileId} using {ctx.Store.GetType().FullName}");
                    return Task.CompletedTask;
                },
                OnFileCompleteAsync = ctx =>
                {
                    logger.LogInformation($"Upload of {ctx.FileId} completed using {ctx.Store.GetType().FullName}");
                    // If the store implements ITusReadableStore one could access the completed file here.
                    // The default TusDiskStore implements this interface:
                    //var file = await ((tusdotnet.Interfaces.ITusReadableStore)ctx.Store).GetFileAsync(ctx.FileId, ctx.CancellationToken);
                    return Task.CompletedTask;
                }
            },
            // Set an expiration time where incomplete files can no longer be updated.
            // This value can either be absolute or sliding.
            // Absolute expiration will be saved per file on create
            // Sliding expiration will be saved per file on create and updated on each patch/update.
            Expiration = null
        };
    }`

How large is your test.zip file? Currently the upload piece size is 3MB so if it is less than that it would start from 0 because no segment/piece uploaded completely.

I test  nodeJs server site today .It do not work.

PS .demo code as follows
`const tus = require('tus-node-server');
const express = require('express');
const server = new tus.Server();
server.datastore = new tus.FileStore({
path: '/files'
});
const app = express();
const uploadApp = express();
uploadApp.all('*', server.handle.bind(server));
app.use('/uploads', uploadApp);
const host = '<IP_ADDRESS>';
const port = 1080;
app.listen(port, host);`

holy ， It is working... thank you very much

Glad to hear 👍


Fixed git gud levels formatting

Fixes #140

Can you change the double brackets to be something else? I think it makes the code less readable, and we want to make sure all of the code is well-documented and understandable

See 456c995f7aeac410526d5076a19f3e1d4ca95d97

This should be updated with human names. See #185

Closing this PR because #185 will fix this.


fix(obstacle_cruise_planner): use absolute value in lateraloffset to filter on obstacles

Signed-off-by: jack.song<EMAIL_ADDRESS>Description
In obstacle_cruise_planner,During obstacle filtering, the absolute value should be taken for comparison with the parameters after calculating the horizontal distance of the obstacle, because the offset distance is positive or negative.
The current code is as follows:

The "dist_from_obstacle_to_traj" should be changed  to "std::fabs(dist_from_obstacle_to_traj)".

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[x] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@shulanbushangshu Thank you so much!
Let me know if there is any other wrong behavior.

@shulanbushangshu Thank you for your contribution. Since this PR already gets approval, you can merge the PR whenever you want.


homework-10

BASIC USER ->

ADMIN USER ->

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
8

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Phenomenon that type definition does not work when dynamic route of @hono/zod-openapi is assigned to hono/client.

What version of Hono are you using?
4.4.12
What runtime/platform is your app running on?
local
What steps can reproduce the bug?
Please use the tsuki-lab/verification-hono-openapi-rpc repository me created as a reproduction environment.
const app = new OpenAPIHono();

const route = createRoute({
  method: "get",
  path: "/users/{id}", // Use path parameter.
  request: {
    params: z.object({
      id: z.coerce.number().openapi({ description: "userId", example: 1 }),
    }),
  },
  responses: {
    200: {
      content: {
        "application/json": {
          schema: userSchema,
        },
      },
      description: "Get a user",
    },
  },
});

const routes = app.openapi(route, (c) => {
  return c.json([{ id: "1", name: "John" }]);
});

const client = hc<typeof routes>("");
// The type does not match the intended shape
const client: {
  [x: string]: ClientRequest<{
      [Method: `$${Lowercase<string>}`]: Endpoint;
  }>;
}

What is the expected behavior?
const client is correctly type-stored.
const client: {
  user: {
    ":id": {
    // ...
    }
  }
}

What do you see instead?
const client: {
  [x: string]: ClientRequest<{
      [Method: `$${Lowercase<string>}`]: Endpoint;
  }>;
}

Additional information
No response

I found that this condition reproduces itself when using z.numer() and z.coerce.number().
Using z.string() is a workaround at this time.
  // ...
  request: {
    params: z.object({
      id: z.string().openapi({ description: "userId", example: 1 }),
    }),
  },
  // ...

However, when using $get(), the id passed to param must be converted to a string type each time it is used.
const r = await client.api.users[":id"].$get({ param: { id: "1" } });
console.log(await r.json());

Hi @tsuki-lab !
This is a bug in @hono/zod-openapi. I've fixed it and released a new version 0.14.8. Try it!

@yusukebe
Thank you for fixing the bug in @hono/zod-openapi and releasing the new version 0.14.8! I'll try it out right away.


[typescript] Use Class in type's definition to allow use 'new' operator

After struggle a lot with the message [ts] Only a void function can be called with the 'new' keyword, every time that I try to use the code above, I got to resolve the issue  by using export class insted a namespace.
import Ajv from 'ajv';
...
const ajv = new Ajv(ajvOptions);

This change allows to type cheking the options, outside of the parameter like this:
import Ajv, {Options} from 'ajv';
const ajvOptions: Options = {...}
const ajv = new Ajv(ajvOptions);

Looks ok to me, @blakeembrey I'd appreciate if you could look as well...

This shouldn't be merged. It's not correct for TypeScript and work in the runtime. You can read the difference between ES6 export default and export = here: https://www.typescriptlang.org/docs/handbook/modules.html.

Also, changing it to a class instead of a function would completely break backward compatibility for people who do use it as a function currently. You should instead enable support for both ways.

You should change it to this instead:
declare var ajv: {
  (options?: ajv.Options): ajv.Ajv;
  new (options?: ajv.Options): ajv.Ajv;
}

@blakeembrey thank you, I was thinking about it not being backwards compatible...

Ok. I understand. I going to make another PR fixing this.

@dalcib Just FYI, but you would usually update the PR instead. It helps maintainers keep track of the discussion and changes, and generally avoids unneeded churn. To update the PR, you can just keep working on the branch and the maintainers and anyone else subscribed to the PR would see updates.


🔖 Bump version and Changelog

Coverage remained the same at 83.795% when pulling e664ce6b34141d2687d545affa8c0e35dd61d7b7 on kyrylo/bump_batch_scoring into c8310661ecc4b769a251d1f88c4a746041c3df00 on master.


ci: update golangci-lint to v1.54

I'm going to update to v1.55 shortly but this can be landed as-is - I was initially on the fence about tagalign mainly because it has to be done manually, but it wasn't that much work and I think might actually be nice so I vote we live with it for a while and revisit if it ends up being too annoying.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (c1ea22e) 78.09% compared to head (bffdbc3) 78.09%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #661   +/-   ##
=======================================
  Coverage   78.09%   78.09%
=======================================
  Files          81       81
  Lines        5866     5866
=======================================
  Hits         4581     4581
  Misses       1084     1084
  Partials      201      201

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


Expose platform specific Configuration via expect actual

Problem Statement
I want to configure underlying configurations in android and ios SDK, which are not jetzt available in the common configuration.
Solution Brainstorm
Create an expected type for the configuration in common code and add actual implementations of it in the platforms. The users can then also use expect/actual to configure the actual platform configuration as needed, without there being a common mapping between them.

We would like to try out the React Native session replay alpha (https://github.com/getsentry/sentry-react-native/issues/2703), but can not apply the required configuration in sentry android and ios SDK.

Create an expected type for the configuration in common code and add actual implementations of it in the platforms. The users can then also use expect/actual to configure the actual platform configuration as needed, without there being a common mapping between them.

can you give a simple example?

// common
expect class PlatformConfigurationOptions

// android
actual typealias PlatformConfigurationOptions = SentryAndroidOptions // class from android sdk

// ios
actual typealias PlatformConfigurationOptions = SentryiOSOptions // class from ios sdk

Okay understand
This should work, theoretically, but testing it, there's some issue with typealiasing the JVM sentryoptions and Android sentry options where you cannot access the properties. have you been able to successfully implement this locally?

Odd thing is that it works directly in the SDK but when consumed by another application (e.g our sample app) it fails

Okay got it why it didn't work, it requires us to expose the internal native API's with api('java_sdk_dependency).
It is possible and simplifies the customization of the native SDK tbh. We generally haven't used actual typealias yet because it requires exposing the internal Sentry Native SDK dependencies.
Tbh might be worth it but I'll talk to my team and see if we have any concerns about that

If you don't want to expose the dependency on the android SDK, you can specify it in the documentation, that users that want to use the platform config should add the dependency on the android SDK themselves to their projects.

I'm generally thinking about exposing them since it would reduce the internal jvm/android implementation side immensely if we can internally use typealias as much as possible as well
But good point, regardless we can document that in the docs

@romtsn thoughts on exposing Sentry/Sentry-android API with api("java/android_sdk_dependency") so we can use actual typealias?
One of the major cons would be that user could have access to io.sentry.Sentry and io.sentry.kotlin.multiplatform.Sentry in the jvm/android sourceSets.

This would also mean user can bypass some code in the multiplatform wrapper, I don't know if there is anything important. But this can also happen already if users manually included the android sdk or are migrating from android sdk to multiplatform sdk.

yeah tbh, I think the way forward is to tell the users to manually add and configure the Android SDK and have a flag to disable native initialization (similarly to how we do that in RN and Flutter), to align with other hybrid SDKs, what do you think?

on the other hand, RN and Flutter don't have expect/actual mechanisms, so this might be a nice feature of the KMP sdk, so I'm on the fence here

Even though it might not align exactly with the other hybrid SDKs I think doing it the "KMP" way is more intuitive and easier for the user.
As an example of how it could look like from the user perspective
// commonMain
fun init() {
  Sentry.init(getPlatformOptions())
}

expect fun getPlatformOptions(): PlatformOptions

// iOS
actual fun getPlatformOptions(): PlatformOptions {
  return SentryCocoaOptions(...)
}

// Android
actual fun getPlatformOptions(): PlatformOptions {
  return SentryAndroidOptions(...)
}

We are using the KMP sdk together with the RN sdk and use the disable native initialization in RN, because it would else override settings done in kotlin. However I think it lacks documentation on what features of the sdk are not working if you disable native initialization. Example we updated RN sdk to use session recordings and configured the options in RN, but because native initialization was disabled it did not work, because we now have to enable the correct settings in the KMP/Native sdk our self and there is not much documentation for which are the corresponding options. What we are doing is looking at the source code of RN sdk to find out what settings it uses in the native SDK, so we can set this then manually.
So there is a drawback on allowing to disable native initialization.

@buenaflor yeah looks reasonable, let's do that then!

Cool!
Now we have two options for making this work:

We set api("java/android_sdk_dependency") internally instead of implementation
The user adds implementation("java/android_sdk_dependency") to their android's build.gradle dependencies and we document it

The pro of the first option is that we can freely use actual typealias in the SDK ourselves which would be nice for us and the user doesn't have to do anything else


Location of project.androidGroovy

Could you, please, specify the correct location of
project.androidGroovy {...}
Should it be inserted as top-level item in .gradle or, for example, inside the android {...} block?
groovy-android-sample-app seems to be a good place for this example.

Top level like you have your screenshot.
I'll add an example when I get the chance, otherwise adding this to the example would be a very helpful contribution.


How to use PointCloud from pedsim_sensors to update costmap obstacles

Hi, all
I have been trying to navigate pedsim_ros built-in diff-robot in environment created pedsim_similator
I have been able to navigate robot to desirable goal by navigation stack:

I have recorded and placed at https://www.youtube.com/watch?v=d6atQ8QfVFs

But as you seen, robot can't avoid obstacles (men) because I think that local costmap didn't update obstacles
To update obstacles to local costmap, I have done as following:

I don't use static map for global map
In local_costmap_params.yaml, I place two layers obstacle_layer and inflation_layer, obstacle_layer updated from pedsim_people_sensor/point_cloud_local topic:

   plugins:
     - {name: obstacle_layer,          type: "costmap_2d::ObstacleLayer"}
     - {name: inflation_layer,         type: "costmap_2d::InflationLayer"}

   obstacle_layer:
     observation_sources: bump
     bump: {data_type: PointCloud , sensor_frame: odom, topic: pedsim_people_sensor/point_cloud_local, marking: true, clearing: true, min_obstacle_height: 0.0, max_obstacle_height: 3 }

Hint: Pls view full source code at: https://github.com/AMRobots/pedsim_teb

I checked pedsim_people_sensor/point_cloud_local topic, it is ok and can view it in rviz

--
I already have tried to solve this issue by:

Method 1: set min_obstacle_height to 0.0 cm as suggestion at here
Method 2: convert pedsim_people_sensor/point_cloud_local(PointCloud) to PointCloud2 (by point_cloud_converter package), then convert PointCloud2 to LaserScan (by pointcloud_to_laserscan package), finally use LaserScan to update local costmap

With two methods I failed.
I don't know misunderstand or make mistakes at any place, pls suggest me the way to fix it.

Here is a good example of a config that uses these sensors with move_base
# use rolling window version of costmap, as local costmap is not static
rolling_window: true
static_map: false
publish_frequency: 10.0
update_frequency: 10.0
combination_method: 1

width: 5.0
height: 5.0
resolution: 0.05
global_frame: odom

plugins:
 - {name: obstacles, type: "costmap_2d::ObstacleLayer"}
 - {name: inflater, type: "costmap_2d::InflationLayer"}

obstacles:
  observation_sources: /pedsim_people_sensor/point_cloud_local /pedsim_obstacle_sensor/point_cloud_local
  /pedsim_people_sensor/point_cloud_local:
    data_type: PointCloud
    sensor_frame: base_footprint
    marking: true
    clearing: true
    obstacle_range: 4
    raytrace_range: 4
    map_type: costmap
  /pedsim_obstacle_sensor/point_cloud_local:
    data_type: PointCloud
    sensor_frame: base_footprint
    marking: true
    clearing: true
    obstacle_range: 4
    raytrace_range: 4
    map_type: costmap

inflater:
  observation_sources: /pedsim_people_sensor/point_cloud_local /pedsim_obstacle_sensor/point_cloud_local
  /pedsim_people_sensor/point_cloud_local:
    data_type: PointCloud
    sensor_frame: base_footprint
    marking: true
    clearing: true
    obstacle_range: 4
    raytrace_range: 4
  /pedsim_obstacle_sensor/point_cloud_local:
    data_type: PointCloud
    sensor_frame: base_footprint
    marking: true
    clearing: true
    obstacle_range: 4
    raytrace_range: 4

A similar one for a global map looks very similar.


Any plan to support Partitioned session cookies?

Flask supports setting the partitioned attribute to the session cookie starting from version 3.1.0 in the default SecureCookieSessionInterface implementation, see the commit https://github.com/pallets/flask/commit/9efc1ebeeb431769897798cdf8de2c67dcbbc9c0. Is there any plan to implement the partitioned session cookie in this project?

Sure. Thanks for bringing this up as I was not aware. If you have any more information or want to start a pr that will be great too


DEPRECATED Symfony 3.2

User Deprecated: The "assets.packages" service is private, getting it from the container is deprecated since Symfony 3.2 and will fail in 4.0. You should either make the service public, or stop using the container directly and use dependency injection instead.
AND
User Deprecated: The "ivory_ck_editor.renderer.json_builder" service is private, getting it from the container is deprecated since Symfony 3.2 and will fail in 4.0. You should either make the service public, or stop using the container directly and use dependency injection instead.
I'm currently using Symfony 3.4 and planning on migrating to Symfony 4 but I found this log message on Deprecations section.
Hope you guys can fix this. Thank you.

Duplicate of #329
This bundle isn't being actively developed. It has been forked by FOS, see https://github.com/FriendsOfSymfony/FOSCKEditorBundle/blob/1.x/docs/migration.rst for migration instructions.


List all open channels and connected peers

An endpoint on the maker side to list all open channels and connected peers.
This way it would be easier to extract the necessary data to force close / close a channel.

such endpoint exists already - /api/channel/list  - it's pretty rudimentaly, as it only shows things in the logs, we might want to make in nicer eventually by returning some nice json.


Update site wide settings

Resolves #9

Codecov Report

Merging #11 (cf64218) into master (72f987b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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

LGTM


Rx Interrupt and UARTsend function

Addresses Issue #15 as well as introduces UART2 send function.

I apologize that I'm making a lot of noise on the changes. Hopefully as I gain experience doing this that noise will reduce.

will need to conditionally compile enabling the RX interrupt, otherwise SerialKeyPressed() doesn't work and there is no way to start the motor or any other input from the terminal. This will have to be the case until the interrupt driven RX has enough in place to provide the present level of UI input functionality.
stm8_mcp_JS.patch.txt

I reviewed all changes and verified that basic motor function is not affected (ran on stm8s105 black board and also stm8s Discovery) - all looks good! Thank you and nice job!


How to load external urls?

For example, I have an image tag with its src set to point to an external link say 'https://via.placeholder.com/350x150'. Setting nodeIntegration to false should work but the rest of the app doesn't work as it uses relative URLs.

Docs for that are here. You can add an onClick prop to the image element.

I see. However, I was hoping for another way(no offense).
I'm using the react-player module to which I'm passing an external URL. If it made a fetch request, all would be fine but it is requesting it through the file protocol. Would I be using the same solution to this as well?

Provide an example. I think that would help me better understand your setup

Ok. This was a silly mistake on my part. I specified the wrong port.
Apologies for raising a false alarm.
Thanks for the great repo!

Hi @amilajack !
I am getting  Loading PDF as base64 strings/URLs might not work on protocols other than HTTP/HTTPS when trying to load an external URL of a PDF file. I am using the react-pdf library which won't load the file over any other protocol other than the HTTP/HTTPS due to security reasons.
How can I force the file to load over HTTP. Also, I do not see any network request for the file and I've double checked the location of the resource just to be sure.

Share the stacktrace here

Here is the stacktrace.
Loading PDF as base64 strings/URLs might not work on protocols other than HTTP/HTTPS.
It also points to the two lines below
var parentHotUpdateCallback = window["webpackHotUpdate"];   at bootstrap:1
return hotCreateRequire("./node_modules/pdfjs-dist/build/pdf.worker.js")(__webpack_require__.s = "./node_modules/pdfjs-dist/build/pdf.worker.js"); at bootstrap:788
Any ideas on how to fix it?

How are you getting react-pdf to load an external url

Try this example https://github.com/amilajack/erb-pdf-example

@amilajack thanks for the example. It does work when importing directly from react-pdf. However, this would cause performance issue and hence the library author suggests importing like so to have the rendering done off the main thread:
import { Document } from 'react-pdf/dist/entry.webpack';

Enable PDF.js worker
It is crucial for performance to use PDF.js worker whenever possible. This ensures that PDF files will be rendered in a separate thread without affecting page performance. To make things a little easier, we've prepared several entry points you can use.
Webpack
Instead of directly importing/requiring 'react-pdf', import it like so:
import { Document } from 'react-pdf/dist/entry.webpack';

This is where the window is not defined error pops up and the pdf isn't rendered.

This is an issue with the library and webpack. Not related to the boilerplate. See https://github.com/webpack/webpack/issues/6642

Thanks for your time!


Disable ability to change the date manually in input

Is it possible to disable manipulating with date in input? only by choosing date and time from datetimepicker. I use datetimepicker with moment.js support.
When i delete some value and write another i receive error like

Invalid date: Thu Oct 13 2016 18:50:00 GMT+0300;

And then moment.js cannot work with 'manually formatting date'.
Thank you!

I think you have at least options:

Remove the <input/> and only display the value.
 It's  day end and I forgot about input above. Thanks.


Reflovex main API design

Reflovex API
This API is designed to create reflow sessions and their data, but also to retrieve information like profiles sessions.
Every request to API will use the board UUID to verify if the request is valid.
The format to use is JSON for requests and responses.
Common processes
Automatically, every request to server will reflect and active status for board.
Create a reflow session
To reach this goal, first of all is necessary create a session for required board.
The session requires a profile. So I propose have two default profiles on microcontroller and if user wants, retrieve profiles from server to chose one by id.
The session id will be returned and with this session id the board will be capable of send reflow measurements in order to relate that session id.
 However I think the URLs are wrong.The Url to create a session should be POST /api/sessions, also the data and example don't match (I agree with the example, but don't agree with the data).
Finally, I think the UUID could be placed instead in an HTTP Header :thinking:


bug: toggle disable not working as expected

Reproduce link:
https://react-ts-c7hgfd.stackblitz.io
https://stackblitz.com/edit/react-ts-c7hgfd?file=ToggleTest.tsx
cause:
currently the disable callback in use-toggle hook are setting the state to true.
possible fix:
set disable callback in use-toggle to set state to false
https://github.com/kodingdotninja/use-toggle/blob/main/src/use-toggle.ts#L31

My bad, was half awake when I worked on this. 😂
Fixed in 4512fc700b3b3d7aac208469604d7b176615624d.


When subfolder of a repo is used as working directory unable to see the local GIT identity in status bar item.

Issue Description
As of now, I see that if a user opens the GitRepo as a working directory in VSCode. Am able to view the git identity in the status bar. Whereas when we try to open a subfolder of GitRepo as APIs or Client. git Identity is no longer visible in the status bar. But ideally, we are still working under the same git repo.
Folder structure of a Git Repo

Additional Notes

git global credentials are not set (user. name & user.email).

Sorry, for some reason I wasn't subscribed to my own repo.
The issue is that the extension gets activated by the presence of the .git folder. For performance reasons, I don't want to activate the extension every time VS Code starts & scan for an active version control system or scan the parent folders.
For the newest version I implemented an easy workaround: Just create a .gitignore file in those folders and the extension should be automatically loaded.


Build instructions

The README should contain instructions on how to build the artifacts (specifically the zip and exe file). Also, I'd suggest having a main branch with no generated artifacts and have a Github action create them for the gh-pages/deploy branch.

Hi, I'm sorry but yes this repository is very deprecated. Please feel free to add it yourself if you have time. :)

If you tell me how they are built, I'm happy to add that info to the README.

Well, there's no build. It's just an index file. If you are talking about the downloads of the Open Hue Controller I don't have the code publicly available. But it's just a basic electron app that I made to get it to work. Does that answer the question or did I misunderstand?

Okay. I'll see if it could be done in a GitHub action: https://medium.com/@johnjjung/building-an-electron-app-on-github-actions-windows-and-macos-53ab69703f7c


ML Streaming docstrings

Reference Issues/PRs
Add docstrings to the streaming module
What does this implement/fix? Explain your changes.
Any other comments?

@sadrasabouri Hi Sadra, Thank you for your comprehensive review, I will apply them all in the coming 4-5 hours.


Feature: Add connection fields in create new connection

Add host & port input field in add new connections dialog

Added host & port input field in add new connections dialog


stern: Include bash & zsh completion support

Motivation for this change
I want tab completion
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@grahamcofborg eval

Success on x86_64-linux (full log)
Attempted: stern
Partial log (click to expand)
github.com/wercker/stern
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/x3il84l3p9rsygn3fpgp6z6svfd7blib-stern-1.10.0-bin
shrinking /nix/store/x3il84l3p9rsygn3fpgp6z6svfd7blib-stern-1.10.0-bin/bin/stern
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/x3il84l3p9rsygn3fpgp6z6svfd7blib-stern-1.10.0-bin/bin
patching script interpreter paths in /nix/store/x3il84l3p9rsygn3fpgp6z6svfd7blib-stern-1.10.0-bin
checking for references to /build in /nix/store/x3il84l3p9rsygn3fpgp6z6svfd7blib-stern-1.10.0-bin...
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip

Success on aarch64-linux (full log)
Attempted: stern
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/vzbaw3dhkrsnqiyzx9vyq7yflsyscvld-stern-1.10.0-bin
shrinking /nix/store/vzbaw3dhkrsnqiyzx9vyq7yflsyscvld-stern-1.10.0-bin/bin/stern
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vzbaw3dhkrsnqiyzx9vyq7yflsyscvld-stern-1.10.0-bin/bin
patching script interpreter paths in /nix/store/vzbaw3dhkrsnqiyzx9vyq7yflsyscvld-stern-1.10.0-bin
checking for references to /build in /nix/store/vzbaw3dhkrsnqiyzx9vyq7yflsyscvld-stern-1.10.0-bin...
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
/nix/store/vzbaw3dhkrsnqiyzx9vyq7yflsyscvld-stern-1.10.0-bin

Sadly, it does break cross-compilation. I see two ways to fix it, either

don't generate completions when cross compiling
hardcode the completion scripts and turn the non-cross build into an equality test

The latter may or may not be feasible, depending on what's in the script. If the script defers most/all of the logic to the binary it should be fine.

Success on aarch64-linux (full log)
Attempted: stern
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/2mc8j39gbsf3hcfyd2zpc70k4pzsha6d-stern-1.10.0-bin
shrinking /nix/store/2mc8j39gbsf3hcfyd2zpc70k4pzsha6d-stern-1.10.0-bin/bin/stern
strip is /nix/store/qjrnv0qw44bw1hc23zhfh26xd1c25dfs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/2mc8j39gbsf3hcfyd2zpc70k4pzsha6d-stern-1.10.0-bin/bin
patching script interpreter paths in /nix/store/2mc8j39gbsf3hcfyd2zpc70k4pzsha6d-stern-1.10.0-bin
checking for references to /build in /nix/store/2mc8j39gbsf3hcfyd2zpc70k4pzsha6d-stern-1.10.0-bin...
strip is /nix/store/qjrnv0qw44bw1hc23zhfh26xd1c25dfs-binutils-2.30/bin/strip
/nix/store/2mc8j39gbsf3hcfyd2zpc70k4pzsha6d-stern-1.10.0-bin

Success on x86_64-linux (full log)
Attempted: stern
Partial log (click to expand)
github.com/wercker/stern
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/5x2is76b1k9wlq1wcninl48xz0zqn3kg-stern-1.10.0-bin
shrinking /nix/store/5x2is76b1k9wlq1wcninl48xz0zqn3kg-stern-1.10.0-bin/bin/stern
strip is /nix/store/n4hb93w6j076xcjw5pm09rdmc09s075b-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/5x2is76b1k9wlq1wcninl48xz0zqn3kg-stern-1.10.0-bin/bin
patching script interpreter paths in /nix/store/5x2is76b1k9wlq1wcninl48xz0zqn3kg-stern-1.10.0-bin
checking for references to /build in /nix/store/5x2is76b1k9wlq1wcninl48xz0zqn3kg-stern-1.10.0-bin...
strip is /nix/store/n4hb93w6j076xcjw5pm09rdmc09s075b-binutils-2.30/bin/strip


Blanket rename of SIG to TAG, breaking a myriad of external links for now

Actually things were fixed here https://github.com/cncf/tag-observability/pull/46


examples fix: users is list

this brings the example in line with
https://s3.amazonaws.com/cloudformation-templates-us-east-1/IAM_Users_Groups_and_Policies.template
The current example CFN template will fail if you try to create-stack (I modified it for my use and found this bug).
FYI I wrote a test harness to validate the example templates against the AWS CLI, but the validate-template does not catch this bug.  Let me know if you're interested in having this test harness, unfortunately it introduces a need for the AWS CLI and real IAM credentials that has cloudformation:ValidateTemplate rights, so it might be too heavy to be useful.

Thanks Alex!  Not sure we have the setup in place for troposphere to have it connect to AWS to do the validation.  If we made it a part of regular testing I'd be worried that changes that people would submit would be blocked whenever AWS changed their CF templates, and that would probably be bad.  Might be something we should check out for regular testing though, just to see if we can catch the changes a bit better.
Anyway, merged this change - thanks!

@phobologic Do we think AWS is going to change their CF templates very often?  Or you mean the example templates?  This test is based on the tests/example_output that is already versioned in the troposphere repo (the regression set).  I have it up here if you want to check it out: https://github.com/cloudtools/troposphere/compare/master...alexlo03:alexlo03/test_templates


add mongo-express

add mongo-express for visualize mongoDB database content in browser.

A stand-alone package such as MongoDB Compass can be used to access the DB for this function, without having to add it to the docker image.  I rather not bloat the project as much as possible.


[WIP] Add cutoff date for stats page

This makes it so one can append ?weeks=4 to the URL and only the last 4 weeks will be shown. Useful so we can evaluate which actions are actually in use currently.

It's not actually working at the moment 😅


Is it possible for random vehicles spawn with random upgrades?

Basically is it possible to code it that way or too much hassle?

I haven't tried to play around with upgrades.  I don't know much about them.  I'll try to look into it, but it I'm a little busy doing other things.  If it looks too complex, I'll probably not implement it.

alrighty


Don't launch on TheLastBen Colab

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
I did this step
In your webUI installation directory, open webui-user.bat/sh in a text editor, and ensure the line starting with set COMMANDLINE_ARGS= contains the following: --api --cors-allow-origins=http://<IP_ADDRESS>:3456, then save the file.
but I couldn't found/understand this step NOTE: the commandline flag --gradio-debug disables custom API routes and completely breaks openOutpaint. if applicable, please remove it from your COMMANDLINE_ARGS before running openOutpaint.
Additional context
openOutpaint init
Error loading script: main.py
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/scripts.py", line 205, in load_scripts
module = script_loading.load_module(scriptfile.path)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/script_loading.py", line 13, in load_module
exec(compiled, module.dict)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/openOutpaint-webUI-extension/scripts/main.py", line 40, in
print(run(f'"{git}" -C "' + scripts.basedir() +
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/launch.py", line 64, in run
raise RuntimeError(message)
RuntimeError: Error running command.
Command: "git" -C "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/openOutpaint-webUI-extension" submodule update --init --recursive --remote
Error code: 1
stdout:
stderr: fatal: destination path '/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/openOutpaint-webUI-extension/app' already exists and is not an empty directory.
fatal: clone of 'https://github.com/zero01101/openOutpaint.git' into submodule path '/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/openOutpaint-webUI-extension/app' failed
Failed to clone 'app'. Retry scheduled
fatal: destination path '/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/openOutpaint-webUI-extension/app' already exists and is not an empty directory.
fatal: clone of 'https://github.com/zero01101/openOutpaint.git' into submodule path '/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/openOutpaint-webUI-extension/app' failed
Failed to clone 'app' a second time, aborting

Hi! have you added the %cd?
Yours seems to be a different problem, but here is a quick fix for some problems TheLastBen colab users were facing with extensions.

For now, a fix is to delete on GDrive the folder SD/extensions/the extension/app and run the webui again

thank you! Now I can launch and see extension tab on webui but now I see  {"detail":"Not Found"} error. Sorry to bother you my newbie questions :(

The
%cd /content/gdrive/MyDrive/sd/stable-diffusion-webui

command should be added before the last try: tag.
Remember to also enable the API:

If that still didn't work, delete the whole extension from your gdrive and reinstall it from the webui extensions tab.

After that Enable_API = True and %cd /content/gdrive/MyDrive/sd/stable-diffusion-webui command, I delete extensions folder and reinstalled the extension and it worked now. Thx so much my friend!

No problem! Glad to help.
For Colaborators: It seems the installation process has been... A bit unstable recently. Will keep an eye on this and close the issue for now.


[Feature] Exclude antd the build

As I see it, @pixas/babel-preset-lib will include antd in the build. Wouldn't it make sense to remove this dependency?

As I see it, @pixas/babel-preset-lib will include antd in the build. Wouldn't it make sense to remove this dependency?

this preset only applied for build, I don't think it will affect your project.


configure radius change does't  work

When I use  heatmapInstance   configure method to change radius ,for example:
var newConfig = {
radius: Math.random() * 20
}
heatmapInstance.configure(nuConfig);
that does't  work . I see the code and find  radius changed  but  store._radi not change

i have the same question too, is there a good solution?

You can't change the radius by the method .configure() but you can do it by using .setData().
For example:
var dataPoint = {
     x: 50,
     y: 50,
     value: 20,
     radius: 50
};

var data = {
   max: 100,
   min: 0,
   data: [dataPoint]
};

heatmapInstance.setData(data)

Then you can change the radius.

The google heatmap plugin doesn't know the method configure(). Therefore I cannot resize radius. How can I remove from map? (to readd)

Meet the same problem. I add a function to "Store.prototype", like this:

 ,resetRadius: function(radius){
   if (radius) {
     this._cfgRadius = radius;
     var radi = this._radi;
     for(var x in radi){
       for(var y in radi[x]){
         radi[x][y] = radius;
       }
     }
   } // end if
 }

and it will be used in "Heatmap.configure" function. like this:

configure: function(config) {
    this._config = Util.merge(this._config, config);
    this._renderer.updateConfig(this._config);
    this._store.resetRadius(config["radius"]); // reset radius
    this._coordinator.emit('renderall', this._store._getInternalData());
    return this;
},

Hope it helps :)

@quietsnowyday It's very good!

Just wanted to second @quietsnowyday's solution here for updating the radius from configure - works really well, thank you!
If you're having a similar problem with the blur not being updated, I found the easiest solution was:
        var tpl;
        if (!this._templates[radius + "_" + blur]) {
          this._templates[radius + "_" + blur] = tpl = _getPointTemplate(radius, blur);
        } else {
          tpl = this._templates[radius + "_" + blur];
        }

(inside the _drawAlpha_ function; essentially it's caching the template based only on the radius, not on the blur as well).  There's a pull request for something similar from someone else too but I thought it'd be handy to have the two in the same place in this issue in case anyone else, like me, is trying to generate dynamically reconfigurable heatmaps.

Just wanted to second @quietsnowyday's solution here for updating the radius from configure - works really well, thank you!
If you're having a similar problem with the blur not being updated, I found the easiest solution was:
        var tpl;
        if (!this._templates[radius + "_" + blur]) {
          this._templates[radius + "_" + blur] = tpl = _getPointTemplate(radius, blur);
        } else {
          tpl = this._templates[radius + "_" + blur];
        }

(inside the _drawAlpha function; essentially it's caching the template based only on the radius, not on the blur as well). There's a pull request for something similar from someone else too but I thought it'd be handy to have the two in the same place in this issue in case anyone else, like me, is trying to generate dynamically reconfigurable heatmaps.

Thanks -- hours of searching, writing on the git page, nothing helped until I saw this. Updating the blur (radius, opacity were ok) change was a headache and this was the Tylenol !