NumberPlane and/or other objects not positioned at the center of screen since 0.6.0

Description of bug / unexpected behavior
A circle or rectangle object does not seem to be positioned at the center in NumberPlace since 0.6.0 release.
Expected behavior
Both NumberPlane and Circle (or other Mobject) should be aligned at the center of the screen by default
How to reproduce the issue
class TestCentering(Scene):
    def construct(self):
        plane = NumberPlane(axis_config={"include_numbers": True})
        circle = Circle(radius=1.)
        self.add(plane, circle)

Additional media files
This is the image generated by 0.5.0 (Expected Behavior)

This is the image generated by 0.6.0 and 0.7.0 versions (Not Expected Behavior)

Logs
System specifications

OS (with version, e.g Windows 10 v2004 or macOS 10.15 (Catalina)): macOS 11.3.1
RAM:
Python version (python/py/python3 --version):3.7
Installed modules (provide output from pip list):

LaTeX details

LaTeX distribution (e.g. TeX Live 2020):
Installed LaTeX packages:

FFMPEG
Output of ffmpeg -version:
PASTE HERE

Additional comments

The issue seems to be that, when centering the plane mobject at some point during the creation process, the numbers are also included and the overall plane is skewed. The same problem exists for Axes where the lines are shifted away from the center. The fix would be to have the lines of the plane handled seperately and centered without including the numbers. I've gotten this solution to work and I'll PR it when it's more polished


[FLINK-9091][build] Dependency convergence run against dependency-red…

What is the purpose of the change
As outlined in the JIRA the enforcer-plugin may not always work against dependency-reduced poms leading to convergence failures. This heavily conflicts with our approach to shading dependencies.
This PR ensures that the dependency-convergence , at least on travis, is run against dependency-reduced poms.
The easiest way to achieve this is to run the enforcer plugin separately for each leaf module, for which maven works against the local repository, and thus dependency-reduced poms.
Naturally this cannot be nicely modeled into the maven life-cycle, hence we rely on a bash script to iterate over all modules.
To prevent other devs from running into this problem the enforcer plugin is now disabled by default. It must be explicitly enabled with a profile. To check the convergence locally one may use tools/check_dependency_convergence.sh.
This PR subsumes #6073.
Brief change log

disable enforcer by default
add profile to enable enforcer
add tools/check_dependency_convergence.sh script to run convergence check against dependency-reduced poms
modify travis scripts to run convergence check after compilation

Verifying this change

add a dependency-convergence violation and run on travis; optionally run script locally

Thanks for the update @zentol. So do we want to support this script also on MacOS or not? Otherwise it is good to merge. What is your opinion about the static MODULES_CONNECTORS and MODULES_LIBRARIES should we create a separate issue for it?

We should make sure it runs on mac, I just haven't had the time to search for a cross-platform solution.
I also wouldn't be able to verify that it actually runs on mac, so someone else may have to do that.
I'm rather hesitant about generating the lists automatically. We could only generate part of them anyway (since contrib is split among them) and partially generating them is messy. What I would prefer is failing the build if a library/connector doesn't show up in the list to create more awareness for the whole thing.


Property 'resourceUsage' does not exist on type 'Process'

Bug Report
Get Property 'resourceUsage' does not exist on type 'Process'. when running process.resourceUsage().
Also get property does not exist when doing process.memoryUsage.rss();.
🔎 Search Terms
memoryUsage, rss
🕗 Version & Regression Information
node version: 15.11.0
typescript version: 4.2.3
Please keep and fill in the line that best applies:

This is a crash

