gcc5 & gcc6 when MacOS::CLT not installed

I have a custom homebrew instance where I keep gcc5 and gcc6 for my test builds on macOS.
The homebrew instances are installed with several scripts, available from gist.
Recently I upgraded my mac to Sierra, and ran the install-homebrew-gcc.sh script, which completed properly.
But, when I tried to use the compilers, my build failed, claiming some system headers were not available.
After inspecting the formulas, I discovered this:
      unless MacOS::CLT.installed?
        # For Xcode-only systems, we need to tell the sysroot path.
        # "native-system-headers" will be appended
        args << "--with-native-system-header-dir=/usr/include"
        args << "--with-sysroot=#{MacOS.sdk_path}"
      end

In other words, if the Xcode Command Line Tools are not installed, the gcc build will not add the system headers. That was also my case, after the upgrade to Sierra, I installed Xcode, but forgot to install the new Command Line Tools.
The question is what would be the usability of the resulted incomplete compilers, since simple builds that include system headers will fail?
Suggestion:

make the formula depend on the presence of MacOS::CLT; if CLT is not installed, abort the build; for special cases, possibly add a --force, to build an incomplete compiler.

The question is what would be the usability of the resulted incomplete compilers, since simple builds that include system headers will fail?

The headers are in the Xcode SDK folder:
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk/usr/include

make the formula depend on the presence of MacOS::CLT

There's generally no reason to depend on the CLT because everything is available in Xcode itself too. brew doctor will warn you about the non-generally cases.

sorry, I don't understand your answers, and I'm afraid you did not understand the problem.
if you install homebrew and gcc on a system without CLT, you do not get any warnings or errors, and after half an hour of waiting to compile the sources, you get a completely useless compiler, that will fail complaining about missing system header files.
is this the expected/desired behaviour?
in my opinion this is definitely time lost, because after you install CLT you need to reinstall (rebuild) gcc again in homebrew, to have configure add the missing option.

you do not get any warnings or errors

You get a warning:
Warning: Building gcc from source:
  The bottle needs the Xcode CLT to be installed.

@ilg-ul this is on a system without the CLT:
Josephs-Mac:~ joe$ brew test -vd gcc
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FormulaLoader): loading /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/gcc.rb
Testing gcc
==> Using the sandbox
/usr/bin/sandbox-exec -f /tmp/homebrew20161115-93679-1utyse.sb /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby -W0 -I /usr/local/Homebrew/Library/Homebrew -- /usr/local/Homebrew/Library/Homebrew/test.rb /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/gcc.rb -vd
/usr/local/Homebrew/Library/Homebrew/test.rb (Formulary::FromPathLoader): loading /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/gcc.rb
==> /usr/local/Cellar/gcc/6.2.0/bin/gcc-6 -o hello-c hello-c.c
==> /usr/local/Cellar/gcc/6.2.0/bin/g++-6 -o hello-cc hello-cc.cc
==> /usr/local/Cellar/gcc/6.2.0/bin/gfortran -c in.f90
==> /usr/local/Cellar/gcc/6.2.0/bin/gfortran -o test in.o
Josephs-Mac:~ joe$

The tests pass.

if you install homebrew and gcc on a system without CLT, you do not get any warnings or errors, and after half an hour of waiting to compile the sources, you get a completely useless compiler, that will fail complaining about missing system header files.

What do you mean by "completely useless compiler, that will fail complaining about missing system header files"? Can you provide an example?

What do you mean by "completely useless compiler, that will fail complaining about missing system header files"? Can you provide an example?

my test is in C++ and uses <cassert>, which refers to <assert.h>, reported as missing by gcc-5 (and possibly 6, but I already deleted the bad install, so I cannot test).
so I suggest you try to include <assert.h>; if you built gcc with MacOS::CLT.installed == false, the --with-native-system-header-dir=/usr/include --with-sysroot=#{MacOS.sdk_path} were not defined, and the compile will fail.

this is on a system without the CLT:

@ilovezfs, could you run gcc -v and check if --with-native-system-header-dir=/usr/include --with-sysroot=#{MacOS.sdk_path} were present when you built the compiler?

@ilg-ul
Josephs-Mac:~ joe$ gcc-6 -v
Using built-in specs.
COLLECT_GCC=gcc-6
COLLECT_LTO_WRAPPER=/usr/local/Cellar/gcc/6.2.0/libexec/gcc/x86_64-apple-darwin16.1.0/6.2.0/lto-wrapper
Target: x86_64-apple-darwin16.1.0
Configured with: ../configure --build=x86_64-apple-darwin16.1.0 --prefix=/usr/local/Cellar/gcc/6.2.0 --libdir=/usr/local/Cellar/gcc/6.2.0/lib/gcc/6 --enable-languages=c,c++,objc,obj-c++,fortran --program-suffix=-6 --with-gmp=/usr/local/opt/gmp --with-mpfr=/usr/local/opt/mpfr --with-mpc=/usr/local/opt/libmpc --with-isl=/usr/local/opt/isl --with-system-zlib --enable-libstdcxx-time=yes --enable-stage1-checking --enable-checking=release --enable-lto --with-build-config=bootstrap-debug --disable-werror --with-pkgversion='Homebrew gcc 6.2.0' --with-bugurl=https://github.com/Homebrew/homebrew/issues --enable-plugin --disable-nls --enable-multilib --with-native-system-header-dir=/usr/include --with-sysroot=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk
Thread model: posix
gcc version 6.2.0 (Homebrew gcc 6.2.0)
Josephs-Mac:~ joe$

@ilovezfs,
as I guessed, your compiler was built with MacOS::CLT.installed.
the problem I am reporting occurs when you do it without MacOS::CLT.installed.
to be sure we are speaking about the same formula, I suggest you try it with gcc-5.

No, it's built with CLT uninstalled.
See the
 --with-native-system-header-dir=/usr/include --with-sysroot=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk

which is only passed when CLT is uninstalled.

which is only passed when CLT is uninstalled.

