Datamachine exception

While trying to use the Datamachine on one of the recent versions of Wikipedia (1st March 2017), I get the following error :-

"2017.03.13 17:20:53","1552941056","1398379112","org.xml.sax.SAXParseException; lineNumber: 64370855; columnNumber: 559; JAXP00010004: The accumulated size of entities is "50,000,001" that exceeded the "50,000,000" limit set by "FEATURE_SECURE_PROCESSING".
de.tudarmstadt.ukp.wikipedia.wikimachine.dump.xml.AbstractXmlDumpReader.readDump(AbstractXmlDumpReader.java:209)
de.tudarmstadt.ukp.wikipedia.datamachine.dump.xml.XML2Binary.(XML2Binary.java:47)
de.tudarmstadt.ukp.wikipedia.datamachine.domain.DataMachineGenerator.processInputDump(DataMachineGenerator.java:70)
de.tudarmstadt.ukp.wikipedia.datamachine.domain.DataMachineGenerator.start(DataMachineGenerator.java:64)
de.tudarmstadt.ukp.wikipedia.datamachine.domain.JWPLDataMachine.main(JWPLDataMachine.java:64)"

https://github.com/dkpro/dkpro-jwpl/issues/139

Thanks! Will try out the proposed solution and will close the issue.

For the record, that's how the original author of the issue addressed this problem :- https://groups.google.com/forum/#!msg/jwpl/EhuqKpF2jag/jJO153TdBAAJ

I am not sure why that has not been done already, I will look into it

The solution suggested in #139 works; refer to https://groups.google.com/forum/#!msg/jwpl/EhuqKpF2jag/jJO153TdBAAJ if unclear.


No bean named 'Microsoft.Targeting' available when using Feature Flag with targeting filter

Following closely the documentation here, I've set up a minimal spring sample application, where I'm using feature flags. Simple feature flags which toggle on/off can be loaded by the application as expected.
But if a feature flag has a filter criteria, I'm getting an exception:

org.springframework.beans.factory.NoSuchBeanDefinitionException: No bean named 'Microsoft.Targeting' available

The following value was generated via App Configuration in Azure Portal
{
	"id": "ff-sometimes-on",
	"description": "",
	"enabled": true,
	"conditions": {
		"client_filters": [
			{
				"name": "Microsoft.Targeting",
				"parameters": {
					"Audience": {
						"Users": [],
						"Groups": [],
						"DefaultRolloutPercentage": 50
					}
				}
			}
		]
	}
}

In the application, I call the endpoint
	@GetMapping("/hello")
	@FeatureGate(feature = "ff-sometimes-on")
	public String hello() {
		return "hello";
	}

The exception is raised everytime this endpoint is called responding Internal Server Error. The error originates from com.azure.spring.cloud.feature.manager.FeatureManager::isFeatureOn, even though during the application's startup phase, this class correctly loads its hashmap values and initializes the expected feature flag object under the expected key ("ff-sometimes-on").
As documented I've added maven dependency

azure-spring-cloud-feature-management-web

aswell as

spring-boot-starter-web
azure-spring-cloud-appconfiguration-config-web
spring-boot-starter-thymeleaf

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

ID: 7af9b1df-7ce6-465b-bb32-f1cc91039085
Version Independent ID: a59d2917-6fe5-eb90-8222-f6f23933db56
Content: Tutorial for using feature flags in a Spring Boot app - Azure App Configuration
Content Source: articles/azure-app-configuration/use-feature-flags-spring-boot.md
Service: azure-app-configuration
GitHub Login: @mrm9084
Microsoft Alias: mametcal

@ts-62020 it looks like you need to follow https://github.com/Azure/azure-sdk-for-java/tree/main/sdk/appconfiguration/azure-spring-cloud-feature-management#targetingfilter. And name the TargetingFilter Bean Microsoft.TargetingFilter.
The name of the bean matches the name field.

Thanks, so I guess the missing piece was

TargetingFilter relies on another @Bean to be added to the application, ITargetingContextAccessor. The ITargetingContextAccessor allows for defining the current TargetingContext to be used for defining the current user id and groups

Sorry, files were moved https://github.com/Azure/azure-sdk-for-java/tree/main/sdk/spring/spring-cloud-azure-feature-management#implementing-a-feature-filter. The issue you are having is that the filters aren't created by default and need to be provided.


fixed/Eslint issue

This pull request introduces Fixing the issue related to ESlint violations such as missing explicit any  and  unused vars etc . i already resolved these issue.
Key highlights include:
Resolved all the issues related ESLint . first please check the code and then merge
the issue had resolved on following github branch
https://github.com/Ankuristic/html-reporter/tree/fixing_eslint_issue.
Resolve #44

Resolve Eslint  issue of #44


OOP Library

This isn't currently done, and i'm more opening this as a way to go "Hey, this exists."
This pull request basically adds a library that abstracts Raylib into a more Vala-like experience. The reason I went down the library route instead of doing it directly through the vapi was because I quickly realized it would make the raylib vapi a massive 10,000+ line unmaintainable monstrosity. This also allows the ability to keep the more c-like API alongside the OOP API.
This also has an unattended consequence of making it work on any language that supports GObject Introspection.
Another thing I did was add Raylib as a subproject, so people could use this binding as a subproject for their own projects and have the full stack.
The Util namespace just contains functions that I made to help me out, and thought it'd be useful in general.
The code is also documented, so using valadoc will create pretty good documentation.
Now, I don't think i'll be able to get around to doing the 3D stuff, as I don't really do anything with 3D.
With a bunnymark score of around 62,000 before it dips below 60fps. I'd say it performs pretty well, as well!

Hey!
This looks awesome and I agree going down the library path is best, main thing that prevented me from working on a VAPI for it was; as you said, how massive it would end up!
Let me know when you think everything 2D wise is sorted and I have no issues with merging this across :-)

Wouldn't be a bad idea, might help it get out there more!
Sorry I haven't updated the VAPI for 5.0 yet, been away from Vala for a while and working in straight C, had completely slipped my mind!

Wouldn't be a bad idea, might help it get out there more!
Sorry I haven't updated the VAPI for 5.0 yet, been away from Vala for a while and working in straight C, had completely slipped my mind!

Ha, no worries. I've actually been away from Vala quite a bit too. Been f'ing around with objects in C directly. Surprisingly works well.

Oh yeah, forgot to bring this up. But you fine with me removing rini? GLib already has a built-in object called GLib.KeyFile which also handles ini files.
https://valadoc.org/glib-2.0/GLib.KeyFile.html

That should be fine :-)
Ngl I am not sure why I added it as I never actually tested whether it worked

So, I added a docker compose file to make testing on 3 major types of distros (stable, rolling and bleeding edge) easier. I actually caught some compilation issues this way on Almalinux 8, so it should work on any distro made past 2018, and it compiles on bleeding edge (Fedora Rawhide), so no future incoming compatibility issues =)

Hey I want a 2nd opinion on this. Should the cursor functions be under the Window object, or the Input.Mouse object? I'm conflicted since it's a graphical thing rather than a input related thing.
Refering to these functions:
void ShowCursor(void);
void HideCursor(void);
bool IsCursorHidden(void);
void EnableCursor(void);
void DisableCursor(void);
bool IsCursorOnScreen(void);

For those functions specifically I would say the window object would be better considering they are more directly tied to the windows functionality, might be a bit strange to have it out of the 'mouse' object but think it makes more sense logically.

I'm also gonna skip the VR stuff. Because I'm poor, and will probably never have access to a VR headset to test this crap lol

VAPI has been updated to match Raylib 5.0, changes shouldn't break anything.

So, if you've noticed a bit of a decrease in activity in my repo. Blame Helldivers lol
But I'll get it done! Eventually...

110% understandable, the star wars battlefront collection is about to remove all free time I have so I feel that!

Aw yeah, back in action! Gonna start re-creating the examples from Raylib upstream as that'll be a good way to indicate what needs to be implemented.
As a sidenote, I love that Raylib is API stable, so I don't feel rushed at all to accomplish this =D
Like, 6.0 could come out tomorrow, and i'd probably have to do minimal effort to get it working with it.

Found out if I want to make all the examples, I gotta add raymath to the vapi file, and then OOP it 😅

Ah yes!
I was working on a raymath VAPI a while ago but couldn't work out an easy way to make it 'portable' and usable without the raylib VAPI itself

Ah yes!
I was working on a raymath VAPI a while ago but couldn't work out an easy way to make it 'portable' and usable without the raylib VAPI itself

Interestingly, a lot of the float math functions are already methods in vala's float object (Such as clamp). I might ask around to see if I could somehow extend the float object to have the other functions such as lerp

changing the parameters to use ints instead of enum types. Since enums in vala are not really the same kind of enum in C, and it causes issues.

Hi, I was wondering what your specific issue here was?

changing the parameters to use ints instead of enum types. Since enums in vala are not really the same kind of enum in C, and it causes issues.

Hi, I was wondering what your specific issue here was?

If I remember correctly, I was getting type errors at compilation.

ah I see. If the errors are still there it might be fixable with attributes.

Just a heads up I will be updating the base VAPI to raylib 5.5 in the coming days, will try not to touch anything else to avoid further conflicts! 😄


[@angular/cli] input decorator is removed when using the AoT build

The resolve data doesn't supplies component inputs anymore in AoT build since the last version of @angular/cli as it remove all angular decorators in order to have less bundle size.
I propose to introduce custom decorator for resolve data:
@Component({ ... })
export class App {
    @Resolve() foo;
}

https://github.com/angular/angular-cli/pull/4301

Great idea! I'm facing the same problem on my project. However I gave your Idea a try.
See my commit: Resolve Decorator
But I got stuck. The AoT compiler prints the annoying error:

ERROR in Error encountered resolving symbol values statically. Calling function 'makePropDecorator', function calls are not supported.

I don't know how to get rid of this. I tried to export and wrap the function call, without any success.
@aitboudad I'd appreciate if you could have a look on it.

I really want to use @Input() because it allows components to be agnostic towards being routed or not routed.

another approach is to pass the inputs through state config:
    {
        name: 'app',
        component: App,
        resolve: [
            {
-               token: 'app',
+               input: 'app',
                deps: ...
            },
        ],
    },

another approach is to pass the inputs through state config:

That would not allow resolves declared on a parent state to be bound to a (routed) child component.  It would have to be mapped on the view level.  We already have the bindings: property on a view declaration which serves this purpose.  However, making it mandatory would introduce a lot of boilerplate.
    {
        name: 'app',
        component: App,
+        bindings: { app: 'app' },
        resolve: [
            {
               token: 'app',
                deps: ...
            },
        ],
    },

AOT already introduced a lot of boilerplate :) so I'm ok with the bindings: property as I think the bundle size is important too (in my case removing decorators shrunk the bundle size by ~170 Kb)

