Using nestia in nx/webpack

Question
I'm planning to use nestia in my nx+nest.js project.
I have to do this to make typia and nestia transformers work without ts-patch.

I would suggest to have built-in support for this just by adding before export.

// project.json
  "targets": {
    "build": {
      "executor": "@nrwl/webpack:webpack",
      "options": {
        "target": "node",
        "compiler": "tsc",
        ....
        "webpackConfig": "apps/api/webpack.config.ts",
        "transformers": [
          "apps/api/transformers/nestia.ts",
          "apps/api/transformers/typia.ts"
        ]
      },

// nestia.ts
import type * as ts from 'typescript';
import transform from '@nestia/core/lib/transform';

export const before = (options: Record<string, any>, program: ts.Program) => {
  return transform(program, options);
};

// typia.ts
import type * as ts from 'typescript';
import transform from 'typia/lib/transform';

export const before = (options: Record<string, any>, program: ts.Program) => {
  return transform(program, options);
};

But unfortunately it's not the case with npx nestia sdk and npx nestia swagger.
Is there other way to use these commands without having to install ts-patch? e.g. having the generator as transformer that can be used after code compilation with options instead of having nestia.config.ts.

I do not have insight about this issue. Therefore, I will ask you something:

Why do you use webpack in backend project?
The before functions just look like parameters atlre reversed. Why it required?

Also, not possible to replace nestia.config.ts file by transform options, as nestia do not really execute the backend server, but just only analyze its codes and structuress.
Anyway, if you have clear insight and know how to solve, it is okay to send a PR including website content (maybe setup.mdx)

I'm in the middle of moving our codebase over to Nx and the way @nx/nest uses webpack has struck me as very odd, with significant numbers asking for a way to disable it: https://github.com/nrwl/nx/discussions/8482.
So my (relatively uninformed) answer to 1. would be: it's just a weird part of Nx's design and probably quite hard for them to change it.
Doesn't really seem like an issue with this repo, rather Nx's fault for not being flexible enough.

I do not have insight about this issue. Therefore, I will ask you something:

Why do you use webpack in backend project?
The before functions just look like parameters atlre reversed. Why it required?

Also, not possible to replace nestia.config.ts file by transform options, as nestia do not really execute the backend server, but just only analyze its codes and structuress.
Anyway, if you have clear insight and know how to solve, it is okay to send a PR including website content (maybe setup.mdx)

Ability to use terserPlugin and minify code
Possibility of bundling node_modules inside bundle, output would be completely independent of node_modules folder
Usage of pkg to build executable instead of js server)
currenlty I'm using babel instead of tsc
with config

module.exports = {
  presets: ['@babel/preset-typescript'],
  plugins: [
    ['@babel/plugin-proposal-decorators', { legacy: true }],
    'babel-plugin-transform-typescript-metadata',
    '@babel/plugin-transform-runtime',
  ],
};

I'm getting
Error: Error on nestia.core.TypedRoute.Post(): no transform has been configured. Run "npx typia setup" command.
    at t.TransformError
    ```

I do not have insight about this issue. Therefore, I will ask you something:

Why do you use webpack in backend project?
The before functions just look like parameters atlre reversed. Why it required?

Also, not possible to replace nestia.config.ts file by transform options, as nestia do not really execute the backend server, but just only analyze its codes and structuress.
Anyway, if you have clear insight and know how to solve, it is okay to send a PR including website content (maybe setup.mdx)

Ability to use terserPlugin and minify code
Possibility of bundling node_modules inside bundle, output would be completely independent of node_modules folder
Usage of pkg to build executable instead of js server)
currenlty I'm using babel instead of tsc
with config

module.exports = {
  presets: ['@babel/preset-typescript'],
  plugins: [
    ['@babel/plugin-proposal-decorators', { legacy: true }],
    'babel-plugin-transform-typescript-metadata',
    '@babel/plugin-transform-runtime',
  ],
};

I'm getting
Error: Error on nestia.core.TypedRoute.Post(): no transform has been configured. Run "npx typia setup" command.
    at t.TransformError

Babel removes every type info, therefore cannot be used in nestia

Is there a list of supported compilers)? instead of babel)

Only TypeScript

By the way, may I ask plz why ts-runtime-type-checking is faster then just using decorators)?
Seems none, babel and swc are not supporting runtime type checking

Are there any examples of nestia+webpack)?

I tried to use
 {
            test: /\.(ts|tsx)$/,
            use: [
              {
                loader: 'ts-loader',
                options: {
                  transpileOnly: true,
                  allowTsInNodeModules: true,
                },
              },
            ],
          },
          {
            test: /\.(js|cjs|mjs|jsx)$/,
            use: [
              {
                loader: 'swc-loader',
              },
            ],
          },
          ```
         but when trying to import
         import { TypedParam } from '@nestia/core';
         ```
ERROR in ./src/app.controller.ts
Module build failed (from ./node_modules/.pnpm/ts-loader@9.4.3_typescript@4.9.5_webpack@5.84.1/node_modules/ts-loader/index.js):
TypeError: Cannot read properties of undefined (reading '0')

@Vector-Green This article may help you to understand why typia (nestia) is faster than class-validator
https://dev.to/samchon/typia-15000x-faster-validator-and-its-histories-1fmg

Also, I've not used webpack in NestJS environment at all.
Can anyone help us?

We also use nest with nx. Found out about typia and nestia. Our company currently uses nx and nest with class-validators.
Would be great if there is support for webpack there.  Would help, but i am not that good with webpack. Will also investigate it.
@samchon Thank you for you help.

As I've no chance to use it, no way for me. Can anybody help them?

Will someone try below way?
1. Install webpack manually
npm install --save-dev webpack webpack-cli webpack-node-externals

2. Configure webpack.config.js file
const path = require("path");
const nodeExternals = require('webpack-node-externals');

module.exports = {
    externals: [nodeExternals()],
    entry: ['./src/main.ts'],
    output: {
        path: path.join(__dirname, 'dist'),
        filename: 'main.js',
    },

    mode: 'development',
    target: 'node',
    module: {
        rules: [
            {
                test: /\.ts$/,
                exclude: /node_modules/,
                loader: 'ts-loader',
            }
        ]
    },
    resolve: {
        extensions: ['.tsx', '.ts', '.js'],
    },
    optimization: {
        minimize: false
    },
};

3. Run webpack command
npx webpack

Sounds like a good path, what i tried did not work. But i think it still needs some tweaks, so that you can use it with nx executors. Will try it  next week and check if i can bring it to work.
Thank you.

I just fix this by updating transformers in project.json
"transformers": [
  "typia/lib/transform",
  {
    "name": "@nestia/core/lib/transform",
    "options": {
      "validate": "assert",
      "stringify": "assert"
    }
  }
]

Good job, can you write a documentation PR about it?
https://github.com/samchon/nestia/blob/master/website/pages/docs/setup.mdx

@samchon I just update the documentation.
https://github.com/samchon/nestia/pull/536

Published

I just fix this by updating transformers in project.json
"transformers": [
  "typia/lib/transform",
  {
    "name": "@nestia/core/lib/transform",
    "options": {
      "validate": "assert",
      "stringify": "assert"
    }
  }
]

Looks like it's not working on latest nx version, also I have broken import of libs when this transformers specified in project.json

Hi @AwesomeAdventure , I think the default of webpack config of nx is {transpileOnly: true}. I think real solution is turn it off.
const { composePlugins, withNx } = require('@nx/webpack');

module.exports = composePlugins(withNx(), (config) => {
	config.module.rules.forEach((rule) => {
		if (rule.options?.transpileOnly) {
			rule.options.transpileOnly = false;
		}
	});
	return config;
});

I make the newest version nx (18.2.4) run with nestia in this example repo. See commit: https://github.com/honguyenhaituan/nx-typia/commit/c1df98cf949b84633448d9ec4edbcef3635a0927


brew services start yabai not working

I updated to macOS ventura and now when i try to run brew services start yabai
This is the output:
brew services start yabai
Bootstrap failed: 5: Input/output error
Try re-running the command as root for richer errors.
Error: Failure while executing; `/bin/launchctl bootstrap gui/501 /Users/itaysharir/Library/LaunchAgents/homebrew.mxcl.yabai.plist` exited with 5.

I ran the command as root and i got:
sudo brew services start koekeishiya/formulae/yabai
Password:
Warning: Taking root:admin ownership of some yabai paths:
  /opt/homebrew/Cellar/yabai/HEAD-206c0d6/bin
  /opt/homebrew/Cellar/yabai/HEAD-206c0d6/bin/yabai
  /opt/homebrew/opt/yabai
  /opt/homebrew/var/homebrew/linked/yabai
This will require manual removal of these paths using `sudo rm` on
brew upgrade/reinstall/uninstall.
Warning: yabai must be run as non-root to start at user login!
/Library/LaunchDaemons/homebrew.mxcl.yabai.plist: service already bootstrapped
Bootstrap failed: 37: Operation already in progress
Error: Failure while executing; `/bin/launchctl bootstrap system /Library/LaunchDaemons/homebrew.mxcl.yabai.plist` exited with 37.

Nevermind, i fixed it by running launchctl remove homebrew.mxcl.yabai

Not sure if this is the correct thing to do but launchctl remove homebrew.mxcl.yabai fixed similar issue


Install Error - "CSET Database Setup Error"

🐛 Bug Report
A clear and concise description of what the bug is.
Whenever I install the CSET standalone, I go to open it up and get the setup error. I did some googling and there seems to be issues with SQL Server and Windows 11. I set the registry key as described here, but the program will still not run. Here is the Microsoft link that others posts have mentioned as well.
To Reproduce
Steps to reproduce the behavior:
Running on Windows 11 Pro. I also tried with and without the firewall on, still does not work.

Download Standalone exe.
Install Standalone.
Open up CSET application.

I tried to install server 2019 express to see if that my help but the Standalone exe installs 2022 regardless.
Expected behavior
A clear and concise description of what you expected to happen.
The program would open and run normally.
Any helpful log output
Paste the results here:

Same issue here!

@randywoods @randywoods1 @Marcus-Goectau - Can you help us?
Thanks a lot!

@xjimbob @darcjr Thank you for submitting this.
I've done some research on this, and I suspect it could be an issue with SQL server running on Windows 11 machines using SSDs with sector sizes bigger than 4096 bytes. Can you look at the error.log files located here to verify if the issue is being caused by SSD sector size:
%USER%\AppData\Local\Microsoft\Microsoft SQL Server Local DB\Instances\INLLocalDB2022
If you see something about misaligned log IOs, that would verify this is the issue.
I found this guide on a potential fix: https://blog.nimblepros.com/blogs/sql-server-windows-11-fiasco/
If LocalDB still won't start after adding the new registry key, you should try deleting and recreating the INLLocalDB2022 instance if you haven't already done so.
Sqllocaldb delete INLLocalDB2022
Sqllocaldb create INLLocalDB2022

@xjimbob @darcjr Thank you for submitting this.
I've done some research on this, and I suspect it could be an issue with SQL server running on Windows 11 machines using SSDs with sector sizes bigger than 4096 bytes. Can you look at the error.log files located here to verify if the issue is being caused by SSD sector size:
%USER%\AppData\Local\Microsoft\Microsoft SQL Server Local DB\Instances\INLLocalDB2022
If you see something about misaligned log IOs, that would verify this is the issue.
I found this guide on a potential fix: https://blog.nimblepros.com/blogs/sql-server-windows-11-fiasco/
If LocalDB still won't start after adding the new registry key, you should try deleting and recreating the INLLocalDB2022 instance if you haven't already done so. Run these commands in an elevated command prompt to accomplish this:
Sqllocaldb delete INLLocalDB2022 Sqllocaldb create INLLocalDB2022

Thanks @Marcus-Goectau !
I had done the registry edit before but I had not done deleting and recreating the db. After deleting the db and then recreating it, it now works!


Replace icon assets with incompatible ratios

Description
Some Quickstart / Observability Packs have logo assets that are incompatible with the cards where the logos will be rendered.

The problem seems to be either:

A given logo has a vertical orientation (e.g. 1:2)
A given logo has vertical whitespace above or below the icon, making it look smaller than it should look

Logos to replace:
Vertical orientation

[ ] https://raw.githubusercontent.com/newrelic/newrelic-observability-packs/v0.10.0/packs/jmx/logo.png

Whitespace above/below

[ ] https://raw.githubusercontent.com/newrelic/newrelic-observability-packs/v0.10.0/packs/consul/logo.png
[ ] https://raw.githubusercontent.com/newrelic/newrelic-observability-packs/v0.10.0/packs/memcached/logo.png
[ ] https://raw.githubusercontent.com/newrelic/newrelic-observability-packs/v0.10.0/packs/mysql/logo.png
[ ] https://raw.githubusercontent.com/newrelic/newrelic-observability-packs/v0.10.0/packs/varnish/logo.png

Example of a logo that's rendering well and has no excess whitespace

https://raw.githubusercontent.com/newrelic/newrelic-observability-packs/v0.10.0/packs/mongodb/logo.jpeg

Steps to Reproduce
The UI where these tiles render is in development currently. Please contact DevEx in #help-dev-experience if you'd like to reproduce locally or confirm resupplied assets are rendering nicely. Action to resolve this bug should simply be uploading more ratio-friendly assets to replace the problematic icons.
Additional context
Slack thread from #observability-packs-internal: https://newrelic.slack.com/archives/G01N33Y8P3K/p1629420120092200?thread_ts=1629394255.078300&cid=G01N33Y8P3K

this is probably noise, but as a heads up, we are working on adding more packs to the repo and as part of this work we added ~300 packs, so I've probably made this worse 🙈
pr for reference: https://github.com/newrelic/newrelic-observability-packs/pull/147
would you happen to have a way to easily check this?

Since those are all SVG logos, I hope we'll be okay actually 🤞 Thanks for the heads up!

Hm, going through some of those SVGs at random, we might end up with a few that have excess whitespace above/below. Hard to say until they're loaded in the card... But most look good.
Suppose we'll need to load them up in the UI and assess if any look really off (like the tiny MySQL dolphin 🐬)
https://github.com/newrelic/newrelic-observability-packs/blob/15725c4d73941f1e08eeecb1bdd74132e02e6be8/packs/ansible/icon.svg
https://github.com/newrelic/newrelic-observability-packs/blob/15725c4d73941f1e08eeecb1bdd74132e02e6be8/packs/android/icon.svg

Dustin Finkle asked me to submit a request to marketing so I did that and targeted it for 10/8.

Got it.  Captured here: https://app.asana.com/0/1200592555863571/1201082122971816/f
I've assigned to Dan Weber on the Creative Team.

@jpvajda , this work? In the zip...

Horizontal version of the Java logo
4 PNG logos with whitespace cropped out
2 versions (black and white) of Cribl & Trend Micro
Quickstart Logos.zip

@DustinFinkle these should do the trick! we have no good way to test the quickstarts before they actually get updated to the site, but I can make the changes and see how they improve.

We can't use the dark mode images yet, as we only support one image per quickstart, so this will be a future improvement to consider. https://github.com/newrelic/newrelic-quickstarts/issues/421


Note to author about outdated code

Just a word of warning, some of your imports have been depreceated since recent version releases of Flask. Please consider updating the code to reflect the new method of importing or alternatively hide the source. I only say this as unsuspecting new devs who might try and use this code after installing Flask and relevant extension from newer versions.
Just a suggestion.

Hello @Swiftkilla can you send the PR with updated imports? I don't use Python these days and away from its ecosystem. I will be more than happy to accept your contribution.

Sure, will do when I have some spare time bud. No problem :)

https://stackoverflow.com/questions/59525220/which-package-should-i-install-for-flask-ext-login-and-flask-login


Minor updates to the doc files to Get-PnPHomeSite.md and Set-PnPHomeSite.md

Type

[ ] Bug Fix

What is in this Pull Request ?
Minor updates to Get-PnPHomeSite.md and Set-PnPHomeSite.md files

Thanks @reshmee011 !


Pdf.js not work in Iphone

Hello, please help, why this page http://mozilla.github.io/pdf.js/web/viewer.html#page=2 not work in iphone 4 5 or 6 versions? but in android good work. Please help. Big Thank!

We really need more information here. What does not work? Is it not rendering at all or are there rendering artefacts? Are there errors mentioned in the console? Please be as precise as possible to explain the problem.

fwiw, tested on iphone 4s and few configurations in the emulator -- works for me.

link to project http://adebiportal.kz/pdf-js-master/web/viewer.html?lang=ru&book=/upload/iblock/9cc/9cc88932e09a0b2278c9f3fafb7b8139.pdf work in android, but not work iphone 4,5,6 ... in console has errors http://adebiportal.kz/pdf-js-master/web/locale/locale.properties not found. File is exists. Please help with this problem.

I also see TypeError: document.getElementById(...) is null because you removed the download button without removing the calling JS code (wait a few seconds after opening the page to see that in the console). This really seems like a custom implementation issue rather than a bug in PDF.js though...

ok. I dont see this error, but we remowed all  js code that calling downloa button. Nothing happens. We testng viewer at http://www.mobilephoneemulator.com/ and we see that kind of error http://joxi.ru/aD2Pgw0HLbw0m3 I think it's because of access. How can I test it right without iphone?
Mobile phone emulator that enables you to test the display of any website in many cell phones.

This is a CORS issue. See https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#can-i-load-a-pdf-from-another-server-cross-domain-request for more information. Basically you cannot load a PDF from another server by default: you have to configure the server to allow that.
Closing as answered as there appears to be no PDF.js bug here, but rather a custom implementation bug. For more help, please use the IRC channel.


Move prop-types to externals

Exclude prop-types from bundle moving the package to externals

Hey! Thanks for creating a PR, but could you explain what this helps accomplish? If this happens to be for dealing with #19 (the "Calling PropTypes validators" react warning), that should be solved now.

Hi @iqnivek ,
Moving referenced package to externals will exclude it from the bundle (less size, less download time). The most react apps do already use prop-types package, so there is no need to include it in this component's webpack bundle. As a side effect, it solves #19 :)
Also see https://github.com/OscarBarrett/prop-types-externals-example

Got it. #19 is already fixed by updating prop-types, though. I think adding to externals is problematic because there's no guarantee that a react project has prop-types (if they're still using React.PropTypes, or if they're just not using proptypes at all). If prop-types were specified in peerDependencies, having it in externals would be fine, but the prop-types documentation recommends to have the package in dependencies, not peerDependencies. So I don't think I can take this change in. If I'm actually wrong in my assessment, point me to the relevant details and I'd be happy to reopen. Thanks!

Please, read the Note section in How to Depend on This Package? section:

For UMD bundles of your comoponents, make sure you don’t include PropTypes in the build. Usually this is done by marking it as an external (the specifics depend on your bundler), just like you do with React.

@iqnivek Did you get a chance to read the notes?


feat: add jaeger rest endpoints for grafana tracing support

Description
Add jaeger rest endpoints for grafana tracing support
TODO:

[ ] Remove go_parse_duration dependency
[ ] Move functions in util.rs into models.rs (can we get rid of some of them?)
[x] Check if the swagger UI is working correctly with the new endpoints

Related issues:
https://github.com/quickwit-oss/quickwit-datasource/issues/5

Checked the swagger UI, the image below

😅🎉


(gdk): Can't publish component with s3 artifact URIs

Describe the bug
Can't publish component that has s3 artifact URIs in the recipe that don't exist locally.
To Reproduce
Create manifest with s3 URIs other than the source code zip, that don't exist locally.  run gdk component publish
Expected behavior
Component is published using s3 URIs in its artifacts.  URI with s3://BUCKET_NAME/COMPNENT_NAME/COMPONENT_VERSION gets values replaced, other URIs are left alone
Actual behavior
Error was raised
Could not find the artifact file specified in the recipe '<my s3 file>' inside the build folder '<my build folder>'
Environment

OS: Darwin 19.6.0
JDK version: n/a
Nucleus version: n/a
GDK version: 1.1.0

Additional context
Documentation says "If you use GDK CLI v1.1.0 or later, the component recipe can specify artifacts that exist in an s3 bucket but not in the local component build folder".  That's basically what I'm trying to do
Bug prevents me from publishing my component using GDK CLI

A work-around that allowed me to publish was to log.info instead of raise Exception here.  just skip renaming URI if file doesn't exist locally.  Let me know what you think

Hi BillMcGuinness,
Thank you for using our tool. The fix for this bug is already in the development branch of the tool (commit) and we used similar workaround like yours :). We will roll this fix out with next version of our tool.
Closing this issue for now. Please feel free to open it again if you have more concerns. Thanks!


Windows Compatibility

Latest release doesn't work correctly on Windows. So...

Non working Windows related hack was removed.
Process call code was refactored. Now it's cross-platform.

closes #56 and not breaks #55

I do confirm that this is working on Ubuntu. Can someone check on Mac OSX?
@hordijk This works for you right?

@nunomaduro year, it works on macOS.

And you have tested that on non bash windows right ?
On Mon 27 Aug 2018 at 21:31, Sergey Chursin<EMAIL_ADDRESS>wrote:

@nunomaduro https://github.com/nunomaduro year, it works on macOS.
[image: image]
https://user-images.githubusercontent.com/980845/44681284-d99b4900-aa48-11e8-9067-dcfa8eed4d7d.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nunomaduro/larastan/pull/128#issuecomment-416341016,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFNFVMYTrBPZYEvWhzVfaftsWGM7ig2yks5uVEj2gaJpZM4WE5R6
.

@nunomaduro Yes, right.

Thanks!

The fix in OptionsResolver.php is the same as #110. I can confirm it works fine for me.


UNIQUE constraint failed

Hi,
I am trying to add django-vote into my project but have come across a problem :(
I am getting the following error message.
UNIQUE constraint failed: vote_vote.user_id, vote_vote.content_type_id, vote_vote.object_id
Here are the following files:
views.py
def vote_up(request, id):
entry = Entries.objects.get(id=1)
entry.votes.up(request.user)
return HttpResponseRedirect(reverse('playlists.views.index'))
models.py
from vote.managers import VotableManager
class Entries(models.Model):
votes = VotableManager()
title = models.CharField(max_length=255)
url = models.CharField(max_length=255)
datetime = models.DateTimeField(auto_now_add=True)
posted_by = models.ForeignKey(User, unique=True)
def str(self):
return (self.title)
It would be awesome if anyone could point me in the right direction :)

If you call vote.up twice with the same user as the argument, you will get this exception.
up method will raise an IntegrityError when this happened, you can catch this error in your code, and return some message to user

Hi Shellfly,
Awesome, I must have got confused and thought the error was actually something wrong with my code, Thanks for responding so soon I will do some research on how to catch the error code and return a message to the user saying they have already voted on this item :+1:


Changing position of the button in the collection section of home screen

here the button and the text above the button should come in the below of the card to look more good
opening button above the cards give the wrong impact .
i want to work on it and fix this.
please assign me this under the gssoc

Not required


Stall in the C++11 when overflowing the pipe in AddTaskSetToPipe

When adding more than (1<<PIPESIZE_LOG2) elements using AddTaskSetToPipe, once the pipe is full it will start executing subtasks inplace but if the other worker threads are asleep, they will not be awaken.
My simple fix was to add
        // wake up threads early
        if( ( numAdded % 128 ) == 0 )
        {
            if( m_NumThreadsActive.load( std::memory_order_relaxed ) < m_NumThreadsRunning.load( std::memory_order_relaxed ) )
            {
                m_NewTaskEvent.notify_all( );
            }
        }

just after the if( !m_pPipesPerThread[gtl_threadNum].WriterTryWriteFront(...) ) { ... } block.

With the current task scheduler this shouldn't happen unless the number of threads in the system is > 16, which is potentially the case with some systems.
Note that @fstrugar has a modified scheduler which has grain size control allowing tasks to be split up with more control than the current enkiTS.
I'm marking this as both a bug and a feature as this is related to two features I have been looking into doing:

Adding grain size control
Optimizing task insertion by offloading some of the splitting to other threads (note offloading all splitting is slower in tests).

See Issue #10 as I've started grain size control and optimization of task insertion for systems with large numbers of hardware threads.

I believe the recent commit linked to above along with the grainsize and task addition changes should fix this. As I'm not heard back from you since the creation of this issue I'll close it.


TypeScript usage?

I would love to have some docs showing how this is supposed to be used with TypeScript!

Agreed. Anything in particular you're curious about?
The demo has some typescript code examples that might be useful in the meantime :)

Well, mostly how I should type the block content I pass in :) I see in the demo app there's a repo called @portabletext/types - would be great to have a link to this in the docs, for instance :)

I struggled with this for a good while. I'm still not sure I fully understand it, that being said I have auto completion so I'm happy. I also had a pretty simple case, in that I wanted to style my text blocks with Chraka.
Example type definitions I create, I'm sure there is a better way to do this with the included types but I couldn't figure it out.
type Block = {
  _key: string;
  _type: string;
}
type BlockChild = Block & {
  marks: string[];
  text: string;
}
export type BlockDefault = Block & {
  children: BlockChild[];
  style: string;
}

//My object shape in sanity.
type Author = SanityDocument & {
  name: string;
  picture: string;
  bio: BlockDefault;
  image: Image;
  projects: any[];
}

In my component I pass bio to  <PortableText /> pass in which custom component to render for the type block.
<PortableText
            value={bio}
            components={{
              types: {
                block: PortableBioBlock,
              },
            }}
          />

My custom component PortableBioBlock looks like this:
import { Text } from "@chakra-ui/react";
import type { PortableTextTypeComponent } from "@portabletext/react";
import { BlockDefault } from "src/@types/Post";

export const PortableBioBlock: PortableTextTypeComponent<BlockDefault> = ({
  children,
}) => {
  return (
    <Text fontSize={"1.2rem"} letterSpacing={1.3} lineHeight={1.4}>
      {children}
    </Text>
  );
};

I'm not sure if this is any help, or if there are ways I could improve it

// @ts-ignore is your friend.

This is how I did it:
import type { PortableTextBlock } from '@portabletext/types'

export type Recipe = {
  _id: string
  title: string
  tags?: [{ title: string; slug: { current: string } }]
  slug: { current: string }
  body: [PortableTextBlock] // <-- 🎉
  mainImage: {
    asset: {
      url: string
    }
  }
}

I did something along these lines which worked nice for our use-case:
interface SanityResponse extends SanityDocument {
    beskrivelse: string;
    embeddedInnhold: PortableTextBlock[];
    maal: string;
    tittel: string;
}
function MyComponent() {
    const data: SanityResponse[] = await sanity.fetch(
        '*[_type == "Aktivitet"]'
    );
    return (
        <PortableText
            value={data.map(({ embeddedInnhold: innhold }) => ({ innhold }))}
            components={portableTextComponents}
        />
    );
}

I can write a simple PR updating the readme on simple typescript use/gotchas if preferable :)

We just added a "typing portable text" section to the README, let me know if it is helpful https://github.com/portabletext/react-portabletext#typing-portable-text


Load from engine got no result

When I load from caffemodel, it work fine.
But when I load it from engine directly from previous saved trt engine file, it does not got any result. What's wrong?

Could you show me your shell to load the engine file?


YouTube Music Does Not Update Feeds / Episodes

When adding links to YouTube Music, it loads episodes fine, and they also play fine. The problem is that those feeds don't load new episodes from that point forward.
This kind of behavior looks like it does not happen with other podcast clients (Antennapod, Podcast Republic, etc)

I did not know youtube music supported rss, is your server public or only available inside your lan?

For quite a while YouTube Music allows to add podcasts via RSS.
My server can be accessed outside my home. Other podcast clients can fetch all the episodes.

Can you share some rss feed that do work? It's probably an issue with youtube music being picky about something in the feed or in how to fetch the feed. (I would try with a youtube channel that has an official podcast feed and compare those)
If you can, also check yourself if you see something weird comparing them, like some fields missing from the vod2pod one.
Another issue could be that something about your server does not work well with youtube, to troubleshoot that I've added a podcast to my yt music to see if I can replicate the issue


feat: Ability to configure the use of private API keys in an environment

Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[x] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[ ] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

Introduce a configuration option to disable private API key use and generation in an environment. When disabled, only JWT can be used to authenticate, helping to prevent potential misuse. By default private API keys are still enabled.

Added

A new configuration option ENABLE_API_KEY_AUTH to control private API key use in an environment.

Changed

Prevent authentication using API key, if ENABLE_API_KEY_AUTH config is disabled
Prevent creation of API keys, if ENABLE_API_KEY_AUTH config is disabled
Hide UI section for managing API keys (under Settings > Account), if ENABLE_API_KEY_AUTH config is disabled
Import of WEBUI_AUTH in auths.py was moved to be imported from open_webui.env where it originates from

Additional Information

This is a reopened version of #6950. This PR resolves merge conflicts against dev & fixes an issue with configuration option use. Updating locales has been left out.
Discussion for this functionality was started in #6756
A slight overlap with discussed functionality in #6692

Screenshots or Videos

Sample API Requests
When ENABLE_API_KEY_AUTH=False.
Making calls to endpoints using API key
curl 'http://localhost:8080/api/v1/models/' \
  -H 'Authorization: Bearer <API-KEY>' \
  -w "\nHTTP Status: %{http_code}\n"

{ "detail":"Use of API key is not enabled in the environment." }
HTTP Status: 403

Trying to create a new API key
curl -X 'POST' \
  'http://localhost:8080/api/v1/auths/api_key' \
  -H 'Authorization: Bearer <JWT>' \
  -H 'accept: application/json'
  -w "\nHTTP Status: %{http_code}\n"

{ "detail": "API key creation is not allowed in the environment." }
HTTP Status: 403

Generally looks good, I'll push some refacs and have this merged before 0.4

Closing in favour of 7a585fbaf34a8864309d85350c0b8fda97dae8c5, Appreciate the effort 🙌


AsDecimal() in TestPage does not work for empty value

1. Describe the bug
If i have a field in a testpage with an empty value and i want to convert the value into a decimal value with AsDecimal() function an error occurs "Input string was not in a correct format".
2. To Reproduce
write code in a test codeunit with a testpage and a field with an empty value:
ExampleTestPage.Field3.AsDecimal()

3. Expected behavior
I would expect that the value "0" as return value. (As in older versions <BC20.3)
4. Actual behavior
An error is occurring with message: "Input string was not in a correct format"
5. Versions:

AL Language: 9.3.646020
Visual Studio Code: 1.69.1
Business Central: 20.3.42673.43026

Hi, could you provide a test page and a test codeunit that makes the error occur? This would help in trying to reproduce the problem. Thanks

This issue has been closed because it needs more information and has not had recent activity. Please open a new issue if you believe this has not been resolved and reference the current issue.


S3DocStore

Feature request
Hi, it seems the only DocStore available are InMemory or Google. I'd like to submit a feature request for an S3DocStore.
Motivation
Many people have raised issues related to limited DocStore options.
Your contribution
Not sure.

Also, an Azure docstore would be great.  Perhaps based on Cosmos.

Bump.


Cursor always jumps to the end when editing text

General Information

Project version:
1.1.0

Platform/OS version:
iOS 14 and 15

IDE version:
Xcode 13

Devices:
iPhone 12, iPhone X

Describe the bug
Cursor always jumps to the end when editing text
Steps to reproduce

Type "Hello World"
Move cursor to the beginning
Type "A"

Expected behavior
Cursor should be after "A"
Actual behavior
Cursor automatically moves to the end of the text.
Video
https://user-images.githubusercontent.com/3197049/157152747-5c38b183-233c-4a24-8b9f-0d03d98d5554.mov
Additional context
Since you set the text on update ui, the cursor naturally moves to the end. I have a forked version with a fix. It's something I quickly put together so I'm not entirely confident if its clean enough for a PR request (I'm also particularly new to SwiftUI) . I can send a PR if you're fine with it but if not I hope you could roll out a minor fix.
Hope this helps. And thanks for putting this package together.
EDIT:
Replace Imgur video link with embedded video.

Are you able to include a Demo app at all? @teffi I'm unable to reproduce sorry ¯_(ツ)_/¯

@shaps80 Updated description with demo video :)

Hi,
To confirm the bug, I'm seeing it as well and it seems teffi's fix works.

@joemichels / @teffi yep I can confirm as well. However the suggested implementation isn't a great fit for SwiftUI and/or as a reusable framework. I'm on holiday atm, but when I'm back next week I'll review properly and consider an alternative. Thanks!

@shaps80 Has this fix been integrated into main?

I thought so, was a few months back so I admit my memory is fuzzy. I'll look into this tomorrow (out atm) and get back to you. Or get it done, either way.

@shaps80 Great TextView for SwiftUI! Just wondering if you have had a chance to fix the issue about the cursor?

@Matt54 / @TToldi – so actually I have a complete re-write in-progress for this. Solves the above issue but also enables a far more SwiftUI approach. Will let you know once its ready 👍🏻


Batch data transfer. Multiple rows per INSERT.

Hello.
How about create an option for using multiple rows with per INSERT statement during table transfer to another database? That would speed up transfer exponentially. I know, there may be issues with DEFAULT values and triggers, but it would be just an option for those, who know the risks.
I use some app that transfers tables with bulk INSERTs and it is about 10 times faster, then DBeaver's transfer.
Thank you.

already implemented in 22.0.1


fix: new fork 'withdrawals' and fixes for asset unlock signature verification

Issue being fixed or feature implemented
https://github.com/dashpay/dash-issues/issues/83
What was done?
Introduces new fork "withdrawals" which let Asset Unlock be valid for any active quorum + 1 extra inactive (in opposite of hard-coded 2 of them).
How Has This Been Tested?
See new test section test_withdrawals_fork in functional test feature_asset_locks.py
Breaking Changes

new fork "withdrawals"
new logic of validation of Asset Unlock transactions's signature. Even no asset-unlock is mined yet, previous version of clients will refuse blocks which is fine for current system

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone

Needs rebase?

Needs rebase?
depends on #6275 (bury mn_rr) also, which is not merged yet; will rebase after

tests failed

@knst knst force-pushed the fix-withdrawal-quorums branch from 7bc3698 to 5fd7b07

fixed test, activation date and rebased to the newer commit - base commit is pretty old

int CAssetUnlockPayload::getHeightToExpiry() const
{
    const auto& llmq_params_opt = Params().GetLLMQ(Params().GetConsensus().llmqTypePlatform);
    assert(llmq_params_opt.has_value());
    return requestedHeight + llmq_params_opt->dkgInterval * (llmq_params_opt->signingActiveQuorumCount + 1);
}

@UdjinM6 no, accordingly DIP and by design, getHeightToExpire is not related to the amount of active quorums at all, it's a mechanism to refuse withdrawal in a short time (48 blocks) if it has not been mined, for example due to limits, or due to small fee; so it can be re-issued without waiting 24+ hours


I'll propose other ones, I was waiting for merging these PRs as I didn't want to spam with a bunch of PRs 😄


Why does SugarORM contain the default ic_launcher.png icons? They are merged with my own ic_launcher.png and I don't want that

Some background:
Using SugarORM in my app and currently I have designed a single app launcher icon that I have named "ic_launcher.png" and put in my app's /drawable folder. I'd like to use that icon globally throughout my app for all device resolutions so I removed ic_launcher.png from the other drawable folders.
However, SugarORM also includes the default ic_launcher.png in all of  it's drawable folders and since I am using SugarORM in the <application> tag, those are merged with my designed icon and device resolutions for which I have not provided ic_launcher use the one coming from the SugarORM folder.
One way to overcome this is to rename my custom designed icon something different than ic_launcher and put the new name in android:icon but I think most people that use SugarORM will just go for the default configuration and not even notice the merged ic_launcher drawable.
I think that in the future we should either remove this launcher icon from the SugarORM library or rename or just leave it in the default /drawable folder (no drawable-hdpi, etc).

I found this too after nearly going crazy about my icon being ignored. Lucky you that were warned by Android Studio, for me it was silently failing.
In my case, renaming my icon to another name and using the 'tools:replace' tip did the trick while the fix arrives.
Error:(11, 9) Execution failed for task ':app:processDebugManifest'.

Manifest merger failed : Attribute application@icon value=(@drawable/icon_fatofacts) from AndroidManifest.xml:11:9
is also present at com.github.satyan:sugar:1.3:13:9 value=(@drawable/ic_launcher)
Suggestion: add 'tools:replace="android:icon"' to  element at AndroidManifest.xml:10:5 to override

I confirm that it's solved in version 1.3.1; this issue should be closed. Maybe it was a build error @satyan @whoshuu? Also, it would be nice to have a tag available for version 1.3.1 :)
Generated AndroidManifest for version 1.3:
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.orm"
    android:versionCode="1"
    android:versionName="1.0" >

    <uses-sdk
        android:minSdkVersion="9"
        android:targetSdkVersion="19" />

    <application
        android:allowBackup="true"
        android:icon="@drawable/ic_launcher"
        android:label="@string/app_name" >
    </application>

</manifest>

Generated AndroidManifest for version 1.3.1:
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.orm"
    android:versionCode="1"
    android:versionName="1.0" >

    <uses-sdk
        android:minSdkVersion="9"
        android:targetSdkVersion="19" />

    <application>
    </application>

</manifest>

same thing here

Yes, it's fixed in 1.3.1.

Are you guys sure? I got it on 1.3.1 and downloaded the *.aar from maven central.  There must be some auto-generated ic_launcher.png because it's giving me an ic_launcher.png duplicate file error despite it not being in my project. Removing sugarORM clears the error.

You should release a new version so that jcenter and mavenCentral pickup the changes.

I wanted to use this for a new project but i cant because of this issue

@MichaelObi Like described in the first post, you could just rename your own ic_launcher

That's a very hacky fix.

I was using 1.3 just updated to 1.3.1 and I'm not getting the error anymore.

This issue should be resolved with the new version 1.3.1.

@satyan Please check the following link from jCenter: https://bintray.com/artifact/download/bintray/jcenter/com/github/satyan/sugar/1.3.1/sugar-1.3.1.aar You can see that it still contains the resources.

Still getting this issue on 1.3.1 with Android Studio.

@twig Double check your gradle file for SugarORM dependency
It should be
compile 'com.github.satyan:sugar:1.3.1'
The official doc on the site suggest version 1.3 but that's bugged ( http://satyan.github.io/sugar/ ) can i suggest an update to the docs @satyan ? Thank you in advance

1.4 seems to have fixed it.

@quivalen I was already using 1.3.1. Still getting the same error with 1.3.1, but I just renamed the icon file to avoid the issue. Haven't tried 1.4 yet though

I also had this problem. I was using 1.3 though. Im glad I found this forum. I was totally confused as to why my apps Icon suddenly changed.

Unfortunately, I moved my "ic_launcher" from "mimap" to "drawable". It's not a good solution, but it worked.
I'm using version 1.3.

thanks @masoomyf work for me.


Fix build compilation when using coursier

I am using coursier in my global sbt configuration. The current scalatest master build does not compile without these changes. I am not completely sure what is causing the issue, but it seems that io package gets shadowed by "io.get-coursier" % "sbt-coursier" % "1.0.0" sbt plugin.

@cla-bot[bot] check

I think this PR is not relevant anymore. :)


Add XML Schema Validation

Breaking #40 into smaller chunks for Hacktoberfest. We'd really like to start adding schema validation in addition to syntax validation. XML seems like a good one to start with. We'd like to support validation against DTD as well as XSD.

@kehoecj Looking into this and see what’s possible.

@kehoecj Can you please assign it to me? Just so I don't lose track :)

@kehoecj Do you have a specific file naming convention for the schema and syntax validators to differentiate them? Or should I create a new directory of schema validators? Just getting some information to keep it maintainable for everyone. Thanks 😄

Schema validation should be in the validator package and inside the xml.go validator

@kehoecj Sorry if I'm bombarding you, but I need some guidance. If we want to enable validation against XSD or DTD, we would also need to provide a way for the user to specify a path to the schema file, right? But this would end up in a new flag? Or am I thinking the wrong way?

@kehoecj Just tagging you so that you can give me a little hint as to how I might do it. Thanks :)

@kehoecj Sorry if I'm bombarding you, but I need some guidance. If we want to enable validation against XSD or DTD, we would also need to provide a way for the user to specify a path to the schema file, right? But this would end up in a new flag? Or am I thinking the wrong way?

You're on the right track. With XML the XSD/DTD URL or local path can be declared in the XML. We may want to just start with that as the MVP/POC. It's likely one of the go XML libraries already does some of this. Here's what I'm thinking:

If there is not XSD/DTD attribute declared in the file just check the syntax
If there is is an XSD/DTD declared AND the file can be obtained then validate schema and syntax. Prioritize outputting the syntax error first then schema issue.
Add a flag to just check the syntax, i.e. the current behavior. Something like --syntax-only

@timwehrle Does that help?

@kehoecj Ah, yes. Thank you. I was probably overthinking it and missed the simpler solution.

@kehoecj  So basically my code can retrieve the XSD/DTD schema file (had to implement a custom function using regex because the go xml libraries don't provide a way). But the problem is that I can't read the file because I would need to locate it first. But I can't manage to use the already implemented FileFinder that is in the project because it would resolve in import cycle errors. How would you proceed from here? Should I implement a function myself? Thanks for helping out :)


What's the benefit of copy_op when generating gradient graph?

@tqchen @ZihengJiang
I don't get the point about what's the  benefit of copy_op in gradient.cc.   Why not just use ret.outputs[counter] = entry.sum? Thanks.
    if (copy_op != nullptr) {
      auto kv = unique_grads.find(entry.sum);
      if (kv == unique_grads.end()) {
        unique_grads.emplace(std::move(entry.sum), std::make_pair(1, counter));
      } else {
        NodePtr copy_node = Node::Create();
        std::ostringstream os;
        os << entry.sum.node->attrs.name << "_" << kv->second.first << "_copy";
        kv->second.first++;
        copy_node->attrs.op = copy_op;
        copy_node->attrs.name = os.str();
        copy_node->inputs.emplace_back(entry.sum);
        if (copy_node->attrs.op->attr_parser != nullptr) {
            copy_node->attrs.op->attr_parser(&(copy_node->attrs));
        }
        unique_grads.emplace(NodeEntry{std::move(copy_node), 0, 0}, std::make_pair(1, counter));
      }
    } else {
        ret.outputs[counter] = entry.sum;
    }

https://github.com/dmlc/nnvm/blob/e842c098decf9f5eb6bd84e307c58e50078596b7/src/pass/gradient.cc#L208

According to the code, this is for situation where grad_xs contains duplicate NodeEntry so their gradient entries are also the same. It tries to make those gradient entries distinct by doing copy. Don't know the real application for this though.

@jermainewang Can you give me an example in practice?

Let's say you have a very simple graph that just adds 2 tensors: c =a+b. Normally when you see such addition you would want to omit copying (the gradient to a and b is the same so why make 2 tensors with the same information). But if this addition is on the edge of the graph as in this example, you may bind 2 actual tensors to those 2 output gradients. This fails because those 2 gradients are actually the same thing. That is why this code looks at the outputs of the graph and when it sees duplicates it makes them unique by applying copy. Does that make sense?

Got it. Thanks @ptrendx


Add docs for Manual OpenSUSE install

Name says it all. =]

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Signed CLA, so that's done. =]

Bumping this up for review.

Bumping again and hoping for a quick review. =]

@purp thanks for the additions! Sorry for the delay here


Allow GitHub compatible HTML tags in markdown contexts

It would be great if the set of HTML tags that work on GitHub also were allowed in MarkdownStrings.
AFAICT, those are
h1 h2 h3 h4 h5 h6 h7 h8 br b i strong em a pre code img tt
div ins del sup sub p ol ul table thead tbody tfoot blockquote
dl dt dd kbd q samp var hr ruby rt rp li tr td th s strike summary
details caption figure figcaption
abbr bdo cite dfn mark small span time wbr

(source).
Some work in that direction is already happening in #97793.

This is a duplicate. Unfortunately, I don't recall the exact issue number nor its title :flushed: Anyways, searching for issues with the same labels will likely help you :point_right:
https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aopen+label%3Afeature-request


Point GitHub links to csnx in source storybook docs

What are you changing?

Changes links to components in the Source stories from https://github.com/guardian/source/tree/main/packages/* to https://github.com/guardian/csnx/tree/main/libs/*
Changes blob to tree in some of the URIs for consistency

Why?

So we aren't pointing users to old copies of components stored in the archived Source repositories.

closes https://github.com/guardian/csnx/issues/290

great - can you add a patch release so these are installable?

thanks for reminding me, done 👍


(Suggestion) Highlight selected timeslot

I would suggest that there should be a way to highlight the selected timeslot, to increase user experience.
Maybe there is a way to do that, but I couldn't find

Okay i just published a new major version 1.2.0 which includes an internal state for the selected start time and a selectedStartTime Date prop if you want to override the internal state especially for displaying a pre-existing selection on mount.


Feature request - webconfig, OSC proxy, iPad commandset

Hi Marc!
This app is great! However, I'd like to suggest a few improvements. Before I start coding them myself, I thought I'd just asked if you wanted to consider them worth your time.

Add console IP & port config to the web frontend. The rest can remain in the default.json, but we're touring with our session and renting different consoles with various (fixed) IPs. This would make our setup easier and quicker.
Add a simple OSC proxy. The user would define a set of IPs of proxy clients. OSC messages originating from those on the list would be just resent to the console. Also, messages incoming from the console would be relayed to all of those clients on the list. Why? We're using an Ableton Live to feed click to the IEMs and the session also send commands to the console (FX send levels, channel levels etc.). As there is only one External control allowed, we sadly can't combine our solution with OSCWebMixer.
Add an option to use the iPad set of commands. With the functioning OSC proxy this would allow engineers to use their iPad as they're used to while using OSCWebMixer too. I guess it is just a handful of OSC addresses to sniff from a console. Besides that, there would be no need to copy the OTH OSC dosc file to the console anymore.

Let me know your thoughts, please. I believe these changes would skyrocket the app among users.
Best regards,
Lukas, vocal group SKETY, Czech Republic

Hi Lukas,
These are great suggestions. I really like the proxy idea as I've been wanting to trigger macros from a streamdeck but haven't had a way to send the messages to the desk with webmixer connected. I will get onto these and respond here as I make progress.

Hi Lukas,
I've made some progress with using the iPad set of commands. I managed to connect two iPads at once by bridging the connection with my laptop. The laptop is passing around the messages to each device. Its working just like a proxy so you should be able to send any message to the laptop and it will pass it on to everyone else. It should be fairly straight forward to add add webmixer connections as well!
Here is a video I made with my proof of concept:
https://www.youtube.com/watch?v=WYZMACbnirU
What do you think?

Hi Marc! Good job!!! Is this a standalone PoC or did you add the proxy functionality to the existing code?
Can't wait to test it myself!

This proof of concept is a standalone version that doesn't currently work with webmixer. I think I might start a separate github project so that you can test it. The changes are going to be fairly major and I don't want to break the existing setup.

So you don't even think of merging the two projects? Or you wanna make a fork of the original WebMixer?

I think they will merge together. I'm planning on creating an admin area so all the settings can be managed in the browser. This will save editing the config file and will make it easier for people to set it up. I'm also thinking about dropping the favorites feature because I have found that people don't use it and prefer to be able to see the entire list of channels. I think I will replace it with some way to group the channels together.

OK, cool! Be it with the basic functionality only - webmixer and oscproxy, even without the webconfig - I could give it a crash test on May 4+5 when we have rehearsals and then on May 15 in a live show. Just sayin' ;-) Keep up the good job! L.

Here is the new version I've been working on. It still needs a bit of testing and documentation created but we used it on the weekend and didn't have any issues. Please have a play with it and let me know what you think:
https://github.com/castles/OSCWebMixer2
I will close this issue now. Feel free to open up a new one in the new repository for any feedback.

Cool Marc! Can't wait to test it this weekend! Thanks!
Lukáš Prchal
t: +420 776 558 527
po 29. 4. 2024 v 14:38 odesílatel Marc @.***> napsal:

Here is the new version I've been working on. It still needs a bit of
testing and documentation created but we used it on the weekend and didn't
have any issues. Please have a play with it and let me know what you think:
https://github.com/castles/OSCWebMixer2
I will close this issue now. Feel free to open up a new one in the new
repository for any feedback.
—
Reply to this email directly, view it on GitHub
https://github.com/castles/OSCWebMixer/issues/6#issuecomment-2082622071,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHLB2YAFNWFSDXX7L6YB6ADY7Y5MDAVCNFSM6AAAAABFLGJIPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSGYZDEMBXGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


gru convert tflite err(KeyError: 'kernel')

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):windows 10, notebook(anconda)
TensorFlow installed from (source or binary): binary
TensorFlow version (or github SHA if from source):
tensorflow2.0.0 (use pip install)

Command used to run the converter or code if you’re using the Python API
converter = tf.lite.TFLiteConverter.from_keras_model(model)
tflite_model = converter.convert() # where is wrong
Copy and paste here the exact command
input_tensor = tf.keras.Input(shape=(28, 28))
x = tf.keras.layers.GRU(154)(input_tensor)
x = tf.keras.layers.Flatten()(x)
out_tensor = tf.keras.layers.Dense(10, activation="relu")(x)
model = tf.keras.Model(input_tensor, out_tensor)
model.summary()
converter = tf.lite.TFLiteConverter.from_keras_model(model)
tflite_model = converter.convert() # where is wrong
open("converted_keras_model.tflite", "wb").write(tflite_model)
The output from the converter invocation
KeyError: 'kernel'
# Copy and paste the output here.
KeyError                                  Traceback (most recent call last)
<ipython-input-7-b14fd83e63c7> in <module>()
     13
     14 converter = tf.lite.TFLiteConverter.from_keras_model(model)
---> 15 tflite_model = converter.convert()
     16 open("converted_keras_model.tflite", "wb").write(tflite_model)

~\AppData\Roaming\Python\Python36\site-packages\tensorflow_core\lite\python\lite.py in convert(self)
    403
    404     frozen_func = _convert_to_constants.convert_variables_to_constants_v2(
--> 405         self._funcs[0], lower_control_flow=False)
    406     input_tensors = [
    407         tensor for tensor in frozen_func.inputs

~\AppData\Roaming\Python\Python36\site-packages\tensorflow_core\python\framework\convert_to_constants.py in convert_variables_to_constants_v2(func, lower_control_flow)
    412
    413   # Get mapping from function name to argument types.
--> 414   function_data = _get_control_flow_function_data(node_defs, tensor_data)
    415
    416   # Get variable data for all nodes in `node_defs`.

~\AppData\Roaming\Python\Python36\site-packages\tensorflow_core\python\framework\convert_to_constants.py in _get_control_flow_function_data(node_defs, tensor_data)
    260         if arg_types[idx] == dtypes.resource:
    261           input_name = node.input[idx]
--> 262           arg_types[idx] = get_resource_type(input_name)
    263           output_shapes[idx] = get_resource_shape(input_name)
    264

~\AppData\Roaming\Python\Python36\site-packages\tensorflow_core\python\framework\convert_to_constants.py in get_resource_type(node_name)
    226
    227   def get_resource_type(node_name):
--> 228     numpy_type = tensor_data[node_name]["data"].dtype
    229     return dtypes.as_dtype(numpy_type).as_datatype_enum
    230

KeyError: 'kernel'

**Failure details**
conversion is  wrong.

**Any other info / logs**

Hi Renjie,
Any thoughts / guidance on the RNN stuff?

Hi Nupur, can you help take a look? thanks

@mkz0930 When I added experimental flag, everything worked as expected. Please add one line to your code after converter.
converter.experimental_new_converter = True
Please check the gist here. Thanks!
Please close the issue if it was resolved for you. Thanks!

@mkz0930 When I added experimental flag, everything worked as expected. Please add one line to your code after converter.
converter = tf.lite.TFLiteConverter.from_keras_model(model)
converter.experimental_new_converter = True
tflite_model = converter.convert()

Please check the gist here. Thanks!
Please close the issue if it was resolved for you. Thanks!

@jvishnuvardhan
I'm sorry, it didn't work by adding the flag. The error is the same.
My version is 2.0.0, is your version 2.1rc?

@mkz0930 The flag experimental_new_converter is not available in 2.0. Please use the tf-nightly (pip install tf-nightly).

thanks very much ,it works@gargn

This sounds familiar to me (see https://stackoverflow.com/questions/63303926/tfliteconverter-throws-keyerror-cond-1-input-1-0).
I sometimes get KeyError: 'cond_1_input_1_0' or KeyError: 'kernel'. Not quite sure yet what exactly causes what but does anybody else know maybe?


Vendor code still in my bundle

When I run my production build, I see script tags to cdn files for all my vendor packages.

However my app bundle still contains react in it. What am I missing here?

Hey @nojaf, thanks for opening an issue!
Can you please post your webpack config, if possible? Can you also post the versions of dynamic-cdn-webpack-plugin and webpack you are using?
This information will help in solving this issue as soon as possible.
Thanks for your patience.

Hi @aulisius
Sorry for not posting this right away.
const path = require("path");
const webpack = require("webpack");
const HtmlWebpackPlugin = require('html-webpack-plugin');
const MiniCssExtractPlugin = require("mini-css-extract-plugin");
const CopyWebpackPlugin = require('copy-webpack-plugin');
const DynamicCdnWebpackPlugin = require('dynamic-cdn-webpack-plugin');

function resolve(filePath) {
    return path.join(__dirname, filePath)
}

var babelOptions = {
    presets: [
        ["@babel/preset-env"],
        "@babel/preset-react"
    ]
};

var isProduction = process.argv.indexOf("-p") >= 0;
console.log("Bundling for " + (isProduction ? "production" : "development") + "...");

var commonPlugins = [
    new HtmlWebpackPlugin({
        filename: resolve('./output/index.html'),
        template: resolve('./src/App/index.html')
    })
];

const mode = isProduction ? "production" : "development";

const devServer = {
    contentBase: resolve('./static/'),
    publicPath: "/",
    port: 9090,
    hot: true,
    inline: true,
    historyApiFallback: true,
    proxy: {
        '/remote/*': {
            target: "http://localhost:8800",
            changeOrigin: true
        }
    }
};

const fsprojRule = {
    test: /\.fs(x|proj)?$/,
    use: {
        loader: "fable-loader",
        options: {
            babel: babelOptions,
            define: isProduction ? [] : ["DEBUG"],
            extra: {optimizeWatch: true}
        }
    }
};

const babelRule = {
    test: /\.js(x?)$/,
    exclude: /node_modules/,
    use: {
        loader: 'babel-loader',
        options: babelOptions
    },
};

const appConfig = {
    devtool: undefined,
    mode,
    resolve: {
        modules: [
            "node_modules/",
            resolve("./node_modules/")
        ]
    },
    entry: isProduction ? // We don't use the same entry for dev and production, to make HMR over style quicker for dev env
        {
            app: [
                resolve('./src/App/RonniesClient.fsproj'),
                resolve('./src/App/scss/main.sass')
            ]
        } : {
            app: [
                resolve('./src/App/RonniesClient.fsproj')
            ],
            style: [
                resolve('./src/App/scss/main.sass')
            ]
        },
    output: {
        path: resolve('./output'),
        filename: isProduction ? '[name].[hash].js' : '[name].js',
    },
    plugins: isProduction ?
        commonPlugins.concat([
            new MiniCssExtractPlugin({
                filename: 'style.css'
            }),
            new CopyWebpackPlugin([
                {from: './static'}
            ]),
            new DynamicCdnWebpackPlugin({
                env: mode
            })
        ])
        : commonPlugins.concat([
            new webpack.HotModuleReplacementPlugin(),
            new webpack.NamedModulesPlugin()
        ]),
    module: {
        rules: [
            fsprojRule,
            babelRule,
            {
                test: /\.(sass|scss|css)$/,
                use: [
                    isProduction ? MiniCssExtractPlugin.loader : 'style-loader',
                    'css-loader',
                    'sass-loader',
                ],
            },
            {
                test: /\.(png|jpg|jpeg|gif|svg|woff|woff2|ttf|eot)(\?.*$|$)/,
                use: ["file-loader"]
            }
        ]
    },
    devServer
};

module.exports = appConfig;

Hi
Can you try a few things:

do you have multiple version of react installed? (npm list react, yarn ls react)
try removing resolve.modules

Hello,
The suggestions didn't help although I believe this might be a false alert.
If I remove the cdn links from my index.html, my app doesn't work.
So I'm starting to believe that the screenshot with the React comment only contains the license part and not the actual React code. React is not part of my bundle  so no issue here.


Make devnet mergable with main

Description
Resolve merge conflicts between main and testnet branches.
Related Issue(s)
n/a
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[x] Other (please describe): merge conflicts

Breaking Change
n/a
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have run cargo fmt and cargo clippy to ensure my code is formatted and linted correctly
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable)
n/a
Additional Notes
n/a

closing in favor of merging devnet-ready => devnet


Padding issue from a timeline post

Is there a reason why the right padding value is set to 0 for li child ?

Another fix may be to add padding-right: 15px to the parent ul block.

Yes, you are right. The class follow-bubbles which holds the properties of the ul block comes from a third-party license file (). Where do you want to overload the ul property ?
The easiest solution would be to include at the beginning of the templates/user/index.hbs :
<style type="text/css">     .follow-bubbles{         padding-right: 15px;     } </style>
Don't really like this solution though...


Create per-instance configuration options

Concept taken from #9
These instance configuration options will be stored in a file, named milli_instance_settings.json, which will reside in the instance directory alongside the index directories. It can have a format similar to the following:
{
  "someDefaultSetting": "someDefaultValue",
  "indices": {
    "someUserGivenIndexName": {
      "milliVersion": "v1",
    }
  }
}

Instead of using a .json file, I am wondering if this should be done in heed directly.


Species subsetting for n= 1

In sampspec, subsetting by only a single species (e.g., sampspec(species = 'CHIL', gear = 'Drift')) does not work, Need a new else statement in the vicinity of line 134 in the code to accommodate such a condition.

Temporary workaround is to add a real, but obviously not-represented-in-the-data taxa. E.g., sampspec(species = c('CHIL', 'RBT'), gear = 'Drift')

Fixed in v 2.0 by adding a new else statement.


POL-128 - Schedule Report with Markup

Description
[Describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves]
Contribution Check List

[x] All tests pass.
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] New functionality has been documented in CHANGELOG.MD

@Ergocho69  @dfrankel33   This policy still need some updates.   please see my comments.


Extension not working on OperaGX

to my knowledge, operagx uses chromium like google chrome so i assumed the extension would work. added the url into the subscription + re-opened web browser as per instructions, but the ai images still appear on search

You shouldn't be using Opera GX in the first place: https://www.spacebar.news/stop-using-opera-browser/

Sorry, I know it doesn't solve the issue but you shouldn't be using Opera GX in the first place: https://www.spacebar.news/stop-using-opera-browser/

I agree, don't use Opera, i recommend switching to brave or chrome or something not Opera. Opera as been bought in 2016 by a chinese company and is collecting your data without you knowing or consenting to.

Use Vivaldi if you want a Chromium base or Librewolf if you want a Firefox base


Generate attribution from zip file with component list not working

Generate attribution with component list from a directory work fine.
However, if user zips the about_files directory to the about_files.zip and run attribution from there, it will not work.
The code will create a directory named about_files, the name of the zip file, under some tmp directory and extract the about_files.zip content to that directory and run from there
for instance,
/home/user/tmp/xxxbxbxbxx/about_files/about_files/test.h

The input has the /about_files/test.h as the about file path, so when running the attribution from the created tmp directory, /home/user/tmp/xxxbxbxbxx/about_files/, it will look for /about_files/about_files/test.h and that's why it is not working (because of the extra /about_files)
@DennisClark I would like to know if this also happens to MacOS or is it a windows issue?

Hi @chinyeungli  -- I am afraid that I don't completely understand what my test data should look like.  Based on your description, the data is what I normally do:

Run genabout to write everything under an /about_files/ directory.
Compress that to about_files.zip
Use about_files.zip as input when I run genattrib.

Are you saying that if I had another /about_files/ directory under my top-level /about_files/ directory that it would fail?  or some other condition?

@DennisClark Yes.
For instance, your non-zipped input is something like
/home/User/abc/about_files/

When you zip that directory and use it for attribution generation, the tool will create a about_files directory under a tmp location such as
/home/User/tmp/xxxxxaaaaa/about_files/

and then extract the content under it and become
/home/User/tmp/xxxxxaaaaa/about_files/about_files/

It is nothing wrong with your test data, I will see what can I do in the code side. Thanks for looking at it.

The code keeps the last segment of the base_loc to become the first segment of the returned path in the get_relative_path function
i.e.
the base_loc is
/home/User/tmp/xxxxxaaaaa/about_files/

and the about_file_path from the input begins with about_files, so when keeping the last segment from the base_loc and join the about_file_path, it will has 2 about_files segments in the returned path and this will not match with the about_file_path in the component list and causing errors.
In another example, if user has the following about_file_path for input:
/about_files/test.c

and generate about file at
/home/User/abc/about_files/test.c.ABOUT

Then, if the user zip it at the about_files/ level under the named 'metadata.zip', the code will extract this zip file in a tmp location such as /home/User/tmp/xxxxxaaaaa/metadata/about_files/test.c.ABOUT with base_loc = ``/home/User/tmp/xxxxxaaaaa/metadata/`
The previous code will use the last segment of the base_loc as the root of the returned path, the tool will try to match the /metadata/about_files/test.c.ABOUT with the about_file_path from the input component list in genattrib, and it will throw error because there is no /metadata/about_files/test.c.ABOUT about_file_path in the component list.

This has been fixed.


add support for update user profile

fixes #878

payload for updating user will not contain 'ID' ,  the ID would be deduced from the context token. Hence created new payload type in design/account.go
refactored existing tests by putting re-usable code into method(s).
added controller method for Update(..) in controller/users.go

./bin/alm-cli update users --key $ALM_TOKEN  --payload '{ "data":{ "type":"identities" ,  "attributes":{"bio":"something"}}}'  -H localhost:8080 --pp
2017/03/16 20:50:55 [INFO] started id=td0HtVgV PATCH=http://localhost:8080/api/users
2017/03/16 20:50:55 [INFO] completed id=td0HtVgV status=200 time=13.740488ms
{
    "data": {
        "attributes": {
            "bio": "something",
            "email"<EMAIL_ADDRESS>            "fullName": "Test2 User2",
            "imageURL": "https://www.gravatar.com/avatar/47e397200125043ae3ef960944b2a5e1.jpg",
            "providerType": "kc",
            "url": "",
            "username": "testuser2"
        },
        "id": "71f343e3-2bfa-4ec6-86d4-79b91476acfc",
        "links": {
            "self": "http://localhost:8080/api/users/71f343e3-2bfa-4ec6-86d4-79b91476acfc"
        },
        "type": "identities"
    }

Codecov Report

Merging #906 into master will decrease coverage by -0.01%.
The diff coverage is 70%.

@@            Coverage Diff             @@
##           master     #906      +/-   ##
==========================================
- Coverage   70.02%   70.02%   -0.01%
==========================================
  Files          86       86
  Lines        6495     6535      +40
==========================================
+ Hits         4548     4576      +28
- Misses       1534     1542       +8
- Partials      413      417       +4

Impacted Files
Coverage Δ

controller/users.go
80.74% <70%> (-4.53%)
:x:

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 fff472b...89447bc. Read the comment docs.

@sbose78 can the user update their username on the API? I don't think we can allow this as it changes all the URLs for their spaces on the UI and we don't plan to put any support for that in yet.

@sbose78 what validation rules are you applying on the server?

@pmuir , yeah the api doesn't support updating the username..

		updatedEmail := ctx.Payload.Data.Attributes.Email
		if updatedEmail != nil {
			user.Email = *updatedEmail
		}
		updatedBio := ctx.Payload.Data.Attributes.Bio
		if updatedBio != nil {
			user.Bio = *updatedBio
		}
		updatedFullName := ctx.Payload.Data.Attributes.FullName
		if updatedFullName != nil {
			user.FullName = *updatedFullName
		}
		updatedImageURL := ctx.Payload.Data.Attributes.ImageURL
		if updatedImageURL != nil {
			user.ImageURL = *updatedImageURL
		}
		updateURL := ctx.Payload.Data.Attributes.URL
		if updateURL != nil {
			user.URL = *updateURL
		}

Above are the only attributes that could be updated.
The current validation we have is all implicit ones based on the data type.


Using port 1935 as default.

When the URI specified on connection options does not have a port specified, it should use 1935, since its the default RTMP port.

thank you very much! this made it into rtmpsharp v0.4, now on nuget :D.
i appreciate all the pull requests, joao! 🎉


Rename proxy

another minor fix for proxy
% psql -h pg.neon.tech
NOTICE:  ☀  Welcome to Zenith!

Zenith should be Neon

wrong base, will reopen


nixos/containers: add wants and after dependency for network interfaces

Make sure network interfaces that should be moved into the container via
the --network-interface flag are available before starting the
container.
See the systemd-nspawn man page  in the section on the --network-interface flag https://www.man7.org/linux/man-pages/man1/systemd-nspawn.1.html
Motivation for this change
Autostart of a container fails if the interface is not available. A general dependency on network.target is not sufficient, e.g. in the case of VPN interfaces, that only get created after the network is online.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Any update?

This feature seems useful and important. Any update on this or reasons why this PR is not approved?

This feature seems useful and important. Any update on this or reasons why this PR is not approved?

Sadly only just noticed this now. Let's get it merged. :rocket:


outgoing-webhooks.md: Document example endpoint responses.

This is a follow-up to Tim's feedback on #9628.
@timabbott: FYI :) Are there any other example responses we should display here that I am not aware of perhaps?

This is helpful, but actually there's a few things that are wrong:

These are sample requests the Zulip server makes to your server, not sample responses.  So the text needs editing :)
We needs sample responses for what we'd like your server to return to implement this API correctly.  @rheaparekh may be able to provide advice on that.

