BUG REPORT: webpack-isomorphic-tools error - asset not found

I'm upgrading to Node 6 (v6.1.0), so I've cloned the last version of este and followed all steps.
The first gulp launch was successful, but the second caused these errors:

$ gulp
[12:08:55] Requiring external module babel-register
[12:08:57] Using gulpfile c:\este-app\gulpfile.babel.js
[12:08:57] Starting 'env'...
[12:08:57] Finished 'env' after 16 ms
[12:08:57] Starting 'server'...
[12:08:57] Starting 'server-hot'...
[bg] Starting node ./webpack/server
[12:08:57] Finished 'server-hot' after 3.31 ms
[12:08:57] Starting 'server-nodemon'...
[bg] Starting node_modules.bin\nodemon.cmd --ignore webpack-assets.json src\server
[nodemon] 1.9.2
[nodemon] to restart at any time, enter rs
[nodemon] watching: .
[nodemon] starting node src\server
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/app/App.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/auth/Login.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/fields/FieldsPage.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/firebase/FirebasePage.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/firebase/Login.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/firebase/Users.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/lib/Loading.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/me/MePage.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/todos/Buttons.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/todos/NewTodo.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/todos/Todo.scss
Server started at http://localhost:8000
Hash: e10979d6dbf3a922bee8
Version: webpack 1.13.1
Time: 9061ms
Asset     Size  Chunks             Chunk Names
app.js  8.17 MB       0  [emitted]  app
1-fcd4ade39144b06dfa4a.js   898 kB       1  [emitted]
webpack built e10979d6dbf3a922bee8 in 9061ms
I didn't have any problems with older Node version (v5.10.1).
Do you have any idea how to fix this problem?

Did you try restart? No kidding, I think it's cached somehow.

Feel free to reopen it.

After restart it still throws these errors.

Please try to update dependencies. It works for me and the others so the problem is probably on your side.

I deleted node_modules and install dependencies again and it still doesn't work.

Check your package.json versions please.

Anyway, there is nothing specific in Este, so you should check webpack-isomorphic-tools issues instead.

Same:

[17:58:34] Requiring external module babel-register
(node:8788) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
(node:8788) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
[17:58:35] Using gulpfile f:\vixusapp\este-app\gulpfile.babel.js
[17:58:35] Starting 'env'...
[17:58:35] Finished 'env' after 40 ms
[17:58:35] Starting 'server'...
[17:58:35] Starting 'server-hot'...
[bg] Starting node ./webpack/server
[17:58:35] Finished 'server-hot' after 14 ms
[17:58:35] Starting 'server-nodemon'...
[bg] Starting node_modules.bin\nodemon.cmd --ignore webpack-assets.json src\server
[nodemon] 1.9.2
[nodemon] watching: . any time, enter rs
[nodemon] starting node src\server
[nodemon] restarting due to changes...
[nodemon] starting node src\server
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/app/App.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/auth/Login.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/fields/FieldsPage.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/firebase/FirebasePage.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/firebase/Login.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/firebase/Users.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/lib/Loading.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/me/MePage.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/todos/Buttons.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/todos/NewTodo.scss
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/todos/Todo.scss
Server started at http://localhost:8000
Hash: 94c1c8358461675fe896
Version: webpack 1.13.1
Time: 6347ms
Asset    Size  Chunks             Chunk Names
app.js  8.2 MB       0  [emitted]  app
1-73e00e053f2297e9bd60.js  899 kB       1  [emitted]
webpack built 94c1c8358461675fe896 in 6347ms

