Validation fails when using expressions in GCS Artifact bucket/key (only when in subordinate step/task)

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issue exists when I tested with :latest
[X] I have searched existing issues and could not find a match for this bug
[X] I'd like to contribute the fix myself (see contributing guide)

What happened/what did you expect to happen?
The following workflow fails validation with the following error:

Failed to submit workflow: rpc error: code = InvalidArgument desc = templates.steps-with-steps.steps[0].step templates.steps-with-artifact.steps[0].call-template-with-artifact.arguments.input.from, artifact location, or key is required

This only happens when the steps template that instantiates the input artifact is being called in another template. If one submits the workflow with the entrypoint steps-with-artifact, then validation passes and the workflow runs as expected.
Another way to bypass the validation error is to move the calculation of the expressions for bucket/key one level up, by creating input parameters as shown here:
    - name: steps-with-artifact
      inputs:
        parameters:
          - name: gcs-uri
          - name: bucket
             value: "{{= ( inputs.parameters['gcs-uri'] | split('/') )[2] }}"
          - name: key
             value: "{{= ( inputs.parameters['gcs-uri'] | split('/') )[3:] | join('/') }}"
      steps:
        - - name: call-template-with-artifact
            template: template-with-artifact
            arguments:
              artifacts:
                - name: input
                  gcs:
                    bucket: "{{ inputs.parameters.bucket }}"
                    key: "{{ inputs.parameters.key }}"

Version
v3.5.5
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: test-artifact-expr-
  namespace: argo
spec:
  entrypoint: steps-with-steps
  arguments:
    parameters:
      - name: gcs-uri
        value: gs://bucket/file.txt
  templates:
    - name: steps-with-steps
      inputs:
        parameters:
          - name: gcs-uri
      steps:
        - - name: step
            template: steps-with-artifact
            arguments:
              parameters:
                - name: gcs-uri
                  value: "{{inputs.parameters.gcs-uri}}"

    - name: steps-with-artifact
      inputs:
        parameters:
          - name: gcs-uri
      steps:
        - - name: call-template-with-artifact
            template: template-with-artifact
            arguments:
              artifacts:
                - name: input
                  gcs:
                    bucket: "{{= ( inputs.parameters['gcs-uri'] | split('/') )[2] }}"
                    key: "{{= ( inputs.parameters['gcs-uri'] | split('/') )[3:] | join('/') }}"

    - name: template-with-artifact
      inputs:
        artifacts:
          - name: input
            path: /tmp/input
      script:
        image: alpine:3.7
        command: [sh]
        source: |
          cat /tmp/input

Logs from the workflow controller
N/A

Logs from in your workflow's wait container
N/A

Hi @ezk84
Does this issue occur in the previous version(s)?


add horizontalDrag attr to disable or enable the layer horizontal drag

So the SlidingLayer can include ViewPager.

This will allow you to put a view pager up there, and it wont trigger swipe issues. So I guess I am confused by this. You want to be able to turn that ability on or off? Then just set the onTouchListener again
        vpPager.setOnTouchListener(new View.OnTouchListener() {

            @Override
            public boolean onTouch(final View v, final MotionEvent event) {
                if (event.getAction() == MotionEvent.ACTION_DOWN) {
                    getSlidingLayer().requestDisallowInterceptTouchEvent(true);
                }
                return false;
            }
        });

@elmateo487 I am not sure if I understand it the right way.
My layout looks like:
SlidingLayer

Linearlayout

VerticalSeekBar

Sometimes the SlidingLayer consumes the touch events preventing the onStopTrackingTouch for the SeekBars to work.
Do I need to handle the ACTION_UP event somehow to solve this?

I've just tried tested it with this layout
`<com.wunderlist.slidinglayer.SlidingLayer
android:id="@+id/slidingLayer"
android:layout_width="@dimen/layer_width"
android:layout_height="match_parent">
    <LinearLayout
        android:id="@+id/slidingLayerLinLayout"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:background="@color/Vincent_Dark"
        android:orientation="vertical">

        <TextView
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:text="Test" />

    </LinearLayout>
</com.wunderlist.slidinglayer.SlidingLayer>`

Even if I set the touchlistener to the LinearLayout and the requestDisallowInterceptTouchEvent to the sliding-layer, I am still able to swipe to clase the sliding-layer!?
`mLinLayout.setOnTouchListener(new View.OnTouchListener() {
        @Override
        public boolean onTouch(final View v, final MotionEvent event) {
            if (event.getAction() == MotionEvent.ACTION_DOWN) {
                mSlidingLayer.requestDisallowInterceptTouchEvent(true);
            }
            return false;
        }
    });`


Is it necessary to make the encoder layers equal to decoder layers?

I tried to train a 40-layers-encoder and 6-layers-decoder base model on a 20M-scale dataset, with a learning rate of 5e-4, 4000 decay steps, dropout=0.3 and batch_size=8k. The training runs on 8 GPUs, so the effective batch_size is 64k tokens per update.
But the gradients will go to very large soon and the training is diverged. If I set the lower learning rate, this phenomenon will occur late but still diverged. And if the learning rate is too low, the loss will keep high, and the BLEU is very low. Is there any tricks for you methods?
Any help is appreciate.

Answer to the question: no you don't need to have encoderlayer#=decoderlayer#.
For the diverging phenomenon, considering such a deep model and the large dataset, I would suggest combining a lower learning rate (as you have already tried) with a larger dropout. The effective batch size is also much larger than what we tested, but I'm not sure what kind of effect it has on training (typically Transformers benefit from larger batches, but in our case it might make it more fragile). If you're using fp16 during training, considering training under fp32, as it is much less likely to blow up.
Regarding the problem of low LR-> high loss, please consider combining a moderate learning rate with maybe slightly longer decay period, so the learning rates lingers in the higher value region a bit longer.
Please let me know if the above changes helped.


minor typo JS3 - answer section

highlighted part should say 'mapOfElements'

https://github.com/garageScript/c0d3-app/pull/2218


Completions (zsh, fish): check for python3

Both the zsh and fish completions depend on python3. When the completitions kick in with those shells where no python3 is found it throws errors.
Check python3 is installed, otherwise return no completitions to avoid throwing errors

If you're using fish (or zsh with completions turned on) and the completions kick in and you lack python3, then it throws an error.  eg
[17:43] adavis@carbon ~> fastfetch  fish: Unknown command: python3
/usr/share/fish/vendor_completions.d/fastfetch.fish (line 101):
' | python3 | source
    ^~~~~~^
from sourcing file /usr/share/fish/vendor_completions.d/fastfetch.fish
fastfetch                fastfetch

As far as i can tell no major (if any) distro packages list python3 as a dependency, so it'd be really nice to do something to avoid this.  Whether my PR is the right answer or not is in the eye of the beholder :)

May I ask what os are you using?

Thanks for merging.

May I ask what os are you using?

I noticed this on a new Arch install


Updates Default Fields When Creating a GKE Cluster

Changes
This PR update default fields when creating a GKE Cluster.

Removes --image-type=cos because "Creation of node pools using node images based on Docker container runtimes is not supported in GKE v1.24+ clusters as Dockershim has been removed in Kubernetes v1.24."
Sets default machine type as "e2-standard-4" because it's cost-optimized.

/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] ~Has Tests included if any functionality added or changed~
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/kind documentation

/retest

/retest

/retest

@dibyom thanks for the quick approval! It seems like lgtm is also required for merging the pr

/lgtm

/retest


How to avoid an intellectual property in text output when using BLIP2?

I am using the Salesforce/blip2-opt-2.7b model (on RTX 3070 8Gb) with blip2 for images captioning.
Sometimes, the generated text includes irrelevant or unwarranted intellectual property, such as 'Pineapple wallpaper iphone 6' in response to a simple pineapple image. It also occasionally includes names of TV shows even when they are not related. Moreover, when the image is a generic smartphone, it often generates 'iPhone.'
How can I avoid this issue?  The presence of intellectual property in the generated text is highly undesirable.

我使用 Salesforce/blip2-opt-2.7b 模型（在 RTX 3070 8Gb 上）和 blip2 进行图像字幕。
有时，生成的文本包含不相关或无根据的知识产权，例如响应简单的菠萝图像的“菠萝壁纸 iphone 6”。它偶尔也会包含电视节目的名称，即使它们不相关。此外，当图像是通用智能手机时，它通常会生成“iPhone”。
我怎样才能避免这个问题？生成的文本中存在知识产权是非常不受欢迎的。

Hi, brother, have you solved your problem? I also encountered the same problem. For things that are not in my image, he also generates text descriptions, which is very annoying for me. May I ask how to solve him? Or can we communicate to solve it?
Wishing you a thriving academic career


Forward on trace context to upstream services

When the OTEL_SERVICE_NAME environment variable is provided, we use the Open Telemetry requests library to instrument the requests.Session to ensure trace context is forwarded on to upstream services for functionality like webhooks and uploads.
As we only want the internal sessions used by cog to be instrumented rather than the global session available via request.get etc, we do some awkward monkey patching in the new telemetry.requests_session() method.
Tests have been added to verify the telemetry is added to the correct clients and not the globals.

[!NOTE]
Currently there is no support for configuring any exporters, and no telemetry will be collected for cog. This PR is primarily about ensuring that the trace context is properly propagated.

@nickstenning I'm looking at this and wondering for our needs whether it would be simpler to capture the value of the traceparent header in the prediction & training HTTP handlers and just manually forward that on in the upload and webhook handlers, it would require a little more code but the intent would be clearer and more straightforward than using the otel library.

whether it would be simpler to capture the value of the traceparent header in the prediction

Yeah, looking at this I think you're probably right. It would be traceparent and tracestate headers: see https://www.w3.org/TR/trace-context/.

Closing in favor of https://github.com/replicate/cog/pull/1698


Error on install

On install i obtain this error:
SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'core_modules' already exists (SQL: create table core_modules (idint unsigned not null auto_increment primary key,aliasvarchar(100) not null,installed_versionvarchar(255) null,created_attimestamp null,updated_at timestamp null) default character set utf8mb4 collate utf8mb4_unicode_ci engine = InnoDB)

https://github.com/sgsoft-studio/modules-management/compare/3.1...trippo:patch-1

Hello,
How can you see this issue? Because this table will be created when you run
php artisan cms:install

Simply installing the plugin from admin plugins page


Login not immediate

When you login, does not immediately show manage submissions.

Fixed

Still it be flaky.  It works sometimes but not always especially on sign-out.

Fixed.  Solution needed to be copied to logout.


multiple compiler issues

