Converting a class object to a string should change '.' to '::'

Converting a class object to a string should change . to ::.  Some code checks the classname, like in Date::Manip::Obj.

Fixed in v1.030


Add emulator

Coverage increased (+0.3%) to 97.816% when pulling 4ade38d8261017fa33d01e0e57fba41657ef8361 on add-emulator into dcc8246e30639f9abd50741067a67261c8b20e82 on 0.1.x.

Coverage increased (+0.4%) to 97.897% when pulling a531b68f42b07efcec01b278e740b4d0c009544a on add-emulator into dcc8246e30639f9abd50741067a67261c8b20e82 on 0.1.x.

Coverage increased (+0.4%) to 97.911% when pulling 753be8e723ef1dfb7016d5bfd859c62bc6ac789b on add-emulator into dcc8246e30639f9abd50741067a67261c8b20e82 on 0.1.x.

Coverage increased (+0.4%) to 97.911% when pulling b0e580ab0af6ea296a873819f2e9714cd4bbf6fd on add-emulator into dcc8246e30639f9abd50741067a67261c8b20e82 on 0.1.x.


Multiline help text with indentation

Sample
I altered the test.fsx file to try this out.
let fileOption =
  commandOption {
    names ["f"; "file"]
    description "Name of a file to use\n\
                 Default index: 0\n\
                 Look multiple lines!\n\
                 Neat!"
    takes (format("%s:%i").withNames ["filename"; "index"])
    takes (format("%s").map (fun filename -> (filename, 0)))
    suggests (fun _ -> [CommandSuggestion.Files None])
  }

I did not push those changes ^ they were just for manual testing. Would you prefer some code added for that?
Previous output
$ dotnet fsi test.fsx
test> help
num: 0
error: showing help.

usage: main [options] <command>

The main command.

commands:
  echo [options]                    Echo the input.

options:
  -n, --number=<num>                integer number.
  -v, --verbosity={q(uiet)?$|n(ormal)?$|f(ull)?$|c{ustom:<level>|:<level>}}
                                    Display this amount of information in the log.
  -f, --file=<filename>[:<index>]   Name of a file to use
Default index: 0
Look multiple lines!
Neat!
test>

New output
$ dotnet fsi test.fsx
test> help
num: 0
error: showing help.

usage: main [options] <command>

The main command.

commands:
  echo [options]                    Echo the input.

options:
  -n, --number=<num>                integer number.
  -v, --verbosity={q(uiet)?$|n(ormal)?$|f(ull)?$|c{ustom:<level>|:<level>}}
                                    Display this amount of information in the log.
  -f, --file=<filename>[:<index>]   Name of a file to use
                                    Default index: 0
                                    Look multiple lines!
                                    Neat!
test>

Thank you very much for your contribution! Looks good to me in general. I will be doing the versioning and creating a release myself, so can you revert the last commit bumping the version?

Thank you very much for your contribution! Looks good to me in general. I will be doing the versioning and creating a release myself, so can you revert the last commit bumping the version?

Alright! Sure enough :)

Alright! Sure enough :)

Thank you 😊

Thank you 😊


Update namespaces

Prepend Microsoft.JavaScript. to all NodeApi namespaces.
Add Interop namespace for that subfolder in main library.
Rename Hosting namespace to DotNetHost.

I am NOT creating a Microsoft.JavaScript.NodeApi.Native namespace yet, because that will require more refactoring. Lots of code depends on extension methods in JSNativeApi. I think those should be refactored to methods or extension methods in the root namespace, then JSNativeApi should be internal. Also maybe code within JSNativeApi can be organized into more separate types, since they will all be in an internal namespace.

I am currently changing the way we do the interop with Node-API functions. Could you postpone changes to JSNativeApi for a couple of days? I agree with you: we should probably delete this class completely and spread its code between other types.

Could you postpone changes to JSNativeApi for a couple of days?

I made some minor changes in that area over the weekend, for this PR: https://github.com/jasongin/napi-dotnet/pull/39
After that I don't expect to be working there any time soon.


Added warbler and updated documentation

Added some documentation and an example where warbler is relevant.
Let me know what you think.

Hey so overall this looks good and really thankful for your help, not just this PR but already throughout the investigation in the issue you have logged. I saw you already started making some more changes based on the comments, so let me know when you are all done and then I'll merge it.

Ok, I think I've got everything in place. Glad we were able to resolve #31.
It's nice working on this project, happy to help.

Awesome, many thanks!!


Issue: Unzip archives which contains large files

I have a zip archive which contains several files and one of this file has 421MB size.
Grunt unzip throws the following error:
Warning: Invalid string length Use --force to continue.
Example:
unzip: {
    src: "dir/*.zip",
    dest: "dir"
}

The archive doesn't seem to be corrupt and it works if I used 7z for example.
I can fix this issue by setting checkCRC32: false but I don't know why it is needed.
P.S I can't provide the archive since it contains client information.

Would it be possible for me to get a copy of the file? It will be hard for me to debug it otherwise =/

Sorry, but unfortunately I can't provide you that file, it is confidential.
I know it is hard for you to debug, you can try with different types of files and with larger size than 400MB, maybe you can find out something. Thank you!

Maybe you can generate another ZIP file via the same confidential process and see if it generates the same error? Based on your CRC32 patch, I have a feeling it's specific to the file

I have more than one archive with at least one file larger than 400MB inside and I got the same issue. I'll continue to investigate on my side for those specific files.

Cool, I'm going to close this issue for now since it's blocked on my side. We can re-open if we have something to iterate on


Fix grammar in contoso-case-study.md

Fix grammar based on issue #3805

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@Kateyanne Please merge this PR. Thanks.

@Kateyanne Please merge this PR. Thanks.


Fix the "Not Found" view and implement an e2e test

ref #250

Can someone review this? (Ping @diegovdc) Would be nice to have a working 404 in the next update.

Looks good. Tested adding a scale, going to a non existent page, playing the synth, and going through the octaplex flow.


New extension PlatformLedgeGrabber.

This behavior makes the platformer character automatically grabs platforms ledges when they are within reach.
Additional constraints can be added to:

make the grabbing range smaller
only grab ledges the character is facing

Note that it won't work with a custom collision mask, but it's not advised to use one with a platformer character anyway.
This extension tries to answer this ticket:

https://trello.com/c/zezNvn1J/269-add-an-option-to-automatically-grab-the-ledge-of-platforms

Example

Project: https://www.dropbox.com/s/7mz13em7a4dr05a/platformer-autograb.zip?dl=1
Build: https://liluo.io/instant-builds/f952fb6a-81b5-493d-a029-c8746c966d79

JavaScript
const behaviorName = eventsFunctionContext.getBehaviorName("Behavior");
const object = objects[0];
const behavior = object.getBehavior(behaviorName);

/** @type {gdjs.PlatformerObjectRuntimeBehavior} */
const character = object.getBehavior(behavior._getPlatformerCharacter());

const isUsingLeft = character.isUsingControl("Left");
const isUsingRight = character.isUsingControl("Right");

behavior.usedLeftLast = (behavior.usedLeftLast || isUsingLeft) && !isUsingRight;

if (!character.isFalling()
    || (isUsingLeft && !behavior._getIsMovingLeftAutomatically())
    || (isUsingRight && !behavior._getIsMovingRightAutomatically())) {
    return;
}
const wasMovingRightAutomatically = behavior._getIsMovingRightAutomatically();
const wasMovingLeftAutomatically = behavior._getIsMovingLeftAutomatically();
behavior._setIsMovingRightAutomatically(false);
behavior._setIsMovingLeftAutomatically(false);

/** @type {gdjs.PlatformObjectsManager} */
const manager = gdjs.PlatformObjectsManager.getManager(runtimeScene);
/** @type {float} */
const margin = behavior._getLedgeMarginX();
/** @type {boolean} */
const isOnlyGabbingFacingLedges = behavior._getIsOnlyGabbingFacingLedges();

if (!behavior.potentialGabbablePlatforms) {
    behavior.potentialGabbablePlatforms = [];
}
/** @type {PlatformRuntimeBehavior[]} */
const platforms = behavior.potentialGabbablePlatforms;
manager.getAllPlatformsAround(object, margin, platforms);

// Find the nearest platform that can be grabbed.
let platformDistanceMin = margin;
let shouldMoveLeft = false;
let shouldMoveRight = false;
const canGrabLeft = !isOnlyGabbingFacingLedges || !behavior.usedLeftLast;
const canGrabRight = !isOnlyGabbingFacingLedges || behavior.usedLeftLast;
const aabb = object.getAABB();
for (const platform of platforms) {
    /** @type {gdjs.AABB} */
    const platformAABB = platform.owner.getAABB();
    const characterIsAtTheLedgeLevel = aabb.max[1] > platformAABB.min[1] && aabb.min[1] <= platformAABB.min[1];
    if (characterIsAtTheLedgeLevel) {
        const leftDistance = platformAABB.min[0] - aabb.max[0];
        if (canGrabLeft && 0 <= leftDistance && leftDistance <= platformDistanceMin) {
            shouldMoveRight = true;
            shouldMoveLeft = false;
            platformDistanceMin = leftDistance;
        }
        const rightDistance = aabb.min[0] - platformAABB.max[0];
        if (canGrabRight && 0 <= rightDistance && rightDistance <= platformDistanceMin) {
            shouldMoveLeft = true;
            shouldMoveRight = false;
            platformDistanceMin = rightDistance;
        }
    }
}