And in browser looks like (http://i.imgur.com/htcUvzx.png) with error.

I saw this, and if I remember correctly, you need to delete /webpack-assets.json .

Nope, after delete and run gulp /webpack-assets.json generated same as before.

Feel free to investigate it, because it just works for me.

Just had it happen again on a new project. It happens (at least) when you change the scss structure. I removed the all of the SCSS files from the src/client as well as their imports. Changed the src/client/App/App.react.js to import:
import '../styles/main.scss';
Ran gulp and saw the error. Stopped gulp. I deleted the /webpack-assets.json, ran gulp again and it worked:
[rkoberg@imac ~/Sites/fwi/apps (master *)]$ gulp
[13:54:13] Requiring external module babel-register
(node:8813) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
(node:8813) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
[13:54:21] Using gulpfile ~/Sites/fwi/apps/gulpfile.babel.js
[13:54:21] Starting 'env'...
[13:54:22] Finished 'env' after 406 ms
[13:54:22] Starting 'server'...
[13:54:22] Starting 'server-hot'...
[bg] Starting node ./webpack/server
[13:54:22] Finished 'server-hot' after 16 ms
[13:54:22] Starting 'server-nodemon'...
[bg] Starting node_modules/.bin/nodemon --ignore webpack-assets.json src/server
[nodemon] 1.9.2
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: *.*
[nodemon] starting `node src/server`
[webpack-isomorphic-tools] [error] asset not found: ./src/browser/styles/main.scss
Server started at http://localhost:8000
Hash: e30e7c19017ffb934e7f
Version: webpack 1.13.1
Time: 14001ms
                    Asset    Size  Chunks             Chunk Names
                   app.js  8.2 MB       0  [emitted]  app
1-0c97fff805f86d1c9a23.js  898 kB       1  [emitted]
webpack built e30e7c19017ffb934e7f in 14001ms
^C
[rkoberg@imac ~/Sites/fwi/apps (master *)]$ gulp
[13:55:07] Requiring external module babel-register
(node:8833) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
(node:8833) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
[13:55:08] Using gulpfile ~/Sites/fwi/apps/gulpfile.babel.js
[13:55:08] Starting 'env'...
[13:55:08] Finished 'env' after 8.06 ms
[13:55:08] Starting 'server'...
[13:55:08] Starting 'server-hot'...
[bg] Starting node ./webpack/server
[13:55:08] Finished 'server-hot' after 3.05 ms
[13:55:08] Starting 'server-nodemon'...
[bg] Starting node_modules/.bin/nodemon --ignore webpack-assets.json src/server
[nodemon] 1.9.2
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: *.*
[nodemon] starting `node src/server`
[webpack-isomorphic-tools] (waiting for the first Webpack build to finish)
[webpack-isomorphic-tools] (waiting for the first Webpack build to finish)
[webpack-isomorphic-tools] (waiting for the first Webpack build to finish)
Hash: e30e7c19017ffb934e7f
Version: webpack 1.13.1
Time: 7143ms
                    Asset    Size  Chunks             Chunk Names
                   app.js  8.2 MB       0  [emitted]  app
1-0c97fff805f86d1c9a23.js  898 kB       1  [emitted]
webpack built e30e7c19017ffb934e7f in 7143ms
Server started at http://localhost:8000

Can you suggest fix please?

I removed all scss imports, but still have javascript errors.
I think problem may be around how node 6.x process path on windows platform.
Developer of webpack-isomorphic-tools create new project without hack Node require() https://github.com/halt-hammerzeit/universal-webpack
May be think to move on it?

Interesting! I have to check it.

Like I wrote, the fix for me was to, stop gulp (the server), remove /webpack-assets.json, run $ gulp again

I try on clean Ubuntu for experiment.

Hmm, can you guys just try to reload browser manually after that error?

Already.
Every time: _object-gopn-ext.js?d238:6Uncaught TypeError: (0 , _typeof3.default) is not a function

Try to found .default in /src and replace it with nothing. Maybe Babel add-module-exports issue.

As I understand, everywhere default needed.
Pavel@P-LPM-PC-1 /f/vixusapp/este-app/src (master) $ grep -rnw './' -e '.default' ./common/configureBluebird.js:7:require('babel-runtime/core-js/promise').default = Bluebird; ./common/lib/redux-fields/fields.js:35:        default: return path; ./native/index.js:5:  require('./initialState').default.intl.locales ./server/intl/loadMessages.js:14:      descriptors: require(../../../messages/${fileName}).default, ./server/lib/errorHandler.js:24:    default() {

And additional error in server console:

asset not found: ./src/browser/home/50x50.png

reinstall npm to 3 and npm i again pls

npm install npm -g
`--<EMAIL_ADDRESS>

npm i
done

But all the same.

Thank you. So please try plain webpack-isomorphic-tools usage, probably some their boilerplate. If it will work, try to compare what Este does differently. Thank you again.

On clean Ubuntu, clean install este  - fly normal.

So it's only Windows problem?

I think so. And as I say before it's problem with processing pathes in Node 6.x on Windows.
Before I meet similar problem with different packages in Node 6.x on Windows, cannot remember issue in Node, but they already work on it.

At now I create linux dev environment for dig este. )

I saw the problem on OSX. You could probably see this by
I saw the problem on OSX, node 6.2.0, npm 3.8.9
To reproduce:

doing an install,
start server and hit a page,
stop the server,
change the scss import in App.react.js to some new scss file
restart server and you should see the error for asset not found [new scss]

I just added Foo.scss and it works. OSX and Node 6.1.0 here.

So I got [webpack-isomorphic-tools] [error] asset not found: ./src/browser/app/Foo.scss, but new stylesheet works and after app restart the message disappeared.

Please check https://github.com/halt-hammerzeit/webpack-isomorphic-tools/search?q=asset+not+found&type=Issues&utf8=✓ for possible solution.


Fix CVE-2018-12356 by hardening the regex.

Detailed write-up here:
https://neopg.io/blog/pass-signature-spoof/

Can you describe how this change makes the script no longer vulnerable?

Hu Earlz. Using [GNUPG:] is incomplete due to the complexity of parsing gpg output. Forcing the regex to check at the beginning tightens it down, and prevents signature spoofing.


Make use of TextUtil#pluralize

This is fairly negligible, but I came across some code that should be making use of the pluralize method.

I have added in the blank lines.
Do you think a cop could be written for this? I was thinking that I needed the blank lines, but RuboCop passed so I left them out.

We don't have a cop that check for blank lines after a block of include/extend/etc, but I'm pretty sure we have a cop checking for blanks lines after method definitions. Guess it might be buggy.
If there's one pretty important cop missing it's probably this one - https://github.com/bbatsov/ruby-style-guide#consistent-classes


feature/py3 compatibility

update gitignore
'modernize' codebase for py-3
Update knob_scripter

welp. i ended up doing most of this manually so i learn what the differences in python3 are ... thank you kindly anyway for this pull request from which I still learned things!


Getting exception configuration must contain projectId

Using a very basic configuration, I get an exception configuration must contain projectId. see below for usage. absolutely certain that the instantiated sanityClient is working since all my groq queries are fine. Is there anything that I might be missing?
sanity/client 2.1.4
next 10.0.5
// Image wrapper
import Img from 'next/image';
import {useNextSanityImage} from 'next-sanity-image';
import sanityClient from '@/services/sanityClient';

const Image = ({src}) => {
  const imageProps = useNextSanityImage(sanityClient, src);

  return <Img {...imageProps} />;
};

export default Image;

edit: I added @sanity/image-url and updated it 0.140.19, and the error changed to  Cannot read property 'split' of undefined coming from the hook 🤔

edit: I added @sanity/image-url and updated it 0.140.19, and the error changed to  Cannot read property 'split' of undefined coming from the hook 🤔

Thanks for opening this issue. From the looks of it this has to do with instantiating SanityClient. I'm not sure if this has anything to do specifically with this library. The specific error occurs from the following line:
https://github.com/sanity-io/sanity/blob/a3f7158016d63728a9b435e6ab444ff2b90fd424/packages/%40sanity/client/src/config.js#L56. Could you confirm that this is the case from your error logs?
This library only passes the SanityClient instance to @sanity/image-url, and even has @sanity/client as a peer dependency so it should not be in the way of your own dependency versioning. Does the issue only occur when having next-sanity-image installed?

Thanks for opening this issue. From the looks of it this has to do with instantiating SanityClient. I'm not sure if this has anything to do specifically with this library. The specific error occurs from the following line:
https://github.com/sanity-io/sanity/blob/a3f7158016d63728a9b435e6ab444ff2b90fd424/packages/%40sanity/client/src/config.js#L56. Could you confirm that this is the case from your error logs?
This library only passes the SanityClient instance to @sanity/image-url, and even has @sanity/client as a peer dependency so it should not be in the way of your own dependency versioning. Does the issue only occur when having next-sanity-image installed?

Closed due to inactivity.

Was there a solution found for this @ezekielaquino ?

quitequinn have you found solution?

I'm now receiving error when visiting some pages.
Configuration
const client = createClient({
  projectId: process.env.SANITY_PROJECT_ID,
  dataset: process.env.SANITY_DATA_SET,
  apiVersion: "2022-03-25",
  useCdn: false

npm: 8.11.0
node: v16.15.1:
next-sanity": "^4.1.1",

I did.. but I think it was something silly like I had the wrong character in the .env or something like that. Make sure your environmental variables are being loaded in correctly!

when saving a variable on .env file
VARIABLE_NAME(space) = (space) variable
Otherwise without the space it generates this error

import { SanityClient } from '@sanity/client';
import {imageUrlBuilder} from '@sanity/image-url';
export const client = new SanityClient({
projectId: process.env.ReactTokenId,
dataset: 'production',
apiVersion: '2023-07-04',
useCdn: true,
token: process.env.ReactTokenId,
});
const builder = imageUrlBuilder(client);
export const urlFor = (source) => builder.image(source);
why is this not running

have you found a solution yet? that is the same error I am also dealing with

@mishraji900 does it work when you expicitly provide the projectId and token properties to your SanityClient instance? If that's the case, it might have something to do with your environment variables. If you're making this call from the client-side, the projectId environment variable should be present. However i would not recommend to expose your token on the client-side.
Besides that check which version of @sanity/client and next-sanity-image you're running. Make sure to check for compatibility in the peerDependencies of the package.json: https://github.com/lorenzodejong/next-sanity-image/blob/main/package.json.

@lorenzodejong
import { SanityClient } from '@sanity/client'
import imageUrlBuilder from '@sanity/image-url'
export const client = new SanityClient({
projectId : 'z2ptzyq6',
dataset : 'production',
apiVersion: '2023-7-26',
useCdn: true,
token: 'sk2v5YoYB5yatNa46F8GwhxfnRWzR5BvuLJSZe2vnw26nLl5mREpcHW9VCkXCL3qv9iX5JPANxjZ2kcclDZCyi3SfCcprnoFFXpuLzRDOQNcKgLJVZpLcwmWRh8wRyYVJGjWNCBXDDJMnDKw1xtFjZ4s3W1xZTcvD9PFTBhFzGm1OKEdzeMU'
}
)
const builder = imageUrlBuilder(client);
export const urlFor = (source) => builder.image(source);
This is my code, I am still experiencing the error:
Configuration must contain projectId

Go to sanity.config.js and input your projectId
projectId: 'projectId',
This worked for me

Configuration must contain projectId
i have checked all the above methods but this issue still persists


⚠ ClamAV has degraded performance

In 2934183, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 228 ms

Resolved: ClamAV performance has improved in d362b5a.


sqs: return the error when GetQueueUrl error is not QueueDoesNotExist…

When the GetQueueUrl fails we are not returning the error if it is not QueueDoesNotExist which makes it hard to find out what why the sqs consumer is not working.

rebasing should fix the failing static code analysis job


Flickering on first render (only ios, android is ok)

Current behavior
On first render, data is empy list, flickering on ios, android is ok
https://github.com/Shopify/flash-list/assets/1215843/1d0c6f18-0697-4201-bece-540288455823
Expected behavior
not flickering
To Reproduce
        <FlashList
            data={articles}
            estimatedItemSize={130}
            keyExtractor={({ id }, i) => id}
            showsVerticalScrollIndicator={false}
            renderItem={({ item, index }) => <ReplyItem reply={item}  />}
            onRefresh={ () => dispatch(fetchDetailTopic({ id, page: 1, order })) }
            refreshing={status === 'ing'}
            // refreshing={status === 'ing' && articles.length > 0}
            onEndReachedThreshold={0.3}
            onEndReached={() => { if (page < total && status != 'ing') dispatch(fetchDetailTopic({ id, page: page + 1,})) }} />

tmp fix
set refreshing = false, while data is empty.
Platform:

[x] iOS
[ ] Android

Environment
1.6.2

I'm also facing same kind issue when when list screen opened the entire screen moves up a bit and resets

It looks like the pull to refresh animation is being triggered. Is is possible that you're setting refreshing to true during mount?

Closing due to inactivity.


Error 'Condition "!is_visible_in_tree()" is true' when moving cursor in window with hidden TouchScreenButton

Godot version:

3.3-rc7
OS/device including version:

Windows
Issue description:

With rc7 I get lots of errors about _input: Condition "!is_visible_in_tree()" is true for touch screen buttons, which weren't showing before. Is this normal ?

Steps to reproduce:
With the minimal reproduction project, when the visibility of the touch screen button is turned on in the editor, it's ok. When the visibility is turned off, running the project and moving the mouse inside the window generates an error for every mouse event.
Minimal reproduction project:

TouchScreenButton.zip

My bad. I had a get_global_position() in the TouchScreenButton which fails when the button is hidden.

Still have them and these errors are new compared to previous versions.

Can you clarify which versions are "previous versions"?

A minimum reproduction project, or more detailed steps to reproduce, would be greatly appreciated. I tried to add a TouchScreenButton and hide/show it, doesn't raise any errors here in 3.3 RC 7.

The ones I've tested are 3.3-rc6, 3.2.rc4 and before. I also tried to isolate a quick MRP from the existing code base but didn't succeed so far. I'll dig it up and come back with details.

That's interesting, there hasn't been much change from 3.3 RC 6 to RC 7, and none that should affect TouchScreenButton: https://github.com/godotengine/godot/compare/15ff752737c53a1727cbc011068afa15683509be...cca2637b9b9dcb16070eb50a69c601a5f076c683

I nailed it down. Try the MRP

Thanks. To clarify, it's moving the mouse around in the window with visibility off when running the game (not in the editor).
I have the same issue in 3.3 RC 7, 3.3 RC 6, 3.2.3-stable, and all the way down to 3.1-stable (didn't test earlier releases) so it doesn't seem to be a regression.

Strange indeed that I noticed this for the first time. It's nasty enough as well, as there doesn't seem to be a workaround with code. The simple presence of the _input() function generates the error for mouse (any?) events.

I tried the reproduction project and I can confirm this issue is still present in Godot 3.4-stable.


patches table widget to send values on locale switch

This will allow datatable formwidgets within an mlrepeater or mlnestedform to send it's data via POST on a locale switch.
It patches the onFormSubmit function of the Table Widget and temporary sets the postback handler to the onSwitchItemLocale handler beeing executed.
closes #33

Closing in favour of a fix to the widget in the core, see https://github.com/wintercms/wn-translate-plugin/issues/33#issuecomment-1125769772


Depth Color turning on when preference is off

Sometimes I'll start gide and depth color will be off but then later it appears in the textview even though the flag is still set to false in the prefs.

Well first, I see now that all of the editing prefs are in both gide and project prefs and I'm always forgetting about project prefs.
Second, I'm wondering if it may be that the depth color is not spontaneously appearing but that I am moving from a part of the file without much depth coding to a portion where there is a lot.

Closing - will reopen if the problem is real!


accumulate: PEP8 compliance updated

The exercise accumulate had some inconsistency with PEP8 which I fixed (#214).
tmo$ flake8 accumulate/ --ignore=E501
./accumulate/accumulate_test.py:8:56: E226 missing whitespace around arithmetic operator
./accumulate/accumulate_test.py:12:37: E226 missing whitespace around arithmetic operator
./accumulate/accumulate_test.py:22:26: E226 missing whitespace around arithmetic operator
./accumulate/accumulate_test.py:22:31: E226 missing whitespace around arithmetic operator
./accumulate/accumulate_test.py:33:59: E226 missing whitespace around arithmetic operator
./accumulate/example.py:4:19: E231 missing whitespace after ','

Thanks


Lint status

Coverage remained the same at 90.291% when pulling 8b0240b81bb96525d783868a0393989058c50316 on lint-status into 66ce41ee779246e6e39c2292d72c35d838d15e33 on integrations.

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

Merging [#4][cc-pull] into [integrations][cc-base-branch] will not change coverage

@@           integrations         #4   diff @@
==============================================
  Files                 5          5
  Lines               662        662
  Methods              66         66
  Messages              0          0
  Branches            164        164
==============================================
  Hits                599        599
  Misses               63         63
  Partials              0          0

Powered by Codecov. Last updated by [66ce41e...df73967][cc-compare]
[cc-base-branch]: https://codecov.io/gh/quinkennedy/spacebrew/branch/integrations?src=pr
[cc-compare]: https://codecov.io/gh/quinkennedy/spacebrew/compare/66ce41ee779246e6e39c2292d72c35d838d15e33...df7396770777d6b80f80883fc25e91116846da34?src=pr
[cc-pull]: https://codecov.io/gh/quinkennedy/spacebrew/pull/4?src=pr

Coverage remained the same at 90.291% when pulling f40f39fdbba17c6f668636601624b5ddc3a34d4d on lint-status into 66ce41ee779246e6e39c2292d72c35d838d15e33 on integrations.


fix a typo

Signed-off-by: zhuanlan<EMAIL_ADDRESS>
What this PR does / why we need it:
fix a typo, needsSCSIControler -> needsSCSIController
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

/ok-to-test

/approve

/retest-required

Hi @Longchuanzheng, you need to add a release note otherwise the PR cannot be merged, in this case NONE is fine.

/retest

/retest

/retest

/retest

Hi @acardace, the pull-kubevirt-fossa test failed for many times and I think /retest can't deal with it. I would appreciate it if you can give me some advice.

/lgtm
it's ok, pull-kubevirt-fossa is not a required lane :)

/retest


When a value is unset, the URL retains the un-interpolated variable with curly brackets

In this use case, I'm using a "page" content type, and the site's Home page is the only page to have an empty slug. When previewing other pages with slugs set, everything works as expected, but when trying to preview the homepage, the created URL path is /en/{slug} (or /en/%7Bslug%7D after the browser's URL encoding).
Maybe the default for an empty variable such as the slug should be to interpolate the query URL with an empty string?
Here is my config:
'preview-button': {
  enabled: true,
  config: {
    contentTypes: [
      {
        uid: 'api::page.page',
        draft: {
          url: env('STRAPI_PREVIEW_DRAFT_URL'),
          query: {
            type: 'page',
            slug: '{slug}',
            secret: env('STRAPI_PREVIEW_SECRET'),
            locale: '{locale}',
          },
        },
        published: {
          url: `${env('STRAPI_PREVIEW_PUBLISHED_URL')}/{locale}/{slug}`,
        },
      },
    ],
  },
}

Followup: this seems to happen when a default value is not configured for a field, and Strapi has the value set to null. Specifying an empty string as the default value for slug here is a workaround for the issue.

Hi @zrothauser I can certainly use an empty string instead of ignoring the empty values 👍🏻 I'll be sure to include this in an upcoming release.

@zrothauser The latest release now includes this update so unmatched values are replaced with an empty string 👍🏻


ModuleNotFoundError: No module named 'asyncore' when using auto-cpufreq

Describe the bug
When using auto-cpufreq, an error occurs: ModuleNotFoundError: No module named 'asyncore'.
Steps To Reproduce
Steps to reproduce the behavior:

Enable auto-cpufreq

Expected behavior
The auto-cpufreq tool should run without any errors, and the asyncore module should be found and used properly.
Additional context
Additional logs: https://privatebin.net/?86b146232ad1c856#HJJ6dFyoQm3UosFftGsJZEFZt9AgFxiagDkMxdD4dvHy
Notify maintainers
@adamlwgriffiths
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 6.9.9, NixOS, 24.11 (Vicuna), 24.11.20240712.7e7c39e`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix Super) 2.22.0pre20240705_1b14f2e`
 - nixpkgs: `/nix/store/qmh8bas1qni03drm0lnjas2azh7h87cn-source`

#325685

This does not resolve the issue. Auto-cpufreq is still producing the same error, even after I updated the inputs.

You have to wait for the branch to advance, see nixpk.gs to know when it does


Fix broken link in README.md

Fix broken link

Badr can you give me the permission to merge without approval or at least decrease the number of necessary approvals? Some changes like this one do not need an approval.


quick demo about pointrcnn

Using the pre training model and the point cloud data file 000008.bin in the training set by Kitti data set, Visualizing the point cloud to view the prediction results, and I find that it is very different from the demo,
python demo.py --cfg_file cfgs/kitti_models/pointrcnn.yaml     --ckpt pointrcnn.pth     --data_path /home/data/kitti/000008.bin

Print my pred_ dicts
[{'pred_boxes': tensor([[  3.9855,   2.6731,  -0.8388,   3.4149,   1.5907,   1.5326,  -0.2600],
        [  8.1183,   1.1990,  -0.8320,   3.7272,   1.5938,   1.5666,   2.8324],
        [  6.4710,  -3.8736,  -1.0183,   2.9478,   1.5104,   1.4823,  -0.3161],
        [ 14.6815,  -1.0701,  -0.7700,   3.5354,   1.5662,   1.4859,  -0.3415],
        [ 20.5447,  -8.5958,  -0.9451,   3.1763,   1.6279,   1.5571,  -0.3452],
        [ 33.5792,  -7.0383,  -0.4803,   3.9992,   1.6634,   1.5867,   2.8569],
        [ 55.4581, -20.2513,  -0.5242,   4.1806,   1.7357,   1.5449,   2.8281],
        [ 25.0188, -10.1037,  -1.0143,   3.7758,   1.6138,   1.4882,  -0.3696],
        [ 28.6975,  -1.8450,  -0.2476,   3.5511,   1.5018,   1.4666,  -1.6337],
        [ 40.9106,  -9.1784,  -0.4699,   3.7668,   1.6025,   1.5252,   0.1146]],
       device='cuda:0'), 'pred_scores': tensor([1.0000, 1.0000, 1.0000, 1.0000, 0.9921, 0.9764, 0.8783, 0.7433, 0.7345,
        0.4599], device='cuda:0'), 'pred_labels': tensor([1, 1, 1, 1, 1, 1, 1, 1, 1, 1], device='cuda:0')}]

Why is my data much less than others? and how to solve this problem? please

same problem
how to solve ?


[Problem] Pythagorean triplet

Pythagorean triplet
Pythagorean triplet is a set of three natural numbers where a < b < c.
$a^2 + b^2 = c^2$
There is exactly one Pythagorean triplet for which $a + b + c = 1000$.

Find a, b and c and print these numbers out.

@Legoeggolas Your code doesn't work on g++ and clang. Update the project with your working code.

@Legoeggolas Your code doesn't work on g++ and clang. Update the project with your working code.

Double checked, works just fine for me.

What problems are you facing with the code?
I have intentionally commented out an include directive in the header file, maybe that is the cause of your issue?

@Legoeggolas Your code doesn't work on g++ and clang. Update the project with your working code.

Double checked, works just fine for me.  What problems are you facing with the code? I have intentionally commented out an include directive in the header file, maybe that is the cause of your issue?

It's not my issue. I tried my linux machines.  search starts here:
#include <...> search starts here:
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0/x86_64-pc-linux-gnu
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0/backward
/usr/lib/clang/14.0.6/include
/usr/local/include
/usr/include
End of search list.
 "/usr/bin/ld" -pie --eh-frame-hdr -m elf_x86_64 -dynamic-linker /lib64/ld-linux-x86-64.so.2 -o a.out /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64/Scrt1.o /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64/crti.o /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/crtbeginS.o -L/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0 -L/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64 -L/lib/../lib64 -L/usr/lib/../lib64 -L/lib -L/usr/lib /tmp/solution-f8d086.o -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s --no-as-needed /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/crtendS.o /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64/crtn.o /usr/bin/ld: /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64/Scrt1.o: in function _start': /build/glibc/src/glibc/csu/../sysdeps/x86_64/start.S:103: undefined reference to main'
clang-14: error: linker command failed with exit code 1 (use -v to see invocation)

@onurbbr
It is because my code is not supposed to run as-is.
You're supposed to either include the header (where a function is provided) and compile with that, or uncomment the main function (and necessary includes) in the header and source files.

@onurbbr It is because my code is not supposed to run as-is.
You're supposed to either include the header (where a function is provided) and compile with that, or uncomment the main function (and necessary includes) in the header and source files.

Activating main does not change the result. This time I'm getting a namespace error. When I enable Namespace, the linker cannot link the code you wrote.

[onur@archlinux pythagorean_triplet]$ clang solution.cpp
solution.cpp:37:10: error: no member named 'cout' in namespace 'std'     std::cout << "The values are: ";     ~~~~~^ solution.cpp:39:14: error: no member named 'cout' in namespace 'std'         std::cout << val << " ";         ~~~~~^ solution.cpp:42:10: error: no member named 'cout' in namespace 'std'     std::cout << std::endl;     ~~~~~^ solution.cpp:42:23: error: no member named 'endl' in namespace 'std'     std::cout << std::endl;                  ~~~~~^ 4 errors generated.
When I enable std and iostream it gives another linker error.
[onur@archlinux pythagorean_triplet]$ clang solution.cpp
/usr/bin/ld: /tmp/solution-aaf7f3.o: in function main': solution.cpp:(.text+0x1bf): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x1cb): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::operator<< <std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&, char const*)' /usr/bin/ld: solution.cpp:(.text+0x22a): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x22f): undefined reference to std::ostream::operator<<(int)' /usr/bin/ld: solution.cpp:(.text+0x23e): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::operator<< <std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&, char const*)' /usr/bin/ld: solution.cpp:(.text+0x256): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x25d): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::endl<char, std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&)' /usr/bin/ld: solution.cpp:(.text+0x262): undefined reference to std::ostream::operator<<(std::ostream& (*)(std::ostream&))' /usr/bin/ld: /tmp/solution-aaf7f3.o: in function __cxx_global_var_init': solution.cpp:(.text.startup+0xc): undefined reference to std::ios_base::Init::Init()' /usr/bin/ld: solution.cpp:(.text.startup+0x13): undefined reference to std::ios_base::Init::~Init()' clang-14: error: linker command failed with exit code 1 (use -v to see invocation)
Error details (--verbose)
[onur@archlinux pythagorean_triplet]$ clang solution.cpp -v
clang version 14.0.6 Target: x86_64-pc-linux-gnu Thread model: posix InstalledDir: /usr/bin Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-pc-linux-gnu/11.3.0 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-pc-linux-gnu/12.2.0 Found candidate GCC installation: /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/11.3.0 Found candidate GCC installation: /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0 Selected GCC installation: /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0 Candidate multilib: .;@m64 Candidate multilib: 32;@m32 Selected multilib: .;@m64 Found CUDA installation: /opt/cuda, version   "/usr/bin/clang-14" -cc1 -triple x86_64-pc-linux-gnu -emit-obj -mrelax-all --mrelax-relocations -disable-free -clear-ast-before-backend -disable-llvm-verifier -discard-value-names -main-file-name solution.cpp -mrelocation-model pic -pic-level 2 -pic-is-pie -mframe-pointer=all -fmath-errno -ffp-contract=on -fno-rounding-math -mconstructor-aliases -funwind-tables=2 -target-cpu x86-64 -tune-cpu generic -mllvm -treat-scalable-fixed-error-as-warning -debugger-tuning=gdb -v -fcoverage-compilation-dir=/mnt/SSD/Projects/GitHub/callofcode/challenges/C++/pythagorean_triplet -resource-dir /usr/lib/clang/14.0.6 -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0 -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0/x86_64-pc-linux-gnu -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0/backward -internal-isystem /usr/lib/clang/14.0.6/include -internal-isystem /usr/local/include -internal-isystem /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../x86_64-pc-linux-gnu/include -internal-externc-isystem /include -internal-externc-isystem /usr/include -fdeprecated-macro -fdebug-compilation-dir=/mnt/SSD/Projects/GitHub/callofcode/challenges/C++/pythagorean_triplet -ferror-limit 19 -stack-protector 2 -fgnuc-version=4.2.1 -fcxx-exceptions -fexceptions -fcolor-diagnostics -faddrsig -D__GCC_HAVE_DWARF2_CFI_ASM=1 -o /tmp/solution-ed3d52.o -x c++ solution.cpp clang -cc1 version 14.0.6 based upon LLVM 14.0.6 default target x86_64-pc-linux-gnu ignoring nonexistent directory "/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../x86_64-pc-linux-gnu/include" ignoring nonexistent directory "/include" #include "..." search starts here: #include <...> search starts here:  /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0  /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0/x86_64-pc-linux-gnu  /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../include/c++/12.2.0/backward  /usr/lib/clang/14.0.6/include  /usr/local/include  /usr/include End of search list.  "/usr/bin/ld" -pie --eh-frame-hdr -m elf_x86_64 -dynamic-linker /lib64/ld-linux-x86-64.so.2 -o a.out /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64/Scrt1.o /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64/crti.o /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/crtbeginS.o -L/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0 -L/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64 -L/lib/../lib64 -L/usr/lib/../lib64 -L/lib -L/usr/lib /tmp/solution-ed3d52.o -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s --no-as-needed /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/crtendS.o /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/12.2.0/../../../../lib64/crtn.o /usr/bin/ld: /tmp/solution-ed3d52.o: in function main': solution.cpp:(.text+0x1bf): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x1cb): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::operator<< <std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&, char const*)' /usr/bin/ld: solution.cpp:(.text+0x22a): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x22f): undefined reference to std::ostream::operator<<(int)' /usr/bin/ld: solution.cpp:(.text+0x23e): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::operator<< <std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&, char const*)' /usr/bin/ld: solution.cpp:(.text+0x256): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x25d): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::endl<char, std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&)' /usr/bin/ld: solution.cpp:(.text+0x262): undefined reference to std::ostream::operator<<(std::ostream& (*)(std::ostream&))' /usr/bin/ld: /tmp/solution-ed3d52.o: in function __cxx_global_var_init': solution.cpp:(.text.startup+0xc): undefined reference to std::ios_base::Init::Init()' /usr/bin/ld: solution.cpp:(.text.startup+0x13): undefined reference to std::ios_base::Init::~Init()'

I tried ubuntu server and ubuntu docker image. The result is the same.
root@ubuntu:~# clang solution.cpp
/usr/bin/ld: /tmp/solution-6d7d40.o: in function main': solution.cpp:(.text+0x182): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x18e): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::operator<< <std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&, char const*)' /usr/bin/ld: solution.cpp:(.text+0x1ed): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x1f2): undefined reference to std::ostream::operator<<(int)' /usr/bin/ld: solution.cpp:(.text+0x201): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::operator<< <std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&, char const*)' /usr/bin/ld: solution.cpp:(.text+0x219): undefined reference to std::cout' /usr/bin/ld: solution.cpp:(.text+0x220): undefined reference to std::basic_ostream<char, std::char_traits<char> >& std::endl<char, std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&)' /usr/bin/ld: solution.cpp:(.text+0x225): undefined reference to std::ostream::operator<<(std::ostream& (*)(std::ostream&))' /usr/bin/ld: /tmp/solution-6d7d40.o: in function __cxx_global_var_init': solution.cpp:(.text.startup+0xc): undefined reference to std::ios_base::Init::Init()' /usr/bin/ld: solution.cpp:(.text.startup+0x13): undefined reference to std::ios_base::Init::~Init()' clang: error: linker command failed with exit code 1 (use -v to see invocation)

@onurbbr
You are using clang instead of clang++
The former cannot compile C code unless you explicitly set some flags and linkers (even then I am unsure about that, given standards and all) and clang++ is the recommended way to compile C++ code. I'm sorry but I did not notice this earlier, since I am the most familiar with GCC and MSVC.
After you have added the appropriate include in the header file and uncommented the main function, it should compile properly.
Or, you can write your own main(), include the header, and compile that file with the given source file (the way I intended the program to be used). In hindsight, I should have used flags instead of commenting out deprecated code, which I will be sure to do in any other solutions I submit.

@onurbbr You are using clang instead of clang++ The former cannot compile C++ code unless you explicitly set some flags and linkers (even then I am unsure about that, given standards and all) and clang++ is the recommended way to compile C++ code. I'm sorry but I did not notice this earlier, since I am the most familiar with GCC and MSVC.
After you have added the appropriate include in the header file and uncommented the main function, it should compile properly. Or, you can write your own main(), include the header, and compile that file with the given source file (the way I intended the program to be used). In hindsight, I should have used flags instead of commenting out deprecated code, which I will be sure to do in any other solutions I submit.

Hmm. I tried with ubuntu vm and it worked. That's weird because my main machine is not compiling this code. Anyway problem is my main machine. It works on Raspberry Pi 4 too. Sorry for misunderstanding.

@onurbbr
You're very welcome, and I'm glad that we could come to a resolution. Have a nice day too, and happy coding!


improve changelog xtask

oops, I had already merged this, so the commits make no sense really, oh well!


[Feature] Google Ads search report to be  included

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
Similar to MIcrosoft search/ bing, the search report should be generated for Google platform.
Describe alternatives you've considered
No response
Are you interested in contributing this feature?

[ ] Yes.
[ ] Yes, but I will need assistance and will schedule time during your office hours for guidance.
[ ] No.

Anything else?
No response

Hi @mxmreporting, we are investigating this currently and plan to start work on this in the near future after we've properly scoped out the work that goes into bringing this report in.


⚠ Bofrost.lu has degraded performance

In ae34227, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 855 ms

Resolved: Bofrost.lu performance has improved in de7fd4d after 16 minutes.


NagiosPB: Populate Nagios Server Config Files From j2.templates For Windows Machines

NagiosPB: Populate Nagios Server Config Files From j2.templates
In an earlier part of the automation of nagios configuration, files with names similar to these
test-azure-win2012r2-x64-1.cfg
build-ibmcloud-win2012r2-x64-2.cfg
were created as a place holder to be used for populating the config files appropriately
Starting off with Windows on this issue, write ansible code and any supporting scripts to populate the content of these files appropriately. This can be done using jinja2 templates , such as the example ones provided
test-windows-template.j2
build-windows-template.j2
The logic should run along the lines of, if a host is a test & windows machine, its configuration should use the test-windows-template.j2 to populate its configuration file.
Thought should be given to how this mapping between service types, and machine description should be performed, in order to simplify the expansion. As this task will expand to cover all other service and host types.

Next issue in line , can you assign me ?

@lumuchris256 done!

Main PR is now merged


increase jit batch size progressively

Saves some enqueue time for big jit caches
llama 7b gpu6
enqueue in  39.35 ms
total  60.86 ms, 16.43 tok/sec

vs
enqueue in  52.92 ms
total  65.90 ms, 15.17 tok/sec

resnet beam=4
30  339.68 ms run,   36.58 ms python,  26.77 ms fetch data,  276.33 ms HSA * 6,  7.03 loss, 0.00 acc, 0.007821 LR, 94.53 GB used,  45560.44 GFLOPS

vs
30  369.88 ms run,   57.52 ms python,  24.03 ms fetch data,  288.33 ms HSA * 6,  7.03 loss, 0.00 acc, 0.007821 LR, 94.53 GB used,  41840.78 GFLOPS

This is great! Is there a upper limit how big a single batch can be?

llama 70B 4.66 tok/s!

I decided not to set an upper limit. I tested it (4096), and it really doesn't change anything. We could add one if something goes wrong

FYI this one might cause resnet training to hang on the second epoch. It might not be this one and I need to do more test, but 3 runs I included this hung, and the one that is on previous commit has trained more than 5 epochs as of now


Request: PreDB intergration

Is it possible to integrate http://pre.corrupt-net.org/

Im not sure what that is, i have to check it when i get home from work. The link was blocked websense...

@unixjeff I checked on my phone. what is this site used for? Can you download stuff from there or is it only a list of releases?

it keeps track of when release are released so if you wanted to check and see if a ep of a show is out there or a movie you can search "Scene" and see if its out

Honestly I don't see the point of there isn't a api or atleast you can download torrent s from there. I'll rather add more torrent providers. If any takes the time to make this in decent way I'll still merge it. But I won't be coding this myself :)

you can see what 'officially' got released and can make the usenet searches more pointed. but torrent sites usually have moderation anyways on what gets in so i dont see much need as well.

nZedb uses this and ZNC to keep an IRC connection. I use it there.

Ok


Kindeauth integrated using react

-Installed @kinde-oss/kinde-auth-react library for KindeAuth.
-Updated Navbar component to reflect the authentication state (login, register, logout).
-Created .env file to handle environment variables securely.
-Updated App.js to wrap the application in the KindeProvider for authentication context.

Summary by CodeRabbit

New Features

Added Kinde authentication support, enhancing user login and registration processes.
Updated Navbar to conditionally display authentication options based on user status.

Documentation

Introduced new environment variables in the example configuration file for Kinde integration.

Chores

Updated .gitignore to exclude sensitive environment files from version control.
Added dependency @kinde-oss/kinde-auth-react to the project.

@RamakrushnaBiswal please review

@17arindam resolve coderabit suggestions

@RamakrushnaBiswal please review my pr

add only login

@RamakrushnaBiswal so should i create register somewhere else because otherwise it is showing


Middleware Extensibility: Support "Multi-tenancy" In AuthenticationOptions (and more)

Copied from https://katanaproject.codeplex.com/discussions/561673 as I was advised this project is moving from CodePlex to GitHub.
Hello Katana team,
Problem
I've been unable to cleanly extend the "built-in" authentication middleware because of its dependency on a concrete instance of AuthenticationOptions (and subclasses per implementation).
Presently the auth middleware is initialized at start-up and any configuration (API KEY, API SECRET, etc) cannot be changed on a per-request or per-tenant basis.  I've seen some suggestion of multiple middleware instances per tenant as a solution but this is not very efficient or (feasible) when you have N tenants.
In my case each of my customers has their own OAUTH api credentials for each service and I need to be able to set them per-request based on the tenant scope.  My problem, although seemingly common, is not the only reason to consider my proposed change.
The current dependency on a concrete implementation of [Middleware]AuthenticationOptions and AuthenticationOptions limits extensibility to the point that many are forced to "roll their own" copies of existing middleware with minor changes for their individual needs.
Proposed Design Change
I propose a design change, of which I'd like to contribute code, to address this limitation and allow for an optional alternative way to load middleware AuthenticationOptions.  To achieve this we would depend on an interface IAuthenticationOptions and I[MiddlewareName]AuthenticationOptions for each specific middleware implementation.
These interfaces would have default implementations for each middleware.  For example, for Facebook there would be a DefaultFacebookAuthenticationOptions which would implement the interface and use the current (static) approach to loading AppId and AppSecret.
A second scenario-specific implementation (which would be left to someone else to implement) could be MultitenantFacebookAuthenticationOptions which also implements the interface and identifies the tenant based on the request and then uses a datastore to load the AppId and AppSecret.
It would probably make most sense to have a FacebookAuthenticationOptionsBase abstract class which contains most of the needed functionality.  We would also need to modify the existing FacebookAuthenticationOptions class to support the new interface but without breaking backward compatibility.
I have already proven this concept in my own project.  In order to implement this we would need to change several authentication middleware classes which have dependencies on AuthenticationOptions as well as existing middleware which has dependencies on concretes of their respective implementations (e.g. FacebookAuthenticationOptions).  Likewise, Extension Methods related to adding middleware to the pipeline would be modified to depend on the aforementioned interfaces instead of concretes.
Backwards Compatibility
This design change would have no impact on existing implementations.  Developers would still be able to use app.UseFacebookAuthentication (et al) as well as the existing FacebookAuthenticationOptions class.
A new option for developers will be to provide/inject an IAuthenticationOptions when adding the middleware to the pipeline.  Typically that would be done as follows:
// Old approach still supported
app.UseFacebookAuthentication(appId, appSecret);

// A new approach to inject an IFacebookAuthenticationOptions implementation
app.UseFacebookAuthentication(new MultitenantFacebookAuthenticationOptions());

Both approaches would be supported under the hood as such:
public static class FacebookAuthenticationExtensions
{
    public static IAppBuilder UseFacebookAuthentication(this IAppBuilder app, IFacebookAuthenticationOptions options)
    {
        if(app == null) throw new ArgumentNullException("app");
        if(options == null) throw new ArgumentNullException("options");

        app.Use(typeof (FacebookAuthenticationMiddleware), app, options);

        return app;
    }

    public static IAppBuilder UseFacebookAuthentication(this IAppBuilder app, string appId, string appSecret)
    {
        return UseFacebookAuthentication(
            app,
            new DefaultFacebookAuthenticationOptions
            {
                AppId = appId,
                AppSecret = appSecret,
            });
    }
}

Existing custom middleware, such as those provided by the owin-middleware/OwinOAuthProviders project should continue to work but will need to update their Microsoft.Owin.Security dependencies if they want to stay current.  I believe this is the only burden added by my proposed design.
Finally...
I'm eager to see this implemented as I believe it is a design improvement which makes authentication middleware much more extensible without making it more complex.  I wanted to get buy-in before I started the code changes required to implement the change.
So, Katana team, what do you think? :)

This is definitely a show stopper for many of us with many customers requiring us to support multiple websites using shared code bases.
I have already create a similar issue (before seeing this one) at aspnet/Home#743

Moved from https://github.com/aspnet-contrib/AspNet.Security.OpenIdConnect.Server/issues/107:

Multi-tenancy has always been a thorny topic, even back to the Katana era :smile:
Sadly, it's still not implemented in ASP.NET 5: https://github.com/aspnet/Security/issues/35
In theory, you should be able to create a custom IOptions<OpenIdConnectServerOptions> and return a different instance for each request, depending on the hostname, the path, a query string parameter, a header... :
public void ConfigureServices(IServiceCollection services) {
    services.AddScoped<IOptions<OpenIdConnectServerOptions>, MyOptions>();
}

public class MyOptions : IOptions<OpenIdConnectServerOptions> {
    private readonly IHttpContextAccessor context;

    public MyOptions(IHttpContextAccessor context) {
        this.context = context;
    }

    public OpenIdConnectServerOptions Options => GetNamedOptions(name: null);

    public OpenIdConnectServerOptions GetNamedOptions(string name) {
        // Extract the tenant identifier and use it to return different options.
        // You could extract if from a header, from the query string,
        // from the path and of course, from the hostname.
        var tenant = context.HttpContext.Request.Headers["Tenant-ID"];

        return new OpenIdConnectServerOptions {
            Issuer = new Uri("http://www.google.com/")
        };
    }
}

Sadly, this won't work, as AuthenticationMiddleware calls IOptions<>.Options directly in its constructor (where you can't access the current request, since there's simply none :smile:) and stores it as a singleton: https://github.com/aspnet/Security/blob/dev/src/Microsoft.AspNet.Authentication/AuthenticationMiddleware.cs#L26-L34
To support multi-tenancy, AuthenticationMiddleware's inheritors would also have to update their constructor, to avoid accessing the options instance too early. I guess we'd need a ValidateOptions directly in AuthenticationMiddleware to delay options validation.

/cc @blowdart @brentschmaltz @HaoK @Tratcher

It seems that this issue has been marked as an "enhancement". I cannot agree with that. I would say that for the new ASP.NET to rock, it is a "requirement" and a "show stopper". The issue is open for almost a year now.
I am an architect/developer with more or less 16 years of web development background and have been awarded as Microsoft MVP for ASP.NET for 4 times. With that experience in my past, I can clearly state that even though not every client (and project as a result) needs it, there are many enterprise clients that like to manage multiple websites from one instance.
Look at the CMS frameworks for instance. One of the best selling features of enterprise level CMS frameworks is that one can manage multiple websites on a single installation. Name it Umbraco, WordPress, EPiServer, Drupal, Django, Sitecore whatever. The market needs that technical ability.
Now that we have to go to clients and say "well, we could do that with the previous stack of .NET but not anymore with the new .NET release because the AuthenticationMiddleware does not support it". That does not make sense to me.
I could name you (which I would not of course as per the NDAs I signed) clients that host more than 70 websites on a single CMS installation. When I say single, I don't mean to say on a single server. If it is a webfarm of 5 servers, then they of course have 5 installations, instead of 70x5=350 installations.
If someone tells me how to explain a client that we have to charge them for maintenance of 350 installations instead of 5, I could be happily developing with a framework that does not support multi-tenant applications. Even if they are fine to pay for that, let's say that we need to make an update to the application, how would you update 350 installations in a timely manner? It would take days, if not weeks.
I don't really know what is the reason behind security modules not supporting multi tenant applications but I would really like to hear whatever it might be from the team that develops it.

It's an enhancement simply because it's never been like this. It's a new feature, an enhancement to the existing feature set.

I did not literally mean "it is not an enhancement" and imho, it was not the most important thing in my previous comment.
Thank you for your explanation but I am rather looking for feedback regarding at which milestone you are planning to release the support for multi tenant applications if you plan to support it at all and if not, how do you propose us to solve the maintenance issues I shared previously.

There is currently no milestone attached to it.

I can clearly see that on the right side of the original post where it says "Milestone: Backlog".
Does not really answer any of my concerns I shared or questions I asked above.

@blowdart's secret name is "Captain Obvious" :smile:

Admittedly I haven't tried anything code-wise, but why can't you build a concrete implementation that doesn't STORE the credentials it is given, rather it proxies the properties as context-specific values that get computed using the in-flight context?  I did something like this for view models in a multi-tenant app a while back.

I would like to add my voice and my vote in support of this issue. I need to be able to implement multi-tenancy configurable as either per hostname or based on the first folder segment of the url and I need to be able to configure different authentication settings for each tenant. If it is going to be impossible or very very difficult to do that with asp.net 5 that will be a big problem and certainly indicative of a design flaw in the framework IMHO.

I have to back-pedal my previous comment. While it would be nice if the middleware supported multi-tenancy by default, I have to admit I did not find it that difficult implementing my own multi tenant cookie middleware especially since the single tenant cookie middleware is open source. It was wrong of me to say it is indicative of a flaw in the framework, I really like the framework.

I'm currently working on creating a multi-tenant app in DNX as well. And apparently, (according to @PinpointTownes :P), this is one of the problems WRT that. So +1 from me.

I'd like to this prioritised too. It was confirmed some time ago that multi-tenancy wouldn't be supported out of the box in v1. However, it should at least be considered so that we have the extensibility points necessary to implement it ourselves.
I recently released SaasKit for ASP.NET Core that provides lightweight middleware to perform tenant identification and makes the current tenant available via DI.
When it came to ASP.NET Identity (cookie based auth) it was fairly trivial to support multi-tenancy since we could just provide the underlying DbContext (see http://benfoster.io/blog/aspnet-core-multi-tenancy-data-isolation-with-entity-framework). I know @joeaudette does have some concerns about the cookie middleware but since each of our tenants have separate domains, we don't need to set the cookie path per tenant so it was less of an issue for us.
However, when it comes to the external auth middleware this is a bit of a showstopper for us. Whilst it is possible to register multiple instances of the same auth provider by specifying a unique AuthenticationType, this does not work if you're provisioning tenants on-demand or if you have n number of tenants.
The main issue as a others have pointed out is that the AuthenticationOptions instance is created once per application rather than being resolved per request. This makes it virtually impossible to use for multi-tenant applications without essentially re-writing the underlying middleware and handlers.
The most logical solution seems to be to leverage DI and have the options resolved per request. Given that ASP.NET Core supports parameter injection into a middleware's Invoke method and as far as I can tell (please correct me if I'm wrong) the options are not actually used until the handler is created (on each request) would this not be a viable option?
It would then be easy to customise the options per tenant:
        services.AddScoped<GoogleOptions>(x => {
          var tenant = x.GetRequiredService<AppTenant>();

          return new GoogleOptions {
            ClientId = tenant.GoogleCliendId,
            ClientSecret = tenant.GoogleClientSecret
          };
        });

It would be great to get some feedback on this.

We realise this is wanted but we're at the point now where we are not doing any more feature work, hence this being in the backlog.
We will look at this once we get v1 out the door.

We have a pretty good workaround in the latest release of SaasKit.
This allows the auth middleware to be configured per tenant by creating forked middleware pipelines per tenant:
    app.UsePerTenant<AppTenant>((ctx, builder) =>
    {
        builder.UseCookieAuthentication(options =>
        {
            options.AuthenticationScheme = "Cookies";
            options.LoginPath = new PathString("/account/login");
            options.AccessDeniedPath = new PathString("/account/forbidden");
            options.AutomaticAuthenticate = true;
            options.AutomaticChallenge = true;

            options.CookieName = $"{ctx.Tenant.Id}.AspNet.Cookies";
        });

        builder.UseGoogleAuthentication(options =>
        {
            options.AuthenticationScheme = "Google";
            options.SignInScheme = "Cookies";

            options.ClientId = Configuration[$"{ctx.Tenant.Id}:GoogleClientId"];
            options.ClientSecret = Configuration[$"{ctx.Tenant.Id}:GoogleClientSecret"];
        });
    });

@blowdart any update now v1 is out the door 🤔 I also have a need of this type of feature

This is in progress for 2.0: https://github.com/aspnet/Security/pull/1170

Tracked via https://github.com/aspnet/Security/issues/1179


Fix OpenMM post-link.{sh,bat} files

You forgot to back-port those fixes into the openmm-dev recipe.


Feature preserve_order is not additive

Cargo unions all features that all rdeps in a project of a crate want, and compiles the crate with that. This requires features to be additive. Since this feature changes the types used in the public API of the crate, it is not.

We can avoid affecting types in the public API by replacing serde_json::value::Map (which is currently either BTreeMap or LinkedHashMap) with an opaque newtype wrapper around BTreeMap or LinkedHashMap. @oli-obk do you see a better approach?


[BUG] Broken page layout

I'm aware of a bug where the Friend Activity panel is shifted to the bottom right corner of the Spotify page layout.
This is a result of Spotify changing their internal styling - which broke the styling of the Friend Activity panel.
I pushed a hotfix for this bug here and submitted it for review to the Chrome Web Store.
The Chrome Web Store review process takes some time - while waiting for the hotfix to publish, you can follow the steps below to install the new version manually:

Download spotify-friend-activity-v1.0.9.zip from this repository's release log here. (The first link under the Assets section)
Extract the downloaded zip file to a folder. (Mac: Double click the zip file. Windows: Right click > Extract All)
Open Chrome and navigate to chrome://extensions/
If you already installed Spotify Friend Activity from the Chrome Web Store, you must temporarily disable it by clicking on the toggle at the bottom right of the Spotify Friend Activity card in chrome://extensions/
Enable Developer Mode. (Top right toggle)
Click on Load unpacked. (Top left button)
Select the extracted folder from Step 2.
Navigate to https://open.spotify.com/ to verify that the extension is working. (You may have to refresh the page)

The hotfix is now published to the Chrome Web Store. Here's how to get the update:

Open Chrome and navigate to chrome://extensions/
Enable Developer Mode. (Top right toggle)
Click on Update. (Top left button)
Navigate to https://open.spotify.com/ to verify that the extension is working. (You may have to refresh the page)


should vtype vill  bit still read 1 for invalid values in minimum configuration.

In section 3.2. Vector type register:
"The illegal value represented by vill can be encoded
using the illegal 64-bit combination in vsew[1:0] without requiring
an additional storage bit."
Must bit XLEN-1 still read 1 for such? i.e. vill = vsew[1]& vsew[0], so only two storage bits but msb be derived from them.
And because the statement in 3.2.3. Vector Type Illegal vill
"When the vill bit is set, the other XLEN-1 bits in vtype shall be zero."
Would require vsew[1] and vsew[0} be masked to zeros on CSR read when both are set internally.
Is this zero requirement that all other bits be zero too restrictive for the minimal case?

Re: "Must bit XLEN-1 still read 1 for such?" -- I think the definition of vill is sufficient to imply the answer is "yes."
Re: "Is this zero requirement [...] too restrictive?" -- I would be OK with eliminating this requirement, and instead saying that when (and only when) vill is set, the other fields are permitted to assume invalid values.  Under this alternative, it would be OK for vsew to read as 11 when vill is high in the simple 32-bit case.

We've tried to avoid implementation-defined behavior in unprivileged spec where possible.  While this one seems fairly innocuous, I can see verification people upset at having to deal with model differences here.

Without taking traps on vsetvl, "Zero all" as well as "let it be undefined" rules, would make it impossible to emulate future or unsupported vtype configurations.

The current spec uses illegal bit primarily to avoid having to trap CSR writes dependent on data value written, and to allow the vtype register to only support legal values for the current implementation.  It has the other benefit of allowing unprivileged software to determine what vector types are supported but I think this is a minor benefit.
An alternative approach would be trap any illegal write to vtype for emulation purposes (and possibly debugging), but that would then prevent use of this mechanism for user configuration discovery and would require data-dependent traps for this instruction.
A privileged architecture extension could add feature to trap on illegal CSR writes to support this emulation without requiring that all implementations support traps on CSR data values.

I think that the current trapping scheme with an exact vtype register storage is the most suitable approach for emulation. (We can also move the "exact vtype" requirement to OS profile as to let embedded targets save some silicon)

Closing this issue for now, as we seem to have agreed that future version of privileged architecture can add facility to trap on bad type values for emulation purposes, without adding new awkward data-dependent trap for all implementations.

Sorry to reply to an closed issue, but I wonder if there's been justification for zeroing the entire vtype when vill=1 ?
Leaving the other bits in vtype unchanged seems easier to implement and more useful -- upon requesting an invalid vtype, the application can more easily restore the previous legal version by reading vtype, clearing the MSB, and running vsetvl. If vtype is cleared upon an illegal setting, then the application must always save the value vtype before any vsetvl instruction.
Actually clearing vill could be made even easier. The first way is to force vsetvl to always clear vill upon writing (ie, ignore setting of vill in rs2). This would eliminate the "clearing the MSB" step. The second way would be to allow vtype CSR to be writable, and use the CSRRC instruction to immediately clear vill atomically with reading vtype. There are other hazards of making vtype writable, which I will just leave open for consideration.


Node stopped syncing at Block 105234895 (Target client not connected)

I have upgraded my node to Bedrock - But still node is not syncing further.
.env

Fault Detector Bedrock Logs:

Node Healthcheck Bedrock Logs

Op-geth logs

Bedrock init logs

Hi, did you resolve it?

Hi, did you resolve it?

No. Unable to find the fix. Can you please help?

I can't help you. I have the same problem :-)

Hi, my fork has fixed this issue and uses an official snapshot. It's related to snapshots can't be downloaded properly.
Check out https://github.com/Chomtana/simple-optimism-node/tree/main/scripts
Pull request https://github.com/smartcontracts/simple-optimism-node/pull/97

#97 should have fixed this


[Bug] OOM for concurrent long requests

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.
[X] 4. If the issue you raised is not a bug but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.

Describe the bug
I am trying to benchmark inference of llama3-8b with long requests, I send 20 concurrent requests each with length of 1k tokens and I set the stream to True and max_tokens to 1024.
This is how I start the server:
python -m sglang.launch_server --model-path NousResearch/Meta-Llama-3-8B-Instruct  --host <IP_ADDRESS>  --port 8000 --context-length 4096 --dtype bfloat16  --chat-template llama-3
I add llama-3 template in conversaitons.py as they are present in conversions file of FASTCHAT.

Note: when I send this to VLLM entrypoint, it works without OOM error!

Error:
INFO:   - "POST /v1/chat/completions HTTP/1.1" 200 OK
[gpu=0] Prefill batch. #new-seq: 17, #new-token: 13938, #cached-token: 3260, cache hit rate: 17.86%, #running-req: 1, #queue-req: 2
Exception in ModelTpServer:
Traceback (most recent call last):
  File "/home/ubuntu/sglang/python/sglang/srt/managers/tp_worker.py", line 219, in exposed_step
    self.forward_step()
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/sglang/python/sglang/srt/managers/tp_worker.py", line 235, in forward_step
    self.forward_prefill_batch(new_batch)
  File "/home/ubuntu/sglang/python/sglang/srt/managers/tp_worker.py", line 545, in forward_prefill_batch
    output = self.model_runner.forward(batch, ForwardMode.EXTEND)
  File "/home/ubuntu/sglang/python/sglang/srt/model_executor/model_runner.py", line 388, in forward
    return self.forward_extend(batch)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/sglang/python/sglang/srt/model_executor/model_runner.py", line 356, in forward_extend
    return self.model.forward(
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/sglang/python/sglang/srt/models/llama2.py", line 314, in forward
    hidden_states = self.model(input_ids, positions, input_metadata, input_embeds)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/ubuntu/sglang/python/sglang/srt/models/llama2.py", line 281, in forward
    hidden_states, residual = layer(
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/ubuntu/sglang/python/sglang/srt/models/llama2.py", line 239, in forward
    hidden_states = self.mlp(hidden_states)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/ubuntu/sglang/python/sglang/srt/models/llama2.py", line 79, in forward
    gate_up, _ = self.gate_up_proj(x)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/vllm/model_executor/layers/linear.py", line 330, in forward
    output_parallel = self.quant_method.apply(self, input_, bias)
  File "/home/ubuntu/miniconda3/envs/sglang-env/lib/python3.9/site-packages/vllm/model_executor/layers/linear.py", line 122, in apply
    return F.linear(x, layer.weight, bias)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 764.00 MiB. GPU

Reproduction
Same as describe the bug section
Environment
$ python -m sglang.check_env

Python: 3.9.19 (main, May  6 2024, 19:43:03) [GCC 11.2.0]
CUDA available: True
GPU 0: NVIDIA A10G
CUDA_HOME: None
PyTorch: 2.3.1+cu121
sglang: 0.2.9.post1
flashinfer: 0.1.3+cu121torch2.3
requests: 2.32.3
tqdm: 4.66.5
numpy: 1.26.4
aiohttp: 3.10.0
fastapi: 0.112.0
hf_transfer: 0.1.8
huggingface_hub: 0.24.5
interegular: 0.3.3
packaging: 24.1
PIL: 10.4.0
psutil: 6.0.0
pydantic: 2.8.2
uvicorn: 0.30.5
uvloop: 0.19.0
zmq: 26.1.0
vllm: 0.5.3.post1
multipart: 0.0.9
openai: 1.38.0
anthropic: 0.32.0
NVIDIA Topology:
	GPU0	CPU Affinity	NUMA Affinity	GPU NUMA ID
GPU0	 X 	0-15	0		N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

ulimit soft: 1024

Thanks for reporting the errors.

You do not need to specify the chat template. The model you used already has the chat template. https://huggingface.co/NousResearch/Meta-Llama-3-8B-Instruct/blob/main/tokenizer_config.json#L2053
Could you try to add --max-prefill 8192 when you launch the server? i.e.

python -m sglang.launch_server --model-path NousResearch/Meta-Llama-3-8B-Instruct  --host <IP_ADDRESS>  --port 8000 --context-length 4096 --dtype bfloat16  --max-prefill 8192

If it still OOMs, try to add --max-prefill 4096 or --mem-fraction-static 0.83

@merrymercy thanks for the prompt response. it works with --max-prefill 4096.   btw is the backend VLLM? what are the available backends?
for tokenizer, how it should be if i didn't pass the chat-template, the LLM is loaded without chat template as following:
chat_template=None,
server_args=ServerArgs(model_path='NousResearch/Meta-Llama-3-8B-Instruct', tokenizer_path='NousResearch/Meta-Llama-3-8B-Instruct', tokenizer_mode='auto', load_format='auto', dtype='bfloat16', trust_remote_code=False, context_length=4096, quantization=None, served_model_name='NousResearch/Meta-Llama-3-8B-Instruct', chat_template=None, host='<IP_ADDRESS>', port=8000, additional_ports=[8001, 8002, 8003, 8004], mem_fraction_static=0.88, max_prefill_tokens=4096, max_running_requests=None, max_num_reqs=None, max_total_tokens=None, schedule_policy='lpm', schedule_conservativeness=1.0, tp_size=1, stream_interval=1, random_seed=746231706, log_level='info', log_level_http=None, log_requests=False, show_time_cost=False, api_key='', file_storage_pth='SGlang_storage', dp_size=1, load_balance_method='round_robin', chunked_prefill_size=None, disable_flashinfer=False, disable_flashinfer_sampling=False, disable_radix_cache=False, disable_regex_jump_forward=False, disable_cuda_graph=False, disable_disk_cache=False, enable_torch_compile=False, enable_p2p_check=False, attention_reduce_in_fp32=False, efficient_weight_load=False, nccl_init_addr=None, nnodes=1, node_rank=None)

btw is there any factors that influence the concurrent requests should I check?

We use the chat template in the huggingface tokenzer by default if it is available https://github.com/sgl-project/sglang/blob/8207637029082563cab74951fe8d5f86b574b85e/python/sglang/srt/openai_api/adapter.py#L701-L704

SGLang is a standalone server, it uses attention CUDA kernels from flashinfer, linear layers from vLLM, and triton operators from torch.compile. For your 8B model, you can try to add --enable-torch-compile, which will be faster. It is not very precise to say that "the backend is vLLM", as SGLang integrates many high-performance components.

See this guide on tuning the parameters for high throughput https://github.com/sgl-project/sglang/blob/main/docs/en/hyperparameter_tuning.md

There is definitely a much bigger regression with recent versions around OOM. Previous workloads and configurations that had no issues running large concurrency for long periods of time are now OOM'ing even when running with a fraction of static mem pool and lowered concurrency.

There is definitely a much bigger regression with recent versions around OOM. Previous workloads and configurations that had no issues running large concurrency for long periods of time are now OOM'ing even when running with a fraction of static mem pool and lowered concurrency.

Hello, is there any solution now

This commit seems to have improved things (too early to say if resolved, but early testing is working better): https://github.com/sgl-project/sglang/commit/df191254abc002b3284560d9c4b94214a4656265

this should be fixed with the latest version.
see also https://github.com/sgl-project/sglang/blob/39bb49d156f2319d2aec67c458c2db980bb0f4c3/README.md?plain=1#L221-L224
https://github.com/sgl-project/sglang/blob/main/docs/en/hyperparameter_tuning.md?plain=1#L28-L32


feat: add blackfire support

Allows to use https://blackfire.io together with PHP. Didn't tested Go / Python profiling.
Example Bllackfire together with PHP:
{ pkgs, config, ... }:

let
  phpPackage = pkgs.php.buildEnv {
   # Will be fixed with https://github.com/NixOS/nixpkgs/pull/202934
+   extensions = { all, enabled }: with all; enabled ++ [ (blackfire// { extensionName = "blackfire"; }) ];
    extraConfig = ''
      memory_limit = 256M
+     blackfire.agent_socket = "${config.blackfire.socket}";
    '';
  };
in
{
+ blackfire.enable = true;
+ blackfire.server-id = "xxxxxxxx";
+ blackfire.server-token = "xxxxxxxx";
+ blackfire.client-id = "xxxxxxxx";
+ blackfire.client-token = "xxxxxxxx";

  languages.php.enable = true;
  languages.php.package = phpPackage;
  languages.php.fpm.pools.web = {
    settings = {
      "pm" = "dynamic";
      "pm.max_children" = 5;
      "pm.start_servers" = 2;
      "pm.min_spare_servers" = 1;
      "pm.max_spare_servers" = 5;
    };
  };

  caddy.enable = true;
  caddy.virtualHosts."http://localhost:8000" = {
    extraConfig = ''
      root * public
      php_fastcgi unix/${config.languages.php.fpm.pools.web.socket}
      file_server
    '';
  };
}

Could you add the example into the enable section?

@domenkozar fixed. I added the generated markdown file too to the PR. So you can review the result :)


Merging master branch to live branch to publish docs

Docs Build status updates of commit 7784dae:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/API-references.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


pytlsd submodule problem

You have some problem with pytlsd, it's not working.

Hi, can you be more specific about your issue, e.g. by describing and pasting your error message?

The link pytlsd @ 3b36289 in the DeepLSD/third_party folder does not open, the other links in the folder do. Also, I can't install this library with git clone --recurse-submodules.

Thanks for reporting it. Can you try again? It should be fixed now and you should be able to link correctly to pytlsd.

I am closing this now as it should have been resolved now. If this is not the case, feel free to reopen it.


TypeError when using resnet18 for classification

I generated a classification task following the instruction on the page. Everything was fine at this stage.
When I tried to run the experiment with m-eurosat dataset, I saw the following error message:

The pytorch version of my computer is torch 2.3.0 with CUDA 11.8, python version is 3.10.14.
I tried convnext model on the same dataset, this error didn't show up.
It would be appreciated that you can fix this problem. Thank you!
By the way, I find another error when using resnet50 model. It seems that pytorch no longer accepting monument which exists in the resnet50.yaml as a parameter for Adam optimizer. Keeping the parameter will cause an error when running the experiment. I modified optimizer part in resnet50.yaml (on my computer) according to that in resnet18.yaml, and this error got fixed.
Could you please update resnet50.yaml in your project to fix this error? Thank you!

Hi @Purin100 , thanks for the issue. The error you are seeing occured, because the torchgeo dependency changed the return statement of the function we used. I just pushed to main, and should be fixed now. I can run classification experiments.

Hello @nilsleh, I tested the code on my computer and it worked well. Thank you!


Slack invite link is expired

Hi!
Could you please renew slack invitation link?

PS: Sorry,  i've accidentally added wrong label for this issue

Oops! This link should never expire. Anyway, It's updated on the website. Thanks for reporting!

It's now updated on the repo too. Thanks!

Thank you!


Request for vector return from normal_cdf given vector arguments

I wish to execute the normal CDF on vector inputs to obtain a vector of cumulative probabilities. However, I am getting a dimension mismatch error. The error says the return on the right-hand side is of type “real”. The Stan documentation says that the cdf function accepts “reals” arguments and has “reals” returns. I thought this psuedotype “reals” included vectors yet it appears to be returning a scalar.
Example code:
vector[N] mu;
vector[N] sigma;
vector[N] x;
vector[N] phi;

phi = normal_cdf(x, mu, sigma);

Example error:
SYNTAX ERROR, MESSAGE(S) FROM PARSER:
Dimension mismatch in assignment; variable name = phi, type = vector; right-hand side type = real.
Illegal statement beginning with non-void expression parsed as
  phi

Yes all the lpdf and cdf functions return back a scalar. A real in the stan language is a scalar
It would be nice to have a vlpdf style function that returned a vector but we haven't had time to implement it

This would be nice to have for all the copula stuff as well. As the copulas use the marginal cdfs evaluated at each data point.

But I think there would not really be a performance gain..so these functions can live in Stan language is what I would think.

But I think there would not really be a performance gain..so these functions can live in Stan language is what I would think.

+1 ^
also, the row/col-wise framework proposed by @andrjohns would solve this issue.

This feature would really help when we need access to individual observation-level log likelihoods but still want to take advantage of the vectorization in the _lpdf functions. Example: https://arxiv.org/abs/2209.09190

It would also help in case of interval censoring. See this comment: https://github.com/paul-buerkner/brms/issues/1657#issuecomment-2367156113

I have a PR that examples how we can do this. The main thing is it's a huge rewrite of our distributions that would be a lot of elbow grease effort. It would be nice to have a Stan hackathon to go through the distributions with people and do this split up over several people. We also need a test suite change so that all these are tested for vector returns
https://github.com/stan-dev/math/pull/2751


Added Pancake Sort in Python

Closes Issue Number #1299 Partially

This issue wasn't assigned to you!


A conflict with VFE: Classical

Unlike the AR core, AR:SPQR causes cross-reference errors on startup when loaded alongside VFE: Classical. For some reason the amount of errors is different when AR: Meditarranean Factions is loaded with AR: SPQR, but I can't draw any proper conclusion from that and I didn't check the other addons.

I am the author of the VFEC Mod changes.
What do you mean by cross-reference errors? The Patches in this Mod extensively alter and integrate the VFE Classical Mod. Both the SPQR Ancient Rim Mod and the Mediterranean Factions Ancient Rim Mod affect different parts of that Mod differently.
What specific errors are you seeing?


Use PROTECTED_AUTH to activate the PIN Pad

Description of the changes
Use PROTECTED_AUTH instead of None as the user pin to activate the PIN Pad with PKCS#11 Devices. Resolves #133
Checklist
Please go over this checklist to increase the chances of your PR being worked on in a timely manner. Deviations are allowed with proper justification (see previous section).

[x] I have read the project's CoC and contribution guidelines.
[x] I understand and agree to the terms in the Developer Certificate of Origin as applied to this contribution.
[x] All new code in this PR has full test coverage.

For bug fixes (delete if not applicable)

[ ] My changes do not affect any public API or CLI semantics.
[ ] My PR adds regression tests (i.e. tests that fail if the bug fix is not applied).
[ ] All new code in this PR has full test coverage.

Thanks, @pquL1!
Hmm, unfortunately this breaks in some cases where None would otherwise work. I'm getting this error with one of my devices after applying your change:
pkcs11.exceptions.ArgumentsBad: Protected authentication is not supported by loaded module

For context: this module handles its own PIN dialog, but doesn't expose that fact in any discernible way in the PKCS#11 API. Maybe it should, though. I'll investigate.
On the other hand, even if the module I'm using is somehow bugged and/or weird, there probably exist PKCS#11 tokens that don't require any authentication at all (e.g. signing hardware intended for unattended operation). Passing PROTECTED_AUTH to those also doesn't seem right.
Anyway, the fact remains that there exist implementations that work with PROTECTED_AUTH but not with None and vice versa (unfortunately). I'll try to come up with some tweaks to your PR to allow both variants to be used.

Update: I've been working on some tooling to automate PKCS#11 testing lately, in particular to be able to run it in CI (see MatthiasValvekens/certomancer@d0e8996 and MatthiasValvekens/certomancer@1fea804). I'll get back to this PR after I get that set up!

Codecov Report

Merging #134 (f689fb6) into master (b831bf1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #134   +/-   ##
=======================================
  Coverage   98.69%   98.69%
=======================================
  Files          75       75
  Lines       11189    11209   +20
=======================================
+ Hits        11043    11063   +20
  Misses        146      146

Flag
Coverage Δ

unittests
98.69% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pyhanko/config.py
100.00% <100.00%> (ø)

pyhanko/sign/pkcs11.py
100.00% <100.00%> (ø)

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

So, I've turned the prompt-pin config setting from a boolean into a tri-state enum with values prompt, defer and skip. prompt and skip correspond to the old true and false, respectively (and passing in boolean values will get you those results, for compatibility's sake). The new defer option should trigger the behaviour you want.
Can you check if that works as expected on your end?
FWIW, I added a test against SoftHSM with PROTECTED_AUTH, but since SoftHSM doesn't support the PROTECTED_AUTHENTICATION_PATH flag at all, the test is of the "I expect this operation to fail in this very specific way" type, so it's not quite the same thing.

Hi @MatthiasValvekens, unfortunately it doesn't work. I haven't been able to figure out what's going on, but regardless of the setting for prompt_pin, I now get pkcs11.exceptions.PKCS11Error: No token with label 123456789 found (even if I let it promt the pin and enter the correct one). The tri-state enum seems to work (I think it does select PROTECTED_AUTH correctly), but something else seems to have gone awry with the PCKS11 setup so it never really gets that far. I first thought my device had an issue, but it works fine with the regular pyhanko version.

Hmm, that's curious. Could you try applying your original change (the None -> PROTECTED_AUTH one) on the tip of the current master branch and run that? If that also doesn't work, then I probably inadvertently changed something while refactoring things to make them more testable...
Are you specifying both a slot index and a token label, or only a label?

Seems like it's the same problem with the current master branch. I'm only supplying the token label.

That's really weird. So, you mentioned letting the device prompt for PIN and then getting the error? Because the routine that raises that exception runs before the token.open(...) method is ever called... https://github.com/MatthiasValvekens/pyHanko/blob/b831bf1a6d22111f634fa3ad1cdc1d6516905d04/pyhanko/sign/pkcs11.py#L52-L87
I'll try debugging this further tomorrow, but since it's kind of hard to repro without having your setup in front of me, it would be extremely helpful if you could help me figure out which of the commits between @551c8bc and the current master branch ended up breaking things... :/.  It's probably something silly.
Anyway, big thanks for the input so far!

Hmmm, I'm not immediately seeing what could cause the issue you mentioned. Would you mind sharing the config and CLI invocation you're using?

Hi @MatthiasValvekens, 60a9dbc introduced the issue. Starting with that commit, my device doesn't work at all anymore. Even if I set prompt_pin to true, it doesn't find the token. 551c8bc0 still works.
My PKCS11 config is
        module-path: /usr/local/lib/libpkcs11tcos3SigG_x86_x64-1.8.0.dylib
        token-label: 123456789
        cert-label: ABC
        key-label: XYZ
        raw-mechanism: True
        prompt_pin: True <-- using this for testing

My CLI invocation is long, but with regard to PKCS11, it only specifies the p11-setup.
Thanks for your efforts to solve this!

Hi Matthias, I think I found the issue. I didn't put quotes around my token label in the config and after the change it seems to have been interpreted as an int so the comparison failed.

Hi @pquL1, thanks a lot!

Hi Matthias, I think I found the issue. I didn't put quotes around my token label in the config and after the change it seems to have been interpreted as an int so the comparison failed.

Huh, weird. I didn't really touch that part of the config deserialisation logic, but YAML sometimes does bizarre things... Glad you got it sorted, though.

One more thing: I get a Segmentation fault: 11 with almost every signature I make. The signature comes out fine, though. I haven't been able to figure out what causes it. However, sometimes I don't get the error and I think that this often times goes along with the PIN PAD activating faster than usual (quite difficult to know for sure because I can't intentionally reproduce it yet). I would guess that this has nothing to do with anything in this issue but I still thought I'd mention it, maybe it rings a bell...

Interesting... I have some stress tests in the test suite that also segfault more often than I'd like them to... I wasn't sure if it was a bug in python-pkcs11, in SoftHSM or in the way pyHanko calls them, though. There's a chance that those are related. I never run into issues triggering signatures one by one, though, so it might be something else.
Let me know if you manage to figure out anything more!


Close app from notification

Hi, why don't you put a little "x" in the corner of the notification to completely close the YouTube app (and the notification) running in background? (Maybe it was present some times ago?)
Thanks!

Yes, the "X" button in notification panel for closing youtube app is missing. Without this button closing youtube app is much more difficult. It would be great if this close button is back.
Thanks for great module!

The module does not change notification design and its contents... if you want to change something in that - write feedback to Google :)

You mean that the notification is in the YouTube app? I thought it was designed by you!

Nope :P Module makes it visible for everyone, not only for those with YT Red subscription :)

Oh now I understand..thanks


about efficientnet-b7  train on cifar-10,but i set up batch size is 8 ,gpu(1080ti 11g) out of memory

i guess if the efficientnet-b7 model size is large?

Yes, the efficientnet-b7 is too large to load on one 1080ti GPU.

I believe actual blocker for training/tuning is that with current implementation we cannot use torch.utils.checkpoint.checkpoint_sequential - PyTorch way to reduce memory during training. Some minor refactor should do the trick. Possible guide: https://github.com/prigoyal/pytorch_memonger


refactor: simplify and inline is_disabled()

Thanks I learn a lot. I didn't know about vim.tbl_contains and vim.bo.ft. It's just so beautiful...


[jdbc] Make raised AuthenticationRequired exception syntax Py3 compatible

What changes were proposed in this pull request?

Syntax changes for keeping both python versions 2 and 3 compatible

How was this patch tested?
Locally build
Please review Hue Contributing Guide before opening a pull request.

Hi @Harshg999, Implemented the changes. But circleCI build is failing. Is it something that requires some kind of access or privileges to have for running?

Ahh you force pushed and might be your local branch wasn't rebased with latest master changes. Let me rebase the branch and then hopefully it should be good.


Support passing string to splitChunks.{cacheGroup}.test

https://webpack.js.org/plugins/split-chunks-plugin/#splitchunkscachegroupstest
Guidance

Add options scheme. See #3378
Rewrite the binding_options part

https://github.com/web-infra-dev/rspack/blob/4b2a61d26a8063c4289e84a66d8a87e721474692/crates/rspack_binding_options/src/options/raw_split_chunks.rs#L120
Change to Opton<Either<JsString, JsRegExp>>. See https://github.com/web-infra-dev/rspack/pull/3473 for example.

Port the following code
https://github.com/webpack/webpack/blob/f921a5baca9c7d200eee8204fafaaaf36b7c1451/lib/optimize/SplitChunksPlugin.js#L526-L529

to this part
https://github.com/web-infra-dev/rspack/blob/4b2a61d26a8063c4289e84a66d8a87e721474692/crates/rspack_binding_options/src/options/raw_split_chunks.rs#L238
PS: Only support this feature in the rspack_split_chunks_plugin_ new crate.
https://github.com/web-infra-dev/rspack/blob/4b2a61d26a8063c4289e84a66d8a87e721474692/crates/rspack_binding_options/src/options/raw_split_chunks.rs#L163
Block tests

https://github.com/hyf0/webpack/blob/671ac29d462e75a10c3fdfc785a4c153e41e749e/test/configCases/split-chunks/incorrect-chunk-reuse

@hyf0 can you assign this to me?

Done


Migrate the community to discord

Description

Gitter was initially chosen as the community platform due to it being FOSS. However, I've come to realise that not many people are on gitter and having a community platform without a community defeats the purpose. DIscord is the most common platform these days, so we are moving to discord.
Expected Behavior

Move to discord.

All done!


SAA-366 chore updating 3rd party dependencies.

Bumping to latest 3rd party dependencies where available.

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

Coverage data is based on head (3543c1d) compared to base (a39192b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #183   +/-   ##
=========================================
  Coverage     70.76%   70.76%
  Complexity      799      799
=========================================
  Files           160      160
  Lines          3687     3687
  Branches        130      130
=========================================
  Hits           2609     2609
  Misses         1000     1000
  Partials         78       78

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

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