Requested a reward [.wanrick]

Guild id<PHONE_NUMBER>402363984 | Mowgli's GYM
Member id<PHONE_NUMBER>05653288
Locale Russian
Nickname VolanDeFor
Морковь - 5 шт. | Carrot - 5 pcs.


Changefeed - scale out support

Is your feature request related to a problem? Please describe.
Long running ChangeFeedIterator has no support for scale-out scenario, e.g. when data and nr. of partitions is growing, it's not possible to split the FeedIterator when partition was split.
In our scenario, we create new container, iterate FeedRanges, and for each FeedRange, we create FeedIterator. So at the beginning we end up with very few FeedIterators.
As the data grow to thousands of partitions, we still end up with the initial number of FeedIterators and no way to scale-out.
And the processing is a bit CPU/IO heavy to process everything on one server.
Describe the solution you'd like
Best case scenario would be notified, when the partition split happens, which FeedRange was split and what are the new FeedRanges.
Or to be able to check if FeedRange represent multiple partitions with possibility to split it into multiple FeedRanges
Describe alternatives you've considered
Changefeed processor. Not good for our use case. We need to be more in control how things are processed. e.g. more realtime processing, some ordering/dependency logic between partitions etc. And would require duplicating some of our infrastructure.
Additional context

@zbynek001 When looking at the processing of the change feed events, how is the time spent once the changes are received? Which is the biggest time sink in the processing?
The FeedRanges do not necessarily represent 1 partition (though they might), based on this decoupling, if there were no relationship with the number of partitions, and you could scale up to any number of iterators, how would you decide this number and why?

@ealsur we're running multiple different types of processing, each using different resources, some use only IO, some also need more cpu. At the end, we're not able to process the changefeeds without scaling-out.
Well, we don't necessarily need to do the scale-out on partition level. It was done that way, because it was the only way how to parallelize the changefeed in v2. And it has an advantage, that the changefeed of one partition represent the documents in the order they were created, which we're using for some additional logic.
Basically, as long as we're able to scale-out the processing to some desired number of parallel operation then it'd be good.
If the FeedRange would not represent partitions, what would then GetFeedRangesAsync return? In such scenario might be also good to support not only splitting of FeedRange, but also merging them together, but that might lead to different discussions.

When one vm cannot handle the amount of data it needs to process from changefeed than it needs to scale out.
Whether GetFeedRangesAsync returns FeedRanges representing partitions or not is not really important. What's important is to be able to split individual FeedRanges independently from other and to be able to continue processing where we left off before the split. One such scenario is when adding more instances to vm scaleset and wanting to distribute the load also to the new instances without as small interuption of processing as possible, i.e. without recreating all of the FeedRangeIterators


Devel

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

3 of 9 (33.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 94.257%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

jsonify.go
3
9
33.33%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.3%

Covered Lines:
1592

Relevant Lines:
1689

💛  - Coveralls


Modifies coord attrs lookup for non-coord levels on GRIB2 files

I ran across an interesting edge case while trying to read in some GRIB files from the NOAA HRRR model using @martindurant's awesome improvements from #198. The edge case occurs when you supply a typeOfLevel filter, but the level is not actually a coordinate in the dataset. The most common use case for this is selecting the surface fields (including certain thermodynamic fields like skin temperature, or winds). In this case, the attribute lookup into the decoded COORD_ATTRS on the dataset hat cfgrib produces will fail, but the scan_grib will keep on running through, producing empty references that need to be dealt with.
The changelist here is a very, very naive way of fixing this - simply grabbing the basic attributes that have already been inspected. I'm sure there are better ways. But this PR is primarily to exposure to the community and get feedback for a better solution.
To Reproduce

Grab a recent HRRR file from NOMADS (e.g. here, subbing in the most recent YYYYMMDD for the date)
Run the following code snippet:from kerchunk.grib2 import scan_grib
hrrr_fn = "/path/to/hrrr/grib"
grib_references = scan_grib(
    hrrr_fn,
    common=['time', 'latitude', 'longitude', 'valid_time']
    storage_options={},
    inline_threshold=0,
    filter={'typeOfLevel': 'surface', 'shortName': 't'},  # for selecting the single surface temp message
)

That's right, on that branch the debugs are still in, but they are gone on main.

Thanks for the feedback @martindurant and @emfdavid. I’ll clean this up tonight.
Apologies @martindurant - if I realized you wanted to cut a new release with the GRIB re-factor I would’ve expedited this :(

No problem, @darothen , releases are cheap

@darothen Thank you for solving this issue!

Will merge when green. We can decide later if we want to make these attributes of the top level dataset rather than coordinates.


R119 bcfix

Temporary fix for multidimensional broadcast

Build triggered. sha1 is merged.

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/196/

Build finished. No test results found.


sys-kernel: enable kernel virtio vnc output

Fix for: https://github.com/flatcar/Flatcar/issues/1553
TBD: check whether all the changes are actually required and the initrd / kernel size increase.

Not that it's a bad idea, but why do you need SIMPLEDRM? I found that the existing kernel works with QEMU's -display virtio-gpu-pci, but you just don't see anything between GRUB and pivot root, presumably because the module is not in the initrd.

I also wonder why CONFIG_EFI_FB isn't enough on arm64. That's what amd64 normally uses given that we don't pass -device virtio-vga there.

I also wonder why CONFIG_EFI_FB isn't enough on arm64. That's what amd64 normally uses given that we don't pass -device virtio-vga there.

made some tests and verified the minimal patch, and it seems that DRM and VIRTIO_GPU is needed at the same time (makes sense as they are in the same config tree).


benchmarking campaign

I would like to run a nice benchmarking campaign for this project.
The first thing I thought when I saw streama was: who knows if I can run it on my old orangepi zero? so, for fun.
Does anyone with me have many single boards and differentiated hardware? Help me do some tests to publish on the wiki
I would like to monitor the following things:

CPU
RAM
Disk usage
Bandswitch
Number of users
DB performance
SSD vs HDD vs MicroSD vs Floppy vs USB key/disk etc.
Virtual vs physical
Local storage vs remote storage
And many others.

Sorry for my english...

Hi Johnny,
This idea is brilliant a few things that we may need to standardise or account for would be the OS that the test is run on and also then the tools for recording performance doing the basics like CPU and RAM are easy but how would we record DB performance and bandwidth.
Also if dularion likes the idea then we can put something on the official Wiki when a certain amount of devices have confirmed that it works that its "stable" and if it works but not many people have confirmed then its "community tested"
Another category would be what IOPS are required as I know that some users are running streama on hosted servers.
I would be happy to test this on some different hardware, I have an assortment of raspberry pi's of various generations and a hyper-v server where my current instance where I have tested various memory amounts before and also he availability to spin up cloud servers to test this.


i have bing in trouble of this.

ImportError: cannot import name 'keys' from 'eth_keys' (unknown location)

I'm not able to replicate with v0.5.0. I'd recommend you try to reinstall with a new virtualenv.

i have found my problem,that because i install sloc.sloc have diference with solc.


react-stampit

https://github.com/stampit-org/react-stampit
I love it. Though I understand people prefer the somewhat broken/incomplete classes.

Sorry, mixins are no go. Feel free to convice me to opposite.

In order to do that I would need to know why they are no go?

For me classes + higher order component is a no go since statics are not forwarded.

As long as we keep using higher order components to approach the same thing as with mixins, I can't see any point of stopping mixins usage in React Native. Components composition is something I really like when using mixins, it's also less boilerplate to write. Waiting for the official solution made specifically for ES6.
Some examples:

AnimateMixin - adds animation to a component when state changes (LayoutAnimation in iOS)
TrackKeyboardMixin - tracks keyboard and adds this.state.keyboardPosition to your component so you can adjust position of e.g. your form when keyboard is toggled.

Mixins is composition...

Mixing is merging, HOC is composition. What does not work for you with higher order components? Mixings is super brittle, once someone will use handleChanges or similar method. React ES6 classes are ok, they allow you to add local state, which is ok for experimenting.

Mixins are anti-functional approach, exactly as Dan wrote. There are many most of the time "nice" features in any programming language, but the task of constant/linear development complexity system is to prefer pure funtional before any "nice" features whenever possible. If it's not possible due to environment, then that's the right place for low level abstraction to keep the app as functional as possible.

As I wrote, HOC does not forward statics.

If you already use statics, then is your "functional" level messed already so much, that you can copy/paste them. :-D

I'll refer to Eric Elliot to debate mixins. I'm not qualified further than having an opinion.
http://ericleads.com/2013/06/classical-inheritance-is-obsolete-how-to-think-in-prototypal-oo/
https://github.com/stampit-org/stampit
http://ericleads.com/2014/02/prototypal-inheritance-with-stamps/

If you already use statics, then is your "functional" level messed already so much, that you can copy/paste them. :-D

I don't quite understand this.

I've read those things about stampit. And I actually like it. But I would need more time to study it. About the statics - did you mean that higher order component doesn't automatically get the static properties of the wapped one?

did you mean that higher order component doesn't automatically get the static properties of the wapped one?

Yes.

From functional view it would be better to have "utility object" injected in constructor, which would pass without problems. But if you really want static methods then it's better to create higher order component factory, which can copy the static props to the HoC.  ;-)

Ran into this problem again today. I can't use willTransitionTo in a component that is wrapped by a exposeRouter.

I ran into the same problem @ronag. I need prevent user closes the form while inserting data, like the docs
willTransitionFrom: function (transition, component) {
  if (component.formHasUnsavedData()) {
    if (!confirm('You have unsaved information,'+
                 'are you sure you want to leave this page?')) {
      transition.abort();
    }
  }
}

then when wrap requireAuth using another HoC the willTransitionFrom method never is called
Here there are some workaround for this kind of problem: https://github.com/acdlite/flummox/issues/173
Any suggestion?

willTransitionTo will not be static method with new react-router api anyway, so 1) wait for new api 2) use some temp workaround 3) don't use willTransitionTo. Anyway, I guess you can not prevent accidental leaving through browser back button, so there is no point to use confirm for that. I would just save all data into global app state, then data will be never lost, and you can use onBeforeUnload for protecting accidental app closing. Makes sense?

My work around is to in exposeRouter do:
static willTransitionTo = Component. willTransitionTo

Ugly, but it works until I personally switch to react-stampit.


IHostedService Under Load Balancer?

IHostedService Under Load Balancer
I read below line
"you can even make sure that it runs a single instance of that microservice process/container." but its confusing
I have multiple container and instance of my service under load balancer.
will this background job run in all instance or one among them??
if no how multiple instance inform that background process already started.

Can you point to where you read that line so we can see context? thanks!


Get the yarn cache directory to build de cache from

