Check return codes in Java agent

We resolved an issue with Java in Jenkins that resulted from not checking the return code from a JNI call. There are numerous places where we don't check return codes. Probably should look into this.

This was resolved with #800.  Closing because no further action is needed.


setup install, get error bad marshal data (unknown type code)

Ubuntu with python3.8.9.
➜  binwalk-2.3.3 python3 setup.py install  --user
/usr/lib/python3.8/distutils/dist.py:274: UserWarning: Unknown distribution option: 'long_description_content_type'
  warnings.warn(msg)
/usr/lib/python3.8/distutils/dist.py:274: UserWarning: Unknown distribution option: 'python_requires'
  warnings.warn(msg)
running install
running bdist_egg
running egg_info
writing src/binwalk.egg-info/PKG-INFO
writing dependency_links to src/binwalk.egg-info/dependency_links.txt
writing top-level names to src/binwalk.egg-info/top_level.txt
reading manifest file 'src/binwalk.egg-info/SOURCES.txt'
writing manifest file 'src/binwalk.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
installing scripts to build/bdist.linux-x86_64/egg/EGG-INFO/scripts
running install_scripts
running build_scripts
changing mode of build/bdist.linux-x86_64/egg/EGG-INFO/scripts/binwalk to 775
copying src/binwalk.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying src/binwalk.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying src/binwalk.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying src/binwalk.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
zip_safe flag not set; analyzing archive contents...
Traceback (most recent call last):
  File "setup.py", line 330, in <module>
    setup(
  File "/usr/lib/python3.8/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/usr/lib/python3.8/distutils/dist.py", line 966, in run_commands
    self.run_command(cmd)
  File "/usr/lib/python3.8/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "/usr/lib/python3/dist-packages/setuptools/command/install.py", line 67, in run
    self.do_egg_install()
  File "/usr/lib/python3/dist-packages/setuptools/command/install.py", line 109, in do_egg_install
    self.run_command('bdist_egg')
  File "/usr/lib/python3.8/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/lib/python3.8/distutils/dist.py", line 985, in run_command
    cmd_obj.run()
  File "/usr/lib/python3/dist-packages/setuptools/command/bdist_egg.py", line 209, in run
    os.path.join(archive_root, 'EGG-INFO'), self.zip_safe()
  File "/usr/lib/python3/dist-packages/setuptools/command/bdist_egg.py", line 245, in zip_safe
    return analyze_egg(self.bdist_dir, self.stubs)
  File "/usr/lib/python3/dist-packages/setuptools/command/bdist_egg.py", line 355, in analyze_egg
    safe = scan_module(egg_dir, base, name, stubs) and safe
  File "/usr/lib/python3/dist-packages/setuptools/command/bdist_egg.py", line 392, in scan_module
    code = marshal.load(f)
ValueError: bad marshal data (unknown type code)

solved: upgrade  setuptools


OTP generator return 5 digits instead of 6

Sometimes generateOneTimePassword returns 5 digits instead of 6. Not sure how to reproduce this

generateOneTimePassword returns an int, and an int doesn't have any built-in notion of "number of digits." If you were to do this:
final int example = 12;
System.out.println(example);

…then you'd expect the output to be "12" and not "000012," right? The same thing is true with one-time passwords. When you generate a (six-digit) one-time password, you could wind up with a value anywhere between 0 and 999,999. If that value is 12, System.out.println(oneTimePassword) will just print "12."
What you need to do is:
System.out.format("%06d\n", oneTimePassword);

…or:
final String oneTimePasswordString = String.format("%06d", oneTimePassword);


Support TargetFramework of netstandard2.0 and version bump to 5.0.4.

Update: Support the netstandard2.0 target framework (in addition to the existing ones)
Description
I have a project where I need to use this library from .NET Framework for backward compatibility purposes. By conditionally referencing the System.Text.Json package, it's possible to build the entire solution to target netstandard2.0 and therefore be consumed from .NET Framework.
Motivation and Context
The netstandard2.0 target framework allows this package to be used by additional .NET runtimes.  In my case, I need to run my solution under .NET Framework for legacy reasons.
The only issue was the System.Text.Json package which isn't included in the framework for netstandard2.0, but there is a Microsoft-supplied external package that can be used to support netstandard2.0.  By conditionally including this PackageReference it only affects the netstandard2.0 target (keeping the other targets using their internally included packages).
Testing
Successfully ran the Amazon.Extensions.Configuration.SystemsManager.Tests unit test suite.
Screenshots (if appropriate)
NA
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have read the README document
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

License

[x] I confirm that this pull request can be released under the Apache 2 license

So you are using the Microsoft.Extensions.Configuration configuration framework in a .NET Framework application?

So you are using the Microsoft.Extensions.Configuration configuration framework in a .NET Framework application?

Yeah.  Long story, but we need to use some of the old capabilities of .NET Framework libraries that have been (rightly) removed in their .NET Core re-incarnation.  While being primarily a runtime, there are some Windows-specific capabilities that can only be accessed using it.
However, because .NET Framework 4.8 supports netstandard2.0 we're able to use all the newer ecosystem to build the application itself (e.g. Microsoft.Extensions.Configuration, Kestrel, WebHost, etc).  Makes it a little less jarring.

So you are using the Microsoft.Extensions.Configuration configuration framework in a .NET Framework application?

Yeah. Long story, but we need to use some of the old capabilities of .NET Framework libraries that have been (rightly) removed in their .NET Core re-incarnation. While being primarily a runtime, there are some Windows-specific capabilities that can only be accessed using it.
However, because .NET Framework 4.8 supports netstandard2.0 we're able to use all the newer ecosystem to build the application itself (e.g. Microsoft.Extensions.Configuration, Kestrel, WebHost, etc). Makes it a little less jarring.

Sounds like an interesting scenario. I'm fine adding the .NET Standard 2.0 support to unblock you. I just had a couple minor comments and then we can get this released.

Great, thanks for that.


UI development for #SB-6013 (Header Redesign)

@lakhanmandloi

@harishgilimi This pr is good to merge


Community Patch Bug

https://imgur.com/gallery/7g31mF0
Let’s me load into my saved game if I turn community patch off on launcher
(Playing on 1.3 beta)

I got the same thing when trying to load my save just now. I've been using 0.1.0-rc2 since the start of my campaign, was able to play and save and load just fine, however when closing the game coming back after a while and trying to load I got the same crash.
Interestingly, a workaround is to start a new campaign and immediately click back (before selecting a cultural background) and then load the save. This will let me load the save and play.

Interestingly, a workaround is to start a new campaign and immediately click back (before selecting a cultural background) and then load the save. This will let me load the save and play.

This works XD

Genius it works

Yeah some of our patches account for this, some don't - had to do w/ the way the campaign initializes static data when starting a new one vs. loading a game. We'll fix this.

I got the same thing when trying to load my save just now. I've been using 0.1.0-rc2 since the start of my campaign, was able to play and save and load just fine, however when closing the game coming back after a while and trying to load I got the same crash.
Interestingly, a workaround is to start a new campaign and immediately click back (before selecting a cultural background) and then load the save. This will let me load the save and play.

Oh god this really works


Testing

Test all features. To avoid side-effects, each feature should be contained in its own test file
Changes

[x] creation
[x] singleton
[x] disable
[x] restore
[x] capture
[x] flush
[x] compat

Added in v.1.0.2


question about conf_thresh

How much did you get conf_thresh when validating the test set? 0.18 or 0.1？
This threshold has an influence on the detection results, so I would like to ask what threshold is usually used in your own experiments and the comparative tests you do.

I used 0.1


Fix content.integration tests warnings

About the PR

Fixes most of the warnings in the Content.IntegrationTests module.
Does not fix CreateGrid related warnings as that would require engine work.

Media

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

Should be good to go. About half of the lines added are two new mispredict reconciliation tests. They're for the source gen and ECS based versions.


chore: Add coverage

Reports are here (for PR's), need to merge it to have base coverage

:tada: This PR is included in version 1.4.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Strange inference with a ternary operator and Lists

On the following code:
void fn(Iterable iter) {}

void main() {
  bool condition = true;
  Iterable<String> iter = [];
  fn(condition ? [] : iter.toList());
}

The analyzer reports the following warning:
[warning] The argument type 'EfficientLength' cannot be assigned to the parameter type 'Iterable'

This seems wrong. The first branch of the ternary operator is a List, which is assignable to Iterable.
Note that if <String> is removed from the type annotation for iter, or if <String> is added to the list literal, the warning goes away.
I'm using 1.13.0-edge.9e18a7519d429f87c70bb3b8871c6cc666ee3fa6.

This looks like an instance of issue 19425.

Yes, sadly, unless we get a different definition of least upper bounds we're stuck. The current specification of LUB says that LUB(List, List) is EfficientLength. The answer that everyone expects is List, so it surprises lots of people fairly frequently.
To work around this, you either have to write "[]" for the literal, or stop using the conditional operator (the only place where LUB is used in the spec.).


feat(datepicker): dispatch events when datepicker is opened and closed

Adds the opened and closed events that are dispatched when a datepicker is opened/closed.
Fixes #7626.

Hello this PR solve problem in my project. Thank you, look forward merge it.

Is there some time, this merge is going to be released?

This was included in the 5.0.0-rc1 release


Adding metrics to the metric model mindmap

The metrics-model-libs/libs-mindmap is a great starting point to navigate metrics.
From my perspective, mindmap enables related metrics to come into plain view. Thus, I suggest adding
metrics that are included in each metric model.

Markmap supports generating online mindmaps based on markdown files, and it also supports displaying hyperlinks, so it's another solution I'm suggesting. This could be an ongoing discussion.

@xiaoya-yaya I have add mindmap https://github.com/chaoss/chaoss.github.io/pull/3 to chaoss io page.
Similarly like https://eyehwan.github.io/wg-metrics-models/libs-mindmap/mindmap

Is this done? If so, could the issue be closed?

Yes, I think we can close it.


Transforming into a Pokemon gives you info on their ability that you shouldn't have

This is a bug. Using transform causes some issues with the tracker, one major and one minor, detailed below.
The major issue
Immediately after using Transform, your Pokemon's ability on the tracker changes to the targeted Pokemon's ability, giving you information you shouldn't be able to access (you can't see it by viewing the Pokemon's summary in-game). Looking at the reference image below, Poliwhirl has the ability Shed Skin, while Charmeleon has Rough Skin.
Is this similar to #30 ?
The minor issue
After transformation, the tracker continues to show your original Pokemon's moveset and stats. I image not revealing the stats is a good thing. The moveset not reflecting the Pokemon it became is only a small concern; or maybe not one at all. It's not really important to track this data necessarily. However, many people find it useful to know what move is effective against the opposing Pokemon. There may be other reasons to temporarily see the transformed moves show in the tracker.
Reference image

In regards to the ability, i'm pretty sure the cause is the same as both #30 and #13 showing the opponent's abilities
Since the ability is being read from sBattlerAbilities, which is the in-battle ability data, my guess is when the player's ability is changed due to transform / skill swap / trace and probably similar things like role play the game changes the ability ID stored in sBattlerAbilities to the ability that was copied/swapped over to the player
So if this is the case, then the most optimal solution would be to find a better memory address that permanently stores the pokemon's original ability, ideally one that also stores the ability outside of battle too which would also resolve #1. Whether this exists or not I don't know
I've tried to have a look for this but i don't know much regarding pulling data from game memory or what memory addresses to try so not found anything too promising yet but the decomp code has a big list of them if anyone else wants to try looking (for FRLG they're here - https://github.com/pret/pokefirered/tree/symbols)

That's a good point. I suppose one of the cleanest ways to handle this is to treat being transformed as being in a special state, a state such that the game doesn't reveal data on the tracker about yourself. Similar to how we have being in a state of "Must Check Summary" to hide data from the player.

yeah possibly, there is a boolean variable I introduced (Program.transformedPokemon.isTransformed) to fix the issue where the tracker will overwrite a pokemon's tracked moves with your moves if they transformed (PR #61).
This could maybe be used to also hide copied abilities for the player when the player is transformed. Wouldn't need to worry about a scenario where both pokemon are transformed since in gen 3 transform fails when used on a transformed pokemon, so only 1 can be transformed at a time.


Split k8s and az actions

Which issue this PR addresses:
Fixes https://github.com/Azure/ARO-RP/issues/1061
What this PR does / why we need it:
Currently, all admin actions only work if k8s clients can be created, even if they don't require the use of k8s clients. In clusters that failed to be created, this may not be the case. If they don't have a value for PrivateEndpointIP, restconfig cannot be created. Therefore, the k8s clients can't be created, either. This PR enables us to do things like list Azure resources even if the k8s clients can't be created, which would help debug some failed cluster creations.
Test plan for issue:
Run existing unit and E2E tests
Is there any documentation that needs to be updated for this PR?
No.

Aren't Azure geneva actions and k8s geneva actions completely disjoint?
Please could you see if you can separate them so that each is off a separate struct?

Aren't Azure geneva actions and k8s geneva actions completely disjoint?
Please could you see if you can separate them so that each is off a separate struct?

E2E failed with
unknown blob
make: *** [publish-image-aro] Error 1

E2E failed with
unknown blob
make: *** [publish-image-aro] Error 1

/azp run e2e

/azp run e2e


Setting serial port to low latency

In the VectorNav drivers which I have used in the past, there was a method to set the serial port to use low latency mode.
You can see the discussion here: https://github.com/dawonn/vectornav/pull/82
Can something like that potentially be useful for the inertial sense driver too?

We provide what we call RMC messages. See this information: https://docs.inertialsense.com/user-manual/com-protocol/isb/#realtime-message-controller-rmc


Gradients do not exist for variables

When i'm training,it shows like this:
_initial_net_size
288 288 288
_initial_net_size
288 288 288
Training samples : 6537, Validation samples : 1044
Original yolov3 weights loaded!!
WARNING:tensorflow:Gradients do not exist for variables ['stage0/layer_0/moving_mean:0', 'stage0/layer_0/moving_variance:0',
...
, 'private__conv5_2/layer_103/moving_vari
ance:0', 'detection_layer_3_21/layer_104/moving_mean:0', 'detection_layer_3_21/layer_104/moving_variance:0'] when minimizing the loss.
0%|                                                                                | 3/3268 [00:20<6:26:03,  7.09s/it]t
There are something wrong?

Will the code in 2-training-from-scratch run properly

That‘s ok,despite the warning.

Will the code in 2-training-from-scratch run properly?


How can I host multiple Blazor(WebAssembly) clients in one AspNetCore site

I am trying to host two Blazor(WebAssembly) clients with different route in one AspNetCore site,is there any way to do this?
Could I embed the blazor client files (include blazor dlls) in to different dll,then use EmbeddedFileProvider to provide it. Is this the best way?

@liyanjie8712 thanks for contacting us.
This is not something that we currently support but something that we might look to support in the future.
Thanks.

/cc: @danroth27


Game menu

This is an upgrade of friends, chat and menu.
All of them are, with this pr, working in the url just like the profile does.
The game is accessible in the front end via game->multiplayer->online->join a room. This is basically the file from the backend at the moment, i intend on making it functionnal with players tomorrow.
I've finally patch the weird subliminal redirection at every changes.

I still have an issue in chat when we try to send a message to a player, it will not redirect to /private/nananan when we type his alias.
Chat rooms named are not changed at the moment.

Oh and i forgot to mention i deleted the game files that were still in the front, Jean you need to check if it's ok with you, i might have done some extra cleaning.

Oh and i forgot to mention i deleted the game files that were still in the front, Jean you need to check if it's ok with you, i might have done some extra cleaning.

I have no use for these files, they are documented here anyway in case we need them for some reasons.

oh and i added this verification
if (page.startsWith("profile/")) {
if (page.split("/").length > 2) {
set404View(innerContent);
}
const profileUsername = page.split("/")[1] || data.alias;
setProfileView(innerContent, profileUsername);
} else if (page.startsWith("chat/")) {
const pageSplit = page.split("/");
if (pageSplit.length > 3) {
set404View(innerContent);
} else {
const roomType = page.split("/")[1];
if (roomType) {
const aliasOrRoomToJoin = page.split("/")[2];
if (!['public', 'private'].includes(roomType)) {
set404View(innerContent);
} else if (aliasOrRoomToJoin) {
setChatView(innerContent, roomType, aliasOrRoomToJoin);
} else {
setChatView(innerContent);
}
} else {
if (page.split("/").length > 2) {
set404View(innerContent);
} else {
setChatView(innerContent);
}
}
}
					Wich i feel like it's a very clunky logic, i've done it pretty late, i think some changes will need to be made,
					It's like to prevent url like /chat//alias to work


Mesos: Use slave.PID.Host as task SlaveIP.

What does this PR do?
This is the proper way to set the SlaveIP as done by the underlying mesos-dns helper library
Motivation
Fixes #2545
More

[x] Added/updated tests
[ ] Added/updated documentation

(Note: I don't think there's any documentation to update; the fix is more about an implementation detail to me.)


Extra files for 4-3 and 16-9?

Why are there two separate files, is this really necessary?

Seems to me it useful.


Updating error message

Error message now complains on the max number of 16, as correctly explained in this question's answer.

@mrbullwinkle
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Entity hider- hide players on ignore list

Would be useful to have this as an option. My use case would be to hide bots I've already reported at air obelisk both so I don't bother reporting them again, and so I don't have to right-click through them to use the obelisk. Hiding all players would make me unable to see potential PKers. This feature would probably be useful in other places as well.
This suggestion was brought up before in #1978 but the developer who was working on it seems to have been inactive for a while.

https://github.com/runelite/runelite/issues/3349

https://github.com/runelite/runelite/wiki/Rejected-or-Rolled-Back-Features (actually I should close #1978 too)


Ask for confirmation before closing tabs in IPython console

Feature request: Closing a tab in IPython console should ask for confirmation, or at least there should be an option in preferences to enable such a confirmation.

We offer a confirmation when you have more than one tab connected to the same kernel.
@michaelklachko, why do you think this is important?

This is important to me because when I run multiple simulations, I have several tabs open in IPython console, and these tabs accumulate lots of data (simulation results). More than once I accidentally clicked on X mark on the tab label, closing it - terminating simulation and losing all that data.
Currently there's no confirmation when closing the tabs. I'm using Spyder 2.3.6 on Windows 8.1.
Note that I have "Ask for confirmation before closing" checkbox enabled in Tools - Preferences - IPython Console, so perhaps this is a bug?
Also, there's no confirmation when closing the entire Spyder application, but that's another issue.

I guess asking spyder to confirm only makes sense if there is an ongoing process in a console, is this what you imply?

@goanpeca: no, it shouldn't matter if the code is still running or not, if that's what you mean by "ongoing process". The UI should try to prevent accidental loss of data (computation results shown in the console tabs).
I'm still not sure if this is a feature request or a bug report. What is the purpose of the checkbox in Tools - Preferences - IPython Console - "Ask for confirmation before closing"?

Sure, we could add asking to confirm close as a global option.
Although I would personally disbale it as I would get annoyed with it after a couple of times
Not sure what the "Ask for confirmation..." does, @ccordoba12 ?

@michaelklachko could you create an issue with that ?
Ask for confirmation before closing spyder as a global option in preferences?

There is no need to open a new issue. The option is already there, under the IPython Console preferences, as @michaelklachko pointed out.
The thing is (as I said above) that that option only works when you have more than one console connected to the same kernel. So it's not a bug, but we can change that behavior to ask for confirmation in all cases.
I think @michaelklachko has a good point and we should definitely do that for 3.0 :-)


Search field resets automatically and blocks further entries

Description:
A problem occurs where the search field is automatically reset after the user has entered a few characters. If the user types too slowly or only enters a few characters, the field is reset undesirably. It is then no longer possible to continue typing without clicking on the search field again. This impairs the user experience and leads to interruptions in the input process.
Steps to reproduce:
Click on the search field and start typing a term.
Enter a few characters slowly (e.g. two characters).
Wait briefly while you want to enter the next letter.
Observe that the search field resets automatically.
Try to continue typing without clicking again - this is not possible.
Expected behaviour:
The search box should not reset while the user is still busy typing. The user should be able to continue typing without clicking the search box again.
Environment:
Browser: Google Chrome
Operating system: Windows 11
Website: https://vocabularies.cessda.eu/?q=mo&f=language:en&sort=relevance
Actual behaviour:
The search box resets unexpectedly, resulting in the user not being able to continue typing without clicking in the box again.

@Lorenz1983 tested on Windows with Chrome build: Version 131.0.6778.86 (Official Build) (64-bit), Brave build:
Version 1.73.91 Chromium: 131.0.6778.85 (Official Build) and Firefox build: 132.0.2 (64-bit) and no issues found. All of them are latest build.

Added code to set focus on the search input after the view is initialized. Created PR #970.

Noticed that the Agency filter behaviour is weird. Look like a new bug here.
Steps to reproduce:

The CVS page is opened at the root path; i.e.:  https://vocabularies-staging.cessda.eu/
When setting agencies in the Agency filter, only 1 agency can be set. However, multiple agencies should be possible to specify in the filter. Anyway, proceed by setting 1 agency in the filter.
Click in the search and type something to perform a search.
The Agency filter is empty, but is should stay as set in step 2)
If repeating step 2) here, another or the same agency can be added to the filter.
We can repeat steps 3) to 5) and we can end up having the same Agency in the filter 3 times.