// Don't try to move to a ledge that is too far to be actually grabbed.
let canReachLedge = false;
if (shouldMoveLeft || shouldMoveRight) {
    const alreadyDecided = wasMovingLeftAutomatically || wasMovingRightAutomatically;
    if (alreadyDecided) {
        canReachLedge = true;
    }
    else {
        // Check if the ledge is actually reachable
        const currentSpeedY = Math.abs(character.getCurrentFallSpeed() - character.getCurrentJumpSpeed());
        const timeDelta = object.getElapsedTime(runtimeScene) / 1000;
        // Rough approximation of the time frame
        const t = (aabb.max[0] - aabb.min[0]) / ((currentSpeedY + character.getMaxFallingSpeed()) / 2) - 2 * timeDelta;
        //console.log("time frame: " + t);

        const accelerationX = character.getAcceleration();
        const maxSpeedX = character.getMaxSpeed();
        const currentSpeedX = Math.abs(character.getCurrentSpeed());

        let displacementMaxX = 0;
        if (currentSpeedX === maxSpeedX) {
            displacementMaxX = maxSpeedX * t;
        }
        else {
            const maxSpeedXTime = Math.min(0, (maxSpeedX - currentSpeedX) / accelerationX);
            if (t < maxSpeedXTime) {
                displacementMaxX = currentSpeedX * t + accelerationX * t * t / 2
            }
            else {
                displacementMaxX = currentSpeedX * maxSpeedXTime
                    + accelerationX * maxSpeedXTime * maxSpeedXTime / 2
                    + maxSpeedX * (t - maxSpeedXTime);
            }
        }
        //console.log((platformDistanceMin - character._xGrabTolerance) + " < " + displacementMaxX);
        canReachLedge = platformDistanceMin - character._xGrabTolerance < displacementMaxX;
    }
}
// Move toward the ledge.
if (canReachLedge) {
    if (shouldMoveLeft) {
        //console.log("Move left");
        behavior._setIsMovingLeftAutomatically(true);
        character.simulateLeftKey();
    }
    else if (shouldMoveRight) {
        //console.log("Move right");
        behavior._setIsMovingRightAutomatically(true);
        character.simulateRightKey();
    }
}
// Need to hold the key one frame after reaching the ledge to grab it.
if (wasMovingLeftAutomatically && !shouldMoveRight) {
    character.simulateLeftKey();
}
else if (wasMovingRightAutomatically && !shouldMoveLeft) {
    character.simulateRightKey();
}

This isn't better in the engine itself?
On the PlatformerObject we have already a few settings about ledge grabbing.

This isn't better in the engine itself? On the PlatformerObject we have already a few settings about ledge grabbing.

Yes, we should at least add an option to grab without pressing Left or Right in the engine, but this extension also moves the character automatically toward the ledge and I'm not sure if this part should be in the engine. The huge benefit of having it in the engine would be to avoid this 2nd spacial search (which also breaks encapsulation a bit).
Do you think the whole extension should be integrated in the engine?

Hello @D8H 👋
A review has started for this extension, but a new option has presented itself in the meantime. We now have a list of unreviewed "Community extensions", and your extension can be added to it, bypassing the review phase.
If you chose to not take a review, you need to keep the following in mind:

As of now, community extensions are not yet shown to users, but they should be until next update
Community extensions will never be shown to GDevelop users directly, they will have to check a checkbox for them to be shown
All community extension have a warning about them being unstable and unreviewed displayed on their store page
Your extension will be added immediately, no question asked, and without you needing to do any changes to your extension
You will still need to make an extension that passes the bot's tests (meaning you still should respect the extension best practices)

tl;dr You will not need to do any more (or only minimal) work on your extension, but it might not meet the standards of all users, and reach less users.
If you chose to continue on with the review:

The reviewing process might require of you to make some changes to your extension until it meets our reviewer's standards
Your extension will reach more users
You will be delivering a better extension to the community
It will take a bit more time for your extension to be added to the store

tl;dr You will need to do a bit more work but deliver a high quality contribution that will reach more people
The choice is yours, please tell us how you would like to continue.

I'll continue with the review process, but probably push it in the community folder too at some point.

I think this extension doesn't worth a full review.


Update spectests to current codestatus

When running on Windows, all rspec tests are working again.

Fixed as part of #63


MPVERIFY instruction potential refactoring

MPVERIFY computes the Merkle root of a path for a given specified node in the stack. The stack is arranged as follows (from the top):

depth of the path, 1 element.
index of the node, 1 element.
value of the node, 4 elements.
root of the tree, 4 elements.

Currently, post this operation, we are replacing the node value with the newly computed root calculated by the hasher co-processor. The stack will look like this post this operation:

depth of the path, 1 element.
index of the node, 1 element.
computed root of the path, 4 elements.
root of the tree, 4 elements.

We are already asserting the computed root of the path with root of the tree and repeating this in assembly(mtree.get) wastes couple of clock cycles. As noted by @bobbinth:

"Instead of replacing the node with the newly computed root, we should just copy over the stack to the next cycle. The instruction should fail if the roots are not the same".

Operations relying on MPVERIFY might get a bit simplified due to fewer VM cycles needed now.
edit:
The instruction also needs to be populated with six helper registers which are as follows:

row address of the start of the computation in hasher chiplet - 1 element
the least significant bit of the node index - 1 element
sibling node of the leaf - 4 elements

The mtree.get assembly instruction also needs to be updated to be in line with the new changes made in MPVERIFYas there are some potential optimization possibilities..

Closed by #318


Custom routes + Further notification cleanup

Summary of Changes
This PR adds a new feature to allow the configuration of custom and completes some cleanup in the notifications code.

There is a new config variable INIT_CUSTOM_ROUTES. It is a Callable that takes a Flask application and is called during create_app. This allows individuals to configure custom endpoints to support custom logic. In our case at Lyft, we are using it to hook on a new endpoint to our frontend application that will process some internal callback to generate some custom metrics.

In our python code I have added an enum class NotificationType, that matches the NotificationType interface in the frontend application, and replaced all usages of hardcoded notification type strings to leverage the enum class. This makes it easier to maintain the code snd prevent some human error when it comes to writing our notification logic.

For notification emails, instead of passing "notification" as the email type, we will pass the exact notification_type instead. This is more helpful because it will allow custom mail clients to execute specific logic based on the notification_type.

In _build_metrics() in action_log.py we now catch any exceptions that occur when getting the user for metrics, and fallback to the default. The reason is that actions that are logged in any of the custom routes, if that endpoint was hit by another service as opposed to being the result of a user action -- the AUTH_USER_METHOD might encounter some error.

Tests
Added a python test to cover calling get_notification_subject() with an unsupported notification type.
Documentation
Added documentation for INIT_CUSTOM_ROUTES to flask_config.md.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes, including screenshots of any UI changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes all tests documented in the developer guide

Codecov Report

Merging #309 into master will increase coverage by 0.02%.
The diff coverage is 83.33%.

@@            Coverage Diff            @@
##           master    #309      +/-   ##
=========================================
+ Coverage   71.57%   71.6%   +0.02%
=========================================
  Files         143     143
  Lines        3684    3701      +17
  Branches      459     462       +3
=========================================
+ Hits         2637    2650      +13
- Misses       1014    1018       +4
  Partials       33      33

Impacted Files
Coverage Δ

amundsen_application/config.py
100% <100%> (ø)
:arrow_up:

..._application/static/js/interfaces/Notifications.ts
100% <100%> (ø)
:arrow_up:

...undsen_application/api/utils/notification_utils.py
91.83% <76.47%> (-3.35%)
:arrow_down:

Continue to review full report at Codecov.

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

@jinhyukchang can you give your input on the changes in amundsen_application/log/action_log.py?
Now that I think about it, we could also modify our our custom authentication method and return some default "service user" for our new use case where a service is hitting a custom endpoint as opposed to a real user. What do you think?

Reverted the change to _build_metrics in favor of a modification to our custom AUTH_USER_METHOD.

@danwom PTAL

Looks fine on my end, but maybe you'll want to get @jinhyukchang to look as well?

Jin was fine with the changes I tagged him to look at in our private repo, will go ahead and merge this.


fix: increasing Postgres connection pool size

Description
This PR increases the Postgres connection pool size from 5 to 30 to handle request spikes properly.
During the request spikes there are Store(Database(PoolTimedOut) errors, that should be fixed by increasing the pool size. The RDS state at this moment remains stable (ACU <= 30%), so it should be handled without scaling.
How Has This Been Tested?
Unit testing.
Due Diligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

Why is the change under /tests?

last git push was missed, I didn't think you would review so fast 😅👍


--fmt-report option for cspt does not work

The option creates an output file mt-report.txt instead of writing it out in the terminal

remember to link to this to log your community badge


Crash with v2.1.2

Before upgrading to v2.1.2, my program worked well with v2.0.9. However, after upgrading to v2.1.2, the program crashes when starting up.
I also tried v2.1.0 and v2.1.1, they are good.
I’m using VS2022 with C++ 17 and running on Windows 10 (64-bit). I’ve set the following options:
#ifdef _WIN32
	mi_option_enable(mi_option_t::mi_option_verbose);
	mi_option_set_enabled(mi_option_e::mi_option_verbose, true);
	mi_option_set_enabled(mi_option_e::mi_option_abandoned_page_purge, true);
	mi_option_set(mi_option_e::mi_option_purge_delay, 50);
#endif

“If I comment out the lines below, it works well.
//	mi_option_enable(mi_option_t::mi_option_verbose);
//	mi_option_set_enabled(mi_option_e::mi_option_verbose, true);

Hi @daanx , sorry for bothering, could you please help to check this?  It works fine with v2.0.9, 2.1.0, and 2.1.1.
Please let me know need I provide more information.
Thanks

Is this using mimalloc redirection?

Is this using mimalloc redirection?

Yes, also override the new delete as the guide.

I think I know what is the cause; thanks for reporting!
The workaround would be to disable verbose, or comment out just the 2 verbose calls in mi_reserve_os_memory_ex (in arena.c).
However, the root cause is troublesome; mimalloc uses vsnprintf before the C runtime is  initialized; I thought that was ok but it seems not the case (it might depend on initialization order as I cannot repro) -- this might be due to locale's I guess. Not sure yet how to fix this.. tbc

It's possible that the vsprintf_l versions might work - https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/vsnprintf-s-vsnprintf-s-vsnprintf-s-l-vsnwprintf-s-vsnwprintf-s-l?view=msvc-170 - where you explicitly provide the locale, so the thinking here is that there is no need to create global variable (from the CRT). Since the source code is available (as part of the SDK / Visual Studio) it can be checked (I'm going to do that myself, and put some findings here)

What might work in this case, if instead of using vsprintf, use the Windows SDK function - e.g. some variant of these - https://learn.microsoft.com/en-us/windows/win32/api/strsafe/nf-strsafe-stringcchvprintf_la


input array from shape error

