oEmbed plugin

This plugin intercepts the first links in the posts, searches its providers to see if it knows about an embed preview for each link based on their domain, and if it does, renders the enhanced preview for them.
The patch of this PR is the difference between the plugin template and the final oEmbed plugin.

How this is preventing pops?

After talking with @enahum, the plugin needs some more work to fix the scroll pops issue, tagging as work in progress

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@mgdelacroix Is this ready to merge?

@hanzei it was waiting for a solution to the scroll pop problem, which I've just pushed :)
@enahum please review the new approach to the embed frame dimensions, hopefully it will work nicely with the chat scroll

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Raised the concerns related to the embeding of HTML and the use of iframe with @DSchalla so he can help checking the security aspects of the plugin

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@mgdelacroix Have you had a chance to talk with @DSchalla about this yet?

@hmhealey This is blocked currently on my side.

Thanks for the review @DSchalla :)

Should we potentially aim to a whitelist of trusted souces?

I think this is the way to go. Sadly it has proven to be difficult and risky to use the generic idea that oEmbed offers, but maybe an intermediate approach would be a good fix. I'm thinking on adapting the plugin and create a static component per oEmbed provider, I mean, putting the oEmbed response for Spotify that we know that is an iframe on their domain in a static component and use it for the Spotify url matches.
It would be a more iterative approach of adding those providers that we know are safe, but we can reuse the current plugin structure and some core ideas of the current implementation, and it should be safe as we would basically use the iframes that we know are safe in a static fashion.
Any thoughts or ideas?

Alternatively, can we just merge this PR and take note of the outstanding security concerns. There is a lot of good stuff in this PR !

Agree, I think I can fix the conflicts and merge it easily, and then we can decide if it's worth to move forward and fix the security issues taking advantage of what has already been done :)

Codecov Report

Merging #1 (f5d0bfc) into master (16c74b8) will increase coverage by 20.23%.
The diff coverage is 55.55%.

:exclamation: Current head f5d0bfc differs from pull request most recent head cddd3d0. Consider uploading reports for the commit cddd3d0 to get more accurate results

@@             Coverage Diff             @@
##           master       #1       +/-   ##
===========================================
+ Coverage    4.76%   25.00%   +20.23%
===========================================
  Files           3        3
  Lines          21       44       +23
===========================================
+ Hits            1       11       +10
- Misses         20       31       +11
- Partials        0        2        +2

Impacted Files
Coverage Δ

server/plugin.go
57.89% <55.55%> (-42.11%)
:arrow_down:

server/main.go
0.00% <0.00%> (ø)

server/configuration.go
0.00% <0.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 16c74b8...cddd3d0. Read the comment docs.

@mgdelacroix I fixed the merge conflicts as I caused them initially. Leaving it to you to merge the PR.

Thanks @mgdelacroix @hanzei 👍

Thanks a ton for fixing the conflicts @hanzei!! Merged!


Initial values for solving the steady state

Hi!
I'm running the example with initial values for solving the steady state in a separate file (RISE_toolbox/examples/NonlinearModels/), but I get the error below when trying to solve the model. Is there a bug, or am I doing something wrong?
Best,
Tord
************************** ERROR MESSAGE *****************************
Undefined function or variable "nvar".
Error in locate_variables (line 40)
IDs=nan(nvar,1);
Error in dsge/compute_steady_state>prepare_steady_state_program (line 947)
locs=locate_variables(updated_param_list,obj.parameters.name);
Error in dsge/compute_steady_state (line 129)
[obj,sscode,blocks,unsolved]=prepare_steady_state_program(obj);
Error in dsge/solve/solve_zeroth_order (line 594)
[obj,structural_matrices,retcode]=compute_steady_state(obj);
Error in dsge/solve (line 313)
retcode=solve_zeroth_order();

Hi Tord,
The example was broken because the syntax changed a while back. I have now updated the example and checked that it runs as it should.
Cheers,
Junior


implement auth middleware for s3

Signed-off-by: Chanwit Kaewkasi<EMAIL_ADDRESS>

After the review I noticed that upstream exposes some of this functionality:
https://github.com/aws/aws-sdk-go-v2/blob/main/aws/signer/v4/v4.go
We may benefit from using their implementation for the signing and verification part. They also have a middleware implementation, but I am not sure how useful that would be for us.

After the review I noticed that upstream exposes some of this functionality: https://github.com/aws/aws-sdk-go-v2/blob/main/aws/signer/v4/v4.go
We may benefit from using their implementation for the signing and verification part. They also have a middleware implementation, but I am not sure how useful that would be for us.

Here's some background.
The codes I'm using was taken from the v4 signing of the MinIO lib (Apache license, also used by Flux).
Instead of signing, I reverse its process to implement the verification method.

Here's some background.
The codes I'm using was taken from the v4 signing of the MinIO lib (Apache license, also used by Flux).
Instead of signing, I reverse its process to implement the verification method.

@chanwit thank you for the context. Unfortunately neither MiniIO nor AWS seems to expose the entire logic, so it does make sense the approach.


Binder

Binder links do not work as expected.
After building a Docker image, an error occurred.
https://notebooks.gesis.org/binder/jupyter/user/spm-lab-sparse-ir-doc-c33q2ddq/tree/src/src/transformation_py.ipynb

Binder works well with jupyter book here. Is it crucial to include both of *.md and *.ipynb files?
https://geographicdata.science/book/intro


Updates to custom titlebar docs post appwindow titlebar merge

With Winappsdk 1.4, WinUI 3 custom titlebar now uses Appwindow titlebar underneath. As a result, there are a few changes which require to be documented. This PR captures all those changes and adds them to the doc.

assigned to @jwmsft


merge from arlac77/npm-package-template

README.md

docs(README): update from template

Pull Request Test Coverage Report for Build 2408

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

Totals

Change from base Build 2403:
0.0%

Covered Lines:
361

Relevant Lines:
375

💛  - Coveralls


fix: add Datadog dashboard service in the readme

Add the new Datadog dashboard service in the readme.
Dashboard endpoint in Datadog API unifies screenboard and timeboard APIs.

@googlebot I signed it!


Updating sub-nav margin

Removing margin at 480 breakpoint so that sub-nav border width matches that of the segment border width between 480 and 720.

:shipit:


Add the dyn keyword where trait objects are used

This is just because trait objects without dyn have been deprecated in nightly. A proper conversion to Rust 2018 edition will be done at a later time.

bors r+


Refactor Falcon7b to use ttnn multi-device tensors (data parallel)

Ticket
#5383
Problem description

Python loops were being used to achieve data parallel instead of ttnn multi-device tensors

What's changed

Replaced all device usages for wormhole, t3000, tg to device_mesh and replaced tensor lists with ttnn multi-device tensors (while maintaining support for the grayskull model which still uses single device)
Various miscellaneous cleanup in the demo and submodule tests
Deleted obsolete untested prefill-decode test

Checklist

[x] Post commit CI passes
[x] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[x] New/Existing tests provide coverage for changes

Pipelines (in progress):
Device perf: https://github.com/tenstorrent/tt-metal/actions/runs/10411239032
Model perf: https://github.com/tenstorrent/tt-metal/actions/runs/10411243503
T3k model perf: https://github.com/tenstorrent/tt-metal/actions/runs/10411247207
T3k perplexity: https://github.com/tenstorrent/tt-metal/actions/runs/10411252283
T3k demos: https://github.com/tenstorrent/tt-metal/actions/runs/10411255879
Single-card demos: https://github.com/tenstorrent/tt-metal/actions/runs/10411261417
Post-commit model tests: https://github.com/tenstorrent/tt-metal/actions/runs/10411266598

Added @cfjchu as a reviewer.

Looks great, were there any notable perf deltas?

Looks great, were there any notable perf deltas?

should be faster since i see you're now using async mode for multi-device tests

Looks great, were there any notable perf deltas?

should be faster since i see you're now using async mode for multi-device tests

We were already using async for our multi-device perf and demo tests (I also updated the submodule tests to use async in this PR), but I am seeing some perf differences from the multi-device tensor changes:
T3K (8-chip) demo perf | seq 1024: 0.8x for prefill, 1.5x for decode
TG (32-chip) demo perf | seq 1024: 0.9x for prefill, 2.5x for decode
I'm not fully sure why there's a degradation for prefill, will try to investigate further.

@djordje-tt could you please also review as well?

Looks great, were there any notable perf deltas?

should be faster since i see you're now using async mode for multi-device tests

We were already using async for our multi-device perf and demo tests (I also updated the submodule tests to use async in this PR), but I am seeing some perf differences from the multi-device tensor changes:
T3K (8-chip) demo perf | seq 1024: 0.8x for prefill, 1.5x for decode
TG (32-chip) demo perf | seq 1024: 0.9x for prefill, 2.5x for decode
I'm not fully sure why there's a degradation for prefill, will try to investigate further.

nice decode perf improvement!


[tensorflow] upgrade to 2.4.1

upgrade tensorflow to 2.4.1
this PR is branched off PR #14394, so please wait until 14394 is merged, then the review of this PR will be minimal

@NancyLi1013 as the TensorFlow UWP branch won't get merged in the foreseeable future, I removed the UWP changes from this PR, so that it only contains the TensorFlow v2.3.1 -> v2.4.1 upgrade now, so you can remove the "depends:different-pr" label.
Contents of this PR are now just a) removal of patches that were merged upstream and b) new patches that got necessary.

@jgehw
Is this PR ready for review now?

Is this PR ready for review now?

@NancyLi1013 yes, please

@NancyLi1013 please trigger a full Linux build, then the CI should build fine. The current CI error is because there were still files from the previous build:
Installing package tensorflow-common[core]:x64-linux...
The following files are already installed in /mnt/vcpkg-ci/installed/x64-linux and are in conflict with tensorflow-common:x64-linux

/azp run

Converting to draft while this PR is blocked on another.

Converting to draft while this PR is blocked on another.

@ras0219-msft It's no longer blocked as I reverted the UWP specific changes on March 2nd. Afterwards, this PR has already been reviewed by @NancyLi1013, so it's actually ready for merge into master.

Relate to UWP PR https://github.com/microsoft/vcpkg/pull/14394

@ras0219-msft, since this doesn't include the uwp changes any more, could you help review and merge this PR?

Thanks, this looks great to me!


ng-derective sub generator is not working

