glsl: matrix qualifier on shader attributes

This slang code currently miscompiles:
[shader("vertex")]
float4 main(float3 position, float4x4 mvp) : SV_Position {
    return mul(mvp, float4(position, 1.0)));
}

It compiles to something like this GLSL code:
#version 450
layout(location = 0)
in vec3 _S1;

layout(row_major) // glslang produces: error: 'layout' : matrix or packing qualifiers can only be used on a uniform or buffer
layout(location = 1)
in mat4x4 _S2;

void main()
{
    gl_Position = (((((vec4(_S1, 1.0)) * (_S2))));
    return;
}

Reading matrices from an input stream is very rare. Do you intend this, or do you mean uniform float4x4 mvp?

GLSL does not allow matrices as vertex stream elements IIRC.

The fix here is to issue an error message that forbids defining a non-uniform matrix typed parameter.

Yes I did intend to do that and it's useful with per-instance vertex input
attributes. It worked just fine with glsl, I'm pretty sure it's legal.
On Wed, Jan 17, 2024, 02:13 Yong He @.***> wrote:

The fix here is to issue an error message that forbids defining a
non-uniform matrix typed parameter.
—
Reply to this email directly, view it on GitHub
https://github.com/shader-slang/slang/issues/3447#issuecomment-1894774931,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACUNUA7ZSIC4VFYDVDBYOTYO4QTRAVCNFSM6AAAAABBZKPIEWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJUG43TIOJTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

The issue here is that the code is being compiled with "column-major" layout, which maps to glsl's "row-major". Can you share you compilation options? Compiling with default row-major layout should get rid of the decoration.
You can also try -emit-spirv-directly and see if it fixes the problem.

My args are -I src/shaders -g3 -line-directive-mode none -entry main -target glsl -force-glsl-scalar-layout -o out.glsl -- shader.slang
I'm not passing the -matrix-layout-row-major option. Are you saying that if I did pass it, there would be no layout(row_major) or layout(column_major) annotations on these GLSL attributes? I may be approaching it incorrectly in that case, since I translated my GLSL's matrix * vector snippets into mul(matrix, vector) in Slang.

You need to either pass -matrix-layout-row-major or specify row_major modifier on the parameter.

Closing since the matrix layout qualifier should work here. If not, please reopen.


N64 switch gameplay measures as 1fps

I wanted to measure the fps of Majora's mask played with the switch's N64 emulator but gameplay is measured at a constanct 1fps for some reason, cutscenes are measured at the correct 20fps which I think is because of the noise filter they put over the cutscenes but actual gameplay measurements are unusable.
The video is a lossless HEVC video recorded using OBS' StreamFX plugin and NVENC encoder directly from my AverMedia Live Gamer 4K.
Video
I was able to measure it just fine with a pretty jank python method, using OpenCV to basically create a difference map and counting each frame that has any pixels that aren't pure black.
No idea if this method is useful to you but I thought I'd share either way.
Code

Added link to the video! fair warning though it's 40GB

It works on my machine at 0 pixel difference. Did you ignore tearing analysis and set the dismissed tear percentage to 100%?


Tag entity exclusion

This pr adds a new way to exclude entities from raycasting using the Tag component.

excluded tag names were added as a field to the scene configuration component.
added a way to dynamically handle entity exclusion from raycasting based on their tags.

To exclude an entity from raycasting tag it using the tag component and include the tag name in the scene configuration componet.

Good job 👍
When testing I got the following bug:

I added cube with tag that is excluded in scene configuration.
I ran the simulation and the cube was ignored by RGL 👍
I stopped the simulation and removed tag from the cube (Tag component is still present)
I ran the simulation again and the cube was ignored by RGL even though the tag was removed.

Please investigate my test case. Also, I left some comments to address.

@msz-rai
This must be a bug in the engine. I read the entity's tag list on demand. I've checked this and the tag component lists the excluded tag after edition in the editor.


ScrollView Inside a KeyboardAvoidingView straight up doesn't work

🐛 Bug Report
Basically the title. No events are fired either (onScroll etc)
To Reproduce
Put a scroll view inside a keyboardavoiding view and run on Android. Works fine on iOS!
Expected Behavior
Should work
Code Example
https://snack.expo.io/@bobber205/quiet-kiwi
Environment
React Native Environment Info:
System:
OS: macOS High Sierra 10.13.6
CPU: (4) x64 Intel(R) Core(TM) i5-5287U CPU @ 2.90GHz
Memory: 36.16 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 11.6.0 - /usr/local/bin/node
Yarn: 1.13.0 - /usr/local/bin/yarn
npm: 6.5.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
Android SDK:
API Levels: 26, 27, 28
Build Tools: 26.0.2, 27.0.3, 28.0.3
System Images: android-27 | Google APIs Intel x86 Atom, android-27 | Google Play Intel x86 Atom, android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.3 AI-182.5<IP_ADDRESS>14842
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.6.3 => 16.6.3
react-native: 0.58.6 => 0.58.6

Update:
Using the enabled properly to selectively turn KeyboardAvoidingView "off and on"  doesn't fix it.
KeyboardAvoidingView seems to be capturing some set of events that ScrollView needs

Unable to repro with minimal example: https://snack.expo.io/@jkcooper/rn23826-scrollview-inside-keyboardavoidingview
Im not sure whats going on in yours, I believe its something to do with flex and fixed heights, but couldn't figure it out. If you replace the KeyboardAvoidingView with a plain view you still cannot scroll, so in any case the issue is not with KeyboarAvoidingView and ScrollView.


Fixed port conflict of json_run_localhost

This makes the json_run_localhost assign ports to workers more robustly to avoid port conflict with a small change.
Json_run_localhost works like this;

json_run_localhost

spawning QpsWorker1 as a server (bind driver_port and server_port)
spawning QpsWorker2 as a client (bind driver_port) which connects to the server worker via server_port.
spawning the Driver which connects to QpsWorkers via driver_port

Previously the driver_port of QpsWorkers were chosen by json_run_localhost but the server_port of Server Workers were picked pu by itself so chances are that they're using the same value for  driver_port and server_port.
This change makes json_run_localhost pick up the all ports in advance although some QpsWorkers don't use server_port because they will be clients. At lease, this guarantees that all ports used here will be unique. (This doesn't prevent port conflict from multiple json_run_localhosts runnin on the same space)
In addition to this, logs printing out the port are added to trace this problem easily in future and the driver port is changed not to be reused with others.
This will supersede #23178.

@vjpai Main point is that two different processes (json_run_localhost and one of QpsWorker) assigned the port independently, thus it sometimes assigned the same port to two different entities because they maintained available port set locally. (instead of using global port-server, etc) That's why this has been flaky for a while.
This fix, rather, changed this behavior by making only json_run_localhost assign the port, resulting in unique ports assigned to all entities although json_run_localhost still manages ports locally.

Ok, so is there a race condition between picking the port and actually using it? That is, both processes pick the port and then only one is able to use it? I would argue that that points to a problem in the port server design; otherwise this problem could end up being much more widespread since we always have lots of processes running at the same time on the same machines and these could flake.
IOW, I would suggest that the port server needs to mark a port used immediately after it returns a port or something like that; I'm surprised that that isn't the case.

