Incomplete info in "How to reference secrets in components" leads to dapr run errors

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
  I have not made any changes since I do not know how to use the document technology, but below is what needs to be added to the document.  Text is copied from Issue #1116.
The current version of "How to reference secrets in components" provides inaccurate examples and no explanation for self hosted situations referencing secrets that are stored in local files secret stores, environment variable secret stores, and perhaps the Azure Key value. For these stores, setting the secretRef name and key to DIFFERENT string values results in dapr run error messages like:
time="2021-01-22T14:48:29.5541945-05:00" level=error msg="failed to init output binding svs-servicec-inputq-localsecret-asbqueue (bindings.azure.servicebusqueues): failed parsing connection string due to unmatched key value separated by '='" app_id=serviceA instance=A-GeorgeXPS1 scope=dapr.runtime type=log ver=1.0.0-rc.2
time="2021-01-22T14:48:29.5562059-05:00" level=error msg="failed to init output bindings: failed parsing connection string due to unmatched key value separated by '='" app_id=serviceA instance=A-GeorgeXPS1 scope=dapr.runtime type=log ver=1.0.0-rc.2
time="2021-01-22T14:48:29.5562059-05:00" level=fatal msg="process component svs-servicec-inputq-localsecret-asbqueue error: failed parsing connection string due to unmatched key value separated by '='" app_id=serviceA instance=A-GeorgeXPS1 scope=dapr.runtime type=log ver=1.0.0-rc.2
URL of the docs
https://v1-rc2.docs.dapr.io/operations/components/component-secrets/
Expected content
In addition to the existing content, yaml needs to be shown for local file and environment variable secret stores in self hosted scenarios. In these stores the secretRef name and key must be set to the SAME string value. The Azure Key Vault may be the same. Plus there needs to be text that warns the user of what causes the above errors.
Screenshots
Additional context
Here is the conversation on Dapr Discord that revealed this fact -- See CMendible3's comment near the end of the thread for a terse summary of the actual behavior (https://discord.com/channels/778680217417809931/781589820128493598/802585525655568414)
Issue reference
 Issue #1116

Closing as this PR includes unrelated changes from rc.3 branch that should not go into v0.11 branch. Please open a new PR from a dev branch that addresses the specific issue only

Closing as this PR includes unrelated changes from rc.3 branch that should not go into v0.11 branch. Please open a new PR from a dev branch that addresses the specific issue only


new(userspace/libsinsp): support lxc 4.0 cgroup layout.

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area libsinsp
Does this PR require a change in the driver versions?
What this PR does / why we need it:
LXC 4.0 uses a new cgroup prefix/layout: https://linuxcontainers.org/lxc/news/2020_03_25_13_03.html
This PR adds support for it.
Which issue(s) this PR fixes:
Fixes #
See https://github.com/falcosecurity/falco/issues/3213
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

new(userspace/libsinsp): support lxc 4.0 cgroup layout.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I propose to backport this to the 0.17.1 tag, since it is an handy fix.
/milestone 0.17.1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add indent based code folding

@alexandrudima Alex, FYI

@aeschli
Is there a way for extensions to figure out which parts of code are folded?
For example vim extensions need this to properly move the cursor when different parts of code are being folded.
I think this is not possible right now and if that's the case, what do you think about exposing getHiddenAreas() function to extensions?

@aeschli atom editor uses the same indentation based folding but also includes the line after a fold-range with a foldEndPattern regex check. The regex pattern is different for each language.
For css: 'foldEndPattern': '(?<!\\*)\\*\\*/|^\\s*\\}|\\/*\\s*@end\\s*\\*\\/'
For js : 'foldEndPattern': '^\\s*\\}|^\\s*\\]|^\\s*\\)'
Some language extensions in vs code already have this pattern (php but not all of them.
Clearly this needs to be added to all language extensions but it doesn't seem to be easy.
Specially because each extension has it's own syntaxes files (even with different formats).
Do you think supporting C based languages with this approach is a good idea (just for now)?
Should i send a PR only supporting languages with block folding ( {  } ).

@aminroosta There is currently no API in the extension host to find out about folded regions. But there is ongoing work for improving VIM support to enable operations in the view model, see #9143
Supporting folding end patterns is a good suggestion. For now we have no work planned as other issues have higher priority.

@aminroosta we already have new APIs for cursor movement across visible lines, you can run
vscode.commands.executeCommand("cursorMove", {
      to: "lineUo", // or "lineDown"
      noOfLines: 1
    });

to move across folded lines.

@aminroosta current implementation of VSCodeVim is not handling cursor moving correctly. We are changing cursor position ourselves, which breaks the cursor moving in folded code. But now I find that we should leverage Code's existing command cursorLeft, cursorLeftSelect, etc to perform the real cursor moving. In this case, we don't need to know the hidden lines information any more.

@aeschli I just found another use case :-), implementing toggle-fold command.
without information on folding areas that would be impossible.

@rebornix I am seeing following behavior in VS Code and it is similar to Vim behavior you mentioned in your example

After discussing with @rebornix fixing the issue #9619 will let Vim to completely use VS Code apis for cursor movements, thereby, fixing the folding issue automatically.


Fix weave default iprange

Default iprange conflicts with DigitalOcean, this should fix it (fix taken from weave master)

looking good!


ScaNN normalizing dataset pre-training

As mentioned in Accelerating Large-Scale Inference with Anisotropic Vector Quantization

"MIPS is equivalent to cosine similarity search when all datapoints are equal-norm, so we adopt our technique to cosine similarity search by unit-normalizing all datapoints at training time."

Does this mean that it is not necessary to normalize dataset pre-training as seen in example code?

Does this mean it's OK to normalize data set even if it's not cutting through?￼

As seen in the example code￼

Thank you baby donny, your very helpfull ;)


Event volume

The introduction to the article states "Event volume: Low", but then later, it states multiple times that "This subcategory typically generates huge amount of ... events."

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

ID: 986f0d28-6df3-f44c-b500-dd6fe4b65bc1
Version Independent ID: bd1b7ab1-2dbc-7c85-57d9-0dca8572537c
Content: Audit Logoff (Windows 10)
Content Source: windows/security/threat-protection/auditing/audit-logoff.md
Product: w10
GitHub Login: @Mir0sh
Microsoft Alias: justinha

@officedocsbot assign @mypil

@dkbriley Point taken, thanks for helping to improve the documentation. I will post back here when we have finished making changes.

@mypil This issue can be closed https://github.com/MicrosoftDocs/windows-itpro-docs/commit/e72fc555d62ed74f72dc5fbf5a3effce5a41ea06#diff-f69222beb14c8028db01965f5d33a800

@dkbriley - Thank you for submitting this feedback.
From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Adapt sample jobs to secure setup

This igoes hand-in-hand with this issue
This will likely require introducing data transfer components, or something equivalent.

Duplicate of #33


Can't trigger autocomplete

Not sure what I'm doing wrong or if it's broken.
Tried "::" in frontmatter and ":::" in an inline field.

Can you confirm that you currently have the 0.0.5 version installed?
Do you see any error message in the console (alt+command+i on mac)?

I do have 0.0.5 installed.
I do not see any errors for that plugin in the console.

I've experienced something similar right now on one of my vault.
I've disabled then enabled again the plugin (without removing it) and the suggestor worked again.
Can you try it?

I think I found the issue! I disabled the Various Compliments plugin and now it works!

Ok thank you for the report, I'll add a "know issues" section in the read.me file


Auto Light Estimation (Three.js)

Hey there,
it would be amazing to have automatic light estimation. Right now it's hard to make an object look realistic since the lighting need to get set manually.
Are there any future plans on adding something like this?
I found a video where someone did this with a pingpong-ball but that's not what i call automatic.
Kind regards

I build a lib for location-based ar with includes light estimation based on the AmbientLightSensor API https://developer.mozilla.org/en-US/docs/Web/API/AmbientLightSensor
For a live example see: https://0b01001101.github.io/locationar/examples/ambientlight.html
The use "in production" is limeted, because the Chrome flag #enable-generic-sensor-extra-classes need to be enabled in the current version [Chrome 75]

Closing this issue due to dismission of this repository. New repository of AR.js will be up at https://github.com/AR-js-org/AR.js/
Feel free to re-open the issue there if still valid.


Use $ref from swagger annotation as suggested identifier

The $ref property of swagger annotation as suggested identifier. This is accessible in the following code places (incomplete list):

DefaultApiResponseAnnotationMapper
DefaultHeaderAnnotationMapper
DefaultRequestBodyAnnotationMapper / DefaultRequestBodyOperationCustomizer

The string provided is actually intended to manually reference another item, but I think using it as the suggested identifier is the better usage of that annotation property.

Might be related to #10 and #15


fix(host): cold migration disk removed

What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:

release/3.8

/cc @swordqiu @ioito
/area host

/lgtm


Provide, and use, a buildah script to construct zbm-builder images

Using buildah directly provides flexibility that can not be achieved with a Dockerfile. It also prevents the layer problem that bloats image sizes, avoiding the need to squash the image.
Eventually, we can do all sorts of neat things, like mount the container used to construct the image and manipulate the image from within a chroot.
Still testing, but I think it does what we expect.

Needs a rebase onto master, otherwise this looks good! buildah sure beats a Dockerfile for ease of use.


Reenable ignored test

The test case DatastoreWrapperShould.generate_key_factories_aware_of_tenancy is currently @Ignored due to the namespace removal policy of the Datastore.
Reenable the test as soon as the outdated namespaces are removed from the Datastore.

Will be fixed in the nearest PR.


Deploying a python function to azure does not work

Repro steps:1.2.Action: azureFunctions.deployError type: ErrorError Message: Python projects are not supported on Windows Function Apps.  Deploy to a Linux Function App instead.Version: 0.16.0OS: win32VS Code Version: 1.33.1Call Stack```Object. (dist/extension.bundle.js:391:67390)Generator.next ()o (dist/extension.bundle.js:391:64243)```

@yannickmorel please add repro steps to the issue. If you selected a Windows function app when deploying a python project - that is not supported


Updated icon and installer with new graphics. 💄

Fixes #1259, #1931
===
Changed the current Windows installer from a one-click installer to an assisted installer that should give users more control over the installation process. This includes which users to install the program for, and also where to install it.
This PR also updates the icon for the application.

Application Icon
Old Icon

New Icon

Windows Installer

Coverage remained the same at 68.031% when pulling f5c13cd8b0862eb012ae8eb415e1d0da988d242d on toanzian/install-pic into 23ca21796e05932120b60fba60975f123cc9594d on master.