coverage: 100.0%. remained the same
when pulling 944a3cea6199dcb4e2cc84325d6d862e0822039f on get_cache_yarn_directory
into bfd05970d278463da3fcc10a5e6f7b9e2050f001 on master.


Reland "Measure iOS CPU/GPU percentage (#39439)"

This reverts commit 652be88ecd259407b47d0dbeb243f5fbbef81638.
This fix is in https://github.com/flutter/packages/pull/37

@goderbauer : good catch! I was actually just googling how to do it. Added the version to https://github.com/flutter/flutter/pull/41234/commits/703ea94155a7d7d07417dcb95226bd5b36c5608e :)


keep Webview without Edge Core and Edge Updater

The variant that keep Webview looks like it will also keep Edge Updater, is there a variant that only keep Webview without Edge Core and Edge Updater?

Looks like Edge Update is used by WebView.
Brought WebView removal back - Remove-EdgeWeb.exe


feat(react-native): persist last displayed story

Issue: N/A
What I did
I've implemented this feature from the react-native roadmap:

Easily select initial story: We could use async storage to remember last selected story. It would help users who cannot use Hot Reload to have similiar functionality just by using Live Reload.

The last displayed store will be persisted just in case storybook is not running.
PS: I haven't added a getStorybookUI option to toggle this feature.  Please let me know if it's the case of adding one.
How to test
Is this testable with Jest or Chromatic screenshots? N
Does this need a new example in the kitchen sink apps? N
Does this need an update to the documentation? N
If your answer is yes to any of these, please make sure to include it in your PR.
For maintainers only: Please tag your pull request with at least one of the following:
["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

Hey, I believe this one should use a toggle in getStorybookUI. Maybe not everyone wants to use this behaviour.
Also it would be great to have an option to pass in initial story as an option manually to getStorybookUI.

Also I've just created an issue #4716 . Seems like I broke old behaviour in v4. It is a little bit relevant to this task.

Also it would be great to have an option to pass in initial story as an option manually to getStorybookUI.

@Gongreg I just added a shouldPersistSelection option. Please let me know what you think. 557d6f48254efa62d405a68b83091a1eefbe000f

Hey, I believe this one should use a toggle in getStorybookUI. Maybe not everyone wants to use this behaviour.

I don't think I get it. Isn't the shouldPersistSelection enough in this case?

Hey @elucaswork, I think I would even make this option true by default. This is a cool feature.
What I meant about separate option (like initial story) is so that users could control the initial story themselves out of storybook scope. Now they have to get the channel and emit the initial story. But it can be done separately, it was just an idea :)
In the long run we just have to make sure that there is this priority:
Server, manually setting up, reading from storage.
If user is running server, just allow the server to set the story, if not and user manully passes initial story, set to that story, if not try to read from storage and set it, and if not just open up the first story.
I hope the idea is clear :)
So to finish this PR I think just setting option to true by default should be enough.

@Gongreg thanks. got it now. Nice idea.
I just implemented initialSelection option here, in the same PR. let me know what you think about it.
getStorybookUI({
  initialSelection: { kind: 'Knobs', story: 'with knobs' },
})

Looks great! :)

@elucaswork it might be useful to update console logs to be more explicit about what is wrong.


Remove concept of "full" push

For a while, we had kind.ServiceEntry for Service changes and Endpoint changes. We implicitly distinguish the two based on Full=true or false.
Today, the line between a full push and a partial push is blurred. We have a lot of things doing incremental pushes. Largely this is about reverse engineering whether any types that operate on Full push only need it. We have also this weird mix of endpoints and service updates under kind.ServiceEntry.
I think we could get away with removing the entire Full concept, without any behavioral changes. Basically we would make endpoint updates their own kind.Endpoint, while service changes are kind.Service.

@stevenctl @ramaraochavali @hzxuzhonghu after 1.22 I may try out a prototype to see if this actually makes sense. LMK if you have ideas or objections

From my perspective, we may maintian a resource->xds type map

We do the reverse of it while during full push in every xds generator.
I agree it is a good idea but we need to think through what xds types to push when config changes

To make it more concrete ill put up a small prototype. I feel like i cannot tell if its a good idea unless I try it :slightly_smiling_face: . But after 1.22 probably, a bit busy until then

Not stale. I have a prototype somewhere, need to dig it up..

Early prototype was/is in https://github.com/howardjohn/istio/tree/exp/drop-full-push


check child visibility for cases

If all the children of a case statement have the same
visibility then the case should inherit that value. For
example a case that has only one leaf in it that's marked
"obsolete" should mark the case itself as obsolete.

Looks at those unit tests, a beautiful sight for sore eyes

I presume you know how to publish a new version after this goes in?

uhh I could take a guess or two, but no, I can't say I know how to cut a new release

uhh I could take a guess or two, but no, I can't say I know how to cut a new release

Basically just bump the versions like this:
https://github.com/128technology/yinz-json/commit/44b92c298c75264f74eac6b6279d99cb35f04284
In a PR, I'll approve. Then just create a new release via Github and when you do it make a new tag on master as part of the release and use a tag name following the other ones we have. CI will handle the rest and publish it.


Load custom resources at startup to improve autocompletion

This is to allow auto-completion for custom resources, so instead of hardcoding resource types they are loaded at startup.
The drawbacks is that it slows the startup a bit.

Closing this, as the project seems to be dead

Closing this, as the project seems to be dead


Speed up inclusion of abbreviations on every page

The including of those pages through auto_appending with the snippets extension slows don the build by a factor of 3.

Abbreviations are now disabled with  ./gradlew serverFast


CompileMacro Fix

Fixed CompileMacro problem when used with 'val i: Int = null'.

This is the fix for the following issue:
https://github.com/scalatest/scalatest/issues/855


libcxxStdenv broken on linux