Hi, I just try to get this running and found multiple issue. I hope I didn't screw up myself.
I have Netbeans 8.1 and downloaded modern-jogl-examples and jglm.
I am a little confused why you sometimes import glm...from the Jglm project and sometimes import framework.jglm.
File ObjectPole.java:  Import is for glm.quat.Quat but the called method mult(Quat) is only defined in framework.jglm.Quat. There's also a call for Quat.cast_() which is not defined in in both …?
Some other issues I had:

Timer.Type.Loop -> LOOP should be all capitals
cannot convert frameworkjglm.Vec3 to glm.vec._3.Vec3
java.awt.event.MouseEvent cannot be converted to com.jogamp.newt.event.MouseEvent

As said I hope I didn’t missed anything big.
Anyway thx for the project I am really eager to try it out.
Rgds
tmg

Hi,
I'll be back on Monday, however tutorial/jglm is a relict, needs to be removed
If you prefer loop capital, feel free to submit the changes :simple_smile:
java.awt.event.MouseEvent is wrong, newt is the right one
Thanks

Hi!
Could you provide instructions how to build and run each tutorial? Only Java, without Kotlin. And Gradle is new for me - is it necessary here? Intellij Idea can not build & run it by yourself?
And this:
Error: Kotlin: Incompatible classes were found in dependencies. Remove them from the classpath or use '-Xskip-metadata-version-check' to suppress errors
Error: Kotlin: Class {class_name} is compiled by a pre-release version of Kotlin and cannot be loaded by this version of the compiler

I prefer to use Java 1.8 - does it problem?


chore(ci): validate-rust should ignore docs

Closed since it breaks required workflows
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-including-paths


Add convenience method for local DOM querySelectorAll, much like $$ for local DOM querySelector

Polymer has a utility method $$ that provides a shorthand for querying individual elements on an element's local dom. This is very helpful and my colleagues and myself enjoy using this utility method. So much so, in fact, that we would like something similar for querySelectorAll against the local DOM.
I have opened up a pull request that adds a $$$ utility method which performs querySelectorAll against an element's local DOM, in a very similar fashion to $$: https://github.com/Polymer/polymer/pull/3186

+1, just had the same idea. beats having to write Polymer.dom(this.root).querySelectorAll(...)

+1

+1

$$ has been removed, as the native API is a lot more intuitive: this.shadowRoot.querySelector Moreover, all other native DOM APIs are available on this.shadowRoot, including querySelectorAll. For more information on the discussion, see #3186. Therefore, I am closing this issue.


⚠ Adacta-fintech.com has degraded performance

In 54586c2, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 1896 ms

Resolved: Adacta-fintech.com performance has improved in d8660fb after 10 minutes.


thumb_size not working

Not able to increase the size of thumb and background track @warkiz

@sakshiigoyal  thumb will be limited in 30dp，and track‘ size will less than thumb‘s.


Problem with adding new indicator

Hey,
I have hard time adding new indicator. I want to add RSI 8, by default it is RSI 14 and I don't know how or is this even possible.
if there is anyone willing to help i would appreciate

Hi @m100jek,
Only RSI14 and RSI7 are supported by the TradingView scanner API. At this time, there is no way to customize the RSI length. However, you can still use RSI7 as an alternative, which is the closest to RSI8. To add RSI7, call the add_indicators() function:
from tradingview_ta import *

handler = TA_Handler(
    screener="crypto",
    interval= Interval.INTERVAL_1_HOUR,
    symbol="ETCUSDT",
    exchange="binance",
)

handler.add_indicators(["RSI7"])

analysis = handler.get_analysis()
print(analysis.indicators["RSI7"])
# Output: 44.93608397

Thank you so much for the answer. U are awsome


Test SW generation with source-maps

v3 has new generation for SW which potentially may have problems with #source-map. Has to be tested.

Works fine.


⚠ SceneTime has degraded performance

In 180943f, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 9849 ms

Resolved: SceneTime performance has improved in 690ebda.


world coordinate system scale is off compared to node-gles3 version

problem: when rendering a scene created in the node-gles version, the ops in webXR are stacked on top of each other. this affects both the position and quaternion

param is found in file systemSettings.js -- but when updating position variables in meshes.js, geometries.js, the meshes are not positioned with the panel positioning


Mocha support

Thanks! :smile:
I just tried it but I'm getting this error:
ReferenceError: QUnit is not defined
    at assets/test-support.js:6284:16
    at mod.state (assets/vendor.js:150:29)
    at tryFinally (assets/vendor.js:30:14)
    at requireModule (assets/vendor.js:148:5)
    at s (assets/test-support.js:1686:142)
    at s (assets/test-support.js:1686:122)
    at requireFrom (assets/vendor.js:121:12)
    at reify (assets/vendor.js:106:22)
    at mod.state (assets/vendor.js:149:17)
    at tryFinally (assets/vendor.js:30:14)

No worries - will double check that issue

Oh wait testem.json isn't automatically updated by ember-cli-mocha. Wouldn't it be better to look for that package in package.json?

After I set the testem.json framework to mocha it worked! Nice work.

Cool - am happy to change the framework detection to checking for ember-cli-mocha in the package.json if you think that would be better.

It's a bit more fool proof.

Yeah, make sense - will update

Awesome thank you very much!

no problem :)

pushed a new version to npm

Thanks


Come up with plan for keeping in sync with nf-core

Problem
The nf-core tools and pipeline templates will periodically be updated and require synchronization. The overall sync process is controlled via the nf-core sync command, but relies heavily on git branches (the TEMPLATE branch) and manual work to resolve merge conflicts. This is described in details here: https://nf-co.re/docs/contributing/sync
Not keeping pipelines in sync might lead to test/lint failures in a pipeline.
Solution 1: Keep our pipelines in sync with nf-core
This solution involves spending active effort to keep our pipelines in sync with nf-core. Mainly, by going through https://nf-co.re/docs/contributing/sync whenever there's a new release.
Advantages

We get latest updates for files by nf-core team. These could include fixes or additional features, or ways of running pipelines that keep us up-to-date.
Our tests in GitHub actions continue to work after syncing

Disadvantages

Syncing is a complicated process that may involve manual review of changes to many files and fixing merge conflicts in git
Syncing might make modifications to files in our pipelines that break integration with IRIDA Next (this would have to be part of review).

Solution 2: Make only minimal changes necessary to keep GitHub actions tests working
In this solution, we only make the minimal changes necessary to keep our test suite working in GitHub. The pipeline code itself should largely remain unchanged.
Advantages

Less complex process to update
Aren't changing pipeline code as much, hence less concern about introducing errors or breaking integration with IRIDA Next.

Disadvantages

Don't keep in-sync with latest changes by nf-core team
Trying to synchronize later may be a much larger task

Decision
The current decided solution will be in between Solution 1 (keep pipelines in sync) and Solution 2 (minimal changes).

We will make an effort to keep the iridanextexample pipeline in-sync with nf-core as new changes are made.

This pipeline is used to test out integration with IRIDA Next but is not used for data analysis, so provides an opportunity to test out any changes from nf-core prior to incorporation into other pipelines.
We are using this pipeline as a template for developing other pipelines, and so it's more useful to keep in sync.

For other pipelines we develop, we will only look into synchronizing with nf-core when making a new major release.

Major releases will require testing integration with IRIDA Next anyways.
These will be less frequent and so less work for us to manage.
We could try and re-use work done in keeping iridanextexample in sync with nf-core to synchronize code for other pipelines.


sleigh -x : unknown option: -x

Describe the bug
Can't specify -x to get debug information. Flag is listed in the Usage info but not accepted ?
To Reproduce
$ /opt/ghidra/Ghidra/Features/Decompiler/os/linux_x86_64/sleigh

USAGE: sleigh [-x] [-dNAME=VALUE] inputfile [outputfile]
   -a              scan for all slaspec files recursively where inputfile is a directory
   -x              turns on parser debugging
   -u              print warnings for unnecessary pcode instructions
   -l              report pattern conflicts
   -n              print warnings for all NOP constructors
   -t              print warnings for dead temporaries
   -e              enforce use of 'local' keyword for temporaries
   -c              print warnings for all constructors with colliding operands
   -o              print warnings for temporaries which are too large
   -s              treat register names as case sensitive
   -DNAME=VALUE    defines a preprocessor macro NAME with value VALUE

$ /opt/ghidra/Ghidra/Features/Decompiler/os/linux_x86_64/sleigh -x
Unknown option: -x

Steps to reproduce the behavior:

run "sleigh -x" from CLI

Environment (please complete the following information):

OS: Arch
Java Version: 19
Ghidra Version: 10.2.2
Ghidra Origin: distro

 $ find /opt/ghidra/ | grep "sleigh"
/opt/ghidra/Ghidra/Features/Decompiler/os/linux_x86_64/sleigh
...
...
/opt/ghidra/support/sleigh

