[native_assets_builder] Tree shaking

We should consider running the build after kernel compilation instead.
Use cases:

If native assets are not reachable, then the whole build.dart for that package does not need to be run.
For localization, we want to bundle a message file, and we want to tree-shake that message file based on what is reachable from the code. (If we would have icons, it would probably follow something similar to locazation.) https://github.com/dart-lang/sdk/blob/main/pkg/compiler/doc/resource_identifiers.md

For use case 2, we might want to standardize the annotations and output format for what is reachable, so that any user can implement tree-shaking for any type of resource they want to bundle (e.g. icons, images, localization, nothing is special.)

Notes from discussion with @mkustermann:
Maybe we should not

conflate user-defines and tree-shaking "inverse meta data", and
run builds after kernel compilation.

Instead, we should consider

having user-defines being fully global (not scoped by only having them visible to dependencies),
doing "build" in order from dependencies to dependents before the Dart build, and
doing "tree-shaking" in reverse order after the Dart build.

Step 2 and 3 would be similar to C++ builds where the build-step is separate from the linking step.
The data-flow in 2 would be as meta-data currently works.
The data-flow in 3 would be in reverse order ("reverse meta data").
This approach should also cover the use case for package:messages. Packages with a data/messages.json would declare it in the the tree-shaking phase, and package:messages would pick up this inverse metadata and output the data assets.
We have multiple options for what file this tree-shaking script should be in:

link.dart (next to build.dart)
build.dart and introduce a --linking phase.

@mosuem Any thoughts about this approach?

Having a build phase and tree-shake phase does raise the question what the CLI interface should be for the tree-shake phase.

Should both the build phase and tree-shake phase be able to output code assets and data assets? Or should the code and data assets output in the build phase be input to the tree-shake phase?
Should the build meta-data also flow to the tree-shake phase? Or should info from build -> tree-shake be a separate data structure? Or should it only be data/code assets? (Data assets can be arbitrary files, so it could contain anything.)

What does in reverse order mean?
But generally, I believe this would correspond to the initial idea of having a build_after.dart.
I have no thoughts on whether having a second script is better or worse than different calls to build.dart. Also, there is the still the question of whether the latter would be implemented through a CLI or an implementation of an interface having build(); and tree_shake(); methods.
An advantage to different scripts we discussed before would be the lower overhead of running only the available scripts, instead of running a build.dart with possibly empty instructions.

What does in reverse order mean?

Dependencies get the "meta data" / "config" from the dependants. E.g. package:foo depends on package:messages, so the config from package:foo data/messages.json flows to package:messages.

Notes from discussion with @mosuem:
The interaction between build.dart and link.dart should probably be as follows:

build.dart outputs code/data assets if they should not be tree-shaken
build.dart outputs "meta data" which are keyed on target package link.dart for tree-shaking by that package link.dart
link.dart outputs code/data assets

