Proposal - add subs.static property

I have a city picker in my proj with 5000 cities in it (each - a different doc). This picker is present on every page.
Before using subs-manager I subscribed to 'cities' publication only when user clicked on city picker, showing a loader while the data was downloaded to the client.
After installing subs-manager my data is there almost instantly (if user doesn't rush in and clicks city picker instantly after page was rendered). But this means that each of my users will be subscribed to this pub, inducing overhead on the server, related to maintaining all of these reactive subscriptions.
So, I'm in doubts here:

with subs-manager I get nice UX, but server overhead
w\o it I subscribe only when necessary, but have suboptimal UX

Thus I propose to add to subs-manager the ability to download only the initial data and then sever connection to server if it doesn't need to be reactive. Like adding a subs.static: true property.

You could just create a meteor method to retrieve the cities and call it inside a Meteor.startup block on the client which would have the same effect. Save the cities to some kind of global variable like the Session var or a local collection var localCollection = new Mongo.Collection(null)?

I agree with @mjmasn.  This is exactly the wrong way to approach that problem.  Either use a method and locally store it, or have a non-reactive publication (One that just fetches the records and dumps them using this.added in the publication method).

@lassombra how do I create a non-reactive publication?

@avalanche1 Another way is to do a null publish. The best way to explain it is with the Meteor.user() function. Where do your user details come from to allow the function to work? They are published using a null publish [1], which means the data is automatically sent to the client when the page first loads.
e.g. if your collection is called 'Cities':
Meteor.publish(null, function() { return Cities.find(); });

No need to subscribe, Meteor will handle it automatically because the publish has no name. Now you will always have access to the cities on the client without having to reload the subscription on each page. The list will still be reactive, but there's no overhead if nothing changes so you shouldn't need to worry about 'overloading' the server at all.
[1] https://github.com/meteor/meteor/blob/devel/packages/accounts-base/accounts_server.js#L699-L714

@mjmasn this still has the problem of the entire subscription being reactive.  What @avalanche1 is looking for is to not take that performance hit.
@avalanche1 to do a "non-reactive" publication which has the benefit of being tied to the mergebox still and still showing up as an active subscription, in your server code:
Meteor.publish('publicationName'/* or null for autopublish */, function() {
	// this in a publication context represents a specific publication.
	// save self to this to make it available in closures later.
	// (Or could use arrow functions with es6)
	var self = this;
	// get the entire current cities array.
	// At this point we're acting like a non-reactive controller
	var cities = Cities.find.fetch();
	// loop through the cities to send them to the client.
	_.each(cities, function(city) {
		// self.added from http://docs.meteor.com/#/full/publish_added
		// sends an item to the client to be stored in the client's
		// minimongo collection with the provided collection name.
		self.added('collectionName', city._id, city);
	});
	// don't forget to send this or the subscription will never know
	// that it's done and the data is ready to be used.
	this.ready();
});

The only real benefit this has compared to meteor methods is it handles the integration into minimongo for you both on the adding side, and on the removing side when you stop the subscription.
Otherwise you can have a meteor method that does something similar and you load it into minimongo yourself.

@lassombra , thank you, this is educative.
But how can we stop the publication after receiving the initial set of data. I don't want the server to spend resources on maintaining that pub.

@avalanche1
In the example I showed you, there is only a slight memory overhead on the server as it maintains the map of items sent so that on stop it can remove them.  No observers are created which is where the real pain of publications exist.  It can also be further optimized by having cities loaded into memory once on startup and always pumped down on request.  If you want a method style:
//Server
Meteor.methods({'getCities': function() {
	return Cities.find().fetch();
}});

//Client
Meteor.startup(function(){
	Meteor.call('getCities', function(error, result){
		if (result){
			_.each(result, function(city){
				Cities.insert(city);
			});
		}
	})
});

This is the way to have the data stay in minimongo if you do not want to keep an active subscription.
But, as I said earlier, a non-reactive publication like I provided above is not expensive and you get minimongo integration automatically.
Back to subs.static, besides being outside of the scope of what subsmanager is, it's impractical.  subs.static would never be useful here because it couldn't force the server to not be reactive.  You'd still get an observer created (with all the cost there) and then quickly destroyed according to the design you offer which means then garbage collection.  Once you call this.stop() the data is removed from local mongo.  Subs manager would have to alter the core behavior of meteor in order to change that.  This is not something we want.  Also, you really don't want the client telling the server how to handle the publication.  The code I provided above is really not that complicated, and if you're going to be doing it in multiple places, you could create a utility function to do it.  If you want non-reactive data that you keep handy in local mongo, then you need a method.  A method can retrieve information server side, and return it to the client which can then load it into either a local variable, or if needed, minimongo.  A method once complete is done, there are no more references to it hanging around.
I've used the non-reactive publications more than a couple times on production applications which had just way too much data.  In one case, the publication is cached so that every client that ever connects gets the same result until the server process is restarted.  In another case the publication is refetched every time but doesn't react so I don't incur that overhead.


Enforce type checking on winpdh_base.py

What does this PR do?

Follow-up to #7752
Add and enforce type checking on winpdh_base.py, so that bugs that can be caught by mypy like the one fixed by #7752 don't creep up again in the future.
Motivation

Type safety
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

/azp run

Couch and HAProxy failures are unfortunate flakes… Merging


Bug: parse error near `-a' from 'source..' command during new install on fresh OS X 10.15.7 Catalina

Bug report
To reproduce

Open a new Terminal window
Run curl -s "https://get.sdkman.io" | bash
Run source "$HOME/.sdkman/bin/sdkman-init.sh"

Expected behavior:
The sdkman-init.sh script is executed and the sdk command is available
Actual behavior:
danbitter@Dans-MBP ~ % source "/Users/danbitter/.sdkman/bin/sdkman-init.sh"
find: -type: open: unknown type
/Users/danbitter/.sdkman/bin/sdkman-init.sh:83: parse error near `-a'
System info

OS and version: Mac OS X 10.15.7 Catalina
Shell and version (e.g. bash --version/zsh --version): zsh 5.7.1 (x86_64-apple-darwin19.0)
The output of sdk version: zsh: command not found: sdk

Screenshot of my Terminal window

Hi @BitterDone, can you confirm that zsh has in fact been initialised in your shell? What is the console output for echo $SHELL and echo $ZSH_VERSION?
The fact that find is not working properly makes me think that your current shell is not fully featured. The code is thoroughly tested and runs perfectly on both bash and zsh.

`echo $SHELL` produces /bin/zsh
`echo $ZSH_VERSION` produces 5.7.1
I agree, something feels weird, but my work provided this brand new MBP, unmodified by them. I recently upgraded to OS X 10.15.7 Catalina, and I've used the laptop for maybe 10 or 15 hours so far.

@BitterDone let's get to the bottom of this :smile:
Next, I want to ask you to please run the following and paste the console dump:
find "${SDKMAN_DIR}/src" "${SDKMAN_DIR}/ext" -type f -name 'sdkman-*'

Also, if you could paste the output of find --version please.

I appreciate your willingness to help :-) I've used sdkman religiously the past couple years and I'd love to have it working again.
danbitter@Dans-MBP spring-boot-app % find "${SDKMAN_DIR}/src" "${SDKMAN_DIR}/ext" -type f -name 'sdkman-*'
find: -type: open: unknown type
danbitter@Dans-MBP spring-boot-app % find --version
find: illegal option -- -
usage: find [-H | -L | -P] [-EXdsx] [-f path] path ... [expression]
find [-H | -L | -P] [-EXdsx] -f path [path ...] [expression]

find -version should give you the correct version.
This is very strange, as both GNU and BSD find supports the -type parameter.
A brew install findutils would probably fix the problem but it would alsbe interesting to get to the root of the problem.

Could you also run which find?

Neither appears to be cooperating ¯\_(ツ)_/¯
danbitter@Dans-MBP spring-boot-app % find -version
find: illegal option -- v
usage: find [-H | -L | -P] [-EXdsx] [-f path] path ... [expression]
find [-H | -L | -P] [-EXdsx] -f path [path ...] [expression]
danbitter@Dans-MBP spring-boot-app % which find
type: bad option: -l