I'd like to point out that in 1.0.0-beta.4 the only workaround is to inject the resolves into the component constructor:
@Component({
})
export class MyComponent {
  constructor(@Inject('myResolve') public resolveData: any) {
  }
}

since bindings is implemented what do you think about adding a new option autoBindings through the config part which will pass all resolved values into the component if the property have the same name of resolved value.
UIRouterModule.forRoot({ autoBindings: true })

I've been considering a similar thing, but how can you tell if a newly constructed component has a property or not?
class FooComponent {
   someInput: string;
}

it seems not possible :(

@aitboudad  For now I'm going to use your original idea of a Decorator like @ResolveData() resolveInput
I don't like not using @Input(), but this seems like a simple way to manage until Angular comes up with a solution for exposing metadata to for third parties, when we will remove @ResolveData and have users switch back to @Input()

It should be solved soon see https://github.com/angular/angular/pull/15214/commits/bd64ab5925e874ccc2a1ef80f08c6c28158e104d

@aitboudad that's very promising!

I am going to remove the ResolveData decorator before releasing beta.5 which will support angular 2.x  I will then release beta.6 using the inputs as exposed in that commit.


Example using this with a webpack

Trying this to integrate this in webpack3. Is it posible?
Could you please provide an example

            {
                test: /\.s?css$/,
                use: extractSass.extract({
                    use: [
                        {
                            loader: 'css-loader',
                            options: {
                                module: true,
                                sourceMap: true,
                                importLoaders: 1
                            }
                        },
                        {
                            loader: 'postcss-loader',
                            options: {
                                sourceMap: true,
                                plugins() {
                                    return [
                                        esCssModules({
                                            jsFiles: 'src/index.js',
                                        }),
                                        autoprefixer
                                    ];
                                }
                            }
                        },
                        {loader: 'resolve-url-loader'},
                        {loader: 'sass-loader', options: {sourceMap: true}}
                    ],
                    fallback: 'style-loader'
                })
            },

Getting error Module build failed: TypeError: esCssModules is not a function

If I try to load module like const esCssModules = require('es-css-modules').default;
then I get warning
Unknown error from PostCSS plugin. Your current PostCSS version is 6.0.9, but es-css-modules uses 5.2.17. Perhaps this is the source of the error below.

then some errors in build process like
ERROR in ./assets/scss/main.scss
Module build failed: ModuleBuildError: Module build failed: TypeError: Cannot read property 'Symbol(Symbol.iterator)' of undefined

in main.scss nothing special :)
@import "~bootstrap-sass/assets/stylesheets/bootstrap";

P.S. https://github.com/jacobp100/es-css-modules/blob/master/rollup.config.js#L5
according documentation

For Node.js:
compile to a CommonJS module
$ rollup main.js --format cjs --output bundle.js

Related? https://github.com/babel/babel/issues/5893

I'm not sure this can be run with webpack to be honest, but feel free to try and prove me wrong! It gets its own dependency graph, and generates files that are used by webpack. It was really intended to be run before webpack. Hope this helps!


Time of Release of Feed is EARLIER Than Actual Time

In the main Vienna RSS window, the time of the release of a particular feed entry is EARLIER than the actual current time.
A small problem but an important detail which should be working differently. It does not always happen and i am unclear why and when it does and doesn't.
Screenshots
See screenshot and look at dates of the first few entries and compare to the time on the bar at the top right of the

screen.

Vienna version 3.8.7 :15608906: (8038)
macOS Ventura 13.5.1

Thanks Eitot.
Those are some good things to start observing and noting. I appreciate it. I can help solve it having this bit of info.

Thanks for continuing to look into this Eitot. I will keep this in mind about the wrong times when looking at Freedom's Phoenix feed.
I appreciate your perseverence, continuity and attention to detail.
Louis
-------- Original Message --------
On Oct 21, 2023, 07:00, Eitot wrote:

I have observed the feed you are using and I found the problem. The feed itself is using the wrong dates.
Screenshot 2023-10-21 at 12 55 26
When I check the source for those entries that are dated in the future, I find e.g. this line:
<
pubDate

Sat, 21 Oct 2023 13:00:00 GMT</

pubDate

At the time of writing this, the current time in GMT is Sat, 21 Oct 2023 11:00:00 GMT.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


libnmstate: expose the 'state_match()' for different states comparison

The network role needs such a api for determining if the network state
changes between the desired state and the current state.
Signed-off-by: Wen Liang<EMAIL_ADDRESS>

nmstate-0.3 branch is no maintained anymore. Please patch 1.3 branch and base branch.

nmstate-0.3 branch is no maintained anymore. Please patch 1.3 branch and base branch.

Thanks.


feature gate ipc reader/writer

Which issue does this PR close?
This closes #310. This was as easy as bumping #[cfg(feature = "ipc")] on the module. This make flatbuffers an optional dependency. To reduce migration friction, the feature ipc was added to the default features.

Codecov Report

Merging #336 (3cabcbb) into master (71c2159) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #336   +/-   ##
=======================================
  Coverage   82.53%   82.53%
=======================================
  Files         162      162
  Lines       44043    44043
=======================================
  Hits        36350    36350
  Misses       7693     7693

Continue to review full report at Codecov.

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

(note, the failure is unrelated; it seems that rust stopped being available on windows, still investigating why)


README has not been updated with recent changes, review and update

Write the wiki entry, then this will allow a README review. Address any shortcomings

Add some more badges also
https://www.tidyverse.org/lifecycle/#experimental


Multiple mysql connection crash on Heroku

When I try to make multiple MySQL Connection on Heroku it crash

extension Droplet {

    func setupRoutes() throws {
        var mysql = MySQL.Database.makeTest()

        get("hello") { req in
            var json = JSON()
            try json.set("hello", "world")
            return json
        }

        get("test") { req in

            let conn = try mysql.makeConnection()

            try conn.execute("DROP TABLE IF EXISTS spam")
            try conn.execute("CREATE TABLE spam (s VARCHAR(64), time TIME)")

            for _ in 0..<10_000 {
                try conn.execute("INSERT INTO spam VALUES (?, ?)", ["hello", "13:42"])
            }

            let conn2 = try mysql.makeConnection() //HEROKU CRASH HERE
            try conn2.execute("SELECT * FROM spam")
            return req.description

        }

        // response to requests to /info domain
        // with a description of the request
        get("info") { req in
            return req.description
        }

        get("description") { req in return req.description }

        try resource("posts", PostController.self)
    }
}

extension MySQL.Database {
    static func makeTest() -> MySQL.Database {
        do {
            let mysql = try MySQL.Database(
                hostname: "****",
                user: "*****",
                password: "******",
                database: "*****"
            )
            let connection = try mysql.makeConnection()
            try connection.execute("SELECT @@version")

            return mysql
        } catch {

            fatalError("MySQL ERROR")
        }
    }
}

I can't reproduce this. Sure, Heroku returns an Application Error – a request timeout as inserting 10000 rows in separate transactions takes more than 30 seconds –, but no crash whatsoever.

Here is the code you can reproduce
test.zip

I still don't get a crash, but serverLost exceptions.
Regardless, your code is incorrect.
MySQL connections are not thread-safe, so creating one connection during route setup and trying to reuse it inside the route handlers will not work. By design.

If i dont reuse the connection it will use up all the mysql connection i have only 10 connection Any solution on this?


Quickguide for beta release?

Hi, I like to try the beta release, v2.6.0b1.
Could someone guide a recommended way to install it on my Unity project?
I can't find documentation for that.
Thanks in advance.

There is a quickstart guide in the readme. Information on getting the examples set up with screenshots is found at https://valvesoftware.github.io/steamvr_unity_plugin/articles/Quickstart.html .

@keymastar Hi, Thanks for your answer. But I was wondering how to use the beta release. Can I just copy/paste files from this repo to my folder after installing 2.5.0 version?


[Bug]: evt.target.validity.tooShort not working as expected.

Version
latest version in my projects but not in create-react-app (example repo)
Steps to reproduce

Clone my repo at https://github.com/007tom/jest-tooShort
yarn install
yarn test
You should see the error come up

Expected behavior
I expected to see evt.target.validity.tooShort logs true in terminal when run tests.
Actual behavior
evt.target.validity.tooShort logs false.
Additional context
Whatever I use (userEvent or fireEvent), it's the same result, but not in the browser (run yarn start).
Environment
System:
    OS: macOS 11.6
    CPU: (4) x64 Intel(R) Core(TM) i5-4258U CPU @ 2.40GHz
  Binaries:
    Node: 16.10.0 - /usr/local/bin/node
    Yarn: 1.22.15 - /usr/local/bin/yarn
    npm: 7.24.0 - /usr/local/bin/npm

Hi! Thanks for taking the time to open up an issue. However, this is an issue for JSDOM - which is the implementation of the DOM your get by using Jest.
https://github.com/jsdom/jsdom

@SimenB
Yes, you're right!
jsdom has not implemented tooShort for now:
https://github.com/jsdom/jsdom/blob/0778616c35b698e72a125993545576f00a8e7155/lib/jsdom/living/nodes/HTMLInputElement-impl.js#L1001-L1002
and major form library like react-hook-form implement it using setCustomValidity: https://github.com/react-hook-form/react-hook-form/blob/ab106ab57f4ee049a4fec89bb83fb8deb1d34e31/src/logic/validateField.ts#L165
so it can be handled by jsdom: https://github.com/jsdom/jsdom/blob/0778616c35b698e72a125993545576f00a8e7155/lib/jsdom/living/constraint-validation/DefaultConstraintValidation-impl.js#L36-L38.
Thank you!


Stylistic changes to improve repository navigation

This pull request does some stylistic changes to improve repository navigation
TODO would be to make the individual readme's link back to the main readme.

I prefer a link to the blog post in the main README.
But it was a really good idea to have mini-readmes in the project folders!
Thank you so much!


Next.js 13 - Builds not outputting css on Windows

Describe the bug

Clone the "should be working repo" in Windows

Inspect with DevTools, and see footer class being generated without styles

Need help @markdalgleish @SuttonJack
Reference in discussion
Edit:
Running the app inside container seems fine.
Reproduction
https://github.com/SuttonJack/vanilla-extract-app-dir
System Info
Windows 11

Used Package Manager
pnpm
Logs
No response
Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

I'm having the same issue here on Windows 10

I'm still in the process of setting up a windows dev environment, but I'll take a look at it when I can.

Also happening in Linux (Ubuntu).

Also happening in Linux (Ubuntu).

It works fine for me, on MacOS though, but I wouldn't expect it to behave very differently between ubuntu and macos.
Also general update, I'm reproduce the issue on windows. It's odd that classnames are in the markup, but there are no CSS files in the document head. This seems like an issue with Next, but it might not be. We've made a discussion in the next repo about Vanilla Extract support, pointing to various next bugs https://github.com/vercel/next.js/discussions/49892.