/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:579
throw e;
^
TypeError: Cannot read property 'capitalize' of undefined
at NgDirectiveGenerator. (/usr/local/lib/node_modules/generator-mean-seed/ng-directive/index.js:107:80)
at NgDirectiveGenerator. (/usr/local/lib/node_modules/generator-mean-seed/node_modules/yeoman-generator/lib/base.js:233:7)
at PromptUI.onCompletion (/usr/local/lib/node_modules/yo/node_modules/inquirer/lib/ui/prompt.js:69:10)
at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.completed (/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:1977:12)
at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:1914:14)
at AnonymousObserver.tryCatcher (/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:567:29)
at AutoDetachObserverPrototype.completed (/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:5032:56)
at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:1914:14)
at InnerObserver.onCompleted (/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:3345:65)
at InnerObserver.tryCatcher (/usr/local/lib/node_modules/yo/node_modules/inquirer/node_modules/rx/dist/rx.js:567:29)

Thanks for the logs @awaemmanuel
Seems like the underscore dependency was missing. I just added it and pushed an update - could you try again? And if that still does not work, try installing underscore globally and then see if it works?

Hello Luke,  Thanks I'll try again.

So do npm install - g underscore?

So do npm install - g underscore?

@awaemmanuel First try updating the generator and running as is (I added underscore as a dependency there). If that does not work then yes, run npm install -g underscore

Actually this should be fixed with https://github.com/notorii/generator-mean-seed/issues/19

Yea it seems as a duplicate ticket.


Task - VnV Plan - Tests for Non-Functional Requirements

Description

[ ] #197
[ ] #198
[ ] #199
[ ] #200
[ ] #201
[ ] #202
[ ] #203
[ ] #204

Definition of Done

Note that the boundary between each requirement type isn't strict and a test for one requirement may also hit on others, but we just need to make sure that we have at least one for each requirement


[PDI-19292] ERROR messages related to LoggingRegistry Purge keeps get…

…ting logged.

:white_check_mark: Build finished in 26m 34s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core

:ok_hand: All tests passed!
Tests run: 5170, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


Remove unused robots in dev mode too

I'm not entirely sure what the side-effects are of reordering these events, advice would be more than welcome!
But the benefits of this is my simulator runs at 5x speed with 1 robot in the arena, and 3x speed with 4 robots in the arena, so it's about a 66% simulation speed improvement

Immediate thoughts, but haven't thought much yet:

we should test this in each of the cases of
the default robot code location
just zone-0
just some other zone
what happens if there is no user-code found
does this make it less obvious to competitors that there will be other robots in the arena?
does this make it less obvious to competitors that they can test their code in other starting zones?

the default robot code location
Robot was deleted :( will have to fix before submitting.

just zone-0
This works

just some other zone
This also works

what happens if there is no user-code found

Seems to behave correctly, a robot.py was created!

does this make it less obvious to competitors that there will be other robots in the arena?

I don't think so? Maybe we can mitigate this by adding a starting box under where the robots start

does this make it less obvious to competitors that they can test their code in other starting zones?

I think we should fix this by documenting it better


Migrate to Maven to build

Not sure if this is something you'd be interested in or not. If not, feel free to close.
Only way I know how to set up Maven is to use the "recommended" folder structure. Basically had to move the "io" directory down two levels. It looks like:
|-- 1.13CommandAPI
|   |-- pom.xml
|   `-- src
|       `-- main
|           |-- java
|           |   `-- io
|           |       `-- github
|           |           `-- jorelali
|           |               `-- commandapi
|           |                   |-- api
|           |                   |   |-- arguments
|           |                   |   |   |-- Argument.java
|           |                   |   |   |-- ArgumentUtil.java
|           |                   |   |   |-- BooleanArgument.java
|           |                   |   |   |-- ChatColorArgument.java
|           |                   |   |   |-- DoubleArgument.java
|           |                   |   |   |-- EnchantmentArgument.java
|           |                   |   |   |-- EntityTypeArgument.java
|           |                   |   |   |-- FloatArgument.java
|           |                   |   |   |-- IntegerArgument.java
|           |                   |   |   |-- ItemStackArgument.java
|           |                   |   |   |-- LocationArgument.java
|           |                   |   |   |-- ParticleArgument.java
|           |                   |   |   |-- PlayerArgument.java
|           |                   |   |   |-- PotionEffectArgument.java
|           |                   |   |   |-- StringArgument.java
|           |                   |   |   `-- TextArgument.java
|           |                   |   |-- CommandAPI.java
|           |                   |   |-- CommandExecutor.java
|           |                   |   |-- CommandPermission.java
|           |                   |   `-- SemiReflector.java
|           |                   |-- Example.java
|           |                   |-- Main.java
|           |                   `-- Main.java.OLD
|           `-- resources
|               `-- plugin.yml
|-- LICENSE
|-- README.md
|-- v1.0 Documentation.md
|-- v1.1 Documentation.md
`-- v1.2 Documentation.md

You can compile this with mvn clean install from the command line if you have maven installed. You can also do the same with eclipse by importing the project as an "Existing Maven Projects" (File->Import->Maven->Existing Maven Projects)
To build, you can make a new Run As -> Maven Build... (with the ...) and in the Goals: field put clean install. Output jar shows up under target directory.
Again, feel free to close if you don't want to go this route - I just needed a way to compile it :)

For some absurd reason, Eclipse complains with having the "io" directory down two levels, but maven compiles fine. It was a nice suggestion, but for now I'm choosing not to go with this method.


Help injecting image pull secrets into my deployment manifest.

What happened?
I cannot inject imagePullSecrets into my manifests/deployments via devspacy.yml
What did you expect to happen instead?
I'm expecting this to be possible. I think I can do something like this (I'm using Dockerhub):
pullSecrets:
  dockerhub:
    registry: docker.io
    secret: dockerhub

How can we reproduce the bug? (as minimally and precisely as possible)
In my deployment manifest, I'd expect to reference the above secret via:
      imagePullSecrets:
        - name: dockerhub

But the secret is not being created.
My devspace.yaml:
pullSecrets:
  dockerhub:
    registry: docker.io
    secret: dockerhub

Local Environment:

DevSpace Version: [use devspace --version]
Operating System: linux (Ubuntu 22.04)
ARCH of the OS: AMD64
Kubernetes Cluster:
Cloud Provider: k3d (local)
Kubernetes Version: v1.26.7

Anything else we need to know?

@lizardruss yes, I did. User error on my part. :)


Substitute java awt jni

Integration test added, validating that we can read an example project file (kv-solsidan).
Awt loading a jni library and a native-method substituted with nothing. Awt-Color is used for some fields in Primavera projects causing exceptions to be thrown at runtime, and colors falls outside our domain.

@NiklasJonsson6 Is this PR what we've been running the last 6 months? In that case I have reviewed now 😄

@NiklasJonsson6 Is this PR what we've been running the last 6 months? In that case I have reviewed now 😄

It seems that way, judging by the dependency version we use and the time of commits. Thanks for the review, lets merge! :)

Now I'm sure after finding this commit https://github.com/Yolean/yolean.github.io/commit/d7b034e992b93a1d0cbb3d44f7603706b6c37d80


Create package / custom element manifest validator page

To help package maintainers publish valid custom elements manifests, we should add a validator page.
This page will have an input for an npm package name. Entering a package name will trigger a standard import of the page into the registry, and report validation errors in the UI. This means that we will have to store validation errors in the database in order to show errors without performing a new import on already imported packages.
Validation checks can be categorized as those strictly required to process a CEM and those the registry add for inclusion criteria (package type module, etc).
Some validation checks:

✅ package.json has customElements field
✅ Manifest pointed to by customElements exists
✳ package.json has a description field.
✅ Manifest parses as valid JSON
✅ Manifest schemaVersion number is known
✅ Manifest validates with the corresponding JSON schema for manifests of that version
✳ The manifest has a readme field: the file exists, and is markdown

✳ The file exists
✳ Parses as GitHub Flavored markdown
✳ Doesn't contain unsupported HTML elements

For every module listed in the manifest:

✅ The module exists in the published package
✳ The module is in the package exports (Maybe? We don't differentiate between the exported and internal modules in CEM, but internal modules may need to be described because they have the original declarations)
✳ The module parses/validates as a module (strict mode, doesn't use CJS APIs)
For every export of the module

✅ Type of export is known (js or custom-element-definition)
✅ declaration fully resolves to a valid declaration
✳ Warn if declaration points outside the package (until have cross-package references)
✳ custom-element-definition export:

✳ name is defined and contains a -
✳ Resolved declaration is kind: "class" with customElement: true

✳ js export:

✳ name is default, *, or a valid identifier

For every declaration of the module:

class:

✅ superclass reference resolves
✅ mixins references resolve
✅ ... all member's references resolve

✅ field: type validates

'function':

TODO

'mixin':

TODO

'variable':
TODO

If a package has a package export map, the manifest should also be added there


Implementing contact wrench cone

Implementing contact wrench cone cost and its unittest.
You can see the com trajectory with contact-wrench cone through the existed example, "bipedal_walk.py"

Thanks a lot @jcarpent  and @cmastalli
With your valuable comments, I modified codes and made unittest code for wrench cone.
Could you review it once again?

@ggory15 I have cleaned up the code as discussed internally. I had to do other changes that were not fully related with your own contribute but connected with it. An example of it was to modify a couple of examples that were working badly due to an infeasible step timings when the wrench cone is included.
The code is ready to be merged if everything passes.

@ggory15 the code is good but there is a failure in the Xenial job. This is the error:
/home/travis/build/loco-3d/crocoddyl/unittest/test_wrench_cone.cpp:35:26: error: ‘UnitRandom’ is not a member of ‘Eigen::Quaterniond {aka Eigen::Quaternion<double>}’

   Eigen::Quaterniond q = Eigen::Quaterniond::UnitRandom();

We need to solve them, but I don't know what it could be the reason. @nim65s, @wxmerkt  or @jcarpent  do you know why it can't be found UnitRandom in Xenial? Travis is installing Eigen as:
sudo apt install -y -qq libeigen3-dev

@ggory15 the code is good but there is a failure in the Xenial jobs. This is the error:
/home/travis/build/loco-3d/crocoddyl/unittest/test_wrench_cone.cpp:35:26: error: ‘UnitRandom’ is not a member of ‘Eigen::Quaterniond {aka Eigen::Quaternion<double>}’

   Eigen::Quaterniond q = Eigen::Quaterniond::UnitRandom();

We need to solve it, but I don't know what it could be the reason. @nim65s, @wxmerkt or @jcarpent do you know why it can't be found UnitRandom in Xenial? Travis is installing Eigen as:
sudo apt install -y -qq libeigen3-dev

For a quick fix, I write unittest code without using "UnitRandom".

UnitRandom appeared in 3.3~beta2, while 16.04 have 3.3~beta1.
Before that, this was UniformRandom: https://github.com/eigenteam/eigen-git-mirror/commit/04a67a761e0387b804cee23757491dab098d6265

But UniformRandom was added by @jmirabel, also in 3.3~beta2: https://github.com/eigenteam/eigen-git-mirror/commit/d26731c65512fe753af2827d9015a0c1accf732a

@nim65s thanks for the explanation :)

You can use the one internally present in Pincchio.

You can use the one internally present in Pincchio.

Indeed, @ggory15 implemented a solution based on Pinocchio


StringHelper::truncate with default horizontal ellipsis #16

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Tests pass?
✔

Fixed issues
#16

Merged.


Crashing entering abyss

Describe the bug
Almost every time i enter a dungeon/go into an instanced fight/ npc teleports me Runelite crashes. Just being in the abyss seems to crash as well. Hope these are the logs and help.
launcher_2019-06-21.0.log
launcher_2019-06-23.0.log
launcher_2019-06-28.0.log
launcher_2019-06-20.0.log
.
To Reproduce
Steps to reproduce the behavior:
Go to 'abyss'
See error
Expected behavior
Issue to be fixed. Its been happening since two updates ago and was hoping it would have been fixed, but it hasn't.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):
OS: [ Windows]
RuneLite version: [e.g 1.5.27]
Launcher version: [e.g 1.6]
Additional context
Add any other context about the problem here, e.g. logs. Your client logs can usually be found in
C:\Users<your_user_name>.runelite\logs on Windows and ~/.runelite/logs on Linux and macOS.

We need client logs, because the client is crashing, not the launcher.

client_2019-06-23.0.log
client_2019-06-28.0.log
client_2019-06-21.0.log
client_2019-06-22.0.log

Are you still seeing this crash? If so, can you please post your client and launcher versions (found on the info tab in the side panel), your client logs, and any jvm_crash files you may have in your logs folder?


Create gitattributes

This prevents GitHub from overwhelmingly representing the repository as GLSL/HLSL shaders, that being from the Alloy directory.

Thanks for the suggestion. I've done some research on the gitattributes file and discovered that there's a lot more that should probably go in there. I've made a commit directly to the main branch with it.


[CP-SAT] Developing custom decision strategies (C++)

Hi!
Is there any guide or example solutions about developing custom VariableSelectionStrategies or DomainReducionStrategies?
I want to implement domain specific heuristics to speed up the search. I imagine something like choosing variables in an order which depends on other variables (e.g. classrooms has a capacity and the scheduling of their id-s is in the order of decreasing capacity).
Thanks!

There are none :-) There are static strategies, and you can sort the
variables beforehand.
In general, using a black-box solver, I would solver sub-models in a
russian doll way, hinting the solution of the smaller doll when solving the
bigger doll.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
*De : *Bendegúz Módli<EMAIL_ADDRESS>*Date : *mer. 15 mai 2019 à 16:55
*À : *google/or-tools
*Cc : *Subscribed
Hi!