ah, yes, sorry, I understood it wrong.
anyway, I regret that I removed the version that did not work, since now I cannot reproduce the problem; I ran two gcc5 builds, first without CLT and second with CLT, and both seem functional.
I guess either something went wrong between Xcode 8.0 and Xcode 8.1, or removing Xcode was not enough to simulate the case when CLT were not installed; unfortunately I cannot tell what was wrong. :-(
so we'll leave it for now, and if I encounter the problem again I'll preserve the install folder, for further analysis.

You get a warning:

Warning: Building gcc from source:
  The bottle needs the Xcode CLT to be installed.

I checked and this warning was not issued. Something may still be wrong with the builds, but the stars are in a favourable position, and the compilers are functional.
For reference, the build consoles are packed in the attached archive.
gcc5-builds.zip


updating flask dependencies

Motivation and Context
raised because of security issues or bumping version.
Check not breaking loads of stuff
What has changed
Updated flask version from 2.3.3 to 3.0.0 and flask-babel from 3.1 to 4.0.0
How to test?
Run against docker-acceptance- test
Links
https://trello.com/c/ZJX4yG3D/1292-fix-dependabot-test-failures-in-the-new-rh-ui-5
Screenshots (if appropriate):

 Fix Dependabot test failures in the new RH-UI (5) KM


caught SyntaxError: Cannot use import statement outside a module

When I first import the project, it gives the error "require is not defined". When I click on "Quick Fixe", I get the error "not use import statement outside a module". How can I solve it?

go to your package.json, change the type property to module


Pico_timer.py missing

In your blog post, you refer to this file (for measuring frequency using the reciprocal method), but I don't see it in the main branch of the repository. Thanks for the great post.

@gregoryng - in the clone i just took now, there's https://github.com/jbentham/picofreq/blob/main/pico_timer.py in main, is that what you're looking for?

Yes, that is what I was looking for and I guess it was there all along.
I was originally browsing on mobile and I see now that github only shows the first few entries unless you click "view more" so maybe that's what happened -- me not noticing that in the web site.


Does our sealer support non-root users

What happened:
usually ,  not all user with root permission when he execute sealer command , so this will brings the following problems.
sealer images :failed, when local machine is not have sealer image.
sealer rmi,;sealer build;sealer pull ;sealer tag: cmd who need to write image or image metadata.
all of this error caused by  the same reason :  sealer images saved at '/var/lib' that its  permission  is 755, means non root can not write in.

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

sealer version (use sealer version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

I think there is not way to support this feature completely. unless we put sealer file system under /home/{user} @fanux

I think there is not way to support this feature completely. unless we put sealer file system under /home/{user} @fanux

agreed


Submit files and script reorgnization and cleanup

A successful run from this branch here: http://vdt.cs.wisc.edu/tests/20191029-1256/results.html. Highlights include:

Cleaned up lots of cruft that we don't need
Scripts for jobs and rundir generation moved to bin/
Submit files and DAGs moved to jobs/. This dir is the new working dir for the jobs, i.e. it's where the jobs expect their input and will dump their output

After this is merged, we'll need to:

Update osg-run-tests on osghost
Probably update @edquist's compare-rpm-versions and list-rpm-versions if they make any assumptions about a run's directory structure

Then update them on osghost
Move those scripts to this repo and include them in the vm-test-runs package

Why are you deleting the scripts and files needed for image generation (like prepare-images)? Do they exist elsewhere now?

Those scripts were mostly used for the initial vm-test-runs implementation. Base images are created off in CHTC somewhere: I'd have to dig to find where exactly and they're broken so they need to be overhauled. The input image creation is done by bin/create-io-images

Good from my point of view. Feel free to merge if you don't want to wait for Carl.


Serverurl empty error

Add mustMatch check logic in GetConfigServerUrl(string uri).
Add constructors in ServerUrl class.

working with @guo7711 to make few tweeks

nothing wrong with leaving a PR open and laying down more commits to get things perfect prior to merging, but either way... :smile:

@esoekianto #297 is the updated PR.

@guo7711 and @esoekianto thank you both for your quick fix!


Feature/add style cop analyzers

Fixes #253
Proposed Changes
Fix violations of these rules...
CS0105
CS0169
CS0414
CS0649
SA1025
SA1107
SA1216
SA1505
SA1507
SA1508
SA1509
SA1510
SA1512
SA1513
SA1515
SA1517
SA1518
SA1649
Pre-existing violations that I've not touched:
CS0618
CS1998
All other rules are either off or do not cause violations.
Once this is merged in we can think about turning on other rules, if we want to.

@binarymash man you are a hero!  I thought you were just gonna merge and then we would go rule by rule!!!
I assume codes in PR match up to http://stylecop.soyuz5.com/StyleCop Rules.html
Ill go through them and make sure I understand everything then merge.

Dont worry about CS0618 thats for me to sort at some point..
CS1998 really annoys me because you have an async API but some implementations are not async. If you wrap it with a task you are spinning up a task for no reason..is there a better way with this?

If you want me to fix the merge conflicts let me know. You have already done loads and I really appreciate it!!

Not sure about the CS1998 as I didn't really look into them. If it's not async then can you just you Task.FromResult(...)?
Actually, forgot to say before, there's still a couple of violations for CS0649 (variables which are being used but never assigned to). These are both in test code - not sure if the tests are incorrect, or what. Any ideas?

I'd imagine the CS0649 in the tests are being used in constructors or something but they get passed somewhere else...there is quite a lot of data in transit in the whole project.

@binarymash you happy to merge this?

yep.

let the arguments begin :)


Deletion repeating directory name from src path

ERROR in ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
Module not found: Error: Can't resolve '../../fonts/Roboto-Condensed/Roboto-Condensed/Roboto-Condensed-Regular.eot' in 'F:\Projects\roboto-fontface-bower\css\roboto-condensed'
@ ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css 6:953-1038
@ ../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
@ ./src/App.js
@ ./src/entry-client.js
@ multi webpack-hot-middleware/client ./src/entry-client.js
ERROR in ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
Module not found: Error: Can't resolve '../../fonts/Roboto-Condensed/Roboto-Condensed/Roboto-Condensed-RegularItalic.eot' in 'F:\Projects\roboto-fontface-bower\css\roboto-condensed'
@ ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css 6:2668-2759
@ ../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
@ ./src/App.js
@ ./src/entry-client.js
@ multi webpack-hot-middleware/client ./src/entry-client.js
ERROR in ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
Module not found: Error: Can't resolve '../../fonts/Roboto-Condensed/Roboto-Condensed/Roboto-Condensed-Light.eot' in 'F:\Projects\roboto-fontface-bower\css\roboto-condensed'
@ ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css 6:4359-4442
@ ../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
@ ./src/App.js
@ ./src/entry-client.js
@ multi webpack-hot-middleware/client ./src/entry-client.js
ERROR in ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
Module not found: Error: Can't resolve '../../fonts/Roboto-Condensed/Roboto-Condensed/Roboto-Condensed-LightItalic.eot' in 'F:\Projects\roboto-fontface-bower\css\roboto-condensed'
@ ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css 6:6040-6129
@ ../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
@ ./src/App.js
@ ./src/entry-client.js
@ multi webpack-hot-middleware/client ./src/entry-client.js
ERROR in ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
Module not found: Error: Can't resolve '../../fonts/Roboto-Condensed/Roboto-Condensed/Roboto-Condensed-Bold.eot' in 'F:\Projects\roboto-fontface-bower\css\roboto-condensed'
@ ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css 6:7706-7788
@ ../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
@ ./src/App.js
@ ./src/entry-client.js
@ multi webpack-hot-middleware/client ./src/entry-client.js
ERROR in ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
Module not found: Error: Can't resolve '../../fonts/Roboto-Condensed/Roboto-Condensed/Roboto-Condensed-BoldItalic.eot' in 'F:\Projects\roboto-fontface-bower\css\roboto-condensed'
@ ./~/css-loader!../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css 6:9370-9458
@ ../roboto-fontface-bower/css/roboto-condensed/roboto-condensed-fontface.css
@ ./src/App.js
@ ./src/entry-client.js
@ multi webpack-hot-middleware/client ./src/entry-client.js

Thx and sorry for the delay. Merged and released as @0.7.1


Update equality comparator for strings

What current issue(s) from Github does this address?

changes == to use EQUAL operator rather than NUMEQUAL

How did you make sure your solution works?

added tests

Are there any special changes in the code that we should be aware of?

no

Coverage decreased (-0.06%) to 93.274% when pulling 1a938dc044011041dde9105cb55aec3e619f352f on equality-update into e86e4839994927c37ca3a60884d7ae9d15cb167d on development.


Upgrading to CommunityToolkit.WinUI v8.0

With v8.0 available on NuGet, this PR adopts the new version of the Toolkit.
The following changes are included:

SettingsCard/SettingsExpander are now in the main toolkit, so no need to use the Labs packages anymore.
Updating to the new Toolkit namespaces
Adopting the FontIconExtension where possible, and removing the FontFamily=Segoe Fluent Icons parameter since that's now fixed in 8.0.
Replace custom converters with equivalents from the Toolkit.
Applying XAML styler

There are a ton of changes, but mostly due to the new namespaces and XAML styler. There are no visual or code changes.
Keeping this as DRAFT for now since we might be able to switch to the GA version any day now.

Nice job @niels9001!
Need to update https://github.com/microsoft/PowerToys/blob/main/NOTICE.md#nuget-packages-used-by-powertoys in order to make CI pass.
What do you think about consolidating view models and commands using source generators provided by CommunityToolkit.Mvvm across the solution (OFC as a separated work )?

Nice job @niels9001!
Need to update https://github.com/microsoft/PowerToys/blob/main/NOTICE.md#nuget-packages-used-by-powertoys in order to make CI pass.
What do you think about consolidating view models and commands using source generators provided by CommunityToolkit.Mvvm across the solution (OFC as a separated work)?

@davidegiacometti
Thanks! Updated the file as well 🤞.
I think our ViewModels (especially in Settings) could use some love - and I think using source generators would be amazing!

Hi @niels9001 , I think the current issues with CI are a bad merge with the contents of this PR: https://github.com/microsoft/PowerToys/pull/28367/

Hi @niels9001, I've added a commit to this PR to try and fix the merge: https://github.com/microsoft/PowerToys/pull/28205/commits/7fbbd46bc12a00274a1a28ddb38fbaf592f6a8d9
Can you please check if your changes are still as intended?

Hi @niels9001, I've added a commit to this PR to try and fix the merge: 7fbbd46
Can you please check if your changes are still as intended?

Thanks!! Yeah some weird things were going on at some point. Just tested, and everything seems to work as intended :)! Thanks!


weather-tools as a library

Fixes #206 .

This is a good start. However, the abstractions we provide in the Pipeline object should make adding variations to the pipeline easy (where we expect there to be changes). Our internal pipelines should be a guide for what kinds of changes we can expect from users. For example, users should be able to:

choose custom file openers (for their own file types) / ways users can create EE assets
create filters or combine URIs into groups to suite specific pipeline needs
specify if they want a batch or streaming pipeline

These only reflect our initial variants -- and only for the EE destination. I think that we could benefit from implementing this feature later, as we've still only seen a handful of variations of our pipeline.

Yes, It is an abstraction of what weather-mv pipeline is doing which the user can inherit and override pipeline-level functions.
Like this going one step deeper, the user should be able to override bq, ee, rg level functions. For that we can import them in weather_mv's init and make extendable classes.
Thoughts?


Installer hangs on npm install command

There is an upstream issue affecting installs. It can be resolved by exporting a variable before running the install.sh script.
Modified instructions until upstream is resolved:
# Download the install script
wget https://raw.githubusercontent.com/aws-samples/document-translation/main/util/installer/install.sh

# Make the script executable
chmod +x ./install.sh

# Temporary
export UV_USE_IO_URING=0

# Run the install script
./install.sh

Ref: https://github.com/amazonlinux/amazon-linux-2023/issues/840
Thanks to @domaws for finding this

https://github.com/aws-samples/document-translation/commit/f8c9532a595646f8d0ec14cba86f18c397612916


Lux command not found

After installing Lux using $ go install github.com/iawia002/lux@latest I get the output: "command not found" when running lux --version or just lux.
Why does this happen? And if is something that can't be solved how do I uninstall the package using go?
Thanks for you time and consideration.

You can find the binary in the ~/go/bin/ or your_dir/go/bin by default, you have to export it to path if you want to use it from everywhere. Assuming you're using *nix, not sure about Windows.


[Preview Version] the app closes when you change the compression percentage of Data Saver

Steps to reproduce
https://user-images.githubusercontent.com/96607644/148828647-62bcbc99-5774-4904-bd2b-ae82f1b36159.mp4
Expected behavior
the app does not close when changing the percentage.
Actual behavior
the app closes when I change the percentage
Crash logs
No response
Tachiyomi version
App version: 1.7.0 (standard, 04f284e27, 23, 2022-01-09T20:47Z) Preview build: 426
Android version
Android version: 7.0 (SDK 24)
Device
Device model: HUAWEI NXT-L09
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.7.0.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Addressed by https://github.com/jobobby04/TachiyomiSY/commit/479950e60e29fe11e8f6d40721c4cd4c7fafba75


