You did not set any plugins, parser, or stringifier. Right now, PostCSS does nothing.

 I had the configuration working on webpack v3 using older versions of the libraries/plugins. Upgraded to webpack v4 and latest plugins versions and now postcss can't pickup the plugin configuration.
Details
Configuring postcss-loader with following plugins:

precss
autoprefixer
lost

Error (Logs|Stacks)
You did not set any plugins, parser, or stringifier. Right now, PostCSS does nothing. Pick plugins for your case on https://www.postcss.parts/ and use them in postcss.config.js.
Reproduction (Code)
Please see:
https://gist.github.com/trigun539/2b8ebf739b64881ec0b3ea2a9e37ba92
Environment
OS: Windows
node: 8.15.0
npm: 6.4.1
postcss-loader: 3.0.0
precss: 4.0.0
autoprefixer: 9.4.10
lost: 8.1.0
mini-css-extract-plugin: 0.5.0

Because you do not have plugins for postcss-loader, it is error from postcss

Rename postcss.conf.js to postcss.config.js and it should work

@michael-ciniawsky Sorry the file actually is postcss.config.js. It is still not working.

Because you do not have plugins for postcss-loader, it is error from postcss

I have precss/autoprefixer/lost postcss plugins that should work.

https://gist.github.com/trigun539/2b8ebf739b64881ec0b3ea2a9e37ba92#gistcomment-2866264

If you have the same problem, i've found an explication/solution here :
https://github.com/postcss/postcss/issues/1247

any solution? i getting the same issue

any solution? i getting the same issue
You did not set any plugins, parser, or stringifier. Right now, PostCSS does nothing. Pick plugins for your case on https://www.postcss.parts/ and use them

I got a solution. i using a laravel mix. solution here

As an aside, I have discovered that despite a valid (webpack) postcss-loader configuration with plugins present, from what I can tell postcss-advanced-variables will cause this error to be emitted since it apparently invokes postcss internally without plugins; I'm not sure if this is a bug per se, or whether there is some way to refrain from warning when explicitly invoked like this.
https://github.com/jonathantneal/postcss-advanced-variables/blob/master/src/lib/transform-import-atrule.js#L75

Is it possible to get more of a stack trace for this error? I'm getting it now but everything seems to be working correctly otherwise. I presume it's some other dependency somewhere in the tree.

@wadtech I just dumped a couple of console.logs in postcss processor.js process function, that's how I tracked the source of mine down

I was receiving this error on console even when my CSS was processed with no problems. In my case was due to @import statements on CSS files. Adding postcss-import fixed the issue but this plug-in is redundant since @import statements are already tackled by css-loader.

I am trying to upgrade to<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>but I am getting this issue as well. Webpack seems to build as expected but prints these warnings a lot of times before printing the final output. I noticed that I only get these warning in development mode.
Here's my updated loader config:
loader: 'postcss-loader',
    options: {
        postcssOptions: {
            plugins: [
                ['autoprefixer', {
                    overrideBrowserslist: targets,
                }],
            ],
        },
        sourceMap: true,
    },
}

Any idea what could be causing this issue?

@evilebottnawi yes, I do. Is that the cause of the issue?

@bensampaio some package doesn't have postcss.config.js, you need to put it in each packages or in root or setup in loader using config

@evilebottnawi I am putting it in the loader as I showed above:
loader: 'postcss-loader',
    options: {
        postcssOptions: {
            plugins: [
                ['autoprefixer', {
                    overrideBrowserslist: targets,
                }],
            ],
        },
        sourceMap: true,
    },
}

I can't move this configuration to postcss.config.js because the value of overrideBrowserslist is dynamic. In any case I tried creating a postcss.config.js on every package in my monorepo and this issue still remains. In my webpack config I use mini-css-extract-plugin, css-loader, sass-loader, postcss-loader and optimize-css-assets-webpack-plugin. Could it be related to any of these? Do they use postcss internally without configuration?

If I remove postcss-loader from my configuration then the problem is gone. But this isn't exactly what I want xD

@bensampaio Please:

migrate from optimize-css-assets-webpack-plugin on https://github.com/webpack-contrib/css-minimizer-webpack-plugin, it is official css plugin for minimizing with cache/parallels/source maps/etc
In any case I tried creating a postcss.config.js on every package in my monorepo and this issue still remains, you need to put it in root, because you compile files from node_modules and we don't know which plugins should be used there

@evilebottnawi I did create a ´postcss.config.js´ file in the root of my monorepo. That didn't fix the issue :/ And this was just a test anyway because as I mentioned my postcss config needs to be in the loader. I have multiple webpack configs running with different browserslist configurations and as far as I know there's no way I can move that out from my webpack configs. If this loader offers options to configure postcss then I would expect that to be enough... Is there anyway I can help figure out the source of these warnings since none of the solutions so far helped?
I will migrate to ´css-minimizer-webpack-plugin´ as you mentioned, thank you!

@bensampaio Please create reproducible test repo

@bensampaio did you solve the problem? I have exactly the same error! I also use mini-css-extract-plugin, css-loader and monorepos

@evilebottnawi I am trying to reproduce the issue in this repo: https://github.com/bensampaio/postcss-monorepo-issue. However, webpack doesn't even seem to be able to resolve the postcss-loader while it is able to resolve all the other loaders. I am not sure if this is related but it is definitely weird that I only get this issue with postcss-loader.

@bensampaio Because you run run build:dev in @test/app workspace but there are no loaders for @test/app, sorry problems is not related to webpack or postcss-loader, why it is work for babel-loader, because it is hoist to node_modules/babel-loader, but no node_modules/postcss-loader

More information https://github.com/yarnpkg/yarn/issues/7572

I was getting this warning in my NX workspace . I don't know how to find the webpack in my NX workspace and post config.css , I tried creating a webpack.conf.js and post.config.js but didn't solved my problem . Can anyone guide me on this  ?

This is my project structure

@Bessonov feel free to send PR to docs

@alexander-akait I don't feel like I have enough background information to put it into proper and precise wording. For example, it isn't clear for me why it works this way. I start the process from a sub-module and expect that the configuration in this sub-module should be enough. And how it's expected to work, if there are multiple frontend modules which requires different postcss configuration.

it is due module hoist in monorepos and default cwd...

I meet the same problem when I use autoprefixer plugin.  I tried many solutions I've found by Google , none of they woked. But ironically, it's just beacuse I just wrote plugin rather plugins in postcss.config.js.

Is there any way to disable this yet? I'm using next.js and the docs clearly state you can disable postcss during development https://nextjs.org/docs/advanced-features/customizing-postcss-config#customizing-plugins however if I do this, my terminal is flooded with warnings.

In my case, changing the value of plugins property from function to array has solved the issue.
Before:
module.exports = {
  plugins: () => [
    require('postcss-preset-env'),
    require('cssnano'),
  ],
};

After:
module.exports = {
  plugins: [
    require('postcss-preset-env'),
    require('cssnano'),
  ],
};

Seems like people have different experience on this, I just want to share mine.

I am trying to upgrade to<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>but I am getting this issue as well. Webpack seems to build as expected but prints these warnings a lot of times before printing the final output. I noticed that I only get these warning in development mode.
Here's my updated loader config:
loader: 'postcss-loader',
    options: {
        postcssOptions: {
            plugins: [
                ['autoprefixer', {
                    overrideBrowserslist: targets,
                }],
            ],
        },
        sourceMap: true,
    },
}

Any idea what could be causing this issue?

try this:
plugins: [     require('autoprefixer'),   ],
this worked for me.

npm i --save-dev postcss postcss-focus postcss-preset-env postcss-flexbugs-fixes
copy to
"peerDependencies": {
    "postcss": "..."
 }

more config checkout this link: https://evilmartians.com/chronicles/postcss-8-plugin-migration
create postcss.config.js:
module.exports = {
    plugins: [
        'postcss-flexbugs-fixes',
        [
            'postcss-preset-env',
            {
                autoprefixer: {
                    flexbox: 'no-2009',
                },
                stage: 3,
                features: {
                    'custom-properties': false,
                },
            },
        ],
        require('postcss-focus'),
    ]
}

in the nuxt.config.js file
build: {
postcss: null,
},


feat(code-owner): add enrichment maintainer as code owner

Change request
General information
Summary of change:
Add enrichment maintainer as codeowner for fasttext.
Blast radius
Could impact email notifications for the fasttext repo.
How will you measure the impact on production and potential failure?
Does not impact production.
How will this change be rolled back if this breaks?
Remove CODEOWNERS file and re-commit.
Checklist

[x] I have adhered to the Tink code review
best-practises.
[x] If other teams may be affected by the changes, I have assigned them as
reviewers.
[x] The code I'm changing has a clear code owner. If not, please help out to
find and assign
one!
[ ] I have verified that this PR works in production. (check this after
merging the PR)

If you are unsure/in disagreement about the solution/code, bring in an
additional reviewer!

You're so much better at reviewing than me, @matijabruncic 😢

That's just because up until today I thought that code owners is something Tink specific (😊 ) and started reading about it

To use a CODEOWNERS file, create a new file called CODEOWNERS in the root, docs/, or .github/ directory of the repository, in the branch where you'd like to add the code owners.

Ref: https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/creating-a-repository-on-github/about-code-owners
Should this file be in some other folder?

Whoops! Brain fart. I'll fix and do a commit --amend.


fix: collect actor sender in actor creation

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Currently, the barrier sender of actors are registered via a channel in the local barrier manager. This happens in the process of creating actors. It may happen that the register sender request is handled after the actors have been created and spawned. However, after the actors have been created and spawned, the meta node will start injecting barrier. If the register sender request is handled after the inject barrier request, the inject barrier request will fail at sender not found like in https://github.com/risingwavelabs/risingwave/issues/15703. We try to prevent this by using a biased select like the following
select! {
    biased;
    register_sender = register_sender_rx.recv() => {...}
    inject_barrier = inject_barrier_rx.recv() => {...}
}

Intuitively this should work, because the register sender request is added to the channel before actor creation finished, and inject barrier request is sent after actor creation. When we do biased select, we always poll the register sender channel before the inject barrier channel, and then we should have ensured that the sender is registered successfully before injecting barrier. However, https://github.com/risingwavelabs/risingwave/issues/15703 still happened.
Looking into the code of tokio mpsc channel, we can see that it's using memory order AcqRel rather than SeqCst  maybe out of performance issue. However, when using AcqRel, we can only ensure that the memory order on a single channel is consistent, which means in a single channel if event A is sent before event B, event A must be received before event B. However, we cannot ensure that if event A is sent to channel A before event B is sent to channel B, in another thread event A must be visible in channel A before event B is visible in channel B. As an example, the following code can panic after several iteration.
use std::collections::HashSet;
use std::thread::{sleep, spawn};
use std::time::Duration;
use tokio::select;
use tokio::sync::mpsc;

#[tokio::main]
async fn main() {
    let (tx1, mut rx1) = mpsc::unbounded_channel();
    let (tx2, mut rx2) = mpsc::unbounded_channel();
    let _thread_handle = spawn(move || {
        let mut i = 0_usize;
        loop {
            // for any i, tx1 is sent before tx2.
            tx1.send(i).unwrap();
            tx2.send(i).unwrap();
            i += 1;
            sleep(Duration::from_micros(10));
        }
    });
    let mut set = HashSet::new();
    loop {
        select! {
            biased;
            first = rx1.recv() => {
                set.insert(first.unwrap());
            },
            second = rx2.recv() => {
                let i = second.unwrap();
                // since for any i, tx1 is sent before tx2, and the select is biased to rx1
                // we used to expect that for any i, we will receive i from rx1 before rx2.
                assert!(set.remove(&i), "{}", i);
            }
        }
    }
}

