layan-gtk-theme: init at 2021-06-30

Motivation for this change
Layan is a flat Material Design theme for GTK 3, GTK 2 and Gnome-Shell which supports GTK 3 and GTK 2 based desktop environments like Gnome, Budgie, etc.
-- quote from vinceliuice/Layan-gtk-theme
Things done

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

[x] NixOS
[ ] 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 nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 130064 at 86cfd3c0 run on x86_64-linux 1

1 package built successfully:

layan-gtk-theme

2 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/data/themes/layan-gtk-theme/default.nix:17:3:
   |
17 |   installPhase = ''
   |   ^

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/data/themes/layan-gtk-theme/default.nix:26:5:
   |
26 |     license = licenses.gpl3;
   |     ^

@musfay Changes performed, thanks for reviewing!

The name env variable will affect the install.sh script, so I unset it before install.
see https://github.com/vinceliuice/Layan-gtk-theme/blob/2021-06-30/install.sh#L189

Also, gpl3 changes to gpl3Only.


bzlmod: Do not register toolchain

The auto-configured toolchain is already provided by the bazel_tools built-in module. Until it is moved to rules_cc, there is no need to register it again in this module.

Stacked on #165


Bump version: (to re-release) Add missing conda dependencies

To fix the CI: https://github.com/populationgenomics/analysis-runner/runs/3024547558
Do you think there are other new dependencies in the Cromwell update that we need to specify?

😓  mean't to squash merge!


About Views

Can I make schema.json using a VIEW as "table", or there some schema generator to make this JSON analyzing tables used on my VIEW query?

Views are currently not supported yet. Some work has been done but its still WIP.

Congrats for your job, i'm using to replace materialized views. I will try simulate views on schema, tnks.

@toluaina  hi, are you still working with this? I have a table with tree-like struct and I want get a path for every item for searching. Since there is no such method for recursive retrieving I create a recursive view to achieve that.
But , I found view cannot be synced when change the data.
Can you tell me how to make view work? I can implement it if I can.

@toluaina yes, i have try it a little, as far as I upderstand because of there is no such thing like permarary key, so they cannot make the trigger run.
In My case, I create a trigger to make that view as a real table.
Thank you for your reply.

Sorry for the long wait. Support for views is detailed here

view are finally supported.


Should we update JSON2 to JSON3?

JSON3: https://github.com/bestiejs/json3/
Benefits:

No use of eval or regexps
Fixes some edge-case issues in IE8 as well

Cons:

Do we want to bother updating JS for IE7 at all?
Slightly larger file size after minification
We’re only including it in ie.js so the other browser edge case fixes wouldn’t get applied

No, we don’t. Removed it from the GOV.UK Frontend Alpha and turned off JS for IE7 and lower.


Nested modules outright broken

module Foo =
  module Bar =
    let x = 1

module Foo.Baz =
  let y = 2

One would expect this to be equivalent to:
module Foo =
  module Bar =
    let x = 1
  module Baz =
    let y = 2

One would be wrong:
> Foo.Baz.y
2
> Foo.( Baz.y )
  Variable not in scope: `Baz.y`

  Arising from use of the expression Baz.y
  │
1 │ Foo.( Baz.y )
  │       ^^^^^