Ah yes... I had only tried the first one. Never noticed the other until now (it's not in $PATH by default).
Sorry for the noise.

Was gonna ask if arch put specific things on PATH or not.
There's those c versions and this java version, I've only ever used the Java version.  I imagine the c is just compatibility checks for decompiler code maybe.


Migrate to Manifest V3

More details: https://developer.chrome.com/docs/extensions/mv3/intro/mv3-migration/
Checklist: https://developer.chrome.com/docs/extensions/mv3/mv3-migration-checklist/

Chrome version is using manifest v3 now. Firefox do not support it yet, so I'm closing this issue. I'll open another issue when Mozilla announce more information about manifest v3.


Add signal support to Godot Android plugin

Supports registering and emitting signal from a Godot Android plugin.
Fixes godotengine/godot-proposals#537
Sample project
3.2 backport

cc @DrMoriarty

Needs a rebase, otherwise seems good to me.

Needs a rebase, otherwise seems good to me.

@akien-mga Done!
The 3.2 backport (needed because of the new Callable class in master) is also ready to go.

Thanks!


Bag bar disappearing when addon used in tandem with Bartender

Addon Version
0.1.7
World of Warcraft Version
Retail
Describe the bug
When betterbags is used in combination with bartender, the blizzard bag bar is hidden by default, despite its visibility being enabled in both betterbags and bartender.
Bartender4-4.14.14
Steps to reproduce
Presents itself by default.
To temporarily resolve, one must toggle the bag bar visibility off and on again in bartender to resolve the issue. Toggling off and on in betterbags has no effect. This fix is not persistent needs to be re-done any time the UI reloads itself.

Better bags also doesn't follow Bartenders anchors currently - would be a nice to follow that as it currently can shift the bag position.

relevant screenshot:
https://discord.com/channels/1063213796845428876/1063213797633966122/1266917791429169245


Acceptance of short command line options

Just a wish: Currently command line options need to be explicitly given (e.g. -out, -overwrite). It would be convenient to accept the short (one letter, or more) versions of these as well.
If this is considered 'overwrite' should be renamed to something else, like 'force'

I think a bigger cleanup is required here. Long options should use '--' as a prefix. Short options should use '-'.

If you're talking about long options where you don't write out the full name, I know that Python supports this in argparse. I'm not sure what to think about it. I always leave it on, but I read somewhere that this might not be the best practice, because it could break scripts when adding new options.

I was not aware of that. I was exactly thinking about argparse here. Do you plan to rewrite the scripts in python?

I was thinking of it, yes. It has to be Python 2.6, to be able to work on a wide variety of OSes.


Swagger errors for TapiTopology.swagger

Swagger Error
Cannot have multiple operations with the same operationId: retrieveContext_topology_node_transferCostCostCharacteristicCostCharacteristic
Jump to line 1008
Details
Object
Swagger Error
Cannot have multiple operations with the same operationId: retrieveContext_topology_node_transferTimingLatencyCharacteristicLatencyCharacteristic
Jump to line 1129
Details
Object
Swagger Error
Cannot have multiple operations with the same operationId: retrieveContext_topology_link_transferCostCostCharacteristicCostCharacteristic
Jump to line 1777
Details
Object
Swagger Error
Cannot have multiple operations with the same operationId: retrieveContext_topology_link_transferTimingLatencyCharacteristicLatencyCharacteristic
Jump to line 1898
Details
Object
Swagger Error
Cannot have multiple operations with the same operationId: retrieveContext_topology_link_validationValidationMechanismValidationMechanism
Jump to line 2084
Details
Object

Fixed


Need UWP infrastructure for flutter/plugins

I'm starting on bringing up UWP support in flutter/plugins, and need the infra brought online for that:

A recipe that looks like the existing Windows plugin recipe except:

It runs --enable-windows-uwp-desktop instead of --enable-windows-desktop
It passes --winuwp instead of --windows to script commands

Ideally it should be a separate stand-alone recipe so that we can see UWP failures separately from Win32 failures, but if we have to combine them that would be livable.
(I'll have a test PR available shortly to try running the new recipe against to see if it's working.)

I think this can be done with changes to the existing recipe and a property.

@stuartmorgan The recipe change is merged, please let me know if anything else if needed. To run against the new recipes, please inject a new property for a new uwp builder in flutter/infra side: uwp: true.


Revert pinned nightly

Checking https://github.com/tensorflow/tensorflow/commit/f6a40276a1eaa472b25e87a6371310bc80e3fb56 fix

Pending #663

@tensorflow/sig-addons-maintainers ...ready for merge :D


Change The widget name to child and add decoration: BoxDecoration

Change The widget name to child and add decoration: BoxDecoration

Change The widget name to child and add decoration: BoxDecoration

@MdAshrafUllah thank you for the suggestion, will do and update.


Video List Page

Look into error handling for this page.  Currently giving a 500 error if user doesn't visit the create stream page first.  Possibly giving an error when gun db looks for data.

#53 resolves when user is navigating to the page, but issue is still there if user refreshes the page.  The error is when using the gun db


[Build Microservices with Spring Data, Cassandra, and Stargate.io] Razvan Diaconu

Name: Razvan Diaconu
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/razvan-diaconu-240b0566/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

Hey Razvan Diaconu, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/_2-vretZTMqP1tkDt4U_zQ


"null" at negative values

Once a value change to negative the adapter don't work anymore.
I would like to record the overall perfomance of my heating, so I have one value which is able to change between +15000W to -5000W, but everytime it change below "0" it shows "null".

thank you need to check this one out missed it during my research of other items

I tried to reproduce this without luck, can you give me a little more background how and when this occurs ?

fixed 0.4.7-alpha.0 https://github.com/iobroker-community-adapters/ioBroker.sourceanalytix/commit/a47eb48394e58548ae58a9e9d75cea6401d039d1


[Feature] Poster for audio

Can we have a poster for audio tag?

No, but you can have a poster for a <video> tag with an audio file: http://jsfiddle.net/mm2qdevb/

Yes, that's how I'm doing it but wanted to have the same behaviour for audio and video tag delivered by mediaelement

@LukaszGrela What's the behavior you are trying to get exactly by using that approach with MediaElement?

To have poster for audio file, and not need to convert audio into video with static image:)

Ok I'll check this and keep you posted on any progress.

@LukaszGrela Please check 3.x-dev branch; there's a test file for HTML5 that solves your issue. Let me know if it works for you

hey this its  theodore burgess would like to have a chat  in regards
to u all getting out of my pc
On 9/19/16, Rafael Miranda<EMAIL_ADDRESS>wrote:

@LukaszGrela Please check 3.x-dev branch; there's a test file for HTML5 that
solves your issue. Let me know if it works for you
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/johndyer/mediaelement/issues/1422#issuecomment-248150452

Closing this issue since no answer has been posted in more than 2 weeks. If you'd like to reopen this, let us know. Thanks!

@ron666 I'm wanting to do the same thing, have a poster image for the audio tag (without having to convert an audio file to video)...I downloaded the 3.x-dev branch, but am not seeing where there is sample code that is using a poster image with the audio player. Can you point me in the right direction?

Check the test/html5.html file. There should be an example of audio. Ring played with video tag there

Ah, got it. Thanks!


We should also test doc values

Doc values are an important part of most search applications, and we're not testing them today.

That would be extremely interesting. This is one of the piece where tantivy diverged quite drastically from lucene too.
Our codec is similar but is tweaked for random access, and we do random access on it. I would love ot see some comparison.

The SIMD stuff discussed in the other ticket only kicks in for range queries.

OK got it.

That would be extremely interesting. This is one of the piece where tantivy diverged quite drastically from lucene too.
Our codec is similar but is tweaked for random access, and we do random access on it. I would love ot see some comparison.

Ahh that is interesting!  Lucene used to be (long ago) random accessed, and then moved to iterator only, which was problematic e.g. for KNN search.
Hmm, for the default BM25 rank, is Tantivy using any doc values, e.g. maybe field or doc length (norms)?  Lucene does store norms as doc values, so we are testing that in the search benchmark now.  I'll open a spinoff issue to also test non-relevance sort (though that, too, will likely be another doc values field).

Yes... But we did not have enough time to do proper research on whether this was a good idea or not, so I am super interested in whatever benchmark you can come up with.
If you want to experiment with random access docvalues, I am happy to share details about our codec. I really need to write a blog post about that.

Hmm, for the default BM25 rank, is Tantivy using any doc values, e.g. maybe field or doc length (norms)? Lucene does store norms as doc values, so we are testing that in the search benchmark now. I'll open a spinoff issue to also test non-relevance sort (though that, too, will likely be another doc values field).

In tantivy, Fieldnorms are stored over a single byte, using the same "scale" as Lucene.
The fieldnorms are not exactly docvalues. They will always take one byte per doc, even if the field is absent from the document. In that case, it will get the fieldnorm 0.
We have a way to index stuff in a schemaless manner -mostly useful in quickwit-, but in that case, you don't have any fieldnorms.


Fixed link and improved Reader role naming

Fixed link to enabling enhanced security features and improved Reader role naming, as there is no such "Global Reader" role in Azure Resource Manager.

#sign-off

@helderpinto the intention of 'Global reader' here was the Reader permissions on the full scope. So for access to all compliance info in the tenant, the user would need Reader permissions for the whole tenant. (of for all relevant subscriptions... whatever that scope might be)
I will be further updating the document to reflect this


Cannot read properties of undefined (reading 'canvas')

Describe the bug
A clear and concise description of what the bug is. If you intend to submit a PR for this issue, tell us in the description. Thanks!
Hi! Fist of all, What a great library! I'm playing with it, and so far... I LOVE IT! now... to the error, I am currently trying to use the post-processing package on a scene and I get the following error when I use it:

Reproduction
This same happens whenever you check the examples:
https://playground.tresjs.org/experiments/nuxt-stones

Steps
Steps to reproduce the behavior:

Just adding the EffectComposer with a default bloom effect.

Expected behavior
For the effect composer component to work
Screenshots
If applicable, add screenshots to help explain your problem.
System Info
Output of npx envinfo --system --npmPackages '{vite,@tresjs/*, three, vue}' --binaries --browsers
System:
    OS: macOS 13.4.1
    CPU: (8) arm64 Apple M1
    Memory: 323.22 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.15.0 - ~/.nvm/versions/node/v18.15.0/bin/node
    npm: 9.5.0 - ~/.nvm/versions/node/v18.15.0/bin/npm
  Browsers:
    Chrome: 116.0.5845.110
    Safari: 16.5.2
  npmPackages:
    @tresjs/cientos: ^3.1.0 => 3.1.0
    @tresjs/core: ^3.1.0 => 3.1.0
    @tresjs/post-processing: ^0.3.0 => 0.3.0
    vite: ^2.8.0 => 2.9.16

Additional context
Add any other context about the problem here.

I can't reproduce the bug. The playground example does not report any errors in the console. Let me know if I missed something.

Got fixed with a version bump! Thank you for your time!


More Code indentation issues

especially when typing in Input example from docs

Just tested this. After typing in entire Input example, I couldn't reproduce the issue... 🐺


[Feature Request] How to set the el-tree's row height?

Existing Component
是
Component Name
el-tree
Description
How to set the el-tree 's row height ? I want to add a button to the el-tree's every row, and the button's type="success", the button's size="mini", but the el-tree's  is to narrow to contain the button
thank u

{{ node.label }}
<el-button type="text" size="mini" @click="() => remove(node, data)">
Delete

B
T

override style .el-tree-node__content, please.


[SPARK-14025][STREAMING][WEBUI] Fix streaming job descriptions on the event line

What changes were proposed in this pull request?
Removed the extra ... for each streaming job's description on the event line.
[Before]

[After]

How was this patch tested?
test suits, manual checks (see screenshots above)

Can one of the admins verify this patch?

@andrewor14 @zsxwing would you mind taking a look at this when you have time? Thanks!

I agree it's a problem, but this seems like a hacky way to band-aid it, with a second version of the description and a flag passed around. Is the problem not just that this is HTML, but not being interpreted as such?
Is it necessary that this description have HTML to begin with?

@srowen thanks for looking at this!
I believe job descriptions were intended to contains only plain texts at first, but HTMLs were introduced in for streaming jobs by https://github.com/apache/spark/pull/8791. Hyperlinks in the CompletedJobs/CompletedStages tables are proven quite useful, so I think may we'll need one more version of the job description? Any thoughts on how to make this elegant would be appreciated :)

Besides, the blue/green bar in the event line itself is a clickable, linking to the specific job page. The  thing is superfluous, let's figure out how to remove it.

