Transform within default variables

Transforming px values to rem is not working inside default values in custom properties.
CSS below would be sufficient to reproduce the issue.
body {
  padding: var(--site-padding, 16px)
}

Desired outcome should be:
body {
  padding: var(--site-padding, 1rem)
}

Nonetheless, such piece of code is not transformed. Is this a flaw possible to fix or design choice?

Yes, it is a design choise, see this regexp
/"[^"]+"|'[^']+'|url\([^)]+\)|var\([^)]+\)|(\d*\.?\d+)px/g

it ignored all the var statements, I think it's because of the variable names, which can include something like --1px. We just need to improve regexp and there will be no such problem:
/"[^"]+"|'[^']+'|url\([^)]+\)|((?<=^|[^\w-])-?\d*\.?\d+)px\b/g

but
left: -10px;
lapse

是的，这是一种设计选择，请参看此正则表达式
/"[^"]+"|'[^']+'|url\([^)]+\)|var\([^)]+\)|(\d*\.?\d+)px/g

它忽略了所有的var语言句子，我认为这是因为变量名，它可以包含类似的--1px。我们只需要改变进入正则表达式就不会出现这样的问题：
/"[^"]+"|'[^']+'|url\([^)]+\)|((?<=^|[^\w-])-?\d*\.?\d+)px\b/g

left: -10px; not working

any progress?

@cuth Please add the ability to convert px to em inside var in the callback, while ignoring custom variables.


Is your feature request related to a problem? Please describe
现GPU引擎能否做出类似跑马灯的循环动画，效果图如下

Describe the solution you'd like
A clear and concise description of what you want to happen
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered
Additional context
Add any other context or screenshots about the feature request here

我通常的做法是
建模-拉直UV-制作透明贴图-制作UV动画

怎么制作UV动画呢？能提供一个示例吗？

参照官方示例：Sample_ConduitGeometry2.ts里面UV流动效果的实现。


Memory Leak

Memory leak on failing XML parser. Test adapted from t/11memory.t
use LibXML;
for ^10000 {
    try {
        LibXML.parse: :string("<X>foo</Y>");
    }
}

I notice the DESTROY method never gets called on the created LibXML::Parser::Context object.
$ perl6 -v This is Rakudo version 2019.07.1-306-gfce964231 built on MoarVM version 2019.07.1-180-gcf3b545e5

Err, it's even more basic. DESTROY method isn't being called on LibXML::Parser::Context for
use LibXML;
for ^10000 {
    LibXML.parse: :string("<X>foo</X>");
}

GC works for the simple case, of creating these objects directly:
use LibXML::Parser::Context;
for ^10000 {
     LibXML::Parser::Context.new;
}

Also worth noting that output LibXML::Document objects (which keep a reference to the context) are being destroyed.

This is enough to stop GC:
use LibXML::Native;
use LibXML::Parser::Context;
for ^10000 {
    LibXML::Parser::Context.new.set-native: xmlParserCtxt.new;
}

Further golfed to the setting of the error handling callback closure. Have raised Rakudo ticket https://github.com/rakudo/rakudo/issues/3184

Fixed in 0.0.8. Switched from closures to dynamic variables ($*XML-CONTEXT). LibXML::Parser::Context objects are being destroyed. 11memory.t is running leaner.


[history] more cautious in deciding domain state to make decisions on…

What changed?
When domain cache returned entity not found error, don't drop queued tasks to be more conservative.

Why?
In cases when the cache is dubious, we shouldn't drop the queued tasks.

Pull Request Test Coverage Report for Build 0186ec6f-f681-41f3-a94d-3710932d9573

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
136 unchanged lines in 17 files lost coverage.
Overall coverage decreased (-0.05%) to 57.072%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

service/history/queue/task_allocator.go
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

common/task/weightedRoundRobinTaskScheduler.go
1
89.64%

service/history/queue/task_allocator.go
1
53.33%

service/history/queue/timer_queue_processor_base.go
1
77.26%

common/peerprovider/ringpopprovider/config.go
2
81.58%

common/task/fifoTaskScheduler.go
2
84.54%

service/frontend/workflowHandler.go
2
59.86%

service/history/handler.go
2
47.15%

service/history/task/transfer_standby_task_executor.go
2
87.66%

service/matching/matcher.go
2
91.46%

common/persistence/statsComputer.go
3
94.64%

Totals

Change from base Build 0186ec4a-0629-4049-8935-b8ff08697b97:
-0.05%

Covered Lines:
85207

Relevant Lines:
149297

💛  - Coveralls

Given the contents of #5140, I think the error(s) in this func won't be logged... tbh we probably do want to log and/or metric issues here, since they'd be unexpected.  Currently they're created and lost forever.
But yea, 👍 for this change.  Accidentally-merged clusters could also be dangerous otherwise, with not-founds.

this might be unit-test worthy

I'll add unit test (along with the log, metrics) in the follow-up


Update Master

Description

Changed how options are exposed
Fixed Aspect Ratio, Rotation & Flip of the RawTexture
Improved Logs
Improved Performance (lower the amount of GC)
Update samples (vsync, disabled auto-rotation)
Tested with iOS (iPhone & iPad)


Add RHEL 6/7 params and remove ploperations/puppet requirement

This probably should have been a few different request/branches, but it's an easy merge if you take a look.
Removes requirement for ploperations/puppet by using built-in $::settings::ssldir
Fixed a pathing problem for generate_bacula_dhkey exec
Added distinct params for RedHat split into RHEL6 and RHEL6 sections.
Thank you,
Jordan

I'd be happy to get rid of the plops-puppet requirement, but I think the ssldir setting should be ported over.  Last I looked I couldn't work out a way from the agent to get the ssl directory the agent was using.

I'm using a non-standard puppetssl directory and it looks like it's working as expected.
[root@bacula-test ssl]# diff /etc/pki/puppet/certs/bacula-test.jordantest.local.pem /etc/bacula/ssl/bacula-test.jordantest.local_cert.pem
[root@bacula-test ssl]#

My pupet.conf is set like this on the agent
ssldir = /etc/pki/puppet
cacert = /etc/ipa/ca.crt

Oddly "puppet config print ssldir" returns /etc/puppetlabs/puppet/ssl (puppet4) but it appears that $::settings::ssldir uses the correct ssldir that is set in my agent config.
However, the ssldir and cacert are the same on the agent and the master.

https://docs.puppetlabs.com/puppet/latest/reference/lang_facts_and_builtin_vars.html#puppet-master-variables
In order to remove the plops-puppet requirement, I'd like to see something that doesn't use the $settings variable to allow for differing paths on master and agent (like in my environment).  Perhaps one approach could just force the user to specify the path to the cert, key and ca files they would like to use, and then the module would just copy them into a place they can be used.
This would allow users to use whatever SSL implementation they desire, and the bacula module gets a bit more flexible.

I'd like to get rid of the ploperations/puppet dependency too. Just a quick thought: Would it be possible to deliver a custom fact which returns the result of puppet config print ssldir?
On the other hand it wouldn't be too dumb to just copy the code from https://github.com/puppetlabs-operations/puppet-puppet/blob/master/manifests/params.pp#L4 to get the ssl_dir right. Currently we're depending on a whole module just to get this parameter. It's a matter of taste, I guess.

https://github.com/xaque208/puppet-bacula/issues/45

https://github.com/xaque208/puppet-bacula/pull/49


Update CLI to pull data same as the api however, write the entire payload instead of reducing it.

Now that we have a new way to get game data, and it can pull the token for an environment variable we can update the CLI to use the new way.

[ ] Update CLI to call the same code as the API
[ ] Write the entire response from the api as JSON. This file should be larger than the one returned by the API.

Code to pull the data already exists so this is just plugging it into the CLI


Session is null

I'm currently having an issue where the getting of the state parameter from the session is returning null and failing the check on callback. Any ideas why that might be?

It seems like the names it was using to get the data were different. When the data was being set it was just "oauth". In the callback it was "Google2Client". Know why that might be?

Very strange! I just tested it successfully on this demo: https://github.com/pac4j/spring-security-pac4j-demo
Either the name of the client is set explicitely or it's the name of the class (by default, in that case: "Google2Client").
Are you really sure about the "oauth" name?


Update to Swift 3

A few changes to make it work with Swift 3

Thank you very much @jbarmanet Much appreciated!
I also added another commit with recommendations that Xcode 8.1 gave, and also migrated the example project that resides within the repo.

Glad I could participate, you made a nice & efficient library !!  :)

Le 16 nov. 2016 à 20:32, Mats Melke<EMAIL_ADDRESS>a écrit :
Thank you very much @jbarmanet https://github.com/jbarmanet Much appreciated!
I also added another commit with recommendations that Xcode 8.1 gave, and also migrated the example project that resides within the repo.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/melke/SlimLogger/pull/12#issuecomment-261047489, or mute the thread https://github.com/notifications/unsubscribe-auth/ADPx1jihCYDFa5mHn6Z5jI9TYNlAH275ks5q-1pigaJpZM4Ksqos.


Can't communicate client to client

Hello, I just installed openvpn on Debian 10 server (Exm IP: <IP_ADDRESS>).
I've create two clients A (<IP_ADDRESS>) & B (<IP_ADDRESS>). VPN works as expected. My server (<IP_ADDRESS>) can ping to both clients if they are connected. Clients also can communicate/ping to server (of course). However, the clients can't ping each other (<IP_ADDRESS> <-> <IP_ADDRESS>). Is there any additional step I need to do?
Thanks.

Nevermind, I've added the below line in server.conf file and internal communication is working.
client-to-client
Thank you