Same here on Windows 11, but not be reproduced on Ubuntu (WSL).

Same here on Windows 10 (19045.3208), Node.js 19.2.0, latest versions of @vanilla-extract/css/next/@vanilla-extract/next-plugin.

Same here.
reproduction: https://github.com/Sharlottes/ve-missing-style-reproduction
interestingly, the deployment on the Vercel works well.
https://ve-missing-style-reproduction.vercel.app/

still doing some debugging, but it's server components specifically that are failing, adding a 'use client' directive results in stylesheets being generated.

still doing some debugging, but it's server components specifically that are failing, adding a 'use client' directive results in stylesheets being generated.

This works for components, what about global styles, any workaround?

still doing some debugging, but it's server components specifically that are failing, adding a 'use client' directive results in stylesheets being generated.

This works for components, what about global styles, any workaround?

I did find a workaround, but ended not needing it, as it's a bit of an inelegant hack, but if you use a wildcard import and ensure you use the resulting variable in some capacity, it should pickup the globalStyles
import * as styles from './global.css';

console.log(styles)

still doing some debugging, but it's server components specifically that are failing, adding a 'use client' directive results in stylesheets being generated.

This works for components, what about global styles, any workaround?

I did find a workaround, but ended not needing it, as it's a bit of an inelegant hack, but if you use a wildcard import and ensure you use the resulting variable in some capacity, it should pickup the globalStyles
import * as styles from './global.css';

console.log(styles)

Uh, really ugly, but it works, thank you!

I've recently been migrating my blog from Gatsby to Next.js and I'm having the same problem. I found a solution at style9-webpack and made another Next.js plugin for vanilla-extract that works fine on Windows/Linux/macOS, if you're having a similar problem, you can try @syfxlin/next-plugin-vanilla-extract.

I've recently been migrating my blog from Gatsby to Next.js and I'm having the same problem. I found a solution at style9-webpack (SukkaW/style9-webpack#1) and made another Next.js plugin for vanilla-extract that works fine on Windows/Linux/macOS, if you're having a similar problem, you can try @syfxlin/next-plugin-vanilla-extract.

Is this something that can be contributed here? I'm sure many of us would prefer not to see fragmentation, especially as vanilla-extract and or next continue to evolve

Hi @GeeWizWow, @syfxlin/next-plugin-vanilla-extract hasn't been fully tested, for example I'm not sure if it works in the Next.js Pages Directory yet. Once it's confirmed to be reliable, I can contribute here, thanks!

I temporarily gave up on the next-plugin and wrote the webpack transformer myself with webpack-plugin.
It works now, just for reference, don not use it in production：
// 2023/10/03; windows-11; next^13.5.3; webpack-plugin^2.3.1; css-loader^6.8.1

const { VanillaExtractPlugin } = require('@vanilla-extract/webpack-plugin/next');
const { default: MiniCssExtractPlugin } = require('next/dist/build/webpack/plugins/mini-css-extract-plugin');

/** @type {import('next').NextConfig} */
const nextConfig = {
  experimental: {
    serverActions: true,
  },
  webpack(config) {
    config.plugins.unshift(new MiniCssExtractPlugin());
    config.plugins.unshift(new VanillaExtractPlugin());

    config.module.rules.unshift({
      test: /vanilla\.virtual\.css$/,
      use: [
        MiniCssExtractPlugin.loader,
        require.resolve('css-loader'),
        VanillaExtractPlugin.loader,
      ],
    });
    return config;
  },
};

module.exports = nextConfig;

I gave up next-plugin and instead write webpack transformer with webpack-plugin directly. It works for now, just for reference, do not use this in production:
// 2023/10/03; windows 11<EMAIL_ADDRESS><EMAIL_ADDRESS>
const { VanillaExtractPlugin } = require('@vanilla-extract/webpack-plugin/next');
const { default: MiniCssExtractPlugin } = require('next/dist/build/webpack/plugins/mini-css-extract-plugin');

/** @type {import('next').NextConfig} */
const nextConfig = {
  webpack(config) {
    config.plugins.unshift(new MiniCssExtractPlugin());
    config.plugins.unshift(new VanillaExtractPlugin());

    config.module.rules.unshift({
      test: /vanilla\.virtual\.css$/,
      use: [
        MiniCssExtractPlugin.loader,
        require.resolve('css-loader'),
        VanillaExtractPlugin.loader,
      ],
    });
    return config;
  },
};

module.exports = nextConfig;

Does the fix to the next plugin from https://github.com/vanilla-extract-css/vanilla-extract/pull/1180 work for you?

https://github.com/plrs9816/vanilla-extract-app-dir
Something broke again in next 13.5?
In 13.4 it works fine but the css is not generated in 13.5. (I only tested in windows)
Strangely the above demo is working, because I added unnecessary appDir: true config.
⚠ App router is available by default now, experimental.appDir option can be safely removed.
It says the option does nothing, but if it's removed it doesn't work.
13.4 works fine without the option.
const nextConfig = {
  experimental: {
    appDir: true,
  },
};

@plrs9816 This should hopefully be fixed by https://github.com/vanilla-extract-css/vanilla-extract/pull/1193


/mod remainder is always a fixnum

This should optimize the fixnum check to t:
IN: scratchpad [ { integer fixnum } declare /mod fixnum? ] optimized.
[ /mod tag 0 eq? ]

Maybe related, this works:
[ { fixnum fixnum } declare fixnum/mod fixnum? ] optimized.
[ fixnum/i \ t ]

This does not:
[ fixnum/mod fixnum? ] optimized.
[ fixnum/mod tag 0 eq? ]

The declarations shouldn't be necessary.

Hm it appears that optimization isn't safe because the return values are automatically promoted to bignums if one of the arguments is a big num:
33 90 2^ /mod [ bignum? ] both?
t
90 2^ 33 /mod [ bignum? ] both?
t

I think #224 is related.

I think the interval-mod word is bugged:
fixnum-interval fixnum-interval interval-mod fixnum-interval interval-subset? .
f

This means (I think) that Factor thinks that modulus of two fixnum values can overflow into a bignum.

Hmm, weird, it works for simple interval:
IN: scratchpad T{ interval { from { -10 t } } { to { 10 t } } }
              [ dup interval-mod ] [ interval-subset? ] bi .
t

But fails for fixnum-interval...
IN: scratchpad fixnum-interval
              [ dup interval-mod ] [ interval-subset? ] bi .
f

This seems the problem?
IN: scratchpad T{ interval { from { -10 t } } { to { 10 t } } } interval-abs .
T{ interval { from { 0 t } } { to { 10 t } } }

IN: scratchpad fixnum-interval .
T{ interval
    { from { -576460752303423488 t } }
    { to {<PHONE_NUMBER>03423487 t } }
}

IN: scratchpad fixnum-interval interval-abs .
T{ interval { from { 0 t } } { to {<PHONE_NUMBER>03423488 t } } }

IN: scratchpad first-bignum .
576460752303423488

In the second case, calling interval-abs seems to grow the interval to include the first-bignum.

Hmm, I think thats not a problem, but the way interval-mod uses interval-abs is probably the problem.

Some test cases?
IN: scratchpad T{ interval { from { -3 t } } { to { 2 t } } }
[ dup interval-mod ] [ interval-subset? ] bi .
f

IN: scratchpad T{ interval { from { -3 t } } { to { 2 t } } }
[ dup interval-mod ] [ swap interval-subset? ] bi .
f

So, just out of curiosity, I tried mod on a range [-5,2]:
IN: scratchpad -5 2 [a,b] dup
               [ [ mod ] [ 3drop f ] recover ] cartesian-map
               concat members  .
{ 0 -1 -2 f -4 -3 1 2 }

And looked at what interval-mod returns:
IN: scratchpad -5 -2 [a,b] dup interval-mod .
T{ interval { from { -5 f } } { to { 5 f } } }

So wrong!

IN: scratchpad -5 2 [a,b] -9 10 [a,b]
              [ [ mod ] [ 3drop f ] recover ] cartesian-map
              concat sift members natural-sort .
{ -5 -4 -3 -2 -1 0 1 2 }

IN: scratchpad -9 10 [a,b] -5 2 [a,b]
              [ [ mod ] [ 3drop f ] recover ] cartesian-map
              concat sift members natural-sort .
{ -4 -3 -2 -1 0 1 2 3 4 }

For the previous examples:
IN: scratchpad -5 2 [a,b] -9 10 [a,b] interval-mod .
T{ interval { from { -10 f } } { to { 10 f } } }

IN: scratchpad -9 10 [a,b] -5 2 [a,b] interval-mod .
 So something -5.0 mod can return values in the interval (-5,5).

Maybe this isn't a bug with interval-mod and has more to do with compiler.tree.propagation.known-words or math.partial-dispatch not configuring the fixnum words properly?
IN: scratchpad [ fixnum-bitand fixnum? ] optimized.
[ fixnum-bitand tag 0 eq? ]

IN: scratchpad \ fixnum-bitand "input-classes" word-prop .
{ fixnum fixnum }

IN: scratchpad \ fixnum-bitand "outputs" word-prop .
[
    [ binary-op-class ]
    [ [ interval-bitand ] binary-op-interval ]
    2bi
    integer-valued
    <class/interval-info>
]

Probably it should output a fixnum-valued interval, not integer-valued?

Yeah I think so too. But there is something tricky, similiar to interval-mod:
fixnum-interval fixnum-interval interval-bitand fixnum-interval interval-subset? .
f

Yeah, this seems wrong:
IN: scratchpad fixnum-interval dup interval-bitand .
full-interval


Remove PyZFP from environment.yml and requirements.txt, move to optional

I'm sure we're all happy to see it go :)

uncontroversial, merging. Thanks


fix argument type of hdf5 formatter write_metadata function

Fixes #440
@unga @core

Looks good, thanks


Lock to Radon 3 0 1

Codecov Report

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

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   79.87%   79.87%
=======================================
  Files           2        2
  Lines         154      154
=======================================
  Hits          123      123
  Misses         31       31

Continue to review full report at Codecov.

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

@tisnik Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-52


Support all pixel formats in samples

The samples are currently hard-coded to 8-bit BGRA.
Add support for all pixel formats.

Blocked by DSOF-17864 as the API is currently reporting RS_FMT_RGBA16 for Uncompressed streams, but only accepting RS_FMT_RGBA32F through rs_sendFrame


dx build --package p1 --bin b1

cargo build --package p1 --bin b1 works, does dx support dx build --package p1 --bin b1?

You can use the bin flag with the CLI and it will search the workspace for a binary that matches that name.
Closing this in favor of https://github.com/DioxusLabs/dioxus/issues/1547, feel free continue in that issue


Log analytics workspace vs storage account?

