Send draft item links to Publishing Api

links information is missing in the publishing-api for draft content.
This is causing errors when importing data into Content Publisher as draft documents are missing primary_publishing_organisation within links.
This task will patch the links information for draft documents. It looks like it was previously only run for published and withdrawn content.
It seems like we may be encountering this issue for similar reasons to: https://github.com/alphagov/whitehall/pull/2792#issuecomment-255812139 - only in our case for draft rather than withdrawn content.

Thanks for putting this together Rosa.  Have we looked into this was a legacy issue for old draft NDA content, and whether patching links occurs automatically if an organisation creates new content now?  My concern is we would need to remember to run this right before kicking off the import process.
Could you also add a link to the Trello card in the description?  A link to this PR will then automatically get added to the card.

@emmabeynon I tested on Whitehall integration and drafts do have the links data when created, so it should be a case of fixing the existing draft documents.
Will link the PR :)


remove import file extension

Really no reason to have this, and it breaks the build when importing src/

Coverage decreased (-0.02%) to 69.96% when pulling 7cfc108fb457b765d3b28c8416a7614b9b9ad838 on jgillich:patch-1 into 66688c19a6fb336fb68fb085b98e121caf9df018 on sourcelair:v3.

Yeah that should not be there, thanks @jgillich 😄


fasthttp plugin

https://github.com/valyala/fasthttp

https://github.com/pinpoint-apm/pinpoint-go-agent/blob/main/doc/plugin_guide.md#ppfasthttp


When ReachableTask decides that a device is unreachable, all further tasks need to be stopped

We have chosen to split Zino 2's work into multiple tasks that run in serial, so they can be neatly and conceptually separate.  However, the ReachableTask always runs first, and if it decides that a device isn't reachable, it should have some way to signal to the task-running-loop that the remaining tasks can be cancelled.
Right now, the logs fill up with TimeoutErrors for the remaining tasks every time a device is unreachable.
A quick suggestions might be to raise some sort of exception after scheduling the extra reachability checks, like a DeviceUnreachableError  or something.

Made something in #124 but not sure if im all too happy with it

A device being unreachable does not cause the RreachableTask to not be able to complete its purpose, because handling a device being unreachable IS the purpose. So raising an exception does not really make sense.

Well, I tend to disagree.  The purpose of ReachableTask is to verify that a device is reachable.  When a device isn't, that is an exceptional situation, and there is nothing wrong with using exceptions for flow control in my mind.


Export as .rs

Would it be possible to somehow parse the .ipynb files and convert them to a .rs file? And maybe put the dependencies in a Cargo.toml file as well.
Not everyone using rust uses evcxr and if I need to share my project with them, this function would come in handy.

Thanks for the suggestion @TahsinTariq. Sounds like a good feature to me. I'll have to think about the simplest way to implement it.

I wrote myself a solution based on scriptisto and jupytext.
scriptisto is a tool to run text files as scripts.
jupytext can convert between text files and jupyter notebooks. It can be run as a plugin for the jupyter server to convert on every save. The percent encoding format works quite well with Rust. It inserts // %% between every cell.
This should be the content of the first cell telling scriptisto how to run the file. The rs-file should then start with #!/usr/bin/env scriptisto and then it can be run as an executable.
// scriptisto-begin
// script_src: src/main.rs
// build_cmd: sed -i '1ifn main\(\)\{' src/main.rs && sed -i '$a;\}' src/main.rs && sed -i 's+^// %%$+;+g' src/main.rs && cargo build --release && strip ./target/release/script
// target_bin: ./target/release/script
// files:
//  - path: Cargo.toml
//    content: |
//     package = { name = "script", version = "0.1.0", edition = "2018"}
//     [dependencies]
//     serde = { version = "*", features = ["derive"] }
//     serde_json = "*"
// scriptisto-end

The problems I encountered were:

A main file is needed for a Rust program. So the code above simply wraps the whole content into a fn main() {...}.
Code cells can end in an expression, which is simply printed in a notebook. As Rust code this is a syntax error. So the snippet above solves this by ending each code cell with a ;.
External dependencies are not synchronized between the :dep entries and the embedded Cargo.toml file. But at least they can both be edited in the notebook and the rs-file.


Common configuration for WFP open data cube layers

We have a number of countries that will use a common set of layers that are accessed through WFP's open data cube instance as WMS layers. Currently, any change to a color ramp, legend text, etc. has to be made for each individual PRISM instance.
Is there a way we can have these common configurations read from a single source so that we can ensure they are always in-sync and avoid errors in repeatedly updating them for each instance? If not a software solution, any suggestions on a file management solution to make this more efficient? Any thoughts @ericboucher?

@wadhwamatic I tried to create common configuration on src/config/shared/layers.json. The file will be referenced by country specific config. Now the layers.json files are smaller because of the shared layers. You can see it on feature/shared-config.
There are many config changes, I wrote a small script for that. The result is here:
summary-changes.txt
Basically, for each country it will show:

matches: The shared layers that will be used
modifies: Shared layers that will be used but then there are some changes or additional attributes like "additional_query_params" or "chart_data" or "content_path", etc.
unique_layers: ODC layers that are not referenced by any other countries. Those layers won't be taken to shared/layers.json file
renames: Shared layers that have the same URL but different layer id, usually because old naming. For example "rain_anomaly_monthly" (old name) will be changed to "rain_anomaly_1month" (new name in shared layers)
unused: Showing layer ids registered in layers.json but not referenced in prism.json, probably old layers that are not used anymore

For now, the src/config/<country>/index.ts will have repeated layer merge code for each country, the function cannot be extracted because of strict typing on layers.json. I think with generics it would be possible, but I haven't tried much. Also if we want to make extending shared layers mandatory, this can be simplified, the merge code will only be needed on config/index.ts. But I don't think that's a good idea either.
Please review, if it's good, I can make a PR for feature/shared-config branch.

Thanks for the work on this @reed1. I would suggest that we remove admin_boundaries and wp_pop_cicunadj from the shared layers file. We may later reintroduce wp_pop_cicunadj since the way we deploy that layer might change.
I'd like to have @ericboucher review how this is implemented, especially the changes to country-level index.ts files. I'm not sure about adding the modifiedLayers logic there.
Please create a PR based on the feature/shared-config branch.

@ericboucher - surfacing this again as part of the potential work on simplifying layer configuration


Change URL to URI

https://github.com/reearth/cesium3dtiles-go/blob/5fab94dc3d59172b982081c89d95e99f71d5473a/tileset/tileset.go#L43
According to the 3d-tile specifications the dataset reference should be done with "URI" instead of "URL"
https://github.com/CesiumGS/3d-tiles/tree/main/specification#content

Fixed in #5


ENH Use v2 of the tag-release action

Updates to using the new major release of gha-tag-release
Issue

https://github.com/silverstripe/gha-ci/issues/137

Approved but not merged? This doesn't rely on anything else now.


Show Free Ram

Suggested by a user, show free ram instead of used ram

Oops this a stats monitor issue, this was moved haha


Specify exe path as TargetObect for non-zero exit code ErrorRecord

PR Summary
This PR changes the TargetObject for the ErrorCode to the full native command's path.  This ErrorRecord is generated as part of the experimental feature PSNativeCommandErrorActionPreference.

PR Context