Hence, we cannot guarantee that the register sender request is handled before inject barrier request simply via biased select.
In this PR, we change to store the register the sender in a struct shared via Arc<Mutex<_>>. After actor creation, the registered sender will be taken out and register in the worker loop before spawn the actor and notify finish of actor creation.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Apart from introducing the shared context back, can we make register_send an async function and await on an oneshot channel to ensure the sender is registered when it returns?

Apart from introducing the shared context back, can we make register_send an async function and await on an oneshot channel to ensure the sender is registered when it returns?

Currently it works, but I am worried that this may introduce potential deadlock easily in our future development. Awaiting on the oneshot channel depends on the worker loop to handle the request, and it blocks actor creation. But the worker loop manages actor creation and polls the finish of actor creation to do further work. This can be dangerous, and may introduce implicit deadlock by accidence.


Json encoding

Hi folks,
I wanted to share one concern with you.
Our .json file couldn't be processed due to some issue with Unicode encoding. Apparently, we had a Line Separator char in the string literal. It was invisible in our text editors and we found with an empirical approach. I found that answer on SO that explains why javascript fails to parse it.
I just think that it would be great to automatically escape this char in json-loader somehow. Or let a user know about the possible problem. What do you think?

I'd like to echo the above comment and provide a minimal example demonstrating the problem:
https://github.com/dwbruhn/webpack-json-example
Looks like this issue has been open for a while...will someone be looking into it?

Oh, looks like it was fixed but never published!
https://github.com/webpack-contrib/json-loader/commit/3b3069fa3f9deb35cc625c6c388e234a240ee312
Can someone publish it, please? @sokra

@dwbruhn - This plugins functionality is the default in Webpack > 2.

Hi @d3viant0ne, but this is clearly a bug. Is there some blocker to publishing the fix?

My point is, if you are using webpack 2, you don't need this plugin.
If you are using webpack 1.x, it's not going to be supported for long & I will deprecate this the second it no longer is.
Fix, no fix, new version .... the right answer here is to use Webpack's native support for json and not a really soon to be moth-balled plugin.

Hi @d3viant0ne, right, but it seems that webpack's "native support for json" simply consists of including this module in its package.json.
And the github link on the npm page for the module redirects here.
You can see in my example that I have no webpack plugins configured at all, and yet the problem appears.
Happy to be corrected, though, if I'm missing something!

iirc it's a lack of documentation issue in regards to Webpack's json support.
I'll get a version out as soon as I get back to my desk & see about getting the documentation issue resolved this weekend.

<EMAIL_ADDRESS>is available on npm.
Change can be found here: https://github.com/webpack-contrib/json-loader/blob/master/CHANGELOG.md#055-2017-07-22

Thanks! 👍


feat(config-preview): added

closes #96
Signed-off-by: Lukas Mertens<EMAIL_ADDRESS>
Stack:

#106
#104 ⬅
#102

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #106


Can't get orientation data from pose

How can I get the orientation data? Right now the only non-zero values I get from the pose is the position (x, y, z). I'm using a swift multi in simulation mode with msg IDs 544 and 545 enabled for tcp server 0.
But in the swift sbp spec document it says swift multi and duro do not produce these messages (544 and 545)...
https://www.swiftnav.com/latest/sbp-specification

Duro does provide orientation, we are using it that way. I never used the Piksi Multi, but I'm guessing Multi also has an inertial and a non inertial version like Duro. Are you using the inertial one? If yes, did you enable the IMU?

Ah, I didn't realise they had "inertial" versions of the products.. I don't think there is an inertial version of the multi though as I don't see one on their store.
I do have the IMU enabled but I'm guessing this feature is reserved only for the Duro Inertial and not the regular Duro?

Yes, i think it is  is reserved only for the Duro Inertial, but if you want to be 100% sure you should ask the Swiftnav support, they reply quite quickly.


Dragonrage: Watch myparfait.com

Dragonrage requests the watch of the watch_keyword myparfait\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cigien in Charcoal HQ


Incorrect ordering in --sort-by

What happened:
Pods are not sorted in the correct order when using --sort-by .metadata.name
What you expected to happen:
Pods are sorted correctly, 1..35 followed by a..z
How to reproduce it (as minimally and precisely as possible):

I think this bug is long standing. Repro as follows
for n in {a..z} ; do kubectl run $n --image=nginx:alpine ; done
for n in {1..35} ; do kubectl run $n --image=nginx:alpine ; done

kubectl get po --sort-by=.metadata.name

Pods u, m, n, k come out first in the order and in that order.
Anything else we need to know?:
I have traced this to here
https://github.com/kubernetes/kubectl/blob/master/pkg/cmd/get/sorter.go#L276
These pod names are incorrectly being identified as SI quantities, when without a numerical quantity they should be treated as plain strings. Having said that, a pod name is never an SI quantity, so perhaps this logic should only be used for fields where a quantity can be expected.
Environment:

Client Version: v1.28.1
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.1

Killercoda playground, although as mentioned I think this is long standing

Ubuntu 20.04 LTS

Raised related issue - https://github.com/kubernetes/apimachinery/issues/165

For these letters "eEinumkKMGTP", ParseQurantity will parse them as SI, which is likely its normal logic. I believe in the 'orderby name' use case, we wouldn't expect 'name' to be parsed as SI, hence we may need to add an option?  "Ok, I want to sort by metadata.name, but I have to remember to use --parse-quantity=false to not get odd results". That'll be a bit much for learners.
Much nicer (and I know, much more work), would be to encode that in the internal representation of an object's schema so that the sorter can use this information. Bit more work for CRD developers too.

Perhaps SI values are an entirely new type in the type system?

Either that, or to be treated as SI, a value must have a numeric component prior to any recognizable unit. Issue is that a string literal value of e.g. k is assumed to be SI, when it clearly isn't.


Module does not sign in after gpgs.init

Hey @jlopezcur ,
Thanks for the great plugin, its easy to setup and compile. But after i have followed your instructions and everything seems to work fine, but after trying to sign in i get no pop up view for the google sign in.
The app is also running fine & does not crash. You can see my logcat below

I have tried to fix this to no luck. Maybe you can point me in the right direction.
I have attached my sample scene and debug.apk

By the logcat you posted, I see that the problem is in the Android installation:
Could not find class 'android.os.UserManager', referenced from method com.google.android.gms.common.zze.zzbs
¿Did you follow the instructions on the godot page? http://docs.godotengine.org/en/stable/development/compiling/compiling_for_android.html#requirements
Under the Requirements:

Android SDK version 23.0.3 [Note: Please install all Tools and Extras of sdk manager]
Android build tools version 19.1
Android NDK r13 or later

I think Android build tools version 19.1 have the missing library.
Run again your Android SDK Manager and be sure to install it.
Look when you can and told me.
I will try to be more quick with the answer.

Thanks for the advice @jlopezcur
I got it to work by adding the RealtimeMultiplayer Class from your plugin and then compiling with this module  https://github.com/FrogSquare/GodotGoogleService and then . I will try your advice and give feedback.
On a different topic, do you have an idea of how to fetch the google sign users avatar or profile picture.
My attempts have not been successful. But i should probably post my attempts on a new issue.


I keep getting the error "fail to fetch user tags"

is there any way to fix it?

me too

is there any way to fix it?

Hi, can you please try to access this link directly?
https://www.pixiv.net/ajax/user/ + ${uid} + /illusts/bookmark/tags
Please replace the uid variable with your own uid.
This is the endpoint that pixiv website uses to fetch your tags information, and I have tried it only on my own accounts. It should return a JSON file including the tags.

me too

Hi, can you please try the link above? I would like to check what pixiv returns for your browser.

me too

Hi, can you please try the link above? I would like to check what pixiv returns for your browser.

{"error":true,"message":"\u62b1\u6b49\uff0c\u60a8\u5f53\u524d\u6240\u5bfb\u627e\u7684\u4e2a\u7528\u6237\u5df2\u7ecf\u79bb\u5f00\u4e86pixiv, \u6216\u8005\u8fd9ID\u4e0d\u5b58\u5728\u3002","body":[]}
如果是把 + ${uid} + 这一段全部替换的话就是这个 旁边的两个+号保留{"error":true,"message":"\u65e0\u6cd5\u627e\u5230\u60a8\u6240\u8bf7\u6c42\u7684\u9875\u9762","body":[]}

me too

Hi, can you please try the link above? I would like to check what pixiv returns for your browser.

{"error":true,"message":"\u62b1\u6b49\uff0c\u60a8\u5f53\u524d\u6240\u5bfb\u627e\u7684\u4e2a\u7528\u6237\u5df2\u7ecf\u79bb\u5f00\u4e86pixiv, \u6216\u8005\u8fd9ID\u4e0d\u5b58\u5728\u3002","body":[]}
如果是把 + ${uid} + 这一段全部替换的话就是这个 旁边的两个+号保留{"error":true,"message":"\u65e0\u6cd5\u627e\u5230\u60a8\u6240\u8bf7\u6c42\u7684\u9875\u9762","body":[]}

这段解码出来是：抱歉，您当前所寻找的个用户已经离开了pixiv, 或者这ID不存在。
能确定手动输入的uid是正确的吗

It says I have too many bookmarks.
What's the limit of bookmarks?

It says I have too many bookmarks. What's the limit of bookmarks?
Oh and I have 8 including uncategorised

I am not sure about the limit but I have 84 tags myself. Can you post the complete JSON file here?

me too

Hi, can you please try the link above? I would like to check what pixiv returns for your browser.

{"error":true,"message":"\u62b1\u6b49\uff0c\u60a8\u5f53\u524d\u6240\u5bfb\u627e\u7684\u4e2a\u7528\u6237\u5df2\u7ecf\u79bb\u5f00\u4e86pixiv, \u6216\u8005\u8fd9ID\u4e0d\u5b58\u5728\u3002","body":[]}
如果是把 + ${uid} + 这一段全部替换的话就是这个 旁边的两个+号保留{"error":true,"message":"\u65e0\u6cd5\u627e\u5230\u60a8\u6240\u8bf7\u6c42\u7684\u9875\u9762","body":[]}

这段解码出来是：抱歉，您当前所寻找的个用户已经离开了pixiv, 或者这ID不存在。
能确定手动输入的uid是正确的吗