Hi,
I am a little confused :-)
This article talks a lot about logs in storage accounts. I thought Azure Monitor default stored log files in a log analytics workspace, but this is not mentioned?
If you have all your log files in a log analytics workspace, how do configure RBAC so users only have access to part of the data? According to Microsoft you should have so few log analytic workspaces as possible, but if you have multiple customers or projects they should only have access to their data.
Regards Peter

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

ID: 50c87636-ca0e-f392-f5c5-502113668adf
Version Independent ID: 20db3749-aef8-f111-1428-a3f00ee5731c
Content: Get started with roles, permissions, and security with Azure Monitor
Content Source: articles/azure-monitor/platform/roles-permissions-security.md
Service: azure-monitor
Sub-service: ****
GitHub Login: @johnkemnetz
Microsoft Alias: johnkem

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

Thanks, @femsulu. All the information is here:
https://docs.microsoft.com/en-us/azure/azure-monitor/platform/manage-access#workspace-permissions-and-scope
I think the ordering of the information is a bit strange and very confusing. Log analytic workspaces should be mentioned in this article and then you could link to the dedicated log analytics workspaces article further down.

You are correct, log analytics access modes will help you define RBAC for your workspace.
Good idea to link on this page. Will raise this with the content owner to see if it is feasible.
Thanks again for your feedback. @MGoedtel  - FYI
We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.
Cheers.


Updating build process to use Go 1.19

Go 1.17, which has been used, is no longer supported by the Go community. This update brings the build process up to the latest version of Go.
golangci-lint is updated because version 1.44 did not work with Go 1.19 due to changes.
A large number of files were updated because gofmt handles comments differently as of Go 1.18. There is now a markdown-ish format. gofmt -ew . was run on the codebase to fix the formatting. This primairly updated code comments to be properly formatted. In some cases it fixed indentation elsewhere in a Go file.

@mattfarina I'll close this PR https://github.com/rancher/rancher/pull/38935 that I was working on.

Note, release/v2.6 is updated to Go 1.19 as part of #38960


[typescript] Define components as ComponentClass, not ComponentType

See e.g. Button:  https://github.com/mui-org/material-ui/blob/b29f1cbec31548291a75959d3627473569343eb4/src/Button/Button.d.ts#L36.
ComponentClass supports ref= whereas ComponentType doesn't (because the component may be a stateless functional component).  So any attempt to use ref= results in a Typescript compile error.
ref's are often used to find the anchor element for popovers and similar.

Doesn’t the buttonRef prop do what you want?

@pelotom I'm sure it does... I didn't see it.  https://material-ui-next.com/api/button/ doesn't mention that Button inherits from ButtonBase, and the examples for Popover on https://material-ui-next.com/utils/popovers/ use ref=.
Is ref= deprecated for some reason?  If not, I'd argue to maybe dump buttonRef=, in line with the other last-change-before-1.0 API trimming lately.

For most Material UI components, the argument to a ref={...} callback would be the withStyles HOC instance... probably not what you want. It seems like a leakage of implementation details to expose that, hence why we use ComponentType everywhere, and instead expose targeted fooRef props where appropriate.

https://material-ui-next.com/api/button/ doesn't mention that Button inherits from ButtonBase

It does: https://material-ui-next.com/api/button/#inheritance

the examples for Popover on https://material-ui-next.com/utils/popovers/ use ref=.

We should fix that.

I've opened a PR to update the Popover demo: https://github.com/mui-org/material-ui/pull/10708

Ready to close, by me.


Need to configure for "Accounts in any organizational directory".

I followed Option 2 for Registering Application.   I believe there is a missing a step to select "Accounts in any organizational directory" for "Who can use this application or access this API?"  Error Message:
AADSTS50194: Application 'ed1129e8-d1f7-4575-b00e-be5a0cdf47ce'(ASPNet-Tutorial) is not configured as a multi-tenant application. Usage of the /common endpoint is not supported for such applications created after '10/15/2018'. Use a tenant-specific endpoint or configure the application to be multi-tenant.

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

ID: bf04450a-028e-cc23-d9b3-880a1b5c0fd5
Version Independent ID: ac05d3c4-0379-0f34-9e17-787fb95b7189
Content: Azure AD v2.0 ASP.NET web server getting started
Content Source: articles/active-directory/develop/tutorial-v2-asp-webapp.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

As I look closer, the correct selection is "Accounts in any organizational directory and personal Microsoft accounts (e.g. Skype, Xbox, Outlook.com)"

@Rich-Lang Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

We're unable to complete your request
unauthorized_client: The client does not exist. If you are the application developer, configure a new application through the application management site at https://apps.dev.microsoft.com/.

After switching app registration to "Accounts in any organizational directory", MS logins aren't able to connect anymore.

@mcurko : please have a look at https://github.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2/

@mcurko, please see @jmprieur comment.  I haven't seen you reply back so I'm going to close this issue.  Please feel free to reopen if you problem is not resolved.  Thanks!


feat(tokens): new button semantic color tokens and generate scss in core

Adding new button color tokens and logic to generate core SCSS button color variables.
Description
Adding a set of new color tokens specifically for the component Button. Also includes templates and logic to generate core/scss variables for button color tokens.
(always generated based off of the latest color tokens)
example: $md-button-primary-lightUi: #007AA3 !default;
Related Issue

Motivation and Context
Creates more consistency and ease when styling the button component.
How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Looks clean !


Implement some kind of pub/sub

It's getting pretty apparent that our codebase is pretty messy due to us not having a proper event system implemented. I would say that the most appropriate system to implement would be a pub/sub, but I'm open for suggestions. My main reason for pub/sub would be that (from my basic understanding) it fits in pretty well with the whole flux way of doing things. Again, I'm very much open for suggestions, but this is what I personally would go for.

Fixed with c2da76c


Unexpected output in React Native

I'm getting different outputs on Web and React Native. For example:
"21 hour ago" on React Native and "21 hours ago" on React Web.
My React Native code:
import {Text} from 'react-native';
import TimeAgo from 'react-timeago';

<TimeAgo
  component={Text}
  date={petition.get('createdAt')} // # => "2017-03-18T04:38:44.000Z"
/>

There was a bug that has recently been fixed but I have yet to cut a new release. I will add this as an explicit test-case for that.
As a workaround, I suggest using a custom formatter.

Would it be possible to just return the raw text with no wrapper so that you are responsible for wrapping it in something on whatever platform you use?

You can already use a custom formatter to control the output as much as you want.

This doesn't sound right. This is not using the platform to generate the strings. There WAS a bug that would print 21 hour ago but it was fixed. Please check that you have the same version of react-timeago on both platforms.

Thanks

The fix is as simple as adding the props component={Text}.
The component tries to use a 'time' html element to render the output. That doesn't exist on React Native of course.


Django CMS 3.3: Page tree burger icon has no effect if your page permissions are restricted

If your user is restricted to edit only a subset of pages, the burger icon has no effect on click. The 'Legend" icon too.

I can see the HTML in the source code related to this, it's rendered there, it just doesn't pop up.
The icon works fine if I have permissions to edit all pages, or if I'm a superadmin. A popup pops up with meta info and copy/paste buttons.
Django-CMS 3.3, Django 1.9.6

Thanks for the report.
I noticed this as well but missed to create ticket 👍

@bartTC could you please check if https://github.com/divio/django-cms/pull/5461 fixes this for you?

@vxsx this fixed the bug, thank you all!


To add calculator in the section circle for calculating Chord of a Circle

Is your feature request related to a problem? Please describe.

I want to add a calculator which will calculate chord of a circle

Hi, I want to work on this issue I will start working on it as soon as I get assigned!! I am a part of GSSoC'21.
Kindly assign this issue to me!
Discord Username : Abhijeet Sinha(P)
Discord Tag: #4018


Add LF names to analysis init

Avoids repetitive boilerplate of extracting LF names for analysis.
A couple decision points:

Didn't add Y to init since you might want to try against a couple label sets (?)
Did LFs rather than LF names to 1. avoid name extraction boilerplate and 2. play defense if we want to extract other properties.

Test plan
Updated LF name unit test