💻 Code
console.log(process.resourceUsage())
🙁 Actual behavior
🙂 Expected behavior
Documented (here)[https://nodejs.org/api/process.html#process_process_resourceusage] , in the nodejs docs

Typing for Node.js come from @types/node and not from TS itself.
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/1e7c3528f8f66824c99c82be040418f66a17c49d/types/node/process.d.ts#L300
What version do you have installed?

my package.json says my @types/node version is 14.14.33

Looks like installing latest fixed


Update Gallery lockfiles for the new version of the video_player plugin

The video_player plugin was rolled in https://github.com/flutter/flutter/pull/139203 and the lockfiles need to be updated to match

Can you share the command you used to generate the lockfile? There is a script and there is a way to manually generate lockfles with gradle and I am surprised to see a different number and name of lockfiles in this pr.

I used gradlew assemble --write-locks


Parsing of AST literals must return errors

Existing implementation of graphql-go/graphql return a nil value, when it fails to parse AST literal (e.g. DateTime). This type of error suppression leads to confusion and hides errors in the code.

Migrated to vektah/gqlparser. Eventually closed by #95. Consistent encoding of Go types to JSON (graphql) is still a subject of future improvements.


perf(plugin-back-to-top): optimize backToTop feature

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Provide a description in this PR that addresses what the PR is solving. If this PR is going to solve an existing issue, please reference the issue (e.g. close #123).

What is the purpose of this pull request?

[ ] Bug fix
[x] New feature
[ ] Other

Description

Adjust the backToTop button, add scrolling progress percentage text, and optimize the interaction effect.
Screenshots

Before

After

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.304%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
698

Relevant Lines:
1255

💛  - Coveralls

I'm not pretty sure about your contribution. I already have the percent computed defing the scroll precent,  Why should and other similar variable is introduced?
Also there is one more thing to be discussed,  In what situation shall people care about the actual and exact percentage that a page has been scrolled?
A progress bar around the icon should have been enough for this situation.


Would like to implement Fill As Default button for Schedule sheet Duration and Start Offset

Duration: START and END will be set to 0, others will be 1.
Start Offset: START, END  and tasks which depends on START will be set to 0, other will be 1.

Issue was properly resolved.


Implement suggestions and some fixes

This PR adds two suggestions as well as some bug fixes and updates.
Player Notes
I've added an option for moderators to add notes to a player's profile. Only mods with the EDIT_PLAYERS perm can edit and view them. A note will also display the mod that adds/edits it as well as the timestamp.

Profile without any notes added

Image of the dialog to add or update a note

Profile with an example note

Unban Date on Profile
Moderators, as well as the user, will now be able to see the unban date on a player's profile.

Bug fixes

Fixed an issue with the notifications component in the navbar where it would send two api requests instead of one
Made the "Mark As Read / Unread" dropdown in the notifications page close after clicking on the option. Previously, it wouldn't close, and it was easy to spam press the button.

Add Notifications To Jobs

Previously, no notification would be sent if the worker job unbanned someone
Similarly, there wasn't a notification when the job removed an expired role from someone

I updated the RemoveExpiredRolesCommand to query for any expired roles before deleting the rows. Since this command uses a few joins and is ran every minute, I've created indices on the user/team/series/tournaments role tables on the expires_on column to make the queries as fast as possible.
While working on the indices, I discovered that the second half of UpdateDbSchemaCommand that checks for changed indices was never ran. Since clean_schema returned a map, the call to get_tables_from_schema caused the iterator to be exhausted because of the dict comprehension, and this causes get_indices_from_schema to return an empty dict. A simple fix was to wrap the map with a list.

Everything looks good to me, feel free to merge whenever you're ready


Admin Main Page Update

This is an update to the admin page, customizing it and altering it to make it cohesive with terrastories. This simply adds cosmetic changes to the UI, no functionality change.

Before:

After:

Before:

After:


Should union graph export exclude ili and oewn?

Currently, it includes these datasets.

Yes, it should + the links to those datasets should be available alongside the other data sets.


sha mismatch for perforce

~$ brew install perforce
==> Installing perforce from homebrew/homebrew-binary
==> Downloading http://filehost.perforce.com/perforce/r15.1/bin.macosx105x86_64/p4
######################################################################## 100.0%
Error: SHA256 mismatch
Expected: a6d39a35f2be437e2fcbc28a5d0a6b520a584bc122a262c8a5f14cf1dd1811c9
Actual: 32a4b1da589ce70d408772268986bf9bd09aa22f30c8883cf5cf47eeff8f73ae
Archive: /Library/Caches/Homebrew/perforce-2015.1.1227227

Yeah, looks like it's just been changed: http://filehost.perforce.com/perforce/r15.1/
These aren't properly versioned binaries (we use the date as the version string), so when they're updated the SHA doesn't match and it needs to be updated as well to match.  Interested in opening a pull request?

Closed via https://github.com/Homebrew/homebrew-binary/commit/e0ce8dc363942179345b192b1603310de42ee7f4.


Add isEmpty() / isNotEmpty() to AbstractPathAssert

Summary
AbstractPathAssert lacks assertions to test if path is empty or not empty, in the case of regular files.
These assertions could rely on java.nio.Files.size().
Example
assertThat(path).isEmpty();
assertThat(path).isNotEmpty();

@scordio Can I work on this issue? IsEmpty refers that the path that you are testing doesn't exist or doesn't have any file, right?

Hi @OmarMorales71, sure. Thanks for helping!
The assertion should check that the Path exists, is a regular file and has size equals to zero. The behavior should be similar to isEmpty() for File, which uses assertIsEmptyFile:
https://github.com/assertj/assertj-core/blob/16166e38199d121c843414a5acd315106fc1f1d0/src/main/java/org/assertj/core/internal/Files.java#L325-L329
The existence and the regular file assertion can be covered by assertIsRegularFile:
https://github.com/assertj/assertj-core/blob/16166e38199d121c843414a5acd315106fc1f1d0/src/main/java/org/assertj/core/internal/Paths.java#L142-L145

@scordio Thank you! Let's work on it

@scordio Hello, I have been working on the issue. I just need to do the tests. But for a while, how you see my solution?
Here are the methods in AbstractPathAssert
https://github.com/OmarMorales71/assertj-core/blob/42cb5e258fed31e747f9b2492f33143339080207/src/main/java/org/assertj/core/api/AbstractPathAssert.java#L1753-L1761
And here are the implementations in src/main/java/org/assertj/core/internal/Paths.java
https://github.com/OmarMorales71/assertj-core/blob/42cb5e258fed31e747f9b2492f33143339080207/src/main/java/org/assertj/core/internal/Paths.java#L502-L519
I'm working on the tests right now, I'm in a good way?

Hi @OmarMorales71, the direction is good and I'd be happy to provide some specific feedback. Would you mind raising a pull request, even on draft state?

@scordio I already did a draft PR, in the afternoon I'm going to work on the tests and documentation. If you can give me some feedback or comments please I'd be really grateful.

@scordio The PR has been sent, If you can check it, it will be great. Thank you!

@scordio I read your comments in the pull request, thank you so much, I'm working in the changes!

@scordio I fixed the changes you requested and added the unit tests. If you can check it, it will be great. :D


New Kenya dataset [Example of adding data]

https://user-images.githubusercontent.com/25067660/205736782-25d00547-8eeb-43dd-a3f3-517630fcf8f3.mov

@MsPixels please approve if it looks good!

Looks good to me @ivanzvonkov!


Any plan for pyclipper2？？

https://github.com/AngusJohnson/Clipper2
The author of clipper1 just released clipper2.
I wonder if you have time for wrapper to clipper 2??

Let's keep open as a reminder

Let's keep open as a reminder
Yep!

@anthrotype Any thoughts on this? Thanks.

unfortunately I don't plan to work on this any time soon, if you'd like to take this on feel free to

To me it looks they Clipper and Clipper2 are quite different, I would guess that the entire wrapper had to be rewritten. For example, they are now using templates in the Clipper2 code. From all I can understand, that is somewhat limited and awkward to deal with in Cython.


Can we use an SPDX Expression in the Bundle-License header?

For the contents of the Bundle-License header, the specification states (in part) that...

This identifier should be one of the identifiers defined by [25] Software Package Data Exchange (SPDX) License List.

In cases where content is multiply licensed, it states that you should separate the individual licenses with a comma.
While it is probably reasonable to assume a dual-licensing scenario in which an adopter can choose the license that they prefer to consume the content under (or in SPDX), this is not necessarily the case. There may be cases where the adopter has to accept the terms of multiple licenses (and in SPDX).
The use of the word should when describing the license-identifier and that the header is described as purely informational gives us some flexibility.
So here's the question... would it be considered within the spirit/intention of the requirements of the Bundle-License header to specify an SPDX expression? And, since this information is described as purely informational can we safely assume that providing an SPDX expression shouldn't break anything?
e.g.,
Bundle-License: EPL-2.0 or GPL-2.0-only with Classpath-Exception-2.0

https://docs.osgi.org/specification/osgi.core/8.0.0/framework.module.html#framework.module-bundle-license also states:

The Bundle-License header provides an optional machine readable form of license information. The purpose of this header is to automate some of the license processing required by many organizations like for example license acceptance before a bundle is used. The header is structured to provide the use of unique license naming to merge acceptance requests, as well as links to human readable information about the included licenses. This header is purely informational for management agents and must not be processed by the OSGi Framework.

While this header is not parsed by the OSGI Framework itself it still is expected to be machine readable and can be parsed by things like management agents (e.g. p2).  The syntax defined by the specification does would not allow you to insert and, or keywords like that without some enhancements to the syntax.

My reference to p2 was only as an example.  I don't think p2 actually reads this header at all.

We should update this in the core specification for the next release


base class for Redis

Haraka might benefit from a base class for Redis. Rather than having each plugin do common things like require('redis'), redis_connect(), redis_ping(), have those implemented in a base class that can be inherited.

DB indexes? Sounds good. How about allowing a plugin have a separate redis config?
for ex.: redis.ini:
[default]
host = 1.com
port = 1234
db = 0

[karma]
host = localhost
db = 1
password = xxx

[queue/random_delivery]
port = 12354

Indeed; you might want one plugin to use in-memory and another to use Redis too.
Equivalent functionality can easily be achieved by adding a Redis plugin and exporting it via server.notes to plugins that want to use it (which is what I do here).

considering this done with the merge of plugins/redis.


Wrong error being merged from yup when using nested fields

🐛 Bug report
Current Behavior
When using a yup schema for nested fields, it shows the last error generated by the schema instead of the first. This is since abortEarly is false.
const schema = object().shape({
        parent: object().shape({
          child: object().shape({
            grandChild: string()
              .required('required')
              .min(3, 'min')
              .max(8, 'max'),
          }),
        }),
      });

// generates an error object like
{
  parent: {
    child: {
      grandChild: 'min'
    }
  }
}

Expected behavior
It should show the first error hit by the yup schema.
{
  parent: {
    child: {
      grandChild: 'required'
    }
  }
}

Reproducible example
https://codesandbox.io/s/p54o80922m
Suggested solution(s)
Looks like the fix is pretty simple: https://github.com/jaredpalmer/formik/blob/master/src/Formik.tsx#L685-L689
The inside of the loop does a setIn which sets a deep property path but the initial check errors[err.path] is a shallow check. I think it should be getIn(errors, err.path) for that initial check.
Your environment

Software
Version(s)

Formik
1.5.1

React

TypeScript

Browser

npm/Yarn

Operating System

Closing due to long inactivity


Move var declaration inside closure to make it available to firefox (#6854)

backport #6854

Coverage remained the same at 78.215% when pulling f2520348e4c83462067488cb669eb3a8dc9e4c2e on FinalAngel:release/3.7.x into 6f0a40d2cfab915ace6823ddec94f4b499e24095 on divio:release/3.7.x.


Converting url to media id

Hello how can I get a mediaid out of a link?
best regards

Example: https://www.instagram.com/p/CClpx7qsYm-/
// url code to pk (media id)
long media_id = IGUtils.fromCode("CClpx7qsYm-");
// vice versa
String url_code = IGUtils.toCode(media_id);

Thats only working for public posts not for private posts. Private posts look like this:
https://www.instagram.com/p/BtTnuFVhN0c-dH1F9Ma1DL134F9T9gdtlvVN8I0/
They are way longer

It looks base64 encoded, but I do not currently know how to reverse it into proper id. Anyways, it is a private post, so I don't think it is right or useful to be able to reverse it. Usually the media id is already accessible by calling the user feed endpoint if the account is allowed to view posts from the private user. Library is only meant to replicate functions available to Android application. Perhaps someone can reverse it or I may try to look further into it if I have time.


[Bug]: Multiverse crashing on server start

/mv version -p output
cannot
Server logs
https://gist.githubusercontent.com/filasik/99169e5fbec91930a93b087ee4fe4549/raw/cf052538bf87fd8c46ddbde1745ea00b3279a88c/log.log
Server Version
PaperMC 1.20.x 1.20
Build: 4

Bug Description
when server starts
Steps to reproduce
start server
Agreements

[X] I have searched for and ensured there isn't already an open or resolved issue(s) regarding this.
[X] I was able to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.

You'll need to provide full logs, The one line saying multiverse is disabling does not provide any information.

His meaning is that MC has been updated to version 1.20, and he hopes that this plugin will be updated and support version 1.20: D
The Paper team has updated 1.20
I also hope your team can update support 1.20 as soon as possible. Thank you


remove sage epg option from the STV.

I think we should probably remove the sage epg option from the STV since it's effectively gone at this point.  avoid any future issues popping up.  I haven't made any STV changes since the open source era, so I'm not positive on the best way to submit changes for it, but it shouldn't be a terribly difficult change to simply omit that option from the selection (which should move schedules direct to the top of the list).

I would approve such a change. :)
On Thu, Jun 29, 2017 at 10:04 AM, Christopher Piper <
<EMAIL_ADDRESS>wrote:

I think we should probably remove the sage epg option from the STV since
it's effectively gone at this point. avoid any future issues popping up. I
haven't made any STV changes since the open source era, so I'm not positive
on the best way to submit changes for it, but it shouldn't be a terribly
difficult change to simply omit that option from the selection (which
should move schedules direct to the top of the list).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/322, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANEIDAWCXjTzAF2PT1gsJRzE0aXUsyb_ks5sI9ktgaJpZM4OJmbj
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

Oh wait...that's what's used by EPGPlugins though...so maybe we shouldn't
remove that...but instead rename it and move it down in the order so SD is
first.
On Thu, Jun 29, 2017 at 10:11 AM, Jeffrey Kardatzke<EMAIL_ADDRESS>wrote:

I would approve such a change. :)
On Thu, Jun 29, 2017 at 10:04 AM, Christopher Piper <
<EMAIL_ADDRESS>wrote:

I think we should probably remove the sage epg option from the STV since
it's effectively gone at this point. avoid any future issues popping up. I
haven't made any STV changes since the open source era, so I'm not positive
on the best way to submit changes for it, but it shouldn't be a terribly
difficult change to simply omit that option from the selection (which
should move schedules direct to the top of the list).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/322, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANEIDAWCXjTzAF2PT1gsJRzE0aXUsyb_ks5sI9ktgaJpZM4OJmbj
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

Right.. those silly plugins...  Probably change the text on the entry to focus on plugins.

Out of curiosity, are there any EPGPlugins in the wild in active use?

Yeah, there's a few....when I run stats on the EPG server it explicitly
lists the number of people using an EPG plugin and IIRC it's a couple
hundred.
On Thu, Jun 29, 2017 at 10:20 AM, Christopher Piper <
<EMAIL_ADDRESS>wrote:

Out of curiosity, are there any EPGPlugins in the wild in active use?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/322#issuecomment-312034155, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDMhdf_VXKTueowRfD7KmkUC540Btks5sI9zAgaJpZM4OJmbj
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

Wow, that's a pretty good size number. At the least maybe the code should be changed so that it no longer accommodates the old EPG service and immediately returns an error if a plugin isn't configured.

Last time I ran the numbers (which was a month ago) there were still about
2000 people using the SageTV EPG server.  I don't know how many are on SD,
but the total user base is probably somewhere in the 3000 range is my guess.
On Fri, Jun 30, 2017 at 11:13 AM, Joseph Shuttlesworth <
<EMAIL_ADDRESS>wrote:

Wow, that's a pretty good size number. At the least maybe the code should
be changed so that it no longer accommodates the old EPG service and
immediately returns an error if a plugin isn't configured.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/322#issuecomment-312337290, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDM7UWFlmTokBTaPkNdKfXYHkURGMks5sJTqugaJpZM4OJmbj
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

That's going to be interesting come 7/14.

Yeah...I just checked and there's still about 1000 users hitting the
existing EPG server over the past 3 days. But I know some people that
weren't planning on upgrading until this weekend...but there will
definitely be many that are caught off guard.
On Fri, Jun 30, 2017 at 12:54 PM, Joseph Shuttlesworth <
<EMAIL_ADDRESS>wrote:

That's going to be interesting come 7/14.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/322#issuecomment-312359138, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDDzRy0dWPItnfK86ozvgCvjc4BNzks5sJVJqgaJpZM4OJmbj
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

I am guessing that there are a few folks like me that have multiple servers running.  I have four servers running currently - two hitting SD and two hitting the old EPG server.  Are you counting SageTV or IP addresses as they should all be coming from the same IP addresses.

There's a few different ways it can count and they all come out pretty
close.
Jeff Kardatzke
Sent from my Android
On Jun 30, 2017 6:50 PM, "wayner9"<EMAIL_ADDRESS>wrote:

I am guessing that there are a few folks like me that have multiple
servers running. I have four servers running currently - two hitting SD and
two hitting the old EPG server. Are you counting SageTV or IP addresses as
they should all be coming from the same IP addresses.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/322#issuecomment-312403525, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDHXBrV2I8SQrAUvgAZTzbk8luIZgks5sJaX6gaJpZM4OJmbj
.

well, past the critical weekend.  how many uniques are hitting the server now?

I can't tell you...access to that port is blocked by firewall policy now so
nobody is actually connecting to it anymore.
On Tue, Jul 11, 2017 at 4:44 AM, Christopher Piper<EMAIL_ADDRESS>
wrote:

well, past the critical weekend. how many uniques are hitting the server
now?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/322#issuecomment-314418211, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDBlYvAjpx3J7kI-ZxDVP4Uur2lmAks5sM1__gaJpZM4OJmbj
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

gotcha.

I think indirectly related is the Zap2It logo still appears in the System =>Information Screen


⚠ Bofrost.at has degraded performance

In ff5a289, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1314 ms

Resolved: Bofrost.at performance has improved in fbc49fc after 9 minutes.


xterm: 331 -> 332

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/xterm/versions.
These checks were done:

built on NixOS
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/koi8rxterm -h’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/koi8rxterm help’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/uxterm -h’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/uxterm help’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/xterm -h’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/xterm help’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.xterm-wrapped -h’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.xterm-wrapped help’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.koi8rxterm-wrapped -h’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.koi8rxterm-wrapped help’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.uxterm-wrapped -h’ got 0 exit code
ran ‘/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.uxterm-wrapped help’ got 0 exit code
found 332 with grep in /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332
directory tree listing: https://gist.github.com/d4955919bb8d06a92848a3ca009f26e9

cc @viric @vrthra for review

Success on x86_64-linux (full log)
Attempted: xterm
Partial log (click to expand)
shrinking /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.resize-wrapped
shrinking /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.xterm-wrapped
gzipping man pages under /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/lib  /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin
patching script interpreter paths in /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332
/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.koi8rxterm-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332/bin/.uxterm-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332...
/nix/store/z5digwa447rm08z7y1krc13km16lqwnl-xterm-332


Make Database connection in the config optional

Also in this pull reques:

Updated libraries
Removal of rhel7

@kouylekov-usit hold on, we can't remove 3.8 tests while the project is still targeting >=3.8. Not just 3.8 having issues either, the updated tsd-api-client broke tests on other versions too

I removed 60f52d553c42d9b987c910632c4585a41336cedb and f9c57b5487366b040538ccaeffb95e8133d6e9b6, also pinning tsd-api-client to <3.6 since current release has broken backwards compatibility with how it's used for tsd-file-api test (and moved it to dev dependencies since it's only used for tests).