{"error":false,"message":"","body":{"public":[{"tag":"CG\u96c6","cnt":1},{"tag":"JK","cnt":1},{"tag":"NTR","cnt":4},{"tag":"P5","cnt":1},{"tag":"R-18","cnt":8},{"tag":"T\u30d0\u30c3\u30af","cnt":1},{"tag":"\u304a\u3063\u3071\u3044","cnt":1},{"tag":"\u304a\u306d\u30b7\u30e7\u30bf","cnt":2},{"tag":"\u304a\u5c3b","cnt":1},{"tag":"\u304f\u3071\u3041","cnt":1},{"tag":"\u3080\u3061\u3080\u3061","cnt":1},{"tag":"\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f3","cnt":1},{"tag":"\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f35000users\u5165\u308a","cnt":1},{"tag":"\u30a8\u30ed\u8863\u88c5","cnt":1},{"tag":"\u30aa\u30ea\u30b8\u30ca\u30eb","cnt":1},{"tag":"\u30af\u30a4\u30fc\u30f3\u30fb\u30a8\u30ea\u30b6\u30d9\u30b9(\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f3)","cnt":1},{"tag":"\u30b9\u30d1\u30f3\u30ad\u30f3\u30b0","cnt":1},{"tag":"\u30d0\u7f8e\u8089","cnt":1},{"tag":"\u30d1\u30a4\u30ba\u30ea","cnt":1},{"tag":"\u30d7\u30ea\u30f3\u30c4\u30fb\u30aa\u30a4\u30b2\u30f3(\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f3)","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca1000users\u5165\u308a","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca100users\u5165\u308a","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca5","cnt":2},{"tag":"\u30da\u30eb\u30bd\u30ca\u30b7\u30ea\u30fc\u30ba","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca\u30b7\u30ea\u30fc\u30ba\u30a4\u30e9\u30b3\u30f3","cnt":1},{"tag":"\u30de\u30b0\u30ed\u30ca","cnt":1},{"tag":"\u30de\u30de\u30b7\u30e7\u30bf","cnt":2},{"tag":"\u30e1\u30a4\u30c9","cnt":1},{"tag":"\u4e2d\u51fa\u3057","cnt":3},{"tag":"\u4e73\u98a8","cnt":1},{"tag":"\u4eba\u59bb","cnt":1},{"tag":"\u5236\u670d","cnt":1},{"tag":"\u5373\u5815\u30612\u30b3\u30de\u30b7\u30ea\u30fc\u30ba","cnt":1},{"tag":"\u539f\u521b","cnt":3},{"tag":"\u5973\u5b50\u9ad8\u751f","cnt":1},{"tag":"\u59c9\u306a\u308b\u3082\u306e","cnt":1},{"tag":"\u5c3b\u795e\u69d8","cnt":3},{"tag":"\u5de8\u4e73","cnt":2},{"tag":"\u6771\u90f7\u4e00\u4e8c\u4e09","cnt":1},{"tag":"\u6975\u4e0a\u306e\u4e73","cnt":1},{"tag":"\u6975\u4e0a\u306e\u5973\u4f53","cnt":1},{"tag":"\u6975\u4e0a\u306e\u8089","cnt":1},{"tag":"\u6b66\u898b\u5999","cnt":1},{"tag":"\u6bcd\u4e73","cnt":3},{"tag":"\u6bcd\u5b50\u76f8\u59e6","cnt":1},{"tag":"\u6d5c\u98a8","cnt":2},{"tag":"\u7206\u4e73","cnt":2},{"tag":"\u7fa9\u6bcd","cnt":1},{"tag":"\u8266\u3053\u308c","cnt":2},{"tag":"\u8266\u3053\u308c1000users\u5165\u308a","cnt":1},{"tag":"\u8266\u3053\u308c5000users\u5165\u308a","cnt":1},{"tag":"\u8266\u968a\u3053\u308c\u304f\u3057\u3087\u3093","cnt":2},{"tag":"\u843d\u9078","cnt":1},{"tag":"\u8fd1\u89aa\u76f8\u59e6","cnt":1},{"tag":"\u9006NTR","cnt":1},{"tag":"\u9670\u6bdb","cnt":2},{"tag":"\u9a0e\u4e57\u4f4d","cnt":1},{"tag":"\u9b45\u60d1\u306e\u8c37\u9593","cnt":3},{"tag":"\u9b54\u738b\u30de\u30b0\u30ed\u30ca","cnt":1},{"tag":"\u9e7f\u5cf6","cnt":2},{"tag":"\u672a\u5206\u985e","cnt":1834}],"private":[{"tag":"\u672a\u5206\u985e","cnt":111}],"tooManyBookmark":false,"tooManyBookmarkTags":false}}
昨天多了个空格

me too

Hi, can you please try the link above? I would like to check what pixiv returns for your browser.

{"error":true,"message":"\u62b1\u6b49\uff0c\u60a8\u5f53\u524d\u6240\u5bfb\u627e\u7684\u4e2a\u7528\u6237\u5df2\u7ecf\u79bb\u5f00\u4e86pixiv, \u6216\u8005\u8fd9ID\u4e0d\u5b58\u5728\u3002","body":[]}
如果是把 + ${uid} + 这一段全部替换的话就是这个 旁边的两个+号保留{"error":true,"message":"\u65e0\u6cd5\u627e\u5230\u60a8\u6240\u8bf7\u6c42\u7684\u9875\u9762","body":[]}

这段解码出来是：抱歉，您当前所寻找的个用户已经离开了pixiv, 或者这ID不存在。
能确定手动输入的uid是正确的吗

{"error":false,"message":"","body":{"public":[{"tag":"CG\u96c6","cnt":1},{"tag":"JK","cnt":1},{"tag":"NTR","cnt":4},{"tag":"P5","cnt":1},{"tag":"R-18","cnt":8},{"tag":"T\u30d0\u30c3\u30af","cnt":1},{"tag":"\u304a\u3063\u3071\u3044","cnt":1},{"tag":"\u304a\u306d\u30b7\u30e7\u30bf","cnt":2},{"tag":"\u304a\u5c3b","cnt":1},{"tag":"\u304f\u3071\u3041","cnt":1},{"tag":"\u3080\u3061\u3080\u3061","cnt":1},{"tag":"\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f3","cnt":1},{"tag":"\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f35000users\u5165\u308a","cnt":1},{"tag":"\u30a8\u30ed\u8863\u88c5","cnt":1},{"tag":"\u30aa\u30ea\u30b8\u30ca\u30eb","cnt":1},{"tag":"\u30af\u30a4\u30fc\u30f3\u30fb\u30a8\u30ea\u30b6\u30d9\u30b9(\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f3)","cnt":1},{"tag":"\u30b9\u30d1\u30f3\u30ad\u30f3\u30b0","cnt":1},{"tag":"\u30d0\u7f8e\u8089","cnt":1},{"tag":"\u30d1\u30a4\u30ba\u30ea","cnt":1},{"tag":"\u30d7\u30ea\u30f3\u30c4\u30fb\u30aa\u30a4\u30b2\u30f3(\u30a2\u30ba\u30fc\u30eb\u30ec\u30fc\u30f3)","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca1000users\u5165\u308a","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca100users\u5165\u308a","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca5","cnt":2},{"tag":"\u30da\u30eb\u30bd\u30ca\u30b7\u30ea\u30fc\u30ba","cnt":1},{"tag":"\u30da\u30eb\u30bd\u30ca\u30b7\u30ea\u30fc\u30ba\u30a4\u30e9\u30b3\u30f3","cnt":1},{"tag":"\u30de\u30b0\u30ed\u30ca","cnt":1},{"tag":"\u30de\u30de\u30b7\u30e7\u30bf","cnt":2},{"tag":"\u30e1\u30a4\u30c9","cnt":1},{"tag":"\u4e2d\u51fa\u3057","cnt":3},{"tag":"\u4e73\u98a8","cnt":1},{"tag":"\u4eba\u59bb","cnt":1},{"tag":"\u5236\u670d","cnt":1},{"tag":"\u5373\u5815\u30612\u30b3\u30de\u30b7\u30ea\u30fc\u30ba","cnt":1},{"tag":"\u539f\u521b","cnt":3},{"tag":"\u5973\u5b50\u9ad8\u751f","cnt":1},{"tag":"\u59c9\u306a\u308b\u3082\u306e","cnt":1},{"tag":"\u5c3b\u795e\u69d8","cnt":3},{"tag":"\u5de8\u4e73","cnt":2},{"tag":"\u6771\u90f7\u4e00\u4e8c\u4e09","cnt":1},{"tag":"\u6975\u4e0a\u306e\u4e73","cnt":1},{"tag":"\u6975\u4e0a\u306e\u5973\u4f53","cnt":1},{"tag":"\u6975\u4e0a\u306e\u8089","cnt":1},{"tag":"\u6b66\u898b\u5999","cnt":1},{"tag":"\u6bcd\u4e73","cnt":3},{"tag":"\u6bcd\u5b50\u76f8\u59e6","cnt":1},{"tag":"\u6d5c\u98a8","cnt":2},{"tag":"\u7206\u4e73","cnt":2},{"tag":"\u7fa9\u6bcd","cnt":1},{"tag":"\u8266\u3053\u308c","cnt":2},{"tag":"\u8266\u3053\u308c1000users\u5165\u308a","cnt":1},{"tag":"\u8266\u3053\u308c5000users\u5165\u308a","cnt":1},{"tag":"\u8266\u968a\u3053\u308c\u304f\u3057\u3087\u3093","cnt":2},{"tag":"\u843d\u9078","cnt":1},{"tag":"\u8fd1\u89aa\u76f8\u59e6","cnt":1},{"tag":"\u9006NTR","cnt":1},{"tag":"\u9670\u6bdb","cnt":2},{"tag":"\u9a0e\u4e57\u4f4d","cnt":1},{"tag":"\u9b45\u60d1\u306e\u8c37\u9593","cnt":3},{"tag":"\u9b54\u738b\u30de\u30b0\u30ed\u30ca","cnt":1},{"tag":"\u9e7f\u5cf6","cnt":2},{"tag":"\u672a\u5206\u985e","cnt":1834}],"private":[{"tag":"\u672a\u5206\u985e","cnt":111}],"tooManyBookmark":false,"tooManyBookmarkTags":false}}
昨天多了个空格

这个返回结果是正确的，但插件还是不能用是吗？也可能是解码的问题，我研究一下

The new version is working

The new version is working

cool


[api] More agresive image searcher while populating objects

Current politic is to search images that have both fanart and thumb.
But that's not always the case - need to make better code for that
All items in filter have thumb, some of them have fanart - but still 'filter' still miss art collection.

Probably 'best of 3' would be good;


React Native app to sumarise notifications

For a given window, give a per-application summary of all notifications. Will inform #874

https://docs.expo.dev/tutorial/create-your-first-app/
This will only work on android as it is not possible to see other app notifications on ios.


Remove expirationDuration

Proposing to remove expirationDuration from the Remote Config getting started flow to simplify usability:

It prevents console changes from being fetched
Even after changing the value, the previous run's fetch is still locked in
I can't think of a use case where I want to call fetch but only fetch old values

cc: @karenyz


Fixed Alignments and added hover in numbers

Would be great if you create issue and attach this PR
also, attach screenshot or video #3

@vaishnavi-2901 ya sure

Fixed the issue #9
https://github-production-user-asset-6210df.s3.amazonaws.com/79904168/344483280-23ec952a-66f9-40ae-8af1-193a4ec02aa7.mp4


/static/js/main.6d274e43.js ContentDownload too long

https://github.com/casdoor/casdoor/assets/19483437/0a8c63d2-93e1-49f3-9719-4251008173d9
hi, main.6d274e43.js is too big, it's 8M and it's taking too long to load, is there anything I can do to optimize it? I am running casdoor from docker.

@moxixuan use latest version Casdoor, it is only 2 MB


Non-local clients need server address for websocket

The UI hardcodes localhost:4550 as a websocket address for communication with the OSC bridge.  The server should replace localhost with whatever the server's IP address is (or a flag value.)

@3d0b4ea7d1b3ae1bbfbbfaa35b8e8b8890360aa0
Usage: ./build/sonic-jam --ui_ip <IP_ADDRESS> (or whatever IP you want to serve on)
There are a bunch of other flags I added as well.  See config.go for the whole list.


[SPARK-48714][PYTHON] Implement DataFrame.mergeInto in PySpark

What changes were proposed in this pull request?
Spark 4.0 added a new df.mergeInto API, but it is missing from PySpark. This PR fixes that.
The support for this API in Spark Connect Python API will be added later by https://github.com/apache/spark/pull/46960.
Why are the changes needed?
Because PySpark does not support df.mergeInto.
Does this PR introduce any user-facing change?
Yes, the user would be able to use the df.mergeInto API.
How was this patch tested?
New unit tests.
Was this patch authored or co-authored using generative AI tooling?
No.

@huaxingao @viirya would you mind taking a look please?

I quickly went through the PR, and it looks good overall. I echo @viirya's  suggestion to add a few more tests to make this more robust.