Ooh, +1 to @paroma's comment about checking for duplicate LF names. I don't see any good reason for LFs to have duplicate names, and this could easily catch the common mistake (that I've made many times) of not updating an LF's name when copying and pasting so I end up adding the same LF twice.

Duplicates should be checked somewhere -  in the applier probably. And it’s not technically invalid but a warning message that two repeated names exist would help I think

Agree a check somewhere would be good, but let's be more intentional about where that happens. The LF class itself could act as a registry, etc. Lots of better options than here. Leaving for now, will make a note to add.


Updates for interactions of retrieving project list in repositories.

Updates for interactions of retrieving project list in repositories.

Coverage remained the same at 33.222% when pulling e28af6ffce81d4d3163dd17943c91e2c130d9f22 on wknet123:dev into 8dd75c9b85e4146f7845a295c043a94146b70a07 on vmware:dev.


Pseudo Classes and Elements are not compiling properly.

Good afternoon, Glenn Marks! I found some issues in compiling of pseudo classes and pseudo elements. When Im trying to add some pseudo class(or element) to element it compiles with space and browser is not reading it. Here are some screenshots. Sorry for my English. It might be not perfect, but I hope youve understood me.

You need to prepend the :hover with an &, so it should be &:hover. This ties it to the preceding element (takes the space away).

Thanks! I`ve realized that I missed that important thing!

NP, glad you sorted it 😁


Remove stale examples/keras_io

Happy to leave these if we still have a reason, but as far as I know we have merged these back into keras-team/keras-io, the proper source of truth.
They have been popping up in code search results making it slightly harder to check things out in the repo (but not a big deal).

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

Comparison is base (2588a15) 75.36% compared to head (8ae92a8) 55.69%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19042       +/-   ##
===========================================
- Coverage   75.36%   55.69%   -19.67%
===========================================
  Files         339      339
  Lines       35618    35618
  Branches     7010     7010
===========================================
- Hits        26843    19838     -7005
- Misses       7215    14295     +7080
+ Partials     1560     1485       -75

Flag
Coverage Δ

keras
55.69% <ø> (-19.54%)
:arrow_down:

keras-jax
?

keras-numpy
55.69% <ø> (?)

keras-tensorflow
?

keras-torch
?

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

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


Add --upgrade-package option

Add --upgrade-package option like that in pip-compile and pass it on to pip-compile when appropriate.

Fixes #139

I fixed the errors due to incompatible pip-tools version. Please rebase your branch on the master.

Thank you very much. Enjoy 1.4.0.

Thank you very much. Enjoy 1.4.0.

Thank you for pip-compile-multi and helping me to make it better.


Uses rs command as default docker entrypoint

At the moment users have to run the docker container with /app/rs as a command. Because robosat always goes through the rs wrapper and the docker containers only package up robosat it's a better idea to make it the default entrypoint.

Before
docker run --rm -it daniel-j-h/robosat:latest-cpu /app/rs extract ..

After
docker run --rm -it daniel-j-h/robosat:latest-cpu extract ..

Without sub-command we show the help printout.

docker build -t daniel-j-h/robosat:latest-cpu -f docker/Dockerfile.cpu .

docker run --rm -it daniel-j-h/robosat:latest-cpu

usage: ./rs [-h]  ...

optional arguments:
  -h, --help  show this help message and exit

Supersedes https://github.com/mapbox/robosat/pull/140 - can be closed.
@jacquestardie please merge.

Thank you @daniel-j-h :+1:


Make the presentation list more flexible in Admin

Not super familiar with P5, so sorry if my use of Globals or DOM manipulation is not up to standard. Just wanted to PR in a more flexible admin menu. Also: Somebody who knows P5 better than I needs to really glam up this admin menu!  Very ugly, I know.

This is awesome thank you @ChrisDrafters! @all-contributors add @ChrisDrafters for code.


When default value of a property has a pipe symbol in it this results in odd rendering as the pipe symbol is also used as a separator for the property attributes in the template

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

I define this class annotated with @ConfigurationProperties:
@Component
@Data
@ConfigurationProperties(prefix = "test-config")
public class TestConfigurationProperties {

    /**
     * Regular expression
     */
    private String myRegex = "(1|2|3)\\.item";
}

I get this result in spring-configuration-metadata.json file:
{
      "name": "test-config.my-regex",
      "type": "java.lang.String",
      "description": "Regular expression",
      "sourceType": "com.vodafone.dxl.myvdf.chatbot.configuration.TestConfigurationProperties",
      "defaultValue": "(1|2|3)\\.item"
    },

My .adoc file is as below:

pom.xml configuration:
<plugin>
				<groupId>org.rodnansol</groupId>
				<artifactId>spring-configuration-property-documenter-maven-plugin</artifactId>
				<version>0.6.0</version>
				<executions>
					<execution>
						<id>generate-adoc</id>
						<phase>process-classes</phase>
						<goals>
							<goal>generate-property-document</goal>
						</goals>
						<configuration>
							<type>ADOC</type>
							<outputFile>path-to-adoc-file</outputFile>
							<failOnError>true</failOnError>
						</configuration>
					</execution>
				</executions>
			</plugin>

Thank you, it is clear now.

AsciiDoc tables are built with the "|" (pipe) symbol so this is what causes the problem, it must be escaped so the basic templates must be changed, in the upcoming days I'll be picking it up. Until then I can not really suggest anything, because it would break your regexp, so please be patient with the fix.
Until then, can I ask you to give me some feedback in this discussion? :)
Thanks in advance.


MacOS follow up

#21 adds MacOS support but while testing I noticed a few things that might be improved, though it's hard to say given the embarrassing state of Apple's documentation. Writing this down now so I don't forget later.

Minidump supports thread names, however the MacOS implementation doesn't write that data, because Breakpad doesn't. This might (again, terrible documentation) be impossible since the thread name is actually part of pthreads, but on the minidump-writer side we are working with mach ports which are (mostly?) paired with a pthread, but AFAICT the API to retrieve the thread name for the pthread is task specific and probably won't work? But this would be extremely nice information to have if we can get it reliably. There might be some shenanigans to be done by finding out where the BSD pthread info is located in the process space and just reading it directly from the process memory like we do with eg the thread stack etc.

We've already implemented this in Breakpad, see: https://hg.mozilla.org/mozilla-central/rev/63e43c6f4114
Porting it should be straightforward.


DetectorRS with one class

I want to use DetectorRS HTC r50 with my dataset have one class follow coco dataset.  I try to follow the tutorial on website when set, (I try faster RCNN follow tutorial and it worked for me)
model = dict(
roi_head=dict(
bbox_head=[dict(num_classes=1),
dict(num_classes=1),
dict(num_classes=1)],)
and don't use mask head
I try list dict num_class because of Cascade RCNN but got an error:
KeyError: 'cfg or default_args must contain the key "type", but got {'num_classes': 1}\nNone
How can i fix it? And how can i modify config file without mask head?

Did you ever figure this out?


fix: wrong branch name in docker-image workflow

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #148 (72ba280) into master (627f85b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #148   +/-   ##
=======================================
  Coverage   35.39%   35.39%
=======================================
  Files          75       75
  Lines       11561    11561
=======================================
  Hits         4092     4092
  Misses       6991     6991
  Partials      478      478

Continue to review full report at Codecov.

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


Floats are being converted to integers / longs

It appears that floating point numbers are being converted into either integers or longs (depending upon the value of the float). I'm not sure if this is generally the case, but it definitely seems to be happening when I am using YamlDynamicSerializer.deserialize.
I believe I've tracked the issue to String.adjustDynamicString.  It looks like this method checks to see if the double value is within the max integer and max long ranges. If you pick a value like 3.1415 this will satisfy the integer condition and therefor be converted to an integer (and in so, losing significant digits).
You might be able to use the rem function to check to see if the number is in fact representing a float or an integer number - https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-int/rem.html .

Thank you for catching this! I will fix it soon.


Test Blade Samples 7.2

Can you rebase with head? I already had a few of these pushed up.

Sure, sorry about that.


Dogs cannot be heard first time entering 212

Start Leon A
Jump to room 212 Back Alley
Dogs are silent

Seems to be fine loading a saved game where they have already jumped down.

We have the same issue in room 204 with the licker.
It does not matter if you go to another room and come back (same with jumping); the licker remains silent.
However, as you mentioned, if you load the game and then return to room 204, the licker's sounds work again.


JX-GitOperator fails to install - failed to download - helm

Hello,
When I run a fresh install of the jxv3 from this repository, my EKS clusters are installed and it fails to install the jx-git-operator with the below error from terraform
module.eks-jx.module.cluster.null_resource.kubeconfig: Creation complete after 2s [id=6616726043172466068]
module.eks-jx.module.cluster.helm_release.jx-git-operator[0]: Creating...
Error: failed to download "https://jenkinsxio.storage.googleapis.com/charts/jx-git-operator-0.0.143.tgz" (hint: running helm repo update may help)
Could you please help me to overcome this? I'm just following the readme from this Git repo.

I had the same problem. With the suggestion of Chris Mellard in slack channel, I've added the following to main.tf and it worked for me:
provider "helm" {
  version = "1.3.2"
}

I had the same problem. With the suggestion of Chris Mellard in slack channel, I've added the following to main.tf and it worked for me:
provider "helm" {
  version = "1.3.2"
}

same issue: https://github.com/jenkins-x/terraform-aws-eks-jx/issues/233


Upload speed not throttled

Hi,
As the title suggests, I can't get the upload speed to throttle. Download is throttling as expected.
Any ideas?
I'm on 10.13.2
Thanks

Hi @Ajaay ok, must be bug then, mostly used this for testing web pages so the upload speed isn't so important. Do you have a good way of testing so I can replicate and look into how we set up the rules?
Best
Peter


ADE-177/[#2]: Replace sprint references with time intervals and sanit…

Forecasting now supports custom time intervals with either weeks or days. Formatting has also been updated.

Just putting it up to fish opinions about the change. Should resolve #2 when merged. I will be adding comments, updating readme and modify the shell script in the meantime.

LGTM
Looks like the conflicts with main might take a while to resolve, though. I guess the first step would be to run Prettier on forecast.ts from main. Hopefully it won't be so bad after that.

LGTM
Looks like the conflicts with main might take a while to resolve, though. I guess the first step would be to run Prettier on forecast.ts from main. Hopefully it won't be so bad after that.

@kn-ad merge into main completed! Feel free to do a sanity check, I will update readme and then should be all good to merge.


Complete list of components

For a better syntax validation component now is defined as a polymorphic type and can be implemented as container, kubernetes, openshift, plugin or volume. (as stated in https://docs.devfile.io/devfile/2.0.0/user-guide/migrating-components.html)

Complete the list of components in https://docs.devfile.io/devfile/2.0.0/user-guide/adding-components-to-a-devfile.html
Add new page and navigation for missing components: openshift and volume

label: area/documentation
cc: @jc-berger

JIRA: https://issues.redhat.com/browse/RHDEVDOCS-2561

JIRA: https://issues.redhat.com/browse/RHDEVDOCS-2561


Fix matrix generator so it doesnt break if no description on a test

Noticed that we didn't have the regenerated matrix in yet, and was going to push it up to the release branch, but found a bug that is fixed here... which led me to morre bugs in us core which are waiting to get pushed through (which is why I didn't include the actual improved xls file in here)

Addressed separately.


ES6 - jsonix variables not defined

https://github.com/highsource/jsonix/issues/238
https://github.com/highsource/jsonix/pull/266

https://github.com/eblondel/OpenFairViewer/commit/66676a8a9d1f22ef9cdd37e668e302a5a8bafca5


CSS of tooltip can be inconsistent

Hi
Thanks for this. I was looking for a quick way to implement tooltips without having to write my own direcive and this seems to be the best I've come across.
However, I've noticed that the CSS of the tooltip can be affected by the element on which it is used. For example, the font weight seems to pick up that of the underlying element. Also the height may change if the element is in a flex container, for example.
Also, the width of full-width block elements may 'shrink' when the tooltip directive is used.
Maybe I'm doing somethng wrong?

Thanks for the super-quick reasponse. I'll stick with it then!

Hi,
Just following up. I'm still in pretty early stages of my project, so I'm not in any major rush. Just curious if there's been any progress.
Cheers


Optimize archive unpacking

This PR propose an approach to optimize checkpoint archive unpacking.
Two new functions are introduced:

untarFiles` to extract specific files from the archive.
getCheckpointAndDiffSizes  as  replacement for getCheckpointSize function. This function read the sizes of the checkpoint directory and rootfs-diff.tar file from the archive without unpacking the entire archive.

Two failing test cases checking the presence of the checkpoint directory. Is it OK to remove those ?
cc: @adrianreber, @rst0git
Fixes: #51

Overall this looks very good.

Two failing test cases checking the presence of the checkpoint directory. Is it OK to remove those ?

I would say you should adapt them. There should be an error message if the tar file has no checkpoint directory. Currently it seems to just says size is 0. You probably should add an error message if there is no checkpoint directory.
Maybe have an early test to see if there is a  checkpoint directory in the tar file at all. If it is missing we should have some error message.

Please also add new test cases for compressed tar achives to ensure that works and also add a rootfs size test.

Codecov Report
Patch coverage: 71.81% and project coverage change: -0.52 :warning:

Comparison is base (f2a5886) 78.20% compared to head (988770f) 77.69%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
- Coverage   78.20%   77.69%   -0.52%
==========================================
  Files           3        3
  Lines         312      399      +87
==========================================
+ Hits          244      310      +66
- Misses         53       64      +11
- Partials       15       25      +10

Impacted Files
Coverage Δ

checkpointctl.go
81.05% <44.44%> (-1.91%)
:arrow_down:

container.go
79.36% <74.25%> (-2.62%)
:arrow_down:

... and 1 file with indirect coverage changes

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

Please take a look at why the test code coverage decreases. Maybe you need to create tests with broken tar files to handle a couple of your new error paths. Maybe just a compressed file without a tar archive or you can create a tar file and then replace parts of if to break it.
I would also prefer if you do not check in the tar file for the test case. You should be able to create it on the fly.

Also, please do not create fixup commits. If you have fixes of your original commit then squash the fixes into the original commit of the PR.
To add the test cases in a separate commit makes sense from my point of view.

Thank you so much for the valuable feedback @adrianreber
I have modified the previous implementation be able to reuse the same logic were we need to iterate over archive header files.
What do you think ?
Also, I am not sure if the test cases are good ?

Thanks for the added tests. This looks really good know. I will need a couple of days to have a closer look.


server.on("connection", ...) returns the server instance instead of the client socket instance

Per this library it doesn't seem possible to simulate socket.io clients emitting events.
I can only emit events from the server.
var Server = require("mock-server").Server
var mockServer = new Server("http://localhost:1111")

mockServer.on("connection", function(socket){
  // socket === mockServer
  // this makes no sense. socket should be the
  // client socket which connected, not the server.

  // in the real socket.io lib, socket is the client instance
})

In my code, I attach event listeners to the socket which connected, and I'm trying to simulate those. Is this possible using another method?

@brian-mann Sorry for the delay in getting back to you on this and thanks for opening the issue. Could you attach the test that you are trying to write so I can use that as a test case?

This is still true, and for WebSocket as well. To be able to work with apollo  subscriptions-transport-ws  ( https://github.com/apollographql/subscriptions-transport-ws/pull/374 ) I've created a hacky fork ( https://github.com/thoov/mock-socket/compare/master...lgandecki:master ) for now, since it looks like you are rewriting this significantly I didn't think it was worth adding tests around it (although I haven't broken any).
Could you tell me a bit what's your plan here? Are you going to push with the rewrite? Or have you decided to stay with the current architecture?

Any updates on this?


Fixed a couple of typos

Checklist

[ ] All new jobs, commands, executors, parameters have descriptions
[ ] Examples have been added for any significant new features
[ ] README has been updated, if necessary

Motivation, issues

Description

Hello and thank you! If you are able to resolve the current conflicts we would be happy to merge.

Should be fixed in the latest major update thank you.


multiple allow/deny entries for UPnP (rowhelper)

This patch will allow the web GUI for UPnP to enter more user specified
entries rather than just 4, I replaced the 4 boxes with one rowhelper
control

now atleast we have an option to add multiple entries, next im planning to add schedules to it and so far im able to add the config for that to the config xml file and read it out on the GUI but how do i trigger events based on the schedule? i mean what file do i need to be looking at to edit so i can trigger upnp daemon on and off based on schedules?

the other issue is the pkg_edit.php doesnt allow schedules dropdown menu control so are there any plans to add that or else without that using schedules wont be possible, i could add that but if there are no plans to merge it then it would seem like waste of time and effort.

i added schedule selection to pkg_edit.php in another pull request so other packages can also use schedules in them, once that is merged then can add the feature to upnp as well

Bump
i have tried the patch on multiple systems and upnp works perfectly fine from past many days

Does this patch work on the 2.2 Release?

Yes tried and tested on 2.1.5 as well as 2.2, all good

thanks...works great...great patch!
On Sun, Feb 1, 2015 at 12:05 PM, xbipin<EMAIL_ADDRESS>wrote:

Yes tried and tested on 2.1.5 as well as 2.2, all good
—
Reply to this email directly or view it on GitHub
https://github.com/pfsense/pfsense/pull/1455#issuecomment-72381391.

--
Antonio Vitti
+1<PHONE_NUMBER> (USA)
<EMAIL_ADDRESS>

Doesn't it need an upgrade config code to avoid breaking current config?

well it doesnt because the miniupnp.inc file rewrites the config everytime u press save

What happens if user has a config with all those 4 fields filled, and upgrade? the rules user had there will get lost, no?

actually it will remain in config but as soon as he activated upnp and clicks save they r overwritten

Then it needs an upgrade config code, to convert old values into new one, so user don't need to click on activate to have uPnP working again, it's going to simply work.

i can code that if u tell me which file handles the upgrade of config

Sure, you need to bump config version in conf.default/config.xml, from 11.6 to 11.7. Also change 'latest_config' in etc/inc/globals.inc.
After that you need to write a function upgrade_116_to_117() in etc/inc/upgrade_config.inc
Let me know if you have questions

ok thanks, doing it now

Thanks!

im having a hard time writing the upgrade config code, i used this but the problem is values from the old permission fields dont get fed in the new fields
$oldperm = $config['installedpackages']['miniupnpd']['config'][0];
$config['installedpackages']['miniupnpd']['config'][0]['row'] = array();
for ($i = 1; $i <= 4; $i++) {
	$config['installedpackages']['miniupnpd']['config'][0]['row'][$i-1]['permuser'] = $oldperm['permuser{$i}'];
}

i think the issue happens at the below line, any suggestion on whats wrong
$config['installedpackages']['miniupnpd']['config'][0]['row'] = array();

any help?

You need to check if
$config['installedpackages']['miniupnpd']['config'][0]['permuser1']

is defined and is not empty, then take its value and save on the new array called:
$config['installedpackages']['miniupnpd']['config'][0]['permuser']

After, you need to make the same check for permuser2, permuser3 and permuser4. And you also need to unset them.

Well the checks I ignored for a start because by default I checked it's set, I was going to do those steps later but the issue is I did a vardump and it returns the array with old values but when I try to assign them to new fields it just remains blank, I tried like a 100 different methods but failed so it's out of my league I guess or there is some other bug that's causing all this issue

And by the way the new array u mentioned actually that needs to be in a sub array called row as the rowhelper control needs that as that's how it works

Unsettling the old fields also doesn't work for me, no idea if it's due to the fact that  I'm working on a virtual box virtual machine for development

Something like this should work (untested):
function upgrade_116_to_117() {
	global $config;

	if (!isset($config['installedpackages']['miniupnpd']['config'][0]))
		return;

	$miniupnpd =& $config['installedpackages']['miniupnpd']['config'][0];

	$miniupnpd['permuser'] = array();

	for ($i = 1; $i <= 4; $i++) {
		if (isset($miniupnpd["permuser{$i}"]) && !empty($miniupnpd["permuser{$i}"])) {
			$miniupnpd['permuser'][] = $miniupnpd["permuser{$i}"];
			unset($miniupnpd["permuser{$i}"]);
		}
	}
}

Oh yeah, I forgot about the rowhelper format, do you have a block of config.xml with rowhelper set?

What about this one:
function upgrade_116_to_117() {
    global $config;

    if (!isset($config['installedpackages']['miniupnpd']['config'][0]))
        return;

    $miniupnpd =& $config['installedpackages']['miniupnpd']['config'][0];

    $miniupnpd['row'] = array();

    for ($i = 1; $i <= 4; $i++) {
        if (isset($miniupnpd["permuser{$i}"]) && !empty($miniupnpd["permuser{$i}"])) {
            $miniupnpd['row'][] = array('permuser' => $miniupnpd["permuser{$i}"]);
            unset($miniupnpd["permuser{$i}"]);
        }
    }
}

with a row helper it should be stored this way
			<row>
				<permuser>allow 1024-65535 <IP_ADDRESS> 1024-65535</permuser>
			</row>
			<row>
				<permuser>allow 1024-65535 <IP_ADDRESS> 1024-65535</permuser>
			</row>

the last one worked, it seems the issue was i was using single quote
['permuser{$i}']
whereas double were required
["permuser{$i}"]

i have committed the last code u gave me, works well and it seems complete now to merge it :)

done
it wouldnt make much difference with empty values as they would anyways go the next time upnp settings would be saved


Run build jobs on pull requests

If someone issues a PR from a fork, we don't run the build job, which then prevents deployment/merging.

i don't like that it builds twice, closing for now.


SocketException in example/lib/scan/mdns_scan.dart

I ran example/lib/scan/mdns_scan.dart and got the following error:
C:/a_dev/flutter-sdk/bin/cache/dart-sdk/bin/dart.exe --enable-asserts C:\Users\xmlspyspring\IdeaProjects\network_tools\example\lib\scan\mdns_scan.dart
Unhandled exception:
SocketException: Failed to create datagram socket (OS Error: 在其上下文中，该请求的地址无效。---> In its context, the requested address is invalid
, errno = 10049), address = , port = 5353
#0      _NativeSocket.bindDatagram (dart:io-patch/socket_patch.dart:1051:7)
<asynchronous suspension>
#1      _RawDatagramSocket.bind.<anonymous closure> (dart:io-patch/socket_patch.dart:2558:15)
<asynchronous suspension>
#2      MDnsClient.start (package:multicast_dns/multicast_dns.dart:135:40)
<asynchronous suspension>
#3      MdnsScannerServiceImpl.findingMdnsWithAddress (package:network_tools/src/services/impls/mdns_scanner_service_impl.dart:73:5)
<asynchronous suspension>

Process finished with exit code 255

My Environment: windows 10 chinese
PS C:\Users\xmlspyspring\IdeaProjects\network_tools> dart --version
Dart SDK version: 3.3.1 (stable) (Wed Mar 6 13:09:19 2024 +0000) on "windows_x64"

I know what the reason is, but I can't change it. This is caused by a bug in the multicast_dns.
The file in this library multicast_dns - 0.3.2 + 6\ lib\ multicast_dns.dart line 63 gets all NetInterfaces, but I have two NetworkInterfaces that are closed, so using a socket to connect the two NetInterfaces makes an error.
  Future<Iterable<NetworkInterface>> allInterfacesFactory(
      InternetAddressType type) {
    return NetworkInterface.list(  // --->   here
      includeLinkLocal: true,
      type: type,
      includeLoopback: true,
    );
  }

我找到了解决办法,但是不知道对不对.
或者换以一种思路,includeLinkLocal: true ,然后
in file  lib/src/services/impls/mdns_scanner_service_impl.dart   line 73
change
await client.start();

to
await client.start(interfacesFactory: (InternetAddressType type){
  return NetworkInterface.list(
    includeLinkLocal: false,  //  old is true
    type: type,
    includeLoopback: true,
  );
},);

or
    await client.start(interfacesFactory: (InternetAddressType type){
      return NetworkInterface.list(
        includeLinkLocal: true,
        type: type,
        includeLoopback: true,
      ).then((List<NetworkInterface> list){
        list.removeWhere((item){
          if(ping item is not ok) return true;   // remove this item
          return false;  // ping item is ok
        });
        return list;
      });
    },);

includeLinkLocal: false will exclude interfaces for everyone. You have 2 interfaces closed but others can have open. If we can determine if a network interface is not closed then we fix this. Probably you can to clone this repo and raise a PR for the fix.

But I don't think that this is the right place to fix it. If we can fix it in package multicast_dns then it would be better. Can you open the same issue in multicast_dns also and let's see if they can fix it.


Using @Rule in a meta annotation

Thanks so much for the quick turn around on the rule name modification.
Now annotating a rule class with just @Rule without a rule name automatically defaults the rule name to the class name, however, the rule class does not like a meta-annotation that has @Rule embedded in it. For example, I have defined my meta-annotation @BasicRule in the following manner. My tests throw an exception while parsing the class, complaining that it cannot find the @Rule annotation, even though I have annotated my rule class with @BasicRule.
@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.TYPE)
@Component
@Scope("prototype")
@Rule
public @interface BaseRule
{
}
Sample Stack Trace:
java.lang.IllegalArgumentException: Rule ‘XYZ' is not annotated with 'org.easyrules.annotation.Rule'
at org.easyrules.core.RuleDefinitionValidator.checkRuleClass(RuleDefinitionValidator.java:34)
at org.easyrules.core.RuleDefinitionValidator.validateRuleDefinition(RuleDefinitionValidator.java:26)
at org.easyrules.core.RuleProxy.asRule(RuleProxy.java:34)
at org.easyrules.core.DefaultRulesEngine.asRule(DefaultRulesEngine.java:209)
at org.easyrules.core.DefaultRulesEngine.registerRule(DefaultRulesEngine.java:101)