This seems to happen only when 6-9 transactions are intended to be displayed.

Link the related issue
Closes #0000
Checklist before requesting a review

[x] Is this PR ready for merge? (Please convert to a draft PR otherwise)
[x] I have performed a self-review of my code.
[x] Did I request feedback from a team member prior to merge?
[x] Does my code following the style guide at docs/CODING-STYLE.md?

Instructions for Reviewer
Functionalities or workflows that should specifically be tested.

clicking various candles display expected number of rows

Environmental conditions which may result in expected but differential behavior.

different candles will have different number of transactions to display, some requiring a 'View More' button

If relevant, list additional work to complete pre-merge (delete logging, code abstraction, etc)

Thanks, @enwong10. Looks good!


Subscription removed from StompClientConnectionImpl cache even before sending UNSUBSCRIBE frame

Questions
Do not use this issue tracker to ask questions, instead use one of these channels. Questions will likely be closed without notice.
Version
Which version(s) did you encounter this bug ?
4.4.5
Context
In StompClientConnectionImpl class when an unsubscribe call is received, it is removed from client cache "List subscriptions" before sending the UNSUBSCRIBE frame. When this call is failed at server side, client cache doesn't have the subscription due to which we are receiving events and we cannot make another unsubscribe call as StompClientConnectionImpl would return an IllegalArgumentException
https://github.com/vert-x3/vertx-stomp/blob/55227790cb75f13cbcda764cc269efa02da652ae/src/main/java/io/vertx/ext/stomp/impl/StompClientConnectionImpl.java#L357-L361
Same goes for subscribe as well.
https://github.com/vert-x3/vertx-stomp/blob/55227790cb75f13cbcda764cc269efa02da652ae/src/main/java/io/vertx/ext/stomp/impl/StompClientConnectionImpl.java#L313-L326

do you have a reproducer for this ?

Nope I'll try to create a test case for it. Fix would be adding the subscription in cache after sending SUBSCRIBE/UNSUBSCRIBE frame, so haven't thought of reproducer.


Remove @electron/remote

Usage of @electron/remote is discouraged
We can replace it with inter-process communication fairly easily, just need to do it piece by piece.

@bmulholland - would you mind confirming that this can be closed.  I did a search and couldn't find any references to remote from the electron package directly.  All seem to have been migrated to @electron/remote

All seem to have been migrated to @electron/remote

That's the package that is discouraged and shouldn't be used. From the linked article, about @electron/remote itself:

⚠ Warning! This module has many subtle pitfalls. There is almost always a better way to accomplish your task than using this module. For example, ipcRenderer.invoke can serve many common use cases.

Oops!  I misread it.  I focused on this comment but missed the warning.  Thanks for the reminder

@electron/remote is a replacement for the built-in remote module in Electron, which is deprecated and will eventually be removed.


Thank you!

Hi there,
First of all I wanted to thank you for your hard work and sharing this project. In my opinion, it is the best Photoshop SD project I've seen so far. The API connection works great, allowing easy access to different models. Being able to make selections instead of masks also works way better than I initially expected. And that for a project which had its first release last week!
Like your username, it really feels like you and Stable.Art are from another world :)
Issue wise there aren't any major "bugs/complaints" at all which surprised me after testing out a lot if not most of the other Photoshop SD projects. The minor bugs like variation in color are easily fixed with a hue/saturation mask, rough edges with a mask and a soft brush and if I remember correctly, the README said you are already working on outpainting.
I'm never going back to another Photoshop SD plugin ever again :)

Wow, thank you for the such wonderful feedback, I really appreciate it! And yep, I am already working on outpainting, I hope to deliver it in the upcoming weeks.
About the bug with variation in color - I will check it after outpainting.
About rough edges - could you provide any examples?
I want to make this plugin perfect, so feel free to create issues even for such minor bugs.

You can just upload them right here, to this issue :)

ah sorry, I forgot that that was a possibility haha. Here are some pictures explaining the rough edge issue.
the file ending with _Original is the source image. The one ending with _1_Default is an example using a wound on the left cheek edited with Stable.Art without any further edits. _1_Moved is the same edit but moved slightly to better line up. The files ending with _2_default and _2_moved follow the same rule, only this time focusing on removing a part of the weapon right of the necktie.
As you might notice, the edits made with Stable.Art are:

Slightly misaligned
Slightly miscolored
Have some hard outer edges (depending on the color)

The misaligning issue is fixed easily enough by using ~2 presses of the arrow keys, usually in a downwards and right direction most of the time as far as I've noticed
The miscoloring issue depends on the source color. In my case, light brown/brown colors got quite a different and noticable hue, lightness and saturation. This is pretty easy to fix with a Hue/Saturation mask though
The hard edges are more noticable with certain colors such as the brown I talked about above. Adding a mask to the layer and using a 0% hardness brush (very soft) to quickly brush around the very edges fixes this in an instant. I hope this is enough information but if it's not, feel free to send another message :)

Ah, got it. Yes, this is enough, thanks!

