Line drawn out of bounds on dataset update

Using a LineChart, when the dataset is updated with a large value, the recomposition results in a line being drawn out of bounds.
LineChart(
            lineData = pagesReadData,
            color = MaterialTheme.colors.primaryVariant,
            modifier = modifier
                .height(250.dp)
                .fillMaxWidth()
                .padding(32.dp),
            axisConfig = AxisConfig(
                showAxis = true,
                isAxisDashed = false,
                showUnitLabels = true,
                showXLabels = true,
                xAxisColor = MaterialTheme.colors.onSurface,
                yAxisColor = MaterialTheme.colors.onSurface,
            )
        )

Resultant:

ScreenCap:
Out of bounds.webm
Please let me know if any other data is needed.

Hey buddy, thank you for this. Can you pass your dataset for me to replicate!
thanks

Initial Data:
LineData(xValue=23 Sep , yValue=10.0)
LineData(xValue=24 Sep , yValue=10.0)

Updated Data:
LineData(xValue=23 Sep , yValue=10.0)
LineData(xValue=24 Sep , yValue=10.0)
LineData(xValue=26 Sep , yValue=20.0)

Result:

Thank you, will check soon!

Hey @hi-manshu sorry to bug you, but any update on this?

Hi,
I'm facing similar problem with CurveLineChart.
When I load the follwing LineData, recomposition draw the CurveLine chart out of bounds.

LineData that I used.
listOf(
  LineData(xValue = "23 Sep", yValue = 0.793f),
  LineData(xValue = "24 Sep", yValue = 0.766f),
  LineData(xValue = "25 Sep", yValue = 0.746f),
  LineData(xValue = "26 Sep", yValue = 0.786f),
  LineData(xValue = "27 Sep", yValue = 0.735f),
  LineData(xValue = "28 Sep", yValue = 0.712f),
  LineData(xValue = "29 Sep", yValue = 0.744f),
  LineData(xValue = "30 Sep", yValue = 0.774f)
)

Fixed this issue in PR #55

Hi @AlexZhukovich ,
Tested the fix with your branch. Found that the issue still exist with the following LineData.
listOf(
  LineData(xValue = "23 Sep", yValue = 0.793f),
  LineData(xValue = "24 Sep", yValue = 0.766f),
  LineData(xValue = "25 Sep", yValue = 0.746f),
  LineData(xValue = "26 Sep", yValue = 0.786f),
  LineData(xValue = "27 Sep", yValue = 0.735f),
  LineData(xValue = "28 Sep", yValue = 0.712f),
  LineData(xValue = "29 Sep", yValue = 0.744f),
  LineData(xValue = "30 Sep", yValue = 0.774f)
)

@khaleeljageer I think here we have different issues. The initial issue is related to adding additional data and updating the chart component. In your case, the data is displaying incorrectly.

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Don't miss out on your $1500 in BNB! Act now to secure your tokens.

Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Don't miss out on your $1500 in BNB! Act now to secure your tokens.
Winners: @qst0, @wtfiwtz, @0xhydrogen, @coolcode, @barrasso, @zubayrrr, @willzhuang, @SofianeBoumedine, @eolszewski, @rayeaster, @pcranaway, @yongseongkim, @TerrorJack, @simudream, @qfengthree, @stas, @eueung, @UnderratedDev, @bmiller59, @TheBlapse, @sheluchin, @cppchedy, @veronicaz41, @notaquant, @sger, @dmitri-dev, @seansummers, @cavb, @maximebrugel, @drizzyrdrgz-exe, @hp-huang-tw, @Nubuck, @keivinonline, @Dyonishw, @makramkd, @seifsg, @julian0zzx, @maaverik, @nimeshneema, @ojhariddhish, @holyshared, @kaleb-keny, @ricoregala, @laizy, @melvinzhang, @durdn, @hc0208, @onlymaresia, @luisgdev, @headsrooms

AltLayer Airdrop Season One Update

🚀 Exciting Announcement! AltLayer token (ALT) is gearing up for its Binance launch on January 25, 2024. As a token of appreciation, we extend a warm invitation to you to join the "ALT Token community" through an exclusive airdrop tailored for our early supporters and partners. Here's a quick overview:

Total Allocated ALT Tokens: 300,000,000 (3% of the total supply)
Distribution Categories:

Ethereum Mainnet
Binance Mainnet

For a comprehensive breakdown of criteria and further details, refer to the complete announcement Oficial Website.
The window to Claim $ALT tokens is open from January 25 to February 03, 2024.
Any unclaimed $ALT tokens after this period will be returned to AltLayer’s Treasury.
We deeply appreciate your contribution to AltLayer's success! Stay tuned for more exciting developments in Season One.
Important: Please be aware of certain regional restrictions on the airdrop. Review the disclaimer before participating.
Mofu2Monarch, @soltantafe, @90tech, @onorjay, @hahaguo, @ahmetcaglayan, @Jackiexiao, @senzen777, @Leo-Lee15, @steven5538, @gengmoqi, @JoseRodrigues443, @whiteclover, @codyphobe, @MM422, @Ubiquitouskiwi, @fang503, @ys1111, @mmuhd, @flyingfoxlee


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/example.justlists.ts
Checklist

[ ] I understand that this is not the place to request a token listing.
[ ] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[ ] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL: https://tronscan.org/#/token/1003452
JustList Name: Bilween
Link to the official homepage of the JustList manager: https://www.bilween.org

Sorry, your issue will be closed as you did not submit your information in the correct format.


CLI Building Algorithm Containers

Work with Jared to make sure that the CLI can build algorithm containers in an easy way for algorithm developers.

I'd say this is OBE, now that we are working toward Seed integration and have the CLI accomplishing this.


Patch 1

Thanks for taking the time to submit a pull request!
If you haven't already, be sure to take a look at the CONTRIBUTING documentation to get up to speed with the practices in this repository.
When you're ready, switch to the "Preview" tab to select an applicable template:

General template (most common)
Wallet integration template

Hi, @Yoomxtt we're closing this PR for now, as it's not clear what the goal is here. Feel free to reopen if needed.


Packaged HPX distribution does not pull in all required packages for Fedora29 nor are they all documented.

Expected Behavior
"sudo dnf install hpx" should install required packages, and those packages should be documented by listing all that are required (in "getting started").
Since the user is installing compiled packages, it should not be presumed that the user has "compiled HPX" and read that section, which does give some clues.
Actual Behavior
"sudo dnf install hpx" does not install hwloc
Steps to Reproduce the Problem
"sudo dnf repoquery --depends hpx"
Does not show "hwloc"
Specifications

HPX Version: 1.2.0
Platform (compiler, OS): Fedora29

[diehlpk@xwing rpmbuild]$ dnf repoquery --depends hpx
Fedora Modular 29 - x86_64                                                                                11 kB/s |  18 kB     00:01
Fedora Modular 29 - x86_64 - Updates                                                                                12 kB/s |  17 kB     00:01
Fedora 29 - x86_64 - Updates                                                                                14 kB/s |  17 kB     00:01
Fedora 29 - x86_64 - Updates                                                                                3.0 MB/s |  21 MB     00:07
google-talkplugin                                                                                895  B/s | 951  B     00:01
RPM Fusion for Fedora 29 - Free - Updates                                                                                9.9 kB/s |  13 kB     00:01
RPM Fusion for Fedora 29 - Nonfree - Updates                                                                                9.8 kB/s |  13 kB     00:01
skype (stable)                                                                                4.8 kB/s | 2.9 kB     00:00
ld-linux-x86-64.so.2()(64bit)
ld-linux-x86-64.so.2(GLIBC_2.3)(64bit)
ld-linux.so.2
ld-linux.so.2(GLIBC_2.3)
libboost_atomic.so.1.66.0
libboost_atomic.so.1.66.0()(64bit)
libboost_filesystem.so.1.66.0
libboost_filesystem.so.1.66.0()(64bit)
libboost_program_options.so.1.66.0
libboost_program_options.so.1.66.0()(64bit)
libboost_regex.so.1.66.0
libboost_regex.so.1.66.0()(64bit)
libboost_system.so.1.66.0
libboost_system.so.1.66.0()(64bit)
libc.so.6(GLIBC_2.14)(64bit)
libc.so.6(GLIBC_2.28)
libdl.so.2
libdl.so.2()(64bit)
libdl.so.2(GLIBC_2.0)
libdl.so.2(GLIBC_2.1)
libdl.so.2(GLIBC_2.2.5)(64bit)
libdl.so.2(GLIBC_2.3.3)
libdl.so.2(GLIBC_2.3.3)(64bit)
libgcc_s.so.1
libgcc_s.so.1()(64bit)
libgcc_s.so.1(GCC_3.0)
libgcc_s.so.1(GCC_3.0)(64bit)
libgcc_s.so.1(GCC_3.3)
libgcc_s.so.1(GCC_3.3)(64bit)
libgcc_s.so.1(GLIBC_2.0)
libhwloc.so.5
libhwloc.so.5()(64bit)
libm.so.6
libm.so.6()(64bit)
libm.so.6(GLIBC_2.0)
libm.so.6(GLIBC_2.1)
libm.so.6(GLIBC_2.2.5)(64bit)
libpthread.so.0
libpthread.so.0()(64bit)
libpthread.so.0(GLIBC_2.0)
libpthread.so.0(GLIBC_2.1)
libpthread.so.0(GLIBC_2.2.5)(64bit)
libpthread.so.0(GLIBC_2.3.2)
libpthread.so.0(GLIBC_2.3.2)(64bit)
libpthread.so.0(GLIBC_2.3.3)
libpthread.so.0(GLIBC_2.3.3)(64bit)
librt.so.1
librt.so.1()(64bit)
libstdc++.so.6
libstdc++.so.6()(64bit)
libstdc++.so.6(CXXABI_1.3)
libstdc++.so.6(CXXABI_1.3)(64bit)
libstdc++.so.6(CXXABI_1.3.11)
libstdc++.so.6(CXXABI_1.3.11)(64bit)
libstdc++.so.6(CXXABI_1.3.2)
libstdc++.so.6(CXXABI_1.3.2)(64bit)
libstdc++.so.6(CXXABI_1.3.3)
libstdc++.so.6(CXXABI_1.3.3)(64bit)
libstdc++.so.6(CXXABI_1.3.5)
libstdc++.so.6(CXXABI_1.3.5)(64bit)
libstdc++.so.6(CXXABI_1.3.8)
libstdc++.so.6(CXXABI_1.3.8)(64bit)
libstdc++.so.6(GLIBCXX_3.4)
libstdc++.so.6(GLIBCXX_3.4)(64bit)
libstdc++.so.6(GLIBCXX_3.4.11)
libstdc++.so.6(GLIBCXX_3.4.11)(64bit)
libstdc++.so.6(GLIBCXX_3.4.14)
libstdc++.so.6(GLIBCXX_3.4.14)(64bit)
libstdc++.so.6(GLIBCXX_3.4.15)
libstdc++.so.6(GLIBCXX_3.4.15)(64bit)
libstdc++.so.6(GLIBCXX_3.4.18)
libstdc++.so.6(GLIBCXX_3.4.18)(64bit)
libstdc++.so.6(GLIBCXX_3.4.19)
libstdc++.so.6(GLIBCXX_3.4.19)(64bit)
libstdc++.so.6(GLIBCXX_3.4.20)
libstdc++.so.6(GLIBCXX_3.4.20)(64bit)
libstdc++.so.6(GLIBCXX_3.4.21)
libstdc++.so.6(GLIBCXX_3.4.21)(64bit)
libstdc++.so.6(GLIBCXX_3.4.22)
libstdc++.so.6(GLIBCXX_3.4.22)(64bit)
libstdc++.so.6(GLIBCXX_3.4.9)
libstdc++.so.6(GLIBCXX_3.4.9)(64bit)
libtcmalloc_minimal.so.4
libtcmalloc_minimal.so.4()(64bit)
rtld(GNU_HASH)