I've submitted a pull request to address your issue.
In the case where you've got lots of Rules that are also Spring-managed @Components that are also prototype-scoped, a custom @Rule annotation can reduce the amount of annotation boilerplate.

@richdouglasevans, many thanks to you for adding support for meta-annotations!
I've merged your PR in the master branch.
@cvsubs Could you please give it a try with the last version of the master branch and keep us informed if this is ok in your use case? Sorry for not being able to push the snapshot version to maven central, I'm behind a nasty proxy :rage:

@cvsubs I've got your email, glad to hear it works fine for you :smile:
kudos to @richdouglasevans for the great PR!
I've planned to release v2.2 by the end of this week. Keep tuned
Regards
Mahmoud


[WIP] list_realms: Print the full domains of the organizations.

Add a new column which prints the domain of the respective realms.
Fixes part of #11015

Testing Plan:
GIFs or Screenshots:

@Hudda Thanks for working on this!
Can you provide some example output?

@showell here is the example outpu

@shubhamdhama Thanks for your feedback. I will make the changes.

@Hudda did you intend to close this?

@timabbott No sir, that was not intentional. It happens as an accident. But I opened a new pull request here.


Data Viz: Resize viewport to fit drawing

Prevents drawing from exceeding boundaries of viewport and becoming unviewable.