Hey, just pushed some changes, issues with misaligning and hard edges are fixed. Could you test it?
In theory, the miscolored problem is also fixed, but I cannot reproduce it, so I am not sure. If it still exists, could you try to reproduce it in the webui itself? I do not manipulate colors in any way, so most likely the problem is related to the webui. Maybe the setting "Apply color correction to img2img results to match original colors" in the webui could fix it.


[1.18.1] Java Error: Caused by: java.lang.ClassCastException

Hi, I always get the same error message when I attempt to use certain origins such as Phantom or Enderian, but not others like Feline or Arachnid. I get an internal error message on my server. All origins work fine on single player, not sure what the issue is.
How does this come about?
 Here's the log

Fixed in .9


Proposal: An ErrorProxy trait for libraries that wish to generically consume errors

Strawman Proposal
Create a minimal ErrorPoxy trait in a new crate and add an implementation to anyhow (and similar error aggregation libraries) so that libraries that wish to generically consume errors can access backtraces and error sources for types such as anyhow::Error that cannot implement std::error::Error.
Background
Due to cohesion limitations anyhow::Error cannot implement std::error::Error without creating an infinite impl From<anyhow::Error> for anyhow::Error loop.
Crates such as async-graphql and sled consume user-defined error types and would benefit from a Trait that provided source and backtrace information across the most common error-like types.
In order to consume an error crates have three options at the moment:

Drop Anyhow support and only accept types that implement std::error::Errors.
Treat all errors as fmt::Display + fmt::Debug dropping error chaining and backtrace information
Choose one error handling library and force all users to use it - eg, async-graphql could choose to depend on error-chain or anyhow but not both (a feature flag could toggle this but then libraries built on async-graphql have the same problem).

Note: This trait would need to be implemented inside anyhow (and similarly error-chain, etc.) because an external implementation leads to a conflicting implementation compiler error since anyhow the compiler cannot guarentee that future versions of anyhow won't implement std::error::Error.
Example Implementation
use std::error::Error;
use error_proxy::ErrorProxy;

#[cfg(backtrace)]
use crate::backtrace::Backtrace;

impl ErrorProxy for anyhow::Error {
    fn source(&self) -> Option<&(dyn Error + 'static)> {
        <self as anyhow::Error>.source()
    }

    #[cfg(backtrace)]
    fn backtrace(&self) -> Option<&Backtrace> {
        <self as anyhow::Error>::backtrace(self)
    }
}

Edit: Created a crate for this: https://crates.io/crates/error-proxy

I would prefer if this were done externally, i.e. putting the following impl in whatever crate defines ErrorProxy.
#[cfg(feature = "anyhow")]
impl ErrorProxy for anyhow::Error {
    fn source(&self) -> Option<&(dyn Error + 'static)> {
        (**self).source()
    }
    fn backtrace(&self) -> Option<&Backtrace> {
        Some(anyhow::Error::backtrace(self))
    }
}

@dtolnay Appreciate you looking into this. Problem there is AFAIK we end up with a compiler error if we also include:
impl ErrorProxy for dyn std::error::Error + 'static {
    fn source(&self) -> Option<&(dyn Error + 'static)> {
        Some(self)
    }

    #[cfg(backtrace)]
    fn backtrace(&self) -> Option<&Backtrace> {
        std::error::Error::backtrace(self)
    }
}

As I understand it the issue is that the compiler can't guarantee that a future version of Anyhow won't implement std::error::Error. I think the idea is that adding traits should never be a breaking change but I could be wrong.

It compiles fine though.
#![feature(backtrace)]
use std::backtrace::Backtrace;
use std::error::Error;

pub trait ErrorProxy {
    fn source(&self) -> Option<&(dyn Error + 'static)>;
    fn backtrace(&self) -> Option<&Backtrace>;
}

impl ErrorProxy for dyn std::error::Error {
    fn source(&self) -> Option<&(dyn Error + 'static)> { unimplemented!() }
    fn backtrace(&self) -> Option<&Backtrace> { unimplemented!() }
}

impl ErrorProxy for anyhow::Error {
    fn source(&self) -> Option<&(dyn Error + 'static)> { unimplemented!() }
    fn backtrace(&self) -> Option<&Backtrace> { unimplemented!() }
}

I'll close the issue since I don't intend to introduce a dependency for this. Thanks anyway!

Oh weird, I was sure that would error. Ok, I'll give it a shot. Generally does a trait sound like a good approach for libraries?

So how do you actually define a method that uses ErrorProxy to generically consume errors? My attempts end up with "trait bound  is not satisfied" errors when passing errors that impl std::error::Error:
https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=4e68530294ba1ebc806e6eca2766c377


docs: Update layer versions

Detail

docs: Update layer versions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 410a6a99a5f35617f424045b085da7c9a6a9b568
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 410a6a99a5f35617f424045b085da7c9a6a9b568
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


what command to use to stdout data

what command to use to stdout data of a live stream in a dirctory..

hlsdl http://xxx.yyy.zzz/xx.m3u8 -o -


maxBufferedItemCount seems to be partially ignored in azure-cosmos

Describe the bug
A clear and concise description of what the bug is.
I have ca 7050 records in my test cosmosdb collection
when I execute
    var cosmosQueryOptions = new CosmosQueryRequestOptions();
    cosmosQueryOptions.setPartitionKey(new PartitionKey("specific pk"));
    cosmosQueryOptions.setMaxDegreeOfParallelism(1);
    cosmosQueryOptions.setMaxBufferedItemCount(1);
    cosmosQueryOptions.setResponseContinuationTokenLimitInKb(4);