I merged the first commit.  I think @showell can take over reviewing the second.

@eeshangarg I made a couple comments.  Feel free to ping me on chat to discuss the bigger picture of the changes I've made to outgoing webhooks.  It was mostly code cleanup, but I changed the field to "content".

Merged, thanks @eeshangarg and @showell!

I also added 8c8cb725bf0c50890025a372814814742e8e09ec on top to clarify some confusion in the documentation.


Contact Us page implemented

Contact Us page implemented
@ashwin31 @chaitu210 @AnjaneyuluBatta505

Coverage decreased (-1.2%) to 80.523% when pulling e6381509746aa2aab856f4812a2b89bc9f6123ae on vidyasagar-r:theme_integration_branch into 76b6223645737cd7dacd8ab0ce105c906e00aa06 on MicroPyramid:master.


1.x: LiveQuery.run() slow during pull replication

1.x: LiveQuery.run() is slower than Query.run().
When conducting an apples-to-apples comparison in testing the use of LiveQuery versus Query, it was observed that the LiveQuery.run() method takes considerably more time to return (between 200ms and 979ms). It was observed that this delay occurs when pull replication is ongoing at the same time.
The developer is querying a single document (startKey = endKey) with an all-docs query:
LiveQuery liveQuery = database.createAllDocumentsQuery().toLiveQuery();
liveQuery.setStartKey(myDocKey);
liveQuery.setEndKey(myDocKey);
liveQuery.setIndexUpdateMode(Query.IndexUpdateMode.AFTER);