The custom UrlGenerator should not typehint the implementation

Typehinting should happen on UrlGeneratorInterface instead (maybe requiring more of the routing interfaces too)

Good point!

Hum... but how to with Symfony\Component\Routing\Generator\ConfigurableRequirementsInterface if the "real" url generator implements or not the interface?
Maybe my UrlGenerator implements ConfigurableRequirementsInterface and methods setStrictRequirements() and isStrictRequirements() check if the "real" url generator implement ConfigurableRequirementsInterface and throw an exception if not?

For setStrictRequirements, make it a no-op if the inner generator does not implement it. For isStrictRequirements, return a fixed value (I would say true as requirements were always strict before we added this feature in Symfony)


CONTRIBUTING.md has placeholders

The setup, testing and style guidelines should be customized for this project (or removed if n/a)

We missed these, thanks for pointing them out.
I added content for the setup and testing, and removed the style guideline

We almost always miss these, I think.  I just got caught on that, so I'm paying it forward.

----- Original message -----From: Ton Ngo<EMAIL_ADDRESS>IBM/tensorflow-kubernetes-art-classification<EMAIL_ADDRESS>Mark Sturdevant<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Re: [IBM/tensorflow-kubernetes-art-classification] CONTRIBUTING.md has placeholders (#2)Date: Fri, Nov 3, 2017 10:28 AM
We missed these, thanks for pointing them out.I added content for the setup and testing, and removed the style guideline
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

fixed


Run tests on CI

Just to know, what do you think about running the automated tests on CI?
I can create a PR adding this feature if you'd like.

Sounds good! Your PR will be merged.

When running the tests locally I actually got some errors. The logs:
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
  Failed assembleTransformation (abbbaba,abbba) [22 ms]
  Error Message:
   Test method Algorithms.Tests.Strings.MinCostStringConversionTests.assembleTransformation threw exception:
System.Reflection.TargetParameterCountException: Parameter count mismatch.
  Stack Trace:
      at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)
   at Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.ThreadOperations.ExecuteWithAbortSafety(Action action)

  Failed assembleTransformation (ababa,ababa) [< 1 ms]
  Error Message:
   Test method Algorithms.Tests.Strings.MinCostStringConversionTests.assembleTransformation threw exception:
System.Reflection.TargetParameterCountException: Parameter count mismatch.
  Stack Trace:
      at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)
   at Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.ThreadOperations.ExecuteWithAbortSafety(Action action)

Failed!  - Failed:     2, Passed:   176, Skipped:     0, Total:   178, Duration: 315 ms - Algorithms.Tests.dll (net6.0)


ob.omit matching all keys containing a string rather than exact match of string

It would seem that, when using ob.omit(myObject, ['my_key']), all keys in myObject that contain 'my_key' are being omitted.
For example, given the following:
myObject = {
  "my_key": "foo",
  "keeper": "bar",
  "my_key_keeper": "baz"
};

this call:
ob.omit(myObject, ['my_key'])

will return this:
{
   "keeper": "bar"
}

even though I was expecting:
{
  "keeper": "bar",
  "my_key_keeper": "baz"
}

It is possible that this behavior is specific to using ob.omit as part of a Swag Doctor module - I didn't dig that deep. :)

Hey man!
Are you using the latest version of objob?
I was pretty sure I patched that issue a few days ago.
If you're using the latest version l, I'll look into it again though! :)

I'll wait for your response. If you're still having the issue and you're on the latest version, I'm pretty sure I hair used a regex that checked for a-Z and 0-9 and also need to check for _ and $

I show that I'm running<EMAIL_ADDRESS>I looked at your regex, and like you say, it may not be casting a wide enough net.
If it helps, my actual keys are elements and elements_board_id.

I'll look into it then! :) in the meantime, Lodash had the same functions, with the only difference being that they're shallow! But should work for your needs

I'm okay just ob.pick()'ing everything but my omitted key for now. :) Just wanted to bring it to your attention for whenever.

Thanks man! :)

This issue should now be patched!
My logic was right but I didn't escape enough times in my regex so while I thought I was checking for a period, I was checking for all characters.
Before:
let re = new RegExp('^'+inputKey+'\..*','g');

After:
let re = new RegExp('^'+inputKey+'\\..*','g');


hitting tab in last cell should select first cell in new row?

Right now if you hit tab in the last cell of a table, it will create a new row below the current one, and move the selection to the new last cell of the table. Instead, I think it should move it to the first cell in that new row, mimicking how tab works normally.

Fixed by 2d49801


Use filters and folder names in the languageConfigurationMapping.properties

Extend the FilePropUtilities.groovy script to handle the use of folder names and filters/wildcards in languageConfigurationMapping.properties.
This way, the user will be able to easier map a group of programs with a common language configuration property file, and avoid to enter all the program names and map each one with the corresponding property file.

Example of the current languageConfigurationMapping.properties structure:
epscmort.cbl=languageConfigProps01
epscsmrd.cbl=languageConfigProps01
epsnbrvl.cbl=languageConfigProps01

Example of the desired languageConfigurationMapping.properties structure:
epscmort.cbl=languageConfigProps01
folder-name=languageConfigProps01
eps*.cbl=languageConfigProps01
$ps*.*= languageConfigProps01

Thanks you!

Delivered via zAppBuild 3.8.0


Update names of variables constructed from TC data

This PR addresses Issues #299 by updating the names used for variables constructed from the Tax Calculator data.

@andersonfrailey @hdoupe Do you all know what the cause of the Travis CI error is here?  It reads;

self = <taxcalc.records.Records object at 0x7f9cb508dc10>
data =       age_head  DSI  EIC  FLPDYR  f2441  f6251  MARS  XTOT   e00200   e00300  ...8798           0    10170         0        0   8798
[8799 rows x 73 columns]
exact_calculations = False
gfactors = <taxcalc.growfactors.Growfactors object at 0x7f9cb64813d0>
weights =        WT2008       WT2009       WT2010       WT2011       WT2012  \
0     335...11201   446.629696   443.611201   446.629696  32095
[8799 rows x 20 columns]
adjust_ratios = 'puf_ratios.csv', benefits = None, start_year = 2011
    def __init__(self,
                 data='puf.csv',
                 exact_calculations=False,
                 gfactors=Growfactors(),
                 weights=PUF_WEIGHTS_FILENAME,
                 adjust_ratios=PUF_RATIOS_FILENAME,
                 benefits=None,
                 start_year=PUFCSV_YEAR):
        # pylint: disable=too-many-arguments,too-many-locals
        self.__data_year = start_year
        # read specified data
        self._read_data(data, exact_calculations)
        # check that three sets of split-earnings variables have valid values
        msg = 'expression "{0} == {0}p + {0}s" is not true for every record'
        tol = 0.020001  # handles "%.2f" rounding errors
        if not np.allclose(self.e00200, (self.e00200p + self.e00200s),
                           rtol=0.0, atol=tol):
            raise ValueError(msg.format('e00200'))
        if not np.allclose(self.e00900, (self.e00900p + self.e00900s),
                           rtol=0.0, atol=tol):
            raise ValueError(msg.format('e00900'))
        if not np.allclose(self.e02100, (self.e02100p + self.e02100s),
                           rtol=0.0, atol=tol):
            raise ValueError(msg.format('e02100'))
        # check that ordinary dividends are no less than qualified dividends
        other_dividends = np.maximum(0., self.e00600 - self.e00650)
        if not np.allclose(self.e00600, self.e00650 + other_dividends,
                           rtol=0.0, atol=tol):
            msg = 'expression "e00600 >= e00650" is not true for every record'
            raise ValueError(msg)
        # check that total pension income is no less than taxable pension inc
        nontaxable_pensions = np.maximum(0., self.e01500 - self.e01700)
        if not np.allclose(self.e01500, self.e01700 + nontaxable_pensions,
                           rtol=0.0, atol=tol):
            msg = 'expression "e01500 >= e01700" is not true for every record'
>           raise ValueError(msg)
E           ValueError: expression "e01500 >= e01700" is not true for every record
../../../../miniconda/envs/ospcdyn/lib/python2.7/site-packages/taxcalc/records.py:146: ValueError

I'm not getting this when I run the test locally.  Is there something up with the PUF used for the tests on Travis CI?

It looks like the part causing the problem was added recently. Is this test using test data for the PUF file?

This is due to a small change in the PUF. Last week we changed how pension income is assigned to non-filers and Martin added the corresponding test. It should be solved by using the PUF issued on March 9th (if I accidentally left you off that email, let me know and I'll send it over to you).

We also changed how e01500 and e01700 were calculated in the CPS file, but that will be included in the latest version of taxcalc.

Thanks @hdoupe and @andersonfrailey.  The test passes locally and I'm using the most recent PUF file from @andersonfrailey for that.
How do I update  the PUF used by Travis CI so that this test passes?

@jdebacker It looks like you are using the test data in the file puf91taxdata.csv.gz.
I'm not sure where this came from or how it could be changed to satisfy the Tax-Calculator requirement.

@martinholmer might be able to help update travis for this.

@hdoupe That's right - what data do you use to test Tax Calc? The puf91taxdata.csv.gz file is used on Travis CI since we can push it to the repo and it can be shared publicly.

I think Tax-Calculator uses the CPS data for testing. The puf 91 file was removed some in TC 0.10.0. Could we use the CPS file for testing?

@hdoupe -- I'll give it a try. Thanks!  Will open another PR for that.

Note that tests of txfunc.py were expanded, but still do not include the most complicated functions.  These tests are involved and will benefit from creating some fake data where the data generating process is known and thus one can verify the parameters returned from estimation match those used to generate the data.
cc @rickecon


feature request

can you add the ability to cancel  the pending orders too after x days?

I made a second hook for that, so users can simply disable or delete it if there is no need for it.

How can we disable invoice cancel and only use pending orders cancel?

How can we disable invoice cancel and only use pending orders cancel?

Just remove everything from line 8 to line 25.
See the image below, delete everything inside the red box. It will only orders and not invoices.


Filechanges I wrote about in the issue

Sorry about this, I'm just now getting used to git...
forking, pull-requests... I used to download your code via npm and was just hacking in that code ;-)
Hope you like my changes,
Nox