Just create a new issue and link them together.
On Thu, Nov 28, 2024, 10:22 Stefan Dlugolinsky @.***>
wrote:

Noticed that the Agency filter behaviour is weird. Look like a new bug
here.
Steps to reproduce:

The CVS page is opened at the root path; i.e.:
https://vocabularies-staging.cessda.eu/
When setting agencies in the Agency filter, only 1 agency can be
set. However, multiple agencies should be possible to specify in the
filter. Anyway, proceed by setting 1 agency in the filter.
Click in the search and type something to perform a search.
The Agency filter is empty, but is should stay as set in step 2)
If repeating step 2) here, another or the same agency can be added
to the filter.
We can repeat steps 3) to 5) and we can end up having the same
Agency in the filter 3 times.

—
Reply to this email directly, view it on GitHub
https://github.com/cessda/cessda.cvs.two/issues/969#issuecomment-2505633135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB23PTXDLF7UPDHGJCSW4FT2C3OGRAVCNFSM6AAAAABSOAZPW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBVGYZTGMJTGU
.
You are receiving this because you were assigned.Message ID:
@.***>


Modify/add test cases for the providers

Update the existing unit tests as well as add new tests to improve coverage

@bpradipt can this issue be closed?

I'll close this issue and instead create separate provider specific issues as needed.