build.dart meta data output is keyed on target package, and link.dart of the target package gets the meta data.
link.dart output from one package is not visible to other link.dart (We don't see any any situation in which one package's link.dart depends on another package's link.dart.)
BuildOutput should contain Map<String packageName, Object config> linkInput (and BuildOutput should have what it currently already has).
LinkInput should contain List<Object> inputFromBuild in addition to some of what BuildInput already has (target OS, etc.)
LinkOutput should contain data & code assets.
Data assets output by either build.dart or link.dart should be bundled in Dart and Flutter apps and accessible by https://api.flutter.dev/flutter/services/AssetBundle-class.html. TODO: figure out what the different bundles mean in Flutter.
The implementation plan would have multiple stages:

Add data assets to build.dart, add AssetBundle to a dart: library, and consume build.dart data assets in Flutters AssetBundle.
Add link.dart CLI to this package,  and introduce a linking phase in both DartDev/FlutterTools for tree shaking.

Sounds good. The deferred loading can be just delegated to the packages own AssetBundle implementation based on a the mapping between loading units and call sites reported from that packages link.dart.

Sounds good. The deferred loading can be just delegated to the packages own AssetBundle implementation based on a the mapping between loading units and call sites reported from that packages link.dart.

If I remember correctly, the shaker would output:

splitting_info.json contains K OR dart-library-uri OR package-name -> file-name
messages_1.json K -> V
messages_2.json K -> V

We still need to flash out how that works with the different deferred loading mechanisms in Flutter and dart2js.

Putting the actual keys in splitting_info.json is going to blow up that file, which nullifies the benefits of deferred loading.
package-name would be preferential, because that would only be one const string. However, I believe flutter splits on dart library. I don't know about dart2js.
library uri (dart file name) seems to be a middle ground, but you'd really only want to use that with code-gen. You don't want users to write their library uri inside the dart file the whole time. (With @Natives we default the optional assetId argument to default to library uri to circumvent this issue.)

Did you give this aspect more thought yet @mosuem?

Or did we want to depend on "component name"? But also that requires code generation. E.g. a package author does not know what the component name is where their file ends up in when that package is used inside an app that defines a split in components.

If I remember correctly, the shaker would output:

splitting_info.json contains K OR dart-library-uri OR package-name -> file-name
messages_1.json K -> V
messages_2.json K -> V

The shaker would output whatever the package needs to load the component. So for package:messages, the link.dart would would build a MessagesResourcesAssets object using the information from the LinkInput/BuildOutput. This object could then be serialized into the AssetBundle. See the example at the example PR.
In this case, it builds a Map<int, int> from message indices to parts. Probably doing it the other way around, Map<int, Set<int>>  parts to set of message indices, is smaller (but has a worse lookup time, so we might want to build the Map<int, int>  at startup).

Putting the actual keys in splitting_info.json is going to blow up that file, which nullifies the benefits of deferred loading.

I don't think it is such a large file, it should be basically number of used message indices*sizeof(int).

Having build.dart specify the linker that an asset is destined for covers our initial use cases.
We can later extend it to have applications overwrite that. https://github.com/dart-lang/native/issues/994#issuecomment-1999363039

Not include an asset at all if it's not referenced from Dart code.
Shrinking the contents of an asset if only a subset of its contents are used.

Both of these can be done in a link hook.
And the first one of these can be done by the Dart/Flutter SDK if all uses of a certain asset ID are const.
@mosuem suggested we should have some kind of enum for BundlingBehavior to control whether the Dart/Flutter SDK throw assets out if their assetId does not occur as a const in the program:

alwaysKeep The link hook writer will take care of throwing the full asset out if it is not used based on the tree shaking information
onlyKeepIfConstReferenced The Dart and Flutter SDK may throw this asset out if there is no reference to it with a const assetId. (NativeCodeAssets currently do not support an API that enables dlopen with a dynamic asset id, @Native is always const asset id, so NativeCodeAssets always work like this.)

@mkustermann suggested to tie this behavior to what hook the asset is output from:

build hooks are onlyKeepIfConstReferenced by default.
link hooks are alwaysKeep by default. The link hook writer will have thought about whether the asset needs to be included already.

I would even introduce two modes instead of onlyKeepIfConstReferenced, for a total of three possible values.
enum Treeshake {
  /// Never treeshake by the SDK
  never,
  /// Treeshake if `loadBytes` never has a non-const arguments passed
  /// and no const argument matches the asset id
  //TODO: find better name
  safe,
  /// Treeshake even if `loadBytes` has non-const arguments passed, but
  /// none of the known const ones matches the asset id.
  unsafe;
}

When using the third mode, it is entirely possible that assets are treeshaken which are actually used, which would only be noticed at runtime. But in the safe mode, any package in the dependency tree which does a single loadBytes with a non-const argument poisons the treeshaking information, and results in nothing being treeshaken.

safe

And then we should probably add a warning on the stderr or stdout when building and one of the hooks reports assets with safe but there are dynamic uses of loadBytes somewhere in the app. "Warning: your app size might have significantly increased due to ....".


i3 Notes

Would you mind  posting your notes on i3wm for the devterm R-01?

I'll try to do that this weekend. Are you having a specific issue? I seem to remember it was reasonably straightforward (install i3 packages, configure in the text files in ~/.local) but I'll have to pull out my R01 core to check what I did.

I haven't received my unit yet, but I am trying to gather resources in advance. Very exciting stuff! Your documentation seems to be the most comprehensive I could find on the R-01, I really appreciate you sharing it.
Sep 15, 2022, 03:33 by @.***:

I'll try to do that this weekend. Are you having a specific issue? I seem to remember it was reasonably straightforward (install i3 packages, configure in the text files in ~/.local) but I'll have to pull out my R01 core to check what I did.
—
Reply to this email directly, > view it on GitHub https://github.com/andypiper/DevTerm-KB/issues/1#issuecomment-1247696524> , or > unsubscribe https://github.com/notifications/unsubscribe-auth/AW5BRB5FQS5PAHZIYEHEGHDV6LGK5ANCNFSM6AAAAAAQMTIJDQ> .
You are receiving this because you authored the thread.> Message ID: > <andypiper/DevTerm-KB/issues/1/1247696524> @> github> .> com>


feat(app): Add Motivational Quote Easter Egg

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[ ] Small bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #16382
Description

Added a feature where a motivational quote is randomly selected from a static list and displayed whenever the user closes all the panes on the page. Also added a quotes.js file with an array of quote objects and a function that returns a random quote object when called.
Edit: Forgot to add double quotes to the quote blocks after moving from test strings to random quote objects from the quotes.js file. Updated to PR to include HTML encoded left and right double quotes.

@Bouncey, thank you for checking out my PR and all of your notes. I'll make the requested changes ASAP!

@Bouncey, I was able to find a way to change the quote when the panes are opened and closed again, but not when the window is resized. Also added some code to ensure that the same quote isn't shown twice in a row. However, it might not be the cleanest way to handle those things.
Let me know what you think and I'll be happy to make any other changes necessary.

LGTM @scissorsneedfoodtoo :+1:
Thanks!

@Bouncey, thank you for all the code reviews and the merge!


Update mobile navigation

1. Update navigation.hbs

Add conditional for hidden property on navigation__link. This has been done for the sign in link to avoid flash of button when logged in. Hide the link, if logged out, show it, if logged in, just replace it.
Add aria-hidden to mobile-navigation__link chevron icon
Delete extra whitespace in mobile-navigation__link

2. Update navigation_component.js

Require template for userAvatarTemplate and userLinkTemplate
Update shop notification for mobile and desktop navs; add matchMedia to change class and remove text for mobile
Add label for ARIA to notification component
On _handleClick method, add preventDefault for links that have "#" as href
If there's not a user.id, show navigation__link by removing hidden prop
Refactor how userPanel and userAvatar get appended for desktop and add them to mobile navigation

3. Update notification.hbs

Add aria-label to markup

4. Update user_panel.hbs

Remove avatar component; moved to it's own partial
Remove hardcoded "sub-navigation" class so that it can be changed in JS for mobile or desktop
Remove extra whitespace
Add else condition to user.unread_message_count so that notification wrap isn't used when there is no notification

5. Update header.hbs

Add space inside of images conditional
Add js-header-mobile class to header__mobile; this is the target for the hamburger notification icon

6. Add user_avatar.hbs

Avatar component has been moved out of user_panel.hbs because it's not needed on mobile and this allows user_panel.hbs to be used in both mobile and desktop

7. Add user_link.hbs

This is the equivalent of user_avatar.hbs but for mobile; we just need the profile link and not an avatar

8. Add mobile_notification_badge.hbs

This is the notification badge that gets placed on the hamburger icon. Since it's being added with JS, this template keeps the markup out of the JS.

9. Update lib/helpers/nav_class_names.js

The items with js- classes needed to be updated. Due to the way this works, the mobile classes aren't correct with the js- classes. Mobile classes become mobile-js- which breaks any functionality tied to the item. This file doesn't seem to change anything as there is a Ruby file that changes the classnames, but this file was updated to stay in sync with the Ruby file.
The ruby file is lib/handlebars_helpers.rb

10. Update header_component.js

Import navigation state to get number of cart items; set state
Import subscribe decorator and add subscribe method
Require mobile_notification_badge.hbs
Add appendMenuIcon method for mobile_notification_badge.hbs template; this method shows the notification dot on the hamburger menu if there's a user notification or an item in the cart

11. Update notification.js

Add label option for ARIA

12. Update notification.scss

Refactor .notification-badge--shop and -inline to remove overriding styles by -inline
Add styles for .notification-badge--mobile-menu

13. Update navigation.scss

Add active and focus states to .navigation__link
Add hidden property to .navigation__link; this is for the sign in link

14. Update sub-navigation.scss
Update styles for .mobile-sub-navigation

Remove horizontal margin on .sub-navigation__item
Add horizontal padding to .sub-navigation__link
Add active and focus states to __link and __item--current __link
Remove horizontal padding on .sub-navigation-feature
Update margin-right and width for .sub-navigation-feature__image
Update padding-top for .sub-navigation-feature__text

15. Update mobile-navigation.scss

Update padding on .mobile-navigation
Add height of 100vh to .mobile-navigation
Remove top border from .mobile-navigation__list
Remove margin and padding reset from .mobile-navigation__list because this is done in the reset.css
Add position relative to .mobile-navigation__item for notification
Remove bottom border on last .mobile-navigation__item
Change a to .mobile-navigation__link; less specific
Add styles for .mobile-navigation__link

Related: https://github.com/lonelyplanet/destinations-next/pull/612

:+1:


Unbreak Travis build

There's a comment thread related to the openjdk10 build failure. In this particular case the issue is triggered when attempting to download junit:junit:jar:4.13-SNAPSHOT from the Sonatype Nexus Snapshots repository.
(Just leaving this comment here in case others building on top of Error Prone hit the same issue. But this may be a good opportunity to ask: are there plans to replace that SNAPSHOT dependency with a released artifact?)

@Stephan202 thanks--I think we're seeing the same issue. I'm testing a work-around in #1162.

are there plans to replace that SNAPSHOT dependency with a released artifact?

https://github.com/junit-team/junit4/issues/1496 :)

I fixed it in 7afde3efa95084147e36fcbf4c4b98f2a7e4d1c3.

@cushon hah, feeling your pain; I opened a post-4.12 PR nearly four years ago (junit-team/junit4#1054) :).
(But at this point Google may consider migrating to the more active JUnit 5 development?)


Translate Debug Running Pod into Chinese

Translate Debug Running Pod into Chinese

whoops, we have a conflict here, @fancc

/lgtm
/approve


fix(react): 🐛 avoid creating browser-specific objects in SSR environment

Description
When using SSR, following error will be throwed:
 ⨯ Internal error: ReferenceError: IntersectionObserver is not defined
    at eval (./node_modules/.pnpm/@lottiefiles+dotlottie-react@0.5.9_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@lottiefiles/dotlottie-react/dist/index.js:1812:5)
    at Object.oN [as useMemo] (/home/sheey/proj/node_modules/.pnpm/next@14.2.2_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:12:68806)

useMemo hook is used to create IntersectionObserver, which will be excuted on server, causing the error.
https://github.com/LottieFiles/dotlottie-web/blob/d59b55d859cafe7c2d2474bf0e9f9e675542ef86/packages/react/src/use-dotlottie.tsx#L82-L96
A solution is to delay the assignment by replacing useMemo with useEffect.
Type of change

[x]  Patch: Bug (non-breaking change which fixes an issue)
[ ]  Minor: New feature (non-breaking change which adds functionality)
[ ]  Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] This is something we need to do.

Sounds good!
I will change it in next commit.

I wonder if there's another bug...
https://github.com/LottieFiles/dotlottie-web/blob/88326766887b315078010c862d455839e23ae86f/packages/react/src/use-dotlottie.tsx#L61
https://github.com/LottieFiles/dotlottie-web/blob/88326766887b315078010c862d455839e23ae86f/packages/react/src/use-dotlottie.tsx#L160-L171
since the dependency list is empty, dotLottie will remain its first value null, meaning the instance will never be destroyed.

I wonder if there's another bug...
https://github.com/LottieFiles/dotlottie-web/blob/88326766887b315078010c862d455839e23ae86f/packages/react/src/use-dotlottie.tsx#L61
https://github.com/LottieFiles/dotlottie-web/blob/88326766887b315078010c862d455839e23ae86f/packages/react/src/use-dotlottie.tsx#L160-L171
since the dependency list is empty, dotLottie will remain its first value null, meaning the instance will never be destroyed.

The dotLottie is created only once when the canvas ref is initialized, which is always going to happen before the useEffect cleanup


Feature Request: Config Option for Adding Prefix to Class Names

Hi all. Firstly thanks for the amazing work.
Feature Request
This issue is a request to add an option in the daisyui config to allow customize the class names with a prefix.
Rationale
This stems from my most recent project where I came across this issue: the words used for naming components in daisy is widely used. This results in two problems:

If daisy has to coexist with another styling system (in our case, we were in the process of migrating bootstrap to tailwind), there will be conflict.

Due to the generous nature of Tailwind's default extractor, if someone is working in a React .jsx file and has the word modal in some comment like the following, for example:
function Component() {
  // this is meant to be my custom modal <-- modal is here and not in class attribute
  return (
...
);
}

Tailwind will include the .modal class in the output. This results in unused css and unoptimized build. Having a prefix for all components and their associated utilities will help with this.

Proposed Implementation
Please look at the pull request linked to this issue for details. There're two parts to this.
The Config Option
We can use postcss-js with the postcss-class-prefix plugin to add prefix on demand in user's tailwind.config.js:
const postcssJs = require('postcss-js');
const postcssPrefix = require('postcss-class-prefix');

//...

const prefix = config('daisyui.prefix');
if (prefix) {
  file = postcssJs.sync(postcssPrefix(prefix))(file);
}

Documentation Site Improvement
In the pull request, I also added instruction about this new option in the config page. But more importantly, I feel like the prefix should be a user-changeable global variable (similar to the theme dropdown) that is actually reflected in the class table and the html code examples.

The change to html codeblocks to support dynamic prefix requires further discussion. Right now my implementation includes a custom svelte action that replaces $$ with the prefix value coming out of a global store. This requires the following change to every pre block that wants to support this:

{
`<button class="$$btn">Button</button>
<button class="$$btn $$btn-primary">Button</button>
<button class="$$btn $$btn-secondary">Button</button>
<button class="$$btn $$btn-accent">Button</button>
<button class="$$btn $$btn-ghost">Button</button>
<button class="$$btn $$btn-link">Button</button>`
}

As this requires a large refactor for all endpoints in the components path, I'm waiting for your discussion and confirmation before including them all in the PR. Right now only the button endpoint is done.
This solution requires work now (which I am happy to do) but will require low maintenance later. I was thinking about having a dictionary for all the classes from daisy and do a one-place-search-and-replace-all type of wrapper/component/action but this comes with a runtime penalty for performance.
How well is this tested?
The first thing I tested was the intellisense of VSCode Tailwind, which works well

So far I have not encountered any breaking change. But please let me know if you think otherwise or found any problem.
Potential Issues & Further Discussion
postcss-js has released 4.0.0 version but it requires postcss 8.3. Unfortunately postcss-clean does not support newer version of postcss (yet?) and will break build step. Eventually I went with 3.0.3 for postcss-js
I think at some point we should switch to cssnano to keep up to date with the postcss ecosystem. Just my opinion that is open for discussion.

It seems that I don't have permission to add myself as the assignee. Please help with that

@saadeghi I see that the PR was merged and you did the hard work of refactoring all the pre blocks - thanks a lot for that. Just so you know I also put the PrefixEdit input in the sidebar to support responsive smaller screens. Maybe we should remove that too?

Yeah I missed that. Thanks.
I got a problem on Tailwind Play after the merge tho: https://play.tailwindcss.com/G6pFZss5N0
I think it's because of the dependency issue you mentioned.
I'm adding postcss (latest) as a main dependency. I'm alos replacing postcss-clean with Parcel CSS

I had to disable the prefix config for now. The problem was not with the dependencies. It was with the prefixer plugin.
I tried postcss-class-prefix and postcss-prefixer but as soon as I import them in index.js (even without using the function) Tailwind Play stops working and throws an error with no useful information.
I'm not sure about the source of the problem.
I found other postCSS plugins too but they add wrapper classes instead of prefixing (.prefix .class. not .prefix-class)

@saadeghi. I see. Thanks for that. I'll spend some time with this and see what I can do.

Version 2.3.5 uses postcss-prefixer
Version 2.3.1 uses postcss-class-prefix
But I had to disable the config on 2.3.6

@saadeghi very interesting, do you recall what the error was? The Tailwind Play you shared is having syntax problem with @2.3.5. It would be useful if Tailwind would have supported loading plugin at a specific version. This is what I am seeing:

I tried to clone Tailwind Play and ran it locally and injected daisy, but can't reproduce this. There was one error but it appeared on tailwindplay master HEAD without anything related to daisy:

You can try it out with the (branch daisyui-prefix-reproduction) I made here.
Make sure you have the daisy repo in the same parent directory as I load daisy dynamically from there as seen here:
parent
  |__ daisy
  |__ play.tailwindcss.com

And of course may sure u turn on the postcss-prefixer import and processing from daisy side.
Let me know. Thanks again for spending time with this. It is greatly appreciated!

Tailwind play does allow loading plugins from a specific version. You can see here:

❌ 2.3.5: https://play.tailwindcss.com/JG5zShMaN2?file=config
✅ 2.3.6 https://play.tailwindcss.com/JG5zShMaN2?file=config
I also downloaded Tailwind Play source. npm run dev give the error you mentioned but npm run build works
(live editor doesn't work but loading plugins work)

And I realized they return every error from plugins as Cannot find module. When I return the real error value, Its:
TypeError: Super expression must either be null or a function, not undefined

And that's not helpful either.

Sounds somewhat like a babel problem. I didn't know they support version-specific import, sorry about that. Will explore this more when I have some free time.

@saadeghi would you mind taking a look at the most recent PR. I bumped all of the postcss-related dependencies and make the imports dynamic because from the error we've got it seems that there is some versioning conflict or some cyclical dependencies. I hope this PR can help in such case?
Also Is there a strategy to maybe do a pre-release or just a branch to test before a tag release? I noticed we can also do this in tailwind play:
module.exports = {
  plugins: [
    require('daisyui#commit-hash'),
  ],
}

Thanks!

@vnphanquang There's pre-release tags but for some reason they don't work as I expect. I'm not sure what's the problem.
For other packages, when it's 1.2.3-alpha.0, installing the latest ignores the 1.2.3 and installs 1.2.2 instead but I don't know why it doesn't work for me.
Anyway. Result is still the same: https://play.tailwindcss.com/PyeOctAwGV?file=config

Tailwind Play loads these module dynamically through skypack. We can reproduce this problem by simply doing this in a browser console:
let im = import('https://cdn.skypack.dev/postcss-prefixer')
// or im = import('https://cdn.skypack.dev/postcss-class-prefix')

All the postcss prefix plugins should end up with the same error like the one you found:
TypeError: Super expression must either be null or a function, not undefined

I tried digging for a while but still could not understand exactly why. I suspect it is most likely because they all import old version of postcss and use its old plugin api; but I didn't have enough time to set up an environment to reproduce & verify this.
Anyway, the latest PR is another attempt. This time I basically copied the source of postcss-prefixer into a module in our source and use the new postcss plugin api.
Please take a look. Thanks @saadeghi

🥳 Works! https://play.tailwindcss.com/SP8HyC9kx5?file=config
Thanks @vnphanquang
I will add the components to document site.

Finally haha!

Amazing work @vnphanquang and @saadeghi, this is going to be a major boost for adoption.  It will make it very easy incrementally adopt Daisy UI to replace an existing library.

@vnphanquang @saadeghi when can we have the prefix option back in daisyui?

@vnphanquang @saadeghi when can we have the prefix option back in daisyui?

It should be there @latest. Is it not working for you?
See docs here

@vnphanquang thanks for the quick response!! I tried it and it didn't work :(

Hmmm. Can I see your config object? Does the default class names still work or is everything breaking? Some dev servers require restart to apply tailwind config changes FYI.

`module.exports = {
darkMode: 'class',
content: [
'./pages//*.{js,ts,jsx,tsx}',
'./components//*.{js,ts,jsx,tsx}',
],
theme: {
},
plugins: [require("daisyui")],
// daisyUI config
daisyui: {
styled: true,
themes: ["light", "dark", "cyberpunk"],
base: true,
utils: true,
logs: true,
rtl: false,
prefix: "ds",
darkTheme: "dark",
},
}`
I'm using it with nextjs. But I don't think it has anything to do with nextjs setup. Everything is followed from
https://tailwindcss.com/docs/guides/nextjs

Hey @hengsok sorry about this late reply. Things seem to be working fine for me in this tailwind play demo.
I don't see anything wrong with your config either.
Are you positive you're using @latest (maybe check package-lock.json) and are writing the names correctly from your markup (if prefix is ds, for example, then you need to write dsbtn not ds-btn)?
Let me know if none of the above helps, I'll setup a nextjs to try and reproduce this, although I doubt it's because of nextjs.

@vnphanquang oh wow you solve the problem straight away! You are so kind to help. Thanks so much.
So the issue is we should call 'dsbtn' instead of 'ds-btn'. It is either my fault in understanding the doc or the doc is a bit unclear there. https://daisyui.com/docs/config/
Perhaps, you can suggest to the author to update it.
In the doc, it says " If you use daisyUI prefix option (like daisy-)", so I thought we write 'daisy' as prefix and call it as 'daisy-'. Because this is the case for tailwind prefix. In tailwind, we set as 'tw' and call it as 'tw-'

@vnphanquang ok my bad, sorry. I understand it wrongly when I was a bit in a rush to set this up for my project. Nonetheless, you are awesome! Have a great day and thanks for contributing to open source. Amazing work there!

No worries @hengsok . Glad to see this option is being used. Good day to yourself!

Quick question, I added the prefix option in my tailwind.config.cjs and everything works great except for the prose style from @tailwindcss/typography. For some reason the text colors are different now. Any ideas on how to fix this?

@ehildebrandtrojo This is a special case. when we use prefix, all class names coming from daisyUI get a prefix.
However, daisyUI has some styles to make prose colors work with daisyUI themes. This way, prose class name is being used by both daisyUI and Typography plugin.
And because of that, the only way to make it work in this scenario, would be using both prose and [PREFIX]prose class names together. One of them get the style from Typography plugin and the other one applies the style coming from daisyUI:
https://play.tailwindcss.com/nZm7MkCpoL
This is not easy to fix because if the prose class, coming from daisyUI totally ignores the prefix, it still will cause problems if Tailwind CSS uses a prefix and daisyUI doesn't use a prefix.

Thank you for the quick reply and the explanation @saadeghi, I did not know that. Originally I just did [PREFIX]prose and it didn't work, leading me to make the above comment. Knowing I just needed to add prose in order to get it to work is essentially what I was looking for when I said 'have it working out-of-the box'.


Invalid: afterAll() may not be used in a describe block containing no tests.

Just updated to jest 27.0.3 and got the following error:
 Invalid: afterAll() may not be used in a describe block containing no tests.

describe('something', () => {
	beforeAll(async () => {
		// [...]
	});

	afterAll(async () => {
        // [...]
    });

    for (const data of testData) {
    	it(`Test ${data.testName}`, () => {
    		// [...]
    	})
    }
});

I have the tests generated in a for loop, as shown. Any idea how to fix this?
jest: ^27.0.3
ts-jest: ^27.0.1

you should use it.each api from jest

Your loop should be inside the describe block, not outside of it.

Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.


Dwarf Fortress: help SDL_ttf find freetype

Closes #418.
Closes #429.

This is a much better solution assuming it works! :)

It Works For Me (TM) but let me know if you have issues. :)