Cleaning up PRs. This is 5 years old :older_woman:


QueryBuilder return in anonymous function with if-condition not recognized

Since Version 1.3 I get the error Anonymous function never returns Doctrine\ORM\QueryBuilder so it can be removed from the return type.
I use the anonymous functions within the buildForm method for my Symfony forms. The following code is working:
                    'query_builder' => static function (EntityRepository $er): QueryBuilder {
                        return $er->createQueryBuilder('p')
                            ->orderBy('p.sort', 'ASC');
                    },

But the following code returns the mentioned error:
                    'query_builder' => static function (EntityRepository $er) use ($country): QueryBuilder {
                        $qb = $er->createQueryBuilder('addr');
                        if ($country instanceof Country) {
                            $qb->where('addr.country = :country');
                            $qb->setParameter('country', $country->getCountryKey());
                        }

                        return $qb;
                    },

I also tried different approaches like
                    'query_builder' => static function (EntityRepository $er) use ($country): QueryBuilder {
                        if ($country instanceof Country) {
                            return $er->createQueryBuilder('addr')
                                ->where('addr.country = :country')
                                ->setParameter('country', $country->getCountryKey());
                        }

                        return $er->createQueryBuilder('addr');
                    },

But this still doesn't work.
If I set queryBuilderFastAlgorithm: true, I don't get this error, but I'm not sure, if this behavious is intended.

This is a regression from https://github.com/phpstan/phpstan-doctrine/commit/88a26a2dabf5f47e7fd9b1c74a2537eca48c0578

@ondrejmirtes do you think you could provide another way to fix #309 that doesn't cause this regression?

This is still a problem

Somehow this just got fixed somewhere in the last ~5 releases


789 housekeeping

Description
Resolves #789
Checklist:

[x] I have reviewed my diff in github, line by line.
[x] Every change is related to the PR description.
[x] I have linked this pull request to the issue(s) that it resolves.
[x] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[x] The branch has been merged or rebased against the head of its merge target.
[x] I'm happy for the PR to be merged at the reviewer's next convenience.

Looks mostly good to me, but I think we actually do want the angle braces for barretenberg includes since it is an external library.. I'm open to being convinced otherwise though!

I thought we have a "mono-repo" including barretenberg ... my bad then. I will revert the last commit and push force on this branch then.

@dbanks12 @suyash67 I have now pushed a commit to have all "barretenberg includes" using some <> brackets in the circuits/cpp (excluding barretenberg sub-directory of course)


[UI] Multi-Context Switcher: style changes and bug fix

Signed-off-by: Aditya Chatterjee<EMAIL_ADDRESS>Description

Fixes style of the switcher.
Fixes the Crashing of UI when no Kubernetes cluster is present.

This PR fixes #5077
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Codecov Report

Merging #5081 (3321ad1) into master (3afcbcd) will decrease coverage by 8.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5081      +/-   ##
==========================================
- Coverage   32.12%   24.07%   -8.06%
==========================================
  Files          62       62
  Lines        5254     5254
==========================================
- Hits         1688     1265     -423
- Misses       3141     3709     +568
+ Partials      425      280     -145

Flag
Coverage Δ

gointegrationtests
?