Use Join-Path

Use Join-Path so that the correct path delimiter is used for the executing platform, rather than concatenating two strings and making assumptions.

Out of curiosity, is this fixing a bug or just "code-health"?

Just the latter - I looked at the script after seeing it mentioned in #1892 and it set off my Spidey-Sense.


Dependency hell for CUDA.jl

v0.3.1 is only compatible with v2.6.3 of CUDA.jl (currently on v3.5.0). This breaks audio processing projects that want to use it with the latest version of Flux (which uses the latest CUDA.jl).

Hi, I feel your pain.
You have to give me a little more context—how is MFCC not compatible with CUDA?  Is it through dependencies that are incompatible with CUDA?

Hello!
I have also encountered this issue while trying to run https://github.com/FluxML/model-zoo/tree/master/contrib/audio/speech-blstm.
Setup is Julia 1.7.1 on Ubuntu 20.04.2 x86_64
Minimal example:

Create a new Julia env
Add Flux and CUDA
Add MFCC
Notice the downgrades

Log:
~/julia-sandbox> julia
               _
   _       _ _(_)_     |  Documentation: https://docs.julialang.org
  (_)     | (_) (_)    |
   _ _   _| |_  __ _   |  Type "?" for help, "]?" for Pkg help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 1.7.1 (2021-12-22)
 _/ |\__'_|_|_|\__'_|  |  Official https://julialang.org/ release
|__/                   |

(@v1.7) pkg> activate .
  Activating new project at `~/julia-sandbox`

(julia-sandbox) pkg> add Flux, CUDA
    Updating registry at `~/.julia/registries/General.toml`
   Resolving package versions...
    Updating `~/julia-sandbox/Project.toml`
  [052768ef] + CUDA v3.6.4
  [587475ba] + Flux v0.12.8
    Updating `~/julia-sandbox/Manifest.toml`
  [621f4979] + AbstractFFTs v1.1.0
  [1520ce14] + AbstractTrees v0.3.4
  [79e6a3ab] + Adapt v3.3.3
  [4fba245c] + ArrayInterface v3.2.2
  [ab4f0b2a] + BFloat16s v0.2.0
  [fa961155] + CEnum v0.4.1
  [052768ef] + CUDA v3.6.4
  [082447d4] + ChainRules v1.23.0
  [d360d2e6] + ChainRulesCore v1.11.6
  [9e997f8a] + ChangesOfVariables v0.1.2
  [944b1d66] + CodecZlib v0.7.0
  [3da002f7] + ColorTypes v0.11.0
  [5ae59095] + Colors v0.12.8
  [bbf7d656] + CommonSubexpressions v0.3.0
  [34da2185] + Compat v3.41.0
  [9a962f9c] + DataAPI v1.9.0
  [864edb3b] + DataStructures v0.18.11
  [163ba53b] + DiffResults v1.0.3
  [b552c78f] + DiffRules v1.9.0
  [ffbed154] + DocStringExtensions v0.8.6
  [e2ba6199] + ExprTools v0.1.7
  [1a297f60] + FillArrays v0.12.7
  [53c48c17] + FixedPointNumbers v0.8.4
  [587475ba] + Flux v0.12.8
  [f6369f11] + ForwardDiff v0.10.25
  [d9f16b24] + Functors v0.2.7
  [0c68f7d7] + GPUArrays v8.1.3
  [61eb1bfa] + GPUCompiler v0.13.10
  [7869d1d1] + IRTools v0.4.4
  [615f187c] + IfElse v0.1.1
  [3587e190] + InverseFunctions v0.1.2
  [92d709cd] + IrrationalConstants v0.1.1
  [692b3bcd] + JLLWrappers v1.4.0
  [e5e0dc1b] + Juno v0.8.4
  [929cbde3] + LLVM v4.7.1
  [2ab3a3ac] + LogExpFunctions v0.3.6
  [1914dd2f] + MacroTools v0.5.9
  [e89f7d12] + Media v0.5.0
  [e1d29d7a] + Missings v1.0.2
  [872c559c] + NNlib v0.7.34
  [a00861dc] + NNlibCUDA v0.1.11
  [77ba4419] + NaNMath v0.3.6
  [bac558e1] + OrderedCollections v1.4.1
  [21216c6a] + Preferences v1.2.3
  [74087812] + Random123 v1.4.2
  [e6cf234a] + RandomNumbers v1.5.3
  [c1ae055f] + RealDot v0.1.0
  [189a3867] + Reexport v1.2.2
  [ae029012] + Requires v1.3.0
  [a2af1166] + SortingAlgorithms v1.0.1
  [276daf66] + SpecialFunctions v2.0.0
  [aedffcd0] + Static v0.4.1
  [90137ffa] + StaticArrays v1.3.2
  [82ae8749] + StatsAPI v1.2.0
  [2913bbd2] + StatsBase v0.33.14
  [a759f4b9] + TimerOutputs v0.5.14
  [3bb67fe8] + TranscodingStreams v0.9.6
  [a5390f91] + ZipFile v0.9.4
  [e88e6eb3] + Zygote v0.6.33
  [700de1a5] + ZygoteRules v0.2.2
  [dad2f222] + LLVMExtra_jll v0.0.13+1
  [efe28fd5] + OpenSpecFun_jll v0.5.5+0
  [0dad84c5] + ArgTools
  [56f22d72] + Artifacts
  [2a0f44e3] + Base64
  [ade2ca70] + Dates
  [8bb1440f] + DelimitedFiles
  [8ba89e20] + Distributed
  [f43a241f] + Downloads
  [b77e0a4c] + InteractiveUtils
  [4af54fe1] + LazyArtifacts
  [b27032c2] + LibCURL
  [76f85450] + LibGit2
  [8f399da3] + Libdl
  [37e2e46d] + LinearAlgebra
  [56ddb016] + Logging
  [d6f4376e] + Markdown
  [a63ad114] + Mmap
  [ca575930] + NetworkOptions
  [44cfe95a] + Pkg
  [de0858da] + Printf
  [9abbd945] + Profile
  [3fa0cd96] + REPL
  [9a3f8284] + Random
  [ea8e919c] + SHA
  [9e88b42a] + Serialization
  [1a1011a3] + SharedArrays
  [6462fe0b] + Sockets
  [2f01184e] + SparseArrays
  [10745b16] + Statistics
  [fa267f1f] + TOML
  [a4e569a6] + Tar
  [8dfed614] + Test
  [cf7118a7] + UUIDs
  [4ec0a83e] + Unicode
  [e66e0078] + CompilerSupportLibraries_jll
  [deac9b47] + LibCURL_jll
  [29816b5a] + LibSSH2_jll
  [c8ffd9c3] + MbedTLS_jll
  [14a3606d] + MozillaCACerts_jll
  [4536629a] + OpenBLAS_jll
  [05823500] + OpenLibm_jll
  [83775a58] + Zlib_jll
  [8e850b90] + libblastrampoline_jll
  [8e850ede] + nghttp2_jll
  [3f19e933] + p7zip_jll

(julia-sandbox) pkg> add MFCC
   Resolving package versions...
    Updating `~/julia-sandbox/Project.toml`
  [052768ef] ↓ CUDA v3.6.4 ⇒ v2.6.3
  [587475ba] ↓ Flux v0.12.8 ⇒ v0.12.1
  [ca7b5df7] + MFCC v0.3.1
    Updating `~/julia-sandbox/Manifest.toml`
  [621f4979] ↓ AbstractFFTs v1.1.0 ⇒ v1.0.1
  [4fba245c] - ArrayInterface v3.2.2
  [ab4f0b2a] ↓ BFloat16s v0.2.0 ⇒ v0.1.0
  [a74b3585] + Blosc v0.7.2
  [052768ef] ↓ CUDA v3.6.4 ⇒ v2.6.3
  [082447d4] ↓ ChainRules v1.23.0 ⇒ v0.8.25
  [d360d2e6] ↓ ChainRulesCore v1.11.6 ⇒ v0.10.13
  [9e997f8a] - ChangesOfVariables v0.1.2
  [717857b8] + DSP v0.6.10
  [b552c78f] ↓ DiffRules v1.9.0 ⇒ v1.5.0
  [7a1cc6ca] + FFTW v1.4.5
  [5789e2e9] + FileIO v1.12.0
  [587475ba] ↓ Flux v0.12.8 ⇒ v0.12.1
  [0c68f7d7] ↓ GPUArrays v8.1.3 ⇒ v6.4.1
  [61eb1bfa] ↓ GPUCompiler v0.13.10 ⇒ v0.10.0
  [f67ccb44] + HDF5 v0.13.7
  [615f187c] - IfElse v0.1.1
  [842dd82b] + InlineStrings v1.1.1
  [d8418881] + Intervals v1.5.0
  [3587e190] - InverseFunctions v0.1.2
  [c8e1da08] + IterTools v1.4.0
  [929cbde3] ↓ LLVM v4.7.1 ⇒ v3.9.0
  [2ab3a3ac] ↓ LogExpFunctions v0.3.6 ⇒ v0.3.0
  [ca7b5df7] + MFCC v0.3.1
  [c03570c3] + Memoize v0.4.4
  [78c3b35d] + Mocking v0.7.3
  [a00861dc] - NNlibCUDA v0.1.11
  [6fe1bfb0] + OffsetArrays v1.10.8
  [69de0a69] + Parsers v2.2.0
  [f27b6e38] + Polynomials v1.2.1
  [74087812] - Random123 v1.4.2
  [e6cf234a] - RandomNumbers v1.5.3
  [c1ae055f] - RealDot v0.1.0
  [3cdcf5f2] + RecipesBase v1.2.1
  [6c6a2e73] + Scratch v1.1.0
  [276daf66] ↓ SpecialFunctions v2.0.0 ⇒ v0.10.3
  [aedffcd0] - Static v0.4.1
  [f269a46b] + TimeZones v1.7.1
  [8149f6b0] + WAV v1.2.0
  [e88e6eb3] ↓ Zygote v0.6.33 ⇒ v0.6.17
  [0b7ba130] + Blosc_jll v1.21.1+0
  [f5851436] + FFTW_jll v3.3.10+0
  [0234f1f7] + HDF5_jll v1.10.5+7
  [1d5cc7b8] + IntelOpenMP_jll v2018.0.3+2
  [dad2f222] - LLVMExtra_jll v0.0.13+1
  [5ced341a] + Lz4_jll v1.9.3+0
  [856f044c] + MKL_jll v2021.1.1+2
  [3161d3a3] + Zstd_jll v1.5.0+0
  [05823500] - OpenLibm_jll
Precompiling project...
  ✗ CUDA
  ✗ Flux
  0 dependencies successfully precompiled in 5 seconds (81 already precompiled)
  2 dependencies errored. To see a full report either run `import Pkg; Pkg.precompile()` or load the packages