input array from shape I use (640, 480, 3) and it always throw out can't into shape (480, 640, 3)
and when i use (480, 640, 3) and it throw out can't into shape (640,480,3)
How can i fix this problem?
I use Python3.5 and Keras 2.2.4
Traceback (most recent call last):
  File "train/train_pistachio.py", line 104, in <module>
    workers=10
  File "/usr/local/lib/python3.5/dist-packages/keras/legacy/interfaces.py", line 91, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/keras/engine/training.py", line 1418, in fit_generator
    initial_epoch=initial_epoch)
  File "/usr/local/lib/python3.5/dist-packages/keras/engine/training_generator.py", line 181, in fit_generator
    generator_output = next(output_generator)
  File "/usr/local/lib/python3.5/dist-packages/keras/utils/data_utils.py", line 601, in get
    six.reraise(*sys.exc_info())
  File "/usr/local/lib/python3.5/dist-packages/six.py", line 693, in reraise
    raise value
  File "/usr/local/lib/python3.5/dist-packages/keras/utils/data_utils.py", line 595, in get
    inputs = self.queue.get(block=True).get()
  File "/usr/lib/python3.5/multiprocessing/pool.py", line 608, in get
    raise self._value
  File "/usr/lib/python3.5/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/lib/python3.5/dist-packages/keras/utils/data_utils.py", line 401, in get_index
    return _SHARED_SEQUENCES[uid][i]
  File "/usr/local/lib/python3.5/dist-packages/keras_preprocessing/image.py", line 1441, in __getitem__
    return self._get_batches_of_transformed_samples(index_array)
  File "/js/data/RotNet/utils.py", line 302, in _get_batches_of_transformed_samples
    batch_x[i] = rotated_image
ValueError: could not broadcast input array from shape (480,640,3) into shape (640,480,3)

Please post a short code snippet or some sort of description of what you are doing, so that I can reproduce the error and help you.

Hello @imycc
did you solve this issue i have same issue??


Remove regex cludge for docs version checking

Now we have denormed sortable version numbers, we can use those. Saves
us getting in a mess when discontinued docs versions start reaching
2.x, at least.

+10 for "cludge" 😂


Improve punishment handling for shared IPs

Made the changes you requested @Zarel

Ok I also did something else. This PR is kinda broader than the original intention now, so I changed the title accordingly.


⚠ Bofrost.de has degraded performance

In 394b629, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1149 ms

Resolved: Bofrost.de performance has improved in 8e1a882 after 20 minutes.


Clear telescope search results

After populating the Trouble list with Telescope search results, is it possible to clear them somehow? Otherwise they continue to clutter up the list and there seems to be no way to "resolve" them.

That's actually intentional. But if you want to clear them you could to the following (or bind it to a key):
:lua require("trouble.providers.telescope").results = {}; require("trouble").close()


[question] Native kubernetes spark vs spark-on-k8s-operator?

As far as I can tell there is a native version of spark on kubernetes:
https://spark.apache.org/docs/latest/running-on-kubernetes
And there is the spark-on-k8s-operator:
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator
In the native version you can directly submit spark jobs while on the k8s version we would write a SparkApplication yaml file. Now I'm wondering if the spark-on-k8s-operator makes use of the native application? And If so would it then be possible to directly submit jobs using spark-submit.

There is only one native version of spark on k8s. Spark operator wraps it and provides a k8s-native interface so that you can run your apps using yamls+kubectl (or another k8s client). But it doesn't make it a part of apache spark. Just in the same way as jupyter notebooks can run spark code, but you can't work with a notebook from spark-submit. Jupyter is not part of spark. So as spark operator.
In theory, your case is possible, but you've asked this question in the wrong place: spark-submit script is owned by apache spark community.

As far as I know Spark operator lets users do spark-submit commits using yaml and takes care of complex configuration issues.
As I answered here, if you want spark application on k8s, you can use client mode and configure it on k8s (spark.master, etc.) in the application. Pyspark users can use the sparglim package to build client mode applications quickly.


IEnumerable.IsNotEmpty

Hey!
I do like the lib, can you please add IEnumerable.IsNotEmpty to prevent checking the IEnumerable.IsEmpty == false ?
I can do a PR, if you want =)

You can call the existing IEnumerable.Any() which actually is the same as IEnumerable.IsNotEmpty.


Compaction is failing due to invalid data and index file path being set

When compaction process is about to flush the data to .data and index files, the file location it has is not valid. The file location has root directory repeated multiple times, such as /tmp/tmp/179845.data

Fixed now, updated the code to not repeat the root path


Refactor startup of notebooks/sessions into a separate class

See here https://github.com/microsoft/vscode-python/pull/9227#discussion_r360157935
E.g. interactiveBase calls into DataScience class to get a list of kernels...
Feels wrong.
DataScience class turns into a utility class (kitchen sink)...

Closing in favor of https://github.com/microsoft/vscode-python/issues/9269