Fail to run example on Apple Silicon M1 with Rust version 1.66

  = note: ld: warning: directory not found for option '-L/Users/mac/workspace/opensource/nacos-sdk-rust/target/debug/build/libz-sys-1fdb361784ca5052/out/build'
          Undefined symbols for architecture arm64:
            "_res_9_getservers", referenced from:
                _init_by_resolv_conf in libgrpcio_sys-63fbe17b71987d12.rlib(ares_init.c.o)
            "_res_9_ndestroy", referenced from:
                _init_by_resolv_conf in libgrpcio_sys-63fbe17b71987d12.rlib(ares_init.c.o)
            "_res_9_ninit", referenced from:
                _init_by_resolv_conf in libgrpcio_sys-63fbe17b71987d12.rlib(ares_init.c.o)
          ld: symbol(s) not found for architecture arm64
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

warning: `nacos-sdk` (example "simple_app") generated 1 warning

I have the same problem in M1 Rust 1.66.1

This needs tikv/grpc-rs to deal with it.

At present, I only reduce the version of Rust to avoid this problem. rustup default 1.65.0

@shaodan It is work in M1 Rust 1.68.0 , maybe have a try.


Database Error  Failed to fetch invoices  in Ch 11

I encountered some problems when adding the  component in Chapter 11. There's an error occurring when rendering the invoice page, and below is the error message. I followed the tutorial exactly as it is. Could there be an issue with this part of the tutorial?
Thanks.

which step which you did?

I did,
check the result at
https://nextjs-dashboard2-plum.vercel.app/dashboard/invoices?page=2&query=l
and github 👍
https://github.com/liwenz/nextjs-dashboard2/tree/chapter11

I did, I only follow the tutorial. check the result at https://nextjs-dashboard2-plum.vercel.app/dashboard/invoices?page=2&query=l and github 👍 https://github.com/liwenz/nextjs-dashboard2/tree/chapter11

I fixed it already, something went wrong with my lib/data.ts. Thank you so much👍


Fix the highlighter preview

Enable high DPI support. @see MSDN Doc
Action items

[ ] Test with different monitors having different resolution configurations.
[ ] Ensure that the application settings are properly configured. @see: Enabling Dpi-awareness

Fixed.


Tooltip tests occasionally fail.

Particularly annoying at the end of a ~10min Travis build, the tooltip tests sometimes screencap before the tooltip has asynchronously loaded.

Fixed and release in 0.2.10


Pull #135 fix and test together for merge

Just a PR that pulls in the work from @rbarbey and @ducarroz together.  For more information about these please see #135 and #137

@krisselden would love your eyeballs on this.

We need to fix this in Ember.  Who should I hound for review?

@krisselden

I'll try to check it out in a moment, but setting wait=0 isn't the type of code I'd generally merge, either fix the bug in calculating the wait time or remove the code that waits, don't set wait to 0.

in my understanding, we reach that code because we are already over due with the timer, therefore the waiting time should be 0 as we need to fire right away...

@krisselden I believe @ducarroz explanation if correct. I just want to double check with you to make sure we aren't missing something

in my understanding, we reach that code because we are already over due with the timer, therefore the waiting time should be 0 as we need to fire right away...

That's exactly right.

@ducarroz if that is the case why did we cancel it? I still feel like this fix is too far downstream of the problem.

The problem is when you do too many fixes without root cause analysis it just makes the library unmaintainable.  From what you are describing, this seems to be the root cause https://github.com/ebryn/backburner.js/blob/master/lib/backburner.js#L503-L506 we are canceling a good timer and rescheduling.
Furthermore, I find it dubious that we deal in 'wait' and 'executeAt', relative should be immediately converted to absolute, and reconverted to wait just before scheduling the next flush.
I won't block merging, but this path is just going to continue down the path of more needless complexity.

@krisselden backburner is using a single timeout to handle all the tasks scheduled with run.later. The problem is that is possible the timeout does not fire on time because some JS takes too much time to execute. If another task is scheduled before JS had a chance to fire the timeout (after the expected time), the current code believe the timeout is expired and cancel it, then reschedule a new one for what it believe should be the next task to run, causing the initial task to be delayed until the next one fires.

I'm still not understanding @krisselden's concern.

Ah actually a quick skim of the code pointed to does make me nervous. This isn't how I remember it :p

@ducarroz that is the problem, the timer shouldn't be canceled and rescheduled, the code on the line I linked to shouldn't cancel it and shouldn't call this method. None of this code should be in this method, the fact this method takes both a wait and executeAt means that it has had too many issues fixed in the same manner.

@krisselden I totally agree that it would be better to overhaul the approach how timers are handled. I would suggest to merge the fix now and create a separate issue which deals with a fundamental refactor of the code. I'd be more than happy to work on it.


patch: Remove CircleCI in favor of Flowzone

Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

@balena-ci I self-certify!


Automation documentName property does not name the document

I create an automation document with the documentName property:
new AutomationDocument(scope, id, { documentName: 'MyDocument', ... });

I expect the AWS::SSM::Document resource to have a Name property matching MyDocument, but the property is missing:
Resources:
  <resource id>:
    Type: AWS::SSM::Document
    Properties:
      Content: ...
      DocumentFormat: YAML
      DocumentType: Automation
      Tags:
        - Key: CdkGenerated
          Value: "true"
      VersionName: v2.0.0
    Metadata:
      aws:cdk:path: <omitted>
    Condition: <omitted>

To work around this, I can override the name property manually
document.cfnDocument.name = 'MyDocument';

Is there a reason the documentName property does not work this way? It looks like when creating the cfnDocument the SsmDocument could also forward the name property:
this.cfnDocument = new CfnDocument(this, 'Resource', {
  ...this.props,
  ...{
    name: this.documentName,
    content: Lazy.any({
      produce: () => {
        const doc = this.buildSsmDocument();
        return JSON.parse(JSON.stringify(doc));
      },
    }),
    documentFormat: isYaml ? 'YAML' : 'JSON',
    documentType: this.documentType(),
    tags: [{ key: 'CdkGenerated', value: 'true' }, ...(this.props.tags || [])],
  },
});

Correct - your proposal was created in this PR #49

Merged. Closing this one.


[patch] Update help find

Ok, here is my attempt at flattening the operations runner into something a little easier to follow.  The gist is that it splits the "join" code into two paths -- one for when a junction table is used, one when it's not -- and either way, always runs 1 query (with user criteria, if present) on the child table per parent, rather than trying to do things like skip and limit in memory.  This is the least-optimized way to do non-native or cross-adapter joins, but it's the cleanest and we can add optimizations piecemeal while hopefully still maintaining some sense of order.

Cool beans @sailsbot


metacafe extractor: deal with family filter

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

What is the purpose of your pull request?

[X] Bug fix
[ ] New extractor
[ ] New feature

Description of your pull request and other information
If you attempted to download a video that required the family filter to be off from metacafe, youtube-dl would fail while attempting to find the title, as the site will have redirected you to a URL like /family-filter/off/. This suppresses that redirect by sending a user=ffilter: false cookie.

Add a test for such video.

@dstftw: done

Tests should be located here: https://github.com/Krenair/youtube-dl/blob/9d87eb84b9508b497372ee350985bb21afd5d550/youtube_dl/extractor/metacafe.py#L25-L110.


Multiple go clients

I am totally confused there seems to be two Go clients in existence:

github.com/tikv/client-go which you would think is the preferred and most up-to-date as it is part of the github.com/tikv project account
and
github.com/pingcap/tidb/store/tikv which is used in the tikv.org documentation linked to by https://github.com/tikv/tikv

Can you explain (maybe add to a readme on tikv/client-go) the difference, pros and cons of using either, clarify which is the recommended to be used.

github.com/tikv/client-go aims to provide a clean library to use tikv. The good side is less dependencies, and the bad side is that it does not have some new features and new interfaces added by tidb/store/tikv.


Replace custom attributes usage with a RedirectService

The DiscoveryNodeService and the ability to inject custom attributes is
gone in ES5.

hm, i think we should patch ES and add the attributes to the discovery node. because issuing a request for every node added might be a problem if the cluster is in a problematic state where a lot of cluster state changes are happening. let's discuss @mfussenegger


Convolutional Neural Net bug

I am trying to run the specific code on a pc with a Titan X gpu:

from keras.models import Sequential
from keras.layers.core import Dense, Dropout, Activation, Flatten
from keras.layers.convolutional import Convolution2D, MaxPooling2D, MaxPooling1D, Convolution1D
from keras.optimizers import SGD
import numpy as np

X = [
[
[
[1,2,3],
[1,2,3],
[1,2,3],
[1,2,3]
]
]
]

Y = [
[1,2,3,4,5]
]

X = np.array(X)

Y = np.array(Y)

Conv_size = 10
Dense_size = 10

filters = 5
rows= 2
cols= X.shape[-1]

model = Sequential()
model.add(Convolution2D(filters, rows, cols, activation='sigmoid' , input_shape = X.shape[1:]))
model.add(MaxPooling2D(pool_size=(rows, cols)))
model.add(Flatten())
model.add(Dense(Dense_size, activation='sigmoid'))
model.add(Dropout(0.5))
model.add(Dense(5, activation='linear'))
model.compile(loss='rmse', optimizer='sgd')
model.fit(X, Y, nb_epoch=50)

and it yields the following error