Ah I thought this was a tooltip, where HTML can't render, but it's not. I wonder, is the problem just that this is rendered as text and not HTML? if it's controlled by the UI, it seems safe to just let it render as HTML if possible?

Actually we've intentionally escaped the description for the event line, so that it will be rendered as plain texts; please see https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala#L85.
So I think it was TD's intention to add hyperlinks to the text tables(where descriptions are not escaped), but he hadn't noticed that the same description would be escaped and used in the tooltip too.

Maybe let's summarize a little bit:

job/stage descriptions are used at 2 places: the Event Timeline and the text tables (CompletedJobs/CompletedStages)
job/stage descriptions were intended to contains only texts at first, so everything's well
https://github.com/apache/spark/pull/8791 introduced HTML in job/stage description for streaming jobs
the text tables works well, because it renders HTML links directly, without escaping the description
the event time line escapes the description intentionally, and then displays the description, leading to the plain  thing

My question is, why escape it? the library will accept and render HTML and that is desirable here right?

Ah, I guess the reason is:

for the blue box in the event timeline, itself is clickable both for streaming/non-streaming jobs, so it's unnecessary to contain any HTML like
for the black tooltip, it is designed to contain only plain texts. The tooltip would just move with the cursor, so even it does contain a link, no one can manage to click on it.

The original tooltip library does accept and render HTML, but I guess we Spark developers decide we only need plain texts here (see the above 2 points), so we'll escape things to prevent from malicious codes. However then HTMLs() slipped in along with PR#8791 :)

Maybe @andrewor14 will explain better :)

ok to test

Test build #53688 has started for PR 11845 at commit 0f32845.

Test build #53688 has finished for PR 11845 at commit 0f32845.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53688/
Test FAILed.

@lw-lin thanks for summarizing the issue. Yes, we don't want to process arbitrary HTML so we don't have to worry about potential XSS threats. However, I agree with @srowen that the approach you chose here seems kind of arbitrary. Instead, a better place to do this might be in UIUtils.makeDescription. There we already parse the description for anchor tags, so we can just pass in an additional flag there to optionally remove the tags.

@andrewor14 thank you for the informative review. Will soon update this PR accordingly.

Test build #53755 has started for PR 11845 at commit 22ac2d9.

Test build #53755 has finished for PR 11845 at commit 22ac2d9.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53755/
Test FAILed.

Merged build finished. Test FAILed.