fix #4137

fix #4137
set $base-font-weight to 400

Thanks @TitanSnow! Can you change this to 500 as @nlhkabu suggested here? https://github.com/pypa/warehouse/issues/4137#issuecomment-402778101

https://github.com/pypa/warehouse/blob/dc120b9c0f8fba9dc98809e68e1b56a1d1bda5e6/warehouse/templates/base.html#L57
But it loads Source Sans Pro with weight 400, 600, 700, no 500. If $base-font-weight is set to 500, the font weight will be 500 before the web font is loaded and be 400 after the web font is loaded, which makes a blink. In addition, other common-used fonts just not have a weight 400 so there's no difference between 400 and 500 for them.

Thanks for pointing that out. It also looks like Source Sans Pro is not available from Google Fonts at a 500 weight. I'll let @nlhkabu review, but this looks good to me.

Oh, well, sorry that I was massed. 400 is the normal weight, while 500 is the weight between normal and bold. I guess few fonts have weight 500.

thanks @TitanSnow :)


fix: allow local development hosts in TrustedHostMiddleware

Allow All Local Development Hosts in TrustedHostMiddleware
Problem
The application was returning 400 Bad Request errors when accessed through localhost or other local development URLs. This was caused by the TrustedHostMiddleware's restrictive allowed_hosts configuration, which only permitted production domains.
Solution
Updated the TrustedHostMiddleware configuration to include all common local development hostnames and a wildcard pattern. The allowed hosts now include:

localhost
<IP_ADDRESS>
* (wildcard for all hosts)

This change maintains the security benefits of TrustedHostMiddleware in production while allowing for easier local development.
Changes

Modified main.py to expand the allowed_hosts list in TrustedHostMiddleware configuration
Kept existing production hosts (gitingest.com, *.gitingest.com, gitdigest.dev)
Added local development hosts and wildcard pattern

Testing

Verified server starts successfully with uvicorn
Confirmed 200 OK responses for:

Main application (http://localhost:8000)
Static files (/static/js/utils.js, /static/favicon.ico)

Notes

The wildcard pattern (*) should be removed for production deployments if strict host checking is required
Consider adding a configuration variable to control allowed hosts based on environment (development/production)

Using a .env file for allowed hosts could be a good idea to keep the configuration adaptable and avoid hardcoding URLs in the app. What do you think? @hannesrudolph @cyclotruc

I agree

Me too, that is what I will do

Closing because it's solved in  https://github.com/cyclotruc/gitingest/pull/31


[SPARK-46356][BUILD] Upgrade sbt-assembly to 2.1.5, sbt-checkstyle-plugin to 4.0.1

What changes were proposed in this pull request?
The pr aims to upgrade sbt-assembly from 2.1.1 to 2.1.5.
Why are the changes needed?
1.sbt-assembly, the full release notes:

v2.1.5: https://github.com/sbt/sbt-assembly/releases/tag/v2.1.5
v2.1.4: https://github.com/sbt/sbt-assembly/releases/tag/v2.1.4
v2.1.3: https://github.com/sbt/sbt-assembly/releases/tag/v2.1.3
v2.1.2: https://github.com/sbt/sbt-assembly/releases/tag/v2.1.2

2.sbt-checkstyle-plugin v4.0.0 VS v4.0.1
https://github.com/stringbean/sbt-checkstyle-plugin/compare/v4.0.0...v4.0.1
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

It sounds good if it passes the CIs.
BTW, could you check other plugins' versions too, @panbingkun ? You don't need to include them into this PR. I just want to know if we have many outdated plugins or we are blocked by something.

Okay, let me check all the plugins.

Merged to master.


Prepare Premal's yeast data for use with RiboViz

https://www.ncbi.nlm.nih.gov/geo/query/acc.cgi?acc=GSE124428
Requires UMI handling and deduplication.

This requires demultiplexing, so will be our test real dataset for #84. Perhaps also test dataset for #38
@FlicAnderson to

write regexp for UMI-tools extract
downsample to test dataset with 1mi reads, see #38.

We would need this done by 20 November.

We have now made these substantial changes, including refactoring #32, so we would like to test this dataset and ideally add a config.yaml to example-datasets.
To discuss with @shahpr and @john-s-f at 10th June meeting.

I've been trying to run the nextflow pipeline on some data and am getting this error when I try to use nextflow run prep_riboviz -params-file test.yaml --validate-only
Unparseable date: "2020-06-09 10:45:45"
That's the incorrect time and it never changes. I've attached the error log, perhaps this has something do without respective locales? At least, this post suggests that (nextflow-io/nextflow#962). Perhaps @mikej888 knows?
nextflow_date_error_log.txt

@john-s-f I've never seen that error! When you say "resolved itself" did you try any of the suggestions from nextflow-io/nextflow#962? Did you change the content of your test.yaml file? Or did you any of the files it mentions? (anything that might have changed a timestamp)

Initially it was installed in my home directory but as that is quite low on space, I moved it somewhere else. I have a feeling this had to do with whether or not I used sudo with the quick install scripts. I ran them without sudo and that seems to have fixed it. I can attempt to figure it out if you'd like.

Hi @john-s-f. Thanks for your help on this ticket. We think it may be too hard to reproduce.
Are you able to add this to the example datasets?

@john-sf. I can't see how the location where you installed Nextflow, or whether or not you used sudo, would give rise to this error, given that the discussion on nextflow/issues/962 relate to the files being accessed by Nextflow rather than where/how Nextflow was installed. We discussed this at a meeting today and think that it might be time-consuming to try and reproduce the error. Also we have no idea as to how many others may encounter this (if any!) So we're OK to leave it as an issue that resolved but sorted itself for now.


Unpack support

Added initial Sentinel 1 support (need to add preprocessing)
Added batch data unpack tool for WV and S1
Possibly fixed epoch processing bug

Is this ready to merge? Also how intensive is this preprocessing? Maybe we should do it before it goes to DELTA if it takes significant time and involves writing files?

There is an unpacking script which I run before training which is the same as we are doing for Worldview.  At this point merging can probably wait until we see how the results are.

Sounds great, thanks!

Ready to merge now.


:bug: Fix empty code snippet in XML when the match is a key of the node, not an inner value

Fixes #503

@shawn-hurley added


Error when building ad3 on Windows 8.1 (64bit) + MSVC 9

Hi,
I've searched this error and understand that it's due to the header file for timing in ad3. But this problem still exists when building on Windows 8.1 (64bit) + MSVC 9. Could you fix this on your ad3 fork (which I believe is automatically downloaded when installing with pip install pystruct command)?  Or is there any workaround? Below is the error message stub:
Installing collected packages: ad3
  Running setup.py install for ad3
    building 'ad3' library
...
FactorGraph.cpp

C:\Users\zzz\AppData\Local\Programs\Common\Microsoft\Visual C++ for Python\9
.0\VC\Include\xlocale(342) : warning C4530: C++ exception handler used, but unwi
nd semantics are not enabled. Specify /EHsc

ad3/FactorGraph.cpp(22) : fatal error C1083: Cannot open include file: 'gettimeo
fday.h': No such file or directory

error: command 'C:\\Users\\zzz\\AppData\\Local\\Programs\\Common\\Microsoft\
\Visual C++ for Python\\9.0\\VC\\Bin\\amd64\\cl.exe' failed with exit status 2

----------------------------------------
Cleaning up...
...

Have you checked if this is fixed in AD3 master? Then I can try to sync it.

Good news, I just installed ad3 manually by git cloning from its master branch and it worked well.

That is great. I need to update my package then.


Nmap.quiet not working.

I could be doing something wrong here, but I have tested as much as I possibly can and have come to the conclusion that nmap.quiet is not working properly.
  def run_host_discovery(cidr)
    Nmap::Program.scan do |nmap|
      nmap.quiet = true
      nmap.targets = cidr
      # TODO: Get rid of Nmap output
      nmap.ping = true
      nmap.xml = "#{@scan_time}/#{cidr.sub('/', '-')}.xml"
    end
  end

I get a long list of hosts that are alive and from my understanding this should quiet all output from nmap.

Ensure your nmap supports -v0


Paket run multiple times for multi project sln file

Description
when running dotnet build in directory containing sln file with two projects paket restore is being run multiple times and showing error
Microsoft (R) Build Engine version <IP_ADDRESS>362 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Paket version 5.163.2
  Paket version 5.163.2
  C:\projects\temp\pakettest\src\PaketTest1\obj\paket.locked is locked by paket.exe (PID = 5304). Waiting...

dotnet --info:
.NET Core SDK (reflecting any global.json):
 Version:   2.1.300-rc1-008673
 Commit:    f5e3ddbe73

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.16299
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.300-rc1-008673\

Host (useful for support):
  Version: 2.1.0-rc1
  Commit:  eb9bc92051

Repro steps
mkdir pakettest
 cd .\pakettest\
dotnet new classlib -lang f# -o src/PaketTest1
dotnet new classlib -lang f# -o src/PaketTest2
dotnet new sln
dotnet sln add .\src\PaketTest1\PaketTest1.fsproj
dotnet sln add .\src\PaketTest2\PaketTest2.fsproj
cd .\src\PaketTest1\
dotnet add reference ..\PaketTest2\PaketTest2.fsproj
 cd ..\..
 mkdir .paket
download paket bootstrapper to .paket
.\.paket\paket.exe convert-from-nuget
dotnet build
test repository https://github.com/landy/pakettest
Expected behavior
no error shown
Actual behavior
after running dotnet build error is shown
dotnet build
Microsoft (R) Build Engine version <IP_ADDRESS>362 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Paket version 5.163.2
  Paket version 5.163.2
  C:\projects\temp\pakettest\src\PaketTest1\obj\paket.locked is locked by paket.exe (PID = 17256). Waiting...

Same here, running "dotnet build" for solution with few projects never finishes. Parameters --no-restore or --disable-parallel doesn't show any effect.
C:\Projects\%MyProjectName%>dotnet build %MyProjectName%.sln
Microsoft (R) Build Engine version <IP_ADDRESS>362 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Paket version 5.164.0
  Paket version 5.164.0
  Paket version 5.164.0
  Paket version 5.164.0
  Could not acquire lock to C:\Projects\%MyProjectName%\docs\Storage\UbiqLanguage\obj\paket.locked.
  The process cannot access the file 'C:\Projects\%MyProjectName%\docs\Storage\UbiqLanguage\obj\paket.locked' because it is being used by another process.
  Trials left: 99.
  C:\Projects\%MyProjectName%\docs\Storage\UbiqLanguage\obj\paket.locked is locked by paket.exe (PID = 4072). Waiting...
  C:\Projects\%MyProjectName%\docs\Storage\UbiqLanguage\obj\paket.locked is locked by paket.exe (PID = 4072). Waiting...  ```

@matthid this should be fixed in latest after one paket install?

tested it with latest Paket and it is working correctly, thanks


[Xero] Kill the XeroOnNewExpensify & AccountingOnNewExpensify betas

👋 I've added * to the XeroOnNewExpensify beta in prep to do the same with AccountingOnNewExpensify to reveal the Accounting page for access to both Xero and QBO when everything is deployed in the next couple of days.
I'm creating this issue to just remind us to kill these betas at some point next week when the dust settles and everything is released. 👍

Xero was released to all yesterday, so I'm going to settle up with the C+ team that worked on this project. Payment summary as follows:
@mananjadhav @hungvu193 @rushatgabhane  - $5k each for both contributor and contributor plus duties across all issues for the Xero project. Hans, I've sent an offer on Upwork. Manan and Rushat, you can request.

Thanks, I've requested the payout.

Accepted the offer. Thank you 🙏

Created a manual request. Thank you!!

$5,000 approved for @mananjadhav

$5,000 approved for @rushatgabhane

@mountiny I reckon we can move forward and kill these betas this week. You still up for helping on that?

@trjExpensify sure
@mananjadhav or @rushatgabhane or @hungvu193 either of you like to create an App PR to remove these betas from App? I can remove it from the backend after they are gone from App for couple of releases

hey yeah should be simple

I created a PR here @mountiny

App PR merged and Web PR created https://github.com/Expensify/Web-Expensify/pull/42479
I am going to get that merged only once the App changes are in production

That's on prod now, right? So can we get the Web PR merged?

That's on prod now, right? So can we get the Web PR merged?

It is, but I would like to wait a bit to make sure users have time to update the app. If the beta is removed and user has an old client, the connection would not show up for them.
I know many real customers still not using this but just to be safe do not want to rush it

Cool, sounds good!

Payment won't be due here, right? But we're still holding on this WebPR of Vit's before closing this out.

yep

Yeah I think we can merge it tomorrow

Cool, let's get it merged today then. 👍

Web PR hit staging 4 hours ago, hopefully we auto-close after tomorrow's deploy.

Didn't auto-close, but closing.


want to add button with hover effect.

Details:
It's a button I will make using HTML & CSS and look good on HOVER.
PLEASE ASSIGN THIS ISSUE TO ME.

solved on pr  #485 please review and merge!


Add ack for worker connection

adding test for worker connection.
Details is issue: https://github.com/locustio/locust/issues/2044

@heyman What do you think?
Would it be possible to make a test case that really tests the functionality? (maybe by patching master to discard the first connect message)
Is this backwards compatible, assuming an old worker discards the ack? I'm thinking mainly of other worker implementations like @myzhan 's Boomer

it requires fixes for sure. I'm working on it

@Nosibb Did you see my response?

@Nosibb ?

@cyberw
I remember about this. I was busy :) I'll finish it next week

No rush, I just wanted to make sure you were't waiting for me or something :)

Let me know what do you think about the new test.
In my opinion, it is backwards compatible. In older versions, we will see a warning. Is it enough?

@heyman Any comments? Looks reasonable to me but I didnt originally write the connection logic.
I’d prefer the constants be named CONNECT_* instead of CONNECTION_* (to indicat that they relate to connection setup, not once connection has been enabled)

Thx! I’ll make sure to bump the minor version when I do the next release.


Social Workers unable to edit Beanstalk 2020 event registration

The website encounters a server error when trying to edit registration for this event.
@Acolli327 spoke to a SW who encountered this issue and grabbed some details:

I just wanted to pass on an error report from a social worker, Isolina Dacosta, from the New Bedford Area Office. She has been trying to register a couple of her kids for the Beanstalk event all day, but she keeps getting an error message when trying to updating the registration to add a sibling. I’m sorry I didn’t catch all of it, but she said it was telling her that the “page could not be served” and there was something like an error key CLI from Heroku. I logged into get to the event page and add them in the back end, to save her some time and I got a brief error page that came up citing Heroku, but I hit refresh before I thought to grab a screen shot.

It was around 4:00 pm on 2/3/2020, she called me while she was in the middle of trying to register the kids, trying different approaches, to no avail.

I tried to replicate the error or at least accurately describe the part where she was having the trouble. I logged into my social worker test email and unfortunately I do not have any test child records to work with but when I registered for the event and then typed in one name, I got the attached error message when I went back into the event registration and tried to add a second sibling. The issue was definitely occurring in the “Edit Registration” section in the Beanstalk Event.

Good news - I tracked down the source of this issue and it should be an easy fix.  One of the children that is registered for the Beanstalk event doesn't have a recruitment worker associated with their record, which is causing some of the Edit Registration logic to fail.  Not sure if that's something that should be fixed in the child record as well (let me know if so and I can direct someone to the record), but I can deploy a code fix to production tonight if that sounds reasonable.  @joesmare please advise.


Publish 8.6-20240120014516+0000

Publish 8.6-20240120014516+0000

@bot-gradle test and merge

The merge queue build has started. Click here to see all failures if any.

The merge queue build has started. Click here to see all failures if any.


chore: add lower version constraint to Terraform Google provider

add lower version constraint to Terraform Google provider

@apeabody Cloud you PTAL?
in  a PR in another repository you asked to upgrade
- required_version = ">= 0.13"
+ required_version = ">= 1.3"

does this change need to be applied in this repo too?

@apeabody Cloud you PTAL?
in a PR in another repository you asked to upgrade
- required_version = ">= 0.13"
+ required_version = ">= 1.3"

does this change need to be applied in this repo too?

It doesn't have to be in this PR, but it should be done eventually.

/gcbrun

(changed to fix to trigger release)


Height unavailable

Which OS ?
MIUI
Version
Which versions are you using:

react-native-swiper v?
react-native v0.?.?

Expected behaviour
Actual behaviour
How to reproduce it>
To help us, please fork this component, modify one example in examples folder to reproduce your issue and include link here.
Steps to reproduce

Not enough info


Configure Zerotier devicemap

Describe the bug
Zerotier networks can be joined declaratively, but the name of the interfaces created seem to be non-deducible from the network ID (and maybe non-deterministic?). This makes it challenging to address the Zerotier interface in other parts of config, such as networking.firewall.interfaces.<name>.allowedTCPPortRanges.
There exists a mechanism to provide a key=value mapping of network IDs to interface names at /var/lib/zerotier-one/devicemap, although I can't seem to find any official documentation on this, only forum, GH, and SE posts referencing this functionality.
Steps To Reproduce
Steps to reproduce the behavior:

Join networks using services.zerotierone.joinNetworks.
Try to reference the created interface in networking.firewall.interfaces.<name>.
Be sad that there isn't a declarative way to obtain the interface name.

Expected behavior
Perhaps there should be either a services.zerotierone.devicemap attrset option, or more intuitively the joinNetworks option could be extended to allow <network-id>=<interface-name> entries which behave as a <network-id> entry currently does, while also adding <network-id>=<interface-name> to /var/lib/zerotier-one/devicemap.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers
@sjmackenzie @zimbatm @ehmry @obadz @danielfullmer

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.40, NixOS, 24.05 (Uakari), 24.05.20240716.2af21f1`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - nixpkgs: `/nix/store/fannjg1hnhs76s8sin8dnddiqnwv6dky-source`

Add a :+1: reaction to issues you find important.

the name of the interfaces created seem to be non-deducible from the network ID (and maybe non-deterministic?)

There's actually a mapping from the network ID to the interface name as defined in the zerotier code under osdep/LinuxEthernetTap.cpp.   It's deterministic except for the unlikely case of a name collision with another interface.
In case it's of interest to you, I had previously replicated this same logic in Nix in my personal configs, mostly for fun to see if I could do the hex math in Nix.   See the ifrname function from: https://github.com/danielfullmer/nixos-config/blob/05222bb666808b3330f060f49c044161e4e24b0d/profiles/zerotier.nix#L2-L43


Failed to load url isomorphic-git

Describe the bug
12:09:29 AM [vite] Pre-transform error: Failed to load url isomorphic-git (resolved id: isomorphic-git) in /app/app/lib/hooks/useGit.ts. Does the file exist?
app-dev-1  | 12:09:29 AM [vite] Pre-transform error: Failed to load url isomorphic-git/http/web (resolved id: isomorphic-git/http/web) in /app/app/lib/hooks/useGit.ts. Does the file exist?
app-dev-1  | 12:09:32 AM [vite] Error when evaluating SSR module /app/lib/hooks/useGit.ts: failed to import "isomorphic-git"
app-dev-1  | |- Error: Cannot find module 'isomorphic-git' imported from '/app/app/lib/hooks/useGit.ts'
app-dev-1  |     at nodeImport (file:///app/node_modules/.pnpm/vite@5.4.11_@types+node@22.10.1_sass-embedded@1.81.0_sass@1.77.6/node_modules/vite/dist/node/chunks/dep-CB_7IfJ-.js:53047:19)
app-dev-1  |     at ssrImport (file:///app/node_modules/.pnpm/vite@5.4.11_@types+node@22.10.1_sass-embedded@1.81.0_sass@1.77.6/node_modules/vite/dist/node/chunks/dep-CB_7IfJ-.js:52914:22)
app-dev-1  |     at eval (/app/app/lib/hooks/useGit.ts:8:37)
app-dev-1  |     at instantiateModule (file:///app/node_modules/.pnpm/vite@5.4.11_@types+node@22.10.1_sass-embedded@1.81.0_sass@1.77.6/node_modules/vite/dist/node/chunks/dep-CB_7IfJ-.js:52972:5)
Link to the Bolt URL that caused the error
https://github.com/coleam00/bolt.new-any-llm
Steps to reproduce
Steps to Reproduce in a Docker Environment**

Prepare the Docker Environment:

Ensure you have Docker installed and running.
Verify the Docker version:docker --version

Example output:Docker version 24.0.6, build ed223bc

Clone the Repository or Access the Project:

Clone the repository or navigate to your project directory:git clone <repository-url>
cd <project-directory>

Verify the Dockerfile:

Open the Dockerfile and ensure it includes steps to install dependencies. For example:FROM node:18-alpine
WORKDIR /app
COPY package.json ./
COPY package-lock.json ./
RUN npm install
COPY . .
CMD ["npm", "run", "dev"]

Make sure the isomorphic-git dependency is listed in package.json.

Build the Docker Image:

Run the build command:docker build -t my-app .

Run the Docker Container:

Start the container:docker run -p 3000:3000 my-app

Trigger the Issue:

Open your browser and navigate to http://localhost:3000.
Perform actions that invoke the code in useGit.ts.

Observe the Error:

Note any errors in the container logs:docker logs <container-id>

Example output:Cannot find module 'isomorphic-git' imported from '/app/app/lib/hooks/useGit.ts'

Expected behavior
run bolt,new in my browser
Screen Recording / Screenshot
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Provider Used
No response
Model Used
No response
Additional context
No response

pnpm install isomorphic-git
pnpm install
That's will add it to your package.json which should be then updated in the main repo.

this is a duplicate #630


Added new config param animate_duration

Added ability to control animate duration via config param animate_duration

Thank you very much for your PR.
What is your use case? It is too fast or too slow for you? Or do you just want to add customization?

Thank you very much for your PR. What is your use case? It is too fast or too slow for you? Or do you just want to add customization?

it's a little bit slow for my experience :) wanna reduce to 100 ms

running tests:

waiting for approve, merge and release 😏

I took the liberty of fixing some minor issues and rewrite the commit message to adhere to conventional commits.

It's on the pre-release v1.12.0. Do you confirm it's working?

yes, just switched to pre-release version on my  ha instance, set duration to 100 ms - it works as expected 🎉

and i'm happy now :)

Me too, and thank you very much for your contribution! Much appreciated 😊


Rake tests fail on MongoDB version 3.6.8

When running bundler exec rake I got two errors. Attached is the stdout of that command:
tests.txt (if you cat it, it will look nice with colors just as with rake)
When trying to get to the bottom of it, I found out that after MongoDB version 3.2, a new version of indexes is being used https://docs.mongodb.com/manual/core/2dsphere/ . I have MongoDB version 3.6, so that could be the cause, but apparently your travis tests are passing with a MongoDB version 3.2 so I'm not so sure
One possible fix would be to force the creation of Version 2 indexes. Or to just ignore the version number when doing the check on the tests. Or to check against which version to compare, according to the MongoDB version

I think the first step is to reproduce this in CI. Make a pull request that swaps the version of MongoDB? I believe you should be able to have a set of tests use 3.2 and another use 3.6 within the same .travis.yml.


fix: knative kafka extensions are short enough to meet the SHOULD req in spec

Fixes #3992

Proposed Changes

Shorten the extensions

Release Note

BREAKING CHANGE: the knative kafka partition/offset ce extensions have been renamed to knkafkapartition and knkafkaoffset to keep them under 20 characters in length

/cc @matzew @Leo6Leo @aliok

/retest

/hold
I'm worried this will break users, given that it is a should is it really worth it? I think the JS SDK should not throw for this as it's a SHOULD

I'm worried this will break users, given that it is a should is it really worth it? I think the JS SDK should not throw for this as it's a SHOULD

Sure, let me try opening a PR to the SDK to fix that

I opened https://github.com/cloudevents/sdk-javascript/pull/593

@Cali0707 closing.
the fix in JS seems better addressing this SHOULD.
/close


"Run in Jupyter" in nbgallery is not working

When trying to run a notebook in jupyter from my nbgallery App, i get an error:
Run failed
Could not send notebook to https://<IP_ADDRESS>:443/.
The Jupyter server is running and i can run the notebook directly using https://<IP_ADDRESS>:443 in my browser . But when trying to do the same from nbgallery it does not work. (The "Jupyter Enviroment" is set to that same address)
Any clues or hints that could point me in the right direction would be much appreciated
B r
/Junior

Sorry, I overlooked this issue.  Were you able to get it working?  My first suggestion would be to make sure that both nbgallery and Jupyter are running secure https (or both insecure http) - the web browser won't allow mixed content.  Also check your Jupyter configuration settings.

Oh, right, you opened an issue in the nbgallery repo too.  I'm going to close this one as a duplicate.

No problema, thanks anyway...:-)
From: Jeff<EMAIL_ADDRESS>Sent: den 8 september 2019 00:03
To: nbgallery/jupyter-alpine<EMAIL_ADDRESS>Cc: Junior Alvarez<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [nbgallery/jupyter-alpine] "Run in Jupyter" in nbgallery is not working (#23)
Sorry, I overlooked this issue. Were you able to get it working? My first suggestion would be to make sure that both nbgallery and Jupyter are running secure https (or both insecure http) - the web browser won't allow mixed content. Also check your Jupyter configuration settingshttps://protect2.fireeye.com/url?k=d81c8360-84c884cc-d81cc3fb-8667c4afe13e-75bce0f013eb2964&q=1&u=https%3A%2F%2Fgithub.com%2Fnbgallery%2Fnbgallery%2Fblob%2Fmaster%2Fdocs%2Fjupyter_integration.md%23manual-configuration.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://protect2.fireeye.com/url?k=48218b95-14f58c39-4821cb0e-8667c4afe13e-149fce7f5abec1f6&q=1&u=https%3A%2F%2Fgithub.com%2Fnbgallery%2Fjupyter-alpine%2Fissues%2F23%3Femail_source%3Dnotifications%26email_token%3DAMLD2DTTPM2WA2FW2PFTOIDQIQQLBA5CNFSM4H3HCXQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6FDEFA%23issuecomment-529150484, or mute the threadhttps://protect2.fireeye.com/url?k=830e1896-dfda1f3a-830e580d-8667c4afe13e-019ca0e5afdca080&q=1&u=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMLD2DWOEK7KK2S3OZI6HR3QIQQLBANCNFSM4H3HCXQA.


Fix "Ios return path" issue

Fixes #22. Modified the source for ios to fix the path returned by the plugin on ios devices after saving the image to the gallery.

hi everyone. the error seems to be fixed and the image is saved to gallery but i'm not getting back the url (in the result I only get "Image Saved to Gallery"). is this the expected plugin behaviour?

the latest version of this plugin which fixes the said issue is 4.1.3. please make sure to update your cordova plugin dependency :)


Disappearing div and span

I'm afraid we have an issue with whatever is inside a div, somehow some tags are disappearing.
If we edit this HTML by just removing the comma:

Mangiare, è incorporare un territorio.
 Jean Brunhes

it will be saved as:

Mangiare è incorporare un territorio.
 Jean Brunhes

Is there a way to be keep all the tags and just change the inner text of them?
May be related to https://github.com/rainlab/editable-plugin/issues/40

@phaberest this plugin is a proof of concept and not officially supported. If you would like any support for it then you'll have to pay for our time. Otherwise you can check out the alternatives https://octobercms.com/plugin/samuell-contenteditor and https://octobercms.com/plugin/netsti-editor

Thank you @LukeTowers, I just replaced it with the netsti and it now works fine.
By the way, would you accept a PR if I fix it in my spare time?

Sure, as long as it's relatively easy to understand the fix :)


Bump django_money version back to 3.2.0, Fix #5825

makemigrations didn't cause any changes, so I guess the migrations are fine after all? Can we somehow test this works with the docker builds without merging this @SchrodingersGat ?
Edit: This fixes #5825

Can we somehow test this works with the docker builds without merging this?

Yes, add the pull_request trigger to the docker workflow by just uncommenting only this line:
https://github.com/inventree/InvenTree/blob/master/.github/workflows/docker.yaml#L23

Thanks! Let's see how this goes ...

@30350n nice, looks like the CI check passed. If you can update the workflow again to disable docker builds for pull request, this looks good to go

@30350n nice, looks like the CI check passed. If you can update the workflow again to disable docker builds for pull request, this looks good to go

Done!


[CHAD-8941] Changed the range of Z-Wave dimmers to [1,100]

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

@bflorian does this look kosher to you?
@Bhavik-T I think you're on the right track. This particular profile is for a zigbee dimming remote, but I do think we'll want to apply the [1,100] range to all profiles which include the Switch Level preference.

Hey @bflorian Can you please check this?

@greens @bflorian updated for the model changes. Not sure why the unit tests are failing, I just changed the profile YMLs

@ctowns when you get time could you update this PR to align to the new format

@ctowns could you resolve the conflict with the inovelli-dimmer profile?


Critical Error

Expected Behavior

Bot starts and runs
Actual Behavior

Critical Error
Your FULL config.json (remove your username, password, gmapkey and any other private info)

Output when issue occurred

2016-09-13 13:27:00,854 [       cli] [CRITICAL] Error with configuration file
Tue Sep 13 13:27:00 EDT 2016 Pokebot  Stopped.
Press any button or wait 20 seconds to continue.
2016-09-13 13:27:21,685 [       cli] [INFO] PokemonGO Bot v1.0
2016-09-13 13:27:21,691 [       cli] [INFO] commit: c9c5332f
./configs/config.json:146:12: Warning: Object contains duplicate key: u'// evolve_list'
|  At line 146, column 12, offset 4126
|  Object started at line 138, column 18, offset 3848
./configs/config.json:147:12: Warning: Object contains duplicate key: u'// donot_evolve_list'
|  At line 147, column 12, offset 4164
|  Object started at line 138, column 18, offset 3848
./configs/config.json:271:4: Warning: Strict JSON does not allow a final comma in an array (list) literal
|  At line 271, column 4, offset 7729
|  Array started at line 9, column 13, offset 197
./configs/config.json:356:6: Warning: Object contains duplicate key: u'// Zubat'
|  At line 356, column 6, offset 10790
|  Object started at line 343, column 15, offset 10061
./configs/config.json:358:6: Warning: Object contains duplicate key: u'// Zubat'
|  At line 358, column 6, offset 10910
|  Object started at line 343, column 15, offset 10061
./configs/config.json: ok, with warnings
2016-09-13 13:27:21,798 [       cli] [CRITICAL] Error with configuration file
Tue Sep 13 13:27:21 EDT 2016 Pokebot  Stopped.
Steps to Reproduce

fresh dev install
Other Information
OS:
Mac

Branch:
Dev

Git Commit:

Python Version:

Any other relevant files/configs (eg: path files)

I will change the values and submit a pull

where is your config file?
check it here : http://jsonlint.com


.NET MAUI Windows 9.0 Preview 5 on WinUI crashes when using SwapChainPanel via SkiaSharp in a game after a while

Describe the bug
We are using .NET MAUI 9.0 Preview 5, .NET SDK 9.0 Preview 5, and SkiaSharp 3.0 Preview 3.1 in our game, GnollHack. SKGLView of SkiaSharp uses SwapChainPanel to render GPU accelerated graphics on Windows/WinUI3. Things work fine for a while, but then the game crashes with the following exception (found in the Output→Debug window):
Exception thrown at 0x00007FFB236EF39C (KernelBase.dll) in GnollHackM.exe: WinRT originate error - 0x80004005 : 'Reentrancy was detected in this XAML application. Use a debugger to locate the reentrant code and, if necessary, move that code to an asynchronous event handler. Press OK to exit the application.'.
Microsoft.ui.xaml.dll!00007FFA8F1137D8: CallContext:[\ImageDecodeActivity] 8000FFFF - E_UNEXPECTED
Unhandled exception at 0x00007FFA8F4A52F5 (Microsoft.ui.xaml.dll) in GnollHackM.exe: 0xC000027B: An application-internal exception has occurred (parameters: 0x0000022A8ED086C0, 0x0000000000000003).
This is in the Debug mode, but it happens also in the Release mode.
Steps to reproduce the bug

Compile .NET MAUI Windows port using these instructions:
https://github.com/hyvanmielenpelit/GnollHack/wiki/Development-Information
(It's a relatively big task, I warn you.)

Start the game and enable GPU acceleration in the settings. Play the game for a while until it crashes. 10 minutes is usually enough.

Expected behavior
The game would not crash.
Screenshots

Stack Trace
     GnollHackM.dll!GnollHackM.GamePage.PaintMainGamePage(object sender, SkiaSharp.Views.Maui.SKPaintSurfaceEventArgs e) Line 7517    C#
     GnollHackM.dll!GnollHackM.GamePage.canvasView_PaintSurface(object sender, SkiaSharp.Views.Maui.SKPaintSurfaceEventArgs e) Line 3860    C#
     GnollHackM.dll!GnollHackM.SwitchableCanvasView.internalGLView_PaintSurface(object sender, SkiaSharp.Views.Maui.SKPaintGLSurfaceEventArgs e) Line 213    C#
     [External Code]
     GnollHackM.dll!GnollHackM.SwitchableCanvasView.InvalidateSurface() Line 75    C#
     GnollHackM.dll!GnollHackM.GamePage.UpdateMainCanvas() Line 1325    C#
     GnollHackM.dll!GnollHackM.SwitchableCanvasView.OnPropertyChanged(string propertyName) Line 352    C#
     [External Code]
     GnollHackM.dll!GnollHackM.SwitchableCanvasView.GeneralAnimationCounter.set(long value) Line 331    C#
     GnollHackM.dll!GnollHackM.GamePage.StartMainCanvasAnimation.AnonymousMethod__415_0(double v) Line 1611    C#
     [External Code]

NuGet package version
None
Windows version
No response
Additional context

Windows 11 Pro 23H2. 64-bit. 22631.3737.
Visual Studio 2022 Preview 17.11.0 Preview 2.0.
.NET MAUI 9.0 Preview 5
.NET SDK 9.0 Preview 5
SkiaSharp 3.0 Preview 3.1

It turned out that we had to add DispatcherQueue.TryEnqueue to our drawing routines. We had to use the window's DispatcherQueue for it to be performant.


interface function debug_error_string  of class ClientContext  can not get error message

grcp-cpp
What version of gRPC and what language are you using?
version 1.17.2
What operating system (Linux, Windows,...) and version?
linux centos 7.3
What runtime / compiler are you using (e.g. python version or version of gcc)
gcc version 4.8.5
What did you do?
If possible, provide a recipe for reproducing the error. Try being specific and include code snippets if helpful.
I write a code snippet, it is a rpc, to watch a key of etcd, like that:
std::shared_ptr<grpc::ClientReaderWriter<WatchRequest,WatchResponse>> stream
	(watchServiceStub_->Watch(&context));

	if (stream->Write(req))
	{
		stream->WritesDone();

		//read for events
		while(true)
		{
			if (stream->Read(&resp))
			{
				if (resp.events_size() > 0)
				{
					bool found = false;

					for (int32_t i = 0; i < resp.events_size(); i++)
					{
						const ::mvccpb::Event& event = resp.events(i);
						if (event.type() == watch_type)
						{
							found = true;
							break;
						}
					}

					if (found)
					{
						break;
					}

				}

				if (resp.canceled())
				{
					break;
				}
			}
			else
			{
				grpc::string strErr = "read watch response fail with reason: ";
				strErr += **context.debug_error_string()**;
				Status ReadErr(::grpc::INTERNAL, strErr.c_str());

				return ReadErr;
			}
		}
	}
	else
	{
		grpc::string strErr = "write watch request fail with reason: ";
		strErr += **context.debug_error_string()**;
		Status WriteErr(::grpc::INTERNAL, strErr.c_str());

		return WriteErr;
	}

when enter else procession, use debug_error_string to get error message fail?
questions:

the function works in what enviroment?

if it does not work , any other interface can instead of it?

What did you expect to see?
get some tips to get error message from the function
What did you see instead?
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

sorry, it fixed, thank you


Master Branch

Hi,
It wont optimize master branch images.. its operate seperate branch only not to master branch

Hey @dkarthe thanks for opening an issue here.
I'm not sure I totally get what the problem is you are reporting. Can you link to a specific example?

Hmmm... I came to the issues tab to talk about something similar to this.
https://github.com/jasikpark/jasik-xyz/ has only received pull requests from ImgBot for the master branch and not the blog branch (the branch that actually has media I want to compress).
I just removed ImgBot from my github account and reinstalled it with permissions for jasikpark/jasik-xyz so this might change hopefully.

I believe @jasikpark and possibly @dkarthe are talking about the same things, we basically want to be able to set which branch we want imagebot to optimize images from. For instance, I would like my develop branch to be the one checked rather than master which receives infrequent commits, whereas develop is often being worked on and new images are being added.

thanks @danner26 for clarifying :)
We have an open issue #155 where we had some conversations a while back - is this in line with what you were thinking? Want to chime in over there?


chore(package): update kronos-interceptor to version 2.5.1

Codecov Report

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

@@           Coverage Diff           @@
##           master     #225   +/-   ##
=======================================
  Coverage   97.54%   97.54%
=======================================
  Files           1        1
  Lines         122      122
=======================================
  Hits          119      119
  Misses          3        3

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 3f4d558...64a391d. Read the comment docs.


Improve callback function support

Right now we are generating:
# typedef void ( * pmix_event_notification_cbfunc_fn_t ) ( pmix_status_t status , pmix_info_t * results , size_t nresults , pmix_op_cbfunc_t cbfunc , void * thiscbdata , void * notification_cbdata )
const pmix_event_notification_cbfunc_fn_t = Ptr{Cvoid}

And I find myself writing:
function notification_fn(evhdlr_registration_id::Csize_t, status::PMIx.API.pmix_status_t, source::Ptr{PMIx.API.pmix_proc_t},
                         info::Ptr{PMIx.API.pmix_info_t}, ninfo::Csize_t,
                         results::Ptr{PMIx.API.pmix_info_t}, nresults::Csize_t,
                         cbfunc::PMIx.API.pmix_event_notification_cbfunc_fn_t, cbdata::Ptr{Cvoid})::Nothing
...
end

callback = @cfunction(notification_fn, Cvoid, (Csize_t, PMIx.API.pmix_status_t, Ptr{PMIx.API.pmix_proc_t}, Ptr{PMIx.API.pmix_info_t}, Csize_t, Ptr{PMIx.API.pmix_info_t}, Csize_t, Ptr{PMIx.API.pmix_event_notification_cbfunc_fn_t}, Ptr{Cvoid}))

The type annotation in notation_fn is not necessary, but I do find it helps with these long signature.
I was thinking of defining a custom struct, but that requires @cfunction($) and that is not fully supported everywhere.
So maybe a macro that makes the cfunction easier?

Since the function signature that @cfunction needs is very tedious to write, I think we can export the Cvoid, (Csize_t, PMIx.API.pmix_status_t, Ptr{PMIx.API.pmix_proc_t}, Ptr{PMIx.API.pmix_info_t}, Csize_t, Ptr{PMIx.API.pmix_info_t}, Csize_t, Ptr{PMIx.API.pmix_event_notification_cbfunc_fn_t}, Ptr{Cvoid}) part as some constant pmix_event_notification_cbfunc_fn_t_signature, then we can write
callback = @cfunction(notification_fn, $(pmix_event_notification_cbfunc_fn_t_signature...))

But this won't help you write the callback function itself.

I'm thinking of doing something like this:
julia> function foo_callback(x::Csize_t)::Cint
           __impl_foo_callback(x)
       end
foo_callback (generic function with 1 method)

julia> cb = @cfunction(foo_callback, Cint, (Csize_t,))  # this should be generated in `__init__()`
Ptr{Nothing} @0x0000000111e59180

julia> ccall(cb, Cint, (Csize_t,), 0)
ERROR: UndefVarError: __impl_foo_callback not defined
Stacktrace:
 [1] foo_callback(x::UInt64)
   @ Main ./REPL[1]:2
 [2] top-level scope
   @ ./REPL[3]:1

julia> __impl_foo_callback(x) = x + 1
__impl_foo_callback (generic function with 1 method)

julia> ccall(cb, Cint, (Csize_t,), 0)
1

In this way, users can define __impl_foo_callback without annotating argument types.


C++ unit test failure under GPU config: //tensorflow/core:ops_array_grad_test

Using Python 2, Linux. This failure doesn't occur under the CPU config
bazel test -c opt --config=cuda //tensorflow/core:ops_array_grad_test
Error log:

I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcublas.so.7.0 lo
cally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcudnn.so.6.5 loc
ally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcufft.so.7.0 locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcuda.so locally
I tensorflow/stream_executor/dso_loader.cc:99] Couldn't open CUDA library libcurand.so.7.0. LD_LIBRARY_PATH:
I tensorflow/stream_executor/cuda/cuda_rng.cc:333] Unable to load cuRAND DSO.
Running main() from test_main.cc
[==========] Running 10 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 10 tests from ArrayGradTest
[ RUN      ] ArrayGradTest.PackGrad
I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:900] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
I tensorflow/core/common_runtime/gpu/gpu_init.cc:102] Found device 0 with properties:
name: Quadro K620
major: 5 minor: 0 memoryClockRate (GHz) 1.124
pciBusID 0000:02:00.0
Total memory: 2.00GiB
Free memory: 1.57GiB
I tensorflow/core/common_runtime/gpu/gpu_init.cc:126] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_init.cc:136] 0:   Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:713] Creating TensorFlow device (/gpu:0) -> (device: 0, name: Quadro K620, pci bus id: 0000:02:00.0)
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 1.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 2.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 4.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 8.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 16.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 32.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 64.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 128.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 256.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 512.0KiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 1.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 2.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 4.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 8.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 16.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 32.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 64.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 128.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 256.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 512.00MiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 1.00GiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:51] Creating bin of max chunk size 2.00GiB
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:73] Allocating 1.37GiB bytes.
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:83] GPU 0 memory begins at 0xb011c0000 extends to 0xb58f68000
[       OK ] ArrayGradTest.PackGrad (65832 ms)
[ RUN      ] ArrayGradTest.UnpackGrad
I tensorflow/core/common_runtime/gpu/gpu_device.cc:713] Creating TensorFlow device (/gpu:0) -> (device: 0, name: Quadro K620, pci bus id: 0000:02:00.0)
[       OK ] ArrayGradTest.UnpackGrad (4 ms)
[ RUN      ] ArrayGradTest.ConcatGrad
I tensorflow/core/common_runtime/gpu/gpu_device.cc:713] Creating TensorFlow device (/gpu:0) -> (device: 0, name: Quadro K620, pci bus id: 0000:02:00.0)
external/bazel_tools/tools/test/test-setup.sh: line 51: 112821 Segmentation fault      (core dumped) "$@"