liveQuery.addChangeListener(new LiveQuery.ChangeListener() {
public void changed(ChangeEvent event) {
//do something with event.getRows()
}
});

// This call takes up to 979 ms to return when pull replication is happening at the same time
QueryEnumerator run = liveQuery.run();

In general, LiveQuery.run() method executes more complex codes than Query.run() method. However, it should not cause 5 times slower.
LiveQuery.run() executes the query with a background thread which is a single thread work executor that is instantiated by Manager class. This thread is used from other methods including the Replicator's method. If this thread is busy for other tasks, it could make LiveQuery.run() task in the execution queue.
By creating other work executor thread for LiveQuery, LiveQuery will not be affected by the replication.


:zap: Simple get method for instance details

@durran If this looks good to you i ll merge and npm publish.  This way you can just use something like the below.
const getInstance = require('mongodb-instance-model').get;

class NativeClient {
  instance(done) {
    getInstance(this.database, done);
  }
}

:+1:

published in<EMAIL_ADDRESS>


Events placeholder

Created EventCard component and added it to the homepage as a placeholder for the events section
Resolves #9

I would also switch to using the NextJS image component for consistency as it works well with the vercel deploy

@RubenOlano I switched to NextJS Image but I'm having trouble resizing it. My code is in the image below. Right now the background fills up the whole screen. The image isn't being contained inside the div and just covers the whole screen. Any idea on how to fix this?