    container.queryItems(query, cosmosQueryOptions, MyEntity.class).iterableByPage(token, 100).iterator().next()

my expectation is that only one query will be executed against cosmosdb
but when I check

cosmosdb metrics - these show 71 query operations
cosmos diagnostics (CDBDataPlaneRequests) - also 71 queries are displayed

what I assume means that all records are cached (71*100=7100)
in our production env
for ca 500 calls to similar function
cosmosdb metrics show over 100 000 queries
obviously this impact RU and also amount of data transferred to our pods
In our current setup when user requests next page of data we call above function
so I'd prefer to have limited bufferedItemCount
but this doesn't seem to work
Can I ask for suggestion how can I solve this?
Environment summary
SDK Version: azure-cosmos-4.62.0
Java JDK version: 17
OS Version MacOS

@gawlikt this repository is not maintained anymore and has been deprecated. Can you please open this issue here - https://github.com/Azure/azure-sdk-for-java


Add support for mimir in the metrics test.

What this PR does
Towards:
https://github.com/giantswarm/giantswarm/issues/30424
adds support for mimir to the metrics test.
Checklist

[x] Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites


How to load Pharo project (existing branch) programatically?

Describe the bug
I'm trying to implement create fromPR command on Pharo Launcher (CLI for now).  My progress so far:
I can obtain from PR number remote branch and Github project name (owner/repository).
On created image (from dev template or stable P10 template) I try to load programmatically Pharo project using MC command.
But Pharo project remote branch (from contributor) cannot be loaded using MC command, it fails.
To Reproduce
Manual steps to reproduce the behavior (UI way):

Download and create P10 stable image (e.g. via launcher UI).
Open Playground and evaluate following (obtained from PR 9588):

Iceberg remoteTypeSelector: #httpsUrl.
Metacello new
	baseline: 'Pharo';
	repository: 'github://dionisiydk/pharo:refactorCompiler9/src'; 	onConflictUseIncoming;
	load

Following error occurs:

MetacelloPackageSpecResolutionError:
Could not resolve: BaselineOfPharo [BaselineOfPharo] in /Users/dbajger/Documents/Pharo/images/P10-launcher/pharo-local/package-cache https://github.com/dionisiydk/pharo.git[refactorCompiler9]

Expected behavior
Pharo project is successfully loaded.
Screenshots

Version information:

OS: MacOS 12.2.1 (Monterey)
Version: P10 latest stable

Additional context
I tried a different PR but resulted in same issue. Any idea what can be wrong with this approach? Help appreciated! Thanks much!

@demarey or @guillep If you have some time (silly precondition, I know), can you look at this? I have no idea, how to load Pharo project remote branch from cloned repo programmatically. Thank you for any hint!

Resolution
1)
for Pharo-project use templates created from CI build, where PR is part of URL link, e.g.:
https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/view/change-requests/
https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/view/change-requests/job/PR-10553/lastSuccessfulBuild/artifact/
Pharo10.0-PR-64bit-5ce32be.zip
full url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/view/change-requests/job/PR-10553/lastSuccessfulBuild/artifact/bootstrap-cache/Pharo10.0-PR-64bit-5ce32be.zip

For other projects use latest stable image and load project baseline (PR number is optional here, project name is enough).


⚠ Jenkins Plugins Artifactory has degraded performance

In ee18305, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1589 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 36d0489 after 24 minutes.


feat: add additional column ordering to keysetpagination

cc @zepatrik
Related Issue or Design Document

Checklist

[ ] I have read the contributing guidelines and signed the CLA.
[ ] I have referenced an issue containing the design document if my change introduces a new feature.
[ ] I have read the security policy.
[ ] I confirm that this pull request does not address a security vulnerability.
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact<EMAIL_ADDRESS>from the maintainers to push the changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added the necessary documentation within the code base (if appropriate).

Further comments

@zepatrik I'd like to postpone the support for multiple fields for now. I've spent too much time on this already, and that would just complicate it, IMO. I'll add some more tests, but then it should be good to go.


Product Name - Active Directory

Why:
Avoid confusion with on-premises Active Directory
What's being changed:
Update product name from on-premises Active Directory to cloud based Azure Active Directory
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@ljtill
Thanks so much for opening a PR! I'll get this triaged for review :zap:


Add logo to the title bar.

There is no logo in the title bar of the website, it would be great to add one.

Can you please assign this task to me

hey can i get this assigned under gssoc 24 .
i agree to follow all the rules and conditions of gssoc 24

@khushi-joshi-05 ,I think every site must be having some logo.
Kindly assign this issue to me and i know FIGMA using which i will add a beautiful and unique logo to it.


Update AUTHORS file and .mailmap

added LC_ALL=C due to osx
http://www.inmotionhosting.com/support/website/ssh/speed-up-grep-searche
s-with-lc-all
Signed-off-by: Patrick Stapleton<EMAIL_ADDRESS>

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to the last commit:
git commit --amend -s --no-edit && git push -f

@jfrazelle thanks for your help, I was able to sign my commit using my real name

LGTM
ping @tianon

@tianon any more suggestions for this PR?
most the of people who aren't signed off are fine since they're small PRs

Naw seems good enough.  Thanks for sticking with it!
LGTM

LGTM


add calc_nobs

Adds a function to calculate the number of observations in source, with option to do it just for the total, or the total and a count for each band.
Addresses #246.

Looks good to me!


ExtractFiles to support override files on the target path

I'm facing an issue that I don't want to delete all files on the target path prior to extraction from the archived file. Would be great to have an option to allow overriding or not the target files.

Have you tried unchecking the Clean destination folder before extracting option?  If so, could you clarify your issue?

Yes, I did. But when I try to uncheck this flag, it raises an error saying that files cannot be extracted because they already exist in the target folder and tries to ask on the console if I want to replace it or not (see the attachment). Would be great o have a flag to force always to 'Yes' option during the extraction.

Ah I see, thanks for the additional info.  What type of archive was this (zip/tar/7z)?

In this case, I'm using zip files but I believe for other types may face the same issue.
Thanks a lot.

Closing since this is an old issue. If you are still seeing issue, please open a new issue .


Avoid duplicate depencency declare via package re-export

Currently it's not really convenient to add new deps to node/runtime and core/test-runtime. One would need to declare the dep twice and remember to set features in those two cargo files correctly. This PR improves QoL by removing the link, but re-export non-wasm package for wasm compile.

Seems like a reasonable approach to get rid of the symlinks.


Make statistics

Some statistics would be interesting, e.g

time of last backup run
total number of successful backups
total number of currently stored snapshots
etc.

Those values could be published as json via mqtt

Handled by the sensor introduced in v4.0


Fixes error on DOM element removal

In certain cases attempts to delete the parent node of DOM elements that did not exist. This makes sure the element exists before attempting to remove it's parent.

Thanks @battlejj


ARTEMIS-1327 - Support checked exceptions in ActiveMQServerPlugin

This will allow plugin writers to use checked exceptions when writing
plugins

I pushed up a new version that logs the error a the warn level

@clebertsuconic - So I just realized that I don't think we want to swallow exceptions.  If there's an exception in the plugin don't we want it to propagate?  There are 2 cases for exceptions, the first is a buggy plugin and that just means the plugin should be fixed.  The second is if there is an actual error and I think that error should be propagated and not just logged.
For example, one of my use cases for this is I want to validate clientIds.  So I want to throw an exception and reject a clientId in the beforeSessionMetadataAdded method if it doesn't meet some criteria and have it propagated back to the client.  (There's actually another Jira I need to open to make this work properly to make sure the exception is thrown back to the client).

@clebertsuconic - Or another use case...say you want to write a custom authentication plugin.  You would normally want to throw something like a security exception, etc in the plugin methods but now you can't do that.

What about expose activemqexception.  But swallow others?
Let me amend my PR and will and you feedback.

That would probably be ok, just as long there is some way to propagate an exception when the plugin writer wants to.  If we do that should we change the plugin interface to throw ActiveMQException instead of Exception?  I'm about to head out for the day but I'll take a look at your new PR in the morning.

@cshannon take a look on #1446 please?

@cshannon I will merge this now as I need to add something on the server plugins...
if there's anything wrong with it.. we can review it later, ok?

@clebertsuconic - This looks good to me


java.lang.NullPointerException when using customMM2Replication Policy

when is use the CustomMM2ReplicationPolicy (https://github.com/aws-samples/mirrormaker2-msk-migration/blob/master/CustomMM2ReplicationPolicy/src/main/java/com/amazonaws/kafka/samples/CustomMM2ReplicationPolicy.java) in a active-passive replication setup I get java.lang.NullPointerException when I create MirrorSourceConnector.
The replication works complete fine if I use the default apache replication policy.
looks like the the latest release of MirrorSourceConnector they added null check for upstream topic and looking at the customMM2Replication class, the upstream topic is set as null causing this issue
Here is the screenshot of the error

kafka3.x  supports this custom policy by default (CustomMM2ReplicationPolicy cannot be used.)
https://stackoverflow.com/a/70994290/5986291
if we use kafka2.x, this error can be avoided.


mesa: fix cross compiling

This PR prevents weird behavior when cross compiling to other architectures and mesa is wanted.

I do not understand the motivation here. I'm not sure I can support cross-compiling our Mesa, though I am a little surprised that it does not work.
If you need to make this conditional, it should suffice to check in your own configuration and set hardware.asahi.useExperimentalGPUDriver appropriately rather than modifying nixos-apple-silicon.
In any case, this modification here puzzles me as it keeps the kernel mode components enabled, but the driver can't work without the user mode components too.

I do not understand the motivation here. I'm not sure I can support cross-compiling our Mesa, though I am a little surprised that it does not work.

This fixes using Mesa when it's an overlay and you're cross compiling for a different architecture. Without this, if I were to use pkgsCross.gnu64.mesa it would build Asahi Mesa instead of the x86 one. This PR prevents the Asahi Mesa to be used outside of aarch64.

In any case, this modification here puzzles me as it keeps the kernel mode components enabled, but the driver can't work without the user mode components too.

I had more stuff in this PR without realizing how the module worked but forgot to change the title. Disregard the part that says this allows for disabling. This PR only allows for the Asahi Mesa to be used on aarch64.

Oh okay, so you are saying if you are on a machine with nixos-apple-silicon and want to cross-build Mesa for an x86 machine? That makes sense now, but how would this overlay get into that version of nixpkgs? I am reasonably sure it will not if you do e.g. nix-build -A pkgsCross.gnu64.mesa '<nixpkgs>'.
I suppose if you have everything in the same config somehow? Why doesn't setting hardware.asahi.enable to false take care of that?

Oh okay, so you are saying if you are on a machine with nixos-apple-silicon and want to cross-build Mesa for an x86 machine?

So I could play Steam games using box64.

That makes sense now, but how would this overlay get into that version of nixpkgs?

Because I enabled the Steam module in NixOS.

I suppose if you have everything in the same config somehow? Why doesn't setting hardware.asahi.enable to false take care of that?

Because I don't know what else that would disable.

Okay, I see now, that makes sense. Thank you for explaining. I don't think that is an issue with the other options because presumably the Steam module doesn't use hardware.opengl for the cross build and system.replaceRuntimeDependencies won't have the same source Mesa.
Please add a comment summarizing the purpose like # prevent cross-built Mesas that might run on this system under emulation (e.g. box64 and Steam) from using the special Asahi Mesa and I will merge.

Please add a comment summarizing the purpose like # prevent cross-built Mesas that might run on this system under emulation (e.g. through Steam and box64) from using the special Asahi Mesa and I will merge.

Will do when I am able to this evening. 👍

Done


[5.1 EARLY] [Swift] The AST printer now qualifies type names more consistently

Cherry-pick of #1417 to the early 5.1 branch for Apple purposes. Reviewed by @dcci and @adrian-prantl.

Testing in apple/swift#23667.


Implement bulk loading

Terrastore should implement documents bulk loading: that is, a way to insert
more documents with a single call.

Bulk loading shouldn't anyways provide any atomicity or isolation guarantee.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 16 Dec 2010 at 9:42

Original comment by<EMAIL_ADDRESS>on 12 Mar 2011 at 9:39

Added labels: Milestone-0.8.2
Removed labels: Milestone-0.8.x

Issue 163 has been merged into this issue.

Original comment by<EMAIL_ADDRESS>on 12 Mar 2011 at 9:43

cool!

1) how does it look in java api?

2) do you have a maven snapshot or do I have to build myselv?

thanks.

Original comment by Andrei.Pozolotin on 19 Mar 2011 at 7:47

Implemented on Java Client too: see issue 172.

Original comment by<EMAIL_ADDRESS>on 26 Mar 2011 at 4:58

Hey Andrei,

the Java Client doesn't currently provide bulk put/get APIs: I'll add them in
the next few days.

Original comment by<EMAIL_ADDRESS>on 20 Mar 2011 at 9:45

Implemented.
Exposed as follows: POST /bucket/bulk/put

Not sure about the URL, which contains the operation name in order to
disambiguate with bulk get.

Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 7:18

Changed state: Fixed


Improve "Add trusted nodes" tutorial to avoid confusion about identical generated keys for both nodes

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

[X] This is not a support question.

Content request
Hello everyone.
I was going through this tutorial:
https://docs.substrate.io/tutorials/get-started/add-trusted-nodes/
and had no complaint until the place where you look at generated keys for the nodes:
ls /tmp/node01/chains/local_testnet/keystore
Tutorial says it should display similar to the following:
617572611441ddcb22724420b87ee295c6d47c5adff0ce598c87d3c749b776ba9a647f04
6772616e1441ddcb22724420b87ee295c6d47c5adff0ce598c87d3c749b776ba9a647f04