@grantrostig Could you please post your output too, because on my machine
libhwloc.so.5
libhwloc.so.5()(64bit)

I can see hwloc as one of the dependencies.

Please not that dnf install hpx will install only missing dependencies and if you have hwloc already installed, it will not be shown there.

Yes, hwloc-libs is installed, but hwloc is not.  Apparently hwloc is needed, so that is a problem.

Ok, hwloc is just the name of the library. It is an fedora package name again, the package hwloc in fedora just contains the executables:
/usr/bin/hwloc-annotate
/usr/bin/hwloc-assembler
/usr/bin/hwloc-assembler-remote
/usr/bin/hwloc-bind
/usr/bin/hwloc-calc
/usr/bin/hwloc-compress-dir
/usr/bin/hwloc-diff
/usr/bin/hwloc-distances
/usr/bin/hwloc-distrib
/usr/bin/hwloc-gather-topology
/usr/bin/hwloc-info
/usr/bin/hwloc-ls
/usr/bin/hwloc-patch
/usr/bin/hwloc-ps
/usr/bin/lstopo-no-graphics
/usr/lib/systemd/system/hwloc-dump-hwdata.service
/usr/sbin/hwloc-dump-hwdata
/usr/share/man/man1/hwloc-annotate.1.gz
/usr/share/man/man1/hwloc-assembler-remote.1.gz
/usr/share/man/man1/hwloc-assembler.1.gz
/usr/share/man/man1/hwloc-bind.1.gz
/usr/share/man/man1/hwloc-calc.1.gz
/usr/share/man/man1/hwloc-compress-dir.1.gz
/usr/share/man/man1/hwloc-diff.1.gz
/usr/share/man/man1/hwloc-distances.1.gz
/usr/share/man/man1/hwloc-distrib.1.gz
/usr/share/man/man1/hwloc-dump-hwdata.1.gz
/usr/share/man/man1/hwloc-gather-topology.1.gz
/usr/share/man/man1/hwloc-info.1.gz
/usr/share/man/man1/hwloc-ls.1.gz
/usr/share/man/man1/hwloc-patch.1.gz
/usr/share/man/man1/hwloc-ps.1.gz
/usr/share/man/man1/lstopo-no-graphics.1.gz

So hpx needs the packaged named hwloc [0], but we do not use these executables within hpx. We use the libraries hwloc-libs [1] when we install hpx and hpx-examples. For hpx-devel we need to install hwlox-devel [2]. So it is might be confusing due to the package name in fedora, but we do not meed the hwloc package with the executables, but we need hwloc-libs and hwloc-devel.
So to summarize, we do not miss the hwloc package for the fedora packages.
[0] https://www.open-mpi.org/projects/hwloc/
[1] https://rpmfind.net/linux/RPM/fedora/devel/rawhide/x86_64/h/hwloc-libs-1.11.12-1.fc30.i686.html
[2] https://rpmfind.net/linux/RPM/fedora/devel/rawhide/x86_64/h/hwloc-devel-1.11.12-1.fc30.i686.html

@grantrostig Does this clarify your concerns? If so, the ticket can be closed.

The procedure for building the helloworld from source is the problem I reported.  Your fine explanation does not fix the problem.  A fix in the documenation is needed to discuss manual workarounds for what I call an error or a simple change in the dependencies on the package "hpx" is required, as well as all the other hpx packages, and also rewording of their description to be more clear.  Then it should be tested on a clean "fedora29 workstation" system using the new updated package via fedora dnf.  Then I would think the ticket should be closed.  Do we mark issues fixed before that?  No not in my opinion, but I don't know the policies of the hpx project.  If one wants automated testing, I guess that would also have to be fixed.

I'll try to summarize the state of this right now. First, keep in mind that the Fedora package is from December and we're learning as we speak. Unfortunately this leaves you, @grantrostig, as a guinea pig, and we appreciate your patience with us.

