Page side navigation not scrollable and buttons are hidden.

The Page navigation side navigation is currently not scrollable and tend to hide the buttons when on smaller screens.

we should ensure we use a list scroll.

we could as well get rid of the titles and leave just the icon when the drawer is minimized, that would give a smaller surface area

we could as well get rid of the titles and leave just the icon when the drawer is minimized, that would give a smaller surface area

I believe it is a great idea


dice game step 9 (js beta) needs to fix one of the hints

Describe the Issue
In step 9 of the review algorithmic thinking by building a dice game project (js beta), if you make a mistake in the line of code that is supposed to add the new li element, you will see this hint:
When updateScore(10, "hi") is called, your new list element should have the text hi: 10.
Notice that this hint says that the end result is hi: 10 with no space to the left of the colon?
But the instructions say:
add a new list item to the score history with the format ${achieved} : ${selectedValue}.
Here we see a space to the left (and right) of the colon.
The test will not pass if you do not add that space to the left. So it is quite confusing when people are seeing the exact result being printed and logged from the hint but their code does not pass.
A fix for this is to add one space to the left of the colon in the hint text.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/review-algorithmic-thinking-by-building-a-dice-game/step-9
Your code
const updateScore = (selectedValue, achieved) => {
  score += selectedValue
  totalScore.textContent = `${score}`;
  scoreHistory.innerHTML += `<li>${achieved}: ${selectedValue}</li>`
}

(this is sample code from a camper on the forum)
https://forum.freecodecamp.org/t/review-algorithmic-thinking-by-building-a-dice-game-step-9/701542
Expected behavior
The hint should show the correct spacing.
Screenshots
No response
System
N/A
Additional context
No response

:)


Delete package alert

Hello, it was my mistake to publish a flutter package under the generic name "alert".
I already published this small util package under "flutter_alert".
If its possible it would be great if you can remove the first one.
Sorry & thank you.

No problem - thanks for reaching out.
The package is deleted now


Add android.permission.GET_TASKS to Review List

Retrieving running apps permission allows apps to retrieve info about currently and recently running tasks. This allows the apps to discover info about which apps are used on the device. Only specific kinds of apps will need such a permission.

It doesn't seem like this is an issue on Android 5+ devices, as GET_TASKS is deprecated and restricted to system apps:
https://developer.android.com/reference/android/Manifest.permission#GET_TASKS

This constant was deprecated in API level 21.
No longer enforced.

https://developer.android.com/reference/android/app/ActivityManager.html#getRecentTasks(int, int)

This method was deprecated in API level 21.
As of Build.VERSION_CODES.LOLLIPOP, this method is no longer available to third party applications: the introduction of document-centric recents means it can leak personal information to the caller. For backwards compatibility, it will still return a small subset of its data: at least the caller's own tasks (though see getAppTasks() for the correct supported way to retrieve that information), and possibly some other tasks such as home that are known to not be sensitive.

https://cs.android.com/android/platform/superproject/+/master:frameworks/base/services/core/java/com/android/server/wm/ActivityTaskManagerService.java;l=3247?q=Manifest.permission.GET_TASKS&ss=android%2Fplatform%2Fsuperproject.


How to use function in filter row

In old version of angular gantt (may 2014), filterRow worked with a function but not with current (or I don't understand how to use it).
Example : http://plnkr.co/edit/nsUbtj6DlaMwQ5qVojuX
Enter for example "kickoff" in the text input : the simple list will be filtered but not the gantt chart...

You can ignore it... api.rows.refresh() works fine...

How have you managed to use "api.rows.refresh()"?
Thanks.

@VincentLemordant Please read the docs, almost each of your question have an answer in the docs. If you still don't have the answer after reading the doc, please just read it again. And then, ask questions.


Prepare project for deployment to Maven central repository

Update the project to meet requirements of Maven central.

Use a registered group ID

Update Maven project to sign jars for executables, source, javadoc

Add deployment configuration to project file

Registered group id in OSSRH is io.fixprotocol

Java packages renamed to use reverse domain io.fixprotocol.


GameObject.DomElement still exists after scene sleep

Version

Phaser Version: 3.80.1
Operating system: Windows

Description
I am using a scene as an inventory window, so it is above another scene.
When I try to hide the scene using scene.sleep(), GameObject.DomElement still remains to be seen after scene's visibility is set to off.
I expected it to disappear, like other gameobjects like image.
Example Test Code
class Test1 extends Phaser.Scene {
constructor() { super('test1')}
create() {

scene.launch('test2');
}
}

class Test2 extends Phaser.Scene {
constructor() { super('test2')}
create() {

const text = this.add.dom(0, 0, 'p', {}, 'test');
this.scene.sleep();
// text remains in the screen.
  });
}
}

const game = new Phaser.Game({
    type: Phaser.AUTO,
    width: 800,
    height: 800,
    backgroundColor: '#111111',
    scale: {
        mode: Phaser.Scale.FIT,
        autoCenter: Phaser.Scale.CENTER_BOTH
    },
    scene: [  Test1, Test2  ]
})

Works for me: https://phaser.io/sandbox/cyBjdyjq


error when using for densenet121, for 2 output classes.

grayscale_cam = cam(input_tensor=input_tensor, targets=targets)