The reason why this test hasn't use port_server is because bazel RBE tests are configured to use local port function. (cs) This local port function is simple so it cannot prevent two different processes from picking up the same port. This behavior was introduced by #13442.
Probably having a port_server for bazel rbe would be good-to-have to avoid this kind of problem (not sure it's feasible). Once port_server becomes able to run along with the tests under bazel rbe, this workaround can be removed.


Setting component state onClick causes the tree to lose focus position during keyboard navigation and go to top of the rendered tree

This is during keyboard navigation: in my use case, I need to keep track of the "selected" node in state so I can apply specific styles to it - when a node is clicked setState is called.
In this simplified example, I have a state variable called 'test' and am calling setTest in the onClick of the node. If you press 'tab' down the nodes and then press 'enter' and 'tab' again, you'll see that the focus is set to the top of the tree instead of staying in place.
Example: https://codesandbox.io/s/react-vtree-forked-n8xl7l?file=/src/App.tsx
Using react-dev-tools I see that each Row has a unique key set as expected, so I'm not sure what the issue is, if anyone has any ideas please let me know!

I think the problem is that the Node function is defined inside the TreePresenter component, which is re-rendered whenevere setTest is called.
The solution should be:

move Node definition outside
pass the state {test, setTest} through the itemData prop of the tree and read it from the treeData prop of the node

Working example: https://codesandbox.io/s/react-vtree-forked-b6ed5c?file=/src/App.tsx

ah that makes sense, thanks!


Editable object should not support the struct.

Here's the reason:
Here's the code for change the property in the editable object.
public class PropertyChangeAction<TProperty>
{
    private readonly IEditableObject obj;
    private readonly string propertyName;
    private readonly TProperty? oldValue;
    private readonly TProperty? newValue;

    public PropertyChangeAction(IEditableObject editableObject, string propertyName, TProperty? value)
    {
        obj = editableObject;
        this.propertyName = propertyName;

        // should backup the current value.
        var propertyInfo = getPropertyInfo(obj, propertyName);
        oldValue = (TProperty)propertyInfo.GetValue(obj);
        newValue = value;

        // perform the action after method call.
        Redo();
    }

    public void Undo()
    {
        var propertyInfo = getPropertyInfo(obj, propertyName);
        Undo(obj, propertyInfo, oldValue);
    }

    public void Redo()
    {
        var propertyInfo = getPropertyInfo(obj, propertyName);
        Redo(obj, propertyInfo, newValue);
    }

    private static PropertyInfo getPropertyInfo(IEditableObject obj, string propertyName)
        => obj.GetType().GetProperties().Single(x => x.Name == propertyName);

    protected virtual void Undo(IEditableObject editableObject, PropertyInfo propertyInfo, TProperty? property)
    {
        propertyInfo.SetValue(editableObject, property);
    }

    protected virtual void Redo(IEditableObject editableObject, PropertyInfo propertyInfo, TProperty? property)
    {
        propertyInfo.SetValue(editableObject, property);
    }
}

Here's the code using the change:
[Test]
public void TestUndoRedoPropertyInStruct()
{
    var obj = new EditableObjectStruct
    {
        TestProperty = "test",
    };
    var action = new PropertyChangeAction<string>(obj, nameof(EditableObjectStruct.TestProperty), "test2");

    Assert.AreEqual("test2", obj.TestProperty);
    action.Undo();
    Assert.AreEqual("test", obj.TestProperty);
}

And we can see that the EditableObjectStruct in the PropertyChangeAction has been copied.

Guess we should not support the structure.
Maybe we can revert some changes in the #3:

Remove that stupid guid property in the interface.
change in interface into the property because property is more eye-friendly.


[experimental] Fix the release folders of experimental

We got experimental/DPN and experimental/experimental accidentally pushed to main.
Most probably the cargo run command was executed from experimental dir (it's supposed to be run from the diem-framework dir)

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

/land


TypeConverters within a Library Module

DBFlow Version:4.0.2
Issue Kind (Bug, Question, Feature): Question/Feature
Description: Is it possible to have TypeConverters within a library module? In the section about library modules within your (very thorough) documentation it doesn't seem like that topic is covered. I have a couple of TypeConverters that I use across multiple projects that I would like to include within my library module instead of having a copy of them in each of the individual projects.
Thanks!

not currently an easy way, but good idea. I can add a typeConverters field to the @Database annnotation to allow you specify which ones are included. or I might allow you to define them with annotations and then just load the empty database holder into the FlowManager via. FlowManager.loadDatabaseHolder().


Developer Guide

Any plan to develop documentation for developers who want to build new models on top of HyperGAN APIs?

Yes! We've made strides on this in 1.0 and have a gitbook released. There's always more to do with documentation but we're excited to make progress with this in the upcoming release.


UI material border radius

Objective
I wrote a box shadow UI material naively thinking I could use the border widths attribute to hold the border radius but it
doesn't work as the border widths are automatically set in the extraction function. Need to send border radius to the shader seperately for it to be viable.
Solution
Add a border_radius vertex attribute to the ui material.
This PR also removes the normalization of border widths for custom UI materials. The regular UI shader doesn't do this so it's a bit confusing and means you can't use the logic from ui.wgsl in your custom UI materials.
Testing / Showcase
Made a change to the ui_material example to display border radius:
cargo run --example ui_material

It's just to show that the correct border radius values are passed to the shader, so it doesn't do anything fancy like rounded corners.

I don't know if it's relevant but I don't see the same, the corner squares are smaller:

Oh yeah I changed this from the screenshot in the PR description. It uses a corner radius smaller then the border width to make sure it's using the correct values for both.


Support D1E-D2E Conversion Kit

We could create fixed monster information for the conversion kit version of all monsters.  This can be lower priority some H&M packs monsters will be used if the conversion kit is also available.
I am not planning to spend time on this, but open to someone keen.

Heroes are done.


What is the gain from using Rollup loader?

Hi there,
Looking through Rollup and Webpack 2 it seems that Webpack 2 already includes Tree Shaking by default: http://www.2ality.com/2015/12/webpack-tree-shaking.html and https://moduscreate.com/webpack-2-tree-shaking-configuration/
What is the gain from using Rollup?
Cheers

https://github.com/egoist/rollup-loader/issues/1 discussions here might be helpful


Implement IP Display UI Component

Post pics!

@nepoche  do you think we can wait for the Relayer integration and then update this pr to use it, or we can do it later.
Looks good so far, for the white theme can you reduce the shadow to be less than the Mixer component?


Destination Reached, but is not indicated on Remote Clients

When the Destination is Reached through Map Graph Detection, the effect to set the Destination Reached is not passed to the Remote Clients.

The ReachedDestinationEffect had the Actor as the Operating Round, when it should be the Share Company so it can be applied correctly.


Bash Interface to automatise the tasks in Xschem NGSpice

Hello,
I am using Xschem with NGSpice at the backend through the Docker station provided by Efabless.com for circuit simulation. I have 100 different trials of the input signal, each saved in a separate text file (.txt). Manually loading timestamps and voltage values for each trial in the piecewise linear function, and modifying the output file path (as it's different for every trial) is time-consuming for 100 trials.
I'm looking to automate this task of loading different input files, simulating, and saving as different output files through a bash script. Has anyone implemented a bash interface to automatically load input data and save it as different output files through a loop? If so, is it possible to implement it inside the Docker container? If not, I'd be happy to contribute in this direction.
Thanks and Regards,
Shavika Rastogi
PhD Candidate
International Centre for Neuromorphic Systems
Western Sydney University, Australia
and
Biocomputation Research Group
University of Hertfordshire, UK

This is already possible with our container, but requires a little bit of manual setup. First, a good starting-point to "generate" a starting command is to use the "DRY_RUN" option of our start scripts. For example you can run DRY_RUN=1 ./start_shell.sh, and the shellscript prints out the commands it would actually use to start the container, in my case this looks like:
docker run -it --name iic-osic-tools_shell_uid_1000 --user 0:0 -e DISPLAY= -v /home/georg/eda/designs:/foss/designs:rw hpretl/iic-osic-tools:latest -s /bin/bash
For your application you can change a few things in this command line.

After -it, you should add --rm, which automatically deletes the container, when your job is done. Just keep an eye on the output files, everything must be in the /foss/designs directory in the container, or it will be deleted.
You can remove the name parameter, as it is deleted automatically and therefore not really useful.
The --user 0:0 should be adapted to fit your current user id and group id. Typically this is 1000:1000 on a local Linux install. You can find it with id -u and id -g
The actuall command you want to run is placed after -s. In our startup script, this mode is basically "skip startup" and it just runs the given command and exists. As you can see in this mode for start_shell, only a bash is launched and nothing else happens. For your application, you put the script to execute into your hosts design directory. When launching the container, the final command line would be something like:

docker run -it --rm --name batch-runner-container --user 1000:1000 -v /path-to-host-home/eda/designs:/foss/designs:rw hpretl/iic-osic-tools:latest -s /bin/bash /foss/designs/my_runner.sh my_runner_parameters
Thank you for the input, we will add a new start script with the comming release!

Hello,
I did the required DRY RUN and ran the final command which looked something like this in my case:
docker run -it --rm --name batch-runner-container --user 1000:1000 -v /home/shavika/eda/designs:/foss/designs:rw hpretl/iic-osic-tools:latest -s /bin/bash/foss/designs/my_runner.sh my_runner_parameters
But it couldn't find the directory :
[INFO] USER_ID: 1000, GROUP_ID: 1000
[INFO] Final PATH variable: /foss/tools/bin:/foss/tools/sak/:/usr/local/sbin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/foss/tools/covered/19d30fc/bin:/foss/tools/cvc_rv/df85a63/bin:/foss/tools/fault/54e8665:/foss/tools/gaw3-xschem/a4bb956/bin:/foss/tools/gds3d/173da0c/bin:/foss/tools/ghdl/v3.0.0/bin:/foss/tools/gtkwave/8e0bcdb/bin:/foss/tools/irsim/16777a6/bin:/foss/tools/iverilog/09f3ebf/bin:/foss/tools/klayout/44a2aa9:/foss/tools/magic/0afe4d8/bin:/foss/tools/netgen/87d8759/bin:/foss/tools/ngspice/ngspice/bin:/foss/tools/nvc/r1.10.1/bin:/foss/tools/openlane/2023.08:/foss/tools/openroad/0cfb9a4/bin:/foss/tools/osic-multitool:/foss/tools/padring/b2a64ab/bin:/foss/tools/pyopus/0.11/bin:/foss/tools/qflow/b0f76bf/bin:/foss/tools/qucs-s/1.1.0/bin:/foss/tools/rftoolkit/bin:/foss/tools/slang/4decdb5/bin:/foss/tools/verilator/947402b/bin:/foss/tools/xschem/e3b9e67/bin:/foss/tools/xyce/Parallel/Release/bin:/foss/tools/yosys/14d50a1/bin:/foss/tools/align-utils
[INFO] Final PYTHONPATH variable: /usr/lib/python310.zip:/usr/lib/python3.10:/usr/lib/python3.10/lib-dynload:/usr/local/lib/python3.10/dist-packages:/usr/lib/python3/dist-packages:/foss/tools/ngspyce/154a272/local/lib/python3.10/dist-packages:/foss/tools/align/d3954af/local/lib/python3.10/dist-packages:/foss/tools/pyopus/0.11/local/lib/python3.10/dist-packages
[INFO] SKIPPING UI STARTUP
[INFO] Executing command: '/bin/bash/foss/designs/my_runner.sh my_runner_parameters'
/dockerstartup/scripts/ui_startup.sh: line 38: /bin/bash/foss/designs/my_runner.sh: Not a directory
Is there a way to extract the correct location of the shell script my_runner.sh ?
Thanks in advance.

Sorry, this was a bit of a missunderstanding here. The script name "my_runner.sh" was just a placeholder for something you could use to call the simulator, or whatever jobs you want to run.
With what I proposed, you copy your data for the simulations to the directory that will be mounted. With the command, the container gets spun up, the script will be run, and after it is done, all results in the mounted directory are kept, and everything else is thrown away and does not eat up storage.
For your specific command, the my_runner.sh does not exist, but also, you are missing a space between /bin/bash and the path to the actual script.


chore(master): release 4.2.3

:robot: I have created a release beep boop
4.2.3 (2024-08-19)
🐞 Bug Fixes

account for all stealing worker nodes (d967235)

✨ Polish

define a variable only when needed (4f28c7a)
display back pressured worker nodes in pool info (8c7a518)

📚 Documentation

publish documentation (e4db94f)

🤖 Automation

deps-dev: apply updates (5a96b64)
deps-dev: bump @types/node in the regular group (#2533) (b2cd95b)
deps-dev: bump eslint-plugin-jsdoc in the regular group (d20645f)
deps-dev: bump the regular group across 11 directories with 1 update (#2534) (a9c233d)
deps: bump the regular group across 11 directories with 2 updates (c657b5d)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v4.2.3
:sunflower:


Add actor_types_taxonomies relationship

Implements #23
We need a many to many join table to create a relationship between
actor_types, introduced for #22, and taxonomies, from impact.
Permissions by role

public: none
registered, no role: none
analyst: R
manager: R
admin: R

@tmfrnz I figured it would be easier to ask this question with the pull request open:
Is the join table supposed to join between taxonomies, or framework_taxonomies?
The issue #23 says:

corresponds to existing table "framework_taxonomies" and new table "actortype_taxonomies"

However, the schema diagram indicated the link came from taxonomies, so that's what I implemented. Let me know if that's not correct 😄

@tmfrnz I figured it would be easier to ask this question with the pull request open:
Is the join table supposed to join between taxonomies, or framework_taxonomies?
The issue #23 says:

corresponds to existing table "framework_taxonomies" and new table "actortype_taxonomies"

However, the schema diagram indicated the link came from taxonomies, so that's what I implemented. Let me know if that's not correct 😄

sorry for the confusion!
but you are right to trust the diagram. what I was trying to highlight was that conceptually measuretypes have the same relation and meaning for measures, as frameworks currently have for recommendations (framework = recommendationtype) and therefore measuretype_taxonomies are analogous to framework_taxonomies. in that sense actortypes is a "framework" for actors

sorry for the confusion!
but you are right to trust the diagram. what I was trying to highlight was that conceptually measuretypes have the same relation and meaning for measures, as frameworks currently have for recommendations (frameworks can be seen recommendationtypes) and therefore measuretype_taxonomies are analogous to framework_taxonomies. in that sense actortypes is a "framework" for actors

awesome, that is what I thought you were getting at 👍

@parndt actor_type_taxonomies Thanks!
On Tue, 2 Nov 2021, 08:50 Philip Arndt, @.***> wrote:

@.**** commented on this pull request.
In app/serializers/actor_type_taxonomy_serializer.rb
https://github.com/dumparkltd/marine-defrag-server/pull/53#discussion_r740789018
:

@@ -0,0 +1,7 @@
+class ActorTypeTaxonomySerializer

include FastApplicationSerializer

attributes :actor_type_id, :taxonomy_id

set_type :actor_type_taxonomies

@tmfrnz https://github.com/tmfrnz what should this type be?
actor_type_taxonomies or plural to both actor_types_taxonomies ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dumparkltd/marine-defrag-server/pull/53#pullrequestreview-794969213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUMLM42QT22BM5WU7FWW4DUJ6J5DANCNFSM5HDNZMQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This contains code from #61 so needs that merged first


msd-eureka-svr to 0.0.2

Promote msd-eureka-svr to version 0.0.2

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Sample image zips have been removed

The sample image data for building the model has been removed from the repository. The readme still references these packages in step 3:
When the app loads, the app will create 3 classifiers for each of the zip files ResumeAR/sanjeev.zip, ResumeAR/steve.zip and ResumeAR/scott.zip.

If users are no longer supposed to use these packages, we should remove the links to them from the readme and provide somewhere else to get sample data from.

@nickmazzi looking into it. We are not using it as we are using the ML model. Let me update the readme with proper instructions.


Adding a rasa exception warning for sentry plot_histogram error

Proposed changes:
Adding a rasa exception for plot_histogram error message.
Steps to replicate the error:
I tried to replicate the error message shown in this issue by

force setting thresh = None in line 127 of this file
and then running the command
rasa test nlu --cross-validation
from an example bot (like formbot)

These settings/run, after quite a while, will produce the following error message :
TypeError: '>' not supported between instances of 'int' and 'NoneType'
This error message looks similar to the original one in the corresponding ticket but it makes me doubt because :

it is actually half of the original sentry error message
it is not in the line mentioned in the sentry
[max(hist_data[0], default=0), max(hist_data[1], default=0)], default=0

Status (please check what you already did):

[x] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

Awesome!!! 💯🥇 Yes, I could not figure how to reproduce it! I will try that then!!! Thanks a bunch !!! 🙏🏽

Hey @wochinge Thank you so much for your very insightful comment! 💯💜 These are very valid points to take into account!
Hey @dakshvar22 ! Any feedback on any of those 3 points mentioned by Tobias would be more than appreciated! 💜In case I can check / test something that enables me to answer them, please let me know! 🙏🏽 Thank you so much in advance!

I am not sure how / when confidence property of ResponseSelectionEvaluationResult can be None. I tried the following:

Go into responseselectorbot under examples folder.
Train with rasa train nlu
Eval with rasa test nlu

I don't see an error. Next, I remove all retrieval intent examples from the NLU data but keep ResponseSelector in my config. Eval still doesn't fail after retraining. Other than that I can't see a situation where confidence would be None. Have we been able to reproduce the error?

I see this now -

I tried to replicate the error message shown in this issue by
force setting thresh = None in line 127 of this file
and then running the command
rasa test nlu --cross-validation
from an example bot (like formbot)

But looks like thresh variable is no longer used in the histogram plotting method of that file. Do we know which exact line was Dominique referring to?

Have we been able to reproduce the error?

@dakshvar22 Not really from the user perspective - but rather took a shortcut to get the specific error message caught by Sentry. So we're right to think that this could be a usage error and that the suggested fix is appropriate?

but rather took a shortcut to get the specific error message caught by Sentry.

I don't think that the shortcut proves anything. We need to actually try to reproduce this (Daksh might just help us to be able to find a fitting scenario quicker)

Hey @dakshvar22 could you please help us with establishing the way forward if we cannot reproduce the user issue?

Since the last time this error was generated, the exact implementation of plotting histograms has changed significantly (PR). Also this now ensures that an exception is raised if the input data to histogram plotting function is empty. So, I am not sure how to reproduce the error anymore. Maybe we can wait out and see if these errors pop up again in the near future?

Since the last time this error was generated, the exact implementation of plotting histograms has changed significantly (PR). Also this now ensures that an exception is raised if the input data to histogram plotting function is empty. So, I am not sure how to reproduce the error anymore. Maybe we can wait out and see if these errors pop up again in the near future?

Thanks for the quick update - I guess we can close this PR considering the new changes @wochinge? The exception that gets raised now is ValueError so Sentry will pick this up in the future if the problem re-appears. Not sure what we should do about the past issue, if to close it as well?

👍🏻 Let's close the PR and mark the sentry issue  as resolved in release xy (not sure which release contains the updates to the plotting) (both in GitHub and Sentry itself)

This PR is being closed as a result of the issue being handled in a different PR.


Add type checking

Make sure that expressions are the correct type by for example checking that the LHS and RHS of an assignment are the same. Same for functions, make sure that the expression in a function call matches the function definition of the arguments

For type checking, there is a need to add compiler-only types for ID. For the member access operator, the second argument is not a variable or anything real. It is the name of a member variable. I propose this to be solved in the type system where we allow for a nice notation to define operator argument types this can be called "MemberIdentity" or something along those lines.
I am, however, unsure if this should be allowed to be used to define from user-code. It seems hard to implement from a compiler standpoint as all uses for the MemberIdentity is to store the name of a member. This has no use in the runtime as the runtime does not know of members.


📝 Docs: Suggest filtering out bots

We're using this action in github.com/typescript-eslint/typescript-eslint and although it works well, it takes up a lot of workflow space. Since it needs to run on all comments & comment edits, every time another bot such as Netlify updates a comment -which is very often in auto-deploys!- the workflow runs. We filed https://github.com/typescript-eslint/typescript-eslint/issues/8205 on ourselves to disable the action on bots.
Can I suggest having the README.md / first-run docs for accessibility-alt-text-bot disable the workflow for bot accounts?
There's something funny about the default docs of an action authored in the GitHub organization causing many many more workflow GitHub Action runs to take up more of GitHub's $.

Really good point! Thanks for opening this issue. cc. @khiga8


Parameter to display real values for axis with exponents or offsets

Hello,
I'm using the MplDataCursor library with MatPlotLib.
I have a problem when I'm trying to add a cursor based on plot with exponent axis.
My axis bounds are 10000000 and 12000000, so MatPlotLib displays them like 1.0 and 1.2 with an 1e7 exponent.
However the cursor displays its values like 1.1 and 1.1 but without the exponent.
Could you add some True/False parameter which displays like 11000000 or 1.1 * 1e7 depending on its values? Is this option already available?
Thanks.

The formatter knows about the offset, you should be able to just ask the formatter for it.

Right, I found the settings.
(Time to close the issue)


Questions about generating SBOM using the sbom-tool

Hello, recently we've been trying to use the sbom-tool, which is a very user-friendly tool. However, what piqued our curiosity is whether the sbom-tool only analyzes dependency files. If we have the source code of a project and dependency files but haven't compiled the project or installed related dependencies, can the sbom-tool still generate an SBOM?
In fact, when we attempted to generate an SBOM for the java-tomcat-maven-example (https://github.com/daticahealth/java-tomcat-maven-example), we encountered some difficulties. The sbom-tool failed to detect any relevant dependencies. Here is the command we used:
sbom-tool generate -b /home -bc /home/java-tomcat-maven-example-master -pn example1 -pv version1 -ps sbom-test

The system version is Ubuntu 20.04.

Or could it be related to the current network conditions, as I performed the above tasks in a situation of poor network connectivity?

The sbom-tool is a build-time SBOM generator. The tool expects a minimum of package restore to be able to identify the dependencies. The SBOM also contains a section where it enumerates the build output files, if there is no build, this section would be empty. Let us know if you have further questions.


Skip test_can_validate_ip_san when unsupported

Resolves #1117.

Add a pragma to catch the linting error, and it looks good to go!

Codecov Report

Merging #1118 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1118   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          21      21
  Lines        1926    1926
======================================
  Hits         1926    1926

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 3bf3009...b02bf41. Read the comment docs.

@sigmavirus24 Addressed that.

🍰


Revert "Fixed 330db Shims to Adopt the PythonRunner Changes [databricks]"

Reverts NVIDIA/spark-rapids#10232 and fixes the plugin build on Databricks 11.3

Closed in favor of #10434


Scaffold smart contract event storage

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#105  👈
#103
#102
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @neekolas and the rest of your teammates on  Graphite


Are steps from Docker.com same as These?

Given Docker for Windows has been released, it's not clear if their install is the same as the install you identified in this article.
So, is https://docs.docker.com/docker-for-windows/  which points to https://download.docker.com/win/stable/InstallDocker.msi
Going to get me the same as this one articulated in the article?
Invoke-WebRequest "https://master.dockerproject.org/windows/amd64/docker-1.13.0-dev.zip" -OutFile "$env:TEMP\docker-1.13.0-dev.zip" -UseBasicParsing

Which is preferred or at least recognize the different paths that might get the same thing.

@cicorias These install different things:

Docker for Windows installs a Docker Linux container development environment on Windows
This documentation takes you through installing Docker Windows container engine for running Docker Windows containers

Here's how you can run them side-by-side: https://forums.docker.com/t/linux-and-windows-images-side-by-side/20518/4?u=friism
At Docker, we're working on making Docker for Windows also set up the Docker Windows container engine (in addition to the Linux components)


[iOS] native event on status bar tap is suppressed by ScrollableTabBar

When you have another scrollview in tabs the native event - scroll-to-top on status bar tap - is suppressed because there are more scrollviews on screen: https://developer.apple.com/library/ios/documentation/UIKit/Reference/UIScrollView_Class/index.html#//apple_ref/occ/instp/UIScrollView/scrollsToTop
The ScrollView inside ScrollableTabBar.js should have set the scrollsToTop={false}.

https://github.com/skv-headless/react-native-scrollable-tab-view/blob/master/index.js#L170
Or do you mean another ScrollView?

the one in index.js is ok, I mean the one in ScrollableTabBar.js https://github.com/skv-headless/react-native-scrollable-tab-view/blob/master/ScrollableTabBar.js
around line 170:
    <ScrollView
        ref={(scrollView) => { this._scrollView = scrollView; }}
        horizontal={true}
        showsHorizontalScrollIndicator={false}
        showsVerticalScrollIndicator={false}
        directionalLockEnabled={true}
        bounces={false}
      >

Would you like to send a pull request?

Sure, I can, I guess that for this one, its not necessary/expected to have possibility to set it true

In this case I see no sense to keep scrollsToTop=true


plugin tutorial/help/cheat sheet

i would like to add new sites for my wonts.
but from the youtube code i didn't know how to exactly implement new sites
do you have any help?

What is a "wont"?

sorry for my bad english skills. i'm not a native speaker in that language :/
with my "wonts", i mean the things i usally do. other words for that in englisch (dict.cc) are "habits" or "usualness". in german so called "Gewohnheit" :)

What sites would you like to implement?

g+, facebook, twitter, please.

sites like ARD or ZDF Mediathek. Those sites are blocked in my school network and have have good documentations.
http://www.ardmediathek.de/tv
http://www.zdf.de/ZDFmediathek#


[FEATURE] Equals method for Chore and ChoreDTO

User story
As a Test Automation Developer in the Chores team, I want to be able to check for value equality on Chore and ChoreDTO objects so that I can ensure the Chores Controller is returning the correct ChoreDTOs / putting the correct Chores into the database
Acceptance criteria

[ ] ChoreDTO has an implementation for .Equals
[ ] Chore has an implementation for .Equals

Additional context
The reason for creating this issue is that the functionality described here doesn't fit within the scope of #213, but is required for it to be completed satisfactorily
i.e. Blocks #213

Upon reconsideration, it's better for the test code to bear the burden of checking equality, since it isn't needed anywhere else


Extract tmt try and tmt init into their own tmt.cli modules

Ongoing process of splitting tmt.cli.
Pull Request Checklist

[x] implement the feature

/packit build

/tests/unit/with-system-packages/basic is failing, looks like some further adjustments are needed:
AttributeError: module 'tmt.cli._root' has no attribute '_construct_trying_provision_options'

One last change needed, fixed in 4b0eb32b, ready for merge.

The provision job failures are irrelevant, merging.


fix(bpf): fix pid key for querying processes map

bpf_get_current_pid_tgid gives both process id and group id, need to use mask to extract pid from the return value.
Same is used while creating the map entry in kepler_sched_switch_trace
Cc: @rootfs @dave-tucker

if we see the overall picture:

In the kernel, the entity that gets scheduled on the cpu is a process (aka thread in user space), and the key in kernel space is pid. when we are counting instructions, time, cache misses, page reads etc , the key there should also be pid.
In user space we read the power from power sources (rapl etc) and divide that power among the entities that get scheduled (threads) as per kepler logic (ratio, lr etc). then when we sum the power usage for a user space process, we should use tgid which is the pid in userspace.
checking further.

the methods kepler_irq_trace , kepler_read_page_trace, and kepler_write_page_trace are getting the correct value for key in the processes bpf map.
the changes in this PR are not required. Closing the PR.

Unless the 2 are being switched around anywhere else in the codebase.

@dave-tucker it is getting switched, my bad i should have read more code.
kepler.bpf.h:
typedef struct process_metrics_t {
	u64 cgroup_id;
	u64 pid;  // pid is the kernel space view of the thread id
	u64 tgid; // tgid is the user space view of the pid
	u64 process_run_time;
	u64 task_clock_time;
	u64 cpu_cycles;
	u64 cpu_instr;
	u64 cache_miss;
	u64 page_cache_hit;
	u16 vec_nr[10];
	char comm[16];
} process_metrics_t;

bpf_perf.go:
// must be in sync with bpf program
type ProcessBPFMetrics struct {
	CGroupID       uint64
	ThreadPID      uint64 /* thread id */
	PID            uint64 /* TGID of the threads, i.e. user space pid */
	ProcessRunTime uint64 /* in ms */
	TaskClockTime  uint64 /* in ms */
	CPUCycles      uint64
	CPUInstr       uint64
	CacheMisses    uint64
	PageCacheHit   uint64
	VecNR          [config.MaxIRQ]uint16 // irq counter, 10 is the max number of irq vectors
	Command        [16]byte
}


Rename master to main

This is failing because it is using the now unsupported method for setting environment variables in GH actions.

This is failing because it is using the now unsupported method for setting environment variables in GH actions.

My PR https://github.com/cnabio/signy/pull/89 fixes this issue as well, so after that's approved you'll be able to get this in.


Inform user about migrated certificate grouping (EXPOSUREAPP-11988)

[x] https://github.com/corona-warn-app/cwa-app-android/pull/4872 should be merged at first

Testing

Scan some certificates using CWA 2.18 where certificates will belong to the same user but the names will be slightly different. Eg. switch last name and first name

cwa tc gen -e wru -s dob="1988-01-01" nam.fn="Max" nam.gn="Mustermann" -o
cwa tc gen -e wru -s dob="1988-01-01" nam.fn="Mustermann" nam.gn="Max" -o

Update to CWA with this branch, and go to the certificates tab
You should see a dialog informing you that certificates have been regrouped (only once and only if the regrouping really happened)

Jira Ticket
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-11988

Dialog Text formatting (Bold Title, more line spacing in subtitle) looks a bit different in figma. Not sure if we can adjust that easily...
see here


Removed dependency on underscore (indexOf instead of _.contains)

Thanks for sharing this function.  I made a small change to remove the dependency on underscore. Now it's all vanilla javascript.

v0.0.4 https://www.npmjs.com/package/heroku-ssl-redirect Thanks


Wrong Centerpoint demo result and eval error

Hi openpcdet team @cgerum @lindahua @chreisinger @acivgin1 @sshaoshuai ,
Thanks for sharing this framework.
I follow the getting_start.md, my environment setups(part) is as follows:
pytorch 1.11.0
cuda 11.3
spconv 2.1.21
numba: 0.55.1
open3d 0.15.2
waymo: 1_3_0
waymo-open-dataset-tf-2-5-0 1.4.1
gpu: 3090 *1
I pulled the newest code of pcdet repo and tried to reproduce the mentioned centerpoint result here:
CenterPoint	71.33/70.76	63.16/62.65	72.09/65.49	64.27/58.23	68.68/67.39	66.11/64.87
I did the following tests:

I did not modify any code, and trained centerpoint for 30 epoch(with default centerpoint.yaml)
python train.py --cfg_file cfgs/waymo_models/centerpoint.yaml

tried the demo.py with the following command
python demo.py --cfg_file cfgs/waymo_models/centerpoint.yaml --ckpt ../output/waymo_models/centerpoint/default/ckpt/checkpoint_epoch_30.pth --data_path ../data/waymo/waymo_processed_data_v0_5_0/segmentxxx/0000.npy --ext '.npy'

The demo result seems incorrect:

tried the test.py
python test.py --cfg_file cfgs/waymo_models/centerpoint.yaml --ckpt ../output/waymo_models/centerpoint/default/ckpt/checkpoint_epoch_30.pth --batch_size 4

get logs and erros (full log
log.txt
is attached):
2022-04-24 15:15:00,441   INFO  *************** EPOCH 30 EVALUATION *****************
eval:   0%|                                                                                | 0/9997 [00:00<?, ?it/s]/data/lin/research/github/OpenPCDet/tools/../pcdet/models/model_utils/centernet_utils.py:142: UserWarning: floordiv is deprecated, and its behavior will change in a future version of pytorch. It currently rounds toward 0 (like the 'trunc' function NOT 'floor'). This results in incorrect rounding for negative values. To keep the current behavior, use torch.div(a, b, rounding_mode='trunc'), or for actual floor division, use torch.div(a, b, rounding_mode='floor').
topk_ys = (topk_inds // width).float()
/data/lin/research/github/OpenPCDet/tools/../pcdet/models/model_utils/centernet_utils.py:146: UserWarning: floordiv is deprecated, and its behavior will change in a future version of pytorch. It currently rounds toward 0 (like the 'trunc' function NOT 'floor'). This results in incorrect rounding for negative values. To keep the current behavior, use torch.div(a, b, rounding_mode='trunc'), or for actual floor division, use torch.div(a, b, rounding_mode='floor').
topk_classes = (topk_ind // K).int()
eval: 100%|██████████████████████████████████████████████████████████████████████████████████████| 9997/9997 [27:20<00:00,  6.09it/s, recall_0.3=(0, 1541320) / 1800720]
2022-04-24 15:42:21,291   INFO  *************** Performance of EPOCH 30 *****************
2022-04-24 15:42:21,301   INFO  Generate label finished(sec_per_example: 0.0410 second).
2022-04-24 15:42:21,302   INFO  recall_roi_0.3: 0.000000
2022-04-24 15:42:21,302   INFO  recall_rcnn_0.3: 0.855947
2022-04-24 15:42:21,302   INFO  recall_roi_0.5: 0.000000
2022-04-24 15:42:21,302   INFO  recall_rcnn_0.5: 0.768198
2022-04-24 15:42:21,302   INFO  recall_roi_0.7: 0.000000
2022-04-24 15:42:21,302   INFO  recall_rcnn_0.7: 0.503220
2022-04-24 15:42:21,353   INFO  Average predicted number of objects(39987 samples): 111.948
2022-04-24 15:42:24.638565: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcudart.so.11.0
Start the waymo evaluation...
Number: (pd, 4476454) VS. (gt, 1630510)
Level 1: 1377451, Level2: 253059)
Traceback (most recent call last):
File "test.py", line 200, in
main()
File "test.py", line 196, in main
eval_single_ckpt(model, test_loader, args, eval_output_dir, logger, epoch_id, dist_test=dist_test)
File "test.py", line 62, in eval_single_ckpt
eval_utils.eval_one_epoch(
File "/data/lin/research/github/OpenPCDet/tools/eval_utils/eval_utils.py", line 110, in eval_one_epoch
result_str, result_dict = dataset.evaluation(
File "/data/lin/research/github/OpenPCDet/tools/../pcdet/datasets/waymo/waymo_dataset.py", line 315, in evaluation
ap_result_str, ap_dict = waymo_eval(eval_det_annos, eval_gt_annos)
File "/data/lin/research/github/OpenPCDet/tools/../pcdet/datasets/waymo/waymo_dataset.py", line 298, in waymo_eval
ap_dict = eval.waymo_evaluation(
File "/data/lin/research/github/OpenPCDet/tools/../pcdet/datasets/waymo/waymo_eval.py", line 206, in waymo_evaluation
metrics = self.build_graph(graph)
File "/data/lin/research/github/OpenPCDet/tools/../pcdet/datasets/waymo/waymo_eval.py", line 122, in build_graph
metrics = detection_metrics.get_detection_metric_ops(
File "/home/server/.local/lib/python3.8/site-packages/waymo_open_dataset/metrics/python/detection_metrics.py", line 134, in get_detection_metric_ops
ground_truth_speed = tf.zeros((num_gt_boxes, 2), tf.float32)
File "/home/server/.local/lib/python3.8/site-packages/tensorflow/python/util/dispatch.py", line 206, in wrapper
return target(*args, **kwargs)
File "/home/server/.local/lib/python3.8/site-packages/tensorflow/python/ops/array_ops.py", line 2911, in wrapped
tensor = fun(*args, **kwargs)
File "/home/server/.local/lib/python3.8/site-packages/tensorflow/python/ops/array_ops.py", line 2960, in zeros
output = _constant_if_small(zero, shape, dtype, name)
File "/home/server/.local/lib/python3.8/site-packages/tensorflow/python/ops/array_ops.py", line 2896, in _constant_if_small
if np.prod(shape) < 1000:
File "<array_function internals>", line 5, in prod
File "/home/server/anaconda3/envs/torch11/lib/python3.8/site-packages/numpy/core/fromnumeric.py", line 3051, in prod
return _wrapreduction(a, np.multiply, 'prod', axis, dtype, out,
File "/home/server/anaconda3/envs/torch11/lib/python3.8/site-packages/numpy/core/fromnumeric.py", line 86, in _wrapreduction
return ufunc.reduce(obj, axis, dtype, out, **passkwargs)
File "/home/server/.local/lib/python3.8/site-packages/tensorflow/python/framework/ops.py", line 867, in array
raise NotImplementedError(
NotImplementedError: Cannot convert a symbolic Tensor (strided_slice:0) to a numpy array. This error may indicate that you're trying to pass a Tensor to a NumPy call, which is not supported
The tensorbaord visualization seems correct:

I tried the pretrained second model on kitti dataset, the demo result seems correct.
Do I miss anything (that lead to the wrong demo result)?
Thanks in advance,
Lin

demo.py uses a "lightweight" DemoDataset
https://github.com/open-mmlab/OpenPCDet/blob/846cf3ed21c274f95f041e7e21fd039ca5c99a1f/tools/demo.py#L23
Replace the DemoDataset with the "original" WaymoDataset and it should work.
https://github.com/open-mmlab/OpenPCDet/blob/846cf3ed21c274f95f041e7e21fd039ca5c99a1f/pcdet/datasets/waymo/waymo_dataset.py#L21

hello @MartinHahner ,
Use the WaymoDataset sovle the demo problem.   The error("NotImplementedError: Cannot convert a symbolic Tensor (strided_slice:0) to a numpy array")  is solved by downgrading numpy from 1.21.1 to 1.19.2.  I guess it's caused by the package waymo-open-dataset-tf-2-5-0 (possibly not compatible with python3.8 and numpy 1.21). There is a newer one: waymo-open-dataset-tf-2-6-0. maybe update to this version solving it automatically.

Hello @bigsheep2012 ,
This is my configuration：
numba    0.55.1
numpy    1.21.0
torch       1.11.0+cu113
waymo-open-dataset-tf-2-6-0    1.4.9
GPU    A100*8
When I run
bash scripts/dist_test.sh ${NUM_GPUS}  --cfg_file cfgs/waymo_models/centerpoint_4frames.yaml \ --ckpt ../output/waymo_models/centerpoint_4frames/default/ckpt/checkpoint_epoch_36.pth \ --set DATA_CONFIG.DATA_SPLIT.test train
use torch1.13+numpy1.23.5, I got same error with, but I use this configuration I got other error:
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xe
Traceback (most recent call last):
File "test.py", line 14, in
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xe
RuntimeErrorTraceback (most recent call last):
: module compiled against API version 0x10 but this version of numpy is 0xe  File "test.py", line 14, in
Traceback (most recent call last):
File "test.py", line 14, in
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
import SharedArray
ImportError: numpy.core.multiarray failed to import
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xe
Traceback (most recent call last):
File "test.py", line 14, in
import SharedArray
ImportError: numpy.core.multiarray failed to import
import SharedArray
ImportError: numpy.core.multiarray failed to import
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
import SharedArray
ImportError: numpy.core.multiarray failed to import
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xe
Traceback (most recent call last):
File "test.py", line 14, in
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
import SharedArray
ImportError: numpy.core.multiarray failed to import
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xe
Traceback (most recent call last):
File "test.py", line 14, in
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
import SharedArray
ImportError: numpy.core.multiarray failed to import
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xe
Traceback (most recent call last):
File "test.py", line 14, in
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
import SharedArray
ImportError: numpy.core.multiarray failed to import
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xe
Traceback (most recent call last):
File "test.py", line 14, in
from eval_utils import eval_utils
File "/home/jovyan/OpenPCDet/tools/eval_utils/eval_utils.py", line 8, in
from pcdet.models import load_data_to_gpu
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/init.py", line 6, in
from .detectors import build_detector
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/init.py", line 1, in
from .detector3d_template import Detector3DTemplate
File "/home/jovyan/OpenPCDet/tools/../pcdet/models/detectors/detector3d_template.py", line 6, in
from ...ops.iou3d_nms import iou3d_nms_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 8, in
from ...utils import common_utils
File "/home/jovyan/OpenPCDet/tools/../pcdet/utils/common_utils.py", line 7, in
import SharedArray
ImportError: numpy.core.multiarray failed to import
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 42457) of binary: /opt/conda/bin/python
Traceback (most recent call last):
File "/opt/conda/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/opt/conda/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/jovyan/.local/lib/python3.8/site-packages/torch/distributed/launch.py", line 193, in
main()
File "/home/jovyan/.local/lib/python3.8/site-packages/torch/distributed/launch.py", line 189, in main
launch(args)
File "/home/jovyan/.local/lib/python3.8/site-packages/torch/distributed/launch.py", line 174, in launch
run(args)
File "/home/jovyan/.local/lib/python3.8/site-packages/torch/distributed/run.py", line 715, in run
elastic_launch(
File "/home/jovyan/.local/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 131, in call
return launch_agent(self._config, self._entrypoint, list(args))
File "/home/jovyan/.local/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 245, in launch_agent
raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:

hi @SkywalkerHe
The sharedarray problem seems like a duplicate issue
Try this solution here
good luck


Image preview of points in message (static maps)

Offer enable showing static maps for better location message as image, with marker(s).
Implementation notes:

Link generated from staticmaps should be hidden behind some other URL to prevent misusing token to static maps provider.
Instead of sending image and converting all logic from message to caption (which has much lower character limit), set disable_web_page_preview to false and set link to this static map image as first in message. Telegram will show image below message automatically.
Depend of pricing of static map providers, but generating static maps could be disabled in default in user settings.

Notes:

nice examples are at http://staticmapmaker.com/
Bing maps ~seems to have no limitations or at least plans big enough~ has limitation to 125 000 request per year which is ~342 requests per day. They also offer plans for non-profit organizations to 50 000 request per 24 hours.
More info on https://www.microsoft.com/en-us/maps/licensing/ and https://www.microsoft.com/en-us/maps/licensing/licensing-options/
HERE maps doesn't have support for markers at all
Google doesn't offer free plan but free mothly credit

Generating and caching image preview of static map is done, just disable_web_page_preview is set to false. Waiting for user/chat settings, where user or admin can enable or disable this preview.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1083 ms

Resolved: Bofrost.at performance has improved in 88a7ed6.


fixed the width issue of nevermissanupdate container.

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!

Sumarise what this PR fixes:
changed the width of the nevermissanupdate container in css.
Fixes #: #134
Checklist:
(Add an x between the brackets to check the items)

[ x] I have sumarised what this PR fixes above
[ x] I have provided the issue ID/URL above
[x ] This PR fixes only the issue mentioned above (create seperate PRs for seperate issues)
[ x] I was the first one to claim the issue or I created the issue

@MattCSmith and @brittneypostma please review the PR.

Hey @armedev
Apologies there is a conflict here I am unsure how to resolve. Any chance you could take a look and resolve it for me.
Regards
Matt

@MattCSmith now you can review my pr.

I found the pull request, I'm not sure how this is solving a responsive issue though. 100vw takes up the full viewport width, 100% will take up 100% of the parent container. I don't think this aligns with the quality pr that Digital Ocean expects for Hacktoberfest. I am going to close this one out.

I found the pull request, I'm not sure how this is solving a responsive issue though. 100vw takes up the full viewport width, 100% will take up 100% of the parent container. I don't think this aligns with the quality pr that Digital Ocean expects for Hacktoberfest. I am going to close this one out.

actually my bad the issue was solved by setting X-scroll to hidden by @AdityaSaroj.


Is there a way to disable query cache?

Hi there,
I'm trying to benchmark OmnisciDB via pyomnisci. Is there a way to disable query cache (both for CPU and GPU) between runs for a single query, i.e., disabling omnisci to use any (final or intermediate) result of the previously run queries while letting it use the tables loaded in memory from disk?
Thanks so much,
Dong

HI @dongheuw,
We don't use ant result cache for final or intermediate results with just one exception; the estimator result cache for some high cardinality queries.
You can disable setting the parameter --use-estimator-result-cache to false, but I'm not sure you want to disable it.
Other than that, we cache the data reads to CPU and GPU caches, the query plans, and the index we build for hash and range joins, but right now, we don't cache any results, so every result of your query reflects the data you have in the DB.
Regards,
Candido

Hi @cdessanti,
Thanks so much for your prompt reply.
Is there a way to disable the cache for "the index we build for hash and range joins"?
Thanks,
Dong

Hi,
the parameter to disable is called --use-hash-table-cache yes.
the data-recycler is about metadata and it's relatively a new parameter, but you can disable it too.

Thank you!


WIP: 1.x Release Branch

The goal of for the 1.x release is parity with npm init -y with a few better defaults which were already feature complete. Things that need to be done to achieve this:

[x] A test which exercises npm init as a comparison

[x] From that we should be able to make a better list

[x] Documentation
[ ] what else?

@Ethan-Arrowood guessing you are not a member of this org, but would you mind giving this a once over?
This is barely more than npm init but I think that is fine for us to cut the 1.x. I have tested with both the programmatic api and the cli in a few projects. And since it is a 1.0.0 I am comfortable just publishing but thought I would get your input first.  Same goes for anyone else who wants to review this, but Ethan and I discussed this as part of the package metadata collab space.

Sure! We could probably Bikeshed a lot of this (typescript, esm, dependencies/dependency less, node version support). But I'm all about iteration so we should merge this and work from there.

Yeah, I would prefer we avoid much bikeshedding if we can. That said, just to address a few things:
Especially for small tools like this, simplicity is key. So:

typescript: We wouldn't get much internal value out of types because it should be a tiny module. I do think we should add jsdoc annotations and generate a .d.ts.
ESM: since the complexity to dual publish is still high and ESM consumers can easily work around a cjs package I see this as a simplicity play as well.
dependencies: this for sure can be cleaned up imo. But right now maybe not necessary.
node versions: So I went with current (and using the nv tool to keep that updated). Ideally we would keep support to just node lts versions at the time of publish I think


Allow for nested schema tests without specifying all keys

Currently, if you have a return_json like this:
{"some_dict": {"key_1": "value_1", "key_2": "value_2"}}

And you write a schema test like this:
schema: {"some_dict": {"key_1": "value_1"}}

It will fail. Since some nested JSON can have many keys that might be irrelevant to your test, this PR adds support for specifying just the keys you are looking for. In particular, if you have the same nested key in many places, you may want to have the parent key listed. For example:
Payload:
{"price": {"value": "1.0", "type": "bid"}, "quantity": {"value": "10", "type": "bid"}}

You need to write tests like:
schema: {"price": {"value": "1.0"}}
schema: {"quantity": {"value": "10"}}

Since negative tests are allowed, you can still force components (empty or null) to be in your tests if you desire.

Probably in the README... I'll add something.
On Thu, Nov 21, 2019 at 10:10 AM Matthew Goldey<EMAIL_ADDRESS>wrote:

@mgoldey approved this pull request.
Nice value add. Where should information around our test spec live? Might
affect clients
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/finos/greenkey-discovery-sdk/pull/106?email_source=notifications&email_token=ACPBTN2O2HNY5OMIG6BLG63QU2XGPA5CNFSM4JP3ZJHKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCMRCATA#pullrequestreview-321003596,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPBTN7ZMAHO7TW3OWECQHLQU2XGPANCNFSM4JP3ZJHA
.

--

© 2019 Green Key Technologies, Inc.  All
rights reserved.The information contained in this e-mail message is only
for the personal and confidential use of the intended recipient(s).  If you
received this communication in error, please notify us immediately by
e-mail, and delete the original message.  Unauthorized use, disclosure or
copying of this communication or any part of it is strictly prohibited and
may be unlawful.
www.greenkeytech.com http://www.greenkeytech.com/


Follow up to reauthentication automated test PR

@tgolen  Just added some extra comments and things to add more context in the tests and also added assertions for Onyx data.
Details
Follow up to comments made in https://github.com/Expensify/Expensify.cash/pull/1152
Fixed Issues (Related to PR)
https://github.com/Expensify/Expensify.cash/pull/1152
Tests
❌

@tgolen simplified this a bunch. Things feel clearer now I think. Lemme know if anything could use explaining.

@tgolen simplified this a bunch. Things feel clearer now I think. Lemme know if anything could use explaining.


chore: add deprecation note to acr plugin

RHIDP-3546: Add deprecation note to acr plugin

@nickboldt can we remove the source from main now too? I think it reduces risk of things getting out-of-sync (refs: #2216 (comment)), and reduce uncessary noise from dependency updates.

yes, you can delete everything and just leave the readme. https://github.com/janus-idp/backstage-plugins/tree/main/plugins/feedback for example


[FEA] Import from S3

 Issue by benfred
Saturday May 23, 2020 at 02:42 GMT
Originally opened as https://github.com/rapidsai/recsys/issues/169

We should be able to load files from S3 for both preprocessing and dataloading. We need to add the ability to cudf to efficiently read from S3 , and then wire up nvtabular to support this.

https://docs.dask.org/en/latest/remote-data-services.html

cuDF supports reading from s3, gcs etc. (added https://github.com/rapidsai/cudf/pull/2635) similar to dask using the fsspec library to handle the filesystem abstraction. cudf readers should also support the storage_options kwarg similar to what's described in remote-data-services doc linked above.
Note: The current codepath for remote filesystems involves reading a BytesIO from the python file buffer object which is passed down to libcudf.

thanks ayush! We've updated to use fsspec everywhere with this PR - it was pretty straightforward because like you mentioned cudf supports this under the hood (aside from in some cases like cudf.io.parquet.read_parquet_metadata doesn't yet support this, and we had to read into a bytesio object ourselves). I think this even gets us GCS/HDFS support too =)


How to create a Grid with Custom List

Hi
My name is Edison and i want to show in Grid a custom list object with search (Serene Framework Style). I create this object getting the data from Oracle Database only for reading not for update/edit/delete transaction.  Do you have any sample code implement serene framework with oracle database.
Thanks...

If I understand you correctly, you are looking for read only. Volkan has given us a sample of this located here. Take a look around and see if that is what you are looking for.

Hi
Thanks for answer.  My default repository belongs to SQL Server Database but I want to access some tables from Oracle Database for validation purpose.  That's I want to populate the grid. I Already create the connection string on web.config to oracle database but I don't know exactly to do use the serene framework on it to access and display the data on that grid.. I read that sergen.exe doesn't work for oracle database and I had to create manually. Any help on this its appreciate..

Let me do a little research on it and I'll get back to you. Everyone else -
feel free to jump in and help.
On Tue, Nov 29, 2016 at 4:00 PM u0122300<EMAIL_ADDRESS>wrote:

Hi
Thanks for answer. My default repository belongs to SQL Server Database
but I want to access some tables from Oracle Database for validation
purpose. That's I want to populate the grid. I Already create the
connection string on web.config to oracle database but I don't know exactly
to do use the serene framework on it to access and display the data on that
grid.. I read that sergen.exe doesn't work for oracle database and I had to
create manually. Any help on this its appreciate..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/volkanceylan/Serenity/issues/1523#issuecomment-263697101,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK6u3TOnIhVBAYa1JMwBYyj5-0YJS1r4ks5rDJJVgaJpZM4K-foy
.

Hi, @u0122300 Where did you read that sergen doesn't work with oracle?
https://volkanceylan.gitbooks.io/serenity-guide/content/howto/how_to_work_with_other_database_types.html
The sample already works with more than one connection string. There is the default and Northwind keys... Take a look around sample, guide, etc for more intel on how to work with it.
Best Regards

SerenityDeveloperGuidePage224.pdf
Here in this page says Sergen doesn't works with oracle Serenity Developer Guide

Yes, indeed. I corrected that a few seconds after I've sent it.
See my answer again. Unless you get stuck with something related to a db oracle behavior, I hope It helps. In fact, creating your own rows is just... boring, but it's not that slow... but sometimes, correcting db old fashioned designed names is just overboring. :-)
Best Regards

Hi
When  I execute sergen.exe with the Oracle Connection String configured in web.config its display the following error message.

The created connection string its all ready tested and interact with Oracle Database.

Yes. It does not work. You have to write your own row.cs, etc.
ConnectionKey("Northwind") orConnectionKey("Default")
or in services
var connection = SqlConnections.NewByKey("Default")
are for configuring that in code.
Best Regards

As noted in guide, Sergen doesn't support Oracle


feat: add loading and error to cms tree state

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:
What Is the Current Behavior?

Issue Number: Closes #
What Is the New Behavior?
Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No
Other Information
AB#78961

We will not add error and loading state for the CMS page tree as long as it is not used in the PWA.


Topic information

Connects to ros2/ros1_bridge#3
The rmw_connext_cpp implementation is ready for review.

lgtm

+1

+1

BTW, here's an idea, we could expose this information as a parameter too. I haven't given much thought to it, though.


Miniserve is Webhosting Panel not a Webserve

Platform
Which platform does the bug apply to?

WebExtension (Google Chrome, Mozilla Firefox)
NPM
Bookmarklet
Website

Describe the bug
Miniserve is reported as a webserver but its webhosting plane.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '...'
Scroll down to '...'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Please provide additional information.


 You will notice that the blocks will rearrange as the module doesn't restrict drag events from external sources.

Yeah.  I tested this out on safari and it wasn't a problem but then I opened up chrome and it acts as @Georgette states.
There are also a lot of issue with elements being restricted from spots they should be able to be dropped in.  Mostly at the end of a subarray.  I worked on nester sortable UITableView (iOS version of a list) a while ago and I know the logic gets a little ugly.  It would be great to see this project carried on to a useable state.

This is an issue of browser implementation. We can't do anything about it here.


Error with ngMock / angular-mock since version 1.3.12

I tried to move angular-mock past 1.3.12 and I couldn't so far.
The other packages are sitting at 1.3.14 at the moment but if I run my tests (Karma + Jasmine) with a recent with a recent version of angular-mock I get something like this:
Error: [$injector:modulerr] Failed to instantiate module ng due to:
Error: [$injector:modulerr] Failed to instantiate module ngLocale due to:
RangeError: Maximum call stack size exceeded
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
    at Function.angular.injector.$$annotate (http://localhost:9876/base/tmp/lib/angular-mocks.js:2147:36)
http://errors.angularjs.org/1.3.14/$injector/modulerr?p0=ngLocale&p1=RangeError%3A%20Maximum%20call%20stack%20size%20exceeded%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)
    at REGEX_STRING_REGEXP (http://localhost:9876/base/tmp/lib/angular.js:63:12)
    at http://localhost:9876/base/tmp/lib/angular.js:4120:15
    at forEach (http://localhost:9876/base/tmp/lib/angular.js:323:20)
    at loadModules (http://localhost:9876/base/tmp/lib/angular.js:4081:5)
    at http://localhost:9876/base/tmp/lib/angular.js:4098:40
    at forEach (http://localhost:9876/base/tmp/lib/angular.js:323:20)
    at loadModules (http://localhost:9876/base/tmp/lib/angular.js:4081:5)
    at Object.createInjector [as injector] (http://localhost:9876/base/tmp/lib/angular.js:4007:11)
    at Object.workFn (http://localhost:9876/base/tmp/tools/angular-mocks.js:2354:52)
http://errors.angularjs.org/1.3.14/$injector/modulerr?p0=ng&p1=Error%3A%20%5B%24injector%3Amodulerr%5D%20Failed%20to%20instantiate%20module%20ngLocale%20due%20to%3A%0ARangeError%3A%20Maximum%20call%20stack%20size%20exceeded%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0A%20%20%20%20at%20Function.angular.injector.%24%24annotate%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular-mocks.js%3A2147%3A36)%0Ahttp%3A%2F%2Ferrors.angularjs.org%2F1.3.14%2F%24injector%2Fmodulerr%3Fp0%3DngLocale%26p1%3DRangeError%253A%2520Maximum%2520call%2520stack%2520size%2520exceeded%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%250A%2520%2520%2520%2520at%2520Function.angular.injector.%2524%2524annotate%2520(http%253A%252F%252Flocalhost%253A9876%252Fbase%252Ftmp%252Flib%252Fangular-mocks.js%253A2147%253A36)%0A%20%20%20%20at%20REGEX_STRING_REGEXP%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A63%3A12)%0A%20%20%20%20at%20http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A4120%3A15%0A%20%20%20%20at%20forEach%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A323%3A20)%0A%20%20%20%20at%20loadModules%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A4081%3A5)%0A%20%20%20%20at%20http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A4098%3A40%0A%20%20%20%20at%20forEach%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A323%3A20)%0A%20%20%20%20at%20loadModules%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A4081%3A5)%0A%20%20%20%20at%20Object.createInjector%20%5Bas%20injector%5D%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Flib%2Fangular.js%3A4007%3A11)%0A%20%20%20%20at%20Object.workFn%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Ftmp%2Ftools%2Fangular-mocks.js%3A2354%3A52)%0A%20%20%20%20at%20attemptSync%20(http%3A%2F%2Flocalhost%3A9876%2Fbase%2Fnode_modules%2Fjasmine-core%2Flib%2Fjasmine-core%2Fjasmine.js%3A1741%3A24)
    at REGEX_STRING_REGEXP (http://localhost:9876/base/tmp/lib/angular.js:63:12)
    at http://localhost:9876/base/tmp/lib/angular.js:4120:15
    at forEach (http://localhost:9876/base/tmp/lib/angular.js:323:20)
    at loadModules (http://localhost:9876/base/tmp/lib/angular.js:4081:5)
    at Object.createInjector [as injector] (http://localhost:9876/base/tmp/lib/angular.js:4007:11)
    at Object.workFn (http://localhost:9876/base/tmp/tools/angular-mocks.js:2354:52)

The code added between 1.3.11 and 1.3.12 is related to those annotated functions. It seems it is entering some sort of infinite loop as soon as is loads the ng module.
Any idea ? Thanks.

This is probably related to https://github.com/angular/angular.js/commit/0baa17a3b7ad2b242df2b277b81cebdf75b04287
@richard-lopes can you plz share a minimal reproduce for this issue? (preferably on plunker or something similar, but a bit more context to what you might be doing differently in your tests might also help)

@richard-lopes Just an idea from looking at the stack - Is it possible that you are loading angular-mocks twice into karma

Thanks. We have a rather huge project to get something minimal, but I do know is that as soon we have inject() used in a test it will fail with stack trace I gave here. As soon as I find a minute, I will try to make something minimal.
I am going to check if we do have angular-mocks loaded twice by any chance.

I tried to run the tests with a debugger. The first time it seems to get into the $$annotate function, the annotatedFucntions array is defined and empty.

I can close the issue !
Good call. It was loaded twice indeed.
I was concatenating libraries in a vendor.js file that contained angular-mocks.js as well and also loading it again in Karma from a different location.
So not only getting rid of it from the vendor.js file solved the duplication issue but it also removed that unnecessary payload from the vendor.js served to end users.
Many thanks.


Removing premature return on lookahead

I think this prematurely ended the BnB algorithm. The fact that we cannot go farther in depth and take a shortcut to exclusion branch doesn't mean we should exit if the "newly excluded" is "first" - we still have a lot of branches to explore.

The logic should be the same as on line 93

I might be wrong; but I am now running javascript and scala implementations (javascript is basically C++), looking where they differ in what they choose, and this returned from BnB algorithm when Javascript didn't

...and yeah, it seems this fix does decrease the total amount significantly.

...and yeah, it seems this fix does decrease the total amount significantly.

You mean it becomes even more efficient?

Yes, it has lower fees.
Because now it gives up searching for match when the lookahead "cuts" when the previously selected (but not the one we are cutting!) is "first" (meaning, all before it are not selected).
So sometimes it gives up the whole search, when there is still a match to be found.
The c++ code in the PR doesn't do this, so it's fine.
I wrote much much worse code in my thesis :D take it as a good sign someone cares about it. :)


prevent default event of open trigger event

I have a modal and my open trigger is an link a[data-micrmodal-trigger]
It's a link so the no-js fallback can be a link to a normal page. The problem is micromodal does not seem to prevent the default action of the trigger so the modal opens then the page changes.
Any reason not to do event.preventDefault() on the open trigger?

@zebapy Fair point. Can't think of a scenario where event.preventDefault() would adversely effect the use case. Would be open to submitting a PR?
@kalpeshsingh Any thoughts on this?

Looks like @gautamz07 opened one

@ghosh I thought this was merged but I still have the same problem with 0.4.0. Here's reproduction https://codesandbox.io/s/micromodal-bug-wnibs

@zebapy, yes its merged but not yet released. Working on some other things to release with this.

Fixed in 0.4.3

@ghosh This doesn't seem to be working in v0.4.6. Was the fix rolled back at some point?

@ghosh This doesn't seem to be working in v0.4.6. Was the fix rolled back at some point?

Try
MicroModal.init({
    onShow: function(modal, element, event) {
        event.preventDefault();
        event.stopPropagation();
    }
});


create automated alerts from jobs

All jobs are now scheduled via manual-jobs

All jobs publish a -done metric when they ran without an error

When scheduling jobs, an alarm is also created on cloudwatch

The alarm is set to go off if the cron did not run once in the doubled interval

e.g. a cron runs every minute, alarm is triggered if the crons runs less or equal than once every two minutes
very short intervals (<30s) will be checked to run once per minute

Small fix in the scheduling to track sentry transactions correctly on error.

addresses #58


Axios Vulnerability

NPM is warning me about a vulnerability in Axios. Neither npm audit fix nor npm audit fix --force fixes the issue.

This is becoming an issue as NexusIQ is also reporting it as a severe vulnerability.
https://nvd.nist.gov/vuln/detail/CVE-2022-1214