1.2.0 was a broken release, which is why we're preparing 1.2.1 which will contain fixes to the CMake configuration (and the HPX library itself). Trying to use 1.2.0 is likely a doomed exercise unless you have too much time.
The hpx-devel package might need libatomic as a dependency (#3673)
We would prefer to remove build instructions for Fedora from the documentation and make sure that dnf install hpx/hpx-devel does what it should instead.

@msimberg , Thank you for your response.  I'm glad my work in testing is useful.  The steps you outlined and quoted below seem like a good plan.  I'm not sure who closes issues and if they are only closed after the issue has been tested and passed?  Another issue might have to be created to address your third point?

1.2.0 was a broken release, which is why we're preparing 1.2.1 which will contain fixes to the CMake configuration (and the HPX library itself). Trying to use 1.2.0 is likely a doomed exercise unless you have too much time.

The hpx-devel package might need libatomic as a dependency (#3673)

We would prefer to remove build instructions for Fedora from the documentation and make sure that dnf install hpx/hpx-devel does what it should instead.

Status of the three points I made earlier:

1.2.1 is just about to be released, and fixes the packaging issues that we were aware of
@diehlpk is libatomic now a dependency of HPX in the new Fedora spec files?
@diehlpk could you take a look through the Fedora section in the manual and make sure it doesn't contain any out-of-date or wrong information, and clean it up if needed?

* @diehlpk is libatomic now a dependency of HPX in the new Fedora spec files?

Yes, it is a new requirement

Let's close this as the V1.2.1 release was published. Feel free to re-open, if necessary.


Update gem thin to version >1.8.0 to resolve installation issues

Gem thin is required by mailcatcher and is specified within a minor version of v1.5.0:
https://github.com/sj26/mailcatcher/blob/30b10049b806a64d1c12419574334f6bd1634223/mailcatcher.gemspec#L42
The installation of gem thin requires a workaround that is documented in the readme:

If you encounter issues installing thin, try:
gem install thin -v 1.5.1 -- --with-cflags="-Wno-error=implicit-function-declaration"

These issues have since been resolved in https://github.com/macournoyer/thin/pull/364 and the fix is available since version v1.8.0.
Is it possible to upgrade the thin dependency or are there any blocking requirements to keep the version as is?

Yes there were/are a few issues preventing the upgrade. They are being worked through. Please use the documented workaround in the meantime.

Thanks for the response.
Of the remaining work required to upgrade, is any of it documented anywhere such that people can contribute towards helping?

No, it is not.
If you'd like to help, please try the latest pre-release version which has swapped out the websocket library which was preventing upgrade:
gem install --pre mailcatcher

If you find any issues please report them.

gem install --pre mailcatcher => mailcatcher-0.9.0.beta1
@sj26 this appers to have the same problem?

@alistairholt could you please be more specific?
The beta doesn't upgrade thin, it upgrades the websocket integration. I'd like to see that stable before committing to upgrading thin.

@sj26 sure. When I tried the beta I still got the same error from thin on install.

I was struggling with installing thin for an entire night, finally I figured out I had to add "-fdeclspec" flag.
I'm still running Mac OS Catalina with xcode 12.2, ruby 3.1.0 installed via rbenv.
Maybe someone finds it useful.
gem install thin -v 1.5.1 -- --with-cflags="-Wno-error=implicit-function-declaration -fdeclspec"
gem install mailcatcher -N
Inspiration:
https://github.com/nginx/unit/issues/653

Had this issue. Did not need the -fdeclspec flag.
Just had to do:
gem install thin -v '1.5.1' -- --with-cflags="-Wno-error=implicit-function-declaration"
then was able to do:
gem install mailcatcher
Qiita Source. This is a Japanese site; I used Google Translate to read. I did not need to use sudo.
Translated Qiita Source.
StackOverflow Source.
System details:
MacBook Pro
16-inch 2021
Apple M1 Max
32 GB Memory
macOS Ventura 13.2

I have pushed another pre-release, v0.9.0.beta2, which relaxes the thin requirement and should support ruby 3.2. Please try it:
gem install mailcatcher --pre


Add String::create_and_take_ownership

This takes a buffer as argument, and takes ownership of the buffer. Mostly useful if you already have a read buffer and just need to wrap it into a String object.
Closes #10

https://bugs.ruby-lang.org/issues/20878
It's almost as if people are watching this and taking inspiration from it :D


Aggregation operators refactor

Description
Closes #2317.

Readiness checklist

[ ] I added/updated unit tests.
[ ] I added/updated integration/compatibility tests.
[ ] I added/updated comments and checked rendering.
[x] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[x] I ensured that PR title is good enough for the changelog.
[x] (for maintainers only) I set Reviewers (@FerretDB/core), Labels, Project and project's Sprint fields.
[x] I marked all done items in this checklist.

One fix needed...


Documentation page for bootstrapper contains HTML tags in title element

That probably shouldn't be there, right?

You're right - if you look at the other titles in the same part of the docs menu, you can see they follow a pattern of "# The <something> file" (replace "" with the item in question), whereas the paket.bootstrapper.exe page does not, but instead includes the backtick notation around file paths, that are used everywhere else in the page.
Would you like to have a go at it, as a PR? It seems to be fixable by just utilizing the same header pattern as on the other pages. If not, I'll get around to it. :)


rktlet: make stage1-name configurable

The current rkt integration makes it possible to configure the stage1
image so we should do the same in the rktlet.

Two doubts on this:

we used to have a per-pod annotation in rktnetes, it would be nice to have it here as well.
I think stage1 name should also specify tag version, as we don't publish a latest tag for them. If so, better to say it explicitly in here.

@lucab I see the annotation bit as orthogonal to this since that's about per-pod override and this is about a sane default.
The version is specified explicitly, right?

Right. I moved my wish/request over to https://github.com/kubernetes-incubator/rktlet/issues/92. Better to add the version in the help text example, though.

A few small naming things, but otherwise looks good. Do you want to also tackle #92 in this PR?

@squeed thanks a lot for the review! Yes, I think I'll collect #92 in conjunction with this for the next round.

/cc @grahamwhaley
This is the first step to bring forward kvm (and other stage1) to the CRI world from previous rktnetes world. I don't expect this to be in a working state right now, especially for the logging part.


Rendering Images

Could you please provide the rendering images in single view reconstruction task or the rendering scripts? Thank you very much.

We use the ShapeNet rendered images from http://cvgl.stanford.edu/data2/ShapeNetRendering.tgz, provided by 3D-R2N2. Since each shape of PartNet has the corresponding ShapeNet ID, we can find its rendered images from the downloaded files.


Not able to build with carthage

Hi,
I upadated my Mac os to 10.13.6 and updated my Xcode to 10.1. After that when I tried to build Xcode gave error "Module Locksmith compiled with swift 4.0.3 can not be imported by swift 4.2 compiler".
So I tried to fetch and build latest Locksmith version using "Carthage update" but I got
"The file “Locksmith.framework” couldn’t be opened because there is no such file." UserInfo={NSFilePath=/Users/xxxxxxxx/Library/Caches/org.carthage.CarthageKit/DerivedData/Locksmith/4.0.0/Build/Products/Release-appletvos/Locksmith.framework, NSUnderlyingError=0x7f8bcf2ce610 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}
Let me know if I'm doing anything wrong from my side.

It builds just fine for iOS at my end, but blows up with macOS:

*** Building scheme "Locksmith iOS" in Locksmith.xcworkspace
*** Building scheme "Locksmith OS X" in Locksmith.xcworkspace
Build Failed
Task failed with exit code 65:

In the build log file I noticed this:

error: SWIFT_VERSION '3.0' is unsupported, supported versions are: 4.0, 4.2, 5.0. (in target 'Locksmith OS X')

It's possible that someone hasn't updated macOS' build to use a newer Swift. I'm going to checkout the code and give it a try. If it works, I'll do a pull request.


Fixes FormatException when Tracing is enabled

This is a fix for the bug #1290 by simply removing the redundant argument from the overload.

The use case is related to refreshing access token before expiry which will be a long wait in test suite.
The changes look reasonable, we should prevent overriding of methods with optional params when using the generic types. The right method that should have been called in this case is on L371.

Ok, making the tests take even longer isn't a good idea.

@shueybubbles I will add this to our list.


Any Limitations while in Preview?

Is there currently a max on the amount of transaction/requests with this feature while in preview?  I have a policy to delete any blob in my container that is older than 90 days.   There is a folder structures that ultimately breaks down to a folder per day, which each folder should hold 3million+ blobs.  When the policy runs, its seems to cap out around 300K requests, which is far below what should be the number.  This falls back into my previous submitted question about logging as I have no visibility into troubleshooting this.
Thanks.

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

ID: 9a558a9e-10d0-ea34-8780-d9084f87f952
Version Independent ID: 60dda4e0-7630-f358-aa3a-6a992eae61e2
Content: Managing the Azure Storage Lifecycle
Content Source: articles/storage/blobs/storage-lifecycle-management-concepts.md
Service: storage
GitHub Login: @yzheng-msft
Microsoft Alias: yzheng

@JRiggenbach Thanks for your question. We are checking on this and will respond to you soon.

@JRiggenbach There was no performance tuning in preview. We will GA this feature by end of the month and this issue will go away at GA.

It’s going to be updated soon, You may keep a track on Azure Updates
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issu


[webapi][XWALK-2955] Update background-size-30.html test

Change the word squares to square in this test so that it can share a same reftest with other tests.

Impacted tests(approved): new 0, update 1, delete 0
Unit test platform: [TizenIVI]
Unit test result summary: pass 0, fail 1, block 0

Please make proper indent of the commit message.
Another, has this update applied to W3C?

I have committed this update to W3C already.

Then update the commit message; this shall be an integration or sync test code with W3C test.


Refactor LinearActQuantizedTensor

Summary:

rename to LinearActivationQuantizedTensor
using implements util to implement torch function and torch dispatch overwrites
refactored tensor subclass dispatch related utility functions: _implements and added two dispatch helpers: _dispatch__torch_function__ and _dispatch__torch_dispatch__ that saves more lines of code from users

Test Plan:
CI
Reviewers:
Subscribers:
Tasks:
Tags:

@jcaip please take a look again, the scope increased a bit to include a refactor for torch_function/torch_dispatch related utils functions
also @msaroufim please take a look as well

nice clean change, just double checking this won't cause any BC issues on tune side for their nf4 recipe?

oh we haven't changed nf4 yet, I can do it in a separate PR


when are the enterprise version comming.

we're still waiting...
considered it was scheduled to be released 2 years ago.
would be easier if you had MVC - build into software. so we don't have to do it. - building the singleton classes.  can become bothersome.
and introduces redudant code. which later has to be refactored.

mvc stands for massive virtual cock??????? i have fear


cat: /sys/devices/virtual/dmi/id/product_name: No such file or directory

When running confetch from WSL (Windows Subsystem for Linux), attempting to automatically grab the model identifier from /sys/devices/virtual/dmi/id/product_name results in the error:
cat: /sys/devices/virtual/dmi/id/product_name: No such file or directory

Here's how I'm working around this issue:
new code:
if [ "$MODEL" = true ] ; then
    if ! cat /sys/devices/virtual/dmi/id/product_name 2>/dev/null; then
        if grep -q -i "Microsoft" /proc/sys/kernel/osrelease || grep -q -i "WSL" /proc/sys/kernel/osrelease; then
            tMODEL="Windows Subsystem for Linux"
        else
            # Failed to grab the model identifier. (Probably running on WSL).
            tMODEL="Computer"
        EF1="Warning: Model grab failed. Defaulting to "Computer"; Set MODEL manually in config.txt to dismiss this message."
        fi
    else
        tMODEL=$(cat /sys/devices/virtual/dmi/id/product_name)
    fi
else
    tMODEL=$MODEL
fi

Now, the script first checks to see if it can't cat the file.  If it fails to cat the file, it first checks to see if the user is running on WSL. If they are, it sets tMODEL to "Windows Subsystem for Linux", otherwise it defaults to setting tMODEL = "Computer". It then flags EF1 (Error Flag 1) with the warning message:
Warning: Model grab failed. Defaulting to "Computer"; Set MODEL manually in config.txt to dismiss this message.

Fixed in fd0abea.


local package definition missing from bundle when using jspm 0.17.0-beta.40

I'm submitting a bug report or I'm submitting a feature request I'm not sure.

Library Version:
0.5.0

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
7.5.0

NPM Version:
4.2.0

JSPM Version
JSPM  0.17.0-beta.40

Language:
TypeScript 2.1.5

Current behavior:
Given a jspm.config.js like the following:
SystemJS.config({
  browserConfig: {
    "baseURL": "/dist",
    "paths": {
      "*": "dist/*",
      "github:": "/jspm_packages/github/",
      "npm:": "/jspm_packages/npm/",
      "yw-event-viewer/": ""
    }
  },
  devConfig: {
    // etc
  },
  transpiler: false,
  packages: {
    "yw-event-viewer": {
      "main": "main.js",
      "format": "amd"
    }
  }
});

SystemJS.config({
  packageConfigPaths: [
<EMAIL_ADDRESS>    "npm:*.json",
    "github:*/*.json"
  ],
  map: {
    "aurelia-animator-css"<EMAIL_ADDRESS>    "aurelia-api"<EMAIL_ADDRESS>    //etc
  },
  packages: {
<EMAIL_ADDRESS>{
      "map": {
        "aurelia-metadata"<EMAIL_ADDRESS>        "aurelia-pal"<EMAIL_ADDRESS>        "aurelia-templating"<EMAIL_ADDRESS>      }
    },
<EMAIL_ADDRESS>{
      "map": {
        //etc
      }
    }
  }
});

While the bundle task works without any errors the resultant file is missing one section:
  packages: {
    // this is the local package definition
    "yw-event-viewer": {
      "main": "main.js",
      "format": "amd"
    }
  }

I suspect (guess) this might be because the systemjs-builder dependency is old (0.15->0.16) and rejects the local package definition as invalid. I tried cloing this repo but I can't get anything to build and so couldn't test that theory.
Another note to make: jspm 0.17 appears to separate the jspm.config.js file into two parts (i.e. two SystemJS.config calls) as you can see above. The I've checked the bundler code and the reader doesn't take into consideration that a single file could have multiple calls although I don't think this is an issue here, A single SystemJS.config call will work so long as this missing package entry is included (I've manually included the missing entry and it works).
Expected/desired behavior:
The local package definition is included in the bundle so that the bundler functions for jspm 0.17 projects. Especially now that loader-default is compatible (https://github.com/aurelia/loader-default/commit/a4607b9)
/cc @ahmedshuhel

Please excuse the ping ahmed, I'm in a rather desperate situation (https://github.com/jspm/jspm-cli/issues/2252) so am exploring all options.

@ahmedshuhel Can you take a look at this?

@AdamWillden  @EisenbergEffect  Will look into it tonight.

@EisenbergEffect @ahmedshuhel I've just realised I completely missed whats happening and where the main bug with bundling lies and it's not the dependency version.
See this section of code in the config-serializer with special note to cfg[key] = systemCfg[key]; given the new config has two calls to System.config and two packages entries we end up overwriting the cfg[key]. We should be merging them (or handling them independently).

@EisenbergEffect @ahmedshuhel I've just realised I completely missed whats happening and where the main bug with bundling lies and it's not the dependency version.
See this section of code in the config-serializer with special note to cfg[key] = systemCfg[key]; given the new config has two calls to System.config and two packages entries we end up overwriting the first cfg.packages entry. We should be merging them or handling them independently.

Incidentally

Crap.. sorry I just pressed a load of incorrect buttons one after another. Here's the post I just deleted:

@EisenbergEffect @ahmedshuhel I've just realised I completely missed whats happening and where the main bug with bundling lies and it's not the dependency version.
See this section of code in the config-serializer with special note to cfg[key] = systemCfg[key]; given the new config has two calls to System.config and two packages entries we end up overwriting the first cfg.packages entry. We should be merging them or handling them independently.

@AdamWillden What's your bundle config looks like? It would be great if you could point me to a github repo with minimum code that reproduces this bug. However, I am trying to setup it up with skeleton-navigation and see if I can reproduce it. Thanks.

@ahmedshuhel I should be able to sort you out a repo and share a link on gitter but won't be till tonight at best (0830 UK time currently)

@AdamWillden Sure. That would be great.

@ahmedshuhel I tried doing this last night but came up against unrelated issues post-upgrade. I'm working on it and you'll have it as soon as I can (home-life permitting)

@ahmedshuhel I've just linked you a repro on gitter
npm install (this will do jspm install)
gulp build
gulp bundle
Notes:

In bundles.js having the depCache option enabled causes me issues on our CI server (if it caused issues locally I didn't notice). I'm having to run with depCache disabled
gulp unbundle will need fixing once bundle is fixed
I had to update the export-release task in the skeleton as the jspm.normalize function seemingly no longer returns the base path (e.g<EMAIL_ADDRESS>but the main

Any issues let me know and I'll be happy to help! The code I've linked is part of our internal company library, I'm pretty sure this skeleton doen't reference anything on our NPM enterprise server so you should be okay running npm install

:+1:

@AdamWillden Pushed a fix. Give it a try and let us know. Install with npm i aurelia-bundler@beta -S

@ahmedshuhel it still seems to be removing the same local package definition for me (tried with npm i ... --save-dev also)
Could this be a possibility?

the systemjs-builder dependency is old (0.15->0.16) and rejects the local package definition as invalid

It would be much nicer if the bundles section were simply added to one of the existing config calls rather than it rewriting the entire file with a new single config call.
Though I guess that may be at odds with the ability to pass multiple config files?

@ahmedshuhel I tried beta.3 and that worked to keep the required entry! :-) When i did the install the other day I only got beta.2.
I stand by my previous comment with regards to not rewriting the entire config with a single call, it'd be much nicer.

@ahmedshuhel yeah if I gulp bundle (which makes it a single config call), gulp unbundle (maintains the single config call) then jspm install some-package the file reverts to having two calls again. Which means the file is never consistent. Not a big deal but a bit 'untidy'. Feel free to keep this closed as the root issue is fixed but perhaps we should open an issue for a future enhancement?

@AdamWillden We are aware of that problem (bundler collapsing it into a single SystemJS.config({...}) call). Let's create a separate improvement ticket for that.

Cheer @ahmedshuhel and thank you very much for your work on this!


chore: download dialog code snippet

#447

Add dedent package to convert code from React nodes to strings
Adds class name for code box
Adds code snippets for dataset and download all annotations

Demos
Download dataset

Download tomogram

Download all annotations

Download annotation

For the tomogram download, do we have a screenshot for zarr file?
Similarly, for the single annotation download, do we have a screenshot for the zarr and ndjson files?
Otherwise these look good to me.

the code snippets look good to me.


Deprecate ILogger extensions in favor of Microsoft.Extensions.Logging.ApplicationInsights package

Fix Issue #817.
ApplicationInsights Logger is now implemented in Microsoft.Extensions.Logging.ApplicationInsights nuget package.
Please install the new package and configure logging using ApplicationInsightsLoggingBuilderExtensions.AddApplicationInsights overloads.
See examples in https://github.com/Microsoft/ApplicationInsights-dotnet-logging/pull/239

cc @RamjotSingh

Shouldn't the classes implementing ILogger be marked as Obselete too? Files under
https://github.com/Microsoft/ApplicationInsights-aspnetcore/blob/develop/src/Microsoft.ApplicationInsights.AspNetCore/Logging/Implementation

Those are internal. So no need to mark obsolete really.

Also we need a good strategy for the longer run (3.x). Are we going to ask users to install both packages or are we going to introduce a dependency?

Also we need a good strategy for the longer run (3.x). Are we going to ask users to install both packages or are we going to introduce a dependency?

@cijothomas  and @Dmitry-Matveev  decided to ask users to install both packages.
I think based on the feedback we may introduce a dependency.
We should document new logger (https://github.com/Microsoft/ApplicationInsights-dotnet-logging/issues/251) and consider changing VS templates.


Add a PublishClient and a PublishHttpClient implementation

Add a PublishClient to the Dapr.Client library to allow the use of the Publish API Endpoints from within the .NET client

This seems like an obvious addition, especially as there is already code to support handling subscriptions.

@darraghjones absolutely agree! I'm planning on working on it over this weekend

@amanbha not sure if you want to assign this to me, but I'm working on it now


Set node_from_env_var filter for k8sattributes

Addingnode_from_env_var filter for the k8sattributes processor, so the agent will query pods running on the same node only.
its reducing resource usage.

 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.


Added dapper for mapping SQL Response to Domain Entity

Updated Mapping between API and Domain Vacancy type to transform Enums where applicable

@dev4valtech can you move the IVacancyRepository into the Domain project - entity repos are typically defined in there and then realised in the infrastructure. This makes the repos available to all other areas of the code

@dev4valtech can you add a call in VacancyOrchestrator to log the fact that the vacancy has been requested. eg. in commitments we log such as:
_logger.Info($"Getting Provider Details, Provider: {providerId}");

is this ready for review again?


cmd/definitions: Implement AOS-40

Signed-off-by: Xuanwo<EMAIL_ADDRESS>

All service should update their usage of service metadata before this PR get merged.


GeoIP database broken?

Most (all?) of the GCP IPs used in prod only resolve to lat, lng pairs for the entire US.
for example:
http://www.google.com/maps/place/38.0,-97.0

However, other sites correctly report as South Carolina:

This causes incorrect Relays to be picked for clients and Gateways.

Confirmed that it's only for some cloud IP blocks


Investigate MAUI support to make VGraph work across multiple platforms

I may be able to port VGraph using MAUI. Not sure how easy or difficult that would be, since the GUI is pretty much married to WPF (and to a lesser degree, Windows Forms), but it's worth looking at if I ever get bored. Alternatively, I may be able to rewrite the front-end using Xamarin or something.

I have looked into making a MAUI version of VGraph. Unfortunately, MAUI lacks some fundamental features that I would need to account for. Most notably, MAUI does not have dedicated mouse and keyboard support at this time. The only method of input I have is tapping on the screen, which can be done with either a cursor or my fingers on a mobile device. It would be possible to invent a new control scheme, but I cannot imagine using this application without keyboard shortcuts.

Looking at the MAUI repository, this now merits another look - It appears that some mouse and keyboard support may have been added to MAUI.


How to measure real size of the created face mesh in .STL file?

Hello @heinrisch ,
I have used this demo and able to export .STL file. But when I import that file in Blender or Autodesk MeshMixer  , size/dimension of the the face mesh are something like ;
x: 0.153 mm
y: 0.104 mm
z: 0.191 mm
So I want to know are these dimensions are of real face? Or I need to convert them ?
I am stuck on this, any support or guidance is highly appreciated.
Thank you.

just multiply with 1000 and you have the correct dimensions.
From your dimensions I would say you are caucasian, male, mid 30's, and you have strong chin bones ;)

Verified. Indeed it will be 1000 for the scale. This was done with very fast measuring.


cranelift-frontend: Support moving GCs with user stack maps

This refactors the way that user stack maps work such that they are compatible with moving GCs. We now keep every GC value in a stack slot, spill to the stack slot on the value's definition, and reload from the stack slot on every use. This means that we can generate a bunch of redundant loads, but we let the mid-end's alias analysis clean that up. This has the added benefit of reducing the length of live ranges for GC values and also means we don't need to proactively spill every single live GC values at each safepoint, because we know they are already spilled. This latter bit actually helps us avoid an accidentally quadratic issue with many, long, overlapping live ranges where we would do O(n^2) spills for a series of safepoints that keep creating more GC refs.
We also implement two further optimizations:

We lazily allocate slots for live GC values, which means that if a GC value is not ever live across a safepoint, then we do never allocate a stack slot for it and never spill it to the stack slot. If we didn't do this, then frames would be larger and we'd have a dead store to the stack slot that would otherwise require the mid-end to grow a dead-store-elimination pass.

We keep a free list of available slots that will never be used again, and we reuse slots from here when possible. This means that a chain of non-overlapping live ranges, each of which still needs to appear in some safepoint's stack map, will all reuse the same stack slot, keeping frames from being bloated.

Finally, this commit also introduces some logs for the liveness analysis to ease future debugging.

operate closer to the level of the frontend's "variables"

I also thought about creating a new entity for these things but I thought through how it would be used in cranelift-wasm and wasmtime-cranelift and my conclusion was that it would be a bit of a mess. First, we aren't just dealing with wasm locals, which are Variables; we are also dealing with the results of calls and things like that, which produce Values. So we have to have a solution for Values either way. Second, cranelift-wasm doesn't make the decision whether certain types are managed by the GC or not, it is agnostic of runtime structures and implementation, wasmtime-cranelift makes those decisions based on its Wasmtime-specific knowledge. That means that cranelift-wasm would have to have some kind of enum of either a GC-managed Variable (or other new entity) or a plain Value and use that for its abstract operand stack, at which point the whole crate, essentially, is infected with this change and things are getting really hairy.
For this reason, we decided it was better, at least for now, to play nice with the existing system and its existing entities, rather than overhaul everything.

I think that might point the way to an implementation solution that also avoids the need for the extra pass: branch in the variable-to-SSA-value translation to an alternate "value just lives on stack" path.

Unfortunately this won't work because the stack slots and liveness and all that are not created until we are finishing the function builder, long after calls to builder.use_var.
Trevor and I talked about another solution for this specific problem though: if we maintain a backwards map from Value to the Variable that it is associated with (if any) then we can force all Values for the same Variable to use the same stack slot. For Values that are not associated with a Variable, they would use the same approach we have in this PR. But this is work for the future.

I think we can land as-is if you prefer

I'm going to go ahead and enqueue this for merging, but I am still interested in your responses above and in the inline thread!


[fix] plenty PRs

#633

What does this change?

@indexzero if it's good why wasn't it merged in at any point??


Fixed crash #861: LatexCiteBeforePeriod quickfix at the end of a file.

Fixes #861.
Changes

Quick fix no longer tries to access non-existent characters with index larger than the document length.

Can reproduce bug and fix. Don't know how you found it but good job :)

Separate section contents to a new file and writing a citation at the end wrongly. Pretty easy :)


Implement new functions and few improvements for supervisor

Code/Docs-wise LGTM, but let's wait for @pcarranzav to confirm the API calls to device.

LGTM


[http2] Dont drop connections on metadata limit exceeded

If we get an error on a stream, all we need to do is cancel that stream.
If instead we get an error not associated with a stream, we should close the transport.
Current behavior was to just close the transport always, and this was a bad bug.

@ctiller Hi.
I want to know if this PR can fix CVE-2023-32732/CVE-2023-32731?

Folks coming here for CVE-2023-32731: An error was made in the original CVE text; this change introduced the bug.
The fix is at https://github.com/grpc/grpc/pull/33005.
We're in the process of updating the CVE text.

Thanks! One more question, is CVE-2023-32732 also fixed by 33005?

One more question, does this PR #32309 introduce CVE-2023-32731 and CVE-2023-32732?

Correct on both.

Okay. Thanks!

@ctiller Sorry, but I'm a little confused.
https://nvd.nist.gov/vuln/detail/CVE-2023-32732 says:
"""
We recommend upgrading beyond the commit in  https://github.com/grpc/grpc/pull/32309
"""
But looking at the above comments, it seems that PR#32309 is not a fix, instead, it's the cause of CVE-2023-32732.
Which one is correct?

@ctiller Hi .
I want to know how to reproduce the issue. I tried using Nginx as an HTTP/2 proxy and passed an HTTP header with incorrect Base64 encoding, but I couldn't reproduce the issue related to CVE 32732. All the functionality worked fine and the data was correctly transmitted to the server.
context.AddMetadata("my-bin", "4rdHFh%2BHYoS8oLdVvbUzEVqB8Lvm7kSPnuwF0AAABYQ%3D");


Should we use Action or EventCallback for binding?

In child component example in "Data binding" section you have this code:
@functions {
    [Parameter]
    private int Year { get; set; }
    [Parameter]
    private Action<int> YearChanged { get; set; }
}

Near the end of "Event handling" section we can read:

Use EventCallback and EventCallback<T> for event handling and binding component parameters.

So the question is: which type should we use for binding?

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

ID: be4c4d0c-149c-bde1-c7fa-f082916390ca
Version Independent ID: 9d945996-ad02-bd4b-58ea-c465cc5cc74c
Content: Create and use Razor components
Content Source: aspnetcore/blazor/components.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

It's a little further down, but I admit a bit cryptic when it comes to specifics:

Prefer the strongly typed EventCallback<T>, which provides better error feedback to users of the component. Similar to other UI event handlers, specifying the event parameter is optional. Use EventCallback when there's no value passed to the callback.

The last sentence makes sense.
The part that I find a hair cryptic is the "better error feedback" bit. It leaves me wondering what kind of improved feedback is provided and for what types of errors? I imagine that we can be more specific there.
I'll mark this for engineering to take a look and respond. No ETA on a response, but we should get an answer soon-ish.

... oh ... good :eye: ... let me work these on the current open PR.

@guardrex it looks that you are confused with my question

which type should we use for binding?

which is written immediately below this text

Use EventCallback and EventCallback<T> for event handling and binding component parameters.

I'm sorry, it is really confusing. The real and most important question is in the title of this issue:

Should we use Action<T> or EventCallback<T> for binding?

and I see that you have already made a correction.
BTW EventCallback is really well described in this issue: https://github.com/aspnet/AspNetCore/issues/6351

@Andrzej-W It seems the PR that I put in almost completely resolved this. Engineering (and docs) are 100% :+1: with EventCallback coverage now.

Action<T> is outta there! ⚾ That's updated to EventCallback.
For EventCallback versus EventCallback<T>, we have:

EventCallback<T> is strongly typed and requires a specific argument type. EventCallback is weakly typed and allows any argument type.

Use EventCallback when there's no value passed to the callback.

We still have that generic remark that "EventCallback<T> ... provides better error feedback to users of the component." However, that's a small, low priority thing to address. I'm 🏃 at the moment. 😅
Is there anything else that requires attention?

@guardrex In Event handling section, EventCallback subsection below second example we have this sentence:

When the button is selected in the Child component:

I think that selected should be changed to clicked. Selected in case of button for me means button got focus but here button have to be clicked (or tapped on touchscreen).
A few lines below we have this sentence:

Don't use EventCallback and EventCallback for child content—continue to use RenderFragment and RenderFragment for child content.

I have to say that I don't understand this sentence at all. It suggests that EventCallback has something in common with RenderFragment and people can confuse them. Maybe someone was thinking about similarities between EventCallback and Action but of course we don't use them for child content. Maybe we should ask someone for a good example when to use Action instead of EventCallback and then rephrase this sentence or remove it completely.

I think that selected should be changed to clicked. Selected in case of button for me means button got focus but here button have to be clicked (or tapped on touchscreen).

Can't change that ... Microsoft documentation guidelines call for "selected." We have to say "focused" (for focused) and "selected" (for clicked).

Don't use EventCallback and EventCallback for child content—continue to use RenderFragment and RenderFragment for child content.

@danroth27, do we need this retain this remark?

Don't use EventCallback and EventCallback for child content—continue to use RenderFragment and RenderFragment for child content.

@danroth27, do we need this retain this remark

I think this sentence can be removed, especially if it's causing more confusion than good


Remove need for local docker

If docker is not configured kompose k8s convert will fail.
Try to not fail if docker is not there (because potentially not needed).

I don't think so. I tried on my machine without docker service, it still works.

This issue was solved on kompose2


Change collector response to be gRPC client retries friendly

Change the collector response status code to be gRPC client retries friendly.

The expected retryable status codes for the client are [UNKNOWN, UNAVAILABLE]
Instead of sending a failure message with status code OK, send the status code UNAVAILABLE as a response.

Compatibility with old agents needs to be checked
Change response only when the client request includes the gRPC metadata entry retry-friendly-response grpc.built-in.retry with a value of true

Change header key for collector response to grpc.built-in.retry


.sphinx: Add goroutines and uptime to wordlist

Signed-off-by: Thomas Hipp<EMAIL_ADDRESS>

Will let it up to @ru-fu whether it's appropriate to add those two or if they should instead be quoted.


fix: wait between multi-target firmware updates

fixes: #5081
Turns out the only thing that prevented multi-target firmware updates from working was actually waiting between the updates, not just logging it...

@zwave-js-bot automerge


Is it possible to add support for exclusive controller access?

Is it possible to get exclusive access to add a feature that gives python exclusive access to a controller?
What I mean is that, for example DS4Windows can get exclusive access to a controller, so no other program can use said controller. Is it possible to add something similair to this library?

Hmm, it should be possible I believe. I'll check that and get back to you.
Although I must ask out of curiosity, why do you want such support ?

It’s for a project that I am working on that requires the program to have exclusive access to said controller, because else the game where it is for will splitscreen
It’s a sort of challenge

Oh, that seems like an interesting challenge. I'd be up for it.
You could really help me out by

providing sample code that demonstrates your use-case
mentioning how this library currently behaves with respect to that sample code
telling me how you think this library should behave with that sample code.

Rest assured, I'll do some research into how I can cater to your use-case, and if at all.
I'm currently closing this issue, please feel free to open it again if you get hold of any of the above resources. Else I'll reopen it once I have a PR to link with it.
P.S. Thank You for taking interest in this library, it made my day :)

Hey, thank you for the clarification. I'll certainly look into it.


Easier access to, and more control of font data

Accessing multiple fonts is cumbersome in the current setup. Also, order of fonts is not guaranteed because of the nondeterministic way 11ty processes _data files:

The non deterministic order seems to be caused by 11ty. 11ty adds global data files to the global data file in an non-deterministic order. Even if it didn't, relying on ordering of object keys is not very safe. So best is probably to generate a list of fonts somewhere, where we control the order of that list.

@pascalw proposes this:
Generate src/fonts/meta.json and fill it:
{
  "fonts": [
    {
       "name": "font one",
       "selector": "font-one"
    },
    {
       "name": "font two",
       "selector": "font-two"
    },
  ]
}

Then we can avoid generating fonts.js, as this JSON can directly imported instead via Webpack, giving us access to the same data we need. An author can tweak the order of fonts by editing the meta.json file. We'd also be able to work with arrays instead of objects, further simplifying things.

This approach makes a lot of sense for me from an authoring perspective.

yarn fontdata now produces a metafile in which you can change the order or even append custom variables.


Use BlueZ 5 by default, update blueman

Now that we have https://github.com/NixOS/nixpkgs/pull/12683 merged, we should start building with BlueZ 5 by default. I've also updated blueman (the new version supports BlueZ 5) and fixed ObexFS. Tested with nox-review wip.

Merging this in few days.

@abbradar :+1:


Article display is suddenly wonky on boingboing.com

Boingboing.com has recently started displaying oddly -- title, then a large gap of white space a good 15+cm, and then the text.
I am sure the the underlying problem is their issue, not yours, nonetheless, it would be nice if you looked at it and would see if you can make it work better.
They also do not do the right thing when one clicks reader view, and haven't for ages. This is why I think they have a wonky web site. Nonetheless, NNW is my way to read them and it's annoying.

That gap is actually an image that you can't see. The problem is that the srcset attribute of the img tag is set with a base64 encoded data gif. As best I can tell, this is supposed to be a placeholder image for WordPress lazy loaded images. Since WordPress lazy loading doesn't work in NetNewsWire, it never gets replaced.
Other RSS readers that I tested also seem to have this problem.
The solution to this problem is for Boing Boing to remove the srcset attribute.

Addendum:
I contacted Boing Boing and got a reply:

You’ve actually solved the problem! We can fix it now!

Thank you for being the ones who actually solved the problem.

Thanks for the follow up!

And thank you for doing all the work. There's also funny stuff on their site about Reader View, that often does the wrong thing, as well. I've mentioned it to them.


Update reasonable testing suite

On my to-do list, not sure when I'll get to it:
The warn-scraper testing suite hasn't been performing real-world testing, in some ways, in years. Nor has warn-transformer. Nor bln-python-client.
Everything is locked around old versions of Python and libraries to support them. Python 3.12 went mainstream in the Ubuntu LTS last month; Python 3.13 goes live in a month. Nothing here is testing around 3.11.
In June I started to go through and identify some possible version upgrades; this is itself now a bit out of date.
https://docs.google.com/spreadsheets/d/1NSqxIhSeCNA4Oxx-5IW5lKFxBnFUyPGp0722eZk3xQY/edit?usp=sharing
Incorporated into this perhaps should be a holistic look at what actually makes sense here. This suite until yesterday was running a pypi test that wasn't actually needed (thanks, @palewire !)
@chriszs is suggesting several pieces of the testing suite could be substituted more efficiently with one, in https://github.com/biglocalnews/warn-scraper/issues/623
I've been running Python 3.12 with this cluster of programs for months now, flawlessly. But changes I run with newer versions never actually get tested with them. Testing suite should reflect real-world operating conditions.

If ruff is not adopted bandit should be considered; bandit should also be considered for a one-off improvement run.


2.0.0-rc.0 copies files from dependencies within pnpm monorepo

I have packages within a pnpm monorepo, and duel is copying files from dependency packages when compiling.
E.g. package1 depends on package2, files from package2 are copied into pacakge1 upon compilation.

Thank you for reporting this. If you don't mind providing a general outline of your project structure, that would help. You can use an online tool like https://tree.nathanfriend.io/ to help with that.

@gsong I've released v2.0.0-rc.1 which should fix this issue.

@gsong I've released v2.0.0-rc.1 which should fix this issue.

I’ll test it out soon and let you know.

@gsong I've released v2.0.0-rc.1 which should fix this issue.

Can confirm this fixed my issue. Thank you!


links check: getting 429 too many requests on CI

https://circleci.com/gh/iterative/dvc.org/2626?utm_campaign=workflow-failed&utm_medium=email&utm_source=notification
@casperdcl should we ignore this and consider as 200?

Hmm looks like it. There's no easy way to avoid this.
Right now we process each file individually (makes it easy to associate errors with the correct file).
If instead we first grep links in all files, then process together we can avoid checking the same link multiple times - but it would be harder to associate errors with the originating file. Possible but a little bit of scripting involved.
At that stage I'd be tempted to create a separate repo for link checking and install it as a hook :)
For now maybe best to ignore 429?

yes, agreed - let's just ignore them


Create VIF with MAC

Hi,
I'm trying to create a VIF with a custom MAC address (since later is not possible to change it). But the mac address is silently discarded:
connection.vifs.create(server: vm_object, network: network_object, MAC: 'mac address')
connection.vifs.create(server: vm_object, network: network_object, mac: 'mac address')

Both will create a vif but with autogenerated MAC.

Is there a reason you want a specific MAC address?  You can query by VM name / uuid and return the generated one.  You just do:
server = connection.servers.find { |server| server.uuid == #{uuid} }
Mac = server.vifs.mac.first
Sent from my iPhone

On Mar 6, 2015, at 6:42 AM, Francisco Miguel Biete<EMAIL_ADDRESS>wrote:
Hi,
I'm trying to create a VIF with a custom MAC address (since later is not possible to change it). But the mac address is silently discarded:
connection.vifs.create(server: vm_object, network: network_object, MAC: 'mac address')
connection.vifs.create(server: vm_object, network: network_object, mac: 'mac address')
Both will create a vif but with autogenerated MAC.

Reply to this email directly or view it on GitHub.

Yes, I need to use a specific MAC because I will be using a static DHCP server, and the MAC will tell some attributes (DNS, gateway...).
For example if the MAC starts with 02:01 thats CPD 02 client 01, use their config.

@fmbiete I'm not sure the reason behind this. When using fog i had this same behavior so i guess it was just extracted as well. Perhaps you have time to dig it further?

The problem was that I only had fog in my Gemfile, adding fog-xenserver fixes the xenserver support in fog, and the MAC is added at creation time.
Sorry for this.

@fmbiete. Awesome to know. Thanks


Translated document won't save and throws error

If a document has a hasOne relation that is set to translate, then it will not be translated. Said document will not be able to save and will throw the following error in the console:
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'data')
    at qc (main.96060888.js:13200:3220)
    at content-manager.9fb36cc0.chunk.js:9856:140
    at Generator.next (<anonymous>)
    at fulfilled (content-manager.9fb36cc0.chunk.js:9534:24)

Here's a reproduction repo: https://github.com/anatolykopyl/plugin-translate-reproduction
https://user-images.githubusercontent.com/33553182/214669996-d92daff7-2403-4275-9b5a-f331f3306d66.mp4

Probably connected to #74 although the error is different

This issue exists in latest version 1.1.5 @fekide-bot


prisma2 generate requires POSTGRESQL_URL (or any env var)

given this schema.prisma:
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "postgresql"
  url      = env("POSTGRESQL_URL")
}

model SomeModel {
  id        Int       @id
}

willl throw error: Environment variable not found: POSTGRESQL_URL. if POSTGRESQL_URL is not defined. This is of course not a problem on local development, but if you do a prisma2 generate during build-time in a CI/CD-environment, you probably don't know the POSTGRESQL_URL and  prisma2 generate  does actually not require that environment variable to run successfully. In fact as a workaroud you can pass some random string or empty string as POSTGRESQL_URL and it will work.
I have not a working CI/CD setup yet, so i am not sure if having env() in the schema does work both on build and runtime?

duplicate of https://github.com/prisma/prisma2/issues/800


I have a module as follows
define(function (
					require
				){
						var tk=require("tk");
						var bb=require("bb");

						var ba = tk.App.extend({
							onStart: function(options){
								this.showView(options.view);
								bb.history.start();
								},
							onBeforeStart: function(options){
								this.model = this.options.data;
								},
							onBeforeStop: function(options){

								},
							onStop: function(options){

								}
						});

						return new ba();
				},
		function(
					err
				){
					//The errback, error callback
					//The error has a list of modules that failed
					var failedId = err.requireModules && err.requireModules[0];

					console.log("In err "+failedId);
						console.log(err);
				}
	);`

the view is as follows
define(
	          function (
					require
				){

					var Mn = require('mt');
					var aview = Mn.View.extend({

					el:'.layout' ,
					template: false,

					return new aview();
				}
	);

I have got the following error
Uncaught TypeError: Cannot read property 'apply' of undefined
at child.showView (backbone.marionette.js:3309)
at child.onStart (bap.js:9)
at child.triggerMethod$1 [as triggerMethod] (backbone.marionette.js:159)
at child.triggerStart (marionette.toolkit.js:865)
at child.start (marionette.toolkit.js:810)
at main.js:57
at Object.execCb (require.js:1693)
at Module.check (require.js:881)
at Module. (require.js:1136)
at require.js:134
Chrome point me at
  showView: function showView(view) {
    var region = this.getRegion();
    for (var _len = arguments.length, args = Array(_len > 1 ? _len - 1 : 0), _key = 1; _key < _len; _key++) {
      args[_key - 1] = arguments[_key];
    }
    return region.show.apply(region, [view].concat(args));

in backbone.marionette.js
Please help me to fix the problem.
Zazakallah khair for reading.
Allah-hafiz
Nadvi

showView on a Toolkit.App is calling showView on Marionette.Application  That function needs a defined region on the App.
https://github.com/marionettejs/backbone.marionette/blob/master/docs/marionette.application.md#root-layout
So your code will need something like:
var ba = tk.App.extend({
  region: '.container',
  onStart: function(options) {
    this.showView(options.view);
  }

which will put the view inside a

However your view is setting the el to .layout which will attempt to attach the view to DOM that is already rendered on the page.
I assume that is still the case here in which case you may want to try:
var ba = tk.App.extend({
  region: '.layout',
  onStart: function(options) {
    this.showView(options.view, { replaceElement: true });
  }

But.. if you can add a single wrapping
 for a starting container in your HTML that is an easier setup than making your layout the top DOM element.

Zazakallah khair for quick answer my issue.
But I get confused by this in another issue.Please pardon me .I have a code as follows
jQuery(function($) {
	var container = $('.layout');

	function layout() {
	       container.layout({
	                 resize: false,
		         type: 'border',
		          vgap: 1,
		          hgap: 1
		});
	};
       $(window).resize(layout);
});

i.e when the top window is resized the layout() will fire.My idea is to attach $(window) with the application and container with a view. Is it possible? Please give me a solution.
Zazakallah khair for reading.
Allah-hafiz
Nadvi


GUI: allow filtering out certain predicates, like ConceptNet /r/RelatedTo

I expect this mostly applies to the NodePage.


Remove Microsoft.DotNet.Watcher.Core package

The shared watch code isn't an API we really want to support. This removes the Microsoft.DotNet.Watcher.Core and merges code into once single ".Tools" project.
History: in the days of DNX, there was a plan to share code between dnx-watch and Visual Studio tooling to watch the system. We split the package into two pieces with this intention. Since then, dotnet-cli has replaced DNX and MSBuild will replace project.json.
cc @muratg @victorhurdugaci nothing in aspnet/Universe is using this package. Is there anyone we need to notify about this removal?

I like it, but isn't it a breaking change for 1.x?

We never RTMed the package. It was only a "preview" so I think we are clear.

As long as you don't remove the old packages, it shouldn't break.

As long as you don't remove the old packages, it shouldn't break.

Yeah, we'll leave it on nuget.org. @muratg do we plan to unlist deprecated, prerelease packages from nuget.org?

@natemcmaster There's a longer term plan to unlist the older packages on nuget.org. We'll eventually do this, up to RC2 for now, I think.


Clip plans to joint limits before displaying them to avoid float rounding issues

This option works and shows the augmented plan before it gets executed.
It does not account for joint limits being temporarily adjusted using the joint limit warning button in the status bar.
It's turned off by default.
This fix could live in the translator as well. Please discuss.

Rather than testing the conditional self.clipFloat32SafeJointLimits inside the clipState and clipPlan functions, I think it would be better for the caller to test and conditionally call the functions.  The functions themselves should always have predictable behavior.
Also, I'm still not convinced this is clipping the visualized plan.  I think the plan playback panel gets plans from a different lcm subscriber and not the ikplanner.  Am I wrong?

Note, the plan message that is clipped is also committed here:
https://github.com/oh-dev/oh-distro-private/blob/a1ce7c9799204fc509d9ef7f055932af90da529d/software/modules/ohdirector/tableboxdemo.py
However, the visualized plan, and the plan that is committed if you click "Execute" in the plan playback panel, is a different plan message.  The plan message returned from ikPlanner is received by one lcm subscriber, but there is a different subscriber that receives plans and passes them to the plan playback panel.  We could fix this, so that there is exactly one subscriber that ever receives messages on the CANDIDATE_MANIP_PLAN channel, but currently this is not the case.


[backport] Add documents of cupy.all and cupy.any function

Backport of #511

jenkins, test this please.

LGTM!


Update workflow.yml

Added codecov step

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


DocsArchive-1.0.20323.1Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/MicrosoftDocs/globalization-docs-archive/pull/8 is merged into live branch.
Auto archive content to https://github.com/Khairunj/globalization-docs-archive.git

Docs Build status updates of commit c861940:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

globalization/index.yml
:white_check_mark:Succeeded
View

globalization/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

This was just a test PR and hence closing it.


restore: Close files tested with --verify

What is the purpose of this change? What does it change?
Before, when running restic restore --verify, the target files were opened once per blob and never closed, this commit fixes that.

Was the change discussed in an issue or in the forum before?
Nope.

Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Codecov Report

Merging #2044 into master will decrease coverage by 4.57%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2044      +/-   ##
==========================================
- Coverage   51.01%   46.44%   -4.58%
==========================================
  Files         176      172       -4
  Lines       14060    14032      -28
==========================================
- Hits         7173     6517     -656
- Misses       5842     6536     +694
+ Partials     1045      979      -66

Impacted Files
Coverage Δ

internal/restorer/restorer.go
42.28% <0%> (-1.25%)
:arrow_down:

internal/backend/b2/b2.go
0% <0%> (-80.69%)
:arrow_down:

internal/backend/swift/swift.go
0% <0%> (-78.45%)
:arrow_down:

internal/backend/gs/gs.go
0% <0%> (-73.61%)
:arrow_down:

internal/backend/azure/azure.go
0% <0%> (-69.46%)
:arrow_down:

internal/backend/swift/config.go
36.95% <0%> (-54.35%)
:arrow_down:

cmd/restic/cmd_mount.go
12.98% <0%> (-44.16%)
:arrow_down:

internal/archiver/file_saver.go
81.57% <0%> (-4.39%)
:arrow_down:

internal/restic/node.go
39.52% <0%> (-1.31%)
:arrow_down:

internal/backend/s3/s3.go
59.39% <0%> (-0.76%)
:arrow_down:

... and 4 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 4c2072d...c0fca3f. Read the comment docs.


Eddystone TLM

Added Eddystone TLM feature to the library.
Usage can be found in the examples folder.

@arun271 thank you for submitting!
However these changes cannot be merged, although they are functional, they do not meet the Eddystone specification.
When in TLM mode, the "Advertising PDU count" and "Time since power-on or reboot" will need to be updated as per https://github.com/google/eddystone/tree/master/eddystone-tlm#frame-specification.
This could be achieved in the EddystoneBeacon::loop() method, by calculating the PDU count and uptime via mills(). Then updating an internal TLM data buffer.

@sandeepmistry
Sir,Thanks a lot for the reviews.I have come up with few changes and tried to make the Eddystone TLM API easier to use.These changes include temperature,PDU count and uptime.
In Eddystone.cpp,added the following:
1.
uint8_t buffer[14]= {
0x20,
0x00,
0x00,
0x00,
0x00,
0x80,
0x00,
0x00,
0x00,
0x00,
0x00,
0x00,
0x00,
0x00
}; //global
unsigned char len = 14; //global
2.
void EddystoneBeacon::tlm_begin(){
this->_bleCharacteristic.setValue(buffer, len);
BLEPeripheral::begin();
this->_bleCharacteristic.broadcast();
}
3.
void EddystoneBeacon::tempLoop(int analogPin) {
this->poll();
pdu_count++;
/** temperature **/
pinMode(analogPin,INPUT);
int a = analogRead(analogPin);
resistance=(float)(1023-a)*10000/a; //get the resistance of the sensor;
temperature=1/(log(resistance/10000)/B+1/298.15)-273.15;//convert to temperature via datasheet
buffer_1[4] = ((float2fix(temperature))&0x0000FF00)>>8; //LSB
buffer_1[5] = (float2fix(temperature))&0x000000FF;  //MSB
//temperature
unsigned long val = millis()/100;
ulong2adv(buffer,10, val);  //UP-TIME SINCE POWER ON OR REBOOT
ulong2adv(buffer,6, pdu_count);  //PDU COUNT
_bleCharacteristic.setValue(buffer, len);
}
4.
void EddystoneBeacon::ulong2adv(uint8_t* adv, int off, unsigned long val) {
off+=3;
for (int i = 0; i < 4; ++i) {
uint8_t x = val & 0xff;
adv[off] = x;
val >>= 8;
off--;
}
}
5.Corresponding changes will be done for defining these new methods in Eddystone.h
6.The usage example is here : https://github.com/arun271/arduino-BLEPeripheral/blob/eddystone/examples/Eddystone/EddystoneTLM
Except for the battery-voltage data, the frame is complete I guess.Please review the changes and let me know what more can be added.
Thanks!

@sandeepmistry
In case above link doesn't open,I have attached the usage code below:
`// Import libraries (EddystoneBeacon depends on SPI)
#include <SPI.h>
#include <EddystoneBeacon.h>
// define pins (varies per shield/board)
//
//   Adafruit Bluefruit LE   10, 2, 9
//   Blend                    9, 8, UNUSED
//   Blend Micro              6, 7, 4
//   RBL BLE Shield           9, 8, UNUSED
#define EDDYSTONE_BEACON_REQ   9
#define EDDYSTONE_BEACON_RDY   8
#define EDDYSTONE_BEACON_RST   7
int analogPin = 0;
EddystoneBeacon eddystoneBeacon = EddystoneBeacon(EDDYSTONE_BEACON_REQ, EDDYSTONE_BEACON_RDY, EDDYSTONE_BEACON_RST);
void setup() {
pinMode(analogPin,INPUT);
Serial.begin(57600);
delay(1000);
eddystoneBeacon.tlm_begin();
}
void loop() {
eddystoneBeacon.tempLoop(analogPin);
}`

@arun271 overall great progress!
See my new comments, feel free to reply to them on Github before making the changes.

Closing this for now due to lack of activity.


[SUPPORT] The query results of the Hive and Spark columns are different

This patch is great, but there is one small problem，following：

when I delete record with id 5
hive query reslut：

spark query reslut：

flink query reslut is right

Originally posted by @mincwang in https://github.com/apache/hudi/issues/3703#issuecomment-978867541

spark reslut lost the   _hoodie_operation column

cc @garyli1019

hope to contain the  _hoodie_operation column for the spark sql query

@mincwang makes sense. tracking this in https://issues.apache.org/jira/browse/HUDI-2941

hi @mincwang You are using spark2's sql query, right?

hi @mincwang I put a pr for fix this. https://github.com/apache/hudi/pull/4649

hi @mincwang I put a pr for fix this. #4649

thanks a lot for your support


port service to golang

To avoid having too many frameworks in use for our team let's port this to golang and use it's internal http code.
See: https://golang.org/pkg/net/http/

Merged, closing this


load adapters from safetensor files (enhancement)

I tried loading my adapter after converting the pickle files into safetensors.
AutoAdapterModel.from_pretrained loads with safetensors (GOOD)
For example: model = AutoAdapterModel.from_pretrained("roberta-base", use_safetensors=True)
However when loading the adapter to the model, it is always looking for the pickle (bin) versions. (NOT SUPPORTED)
Moving forward, I think support for safetensors should be provided.
Afterwards, I uploaded the pickle versions and it did work with those.
For example load_adapters should be enhanced with:
model.load_adapter("solwol/my-awesome-adapter", source="hf", set_active=True, use_safetensors=True)

I mentioned the use of use_safetensors  in AutoAdapterModel.from_pretrained("roberta-base", use_safetensors=True).
But the flag is not required as models like roberta-base are converted into safetensors.
The issue with the load_adapter method is that it always looks for the pickle binaries to load the weights. When I put the pickle bins back, it downloaded all files including safetensors, but uses the pickle bins.
Moving forward the enhancement should work with no pickle files present but all binaries provided in safetensors. I hope this makes sense.

Support for safetensors is currently being added in #692, thanks for your suggestion!


Add an opt-in "typegen" parameter to .nodegenrc which allows us to use external type generators.

Adds a new .nodegenrc opt-in parameter called typegen which overrides the default QuickType schema-to-typescript generator. In case this parameter is not specified the previous code inside the generate-it core is triggered and the same old quicktype types are generated.
This will give us the possibility to have multiple internal or external libraries that generate types for generate-it. It is also the first step to decouple from quicktype (which doesn't see much love from its maintainers lately).
Example .nodegenrc follows:
{
   // ...
  "typegen": "./schema-to-typescript.js",
}

Example ./schema-to-typescript.js:
module.exports.default = async (
  name,
  schema,
  config
) => {
  const typeString = doSomeMagicToGetTypesFromSchema(schema);

  return {
    outputString: `export type ${name} = ${typeString};\n`
  };
};

The following extra changes were required for this to work:

The generation of parameters schemas objects was improved in a way that conforms with standards compliant schema-to-types generators (has no effect for QuickType as it still generate the previous types even with the slightly modified schema);
An extra type called "TypegenFunction" is now exported from generate-it which would allow us to have external modules for type generation in the future;
Unit tests were added where applicable;

The nodegenrc docs says how to inject a file, but it is not easy to know what the file should contain.
@zewish Could we get an example file in the readme docs?

@j-d-carmichael, example is given here in the issue - I can add it to the docs if you this this would be sufficient?

Yea that looks like it would be enough... else nobody is looking at this issue when it is closed, so that info will be lost :)

@j-d-carmichael, @p-mcgowan, docs improved, example added, including a link to the example heading on the config options page - should be way easier to comprehend now.

Thanks @zewish!


Add validation when loading resource template in Resource State Builder

In other places in the code, resource templates are validated on load.
Perhaps this should be moved up into getResourceTemplate()?

Now that resource templates loaded by resource state builder are added to state, this is important.


🐛 BASE_PATH configuration failing to load all resources when used with a basic reverse proxy setup

:bug: Bug report
Description

Passing the BASE_PATH environment variable per the customization docs does not work properly behind a reverse proxy with a standard path prefix rule. There are still some resources that are trying to be loaded from the root / instead of from $BASE_PATH/. Some resource are loading correctly (page background color and cookies consent pop-up load just fine).
Is this a regression?

I'm not sure. But I think this PR was supposed to "fix" it or was an attempt at dealing with this
:microscope: Minimal Reproduction
See Environment section for minimal docker-compose/ docker-swarm configuration to reproduce

:fire: Exception or Error

As you can see in the above screenshot, many resources load fine while using a basePath of player-1, however starting with runtime.js the application tells the browser to load it from root.

As you can see above, when I manually try to load runtime.js from /player-1/runtime.js it loads correctly.
:deciduous_tree: Your Environment

I want to be able to set up multiple juice-shop instances behind a single hostname, however the added BASE_PATH configuration does not seem implemented correctly. Here is a minimal docker-compose file that I am using with docker-swarm. I have omitted the traefik configuration here as it is not relevant.
# docker-compose.yaml

version: '3.8'

services:

  shop-1:
    image: bkimminich/juice-shop
    environment:
      - PORT=3001
      - BASE_PATH=/player-1
    deploy:
      labels:
        - traefik.enable=true
        - traefik.http.services.shop-1.loadbalancer.server.port=3001
        - traefik.http.services.shop-1.loadbalancer.sticky.cookie=true
        - traefik.http.routers.shop-1.rule=Host(`gameday.example.net`) && PathPrefix(`/player-1`)
        - traefik.http.routers.shop-1.service=shop-1
    networks:
      - traefik_network

networks:
  traefik_network:
    external: True
    name: traefik_network

Additional Information

This is not specific to Traefik or the reverse proxy or the web browser. Juice shop should correctly use the basePath config item for ALL relative links/resources. It looks like the main index.html or equivalent is telling the browser to look from  root / not taking the base_path into account perhaps?

Okay I just tried configuring Traefik to strip the pathPrefix before sending the request to the backend container, and everything seems to load correctly.
This is an odd implementation choice in my opinion. Typically you would only strip the base paths for backend containers that ARE NOT "aware" of the base path, but here we are "informing" the backend of the base path, except that its only used for redirects..
I think at least this quirk should be clearly mentioned around the documentation for the basePath. Maybe something like This application expects all incoming requests to be relative to /, regardless of the basePath setting ?

Did we just miss some root-based relative URLs during the intended fix or do you think the entire implementation is flawed?
As we're presumably running the full e2e suite successfully with subfolder setup, I thought everything is working. Could you help me pinpoint how our test setup would need to look like to reproduce the issues you had?

I don't think the entire implementation is flawed, and I personally don't have the time right now to dive into this right now unfortunately, so feel free to close this issue.
The docs DO in fact mention that the base_path variable is used for redirects, so maybe that's enough... I just think there needs to be a clear note that setting the base_path variable does not actually change the base path of the juice-shop web server at all (which makes the variable name very misleading)
An alternate suggestion could be to change the variable name to external_base_path or redirect_base_path instead, so that it's clear
It's a very minor nuance that I think should be clarified, is all.
Like I stated earlier I got it all working by informing the proxy to strip the same base_path value off of request URLs before sending the request to the juice-shop docker container (behind the proxy)

Ok, thanks! I've updated the documentation accordingly!


Remove types, add TypeGuesser to public API

@pwalsh this issue also can be closed, while it was in the feature/docs branch

DONE.


RLException: Invalid  tag: environment variable 'TURTLEBOT3_MODEL' is not set.

hyx@ubuntu:~$ roslaunch turtlebot3_teleop turtlebot3_teleop_key.launch
... logging to /home/hyx/.ros/log/360a6528-a592-11ee-9ef6-1b5675040354/roslaunch-ubuntu-5685.log
Checking log directory for disk usage. This may take a while.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.

RLException: Invalid <arg> tag: environment variable 'TURTLEBOT3_MODEL' is not set.

Arg xml is <arg name="model" default="$(env TURTLEBOT3_MODEL)" doc="model type [burger, waffle, waffle_pi]"/>
The traceback for the exception was written to the log file

打开主机上的~/.bashrc文件，添加或修改环境变量TURTLEBOT3_MODEL，即改为：
export TURTLEBOT3_MODEL=burger

打开主机上的~/.bashrc文件，添加或修改环境变量TURTLEBOT3_MODEL，即改为：
export TURTLEBOT3_MODEL=burger


Software requirements

Please clarify which versions of the software require which versions of PHP?  Do later versions require PHP7?

Thank you.  Are you sure, because I am finding return type declarations in the code of version 1.8.3, which I believe is a feature only compatible with PHP 7?

That's true. Where are this return type declarations? We removed the PHP 5.x tests in 1.8.3, so it could be the case.

I'm getting a PHP parse error in 'Oneup/UploaderBundle/DependencyInjection/Compiler/ControllerPass.ph
p' on line 10 - but I think that file may be dynamically generated by Symfony?

If you have this file (and this error), you'll have the 2.x version installed.

I've currently got v1.8.3 installed, but I installed the git source code because I'm hoping to make an amendment to the code.

1.8.3 has no ControllerPass.

Thanks.  Not sure how that happened, but reinstalled 1.8.2 and problem has gone away.


Change URL in package.json

Coverage remained the same at 91.068% when pulling b0ca8a85edabb878028f8c311a03344d6fe01a1f on jansav:packagejson into 062e99bce73baddf876713f4d76a44f1c1232a9c on Houston-Inc:master.


Document logging

We should document exactly what robot.logger is (it's only mentioned in passing in an example). Also we should document HUBOT_LOG_LEVEL.

Would like the evolution process to maybe consider an update to the underlying log package. Maybe using Winston, with Morgan for auto logging HTTP requests, e.g: http://tostring.it/2014/06/23/advanced-logging-with-nodejs/
But yeah, in the interim, docs would be good.

stalebot: shut up. This is so obviously still an issue.
Can someone seriously turn that thing off? It makes sense in high-traffic repositories but this one just... isn't.

This project is abandoned. I tried keeping stale bot away from a couple of my issues but it just keeps coming back and no one is reviewing them anyway. I've messaged the core team through slack and no one responds. The community is still supporting each other and some of the external contributors with limited access are still approving PRs from time to time, but there's no one at the reigns. There's some murmurs of waiting for direction from higher ups at GitHub, but it's very disappointing that they've not even tried to communicate with the community or respond to basic requests like turning off stalebot, which is just carpet bombing legitimate issues.

@timkinnane time for a community fork maybe? Idk

Indeed. I've been working on one, but I've refactored a lot without updating docs, so it's not ready for mainstream consumption. I'll email you.

Neat.

Hola de 10

Hey, activity: https://github.com/hubotio/hubot/pull/1581
This is a good idea. https://github.com/hubotio/hubot/blob/master/docs/scripting.md would be the place to include it.
robot.logger uses https://www.npmjs.com/package/log

Hubot logger is now an instance of pino


fix file circular reference

Signed-off-by: liuying<EMAIL_ADDRESS>
What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #27
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

/cc @moweiwei


WIP: New auth extensions

(It is very important that you created an Apache Knox JIRA for this change and that the PR title/commit message includes the Apache Knox JIRA ID!)
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. For instance: running automated unit/integration tests, manual tests. Please write down your test steps as detailed as possible)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Knox Contributing Process before opening a pull request.

Is there a JIRA for this?

Is there a JIRA for this?

I thought i had created one, looks like I did not.

These changes are handled separately in

#623
#625
#627


ingress-nginx appears twice in 'arkade install --help'

I played with arkade for the first time today and noticed this in the output of the aforementioned command:
  gitlab                  Install GitLab
  grafana                 Install grafana
  info                    Find info about a Kubernetes app
  ingress-nginx           Install ingress-nginx
  ingress-nginx           Install ingress-nginx
  inlets-operator         Install inlets-operator
  istio                   Install istio

Just a small detail but I thought I could report anyway.

@nitishkumar71 would you like to take this?

Sure @alexellis .
Thanks @antoineco for raising it.

@nitishkumar71 @alexellis I couldn't assign myself but just sent a PR ☝  Feel free to close it if that work has already been done.


feat: Added about page

Description

i renamed the contact layout to alternate layout so i can also use it in the the about page
i created the about page based on the design provided
added the about content & about page as stories
made the about page responsive
i also changed the href mailto in the contact page

Related Issue

#39
Motivation and Context

#39
How Has This Been Tested?

using eslint
Screenshots (if appropriate):

Nice job, @dev-phantom clap
Just a couple of questions (just asking, you're not suppose to make changes sweat_smile ) and suggestions:

oh nice catch i'll try to fix that in the next commit
thanks for that

@chadstewart can i get a review sir
so i can open another PR

All these  issues has been resolved in my next PR

All these issues has been resolved in my next PR

Hey @dev-phantom, I really appreciate the effort your putting into fixing issues and putting together good PRs but it would be better if you kept the work you've done in the PR you've created.
When you create multiple PRs on the same issue, it fragments the conversation across multiple PRs and makes it harder to learn the context of this change when people are looking through the project's history. I'd say avoid closing PRs in general unless the work is no longer needed or the scope of the PR has drastically changed.

Noted sir
Never thought of this
Sorry about that sir


mx.sym.WarpCTC cuda memcpy or memset failed issue

Environment info
Operating System: Ubuntu 14.04
GPU: GTX 1080
Compiler:
Package used (Python/R/Scala/Julia): Python
MXNet version: 0.9.5 in python
Or if installed from source:  git clone https://github.com/dmlc/mxnet.git ~/mxnet --recursive
Error Message:
[16:36:11] src/operator/././cudnn_algoreg-inl.h:65: Running performance tests to find the best convolution algorithm, this can take a while... (setting env variable MXNET_CUDNN_AUTOTUNE_DEFAULT to 0 to disable)
[    INFO][2017/05/05 16:36:25.768] ---------train---------
terminate called after throwing an instance of 'std::runtime_error'
what():  Error: compute_ctc_loss, stat = cuda memcpy or memset failed
Minimum reproducible example
cd ~
git clone https://github.com/baidu-research/warp-ctc
cd warp-ctc
mkdir build
cd build
cmake ..
make
sudo make install
git clone https://github.com/dmlc/mxnet.git ~/mxnet --recursive
cd ~/mxnet
cp make/config.mk .
modify the config.mk as following:
USE_BLAS = openblas
USE_CUDA = 1
USE_CUDA_PATH = /usr/local/cuda
USE_CUDNN = 1
WARPCTC_PATH = /home/nd/warp-ctc   (which my wrap ctc installed)
MXNET_PLUGINS += plugin/warpctc/warpctc.mk
CUDA_ARCH := -gencode arch=compute_30,code=sm_30
-gencode arch=compute_35,code=sm_35
-gencode arch=compute_50,code=sm_50
-gencode arch=compute_60,code=sm_60
-gencode arch=compute_61,code=sm_61
-gencode arch=compute_61,code=compute_61
make
cd python
sudo python setup.py install
Steps to reproduce
python main.py --python main.py --configfile default.cfg
the ctc layer code:
net = mx.sym.WarpCTC(data=net, label=label, label_length=num_label, input_length=seq_len)
How can I fix it?
Many thanks
Xin.q.

@sbodenstein Could you look into this?
We can

revert the ctc_loss pr
disable ctc_loss when using WarpCTC
fix the incompatibility.

Looking at this now. I think option 2 or 3 should be aimed at.

@piiswrong: I'm having difficulties trying to reproduce this due to #6032 (and I'm using a Mac).

Ok then could you disable ctc_loss when using warp ctc plugin first? We don't want to break existing features.
Also could you rename ctc_loss to _contrib_ctc_loss? so that it goes into mx.sym.contrib.

Sure, I will do those things.
For disabling: how would you recommend I do this?

I'm actually not sure.
One way I can think of is making this another plugin. Otherwise the .cc files will always be compiled unless you guard everyone with macros.
I think the problem here is you have two implementations of the same function (one from original baiductc.so, one from absorbed code). One solution I can think of is to wrap all absorbed ctc code in a different namespace.

I do think it shouldn't be too hard to fix (eg with namespaces). But its annoying that I can't build MXNet on GPU at the moment due to the OSX bug.

you can workaround it temporarily by deleting quantize/dequantize ops under contrib

@piiswrong
Is it OK to use the version of v0.9.3 Official Release to run the example correctly?
how can I using the source code to setup mxnet? I found there are folds missing when I use the source code to install the mxnet. While installing with git clone, the latest version is gotten.
Many thanks
xin.q.

@xinq2016, @piiswrong: I can finally reproduce this problem. I have confirmed that this example works when you delete all the new ctc_loss files (the folder ctc_include, ctc_loss.cc, ctc_loss.cu and ctc_loss-inl.h), but breaks when you include them.

@sbodenstein
OK, thanks a lot. I will have a try right now.
Many thanks

I found the same problem.  @sbodenstein or @xinq2016, can you provide more details on how to fix this?

@sbodenstein I delete all ctc_include, ctc_loss.cc, ctc_loss.cu and ctc_loss-inl.h below  ~/mxnet/src/operator/contrib but i still get the error message:
terminate called after throwing an instance of 'std::runtime_error'
 what():  Error: compute_ctc_loss, stat = cuda memcpy or memset failed

@KeyKy: did you do make clean, and then make? Also: I should have fix for this by the end of the weekend.

@KeyKy sbodenstein is right，It works now，thank you

Fixed yet? I got the same error:
terminate called after throwing an instance of 'std::runtime_error'
what(): Error: compute_ctc_loss, stat = cuda memcpy or memset failed

Fixed yet? I got the same error with mxnet 0.10.1

This doesn't appear to be fixed, it does not work for me either.

@jcftang: this is strange. Its fixed for some (like myself), and others not.
Could you give info about the GPU you are using?

@sbodenstein I have nvidia 1080Ti's (founder's edition)


running shell scripts fails on Sequoia macOS

Latest macOS update, Sequoia (15.0) broke running shell scripts using mirrord. Specifically, it looks like it broke the SIP detection somehow.
To reproduce create a script called fun.sh:
#/bin/sh
curl http://ip-info

then
on
MIRRORD_PROGRESS_MODE=off RUST_LOG=mirrord=trace mirrord exec -t deployment/ip-visit-counter -- ./fun.sh
you get
2024-09-27T15:23:39.388168Z TRACE start: mirrord_sip::main: No SIP detected on "./sha.sh"
* Could not resolve host: ip-info
* Closing connection
curl: (6) Could not resolve host: ip-info

it looks like currently SIP detection for scripts like fun.sh relies on a shebang being present - a step should be added to the SIP check to determine if the default interpreter is SIP protected when the binary to be run is a script without a shebang