Fix #15992
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[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: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

I can't merge this.  @rjmholt or @iSazonov - can you merge this?


request attribute voter

hello,
could you please add a voter based on request attribute?
i am using this a lot and i believe it could be useful as a core part of the bundle:
use Knp\Menu\ItemInterface;
use Knp\Menu\Matcher\Voter\VoterInterface;
use Symfony\Component\HttpFoundation\RequestStack;

class RequestAttributeVoter implements VoterInterface
{
    private RequestStack $requestStack;

    public function __construct(RequestStack $requestStack)
    {
        $this->requestStack = $requestStack;
    }

    public function matchItem(ItemInterface $item): ?bool
    {
        $request = $this->requestStack->getMainRequest();

        if (null === $request) {
            return null;
        }

        $menuItem = $request->attributes->get('_menu_item');

        if (null === $menuItem) {
            return null;
        }

        return $item->getName() === $menuItem;
    }
}

with this as last registered voter, one can easily "override" the Route & Uri voter
class SomeController
{
    public function someAction(Request $request)
    {
        $request->attributes->set('_menu_item', 'someMenuItem');
        //...
    }
}

Thank you for your proposal.
I'm not sure that this can be of general interest, let's hear what @stof think of it.

I don't think this voter makes sense in the core (it is flawed, as item names are only unique among their siblings, not globally). I suggest keeping it as a custom voter in your project.

Closing, with two negative opinions.
Thanks anyway for your suggestion @kor3k
Don't hesitate to submit further proposals.


A new button or filter in the Issues menu

Can you make a new button or filter in the Issues menu called Closed as not planned and Closed? Because there should be a difference between the two.

Still doesn't happen.


How to add support for audio files?

Hi @indrajithi
There are some restrictions in the javascript library used (jquery-file-upload). If you search for "png", you should see where. Just add the allowed mimetypes where the library are restricting mimetypes (acceptFileTypes).
The Python/Django code should be allowing these files already.


JAVA-2740: Ignore peer entries with data_center null

This PR intention is to provide a fix JAVA-2740 to ignore peer entry when a null entry for schema_version is found.
Right now it will just throw a java.lang.IllegalArgumentException: Multiple entries with same key
edit: After @adutra review, I have moved to the check from schema_version to data_center column

rebased from 4.x branch


Add convenience function for verifying token flags

Current dependencies on/for this PR:

main

PR #18

PR #19   👈

PR #20

This comment was auto-generated by Graphite.


Switch to dot.js

http://olado.github.io/doT/index.html
Project looks interesting and it's quite light. With this, bootform will be much more suitable for
a browser environment.

This library is no longer in use.


Missing legalization for mhlo.round_nearest_afz

Repro:
from iree.compiler import compile_str

CODE = """
module  {
  func @main(%arg0: tensor<1x4xf32>) -> tensor<1x4xf32> {
    %0 = "mhlo.round_nearest_afz"(%arg0) : (tensor<1x4xf32>) -> tensor<1x4xf32>
    return %0 : tensor<1x4xf32>
  }
}"""

compiled_flatbuffer = compile_str(CODE, target_backends=["dylib"], input_type="mhlo")

Get:
$ python t.py
Traceback (most recent call last):
  File "t.py", line 11, in <module>
    compiled_flatbuffer = compile_str(CODE, target_backends=["dylib"], input_type="mhlo")
  File "/usr/local/google/home/phawkins/.pyenv/versions/py373a/lib/python3.7/site-packages/iree/compiler/tools/core.py", line 262, in compile_str
    result = invoke_immediate(cl, immediate_input=input_bytes)
  File "/usr/local/google/home/phawkins/.pyenv/versions/py373a/lib/python3.7/site-packages/iree/compiler/tools/binaries.py", line 201, in invoke_immediate
    raise CompilerToolError(process)
iree.compiler.tools.binaries.CompilerToolError: Error invoking IREE compiler tool ireec
Diagnostics:
<stdin>:4:10: error: failed to legalize operation 'mhlo.round_nearest_afz' that was explicitly marked illegal
    %0 = "mhlo.round_nearest_afz"(%arg0) : (tensor<1x4xf32>) -> tensor<1x4xf32>
         ^
<stdin>:3:3: note: called from
  func @main(%arg0: tensor<1x4xf32>) -> tensor<1x4xf32> {
  ^
<stdin>:2:1: error: conversion from source -> vm failed
module  {
^

Invoked with:
 ireec /usr/local/google/home/phawkins/.pyenv/versions/py373a/lib/python3.7/site-packages/iree/compiler/tools/../_mlir_libs/ireec - --iree-input-type=mhlo --iree-vm-bytecode-module-output-format=flatbuffer-binary --iree-hal-target-backends=dylib --iree-mlir-to-vm-bytecode-module --iree-llvm-embedded-linker-path=/usr/local/google/home/phawkins/.pyenv/versions/py373a/lib/python3.7/site-packages/iree/compiler/tools/../_mlir_libs/iree-lld --mlir-print-debuginfo --mlir-print-op-on-diagnostic=false

Fix was upstreamed and was locally validated.

Fix was upstreamed and was locally validated.

These tests are still disabled: https://github.com/google/iree/blob/e9c786761f91f02e854875ca5b8b257cc706a993/iree/test/e2e/xla_ops/round.mlir#L1-L6
https://github.com/google/iree/blob/e9c786761f91f02e854875ca5b8b257cc706a993/iree/test/e2e/xla_ops/BUILD#L84-L86
and I get compile errors trying to enable them:
[build] FAILED: iree/test/e2e/xla_ops/check_vmvx_vmvx_round.mlir_module.vmfb D:/dev/projects/iree-build/iree/test/e2e/xla_ops/check_vmvx_vmvx_round.mlir_module.vmfb
[build] cmd.exe /C "cd /D D:\dev\projects\iree-build\iree\test\e2e\xla_ops && D:\dev\projects\iree-build\iree\tools\iree-translate.exe -iree-mlir-to-vm-bytecode-module -mlir-print-op-on-diagnostic=false --iree-hal-target-backends=vmvx -iree-input-type=mhlo D:/dev/projects/iree/iree/test/e2e/xla_ops/round.mlir -o check_vmvx_vmvx_round.mlir_module.vmfb -iree-llvm-embedded-linker-path=\"D:/dev/projects/iree-build/third_party/llvm-project/llvm/bin/lld.exe\""
[build] D:/dev/projects/iree/iree/test/e2e/xla_ops/round.mlir:3:13: error: failed to legalize operation 'math.copysign' that was explicitly marked illegal
[build]   %result = "mhlo.round_nearest_afz"(%input) : (tensor<7xf32>) -> tensor<7xf32>
[build]             ^

What fix was upstreamed? Should we update our tests to use a different op?

Or maybe mhlo.round_nearest_afz now lowers further, but we also need math.copysign to support the op end to end?

Tested how math.copysign is handled a bit more

backend
result

llvm CPU
lowers successfully

llvm WebAssembly
same error as on https://github.com/google/iree/issues/8514

CUDA
success, ConvertToNVVM lowers to llvm.intr.copysign

SPIR-V
missing lowering in ConvertToSPIRV

VMVX
missing lowering in IREE::VM::ConversionPass

mhlo-to-linalg was changed to lower to math.round. This works for llvm.round however glsl.round does not guarantee rounding away-from-zero.

Yeah. GLSL's round is fuzzy on how to handle the 0.5 situation; we'd need to emit additional ops to check and handle that case by ourselves.

Related: https://github.com/iree-org/iree/issues/9617


-q isn't always effective

No a big issue, but -q doesn't suppress the informational output from create, at least.

Ahh, good catch. I'll hammer on that shortly!

Fixed in [master 05a0f7d], thanks!


Display shortcuts in full screen mode?

I use this code to display shortcuts on my WebRTC cameras:
style: >-
.shortcuts {left: 40px; top: 10px; right: unset; display: flex;
flex-direction: line}
.mode {display: none}
but they don't show if I click the card to display the camera full screen. Is there a way to make the shortcuts show on full screen, or could that be added? Thanks.

You need to enable option ui: true


Support for iOS and Android

Hello,
Looks exciting with the 2.0 version SikuliX :-)
Will it support iOS and Android?
Best,
Goncalo

If you are asking, wether SikuliX might run natively on iOS or Android:
I still have no idea, wether this is possible at all (depends on Java and OpenCV support natively).
I might have an idea though in the second half of this year, when the basic features are stable.
Nevertheless: feel free, to make any suggestions or give me hints how to proceed, to get nearer to more than a vague idea.

@TheJusticeLeague   I actually managed to get Sikuli working with an Android app.  A bit convoluted but works fine.  I USB tethered an Android phone, turned on developer mode on the phone and connect with this app (https://www.vysor.io/) on my laptop.
Sikuli can click around just fine.
However, doing mobile things like "swipe" and other gestures are a bit difficult on Sikuli.
@RaiMan   Perhaps you can share how to do mobile gestures input with Sikuli.  I found some
info online but have not tried them yet.

And to be honest, I suspect Sikuli can work with iOS simulator.  But I have not tried yet.

Hey! Sorry for the late response!
I was just trying to use image recognition in testing an iOS app via Sauce Labs. I have had problems with taking a screenshot just using the appium driver. It worked ok with Android. But I could see that SikuliX has the capability as well and I was wondering if you guys have tried it with iOS apps. In Android it worked fine :-)

@TheJusticeLeague
SikuliX currently does not have any features, that do something directly on an attached iOS device.
The here mentioned experimental Android feature is special for Android using ADB, which in turn is not available for iOS.
Later this year, when version 2 is stable I will surely come back to this area.

@RaiMan
Just to follow up on this, I got it to work with the following.
Using Sauce Labs and local device:

Image Recognition in Android
Image Recognition in iOS
Actions (Tap, long tap, etc..) on Android

What I am still in doubt on how to do is the Touch actions in iOS based on coordinates! :-)
I am trying to figure something out that I will gladly share with you :-)

I'm using bignox.com android virtual machine (virtual box underneath) to automate games, works great. Running 5x VMs with 5x accounts.
Would be great if there was a click and hold action - for now I just click a bunch of times (the game allows for holing on the action button to then pop up a dialog to say "click/apply x times")

You can click and hold by moving the cursor to a match then using
"MouseDown" and "MouseUp" when you are done holding. This is also how you
drag or swipe ( by moving mouse during hold)
On Sun, Apr 9, 2017, 9:52 PM Nicholas Orr<EMAIL_ADDRESS>wrote:

I'm using bignox.com android virtual machine (virtual box underneath) to
automate games, works great. Running 5x VMs with 5x accounts.
Would be great if there was a click and hold action - for now I just click
a bunch of times (the game allows for holing on the action button to
then pop up a dialog to say "click/apply x times")
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RaiMan/SikuliX2/issues/21#issuecomment-292849423, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAsmexke0YpzHhgqJerl0-mF4NXdwXWSks5rubWpgaJpZM4LwlGm
.

--
-Sean
http://bellriverranch.com

@goncalosoares  or @RaiMan  I am using Sikulixapi.jar, in my local it working fine but in sauce lab it failing can anyone help me on this.


[Feature] Filter Criteria for Late Filtering

PR Summary
This PR addresses #121 and introduces a query transformer, LateFilter, which enables filtering retrievables at a later stage in the query processing pipeline.
Similar to the FieldLookup transformer, LateFilter fetches a specified field. LateFilter then compares its value against a predicate. Only retrievables that satisfy the predicate are emitted.
Setup and Usage
The LateFilter can be configured by adding the transformer to the query, as shown in the following example:
{
  "inputs": {
    "input-text": {"type": "TEXT", "data": "an orange starfish on the seafloor"}
  },
  "operations": {
    "clip": {"type": "RETRIEVER", "field": "clip", "input": "input-text"},
    "relations": {"type": "TRANSFORMER", "transformerName": "RelationExpander", "input": "clip"},
    "aggregator": {"type": "TRANSFORMER", "transformerName": "ScoreAggregator", "input": "relations"},
    "filter0": {"type": "TRANSFORMER", "transformerName": "LateFilter", "input": "aggregator"},
    "filter1": {"type": "TRANSFORMER", "transformerName": "LateFilter", "input": "filter0"},
    "filter2": {"type": "TRANSFORMER", "transformerName": "LateFilter", "input": "filter1"}
  },
  "context": {
    "global": {
      "limit": "100"
    },
    "local": {
      "relations": {"outgoing": "partOf"},
      "filter1": {"field": "time", "keys": "start", "comparison": ">=", "value": "22480000000", "append": "true"},
      "filter2": {"field": "time", "keys": "end", "comparison": "<=", "value": "34868000000", "append": "true", "limit": "3"},
      "filter0": {"field": "file", "keys": "path", "comparison": "~=", "value": "%.mp4", "append": "true"}
    }
  },
  "output": "filter2"
}

Supported Comparison Operations
The LateFilter supports the following comparison operations:

== : Equals
!= : Not equal
< : Less than
> : Greater than
<= : Less than or equal to
>= : Greater than or equal to
~= : Like (pattern matching)

Additional Configuration Options
LateFilter provides the following options:

"append": Boolean        If set to true, the specified property is added to the retrievables (similar to FieldLookup).
"limit": Int Limits the number of emitted retrievables to the specified value.

I'm not sure this is the best approach. I would either have an operator that pushes down the filtering to the database or one that filters based on existing information, such as generated by the FieldLookup.

I'm not sure this is the best approach. I would either have an operator that pushes down the filtering to the database or one that filters based on existing information, such as generated by the FieldLookup.

It certainly isn't the best approach, but that's not the goal here.

Your first suggestion is - of course - the desired outcome. But the facilities in place are not geared towards this yet and making this happen (end-to-end) constitutes a major undertaking.
The latter may be better in terms of code repetition. But if we think in terms of execution planning (i.e., thinking towards the ideal end goal), spreading such a primitive over multiple operations will complicate the planning process.

Then at the very least, the logic should check if the field is already present before again performing a lookup.


test(graphql-transformers-e2e-tests): prevent SES ratelimit errors

Description of changes
This commit prevents emails being sent when new users are created during e2e tests.
Without this commit, running e2e tests could cause this error:
Failed when signing up user
LimitExceededException: Exceeded daily email limit for the operation or the account. If a higher
limit is required, please configure your user pool to use your own Amazon SES configuration for
sending email.