Updated with new commits.
Tests passed on my local machine; I wonder why scalastyle checks failed (this PR doesn't touch this code snippet):

Scalastyle checks failed at following occurrences:
[error] /home/jenkins/workspace/SparkPullRequestBuilder/core/src/test/scala/org/apache/spark/ui/UIUtilsSuite.scala:64:0: Whitespace at end of line
[error] (core/test:scalastyle) errors exist

It failed on a line you modified so it is due to this change. The error message explains what you need to change.

Test build #53770 has started for PR 11845 at commit ebe1182.

Test build #53770 has finished for PR 11845 at commit ebe1182.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/53770/
Test PASSed.

Looks pretty good. @zsxwing can you have a look?

Test build #53869 has started for PR 11845 at commit 0761e5f.

Test build #53869 has finished for PR 11845 at commit 0761e5f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/53869/
Test PASSed.

LGTM. Merging to master. Thanks, @lw-lin

Thank you all for your kind review & comments, @srowen @andrewor14 @zsxwing !


Discord Notification Support

As the title says, apprise supports discord, but the notifications are specifically made for telegram, so they are broken on discord.

Yes, config file example for plugin gameshub.official.notification.apprise is for telegram. If you want to use apprise to notify discord, you should change the config.json5, especially the notifications->body part.


Run the python-tests against multiple OS versions

Refactor the CIs to run against multiple OS versions. Enables active ubuntu versions and debian.
This demonstrates the issue described in #26 (which is fixed by #29).

FYI - It is expected that ubuntu:23:04 and debian:unstable fail due to #26.

@antonblanchard - I believe this pull request is ready to review.

Thanks @mithro !


Odd location of iota

Why is iota hidden in <flux/source/*>? Other views are in <flux/op/*>.

The types in the source directory are "sequence factories" -- things that (only) go at the start of a Flux pipeline, that is, the "source" of some data. iota is one of those, along with things like from_range, single, empty, istream etc.
The op directory is for adaptors (things that go in the middle of a pipeline and return a lazily evaluated sequence, like filter, map or chunk) and for algorithms (things that go at the end of a pipeline and either return a value or modify a sequence in-place, like sum, fold or sort).
Admittedly this distinction is fairly arbitrary, but hopefully that makes sense!


GAX replayer/tracker checklist

I have gotten most of the stuff done on this. Right now there is a replayer engine that aims to closely mimic GAX's functionality, and an editor/tracker for the .gax files. These two are not released yet, as there are still glaring issues that need to be worked on:

[x] Figure out accurate pitch slide calculation (Most noticeable in Sigma Star Saga)
[x] Implement tone portamento
[ ] Figure out accurate tone portamento calculation (Most noticeable in The SpongeBob SquarePants Movie, SpongeBob Lights Camera Pants, etc.)
[x] Fix bug in which tone portamento cannot go downwards
[x] Figure out note pitch tuning problems (this affects Drake & Josh, Unfabulous, etc.)
[ ] Implement note delay
[x] Implement vibrato
[ ] Figure out vibrato depth calculation problems (it's slightly imperfect)
[x] Fix the volume envelope sustain bug
[ ] Fix a few packing/unpacking bugs

Recently just implemented note delay. It is a bit of a hack (at least for me), because it has a channel object's ticking rely on not only the replayer, but the GAX object's instrument set as well, and I usually like things more self-contained, but it seems to mostly work so far (there are a few edge cases regarding Rayman Raving Rabbids, though).


NR-102794 MERN stack meta details

Observed that MERN stack is live on the below URL
URL: https://newrelic.com/instant-observability/mern-stack
Hence, I am creating this PR to assign meta title and description to the mern stack quickstart
Ticket: https://issues.newrelic.com/browse/NR-102794

closing this PR as the ticket https://issues.newrelic.com/browse/NR-102794 was closed


Path fix and lodash babel dev

Two small changes:
First, a fight to the death between path.join and path.resolve (per #2378). I think path.join wins. Unless it creates another issue I didn't think of?
Second, I added lodash's official babel-plugin-lodash to production plugins in Babel. It replaces import { get } from 'lodash'; with import get from 'lodash/get';. I know we already use the latter by default but in case a user forgets to do the same, it saves them from adding an additional 70k to their bundle.

Coverage remained the same at 100.0% when pulling 1558bcc516c4f2469bb977548c066eeb4437cfbb on path-fix-and-lodash-babel-dev into ff0efa1c2faf8d7d742f27102c73fd5420ead259 on dev.

Coverage remained the same at 100.0% when pulling 1558bcc516c4f2469bb977548c066eeb4437cfbb on path-fix-and-lodash-babel-dev into ff0efa1c2faf8d7d742f27102c73fd5420ead259 on dev.

@gretzky i think we can keep the full lodash toolchain available, since we are using module imports eg import isEqual from 'lodash/isEqual' webpack tree-shaking excludes any unused modules from the final build... so, even with current setup the output bundles would be the same as if we were installing the individual modules from npm, which may not be the most ergonomic solution imo. while i'm also reticent to add anymore plugins, i think in this case it's probably ok since it's just a development dependency
@julienben if we are adding lodash babel transform, let's update existing lodash module imports to named exports, eg.
import conformsTo from 'lodash/conformsTo';
import isFunction from 'lodash/isFunction';
import isObject from 'lodash/isObject';

can be
import { conformsTo, isFunction, isObject } from 'lodash';

since the plugin will transform this to former module import format anyway :)

@justingreenberg makes sense, I'm only asking because we could significantly reduce the size of modules by removing the full lodash toolchain and just adding the modules we need-- lodash is ~530k (64k gzipped) and all of the modules we currently use here combine to 181k (8k gzipped).

@gretzky totally agree regarding DX. unfortunately devs using RBP will be still have to install the full lodash package since it's part of the extended graph (lodash is also a dependency of our other deps like webpack, babel, enzyme, plop etc.)

The way we imported lodash modules until know (import xxx from 'lodash/xxx') already ensured that we benefit from tree-shaking. This was only meant as a protection for users who forget to import this way. But now (per @justingreenberg 	) it allows us to make our imports look cleaner and make sure we always benefit from tree-shaking, no matter how we import.
The plugin itself is only used in production so there's no hit on dev performance. (We have some leeway in dev anyway since we just removed a whole bunch of plugins 😅)
My reasoning was based on this article where he shows how to reduce the lodash part of a bundle. I didn't add lodash-webpack-plugin as it has a few gotchas and should be a per-project decision. I also tried lodash-es but that came with its own set of issues.
@gretzky I'll wait for you to let me know if you're still on the fence about this or if it makes sense to you before I merge.

Yeah makes sense to me overall, I'll merge


Add x86 Windows 32bit build variant on AppVeyor

platform:
  - x64
  - x86
install:
  - ps: Install-Product node $env:nodejs_version {platform}
[...]
  - node_modules\.bin\grunt package-win-{platform}
[...]
artifacts:
  - path: dist\juice-shop-*_node4_win_{platform}.zip

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/36241525-add-x86-windows-32bit-build-variant-on-appveyor?utm_campaign=plugin&utm_content=tracker%2F6283055&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F6283055&utm_medium=issues&utm_source=github).

No x86 but versions for node.js 4/5/6 added.


Partial caching with t strings

Hello
If I have t strings (<%t Buttons.SeeDetails 'See details' %>) in a partial cached block of markup and switch the language, the t strings dont get updated.
Any ideas on how to make that happen? Or how to get partial cached content to update on language change?

did you flush after you updated the locale in your /lang/xy.yml file?
Are the strings updated outside the cached block?
I encountered strange problems with umlauts in language files and wrong file encoding, then the yml parser ignored the whole module's lang files...
Though this issue doesn't seem to be fluent related...

Yes I did flush and the strings are updated outside the cached block.
This issue might indeed not be Fluent related, but I figured I'd give it a shot

So you need to invalidate the cached block using a better cache key? See https://docs.silverstripe.org/en/3.1/developer_guides/performance/partial_caching/
PS: you might also close this issue or move this discussion to another place like forum, IRC, or stackoverflow.

Alright, thanks a lot!

For completeness: It's also mentioned in the fluent docs: https://github.com/tractorcow/silverstripe-fluent/blob/master/docs/en/templating.md

Ah I see. Sorry for the inconvenience!

It's all good. :) I don't remember writing those docs!


Added the LockState property to SiteEntity

Exposed the site lock state information through the LockState property on SiteEntity

Hi @waldekmastykarz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Bug report xxx

Contribution guidelines

[X] I've read the contribution guidelines and wholeheartedly agree

I've found a bug and checked that ...

[X] ... the problem doesn't occur with the server or web themes
[X] ... the documentation does not mention anything about my problem
[X] ... there are no open or closed issues that are related to my problem

Description
Cannot init RedisConnection cannot init redisconnection The account password is correctly configured
Expected behaviour
You can use the Redis data source
Actual behaviour
Not working
Steps to reproduce
Add the Redis connection source configuration
Package versions

Java: 1.8
Node: 16
Npm: 7
Server: <IP_ADDRESS>21015

Configuration
None

System information

Operating system: Linux
Browser: Chrome

Message that will be displayed on users' first issue


Plugin isn't working after latest update

I updated to Sketch version 39.1 (31720) and the plugin doesn't seem to be working any longer. Does it still work for others?

Please load the latest version of the plugin (0.7)... some change in Sketch broke my older versions of the plugin but the latest should work fine.


Load a UIViewController instead of an image when animation is done

I want to load a UIViewController instead of an image when animation is done. How can I do that?

It's already loading UIViewController. It has UIView and it has imageview as child. Lookup in the view hierarchy.


Dev

rc <- dev

Codecov Report
Attention: 177 lines in your changes are missing coverage. Please review.

Comparison is base (1f932d9) 99.36% compared to head (e56b895) 88.30%.

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

Files
Patch %
Lines

R/combineTreatments.R
0.00%
162 Missing :warning:

R/collapseTreatmentEra.R
0.00%
15 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##               rc     #210       +/-   ##
===========================================
- Coverage   99.36%   88.30%   -11.06%
===========================================
  Files          13       15        +2
  Lines        1414     1591      +177
===========================================
  Hits         1405     1405
- Misses          9      186      +177

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fix: inventory csv pod container creating (#535)

fix: inventory.csv creation when it is commented out

fix: edit the default inventory line

Description
When no poller.inventory is specified, there is a problem with an infinite ContainerCreating state.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Bug fix

How Has This Been Tested?
Integration and manual tests
Checklist

[x] My commit message is conventional
[x] I have run pre-commit on all files before creating the PR
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

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


Change interfaces default policy to load all interfaces

with the option  of disabling interfaces by configuration.

ping @fabdrol


"Scoroșești", "Valea Fântânei" and "Valea Ștefanului" are listed in the drop-down menu

Description:
"Scoroșești", "Valea Fântânei" and "Valea Ștefanului" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Scoroșești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Scoroșești, BUZĂU (Odăile)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Valea Fântânei" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Fântânei, BUZĂU (Odăile)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Valea Ștefanului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Ștefanului, BUZĂU (Odăile)".


Project blocked

Hi There,
I have published multiple projects on TechNet Gallery related to Azure and SCOM but they all has been blocked by admin now.
Kindly help me with the same.
Thanks,
Gourav Kumar
https://social.technet.microsoft.com/profile/gouravin/

Hi There,
Do we have any update on it.
Thanks,
Gourav Kumar
From: welcome[bot]<EMAIL_ADDRESS>Sent: 27 June 2019 03:13 PM
To: MicrosoftDocs/feedback<EMAIL_ADDRESS>Cc: Gourav Kumar<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/feedback] Project blocked (#1666)
Thank you for creating the issue! One of our team members will get back to you shortly with additional information. If this is a product issue, please close this and contact the particular product's support instead (see https://support.microsoft.com/allproducts for the list of support websites).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/feedback/issues/1666?email_source=notifications&email_token=AIA6KI432KU3A5N47STWJODP4SDSPA5CNFSM4H32EXZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYWSJUQ#issuecomment-506275026, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIA6KI635WIY2M6DUHSO5PLP4SDSPANCNFSM4H32EXZA.

@dend Can you please take a look at this issue?

Thank you for the feedback @unpredictablegourav - this likely has something to do with you not having enough reputation in the TechNet Gallery. We recommend you submit your code contributions to one of the official Microsoft code sample repositories for a more collaborative workflow.


deinit not called when using this library?

I am running into performance issues in my app and I believe they are related to this library, or at least how I am using it.
I noticed that deinit is never called on the controller that implements Player!
I have tried a lot of methods for getting it to fire and nothing seems to work:
dispatch_async(dispatch_get_main_queue()) {             if self.player != nil{                 self.player!.pause()                 self.player!.stop()                 self.player!.view.removeFromSuperview()                 self.player!.delegate = nil                 self.player = nil             }             self.dismissViewControllerAnimated(true, completion: { _ in })
this helps with memory but does not completely remove the viewController. any ideas?

hey @uknowmeright thanks for the report. that's interesting, i wonder if some object is registered for KVO and is just being held.

@uknowmeright if you're using this as a child view controller, you'll have to remove it from the VC hierarchy too. I updated the sample project as an example:
commit 497258fe6fedd38542615fb49eeaad8121384812
Author: patrick piemonte<EMAIL_ADDRESS>Date:   Thu Dec 15 19:29:29 2016 -0800
update sample project

hmm. This seems to help a bit. My memory keeps rising each time I reload this view controller and deinit never gets called. Cant help but think there is some kind of memory leak. i will keep investigating. Thanks!

I have a similar problem. After dismissing the VC that contains the player, the sound is still playing.
override func dismiss(animated flag: Bool, completion: (() -> Void)? = nil) {
    self.player.stop()
    super.dismiss(animated: flag, completion: completion)
}

📎 https://github.com/piemonte/Player/issues/186


Alkimi bid adapter: added features

Thanks for improving the documentation 😃
Please give a short description and check the matching checkboxes to help us review this as quick as possible.
🏷 Type of documentation

[ ] new bid adapter
[ ] update bid adapter
[*] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[*] Related pull requests in prebid.js or server are linked
[ ] For new adapters check submitting your adapter docs

@muuki88 Hello. Do we need to do any changes in scope of this pull request?

Nope. Sorry, I missed the email notification.


add 'onBlur' prop

Another PR (#40) introduced an onFocus prop, which was cool, but it would be helpful if we could handle blur too!

I could definitely use this too!


Support extractor for hanime.tv

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running yt-dlp version 2022.05.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and am willing to share it if required

Region
India
Example URLs

Single Video: https://hanime.tv/videos/hentai/kanojo-x-kanojo-x-kanojo-1
Single Video: https://hanime.tv/videos/hentai/kanojo-x-kanojo-x-kanojo-3
Single Video: https://hanime.tv/videos/hentai/fuzzy-lips-2
Playlist: https://hanime.tv/playlists/duuilob7bmyefj5kikbx
Playlist: https://hanime.tv/videos/hentai/first-love-1?playlist_id=duuilob7bmyefj5kikbx

Description
hanime.tv is a large video publishing platform catering to Doujinshi consumers. It helps reducing testosterone or estrogen levels of adolescent or adult people. Therefore I conclude that the aforementioned feature will be helpful for many.
For the implementer: https://github.com/rxqv/htv/tree/master/hanimetv
Verbose log
[debug] Command-line config: ['-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8 (No ANSI), error utf-8 (No ANSI), screen utf-8 (No ANSI)
[debug] yt-dlp version 2022.05.18 [b14d52355]
[debug] Python version 3.10.4 (CPython 64bit) - Linux-5.18.1-zen1-1-zen-x86_64-with-glibc2.35
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: ffmpeg -bsfs
[debug] exe versions: ffmpeg 5.0 (setts), ffprobe 5.0
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2<IP_ADDRESS>, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
Latest version: 2022.05.18, Current version: 2022.05.18
yt-dlp is up to date (2022.05.18)

Duplicate of https://github.com/yt-dlp/yt-dlp/pull/1312

That's unfortunate. But don't all the pornographic sites (which it has support for) infringe copyright?


[FEATURE]: native CASE WHEN support

Using case in queries can be a really nice way to restructure data, or omit things entirely if unset. https://www.postgresql.org/docs/current/functions-conditional.html
It's also very useful in orderBy queries, such as like this:
sql`
	case
		when ${table.scope} = 'a' 	then 1
		when ${table.scope} = 'b' 	then 2
		when ${table.scope} = 'c' 	then 3
		when ${table.scope} = 'd' 	then 4
	end
`,

Or when querying from multiple tables but wanting to conditionally change where data is coming from:
sql`
	case
		when ${table.type} = 'a' then ${someTable.name}
		when ${table.type} = 'b' then ${otherTable.title}
	end
`,

This is fine, but could be more ergonomic I think with a native helper.
Psuedo-example without much thought of what this could look like:
case(
	eq(table.type, 'a'), someTable.name,
	eq(table.type, 'b'), otherTable.title,
)

I also need this when treating INTEGER as enum in SQLite

This is an implementation I've just drafted but couldn't test yet. Feel free to give feedback.
function nehw<T>(condition: SQL<unknown>, value: T): SQL<T> {
    return sql`when ${condition} then ${value}`;
}
function esle<T>(value: T): SQL<T> {
    return sql`else ${value}`;
}

function esac<TWhen>(...rules: SQL<TWhen>[]): SQL<TWhen> {
    const s = sql<TWhen>`case`;
    rules.forEach((rule) => {
        s.append(rule);
    });
    s.append(sql`end`);
    return s;
}

// Example usage:
const caseSQL = esac(
    nehw(companyIsNew(), 'new'),
    nehw(companyIsInitializing(), 'initializing'),
    nehw(companyIsActive(), 'active'),
    esle('archived')
);

Any update on this?


Modify überscript to run HG19 and GRCh38

@jpdna has reheadered the HG19 files, which means that we can now run the adam-vs-gatk überscript pipeline on both HG19 and GRCh38.
@jpdna can you:

Add a file similar to https://github.com/BD2KGenomics/toil-scripts/blob/master/src/toil_scripts/adam_uberscript/input_files.py for HG19
Modify the überscript to select between GRCh38 and HG19?

Once you open a PR with these changes, I, @hannes-ucsc, and @almussel will review. Once it looks good, we can kick off the 10 sample HG19 run. I'll add AIs for this on #113.

PR #133 resolves this hopefully.
@fnothaft  - I want to now do a test run of this on the cluster.    Doing so should not  create any conflict with your runs given we are using seperate CGCLOUD namespaces, but wanted to double check with you before I start as if there is any danger  I'd wait.

Reopening this; @jpdna @hannes-ucsc the permissions for s3://cgl-pipeline-inputs/variant_hg19_reordered are screwy. We were using https:// URLs, but the files aren't public and the AWS console won't let me make them public. I've added a commit to #171 to WAR this, but it'd be good to have a second set of eyes on the issue.

Probably ACLs that get in the way.

Probably ACLs that get in the way.

Actually, the odd thing is that it looks like it is more the absence of ACLs that is getting in the way.

This was resolved by #159.


numberinfo does not use real data

Pokemon and Inventory count does not use the actual player limit (it is hard coded to 250 and 350 even if you purchased upgrades).

true. I didn't know you can do that :)
I'll have to check where I can get this info


Use HttpUrl in Request.

This makes URL validation more eager than it is today.

LGTM

  MainTest.simple:32 expected:<http://example.com[]> but was:<http://example.com[/]>
  MainTest.contentTypeHeader:66 expected:<http://example.com[]> but was:<http://example.com[/]>
  MainTest.put:39 expected:<http://example.com[]> but was:<http://example.com[/]>
  MainTest.userAgent:82 expected:<http://example.com[]> but was:<http://example.com[/]>
  MainTest.referer:74 expected:<http://example.com[]> but was:<http://example.com[/]>
  MainTest.dataPut:56 expected:<http://example.com[]> but was:<http://example.com[/]>
  MainTest.dataPost:47 expected:<http://example.com[]> but was:<http://example.com[/]>

https://github.com/square/okhttp/issues/1486


spike(api): research into techniques to start a protocol from any point.

Overview
Can a Python protocol be started from any line? Any function?
Implementation details

User can select a start line and end line in a python protocol
Use hardware simulator for execution from beginning of protocol.
Using python's debugger hook, sys.settrace, attempt to follow a protocol until start line.
When start line is reached, we transition to hardware controller*. Copying state from simulator to hardware controller*.
Continue following along in sys.settrace until end line at which point return to simulator, copying state from hardware controller* to simulator.

Acceptance criteria
Our goal is to prove that this is feasible and useful.
The result should be a simple demonstration.

for the purposes of the spike, a second hardware simulator instance can be used.

Resulted in a demo of running a python protocol line by line. Consensus was that having line-by-line (or command-by-command) functionality is attractive. The ability to switch between hardware and simulator mid-run is also appealing.
The sys.settrace function was used to install a call back just as any debugger would.  Code is in this branch:
https://github.com/Opentrons/opentrons/tree/proto_line-by-line


nohup --help doesn't work

$ sha1sum toybox-i686
2d735feafb2afc1127180a87bb27aac34c154e55  toybox-i686
$ ./toybox-i686 --help
Toybox 0.8.8 multicall binary (see https://landley.net/toybox)

usage: toybox [--long | --help | --version | [COMMAND] [ARGUMENTS...]]

With no arguments, "toybox" shows available COMMAND names. Add --long
to include suggested install path for each command, see
https://landley.net/toybox/faq.html#install for details.

First argument is name of a COMMAND to run, followed by any ARGUMENTS
to that command. Most toybox commands also understand:

--help		Show command help (only)
--version	Show toybox version (only)

The filename "-" means stdin/stdout, and "--" stops argument parsing.

Numerical arguments accept a single letter suffix for
kilo, mega, giga, tera, peta, and exabytes, plus an additional
"d" to indicate decimal 1000's instead of 1024.

Durations can be decimal fractions and accept minute ("m"), hour ("h"),
or day ("d") suffixes (so 0.1m = 6s).
$ ./toybox-i686 nohup
nohup: Needs 1 argument (see "nohup --help")
$ ./toybox-i686 nohup --help
nohup: exec --help: No such file or directory
$

Hmmm, looks like printf and chroot would also have that problem: anything with ^ but no flags in the argument string, so stopearly gets incremented twice before the loop starts in lib/args.c...

FYI commit 42ce6272f5a3 back in April is what broke this, but that fixed another bug so I can't just back it out. Needs a little design work...

No it doesn't.
commit 481a19c48959

Did that fix it?

In the absence of a reply, I'm assuming that fixed it.


Image Files-White Background

When I attempt to insert an image file, (background image) my background is a white square box, even when I copy and paste your same beer stein example.  How do I get just the image figure, weapon, character, to show up without the white square box image?
Thank you
RohanJax
Additional Details
Share Link :
or
Brew code to reproduce : Click to expand
PASTE BREW CODE HERE

Your "additional details" section of the report has not been filled out
properly. There is no way to help if we can't see what your code is doing.
On Mon, Jul 17, 2017, 2:58 PM RohanJax<EMAIL_ADDRESS>wrote:

When I attempt to insert an image file, (background image) my background
is a white square box, even when I copy and paste your same beer stein
example. How do I get just the image figure, weapon, character, to show up
without the white square box image?
Thank you
RohanJax
Additional Details
Share Link :
or
Brew code to reproduce :
Click to expand
PASTE BREW CODE HERE
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stolksdorf/homebrewery/issues/447, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFmzBmLuFKA0oCnfc3XSSx3G4_HpMHg3ks5sO67egaJpZM4OaZFk
.

I copied and pasted a screen shot of what I am seeing, and my code.  Hopefully it translates, and works.
Thanks

Without your actual code helping is really hard. The screenshot is too blurry to read any of the text.  Are you sure you can't just paste your code in here?
Looking at your screenshot though, my guess is that your image is showing that way because the image file actually has a white background on the image.  The beer stein in the front page example has a transparent background. I would suggest opening up Photoshop or Gimp and getting rid of the white background. Pixlr.com is a free web-based tool that could do that as well. Save the file as a PNG and the transparent parts will stay transparent.
There is also this tutorial someone made which explains how you can try adding some CSS code to turn a white background transparent.  If it were me though I would just edit the image so its transparent on its own.

I tried pasting code, but it looks like just the image showed up, here it is again....  If this doesn't work, no worries.  I'll try your other methods.  Thanks.

Ok. Well yes, this image that you have here does have a white background which you will need to remove in an image editing program. That should solve your issue. Pixlr is great and free and you can do it in about 2 minutes.

Super, thanks.  I'll check that out.


LICENSE file

hi guys...
Looks good!
Could you do me a kindness and add a LICENSE file.  It will save me a trip to the lawyers.
Thanks!
nickg

This is Apache2. We'll get the project template applied.
@thaJeztah Let's get this project into proper shape.

Yup, let me get that fixed. Wasn't aware of this project :smile:


Verify message checksum

Description
Closes #1626.
This PR solves the issue with checksum verification for an OP_MSG

[ ] I added/updated unit tests.
[ ] I added/updated integration/compatibility tests.
[ ] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[ ] I ensured that PR title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

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

@AlekSi if I use an io.TeeReader, that means the rest of the implementation for ReadMessage would have to rely on an io.Reader instead of the original bufio.Reader since the io.TeeReader would have already read the data from the underlying buffer. Is this the suggested behaviour?

For this implementation to work, the data can first be read into a TeeReader, then the checksum is calculated, then the data is then read again from the TeeReader into a bufio.Reader so that it can used for the rest of the implementation. However, this is duplicated work

Sorry, I don't follow. TeeReader, MultiWriter, and other IO helpers like those two do not keep read or written data in memory. So there is no duplication of work.
I also don't think we should parse the whole message to calculate the checksum. After all, we calculate the checksum of bytes, not parsed data structure.
I think the algorithm could be something like that:

Read the message header.
If it does not contain ChecksumPresent, do change anything.
If it does contain that flag, create a TeeReader that wraps the existing reader and writes to the CRC hasher. Write read header bytes to hasher.
Use that reader to read bytes for parsing while also hashing them.
In the end, check the checksum.

I found a way around using a TeeReader and made use of the read bytes instead. Please kindly review and let me know what you think.

@AlekSi some of the tests are failing and I'm not sure why. My assumption is that some messages in the records do not contain valid checksums.

I will check it shortly

Unfortunately, the current solution doesn't work.
I run this branch locally (task run) and I insert a document in mongosh: db.hellowo.insert({"foo":"bar"}).
I receive an error and these are FerretDB logs:
2023-05-09T15:56:30.394+0200    DPANIC  // <IP_ADDRESS>:54836 -> <IP_ADDRESS>:27017   clientconn/conn.go:167  [msg_body.go:86 wire.ReadMessage] [msg_body.go:159 wire.verifyChecksum] OP_MSG checksum does not match contents
(err = <nil>)
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run.func3
        /Users/elena/workdir/FerretDB/internal/clientconn/conn.go:167
runtime.gopanic
        /opt/homebrew/Cellar/go/1.20.3/libexec/src/runtime/panic.go:890
github.com/FerretDB/FerretDB/internal/handlers/proxy.(*Router).Route
        /Users/elena/workdir/FerretDB/internal/handlers/proxy/proxy.go:67
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run
        /Users/elena/workdir/FerretDB/internal/clientconn/conn.go:312
github.com/FerretDB/FerretDB/internal/clientconn.acceptLoop.func1
        /Users/elena/workdir/FerretDB/internal/clientconn/listener.go:310
panic: [msg_body.go:86 wire.ReadMessage] [msg_body.go:159 wire.verifyChecksum] OP_MSG checksum does not match contents [recovered]
        panic: [msg_body.go:86 wire.ReadMessage] [msg_body.go:159 wire.verifyChecksum] OP_MSG checksum does not match contents
(err = <nil>)

goroutine 87 [running]:
go.uber.org/zap/zapcore.CheckWriteAction.OnWrite(0x2, 0xc0001f35f0, {0x1017cb92f?, 0x42?, 0xc000208a00?})
        /Users/elena/go/pkg/mod/go.uber.org/zap@v1.24.0/zapcore/entry.go:198 +0xb0
go.uber.org/zap/zapcore.(*CheckedEntry).Write(0xc0001f35f0, {0x0, 0x0, 0x0})
        /Users/elena/go/pkg/mod/go.uber.org/zap@v1.24.0/zapcore/entry.go:264 +0x3c8
go.uber.org/zap.(*SugaredLogger).log(0xc00011a2a0, 0x3, {0x1011c2874, 0xd}, {0xc00045d108, 0x2, 0x2}, {0x0, 0x0, 0x0})
        /Users/elena/go/pkg/mod/go.uber.org/zap@v1.24.0/sugar.go:295 +0x10c
go.uber.org/zap.(*SugaredLogger).DPanicf(...)
        /Users/elena/go/pkg/mod/go.uber.org/zap@v1.24.0/sugar.go:184
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run.func3()
        /Users/elena/workdir/FerretDB/internal/clientconn/conn.go:167 +0x124
panic({0x1015effe0, 0xc0008aabd0})
        /opt/homebrew/Cellar/go/1.20.3/libexec/src/runtime/panic.go:890 +0x258
github.com/FerretDB/FerretDB/internal/handlers/proxy.(*Router).Route(0xc0001ec960, {0x1016ed560, 0xc0005cf830}, 0xc0006d50a0?, {0x1016ee7e0, 0xc0001ece20})
        /Users/elena/workdir/FerretDB/internal/handlers/proxy/proxy.go:67 +0x1b8
github.com/FerretDB/FerretDB/internal/clientconn.(*conn).run(0xc0001b6380, {0x1016ed560, 0xc0005cf710})
        /Users/elena/workdir/FerretDB/internal/clientconn/conn.go:312 +0x1370
github.com/FerretDB/FerretDB/internal/clientconn.acceptLoop.func1()
        /Users/elena/workdir/FerretDB/internal/clientconn/listener.go:310 +0x9e4
created by github.com/FerretDB/FerretDB/internal/clientconn.acceptLoop
        /Users/elena/workdir/FerretDB/internal/clientconn/listener.go:269 +0x5d

As I send a correct request through a correct drive, I expect the checksum to be correct.


TypeError: Cannot set properties of undefined (setting 'intArrayToUtf16')

I have installed the package and even after just importing it ,
import { RNSerialport, actions } from 'react-native-serialport'
its giving these errors:
TypeError: Cannot set properties of undefined (setting 'intArrayToUtf16')
TypeError: Cannot read properties of undefined (reading 'actions')
I am trying to run the applications on windows.

Usa o yarn


Error with Winston 3 ::: TypeError: winston.Logger is not a constructor

Seeing following error after upgrading to Winston 3.1 and express-winston 2.6.0

TypeError: winston.Logger is not a constructor
at Object.logger (/path/test-service/node_modules/express-winston/index.js:190:59)

The source code that's causing the problem:
index.js:190
    options.winstonInstance = options.winstonInstance || (new winston.Logger ({ transports: options.transports }));

Usage in Code:
expressWinston.logger(requestLoggerOptions)

Package Versions:
"express-winston" : "2.6.0",
"winston" : "3.1.0",

Apologies. Just realised there is a newer express-winston version 3.0.1. Closing this off. Sorry for spamming.


Secure the network flow - How will this work

In the secure network flow diagram, it depicts outside initiated traffic in green and cluster initiated traffic in orange: https://docs.microsoft.com/en-us/azure/architecture/reference-architectures/containers/aks/secure-baseline-aks#secure-the-network-flow
I don't see the green traffic request/response will work as in the diagram
The dotted green return traffic should go through Azure Firewall because of UDR setup. That's why AKS docs mentions the need for using inbound NAT on Az Firewall
Thanks,
Mahmoud

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

ID: 46723a24-75ac-f6e2-c172-d3827ca0b7d6
Version Independent ID: c975744e-e594-5667-14b3-c38824bbe24c
Content: Baseline architecture for an Azure Kubernetes Service (AKS) cluster - Azure Architecture Center
Content Source: docs/reference-architectures/containers/aks/secure-baseline-aks.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @PageWriter-MSFT
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#271079

Thank you for reaching out @melzayet.  The two traffic flows are from distinct origins.  The top (green if you can see the colors) flow is HTTP traffic destined for your cluster's workloads (the web app/api, etc) running on nodes in the cluster. the HTTP response for those requests will egress back out to the WAF and to the client.
The bottom (orange-ish) flow is traffic that originated in the subnet containing the nodes.  This would be traffic out to external container registries, the Kubernetes API server, etc.  Basically any cluster-originated traffic that we want to protect with our corporate egress firewall rules.
Because the traffic originated from outside of the UDR-ruled network and the fact we are using internal load balancer in a separate subnet, it takes the same return path.  The UDR-rule applies to traffic originating within its scope.
Also, to be clear, this solution presented solves for egress lockdown (as defined as traffic originating from the cluster) only, it doesn't funnel both ingress and egress through the firewall.
I'm assuming the DNAT part you're talking about is coming from this section in the AKS docs (specifically this).  In that case, they are also forcing all ingress traffic to also flow through the firewall. In their case, they are using a public load balancer for ingress, our solution does NOT use a public load balancer for ingress.  We are making contact with AAG, which is then routed through a internal load balancer in an ingress subnet, which then will not be subject to the asymmetric routing issue.  If you exposed your services directly via the public load balancer IP, then you'd for sure run into the asymmetric routing issue.  Here is the docs where we talk about Internal load balancers note being subject to this asymmetric routing issue.
Let me know if I can help clarify any more - I know this seems a bit "counter intuitive."  Feel free to deploy using the reference implementation, at the end of that journey you'll have a website where workload HTTP traffic is handled via the ingress route (Client -> WAF -> internal LB -> nodepool -> WAF -> Client) and cluster-originated traffic (nodepool -> FW -> external resource -> FW -> nodepool) is handled via the egress route.

@ckittel Thanks for sharing insights on this! I am assigning this issue to you.

@ckittel thank you very much for the detailed and informative reply. I think my confusion came from the asymmetric issue but you clarified very well. Thanks again for taking the time


crash report

CentOS release 5.7 (Final)
Linux server14 2.6.18-274.el5 #1 SMP Fri Jul 22 04:43:29 EDT 2011 x86_64 x86_64 x86_64 GNU/Linux
what should i do if dumps file needs upload?

I upload dump file here, thanks
http://speedy.sh/DYSqQ/7411bc2b-0e07-b58c-169a59f4-0d5f780c.dmp.gz

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain castonguay.org has exceeded the max emails per hour (232/200 (115%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext1.iad.github.net ([<IP_ADDRESS>]:49086 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by server191.web-hosting.com with esmtps (TLSv1.2:DHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.86_1)
(envelope-from<EMAIL_ADDRESS>id 1apkR8-003Nfb-W6
for<EMAIL_ADDRESS>Mon, 11 Apr 2016 18:36:03 -0400
Date: Mon, 11 Apr 2016 15:35:22 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1460414122;
bh=3P+n1qghITR10C96TV2u37+rE0EaTsW+WMDKfoBtu5E=;
h=From:Reply-To:To:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=j9j+r+Y3Mkc6iWizBdhtr/8dvzbWfPrOG8TWO0f6mMRQQyVa7IvTp1xg4gcUIe6wZ
1Hij+6KKO2BeS4xezBg1+2FhHNLMZj3ehEVbKgFeThGgTUkBJCqrufGVAMFtas1Cov
SyZT/K72GnLYfbLGMEetirTIKA6yE+ZWNQBKckLY=
From: Vitaly Slobodin<EMAIL_ADDRESS>Reply-To: ariya/phantomjs<EMAIL_ADDRESS>To: ariya/phantomjs<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [ariya/phantomjs] crash report (#11865)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_570c26aa9e285_8fd3f86a19792a0480556";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: Vitallium
X-GitHub-Recipient: dREc
List-ID: ariya/phantomjs <phantomjs.ariya.github.com>
List-Archive: https://github.com/ariya/phantomjs
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AAF3uqz6ScMbcQ4hyjj24AHoOB0GZaZ8ks5p2syqgaJpZM4BW7Rj
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-Spam-Status: No, score=-3.5
X-Spam-Score: -34
X-Spam-Bar: ---
X-Ham-Report: Spam detection software, running on the system "server191.web-hosting.com",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  Please upgrade to PhantomJS 2. 1.9.x is no longer supported,
and known to have many crasher bugs that are fixed in 2 version. --- You
are receiving this because you are subscribed to this thread. Reply to this
email directly or view it on GitHub: https://github.com/ariya/phantomjs/issues/11865#issuecomment-208594437
[...]
Content analysis details:   (-3.5 points, 5.0 required)
pts rule name              description

0.0 URIBL_BLOCKED          ADMINISTRATOR NOTICE: The query to URIBL was blocked.
See
http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block
for more information.
[URIs: github.com]
-5.0 RCVD_IN_DNSWL_HI       RBL: Sender listed at http://www.dnswl.org/, high
trust
[<IP_ADDRESS> listed in list.dnswl.org]
-0.0 RCVD_IN_MSPIKE_H2      RBL: Average reputation (+2)
[<IP_ADDRESS> listed in wl.mailspike.net]
1.6 HTML_IMAGE_ONLY_12     BODY: HTML: images with 800-1200 bytes of words
0.0 HTML_MESSAGE           BODY: HTML included in message
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
X-Spam-Flag: NO
----==_mimepart_570c26aa9e285_8fd3f86a19792a0480556
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ariya/phantomjs/issues/11865#issuecomment-208594437
----==_mimepart_570c26aa9e285_8fd3f86a19792a0480556
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.
—You are receiving this because you are subscribed to this thread.Reply to this email directly or view it on GitHub

----==_mimepart_570c26aa9e285_8fd3f86a19792a0480556--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain castonguay.org has exceeded the max emails per hour (238/200 (119%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o3.sgmail.github.com ([<IP_ADDRESS>]:41945)
by server191.web-hosting.com with esmtps (TLSv1.2:DHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.86_1)
(envelope-from<EMAIL_ADDRESS>id 1apkRs-003OkH-Le
for<EMAIL_ADDRESS>Mon, 11 Apr 2016 18:36:49 -0400
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=github.com;
h=from:reply-to:to:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=qkm14N/MZMrPb6Fwm4GbPKWl81I=; b=Efsj4SZN24QdM61/
jFx6GoW0zXXWeq7BjnFQADbn1gbu9B3MSApENcuyR1PQaQ1njYRs4xFJioBIBIyX
xtlp10St8Fu4EFG/mSv2gQvdS+kJYOQICpfCOVt8XMwu/9LoMlabP2KO0RwONluU
KWOQYrPCDU3V68UYwhy9oppSmmA=
Received: by filter0551p1mdw1.sendgrid.net with SMTP id filter0551p1mdw1.18345.570C26AB38
2016-04-11 22:35:23.471199872 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0003p1iad1.sendgrid.net (SG) with ESMTP id 1LFjQPNuR7uIkDtyWWYtNg
for<EMAIL_ADDRESS>Mon, 11 Apr 2016 22:35:23.242 +0000 (UTC)
Date: Mon, 11 Apr 2016 15:35:23 -0700
From: Vitaly Slobodin<EMAIL_ADDRESS>Reply-To: ariya/phantomjs<EMAIL_ADDRESS>To: ariya/phantomjs<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [ariya/phantomjs] crash report (#11865)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_570c26ab17b7e_5f083fefca0552b85994dd";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: Vitallium
X-GitHub-Recipient: dREc
List-ID: ariya/phantomjs <phantomjs.ariya.github.com>
List-Archive: https://github.com/ariya/phantomjs
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AAF3uqWKZSs9G7wK8689ZcWvbsvui7gkks5p2syrgaJpZM4BW7Rj
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: CjWC6vIGLGo0uWsXQ9th+U0br113QN4BhNL8BXzrk7uKrR9grGpwuLZANn8+bCOGw0RUsV0Lx9pFDM
snKrjEpebuU2a/K49K4Yi5Q1c4FFZ0fCxuKb5xWRM7zA0viAhJHBt21xf+hpUKkfbIJwYu/XfhjMJx
6OVl6hex7ooz8oaq7a9Q44AT4uPtnZf8VyQt0ZbHZRmH8vmg81YbZmMjBenHVCRErp3m+I7+S0vRAM
Y=
X-Spam-Status: No, score=-1.8
X-Spam-Score: -17
X-Spam-Bar: -
X-Ham-Report: Spam detection software, running on the system "server191.web-hosting.com",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  Closed #11865. --- You are receiving this because you are
subscribed to this thread. Reply to this email directly or view it on GitHub:
https://github.com/ariya/phantomjs/issues/11865#event-623253181 [...]
Content analysis details:   (-1.8 points, 5.0 required)
pts rule name              description

0.0 URIBL_BLOCKED          ADMINISTRATOR NOTICE: The query to URIBL was blocked.
See
http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block
for more information.
[URIs: github.com]
-1.0 RP_MATCHES_RCVD        Envelope sender domain matches handover relay domain
-2.3 RCVD_IN_DNSWL_MED      RBL: Sender listed at http://www.dnswl.org/, medium
trust
[<IP_ADDRESS> listed in list.dnswl.org]
1.6 HTML_IMAGE_ONLY_12     BODY: HTML: images with 800-1200 bytes of words
0.0 HTML_MESSAGE           BODY: HTML included in message
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
-0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)
[<IP_ADDRESS> listed in wl.mailspike.net]
-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders
X-Spam-Flag: NO
----==_mimepart_570c26ab17b7e_5f083fefca0552b85994dd
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #11865.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ariya/phantomjs/issues/11865#event-623253181
----==_mimepart_570c26ab17b7e_5f083fefca0552b85994dd
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #11865.
—You are receiving this because you are subscribed to this thread.Reply to this email directly or view it on GitHub

----==_mimepart_570c26ab17b7e_5f083fefca0552b85994dd--


Reduce Spacing Between  and Slightly Indent Unordered List Item

Would we be able to do an alternative? Use main unordered list as a header and the nested list as the unordered list

@parkjc @billymeinke This is what I get when I use Header 5 ##### and and Header 6 ###### respectively

@parkjc @billymeinke Sorry, misinterpreted that... used Header 4 and nested an unordered list under one of the Header 4 points

This looks better, thanks @fjumawan. We'll just need to be careful about line spacing between the paragraph above and the beginning of the lists.


Exp: modify isReadOnly to support more read cases

Added handling for CallExpressions for just reading of snapshots.
Simplified the eslint fires for read cases and write cases
Adds specific checks for hooks and custom hooks (might need to re-consider)

Fix: #28

The other pending test cases are yet to be checked, in this commit, do not merge.

Removed the cache file
added the remaining 2 cases


Jhbate/upload meta check

WIP but should be done, discussing with @jebeck

This did break the CLIs @jh-bate
Can you fix please?

sorry @jebeck - after saying I would test that I totally forgot about it.

So both cli's now pass in the id of the user into the driver.

One of the log statements ends up with undefined both in CLIs and through UI: Api              | saved upload metadata  undefined
Trying to log the count?

yeah it was but  we just care if there is an error so updated that

So this mostly works as I'd expect now. There's still an undefined being logged - maybe it's just a common issue for all the drivers? It occurs here and here, for instance...

Oh and also the UTC time isn't being added to these errors...is that expected?
I think it'd be great if we could customize the error message for these too. Is that possible? "Upload failed due to bad metadata" or something?

Yeah I think there are a few places like that in the uploader as there was a lot of logging done in order to see what was going on.
With regards to the other Q. Yes, I was going to tackle that as part of the card https://trello.com/c/FccsTSO7. If your happy with that answer then shall we merge this and tackle the nicer consistent messages for the next card

Leaving nicer messages for the next task is fine, but maybe remove the attempts at logging result in the drivers? I'd rather that not get left behind - I find it confusing (and potentially red herring) when there are undefined things being logged.

cool - log statement removed from all drivers when there is an error


partials doesn't show up in {{>context}}

Here the small snippet:
	var ractive = new Ractive({
		el: '#target',
		template: 'Outside: {{>foo}} - <mycomp>Inside: {{>foo}}</mycomp>',
		partials: {
		    foo: 'fooooooo'
		},
		components: {
		    mycomp: Ractive.extend({template: '<i>{{>content}}</i>'})
		}
	});

Actual output:
 Outside: fooooooo - Inside:

Expected output:
 Outside: fooooooo - Inside: fooooooo

This sounds familiar. Off the top of my head, since it uses {{> content }}, {{> foo }} is being resolved on the inside the child component, not the parent. https://ractive.js.org/api/#content

Seems you should use ‘’{{yield}}’’ instead of ‘’{{>content}}’’.

I can confirm that {{>content}} is not supposed to find partials on the parent unless the component is isolated: false. If you need stuff from the parent, {{yield}} is indeed the way if you don't want to break isolation.
It may make sense to have a way to pass a partial into a component from the parent via template without using {{#partial}}. I haven't run across a scenario where {{yield}} doesn't work yet, though.

I'm aware of the difference between {{>content}} and {{yield}}. Both have their use cases. To illustrate my point better, I provided a small example:
https://ractive.js.org/playground/#N4IgFiBcoE5SBTAJgcwSAvgGhAF3gDxICWAbgATFIC8AOngIYxq70B8BA9CaW7QHYCCAZwDGMYgAdclGvVwIAtpIA2DBfXK4AnpIR08CAB65OMBqNxkE7AbVwEwAZjYBCd+QAqYYsMp+GcmFiZRViADNiZHJjBlCELHJ+AHsZQIB3ZJgAa2J+FHIkJWTyZPDyBn4KgFdcMuIVFXJ3Vy5nPkFBB0k2AHUwBCrcAcKYZMkkZPSq4gVFPzKtEYZa+sagsCm-aslE9LB1LRKSYVUGbXJwrJiLMEo5jeTqlSRyACMEclFq4TrFYgAXgw3ioEAA6Lg9OwOFZ1SLrWZKYQGYDAH4IGDCDAYWxdewEN5sVFUbFcQnkAC05FR-DiCGxlPIBGIbAAFKilAwGtiAJRcFnQriwtYqDr2QU9ADKeVEn2E1VEd2F8JUARgn3Rrzy5EUlQuZ1lwkSoNwAHI-JIxqQqJ9Zn5RMl+AonRULUwrAwVGDyAAhaqKRQYxLq+UqGS+GLKHQQzhQroEZUNJqI+YotHCDFYnEgMWsXCotjomAAfTeDGCoizgs4icauehqIAxJJ3cRPeQi6Xy8RK9jofjCcSkKTOOSqTS6Qyqcy2RzddyMHzOAKuqjY63PX3+FwxBJpGwQDhhPAAEoWKykcEO5SOwa4YQAbVNtZUpoAuuRqOQz5ZrGDjM6SDstCcxnAokDkAABv2Dh5JItTkJw9YOM8yH2E2CC3PcSJbuKeb4mEQQ6KCBhlqI2QoGM1T8EgFIOioWSQI2RThAA3LqzB5JAACskhGKxuJ4UJ+bAGwDpOneuH4XmXBhGhImcJhipSfinCodC5BQQIi52J0pBMOQ5i-pen5JAg6Tfue1jAV0uBMCwEGmnZzAIGaWACJpoFqOB5Cmo2XnqAgpruVUhTqAwEHAB55D2EWwgQQ+UWhXhVAQQAjBlIWaXhtKBo5ACCYSysF0V4ZyDSOZ6PYIAAApkNEYmCag0aapW4Ng1JtZpqXkAATP1WUxXmuUII5PrJG8JXJXm5UqI5bwTTVsTxGC16tdNHVJdlMiyBBTj7YNOV0o5ADCBwwGEQWHTN85zb5ipMJdNWVGgqpgjK63bRgb7RTi-CLqxmBAA
IMHO isolated should solely apply to the data, not the other stuff. Or at least include the partials. From a ractive user perspective, I find it rather counter-intuitive that the second example doesn't work.

Thanks for pointing that out. You are right. I was stuck in old versions of ractive's {{yield}} in my mind and wasn't aware of this "aliasing". It's a quite an advanced but nifty feature


Update hacs.json

Set the HASS version to the most recent non-BETA version

Why is this needed? I understand from HACS documentation that the non-beta releases will also be passing this condition.

You have the following in hacs.json: homeassistant: "2023.4.0b1"
The docs state: "Allow Home Assistant beta versions by appending b0. Without b0, only official releases will be allowed."
This is ofcourse perfectly valid, since you only want to deploy on official HASS releases.
However, since 2023.4.0b1 is HIGHER than 2023.3.6 nothing gets installed, which is the issue.
The 'homeassistant" value should contain the minimum required version. Since practically no-one has a BETA running, which is higher than the current HASS release installation will not happen.

My intention was to have v1.9.7 show up as latest available version for users who are on HA 2023.3 or lower.
And to have v1.9.8 show up for users who are on HA 2023.4 (beta or alpha).
If I understand you correctly, pinning to HA 2023.4 in the hacs.json file causes the entire Alarmo project to require HA 2023.4, which is not what I wanted.
In this case I rather remove the pinning completely, HACS documentation is not very clear on this.

I tried this in my own HA installation.
With HA 2023.3 installed.
V1.9.7 shows up as OK for download:

V1.9.8 shows up as incompatible:

After updating to HA 2023.4, V1.9.8 shows up as available for download:

All seems to work as intended so we can close this PR.


feat(signup): Add disabled signup page

Add a beta signup page and logic that will disable the signup page using
a environmental flag

Tested locally. LGTM! Thanks for your fast work on this!


Moya compatible with RxSwift 4.0

I created a pull request for that 👍

I forget to check the 10.0.0-dev branch ;)

Oh, no worries @joanii! FYI we just released 10.0.0 version so you don't have to use 10.0.0-dev branch anymore 😉


weed server -port not work

when use:
weed help server
Usage: weed server -port=8080 -dir=/tmp -volume.max=5 -ip=server_name
then input in cmd:
weed server -ip=<IP_ADDRESS> -port=8080 -dir=./data
display as follow:
flag provided but not defined: -port

that's expected. see weed server -h for the options.

sorry, saw the problem. adding a fix.