Something along these lines, contain and fill will just fill up its parent container which I think is being set to the full screen.

@RubenOlano I switched to NextJS Image but I'm having trouble resizing it. My code is in the image below. Right now the background fills up the whole screen. The image isn't being contained inside the div and just covers the whole screen. Any idea on how to fix this?

You're gonna have to set the height/width manually. The `layout='fill'` and 'objectFit='contain'` makes it fill up it's parent component which is why its taking up the entire screen


RSS feed is dead

https://blogs.technet.microsoft.com/sysinternals/rss.xml (from https://technet.microsoft.com/de-de/sysinternals) is dead.

See if this helps https://blogs.technet.microsoft.com/sysinternals/feed/

Thanks!
Looks good so far.
Looks like i already change it to this one and just forgot to post it here :D

@beerisgood  Issue may be closed now.


Add support for env and fatalError options

env allows to specify environment variables that will be passed to the command being executed.
fatalError allows to change the default behaviour that will abort the test execution, allowing for expected failures to not break k6.
When the command exits with a non zero error code, the stderr will be returned instead of the stdout

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Ronaldo Maia seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

The env option here would be quite awesome, wondering if there's a chance to decouple the two features of this PR and get env options in sooner @romaia ?

Hi @romaia. We are closing this PR due to inactivity. If you are still interested in moving this PR forward based on our discussion please re-open it.


ArchLinux (Monjaro)

Hi, how to install on ArchLinux?
I install only "maven".
These packages are not in the repository: default-jdk default-jre openjdk-8-jdk openjdk-8-jre zlib1g-dev libncurses5-dev lib32z1 lib32ncurses6
sAINT start up but there are mistakes
          +---------------------------------------------------+
    (__)  | WARNING: Use Gmail account only!                  |
 (|)(00)  | E-mail will be sent when it reaches the specified |
  |/(__)\ | number of characters. Optionally you can enable   |
  |_/ _|  | Screenshot, Webcam Capture and Persistence.       |
          +---------------------------------------------------+

 GENERATE SPYWARE
 --------------------------------------------

 [*] Enter your E-mail<EMAIL_ADDRESS>
 [*] Enter your Password: ***
 [*] Enable Screenshot (Y/n): y
 [*] Enable WebCam (Y/n): y
 [*] Enable Persistence (Y/n): y
 [*] Keep data on the computer? (Y/n): y
 [*] Enter the number of characters to send E-mail: 500

 +------------------------------------------+
   Email<EMAIL_ADDRESS>   Password: ***
   Screenshot: true
   Webcam: true
   Persistence: true
   Keep Data: true
   Number of characters: 500
 +------------------------------------------+

 [*] The information above is correct? (y/n): y

 [*] Compiling...
 [*] Successfully compiled in target/ folder.

 [*] You would like to generate .EXE using lauch4j? (y/n): y

 [*] Generating...
java.io.FileNotFoundException: launch4j/saint-1.0-jar-with-dependencies.exe (Нет такого файла или каталога)
        at java.base/java.io.FileInputStream.open0(Native Method)
        at java.base/java.io.FileInputStream.open(FileInputStream.java:213)
        at java.base/java.io.FileInputStream.<init>(FileInputStream.java:155)
        at saint.ui.sAINT.copyFile(sAINT.java:168)
        at saint.ui.sAINT.showGenerator(sAINT.java:399)
        at saint.ui.sAINT.showMenu(sAINT.java:221)
        at saint.ui.sAINT.main(sAINT.java:71)
 [*] Failed to generate file!

 NOTE: Allow access to less secure apps on your gmail account.
 -> https://www.google.com/settings/security/lesssecureapps

You should install:
pacman -S maven, jdk8-openjdk, jre8-openjdk, ncurses, lib32-zlib

You should install:
pacman -S maven jdk8-openjdk jre8-openjdk ncurses lib32-zlib

hey bro I don't receive any emails while the saint-1.0-jar-with-dependencies.jar is runing successfully on target host and the https://myaccount.google.com/lesssecureapps is turned on.
please help me !

You should install:
pacman -S maven jdk8-openjdk jre8-openjdk ncurses lib32-zlib

hey bro I don't receive any emails while the saint-1.0-jar-with-dependencies.jar is runing successfully on target host and the https://myaccount.google.com/lesssecureapps is turned on.
please help me !

Sorry, but I'm not a creator and have not used it since the last comment was written, for reasons I have already forgotten =)


[DNM] kv(ticdc): group events by region to vectorize events processing.

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/run-kafka-integration-test
/run-integration-test

/run-verify-ci
/run-kafka-integration-test

/run-all-tests


c2me incompatibility / thread safety issue

Goety is currently incompatible with Concurrent Chunk Management engine. It crashes/errors trying to spawn an obsidian monolith during structure generation. It looks like the getLevel().random in this line is not thread-safe: https://github.com/Polarice3/Goety-2/blob/9d6215dfb3b2fc03ce4e56b62249574105bfd618/src/main/java/com/Polarice3/Goety/common/entities/neutral/AbstractMonolith.java#L84
Crash log here, but it's not a clean install. I'll try to get a minimal reproduction later. I did not upload the full log because it's 76k lines long, but I can attach it if necessary.

I have no clue if this would fix this issue or not, but hopefully it does:
https://github.com/Polarice3/Goety-2/blob/ab572945245c68956279a03135164e3fa9865ae3/src/main/java/com/Polarice3/Goety/common/entities/neutral/AbstractMonolith.java#L84

Is that change included in <IP_ADDRESS>?

Yes

Thanks! I'll give it a shot.

Works perfectly! Thanks a bunch.


[ServiceBus] Remove unneeded mgmt properties

Remove is_anonymous_accessible from management entities.
Remove support_ordering from create_queue and QueueProperties
Remove enable_subscription_partitioning from create_topic and TopicProperties

/azp run python - servicebus - tests


adding token data and logo png

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address: ADcrbtXkfpos5Z989zAr1KbjG4mXanwJYboaXKS749sm
Token Name: Reedify Finance
Token Symbol: REFI
Logo : https://cdn.jsdelivr.net/gh/reedifyfinance/reedify/logo.png
Link to the official homepage of token: https://reedify.finance/
twitter : https://twitter.com/ReedifyFinance/
telegram : https://t.me/ReedifyFinance/
medium : https://reedify.medium.com/

@bartosz-lipinski


[Improvement] Completely solve the issue about Gravitino java docs link

What would you like to be improved?
We should permanently resolve the link between Java docs.
How should we improve?
No response

After discussing with @ch3yne , the Java docs links are correct, so I temporarily close it.


Updates for compatibility with rspec-expectations

Thanks for doing this. Unfortunately it exposes the fact that we can't release the BuiltIn module until a 2.9 release, but I guess that's good to know :)

@myronmarston assuming you mean doing that in rspec-expectations for the constants that rspec-rails depends on, yes, I think that could work, though I don't think there's any need to visibly deprecate them since rspec-rails is the only consumer. Agree?

@myronmarston actually we'll have to do it in both libs because people might upgrade to rspec-rails-2.8.2 and not rspec-expecations-2.8.1. All the combinations within 2.8.x need to play nice together.

Hmm...one other potentially simpler idea.  In rspec-rails, we could do something like:
module RSpec
  module Matchers
    unless defined?(BuiltIn)
      module BuiltIn
        include ::RSpec::Matchers
      end
    end
  end
end

Essentially, have rspec-rails make a dummy BuiltIn module with all of the constants in RSpec::Matchers for compatibility, if there's not a BuiltIn module already there.


Add user application records creation

Related to #666
@JS-BC - please review - did this with Copilot Workspaces as  a fun demo

For more details, open the Copilot Workspace session.

@AN-BC code generated was close, I made some adjustments

Ok. I’m fine with the developer role being default then, I don’t want to do the other issue for now, let’s close that one out. We need a broader think on users/roles/settings in the system for authorization as we look ahead to version 3.0 of the framework. Let’s make sure our installation docs explain the config file and that this is the default role setup.
From: JS-BC @.>
Sent: Thursday, December 19, 2024 11:29 AM
To: MemberJunction/MJ @.>
Cc: Amith Nagarajan @.>; Mention @.>
Subject: Re: [MemberJunction/MJ] Add user application records creation (PR #668)
@JS-BC commented on this pull request.

In mj.config.cjshttps://github.com/MemberJunction/MJ/pull/668#discussion_r1892826255:

@@ -201,10 +201,12 @@ const mjServerConfig = {

 autoCreateNewUsers: true,

 newUserLimitedToAuthorizedDomains: false,

 newUserAuthorizedDomains: [],

newUserRoles: ['UI'],

newUserRoles: ['UI', 'Developer'],

If we modify the UI role to have more capabilities, then I think it should work like how the Basic role is defined in this issue: #671https://github.com/MemberJunction/MJ/issues/671
—
Reply to this email directly, view it on GitHubhttps://github.com/MemberJunction/MJ/pull/668#discussion_r1892826255, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXGYIQJS7XWU64RCVNY2APD2GL64XAVCNFSM6AAAAABTGQB73KVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMJVGY4TSOJXGY.
You are receiving this because you were mentioned.Message ID: @.***>


Update the icon and name of Twitter to X(Twitter)

Fixes 43206
Update the icon and name of Twitter to X(Twitter)

Please look at https://deploy-preview-43583--kubernetes-io-main-staging.netlify.app/community/
That page still shows Twitter's old icon.

This is a fix for issue https://github.com/kubernetes/website/issues/43206
/retitle Replace "Twitter" logo with "X" logo on Community page

Hi @chansuke Please rebase your changes.

/lgtm

/approve


Clean up value value set descriptions from testing metadata

This will avoid having to regenerate all baselines every time we configure a new calling form for a language so it's included in tests.

Codecov Report

Merging #2098 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2098   +/-   ##
=========================================
  Coverage     86.83%   86.83%
  Complexity     4987     4987
=========================================
  Files           448      448
  Lines         19722    19722
  Branches       2080     2080
=========================================
  Hits          17125    17125
  Misses         1889     1889
  Partials        708      708

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 a1c4651...31dcfa2. Read the comment docs.


Make convenient injection points for baseline reads.

See #313. This cherry-picks the parts that affects plans. Needs work.

It will be easier to rewrite this to to refactor it over #345. Closing.


Import Error

Hi
there is error Coming in:
good-first-issue/gfi/tweet.py
Line -18              => LOGGING_CONFIG
Line-22,23,24     => Import error (Unable to Import toml)

pip3 install toml  ?

@sonuishaq67  I have tried but not get Result.
you also try it and let me Know the result
This is My first time.

works fine for me

ok, Maybe Installation Problem form my side.
Thank you


adding single file module functionality

This change seemed to work for me. (ie I get the same results when running realreq -s tests -d vs  realreq -s tests/test_realreq.py -d
I tried adding a few new test cases in the parameters to the test fixture for source_files, but wasn't able to get them to pass before I had to step away.  The test changes are in the draft (even though they aren't passing) so that you can take a look in case you have a chance to look this over before I do.

Nice! I'll see if I can get a look at this this weekend. Have a great Thanksgiving!


[WIP] Admin can configure slider content

PT: https://www.pivotaltracker.com/story/show/161037720

Pull Request Test Coverage Report for Build 1

18 of 18 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 96.829%

Totals

Change from base Build 86:
0.1%

Covered Lines:
397

Relevant Lines:
410

💛  - Coveralls


[Bug]: API Enums spaces as x0020 , enum parsing API.

Describe the issue
All space signs are converted to

Therefore enums are not working as expected. For example blocked enum " " was parsed to "0020"
Some endpoints are not even recreated, there are not existent in metadata when API has field with enum datatype..
Expected behavior
Parsing should be fixed.
Steps to reproduce
Create enum with space in name of value.
Create Page api with such enum field.
Check endpoint data with http GET query.
Additional context

I will provide a fix for a bug

[X] I will provide a fix for a bug

Hi @Klechaa. We have just experienced this issue by a customer.
Is there any information on when this version should be live for OnPrem installations? Also any additional information would be great to provide some context on why this happens.
AJ mentions this on his blog, but provides no further explaination on the values:
https://www.kauffmann.nl/2020/12/10/business-central-api-v2-0/


Testing grove with curl -x or curl --proxy results in a parent selection error.

When using curl -x or curl --proxy with grove, the remapped request uri is not created properly as remap.RequestRemapURI() expects a relative uri without scheme and host.

This isn't really a bug, Grove has never supported being used as a Forward Proxy.