release 1.1.0

GeoLambda is now a runtime agnostic Docker image, but there is a Python example project.
On a merge to master:

Docker image built and pushed to docker hub developmentseed/geolambda:${VERSION}
Example python36 build Docker image built and pushed to developmentseed/geolambda:${VERSION}-python36
Public Lambda layers are automatically deployed to us-east-1, us-west-2, and eu-central-1. They can be used with any Lambda runtime. See the README for the ARNs

Thanks @vincentsarago , one thing I was wondering about is the http2 support. It seems to have built properly, but the nghttp2 library wasn't needed in the deploy package so I wasn't sure if it was included statically.
What's the best way to test that http2 support is working?

@matthewhanson
$ ldd libgdal.so | grep "http2"
	libnghttp2.so.14 => /usr/local/lib/libnghttp2.so.14 (0x00007fc219c2c000)

GDAL seems to be built correctly and will work for the lambda layer. For packaging, it will not work because AWS Lambda will fetch libcurl pre-installed in the image (AWS AMI) (ref: https://github.com/RemotePixel/amazonlinux-gdal/issues/7)
To solve this we need to hardcode the libcurl path (and to other lib) inside goal shared lib but then it can makes things a bit difficult (I haven't find the way to make the hardcoded path relative).

ah, read the issue, need to use rpath, ugh.
However, you should be able to set rpath to $ORIGIN which will look for the library in the same directory as the parent lib.

@matthewhanson I tried to use $ORIGIN without success sadly but I'll put that on my unix skills being 👎

changing this to a release candidate (1.1.0rc2) and merging
gdal updated to use rpath to find correct libraries included in the deployment package


If add new files to sln, did any script to generate vcxproj file？

Description
If add new files to visual studio sln, is any script to generate vcxproj file？

This question doesn't make any sense, files aren't added to the solution they are added to the project, and projects are included by the solution.
To be clear, none of the visual studio projects or the solution itself are generated with scripts. They are all managed by visual studio itself.


Update boilerplate

Depends on #126
I have made a new project using vuejs-templates/webpack and synced the created configuration with the configuration of this project.
Most significant changes:

Webpack 3 instead of Webpack 2
Jest instead of Mocha, Chai, and Sinon
eslint-plugin-vue instead of eslint-plugin-html

Bundle before:

vendor.js parsed size: 7.13mb
app.js parsed size: 391.83kb
After:

vendor.js parsed size: 2.74mb :tada:
app.js parsed size: 185.18kb

Is build/logo.png needed?

build/logo.png is for notifications produced by node-notifier. I'm not sure if it is useful, would like to test.


fix: update vendor and add build check

Signed-off-by: zhuangqh<EMAIL_ADDRESS>
update vendor
create building check in circleci

LGTM
CI was not tricked, weird!


Cairo 0 artifact assertion is failing

Currently the newly added tests from PR #380 are failing (see PR description for more info and links).
Your task, should you choose to accept it, is to find out why the tests are failing, and do one of the following:

Write an explanatory report as a comment on this issue.
Create a PR with an explanation in the description. The PR should target the branch expand-get-class-test (used by #380)

Hello! I want to investigate this one. Can you please assign it to me?

@kfastov Hey, any update on this?


Updating layer validations

Layer Validation rules

if a layer is specified in a package.yml, it must be one of the layers defined in packwerk.yml
if enforce_architecture is true or strict, a layer must be specified
a packages' dependencies should not be used in layer validations. check will find layer violations

Looks good @perryqh but I had some questions. I still think we should validate dependencies do not contradict the architecture rules.
I can see the argument that if:

Unnecessary dependencies are caught by the unnecessary dependencies check
If it's not unnecessary, the reference checking will find the violation

I just think it makes the constraints of the architecture checker slightly more coupled to using the unnecessary dependencies to be fully valid. Right now, that's at the expense of potentially slightly duplicate feedback (i.e. the user sees check AND validate fail).
However, I think packs should move toward merging check and validate into one call. IMO they are conceptually identical from a UX POV and the only reason they were separate was because packwerk was originally too slow to do them both at the same time. However, I don't see why they couldn't just be a single call, what do you think? If we had that, then the architecture checker can de-duplicate its messages before delivering them to the user.

Agree that combining check and validate could be beneficial. Maybe we could support a new options to validate? pks validate --include-check or something similar.

Agree that combining check and validate could be beneficial. Maybe we could support new options to validate? pks validate --include-check or something similar.

Yeah I'm not sure 🤔 I think any system/person that is using packs in a codebase that uses packwerk definitely wants it to give them the correct set of things that are correct and incorrect, but I think as a user I'd prefer if check just did everything and if I want packs to not validate, it should be the other way around, e.g. --no-packwerk-validate or something that shims the checks that relate to validate in packwerk OUT of packs.
But not something we need to do right now :)

@perryqh Merged this PR.
I don't 100% agree with not having the architecture checker check the list of dependencies, but since that only causes a behavioral difference when there are no references (otherwise it would result in an architecture violation) and there's the "unnecessary violations check" as a stop gap, it's not that important to me.


removeUnusedAtRules doesn't work when there are multiple rules per font

Minimal example:
@font-face {
    font-family: font;
    font-weight: regular;
}
@font-face {
    font-family: font;
    font-weight: bold;
}

This will minify to:
@font-face{font-family:font;font-weight:regular}

Which is wrong.

It appears to be an error in this function: https://github.com/jakubpawlowicz/clean-css/blob/f273443f3d697764777e4618121d780c6894a922/lib/optimizer/level-2/remove-unused-at-rules.js#L112
The check for used font-family rules doesn't check if the block is itself a @font-face block.

👍 it should be checked that way indeed, thanks for digging for the solution

Fixed in 4.1.4!

Thank you!


Sprint 4 Team Issue

Book Reviewing and Discussion Database
Lit Connect
Our project allows users to interact with various books, rate them out of 5 stars, and have discussions with other young scholars in a chat box. The purpose is to understand different perspectives on books.
Repositories

Frontend: sprint4_flocker_frontend
Backend: sprint4_flocker_backend

Features

Home Page: Prompts users to log in before they can access the book reviewing and discussion features.
User Profile Adjustments: Users can customize profile details, including password, username, name, profile picture, and screen name.
Book Reviews: Users can view summaries and rate books up for review.
Chat Room: Provides a discussion space for readers to share opinions on books and engage with others.

Team Roles and Responsibilities
Trio 1 - Home Page, User Profile

Ahaan Vaidyanathan (Scrum Master)

Main page layout
Blogging features
Main page styling

Arnav Mital (Scrummer 1)

User login and logout functions
Profile customization

Trio 2 - Product Page / Backend Development

Noah (Assistant Scrum Master)

Kanban board management
Backend SQLAlchemy data definition and initialization

Jacob (Scrummer 2)

AI integration for book recommendations
Book reviewing features and book database

Francis (Scrummer 3)

Chat/discussion integration with backend data
Styling for a cozy book-themed discussion area
Programming and styling a page to announce upcoming books

Next Steps

[ ] To-do 1: Begin backend programming for data processing and chat functionality
[ ] To-do 2: Start frontend programming for the main page and profile page

Books is a good topic.   It is something that with a Class or a Club you could get a topic or some activity.   I am not sure how you select a book or how you would get audience.  Often time when we start a topic we want to design a verbal and then a visual flow of what we want to do.  You don't have that.
Establish aa admin or moderator for a book.  The moderator publishes book and writes a summary of what they thought about it.  The rate the book.  They then invite or advertise the book to others to request opinions by a certain data.   This could be broken down into several dates to get opinions and discussion as group is reading the book.    The moderator and other invitees could comment on the participants comments to generate discussion.   Perhaps the moderator could be a teacher, or just a student that is trying to generate interest in an assigned book.
You will need a lot more detail planning to figure out how this will work.
.88


IntelliJ instructions incorrect

Search for the official Sponge API Maven artifact under Search by artifact, and click Add.

Doing this only results in Spongeapi:1.0 being displayed in the list, either the newer versions of spongeapi need to be added to the default repositories, or alternate instructions need to be added on this page, https://docs.spongepowered.org/en/plugin/basics/workspace/idea.html#gradle

Isn't this a Docs issue?

Yes, posted to the wrong issue tracker somehow. However doesn't it rely on re=publishing the API to fix if it's intended to be on the maven repo?
Moving to docs.


[Feature Request] Set the order for the labels

It would be insanely useful to be able to tel ipyplot the order of the categories.
For example, I have attempting to print a list of books ordered by age. Unfortunately, even after sorting both the array of file paths and the array of labels, the pictures still come out in the wrong order.
To be clear, what I'm asking for is this:
2, 3, 4, 5, 6, 7 8, 9, 10, 11, 12, 13, 14, 15, 16, 17
instead of this:
10, 11, 12, 13, 14, 15, 16, 17, 2, 3, 4, 5, 6, 7, 8, 9

Thank you once again for this amazing tool :smile:

Hi @educatorsRlearners
Most likely your problem is coming from the fact that plot_class_representations is using numpy.unique function under the hood. And numpy.unique is always sorting the results.
Now in case of strings, 12 will be sorted before 2 and that's our problem.
Here's a simple example:
# for numerical labels as strings:
np.unique(["1", "2","12"])
OUT: array(['1', '12', '2'], dtype='<U2')

# for numercial labels as ints:
np.unique([1, 12, 2])
OUT: array([ 1,  2, 12])

So as you can see the simplest workaround for your case would be to convert your labels from string to integers to keep the correct order.
Nevertheless I hear your voice and I think it makes perfect sense to add a parameter for plot_class_representations which would help to force a specific order of labels. The same features should also be available for plot_class_tabs function to define the order of tabs to be displayed.
Thanks for your feedback! Glad you like IPyPlot!

HI @karolzak,
"So as you can see the simplest workaround for your case would be to convert your labels from string to integers to keep the correct order."
Now I thought for sure I had tried to do that only to realize I hadn't :man_facepalming:
Thank you so much and I"ll be sure to link to the project when I publish my post. Cheers!

Glad the workaround did it for you!  Nevertheless I still want to add the possibility to force some specific order for classes both in plot_class_representations and plot_class_tabs so I'll keep this issue open until it's implemented, if you dont mind :)


update readme for easier iOS devilment setting

sorry for the late reply.
I think the readme should be updated as well.
As a pure iOS dev, I don't really need the per-project setting. Single global settings is good enough.
The current readme is really scary for the new users (like the guy who wrote https://funnyitworkedlasttime.dev/posts/2020-01-10-swift-vscode-sourcekit-lsp/)
Besides, the original example only works for sourcekite. The sourcekit-lsp implementation doesn't read swift.targets.compilerArguments.
At this moment, it's not possible for sourcekit-lsp to support iOS and WatchOS in the same project unless swift.targets.compilerArguments is handled correctly.  It's not an easy task because we will need to activate two sourcekit-lsp servers with different SDK settings (iOS and WatchOS) at the same time.
Anyway, thank you for your great extension :)

sorry for the late reply.
I think the readme should be updated as well.

Don't worry. It was far from being time critical. 😊

As a pure iOS dev, I don't really need the per-project setting. Single global settings is good enough.

I do feel your pain. There shouldn't be that many building and absolute path details within the project configuration. Though changing this would require far more work.

The current readme is really scary for the new users (like the guy who wrote https://funnyitworkedlasttime.dev/posts/2020-01-10-swift-vscode-sourcekit-lsp/)
Besides, the original example only works for sourcekite. The sourcekit-lsp implementation doesn't read swift.targets.compilerArguments.

Well, yes. The readme has grown historically especially as the current situation with supporting different LSPs is quite complex.

At this moment, it's not possible for sourcekit-lsp to support iOS and WatchOS in the same project unless swift.targets.compilerArguments is handled correctly. It's not an easy task because we will need to activate two sourcekit-lsp servers with different SDK settings (iOS and WatchOS) at the same time.

That's right. Though I am not sure if we should even spawn multiple language servers at the same time. Either the configuration would be far too low level, or we were required to actually parse Xcode projects.

Anyway, thank you for your great extension :)

Thanks for your feedback and contributions!
A few quick questions:
What is your opinion on sourcekite versus sourcekit-lsp? Do you use both depending on the actual projects?
How do you debug your iOS/watchOS apps? Do you currently fall back to Xcode for those purposes?


Dependency on empty ivy configuration causes resolve fail

We use quite a few ivy libraries which include statements such as "sources->sources()" in their defaultconfmapping, indicating that the sources configuration should be pulled in from dependencies only if it's available. This works with standard ivy, but fails in gradle. It attempts to resolve an actual empty ("") conf from the dependencies rather than accepting it as a noop fallback. I couldn't find this in the actual ivy specs, so it may just be an undocumented "feature."
Expected Behavior
Resolving an ivy dependency with conf="sources->sources()" should not fail if the dependency does not contain a sources configuration.
Current Behavior
Performing the above causes the entire resolve to fail due to a missing "" configuration.
Context
I'm attempting to convert a project to gradle which uses a large collection of upstream dependencies affected by the described bug.
Your Environment
Gradle 3.4.1

Closing since this fluke isn't documented in ivy, isn't widespread, and we've created an internal fork to handle our strange use case.


update README for Xcode 9.

Since Xcode 9 support is actually available in another repository, let's point people to that :)

@siong1987 should not be https://github.com/XVimProject/XVim2 ?

@pawurb good catch. updated it.


y axis doesn't display properly when plotting the function y=0

Every day I have to plot the cumulative rain of a certain geographic area. Fortunately it doesn't rain every day and sometimes my application just ends up by plotting the function y=0.
The problem is that the y axis completely disappears.
Any idea how to solve this?
Thanks
Francesca

Hi @francescadamico
You can avoid this by setting the max setting for the y-axis. There are 2 possibilities: either you set it to a fixed upper bound, or you can set it to a "usual" value if the maximum value is 0.
Here is an example for the second solution: http://plnkr.co/edit/1QxTDfzC6a3Hz4hTT5nt?p=preview
Let me know if this solves your problem.
Best
Christoph

I have tried your solution in the simplest way to see if this changes something.
Now the following are my options with min and max set for the y axis but nothing changes. I still have the same issue.
$scope.options = {
axes: {
x: {key: 'tick', type: 'date'},
y: {type: 'linear'}
},
/* only one series is displayed by default. Its label will be added after the query is performed /
series: [
{y: 'value1', color: 'steelblue', thickness: '2px', striped: true, min:0, max: 1}
],
tooltip: {
mode: 'scrubber',
formatter: function (x, y, series) {
return moment(x).format('DD MMMM YYYY, h:mm a') + ' : ' + Math.round(y100)/100;
}
},
lineMode: 'linear',
tension: 0.7,
drawLegend: true,
drawDots: false//true
};

the min and max settings need to go to the axis: { y: ... } - you put them to the series ;)

Ups, I was trying to do too many things at the same time ...
Thanks a lot, it worked perfectly!
But maybe it is something that can be fixed be default

perfect :) yeah this should be fixed definitely.


Unify enums used for internal representation of quoting style

Summary
Currently we have four enums across the codebase for representing the fact that Python strings can either use single quotes (') or double quotes ("):
https://github.com/astral-sh/ruff/blob/c269c1a7063bb27e75dc81c2a598eab13aa8b754/crates/ruff_python_ast/src/str.rs#L6-L13
https://github.com/astral-sh/ruff/blob/c269c1a7063bb27e75dc81c2a598eab13aa8b754/crates/ruff_python_formatter/src/string/mod.rs#L234-L242
https://github.com/astral-sh/ruff/blob/c269c1a7063bb27e75dc81c2a598eab13aa8b754/crates/ruff_python_codegen/src/stylist.rs#L97-L103
https://github.com/astral-sh/ruff/blob/c269c1a7063bb27e75dc81c2a598eab13aa8b754/crates/ruff_python_literal/src/escape.rs#L1-L5
This means we have a lot of impl From<QuoteEnum> for IdenticalOtherQuoteEnum methods, and it's generally somewhat confusing to keep track of the subtle differences between them. This PR unfies them into one enum in ruff_python_ast::str::QuoteStyle.
Using ruff_python_ast::str::QuoteStyle in the formatter would have created an awkward name clash with ruff_python_ast::options::QuoteStyle, so I renamed ruff_python_ast::options::QuoteStyle to ruff_python_ast::options::QuotePreference. This seemed to me to more accurately reflect what that enum was for, but an alternative here would have been to rename ruff_python_ast::str::QuoteStyle. (Yet another alternative would be to always use the fully qualified name of ruff_python_ast::str:QuoteStyle in the formatter crate, but that felt too cumbersome to me.)
Test Plan
cargo test

here's a chance that I might get confused with QuoteStyle's value coming from the quote-style config option 😅. This would mean that the enum usage would be Quote::Single, Quote::Double, quote.is_single, quote.is_double, etc. What do you think? @AlexWaygood

Ahh, that makes sense... I should have checked what the config option was called, sorry!
I'll change the formatter enum's name back, and change the ruff_python_ast enum to Quote 👍


[User Story]: NEXT app needs the ability to send email to users.

[User Story]
Mail Daemon

Epic Title

User Story Title
Implement Mail Daemon

T-shirt size: S(mall), M(edium), L(arge)

User Story

Description
A daemon to send outbound emails, with relevant transports options.

Business Need (the why):
There are several scenarios where NExT is expected to send an email to users...  forgotten password, weekly snapshot, etc.

Acceptance Criteria/Tests
For this TASK, the acceptance criteria is that an email gets sent on the correct trigger -- this really isn't a story on its own.

Questions/Clarifications
This is probably just a Task under one of the various "send an email" use cases...  foundational work (shudder)

migrated to new tmobile/sawtooth-next-directory repo


Add Markdown changelog for Data Factory SDK

Any public-facing change should be documented in the changelog.
The changelog will be used to populate a Release Notes page on
azure.com (not yet completed).
Includes sections for breaking changes, feature additions and bug
fixes.

Hi @bgold09, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (brgold). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Search Box

A way to filter profiles at the overview.
Could be multiple choice, switches or similar.

What should the filters be over? Skills? Location?


Brigade Gateway with Draft issue

Validating Brigade & Draft
Versions:
Brigade 0.14.0
Draft 0.15.0
Helm 2.8.2
Followed the instruction under:
Related Projects

Draft Pack for Building Custom Gateways: Build your own gateway in 5 minutes
https://github.com/technosophos/draft-brigade

The project could be used with brig and brigade.js file ->
brig run -f brigade.js  brigade-3c505132ca6bd762bfe6732256e15684510bbcdb72333ed1cf1b88
output: successful
But when trying to use the same Brigade project with Draft, then it's failing:
$ curl -XPOST http://localhost:57399/v1/webhook/myevent/brigade-3c505132ca6bd762bfe6732256e15684510bbcdb72333ed1cf1b88
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100    21  100    21    0     0     70      0 --:--:-- --:--:-- --:--:--    70Internal Server Error
$ draft connect
Connect to chart:8080 on localhost:57399
[chart]: yarn run v1.5.1
[chart]: $ node index.js
[chart]: Running on <IP_ADDRESS>:8080
Your connection is still active.
Connect to chart:8080 on localhost:57399
[chart]: project brigade-3c505132ca6bd762bfe6732256e15684510bbcdb72333ed1cf1b88 could not be loaded
Healthz is working
curl localhost:57399/healthz  -> ok
Any ideas?

@emichaf - do you have RBAC enabled on your cluster?

Additionally, can you run a helm list for your cluster? According to the error, the custom brigade gateway cannot find the project in the same namespace as it was deployed. By default Draft is configured to deploy applications in the default namespace, so if you have brigade running in another namespace (say, brigade), then you'll need to change the namespace field in draft.toml, run draft delete and draft up again. :)
Hope this helps!

Oh, @radu-matei is right. I am missing the role definition in that chart. Fixing.

All components are in the "default" namespace:

brigade-server (started with rbac.enabled=true)
brigade-project (started with rbac.enabled=true)
mygateway (started with rbac.enabled=true)

Still same fault when trying
curl -XPOST http://localhost:55622/v1/webhook/myevent/brigade-ae8de7d2d6004665f92fa350d2cd6f260a6a326f580df3c9502574
Connect to chart:8080 on localhost:55622
[chart]: yarn run v1.5.1
[chart]: $ node index.js
[chart]: Running on <IP_ADDRESS>:8080
[chart]: project brigade-ae8de7d2d6004665f92fa350d2cd6f260a6a326f580df3c9502574
could not be loaded
Trying to run via brig -f, and it's working for the project:
brig run -f brigade.js brigade-ae8de7d2d6004665f92fa350d2cd6f260a6a326f580df3c9502574
ok

Okay, I know this is a bit tedious, but it will help us figure out where the problem lies.
While your mygateway is running, can you:
kubectl exec -it YOUR_GATEWAY_POD_NAME /bin/sh
Then install kubectl in the pod:
$ curl -LO https://storage.googleapis.com/kubernetes-release/release/$(curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt)/bin/darwin/amd64/kubectl
$ chmod +x ./kubectl

Then check to see if you can see the project secrets:
$ ./kubectl get secret -l component=project

If you. see the projects there, we can definitely rule out RBAC and we are down to two things:

Did the gateway auth correctly?
Is there a bug in the project-fetching code?

If the kubectl command doesn't show the project, then the most likely culprit is RBAC.

found some errors in the draft-brigade proj, se PR:
https://github.com/technosophos/draft-brigade/pull/1


CX Reflected_XSS_All_Clients @ website/comments/preview_comment.php [master]

Reflected_XSS_All_Clients issue exists @ website/comments/preview_comment.php in branch master
The application's http_redirect embeds untrusted data in the generated output with header, at line 5 of website\include\functions.php. This untrusted data is embedded straight into the output without proper sanitization or encoding, enabling an attacker to inject malicious code into the output.
The attacker would be able to alter the returned web page by simply providing modified data in the user input _POST, which is read by the <?php method at line 1 of website\comments\preview_comment.php. This input then flows through the code straight to the output web page, without sanitization.
This can enable a Reflected Cross-Site Scripting (XSS) attack.
Severity: High
CWE:79
Vulnerability details and guidance
Checkmarx
Lines: 44

Code (Line #44):
      http_redirect(".." . Pictures::$VIEW_PIC_URL . "?picid=" . $_POST['picid']);

Issue still exists.

Issue still exists.


fix(checkbox, input, radio, slide-toggle): implement setDisabledState from ControlValueAccessor

Implements the setDisabledState method from the ControlValueAccessor interface in all of the input-related components, in order to support disabling via reactive forms. Note that the select component is missing the implementation, however there's a pending PR for it already (#1667).
Fixes #1171.

Could you add a unit test for each component that sets disabled on the component and then asserts that the form model is correctly marked as disabled?

Added unit tests @jelbourn.

LGTM

@crisbeto can you rebase this PR?

Rebased @jelbourn. For some reason the entire slide-toggle.spec got marked as a conflict.

@crisbeto Yeah that's because we changed the Line endings on the whole file.

so how it fix? npm from master? thanks.


[Bug] excluse or remove the mysql:mysql-connector-java:jar:8.0.21 package

What happened
The parent issue is: https://github.com/apache/incubator-inlong/issues/4276

What you expected to happen
compile
How to reproduce
compile
Environment
No response
InLong version
master
InLong Component
InLong Sort
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I am willing to submit a PR.

@GanfengTan, thanks!


Followers sometimes pick up gear they don't equip

I've seen someone pick up studded leather armor while wearing studded leather armor; when I manually swapped it out for them, they didn't acquire the other one, must have been a difference we measure that PerformReequip doesn't? Once we can reproduce it, it can be addressed.

Still not sure exactly why this happens, but with 83c0eaedd606ef9a9140bbe748ae487f2793d493 they at least shouldn't fill their inventory with things they won't wear.


Fix getwuid for source container

Description
Fix getpwuid for source image
All Submissions:

[x] Have you ensured there aren't other open Pull Requests for the same update or change?
[ ] Have you created a new branch for your changes and PR from that branch and not from your master branch?

New Image Submissions:

[ ] Have you added your image to the Github workflows?
[ ] Have you updated the README list to contain your new image?

how you ever tryed if it worked as in my testing it does not? also the default uid for the container user is 988

I am closing this as this will cause some issues as the UID en GID will not always be 994. For example, if the container is running in docker, it will be 988. It is been worked on, on the wings side. you can always make your own docker image and use a wrapper to fake this as example here: https://github.com/parkervcp/yolks/blob/dbe89a4771cba2ed313fbe58cdda6dd00e2724bb/games/dayz/Dockerfile#L35

The ID used changes per host as well. We assign the id based on the sustem users already in place. It's part of the adduser command that sets the uid


feat!: esbuild as a peer dependency

This allows users to upgrade esbuild independently of esbuild-loader.
This is great because esbuild has a new release every other day.
Unfortunately, this is going to be a breaking change.
Hope to queue this with some other BCs.
Closes #25

This makes a lot of sense to me, would love to get this in so esbuild can be upgraded independently!

Oh, I think this commit is actually on master already and this PR can be closed. Would be dandy to get a release with it out!

Still holding this because I want to batch it with another breaking change, and also waiting for esbuild v1 because I imagine there's going to be a breaking change there.
Do you want me to do a dependency upgrade release instead?
Alternatively, if you're using yarn, you can use the resolutions feature.

Oh sorry I was looking at the wrong branch on the root page 😅
No rush, I am just using resolutions :)

Adding a note that this might not be a good idea to release till esbuild is v1 (stable) because they can still technically do breaking changes (they did a breaking change on the Node API recently).


https://facebook.github.io/react-native/releases/0.23/docs/tutorial.html

Yes

Have you read the Contributing Guidelines?
Yes
OSX

react-native -v: react-native-cli: 2.0.1
react-native: 0.47.2
node -v: v7.10.1
npm -v:4.2.0

Target Platform:

Development Operating System:

Build tools:

Steps to Reproduce

Go to https://facebook.github.io/react-native/releases/0.23/docs/tutorial.html
npm install -g react-native-cli
react-native init AwesomeProject

Expected Behavior
Not sure. It's a tutorial. It should do SOMETHING.
Actual Behavior
Installs a megaton of... stuff and then throws an error.

Reproducible Demo

https://facebook.github.io/react-native/releases/0.23/docs/tutorial.html
Follow the first two instructions to the letter.
After installing megatons of... stuff, you get this
** BUILD FAILED **

The following commands produced analyzer issues:

	Analyze /Users/nige/react-native/AwesomeProject/node_modules/react-native/ReactCommon/yoga/yoga/Yoga.c
	Analyze /Users/nige/react-native/AwesomeProject/node_modules/react-native/ReactCommon/yoga/yoga/YGNodeList.c
(2 commands with analyzer issues)

The following build commands failed:
	CompileC /Users/nige/react-native/AwesomeProject/ios/build/Build/Intermediates/React.build/Debug-iphonesimulator/React.build/Objects-normal/x86_64/RCTTabBarItem.o Views/RCTTabBarItem.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)

Installing build/Build/Products/Debug-iphonesimulator/AwesomeProject.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=2):
Failed to install the requested application
An application bundle was not found at the provided path.
Provide a valid path to the desired application bundle.
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/AwesomeProject.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

It's not good. The first tutorial page, from the framework's authors. Overcomplicated. And throws an error. Why would anybody bother?

Current react native version is 0.47 the tutorial is from version 0.23.

@facebook-github-bot answered


Multi language support

https://github.com/Anthony-Gaudino/jekyll-multiple-languages-plugin

Fixed


not an editor command CmpTabnineHub

Looks like this does not work. (not an editor command). Let user configure Tabnine afterwards.

oh. Github ui is terrible, i will create roper issue.


Does or will nnabla support dynamically loading custom function?

Hey, I am interested in this NN tool. And I wonder if it will support custom function? Besides, I wonder is there any difference between nnabla with other NN tool, such as mxnet or tensorflow?
Much thanks.

Thanks for your question.
As a short answer, dynamic loading of a custom function definition is not currently supported. I'll describe it more details below.
There may be two different definitions of custom functions.

A new function definition (e.g adding SpatialPyramidPooling)
A new implementation of an existing function (e.g. adding a CUDA implementation of SpatialPyramidPooling)

For 1, NNabla only provides a way to add a new function definition statically. We're still investigating how we achieve a dynamic function definition.
For 2. a special implementation of a base function (e.g. Convolution --> ConvolutionCuda) is dynamically registered as we do in a CUDA extension of NNabla, and a user can call it by setting an appropriate context descriptor.
There is actually an alternative way to define a custom function (1). We provide a way to write a custom function in Python side, which is not documented so far (i.e. not officially supported so far). You can find it in a unit test.  The test demonstrates how we implement an operator which adds two tensors using Numpy array.
Although it is off-topic on this thread, I will give you a bit of answer on differences between NNabla and others (Please post a question like this to a user group hosted in Google Groups).
I believe the dynamic loading of a special implementation is one of the differences from some of the other libraries. Another difference is that NNabla supports both paradigms of static and dynamic neural network programming, which enables the users to choose the performance efficiency of the static one or the flexibility of dynamic NN, depending on their purposes. Also, NNabla's core is implemented in C++11, including a computation graph and its construction. We'll soon make it available the C++ API, which can be used in training and inference of NN even in a system which prohibits running interpreter languages like Python.

Much thanks for your answer!


there is a bug with Chinese character！！！

how to fix it？

you can update tomcat server.xml

no, i had already configured


Sugestão de melhoramentos na formatação do texto

Sugestão de melhoramentos na formatação do texto.

[x] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks for the contribution! Congratulations on your first PR for this repo and happy coding.


Ingestion bug when using partition key and unsorted data

Describe the bug
Cant ingest data while using a partition key if number of partition key is high and ndjson data is not sorted per partition key.
Steps to reproduce (if applicable)
Ingest 100M more docs with partition key and unsorted data with default configuration
Expected behavior
Data to be ingested in the index without any issue.
Configuration:
Default configuration, version 0.8.1, please check this thread (https://discord.com/channels/908281611840282624/1233364603480576092) where we discuss the issue and hopefully find a solution for further details

The number of published document is probably a limitation of the UI. I think we download a partial list of splits metadata and do the computation in javascript.
@piereissyet Concretely what is the other issue you observe?
Ingest blocks at one point for a few minutes and then resumes or soemthing like that?

My hunch is that heaving partitioning makes the load on merge heavier, especially if the merge policy is inappropriate and the number of partiiton is high.
In that case the default setting in merge_concurrency is not sufficient, and you need to increase it.
merge_concurrency is an undocumented property in the indexer config. (in the quickwit.yaml)
You can confirm this hypothesis by looking at the pending merge / ongoing merge curves.
At each commit the pending merge should increase strongly and then decrease as merge get done.
After 10 * commit_timeout large merges will occur and the peak will get even higher.
After 100 * commit_timeout same thing.
The graph should show the number of pending merge to eventually come back to 0 or close to 0 and not diverge.

Ingestion gets blocked, numbers go crazy on the UI then eventually the insertion keep going after a while. We had use cases where it was stuck for many hours and others where it eventually went back depending on the numbers.
We did not dig the issue more since we managed to get rid of the issue by just sorting our data per partition key.
I will definitely try the merge_concurrency thing if this happen again. Any idea what value should I use ?

Ingestion gets blocked, numbers go crazy on the UI then eventually the insertion keep going after a while.

yes that is in line with what I thought.
If you have prometheus, looking at the graph of pending/ongoing merging should be telling.

We did not dig the issue more since we managed to get rid of the issue by just sorting our data per partition key.
If this is something you can do, it is actually a nice hack to make the work of quickwit much lighter.

merge_concurrency value:

Actually scratch that, I am wrong. merge_concurrency would only help if you had a single index. We do not allow parallel merges within a single pipeline at the moment. Right now the only thing you can do is wait for merges to progress.
What is the volume here?

We had issues with ~130M docs / ~45go of data uncompressed


Comparison to github.com/alexedwards/scs

A quick look at both libraries seems to show an important difference: github.com/alexedwards/scs buffers all response bytes. A bufferedResponseWriter{} buffering all output to catch session changes before headers are sent (

https://github.com/alexedwards/scs/blob/master/session.go#L118
Which at one point supported websockets by implementing hijack?.

Jeff seems to handle this better (but not easier) by making you call Set manually when headers should be set without wrapping your http.Handler.
Don't use alexedwards/scs if you send large payloads to clients, stream response bodies, use websockets(?), etc.

Yeah, that's weird.  I've got no idea why he's using a buffered response writer.  I guess it's for transparently handling setting the cookie?  This looks pretty rough too:
https://github.com/alexedwards/scs/blob/efc4b6b318277be948e5fc6eb5c218a1f160f090/session.go#L137
Maybe I can write a bit more of an in-depth comparison of the two.  His was definitely the strongest contender when I was looking around for a library.

I've got no idea why he's using a buffered response writer. I guess it's for transparently handling setting the cookie?

Yep, that's right. The LoadAndSave() middleware in SCS is basically a helper for the common scenario where you want to communicate the session token (or session id) to the client in a cookie. One of the things it does is add the cookie to the Set-Cookie response header, if necessary, as the middleware chain is unwound.
If the ResponseWriter write wasn't buffered by this middleware, when a handler calls w.WriteHeader() --- either explicitly, or implicitly via w.Write() --- the response headers would be written before the middleware was able to add the Set-Cookie header. And the session token would never make it back to the client.
One of the big design decisions I made with v2 of SCS was to make the saving of session data and communication of the session token to the client independent processes. It makes the package a lot more flexible, as people can write their own middleware to communicate the session token in other ways if they want to (like in a JSON body, or in an X-Session header).
Well done on the jeff package, it's good to see some more alternatives to gorilla sessions :+1:

Thanks for the discussion here guys!  Closing this out for now. :-)


Extract series labels from metric regex

To provide more sophisticated series names, it would be very helpful, if series names could be extracted from the metric regex using capturing groups.

This feature will come with the next release. I hope to be done with this by the end of next week.

Implemented with<PHONE_NUMBER>abd514985572d172815fbf9bb707b4 / release v0.4.1


Docs Enchancement

Summary
Better readability and less confusion
Problem
Added a warning box that warns about some of the less known slash commands mechanics
that are undocumented here

i suck at using github

#94 is this how u do it?

#94
^ i have no idea how to link a PR so here it is


Add A[i=>2,j=>3] syntax for getting and setting values

Also fills in a few more broadcasting features, like A .= c for fill!(A,c) and A .= B for copyto!(A,B).

Codecov Report

Merging #227 into master will increase coverage by 0.29%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
+ Coverage   81.12%   81.41%   +0.29%
==========================================
  Files          46       46
  Lines        4790     4752      -38
==========================================
- Hits         3886     3869      -17
+ Misses        904      883      -21

Impacted Files
Coverage Δ

src/itensor.jl
64.60% <33.33%> (+2.92%)
:arrow_up:

src/index.jl
75.00% <50.00%> (-3.49%)
:arrow_down:

src/mps/mpo.jl
99.38% <0.00%> (ø)

src/iterativesolvers.jl
90.27% <0.00%> (ø)

Continue to review full report at Codecov.

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


About the UCF-HMDB_full Dataset

Where can I download the UCF-HMDB_full dataset used in the paper?

sorry for the late response.
I am busy with a paper deadline.
I will release the dataset once I finish the paper submission.

Released.


add contrubution and chat badges

Fixes #
Proposed Changes

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


docs: add required token field when calling external workflow

Since the token field is required when calling an external workflow, I think it should be added to the example as well. Would have saved me an hour or so 😄

Fixed in latest


Fix bug with middleware mounting

Variable name in bootstrap.use method was incorrect. Should have been userMiddleware and not router. This caused an error to be thrown when mounting middleware.
Fix and add a test to prevent regression.


Plan a round trip

Would it be possible to plan a roundtrip in one step?
My current way to do it is to plan a route from > to and to > from, and choose from the original and alternative the routes with biggest difference from each others. This should be possible to combine into one step, and offer to the user several possible alternatives.
For an easier method, simply take the from > to route (original + alternatives) and get the roundtrip from this one, again choose the combination with biggest difference. There is the half manual way to do it as well - simply let the user to choose the combination of the routes, just show them which routes are suggested for the route to the selected place and back again.

Hello @majkaz, indeed it could be a nice feature - not sure how we'd do that, I think this is (technically) related to #68 since we would need to ask brouter multiple routes, but not sure how to tell it to avoid going back on the same route?

@bagage Maybe we could use the nogo polyline feature, with a buffer on the second (way back) route?

We currently can only assign route options (profile and alternatives) to the whole route and not individual segments, both in the client and the server (Export requests the whole route). Maintaining, passing and processing options per segment would be useful for several features.

not sure how to tell it to avoid going back on the same route

I think that is how the alternatives work: the engine always starts with the best route (original) and then calculates again, avoiding ways taken in the previous ones, repeating n times for the n-th alternative.
So as majkaz suggests, combining e.g. the original "alternative" with some other n-th alternative on the way back would do the trick to get some sort of round trip when setting two points.
A similar feature that I would like to have is suggested round trips by specifying a start point and a distance. But that would require some special route optimization algorithm in the engine.
As an example, cycle.travel has both ("Round-trip" and "Suggest a ride").


Parse and render expert commentary identifiers

Implemented only partially, waiting for CEN/EN identifiers implementation to finish.
closes #11

@ronaldtse ready, now full implementation


OpenAI error: Premature close (ERR_STREAM_PREMATURE_CLOSE)

Sometimes when running OpenAI streaming I get the following error:
Error: Premature close
    at IncomingMessage.<anonymous> (/app/node_modules/node-fetch/lib/index.js:1748:18)
    at Object.onceWrapper (node:events:631:28)
    at IncomingMessage.emit (node:events:517:28)
    at emitCloseNT (node:internal/streams/destroy:132:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:81:21) {
 code: 'ERR_STREAM_PREMATURE_CLOSE'
}

I don't know how to repro, but it happens consistently when I run big batches of jobs with many prompts. Sometimes it happens just when starting the request, sometimes it happens a while in (e.g. 10+ seconds into the generation).
ChatOpenAI.onFailedAttempt never gets called, so somehow p-retry inside ChatOpenAI isn't retrying requests when this happens. Instead, the agent runnable that invokes this.model catches the error.
Any idea how to make this error caught inside ChatOpenAI so that the request can be retried?
This is how I create my Langchain ChatOpenAI runnable:
import winston from "winston";

const STATUS_NO_RETRY = [
  400, // Bad Request
  401, // Unauthorized
  402, // Payment Required
  403, // Forbidden
  404, // Not Found
  405, // Method Not Allowed
  406, // Not Acceptable
  407, // Proxy Authentication Required
  408, // Request Timeout
  409, // Conflict
];

function createFailedAttemptHandler(logger?: winston.Logger) {
  return (error: any) => {
    logger?.warn('OpenAI failed attempt', { error, stack: (error as Error).stack });

    if (
      error.message.startsWith('Cancel') ||
      error.message.startsWith('AbortError') ||
      error.name === 'AbortError'
    ) {
      logger?.warn('OpenAI failed because of cancel/abort', { error });
      throw error;
    }

    const status = (error as any)?.response?.status ?? (error as any)?.status;
    if (status && STATUS_NO_RETRY.includes(+status)) {
      logger?.error('OpenAI failed because of status', { status });
      throw error;
    }

    if ((error as any)?.error?.code === 'insufficient_quota') {
      logger?.warn('OpenAI failed because of insufficient quota', { error });
      const err = new Error(error?.message);
      err.name = 'InsufficientQuotaError';
      throw err;
    }
  };
}

this.model = new ChatOpenAI({
      temperature: 0,
      modelName: 'gpt-4-1106-preview',
      maxTokens: 1000,
      streaming: true,
      cache: fields.cache,
      configuration: fields.config,
      maxRetries: 10,
      onFailedAttempt: createFailedAttemptHandler(fields.logger),
    })

Do you need to have streaming enabled? Catching streaming errors is much more nuanced (e.g. what to do with partial responses).

Do you need to use .stream() for this or could use .batch() instead? Catching streaming errors is much more nuanced (e.g. what to do with partial responses).

I need to use stream (because this is streamed to the user on the frontend)! I'd be fine with just rerunning the request if I get a partial response. Do you know how I'd catch the streaming errors in the first place? Right now they seem to be falling through the retry logic

@jacoblee93 I think I see what's going on: when using streaming, completionWithRetry is used in ChatOpenAI._streamResponseChunks, which awaits (and should retry) the creation of the stream iterable. However, when later iterating over the stream, any errors thrown there won't be retried. Might that be right?
https://github.com/langchain-ai/langchainjs/blob/6e5d02188bd29c7fe51ff1440a097150f88c547d/libs/langchain-openai/src/chat_models.ts#L463
A somewhat related issue: I'm also getting streams that sometimes end unexpectedly while generating JSON. This usually happens when OpenAI API reqs take an unreasonable amount of time (e.g. 4 min vs the regular 20s), so something is prob wrong on OpenAI's side, but Langchain doesn't allow me to catch these problems anyway. That throws errors in getEstimatedTokenCountFromPrompt because JSON parsing fails:
https://github.com/langchain-ai/langchainjs/blob/6e5d02188bd29c7fe51ff1440a097150f88c547d/libs/langchain-openai/src/chat_models.ts#L711-L715
Those errors are also just thrown and not retried because it happens outside of the retry logic.
One potential solution:

Wrap the full streaming logic in the retry logic
In the defaultFailedAttemptHandler, rethrow the errors that are streaming related: https://github.com/langchain-ai/langchainjs/blob/6e5d02188bd29c7fe51ff1440a097150f88c547d/langchain-core/src/utils/async_caller.ts#L18
In this way, ppl still get the same behavior as now, but for use cases like mine that want more control over error handling, it's possible to provide a custom onFailedAttempt handler that e.g. decide to retry these errors.

Any reason you wouldn't want to wrap the whole streaming logic in the retry logic?
https://github.com/langchain-ai/langchainjs/blob/6e5d02188bd29c7fe51ff1440a097150f88c547d/libs/langchain-openai/src/chat_models.ts#L526-L559

@jacoblee93 I think I see what's going on: when using streaming, completionWithRetry is used in ChatOpenAI._streamResponseChunks, which awaits (and should retry) the creation of the stream iterable. However, when later iterating over the stream, any errors thrown there won't be retried. Might that be right?

Yes, this is the intended behavior - we basically catch immediately thrown errors (missing auth tokens) for convenience. When we were implementing the original version of runnables, it wasn't 100% clear what default retry behavior should be mid-stream (abstractly, you could stream from some stateful API with consequences for just re-requesting like that), so we don't do anything by default and just throw it.
I think some way to hook a handler in would be reasonable though - for now, would it unblock you to implement it in your own application logic?

I think for token counting logic we should absolutely not error the entire call on parsing failures, so yes that's definitely a bug.

Sure, I'll implement the retry logic myself

@davidfant curious if you ended up finding a solution to this you could share?


Optimize README images

Idea Description
Resize and use some tool to optimize it 🙏
Reference images and links

How about this imgbot? 🤔

I didn't know that 👀 Let me think it over.

Thanks. I introduced it for trying in this repository.
But as far as this issue is concerned, the images are in https://user-images.githubusercontent.com/ . So Probably we can't use them for it.

@tatsuyafujisaki  Yes you can, but @takahirom seems to have used imgbot before, 🤔, BTW, I recommend using this to optimize images

@Nthily Thank you. I will use Squoosh.
@takahirom Yes, please.:pray:


Widget Transparency broken

It seems this widget's transparency wont change no matter what its set to.

The browser is not using the version of the Game Bar SDK that provides the transparency feature.
It could be updated but I don't think it's worth the time until Game Bar supports WinUI 3.


Fix module doc example

The :interval option is passed via the :reporter_options keywords list.
Also used the opportunity to apply mix.format to all files, making contributions easier for people that have it configured in their editor.

thank you for this!

I just pushed v0.1.2 with this fix, thanks again!


feat (wip): add playground settings for API playground

Adds ability to add playground settings in docs.yaml:
playground:
  allowedEnvironments:
    - Prod
    - Dev

These can be nested under API reference, packages, and endpoints.

Hey @RohinBhargava --  what if an environment id is used that doesn't exist?

In the FE, the fallback is to use the first environment in the list of environments by endpoint. In Candid's case, this would be the production URL, so the right fix might be to block CI if environmentid is not specified in api.yml?


Fixes SteamVR compatibility with Unity 2017.1

Fixes issues with VRTK in Unity 2017.1 where SteamVR camera remains black.
Tested in latest beta 9

There were the following issues with your Pull Request

Commit: d31502fc905d5ab5159f7506e78ed5cfdc753009
Commits must be in the following format: %{type}(%{scope}): %{description}
Your subject line is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

How I crave for the day they finally merge https://github.com/ValveSoftware/steamvr_unity_plugin/pull/14 😩

Are we better off just doing Unity_5_6 and not doing a negation of Unity_2017?

I agree with @thestonefox.

There were the following issues with your Pull Request

Commit: d31502fc905d5ab5159f7506e78ed5cfdc753009
Commits must be in the following format: %{type}(%{scope}): %{description}
Your subject line is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

agree! changed it

@AndreGroeschel You'll have to squash your two commits, so only the newest one remains. Make sure the commit message stays good. Also: A small description in form of a commit message body will help to understand the commit in case we later look at it.
May I suggest this?
fix(SteamVR): only add SteamVR_UpdatePoses for Unity 5.6

The `SteamVR_UpdatePoses` script was added on Unity 5.6 and newer, but
it's no longer needed to do that in Unity 2017.1.


watchObject folder verification doesn't handle globs and causes startup to fail

Describe the bug
I upgraded to the latest version and found that the docker container fails to startup. It fails with the following message in the log. Not sure why since I've changed nothing else in any of the config files.
Note that the watch pattern in the trigger.json file is "watchPattern": "/aiinput/*/*/*/Camera1*.jpg"
I reverted back to v5.0.0 and everything works as before.
Log entries
2020-07-12T00:01:24+01:00 [Trigger manager] Unable to read contents of watch folder /aiinput/*/*/* for trigger Person detector. Check and make sure the image folder is mounted properly. Error: ENOENT: no such file or directory, scandir '/aiinput/*/*/*'

Installation details

QNAP TS-451+ NAS server running Container Station

All the changes in 5.1.0 were logging/startup retry related. There weren't any changes to the trigger code at all.
Are you sure you're using the same settings.json, docker-compose.yaml, and triggers.json file for 5.1.0 as you are for 5.1? Can you post your complete docker-compose.yaml and triggers.json?

Also post the complete logs from startup. There's a new log message in 5.1.0 that specifically reports whether the image folder is mounted correctly.

I did not make changes to any of the config files. Literally just did removed the old container, pulled the latest docker image, and started the container. Here are the settings.json, triggers.json and docker-compose.yaml files.
settings.json
{
  "$schema": "https://raw.githubusercontent.com/danecreekphotography/node-deepstackai-trigger/main/src/schemas/settings.schema.json",

  "deepstackUri": "http://192.168.xxx.xxx:5000/",

  "verbose": true,

  "awaitWriteFinish": true,

  "mqtt": {
    "uri": "mqtt://192.168.xxx.xxx:1883",
    "username": "<username>",
    "password": "<password>",
    "retain": true
  },

  "telegram": {
    "botToken": "insert bot token here",
    "enabled": false
  },

  "pushover": {
    "apiKey": "api key here",
    "userKey": "user key here",
    "enabled": false
  }
}

triggers.json
{
  "$schema": "https://raw.githubusercontent.com/danecreekphotography/node-deepstackai-trigger/master/src/schemas/triggerConfiguration.schema.json",
  "triggers": [
    {
      "name": "Person detector",
      "watchPattern": "/aiinput/*/*/*/Camera1*.jpg",
      "enabled": true,
      "threshold": {
        "minimum": 0,
        "maximum": 100
      },
      "handlers": {
        "webRequest": {
          "triggerUris": ["http://192.168.xxx.xxx:8080/xxx/motion/xxx/Camera1?data={'plug':'Camera1'%2C'name':'Camera1'%2C'reason':'motion'%2C'confidence':100}"]
        },
        "mqtt": {
          "topic": "aimotion/triggers/person"
        }
      },
      "watchObjects": ["person"]
    }
  ]
}

docker-compose.yaml
version: "3"

networks:
  qnet-dhcp-eth0-1c1ca1:
    external: true

services:
  deepstackai-trigger:
    container_name: deepstackai-trigger
    image: danecreekphotography/node-deepstackai-trigger:latest
    networks:
      - qnet-dhcp-eth0-1c1ca1
    ports:
      - 4242:4242
    environment:
      - TZ=Europe/London
    volumes:
      - /share/CCTV/AI_Input:/aiinput
      - /share/Containers/Volumes/DeepStackAI_Trigger/config:/config
      - /share/Containers/Volumes/DeepStackAI_Trigger/node-deepstackai-trigger:/node-deepstackai-trigger
    restart: always

Log Entries
2020-07-12T01:13:04+01:00 [Main] ****************************************
2020-07-12T01:13:05+01:00 [Main] Starting up version 5.1.0
2020-07-12T01:13:05+01:00 [Main] Timezone offset is -60
2020-07-12T01:13:05+01:00 [Main] Current time is Sun Jul 12 2020 01:13:05 GMT+0100 (British Summer Time)
2020-07-12T01:13:05+01:00 [Settings] Unable to read the configuration file: ENOENT: no such file or directory, open '/run/secrets/settings'.
2020-07-12T01:13:05+01:00 [Settings] Loaded settings from /config/settings.json
2020-07-12T01:13:05+01:00 [MQTT] Retain flag set in configuration. All messages will be published with retain turned on.
2020-07-12T01:13:34+01:00 [MQTT] Connected to MQTT server mqtt://192.168.xxx.xxx:1883
2020-07-12T01:13:34+01:00 [Local storage] Creating local storage folders in /node-deepstackai-trigger.
2020-07-12T01:13:34+01:00 [Local storage] Enabling background purge every 60 minutes for files older than 30 minutes.
2020-07-12T01:13:34+01:00 [Local storage] Running purge
2020-07-12T01:13:34+01:00 [Triggers] Unable to read the configuration file: ENOENT: no such file or directory, open '/run/secrets/triggers'.
2020-07-12T01:13:34+01:00 [Triggers] Loaded configuration from /config/triggers.json
2020-07-12T01:13:34+01:00 [Triggers] Loaded configuration for Person detector
2020-07-12T01:13:34+01:00 [Trigger manager] Unable to read contents of watch folder /aiinput/*/*/* for trigger Person detector. Check and make sure the image folder is mounted properly. Error: ENOENT: no such file or directory, scandir '/aiinput/*/*/*'
2020-07-12T01:13:34+01:00 [Main] Unable to access one or more watch locations.
2020-07-12T01:13:34+01:00 [Main] ****************************************
2020-07-12T01:13:34+01:00 [Main] Startup failed due to errors. For troubleshooting assistance see https://github.com/danecreekphotography/node-deepstackai-trigger/wiki/Troubleshooting.
2020-07-12T01:13:34+01:00 [Main] Startup reattempt 1 of 5 in 30 seconds.
2020-07-12T01:13:34+01:00 [Local storage] Purge complete

Yup, you found a bug! The new logic that tests whether the folder exists at startup (since so many people have folder mapping issues) doesn't handle globs in the path names. Instead of tossing an error and halting I should just warn. Hang tight, I'll have a fix shortly.

Ok the fix is ready for you to try out. Change your docker-compose.yaml file to point to danecreekphotography/node-deepstackai-trigger:dev for the image and it should start up now. There will still be a warning about how it can't verify the directory location, but startup won't fail anymore.
Please let me know if this works. If it does I'll push it out as an official release.

Just tried the :dev image and it starts up without failing. Thanks!

Great! This is now released as version 5.1.1. You can switch back to the latest Docker image.

Just switched back to :latest and it is working as expected. Thanks!
BTW, I still get the following log message -

Unable to read contents of watch folder /aiinput/*/*/* for trigger Person detector. Check and make sure the image folder is mounted properly. Error: ENOENT: no such file or directory, scandir '/aiinput/*/*/*'

but then proceeds to startup and watch for images correctly; why am I getting this message?

The check is still there and it can't confirm the folder exists because you're using wildcards in the folder path. It's safe to ignore.

Ah! Got it. Thanks again.
Just in case it helps others, I have to use wildcard in the watch folder path because Reolink cameras FTP images using Year/month/day folder structure.


[backport] Move the NVTX module to cupy_backends.cuda.libs

Backport of #4930

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 36adc8a0e41d0d97032013dc514839a430ed490b, target branch v9) succeeded!


Items on list not sorting when clone option is true

Hello,
This is an excellent plugin and is helping me a lot. I need the clone option, but when it's set to true, the item inside the list doesn't change order each other anymore.

@tbthiago shouldn't the clone option expected to give you the copies? If it is sorting by itself the how do u perform clone?

@a5hik Sorry for not explain correctly. When clone option is set to true, I clone it in other list, but this item doesn't change the order inside the list when you drag and drop it in other position. The item remains in same position. Is that clear?
Other problem is when I use filter, the filtered item after drag and drop in other list is not correct. I changed the code passing the element as parameter instead of index. It's better, but didn't solve the problem if I do it few times.

the two are not supported by this plugin.


Kendall's Tau metric, based loosely on scipy.

Description
Adds Kendall's Tau, a measure of ordinal concordance as a Keras compatible metric,
Type of change

[ ] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ x ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[ ] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
Multiple tests have been included, and because this metric exists in non-tf form in scipy it's easy
to verify correctness.
If you're adding a bugfix or new feature please describe the tests that you ran to verify your changes:
*

@marload
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

Thanks for the contribution. I am not very familiar with scipy implementation. @WindQAQ @bhack are you familiar with this one?

LGTM! Thanks again. Some test are failing but they are on our end and not yours. Once we fix them, I think we can merge this.

/cc @brianwa84 @jvdillon Are you interested in this for the Tensorflow Probability repo? It could fit in the stats sub package: https://www.tensorflow.org/probability/api_docs/python/tfp/stats

Hello! I'd love to see this in TFP!
Id also love to see the top-k loss, as described here: https://arxiv.org/pdf/1206.5280.pdf

@jvdillon Thanks, I have question It seems that the TFP repository currently doesn't hosts metrics right? We could have case like this or our Matthews correlation impl where the user could need to consume it as inherited from tf.keras.metrics.Metric object.
So if the ops is in TFP but the metric is here we need to depend on TFP. Do you plan to support metrics directly?

We dont have keras metrics but we do have standalone functions which might
be regarded as metrics. We strive to be as library agnostic as possible. If
possible we prefer simple functions and have a strict policy that all
functions must support batch input.  Take a look at other tfp.stats for
details.
On Mon, Sep 21, 2020 at 4:35 PM bhack<EMAIL_ADDRESS>wrote:

@jvdillon https://github.com/jvdillon Thanks, I have question It seems
that the TFP repository currently doesn't hosts metrics right? We could
have case like this or our Matthews correlation
https://github.com/tensorflow/addons/blob/master/tensorflow_addons/metrics/matthews_correlation_coefficient.py
impl where the user could need to consume it as inherited from
tf.keras.metrics.Metric object.
So if the ops is in TFP but the metric is here we need to depend on TFP.
Do you plan to support metrics directly?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/2169#issuecomment-696435439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIVTNWWRTQXH4R63APFN33SG7PLPANCNFSM4RQOFWYA
.

@jvdillon Ok but I see that you have Keras layers and optimizers in TFP.
I think that it is ok to have an op like this in TFP but then why not to have also metric that wraps the operator (when it is useful to be consumed as metrics)?
I think that It will be hard for TF Addons to depend on TFP currently just to implement metrics.

On Tue, Sep 22, 2020 at 10:53 AM bhack<EMAIL_ADDRESS>wrote:

@jvdillon https://github.com/jvdillon Ok but I see that you have Keras
layers and optimizers in TFP.
I think that it is ok to have an op like this in TFP but then why not to
have also metric that wraps the operator (when it is useful to be consumed
as metrics)?
I think that It will be hard for TF Addons to depend on TFP currently just
to implement metrics.

I will bring this question to the team. In the meantime, maybe proceed with
adding the core function?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/2169#issuecomment-696880578,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIVTNW3IOKABCQ4ICYTIWDSHDP7ZANCNFSM4RQOFWYA
.

@jvdillon It is ok for me.
@AakashKumarNain @sorensenjs what do you think?
P.s. Are you interested also in https://github.com/tensorflow/addons/blob/master/tensorflow_addons/metrics/matthews_correlation_coefficient.py?

I guess keras metrics have some streaming functionality? At least the old tf.metrics did.
Note that we have a similar code for streaming reduction baking under tfp.experimental.mcmc (covariance, moments, etc). This uses tensors instead of Variables, though.

I guess keras metrics have some streaming functionality? At least the old tf.metrics did.
Note that we have a similar code for streaming reduction baking under tfp.experimental.mcmc (covariance, moments, etc). This uses tensors instead of Variables, though.

@brianwa84 Yes, the Metric api has streaming functionality.
PS: I am okay with adding the core functionality for the time being.
cc: @seanpmorgan

@seanpmorgan Before we lost this PR I am still oriented to include this TFP.

@jvdillon @brianwa84  Do you have permission to migrate this in TFP?

I don't know what you mean by "migrate". I think you can just open a new PR against TFP.
For now, we could perhaps put something like this under tfp.experimental.stats, where we have other streaming metrics like covariance etc. Or you could modify to drop the streaming component and simply compute on a single tensor of observations, which would be more consistent w/ e.g. tfp.stats.covariance. We would likely want to eliminate the for loops for TFP. If you really need a loop, we usually use tf.while_loop or tf.scan to avoid bloating the tf.function graph.

I don't know what you mean by "migrate".

Yes I supposed that we had a related issue about this features as it was mandatory in our contribution policy. But in this case we have a PR directly.

For now, we could perhaps put something like this under tfp.experimental.stats, where we have other streaming metrics like covariance etc. Or you could modify to drop the streaming component and simply compute on a single tensor of observations, which would be more consistent w/ e.g. tfp.stats.covariance. We would likely want to eliminate the for loops for TFP. If you really need a loop, we usually use tf.while_loop or tf.scan to avoid bloating the tf.function graph.

The plan is ok for me
@sorensenjs
Can you open the PR at https://github.com/tensorflow/probability/ ?

Created https://github.com/tensorflow/probability/pull/1147, following up there with requested changes from this thread.

This was added to tensorflow probability
https://github.com/tensorflow/probability/commit/9999dd41df50157e09656d07b2d4dd49136a9288?branch=9999dd41df50157e09656d07b2d4dd49136a9288&diff=unified
See https://github.com/tensorflow/probability/blob/master/tensorflow_probability/python/stats/kendalls_tau.py for more details.

It would be nice if it could be exposed as metrics.

Kendall's tau doesn't fit the metrics streaming style well. There are approximate versions (see https://arxiv.org/pdf/1712.01521.pdf) that are more interesting and well suited to the metrics. I might try doing that.

Kendall's tau doesn't fit the metrics streaming style well. There are approximate versions (see https://arxiv.org/pdf/1712.01521.pdf) that are more interesting and well suited to the metrics. I might try doing that.

Interesting. Thanks for the resoruce

I've created a completely new streaming based algorithm for this abandoned PR
https://github.com/tensorflow/addons/pull/2423


Remove cloning of obsolete repos in internal packer build

/gcbrun

/gcbrun

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2302

Relevant Lines:
3996

💛  - Coveralls


Use cita-database of evm.

Description of the Change
All the files should be deleted.

Just check the commit: https://github.com/cryptape/cita/pull/710/commits/35e9f10bbb7b626f0a8ead291c8a4ae2a7202ee1


Improve Istio components PDB default setting

Motivation
The PDB of Istiod created in IstioOperator is set with minAvailable: 1 by default, if the replicas of Istiod is 1, the PDB will block the node scale down.
For making disruptionsAllowed>0, we need to set PDB with maxUnavaible: 1.
Same for Ingress Gateway
Modification

Pilot(istiod) PDB
Ingress Gateway PDB

Ref:
https://istio.io/latest/docs/reference/config/istio.operator.v1alpha1/#PodDisruptionBudgetSpec
Target effect:

by default, if the replicas of Istiod is 1, the PDB will block the node scale down.
@ciiiii @fantapsody how can we change istiod replicas to 2 for existing and new pool members? Otherwise this change might cause unexpected downtime.

We can change min size of hpa to 2. @gengmao


Bump default E2E Kubernetes version

What type of PR is this?
Bump default E2E Kubernetes version from 1.26.0 to 1.27.0

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm


Rename Eve Charlie in Superdense Coding

Description of Issue (if Applicable)
Eve is an eavesdropper most of the time in this context, so Charlie is better here.
What Changes Were Made?
Rename Eve Charlie.
How Was This Tested?
Anything Else We Should Know
I also fix Eve's message to Alice's message because message sender is Alice.
see: https://github.com/Qiskit/qiskit-textbook/compare/master...ikkoham:rename-eve-charlie?expand=1#diff-399841aaf8bdfb6d2b5cd0071ba898c1R111

Thank you!


[AUCT-813] link to non-auction sale WIP

I need help figuring out how to link the sale. It shows up as underlined text, but a click in the simulator does nothing.

Closing this, as the app sale pages aren't designed for Buy Now sales.


Transactions - Retiring as Sole (Direct Voter) last long until page reload, finishes as expected but with error message

To reproduce on test-sancho:

Be registered or - Register as Direct Voter
Retire - wait couple minutest if needed.
If status is not changing - Reload page
Wait until transaction finishes observe transaction update and card related to Direct (Sole voter) text change
Transaction is reported as invalid but result of Retirement is as desired

Compare with jam recording 3:39s

@pmbinapps Please revalidate that bug if it is still occurring after recent respins and wallet updates


Restriction parameter to prevent more than 7 parameters in the next PRs

i have a PR waiting to be pushed/suggested but i need these methods to be refactored first so i dont fall into more than 7 parameters compiler errors + the new design i suggest makes the methods more easy to maintain.
i know this does not add any more features but i hope this can be usefull for the quality and ease of maintenance there after.
the basics are sending a Restriction parameter instead of 4 (lower/upper bound and accept booleans), thus gaining 3 more free params to send for other features (next PR to filter RCs is waiting for this one to be accepted)
Thank you for your time

You can also suppress the checkstyle error by using
SuppressWarnings("checkstyle:maxParameterNumber")

Thanks, good to know there are such options !
i find it quite harsh for me to bypass the checkstyle and have like 4 to 5 booleans in the final methods of the next PRs.
I find it easier to digest to cut my PR into two (this one for the refactoring, and another for the exclude milestone/previews feature)

would it be ok for the team to merge this PR before the next feature, or do you prefer i merge my work into one PR ?

simpler PR easier to review - it is ok for me for splitting


increase the height of preview-area and input-area

Description
This PR Fixes #36
Checklist

[x] I have tested the changes.
[X] I have updated documentation if necessary.
[X] I have assigned this PR to myself.
[X] I have added the appropriate

Related Issues
"When I write something in the Markdown input area, the content may extend beyond the boundary of the preview area."

@brijeshbhikadiya, you are targetting wrong html elements, target specific divs and give height : 100vh

okay


Refactor actions script

@Jamm02 the installation fails on macOS check this https://github.com/SRA-VJTI/Wall-E/actions/runs/3951413427/jobs/6765234345#step:3:58220

Add at the top of walle_install.sh
set -e

Exit immediately if a pipeline (see Pipelines), which may consist of a single simple command (see Simple Commands), a list (see Lists of Commands), or a compound command (see Compound Commands) returns a non-zero status.

Installer fails for Ubuntu as well https://github.com/SRA-VJTI/Wall-E/actions/runs/3951413427/jobs/6765234345#step:4:17

@Jamm02
Define a _sudo function
_sudo(){
    if ! [ $(id -u) = 0 ]; then
       echo "The script need to be run as root." >&2
       exit 1
    fi
    if ! command -v sudo &> /dev/null
    then
        "$@"
    else
        sudo "$@"
    fi
}

Replace sudo with _sudo


Include Zarr example/notebook

For users getting started, it would be nice to have either an example in a tutorial or a notebook showing how to plug KvikIO into Zarr.

It would be really great if this example showed how to read compressed data.


Patch for #10 and fixed changed boolean for unchanged "present" state

Patch for #10. Also I noticed when requesting compatible version like 11.0.* for state='present', and no newer version is available, it's returning changed = True. I've modified it to check the presence of magic text to say changed = False (same way as for state='latest').

This looks fine, but I don't use this anymore so can't put it to test. @ellisonbg would you be able to test this?

Hi, sorry for taking so long on this review. It looks like a859e6d has some overlap with #14, can you check and make sure it still makes sense to merge this as is or if it'd be best to only merge the checks implemented in c7f8f91?


[feature request] donation

Is it possible to donate you and your development?

Thank you for your support. I haven’t joined the github sponsorship program yet. I’ll apply for it now and I will update it in the document later

I'm in China, it seems that GitHubSponsors is not supported temporarily

Hmm... Is Bitcoin or Paypal allowed in China?

Yes, I have a paypal account: https://paypal.me/benboba

Done.


Decide version convention

We can do semver or a variant of calver.

I think we can start with semver and if there is a good argument for calver we can make the switch later (doing it the other way round is a lot more tricky!)

I'll take semver as decided, then.


remove excess step and explain better in house_of_lore

Making fd pointer of victim chunk point to itself should be excess according to the latest malloc.c source code and I have tested it to prove it works fine :)

Thanks!


Upgrade Web3j to v4.8.4

[ ] Remove the Web3j folder and re-wire the project to use the StructuredData and StructuredDataEncoder from Web3j which should now be working.
[x] Test EIP712v4 (Arrayed version of EIP712) using https://danfinlay.github.io/js-eth-personal-sign-examples/
[x] Test EIP712v4 using Gnosis transaction.
[x] Merge into the EIP1559 branch and remove the extra handling for EIP1559, pulled from v4.8.4 which is now in v4.8.4
[x] Test EIP1559 and resolve the unknown params
[x] Test ETH transaction send.
[x] Test Contract sending (can use PokeToken Token script)
[x] Test Token sending; ERC20, ERC721, ERC1155

All complete except for StructuredData and StructuredDataEncoder which require a private -> public conversion for displaying the signing message for the user.


question regarding encoding of Siksa guru

@aditiagunawan,
Can you check this encoding please?
<app><lem type="norm">tumiṅgalakəna</lem><rdg wit="#A">tumigala:kna</rdg><rdg wit="#B">tumigalakna</rdg><rdg wit="#C">tu<unclear>maṅga</unclear><gap reason="lost"/></rdg></app>

Something seemed wrong, while I was working on the punctuation.
Thanks,
Best

same here
<app><lem type="emn">utər</lem><rdg wit="#A">Utar</rdg><rdg wit="#B">Utir</rdg><rdg wit="#C">Uta<gap reason="lost"/></rdg></app>

@ajaniak : what is the irrgularity you observe and what is the connection with punctuation? do you mean that <gap reason="lost"/> should be complete with @quantity and @unit?

Yes in both those context, the <gap/> needs to be completed. It is an old version of the encoding before the introduction of the <lacunaStart/> and <lacunaEnd/>.

In SMG:
<app><lem wit="#A">paknanya pasuklasvanita lavan maṅəyəh</lem><rdg wit="#B" cause="lexical">Anada Aṅəyə pa<gap reason="lost"/></rdg></app>

<app><lem type="norm">hyaṅ</lem><rdg wit="#A">hya</rdg></app> atma, lumaku paknanya, pamətuan ikaṅ hədap, ya</lem><rdg wit="#B"><gap reason="lost"></gap></rdg></app>

@ajaniak : thanks!
@aditiagunawan : please make the necessary corrections and then close this issue.

@ajaniak
Done! Would you like to check it?

It should be good! I will let you know if I see something else


Stop using .vspreview

should mention this depends on https://github.com/Setsugennoao/stgpytools/pull/10


Verify the Snyk OSS test (SCA) and limits

Snyk OSS SCA test appears to be failing due to Gradle multi-project.  And hit Snyk scan limits.  Perhaps have this test run on demand rather than on checkin.

removed --report to hopefully avoid this.  also added --all-projects to SCA to get both client and server.


Registry Template

A template for registering existing data formats including legacy formats for oracle data serialisation.

merged.


Couchbase client upgrade to v3.0.0

Upgrade Neon.Couchbase to use the Couchbase v3.0.0 client.  This will probably require some changes to support new functionality.  We're upgrading from v2.7.15.
The Couchbase v3 SDK is a huge change from v2.x:
https://docs.couchbase.com/dotnet-sdk/3.0/project-docs/migrating-sdk-code-to-3.n.html
I'm not entirely sure that it's worth migrating to the new SDK and if we did, we should probably name the nuget something different like Neon.Couchbase3 or something and continue to publish the older one.
I also wonder how much more effort we'll want to put into Couchbase, especially if YugabyteDB ends up being any good.
This is going to be non-trivial.  I'm going to up this on the backlog.

After thinking about this, I believe we may end up deprecating Couchbase support.  The 3.0.0 change is just to big and Couchbase community is really too crippled to be of use for many use cases.


Bypassing MFA for a subpath

Describe the issue
I am trying to setup a service so that when a user goes to /service they are prompted for MFA (caddy-auth-portal).  The service sits at /service/app and other sub paths.  I need to be able to exclude /service/rest from MFA as the mobile app does not support it.
Configuration
    route /service* {
		jwt
		@whitelisted  {
			path /rest*
		}
		reverse_proxy @whitelisted <IP_ADDRESS>:9999
	}

Version Information
http.authentication.hashes.bcrypt v2.4.3
http.authentication.hashes.scrypt v2.4.3
http.authentication.providers.http_basic v2.4.3
http.handlers.authentication v2.4.3
http.authentication.providers.jwt v1.3.15
http.handlers.authp v1.4.21

Expected behavior
A user is not prompted for MFA when they go to /service/rest*, but they are for all other subdirectories under /service.
Additional context
I thought this article would get what I needed, but I am not sure whitelisting is what I need as this example is not using jwt.

@samcro1967 , have you looked at “bypass” in jwt docs?

@samcro1967 , I also suggest considering setting up two portals with their own keys and endpoints. One regular, another mobile.

Thank you!  Bypass seems to be working.  That is what I needed.  Here is the config I ended up with for future reference by others.
    route /service* {
		jwt {
			bypass uri prefix /service/rest
			acl rule {
				match role user admin authp/admin authp/user
				allow stop log debug
			}
			acl default deny
		}
		reverse_proxy <IP_ADDRESS>:4533

What would be the advantage of separating them into different portals?  Is it just to keep jwt protected URLs in one portal and non protected in another so it is easier to tell what is protected and what is not when looking at Caddyfile or is there another advantage?

What would be the advantage of separating them into different portals?

You could potentially implement different security controls for different paths. For example, if a user accesses mobile frontend check user agent, etc. This way you could protect against bots.


not works with search_fields

is there any chance to support searching the key directly
// Works
search_fields = ('name_en','name_de', 'key')
// Not Works
 search_fields = ('name', 'key')

No, since name isn't a real field, just a descriptor.
You can write search_fields = [*Model.name.fields, "key"] if you want to automatically include all concrete fields.
The rationale for this design decision is here: https://github.com/matthiask/django-translated-fields#other-features

Thank you <3
how i can buy you a coffe for this great pip.


Don't call last_insert_id if not needed

Fixes #1357

I don't see issues/1357 being run, but I assure it works on my machine(TM)

Refactored. How about now?


fix: postgres datasource not able to connect to schema other than public

Describe the bug
The configured schema is not propagating to the Postgres connection. The current status is
 url = URL.create(
        drivername="postgresql",
        username=self.data_connection.get("username"),
        password=self.data_connection.get("password"),
        host=self.data_connection.get("host"),
        port=self.data_connection.get("port"),
        database=self.data_connection.get("database"),
    )
    engine = create_engine(url)

schema name should be part of engine creation.
Expected behavior
Postgres Datasource should table the schema name in the configuration. And the data source should connect to only that schema.
  - name: search_staging_db       # Data source name
    type: postgres                # Data source type is Postgres
    connection:
      host: <IP_ADDRESS>
      port: 5422
      username: !ENV ${DB2_USER}  # Username to use for authentication ENV variables
      password: !ENV ${DB2_PASS}  # Password to use for authentication ENV variables
      database: dc_db_2
      schema: custom_name

The code should accommodate the schema name
schema = self.data_connection.get("schema")
#....
engine = create_engine(
    url,
    connect_args={'options': f'-csearch_path={schema}'},
    isolation_level="AUTOCOMMIT"
)

fixed with #62


Smart Contract MVP: UBQ bonding curve to burn UBQ and mint NFTs

To do
Implement the logic "Buy with any token" according to the discussion
https://github.com/ubiquity/ubiquity-dollar/discussions/428
The issue is related to
https://github.com/ubiquity/ubiquity-dollar/issues/429
https://github.com/ubiquity/ubiquity-dollar/issues/434

I am interested in this task. I will get on it with your approval. This should be my first contribution, and I am looking forward to distinguishing myself.

I am interested in this task. I will get on it with your approval. This should be my first contribution, and I am looking forward to distinguishing myself.

Hey @Hendobox before you get assigned:

This issue will be devided to smaller issues with separate pricing.
This issue relates to #429

@Hendobox Meanwhile, if you have clarifing questions on this issue, please post them here https://github.com/ubiquity/ubiquity-dollar/discussions/428

I am interested in this task. I will get on it with your approval. This should be my first contribution, and I am looking forward to distinguishing myself.

Hey @Hendobox before you get assigned:

This issue will be devided to smaller issues with separate pricing.
This issue relates to UI component: UBQ bonding curve to burn UBQ and mint NFTs #429

@Hendobox Meanwhile, if you have clarifing questions on this issue, please post them here #428

Oh, clear. Sounds cool.

Still an active issue? Would love to get assigned this.

I would love to make suggest a pull request for this. Still active @sergfeldman @pavlovcik

@sergfeldman you should include context/the title at least next to your referenced issues because its confusing for me to follow a chain of these

I would love to make suggest a pull request for this. Still active @sergfeldman @pavlovcik

Yes but to clarify we have two parts of this. The first part must be completed first, and then the second part is possible. You are able to do only the first part and not do the second part if you wish.

The first part is to fork a bonding curve smart contract. Please let us know which you plan to work with and why before beginning. Along with this, you must make a UI to interact with this smart contract (it likely will simply be a deposit UBQ input field with a submit button.)

The second part is to make another UI that will leverage Uniswap to make it more convenient for users to mint a UbiquiStick even if they do not own UBQ. The Solidity development would be minimal as it would simply be a function to swap on Uniswap from any asset to UBQ, and then call the mint function by depositing that UBQ. This will be more involved in the UI because it should also query the uniswap-router for price estimation.

Comment here which bonding curve contract you plan to fork and then if there are no issues with your selection based on the discussion here, I'll assign you!

@sergfeldman For the issues I understand simple deposit UBQ and mint NFT contract. I would like to work on this using a fork from Bancor Protocol. https://github.com/bancorprotocol/contracts

One small detail: we should be able to recover the UBQ from the bonding curve. The plan is that we can do more with the tokens, like manually add it as liquidity in the market. You should be able to copy the access control code UBQ_MINTER or something like that from another contract to enable this feature only for the admin wallet.

/wallet 0x43BbdE754e2e4e8A3b1282D1037Ce1a1B7c577C2

https://etherscan.io/tx/0xda3cd905c7a6b40a2effa41a5d96ce4919afe6047e5fc25b963be754e7863a44


⚠ Hartman Foundation has degraded performance

In 849bf73, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 3072 ms

Resolved: Hartman Foundation performance has improved in b4831b0.


function and test implemented

Description of the Change
method
Buffer &Buffer::putBuffer(const Buffer & other); implemented
as well as test for this method.

Benefits
you can now use putBuffer for putting another buffer without converting it to vector manually.

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a202a19). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #52   +/-   ##
=========================================
  Coverage          ?   75.91%
=========================================
  Files             ?       76
  Lines             ?     2877
  Branches          ?        0
=========================================
  Hits              ?     2184
  Misses            ?      693
  Partials          ?        0

Impacted Files
Coverage Δ

core/common/buffer.hpp
100% <ø> (ø)

core/common/buffer.cpp
93.75% <100%> (ø)

test/core/common/buffer_test.cpp
100% <100%> (ø)

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 a202a19...7d6c9ec. Read the comment docs.


ParseArguments explodes with uncaught exception when provided malformed arguments

I have the following options class:
class Options
{
    [Value(0,
        MetaName = "size",
        Required = true,
        HelpText = "Desired size of the target database")]
    public string Size { get; set; }
}

My Main method is set up like so:
public static int Main(string[] args)
{
    var result = Parser.Default.ParseArguments<Options>(args);
    return result.MapResult(...);
}

When the application is invoked with the arguments --size "300 GB", the following uncaught exception occurs in ParseArguments (the first statement in Main):
Unhandled Exception: System.ArgumentException: author
   at CommandLine.Text.CopyrightInfo..ctor(Boolean isSymbolUpper, String author, Int32[] copyrightYears)
   at CommandLine.Text.CopyrightInfo.get_Default()
   at CommandLine.Text.HelpText.AutoBuild[T](ParserResult`1 parserResult, Func`2 onError, Func`2 onExample, Boolean verbsIndex)
   at CommandLine.Text.HelpText.AutoBuild[T](ParserResult`1 parserResult)
   at CommandLine.Parser.<>c__DisplayClass17_0`1.<DisplayHelp>b__1(IEnumerable`1 _, TextWriter writer)
   at CSharpx.MaybeExtensions.Do[T1,T2](Maybe`1 maybe, Action`2 action)
   at CommandLine.ParserResultExtensions.WithNotParsed[T](ParserResult`1 result, Action`1 action)
   at CommandLine.Parser.DisplayHelp[T](ParserResult`1 parserResult, TextWriter helpWriter)
   at AgentAutomation.DataFiller.Program.Main(String[] args) in D:\depot\git\AgentAutoUtils\execution\DataFiller\Program.cs:line 24

It is totally correct behavior for parsing to fail, since I've configured the Size argument as a positional value rather than an option, and so I need to remove the --size.
However, there should be no input that results in an uncaught exception: all errors arising from malformed input should be passed to the error handler lambda. A user entering nonsense should at the very least see the help screen.

Have you defined an AssemblyCopyrightAttribute or AssemblyCompanyAttribute in your application properties and is it non-null/empty? It isn't breaking because of your input but because it's trying to print the help info (including copyright) and there is no author available.

No, I don't have those. In that case it should probably skip whatever
assembly metadata it was trying to output instead of crashing with an NRE.
On Jul 8, 2016 7:19 PM, "Dan Nemec"<EMAIL_ADDRESS>wrote:

Have you defined an AssemblyCopyrightAttribute or AssemblyCompanyAttribute
in your application properties and is it non-null/empty? It isn't breaking
because of your input but because it's trying to print the help info
(including copyright) and there is no author available.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gsscoder/commandline/issues/332#issuecomment-231494836,
or mute the thread
https://github.com/notifications/unsubscribe/ADgPyEFCbfCoEdUaxQBrLp7-60ZsIeC6ks5qTtrvgaJpZM4JIQDj
.

Yes, that's a really good idea.

I've just merged a fix for this. Thank you @anthonylangsworth!
Let me know if there are further issues around missing assembly attributes.


Orientation listener not longer firing after calling lockToPortrait() or lockToLandscape()

Pretty much what the title says. When I lock the orientation in any direction, the event listener for the orientation change is no longer fired until I call unlockAllOrientations. I'm not sure if this is the expected behaviour. The example code seems to suggest otherwise as it locks the screen on componentDidMount.
Here's my code:
export default class App extends Component {
  state = {
    showCard: false
  }

  constructor() {
    super()

    Instabug.startWithToken(Config.INSTABUG_TOKEN, [
      Instabug.invocationEvent.floatingButton
    ])
  }

  componentWillMount() {
    Orientation.lockToPortrait()
    Orientation.addOrientationListener(this.orientationDidChange)
  }

  componentWillUnmount() {
    Orientation.removeOrientationListener(this.orientationDidChange)
  }

  orientationDidChange = (orientation) => {
    const showCard = orientation === 'LANDSCAPE'
    this.setState({ showCard })
  }

  render() {
    return (
      <Provider store={store}>
        <PersistGate loading={null} persistor={persistor}>
          <SwitchNavigatorContainer />
          {this.state.showCard && <LandscapeWallet>
            <Wallet orientation='LANDSCAPE' />
          </LandscapeWallet>}
          <AppMessage />
          <OfflineSnack />
        </PersistGate>
      </Provider>
    )
  }
}

Since the orientation is locked in portrait mode, the event listener will never fire when the phone is in landscape mode and therefore the showCard will never be set to true.
Cheers

@pouyaemami I'm curious to if you're running this on iOS.
I just installed and set up this package in my project and I got the exact same thing. I could set orientation and the change would fire, but it wouldn't unlock.
I'm asking if you're on iOS because if you added this from the configuration:
`

(UIInterfaceOrientationMask)application:(UIApplication *)application supportedInterfaceOrientationsForWindow:(UIWindow *)window {
while ([[UIDevice currentDevice] isGeneratingDeviceOrientationNotifications]) {
[[UIDevice currentDevice] endGeneratingDeviceOrientationNotifications];
}
return [Orientation getOrientation];
}
`

I resolved this issue by commenting out the while loop that unsubscribes:
`

(UIInterfaceOrientationMask)application:(UIApplication *)application supportedInterfaceOrientationsForWindow:(UIWindow *)window {
//  while ([[UIDevice currentDevice] isGeneratingDeviceOrientationNotifications]) {
//      [[UIDevice currentDevice] endGeneratingDeviceOrientationNotifications];
//  }
return [Orientation getOrientation];
}
`

Unsure if that will help you or not, but it helped me move past this initial hurdle. If you found another solution I'd love to hear it.

@deefourple thanks for the tip! I am using both Android and iOS. I'm not sure where I would make a similar change for Android though 🤔

One more thing, I just looked at my AppDelegate.m file and I don't have the same thing as you. This is what I have and it's working as expected:
- (UIInterfaceOrientationMask)application:(UIApplication *)application supportedInterfaceOrientationsForWindow:(UIWindow *)window {
  return [Orientation getOrientation];
}

@pouyaemami I haven't actually tested android yet as I wanted to ensure this worked for iOS first.
Actually, in terms of the AppDelegate.m code if you remove the commented out part in my example, we do share the same code :).
The while loop was definitely messing up the orientationDidChange for me. If I find any issues/discoveries for android I'll be sure to reach out.

not sure why its not in the readme, but addDeviceOrientationListener can be setup the same way as addOrientationListener but fires regardless if a lock is set or not


Add warning for macOS users: df is size in 1K blocks

Fixed in v1.3.1: change Linux df to blocks of 1024 bytes to unify results


Fix compatible with casbin  v2.75.0

This PR added new LogError method to Logger:
https://github.com/casbin/casbin/pull/1294

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