File "/home/melkor/projects/pytorch-grad-cam/pytorch_grad_cam/base_cam.py", line 188, in call
return self.forward(input_tensor,
File "/home/melkor/projects/pytorch-grad-cam/pytorch_grad_cam/base_cam.py", line 95, in forward
cam_per_layer = self.compute_cam_per_layer(input_tensor,
File "/home/melkor/projects/pytorch-grad-cam/pytorch_grad_cam/base_cam.py", line 127, in compute_cam_per_layer
cam = self.get_cam_image(input_tensor,
File "/home/melkor/projects/pytorch-grad-cam/pytorch_grad_cam/base_cam.py", line 50, in get_cam_image
weights = self.get_cam_weights(input_tensor,
File "/home/melkor/projects/pytorch-grad-cam/pytorch_grad_cam/grad_cam.py", line 22, in get_cam_weights
return np.mean(grads, axis=(2, 3))
File "<array_function internals>", line 180, in mean
File "/home/melkor/miniconda3/envs/graph-classificatio/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 3474, in mean
return _methods._mean(a, axis=axis, dtype=dtype,
File "/home/melkor/miniconda3/envs/graph-classificatio/lib/python3.9/site-packages/numpy/core/_methods.py", line 167, in _mean
rcount = _count_reduce_items(arr, axis, keepdims=keepdims, where=where)
File "/home/melkor/miniconda3/envs/graph-classificatio/lib/python3.9/site-packages/numpy/core/_methods.py", line 76, in _count_reduce_items
items *= arr.shape[mu.normalize_axis_index(ax, arr.ndim)]
numpy.AxisError: axis 2 is out of bounds for array of dimension 0
target_layers = [model.features[-1]] # as mentioned in example, #[BatchNorm2d(1024, eps=1e-05, momentum=0.1, affine=True, track_running_stats=True)]. densenet121
target = 0 or 1
targets = [ClassifierOutputTarget(target)]

How many output neurons does the model have in this case? Is it 1 or 2?
If 1 (representing two classes), you can use
https://github.com/jacobgil/pytorch-grad-cam/blob/master/pytorch_grad_cam/utils/model_targets.py#L26
from pytorch_grad_cam.utils.model_targets import  BinaryClassifierOutputTarget

Hi, I am getting the same error. I have 2 output neurons in my case

Hey, did anyone find a solution to this? this is roadblock for me as well.

I think u should modify your output be one-hot, then choose which class to visualize. E.g. your model output is one dimension with [0,0,1,1,0], set it to be [[1,0],[1,0],[0,1],[0,1],[1,0]].

numpy.AxisError: axis 2 is out of bounds for array of dimension 2
I meet the same question, do you solve it? thanks

How to add it to training as a loss


Explicitly add content of client/ in MANIFEST.in

Avoids extremely long package build times (and pyroma evaluation time) by explicitly avoiding ever adding node_modules to the source dist.
Previously we had these two lines in MANIFEST.in:
graft client
prune client/node_modules

which during building would first add, then remove node_modules. On some systems this can take up to a minute, and is done even during pyroma which means it happens on every commit (with pre-commit enabled).
NOTE: on my system it seems like package-lock.json is included in the dist file, which can be quite large (> 1 MB). If we are building from a clean checkout this would not happen but it's something to be aware of (or we can exclude it explicitly).
Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed
[x] No import of GPL code from MIT code

Verified that wheel and sdist still contain the same list of files, thanks!

NOTE: on my system it seems like package-lock.json is included in the dist file, which can be quite large (> 1 MB). If we are building from a clean checkout this would not happen but it's something to be aware of (or we can exclude it explicitly).

Ok, it's also included in the sdist, but as it compresses well it wasn't caught before. I'm fine with keeping it for now.


Is there a way to force truncating the text in the nav bar?

I achieved a temporary fix by changing the source code for NavBar, adding a "numberOfLines={1}" prop, along with passing in some styling into the root scene. Is there a more elegant/permanent way to do this without changing the underlying source code (since this will get erased every time I update)?
/* NavBar.js */
...
_renderTitle(childState: NavigationState, index:number) {
        return (
            <Animated.Text
                key={childState.key}
                numberOfLines={1}

We could make it configurable option, please send PR

Will look into it, thanks for the quick response :)


Do not run jobs when manual_trigger=True

This PR adds logic that when user specify manual_trigger: True in job definition, the handler will not consider this job as executable automatically, only via comment.
TODO:

[ ] Write new tests or update the old ones to cover new functionality.
[ ] Update doc-strings where appropriate.
[ ] Update or write new documentation in packit/packit.dev.
[ ] ‹fill in›

Fixes https://github.com/packit/packit-service/issues/1806
Related to https://github.com/packit/packit/pull/1984
Merge after https://github.com/packit/packit/pull/1984

RELEASE NOTES BEGIN
￼ TODO
RELEASE NOTES END

recheck

@lbarcziova done, thanks


Fix component analyzer root view

Pull Request Test Coverage Report for Build 487

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-9.9%) to 53.201%

Totals

Change from base Build 481:
-9.9%

Covered Lines:
1473

Relevant Lines:
2506

💛  - Coveralls


Add version pinning to all dependencies

Fixes #27.
I didn't always use semver, sometimes I pinned to the current minor version to be on the safe side.

Codecov Report

Merging #28 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #28   +/-   ##
========================================
  Coverage    58.91%   58.91%
========================================
  Files            9        9
  Lines         1273     1273
  Branches       119      119
========================================
  Hits           750      750
  Misses         476      476
  Partials        47       47

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 bb12232...2653109. Read the comment docs.

Handled in 63b3268.


nvidia-vaapi-driver: init at 0.0.5

Motivation for this change
depends on https://github.com/NixOS/nixpkgs/pull/159607
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build nvidia-vaapi-driver


[Enhancement] FFD emulation based on LF sound stream

I remember around 15 years there was software from company that invented ForceFeedBack, that allowed to add FFD to any game or program like movie players (yep i used and watch movies with gamepad in hand to feel FORCE FEEDBACK from LFE channel), it worked very simple - it listened program sound output, and based on frequency thresholds that you can set, but mostly low frequency bass sounds, it turned those loud basses into force feedback signals.
It worked pretty well, vibrated at sounds of most of gun fires, explosions, collisions, heart beats... and Drum'N'Bass soundtracks XD
Usually native FFD effects happens at same events that has those low freq bass sounds, so mostly it works fine.
If such feature could be added in Xidi in addition to native FFD support (which as i saw is planned), it could be used to add FFD to games that never had it. It would be very good if this would implemented with selection of input channel and frequency range to feedback on, to allow transformation ot LF 5.1\7.1 subwoofer channel into source of FFD.

That's amazing! How have I never heard of this before?

This is an interesting idea as a possible stretch goal. I hadn't heard of it before either (and I can't promise to get to this anytime soon), but it would be interesting to try for those games that do not natively support force feedback effects.


Added some words in Readme.md

Type of Change:
Readme.md changed

This PR not following CONTRIBUTING.md, please create new issue if you want to update existing code and readme.md files!


Use link_args to alias arm builtins

This is a bit of a hack, but it seems to me like the best way we have to create function aliases right now. It's up to you whether this is better than just having more function stubs.
Note that I haven't actually tested this yet as I don't have access to my arm toolchain.

arm-linux-gnueabi-gcc: error: unrecognized command line option '--defsym=__aeabi_memcpy8=__aeabi_memcpy'
Try using -Wl,--defsym=__aeabi_memcpy8=__aeabi_memcpy instead so that gcc passes the arguments to the linker.

I couldn't get this to work locally. When linking an executable (from cu), these --defsym arguments didn't appear in the linker arguments (as seen from -Z print-link-args) and I got "an undefined reference to __aeabi_memcpy4" error.
But if we can get this to work, then it seems fine to add to me. I don't know if these arguments are gcc specific but rustc only knows how to gcc as a linker, and no other linker, anyway so it doesn't matter. This change also seems easy to revert if link_args gets removed and removal shouldn't be visible by downstream users.

You need to mark these with #[cfg(not(test))].

Thanks for helping me code-by-proxy @Amanieu. I'm finally back on my real computer so I can test this myself, but I expect there aren't any changes left to make after that last one :P

@japaric I'll see if I can actually get this working locally now.

This is blocked by rust-lang/rust#36176. link_args are not propagated from rlibs to binaries.

re: link_args. Alex said:

This "feature" has basically always been slated for deprecation/removal.

So we can't really on link_args to do this :-(. Closing ...
Regardless, thanks for the PR @mattico!


Asset - More Info Button

The More Info button in heatwallet does not seem to work for user created assets, even if the required info was entered when asset was created. Link below shows More Info button for Euro Asset on testnet. See: https://dl.dropboxusercontent.com/s/npzn8klfo1u9k2v/heat-more-info.jpg

Pick this up when we integrate new design!

Now the design is:

The text "No description available ..." appears when heat-ui cannot get the resource on asset descriptionUrl. The cause is "No 'Access-Control-Allow-Origin' header is present on the requested resource." This is a CORS thing, browser does not allow cross domain request. This can be resolved by delegate the request to the server side then return response to the browser. But heatledger server is not intended for this.
I suggest instead show a link to a resource opening in new browser tab, for example:
Description provided by asset: https://en.wikipedia.org/wiki/Mongolian_tögrög


Temporary contributing guide

Motivation
Contributing guide (developer getting started guide).
Guide will evolve over the time. I wanted to provide just minimal set of instructions and general idea how SDK will be used.

@wtrocki I'll give this a spin


Karabiner causes ImageJ/Fiji plugin to freeze

When Karabiner with activated AXNotifier is used on El Capitan in conjunction with ImageJ or Fiji (http://fiji.sc), at least the ClearVolume viewer (github.com/clearvolume/clearvolume) freezes upon startup. I feel this issue is related to #259 and #255, even though the Event Viewer window tells all of Fiji's windows are already exempt:

If I disable AXNotifier, Fiji and ClearVolume work as expected.
OS X version is 10.11.1,
Karabiner version is 10.14.18,
Java JRE/JDK is 1.8.0_66
Edit: Fiji's process name is ImageJ-macosx, the bundle identifier is org.fiji.

Thank you for feedback.
I've fixed this issue at Karabiner-10.14.19.
Please upgrade Karabiner to the latest beta version.
https://pqrs.org/osx/karabiner/document.html#check-for-updates

Thanks for the quick fix, I just verified it works :)
Out of curiosity: Do you know what the root cause of these hangs is? When looking at the thread dump of our plugin - while it was hanging - I noticed an AWT thread being blocked.

Thank you for confirmation!
This issue is caused by AXObserverCreate.
https://developer.apple.com/library/mac/documentation/ApplicationServices/Reference/AXUIElement_header_reference/
I believe it's an issue of Java.
Java apps will be hang when we observe java apps using the accessibility API.

Interesting, thanks for the info! I'll close the issue now. Also thanks again for fixing it so fast :)


Removing fcntl calls to get pywal16 to work on windows

pywal originally worked on Windows to generate color schemes.
It had to be run with wal -n, which skipped setting the wallpaper, but the colorscheme generation worked fine, and you were able to theme bash, vscode, your browser, etc.
pywal16 however does not seem to work because of the fcntl calls present in one function here.
fcntl is not available on Windows, so could it be replaced with something more portable or removed altogether?
Thanks.

For what it's worth, if you install git for Windows, you get a bash.exe with a posix environment which you can use in any terminal. Ctrl-S suspends works in bash to suspend things.

So you can do have write blocked processes, files and file descriptors on windows... then simply using an if to decide if the platform supports flow control is not enough, there needs to be a way to avoid writing to write blocked file and file descriptors on windows...
Okay got it, will have to check if portalocker supports the functionality or find a library that supports the functionality and supports unices and windows.

okay, i'm not sure portalocker supports the functionality on windows to check if a file is not accepting input, or at least i'm not capable of navigating the portalocker documentation to find a function equivalent of the calls to get an io error in:
https://github.com/eylles/pywal16/blob/b9ce25b085a6d0801a48c85574e1b310f78305de/pywal/util.py#L180-L186
so the way i will proceed will be a check to see if pywal is running on a platform that supports fcntl before even importing the module.

couldn't figure out migrating the fcntl calls to portalocker so went with a try except and an if on #53

i plan to push the next release before the next month, will close this on the next version.

yeh sure, before next month... anyway, closing now since i want to have the release this week.


0.5.0 release

[ ] Local model loading * improved API/encapsulation

[x] Basic implementation: #39
[ ] Demo model introspection in some way & add local model introspection

[ ] PGVector support in embed_helper.py (will assume the user already has a managed PG instance running & all they need is to specify host/dbname/user/pass & such)

[x] contingent on finishing #33

OK we should be good to release today or tomorrow. Some work on test suite will be necessary.


Risk of undesired overwriting

We now have resuming capabilities for simulations, which is totally great, thanks @mikemhenry https://github.com/choderalab/perses/pull/830 . As far as I understand, to accomplish this for the CLI the user is now able to change/overwrite contents in a previously existing directory. What if the user doesn't realize there is a previous existing directory and runs a simulation with the same working directory name? Would it overwrite its contents without warning?

For now, the default behavior should be

check if a simulation is already there
if so, resume and finish (or extend) it

We could add more complicated checking to make sure the YAML parameters and perses version match, and adding more complex control over whether to overwrite, extend, or fail if output files are encountered but it would not be worth the effort to do this right now because we are planning to overhaul the whole top-level API and YAML structure immediately after releasing 0.9.2.


Break out plan validation logic to a common pkg

Since the MigStage and MigMigration objects share a need to validate planRefs, we should break out the Plan validation to a common package.
We should change https://github.com/fusor/mig-controller/blob/master/pkg/controller/migmigration/validation.go#L63 to take a MigPlanRef as an input and have a common validation pkg.

@dymurray This can be closed, right?

Yes thanks!


Orientations of solar radiation receiving planes

Description of the issue
Solar radiation receiving planes as solar energy converting devices or measurements can have different orientations
Ideas of solution
Horizontal plane
Suggested defintions, but still need some more deep discussion

A plane with its normal facing straigt upward
A plane that is perpendicular to the direction of gravity
Wikipedia: In astronomy, geography, and related sciences and contexts, a direction or plane passing by a given point is said to be vertical if it contains the local gravity direction at that point. [1] Conversely, a direction or plane is said to be horizontal if it is perpendicular to the vertical direction. In general, something that is vertical can be drawn from up to down (or down to up), such as the y-axis in the Cartesian coordinate system.”

Vertical Plane
Suggested defintion:
A vertical plane that is perpendicular to the horizontal plane. The slope is 90°, It needs a surface azimuth angle to be defined.
Normal Plane
Suggested defintion: A normal plane is oriented that the normal of the plane is pointing towards the sun.
Inclined Plance
Suggested defintion: An inclined plane is a plane with a defined slope and solar azimuth angle.
Comment: A typical inclined plane is a tilted rooftop.
Single Axis Tracking
Suggested Definition:
A plance that is following the path of the sun by roting along a fixed axis. Typical axis orientations are North-South or East-West.
Two Axis Tracking
A plane that is following the sun by rotating around two axes. The plane is always oriented normal.
Workflow checklist

[x] I discussed the issue with someone else than me before working on a solution
[x] I already read the latest version of the workflow for this repository
[x] The goal of this ontology is clear to me

I am aware that

[x] every entry in the ontology should have a definition
[x] classes should arise from concepts rather than from words

I think, we first need a definition for plane.

Alternative term for inclined plane: titlted plane

The plane is to be defined in #1074. These should then be sublasses to #1074.

Should we replace "plane" in all the above defintions by "solar radiation receiving surface" as suggested in #1074

@carstenhoyerklick : Any thoughts on the proposal of my comment from 17 May?

Also see OEO-DEV 46
https://etherpad.wikimedia.org/p/oeo-dev-46

Is there something from the discussion of OEO-DEV 46 that can already be implemented?

In PR #1448, we implemented the following classes:

solar tracking
single axis tracking
two axis tracking
solar tracked receiving surface
single axis tracked receiving surface
two axis tracked receiving surface

@carstenhoyerklick : Is there still something missing from this issue or can we close it?

@carstenhoyerklick : Is there still something missing from this issue or can we close it?

Reading again the issue start, all those different planes are still missing. We won't solve this in the next days, thus I move the milestone.

I think we can close this one.


Inherit relevant border-radius from the parent

Fixes #2818
Turns the :before did not inherit its parent border-radius. So the small grey is actually the square / rectangle element with its alpha channel on 0.2. I've made it inherit its parent radius, because it can also be the case that the parent doesn't have a border-radius.
Normally overflow: hidden would fix this for children, but it doesn't seem to make a difference.

Interesting Chrome bug! Toggling body { margin: 0 } from normalize.css seems to make it work.

Merged in 9f72e121592669deed1204a42051402739bd5f23 - thanks!


[FLINK-15086][client] Explictly close JobCluster in attach mode

What is the purpose of the change
If a JobCluster started in attached execution mode, it will wait for a client requesting its result and shutdown itself. However, there is no permission that a user got JobClient from executeAsync must call getJobExecutionResult so the cluster may leak.
Since currently the choice detach or attach can be done by user itself, my opinion is that we doesn't close JobCluster in MiniDispatcher in attach mode, but let the client send a shutdown request on close.
Verifying this change
Maybe an end-to-end test for resource leak? I don't have an idea to verify per-job resource leak in UT/IT framework.
Does this pull request potentially affect one of the following parts:
N/A
Documentation
N/A

Verified locally

without patch
with patch

execute + detach
cluster correctly closed
cluster correctly closed

execute + attach
cluster correctly closed
cluster correctly closed

executeAsync + detach
cluster correctly closed
cluster correctly closed

executeAsync + attach
cluster doesn't closed
cluster correctly closed

actually here we have some statements.

executeAsync + detach: cluster will close itself on job finish, so the job client might cannot query the cluster before JobClient closed. but since it somehow means attach execution because you still communicate the job with job client but config detach, it is an undefined behavior. In this case, the recommended practice would be: fire env.executeAsync and forget it.
executeAsync + attach:

resource leak if user doesn't call JobClient#close in main method. OK we document it and highlight it. Now that the ownership of the cluster move to user program, he should take care of it.
cluster shutdown before job finished. It happens if user doesn't wait for job result but close job client before job finished. The job client is the handle you talk with the job; if you release it, the job is an orphan so close it would be OK.

TBH for streaming workload which doesn't have a result anyway, it doesn't matter we just fire & forget. And close it when needed.

I think for executeAsync() we should always execute the cluster in detached mode. This has the unfortunate consequence that the cluster might go down before the user can query anything, but I think this is ok for now. In the future, we can enhance the JobClient/ClusterClient to talk directly to YARN (or the underlying execution system), this way, we can still get the job status (and some results) after a per-job YARN cluster has shut down.

OK make sense. I will add this constraint in document.
Here is the status from my side: I'm afraid I cannot provide document today but IIRC document can be checked in concurrently with or after releasing since it is not a feature.


Do not expand files, add fileglob

The first argument to utils.getFileList is now a list of already
expanded files, the second argument is a list of fileglob patterns.
Boths lists are concatenated after expansion and made unique.
This allows a user to specify a set of known files or have node webkit
builder expand it on the fly. This is particularily interesting for
grunt, which does the file expansion already.
This changes the NwBuilder interface with a backwards incompatible
manner: The constructor now takes both a files and a fileglob
array. However, what is now called fileglob was previously called
files and will NOT be expanded further.
This is one possible implementation to resolve #171

See https://github.com/nwjs/nw-builder/issues/171#issuecomment-73319969


Perfect theme -- two ideas

As simple as your theme is, it is exactly what I needed.  Thank you!  Suggestions:

implement the theme with either documentation or a function that enables quick toggling between command promps (e.g., between nothing and robbyrussell).  Right now the use of the autoloading hook function requires I quit terminal (or iTerm) to use another prompt.

have a setting to disable the pseudo line number count.  Go pure nothing!

Again, just ideas.

Thanks for the great ideas @EdmundsEcho .
I am not sure about no. 1, IMHO it seems like the quicker prompt switching is something better to be done  in a separate plug-in of some kind.
Absolute love the idea of disabling the line number count. :+1:

Great!  I agree that another plugin makes sense for the swapping idea.  Here is something I did over the weekend.  It might be a useful repository.
#!/usr/bin/env zsh

# change-zsh-theme.sh new_theme
#---------------------------------------------------------------
# TODO: verify that the new theme exists
#       use autoload -U add-zsh-hook to force cursor update
#
# Note: If a theme is replaced with one that does not exist
#       oh-my-xsh defaults to `robbyrussell`.  The requested
#       theme must already have an entry in the config
#       file.  This script only toggles commenting in/out
#       to activate a theme entry. Finally, the zshrc file
#       is copied to a backup before changes are maded file
#       if required (e.g., zshrc_bk).
#
#---------------------------------------------------------------
ZSHRC=$HOME/dotfiles/zshrc
#---------------------------------------------------------------

PROGNAME=${0##*/}
ARGS=1

main () {
  validate_input $#
  swap_themes $@
  echo "Updated zshr. Source the file to see changes."
  exit 0
}

swap_themes () {

  NEW_THEME=$1
  TEMP_FILE=/tmp/swap_themes.$$.$RANDOM

  # backup .zshrc
  cp $ZSHRC ${ZSHRC}_bk

  # Comment out the line with an active theme
  sed -E "s/^(ZSH_THEME=.+\")/# \1/" $ZSHRC > $TEMP_FILE

  # Activate the chosen theme
  sed -E "s/^# +(ZSH_THEME=\"$NEW_THEME\".*)/\1/" $TEMP_FILE > $ZSHRC

  rm -f $TEMP_FILE
}

err () {
  echo "[$(date +'%Y-%m-%dT%H:%M:%S%z')]: $@" >&2
}

validate_input () {
  if [[ $1 -lt $ARGS ]]; then
    err "Usage: $PROGNAME new-theme-name"
    exit 128
  fi
}

main "$@"
exit 0

You can start writing a plugin with it. Also sourcing the .zshrc file automatically after changing the theme would be great.


setup: exclude tests for default install

We probably don't need install tests for "normal" users and this is required to get gentoo ebuild working (package manager).
ps. You forgot to push new release/archive on github - latest is 1.2.9

Yea I need to rewrite all the tests and move them out of a requirements.txt and onto their own dev_requirements.txt, I've been focusing on all changes occurring on Cloudflare but it is on my todo
Re github release ... thanks ... I will release with when i release 1.2.15 currently in dev, theres quite a number of changes happening
Re this PR, can you do dev branch instead of the master as master is based off dev.


Add a way to specify a different favicon provider than google

Currently, the getFavicon method does not allow changing the favicon provider, which is google.com by default. This could be changed by adding a second "options" parameter, so that we could call for example:

getFavicon('https://applecom'); // Uses the default google.com/s2/favicons provider
getFavicon('https://applecom', { provider: FAVICON_KIT })

Where FAVICON_KIT would be an enumeration value, along with GOOGLE.

I want to work on this issue.

Alright @suprabhat25, here you go, good luck :slightly_smiling_face:

Currently, the getFavicon method does not allow changing the favicon provider, which is google.com by default. This could be changed by adding a second "options" parameter, so that we could call for example:
getFavicon('https://applecom'); // Uses the default google.com/s2/favicons provider
getFavicon('https://applecom', { provider: FAVICON_KIT })


Downloading configured favicon can break server on startup

The following code from server.js will run into errors similar to Error: ENOENT: no such file or directory, stat 'C:\Data\juice-shop\app\public\favicon.ico' if (and only if) a configured application.favicon is slow (or fails) to be downloaded.
/* Favicon */
let icon = 'favicon_v2.ico'
if (config.get('application.favicon')) {
  icon = config.get('application.favicon')
  if (utils.startsWith(icon, 'http')) {
    const iconPath = icon
    icon = decodeURIComponent(icon.substring(icon.lastIndexOf('/') + 1))
    utils.downloadToFile(iconPath, 'app/public/' + icon)
  }
}
app.use(favicon(path.join(__dirname, 'app/public/' + icon)))

Test cases
:heavy_check_mark: mozilla.yml reliably works fine
:x: 7ms.yml fails on startup with above error

Actually all downloads are too slow, the Mozilla theme just happens to download an image with an existing filename (favicon_v2.ico). Simplest fix is probably to just create a dummy file with the name of the to-be-downloaded file up front.


[DateTimePicker] Failed prop type: Invalid prop views[3] of value seconds supplied to ForwardRef(DateTimePicker), expected one of ["day","hours","minutes","month","year"]

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

DateTimePicker allows to work with seconds view but gives an error message if you put seconds in view:
Warning: Failed prop type: Invalid prop `views[3]` of value `seconds` supplied to `ForwardRef(DateTimePicker)`, expected one of ["day","hours","minutes","month","year"].
    at DateTimePicker (https://c2b3l.csb.app/node_modules/@mui/lab/DateTimePicker/DateTimePicker.js:23:58)
    at BasicDateTimePicker (https://c2b3l.csb.app/demo.js:38:33)
    at StyledEngineProvider (https://c2b3l.csb.app/node_modules/@mui/styled-engine/StyledEngineProvider/StyledEngineProvider.js:18:23)
Warning: Failed prop type: Invalid prop `views[3]` of value `seconds` supplied to `ForwardRef(MobileDateTimePicker)`, expected one of ["day","hours","minutes","month","year"].
    at MobileDateTimePicker (https://c2b3l.csb.app/node_modules/@mui/lab/MobileDateTimePicker/MobileDateTimePicker.js:32:76)
    at DateTimePicker (https://c2b3l.csb.app/node_modules/@mui/lab/DateTimePicker/DateTimePicker.js:23:58)
    at LocalizationProvider (https://c2b3l.csb.app/node_modules/@mui/lab/LocalizationProvider/LocalizationProvider.js:17:20)
    at BasicDateTimePicker (https://c2b3l.csb.app/demo.js:38:33)
    at StyledEngineProvider (https://c2b3l.csb.app/node_modules/@mui/styled-engine/StyledEngineProvider/StyledEngineProvider.js:18:23)
Warning: Failed prop type: Invalid prop `views[3]` of value `seconds` supplied to `ForwardRef(DesktopDateTimePicker)`, expected one of ["day","hours","minutes","month","year"].
    at DesktopDateTimePicker (https://c2b3l.csb.app/node_modules/@mui/lab/DesktopDateTimePicker/DesktopDateTimePicker.js:32:76)
    at DateTimePicker (https://c2b3l.csb.app/node_modules/@mui/lab/DateTimePicker/DateTimePicker.js:23:58)
    at LocalizationProvider (https://c2b3l.csb.app/node_modules/@mui/lab/LocalizationProvider/LocalizationProvider.js:17:20)
    at BasicDateTimePicker (https://c2b3l.csb.app/demo.js:38:33)
    at StyledEngineProvider (https://c2b3l.csb.app/node_modules/@mui/styled-engine/StyledEngineProvider/StyledEngineProvider.js:18:23)

Expected Behavior 🤔
No prop type error if there is seconds in views prop.
Steps to Reproduce 🕹
Quick code example:
<DateTimePicker
	renderInput={(props) => <TextField {...props} />}
	label="DateTimePicker"
	value={value}
	onChange={(newValue) => {
	  setValue(newValue);
	}}
	inputFormat="yyyy-MM-dd hh:mm:ss"
	views={["day", "hours", "minutes", "seconds"]}
  />

Full code example: https://codesandbox.io/s/basicdatetimepicker-material-demo-forked-c2b3l?file=/demo.js
I found a workaround but it's not very good - i'm hiding the error inside console.error:
import { useEffect } from 'react';
import AdapterMoment from '@material-ui/lab/AdapterMoment';
import LocalizationProvider from '@material-ui/lab/LocalizationProvider';
import DateTimePickerOrigin from '@material-ui/lab/DateTimePicker';

// It's workaround to force DateTimePicker display `seconds` view
// If error is a proptype error and includes the following string: `Invalid prop `views[3]` of value `seconds` supplied to`
// suppress the error and don't show it
// if it is not a proptype error, we show it

// eslint-disable-next-line no-console
const consoleErrorOrigin = console.error.bind(console.error);
// eslint-disable-next-line no-console
console.error = (message, ...params) => {
  const suppressedErrors = (params || []).length > 1 && params[1].toString().includes('Invalid prop `views[3]` of value `seconds` supplied to');
  if (!suppressedErrors) {
    consoleErrorOrigin(message, ...params);
  }
};

/**
 * DateTimePicker component
 */
const MyDateTimePicker = (props) => {
...

  // This is a part of workaround to force DateTimePicker display `seconds` view
  useEffect(
    () => () => {
      // eslint-disable-next-line no-console
      console.error = consoleErrorOrigin;
    },
    [],
  );

  return (
    <LocalizationProvider dateAdapter={AdapterMoment}>
      <DateTimePicker
      ...

Context 🔦

Your Environment 🌎

`npx @mui/envinfo`
  System:
    OS: Windows 10 10.0.19042
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 3.81 GB / 15.82 GB
  Binaries:
    Node: 12.22.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - C:\Program Files\nodejs\yarn.CMD
    npm: 6.14.12 - C:\Program Files\nodejs\npm.CMD
  Managers:
    pip3: 21.0.1 - C:\Python39\Scripts\pip3.EXE
  Utilities:
    Git: 2.33.0. - /mingw64/bin/git
  Virtualization:
    Docker: 20.10.8 - C:\Program Files\Docker\Docker\resources\bin\docker.EXE
  Languages:
    Bash: 4.4.23 - C:\Program Files\Git\usr\bin\bash.EXE
    Perl: 5.34.0 - C:\Program Files\Git\usr\bin\perl.EXE
    Python: 3.9.2 - /c/Python39/python
  Browsers:
    Chrome: 92.0.4515.159
    Edge: Spartan (44.19041.1023.0), Chromium (93.0.961.38)

views: PropTypes.arrayOf(
    PropTypes.oneOf(['day', 'hours', 'minutes', 'month', 'year']).isRequired,
),

views does not have option for second, i think this could be feature request.

@khalid283 yes, maybe i chose the wrong issue type: i thought that it's a propType bug because component supports seconmds view and only propType doesn't.

Fixed in  https://github.com/mui-org/material-ui/pull/25095 which is available in<EMAIL_ADDRESS>


OpenCV Error: Assertion failed (ssize.width > 0 && ssize.height > 0) in resize

Ranadeep Biswas I have made a new python3 installation. The installation of pip3 install -r requirements. txt went through without errors. I also created the dual_points. json file on the html page.
The first stitch with "python3 /fishpypano. py -i /Volumes/500gb/i. JPG -p dual_point. json" worked, but didn't look good.
Then I created the calibrated_data. json with the -c option. Now I get the following error code:
macbook-pro:FishPyPano-master omek$ python3 ./fishpypano.py -i /Volumes/500gb/i.JPG -p dual_point.json OpenCV Error: Assertion failed (ssize.width > 0 && ssize.height > 0) in resize, file /Users/travis/build/skvark/opencv-python/opencv/modules/imgproc/src/resize.cpp, line 4044 Traceback (most recent call last): File "./fishpypano.py", line 514, in <module> img0, img1 = dual.warp_img() File "./fishpypano.py", line 401, in warp_img img0 = self.crop_fisheye_img(self.dual_img, self.cir0_c) File "./fishpypano.py", line 309, in crop_fisheye_img (self.fisheye_size, self.fisheye_size)) cv2.error: /Users/travis/build/skvark/opencv-python/opencv/modules/imgproc/src/resize.cpp:4044: error: (-215) ssize.width > 0 && ssize.height > 0 in function resize

any news?

This issue is fixed. But there are other issues in the stitching engine. I may need to rewrite some parts. So I have not pushed the fix yet. I will announce when I will push the reworked code.

Ok, i hope the best.

the calibration won't work:
macbook-pro:FishPyPano-master omek$ python3 calibrate.py -p  i.JPG Traceback (most recent call last):   File "calibrate.py", line 39, in <module>     points = np.array(json.load(jh))   File "/usr/local/Cellar/python3/3.6.4_2/Frameworks/Python.framework/Versions/3.6/lib/python3.6/json/__init__.py", line 296, in load     return loads(fp.read(),   File "/usr/local/Cellar/python3/3.6.4_2/Frameworks/Python.framework/Versions/3.6/lib/python3.6/codecs.py", line 321, in decode     (result, consumed) = self._buffer_decode(data, self.errors, final) UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 0: invalid start byte

you were supposed to pass the JSON file containing matched points. use the HTML page to generate that file.

macbook-pro:FishPyPano-master omek$ python3 stitch.py  -c calib_data.json -i a.jpg
Loaded calibration data..
Projecting fisheye images to equirectangular projection..
Projecting left fisheye image..
Projecting right fisheye image..
Done projecting, starting stitching..
Calculating optical flow and Warping..
OpenCV Error: Assertion failed (dims <= 2 && step[0] > 0) in locateROI, file /Users/travis/build/skvark/opencv-python/opencv/modules/core/src/matrix.cpp, line 991
Traceback (most recent call last):
  File "stitch.py", line 145, in <module>
    warped_overlap1, warped_overlap2 = util.flow_warp(equi1_overlap, equi2_overlap)
  File "/Users/omek/Downloads/FishPyPano-master/util/flow.py", line 32, in flow_warp
    bot_frame_ = _opt_flow_warp(top_frame, bot_frame, False)
  File "/Users/omek/Downloads/FishPyPano-master/util/flow.py", line 13, in _opt_flow_warp
    img1_gray, img2_gray, None, 0.5, 3, 15, 3, 5, 1.2, 0)
cv2.error: /Users/travis/build/skvark/opencv-python/opencv/modules/core/src/matrix.cpp:991: error: (-215) dims <= 2 && step[0] > 0 in function locateROI

can you share the calib_data.json and a.jpg with me?

calib_and_dual.zip

I see it didn't calibrate properly. can you share your matched point file too?

I have uploaded both files as .zip at the github issue

Am 26.02.2018 um 19:50 schrieb Ranadeep Biswas<EMAIL_ADDRESS>I see it didn't calibrate properly. can you share your matched point file too?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Sorry. Didn't see that. Anyway, I looked into the points, I see the points are not correctly chosen. I will upload a video illustrating how to choose the points. If the points are okay, it should work fine.

Ok, i will try it again tomorrow. Thank you for the help

Am 26.02.2018 um 20:02 schrieb Ranadeep Biswas<EMAIL_ADDRESS>Sorry. Didn't see that. Anyway, I looked into the points, I see the points are not correctly chosen. I will upload a video illustrating how to choose the points. If the points are okay, it should work fine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@rnbdev Can you upload the video to show how to choose the points for calibration?


feat: add apigw auth config

:tada: This PR is included in version 1.0.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


What about docs?

Hey there - I wonder what we should do about documention on this project @RichBiker


Investigate ML image recognition

Function implemented on related branch. Feature won't be merged into production app for reasons documented in related PR.


[Xcode10] ValidateRequiredFields Unknown selected data source for Port Speaker

Hi,
After upgrading xcode version, there are following log message every seconds.
 2018-09-20 11:58:09.622656+0400 MyApp[1027:137400] [avas] AVAudioSessionPortImpl.mm:56:ValidateRequiredFields: Unknown selected data source for Port Speaker (type: Speaker)

Does any one know the solutions? Thanks

@ya-watanabe Getting the same issue after upgrading xcode as well BUT it is only happening on iPhone XS models with iOS 12, not on any previous iPhone with older iOS versions. Not entirely sure if it is hardware induced (stereo speaker setup on XS models, perhaps?) or iOS 12 software level requirements (introduced recently?).

I have the same log message, and it is after I've added AdMob to my app.


Proj6

Not sure this will work, if it does I'll create a branch and change the target.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.


Continue to support Security Context mocked using Spring Security Test

The change in #360 breaks a test in my project where I am using SecurityMockMvcRequestPostProcessors from Spring Security Test 4.0.0.M2 by Rob Winch to mock the Security Context in Tests of a REST API based on MockMvc.
see http://spring.io/blog/2014/05/23/preview-spring-security-test-web-security#populating-a-test-user-with-a-requestpostprocessor
The mocked Security Context is naturally not based on an OAuth2 Access Token and the Authorization Header is not present in the request. Rather the Security Context it is just installed within the test runtime using the new MockMvc with(user(...)) and with(authentication(...)) patterns.
Therefore the code introduced by c5d6f1f... and 3e015a9... clears any non-anonymous Security Context populated via Spring Security Test and leads to a 401.
I think setting stateless to false is not a good way to solve this in tests.

Options just for consideration, I am trying to find some ideas how to handle this:

Securely recognize a mocked Authentication and if present, do not clear the Security Context.
Move relevant framework code in OAuth2AuthenticationProcessingFilter to a Spring Bean that can be replaced in tests using @Primary and the like.
Introduce another configuration property relevant to the decision to clear or not, which can easily be set from outside.
Somehow support a SecurityContextHolder for tests that can be programmed to not clear the Security Context ever (surely not the best thing to do and very bold...).
Provide a way to create and use a kind of mocked (forged?) Access Token in tests.

Some of these options could be difficult and/or introduce unwanted security risks.
Perhaps there is already a way to go, please let me know.

Isn't the best option simply to set the "stateless" flag to false (either for the whole app or just for the tests)? The reason it's configurable is so you can choose the behaviour you want.

Hi,
I'm referring code on https://github.com/spring-projects/spring-security-oauth, I see there is a problem in RestOperation API when trying to attempt to make request to Facebook where causing an exception.. Could you please check at your end too and try to fixed it ASAP?

What would be the easiest way to configure stateless = false in a test (not for the whole app)?
I mean, normally you do not configure OAuth2AuthenticationProcessingFilter yourself, the configuration is done within the framework via ResourceServerConfigurerAdapter. The flag also does not seem to be a Spring Boot environment parameter out of the box etc.

In a test you can just add a ResourceServerSecurityConfigurer to your application context and set the flag to false. I don't know if that's the best solution (cc @rwinch) but it certainly should work.
It's good that you have Spring Boot in mind, but remember this project does not depend on Spring Boot (and Spring Boot doesn't yet have any OAuth integration).

I ended up doing the following.
In my existing production @Configuration class that is a ResourceServerConfigurerAdapter I added:
    @Autowired(required = false)
    @Qualifier("springSecurityOAuth2StatelessSecurityContext")
    private Boolean stateless = Boolean.TRUE;

This Boolean member variable is used to set the flag via the configurer in the same file:
    public void configure(ResourceServerSecurityConfigurer resources) {
        resources
                .stateless(stateless)
// ...

In the test sources I added a configuration with the following Bean (which could also be restricted to a particular Spring Profile if necessary):
    @Bean
    @Primary
    public Boolean springSecurityOAuth2StatelessSecurityContext() {
        return Boolean.FALSE;
    }

This works for me and is an acceptable change in the production code while allowing the IntegrationTest classes based on MockMvc to test with the production OAuth2 configuration while using Spring Security Test in some of the tests (which is a mixture that I admit is not necesarily mainstream, but in my case desired).

I'm not sure I have a better way around it at the moment. Dave and I will need to look into adding testing support for Spring Security OAuth in the future.

Just to be more clear on my earlier query - Spring OAuth2 is not working properly as Client (don't any working find code) in order to connect to the Social Networks (like Google, Facebook, Linkedin etc,) or Cloud API's (like Misfit etc..) as follow proper OAuth2 mechanism, But on other hand, I see Apache Oltu (earlier Apache Amber) is useful enough to work as a OAuth2 client and its working fine in my sample example. Why such implementation can't be developed using Spring OAuth2 or please someone look at this following code which is causing problem?

Just to be more clear on my earlier query - Spring OAuth2 is not working properly as Client (don't any working find code) in order to connect to the Social Networks (like Google, Facebook, Linkedin etc,) or Cloud API's (like Misfit etc..) as follow proper OAuth2 mechanism, But on other hand, I see Apache Oltu (earlier Apache Amber) is useful enough to work as a OAuth2 client and its working fine in my sample example. Why such implementation can't be developed using Spring OAuth2 or please someone look at this following code which is causing problem?

Just to be more clear on my earlier query - Spring OAuth2 is not working properly as Client (don't any working find code) in order to connect to the Social Networks (like Google, Facebook, Linkedin etc,) or Cloud API's (like Misfit etc..) as follow proper OAuth2 mechanism, But on other hand, I see Apache Oltu (earlier Apache Amber) is useful enough to work as a OAuth2 client and its working fine in my sample example. Why such implementation can't be developed using Spring OAuth2 or please someone look at this following code which is causing problem?

Just to be more clear on my earlier query - Spring OAuth2 is not working properly as Client (don't any working find code) in order to connect to the Social Networks (like Google, Facebook, Linkedin etc,) or Cloud API's (like Misfit etc..) as follow proper OAuth2 mechanism, But on other hand, I see Apache Oltu (earlier Apache Amber) is useful enough to work as a OAuth2 client and its working fine in my sample example. Why such implementation can't be developed using Spring OAuth2 or please someone look at this following code which is causing problem?

@pashtika if you have a specific problem please open a separate issue (this one is about something different). Please look at the existing samples (there are plenty on the client side, and one that definitely uses Facebook successfully), and be very specific about what the problem is. And please don't duplicate posts.

OK. Could you please provide client links (if you're aware) ? As I can't find anything working OAuth2 client example? I appreciate your help on this.
Multiple post was by mistakenly happened due to network issue at my end. I've deleted those in order to remove duplicates.Thanks.

Hi, I encounter issue testing with @WithMockUser and spring security/security-oauth2.
Using @sourcekick snippet, I suddently swap my 401 error to a 403 error.
The endpoint I am trying to access is a endpoints not protected by the OAuth filter but only security.
i see few information about people testing both framework together, I don't really understand what is the snippet for. Can this help ?

The recommendation provided at https://github.com/spring-projects/spring-security-oauth/issues/385#issuecomment-73051286 does work for tests, but a better way to do this without modifying any production code is the following:

Create a new ResourceServerConfigurerAdapter for your tests.

public class AuthStatefulSecurityConfig extends ResourceServerConfigurerAdapter {

    @Override
    public void configure(ResourceServerSecurityConfigurer resources) {
        resources.stateless(false);
    }
}

Include this in your tests with either of the following
a. explicitly in your test class with @Import(AuthStatefulSecurityConfig.class)
b. dynamically by including in spring.factories:

org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc=\
  com.broadleafcommerce.auth.AuthStatefulSecurityConfig


Fixes bug where leak_info was not being set in evaluate_final_action.py

First of all, thank you very much for the interesting paper, dataset and well written code.
I am currently reproducing the results and noticed this bug after having leakage rate of 0 even though secrets were being leaked. The bug was that in the judge_leakage step, leak_info was never being set to true in case of a leakage of any secret, and thus leakage rate was always 0.
This pull request fixes this bug.
Thank you again for your work.
Kind regards

Thank you so much for your interest! Yes, the bug is introduced when we clean up the code for release. Sorry for the trouble and thanks for fixing it.


Add more ignore and skip tests to the list

Based on the initial Salt Shaker runs, these are some tests we need to ignore to prevent errors and also skip to avoid hanging executions.

Thanks for the feedback @agraul. I've removed the ones that are not longer needed after the changes we are introducing here: https://github.com/openSUSE/salt/pull/616


Fix PM mode auth setting when using embedded OIDC

Also: uptick to next release version

any chance to add a readme command to copy/paste and run ?


assh config build drops extra IdentityFile parameters.

IdentityFile: /Users/2/.ssh/2/id_rsa
Here is a snippet of my assh.yml file.
defaults:
    ControlMaster: auto
    ControlPath: ~/tmp/.ssh/%h-%p-%r.sock
    ControlPersist: yes
    IdentityFile: /Users/1/.ssh/1/id_rsa
    IdentityFile: /Users/2/.ssh/2/id_rsa
    IdentityFile: /Users/3/.ssh/3/id_rsa
    UseKeychain: yes
    Port: 22
    StrictHostkeychecking: no
    Forwardx11: no
    ForwardAgent: yes
    Protocol: 2
    ServerAliveInterval: 5

When I assh config build it only uses 1 IdentityFile. I add them back in manually, so it's not a huge deal, and I don't know if this is operating as expected or if this is a bug. You can support multiple IdentityFile(s) in a config.
Host *
  ControlMaster auto
  ControlPath ~/tmp/.ssh/%h-%p-%r.sock
  ControlPersist yes
  ForwardAgent yes
  ForwardX11 no
  IdentityFile /Users/1/.ssh/1/id_rsa
  Port 22
  Protocol 2
  ServerAliveInterval 5
  StrictHostKeyChecking no
  UseKeychain yes
  ProxyCommand assh connect --port=%p %h

Hi @bdmorin,
This is due to a limitation in the YAML protocol which only supports one unique entry per key, but it's easily bypassable by replace, this:
    IdentityFile: /Users/1/.ssh/1/id_rsa
    IdentityFile: /Users/2/.ssh/2/id_rsa
    IdentityFile: /Users/3/.ssh/3/id_rsa

to this:
    IdentityFile:
      - /Users/1/.ssh/1/id_rsa
      - /Users/2/.ssh/2/id_rsa
      - /Users/3/.ssh/3/id_rsa

Example here:

assh.yml
generated ssh_config

Thank you for your feedback

Thank you! I should have figured that out myself.


instantiated error types not being generated/outputted

type ErrNotFound error

var (
	ErrFoodNotFound            ErrNotFound = errors.New("food not found")
	ErrCompanyNotFound           ErrNotFound = errors.New("company not found")
	ErrUserNotFound            ErrNotFound = errors.New("user not found")
)

output:
export type ErrNotFound = error;

I am trying to use this common pattern in go, but none of the instantiated errors are being generated
Is there a work around for this @gzuidhof

Hmm there is no obvious way to do it, it's hard for Tygo to have some generic way of dealing with instantiated variables.
I would propose that the most straightforward way is to add some frontmatter manually, you can define frontmatter in the yaml file and it will be put into the output file.
It's not often I see raw error being serialized into Typescript/JSON, I don't even know what it looks like. I guess in the end it becomes a string? In that case, you could use something like the following as frontmatter:
export type error = string;

export const ErrFoodNotFound: error = "food not found";
export const ErrCompanyNotFound: error = "company not found";
export const ErrUserNotFound: error = "user not found";

Of course it's less than ideal to do it manually, but perhaps for your usecase it's fine?

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error
https://go.dev/blog/error-handling-and-go
In golang an error satisifes this interface
type error interface {
    Error() string
}

I was hoping I could at least create a struct called Error with that interface and assign it. But if I can't use var I will have the same issue.

I found a good Go article for anyone interested in this issue:
https://dave.cheney.net/tag/errors


Update to docker files

I tried to run as a docker image, but this failed with an error saying "unable to find config file".
This seems to be because "run.sh" executes in "/" (and so does "codecept.js"), but the config file is in "/tests/".  I've made a couple of small changes (to the Dockerfile and docker/run.sh) to cd to the /tests/ directory and also to allow a CODECEPT_ARGS rather than GREP env var (I'm assuming the docker file is based on an earlier codecept.js syntax).
Happy to discuss.

Yes. Thanks for the update

Ah, this has me pulling my hair. My first time trying out docker images, and could not figure out what was wrong. (I downloaded the docker image this morning :-)
Its working now.


Make it work

Hey @ArtSuslov 👋
Without any comment or description I can only make assumptions about the intent of this PR.
I'm always open to improving usability of the docker containers, etc... but there are a bunch of changes here that I don't see helping the primary users, Graph network indexers.

we will not be relying on '@tokene-q/common-ts' instead of @graphrotocol/common-ts, are there differences there that you require?
we are hoping to upgrade our docker images to use a newer version of the Node runtime.  Version 14 is quite old now and even the version we are using, 16, is no longer technically supported.

I like that you included some instructions for interacting directly with the smart contracts without relying on a browser app.

It's been almost a year with no response or activity from the submitter. I'm closing this.


feat: add additionalvarbinds to enricher

Codecov Report

Merging #126 (439d433) into develop (a003fbd) will increase coverage by 0.59%.
The diff coverage is 70.65%.

:exclamation: Current head 439d433 differs from pull request most recent head 972f572. Consider uploading reports for the commit 972f572 to get more accurate results

@@             Coverage Diff             @@
##           develop     #126      +/-   ##
===========================================
+ Coverage    59.53%   60.13%   +0.59%
===========================================
  Files           33       33
  Lines         1389     1440      +51
===========================================
+ Hits           827      866      +39
- Misses         562      574      +12

Impacted Files
Coverage Δ

splunk_connect_for_snmp_poller/manager/tasks.py
36.98% <ø> (ø)

tests/test_hec_sender.py
100.00% <ø> (ø)

splunk_connect_for_snmp_poller/mongo.py
33.89% <15.38%> (-1.96%)
:arrow_down:

..._connect_for_snmp_poller/manager/task_utilities.py
22.69% <22.22%> (+0.21%)
:arrow_up:

...poller/manager/static/interface_mib_utililities.py
75.67% <35.71%> (-24.33%)
:arrow_down:

...lunk_connect_for_snmp_poller/manager/hec_sender.py
33.76% <100.00%> (+0.87%)
:arrow_up:

...ect_for_snmp_poller/manager/static/mib_enricher.py
100.00% <100.00%> (+5.88%)
:arrow_up:

tests/test_config_input_data.py
100.00% <100.00%> (ø)

tests/test_mib_enricher.py
100.00% <100.00%> (ø)

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 a003fbd...972f572. Read the comment docs.

:tada: This PR is included in version 1.5.0-develop.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add ability to add tags to Asana tasks

Tested in my Windows PR that resulted in https://app.asana.com/0/69071770703008/1208615375126836/f.

Tested and works as expected. Thanks @nshuba ! We’ll need to make a new release so this is available to you without pointing to the main branch.


Rework documentation

Need to rework docs surrounding variants to make sure it matches the variants file

Duplicate of #201


STAR-1112 Add QueryInterceptor to QueryProcessor

Do we actually need this change in Cassandra itself?  Given we can have our own QueryProcessor in CNDB can we not just add this there?

@JeremiahDJordan We do need this because the schema change messages do not go through Stargate so we can't use the Stargate QueryInterceptor.