Known -- test is currently not expected to run on GPU.  Will keep it open until we fix.

Just a related question --- If test is not run under GPU (CUDA) configuration, how can we convince ourselves that CUDA kernels for several TF operations do work correctly?

@vrv, @zffchen78: Is this still an issue?

Should have been closed.

Known -- test is currently not expected to run on GPU. Will keep it open until we fix.
Does the issue be solved?


Additional updates to  Security Update Support for Pub

This adds additional improvements for Security Update Support for Pub beyond the basic implementation over in:

#6230

This is now ready for review... there's a couple of TODOs that I expect after further discussion probably will be handled as separate PR's, but in the meantime this is ready for review / discussion

Confirmed working locally using dry-run:
[dependabot-core-dev] ~/dependabot-core $ SECURITY_ADVISORIES='[{"dependency-name":"retry","patched-versions":[],"unaffected-versions":[],"affected-versions":["<= 3.0.2"]}]' bin/dry-run.rb pub org/test-repo --cache=files --dep=retry

=> reading cloned repo from /home/dependabot/dependabot-core/tmp/dsp-testing/dependabot-all-updates-test-staging
=> parsing dependency files
I, [2022-11-26T02:24:01.438558 #6936]  INFO -- : Installing the Flutter SDK version: 3.3.9 from channel stable with Dart 2.18.5
I, [2022-11-26T02:24:01.438688 #6936]  INFO -- : Checking out Flutter version refs/tags/3.3.9
I, [2022-11-26T02:24:04.025229 #6936]  INFO -- : Running `flutter doctor` to install artifacts and create flutter/version.
I, [2022-11-26T02:25:58.674844 #6936]  INFO -- : Running `flutter --version`
I, [2022-11-26T02:26:00.915668 #6936]  INFO -- : Installed the Flutter SDK version: 3.3.9 with Dart 2.18.5.
=> updating 1 dependencies: retry

=== retry (2.0.0) (vulnerable 🚨)
 => checking for updates 1/1
I, [2022-11-26T02:26:01.273093 #6936]  INFO -- : Installing the Flutter SDK version: 3.3.9 from channel stable with Dart 2.18.5
I, [2022-11-26T02:26:01.273206 #6936]  INFO -- : Checking out Flutter version refs/tags/3.3.9
I, [2022-11-26T02:26:02.841201 #6936]  INFO -- : Running `flutter doctor` to install artifacts and create flutter/version.
I, [2022-11-26T02:26:07.200580 #6936]  INFO -- : Running `flutter --version`
I, [2022-11-26T02:26:09.369335 #6936]  INFO -- : Installed the Flutter SDK version: 3.3.9 with Dart 2.18.5.
 => latest available version is 3.1.0
 => earliest available non-vulnerable version is 3.1.0
 => latest allowed version is 3.1.0
 => requirements to unlock: own
 => requirements update strategy:
 => updating retry from 2.0.0 to 3.1.0
I, [2022-11-26T02:26:10.227446 #6936]  INFO -- : Installing the Flutter SDK version: 3.3.9 from channel stable with Dart 2.18.5
I, [2022-11-26T02:26:10.227632 #6936]  INFO -- : Checking out Flutter version refs/tags/3.3.9
I, [2022-11-26T02:26:12.050901 #6936]  INFO -- : Running `flutter doctor` to install artifacts and create flutter/version.
I, [2022-11-26T02:26:15.777269 #6936]  INFO -- : Running `flutter --version`
I, [2022-11-26T02:26:17.972313 #6936]  INFO -- : Installed the Flutter SDK version: 3.3.9 with Dart 2.18.5.

    ± pubspec.yaml
    ~~~
    6c6
    <   retry: ^2.0.0 # Can update with updated constraint, no further constraints.
    ---
    >   retry: ^3.1.0 # Can update with updated constraint, no further constraints.
    ~~~

    ± pubspec.lock
    ~~~
    38c38
    <     version: "2.0.0"
    ---
    >     version: "3.1.0"
    ~~~
🌍 Total requests made: '0'

In order to keep the Security Updates work on track, I split out the straightforward work as an initial PR in:

#6230

And then stack this PR on top of it, with the additional questions/notes about transitiive deps, the unclear exception raising, and unit test improvements.


Change endpoint

for https://github.com/Samedis-care/identity-management-backend/pull/23

is app here the id (required) or the name?

is app here the id (required) or the name?

ID


Reduce allocations in UnpackDomainName by better sizing slice

This simple change greatly improves the performance and memory consumption of UnpackDomainName. It's perfectly safe to size s this large as it never escapes and will be stack allocated.
After this PR, UnpackDomainName will never be the source of memory garbage.
In terms of number of allocations and allocated memory, this is absolutely the best possible results, i.e. UnpackDomainName can never be more efficient with regards to memory usage.
name                                   old time/op    new time/op    delta
UnpackDomainName-12                       169ns ±13%     167ns ±19%     ~     (p=0.955 n=10+10)
UnpackDomainNameUnprintable-12            150ns ±13%     160ns ± 6%   +6.61%  (p=0.015 n=10+10)
UnpackDomainNameLongest-12               1.45µs ±30%    0.74µs ± 7%  -48.74%  (p=0.000 n=10+9)
UnpackDomainNameLongestUnprintable-12    6.05µs ±16%    3.21µs ± 7%  -47.00%  (p=0.000 n=10+10)

name                                   old alloc/op   new alloc/op   delta
UnpackDomainName-12                       64.0B ± 0%     64.0B ± 0%     ~     (all equal)
UnpackDomainNameUnprintable-12            48.0B ± 0%     48.0B ± 0%     ~     (all equal)
UnpackDomainNameLongest-12                 640B ± 0%      256B ± 0%  -60.00%  (p=0.000 n=10+10)
UnpackDomainNameLongestUnprintable-12    2.94kB ± 0%    1.02kB ± 0%  -65.22%  (p=0.000 n=10+10)

name                                   old allocs/op  new allocs/op  delta
UnpackDomainName-12                        1.00 ± 0%      1.00 ± 0%     ~     (all equal)
UnpackDomainNameUnprintable-12             1.00 ± 0%      1.00 ± 0%     ~     (all equal)
UnpackDomainNameLongest-12                 3.00 ± 0%      1.00 ± 0%  -66.67%  (p=0.000 n=10+10)
UnpackDomainNameLongestUnprintable-12      5.00 ± 0%      1.00 ± 0%  -80.00%  (p=0.000 n=10+10)

/cc @semihalev (for https://github.com/miekg/dns/issues/836#issuecomment-442413416 & https://github.com/miekg/dns/pull/839#discussion_r237073104)

Codecov Report

Merging #844 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #844      +/-   ##
==========================================
- Coverage   57.69%   57.65%   -0.04%
==========================================
  Files          43       43
  Lines       10856    10856
==========================================
- Hits         6263     6259       -4
- Misses       3504     3507       +3
- Partials     1089     1090       +1

Impacted Files
Coverage Δ

msg.go
78.77% <100%> (-0.65%)
:arrow_down:

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 fa58975...b1b868a. Read the comment docs.

thanks, looks good.


Write typescript code in vue3 setup template without any hint

Without any hint
The desired results are as follows

Automatic import is expected

Duplicate of #823


leam-base.jackson-databind.mid-5577.idx-138418.8.mutant

This changes code needs more review

@Mariam22-hub please revise this PR as well as all others to only change formatting not the code behavior.

It's okay that you closed this PR this time, but in the future please revise existing PRs rather than closing them and opening new ones.


Any work to publish package on Maven Central ?

As you mention here, we have to modify our settings.xml file to be able to use Github packages. But I don't want to use this kotlin package in this way. That's why I wonder that do you have some future plan to publish this package on maven central ? @dslmeinte

Apologies @dagdelenmustafa : your issue slipped through. It wasn't my intention to let you go unanswered.
The simple answer is: would be good to have this, but haven't found any time yet to get it working. (It's a bit trickier than e.g. publishing an NPM package.) I reopened this issue, as it's a good reminder.

Hi @dslmeinte,
No problem at all and thanks for your reply. I look forward to new updates.

Closed per PR #124


added setting to disable automatic strigify of pdo mysql

Fixes https://github.com/Shardj/zf1-future/issues/362 as prescribed by
hungtrinh

@roshanbudhathoki seems like some Unit tests are failing after the change. Could you pls review?

@develart-projects  I am getting following error.
Testing fetchLatersVersion only works when TESTS_ZEND_VERSION_ONLINE_ENABLED is set.
Am I missing to setup something. Even unsure where do I setup this.

@roshanbudhathoki  here is my local testing command:
php /usr/share/NetBeansProjects/ZF1-Future/bin/phpunit --colors --log-junit /var/tmp/nb-phpunit-log.xml --configuration /usr/share/NetBeansProjects/ZF1-Future/tests/phpunit.xml /usr/share/NetBeansProjects/ZF1-Future/tests/AllTests.php

@develart-projects  I have fixed and updated the code, should work now. Please approve so that CI can run and give us the test results.


Universal Support

Hey, thanks for this starter, i try so many starter but best this, have trouble adding universal support this starter u want add next release add universal support this starter ?

It's on the milestone, waiting for universal package going to @angular repo
https://docs.google.com/document/d/1rC5dz7AkCZP813daT_bccbS-20OLmXgBVi50CSKx148/pub

contributions / PR is welcomed!

https://github.com/angular/angular/issues/13822


Unicode support

Text written with utf8 encoding is garbled in the terminal with no apparent reason, no matter what terminal window is used. At least in konsole and x-terminal-emulator, on UTF8-enabled system.
Result in app:

[ ] x  +2x=0
[ ]      ~B ~G
[ ] duck:  ~_ ~F
[ ] normal text works

Text in TODO:
TODO: x²+2x=0
TODO: żółć
TODO: duck: 🦆
TODO: normal text works

At least in konsole and x-terminal-emulator

Today, I installed todo-rs and ran it from mate-terminal on Arch Linux  and can't reproduce this bug. Can you check the fonts/terminals you installed?

unfortunately I can't. It was more THAN ONE YEAR AGO, so sorry dude, it was a too-slow reply. I had another two versions of the system right now, since then, now it's on 23.4.
but...
I believe it was on MX Linux 23.1 or .2 system version, with xfce4 as a manager. Text was entered into a file with vim with no problems, displayed with no problem, cat/less/more were working and all chars were OK; and when it was displayed on the very same window from rust, it has this garbled content.
I think there may be something with terminal configuration, maybe some flag was missing in the program, dunno. Maybe there is a single another step required. And right now I unfortunately have other important issues to fight - so I can only return to that in the future, hopefully.


remove py33-twtrunk from travis

Codecov Report

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

@@           Coverage Diff           @@
##           master     #947   +/-   ##
=======================================
  Coverage   60.96%   60.96%
=======================================
  Files          71       71
  Lines       12664    12664
  Branches     2021     2021
=======================================
  Hits         7720     7720
  Misses       4286     4286
  Partials      658      658

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 1ed967a...1dc3aec. Read the comment docs.


[bug] /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.29' not found"

Describe the bug

Reproduction
No response
Expected behavior
No response
Platform and versions
ubuntu 20 build, ubantu 16 run app.

Stack trace
No response
Additional context
No response

Hi, this is a known limitation explained here: https://tauri.app/v1/guides/distribution/linux#limitations
You have to build on the oldest system you want to support.


[BUG] ModalBottomSheet NavHost animateContentSize issues

There appears to be some animation issues when using animateContentSize on a NavHost Modifier when the content is inside the material3 ModalBottomSheet. Here's the demo of how the same code behaves with the NavHost and without the NavHost.
Recording:
https://drive.proton.me/urls/P8STB5F53G#mZ9le5bjPJB3
Issues with comparison to code without NavHost:

Showing bottom sheet animation hangs for ~0,6 seconds at the bottom of the screen. By default, ModalBottomSheet finishes showing animation in around ~400ms, but with NavHost content and animateContentSize modifier it hangs at the bottom barely visible for ~600ms and then performs ~400ms showing animation.
Sheet doesn't animate alongside the Android keyboard (IME padding) but does it one after the other. E.g. when navigating inside the sheet to a route that has different size while hiding the keyboard, it first animates the content size of the new route and then hides the keyboard as two separate animations. For example, when not using NavHost compose animates both events at the same time (see example video).

Example code
@Composable
fun App() {
    Column(modifier = Modifier.fillMaxSize()) {
        var showBottomSheet by remember { mutableStateOf(false) }
        val sheetState = rememberModalBottomSheetState(skipPartiallyExpanded = true)
        if (showBottomSheet) {
            ModalBottomSheet(
                sheetState = sheetState,
                onDismissRequest = { showBottomSheet = false },
                content = {
                    val navigator = rememberNavigator()
                    NavHost(
                        modifier = Modifier.animateContentSize(),
                        navigator = navigator,
                        navTransition = NavTransition(),
                        initialRoute = "/sheet1",
                    ) {
                        scene(route = "/sheet1") {
                            Content(
                                modifier = Modifier.fillMaxWidth().height(200.dp),
                                title = "NavHost 1",
                                onClick = { navigator.navigate("/sheet2") }
                            )
                        }
                        scene(route = "/sheet2") {
                            Content(
                                modifier = Modifier.fillMaxWidth().height(350.dp),
                                title = "NavHost 2",
                                onClick = { showBottomSheet = false }
                            )
                        }
                    }
                }
            )
        }
    }
}

@Composable
private fun Content(
    modifier: Modifier,
    title: String,
    onClick: () -> Unit,
) {
    Column(
        modifier,
        horizontalAlignment = Alignment.CenterHorizontally
    ) {
        var field by remember { mutableStateOf("") }
        Text(title)
        TextField(
            value = field,
            onValueChange = { field = it })
        Button(onClick = { onClick() }) {
            Text("Next")
        }
    }
}

There are no issues/interference with the ModalBottomSheet animations without the animateContentSize Modifier on the NavHost, but unfortunately ModalBottomSheet doesn't animate its content height by default so without this modifier when navigating to routes with different heights the content simply jumps instead of showing nice animation.
Here's the demo code from the recording: https://github.com/tomczyn/PreCompose-Sample/tree/animations

The issue might be related to AnimatedContent, as PreCompose uses it internally to manage navigation transitions. A quick workaround could be to configure the navTransition so that it does not trigger any animations.

The issue with the keyboard is actually because when the content goes off the screen, the field that had a focus is still being composed for the animation. That's why defining no animation solves it, and with animation the keyboard is waiting until the end to be hidden. Adding manual clear focus through LocalFocusManager solves the issue. So the only real problem was animateContentSize which is now fixed.


Fix: Add token to wallet Issue 449

This fixes https://github.com/OlympusDAO/olympus-frontend/issues/449
Metamask was not opening the add token prompt because the image asset
was not passed as a string
This change replaces the image asset with a string path.

Superceded by: https://github.com/OlympusDAO/olympus-frontend/pull/448

oh dang. i just saw this. i liked this solution better. Kek.


Staking/Slashing on Abuse

Closes #51
Closes #62
Closes #88

[x] Update core smart contracts.
[x] Update Python SDK.
[x] Update Node.js SDK.
[x] Update Fortune.
[x] ~Update subgraph if necessary.~
[x] ~Update ETH KVStore GUI if necessary.~
[x] ~Update Escrow Dashboard if necessary.~

@leric7 please add tests to the proxies:

Check owner can update
Check no owner cannot update
Check updated contract still has same storage
.....


Progress bar with sliding control

feature request, or proposal:
A feature request for adding slide control in progress bar itself. Now we can set it programmatically binding a data value, but user couldn't change value by dragging. Slider doesn't have buffer line.
What is the expected behavior?
Ability to slide/change the progress value of progress bar
What is the current behavior?
A progress bar without an ability to change the value like a slider control.
What is the use-case or motivation for changing an existing behavior?
A very general use case will be a video/audio control where, user can go back or forward by sliding within a buffered zone. Could use slider but it doesn't have a buffer feature

This isn't really the intended interaction pattern for the progress bar (especially WRT accessibility). I recommend using a combination slider + progress bar.