Pull Request Test Coverage Report for Build 2439

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 35.027%

Totals

Change from base Build 2430:
0.0%

Covered Lines:
2174

Relevant Lines:
5509

💛  - Coveralls

The current deployed version can draw circular lists.
However, with this PR, circular lists cannot be drawn and i get this error instead.
Line 8: RangeError: Maximum call stack size exceeded

Ah, brainfade, sorry. Fixed now.

Resizing not working yet, as seen in this example:
draw_data(build_list(100, x => x));

@lumos309 can you resolve the conflicts?


Upgrade to Wagtail 4.x

Wagtail 4.x supports Django 3.2 according to https://docs.wagtail.org/en/stable/releases/upgrading.html#compatible-django-python-versions
Short term we want access to Global settings, see https://docs.wagtail.org/en/stable/releases/4.0.html#global-settings-models
See https://docs.wagtail.org/en/stable/releases/4.0.html#upgrade-considerations

Looks great!

@theskumar Opening/edit a Rounds leads to a server 500 error, https://dev-apply.opentech.fund/admin/funds/round/.

@theskumar Opening/edit a Rounds leads to a server 500 error, https://dev-apply.opentech.fund/admin/funds/round/.

It fails with the missing template error, looking into it.
TemplateDoesNotExist at /admin/pages/18/edit/
wagtailadmin/panels/single_field_panel.html

Add a fix for it. This is how it looks now on my local machine, using the sandbox db dump.

@fourthletter found another issue, creating a round.
TemplateDoesNotExist at /admin/funds/round/choose_parent/
modeladmin/includes/header_with_breadcrumb.html
Request Method:
GET
Request URL:
http://apply.hypha.test/admin/funds/round/choose_parent/
Django Version:
3.2.18
Exception Type:
TemplateDoesNotExist
Exception Value:
modeladmin/includes/header_with_breadcrumb.html

In hypha/apply/funds/templates/funds/admin/parent_chooser.html it works to replace the header block with:
    {% block header %}
        {% include "wagtailadmin/shared/header.html" with title=view.get_page_title subtitle=view.get_page_subtitle icon=view.header_icon %}
    {% endblock %}

But might not be optimal.

@frjo The above works fine and doesn't seem to be create any issue. I've pushed the fix for it to #3045 . I'm not sure if you need to redeploy the test.

@fourthletter Deploying a fix to test now.

@theskumar In current Hypha we have code to stop staff from using "File Field" (single) since they by mistake used it in place of "Multiple File Field".
On old applications forms where it was already added it should work as normal. But it should not be possible to add to any new forms. And if deleted fron an old form it should not be possible to add it again.
With this PR it is back.

ah. This is a good catch, I'll look into this.

@theskumar Another issue that I believe is Wagtail 4 related.
We have code to make the category fields use the default value if empty on the field instance.
https://github.com/HyphaApp/hypha/blob/278d8a7dc18e106e924acc7ee9346d46ede5072a/hypha/apply/categories/blocks.py#L88-L95
In line 94 value['category'] used to return the category object <Category: Region(s) archive> but on test now it just return an int, the Category ID.
[2023-04-22 07:18:55] DEBUG [hypha:98] StructValue([('field_label', 'Region'), ('help_text', ''), ('help_link', ''), ('required', True), ('category', <Category: Region(s) archive>), ('multi', True)])

[2023-04-22 07:14:20] DEBUG [hypha:96] StructValue([('field_label', 'Region'), ('help_text', ''), ('help_link', ''), ('required', True), ('category', 5), ('multi', True)])

See https://otf.sentry.io/issues/4116177815/?project=5823154&query=is%3Aunresolved&referrer=issue-stream&stream_index=0

@theskumar In current Hypha we have code to stop staff from using "File Field" (single) since they by mistake used it in place of "Multiple File Field".
On old applications forms where it was already added it should work as normal. But it should not be possible to add to any new forms. And if deleted fron an old form it should not be possible to add it again.
With this PR it is back.

@theskumar Could we please change 'File field' to 'Single file field'? We have historical knowledge as to why this field is faded out, but other organizations adopting this platform will not have this historical knowledge.

If easy we can hide the "File Field" (single) instead of disabling it, then no confusion.

Right, I looked the internal changes and it feels like hiding might be much easier than disabling it.

Update: the disabling/hiding of "Single File Field" is turning out to be harder than a expected, as the core of the wagtail is now react and it's harder to hack around it. I've spent sometime on it today and over the weekend. If I'm not able to solve it by tomorrow. We'll have a make a decision.
For the categories issues, will take a look, have got to that yet.

Cool. Will push the code with "Single File Field"


Unexpected regex behavior, dash in character class

