Add support for script comment/description

I have been playing around with wire it and I love it!
I have always wished for package.json  comments, but we don't have them even in 2024!
Many times we have commands that are too complex, and an explanation would be of great help.
What about adding a comment or description prop to the wireit scripts?
I know we can add any prop to the script object that is unrelated to wireit and wireit won't mind those, but my suggestion is primarily to have best practices in place and allow auto completions via VS Code extension.
Something like this
"wireit": {
	"cleanup": {
		"comment": "Delete the paths that are not version controlled.",
		"command": "rm -rf some/path/* && rm -rf some/other/path/* && rm -rf some/other/related/path/*"
	},
}

I like description. We could also display it when you hover over a dependency.

even if you just reserved the "description" key for use only by comments, without any added functionality, it would be good. I am already using "description".
Ideally the CLI should support some way to show the description for every wireit-based command, optionally showing descriptions recursively for all dependencies. Perhaps that's what should happen when one executes wireit directly, as opposed to via npm run.

I like description. We could also display it when you hover over a dependency.

I think description and comment are two different things. One provides a description of the command, to be shown on hover as you suggest, or via CLI as I suggested in my prior comment. But comment is a note about the implementation, perhaps a reminder to self of why it is done the non-obvious way it is done.


Remove keyring entirely and use config json

This removes entirely the fancy keyring integration in favour of a much simpler config file.
Why? Every time I come back to this tool to iterate on it, a weird native interface library is broken and it won't build. Beyond that, it means this tool needs binary builds (for code-signing) which is awkward and impractical.
This would be a very breaking change! Closes https://github.com/buildkite/cli/issues/114.

I built your branch, and it worked great

Thanks @lox. I mostly agree, and err on the side of development productivity.
How painful do you think it is to have tokens in files? Is there another way to do keychain tokens nicely? Every time I explore that space it seems pretty spicy.
This repo also doesn't have a clear caretaker at the moment. I suspect it would be the decision of whoever wants to pick this up and start iterating on it. So this might need to wait for that moment.

Yeah, seems reasonable.
I think this project needs to be as easy to occasionally-maintain as possible, so this would be a win.
I wonder if there's anything else that should be bundled into a v2.x release, given we'll probably need to bump major version for the breaking change 🤔
I guess we can merge in the mean time, to at least get this on main.

Rebased onto main to get https://github.com/buildkite/cli/pull/131 and added a small README tweak.
@moskyb and I have tested it locally, including the whole GraphQL token and GitHub OAuth flow, verifying the keys end up in the config file, and can be used to list pipelines 👌🏼

Hrmm… not sure if this is relevant to this PR, but I thought the removal of keyring would ideally make this not happen when running bk --help:

I think this is just macOS these days… it's a downloaded binary, so it “warns” the user.
xattr -d com.apple.quarantine <filename> did the trick.


UI: group similar violations