I fixed this issue in 172ed10426df966ee5df76a1ef18f683698fe23f (see also #8546 and #8549). @pikajude reverted this commit without discussion in b62baacbc177cf799d88c2b519015a13ef91891c. I assume this was due to an unspecified error he alluded to at https://github.com/pikajude/nixpkgs/commit/172ed10426df966ee5df76a1ef18f683698fe23f#commitcomment-11921957. As can be seen there, I tried to get more information so his issue could be correctly fixed, but now we're back where we started.
I use libcxxStdenv for some of my projects, and need it to be functional on linux. I had assumed the merge was good until I pulled nixpkgs master today, seeking to update my development tools, and found the revert. I don't understand why this commit was reverted with no meaningful discussion. At the very least, I'd like to hear the details of what, if any, issues actually exist with the fix on OSX, so that I can attempt to fix them. As discussed in the above links, the previous--and now once again current--behavior of libcxxStdenv was clearly wrong, and should not remain on any platform.

I got in touch with @pikajude on IRC, and we resolved the OSX breakage and restored the changes to staging as of fa6c8445c768a64c908fcd4d2955843622ba2550.

Good!


Typo on wiki path.

The wiki regarding creating a localhost override has a typo in the file path.
It should be "config/initializers/geocoder.rb" not "config/initilizers/geocoder.rb".
https://github.com/alexreisner/geocoder/wiki/Provide-a-hardcoded-IP-to-`development`-and-`test`-environments

Thanks--fixed!


Gradle version update and 1.20.6 support

There was a name change in NMS packages in version 1.20.6.  That's why I updated it to the system recommended by the Paper community. It became working for 1.20.6. It also supports older versions.

I also updated the gradle version to 8.8

Does Paper's update include legacy versions? I'll check this jar manually before merge. We have to ensure all versions from 1.8 to 1.20.6 are supported and work well still.
Thanks for it.

@hamza-cskn I also added a method in PaginationManager that clears all items. I tested, it works on older versions.


I sent an order and the balance doesn't match?!

I sent the following order against a balance of USD $100.00
{
  "client_oid":"17215070-e815-40b0-a6ea-680f48e61303",
  "time_in_force":"GTC",
  "type":"limit",
  "price":"70.00",
  "post_only":true,
  "side":"buy",
  "product_id":"BTC-USD",
  "stp":"dc",
  "size":"0.01000000"
}

The order shows up as expected when I go see the website interface, however, my balance was updated and now looks like: $99.30 when I expected $30.

The returned JSON seems to look just fine:
{
  "fill_fees": "0.0000000000000000",
  "id": "08d3b9bd-2f2f-4406-8f9a-dd7917272278",
  "size": "0.01000000",
  "status": "pending",
  "filled_size": "0.00000000",
  "post_only": true,
  "side": "buy",
  "executed_value": "0.0000000000000000",
  "price": "70.00000000",
  "product_id": "BTC-USD",
  "stp": "dc",
  "created_at": "2018-03-31T22:12:59.337479Z",
  "type": "limit",
  "settled": false,
  "time_in_force": "GTC"
}

It could be that this has something to do with this issue:
https://github.com/coinbase/gdax-tt/issues/199

Hey @AlexisWilke, as this issue doesn't relate to the PHP library itself and is about API behavior, the best way to find an answer or report a bug is by opening a support ticket so that someone can provide you with an authoritative answer.
The API this library integrates with is for Coinbase, which is a separate product, hence why we strongly recommend you find your answer from GDAX support. Thanks!


Doesn't seem to do anything...

I run the app, I get the gui and enter a model hash and select a folder to DL to, but after I press the download button nothing happens. Nothing in the console, nothing downloaded. On Windows 10, of course ran git clone and then ran pip install -r requirements.txt.

Hey there, sorry I haven't been active on here as frequently as I should be. I'm finishing up my final semester of college and it's kept me busy. Did you figure this out or is it still happening?

I didn't figure it out actually. But ended up using other image grabbers and then altering the code with GPT4 to meet my needs. But thanks for responding!


Request: Add gr-m2k

The ADALM2000 is a similar to the PLUTO, but needs a different Gnuradio module.
Could you please integrate gr-m2k https://wiki.analog.com/university/tools/m2k/libm2k/gr-m2k?
It's probably trivial to include, but I am not familiar with the Conda gnuradio-feedstock setup.

Thanks for the report. Without having tried to build it or make a recipe, I see two potential blockers for making a gr-m2k conda-forge package (but that's not saying it won't happen):

Currently, it only appears to be compatible with GNU Radio 3.8. I'd probably want compatibility with 3.10 to keep up with the rest of the ecosystem, although it's not a blocker.
Conda-forge requires that packages have a versioned release, and so far that is not true for gr-m2k. The only way around that is to make a release, or host the package on a personal anaconda.org channel instead of through conda-forge. (This is a problem for many GNU Radio OOT modules, for what it's worth.)

If you're motivated to speed this up, put in a kind word with the developers of gr-m2k to see if they would be willing to add support for GR 3.10 and make a release. Thanks!

Thanks Ryan, I opened an issue with gr-m2k (https://github.com/analogdevicesinc/gr-m2k/issues/22).
But activities are quite low over there. Let's see...

Hi,
We have created a release tag for the first official release version of gr-m2k. This one is GNU Radio 3.10 compatible.
The maint-3.8 branch is 3.8 compatible.
Is this what was needed in order to start the integration of gr-m2k?
Thank you!
-Alexandra

PR to add gr-m2k to conda-forge: https://github.com/conda-forge/staged-recipes/pull/21170

Now available at https://github.com/conda-forge/gnuradio-m2k-feedstock.


加入 Keynote 的 og 資訊

(issue #5)

Keynote og 分享圖
Keynote 中、英文 og 標題與描述文字

Current coverage is 94.20%

Merging #119 into master will not affect coverage as of 1e74f9a

@@            master   #119   diff @@
=====================================
  Files           31     31
  Stmts         1000   1000
  Branches         0      0
  Methods          0      0
=====================================
  Hit            942    942
  Partial          0      0
  Missed          58     58

Review entire Coverage Diff as of 1e74f9a

Powered by Codecov. Updated on successful CI builds.


Translation into other languages: Portuguese - Brazil

The first module of the game was translated. 😃
(Hacktoberfest, pls)

Hi @Kouthk, this is on WIP? Because some of the room is not translate to portuguese

@ming-tsai, thanks!. =)
I will translate the remaining "Modules". could you put a hacktoberfest-accepted tag?

@Kouthk,
Please configure your GitHub with the email that is committing or the other way around


vv-as for rules like 'after' or 'before'

Versions:

VueJs: 2.5.2
Vee-Validate: 2.0.0

Description:
When you set the rule after or before, the error message display the other field name even if it has a vv-as attribute defined.
Example:
Given a field having "start-date" name and "Start date" vv-as attribute
And another one having "end-date" name and "End date" vv-as attribute
When you set the rule "after:start-date" to end-date field, the message is 👍
"End date must be after start-date"
Instead of
"End date must be after Start date"

I have it failing for me in this demo:
https://jsfiddle.net/to014ycs/5/
…but I can't really see what's different, apart from the vee-validate versions (2.1.1 vs 2.0.0) and vue (2.5.17 vs 2.5.13).
I've also noticed the error doesn't always clear once the end date has been changed to something valid — and it doesn't appear at all if the start date field is edited to be after the end date. Is there a way to have an edit to the start date trigger the end date's validation?


kubeless deployment security context and practice

kubeless starts deployment that runs as root, which arguable may not be needed especially because it only binds to unprivileged port.   Even if it needs to bind to privileged port it may not need to run as root if the underlying kernel is configured to support binding to privileged port by non-root user.
I suggest the following for considerations:
The deployment should create container that does not run as root (runAsNonRoot: true) and readOnlyRootFilesystem: true to make it truly stateless, or at least allow users to specify that.

Are you running on openshift :)

No we are not.
We run vanilla Kubernetes with reasonable policy and restriction using RBAC/PSP/Initializer/admission controllers.   We have a multi-tenant environments so these practices are important for us to get right early on.   :)
Happy to chat more if you are interested in our environment.

@phsiao yes would love to know more. You can reach me at<EMAIL_ADDRESS>

I think this was made possible with https://github.com/kubeless/kubeless/pull/217, it just isn't exposed in the CLI.  So .. if you're comfortable creating API objects from kubectl create -f $yaml (or similar), then you can construct all sorts of podspec templates right now, including this one.
Having said that, I sympathise with the original proposal - theoretically we should be able to enable these options by default given the conceptually "stateless" nature of the functions we're exposing.  Unfortunately the current architecture relies on runtimes being able to do some initialisation at container-startup, potentially including (eg) installing packages into the container rootfs, as container-root.  Thus at the moment, this is in direct conflict with that sort of flexibility.  We could enable these options conditionally for runtimes that we know don't need this (most of them), but again this is in conflict with trying to minimise the embedded "knowledge" about runtimes.
So:

I think this needs to be conditional, and disabled by default, if we want it anytime soon - and rely on human knowledge as to whether it is appropriate to enable it for this runtime/language/code combination.  - In that case, do we want to expose that via yet another command line option (the command line UI is already overwhelming), or leave it up to "power users" who are comfortable working with YAML and the raw Function CRD?

I can go either way on that last question.  @phsiao Do you need kubeless cli support for this, or are you going to be working with the API objects directly anway (perhaps via serverless framework or some other wrapper)?

I have not looked very closely yet, but it appears to me that kubeless is leveraging initContainers to do some initialization work such as installing runtime's packages so we can separate the securityContext for the initContainers and the regular runtime.
I think sometimes we need to modify the rootfs to make things flexible, such in the #364 so additional commands can be installed, but maybe it is time to make it a design goal that the runtime container has to be immutable and run as non-root?   The bash use case I think can still be done via initContainers, just the command needs to be installed in specific locations in a shared mount that the runtime can pick up the commands in its path.   It is already making assumptions that it is a debian variant, so it is already not without restriction.
Separately, on the topic of pod template, I think it might be interesting to carry the customizable portion of the pod template (and other template that is not varying from function to function) in ConfigMap and reduce the needs to keep adding more flags to CLI.

maybe it is time to make it a design goal that the runtime container has to be immutable and run as non-root?

Agreed, we should decide if this is a thing we want, and follow-through on that decision.  Such a decision might push us strongly towards a more significant "build" step, for example (although I imagine your original security concern would also impose other restrictions like "no in-cluster docker build daemon")

I think it might be interesting to carry the customizable portion of the pod template in ConfigMap

Have you looked at the actual Function k8s API object (see https://github.com/kubeless/kubeless/blob/43ceb591696a68c746b807578a77ca5d7b1d8870/pkg/spec/spec.go#L33)?  As mentioned above, you can already provide a mostly-arbitrary pod template.

Right, was looking around the various runtime docker images as well as how customization such as ports to listen on for http-trigger and kafka topic to subscribe to for event-trigger, I become clear to me the interface must also be clearly defined for kubeless to be successful.   For example, one of the use case I have is to provide a simple way to build mashups API for internal APIs, but to do that I need to bring in significant infrastructure bits and pieces, and to rebuild that every time is going to be very inefficient and building a custom runtime that kubeless can consume is more scalable.   To do that today is not very easy, as you need to update the code to do that.
I'd be happy to participate in drafting of design documents for these three topics:

What does a runtime image looks like, and what interface it needs to provides and implements.   For example, a python http-trigger image needs to bind to port specified in the environment, implements a health/readiness check, and so on.
How does a runtime initializes before it can become ready, via initContainers or other mechanism.
How does a runtime behave once it is up and running.  This is where the securityContext, readonly filesystem (or state-keeping in general) and process privilege comes in.

Regarding the pod template, I looked at it but did not quite get what you were saying, but I think I do now.   Yes, I can modify the function spec at runtime, but if I have to do that for every function for same/similar things than it is not very efficient.   I think we can introduce a concept of global default pod template, and namespace default pod template, that the controller will import when building function for the namespace.
Things that I would love to be able to customize, and why, include, but not limited:

securityContxt (both pod and container level so it can match our podsecuritypolicy requirements and limits)
nodeSelect, tolerations, affinity and anti-affinity (runtime that may need special resources such as GPU, and move runtime to where there dependencies are, or allocate runtime on unique nodes)
dnsPolicy (we encourage people to use Default, because our environment's resolvers know cluster domain)
imagePullSecrets (private docker registry ..)

@phsiao would love your help with https://github.com/kubeless/kubeless/pull/396
that's where we are going to work on a better design for the function interface.

It is now possible to specify a Deployment template in the Kubeless configuration so every pod uses that by default. See the documentation: http://kubeless.io/docs/function-controller-configuration/


Add equatable_if_let

What it does
Checks for pattern matchings that can be expressed using equality.
Why is this bad?
It reads better and has less cognitive load because equality won’t cause binding.
It is a Yoda condition. Yoda conditions are widely criticized for increasing the cognitive load of reading the code.
Equality is a simple bool expression and can be merged with && and || and reuse if blocks
Example
if let Option::Some(2) = x {
   do_thing();
}

Use instead:
if x == Option::Some(2) {
   do_thing();
}

Hi @0xLucqs, can i try this?

sure can you join the telegram group as well so i can help if you're stuck or don't know how to start ? https://t.me/cairolint

Thanks @0xLucqs


Don't translate the GUI by google

From what i feel , you want multi-language UI in the new client beta.hack.chat and decide to not hire a real translator ,
if you want. i can translate the text in chinese , if you can gave me a sheet to match

The new client source has been released, you can find the Chinese translations here


[now-cli] Implement continue: true with dest

Closes #2685.

Codecov Report

Merging #2908 into canary will decrease coverage by 2.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           canary    #2908      +/-   ##
==========================================
- Coverage   13.17%   11.08%   -2.09%
==========================================
  Files         266      265       -1
  Lines       10394    10075     -319
  Branches     1170     1257      +87
==========================================
- Hits         1369     1117     -252
+ Misses       8964     8868      -96
- Partials       61       90      +29

Impacted Files
Coverage Δ

src/util/output/create-output.ts
31.25% <0%> (-12.34%)
:arrow_down:

src/util/prefer-v2-deployment.ts
56.25% <0%> (-5.66%)
:arrow_down:

src/util/dev/server.ts
61.39% <0%> (-5.51%)
:arrow_down:

src/util/dev/yarn-installer.ts
85.71% <0%> (-5.03%)
:arrow_down:

src/util/dev/builder.ts
73.07% <0%> (-4.35%)
:arrow_down:

src/util/get-files.ts
87.77% <0%> (-3.89%)
:arrow_down:

src/util/dev/router.ts
78.57% <0%> (-3.79%)
:arrow_down:

src/util/projects/get-project-by-id-or-name.ts
25% <0%> (-3.58%)
:arrow_down:

src/util/dev/builder-cache.ts
77.44% <0%> (-3.43%)
:arrow_down:

src/commands/certs/ls.js
0% <0%> (ø)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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


Docs: add option to visual examples that allows generating GIF images

It would be nice to have animated visual examples for some interactive widgets (buttons, dropdowns etc.)
It should be easy to implement in our of-screen documentation renderer because Pillow allows for simple and easy creation of GIFs (see: https://pillow.readthedocs.io/en/latest/handbook/image-file-formats.html#gif).

Another useful link: https://gist.github.com/xyfeng/559e56401dee3eccfa23

It may be worth considering post-processing any gifs with gifsicle to reduce the size if it's available.


Beer Factory trading routes are affected multiple times by multipliers

Game speed on x0.5 results in the correct trade amount shown in the management modal (the trading amount includes the game speed):

But the real traded amount is halved again to a speed of x0.25:

Fixed in version 1.69.0


#47 - removeImage Mojo added

@krystiannowak Nice. We should also mention how to use this with an example in the README's Usage section.

@davidxia both usage modified and put some additional comments on params around client call

@davidxia - do you think it's acceptable for now to merge and release it in a new version?

@krystiannowak great thanks. Can you also take a look at my comment here https://github.com/spotify/docker-maven-plugin/pull/62#discussion_r26771743 and squash everything into 1 commit?

@davidxia now it's all in one commit

@davidxia thanks! now can't wait for 0.2.5 :-)


Update startup.ps1

Updates startup.ps1 examples, since they require $Env qualifier

@GregoryGoldshteyn : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Where are the  ChatVectorDBChain docs?

I can't find the ChatVectorDBChain  doc, the link in the readme file appeared to be a 404 error

@CreationLee According to blog.langchain.dev, ChatVectorDBChain is now ConversationalRetrievalChain

@CreationLee According to blog.langchain.dev, ChatVectorDBChain is now ConversationalRetrievalChain

thanks


Improve diffing of Jest Matchers

Love the library. It is much more ergonomic compared to the mocking I was doing by hand!
Jest can show a diff between the received and actual calls when I use .toHaveBeenCalledWith(...) including when there are multiple non-matches.

Diff Examples in a Details so this ticket doesn't look super long
describe('diff demo', () => {
    it('diffs the real call and expected', () => {
        const mockFn = jest.fn();
        mockFn({ RoleArn: 'abc', RoleSessionName: 'def' });
        expect(mockFn).toHaveBeenCalledWith({
            SomeField: 'xyz',
            RoleArn: 'ABC',
            RoleSessionName: 'def',
        });
    });
});

expect(jest.fn()).toHaveBeenCalledWith(...expected)

- Expected
+ Received

  Object {
-   "RoleArn": "ABC",
+   "RoleArn": "abc",
    "RoleSessionName": "def",
-   "SomeField": "xyz",
  },

Number of calls: 1

it('diffs the real call and expected with multiple calls', () => {
    const mockFn = jest.fn();
    mockFn({ RoleArn: 'abc', RoleSessionName: 'def' });
    mockFn({ SomeField: 'xyz', RoleSessionName: 'def' });
    expect(mockFn).toHaveBeenCalledWith({
        SomeField: 'xyz',
        RoleArn: 'ABC',
        RoleSessionName: 'def',
    });
});

expect(jest.fn()).toHaveBeenCalledWith(...expected)

Expected: {"RoleArn": "ABC", "RoleSessionName": "def", "SomeField": "xyz"}
Received
       1
          Object {
        -   "RoleArn": "ABC",
        +   "RoleArn": "abc",
            "RoleSessionName": "def",
        -   "SomeField": "xyz",
          },
       2
          Object {
        -   "RoleArn": "ABC",
            "RoleSessionName": "def",
            "SomeField": "xyz",
          },

Number of calls: 2

Whereas when I use aws-sdk-client-mock-jest matcher .toHaveReceivedCommandWith
it("Doesn't diff the real call and expected", async () => {
    const mockedClient = new STSClient();
    await mockedClient.send(
        new AssumeRoleCommand({
            RoleArn: 'abc',
            RoleSessionName: 'def',
        }),
    );

    expect(stsMock).toHaveReceivedCommandWith(AssumeRoleCommand, {
        RoleArn: 'ABC',
        RoleSessionName: 'def',
        SourceIdentity: 'xyz',
    });
});

Expected STSClient to receive "AssumeRoleCommand" with {"RoleArn": "ABC", "RoleSessionName": "def", "SourceIdentity": "xyz"}
STSClient received matching "AssumeRoleCommand" 0 times

Calls:
  1. AssumeRoleCommand: {"RoleArn": "abc", "RoleSessionName": "def"}

particularly when the difference between expected and actual is subtle (the example that prompted this feature request was an arn with a : v.s. :: it would be nice to see the diff output (which I naively assume Jest can do for us? 🤞)
If this is a feature you'd support I'd happily contribute some time to making it happen.

The main problem here is that toHaveReceivedCommandWith() does not simply compare 2 objects but compares the expected object with all received values, of which there can be 0, 1, or multiple. So if we received 2 calls and none of them match, we don't know to which we should compare in a "nice" output.
However:

In most cases, like your example, we receive only a single call, so we could show the nice diff then (because 99% it is the call we expected and want to compare to), otherwise defaulting to the current behavior (listing all received calls).
There is toHaveReceivedNthCommandWith() which compares expected value and single received value, so we can show the nice diff here as well.

This sounds like a nice improvement. If you are willing to do so, please create a PR.


Stroke with antialiasing has black edges

Hi, Ion! Firstly, thanx for extension - it's very nice!

I have some problem when create line, based on Stroke with 0-1-0 alpha texture for aa. Here is a code:
public class PathTest extends Sprite
{
[Embed( source = "/assets/strokeTexture.png" )]
private var WhiteBMP		:Class;
private var stroke			:Stroke;
private var whiteBMP		:Bitmap;
private var whiteTexture	:Texture;
 public function PathTest()
 {
 	addEventListener(Event.ADDED_TO_STAGE, onAdded);
 }

 private function onAdded ( e:Event ):void
 {
 	stroke = new Stroke();
 	addChild(stroke);
 	stroke.x = 20;
 	stroke.y = 20;

 	whiteBMP = new WhiteBMP();
 	whiteTexture = Texture.fromBitmap( whiteBMP, false );

 	runExample();
 }

 private function runExample():void
 {
 	stroke.material = new TextureMaterial(whiteTexture, 0xFFF440);

 	var l_x:Number = 0;
 	var l_y:Number = 0;
 	for (var i:int = 0; i < 100; i++)
 	{
 		l_x += Math.random() * 10;
 		l_y += Math.random() * 10;
 		stroke.lineTo(l_x, l_y, 16);
 	}
 }

}

The problem is that there are black edges around the line. Is it possible to make its transparent or may be set the color of that (same as the background color)?
Look what i mean:

here is a texture:

and project src:
https://www.dropbox.com/s/om7v84i2iibpp53/pathTest.zip?dl=0
.. and another small question. May be it's possible to update stroke thickness without clear and redraw it by lineTo method? I need to animate at each frame long polyline thickness (by zoom-in/out a route in my geo-map application). Now a use that:
public function update():void
    {
		clear();
		for (var i:int = 0; i < _basePoints.length; i+=3)
		{
			lineTo(_basePoints[i], _basePoints[i+1], _thickness);
		}
	}

Is there fastest way to do that job?
Thanks!

Looks like a problem related to pre-multiplied alpha to me
http://www.quasimondo.com/archives/000665.php
If you stored your alpha in a seperate bitmap, then used a bit of BitmapData.copyChannel, you should be able to construct an un pre-multiplied version of your bitmap.
Or if you only needed greyscale, you could save yourself another bitmap and just pack the alpha into the green channel.

I will answer the second issue first:
If you use ShapeEx instead of Shape, the shapeEx.graphics - API opens up the ability to post process Line properties, like Thickness. However, it isn't a whole lot faster, since the underlying geometry has to be recreated anyway.
If you look at the "postProcess" method on GraphicsEx, you can supply a startIndex and an endIndex, and a GraphicsExThicknessData variable that holds the thickness you wish to apply along the line.
This allows you to say that the line should be 10 thick in the beginning, 20 thick in the middle, etc.
But the performance gains will be slight, unfortunately. It does give you the ability to make more fun lines, however. You can check Example 08 in the Extension, the NaturalSplineExample , it will show you how to use the postProcess method.
For the first question, unfortunately I dont know how to solve that. I think it has something to do with premultiplied alpha and the shaders that the Graphics extension use by default, but I dont know what exactly. It should be possible to fix, but I have so very little time on my hands.
I will look into it as well, but please have a look yourself too, if you want, in the StandardMaterial, the StandardVertexShader and the TextureVertexColorFragmentShader classes

Yes, it is the StandardVertexShader and the TextureVertexColorFragmentShader classes that handles premultiplied alpha wrong.

Ion, Thank you so much for your answer! Thank you for try to help solve the problem quickly!
But "shader"... ohhmg! It's so difficult for me... %( Do you have any advice for me, that what I need to look specifically? "Premultiplied Alpha", it's a magic words ) Well, thanks anyway, I understand that the solution to this problem requires a lot of time from you. No problem, thanks!

Fixed by d59dad98088b5709b1ff63eec4a4a6744953e482.
This fix doesn't touch that shaders (un-premultiplying in shaders would cause precision artifacts) - Starling already supports correctly rendering shader output with premultiplied alphas via hardware blend modes - so I've just exposed the flag for that from the materials.
IMaterial's have a new 'premultipliedAlpha' field that's set to 'true' by default on TextureMaterial. This assumes most people will be supplying image data via Flash's BitmapData class (which premultiplies) - users supplying texture data via their own loading mechanisms (which don't premultiply) will need to set premultiplyAlpha=false on their material to get correct blending.

This is most excellent, and very very good news. Great fix, @jonathanrpace

@jonathanrpace, Thank you so much!!!! Your help is invaluable!!!!


[Bug]: When defining dev.assetPrefix to a full URL (http://localhost:3010/subpath/) static files can't be found under said subpath

Version
System:
    OS: macOS 14.5
    CPU: (10) arm64 Apple M1 Pro
    Memory: 7.46 GB / 32.00 GB
    Shell: 6.21.00 - /bin/tcsh
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 126.0.6478.127
    Edge: 126.0.2592.81
    Safari: 17.5

Details
I have a rather complicated setup with multiple module federation apps running in parallel. We have start script where you select which apps to start locally and which to proxy in from a deployment (very handy when working with MF). So much for the introduction.
Now because of said setup, we need to define the dev.assetPrefix path to a full URL (like i.e. http://localhost:3010/subpath/). Because if we only set an absolute URL, when trying to fetch async packages from one of the MF modules, it wouldn't know in which app to search (it would search it in the host app, where it's most probably not).
I noticed that when I set an absolute URL, it will make sure that the static files can be accessed through this absolute URL. Not so much if the assetPrefix is a full URL. The script tag points to the right place, but the JS files will still be located outside the subpath.
If there's another way to achieve this, then I'm happy, but currently, I'm really struggling to get this usecase (different subpath isntead of every app on root) to work.
Reproduce link
https://github.com/nate-summercook/rsbuild-asset-prefix
Reproduce Steps
I guess this could be reproduced without a module federation setup, it's probably a general limitation of rsbuild, that static files won't be accessible on a subpath if the dev.assetPath is a full URL. It's probably also a desired behavior, but it would be nice to have a way to deal with this. Searched the documentation up and down but didn't find anything.

Checkout my repro repository
run npm i
run npm start
If you change assetPrefix to /subpath/, it will work, but that doesn't help me in my module federation situation.

@9aoy can you help to check this?

I think this is a scenario that rsbuild misses, let me figure out how to support this.


Improve docs.rs documentation

Part of gtk-rs/gir#820.
cc @EPashkin @sdroege

👍, CI agree with me

Then I merge! :)


[Documentation] document Hadoop/Spark support matrix on lakeFSFileSystem

lakeFS Hadoop Filesystem
Our direct-access Hadoop Filesystem implementation currently depends on some hadoop-* libraries in version 2.7.7.
This was done with the decision to support (only) Spark version 2.4.7 and 3.0.1 which both depend on Hadoop 2.7.7.
Metadata client
Metadata client is compiled and published twice, once for Spark 2.4.7 and one for Spark 3.0.1.
The problem
Times are changing and these Spark versions are getting old.
Research how we can support more versions.
The user needs to be able to understand easily which version to use and how to get this version.
Related issues
#2336 - describes the need to support Hadoop 3.1 specifically.
#2731 - caused due to a missing feature in hadoop 2.7.7.
#2597 - task to test flows based on the metadata client (export/GC) in Spark 3.1 on EMR.

duplicate of #2800


The revocation function was unabel to check revocation

Hi
I am on Win 11, using 2.1.14 I am trying to install PhotoMaker2 and hitting this issue. I have no idea how to fix it. It seems like a git issue. I have Git Windows install and that version does not have this type of thing. The Pinokio version has an issue and I do not know how to fix it.

Thanks

Try to delete the photomaker2 folder and try it again.

Try to delete the photomaker2 folder and try it again.

Thanks but I already did that unfortunately, unless there is a new fix?

I'm getting this same issue with every project I try to install. I've tried resetting the cache and resetting the bin folder. All the prerequisites install perfectly (conda, git, etc) but when it comes to the actual project, they all throw the error above, so I've not been able to use Pinokio at all.


Assets not visible for "Child

Affected module
Does it impact the UI, backend or Ingestion Framework?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Screenshots or steps to reproduce
Expected behavior
A clear and concise description of what you expected to happen.
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: [e.g. 0.8]
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

duplicate. Created a new issue by mistake.


Dev env[merging the DEV to QA]

There is no any secret in application properties files. It has been already reverted.


Follow fails

💡 Summary
Attempting to follow someone fails.
🙂 Expected Behavior
Search for someone
See their profile
Click the Follow button
Follow succeeds
☹ Actual Behavior
Search for someone
See their profile
Click on the Follow button
Follow button changes to Processing and never completes
📝 Steps to Reproduce

New installation of Misskey on Ubuntu 20.04 Intel NUC.
Following another user fails

📌 Environment
Ubuntu 20.04
Intel NUC i3
Misskey v12.61.0
nodejs version 14.15.1
PostgreSQL v12.5
Elasticsearch v7.10.0
Lucene v8.7.0
My domain: https://fedi.place
Apache/2.4.29 front-end config: https://pastebin.ubuntu.com/p/x6S7bZsY4H/
Misskey config: https://pastebin.ubuntu.com/p/kh8VQmTyhk/

Added ProxyPreserveHost on to the Apache front-end configuration. Follow works now. Thanks to<EMAIL_ADDRESS>for the tip.


added fci bsu

Website: https://fci.bsu.edu.eg

@MuhammadMagdy98 lib/domains/eg/edu/fci.bsu.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@philipto So changing the file name to bsu.fci.txt will resolve this?

@MuhammadMagdy98 I am sorry, the domain fci.bsu.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

@philipto I didn't find it in abused.txt maybe it's confused with bu instead of bsu?


Fix Docker Build by Ensuring Cache Directories Exist

With a fresh build, errors occur which show the following:
Check for working C compiler: $BUILD_PREFIX/bin/x86_64-conda-linux-gnu-gcc - broken
...
ccache: error: Failed to write to /workspace/.cache/ccache/ccache.log: No such file or directory

This PR fixes that by ensuring the cache directories exist

Can one of the admins verify this patch?

Closes #7


[button] Missing declaration file when using path import

When trying to import a component with import path, typescript shows an error about missing declaration file.
Note: There is no runtime error
Button

ButtonIcon

Checking the dist folder, declaration files are generated in a sub-folder:

Declaration files should be in the same folder of its js files.


MP4 file missing in example

In the example https://github.com/PyAV-Org/PyAV/blob/main/examples/basics/save_keyframes.py the file
'pexels/time-lapse-video-of-night-sky-857195.mp4' is missing from the package
At least, say where one can find it. Or use "yourfile.mp4" as filename  to indicate that any MP4 file will do.
Common sense, man, Common sense. And a little less programming carelessness.

That file is hosted online. The function will automatically get it.


dependency on multiple days of task

I'd like to setup a weekly hive task that depends on 7 days worth of hive data from another task.  In looking at setting up upstream dependencies - it looks like:

You can set up same day dependencies on a task via the t2.set_upstream(t1), but you can't specify a date here.
You can set up a HivePartitionSensor, and specify a certain relative date - but maybe you have to setup one of these for each date you want to wait for.

Something that would be nice:
HivePartitionRangeSensor(
table=my_schema.my_table,
date_start='{{macros.ds_add(ds, -7)}}',
date_end='{{ds}}'
)
Also, another option that would be nice would be waiting for jobs with a different date to finish.  So something like:
t2.set_upstream_range(t1,'{{macros.ds_add(ds, -7)}}','{{ds}}')

Hey Charlie,
For 2) one solution would be to create them programmatically(see below), but I agree that there is room for a new operator.
day_span = 7
for i in range(day_span):
    s = HivePartitionSensor(
    task_id="wf_myschema_mytable_" + i + "_days_before",
    table="my_schema.my_table",
    partition="ds='{{ macros.ds_add(ds, -"+ i +") }}'",
    dag=dag
    )
    t2.set_upstream(s)

For your second remark, if you toggle depends_on_past for your DAG, then the task for ds will only run if the task ran successfully for the previous execution date. This way, and provided you boostrapped your   jobs by running the first 7 days manually, the constraint would work going forward.
Though I will let @mistercrunch comment more if he'd like.
Best,
Arthur

thanks @artwr - your programmatic solution definitely works.
And that's a fanstastic point about setting depends_on_past being true - that could simplify things a lot for my dependency management.  but if we set depends_on_past - we can't backfill all sorts of things simultaneously.   maybe i will use depends on past= false for the backfill of earlier jobs, and then depends_on_past going forward.
Much appreciated!
Charlie

@cgelman that sounds like the right approach for now. I wouldn't oppose a feature that would add partition ranges to HivePartitionSensor, though the two options above are good enough that it might not be super high value.


pthread_getattr_np is not present on illumos

In 9f6a7ceee819e847eb4e0218c4452ac1d0cffad4, the symbol for pthread_getattr_np was added to the file common to both Solaris and illumos.  This symbol does not yet exist on illumos and causes the libc-test suite to fail there.  It should be moved into the Solaris-specific file where its visibility is limited.
With this change, the libc-test suite passes once again on an illumos machine.

r? @JohnTitor
(rust_highfive has picked a reviewer for you, use r? to override)


Recommended Extensions for Skuba Projects

We can create a default .vscode folder with an extensions.json file:
{
    "recommendations": [
        "esbenp.prettier-vscode",
        "dbaeumer.vscode-eslint"
    ]
}

to recommend plugins we suggest. It comes up with a prompt when you open the project which I thought is kinda neat.

example with more recommendations https://github.com/SEEK-Jobs/noti-experience-resources/blob/master/.vscode/settings.json#L29


fix typo RUBUCOP --> RUBOCOP for slim lint

this actually makes 6ebd8f355c974cb6b7c5d5aff20603c8c4b38feb work
... looks through all that refactoring a typo slipped in, sorry!

Cheers! :beers:


[test] Transpile more similar to prod bundle

Generally we want to test as similar as possible to production environments to make debugging easier. I moved some babel plugins into the default plugins to make testing more similar to prod. I don't think we've ever had a problem but let's be safe.
date-mui-test will ultimately be removed as well. Though it's used fairly often in pickers so I want to take a gradual approach or rather do it while refactoring the pickers.

Did you look into the cost of the change? How much slower will the dev env will be?

That's a non-goal of this change. We shouldn't ever dissect the plugins based on performance. If this is/was/will be a problem we need to set up test infra for dev and prod.

I have tried to rerun the test but it fail, it doesn't seem to be a flaky one.

I have tried to rerun the test but it keeps failing, it doesn't seem to be a flaky one.

Yeah, already investigating.

I have tried to rerun the test but it keeps failing, it doesn't seem to be a flaky one.

Test failure was expected but not anticipated. Fixed with b99da0e (#25406)


Why the lack of updates?

Hello, I am curious as to why there have been no updates for this project in over a month; is the developer still actively working on it or is there some alternative out there?
Thank you. :)

@Ainaemaet Hi, I'll try to update as long as I make progress in improving quality.


Add more watchers in deploy command

watch for jobs that are not hooks
watch for pods, daemonsets, statefulsets

A support of https://github.com/flant/kubedog/pull/33

https://github.com/flant/dapp/issues/1050


Prepare jupyter book for discussion

See VACILT/trend_paper/#4 or my book here.

The Github page is ready on this address: https://vacilt.github.io/SSW_project/. So far it is set to be built from the /docs folder in the master branch. We can manually put the content there as stated here.

I would like to add hide-input tag to all notebooks. It will enhance readability. You can adapt the following script:
import os.path
import nbformat as nbf
from glob import glob

notebooks = glob(os.path.join(path, "*.ipynb"))

for ipath in notebooks:
    ntbk = nbf.read(ipath, nbf.NO_CONVERT)

    for cell in ntbk.cells:
        cell_tags = cell.get("metadata", {}).get("tags", [])
        if len(cell_tags) == 0:
            cell_tags.append("hide-input")
        cell["metadata"]["tags"] = cell_tags

    nbf.write(ntbk, os.path.join(path, os.path.splitext(os.path.basename(ipath))[0]+"_clean.ipynb"))


Provide a preview feature for tox.ini files

Addressed Changed
According to issue #19:

Show previews via hover messages for environment variables in tox.ini files for passenv and setenv variables.
Variables could be assigned directly in a section (environment) or in the inherited section testenv which can be reused by all other sections (environments).
passenv variables are read from the current environment.
setenv variables are assigned either directly inside the tox.ini file or indirectly referenced from an external .env file.

Changes

Added class src/environment_variables_service.ts which contains the logic to provide hover messages with preview data for tox environment variables.
Added unit tests and sample files (in src/test/examples/envvars/) to test new features for previewing environment variables in tox.ini files.
Added register command for VS Code hover integration to src/extension.ts.
Added activation event in package.json for extended language support in ini files.

How to test

run unit tests via npm test from terminal or run Extension Tests in VS Code
run extension, open a tox.ini file, and hover with the mouse over an environment variable. You will see a preview of the value of the variable.

Thanks! While I'm not too happy about parsing the ini file with regex, I suppose I can't think of a much better solution either - I thought about using a library, but I suppose that might just introduce new issues with tox' particular flavor of ini and perhaps also the "partial" parsing based on the cursor position and such.
It looks like the tests failed because they expected an empty string in the hover message while the code actually uses n/a, so I fixed that in fde12c172dc7ecb15c74cf012d9b61d811585323. Other than a few changes for (probably personal) style preferences, this is looking great! Appreciate how the code is quite self-contained, well tested, and the great overview in your PR description!
Also, TIL that tox can include an .env file via file|, I had no idea!

Thanks for fixing that empty string flaw! Must somehow have missed it.
I've indeed spent some time on the special ini usage of tox. Especially how multiline values are used.
If there is a better approach in the future, I will be happy to see or help with it.


Add stm32g070 and stm32g071 to replace stm32g07x

The main goal of this PR is to create separate crates for stm32g070 and stm32g071 since the g070 belongs to the value line that is missing a lot of features compared to the 'access line'.
I extracted the latest SVD files from STM32CubeIDE. They're a bit better than previous version, still quite some bugs in the files for parts other than G081 (since it is the part with all features). However the bugs are mostly peripherals that actually don't exist on that specific part. I manually edited the SVD files for G071 and G070 to remove some of those.
This addresses https://github.com/stm32-rs/stm32-rs/issues/394 and is related to https://github.com/stm32-rs/stm32g0xx-hal/issues/18

Thanks for the update! I think it does make sense to split the 70/71 apart.

I manually edited the SVD files for G071 and G070 to remove some of those.

We only put completely stock unmodified SVDs in the zip files; ideally these zip files should be directly downloaded from ST and not modified at all. Would it be possible to redo your modifications as patches in the device YAML files or in common_patches/ please?

Ok I see. It is gonna take way more time then to do. I can revert to the stock files, but the stock ST version literally has typos:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: In peripheral `ADC`

Caused by:
    0: In register `HWCFGR6`
    1: 0x1F1F1F11F invalid
    2: number too large to fit in target type', /Users/ingmar/.cargo/registry/src/github.com-1ecc6299db9ec823/svd2rust-0.17.0/src/main.rs:96:18
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
make: *** [stm32g0/src/stm32g070/mod.rs] Error 101

So I fix that now with a patch and then in the next revision ST also fixes it and then we undo the patch again? Or can svdtools/svd2rust ignore those cases then?

Now it contains the stock SVD files v1.3 with a patch to make it build. The other changes I might add back later with patches, it is a different issue anyway.

@adamgreig, anything else I should change?


Implement the file settings UI

The file settings UI should include the following:

Kinds, with the default being 'live-action', 'animated' and 'documentary'
Colors for movies and series that are not yet watched and released
Default names for a new season

Colors for movies and series that are not yet watched and released are now handled by kinds, so it won't be implemented separately.

The settings UI will also include the list name which may differ from the file name.


No such thing as oauth_token

I get the NeedsBrowser error on all 3 of my accounts. I tried following this but it wants me to find a cookie by the name of oauth_token. This cookie does not exist.

can you put a screenshot of the cookie you can see? not sure why that page doesn't give the actual URL, but this is it:
https://accounts.google.com/embedded/setup/android

This page hangs somehow. After I press "I agree", it sits there loading forever.
It does have the oauth_token cookie though so I guess this is solved lol


Recover Wallet grey button looks like the button is disabled

Replacement from @denyszayets attached.

@raedah Updated version. Including Main wallet and secondary wallet.

Will hold off on second version till we can get the multiple wallets feature flushed out.

https://d.pr/f/MQDDy3


Add Arbitrum Integration Testing

https://github.com/IndexCoop/index-coop-smart-contracts/tree/master/test/integration/arbitrum


Node.js vx.y.z is not released or avaliable, official list returns 503

My Environment

[ ] Windows 7 or below (not truly supported due to EOL - see wiki for details)

[ ] Windows 8

[x] Windows 8.1

[x] Windows 10

[ ] Windows 10 IoT Core

[ ] Windows Server 2012

[ ] Windows Server 2012 R2

[ ] Windows Server 2016

[x] My Windows installation is non-English.
Have one english and one spanish

I have already...

[x] read the README to be aware of npm gotchas & antivirus issues.
[x] reviewed the wiki to make sure my issue hasn't already been resolved.
[x] verified I'm using an account with administrative privileges.
[x] searched the issues (open and closed) to make sure this isn't a duplicate.
[x] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[ ] settings.txt
[ ] proxy support
[ ] 32 or 64 bit support

Expected Behavior
nvm install x.y.z
should install node version x.y.z and related npm
Actual Behavior
Node.js vx.y.z is not released or avaliable
And nothing installs
Steps to reproduce the problem:
nvm install 7.8.0
In fact failed even with LTS so try any
Tried to do it both in w 8.1 and w10, I have before used nvm on these machines, I got other installed versions in the list and also completely usable, but can't install anymore, also I can't reach your list
nvm4wList, returns 503, is the service down?

The old node distro list was deprecated in v1.1.1 and stopped being updated late last year. You can type nvm -v to see which version you're using, but I suspect it's 1.1.0 or below. Make sure you're using the latest version from the releases.

I was not aware, That just came from a little google search and not much reading, I apologize for that, but I'm running 1.1.3, so... what else could it be?
C:\>nvm v
1.1.3

same here, can install 7.10.0 but not 7.9.0

I'm pretty sure https://github.com/coreybutler/nvm-windows/blob/5a235cad46689972325807152bb6a1610dad8dcb/src/nvm.go#L140 is the offender. I've got OSCON to deal with this week and launching a new site next week... so it may take me a little while to get to this. I'd accept a PR.

Submitted a PR I believe It's solved


Custom dataset causing ValueError in add_tier

Hi,
I am trying to use the DeepFry model with my own dataset.
My data is downsampled to 16k, has TextGrids, and uses the same arpabet phone set.
However, when it is time to write the new TextGrids in add_tier() in utils.py, at the end of the tier there is a difference between the calculated end of the interval (prev_end) and the overall end of textgrid (max_time). In a lot of my recordings, this difference is larger than
the window size (0.005 seconds), easily twice as large. Currently, the code does not adjust the interval end time because the difference is too large, which means the interval can't be written to the textgrid, which doesn't accept intervals with max_time's after the textgrid's max_time, resulting in a ValueError.
Is this something to worry about (e.g. all the boundaries are incorrect or imprecise) or that can be fixed (other than removing the requirement that the difference between prev_end and max_time is less than f_window?
Thanks,
Emelie

Hi,
Could you provide the full error?
Alternatively, could you check if the waveform end time is greater than the textgrids max time? If so, changing the textgrid's max time will prevent the exception.


Wrong code tag included in 'Opening Screens'

Wrong include tag in Backoffice UI > Opening Screens > Executing Code after Close and Returning Values
('screens' instead of 'screen-builders')
Mistake output in rendered content:
When using ScreenBuilders, the listener can be provided in the withAfterCloseListener() method:
@Autowired
private Screens screens;
@Autowired
private Notifications notifications;
private void openOtherScreen() {
screenBuilders.screen(this)
...

 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.

Thanks!


Wrong repo language for Scala project

Github url : https://github.com/tahakashaf/LearnSpark_2.X
Launguage showing as XSLT but it needs to be scala

Preliminary Steps
Please confirm you have...

[x] reviewed How Linguist Works,
[x] reviewed the Troubleshooting docs,
[x] considered implementing an override,
[ ] verified an issue has not already been logged for your issue (linguist issues).

Problem Description

URL of the affected repository:
Last modified on:

Expected language:

Detected language:

The language shown in the search results is the first in the language statistics for your repository (in your case, XSLT). Please see Linguist overrides to tell Linguist to ignore XSLT and CSS files.

Got it ..Fixed...thanks


Autostart based on last state

As for now (latest LXD 4.20 from snap) all containers with enabled autostart starts:

when server starts and snapd starts
when LXD in snap refreshed to any new LXD version or when snapd restarts

The second case often is not desired behavior if container was stopped manually. The worst case as an example - multiple containers with routed network on different servers with the same public ip for CT - when LXD refreshes on some server in this case it starts stopped containers on current server (if autostart is enabled for them) and all traffic for this IP is going to the wrong server to container which was stopped and not supposed to be running.
Is it possible to change autostart behavior to be based on last state? I suppose volatile.last_state.power can be used as source for last state.
This can also be connected with https://github.com/lxc/lxc/issues/4029 as an alternative way to monitor the situation and autostart stopped containers which last state was "running".
Also, I suppose some ARP checks must be done during CT start and containers should not be starting if the same public ip is already running somewhere. At least for routed/macvlan/ipvlan network with public IPs. Should I open different issue about it?

snap refreshes do not restart containers, snap restart lxd would do it though but we usual advice people against doing that.
LXD default autostart is based on last state.  What you're describing suggests that you actually have boot.autostart set to true which would have LXD always start the instance every time it starts itself.
Just leave boot.autostart unset and it will follow the last state.

Key
Type
Default
Live update
Condition
Description

boot.autostart
boolean
-
n/a
-
Always start the instance when LXD starts (if not set, restore last state)

So, may be some bug occures somewhere here.
My setup:
server1: ct1, routed network, public ipv4, boot.autostart=1
server2: ct2, routed network, same public ipv4, boot.autostart=1
lxc start ct1 # now all traffic goes to ct1 at server1
lxc stop ct1
lxc start ct2 # now all traffic for the same ip goes to ct2 at server2
after some time and, I suppose, new version of LXD from snap with autorefresh enabled
and now I can see both CT is running and traffic goes to ct1 at server1 while it should not be started and ct2 with this ip must be available.
Last time it happened about yesterday nearly the same time the new LXD version arrived from snap.
How to debug this issue?

oh, I did not notice  that!

Just leave boot.autostart unset and it will follow the last state.

I suppose I understand this setting wrong before, I think it will solve the issue. Thanks!

But ARP checks still should be done, I think. I will open different issue with bash code to reproduce that.

For the ARP side of things, I'd like @tomponline's opinion, but in general, we don't prevent users from shooting themselves in the foot or doing what feels like odd setups.
What if you're trying to show a class what happens when you get a MAC or IP conflict on a network?

@stgraber How exactly can I do it?
How it looks like described in https://github.com/lxc/lxd/issues/9545#issuecomment-973317307
cisco c4948efs used before both servers.

So it does not look like ip conflict, rather ip stealing

As @stgraber in general none of the LXD NIC types prevent IP conflicts (in principle it would be possible, for example, to use the bridged NIC type with the ipv{n}.address setting to setup a static DHCP allocation that conflicts with another host on the network that has had their IP configured manually).
However routed and ipvlan NIC types are somewhat different in that they alone take a specified ipv{n}.address setting and setup ARP/NDP proxy entries on the host.
We could add an IP neighbour detection feature before starting the instance and preventing starting in that case. However it would require that the instance start be slowed down (as we would be waiting for an absence of response rather than a positive one to allow the instance to start). For that reason I suspect we would want it under a NIC level option with warnings about slowed instance start time.

@tomponline

it would require that the instance start be slowed down

It don't took much time in real life I think. I'm still using OpenVZ 6 on some legacy projects and there are ARP checks in OpenVZ and they are fast enough.
With OpenVZ ct will start without working network in case ip is already used if I recall correctly, but I'd prefer do not start at all in such case if it is possible.

as we would be waiting for an absence of response rather than a positive one to allow the instance to start

btw, why not invert this logic?
It's faster do not start in case of response (it's nearly instant) and do not wait for "no responce" (even if timeout still can be configured to small enough time)?

it's nearly instant

We would need to define how long "nearly instant" is and wait that long.

1.5 second looks fine (but may be it can be done faster):
# time arp
...
real	0m0.104s
user	0m0.002s
sys	0m0.000s

# time arping -c1 x.x.x.234 # not exists
ARPING x.x.x.234 from x.x.x.2 eno1
Sent 1 probes (1 broadcast(s))
Received 0 response(s)

real	0m1.019s
user	0m0.011s
sys	0m0.000s

# time arping -c1 x.x.x.82 # exists
ARPING x.x.x.82 from x.x.x.2 eno1
Unicast reply from x.x.x.82 [00:25:90:FC:D2:A4]  322.530ms
Sent 1 probes (1 broadcast(s))
Received 1 response(s)

real	0m1.016s
user	0m0.000s
sys	0m0.011s

Yep that sounds fine, but 1.5s is quite a long time with respect to a container start time, so I do think it should be optional (perhaps configurable timeout). We'd also need to figure out how to do this from go, as don't really want to be running a process for each IP specified on the NIC.
Can you open an issue https://github.com/lxc/lxd/issues here?

OpenVZ 6 definitely do it instantly, they do
ARPING=$(/sbin/arping -q -c 2 -w ${ARPING_WAIT:-3} -D -I ${parent_device} ${IPADDR})

if [ $? = 1 ]; then
    ARPINGMAC=$(echo $ARPING |  sed -ne 's/.*\[\(.*\)\].*/\1/p')
    net_log $"Error, some other host ($ARPINGMAC) already uses address ${IPADDR}."
    return 1
fi

it's from /etc/sysconfig/network-scripts/ifup-aliases from OpenVZ 6, may be just make the same?

That takes about 2s:
time sudo arping -q -c 2 -w -3 -D -I enp4s0 <IP_ADDRESS>
..	100% packet loss (0 extra)

real	0m2.116s
user	0m0.003s
sys	0m0.003s

strange, but even 4s on centos/openvz6, but...
# time vzctl start 2037
Starting container...
Container is mounted
Adding IP address(es): x.x.x.82
Setting devices
Container start in progress...

real	0m1.174s
user	0m0.044s
sys	0m0.056s

I will attach the full code from OpenVZ a little bit later, may be some optimization for this process is possible and used by them.

Likely ARPING_WAIT is coming from somewhere set to 1.


POE / 2.5GE option added - updated config and workflow yaml, also removed some account name hardcoding.

Updated the bpi-r4-BE14-janusz.yaml to take an optional parameter of 'poe' to build the poe version.
Updated the poe config file to have the correct target and explicitly load the 2.5GE firmware.
Replaced the hardcoded github account name in the curls to make forking easier (might make people more willing to update their own config files.)

That's very interesting PR! Could you rebase it?


Supports multiple Angular applications with shared codebase in a single instance of the seed

Is it possible to make a short example on a wiki page for this feature ?

Yes please! I am trying to find the best practices to do this myself. Adding 2 NG2 apps using this seed. Thanks!

I'll write the wiki today. Once this PR lands in master we'll have both - AoT enabled and standard production builds working for multiple apps with shared codebase!

Here's the wiki page https://github.com/mgechev/angular-seed/wiki/Multiple-Applications.

Awesomeness!

thanks for the wiki page ;)


Album Art?

When using Spotify, it sets the large image to the album art.  Is that possible here?
The art should be readable via dbus from rhythmbox failing that could be extracted by another module from the file. However, I can't see where it's currently getting the image from it sets large_image="rhythmbox" but I can't see how discord identifies the icon that "rhythmbox" is referring to.

Discord has special integration with Spotify which is why it can show album art. All image assets for normal "games" need to be uploaded in the discord developer panel (where you would give a key, in this case the key for the Rhythmbox logo is rhythmbox).
It is possible to upload images via the API and provide a unique key, but this would require the end user to make an application on the Discord developer portal and provide the client ID themselves along with their token for authentication. I have done this for other projects but I do not currently plan to implement it in this simple plugin. If anyone who is better at python than me is wanting to, please do!

I began working an implementation of this for myself and will probably upload the additions but can you clarify what you mean by

It is possible to upload images via the API and provide a unique key
Do you mean automatically or manually here? I couldn't find any reference to an automatically uploading an image through pypresence.
If it is done manually, then the most that can be added is an output folder with the covers with the fitting naming scheme that must be uploaded to the user's own application on Discord's portal.

pypresence does not provide a way to upload assets. Discord doesn't really want you to do it like this but you can by making a POST request to the API endpoint https://discordapp.com/api/oauth2/applications/APPLICATION_ID/assets with the payload:
{
  "name": "unique_key",
  "image": "data:image/png;base64,base64 image data"
}

Keep in mind that when you do this, you have a maximum of 150 assets per application on discord at a given time and you will have to keep track of how many you have uploaded and probably the timestamp that you uploaded them at. When you reach 150 assets you could for example delete the oldest asset, or delete the least used asset, etc.
It's not too hard to implement, but I don't think that this API is documented and I don't know if we are supposed to be using it. So if we're not, shhh!

You can also set it to "Listening to" as of a recent update. The API is here: https://discord.com/developers/docs/game-sdk/activities#data-models-activitytype-enum
I'm not sure it will help with album art but it may open up more possibilities.

I've uploaded my changes to a fork and finding the album art works mostly fine, excluding that it gets the image by looking through the folder of the playing song, so if there isn't a cover.png nothing is found. It can be worked over by getting the cover from the embed data of the song but it's annoying to implement with all the different file extensions. Rhythmbox also saves all it's cover art to ~/.cache/rhythmbox/album-art but I'm unsure of how they're named (3 character something, some alphanumeric others purely numeric) so I can't use it. The images also aren't always .png's so that's something else that I didn't account for.
However, the images aren't showing up in the discord developer portal meaning something is wrong with the request. It's most likely to be something out of

How I've got the base64 data in some way
The payload, particularly with data:image/png. As not all the images are pngs I decided to change it to just data:image/ but it's probably malformed in some way there
The image doesn't meet the 512*512 minimum for assets

I did manually upload an image and it worked, so the problem is somewhere in the request.

After uploading the bases to a test html (<img src="data:image/;base64, ...>) site I found that the encoding is not the problem as most of the data read fine, and the data:image/ wasn't a problem either so the cause is most likely the  API request. Did you get it wrong somehow  or am I doing something wrong?

Are you authenticating? Is that payload being encoded as JSON? Did you even look at the response from your POST?

I've now (hopefully) properly set up the headers, but I haven't implemented a way to get the auth token. How did you get yours in this application? Did you set up a server to use as the URI for discord?

I just had the user provide it in the settings.

I got it working, although for songs without a cover.png it won't work unless a different method to get the album art is used.
Can I get the icons (rhythmbox, play, pause) that you used in your application so I can upload them to Discord's assets?

Sorry, got busy this week.
Unknown: https://cdn.discordapp.com/app-assets/589905203533185064/589905260307546149.png?size=2048
Stop: https://cdn.discordapp.com/app-assets/589905203533185064/589905259972001792.png?size=2048
Pause: https://cdn.discordapp.com/app-assets/589905203533185064/589905259967545378.png?size=2048
Play: https://cdn.discordapp.com/app-assets/589905203533185064/589905260001099778.png?size=2048
Rhythmbox: https://cdn.discordapp.com/app-assets/589905203533185064/589905260328255493.png?size=2048

I figured out how to get the images from the database, had to copy a lot of code from the webremote plugin first. I'll clean up and upload this new code soon.
Should I write some code to upload all these assets you sent as a first time set up or just instruct the user to do it manually? They also need to use their own discord application, so uploading the assets shouldn't be too much extra effort lol

I would have implemented it in a way that the album art is automatically uploaded whenever it is needed. Such as when they play a new album.
I also am planning on rewriting this plugin, and I'll include this feature as well. If you want, you can put what you have into a pull request I'll merge it to another branch where I'll be doing some work on the whole plugin. Just so we don't have anything broken in master. Thanks so much!

Oh sorry, the album art is automatically uploaded but these other icons are manual. I'll add my code into another branch soon!

I've uploaded the code to my own fork since i don't have branch access lol, you can test it and add it to the main branch of this


Show aggregation if one column is grouped

Resolves #19

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

Project coverage is 99.88%. Comparing base (e5c4c2d) to head (f489763).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   99.88%   99.88%
=======================================
  Files          54       54
  Lines         850      853    +3
  Branches      203      203
=======================================
+ Hits          849      852    +3
  Misses          1        1

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


Added code for date format identifier

@abh33 moving forward, please use 4 spaces indentation. Have updated the same in the PR. Thank you for your work.


Tooltip label with multiple text colors

Is there a way to set a text with multiple colors and fonts on a tooltip?

Hello, can you take a screenshot of which tooltip you want to change ?

@hmleit you can style individual lines in a single label independently, but not more granular than that. To style individual lines, make your labels function return an array or lines of text. Then supply a style prop to VictoryTooltip with an array of style objects. Each element in your style array will be applied to an individual line.


Error running release finish

Node: 20.9.0
package manager: pnpm
CLI: gitflow-cjs 2.2.1
gitex-flow: 2.4.0
cmd: gitex-flow release finish
Config:
{
  "projectConfig": {
    "changelog": {
      "type": "ConventionalChangelog"
    },
    "bumpVersionFiles": [
      "package.json"
    ]
  }
}

Error:
[2024-01-16T03:45:26.455] [INFO] gitex-flow [release] - Finishing release branch "0.3.0"
[2024-01-16T03:45:33.709] [ERROR] gitex-flow - RangeError [ERR_CHILD_PROCESS_STDIO_MAXBUFFER]: stdout maxBuffer length exceeded
    at new NodeError (node:internal/errors:406:5)
    at Socket.onChildStdout (node:child_process:490:14)
    at Socket.emit (node:events:514:28)
    at addChunk (node:internal/streams/readable:376:12)
    at readableAddChunk (node:internal/streams/readable:345:11)
    at Readable.push (node:internal/streams/readable:286:10)
    at Pipe.onStreamRead (node:internal/stream_base_commons:190:23) {
  code: 'ERR_CHILD_PROCESS_STDIO_MAXBUFFER',
  cmd: 'git flow release finish -m 0.3.0 "0.3.0"'
}

Checking the commits and seems to be while creating the tag as the release branch was merged to master.

Also didn't merge back to develop.

Hi @bsimakis,
thank you very much for your report.
It seems that the response of the executed bash command is much larger than expected. Can you please run the raw git flow command (git flow release finish -m 0.3.0 "0.3.0") directly in your console and post the result here?
Which operating system do you use?

@jrebmann -
I am running within a Bitbucket Pipeline so it is running in Docker, specifically the node:20.9.0 image.

I have attached the bitbucket-pipelines.yml with the release pipeline using gitflow. I recently updated to use pnpm but had successfully create the release using this pipeline with npm last summer.
I have attached the logs from the pipeline run using the raw git flow command. It was successful but as you can see the output is huge as in a previous run I accidentally committed the pnpm cache.

NOTE: I have removed the specific repo details but the rest of the logs are present.
pipelineLog-{db2f0e6e-ece6-4bed-98a7-ac929e1dbbc2}.txt
bitbucket-pipelines.yml.txt

@jrebmann Thank you for providing a way to get around the stashing error and while annoying, it was not the bug I was reporting.
The log file you are looking at is for when I used the raw gitflow command you suggested, instead of release:finish.  You can see all the output that that command makes (Starts on line 244) which maxes out the buffer and causes the error I reported originally.

Okey thanks for the hint. After the command git flow release finish -m 0.3.0 "0.3.0" there is a lot of console output.
This explains why the string buffer of gitex-flow overflows.
Now the question is why there is so much console output. The git flow release finish obviously causes a huge merge with the 'master' branch.
Unfortunately, there is no environment variable to increase the buffer.
According to the documentation, the standard buffer is only 200KB (https://stackoverflow.com/a/23429654), which is too little for such a large log.
The maxBuffer option can be set centrally in the following line:
https://github.com/gitex-flow/gitex-flow-node/blob/45f3bca82663fbfe05e1cc5984f37a4e3161450e/src/avh/GitFlowBashExecuter.ts#L79
I will do this for the next bugfix release.
Thank you very much for reporting this issue.


Make more friendly when gpg agent isn't running

Removed stack trace, now prints
can't connect to `/tmp/gpg-eRshRR/S.gpg-agent': No such file or directory
gpg-connect-agent: can't connect to the agent: IPC connect call failed
ERROR connecting to gpg-agent. Try removing the file '~/.gpg-agent-info' and relogging


diffrax version 0.5.1 creates leaked tracer

when using diffrax version 0.5.1 the jax.check_tracer_leaks() test fails, while when using 0.5.0 it runs just fine.
We should investigate this at some point but not highest priority for now

something has changed and diffrax 0.5.1 is no longer compatible with the current packages in our poetry env. This may be due to packages updating their dependencies due to issues filed. Because we do not need any functionality from diffrax > 0.5.0 I will mark this issue closed.


Inaccessible HTML table markup due to alt text

Describe the bug
The HTML markup of a typical table cell generated by All Contributors looks like this:

Anand Chowdhary

💻 📖 🎨

As you'll notice, the  has alternate text with the user's name, which is then repeated below in the  tag.
The problem is that a screen reader would read this is "Anand Chowdhary Anand Chowdhary", when it should actually read it as "Anand Chowdhary". The solution is for the image to be . Empty alternate text means a screen reader should not read this image out, and is perfectly valid markup.

I'll open this issue in the cli project instead.


TS2339 triggering inside if only with dynamic comparison

TypeScript Version:  3.0.0-dev.201xxxxx
Confirmed also in next version

Search Terms:
TS2339, Property does not exist on type, Property
Code
            someFuncton (needle: User | string | null): any[]=> {
                let toReturn = [];
                // Case string
                if (typeof needle === typeof whitespace) {
                    toReturn = needle.length > 2 ? this._filterUsers(needle) : [];
                }
                // Case User
                else if (!!needle) {
                    return _.filter(this.usersList, needle)
                }
                // Case NULL
                else {
                       //Do something
                }
                return toReturn;
            })

Expected behavior:
No error is triggered
Actual behavior:
A TS2339 error is triggered:

error TS2339: Property 'length' does not exist on type 'string | User'.
error TS2345: Argument of type 'string | User' is not assignable to parameter of type 'string'.

The second error is due to the fact that the _filterUsers function accepts User as argument type.
Playground Link:
http://www.typescriptlang.org/play/#src=export class User {
    firstName%3A string%3B
    lastName%3A string%3B
    userId%3A string%3B
    division%3A string%3B
    businessUnit%3A string%3B
}

const whitespace %3D " "%3B

export function myCallback(needle%3A User | string | null) {
    let toReturn %3D []%3B
    %2F%2F Case string
    if (typeof needle %3D%3D%3D typeof whitespace) {
        toReturn %3D needle.length > 2 %3F ["A"%2C "B"%2C "C"] %3A []%3B
    }
    %2F%2F Case User
    else if (!!needle) {
        %2F%2F Do something else
    }
    %2F%2F Case NULL
    else {
        %2F%2F Do awesome
    }
    return toReturn%3B
}%3B
Related Issues:
Not precisely like this

Note: the problem solves if I use typeof needle === "string" or by adding a cast expression inside the if statement: <string>needle.length


The weights too big !!!

OMG.....

LLM Models are usually large. grok-1 is one of the largest. Hence it is a huge download. Also the system requires 8 GPUs with a total memory (VRAM) that is larger than the model size to run.
Please close this issue and move to:
https://github.com/xai-org/grok-1/discussions
Reason:
Not a real issue
#69
#108


fix: Enhanced cross-fading of Boombox

Set TransitionType and EaseType in Boombox Tween to enhance cross-fading.
Fixes https://github.com/deep-entertainment/issues/issues/39

Thanks a lot. 🎁


Inspect macro

Man, the amount of work you're getting done on this since updating the backend to TS is incredible. Nice work!


GPU Memory error when running DenseFlow

Running denseflow using the mmaction 'rawframes.py' script works for the RGB extraction but fails when tryign to extract flow images, throwing the following error:
terminate called after throwing an instance of 'cv::Exception'
what():  OpenCV(4.2.0) /home/******/opencv-4.2.0/modules/core/src/cuda_host_mem.cpp:95: error: (-217:Gpu API call) out of memory in function 'allocate'
I only use 1 gpu (eGPU), but set it as such when calling script:
python build_rawframes.py /mnt/d/Data/ava/videos_15min/ /mnt/d/Data/ava/rawframes/ --task flow --level 1 --mixed-ext --flow-type tvl1 --num-gpu 1 --resume
Any ideas?

See https://github.com/open-mmlab/denseflow/issues/18

Please post the full stacktraces

Try making this number smaller
https://github.com/open-mmlab/denseflow/blob/master/include/dense_flow.h#L95

I've tried to set the batch size to 32, but unfortunately to no avail. This is a bit of a differant beast when it comes to setup, as it is a WSL ubuntu VM on Win 10 bootcamp (Macbook Pro) using an Aurous RTX 2080ti 11Gb GPU. So it might be a result of such a daisy chain of connections, but it is a bit odd given that at 11Gb I would have though it would be sufficient. Or possibly does the DenseFlow script not run through the GPU?
Unfortunately I can't seem to get a full stack trace from error, as it never really kills the process / script, and terminal / PyCharm just prints out as below.

flow 36 bhlFavrh7WU.mp4 tvl1 done
push frames gray, video_flow_idx 127, batch_size 32
frames_gray_queue full, waiting...
"/mnt/d/Data/ava/videos_15min/3IOE-Q3UWdA.mp4", frames ≈ 30600
"/mnt/d/Data/ava/videos_15min/z3kgrh0L_80.mp4", frames ≈ 30600
"/mnt/d/Data/ava/videos_15min/_-Z6wFjXtGQ.mp4", frames ≈ 30600
push frames gray, video_flow_idx 0, batch_size 32
push frames gray, video_flow_idx 0, batch_size 32
push frames gray, video_flow_idx 0, batch_size 32
push frames gray, video_flow_idx 31, batch_size 32
push frames gray, video_flow_idx 31, batch_size 32
push frames gray, video_flow_idx 31, batch_size 32
push frames gray, video_flow_idx 63, batch_size 32
push frames gray, video_flow_idx 63, batch_size 32
push frames gray, video_flow_idx 63, batch_size 32
push frames gray, video_flow_idx 95, batch_size 32
push frames gray, video_flow_idx 95, batch_size 32
push frames gray, video_flow_idx 95, batch_size 32
terminate called after throwing an instance of 'cv::Exception'
what():  OpenCV(4.2.0) /home/*****/opencv-4.2.0/modules/core/src/cuda_host_mem.cpp:95: error: (-217:Gpu API call) out of memory in function 'allocate'

terminate called after throwing an instance of 'cv::Exception'
what():  OpenCV(4.2.0) /home/*****/opencv-4.2.0/modules/core/src/cuda_host_mem.cpp:95: error: (-217:Gpu API call) out of memory in function 'allocate'
frames_gray_queue full, waiting...
Aborted
flow 37 z3kgrh0L_80.mp4 tvl1 done
Aborted
flow 23 3IOE-Q3UWdA.mp4 tvl1 done
"/mnt/d/Data/ava/videos_15min/X5wWhZ2r9kc.mp4", frames ≈ 30600
"/mnt/d/Data/ava/videos_15min/uq_HBsvP548.mp4", frames ≈ 30600
push frames gray, video_flow_idx 0, batch_size 32
push frames gray, video_flow_idx 0, batch_size 32
push frames gray, video_flow_idx 31, batch_size 32
push frames gray, video_flow_idx 31, batch_size 32
push frames gray, video_flow_idx 63, batch_size 32
push frames gray, video_flow_idx 63, batch_size 32
push frames gray, video_flow_idx 95, batch_size 32
push frames gray, video_flow_idx 95, batch_size 32
terminate called after throwing an instance of 'cv::Exception'
what():  OpenCV(4.2.0) /home/*****/opencv-4.2.0/modules/core/src/cuda_host_mem.cpp:95: error: (-217:Gpu API call) out of memory in function 'allocate'