Description of how you validated changes
I ran an integ test without this change. Failed with the above error. I then ran the same test with this change built in, and it passed.
Checklist

[x] PR description included

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #7412 (0bb513a) into master (be441c9) will increase coverage by 0.00%.
The diff coverage is 64.28%.

@@           Coverage Diff           @@
##           master    #7412   +/-   ##
=======================================
  Coverage   52.74%   52.75%
=======================================
  Files         514      514
  Lines       25919    25916    -3
  Branches     5059     5056    -3
=======================================
  Hits        13672    13672
+ Misses      11280    11277    -3
  Partials      967      967

Impacted Files
Coverage Δ

...ider-awscloudformation/src/display-helpful-urls.js
16.90% <33.33%> (+0.35%)
:arrow_up:

...wscloudformation/utils/auth-sms-workflow-helper.ts
45.45% <80.00%> (+21.64%)
:arrow_up:

...r-utils/awscloudformation/utils/message-printer.ts
50.00% <100.00%> (-10.00%)
: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 53c115f...0bb513a. Read the comment docs.


Errors with subpackage handling

Using 2.8.3 to compile docs for the Joomla CMS repo, we're seeing some errors with the handling of the @subpackage tags in our non-namespaced classes.  For reference, our existing style rule is to only include the tags in our file doc blocks.

Classes are listing two @package tags in the main class section (output from IIRC the 2.8.1 release, same issue - https://api.joomla.org/cms-3/classes/JPlugin.html)
In the package listing, the @subpackages are correctly listed in the sidebar but the @subpackage pages are empty (example - https://api.joomla.org/cms-3/packages/Joomla.Platform.Controller.html)

There are a couple of places where there is still a @package/@subpackage tag in the class doc block, such as https://api.joomla.org/cms-3/classes/JRouteWrapperRoute.html, and its routing chain is correct.  FWIW our decision to remove these tags from the class blocks came after https://github.com/joomla/coding-standards/issues/80.
The issue exists in the main responsive-twig template as well, just to rule out any customizations we've made in wrapping elements from that template in Joomla's site template.

I am seeing the same problem with the responsive template on a project that has substantial legacy coding standards which there is no reason to change. None of the subpackage elements are being listed correctly with phpdoc2.
We are still using phpdoc1 for the live documentation, but that has not been updated for some time so I am just starting the exercise to move to using phpdoc2 ... but with substantial new errors appearing.

Updated original issue description for latest build on 2.8.5.


Bulk upsert to TopicAssignment table does not work

Summary
force:data:bulk:upsert does not work for the TopicAssignment table. I've tested using workbench and other anon apex and can confirm my source data is good and permissions enabled.
Steps To Reproduce:

sfdx force:data:bulk:upsert -s TopicAssignment -i Id -f ~/path/to/csv

CSV Header:
TopicId, EntityId
0TO...,001...
Expected result
The data load should complete with the bulk api like other tools.
Actual result
ERROR running force:data:bulk:upsert:  No create/update access for object:TopicAssignment
Additional information
Feel free to attach a screenshot.
SFDX CLI Version(to find the version of the CLI engine run sfdx --version):
sfdx-cli/7.6.0-28071a499b darwin-x64 node-v10.15.3
SFDX plugin Version(to find the version of the CLI plugin run sfdx plugins --core)
salesforcedx 45.13.0 (core)
OS and version:
macOS 10.14.5

@rickschmidt we've got an updated version of the force:data commands coming out.  Would you be willing to re-test this bug using that?  And if it still doesn't work, let me know?
Here's the instructions for previewing that new version.
https://github.com/forcedotcom/cli/issues/907

No it does not work after updating to the preview. I verified my CSV with another tool.
After looking at this with 2021 eyes I don't think it will ever work as long as you can not discriminate between an insert and an update operation(unless I'm missing something in the preview or existing functionality). Since Externalid is required an upsert is implied.  TopicAsssignment only permits inserts as verified through describe calls.


Add abidiff check to CI

Description
Closes #1085. Add new CTest target and CI job that runs it. ABI snapshots are committed to the repository and checked against with every PR.

[ ] I have updated the release notes.
[x] I have updated all relevant user documentation.

Reviewer Notes
Deliberately breaking commits are provided to show the output.

Passing CI job: https://github.com/OpenAssetIO/OpenAssetIO/actions/runs/9315770655/job/25642641211?pr=1319
Deliberate failures incoming.

drop! commit added to illustrate the output of breakages (see commit message for what breakages were introduced).
I tried to investigate the rather odd entry in the output
'class openassetio::v1::pluginSystem::CppPluginSystemManagerImplementationFactory at CppPluginSystemManagerImplementationFactory.hpp:48:1' changed:
  type size hasn't changed
  there are member function changes:

when CppPluginSystemManagerImplementationFactory hasn't changed.
Using the --dump-diff-tree can help.  Ultimately its because of the change to ManagerInterface, and that warning is saying the CppPluginSystemManagerImplementationFactory is affected by it.  However, this shouldn't show up as an independent ABI change. It's also odd that the list of "member function changes" is empty. So I think its a bug in libabigail.
libabigail 2.5 is out, but not available on CentOS 7 (i.e. our Docker image) so we're stuck on 2.4 for now..


Error message does not display after user try to delete Region and PhysLocation and Divisions

What does this PR (Pull Request) do?

[x] This PR fixes #4939

Which Traffic Control components are affected by this PR?

Traffic Portal

What is the best way to verify this PR?
Follow recreation steps on issue and verify it is not happening anymore.

If this is a bug fix, what versions of Traffic Control are affected?
master (0cf375d)

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Should be fixed now


Could not see the wifi printer

We setup a wifi printer. The android device sees it, and can send document to it. But the demo app can not see the printer. How to solve it? Thanks!

@bpcuong2013 What is the printer model? also try testing this on the Official Star Demo application for android that comes with the SDK  and make sure it works there so we know if it is a connection problem or the plugin. Also check that you are using the correct emulation for your printer model

Our printer model is EPSON - Thermal printer.
We also tried the app below, and it has the same issue.
https://play.google.com/store/apps/details?id=com.StarMicronics.StarIOSDK&hl=en

Hi @bpcuong2013 this plugin is only compatible with STAR Micronics printers. You need to search for a plugin for EPSON printers


Add Nong Song: Imperial Circus - Dead Decadence (Yomi Yori Kikoyu / Koukoku No Tou To Honoo No)

Song Name
Imperial Circus - Dead Decadence (Yomi Yori Kikoyu / Koukoku No Tou To Honoo No)
Artist Name
Imperial circus
Source
Youtube
Youtube Link
https://www.youtube.com/watch?v=DWtbMzghaMM
Direct File Link
No response
Song ID
847746
Start Offset [ms]
No response
Extra
Orochi uses this song. I thought people would have added it by now but it apparently isn't.

Accept

Oops


Could you post the code of "Sales_item.h"? where is that? thanks

Just in the repo: https://github.com/Mooophy/Cpp-Primer/blob/master/include/Sales_item.h

Thanks for the quick response! How to download this? If I copy this (or any solution you posted), it will take the number ahead of each line, it is tedious to delete the numbers one by one. （这个网站帮助真大！没想到是中国人办的，请教怎么把程序从你们的网页复制到txt里呢？我直接粘贴的话，每行前面都会有那个代表行号的数字，一个个删太麻烦了，谢谢！）

If I copy this (or any solution you posted), it will take the number ahead of each line, it is tedious to delete the numbers one by one.

clicked Raw, and then copy it more easily.

这个网站帮助真大！没想到是中国人办的，请教怎么把程序从你们的网页复制到txt里呢？我直接粘贴的话，每行前面都会有那个代表行号的数字，一个个删太麻烦了，谢谢！

这个网站是 Github 的。简介在这里：https://github.com/about
如果有使用上的疑惑，在这里查询：https://help.github.com/

thanks


Duplicate Symbol DEVICE_PERIPHERALS

The last commit changed the nrf9160-hal to version 0.16.0.  I can't get the examples to build with this version due to a duplicate symbol error
  = note: rust-lld: error: duplicate symbol: DEVICE_PERIPHERALS
          >>> defined at lib.rs:3227 (src/lib.rs:3227)

Changing it back to version 0.15.1 fixes it.  I'm not sure that is the best fix though.

I've created a PR to address this https://github.com/tweedegolf/nrf9160-rust-starter/pull/6

PR merged :)


kubectl get pod existing not-existing does not show the existing pod

expected: a table with existing and not existing
actual: kui only presents the error part ("not-existing" not found)

i forgot to add the new test


”Breaza” is listed in the drop-down menu

Description:
"Breaza" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Breaza" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Breaza, MUREȘ".


Worked with Faless(Fabio Alessandrelli) to update server platform.

For https://github.com/godotengine/godot/issues/8361
It's an updated version of the server platform. Uses dummy for the driver name. (Any name changes can be discussed elsewhere.)
Add dummy rasterizer driver.
Disable GLES builders and source from server compilation.
Add dummy texture handling.
Server platform now compiles and run on linux.
Seems to also be able to do exports of some demos I tried.
Fixes to OS_Server and DummyRasterizer to match new signatures.
Add dummy audio driver.
Update to newer api.

I've rebased your changes on top of my commits so It's easier to review, and nicer for the commit log.
https://github.com/godotengine/godot/compare/master...Faless:server_again_ifire .
Additionally, I would discourage the usage of the override keyword as there is no need for it.

Looking for review and merge.

I don't understand, what branch should be considered for merging? ifire:server_again_2 from this PR or  Faless:server_again_ifire linked by @Faless?

Looking for review and merge.

Need an answer to @Faless' comment before that:

Additionally, I would discourage the usage of the override keyword as there is no need for it.

Personal opinion, override (c++11) should be used whenever implementing a virtual function. It uses the compiler to help detect if the method made is the exact method overridden.
But consistency in the engine source is a good goal too.

We still have to decide the fate and rules for c++11 in the Engine, I think
we can add it later we those rules are set.
On Feb 14, 2018 16:29, "K. S. Ernest (iFire) Lee"<EMAIL_ADDRESS>wrote:

@akien-mga commented on this pull request.
In drivers/dummy/rasterizer_dummy.h
https://github.com/godotengine/godot/pull/16653#discussion_r168210232:

@@ -39,584 +39,584 @@