~/julia-sandbox> cat Project.toml
[deps]
CUDA = "052768ef-5323-5732-b1bb-66c8b64840ba"
Flux = "587475ba-b771-5e3f-ad9e-33799f191a9c"
MFCC = "ca7b5df7-6146-5dcc-89ec-36256279a339"

Hope this is helpful

A very important downgrade is LLVM v4.7.1 ⇒ v3.9.0, which broke Flux precompilation for me
Tried updating it back, got the following; hope this is also helpful
(julia-sandbox) pkg> add<EMAIL_ADDRESS>   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package CUDA [052768ef]:
 CUDA [052768ef] log:
 ├─possible versions are: 0.1.0-3.6.4 or uninstalled
 ├─restricted to versions * by an explicit requirement, leaving only versions 0.1.0-3.6.4
 ├─restricted by julia compatibility requirements to versions: [2.3.0, 2.5.0-3.6.4] or uninstalled, leaving only versions: [2.3.0, 2.5.0-3.6.4]
 ├─restricted by compatibility requirements with LLVM [929cbde3] to versions: 3.3.3-3.6.4 or uninstalled, leaving only versions: 3.3.3-3.6.4
 │ └─LLVM [929cbde3] log:
 │   ├─possible versions are: 0.9.0-4.7.1 or uninstalled
 │   └─restricted to versions 4.7.1 by an explicit requirement, leaving only versions 4.7.1
 └─restricted by compatibility requirements with SpecialFunctions [276daf66] to versions: 0.1.0-2.6.3 or uninstalled — no versions left
   └─SpecialFunctions [276daf66] log:
     ├─possible versions are: 0.7.0-2.0.0 or uninstalled
     └─restricted by compatibility requirements with MFCC [ca7b5df7] to versions: 0.7.0-0.10.3
       └─MFCC [ca7b5df7] log:
         ├─possible versions are: 0.3.1 or uninstalled
         └─restricted to versions * by an explicit requirement, leaving only versions 0.3.1

We can reveal the culprit by explicitly requiring recent LLVM and Flux
(julia-sandbox) pkg> add LLVM@4, <EMAIL_ADDRESS>MFCC
   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package CUDA [052768ef]:
 CUDA [052768ef] log:
 ├─possible versions are: 0.1.0-3.6.4 or uninstalled
 ├─restricted by julia compatibility requirements to versions: [2.3.0, 2.5.0-3.6.4] or uninstalled
 ├─restricted by compatibility requirements with Flux [587475ba] to versions: 2.1.0-3.6.4, leaving only versions: [2.3.0, 2.5.0-3.6.4]
 │ └─Flux [587475ba] log:
 │   ├─possible versions are: 0.4.1-0.12.8 or uninstalled
 │   └─restricted to versions 0.12 by an explicit requirement, leaving only versions 0.12.0-0.12.8
 ├─restricted by compatibility requirements with LLVM [929cbde3] to versions: 3.3.3-3.6.4 or uninstalled, leaving only versions: 3.3.3-3.6.4
 │ └─LLVM [929cbde3] log:
 │   ├─possible versions are: 0.9.0-4.7.1 or uninstalled
 │   └─restricted to versions 4.7.1 by an explicit requirement, leaving only versions 4.7.1
 └─restricted by compatibility requirements with SpecialFunctions [276daf66] to versions: 0.1.0-2.6.3 or uninstalled — no versions left
   └─SpecialFunctions [276daf66] log:
     ├─possible versions are: 0.7.0-2.0.0 or uninstalled
     └─restricted by compatibility requirements with MFCC [ca7b5df7] to versions: 0.7.0-0.10.3
       └─MFCC [ca7b5df7] log:
         ├─possible versions are: 0.3.1 or uninstalled
         └─restricted to versions 0.3.1 by an explicit requirement, leaving only versions 0.3.1

MFCC requires an old version of SpecialFunctions (0.7.0-0.10.3), which requires old CUDA (2.6.3), which is only compatible with old LLVM (3.3.3-3.6.4)

Hello,
I am sorry to hear that this happened to you.  Any clue what special function MFCC needs?  It is probably not such a fancy function, I am sure we can make this more modern

There is a comment about erfinv in the code: https://github.com/JuliaDSP/MFCC.jl/blob/c87c6ab8a6773fc7dca4a4e1cd641b9695d9e5fe/src/MFCC.jl#L16
Maybe it's the only one MFCC.jl uses, hard to tell

The Project.toml doesn't specify any version in dependencies, where is this coming from?

I think I found it. For some reason, Julia General Registry thinks that SpecialFunctions has to have version under 0.10
https://github.com/JuliaRegistries/General/blob/master/M/MFCC/Compat.toml
Current workaround
Install MFCC from git:
(sandbox) pkg> add https://github.com/JuliaDSP/MFCC.jl<EMAIL_ADDRESS>
To avoid explicit HDF5 line in Project.toml, one can use branch from #25:
(sandbox) pkg> add https://github.com/maetshju/MFCC.jl#fixhdf5

I think I managed to update all dependencies (they weren't so strong anyways), basically because I figured out how to do it:
(@v1.8) pkg> activate .
(@v1.8) pkg> update
## try new version of dependency in Project.toml
(@v1.8) pkg> update
(@v1.8) pkg> test

fix problems in the code along the way, and repeat.
Can you maybe check if v0.3.3 still causes these issues for you?


UI: Use bignumbers until value is displayed

enforce using typed parameters in conversion utils

closing due to lack of clarity / examples where this causes bugs


Add support for advanced LightningJ APIs

Right now lnd-grpc-client creates injectable beans for the sync / async versions (SynchronousLndAPI, AsynchronousLndAPI) of the main LND API.
The main LND API exposes basic LND functionalities, but LightningJ also has more advanced ones.
As per its documentation, it exposes several other API services, each with a sync + an async variant, for example:

Wallet Unlocker API

org.lightningj.lnd.wrapper.walletunlocker.SynchronousWalletUnlockerAPI
org.lightningj.lnd.wrapper.walletunlocker.AsynchronousWalletUnlockerAPI

Invoices API

org.lightningj.lnd.wrapper.invoices.SynchronousInvoicesAPI
org.lightningj.lnd.wrapper.invoices.AsynchronousInvoicesAPI

etc

Could you please create beans for the other ones as well?
From what I can tell it comes down to extending

org.tbk.lightning.lnd.grpc.LndJsonRpcClientFactoryImpl and
org.tbk.lightning.lnd.grpc.config.LndJsonRpcClientAutoConfiguration

Thanks.

Hey @cd2357 - thank you very much! this is really appreciated!
This is implemented with commit https://github.com/theborakompanioni/bitcoin-spring-boot-starter/commit/a919dd44ae747b2316b1476cae1c8c548c189111 .
I will do some further testing and will release a new version soon.
Your issue was also the trigger to finally get rid of the erroneous term 'JsonRpc' in class names of this package.
This has also been fixed, as well as the health/info actuator property names (also contained the term 'jsonrpc').
The class LndJsonRpcClientFactory has been removed, as it served no real benefit to other users.
Thank you for your time, sir.
Have a nice day.

Excellent, that's great to hear @theborakompanioni  :)
Looking forward to the next release.

Looking forward to the next release.

@cd2357 v0.2.0 has been released.
Would you report if that solved your problem, or if there is anything that you are missing from this release?
I'll close the issue if it satisfies your needs.
Have a nice day. Thank you.

Works, just tested 👍
I can close it right away.


manifest

I am new to puppet so I need some advice. In my /root/.puppetlabs/bolt/modules/wsusserver/manifests I have several manifests already in place; approvalrule.pp  built_in_computer_target_groups.pp  config.pp  init.pp  install.pp  params.pp  service.pp, do I add the code to one of these or create a new one. I apologize for my noobieness.

I've not used Bolt but I do not think this module supports Bolt. You need to install and configure a Puppet Server (either with Puppet Open Source free or a paid version of Puppet Enterprise) as well as install Puppet Agents on your client machines that link back to your server.  You generally do not edit the modules themselves.


Parsing function info from DWARF is slow

The way we handle function information in DWARF doesn't scale well at all, and slows down exponentially with number of functions.
As a benchmark: parsing the debug info from factorio's headless linux client takes ~1300 seconds on my computer

Fixed in 4.1.5427-dev
Parsing factorio debug info takes ~14 seconds now


WIP Initial implementation of preprocessing tool

The scope is tracked at https://github.com/src-d/backlog/issues/648#issuecomment-300134120
The plan for this PR is to finish initial scope from TODO section of above. Further improvements will be handled in subsequent PRs.
Details on how to run it are inside README.md

@bzz @mcuadros before this merged we need to change the license to Apache License 2.0, we're no longer using MIT

All reviews are addressed now, please, let me know what you think!


test cases for all current rest specifications (get, put, post)

https://github.com/sopra-fs22-group-34/server/commit/b300ee0df0f52dbfdfa58c04bf4982a22791f9c5


Backport #2271

Backport #2271 per request as it wasn't applying cleanly.

Attempting to get rid of jenkins branch build check via close / reopen.

Reopening for review.


Similar tools or datasets

I'd be curious to know how hyRefactor output compares to the ENHD data (https://doi.org/10.5066/P9PA63SM).
.. and also to these similar toolsets:

https://github.com/mbtyers/riverdist
https://github.com/MiKatt/openSTARS

Hi @jsta -- sorry to be slow here -- was out on family leave.
ENHD is largely updates to the NHDPlusV2 network topology. I am actually using that in this project https://code.usgs.gov/wma/nhgf/gfv2.0 in conjunction with hyRefactor.
Riverdist is a toolbox for analyzing hydrographic data -- not entirely incompatible with hyRefactor but a different set of capacities.
openSTARS looks like a soup to nuts network creation utility for people who want to start from a DEM and build their network from whole cloth.
hyRefactor is focused on building multi-scale river networks and corresponding catchments that hang together and suit multiple purposes at a variety of scales. I've used hyRefactor with datasets I've built as well as with the NHDPlusV2 and HR ... so in theory all these things could be used together in workflows.
Does that help?


envoy: add mongo filter feature

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@kyessenov: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
245e87a4bb6cc1e1aa784ff4d0228e96691133d7
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Details activity: Open category in-app

Once a form of category navigation is available, it would be great if it were accessible by clicking a category of an existing media.

Can I get more insight into this issue. On what to actually do so that I can try working on this issue.
Thanks :)

I don't think this can be implemented until we actually have category navigation?

Yes, this can not start until #1221 is done, which will take at least a few months.


Rename github master branch to main

Rename github master branch to main

Happy to take a crack at this. Should be fairly straightforward, we'll need to create main, update a few references, and then set it as the default branch. I can push up the branch and ensure it's inline with our old mainline and then put together a PR to update references before we switch the default branch.

Perfect. Existing PRs can be edited to change the base branch easily once the main branch exists.
And for context on why this change is being made (I should have included this information when creating the issue):
https://www.zdnet.com/article/github-to-replace-master-with-alternative-term-to-avoid-slavery-references/

I've created the main branch and put together a PR of the references in the README and CHANGELOG that needed updating. @andygrove I think you'll have to update the default branch when you're ready.

main is now the default branch.


ms_calc_flux_rsfme, and attendant testing

