_this.emit is not a function

I'm trying to run the yandex-search example you have on the main page, but it's coming up with this error:
✘ yandex-search plain [PhantomJS]
_this.emit is not a function
Total: 1 Passed: 0 Failed: 1 Skipped: 0 Retries: 0
Critical error while running " yandex-search plain" in PhantomJS:
TypeError: _this.emit is not a function
at /Users/van.patton/node_modules/gemini/lib/capture-processor/screen-shooter.js:31:23
at _fulfilled (/Users/van.patton/node_modules/q/q.js:834:54)
at self.promiseDispatch.done (/Users/van.patton/node_modules/q/q.js:863:30)
at Promise.promise.promiseDispatch (/Users/van.patton/node_modules/q/q.js:796:13)
at /Users/van.patton/node_modules/q/q.js:604:44
at runSingle (/Users/van.patton/node_modules/q/q.js:137:13)
at flush (/Users/van.patton/node_modules/q/q.js:125:13)
at doNTCallback0 (node.js:430:9)
at process._tickCallback (node.js:359:13)
I start phantomjs using phantomjs --webdriver=4444
then I run the test using   gemini gather test
.gemini.yml
rootUrl: http://yandex.com
browsers:
PhantomJS:
desiredCapabilities:
browserName: phantomjs
test.js
var gemini = require('gemini');
gemini.suite('yandex-search', function(suite) {
suite.setUrl('/')
.setCaptureElements('.main-table')
.capture('plain')
.capture('with text', function(actions, find) {
actions.sendKeys(find('.input__control'), 'hello gemini');
});
});

nm just installed 1.1.2 and its working now

yes, sorry. It was bug in the gather command in v1.1.1 version. Was fixed in v1.1.2


quat.lerp is not sign-preserving

I was wondering why I was seeing incorrect results with quat.lerp, until I realized it's just a vec4 lerp and isn't a quaternion lerp at all! A proper quaternion lerp would look something like this:
export function lerp(out, a, b, t) {
  var ax = a[0],
      ay = a[1],
      az = a[2],
      aw = a[3];
  var bx = b[0],
      by = b[1],
      bz = b[2],
      bw = b[3];
  var cosom, scale0, scale1; // calc cosine

  cosom = ax * bx + ay * by + az * bz + aw * bw; // adjust signs (if necessary)

  scale0 = 1.0 - t;
  scale1 = cosom >= 0.0 ? t : -t;

  out[0] = scale0 * ax + scale1 * bx;
  out[1] = scale0 * ay + scale1 * by;
  out[2] = scale0 * az + scale1 * bz;
  out[3] = scale0 * aw + scale1 * bw;
  return out;
}