>>> model.add(Dense(Dense_size, activation='sigmoid'))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/keras/layers/containers.py", line 68, in add
    self.layers[-1].set_previous(self.layers[-2])
  File "/usr/local/lib/python2.7/dist-packages/keras/layers/core.py", line 82, in set_previous
    assert self.input_ndim == len(layer.output_shape), ('Incompatible shapes: layer expected input with ndim=' +
  File "/usr/local/lib/python2.7/dist-packages/keras/layers/core.py", line 814, in output_shape
    '(got ' + str(input_shape[1:]) + '. '
Exception: The shape of the input to "Flatten" is not fully defined (got (5, 1, 0). Make sure to pass a complete "input_shape" or "batch_input_shape" argument to the first layer in your model.

also the versions are
pip show theano | grep Version
Metadata-Version: 1.1
Version: 0.8.0.dev0

pip show keras | grep Version
Metadata-Version: 1.1
Version: 0.3.1

When i run the same code on another pc it passes!
>>> model = Sequential()
>>> model.add(Convolution2D(filters, rows, cols, activation='sigmoid' , input_shape = X.shape[1:]))
>>> model.add(MaxPooling2D(pool_size=(rows, cols)))
>>> model.add(Flatten())
>>> model.add(Dense(Dense_size, activation='sigmoid'))
>>> model.add(Dropout(0.5))
>>> model.add(Dense(5, activation='linear'))
>>> model.compile(loss='rmse', optimizer='sgd')

>>> model.fit(X, Y, nb_epoch=5)
Epoch 1/5
1/1 [==============================] - 0s - loss: 3.4482
Epoch 2/5
1/1 [==============================] - 0s - loss: 3.7623
Epoch 3/5
1/1 [==============================] - 0s - loss: 3.7835
Epoch 4/5
1/1 [==============================] - 0s - loss: 3.8206
Epoch 5/5
1/1 [==============================] - 0s - loss: 4.4849
<keras.callbacks.History object at 0x5751610>

It is a machine without gpu and versions
pip show theano| grep Version
Metadata-Version: 1.1
Version: 0.7.0

pip show keras| grep Version
Metadata-Version: 2.0
Version: 0.3.0

What could i do in order to use such a good gpu ?
Should i fall back to keras 3.0 or is it a theano problem ?
Thank you in advance

You seem confused. The error message is clear enough:

Exception: The shape of the input to "Flatten" is not fully defined (got (5, 1, 0). Make sure to pass a complete "input_shape" or "batch_input_shape" argument to the first layer in your model.

So your Flatten layer is getting an input of shape (5, 1, 0), i.e. with 0 filters, which is not a valid input shape. Hence an error. You are pooling 0 columns of your previous input.

Hello Mr. Chollet
I understand the error printed.
The problem is that it should not be there.
If you observe the code the Flatten layer should not get this input.
Also i have posted the output of two different machines.
The one with the gpu yields an error.
The second without the gpu runs smoothly the same code and presents an output.

If you haven't solved it by now, its because you're pooling layer has completely shrunk the input. remove that and you should be fine

Have you find the solution?
I have the same problem. My code was running on my computer without GPU, but after I added a GPU, I got the same error.

This problem occurred when i used flatten after a pooling layer.
I had used pool_size in a wrong way.
I found out the solution carefully reading the documentation,
as well as using
model.layers[-1].get_weights()[0].shape
which was extremely helpful in order to place the appropriate values in pool_size.
Also try using dummy data created with numpy (or a really small subcorpus) just to make it compile.
Then use the full data set in hand.
I hope that helps.

Thank you for your reply.
My problem was that I had forgotten to change image_dim_ordering to TH when I changed the backend to theano.

@hadikazemi AHA! there is my answer. Thanks so much!


Add ros_bt_py technology info + examples

@sea-bass As you are the only one I know that tried ros_bt_py, do you think the README is sufficient for people who have not worked with it before? Should I put the examples from the presentation in the readme as well?

Otherwise, this is finished on my end.

I'm also still running somewhat frequently into the PyRoboSim display sync issue. I will take a look to see if I can improve this...

Problem 4
I think you're navigating the robot to the kitchen room instead of one of the actual locations (fridge/pantry), so my robot got stuck trying to place the bread0 object over and over again because it was not in the right location. Please take a look at that.

I think this specifically happens when the robot is holding the bread0 or butter0 object and, on the way to place it on the table, it has to go to the charger due to low battery. There may be a subtle behavior bug there.

Problem 4
I think you're navigating the robot to the kitchen room instead of one of the actual locations (fridge/pantry), so my robot got stuck trying to place the bread0 object over and over again because it was not in the right location. Please take a look at that.

I think this specifically happens when the robot is holding the bread0 or butter0 object and, on the way to place it on the table, it has to go to the charger due to low battery. There may be a subtle behavior bug there.

I briefly looked into this just now, and I think it is an issue with the interrupt during the execution of the path following. As far as I can see it flags the interrupted movement as successful and just continues with the placing after returning from the charger.
I will follow up on this tomorrow morning.

I briefly looked into this just now, and I think it is an issue with the interrupt during the execution of the path following. As far as I can see it flags the interrupted movement as successful and just continues with the placing after returning from the charger.
I will follow up on this tomorrow morning.

Are you officially canceling the original ROS action, or just sending a new one? If it's the second, then I could see how this happens.

The examples all ran for me -- just a few things.
README / Instructions
I think we need some instructions on how to load trees from the ros_bt_py_pyrobosim package. It took me a while to figure out.

I added a section about loading the example trees. Hopefully, this is more clear now.

Also, generally the UX of this is interesting. It seems like if I select the wrong tree by accident by single-clicking, I have to hit "Back"? Also I get no visual confirmation on which tree I have actually selected.

I am aware that the UX needs some polishing. I will try to find some time to look into it,

Problems
Problem 1
Worked well!
Problem 2
Worked well. Suggest renaming PlanToDump (and similar) to say Dumpster instead of Dump, as for English speakers this could... be funny naming.

I have renamed the nodes.

Problem 3
Worked well. Question, is there a way to look inside the subtrees here while the trees are running? I couldn't figure out how to expand them.

I added a section about this to the README. You need to check the PublishSubtree box in the topbar and then you can inspect them via the Tree Drop Down above the editor.

Problem 4
I think you're navigating the robot to the kitchen room instead of one of the actual locations (fridge/pantry), so my robot got stuck trying to place the bread0 object over and over again because it was not in the right location. Please take a look at that.
I tried again and it seems it ran fine, but only partially. The bread and butter was brought to the table, but nothing was done with the waste/dumpster part of the objective. Maybe that is not implemented yet in this solution?

I indeed forgot to implement the dumpster part in the previous version. I added it alongside a fix for the issues regarding the recharging being triggered while holding an object.

... after the lint fixes are resolved.


[AutoPR cognitiveservices/data-plane/LUIS/Runtime/cognitiveservices/data-plane/LUIS/Authoring] LUIS Python conf

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3455

(message created by the CI based on PR content)

Replaced by https://github.com/Azure/azure-sdk-for-python/pull/2995


update sts to bring in a couple fixes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 41.816%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
30265

Relevant Lines:
67026

💛  - Coveralls


Keybase UI does not start

Keybase UI will not start on my miix-320 running pop! linux 19.10
my log id: 54dd330547e546d45be0f51c

Sorry, your log didn't have too much information in it.
What version of keybase do you have? keybase version.
Is the service running? Can you login and use keybase chat ls?
What happens if you run /opt/keybase/Keybase directly to start the gui?


Maybe a Wiki entry (like: dont build external libs as standalone) is usefully?

Don't try to build jpeg in engine-x alone, because I modify the cmake of it, and have many config in external/CMakeLists.txt

Maybe a Wiki entry (like: dont build external libs as standalone) is usefully?

I think no longer on the focus.


Modified Korean translations more natural

[V] Have you followed the contributing guidelines?
[V] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Oh i didn't know that i have to change heading title.
I'll change it! Thank you

I don't know why External Link 403 Error occurring for 'https://www.drupal.org/community-initiatives/drupal-core/usability'
How can i solve this error?

I think the dupal.org site must be doing some kind of throttling, but I can't tell what -- the site works for me from a web browser or wget, but not curl or tests, and changing the user agent doesn't help. So ignoring for now.
@pilgwon I noticed there's an older PR in progress to improve the Korean translation, #709. I haven't looked to see whether your updates touch the same material as that one. Maybe you can review #709 and/or @reverince can review this one. I guess that this one might be smaller in scope and could be either its branch could be merged into #709 or merged into master quickly?

Oh I don't know that translation in korean project is wip.
I'll close and wait for that project!


Confusing directory structure for lab users

Exercise 1.6
- very confusing on the wording of where you are supposed to be after everything else being done in the “networking_workshop” folder, confused people with the “test” folder structure. Perhaps reword or make it more clear the change in the directory.

I think I also fixed this with PR https://github.com/network-automation/linklight/commit/370e3ce72796b9a5fefaab10a42ec19e4f513cd5
can you check and make sure this is good?


support for custom checkboxes in multiselect

Support custom checkboxes in multiselect. Is necessary a customComponent in selectRow props.

@jfernandogt, that's cool, it's a todo item in v3.0.0. Thanks your contribution, I'll check this out :)

@jfernandogt, why to build the https://github.com/AllenFang/react-bootstrap-table/pull/627/commits/c6426cf5ff9a9fdf1b8f99f6978d22d40c36d37c ?

Released on v2.4.2. Thanks your contribution :)


gRPC streaming client WIP.

Looking for design/API level feedback on this approach to providing a gRPC streaming client for
Envoy to use with the v2 APIs. There are no tests (or even manual validation of behavior yet),
when we settle on design I can add these.
DO NOT MERGE.

@lizan @fengli79 @mattklein123

At a very high level (interface level), this approach LGTM.

Do you want to get the API closer to the gRPC async stream API?
https://github.com/grpc/grpc/blob/master/include/grpc%2B%2B/impl/codegen/async_stream.h

@lizan @qiwzhang

@fengli79 I don't think it makes sense to use the exact gRPC async stream API, as what I'm doing is closer to the Envoy model for callbacks (declaring a callback object rather than supplying individual callbacks). With that said, I think there's useful stuff in the interface you point to wrt stream termination, status/error handling and metadata that we can use to make the Grpc::AsyncClient interface a bit richer and closer in terms of capabilities.

@fengli79 @wattli @lizan @mattklein123 This is now ready for review.

@lizan Can you comment on the HTTP/gRPC async client timeout question above? If I don't get any further feedback, I'll merge and followup in a separate issue (in the interest of moving the v2 API implementation along).


Metric Webapp

I have completed the task of building the Metrics web app according to the stipulated requirements:
The following implementations were made: 🌱

[x] Created a single-page application (SPA) built with React and Redux.
[x] Connected to an API to display metric data on the home page
[x] Implemented the Home component to display a list of items from the API filtered by Air quality parameters.
[x] Implemented detail view to display more details about a specific metric index.
[x] Selected one main color for the Ui theme of the application
[x] Implemented search filter to Home and Detail Component
[x] Implemented a navigation button in the header to enable users to navigate to the home page.
[x] Implemented redux and redux store using redux-toolkit.
[x] Create integration tests for the application using the React Testing Library
[x] Setup redux store to ensure the data retrieved from the API is be stored in the Redux store
[x] Hosted the application and included the link in readme
[x] Recorded a presentation video for the  project and included the link in readme
[x] Created unit tests for pure functions in the Redux code.
[x] Created integration tests using react testing library and mocked the components.
[x] Recorded presentation video showcasing the file structure,objective,and special features.

In addition, I have ensured that this project fulfills the following default requirements :
✅ - There are no linter errors.
✅ - Used correct flow Gitflow.
✅ - Documented my work in a professional way.
✅ - Followed the list of the best practices for HTML & CSS and  JavaScript.
Please review my implementation and suggest any improvements and changes.
Thank you 🙏

I have completed the task of implementing the suggested changes as requested by the reviewer.
The following implementations were made: 🌱

[x] Attributed Nelson Sakwa to the thank you section in the readme file.
[x] Removed the useEffect hook from home and implemented the exact route of the details component
[x]Implemented a checker pattern to the cards to match the original Ui design layout.

In addition, I have ensured that this project fulfills the following default requirements :
✅ - There are no linter errors.
✅ - Used correct flow Gitflow.
✅ - Documented my work in a professional way.
✅ - Followed the list of the best practices for HTML & CSS and  JavaScript.
Please review my implementation and suggest any improvements and changes.
Thank you 🙏


Parser problem with IF

Now alasqlparser.jison file has a problem with grammar of IF operator (see the message below). Even with this warning all tests passed ok. How to modify grammar file to remove this warning?
    Conflict in grammar: multiple actions possible when lookahead token is ELSE in state 231
    - reduce by rule: If -> IF Expression AStatement
    - shift token (then go to state 381)

    States with conflicts:
    State 231
      If -> IF Expression AStatement .ElseStatement #lookaheads= GO SEMICOLON EOF END ELSE
      If -> IF Expression AStatement . #lookaheads= GO SEMICOLON EOF END ELSE
      ElseStatement -> .ELSE AStatement #lookaheads= EOF SEMICOLON GO END ELSE

Is this still a problem?

is this still relevant?

At least it works, so we can close it.

:+1:


Fixed setup when using ngrok

Fixed the env var name to properly set Agent's endpoints when using ngrok

Codecov Report

Merging #459 (60f764b) into main (f3790c9) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #459      +/-   ##
==========================================
- Coverage   85.67%   85.65%   -0.02%
==========================================
  Files         253      253
  Lines        5451     5467      +16
  Branches      839      878      +39
==========================================
+ Hits         4670     4683      +13
- Misses        781      783       +2
- Partials        0        1       +1

Impacted Files
Coverage Δ

...ckages/core/src/modules/routing/RecipientModule.ts
63.35% <0.00%> (-0.05%)
:arrow_down:

packages/core/src/agent/MessageSender.ts
71.68% <0.00%> (+0.25%)
:arrow_up:

...onnections/messages/ConnectionInvitationMessage.ts
98.03% <0.00%> (+0.26%)
:arrow_up:

packages/core/src/transport/WsOutboundTransport.ts
9.23% <0.00%> (+1.41%)
:arrow_up:

Continue to review full report at Codecov.

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


Add installation instructions for ubuntu

Figuring out exactly which packages to install wasn't trivial. I added some installation instructions to make it easier for others.
I haven't used it on Windows, so I'm not sure if there are any additional installation instructions that one needs to follow there.

Thank you so much for the PR!
I've merged it and will push these changes to the website now.

The changes are live now! Thank you once again!


generate html and sarif report

Hi
Is it possible to generate both a sarif and html report using this action?
Thanks

hi @javixeneize - today this is only possible if you run the action twice, one with SARIF and one with HTML format. We don't support multiple formats at once.

Hi @simar7. Could you show an example of how to generate a report in HTML format? I couldn't find any examples in the repo.


⚖ Register Validator:

Network Choice
dentrite-1
Who are you?
Lavender.Five Nodes
Delegator address
axone14qekdkj2nmmwea4ufg9n002a3pud23y8xyvu9d
gentx
{
  "body": {
    "messages": [
      {
        "@type": "/cosmos.staking.v1beta1.MsgCreateValidator",
        "description": {
          "moniker": " Lavender.Five Nodes 🐝",
          "identity": "F87ADDB700C0CC94",
          "website": "https://www.lavenderfive.com/",
          "security_contact"<EMAIL_ADDRESS>          "details": "Fortifying crypto networks with Horcrux security, slash insurance, and open source contributions. Connect with us at https://linktr.ee/lavenderfive."
        },
        "commission": {
          "rate": "0.050000000000000000",
          "max_rate": "0.200000000000000000",
          "max_change_rate": "0.050000000000000000"
        },
        "min_self_delegation": "1",
        "delegator_address": "",
        "validator_address": "axonevaloper14qekdkj2nmmwea4ufg9n002a3pud23y8k3qwl0",
        "pubkey": {
          "@type": "/cosmos.crypto.ed25519.PubKey",
          "key": "DJiHdnD2gZl5PpJCholQhYBor4CTtSfXPgrOD8MRygs="
        },
        "value": {
          "denom": "uaxone",
          "amount": "1000000000000"
        }
      }
    ],
    "memo": "7d610ad1f75857c01b45e8b6f25d73457a543611@<IP_ADDRESS>:26656",
    "timeout_height": "0",
    "extension_options": [],
    "non_critical_extension_options": []
  },
  "auth_info": {
    "signer_infos": [
      {
        "public_key": {
          "@type": "/cosmos.crypto.secp256k1.PubKey",
          "key": "AgDrjXJ1+/lwJdf+fekEXKwLpA30N7F/Z3ESi/Sn7Rw4"
        },
        "mode_info": {
          "single": {
            "mode": "SIGN_MODE_DIRECT"
          }
        },
        "sequence": "0"
      }
    ],
    "fee": {
      "amount": [],
      "gas_limit": "200000",
      "payer": "",
      "granter": ""
    },
    "tip": null
  },
  "signatures": [
    "SEVRqMSw0mW5GjUJO8nncsWn9dDLrWemucn1nrSJ+h1iOyCehh+wT1W82eI7KIBCtraJw4amwhJ1IGzApisdhA=="
  ]
}

😉 Here is the corresponding PR: https://github.com/axone-protocol/networks/pull/770

Done by https://github.com/axone-protocol/networks/pull/770


samples: sensor: bme280:  Demonstrate RP2040 PIO

Demonstrate use of the Raspberry Pi Pico PIO SPI driver.

@yonsch - Is this more in line with your recommendation?

@yonsch - Thanks for spotting that.  I also added a note to the Pico docs as you suggested.

@MaureenHelm - Being a relative newbie to Zephyr, I'm not familiar with the purpose or structure of the sample.yaml file.  Can you please point me to appropriate documentation on how those files are used and how to successfully make additions?
Thank you,
Steve

@MaureenHelm - Being a relative newbie to Zephyr, I'm not familiar with the purpose or structure of the sample.yaml file. Can you please point me to appropriate documentation on how those files are used and how to successfully make additions?
Thank you, Steve

You would need to add rpi_pico to the integration_platforms to make sure it's built in CI. Otherwise it'll get skipped.

@MaureenHelm - Being a relative newbie to Zephyr, I'm not familiar with the purpose or structure of the sample.yaml file. Can you please point me to appropriate documentation on how those files are used and how to successfully make additions?
Thank you, Steve

https://docs.zephyrproject.org/latest/develop/test/twister.html

@MaureenHelm @yperess - I added my best guess at test configurations to samples/sensor/bme280/sample.yaml.  Please let me know what additions are still needed.

@MaureenHelm @yperess - I updated with another version of the samples/sensor/bme280/sample.yaml.  Please let me know what additional changes are needed.

@yperess @yonsch - Can you please verify the changes I made in response to your review comments?  Thank you!


Page in pages?

Would it be possible to create a 'page in page'?
The goal here is to make a sub-page that opens when you click & hold a button where i can add more tiles and actions related directly to the parent (in my case I'm planning on making a TV Remote Control as a sub-page item).
Thanks,

No, but you can play with pop-up IFRAME.

No, but you can play with pop-up IFRAME.

This is what I'm currently doing, and integrating my HA Remote - It's not the most elegant solution so was wondering if there was a better alternative.
Thanks,


Bug 1827364: routes/ingress: add documentation for HTTP2 ALPN routes

Add some clarification/documentation around HTTP/2 enabled routes.

/cc @danehans

ner/v1alpha1/types.go': Operation not permitted
/test verify

@Miciah @danehans updated based on your feedback. Probably worth a clean sweep reread as it has been a while.

Rebased in https://github.com/openshift/api/commit/ff587549773485c486187a049b0101fb5db027db

/lgtm

/approve


Names can't be more than 1 word long

Version: v1.0.0
Issue: Command
Details: When running the v!user edit username command, the bot only recognizes the first word of a username. Usernames can be multiple words long so this should allow more than 1 word.

Should be easy to fix, we just need to combine all arguments after the first one.


Overloaded Function Signature: Can't delegate to function with same signature

TypeScript Version:  2.2.1
Code
interface NumberOption {
    returnNumber: true;
}

interface NoNumberOption {
    returnNumber?: false;
}

function eitherOuter(option: NumberOption): number;
function eitherOuter(option: NoNumberOption): string;
function eitherOuter(option: NumberOption | NoNumberOption = {}): string | number {
    // return eitherInner(option); // Does not compile
    if (option.returnNumber === true) {
        return eitherInner(option);
    } else {
        return eitherInner(option);
    }
}

function eitherInner(option: NumberOption): number;
function eitherInner(option: NoNumberOption): string;
function eitherInner(option: NumberOption | NoNumberOption = {}): string | number {
    if (option.returnNumber) {
        return 1;
    } else {
        return 's';
    }
}

const n = eitherOuter({ returnNumber: true });
const s1 = eitherOuter({ returnNumber: false });
const s2 = eitherOuter({});

Expected behavior:
I would expect to be able to delegate from eitherOuter to eitherInner without any special annotations.
Actual behavior:
Delegating from eitherOuter to eitherInner is not possible, instead the simplest workaround I was able to find was to write an if statement, which had the same statement in both branches.

Type Checking the implementation of eitherOuter fails because there is no overload declaration of eitherInner that takes NumberOption | NoNumberOption. It looks at the first declared overload and rejects it since it cannot except the union type. It rejects the second declared overload for the same reason.
Simplest solution
function eitherOuter(option: NumberOption): number;
function eitherOuter(option: NoNumberOption): string;
function eitherOuter(option = {}): string | number {
  return eitherInner(option); // Compiles
}

I am not seeing any errors on the latest drop. closing as fixed. please reopen if this is not the case.


Dependency to 'pytorch_pretrained_bert' left in train_extractive.py

Thanks for the amazing code repo ! 😃
In the dependencies list, you mentioned pytorch_transformers, so you moved to the new version of HuggingFace library.
However there is still a dependency to pytorch_pretrained_bert in train_extractive.py.

Sorry for my neglection, this has been fixed now


Spanish translation for Ecto lesson

This lesson is almost done, I just need to translate the following sections:

[x] Fragments
[x] Changesets

/cc @doomspork @erickgnavar

@doomspork My condolences!
No problem at all, review this when you can, I'm happy to help!
Happy new year for you too! :boom:

@doomspork My condolences!
No problem at all, review this when you can, I'm happy to help!
Happy new year for you too! :boom:

@erickgnavar how would you like to go about merging these lessons?  Are you almost finished with the basic ones and the configuration?

ping @erickgnavar :grinning:

Hi @doomspork, sorry for the late response, I would like to finish the basic lessons before doing the merge, I hope to finish in 2 days.

Thanks for the update @erickgnavar :+1:

Hi @erickgnavar!  I want to see whether you are be open to merging both this PR and your's (#43) even though the remainder of the Basics are not translated.  My thinking behind it is:

The Spanish language setup will be completed.  This unblocks other contributors who might be interested in lending a hand (see @milmazz / @ZuraGuerra)
I'd like to merge in @milmazz changes; I don't want PR rot to discourage contributors.

Open to your thoughts and feedback as well as others :grinning:
Thanks again for all your work

Sure, no problem, I have 6 lessons translated, you can merge with this PR.
I'm still working on the remaining lessons.

Thanks @erickgnavar.  I left you a comment on your PR RE: rebasing, everything else looks good.
@milmazz once I merge in #43 I'll merge and deploy this.  Thank you again.  Sorry for the delays.


Email authentication and verification Node.js and Firebase

Article checklist
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines
This fixes #1822

@phinaomondi Can I start reviewing this now (If the article is complete)?

Yes it's complete. @cyberShaw

@phinaomondi Can I start reviewing this now (If the article is complete)?

@cyberShaw  Yes you can, I completed the article yesterday.

Alright, I'll do the review now. 🚀
On Fri, May 28, 2021, 3:26 PM Phina Kersly @.***> wrote:

@phinaomondi https://github.com/phinaomondi Can I start reviewing this
now (If the article is complete)?
@cyberShaw https://github.com/cyberShaw Yes you can, I completed the
article yesterday.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/1996#issuecomment-850301838,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIF2IPM4OWCH4FUT44UWPU3TP5SFVANCNFSM42SWNEHQ
.

Thank you @cyberShaw

Thank you v much for your patience as we worked through our Repo re-structuring process.
The purpose behind this restructure - was to better allow us to handle more volume and allow students to publish more articles and get more involved with the community (with open-source contribution and suggestions).
With most of the changes coming to completion - we will be sure to get your article published asap.
Thank you again - and we are looking forward to more amazing articles.

plagiarism check done

Hey @cyberShaw. I made the following changes:

Fixed typing mistakes

Moved the article to the suggested directory

Kindly check them out. Thank you.

Hey @cyberShaw. I made the following changes:

Fixed typing mistakes
Moved the article to the suggested directory

Kindly check them out. Thank you.

Thanks for making the changes, we can now do the tech review

@cyberShaw  Fixed the changes and inserted the missing segment. Kindly check.

@cyberShaw Fixed the changes and inserted the missing segment. Kindly check.

Please add the extra section to the Table of Contents, please

@cyberShaw Fixed the changes and inserted the missing segment. Kindly check.

Please add the extra section to the Table of Contents, please

@cyberShaw Done!

@phinaomondi Thank you very much - the article is now ready to publish.
Please do be sure to mind the formatting (not sure why but many lines broke our page margins) perhaps the text was written in word pad or somewhere other than a text editor (my wild guess)
A few images were incorrectly named - I believe one is included and not needed??
Not sure.  we can add later if need be.
Go through all the suggestions made by your Peer reviewer and myself during my final to help speed publishing along and avoid potential future article closure. 🚀
Grear work - looking forward to more awesome tutorials.

cc @cyberShaw please do the same go through my final review - articles ready for final review should be more polished before the final label. 👍


CRUD how to see create/edit/delete ?

**I have generated a new module  works, with a fresh install (Laravel 5.7),  created the migration, executed the php crud command and all is fine. But when I publish the module, and migrate, the only thing I can see by clicking in the menu is the title of the module. **

To Reproduce
create a module, crud and publish
the route is defined as the following for a new module works
<?php

Route::group(['namespace' => 'Cms', 'middleware' => ['web']], function () {

/*
|--------------------------------------------------------------------------
| Work App Routes
|--------------------------------------------------------------------------
*/

Route::resource('works', 'WorksController', ['only' => ['show', 'index']]);

});

also the controller does not have the store/search/edit/destroy/history....
<?php

namespace Cms\Modules\Works\Controllers;

use Cms;
use CryptoService;
use Illuminate\Http\Request;
use App\Http\Controllers\Controller;
use Cms\Modules\Works\Services\WorkService;

class WorksController extends Controller
{
    public function __construct(WorkService $workService)
    {
        $this->service = $workService;
    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index(Request $request)
    {
        return view('works::works.index');
    }
}

How can I create/edit new records and delete ? Did I miss a step ?
I have read doc (a little light at this time) and searched but cannot find something about that.
Or is it a bug ?
Thanks for your support.

What command did you use? To get a full CRUD component you need to run artisan module:crud {name}

ok, same remark than for migration issue
I did try to follow doc and used as a first step the

php artisan module:make Toto
php artisan module:crud Toto
and the controller, after exception is catched, we see at the end is the one from the make command

Maybe the documentation is not too clear,
I do agree with only the crud command, all is running
I wiill continue to test... thanks for support !

php artisan module:make Toto this is intended as a non CRUD based module. You do not need to run both, its one or the other


Note box incorrect?

There is a purple note box that states "In order to successfully perform the last two actions, this permission needs to be granted at the resource group or subscription level."
Yet there 3 bullet points after it.

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

ID: c62e3725-b0dc-5ce5-c049-70fd3ae106e3
Version Independent ID: 9c0c5ef7-bd49-cb39-ca10-d98386004975
Content: Manage Log Analytics workspaces in Azure Monitor - Azure Monitor
Content Source: articles/azure-monitor/platform/manage-access.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@garybushey  Thanks for your comment. We are actively looking into it and will get back to you shortly.

@garybushey When a user is provided "Log Analytics Contributor" role at resource scope, then the user will be able to perform the operations mentioned in purple box as those permissions should be provided at resource group or subscription scope. Only for "Create and configure Automation accounts" and "Add and remove management solutions" , the permissions are to be provided at  resource group or subscription scope.
When a user is provided "Log Analytics Contributor" role at resource group or subscription scope, user will be perform all the operations as stated by in the role.
Hope this helps!
As there are no further changes to documentation, we will close the issue for now. Kindly revert if you have further questions.


Load server data?

I'm using basically the same server data on all my projects but everytime I make a new project or rename a folder, I have to re-input all my server data again and again and again...
Is there any way to store / load server data? would be a great feature in the server setup window!
Loving this extension otherwise!

I agree. Please add server profiles. This would be a great feature! Love your extension, keep up the good work.


[features request] Support for  Loading From Localhost

Microsoft new webview control (WebView2) build on chromium and support javascript SPA ( react, vue, angular ... )
it will be amazing if you use support webview2.
can't load from localhost server.

Hi,
to load from local host have a look at https://github.com/webview/webview_csharp and look at the end of the README.md. The part with NetIsolation had fixed that problem for me.
Greetings

Thanks, so there is no way to do it in Go ? loading from loaclhost !!

Ah sorry, I missed some information.
Go uses the same DLLs, so if you do the step with the NetIsolation it also works for Go based project on Windows.

Here's what I want to do. am serving website in localhost through the default port 80 and i want to load it.

OK, I served my files from inside the Go application but I think its the same issue.
In the webview/webview_csharp repo the following is mentioned:

HostedContent on Windows systems
The Edge webview uses a UWP application context on windows. UWP applications disallow loopbacks. For development purpose it is necessary to run the following command in an administrative command prompt:
CheckNetIsolation.exe LoopbackExempt -a -n="Microsoft.Win32WebViewHost_cw5n1h2txyewy"
This adds the Edge Webview Host to the exception list of this limitation. Your best bet for application distribution is to create an installer which executes this command on installation.

And because the DLLs used by Go are the same as used by CSharp you can apply the command, too.
Just close your application and run the following command in an elevated command prompt:
CheckNetIsolation.exe LoopbackExempt -a -n="Microsoft.Win32WebViewHost_cw5n1h2txyewy"
After that your application should be able to navigate to localhost / <IP_ADDRESS>.

@stevenkl  Thanks, it worked, also what about adding icon to the window ?

You can use Resource Hacker to change the icon of your windows executable.
I've also seen a go module/helper to change the icon on build, but currently can't remember the name.
You could also change the window icon programmatically but then you need some CGo code and I'm not able to provide an example for this, never tried it this way.

Thank you very much.

First that documentation is no longer there, Second the suggested:
CheckNetIsolation.exe LoopbackExempt -a -n="Microsoft.Win32WebViewHost_cw5n1h2txyewy"

Isn't a fix, it's an ugly workaround that makes it impossible to distribute the application. Thanks for nothing MS.

If you're willing to accept the "workaround" of my last comment, here's how you make it load an index.html file in the same directory as the executable:
#include "webview.h"
#include <iostream>
#include <filesystem>
#include <windows.h>

#ifdef _WIN32
int CALLBACK WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
#else
int main()
#endif
{
    webview::webview w(true, nullptr);
    w.set_title("Window");
    w.set_size(980, 650, WEBVIEW_HINT_NONE);
    w.set_size(980, 650, WEBVIEW_HINT_MIN);

    wchar_t rawPathName[MAX_PATH] = { 0 };
    GetModuleFileNameW(NULL, rawPathName, MAX_PATH);
    std::string executablePath = std::filesystem::path(rawPathName).parent_path().string();

    w.navigate(executablePath + "\\index.html");

    w.run();
    return 0;
}

Enjoy.


Improve lint-chromium to output actionable information

Before this commit, lint-chromium complained without an obvious course of action:

Warning: Pref objects doesn't have the same length.
Error: chromium/preferences_schema is not in sync

With this commit, the error message is more actionable:

Warning: extensions/chromium/preferences_schema.json does not contain entry for pref: enableFloatingToolbar
Error: chromium/preferences_schema is not in sync

Note: this is intentionally branched off an older version of the master branch that did not include #16605, to show the new effect in CI. The old result can be seen in https://github.com/mozilla/pdf.js/actions/runs/5369999621/jobs/9760078018?pr=16601 from PR #16601.

Good idea; thanks!


[Feature] adding option to configure FileStore root directory

Hi
I am using S3Mock for my integration tests. Sometimes on my workplace VM or on the test environment, housekeeping jobs deletes all contents in the /tmp folder. It will be a good feature if we could configure the FileStore root directory instead creating it in default construct.
Thanks,
Sujith

opened a pull request for this
https://github.com/adobe/S3Mock/pull/28
providing a builder for the classrule and the option to provide an argument(root) when running the application


Npm audit fix

What does this PR do ?
In this PR, we fixed the main vulnerabilities found by npm audit. We fixed the small bugs introduced by newly updated packages.
In the future, this practice should become normal boyscouting.
How should this be manually tested?
npm i should report 40 vulnerabilities instead of the ~140 previously found.

Codecov Report

Merging #420 into 2.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.x     #420   +/-   ##
=======================================
  Coverage   85.79%   85.79%
=======================================
  Files          38       38
  Lines         725      725
=======================================
  Hits          622      622
  Misses        103      103

Continue to review full report at Codecov.

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


Feat add attr decorator

What?
This adds a new behaviour as part of the @controller decorator; any property tagged with @attr will turn into a getter/setter mapped to an attribute with the equivalent data-* name, somewhat similar to how dataset works, but easier to use, more intuitive and with more type guarantees.
Why?
We want a way for controllers to easily enumerate the properties they expose for customisation. It's important to have these be enumerated in a way which:

Is easy and intuitive to use for developers; there's little that can go "wrong".
Avoids combinatorial complexity. Nothing to "configure" in of itself.
Has standard behaviours, so that  - prepared with the knowledge of the docs - seeing a property in a class definition lets you rely on the semantics you know to be true.
Aligns with our ideals for best practices in how to manage the configuration/state of an element.
Enhances Web Components ergonomics & behaviours, does not replace them.
Has an easy escape hatch; easy to rewrite code to drop out of decorators or drop out of using Catalyst entirely.

This is a effectively second attempt at https://github.com/github/catalyst/pull/63, which aimed to achieve a similar result but used entirely new semantics via properties with a custom observedProperties field, introducing a whole set of new mechanics to the library.
This PR builds on the lessons from https://github.com/github/catalyst/pull/63 and provides decorators which lean on existing semantics of Attributes, providing slight additional conveniences which make intuitive sense for a TypeScript codebase, such as better type-strictness. Having these lean on attributes also avoids any synchronisation issues between DOM and class state.
As this leans on attributes, this behaves similarly to native built-in HTML elements where they have properties that map to attributes, for example hidden or required for boolean attributes, maxlength for numerical attributes or placeholder for string based attributes. Attention has been paid to these example attributes to make sure our semantics align closely, so that developers can "intuit" the behaviours of these based on existing precedent in the web platform.
How?
Much of this is explained in the guide & code comments, but essentially: @attr "tags" a property (read: adds it to a WeakMap), and @controller calls initializeAttrs on the instance. initializeAttrs then iterates over all "tagged" properties, using Object.defineProperty to assign an appropriate get/set combo. The get/set combo maps the property name to a data attribute name and calls the necessary APIs to get/set the data so that it is directly synchronised with the elements attributes.

Great piece of work and cool feature

Great piece of work and cool feature


Closures that accept a generator do not seem repeatable

e.g. Closure(sum, tally(10))
when invoking such a closure via a Call action, the Result.value is 0 instead of 10.

Turns out things work as they should.
For a Closure with an iterator arg, the iterator should not be regenerated as it is a finite resource. If such a closure is used in a an Action, repeated performances should reflect depletion of the iterator arg of the Closure. If this behavior not be desired, then an interable can be used instead.

slated for v1.7.9 release


iot-operator doesn't watch for IoTConfig

https://github.com/EnMasseProject/enmasse/blob/2d3f74cf8ac7e97caa03d747d674c25a89e15b80/templates/iot/operator/050-Deployment-iot-operator.yaml
the iot-operator deployment doesn't set the env var CONTROLLER_ENABLE_IOT_CONFIG and therefore iot-operator doesn't watch for IoTConfig resources ...
https://github.com/EnMasseProject/enmasse/blob/2d3f74cf8ac7e97caa03d747d674c25a89e15b80/pkg/controller/add_iotconfig.go#L14-L15

The iot-operator should not watch that resource. The IoTConfig resource is processed by the enmasse-controller-manager. So I think that is fine.

in that case, the problem is in enmasse-operator because it isn't watching for IoTConfig too
https://github.com/EnMasseProject/enmasse/blob/2d3f74cf8ac7e97caa03d747d674c25a89e15b80/templates/enmasse-operator/050-Deployment-enmasse-operator.yaml

Shouldn't this be the responsibility of the iot-operator though?

No the "iot-operator" works cluster wide while the enmasse-controller-manager only works in the enmasse infrastructure project. And thus has different permissions.
It looks like @lulf deactivated the IoTConfig module here: https://github.com/EnMasseProject/enmasse/commit/5c25b06be27dff35d3396356bc0b96eb12971a48#diff-b5797c7936e90c9bb18dc959ef2a73c9L38

In combination with the IoT system tests being called wrongly in travis, no one noticed. I created PR #2488 which should fix calling the systemtests on travis.
So they should run into an error now. And then we can think how to fix this.

@ctron Hmm, perhaps we can grant iot-operator cluster-wide permissions to get watch iot configs as well? I would like to avoid depending on the IoTConfig CRD being installed for the enmasse-operator to work.

So whole cause for this was to limit the capabilities of the two different functionalities. IoTConfig is only valid for "enmasse-infra" and must create deployments, read/write secrets, … while the IoTProject part requires cluster wide access to only IoTProject, and Address, AddressSpace, MessagingUser.
However, the controller runtime can only run either namesspaced or non-namespaced … but not both in a single operator instance.
So mixing those two functionalities again would mean to set the watch namespace to cluster wide, and that would cause it to also watch for secrets cluster wide, which we did not want to grant.

Good point, I was a bit confused about the IoTConfig vs. IoTProject. We should move the IoTConfig CRD to 'templates/crds' then and add the necessary rules to enmasse-operator/20-Role-enmasse-operator.yaml


I want to read data from a firebase node and store it in a variable. Is this possible?

Yes, look at the FirebaseArduino.get methods: http://firebase-arduino.readthedocs.io/en/latest/#_CPPv2N15FirebaseArduino6getIntERK6String


Standard / Reference for this repo

Hey everyone!
I'm currently working on adding a Timed Metadata Track feature to this project and have been referring to the Apple QTFF documentation. However, I've noticed some inconsistencies (e.g. between the Apple Doc and MinfBox implementation in crate) between this document and the current state of the repo.
I saw in the README that the project references the ISO standards (that links to Wikipedia, which doesn't provide much useful information), which are often behind a paywall and thus not readily accessible for reference. Could you please provide some information about documents or resources that were referenced while developing this repo? It will be very helpful for aligning my work with this project.:)
Regards,
WERDXZ
PS: I'm using this repo for the QuickTime MOV part of creating Apple's Live Photo in rust, and here is a reference for it.

Unfortunately, the ISO standards are behind a paywall, so I can't link or upload them here directly. But if you search around on Google, you can find them easily.

Thank you very much


Updated version of Microsoft.Internals.Testplatform.Internals package…

Updated version of Microsoft.Internals.Testplatform.Internals package, fixed sign proj.

@harshjain2,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


examples: run as root or change ownership

The pmem-app.yaml files run the PMEM-CSI container with the pmem-csi user. That makes the /data directory in the example unusable.
We either must change the ownership of it or change the user.

@pohly As it is an example application, what about using other than pmem-csi container, say busybox?

That container then wouldn't have our fsdax check binary installed.

That container then wouldn't have our fsdax check binary installed.

Yes, but at least in our example, we do not mention using that binary. We mentioned running mount and df on the /data directory. As this is an example application I prefer we better use some non-pmem-csi image.

The ascii animation mentions it.

How is the busybox image better than our own?

The ascii animation mentions it.

The ascii animation mentions it.

Okay, got it.

How is the busybox image better than our own?

It is not about better, I felt like it is an example app better use some generic not-pmem-csi specific image to avoid any confusion for the user.

The pod name should make that clear. I am not even sure whether users will notice what image it is based on.

The ascii animation mentions it.

I don't want to make this issue any larger than it currently is, but we probably should rethink how we document the examples:

pull them out of the https://intel.github.io/pmem-csi/0.9/docs/install.html#install-from-source section, using URLs instead of relative file paths
document that the example apps are based on Debian and thus additional software can be installed with "apt update && apt get"
mention pmem-dax-check

I don't want to make this issue any larger than it currently is,

We could make the examples work as we currently documented by setting the setuid bit of the pmem-dax-check binary as in #937.

I think the cleanest solution is to change permissions on the /data mount. That mirrors best practices. It can be done like this:
diff --git a/deploy/common/pmem-app-generic-ephemeral.yaml b/deploy/common/pmem-app-generic-ephemeral.yaml
index 0eeccf9c7..d4aa4fa98 100644
--- a/deploy/common/pmem-app-generic-ephemeral.yaml
+++ b/deploy/common/pmem-app-generic-ephemeral.yaml
@@ -6,6 +6,8 @@ apiVersion: v1
 metadata:
   name: my-csi-app-inline-volume
 spec:
+  securityContext:
+    fsGroup: 1000
   containers:
     - name: my-frontend
       image: intel/pmem-csi-driver-test:canary
pohly@pohly-desktop:/nvme/gopath/src/github.com/intel/pmem-csi$ kubectl exec -ti my-csi-app-inline-volume -- /bin/bash
pmem-csi@my-csi-app-inline-volume:/$ ls -l /data
total 16
drwxrws---. 2 root pmem-csi 16384 Apr 16 06:29 lost+found
pmem-csi@my-csi-app-inline-volume:/$ pmem-dax-check /data/test-file
/data/test-file: supports MAP_SYNC, content is zeroed: true

A short comment about the security context would be good, too:
# The container runs apps as user/group 1000/1000.
# This security context causes permissions of volume mounts
# to be adapted accordingly, see
# https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-pod

I think the cleanest solution is to change permissions on the /data mount. That mirrors best practices. It can be done like this:
diff --git a/deploy/common/pmem-app-generic-ephemeral.yaml b/deploy/common/pmem-app-generic-ephemeral.yaml
index 0eeccf9c7..d4aa4fa98 100644
--- a/deploy/common/pmem-app-generic-ephemeral.yaml
+++ b/deploy/common/pmem-app-generic-ephemeral.yaml
@@ -6,6 +6,8 @@ apiVersion: v1
 metadata:
   name: my-csi-app-inline-volume
 spec:
+  securityContext:
+    fsGroup: 1000

I have initially tried with this solution, but the issue is that pmem-dax-check in our animation demo, tries to access /home/test-file to demonstrate that the non-pmem backed volume does not support MAP_SYNC. I couldn't figure out how to make make that work with the 'securityContext' solution:
$ kubectl exec my-csi-app-1 -- ls -dl /data
drwxrwsr-x. 3 root pmem-csi 4096 Apr 16 07:20 /data

$ kubectl exec my-csi-app-1 -- pmem-dax-check /data/test.txt
/data/test.txt: supports MAP_SYNC, content is zeroed: true

$ kubectl exec my-csi-app-1 -- ls -dl /home
drwxr-xr-x. 2 root root 4096 Mar 19 23:44 /home

$ kubectl exec my-csi-app-1 -- pmem-dax-check /home/test.txt
open /home/test.txt: permission denied
command terminated with exit code 2

Do you have any idea how to make it work even with pre-existed directories like /home.

tries to access /home/test-file

The demo should use /tmp/test-file instead of that.
I already told Eric that the animation must be updated for 0.9. He is currently waiting for this issue here to be resolved and a new 0.9.1 release with the fix backported.


Fix API link header

Both added test for link header generation and will fix #7296

Codecov Report

Merging #7298 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7298      +/-   ##
==========================================
+ Coverage   41.15%   41.21%   +0.05%
==========================================
  Files         464      464
  Lines       62763    62788      +25
==========================================
+ Hits        25833    25878      +45
+ Misses      33537    33520      -17
+ Partials     3393     3390       -3

Impacted Files
Coverage Δ

modules/context/api.go
74.78% <100%> (+19.47%)
:arrow_up:

routers/repo/view.go
43.25% <0%> (+1.01%)
:arrow_up:

modules/log/event.go
65.64% <0%> (+1.02%)
:arrow_up:

models/unit.go
67.56% <0%> (+5.4%)
:arrow_up:

Continue to review full report at Codecov.

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


impossible to run it on ubuntu 16.04?

Hi ,
is crnn tested on ubuntu 16.04 ? this is not worked on 16.04 .
TPHH library doesn't support 16.04 , this tested on 13.04 , 14.04.

same question

I can run all these dependencies on Ubuntu 16.04, but the crnn can not be built correctly.

Same issue :(

@umdreamer : Downgrading flualib didn't work for me unfortunately.

I see your new suggestions though. Will try them today. Thank you so much! @umdreamer


Integration between drone and Gitea (bleeding edge) erroring out on webhook

Gitea version (or commit ref): e6a97b6
Git version: 2.13.5
Operating system: centos (Docker)
Database (use [x]):

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

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Description
When I push a commit to a repo that has a webhook that is connected to drone the webhook fails. Checking the logs from gitea I see that drone fetches a URL from the API, but is returned a 404 (not found) error.
I've tested calling that route directly via browser and I also get a 404. However when I call /api/v1/repos/sdstack/atomic/ directly via my browser (when I am logged in) I get information about the repo, and when I am logged out (testing via incognito mode) I call the /api/v1/repos/sdstack/atomic/ URL and it gives me an expected 403 (as it is a private repo so I wouldn't want someone not logged in to be able to get that information).
Drone version: 0.8.1+build.1318
<IP_ADDRESS> - - [01/Nov/2017:20:39:28 +0300] "GET /api/v1/repos/sdstack/atomic/raw/6d31739044602484ef3b658270a973c5ebedb461/.drone.yml HTTP/1.1" 404 11349 "-" "Go-http-client/1.1" "-"

Just confirmed this happens on latest commit of gitea 122a66f
[Macaron] 2017-11-02 03:12:13: Started GET /api/v1/repos/whff/whff/raw/ad61956db6300cad2d0f72fbe4c5128f041c05dc/.drone.yml for <IP_ADDRESS>
[Macaron] 2017-11-02 03:12:13: Completed /api/v1/repos/whff/whff/raw/ad61956db6300cad2d0f72fbe4c5128f041c05dc/.drone.yml 404 Not Found in 9.807562ms

I would like to confirm that I have the same odd issue.

@LFlare it should not be related

@lafriks, I've tried using my front-end web proxy to dynamically rewrite the URLs on-the-go and from what I found out, accessing the API endpoint over web browser and having it rewrote works but having drone access the API endpoint to query for .drone.yml and having it rewrote does not work. Since Gitea responds to unauthenticated access to private repositories with a 404, I feel like it might have some relevance.
To that extent, I'm still scrolling through the code trying to figure out what changed recently that screwed up the integration.

@lafriks, @techknowlogick, this has been fixed in #2836.

is Gitea-1.2.3 immune from this ?

@strk yes it was broken on master

I upgraded Gitea to version 1.4.0+rc2-3-g5d0c987 but still see this in the logs:
[Macaron] 2018-03-12 09:05:48: Started GET /api/v1/repos/org/repo/raw/refs/pull/70/head/.drone.yml for <IP_ADDRESS>
[Macaron] 2018-03-12 09:05:48: Completed GET /api/v1/repos/org/repo/raw/refs/pull/70/head/.drone.yml 404 Not Found in 63.545961ms

I upgraded Gitea to version 1.9 but still see this in the logs:
[Macaron] 2019-09-05 08:12:33: Started GET /api/v1/repos/chaplin/xshop_new/raw/eaeb7e075e42001b97b89b348cd2df31156bf1aa/.drone.yml for <IP_ADDRESS>
[Macaron] 2019-09-05 08:12:33: Completed GET /api/v1/repos/chaplin/xshop_new/raw/eaeb7e075e42001b97b89b348cd2df31156bf1aa/.drone.yml 404 Not Found in 78.427818ms


Create changelog

Must be done before v1.0.0

I've been looking into Conventional Commits as a lot of the big boys (angular, aurelia, ng-bootstrap, etc) are using it as a system to automatically generate a changelog and bump their package version.
By using this approach via Standard Version our release process could be as follows:
npm run release - standard-version
npm run publish - git push --follow-tags origin master; npm publish
And we'd be done, that would auto-generate a changelog, bump the version number, and publish to npm/push to git.
It's well worth a read and I'd be up for trying it out. Commit messages on branches can still be whatever we like, as long as when the branch is merged it gets squashed into a single commit that is conventional, and any commits to master should be conventional.

Also, I've thought about creating new development branch where we can merge several PRs before merging changes in master and bumping the version. What do you think about it?

Yes gitflow works for me, master essentially being the production branch.


Texas HIV Pilot: Align with internal stakeholders on how/when to approach Texas

Acceptance criteria

[ ] Identify strategy for approaching Texas (work with Glenn, Audrey, Jillian, Chris)

[ ] What will we tell them
[ ] In what order will we tell them
[ ] What are the questions/feedback we want to get from Texas
[ ] How to handle any questions we may get from them

[ ] Identify and create materials needed (if any) for the conversation with Texas
[ ] Determine/schedule (with Glenn) time to meet with Texas
[ ] Share with Max for awareness and any feedback

Meeting schedule with Jillian and Audrey for May 8th to develop initial strategy ideas as Chris and Glenn are OOO. A meeting has been schedule with all of us on May 13th to finalize plan before meeting with Texas on the 15th.

Will span to following sprint

Link to meeting plan


You are using ScoreCAM with target layers, however ScoreCAM will ignore them is occured

Thank you for your work, When i run swinT_example.py, Warning: You are using ScoreCAM with target layers, however ScoreCAM will ignore them is occured, what is  the meaning of it? looking forward to your reply!

Hi, this can be ignored.
I will remove the warning in the next version.


Can't sort after filter

grimmaple95 (@GrimMaple) reported this on 2023-07-18T15:16:42Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=24048
CC List

nick (@ntrel)

Description
This code fails to compile

```
auto a = [1, 2, 3, 4, 5, 1, 2, 3, 4, 5];
a.filter!(x => x > 2).sort!((x, y) => x > y).writeln;
```

because you can't sort over filter result. Adding `.array` before `.sort` "fixes" the problem. Probably, it should work without the `array` inbetween

grimmaple95 commented on 2023-07-18T19:23:35Z
(In reply to Nick Treleaven from comment #1)
> filter returns an input range.
> sort requires a random access range and works in place and has good time
> complexity.
> How do you want to sort an input range?

As a user, I shouldn't care about what `filter` returns. I should care about getting a filtered & sorted array/range.

As for `sort`, in my user opinion, if it can't do its job "in place", then it could as much do it out-of-place, because this code

```
auto a = [1, 2, 3, 4, 5, 1, 2, 3, 4, 5];
a.sort!((x, y) => x > y).writeln;
```

fails to compile just as much, and it's staright up inconvenient

dfj1esp02 commented on 2023-07-18T18:54:33Z
It would do inputRange.array.sort!predicate, that's how sort usually works on input ranges.

grimmaple95 commented on 2023-07-19T07:20:32Z
(In reply to Grim Maple from comment #3)
> (In reply to Nick Treleaven from comment #1)
> > filter returns an input range.
> > sort requires a random access range and works in place and has good time
> > complexity.
> > How do you want to sort an input range?
>
> As a user, I shouldn't care about what `filter` returns. I should care about
> getting a filtered & sorted array/range.
>
> As for `sort`, in my user opinion, if it can't do its job "in place", then
> it could as much do it out-of-place, because this code
>
> ```
> auto a = [1, 2, 3, 4, 5, 1, 2, 3, 4, 5];
> a.sort!((x, y) => x > y).writeln;
> ```
>
> fails to compile just as much, and it's staright up inconvenient

I meant to change `auto a` to `immutable a`. Then it won't compile, but should

nick (@ntrel) commented on 2023-07-18T17:16:16Z
filter returns an input range.
sort requires a random access range and works in place and has good time complexity.
How do you want to sort an input range?


Add Base-64 encoding/decoding to GREL

Original author<EMAIL_ADDRESS>(September 30, 2010 15:41:25)
Would be nice to be able to perform base64 encoding/decoding directly in GREL
Original issue: http://code.google.com/p/google-refine/issues/detail?id=148

Rather than implementing this as it's own function, it might be a good idea to introduce a pair of encode() / decode() functions which take an algorithm as an argument. e.g. "foo".encode("base64")

@tfmorris Totally agree.  I can envision how that proposed pair of encode() / decode() functions would be similar in syntax with how we did the escape() / unescape functions.


Translation CURIEs (XML) to bare string notation (JSON/JSON/LD) for standard values

Dear @sboeckelmann ,
Im using your (great) web application, I just noticed that if an XML document makes use of CURIEs in e.g. the bizStep field, the same value is taken over in the JSON/JSON LD document as well. At the same time, a JSON/JSON-LD doc, IMHO, should only contain the corresponding bare string notation for all CBV standard values. Interestingly, it works for type in the sensorReport field: in this case, the tool correctly translates the value from 'gs1:Dimensionless' to 'Dimensionless'. But e.g. for 'cbv:Disp-in_progress', it leaves the value as is.
AFAIK, we never provided message examples for this case, so I am happy to provide the following exemplary EPCIS XML Doc:
<?xml version="1.0" encoding="UTF-8"?>
<epcis:EPCISDocument xmlns:epcis="urn:epcglobal:epcis:xsd:2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:gs1="https://gs1.org/voc/" xmlns:cbv="https://ref.gs1.org/cbv/" xmlns:ext1="https://ns.ext.de/epcis/" schemaVersion="2.0" creationDate="2023-01-11T12:00:00.000+01:00" xsi:schemaLocation="urn:epcglobal:epcis:xsd:2 EPCglobal-epcis-2_0.xsd">
	<EPCISBody>
		<EventList>
			<ObjectEvent>
				<eventTime>2023-01-11T12:00:00+01:00</eventTime>
				<eventTimeZoneOffset>+01:00</eventTimeZoneOffset>
				<certificationInfo>https://accreditation-council.example.org/certificate/ABC12345</certificationInfo>
				<epcList>
					<epc>https://id.gs1.de/01/04012345999990/21/XYZ-1234</epc>
				</epcList>
				<action>OBSERVE</action>
				<bizStep>cbv:BizStep-receiving</bizStep>
				<disposition>cbv:Disp-in_progress</disposition>
				<readPoint>
					<id>https://id.gs1.de/414/4012345000115</id>
				</readPoint>
				<sensorElementList>
					<sensorElement>
						<sensorMetadata ext1:someFurtherMetadata="metadataText" time="2023-01-11T11:05:00Z" bizRules="https://id.example.com/253/4012345000054987"/>
						<sensorReport exception="gs1:ALARM_CONDITION" coordinateReferenceSystem="http://www.opengis.net/def/crs/EPSG/0/27700" ext1:someFurtherReportData="reportText" type="gs1:Dimensionless"/>
					</sensorElement>
				</sensorElementList>
			</ObjectEvent>
		</EventList>
	</EPCISBody>
</epcis:EPCISDocument>

Would you agree that it make ssense to adjust this behaviour?
Kind regards,
@RalphTro

Dear @RalphTro, thank you so much for testing, this really helps us a lot to improve our code-base!
@Aravinda93 will pick it up immediately 😃

@RalphTro
Thanks for reporting the issue, I just analysed the code and found that its happening because we have only taken care for the CBV formatted urn that starts with urn:epcglobal:cbv:*** hence we were not supporting the cbv:***- but we are currently in process of adding them as well.
During the analysing of your XML I noticed that when converting to JSON the sensorReport -> exception in converted JSON appears as gs1:ALARM_CONDITION should it also be converted to barestring something like ALARM_CONDITION similar to how we are handling the sensorReport->type field?

Dear @Aravinda93 ,
Many thanks for taking a look into this matter. Turning to your question: That is at least my understanding given that they are embedded in the EPCIS JSON schema in a simlar fashion (see
https://ref.gs1.org/standards/epcis/epcis-json-schema.json). Do you share my POV?
Kind regards,
Ralph

@Aravinda93
I remember that you changed voc to cbv with one of your latest changes on the digitallink converter and I want to make sure that we get the vocab stuff right.
https://ref.gs1.org/standards/epcis/epcis-context.jsonld:
"cbv": "https://ref.gs1.org/cbv/"
"gs1": "https://gs1.org/voc/"

CBV - Core Business Vocabulary
https://ref.gs1.org/cbv/
see https://ref.gs1.org/standards/cbv/ 3.3.1 (page 27/28)
https://ref.gs1.org/cbv/X-Y -> urn:epcglobal:cbv:X:Y
with JSON-LD using the shortened Y-only expression
see bizStep defined in https://ref.gs1.org/standards/epcis/epcis-context.jsonld
GS1 Web Vocabulary
https://www.gs1.org/voc/
see https://ref.gs1.org/standards/cbv/ 7.6.1 (page 55)
https://www.gs1.org/voc/X -> gs1:X
@Aravinda93
can you please double check if we got all of this right, also in the digitallink converter?

Dear @RalphTro
The reported issue has been resolved and the fix has been successfully deployed to the web application. The updated functionality is now available for use. However, it appears that we may have missed informing of this update. We apologize for the same.
After implementing the fix, when the XML provided by you above is input into the system, we are now able to generate the following JSON output:
{
  "@context": [
    "https://ref.gs1.org/standards/epcis/2.0.0/epcis-context.jsonld",
    {
      "ext1": "https://ns.ext.de/epcis/"
    }
  ],
  "type": "EPCISDocument",
  "schemaVersion": "2.0",
  "creationDate": "2023-01-11T12:00:00.000+01:00",
  "epcisBody": {
    "eventList": [
      {
        "type": "ObjectEvent",
        "eventTime": "2023-01-11T12:00:00+01:00",
        "eventTimeZoneOffset": "+01:00",
        "certificationInfo": "https://accreditation-council.example.org/certificate/ABC12345",
        "epcList": [
          "https://id.gs1.de/01/04012345999990/21/XYZ-1234"
        ],
        "action": "OBSERVE",
        "bizStep": "receiving",
        "disposition": "in_progress",
        "readPoint": {
          "id": "https://id.gs1.de/414/4012345000115"
        },
        "sensorElementList": [
          {
            "sensorMetadata": {
              "time": "2023-01-11T11:05:00Z",
              "bizRules": "https://id.example.com/253/4012345000054987",
              "ext1:someFurtherMetadata": "metadataText"
            },
            "sensorReport": [
              {
                "type": "Dimensionless",
                "exception": "gs1:ALARM_CONDITION",
                "coordinateReferenceSystem": "http://www.opengis.net/def/crs/EPSG/0/27700",
                "ext1:someFurtherReportData": "reportText"
              }
            ]
          }
        ]
      }
    ]
  }
}

If the issue has been resolved, could you please confirm the same so that we may proceed with closing this ticket as resolved? Thank you.
Kind regards,
Aravinda

Dear @Aravinda93 and @sboeckelmann ,
Many thanks for adjusting this! Happy to close this issue.
Kind regards,
Ralph


generate kube with volumes

Add volume functionality for generate kube, including adding the fields for containers, as well as the pod. Test added as well
Fixes: https://github.com/containers/libpod/issues/2303

/approve

/test images

Changes LGTM

@baude @jwhonce PTAL

@baude @jwhonce @rhatdan @TomSweeneyRedHat @vrothberg  Can we get this landed? I'm messing around in Inspect and I don't want to make merge conflicts...

Changes LGTM, if you're happy with the one question you had @mheon, feel free to merge it.
At some point, I'd like to see some expanded doc covering this new functionality.

/lgtm


Decompiler creates "continue label;"

Highly mangled loop structure with other elements makes continues with unknown closures. https://gist.github.com/SuperCoder7979/804959ce7745dc2aba57a99d9e3dc007 compiled with JDK 17.

Edited with more information- Turns out if you hit ctrl+shift+enter when writing an issue GitHub will immediately send it out.


Fixing invalid query building on dimensions with aliased join keys

Summary
During query building, joining in dimensions with an aliased join key should use the alias directly, rather than constructing an invalid join ON with the AS clause. This fix "promotes" the alias name to the actual join column name, which results in a valid query.
Test Plan
Wrote unit tests. Also tested on some actual queries running locally and they're generating runnable queries.

[X] PR has an associated issue: #438 438
[X] make check passes
[X] make test shows 100% unit test coverage

Deployment Plan
N/A

Oh wait I was going to add more tests... oh well, different PR.

Whoops 😅


Add the page structure and all functionalities.

Hello!

In this Branch I implemented the following things:-

Add todo list components (Form, add, and todo list js files)
Add event handlers.
Implement CSS in the App.css file.
Use react hooks like useState and useEffect to handle states.

Project Approved :trophy: :tada: 🟢
Hi @Besufekad-HAZ
Your project is complete!  You have done it exceptionally well. it's time to merge it :100:
Congratulations! 🎉
To Highlight :+1:
:heavy_check_mark: No linters error.
:heavy_check_mark: Details PR title and description.
:heavy_check_mark: Good commit messages.
Cheers and Happy coding!👏👏👏

Thanks for the review 🙏🙏....


ISSUE-993 RollingWindow Join backend bugs

tested. Works correctly with latest UI changes.  @shahsank3t  please go ahead and merge this.


Production deploys version in advance of what's been released to it

https://deployment.biosimulations.org/applications/biosimulations-prod shows v9.13.1 even though this hasn't been released. Critically, this could be out of sync with the frontend. Not sure if this comes from deleting the HSDS and resyncing.

I am not seeing the issue.
The message on the top is simply the latest commit message of the git repo that has been synced.
The message states that dev was updated to 9.13.1. The actual version of the deployed pods in prod is older


Nothing bites.

Stops running if the hook result is "Nothing bites." during a mooch.

Same issue - I'd flag it as a bug.


Improvement to delete all Tracking Cookies (Local Storage, Channel ID etc.)

I really love your Chrome extension approach, unfortunately it doesn't help at all because most big intrusive services track you down much deeper (they already know, Cookies are out)
Please have a look in chrome://settings/cookies on your own Cookies (i call Local Storage stuff Cookies as well) which are not deleted by your Cookie AutoDelete

Nice GIF. ;)
I wrote this in the description of the extension page:

This Extension can't clear LocalStorage yet
See:
https://bugs.chromium.org/p/chromium/issues/detail?id=78093

What I'm looking for is an API that selectively clears site data by hostname. The only APIs is browsingdata which is an all or nothing approach by time frame which is not what I'm looking for. There's probably extensions that already use that API
FWIW, it seems like that bug might never get fixed in Chrome. Firefox might implement the API that I need sooner than Chrome.

@mrdokenny 1st I would like to thank you for doing this, really. :)
2nd: is there any ETA known for Firefox implementation? I don't mean your ETA, but Mozilla's. ;)

@crssi

Thanks!
Not sure about ETA since this bug is P5.

I made #44 to discuss and deal with this issue. Please direct future discussion over there.