duuuude!!!!!!!! hope you're doing amazingly well!!! Miss you and the whole
team, and MAJOR congrats on the Q sim paper!!!
Still up here in Maine living in a hut I made out of sticks (extremely
literally) and canoeing around with a 12 foot pole, starting to get stoked
to dive back into our lovable world of science soon tho :)
wishing you the best and can't wait until I get a chance to hear an update
on things!!
Best,
Wes


./nvidia.s:1: unexpected token after '#': 首先添加一个blacklist exit status 2

Please answer the following before submitting your issue:
Note: Please include any substantial examples (debug session output,
stacktraces, etc) as linked gists.

What version of Delve are you using (dlv version)?


Selected conversation missing "select" state

PURCHASE-1907
https://www.notion.so/artsy/Selected-conversation-missing-select-state-dfc1c1191eca40bb9c2a9b294be7a497

Fails

:no_entry_sign:

Danger failed to run dangerfile.ts.

Error RangeError
Maximum call stack size exceeded
RangeError: Maximum call stack size exceeded
    at resolveStandardizedName (/home/circleci/project/node_modules/@babel/core/lib/config/files/plugins.js:97:33)
    at resolvePlugin (/home/circleci/project/node_modules/@babel/core/lib/config/files/plugins.js:54:10)
    at loadPlugin (/home/circleci/project/node_modules/@babel/core/lib/config/files/plugins.js:62:20)
    at createDescriptor (/home/circleci/project/node_modules/@babel/core/lib/config/config-descriptors.js:154:9)
    at /home/circleci/project/node_modules/@babel/core/lib/config/config-descriptors.js:109:50
    at Array.map (<anonymous>)
    at createDescriptors (/home/circleci/project/node_modules/@babel/core/lib/config/config-descriptors.js:109:29)
    at createPluginDescriptors (/home/circleci/project/node_modules/@babel/core/lib/config/config-descriptors.js:105:10)
    at /home/circleci/project/node_modules/@babel/core/lib/config/config-descriptors.js:63:49
    at cachedFunction (/home/circleci/project/node_modules/@babel/core/lib/config/caching.js:33:19)

Dangerfile
-----------------------------------^

  Generated by :no_entry_sign: dangerJS against 8780f51f75ceb353cc7bc81aca472d3755ea01b5

Looks great @lilyfromseattle. I think there's an opportunity to follow up and simplify some of the logic around when we add dividers to the snippet, but what you have here is good for now. 👍


top level Args while using commands

Hi,
I currently try to reimplement the commandline handling an application
has using go with your kingpin(v2) interface, and ran against a wall.
It might be im too stupid or I may try something non-supported (and am
also entirely new to Go too), but I ran into:
can't mix top-level Arg()s with Command()
What I try is having an application have a number of commands, with
some arguments to them - but when none of the commands fit, the
argument is stored in a variable.
Say,
tm s la la   - its doing command s with 2 strings as an arg.
tm ms foo    - command ms, one arg
tm bar       - no command, just one arg.
Here, bar can be ANY random string, not predictable, but I need to
know it to handle it.
Is it possible?
Thanks!
Joerg

Unfortunately this is not yet possible, but I agree it would be very handy. It would also be nice to be able to mix args and commands like so:
mycli command0 <arg> command1

Now that would be top, yes.
Hrm, not possible means either a huge change in the ui, which is bad, finding another commandline parser that does support it, implementing my own (oh gosh, brrr) or me using something else to learn go. I guess it will be the latter. :)
Do you think it can be possible in v3? Or is this more for a v4 somewhen?
Joerg

Ideally for v3, definitely.

This is implemented in the v3-unstable branch (gopkg.in/alecthomas/kingpin.v3-unstable).

And by "this" I mean the ability to do command0 <arg0> <arg1> command1 <arg2>. Interspersed arguments must be .Required() to avoid ambiguity.


refactor: remove unnecessary allocations and apply clippy fixes

The following pull request removes lots of instances of unnecessary heap allocations through either .clone(), .collect(), or .to_string() by using references or utilizing Display instead of ToString.
Other than that, this pull request also refactors many parts of the code to be a lot cleaner! :D
This pull request is a redo of #33 by using signed commits.

This is amazing! Love contributions that help reduce the footprint of Reaper. I'll take a look at this today and if everything is good, I'll merge it in
If you're in the Reaper Discord, please @ me and I'll give you a contributer role!

:heart:


Remove 2 non-.govs (HHS)

HHS has requested that we remove tslms.org and convertlanguage.com from their BOD 18-01 reports.
Upon review, it appears that while tslms.org used to redirect to the AHRQ website (as of 2019 when we added it), it no longer appears to be tied to HHS (no longer registered to anyone, no redirects, no site:tslms.org results on Google).
It also looks like convertlanguage.com is a domain used for hosting various sites all owned by different orgs (e.g.eshhsorgandonor.convertlanguage.com and alaskaair.convertlanguage.com), but the base domain itself is not specifically managed or operated on HHS' behalf.
Please remove tslms.org and convertlanguage.com.

See also cisagov/scan-target-data#52.


High-Scale IPcache: Chapter 1

This new feature has a CFP at https://github.com/cilium/design-cfps/pull/7. The goal of this first PR is to define the new flag and remove all pod entries (see exceptions in CFP) from the ipcache when that flag is set.

/test

/test

/test-1.25-4.19

/test


Process exits with code 1 even though all tests passed.

For some reason both the threads pass perfectly fine and report that all specs passed successfully but the summary table stays empty.

Any suggestion on how to fix the threads reporting their results correctly so that the summary table is filled properly?

I'm having the exact same issue, all test passing yet the Git action is failing :/

Do you have cypress-multi-reporters installed ?
Make also sure it used for reporting, not any other

I had the same issue. I followed this solution and now it works great! https://github.com/tnicola/cypress-parallel/issues/90#issuecomment-1693585182


Release 2.5.1?

Can we get a release containing #56 ?

v2.5.1 is available.

@dkoszewnik Awesome, thanks!


Include DISQUS comment alternative

I can implement manager comments DISQUS, some people using alternative to manager comments. 👍

@felipesousa thanks , but now new version is support DISQUS comments : ） hope u like it~


FAILED deepdive check compiled_base_relations_have_input_data

I ran into this error when building own app.  I had to create dummy files with the right basename to circumvent it. if one could add to the current documentation (great documentation btw) about how to properly setup the input requirement, that'd be even greater.

Thanks for reporting this, @cognitronz . Yes, we've learned that this dummy file requirement is a bit cumbersome. We'll address it in the upcoming release (likely by removing this requirement).

Hey @cognitronz , can you explain in brief how you created the dummy file? Thanks!


IE cross tab updates (MIX-713)

I also updated the library to the latest version.  You could either review it by commit, or I'll try to update it first once I have permission.
Main fix: Add a new "ngStorageUpdated-..."


Restore the TreehouseView parameter in ViewBinder.widgetFactory

This is necessary for View-based factories which need a Context to create the child views.

This seems like a really weird way to get a context other than just bringing one you capture yourself.

+1 to what Jake said. view is only necessary for its context when using View-based widget factories. Compose UI and UiKit don't need this param. I wonder if there's a better place for widgetFactory to live?
Could you grab the view from the other ViewBinder methods and hold a reference to it?

I don’t follow?
In our application, the TreehouseApp can be an host-application-scoped object. Ie. it can be a Dagger singleton. The TreehouseView a screen-scoped object. It has a narrower Context that may have scoped things in it, like a theme.
We need a way for the TreehouseApp to create child views for the TreehouseView. How should it get a Context for that? It used to work!

I think it's weird to have to unsafe cast this to a subtype to get access to one of its properties that's effectively a dependency of the factory. What about if you need an image loader or a feature flag manager or something else? Do we have a missing layer that sits between app and each view?

This is specifically about Context in Android. It travels through the view hierarchy, and we can’t/shouldn’t get it from elsewhere.