Adjust notifyObservers iteration when unwatch function is called during iteration [Fixes #151]

Proposed fix for #151.
This PR takes the somewhat ugly approach of reading/modifying the notifyObservers iteration from within the unwatch function. This won't work if notify loops can be nested, but I don't think that is possible/allowed?
The cleaner approach of taking a copy of __observers to iterate would mean that observers later in the array that might be unsubscribed during the iteration would still receive a notification.
(Might cause conflict with merging #115 which touches the same code, but retains the same bug.)

This also means that new notifiers will be called if registered during
notify?
Making a copy would make more sense to me and is probably more predictable
behavior for new developers.
On Sat, Aug 15, 2015, 09:45 Chris Spencer<EMAIL_ADDRESS>wrote:

Proposed fix for #151
https://github.com/optimizely/nuclear-js/issues/151.
This PR takes the somewhat ugly approach of reading/modifying the
notifyObservers iteration from within the unwatch function. This won't work
if notify loops can be nested, but I don't think that is possible/allowed?
The cleaner approach of taking a copy of __observers to iterate would
mean that observers later in the array that might be unsubscribed during
the iteration would still receive a notification.
(Might cause conflict with merging #115
https://github.com/optimizely/nuclear-js/pull/115 which touches the
same code, but retains the same bug.)
You can view, comment on, or merge this pull request online at:
https://github.com/optimizely/nuclear-js/pull/152
Commit Summary

Adjust notifyObservers iteration when unwatch function is called
during iteration [Fixes #151]

File Changes

M src/change-observer.js
https://github.com/optimizely/nuclear-js/pull/152/files#diff-0 (16)
M tests/change-observer-tests.js
https://github.com/optimizely/nuclear-js/pull/152/files#diff-1 (11)

Patch Links:

https://github.com/optimizely/nuclear-js/pull/152.patch
https://github.com/optimizely/nuclear-js/pull/152.diff

—
Reply to this email directly or view it on GitHub
https://github.com/optimizely/nuclear-js/pull/152.

The current behaviour is that up to n new notifiers are currently called when registered during a notify, where n is the number of observers removed during that notify. If there's a desired behaviour one way or the other it can be changed (by adding iterationLength and adjusting that too).
Agreed on an array copy being nicer, however I don't think it gives predictable behaviour, as calling unregistered handlers could lead to very hard to track bugs due to dependency on observer ordering.
For instance when working with React, handlers a, b where a unmounts a component and b does something with that same component. If they skip an isMounted() check then certain component APIs will throw if b lacks an isMounted check. But it'll work fine whenever they're ordered b, a. (Handler a could be way up the component hierarchy than handler b, giving spooky-action-at-a-distance.) In development you'd just get a mounting error and fix it, but in production it could cause your application to break seemingly at random.
An alternative approach would be to .slice(0) the observers, and have unwatch functions set some __unwatched property on themselves that the notifyObservers iteration can check for to avoid running unregistered handlers. I can raise that alternative PR if wanted.

What if the isMounted() check would be added to the react mixin?

I think the ReactMixin only calls setState which React recently made tolerant (well, exception -> warning) to being called on unmounted components. In any case, #153 is probably a simple more palatable fix which also does not call new notifiers.

This is fixed in #153


Custom fonts in Workbench session

In non-k8s installations placing custom fonts in /etc/rstudio/fonts works fine.
In k8s sessions however, fonts in this place are not picked up by the IDE. The installation procedure is scripted and the font files are present in the session pods under /etc/rstudio/fonts.
Is additional linking needed to make the custom fonts being picked up in a k8s session? Or do they need to be placed in /mnt/rstudio/session-configmap instead?

Yeah, you probably want to put them in session-configmap. If they are baked into the image, you should bake them into /mnt/dynamic/rstudio, because session-configmap gets overwritten by a configmap 😄
Check out XDG_CONFIG_DIRS env var to see where config gets picked up from 😄

Thanks!
echo $XDG_CONFIG_DIRS
/mnt/dynamic:/mnt/session-configmap:/mnt/secret-configmap:/mnt/configmap:/mnt/load-balancer/

And yes, /mnt/dynamic/rstudio/fonts did the trick! 🎉


(Question) : Proxy Frontend rule to another machine

Before moving to traefik, I look uppon the documentation and couldn't find if it's possible to have a front end rule to forward to another machine which doesn't run any containers?
For example, I have machine A and B.
A receives all trafic and is running traefik
B is running apache and handle some specific (sub)domains: let's say magic.example.com
So if I try to access to code.example.com, traefik will redirect to the according docker container, same for drone.example.com.
But when I try to access to magic.example.com, it should redirect to machine B which will handle its own way.
Is it possible to do that?
If yes, could you point me to the doc (because I couldn't find anything) or the configuration I should have?
Thanks in advance.

:+1:

I'm also looking for similar features. I have services which only offers proxy (HTTP or SOCKS) connection mode to manage traffic but need to configure consumer based only on URL. I'm expecting to use solutions like traefik to expose infrastructure as URL and then reroutes traffic to these services as proxy.

I'll close the issue due to inactivity but feel free to re-open if needed.


Migrate create_comment.jsx to be pure and use Redux

Summary
Migrate create_comment.jsx to be pure and use Redux
Ticket Link
https://github.com/mattermost/mattermost-server/issues/7672
Checklist

[x] Added or updated unit tests (required for all new features)
[x] Touches critical sections of the codebase (auth, posting, etc.)

Thanks @QuantumKing for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

Assigning myself so I remember to follow up. Also, Jenkins isn't building because there's merge conflicts

@QuantumKing Going to merge this now. Thanks for making those changes


Multiple Navigators: When pushing a new route on a separate navigator textfield loses focus

Steps to Reproduce
I have a Master/Detail view on a tablet/desktop app where i display two Navigators side by side. When the user is searching in the Master View (ie. text field is focused) it loses focus if i push another route into the second (detail) navigator.
I have created a simple example: https://dartpad.dartlang.org/aa423731de97aef1160c8b9b812ad1c9
setting the focus in the text field and typing a key will blur the text field.
              child: TextField(
                onChanged: (newValue) {
                  _navigatorKey.currentState.pushAndRemoveUntil(
                      MaterialPageRoute(builder: (_) => PageWithTextField()),
                      (route) => false);
                },

Expected results: Right (detail) navigator navigates to the new page, Focus remains in the text field on the left side.
Actual results: TextField loses focus, user cannot keep typing.
As far as I understand it, having multiple Navigators is perfectly fine? I am not 100% sure if it is supported behavior to have two navigators side by side?
Is there some way to push a route without losing focus on the textfield?

Hi @hpoul
I don't think there's anything wrong with multiple navigators,
but for my understanding losing focus has to be expected;
I'd interpret your issue more as feature request rather than a bug
if you agree with that.
Thank you

@iapicca ok sure.. although I personally did not expect it, since the route i'm pushing does not contain any input elements with autofocus = true, so i did not expect that it would change the focus. But I guess I can understand why it's happening looking at the code..
thanks for taking a look.

Chiming in, I'm facing this issue in my app and it's mind blowingly annoying. This behavior needs to be configurable. I have a textfield that updates a navigator as text is entered. Because navigator keeps stealing focus, this means that the keyboard keeps disappearing while the user is trying to type.
This needs to be configurable. eg add the following parameter to navigator: bool takeFocus = true.
@hpoul  did you ever find a work around to stop navigator from stealing focus?

did you ever find a work around to stop navigator from stealing focus?

@caseycrogers yeah, I've got a custom page route which simply force the focus back into the text field a frame after the push happened https://github.com/authpass/authpass/blob/6a019bb6bfeb4b990dc6d31f92d80f6b7d624f4d/authpass/lib/ui/screens/main_app_scaffold.dart#L61 ... Works surprisingly well.. to the point that I forgot that there is a workaround in place 😅

did you ever find a work around to stop navigator from stealing focus?

@caseycrogers yeah, I've got a custom page route which simply force the focus back into the text field after the push happened https://github.com/authpass/authpass/blob/6a019bb6bfeb4b990dc6d31f92d80f6b7d624f4d/authpass/lib/ui/screens/main_app_scaffold.dart#L61 ... Works surprisingly well.. to the point that I forgot that there is a workaround in place 😅

Inspired by your workaround, I created the following which is a little bit simpler and may be more desirable in some cases (it prevents the route from ever taking focus instead of letting it take focus and then stealing focus back):
  @override
  TickerFuture didPush() {
    navigator!.focusScopeNode.canRequestFocus = false;
    final ret = super.didPush();
    // Could have a race condition because ret is just waiting on ticker future,
    // NOT waiting on it's callback on top of ticker future.
    // This code assumes that callbacks are called in FIFO order, this may not
    // be a valid assumption.
    ret.whenComplete(() {
      navigator!.focusScopeNode.canRequestFocus = true;
    }); // focusNode.requestFocus());
    return ret;
  }

  @override
  void didAdd() {
    navigator!.focusScopeNode.canRequestFocus = false;
    super.didAdd();
    // This probably has a race condition with `TransitionRoute.didAdd`. I don't
    // think there's a way around it because the former is waiting on an
    // internal future that I have no access to.
    TickerFuture.complete().then((value) {
      return navigator!.focusScopeNode.canRequestFocus = true;
    });
  }```

If the Flutter team won't take my PR I guess we'll just have to live with the workarounds. That said I really don't trust them. They set off my race condition spidey sense.


Please add author field

Like this one: https://github.com/ioBroker/ioBroker.hm-rpc/blob/master/io-package.json#L103

Hello Bluefox,
Done :)


Menu works on Android phone with Web browser, but not as phonegap app

Following the instructions, I built a presentation using Reveal. I then added the menu. It works on my lap top. I deployed it at github.io and it works there. When I put that same respository thru build.phonegap.com, the resulting app has the navigation from reveal, but no hamberger menu.
Any suggestions where to look to debug this?

Sorry, I don't have any experience with phonegap. Not sure I can be of any help here.


Installing in Android gives  "package com.learnium does not exist"

error: package com.learnium does not exist
import com.learnium.RNDeviceInfo;
                   ^

You are missing the .* at the end of RNDeviceInfo:
import com.learnium.RNDeviceInfo.*;

In MainActivity.java I included the .* but when the error appears the .* doesn't show.

Have you included it in build.gradle?

OK, I uninstalled it, did all the steps again and now it's fine.
Maybe some problem with npm install..
Thanks!


Custom recipes doesn't work.

I want to edit some recipes from EIO machines so I wrote  this code in user config but... it didn't work.
It is a bug or my code is incorrect?

Did you check the log files to see if there are any errors?

Please first start the game and look in the logfile. There should be some messages about recipe loading.

<a href"http://pastebin.com/gWWAxWTx">This?<a>

This?
http://pastebin.com/gWWAxWTx

Yes. However, I didn't expect it to fail that hard. Have me look into the code, why it crashes instead of giving you an error message. Which Ender IO version are you using?

Code is in link in first post.
I try to use different versions of EnderIO. The newest beta version and the last release.
EnderCore which I use is always the newest beta.

From the crash it looks like you are creating a recipe group without a name. However, the xml you posted has only one group and that one has a name...

And?


Enchiridion not opening

The book's not doing anything when I right-click it. Im doing this in the Hex MC Modpack and also its the 1.18.2 version of it (Conjuring 1, not 2). Any ideas?

Do you have a link to the modpacks page? As long as you have Patchouli installed the book should open.

Could you please post your latest.log? It should contain an error when you open an invalid book

Uhhh... Im still playing but havent tried to open it for a few days already.

Where can I find it?

https://fabricmc.net/wiki/player:tutorials:logs_ml:windows

nvm I think I found it
latest.log

https://pastebin.com/vyuTSJkT

AFAIK the Patchouli compat was broken in 1.0.21+1.18, so you should probably upgrade Conjuring to 1.0.22+1.18

Huh?

But the only 1.0.22 is the Conjuring 2, not 1.

Ok! Thanks guys! Im gonna try that now and if it fixes it, ill close the issue.

Indeed it works now! Thanks a lot again and... CLOSED!

... Actually Idk how to close it but its definitely close-able now.

This button would have been your best friend, but fret not, I've now done it for you
Cheers


Add swift driver

Adding a PoC swift-all-in-one driver. The example
deploys a single instance of an SAIO that can be used
to test the Swift API.

Verified:
$ curl -i http://<IP_ADDRESS>:31558/healthcheck
HTTP/1.1 200 OK
Content-Length: 2
Content-Type: text/plain
X-Trans-Id: tx134c3fd7890c4495bc5d1-0058e57f64
Date: Wed, 05 Apr 2017 23:36:04 GMT

OK


Feature request: permutation test

This should be relatively straightforward to implement: type permutationTest :: (v a -> v b -> Double) -> Int -> PValue, i.e. takes in a function that takes a vector of labels and a vector of samples to report a test statistic, a number of random runs, and returns the p-value (possibly with the observed test statistic of the samples with labels).

I don't understand what this test should do. Function with provided signature could only compute PValue from Int and doesn't seems to be useful

You're absolutely right. I meant:
permutationTest :: (v a -> v b -> Double) -> Int -> v a -> v b -> PValue
That is, (function of labels, samples to test statistic) -> number of runs -> labels -> samples -> p-value.

It would be basically (with mwc-random):
-- | Permutation test returning p-value.
permutationTest :: (G.Vector v a, G.Vector v b)
                => (v a -> v b -> Double)
                -> Int
                -> v a
                -> v b
                -> IO Double
permutationTest f runs labs samples = do
  let obs = f labs samples
      run r = do
        g <- R.initialize . V.singleton $ fromIntegral r
        ls <- R.uniformShuffle labs g
        return $ f ls samples

  n <- genericLength . filter (>= obs) <$> mapM run [1..runs]

  return $ n / fromIntegral runs

Although it should be permutationTest :: (v a -> b -> Double) -> Int -> v a -> b -> PValue to generalize the sample structure (it could be a matrix or tree, for instance).


[Feature] Choose direction of shadow

Sometimes having a shadow all around the pancake view is too much. Sometimes I would like a shadow only below the item. Could this be done by changing the shadow's shape to a semicircle?

Will merge all shadow improvements into #86


Add Data to widget::prelude

Yes I fought this for a while on the condition that the prelude was exclusively things required explicitly to implement widget, and data isn't in any of the widget signatures, but it's a big enough gotcha I'm happy to do this now.

Yes I fought this for a while on the condition that the prelude was exclusively things required explicitly to implement widget, and data isn't in any of the widget signatures, but it's a big enough gotcha I'm happy to do this now.


[topic] fix missprint for topictypes

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] 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?

Other information

Codecov Report

Merging #354 (320ce68) into master (8abc51f) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #354      +/-   ##
==========================================
- Coverage   35.05%   35.01%   -0.05%
==========================================
  Files         116      116
  Lines       16151    16151
==========================================
- Hits         5662     5655       -7
- Misses      10267    10273       +6
- Partials      222      223       +1

Flag
Coverage Δ

1.16.x
34.62% <ø> (ø)

1.18.x
35.00% <ø> (-0.07%)
:arrow_down:

1.19.x
35.02% <ø> (-0.05%)
:arrow_down:

macOS-latest
34.99% <ø> (ø)

ubuntu-latest
35.01% <ø> (+0.01%)
:arrow_up:

unit
35.01% <ø> (-0.05%)
:arrow_down:

windows-latest
?

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

Impacted Files
Coverage Δ

...al/topic/topicreaderinternal/stream_reconnector.go
81.16% <0.00%> (-4.49%)
:arrow_down:

internal/table/client.go
79.46% <0.00%> (+0.47%)
:arrow_up:

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


LGPL license?

Would you consider a license more suitable for use as a library in non-GPL-licensed project?
Thank you!

We decided to move to an apache license.

@adamdeprince Thank you for following up on this.


Make the library more 'robust'

Hi @BenjaminFavre
Thank you for your great library. It does its job perfectly!
I would love to make the library more robust by

[x] adding types (you already require PHP8)
[x] adding Psalm for static analysis
[ ] adding tests
[x] adding coding standard
[x] and adding a github actions workflow for verifying the above

Wdyt?
Have a great day 👋

Hi @loevgaard
The lib already fits my personal needs so I don't spend more time on it, but I would definitely welcome your contribution.
Feel free to make the library more robust, I will merge any PR in this sense, your 4 suggestions are relevant!
Thanks!

Good morning, @BenjaminFavre
I have added a few PRs. Let me know what you think :)

Let me know when you're done working on the lib so I create a new tag ;)
I can also create an intermediary tag if you need one for your personal use.

My plan is to add some tests at some point to this library, Benjamin, but I am not sure when. Inspired by your decorator I created another decorator that you might find useful in debugging scenarios: https://github.com/Setono/request-aware-http-client.
Again, thanks for creating this library 🤜 🤛

Nice ! I used to write all this painful logging around HTTP calls in each and every project that required it, next time I'll try to go with your decorator !
Thanks again for contribution, I created a tag containing all your work so far :pray:

Nice, thanks :)


[dv/top] Update wdog_lc_escalate test

the test was previous faililng because the busy_spin_macro
has a slightly different cycle count from the escalator phase.
this caused the test to mistakenly think the alert was not triggered
even though it just needed to wait a little bit more time.

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

Thanks @tjaychen for the PR, I think this is a great work around, but in the long term we should fix the busy_spin_micros. So I suggest merging the PR and creating an issue to fix the busy_spin_micros latter. Feel free to assign it to me, I do have some ideas that I would love to try.

sounds good. let me do that. I was actually looking at busy spin micro yesterday... if we're okay with specifying the frequency in MHz (instead of Hz), we could actually go all the way down to uint32 (with a limit on the max value of usec) and remove the need to use a long division.
But let me merge this and assign that to you first.

MHz would not work as the clock for verilator is 500kHz. But something in this line would be the definitive solution I guess.
I quickly came up with a very pontual workaround for the busy_spin_micros. But feel free to ignore it.
  // Some tests on Verilator are sufficiently time-sensitive that generalized
  // slow division can cause tests to time out (e.g. uncessary watchdog bites.
  //
  // So, instead, 1M ~= 16384 * 61 = (1 << 14) * 61 = 999424, we can get rid of the 64 bytes division with the smallest error.
  uint64_t cycles = (((uint32_t)kClockFreqCpuHz * usec) >> 14) / 61;

In any case we need the issue opened to implement the definitive solution later.

MHz would not work as the clock for verilator is 500kHz. But something in this line would be the definitive solution I guess. I quickly came up with a very pontual workaround for the busy_spin_micros. But feel free to ignore it.
  // Some tests on Verilator are sufficiently time-sensitive that generalized
  // slow division can cause tests to time out (e.g. uncessary watchdog bites.
  //
  // So, instead, 1M ~= 16384 * 61 = (1 << 14) * 61 = 999424, we can get rid of the 64 bytes division with the smallest error.
  uint64_t cycles = (((uint32_t)kClockFreqCpuHz * usec) >> 14) / 61;

In any case we need the issue opened to implement the definitive solution later.

sounds good, i'll merge this and open a separate issue. thanks @engdoreis


rofi-wayland: add patch for niri

Hi,
This add a patch to fix rofi-wayland use with the niri window manager.
ref. https://github.com/davatorium/rofi/discussions/2008
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build rofi-wayland


Add started at

This prevents GitHub from reporting:
ran ... days ago in less than 5 seconds
when in fact GitHub is thinking "it ran in negative 1 minute".

See probot/probot#803


Changing the AI model results in DRAM compiling error

@OOHehir
We would like to test some models in the jungle, therefore i just tried to implement a new model (using this commit 8339ced2ec9f617414e5dc075a878fdef896f7a1  ) using esp32dev-ei-windows. But getting a compiling error. Haven't tried to change the model for a while so i am not sure since when this issue is happening. If the error doesn't point you in the right direction. I can try to find out which commit introduced that issue.
Archiving .pio\build\esp32dev-ei-windows\libe32\libaudio_input.a
Archiving .pio\build\esp32dev-ei-windows\lib882\libedge-impulse.a
Linking .pio\build\esp32dev-ei-windows\firmware.elf
<EMAIL_ADDRESS>.pio/build/esp32dev-ei-windows/firmware.elf section `.dram0.bss' will not fit in region `dram0_0_seg'
<EMAIL_ADDRESS>DRAM segment data does not fit.
<EMAIL_ADDRESS>DRAM segment data does not fit.
<EMAIL_ADDRESS>region `dram0_0_seg' overflowed by 12088 bytes
collect2.exe: error: ld returned 1 exit status
*** [.pio\build\esp32dev-ei-windows\firmware.elf] Error 1

Let me know if you need more infomration about it. The model i tried to implement is this one:
test_Trumpet_Trimmed_1.0.54.zip

@OOHehir
After doing a fresh github clone. The model import works. Not sure what actually caused it.
I am sorry if you have wasted time to investigated this issue.

@EDsteve No problem,  I didn't get a chance to look at it. I think different EI models might pull in different files, so perhaps could be the issue.

I closed this mistakenly. I still can't change the model.
@OOHehir if you can take a look. That would be great. So i can start testing models and hopefully deploy some in the jungle for testing soon.

@EDsteve Its difficult to examine the model from the files but I presume its quite complex? The compiler is complaining that there's too much data to store in the assigned region. I added this build flag for the AI build, you could comment it out & see if the code compiles. It's not an ideal fix as it will result in the memory getting more fragmented during runtime.

@EDsteve Additionally to this you could try uncommenting
#define EI_BUFFER_IN_PSRAM

in project_config.h to a allocate the bufer in PSRAM & see if that solves the problem.

@OOHehir Haven't had much luck with the build flag. But thanks for the new tip. I will try that hopefully tomorrow.
The models i am using are the same as i used before. Not complex at all i would say. But let's see if #define EI_BUFFER_IN_PSRAM solves that problem :)

@EDsteve Just tried this. It fails to build as you described but commenting out this line allows a build. Probably best to do a 'full clean' first to ensure there's no left over build artifacts.
Please keep in mind the caution above:
It's not an ideal fix as it will result in the memory getting more fragmented during runtime.
I'll close this, please re-open if it persists for you.


chore: use "bugfixes: true" in babel-preset-env

Using bugfixes: true as available in<EMAIL_ADDRESS>https://babeljs.io/blog/2020/03/16/7.9.0 will help to ensure that the smallest possible output when building with Babel in es-dev-server, Rollup, webpack, and more!

@LarsDenBakker these tests that don't pass after the update (https://app.circleci.com/pipelines/github/open-wc/open-wc/5418/workflows/af48d8a7-d752-497c-87de-c0c9832a1e8e/jobs/6951/parallel-runs/0/steps/0-104), I expect them to be different, that's sort of the win here, but is there a way I can know the output still serves the purpose of the test before I update the snapshots?

Yeah those need to be updated. Snapshots can be updated with yarn test:updates-snapshots, but there are also compatibility mode e2e tests which check if the babel compile did what it needed to do. They're a bit annoying to maintain, but valueable :)

As long as they're backed up in the e2es so that I don't have to parse them for "correctness" as much as make sure they're tuned to pass, then I'm fine with it. Got this change passing CI and am looking forward to doing some before and afters of the possible byte savings...


color code for chart at top needs to go

      plugins:{         legend: {           display: false       }     },
needs to be added to the right place. see slack

Chart has been murdered by Jeremy. Henceforth, said issue is now deemed resolved.


feat(scale_pattern): add fromChordPattern factory constructor

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
703

Relevant Lines:
703

💛  - Coveralls


Package changes for Adoptium Temurin binaries (add temurin8) 🧪

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Related to #28737
Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/pull/28744#issuecomment-927313703


[FEATURE] Localize datetime and return it in API response

Is your feature request related to a problem? Please describe.

Data is stored in the DB in the UTC time zone.
Currently, most APIs convert the data to JST and return it.
We want to localize the data to the time zone set in the environment variable and return it.

Describe the solution you'd like

Converts the datetime to the time zone set in the environment variable and returns it.

Allow arbitrary time zones to be set as with ibet-prime.


bicep build/bicep decompile service

In order to support both Portal template export and portal deployment with bicep files, we need to stand up APIs for sending bicep code and returning the generated template and vice-versa.
Related to #2369

For export as Bicep in the portal, we would need to decompile a single template into a single Bicep file, which should be fairly easy to model in an API. Having a single compile API is also easy to model.
We will need to decide whether we should support multiple files with this API. For example, would we need to support compiling a file referencing local modules? Would the API also need to handle any private registries once they're available?

Additionally to the API service it would be useful to have other tools to compile into arm. For go/python developers like us, msbuild or cli commands are not really feasible to process the bicep files. At the end of the day we need to send the json templates to Azure RM and we need a way to programmatically compile with go or python.

The referenced sample I had to compile a JSON template so users could easily deploy it. While not a big issue, Bicep templates seem a lot easier to read for the purposes of samples, which pair nicely with using the typical Deploy to Azure buttons.

Closing this as we have built a "decompile service" that will go live in January to support Portal-based export to Bicep. At the moment, there are not plans to add a build service, so I am not going to open a different one for tracking that.


Incorrect compilation of type synonyms

Bug reported by @liesnikov.
private variable @0 a : Set

Ap : (p : @0 a → Set) → @0 a → Set
Ap p x = p x
{-# COMPILE AGDA2HS Ap #-}

Expected output:
type Ap p = p

Actual output:
type Ap x = p

This seems to be happening here:
https://github.com/agda/agda2hs/blob/9388b77207e97b3934af9fb10d27459b30832078/src/Agda2Hs/Compile/TypeDefinition.hs#L31
The namedClausePats has both arguments, but the compileDom of the first argument returns DODropped, causing it to be ignored by the compileTypeArgs, whereas it should select the other variable to be DODrpped.


chore(security): Updating assert logic

SUMMARY
This PR updates the security manager assertion logic making it more flexible for deployments to configure their security manager. Currently there exists two types of methods:

can_access_*, e.g. can_access_datasource, which returns a boolean in response to whether the user can access said resource.
assert_*_permission, e.g. assert_datasource_permission, which returns None but raises an exception if the user cannot access said resource.

The issue with this approach is that some methods like access_datasource have all the logic and assert_datasource_permission calls can_access_datasource and raises an exception if the response is False, whereas others like can_access_table have no assert equivalent.
Additional context as to why a user cannot access said resource is lost when the logic resides in the can_access_* and thus the preferred approach is to have the logic in the assert_*_permission method and thus the can_access_* methods (which are mostly just convenience methods) can take the form,
def can_access_datasource(self, datasource) -> bool:
    try:
        self.assert_datasource_permission(datasource)
    except SupersetSecurityException:
        return False

    return True

Note I’ve only translated a few of the can_access_* methods. I think in the future it could make sense that more of these methods could leverage this pattern depending on how deployments with custom security managers would want to proceed.
Secondly the term assert seems strange as one would expect it to raise an AssertionError if the assertion fails, whereas these methods raise a SupersetSecurityException, hence I opted for the requests approach (which uses raise_for_status) to call these methods raise_for_access and thus it seems clearer than an exception may be raised (and thus should be handled if necessary). I used the term access rather than permission for consistency with the can_access_* methods. I also added a wrapper function to various classes so rather than,
query_context = QueryContext(**json.loads(request.form["query_context"]))
security_manager.assert_query_context_permission(query_context)

the logic is now,
query_context = QueryContext(**json.loads(request.form["query_context"]))
query_context.raise_for_access()

Finally rather than having a slew of raise_for_*_access which can add to the confusion (and introduce unnecessary complexity) if a deployment needs to override these, i.e., they intrinsically need to know how these interact, I defined a single raise_for_access method which contains all the logic regardless of the resource type (table, datasource, visualization, etc.). This can be helpful say if a user cannot access to an outright datasource but due to column level security can access the datasource in the context of a visualization (where the columns are defined).
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN
CI and added unit tests.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

@etr2460 SQL Lab uses the rejected_tables method which calls can_access_datasource which leverages this the raise_for_access method.

@etr2460 the can_access_datasource (renamed to can_access_table in https://github.com/apache/incubator-superset/pull/10031) is used by SQL Lab which leverages this the raise_for_access method.

@dpgaspar and @villebro I was wondering whether you had any updated thoughts regarding this approach.

@john-bodley this needs a rebase

@villebro I've rebased. Note in commit https://github.com/apache/incubator-superset/pull/10034/commits/3c5f3ef41977bf3720fb3e4359209cb6f13c122c I also extended the logic to replace the rejected_tables method. I think this provides clarity/simplicity in the views but possibly adds complexity to the raise_for_access method.

@villebro I agree that the raise_for_access method is fairly loaded (though in general I like the standardization it has provided in terms of checking for access from the perspective of a query, viz, etc.).
The mixin is an interesting idea, though I sense it mightn’t simply the logic in raise_for_status as one would need to do a number of isinstance checks to work out whether the DatasourceMixin is a query context, viz, etc to pull out the appropriate attributes. Also this doesn’t address the issue of the Table argument as there’s no mixin related to that.

@etr2460, @john-bodley
I'm aware of this change, will take a look today


[REQUEST] Task to Programatically Updating the Secure Configuration of a Plug-in Step

Would you consider adding a task that encapsulates the following?
http://www.tomasschulkes.com/index.php/programatically-updating-the-secure-configuration-of-a-plug-in-step-after-deploying-a-solution/
The goal would be to use it to update dev/test/prod with appropriate values through automation.

Any feedback on this one?  I have a sample console application that does what I would love to see embodied as a VSTS task from this project here:
https://github.com/devkeydet/UpdateSecureConfiguration

Do you take contributions?  I'd be happy to take a crack at porting my console application to a task.  I can fork, give it a shot based on your other tasks, and submit a pull request.  Thoughts?

Hi, the online management  tasks has been consuming most of my time dedicated to this framework recently. Contributions are always welcome. Let me know if you need any guidance.
Have you considered storing the configuration in a custom entity in CRM which you can secure using roles and deploy/manage using configuration migration/package deployer if needed. The plug-ins could retrieve this at runtime and cache if needed.

I have, but prefer to use secure config.  I'll look at your other tasks and try building this out and submit a pull request.  If it meets your bar, let me know and I can help with others.

Published in 8.2.1


Add a struct scope to the symbol files

It would be nice to have something similar as the +m flag we have for C++ files in exhuberant ctags. It adds to the tags the name of the owning class. The tag file could also adds the package scope to the symbol.

has been fixed with pull request #27


Support custom switch-case indentation for JavaScript

Feature request https://github.com/microsoft/TypeScript/issues/18682 and PR https://github.com/microsoft/vscode/pull/179670 added typescript.format.indentSwitchCase setting, so as it's possible to no-indent case in switch command. Unfortunately, it works for TypeScript only.
Please, add make it work for JavaScript, too. E.g. add some javascript.format.indentSwitchCase setting.
Note: See also #32925 which was originally created for JavaScript, but closed as completed, while in fact it's not completed, because the current implementation is for TypeScript only.

Hi, would I be able to try out this issue?

I want to work on this

i want to work on it


Exponent Calculator Extension:

Is your feature request related to a problem? Please describe.
It will helps us to find any power of a number (x)^y.
for examle: let x=2 and y=10 then (2)^10=1024
Describe the solution you'd like
I would like to add Exponent Calculator Extension:
I agree to follow this project's Code of Conduct
I'm a GSSoC'24 contributor
I want to work on this issue

I have added that issue but that was only calculator  so that wasn't assigned to me now i want add extension for that
Please look into it


Setting up Cognito infrastructure - to use Amplify or not to use

Create Cognito infra through Amplify or separately create and manage CF templates?
I understand Amplify provides a way to setup and deploy a User Pool through the CLI, but it seems to create a soft dependency between the app in which Amplify is being used for and the backend infra (the CF templates are created in the frontend app dir, changes must be made through it).
We have an immediate use case for Cognito User Pool authn, and while the use case is against a React app, the pool will be leveraged by other clients in the future, as well extended into Identity Pools for authz. I am concerned that taking advantage of Amplify comes at a cost (full disclosure I have barely touched its surface so not sure what the full cost is). However, if we don't create the infra through Amplify, it seems connecting an external Cognito User Pool isn't fully supported (workarounds mentioned here https://github.com/aws-amplify/amplify-cli/issues/779) - which we'll need to do no matter what as we have multiple apps that will want to use it.
Just looking for some suggestions for managing and deploying infra - seems there are tradeoffs either way.

Perhaps best practices are documented somewhere that I'm not able to find?

@NTonani It seems like you want to use the same AWS resources accross multiple frontend apps. We have documents this use case out here - https://aws-amplify.github.io/docs/cli-toolchain/quickstart#multiple-frontends

Yes that is a helpful guide once committing to using Amplify for managing infrastructure. Is that currently the preferred method, over managing it independently?
I'm mainly concerned with the coupling between the app and the backend infra it seems to force. Take, for example, managing infra for multiple apps (app1, app2, app3) within app1. Suppose app2 has a use case which requires extending the infra. Now, per best practices (backend definition residing in initial app - app1), and maintaining a single source of truth, app1 must be involved. Hence the coupling.
Managing independently would resolve this, however I have read that Amplify support becomes inconsistent when connecting pre-existing infra. Is there a doc on such tradeoffs - managing infra for multiple apps in Amplify vs not? If not, it would be beneficial. Thanks.

Ah - seems we can manage a separate repository for Amplify. At least it decouples the apps. Will move forward with this.

@NTonani Cool! Let us know if you are stuck with any other issues.

@kaustavghosh06 After creating an Amplify app in a new repo, multiple envs, and pushing infra, I tried connecting my first frontend and when I ran amplify pull I got the following:

No Amplify apps found. Please ensure your local profile matches the AWS account or region in which the Amplify app exists.
Failed to pull the backend.

This lead me to manually attach my repo to a new Amplify Console app, though it links it as the frontend and not backend. Even still, after reading Amplify Console now provides visibility into backend environments provisioned by the Amplify CLI I presumed I wouldn't have to do that at all! Article states:

When you initialize a project with the Amplify CLI, the Amplify Console creates an app project that provides a list of backend environments

Not sure if there has since been a rollback of the feature, but amplify console brings me back to the create page, indicating no app was setup when I initialized my repo.
How can I best setup an Amplify app with my backend repo? Thanks!

@NTonani Did you deploy the AWS resources with 4.6.0 as well?

@kaustavghosh06 Just checked that as well. I believe what happened is I updated when amplify pull failed, but earlier provisioned resources with <4.0...
Is there a way to retroactively create/assign an app? Or do I need to teardown?

I ended up deleting and starting over as I wasn't far in. I will note the quickstart#workflow guide will not work as mentioned if frontends are not sharing a directory unless Amplify version is >=4.0. Might be worth mentioning there.
Thanks for the help.


PrimitiveArrayDiffer/Field properties

new PrimitiveArrayDiffer
allow using Field properties of bean properties

Hallo Daniel,
ich wünsche dir ein frohes neues Jahr. Es fängt gut an denn ich habe jetzt Zeit, noch etwas in java-object-diff zu machen. Was möchtest du noch geändert haben?

Hi Wolfgang,
Happy New Year to you too. Great to hear you have some time for java-object-diff! I'll add some comments to the lines of code I have questions or change requests about.
P.S.: I'll stick to English, because these conversations are usually a very good documentation for others. But feel free to keep on writing in German if you like and I'll do my best to provide enough context for others to follow in my answers.

Regarding the PropertyAccessor: how about changing it to the following property resolution hierarchy:

public field, getter & setter

use getter for read access
use setter for write access
getAnnotations returns annotation from getter

public field, no getter, no setter

use field for read access
use field for write access
getAnnotations returns annotations from field

public field, no getter, setter present

use field for read access
use setter for write access
getAnnotations returns annotations from field

public field, getter, no setter

use getter for read access
use field for write access
getAnnotations returns annotation from getter

no public field, but getter & setter present

use getter for read access
use setter for write access
getAnnotations returns annotation from getter

This way public, non-static fields would automatically be treated as properties, yet it is possible to alter read and write access via getters and setters.
I wouldn't go as far as automatically including private fields though, since it would introduce a much higher risk of breaking client code. If you really need access to private and protected fields, it would be good to either make it opt-in via new configuration setting or just adding a second introspector.

I fear this is not a solution for me. I am comparing JPA entities and JPA annotations can be applied to the field OR the getter method(). It is up to the developer. Therefore I must check existence of the annotations on the field (if existing) AND on the read method.
Normally entity beans should follow Java beans conventions but there are indeed cases where an entity has a property without having a corresponding field. Some JPA providers use byte code enhancement to modify the entity classes. I observed this in integration tests of object comparing and merging. I took that into account in the provided code.
I am sorry I cannot provide you with test cases that fit into your environment as I have no knowledge of TestNG, Groovy nor Spock. I use junit and arquillian in a complex integration environment.
Von: Daniel Bechler<EMAIL_ADDRESS>Gesendet: Freitag, 2. Januar 2015 23:08
An: SQiShER/java-object-diff
Cc: Wolfgang-Winter
Betreff: Re: [java-object-diff] PrimitiveArrayDiffer/Field properties (#123)
Regarding the PropertyAccessor: how about changing it to the following property resolution hierarchy:
·         public field, getter & setter

use getter for read access
use setter for write access
getAnnotations returns annotation from getter

·         public field, no getter, no setter

use field for read access
use field for write access
getAnnotations returns annotations from field

·         public field, no getter, setter present

use field for read access
use setter for write access
getAnnotations returns annotations from field

·         public field, getter, no setter

use getter for read access
use field for write access
getAnnotations returns annotation from getter

·         no public field, but getter & setter present

use getter for read access
use setter for write access
getAnnotations returns annotation from getter

This way public, non-static fields would automatically be treated as properties, yet it is possible to alter read and write access via getters and setters.
I wouldn't go as far as automatically including private fields though, since it would introduce a much higher risk of breaking client code. If you really need access to private and protected fields, it would be good to either make it opt-in via new configuration setting or just adding a second introspector.
—
Reply to this email directly or view it on GitHub https://github.com/SQiShER/java-object-diff/pull/123#issuecomment-68567139 .  https://github.com/notifications/beacon/AJovAG_1MIVNFTDdlUNPEPOcqcxLVGsTks5ndw4ngaJpZM4DFgj0.gif


Added support for center crop and resizing

I've added some support for performing central crop with a specific fraction, since this is a common preprocessing technique used in ImageNet classification (2012) validation preprocessing. I have tested the performance with Inception V3 pretrained and with the preprocessing transformation seen below and some arithmetic operations, I can get a validation accuracy of up till 77.61% for all 50k images, which is close to the original reported 78.0% accuracy from Google. This should be close enough for the sake of reproducibility.
transform = transforms.Compose([
    transforms.CenterResizedCrop(299),
    transforms.ToTensor(),
])

On top of the transformation above, I also needed to perform something as simple as (input - 0.5)  * 2.0 in my data loading, in order to match the preprocessing operation used by google as seen here:
https://github.com/tensorflow/models/blob/master/research/slim/preprocessing/inception_preprocessing.py#L243-L281
This got the val accuracy up by another 0.3%. I believe the remaining difference in val accuracy could be attributed to training differences between the 2 frameworks.

Regarding changes
To prevent breaking programs using existing function, I created the new resized_center_crop function instead of editing the RandomResizedCrop (which can technically achieve the same thing, although you would have to ignore many paramters and set the scale to a tight bound). I think this will be a nice addition to researchers working on the ImageNet dataset esp. when they want to replicate the performance seen in TensorFlow.
Please check if there are any issues with this pull request -- I'll figure a way to fix any issues as soon as I can.

@kwotsin : First of all thanks a lot for your efforts. I would like to use your changes as well.
based on your latest changes, what do I need to change ?
Can you share your training/validation transformations as well ?


document is not defined at Carousel:26

I tried to just copy-paste the example noted here, the "With Captions" one, but I get this instead of the Carousel:
document is not defined

ReferenceError: document is not defined
    at [...]/node_modules/sveltestrap/src/Carousel.svelte:26:12

Which points to this:
  [...]
  onMount(() => {
    setRideTimeout();
  });

  _removeVisibilityChangeListener = browserEvent(
    document,     <------  this line!
    'visibilitychange',
    () => {
      if (document.visibilityState === 'hidden') {
        clearRideTimeout();
      } else {
        setRideTimeout();
      }
    }
  );
  [...]

According to this StackOverflow answer, the problem appears to be connected to the fact that document is only available after the component has been mounted.
I though, as a very noob on Svelte, to just chuck the assignment to _removeVisibilityChangeListener inside the onMount callback, and appears to be working just fine.
  onMount(() => {
    setRideTimeout();

    _removeVisibilityChangeListener = browserEvent(
      document,
      "visibilitychange",
      () => {
        if (document.visibilityState === "hidden") {
          clearRideTimeout();
        } else {
          setRideTimeout();
        }
      }
    );
  });

Node version: v12.19.0
Svelte: v3.17.3
Sveltestrap: v3.9.4
I will soon create a PR for this, hope this helps out anybody!

Hi, are you. doing this in Sapper or server side rendering by chance?

Yep, I have updated the issue with my current Sapper version: v0.28.0

Thanks, and just to confirm, are you importing like mentioned here:
https://github.com/bestguy/sveltestrap#note-on-server-side-rendering-ssr-usage-fixed
and is sveltestrap in your dev dependencies?
(Just checking as there are always weird issues with Sapper SSR)

Currently, my devDependencies are as follows:
  "devDependencies": {
    "@babel/core": "^7.0.0",
    "@babel/plugin-syntax-dynamic-import": "^7.0.0",
    "@babel/plugin-transform-runtime": "^7.0.0",
    "@babel/preset-env": "^7.0.0",
    "@babel/runtime": "^7.0.0",
    "@rollup/plugin-babel": "^5.0.0",
    "@rollup/plugin-commonjs": "^14.0.0",
    "@rollup/plugin-node-resolve": "^8.0.0",
    "@rollup/plugin-replace": "^2.2.0",
    "@rollup/plugin-url": "^5.0.0",
    "fa-svelte": "^3.1.0",
    "rollup": "^2.3.4",
    "rollup-plugin-svelte": "^6.0.0",
    "rollup-plugin-terser": "^7.0.0",
    "sapper": "^0.28.0",
    "svelte": "^3.17.3",
    "sveltestrap": "^3.9.4"
  }

And my copy-paste component is this:

<Carousel {items} bind:activeIndex ride={true} interval={2000}>
  <CarouselIndicators bind:activeIndex {items} />

    {#each items as item, index}
      <CarouselItem bind:activeIndex itemIndex={index}>

        <CarouselCaption
          captionHeader={item.title}
          captionText={item.subTitle} />
      </CarouselItem>
    {/each}

  <CarouselControl direction="prev" bind:activeIndex {items} />
  <CarouselControl direction="next" bind:activeIndex {items} />
</Carousel>

Hi @IAL32 this is released in<EMAIL_ADDRESS>Please let me know if looks good and welcome to close this.

Will close, should be fixed in 3.9.5.  Please reopen if still seeing issues


Codeunit 2000001 CheckPaymJnlLine "CheckBeneficiaryBankForSEPA"

can following change be made to codeunit 2000001 CheckPaymJnlLine function "CheckBeneficiaryBankForSEPA"
WITH PmtJnlLine DO BEGIN
CheckIBANForSEPA(PmtJnlLine,EuroSEPA);
IF IfForeignBank(PmtJnlLine."Bank Account") THEN //new code
ErrorEmptyFieldInLine("Line No.","SWIFT Code",FIELDCAPTION("SWIFT Code"));
IF NOT ErrorEmptyFieldInLine("Line No.","Bank Country/Region Code",FIELDCAPTION("Bank Country/Region Code")) THEN
CheckSEPAAllowed(EuroSEPA,"Line No.","Bank Country/Region Code");
END;
new function:
IfForeignBank(codBank; code[20])
GetBankAccount(codBank);
CompanyInformation.GET;
IF BankAcc."Country/Region Code" <> CompanyInformation."Country/Region Code" THEN
EXIT(TRUE);
this because the SWIFT code is no longer mandatory for belgian bank transfer and therefor is not always present.
or event in place of this code is also a possibility, so we can skip the error.

Thanks for submitting this issue. We will publish a fix for it in the next minor update for release 15.

Thanks for submitting this issue. We will publish a fix for it in the next minor update for release 15.


Error when creating vwan hub

This is autogenerated. Please review and update as needed.
Describe the bug
I get an error when trying to create a vwan hub.
Command Name
``
network vhub create ....
Errors:
The command failed with an unexpected error. Here is the traceback:
progress_patch() got an unexpected keyword argument 'det'
Traceback (most recent call last):
  File "C:\Users\deckel\.azure\cliextensions\interactive\azext_interactive\azclishell\app.py", line 663, in cli_execute
    result = invocation.execute(args)
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 657, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 702, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 878, in __init__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/progress.py", line 179, in __init__
TypeError: progress_patch() got an unexpected keyword argument 'det'
To open an issue, please run: 'az feedback'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
``

Expected Behavior
vwan hub creation.
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.24.0 *

Extensions:
azure-firewall 0.10.0
import-export 0.1.1
interactive 0.4.4
ssh 0.1.5
virtual-wan 0.2.6

Additional Context

network

Can you use az network vhub create in terminal directly instead of in az interactive. There maybe some bug in interactive.
You can use az network vhub create -h to get help messages.

Duplicate of #17740


fix default sigma/tau, close #74

See https://nbviewer.org/github/TomographicImaging/CIL-Demos/blob/fix_default_sigma_tau_values/training/2021_Fully3D/Week2/03_PDHG.ipynb
nbstripout before accepting merge.

If you are ok with the changes, yes. and merge in master

However, I am happy with the fix to the number of angles


crashes my game for some reason

hi for some reason this exact mod is crashing my game and giving me an error code 1. thanks

I'm having the same issue. Crash log attached.
latest.log


Add slots to class GameTreeReport

Low priority but this is syntactically safer. Right now I have a commented block of init just to memorialize what the attributes are. However, then I'd need to use the attributes as instance attributes rather than class attributes.

Closed as out of scope for this repository.


No way to "reverse" parse from SQL_DML_Command back to SQL (text)

This will be needed on rare occasions, at least in my project it is.  I am currently working on a solution.

So far, I am thinking a "reverse parse" grammar will be needed in order to do this the "right" way.  What are your thoughts?

For further context, here is my particular use-case:

first, I have developed an API that "enforces" referential integrity - i.e. I have documents that refer to other document id's as a foreign key

and when a user attempts to delete a document with a dependent "foreign key" that still exists, I hook my api call to delete the PK document

if there are any documents that depend on the document to be deleted, an exception is thrown indicating that the PK document has dependencies and cannot be deleted

to search for dependent documents, I need to parse the delete SQL and convert it to select SQL to first find the document ids that would be selected by the delete SQL

then I query the dependent collections for any occurrences of the primary document id as a FK

if any exist, the exception (as previously) described is thrown

this depends on:

parsing the delete SQL and using it's "from" and "where" clause to craft/convert to a select SQL statement, specifying the docid as column in the select SQL statement
I have built a preliminary treed traversal algorithm (in-order, typical) but it is getting messy
I would like to utilize the parse-tree semantic currently in-place if possible

Your thoughts?

I've found an alternate method to accomplish this.

After successfully implementing this in my API, I wonder if this feature would be desirable in PyFireSQL itself - that is, would enforcing Referential Integrity restraints be a desirable feature in PyFireSQL?  If so, we ought to have a more thorough design discussion.  Let me know what you think.


Run Unit Tests in Node

https://github.com/jvilk/BrowserFS/issues/192
By @jvilk

Unit tests already run in Node with Jest, albeit they don't work (that is a separate issue though)


DotNetCoreTest fails to use XUnit2Settings on Cake 0.27

What You Are Seeing?
I'm using the DotNetCoreTest command using both DotNetCoreTestSettings as well as Xunit2Settings:
DotNetCoreTest(dotnetTestSettings, project, xunitSettings);

which results in the following exception:
Error: One or more errors occurred.
        Method not found: 'Cake.Common.Tools.XUnit.ParallelismOption Cake.Common.Tools.XUnit.XUnit2Settings.get_Parallelism()'.

What is Expected?
Expected is that the tests will be executed.
What version of Cake are you using?
The error occurs in 0.27.0 and 0.27.1.
The error does not occur in 0.26.1.
I think the regression has been introduced by the change of ParallelismOption to a nullable.
See https://github.com/cake-build/cake/pull/2040/files#diff-133ec58ad83fd466eed3b129d32daefaL84
Are you running on a 32 or 64 bit system?
64 bit
What environment are you running on?  Windows? Linux? Mac?
Windows
Are you running on a CI Server?  If so, which one?
Yes, Jenkins.
How Did You Get This To Happen? (Steps to Reproduce)
See "What You Are Seeing?".

@erik-am It would help to have more of the stack trace and/or verbose output, but it appears that the exception is coming out of the Cake.Incubator assembly at the method below, right?
void DotNetCoreTest(this ICakeContext context, FilePath project, XUnit2Settings xunitSettings)

Thank you @kcamp, indeed it seems the issue is in Cake.Incubator. So, Cake 0.27.0 did include a breaking change in contrast to the release notes,
For now, I changed my usage of DotNetCoreTest, so that I'm using the method provided by Cake itself instead of the extension. Thanks for pointing me in the right direction! 👍

@wwwlicious are you in a position to take a look at this?
@erik-am unfortunately, this does look like it was an unintended breaking change in Cake 0.27.0 😢
@cake-build/cake-team FYI...

@gep13 I'll have a look..


Update asset-structure-guidelines.md

Added some terse guidance on prim, attribute, and relationship naming conventions.

This is great @jcowles! Do you think the example prim paths should have slashes?
Maybe an extra bullet-point could be name-spaced primvars/attributes for DCC or studio-specific applications?

Done!


Why do we need NAT gateways in module 2

Hi,
Can someone please tell me why the containers hosted in private subnet need to talk to internet? It is hosted behind NLB and the internet traffic comes via nlb only. Can i exclude NAT gateways from module 2?

Good question!   I understand that internet access is requiredto allow updates to be downloaded, but I wonder if this functionality is required for this lab.

The CDK variant of the lab makes it much easier to tear down the environment and recreate it, thus helping to minimize unexpected charges.

Container instances need access to communicate with the Amazon ECS service endpoint. This can be through an interface VPC endpoint or through container instances having public IP addresses.
In our case we do not have interface VPC endpoint configured and the container instances do not have public IP addresses, then we must use network address translation (NAT) to provide this access.
Same logic goes with with Fargate also,
Fargate(without internet access in private subnet) can’t make a network request to Amazon ECR to download the container image, or communicate with Amazon CloudWatch to store container metrics. To run a Fargate task in a private subnet, we must add this NAT gateway
I experimented this by launching fargate cluster and task without a NAT gateway. The task was never moved to running state as it was not able to download the container image from ECR because there is no internet access in private subnet. I then launched a NAT gateway and added it to private routing table and then immediately my task status moved to running state and private IP of my container is successfully registered to my target group of ALB.
I then deleted the NAT gateway immediately as i don't need it afterwards. :)
https://docs.aws.amazon.com/AmazonECS/latest/developerguide/launch_container_instance.html
https://aws.amazon.com/blogs/compute/task-networking-in-aws-fargate/

It's possible to set up Fargate with public subnets only, and application load balancer. I had to change the python application to expose port 80.
In this setup, you actually don't need the API Gateway (or else you have to modify its integrations).


Implemented feature: 'go to main activity'

Suggested here #1423
Description
Added a toolbar button and an action under navigation menu, that opens main activity class in a code tab

@Mino260806 thank you for great work :+1:
I made a small changes to search main activity class by original name (not renamed alias) and also handle exceptions with additional info about failed step.
About left todos: looks like it is better to make a new class (in jadx-core) to handle all manifest parsing and output ApplicationParams as result, so it will be possible to use it both in export and main activity open. But this needs a lot of refactoring, so new PR is preferred.

looks like it is better to make a new class (in jadx-core)

Sure. In which package do you suggest to create the new class ?

Sure. In which package do you suggest to create the new class ?

jadx.core.utils.android will be fine, it already has various utility classes for work with android resources


GB zapped web bug

Strange issue: string "GB zapped web bug" appended to rss item text. This string is not present in the original RSS file when I curl'd it:

Feed source: http://feeds.feedburner.com/i-ljosi-sogunnar

Hmm that's weird. I'm not able to reproduce. Would it be possible for you to delete fhe feed, re-add it to Vienna, and see if the problem is still occurring?

Deleted feed, re-added and same thing occurs.

When I look at the source in Vienna via Show XML Source, the feed XML has a comment "" appended to the description tag.

OK, which version of Mac OS and Vienna are you using?

This old bug report suggests it's due to the use of glimmer blocker: https://www.cocoaforge.com/viewtopic.php?f=18&t=23854
are you using glimmer blocker?

Yes, I am using GlimmerBlocker. Disabled it and the problem disappeared.


⚠ Bofrost.be has degraded performance

In 9cc1c26, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2858 ms

Resolved: Bofrost.be performance has improved in b98fa63.


Use --url instead of --host/--port for login and product command-line too

Merge #857 first!

I fixed the merge conflict in the documentation.


How to hide senstive information

Can Allure Report hide sensitive information, such as URLs, usernames, passwords, because this report is accessed publicly, I don't want other people to know this information?

Report will not be accessed publicly unless you serve it on a public web server.
Anyways, plugin doesn't know what is sensitive information what is not, but it has setting to disable logging for specific commands - allureSkipCommands
you can read about it in documentation - Environment Variables

OK, thank you for the info sir


Establish structure for a dedicated neovim plugin: neovim

#62 as well as the complex setup process shows the value of a dedicated neovim
How does this relate to #40 and #65: this plugin could be a dependency of Obsidian.nvim, which would call the plugin's setup function.
This would also enable #71 (though the plugin would become not so minimal)

I am not thinking about this very much ATM! Maybe in the future. For now, closing.


COOK-123 Add amazon to rhel platform_family conditional

This is needed for Chef 13 compatibility on Amazon Linux.

LGTM 👍


trying to click "install from URL"

Describe the issue in detail here. Attach any relevant screenshots or files. The section below was automatically generated. Remove any information that you do not wish to include in the report.

Debug Data

Property
Value

Name
"Wolfram/Chatbook"

Version
"0.0.29"

EvaluationEnvironment
"Session"

FrontEndVersion
"Desktop: 13.2 for Microsoft Windows (64-bit) (January 31, 2023)"

KernelVersion
"13.2.1 for Microsoft Windows (64-bit) (January 27, 2023)"

SystemID
"Windows-x86-64"

Notebooks
True

DynamicEvaluation
True

SynchronousEvaluation
False

TaskEvaluation
False

Settings

Property
Value

Stack Data
throwFailure
trimURL

Failure Data
<|
  "Evaluation" :> Wolfram`Chatbook`Actions`Private`PersonaURLInstall[$Failed]
  "Arguments" -> {}
|>

This is a direct result of issue #187.


demumble swift symbols as well

Fixes #32.
A bit of a mess since swift's demangler is much less hermetic than LLVM's.

…looks like nobody ever tried building swift with gcc??


checkDBStatus.sh does not work

I build image from:
docker-images/OracleDatabase/SingleInstance/dockerfiles/<IP_ADDRESS>
When I create DB my oratab is empty.
from checkDBStatus.sh:
... ORACLE_SID="grep $ORACLE_HOME /etc/oratab | cut -d: -f1" ...
It is not good to use data that is not in the volume.

Also checkDBStatus.sh work only for plug-able db.


Allow top-level return in script parsing. Fixes issue 8509 in master,…

… like PR 8510 fixed it for 'next'

Please accept CLI

Thanks


Some basic updates

There's plenty more here to work on. As always, comments/criticism welcome :)

@bjd2385 I suggest pull latest changes from type-hinting branch.

@lk-geimfari Will do. As you can probably tell, I'm still learning about version control and GitHub's workings. Thanks for being so patient with me :)

@bjd2385 Thank you for the contributing!

@bjd2385 Ping.


Pinned.fromSize and onTap not working

Hi, I have this Pinned.fromSize:
Pinned.fromSize(
  bounds: Rect.fromLTWH(29.0, 11.0, 42.0, 23.0),
  size: Size(100.0, 44.0),
  fixedWidth: true,
  fixedHeight: true,
  child:
      // Adobe XD layer: 'Sign in' (text)
      Text(
    'Login',
    style: TextStyle(
      fontFamily: 'Poppins-Medium',
      fontSize: 16,
      color: const Color(0xffffffff),
    ),
    textAlign: TextAlign.center,
  ),
),

I need to implement onTap on Login Text. I tried with InkWell and GestureDetector (with Text as child) without success.
How can I handle this?

did you fix this?
I'm having the same error

We use GestureDetector in PageLink and for onTap params, and it works as expected (at least here). Any chance you could provide more info?

Closing due to lack of detail. Please feel free to reopen with more information.


Fix layout for e.g. properties/P000053

@StevenClontz - for your reference, https://github.com/pi-base now has an "editor" repo which is the (very early stage) Electron desktop editor and a "core" repo which is an npm package that will contain the shared logic between the viewer and the editor (so all the formula and deduction stuff).