So you can clearly see two IDENTICAL keys (disregard first 7 digits of both keys, they present Sr25519/EdSr25519 differential part I would imagine).
However if you follow the tutorial precisely (like I did), you will notice that you actually get two different keys generated.
My output:
617572617e016f19ab623ba5f487f540017c1edbab06c0b211a16d40531dbd62d94ceb24
6772616e4ac976937e53fd836512cfd288bb438584ba366cbf9be403a0acd82c1c7c0739

And similar for the second node.
TUTORIAL says you should get:
617572611a4cc824f6585859851f818e71ac63cf6fdc81018189809814677b2a4699cf45
6772616e1a4cc824f6585859851f818e71ac63cf6fdc81018189809814677b2a4699cf45

And what I've got is:
617572610a6cadb3d6f55a121de4c89754dd835e634ae83249734dfad01c2fae7e9ac102
6772616e5f273f61a4910897cec969b598a70a832fb7894ad7c741e2a559617898426f20

Did I do something wrong? Or have I potentially found some misleading information in the tutorial?
Wouldn't necessarily call it "a bug", as nothing breaks, but seeing identical keys in the tutorial someone might expect the same in their  own set up, and wasting hours figuring out why his generated keys ain't identical.
What needs to be updated:
https://github.com/substrate-developer-hub/substrate-docs/blob/main/content/md/en/docs/tutorials/get-started/add-trusted-nodes.md
Lines 459, 460
Lines 546, 547
Shall we fix it? I can update with my locally generated keys as an example, if approved.
Are you willing to help with this request?
Yes!

@gogosama-dev, great catch! I didn't notice that they were identical after the first 7 digits.
Feel free to open a PR and fix the issue.
Thanks again for helping to improve the tutorials.

https://github.com/substrate-developer-hub/substrate-docs/blob/main/content/md/en/docs/tutorials/get-started/add-trusted-nodes.md

Thanks, Lisa!
I've submitted my PR:
https://github.com/substrate-developer-hub/substrate-docs/pull/1724


Build menu