var token = /^[a-zA-Z0-9_!#$%&'*+.^`|~-]+$/;
console.log( token.test('Content-Type') );

The above prints false while it should be true
tested with duktape v 1.4 & nodejs v6.9.5  and latest chrome / firefox all give true
it seems that removing end regex match will fix this
var token = /^[a-zA-Z0-9_!#$%&'*+.^`|~-]+/;

The root cause is related to parsing a literal minus vs. a codepoint range, a simplified repro case is:
duk> /[x-]/.test('-')
= false

The simplest fix for now is to escape the slash:
duk> /[x\-]/.test('-')
= true

I'll look into the root cause a bit deeper later.

yes I also just found that moving dash any where but the end will fix this too
var token =  /^[a-zA-Z0-9-_]+$/;
console.log( token.test('Content-type') ); // true

I think I found the cause: regexp class range '-' parsing checks whether the dash is followed by a ']', but that lookup was wrong in Duktape 2.0 and later.

Ok, #1570 has a bug test case and a fix. I'll merge it once the tests pass.

I'm not sure how you can be this fast in fixing issues, but you're awesome :+1:

Hehe, thanks :) This one was luckily a small bug probably introduced when reworking some lexer stuff for 2.0 release.
Thanks for reporting it, now it's covered by testcases too :)


overlaps in Interval

The following code in models.py would return false for two zero length intervals, e.g., Interval a(42,42) and Interval b(42,42).
Is this desired behavior?
def overlaps(self, other):
assert isinstance(other, Interval)
return self.end > other.start and other.end > self.start

Peter- sorry for not getting to this earlier.
I think this gotcha is a consequence of logic that's required for more typical cases.
First, recall these are interbase intervals. So, a(42,42) and b(42,42) are zero-width points (not width 1 sequences).
Let's consider non-zero intervals firs. Say c(5,7) and d(6,8), both length 2. The condition for overlapping is
c.end > d.start and d.end > c.start, which is true.
(For inclusive coordinates, this would be written as ≥)
Imagine e(7,9). Intuitively, we would not think that c and e overlap (again, interbase coordinates, so they abut at 7).
Finally, imagine moving c.start rightward to 7 (so that it becomes c2(7,7) and e.end leftward to e2(7,7). Obviously, if c(5,7) and e(7,9) don't overlap, their shrunken derivatives c2 and e2 shouldn't be seen to overlap either.
So, in effect, the condition is essentially asking whether the overlap length is non-zero. That reinterpretation is critical for thinking about whether any segments overlap, including the odd-seeming case of two zero-width points.
Does that pass the Peter logic filter?

Closing old question.


Prepare for v0.9.5

Summary

Code documentation and documentation site configs.
Integration and unit tests.
Bugfix: deserialization of a deploy with StoredVersionedContractBy* executable deploy item failed because of a typo.
Serialize BigIntegers as unsigned always.

TODO
Checklist

[X] Code is properly formatted
[X] All commits are signed
[X] Tests included/updated or not needed
[X] Documentation (manuals or wiki) has been updated or is not required

@davidatwhiletrue another thing I noticed is that we should add a link to the dockerized NCTL instructions in the README to make it easier to set up the test environment. Atm one should know how to use NCTL in order to proceed. Do you agree? Please make a note to add it once it's ready if you do.

@ihor It's a good idea. I've added to the README the command to start the container and a link to the full instructions. Similarly, I added a new page to the documentation website.


Catch AttributeError if someone passes a bool

Summary of changes (Bug fix, feature, docs update, ...)

Catch AttributeError if someone passes a bool to retrieve_atmospheric and update_mismatch=True

Please check if the PR fulfills these requirements

[x] Closes #1028
[x] Closes #1027
[x] Tests added and passed if fixing a bug or adding a new feature
[x] All code checks passing - black --line-length 110 run over code, mypy and flake8 not showing any errors
[x] Documentation and tutorials updated/added
[x] Added an entry in the latest CHANGELOG.md file if fixing a bug or adding a new feature

Based on type definitions would no longer expect a bool to be passed for the update_mismatch argument. In general it would be better to include generic type checking to catch these sort of error but, in this case, because this is a change from a previous set up (update_mismatch used to allow a bool) this fix seems reasonable.


referenceProxy

I test reference proxy , it's increase class size and generate two, three... same method, i write small class no need generate new method every time, just generate one time and use, can you add this?
// Test
        invoke(getDeclaredMethod("test.Main" /* Find Class */, "hello" /* Find Method */), "test.Main" /* Find Class */);
// Test 2nd
        invoke(getDeclaredMethod("test.Main" /* Find Class */, "hello" /* Find Method */, String.class /* Arguments Type */), "test.Main" /* Find Class */, "Hello World" /* Arguments */);

Full Code:
https://pastebin.com/NHPCfQQ3

#2 Is similar.


Initial Stripe and Charges Object

Little rough around the edges but works well, This is the initial charges and core stripe object.

Don't forget about self.class to access the static variables


Pass apply checks as functions to Applicators

Signed-off-by: hasheddan<EMAIL_ADDRESS>
Description of your changes

Updates Applicator interface to take functions which can check conditions before executing patch. Allows for package consumer to pass in custom checks (ref: https://github.com/crossplane/crossplane/pull/1341/files#diff-ba7cd8f84dd4227171ca0d3ed2bf46baR275). Also allows for any transformations to be applied to object before patch (ref: https://github.com/crossplane/addon-oam-kubernetes-remote/pull/1/files#diff-c4486ee3d6e8e3a194c5722f78c15955R73).
@negz I'll go back and update any places we are already using Apply after this is merged :+1:
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant documentation, examples, or release notes.
[ ] Updated the RBAC permissions in clusterrole.yaml to include any new types.

@negz updated :+1:


Windows installation is behaving like malware

What is the issue?
This is beyond bad.

It installs stuff all over the place, without informing the user.

	C:\Users\xxxx\AppData\Roaming\Microsoft\Windows\Start Menu\Programs
	C:\Users\xxxx\AppData\Local\Temp\ollamaNNNNNN
	C:\Users\xxxx\.ollama\id_ed25519
	C:\Users\xxxx\AppData\Local\Programs\Ollama
	C:\Users\xxxx\AppData\Local\Ollama

It doesn't ask the user where they want it to be installed!
It opens ports and starts a server under different application name and on unspecified sever ports that can only be found in a temp directory.
Once the original (cmd) windows is closed, the server is still running!
There are no instructions on how to shut down the server

In the current state this application is acting more like malware than something serious.
OS
Windows
GPU
Nvidia
CPU
Intel
Ollama version
No response

Did you download from the website?
If you did, those file locations look correct, and Ollama runs in the background when your computer starts so you can use the models supported by Ollama at anytime without having to start the app.

The issue with downloading without asking the path might be an issue with the installer. Otherwise, Ollama was built to run in the background. Ollama is not malware.

Did you download from the website?
If you did, those file locations look correct, and Ollama runs in the background when your computer starts so you can use the models supported by Ollama at anytime without having to start the app.

The issue with downloading without asking the path might be an issue with the installer. Otherwise, Ollama was built to run in the background. Ollama is not malware.

I'm having the same problem, I'm trying to install ollama as well as the models to other partitions instead of the system partition, but I can't do it.

In 0.1.33 we've moved to a more conventional packaging model on Windows where all the executables are in the installer instead of carrying them as payloads in the primary CLI.  We've also added a zip file with all the components for folks who want to roll their own installation models in advanced use-cases.  The pre-release is available at https://github.com/ollama/ollama/releases
@liaosu0755 you can adjust the location of the models by adjusting an environment variable - see https://github.com/ollama/ollama/blob/main/docs/faq.md#where-are-models-stored
Fine-grain control over the installer location is tracked in #2776

@dhiltgen
Please reconsider your install paths.
No serious package is putting huge models under a dotted sub-directory in home, especially on Windows.
Comfortably closing the issue, does not solve the problem. The problem is that ollama is behaving like malware, for all sorts of reasons, including and not limited to the ones I stated above. But mostly for lack of informing the user what the installer is about to do.

Fine-grain control over the installer location is tracked in #2776

You could also have answered:
$Env:OLLAMA_MODELS = 'C:\LLM\models'                                    # User
[Environment]::SetEnvironmentVariable('OLLAMA_MODELS','C:\LLM\models')  # System

Please put all environment variables in your README, so users doesn't have to waste time finding out about this, only after having completed installation.

I agree with eabase. I have a 16Tb disk + 2Tb SSD + 500 Gb nvme but most of the time I can't even use them because so many programs force themselves in C: system disk, I hate it.
Here I just installed Ollama with 1 model, I have now 495Mb left. This is a neverending fight with the programs even though I have plenty of storage space, a custom install option is essential

@eabase Ollama is a client-server architecture.  The server keeps track of the models, and the client keeps track of history.  We don't currently have a way to alter where the client stores state; it will always be in your home directory.

@dhiltgen
Thanks for clarification. One last, question:
(a) What is placed in this history folder?
(b) How large can it get?


Fix non-ghosts and admins counting toward most followed

About the PR

Anomaly cores and admin ghosts no longer count as followers for the purposes of the "Warp to most followed" button.
Why / Balance

Counting cores is silly. Counting admins is bad.
Fixes #28106
Fixes #27957
Technical details

Instead of using Followers.Count, the code now loops through the followers and checks for GhostComponent. To prevent counting admins, only ghosts with CanGhostInteract true are included.
Slightly less efficient since we now iterate through the followers on each followed object, but this isn't exactly a hot code path.
Media

Warping to the character with one ghost follower, ignoring the other character with more anomaly core followers:
https://github.com/space-wizards/space-station-14/assets/85356/a29ebaae-7e79-44ea-8966-cead995dfb8c
The same thing, but now the one ghost is an aghost, so there is no "most followed" and no warp happens:
https://github.com/space-wizards/space-station-14/assets/85356/d190b0cd-6f44-481b-9fa0-7f772b98888d

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: "Warp to most followed" ghost option no longer includes admins and non-ghost followers.

No more admin ghost gang, damn. Also it only just hit me yeah anom cores share code with ghost follow huh?

anom cores share code with ghost follow huh?

Yep, both use FollowerSystem.

If you didn’t already, please do that with the ghost plushie and revenant plushies because they have the same issue

If you didn’t already, please do that with the ghost plushie and revenant plushies because they have the same issue

They should be covered, but let me test and confirm that.

So, checking IsAdmin isn't great, since it excludes all admins who are ghosts, even if they're just regular ghosts and not aghosts. Unless that's desired, I'm sticking with basing it on CanGhostInteract. I did switch it over to better leverage EntityQueryEnumerator for filtering though.

Should be the last two changes. Tested it and everything works.


Add exclude_members param to channels.list

Docs: channels.list

Done: https://github.com/SlackKit/SKWebAPI/pull/5


docker build failed

Failed when try to build docker image from source of simple-worker
Step 9 : RUN godep go build -ldflags "-X main.GitVersion git rev-parse HEAD -X main.BuildTime date -u '+%Y-%m-%d_%I:%M:%S'"
---> Running in 375c03b85fb7
fatal: Not a git repository (or any of the parent directories): .git
I guess reason is .dockerignore of your source  exclude .git directory.  Do you need to modify your .dockerignore ?
Thanks
Kevin

Nice catch and thanks for your attention.
You're right. Now we rely on git directory to get the build version and build time. I have removed it in .dockerignore and thanks again :+1:


Compilation error on Windows for VisibleBoundsPadding

Current behavior
Trying to set
toolkit:VisibleBoundsPadding.PaddingMask="All"
Throws error compiling on Windows
Also tooltip can't resolve VisibleBoundsPadding

Expected behavior
Should be ale to set this property on all targets
How to reproduce it (as minimally and precisely as possible)
dotnet new unoapp
Set utu:VisibleBoundsPadding.PaddingMask="All"  on the root grid of MainPage
Try to build and run
Nuget Package:
Stable uno.templates (ie Uno 4.10 release)
Package Version(s):
Affected platform(s):

[ ] WebAssembly
[ ] Android
[ ] iOS
[ ] macOS (AppKit)
[ ] Mac Catalyst
[ ] Skia

[ ] WPF
[ ] GTK (Linux)
[ ] Linux Framebuffer
[ ] Tizen

[ ] Windows

IDE:

[ ] Visual Studio 2022
[ ] Visual Studio 2019
[ ] Visual Studio Code
[ ] Visual Studio for Mac
[ ] Rider Windows
[ ] Rider macOS

Relevant plugins:
Anything else we need to know?

Ignore this - wrong toolkit


Using -r will remove build tags above license

Firstly, thanks for the amazing tool!
I have one small issue to report. In source files with a build tag above the license, running the tool with -r will accidentally remove those build tags if the license is different.
Example diff after running the tool when I update the license:
diff --git a/a.go b/a.go
index 3b5f281..e5fdaaa 100644
--- a/a.go
+++ b/a.go
@@ -1,8 +1,5 @@
-//go:build linux
-// +build linux
-
 /*
- * Copyright 2022 <Name>
+ * Copyright 2022 <New Name>
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/hack/boilerplate.go.txt b/hack/boilerplate.go.txt
index 0cfe2ae..696bdd8 100644
--- a/hack/boilerplate.go.txt
+++ b/hack/boilerplate.go.txt
@@ -1,5 +1,5 @@
 /*
- * Copyright 2022 <Name>
+ * Copyright 2022 <New Name>
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Hi @irvinlim!
I am glad that you found the tool useful.
Thanks for the feedback, I'll provide a new version soon that will not accidentally remove the build tags when replacing a license.

Hi again @irvinlim,
You can now download the new release v1.3 that does not remove the build tags.
Regards,
Lluis

Amazing, thanks @lsm-dev for the fix!


feat: add struct (root included) field name aliases

With aliases supported, we can more freely rename fileds. e.g.
if a field is renamed to new_field, all we need to do is to keep the old name field as an alias in the schema like this:
{new_filed, #{type => integer(), aliases => [field]}

tagged 0.37.0