( Reference, if necessary: https://fabiensanglard.net/doom3_documentation/37725-293747_293747.pdf#page=4 )

Oh, that is a bug indeed. I'd gladly accept a PR that fixes this.
(Other reference for myself https://github.com/FlaxEngine/FlaxAPI/blob/master/FlaxEngine/Math/Quaternion.cs#L743 )

And one more: Unity as well (https://github.com/Unity-Technologies/Unity.Mathematics/blob/master/src/Unity.Mathematics/quaternion.cs#L563-L572).
This is far from the first time I've seen this :) . I'm slightly worried about back-compat, but I think pretty much everybody wants the fixed behavior.

If you're worried about backwards compatibility, I can add this to the 4.0 milestone.
(Though, the 4.0 milestone isn't progressing very fast, because on one hand I'm waiting for WebAssembly to become better and on the other hand, I'm busy with other projects and pretty much have abandoned WebGL. Thus, I'm not using gl-matrix for anything.)


Probably done incorrectly

This is my first pull request and im pretty sure i did not do it right

Please note there's an authors.txt file you may wish to add your name to now! :-)


Cosmos CFP: fix for race condition issue

ChangeFeedProcessor: fix a race condition issue where the lease document has been updated while we are trying to write the current state like when check-pointing the continuation token.

/azp run cosmos - java - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests


Can typescript promises be used, and do they translate nicely to Lua coroutines?

Can I just use promises and async / await with this or not?

Yes.


Created new section with link to bulma website

Created new section with link to bulma website for more information

[x ] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @whiskerwind
It seems that similar changes have already been accepted earlier for this article you're editing, sorry about that. 😓
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Add new jbrowse formula version 1.12.1

Adding a new cask

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

@fanquake went ahead with this change!


Roles command does not exist

Torus roles command, as per docs, does not seem to be implemented :/
Is there another way to add permissions to a machine?
» torus machines roles create local
No help topic for 'roles'
» torus machines
NAME:
   torus machines - Manage machine for an organization

USAGE:
   torus machines command [command options]

COMMANDS:
     create   Create a machine for an organization
     view     Show the details of a machine
     destroy  Destroy a machine in the specified organization
     list     List machines for an organization

OPTIONS:
   --help, -h  show help

Installed via Homebrew
» torus -v
CLI         0.15.0
Daemon      0.15.0
Registry    0.20.0

You need to upgrade to the latest version of torus v0.22.0!

That'll be it, thanks Ian


[Feature Request] Intermediary for separate networks

I would like to be able to create a network to network link without having to rely on an external storage source to do so. This could be resolved a few ways however IMO the easiest solution is to allow Flux Plugs to connect directly to flux points.  Then you could set up a Point set to "Network A" feeding a Plug set to "Network B".  Currently the only way to do this is to have energy storage from another mod hooked to a point and a plug.

So you are asking to be able to send energy to a separate network?

Yeah that's the gist of the request. Currently it can be done just by hooking a plug from Network A to say an Ender IO capacitor bank, and then outputting from the capacitor to a point on Network B. However I'd like to see this functionality built into the mod directly as Ender IO has different (lower) RS transfer limits than your mod does.


Add support for multiple fields in CUDA

Add support for multiple fields in CUDA
Description
This PR adds support for multiple fields in CUDA, by making the current radix2_dit_butterfly kernel a template, that will be wrapped by field-specific implementations like radix2_dit_butterfly_stark256. On the rust side all libraries are loaded into the device at CudaState::new, and later matched by field name.
Type of change
Please delete options that are not relevant.

[ ] New feature
[ ] Bug fix
[ ] Optimization

Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] This change is an Optimization

[ ] Benchmarks added/run

Codecov Report

Merging #320 (0c1930e) into cuda_fft (3a02a71) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           cuda_fft     #320   +/-   ##
=========================================
  Coverage     96.29%   96.29%
=========================================
  Files            92       92
  Lines         11621    11621
=========================================
  Hits          11191    11191
  Misses          430      430

Impacted Files
Coverage Δ

...field/fields/fft_friendly/stark_252_prime_field.rs
0.00% <0.00%> (ø)

math/src/field/traits.rs
78.37% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Bugfix/windows

This PR fixes a few issues with the windows build, namely:
1: #define guards to bring in rmdir or _rmdir depending on platform
2: Lowers the found points comparison threshold in the inversetransform unit test; likely due to round-off
3: Modifies the appveyor script to build test target RUN_TESTS which works on windows
Good to go!

@jonesholger, before running the RUN_TESTS target with cmake, we need to build all the required executables which can be done by building the exec cmake target:
cmake --build build_serial --target exec

@v-dobrev Thanks for catching that right away.

This PR should fully resolve #784.


Single CLI command with subcommands for deploy/run/restart/render

We should make sure that the new command is easy to test (and well tested).
Evaluate:

https://github.com/erikhuda/thor
https://github.com/Shopify/cli-kit

Add this to the list too https://github.com/commander-rb/commander

@KnVerey Has a library on which to build this been chosen? I would be happy to prep a PR for this if so.

I think Thor makes the most sense for this project. However, please note that we have a few other changes that we'd to take the opportunity to make in this transition, beyond just splitting things like kubernetes-deploy -> krane deploy. Unfortunately these aren't documented in a single place yet. I've added a work item to our internal tracker to produce a proper design document for this.

@KnVerey I have a little time available to deal with this. Is it worth me making a start? Do you have any references to those other changes you'd like to make?

Hey @benlangfeld thanks for letting me know you'd like to start this soon. I started a draft of the design doc and will post it here for comment by the end of the week.

Here is the proposed design as promised @benlangfeld : https://docs.google.com/document/d/1oInUsKplYGNWTymPY48xtDCx3x4Cwc-phcEKUkt2v4U/edit?usp=sharing. Let me know if you have any trouble accessing it. Everyone should be able to add comments and suggestions.
cc @Shopify/cloudplatform

Excellent, thanks @KnVerey . Here's what I propose as an approach for attacking this, each step being a PR with the intent of being merged before moving on to the following step (to reduce rebase overhead) and the possibility of being included in intermediate releases:

Add krane CLI in parallel with the existing CLI, as close to a pure addition as possible.
Add support for new annotation names, retaining backward compatibility with (but raising warnings on) the old annotation names.
Rename internals (class names, filenames).
Rename the project (gemspec).
Remove backward compatibility (old annotations, old CLI) some number of weeks later.

Does this sound reasonable to you?

Overall this sounds like a good approach. 2 minor concerns:

Having to do something like require 'kubernetes-deploy'; Krane::DeployTask.new is odd. We may want to merge step 3 and step 4 at the same time.
(5) It's possible we'll want to support the old annotations longer than the old CLI, so we may want to break those apart, but I think we can discuss that when you're ready to start step 5.

Krane 1.0.0 has been released tracking removal of annotations in https://github.com/Shopify/krane/issues/526


[release-4.14] OCPBUGS-36452: Can't install operator on 4.15 after uninstalling it on a prior version

Manual Cherry pick of commits from https://github.com/openshift/operator-framework-olm/pull/758

/test images

/label cherry-pick-approved
/label qe-approved

/jira refresh
/retest

/retest

/retest

/retest

/retest

/approve
/lgtm
/label backport-risk-assessed

/unhold


Loss cardinality does not make sense in the setting when using focal loss

Hi,
I see that starting from deformable DETR, it doesn't make sense to have cardinality loss. In the original DETR paper, the class_embed head outputs num_classes + 1, to also output a value for the no object class, and if we keep the same structure then the Cardinality loss specifically the line would work as commented in the code. However, in all the versions of DETR from IDEA-Research that uses focal loss, uses class_embed head that only outputs num_classes, so no output for the no object class, and then the Cardinality loss instead considers the last class the no object class. Can you explain its use?
Also, why did you make the choice to not use the no-object class for focal loss, is there some explanation that I am missing?

I have the same question. I just understand that if add no-object it will cause category imbalance problem.
I know the focal loss is aimmed to solve the problem, but I find the prediction still closse to no-object class. so I feel confused a lot.
Do you understand how to explain the question? if so, can you explain it for me?
Thanks!


Add IPFire

The Open Source Firewall Distribution
https://www.ipfire.org/
That one is not really following the style of the existing distributions. It feels not very smooth.

Good find, it looks like we can add it.  We can probably skip all the loading and load the bits directly using the sources from here:
http://boot.ipfire.org/menu.cfg?arch=x86_64&platform=pcbios

Yeah, your latest update is along the lines of what I was thinking of doing, just pointing at the kernels directly and bypassing all of their menus.  Let me know once it's ready to review and I'll kick the tires on it.
Thanks!

Closing since I put together an updated PR.  Thanks for your work on this!
https://github.com/antonym/netboot.xyz/pull/221


Update streamer list

shenetworks: HTB + THM streamer, content creator from other platforms who uses twitch for htb / thm streams (and other tech-thing streams)

Thank you much @koronkowy


Step 13 "storage-rules.json" ?

Hi. Thanks the nice tutorial.
Here is my confusing point. These are typo?

Related info:

Goo dcatch thanks! I'm updating the instructions to have "storage.rules" everywhere :)


Pressure values exactly half of expected value?

Temperature is correct.  Pressure values are around 505 which is half what it should be.  Any ideas how this could happen?

Thanks for this issue,
I'll try to pick this up tomorrow. I do not recall seeing this before, also in other issues this is never mentioned.
Do you use 3V3 or 5V device?
Can you tell more about your setup?

It is connected to a Feather M4 Express with 3V3 power.  I'm calling the library in the default configuration with no arguments:
MS5611.begin() in setup and MS5611.read() and MS5611.getPressure() in loop.

Had a quick look,
It might be in line 133 in ms5611.cpp
C[3] = 3.90625E-3;      // TCS      = C[3] / 2^6
==>
C[3] = 0.015625      // TCS      = C[3] / 2^6
3.90625E-3 = 1/(2^8)

mmm the above should give an error factor 4.
tomorrow I will dive into the datasheets

Just in case it helps, these are the PROM values:
MS5611_LIB_VERSION: 0.3.8
140
38252
37147
24739
23894
32339
28498

Thanks, those are device specific calibrations. Cant draw conclusions from them.
Q: do you have a link where you bought the device?
Just to check if the type is exactly the same.
A factor 2 can also be caused if a bit is missing in xommunication. Do you have appropriate pull ups on data and clock line? .just to check.

Good morning,
Analysis
I have looked this morning into two documents, a datasheet and an application note. (AN520_004 www.meas-spec.com Aug. 09, 2011) and I noticed the following:

that the comment in my code 2^6 should be 2^8 according to the datasheet.
that the constants used in the algorithm to calculate the pressure differ between the two documents.

Conclusions (so far)
Point 1 is easy to fix, and as the constants in thecode are in line with the datasheet, and there was no factor 2 problem before I consider them correct.
Point 2 is remarkable and indicates that there might be (at least) two different series of the sensor.
Actions

I will make a test33 branch with the "other constants" and if you can test this with your sensor that would be great.
The constant 0 in EEPROM is the manufacturer code and is not used in the library. I will make a function to access it so it is possible to see if there are differences between these numbers.

To be continued.

differences in the constants.
                                          datasheet     |    appNote
  C[0] = 1;
  C[1] = 32768L;          //  SENSt1   = C[1] * 2^15    |    * 2^16
  C[2] = 65536L;          //  OFFt1    = C[2] * 2^16    |    * 2^17
  C[3] = 3.90625E-3;      //  TCS      = C[3] / 2^8     |    / 2^7
  C[4] = 7.8125E-3;       //  TCO      = C[4] / 2^7     |    / 2^6
  C[5] = 256;             //  Tref     = C[5] * 2^8     |    * 2^8
  C[6] = 1.1920928955E-7; //  TEMPSENS = C[6] / 2^23    |    / 2^23

C5 and C6 are for Temperature and they are the same.
The others are for pressure and they all differ a factor 2.
So it might explain your issue!

Please download and try - https://github.com/RobTillaart/MS5611/tree/test33

The test33 branch also contains updates for the coming 0.3.9 release.

rp2040 in build
changelog.md
experimental functions uint16_t getManufacturer() and uint16_t getSerialCode()

These should not interfere with the changed constants.

Added setMathMode(mode) to the branch to switch between the default mode and the mode with the constants used in the application notes. Although the latter are not tested yet I expect these to solve the issue.

You did it, nice work!

I'm not 100% happy yet with the API so there might be minor changes.
I'm thinking of putting the call to reset() within the setMsthMode() call + some param checking.

Can you post the info of the type of device you have?
The n I can give it as example in the readme.md when to use the mathMode.

@sathibault
Refactored the library, now the mathMode is a parameter of reset(mathMode = 0). Default the datasheet mode for backwards compatibility, option mathMode =1 for the factor 2 pressure correction.
The separate functions to set the mode are gone (no need to call reset() after setting the mode anymore.
The interface of the lib is almost identical so most people won't notice.
And for those who need the factor2 correction, they only need to add one call in the setup() of their code.
I will merge later today

Can you post the info of the type of device you have? The n I can give it as example in the readme.md when to use the mathMode.

I don't really have any detail on it.  It came from a drone development kit and seems to be a custom board.  There's nothing readable on the chip.  The PROM reg 0 is 140.  Not sure if that is an indicator or not.

I don't really have any detail on it.

Thanks for checking, most important is that the sensor now works as intended and that I have docs that describe the math.
if you are interested - https://www.amsys-sensor.eu/sheets/amsys.fr.an520_e.pdf

The PROM reg 0 is 140. Not sure if that is an indicator or not.

I have 5 or 6 dumps and they are all different. I do not have any clue what reg 0 means.
The other regs are at least in the same orde of magnitude so they are calibration data.


[BUG] Database Corruption After Live Migration of Windows Server 2022 with SQL Server

Bug: Database Corruption After Live Migration of Windows Server 2022 with SQL Server
Description
During the process of live migrating a Windows Server 2022 with SQL Server installed on it, a bug is encountered where the database becomes corrupted. To avoid this issue, the only known workaround is to manually stop the virtual machine (VM) and load it onto a new host.
Steps to Reproduce

Have a cluster with at least three nodes.
Run a VM with Windows Server 2022 Standard and SQL Server 2019 installed on it.
Begin the live migration process or put the host into maintenance mode to initiate the live migration.
Ensure that the database on the VM is actively in use during the migration process.
Allow the live migration of the VM with Windows and SQL Server to complete.

Expected Behavior
The live migration should proceed without causing any database corruption.
Possible Solution
It would be helpful to have a feature to tag a VM to prevent it from being live migrated. This would allow users to safeguard against potential database corruption during live migration.
Environment

Harvester ISO version: v1.1.1
Baremetalx3 OVHCloud

You can specify the VM to run on on a specific node.  This will make it ineligible for live migration.  This does help.

Live Migration of Windows server 2022 cause WID corruption, which can broke RDP server manager, WSUS and other services.

Has anyone tested this data corruption problem on recent versions? I hope I can use live migrations on windows VMs again.


Bye gif pooja

Added another method of reply using GIF resolving issue #35

Thanks @poojasrinivass! 👍


Add LaunchDarkly (Actions) documentation.

Proposed changes
This PR adds documentation for the newly developed LaunchDarkly action destination (https://github.com/segmentio/action-destinations/pull/546).
Merge timing

ASAP once approved

Related issues (optional)

https://github.com/segmentio/action-destinations/pull/546
https://github.com/segmentio/action-destinations/pull/556
https://github.com/segmentio/action-destinations/pull/561

Thanks again @stayseesong!


Lodestar enagages all cores 100% when doing countdown for genesis

Describe the bug
When trying to run a lodestar interop locally, noticed that lodestar starts consuming 100% cpu on all cores

which returns to normal as soon as genesis countdown is finished and chain starts moving.

Bug seems to be lodestar doing an active wait on some multi threaded workers
Expected behavior
Should not engage CPU cores while genesis countdown

Steps to Reproduce
run local interop and observe cpu usage while countdown to genesis
cd packages/beacon-node
EL_BINARY_DIR=g11tech/geth:withdrawalsfeb8 EL_SCRIPT_DIR=gethdocker yarn mocha test/sim/withdrawal-interop.test.ts

OS: ubuntu 22.04

@g11tech Run locally and could recreate. Tho:

Extending the genesis delay the 100% CPU only lasts for about 10 seconds, then goes back to idle. So your observation that it lasts for the entire genesis waiting period so happens that the waiting period is about 10 seconds
I attached a CPU profiler and it was empty. There must be a bug or something so I can't tell what's taking CPU

@g11tech Run locally and could recreate. Tho:
* Extending the genesis delay the 100% CPU only lasts for about 10 seconds, then goes back to idle. So your observation that it lasts for the entire genesis waiting period so happens that the waiting period is about 10 seconds

* I attached a CPU profiler and it was empty. There must be a bug or something so I can't tell what's taking CPU

Could geth CPU be reported as node's process since it is run a child process? I can't think of anything on start that could max out CPU. We only multithread BLS sig verification

hmm yes, it goes back to idle, but CPU during this 10 sec period scales with number of cpu cores (tested on 24 core) and all cores engaged there as well.
I will try running geth separately as see what happens

@dapplion didn't notice with other EL clients so closing the issue as of now


Development: Add LTI Authentication for email address in use

[!WARNING]
Deploy only locally or on ts11! The LTI feature is only available on TS11.

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I followed the coding and design guidelines.

Motivation and Context

We identified a security issue where someone could potentially log in as any user, including admin, using just an email address.
Description

When a user attempts to access Artemis and isn't already logged in on the same browser, the Artemis server identifies this and sends a 401 error to the client, indicating that authentication is required. This error is then captured on the client side for appropriate handling. Conversely, if the user is already logged into Artemis on the same browser, the client recognizes this authenticated state and automatically redirects the user to the appropriate page without further prompts.
Steps for Testing

Prerequisites:

1 Student

User not logged into Artemis:

Log out from Artemis if you are already logged in with a user.
Log in to https://moodle.ase.in.tum.de/ using credentials:

username: student1
password: same as artemis_test_user_1

Navigate to "TS11 Authentication Test Exercise" in "LTI Testing Course" or simply go to https://moodle.ase.in.tum.de/mod/lti/view.php?id=127
Verify you are forwarded to login page
Login Artemis using artemis_test_user_1 credentials
Verify you are directed into the text exercise

User logged into Artemis:

Login to Artemis using artemis_test_user_1 credentials
Log in to https://moodle.ase.in.tum.de/ using credentials:

username: student1
password: same as artemis_test_user_1

Navigate to "TS11 Authentication Test Exercise" in "LTI Testing Course" or simply go to https://moodle.ase.in.tum.de/mod/lti/view.php?id=127
Verify you are directed into the text exercise

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

no change
Screenshots

Manual test on TS11 worked just like you described 👍

Manual tested on ts11.

User not logged into Artemis: Worked successfully
User logged into Artemis: Worked successfully


Display Textures in MatPlotLib

I have managed to read STL files and display them with Mat Plot Lib. However, all the textures in the files and fine details are lost. What could be wrong?
For example, this is how the original file looks like with different viewer:

This is how it looks like on our system with Numpy STL:

Unless you've activated one of the cleanup commands such as duplicate removal it shouldn't change the stl.
Without any code or stl files I can't say much more though

looks like matplotlib is patching the surfaces "wrongly". That can have a plethora of causes, so in terms of SO, a MVRE would be nice. :)

Thank you for your reply;
Here you're my code:
    def extract360(self, path_to_file, nr):
        ret = []
        directory = "LEGO"
        if not os.path.exists(directory):
            os.makedirs(directory)
        directory = "%s/%s" % (directory, nr.zfill(10))

        next_idx = len(os.listdir(directory))

        if not os.path.exists(directory):
            os.makedirs(directory)
        angels = [0, 45, 90, 135, 180, 225, 270, 315, 360]
        axes = [
            [0, 0, 0.5],
            [0, 0.5, 0],
            [0.5, 0, 0],
            [0, 0.5, 0.5],
            [0.5, 0, 0.5],
            [0.5, 0.5, 0],
            [0.5, 0.5, 0.5]
        ]
        i = 1
        for angel in angels:
            for axis in axes:
                # Create a new plot
                figure = plt.figure()
                plt_axes = mplot3d.Axes3D(figure)

                plt_axes.set_axis_off()

                # Load the STL files and add the vectors to the plot
                your_mesh = mesh.Mesh.from_file(path_to_file)

                your_mesh.rotate(axis, math.radians(angel))

                plt_axes.add_collection3d(mplot3d.art3d.Poly3DCollection(your_mesh.vectors))

                # Auto scale to the mesh size
                scale = your_mesh.points.flatten(-1)
                plt_axes.auto_scale_xyz(scale, scale, scale)

                # Show the plot to the screen
                f_p = "%s/%010d.jpg" % (directory, next_idx+i)
                plt.savefig(f_p)
                ret.append(f_p)
                plt.close()
                i += 1
        return ret

And here you're the STL file:
https://printabrick.org/bricks/30363/zip

I have used VTK for the same thing, and I get good results with Textures:
https://www.youtube.com/watch?v=2dk2k_9ubpg

I am glad that other packages yield good results, but step one is the same as I notice with a lot of others: repairing an invalid mesh. I added the fixed stl as well. I hope that one is okay enough for further discussion purposes.
Also attached to this reply: a picture snippet of my matplotlib plotted lego block. This without the axes_off feature or the autoscale. Just fyi, a somewhat similar plot can be generated from the original stl, albeit with some more distorted triangles.
So, matplotlib intricacies seem to be the issue here, rather than numpy-stl.
I am unable to reproduce the exact plot you showed at the start though, so it is going to be a tough cookie to help you out with the plotting issues.
lego.zip

As extra bonus, I'll add an example of misdefinition in the mesh:
stl.vectors[679] Out[268]:  array([[ -0.91843998, -22.21730995,  -7.6724    ],        [  0.        , -22.39999962,  -7.73329973],        [  0.        , -22.39999962,  -7.73329973]], dtype=float32)
As you can see, two of the points in the polygon have the same spatial coordinates, leading to a triangle with zero area (a line). It is now undefined how to handle normalization and other operations on this segment; so I am not surprised if scaling operations fail big time in this case. Also, in some cases if the flag is set, I guess these polygons will be cleaned up. There are at least 12 such polygons I could find.

Thank you very much.
1- How have you fixed the STL? Is there an automated utility for such?
2- Numpy STL apparently has no issue here, but what's the issue with MatPlotLib? I don't quite get it? Is it edgecolor? axis off?
Thanks again


fix: add "waytoodank" to the waytoodank emote keywords

Added "waytoodank" to the waytoodank emote keywords
Related to issue https://github.com/benawad/dogehouse/issues/1914

@TheOtterlord I was gone for a long time - can we merge more emotes?

@TheOtterlord I was gone for a long time - can we merge more emotes?

Not unless they remove the same number of emotes.
But we pretty much have a pause for now

ok

thank you

This is a fix, so I'm pretty sure this one is fine


feat: create a dedicated page for the faq

Checklist:

[X] I have read and followed the contribution guidelines.
[X] I have read and followed the how to open a pull request guide.
[X] My pull request targets the main branch of freeCodeCamp.
[X] I have tested these changes either locally on my machine, or GitPod.

Related to #49949

I turned the FAQ component into a FAQ page. When I was doing this, I also made sure to use the Spacer from the FreeCodeCamp UI.

Merge conflict removed

Probably could use a rebase/merge.

Thanks for working on the help page. The FAQ section is not sufficient to serve as a FAQ page for the website.
I think we should close this pr in favor of https://github.com/freeCodeCamp/freeCodeCamp/issues/49949#issuecomment-2470260286


Dev rares

added UI for MedicalRecord and made the MedicalRecord 100%

Codecov Report

Merging #58 (0a8c8e4) into dev (27e5bff) will increase coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev      #58      +/-   ##
==========================================
+ Coverage   84.84%   84.99%   +0.14%
==========================================
  Files          71       71
  Lines        3491     3485       -6
  Branches      163      163
==========================================
  Hits         2962     2962
+ Misses        508      502       -6
  Partials       21       21

Impacted Files
Coverage Δ

...cksNet.API/Controllers/MedicalRecordsController.cs
30.30% <ø> (ø)

src/DucksNet.Domain/Model/MedicalRecord.cs
100.00% <ø> (+21.42%)
:arrow_up:

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


Bundle response links are broken if ILocalhost.DefaultBase ends with /

Describe the bug
If the Spark configuration specifies a base address with a trailing /, notably covering the case of hosting FHIR at the domain root (e.g., https://localhost:5001/), all Bundle response Snapshot links are broken: they are created with doubled / characters, resulting in NotFound responses when the snapshots are requested.

NOTE: This is not seen in the Spark reference implementation example site because its base route ends with fhir.

To Reproduce
Steps to reproduce the behavior:

Initialize a Spark project. Make sure FHIR is hosted at the application root.
Provide a configuration where the SparkSettings.Endpoint is set to the application's root (e.g., http://localhost:5000/). (NOTE: Because Endpoint is a Uri, even if the Uri constructed without a path, e.g., http://localhost:5000, the Uri.ToString() will have the path /, e.g., http://localhost:5000/.)
Run the server and perform a search or transaction. For most useful result, use a request which will generate multiple response entries and provide a smaller _count parameter (so pagination will be needed to retrieve all results).
Examine the response Bundle.Link contents. You'll see something like the following (however, the host will match whatever you configured):

[{"relation":"self","url":"https://fhir-service:9443//_snapshot?id=c5107296-6dd4-4762-adcc-9809bd5c266d&_offset=0"},{"relation":"first","url":"https://fhir-service:9443//_snapshot?id=c5107296-6dd4-4762-adcc-9809bd5c266d&_offset=0"},{"relation":"last","url":"https://fhir-service:9443//_snapshot?id=c5107296-6dd4-4762-adcc-9809bd5c266d&_offset=3"},{"relation":"next","url":"https://fhir-service:9443//_snapshot?id=c5107296-6dd4-4762-adcc-9809bd5c266d&_offset=1"}]

Expected behavior
Expectation is that the links will contain no empty segment (i.e., /, instead of //) between the host and _snapshot path.
Example valid responses from Spark reference example server response:
[{"relation":"self","url":"https://spark.incendi.no/fhir/_snapshot?id=c85ca586-4b31-4cc4-88d2-91045d4a365c&_offset=6"},{"relation":"first","url":"https://spark.incendi.no/fhir/_snapshot?id=c85ca586-4b31-4cc4-88d2-91045d4a365c&_offset=0"},{"relation":"last","url":"https://spark.incendi.no/fhir/_snapshot?id=c85ca586-4b31-4cc4-88d2-91045d4a365c&_offset=64"},{"relation":"previous","url":"https://spark.incendi.no/fhir/_snapshot?id=c85ca586-4b31-4cc4-88d2-91045d4a365c&_offset=5"},{"relation":"next","url":"https://spark.incendi.no/fhir/_snapshot?id=c85ca586-4b31-4cc4-88d2-91045d4a365c&_offset=7"}]

Research
The problem appears to be caused by the manual construction of the snapshot base URL:
https://github.com/FirelyTeam/spark/blob/d9889ad7003c91eabfc6a302416fdf1ef4de8766/src/Spark.Engine/Service/FhirServiceExtensions/SnapshotPaginationService.cs#L209
At that line, the URL is constructed using baseurl = new Uri(_localhost.DefaultBase + "/" + FhirRestOp.SNAPSHOT).AddParam(FhirParameter.SNAPSHOT_ID, _snapshot.Id);. As a result, it appends an additional / if the ILocalhost.DefaultBase already ends with /.
I imagine that updating the line to baseurl = _localhost.Absolue(FhirRestOp.SNAPSHOT).AddParam(FhirParameter.SNAPSHOT_ID, _snapshot.Id); would resolve the issue, given that Localhost.Absolute already accounts for trailing / (see: https://github.com/FirelyTeam/spark/blob/d9889ad7003c91eabfc6a302416fdf1ef4de8766/src/Spark.Engine/Core/Localhost.cs#L22 ).

Thanks for the thorough report! Fix is on r4/master and stu3/master, and I'll put up a new release pretty soon.


Broken image in the examples README

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Steps:

Open https://github.com/mui/mui-toolpad/tree/master/examples/qr-generator
Check the screenshot

Current behavior 😯
No screenshot, just a JSON blob
Expected behavior 🤔
Screenshot visible in the markdown
Screenshot file included in the example folder
Context 🔦
This happens on all examples
Your environment 🌎
No response

It is happening because this .md file is being used for the docs page as well and this image is being rendered there.
I don't know how can I make image selection conditional from this .md file.


Routing by base path

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] other (Please do not submit support requests here (see above))

Current behavior:
Routing occurs only within one basic path. If the site has several applications, then for them it is necessary to define the paths separately.
Expected / new behavior:
Define the path along with the base path if necessary.
What is the motivation / use case for changing the behavior?
I do not want to configure paths in different places and load the necessary settings, when I could configure it in one place.
Other information
I have a solution (see https://github.com/lightapp24/angular.js/tree/routing-by-basePath_Href). If this feature is approved, I will prepare PR.

Sorry. Saying "several applications", I mean different pages that contain one and the same ng-app directive.

Sorry, I am still not sure what this is about. Can you give an example or describe the usecase in a little more detail?

OK, suppose we have two pages, let's name their profile and community.
Profile is located in http://example.com/profile and contain:
<html>
<head>
<!-- definition of the head -->
<script src="js/app.js"></script>
<script src="js/profile_route.js"></script>
</head>
<body ng-app="myApp">
<!-- part of profile content generated on the server side -->
<ng-view>
<!-- part of profile content that uses ng-view -->
</ng-view>
</body>
</html>

Community is located in http://example.com/community and contain:
<html>
<head>
<!-- definition of the head -->
<script src="js/app.js"></script>
<script src="js/community_route.js"></script>
</head>
<body ng-app="myApp">
<!-- part of community content generated on the server side -->
<ng-view>
<!-- part of community content that uses ng-view -->
</ng-view>
</body>
</html>

We have app.js:
var app = angular.module("myApp", ["ngRoute"]);

// some controllers, directives and etc.

profile_route.js
app.config(function($routeProvider) {
    $routeProvider
    .when("/", {
        templateUrl : "tmpl/profile/main.htm"
    })
    .when("/contacts", {
        templateUrl : "tmpl/profile/contacts.htm"
    })
    .when("/security", {
        templateUrl : "tmpl/profile/security.htm"
    });
});

community_route.js
community_route.js
app.config(function($routeProvider) {
    $routeProvider
    .when("/", {
        templateUrl : "tmpl/community/main.htm"
    })
    .when("/settings", {
        templateUrl : "tmpl/community/settings.htm"
    })
    .when("/security", {
        templateUrl : "tmpl/community/security.htm"
    });
});

This example of an incomplete SPA, it is something hybrid.
We have to load the router separately so that they do not conflict. The only way out of this situation is to use the base path (in an example this is /profile and /community) in routing. My solution uses the format "[/basePath1,/basePath2,...]/somePath", where "[/basePath1,/basePath2,...]" is an optional part. For this example, this will look like:
app.config(function($routeProvider) {
    $routeProvider
    .when("[/community]/", {
        templateUrl : "tmpl/community/main.htm"
    })
    .when("[/community]/settings", {
        templateUrl : "tmpl/community/settings.htm"
    })
    .when("[/community]/security", {
        templateUrl : "tmpl/community/security.htm"
    }).when("[/profile]/", {
        templateUrl : "tmpl/profile/main.htm"
    })
    .when("[/profile]/contacts", {
        templateUrl : "tmpl/profile/contacts.htm"
    })
    .when("[/profile]/security", {
        templateUrl : "tmpl/profile/security.htm"
    });
});

Sorry for my English.

I see what you mean now. I am not too keen on this change. If nothing else, if you have two distinct pages (different apps) it makes most sense to keep the routes separate (most people even separate their routes per view).
Complicating things (e.g. in terms of API surface, maintainance cost, LOC) just in order to allow (very few) people do what doesn't seem like a good idea (putting routes from different apps in the same file) is something I don't like.
Putting this in the Ice Box, but leaving it open to see what others think.

I agree that this is a feature that would only benefit very few developer and the added code / complexity is not worth implementing it.


Wrong Z-Wave entity shown in the overview "bubble" from devices with instanceid > 1

Home Assistant release (hass --version):
0.51.2
Python release (python3 --version):
3.5.3
Component/platform:
zwave
Description of problem:
According to my thread at https://community.home-assistant.io/t/z-wave-value-instance/25006/5?u=nodomain there seems to be a little UI bug that prints the wrong entity ID if the device has a instance ID >1. This led to some confusion with me trying to hide/regroup the device using customization.
In the "developer" view with all entities the device shows up correctly.
I assume something has been mixed up when changing the entity IDs some releases ago.
Expected:
Show the correct instance ID, e.g. sensor.dimmer_livingroom_energy_2 instead of sensor.dimmer_livingroom_energy
Problem-relevant configuration.yaml entries and steps to reproduce:
n/a
Traceback (if applicable):
n/a
Additional info:

Could you post an image of both the developer section where the device is showing correctly and the more-info where it shows incorrectly?

I'm having a similar problem with the Inovelli 2-channel switch. Sometimes I get one channel, sometimes the other, but since 0.60.1 I can't get all my switches all the time.

I have the same problem with the Inovelli 2-channel switch as well.  Sometimes it shows all the switches, sometimes it just shows one, and it shouldn't be overwriting "old" with "new" that is missing the sub-IDs
friendly_name: Dual Outlets Switch
new_entity_id: switch.dual_outlets_switch
node_id: 5
old_entity_id: switch.dual_outlets_switch_5_0_6
value_id:<PHONE_NUMBER>16354208
value_index: 0
value_instance: 6

Have the same problem with a fibaro double switch. While the old ids for the 2 switches will be different, the new entity ids are the same. I renamed the zwave node bed3_double_switch and get duplicate: switch.bed3_double_switch_switch


Add option for debug build to build hook

[ ] main
[x] 4-3-stable

Re-opening because we need to cherry-pick to main...


Fix tiny typo

Use the proper em dash. (Recall that "&" interpolates to the empty string. It's there so the digits of "2016" can follow the code point number of the em dash.)

Thanks!


"Make default payment method" option isn't showing up after you get error - reported by @Puneet-here

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to settings > Click on bank account
Click "Make default payment method"
Type wrong password > Enter
Tap on "Make default payment method" again

Expected Result:
Make default payment method" option should be shown again
Actual Result:
"Make default payment method" doesn't show up again
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.2.7-0
Reproducible in staging?: Y
Reproducible in production?: need PROD account
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1662753199634729
View all open jobs on GitHub

I think We Should close this issue in favor of https://github.com/Expensify/App/issues/10493#issuecomment-1229930529

I'm unable to reproduce. Can you specify reproduction steps? Or can you try again, and reopen if this issue is found again:


Consume rotary events from Hue bridge event stream

While v0.13.42 adds support for the Hue tap dial switch, (see https://github.com/ebaauw/homebridge-hue/releases/tag/v0.13.42), it doesn't yet consume rotary events from the Hue bridge API v2 event stream.  Also applies to Lutron Aurora.

In v0.13.43.


Initial Website Setup - 1

An initial PR for setting up the base of our docusaurus website!
Work-in-progress:

fixing some issues in the searchBar component
preparing a guide on "How to set up locally" for folks
hosting on gh-pages

Related to #5

Hey @verma-kunal i see some blogs in blogs/  folder and tutorials in docs folder i was wondering if we need those things in our website. Are those mandatory components without which things won't work

Hey @verma-kunal i see some blogs in blogs/  folder and tutorials in docs folder i was wondering if we need those things in our website. Are those mandatory components without which things won't work

Hmm, will surely look into this tomorrow @AvineshTripathi !! Thanks for pointing out :)

I think we can cleanup a lot of stuff before putting the base template to the website , we do not needs the docs, tutorials as they can be looked up from the product website itself, we need to have a clean repo with basic structure and theme , then we start adding blogs and stuff

I think we can cleanup a lot of stuff before putting the base template to the website , we do not needs the docs, tutorials as they can be looked up from the product website itself, we need to have a clean repo with basic structure and theme , then we start adding blogs and stuff

Sure @saiyam1814 , will work on this part!

Closing in favour of #16


Finish Team Goals Document

doc: https://docs.google.com/document/d/1fNKaOiV-D2VHdNUPJoGFsM2D10RWNsTGj5WjGHLboqg/edit?usp=sharing

[ ] Incorporate any feedback from the rest of team cc @badboy @Manishearth @skade
[ ] Get sign off by 31st of Jan

[ ] @badboy
[ ] @Manishearth
[ ] @skade

[ ] Send finished document to @ashleygwilliams by 2nd of Feb

completed http://blog.community.rs/content-team/2018/07/16/hello-content-o-tron.html


Left menu in examples not visible on small screens

When the screen width is smaller than 1400px (!), the menu on the left goes off the screen, because it is positioned with a negative margin left instead of using the responsive grid.

ya I just haven't had a chance to fix it. PR's welcome

here you go :)


invalid TypeScript declarations generated

when i run npm install && gulp build i get invalid TypeScript declarations file that contains following invalid lines:
    abort(null): void;
    open(method: string, url: string, isAsync: boolean, user?: string, password?: stringnull): void;
    send(content?: anynull): void;

but instead these lines i'm expecting following lines to be copied without any modifications from the source: https://github.com/aurelia/http-client/blob/6a3f44a7eedd6d684ab8618592097bc2a705690b/src/request-message-processor.js#L62-L70
You can verify presence of errors using following command (tsc is TypeScript compiler - i'm using version 1.6.2, but it's not significant):
tsc dist\aurelia-http-client.d.ts

produces following output:
dist/aurelia-http-client.d.ts(57,11): error TS1138: Parameter declaration expected.
dist/aurelia-http-client.d.ts(57,15): error TS1005: ';' expected.
dist/aurelia-http-client.d.ts(57,16): error TS1128: Declaration or statement expected.
dist/aurelia-http-client.d.ts(57,22): error TS1109: Expression expected.
dist/aurelia-http-client.d.ts(58,16): error TS1005: ',' expected.
dist/aurelia-http-client.d.ts(58,29): error TS1005: ',' expected.
dist/aurelia-http-client.d.ts(58,46): error TS1005: ',' expected.
dist/aurelia-http-client.d.ts(58,62): error TS1109: Expression expected.
dist/aurelia-http-client.d.ts(58,81): error TS1109: Expression expected.
dist/aurelia-http-client.d.ts(58,94): error TS1005: ';' expected.
dist/aurelia-http-client.d.ts(58,100): error TS1109: Expression expected.
dist/aurelia-http-client.d.ts(59,18): error TS1109: Expression expected.
dist/aurelia-http-client.d.ts(59,28): error TS1005: ';' expected.
dist/aurelia-http-client.d.ts(59,34): error TS1109: Expression expected.
dist/aurelia-http-client.d.ts(666,1): error TS1128: Declaration or statement expected.

We already have a fix in the new d.ts plugin. We are waiting to sync with them and the new babel release.


DVC badge

thought it would be great to have a DVC badge - mainly for devs to note we're using it

We finally have shields.io & simpleicons support! :tada: Here's what your README can look like:
My Project README.md

Yes, yes the accuracy metric example above is dynamic and not hard-coded :rocket:

![DVC](https://img.shields.io/badge/-tracked-white.svg?logo=data-version-control&link=https://dvc.org)

[![DVC](https://img.shields.io/badge/-Data_Version_Control-white.svg?logo=data-version-control&style=social)](https://dvc.org)

https://img.shields.io/badge/dynamic/json?logo=data-version-control&colorA=black&colorB=F46737&label=Model%20Accuracy&url=https://github.com/iterative/get-started-experiments/raw/main/metrics.json&query=categorical_accuracy

Wow! That's amazing! 🎉  Perhaps related to https://github.com/iterative/cml/issues/219?

Ooooo!  Que fancy!  Love it @casperdcl !!!  Is there a reason why you chose the purple? Not finding fault, just wondering.  I am a fan of purple.

I think a View in Studio badge would be also a nice touch for our example repos.
Currently, unless someone has already created a public view, it is kind of tedious to go from a  Github repo to a Studio view.
cc @iterative/studio

Closing for now (please reopen if I'm missing something)

I think a View in Studio badge would be also a nice

@daavoo mind opening that in https://github.com/iterative/studio-support/issues ?

Maybe somewhere in https://dvc.org/doc/user-guide/contributing ? Please do/open new issue/reopen this one & self assign or otherwise, as you see fit. Thanks


Oauth

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (yarn build) was run locally and any changes were pushed
[x] Lint (yarn lint --check) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

twitter oauth login

Does this introduce a breaking change?

[x] Yes
[ ] No

Other information

There's a problem, I don't know how to make test:e2e succeed in frontend, since the signup now will redirect to twitter page and redirect back then.

Looks like the signup new user command in support/commands.ts will need to be refactored to test Twitter's OAuth and make the tests pass on CI. Cypress should support this. Here is an example found in their docs.


Eap7 606 statistic master 2 nd

can you either use the WFLY jira reference or create a RESTEasy jira (and reference it) for this feature?


Add public execution / consensus support

Changes:

Adds Possibility to use remote Execution node with new flag for sedge cli

Types of changes
What types of changes does your code introduce?

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[x] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[x] No

Comments about testing , should you have some (optional)
Must test different deployment levels (only deploy validator, deploy validator and consensus)

Codecov Report

Merging #76 (322d49c) into main (61cc4b1) will decrease coverage by 0.65%.
The diff coverage is 70.27%.

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   84.55%   83.90%   -0.66%
==========================================
  Files          31       31
  Lines        1800     1845      +45
==========================================
+ Hits         1522     1548      +26
- Misses        224      241      +17
- Partials       54       56       +2

Flag
Coverage Δ

unittests
83.90% <70.27%> (-0.66%)
:arrow_down:

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

Impacted Files
Coverage Δ

cli/cli_utils.go
87.85% <38.46%> (-2.93%)
:arrow_down:

internal/pkg/generate/generate_scripts.go
90.05% <64.86%> (-5.83%)
:arrow_down:

cli/cli.go
70.46% <92.85%> (+0.89%)
:arrow_up:

internal/pkg/clients/clients.go
92.85% <100.00%> (ø)

internal/utils/utils.go
100.00% <100.00%> (ø)

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

Closed in favor of #79


Set valid Accept header type for plain text

A very minor change to ensure compatibility with GDMC-HTTP v0.7. Since this version, GET /chunks returns binary ("Accept": "application/octec-stream") by default instead of plain text. This isn't a problem if it weren't for that "text/raw" being a type that the API does not recognize. Nor should it, since "text/plain" is much more conventional.

This was a bug I caused in GDMC-HTTP v0.7.0. I wrote a fix for it in v0.7.1.


Update plan-cloud-voicemail.md

Need to update licensing requirements for on-premise only environments.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


[Bug] clickhouse replicated error & freeze

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
DBT Core works properly with replicated tables in ClickHouse, and DBT Core does not randomly freeze while updating replicated tables.
Expected Behavior
DBT Core is unable to update tables with the Replicated engine, as DBT creates an empty copy of the destination table and, after completing the update (in my case, with materialized='incremental' and incremental_strategy='delete+insert', inserts data from the copy into the destination table and then deletes the copy. However, this process ignores the fact that Clickhouse is unable to create a copy of a replicated table, resulting in an exception being thrown (Code: 253. DB::Exception: Replica /clickhouse/tables/db/table/replicas/replica_name already exists).
Although this was expected, I attempted to use the MergeTree engine for updating, since it does not require replicating the table with new data. However, unexpectedly, DBT begins to freeze randomly while performing the updates. Sometimes, it works fine, but at other times, it appears to be deadlocked for some reason, regardless of the number of threads I use.
If this issue is not related to the DBT team, the problem with the 253 exception may not be specific to this repository. However, the freezing issue may be due to some other factor.
Steps To Reproduce

create Replicated table

run update with incremental strategy

create replicated table

switch model engine to not replicated

run update with incremental strategy

Relevant log output
No response
Environment
- OS: Ubuntu 22.04.5 LTS
- Python: 3.11.10
- dbt: 1.8.8
- clickhouse: <IP_ADDRESS>

Which database adapter are you using with dbt?
other (mention it in "Additional Context")
Additional Context
dbt-clickhouse: 1.8.4

Thanks for reaching out @V4kodin !
This looks specific to the dbt-clickhouse adapter, so could you please open an issue here instead?
Closing this report in the dbt-core repository in favor of creating one in dbt-clickhouse instead.


update download url and hash for 4.10.12 bundles

/hold
bundles are not synced to mirror.openshift.com yet!

Looks good, though we really need to build a new version of the podman bundle (not sure how to version it). I guess too late for this release :(

/unhold


Azure IoT Hub/Central/DPS integration

Integrate into Azure IoT suite of tooling. There is a documentation page stating 'soon' but no actual issue for this (#7 is closed).
Ideally, in addition to IoT Hub it would be integrated into DPS and also Central to have complete end-to-end lifecycle of devices.
Although there is no 'nice' Rust APIs, there is autogenerated ReST APIs for most things:

https://github.com/Azure/azure-sdk-for-rust/tree/main/services/svc/iotcentral
https://github.com/Azure/azure-sdk-for-rust/tree/main/services/mgmt/iotcentral
https://github.com/Azure/azure-sdk-for-rust/tree/main/services/mgmt/iothub

Device Provisioning Service does seem to be missing though at this time.

I will remove this from the documentation. At this moment I have no plans to implement this. I know a few companies that have used the Azure IoT Hub with ChirpStack v3, but it brought them a lot of troubles. These issues were related to both the unstable SDKs and latency that the integration with IoT Hub added. Under high throughput, Azure started to throttle the ChirpStack -> Gateway messages. They had discussions with the Azure team which confirmed that the outbound throughput is limited per tier-level, the only option for these companies was to upgrade their tier (with a significant price increase) to avoid throttling of downlink payloads (and thus failed transmissions).
The companies that supported have migrated to their own MQTT broker instance with gave them better performance and a lower bill.


Selection in the bar type starts with spacing to the right

Bug report
When type is area, that is the behavior with I expect

When type is bar, starts with spacing to the right

What are your x-axis categories?
Can I see a CodePen?

https://codesandbox.io/s/sweet-stallman-2sxgb?file=/src/App.js

Hey,
This is still happening, if you use x-axis datetime the mouse click has an offest from the selection area

The issue also appears in the zoom selection as well.
A workaround could be disabling a grid.
Then the selection tool works as expected, without spacing to the right.
// Options
grid: {
        show: false,
}


Getting custom Headers from response

Hi,
great work you perform with this framework. I have a question: Is it possible to get (custom) response headers from a download operation? I need to evaluate them.
As far as I understand, the only way would be to listen to SDWebImageDownloadReceiveResponseNotification and read response from the SDWebImageDownloaderOperation object.
Thanks

Ok, after a little bit more digging I found out that the response can be accessed through the downloadOperation which is returned by the downloadImageWithURL... call, eg:
operation = SDWebImageManager.sharedManager().downloadImageWithURL(url, options: SDWebImageOptions(), progress: progressBlock, completed: completionBlock)
In the completion block the full response can be accessed with operation.response
I close the issue.

... completed:^(UIImage *image, NSError *error, SDImageCacheType cacheType, BOOL finished, NSURL *imageURL) { ...
how to get the response from here?

Seems like there were some changes done since the time I opened this issue. I think there still must be a way to access the download operation and its response property.

Hmm, I'm not sure you can. At least from this level. you'll need access to the SDWebImageDownloaderOperation which has a response property.

Hi There
Is there any news about this ?
I'm using sd_setImageWithURL to download an image, and I need to retrieve the session cookie, any tip to achieve this?
Any help would be greatly appreciated as I'm stuck on this issue.
Thanks

Already available in 5.x via #2261
SDWebImageCombinedOperation.downloadOperation.response.allHeaderFileds. Use necessary force cast during this chain propery call.


Add support of ephemeral_storage

Hi,
AWS ECS introduced ephemeral storage earlier this year, and aws provider supports it from v3.45.0.

Just checked hashicorp/terraform-github-actions and it was officially replaced by hashicorp/setup-terraform
Not quite sure which version is preferred, simply replace it with the code snippet in its README.md


Events scheduled for the same time at the same time should stay ordered

Fixes #36
I am not sure if there is a reliable way to test this change. Wanted to get some feedback on this as it seems a little heavy-handed.

This was taken care of in https://github.com/ReactiveX/RxPHP/issues/50


Can't use eglot with latest Emacs & straight, due to project.el location

I install eglot via straight, and it's been working fine for a long time, until I just updated to the latest emacs master branch and pulled all the latest straight packages. Now emacs starts OK, but when I invoke eglot I get an error: "funcall: Feature provided by other file: project". This comes from require-with-check which looks to see if the feature project was loaded, using load-history. In my case, load-history does not contain `project.
In fact just calling (require-with-check 'project) fails with the same error. I think it's using the built-in version rather than what's loaded from straight.
I checked, and the built-in version is byte-for-byte identical with the version in straight/repos, but I think require-with-check checks the file path.
I also deleted straight.el's copy of the project repo and restarted, and straight clones it again, but I still get the error.
If I debug into require-with-check, I see it calls locate-file on project in load-path and that returns ~/.config/emacs/straight/build/project/project.elc but that is not found in load-history. That only contains "/Applications/Emacs.app/Contents/Resources/lisp/progmodes/project.elc" (or similar on Windows).
I looked at #551 and other related issues, but they don't seem quite the same.
Here's a test case. Load this file into an emacs -Q session:
;;; Set up package system -- straight.el
(defvar bootstrap-version)
(or (boundp 'native-comp-deferred-compilation-deny-list)
    (setq native-comp-deferred-compilation-deny-list '()))

(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

;;; Meta-package system: use-package. Auto-installs and configures packages.
(straight-use-package 'use-package)
(defvar straight-use-package-by-default)
(setq straight-use-package-by-default t) ; make use-package use straight

;;; code to test
(set-variable 'debug-on-error t)
(use-package projectile)
(use-package eglot)
(require-with-check 'project)

and you'll see the error.
If you add (use-package project) before projectile, then it works OK.

Huh... interesting... let me have a look at what is going on there. Thanks for the ideas on debugging.

Ok, I see that require-with-check is a very new function, implemented last month in Emacs core: https://gitlab.com/ganserla/emacs/-/commit/2a8e6c8c84ed33674e525625644d5ce84ee8c59a
Maybe there is an edge case when the same package is provided from multiple locations, that is not taken into account properly by one system or the other. I'll build a more recent Emacs snapshot and try it out since I have not used this function before.

Yes, okay, this makes sense. The issue is that projectile loads project, but doesn't declare it as a dependency in this line:
https://github.com/bbatsov/projectile/blob/0163b335a18af0f077a474d4dc6b36e22b5e3274/projectile.el#L9
So straight.el does not know that it needs to install project, and the built-in version gets loaded. But then eglot does declare project as a dependency:
https://github.com/joaotavora/eglot/blob/db91d58374627a195b731a61bead9b4f84a7e4bc/eglot.el#L10
Which causes straight.el to install project. But the other version was already loaded, so require-with-check correctly identifies that there is a confusing situation.
I think the correct solution here is for projectile to declare a dependency on project. But, straight.el can be improved by reporting a warning if a package is made available when a different version of that same package has already been loaded into the current Emacs session. I'll file another issue to track that improvement, and link it below.

We can leave this issue open until Projectile is updated to declare the dependency, and new users will not encounter this problem anymore.
The workaround is as you observed; (straight-use-package 'project) or equivalent before running any code that would (require 'project).

Forgive the intrusion, but I wonder if this is another instance of the problem reported at these issues, relating to a master version of the map library being installed for a non-master build of Emacs, which fails since it depends on the version of pcase from master, which has associated changes from https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=806759dc0a6a3b049ce35d0497011464e5fc4dcb:

https://github.com/alphapapa/burly.el/issues/64
https://github.com/alphapapa/bufler.el/issues/70#issuecomment-1949928135

Thank you for taking the time to identify the problem. Indeed, I am using straight.el.

Yes, look here: https://github.com/alphapapa/burly.el/issues/64#issuecomment-1955989287  The repo at https://github.com/emacs-straight/map.git is importing map.el from Emacs master, which is ending up installing changes to be released in Emacs 30 into configurations running earlier Emacs versions, which is producing "chimera" configurations that are broken.  And since the map.el version header still shows 3.3.1, users think they have map v3.3.1 installed, same as the one available from GNU ELPA, while in fact they have map.el from Emacs master ("30.0.50") which will probably be tagged as map v3.4 someday.

It may indeed be related, but not quite the same. In my case both project.el versions are identical; it's the new require-with-check function in emacs itself which checks the on-disk location of the required file that causes trouble.

I'm currently struggling financially, so I won't have time to implement any of the above for now

Take care, I will continue triaging issues with the time that I have available, never feel obligated to spend time you don't have.

We could prevent this and similar issues if straight didn't install external versions of core packages unless they were explicitly requested by the user

Ugh... this is really making it sound like we may actually need to properly support package version numbers. Because we really do want to install an external version of a package when the version shipped with Emacs is insufficiently recent to satisfy a dependency. But of course then if that is determined after the built-in version was already loaded, we are stuck again...
I think you're right and disabling built-in packages by default is the right solution, even if it will be disruptive. We can implement it in a way that will signal a warning on first use, perhaps. Or make it as an opt-in feature, and report a warning when a package is added to load-path when it was already loaded, with a pointer to the user option.
I'll add the latter idea to my to-do list, to resolve that long-standing issue. And then Nicholas' idea about installing built-in packages only when requested.

In case anybody runs into this issue and the original author's solution doesn't work (as it did not for me).
You can also do this:
(use-package straight
  :custom
  (straight-built-in-pseudo-packages '(emacs nadvice python image-mode **project** **flymake**))
  (straight-use-package-by-default t))

Which I gleaned from this comment by @raxod502 https://github.com/radian-software/straight.el/issues/551#issuecomment-667540139
eglot starts up as expected now. I am using Emacs 29.3
straight.el hash: b3760f5829dba37e855add7323304561eb57a3d4
I don't know what the eglot hash is since it's the mirror. Idk if they'd be the same hash...?

I ended up having to add xref as well in order to use eglot. In my case, I just customized straight-built-in-pseudo-packages: it is currently (emacs nadvice python image-mode project flymake xref).

I don't know what the eglot hash is since it's the mirror. Idk if they'd be the same hash...?

You can check the revision in ~/.emacs.d/straight/repos/eglot, same as any other repository. Then you can cross-reference that commit to see what version of the package on GNU ELPA Devel it corresponds to - https://github.com/emacs-straight/eglot/commit/6b9c4faa5201915ab81fe470a872ebce8921fe51
I don't believe GNU ELPA tells you which commit of a repository was used to build a particular package version, which is part of the reason I don't like it. You can probably get a good enough idea by extracting the timestamp embedded in the version number and checking to see where in the upstream commit history that fell.


Remove pkg/ from .gitignore

Description

This was causing linting failures due to subdirectories of vendored modules being ignored.
I think the .gitignore entry dates from when this repo set itself as the GOPATH and Go would generate files to that directory.

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing performed?

[X] Unit tests
[ ] Integration tests
[ ] Acceptance tests

Checklist:

[X] This PR is being made against the main branch, or relevant version branch
[ ] I have made corresponding changes to the documentation
[ ] I have added testing for my changes

If you have any questions, or want to get attention for a PR or issue please reach out on the #logging-and-metrics channel in the cloudfoundry slack

It looks like the tests and linting are still failing?

I'd planned on having our tooling commit the missing files once the exclusion was removed but have now pushed a second commit with the previously excluded files.


Update v0.2.1

@JuliaRegistrator register
Release notes: Project.toml fix

Error while trying to register: "Tag with name v0.2.1 already exists and points to a different commit"

@JuliaRegistrator register
Release notes:

Registration pull request created: JuliaRegistries/General/49082
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.1 -m "<description of version>" 2a31ec75520aebc80909bc272e0c4e97bf6e230c
git push origin v0.2.1

Also, note the warning: This looks like a new registration that registers version 0.2.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.

@JuliaRegistrator register

Error while trying to register: "Tag with name v0.2.1 already exists and points to a different commit"

@JuliaRegistrator register

Error while trying to register: "Tag with name v0.2.1 already exists and points to a different commit"

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/49245
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.2 -m "<description of version>" d0efeea0de8a0deb22aba4638fb990d136744482
git push origin v0.2.2


Remove DisposedException

This removes the need to wrap Queue::push() in try/catch if the iterator is disposed. If the producer cares if the iterator is disposed, they can check with Queue::isDisposed(), but often piping into a blackhole is acceptable (e.g., Pushing websocket frame data into a message stream. The data must be consumed anyway, and it doesn't matter if no one is listening on the other end).
If an iterable is used as the source, iteration stops once the iterator is disposed.
I have verified the behavior here works well in the postgres and websocket libs, which make heavy use of this library.

Closing after discussing with @kelunik. Throwing an exception from Queue::push() is the "safe-by-default" way for infinite producers. Such producers would need to check Queue::isDiposed() after each push to avoid creating more data than necessary, whereas throwing an exception cannot be ignored/forgotten. Consuming websocket frames is likely an exceptional use-case where the producer needs to keep consuming data from the stream.
TL;DR: There's no perfect solution, but an exception from Queue::push() prevents infinite loops or producing unnecessary data.


Figure out a strategy for release.json

release.json is the file that the census library uses to determine the location of the artifacts. This is currently hosted on the public bucket and needs to change. We have a few options:

Keep it on the existing bucket (in single-cell-prod). The bucket will need to become private and a Cloudfront rule needs to be created so that the file can be publicly accessible.
a. This has the advantage that existing clients will not break
b. Since the builder needs to write to this file, an IAM permission could be necessary if the builder will run on the new account
Move the release.json file to the new S3 bucket
a. This will break existing clients
b. This might pose additional security concerns, as this file is effectively vulnerable to potential injections
c. Easiest solution since the builder will write this file after the copy is done to the same bucket
Move the release.json file to a different location (API, GitHub, ...)
a. Will still break existing clients
b. Likely more complex than necessary.

If we care about existing api clients (but I don't think we should), we could do 1 and update the s3 urls to the new bucket, but otherwise consider it end-of-life, no further updates. New api clients will find a new release.json in the new bucket.

As per our conversation we should not consider existing client as the API has been flagged unstable, we will make sure to communicate the change to users once we implement it.


add riftguardian plugin

Highlighting/overlays for Guardians of the Rift minigame.

Don't clear the hint arrow if you're not in the minigame area: https://github.com/rdutta/runelite-riftguardian/blob/173144ec4a09faf10cdc1375a435daf9e4851f0c/src/main/java/ca/riftguardian/RiftGuardianPlugin.java#L145 (other plugins, and the client itself, use the hint arrow)

Updated. It will now check if the portal gameobject is present before remove the hint arrow.


[KYUUBI #2085][FOLLOWUP] Fix kyuubi-common wrong label

Why are the changes needed?

Change kyuubi-common to kyuubi-ctl in ctl label.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

cc @cfmcgrady

Since no test can cover this change
thanks, merging to master


Better memory handling

Main issue: No logic to handle more data than there is memory available
Additional issues:

Share the same memory block between multiple streams - If UDMA memory blocks are too large then there is a lot of fragmentation and a lot of unused available memory
Check if UDMA memory blocks are consecutive - If they are, then smaller memory blocks can be added together to be able to support larger data streams
Add a way to split larger data files into smaller ones - Probably needs HW support to have multiple addresses. If a stream is too large then it should be split into multiple temporary files.
Make more memory available on the FPGA - Currently only 80MB of RAM available for data, even thouch the ZCU102 supports way more.

Closing this as more specific issues have been made: #78 and #79


[APPS-3271] Ensure requirements only apps specify Support as their product

:v:
/cc @zendesk/vegemite
Description
Ensure requirements only apps specify Support as their product. This is to pass validations that the app can be uploaded from within Support.
References

JIRA: https://zendesk.atlassian.net/browse/APPS-3271

Risks

low: breaks product-related validations / features

is it possible to add a regression test? does this affect all requirements apps?

Will this break channel framework and requirements apps based on current docs?

No, this is the fix for currently-broken requirements only apps.

@sandlerr we had a browser test for this, but it was disabled, @token-cjg submitted a PR to fix it: https://github.com/zendesk/zendesk_browser_tests/pull/3720

this seems unit testable, why would we not have one? With a tight merge / qa / deploy cycle those tests might not pick it up before it hits production anyway.

By unit testable, do you mean for the old behaviour? Originally I left it out since we were planning for a production patch, however we can probably add it if we have time now.


Added External Secrets Support (cont.)

A continuation of #68 that does not come off our main branch.

OK; pinky promise will get to this tomorrow! All ready to go @clbx? :D

We've been using it pretty heavily in our pipelines and its been working great.
I misnamed the ClusterSecretStore type at some point between my testing and the final result, but that is fixed.
I believe that it is all ready to go, and if any issues come up during our use, I will contribute them back upstream.

Thanks!
I've never figured out how to do buildkite builds for forks so pushed an in-org branch and built here:
https://buildkite.com/embark-studios/k8s-buildkite-plugin/builds/274#0187a44f-33ad-49ba-b59c-85fffb375b51
Looks like a bunch of shellcheck issues that'd be lovely if you could fix them and then I'll merge :-)

I will take a look at them on Monday

I completely forgot about this PR, I put it in our next sprint to finish these up so we can merge this in.

I can't run the CI to check, but I believe I fixed them.

@clbx There's still some missing -- I tried to push a fix to this branch but I don't think you've granted maintainer push permissions on this PR. https://github.com/EmbarkStudios/k8s-buildkite-plugin/commit/216e8fb909a93f735f75c912271f05515def0107 is the missing fixes :-)

Ah; I can just merge it from my branch tomorrow then, let's not make life overly complicated. And good luck on your new adventure @clbx! :-)

Thank you, I'm glad I was able to contribute to this even if it has taken months to get it in 😆

@tgolsson can we close this pr then?

Back from vacation now, merged in #72!


Missing SuperPuTTY x64 release build (64bit/64 bit)

Dear SuperPuTTY team, @jimradford,
Thanks for <IP_ADDRESS> release from today but SuperPuTTY x64 (64bit/64 bit) release build is always missing.
Can you create this important missing release build?
Thanks in advance.

@jimradford: Sorry for the delay!
It is like to create 32 bit application because 16 bit works?
You can look information on official PuTTY website:

https://www.chiark.greenend.org.uk/~sgtatham/putty/faq.html#faq-32bit-64bit

SuperPuTTY, written in .NET doesn't care if you use a 32 or 64 bit version of putty you can use either. Did you read the link I provided?

It is time to enter in the future, we are in 2023, x64 is here since Windows XP Pro.
The goal is to have like other softwares, a x64 version in more x86.
Some links:

https://learn.microsoft.com/en-us/dotnet/framework/64-bit-apps
https://www.youtube.com/watch?v=2jdH85teO5Q
https://www.csharp411.com/determine-if-your-c-application-is-64-bit/
https://www.ibm.com/docs/en/spectrum-symphony/7.2.1?topic=symphony-support-64-bit-applications
https://www.geeksforgeeks.org/c-sharp-program-to-check-if-operating-system-is-64-bit-os-or-not-using-environment-class/

I think you might not be understanding, for the SuperPuTTY application, there is zero advantage or purpose to having two binary releases, you literally get nothing from it other than the zip file being labeled "64 bit". It does not/will not perform any different from what you are downloading, installing, and using right now.


jquery $ not defined error

Hi,
i am using bsb material admin theme for my project.design is working fine.but i got error $ not defined when using jquery and javascript functions. so i cant use any jquery functions when theme is applying .please help to solve this.
thanks in advance.

Hi,
I need more information about this, i didn't understand because the theme already built with jQuery v1 and did not test with jQuery version 2 and 3. Maybe you can send console screenshot, some codes etc...

Check your HTML, it sounds like jQuery library is added to the page more than one.

thank you for all solutions.the problem solved,the error shows the meaning of $ is overridden, solved it by declaring $(jquery)


Refactors AgoraClient's initialization

Closes #13.
Moves initialization of AgoraClient from its constructor (implicit) to a proper function (explicit).
This means that it's the user responsability to check for it, but it also gives the power to know when it's ready.
Release Notes

BREAKING CHANGE: Must call initialize on AgoraClient before further usage.

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] The GitHub Actions pass building and linting. Linter returns no warnings or errors.
[X] The QA checklist below has been completed

Pull request type
Please check the type of change your PR introduces:

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
The initialization is triggered by the constructor of AgoraClient at initialization and if it happens to failure, it can't recover from it.
Issue Number: #13
What is the new behavior?
The initialization is triggered by a specific function by the client. If failure happens the client gets to know and can handle it. Also the same instance can be reused.
Does this introduce a breaking change?

[X] Yes
[ ] No

QA Checklist
UIKit Update Checklist (Minor or Patch Release)

[X] Using the latest version of Agora's Video SDK
[X] Example apps are all functional
[X] Core features are still working (both ways across platforms)

[ ] Camera + Mic muting works for local and remote users
[ ] Users are added and removed correctly when they join and leave the channel
[ ] Older versions of the library gracefully handle changes (Create issue if not)
[ ] Builtin buttons all work as expected

[ ] Any newly deprecated methods are flagged as such inline and in documentation

Me and @Meherdeep discussed this yesterday, and we think this is the right approach. But we want to go over a couple more things with this including

Making sure this breaking change doesn't have adverse effects
Adding dispose as well

@tadaspetra that is true, I've noticed there was no dispose and that bugs me. Should I include it in this PR? I think the SessionController should be disposed, but I'm not sure in regards of RtcEngine.

@tadaspetra that is true, I've noticed there was no dispose and that bugs me. Should I include it in this PR? I think the SessionController should be disposed, but I'm not sure in regards of RtcEngine.

Of course, we would love it if you could add it 😄

@tadaspetra I've found out that RtcEngine has a destroy method which unfortunately is async. How would we address that? Im thinking on disposing SessionController both RtcEngine and SessionController just because we can't make SessionController's dispose async.

@tadaspetra I've found out that RtcEngine has a destroy method which unfortunately is async. How would we address that? Im thinking on disposing both RtcEngine and SessionController on AgoraClient's dispose just because we can't make SessionController's dispose async.

So that might work, but one thing to look out for in the endCall() function we are already calling engine.destroy. I'm thinking this should be done in a dispose as well somewhere in case someone leaves without "ending the call" (maybe using back button or something).
It might be a little more complex solution for this.

Thank you for the idea, and your code was really useful in the merge of #45.
Thank you for all the contributions and hope to see you around here more :)


Generate string IDs for compatibility with Buffer

When keys are provided as [large] numbers, the Buffer constructor fails with an error.

Awesome! Could I also request a passing test for this?

I have quite the stack of yaks piled up, so no bandwidth to do so in the near future.


[Feature] Load images from tar file

sometimes one want to load images that were exported via
docker save -o myimage.tar myimage

to get loaded with
docker load -i myimage.tar

This PR achieves this

@paschdan thanks, this is great. Much appreciated.


Resolving environment file conda selector syntax

Making MPI explicitly an optional dependency that users need to install will simplify dev installation, and mean we don't have to worry about platform specific installation.
Changes:

Remove MPI packages from dev installation (ie, environment.yml)
Update install instructions
Update MPI documentation notebook
Remove mamba from GHA testing to reflect new installation instructions

Supercedes #1471 and #1513

Resolved the main test failures, but still getting a failure for the following 2 tests. Would you mind looking into it @erikvansebille ?
=========================== short test summary info ============================
FAILED docs/examples/example_peninsula.py::test_peninsula_fieldset_AnalyticalAdvection[flat-scipy] - assert np.False_
 +  where np.False_ = <built-in method all of numpy.ndarray object at 0x7fddfe59a9d0>()
 +    where <built-in method all of numpy.ndarray object at 0x7fddfe59a9d0> = array([0.22306108, 0.09739923, 0.00405693, 0.04935837, 0.04673195,\n       0.00358963, 0.02781296, 0.01982117, 0.01034927, 0.01644897],\n      dtype=float32) <= 0.1.all
FAILED docs/examples/example_peninsula.py::test_peninsula_fieldset_AnalyticalAdvection[spherical-scipy] - assert np.False_
 +  where np.False_ = <built-in method all of numpy.ndarray object at 0x7fddfe5defd0>()
 +    where <built-in method all of numpy.ndarray object at 0x7fddfe5defd0> = array([0.22306108, 0.09739923, 0.00405693, 0.04935837, 0.04673195,\n       0.00358963, 0.02781296, 0.01982117, 0.01034927, 0.01644897],\n      dtype=float32) <= 0.1.all
= 2 failed, 329 passed, 112 deselected, 4 xfailed, 210 warnings in 934.38s (0:15:34) =

I did some testing, and those tests were also failing on master in my local development. Diffing the environment files, numpy 1.26.4 was working but numpy 2.0.0 (!! new major version!) was breaking. I'll look into what a migration will look like, but for now let's update the env file.
NumPy 2.0.0 Release Notes https://github.com/numpy/numpy/issues/24300

Doc build failure is due to the https://www.aoml.noaa.gov/phod/argo/images/argo_float_mission.jpg link not working (https://www.aoml.noaa.gov is down). Happy to merge pending your PR approval (and that the tolerances are ok) @erikvansebille

Increased error in the test_fieldset_sampling.py::test_fieldset_sample and test_fieldset_sampling.py::test_fieldset_sample unit tests are due to changes in the grid interpolation. Likely due to numpy2's changes in how they related to float32 type promotion.

@erikvansebille Would you be able to merge here? I don't think I can due to CI failure/branch protection rules, and I don't have access to the settings tab for this repo :)


Strange buttons behaviors since last update

Hello there, let me say that your addon is simply wonderful, but from the last update i noticed that the answer buttons edited by this addon (Minimal Flat Big Buttons https://ankiweb.net/shared/info/1042429613) are super compressed in the center of the screen while usually they spread all across the screen (which is super useful if you have a touch pc and want to smash some reviews with those enormous buttons). Any idea why this happens?

Ah thanks for the report.
The new way the addon renders actually makes a sub-window that only contains the answer buttons instead of stretching the bottom bar over the entire screen and making the background transparent (which was causing graphical issues for some users)
I took a look through the addon code and unfortunately the way it formats the buttons isn't that compatible with the way I'm showing them. I'll look into some alternatives, but it may take a bit since it isn't a straightforward fix.
In the meantime, if you want to use the old version it's available here
I'll post here when I figure this issue out

Since I was the one who made that addon by editing the css of large and colorful buttons a bit through some guides found online, but i don't know much about CSS or programming in general, it's not a problem for me to edit the addon if you explain what should i change exactly to make it compatible! I think the problem is that i used vw to define the width and since the container is much smaller with this new update, the buttons are much smaller too.
Alternatively since i saw that with your last update you added custom answer buttons, if you want and have time, you could add a feature that allows the user to select different presets for the answer buttons, add the styling that i used in my addon (big fat buttons useful for review through touch), and i would delete my addon / make a redirect to yours. Something like merging my addon in yours. But i understand that this would require much work and you may not be interested or have the time to do it!

Oh haha it's great to meet another GUI addon maker! I think the way you did it is perfectly valid, and I think it would be good if my addon supported dynamic layouts. I'm testing some solutions out, so hopefully you won't have to make any changes.
I think presets would also be a great idea - I'll look into it

I improved compatibility with the latest version! It should work properly now, and dynamically resize.
However there's still a bug where it doesn't resize properly upon zooming with Anki Zoom. Unfortunately anything that uses view units (vh or vw) won't work properly. If you can modify your addon to use anything besides vh or vw (pixels, em, percentage all should work fine), it should work better with zoom.


Context manager

We need to make the secret manager a context based system so we can properly close AWS client connections.

Fixed in 0.2.2

Fixed in commit: https://github.com/gdoermann/supersecret/commit/d9d25984da5b66c3324b3d27449c79e4a5f1d35c


All tab view of tabs are mounted once.

Version
Tell us which versions you are using:

react-native-router-flux 4.0.0-beta.16
react-native 0.45.1

I have tabbar which have 4 tabs.
When I enter the first tab of tabbar, other tabs are mounted ( checked this in componentDidMount life cycle function)
Additionally, When I pressed another tab, old tab view component was not unmounted.
Is it normal behavior?
This makes App to call unnecessary API calls.

Same here
I had to move the <Scene> to the tab component, return it inside of a function and call it where I need the scene. Then, I used onEnter instead of componentWillMount or componentDidMount.
I can show you the code if you need it
Besides, we should have the option to not mount the components until we enter

@giulianok
I want to see your codes.
You mean that to work around this bug(?) you use 'onEnter' method. right?
I think so that we need to option to not mount the components until we actually enter.
When a tab is switched to another scene, previous tab component needs to be unmounted too.

@lazyhoneyant Use lazy option. Will add it to docs. Component will NOT be unmounted after switching - it is how tabbar works.

@aksonov
thank you. 'lazy option' solves this problem.


Uses flagged property to filter out flagged reportbacks

Closes #3652, updates Reportback Totals function based on new Reportback.flagged property.

:+1:


Adds automatic formatting and linting

summary
This PR adds in automatic code formatting and linting. It also adds make commands as well as a git hook so that these commands are automatically run when performing a git push. If you'd like to skip the hook, you can use git push --no-verify.
I recommend installing a prettier plugin for whichever IDE/code editor you are using.
new commands:

make install: install the dependencies needed for format and lint.
make lint: run eslint on all of your javascript files.
make format: run prettier on the html/css/js files, as well as google-java-format on the java files.

next steps

@codyjrivera to run make format and make lint until everything is passing :). May also need to install node/npm if not already available.

@codyjrivera: do you mind if we keep these two changes as two separate PRs:

Add in linting rules / autoformatting
Format all the code / fix lint errors

I cherry-picked my lint fixes out of this PR and put them into another feature branch.

LGTM @codyjrivera


Added validation for account creation + bug fixes

Coverage increased (+2.5%) to 40.975% when pulling 7bf717abd8be477aa88a329eca5f08e07dfe8e6a on egopher:master into 11bbcf83168abee3087b0f6e4168a55aabd8fd0c on supergiant:master.


Updated changelog.md for 0.8.x

Documented all fixes and changes in the changelog with attributions to the PR authors

Great job! We should probably find some way to automate this in the future

@seemethere once I pull the contributors guide together, I think that'll be pretty straightforward.

@r0fls Sorry about that - corrected in my fork.


Fix custom event arguments

Any reason for not merging this?

Yes! Inactivity from contributors https://github.com/twitter/typeahead.js/issues/1373

A fix for this is now merged over in corejavascript/typeahead.js#65

 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.Jake Harding seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[AST Builder] Implement GenerateUuid Funciton

Resolve : https://github.com/gluesql/gluesql/issues/739

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
11 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 97.844%

Files with Coverage Reduction
New Missed Lines
%

core/src/executor/aggregate/mod.rs
11
92.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
29683

Relevant Lines:
30337

💛  - Coveralls


ACA-Py integration with WS ( jvm part)

Hello Krzysztof!
I have left several comments, but I have several general concerns

Existing functionality of working with .NET agent with JVM/Android/Ios application should not be broken. As I see it could be broken in the PR
Why do we introduce port and ip in builder parameters. Does it mean our library will work only with one agent at a time? How can it connect simultaneously to several agents

Those two agents aren't compatible, I don't have a working .net agent to make assure that both agents are working correctly. I think we should fork the repo into two parts. (.net and python one)
I'm doing a setup of ACA-Py it has its own one IP address that behaves as Issuer/Verifier, and mobile wallet as Holder. Good point on how to extend it.