Merged to master.

Hi @HyukjinKwon  @viirya ,
It looks like this change has been merged, but I still don't see any documentation on mergeInto in the Spark 3.1.3 Python API documentation.

this was merged for Spark 4.0.0

New feature won't be backported to previous versions.
You should see the documentation after Spark 4.0.0 releases.

@viirya Do you know when is the Spark 4.0.0 release scheduled?

I'm not sure. I think the original schedule is Jun, 2024. The 4.0.0 preview was released in Jun, 2024. But I don't find the date of the final release.


[Windows] Set toast's duration and image

Hi! Thanks for useful plugin!
Added passing arguments for toast's duration ("short" is about 5 sec) and image path (when it's null, using Text02, either ImageAndText02 template)

Is it good news here?


DOM event handling

Add proper DOM event hander (addEventListener) and full event support. See http://dvcs.w3.org/hg/progress/raw-file/tip/Overview.html.

@driverdan I have code like this:
xhr.addEventListener('load', e => {
  ....
});

but e is undefined. Is it expected?


#27 fixed corrected the delta definition with the reverse order of the records.

Awesome, thanks!
Would be good to have a test for it, of course.
Please consider adding one.

#29


There is a blank at the bottom of the window when the hidden taskbar is enabled.

Windows Terminal version
1.17.11461.0
Windows build number
10.0.22621.1848
Other Software
None.
Steps to reproduce
1.Enable the auto hide taskbar option.
2.Open the Windows Terminal.
3.Maximize the window.
Expected Behavior
At maximization, the window should cover the entire screen.
Actual Behavior

There is a blank at the bottom of the window when the hidden taskbar is enabled.

I believe this has always been like that on Windows. See here: https://github.com/microsoft/terminal/issues/12959#issuecomment-1337588535
I've updated the title of #12959 to make it easier to find in the future. I'll mark this issue as a duplicate of it, so that we can continue to use it to track this issue, even if we likely cannot properly fix it. /dup #12959


feat: read contracts (multicall, batching, pagination/infinite scroll)

Description
This PR adds the following hooks:
useContractReads
Hook for calling multiple ethers Contract read-only methods.
Simple usage
import { useContractReads } from 'wagmi'

const wagmigotchiContract = {
  addressOrName: '0xecb504d39723b0be0e3a9aa33d646642d1051ee1',
  contractInterface: wagmigotchiABI,
}
const mlootContract = {
  addressOrName: '0x1dfe7ca09e99d10835bf73044a23b73fc20623df',
  contractInterface: mlootABI,
}

function App() {
  const { data, isError, isLoading } = useContractReads({
    contracts: [
      {
        ...wagmigotchiContract,
        functionName: 'getAlive',
      },
      {
        ...wagmigotchiContract,
        functionName: 'getBoredom',
      },
      {
        ...mlootContract,
        functionName: 'getChest',
        args: [69],
      },
      {
        ...mlootContract,
        functionName: 'getWaist',
        args: [69],
      },
    ],
  })
}

Read more
useContractInfiniteReads
Hook for calling multiple ethers Contract read-only methods with "infinite scrolling" ("fetch more") support. Useful for rendering a dynamic list of contract data.
Simple usage
import { useContractInfiniteReads } from 'wagmi'
import mlootABI from './mlootABI.json'

const contractConfig = {
  addressOrName: '0x1dfe7ca09e99d10835bf73044a23b73fc20623df',
  contractInterface: mlootABI,
}

function App() {
  const { data, fetchNextPage } = useContractInfiniteReads({
    cacheKey: 'mlootAttributes',
    contracts: (param = 0) => [
      { ...mlootContractConfig, functionName: 'getChest', args: [param] },
      { ...mlootContractConfig, functionName: 'getFoot', args: [param] },
      { ...mlootContractConfig, functionName: 'getHand', args: [param] },
    ],
  })
}

Read more
Also a few other changes, read the changesets here

It seems useContractReads  can only call 62 ERC20 contracts' balance. Can the hook fetch more contracts?

@hskang9 — can you provide me with an minimal reproducible example? Should work with indefinite number of contracts.


New behavior of wildcard in 2.2 silently breaks alerts

Previously bosun would let you use both * and | in the match criteria for tags. The new 2.2 default will silently not return any data if you specify a name=optiona|any*|optionb or any other combination of | and *. Kyle already filed an issue with https://github.com/OpenTSDB/opentsdb/issues/650 but if it is not fixed upstream before releasing the next version we should return an error for any tag criteria that include a * and | but are not using the regexp() filter. Otherwise we will silently break existing alerts (we had multiple broken in our config for the last week or so)

Hmm, 2.2 should treat these as literals in the same way 2.1 did, e.g. it would match name=optiona || name=any* || name=optionb but not match name=anysomthing since TSDB didn't support wildcards previously. Did Boson do post-filtering when a wildcard was given?

Yes, Bosun did its own wildcard expansion when there was an asterisk. So it
is the ability to mix or and wildcard.
On Dec 3, 2015 8:51 PM, "Chris Larsen"<EMAIL_ADDRESS>wrote:

Hmm, 2.2 should treat these as literals in the same way 2.1 did, e.g.
it would match name=optiona || name=any* || name=optionb but not match
name=anysomthing since TSDB didn't support wildcards previously. Did
Boson do post-filtering when a wildcard was given?
—
Reply to this email directly or view it on GitHub
https://github.com/bosun-monitor/bosun/issues/1495#issuecomment-161848425
.

With #1710 it made me think of another workaround (I haven't tried it yet though, and it was really meant for this):
$a = q("min:ping.timeout{dst_host=iwildcard(foo*)" ....)
$b = q("min:ping.timeout{dst_host=iwildcard(bar*)" ....)
$c = merge($a, $b)

Merge is meant for graphing and not alerting, but this may work. I've pinged them on the opentsdb issue, maybe it can get added to 2.3.

In terms of the Milestone, we need to make the config error out on the condition so alerts do not silently break.

Documented in 0.5.0 release notes draft I have seen form kyle. Closing.


DBZ-7813: waitForAvailableRecords using Awaitility

It appears that under some conditions the test runner thread does effectively never yield during the busy-wait loop in the original waitForAvailableRecords(). With constrained CPU resources this can cause the engine thread to not run and thus not generate records in time. Awaitility sleep-wait alleviates this.

@nrkljo Does it make sense to intorduce a new method? There is already waitForAvailableRecords(...) on the class. Maybe it would make sense to modify that one to use Awaitility?

@nrkljo Applied, thanks


MYWO popup dialogs are not scrollable

Overflowing dialog content is hidden.
(MYWO tangent issue. Assigning to Andreas.)

Fixed


Create EasyDNS.ps1

Added script for EasyDNS and wildcard DNS verification

Thank you for creating the EasyDNS validation plugin. Can you please provide an example command-line usage for the plugin? How are the token and key values that are obtained from EasyDNS to be specified on the command line? Let's assume that my key is "api123x4567y89z10.12345678" and my token is "u123x4567y89z10.12345678".
wacs --test --verbose --accepttos --emailaddress<EMAIL_ADDRESS>--source manual --friendlyname "<My Friendly Name>" --commonname "<hostname.mydomain.com>" --host "<san.mydomain.com>" --store pemfiles --pemfilespath "<C:\Temp\MyPEMfiles>" --validationmode dns-01 --validation script --dnsscript "C:\ProgramData\chocolatey\lib\win-acme\tools\Scripts\EasyDNS.ps1"
I have tried using --EDKey api123x4567y89z10.12345678 and --EDToken u123x4567y89z10.12345678, and I have tried using ----eab-key and ----eab-key-identifier but neither have worked.

In the file EasyDNS.ps1 I noticed something that may be a documentation bug
In the following lines
.EXAMPLE

EasyDNS.ps1 create {RecordName} {Token} EDToken EDKey

EasyDNS.ps1 delete {RecordName} {Token} EDToken EDKey

should {Token} be replaced with {TxtValue} ?

I responded to your question here #2242


Add requirements.txt with pip freeze

Installed following packages with pip:

mercantile
fiona

cc: @rodowi


French localization

Hello,
I've made the French localization, can you add it?
https://www.dropbox.com/s/b4qagnn7m61zrgb/fr.lproj.zip?dl=0
I'll be available for future translations of Messenger.

Updated translation:
fr.lproj.zip
Can you add it?
Still available for future translations of Messenger... if you add it.


Live tab values not refreshing

Hi,
I do not know if it is supposed to refresh, but my live values are not live. I need to manual use refresh (f5) on chrome to get new values even tough the updated states keeps flashing "a few seconds ago".
There are no adBlockers active or anything.
Anything i'm missing here?

The tables in "live" tab are updating  without the need to refresh the page manually. How do you determine the values are not refreshing? Can you give an example?

I compare the values to those of poloniex itself by looking at the lending page. Just to be complete: I meant the fee values of existing loans. Loans not created by the bot. They existed pre bot. But are shown ok when refreshing manuallyOp 1 aug. 2017 9:10 p.m. schreef dutu notifications@github.com:The tables in "live" tab are updating  without the need to refresh the page manually. How do you determine the values are not refreshing? Can you give an example?
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

There could be a slight sync time gap between the poloLender app and Poloniex site since they don't refresh at exactly same time. The time gap is probably i and the range of milliseconds or very few seconds in the worst case.
Since Poloniex doesn't not provide a faster API method (neither for external apps like poloLender nor for own Web interface), it's best it can be done.
Please also note that poloLender data is refreshed at higher speed than the Poloniex Web interface itself
😊

I understand what you mean, but I actually mean "not updating. Not live". It does not matter how long I wait.

Can you give me a javascript method / variable to start looking at? I'm ok with trying to debug some by console.

I seem to have more issues on chrome, like not continuing after login unless I refresh manually. Might be a local issue for me.... but I hardly experience other issues with websites...

fixed in the new version 1.3.1 that has just been released.
Let me know if it works as wanted

Both issues fixed. Thanks!
Will donate when my bot runs full time with a bit more coins


[DO NOT MERGE] Add ga tracking to find your local council

We add Google Analytics tracking to 'Find your local council'. This aims to provide similar error tracking as already present for 'Local Transactions'.
We add tracking for:

postcodeSearchStarted
postcodeResultsShown (with number of results of shown)
exit link tracking for unitary/district/county homepage URLs
postcodeError tracking, specifcally:

postcodeErrorShown:invalidPostcodeFormat
postcodeErrorShown:noLaMatch
postcodeErrorShown:fullPostcodeNoMapitMatch

For https://trello.com/c/qOtW0fJB/444-analytics-for-find-my-local-council-2
Paired with @emmabeynon
This branched off of https://github.com/alphagov/frontend/pull/986, so will need to be rebased after that branch gets merged.

Good stuff.  GH seems to have the commits in a weird order, but I'm sure it'll be fine when we merge it into the other branch.
One thought is that we could merge the commit where we introduce alert_type and the subsequent commit where we change it to publication_format as the change only occurs a commit apart - we could just pretend we got it right all along.

thanks @h-lame, have addressed your comments and rebased!

Merging this into 'build-find-local-council' branch which already has an open PR


Add two issue templates: bug and enhancement

Create two issue templates in the meshery-istio repo by copying the meshery repo's issue templates.

fixed by #16


Request for Information: Next Build Release Date for Terraformer

Hello,
I would like to inquire about the release date for the new version of Terraformer that includes compatibility with the latest version of Terraform. Additionally, I am interested in knowing if this new release addresses any known bugs or issues present in the previous version.
Could you kindly provide information on the expected release date for this update? This will help me plan my usage of Terraformer accordingly.
Thank you!

Hi, the latest version was released on July 4th. Can you check if the bug is fixed?