unittests
24.07% <ø> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-58.95%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.14% <0.00%> (-55.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.91% <0.00%> (-54.55%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.41% <0.00%> (-50.00%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.92% <0.00%> (-48.08%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.29% <0.00%> (-37.37%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.03% <0.00%> (-23.03%)
:arrow_down:

mesheryctl/internal/cli/root/system/dashboard.go
1.49% <0.00%> (-16.42%)
:arrow_down:

mesheryctl/pkg/utils/healthcheck.go
8.82% <0.00%> (-4.42%)
: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 3afcbcd...3321ad1. Read the comment docs.

The UI integration test failures seem unrelated to this PR.

Thank you. This gets us closer to the design.


installation data per release type

app ids / installation ids
Tom will generate
Pass app id into install new (and user new?)

app ids:
Main playstore: OdyseeAndroidNativePLAYSTORE
apk: OdyseeAndroidNativeAPK
fdroid: OdyseeAndroidNativeFDROID
all should also pass installation type= mobile, domain= odysee.com


Add Makefile to Automate Local E2E Test Setup

Objective
Introduce a Makefile to streamline the local setup and execution of End-to-End (E2E) tests, improving developer efficiency.
Problem
Running E2E tests requires specific processes to be set up before execution, and manually configuring these processes for local testing can be time-consuming and error-prone. This repetitive task reduces developer productivity and can lead to inconsistencies in the testing environment. Automating this setup with a Makefile will make local testing more efficient and reliable.
Changes

Added a Makefile to automate the setup and execution of E2E tests locally.
Added a README with clear instructions on how to run the tests using the new setup.

Scope of Change
This is a non-critical update, as it only affects the local environment for running E2E tests and does not impact production code.

The problem with using '&' to start background tasks is that they must be terminated manually. In contrast, PM2 simplifies this process by assigning an identifier for easy management and control

@theanmolsharma, Using the tail command has also been challenging due to the issues with background PIDs. As an alternative, I opted to use output log files.


WIP: Add podman functional tests (#2159)

Signed-off-by: Sagi Shnaidman<EMAIL_ADDRESS>Add functional tests for podman driver
PR Type

Feature Pull Request

Hey @sshnaidm, thanks for this. I mentioned a comment in the IRC and also on https://github.com/ansible/molecule/issues/2317#issuecomment-534074063 regarding our plans for the drivers. I suppose the best course of action is to get this PR passing soon and merge it OR close the PR and follow the path of removing the driver code (see https://github.com/ansible/molecule/pull/2303) and then create a repostiory in a similar fashion to https://github.com/pycontribs/molecule-inspec. This is all a bit experimental right now but that's the current state. Let me know what you think.

@sshnaidm please fix this, we need better podman coverage now more than ever, especially as we do not have docker on centos8 and rhel8.

The problem is that podman driver will work only with patch https://github.com/ansible/ansible/pull/57741 in ansible and this will be only in 2.9. I've created backport in https://github.com/ansible/ansible/pull/62915 but not sure if it will be merged.
So far we need to limit tests to run on ansible 2.9 minimum.

@sshnaidm I can try to add a fixture that enables tests only on specific ansible versions. Would this help you?

This should help you https://github.com/ansible/molecule/pull/2337

Seems like failed tests are hetznercloud tests, not related to podman. And all podman tests are skipped, because it can't be installed on ubuntu. Need to test it on centos.

Restarted the build :+1:

As I see podman tests are skipped, although podman is installed in last version and ansible version is 2.9rc01. I don't understand why it's still skipped: https://travis-ci.com/ansible/molecule/jobs/240039472
@ssbarnea @decentral1se  any thoughts?

OK, I think I found what is the problem. Current ansible 2.9 has version 2.9rc01 and pkg_resources compares it incorrectly:
pkg_resources.parse_version("2.9rc01") >= pkg_resources.parse_version("2.9")
False

I've changed minimum version to 2.8.999 to avoid this.
@ssbarnea ^^

OK, now podman runs finally, and fails. Will check what is the reason.

@sshnaidm I can help with the requested changes if you're running low on free cycles

@fabianvf thanks, I think we're good now. Waiting for https://github.com/ansible/molecule/pull/2394 to merge, and then it should be ok.


Update postgres disk setup for modern EC2 hosts

Various updates to move this tool to be in line with current hosts.
Testing Plan: Verified as a no-op on existing hosts.  Will verify after merging by standing up a new postgres host.

Great, thanks @alexmv!  I merged all but the last commit as the series ending with b33aa8da7fb215e8421820d3e35334b8abe3cf72; for that one, I'm not sure it's the right change.  We have 3 situations in which we might run setup_disks:

A totally new server, where need to run setup_disks
That server later, where we expect running setup_disks to be a noop
A server where we just did the operation we did with postgres2 a few nights ago, of destroying its local disk and then restarting it.

I think in that last case, this logic isn't correct, in that /etc/fstab will have /srv on there, but /srv will have no filesystem.
I guess the other thing I notice is our postgres_master rule to actually run this:
  exec { 'setup_disks':
    command => '/root/setup_disks.sh',
    require => Package["postgresql-${zulip::base::postgres_version}", 'xfsprogs', 'mdadm'],
    creates => '/dev/md0'
  }

no longer makes sense; maybe we want that condition to be more like an onlyif for /var/lib/postgresql not being a symlink to a directory?
Looking at the other entries in postgres_master.pp, I think we can also delete give_nagios_user_access; because that state lives in data in the permissions tables in database, it doesn't make sense as a puppet rule (we're not likely to need to  create a new empty database from scratch with no data).  Does that seem right?
BTW you should probably set your GIT_AUTHOR_EMAIL to the Zulip one, at least for this repository.

Pondering this some more, the third case is partially fixable via:
if ! mountpoint -q /srv; then
    mkfs.xfs $LOCALDISK
    mount /srv
fi

...except that we end up with two possible end states: either an empty database that was just moved into /srv/postgres after initial configuration, or (post-reboot) a /srv mounted and no /srv/postgres at all.
I thus question if we ever actually want to move /var/lib/postgres over; it's more consistent to have the end state be a created (but empty) /srv/postgres which is ripe for running wal-g backup-restore LATEST into, rather than this potential other state.
It seems a little scary to rm -rf /var/lib/postgresql in the case it's not currently a symlink, but since we never actually store real data in the durable /var/lib/postgresql, that should always be safe.

Looking at the other entries in postgres_master.pp, I think we can also delete give_nagios_user_access; because that state lives in data in the permissions tables in database, it doesn't make sense as a puppet rule (we're not likely to need to create a new empty database from scratch with no data). Does that seem right?

The only case it's applicable is if puppet on a clean machine, which means it has its own local database without any permissions.  But per the above, I don't think that should be a potential end state.  Given that, I'll remove it.

How about we just do an mv to /root/empty-postgres for /var/lib/postgresql rather than rm, just for safety in case of future bugs in the code?  mv is always safe :).

I believe the above addresses your comments!

Yep, this is great, merged, thanks @alexmv!


Possiblity to have grey out period where no events can be created (past and into future period)

Hi,
FYI: I am using your calander for a simple planning of appointments.
Is there a simple solution for you to disable the possiblity to add events in the past?
At the moment i am doing it myselve in the dateClick checking the date and ignoring the event when the incoming date is in the past.
But the draw back is that there are no visual hints that you cannot plan an appointment. (time is shown, dates are of normal color instead of greyed out)
It would be nice if there would be some gray period where you cannot add new events.

In the past (make all previous days  AND previous hours in current day : Grey and not clickable and not showing time on hover
Extra: Add a future period to now allow entry of new events; Essentialy point 1, but then the start is not 'now' but a future point in time, possibly given by numbers for  days:hours:minutes

Point 2 is required since there is always some kind of lead time for reacting on the actual event.

Hi @DNWEIJ
Thank you for suggestions! I'd like to keep the size of the library as small as possible. And provide a tool instead of a ready-made solution :smile:
I prepared an example where days in the past are highlighted using background events. I used red, but you can change it to whatever you like. Also, using eventContent, you can not show the pointer for such days. dateClick also has a check to see if we are clicking on a day in the past or not.
Will this work for you?

Thanks! This is indeed what i am search for. You pointed me in the right direction with the eventContent.
I totally missed it from the documentation
Maybe copy the part you have in the jsbin example for eventContent into the documentation as example? That gives a good pointer what you can do with the eventContent
Again, thanks! :-)

Maybe copy the part you have in the jsbin example for eventContent into the documentation as example? That gives a good pointer what you can do with the eventContent

Yes, other tickets also had useful code examples. I'm thinking about taking them out to one place so that others can get to know them.
Thanks for the idea!


Fix wildcard support for multiply nested wildcards

This PR fixes the support for methods that end in a wildcard character *, and include two or more underscores to separate nested parts of the method name.

Some inline comments related to redundant code that's been left over should be addressed.

We should also add a test that includes method names ending with * for the sake of completeness.

Addressed in the rebased 3b72980


 It might be
good to go away once this repository is no longer hosted at GitHub, or
if GitHub ceases to use the line, or GitHub ceases to exist.

I think we can leave this for now, given it's just for GitHub cosmetics.


With timeout: Uncaught TypeError: request.cancel is not a function

When a timeout is set, and when the requested url does not exists or the target server is offline, the request is cancelled with an error:
import Vue from 'vue'
import VueResource from 'vue-resource'

Vue.use(VueResource)

Vue.http.get('http://does-not-exist-or-is-offline.com/something', { timeout: 3000 })
  .then(response => {
    console.log(response)
  }, response => {
    console.log('error')
  })

It returns Uncaught TypeError: request.cancel is not a function and then error
Is it a bug or is there a way to handle this situation?
(UPDATE: i am using ^0.9.1)

Seems to be related to changes in version 0.9.0.

Renamed method Request.cancel() to Request.abort()

https://github.com/vuejs/vue-resource/commit/2f951df7261a5bd940438addf83b2a13de06c873#diff-c825ede355666a78dcca43e2d5bf33e5R745

@michaelhue yes https://github.com/vuejs/vue-resource/blob/master/src/http/interceptor/timeout.js#L11
maybe it should be request.abort();

This file does not reflect the cancel() to abort() change on the timeout.

I am seeing request.abort() defined in function xhrClient and function xdrClient, but that is it.
@hedii Maybe try going into your local Vue source files and locate this:
if (request.timeout) {
    timeout = setTimeout(function () {
        // change this to abort();
        request.cancel();
    }, request.timeout);
}

Then report back.

closed in #311


Migrate from UnsafePointer to UnsafeRawPointer.

Previously:
https://github.com/apple/swift/pull/3724
The lldb errors on the linux bots were not related to these changes.

@swift-ci Please smoke test OS X.

As proposed in SE-0107: UnsafeRawPointer.
Depends on corelibs-foundation PR:
https://github.com/apple/swift-corelibs-foundation/pull/478

This passes build-toolchain on both darwin and ubuntu with the above PR.
These are the same changes that were on master yesterday. The lldb failures were unrelated.


Basic functionality for pcp_exporter

Add all required files to build pcp_exporter, and convert pmwebapi metrics into Prometheus metrics.
Signed-Off-By: Kalyan Mada<EMAIL_ADDRESS>

@joehandzik took care of the modifications, let me know if there is anything else!

It looks like @joehandzik and I are pretty much on the same page - the important ones are the log.Fatal related lines, otherwise everything else should probably wait for future PR's.

The main issues I wanted to see fixed are resolved now, so once Travis passes, I am good to merge under the assumption that the other points @joehandzik and I put here will be fixed in future PRs.


[misc][broker] Update deprecation warnings to use 3.0.0

Motivation
With https://github.com/apache/pulsar/pull/19573, we should replace deprecation warnings that were set to 2.12.0 with 3.0.0.
Modifications

Update 4 deprecation warnings.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
This is cosmetic.
Documentation

[x] doc

This is a documentation change.
Matching PR in forked repository
PR in forked repository: skipping forked PR since this is a trivial change

/pulsarbot rerun-failure-checks


Stressing branch misprediction logic (verification experiment)

Hi :wave:,
I was testing another verification hypothesis. What I am doing is basically generating random legal instruction addresses out of BTB. My assumption is that branch prediction mechanism should not affect functionality of the core and doing this will simply trigger a lot of mispredictions. However, Black Parrot hangs for some tests (watchdog timeout). I was wondering if doing this is okay in Black Parrot?
Thanks!

Hi! Yep, that should be legal.  Could you share an address that's causing a hang? It's possible that it's generating an address that goes to unmapped local memory, which would be < 0x8000_0000 and not mapping to cfg, clint or cache MMIO CSRs.  If a request is sent to an unmapped address region, it could cause a hang.
Another unusual case would be > 0x20_0000_0000, which maps off-chip memory.  This should cause a speculative instruction fault, which will get squashed before causing a trap (since it's speculative).
In the former case, a patch was merged for this earlier today.  https://github.com/black-parrot/black-parrot/pull/519.  If it's a non-local address, we'll have to look deeper!

I am trying to mimic the legal instruction addresses, meaning the addresses are half-word aligned and are in the range 0x8000_0000 to 0x8000_FFFF.
I am trying to isolate the address that causes the hang now, but I have just realized that I might be doing something wrong. Quick question: ports br_tgt_v_o and br_tgt_jmp_o in bp_fe_btb can be randomized as well, can't they?

I would do word-aligned.  We don't have support for misaligned addresses (hardware or trapping) at the moment.
If br_tgt_v_o is forced high when the entry is invalid, that could cause X-propagation.  Similar to our other handshaking discussion, it's generally only allowed to mask helpful signals, not force them high.
I think br_tgt_jmp_o can be freely randomized (subject to the range and alignment restrictions)

Closing for staleness


Fixing the active session count going negative.

Purpose
Avoid the portal graph showing active session count as negative.
Goals
When a session termination event is received for a session that is not traced for the creation, analytics should not consider it as it will corrupt the data, making it possible to go negative. We consider this a highly unlikely situation unless analytics is enabled while there are active sessions(remember me, sessions maybe).
(Contributed by Lasantha)

fixes #495


hmm matlab: Error when running 'wu run none-matlab matlab-vm hmm'

-------------- hmm,matlab,none-matlab,mba-2011,matlab-vm --------------
{ type: 'experiment',
'short-name': 'default',
'input-size': 'medium',
'iteration-number': 1 }
************** Iteration 0 ************
                        < M A T L A B (R) >
              Copyright 1984-2015 The MathWorks, Inc.
               R2015b (<IP_ADDRESS>7246) 64-bit (maci64)
                          August 20, 2015

To get started, type one of these: helpwin, helpdesk, or demo.
For product information, visit www.mathworks.com.
----------------------------------------------------
Your MATLAB license will expire in 52 days.
Please contact your system administrator or
MathWorks to renew this license.
----------------------------------------------------

Academic License

Error: File: /Users/erick/Documents/Recherche/Ostrich2/builds/4693df730f7cb9c94f1508b85a93303488e4660d/runner.m Line: 176 Column: 1
"pi" previously appeared to be used as a function or command, conflicting with its use here as the name of a variable.
A possible cause of this error is that you forgot to initialize the variable, or you have initialized it implicitly using load or eval.

The similar question I have asked in #73
"medium": {
            "model": "n",
            "nb-states": 512
        },

From: benchmarks/hmm/benchmark.json

Solve. See 91142040acaf1c352ac6b370b1f2abd6568eb7f7


Subnautica giving me this error

Output log:
Import: Game file 'globalgamemanagers' has been found
Import: Game file 'level0' has been found
Import: Game file 'level1' has been found
Import: Game file 'level10' has been found
Import: Game file 'level11' has been found
Import: Game file 'level12' has been found
Import: Game file 'level2' has been found
Import: Game file 'level3' has been found
Import: Game file 'level4' has been found
Import: Game file 'level5' has been found
Import: Game file 'level6' has been found
Import: Game file 'level7' has been found
Import: Game file 'level8' has been found
Import: Game file 'level9' has been found
Import: Dependency 'unity_builtin_extra' has been loaded
Import: Assembly 'UnityEngine' has been loaded
Import: Assembly 'mscorlib' has been loaded
Import: Dependency 'unity default resources' has been loaded
Import: Dependency 'globalgamemanagers.assets' has been loaded
Import: Assembly 'Assembly-CSharp-firstpass' has been loaded
Import: Assembly 'Assembly-CSharp' has been loaded
Import: Dependency 'sharedassets0.assets' has been loaded
Import: Assembly 'UnityEngine.UI' has been loaded
Import: Assembly 'System.Core' has been loaded
Import: Dependency 'resources.assets' has been loaded
Import: Dependency 'sharedassets4.assets' has been loaded
Import: Dependency 'sharedassets3.assets' has been loaded
Import: Dependency 'sharedassets5.assets' has been loaded
Import: Dependency 'sharedassets8.assets' has been loaded
Import: Dependency 'sharedassets7.assets' has been loaded
Import: Dependency 'sharedassets6.assets' has been loaded
Import: Dependency 'sharedassets1.assets' has been loaded
Import: SerializedFileException: v:5.6.2p4 n:globalgamemanagers p:C:\Program Files (x86)\Steam\steamapps\common\Subnautica\Subnautica_Data\globalgamemanagers
Message: Read 16 but expected 17 for asset type TimeManager
StackTrace:    at uTinyRipper.SerializedFiles.SerializedFile.ReadAsset(EndianReader reader, AssetInfo assetInfo, Int64 offset, Int32 size)
at uTinyRipper.SerializedFiles.SerializedFile.ReadAsset(EndianReader reader, AssetEntry info, Int64 startPosition)
at uTinyRipper.SerializedFiles.SerializedFile.ReadAssets(EndianReader reader, Int64 startPosition)
at uTinyRipper.SerializedFiles.SerializedFile.Read(Stream stream, Action1 dependencyCallback)    at uTinyRipper.SerializedFiles.SerializedFile.Load(Action1 dependencyCallback)
at uTinyRipper.SerializedFiles.SerializedFile.Load(Parameters pars)
at uTinyRipper.FileCollection.LoadSerializedFile(String filePath)
at uTinyRipper.GameStructure.Load(List1 pathes)    at uTinyRipper.GameStructure.Load(IEnumerable1 pathes)
at uTinyRipperGUI.MainWindow.LoadFiles(Object data)

Is this Subnautica or the new one (below zero). There's an issue logged with below zero already.

It's Subnautica

I am also getting this issue

The only fix I found was to comment out these lines, https://github.com/mafaca/UtinyRipper/blob/d10cfadcb58ce3742bbb49d85656d9cf7012bc05/uTinyRipperCore/Parser/Format/SerializedFile/SerializedFile.cs#L464-L467

The only fix I found was to comment out these lines,
UtinyRipper/uTinyRipperCore/Parser/Format/SerializedFile/SerializedFile.cs
Lines 464 to 467 in d10cfad

Yeah that's generally a way around it, but it doesn't fix the problem just skips over it. The problem is that the size of the asset types are based on the version of Unity used to pack them so UT needs to know what version of the asset and what the expected size is. Would be nice if there was a dictionary of sizes and versions (with a fallback default). Something like a dictionary holding all the sizes for all the versions for each asset type, then as sizes change we just add to the dictionary rather than having all kinds of if() statements scattered throughout the codebase.

Subnautica has custom layout for TimeManager asset type. You can skip it if you temporarily move away globalgamemanagers file


[Suggestion] Is it possible to remove casting shadow for firstperson playermodel?

Is your feature request related to a problem? Please describe.
Is it possible to remove casting shadow from firstperson playermodel? because playermodel covers the flashlight, so i literally cant use the flashlight normally
Describe the solution you'd like
Remove casting shadow from firstperson playermodel

From our Discord server (#commands-list)
Flashlight's shadow fix (when you see ur own shadow):
Add to /GarrysMod/garrysmod/cfg/autoexec.cfg
This: r_flashlightdepthtexture 0


[Question]: My OpenAI resources have been depleted. How should I renew them?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
My OpenAI resources have been depleted. How should I renew them? I am using a Windows system, and I have tried changing the environment variables, but it has not worked.

Hello @newgooooal! 👋 I'm Dosu, a bot here to help you solve bugs, answer your questions, and guide you on your journey to becoming a contributor. While we wait for a human maintainer, know that I'm here to assist you with any issues or questions you might have. I see you're having some trouble with your OpenAI resources - let me take a closer look and I'll provide you with a detailed answer shortly!

My OpenAI resources have been exhausted, and I would like to switch to a new OpenAI API. What should I do?

@newgooooal , 你的问题读起来是因为你的OpenAI账号里余额不足。你可以去他们网站上充钱。这不是一个针对LlamaIndex的问题，所以在这里提问估计不会得到答案。
如果我误解了你的意思，请把你见到的错误提示复制、粘贴到这里，我们好能一起看。
(I wrote Chinese here because I saw you've been translating the webpage when you read replies from Dosu. If my assumption was wrong, you have my apologies.)


Add multiroot-workspace variable scoping for python.defaultInterpreterPath

It's currently not possible to use multiroot-workspace variable scoping to configure python.defaultInterpreterPath, as @kimadeline mentioned in https://github.com/microsoft/vscode-python/issues/18207#issuecomment-1055840598.
My use case is the following. I have multiple python projects in my workspace but I want them all to use the same virtualenv, existing under one of the workspace folders. The tree structure looks like this:
/
├── python-a/
│  ├── .venv/
│  └── foo.py
├── python-b/
│  └── bar.py

To make it work with python.defaultInterpreterPath I need to configure it like this in the workspace file:
"python.defaultInterpreterPath": "${workspaceFolder}/../python-a/.venv/bin/python"

This is fragile, as it will break if python-b/ workspace folder changes its absolute path. With multiroot-workspace variable scoping it would be possible to express this dependency more robustly like this:
"python.defaultInterpreterPath": "${workspaceFolder:python-a}/.venv/bin/python"

@OmeGak I think there may be a bit over overlap here with #18674 (please up vote 😄 )
I don't seem to have this issue specifically, but I have set this in my subworkspace's ./.vscode/settings.json:
{
    "python.defaultInterpreterPath": "${workspaceFolder}/.venv/bin/python"
}

Note https://code.visualstudio.com/docs/editor/variables-reference#_variables-scoped-per-workspace-folder is currently only supported in tasks.json or launch.json (used while debugging), the use for it in settings.json is discouraged.
@OmeGak In this usecase .venv/ does not belong to the first workspace folder and is something the workspace file owns instead, so I'm wondering if the concept of ${workspaceFile} would work better instead. It can be used to access the path to the .code-workspace file.

Thanks for the feature request! We are going to give the community 60 days from when this issue was created to provide 7 👍 upvotes on the opening comment to gauge general interest in this idea. If there's enough upvotes then we will consider this feature request in our future planning. If there's unfortunately not enough upvotes then we will close this issue.

Thanks for your answer @karrtikr!

Note https://code.visualstudio.com/docs/editor/variables-reference#_variables-scoped-per-workspace-folder is currently only supported in tasks.json or launch.json (used while debugging), the use for it in settings.json is discouraged.

The vscode-python extension allows to substitute variables in the python.defaultInterpreterPath setting, however. I knew that variable substitution is only enabled built-in for a few setting keys in settings.json but I didn't know that this practice was discouraged for extensions. I'd be curious to read more info about it, if you have any link at hand. Is workspace scoping going too far?

@OmeGak In this usecase .venv/ does not belong to the first workspace folder and is something the workspace file owns instead

That is one way to look at it but in my use case .venv/ does belong to the python-a workspace. python-a is a full-fledged Python project. The other workspaces are either python-a plugins or python-a Git submodules that are installed in the same virtualenv as python-a and should use the same Python interpreter when I debug them.

I'm wondering if the concept of ${workspaceFile} would work better instead. It can be used to access the path to the .code-workspace file.

This is certainly another workaround. It's still more fragile than being able to use ${workspaceFolder:python-a}, as I will need to hardcode python-a path twice in the .code-workspace file.
"python.defaultInterpreterPath": "projects/my-python-project/python-a/.venv/bin/python"

Versus:
"python.defaultInterpreterPath": "${workspaceFolder:python-a}/.venv/bin/python"

Thank you to everyone who upvoted this issue! Since the community showed interest in this feature request we will leave this issue open as something to consider implementing at some point in the future.
We do encourage people to continue 👍 the first/opening comment as it helps us prioritize our work based on what the community seems to want the most.

@OmeGak Apologies for the delayed response and thanks for the insight!

This is certainly another workaround. It's still more fragile than being able to use ${workspaceFolder:python-a}

Two workspace folders can share the same base name as well where this does not work in its own right. However https://code.visualstudio.com/docs/editor/variables-reference#_variables-scoped-per-workspace-folder also documents this, so maybe it's not considered such a practical scenario. We'll still probably go with ${workspaceFolder:python-a} to be consistent.

I didn't know that this practice was discouraged for extensions. I'd be curious to read more info about it, if you have any link at hand. Is workspace scoping going too far?

Supporting such system variables in settings.json were being a problem for LSP which is probably why they aren't supported by VSCode, IIRC. I'll see if I can find any links.

@OmeGak The reason settings.json substitutions are complicated because settings are hierarchical. In some cases there might not be a good way to resolve. Like in this scenario:

with this as a .code-workspace
{
    "folders": [
        {
            "path": "notpython" // not a python folder
        },
        {
            "path": "python1" // the python folder that contains .pylintrc
        },
        {
            "path": "python2" // the python folder does NOT contain .pylintrc
        },
        {
            "path": "." // root workspace parent of both python1 and python2
        }
    ],
    "settings": {
        "pylint.args": ["--rcfile=${workspaceFolder}/.pylintrc"]
    }
}

What is the right resolution here for .pylintrc for a python file under python2?

C:\GIT\linters\multiroot\multipython\python2\.pylintrc is incorrect because there is no such file under that workspace root, but is the correct substituion based on the variable.
C:\GIT\linters\multiroot\multipython\.pylintrc is the correct file, but for a file under python2 we can't find the right thing to substitute that variable.

The issue boils down to, settings are hierarchical so substitutions are not straight forward. What works is in each folder had a settings.json of its own. Like this:

And you would not need to have variables that require substitution there, at least variables like ${workspaceFolder}.

Hi all, feature should be out in the pre-release version of the extension, use the following to try it out:


Localize UI Strings

Host UI elements should be localized.  I imagine this is a known TODO, but ensuring it is captured

Let's use #792 as the master issue for making sure the right things are localized and localizable.

Ok, wasn't sure if you wanted to treat them separately.  I would have treated them as separate tasks, but 🤷♂  :-)


Upgrade reqwest to v0.12.2

Summary
Still requires...

A reqwest-middleware release.
A reqwest release.
An async_http_range_reader release`.

For now, they're all stubbed in as Git dependencies.
Closes #2814.

Oh, also need to look at https://github.com/astral-sh/uv/issues/2814#issuecomment-2035914083.

Let me know if you want my review.

I'm waiting on a reqwest-middleware release.


Is it possible add shadow above bar???

Maybe...looks already have shadow
But wanna more customize
Ps. Your package is best for me thanks very much

Thank you for choosing our library.
Customizing the shadow isn't supported by now, code change is required.
If you have chance to do it, I am happy to merge your PR.
Cheers.


added open in colab badges at the top of jupyter notebooks

Screenshot:

Thanks for adding this! If we end up with a lot of notebooks we can automate this with a GitHub action :)


Seurat to AnnData test

Issue and/or context:
#1184
Changes:
Adds a Seurat -> SOMA -> AnnData test
Notes for Reviewer:
Relatively simple test that starts from a pre-generated RDS artifact, converts it to SOMA and then to AnnData. Assertions are done in R since we can't read the original Seurat. To do them in Python, we could use Reticulate to read it, but it's probably better to keep it simple.

Codecov Report
Patch coverage has no change and project coverage change: -11.35 :warning:

Comparison is base (e22e1d4) 65.36% compared to head (b5c624f) 54.01%.

:exclamation: Current head b5c624f differs from pull request most recent head a05a98d. Consider uploading reports for the commit a05a98d to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1316       +/-   ##
===========================================
- Coverage   65.36%   54.01%   -11.35%
===========================================
  Files          96       67       -29
  Lines        7720     5352     -2368
===========================================
- Hits         5046     2891     -2155
+ Misses       2674     2461      -213

Flag
Coverage Δ

python
?

r
54.01% <ø> (+0.25%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 44 files with indirect coverage changes

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


https://github.com/wjakob/nanobind/discussions/196

new: is_ndarray_scalar_v, used to limit type_caster<Eigen> to those scalars that it can actually handle. This gives users the chance to add their own specializations for other scalars.
de-bug: correct handling of Eigen/ndarray strides
enhanced: fully support Eigen's dynamic strides; request contiguous memory from ndarray only if really needed.
de-bug: prevent nb::cast<Eigen::Map> and nb::cast<Eigen::Ref<mutable>> from pointing to invalid memory due to conversions.
enhanced: renamed a few test functions to better reflect their signature.

As discussed, type_caster<Eigen::Map> and type_caster<Eigen::Ref<mutable>> do not support conversions any longer, unless a cleanup_list is passed. This prevents nb::cast from returning Eigen types that point into destroyed memory.
Still, I suggest these 2 type casters to not support conversions at all, even if they handle bound function arguments. I believe that the intention of binding a function that expects an Eigen::Map is to modify the passed memory (Map<mutable>) or to avoid copies (Map<const>). Silent conversions may hide errors and show surprising behaviour instead, see e.g. https://github.com/WKarel/nanobind/blob/f339ec0df6ce0d2d185e82df03e4e1905c04a183/tests/test_eigen.py#L99 where the argument remains unmodified.

(FYI, I am working towards a conference deadline and will get back to reviewing PRs in a few days -- sorry for the delay)

Hi @WKarel — I saw that you posted some changes. Please let me know once I should re-review.

Dear @wjakob ,
I  had prepared this comment, but forgot to post them :-D
I hope to have addressed all your remarks in my latest commit.
I'm a friend of comments in code, but I noticed that they are rather sparse in nanobind, and so I initially did not introduce any. Concerning the formatting rules, are they documented somewhere? I was confused by some pre-existing code like
constexpr int NumDimensions ...
right in dense.h, and the rules are still a bit unclear to me.
I have commented out the assert statements. What is the motivation for not including ? I see assertions as a more concise alternative to verbose text comments, with the additional benefit of them being enforced in debug builds (if not commented out). Like comments, they are not really necessary, of course.
Along the way, I discovered a bug when mapping Eigen::RowVector, which I resolved, together with introducing a respective test.
The failed tests on Ubuntu should be gone.

Still, I suggest these 2 type casters to not support conversions at all, even if they handle bound function arguments. I believe that the intention of binding a function that expects an Eigen::Map is to modify the passed memory (Map) or to avoid copies (Map). Silent conversions may hide errors and show surprising behaviour instead, see e.g. https://github.com/WKarel/nanobind/blob/f339ec0df6ce0d2d185e82df03e4e1905c04a183/tests/test_eigen.py#L99 where the argument remains unmodified.

What is your opinion on the above?

Hi Karel,
a few comments:

Formatting rules: they are not 100% consistent, sorry. Usually upper case CamelCase for classes, lowercase snake_case for methods and other globals. Some constexpr members have the former, some the latter. But we don't have any lower case camelCase, which is why I commented.

Commenting: this is basically my hobby project. When I accept a PR, I take on a responsibility to maintain that code in the future. I can usually understand my old code pretty easily. But when I read somebody else's code, I might not follow and ask for more comments. So the commenting requirement is a little asymmetric but needed for the continued maintainability of the project.

Nanobind has a policy of truly minimal include requirements (see the nanobind documentation on "why another library" regarding why this was a problem in pybind11). So that means much of the STL isn't available, including assert(). I think that you were able to call it because of the Eigen headers, which include cassert somewhere.

An alternative would be an if-test followed by nanobind::detail::fail() or similar. But my question was: what are cases where you would expect these assertions to fail? Can it actually happen? Is an assertion truly the right error criterion?

What is your opinion on the above?

I agree that it is confusing and would be willing to support such a change.

Dear Wenzel,
thanks for your explanations. I do not expect any of the assertions to fail under any circumstances, unless there is a not yet discovered bug, of course. It seems that I had not inspected <cassert> for a long time, somehow thinking that it really only contained the definition of a single macro, which is far from true at least for VC++. Still, I think the assertions document the expectations well, and so I'd leave them commented out, if you agree.
Since you agree that Eigen::Map and Eigen::Ref should no longer support conversions, I'll push another according commit. Eigen::Ref will continue to support conversions in general, since this is what it is made for.
Cheers,
Willi.

As discussed, Eigen::Map and Eigen::Ref no longer support conversions in my latest commit.

Hi @WKarel,
I did a pass over the PR (great work!) with some minor stylistic changes and plan to merge it shortly with those edits. However, there was one part of your changes that I did not understand.
The type caster for Eigen::Ref tries two nested map type casters in sequence. The first tries to find a compatible representation, while the second one (dcaster) makes a copy/implicit conversion.
What confuses me is that I don't understand how Ref(dcaster.operator DMap()) can be a valid operation. The text says that this creates a Ref that owns the data it maps, but I did not see anywhere in the Eigen docs how such ownership works. The documentation only mentions that temporaries may be created when calling a function (with Ref argument) that stay alive until the function call is done. But things are different here, because we are returning the Ref instance, so the usual lifetime policies of function argument construction don't apply.

(updated the comments with a more precise question)

Dear @wjakob ,
thanks!
The documentation on Eigen::Ref is quite sparse. I think the reason for using the term temporary is the use case for which this class was introduced, namely to use Eigen::Ref<T const> as function argument type. This makes it easier to define non-template functions that accept Eigen instances without making copies all the time.
If Ref<T const> is constructed from a suitable instance, then it just maps that instance's data. Otherwise, it maps its own member variable, having copy-converted it from that instance. If used as function argument, then this copy only exists during the function call, which may justify to call it 'temporary' in this case. A function may safely return a Ref, however. One only needs to consider its unusual copy constructor that does not copy its member variable. But that is of no concern in nanobind's type_caster, because it constructs the Ref it returns in-place (prvalue), and so the copy constructor is not involved here. For these reasons, your point 3 is of no concern: the returned instance owns the data it maps.
But indeed, there was still a flaw when scalar conversion took place, but no stride conversion! I have therefore pushed another commit that supports scalar type conversions only if either a cleanup_list is passed, or if the Ref is guaranteed to own the data it maps.
Along the way, I have discovered a feature/behaviour/bug in Eigen that makes Ref fail silently if it cannot map its own data. Therefore, stride conversions are now restricted to the common case that Ref's StrideType can actually describe the contiguous memory of Ref's member variable.
I've tried to document all the considerations while not making them too lengthy.
Cheers,
Willi.

Hi @WKarel,
thank you for those changes. I am confused about one last thing:
The trait Eigen::internal::traits<Ref>::template match<DMap>::type::value suggests to me that it represents whether Ref can represent data mapped using DMap.
Should this not be a precondition for calling Ref(dcaster.operator DMap())? (In the sense that that whole code block should be wrapped not just in if constexpr (std::is_const_v<T>) but also take this compatibility into account?
I also don't understand how this flag is combined with the presence of a cleanup list. I remembered that the ndarray caster doesn't even use the cleanup stack.

The only thing that still confuses me is the role of the CompatibleWithDMap part at the end.

Dear @wjakob ,
The constructors of Ref<T const> use one of two methods for construction, depending on the value of Eigen::internal::traits<Ref>::template match<DMap>::type::value. If it is false, then Ref<T const> will own the data it maps, see the relevant constructor, which then calls the method that copy-constructs Ref<T const>::m_object before mapping it.
If the trait is true, then the constructors do not necessarily touch m_object, but they may directly map the constructor argument.
Thus, the trait does not decide on whether the Ref can be constructed from a DMap, but if it is false, then the Ref is guaranteed to own its data.
However, it is true that the construction of a Ref<T const> from an Eigen type may silently fail at run-time, because while it always manages to copy-convert the argument fine, it may fail to map its own (contiguous) data considering the dimensions of the constructor argument and the given StrideType. If have posted 2 respective issues in the Eigen repo:

already the compilation should fail if it is clear at compile-time that the mapping will fail
Ref<T const> should provide a move constructor. type_caster<Ref<T const>> could then own a member variable of type Ref, assign to it in from_python and directly return false if its creation failed, and operator Ref() could then move out the member variable.

So far, type_caster<Ref<T const>> ensures that the creation of Ref will succeed by duplicating internal Eigen logic using can_map_contig_memory, which refuses to call dcaster if the creation of Ref may fail, based on compile time information.
It may be necessary for type_caster<Ref<T const>> to return a Ref that owns the data it maps. That is certainly not the case if a cleanup_list is passed, because that is only the case if the handle to be casted is a bound function argument, which in turn means that the type_caster will stay alive until that function has returned. This guarantees that the memory that Ref maps remains valid even if Ref does not own it, and if type_caster<ndarray> creates a copy due to scalar type conversion or because contiguous memory was requested, because type_caster<Ref> owns both its type_caster<Map> which in turn own a type_caster<ndarray>, which finally owns an ndarray as member variable value, which is assigned by type_caster<ndarray>::from_python. Indeed, the cleanup_list is not touched at all, but it serves as an indicator that Ref does not need to own the data it maps.
If no cleanup_list is passed, then Ref<T const> must own the data it maps, because it will outlive its type_caster. It would be ideal to have some means to force Ref<T const> into constructing itself such that it owns the data it maps, independent of its specialization, and independent of the constructor argument. Unfortunately, there is none, and Ref<T const> guarantees to own the data it maps only if its traits say so - depending on its own specialization, and on the constructor argument type.
Cheers,
Willi.

Hi @WKarel -- many thanks for your hard word on this, I just merged the commit. https://github.com/wjakob/nanobind/commit/7b74d2fa419c6fd1973bc2a69de6640d4721553b contains a pass from me over the code (I hope this looks okay to you, and that I didn't break anything).

Dear @wjakob,
great, and you are welcome. Looking at dense.h, I think that https://github.com/wjakob/nanobind/blob/08b363aaeac0f487261e06d1d2437dec88a06cc2/include/nanobind/eigen/dense.h#LL400C5-L401C69
static constexpr auto Name =
        const_name<MaybeConvert>(MapCaster::Name, DMapCaster::Name);

should be replaced with
static constexpr auto Name =
        const_name<MaybeConvert>(DMapCaster::Name, MapCaster::Name);


iOS 10 show issue

Hi, animation for show and dismiss are broken for iOS 10.

Thanks for the heads up. I'll take a look and update as needed.

Hi, I added a PR fixing show/dismiss animations.

This has been address in this PR: https://github.com/atljeremy/JFMinimalNotifications/pull/35
I'll create a new release with this fix today.


anvilgen: lazy singletons for attributes.

Regarding #58.
Other than increased method count, is there any other issue with this approach?

You're correct, and unfortunately, method count is a big concern for current Anvil design, because it scales very badly.
For example, now there are ~500 view attribute setters in SDK15 DSL. It's not so easy to strip off the unused methods, so we are likely to end up with 500 actual view methods + 500 apply() methods + 500 constructors + 500 static DSL methods.
So a typical Anvil app uses ~2k methods. Including all the support libraries would result in ~3k methods. But if we replace instance with getInstance() - we end up with 25% increase of all the methods - and that's like 6% of the total method limit per dex.
I was considering grouping lambdas somehow to have a few multi-functional apply() methods with some switch/case inside it depending on the actual setter/view type, but I'm still not sure if it's a good idea.

we end up with 25% increase of all the methods

Method limit is something that can be fixed relatively easy with multidex. Performance/memory on the other hand is something that can't be fixed from outside of the library.

On the way to Anvil 0.5.0 we've addressed the issue of singletons in DSL classes in a different manner (#99). They are now replaced with a giant switch/case, which in fact even gives some (very small) performance gain, reduces method count by ~70% and provides a solution to name clashes in various support libraries.
Bad thing is that unfortunately it obsoletes your pull request. I'm sorry, and I really appreciate your input, you've risen a valid concern and it helped to find a right solution, but I have to close this pull request.


[20min.ch] Add new extractor (closes #5977)

[20min.ch] Add new extractor (closes #5977)

This most common pattern is not recognized: http://www.20min.ch/videotv/?vid=425123&cid=2.

@dstftw Works now

Cherry-picked, thanks.
Do you want to be credited as one of the authors of youtube-dl? If so under what name?

Thank you.

Do you want to be credited as one of the authors of youtube-dl? If so under what name?
No, thanks.


Game stutters and becomes unplayable

Hello.
I currently have the problem that my game stutters and becomes nearly unplayable.
I read the reviews here and on ProtonDB and nearly everyone says that the game should run better, but at least equally good compared to running it on Windows.
Back when I used Windows I could play the game on medium settings without any lags, but now that won't work.
The most strange thing is that the stuttering does not change if I switch to low graphic settings so I think the graphic settings are not the issue.
First of all, I tried starting it using truckersmp-cli -g ~/.steam/steam/steamapps/common/Euro\ Truck\ Simulator\ 2 -p -v start (just bare Proton) and it was completely unplayable due to lags.
Then I switched to DirectX 11 using truckersmp-cli -g ~/.steam/steam/steamapps/common/Euro\ Truck\ Simulator\ 2 -p -v -d start and the game didn't start at all.
I enabled the native compiler and the game started an ran a bit better than on OpenGL (I was expecting that).
Enabling WineD3D (truckersmp-cli -g ~/.steam/steam/steamapps/common/Euro\ Truck\ Simulator\ 2 -p -v -d --use-wined3d start) made the game just a bit better and it seems to run flawless in medium settings, but there is weird stuttering that makes it unplayable.
I hope anyone can give me advise on how to fix that if possible.
OS: Arch Linux x64
CPU/GPU: AMD Ryzen 3 2200G (Vega 8 graphics included)
Ram: 8GB
(Yeah, I know, pretty bad setup, but definitely ran the game flawlessly on Windows 10)

Please check if the game also stutters when the compiler message isn't displayed in the lower left running the following with dxvk:
DXVK_HUD=compiler truckersmp-cli …

This environment variable actually improves the performance a bit, but it still isn't comparable to Windows.

Shader compiling takes time and is done for every new shader you encounter. And while the task is done on the fly it causes major stutters by doing so. The variable tells you if that compiling is still going on. So please verify the performance after the "Compiling shader…" message in the lower left isn't visible anymore.

Oh, well, there was no message in the lower left...

Then that's not it ;)
Maybe a demanding task is running in the background? Firefox with a video running, Steam background shader compiling, … just some wild guesses.

Nope, nothing like that. Restarting my machine and not starting anything doesn't change something about that.
As I think Steam would tell me that it compiles shaders in some way (progress bar, idk) I think I can exclude that too.

Alright, my fix was setting up a Linux & Windows dual boot environment to have general-purpose workspace and gaming stuff separated.
I will close this issue as I won't be able to try out other solutions.
Thanks for your help.


upgrade_linux  script that gradeup from davinci0.3 beta.4 to beta.5 is not work

Please provide more details.

https://github.com/edp963/davinci/tree/master/bin/migration