(Previously the `TreehouseView had a widget factory parameter, but you deleted that also. Bringing that back instead would also unblock.)

@swankjesse This is what I did to get access to the view, though it's not ideal:
class ViewBinderImpl : ViewBinder {
  private lateinit var view: TreehouseView<*>

  override fun codeLoading(view: TreehouseView<*>) {
    this.view = view
  }
}

Hmm I don't have better solution so I'd be up for re-adding the parameter until we figure out a better way to pass view-scoped dependencies to Widget.Factory instances.

Yeah things flow through the other hierarchies too. With Compose UI you can't cast a subtype to access anything since it's all contained within the functions. So this is a band-aid for now and we're missing something to support these usages.

I don’t expect we’ll need anything for Compose UI. It can pass themes etc. down the call stack while rendering.
It’s a quirk of Android’s API that you need a Context to construct a View.
What we had previously was even more powerful than this, cause we could configure the factory for each view. But this should be sufficient to unblock Android views.

My point is more that the motivation to include these things is usually a sign of broken layering.
Your refactor when we open sourced it was a first step to fixing things. My refactor helped clean that up further in order to pipe through view-level configuration but it wasn't everything. In that refactor I noted that this specific callback seemed misplaced as it's conflated with event-listener-style callbacks (perhaps now completely obviated by your actual event listener).
I don't think it makes sense for the TreehouseApp to own a ViewBinder and that responsibility should be on the TreehouseView. In doing so, that means the specific subtype of the TreehouseVIew is known without an unsafe cast across unrelated types. But even better, the Context used to create the TreehouseView is directly available (as well as any other dependencies you want to pull).
Perhaps it's easier for me to show you what I mean with a PR. I'll file an issue about it for follow up post-hackweek.


Misc features

Couple of misc features:
x button to clear autocomplete search bar
Whatsapp and Telegram share buttons
Currently the share buttons are placed at the bottom of the restaurants list, and updated whenver we choose a different restaurant. Would be good to get some input on where would be the best location for these buttons.
Currently telegram share link shares a placeholder link. This would be replaced with the link to google maps once that feature is completed.
https://lyeo-sps-summer20.appspot.com/

We can have an infowindow for the selected restaurant similar to the one on the chosen location, to display the restaurant info along with the social media sharing icons.
Also note that there is a small conflict with the recently merged PR.

Alright cool will wait for the infowindow to be merged and sort out the conflicts as well

Moved share buttons to infowindow
https://lyeo-sps-summer20.appspot.com/

the 'x' still seems out of place and feels as though its overlapping with the text .
Reference:

I've experimented with this and I think setting the whole pac-card to flex (I did it in my PR here https://github.com/daekoon/EatGoWhere/pull/16/files#diff-494841b0b2395305cf87e3d10f6911eaR65) along with a few padding adjustments, will help with the placement. Could you give it a try and see if it helps?

Added a bit more padding, should be fixed now. Not too sure about setting it to flex?
https://lyeo-sps-summer20.appspot.com/

LGTM!


Error Serializing Inherited Attributes

Extending the Object.prototype causes Element.outerHTML() to break.
var jsdom = require('jsdom');

Object.prototype.test_name = 'test_value';

var dom = jsdom.jsdom('

');

var div = dom.getElementById('0');

console.log(div.test_name);
console.log(div.outerHTML);

test_value

This tells me that the issue is with the jsdom since its documentAdapter is being passed to the serializer.
I would expect jsdom to skip attributes inherited from Object.
Oddly if Object.prototype is extended after the DOM is initialized it works as expected.
var jsdom = require('jsdom');

var dom = jsdom.jsdom('

');

Object.prototype.test_name = 'test_value';

var div = dom.getElementById('0');

console.log(div.test_name);
console.log(div.outerHTML);

test_value

I suspect the HTMLElement inheritance tree affects the attributes when creating the div, but requires explicit calls through the DOM APIs to change, so the fix will likely be during the attributes initialization.

Great minimal test case, thank you. Shouldn't be too hard to fix.

If there is anything I can help with let me know.
It looks like test/living-dom/attributes.js would be a good place to test this behavior. If you don't get to it before I do, I wouldn't mind contributing some tests.

I think https://github.com/tmpvar/jsdom/blob/master/test/jsdom/parsing.js would be best, actually.

That makes sense.

I thought about adding a nested describe block to handle the global setup in before and after, but I decided to just setup and teardown in the body of the specify since mocha doesn't really offer an after utility for a single test.
https://github.com/deckar01/jsdom/commit/baf7189f608578cfb2bcbf3c7062e7fc5787bbe6


Upgrade libXfont2 to version 2.0.6 for Mariner 3.0

Upgrade libXfont2 to version 2.0.6 and corresponding changes to cgmanifest and signature.json files. README has been renamed as README.md in the upgradded package.
Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[X] The toolchain has been rebuilt successfully (or no changes were made to it)
[X] The toolchain/worker package manifests are up-to-date
[X] Any updated packages successfully build (or no packages were changed)
[X] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[X] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[X] All package sources are available
[X] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[X] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[X] All source files have up-to-date hashes in the *.signatures.json files
[X] sudo make go-tidy-all and sudo make go-test-coverage pass
[X] Documentation has been updated to match any changes to the build system
[X] Ready to merge

Summary

This PR upgrades libXfont2 to version 2.0.6.  README file has been renamed as README.md in the upgraded package.
Change Log

Upgrade libXfont2 to version 2.0.6
No other package impacted
No previous CVEs

Does this affect the toolchain?
NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=501583&view=results

Thank you @kanikanema for the review!


Manually add New Materials.

Before opening an issue:

[ x] My question isn't answered in any other issues (even the closed ones)
[ x] I'm not reporting a bug/error

If you open unnecessary/dumb issues, you'll be rewarded with appropriate tags.

a way to manually add materials, for example events materials and even Class piece/monuments, i am in the middle of grinding Lancer Piece but doesn't have a option to manually do that(i think? at least i tried to find but ddn't find)

wrong lavel -_-


Demo broken

The page redirects to https but references stylesheets using http which Chrome refuses to load:
transfer.frontwise.com/:6 Mixed Content: The page at 'https://transfer.frontwise.com/frontwise/grid-editor/example/' was loaded over HTTPS, but requested an insecure stylesheet 'http://maxcdn.bootstrapcdn.com/bootstrap/3.3.1/css/bootstrap.min.css'. This request has been blocked; the content must be served over HTTPS.
transfer.frontwise.com/:7 Mixed Content: The page at 'https://transfer.frontwise.com/frontwise/grid-editor/example/' was loaded over HTTPS, but requested an insecure stylesheet 'http://maxcdn.bootstrapcdn.com/font-awesome/4.2.0/css/font-awesome.min.css'. This request has been blocked; the content must be served over HTTPS.
transfer.frontwise.com/:1 Mixed Content: The page at 'https://transfer.frontwise.com/frontwise/grid-editor/example/' was loaded over HTTPS, but requested an insecure script 'http://maxcdn.bootstrapcdn.com/bootstrap/3.3.1/js/bootstrap.min.js'. This request has been blocked; the content must be served over HTTPS.
transfer.frontwise.com/:1 Mixed Content: The page at 'https://transfer.frontwise.com/frontwise/grid-editor/example/' was loaded over HTTPS, but requested an insecure script 'http://tinymce.cachefly.net/4.0/tinymce.min.js'. This request has been blocked; the content must be served over HTTPS.
transfer.frontwise.com/:1 Mixed Content: The page at 'https://transfer.frontwise.com/frontwise/grid-editor/example/' was loaded over HTTPS, but requested an insecure script 'http://tinymce.cachefly.net/4.0/jquery.tinymce.min.js'. This request has been blocked; the content must be served over HTTPS.

Thanks so much for bringing this to my attention. Should be fixed now.


CX HttpOnlyCookies_In_Config @ src/main/webapp/WEB-INF/web.xml [master]

HttpOnlyCookies_In_Config issue exists @ src/main/webapp/WEB-INF/web.xml in branch master
The src\main\webapp\WEB-INF\web.xml application configuration file, at line 1, does not define sensitive application cookies with the "httpOnly" flag, which could allow client-side scripts access to the session cookies.
Severity: Medium
CWE:1004
Checkmarx
Training
Recommended Fix
Lines: 1

Code (Line #1):
<?xml version="1.0" encoding="UTF-8"?>

Issue still exists.


Consider using the Webkit logo

The logo right now is hard to decipher in its small size and most people don't have Webkit Nightly installed or even know about it. The solution could be to use the new WebKit logo

The WebKit Nightly logo looks similar to the Safari one (it's just different colors), but I'm OK with switching if you think this one is more clear.
Let's also see what we decide in https://github.com/mozilla/platatus/issues/324, we could use the WebKit Nightly logo for in-development features and this one for released features.

Close in favor of #409?


Repo actively maintained?

Thanks for the awesome project 💪
I'm using it with a mongoose project and it works great.
I'm considering adminJS for another Prisma project. Is this repo maintained? I see issues from months ago w/o responses, and last delpoyment from July.
Thanks :)

Hey @goldylucks,

Perfect! 💕


Make the UI more responsive

Where to start?! There are elements that will be very hard to make repsonsive (e.g., the current Curation experience). However, much of the UI could/should be more responsive than it is.
Need an overall strategy from someone who's been there done that.

Some questions to get us thinking about this stuff:
What device types do we want to support? For each device type we should consider the use cases. What functions make sense? E.g. do users want to curate on their phone while on a bus?

[ ] Mobile (<768px)

[ ] Browse
[ ] Inspect
[ ] Curate
[ ] Harvest

[ ] Tablet (≥768px)

[ ] Browse
[ ] Inspect
[ ] Curate
[ ] Harvest

[ ] Desktop (≥992px)

[ ] Browse
[ ] Inspect
[ ] Curate
[ ] Harvest

[ ] Large Desktop (≥1200px)

[ ] Browse
[ ] Inspect
[ ] Curate
[ ] Harvest

I took a stab at specifying the obvious stuff. Please update/add more specific actions, etc

https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/template/README.md#making-a-progressive-web-app

Probably related to #74 . The UI is not yet structured to be responsive.

Hi,
I'm looking into this issue.
Should we also make the harvest page responsive for mobile size devices?

@gianpaj good point, though I wouldn't spend too much time on /harvest
I would prioritize the other pages first like the homepage, about, browse, definitions, etc

Some work has been addressed in #484 , #486 , #487 and #488
I'm wondering if we need this issue open to track all the work or maybe we can manage it by opening some specific issues.

I'm closing this for now, we can track some other implementations in specific issues.


linux on system z support

I was having issues using phantomJS with linux on system z. Does it not support system Z (SUSE 11 and SLES 11) at all?
The tests we have work fine on other machines but come up with errors on linux on system Z SUSE 11 and SLES 11
running our tests with phantomJS I get:
ERROR: /home/nodebot/jnksws/workspace/angular-live-set/arch/s390x/os/sles-11/node_modules/phantomjs/lib/phantom/bin/phantomjs: /home/nodebot/jnksws/workspace/angular-live-set/arch/s390x/os/sles-11/node_modules/phantomjs/lib/phantom/bin/phantomjs: cannot execute binary file
DEBUG: Process PhantomJS exited with code 126
ERROR: Cannot start PhantomJS
I installed phantom js and ran the tests it is packaged with using npm test and got this error:

<EMAIL_ADDRESS>test /home/nodebot1/elkorep/node_modules/phantomjs
nodeunit --reporter=minimal test/tests.js

tests.js: .FFF.
AssertionError: Test script should have executed and returned run time
at Object.ok (/home/nodebot1/elkorep/node_modules/phantomjs/node_modules/nod                                         eunit/lib/types.js:83:39)
at /home/nodebot1/elkorep/node_modules/phantomjs/test/tests.js:29:10
at ChildProcess.exithandler (child_process.js:758:5)
at ChildProcess.emit (events.js:110:17)
at maybeClose (child_process.js:1015:16)
at Socket. (child_process.js:1183:11)
at Socket.emit (events.js:107:17)
at Pipe.close (net.js:485:12)
AssertionError: 126 == 123
at Object.equals (/home/nodebot1/elkorep/node_modules/phantomjs/node_modules                                         /nodeunit/lib/types.js:83:39)
at /home/nodebot1/elkorep/node_modules/phantomjs/test/tests.js:38:10
at ChildProcess.exithandler (child_process.js:758:5)
at ChildProcess.emit (events.js:110:17)
at maybeClose (child_process.js:1015:16)
at Socket. (child_process.js:1183:11)
at Socket.emit (events.js:107:17)
at Pipe.close (net.js:485:12)
AssertionError: Version should be match
at Object.equal (/home/nodebot1/elkorep/node_modules/phantomjs/node_modules/                                         nodeunit/lib/types.js:83:39)
at /home/nodebot1/elkorep/node_modules/phantomjs/test/tests.js:52:10
at ChildProcess.exithandler (child_process.js:758:5)
at ChildProcess.emit (events.js:110:17)
at maybeClose (child_process.js:1015:16)
at Socket. (child_process.js:1183:11)
at Socket.emit (events.js:107:17)
at Pipe.close (net.js:485:12)
FAILURES: 3/9 assertions failed (137ms)
TAP version 13
1..0
time=155.588ms
npm ERR! Test failed.  See above for more details.

Exit code 126 means that you don't have permission to execute the binary itself (probably something wrong with your setup).
Try to reinstall the package.
I'm closing the issue, because this is not the PhantomJS issue. See also here #11646


Update drug index resolution of MoA and Indications

Updated GraphQL resolving logic to meet specification in ticket opentargets/platform#1308

Indications present on children are also present on parent
MoA are treated as a family of molecules, so parent, child and sibling all have the same MoAs and Linked Targets
Add and expose parentId to model
Expose childIds

I'm confused with the first bullet point: "Indications present on children are also present on parent".
What we would like to have for indications is that the parent indications stay with the parent and the child indications stay with the child. It's the same implementation you had prior to this PR and respects what ChEMBL does on their own website.
Does it make sense?

I'm confused with the first bullet point: "Indications present on children are also present on parent".
What we would like to have for indications is that the parent indications stay with the parent and the child indications stay with the child. It's the same implementation you had prior to this PR and respects what ChEMBL does on their own website.
Does it make sense?

I'm confused with the first bullet point: "Indications present on children are also present on parent".
...
Does it make sense?

Yes, I'll amend. Thanks!

I'm confused with the first bullet point: "Indications present on children are also present on parent".
...
Does it make sense?

Yes, I'll amend. Thanks!


Add support 2018-style macro imports

This PR is a modification of 9a14db2 to use built-in macros correctly in exposed macros.
Fixes #178.

Awesome!
How well tested this is? Do you see any way we could have it automatically tested? Could we add a 2018 crate that we compile from .travis.yml to make sure it keeps working etc.?

added a crate for testing 2018-style macro imports.

A concern is that it is necessary to import __slog_builtin!() when using old-style macro import, since the call of built-in macros will be replaced with it.
There is no effect on the general users (importing macros with just #[macro_use]), but if the users using explicitly macro import (like #[macro_use(slog_o)]) must add __slog_builtin!() to the import list.

So this is technically a breaking change? :(
Well, it's great work. I've added you to project collaborators. I'm having a second child, and looking for increasing the bus factor here.
Maybe there would be a way to have an intermediate release that would  would warn users they have to import this new symbol?
If not, than I guess I'm fine with breakage. Or maybe we should do 3.0. Though the problem with that is it cascades all over the downstream. :/

It might be nervously thought about backward compatibility...
I've accepted the invitation to collaborators.  I'm looking forward to working with you :)
merging this pull request as the first job.

So this is technically a breaking change? :(

Yes, it is and we've run into it already.
It must have been mentioned somewhere (at least in release notes) in order to save time for those who are trying to find the culprit of the broken v2.4.0.


Build fails on Windows 8.1. Upgrade .vsxproj

C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.Cpp.Platform.targets(64,5): error MSB8020: The build tools for Visual Studio 2010 (Platform Toolset = 'v100') cannot be found. To build using the v100 build tools, please install Visual Studio 2010 build tools. Alternatively, you may upgrade to the current Visual Studio tools by selecting the Project menu or right-click the solution, and then selecting "Upgrade Solution...". [node_modules\contextify\build\contextify.vcxproj]

C++ 2010 cannot be installed on Windows 8.1, can it? Where this .vcxproj comes from? Maybe I could help to upgrade it, making it compatible with VS2013..

+1 having the same issue. Have VS 2015 installed, but that doesn't seem to help. I can't find any links to allow me to install VS 2013.

You can still download VS 2013 here: https://www.visualstudio.com/en-us/downloads/download-visual-studio-vs#d-community

closing due to staleness


Line truncation at 197 chars, possible to change this?

Hi there, nice project btw.
I'm using this from Jenkins for most of our infra builds. However, I was wondering if I may either remove/update/make-configurable the truncation of logs coming back from cloudwatch?
Since we do CI/cd for everything, there's frequently fairly long lines (such as CFN updates and the like) and these being truncated is confusing for the average developer who is uninterested in the minutiae of CD pipelines.
Is there any problem or reasoning to not simply raise this this a fair bit (say 2000?). It'd just make the UX a fair bit nicer.

Thank you for this feature request, I have passed it to the team for further review. Your feedback is very much appreciated.

Fixed with cfa1282ca92a4d19192523f2ab3d9253c66d13a1. Thanks for the feedback!


Enhancing CoreAddon with Newer features.

Describe the feature
Enhancing CoreAddon to support Installation of New Addons with following :
Making Version Property Optional to support upcoming Addons Such as ADOT which does have a version number parameter. Also adding a logic to make sure addon creation works with or without version field.
Adding a new field saName to make sure CoreAddon does not populate the AddonName with SA Name. Some Addons require a separate saName for the addon to work perfectly.
Adding a new field namespace field to support creation of Service Accounts in required namespace. Today all Service Accounts needs by Addons are created in kube-system namespace.
Use Case
Enables us to support new EKS Addons and Observability Addons such as AMP, CloudWatch, Xray.
Proposed Solution
Enhancing CoreAddon to support Installation of New Addons with following :
Making Version Property Optional to support upcoming Addons Such as ADOT which does have a version number parameter. Also adding a logic to make sure addon creation works with or without version field.
Adding a new field saName to make sure CoreAddon does not populate the AddonName with SA Name. Some Addons require a separate saName for the addon to work perfectly.
Adding a new field namespace field to support creation of Service Accounts in required namespace. Today all Service Accounts needs by Addons are created in kube-system namespace.
Other Information
No response
Acknowledgements

[X] I may be able to implement this feature request

CDK version used
2.33
EKS Blueprints Version
1.1.0
Node.js Version
18.6.0
Environment details (OS name and version, etc.)
mac

@elamaran11 I assume this has been addressed and published in 1.1.0 and 1.2.0, correct?

@shapirov103 Yes this is addressed now.

Closing this issue @shapirov103


[enhancement] Update to JupyterHub 1.5.0

Description
JupyterHub 1.5.0 has been released and contains a security fix :

JupyterHub 1.5 is a security release, fixing a vulnerability ghsa-cw7p-q79f-m2v7 where JupyterLab users with multiple tabs open could fail to logout completely, leaving their browser with valid credentials until they logout again.

QHub must be updated to use JupyterHub 1.5.0 to include this fix.

This would mean upgrading to use z2jh Helm chart version 1.2.0 - see https://jupyterhub.github.io/helm-chart/

By the way, to fix the vulnerability we (only) need to update the jupyterhub in the user environment:
https://github.com/jupyterhub/zero-to-jupyterhub-k8s/blob/04de3447397bb85b495d0302984d36c61bc75da5/CHANGELOG.md

The new Helm chart is available in my authstate branch if anyone wants to try it. Seems to work so far on GCP.
But for some reason on Digital Ocean, JupyterHub couldn't reach the Keycloak API. Will try again.


iozone: 3.471 -> 3.490

Motivation for this change
update iozone to latest upstream
generate url instead of hardcoding the version in the url
this should make auto-updating possible via nixpkgs-update
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build iozone


Document how transaction service collect offchain signatures (confirmations)

What is needed?
A documentation explaining how confirmation of transactions works and how we collect and store the signatures.

This issue is being implemented on safe-docs https://github.com/safe-global/safe-docs/pull/235


[Suggestion] Additions to No Void Damage to Allies

I've noticed that when the option is enabled, your drones carry over into Void Fields, since they don't die instantly there anymore.
Could I request an addition, or possibly new config option to add onto that which allows broken drones to spawn in Void Fields and Simulacrum. Would be nice to have allies in those places now that they can't take damage to the Void Fog.

Will note this down, though I'm not sure if I'll get the time to implement it anytime soon.


SourceCard: allow entering RHSM bool

This seems to be broken on composer. I create a source with an object of:
{
rhsm: true
...sourceParams
}

No error is reported on creation but on fetch from projects/source/info/ rhsm is set to false.
Can you confirm @bcl @thozza :)
Fixes #1388

Looks like a bug of some sort, it looks like it should be working, store has it copied over, so the state should persist.


[skrifa] tthint tests batch 1

Working toward #803
Covers GETINFO, GETVARIATION, GETDATA, GC, SCFS and MD

I feel like I've been asked to grade the verbal portion of a martian SAT

You and me both?

and it... seems fine I guess?

I’ll take that is “this review has no warranty” :)


cannot locate 'youtube-dl'

Describe the bug
when I use udacimak settoken, I get an error:
throw new Error('ERROR: unable to locate youtube-dl at ' + binPath)
Error: ERROR: unable to locate youtube-dl at
Desktop (please complete the following information):

OS: macOS 10.15.1
Node version: v13.1.0
npm version: 6.12.1
Udacimak Version: when I run udacimak --version, I get the same error

I followed the install guide on the wiki page.

That's a strange issue. Could you try on a different machine to see if it works?

I can not solve this issue. I'm using the docker version of udacimak. However, I don't know the default download folder path. I'm using docker container run -it -v "$(pwd)/:/downloads" udacimak/udacimak download xxx. Any suggestions?

I run this command  docker container run -it -v "/Users/wangtongtong/Downloads/Learning/MOOC/Udacity" udacimak/udacimak  download --targetdir /Users/wangtongtong/Downloads/Learning/MOOC/Udacity ud827 ud359 cs271 ud600 ud730 cs373 ud501
I can download JSON data. But I don't know where these data is. There is nothing in the /Users/wangtongtong/Downloads/Learning/MOOC/Udacity folder

I think the wiki page needs to clarify the default download and render folder paths.

I run this command  docker container run -it -v "/Users/wangtongtong/Downloads/Learning/MOOC/Udacity" udacimak/udacimak download --targetdir /Users/wangtongtong/Downloads/Learning/MOOC/Udacity ud827 ud359 cs271 ud600 ud730 cs373 ud501
I can download JSON data. But I don't know where these data is. There is nothing in the /Users/wangtongtong/Downloads/Learning/MOOC/Udacity folder

You need to be familiar with Docker Volume to get the data to your local machine. Though I've seen Docker on Windows having issues with volume before - it doesn't connect to the volume properly - but I'm not a DevOps expert so I'm not sure how to resolve that.

Me too, I am having this issue on my windows 10. same kind of issue.

The same issue installed with npm install -g udacimak

@udacimak is it working on your machine when you install from npm package?

@vsay01 yes it works on all my computers. I haven't been able to replicate this issue yet. I'd suggesting trying using nvm, Docker image, changing OS. Let me know if any of those work?

@udacimak I use udacimak docker and it's working. For now I'll keep using docker version, Thanks


why i have to reconfigure the config of the code formatter when i start a new project

so, i think once i configured the config then it will be the default config of idea, but it doesn' work. every time i start a new project i must reconfigure again for the new. i think it is troublesome.  maybe the user can choose config all project or config this project. thanks

you can configure it for a default project, when you close all projects

i have a try ,thanks


Need to update the policy path details under the section "Use Group Policy to manage the update location"

Under the section "Use Group Policy to manage the update location",
The policy path details mentioned in the article is valid only till Windows 10, version 1607. It has changed starting Windows 10, version 1703 and remains the same till Windows 10, version 1809 and then again changed in Windows 10, version 1903.

@nenonix  Minor copy editing is needed for this PR. Thanks.

@Dansimp The PR provides a new clarifying note explaining the respective policy paths per Windows version. Please review as the time permits. Thanks

@Dansimp Following up for updates.

@Dansimp Following up for updates.

Just noticed the update. It could have been better if the points were separate. Right now it is one paragraph as follows:
For Windows 10, versions 1703 up to and including 1809, the policy path is Windows Components > Windows Defender Antivirus > Signature Updates For Windows 10, version 1903, the policy path is Windows Components > Windows Defender Antivirus > Security Intelligence Updates
Proposed edit:
For Windows 10, versions 1703 up to and including 1809, the policy path is Windows Components > Windows Defender Antivirus > Signature Updates
For Windows 10, version 1903, the policy path is Windows Components > Windows Defender Antivirus > Security Intelligence Updates

Sure. But to achieve that line break or NewLine, you either need to add </br> at the end of the first paragraph, or insert an extra blank line between the paragraphs.

Okay, so how should I add this edit?

Sorry for the delay. The simplest way to add the necessary line break would be like this:
> [!NOTE]
> For Windows 10, versions 1703 up to and including 1809, the policy path is **Windows Components > Windows Defender Antivirus > Signature Updates** </br>
> For Windows 10, version 1903, the policy path is **Windows Components > Windows Defender Antivirus > Security Intelligence Updates**

Note (sic) that the </br> is mostly only needed after the first sentence (line 2 out of 3).


[FEATURE] Add support for changing storage size of seed node

Is your feature request related to a problem?
The EBS volume size of seed node can't be configured and is hard-coded to 50GB:
https://github.com/opensearch-project/opensearch-cluster-cdk/blob/d3b12a8774835f1a6fdf41d2ea00b8e9fb7f8b8e/lib/infra/infra-stack.ts#L213
When deploying a cluster with no cluster manager node, the seed node will have data role:
https://github.com/opensearch-project/opensearch-cluster-cdk/blob/d3b12a8774835f1a6fdf41d2ea00b8e9fb7f8b8e/lib/opensearch-config/node-config.ts#L25
It means data will be stored evenly on the "seed" node and other data nodes. Because the storage size of other data nodes can be configured and is 100GB by default:
https://github.com/opensearch-project/opensearch-cluster-cdk/blob/d3b12a8774835f1a6fdf41d2ea00b8e9fb7f8b8e/lib/infra/infra-stack.ts#L130
The storage size of the seed node will be a bottleneck.
What solution would you like?
Make the EBS volume size of the seed node configurable.
What alternatives have you considered?
n/a
Do you have any additional context?
n/a

@rishabh6788 , pls have a review and comment.

@tlfeng you can use --context dataNodeStorage to set the storage size on data node. Please let me know if that helps your case.


C cpp: DFS One shot traversal

Description 📜
This PR contributes documentation explaining a unique approach for BST traversal. All of preorder, inorder and postorder traversals are performed in one go using a unique method explained.
Fixes #5068

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] C/CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@Sunainacode hello dear , i have one little confusion could you please help me out .
actually I have to put relative links in my md file like you did . but i am not able to do that . i tried but when i click on the link ..it directly start downloaded and not opening in GitHub repo .. so could you please help me out how to do that ??

@apeksha235 done maam

@Suvraneel done sir. I didn't know about this feature of markdown. Thanks for sharing this. It makes documentation looks so better😊


Error: Cannot destructure property 'blockChain'

[12/14/2024, 6:15:55 PM] [INFO]: === Bridge Transaction from ARBITRUM to BASE value 0.0001 ===
[12/14/2024, 6:15:57 PM] [INFO]: Creating Bridge Request using ID: 9bd57e86-36cd-4142-8d90-cfe67d6fea97
[12/14/2024, 6:15:57 PM] [INFO]: Bridge Max Output Amount: 0.000093634297980989
[12/14/2024, 6:15:58 PM] [ERROR]: Error: Cannot destructure property 'blockChain' of 'createData.transaction' as it is null.
[12/14/2024, 6:15:58 PM] [INFO]: === Santuy, Cooldown 40 Seconds before continue! ===
[12/14/2024, 6:16:38 PM] [INFO]: === Bridge Transaction from BASE to ARBITRUM value 0.0001 ===
[12/14/2024, 6:16:39 PM] [INFO]: Creating Bridge Request using ID: dab68c58-e89f-4df5-a713-41f1bd407a66
[12/14/2024, 6:16:39 PM] [INFO]: Bridge Max Output Amount: 0.000082544403547710
[12/14/2024, 6:16:40 PM] [ERROR]: Error: Cannot destructure property 'blockChain' of 'createData.transaction' as it is null.
[12/14/2024, 6:16:40 PM] [INFO]: === Santuy, Cooldown 40 Seconds before continue! ===

?

 ?

npm install - run this in console

@MrTimonM make sure balance is enough,


Pageserver reconnect v2

Problem
Compute is not always able to reconnect to pages server.
First of all it caused by long time of restart of pageserver.
So number of attempts is increased from 5 (hardcoded) to 60 (GUC).
Also we do not perform flush after each command to increase performance (it is especially critical for prefetch).
Unfortunately such pending flush makes it not possible to transparently reconnect to restarted pageserver.
What we can do is to try to minimzie such probabilty.
Most likely broken connection will be detected in first sens command after some idle period.
This is why max_flush_delay parameter is added which force flush to be performed after first request after some idle period.
See #4497
Summary of changes
Add neon.max_reconnect_attempts and neon.max_glush_delay GUCs which contol when flush has to be done
and when it is possible to try to reconnect to page server
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Also we do not perform flush after each command to increase performance (it is especially critical for prefetch).
Unfortunately such pending flush makes it not possible to transparently reconnect to restarted pageserver.

Hm, why can't we? It seems that if we track in-fly get_page requests we can retry them. E.g. stash outgoing requests in some queue and delete requests from it when we got a successful response. Once disconnect is detected it is possible to retry from the start.

Also we do not perform flush after each command to increase performance (it is especially critical for prefetch).
Unfortunately such pending flush makes it not possible to transparently reconnect to restarted pageserver.

Hm, why can't we? It seems that if we track in-fly get_page requests we can retry them. E.g. stash outgoing requests in some queue and delete requests from it when we got a successful response. Once disconnect is detected it is possible to retry from the start. And no need to change anything in flushing logic -- it is still possible to write in socket without flush. If we delete from that queue if and only if we got a response then we don't care what was sequence of flushes, how much reconnects, etc.

Well, it is non-trivial to implement.
So adding @MMeent  as reviewer

That's code that drops an existing prefetch request for this block. It should remove the request it found from the prefetch system.
Sorry, I do not understand you.
I am asking about the case when *force_lsn == slot->effective_request_lsn
None of two blocks which drops prefetch request is applicable in this case.

I failed to make work my version with resending perfetch requests,
or some reasons backend is blocked in the loop neon_read_at_lsn waiting for prefetch result pagestore_smgr.c:1050
This is why I restored original aproach with resetting prefetch state on reconnect.
But I now handle errors not only in send but also in flush.
So the error reported in https://github.com/neondatabase/neon/issues/4497
should not happen now.


Fixed race condition when changing a notification after sending

Howdy, upstream contributors!
We have been using this awesome repo. I ran into a race condition yesterday in a test. Looked into it a bit and found that the notification was passed through the channel as a pointer and the creator modifies it after. This causes a race with the reader.
Made an one line change and would love to hear what you think! :)

Thanks, but sarama-cluster is now deprecated in favour of https://github.com/Shopify/sarama/pull/1099

Thank you both for fixing my broken code! :)


Update the expression browser example

At the time of this commit when Prometheus' latest release is 2.3.1, http_requests_total expression is no longer valid. The expression example was already updated in the server documentation below:
Prometheus Server Doc: Getting Started
Signed-off-by: Travis Johnson<EMAIL_ADDRESS>

Looping in maintainer as per the contribution guideline<EMAIL_ADDRESS>

This is a duplicate of #1078.

Obsoleted by #1078


toDate(_: ) return value wrong

Hey guy, if you try the follow code, date is "1986-03-02 23:00:00 +0000" instead of nil.
let date = "31/02/1986".toDate(DateFormat.Custom("dd/MM/yyyy"))

The follow code is how the date should be checked:
func toDate(format: String = "yyyy-MM-dd'T'HH:mm") -> NSDate? {
   let dateFormatter = NSDateFormatter()
   dateFormatter.dateFormat = format
   return dateFormatter.dateFromString(self)
}

I think it is a specific behavior of UIKit and not a bug, if you notice Feb 1986 has 28 days (+3 days) = 03 March. It try to "fix" your input date, probability it works only with leap days in feb.

The problem is that I should know if the passed string date is valid or not. Return nil should be the logic to let you know that.
Anyway, I have another question. How come the toDate(_:) method was removed from version 4.x of SwiftDate? Which one should I use now? Thanks in advance for your answer.
Carmelo

I agree but we cannot override the default behavior or UIKit without rewriting the entire logic and this is not in our plan.
We have just made a site with examples and docs about SwiftDate; you can found datetime parsing functions here about date time parsing here.
Basically you can call:
let str = dateString.date(format: DateFormat)

Thanks for the code. I've already tried that but I got a DateRegion object instead of NSDate or Date. Any clue?

In fact we have missed to make public .absoluteDate or .date property. I've scheduled it for the next minor update 4.0.3.

I see. Well, thanks for the info and let me know when that version is out ;)
Cheers!

Sure.
Just follow issue #267

Hi @malcommac, what is the new format of toDate(DateFormat.ISO8601Format(.Extended))?
I'm using
createdAt = try json["createdAt"].stringValue.date(format: DateFormat.extended).absoluteDate

but always failed. Also, there is some other way to get the date without the try and catch?

You can use .withInternetDateTime:
let date4 = DateInRegion(string: "2001-02-03T15:30:00+01:00", format: .iso8601(options: .withInternetDateTime), fromRegion: regionRome)

I tried but always failed.

Ops my mistake, I did not see seconds.
Let's try:
let test = "2016-08-05T18:20:24.388Z"
do {
  let d = try test.date(format: .custom("yyyy-MM-dd'T'HH:mm:ss.SSSZZZZZ"))
  print("Parsed date is: '\(d)'")
} catch let err {
  print("Ops, failed to parse date: \(err)")
}

I thought there was some default settings for that format. I knew I could use the custom one :)
Thanks for your time man. Chat to you soon!

It could be a new option in ISO8601DateTimeFormatter class. See #293

Cool! You rock!


fix(overview): change slack channel name and typo fix

Fixes

https://issues.redhat.com/browse/KFLUXBUGS-1120

Description

Update slack channel and fix typo for Red Hat.

Type of change

[x] Bugfix

Browser conformance:

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

/retest

@rohitkrai03 unit tests are failing

/retest

/retest

/lgtm

/retest

/retest

/retest


BROKERS: Get Labs Data

Closes Task #39590625: BROKERS: Get Labs Data

Wrote a little stub test in the HomeController (that is not committed) to test hitting the live endpoint to get available devices and it worked :)


Query support for *

We have support for multi server queries in the API, but not in JS.
This should work for parity with our HTTP API.
module.exports = function(server) {
  var q = server.from('*').where('type = "photocell"');
  server.observe([q], function(device) {
    console.log(device);
  });
}

Hint: https://github.com/zettajs/zetta/blob/master/lib/runtime.js#L122

See: #199


Removed extra whitespace before and after snippets

This handles whitespace before and after all triple ticks with some
logic to make them format properly to address #36

@ReedCopsey before we have a chance to dig into the R&D aspects of technical writing, maybe it is worth to create a Q&D (quick and dirty) /scripts/quickfixes/trim.all.empty.lines.fsx with whatever you used to accomplish the job?
This might come handy if there are other waves of documentation we want to fix this way.
Nobody would judge on the code :)
And thanks for the PR / huge cleanup!

@smoothdeveloper Hopefully nobody would judge - if there's a desire to add these somewhere, all @dend has to do is ask - would be happy to throw it in there.  It's super ugly, though - definitely not my normal style of F#.

@smoothdeveloper @ReedCopsey I'll just create a new repo for doc tools that we can use - there is a number of those that I built myself that might come in handy.
That said, we are always working on improving our migration tools, so hopefully we'll have less issues moving forward (given that we already found a number of those).


Show default Badge instead of Exception, when calling badges (only composerlock and gitattributes) for non existing repositories

Steps to reproduce:

Call composerlock or gitattributes badge for a non exisiting repository e.g.:
https://poser.pugx.org/foo/bar/composerlock
A badge with an Exception will be generated

I have applied your patch on it: #205