> Foo.(


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack 93ad912...de04b70:
  > de04b70 Do not print debug info when running release.sh (# 70)
bumping knative.dev/pkg 3e1e4d7...a2869ae:
  > a2869ae upgrade to latest dependencies (# 2139)
  > e6552a0 Update community files (# 2138)
  > 79f8b3f Auto-generate webhook certs with ECDSA for FIPS compliance (# 2136)
bumping knative.dev/eventing a4650e0...e27695b:
  > e27695b Update community files (# 5462)
  > 49420f4 Added broker.WithConfig (# 5461)
  > 66dbf4a Added tristate flags and changed experimental config map name (# 5459)

/lgtm
/approve


Weird characters instead of icons

Describe the issue
I can't add much more verbal description beyond the screenshot.
This was just visiting https://datatracker.ietf.org/meeting/120/agenda during IETF 120 MoPS session , using Chrome on Linux.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Same as #7225 - it was apparently a transient situation in the past

Lucas - if this persists, swing by the tools desk during a break so we can look at it in real-time

I seem to get this with Chrome on macOS 10.15 (not on macOS 14).  What are you using?

I forgot to add my OS/browser when I opened but edited shortly after. Linux +chrome

Noticed that this is also probably the same as #7223

This is happening because the icon set is failing to serve in certain cases and browsers use whatever default icon set they have handly.
We have an ongoing conversation with Lucas about what might be happening with our use of Cloudflare that could be leading to those occasional failures.

My current theory, based on not much evidence, is that there is a particular race condition related to HTTP/2 and HTTP/3 connection coalescing. Fonts in particular have constraints on them that require a new connection to be used for the fetch. Javascript is also affected when there are rel=preload or rel=modulepreloaf, or cross orgin attributrs (channeling my friends from the web perf world here). This might be exacerbating some connectivity churn / browser happy eyeballs / racing / looking at cached Alt-Svc state that all culminates in a resource load fail without retry. I stress might here. This sort of error, in my experience, tends to be transient and clear up on a reload, requiring a perfect storm to repeat. Enabling Chrome dev tools also, by default, will cause fetching of css and Js source maps, which also change the way connections are used, sometimes leading to Heisenbugs.
In slow time, I'll keep trying to repro when I in fresh Chrome profiles with netlogs enabled and source maps off ( Cloudflare has a desire to open source bash scripts that make this easy. If we do, I'll put an update here).


Image visualization

With torch, images are fed into the network as BGR (I assume), but RGB convention when displaying intermediate results, so intermediate results show up in false colors.
To show the issue, I created a network with just one multiply(1) layer, here's the inference result:

Thanks for the bug report Marco. Torch receives data in RGB order. Caffe receives them in BGR order. Since the visualization routine was originally written for Caffe it is doing an unnecessary conversion from BGR to RGB in the Torch case. The issue should be fixed in #722. Can you try please?
Thanks!

I'll be back in office on Wednesday, is it ok?

I have updated the commit to address Luke's comment. @pansk if you can try this on Wednesday I think it's fine, thanks!

It works, thank you!


Broken responsive layout in stats page

Describe your issue
In some cases, I find the layout breaking when resizing the screen.
Steps to reproduce
Go to page stats and find a card with a long repo title. Resize the screen and, at around 1000px, the totalCount pill starts to overlap the repo title:

The size at which it starts breaking depends on how long the title is.
When resizing, at around 860px, the card overflows its container when more than one card is displayed:

It doesn't happen with only one card displayed.
What was the expected result?
I expect the layout to be correctly responsive and the content to always be clearly visible.
Put here any screenshots or videos (optional)
No response
Would you like to work on this issue?
Yes

Something was already done in #89 to fix a vertical scrollbar but it seems there's more to be fixed :D
Thanks for reporting it and feel free to submit a PR :)


build.gradle changes for distribution

I made changes to build.gradle for distribution in maven central, etc. for #9
It's publish here:
https://bintray.com/juliantoledo/Maven/archive-patcher
I still need to link it to JCenter.

Sounds good to me. @canamaria , what do you think? I know you have mentioned a desire to upstream a bugfix or two, do you want to do that before we land this change to put it into Maven?
Moving forward, @juliantoledo , can you document what process is needed to update the version in Maven after bugfixes are made in the future, i.e. for incrementing the version?

@andrewhayden sure, I will document it. I still need to generate a sources jar, and optionally a javadoc jar, do be able to link it from JCenter.

Hi everyone,
Yes, there is one internal bug fix in particular that we should upstream
before we put this into Maven. I will look into it today and let you know
as soon as it's in.
Thanks,
Ana
On Thu, Aug 24, 2017 at 7:58 PM, Andrew Hayden<EMAIL_ADDRESS>wrote:

Sounds good to me. @canamaria https://github.com/canamaria , what do
you think? I know you have mentioned a desire to upstream a bugfix or two,
do you want to do that before we land this change to put it into Maven?
Moving forward, @juliantoledo https://github.com/juliantoledo , can you
document what process is needed to update the version in Maven after
bugfixes are made in the future, i.e. for incrementing the version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/andrewhayden/archive-patcher/pull/14#issuecomment-324725991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ack2gq6tmotwHPsiaLmnhQQPJq2vOPSlks5sbce3gaJpZM4PBnGq
.

@andrewhayden @canamaria I fixed the jar generation to include sources and javadoc. I also push the sync to JCenter (that should take few hours/days)
https://bintray.com/juliantoledo/Maven/archive-patcher

Published: https://bintray.com/juliantoledo/Maven/archive-patcher

Yes, please.


settings_notifications.js: Refactor using notification_settings variable.

Create an array of all user notification settings and loop through it to update notification settings and display results.

Automated message from Dropbox CLA bot
@sarahcstringer, it looks like you've already signed the Dropbox CLA. Thanks!

PR #5252 includes the revisions made here.

Merged, thanks @sarahcstringer!


If changing the stack fails app state is not restored

When running change-stack on an application with is currently stopped, if it fails will not restore the state to 'stopped' and will leave the application in the started state.
This is with version 0.1.0 run on cf-deployment v28.1.0 using cf v8.6.1

closing since #38 was merged


display of

please replace "Hands" by "Hand Description"

I have updated the code. It will take some times for it to be applied to all output, since it will need all pipelines to be relaunched.


Updating kysely to latest

Also, shouldn't kysely be a peer dependency?

Hey there, thanks for your pull request!
I'm not sure what build has to do with bumping kysely to latest, can you please explain?

Yes, definitely getting the point of peerDependency and should’ve done it upfront but somehow I missed that.
Glad there are people that do want to improve this package, thanks!
Although even with this pull request this won’t be fixed. I’m not able to access my personal computer for the next 10 days when I’m back I could also fix it myself in case.

Closing in favor of #2.


Add ability to use a lock file to coordinate camera bring up

This is one part of several to get depthcams separated into their own (independent) systemd services.

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

src/astra_driver.cpp, line 39 at r1 (raw file):

#include <stdlib.h>
#include <stdio.h>
#include <sys/shm.h>

I would not change this line. Looks like your editor removed the two spaces here.

src/astra_driver.cpp, line 94 at r1 (raw file):

  }
  else
  {

This might all belong in MULTI_ASTRA, since it's only relevant when running multiple cameras. In fact, it seems like it would obsolete the old way of doing this.

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions.

src/astra_driver.cpp, line 39 at r1 (raw file):
Previously, c-andy-martin (C. Andy Martin) wrote…
I would not change this line. Looks like your editor removed the two spaces here.

Done.

src/astra_driver.cpp, line 94 at r1 (raw file):
Previously, c-andy-martin (C. Andy Martin) wrote…
This might all belong in MULTI_ASTRA, since it's only relevant when running multiple cameras. In fact, it seems like it would obsolete the old way of doing this.

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


GCC8 Werror changes

Debian 10 was released the other day, so GCC8 is now the default.

I'm shocked that it ever worked for atomics the way it was


code folding

From<EMAIL_ADDRESS>on 2013-05-09T06:09:26Z
Spyder Version:  2.2.0
Python Version:  2.7.3
Qt Version:      4.8.3, PyQt4 (API v2) 4.9.5 on Windows
Congratulations on the new enhancements. This may be a repetition but if and when “code folding” is included Spyder will be the ultimate development Python tool.
Best
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1384

From<EMAIL_ADDRESS>on 2013-05-09T04:27:05Z
Thanks for the kind encouragement. As you can see from issue #706 , there are a number of people looking for this feature! Just need to figure out how to get it done.
Status: Duplicate
Mergedinto: 706


decouple observe_on from observable

Decoupled observe_on from observable,
please review.
Thank you,
Grigoriy

Thank You!


memory_network_test_in_flight_message_count hangs nondeterministically on main branch

Make it easier to reproduce: add the following to your justfile:
test_memory_network:
  echo Testing the DA task with async std executor
  ASYNC_STD_THREAD_COUNT=1 cargo test --lib --bins --tests --benches --workspace --no-fail-fast memory_network -- --test-threads=1 --nocapture

Then run
just async_std test_memory_network

Need to run ~5 times to observe the test hang, otherwise it passes with no error.
Assigning to @rob-maron because he was the last person to touch this code. 😄

I have what I believe to be the fix in here https://github.com/EspressoSystems/HotShot/pull/1847


Build instruction in README.md needs update

The current build instruction cmake -DCMAKE_INSTALL_PREFIX=../install .. does not work with latest commit.
Need to use cmake -DSTANDALONE=on -DCMAKE_INSTALL_PREFIX=../install .. instead.

Fixed with https://github.com/p4lang/tdi/pull/30


upgrade(metadata): Support Fargate agents in metadata payloads

What does this PR do?
This is a first step to support Fargate agents metadata.
Adds the UUID field present in the v5 metadata payload to:

Inventories payloads
agent_check payload
Remote Config payloads

And adds a ECS Fargate section field in the inventory_agent payload, containing ecs_fargate_task_arn & ecs_fargate_cluster_name
Motivation
RC-1378
Additional Notes
See RFC
Possible Drawbacks / Trade-offs
The ECS payload part has not been tested as I couldn't build a new image
Describe how to test/QA your changes
UUID in payloads

Build & run your agent
Run the agent diagnose show-metadata <payload> command for every payload
Check that UUID is here, properly filled, and stable

ECS Fargate inventory_agent Payload

Update the image in the remote-config-agent-test Fargate cluster in the Sandbox AWS account
Set the latest RC as the image
Connect to your container using aws ecs execute-command  --region us-east-1 --cluster remote-config-agent-test --task <task_id> --container datadog-agent --command "/bin/bash" --interactive
Run the agent diagnose show-metadata inventory-agent command, check if the ecs_fargate_task_arn & ecs_fargate_cluster_name are there and properly filled

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[x] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[x] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[x] If applicable, the need-change/operator and need-change/helm labels have been applied.
[x] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[x] If applicable, the config template has been updated.

Removed requests from processes & agent-apm as they were mistakenly added after a bad merge of main into this branch.

/merge


Refactor config into Config class

We had a bunch of different Options dict instances that we merged for multiple sources to create the config for the package.
This change merges all config into one class that keeps track of the config options, sources, and potential other state.
[skip changeset]
Needed for #19, to implement the diagnose report.
Closes #36

Thanks for the suggestions! I'm going to merge this and address them in a new PR :)


Flush stream_for_searching_tracking_url

A fix for issue 2999
Description
Flush the stream
Motivation and Context
Issue 2999 - ExternalProgramTask: stream_for_searching_tracking_url is not flushed, causing program output to be lost.
Have you tested this? If so, how?
"I ran my jobs with this code and it works for me."

@dlstadther, do you think this could be merged? It's really a quite annoying regression for debugging Spark tasks.
I can also confirm that the change fixes the problem.

@dlstadther, do you think this could be merged? It's really a quite annoying regression for debugging Spark tasks.
I can also confirm that the change fixes the problem.


Proxy cache-control not set correctly

Setting Cache-Control in server.py here is not done correctly.  This line of code
self.send_header("Set-Cookie", "Cache-Control: no-cache")

is incorrect as cache control is not a cookie, it is it's own header field.  The correct code is:
self.send_header("Cache-Control", "no-cache, no-store")

Note that I added no-store.   I did this because when analyzing what the Testa Powerwall returns when I connect the browser directly to the Powerwall, I see both no-cache and no-store.  I believe this is to support older browsers.
Okay, so, by default pyPowerwall should set no-cache, no-store by fixing this bug.
But not caching has severe performance implications when reloading a page with embedded power flow animation graphics.  We should provide an option that lets a user indicate that they want to permit a browser to cache resources.  I propose a setting.
PW_BROWSER_CACHE=<seconds>
and if this is set to anything non-zero then
browser_cache = int(os.getenv("PW_BROWSER_CACHE", "0"))
if browser_cache > 0:
    self.send_header("Cache-Control", "max-age={}".format(browser_cache))
else:
    self.send_header("Cache-Control", "no-cache, no-store")

This dramatically improves page load (after the first load) until whatever number of seconds expires and then the browser will fetch the resources again (on  page reload).  A user can always force the reload by holding down the shift key.

Notice the resources that are now retrieved from memory.
Now, we could further refine this.  Rather than setting the header before the data is retrieved from Powerwall, wait until after so that the content-type is known.  If content-type is text/css, application/javascript or image/png then use max-age else do not permit caching.  Specifically application/json should never be permitted to cache. This is what appears to be happening by default, but I don't know if that is coincidence or not...  It's not really coincidence (as coded right now) because that data is coming from <ipaddr>:8675/api/xxxx that are in the ALLOWLIST so it is not going through the proxy code.
So, it might be safer to move the header code down a few lines where we already have...
ftype = r.headers['content-type']
and then add
browser_cache = int(os.getenv("PW_BROWSER_CACHE", "0"))
# Allow browser caching, if user permits, only for CSS, JavaScript and PNG images...
if browser_cache > 0 and (ftype == 'text/css' or ftype == 'application/javascript' or ftype == 'image/png'):
    self.send_header("Cache-Control", "max-age={}".format(browser_cache))
else:
    self.send_header("Cache-Control", "no-cache, no-store")

And finally, I would move the first line up to where all the other environment variables are retrieved.
David

I notice that server-r.py is very similar but does not include the (buggy) attempt to set Cache-Control.  But this proposed fix should probably be applied to that file as well?

Also having this same problem. It's breaking other applications on the same host until I clear the bad cookie. Looks like this is blamed on 76a2344c.

Brilliant @dkerr64 !!  I love it.  Do you want to submit the Pull Request? I'm happy to make the changes but prefer to make sure you get credit.
As for server-r.py, that was a prototype that has fallen behind a bit. It was attempt to add a simple auth cookie restriction to help with public access. It's a bit clunky and frankly not very secure. If it worked, the functionality would have been ported to server.py.  Long way to say: don't worry about adding the updates to server-r.py. 😉

Thanks David! This is great. I updated release to t25 and will test and push to latest on dockerhub soon.

Just tested this and it's working great. Thanks!

@jasonacox is this on docker hub yet?  I need it there to easily update my Powerwall-Dashboard server.
Thanks
David

Yes the update is now available as jasonacox/pypowerwall:latest

Yes the update is now available as jasonacox/pypowerwall:latest

Thank you.  I just realized that I needed to run the upgrade.sh script.  I did that and it now flys.  Loading the power flow graphic is so much faster now that it is cached.

Awesome!  Thanks for your help and contribution @dkerr64 !!


Improved control over DialogHost popup

Address issue #655 though not in the way requested.
Added a style to control the Popup in the dialog host and created a default style that would allow consumers to either extend or override the default values as they see fit.
I personally like this option better than simply adding an attached property because it gives greater control and is more discoverable.

@ButchersBoy my changes were initially failing the build because I used C# 7 expression syntax for the property. It looks like AppVeyor just recently updated their build environment images to include VS2017. Any chance on getting that bumped up?

Hi guys,
Can you explain me is something is blocking this request and why it hasn't been pushed witht he 2.3.1?
for your recall, it is linked to : #665, #657, #658.
Thanks,

Hi @elvince right now @ButchersBoy  is out taking care of some personal issues. Once he gets time I am sure he will review the PRs (as there are quite a few outstanding). If you need this change sooner, you can always fork the repository and merge this into it and compile the library locally.

Hi, just caught me at 7am before I move home today!  Merged, thanks @Keboo
@elvince it will be in the latest ci build on nuget shortly.

Thanks Guys.
@ButchersBoy If you will be out for a long (I can understand), you might select the best contributors and might be grant them the ability to merge and deploy nuget package so the project will still be active?
you develop a nice package/tool, it should be a shame that it drop from active to inactive :(
GL for your personal stuff.


Get available purchases and restore purchases fails.

flutter_inapp_purchase: 5.6.1
iOS platform, real device
Sometimes getting available purchases and restore purchases fails with an unknown error: 'An unknown or unexpected error has occured. Please try again later.' - steps are unclear.
Please, add extended logging option to FlutterInappPurchase class, that will be logging everything that is going on inside the plugin in specially provided callback for debugging this issue.

I have the same issue


[Article Submission]

Article Title
dizemba
Heading
dizemba
Article Summary
dizemba
Head Picture URL
dizemba
Body
dizemba
Author Name
dizemba
Date
2024-12-11
Category
Technology
Code of Conduct

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

Thank you for submitting your article! 🎉
We appreciate your effort and will review it within 7 days. Meanwhile, please ensure the following:

[ ] Your submission aligns with our goals and mission.
[ ] It does not market you or your company.

Our team will get back to you once the review is complete. Feel free to reach out if you have any questions or updates. 🚀


How to get the wallet balance like react version in the flutter version

Hi, in the https://react-auth-dapp.walletconnect.com/ demo, after we scan QR code, we can sign in the wallet, and can see the balance for the ETH, but in the flutter demo, we can not get the balance, I can not find any data about balance in the log. So how can I get the balance in ther flutter version?

This is part of the Web3Modal that I will begin building this week in a new repo.
It is not part of the base SignAPI that exists in this repo.
I'll post here later with the new repo so you can follow the progress on this feature.

@taichushouwang  - For EVM compatible chains you can use the web3dart package with a node API provider.
From: https://pub.dev/packages/web3dart#connecting-to-an-rpc-server
// You can now call rpc methods. This one will query the amount of Ether you own
EtherAmount balance = ethClient.getBalance(address);
print(balance.getValueInUnit(EtherUnit.ether));

@taichushouwang - For EVM compatible chains you can use the web3dart package with a node API provider.
From: https://pub.dev/packages/web3dart#connecting-to-an-rpc-server
// You can now call rpc methods. This one will query the amount of Ether you own
EtherAmount balance = ethClient.getBalance(address);
print(balance.getValueInUnit(EtherUnit.ether));

Thx, I will check this

I will be adding this functionality to this new repo:
https://github.com/Luzzotica/Web3ModalFlutter


vNode created via toVNode looses dataset since 3.4.0 release

Forgive me if im wrong, but it seems like since 3.4.0 release and #976 was merged. creating a vNode via toVNode and then patching with that node, dataset is lost.
https://stackblitz.com/edit/js-znkhkf?file=index.js
example:
import { datasetModule, init, toVNode } from 'snabbdom';

const patch = init([datasetModule]);
const $container = document.getElementById('app');

const $tmp = document.createElement('div');
$tmp.innerHTML = `<h1 data-something="yes">hello</h1>`;
const newNode = toVNode($tmp);

patch($container, newNode);

If i roll back to 3.3.1 its works as expected. Perhaps toVNode just missed an update with the above mentioned change?

https://github.com/snabbdom/snabbdom/pull/1020

This is still an issue since @nexteditorjs only partially fixed. It only correct works if data attributes has not having dash in names (which is valid data name see https://reference.codeproject.com/dom/HTMLElement/dataset).
// See https://www.npmjs.com/package/snabbdom documentation.
import * as snabbdom from 'https://esm.run/snabbdom';

const patch = snabbdom.init([snabbdom.datasetModule]);
const $container = document.getElementById('app');

const $tmp = document.createElement('div');
/* $tmp.innerHTML = `<h1 data-something="yes">hello</h1>`; */
$tmp.innerHTML = `<h1 data-some-thing="yes">hello</h1>`;
const newNode = snabbdom.toVNode($tmp);

patch($container, newNode);

This will although a valid html string, throw an error. (can be checked https://jsfiddle.net/u251gbn8/)
This was already reported on https://github.com/snabbdom/snabbdom/issues/1031 but for some reason this issue was closed.
If you using DOMStringMap to set dataset elements but you should also replace - following with a letter with letter which is transformed into its uppercase counterpart;
Best regards,
Nikola

Thanks for reporting this @HT2313. There is a proposed fix in #1109.


[AutoPR] keyvault/resource-manager

Created to accumulate context: keyvault/resource-manager

(message created by the CI based on PR content)


Fix Select max-height and overflow

Issue #45

Thx. Nice job.
But if we add media query like https://github.com/react-component/time-picker/issues/45#issuecomment-309728744  should be better?

cc @fast0490f You can leave you opinion at here.


New method added to stream json upserts to Soda.

This will be helpful for very large upserts and in particular help geospace not have to keep too much in memory during uspert time.

soda-fountain #386 SUCCESS
This pull request looks good

LGTM other than one minor spacing thing

soda-fountain #396 SUCCESS
This pull request looks good

Cool thanks.  @AEscobarCruz how urgent is this change?

No urgency from my end doing making these changes on my "free time", and wouldn't mind having others testing this further.

:+1: Nice @AEscobarCruz !

Can one of the admins verify this patch?

jenkins ok to test


Added new logo from ISLT

Can you remake this pull into dev, it's not quite ready to be pushed live


Bug: App crashed while moving files

Description
Your app dies during move operation.  It closed one of the two windows open and opened a new one in a new (not used by me Location):  C:\Users\Dennis\Downloads
Note I was moving from the configured download location (not in "users"), it seems to have moved all my download files and reconfigured the download directory!
Steps To Reproduce
Don't know more than above
Requirements

Not crash
Not reconfigure my directory settings  (it seems to have also played with "my pictures", I don't seem to be able to start Explorer to check if permanent change to Windows settings or just this app misbehaving.
permanent

Files Version
C:\Users\Dennis\Downloads
Windows Version
11
Log File
debug.log.zip

Thanks for the report. Can you give the following information.

What's your Files and Windows version. You can get these in Files settings > About > Click "Copy" on the first box.
Can you fill out the steps to reproduce with step by step on what you did to have Files crash, as your description is unclear.

@dbareis can you confirm how you were transferring the files? Drag & drop, copy & paste etc.

@yaira2 As your apps drag & drop doesn't support moving (as far as I can see), I cut and pasted the files.
v: <IP_ADDRESS> / 10.0.19045.3086
I'm surprised you need 2 x copy to get the information you want and then without titles...  I'm also surprised the debug log doesn't contain that and way more info.
I've stopped using the app as it reconfigured Windows settings for at least downloads & my pictures.  I think the crash might have done this but either way it shouldn't ever happen.

I've stopped using the app as it reconfigured Windows settings for at least downloads & my pictures.

Can you elaborate on this? Files isn't supposed to change any Windows settings.

For the life of my very old laptop, downloads was configured by me to be at: "D:\TMP### Download ###", running Files now also shows it there currently (as I restored its location), it was also correct before the crash as that was my source directory.  After the crash it points to it's default location under "users", "My pictures" also got moved but not under there, (perhaps a random direction within the library?).

Can you try to reproduce this issue again so we have exact steps to reproduce it? I understand if it's too much of an inconvenience, but it'll help us investigate the issue so that it doesn't happen again.

I have already tried, but I think there was a prompt about conflicting files and I said overwrite.  I tried that again a few times and it didn't crash.  It sounds like the debug log needs to record a lot more information.  I looked for dump files but couldn't find any.

It doesn't look like the crash was related to the file transfer.

@dbareis have you been able to reproduce this issue on <IP_ADDRESS>?

@dbareis are you still able to reproduce this? We've made a number of improvements in v2.5.21 but we'd like confirmation before closing this issue.

Hi,
I'm not using the app, I might try it again in 6 months or so
On Mon, 21 Aug 2023, 7:13 am Yair, @.***> wrote:

@dbareis https://github.com/dbareis are you still able to reproduce
this? We've made a number of improvements in v2.5.21 but we'd like
confirmation before closing this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/12825#issuecomment-1685398542,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACICPW67FRL2HGNO7BWDRLXWJ4ZHANCNFSM6AAAAAAZ3IKT7A
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the response.


Working with Transactions

In particular, how to achieve scenarios that would previously have been done with TransactionScope

Looking for exactly this.  I am using repository pattern with EF 7 and have a method where all my DB updates happen.  I'd like to wrap all of these DB calls (Repository.Add + SaveChanges) into a single transaction so if anything fails everything rolls back for all repositories for that particular set of data.
With what I've read it's currently not supported.  Any work around at this time?

In RC1 you would need to use context.Database.BeginTransaction()/UseTransaction() to setup using the same transaction for multiple contexts. If all the data is being saved thru a single context instance, then it is already transactional.

Hi,
We may have also these scenarios:

Multiple dbcontext type with single connection string (same database, different contexes)
Multiple dbcontext type with multiple database (each dbcontext targets to a different database, in different server or same server)
Single dbcontext type with multiple database (for example, a multi-tenant app which has a single dbcontext type but each tenant has it's own database), so there will be multiple instances of same dbcontext type in a unit of work

For such requirements, how can we create a transaction that will be shared by all these contexes :) Will EF Core support that?

And idea? Thanks.

Hey,
First bullet point is covered in https://docs.efproject.net/en/latest/saving/transactions.html#cross-context-transaction-relational-databases-only. The other two would use the same technique, except you would need to use DbConnection.EnlistTransaction(...) since you would need two separate connections. The one disclaimer is that I haven't actually tried that, but I see no reason that it shouldn't work.
~Rowan

Thank you so much for leading me, and sorry for not finding the document before.
Have a nice day.

Closing as we have docs now - https://docs.efproject.net/en/latest/saving/transactions.html

@rowanmiller
Have any Multiple dbcontext type with multiple database (each dbcontext targets to a different database, in same server) demo code? I have not found any about your say DbConnection.EnlistTransaction(...)
hope your help, thanks!

@hikalkan
Have any about Multiple dbcontext type with multiple database (each dbcontext targets to a different database, in same server) demo code? or more help links?
hope your help, thanks!

@xyting I don't have demo code.

@hikalkan Have any opinion about @rowanmiller point DbConnection.EnlistTransaction(...)?

Maybe he is saying UseTransaction. He can answer better :)

Via the Database APIs you can only share a single transaction if you use the same connection, which won't work if you are targeting different databases. You probably want to spin up two transactions and then call Commit() on both once the operations succeed.


Getting error while creating certs with the DigiCert plugin

I am getting the following error while creating certs with the DigiCert plugin. The DigiCert plugin was able to load successfully.
I tested my DigiCert API account using curl and the API key and it works fine.
2017-03-02 03:26:19,858 ERROR: 'message' [in /www/lemur/lemur/common/schema.py:160]
Traceback (most recent call last):
File "/www/lemur/lemur/common/schema.py", line 158, in decorated_function
resp = f(*args, **kwargs)
File "/www/lemur/lemur/certificates/views.py", line 272, in post
return service.create(**data)
File "/www/lemur/lemur/certificates/service.py", line 232, in create
cert_body, private_key, cert_chain = mint(**kwargs)
File "/www/lemur/lemur/certificates/service.py", line 181, in mint
cert_body, cert_chain = issuer.create_certificate(csr, kwargs)
File "/www/lemur/lemur/plugins/lemur_digicert/plugin.py", line 303, in create_certificate
raise Exception(response.json()['message'])
KeyError: 'message'

So the issue is with the "DIGICERT_URL" setting.
In the old OpenDNS DigiCert plugin, this value point to the full API URL:
DIGICERT_URL = 'https://www.digicert.com/services/v2'
I change it to be the base URL and it is working now:
DIGICERT_URL = 'https://www.digicert.com'
It'd be great if we can have some examples in the documentation so users know what values should be set on these places.

Makes sense, will try to add some additional documentation. In general all of the Lemur plugins use the root URL leaving it up to the plugin to specify the API version it works with.


False negative for Bosnia and Herzegovina

Hi, Bosnian mobile numbers without a trunk code (zero) are being rejected. If you look at the [Wikipedia article].
(https://en.wikipedia.org/wiki/Telephone_numbers_in_Bosnia_and_Herzegovina#Mobile_phone_codes), Bosnian mobile numbers are valid with or without a trunk code. Twilio agrees, and says that the actual phone number value is the same with or without the leading zero. This is a problem for me because my client-side phone number formatting library is stripping leading zero digits.
I was trying to figure out how to make the trunk code optional in the DSL, but I wasn't quite sure how.
Thanks! This library has been awesome (been using it in production for 9~ months now).

Hi Miguel! Thanks for the issue :rocket: – do you have a few confirmed failing examples to help me? Btw, the solution is probably to make trunk be 0?, see Belarus for a similar example.

Thanks for the quick reply @floere!
An example number is<PHONE_NUMBER>1, which is not plausible, vs<PHONE_NUMBER>41, which is plausible. Twilio lookup shows that both numbers are valid, and returns the same phone_number field for both of them (without the 0 trunk).
Changing the trunk code to 0? isn't solving the situation, Phony is running into an issue matching the correct code and fails on a national number length validation.
-Miguel

Thanks! Changing the trunk code is probably solving one issue. The other issue is lengths – and currently, the rules are extremely simple: country '387', trunk('0') | fixed(2) >> split(3,2,2) – as an example, if the last digit group can be longer, then we'd have to change the rule to country '387', trunk('0') | fixed(2) >> split(3,2,2..4). Does that help? Looking at the Wiki page, it seems as if it mostly needs to be country '387', trunk('0?') | fixed(2) >> split(3,3).

OK, I ended up changing the behavior in my client library rather than here. I'm still not quite sure if Bosnian mobile numbers should always be "061" or not. Feel free to close this issue if you think that no work needs to be done on the phony side.

Work probably does need to be done, but I am not sure yet what. So I'll leave it open if you don't mind. Glad you found a solution! :)

Currently closing until it is brought up again. Thanks again! :rocket:


Fix PlayerMPD.prev/next() when stopped

Fix PlayerMPD.prev/next() when stopped

Avoid MPD-related crashes during all prev/next() calls.
Explicitly handle prev() in stopped state, configurable via
playermpd.stopped_prev_action.
Explicitly handle next() in stopped state, configurable via
playermpd.stopped_next_action.
Explicitly handle next() when reaching the end of the playlist:
jukebox-daemon will now ignore the action by default (similar to v2).
It can also be configured to rewind the playlist instead by setting
the new config option playermpd.end_of_playlist_next_action: rewind
or to stop playing.

Fixes #2294
Fixes #2327

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
493

Relevant Lines:
1286

💛  - Coveralls

No action could be a third option, but that could also be added in a later PR.
https://github.com/MiczFlor/RPi-Jukebox-RFID/issues/2327#issuecomment-2050522393

No action could be a third option, but that could also be added in a later PR.

Added now.
As there have been further changes for #2327, this PR needs another review and new testing (I haven't tested the most recent state yet, maybe @igorwalton can help?).


Admin Route

Hi This is more of a question then an issue Ok so here the question I want to be able to hook into your app and be able to use the resource folder to hold a duplicate folder structure and use that to override a page i.e so if post/index.blade.php is found in the resource folder it uses that one if its not it uses yours.
My Way would be to hook the routes but how does laravel know to use your routes instead of the regular routes and is there a way to hook the routes. I think this would be idea to allow to add on features that will not get overwritten on updates

Sorry if I misunderstood your question, but it sounds like you're looking to customize views like so: https://the-control-group.github.io/voyager/docs/#voyager-docs-customization-overriding-views

Closing due to inactivity.


Stalled daemon (writes okay, but no reads)

I'm having an issue where InfluxDB stops accepting query requests from HTTP after about a day.
Writes still happen and get recorded through the graphite plugin I'm using, but when I send in a select query, it never returns. Doesn't time out, either. cURL just sits there waiting for a response.
And there's absolutely nothing in the logs.
 It starts back up fine though and then runs for another day or so before somehow going back into the exact same state.
Any ideas? I'm kind of at a loss. It's currently in its bad state so I can try to debug, but I can't keep it that way for long, since I need to see server stats soon.
Version: "InfluxDB v0.8.5 (git: 9485e99) (leveldb: 1.15)"
Using RocksDB.
On CentOS 6.
Installed from the RPM.

I am now seeing this same issue on another server (using leveldb). I did manage to capture a partial stack trace. I apologize that the data is truncated. I strace'd the process and reconstructed this from the write calls to stderr, and strace wasn't capturing the whole string. This is using the 0.8.7-1 RPM.
https://gist.github.com/shanemhansen/8e77955d9dc9a7900287

Now seeing the same issue linux/x86_64 with 0.8.8

I'm observing this too. Here is my instance info (InfluxDB v0.8.8 (git: afde71e) (leveldb: 1.15)). However as per the blog on v0.9, they have acknowledged this issue (number of open files opened by InfluxDB) and say that this will be optimized.


HACS Compatibility

It would be really nice if this was available to install via HACS.
If i get time i may try to submit a pull request to provide HACS compatibility, but the guidelines are listed here: https://hacs.xyz/docs/publish/start

added and released v0.1b https://github.com/JOHLC/pfSense-FauxAPI-ha/releases/tag/v0.1b

Note: you will likely want to adjust your configs to the new example

Awesome. Thanks! I'll check it out! The Wan stats are so useful.


Images are broken for many of the projects

What happened?
I did some research and found out many PNG files of the project are missing in https://github.com/swapnilsparsh/30DaysOfJavaScript/tree/master/30DaysOfJavaScript/assets
Screenshot:

How can we reproduce this bug?

Visit https://30daysofjs.netlify.app/
Scroll down and you will see blank card with no images

Desktop Information (Optional)
No response
Urgency (Optional)
High priority

@swapnilsparsh Please allow me to fix this I will add the images for missing projects as well as add dummy images if there is an issue with the images

Hii @swapnilsparsh please review the PR #1108


MM-19759 - Added TOS and Privacy Policy default links in About Screen

Summary
Changed Terms of Service and Privacy Policy links in the About screen to always point to MM default links and not rely on the customizable links in the config.
The default links are currently defined in constants here in the mobile app for 5.17 quality release but should be retrieved from the API. This is captured in this ticket for future release.
Ticket Link
MM-19759
Related Pull Requests

WebApp PR https://github.com/mattermost/mattermost-webapp/pull/4084

@marianunez
Can you please confirm these are the default links:
Privacy Policy - https://github.com/mattermost/mattermost-server/blob/master/build/PRIVACY_POLICY.md
Terms of Service - https://about.mattermost.com/default-terms/
Thank you!

@jgilliam17 Default links:
Terms of Service: https://about.mattermost.com/default-terms/
Privacy Policy: https://about.mattermost.com/default-privacy-policy/

Thank you @marianunez
I am testing on a build from this post
Privacy Policy link is not opening to default you mentioned above.
It's opening this instead  https://github.com/mattermost/mattermost-server/blob/master/build/PRIVACY_POLICY.md
Terms of Service is correctly opening to https://about.mattermost.com/default-terms/

It's opening this instead mattermost/mattermost-server:build/PRIVACY_POLICY.md@master

Yes, when you click on https://about.mattermost.com/default-privacy-policy right now it seems to be redirecting to https://github.com/mattermost/mattermost-server/blob/master/build/PRIVACY_POLICY.md.
@wiersgallak can you confirm this is expected?

@jgilliam17 & @marianunez yes that is expected.  That URL redirects to the file in github.

What just happened here? Did we remove the ability to customize this links? Why?

We removed the customizable links only from the About screen. Those links are still customizable and show in other screens like the Login screen.
The decision was made for legal reasons that @wiersgallak could explain more.

@enahum In the about section, we want the links to be permanent and refer to our Mattermost privacy policy & terms of service to be consistent with how we changed this on the webapp for the about section.  This ensures our end-users and customers always have reference to how data is used within the product.
Is there a reason we don't have the customized privacy policy & terms link on the login page for the app to be more consistent here? I recommend a ticket to add them so that we are consistent across the platforms

@marianunez @wiersgallak thanks for your answers... it now makes a bit more sense.
We do not have them on the login screen as they were never needed cause of them being present in the about dialog and initial designs did not include them.
I'm ok if we add them, but some UI/UX needs to be done first. Make sure to talk it over with @esethna


WIP- generate offline map workmanager

Description

Links and Data

What To Review

How to Test

@kaushikrw Is this PR good to be closed? I'm guessing it was set up to help with #70

@kaushikrw Is this PR good to be closed? I'm guessing it was set up to help with #70

Yeah, this can be closed.


Enhancement to edit "Version"

Hi.
Do you plan to add support for version?

Hello @pr0l
Thank you for your request.
It looks like that version field works well when using drop-down list as display parameter in the field's options :

You should use this fix as a workaround until I can update the plugin. I'll try to find a way to make checkboxes display parameter works.

Hello @pr0l
I just updated the plugin. Version field with checkbox display is now supported.
If you find any problem with this update don't hesitate to post it here and we will find a solution
If you find the plugin helpful, please take a minute to star it on Github and rate it on http://www.redmine.org/plugins/redmine_issue_dynamic_edit 👍

@Ilogeek unfortunately the version field is not dynamic. This is the built-in redmine function.

Hi @pr0l
My bad, reading your request I thought you were speaking about custom field "version" (in Administration > Custom fields ).
The last update add the support for Target version and Assignee fields :

Don't hesitate to come back here if it doesn't work as expected.
If you find the plugin helpful, please take a minute to star it on Github and rate it 5 stars on http://www.redmine.org/plugins/redmine_issue_dynamic_edit 👍

everything works fine. Thank you very much for your work. For the plug-in to work completely, it needs to be reinstalled.


Outdated version in LuaRocks and Releases page

Hi. Currently master branch has <IP_ADDRESS>-1.rockspec file, and the reader might think it's for <IP_ADDRESS>-1, but the content of the file says <IP_ADDRESS>-1. However, in tags there is already <IP_ADDRESS> with the same outdated .rockspec, and meanwhile in LuaRocks only <IP_ADDRESS>-1 version is available.
I would like at least to sync the latest version in LuaRocks.

Continues discussion in https://github.com/openresty/lua-cjson/issues/96


[Feature] Disable speedtesting

Description of the feature
i want to be able to disable speedtesting since my internet is kinda slow and when it does run it takes all of my speed. i just want to disable it entirely
Additional context
No response

Hello
You can achieve this by setting the environment variable DASHDOT_SPEED_TEST_INTERVAL to a higher number than default which is 240 (every 4 hours).

I want to disable it entirely, though
On Fri, Aug 19, 2022, 09:00 AuthorShin @.***> wrote:

Hello
You can achieve this by setting the environment variable
DASHDOT_SPEED_TEST_INTERVAL
https://getdashdot.com/docs/config/widget-options/network#dashdot_speed_test_interval
to a higher number than default which is 240 (every 4 hours).
—
Reply to this email directly, view it on GitHub
https://github.com/MauriceNino/dashdot/issues/288#issuecomment-1220322322,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJVL4HL3I67DZJKL7WIEK3VZ4WJ5ANCNFSM565GOT3A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Add a big number like<PHONE_NUMBER>00 and it will never test again.
I understand you want an option for turning it off but till then it's your best bet.

Hey there, thanks for the issue and also thanks @AuthorShin for providing information! :)
You can also hide the results on the frontend by using DASHDOT_NETWORK_LABEL_LIST and excluding the variables speed_up and speed_down.
Unfortunately, I still kind of need one run, so that I have a base value for the network graphs. Otherwise, they would jump around a lot. That's why there is currently no way of disabling it and why I do not plan of adding a way.


Added ProAlanase to proteases.tsv

By popular request, ProAlanase has been added

Lmao, every test broke due to missing tab characters in Proteases.tsv


on /action - turn panels into buttons?

That will make us able to do the mouseover effect right? that would be nice.

Done. Dunno why it took so long....
It is not 100% on mobile. They are not resizing properly for some reason. But it seems to be working, more or less.

Closing, and opening a new issue


You can copy this: ```
Place it before the start and after the end of your example xml.
Thanks!

Hi @lanyuzx , Could you provide the input XML file, expected output, and actual output?


This adds a simple input interface

Formulador-lite is a simple input interface, for converting inputs such as 1+2 or [1/2]+[2/3] into formulador drawings.
Currently fixing the block system to accommodate larger formulas, and fixing the exponent system to allow for closed-parentheses.

Nice idea :)

Thank you! Any suggestions on what to add to it?

I've updated the code to build drawings organically instead of using quoted lists, and have updated the block-unit system. Fiasco is giving me an odd bug- (run-package-tests) returns "invalid number of arguments: 9." I don't necessarily need tests for this project but any illumination would be appreciated.


zh_CN.lang

Um.. Still have some questions:

Translations seems to be wrong at line 6 & 7;
Untranslated statements between line 11 and 14.

@Snownee
'Translations seems to be wrong at line 6 & 7;'
I know about this, but you do not feel translated‘裁剪悠悠球/粘液悠悠球’，is it hard to pronounce（有些拗口）?
‘Untranslated statements between line 11 and 14.’
I generally do not go to translate cfg
Moreover, they are only 'title'（标题）

At least its not so hard to pronounce for me. Now the translations tend to be a attribute of Diamond & Iron Yoyo, but the fact is there are obviously different.

If these look good to the two of you, then I'll pull. Thank you so much for the contribution!

@Snownee

@DYColdWind Nice work! Thanks for taking my suggestion!

Thank you two!


Line characters not working

OSX 10.9.5
used iterm2 and terminal

Can you try the first example in the blessed project readme? Want to know
if this is a blessed or blessed-contrib issue.
On Jan 15, 2015 6:07 AM, "Jared A. Scheel"<EMAIL_ADDRESS>wrote:

OSX 10.9.5
used iterm2 and terminal
[image: screen shot 2015-01-14 at 10 03 50 pm]
https://cloud.githubusercontent.com/assets/211478/5752390/b28af6a6-9c39-11e4-9465-d584f32ef9cb.png
—
Reply to this email directly or view it on GitHub
https://github.com/yaronn/blessed-contrib/issues/3.

Ah, appears to be an issue with blessed. I'll file there. Thanks!


[Feature] Create backend GraphQL APIs for change logs

What is the problem?
Change log modal is rendering dummy codes logs.
Your solution
Create backend GraphQL APIs to query change logs for a given user.

https://github.com/short-d/short/pull/530

Here are some steps need to create this API:

Propose GraphQL API schema for change logs on this issue.
Propose new DB tables for change logs on this issue.
Create DB migrations, create corresponding repositories, and add DB integration tests.
Create a changelog package within usecase package and add the required usecase with unit tests.
Add GraphQL schema and resolvers.
Hook up the GraphQL API with the usecase.
Add required code in wire.go required for dependency injection
Integrate change log API with the frontend

GraphQL schema:
type AuthQuery {
	changeLog(): ChangeLog
}

type AuthMutation {
  	updateLastSeenChangeLogUser(): Time
}

Columns for DB table "change_log":

12_add_changelog_table.sql
-- +migrate Up
CREATE TABLE "change_log"
(
    "id"            CHARACTER VARYING PRIMARY KEY,
    "title"         CHARACTER VARYING VARCHAR,
    "summary"       CHARACTER VARYING VARCHAR,
    "releasedAt"    TIMESTAMP WITH TIME ZONE
);

-- +migrate Down
DROP TABLE "change_log";

13_add_last_seen_changelog_column.sql
-- +migrate Up
ALTER TABLE "user" ADD last_seen_changelog TIMESTAMP WITH TIME ZONE;

-- +migrate Down
ALTER TABLE "user" DROP last_seen_changelog;

@byliuyang

GraphQL schema
type AuthQuery {
  changeLog: ChangeLog!
}

type ChangeLog {
    changes: [Change]!
    lastViewedAt: Time
}

type Change {
  id: ID!
  title: String
  summaryMarkdown: String
  releasedAt: Time!
}

input ChangeInput {
  title: String
  summaryMarkdown: String
}

type AuthMutation {
  viewChangeLog():  Time!
  createChange(change: ChangeInput): Change!
}

DB tables
change_log

Column
Type
Default
Constraints

id
CHARACTER VARYING(5)
no default
primary key unique

title
CHARACTER VARYING(100)
no default

summary
TEXT
no default

released_at
timestamp with time zone
no default

// user_change_log

Column
Type
Default
Constraints

user_id
CHARACTER VARYING(5)
no default
foreign key => user.id

last_viewed_at
timestamp with time zone
no default

@byliuyang  createChange() cannot be written right now until admin dashboard is completed...we would have to manually add changelog to database...user_id is not populated right now...may I add another column email to user_change_log...what would happen if email changes?

@kapillamba4 Let's keep user ID but just not make it foreign key yet. I am now working hard on user email to user ID migration ( hopefully will be done soon ).
createChange is needed. We can use Insomnia to invoke createChange API before the admin dashboard is done. Your next ticket will be creating the admin dashboard and flesh out change log section of it!!!:

@byliuyang @kapillamba4 Hey, I was reading through the thread and had a couple of doubts.

Are we showing the change logs for newly registered users?
If a user is logging in after a long time, are we showing the whole list of change logs?

@rohithbalaji123 We only show the ones they haven't seen yet by default. When clicking on view all button, we will all changes for now. In the future, we want to have pagination.
@rohithbalaji123 We should only show the changes happened after the user created his or her account.

Ohkay, great. If I login after say, an year, will I see all the changes happened since then? Or are we limitting to any threshold number of items in the list?

@rohithbalaji123 We should have some threshold.

Done.


Investigate use of document as target for event listeners

What is the purpose of the work?
A few of our components use the document as the target for event listeners. This could potentially cause issues when there are multiple components on the same page, listening for events with the same name.
Already identified and raised #327 for ic-menu component
Why do we need it?
To prevent running unnecessary code & avoid potential bugs with multiple components on a page

Investigation carried out. The most common event that is listened to at the document level is the themeChange emitted by ic-theme. this is ok to stay as is
Further issues #342 and #343 raised for components that should be modified.


Consider deprecating or unpublishing object-pick?

I know I've installed using the previous name.

thanks, I thought I had!

deprecated, thanks for the heads up. it's used in several libraries currently, once that changes I'll unpublish.


Ig/Met data values do not reset when pressing cancel

Describe the bug
When copying values from one spot to another in the Ig/Met page and pressing cancel the values do not change.
To Reproduce
Are you able to reproduce the problem in a new project after restarting the app?
Yes
Steps to reproduce the behavior:

Go to '.Ig/Met' page from notebook tools
Click on 'Copy Petrology Ig.Met Data'
Press' Cancel

Expected behavior
Form should reset
Screenshots

Fixed with commit 2d78da635efe234ae31b3c699f7f8bae5b2c413f


[Feature Request] add back Copy Symbol in code

In the old version I can easily copy symbol in code like this
(Symbol)0xE7FC,
But it missing in recent releases, can you add it back?

If you want the C# version of the Symbol, there is an option to toggle between C# & XAML in the Settings view. Otherwise it's currently only showing Symbol when it is actually a valid cast - which is only on Segoe MDL2 Asset for support enum values.


Slides doesn't support scrolling large markdown files

Describe the bug
When a large markdown file is opened in slides, it just jumps to the last line.
To Reproduce
Steps to reproduce the behavior:

Install latest slides
Open a markdown file larger than ROWS

Expected behavior
slides should support scroll
Screenshots
No need easily reproducible

What's your use case here? I will consider adding scrolling but I feel like you might just want shorter slides.

What's your use case here? I will consider adding scrolling but I feel like you might just want shorter slides.

I normally use slides to read large READMEs, a very unique use case 😂.

Check this program out! https://github.com/charmbracelet/glow

It might be interested to add scrolling for the use case of executable blocks.
When displaying the output of a code block with Ctrl+E, it could quickly overflow the slide depending on the output.

It might be interested to add scrolling for the use case of executable blocks.
When displaying the output of a code block with Ctrl+E, it could quickly overflow the slide depending on the output.

Do you have a specific example of when you need this? It feels to me that the output should be relatively short (at least should fit on one slide) especially if this is a code demo. I can see why we might want scrolling but I want to avoid preemptively implementing scrolling, if it encourages people to have longer outputs, which the audience won't even be able to read.

In my case i do simple slides for exploits demo, and for example when i have a slide with a simple curl over the database which return a json it could be a bit longer than the slide geometry so i can't really scroll back to the start of the content.
I agree on the fact that smaller is the slide, better will be the audience attention.
Otherwise the workaround i used is to tail -f a file output.txt just next to the slides, but it would be prettier to add scrolling in order to avoid seeing redirection.

Instead of using the file why not just | tail -n X (where X is < terminal_rows - lines(code_block))
I think this will serve you much better than the file and even better than scrolling (IMO) because you won't have to scroll back to your code and it looks like the repetitive data isn't entirely useful to the presentation (a snippet would suffice, let me know if this is not the case though)
What do you think? (As a workaround, I can still probably be convinced that scrolling is a good idea, but I think for the repetitive json data the tail -n 30 solution works better so that you can keep your code in frame and truncate the output, let me know if I'm wrong and you need the entire output displayed though)

Yes that could be a great workaround if the purpose is to show the shape of the json data without concern about the data itself.
But if you want to target a specific value inside the result you can't really predict if the field will be at the begining or the end of the result.
Moreover if the json structure is not the an array of the same "object' you will miss other key value pair which are not at the end of the result.

@Babidiii -

But if you want to target a specific value inside the result you can't really predict if the field will be at the begining or the end of the result.

Maybe you could use jq's select() or match() to target the specific value you are looking for inside the result.
For example (continuing on @maaslalani's example):
$ curl https://jsonplaceholder.typicode.com/todos | jq '.[] | select(.id == 200)'
{
  "userId": 10,
  "id": 200,
  "title": "ipsam aperiam voluptates qui",
  "completed": false
}

$ curl https://jsonplaceholder.typicode.com/todos | jq '.[] | select(.title | test("magnam"))'
{
  "userId": 3,
  "id": 57,
  "title": "pariatur et magnam ea doloribus similique voluptatem rerum quia",
  "completed": false
}
{
  "userId": 4,
  "id": 64,
  "title": "voluptas consequatur qui ut quia magnam nemo esse",
  "completed": false
}
{
  "userId": 4,
  "id": 78,
  "title": "reiciendis est magnam amet nemo iste recusandae impedit quaerat",
  "completed": false
}
{
  "userId": 8,
  "id": 155,
  "title": "voluptatem nobis consequatur et assumenda magnam",
  "completed": true
}
{
  "userId": 10,
  "id": 199,
  "title": "numquam repellendus a magnam",
  "completed": true
}


Support for public API

As mentioned in Mastodon’s documentation some API endpoints do not require an authentication. (e.g. statuses/:id)
https://github.com/tootsuite/documentation/blob/master/Using-the-API/API.md#timelines
If I try to create a Mastodon object without a an access_token I get the following error:
./node_modules/mastodon-api/lib/mastodon.js:345
                    throw new Error('Mastodon config must include \'' + reqKey + '\' when using \'user_auth\'');
                    ^

Error: Mastodon config must include 'access_token' when using 'user_auth'

Is it possible to make unauthorized API calls?

Yes, just pass a dummy-string as access_token: new Mastodon({access_token: 'dummy'})

Ok, in this case I see this issue being done when this cote gets added to the documentation (readme).

I will add publicOnly as a config parameter in the next version, which is cleaner than a dummy token.


Sonoff NS Panel - Removed Temperature Indication

Thank you very much for your contribution and efforts to make the Sonoff integration perfect, it is very important to all of us. In the last update, the thermostat of the Sonoff NS Panel was corrected, but the current temperature indication entity was removed. In my case, I use this entity to monitor the ambient temperature, since I don't use the thermostat, and for me it would be very good if it returned. Thank you very much.

This device using specific logic with temp correction. It's implemented only in climate entity. You may use template sensor if you don't like to work with climate entity.

OK. Thanks for your response. However, I did not find the indication of the
current temperature of the thermostat measured by the sensor, only the
definition temperature. Am I doing something wrong?
Em ter., 3 de mai. de 2022 às 10:23, Alex X @.***>
escreveu:

This device using specific logic with temp correction. It's implemented
only in climate entity. You may use template sensor
https://www.home-assistant.io/integrations/template/ if you don't like
to work with climate entity.
—
Reply to this email directly, view it on GitHub
https://github.com/AlexxIT/SonoffLAN/issues/810#issuecomment-1116092745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU6YXIET2BAMGH73JMPOG33VIESE7ANCNFSM5U63CJBA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Henrique Luiz Serpa
@.***

https://github.com/AlexxIT/SonoffLAN/blob/c1f491abb1b2ae803a192843f917214af7f3493a/tests/test_entity.py#L1023-L1026


Javadocs lied about concurrency safeness, oldschool fix.

Fixes a particularly nasty stacktrace that led to only a single player being up for kickbanning per round.
'Tis what I get for believing the docs. Oldschool fix; just keep track of removals and do it after the loop.
This is up on Civtest, and verified using two accounts; emulated failure condition by having both accounts "be rescanned and cleared" simultaneously. On civmain this failure condition was repeatedly exhibited, so failure to reproduce indicates this is a safe patch.

@ttk2 I'm going ahead and merging this as it's a break-fix, and vital for rollout tomorrow/this morning. If possible please throw it online alongside the config I am sending you in a few moments.


Add checks in setup-kind for existing steps

Summary
This PR fixes up some issue with running hack/setup-kind.sh multiple times.
Ticket Link
Fixes #64
Release Note
Fixes issues with setup-kind.sh on subsequent runs

i think we are not using this script anymore for ci testing, maybe only for local development


Better Optimal Sorting algorithm

Formula should be: sqrt(Entfernung^2 + (1/((2*(1-Belegung))^2 + 1))*Belegung^2) und dabei Entfernung in Meter und Belegung normiert 0...1
1/((2*(1-Belegung) looks like this:

reason is, that crowded parking lots are rated down.

Didn't get to it yet, sry. So I shouldn't merge your open pull request and incorporate this instead?

No hurry, just a reminder. I can do this with the next pull request.
Which units do we have for Entfernung and Belegung?

Mailbox.org - Damit privates privat bleibt

Am 12.10.2015 um 14:55 schrieb Kilian Koeltzsch<EMAIL_ADDRESS>Didn't get to it yet, sry. So I shouldn't merge your open pull request and incorporate this instead?
—
Reply to this email directly or view it on GitHub https://github.com/kiliankoe/ParkenDD/issues/114#issuecomment-147389936.

Units are km and 0...1 (I think :D)

closed in 5a12535c77e98826067fca522d3d2c1f79e6cc35


Collection was mutated while being enumerated

I am developing an app that show users in nearby area in the maps. I use cluster manager to cluster users. The users/cluster items are updated periodically from server. Steps are :

call [clusterManager clearItems]
add all new item to the cluster manager
call [clusterManager cluster]

But sometimes I got error : Collection was mutated while being enumerated in GMUNonHierarchicalDistanceBasedAlgorithm.m, particularly in this place :
/**
 * Returns the set of clusters of the added items.
 */
- (NSArray<id<GMUCluster>> *)clustersAtZoom:(float)zoom {
  NSMutableArray<id<GMUCluster>> *clusters = [[NSMutableArray alloc] init];
  NSMutableDictionary<GMUWrappingDictionaryKey *, id<GMUCluster>> *itemToClusterMap =
      [[NSMutableDictionary alloc] init];
  NSMutableDictionary<GMUWrappingDictionaryKey *, NSNumber *>     *itemToClusterDistanceMap =
      [[NSMutableDictionary alloc] init];
  NSMutableSet<id<GMUClusterItem>> *processedItems = [[NSMutableSet alloc] init];

  for (id<GMUClusterItem> item in _items) {  <--- HERE
  ....

Any advice how to prevent this error from occurring?

Any chance of a minimal reproduction case that demonstrates the issue?

Closing this out as stale. Please re-open when you have uploaded a minimal reproduction sample to a GitHub repo that I can checkout and use to debug what is going wrong, and verifying a fix. Thanks!

I'm facing same issue but cannot reproduce it all the time. It happens sometimes as mentioned in the initial question. @danartinho did you manage to prevent it somehow?


Civic Bosch steering enhancements

add steering enhancements for Civic Bosch:
-limit steer max to a00 to stay outside stock fw deadzone
-add support for modded fw

Will be doing testing on this VS what the original PI values were this week. Back to back, same time of day, same route, and with measure steering accuracy script logs. From my initial testing, using the nidec civic PI values on the hatchback are actually worse than what Joe had originally.

Accuracy testing for shortened request range on bosch civic thanks to discord user jad:
stock 0.7.4:
angle:  0 | error: 0.16 | =: 15% | +:  39% | -: 44% | count: 481996
angle:  1 | error: 0.17 | =: 14% | +:  26% | -: 59% | count: 370763
angle:  2 | error: 0.20 | =: 12% | +:  35% | -: 51% | count: 136276
angle:  3 | error: 0.23 | =: 11% | +:  39% | -: 48% | count: 57672
angle:  4 | error: 0.27 | =: 10% | +:  40% | -: 48% | count: 43897
angle:  5 | error: 0.37 | =:  7% | +:  37% | -: 54% | count: 20784
angle:  6 | error: 0.49 | =:  8% | +:  31% | -: 59% | count: 15127
angle:  7 | error: 1.01 | =:  3% | +:  20% | -: 76% | count: 4568
angle:  8 | error: 1.33 | =:  2% | +:   9% | -: 88% | count: 3734
angle:  9 | error: 1.98 | =:  2% | +:   2% | -: 94% | count: 1928
angle: 10 | error: 3.15 | =:  0% | +:   0% | -: 99% | count: 1862
angle: 11 | error: 4.06 | =:  0% | +:   0% | -: 99% | count: 1006
angle: 12 | error: 5.05 | =:  0% | +:   0% | -:100% | count: 1107
shortened request to a00, kp 1.1, ki 0.33:
angle:  0 | error: 0.19 | =: 12% | +:  39% | -: 47% | count: 467720
angle:  1 | error: 0.21 | =: 11% | +:  24% | -: 64% | count: 367627
angle:  2 | error: 0.25 | =:  9% | +:  34% | -: 56% | count: 127996
angle:  3 | error: 0.30 | =:  8% | +:  32% | -: 59% | count: 48508
angle:  4 | error: 0.34 | =:  8% | +:  32% | -: 58% | count: 36137
angle:  5 | error: 0.53 | =:  7% | +:  27% | -: 64% | count: 16355
angle:  6 | error: 0.84 | =:  5% | +:  14% | -: 80% | count: 11483
angle:  7 | error: 1.53 | =:  0% | +:   3% | -: 96% | count: 4251
angle:  8 | error: 2.55 | =:  0% | +:   1% | -: 98% | count: 3097
angle:  9 | error: 3.38 | =:  0% | +:   0% | -: 99% | count: 1722
angle: 10 | error: 4.38 | =:  0% | +:   0% | -:100% | count: 1701
angle: 11 | error: 5.22 | =:  0% | +:   0% | -:100% | count: 1091
angle: 12 | error: 5.86 | =:  0% | +:   0% | -:100% | count: 1081
steering error seems to be worse with the shortened range. added a third step to request range interp shorten ramp up from a00 to f00. needs testing.


cppcheck of codebase as of September 6, 2019 [code quality]

I have cppcheck 1.88 installed on WSL/Ubuntu; also have a fair chunk of VS 2017 & 2019 materials installed. Ran a test against the current codebase,
cppcheck --std=c++17 --enable=all --force --inconclusive -I "C:\opt\vsbuild\VC\Tools\MSVC\14.16.27023\include" -I "C:\opt\vsbuild\VC\Tools\MSVC\14.16.27023\atlmfc\include" . 2> ../powertoys-cppcheck.txt
powertoys-cppcheck.txt

Thanks!!!


switch to tiered consolidation by default

Please note that for legal reasons we require you to sign the
Contributor Agreement
before we can accept your pull requests.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/4432/

https://jenkins.arangodb.biz/job/arangodb-matrix-pr-linux/5961/

https://jenkins.arangodb.biz/job/arangodb-matrix-pr-linux/5962/


"Cătina" is listed in the drop-down menu

Description:
When user writes "Cătina" the location appears in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Cătina"
Expected results
The location appears in the drop-down menu as "Comuna Cătina, CLUJ"


Dynasty Warriors Next [PCSE00014]

App summary

App name: Dynasty Warriors Next
App serial: PCSE00014
App version: 1.01

Vita3K summary

Version: v0.1.5
Build number: 2658
Commit hash: https://github.com/vita3k/vita3k/commit/48f3e7ce
CPU backend: Dynarmic

Test environment summary

Tested by: gymzatan
OS: Windows 11
CPU: AMD Ryzen 7 5800H
GPU: AMD Radeon(TM) Graphics
RAM: 16 GB

Issues
Game does not boot.
Screenshots

Log
vita3k.log
Recommended labels

Nothing

is ingame

Vita3K summary

Version: v0.1.5
Build number: 2663
Commit hash: https://github.com/vita3k/vita3k/commit/e8151664
CPU backend: Dynamic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:

Game goes to menu but doesn't go ingame, and occasional crashes in menu

Screenshots:

Log
vita3k.log
Recommended labels

Menu
Crash

Vita3K summary
 * Version: v0.1.9

 * Build number: 3502

 * Commit hash: [Vita3K/Vita3K@4ac8a416](https://github.com/Vita3K/Vita3K/commit/4ac8a416)

 * CPU backend: Dynarmic

Test environment summary
 * Tested by: piau9000

 * OS: Windows 10

 * CPU: AMD Ryzen 7 3800x

 * GPU: RTX 2060

 * RAM: 16 GB

Issue:
 * The music in the game plays with a lot of distortion (crackling), even when the game is playing at full speed.
 * Other in-game sounds plays without any problem. In the log, it throws the message "game is using unimplemented filter audio module".

Screenshots:
Log
vita3k.log
Recommended labels
 * Ingame +

 * Audio Quality


Install option in the main header

Currently there is no easy path to find the instruction to install the language. It is very common action so it deserves a place in the main header.
My idea is we can replace the link of Code with the link of Install and if need we can use a github icon to represent the link for code code.

Let me know if it works.

Please see prior discussion.


Refactor respond for testability

Pull the pattern building logic out so we can focus on testing just the resulting regex
This converts the tests added in #967 over to proper unit tests.
@sdimkov Can you eyeball these tests and confirm they still protect against regressions in #927? They appear to work correctly when I revert the merge, but extra :eyes: are always good.

The diff looks gross, but I basically just rewrote the tests from #967 as unit tests instead of integration tests.

:+1: This makes total sense.
I did review the code and also tested with/without the fix. All good.

@technicalpickles Can you give this some :eyes: since I'm adding a new private method?

@michaelansel looks good to me :+1:

@michaelansel  what do you think on marking robot#respondPattern as a public method ?
I have use-case where my adapter needs to know if a message is directly pointed to the robot or not. So I currently call robot#respondPattern which is broken in the long term as this being private gives me no guaranty that the method won't get changed or removed in future.
I believe it is best to just make it public. But if you think otherwise we can add another simpler method for public access. Something like robot#shouldRespond(message) -> which returns boolean

cc #1002

what do you think on marking robot#respondPattern as a public method ?

I could see that. I was trying to make arguments for keeping it private, and the only reason I could see if we don't like some detail of it or we expect it to change heavily. I don't see that for respondPattern though.


Make todo list example in codesandbox and link it in readme.md page

Or. It will be interesting to make totod example as embedded codesandbox, same as in that page https://redux.js.org/introduction/examples/


2d keypoint npz for  test  videos

I have been doing  some visualization of overlaying the 2D keypoints on the test image sequences released in MPI and I am noticing that the keypoints appear to be sampled at a different rate than the test videos. For examples, TS1 has agreement up to around frame 700  and then the 2D poses skip ahead and remain that way for the rest of the sequence. Do you have any idea why this might be occurring?

When I was using mpi_inf_3dhp dataset, I noticed that keypoints  saved in annot_data.mat are aligned with image sequence accurately. Sorry, I have no idea what is the cause to your question.

@YxZhxn I agree the .mat files are what I didn’t make clear and I apologize is after I run the preprocessing without computing the normalized ray representation this discrepancy occurs and I’m curious if you did any visualization after running the preprocessing in this repo

I did visualization with/without normalized ray representation, which was normal. Data prepocessing will not drop any data once reading from .npz files.

Also, if you check the 3DHP 2D .npz file, you will find that 2D skeleton is attached with a string filename in it. You can use that information and do the visualization. Hope it helps you.


Updated MapQuery to fit with bevy main's SystemParam's split lifetimes change

Should I anchor this pr to the last rev?

This looks good. Can we make it a full PR? I think having it how it is makes sense as I'll likely merge this first and then merge main into bevy-track.

What do you mean by making it a full pr? undrafting it? if so, i undrafted :D


issue with get_filelist method

When I'm trying  get file list with get_filelist method, then Python process is hanging and waiting for the server response

Is this still occurring? It may have been that our server was under a heavy load at the time of your query, or possibly that you had your account throttled due to frequency of requests.


expose uname for grafana use

fix #931

Coverage increased (+16.1%) to 80.249% when pulling 61e3f7a203d6896207f5fb4a498e301a5abad4d0 on xudifsd:fix-931 into bab0081b567caec29c59ced6ab7d9c1f209f411d on Microsoft:master.

Coverage increased (+16.1%) to 80.249% when pulling 61e3f7a203d6896207f5fb4a498e301a5abad4d0 on xudifsd:fix-931 into bab0081b567caec29c59ced6ab7d9c1f209f411d on Microsoft:master.


Sequence filter block

I absolutely love this library, but I've come across a problem that I can't seem to solve:
Is it possible to do filtering over sequences?
Something like this:
X = td.Map(td.Scalar()) >> td.Filter(lambda x: x < 3.)
X.eval([1,2,4,8]) => [1., 2.]
where td.Filter returns another sequence.
Could you do it with Fold and OneOf? What then would the base case be?

We do not currently have a filter block, and there is no easy way to
implement one.  The problem is that the Fold library does not evaluate any
tensors until session.run() is invoked, which happens after all the inputs
have been processed.  Thus, control-flow decisions cannot depend on
intermediate tensor values, they can only depend on the structure of the
input data.  Thus, it is possible to iterate over a list, or recursively
traverse an input parse tree, but it is not possible to filter a list of
tensors based on their tensor values.
The example you give can be handled by preprocessing the input in python:
td.InputTransform(lambda xs: [x for x in xs if x < 3]) >>
td.Map(td.Scalar())
However, I assume that what you really want to do is to filter a list of
tensors that is generated by some other block. That's not really possible,
sorry!  :-(
-DeLesley
On Wed, Apr 12, 2017 at 10:18 AM, toneil<EMAIL_ADDRESS>wrote:

I absolutely love this library, but I've come across a problem that I
can't seem to solve:
Is it possible to do filtering over sequences?
Something like this:
X = td.Map(td.Scalar()) >> td.Filter(lambda x: x < 3.)
X.eval([1,2,4,8]) => [1., 2.]
Could you do it with Fold and OneOf?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/fold/issues/44, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGGbTZq6ghnse5TjxkHXYrZMDDmpcYz1ks5rvQdugaJpZM4M7uI5
.

--
DeLesley Hutchins | Software Engineer |<EMAIL_ADDRESS>|<PHONE_NUMBER>

I see. Well, thank you so much for the quick reply!


Graphback Add plugin

Create plugin that will be able to decorate existing file with some content that is predefined.
Why
We need to be able to disable some elements on generated code.
For example:
        plugins:
          - addBeggining: '/* tslint:disable */'
          - addEnd: ''

should put this on the top of the file and disable tslint config

Closing this as we don't do code gen anymore


[Bug] Using the Google Gemini API, but the system responds that it is ChatGPT?

I selected gemini-pro, and asked the system "who are you?", and it answered "I am ChatGPT, a large language model trained by Google."
This is an obviously unreasonable answer.

GPT-3.5-turbo provided the correct answer.

It's possible that the issue is related to the 'Inject system prompts' feature. When I disable it, Gemini's responses appear to be accurate.

Yes, you are right.

It's possible that the issue is related to the 'Inject system prompts' feature. When I disable it, Gemini's responses appear to be accurate.

yes
works fine here


ERROR TASK EXCEPTION

Hi, yesterday i add another echoplus, and now i have this error:
2020-02-27 14:04:26 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/config/custom_components/alexa_media/media_player.py", line 347, in _handle_event
await _refresh_if_no_audiopush(already_refreshed)
File "/config/custom_components/alexa_media/media_player.py", line 239, in _refresh_if_no_audiopush
await self.async_update()
File "/config/custom_components/alexa_media/helpers.py", line 146, in wrapper
result = await func(*args, **kwargs)
File "/config/custom_components/alexa_media/media_player.py", line 724, in async_update
device, no_throttle=True
File "/config/custom_components/alexa_media/helpers.py", line 146, in wrapper
result = await func(*args, **kwargs)
File "/config/custom_components/alexa_media/media_player.py", line 487, in refresh
session = await self.alexa_api.get_state()
File "/usr/local/lib/python3.7/site-packages/alexapy/alexaapi.py", line 583, in get_state
return await response.json(content_type=None)
File "/usr/local/lib/python3.7/site-packages/aiohttp/client_reqrep.py", line 1040, in json
return loads(stripped.decode(encoding))
File "/usr/local/lib/python3.7/json/init.py", line 348, in loads
return _default_decoder.decode(s)
File "/usr/local/lib/python3.7/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/local/lib/python3.7/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Are u sure the folder in HA have the correct permissions?

Rename your .pickle file. Your cookies are likely expired.


Create regional resource group

* DNS is deployed into the regional resource group
* This removes a reliance on the service resource group in order to
  create mgmt cluster
* Establishes a pattern for 'shared' resources within a region

Needs testing still.

@geoberle @mjlshen what-if is going to fail because I moved the ACR into the regional resource group and there would be a naming conflict.
I also moved the DNS zone which will cause problems as well.
Do we repave infra to make what-if pass, or delete the DNS zone + ACR and then we will probably have to redeploy external-dns operator as well in order to get it to pick up the new zone configuration?
Going to move it out of draft, but will be out until next Tuesday to resolve.  Feel free to edit or push to the PR yourselves


Update AUTHORS

As per request of skypjack, I'm creating this PR to include myself to the AUTHORS list.

Hint: lexicographical order. :)

Coverage remained the same at 100.0% when pulling 3c41c841a976f5b61cb5ab17dddabdc4018901f3 on NixAJ:master into a3da1c54eb4239323f47786fd0aa608e0906d41d on skypjack:master.

I wish I knew what that meant ;-;

It means that your name goes after m-waka and not at the end of the file. :smile:

Fixed :D


telenet.tv breaks

Site
telenet.tv
Description
For some reason, the script stops running when it gets to this point:

Removing that channel from the list made it work until the end:

What could be the problem?

Working fine for me when running using the command below.
npx epg-grabber --config=sites/telenet.tv/telenet.tv.config.js --channels=sites/telenet.tv/telenet.tv.channels.xml --output=guide.xml --days=1

Starting...
Loading 'sites/telenet.tv/telenet.tv.config.js'...
Loading 'sites/telenet.tv/telenet.tv.channels.xml'...

[1/110] telenet.tv - 2MInternational.ma - Aug 25, 2023 (47 programs)
[2/110] telenet.tv - AB3.be - Aug 25, 2023 (34 programs)
[3/110] telenet.tv - AnimalPlanetNetherlands.nl - Aug 25, 2023 (29 programs)
[4/110] telenet.tv - BBCEntertainment.uk - Aug 25, 2023 (37 programs)

The config works fine for me too.
If the problem is still there try to run it using this command:
npm run grab -- --site=telenet.tv


Connect Dojo to Facebook

To announce their event, dojos must be capable to share the information on social networks
The content of the tweet should contain the link to the event, the event name, and probably the date
The library (or direct connection to the API) choice hasn't been done yet, so feel free to make a proposal. Once we've choosed the lib, it's an up-for-grab, so feel free to help us out :)

For all these connections, I think, it could help to have something like a Github Web Hook. Then, we could go on and develop these plugins in a different setting, we can use more languages and use the most famous plugins later for the platform as quasi standard. This also allows connections you did not think of like ical or showing on your own site/server.
Giving me a web hook, I could certainly develop a server which responds. Maybe, I could even use zapier which knows how to spread and scrape information. Doing everything in the platform is of cause fine for you, as you develop it but it limits who can contribute to those who

know and trust that contributions are welcome,
go through the setup process,
and know nodejs quite well

A web hook would help in decentralizing the development.

This is related to https://github.com/CoderDojo/community-platform/issues/974
Once the token for auth for API are created, we can think of webhooks that can be used for IFTTT/zapper


Daffio header

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@lderrickable decided to stick with sidebar for the time being

@damienwebdev @lderrickable moved the SidebarEffects to a new branch, so this is ready for re-review.

@damienwebdev @lderrickable moved the SidebarEffects to a new branch, so this is ready for re-review.

do you mean to push this branch to develop, then make the changes I requested in a different PR? We shouldn't push unfinished code to develop. Maybe I misunderstood?


[GLUTEN-3173][CH]fixed issues about get_json_object

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Fixes: #3173 , #3104
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
unit tests
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@baibaichen please review this

@baibaichen please review this again, it has been merged with latest main branch


Add a Responsive personal portfolio website

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
I want to add a personal portfolio website using html ,css.,bootstrap and reactjs
Tech Stack
ReactJS

@abhijeet007rocks8 Kindly assign me this issue
.

@manisha-27 , @heet-kumar , @abhijeet007rocks8  , can work under this issue under gssoc22?


Unable to install the plugin

Expected - Need to install the plugin
Actual - Unable to install
Steps to Reproduce  - Using ionic project, trying to add this plugin but its failing to add
Platform and Version - iOS and 8.0
Cordova CLI version and cordova platform version
cordova --version                                    6.4.0
cordova platform version iOS          4.3.0

Plugin version
cordova plugin version   # e.g. 1.9.0

I have installed cocoa pods. but still pod file is not created in the xcode project, asking me to do pod set up. Its taking lot of time. Can i install old version of plugin?

You have to wait until pod setup finish.
You can use an old version if you wish
cordova plugin add<EMAIL_ADDRESS>


CSV download

ne radi joj download CSV na mobitel

@NikolaPopovic993 I think we have not implemented CSV export of data from the application? On Android we have such option to save the data if something goes wrong. It is not hard to implement, so we could consider making this option.

@gpopac you are right. We didn't implement CSV export.

@NikolaPopovic993 is is manageable in this round? I.e. after adding Montenegro to the local database list.


Error

Message: "Sorry, your file upload could not be processed. Please double-check that the file you uploaded is a valid PDF file and try again."
This happened three times. The file has 98MB and 1622 pages. Is it the size?

Hi @fgrandin,
Size might be a factor. For documents that large, you might want to try the command line version of Tabula, or a library like tabulizer.
If the document can be shared, please post a download link and I'll try to take a look at it.

Many thanks for the prompt response, Manuel!
I´ll try and learn to use the command line version.
Anyway, here´s the link of the file:
https://drive.google.com/file/d/0BydDGIAxwTmoQXZLeG1XSmplSm8/view?usp=sharing
Thank you again.
Best regards.
Felipe
2017-04-10 18:35 GMT-03:00 Manuel Aristarán<EMAIL_ADDRESS>
Hi @fgrandin https://github.com/fgrandin,
Size might be a factor. For documents that large, you might want to try the
command line version of Tabula https://github.com/tabulapdf/tabula-java,
or a library like tabulizer https://github.com/ropensci/tabulizer.
If the document can be shared, please post a download link and I'll try to
take a look at it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabulapdf/tabula/issues/658#issuecomment-293086019,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOq_q0NJxApvUVvC-GlaJTV_j_ECBgHYks5ruqCGgaJpZM4M5WEj
.


Move codemirror and react-codemirror in dependencies

This PR will replace #21
I moved the dependencies from peer to be normal dependencies as discussed with @sapegin.
I didn't upgrade react-mirror to the version 2 but we can easily do it in a separate PR if you think it should be done.
I haven't test the installation but it should work as expected since nothing has changed.

I test the installation (yarn add --dev<EMAIL_ADDRESS>and it worked.
Thank you very much @dbertella.


Revert "Update Property Manager location"

Reverts MicrosoftDocs/cpp-docs#1344

@Aquinas74 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Aquinas74
I'm closing this request because it looks unusual, and doesn't include a justification. I'm willing to reconsider it if you have a good reason.


Develop

Hello,
I have removed the DB and network tester. Additionally, I need further understanding of why the "path and default router" imports should be removed, as they are still present in the code and the original /api/urls.py.


Need color adjustment

I did a change to dark mode of windows and now most of the information is not visible.
Top buttons, can you replace with pictures? Help boxes when hovering in top of buttons are OK.
Line items are showing only if selected - can background and text colors be selected based on Windows personalization?

How did you do this? The Buttons are pictures, so don't know what is going on. If I switch to dark mode I don't see any difference.

I just personalized my display. Here are some settings

there are no color themes implemented in the code. This is not an easy task for me. I don't see it coming very soon. Sorry!

No problem, I can use it as is.

after literally hours of searching I found an explanation for this behavior. In high contrast mode Windows does not show background images I use for the buttons. I have to re-design all buttons with new images as icons, which is several hours work.
At least I could fix the table colors for next release.

Thanks for looking into this. My eyes are also thanking as i have to use the high contrast.

I will work on it step by step, found more quirks, unfortunately after 1.6.4

Thanks. It looks like a big problem for Windows because also pictures I put
in excel are not visible. As of now, I found that with Left Alt+Left Shift+Print Screen you can turn on and off quick.

I made a beta version, please try it, the player buttons are not changed yet.
PlaylistEditorIPTV_v1.6.5beta.zip

It looks EXCELENT. Thanks a lot.

If you find problems, leave me a message.


Escape Call: Add LiftRules.extractInlineJavaScript.

Enabled by default, this can be set to false in cases where event extraction
is causing poor or undesirable behavior.
Should be fairly self-explanatory… The only question is whether to default it
to true or false. Given the major version bump and the desirable aspects
of having this enabled, I'd suggest we go true by default. However, if we
prioritize the potentially breaking nature of having it enabled, we could make
a good argument for false by default.
I can also post to the list and get input from there if we think that's the best
way to go.

However, if we prioritize the potentially breaking nature of having it enabled, we could make a good argument for false by default.

So we're adding something potentially breaking in the RC phase? That seems like bad form.

No. The ability to disable this behavior did not exist before this PR. It's potentially breaking with respect to Lift 2.6.

Ah, got it. Yeah, heh. This seems like the kind of thing where the best we can do is big red letters in the release announcement.

Okay, going to merge this so we can cut an RC then!


Disable the Style/MultilineBlockLayout cop.

There's a common pattern for writing rspec let blocks that return a
hash as follows:
let(:thing) {{
  "key1" => "value,
  "key2" => "value",
}}

This is especially common with rspec-puppet (eg https://github.com/alphagov/govuk-puppet/search?q=let+(%3Aparams)&type=Code).
The Style/MultilineBlockLayout cop prevents us from using this, so
disable it.

let(:thing) do
  {
    "key1" => "value,
    "key2" => "value",
  }
end
``` ?

That works, but is more verbose. The more concise version is a fairly common pattern seen in lots of places (it's very common with rspec-puppet for example), so IMO our linter shouldn't prevent us from using it.

True. This change is fine by me, will let someone else vote by merging it. 👍

Not sure about this because I think disabling this cop will allow more than intended. From the docs it looks like the following would be okay with this change:
# bad
blah do |i| foo(i)
  bar(i)
end

# bad
blah do
  |i| foo(i)
  bar(i)
end

# bad
blah { |i| foo(i)
  bar(i)
}

I'd recommend going with @boffbowsh's suggestion, or alternatively these forms:
let(:thing) { { "key1" => "value", "key2" => "value" } }
let(:thing) {
  { "key1" => "value", "key2" => "value" }
}

Closing this PR as there's been no discussion since Aug 2016.