The default Terminal is zsh (a change Apple made starting with OS X Catalina), so the output is the same as what we've seen:
danbitter@Dans-MBP ~ % source "$HOME/.sdkman/bin/sdkman-init.sh"
find: -type: open: unknown type
/Users/danbitter/.sdkman/bin/sdkman-init.sh:83: parse error near `-a'
However, I changed from zsh to bash via chsh -s /bin/bash and sdkman appears to work as expected:

When I run that source .. command in bash, it returns nothing (appears to work)

Are you using iterm 2? If yes, what happens when you run the same commands in MacOS's default terminal?

Sounds similar to your case
https://stackoverflow.com/questions/44493812/macosx-terminal-which-git-return-error-zsh-bad-option-l

@marc0der @helpermethod Thank you guys so much, you're both what's great about the development community 👍
I disabled everything in my .zshrc, .bashrc, and .bash_profile and reran source "/Users/danbitter/.sdkman/bin/sdkman-init.sh" and it works in the zsh shell.
I'd taken this guy's .bash_profile and made a couple changes, so the issue was totally on my end:
https://natelandau.com/my-mac-osx-bash_profile/

Looks like this was the offender:
alias -g f='open -a Finder ./'                 # f:            Opens current directory in MacOS Finder

I changed it from an alias -g to a function and now my .zshrc works happily alongside sdkman :-)
f() {open .;}

alias -g defines a global alias, which contrary to a normal alias replaces every occurrence of the alias with its value.
So if you have a global alias like
alias -g f=foo

And you type
echo f

the output will be foo.


Feature request: Expo Config Plugin

It would be awesome if this library could ship an expo-config-plugin for EXPO SDK.

I have no plans to deploy on expo

That’s sad, since expo is a no brainer thanks to EAS


New arguments for perSampleDominantFeatures

#420

Otherwise works. Ready to go after changes mentioned above

You can run this for tests, if you haven't already
devtools::load_all()
data("GlobalPatterns")
tse <- GlobalPatterns
assay(tse)[20, ] <- assay(tse)["550960", ]

# Finds the dominant taxa.
sim.dom <- perSampleDominantTaxa(tse)
sim.dom <- perSampleDominantFeatures(tse, complete = FALSE)
tse <- addPerSampleDominantFeatures(tse)
colData(tse)$dominant_taxa
tse <- addPerSampleDominantFeatures(tse, complete = TRUE, n = 5)
colData(tse)$dominant_taxa

Now there is an error in package dependencies (not related to the changes). How should I fix it?

See the log:
2024-01-19T15:00:11.0664027Z make: *** [/usr/local/lib/R/etc/Makeconf:198: readBfaToc.o] Error 1
2024-01-19T15:00:11.0669619Z ERROR: compilation failed for package ‘ShortRead’
2024-01-19T15:00:11.0671074Z * removing ‘/__w/_temp/Library/ShortRead’
2024-01-19T15:00:11.4060117Z ERROR: dependency ‘ShortRead’ is not available for package ‘dada2’
The dependency packages need to be installed --> When dada2 package is installed, its dependencies are installed --> this ShortRead package gives an error when installed.
Why? I don't know to be honest. This might occur when the system is lacking certain system dependencies. However, I had this similar problem just couple days ago in another package. The problem was solved by itself.
How to fix? --> Run the GHA again.
This problem is little bit related to rworkflows issue --> we should standardize GHA by utilizing rworkflows package.

@himmil could you check if it is possible to finalize this..?

This is ready now @TuomasBorman

Seems good!! Bump the version and add info about changes to NEWS file

@TuomasBorman this is ready


Cannot integrate with private repositories

Private repositories need authorization token in parameters when performing communication with GitHub API.
error: https://pastebin.com/Xsh5BDeW

I have successfully reproduced your problem and investigated the code. I have tracked the problem to the request builder in the getJSONReader(uri) method in the GitHubService.java where it seem to occur and I tried to check if there have to be any differences on how to build the request for the public vs private repository. I came to conclusion that the request header does not need any extra content when integrating for the private repository.
After that, I have checked the scope of the personal access token for tyr and it seems like that if you want to have the current version of tyr integrated with a public repository, the current scope "repo:status - Access commit status" is enough, but when integrating with private repository, the scope have to be extended to "repo - Full control of private repositories" otherwise github will refuse the access.
There is written the following in the github docs for the scope "repo:status" :

Grants read/write access to public and private repository commit statuses. This scope is only necessary to grant other users or services access to private repository commit statuses without granting access to the code.

Therefore using scope "repo:status" of the access token should be enough also for the private repositories, but as the tyr is working with more than just commit statuses, we are forced to extend it's permissions.


updates firmware image for increased ZA speeds and 2ms debounce

overview
Tweaks from results of overnight qc tests in production line
SmoothiewareOT commits:
commit that increased the axes speeds
commit that increased the debounce time

What is the expected effect of this on the robot? Will it speed it up, or just increase the max it can achieve?


 If we continue hold it , it will be dangerous for our body. We might even drown. Howewer in the game's current version player can swim and dive in water blocks without any health effects. I want to add new feature into diving mode.
*Step 1
Declare a integer variable to keep a time while character is diving.Increment it if character is diving.(In the game, it is controlled by checkMode() method.Mode is always updated by the game engine.)
*Step 2
Check the variable's value .If it is reached to dangerous level, player will be damaged.
*Step 3
If player finishes the diving ,variable will be updated as 0.

@cagdasgerede Hocam yeni issue ekledim. Onaylarsanız commit edeceğim

Ok


qt5-base[core]:x64-windows build failure

Host Environment

OS: Windows 10
Compiler: MSVC v141

To Reproduce
Steps to reproduce the behavior:
./vcpkg install qt5:x64-windows
Failure logs
PS C:\Users\Hazem PC> vcpkg install qt5:x64-windows                                                 Computing installation plan...                                                                      The following packages will be built and installed:                                                     qt5[activeqt,core,declarative,essentials,imageformats,multimedia,networkauth,quickcontrols2,svg,tools]:x64-windows                                                                                * qt5-activeqt[core]:x64-windows                                                                    * qt5-base[core]:x64-windows                                                                        * qt5-declarative[core]:x64-windows                                                                 * qt5-imageformats[core]:x64-windows                                                                * qt5-multimedia[core]:x64-windows                                                                  * qt5-networkauth[core]:x64-windows                                                                 * qt5-quickcontrols2[core]:x64-windows                                                              * qt5-svg[core]:x64-windows                                                                         * qt5-tools[core]:x64-windows                                                                     Additional packages (*) will be modified to complete this operation.                                Starting package 1/10: qt5-base:x64-windows                                                         Building package qt5-base[core]:x64-windows...                                                      CMake Warning at scripts/cmake/vcpkg_buildpath_length_warning.cmake:4 (message):                      qt5-base's buildsystem uses very long paths and may fail on your system.                                                                                We recommend moving vcpkg to a short path such as 'C:\src\vcpkg' or using                           the subst command.                                                                                Call Stack (most recent call first):                                                                  ports/qt5-base/portfile.cmake:1 (vcpkg_buildpath_length_warning)                                    scripts/ports.cmake:90 (include)                                                                                -- Figuring out qt target mkspec. Target arch x64                                                   -- Target mkspec set to: win32-msvc                                                                 -- Host mkspec not set. Qt's own buildsystem will try to figure out the host system                 CMake Warning at scripts/cmake/vcpkg_buildpath_length_warning.cmake:4 (message):                      qt5-base's buildsystem uses very long paths and may fail on your system.                                                                                We recommend moving vcpkg to a short path such as 'C:\src\vcpkg' or using                           the subst command.                                                                                Call Stack (most recent call first):                                                                  ports/qt5-base/cmake/qt_download_submodule.cmake:13 (vcpkg_buildpath_length_warning)                ports/qt5-base/portfile.cmake:51 (qt_download_submodule)                                            scripts/ports.cmake:90 (include)                                                                                -- Using cached D:/programes/vcpkg/downloads/qtbase-everywhere-src-5.12.8.tar.xz                    -- Extracting source D:/programes/vcpkg/downloads/qtbase-everywhere-src-5.12.8.tar.xz               -- Applying patch patches/winmain_pro.patch                                                         -- Applying patch patches/windows_prf.patch                                                         -- Applying patch patches/qt_app.patch                                                              -- Applying patch patches/gui_configure.patch                                                       -- Applying patch patches/icu.patch                                                                 -- Applying patch patches/xlib.patch                                                                -- Applying patch patches/Qt5BasicConfig.patch                                                      -- Applying patch patches/Qt5PluginTarget.patch                                                     -- Applying patch patches/prl_parser.patch                                                          -- Applying patch patches/Qt5GuiConfigExtras.patch                                                  -- Using source at D:/programes/vcpkg/buildtrees/qt5-base/src/5.12.8-58a5dc6709                     -- Configuring x64-windows-dbg                                                                      -- Configuring x64-windows-dbg done                                                                 -- Configuring x64-windows-rel                                                                      -- Configuring x64-windows-rel done                                                                 -- Building x64-windows-dbg                                                                         CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):                         Command failed: D:/programes/vcpkg/downloads/tools/jom/jom-1.1.3/jom.exe /J 9                       Working Directory: D:/programes/vcpkg/buildtrees/qt5-base/x64-windows-dbg                           Error code: 2                                                                                See logs for more information:                                                                        D:\programes\vcpkg\buildtrees\qt5-base\build-x64-windows-dbg-out.log                                D:\programes\vcpkg\buildtrees\qt5-base\build-x64-windows-dbg-err.log                                                                                Call Stack (most recent call first):                                                                  ports/qt5-base/cmake/install_qt.cmake:65 (vcpkg_execute_required_process)                           ports/qt5-base/portfile.cmake:295 (install_qt)                                                      scripts/ports.cmake:90 (include)                                                                                Error: Building package qt5-base:x64-windows failed with: BUILD_FAILED                              Please ensure you're using the latest portfiles with .\vcpkg update, then                         submit an issue at https://github.com/Microsoft/vcpkg/issues including:                               Package: qt5-base:x64-windows                                                                       Vcpkg version: 2020.02.04-nohash

Missing logs:
D:\programes\vcpkg\buildtrees\qt5-base\build-x64-windows-dbg-out.log
D:\programes\vcpkg\buildtrees\qt5-base\build-x64-windows-dbg-err.log

Also provide the config-<xx>.log logs please

build-x64-windows-dbg-err.log
build-x64-windows-dbg-out.log

used triplet :
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)
set(VCPKG_PLATFORM_TOOLSET v141)

   Creating library ..\..\lib\Qt5Cored.lib and object ..\..\lib\Qt5Cored.exp
double-conversion.lib(double-conversion.cc.obj) : error LNK2001: unresolved external symbol __CxxFrameHandler4
..\..\lib\Qt5Cored.dll : fatal error LNK1120: 1 unresolved externals

Looks like double-conversion was either compiled with a different version of VS or used the /d2FH4  flag. (https://devblogs.microsoft.com/cppblog/making-cpp-exception-handling-smaller-x64/)

exactly double-conversion was built using a different compiler. I rebuild it and all works well thank you very much. ❤


Implement CD for Creating a minified file.

@AidasK
I have a suggestion. Maybe include a way to create a minified file (flow.min.js) and updating that after every merged PR. That will be small in size and not include comments... (I mean, you already know what a minified file is 😉 ). So, yeah, implement a CD for that. Maybe Jenkins.... What do you think?

Ohh, my bad, I realized you are keeping versions... BTW, the last release was 7 months ago... Any timeline on when you will be releasing next?

Code it pretty stale at this moment and this repo only receives 6 pull requests a year, so maintaining jenkins might be too much. Maybe you would like to do it? I can give you the access. Travis needs fixing too

Okay, I can give it a try... Maybe clean the Issues also a bit. I see there are really old issues still open. 😅

I have added you to collaborators list, feel free to do stuff such as close pull requests, issues, push some commits... Glad to see that you are interested ;)

Thank you. 🙇


Windows: Use LoadLibrary to load R library instead of linking to it

LoadLibrary/dlopen

Windows side is done. Reducing the estimate.

Since this is marked as Windows, I guess it is done?

There is broker side change pending. did not make that since we were refactoring everything.

I will close this since the main work is done. However, thee is some impact due to R 3.4 which i will handle separately.


Support file install type

Hi,
I've just make change following the comments in PR #113.

A few things:

[ ] Needs a test-kitchen suite test to make sure if a user a selects file and version 0.10.0 or whatever, then the expectation is that the version output by influxdb version should match 0.10.0
[ ] squash commit if possible.

The idea for "keyed down for version", which I borrowed from chef-elasticsearch, is to be look consistent to the version InfluxDB is going to be installed. However, I don't oppose to your simplify resolution.
Those checksums look redundant, but it would make sense if we switch between versions of InfluxDB.

btw, I've just update test-kitchen suite for file and update default recipe to support file install_type.

Hi @bdangit, its been awhile, could you take a look at my latest comments, see if it make sense. If not, I would revert them.

hi @chrisduong sorry.  I'll check on this today and I'll get back to you.

Hi @bdangit,
Just updated integration test for InfluxDB file installation. Have a nice weekend.

lgtm just tested kitchen locally.
👍

@fishnix what do you think?

alright i'll veto all.  👍
merging in.

thanks @chrisduong !


Wrong Java version

We should find a way to link to common dependencies/prereqs @kkraune
I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

agree, maybe in the root readme for sampleapps maybe? Thanks for fixing


fix(versions): update Activiti/activiti-cloud-notifications-graphql versions into master

UpdateBot pushed maven dependency: org.activiti.cloud.notifications.graphql:activiti-cloud-notifications-graphql-dependencies to: 7.1.138

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.notifications.graphql:activiti-cloud-notifications-graphql-dependencies 7.1.138


[Question] Unable to set breakpoint

I can't set breakpoint using F2 in both Code view and Smali view ( they 're not synchronized either)
I looked up Wiki which said smali debugger is supported, but that doesn't work for me.
Is there any step to enable this feature?
My Jadx-gui version is 1.4.5, my phone is rooted, and I was able to attach debugger on any application.

sorry for setting the lables incorrectly.

@Nova-Noir breakpoints now works only in smali viewer in Show dalvik bytecode mode (checkbox in right click menu in smali viewer) this mode should be enabled automatically on debug start, but looks like something goes wrong this time :cry:

Yes, I tried it and it now working ! Thanks for your kind help.


[INLONG-535][tubemanager] reset offset using offset found by filter consume

JIRA link - [INLONG-535] created by stingpeng

This issue was created a long time ago. Close it first


RHEL does not support --mountpoint change to --target

Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@rhvgoyal Does this work on RHEL?

@rhatdan yes. All the tests passed on rhel after the change.

Looks like you beat me to it.


[enhancement][qa] Make rota more robust to sluggish Slack API

Sometimes when Slack's API is sluggish responding, Rota gets hung up on async calls and doesn't seem to completely process functions before moving on too quickly. Add some functionality to handle race conditions on slow networks or poor connectivity.

Completed; time will tell if this is fully solved.

I'd be surprised if this was entirely due to sluggish responses from slack API. This was happening consistent everyday when I had it installed so had to remove the bot.

I cannot reliably reproduce your issue. If you're able to detect the cause and submit a PR to resolve it, please feel free.


Target .NETFramework4.6.1 instead of 4.6

Resolves https://github.com/AngleSharp/AngleSharp/issues/831

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

What about .NET 4.6 users? This seems to be rather hostile (and is breaking!).

It is due to tooling bug which shipped in "plain" 4.6. If you care much about backward compatibility, 4.5 target can also be included safely. The "net46" is broken, though, as it shipped before .NETStandard2.0 was fully finalized. Microsoft has been so far suggesting bumping to 4.6.1 in order to force users off the broken 4.6.0.

Just for completeness - if we could just target 4.5 we would do it. There is a reason why we went to 4.6 and not 4.5. Regarding 4.6.1 - why would we bother with this and not go to, e.g., 4.7.2 directly?
I can also rephrase: Why should we ship something for a broken system any way?
Now that is quite controversial, but where I was going with this is that there are existing users with 4.6 - and they should still be able to get AngleSharp.
We don't want to force users in any direction. We want to support what is possible (and doable / manageable) to support.
Do you have any link regarding the net46 claim? I don't see any evidence and this was IIRC never discussed (keep in mind I'm an MVP and I'm in the internal mailing lists; even though possible it is in my opinion unlikely that an issue of that magnitude was not discussed in greater detail). I would love to learn more. What I know is that 4.6.1 has problems with the NS2.0 compatibility. The compatibility was added later and that was a mistake - so some things broke / are broken.

I'd be also 100% happy if you'd opt for net46+net472 where the kinks have been iron out, this works for me as well (tried with several other packages, and our internal tooling - went all in on net472 as we don't have internal users on 4.6x). The problematic scenario for me is if package advertises (net46+netstandard20) - in this case 4.7.x tooling goes for net46 as better match.
I've got trouble finding links, unfortunately - my written sources in this case are based mostly on twitter conversations between David Fowler, Immo Landwerth, Marc Gravell ( https://www.nuget.org/packages/Dapper/ ) and Bruno Garcia ( https://www.nuget.org/packages/Sentry ) and what came out of it in terms of targeting their packages - but obviously twitter is terrible for back-references.
Ah, found a decent link, I guess: https://github.com/getsentry/sentry-dotnet/issues/135#issuecomment-438599084

Thanks for taking the time to dig deeper. I think this is the issue I'm also recalling (thus my reference to 4.6.1 being broken).
I guess the way out of this could be to:

Stay with the NS1.3 and NS2.0
Stay with the 4.6
Have a build for 4.7.2

Thus 4.6 and 4.6.1 users will get the same experience, but for 4.7.2 onwards the experience should be better.
Would you agree or do you have a different approach in mind?

That works for me. I'll update the PR.

LGTM - let me know if that works!
Also thanks for the contribution - much appreciated 🍻 !


fix: DataTransfer data types

In 2.0.1 both DataTransferRequest and DataTransferResponse have a field called data which is defined as AnyType so I believe these should both by of type Any here

My read on the spec for this is different. The Field Type for data in both DataTransferRequest and DataTransferResponse is indeed listed as anyType. However, in 2.1.3. Primitive Datatypes, AnyType is defined as "Text, data without specified length or format" (emphasis mine). So I think the correct Python data type for this should be str, not Any.
This is a pretty confusing part of the spec, though, I could certainly be wrong. It looks like it has a mistake in it as well, because the capitalization isn't consistent between the Field Types anyType and the Primitive Datatype name AnyType. Maybe this part of the spec needs some TLC in the next revision.

@malsyned Good catch! You're right. I think we should revert this change.

My read on the spec for this is different. The Field Type for data in both DataTransferRequest and DataTransferResponse is indeed listed as anyType. However, in 2.1.3. Primitive Datatypes, AnyType is defined as "Text, data without specified length or format" (emphasis mine). So I think the correct Python data type for this should be str, not Any.

Interesting, I read "Text, data" to mean "Text OR data" - agreed it's not clear though I think @OrangeTux will ask the OCA for a clarification


About frames

Excellent work! But I want to know how many frames of images and lidars do you use in your work?

Yes,both image and lidar sweeps!

Zero, temporal information fusion, exponential moving average (EMA) for weight parameter and test-time augmentation is not used in all experiments.

OK. Thank you very muach! Looking forward to your release!


Issue/58/forced source db ingest

Hey @jbkalmbach @rbiswas4 !
I was keen to get the Run1 forced source data into MySQL tables so that I could sort out what queries we will be making with qserv and to start playing around with the data.  So over the weekend, I put together the code in this branch, and I ran the ingest on the forced source catalogs of the 985 Run1 visits, using some MySQL tablespace I have on a server in KIPAC.  That ingest took a couple days (we should be able to speed that up though), but once the data were in the tables, querying for the forced source fluxes was very fast, so I wrote a couple classes to serve up light curves based on the Monitor code:
https://github.com/DarkEnergyScienceCollaboration/Twinkles/blob/issue/58/forced_source_db_ingest/python/desc/twinkles/lightCurveFactory.py
and I put it to work in a study on differentiating SN light curves from AGNs:
https://github.com/jchiang87/desc_projects/blob/master/twinkles/ExploringTwinklesRun1.ipynb
That notebook runs in a few minutes.
I'm hoping that the db-based light curve code can eventually go into the Monitor package.   We'll need to have a scalable MySQL db service set up to make use of it though.
Let me know what you guys think.

This is great to have this data in a sql database and yes, I agree this is the kind of thing we would like to have in Monitor. I have some more specific comments, which I will put in sometime today. The quick question: I assume you authenticate via your profile, is there a way I could run this code and query the database from here?

@rbiswas4 I'll need to check on how to get access to those tables for you.  At a minimum, you'll need a SLAC linux account.   Once I have your userid, I'll contact the person administering that server (@slm23) to set up your credentials.

This is really great and the notebook is really interesting. Can you explain more about the changes you made to adding the bandpasses to the registry for sncosmo? I don't quite understand why you need the des and sdss bands as templates?
Also, do you mind if I take the non-database dependent parts of your updated monitor-like code and update the monitor with them now? Finally, do you mind if I point to this when I talk about the monitor during the SSim telecon tomorrow?

@jbkalmbach Sure, feel free to copy any bits over to Monitor that seem useful and to point to this stuff tomorrow at SSims.
Regarding the bandpassses, the problem I was finding was that the sncosmo code would only fit bands that were covered by the model.  All of the LSST throughput data are defined from 300 nm to 1200 nm, so none of them were covered by the model I was trying to use (salt2).   The relevant lines of code in sncosmo appear to be here where b.wave[0] and b.wave[-1] would be 300nm and 1200nm, respectively.  So the overlap always ends up being False.  I tried thresholding the throughput values at 2e-4, but that still caused problems,.  So since I knew the sdss and des bands must already work, I used their ranges to limit the lsst ranges.   In the end, I think we just need to have the lsst throughput files themselves contain the appropriately narrow wavelength ranges.

@jchiang87
Yes, that has been a problem. I think the way we have been doing this is to replace 'salt2' by 'salt2-extended'. This is of course not 'correct' but neither are the simulations, and a more principled approach would emerge from the photometry bake-off.
Also, do you think we could run your ingest script and create that mysql db locally so that we can all try it out?

Also, do you think we could run your ingest script and create that mysql db locally so that we can all try it out?

Definitely.  I'm hoping that bin/ingest_forced.py just works out of the box,  but you'll also need to run a modified version of
https://github.com/jchiang87/desc_projects/blob/master/twinkles/ingest_ref_catalog.py
and if you want the Chisq table, this script:
https://github.com/jchiang87/desc_projects/blob/master/twinkles/chisq_dists.py
I don't think I have a separate script for generating the CcdVisit table, but to do that it would just be
from desc.twinkles.db_table_access import CcdVisitTable
ccdvisit = CcdVisitTable(db=<db_name>, read_default_file=<file with credentials>)
ccdvisit.ingestRegistry(<full path to registry.sqlite3 file>)

Everything except ingest_forced.py should just take a couple minutes.
Let me know how it goes!
PS:  I'm planning to do multiple inserts in the ForcedSourceTable.ingestSourceCatalog method, which should speed things up, but I'm not sure by how much.  I haven't tried running multiple ingest jobs in parallel, but that would be another approach.

BTW, my file with the connection info for the MySQL server I'm using has these entries filled
[client]
host=
port=
user=
password=
#database=

I have a couple different dbs that I use, so I have the database entry commented out.

@rbiswas4 @jbkalmbach
I just pushed b3864bd with the multiple insert per query change.  This speeds up the ingest by about a factor of three or so.  On the rather old MySQL server I'm using, that corresponds to an ingest time of ~30 seconds per 47k object forced source catalog.

Or 8-9 hours for Run 1, I guess. Thanks Jim!
On Thu, Mar 24, 2016 at 1:44 PM, James Chiang<EMAIL_ADDRESS>wrote:

@rbiswas4 https://github.com/rbiswas4 @jbkalmbach
https://github.com/jbkalmbach
I just pushed b3864bd
https://github.com/DarkEnergyScienceCollaboration/Twinkles/commit/b3864bd14aa799b9dc8cb006281b476c2e4f44fa
with the multiple insert per query change. This speeds up the ingest by
about a factor of three or so. On the rather old MySQL server I'm using,
that corresponds to an ingest time of ~30 seconds per 47k object forced
source catalog.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/DarkEnergyScienceCollaboration/Twinkles/pull/184#issuecomment-201012619

Eep, something went wrong over at travis.... Otherwise, can this be merged?

I think so.  The failure was on the last push on the branch, which doesn't yet have the .travis.yml config.  The run on the PR, which is what would run after the merge, succeeded.  (Although, the db related tests were skipped...I'll fix that after the merge...)

Alright! Let's do this then.


Subselect for Join operations

Is there any possibility to build a query which uses a join with subselect, eg:
$t2 = $qb->table('table2')->select(array('t2.id'));
$qb->table('table1 as t1')->select()->join(array('t2' =>$2));

The code above would translate to:
select * from table1 as t1 join (select id from table2) as t2 on t2.id = t1.ref

This is currently not possible. But I've looked into it and it seems to be not that hard to implement it.


Add ability to customize templates

I often use the same labels to configure my reverse proxy (caddy, traefik, etc) and started to customize the template in the generate .js file so I'd have two more fields after the container name to configure the domain name and port for the proxy, which generates the labels for me automatically.

This would then add the labels like this and add the container to the right network:

However those changes get overwritten on every update. Could you add a possibility to add custom templates?
Or would it be easier to fork the project, customize them and create a docker image from that for this purpose?

Hi @nahakiole - we've been discussing this one internally and would like to know a bit more about your specific workflow here, so that we make sure we're coming at the issue from the right direction. Are you able to go into some more detail as to how you're doing things here?

Hey, I'm using portainer to start my docker container and I'm using caddy as a reverse proxy and to provide a ssl certificate for my docker container. To configure the reverse proxy I need to add certain labels to the container like this and add them to the caddy network.
version: '3.7'
services:
  whoami:
    image: jwilder/whoami
    networks:
      - caddy
    labels:
      caddy: whoami.example.com
      caddy.reverse_proxy: "{{upstreams 8000}}"

networks:
  caddy:
    external: true

https://github.com/lucaslorentz/caddy-docker-proxy
This is however tedious to do manually, so I've tried modifying the compiled .js and mounting it inside the portainer docker container with my changes. But after every update the file changes and I have to add my changes all over again.
So it would be nice to be able to overwrite certain template files and add functions in a way. Or for this particular use case make it possible to set labels like in the screenshot and add them to the network automatically.
I hope this clears things up more, else feel free to ask more questions.

Have you considered defining an App Template for that?
https://docs.portainer.io/v/ce-2.9/advanced/app-templates

The main reason I want this feature would be to have new input fields. The App Template could solve part of the problem of providing the default values, but to solve the issue I'd still need the input fields at the top for easy editing.

Can you please elaborate?
For instance the name and database root password fields for the Wordpress App Temlate, are defined in the json and show as

As far as I can see in the examples this sets a environment variable. For my use case I need to set the labels. Or can I set a environment variable and set the label to the value of it?

Ah, you're right. Makes sense now.

The idea of using app templates and then using environment which can be used inside labels could actually be a good solution. But is it possible to create a app template where you can still choose the docker image?

If we'd allow substitution variables anywhere then that would work I think.

The other option, which would work, would be to make a fork of the projekt, add my changes and build the docker container myself. Then I could just pull the changes from your project when you update it.
This might be the easiest option. I also added automatic links to the web interface of the container in the container list.

We are open to contributions, so feel free to create a PR.
For more info, please see:
https://github.com/portainer/portainer/blob/develop/CONTRIBUTING.md


Add some module descriptions and fill package.yml details.

Also update ChangeLog.md similar to https://github.com/coingaming/signable/blob/master/haskell/ChangeLog.md#version-01
And update README.md


[New Port Request] gnutls

Library name: GnuTLS
Library description: GnuTLS implements the TLS/SSL (Transport Layer Security aka Secure Sockets Layer) protocol.
Source repository URL: https://github.com/gnutls/gnutls
Project homepage (if different from the source repository):
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

can you please hurry up to port this library faster?!

It is my understanding that GNUTLS is already available! Not sure what's going on here?

so, you're wrong, cause it's not available


[Impeller] Wire up support in OpenGL for KHR_blend_equation_advanced.

KHR_blend_equation_advanced, needed for fast advanced blends is available on more (> 50%) of devices per GPUInfo.

We already wire up support for GL_EXT_shader_framebuffer_fetch in OpenGL but support for that is lower at ~30%.

Importantly, Skia also wires it up. So Impeller doing that will match Skias level of support.
Forked from the umbrella issue for fast advanced blends via extensions as described in https://github.com/flutter/flutter/issues/120223.

IMO if we do this then we should do the Vulkan equivalent. Its theoretically faster too, since we don't need to do a render to texture.

Actually, the reason we can't easily do this is that it requires decorating shaders with metadata that isn't supported in the GLSL language version we target.

Closing for now, but unless we build a shader compiler pipeline for newer GL ES versions we cannot use this feature.


Create @types for the domparser module

Created at https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/dom-parser .

Thank you @gbidkar!
JFYI, since version 1.x.x types are included in the package.


[sdc] Update USB constraints and move CDC constraints to check-only file

Since it is expected that all CDC paths are short enough, and since set_max_delay interferes with the currently used  set_clock_groups -async directive, the CDC constraints are moved to a separate SDC file that is used for timing analysis only (after P&R) to check that all relevant crossings are within spec.
The USB constraints are also updated so that the skew between differential signals is minimized.
See #13011 for context.

LGTM. Thanks for the hard work: This looks like it took some digging!
I wonder if there is some way to annotate targets for this in the RTL and query them in whatever timing engine we're using. If we always use prim cells and keep a list, though, that might be good enough.

Yeah some methodology to audit this would be nice.
We've tried to stick to prims so that the paths are easily discoverable.
But that also means that paths not using any prims might get missed...

Still needs some minor fixes for CI, though.

I am going to merge this now so that it is part of RC1. We will likely iterate on these constraints as we work towards the next release.


C:\Users\carlo\AppData\Local\Microsoft\PowerToys\PowerToys Run\Logs\<IP_ADDRESS>\2024-10-23.txt

Microsoft PowerToys version
0.85.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
Lately my Chrome browser is crashing about every half hour allegedly due to lack of memory.
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22631.0
IntPtr Length: 8
x64: True
Date: 10/23/2024 12:49:30 PM
Exception:
System.Runtime.InteropServices.COMException (0x80263001): {Desktop composition is disabled} The operation could not be completed because desktop composition is disabled. (0x80263001)
at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

Since the next the last update in Chrome browser, Chrome has been crashing regularly about every half hour due to alleged lack of memory.

/dup https://github.com/microsoft/PowerToys/issues/30507


New version: Tyrrrz.LightBulb version 2.5.1

Created by 🥟 Dumplings in workflow run #1693.
Logs
Updated: 2.5 → 2.5.1
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.
Submitting WinGet manifests
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.

Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Hi @Exorcism0666,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 881.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-05-26 13:33:11.733 [FAIL] Installer failed security check. Url: https://github.com/Tyrrrz/LightBulb/releases/download/2.5.1/LightBulb-Installer.win-arm64.exe Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 881.)

@wingetbot run

Automatic Validation ended with:
(Automated response - build 881.)


Other text areas still have the suggestion button

https://github.com/ordinall/BetterDiscord-Stuff/blob/1de094e48aa400fdba5979b23afe47ee69b906c1/Plugins/DisableStickerSuggestions/DisableStickerSuggestions.plugin.js#L107-L109
To fix this simply repeat the above for the NativeTextAreaContextMenu module.

Thank you for the fix, problem is fixed in 1366694b4b3cfc29e5c71a6474a805f1cac902fd


(Bug) empty error message from FV in amplitude

more than 50% of FV_GENERALERROR event reason field is empty

I checked paths / user records for the cases reason in empty.
All corresponding paths are starting from FV_GENERALERROR, no events were dispatched before
That could mean only one: the user was reloaded the app on the error page.
In such case screenProps.screenState contains no 'error' object. So get(screenProps, 'screenState.error.message') returns undefined
I fixed onmount effects of each error component in FV and prevent event dispatch if screenState doesn't contain an error object
PRs:
https://github.com/GoodDollar/GoodDAPP/pull/2681
https://github.com/GoodDollar/GoodDAPP/pull/2682

@serdiukov-o-nordwhale @sirpy  The issue is not reproduced. After refreshing the page the event is not sent to the sentry again.
Checked on the QA 1.21.0-3
Device:  Desktop // Windows 10 // Chrome v.87.0.4280.88
Attachment: https://www.dropbox.com/s/8itogjryaix12wr/empty error message from FV in amplitude.mp4?dl=0

@serdiukov-o-nordwhale @sirpy  The issue is not reproduced. After refreshing the page the event is not sent to the sentry again.
Checked on the QA 1.21.0-3
Device:  Desktop // Windows 10 // Chrome v.87.0.4280.88
Attachment: https://www.dropbox.com/s/8itogjryaix12wr/empty error message from FV in amplitude.mp4?dl=0


Do not expose fieldupdates as a list. Hide implementation details ins…

…tead.
@vekterli PR

@vekterli I think it is now more conservative. PTAL


Configuration file path enhancement

Hello,
Current configuration file path behavior introduced in v1.1.2 (see https://github.com/replicatedhq/dockerfilelint/commit/b09bff92917adb77470605cd89bb43cc6627f8dd) only allow a directory for -c|--config option.
Is it possible to think to allow a full path name rather than just a folder.
i.e:  dockerfilelint -c /path/to/my_config_rc_file Dockerfile
Rather than : dockerfilelint -c /path/to/.dockerfilelintrc Dockerfile  (because filename .dockerfilelintrc is mandatory)
Thanks in advance

As nobody seems interrested by this feature request, I will close it !


After changing @State, the value of the old state is still obtained under WithPerceptionTracking

Description
Here is my code:
import Perception
import SwiftUI

@Perceptible
class MyFooStore {
  var shouldShow = false
}

struct FooView: View {
  @State var localShouldShow = false
  @State var store = MyFooStore()

  var body: some View {
    WithPerceptionTracking {
      VStack {
        if store.shouldShow && localShouldShow {
          Text("SHOW")
        }
        Button {
          store.shouldShow.toggle()
        } label: {
          Text("TOGGLE")
        }
      }
      .onAppear {
        localShouldShow = true
      }
    }
  }
}

#Preview {
  FooView()
}

I modified the value of localShouldShow in onAppear, and then clicked the button to change the value of shouldShow in the store. However, after clicking, "SHOW" does not appear. I found that the value of localShouldShow is still false when the view is rendered.
But if I change the order of the conditional statement to if localShouldShow && store.shouldShow, it works normally. Is this a bug in Swift Perception?
https://github.com/user-attachments/assets/1e16c95e-1e08-4c1a-b1d9-a5becd8b8f77
Checklist

[X] I have determined that this bug is not reproducible using Swift's observation tools. If the bug is reproducible using the @Observable macro or another tool from the Observation framework, please file it directly with Apple.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
View can update
Actual behavior
View not update
Steps to reproduce
use the code above
Perception version information
1.4.1
Destination operating system
Preview
Xcode version information
Version 16.1 (16B40)
Swift Compiler version information
swift-driver version: 1.115 Apple Swift version 6.0.2 (swiftlang-<IP_ADDRESS>.2 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0

Is this a BUG? 😭

Hi @zhuscat, this is a SwiftUI bug and it has to do with views with escaping closures. WithPerceptionTracking is one such view, but any view with an escaping closure will cause this problem. For example, here is an iOS 18 view that uses @Observable and GeometryReader to reproduce the exact same problem:
@Observable
class MyFooStore {
  var shouldShow = false
}

struct FooView: View {
  @State var localShouldShow = false
  @State var store = MyFooStore()

  var body: some View {
    GeometryReader { _ in
      VStack {
        if store.shouldShow && localShouldShow {
          Text("SHOW")
        }
        Button {
          store.shouldShow.toggle()
        } label: {
          Text("TOGGLE")
        }
      }
      .onAppear {
        localShouldShow = true
      }
    }
  }
}

The fix is to eagerly access @State in the view. Even doing something as simple as let _ = localShouldShow in the view would fix it. And the reason changing the order of && fixed things is because && uses an auto-closure for the right-hand side argument that prevents evaluating that expression if the left-hand side is false.
Since this isn't an issue with the library I am going to convert it to a discussion. Please feel free to continue the conversation over there!


Slick Center Mode - Animations Bug between last and first slide

Hello,
I'm using SLICK in center mode with some CSS animations but i have a bug, like a rebound of animations, between last and first slide.
Do you have any idea to stop this rebound ?
Thanks a lot !
Simon.
====================================================================
http://jsfiddle.net/SimonBak/fmo50w7n/1701/

I have the same problem.
Is there a solution to this problem?

@GoodProject look at my pull request #3307 the solution is in there

@PaulBarrett79 After get your work, I have the same problemes with the transition for first/end slide :/

I'm still having issues with slide index 0 not receiving 'slick-current' positioning. If I set initialSlide: 0 the slide is off the screen, i.e. not active.

For anyone who is still having a problem with this, I fixed mine using these css selectors:
.slick-slide[aria-hidden="true"]:not(.slick-cloned) ~ .slick-cloned[aria-hidden="true"] {  /* Fix when transitioning from last slide to the clone version of the first slide */  } .testimonials .slick-slide[aria-hidden="true"]:not(.slick-cloned) ~ .slick-cloned[aria-hidden="true"] { /* The default look of the slide */ }
Here's a fork of @PaulBarrett79 with this fix https://codepen.io/jemuel23euel/pen/dybGqPj

Can I just say jemuel23euel is The Man. I'd give him a cookie!

/* slide when not active*/
.slick-slide[aria-hidden="true"]:not(.slick-cloned) ~ .slick-cloned[aria-hidden="true"] {

}

/* slide when active (when play last to first) */
.slick-slide[aria-hidden="true"]:not([tabindex="-1"]) + .slick-cloned[aria-hidden="true"]  {

}
/* slide when active (when play first to last) */
.slick-slide[aria-hidden="true"] + .slick-cloned[aria-hidden="true"] {

}

Keep in mind that by setting aria-hidden="true" you are hiding your slider from screen readers.  Not good for accessbility.

Only this solution below worked for me:
    /* slide when active (when play last to first) */     .slick-slide[aria-hidden="true"][tabindex="-1"] + .slick-center {       transform: scale(1.2);     }     /* slide when active (when play first to last) */     .slick-slide[data-index="-1"] .slick-center {       transform: scale(1.2);     }

This works in my case, targeting .slick-center instead as the default and also works when play first to last
/* slide when active (when play first to last) */
.slick-center {
...
}
/* slide when active (when play last to first) */
.slick-slide[aria-hidden="true"]:not([tabindex="-1"]) + .slick-cloned[aria-hidden="true"]  {
...
}

To anyone who is still having a problem with this, I fixed mine using these css selectors:
/* slide when not active/center*/ .slick-slide[aria-hidden="true"]:not(.slick-cloned) ~ .slick-cloned[aria-hidden="true"] { }
/* slide when active/center */ .slick-slide[aria-hidden="true"]:not([tabindex="-1"]) + .slick-cloned[aria-hidden="true"] { }
Here's a fork of @PaulBarrett79's pen with this fix https://codepen.io/jemuel23euel/pen/dybGqPj

Working fine, thanks so much for the solution

I am experiencing  an issue with the first and the last slide when using centre mode. If the number of items is the same as the maximum items shown. This is easily replicated taking a fork of the above.
https://codepen.io/twodareis2do/pen/oNoMMRy

You just saved my day! Many thanks

Guys I know this is really old issue and may be it is out of date but just for help,
The issue is when you target multiple classes for center like:
.slick-slide.slick-active.slick-current.slick-center {
    transform: scale(1.3);
}

this will cause issue.
solution is just target only center class like this
.slick-slide.slick-center {
    transform: scale(1.3);
}
.slick-slide {
    transition: all ease .3s;
}

look at demo: https://jsfiddle.net/faisaljanjuah/9fffrd1r/


allow ticket property config

Summary
The ticket that is created to review the SDL Checklist ticket should be configurable and not hardcoded.
Requirements (place an x in each [ ])

[X] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.

The following point can be removed after setting up CI (such as Travis) with coverage reports (such as Codecov)

[ ] I've written tests to cover the new code and functionality included in this PR.

The following point can be removed after setting up a CLA reporting tool such as cla-assistant.io

[X] I've read, agree to, and signed the Contributor License Agreement (CLA).

Similar like the previous PR. Are you able to visit the following URL to sign the CLA? https://cla-assistant.io/slackhq/goSDL?pullRequest=18

 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.Jason Macgowan 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.


fixing duplicate resource issues

to: @chunyong-lin
cc: @airbnb/streamalert-maintainers
Background
Fixing bug Chunyong found in #1217
Changes

Updating terraform resource names.

Coverage remained the same at 95.386% when pulling 37c4d610e5469070f37d1d59fb6a753343e344b4 on ryandeivert-fix-athena-bug into 6df1ed22d180d4e5db8e732e89e04029c6f97bde on release-3-2-0.


Merge the two list panels into one and fix some GUI bugs

Coverage increased (+1.4%) to 67.437% when pulling 963dc4e03ebc1e7f357fa1d5f21541a9f1d26701 on xllliu:master into 8300a4f2ca15481c8ce791cc3990e1bb1289a1e1 on CS2103-AY1819S1-T12-4:milestone-v1.4.


Decommission kit services VM

This will return for the competition
Summary
:ronseal:
Code review
Testing

[ ] applied the configuration locally
[ ] manually validated the new behaviour
[x] ... is for chumps

Links

@RealOrangeOne as noted elsewhere, please can you test this.

$ ./scripts/apply --limit kitsvcs
[WARNING]: Could not match supplied host pattern, ignoring: kitsvcs
ERROR! Specified inventory, host pattern and/or --limit leaves us with no hosts to target.

Yep, that works.

The physical server was decommissioned to save costs and maintenance.

This will return for the competition

The physical server was decommissioned to save costs and maintenance.

This will return for the competition

:facepalm: sorry, apparently I'm failing to read this evening.


panOnScroll not work

hi~ Is scroll supported as a move option?
I run the example code, but it's not work. could you help me?
https://github.com/wbkd/react-flow/issues/1417

Hello!
panOnScroll does work as intended.
You have to activate it using the zoomActivationKeyCode.
By default it is set to Meta, but if you're for example on Windows it'll be hard to find the "Meta"-Key.
Set it to Shift and you should be able to just press Shift + Mouse Scroll to move around the pane.
If you have a mouse with a horizontal wheel (I have a Logitech here that has one) you can also move horiontally.

<VueFlow
                        :model-value="element"
                        :connection-mode="ConnectionMode.Loose"
                        :default-zoom="0.7"
                        :nodes-draggable="false"
                        :zoom-on-scroll="false"
                        :pan-on-scroll="true"
                        zoom-activation-key-code="shift"
                        @load="onLoad"
                    >
                        <Background />
                        <Controls />
                    </VueFlow>

when i do a scroll action, but the performance is zoom-in or zoom-out

I run the InteractionExample, but the performance is also zoom-in or zoom-out.

It should be Shift not shift (that's a specc from the browser, not my idea to capitalize the key names 😅)

It should be Shift not shift (that's a specc from the browser, not my idea to capitalize the key names 😅)

Oh~ my fault!
I want to display like this. but it's not work.  https://codesandbox.io/s/xenodochial-yalow-1pgk6
do the scroll action, I want to perform this operation to scroll instead of zoom. 😊

You mean panOnScroll without having to press a key?
That doesn't work in 0.3.x currently, sorry 😢
Currently the pan-mode is only working if a key is also pressed (unlike the example you showed where it just activates on scroll).
I can change that for the 0.4.0 release though.

You mean panOnScroll without having to press a key? That doesn't work in 0.3.x currently, sorry 😢 Currently the pan-mode is only working if a key is also pressed (unlike the example you showed where it just activates on scroll).
I can change that for the 0.4.0 release though.

yeah! sometimes panOnScroll means move by scroll. If it's true  and you want to zoom-in or zoom-out, you can specify zoomActivationKeyCode.
thx very much! Looking forward to version 4.0！😝

Yeah, I thought of implementing it this way:
zoomActivationKeyCode will be undefined by default.
If a keyCode is given, it will be mandatory to use said key to activate either zoom or pan/scroll.
If none is given the modes are available without a key.
Disabling the modes would just work by setting zoomOnScroll and/or panOnScroll to false.
Does that cover your use case?

Yeah！ The solution can cover my use case! 😬


Replace react-beautiful-dnd

react-beautiful-dnd has been deprecated

OK, figured out how to do this with pragmatic-drag-and-drop without it being too complicated. Instead of figuring out exactly what index the new location of the card is in onDrop, I'll just figure out:

Which card is it being dropped before/after.
Is it being dropped before or after?
Implement a state action for moveBeforeCard and moveAfterCard that take the key of the card, which is unique. Figuring out the new index etc. can be done there, instead of the onDrop-handler.


Merge preparation for amikai's itereach PR

#483

Sorry if I'm nitpicking here. I'm not sure if user an iter.Seq2 is the right thing to do here. Afaiaa the index part of the iteration is already provided by the range statement itself. The iterator here should be an iter.Seq if I'm not mistaken. Using Seq2 just makes it confusing since that would usually be used with returning errors.


Remove labels for element kinds

See https://bugs.eclipse.org/bugs/show_bug.cgi?id=520771

Got the relevant changes merged into lsp4e. Fixed on master.


feat(Tokens): add spacing tokens

The following spacing custom properties were added and all the components have been updated to use them in terms of margins/paddings spaces
:root {
  --bq-spacing-xs3: 2px;
  --bq-spacing-xs2: 4px;
  --bq-spacing-xs: 8px;
  --bq-spacing-s: 12px;
  --bq-spacing-m: 16px;
  --bq-spacing-l: 24px;
  --bq-spacing-xl: 32px;
  --bq-spacing-xxl: 40px;
  --bq-spacing-xxl2: 56px;
  --bq-spacing-xxl3: 64px;
  --bq-spacing-xxl4: 72px;
}

☁ Nx Cloud Report

⚠ Important: This organization may reach the limit of the FREE Plan and could be disabled at the end of the month. If this happens, remote caching and distributed task execution will no longer work. Your CI may stop working.

We didn't find any information for the current pull request with the commit 9e8ffaaa27d5b30d88612e0d7a5cccb9ac4f00dc.
Please verify you are running the latest version of the NxCloud runner.
Check the Nx Cloud Github Integration documentation for more information.
Alternatively, you can contact us at<EMAIL_ADDRESS>
Sent with 💌 from NxCloud.


don't run ci on jdk11 on windows

fixes #143

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

Coverage data is based on head (5d1d35a) compared to base (eb12b7d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #152   +/-   ##
=======================================
  Coverage   83.13%   83.13%
=======================================
  Files          48       48
  Lines        1269     1269
  Branches      175      175
=======================================
  Hits         1055     1055
  Misses        158      158
  Partials       56       56

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

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


No audio playing from standalone call

Songbird version: 0.4.1
Rust version (rustc -V): 1.7.8
Serenity/Twilight version: N/A
Output of ffmpeg -version, yt-dlp --version (if relevant): N/A
...
Description:
I am running songbird on its own, without serenity/twilight, as a standalone voice server. I am using kafka to read voice state/voice server updates from discord, which I then pass to Call::standalone as connection info. Code is in Github  repo.
A single call is created as shown:
 let cloned_connection_info = connection_info.clone();
                println!("{:?}", cloned_connection_info);
                tokio::spawn(async move {
                    let mut c = Call::standalone(
                        cloned_connection_info.guild_id,
                        cloned_connection_info.user_id,
                    );
                    c.connect(cloned_connection_info);
                    let music =
                        Input::from(File::new("/home/boomermath/discord_jukebox/music.mp3"));
                    c.play_input(music.into());
                });

...
Steps to reproduce:

Run code in repo
Join VC and send voice state and voice server update to kafka, which is then consumed by songbird voice server
Create standalone call
Debug output shows WS Thread finished when I try to connect and play from file, output before that seems normal

After playing around with the library code a bit, the code in src/driver/tasks/ws.rs in the run method of AuxNetworks is terminated due to a flume RecvError
...

I think you may be misinterpreting the shape of the API. If you're running a dedicated voice server to which you pass ConnectionInfo, you instead want to use Driver::new and Driver::connect.
Call::standalone is a bit of legacy cruft we should do away with (emphasis mine):

Creates a new, standalone Call which is not connected via WebSocket to the Gateway.
...
For most use cases you do not want this.

I think its existence predates songbird itself, and I currently know of no one who uses it.

Some other things worth trying:

Can you please await the call to connect and verify that it has actually succeeded?
Would you be able to attach a global event listener on CoreEvent::{DriverConnect, DriverReconnect, DriverDisconnect}?

Updated code:
#[async_trait]
impl EventHandler for EventHandlerCustom {
    async fn act(&self, ctx: &EventContext<'_>) -> Option<Event> {
        println!("{:?}", ctx);
        if let Some(evt) = ctx.to_core_event() {
            return Option::Some(Event::Core(evt));
        }

        return Option::None;
    }
}

 if check_connection_info(&connection_info) {
                let cloned_connection_info = connection_info.clone();
                println!("{:?}", cloned_connection_info);
                tokio::spawn(async move {
                    let mut driver = Driver::new(Config::default());
                    driver.add_global_event(
                        Event::Core(CoreEvent::DriverConnect),
                        EventHandlerCustom {},
                    );
                    driver.add_global_event(
                        Event::Core(CoreEvent::DriverReconnect),
                        EventHandlerCustom {},
                    );
                    driver.add_global_event(
                        Event::Core(CoreEvent::DriverDisconnect),
                        EventHandlerCustom {},
                    );

                    let connect_res = driver.connect(cloned_connection_info).await;

                    match connect_res {
                        Ok(_c) => {
                            println!("Connected!");
                        }
                        Err(e) => println!("{:?}", e),
                    }

                    let music =
                        Input::from(File::new("/home/boomermath/discord_jukebox/music.mp3"));
                    driver.play_input(music.into());
                });

The event listener only prints out the the DriverConnect event like so:
DriverConnect(ConnectData { channel_id: Some(ChannelId(810276988090581015)), guild_id: GuildId(810276987465760800), session_id: "4f43a792dcaac44babd67d48d8a46234", server: "us-east4912.discord.media:443", ssrc: 81474 })

with no DriverDisconnect event, but the songbird::driver::connection prints out Disconnected, followed by songbird::driver:tasks::ws printing out changed speaking state microphone. Tracing shows the message "WS Thread finished" after disconnect/speaking state msgs. No errors from driver connect

Could you please try and store the Driver in a global (or at least longer-lived) Arc<RwLock<HashMap>> ? It looks like it's being dropped when that task ends, which then makes the sudden (no-event) WS closure expected behaviour -- the thread pool etc. all shut down when the last reference to a call is lost to prevent resource leaks.

Creating a global hashmap works! Thank you so much for your help!

Closing


Implement Naive Bayes classification

Related Resources to Follow :-

https://towardsdatascience.com/naïve-bayes-from-scratch-using-python-only-no-fancy-frameworks-a1904b37222d
https://machinelearningmastery.com/naive-bayes-classifier-scratch-python/

I would like to work on this issue.

PR #27 Solves this issue. Hence closing this issue.


adding point ingestion from YGM containers to dnnd API

I'm not sure if we want to add support for YGM container ingestion to dnnd_advanced.hpp. I recall that there was an issue with the metall allocators, but I'm not sure whether my solution for dnnd_simple.hpp avoids it or not. Let me know.

It would also be nice to pass a YGM container to dnnd::query() and dnnd::query_with_features(), although this would require either (a) an internal copy of the contents to a temporary vector, or (b) a significant overhaul of dndetail::dknn_batch_query_kernel, neither of which seems desirable at this time.


Target Platform Intrinsic Functions

@rainersigwald I've updated the NuGet dependency to get the latest version, which has the target platform changes. Is this the right way to go about it?

Looks like the test failures are unrelated to this PR.
@rainersigwald what's the timeline with working to 16.7?

Test failures should have been fixed by #5439.

/azp run

All tests passed. Good to go?

@wli3 see https://github.com/microsoft/msbuild/pull/5429#pullrequestreview-432715609, we're waiting until the fork for 16.7

i see. looks like we are in a lock for releases dates. cswinrt need to be in 5.0.0-preivew 7. I will just communicate that it won't be the final form. as in you can just use net5.0-windows. You need to set <TargetPlatformVersion> separately.

Looks like we need to rush a bit after the branch cut. cswinrt wants to be done in preview 8. 7/15 is the branch cut date. After this is merged, we need to get on the task of parsing out $(TargetPlatformVersion) quickly. Although, it is possible for you to write to code for parsing out $(TargetPlatformVersion) now? So we can just merge the code after 7/2.

Get a build of this local msbuild and then swap the content inside stage 0 for you local SDK might work. We can try together

@wli3, I have the skeleton of the code written and I already did a little bit of testing with this MSBuild change in my stage 0. If you're going to be blocked I can clean it up and test some more tomorrow and put out a draft PR so you can review.

That's great! Let's do that

Once NuGet/NuGet.Client#3479 is merged we'll want to update the NuGet version number to point to the updated implementation, since the one we're using at the moment has known bugs.

We could merge it today?

@wli3 We're going to want to update the NuGet version number once NuGet/NuGet.Client#3479 is merged, so we're still blocked on that. Right now this is implemented on top of the buggy target platform work.

@sfoslund I converted this to a draft so we don't look at it in triage until the NuGet thing is resolved. Can you promote it to "ready for review" when that's done and consumed here?

@rainersigwald I've updated the nuget version so this should be ready to go.

@rainersigwald ?

error MSB3243: (NETCORE_ENGINEERING_TELEMETRY=Build) No way to resolve conflict between "Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" and "Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a". Choosing "Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" arbitrarily.

@rainersigwald updating the master nuget reference seems to create some version mismatches. Is there a simple way to solve this?

@sfoslund add it here:
https://github.com/microsoft/msbuild/blob/fdddb27ebec8f6a51513b6e869f039a32d3a8c39/eng/BootStrapMSBuild.targets#L47-L61


Should the examples be updated to deal with expired sessions

As far I can see, the example code only tries to join on setup() but if the session expires (i.e. as in the case of the V2->V3 migration), the node would not be able to realize and would keep sending messages that don't arrive anywhere.

Yes we indeed have a problem here. I was hoping that the RN2483 had LinkCheck enabled by default, but apparently not:

In other stacks like LMiC the examples also disabled LinkCheck, and it was up to the user to enable ADR and LinkCheck:
https://github.com/matthijskooijman/arduino-lmic/blob/master/examples/ttn-otaa/ttn-otaa.ino#L93
This is therefore a wider topic than just the arduino device lib.
Personally I have many devices out in the field using LMiC, but I believe I enabled LinkCheck and ADR on them. And hopefully that will cause a re-join.


Creating a Blazor Server-Side Application Using ASP.NET Core To Perform CRUD Operations

Topic Suggestion
Creating a Blazor Server-Side CRUD Application Using ASP.NET Core
Writing sample(s):
https://www.section.io/engineering-education/java-timer-tasks/
Proposed title of the article
Creating a Blazor Server-Side CRUD Application Using ASP.NET Core
Proposed article introduction
Server-side processing occurs when a page is requested for the first time and when pages are posted back to the server. For example, validation, saving and retrieving data, and navigating to other pages are all examples of server-side processing.
In this tutorial, we will learn how to make a Blazor Server-Side Application from scratch and perform CRUD operations in it.
The types of CRUD operations we will perform include;

creating a data list
reading data from the list
updating the list
deleting data from the list.

Key takeaways
After reading through this tutorial, the reader should be able to;
have a better understanding of what CRUD operations are.
be able to create a blazor server-side CRUD application using ASP.NET Core that adds, deletes, and edits data in a blazer server app.
If the reader is a beginner, be able to have a syntax of C# from this article.
Article quality
In this article, I will be taking the reader through step by step process on how he or she can be able to create a Blazor Server-Side CRUD Application Using ASP.NET Core and a detailed explanation of what the code he or she writes in each model does.
References
N/A

Sounds like a good topic @Joesongoma - Approved.


file upload not working

i think bee-js file upload is not working. i cloned the ethersphere/examples-js and updated the bee-js to 6.4.0.
when i use the upload-react i get a bzzHash but if i try to open via browser the file/image is not found.
but it is working via swarm-desktop fileupload.

Hi, I have the same issue.
It appears that the problem started with version 6.0.0 due to the change in the http library.
When I revert to version 5.2.0, everything works.
Here is the code I'm using in a Svelte environment:
import { Bee, FileUploadOptions } from "@ethersphere/bee-js";
import { DEFAULT_NAME } from "@lib/common/config";
import { swarmApiEndpoint, swarmApiKey, SWARM_ZERO_APIKEY } from "@lib/nft/storage/swarm";

const getBee = (nodeUrl: string): Bee => {
  return new Bee(nodeUrl);
};

const swarmUploadFile = async (file: File | string): Promise<string> => {
  const nodeUrl = swarmApiEndpoint();
  const batchId = swarmApiKey();

  const bee: Bee = getBee(nodeUrl);

  const isFile = file instanceof File;

  console.log("FILE : ", file);

  const options: FileUploadOptions = {};
  options.pin = true;
  if (isFile && file?.size > 0) options.size = file.size;
  if (isFile && file?.type) options.contentType = file.type;

  const result = await bee.uploadFile(
    batchId,
    file,
    isFile ? file.name : DEFAULT_NAME,
    options
  );

  return result.reference;
};

export { swarmUploadFile };

This appears at page load

and here is i get after upload:

@savage9309 @dlaxcess can you try the latest bee-js v6.7.4 and see if this issue persists.

Oh it works much better now :)
Still have this error in console : Refused to set unsafe header "content-length" but it seems that it doesn't prevent the upload.
Thx


TypeError: jit() got an unexpected keyword argument 'donate_argnames'

Very simple toy neural network runs on tensorflow backend but not on jax. Intel Silicon macOS. Below is a full example and next, the error code when running on jax. I'm sharing this in case it can be helpful to further smoothen the interoperability between backends.
Code:
import keras
import numpy
import pandas as pd
from gingado.utils import list_all_dataflows, load_SDMX_data

df_cpi = load_SDMX_data(
    sources={'BIS': 'WS_LONG_CPI'},
    keys={'FREQ': 'M'},
    params={'startPeriod': 2002}
)

df_cpi = df_cpi['2002-01-01':'2023-09-01']
cpi_cols = df_cpi.dropna(how='all', axis=1).columns
# keep only one observation per country, retaining only the ones ending in 628 (in opposition to 771)
df_infl = df_cpi[[c for c in cpi_cols if '628' in c and c != 'BIS__WS_LONG_CPI_M__AE__628']].pct_change(periods=1).dropna(how='all')
df_infl.columns = [c.split('__')[2] for c in df_infl.columns]

train_cutoff = '2013-01-01'
valid_cutoff = '2020-01-01'
df_infl_train, df_infl_valid, df_infl_test = df_infl[:train_cutoff], df_infl[train_cutoff:valid_cutoff], df_infl[valid_cutoff:]
df_infl_train = df_infl_train[:-1]
df_infl_valid = df_infl_valid[:-1]

train_scale = {
    'mean': df_infl_train.mean(axis=0),
    'std': df_infl_train.std(axis=0)
}

def normalise_df(df, scale=train_scale):
    norm_dfs = []
    for cty in df.columns.tolist():
        norm_dfs.append((df[cty] - scale['mean'][cty]) / scale['std'][cty])
    return pd.concat(norm_dfs, axis=1)

def renormalise_df(df, scale=train_scale):
    renorm_dfs = []
    for cty in df.columns.tolist():
        renorm_dfs.append((df[cty] * scale['std'][cty]) + scale['mean'][cty])
    return pd.concat(renorm_dfs, axis=1)

df_infl_train_n = normalise_df(df_infl_train)
df_infl_valid_n = normalise_df(df_infl_valid)
df_infl_test_n = normalise_df(df_infl_test)

def dense_layer_io(
    df, # Pandas DataFrame with the data
    idx:[Int], # DataFrame indedx
    col:[str], # Column name
    p:Int # Number of lags
):
    # Return an (input, output) tuple for a linear_layer model

    assert len(idx) == len(col)

    batch_X, batch_y = [], []
    for i in range(len(idx)):
        X = df.iloc[idx[i]:(idx[i]+p)][[col[i]]].values.reshape(-1)
        if np.isnan(X).any():
            continue
        batch_X.append(X)
        batch_y.append(df.iloc[idx[i]+p][[col[i]]].values)

    return np.vstack(batch_X), np.vstack(batch_y)

def random_select(df, p, n_samples):
    max_index = len(df) - p
    rand_idx, rand_col = [], []
    for _ in range(n_samples):
        rand_idx.append(random.randint(0, max_index-2)) # instead of -1 only, we also need to account for the target value
        rand_col.append(random.choice(df.columns))
    return rand_idx, rand_col

# get data

n_samples = 1000
p = 24

idx, col = random_select(df_infl_train_n, p=p, n_samples=n_samples)
train_samples = dense_layer_io(df=df_infl_train_n, idx=idx, col=col, p=p)

idx, col = random_select(df_infl_valid_n, p=p, n_samples=n_samples)
valid_samples = dense_layer_io(df=df_infl_valid_n, idx=idx, col=col, p=p)

# build and train model
nn_linear = keras.Sequential([
    layers.Input((p,)),
    layers.Dense(32, activation='relu'),
    layers.Dense(5, activation='relu'),
    layers.Dense(1)
])

nn_linear.compile(
    loss=keras.losses.MeanSquaredError(),
    optimizer=keras.optimizers.Adam(learning_rate=1e-3),
    metrics=[
        keras.metrics.RootMeanSquaredError(name="rmse")
    ]
)

assert nn_linear(train_samples[0]).shape == train_samples[1].shape

history_dense = nn_linear.fit(
    x=train_samples[0],
    y=train_samples[1],
    batch_size=10,
    epochs=500,
    validation_data=valid_samples,
    callbacks=[keras.callbacks.EarlyStopping(patience=10)]
)

Error when backend is jax:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/Users/.../test.ipynb Cell 124 in <module>
----> 1 history_dense = nn_linear.fit(
      2     x=train_samples[0],
      3     y=train_samples[1],
      4     batch_size=10,
      5     epochs=500,
      6     validation_data=valid_samples,
      7     callbacks=[keras.callbacks.EarlyStopping(patience=10)]
      8 )

File ~/.venv/lib/python3.10/site-packages/keras/src/utils/traceback_utils.py:123, in filter_traceback.<locals>.error_handler(*args, **kwargs)
    120     filtered_tb = _process_traceback_frames(e.__traceback__)
    121     # To get the full stack trace, call:
    122     # `keras.config.disable_traceback_filtering()`
--> 123     raise e.with_traceback(filtered_tb) from None
    124 finally:
    125     del filtered_tb

File ~/.venv/lib/python3.10/site-packages/keras/src/backend/jax/trainer.py:254, in JAXTrainer.make_train_function(self, force)
    246     train_step = one_train_step
    248 if not self.run_eagerly and self.jit_compile:
    249     # Note that we mark the state and data to be donated to jax,
    250     # so that jax will reuse the memory buffer for outputs.
    251     # This will reduce the memory usage of the training function by
    252     # half.
    253     @partial(jax.jit, donate_argnames="state")
--> 254     def compiled_train_step(state, data):
    255         return train_step(state, data)
    257     self.train_function = compiled_train_step

TypeError: jit() got an unexpected keyword argument 'donate_argnames'

jax == '0.4.11'
keras == '3.0.1'

try upgrade your jax version

Hi @dkgaraujo ,
The jit function from Jax don't have the argument donate_argnames in jax == '0.4.11' version.It might have added at later versions.
You can refer the same from Jax repo as below.
https://github.com/google/jax/blob/48ad9a6f3e72f19f5b86f3bb014587e6bc6c0c4c/jax/_src/api.py#L148
Please upgrade Jax to higher version where  the argument donate_argnames exists say for eg like jax == 0.4.20.
Thanks.

Thank you all for the responses.


Adding sigmageek in competitive programming

What does this PR do?
Add resource(s)
For competitive programming
Adding Sigmageek on competitive programming section
Why is this valuable (or not)?
How do we know it's really free? just access and let code
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Is it free?

yes
Em qua., 12 de out. de 2022 às 23:40, Eric Hellman @.***>
escreveu:

Is it free?
—
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/pull/8189#issuecomment-1276952977,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADVXNDVDCQXALMJTZF3BXPLWC5ZB3ANCNFSM6AAAAAARDX25SE
.
You are receiving this because you authored the thread.Message ID:
@.***>

sorry for delay. see #9109
Thanks, and welcome to free-programming-books!


Django Rest Framework Theme Integration

Hi, I'm not sure if this is the best way to announce/discuss this but thought it might be useful for the cookiecutter-django community.
This DRF PR was just merged into DRF and has instructions for making a quick website using the DRF base templates and cookiecutter-django's account management templates.
Of course it isn't released yet so in the meantime I just copied the new file to my templates at my_project/templates/rest_framework/base.html.
Best regards

This is good news. At some point, I'll have time to work with @tomchristie on a good integration between DRF and Cookiecutter Django. Perhaps after I finish my current project.

I've tried to take a look at that old one, but infortunately, DRF browsable API is still on Bootstrap v3, while we're currently outputting a project using v5. I'm not sure what you had in mind with this idea, but this is the code I have with a custom bootstrap compilation, I'm reusing the generated CSS and including it in a custom rest_framework/api.html template:
{% extends "rest_framework/base.html" %}
{% load static %}

{% block bootstrap_theme %}
  <link href="{% static 'css/project.min.css' %}" rel="stylesheet">
{% endblock %}

And how it looks:

The navbar seems a bit broken and the dropdown on the button doesn't open. More subjectively, I also think the original theme looks better.
While this might be fixed pretty soon when this PR is merged, I'm not confident in providing this in Cookiecutter Django. When Bootstrap 6 comes out, it would potentially be one more blocker for us to upgrade.
Was it the solution you had in mind @jeffcjohnson? Am I missing anything?


Terraform plan fails with "Invalid provider configuration was supplied" warning when changing cluster subnets

Description
When adding a new subnet, or replacing one of the existing ones with a new one, or removing a subnet, Terraform fails to plan. It seems the provider configuration is not properly initialized, and it tries to access the k8s API server at localhost instead of the proper host when refreshing the aws_auth configmap. Reordering subnets do not cause the issue. The new subnet itself was created by terraform in a previous apply, without any problem.
EDIT: I now created a completely stripped down test case to easily reproduce the issue: https://github.com/gpothier/terraform-aws-eks-issue-1707. I particular, it doesn't use a submodule anymore, everything is in root. This is a pretty much by-the-book usage of the EKS module.
Versions

Terraform:
Terraform v1.0.11
Provider(s):

provider registry.terraform.io/hashicorp/aws v3.67.0
provider registry.terraform.io/hashicorp/cloudinit v2.2.0
provider registry.terraform.io/hashicorp/helm v2.4.1
provider registry.terraform.io/hashicorp/kubernetes v2.6.1
provider registry.terraform.io/hashicorp/local v2.1.0
provider registry.terraform.io/hashicorp/null v3.1.0
provider registry.terraform.io/hashicorp/random v3.1.0
provider registry.terraform.io/hashicorp/template v2.2.0
provider registry.terraform.io/terraform-aws-modules/http v2.4.1

Module:

Reproduction

Steps to reproduce the behavior:
EDIT:
Clone this project: https://github.com/gpothier/terraform-aws-eks-issue-1707
Plan & apply a first time, then change the value of autoscaling_azs in terraform.tfvars from 2 to 3, and attempt to plan again. It will fail. It also fails if initially applying with 3 subnets and then changing to 2.
Expected behavior
Terraform should be able to produce a plan regardless of subnets used in cluster
Actual behavior
Terraform fails to produce a plan
Terminal Output Screenshot(s)
Terraform plan final output:
╷
│ Error: the server is currently unable to handle the request (get configmaps aws-auth)
│
│   with module.couchdb_cluster.module.eks_cluster.kubernetes_config_map.aws_auth[0],
│   on .terraform/modules/couchdb_cluster.eks_cluster/aws_auth.tf line 63, in resource "kubernetes_config_map" "aws_auth":
│   63: resource "kubernetes_config_map" "aws_auth" {
│
╵

And this is somewhere in the middle with TL_LOG=TRACE:
-----------------------------------------------------: timestamp=2021-11-29T16:16:10.047-0300
2021-11-29T16:16:10.058-0300 [DEBUG] provider.terraform-provider-kubernetes_v2.6.1_x5: 2021-11-29T16:16:10.058-0300 [TRACE] [PrepareProviderConfig][Request]
2021-11-29T16:16:10.058-0300 [DEBUG] provider.terraform-provider-kubernetes_v2.6.1_x5: %s
2021-11-29T16:16:10.058-0300 [DEBUG] provider.terraform-provider-kubernetes_v2.6.1_x5: : EXTRA_VALUE_AT_END={0xc0007623f0}
2021-11-29T16:16:10.059-0300 [WARN]  ValidateProviderConfig from "provider[\"registry.terraform.io/hashicorp/kubernetes\"].couchdb" changed the config value, but that value is unused
2021-11-29T16:16:10.059-0300 [TRACE] GRPCProvider: ConfigureProvider
2021-11-29T16:16:10.059-0300 [DEBUG] provider.terraform-provider-kubernetes_v2.6.1_x5: 2021/11/29 16:16:10 [WARN] Invalid provider configuration was supplied. Provider operations likely to fail: invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable
2021-11-29T16:16:10.059-0300 [DEBUG] provider.terraform-provider-kubernetes_v2.6.1_x5: 2021/11/29 16:16:10 [DEBUG] Enabling HTTP requests/responses tracing
2021-11-29T16:16:10.060-0300 [DEBUG] provider.terraform-provider-kubernetes_v2.6.1_x5: 2021-11-29T16:16:10.060-0300 [ERROR] [Configure]: Failed to load config:="&{0xc00192c820 0xc00091e240 <nil> 0xc0000ee480 {0 0} 0xc00192ba60}"
2021-11-29T16:16:10.060-0300 [TRACE] vertex "provider[\"registry.terraform.io/hashicorp/kubernetes\"].couchdb": visit complete
2021-11-29T16:16:10.060-0300 [TRACE] vertex "module.main_cluster.module.iam_assumable_role_admin.data.aws_iam_policy_document.assume_role_with_oidc": expanding dynamic subgraph
2021-11-29T16:16:10.060-0300 [TRACE] Executing graph transform *terraform.ResourceCountTransformer
2021-11-29T16:16:10.060-0300 [TRACE] ResourceCountTransformer: adding module.main_cluster.module.iam_assumable_role_admin.data.aws_iam_policy_document.assume_role_with_oidc[0] as *terraform.NodePlannableResourceInstance
2021-11-29T16:16:10.060-0300 [TRACE] Completed graph transform *terraform.ResourceCountTransformer with new graph:
  module.main_cluster.module.iam_assumable_role_admin.data.aws_iam_policy_document.assume_role_with_oidc[0] - *terraform.NodePlannableResourceInstance
  ------

This issue has been resolved in version 18.0.0 :tada:

I can confirm that with v18.0.1, terraform plan completes without error; however, the apply fails because the cluster must be replaced, and it tries to create the cluster before destroying:
Cluster already exists with name: terraform-issue-1707-eks
Should I open a new issue, or reopen this one?

@gpothier its unclear what issue you are facing and what desired outcome you are trying to achieve. if you are still facing an issue on v18.x, I would suggest opening a new issue

ok, created issue #1752.


Add in Weave support/example in the "Editors and Tools" section?

We can base the installation on the internal conda setup for simplicity

i.e. I think we need to do a Conda.add("nbconvert") or whatever it is.


Pass RID to crossgen

Based on https://github.com/dotnet/buildtools/issues/2050
Contributes to https://github.com/dotnet/corert/issues/5857
cc @weshaggard, @dagood; please note that https://github.com/dotnet/corert/issues/5962 is an existing problem, PR shall not regress from the baseline :)

It looks like that this fixing the crossgen.sh: line 80: $2: unbound variable errror https://github.com/dotnet/corefx/issues/30043 ? If yes, could you please make the same fix in CoreCLR/FX as well?

@jkotas, yes. The second argument was made optional in https://github.com/dotnet/buildtools/pull/2061, so the latest build tools will not break. However, it would be great if all repos start passing this argument as crossgen throws non-fatal errors on Alpine and continues. I will prepare the PRs.


Enable device pairing using homeassistant builtins

This was brought up as https://github.com/gluap/pyduofern/issues/28
Currently pyduofern uses an old workaround: to allow pairing, a "service" has to be called. By now Homeassistant would allow for a nice user-friendly gui workflow. Another relic that is likely related is that we still write duofern.json in the background which shouldn't be necessary any more.
This bug is about
-> move pairing to the GUI framework that other integrations use
-> in the process: maybe get rid of duofern.json
Hopefully one outcome would be a list of services and entities as is provided for other integrations:

Isn't the new pairing flow in 0.5.x releases adressing this?
Did not pair a new shutter yet (but need to in the next days) so not sure about that.


floodlight with styles leaks light through solid brushes

The issue isn't that light leaks through brushes. The issue is that the light radius is ignored; every surface that gets hit by a styled light gets fully illuminated, and theres no intensity falloff based on the origin of the light.


feat(frontend-canister): add take_ownership method

Description
The SNS should be able to clear the Asset Canister’s list of authorized principals with a single call, leaving behind only one principal: the SNS itself.
Fixes https://dfinity.atlassian.net/browse/SDK-928
How Has This Been Tested?
Added an e2e test
Checklist:

[x] The title of this PR complies with Conventional Commits.
[x] I have edited the CHANGELOG accordingly.
[ ] (n/a) I have made corresponding changes to the documentation.

Just a naming suggestion - what about clear_authorized, since that's the language you reached for when describing it?

Just a naming suggestion - what about clear_authorized, since that's the language you reached for when describing it?

Because it doesn't leave the authorized list empty.  The method adds the caller, a controller, to the list.  I updated the description in the PR.