Current situation
When validating large APIs, you often get many similar violations, just for different locations. In the UI this currently looks like this (Nakadi's API used as an example):

This makes the list of violations quite long, while there essentially are only 11 different MUST rules violated here.
Suggestion
Violations with the same title (and rule) should be grouped together. When (as here) they also have the same detail message, we should just have a list of locations (+ a total count).

When they have different details (for cases where the details message is parameterized, like scope ... does not match regex  '^(uid)|(([a-z-]+\.){1,2}(read|write))$', it's a bit more complicated.
Current state:

Simple solution
Just aggregate those with the exact same detail message), with a list of locations, leave ones with different details alone.

More complex solution
Have a second level aggregation by detail under the top-level aggregation.
This likely needs a slight rearrangement, moving the rule link to the top:

(I've mocked up this by editing the DOM in Chromium's dev tools, with proper styling it can look better.)

Could be also seen as a kind of violation optimization, that is handled by the server. Violations are than aggregated outside the rule engine and translated into violations that allow multiple locations.


[request] new method to save the fp32 model

Currently checkpoint saves the original fp32 weights in zero_pp_rank_* files, which makes it inaccessible to users who may need the final result to be used elsewhere w/o Deepspeed (e.g. inference or a different system).
So we need deepspeed.save_model() which will reconstruct the master weights and save them as model.bin.
For context for this need please see: https://github.com/microsoft/DeepSpeed/issues/797
Thank you.

@ShadenSmith shared elsewhere that each fp32 param group is stored in a flattened buffer in the fp16 optimizer here and needs to be exposed as a method.

Any chances this new method to save the fp32 model can be added?
Users are reporting struggles when they customize their saving/loading of the checkpoints
https://github.com/huggingface/transformers/issues/10789
I think there is also a problem to switch from zero2 checkpoint to zero3, correct? Which means that we really need to salvage the fp32 model weights from the zero2 checkpoint before it can be loaded in zero3.
@ShadenSmith, @jeffra

PR fix solution: https://github.com/microsoft/DeepSpeed/pull/892


[GameStudio] Can't close c# tab

Release Type:
official
Version:
beta01-0441
Platform(s):
windows
Describe the bug
c# tabs connot be closed, there is no 'x' and the context menu doesn't have a close option. not even 'close all' does anything.
To Reproduce
Steps to reproduce the behavior:

double click a c# file in the asset view

Expected behavior
normal tab behavior.
Screenshots

looks like #286

Does it happen on a specific project (could be because dock layout is serialized without the CanClose) or also on new projects?

confirmed the project was saved with an older beta, a newly created project doesn't have this issue. might be low prio... do you know how to fix it in the project files? for me this would suffice as a fix.

Workaround:

Open %AppData%\Xenko\GameStudioInternal.conf
Delete the DockingLayout and DockingLayoutEditors for your project.

yes, this works. would it make problems to delete the entire file?

@tebjan no. It will recreate the file with default values the next time you open the Game Studio.


EFR32: PSA crypto validations fixed.

Problem
Multiple unit tests fail due to validation issues.
Change overview

Validations added or corrected in CHIPCryptoPALPsaEfr32.cpp and Efr32PsaOpaqueKeypair.cpp.

Testing
Tested on BRD4164A (MG12) and BRD4186A (MG24).

1.0 accepted: platform specific change.


How to use GAT to generate embeddings to graph nodes

Hi, Petar
Firstly, thanks for your great works!
I want to use GAT for link prediction, my datasets only can give the adjacency matrix as input for GAT. Node doesn't have explicit features like Cora dataset. My question is if I can generate embeddings (features) for each node of the graph, for example, can I assign a unique one-hot vector for every node as input feature?
Thanks

It seems that one hot input is not that appropriate. You can view each dimension as a channel and when you use one hot vector as the input feature, it would be far too sparse for information to aggregate in a single channel.
You can use row vector in Adjacency matrix as feature vector. Besides, I would suggest you to use some random walk based embedding technique to generate the initial embedding. (Node2vec etc.)


UI improvements

In general, the UI should

Not show links that cannot be used
make a clear "affordance" to show when things can be clicked on

To that end, the create project does not belong on the header (logged in or not) and that clicking on the username at the top right  opens a profile page is also not obvious.  The following change is suggested and was approved by Jens 3/28/23 ~1:30pm PST.

Add an icon next to the username (either gear or downward triangle) to indicate something happens when you click the username.

When you click the username, open a menu with the following options;
Profile
Create new project
Logout

These links behave the same as the current links for the profile page, logout, create but are just relocated to the menu, and the menu (and gear icon) are not displayed to users who are not logged in

checked into branch issue_36.  Pull request created


Is it possible to limit the colour

I can see in colours.js you are doing a lot of work with the saturation and lightness. We would like to limit the colours used by e.g. sending some hex values or equivalent. Is this possible?
Thanks in advance

It depends on the degree of control over the colors you need. It is possible to limit (or extend) the saturation and lightness of the generated colors. You can find an example how to do this in this example. However there is currently no option to control the hue of the colored shapes.

Thanks for your reply. It is the hue of the coloured shapes that we would want control of in order to conform with brand identity - although I realise it would probably need a lot of colours in order to make unique identicons. It looks as though it may be possible if I were to fork and extend the library so I might give that a go soon!


Explore text() function to check if it can be improved for more websites - to check

For eg https://www.worldometers.info/coronavirus/
The current logic is return document.body.innerText || document.body.textContent
But document.documentElement.innerText will grab everything. To review how it can retrieved better.

dump `text()` to result.txt

dom return document.body.innerText
dump `dom_result` to body.txt

dom return document.documentElement.innerText
dump `dom_result` to documentElement.txt

All above yields the same results --> result.txt

documentElement includes stuffs like header which shouldn't be included for text extraction.
Will need more examples why document.documentElement.innerText works better than document.body.innerText


Add second AIP for GHO Cross-Chain Launch

https://governance.aave.com/t/arfc-gho-cross-chain-launch/17616

@miguelmtzinf could you fix the lint as well


docs: Update usage instructions in README.md

I made a change to the bundle step.
bundle install GEM is not valid as it does not take a Gemfile name.
bundle install jekyll-optional-front-matter
ERROR: "bundle install" was called with arguments ["jekyll-optional-front-matter"]
Usage: "bundle install [OPTIONS]"

You need just to say bundle install as in my doc change and it will read from the Gemfile.
Or you can use use add as bundle add GEM. But, the add commands appends a line to the Gemfile which is already covered in step 1. Also I don't find add useful at all. So I went with install.
I also cleaned up other steps so that I think make more sense now. I hope those changes are okay.
Thanks

View rendered README.md

Thanks for approving and merging. Glad to contribute.


Investigate using GPU rendering for Skia Scenario tests on iOS

Today these use --enable-software-rendering, presumably to avoid pixel differences on different GPU models in the fleet.
If we do this, it may be necessary to configure the build rules so they only run on one GPU model.
As part of re-enabling them, it may be worthwhile to see if we can upload the images to gold rather than doing the comparisons as part of the test harness. If we did that, we could add the GPU model as one of the dimensions.

Related to https://github.com/flutter/flutter/issues/131888


 I clicked on the 'challenge' and when updated the 'map' of my progress was gone as a rollback. I was missing just do Bonfire 'Simon Game and Zipline' Pinterest Clone '

Lets keep the discussion in #3630

I ate boo boo from a small Labrador retriever.
-----Original Message-----
From: "Berkeley Martinez"<EMAIL_ADDRESS>Sent: 10/12/2015 2:02 PM
To: "FreeCodeCamp/FreeCodeCamp"<EMAIL_ADDRESS>Subject: Re: [FreeCodeCamp] Lose my progress while was testing the 'Claim YourFront End Development Certificate' (#3576)
Closed #3576.
—
Reply to this email directly or view it on GitHub.


Link button with advice

Part of this story on Pivotal.
Link buttons were being rendered with a large margin on the top. This
was fine if there was no advice associated with the button, but meant
that the advice and the button looked ungrouped if there was.
By setting advice to true when using the template, it reduces the margin
to 15px and makes the advice and button look much more cohesive.
Before:

After:

We need this exact same thing for green submit buttons for the new framework agreement signing flow. I'll follow the pattern here (pending approval from @laurenceberry)

Would be super nice if the example one was a bit clearer about what it actually does by including some text (i.e. advice) above it. At the moment the examples look like they do the same thing (spare the different class)


[Build] Remove orphan file

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[X] Other (Build)

Description
MiMaExcludes.scala theoretically has being renamed to SparkMimaExcludes.scala:
https://github.com/delta-io/delta/pull/1952/files#diff-e7ee88f4ceae7019f7bff3eef41ee5cee6da1e54c436df887e3fd4c96c282eda
For some reason both files exist, the former not being used.
@allisonport-db
How was this patch tested?
Unit Tests
Does this PR introduce any user-facing changes?
No

UPDATE: Found the reason. PR renames it, but commit adds a new file instead:
PR:

Commit:


Allow custom help commands to override the auto-generated help command

Hello,
This change allows the auto-generated help command to be overriden by a help.js file in a command directory.

Wow that's actually a really simple fix. At first I thought it wasn't complete lol.


[5h] Spike to investigate feasibility of a BOSH broker

In order to determine the feasibility of using BOSH to dynamically provision managed services via a broker, we want to spend up to 5 hours prototyping to learn more.
Acceptance criteria

[ ] We attempt to create a broker that presents service plans corresponding to a configured set of BOSH manifests and manages instance lifecycles for them, and talk about how far we got.
[ ] We're able to make a decision about whether this is a viable path forward, and we can write stories for what would need to be done to do it properly

@sushiandbeer Can you involve the person who's working on this from the Altoros side in this issue for visibility?

@mogul is this still "in progress"?

Yes, I asked @s-matyukevich to post info on what he's done here.

(He said in today's standup that he's got it working!)

@s-matyukevich, in case you didn't see it, please also check out 18F/cg-product#13 for additional context about what we're after here... Wasn't sure if you've seen the Brooklyn stuff or the CF Summit talk.

@mogul  You can find the broker here https://github.com/s-matyukevich/bosh-broker

Tagging @jmcarp to review and give feedback/PRs.

Update: I read the code a week or so ago and thought the overall approach made sense. If we're going to continue with this, the next step would be https://github.com/18F/cg-product/issues/52.

@jmcarp should we close this out then?

Sounds good to me--closing.


Result-based error handling instead of Ctxt

@oli-obk

I'm not very happy with the Ctxt structure and it's usage. I'd much rather have regular Result type error reporting, but we can leave that to a future refactoring. Maybe the code gets horribly cluttered with Result

The point was to batch up multiple errors like we had when we used span_err with libsyntax. If I have a loop (for example parsing meta items) that produces a few errors, and maybe an error coming from somewhere else, the Ctxt provides a central way to batch those up into a single readable message.
Assigning to Oliver because I already failed to come up with a better way :smile:

I don't think this is going to happen.


Back end specific project add/remove

API endpoint to add/remove a backend asset to/from a specific project

For adding a backend from a specific project:
POST    /Network/{hubName}/Groups/{groupName}/Projects/{projectName}/devices
Data required:
{"name": "<backend_name>", "priority": <int between 1-10000>}
For deleting a backend from a specific project:
DELETE    /Network/{hubName}/Groups/{groupName}/Projects/{projectName}/devices/{deviceName}


Update DF-MP2.py

Description
Please provide a brief description of the PR's purpose here.
What are your new additions? Please provide a brief list.

New Features

[ ] Feature1
[ ] Feature2

Changes

[x] Change1

Any questions for the community?

[ ] Question1

Status

[x] Click when ready for review-and-merge

Fixed such that it prints SCS-MP2 correlation energy instead of same-spin MP2 correlation energy.

Great thanks, this looks good. Can you edit your post with the issue template with the required information and delete fields that are not relevant?

What's with the travis CI build check failed? I only changed the one line....

Looks like a CI error rather than anything you changed. A restart should remediate the issue.

Codecov Report

Merging #81 into master will not change coverage.
The diff coverage is 100%.


Elements with multiple occurrences of same child do not show child as +

For example:
  <key name="CopyFile.RECENT_KEYS">
    <recent name="$PROJECT_DIR$/dtdgen" />
    <recent name="$PROJECT_DIR$/original" />
  </key>

The original Java code lists the key element as:
<!ELEMENT key ( recent+ ) >

whereas the Python code produces this:
<!ELEMENT key ( recent )* >

Work backward from the output

dtd_generator.py:72 shows that ed.sequenced is false.  Should be true.

Debugging both versions with breakpoint at the start of print_dtd.

Java elementList["key"].sequenced is true.
Python element_list["key"].sequenced is False.

In endElement(), there is a difference.

In Java line 474: ed.sequenced is true
In Python line 270: ed.sequenced is False

Good (using bisect logic)
In startElement:

In Java line 412, isFirstInGroup is true
In Python line 217: is_first_in_group is True
The stack entry "parent" is the same in both programs

Good
At the end of startElement:

In Java, line 460
In Python, line 260
se: StackEntry is the same in both programs
ed: ElementDetails is the same in both programs

On continuation, we end up back in startElement again (not endElement yet)

Bad
At the end of the first <key>...</key>, in endElement:

In Java, line 474, ed.sequenced is true
In Python, line 270, ed.sequenced is False


Add eventDispatcher task and solution

Direct to #35

Can you rebase your master... i've fixed the builds

Thanks


az 'aks' command errors with v2.30.0

azure-cli                         2.30.0
core                              2.30.0
telemetry                          1.0.6
Extensions:
aks-preview                       0.5.42
any az 'aks' command now complains of the following:
The command failed with an unexpected error. Here is the traceback:
cannot import name 'Iterable' from 'collections'<EMAIL_ADDRESS>Traceback (most recent call last):
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
cmd_result = self.invocation.execute(args)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/init.py", line 582, in execute
parsed_args = self.parser.parse_args(args)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/knack/parser.py", line 260, in parse_args
return super().parse_args(args)
File<EMAIL_ADDRESS>line 1820, in parse_args
args, argv = self.parse_known_args(args, namespace)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/parser.py", line 284, in parse_known_args
self._namespace, self._raw_arguments = super().parse_known_args(args=args, namespace=namespace)
File<EMAIL_ADDRESS>line 1853, in parse_known_args
namespace, args = self._parse_known_args(args, namespace)
File<EMAIL_ADDRESS>line 2065, in _parse_known_args
stop_index = consume_positionals(start_index)
File<EMAIL_ADDRESS>line 2021, in consume_positionals
take_action(action, args)
File<EMAIL_ADDRESS>line 1930, in take_action
action(self, namespace, argument_values, option_string)
File<EMAIL_ADDRESS>line 1209, in call
subnamespace, arg_strings = parser.parse_known_args(arg_strings, None)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/parser.py", line 284, in parse_known_args
self._namespace, self._raw_arguments = super().parse_known_args(args=args, namespace=namespace)
File<EMAIL_ADDRESS>line 1853, in parse_known_args
namespace, args = self._parse_known_args(args, namespace)
File<EMAIL_ADDRESS>line 2090, in parse_known_args
self.error(('the following arguments are required: %s') %
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/parser.py", line 160, in error
recommender.set_help_examples(self.get_examples(self.prog))
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/parser.py", line 201, in get_examples
return self.cli_help.get_examples(command,
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/_help.py", line 195, in get_examples
else self.group_help_cls(self, delimiters, parser)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/knack/help.py", line 253, in init
child.load(options)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/_help.py", line 306, in load
loader.versioned_load(self, options)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/_help_loaders.py", line 154, in versioned_load
super(CliHelpFile, help_obj).load(parser)  # pylint:disable=bad-super-call
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/knack/help.py", line 198, in load
description = getattr(options, 'description', None)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/knack/parser.py", line 240, in getattribute
self.description = self._description()
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 133, in description_loader
op = self.get_op_handler(self.op_path)
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 59, in get_op_handler
handler = import_module(mod_to_import)
File<EMAIL_ADDRESS>line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/Users/tnaish/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 44, in
from tabulate import tabulate  # pylint: disable=import-error
File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/tabulate.py", line 16, in
from collections import Iterable
ImportError: cannot import name 'Iterable' from 'collections'<EMAIL_ADDRESS>

route to service team

I'm having the same problem as well.

This is an issue in one of the dependencies used by Azure CLI (tabulate).
The issue was fixed in this commit: https://github.com/astanin/python-tabulate/commit/5b7d1ee98631f1909d2f7668746525dcb76e287d
The way the comparison was done with python_version_tuple seems to not work with 3.10.0 (I suppose it assumes the version is 3.1.0).
I manually modified the file with the fix and the command started working again.

Duplicate of #20348


Project Not Successfully Building, Even Though Packages are in the packages folder

My project uses a packages.config and I double-checked to make sure the packages were showing in the packages folder, which they are, but I still get the error:
CreatureStatues.csproj(167, 9): This project references NuGet package(s) that are missing on this computer. Enable NuGet Package Restore to download them. For more information, see http://go.microsoft.com/fwlink/?LinkID=322105.The missing file is packages\ILRepack.Lib.MSBuild.Task.<IP_ADDRESS>\build\ILRepack.Lib.MSBuild.Task.targets.
The file is there, I just don't get why the project isn't finding it.

Issue has been resolved. Error was packet path directory typo.


Block Entity Data is serverbound and clientbound

https://sel-utils.github.io/protocol/pocket113/play/block-entity-data
While in the docs it says only the server sends this packet, actually the client also sends this packet when writing a sign. 😉

Thank you!


JadbException e ;List devices = jadb.getDevices();

please, I have a question: when I use the funcation List devices = jadb.getDevices() it catch that JadbException??

you probably already figured this out but functions can throw exceptions not catch them so you would have to catch that jadb Exception

Can you clarify your question? What is the exception message? Please provide full stacktrace of the error.


Channel topics overlapping with member list.

Channels with really long topics overlap with the member list.

Weird, cannot reproduce...

Maybe you have some custom CSS that may affect the channel info's width? Or maybe a plugin?

I've got no custom CSS and it still happens with all plugins disabled.

May I know what resolution and zoom level your Discord is running at?

Running full screen on my desktop (1920x1080) and my zoom level is 100.

I really can't reproduce this, I tested it on a 1080p monitor and the overflow works.

Issue seems to be fixed after a Discord update(?) Will close the issue now.


[BugFix] fix unstable UT testRollbackExceptionOnSetupCluster

This UT is meant to simulate the case where a newer FE has way too many new logs than the remaining ones, yet after retries several times, there were small chances to recover.
Set retry times to 1 to prevent recovery after retry.
What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)


Android instalation errors solved

Hello, I really like your library! I only had a few problems in installation of library at android app, but after some tests I find a way to fix just changing a configuration, if be useful for another people following this steps to configurate the library on android:

Open up android/app/src/main/java/[YOUR_PROJECT_NAME]/MainApplication.java

Add import com.killserver.screenshotprev.RNScreenshotPreventPackage; to the imports at the top of the file;

Append the following lines to android/settings.gradle:
include ':react-native-screenshot-prevent'
project(':react-native-screenshot-prevent').projectDir = new File(rootProject.projectDir, 	'../node_modules/react-native-screenshot-prevent/android')

Insert the following lines inside the dependencies block in android/app/build.gradle:
  compile project(':react-native-screenshot-prevent')

Hello. Thx) But .. this writes in "manual installation" block)

Hello, I put this because have small details that generate errors in the installation, for example instead MainActivity.java I put here MainApplicationJava, and not add new RNPreventScreenshotPackages. In import of RNScreenshotPreventPackage, instead com.killserver.RNPreventScreenshotPackage; I put com.killserver.screenshotprev.RNScreenshotPreventPackage; because the path package have a folder screenshotprev missing in documentation;

Hello, I put this because have small details that generate errors in the installation, for example instead MainActivity.java I put here MainApplicationJava, and not add new RNPreventScreenshotPackages. In import of RNScreenshotPreventPackage, instead com.killserver.RNPreventScreenshotPackage; I put com.killserver.screenshotprev.RNScreenshotPreventPackage; because the path package have a folder screenshotprev missing in documentation;

thx, fix README. please check


NPM + Browserify

This issue stems from #29 and while support is now in place for Bower, I have still to register the packages on NPM.

Any ETA when it will be available on the NPM?

@Predjo sorry for the slightly delayed reply. We're using NPM heavily in our own build pipelines and so this is moving up my internal todo list, I hope to have published all libraries in the collection to NPM by the end of June.

OK so thanks to the nudge :+1:  I've published each of the libraries under NPM now, so CT is now available here: https://www.npmjs.com/package/ContentTools, and can be installed like so:
npm install --save ContentTools


ConFindr conda install dependencies can't be solved

$ conda --version
conda 4.8.3
$ conda install -c bioconda confindr
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: /
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions

Oh, I think I may have identified the issue - the Bioconda recipe depends on rauth, which isn't actually in bioconda, it's in conda-forge. That might be enough for me to get it working.

Can be solved via
create a new environment in conda and hit
conda install confindr
Ref here


Challenge http://www.freecodecamp.com/challenges/waypoint-create-a-text-field has an issue

When you add <input type='text'> after the ordered list this error pops up:
Syntax error, unrecognized expression
instead of completing the challenge.

We've fixed this. Thank you for your patience, and for bringing this to our attention!


ICON-D2 for more precise 48h forecasts

I was wondering if it would be possible to add support for ICON-D2.
It seems to provide significantly more accurate data (thanks to higher resolutions it seems) than the "normal" ICON model.
Since it's in grib2 format, it would probably require a different method of parsing.
DWD Documentation of ICON-D2 (Section 6.3. Near surface products)
DWD OpenData Folder for ICON-D2 (the subfolders are different generation times if I understand correctly)
Also this data would need to be merged with the normal ICON (or ICON-EU, see below) model for >45h forecast.
I would be happy to help, but I thought getting an overview first (and your opinion on this) would probably help.
If you want to see it in action, windy has this model included.
EDIT: After some further reading, the normal station data seems to depend on the ICON 13km resolution. Seeing this, it would probably also help to switch to ICON-EU for the 45h+ forecast.
Resolutions
ICON: 13km
ICON-EU: 6.5km
ICON-D2: 2.2km
Motivation: ICON data differs significantly where I live (in the 5° range temperature wise, precipitation is even worse), so getting more accurate forecast (into hass) is always helpful.

Sorry for the late reply, I have overlooked your message.
Yes, I agree with you. This would be a nice addition! Unfortunately I currently don't have time implementing this. So it will take a few more weeks before this could happen.
Best regards

Would you be willing to accept a PR?

Of course I would!


Allow usage with Mongoid 4 through 6

@artfuldodger does this actually work? Happy to merge it if so

Hi, at least it works with mongoid 5.2.0 for me.
This gem should be merged in mongoid itself I suppose.

It should, but when I talked to @durran and @mongodb security they dismissed it as "WONTFIX" (maybe I did a bad job of explaining why this is a problem...)
Will push a new version for mongoid 5.2.0

Works for us too. Thanks for the release!


Allow product flavors

Hi. I have an AndroidStudio project with product flavors defined in the gradle file for staging and production environments. Each flavor defines its own applicationId.
I have the args defined with 'assembleStagingDebug' in tasks.json so it can gradlew with the selected flavor profile.
When the debugger is trying to launch the MainActivity, it uses the defaultConfig applicationId/packageName to search for it.
It ends with the error "Launch failed: Error: Activity class {(packageName)/(MainActivity)} does not exist."
I can see the source code uses the command to launch activity 'am start' with the 'this.session.build.pkgname'.
Is it possible to have a launch configuration to specify the pkgname or any other way of searching the correct package of the flavor used?

The latest version of the debugger now attempts to extract the manifest directly from the built APK and uses the source file as a fallback.
This should fix this issue, but I will there needs to be some more testing around different gradle configurations and property overrides.


bindzone runtime: add new DNS record types

Recognize some newer DNS record types - CAA (RFC8659, certificate authority authorization), OPENPGPKEY (RFC7929), SMIMEA (RFC8162). Sort SSHFP alphabetically while there.

Have you tried contacting the maintainer, as mentioned in the header?

Hmm, I thought it was no longer valid because www.mehnle.net came back 'domain not found' and the last updated date on the file was 2011 and there had been several commits to the file in vim since then, but now I see that mehnle.net itself does still exist so I'll try emailing.

No response to my email yet. In the meantime I have found another updated bindzone.vim on github which maybe a better source (it has a few other changes): https://github.com/sebastianw/vim-bindzone/

Hello,
Their website is indeed down, but their mail server is responding:
michel@debian:~$ dig +short MX mehnle.net
10 callisto.mehnle.net.
michel@debian:~$ nc callisto.mehnle.net 25
220 callisto.mehnle.net ESMTP Postfix (Ubuntu)
QUIT
221 2.0.0 Bye

Actually, isn't it @jmehnle ?

let me merge it


Move server autostart into admin services

The autostart feature of the OMAG Server Platform is coded in the Spring server chassis.  This has 2 problems:

The code needs to be duplicated into the UI Server Platform
The code needs to be rewritten if we add support for a different REST framework.

This feature moved the code into the admin-services

It seems that UI platform are using different admin services so even there code needs to be duplicated.
The only common module I see is open-connector-framework, but i don't think it belongs there. Maybe rename http-helper to app-helper and add it there?
Also temporary deactivate is triggered in Spring context closed event. Maybe a better idea is to also use JVM hook (Runtime.getRuntime().addShutdownHook) but

We will be bringing the admin services for the UI into the main admin services module when the basic work is completed.
Even if there were 2 admin services, we still need to bring the function into the admin services so that when we have different REST frameworks, the function still works

Using the Spring hook is fine - it needs to directly invoke the admin services function.

I mean for the deactivate part when shutting down the platform, finding a place to put JVM shutdown hook instead Spring close event will make this part work independent of Spring

I understand your suggestion


Skor Questions

👋🏽 Hello!
First, what a great project! I love Postgres' LISTEN/NOTIFY feature and am using it as a key part of a system I'm currently building.
I have two questions:

Postgres has an 8000 byte limit on message payload size, and an 8 GB queue size limit for all notifies (ref: https://www.postgresql.org/docs/9.6/static/sql-notify.html). It's possible to hit the first limit very easily with a normal-sized jsonb column, and I'm not sure how many instances out there run with less than 8 GB of memory, but in the case that either limit is reached, how does Skor deal with it? Does it just pass the error from Postgres along to the caller or crash?

Since notifications have no access control, meaning any code that can connect to your database can send notification messages, message payloads can't really be trusted. In my own system, I only use them to notify the caller that something has changed. I looked briefly at https://pgxn.org/dist/pg_message_queue/, but ultimately decided to just grab the changed row when notified, instead. How does Skor deal with this issue?

Thanks!

Hi !

The default trigger that init.sh adds captures the entire row. This can be modified to only emit the unique identifier for a row. We'll add documentation for this (https://github.com/hasura/skor/issues/6). When the row limit is hit, the notification is not delivered to skor so skor doesn't notify the webhook. I'm not sure about the behavior when the queue size limit is hit, but I'm guessing it should be the same, i.e, skor doesn't get the event, so it doesn't notify the webhook.

Lacks of permissions on listen/notify is a bit of a pain in Postgres. skor makes no assumptions about the validity of a notification, it merely forwards it to the webhook.


:book: Docs for upgrade with auth

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1288

just small nits!


feat: enable trivy server with self-signed certificates

Signed-off-by: Engin Diri<EMAIL_ADDRESS>fixes #1002
There are situations, where you have a self-signed certifacte for the Trivy Server. e.g. in our company, we have to use our own company wide PKI. Or when you use LE staging ACME. This will appear as x509 certificate signed by unknown authority
With this PR, we can set a new optional variable called trivy.insecureServer. This sets in the background the env variable TRIVY_INSECURE. It's different to existing trivy.insecureRegistry parameter. As we are not really talking about the registry.
Looking for your feedback.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@dirien thank you for the contribution LGTM

Thank you for working on this patch @dirien Please see my comments before we marge this PR.

Thanks for your feedback @danielpacak! Appreciate this very much! I changed the points and will create a PR for the version bump!
Looking forward

Done for version PR bump -> https://github.com/aquasecurity/starboard/pull/1008

@dirien , could you sign our CLA agreement, which is one of required checks before we merge the PR?

@danielpacak, done! I clicked again on the link!


Update dependencies and the template

This PR updates the dependencies in order to fix the vulnerabilities CVE-2024-45491 and CVE-2024-45492 in libexpat.
The microservice template is also updated.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.647%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
367

Relevant Lines:
414

💛  - Coveralls


Add Trinity Installation / Quick start

What was wrong?
Need clear setup instructions on how to setup a Trinity node.
How was it fixed?

Added a Quickstart for Trinity that walks users through the installation process and demos how to sync and interact with the chain via the exposed web3 variable.

Moved the previous Py-EVM Quickstart into guides/evm and promote the Trinity Quickstart in the left nav menu instead. Reasoning is that:

a. The Py-EVM Quickstart is out of date and needs to be revamped anyway
b. I expect more people to be interested in Trinity than Py-EVM so the top down approach seems to make sense

Eliminated an unrelated broken link as well as fixed some style inconsistency

NOTE: We should wait to merge this until the trinity package was released as a pip package.
Cute Animal Picture

This is still missing a quick demo for the JSON-RPC API. Otherwise, ready for review and could potentially be merged once we have the trinity package on pip. Depending on the timing the paragraph on the JSON-RPC API may land in a subsequent PR.


Allow weighted scalar reporting by providing a tuple to the reporter

This PR takes over #4031

@okuta I added some tests. Could you check them again?
@toslunar I fixed it. Thanks!

Jenkins, test this please

Jenkins, test this please

Jenkins CI test (for commit 5fe673fb22fd5f917086cc72f1f40af4fd5f188c, target branch master) succeeded!

Please remove cache file.
.pytest_cache/v/cache/nodeids

Removed.
Jenkins, test this please again

Jenkins CI test (for commit f642dea9eec8fc671c99f42a7ed0965238920be3, target branch master) succeeded!

LGTM!


Standardize on u32 for struct field index and enum variant index

Rename deserialize_struct_field to deserialize_identifier to indicate that it is used not just for struct fields but also for enum discriminants.
Use u32 as the type of variant indices in the serializer API to match the way that our generated visitors expect visit_u32.

Fixes #722.

:umbrella: The latest upstream changes (presumably #858) made this pull request unmergeable. Please resolve the merge conflicts.

@oli-obk I am merging this because it conflicts with all my other changes but lmk if you want to discuss this.

I'm unhappy with the identifier name, but since the docs explain it well and I can't think of anything better, lgtm


Filtering devices in configuration ignored

I have the following configuration:
keymap:
  - name: debugpedal
    remap:
      KEY_A: KEY_F9
      KEY_B: KEY_F7
      KEY_C: KEY_F8
    device:
      only: ["PCsensor FootSwitch Keyboard"]

When I run this config xremap prints the following message:
Selected keyboards automatically since --device options weren't specified:
...

When I provide --device "PCsensor FootSwitch Keyboard" as a command line parameter the events are filtered as expected. I tried the same thing with device paths with the same result.
I expect the tool to obey configuration just as it obeys command line arguments. as it is documented:
https://github.com/xremap/xremap/blob/f39afb076bd5b4e087464fc5327b4fdb09826c25/README.md?plain=1#L381
May be related to #388

That's not device config is for. You see, it's an keymap entry-level config, so it cannot possibly control how xremap globally selects devices. If we were to do that, it must be a top-level config, but xremap currently doesn't have that feature. Since --device solves it with no problem, I would rather not want to complicate the config structure.
device filter is useful when xremap switches between multiple devices using --watch and you want to apply part of the configs for a specific device. It has a completely different use case from --device.


add initial commit to draw group line chart

This is my first to write js, so there must be some deficiencies, any comments are welcome!

Mostly lots of small, styling comments (we call them nitpicks). The code mostly looks good, but I just have questions around the look of the new legend and chart. Could you paste a screenshot?

Thanks for your comments! I will fix it asap.

This PR is to draw a group line chart give a dataGroupsDict. Some demo tests are in dev.js.
https://lh3.googleusercontent.com/-kjW0NN8Eync/Xweh6r7zZ0I/AAAAAAAAACE/mrSvE1i18Ak6_NQILZTj-mXorFCEVYs8gCK8BGAsYHg/s0/2020-07-09.png
https://lh3.googleusercontent.com/-0a5Q7i-HH7c/Xweh-sOraCI/AAAAAAAAACI/QQqkyzqwAnMU25fla6cXAfbsf96pfnqQQCK8BGAsYHg/s0/2020-07-09.png
https://lh3.googleusercontent.com/-EeDOjFRMvO0/XweiB7yM4XI/AAAAAAAAACM/LCEJzvx0GL8jUqpJubP1x_E5pIkdyuszACK8BGAsYHg/s0/2020-07-09.png
These are three screenshots shown in dev.js.

@shifucun @beets The new commit is generally addressing suggestions and refactor codes. PTAL. Thanks!


Docker container

Would it be possible to have this built as a docker container for easier deployment?

There is already a docker solution
https://github.com/TFyre/bambu-farm/blob/main/docker/bambu-liveview/README.md

@Locxion Is there a reason this cant be deployed as a single container? Ideally id like to run this under unraid so using docker compose is a pain.

i have no idea
the livecam containers work per printer
i dont know why the other services are seperated

@Locxion Is there a reason this cant be deployed as a single container? Ideally id like to run this under unraid so using docker compose is a pain.

No reason, but it defeats the purpose of having a docker compose setup based on thin layers with good isolation.
Feel free to submit a PR to add single-container-functionality 😄


logging: Enable string stripping by default for dictionary based backends

Fixing dependency which did not allow strings stripping when LOG_OUTPUT was used which prevented that feature for all dictionary backends.
Setting string stripping to be by default enabled for all dictionary backends.

fyi @optical-o


updated number of days and sessions

Hey, this was already fixed by the time this was reviewed. Thanks for the update buddy. But closing this for now.


Server actions

Changes

Move db calls from db-calls to actions.ts as server actions

Some wrong imports, Fix Them


New Demo GIF

I couldn't help trying to read the code while watching the animation and that makes the animation feel a wee too fast. How about adding a comment before each section of code?
And how about also showing:

a "Hello, World!" one-line bubble
and an example with ANSI colours
?

Good points!
I made a new one.

Very awesome.


Added explicit extension for imported .wasm file for --target bundler

Closes #1441
This update is more in line with esm-module integration. At the moment there's a comment near this code:
// With Bundlers and modern ES6 support in Node we can simply import
// the wasm file as if it were an ES module and let the
// bundler/runtime take care of it.

If I get it right, without extension this will not work in node.js

Looks good to me, thanks!
I don't think this works for some more esoteric use cases but since this is scoped to just bundlers I think it should work out alright.


Reactiveproperty after completed

Fix subscribing ReactiveProperty after OnCompleted. It notifies last value like UniRx.

ReactiveProperty is aligned with BehaviorSubject from dotnet/reactive in behavior, and it does not transmit Value after OnCompleted.
However, ReplaySubject does transmit, and considering overall consistency and convenience, it seems better to transmit Value.
Thank you, I will adopt this approach.


Upgrade Jest and switch to jest-circus

Overview
This PR upgrades all projects to the latest Jest version and switches the test runners over to the new jest-circus runner. The only meaningful change needed to change the runner was to migrate from spyOn to jest.spyOn, which is the majority of what's contained here 🎪👍

out of curiosity - what is the advantage of using jest-circus over default runner?

Currently only being able to use jest.retryTimes and test.todo and move faster for us. But in future it should enable some nice things like getting rid of now- injected globals or fixing some longstanding issues regarding running user code in VM context

Cool, so this leads to another question - why this isnt a default runner yet? 😉


scrapy-splash doesn't render the page, but splash does

Scrapy-splash does not render the page fully, even though Splash alone does. I want to render contents of a table with id "grid". I can see that its rendered correctly in splash browser by http://localhost:8050/info?wait=0.5&images=1&expand=1&timeout=90.0&url=https%3A%2F%2Fwww.homeinspector.org%2FHomeInspectors%2FFind%2FResults%3FMetroAreaID%3D5%26NeighborhoodID%3D%26SearchType%3DMetroArea&lua_source=function+main(splash%2C+args)
++assert(splash%3Ago(args.url))
++assert(splash%3Await(0.5))
++return+splash%3Ahtml()
end
However, its not rendered at all in my scrapy spider. The grid is empty.
I am talking about parse_find_page function
script="""
function main(splash, args)
splash.private_mode_enabled = false
splash.plugins_enabled = true
splash.indexeddb_enabled = true
splash.html5_media_enabled = true
assert(splash:go(splash.args.url)})
assert(splash:wait(7))
splash:set_viewport_full()

return splash:html()
end
"""

class Homeinspectors(CrawlSpider):

name = 'instructors'

def start_requests(self):

    return [SplashRequest('https://www.homeinspector.org/HomeInspectors/Find', self.parse_find_page,
        args={
            'wait': 5,
            'http_method': 'GET',
            'timeout':30
            },
    )]

def parse_find_page (self, response):

    values = response.xpath('//select[@id="ddlMetroArea"]/option/@value').extract()[1:]
    url = "https://www.homeinspector.org/HomeInspectors/Find/Results?MetroAreaID={}NeighborhoodID=&SearchType=MetroArea"
    for v in values:
        yield SplashRequest(url.format(v), self.parse_search_results,
            endpoint='execute',
            args=
                {'lua_source': script,
                'wait': 5,
                'http_method': 'GET'},
            headers={'User-Agent': "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.113 Safari/537.36"})

def parse_search_results( response):

    # print response.body
    instructors_urls = response.xpath('//*[@id="grid"]').extract()
    print instructors_urls

Can you still reproduce this issue?

I'm having the same issue. what should I do?


Error with guild.yml but not guild run train.py

I am using tensorflow 2.0a and guild train.py works fine for me. However, when i specify a guild.yml file and run guild run train it fails with the following error:
You are about to run clamp-detector:train
  batch_size: 2
  epochs: 2
  evaluation_batch_size: 32
  gradient_clipvalue: 1.0
  learning_rate: 0.001
  model_dir: runs/model
  oversample_factor: 15
  seed: 1000
  steps_per_epoch: 100
  validation_split_seed: 71751
  validation_steps: 50
Continue? (Y/n) Y
Traceback (most recent call last):
  File "/home/richard/anaconda3/envs/clamp/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/richard/anaconda3/envs/clamp/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/richard/.local/lib/python3.7/site-packages/guild/op_main.py", line 272, in <module>
    main()
  File "/home/richard/.local/lib/python3.7/site-packages/guild/op_main.py", line 53, in main
    _main()
  File "/home/richard/.local/lib/python3.7/site-packages/guild/op_main.py", line 79, in _main
    _apply_plugins()
  File "/home/richard/.local/lib/python3.7/site-packages/guild/op_main.py", line 107, in _apply_plugins
    _apply_plugin(name)
  File "/home/richard/.local/lib/python3.7/site-packages/guild/op_main.py", line 111, in _apply_plugin
    plugin.patch_env()
  File "/home/richard/.local/lib/python3.7/site-packages/guild/plugins/tensorflow_util.py", line 39, in patch_env
    self._try_patch_tensorflow()
  File "/home/richard/.local/lib/python3.7/site-packages/guild/plugins/tensorflow_util.py", line 52, in _try_patch_tensorflow
    tensorflow.summary.FileWriter, "add_summary",
AttributeError: module 'tensorboard.summary._tf.summary' has no attribute 'FileWriter'

Tensorboard is version 1.13.1, tensorboardX is version 1.7
This is not a show stopper for me since I can run with the other command but I thought it might be helpful to report

The api has changed in tensorflow 2.0 to train_writer = tf.summary.create_file_writer('log_dir')

Ah, indeed - thank you! TF 2.0 is teed up for the next release. I'll update here with status.

@samedii The for this will be out this week. In the meantime, you can work around the problem using the pattern used in this example:
https://github.com/guildai/examples/tree/master/tensorflow2
The Guild file (guild.yml) disables all plugins, which is required to skip the logic triggering the exception you reported. The operation also explicitly defines output-scalars, which capture the output generated by the Keras training loop.
If your operation generates any additional (or alternative) output, update output-scalars accordingly.
If you need any help with regular expressions - just ping us on Slack. We're working on docs with examples for all of this but I wanted to get you an update ASAP.
Thank again for the report!

@samedii Compatibility with TF 2 is now in the latest release 0.6.4. Please let us know if you're still running into issues!

Working nicely!


Added the bonecrusher to the prayer calculator

idk i got bored

@Nightfirecat I assume this is going to be part of the skillcalc refactor?

Added in 9209de31e27a13916dc98812f49cfb07cc369cf0


[NDTensor] Allow general objects are storage backends for Tensor

Description
This relaxes the type constraint on the storage field of Tensor to allow any objects as storage backends.
The idea would be that the storage should adhere to a minimal interface requirement, for example in general that it is an AbstractArray (though that isn't required) and defines some contraction and permutation functionality (though if we provide a general TTGT backend, we could in general only require permutations and matrix multiplications) as well as factorizations like SVD or QR.
You can see the interface requirements demonstrated in arraytensor/array.jl. Some of that can be made more generic, and therefore won't need to be defined for every storage backend.
For now, the design is that non-TensorStorage storage backends are "registered" in the type union ArrayStorage, only for the purpose of ensuring that dispatch doesn't get confused between new functionality we write for generic storage backends as opposed to the current TensorStorage type which has particular requirements, so that we don't have to change current TensorStorage code. Once we replace the current TensorStorage types in favor of AbstractArray backends (Array, BlockSparseArray, DiagonalArray, etc.) we can just remove the ArrayStorage type union (and maybe replace it with AbstractArray where appropriate).
@kmp5VT your PR #1161 aligns with this new design since you are getting rid of the EmptyStorage type (already in the right direction of removing sTensorStorage types) and also the UnallocatedZeros type will still be relevant here since it will either be used directly as a Tensor storage or used as a data backend for one of the other storage types like BlockSparseArray or DiagonalArray. Additionally, I don't anticipate getting rid of our contract or permutedims code, just making sure they both work for the new storage types, so the optimizations you are doing in #1194 are definitely still important.
A strategy for how to proceed with the goal of removing TensorStorage types is to "peel off" the storage types one by one. For example, we can start by making sure that using an Array as a storage backend for Tensor has all of the same functionality as Dense (which I already started to do in this PR, as you can see it already works for a number of operations like addition, permutation, contraction, etc.). Then we can just change the default ITensor constructor to use Array (or UnallocatedZeros) storage instead of Dense storage when there are dense indices. Then, once we develop external BlockSparseArray and DiagonalArray AbstractArrays (BlockSparseArray already started in #1205), once we make them feature-complete with BlockSparse and Diag we can use them as defaults as well, and then just remove BlockSparse and Diag.
The Combiner is a slightly awkward corner case, but I think we can basically just leave it as-is for now and hopefully it get replaced by a more principle fusion tree type in the future.

@emstoudenmire I think as you design the FusionTensor type (or whatever it is called), you should add it to the ArrayStorage type union instead of making it a TensorStorage subtype, and design it as a standalone array-like object independent of the Tensor type at first (whether or not is AbstractArray shouldn't really matter, but I could see an argument for making at an AbstractArray but maybe disallowing getindex and setindex!).

@emstoudenmire @kmp5VT I'll merge this so we can build on top of it going forward.

Looks really good and thanks for the guidance about the fusion storage type. That's what I needed to know.


Bump to 2.50.5

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks!


How Can I see output?

How can I execute the code in SSD.ipynb  and view output?

@Walid-Ahmed you need to open it in Jupyter/iPython notebook.


0.13 support: [fix] Remove deprecation with Component.type

Best I can tell this handles it.

Closed in favor of https://github.com/rackt/react-router/pull/948


chore(NumberField/Discriminant): golf

Use calc to golf a proof and improve readability.

Much nicer, thanks!
maintainer merge

bors merge


Running run_mission.py hangs

I am attempting to go through the tutorial and can launch MineCraft fine, but am unable to play any tutorials or missions. After running run_mission.py, I get a printout of the experiment id and that's it.
I am running Malmo-.37 on Ubuntu 18.04 which Python 3.6.

Can you try
MALMO_XSD_PATH=YourMalmoDir/Schemas/
in your terminal before starting a mission? ( Do replace the path to your own Malmo directory )


[FEATURE] Decouple modules from HTTP client (currently okhttp)

Modules should be decoupled from the HTTP client to allow using other HTTP clients than okhttp (e.g. RestClient for Spring Boot).

Retrofit has a hard dependency on okhttp.. should we throw retrofit in the mix as well?  Retrofit could be replaced by competing libs such as openfeign?

Right, I meant okhttp+retrofit. I would avoid reflection and dynamic proxies as much as possible, I guess we can live without this convenience.

@kchaber @One16-KS could you please review the first draft: https://github.com/langchain4j/langchain4j/pull/1103 ?

Planning to do so this weekend!

Nothing to add really, really good first step, I'll try to chip in if I have spare time.

Hint: Spring-Web's RestClient is the replacement for RestTemplate.
As of version 0.36.2 of langchain4j a streaming chat model always waits for 1 minute to shutdown, even though we can make sure no new requests and all onCompletes are already finished. The OkHttpClient is completely hidden e.g. in OllamaClient. Is there way to gracefully shutdown the "OkHttp Dispatcher" as of now?

Unfortunately no way to shutdown yet. The plan is to migrate Ollama to Java 11 HTTP Client soon and make http client customizable. Since this is an Okhttp issue, it should be solved by this migration.

JDK HttpClient is not bad as an option (had to move away from it, due to e.g. https://bugs.openjdk.org/browse/JDK-8308364). A good comparison can be found https://www.wiremock.io/post/java-http-client-comparison.
What @kchaber suggested will reduce the effort to doctor on langchain4j due to HTTP lib details/issues in the long run (it's not langchain4j's focus anyway). Therefore it is understandable that I also vote for an abstraction similar to Spring-Web's RestClient or RestTemplate instead of depending on one specific HTTP lib.


Match.Optional seems to be null on Bronze Matches.

I think the Optional property on Match object should be nullable.

Hey, thanks for the report. Should be fixed in ea196d9b2e0714d73a1fac89afa902b36b1067c7. NuGet package version 3.0.0 has been uploaded and appears to be available for download now.


feat: new bridge: JsonBridge

This bridge converts json to a feed.
This is a work-in-progress.
Refs https://github.com/RSS-Bridge/rss-bridge/issues/1916 and https://github.com/RSS-Bridge/rss-bridge/pull/2719

As I think more about this, I still maintain that having an extensible abstract bridge similar to XPathAbstract and #2719 is probably the way to go. Otherwise, it will be difficult to use the bridge if the target JSON is even slightly different from the assumptions made by the JsonBridge. Additionally, making an official bridge that does not require tweaking by end users (Like GoogleGroupsBridge) is more difficult with this paradigm.

This is a wip. An abstract json bridge would look very similar.

This was a proof of concept.

@dvikan , any reason we didn't move forward with this? Need help or anything?

it's very niche.
feel free to create a new pr


Auto-fetch benchmark when not available

Based on work @cz4rs done for Kokkos.

I see someone follow the activity on Kokkos Core

Sometimes :) Great artists steal.

Build failures are not related.


main: add prevent_insecure_startup

Fixes #257

Codecov Report

Merging #275 (766718b) into master (fce1954) will increase coverage by 0.33%.
The diff coverage is 57.44%.

@@            Coverage Diff             @@
##           master     #275      +/-   ##
==========================================
+ Coverage   61.06%   61.40%   +0.33%
==========================================
  Files          52       52
  Lines        3344     3368      +24
==========================================
+ Hits         2042     2068      +26
+ Misses        932      930       -2
  Partials      370      370


[ci] Replace Firebase Test Lab deprecated Pixel 4 device with Pixel 5

Pixel 4 is deprecated/less available in Firebase Test Lab, which is causing test timeouts.
Update instead to Pixel 5.
See also https://github.com/flutter/engine/pull/29239
Fixes https://github.com/flutter/flutter/issues/92102
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Tests passed. Merging to open the submit queue.


Toggling the ability to enable/disable BBCode within text boxes?

Is your feature request related to a problem? Please describe.
I'm not sure that this is a 'problem', per se (hence why I'm listing it as a feature request), but I've noticed that it is possible to type BBCode parameters within text input boxes. For example, in some code testing I've been doing, I created a text box which prompts the user to enter their given name (which is then referenced in-text), and by typing in a BBCode parameter like, say, color=x, any iteration of the name is then colored in-text (including iterations which already have color parameters assigned to them, which are then overwritten by the BBCode typed into the initial text box).
Describe the solution you'd like
Since I see how being able to type BBCode into a text input box might be useful, it would perhaps be nice to be able to toggle this ability on and off, or better yet, allow the designer the ability to limit which characters can be allowed within a text input box (being able to limit the number of characters would also be a nice feature).
Describe alternatives you've considered
For the BBCode aspect, I know there is scripting which should automatically re-map either of the brackets used in BBCode so that the generated output doesn't contain usable BBCode, but I'm still new to all of this, so I don't yet know how to actually implement the code (though I will include the scripting I found online; apologies to whomever originally wrote it, as I don't remember the source):
extends RichTextLabel
func escape_bbcode(bbcode_text):
return bbcode_text.replace("[", "[lb]")
For reference, I'm currently using Dialogic 2.0-Alpha-09 in Godot 4.1.1.
Thank you

I was confused initially. You mean the text input box in this case, right? I think that should be doable. Users typing bbcode into the text input didn't occur to me, but we might as well insert a guard.

Yes, sorry, I'm referring to the text input box. Thanks for your response on this. Do you have any thoughts on the further possibilities of limiting the types/amount of characters one can type into a text input box (it seems, at this time, the former can be addressed in Godot's project settings/input map, but I need to learn a bit more to see how to implement this one my end)?
Thanks again


Alt Idle Animation Event is spelled "Speciied" instead of "Specified"

Describe your bug here.

engine is RUINED
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No edits made.

lmfao

Fixed in #2130


Build error "msync not declared" on ubuntu trusty, when building on virtual box guest->host filesystem

I'm trying to compile gperftools-2.7 under ubuntu trusty. Under Vagrant on a mac.  When I configure gperftools-2.7 it confirms libunwind is installed, which I installed first.
But make in gperftools-2.7 gives these errors:
src/heap-checker.cc: In static member function ‘static void HeapLeakChecker::IgnoreLiveObjectsLocked(const char*, const char*)’: src/heap-checker.cc:1436:52: error: ‘MS_ASYNC’ was not declared in this scope          if (msync(const_cast<char*>(object), size, MS_ASYNC) != 0) {                                                     ^ src/heap-checker.cc:1436:60: error: ‘msync’ was not declared in this scope          if (msync(const_cast<char*>(object), size, MS_ASYNC) != 0) {
Am I missing a pre-requisite here?

I have the same error on Ubuntu 18.04.1 LTS.

I also tried docker thing for 18.04. I need someone to attach config.log to try to figure out what happened.

Fresh VM of ubuntu trusty also has no issues building gperftools 2.7. I need more details about your setup.

ok - my tests are on Ubuntu 18.04 LTS Desktop...  I'll try to capture a configure.log later this week.

Here is a config.log of the issue. With gperftools 2.7 release.
This was specifically a vanilla install of Ubuntu 18.04.1 LTS 64-bit on VirtualBox. Only build-essential was installed after install.
config.log
dpkg-l.log

error code 6 is failing of mmap of file that is smaller than mapping length. Can you also give output of /proc/mounts and echo $HOME ? I think we should still get rid of testing mmap, but I would like to understand what is happening here.

Great. This explains "breakage". You're building from directory mounted via vboxsf. I guess it is virtual box thingy that propagates host FS to guest. And apparently it does so in broken, or at least unexpected way.
I am not 100% sure about broken. autoconf code that checks for working mmap is arguably doing it wrong too. It creates files smaller than page size and tries to mmap entire first page. Apparently vboxfs doesn't like it. And I am not sure if it is correctly not liking it.
Anyways, I'll get rid of mmap testing

OK - totally get that @alk . vboxsf just has too many issue. Just as you wrote this I switched this box to using a NFS share from the guest.  Thanks for following up. Hopefully this helps others.

Merged a fix.


spot priced templates can be used along with standard ones

@doktoric

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1284/


feat: Log source state on transitions #37

This also renames state to target_state to make it more explicit.
Using state still works except when filtering using the ORM.

Codecov Report

Merging #75 into master will decrease coverage by 0.32%.
The diff coverage is 91.3%.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   96.08%   95.76%   -0.33%
==========================================
  Files          21       22       +1
  Lines         460      472      +12
==========================================
+ Hits          442      452      +10
- Misses         18       20       +2

Impacted Files
Coverage Δ

django_fsm_log/admin.py
100% <ø> (ø)
:arrow_up:

django_fsm_log/backends.py
71.15% <ø> (ø)
:arrow_up:

tests/test_manager.py
100% <100%> (ø)
:arrow_up:

tests/test_model.py
100% <100%> (ø)
:arrow_up:

.../migrations/0004_rename_target_add_source_state.py
100% <100%> (ø)

django_fsm_log/models.py
95.12% <83.33%> (-4.88%)
:arrow_down:

Continue to review full report at Codecov.

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

Ok, good point. I'll change the PR.

Yes, looks better - makes the patch more concise as well.

@blueyed have you had a chance to test it ?


How to preserve the original one when drag an item from one grid to another?

Lets see this official sample: http://troolee.github.io/gridstack.js/demo/two.html
I want to know if it's possible to preserve the original item when we drag one from left grid to the right? i.e. copy from left to right. I didn't see it from api options.
It's really important to me. Any help is so much appreciated! Thanks.

Hi, @shijistar ! Preserving the original widget isn't currently possible. However, due to an overwhelming request for it, we plan on adding it very soon. #133 has some requests for this feature as well.
For the time being, you can find upcoming features mentioned in our blog post here. We need to work to figure out what we can move to allow for widget cloning (your request here) to be completed sooner. You can follow the blog, where I'll post gridstack updates. I'll be sure to mention our plan for this and other multiple-grid features.
Thanks for using gridstack.js! Your feedback is important to us - we need this continuous feedback to be sure we're delivering the correct features!

Thank you @radiolips for your reply. I'm very happy to hear it's in the plan. I hope it appears in v1.0 :-)
I love gridstack so much, as it provides fantastic user experience that layout the reports like building blocks.   Thank you guys for the such wonderful work!

@shijistar I've been able to create a working example. As with the multi-grid features, it'll be experimental until everyone has had some time to test it, but it'll be in version 1. Continue to monitor my blog for more information. I'll post in the next few days about how this will work.
This also is requested in #396 #499.

@radiolips i had requested this yesterday as well :) Would you be kind enough to share your solution ?
We are trying to solve it in a round about way ( eg: When the drag start happens we copy the html of the widget into their parent container so while the move is happening a copy of widget is left behind. This isn't the exact jquery clone way. But, imagine it as a pack of cards. When you pick the first one, there is another one below it..We keep doing that..round about..

@mandys Sure. For the time being, I am NOT focusing on getting it to work grid-to-grid. If you check the two grid example, there's a widget on the top-left. It's not actually in a grid. That widget will be able to be cloned.
Will that satisfy your requirements? I assume it will - it's actually slightly simpler than needing multiple grids, because I assume that for cloning you don't actually want the initial widget in a grid. Let me know, though.

Yes. We don't need the draggables in a grid. They will live in a sidebar and dragged onto the stage (grid).

Great.  If not, could you join? I could create a patch of the code when I'm done for you to test. If you don't want to, I can push the code to the develop branch when it's complete, but I'd prefer a bit more testers.

Just joined

This works on the develop branch now. Please be sure to use draggable setting helper: clone.

@radiolips Could you possibly show an example of how to set this in the options? E.g. should I use
var options = {
...
draggable: {
                    helper: clone,
                }

I get an Uncaught ReferenceError: clone is not defined when taking this approach.


pythonPackages.ete3: init at 3.1.2

Motivation for this change
The ETE toolkit is a reference python library for manipulation of phylogenetic trees in Newick or Phyloxml format.
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? No
[x] Tested package tests are broken; tested with pythonImportsCheck
[x] Tested compilation of all packages that depend on this change no packages depend on this change
[x] Tested basic functionality of all library files
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thanks for your comments; I fixed them in the last commits.

Should be alright now!

When running nixpkgs-review on this, i got a missing module error when trying to execute bin/ete3: ModuleNotFoundError: No module named 'numpy.core._multiarray_umath'. Nix-code wise though, this looks good to me :relaxed:

Indeed, I was able to reproduce the error. Turns out that nixpkgs-review rev master setup two instances of ete3 (/nix/store/XXX-python3.10-ete3-3.1.2/bin/ete3 & /nix/store/YYY-python3.9-ete3-3.1.2/bin/ete3) for Python 3.9 & 3.10, and that the highest one in $PATH is the one for Python 3.10.
However, for some reason, when importing numpy in Python 3.10, it calls upon numpy for 3.9, which you can observe by running import numpy in python3.10 in the very same environment. Python 3.9 still imports its own numpy though, and thus does not meet the same problem.
Finally, calling directly ete for Python 3.9 (/nix/store/YYY-python3.9-ete3-3.1.2/bin/ete3 or /nix/store/ZZZ/bin/ete3 ) fails with a cannot import name 'TextFace' from 'ete3' – which is expected as I didn't enable Qt5 support by default.
I'm not sure if this problem is stemming from my side or from nixpkgs-review, as I confess I arrive at the very fringe of my Nix competences.

Bump?

Bump again?

Result of nixpkgs-review pr 150972 run on x86_64-linux 1

2 packages built:

python310Packages.ete3
python39Packages.ete3

Thank you!


Canvas: add some missing calls to save/restore state

Some methods were missing calls to save/restore state when used with gradients. The clip was being modified internally and resulting in incorrect results for subsequent drawing operations.

Thanks!


Creating a wrapper component for react-metro

I want to create a generic wrapper component for animations, so I tried this:
import React, { Component } from 'react';
import Metro from 'react-metro';

export class TransitionComponent extends Component {
  props: Props;

  render() {
    const { children, show } = this.props;

    if (!show) {
      return null;
    }

    return Metro.container(
      <div>
        {children}
      </div>
    );
  }
}

And I would use it like this:
      <TransitionComponent transitionName="fade" show={props.show}>
        <MyComponent {...props} />
      </TransitionComponent>

But the animations are not being applied.
Is there something I am missing about rendering children like this?

Hi @dagda1 You are probably just missing the transition group wrapper:
<TransitionGroupPlus>
    <TransitionComponent transitionName="fade" show={props.show}>
      <MyComponent {...props} />
    </TransitionComponent>
</TransitionGroupPlus>

Brilliant thanks, that was it.
If I wanted to use a slide transition, something like
.slide-appear {
  height: auto;
  max-height: 0;
  display: block;
  overflow: hidden;
  transition: max-height 500ms ease;
}

I should pass in a hash of values as the second argument to Metro.container, could you give me a pointer as to what I should be passing in for the max-height transition?

Metro uses Greensock Tweenmax under the hood (specifically fromTo) so I suggest reading their docs at https://greensock.com/docs/TweenMax/static.fromTo() and see what you might find out.
That being said it's important to know that Metro needs animation objects to function properly - apply in / out animations & execute the onMount / onUnmount callbacks.


TG-1897 Delete not needed AppFixture

Hi @ivy-rew ,
I just removed the not needed AppFixture. Now that you are back, can we start the review for this market connector please?

yes sure, just wondering who did the initial review of the first commit? anybody from ivy-core during my vacation? or should it be included in the review?
https://github.com/axonivy-market/stateful-datatable-demo/commit/e71e7355a6b6c14fa80da7895aadc1eb65e6be9c

yes sure, just wondering who did the initial review of the first commit? anybody from ivy-core during my vacation? or should it be included in the review? e71e735

there was no review for the first commit. it was only forked from my repo.

ok thanks: adjusted the master accordingly ... so we can go through the complete changeset

Please be aware that, if in order to appear in the Market, a release of this artifact is necessary.
And furthermore a PR here: https://github.com/axonivy-market/market/tree/master/market/utils


Replace usage of AppInfo with a trait AppMeta

Hi,
there are some use cases (for example using clap and app_dirs in the same app) when you'd want to create your object which you could pass to app_dirs functions. This pull request implements that ability via trait.
I've create AppMeta trait and replaced all function declarations to accept any type that implements this trait. I've also added AppMeta implementations for AppInfo struct, so nothing really changes for the users I think.
Let me know what you think!

I actually didn't check before doing this that there is #19 and now #23 which basically fixes it. I think, that my version doesn't brake existing library users, since the app info struct is left with the same name and in #23 it has a different name.. In my opinion, I would pull in this request (#22) and rework #23 in the following way:
struct AppInfoDynamic {
//
}
impl AppMeta for AppInfoDynamic {
//
}

This way, you would not brake existing users, because AppInfo would still be the same struct, and #19 would be solved by just adding a different struct which owns the strings..

Based on #23 I've added OwnedAppInfo struct, so if this is going to be merged it would require no additional work.

Thought that if AppInfo should be renamed to StaticAppInfo as suggested on #19 , implemented that and added type declaration for AppInfo, so it would not brake existing users.
Bumped version to 1.2.0, since I think such change is quite important for a version change.
Added deprecation declaration, so that existing users would see a warning that AppInfo is deprecated and should use StaticAppInfo instead.

This is exactly what I need - we could potentially derive this trait impl from Cargo.toml info if this crate goes this way...


QuickUnlock deletes KeyFilePath from Keepass.config.xml

There's an "Advanced" option in Keepass to remember key sources. The path to this sources seems to be saved in the config.xml at "KeyFilePath". When using QuickUnlock, it seems to me that QU changes this entry to something QU related in order to work. When Qu now gets invalidated after a given timeframe, the "KeyFilePath" is not being reversed to the entry before. So, after QU is invalidated, I have to chose the file path everytime again despite "remember key sources" is marked at the options tab.
If I just open Keepass, mark the "remember sources" option, close it and then just reopen it without letting QU interfere, it works like it should and the chosen keyfile path is readily selected at startup.
I used Keepass 2.35 for a long time and this worked! This problem seems to have appeared with the new Keepass 2.36. It's really annoying and perhaps you could have a look at it?

That shouldn't have worked with 2.35 too.
See #18 or #13. Both issues contain explanations why this sadly can't work.

I had it running since last year Nov-Dez. I only entered the key file path once and it worked like a charm. But now that you said it, I'm not sure if I did change something else to get this working. Bummer, can't remember.

I don't know if you are familar with the KeePass code but in the 2.36 AceDefaults.cs at line 255 is the SetKeySources method. The code checks if the masterkey contains a "KcpKeyFile" and if so sets the KeyFilePath. After a successfull quick unlock the masterkey contains only a "KcpCustomKey", so the KeyFilePath can't be set (The KeyProvider property gets set to the name of the custom key, "QuickUnlock" in this case.). It may work if name of the custom key returns the old key file path in the case no quick unlock is available. But then the plugin would need to store a list of file paths somewhere which isn't a good solution. The real solution I can think of would be a change in how KeePass requests the key of the plugin but I don't think this will change in the near future because there is no real need for it.

I looked at a few things and I'm pretty sure, if I enter test.txt</KeyFilePath into the Keepass.config.enforced.xml, it seems to override the missing part in the Keepass.config.xml on restart. If QU now gets invalidated, couldn't you just read the KeyFilePath from there?

I looked through the code and I think it is not possible have a workaround for this. If you have enforced the KeyFilePath, doesn't it get added to the list so you can just select the path?

No. I would have to restart, so that the enforced config would put the KeyFilePath back into the normal config. That's why I asked you if it would be possible, at the time QU gets invalidated, to read the KeyFilePath from the enforced config and put it back into the normal config where it was before QU had to overwrite it.


Translate RuleSuppressionID for custom rules as well

PR Summary
Fixes #1143 by translating the RuleSuppressionID for custom rules as well.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[x] Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

Any idea when this will get merged ? @bergmeister

@manigandan-jegannathan-developer
it is reviewed and approved. Hopefully some point next week (this depends on availability of the PowerShell team members that are currently busy with publishing a RC of PowerShell Core 6.2)

@manigandan-jegannathan-developer
it is reviewed and approved. Hopefully some point next week (this depends on availability of the PowerShell team members that are currently busy with publishing a RC of PowerShell Core 6.2)

Thanks!

@bergmeister won't this need any version changes ? How do I get it from PS Gallery

@manigandan-jegannathan-developer
Yes. We are planning to release a new version of PSSA (1.18) this month to the PSGallery. Installation will be as usual via Install/Update-Module


Use libtool with versioned ABI

This uses libtool to build the library and should therefore solve issues like: https://travis-ci.org/bitcoin/bitcoin/jobs/79127433#L1480

Added proper libtool versioning (@jgarzik: please review in detail).

Update looks pretty good, and passes travis.  Seems to match my experience with https://github.com/jgarzik/hail
Comments:

libunivalue needs a pkg-config file a la https://github.com/jgarzik/hail/blob/master/lib/libhail.pc.in and https://github.com/jgarzik/hail/blob/master/lib/libhail-uninstalled.pc.in (with obvious additions at the end of configure.ac)
libhail cleans up the symbol list a bit, perhaps we want that:

libhail_la_LDFLAGS = \
        -version-info $(LIBHAIL_CURRENT):$(LIBHAIL_REVISION):$(LIBHAIL_AGE) \
        -no-undefined \
        -export-symbols-regex "^[^_].*"

In general looks good, thanks!

Added the .pc files.
The -export-symbols-regex "^[^_].*" didn't work out. Gave me an error:
  "UniValue::read(char const*)", referenced from:
      _main in unitester-unitester.o

Maybe striping out _ symbols won't work with C++?


Pass through error messages for failed JWT verification

At this point, hapi-now-auth eats all error messages. I guess this behavior is intended to rely on hapi’s default Missing authentication error message.
Would you consider passing error messages through from the JWT verify method call? Your current code looks like this:
try {
  ++keysTried
  decodedJWT = JWT.verify(token, k, settings.verifyOptions)
  return true
} catch (error) {
  if (keysTried >= settings.keychain.length) {
    throw settings.unauthorized(null, settings.tokenType)   // <-- this line here
  }
  return false
}

Passing the error message to settings.unauthorized would help to provide better error feedback to requesting clients. Missing authentication doesn’t help to find out that the token expired.
An idea could be to pass through the message if there’s a custom unauthorized and not the one from defaults.
What do you think about this idea?

@marcuspoehls I'm in favor of returning better error responses, do you want to take the first shot at modifying the error responses?

@puchesjr Great! Do you have resources for this?
I can't submit a PR before Friday (in 5 days).
Would you start (and probably finish) earlier?

Unfortunately we do not have the resources to complete before your PR
timeline.
Your help is greatly appreciated.
On Sat, Mar 3, 2018 at 11:59 PM Marcus Poehls<EMAIL_ADDRESS>wrote:

@puchesjr https://github.com/puchesjr Great! Do you have resources for
this?
I can't submit a PR before Friday (in 5 days).
Would you start (and probably finish) earlier?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/now-ims/hapi-now-auth/issues/2#issuecomment-370204535,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEyA3lU_8lfc30If_n6e0E-x2l77vhrZks5ta4KlgaJpZM4SMuJT
.

--
google is watching

Let me see what we can do. I do prefer getting some community involvement in the project, but I might be able to free up sometime to accommodate the request.


Update Spanish translations

Hi, can you update the translations on Crowdin instead: https://crowdin.com/project/another-notes-app

Hi, thanks! It´s done.


lark.grammar.Symbol.__eq__ raises an AssertionError when checking if None == Symbol(...)

Describe the bug
When attempting to compare a variable set to None with a Symbol, Lark's implementation of __eq__ within the Symbol class raises an AssertionError.
To Reproduce
from lark.grammar import Symbol
a = None
b = Symbol("abc")

# this throws an error:
a == b

The error message:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "[...]/venv/lib/python3.8/site-packages/lark/grammar.py", line 19, in __eq__
    assert isinstance(other, Symbol), other
AssertionError: None

This class is mainly for internal use, but I will accept a PR that returns False if the other object is None.


Cannot use 'in' operator

I keep getting this, even when trying to just load the sample provided.
Why? What is this?

Seems the problem is with the svgProperties constructor. Not sure exactly, as the svgPath code is super confusing to me. But passing the length  is what the stoke offest it looking for, so I got that from the path and passed it, maybe it's wrong, I don't know. but did this, and it worked:

Ideally, wouldn't need to edit the package though to make it work. :)

Thank you @ezos86 , I'll look into this, the svg-path-properties dependency may be removed already, since it's only purpose is to get the length only
this.length = properties.getTotalLength();

and it can be achieved using what you did.

@73R3WY Yeah, it seems like that library is probably doing a lot more, I'm not sure, I believe it's checking for several Types in the SVG string, I didn't spend too much time following it, I just was looking what the return needed for the dash offset, it seems like the  .length works fine.
Thanks for the reply, I think a lot of developers use Lotti now, but this seems like a simple nice module to use. so works for my need 👍

I just did
constructor(props) {
super(props);
const { d, reverse } = this.props;
// const properties = svgPathProperties(d)
// this.length = properties.getTotalLength();
this.length = d.length;
this.strokeDashoffset = new Animated.Value(!reverse ? this.length : 0);
}
and was able to resolve the issue. But animation does seems to work smoothly.

I just did
constructor(props) {
super(props);
const { d, reverse } = this.props;
// const properties = svgPathProperties(d)
// this.length = properties.getTotalLength();
this.length = d.length;
this.strokeDashoffset = new Animated.Value(!reverse ? this.length : 0);
}
and was able to resolve the issue. But animation does seems to work smoothly.

Thanks for confirming this worked for you also. Maybe we can do a merge request to update package. @73R3WY

Yes, @ezos86 , you guys may open a pull request to have the fix and update it in the meantime. It would be very much appreciated. I'm currently making improvements using Typescript for the library.

None of these solutions worked for me on 61.4 rn version it either gives the same error or gives cannot read length of undefined.

Hi @art1373 , can you try editing the files same with the ones edited in here #27 ?

Closing this, merged #27 and updated npm package to 0.2.1.


Running in docker with same url multiple times not working

Hi,
I have a requirement where I need to open the same url multiple times inside the docker container. The url works only first time but timing out on subsequent opens. I doubt if it's caching issue or something else. I used user agent setting as well but to no avail.
I tried both concurrency context and browser context.
Can you help?
Thanks

Can you provide more information? What docker container, what error messages are you seeing, what's the url (or nature of the site your visiting if it's proprietary)?


File opening errors in logger

I'm currently tracking errors from Amplitude to our error analytics, and I've encountered a significant number of errors in our production app. All the errors are similar in nature, and here are a couple of examples:

The file “v2-80” couldn’t be opened because there is no such file.
“v2-13” couldn’t be removed.

I'm concerned about whether these errors could potentially cause any data issues?
SDK Version: 1.6.0
OS Info: iOS 17.*

Hi @slava-semeniuk-ek - Could you please double check the version of Amplitude-Swift you are using? 1.6.0 was just released two days ago. Any stack traces you can provide would also be very helpful.

For the latest version which uses 1.6.0, I could find the following errors:
NSLocalizedDescription: “v2-8” couldn’t be removed.

Logs
AmplitudeErrorLogger.swift - Line 41
specialized AmplitudeMessageLogger.record(issue:message:) + 41
AmplitudeErrorLogger.swift - Line 24
protocol witness for Logger.error(message:) in conformance AmplitudeMessageLogger + 24
PersistentStorage.swift - Line 104
closure #1 in PersistentStorage.remove(eventBlock:) + 104

partial apply for thunk for @callee_guaranteed () -> ()
libdispatch.dylib
_dispatch_sync_f_slow
PersistentStorage.swift - Line 100
PersistentStorage.remove(eventBlock:) + 100

protocol witness for Storage.remove(eventBlock:) in conformance PersistentStorage
IdentifyInterceptor.swift - Line 124
IdentifyInterceptor.getCombinedInterceptedIdentify() + 124
IdentifyInterceptor.swift - Line 157
implicit closure #2 in implicit closure #1 in closure #1 in IdentifyInterceptor.scheduleTransferInterceptedIdentifyEvent() + 157
IdentifyInterceptor.swift - Line 182
closure #1 in IdentifyInterceptor.scheduleTransferInterceptedIdentifyEvent() + 182
QueueTimer.swift - Line 38
closure #1 in QueueTimer.init(interval:once:queue:handler:) + 38

The file “v2-243” couldn’t be opened using text encoding Unicode (UTF-8).

Logs
AmplitudeErrorLogger.swift - Line 24
protocol witness for Logger.error(message:) in conformance AmplitudeMessageLogger + 24
PersistentStorage.swift - Line 94
PersistentStorage.getEventsString(eventBlock:) + 94

protocol witness for Storage.getEventsString(eventBlock:) in conformance PersistentStorage
EventPipeline.swift - Line 70
closure #1 in EventPipeline.flush(completion:) + 70

thunk for @callee_guaranteed () -> ()
_dispatch_lane_barrier_sync_invoke_and_complete
EventPipeline.swift - Line 65
EventPipeline.flush(completion:) + 65
EventPipeline.swift - Line 34
closure #1 in EventPipeline.init(amplitude:) + 34
QueueTimer.swift - Line 38
closure #1 in QueueTimer.init(interval:once:queue:handler:) + 38

Thanks @slava-semeniuk-ek! This was caused by some concurrency issues that were largely resolved in 1.5.0+, but we missed one callback in IdentifyInterceptor, which I've fixed in https://github.com/amplitude/Amplitude-Swift/pull/176. I've also added some additional diagnostic logging which will help us catch similar issues in the future. Our write patterns are such that events in one of the removed files should have been either uploaded or re-written to another file, so we don't believe this has caused any data issues.

This should be released in 1.6.1

@crleona hey, hey, have you had a chance to check the additional diagnostic logging? It looks like these errors are still appearing

@crleona It seems it's still not fixed for 1.7.0. Could you please check the additional diagnostic logging? Let me know If you need any details from my side


Use compact encoding for runtime names to reduce binary size

Turns out Xaml is very dependent on UTF-16 string literals and any deviation has a negative impact on code size for WinUI.


URI with White Spaces (%20) fails

Hi
I'm using your nginx reverse proxy image, and for most it works perfect. Thanks!
For on of my services
REVERSE_PROXY_BACKEND='$backend$uri$is_args$args'
does not work due to white spaces in the URI.
This is the entry from the proxy log
[22/Aug/2017:21:56:54 +0000] "GET /arcgis/rest/services/Geosok/GeosokRute3/NAServer/Closest%20Facility HTTP/1.1" 400 0 "<HTTP ref>" "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:55.0) Gecko/20100101 Firefox/55.0"
And I don't see any trace of the request at my backend request logs, it seems like the request isn't ever sent to the backend at all.
Requests without white spaces works as expected, like the one below
[22/Aug/2017:22:05:32 +0000] "GET /arcgis/rest/services/Geosok/GeosokRute3/NAServer/Route HTTP/1.1" 304 0 "<HTTP ref>" "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:55.0) Gecko/20100101 Firefox/55.0"
If I query the backend directly bypassing the proxy the requests succeeds.
Changing
REVERSE_PROXY_BACKEND='$backend$request_uri'
will make the request work, but unfortunately it breaks some other features of my backend
This is a snippet of my config (Kubernetes deployment)
        env:
        - name: SERVER1REVERSE_PROXY_LOCATION1
          value: /
        - name: SERVER1REVERSE_PROXY_PASS1
          value: https://my.backend.server
        - name: SERVER1HTTPS_ENABLED
          value: 'true'
        - name: SERVER1HTTP_ENABLED
          value: 'false'
        - name: SERVER1CERTIFICATE_DNAME
          value: '/CN=xx/OU=xx/O=xx.xx/L=xx/C=xx'

Any suggestions?

This will provide some related information
https://serverfault.com/questions/445554/nginx-uri-is-url-decoded

Looks like I will have to escape the uri?

Nginx normalizes the $uri by default, and this is par the documentation. I did a dirty hack just to get passed my problem for now, but It's not a good solution in the long run ;)
  REVERSE_PROXY_BACKEND_VARIABLE='$backend'
  REVERSE_PROXY_BACKEND='$backend$uri$is_args$args'
  REVERSE_PROXY_BACKEND_CS='$backend$request_uri$is_args$args'
  REQUEST_URI='$request_uri'

  if [ -n "${NGINX_PROXY_PASS}" ]; then
    cat >> $configFileReverseProxy/reverseProxy.conf <<_EOF_
          set ${REVERSE_PROXY_BACKEND_VARIABLE} "${NGINX_PROXY_PASS}";
          if (${REQUEST_URI} ~ .*.%20.*) {
             proxy_pass ${REVERSE_PROXY_BACKEND_CS};
          }
          if (${REQUEST_URI} !~ .*.%20.*) {
             proxy_pass ${REVERSE_PROXY_BACKEND};
          }
_EOF_

Is there a good solution for the long run?

The best solution would be to URL encode the $url variable before using it in the proxy_pass declaration
BTW, I had a small bug in my hack, needed to remove $is_args$args from my custom backend, as $request_uri already contains the complete URL including query strings.
REVERSE_PROXY_BACKEND_CS='$backend$request_uri'

How do I Url encode in that context?

I have no idea to be honest. Then I would have done a pull request ;)

What kind of encoding do you need?
https://github.com/openresty/set-misc-nginx-module#set_escape_uri
or
https://www.nginx.com/resources/wiki/modules/set_misc/#set-unescape-uri


239 | Sliding Window Maximum | C++

Sliding Window Maximum -
What will change - A well documented and clean C++ solution will be added.
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Please assign this issue to me

Hey There!! Can You plese assign this to me??

I have written the code for the mentioned problem and will be glad to help if you assign me for the same.

This problem already exists in the codebase.


Make the top nav bar for cell card should be sticky

The top bar with the cell name and filters should be fixed to the top and have a gradient at the bottom.
background: var(--fade, linear-gradient(180deg, #FFF 90%, rgba(255, 255, 255, 0.00) 100%));