BUG: Alignment Issues on About Page on Mobile Screen

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There are alignment issues on the About page when viewed on mobile screens. Elements are not properly aligned, impacting the readability and aesthetics of the page. This needs to be fixed to ensure a consistent and user-friendly experience across all devices.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Go ahead


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #494 (2eae665) into main (8092464) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #494   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

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


Max_bundles not working properly

I recently installed this package. It looks like the max_bundles setting is not working correctly. My setting is set to 5 bundles.

Actually this might be an issue where the process did not finish but uploaded the bundles but did not update the history.json file

Hey @likeadeckofcards, yeah I noticed this too yesterday - the bundles are getting deleted but not being removed from the history.json file. It’s on my list for things to fix but thank you for confirming the bug, I’ll get it sorted soon! ✌

I am closing this because the extra bundles that I was seeing were due to failed deployments, where the code threw an exception.

Great - I did look into this a little bit but couldn't find a bug. Glad it's working!


Add new tap for 1.4.4

Remove Gerrit reference. Remove stalebot config.
I added a tap for 1.4.4 and updated the other taps to point to GitHub for artifacts, not Nexus.

@mastersingh24 @alexvicegrab @christo4ferris since you're the only other people to commit to this repo - I request your reviews specifically


Headlamp doesn't send apiGroup parameter to the SelfSubjectAccessReview API

Hello, I was trying to log into Headlamp using the access token for a user who has RoleBinding to the ClusterRole admin that is restricted with a specific namespace (it means it should have full access to the namespace scoped resources).
After this It was found out that my user cannot edit Deployments and other resources that have apiGroups.
It seems like the issue is because Headlmap doesn't send correct apiGroup parameter when requesting for checking access for the current user:
{"kind":"SelfSubjectAccessReview","apiVersion":"authorization.k8s.io/v1","spec":{"resourceAttributes":{"verb":"update","name":"my-deployment","namespace":"my-namespace","resource":"deployments"}}} - this is the current request sent by Headlamp to the kube api server and returns false as cluster role admin allow update only for the deployments that are under the apiGroup "apps":
- apiGroups:
  - apps
  resources:
  - daemonsets
  - deployments
  - deployments/rollback
  - deployments/scale
  - replicasets
  - replicasets/scale
  - statefulsets
  - statefulsets/scale
  verbs:
  - create
  - delete
  - deletecollection
  - patch
  - update

If I add apiGroups parameter to the permission check request then its working fine:
{"kind":"SelfSubjectAccessReview","apiVersion":"authorization.k8s.io/v1","spec":{"resourceAttributes":{"verb":"update","name":"my-deployment","namespace":"my-namespace","resource":"deployments","group":"apps"}}}
So it seems like Headlamp should send this apiGroup parameter to correctly check permissions for the resources that are under apiGroups.

Hi @Denis220795 . Thanks a lot for reporting this. It's an important issue!
I have been working on this. Soon, we will have a better solution by getting API info from the API server but for now I have added a fix that is IMO good enough.
Can you tell me what flavor of Headlamp you have used? I should be able to get you a build so you can test my solution.

Hi @joaquimrocha , I was using Headlamp in server mode and deployed ghcr.io/headlamp-k8s/headlamp:v0.20.1 with the Helm chart version 0.16.0 to the local Minikube with 1.24 Kubernetes version. So in case you can provide a new image with the fix I can try to deploy and test it.

@Denis220795 , ah, maybe in that case maybe it's easier if you switch to the fix-permissions-api branch and run make image. So you get the image right away. WDYT?

@joaquimrocha yep, I can try this as well, will let you know once finish the testing.

Hi @joaquimrocha I was able to build and test the image from the branch you have provided and can confirm its working fine now. I can view and edit resources that are under apiGroups in the namespaces where I'm admin (with namespace restricted access only).
Also attaching the request payload that is sent now to the SelfSubjectAccessReview API:
{"kind":"SelfSubjectAccessReview","apiVersion":"authorization.k8s.io/v1","spec":{"resourceAttributes":{"verb":"edit","name":"my-deployment","namespace":"my-namespace","group":"apps","version":"v1","resource":"deployments"}}}
Looking forward for the release that will contain this fix 👍
Thanks!


All OpenCL programs hang on Solus

I'm currently trying to package ROC 2.10.0 for Solus and I ran into an issue, I'm using a RX5700XT with an AMD A10-6700 and I'm running kernel 5.4.8.
At the moment I think ROCr-Runtime is working fine as rocminfo reports my GPU correctly, I'm having a problem with ROCm-OpenCL-Runtime though. As soon as a program tries to start OpenCL it hangs and use 100% of a CPU core. For instance, clinfo hangs without printing anything and Blender becomes irresponsive when I try to open the settings panel (that's when it looks for OpenCL devices) I need to CTRL-C clinfo and kill blender or they won't ever exit.
Here are the logs I see you've asked in other posts:
lspci -v
rocminfo
groups
strace clinfo
ldd clinfo
As you can see in Solus we don't use /opt so all binaries and libs are in /usr/bin/ and /usr/lib64, the binaries and libs created by rocm-opencl-runtime are located in /usr/bin/rocm-opencl and /usr/bin/rocm-opencl but since rocminfo runs fine and the strace shows clinfo can find everything it needs to I don't think it's a problem

@Tzigamm Some time ago I tried packaging ROCM for Solus but I gave up. Some days ago was a post in the Solus Forum that someone packaged ROCM up to OpenCl for Solus and needs Tester/Reviewer. I started yesterday the building of those packages and gave some feedback on the corresponding respository. Maybe we can concentrate our efforts here to get the stuff running ?

Hi @Tzigamm, please check latest ROCm Documentation and ROCm 5.7.1 to see if your issue has been resolved.  If resolved, please close the ticket.  Thanks.

Hey @nartmada, sorry for letting this go stale, I haven't used Solus in a while so I can't test if the issue is still there. Closing this


Allow consistency_fail to run into a rails plugin/engine

This 2 commits:

resolve ptyagi16/guard-consistency_fail#2
restore spec/support files require order (to run rspec tests)

Closing for a new PR as errors still remain.


Go to definition should still work for shorthand property declaration with no local binding

declare function foo(option: { [|abcd|]: number }): void;

foo({
    abcd/*use*/
});

Go to definition at /*use*/.
Expected: Language service gracefully finds abcd.
Actual: No definition is available.
This is frustrating because much of the time as I'm actually writing out the type, I want to know what abcd is so I'll jump to its definition. Instead, I can't get any functionality.

Maybe related to #30037


Add details on uninstalling docker deployment

Pull Request Test Coverage Report for Build 77fe4a4cc314c8f37a3829d320897124a9528dd5-PR-1066

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

Totals

Change from base Build d1f5ab34311388120c2057c036ce4c53a756a3ad:
0.0%

Covered Lines:
1148

Relevant Lines:
1740

💛  - Coveralls


Wrong hostname in modsecurity ModSecurity API function, which accepts a server IP address as an argument