Is there any guide or example solutions about developing custom
VariableSelectionStrategies or DomainReducionStrategies?
I want to implement domain specific heuristics to speed up the search. I
imagine something like choosing variables in an order which depends on
other variables (e.g. classrooms has a capacity and the scheduling of their
id-s is in the order of decreasing capacity).
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1279?email_source=notifications&email_token=ACUPL3IYX3QLDKZB6IJYCGDPVQP5RA5CNFSM4HNEEZI2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GT6KOPA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3LTLLNJYSB6A6VQIITPVQP5RANCNFSM4HNEEZIQ
.

Okay, thanks for the answer, I figure out something on the lines what you said.


Getting import limits from org owner on user creation

This closes #9461

@javitonino CR this, please

Nice and clean :sparkles:. I left a comment about also getting max layers in the same way. Also, please test creation from superadmin just in case.

@javitonino CR this again, please. Creation from superadmin won't work like that, as those aree fields. Nevertheless Central gets updated fine.

:+1: Superadmin behaviour makes sense too. :rocket: it!


Implement Semantic Versioning

Notary V2 uses SemVer for plugin versioning. See spec here
At the time of writing this issue, Golang has no standard library support for parsing or comparing SemVer strings. This issue is to track implementing SemVer in Notation if no alternative is available. This issue is blocking using verification plugin minimum version critical attribute in signature verification workflow.
code: https://github.com/notaryproject/notation-go/blob/7e391f7562e4249272b8f3b271cc33ba4380775f/verifier/verifier.go#L127-L128

@rgnote Golang has standard library support for parsing and comparing SemVer strings. Those libraries are live in the go mod repo (see github.com/golang/mod) where the semver package is available here.

@shizhMSFT Couple of issues with x/mod

It requires versions to be prefixed with v
Allows partial strings such as 1, 1.0, etc.

We may need to write a wrapper around it to make it work with Notation.

PR https://github.com/notaryproject/notation-go/pull/237


Add support of visionOS

The platform of the target X (visionOS 1.0) is not compatible with PromiseKit (8.1.1), which does not support visionOS.

Happy to receive PR, cannot do myself RN.


Ena vpc acl fix

This includes the patch from PR 1581 by pdube.  I have manually tested and verified that it works.

Can you explain what this PR is all about?  I will admit that I am a bit confused.  :P

Sorry, I put this pull request against the wrong tree.  That's what I get for being in a rush on a Monday morning.


problem about static files urls