class RasterizerSceneDummy : public RasterizerScene {
public:

/* SHADOW ATLAS API */

/* SHADOW ATLAS API */

Something went wrong here. IDE misconfiguration, or wrong version of
clang-format? (We need 5.0).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/16653#pullrequestreview-96540004,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABnBbquC6SD6hiw7_V_VoFygmeFzxeJsks5tUvxygaJpZM4SDENV
.

But consistency in the engine source is a good goal too.

For now consistency is more important, we shouldn't start doing such changes in some places based on personal preference.
If there's a reason to use those everywhere consistently, it should be discussed, agreed upon and added all at once.

Submitted the whitespace changes, removed override and added a comment on a pointer return that looks suspicious.

@Faless how does this look?

Tried to export a project on WSL (Windows 10)'s Ubuntu.
$ ../fire-godot/bin/godot_server.server.tools.64 --export "Windows Desktop" ../Pong.exe
ERROR: initialize: AudioDriverManager: all drivers failed, falling back to dummy driver
   At: servers/audio_server.cpp:153.
ERROR: create: Cannot find path for config directory!
   At: editor/editor_settings.cpp:771.
handle_crash: Program crashed with signal 11
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x354b0) [0x7f6e2ad754b0] (??:0)
[2] ../fire-godot/bin/godot_server.server.tools.64() [0xd03853] (/mnt/d/Projects/fire-godot/editor/editor_file_system.cpp:1047)
[3] ../fire-godot/bin/godot_server.server.tools.64() [0xcfe067] (/mnt/d/Projects/fire-godot/editor/editor_file_system.cpp:279)
[4] ../fire-godot/bin/godot_server.server.tools.64() [0xcfe642] (/mnt/d/Projects/fire-godot/editor/editor_file_system.cpp:298)
[5] ../fire-godot/bin/godot_server.server.tools.64() [0xc6077d] (/mnt/d/Projects/fire-godot/drivers/unix/thread_posix.cpp:72)
[6] /lib/x86_64-linux-gnu/libpthread.so.0(+0x76ba) [0x7f6e2bbe76ba] (??:0)
[7] /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7f6e2ae473dd] (??:0)
-- END OF BACKTRACE --```

virtual bool has_environment(const String &p_var) const;
virtual String get_environment(const String &p_var) const;
virtual String get_locale() const;

virtual int get_processor_count() const;

Aren't implemented.

https://github.com/godotengine/godot/blob/master/editor/editor_file_system.cpp#L278
Removing these lines allowed the export to finish.
f->close();
memdelete(f);

diff --git a/editor/editor_file_system.cpp b/editor/editor_file_system.cpp
index dca32d749..1251a0e56 100644
--- a/editor/editor_file_system.cpp
+++ b/editor/editor_file_system.cpp
@@ -276,8 +276,6 @@ void EditorFileSystem::_scan_filesystem() {

 	f = FileAccess::open(fscache, FileAccess::WRITE);
 	_save_filesystem_cache(new_filesystem, f);
-	f->close();
-	memdelete(f);

 	scanning = false;
 }
@@ -1044,6 +1042,10 @@ void EditorFileSystem::_save_filesystem_cache(EditorFileSystemDirectory *p_dir,

 	if (!p_dir)
 		return; //none
+
+	if (!p_file)
+		return; //none
+
 	p_file->store_line("::" + p_dir->get_path() + "::" + String::num(p_dir->modified_time));

 	for (int i = 0; i < p_dir->files.size(); i++) {

This is not correct, but the problem is around here.

Well your problem is:
ERROR: create: Cannot find path for config directory!
   At: editor/editor_settings.cpp:771.

Then all files it tries to open will be invalid, and if there aren't null checks (like here), it will crash. But that's likely unrelated to this PR and just related to caveats of running Linux binaries on WSL.

Meanwhile, do you think #22422 can be a good temporary solution?

I merged #22422 for now as it's an OK workaround for the issue until something better can be implemented. It might be worth it to open a new issue to keep track of what needs to be done (and remember to revert/undo #22422 at that time).


Unity 5.5.0b7 changed some Spatial Mapping API

Origami (and all catch up chapters) need some minor mods to ".\Assets\Holograms\Support\Spatial Mapping\SpatialMapping.cs"

This was done in #21, right?


update dameng driver version,add categories

Update to the latest  dameng driver.add categories

Hello @Stkai
What do we need to test with the latest Dameng driver version? Maybe some changes? Are some bugs fixed or?

Hello @Stkai
What do we need to test with the latest Dameng driver version? Maybe some changes? Are some bugs fixed or?

It is not necessary to update the driver. Previously, the Dameng driver did not have a category and could not be found in the category.

It is not necessary to update the driver.

I'm talking about this update that you made: <IP_ADDRESS>.
Do we need to check something related to the updated driver?

It is not necessary to update the driver.

I'm talking about this update that you made: <IP_ADDRESS>. Do we need to check something related to the updated driver?

Sorry. I misunderstood, No testing required

Verified


Cannot install node

Operating system and version: macOS Ventura 13.1
nvm debug output:

nvm --version: v0.39.2
$TERM_PROGRAM: Apple_Terminal
$SHELL: /bin/zsh
$SHLVL: 1
whoami: 'omid'
${HOME}: /Users/omid
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: /usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/Apple/usr/bin
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'zsh 5.8.1 (x86_64-apple-darwin22.0)'
uname -a: 'Darwin 22.2.0 Darwin Kernel Version 22.2.0: Fri Nov 11 02:08:47 PST 2022; root:xnu-8792.61.2~4/RELEASE_X86_64 x86_64'
checksum binary: 'shasum'
OS version: macOS 13.1 22C65
curl: /usr/bin/curl, curl 7.85.0 (x86_64-apple-darwin22.0) libcurl/7.85.0 (SecureTransport) LibreSSL/3.3.6 zlib/1.2.11 nghttp2/1.47.0
wget: not found
sed: /usr/bin/sed
cut: /usr/bin/cut
basename: /usr/bin/basename
rm: /bin/rm
mkdir: /bin/mkdir
xargs: /usr/bin/xargs
git: /usr/bin/git, git version 2.37.1 (Apple Git-137.1)
grep: /usr/bin/grep, grep (BSD grep, GNU compatible) 2.6.0-FreeBSD
awk: /usr/bin/awk, awk version 20200816
nvm current: none
which node: node not found
which iojs: iojs not found
which npm: npm not found
npm config get prefix: nvm:298: command not found: npm
npm root -g: nvm:298: command not found: npm

nvm ls output:

            N/A
mkdir: /Users/omid/.nvm/alias: Permission denied
mkdir: /Users/omid/.nvm/alias: Permission denied
iojs -> N/A (default)
node -> stable (-> N/A) (default)
unstable -> N/A (default)

How did you install nvm? install script in readme.

What steps did you perform?
nvm install node
What happened?
mkdir: /Users/omid/.nvm/alias: Permission denied
mkdir: /Users/omid/.nvm/alias: Permission denied
mkdir: /Users/omid/.nvm/alias: Permission denied
Downloading and installing node v19.3.0...
mkdir: /Users/omid/.nvm/.cache: Permission denied
creating directory /Users/omid/.nvm/.cache/bin/node-v19.3.0-darwin-x64/files failed
Downloading https://nodejs.org/dist/v19.3.0/node-v19.3.0-darwin-x64.tar.xz...
Warning: Failed to open the file
Warning: /Users/omid/.nvm/.cache/bin/node-v19.3.0-darwin-x64/node-v19.3.0-darwi
Warning: n-x64.tar.xz: No such file or directory
curl: (23) Failure writing output to destination

Binary download from https://nodejs.org/dist/v19.3.0/node-v19.3.0-darwin-x64.tar.xz failed, trying source.
grep: /Users/omid/.nvm/.cache/bin/node-v19.3.0-darwin-x64/node-v19.3.0-darwin-x64.tar.xz: No such file or directory
Provided file to checksum does not exist.
Binary download failed, trying source.
Detected that you have 8 CPU core(s)
Running with 7 threads to speed up the build
Clang v3.5+ detected! CC or CXX not specified, will use Clang as C/C++ compiler!
mkdir: /Users/omid/.nvm/.cache: Permission denied
creating directory /Users/omid/.nvm/.cache/src/node-v19.3.0/files failed
Downloading https://nodejs.org/dist/v19.3.0/node-v19.3.0.tar.xz...
Warning: Failed to open the file
Warning: /Users/omid/.nvm/.cache/src/node-v19.3.0/node-v19.3.0.tar.xz: No such
Warning: file or directory
                                                                                0.0%curl: (23) Failure writing output to destination

Binary download from https://nodejs.org/dist/v19.3.0/node-v19.3.0.tar.xz failed, trying source.
grep: /Users/omid/.nvm/.cache/src/node-v19.3.0/node-v19.3.0.tar.xz: No such file or directory
Provided file to checksum does not exist.

What did you expect to happen?
Is there anything in any of your profile files that modifies the PATH? No.

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

I just ran into this issue after a fresh Kubuntu install. Then I remembered I installed curl with snap instead of apt, so I reinstalled it with apt and everything works now.

snap's curl is indeed broken and should not be used; please file a new issue if you're still having trouble


Integration with elasticsearch scala library

Is there any way to use this plugin within a client application with elasticsearch spark scala library?

@Gigitsu this is an elasticsearch plugin,  I am not ware that we can use ES plugins within any client. Can you specify?


fix: update node-fetch version in node 12 and 14 runtimes

<EMAIL_ADDRESS>fixes a security issue.
NOTE: this update will be in the next release, so please check with the Runtime team on Slack before this gets pulled in.

Types of changes

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

Checklist:

[X] I have signed the Adobe Open Source CLA.
[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

fixes deployed, can be pulled in


Improve working with git experience.

We close this for now. We'll open more specific ones as necessary.


refactor(caip): flatten CAIP imports

This imports CAIP utils from the new flattened exports.
⚠ https://github.com/shapeshift/lib/pull/560 should be merged and published first - it contains the flattened exports.
Once that's done, we should run yarn one more time here to update the yarn lockfile.
Closes https://github.com/shapeshift/unchained/issues/396
Partially completes https://github.com/shapeshift/lib/issues/548

unchained client is being moved into lib. these changes will need to be brought over upon introduction there

package officially exists in lib. changes will need to be implemented there.


Add the cargo installation to the build guide for Ubuntu before cargo build

sudo apt install cargo

we mention cargo as a prerequisite though, do you think we need to provide this line of instruction?

@outoftardis Ok, I think this enough. Sorry.


README: small tweaks

Fix a URL & add quotes for style cohesiveness.

I was just looking for something to make OG images easier as a fellow non-graphically inclined person, thanks for putting this up! :D

aww thank you, @lowlydba ! 😊


ehcache artifact without JSR-107 support would simplify apps deployed on app servers w/ a diff. default JSR-107 provider

Without advertising the other JSR-107 provider,
the following exception is thrown:
[2022-06-18T11:36:21.319+0200] [SomeServer 4.1] [SEVERE] [] [com.something.app.someapp.type.web.config.SomeConfigurer] [tid: _ThreadID=15 _Thread     Name=RunLevelControllerThread-1655544943873] [timeMillis:<PHONE_NUMBER>319] [levelValue: 1000] [[
Multiple CachingProviders have been configured when only a single CachingProvider is expected
javax.cache.CacheException: Multiple CachingProviders have been configured when only a single CachingProvider is expected
at javax.cache.Caching$CachingProviderRegistry.getCachingProvider(Caching.java:376)
at javax.cache.Caching$CachingProviderRegistry.getCachingProvider(Caching.java:351)
at javax.cache.Caching.getCachingProvider(Caching.java:142)
...
P.S. Given that setting system property
javax.cache.spi.CachingProvider=com.alernative.cache.AlternativeCachingProvider
hasn't helped,
my current hackish work-around is to build a custom artifact after minimalistic deletion of relevant files:
$ rm org/ehcache/jsr107/EhcacheCachingProvider*
$ rm META-INF/services/javax.cache.spi.CachingProvider
$ rm OSGI-INF/org.ehcache.jsr107.*

P.P.S. My idea has been to use ehcache caches backed by the file-system, which the alternative doesn't seem to support (for free).

If you wish you can use the explicit modules org.ehcache.modules:* rather than the default package that includes 107 support.

Given that we think this an unusual case we're going to say that depending on individual modules is the solution here.

The suggestion seems feasible. I put a comment to try it when i play with that code again.
P.S. I'd appreciate feedback on the following.

Is org.ehcache:ehcache(:3.(9|10)) equivalent to
everything under org.ehcache.modules:*
including org.ehcache.modules:ehcache-107
or some modules are excluded?
Does org.ehcache:ehcache(:3.(9|10))
include everything under
org.ehcache.modules.clustered:*?
P.P.S. My guess is, I'd need to exclude just org.ehcache.modules:ehcache-107, including org.ehcache.modules.clustered:* to start with (further omissions could be considered over time to reduce the size of dependencies)...


Enhance Archive Extractions

The extract service uses 7zip to extract files from archives. In cases of some malformed ZIP archives, this option extracts different files than Windows Explorer and WinRAR. I agree with the 7zip developers that this is not bug in 7zip.
https://perception-point.io/blog/evasive-concatenated-zip-trojan-targets-windows-users/
I expect Assemblyline to identify this as a suspicious Heuristic and extract files from one or more archives appended after the first central directory. The files extracted from archives appended to the initial archive should be correlated in some way to the original file, possibly through an "appended archive" that is extracted and resubmitted without the header.
flowchart TD
    ma[Submitted ZIP Archive
    ZIP 1 + ZIP 2 + ZIP 3]
    ma --extract--> aa1(ZIP 1
    file10
    file11)
    ma --subfile--> a2[ZIP 2]
    a2 --extract--> aa2(ZIP 2
    file20
    file21)
    ma --subfile--> a3[ZIP 3]
    a3 --subfile--> aa3(ZIP 3
    file30
    file31)

Users can use YARA to identify file entries after the central directory, but there is not an easy way to resubmit a file without the first ZIP archive. The user can manually compare the result from extract which shows the first archive's metadata, and the result from characterize which shows the last archive's metadata, but that does not analyze the second archive's contents.
This is the example ZIP archive (base64 encoded) as described in the blog post.
UEsDBAoAAAAAAL2Aa1lvuF2AGQAAABkAAAAJABwAZmlyc3QudHh0VVQJAANmKzJnZisyZ3V4CwAB
BLgj8wsEQQ/zC3RoaXMgaXMgYSBoYXJtbGVzcyBmaWxlIQpQSwECHgMKAAAAAAC9gGtZb7hdgBkA
AAAZAAAACQAYAAAAAAABAAAAgIEAAAAAZmlyc3QudHh0VVQFAANmKzJndXgLAAEEuCPzCwRBD/ML
UEsFBgAAAAABAAEATwAAAFwAAAAAAFBLAwQUAAAACADDgGtZ05tNZBwAAAAdAAAACgAcAHNlY29u
ZC50eHRVVAkAA20rMmdtKzJndXgLAAEEuCPzCwRBD/MLC8nILFYAokSFstSiSoXi5EQgmZuYU55Y
lMoFAFBLAQIeAxQAAAAIAMOAa1nTm01kHAAAAB0AAAAKABgAAAAAAAEAAACAgQAAAABzZWNvbmQu
dHh0VVQFAANtKzJndXgLAAEEuCPzCwRBD/MLUEsFBgAAAAABAAEAUAAAAGAAAAAAAA==

I built two new versions of Extract, v4.5.0.stable49 and v4.5.0.dev51, which should give you the desired behaviour with zip files that contain appended data at the end. Your example file now gets both text files extracted. Thank you for the issue!


Mostly the HelpRequests contained in HelpLists don't contain the requester

I noticed a difference in behavior for the responses created by the "Help Lists" endpoint:

GET /help-lists: all help requests (which are nested in the "HelpList" responses contain the correct requester object
GET /help-lists/{helpListId}: help requests are lacking the requestor entirely
PUT /help-lists/{helpListId}/help-request/{helpRequestId}: requestor is only available for the request which was added through the call.. all others are lacking the requestor.

From a "users" perspective of the API it would be cool if all the call return fully initialized responses to avoid nesting calls..

Thanks, I think it is solved with commit https://github.com/NexdApp/nexd-backend/commit/2aa44bb03399c5a1fe79073fd6e50b6d864df627


How can i display the reminder message without user input i.e without affecting normal conversation with chatbot?

I want to create a reminder in chatbot which will check every minute if the required time is arrived or not. If yes, then display it to user but user should be able to do conversation until that time arrives. Where should i call such a function? Is there any other idea for creating a reminder?
Versions are mentioned below:
node v 6.9.2
npm v 3.10.9
Ubuntu 16.04

I'd recommend adding a message prop in the reply with the required time to send the reminder, and then handling it in the client (express/telnet etc) i.e. setTimeout(/* function to call bot again here */, reply.props.timeout). Note that there may be weird issues with modifying conversation state doing it this way.
Or if you know ahead of time what the reminder message is going to be, then setTimeout(/* function to send reply.props.reminderMessage to user */, reply.props.timeout) may be more useful.


fix: don't refresh whole System for memory

Previously the entire System was refreshed twice for no reason.

Btw, is it intended that disk usage is displayed as Free / Total instead of the more common Used / Total?

thank you for the pr @Luk-ESC . Yes it was intentional to show Free / Total. If we get enough votes for Used / Total then I'll make the changes.

plz make the changes so that the workflows pass. Then I'll merge the pr.


Not working on Android

I'm trying to use the plugin on an Android device (Nativescript 8)
The view appears, I can add an overlay - but there's no preview. All I see is a black screen.
On iOS, the plugin is working (except for the built in camera-toggle - this produces a crash)

Nevermind... I didn't request the camera permissions. That stopped it from working


[WIP] Limit doubling time to smoothed line

We can solve use a decorator to set the Y-limit.
Also, I created a new file plotting_helpers.py, because oscovida.py is embarrassingly big already.
Closes #123

Strangely enough, tests do well on my local machine :man_shrugging:

Does this need further review or can we merge it?

Yes, needs further work: failure in https://travis-ci.com/github/oscovida/oscovida/jobs/404330728#L600

Yes, exactly. There is still the case with twinx, that's why I put [WIP] status.

Codecov Report

Merging #134 into master will increase coverage by 0.37%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
+ Coverage   79.17%   79.54%   +0.37%
==========================================
  Files           4        5       +1
  Lines         869      885      +16
==========================================
+ Hits          688      704      +16
  Misses        181      181

Impacted Files
Coverage Δ

oscovida/oscovida.py
76.85% <100.00%> (-0.59%)
:arrow_down:

oscovida/plotting_helpers.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


I can't Search by name or username.!

It could be cool if we can search, and find names fastly.!

Should be fixed by #115 and partially addressed via lauripiispanen/github-top#221

@mariovarela99 my fork has this implemented via badges (plus the above PR was merged in it)

@ashkulz Alright, I am gonna see it now.


just import README into docs/index.rst rather than duplicating text

Codecov Report

Merging #22 (2cb8e69) into main (b6e0c54) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   48.75%   48.75%
=======================================
  Files         115      115
  Lines       25507    25507
=======================================
  Hits        12435    12435
  Misses      13072    13072

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 b6e0c54...2cb8e69. Read the comment docs.


Update Docker installation instructions

The docs for installing Docker are a bit outdated. kubernetes/kubernetes#82326 (comment)
Sorry, I messed up the branch in the previous PR. Please check again.
https://github.com/kubernetes/website/pull/17347
/assign @neolit123
/assign @daminisatya

/lgtm
/assign @jaredbhatti


Doesn't ModelControlAPI manage unloaded model's GPU memory?

This is just a question.
I've tried to use ModelControlAPI and faced an issue that one model loading failed even after unloading another model that requires huge GPU memory.
For example, the GPU has 12GiB memory, model1 requires ~9GiB to run an inference and model2 requires ~4GiB. In this case, after loading model1, trying to load model2 failed due to OutOfMemory. So, I expected that I can load model2 by unloading model1. However, currently failed. Even if unloading model1, GPU memory wasn't released.
Is that an expected behavior? If so, I hope these details are described in the document.

What type of model are you using. As we note here: https://docs.nvidia.com/deeplearning/sdk/tensorrt-inference-server-guide/docs/model_management.html#model-control-mode-explicit, this mode is experimental because some frameworks have trouble loading/unloading models. We are going to do some more characterization of which frameworks can handle load/unload well and which can't. Right now our suspicion is that both TensorFlow and PyTorch do not manage memory well when you repeatedly load/unload models.

Thank you for the comment.
Currently, ResNet50 and etc are targets, and right now these models are for evaluation. I know this feature is experimental so please consider more flexible memory management by ModelControlAPI.

The model itself (like resnet50) is not the problem. It is the framework backend. We will document our findings once we do more thorough characterization of the frameworks.

@deadeyegoodwin however if i convert tensorflow to graphdef, is it still using tensorflow backend?

Yes. Saved-model and graphdef are two model formats for Tensorflow. Saved-model has a graphdef embedded within it with additional metadata. Once loaded both model formats are handled the same by TF.

@deadeyegoodwin  if i convert the graphdef to TensorRT, but not all the layers can be converted, so it ends up being a mix between tensorflow and TensorRT, is there still going to be a problem with the loading/unloading?
Also, do you know if the problem of the loading/unloading will lead to a memory leak in which memory cant even be recovered? And the only solution is to restart the triton server?
What i mean is this, if i can unload a 40% of a tensorflow model, when when i load it back i only need to load the remaining 60%. As long as the memory is not leaking and becoming unusable i think its still ok. But if i repeatedly load and unload for eg. m1-m2-m1-m2-m1-m2 , and after 100 tries it OOMs, then this suggests some type of memory leak with is unsafe

If you use an offline tool on a TF model to add TensorRT optimization, the resulting model is still a TF model and is managed by the TF framework like any other TF model. We don't know of any memory leak in TF (or other framework) on repeated load/unload and we even have a CI test for it. But as I mentioned in the other ticket TF will not return its memory to the system, even if you unload all TF models. There is a commandline flag to limit how much GPU memory TF can have which can be useful if you want to make sure TF doesn't use all GPU memory.


Build with mtaext - update cloud-mta version for new validations + more

Handle errors in merge (which now returns error)
Improved error message for unmarshaling mtaext files (which now perform
more checks on the input)
Validate mtaext files in "validate" command and write which file the
warnings are for
Check schema version matches between mta.yaml and extensions

Checklist

[X] Code compiles correctly
[X] Relevant tests were added (unit / contract / integration)
[X] Relevant logs were added
[X] Formatting and linting run locally successfully
[X] All tests pass
[ ] UA review
[ ] Design is documented
[ ] Extended the README / documentation, if necessary
[ ] Open source is approved

Approved and reviewed by Alla (only new change was updating the cloud-mta and yaml.v3 versions)


Bug: Import error when using tsconfig module: Node16 or newer

I have a Node.js project with module format commonjs, I noticed the following errors when import LangChain packages:
import { ChatOpenAI } from "@langchain/openai";
import { ChatPromptTemplate } from "@langchain/core/prompts";
import { StringOutputParser } from "@langchain/core/output_parsers";

/*
The current file is a CommonJS module whose imports will produce 'require' calls;
however, the referenced file is an ECMAScript module and cannot be imported with 'require'.
Consider writing a dynamic 'import("@langchain/openai")' call instead.
To convert this file to an ECMAScript module, change its file extension to '.mts',
or add the field `"type": "module"` to package.json
*/

This issue seems to be related to TypeScripts module resolution. If I use "module": "Node16" or "module": "NodeNext", the issue appears. If I switch to the very old "module": "CommonJS", the issue disappears.
I have seen similar issues here with the recommendation to use require("@langchain/openai") instead of import.
However, I really don't think that is the real issue because LangChain includes both ESM and CJS packages and it should not be any problem to import CJS in TypeScript via import.
Here is a minimal reproducible example:
https://github.com/zirkelc/langchain-node16-issue
Changing the "module" in the tsconfig.json will toggle the import error:
{
  "compilerOptions": {
    // "module": "CommonJS", // => works
    // "module": "NodeNext", // => doesn't work
    // "module": "Node16", // => doesn't work
    "target": "es2022",
    "strict": true,
    "esModuleInterop": true,
    "skipLibCheck": true,
    "forceConsistentCasingInFileNames": true,
  },
  "include": [
    "index.ts",
  ],
  "exclude": [
    "node_modules/**/*",
  ],
}

Thank you for the report. Will dig in as soon as I can.

Hi @jacoblee93,
I encountered the same issue for NanoID awhile ago: https://github.com/ai/nanoid/issues/422
Gladly, they were able to fix it by: https://github.com/ai/nanoid/issues/422#issuecomment-1777877914
TLDR: the package.json exports need to reference .js and .d.ts files for both ESM and CJ. That means, exporting .js and .d.ts for ESM, and .cjs and .d.cts for CJS. See TS reference
I was able to verify locally by changing the package.json of @langchain/openai and adding a index.d.cts type file:
Before:
  "exports": {
    ".": {
      "types": "./index.d.ts",
      "import": "./index.js",
      "require": "./index.cjs"
    },
    "./package.json": "./package.json"
  },

After:
  "exports": {
    ".": {
      "require": {
        "types": "./index.d.cts",
        "default": "./index.cjs"
      },
      "import": {
        "types": "./index.d.ts",
        "default": "./index.js"
      }
    },
    "./package.json": "./package.json"
  },

The new index.d.cts is a copy of the existing index.d.ts:
export * from './dist/index.js'

I hope this helps you. Let me know if I can do anything else!

That's extremely helpful! Thanks so much.
I think with all of the integration packages we're splitting out we should get some kind of scaffolding lib in place along with this to avoid excessive copy paste...

I'm facing the same issue, and unfortunately I have to stick to NodeNext for module resolution. Any work around ?

@thibautvdu it's a TS-only error, so you can @ts-ignore these errors as a workaround.

This should be fixed in the latest versions of langchain, @langchain/core, @langchain/community, and @langchain/openai! Please reopen if it persists.

Hi @jacoblee93 thanks a lot for the update ;
The issue is solved for the aforementioned packages, but is still present with @langchain/misralai

Will push a new version of all integration packages today and update you 👍

Hi @jacoblee93
this issue is mainly fixed, but I noticed the @langchain/openai still has an incomplete exports declaration:
https://github.com/langchain-ai/langchainjs/blob/c80d51835832968b4341445084413511a0c079f8/libs/langchain-openai/package.json#L85-L94
Will you going to change this package too?

 Will patch today.

Hey, this slipped my mind. Fixing now.

Wait no @zirkelc I think that is the correct and expected output at this point? We didn't go with exactly what was suggested above.
If it's still causing problems, let me know and we can reopen.


fix: update to eye v4.2.1

fix: update to eye v4.2.1

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

npm package (@latest dist-tag)
GitHub release
v5.2.1

Your semantic-release bot :package::rocket:


Hash Sum mismatch issue caused Unable to fetch some archives

I try to vagrant up with thif config on my Windows PC.
And run into such kind of problem:

msg: 'apt-get install 'autoconf' 'bison' 'build-essential' 'libssl-dev' 'libyaml-dev' 'libreadline6-dev' 'zlib1g-dev' 'libncurses5-dev' 'nodejs' 'python-pip' 'python-dev' 'htop' 'git'' failed: E: Failed to fetch http://archive.ubuntu.com/ubuntu/pool/main/a/automake-1.14/automake_1.14.1-2ubuntu1_all.deb  Hash Sum mismatch

Is this due to my network issue? I tried this in my school serveral times and failed.
This is my console log.

I guess such things can happen because of network glitches. Also try to disable firewall (if you have one). Since some of packages were downloaded successfully (as I see in your console log), I would try to run the following:
sudo rm -rf /var/lib/apt/lists/*
sudo apt-get update

And then re-run vagrant provision.

Yeah. I changed the sources.list mirror and tried again. It worked.


E1-S1-T1: Install tenserflow and toxicity model

Epic : https://github.com/Flossed/obscenity/issues/1
Story : https://github.com/Flossed/obscenity/issues/2
Install both the tensorflow framework, and the model for toxicity:
Commands:
npm i @tensorflow/tfjs
npm install @tensorflow-models/toxicity
This can be tested by requiring the tensorflow framework :
const tf                               = require( '@tensorflow/tfjs' );
And checking with a console.log( ) the version:
console.log( tf.version.tfjs);
example output:
4.21.0

Committed under commitID : 79072ecb376093919a88ddf6df212ba8d660ce30


Group the update package in the depandabot

Follow how https://github.com/karaoke-dev/osu-framework-microphone/pull/345 did.

Technically, osu.framework and osu.resource might not has update package at the same time, but should be better to upgrade together if has release at the same time.


Add Administrator Bot type which only admins can create or own.

This pr is an updated version of WIP pr #12589(which was around 2 years old and had conflicts) , in this I have updated the changes in the code to allow add an administrator bot by admin user/realm only.
Changes that are done from the previous pr are:

Updating backend tests according to current code base to use functions like do_change_user_role for updating user role.
Updating tests to check that last realm owner does not get deleted even if an admin bot is present(to justify the idea of always having a human owner).

This pr implements following parts of what was asked in issue #12424

Added UI for giving option to adminstrators/realm owners to add a administrator bot.
Added backend for a Administrator Bot type to support adding of admin bot.
Checks to ensures realm owner cannot delete itself even if an admin bot is present.
Added cascade demotion of user's admin bots if that admin user is demoted(this was not mentioned in the issue #12424 but was suggested by @showell during a discussion for this issue on CZO).
Added checks to block admin bots from accessing certain endpoints (as discussed here).

It is marked WIP as various works are left on it as asked in issue #12424  and this comment
Testing plan: tested manually by adding bots from frontend as different user. Backend changes have been tested using tools/test-backend.
GIFs or screenshots:

For admin user iago:

For non admins:

Cascade demotion of admin bot:
https://user-images.githubusercontent.com/63504956/105379670-15936200-5c33-11eb-929d-feac26184e52.mp4

Thanks for working on this @m-e-l-u-h-a-n!  I posted a few quick comments.

Thanks for working on this @m-e-l-u-h-a-n!  I posted a few quick comments.

@timabbott this pr has been updated as per your suggestions. Can you please give it another review.
I also left a message on developemnt help > Admin bots, a couple of days back which I think I should also put here for anyone going through it:
Also as per the current status of the work following work is left on it:

Realm export(currently allowed for all admin(so for admin bots too) I think we should block it at least for admin bots(as per previous suggestions from @_Rishi Gupta|131  over it).
Edit which domains are allowed to login(due to a doubt I expressed, here it would be great to get some guidance so this can be done quickly).
Update api docs to mention that this bot exists, and update docs on bots to explain their functionality.(I think I should do it after I get some reviews on this pr so that it is not required to be changed frequently, as we proceed with review of this pr).

@timabbott this pr has been updated as per your suggestions. Can you please give it another review.
I also left a message on developemnt help > Admin bots, a couple of days back which I think I should also put here for anyone going through it:
Also as per the current status of the work following work is left on it:

Realm export(currently allowed for all admin(so for admin bots too) I think we should block it at least for admin bots(as per previous suggestions from @_Rishi Gupta|131  over it).
Edit which domains are allowed to login(due to a doubt I expressed, here it would be great to get some guidance so this can be done quickly).
Update api docs to mention that this bot exists, and update docs on bots to explain their functionality.(I think I should do it after I get some reviews on this pr so that it is not required to be changed frequently, as we proceed with review of this pr).

@zulipbot remove "has conflicts"

@timabbott this is ready for review.

As it was a big change so I tried spliting the commits, due to this few commits in between are failing lint checks in between. I'll fix that along with other updates.

@timabbott updated the pr with latest changes.
Can you please have a look at it. Also status of work on it, is mentioned in https://github.com/zulip/zulip/pull/17038#issuecomment-770185050.

Closing, since I think we've ended up deciding to go with the direction of using the normal role field for bot users, in part based on seeing the complexity of implementing this approach.
Thanks for all your work on this @m-e-l-u-h-a-n!


User service integration into User pages

Closes #284
I've integrated the User service functions into their appropriate pages. Right now, the add user works, we will need to wait for the rest of the endpoints to confirm the integrations works. Wanted to get it up for review though for any possible feedback. I am confident that there won't be much more to change once the endpoints are done. I will also post a demo video once the endpoints are done.

[x] wait for rest of user endpoints

Here's that demo
https://user-images.githubusercontent.com/58532471/161454200-01ba8183-f050-4d24-b443-a629047e4d56.mp4


"select all" doesn't work.

Steps to Reproduce

write few words at text field
double tap the word
tap "select all"

Expected results:
All words would be selected
Actual results:
no words is selected

[√] Flutter (Channel beta, 1.21.0-9.1.pre, on Microsoft Windows [Version 10.0.18363.1016], locale ja-JP)
    • Flutter version 1.21.0-9.1.pre at C:\src\flutter
    • Framework revision be9bc8cb39 (13 days ago), 2020-08-18 09:46:08 -0700
    • Engine revision 267070c17a
    • Dart version 2.10.0 (build 2.10.0-7.2.beta)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    • Android SDK at C:\Users\hatadev\AppData\Local\Android\sdk
    • Platform android-29, build-tools 29.0.1
    • Java binary at: C:\Program Files\Android\Android Studio1\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.6)
    • Android Studio at C:\Program Files\Android\Android Studio1
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] VS Code (version 1.48.2)
    • VS Code at C:\Users\hatadev\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.13.2

[√] Connected device (3 available)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 84.0.4147.135
    • Edge (web)       • edge       • web-javascript • Microsoft Edge 85.0.564.41

• No issues found!

Hi @hatano0x06,

When we double tap the word, we see cut, copy, paste toolbar.
When we long press, then we see paste, select all and upon selecting select all, I am able to see the entire text selected properly.
When we double tap at end of the words, and select select all, then the entire text is selceted properly.

I am on latest Stable (1.20.2) and tested on iOS simulator.

I check at Stable (1.20.2) and it was ok for me too
thank you for your check
does it fixed at next beta version?

@hatano0x06 I am not sure about that. Mean time you may switch to Stable.
I am going ahead and closing this as non-issue.
Thanks.


refactor(utils)!: make snark-artifacts utility functions dynamic

Description

WIP
Related Issue(s)

Closes #272
Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have run yarn style without getting any errors
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@sripwoud I have deliberately minimized the code, but it makes sense to re-integrate parts of your code. Let me reply your comments!


[Access Request] page 973 "Time Sheet Card"

Could you please change the access scope of the following variables to protected?
var
    TimeSheetMgt: Codeunit "Time Sheet Management";
    TimeSheetApprovalMgt: Codeunit "Time Sheet Approval Management";
    RefActionType: Option Submit,ReopenSubmitted,Approve,ReopenApproved,Reject;
//+Change
protected var
    SubmitEnabled: Boolean;
    ReopenSubmittedEnabled: Boolean;
    ReopenApprovedEnabled: Boolean;
    ApproveEnabled: Boolean;
    ManagerTimeSheet: Boolean;
//-Change

Was fixed in version 22.


Address derivation between Libra and Diem codebases is different. (v5)

Address derivation between Libra and Diem Code bases
This is a problem since all the mnemonics/key will generate different account addresses than expected.
This came to our attention with the swarm 5th account problem (the first generated account) :
This is the expected accounts (v4):
[language/tools/vm-genesis/src/lib.rs:405] owner_address = 4C613C2F4B1E67CA8D98A542EE3F59F5
[language/tools/vm-genesis/src/lib.rs:405] owner_address = 88E74DFED34420F2AD8032148280A84B
[language/tools/vm-genesis/src/lib.rs:405] owner_address = E660402D586AD220ED9BEFF47D662D54
[language/tools/vm-genesis/src/lib.rs:405] owner_address = 9E6BB3A75E9618FBA057E86E69338C94
[language/tools/vm-genesis/src/lib.rs:405] owner_address = 5709AFA1D0FBB40B171AA03D457DB72A

This is what we have, the 5th address is different:
OK create_and_initialize_main_accounts ===============
Initializing with env: test
0 ======== Create Owner Accounts
[language/tools/vm-genesis/src/lib.rs:413] owner_address = 4C613C2F4B1E67CA8D98A542EE3F59F5
[language/tools/vm-genesis/src/lib.rs:413] owner_address = 88E74DFED34420F2AD8032148280A84B
[language/tools/vm-genesis/src/lib.rs:413] owner_address = E660402D586AD220ED9BEFF47D662D54
[language/tools/vm-genesis/src/lib.rs:413] owner_address = 9E6BB3A75E9618FBA057E86E69338C94
[language/tools/vm-genesis/src/lib.rs:413] owner_address = 4E1F81F77024B56DBB853CE7ED8E1C7E

https://github.com/OLSF/libra/blob/v5/config/management/genesis/src/storage_helper.rs#L276
Ed25519PrivateKey::generate(&mut rng) in this line  .import_private_key(OWNER_KEY, Ed25519PrivateKey::generate(&mut rng)) is different, seems like this is the reason we have different account addresses (and public key shown in previous comment). And the root cause might be different versions of rand library as seen below:
// v5
--- namespace: "4_owner"
--- seed: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 5, 0, 0, 0, 0, 0, 0, 0]
--- Ed25519PrivateKey(SecretKey: [254, 70, 32, 88, 91, 108, 253, 244, 133, 213, 152, 34, 153, 25, 21, 165, 146, 0, 105, 153, 47, 63, 34, 123, 14, 217, 96, 198, 0, 176, 168, 235])
/home/gsimsek/.cargo/registry/src/github.com-1ecc6299db9ec823/rand-0.8.3/src/lib.rs

// v4
--- namespace: "4_owner"
--- seed: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 5, 0, 0, 0, 0, 0, 0, 0]
--- Ed25519PrivateKey(SecretKey: [14, 17, 228, 135, 90, 218, 215, 235, 219, 9, 64, 116, 201, 243, 14, 48, 232, 177, 15, 67, 123, 13, 128, 82, 123, 213, 200, 223, 30, 232, 252, 43])
/home/gsimsek/.cargo/registry/src/github.com-1ecc6299db9ec823/rand-0.7.3/src/lib.rs

I don't think this is an issue. I'm getting the same addresses deterministically. I added a test with commit 5aff45d3173a1d9a2e664c4e98d56a364f286a99
check_address_derivation_does_not_change()


Docker Build: Python installs not working?

I am trying to use the docker setup for this repo. The strange behavior I am noticing is that the tensorrt_llm wheel and its dependencies like transformers appear to install cleanly while the container is building, but they do not seem to be there when I run the container.
Steps to reproduce:
make -C docker release_build CUDA_ARCHS="86-real"
In the output I see:
#28 11.98 Successfully installed accelerate-0.20.3 build-1.0.3 colored-2.2.3 cuda-python-12.2.0 diffusers-0.15.0 huggingface-hub-0.18.0 lark-1.1.8 pyproject_hooks-1.0.0 safetensors-0.4.0 sentencepiece-0.1.99 tensorrt-llm-0.5.0 tokenizers-0.13.3 transformers-4.33.1
But when I start the container with make -C docker run and execute this  python -c "import transformers" I get the following error:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'transformers'

If I run pip freeze, these packages are missing from the output:
...
tblib @ file:///rapids/tblib-2.0.0-py3-none-any.whl#sha256=9100bfa016b047d5b980d66e7efed952fbd20bd85b56110aaf473cb97d18709a
tensorboard==2.9.0
tensorboard-data-server==0.6.1
tensorboard-plugin-wit==1.8.1
tensorrt @ file:///usr/local/tensorrt/python/tensorrt-9.1.0.post12.dev4-cp310-none-linux_x86_64.whl#sha256=efdcd7d365732499f7c0fa3103138fd769f90bb02cf0ee9ed31b7da178897f16
terminado==0.17.1
thinc==8.1.10
threadpoolctl==3.2.0
thriftpy2 @ file:///rapids/thriftpy2-0.4.16-cp310-cp310-linux_x86_64.whl#sha256=879265d504b99a5acd5ee479d58ac4e82d8debb151037b4f488e3d01f092e00b
tinycss2==1.2.1
toml==0.10.2
tomli==2.0.1
toolz @ file:///rapids/toolz-0.12.0-py3-none-any.whl#sha256=2059bd4148deb1884bb0eb770a3cde70e7f954cfbbdc2285f1f2de01fd21eb6f
torch==2.1.0a0+29c30b1
torch-tensorrt @ file:///opt/pytorch/torch_tensorrt/py/dist/torch_tensorrt-2.0.0.dev0-cp310-cp310-linux_x86_64.whl#sha256=1220e0e3b3d67f2536abc547ecdfa1f0c859dea438e4333e4f3daf1672787757
torchdata @ file:///opt/pytorch/data
torchtext @ file:///opt/pytorch/text
torchvision @ file:///opt/pytorch/vision
tornado==6.3.2
tqdm==4.65.0
traitlets==5.9.0
transformer-engine @ git+https://github.com/NVIDIA/TransformerEngine.git@3f01b4f812e0e501257278ec269499ea02b2d4f3
treelite @ file:///rapids/treelite-3.2.0-cp310-cp310-linux_x86_64.whl#sha256=e7b18d6b4e678bac634c620f7ec2075bcf342c58dcd03c827d38f64a82f22b7d
treelite-runtime @ file:///rapids/treelite_runtime-3.2.0-cp310-cp310-linux_x86_64.whl#sha256=5533c60f8a8f5dd4e3127dbb9f16147db7a77facc2c44361e2bd2c25467ed9b1
triton @ file:///tmp/dist/triton-2.1.0%2B440fd1b-cp310-cp310-linux_x86_64.whl#sha256=3f87412b539d21820e40a7a26791fe50d317a8883eec59958c698e267dd6d3f0
typer==0.9.0
...

Now everything works if I rebuild the wheel file and install it from within the container. But I shouldn't have to do this, why are the python packages disappearing after the docker build? Can anyone reproduce this behavior following my steps?

Oh, my bad, thank you!


fix!: host-metrics system.cpu.utilization calculation fix

Which problem is this PR solving?
this PR is fixing the calculation of system.cpu.utilization metrics which is currently returning values that are out of the range [0,1] specified in semantic conventions. See #1718
Short description of the changes
the calculation now is made based on the previous values returned from the os.cpus NodeJS API and the time it was called. Also Date.now static function is used to avoid unnecessary object creation (date objects) on each measurement.

@legendecas @dyladan could you give another look please?

Thank you for working on this.

Thanks @legendecas :)
I'll prepare another PR for process.cpu.* metrics once this one is merged.


dot_graph writes empty png

Following Jake VDP's tutorial blog post, I am attempting to plot a dask graph. However, dot_graph plots an empty png. Here is the example. The resulting png file is zero bytes in size. As you can see, there is no exception raised.
Using dask 0.7.5 built from master on OS X 10.11, Anaconda Python 3.5. I've installed graphviz and pydot2 from pip.

@fonnesbeck can you verify that your graphviz system library works correctly?  E.g. can you try writing out a .dot file and then using the dot command line utility directly?
>>> # dot_graph(b2_min.dask, filename='b2_min.dot')
>>> b2_min.visualize(filename='b2_min.dot')  # perhaps nicer syntax

$ dot -Tpng b2_min.dot -o b2_min.png

Or, an alternative test would be to install graphviz with a tool like brew.

I have GV installed via brew:
$ brew install graphviz
Warning: graphviz-2.38.0 already installed

I mention the tests above because, when others ran into problems like this (cc @Carreau) it was because the system graphviz library was not compiled with libpng, and so the dot application failed on png files.  Perhaps you're running into a different problem though.  My intention with the questions above was to determine if this was the same issue.
You could also try visualizing with svg, which, if it works, would corroborate this theory.
import dask.array as da
da.ones(5, chunks=2).visualize(filename='foo.svg')

When I did export to a .dot file, I had the following in the terminal:
dyld: Library not loaded: /usr/lib/libltdl.7.dylib
  Referenced from: /Users/fonnescj/anaconda3/bin/dot
  Reason: image not found

Your visualize example fails with the following:
---------------------------------------------------------------------------
ExpatError                                Traceback (most recent call last)
<ipython-input-9-01f98d3386e3> in <module>()
      1 import dask.array as da
----> 2 da.ones(5, chunks=2).visualize(filename='foo.svg')

/Users/fonnescj/anaconda3/lib/python3.5/site-packages/dask/base.py in visualize(self, filename, format, optimize_graph, **kwargs)
     25                   **kwargs):
     26         return visualize(self, filename=filename, format=format,
---> 27                          optimize_graph=optimize_graph, **kwargs)
     28
     29     def _visualize(self, filename='mydask', format=None, optimize_graph=False):

/Users/fonnescj/anaconda3/lib/python3.5/site-packages/dask/base.py in visualize(*args, **kwargs)
    125     dsk = merge(dsks)
    126
--> 127     return dot_graph(dsk, filename=filename, **kwargs)
    128
    129

/Users/fonnescj/anaconda3/lib/python3.5/site-packages/dask/dot.py in dot_graph(dsk, filename, format, **kwargs)
    218         f.write(data)
    219
--> 220     return _get_display_cls(format)(filename=full_filename)

/Users/fonnescj/anaconda3/lib/python3.5/site-packages/IPython/core/display.py in __init__(self, data, url, filename)
    384         self.filename = None if filename is None else unicode_type(filename)
    385
--> 386         self.reload()
    387         self._check_data()
    388

/Users/fonnescj/anaconda3/lib/python3.5/site-packages/IPython/core/display.py in reload(self)
    403         if self.filename is not None:
    404             with open(self.filename, self._read_flags) as f:
--> 405                 self.data = f.read()
    406         elif self.url is not None:
    407             try:

/Users/fonnescj/anaconda3/lib/python3.5/site-packages/IPython/core/display.py in data(self, svg)
    488         from xml.dom import minidom
    489         svg = cast_bytes_py2(svg)
--> 490         x = minidom.parseString(svg)
    491         # get svg tag (should be 1)
    492         found_svg = x.getElementsByTagName('svg')

/Users/fonnescj/anaconda3/lib/python3.5/xml/dom/minidom.py in parseString(string, parser)
   1966     if parser is None:
   1967         from xml.dom import expatbuilder
-> 1968         return expatbuilder.parseString(string)
   1969     else:
   1970         from xml.dom import pulldom

/Users/fonnescj/anaconda3/lib/python3.5/xml/dom/expatbuilder.py in parseString(string, namespaces)
    923     else:
    924         builder = ExpatBuilder()
--> 925     return builder.parseString(string)
    926
    927

/Users/fonnescj/anaconda3/lib/python3.5/xml/dom/expatbuilder.py in parseString(self, string)
    221         parser = self.getParser()
    222         try:
--> 223             parser.Parse(string, True)
    224             self._setup_subset(string)
    225         except ParseEscape:

ExpatError: no element found: line 1, column 0

I suspect that the graphviz executables in anaconda are broken (note that the python graphviz library is probably fine.)  I recommend uninstalling it and going with what brew would install.  Will raise an issue with Anaconda folk.
conda remove graphviz

Yup, that is the issue. Removing conda's graphviz fixed all the examples.

OK, sorry for the runaround.  If you have the time, a quick mention from you in the anaconda-issue linked above might help to get this issue resolved upstream.

Great, thanks for troubleshooting.

We should probably raise an informative error message if this occurs.

I am having this same issue. Installed on Windows 10 using conda-forge for dask (2.12.0) and python-graphviz (0.13.2). Installing python-graphviz also grabbed graphviz (2.38.0).
I was able to resolve this by uninstalling the conda python-graphviz package and using pip install graphviz instead.
Reporting here because I am not receiving a useful error message, as should be the case according to #1027.
import dask.array as da
da.ones(5, chunks=2).visualize(filename='foo.svg')

---------------------------------------------------------------------------
ExpatError                                Traceback (most recent call last)
<ipython-input-49-ceaf49eae531> in <module>
      1 import dask.array as da
----> 2 da.ones(5, chunks=2).visualize(filename='foo.svg')

C:\Miniconda\envs\pyFIRS\lib\site-packages\dask\base.py in visualize(self, filename, format, optimize_graph, **kwargs)
     90         https://docs.dask.org/en/latest/optimize.html
     91         """
---> 92         return visualize(
     93             self,
     94             filename=filename,

C:\Miniconda\envs\pyFIRS\lib\site-packages\dask\base.py in visualize(*args, **kwargs)
    529         raise NotImplementedError("Unknown value color=%s" % color)
    530
--> 531     return dot_graph(dsk, filename=filename, **kwargs)
    532
    533

C:\Miniconda\envs\pyFIRS\lib\site-packages\dask\dot.py in dot_graph(dsk, filename, format, **kwargs)
    253     """
    254     g = to_graphviz(dsk, **kwargs)
--> 255     return graphviz_to_file(g, filename, format)
    256
    257

C:\Miniconda\envs\pyFIRS\lib\site-packages\dask\dot.py in graphviz_to_file(g, filename, format)
    284         f.write(data)
    285
--> 286     return display_cls(filename=full_filename)

C:\Miniconda\envs\pyFIRS\lib\site-packages\IPython\core\display.py in __init__(self, data, url, filename, metadata)
    625             self.metadata = {}
    626
--> 627         self.reload()
    628         self._check_data()
    629

C:\Miniconda\envs\pyFIRS\lib\site-packages\IPython\core\display.py in reload(self)
    651         if self.filename is not None:
    652             with open(self.filename, self._read_flags) as f:
--> 653                 self.data = f.read()
    654         elif self.url is not None:
    655             try:

C:\Miniconda\envs\pyFIRS\lib\site-packages\IPython\core\display.py in data(self, svg)
    754         # parse into dom object
    755         from xml.dom import minidom
--> 756         x = minidom.parseString(svg)
    757         # get svg tag (should be 1)
    758         found_svg = x.getElementsByTagName('svg')

C:\Miniconda\envs\pyFIRS\lib\xml\dom\minidom.py in parseString(string, parser)
   1967     if parser is None:
   1968         from xml.dom import expatbuilder
-> 1969         return expatbuilder.parseString(string)
   1970     else:
   1971         from xml.dom import pulldom

C:\Miniconda\envs\pyFIRS\lib\xml\dom\expatbuilder.py in parseString(string, namespaces)
    923     else:
    924         builder = ExpatBuilder()
--> 925     return builder.parseString(string)
    926
    927

C:\Miniconda\envs\pyFIRS\lib\xml\dom\expatbuilder.py in parseString(self, string)
    221         parser = self.getParser()
    222         try:
--> 223             parser.Parse(string, True)
    224             self._setup_subset(string)
    225         except ParseEscape:

ExpatError: not well-formed (invalid token): line 1, column 0

As this issue is several years old I would suggest opening a new issue


feat: HTML Render option

Description
The switch option for the text widget render by HTML.
Fixes #3682
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: feat/render-as-html-option

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
app/client/src/components/designSystems/blueprint/TextComponent.tsx
42.42 (-1.33)
10 (0)
9.09 (0)
40.63 (-1.31)

@Tooluloope @somangshu Tested this feature and it is working as expected


Logging, asserts and PPP edits

Loggers and asserts for cheatsheet functionalities
Added UI description for my PPP (left diagrams to finish up)

Looks good!


Grid not drawn completely

When opening a job or transformation, the grid is shown in the visible area, also a bit when scrolling down, but once scrolling further down, it disappears:

Mozilla, Fedora 27

Thank you for reporting a bug, but can you tell me the version of webSpoon you were using?
I guess you were using <IP_ADDRESS> or before when you had this issue, right?
As you may have noticed, the drawing grid has changed at <IP_ADDRESS>, where the grid is drawn only when you enable it and you are dragging a step/job entry.
So yes, it was a bug up until <IP_ADDRESS>, but it has been eventually resolved at <IP_ADDRESS>. Please upgrade it.

I downloaded a latest version some time ago. Today I checked with <IP_ADDRESS>: Following problems:

Grid not shown although ticked in Options
Repository Button missing

Firefox Quantum 57.0.4 (64-bit)

The naming convention for the Docker image tag has changed. See https://github.com/HiromuHota/webspoon-docker#tags for how it's changed.


add benchmark for Storm Trident API

How does one trigger the CLA bot?

CLA is valid!

I guess I just found out. :)

@knusbaum we never added in the docs how to run the validation test to be sure the code is doing the right thing.  Could you post it here.  I will add it to the docs/automated script and test the trident code.


[bug] fix hasAvailability

The current hasAvailability only propoagates the experts' hasAvailability urls to the document.   setup.  I propose that we more accurately represent this with the preferred label as well, so...
{
  "_id": "expert/LDdgBTXN",
  "_version": 74,
  "_source": {
    "@id": "expert/LDdgBTXN",
    "@context": "http://localhost/api/schema/1/context.jsonld",
    "@graph": [],
   "hasAvailability": [{
          "skos:prefLabel": "Collaborative projects",
          "availabilityOf": "expert/LDdgBTXN",
          "@type": "Concept",
          "@id": "ark:/87287/d7mh2m/keyword/c-ucd-avail/Collaborative%20projects",
          "skos:inScheme": {
            "@id": "ark:/87287/d7mh2m/keyword/c-ucd-avail/"
          }
        }],
        "orcidId": "0000-0001-9829-8914",
        "@id": "expert/LDdgBTXN"
        }

This is more standard with our practice of just promoting @graph components to the document level.
Furthermore, we base the hasAvailability index in elastic search on the prefLabel  so that filters will look like
/search?q=hart&hasAvailability=Collaborative%20projects

And finnally our schema will promote skos:prefLabel to prefLabel

@UcDust does this seem reasonable? UI effects should be limited to your search formulation only, AFAIK.

@qjhart yea this seems like it would work.