Describe the bug
We have HAProxy Enteprise which uses ModSecuirty v2.x. In the modsecurity logs, such as shown below, the private IP address of HAProxy appears in the host name field, i.e. 10.X.Y.Z, so the host name is missing.  This issue makes it extremely difficult to understand which hostname matches with a ModSecurity rule. I have already contacted HAProxy technical support who answered me like this:
"I've got the answer from the engineering team, the ModSecurity API function, which is used to process the connection, accepts a server IP address as an argument (in our case that's IP address used on haproxy), which is later used in the logs. It says "hostname", but it's actually an IP address. Other ModSecurity implementations, such as the one for nginx also uses IP address, not hostname, as it's the most used implementation and we don't want to fork this code part we will leave it like this."
Logs and dumps
Sep XX 00:XX:XX localhost hapee-lb[2135709]: https-in/owasp_crs ModSecurity: Warning. detected XSS using libinjection ... [tag "attack-xss"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/152/242"] [hostname "10.X.Y.Z"] ... (omitted)
Notice: Be carefully to not leak any confidential information.
To Reproduce
Steps to reproduce the behavior:
Install HAProxy Enteprise and enable ModSecurity module
Expected behavior
hostname as the word itself says should refer to an FQDN (see https://en.wikipedia.org/wiki/Hostname) and not contain an IP address which is the internal one of the proxy.
Server (please complete the following information):

ModSecurity version  with HAProxy Connector
HAProxy Enteprise 2.4r1 LTS
OS (and distro): RHEL 8

Rule Set (please complete the following information):
All rules

Hello @alfonsoalongi ,
I cannot speak directly to HAProxy per se, but in general, in a standard ModSecurity v2 deployment with Apache HTTP Server ...
ModSecurity will populate the audit log's hostname from Apache's ServerName`` configuration directive **if** the UseCanonicalName On``` is set.
If ModSecurity does not have access to the name that way, it will use the value of the 'Host' header. If this value is name like "modsecurity.org" then that will be output. If the value of the Host header is a decimal-dot IP address, then that will be used.

Hello @martinhsv,
thank you so much for your answer. I have reopened a ticket on HAProxy support and am awaiting a response from the engineering team. I'll let you know as soon as I have any updates on this.

Was there anything further on this?
Based on what has been presented so far, this should probably get closed from a ModSecurity perspective.


ci: bump mage-tools

Codecov Report

Merging #54 (53a7979) into master (71a780d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #54   +/-   ##
=======================================
  Coverage   44.88%   44.88%
=======================================
  Files           7        7
  Lines         127      127
=======================================
  Hits           57       57
  Misses         61       61
  Partials        9        9


createSelect: fix initial value settin for select element (form block)

the <select> element does not support the value attribute thus the option.value needs to be compared to the fd.Value instead of select.value
Fix #40

@felix-staud can you please sign the CLA and provide a test url?

@felix-staud can you please sign the CLA and provide a test url?

CLA
So I did sign the CLA (and made sure to close and re-open the issue after this. But doesn't seem like this worked? I'm using a multitude of emails, could that be the issue?)
Test URL
https://main--aem-live-test--felix-staud.hlx.page/form/form-demo
Sheet

Name
Type
Label
Placeholder
Value
Options
Mandatory
Style
ID
Fieldset

select
select
Country

USA, Germany, Switzerland, Canada

selectWithvlaue
select
Favorite Color

green
red, yellow, blue, green, black, white

submit
submit

confirmation

https://main--aem-live-test--felix-staud.hlx.page/form/form-thank-you

URL for testing:

https://main--aem-live-test--felix-staud.hlx.page/form/form-demo


Units warning for special values

DEPRECATION WARNING: span_frames_min_duration: '-1' (source: -javaagent options) is not using a time unit. Please use one of 'ms', 's' or 'm'.

It makes no sense to have units for special values like -1.

I see what you are saying but it means that we do have to allow not specifying the unit. In the next major version, we want to disallow not specifying the unit as it makes the options easier to read (you don't have to remember the default unit). A consequence of that is that you have to always be explicit, also when specifying -1ms, for example.

Documentation should be updated. Is it -1ms or -1s etc. that is the special value, or does any negative value disable the limit?

You can use any negative value, for example -1ms. I've clarified the documentation. Thanks for pointing this out! 🙂


Set default llvm version to 15

One test is failing for me locally: effekt.LLVMTests.examples/llvm/forking.effekt.

==> X effekt.LLVMTests.examples/llvm/forking.effekt (llvm)  0.306s java.lang.RuntimeException: Nonzero exit value: 134
    at scala.sys.process.ProcessBuilderImpl$AbstractBuilder.slurp(ProcessBuilderImpl.scala:164)
    at scala.sys.process.ProcessBuilderImpl$AbstractBuilder.$bang$bang(ProcessBuilderImpl.scala:121)
    at effekt.Driver.evalLLVM(Driver.scala:155)
    at effekt.Driver.evalLLVM$(Driver.scala:25)
    at effekt.LLVMTests$$anon$1.evalLLVM(LLVMTests.scala:31)
    at effekt.Driver.eval(Driver.scala:109)
    at effekt.Driver.eval$(Driver.scala:25)
    ```

CI on the main branch fails due to the install script switching to llvm 16.

@b-studios honestly, I don't know what could have caused that failure.
Version 12 shouldn't work at all at this point, and if 15 is already in use, this shouldn't change anything.

Ok, let's just merge this and I'll try to figure it out on my end.


Making sure MUD dates are not set for overseas

CI: Security warnings found!

CI: Security warnings found!

CI: Security warnings found!


Fix bug of creating instance with local ssd using nvme interface

Local-ssds with nvme interface created by GCE use name like nvme0n1, nvme0n2...
And those local ssds are under the /dev .

LGTM.


Separate prison and police move form functionality

Ticket: https://trello.com/c/eG0uLtUP
It won't make sense to merge this until Cesi's police user work is done.

Superseded by https://github.com/ministryofjustice/moving-people-safely/pull/488


changes/improvements to CONTRIBUTING

The initial commit of Contributing is merged to provide its info to DHD contributors
the improvement suggestion from that PR are still valid and mad into this issue:

Possible further additions/edits are
including some of the rake commands into contributing
referencing contributing from README
adding a section on Reviewing
section or reference about style-guide and use of typographical features, including durations, summaries, yaml-frontmatter etc.

request to compare workflow with atlassian gitflow documentation:
https://www.atlassian.com/git/tutorials/comparing-workflows/gitflow-workflow


Idea for later : add ccya ~ in the pie chart

In the pie chart of the report add the proportion of ccya ~

Was fun. Commited to noquick


Use @MirrorsUsed(..) annotiation

This commit introduces the @MirrorsUsed(..) annotation as described here: https://api.dartlang.org/stable/1.24.2/dart-mirrors/MirrorsUsed-class.html
This leads to less generated code:
Before: Compiled 6,085,576 characters Dart to 9,543,914 characters JavaScript in 19.4 seconds
After: Compiled 6,085,628 characters Dart to 787,162 characters JavaScript in 4.60 seconds

Did you use the reflection_remover transformer when you tried without @MirrorsUsed?
- angular2/transform/reflection_remover:
    $include:
      - web/main.dart

No, I didn't, but tried it out.
I stumbled across this phenomenon, when I was using dart2js to compile
the main.dart of my project using dart-lang/angular. Compiling it
produced a warning which lead me here.
Is using that transformer best practice? I just liked the idea of using
a simple annotation directly in dart-lang/angular and was wondering
why it is not used.

Reflection (dart:mirrors) breaks tree-shaking because this way dart2js can't know anymore which parts of the code are actually used and which not, this is why your build output becomes so big.
@MirrorsUsed() allows to guide dart2js tree-shaking when reflection is used.
The reflection_remover transformer replaces dart:mirrors usage by generates code that avoids reflection entirely.
AFAIK this is way better than guiding dart2js.

There is no more reflection (at all) in AngularDart 4.0.


request for lightweight server wrapper

It would be helpful to offer a standard approach for serving the application from a standalone application on a remote server.
It would seem possible to provide   a package that would  include both the existing web application and a lightweight web server.
In such a model, a process running on a remote server would serve the web page, and also would provide a few GET and POST operations for managing the user settings, which the server would store in a designated directory. The client web page would store settings through these operations, rather than local storage. The server would store the configuration in a  directory that could be passed to the server process on the command line. The server might  keep a password hash as well, so that the web access may be protected by authentication.
This approach would be comparable to existing solutions for other applications. For example, The Deluge Bittorrent application includes a web server that runs as a standalone application that may be accessed by a user through the web browser, and also connects to a daemon process managing the file transfers.
I wonder whether someone might wish to attempt to contribute a project of such  kind. It might require some light modifications to the project core, but mostly would be a thin wrapper around the existing project.

AriaNg does not provide a package / image with web server, but there are many people doing this. You can visit AriaNg third party extensions for more information : http://ariang.mayswind.net/3rd-extensions.html

Thank you for the reference to the existing extension projects.
None of the projects listed is the same as I proposed.
Presently, settings are always stored on the client. When a different client accesses the same server, the settings are different. Each client must be configured independently.
Proposed was a mode of operation in which Aria-Ng operates more as a traditional web application, with configuration stored on the server side. It requires some modification to the core application, so that it may detect which mode to use for storing configuration, based on whether the server supports operations for maintaining the configuration.
You  might keep this issue open, in case someone  would notice it, and wish to contribute by trying to implement the idea.


Bump to v1.8.1-RC2

As the title says

/hold

/lgtm

LGTM

/retest

Rerunning tests

Alright, hell with this. Setting CI:DOCS to force merge.

/hold cancel


Added dir search feature

When downloading the .html file, it downloads a folder and inside it the file of interest lies, so to automate the search of the file of interest added searching feature of the current gyft2 dir

Earlier, while saving the html page, it was essential to  open the timetable frame in a new tab, such that the timetable and only the timetable could be saved. Also, the timetable was saved with HTML only setting enabled.
Now, with your patch this limitation can be removed and thus the user has to just save the page with Web Page, Complete option enabled!
The problem however would be for us to add the rules to ignore both the html file and the folder saved.
The name of the file for me is Welcome Aditya Vikram Singh to ERP, IIT Kharagpur.htm and the folder has a similar naming scheme.
It would be awesome if you could also add the gitignore rules in a new PR.
For now this PR makes a significant improvement in the timetable generation process.
Thanks for contributing! :tada:
PS. Sorry for the late response, was busy with several things and could not test the PR soon enough!

@Raghwendra-Dey Here is the issue relating to .gitignore : #6


Please tell me ))

I didn't quite understand the meaning of step 4, namely "Enter the limits you want in limiteds.text and separate them with a comma. It should look like this: Active1, Active2, Active3" Could you please give an example of this.


Add (custom) Build Status code

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
Hi i was wondering if/how i can change the status for a build order.
reason is i have a bunch of BOM components on stock intended for a few build runs. but these runs are not yet to be ran. in stead i enter them as forcast. hence i'd like to give them the status forcast ro reserved. for me the endgoal is making sure stock is allocated but its very common to make certain agreements with customers to have a few SO or BO scheduled to give insights in pricing/qty etc. therfore it coul be helpfule to be able to manually change the status for e BO or SO or perhaps even PO to be visibla as a forcast or something.
Suggested solution
in the admin pane allow for adding custom status labels
Describe alternatives you've considered
i now change the priority order but prefere to have a more reable status
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

Thanx and agree i'm also currious as ow do other use/see this. but i think my idea could work. lets see what others think.

On that i also noticed that an BO can have more stautes already;

how do i change the status from pending to Production?? i could not find button or way to do so.

Thx for the reply.
so that info (for future reference) can be found here: https://docs.inventree.org/en/latest/build/output/?h=build+output#build-outputs
And if i undertand correct i should now gor for:

and create a new Build Output:

which brings me to the following screen

Now her it geives the omression (because of the *) that one needs to enter serial numbers but this is not the case. just press confirm and ok.
after this step the status is changed from Pending to Production.
Hope this helps others in the future
Br Martijn


Use nginx plus syntax

Replaced "dynamic_server" directive by "server" and only enable the dynamic DNS resolution if the "resolve" parameter was used.
This is the same syntax used on nginx plls.

wonderful!

Indeed. Thanks again for all your work on this module!
I've updated the README to account for this new syntax. I've also created some version tags and added a CHANGELOG.


Photo caption alignment

@tmfrnz quick question re photo captions: some are left-aligned, e.g. T2.4, M1.6, while most are right-aligned. Has the alignment been changed somewhere for these - can I change them individually? or is it a bug?

@sylvhay this is indeed a small issue for captions spanning multiple lines. While currently the caption box is right-aligned, the containing text itself is not. Will fix in next update.

moving this to https://github.com/red-list-ecosystem/typology-map/issues/43


No bold font on a light terminal theme

Steps to reproduce

open any directory in gnome-terminal with a dark theme: vim .

exit vim :q
apply any light theme to the terminal
open a directory again vim .

As you see on the screenshots the text isn't bold on a light theme. It's the same in a code with enabled syntax. There's no boldness of a text on vim with any light theme.
Expected behaviour
The text should be bold on dark and light themes at the same places. At least there should be possibility to enable it.
Operating system
Xubuntu 20.04 xfce4
Version of Vim
GNOME Terminal v3.36.2; Vim 8.1.2269
Logs and stack traces
No response

what theme/colortheme are you using?

I tested on gnome-terminal with "north" and "paper color light" themes. But I also have this problem with my custom colors on xfce4-terminal.

I tested on gnome-terminal with "north" and "paper color light" themes

Is that the dark or light colorscheme?
The thing is, I don't see bold keynames in my netrw. You can check yourself. Netrw uses the highlighting group netrwHelpCmd for those keys. Check the output of :hi netrwHelpCmd to see the definition (or where it links to).  For me it links to Function which again links to Identifier. None of those declare a bold attribute.
So I would think, this is an issue of the light colortheme that you are using.

I don't use any theme in vim.  I only change colors in the  terminal.
And I don't understand why with one color scheme the text is bold and on the other it's not.

you are apparently using a dark colorscheme, that defines the bold attribute for certain highlighting groups. Now if you switch to a light background, either the same colorscheme does not support bold for the light background, or you are simply using the default light colorscheme, which does not define bold colors.

And I don't understand why with one color scheme the text is bold and on the other it's not.

Because when &background is dark, the Identifier highlight group is given the cterm=bold attribute:
SynColor Identifier	term=underline cterm=bold ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#40ffff guibg=NONE
                                   ^--------^

source
But not when &background is light:
SynColor Identifier	term=underline cterm=NONE ctermfg=DarkCyan ctermbg=NONE gui=NONE guifg=DarkCyan guibg=NONE

source
The reason for that is given here:

" Careful with "cterm=bold", it changes the color to bright for some terminals.

As a workaround, you could try this:
vim9script
augroup BoldFontOnLightTheme
    autocmd!
    autocmd ColorScheme * BoldFontOnLightTheme()
augroup END

def BoldFontOnLightTheme()
    var hl: dict<any> = hlget('Identifier')->get(0, {})
    if hl->empty()
        return
    endif
    if hl->has_key('cterm')
        [hl->extend({cterm: hl.cterm->extend({bold: true})})]->hlset()
    else
        [hl->extend({cterm: {bold: true}})]->hlset()
    endif
enddef

This requires the patch 8.2.3578 for the hlget() and hlset() functions.

SynColor Identifier term=underline cterm=bold ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#40ffff guibg=NONE

ah, i did not see that. Thanks for correction.

I forgot there were other highlight groups which could be affected by this issue:
vim9script

augroup BoldFontOnLightTheme
    autocmd!
    autocmd ColorScheme * BoldFontOnLightTheme()
augroup END

const TO_BE_BOLD: list<string> = readfile($VIMRUNTIME .. '/syntax/syncolor.vim')
    ->filter((_, v: string): bool => v =~ '^\s*SynColor\>.*\<\%(cterm\|gui\)=bold\>')
    ->map((_, v: string) => v
    ->matchstr('SynColor\s*\zs\S\+'))
    ->sort()
    ->uniq()

def BoldFontOnLightTheme()
    if &background != 'light'
        return
    endif

    for name: string in TO_BE_BOLD
        var hl: dict<any> = hlget(name, true)->get(0, {})
        if hl->empty()
            return
        endif
        var cterm: dict<bool> = hl->get('cterm', {})->extend({bold: true})
        var newhl: dict<any> = hl->extend({cterm: cterm})
        hlset([newhl])
    endfor