Fix missed gnupg2 in ucx example dockerfiles (#6111) [skip ci]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>fix https://github.com/NVIDIA/spark-rapids/issues/6112
cherry picked #6111 to fix dockerfiles in current github page

lets keep the gh-pages as it is for now. We can wait for the actual 22.08 release to fix the page


Consider using FastBroadcast.jl

I have observed that @.. from FastBroadcast.jl instead of<EMAIL_ADDRESS>can improve the performance. It might be worth trying this out here - at least in the future for models where the elliptic solves do not dominate the runtime.

Sounds good. I can have a look when I find some time. But, PRs are also welcome of course :smile:

I will look into this when implementing the hyperbolic approximation of the Serre-Green-Naghdi equations


Rule34video link redirect and duration filter

Source name and language
Rule34video 13.5 (English)
Describe your suggested feature
It's usually very hard to search on Rule34video (extension) so link redirections/opening-links can help a lot. also there's time filter already avaliable on website so if it's possible to implement that then that would be great.
also, for some reason search on Rule34video site and extension shows completely different results. so syncing it should help a lot. (site results preferred).
opening links example:

Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I have updated the app to version 0.13.5.
[X] I will fill out all of the requested information in this form.

fulfilled, closing.

Reopened as duration filter wasn't implemented

should I open another issue for it or you can add error option in it insted of opening/downloading it and file format not supported things.


log-backup: use row-level memory usage statistic for initial scan

What is changed and how it works?

Issue Number: Close https://github.com/tikv/tikv/issues/15714
What's Changed:
This PR removed the semaphore that may make TiKV panic, and refined the place where we consume the memory quota. Now, we will try to consume all messages as soon as we have reached the memory quota.

Check List
Tests

Unit test
Manual test (add detailed scripts or steps below)

Side effects

Performance regression

Release note
Fixed a bug that may cause log backup's checkpoint get stuck with huge rows.

/merge

/run-cherry-picker


⚠ Processors has degraded performance

In 2d9b501, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 42 ms

Resolved: Processors performance has improved in ceef1f2.


appengine/go11x/cloudsql: allow override of socket file

Make the socket file overrideable for the CloudSQL sample to make local dev more flexible.

lgtm

Pushed, @broady could you merge at will? I don't have access


fix(clerk-js): revert #1391

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[x] npm test runs as expected.
[x] npm run build runs as expected.

Reverts #1391 as it was causing errors when loaded.

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Add "compile" and "render" functions to Environment

Once these exist, run some existing (or new) "compile" and "render" commands through the environment as a front-end.
The arguments to these functions should inform how a manifest could be targeted to perform the invocation (as if it's a Makefile) and possibly how a dependency relationship can be formed between them (also as if it's a Makefile).

We may want / need to re-evaluate what's required in a manifest and what's not. Possibly we just want a manifest to hold these compile and render definitions and the schemas for those elements should probably be able to specify their own directories, so that you can persist with just one monolithic manifest (if you choose).
Probably we just want something like:
job_name:
  configs:
  - "optional"
  variables:
  - "optional"
  templates:
  - "optional"
  schemas:
  - "optional"
  depends_on:
  - "job_name_a"
  - "job_name_b" # should order matter for these or should it be reasonable to build them in parallel?
  command: "render" # or "compile"
    arg1: "test"
    arg2: "test"

The manifest schema should possibly just be in code, so that it's easier to build a complex relationship between elements.

We may need a "noop" command to support "phony" targets.

Determining updates required could get complicated fast, make's strategy of using file timestamps is good but we may want to try something like checksums on the dependencies, or a combination of both where some kind of "audit" can make sure the timestamp strategy is still working, and if not update our cache of information.

Made some progress on this:
configs:
  type: list
  schema:
    type: "string"

schemas:
  type: list
  schema:
    type: "string"

templates:
  type: list
  schema:
    type: "string"

variables:
  type: list
  schema:
    type: "string"

compiles:
  type: list
  schema:
    type: "dict"
    schema:
      name:
        type: "string"
      b:
        type: "string"
      c:
        type: "string"

renders:
  type: list
  schema:
    type: "dict"
    schema:
      name:
        type: "string"
      b:
        type: "string"
      c:
        type: "string"

Compiling and rendering need to be able to specify directories, additive to the ones optionally provided above.
Need to settle on an initial set of arguments / front-end for these schemas. From there, we can perform the invocation to the underlying "compile" or "render".
We can probably get rid of cmd_compile and cmd_render at some point, too. Maybe worth keeping them though

Compile should be able to specify path from overall config root.

Compiling is now possible, what still remains though is loading / accepting new directories as specified in the target (true for both compile and render) and the future scope of having some notion of dependencies that need to be resolved first.
I don't know if the dependency resolution is even necessarily useful, though.

The dependency relationship should be:
compilation depends on the config data, schema data, and variable data. If any of those are changed, and compilation for the target is requested, it should be re-compiled
rendering depends on a set (one or greater) of compilation artifacts, they need to be combined into one monolithic dict, it also depends on the loaded templates and whether or not those have changed
This brings us to a generic solution for evaluating any arbitrary directory at load time. It might be easiest to compute the hash or something of the whole thing (or keep a count of total files, etc)

operation should be an enum, handle needs a stack for the dep tree and a "visited" list

cache should be in terms of hits and misses, we should split out the cache entity entirely

Decided operation shouldn't be enum. Open items:

allow compile to specify a path
don't create a namespace (costly) if an operation doesn't load any new data

Once these two items are complete, we can 1. implement render and 2. add support for dependency resolution.

both of the above tasks are complete

Closing this issue to make more that are more poignant.


docs: fix spelling errors in bt_spp_initiator

when checking the menuconfig, we see that the first option should be 'component config'

 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.

Thanks for your contribution.

@Dazza0 I have squashed them, I believe. The two commits are now one.

Thanks for your contribution again, changes merged with https://github.com/espressif/esp-idf/commit/86d2e09629c8a555284ccf170db53ff69c4340be.


pango: maintenance 1.40.11 -> 1.40.12

The patch is included in this release.
Motivation for this change
Patch URL for pango no longer works so updating to newer build that doesn't need the patch
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build pango

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/pdy1li3wn49qgghmp6vrrl4gzq3lahqq-pango-1.40.12/lib/libpango-1.0.so.0.4000.12
shrinking /nix/store/pdy1li3wn49qgghmp6vrrl4gzq3lahqq-pango-1.40.12/lib/libpangoft2-1.0.so.0.4000.12
shrinking /nix/store/pdy1li3wn49qgghmp6vrrl4gzq3lahqq-pango-1.40.12/lib/libpangoxft-1.0.so.0.4000.12
shrinking /nix/store/pdy1li3wn49qgghmp6vrrl4gzq3lahqq-pango-1.40.12/lib/libpangocairo-1.0.so.0.4000.12
stripping (with flags -S) in /nix/store/pdy1li3wn49qgghmp6vrrl4gzq3lahqq-pango-1.40.12/lib
patching script interpreter paths in /nix/store/pdy1li3wn49qgghmp6vrrl4gzq3lahqq-pango-1.40.12
checking for references to /build in /nix/store/pdy1li3wn49qgghmp6vrrl4gzq3lahqq-pango-1.40.12...
shrinking RPATHs of ELF executables and libraries in /nix/store/qsblqn39843s0cwqdzybzhxxghxl8y3p-pango-1.40.12-devdoc
patching script interpreter paths in /nix/store/qsblqn39843s0cwqdzybzhxxghxl8y3p-pango-1.40.12-devdoc
checking for references to /build in /nix/store/qsblqn39843s0cwqdzybzhxxghxl8y3p-pango-1.40.12-devdoc...

Success on x86_64-darwin (full log)
Partial log (click to expand)
/nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12/lib/libpangoft2-1.0.0.dylib: fixing dylib
/nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12/lib/libpangoxft-1.0.0.dylib: fixing dylib
/nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12/lib/libpango-1.0.0.dylib: fixing dylib
/nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12/lib/libpangocairo-1.0.0.dylib: fixing dylib
/nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12/lib/libpangoft2-1.0.0.dylib: fixing dylib
/nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12/lib/libpangoxft-1.0.0.dylib: fixing dylib
stripping (with flags -S) in /nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12/lib
patching script interpreter paths in /nix/store/k4alk6mn2hlfah62cbyjz8ywaryqzyss-pango-1.40.12
patching script interpreter paths in /nix/store/hd61rhilm842jgyc1nq76fifn8cs32dh-pango-1.40.12-devdoc
/nix/store/00lk1z886v2vzgvqjp36gqx6s7rk7rlf-pango-1.40.12-bin

Thanks!

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/wpbxdf9c3j1ydryahdhg60f1r7wmgysy-pango-1.40.12/lib/libpangoxft-1.0.so.0.4000.12
shrinking /nix/store/wpbxdf9c3j1ydryahdhg60f1r7wmgysy-pango-1.40.12/lib/libpangoft2-1.0.so.0.4000.12
shrinking /nix/store/wpbxdf9c3j1ydryahdhg60f1r7wmgysy-pango-1.40.12/lib/libpango-1.0.so.0.4000.12
stripping (with flags -S) in /nix/store/wpbxdf9c3j1ydryahdhg60f1r7wmgysy-pango-1.40.12/lib
patching script interpreter paths in /nix/store/wpbxdf9c3j1ydryahdhg60f1r7wmgysy-pango-1.40.12
checking for references to /build in /nix/store/wpbxdf9c3j1ydryahdhg60f1r7wmgysy-pango-1.40.12...
shrinking RPATHs of ELF executables and libraries in /nix/store/zx91343jc88a52f2r13dkagi4807spcj-pango-1.40.12-devdoc
patching script interpreter paths in /nix/store/zx91343jc88a52f2r13dkagi4807spcj-pango-1.40.12-devdoc
checking for references to /build in /nix/store/zx91343jc88a52f2r13dkagi4807spcj-pango-1.40.12-devdoc...
/nix/store/xmcpb90avffgvfqnd1isshqhl7226cib-pango-1.40.12-bin


[BEAM-6619] [BEAM-6593] update gradle to include all py3 it tests

This is is part of a series of PRs with goal to make Apache Beam PY3 compatible. The proposal with the outlined approach has been documented here: https://s.apache.org/beam-python-3.
This PR updates Gradle to run the IT tests the same way as for python 2 and skips the tests that have unsolved errors
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

Run Python PostCommit

Run Python PostCommit

cc: @tvalentyn

Run Python Precommit

LGTM to me. Thanks!


connectionFromArraySlice incorrect slice

ArrayConnection::connectionFromArraySlice is not slicing corretly when using "after" or "before" as parametersin the query

Thanks, will check soon.

I have a fix ready, just trying to understand how to create a pull request

merged and released in v<IP_ADDRESS>


Minor Fixes

Background
Add minor changes such as typo fix or tag suggestions.
Changes

Add Okta tag to Okta rules to maintain consistency. Currently, they're tagged as Identity & Access Management whereas other rules such as OneLogin has OneLogin tag
Fix typo in Github.Repo.Created -- the action is repo.create; the rule never detected repo creation because of typo
Add suggested fix for Github.Team.Modified -- current rule uses deep_get(event, 'data', 'team') which always return None

Testing

Observed issues and tested it via our internal Panther instance
Also ran panther_analysis_tool

 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.

Thanks you @adaadb6 for putting up this PR and @reedloden for the useful reference!
Verified with log data that the logs we see align with the provided documentation.
Looks like some merge conflicts -- we'll get those straightened out and get this merged in :)


Telemetry?

Basically the same question for all our other distributable solutions, with the same discussion points as: https://github.com/swellaby/generator-swell/issues/111
@traviskosarek elevating our discussion from Slack here for broader views.
My sense is that whatever we end up deciding can likely be applicable for all our VSTS extensions. Key questions again are around whether or not users should be able to opt out (via task config), I think they should be able to, and the last piece is what data point(s) we should collect.
Some basic usage info would be nice, and some error capture too (environment overview, like task version, node.js version, etc., no user info)

I agree with no user info. We aren't trying to be big brother, just get information on how our extensions are being used :)
Along the same lines, I think being able to opt out is important. In Slack we already mentioned using a checkbox within the task to control this. I think a pre-checked checkbox in a pre-collapsed accordion is best just from a "we can likely get the data we are looking for" and "it is not in a shared context with the task". That said, I am not sure if this can be thought of as "shady" because it would be initially hidden, but fully accessible. Thoughts?

Agreed. Task configuration option to opt out. I think having a fully detailed set of documentation that outlines exactly what our telemetry is capturing, including links to capture/send points in the source code, could help mitigate some concern. The task source be open source (and also viewable post install via tfx cli tools) should hopefully help folks feel more concerned.
In part, i'd really just like to get some info to help us support users when an issue arises

Some proposed data points we could potentially gather/include in Telemetry Data:

Task version
VSTS Agent version
OS?
Node (or Powershell for any PS impl tasks we have) version
Task execution duration
Error info (which needs more discussion)

Some examples of data points that I believe we should not include in telemetry data

don't include the VSTS account name
don't include IP address/hostname from env where task is executing
don't include any sensitive task input values
don't include any VSTS user data

I agree with the above two comments of "proposed data points to capture" and the data points "we should not capture".
As far as "Error info", I think a pure stack trace could potentially add in some of the sensitive information we do not want to grab. This could be any of the "do not include" items or it could be some secret variables that are set in the build or within the vsts instance. Perhaps I am wrong?
I think here the safest bet is some specific error message/code that allows us to pinpoint where in the app this happened. That said, I would like more information if possible.

Closing this as it no longer seems necessary, and the effort/cost of implementing significantly outweigh any potential benefits.
The extension has been workin successfully for a long period of time, and the logging output should provide enough information for any users needing to report bugs.
@traviskosarek - I think Telemetry and the points made here could still be relevant for the git-mirror extension however.


Support detect timeout and available state via pychonet

Rel. https://github.com/scottyphillips/echonetlite_homeassistant/issues/156

Fixed #156


added test for issue #847

testcase to trigger this error as mentioned here
https://github.com/lnbits/lnbits-legend/issues/847

This PR should ideally include the fix, not just a test that fails.

Codecov Report

Merging #848 (7b6e15a) into main (b83fb1d) will decrease coverage by 2.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #848      +/-   ##
==========================================
- Coverage   41.82%   39.68%   -2.14%
==========================================
  Files         216      218       +2
  Lines       11394    11983     +589
==========================================
- Hits         4765     4756       -9
- Misses       6629     7227     +598

Impacted Files
Coverage Δ

lnbits/decorators.py
81.30% <100.00%> (+0.35%)
:arrow_up:

lnbits/wallets/lndrest.py
17.69% <0.00%> (-21.19%)
:arrow_down:

lnbits/wallets/cln.py
22.11% <0.00%> (-12.51%)
:arrow_down:

lnbits/db.py
80.80% <0.00%> (-9.61%)
:arrow_down:

lnbits/wallets/macaroon/macaroon.py
27.53% <0.00%> (-5.80%)
:arrow_down:

lnbits/extensions/watchonly/views_api.py
22.03% <0.00%> (-2.43%)
:arrow_down:

lnbits/bolt11.py
78.57% <0.00%> (-2.39%)
:arrow_down:

lnbits/commands.py
72.85% <0.00%> (-1.43%)
:arrow_down:

lnbits/core/models.py
66.37% <0.00%> (-0.29%)
:arrow_down:

lnbits/core/crud.py
56.83% <0.00%> (-0.10%)
:arrow_down:

... and 11 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


fix: LEAP-344: View all annotation tab is not loading

PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[x] Frontend

Describe the reason for change
The "View All Annotations" tab was experiencing loading issues when accessed with a configuration containing an image. The problem stemmed from the validation process, which was impeded by the lazy loading behavior of the image tag. As a result, the image wasn't added to the viewport until fully loaded, causing the tab to fail to load.
What does this fix?
To address this issue, the fix involves skipping the validation if the image is marked as ready. This modification acknowledges the lazy load behavior of the image, ensuring that the validation process doesn't impede the loading of the "View All Annotations" tab.
What libraries were added/updated?
None
Does this change affect performance?
no
Does this change affect security?
no
What alternative approaches were there?
Consideration was given to removing the lazy load from the ImageView to directly address the loading issues. However, it's crucial to note that this alternative approach comes with potential performance degradation.
What feature flags were used to cover this change?
no
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[x] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
Grid

/git merge master

/git merge master


Files: Format files size as Uplink mobile

Format files size as Uplink mobile

https://docs.rs/humansize/latest/humansize/


Test controller from Fixture passing to TESTS

Hello I am using one condition( if else loop) to use different URLs based on the env. I am mentioning this in fixture.beforeEach(..) and want to hit the URLs in TESTS based on the outcome
fixture('Verifying')
.beforeEach(async t =>{
console.log("Execution Started ");
const args = minimist(process.argv.slice(2));
var environment = args.env;
if(environment == 'Test')
{
t.ctx.Connectionstring=config.Test.Connectionstring;
t.ctx.Url=config.Test.Url;
}
else if(environment == 'Stage')
{
t.ctx.Connectionstring=config.Stage.Connectionstring;
t.ctx.Url=config.Stage.Url;
}
else if(environment == 'Prod')
{
t.ctx.Connectionstring=config.Prod.Connectionstring;
t.ctx.Url=config.Prod.Url;
}
test.page(t.ctx.Url)
("TC1 - Report Batches", async t => {
await t

.typeText(lgpage.username, 'abc)
.typeText(lgpage.password,'abc')
.click(lgpage.submit)

})
test.page(t.ctx.Url) doesnot work. Even if i give fixture.page(t.ctx.Url) doesnot work both the time it is giving the error:
"Cannot implicitly resolve the test run in the context of which the test controller action should be executed. Use test function's 't' argument instead."
I also tried navigateTo(t.ctx.Url) this doesnot work.

Hi @avi0711
I've already answered this question in the StackOverflow thread.
We try to keep the GitHub issues tracker for bugs and feature requests only (see also: Contributing. So, if you have any questions, please ask them using StackOverflow and don't duplicate them on GitHub.


Update lang version

Purpose
$Subject
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the changelog
[ ] Added tests
[ ] Updated the spec
[ ] Checked native-image compatibility

@kalaiyarasiganeshalingam why we need latest lang version

@kalaiyarasiganeshalingam why we need latest lang version

We have to release this change with a minor version update.
https://github.com/ballerina-platform/module-ballerina-xmldata/pull/489


Text resize

I’ve finally decided to change all fixed values to relative ones because those were not increasing/decreasing with the IE9 font size parameter.


"requested by TypeScript but content not available" and very slow diagnostic

Hi guys.
I have been using kak-lsp for quite a while and, I believe, since I started using I noticed that the diagnostic is very slow to run as I type.
I decided to start investigating this issue recently but I think I need some help from you.
I the *debug* buffer, I have a tone of log saying
 .../index.d.ts requested by jTypeScript but content not available

Everytime I type 1 character I get a tone of those.
I believe this is the reason for the very slow diagnostic (and the UI blinking all the time as I type).
I saw this issue appears on Neovim also https://github.com/sourcegraph/javascript-typescript-langserver/issues/432
But I am not sure what to do with this piece of information.
If it is linked to completion, what kind of expression can I set in the lsp_completion_trigger option so that it never triggers?
Thanks for your help

Hi, thank you for reporting! I failed to reproduce the problem, would you mind sharing a repo with a sample project?

Regarding disabling autocompletion, you can do it by setting lsp_completion_trigger to 'fail'

hey @ul thanks for a fast answer.
I tried disabling completion, but it doesn't seem to improve anything.
Sadly I have had the time to create a small project that reproduce the bug. you can try and clone this one and then edit whatever .ts file there is in there (src/script/util/Logger.ts might be a good candidate).
https://github.com/wireapp/wire-webapp

So, I switched from javascript-typescript-langserver to typescript-language-server.
The diagnostic is not really faster but I don't get the error stacking in the *debug* buffer anymore :)
I think I can close this issue now

One question though. Is there a way I can limit the rate at which commands are sent to the server.
It seems like there is absolutely no debounce, everytime I type a single character the whole file is sent to the language server.
I cannot seem to find an option that would limit this

Unfortunately, there is no such option at the moment. Thanks for the idea, I'll consider adding it (and PR is welcome as well!). Meanwhile, you can try wrapping kak-lsp into some script handling debounce.


Handle correctly unsubscription

Subscriptions to observables which do not have a pre-defined completion (such as services) have to be unsubscribed "by hand",  otherwise it stays subscribed for life, which creates a memory leak.

Fixed by #112


[Suggestion] Null value for sender

In my application we have a scenario where logged in users can create comments upon other users' contents and those content creators will be notified about it.
But in some cases, it is also possible for non-registered users to also comment using only a name and email fields.
I have tried something like this, using the extra method to send the user name and email.
$notifynder->category('comment')
           ->from(null)
           ->to(2)
           ->url('url_here')
           ->extra(compact('anonymous_username_and_email'))
           ->send();

but it will not allow me to do so. Could we have this ability added to the package?

@rulrok
Keep it simple and stupid :+1:
$notifynder->category('comment')
           ->from(0)
           ->to(2)
           ->url('url_here')
           ->extra(compact('anonymous_username_and_email'))
           ->send();

set 0 for from_id is the best way?
How to implement for internal system notifications?
ex: "New user registered" 2 minutes ago

@wemersonjanuario
Simply, no.
It is not the best way to do it, it's just a word-around before we make something like @rulrok did possible.
We've talked about with @fenos and the current idea is to assign a default User entity or Polymorphic Model Entity when from is a NULL value, we're still thinking about this feature :smile:

Nice to know it. I hope you can implement this feature soon.
Thanks!

@rulrok @wemersonjanuario What i was thinking is that Notifynder already allow for a System kind of notifications. Of course is not out of the box but i'll think about it, if this functionality has to be given out of the box.
Let's look it in this way:
You can create a table called system_bots
// system_bots table

id | name | img | all fields you like

Then create your model as usual:
class SystemBot extends Model {

   use Notifable;
}

When you want to send a notification from the system you can just do:
// there are missing fields on the notification for the purpose of example
$this->notifynder->from(SystemBot::class,1)->to(User::class,1)->send();

Close for inactivity

@fenos I think that creating extra table just to store one user is not a good solution. I need to create "system" notifications.

@fenos This feature would be great - I've solved it by the 0 value solution. But it would be great if it's possible to don't use the from() method. Or have a alternate method like system() that calls the from method with null or 0 or whatever.
Or having a default value for the System-User in the Config, like system_value where any kind of allowed value can be placed - so it is possible to use a default User or other Model if present in the app or use 0, null, false or anything like this.


Fix double prefix with TTL

Entries set with a TTL are not deleted, the delete call is made with a double prefix which does not delete the entry.
Corresponding issue: https://github.com/lukechilds/keyv/issues/108

Coverage remained the same at 100.0% when pulling f14462232aea56faf1475644becd6648ecb62038 on soundstep:fix-double-prefix into 57c03a4219065a280b470402eb97abb8c03a99d5 on lukechilds:master.

Hum, correct me if I'm wrong, the raw key need to be preserved in order definitely delete it at https://github.com/soundstep/keyv/blob/fix-double-prefix/src/index.js#L67, right?
BTW, @lukechilds can you take a look to this? 🙂

@Kikobeats Yeah both are needed, the param key remains to be sent the delete member function, which also do the prefix (that was the doubling), and the keyPrefixed is used to get the entry from the store (as before).

As an aside, re-assigning a function param is not great, ideally a keyPrefixed should be created every time the _getKeyPrefix is called. That would make things clearer, for example in the set function.
But that's "style" so I haven't done that in the PR.

Thank you so much for this @soundstep, great spot!

As an aside, re-assigning a function param is not great, ideally a keyPrefixed should be created every time the _getKeyPrefix is called. That would make things clearer, for example in the set function.
But that's "style" so I haven't done that in the PR.

Can you explain a little bit more what you mean by this?

Ahh I get it, implemented in https://github.com/lukechilds/keyv/commit/b87902889c0b8be123c3eee429ae55e6716d34ab

Published as<EMAIL_ADDRESS>🎉

No problem @lukechilds !


Projectfile

Description
This will make the init command and the new command interactive.
Checklist

I'll fix the conflicts and make the tests pass.

Thanks! I'll look this over later after I fix master. My initial impression though is I would prefer to enable interactivity with an -i/--interactive flag. Only being interactive makes the config file way less useful and as a user I would want non-interactive by default.

I think you make a good point. Except the package name and the description all the other questions are pretty static and it can be supplied by the config file.
I'll change the behavior to add a -i option for interactive.

The tests are now passing. But I haven't added new tests to cover the code that I have added. I'll work on adding tests tomorrow. In the meantime, I welcome any comments or suggestions.

I've added tests to the interactive init command. I have also fixed the failing tests. The PR is pretty big. I'm happy to do a screen share and walk you through the main points if you're interested.
I'll add more tests to the interactive new command as well, but I think this PR is ready for your review.

In init and new command files can you please change from hatch.settings import load_settings to from hatch.settings import copy_default_settings, load_settings and use that new function in the except FileNotFoundError:? That was an oversight of mine.
You can then use the default settings in the functions.

After those changes, we also need to make ourselves the tool.hatch as we discussed.
Here are some examples I've found:
https://github.com/pypa/pip/blob/54b983c2bd56dfbd61bbcc2dcc9177d4e55e25af/pyproject.toml
https://github.com/saezlab/pypath/blob/b285a3a80815a6f27100966c0ad5da01fe33fe61/pyproject.toml
https://github.com/pandas-dev/pandas/blob/e0fe5cc60b1dc0d777223bba64b8abfc0e0e02ab/pyproject.toml
https://github.com/twisted/twisted/blob/2dde252439267573fb802b5ab64af9517b3ea856/pyproject.toml

https://github.com/pypa/pip/pull/4144#issuecomment-332024594

Sorry, I missed a few.

Looks great! We just need to change the pyproject.toml: https://github.com/pypa/pip/pull/4144#issuecomment-332024594 until end of thread
So, initially I was thinking we could have metadata and commands, but exclude packages for when Hatch supports Pipfiles. Sound good?

Looks like one test is failing on one Python36-x64 Appveyor.
Looks to me it's a timeout failure. Can you take a look?

That's just virtualenv being slow, it happens sometimes :/

The PR looks great except the pyproject.toml needs:
[build-system]
requires = ["setuptools", "wheel"]

and put metadata right under [tool.hatch].

Tests failing due to source -> metadata change.

WooHoo!! 🎉


Update running-elyra-in-air-gapped-environment.md

Air-gapped needs to be defined.

What changes were proposed in this pull request?

How was this pull request tested?

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Air-gapped environment details --> https://en.wikipedia.org/wiki/Air_gap_(networking)

@richlysakowski was your intention to add information that would resolve https://github.com/elyra-ai/elyra/issues/2801?


ARB files support

Hi,
Do you have plans to add support to ARB files format (commonly used in Flutter)?

Not currently. But patches or thumbs-uping this issue welcome.
Spec: https://github.com/google/app-resource-bundle/wiki/ApplicationResourceBundleSpecification

I'd second that request, as the obvious workaround (using arb-convert, see https://github.com/trufi-association/arb-convert) seems not working well:
The following ARB fragment
"stpNumChildren": "Numero di bambini",
"@stpNumChildren": {
"description": "STP number of children",
"type": "text",
"placeholders": {}
},
will be converted to
#. STP number of children
msgctxt "stpNumChildren"
msgid "Numero di bambini"
msgstr ""
That is it puts the english text into msgctxt, the translated text in msgid and leaves msgstr blank, whereas the english text should be in msgid and the translated italian text in msgstr.

Support for Flutter's ARB files was added in version 3.3 (commit 75b6fefa7e544496df6724fc53ba1711504b5d95):
https://poedit.net/news/poedit-3.3-json/


LOBS Analytics

Context:
As mentioned in the chat by Ivan, the DAO will hold back from minting unclaimed LOBS (to save gas).
Because of this it is assumed that OpenSea and the likes will only pick up the traits & rarity of the current supply and not include the (large portion) of the DAO's NFT's.
Proposal:
This is a short proposal to generate analytics on the full set of the 6751 available LOBS.
I will generate for the full list of LOBS;

Holder Address
Image Link
Rarity (%) of all traits
Rarity Score
Relative Rarity Ranking

See the below screenshot for an example of another collection;

Will also group the rarity score's by address so we can rank each address/member :1st_place_medal:
How:
I'll parse the raw JSON metadata of all LOBS and provide the code & a spreadsheet of the above (that we can re-run adhoc).
Holders will be able to filter down to their address and see the data across multiple LOBS.
 This is my way of giving back something to get one.
So, 1 LOB from the treasury (my unclaimed one) for this.
It's likely this doesn't pass before Wednesday's reveal (let's be honest, I'll do the work anyway) but there is the issue of if this passes what LOB do I get?
I'd like to keep it completely random.
We can say that I get the direct next available LOBS id or even just do a random number generator of the DAO LOBS.. if a big brain has a better solution to randomise all the better.
Pros:

Community building / hype for reveal
Find out who has the biggest horseshoe
See if Ivan gmi or ngmi
Pray that some 3 post anon doesn't get the diamond claw together
I will likely add more analytics to this in the future
Keep track of flippers so we can bring out the guillotine
See how much good art the DAO is sitting on

Cons:

1 LOB cost - given from treasury to community member.

Action:

Post feedback if you want this to happen
If enough interest, then we can make a snapshot

I support doing this, 1 LOBs is a fair pay for this job.

Good idea, fully support it. Let's brainstorm what else can you @msirinotis do for community smirk Write a post, Sir after you do analytics.

Yeah, this is a good idea. I'll do a write up too.

idea is pretty good but is this a good way to donate 🦞,, need to brainstorm what comes next like somebody missed and will again ask for it using whatever 😬 buts its a good part community building but i would rather ask to eth giveaways

I won't comment on the value as I just want this to be about having fun and doing something for the community. The vote can decide if it's worthy or not.
Zhongfu kindly posted the ipfs locations & traits earlier so I've been working with those already.
Some pre-processing so far rather than just taking pure rarity.

Add combination trait -> if has both custom artwork & signature (only 2....  rare)
Preliminary weightings on different traits (we care way more about artwork/signature than mood)
Condense typo's for 'untextured' gene's.
Penalise Earth Circle 01, 06, 11 etc genes as they are common amongst each other

Some of highest ranking LOBS so far;
The 2 art+signature combinations.

A rare non-signed piece

Rare non-artwork piece

A... not so rare piece

Upcoming work;

Adjustments to ranking algorithm in line with lobsters values over pure rarity.
Map all holders to the traits/ranking data once ID's are revealed.

Love the idea - am in support.

I second what etthiscam posted, I dont think this is worth the actual market price of Lobs (3 eth atm). Seems too expensive and I have already seen rarity ranks posted on chats.
I also think the DAO should hold off for some time to distribute unclaimed lobsters, dont find it prudent to start using them so early tbh

Agree on this.

Ok, the snapshot period is over.
https://snapshot.org/#/10b57e6da0.eth/proposal/Qmaua3zenAJRHbDaLyxuv7giP1uAeXHu2Y6iibKr7mct8J
Firstly, thank you to all that voted and for such a high percentage of you voting to accept the proposal.
Given that the standardising governance issue is still under discussion - what are the next steps here?
Keep in mind this is meant to be as random as feasibly possible. I know there are constraints that now we know the VRF seed we know which token_id will be minted next but is there a way around this? DAO mints next 5-10 and we run a random number generator? Spit-balling here.

Looks good ser

This one was resolved, the man finally got his LOBS. Closing.


Unable to record more than 6 seconds

Hi there are my options
record.start({
sampleRate: 16000,
compress: false,
threshold: 0.5,
thresholdStart: null,
thresholdEnd: null,
silence: '1.0',
verbose: true,
recordProgram: 'sox'
})
I am unable to record more 6 seconds Thanks here is my output through verbose
Recording with sample rate 16000...
Recording 5548 bytes
End Recording: 56.070ms

hi,are you able to resolve this problem?


Fix Dart Sass & Ruby Sass compatibility

Fixes https://github.com/twbs/bootstrap/issues/29743, also works with ruby-sass: https://www.sassmeister.com/gist/ff625639f5c4e126d121c853d93f3025
Ping @XhmikosR @ysds

The minus isn't causing any issues if I'm not mistaken?

Yep, there is no problem with the subtract().
My point is for reduce the code complexity by using the same format to plus and minus.


Add trickle input and output

Logging is still rough as I'm ironing out issues around tear-down, but wanted to put this out
Should be testable with
 curl -vvv http://localhost:9000/live-video-to-video/ -H 'Content-Type: application/json' -d '{"subscribe_url":"http://trickle-server/input-stream","publish_url":"http://trickle-server/output-stream"}'

@victorges Just a heads up that the publish task hangs after the input stream completes. I am not sure exactly how the other parts work but you will need to call publish_queue.put(None) after receiving a None from subscribe_queue (or waiting for subscribe task completion, eg await subscribe_task that is created here. This can also be used as a signal to clean up the inference job.
For reference, here is how I have been driving the process (without inference)


Can Word JS API Invoke External Applications and Pass it Data?

Hey, I'm not familiar with this API so I wanted to know if it's possible to use it to invoke an external application and pass it data.
I use MS Word to take video notes. An example might look like this (The Two Generals' Problem is a hyperlink to a local video file):

The Two Generals' Problem
• @2:35 Two communicating computing systems can’t guarantee their outgoing message has been received and acknowledged. (@2:40-3:15) Their message can be intercepted/destroyed.
• @4:03 Application: Phone app sending user product order to a payment processing server
• @5:10 idempotency token (idempotency key); e.g., shopping cart ID that’s sent with the order. If the server receives an order with the same shopping cart ID, it is processed once and subsequent orders with the same key are ignored.

I have an idea to build a Word Add-in to create and parse a word document for video timestamps (e.g., @2:35 or @:10-15:00). I'd like for the add-in to associate timestamps in the bulleted list with the hyperlinked video and then launch a video player to play the video, beginning at the specified time. I'm planning to use Qt to build the video player.
Is it possible for the Add-in to launch my Qt app and pass it data? (e.g., video start time).
A crude way for the add-in to pass data to the Qt app would be to write out the start/end time to a file, which the Qt app then reads from. Perhaps there's a better way (e.g., Add-in can invoke an application and pass it command-line args?)

Hi @ElusiveTau,
Yes this should be possible. You can open a new browser window, or a dialog box. For more information see this stack overflow q&a.
I'm not familiar with the QuickTime player specifics, but I would think there is probably a way to load the player in the opened web page and pass the parameters with the URL when you open the page.
Hope this helps,
David


Single-header design

The library looks pretty cool but it's lacking in terms of its use of the single-header design.

No _IMPLMENTATION space
This messes up a lot of use cases. Without this, you cannot compile the library into an object file or a shared object file and then link it.

NO _HEADER space

While you do use ZEROGL_H, the library is not organized using structures first and the function defines last.
Organizing its structures/function declares first and then function designs allow the user to easily understand the library's API and how to use it.
If you prefer to keep it how it is, that's also fine, it depends on how you want the user to be able to use the library. I think the way I described is better for single-header libraries.
For reference, you can check out my single-header library, RGFW or stb_image.

This is done. Just #define ZEROGL_IMPLEMENTATION before importing it. Tks for the tip.

Question: should I have the private functions on the header part?

https://github.com/prcastro/zeroGL/pull/45 does exactly that. Tks again


Waifu2x not available argument for training

python3 train.py waifu2x
usage: train.py [-h] {imagenet} ...
train.py: error: argument {imagenet}: invalid choice: 'waifu2x' (choose from 'imagenet')
I am following the documentation for training and when I run the help command it tells me waifu2x is an invalid choice.

I have fixed that problem at be3faf08d2e1939474a84a55530ec4e70d5234a2 (2023-02-07).
Perhaps the source code is out of date. git pull it again.


linux-firmware: allow passing ref to update-script

This is useful for locally testing newer firmware and doesn't really complicate the script.
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

This would be a lot easier if it wouldn't be a FOD, ref #222187


Cannot view Prototyped APIs in Dev Portal

Description:
When click on View in Dev Portal button or when logged in to the Dev Portal and search for the Prototyped API and try to view it by clicking on it, it gives a blank black color page as below.

Steps to reproduce:

Create a Prototyped API.
Click on "View in Dev Portal" button from the Publisher for that particular API or logged into the Dev Portal and click in that API.
Gives a blank page as shown above.

Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

@npamudika you need to create a deployment for the prototyped API.
Deployments -> Create revision and deploy

Showing the blank page due to deployments not present will be addressed.

Yes, after creating a deployment, is able to view the API.
@malinthaprasan Shall we link to the black page showing issue and close this?

This will be handled via https://github.com/wso2/product-apim/issues/10843


fix(pom): Maven life-cycle "site" (mvn site) fails with message "Could not connect to Central search"

💡 TIP: Select the ↖⎾ Preview ⏋ Tab above help read these instructions.
1. Issue type

⌦ Type the letter "x" in the "checkbox" the best describe this issue.

[ ] Feature: I'm requesting a product enhancement.
[x] Defect:  I encountered an error or detected a flaw.

2. User story summary
As a developer, I want to be able to run "mvn site" and "mvn site:run" to generate the reports that this module promises to deliver, so that I can ascertain the quality of my own Maven project.
3. Acceptance criteria

[ ] 1. The Maven site life-cycle (mvn site) runs successfully with no errors
[ ] 2. The Maven goal "site:run" (mvn site:run) runs successfully with no errors.

Greg,
In my local I tried the fix suggested here, and it worked. I will create a PR. Thaks.

@joquijada , thanks again for the PR and update! I've merged your changes into master.
🎉


Update mongodb-metrics.rb to add SSL mode

The plugin failed to connect to a MongoDB server that only allows SSL connections.
I've patched it with SSL mode options (by default disabled).

@FlorinAndrei
Thanks for this!  Could you also make a PR against sensu-plugins-mongodb.

Okay, I did a PR on the metrics thing, which is exactly like the one I've patched initially, but... why are these scripts kept in two separate places, if they are identical? Just trying to understand.


Allow user to overwrite Collection field default value

Right now, if an option field is a Collection and is initialized with a default value in the code, and at the same time the user code is invoked with parameters for that collection field, these parameters are appended to the existing collection.
However, in my eyes it would make more sense to reinitialize the field with a new collection only containing the user-passed parameters. This is at least, what I expected.
Right now, there is no way of me saying "use only the parameters I passed in, and forget about the default one".

Yes, that makes sense. It will require some work to keep track of the original default values so we only replace the original collection with a new collection when the first user-specified option is matched, and for subsequent options add to (and don't replace) the new collection.

true, that was an oversight I had in the temporary "fix" I incorporated in my fork. Any idea, how to achieve this? Maybe introduce a boolean flag for each option, whether the current field value is user-specified?

If you want to create a pull request, perhaps add a Set<Field> field in Interpreter with fields that have been modified during parsing? Clear the set at the start of parsing, and check if it already contains the field before creating a new collection or array? Any time a field is modified while parsing the command line, add the field to the set.
Otherwise (without modifying the code), the workaround is similar to #213: keep the default value null, document the default value in the option description and initialize the default in the application if the user did not specify any values.
Note that this use case (a collection or array with multiple default values) is not covered nicely by JCommander's IDefaultProvider, so still thinking about what would be a nice way to do #213...

Implemented: parsed values are no longer appended to, but instead replace, the default value of multi-value (array, Collection or Map) options and positional parameters.


Support themes in the theia-trace-extension and it's graphs

Theia supports changing the theme, however, the theia-trace-extension doesn't apply it. The current implementation only supports the dark them. When switching to a light theme, the background of all the different graph types is still black.

@bhufmann Can we plan on supporting 1 light theme and the current dark theme? At least for the MVP?
I can take a stab at this one...

@bhufmann Can we plan on supporting 1 light theme and the current dark theme? At least for the MVP?
I can take a stab at this one...

Yes, I think too, we should support at least one light theme and one dark theme for the MVP.
It currently doesn't give the best impression for the user when using for example the Theia light theme.

From a UX perspective, we're hesitant to support all themes in the plugin:

Colour has an important functional role in complex visualisations and it would be impossible to guarantee that the visuals play nicely with all themes.
Colour consistency/determinism across users allows the brain to develop colour association reflexes which are much faster than consciously identifying elements.

At the moment, I'm imagining leaning towards just a light and a dark theme in the plugin. Having just two colour themes would sacrifice a sense of visual integration for those who have other themes and add visual noise, but I think we can get a sense of how many people find that annoying during user tests and mitigate it later.

@ebugden I agree, initially for the MVP it's ok to support dark and light themes. Please note that colors for the graphs (states, lines) are proposed by the trace server back-end. For the known graphs in the MVP these colors are fine for light and dark them.
Later on, we need to think about the theming in a more broader sense. What I mean we probably need to adjust the graph colors in the front-end depending on the theme (i.e. to avoid something like blue on blue background or bad color contrast). This is not in scope for the MVP and this initial theme support.


[icicle-cpu] Implemented PE file loader

Implemented a PE file loader and done a minor refactoring to the env.rs and elf.rs files. I'm not an expert rust developer so I'm open to suggestions to improve the code :)

thanks for the useful comments! I applied the suggestions to the code.


The Return of "does not work on derived classes?"

Hello,
I have the same problem as #94 with other function calls: An Edge is a subclass of a Feature and point_to_line_distance fails to recognize an edge as a Feature...
I think that there is a more general issue with type/subtype checking.
Also, the documentation does not always reflect the code: For exemple, distance is documented as distance(Feature, Feature). But distance(Point, Point) also works and is not documented.
I have the feeling that more generally, type should be enforced at function parameter level: point_to_line_distance(Feature, Feature) is too loose. The coder must check that the Feature geometry has the appropriate type. point_to_line_distance(Point, Line) is more precise and type checking is done by Python at runtime. There is no need to add more checks inside the code of the function.
BTW, point_to_line_distance should return the closest point somehow as well. May be a function nearest_point_on_line(Point, Line) returns the nearest point, and its distance to the line can be determined with distance.
Here is a piece of code that exhibit the issue:
`
from geojson import Point, LineString, Feature
from turfpy.measurement import point_to_line_distance
class Edge(Feature):
def init(self, node: str, start: Feature, end: Feature):
Feature.init(self, geometry=LineString((start["geometry"]["coordinates"], end["geometry"]["coordinates"])))
self.name = node
p1 = Point((25.25458, 51.623879))
f1 = Feature(geometry=p1)
p2 = Point((25.254626, 51.624053))
f2 = Feature(geometry=p2)
e1 = Edge("e1", start=f1, end=f2)
p0 = Point((25.0, 51.0))
f0 = Feature(geometry=p0)
fline = Feature(geometry=LineString((f1["geometry"]["coordinates"], f2["geometry"]["coordinates"])))
print(point_to_line_distance(f0, fline))
#71.59329853730718
fedge = Edge(node="e1", start=f1, end=f2)
print(isinstance(fedge, Feature))
#True
print(point_to_line_distance(f0, fedge))
Traceback (most recent call last):
File "/Users/pierre/Developer/Internet/js/gip/emitpy/tests/ft3.py", line 28, in
print(point_to_line_distance(f0, fedge))
File "/usr/local/miniconda3/lib/python3.9/site-packages/turfpy/measurement.py", line 1004, in point_to_line_distance
feature_of(line, "LineString", "line")
File "/usr/local/miniconda3/lib/python3.9/site-packages/turfpy/helper.py", line 120, in feature_of
raise Exception(
Exception: Invalid input to line, Feature with geometry required
`
P.

I also have the issue of classes with the following hierarchy:
A Line is a subclass of a LineString (a Line is a LineString with just 2 points, start and end). I create a Feature from the Line.
nearest_point_on_line(Feature, Feature) fails, telling the Line is not a regognized LineString, although isinstance(aLine, LineString) is True.

@devleaks  Thanks again, I think we need to check at the project level and see where ever possible to make types/subtypes specific. It will take some time to fix all the possible functions. Please feel free to open PR. We can do it iteratively each function level.

Issue actually resides in GeoJSON package. See here.


Setting max distance via Python

Required Info

Camera Model
{ D435 }

Firmware Version
05.10.06

Operating System & Version
Jetson TX2

Kernel Version (Linux Only)
4.4.38-tegra

Platform
NVIDIA Jetson

SDK Version
2.0

Language
python

Issue Description
I'm trying to access and set the max_distance on the camera via Python.  Here's the code, which I pulled from the IDE recommendation on the functional call.
pipeline = rs.pipeline()
config = rs.config()
profile = pipeline.start(config)
depth_sensor = profile.get_device().first_depth_sensor()
depth_sensor.set_option(rs.option.max_distance, 16)

I get this error, which is odd for a depth sensor:
RuntimeError: Device does not support option Max Distance!
I'm also trying to set things like visual_presets and color scheme, and I can't seem to find that API data anywhere. Visual Presets uses an enum range I'm guessing of 0 - 6 based on when I called a .get_range() on that option, but that's not documented anywhere.

Hello @lesreaper ,
The specific options are applicable with colorizer object.
To check whether specific option is available use the supports API

depth_sensor = profile.get_device().first_depth_sensor()
if (depth_sensor.supports(rs.option.max_distance)):
...

[Realsense Customer Engineering Team Comment]
Hi @lesreaper,
Try below code to see how it works.
colorMap = rs.colorizer()
print("colorizer - max_distance: %f" %(colorMap.get_option(rs.option.max_distance)))

[Realsense Customer Engineering Team Comment]
Hi @lesreaper,
Still need any support for this topic?


Javascript const selector update

Caused by:
https://github.com/atom/language-javascript/commit/2445663dfb9ef43ba1998
a45a8e0dd241c70c21d
https://github.com/atom/language-javascript/commit/c63453662370c97110103
95b7bd21d7a4c8957f7

Thanks for the pull request!
I've seen that const seems to be buggy at the moment. In some cases it is correctly applied, in others not. This is how the selectors look like for me on Atom 1.13.0; I don't see the storage.type.const.js selector. Which version are you using?

No problem 👍
I am also using Atom 1.13.0. This pull-request is a bit preemptive. The next version of Atom that includes<EMAIL_ADDRESS>will need this fix. You can test now by installing<EMAIL_ADDRESS>like so:
apm install<EMAIL_ADDRESS>
After you'll see the DOM like so:

Ah, that make sense!
Great, I will merge your pull request as soon as the next Atom version is out ✌🏼

@morkro, this change is now released! https://github.com/atom/atom/releases/tag/v1.14.0

🙌🏻 Nice, thanks for the update!


🚀 Feed: initializing app and adding data models

This PR is built upon issue #26
Setting up the feed app and adding data models.

Created the feed app
Created a Post data model
Created a Comment data model
Register the data models to the admin panel
Add tests
Populate the DB

Close #26

./feed/models/init.py:1:1: F401 '.post.Post' imported but unused
./feed/models/init.py:1:1: F401 '.post.PostManager' imported but unused
./feed/models/init.py:2:1: F401 '.comment.Comment' imported but unused
Error: Process completed with exit code 1.
Please fix flake8 errors, no review on failed actions

Tests look solid, I also focused on the creation and deletion of data.
Notice that you can also use the @pytest.mark.django_db() mark above a class and it will be relevant for the methods inside it.

Tests look solid, I also focused on the creation and deletion of data. Notice that you can also use the @pytest.mark.django_db() mark above a class and it will be relevant for the methods inside it.

Cool, Thanks for reviewing :)

I've populated the DB and added tests to the data models.
I needed to populated the DB with preferences which made a job_board test to fail so I changed the deletion tests to delete the specific fixture models and not delete by filters.
Im not sure if the changes I made to the job board are legitimate, so please review it @DeanBiton

@paOmer
When I ran vagrant up, I got at the bottom of the output:
    default:   Your models in app(s): 'feed' have changes that are not yet reflected in a migration, and so won't be applied.
    default:   Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.
    default: + setsid pipenv run python manage.py runserver <IP_ADDRESS>:8000

So I ran:
pipenv run manage.py makemigrations feed

and a new file called feed/migrations/0012_alter_post_prefernces.py was created, according to the output:
Migrations for 'feed':
  feed/migrations/0012_alter_post_prefernces.py
    - Alter field prefernces on post

and the content of the feed/migrations/0012_alter_post_prefernces.py file is:
# Generated by Django 4.0.3 on 2022-03-30 06:32

from django.db import migrations, models
import django.db.models.deletion

class Migration(migrations.Migration):

    dependencies = [
        ('job_board', '0004_test_data_years_of_experiences'),
        ('feed', '0011_comments'),
    ]

    operations = [
        migrations.AlterField(
            model_name='post',
            name='prefernces',
            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, to='job_board.preference'),
        ),
    ]

Is this alright? Do you know this?

@paOmer When I ran vagrant up, I got at the bottom of the output:
    default:   Your models in app(s): 'feed' have changes that are not yet reflected in a migration, and so won't be applied.
    default:   Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.
    default: + setsid pipenv run python manage.py runserver <IP_ADDRESS>:8000

So I ran:
pipenv run manage.py makemigrations feed

and a new file called feed/migrations/0012_alter_post_prefernces.py was created, according to the output:
Migrations for 'feed':
  feed/migrations/0012_alter_post_prefernces.py
    - Alter field prefernces on post

and the content of the feed/migrations/0012_alter_post_prefernces.py file is:
# Generated by Django 4.0.3 on 2022-03-30 06:32

from django.db import migrations, models
import django.db.models.deletion

class Migration(migrations.Migration):

    dependencies = [
        ('job_board', '0004_test_data_years_of_experiences'),
        ('feed', '0011_comments'),
    ]

    operations = [
        migrations.AlterField(
            model_name='post',
            name='prefernces',
            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, to='job_board.preference'),
        ),
    ]

Is this alright? Do you know this?

@taljacob2 thanks for letting me know.
That weird, Iv'e destroyed my VM and deleted the db file, the ran vagrant up but didn't got the output that you've got.
Anyway I ran pipenv run python manage.py makemigrations and also got the same migration file as you did.
I also ran pipenv run python manage.py migrate and everything seems to work find before and after this migration,
I mean that I can see that the post model has the preference model linked to it via the admin panel before and after the migration.
I guess everything seems fine, and I'm wondering if this needs to be commited to repo, what are your thoughts about it?

@paOmer When I ran vagrant up, I got at the bottom of the output:
    default:   Your models in app(s): 'feed' have changes that are not yet reflected in a migration, and so won't be applied.
    default:   Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.
    default: + setsid pipenv run python manage.py runserver <IP_ADDRESS>:8000

So I ran:
pipenv run manage.py makemigrations feed

and a new file called feed/migrations/0012_alter_post_prefernces.py was created, according to the output:
Migrations for 'feed':
  feed/migrations/0012_alter_post_prefernces.py
    - Alter field prefernces on post

and the content of the feed/migrations/0012_alter_post_prefernces.py file is:
# Generated by Django 4.0.3 on 2022-03-30 06:32

from django.db import migrations, models
import django.db.models.deletion

class Migration(migrations.Migration):

    dependencies = [
        ('job_board', '0004_test_data_years_of_experiences'),
        ('feed', '0011_comments'),
    ]

    operations = [
        migrations.AlterField(
            model_name='post',
            name='prefernces',
            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, to='job_board.preference'),
        ),
    ]

Is this alright? Do you know this?

@taljacob2 thanks for letting me know.
That weird, Iv'e destroyed my VM and deleted the db file, the ran vagrant up but didn't got the output that you've got.
Anyway I ran pipenv run python manage.py makemigrations and also got the same migration file as you did.
I also ran pipenv run python manage.py migrate and everything seems to work find before and after this migration,
I mean that I can see that the post model has the preference model linked to it via the admin panel before and after the migration.
I guess everything seems fine, and I'm wondering if this needs to be commited to repo, what are your thoughts about it?

I guess eventually we will add this file anyway because Django wants us to. Although I am not sure this file does anything essential. It is possible to decide about this in the future, and leave it like this for now.


737 -  Add the metadata download on portal level

Issue Number
Closing #737
Purpose/Implementation Notes
I've added the portal metadata download support to the portal.
Changes include:

Updated the useMetadataOnly hook to support the portal metadata
Added a new component PortalMetadataDownload  and rendered in the Header navigation
Added the portal metadata support to DownloadStarted

Types of changes

Refactor (addresses code organization and design mentioned in corresponding issue)
New feature (non-breaking change which adds functionality)

Functional tests
N/A
Checklist

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots
N/A

I've applied the feedback and this PR is ready for your review. Thank you David!


API issues to discuss or change

This list is meant to keep track of things we come across in the API that don't look right and may need fixing (after completing the refactoring):

function names in functions.py need to be longer and clearer (intwave, centrfrq, scal2frq, orthfilt).
qmf already exists in scipy.signal
wavelet parameter to functions should be uniform everywhere (can be Wavelet instance, string and in some cases intwave even a third option).
functions in thresholding.py are (too?) simple and in two cases equal numpy names.
MODES is all capitals. It's properties aren't very Pythonic.
Wavelet has __str__ but not __repr__.

Closed by #55.


Cleanup Repos

Add an getViewModelListBehaviorSubject that simplifies
how to get most model lists in the view
unified update, delete and create methods and removed redundant code from the repos
(where it was possible)
cleaned up the motion detail to not directly use the DataStore
Add information about the "main model" to all ViewModels, to call the
correct constructor in the BaseRepo
Briefly tested all the creation and delteion

Could not easily reproduce. Needed to open the same motion in 10 tabs simultaneously to get the error once.

@jsaalfeld that goes for current master aswell


Core and CDK issues on Joyent

I've been trying to deploy Core and CDK on a Joyent cluster.  The install hooks fail because apt isn't up to date:
root@dbe49d90-ba0d-ef45-dacd-e39bd2dba615:/var/lib/juju/agents/unit-etcd-0/charm# ./hooks/install
Reading package lists... Done
Building dependency tree
Reading state information... Done
python3-yaml is already the newest version (3.11-3build1).
The following additional packages will be installed:
  binutils build-essential cpp cpp-5 dpkg-dev fakeroot g++ g++-5 gcc gcc-5 libalgorithm-diff-perl libalgorithm-diff-xs-perl libalgorithm-merge-perl libasan2 libatomic1 libc-dev-bin
  libc6-dev libcc1-0 libcilkrts5 libdpkg-perl libexpat1-dev libfakeroot libfile-fcntllock-perl libgcc-5-dev libgomp1 libisl15 libitm1 liblsan0 libmpc3 libmpx0 libpython3-dev
  libpython3.5-dev libquadmath0 libstdc++-5-dev libtsan0 libubsan0 linux-libc-dev make manpages-dev python-pip-whl python3-wheel python3.5-dev
Suggested packages:
  binutils-doc cpp-doc gcc-5-locales debian-keyring g++-multilib g++-5-multilib gcc-5-doc libstdc++6-5-dbg gcc-multilib autoconf automake libtool flex bison gdb gcc-doc
  gcc-5-multilib libgcc1-dbg libgomp1-dbg libitm1-dbg libatomic1-dbg libasan2-dbg liblsan0-dbg libtsan0-dbg libubsan0-dbg libcilkrts5-dbg libmpx0-dbg libquadmath0-dbg glibc-doc
  libstdc++-5-doc make-doc python-setuptools-doc
The following NEW packages will be installed:
  binutils build-essential cpp cpp-5 dpkg-dev fakeroot g++ g++-5 gcc gcc-5 libalgorithm-diff-perl libalgorithm-diff-xs-perl libalgorithm-merge-perl libasan2 libatomic1 libc-dev-bin
  libc6-dev libcc1-0 libcilkrts5 libdpkg-perl libexpat1-dev libfakeroot libfile-fcntllock-perl libgcc-5-dev libgomp1 libisl15 libitm1 liblsan0 libmpc3 libmpx0 libpython3-dev
  libpython3.5-dev libquadmath0 libstdc++-5-dev libtsan0 libubsan0 linux-libc-dev make manpages-dev python-pip-whl python3-dev python3-pip python3-setuptools python3-wheel
  python3.5-dev
0 upgraded, 45 newly installed, 0 to remove and 2 not upgraded.
Need to get 837 kB/77.5 MB of archives.
After this operation, 201 MB of additional disk space will be used.
Err:1 http://security.ubuntu.com/ubuntu xenial-security/main amd64 linux-libc-dev amd64 4.4.0-71.92
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:1 http://security.ubuntu.com/ubuntu xenial-security/main amd64 linux-libc-dev amd64 4.4.0-71.92
  404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/main/l/linux/linux-libc-dev_4.4.0-71.92_amd64.deb  404  Not Found [IP: <IP_ADDRESS> 80]

E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
Traceback (most recent call last):
  File "./hooks/install", line 8, in <module>
    basic.bootstrap_charm_deps()
  File "lib/charms/layer/basic.py", line 54, in bootstrap_charm_deps
    'python3-dev',
  File "lib/charms/layer/basic.py", line 160, in apt_install
    check_call(cmd + packages, env=env)
  File "/usr/lib/python3.5/subprocess.py", line 581, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['apt-get', '--option=Dpkg::Options::=--force-confold', '--assume-yes', 'install', 'python3-pip', 'python3-setuptools', 'python3-yaml', 'python3-dev']' returned non-zero exit status 100

I can solve this by running juju ssh [unit] sudo apt update -y across all the units.  I assume the charms that make the bundle aren't running apt update because the "big data" bundles seem to work.
I've tried this via juju deploy and conjure-up.

Hi @joedborg thanks for the report. I thought the apt-layer should be handling a refresh as the first thing it does. If it is not doing that we should look into this problem. If you have a suggested fix please let us know, otherwise we will dig into then when we get a chance to breathe

Hey @mbruzek, no worries at all!  Yes, I'd assumed the same.  I'm not really sure if it's a bundle or Joyent issue (seems to be a bit of both).  No rush whatsoever from my side :), I'll see if I can come up with a fix that doesn't affect the charms too much.

AFAICT the etcd charm wasn't using the apt layer, which explains the original problem. In any case, the etcd charm now uses snaps, so this bug is no longer valid.


Chaincode related questions

The chain code I defined needs to include some external library files, which can only be downloaded in Ubuntu.
How can I deploy the smart contract?

The method I want to implement in smart contracts only works in Ubuntu 20.04. How can I combine it with fabric?

Please reserve github issues for actual code issues.
For questions please use mailing list or discord chat - https://hyperledger-fabric.readthedocs.io/en/latest/CONTRIBUTING.html#communications-and-getting-help

For starters take a look at using external chaincode - https://hyperledger-fabric.readthedocs.io/en/latest/cc_service.html


v0.17.1 Can't resolve '../css/doc-explorer.css' in graphiql/dist/components

Topic

It looks to me like the entire ./css/... directory is missing, as I'm getting this when I try to build for production:
Creating an optimized production build...
Failed to compile.

./node_modules/graphiql/dist/index.js
Cannot find module: './css/show-hint.css'. Make sure this package is installed.

You can install this package by running: npm install ./css/show-hint.css.

oops! will fix this asap

@MichaelBurgess @Fi1osof thanks for the report! can you let me know if that PR helps?
try these steps locally:

yarn
yarn build
lerna run build-bundles
lerna run build-demos (about to introduce a developer docs PR sorry)


fix(readme): no changes, updating docs for GOOGLE_APPLICATION_CREDENTIALS authentication

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


Stowatch JS Extension

Is your feature request related to a problem? Please describe.
Users often find it difficult to get a stopwatch, with a clean and attractive UI, which might help them to concentrate on their work, thus allowing them to record the times taken for doing various tasks.
Describe the solution you'd like
The Stopwatch Extension is aimed to provide its users a stowatch, which has a clean and attractive UI, thus helping them to concentrate on their tasks, and containing features like Start, Pause, Reset and can register unlimited Laps.
Additional context
The Stopwatch Extension should seamlessly integrate into any browser, and should have a clean but attractive UI, thus providing a great User Experience.
What problem is this feature trying to solve?
This feature is aimed to solve the users problem of not finding a stopwatch will a clean UI, which might help the users to concentrate on their tasks.
How do we know when the feature is complete?
The feature will be completed when users will be able to start, stop and reset the stopwatch, as well as, register unlimited laps on the extension.

Levels are defined as per the difficulties and code checking. I will check the PR and change the level then.

Sure mam, thank you!


[WIP] Generic webhook (#602)

An attempt to implement #602, currently WIP
Includes

Generic Webhook API Server
Will start depending on generic-gateway command line flag
Helm chart implementation (I'd appreciate a double/triple check here, first time editing a Chart)
Tests

Does not include

Documentation

Points

feeling a bit weird about this. Explain: all source code files in the 'webhook' folder use Gin, whereas I used go-restful, since all the other methods on API server use that. Thoughts?
added a new field on Project struct with the GenericWebhook secret which can be auto-generated
not sure about this configuration
Generic Webhook handler will refuse to retrieve Projects that have empty secret reference

Looking forward to your comments and reviews, is this what you had in mind?

thanks for the review, agree with all proposed changes 👍

This looks great! I'll test the new gateway early next week, but at first glance it looks really great.
Thanks!

Created PR on brigade-charts here
In any case, linter bit me again :( I suppose we can merge #728, then merge master and it will be fixed. Thoughts? cc @radu-matei @vdice

One thing I don't see here (and I haven't actually gotten to testing just yet) is documentation - do we want to have it when merging? (It's a clear yes from me).

definitely, was planning to work on that as soon as we all conclude that this PR contains all the features we want the Generic Webhook to have.

This looks great!
Could you please rebase and squash the commits with the small fixes? This helps keeping the commit history relevant, and reverting much easier.
Thanks!

squashed a bunch of commits, let me know if it's better now, thanks!


Invalid decoding for original UUencode charset

Hi,
I don't know if I'm using the incorrect options but I don't get UUencode working for decoding the attached string ( see attached file).
I'm using:
let buffer = UUencode().decode(str)
The string contains space and I get an error message for invalid space character. The attache sample represents an encoded binary as jpg picture.
pic_uue.txt

The space is only part of the original charset for UUencode.
You can set this during initialization (or on the fly):
const uuencode = new UUencode("original");
uuencode.decode("str");

I have not tested your input though. If you still have issues, just let me know.

Thank you, unfortunately it is not working. Difficult to understand the problem, tested a lot.
I put a little code together for decoding to uint8array and downloading as file :
https://jsfiddle.net/hleogr/2yuk5s34/32/
I cannot say whether the input is wrong or there is a issue with the decoder.

Hi. The code looks way too complicated. As far as I can see, you feed the decoder line by line, which is not the right way to do it. Where is the UUencoded string coming from? It also contains octal escape sequences. Do you have a pure sample input?

Hi, indeed, the sample string is from a larger source  that contains multiple uue encoded binaries. For simplification purpose, I only included one uue encoded string in the fiddle example. The matchAll regexp finds all uuencoded files and each  is feeded as ${bincontent} to the decoder (not line by line).

OK. I found the problem.
Actually it is immanent to the original uuencode charset. If you take a look at your fiddle example, line 6. There is a space missing at the very end of the line, which can easily get removed by editors or whatever. It is also missing in the original file. This causes the decoder to produce wrong results. I will work on a fix for the original version to add missing spaces if they not present (the very first character indicates the correct number of characters).

Give it a try now.

Hi,
First of all, thank you! I tested and for all graphics it appears to work
now. However, it is not working in other case of uuencoded binaries. I
double checked everything (including used regexp).
In some cases it is just return 3 bytes and those bytes always representing
this uint8array [ 80, 75, 3 ]. For example the first lines when just
returning those 3 bytes are:
M4$L#!!0<EMAIL_ADDRESS> +$    0    9&]C4')O<',O87!P+GAM
M;$V./0L",1!$_\IQO;=!P4)B0-!2L+(/>QLOD&1#LD)^OCG!CVX>;QA&WPIG
MNI#BV&5(C(I(/ !47BK9.7:=N')=HI6-Y #OGD<Z,STA)8O4'J@)I9GF
@.**>K7A.YNJQ<&4GPZ4A!0WJ=0U[R;UEA_6#MI7E!+ P04
M    " #CAFI6/HK7?.\    K @  $0   &1O8U!R;W!S+V-O&ULS9+/
M3L,P#(=?!>7>.FT9AZC+96BG(2$Q"<0M2KPMHOFCQ*C=V].6K1."!^ 8^Y?/
[...]
In other cases, the CPU is running hot and decoding is not stopping or it
takes a very long time but the decoding result is not correct.
Not sure how I can help  to find the reason.
Am So., 21. Mai 2023 um 12:33 Uhr schrieb UmamiAppearance <
@.***>:

Give it a try now.
—
Reply to this email directly, view it on GitHub
https://github.com/UmamiAppearance/BaseExJS/issues/52#issuecomment-1556143781,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABI2CFKRFQIIEQBEL6647RLXHHVQVANCNFSM6AAAAAAYIAFCKI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi. How big are those files? Do you have a real example?

HI, I attach a sample uue encoded jpg file where the CPU (16 threads) is
running hot and decoding is not stopping. The jpg files itself is only
about 300kBytes.
Am Mo., 22. Mai 2023 um 16:03 Uhr schrieb UmamiAppearance <
@.***>:

Hi. How big are those files? Do you have a real example?
—
Reply to this email directly, view it on GitHub
https://github.com/UmamiAppearance/BaseExJS/issues/52#issuecomment-1557278611,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABI2CFNJRJPHLBA377HGD6DXHNWZHANCNFSM6AAAAAAYIAFCKI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Please continue this in this new created issue: #54

Updated #54

There still was an issue, when spaces where missing in last encoded line. Fixed in #56


feat: Add separators on context menu

Fixes #2506 - Check the preview.
This is my first try toward implementing separators in context menu.
One thing which I can improve in this implementation is to use constant varible instead of hard coded numerical value for order and group.
If there is a more clever way to implement this, please suggest me.
Screenshots

Visually it looks great ❤  not sure what to do with the ids though.. needs some thinking :)

If we don't want to have this weird numbering scheme, another option would be to simply list the actions manually, in the order we want, including separators.
IMO it's not so bad — it'd simplify the code (remove the need for actionManager.getContextMenuItems, item sorting, CANVAS_ONLY_ACTIONS constant, remove group, contextMenuOrder, contextMenuGroupOrder...), and make it more explicit and easier to navigate.
We should also move all the adhoc contextmenu items into proper actions (paste..).
It would look something like this:
import { actionPaste } from "src/actions/actionClipboard"; // this action doesn't exist yet
import { actionSelectAll } from "src/actions/actionSelectAll";

const separator = { type: "separator" };

ContextMenu.push({
	options: [
		actionPaste,
		separator,
		actionSelectAll,
		// ...
	]
});

Thank you @dwelle, your solution seems very nice to me. Removing numbering scheme will definitely improve code quality.

How to convert following methods into actions which are using this.setState() and this.canvas?
  private pasteFromClipboard = withBatchedUpdates(
    async (event: ClipboardEvent | null) => {
      // #686
      const target = document.activeElement;
      const elementUnderCursor = document.elementFromPoint(cursorX, cursorY);
      if (
        // if no ClipboardEvent supplied, assume we're pasting via contextMenu
        // thus these checks don't make sense
        event &&
        (!(elementUnderCursor instanceof HTMLCanvasElement) ||
          isWritableElement(target))
      ) {
        return;
      }
      const data = await parseClipboard(event);
      if (data.errorMessage) {
        this.setState({ errorMessage: data.errorMessage });
      } else if (data.spreadsheet) {
        this.addElementsFromPasteOrLibrary(
          renderSpreadsheet(data.spreadsheet, cursorX, cursorY),
        );
      } else if (data.elements) {
        this.addElementsFromPasteOrLibrary(data.elements);
      } else if (data.text) {
        this.addTextFromPaste(data.text);
      }
      this.selectShapeTool("selection");
      event?.preventDefault();
    },
  );

  private copyToClipboardAsPng = async () => {
    const elements = this.scene.getElements();

    const selectedElements = getSelectedElements(elements, this.state);
    try {
      await exportCanvas(
        "clipboard",
        selectedElements.length ? selectedElements : elements,
        this.state,
        this.canvas!,
        this.state,
      );
    } catch (error) {
      console.error(error);
      this.setState({ errorMessage: error.message });
    }
  };

How to convert following methods into actions which are using this.setState() and this.canvas?

Actions can (should) return { appState } which is what will be set as new state. We can technically pass canvas to the action, as well.
That said, maybe it's not a good fit for pasteFromClipboard. That helper needs access to ClipboardEvent, too. Everything is solvable, but maybe we should leave it for later, and instead inline that one/two "actions" into the contextMenu as we've been doing so far?

What's the latest with this one @Kartik1397? Are you stuck anywhere? need help?

What's the latest with this one @Kartik1397? Are you stuck anywhere? need help?

@lipis My exam is going on which will end tomorrow. So I will start working on this again from tomorrow.

@lipis My exam is going on which will end tomorrow. So I will start working on this again from tomorrow.

Priority on the exams! Good luck..

Priority on the exams! Good luck..

@Kartik1397 are you done with this one?

@Kartik1397 are you done with this one?

@lipis Yes, If you fine with not creating action for paste and toggleZenMode for now.

@lipis Yes, If you fine with not creating action for paste and toggleZenMode for now.

Let's see what @dwelle thinks..

Let's see what @dwelle thinks..

Let's hide the shortcuts at the same breakpoint as with the menu changes.

Let's hide the shortcuts at the same breakpoint as with the menu changes.

Let's hide the shortcuts at the same breakpoint as with the menu changes.

Let's do it in a separate PR as it will require a more involved fix.

Let's hide the shortcuts at the same breakpoint as with the menu changes.

Let's do it in a separate PR as it will require a more involved fix.

@Kartik1397 reviewed. I made a few changes:

Change how the canvas is passed. Instead of reusing the formData param which should be used only for the data retrieved from action's accompanying HTML, I added a 4th argument app: App from which we can get canvas directly. This makes it type-safe (which also surfaced a one failure code path — one which in practice wouldn't come up ATM but best be safe).
Made toggleZenMode into an action to align with toggleGridMode, and made both helpers in App use those actions.
Plus some naming/style tweaks.
Reverted the <kbd> → <div> — I think it was a mistake?

If you're ok with the changes, we can merge :rocket:

@Kartik1397 reviewed. I made a few changes:

Change how the canvas is passed. Instead of reusing the formData param which should be used only for the data retrieved from action's accompanying HTML, I added a 4th argument app: App from which we can get canvas directly. This makes it type-safe (which also surfaced a one failure code path — one which in practice wouldn't come up ATM but best be safe).
Made toggleZenMode into an action to align with toggleGridMode, and made both helpers in App use those actions.
Plus some naming/style tweaks.
Reverted the <kbd> → <div> — I think it was a mistake?

If you're ok with the changes, we can merge :rocket:

Let's 🐑 it :)

Let's 🐑 it :)

There was a bug, when we open context menu while cursor type  is move.
Before

After

If you're ok with the changes, we can merge

Yeah :thumbsup:

Also, what's your twitter handle?

https://twitter.com/ktk_pjpt

There was a bug, when we open context menu while cursor type  is move.
Before

After

If you're ok with the changes, we can merge

Yeah :thumbsup:

Also, what's your twitter handle?

https://twitter.com/ktk_pjpt

🎆

🎆


BugFix: Timestamp is not updating.

The timestamp is stuck at creation time.
Both prints the same time in the logs.
logger.info('Hello %s!', 'world');

setTimeout(function() {
		logger.info('Hello %s!', 'world');
	},
	2000);

Oops.

Thanks!

Released as v1.0.1


sharing problem fact on action page

Prompt user to share the problem statement on the action page (this prompt should not appear on the pitch page).
Desktop:

Mobile:

Let's create a field "Problem Fact Share Prompt" on https://beta.dosomething.org/admin/config/dosomething/dosomething_campaign for the copy of the prompt (i.e. "F that, spread the word") so that we can update it if need be without any code deploys.

@sbsmith86 We should also push the FAQs to the right column if there are shares present, to prevent them from stacking up on the left. Furnes was saying in Slack that this is a template thing, so I'm not sure if that is something changed front end or back.

@mikefantini Edited description to read "variable", not a "field" -- to help clarify it's not a Drupal field.

@lkpttn so when shares are present, the FAQ's should display like they do when there is a video?
Also, what are the rules around when the shares are present/when they are not? If there is a problem fact statement, then shares should be present? Would there ever be a case where there is a problem fact statement and shares should not be there?

Yeah yeah.
I think the intention is for there to always be share buttons, so the FAQ link will always move over. I don't forsee us rolling this out onto only certain campaigns, it would be site-wide. This could be implemented in the template logic or we could change the rule to always have FAQs on the left side.