Menu should use the checkbox version to hide/unhide; (so it's navigable by keyboard)
Menu should be responsive
Menu will require dropdown menus

9/9/2024
Desktop and mobile menu layout built. Created a breakpoint at width 1000px between the two.
What's left is to:

Add the hrefs to each nav menu item
Add animations

Hamburger (https://codepen.io/fabean/pen/XXegoQ)
submenu item transition for desktop and mobile

To confirm: @ladyofcode
Can I assume the href will be the same as the nav item text?

upcoming => /upcoming
past jams -> /past-jams
jam 1 -> /past-jams/jam-1

So the actual links will be:
Upcoming > /game-jam-2025
Past Jams > /past-jams
Contact > /contact
Jams links: /game-jam-2024, /game-jam-2023, etc
For Past Jams dropdown, either add the links manually or have it generate a link for every slug on the posts api route and use that array for the dropdown


Color contrast in immersive mode

Right now our primary color is #4d4d4d. To have a contrast that passes WCAG AA for graphical objects and UI Components, it needs to be at least 60% opacity, which makes the difference between normal and immersive modes kinda difficult to notice.

I guess that the implementation of themes could also affect this value. Is there any way to dynamically calculate this contrast ratio as well?

Took a deeper look at MUI, which has some world-class and extensive color manipulation utility to achieve some accessibility in their theming.

utils can be found in colorManipulator – I’m legit sad it doesn’t exist as a standalone package tbh, especially as there is a common approach across contrast utilities so you’re pretty much reimplementing the same functions everywhere;
they have a contrastThreshold you can set in palette/themes but it’s just a number for comparison (if/else) in order to pick between light and dark text depending on the background color;
they made it clear it’s the same logic as Bootstrap in a comment;
this means their theme provider won’t automagically pick a color for you;
very few utilities go the extra mile to provide values that match the contrast criteria, but they do exist, albeit in a complex form.

have you looked into https://www.radix-ui.com/colors ?

For reference, how colors are generated in Radix. Couldn’t take an extensive look yet though, so can’t describe what is happening, I’m just logging the URL so that I remember.

After extending themes properties in the Playground preferences, I guess a sensible first implementation could be to create tonal palettes from main colors e.g. background can be used for hover for instance, text for subdue or disable etc. and pick the first color in the tonal palette that meets a given/preset contrast ratio.
In that case it could either be automatic when a theme property is null or opt-in (an helper function). Results are not going to be necessarily perfect every time but that should at least help get a starting point you can fine-tune if you're not happy with the result.
That said, I’d be happy to hear how others would approach this issue.


My Changing on Tensorflow is not work

GitHub issues are for bugs / installation problems / feature requests.
For general support from the community, see StackOverflow.
To make bugs and feature requests more easy to find and organize, we close issues that are deemed
out of scope for GitHub Issues and point people to StackOverflow.
For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System:
OS X EI Capitan version 10.11.5
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
If installed from binary pip package, provide:

Which pip package you installed.
The output from python -c "import tensorflow; print(tensorflow.version)".

If installed from sources, provide the commit hash:
Steps to reproduce
1.Change the file cifar10_input.py with adding some print in it.
2.rebuild my tensorflow as the https://www.tensorflow.org/versions/r0.8/get_started/os_setup.html#installing-from-sources WHICH Section "Setting up TensorFlow for Development" instruction.
3.And when I back to cifar10 to run the cifar10_train.py, I found nothing have changed about my revised.
What have you tried?
1.I try the instruction from other section "Create the pip package and install"
Logs or other output that would be helpful
(If logs are large, please upload as attachment).

Unfortunately this is too little information, and I am having trouble parsing your sentences.  It sounds like you added a print statement or similar and it didn't show up when run.  This is probably a simple shell error, not an issue with TensorFlow itself.

Not only for print I had ever tried to remove some code, could cause program crash  i.e. def ...., from cifar10_input.py. And When I run cifar10_train.py, it can work well.

Unfortunately I still don't understand your issue, but I suspect it is closer to a StackOverflow question than a problem with TensorFlow.  You could try posting your code changes, or you could try posting them on StackOverflow with the tensorflow tag.

Actually, I think I just want to know how to correctly install my tensorflow with some code changed by myself. And I guess the reason why my change is unavailable, is that my tensorflow library is old, the before code without my change. So I think what I really need, just a correct method to develop and install tensorflow with python

Okay, I'll close for now.  Let us know if you have further issues, but we'll probably need more details for those too if so.

-.-! But I have no idea about how to develop and install my own code....cloud you give me some method with detail?

@SunAriesCN: Questions about how to write new code on top of TensorFlow belong better on StackOverflow.


Added Aho Corasick from returnzerooo blog

Thanks for your contribution. :)

Thanks for this awesome repository and also for including the links!


node version>=5.0.0 instead of LTS 4.2.x

@davezuko Why do you choose to use (package.json) node version 5 instead of a LTS version?

I was wondering the same thing today.

Because it comes with npm3 by default, and I thought it would be weird to require 4.2.x but enforce npm3.
If that's not weird I'd have absolutely no problem dropping the version to 4.2.x.

I didn't notice that I had the npm engine specified, will go update the README/package.json to point to >=4.2.0.

Nice, tnks.

https://github.com/davezuko/react-redux-starter-kit/commit/c4a2732d5bf7bda0ad8443b8cfeddcd17a0ad01b should do the trick (don't want to lock it to 4.2.x because some people do use ^5.0.0).


Shuffle ammo for Buckshot Roulette

About the PR

Ammo in shotguns, Ammo boxes, and magazines now can be shuffled, so you can play buckshot roulette!
Shells should enter the chamber in a hidden sequence, which is impossible without a third party loading a shotgun, which overcomplicates RP process, so now it's possible to load lethals/blanks and shuffle them
Why / Balance

Because of Executions PR: https://github.com/space-wizards/space-station-14/pull/24150
Me, and my friends recognized how cool this feature is. Not for 1984 executions / suicides, but to play Buckshot roulette!
Now, Execution PR has some issues and do have bug when shooting yourself with blank is actually lethal, as mentioned here:
https://github.com/space-wizards/space-station-14/issues/24180
Despite that, execution idea is awesome. This PR assumes this idea stays and helps to play roulette.
Technical details

New rule applies to BallisticAmmoProviderComponent, meaning this action is allowed to magazines, ammo boxes, guns unless they're using magazine, so it can be done with Hristov, Mosin, most of shotguns, but not revolvers, which can be spinned instead.
This may be weird (shuffling loaded shotgun/magazine), but it simplifies the process.
It seems okay to me - people don't inspect their weapons/magazines anyway (where the button is), so it shouldn't be getting in the way of the usual gunplay and its atmosphere.
There's a generic shaky sound to this effect, so other party knows, that ammo is shuffled. It seems okay, but I would like to have a more satisfying sound effect, but I haven't found one.
Media

[ X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

Make sure to take this Changelog template out of the comment block in order for it to show up.
:cl: volodius

add: Ammo can be shuffled to play Buckshot Roulette

So yes, seems like the only way to implement such feature would be spawning ammo and removing unspawned, which is not ideal.
I'll try to implement just that soon with some basic limitations so it cannot be abused - while it's unlikely that someone'll be using macros to spawn a ton of ammo while standing on top of 1000 ammo boxes, I probably should add that then.
If it'll end up bad for whatever reason, I'll close this PR

I would say you could just add a reskinned revolver prototype that has a whitelist for ShellShotgun but the RevolverAmmoProvider system needs reworking because it doesn't use all the properties from cartridges like projectile count.

I'll close this PR for now due to following reasons:
Execution damage multiplier makes it unfair to play using regular gun - shooting opponent  will deal non lethal amount of damage, while shooting yourself instantly crits, making it not fun to play
This and shenanigans with hackish spawn of unspawned ammo would make that implementation of ammo shuffle not fun to play so it's probably not worth even to rely on gunsystem features here.
I kinda get now why in ss13  russian revolver is not a gun, maybe I'll look forward to something closer to this.
I'll see how I can implement buckshot in a more meaningful way in a different PR in future.


WOW + mCustomScrollbar = conflict

I have trouble with WOW when using mCustomScrollbar
I found the reason, it is
.mCustomScrollBox{
		position: relative;
		overflow: hidden;
		height: 100%; <----------

when i remove that hight, custom SB does not display.
Help pls, what i shold do with that problem ?
sry for bad eng XD

Solved

How do you solved it?


"bottom" position won't let me give margin bottom(offset from bottom?)

When I set the modal position with "bottom", I can't move it at all from the bottom. I need to have a marginBottom which does not work, in comparison to marginTop that works with position "top"

Using bottom instead of marginBottom works.


"All Forms" string in top nav

Description
Missing consistency in top nav.
Steps to reproduce

Create a form or go in an existing form to edit
Look at the top nav bar
Notice "All Forms [FR]"
Toggle to English
Navigate through the site and notice changes in reference between "Forms" / "All Forms" / "All forms"

Details

Browser: Edge
Operating system: Windows
Environment: App router PR
Language: French
Viewport: Tablet

Expected behaviour
Forms or All forms in English consistently throughout
Formulaires or Tous les formulaires in French consistently throughout
Screenshots or videos

Let's go with "Forms" in EN and "Formulaires" in FR

Fixed in PR #340


Add StackedTextColor option

Add StackedTextColor options for three action buttons.
Changes

positiveStackedColor, positiveStackedColorRes, positiveStackedColorAttr for positive button
negativeStackedColor, negativeStackedColorRes, negativeStackedColorAttr for neutral button
negativeStackedColor, negativeStackedColorRes, negativeStackedColorAttr for negative button
Attrs: md_positive_stacked_color, md_neutral_stacked_color and md_negative_stacked_color

Why
I was build a dialog like this:

But the button text gets long, then this dialog becomes this:

At first I though the positive button is missing, so I dug a while,  found it's not a bug:

Sure I can change the background with btnSelectorStacked, but it's better to offer the option to change text color while it's stacked.

Version 2.0 is under development as a rewrite, so this will no longer be applicable.


Unify signatures of mesh constructors

It may be nice to unify the signatures of some mesh constructors or provide additional convenience constructors. This can be nice when testing different meshes, e.g., for #1191. Right now, we have
julia> TreeMesh(

TreeMesh(coordinates_min::Real, coordinates_max::Real; kwargs...) in Trixi at /home/hendrik/.julia/dev/Trixi/src/meshes/tree_mesh.jl:161
TreeMesh(coordinates_min::Tuple{Vararg{Real, NDIMS}}, coordinates_max::Tuple{Vararg{Real, NDIMS}}; n_cells_max, periodicity, initial_refinement_level, refinement_patches, coarsening_patches) where NDIMS in Trixi at /home/hendrik/.julia/dev/Trixi/src/meshes/tree_mesh.jl:93
TreeMesh(::Type{TreeType}, args...) where {NDIMS, TreeType<:Trixi.AbstractTree{NDIMS}} in Trixi at /home/hendrik/.julia/dev/Trixi/src/meshes/tree_mesh.jl:80

julia> StructuredMesh(

StructuredMesh(cells_per_dimension, faces::Tuple; RealT, periodicity) in Trixi at /home/hendrik/.julia/dev/Trixi/src/meshes/structured_mesh.jl:86
StructuredMesh(cells_per_dimension, mapping; RealT, periodicity, unsaved_changes, mapping_as_string) in Trixi at /home/hendrik/.julia/dev/Trixi/src/meshes/structured_mesh.jl:47
StructuredMesh(cells_per_dimension, coordinates_min, coordinates_max; periodicity) in Trixi at /home/hendrik/.julia/dev/Trixi/src/meshes/structured_mesh.jl:120

julia> P4estMesh(

P4estMesh(trees_per_dimension; polydeg, mapping, faces, coordinates_min, coordinates_max, RealT, initial_refinement_level, periodicity, unsaved_changes) in Trixi at /home/hendrik/.julia/dev/Trixi/src/meshes/p4est_mesh.jl:146

For example, it wold be nice to just replace TreeMesh by StructuredMesh or P4estMesh and get something that just works.

Bumping to add that we should do the same for DGMultiMesh constructors. For example, cells_per_dimension is a keyword argument here
    DGMultiMesh(dg::DGMulti; cells_per_dimension,
                coordinates_min=(-1.0, -1.0), coordinates_max=(1.0, 1.0),
                is_on_boundary=nothing,
                periodicity=ntuple(_ -> false, NDIMS))

but not for the StructuredMesh equivalent
    DGMultiMesh(dg::DGMulti{NDIMS}, cells_per_dimension, mapping;
                is_on_boundary=nothing,
                periodicity=ntuple(_ -> false, NDIMS), kwargs...) where {NDIMS}

Yes, it would definitely be great to unify these :+1:

I agree, I also do not like that for P4est meshes we explicitly supply the dimensionality as a type (not sure if this is the correct Julian term), i.e., P4estMesh{2} while for TreeMesh, StructuredMesh and DGMultiMesh the dimensionality is inferred from the arguments while again for the unstructured 2d mesh the constructor reads UnstructuredMesh2D.

So something I particularly dislike is the order of the coordinates and the argument that governs the discretization accuracy, i.e., initial_refinement_level, cells_per_dimension, trees_per_dimension which is different for the TreeMesh vs Structured/P4estMesh.
Also with #2129 around the corner we should add the RealT keyword for the TreeMesh. Might be the way to get #2129 itself non-breaking.

In #2129 the RealT keyword got introduced to the TreeMesh constructor:
julia> TreeMesh(

TreeMesh(::Type{TreeType}, args...; RealT) where {NDIMS, TreeType<:AbstractTree{NDIMS}}
TreeMesh(coordinates_min::Tuple{Vararg{Real, NDIMS}}, coordinates_max::Tuple{Vararg{Real, NDIMS}};
n_cells_max, periodicity, initial_refinement_level, refinement_patches, coarsening_patches, RealT) where NDIMS
TreeMesh(coordinates_min::Real, coordinates_max::Real; kwargs...)

StructuredMesh(cells_per_dimension, faces::Tuple; RealT, periodicity)
StructuredMesh(cells_per_dimension, mapping; RealT, periodicity, unsaved_changes, mapping_as_string)
StructuredMesh(cells_per_dimension, coordinates_min, coordinates_max; periodicity)


why the bot dont start???

i follow all the instructions but the bot are not starting (i use replit. release v1.0.4)

Can i see the folders/files by using 'ls' in the cmd?


Added translation to Setup.vue

Added translation to Setup.vue, i think that should be now everything.
New i18n keys:
Create your admin account
Repeat Password
These might need a translation into zh-HK.
Also because of #281, should i redo all the keys and move them to the en.js?

Also because of #281, should i redo all the keys and move them to the en.js?

Not very important in the current stage. I plan to write a script that add missing key in all language files. It could save a lot of time in the future, and it is easier for translators to know what have not translated yet in the file.

Also because of #281, should i redo all the keys and move them to the en.js?

Not very important in the current stage. I plan to write a script that add missing keys in all language files. It could save a lot of time in the future, and it is easier for translators to know what have not translated yet in the file.

Why in all files? Just for en.js. Missing translations will be visible by warnings in console


Add microbenchmark of single-threaded CQ operations

Sample output:
~/grpc (bm_cq *$%) make CONFIG=opt bm_cq -j ; and bins/opt/bm_cq --benchmark_min_time=10
make: Circular /usr/local/google/home/ctiller/grpc/objs/opt/test/cpp/util/create_test_channel.o <- /usr/local/google/home/ctiller/grpc/libs/opt/libgrpc++_test_util.a dependency dropped.
make: Nothing to be done for `bm_cq'.
I0214 12:42:01.718212715  110713 ev_epoll_linux.c:93]        epoll engine will be using signal: 40
D0214 12:42:01.718270025  110713 ev_posix.c:107]             Using polling engine: epoll
Run on (12 X 3501 MHz CPU s)
2017-02-14 12:42:01
Benchmark                     Time           CPU Iterations
-----------------------------------------------------------
BM_CreateDestroyCpp         133 ns        133 ns  104472462
BM_CreateDestroyCore         94 ns         94 ns  151686091
BM_Pass1Cpp                  99 ns         99 ns  141593783
BM_Pass1Core                 94 ns         93 ns  150366025

Now based on #9693

Merged with head

Ping

Reassigning since Sree is out

Jenkins: test this please

Failed: #9727


No license specified

Hi!
I’d love to use this library, but there’s no license specified. Would you mind committing one?
Thanks!

on their NPM homepage it says MIT

Thanks - 7 months ago we needed an actual LICENSE text checked in to the repo, but since then we’ve been told we can rely on the package.json field.
You may still want to check in a LICENSE file for completeness though.


Add push and pull steps for Azure Blob Storage

Adds push_to_azure_blob_storage and pull_from_azure_blob_storage project steps to allow Azure Blob Storage to be used for deployment storage and retrieval. New steps follow the pattern established for existing push and pull steps.
Cleans up docstrings and field description for AzureBlobStorageCredentials
Adds logo and documentation URLs to AzureContainerWorker

Closes #82
Closes #83
Example

prefect_version: ...
name: ...

push:
    - prefect_azure.deployments.steps.push_to_azure_blob_storage:
        requires: prefect-azure
        container: my-container
        folder: my-project
        credentials: "{{ prefect.blocks.azure-blob-storage-credentials.dev-credentials }}"

pull:
    - prefect_azure.deployments.steps.pull_from_azure_blob_storage:
        requires: prefect-azure
        container: "{{ container }}"
        folder: "{{ folder }}"
        credentials: "{{ prefect.blocks.azure-blob-storage-credentials.dev-credentials }}"

Screenshots

Checklist

[x] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[x] Includes tests or only affects documentation.
[x] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[x] Includes screenshots of documentation updates.

Run mkdocs serve view documentation locally.

[x] Summarizes PR's changes in CHANGELOG.md

@serinamarie I added some test coverage based on your suggestions and removed references to 'projects' in this PR, so it should be ready for another review.


Support Querying metrics for multiple resources

Support Querying metrics for multiple resources

This may well already be possible, however, I was not able to find or understand.
Is it possible to get metrics of multiple resources in a single call/query?
For eg. metrics for a list of VMs.
or
metrics for set of VMs of a given resource-group / subscription?

Hi @vipinvkmenon thanks for this issue.
I am not quite sure what is metrics referring for, but as far as I know, Azure only supports get the instance view of one VM at a time.
You could LIST the VMs in a resource group or subscription, but it gives less information than instance view.

Hi @ArcturusZhang .
Maybe I'll give you a context.
Coming from the AWS world.....
If I were to run the command
aws cloudwatch list-metrics --namespace AWS/EC2 --metric-name CPUUtilization

From Cloudwatch I would get the metric-value CPUUtilization for all the VMS present that particular region.
And if run
aws cloudwatch list-metrics --namespace AWS/EC2

Then all metrics related to VMs (at that time-instant)
(the cli is preconfigured with the region along with Credentials)
In their SDK the function is GetMetricData.
In the AZURE world, I would imagine something similar to this under Resource-Groups (if not subscriptions)?
For now, It looks like I would have to get the list of all resources and then Loop through each resource for the resource ID and then get the metrics, Which seems like a lot of API calls and a non-elegant idea... Hence the question.
Is there a better way to approach. (I may have failed to find the right APIs maybe?)

For general metrics the Azure Monitor team is the right team to answer these questions.

Hi @scottaddie any update for batch query metrics API?
We need this feature to resolve the performance issue when we heavily use the Metrics APIs.

Hi @scottaddie any update for batch query metrics API? We need this feature to resolve the performance issue when we heavily use the Metrics APIs.

Hi @yummydsky! We released the first beta for metrics batch query yesterday. You can use this feature with azquery v1.2.0-beta.1.

@gracewilcox Thank you!!! We will try this feature to see if it can enhance the performance. Thanks again!!


Objdump

Develop objdump for better diagnostics
todo list

[ ] Program entry (main function) & arguments
[ ] Implement features from modifying a copy of decode
[ ] Layout
[ ] Documentation

In objdump branch


Tree map fixes

Added lots of various fixes for TreeMap.
Explanation. I get https://github.com/pulseenergy/harmony-treemap/blob/master/src/test/java/org/apache/harmony/java/util/TreeMapTest.java this file (Apache 2.0 licensed), renamed it to TreeMapTest1, adapted to our testcases and noticed multiple failures. Then I fixed them one by one.
So, now 2 options are possible:

Have two test files for TreeMap, one normal and one "verbose".
Remove this TreeMapTest1 file.

I don't like the idea of committing this TreeMapTest1, but if you don't want to merge it with TreeMapTest, at least please resolve checkstyle issues.

Could you please fix checkstyle in gradle?
When I'm running ./gradlew check it fails with:
FAILURE: Build failed with an exception.

* What went wrong:
Could not load the value of field `aliasSet` of `sun.nio.cs.UTF_8` bean found in field `charset` of `org.gradle.api.internal.resources.FileCollectionBackedTextResource` bean found in field `config` of task `:classlib:checkstyleMain` of type `org.gradle.api.plugins.quality.Checkstyle`.
> java.lang.reflect.InvocationTargetException (no error message)

What Java version are you using? Is it 21? Gradle does not support Java 21 yet, so you should run from 17

Yes, it's 21.

Still, it worked earlier on 21. But ok, will downgrade for tests.

I did not change anything significant here. The error is in Gradle, and it clearly states, that Java 21 is unsupported. Gradle doc also says that there's no support for running Gradle on Java 21 yet. I don't know what has changed, perhaps some random caching issues.

Now when I'm compiling with 17 Java it fails with
> Task :tests:compileTestJava FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':tests:compileTestJava'.
> error: invalid source release: 21

Please, read this article: https://docs.gradle.org/current/userguide/toolchains.html
Personally, I run Gradle from IDEA and specify JDK 17 as Gradle run JDK, but project has both 17 and 21 installed, so Gradle toolchains work properly. If it's not an option, I don't have a quick answer.
You can also try keep running on 21, but play with these lines in gradle.properties:
org.gradle.caching=true
org.gradle.unsafe.configuration-cache=true

especially, with the second one

Thanks, setting this properties helped. Now checkstyle and build are successful.


Random output

We can use a special flag for random output, using the blockhash & order of transactions in the block to determine the order of the output. This is used in randomness & casino
edicts: [
 {
  id: 1234,
  amount: 1000,
  output: RANDOM
 }
]

Good idea!

Moving issues to the ord repo.


Delete robots.txt

By using Jekyll sitemap plugin, you already get both

robots.txt
sitemap.xml

Thanks, and it will be fix in next commit


Commit move tolerance to method signature for later use

Here you go @nwilliams-kobold , clean up of the hard-wired tolerance

@domfournier You don't need to keep two separate tolerances?

Seems to work with the same value. Not sure why devin added two

We can ping dccowen on your PR to confirm the change


Variable Observers and External Functions support

Actually, calling ink functions from lua is also be supported by accident (see Story:EvaluateFunction() ), but not tested nor is it documented. Consider that as a "hidden feature"

Should we call this one 1.0 ? :)

Should we call this one 1.0 ? :)

I think yeah in terms of functionality it can be 1.0
Anyway here are the things that I think are still missing, but not essential :

docs for less commonly used functions
better getter/setter handling (should use some metatable trick)
contribution guide
global callbacks (as stated in the modified readme)

Let's goo !


Stream->channel renames in zerver/webhooks/ docs

Note: This doesn't change the occurences in the Zapier integration doc,
since they refer to pieces of the UI in Zapier, which may still be using
the "Stream" terminology.

Merged, thanks @mateuszmandera!


URI malformed error

The following error happened in some builds:
Failed during font subsetting
populate transform: URI malformed
    /opt/build/repo/node_modules/urltools/lib/index.js:140:18 Object.exports.fileUrlToFsPath
    /opt/build/repo/node_modules/assetgraph/lib/assets/Asset.js:1482:20 makeRelativeToCwdIfPossible
    /opt/build/repo/node_modules/assetgraph/lib/assets/Asset.js:1489:9 Image.get urlOrDescription [as urlOrDescription]
    /opt/build/repo/node_modules/assetgraph/lib/transforms/logEvents.js:57:53 outputMessage
    /opt/build/repo/node_modules/assetgraph/lib/transforms/logEvents.js:142:9 AssetGraph.<anonymous>
    events.js:198:13 AssetGraph.emit
    /opt/build/repo/node_modules/assetgraph/lib/AssetGraph.js:169:12 AssetGraph.warn
    /opt/build/repo/node_modules/assetgraph/lib/transforms/populate.js:42:22 processAsset

Do you know what the error might be?

Looks like we are encountering either an invalid URI, or decodeURI gets an unexpected character as part of the URI, which it throws on.
The stack indicates that something probably went wrong during population of the asset, so a warning is emitted, but the warning handler throws while attempting to create the error output while running https://github.com/assetgraph/urltools/blob/master/lib/index.js#L138-L148
It looks like we need to build in a few more safeguards, because at the very least our error handler should be able to display the relevant error message it attempted to.
@papandreou This seems like something for Assetgraph core


Bluetooth: Additional 1.9.2 fix

@cvinayak @jhedberg added last fix to pass conformance

Waiting for EBQ results before approving

All 271 conformance tests passed.


likes/dislikes stay when cycling between specific posts

problem was with comments too. has been fixed by setting :key=postId on specific post to force reload component


Finetune

Can we support instruction finetune?

Nothing special is required to do instruction-finetuning. It's the same as normal finetuning, just with different data.
if you're interested in RLHF, that's not something we plan on supporting, however there are libraries such as trl and trlX that support this. The later library has a WIP integration with GPT-NeoX

Hi @StellaAthena, I think you mean (If I'm not mistaken) to line up the instruct data as an append of instruction and response and continue with regular LM training. However, for most instruction tuning, it is standard to mask the gradient updates for the instruction and only backdrop them for the response. Does the framework make this possible?

There's an open PR for this here. We also have started implementing RLHF features since my comment, e.g., this and this.

Hi @StellaAthena, I think you mean (If I'm not mistaken) to line up the instruct data as an append of instruction and response and continue with regular LM training. However, for most instruction tuning, it is standard to mask the gradient updates for the instruction and only backdrop them for the response. Does the framework make this possible?

There's an open PR adding support for this here. We also have started implementing RLHF features since my comment, e.g., here and here.

Thanks for the reply! Will check it out


use the resourceapply methods in the cert rotation

Our resourceapply methods emit events and handle the update/create logic.  This updates the cert rotator to use it.
/assign @mfojtik

LGTM, good changes, fix the import order and feel free to tag.