Hi guys
I can enable hystrix dashboard successfuly.
But in production environment, my service is after zuul with this domain "http://sample_domain/service_name/..." which cause static files like jquery.js don't load successfully.
(It wants to download files for example from http://sample_domain/jquery.js instead of http://sample_domain/service_name/jquery.js)
Is there any way to fix this issue by for example overriding base urls of static files or ...?
Thanks.

unrelated to hystrix-dashboard


Results progressively slower

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug
when executing sql statements (e.g. debugging a SP, or even repeatedly executing tsql) the results window progressively gets slower and slower to populate
initially execution time and results appearing match
but over a short period of time the results take over a minute to appear, even for a execution time of under a second
e.g.
the message window could state 00:00:00.020
the results will not appear for over a minute, and when they do will be empty [i.e. not headers or rows, just "RESULTS" heading]
then eventually blank rows
and finally several minutes later data will appear
not sure if relevant, but MicrosoftSqlToolsServiceLayer is using over 2GB RAM
Azure Data Studio version: azuredatastudio 1.5.2 (f74080c96310354bc92211f1826c0fdce78ca2f6, 2019-03-22T06:25:26.627Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-2400 CPU @ 3.10GHz (4 x 3093)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.89GB (0.53GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

agent
Mic
0.37.0

Perhaps some kind of memory leak ?
Closing ADS and running a just a few select queries, I'm now in the position of a select statement showing execution time of 2.845 seconds, wiht 6 rows
Results pane, has headers, 6 rows but all empty
and MicrosoftSqlToolsServiceLayer over 5Gb and growing

looks like this might be a dupe of #5249

Closing as duplicate.


Please be compatible with llama.cpp

When I use the OpenAI Base Url to link to llama.cpp
{"tid":"6312","timestamp":1713860426,"level":"INFO","function":"log_server_request","line":2875,"msg":"request","remote_addr":"<IP_ADDRESS>","remote_port":5213,"status":200,"method":"OPTIONS","path":"/v1/moderations","params":{}}
{"tid":"6312","timestamp":1713860426,"level":"INFO","function":"log_server_request","line":2875,"msg":"request","remote_addr":"<IP_ADDRESS>","remote_port":5213,"status":404,"method":"POST","path":"/v1/moderations","params":{}}

It seems to be moderations, but I confirm that my llama.cpp server is fine and other programs can communicate with it through the openAI API.

You can disable moderator
CLI option
--no-use-moderator

You can disable moderator
CLI option --no-use-moderator

Can it be implemented on the web? I prefer to use it on the web.

It is already there
Toggle this one


[breaking][CI] Use CTK 12.4

Closes #10696
Closes #10370
Marking as breaking, as we are dropping support for CUDA 11.8.

Apply https://github.com/abseil/abseil-cpp/commit/cfde5f74e276049727f9556f13473a59fe77d9eb to patch Abseil C++ to be compatible with latest CUDA compiler.
Work around false positive unix.BlockInCriticalSection from clang-tidy. See https://github.com/llvm/llvm-project/issues/104241

Thank you for working on this! Using 12.4 is going to help with the external memory work significantly. Out of curiosity, what's the kernel driver version of the AMI? Is it using kernel-open? Ubuntu package:
$ apt search nvidia-driver
...
nvidia-headless-555-open/jammy 555.58.02-0ubuntu0~gpu22.04.1 amd64
  NVIDIA headless metapackage (open kernel module)
...

As for the GRPC, I use:
>>> import grpc
>>> grpc.__version__
'1.62.2'

from conda-forge, seems to work well with CUDA 12.4/5, if it helps.

@trivialfis Here is the kernel driver version:
$ nvidia-smi
Tue Aug 13 19:57:56 2024
+-----------------------------------------------------------------------------------------+
| NVIDIA-SMI 550.90.07              Driver Version: 550.90.07      CUDA Version: 12.4     |
|-----------------------------------------+------------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id          Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |           Memory-Usage | GPU-Util  Compute M. |
|                                         |                        |               MIG M. |
|=========================================+========================+======================|
|   0  Tesla T4                       On  |   00000000:00:1E.0 Off |                    0 |
| N/A   32C    P8             14W /   70W |       1MiB /  15360MiB |      0%      Default |
|                                         |                        |                  N/A |
+-----------------------------------------+------------------------+----------------------+

+-----------------------------------------------------------------------------------------+
| Processes:                                                                              |
|  GPU   GI   CI        PID   Type   Process name                              GPU Memory |
|        ID   ID                                                               Usage      |
|=========================================================================================|
|  No running processes found                                                             |
+-----------------------------------------------------------------------------------------+

The AMI is based on Amazon Linux 2, so it probably doesn't use kernel-open.

The next release, which is going to be 3.0.0 (due to the rewrite for the JVM packages).

@terrytangyuan BTW, we would love to get your input on https://github.com/dmlc/xgboost/pull/10639

Sounds great!

Hmm, RMM is bringing in a different CCCL. Might have to patch it by back porting the one liner fix.


Git Repo Folder Structure

Summary
Create an intuitive folder structure within the Git repository. This folder structure should align with the Wireframe / Website Design we have chosen. This folder structure should also align with the frameworks (Front and Back End) we have chosen to use to build the website with.
Acceptance Criteria

Supports the backend and front end frameworks
Components Folder with Sub Components Folder: Ex. footerComponent Folder, headerComponent Folder
Pages Folder: This will house each page on the Underdog Devs website. Home Page, Mentee Page, Mentor Page, each individuals pages.
CSS Folder
SCSS Folder
Pictures Folder

Discussion
N/A
Screenshots
N/A
Additional Information
N/A

Summary
Very brief summary of the foreseen changes.
Acceptance Criteria

Write a list here
of all of the expectations
that must be satisfied
in order for the feature or change
to be considered complete

Discussion
Add extended discussion here
Screenshots
optional
Additional Information
relevant links to Slack messages, for example

Hi @juanresendiz813 - I wanted to check in to see how this ticket is coming along. Have you gotten a chance to start on it? If not, no worries. Please let me know if you have any blockers that are preventing from starting this so that we can work to remove them. If there are no blockers I would love to understand where some of your thoughts are so far so please post a comment on this ticket. Talk soon, Matt

Hey Matt! I've been looking up a lot of best practices and I see why I have to work with two more tickets on the board in order to get it structured properly. Jess just reached out to me recently so we're meeting tomorrow in order to get some lead way. Ill check with the other ticket participants as well to see how it is going. By Tuesday I should have something going in order for everyone to review this week. If I run into anything Ill be sure to reach out(which I'm sure I wil be soon lol). Either way Ill keep you posted. Thanks!

Thanks for the update @juanresendiz813! Glad to see you leveraging the other mentees, that is awesome teamwork and something that you will do on a daily basis in a full time role. Please let me know if there is anything I can do as well.

https://whimsical.com/join/AcPtPYGFP7Kn3YFPht6bp2?invite=rcl7koq2
Hey Matt, here is a link to the draft folder structure we are thinking about. We noticed it will most likely be written in React but if that changes we can just edit where needed. You can add and edit yourself with the attached link. Thanks.

@juanresendiz813 - Nice job man, this is perfect. I really don't have any feedback. Once we decide on a framework we may need to adjust slightly but I think the structure you picked out should be pretty adaptable. Nice work again!


Alternate email for PIM notifications

Hello,
Do we have a way to use the alternate email in user profiles for communications sent  by PIM ?
Thanks !

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f5a932ce-6046-f6a9-9b85-188c8530174c
Version Independent ID: 196cb921-fbf8-6819-e0e2-8cd03fda4505
Content: Email notifications in Privileged Identity Management (PIM) - Azure Active Directory
Content Source: articles/active-directory/privileged-identity-management/pim-email-notifications.md
Service: active-directory
Sub-service: pim
GitHub Login: @curtand
Microsoft Alias: curtand

@EmiFlo
Thanks for your feedback! We will investigate and update as appropriate.

@EmiFlo
Thank you for your time and patience throughout this issue!
Unfortunately, there isn't a native way to default the PIM emails to your Alternate Email address. However, there are a few options you can take to ensure your alternate email gets the PIM emails.

Create an email rule within your primary inbox to forward any PIM emails to your alternate address.
Edit the Role Notification settings to include additional recipients (i.e. your alternate email).
Switch your Primary email address with your Alternate email.

If you'd like to have the ability of receiving PIM emails using an alternate email, I'd recommend leveraging our User Voice forum and creating a feature request, so our engineering team can look into implementing this.
Since this issue isn't directly related to improving our docs, I'll go ahead and close this out.
Thank you again for your time and patience throughout this issue.


Update Docs: Guides “nuxt”

Automatically generated by Netlify CMS

I have updated documentation for nuxt for using @nuxt/content module.
Any criticism is appreciated.
Thank You.


[Release] Setting all LC_* explicitly in binary upload script is redundant

Describe the enhancement requested
We can use LC_ALL instead.
Component(s)
Release

Issue resolved by pull request 41248
https://github.com/apache/arrow/pull/41248


How to update the forked repo from original repo

Gitea version (or commit ref): 2f8c65c
Git version: 2.11.0
Operating system: raspberry pi(dietpi)
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Description
Hi everyone,
I am a gitea's new user, I installed gitea on my raspberry pi which runs dietpi.
I have no idea to update the forked repo from original repo on the gitea's web gui.
When I forked a repo and then modified, I can used "Pull requests" to merge changes  to original repo, but if the original repo was modified, I can't use "Pull requests" like Github can press "switch the base" to change source and target.
Thanks

Hey there, if you want to pull changes from the main repository you should probably do this from the command line instead of using pull requests:
git remote add upstream [main_fork_clone_url]
# every time you want to update
git pull upstream master

i agree with @thehowl, my advise do git fetch before git pull
git remote add upstream [main_fork_clone_url]
git fetch upstream
git pull upstream master

Thanks for your answer.
Is it possible add a option on web UI?

I think this is duplicate of #2488 to update fork from web UI


MERN_MongoDB: 3.1 Creating Database in MongoDB(V)

Description
Creating Database in MongoDB

Explain all the steps for creating a MongoDB Database with the CLI.

It should include basic commands to insert data, show and use the database.

Explain all the steps for creating a MongoDB Database with Compass.

Project Directory
Branch MERN>MERN/Topic/Backend/MongoDB/
Name the File as 3.1_Creating_Database_in_MongoDB(V).md
Domain
MERN
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign


Specify gulp process exit code

Is there a way to specify the gulp process exit code? It doesn't seem possible within gulp. I'm hooking the process exit with a global var to do it right now, which is kind of ugly.
I'm trying to echo a subprocess's exit code to gulp's caller, roughly like:
// gulpfile.js
var gulp = require('gulp');
var spawn = require('child_process').spawn;

var gulpExitCode = 0;

process.on('exit', function() {
  process.exit(gulpExitCode);
});

gulp.task('test', function(done) {
  var proc = spawn('ls', [], { stdio: 'inherit' });
  proc.on('error', function(err) {
    done(err);
  });
  proc.on('close', function(code) {
    gulpExitCode = gulpExitCode || code;
    done();
  });
});

There's no good way to override this.  We only exit with 1 upon error or 0 upon success.
@contra yes, undertaker accounts for child_process event emitters.

@phated Where is this done currently?


fix moved asciidoc component back

part of https://github.com/fabric8-launcher/launcher-planning/issues/129

it will work regardless of the version


OpenSearch as replacement for propietary ElasticSearch

Is your proposal related to a problem?
Yes.  I have requirements to use Open Source infrastructure, as we cannot risk vendor lock-in for core components.
Describe the solution you'd like
Full support for OpenSearch, https://www.opensearch.org/ a fork of the open source version of ElasticSearch after then went proprietary on their license.
Describe alternatives you've considered
Use an old ElasticSearch
Create my own OpenSearch integration
Additional context
ElasticCo changed the license of new versions of their products to a proprietary, source available license https://www.elastic.co/pricing/faq/licensing
The Wagtail marketing website refers to ElasticSearch in several locations as being Open Source, however, this is no longer the case and the marketing website should be updated to reflect the Source Available license change in ElasticSearch.  Preferably, an new release of Wagtail would switch to using the Open Source OpenSearch and the marketing would be updated to talk about OpenSearch instead of ElasticSearch.
As a community, we can be thankful for the Open Source work that Elastic has created in the past, and their continued contributions to Open Source projects like Apache Solar, however, we should not continue to refer to ElasticSearch as Open Source when that is no longer the case.  They as a company made a choice to switch to a proprietary non-open license, and with that choice comes consequences.  We as a community need to react accordingly to not spread confusion about what is and is not open source.  The simplest and best approach is to adopt OpenSearch as the standard search interface in Open Source projects like wagtail.  In the meantime, references to Elastic being open source should be removed from the website and marketing materials.

Hi @ensignavenger, thank you for the suggestion. This sounds to me like a welcome addition. At the same time, I have a feeling it might be easier to build this as an external package first (see for example wagtail-whoosh), since Wagtail has good support for swapping search backends.
In our docs, I see we explicitly mention support for Amazon AWS Elasticsearch Service, which I believe is now Amazon OpenSearch Service, so it could be the existing search backend would already support OpenSearch with little to no change?

For the marketing website, I saw two mentions on the Features page as you described:

Elasticsearch is an excellent open source search engine with straightforward integration with Wagtail and great performance.

And:

Elasticsearch
Wagtail uses Elastic for finding content within the admin interface. It provides quick and fluent access to pages, images and documents.
We use Elastic because it’s open source, gives great performance and offers a straightforward integration with Wagtail.

I’m not too sure what I’d replace those with myself - someone else might feel more inspired, if not I’ll let people who look after the site know they’ll have to change this.

Thank you for responding and for your work providing Wagtail!  I agree it probably would not be too much work to get Open Search working with minimal changes to the existing integration code.  I do have some concern for the future maintenance as development diverges between Elastic and OpenSearch.  I haven't had a chance to dig in to see what it would take myself, yet.
As for the marketing site, I think those are the two references I found as well- simply removing the reference to elastic being Open Source would be a good first pass.  It could be removed entirely or changed to "source available".

We use Elastic because it provides great performance and offers a straightforward integration with Wagtail.   Wagtail also supports alternative search back ends via packages...

FYI, I tried dropping in OpenSearch in place of Elasticsearch, but get:
$ ./manage.py wagtail_update_index
Updating backend: default
default: Rebuilding index wagtail__wagtailcore_page
Traceback (most recent call last):
…
  File "/…/venv/lib/python3.10/site-packages/elasticsearch/transport.py", line 421, in perform_request
    _ProductChecker.raise_error(self._verified_elasticsearch)
  File "/…/venv/lib/python3.10/site-packages/elasticsearch/transport.py", line 638, in raise_error
    raise UnsupportedProductError(message)
elasticsearch.exceptions.UnsupportedProductError: The client noticed that the server is not Elasticsearch and we do not support this unknown product
$ echo \U+1F615
😕

FYI, more FOSS organizations are dumping Elasticsearch. For example, Homebrew won't install it, reports:
$ brew info elasticsearch
==> elasticsearch: stable 7.10.2
Distributed search & analytics engine
https://www.elastic.co/products/elasticsearch
Disabled because it is switching to an incompatible license. Check out `opensearch` instead!
Not installed
…

Previously, I've had luck supporting OpenSearch using the elasticsearch7 backend and pinning the elasticsearch dependency to 7.13.4 (the last version before the es library started throwing the UnsupportedProductError when connecting to a non-Elastic product).  However, in Wagtail 5.1 (commit 9203716bae8c55b460e309b16c65d5311ad6c747 ) the ES7 backend has started including settings as a keyword argument in its calls for compatibility for elasticsearch versions >= 7.15 or so.  Running on an older version of elasticsearch now gives an exception like:
Traceback (most recent call last):
  File "<project root>/manage.py", line 23, in <module>
    execute_from_command_line(sys.argv)
  File "<project root>/venv/lib/python3.9/site-packages/django/core/management/__init__.py", line 442, in execute_from_command_line
    utility.execute()
  File "<project root>/venv/lib/python3.9/site-packages/django/core/management/__init__.py", line 436, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "<project root>/venv/lib/python3.9/site-packages/django/core/management/base.py", line 412, in run_from_argv
    self.execute(*args, **cmd_options)
  File "<project root>/venv/lib/python3.9/site-packages/django/core/management/base.py", line 458, in execute
    output = self.handle(*args, **options)
  File "<project root>/venv/lib/python3.9/site-packages/wagtail/search/management/commands/update_index.py", line 158, in handle
    self.update_backend(
  File "<project root>/venv/lib/python3.9/site-packages/wagtail/search/management/commands/update_index.py", line 84, in update_backend
    index = rebuilder.start()
  File "<project root>/venv/lib/python3.9/site-packages/wagtail/search/backends/elasticsearch5.py", line 984, in start
    self.reset_index()
  File "<project root>/venv/lib/python3.9/site-packages/wagtail/search/backends/elasticsearch5.py", line 980, in reset_index
    self.index.reset()
  File "<project root>/venv/lib/python3.9/site-packages/wagtail/search/backends/elasticsearch5.py", line 972, in reset
    self.put()
  File "<project root>/venv/lib/python3.9/site-packages/wagtail/search/backends/elasticsearch7.py", line 25, in put
    self.es.indices.create(index=self.name, **self.backend.settings)
  File "<project root>/venv/lib/python3.9/site-packages/elasticsearch/client/utils.py", line 168, in _wrapped
    return func(*args, params=params, headers=headers, **kwargs)
TypeError: create() got an unexpected keyword argument 'settings'

As far as I can tell, there's no way to run Wagtail 5.1 on OpenSearch (and in particular Amazon OpenSearch Service) at this point.

Marking this for a potential 5.1.2 bugfix release, since if there was a previously working configuration (even if it's not one that we knew about :-) ) then this can be considered a regression.

We're seeing the same error ("IndicesClient.create() got an unexpected keyword argument 'settings'") when using a Bonsai 	Elasticsearch 7.10.2 instance.
Current Django/Wagtail versions are:
Django==4.2.4
wagtail==5.1.1
Previous versions are working fine:
Django==4.1.7
wagtail==4.2.1

Have fixed compatibility with <7.15 versions of the library in #10916, and added Opensearch to our CI. Just needs adding documentation, and I think we can call this officially supported.


Symbolics changes and bump version

Codecov Report

Merging #37 (efab905) into master (8bc04b4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   78.78%   78.78%
=======================================
  Files          11       11
  Lines         476      476
=======================================
  Hits          375      375
  Misses        101      101

Impacted Files
Coverage Δ

src/symbolics_interface.jl
33.33% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.


Multiple Queues / ReceiveEndpoints w/ IoC

I am unable to add multiple endpoints using StructureMap. I'm not entirely sure if I am declaring it wrong or what I might be missing. When I try to add a code block during instantiation I get a "A lambda expression with a statement body cannot be converted to an expression tree" error. If I don't pass in the context I can set up multiple queues but then the auto-wireup isn't working on my IConsumer<>
Here's my original multi-queue setup (without context):
For<IBusControl>(new SingletonLifecycle()).Use(Bus.Factory.CreateUsingRabbitMq(cfg =>
        {
            var host = cfg.Host(new Uri($"{settings.RabbitMqConnectionUri}{settings.MyVirtualHost}/"), h =>
            {
                h.Username(settings.RabbitMqUsername);
                h.Password(settings.RabbitMqPassword);
            });

            cfg.ReceiveEndpoint(host, settings.MyProductQueue, ec =>
            {
                ec.ExchangeType = ExchangeType.Topic;
                ec.AutoDelete = false;
                ec.Durable = true;
            });

            cfg.ReceiveEndpoint(host, settings.MyProductSaga, e =>
            {
                e.StateMachineSaga(productScrapeSaga, productScrapeRepo);
            });
        }));

Forward<IBusControl, IBus>();

My working StructureMap Code, I was able to workaround the expression tree issue for the host by declaring an action variable outside the Func:
Action<IRabbitMqHostConfigurator> hostLoginConfiguration = h => { h.Username(settings.RabbitMqUsername); h.Password(settings.RabbitMqPassword); };

For<IBusControl>(new SingletonLifecycle()).Use(context => Bus.Factory.CreateUsingRabbitMq(cfg =>
        cfg.ReceiveEndpoint(
            cfg.Host(new Uri($"{settings.RabbitMqConnectionUri}{settings.MyVirtualHost}/"), hostLoginConfiguration),
            settings.MyProductQueue,
            ec => ec.LoadFrom(context)
            )
        )
    );
Forward<IBusControl, IBus>();

My Failed Code:
var productScrapeSaga = new ProductScrapeSaga();
var productScrapeRepo = new InMemorySagaRepository<ProductScrapeSagaState>();

Action<IRabbitMqHostConfigurator> hostLoginConfiguration = h => { h.Username(settings.RabbitMqUsername); h.Password(settings.RabbitMqPassword); };

For<IBusControl>(new SingletonLifecycle()).Use(context => Bus.Factory.CreateUsingRabbitMq(cfg =>
    {
        cfg.ReceiveEndpoint(
            cfg.Host(new Uri($"{settings.RabbitMqConnectionUri}{settings.MyVirtualHost}/"), hostLoginConfiguration),
            settings.MyProductQueue,
            ec => ec.LoadFrom(context)
        );

        cfg.ReceiveEndpoint(cfg.Host(new Uri($"{settings.RabbitMqConnectionUri}{settings.MyVirtualHost}/"), hostLoginConfiguration),
            settings.MyProductSaga, e =>
            {
                e.StateMachineSaga(productScrapeSaga, productScrapeRepo);
            });
    }
));
Forward<IBusControl, IBus>();

Is there something I'm missing on setting up multiple queues from the same Container Registry? Is there a better way to configure sagas for Auto-Wireup using Structure Map?
Thanks,
Ross

I was able to work around this by creating a static method returning IBusControl. While this wasn't the most transparent way of resolving this it solves the issue.
public class ContainerRegistry : Registry
{
	public ContainerRegistry()
	{
		For<IBusControl>(new SingletonLifecycle()).Use(context => CreateRabitMqBusControl(context));
		Forward<IBusControl, MassTransit.IBus>();
	}

	private static IBusControl CreateRabitMqBusControl(IContext context)
	{
		var settings = new AppSettings();
		var productScrapeSaga = new ProductScrapeSaga();
		var productScrapeRepo = new InMemorySagaRepository<ProductScrapeSagaState>();

		return Bus.Factory.CreateUsingRabbitMq(cfg =>
		{
			var host = cfg.Host(new Uri($"{settings.RabbitMqConnectionUri}{settings.MyVirtualHost}/"), h =>
			{
				h.Username(settings.RabbitMqUsername);
				h.Password(settings.RabbitMqPassword);
			});

			cfg.ReceiveEndpoint(host, settings.MyProductQueue, ec => ec.LoadFrom(context));

			cfg.ReceiveEndpoint(host, settings.MyProductSaga, e =>
			{
				e.StateMachineSaga(productScrapeSaga, productScrapeRepo);
			});
		});
	}
}

Containers are weird, in how they work against some of the functional composition patterns at times. And of course they all behave differently. I'm trying to find ways to tease apart the internals so that they can be more easily composed inside containers, but they're just hard to get right.
There is a bunch of working going into v3.5 that changes a lot of how the disposable things work to avoid common screw-ups, so hopefully this makes it easier. The fact most containers were built before async was a thing makes it hard to do "async all the way down" when the container is a brick wall.

Thanks for your response. It was an easy enough workaround, I'll go a head and close the ticket. Just figured it would be good in case anyone else bumped into this issue.


I can't open the native process strings, permission error

Brief description of your issue
I can't open the native process memory strings, permission error

Steps to reproduce (optional)
SgrmBroker
MsMpEng
I need to access these two processes to see some records, they both don't open
Expected behavior (optional)
open the string search tab
Actual behavior (optional)
No response
Environment (optional)
System Informer 3.0.7333
Windows 11 10.0.22631 / 22631

These processes have PPL and Microsoft enforces limited access.

Is there nothing I can do to gain access?


tutor.lv and tutor.da is using UTF-8 encoding

https://github.com/vim/vim/blob/master/runtime/tutor/tutor.lv seem to be in UTF-8 and there is no tutor.lv.utf-8
If tutor.[ll].utf-8 should use UTF-8 encoding then what should tutor.[ll] use?

https://github.com/vim/vim/blob/master/runtime/tutor/tutor.lv seem to be in UTF-8 and there is no tutor.lv.utf-8
Same for tutor.da.
If tutor.[ll].utf-8 should use UTF-8 encoding then what should
tutor.[ll] use?

This is not totally consistent.  These days using utf-8 as the default
should be OK, it will work on most systems.
Otherwise, it's handled in tutor.vim.  Thus if there is only one tutor
available, and the current 'encoding' doesn't work for it, then it
should fall back to the English version.
--
Did you ever see a "Hit any key to continue" message in a music piece?
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Not so much fun for lv translator to spend time on translation just to have it fallback to english.

If the translation is in menu_[ll].utf-8.vim.
And menu_[ll].latin1.vim says:
source <sfile>:p:h/menu_[ll].utf-8.vim

Are the translation then not converted to UTF-8?

What controls which file is loaded? latin1 or utf-8.
It seems latin1 is loaded when both are there but why?

What controls which file is loaded? latin1 or utf-8.
It seems latin1 is loaded when both are there but why?

Look in tutor.vim
--
hundred-and-one symptoms of being an internet addict:
174.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I suppose this can be closed?

I found that the command vimtutor lv failed to find tutor.lv.
I think that tutor.lv should be renamed to tutor.lv.utf-8, and tutor.vim needs to be adjusted for that.
I will create a PR later.


Unable to compute the gradient of the wav2vec encoder

Before asking:
I found one closed issue on the gradient, but with no follow-up.
What is your question?
I cannot backprop the gradient through the wav2vec encoder. I have isolated that using the wav2vec encoder no longer backprops the gradient.
Code
Here is the code I run:
import os
import torch
import torch.tensor as T
from fairseq.models.wav2vec import Wav2VecModel

MODEL = "wav2vec_small.pt"
fname = MODEL
os.system("wget -c https://dl.fbaipublicfiles.com/fairseq/wav2vec/%s" % MODEL)
cp = torch.load(fname)
model = Wav2VecModel.build_model(cp['args'], None)
model.load_state_dict(cp['model'], strict=False)
model.eval()

time = torch.linspace(0.0, 2.5, 64000)
def sine(f0):
  return T(0.5) * torch.cos(f0 * 2.0 * 3.14 * time)

def extractor(x):
  return model.feature_extractor(torch.tensor(x.view(1, -1)))
def wav2vecdist(x, y):
  return torch.mean(torch.abs(extractor(x) - extractor(y)))

ft = T(440.0)
# If you enable GPU, you have to add 'device="cuda"'
with torch.enable_grad():
  fp = T(660.0, requires_grad=True)
  fp.retain_grad()
  dist = torch.mean(torch.abs(extractor(sine(ft)) - extractor(sine(fp))))
  print(dist)
  dist.backward(retain_graph=False)
  print(fp.grad)

As you can see, the extractor in wav2vec is not passing a gradient and I'm not sure why.
A minimal google colab is here: https://colab.research.google.com/drive/1l61syPS7GO7O-S0o8cWnv8R0JwzVq58T?usp=sharing
What have you tried?
I have tried to remove as much exterior code as possible.
I have tried fiddling with .train() and .eval() on the wav2vec model.
What's your environment?

fairseq Version (e.g., 1.0 or master): master
PyTorch Version (e.g., 1.0): 1.6.0+cu101
OS (e.g., Linux): Ubuntu 18.04.5 LTS
How you installed fairseq (pip, source): source
Build command you used (if compiling from source): !pip install -q --editable ./
Python version: 3.6.9
CUDA/cuDNN version: 10.1
GPU models and configuration: Tesla P100
Any other relevant information: Google Colab

I have tried also with fairseq 0.10.1 and have the same issue.
I tried further with Wav2Vec2Model and have the same issue.

  return model.feature_extractor(torch.tensor(x.view(1, -1)))

creates a copy, which loses the gradient. The line should be:
  return model.feature_extractor(x.view(1, -1))

The colab is updated.


java.io.IOException: Cannot run program "arc"

the below is the error message:
[api_build_job] $ arc patch --diff 10811 --nocommit --nobranch ********
java.io.IOException: Cannot run program "arc" (in directory "/root/.jenkins/workspace/api_build_job"): error=2, No such file or directory
	at java.lang.ProcessBuilder.start(ProcessBuilder.java:1047)
	at hudson.Proc$LocalProc.<init>(Proc.java:244)
	at hudson.Proc$LocalProc.<init>(Proc.java:216)
	at hudson.Launcher$LocalLauncher.launch(Launcher.java:815)
	at hudson.Launcher$ProcStarter.start(Launcher.java:381)
	at hudson.Launcher$ProcStarter.join(Launcher.java:388)
	at com.uber.jenkins.phabricator.conduit.ArcanistClient.callConduit(ArcanistClient.java:59)
	at com.uber.jenkins.phabricator.tasks.ApplyPatchTask.execute(ApplyPatchTask.java:129)
	at com.uber.jenkins.phabricator.tasks.Task.run(Task.java:58)
	at com.uber.jenkins.phabricator.PhabricatorBuildWrapper.setUp(PhabricatorBuildWrapper.java:151)
	at hudson.model.Build$BuildExecution.doRun(Build.java:156)
	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:534)
	at hudson.model.Run.execute(Run.java:1738)
	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
	at hudson.model.ResourceController.execute(ResourceController.java:98)
	at hudson.model.Executor.run(Executor.java:410)
Caused by: java.io.IOException: error=2, No such file or directory
	at java.lang.UNIXProcess.forkAndExec(Native Method)
	at java.lang.UNIXProcess.<init>(UNIXProcess.java:187)
	at java.lang.ProcessImpl.start(ProcessImpl.java:130)
	at java.lang.ProcessBuilder.start(ProcessBuilder.java:1028)
	... 15 more
[phabricator:arcanist] Error applying arc patch; got non-zero exit code FAILURE

I have arcanist in the host,and can run arc command,also add the path of arc to the PATH.
In the job's directory ,i can run the arc command.

It sounds like you've configured jenkins to point to an arc path that doesn't have an arc binary. What does your configuration point to? Do you have the arc binary installed on your jenkins slave nodes, or just the master? What path is it in?

I have the arc binary installed，and can run  the arc command in the jenkins' workspace。
In the global configuration of jenkins ，using the absolute Path of arc
(Location of arcanist ) can solve this problem.
Thanks a lot

Ok, it sounds like this issue is resolved, I'll close out. Yes, if you install the arc binary somewhere that is not in the jenkins' user's path, you need to specify the path in the global config.

Hi, I met the same problem,:
[pha_wave_driver_android_build] $ arc patch --diff 11164 --nocommit ******** java.io.IOException: Cannot run program "arc" (in directory "/Users/xxx/jenkins/workspace/pha_wave_driver_android_build"): error=2, No such file or directory
But I run arc command in "Execute Shell" block.

Do you have the arc binary installed? What path is it in? Did you set a
custom path in your global configuration? As much information as possible
is usually a good start to debug things like this.
On Tue, May 10, 2016 at 11:13 PM, Yecheng Zhou<EMAIL_ADDRESS>wrote:

Hi, I met the same problem,:
[pha_wave_driver_android_build] $ arc patch --diff 11164 --nocommit

java.io.IOException: Cannot run program "arc" (in directory
"/Users/xxx/jenkins/workspace/pha_wave_driver_android_build"): error=2, No
such file or directory
But I run arc command in "Execute Shell" block.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/uber/phabricator-jenkins-plugin/issues/145#issuecomment-218371003

yes, I have arc binary installed and the path is /Users/xx/CRClientTools/arcanist/bin/arc and it is already in PATH, I can run arc on both Linux command line and "Execute Shell" block in jenkins job step setting.

Sorry, after config arc path in global config, it works.

Ok, cool, I'll leave this closed out then. Let me know if you run into any other issues.

this is happening to me but my project is running on a slave. If I ssh into the slave and try to use arc it works fine. Do I also need to have arc on the jenkins server?

Have you configured the path properly? arc should not be necessary on the
master
On Wed, Jun 1, 2016 at 7:30 AM Nuno Horta<EMAIL_ADDRESS>wrote:

this is happening to me but my project is running on a slave. If I ssh
into the slave and try to use arc it works fine. Do I also need to have arc
on the jenkins server?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/uber/phabricator-jenkins-plugin/issues/145#issuecomment-223010438,
or mute the thread
https://github.com/notifications/unsubscribe/AAF7PwEmga_ggzSBxTPeNQz0GwHcMsZAks5qHZdqgaJpZM4H12DE
.

yeah I set an environment variable PATH for the slave. I restarted it and I can see the path updated under Node > System Information > Environment variables but it's still not working. Also if I go to the script console on that slave and run println System.getenv("PATH") the output is different from the environment variable

Can you try configuring your global Phabricator options with the absolute
path to "arc"? That way it won't depend on environment variables
On Wed, Jun 1, 2016 at 7:57 AM Nuno Horta<EMAIL_ADDRESS>wrote:

yeah I set an environment variable PATH for the slave. I restarted it and
I can see the path updated under Node > System Information > Environment
variables but it's still not working. Also if I go to the script console on
that slave and run println System.getenv("PATH") the output is different
from the environment variable
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/uber/phabricator-jenkins-plugin/issues/145#issuecomment-223019345,
or mute the thread
https://github.com/notifications/unsubscribe/AAF7PwIVxPh7iGw96q_rz1hdK-F6tjiwks5qHZ28gaJpZM4H12DE
.

it's still throwing the same error. The slave is running osx, I'm not sure if that helps

What does your global configuration look like? What path is arc installed
to so your OS X machines?
On Wed, Jun 1, 2016 at 11:12 AM Nuno Horta<EMAIL_ADDRESS>wrote:

it's still throwing the same error. The slave is running osx, I'm not sure
if that helps
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/uber/phabricator-jenkins-plugin/issues/145#issuecomment-223024487,
or mute the thread
https://github.com/notifications/unsubscribe/AAF7P8unxwE6SBpoikGGqPOiqJvoY-_sks5qHaFUgaJpZM4H12DE
.

That's what I added to the global configuration and to the slave. I can see the same path if I go to the slave "System Information". "build" is the name of the user that I setup to launch the slave agent via ssh

Which PATH environment variable does the plugin use by default?
It seems that the plugin is not sourcing the .bash_profile of our jenkins user before running because the arc command is in the PATH specified there. I can run arc from both Terminal.app when I am logged in as the jenkins user and from the "Execute Shell" block in Jenkins.
I could hardcode the location of arcanist but I am in the process of migrating our nodes to a new configuration where the arcanist location will differ than the old nodes. Rather than upgrade them all at once I was hoping I could provisioning the new nodes to put arcanist in the correct location and once they were all up and running go back to the old nodes and fix them.


clean up public classes and methods

only expose LazyGui and maybe Utils

investigate Facade Pattern

investigate putting everything in one package and making most stuff package private

attempt #1 compiled but crashed with no explanation


Bunch of Spf13 issues!!

I admit spf13.vim is the quickest, but when it doesn't work, it doesn't work really hard. Here are the issues I came up with.

I cannot install it in windows 8.1, it just stuck at the end.
I cannot install it using cinst either.
If I accidentally open the .cmd, it just starts everything from beginning, even if everything is present.
Not to mention gVim becomes slower (like Atom) if spf13 is successfully installed.

I wonder if the development for windows is stopped.

the continuing development and usage on windows requires folks who are able to deal with windows issues. I know i have no experience in that.  I wonder what windows versions @spf13 deals with on the day to day. Maybe it's just windows 7.

Only Windows 7 personally
We really need maintainers to step up and shepherd this who use Windows daily for dev. I'm happy to give commit access to anyone who wants to volunteer.

At last it worked.
I'll describe what happened, what worked and what didn't,
Previously I tried on...
Intel processor, Windows 7, 2 GB Ram, Intel HD Graphics
Amd processor, Windows 8.1 (32 bit and 64 bit), (2 and 4) GB Ram, 1 Gb ATI graphics,
Now, I tried to install it in...
Intel processor, windows 8.1 64bit, 4GB RAM, 2GB NVidia Graphics...
I installed Git, Curl, Vim normally... Even if I did that, Choco installed them again (What a pain!!!)... It didn't matter which way I installed them though...

Installing Spf13-vim via Choco...FAILURE...,

I tried to uninstall and reinstall again via Choco...FAILURE...
Showed "Not a git repo" alongside other problems...

Installed using .CMD file AS ADMINISTRATOR...Took a while...didn't get stuck like previous time...
SUCCESS...!!

NOTE: previously, I did the same and stuck at some point...
NOTE 2: Now, it ran way much faster, looks fun! Maybe it depends on Hardware/architecture when it comes to Windows....
I'll try to notify if any other problem occurs/ something comes up in mind in future...

I'm closing this issue since its fixed.


Tweak documentation

Change Summary
I found a few spots in the documentation to impact the reading flow a bit, so I changed them.
Related issue number
none
Checklist

n/a – Unit tests for the changes exist
n/a – Tests pass on CI and coverage remains at 100%
n/a – Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review

please update

please update

I have rebased, resolved conflicts, and removed the commit that adds the changes file.

Thanks @homeworkprod


streamline changest viewing for larger diffs

We basically output the same thing twice, and on large diffs we'll go well over the comment size limit for an issue.
streamlining this by minimize the necessary output, removing the 'detailed' view, in favor of updating the changeset to contain added info about the file sizes
while here I also updated some dependencies
fixes https://github.com/github/webpack-bundlesize-compare-action/pull/296
fixes https://github.com/github/webpack-bundlesize-compare-action/pull/294
fixes https://github.com/github/webpack-bundlesize-compare-action/pull/293

Once this lands, we should version and update internal consumers


Cannot read property 'portalUrl' of undefined

At first, it says deployed successfully.
Then getting this issue when trying to deploy my flask app.
I have imported to libraries flask and pandas.
On the webpage, it says application error.
Everything works fine locally.
Can anyone help?
2022-06-09T05:30:54.922635685Z   File "", line 671, in _load_unlocked
2022-06-09T05:30:54.922639185Z   File "", line 843, in exec_module
2022-06-09T05:30:54.922642885Z   File "", line 219, in _call_with_frames_removed
2022-06-09T05:30:54.922646485Z   File "/home/site/wwwroot/app.py", line 2, in
2022-06-09T05:30:54.922650185Z     import pandas as pd
2022-06-09T05:30:54.922653585Z ModuleNotFoundError: No module named 'pandas'
2022-06-09T05:30:54.930059369Z [2022-06-09 05:30:54 +0000] [63] [INFO] Worker exiting (pid: 63)
2022-06-09T05:30:55.191811839Z [2022-06-09 05:30:55 +0000] [59] [INFO] Shutting down: Master
2022-06-09T05:30:55.192534748Z [2022-06-09 05:30:55 +0000] [59] [INFO] Reason: Worker failed to boot.
It says module not found pandas. I have mentioned it in the requirement.txt file

Hi @karinapatel02! Could you please provide a little more context for this issue - like the name of the failed task and an example of the deployment template?

I am facing the same issue here, i have deployed my flask api to azure web services and i got the same error
11:49:15 AM dms-twitter-test2: Warnings (0)
11:49:15 AM dms-twitter-test2: Triggering recycle (preview mode disabled).
11:49:15 AM dms-twitter-test2: Deployment successful.
11:49:22 AM: Deployment to "dms-twitter-test2" completed.
11:49:42 AM: "dms-twitter-test2" reported a critical error: Exception
11:49:48 AM: Error: Cannot read properties of undefined (reading 'portalUrl')

I am facing the same issue.

I am also facing the same issue while deploying the spacy to azure app service. Is anyone able to fix this.

Please add "tensorflow", it was the solution for me. There seems to be an implicit (unsolved) dependency somewhere.
Let me know if works for you.


[PROPOSAL] Allow sending DatasetEvent and JobEvent using Transport.emit of Java client

Purpose:
Currently Transport.emit method of Python client accepts RunEvent, DatasetEvent, JobEvent. But Java client accepts only RunEvent and String, so both DatasetEvent and JobEvent should be serialized to JSON first.
This is actually a transport implementation detail. Some transport may use other formats for serialization, e.g. MsgPack, Avro, Protobuf and so on.
Proposed implementation
Allow emmiting DatasetEvent and Job Event by calling Transport.emit method. Serialization will be handler by transport, currently this is just JSON.

Draft implementation: #2611

@pawel-big-lebowski have a note about Kafka/Kinesis implementation:

We need first to decide how to handle different event types for Kafka and Kinesis. I would opt for separate topics but I am open to discussion and would love to hear others opinion too.
https://github.com/OpenLineage/OpenLineage/pull/2611#discussion_r1570061883

Kafka shards messages to different topic partitions using the value of key field of the message. Sending message with a key abc to a topic leads to storing this message in partition with number hash(key) % num_partitions. Using null as key value stores message in a random partition. For Kinesis it is almost the same, but partitionKey should always be set, it cannot be null.
Before #2559 Java client used static value for Kafka partition key (total message ordering, bad for performance), but Python used a random key (no ordering at all, bad for event handling). Also Python already supported sending any type of event, not just RunEvent.
#2559 introduced messageKey option for Kafka transport, which by default is generated as:

run:{jobNamespace}/{jobName}
dataset:{datasetNamespace}/{datasetName}
job:{jobNamespace}/{jobName}

This unified the message key generation for both Java and Python clients, allowing both to shard messages to different topic partitions, and to keep the order of events related to the same entity (run/dataset/job).
User may override this value by explicitly passing messageKey config option of Kafka transport, to send messages of parent and child run events in the same topic.
This leads to sending events of all types to just one topic (currently only for Python client). So any message received from the same topic should be first parsed as raw JSON, and then using value of schema field it can be decided which event type should be used to deserialize this message.
This is why run:, dataset: and job: prefixes were added, allowing consumers to route messages related to some specific event type. But this requires all Kafka consumers to share the same implementation detail, and I'm not sure is that correct or not.
Using different topics may solve this issue. But this is not actually only Kafka/Kinesis transport issue, but all of the transport implementations. For example, FileTransport will append different events to the same file, and file reader should then properly deserialize these events depending on event type.
I'm not sure that the solution should be.

The tradeoff is: (1) burden with amount of configuration entries on producers side vs (2) extra complexity on consumer side to detect which even type is actually sent.
FileTransport was introduced for testing purpose. For HTTP we have tested that event type can be resolved with standard Jackson library (https://github.com/MarquezProject/marquez/pull/2495/files#diff-baa4affac174a99d36ad8ae509a893002291665ca3deba5dae878301875b136e)
My Kafka experience (from long time ago) is that its consumers were fragile when reading from topic other data than expected. And, a newer version of producer could possibly pose a threat to process reading events from Kafka.
Would it make sense to at least let client decide whether to send everything in one topic or separately?

Would it make sense to at least let client decide whether to send everything in one topic or separately?

IIRC, default integrations with Airflow, Spark and Flink send only RunEvent. For DatasetEvent and JobEvent user have to write some code, and also configure which transport should be used. So user can set proper topicName for events generated in his code.

. So it's up to user to set topicName value.

Yeah, this makes sense.

So for now keeping the same set of options for all the transport implementations. User should set proper endpoint/topic/file name, which can handle event types.
For Kafka/Kinesis using the same partition key, to distinguish between event types without deserializing value:

run:{jobNamespace}/{jobName}
dataset:{datasetNamespace}/{datasetName}
job:{jobNamespace}/{jobName}

Is that OK?

For Kinesis, should I add explicit partitionKey option to allow user to override the value, just like in Kafka?

Moved changing Kinesis partitionKey format to #2620


[ALLUXIO-3231] Make a log4j.properties.template file.

https://alluxio.atlassian.net/browse/ALLUXIO-3231
The downside here is starting alluxio without making a logj4.properties file copy will cause our logging to fail (the system still comes up, there just errors reported and of course no .log files).
Also note we have log4j.properties files for our test/packages, but I think those are fine to keep.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20252/
Test PASSed.

I think we need to update the tarball generation scripts to work with this change

@aaudiber I updated the script. One downside of this change is that users will get (nonfatal) errors if they do not create a log4j.properties file explicitly. It might be worth the user experience to take a hit for code cleanliness


Update to version 0.4.6

Issue #, if available: #216
Description of changes: The Latin bug in installation is hurting emukit installation, so we need a new version with a fix out.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

As an example, I've run into it here: https://github.com/amzn/xfer/pull/60

Codecov Report

Merging #226 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #226   +/-   ##
=======================================
  Coverage   89.18%   89.18%
=======================================
  Files         110      110
  Lines        3124     3124
  Branches      335      335
=======================================
  Hits         2786     2786
  Misses        273      273
  Partials       65       65

Impacted Files
Coverage Δ

emukit/__version__.py
0% <0%> (ø)
:arrow_up:

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 3cfd510...24e6470. Read the comment docs.


rust: Display isntall isntruction on NotFound

Fixes #122

Here's an example output from running the tests without the lib:
---- load_sample_error stdout ----
thread 'load_sample_error' panicked at tests/load.rs:57:44:
called `Result::unwrap()` on an `Err` value: Shared library file 'libyamlscript.so.0.1.49' not found
Try: curl -sSL yamlscript.org/install | VERSION=0.1.49 LIB=1 bash
See: https://github.com/yaml/yamlscript/wiki/Installing-YAMLScript

Thanks. Can you also look in $HOME/.local/lib after /usr/local/lib?

This is the current behavior. /usr/local/lib is pushed first and looked in first.

Sorry, I was looking in the diff, but I forgot that behavior was already implemented.

I'm getting an error with the tests:
$ make -C rust test
make: Entering directory '/home/ingy/src/yamlscript/rust'
cargo test
    Finished test [unoptimized + debuginfo] target(s) in 0.01s
     Running unittests src/lib.rs (target/debug/deps/yamlscript-908c6d55c850b71c)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/load.rs (target/debug/deps/load-1aff250989321444)

running 5 tests
test load_loop ... ignored

thread 'load_sample_error' has overflowed its stack
fatal runtime error: stack overflow
error: test failed, to rerun pass `--test load`

Caused by:
  process didn't exit successfully: `/home/ingy/src/yamlscript/rust/target/debug/deps/load-1aff250989321444` (signal: 6, SIGABRT: process abort signal)
make: *** [Makefile:15: test] Error 101
make: Leaving directory '/home/ingy/src/yamlscript/rust'

I'm getting an error with the tests:

My bad. x.fmt(f) is a recursive call... I didn't have the correct version installed so the only NotFound test I had passed (the only one not triggering the infinite recursion).

Merged. Thanks!


erlinit: fix download url

Hello,
The erlinit git repository has changed the name of the branches, from "master" to "main", causing errors in the fetch step:
ERROR: erlinit-1.6.0-gitAUTOINC+53b0a4dfc5-r0 do_fetch: Fetcher failure: Unable to find revision 53b0a4dfc5aab5f883652527e29e2db39ac546ae in branch master even from upstream
ERROR: erlinit-1.6.0-gitAUTOINC+53b0a4dfc5-r0 do_fetch: Fetcher failure for URL: 'git://github.com/nerves-project/erlinit;branch=master'. Unable to fetch URL from any source.
ERROR: Logfile of failure stored in: [...]/build/tmp/work/aarch64-poky-linux/erlinit/1.6.0-gitAUTOINC+53b0a4dfc5-r0/temp/log.do_fetch.26591
ERROR: Task ([...]/poky/meta-erlang/recipes-core/erlinit/erlinit_git.bb:do_fetch) failed with exit code '1'

This patch fixes the URL, allowing the build to complete.

Thanks @adrians


Permit multipart filenames to contain non-ascii characters

This commit is partially cherry-picked to include only relevant part of the fix

Not sure why build fails, definitely not relevant to the changes, probably some configuration change in the build?

@swankjesse this was failing with an impossible compile error.  I suspect the install phase being run in a separate maven call than test means there is some cross build artifact pollution going on.  Trying to rebuild now.

Built successfully locally.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  02:53 min
[INFO] Finished at: 2019-05-02T20:28:44-04:00
[INFO] ------------------------------------------------------------------------


Squeezebox

Support for media player Logitech Squeezebox

Looks pretty good, just a few comments. I'm not sure why the checks are failing, but we will need to figure that out before merging.

It looks good to me. Pylint has some issues, we can merge once those are fixed.

I don't get why the coveralls test fails. I have not changed the sun light trigger as far as i know :)
Could I please get some assistance here?

@rmkraus Thanks for the review by the way, I agree on all comments

Coveralls is complaining because overall coverage is decreasing by -2.6% (because your code is untested). Since your component interfaces with an external device, you can add squeezebox to the coveragerc file to be excepted from coverage tracking.


docs: resource_deps

Lgtm! except... where ARE the resource docs?

Lgtm! except... where ARE the resource docs?

git add is hard, ok?

what's the status of this pr? is this ready to be merged, or waiting on something?

what's the status of this pr? is this ready to be merged, or waiting on something?

I still need to address Maia's comment and update the caveats now that the flashing green dot bug is fixed.


Fix Community Extensions markdown

GitHub recently made their markdown parser stricter - and error in the table markdown had broken this page.

@justcla Could this PR be accepted?

Done. Thanks guys.

Thanks @justcla!

You're welcome Justin!


"super" is not working inside an async method

// using closure-compiler-v20170910.jar
// java -jar compiler.jar --rewrite_polyfills --language_in=ECMASCRIPT_NEXT --compilation_level=SIMPLE --js=test.js --js_output_file=test.min.js --language_out=ECMASCRIPT5

class AbstractUrlFetcher {
  async fetchUrl() {
    this.sayDone() // This throws. Expected not to throw.
  }
  sayDone() {
    console.log("done")
  }
}

class UrlFetcher extends AbstractUrlFetcher {
  async fetchUrl() {
    await super.fetchUrl()
  }
}

new UrlFetcher().fetchUrl()

The trouble is that async functions are implemented by creating an inner generator function.
Since generator functions cannot be arrow functions it ends up hiding super.
We handle arguments and this by creating aliases for them. We can't quite do that the same way for super, because you cannot treat super itself as a value, but we can create arrow function wrappers as needed like these.
// handles a call to super.foo()
const $jscomp$generator$super$foo$function = (a, b, c) => super.foo(a, b, c);
const $jscomp$generator$super$foo$get = () => super.foo;
const $jscomp$generator$super$foo$set = (v) => (super.foo = v);


Bump Version

Hello! It's possible to update this plugin to support newer versions of typescript?

Oops, it's impossible to update a plugin if the base isn't updated yet. Sorry xD


completed the current task

Checklist
Required changes:

[x] Center the description (Line 17 in HTML file) using a single change in CSS
[x] Fix JS so that when the search is executed, the search engine is not present in query
[x] Add Google Drive, Google Slides and Google Sheets as a provider

Optional changes:

[ ] Add other search providers (GitHub, Spotify, etc)
[ ] Additional CSS styling (as you like)

Good work

Want to see a hosted version of this


Core subset time in gap fix

@huard and @Zeitsperre , I invite you to review this PR on clisops. There are two main components:

Changes in clisops.core.subset#check_start_end_dates decorator:

see: https://github.com/roocs/clisops/commit/96aceeccf2937b2261546503c571c89423e65c8e
there was an issue that picking times between valid time values would cause an error, I have changed this so that the start_date and end_date will be nudged to the adjacent valid value.

I have restructured our clisops.ops.subset#subset function so that it now correctly calls clisops.core.subset functions before working out how to chunk/split datasets:

see: https://github.com/roocs/clisops/commit/19ddf49179080935f065a63e78d717580de63154

All, I'll fix the problems picked up by Travis.

Don't worry about the Windows build. My suggestion for the formatting fix:
conda install -c conda-forge pre_commit
pre-commit install
pre-commit run --all-files

You'll only need to run this once. After that, changes will be automatically applied when you commit changes.

Don't worry about the Windows build. My suggestion for the formatting fix:
conda install -c conda-forge pre_commit
pre-commit install
pre-commit run --all-files

You'll only need to run this once. After that, changes will be automatically applied when you commit changes.

Thanks @Zeitsperre, I've been needing to learn how to do that for a long time..Much appreciated.


Get components for End Effector

Specifically:
2 Potimeters
2 Servos
This is for messing around with End Effector

Everyone got the components *unsure about Andrew


Added mockgoose

Mockgoose uses an in-memory version of mongodb so tests start fresh each time and no database is polluted should certain tests fail.
Edit: one thing ive noticed with Mockgoose is that if your internet is slow (or not available), the first test that uses mockgoose will fail with timeout exceeded. I think this is because mockgoose is checking if the prebuilt mongodb binary is up to date. Not sure this is acceptable for this project but personally I'm okay with it. See https://github.com/Mockgoose/Mockgoose/issues/20

Coverage decreased (-0.3%) to 89.241% when pulling 62c763fc6c54b387959afb5726bb64ad5fd5b709 on Christilut:develop into 8bc61e2c5dec27014f770059630802aa21d6587c on KunalKapadia:develop.

Coverage decreased (-0.3%) to 89.241% when pulling 6ed47ab7cb5d97112f23fc4017ddfdd2b30fe864 on Christilut:develop into 8bc61e2c5dec27014f770059630802aa21d6587c on KunalKapadia:develop.

This looks wrong. Mockgoose says it spins up a mongod and the example shows how to add this to the chai tests. Why do you implement it in the server?


执行pod install 出错：file: cannot load such file rb

[!] Invalid Podfile file: cannot load such file -- ../lego_flutter/.ios/Flutter/podhelper.rb.

cd ../Lego_flutter
flutter pub get


AEP integration with Places

Description

Added logic to dispatch Edge Request Content Event every time a location entry or exit occurs. The dataset id used for the event is the AJO Push Tracking Experience Event Dataset from the AEP Messaging extension configuration. The XDM field group used to populate the Places data is the PlacesContext field group which is already added to this dataset by default.
Related Issue

Motivation and Context

Motivation behind this change is to be able to use the newly added experience events as triggers for AJO Campaigns and Journeys.
How Has This Been Tested?

Added unit and integration tests to cover the changes. Also tested E2E with AJO Campaigns and Journeys to ensure the triggers are working as expected.
Screenshots (if appropriate):
Types of changes

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

Checklist:

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

addressed in a newer pr with different requirements


Streamdeck plus support

Hello!
Really interesting project here. Note that I don't own any streamdeck yet but I'm looking at it in order to manage my multi-room media.
I'm wondering if you support / plan on supporting Streamdeck plus, it seems to differ by two features:

A (touch) screen that can display information
4 rotary encoders that can be turned infinitely in both directions

For the rotary devices I'd be interested to know if i could use it to control my multiroom audio (ie: one rotary encoders per room), or the lights in the room (going up and down in terms of brightness). It might be a bit tricky to implement in order to not send too many signals to home assistant, but that's a problem i've seen solved on different implementation (controllerX receives and process a rotation start and rotation stop events from zigbee devices)
Thoughts?

I see it's duplicate from https://github.com/basnijholt/home-assistant-streamdeck-yaml/issues/94 ; Couldn't find it initially due to the fact that I was searching for "plus" and not +, sorry about that!

Closing because its indeed a duplicate.
When this PR is merged and released we can support the + (https://github.com/abcminiuser/python-elgato-streamdeck/pull/108).


Added a floating point timestamp to GELF formatter

Problems

Previously the GELF formatter was only outputting integer timestamps. This can causes issues in logging aggregation software that's trying to sort by time. Because many log messages can contain the same timestamp, things get displayed out-of-order resulting in an incorrect chain of events. Alternatively the log shipping software could generate its own timestamps, but why not use the actual timestamp if we can?

The timestamp being utilized by the GELF formatter was generated within the format() function instead of utilizing the time in the logging record in the created instance variable. This is not a huge problem, however if we can use a consistent time why not? This is the time field that is utilized by the default formatter when generating timestamps

Enhancements

To be consistent between releases i've kept the integer timestamp in the field timestamp and created a new field named timestamp_f that is the floating point time. I named it timestamp_f because time.time() can have various amounts of accuracy depending on the system so i didn't want to label it timestamp_ms or timestamp_ns and not be able to provide that level of accuracy.

Example:
$ tail -n 1 st2api.gelf.log
==> st2api.gelf.log <==
{"version": "1.1", "level": 6, "timestamp":<PHONE_NUMBER>, "_python": {"name": "kombu.mixins", "process": 15333, "module": "mixins", "funcName": "Consumer", "processName": "MainProcess", "lineno": 231, "filename": "mixins.py"}, "timestamp_f":<PHONE_NUMBER>.887307, "host": "stackstorm2.xxx.xxx", "full_message": "Connected to amqp://guest:**@<IP_ADDRESS>:5672//", "short_message": "Connected to %s"}

I've converted over to utilizing the record.created value for the timestamp so that the time between the st2api.log and any outputted GELF logs are consistent. In the example below notice the same fractional component in the timestamp_f field in the GELF log and the fractional component in the timestamp in the st2api.log:

$  tail -n 1 st2api.gelf.log st2api.log
==> st2api.gelf.log <==
{"version": "1.1", "level": 6, "timestamp":<PHONE_NUMBER>, "_python": {"name": "kombu.mixins", "process": 15333, "module": "mixins", "funcName": "Consumer", "processName": "MainProcess", "lineno": 231, "filename": "mixins.py"}, "timestamp_f":<PHONE_NUMBER>.887307, "host": "stackstorm2.xxx.xxx", "full_message": "Connected to amqp://guest:**@<IP_ADDRESS>:5672//", "short_message": "Connected to %s"}

==> st2api.log <==
2017-08-09 15:04:07,887 54550928 INFO mixins [-] Connected to amqp://guest:**@<IP_ADDRESS>:5672//

Suggestions?
I'm open to further suggestions?
Not sure if the timestamp field to be updated to a floating point value or if they should remain separate for backwards-compatibility?

Thanks, the approach and change itself looks good to me :+1:

Merged - thanks again!