enddef

Write the script somewhere and source it with :source before setting the color scheme in your vimrc.

So this is not a problem with Vim?  closing.

It seems there are still 2 inconsistencies.  Not sure they can be fixed without annoying users who are accustomed to the current attributes.
First, if cterm=bold is too unreliable (because we don't know whether the terminal will interpret it as "bold", "bright" or "bold and bright"), then it's weird to use it for Identifier on a dark background.  Even weirder, it's not used in the GUI for the same group on the same kind of background.  And it's not used on a light background, neither in the TUI, nor in the GUI.  To be consistent, maybe it should be replaced with cterm=NONE:
diff --git a/runtime/syntax/syncolor.vim b/runtime/syntax/syncolor.vim
index 5b907a3b8..326c403a3 100644
--- a/runtime/syntax/syncolor.vim
+++ b/runtime/syntax/syncolor.vim
@@ -34,7 +34,7 @@ if &background == "dark"
   SynColor Comment	term=bold cterm=NONE ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#80a0ff guibg=NONE
   SynColor Constant	term=underline cterm=NONE ctermfg=Magenta ctermbg=NONE gui=NONE guifg=#ffa0a0 guibg=NONE
   SynColor Special	term=bold cterm=NONE ctermfg=LightRed ctermbg=NONE gui=NONE guifg=Orange guibg=NONE
-  SynColor Identifier	term=underline cterm=bold ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#40ffff guibg=NONE
+  SynColor Identifier	term=underline cterm=NONE ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#40ffff guibg=NONE
   SynColor Statement	term=bold cterm=NONE ctermfg=Yellow ctermbg=NONE gui=bold guifg=#ffff60 guibg=NONE
   SynColor PreProc	term=underline cterm=NONE ctermfg=LightBlue ctermbg=NONE gui=NONE guifg=#ff80ff guibg=NONE
   SynColor Type		term=underline cterm=NONE ctermfg=LightGreen ctermbg=NONE gui=bold guifg=#60ff60 guibg=NONE

Second, for the Statement and Type groups, we currently set the bold attribute in the GUI, but not in the TUI.  That makes sense, but what if the user has set 'termguicolors' to true?  In that case, Vim will use cterm=NONE, even though gui=bold would be more consistent.  And we have the guarantee that using the bold attribute would not cause any issue, because when 'termguicolors' is set, Vim does not send sequences in which the colors are indexed in a limited palette (like CSI 30m) which the terminal can then unexpectedly map to some brighter equivalent.  Instead, the sequences describe the exact red/green/blue components of the color (e.g. CSI 38;2;95;135;135m).  There is no way for the terminal to mess up such a color, because there is no obvious mapping like there is for indexed colors between 0 and 7 (0 into 8, 1 into 9, 2 into 10 ...).
IMO, this would make a default light color scheme nicer to look at, and it would give the results that the OP expected (provided that they write set termguicolors in their vimrc):
diff --git a/runtime/syntax/syncolor.vim b/runtime/syntax/syncolor.vim
index 5b907a3b8..c2841861f 100644
--- a/runtime/syntax/syncolor.vim
+++ b/runtime/syntax/syncolor.vim
@@ -30,31 +30,43 @@ endif
 " too many colors anyway.
 " Careful with "cterm=bold", it changes the color to bright for some terminals.
 " There are two sets of defaults: for a dark and a light background.
+let s:gui = has('termguicolors') && &termguicolors || has('gui_running')
 if &background == "dark"
   SynColor Comment	term=bold cterm=NONE ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#80a0ff guibg=NONE
   SynColor Constant	term=underline cterm=NONE ctermfg=Magenta ctermbg=NONE gui=NONE guifg=#ffa0a0 guibg=NONE
   SynColor Special	term=bold cterm=NONE ctermfg=LightRed ctermbg=NONE gui=NONE guifg=Orange guibg=NONE
-  SynColor Identifier	term=underline cterm=bold ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#40ffff guibg=NONE
-  SynColor Statement	term=bold cterm=NONE ctermfg=Yellow ctermbg=NONE gui=bold guifg=#ffff60 guibg=NONE
+  SynColor Identifier	term=underline cterm=NONE ctermfg=Cyan ctermbg=NONE gui=NONE guifg=#40ffff guibg=NONE
   SynColor PreProc	term=underline cterm=NONE ctermfg=LightBlue ctermbg=NONE gui=NONE guifg=#ff80ff guibg=NONE
-  SynColor Type		term=underline cterm=NONE ctermfg=LightGreen ctermbg=NONE gui=bold guifg=#60ff60 guibg=NONE
   SynColor Underlined	term=underline cterm=underline ctermfg=LightBlue gui=underline guifg=#80a0ff
   SynColor Ignore	term=NONE cterm=NONE ctermfg=black ctermbg=NONE gui=NONE guifg=bg guibg=NONE
+  if s:gui
+    SynColor Statement	term=bold cterm=bold ctermfg=Yellow ctermbg=NONE gui=bold guifg=#ffff60 guibg=NONE
+    SynColor Type	term=underline cterm=bold ctermfg=LightGreen ctermbg=NONE gui=bold guifg=#60ff60 guibg=NONE
+  else
+    SynColor Statement	term=bold cterm=NONE ctermfg=Yellow ctermbg=NONE gui=bold guifg=#ffff60 guibg=NONE
+    SynColor Type	term=underline cterm=NONE ctermfg=LightGreen ctermbg=NONE gui=bold guifg=#60ff60 guibg=NONE
+  endif
 else
   SynColor Comment	term=bold cterm=NONE ctermfg=DarkBlue ctermbg=NONE gui=NONE guifg=Blue guibg=NONE
   SynColor Constant	term=underline cterm=NONE ctermfg=DarkRed ctermbg=NONE gui=NONE guifg=Magenta guibg=NONE
   " #6a5acd is SlateBlue
   SynColor Special	term=bold cterm=NONE ctermfg=DarkMagenta ctermbg=NONE gui=NONE guifg=#6a5acd guibg=NONE
   SynColor Identifier	term=underline cterm=NONE ctermfg=DarkCyan ctermbg=NONE gui=NONE guifg=DarkCyan guibg=NONE
-  SynColor Statement	term=bold cterm=NONE ctermfg=Brown ctermbg=NONE gui=bold guifg=Brown guibg=NONE
   " #6a0dad is Purple
   SynColor PreProc	term=underline cterm=NONE ctermfg=DarkMagenta ctermbg=NONE gui=NONE guifg=#6a0dad guibg=NONE
-  SynColor Type		term=underline cterm=NONE ctermfg=DarkGreen ctermbg=NONE gui=bold guifg=SeaGreen guibg=NONE
   SynColor Underlined	term=underline cterm=underline ctermfg=DarkMagenta gui=underline guifg=SlateBlue
   SynColor Ignore	term=NONE cterm=NONE ctermfg=white ctermbg=NONE gui=NONE guifg=bg guibg=NONE
+  if s:gui
+    SynColor Statement	term=bold cterm=bold ctermfg=Brown ctermbg=NONE gui=bold guifg=Brown guibg=NONE
+    SynColor Type	term=underline cterm=bold ctermfg=DarkGreen ctermbg=NONE gui=bold guifg=SeaGreen guibg=NONE
+  else
+    SynColor Statement	term=bold cterm=NONE ctermfg=Brown ctermbg=NONE gui=bold guifg=Brown guibg=NONE
+    SynColor Type	term=underline cterm=NONE ctermfg=DarkGreen ctermbg=NONE gui=bold guifg=SeaGreen guibg=NONE
+  endif
 endif
 SynColor Error		term=reverse cterm=NONE ctermfg=White ctermbg=Red gui=NONE guifg=White guibg=Red
 SynColor Todo		term=standout cterm=NONE ctermfg=Black ctermbg=Yellow gui=NONE guifg=Blue guibg=Yellow
+unlet! s:gui

 " Common groups that link to default highlighting.
 " You can specify other highlighting easily.

Changing the default colors is more likely to cause problems than solve them.  If someone cares about this then using a color scheme is the most obvious solution. Changing the defaults for consistency is not a good idea.

Understood.  I just post this here for anyone interested:
vim9script

augroup ColorschemeCustomizations
    autocmd!
    # Make sure  the `cterm` and  `gui` attributes of the  "preferred" highlight
    # groups are in sync.  Make sure `cterm=bold` is not used in a terminal with
    # a limited palette.
    autocmd ColorScheme * FixPreferredHL()
augroup END

 # `:help group-name`
const PREFERRED_HL: list<string> =<< trim END
    Comment
    Constant
    Identifier
    Ignore
    PreProc
    Special
    Statement
    Type
    Underlined
END
def FixPreferredHL()
    for name: string in PREFERRED_HL
        var hl: dict<any> = hlget(name, true)->get(0, {})
        var gui: dict<bool> = hl->get('gui', {})
        var cterm: dict<bool> = hl->get('cterm', {})
        # We're going to copy the value of `gui` into `cterm`.
        # But we  don't want to  copy the bold  attribute in a  terminal without
        # `'termguicolors'`, because the terminal could interpret it as "bright"
        # or "bold and bright":  https://askubuntu.com/a/875228
        if gui->get('bold') && !&termguicolors
            gui->remove('bold')
        endif
        # if `gui` and `cterm` are not in sync, sync them
        if gui != cterm
            var fixed_attr: dict<any> = hl->extend({cterm: gui})
            hlset([fixed_attr])
        endif
    endfor
enddef

It's much better than the previous code:

doesn't read any file (so faster)
make sure that cterm and gui are always in sync (so the colorscheme is always consistent)
never use cterm=bold in a terminal with a limited palette (so no surprise, even in a console)


Test on PHP 8.3

@simPod 7 tests are failing for me locally. Do you have any idea why they work in CI but not locally? I have PHP 8.3.3 installed.
There were 7 failures:

1) GraphQL\Tests\Executor\AbstractTest::testWarnsAboutOrphanedTypesWhenMissingType
Failed asserting that null is an instance of class GraphQL\Error\Error.

/home/bfranke/projects/graphql-php/tests/Executor/AbstractTest.php:631

2) GraphQL\Tests\Executor\AbstractTest::testHintsOnConflictingTypeInstancesInResolveType
Failed asserting that null is an instance of class GraphQL\Error\Error.

/home/bfranke/projects/graphql-php/tests/Executor/AbstractTest.php:764

3) GraphQL\Tests\Executor\ExecutorLazySchemaTest::testHintsOnConflictingTypeInstancesInDefinitions
Failed asserting that two arrays are identical.
--- Expected
+++ Actual
@@ @@
 Array &0 [
     0 => 'Test',
-    1 => 'Test',
 ]

/home/bfranke/projects/graphql-php/tests/Executor/ExecutorLazySchemaTest.php:182

4) GraphQL\Tests\Executor\ExecutorLazySchemaTest::testSimpleQuery
Failed asserting that two arrays are identical.
--- Expected
+++ Actual
@@ @@
 Array &0 [
     0 => 'Query.fields',
-    1 => 'SomeObject',
-    2 => 'SomeObject.fields',
+    1 => 'SomeObject.fields',
 ]

/home/bfranke/projects/graphql-php/tests/Executor/ExecutorLazySchemaTest.php:218

5) GraphQL\Tests\Executor\ExecutorLazySchemaTest::testDeepQuery
Failed asserting that two arrays are identical.
--- Expected
+++ Actual
@@ @@
 Array &0 [
     0 => 'Query.fields',
-    1 => 'SomeObject',
-    2 => 'SomeObject.fields',
+    1 => 'SomeObject.fields',
 ]

/home/bfranke/projects/graphql-php/tests/Executor/ExecutorLazySchemaTest.php:385

6) GraphQL\Tests\Executor\ExecutorLazySchemaTest::testResolveUnion
Failed asserting that two arrays are identical.
--- Expected
+++ Actual
@@ @@
 Array &0 [
     0 => 'Query.fields',
-    1 => 'OtherObject',
-    2 => 'OtherObject.fields',
-    3 => 'SomeUnion',
-    4 => 'SomeUnion.resolveType',
-    5 => 'SomeUnion.types',
-    6 => 'DeeperObject',
-    7 => 'SomeScalar',
+    1 => 'OtherObject.fields',
+    2 => 'SomeUnion.resolveType',
+    3 => 'SomeUnion.types',
 ]

/home/bfranke/projects/graphql-php/tests/Executor/ExecutorLazySchemaTest.php:446

7) GraphQL\Tests\Type\DefinitionTest::testRejectsASchemaWithSameNamedObjectsImplementingAnInterface
Failed asserting that exception with message matching "/Schema must contain unique named types but contains multiple types named "BadObject"/" is thrown

😄
For me only EnumTypeTest::testAllowsSimpleArrayAsValues fails locally but it has been failing like 2y already.

Thanks for digging in. Do you think that specifying the INI settings in composer test makes sense the way i did it?


Default Ok-type to () in anyhow::Result typedef

WARNING: This issues is a matter of style and taste! :)
I have a lot of anyhow::Result<()> in my code base. I find it ugly.
If we change the definition of anyhow::Result from
pub type Result<T, E = Error> = core::result::Result<T, E>;

to
pub type Result<T = (), E = Error> = core::result::Result<T, E>;

Then we could rewrite
fn foo() -> anyhow::Result<()> {
    …
}

to
fn foo() -> anyhow::Result {
    …
}

This would NOT be a breaking change (afaict).

I would prefer not to make this change.
You can write your own Result type alias in your crate.

Fair enough!


Enhance contributing.md

The current CONTRIBUTING.md file lacks essential information about how newcomers can contribute to our project and may not be sufficiently clear. To foster a better understanding of the contribution process and to make it more accessible to new contributors, we need to enhance the CONTRIBUTING.md file.

Hello, I am looking to work on my first issue and wanted to know if I could please work on this issue?


chore: fix some comments

Proposed Changes
fix some comments

Types of Changes
docs

Verification
no need

Testing
no need

Linked Issues
no

User-Facing Change

fix some comments

Further Comments

Please fill out the PR form completely

Please fill out the PR form completely

Modified.
Please review again.


Improve display for unsupported debug draw modes in GLES2

Don't disable the Normal option as it technically can be chosen (it's the default).
Use a text suffix in addition to the tooltip.

Suffixes were made as short as possible to avoid making the menu too wide. The menu's width is defined by the length of the longest item and the name of the longest shortcut.

This closes https://github.com/godotengine/godot/issues/58792.
Preview

Thanks!


app-route can’t work with lit-element

Due to the data binding lit-element uses, we can’t use app-route and the others iron components that require data binding.
Is there any solution or router for lit-element ?

If you really really like working with app-route, you can listen to the data change events that app-route emits manually. Everything that might have been 2-way bound in Polymer will emit an *-changed event. These are outlined here: https://www.webcomponents.org/element/PolymerElements/app-route/elements/app-route#events
Beyond that just about any routing should work great with lit-element, some people use page.js or the pwa-starter-kit is employing the router import from PWA Helpers. Hope that helps!

Here is the create-lit-app template which uses Vaadin.Router library, built by our team to fill this gap.
See https://github.com/vaadin/vaadin-router

Lots of different routing approaches should work fine. Take a look at PWA Starter Kit for a great out of the box experience for building web apps with LitElement. It uses a very simple routing approach.
As mentioned, you can listen for the events fired using lit-html (using the new lit-html syntax here that is currently released in the dev tag and will be included in version 0.6 of LitElement).
<app-route .route="${route}" @route-changed="${(e) => this._handleRoute(e)}"></app-route>

Lots of different routing approaches should work fine. Take a look at PWA Starter Kit for a great out of the box experience for building web apps with LitElement. It uses a very simple routing approach.
As mentioned, you can listen for the events fired using lit-html (using the new lit-html syntax here that is currently released in the dev tag and will be included in version 0.6 of LitElement).
<app-route .route="${route}" @route-changed="${(e) => this._handleRoute(e)}"></app-route>

Ah yes, there aren't many router libs out there that support Lit Elements (or Web Components at all), but I've started one over at https://github.com/mkay581/router-component. It's pretty lightweight and supports a more html declarative syntax.

PWA Starter kit routing approach is great! I manage to put all this in a route element here: https://github.com/fernandopasik/lit-redux-router
Feedback and contributions are very much welcomed!!

There's some nice vanilla router like pagejs : https://visionmedia.github.io/page.js/
it works fine with LitElement

I would suggest https://www.npmjs.com/package/lit-element-router

I tried to use app-location/app-route with event handler but as we set the route property with ".route="${route}"" (so in the render method), changes made by the "@route-changed" handler is not  reflected. Indeed, event is sent, caught and processed after the render method but before the updated one ! And changes made during this interval are not rendered as noticed in render and update method documentation "Setting properties inside this method will not trigger the element to update." :(
But, good news, I tried the https://github.com/vaadin/vaadin-router mentioned above and it seems to work very well :) !


System hangs when using Llamacpp as LLM

The following code appears to load the llamacpp model properly, but it just ramps up the CPU load and hangs for hours if allowed.
If service_context=service_context is removed from GPTSimpleVectorIndex.from_documents() then it uses OpenAI's api and works fine. What step is missing here to run llama locally? It outputs all the debug text like loading llamacpp normally does, so it is loading it.
llm_predictor = LLMPredictor(llm = LlamaCpp(model_path="~/Code/llama.cpp/models/30B/ggml-model-q4_0.bin", n_threads=10))
service_context = ServiceContext.from_defaults(llm_predictor=llm_predictor)

ObsidianReader = download_loader('ObsidianReader')
documents = ObsidianReader('~/Documents/Obsidian').load_data()

index = GPTSimpleVectorIndex.from_documents(documents, service_context=service_context)

print(index.query("Any query here"))

@MLTQ did you try with full paths? Please paste full errors.

Stale issue, closing for now, please feel free to re-open if this continues to be an issue.


Default collation and char set

Do you have a way to set the default collation and character set like this role:
https://github.com/ANXS/mysql
- name: foobar
    collation: "utf8_general_ci"        # optional, defaults to "utf8_general_ci"
    encoding: "utf8"                    # optional, defaults to "utf8"

As far as I can tell, Redis stores binary data and doesn't seem to care about collation or encoding. Here's a similar issue filed against Redis itself, which was closed. I'm going to do the same with this issue, but if there's a particular problem I can help solve, I can re-open it.


Create "support" package modules

CC @jzrapp (perhaps link to the Cryo spreadsheet you've developed?)
For parameter groups that are repeated across multiple environmental packages (e.g. sample logistics)
These support packages (or parts of them) would be sourced by the environmental packages that need them, perhaps driven by an import file that would pull in either single parameters or the whole package.
Copy of GenomicsStandardsConsortium/mixs-ng#9 originally filed by @pbuttigieg.

This issue is no longer so relevant, now that we have checklists and extensions as modules in MIxS 6.2 and all terms are in a single list (so it is easier to create modules). INSDC should be able to create any modules they need from the newest release.


Roll Engine from 825c409164c1 to 2d91e19928b5 (1 revision)

https://github.com/flutter/engine/compare/825c409164c1...2d91e19928b5
2021-09-10<EMAIL_ADDRESS>Roll Skia from f17e4008a89b to 9292c806e97d (1 revision) (flutter/engine#28544)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Include types to most upstream modules

This PR adds types to the following files, as per https://github.com/jxnl/instructor/issues/390.
instructor/cli/files.py
instructor/cli/usage.py
instructor/exceptions.py

This ellipsis thing is amazing


fix anchor link in network-setup.md

NB: GH is showing me a second change at line 181 -- no idea what that is about, maybe just a fluke. The only change made here conciously is the one on line 31.

Merged in dbc081913ec3d0dc3d0d5f08d00df7e841fd6ad5, thanks.


New-BCContainer and New-NavContainer build fails with -shortcuts Desktop

I am creating a docker vm with BC14 - if I change shortcuts from None to Desktop, it fails every time.
I am creating an upgrade path which is why I need C/AL with BC14 so I can use the new extension upgrade procedures.
(How do I bring up the C/Side dev client without shortcuts?)
New-BCContainer @parameters `
                -accept_eula `
                -containerName $ContainerName `
                -artifactUrl $artifactURL `
                -auth NAVUserPassword `
                -Credential $credential `
                -updateHosts `
                -assignPremiumPlan `
                -**shortcuts None** `
                -licenseFile $ContainerLicFile `
                -EnableTaskSchedule:$false `
                -memoryLimit 8G `
                -includeAL `
                -alwaysPull `
                -includeCSIDE `
                -isolation hyperv

NavContainerHelper is version <IP_ADDRESS>
NavContainerHelper is running as administrator
Host is Microsoft Windows 10 Pro Build 19041
Docker Client Version is 19.03.12
Docker Server Version is
...

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context

does it happen all the time?
did it use to work?

Using license file C:\Docker\dockercontainer-master\dockercontainer-master.\4829960.flf
Additional Parameters:
--env customNavSettings=EnableTaskScheduler=False
Files in C:\ProgramData\NavContainerHelper\Extensions\V14Base\my:
AdditionalOutput.ps1
license.flf
MainLoop.ps1
SetupNavUsers.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container V14Base from image mcr.microsoft.com/dynamicsnav:10.0.19041.450-generic
9674d7f89a2878bc105e4daf032cd861c2163b1ca053a2e71d7dd4a6ed569308
Docker : Error response from daemon: container 9674d7f89a2878bc105e4daf032cd861c2163b1ca053a2e71d7dd4a6ed569308: driver
"windowsfilter" failed to remove root filesystem: failed to detach VHD: The device is not ready.: rename
C:\ProgramData\Docker\windowsfilter\9674d7f89a2878bc105e4daf032cd861c2163b1ca053a2e71d7dd4a6ed569308
C:\ProgramData\Docker\windowsfilter\9674d7f89a2878bc105e4daf032cd861c2163b1ca053a2e71d7dd4a6ed569308-removing: Access is denied.
At C:\Users\dmachanick\Documents\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\HelperFunctions.ps1:124 char:17

            Docker rm $out -f

            ~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (Error response ...cess is denied.:String) [], RemoteException
FullyQualifiedErrorId : NativeCommandError

The error you are seeing is a docker problem. You probably have to reset your docker to factory defaults (which removes all containers and removes all images) and then start from scratch.
On how to access C/SIDE without shortcuts - I would create a container with shortcuts and see the path of the shortcuts - then you can see which program to launch. (I cannot remember the exact path)

yes - if shortcuts are set to be created, it gets the error.
It is a corporate controlled desktop.
Running Symantec endpoint protection

I have been able to execute finsql by navigating to the path externally

and you are running PowerShell elevated (as administrator)?
have you tried the other options for shortcuts (e.g. startmenu)?

yes - running as administrator
did not try startmenu yet
are there any additional options for shortcuts?

when you press space after writing -shortcuts you will see the options.

running it now - thanks for the tip - I will have to get used to the space feature

Startmenu worked - maybe it was a permission issue with the desktop.
I have noticed every file I download is blocked and I have to unblock it.
Thanks.