[SPARK-48480][SS][CONNECT] StreamingQueryListener should not be affected by spark.interrupt()

What changes were proposed in this pull request?

This PR implements a small architecture change for the server side listenerBusListener. Before, when the first addListener call reaches to the server, there is a thread created, and there is a latch to hold this thread long running. This is to prevent this thread from returning, which would send a ResultComplete to the client, and closes the client receiving iterator. In client side listener we need to keep the iterator open all the time (until the last removeListener call) to keep receiving events.
In this PR, we delegate the sending of the final ResultComplete to the listener thread itself. Now the thread doesn't need to be held stuck. This would 1. remove a hanging thread running on the server and 2. Shield the listener from being effected by spark.interruptAll.
spark.interruptAll interrupts all spark connect threads. So before this change, the listener long-running thread is also interrupted, therefore would be affected by it and stop sending back events. Now the long-running thread is closed, so it won't be affected.
Why are the changes needed?

Spark Connect improvement.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Added unit test
Was this patch authored or co-authored using generative AI tooling?

No

@HyukjinKwon @juliuszsompolski  Finally find a time to finish it, PTAL : D

Merged to master.


Place virtual added mass at link origin

is the goal here to keep the fluid_added_mass inertial properties distinct from the rigid body inertial properties for introspection purposes? It sounds from the title of the PR that this changes the effective 3D point where the inertial parameters are expressed


Fix missing bookmark

Docs Build status updates of commit 0037ad9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/exchange-web-services/ews-client-design-overview-for-exchange.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Should this be deleted? marts/revenue_metric.sql

This code looks like the boilerplate code here: https://docs.getdbt.com/docs/building-a-dbt-project/metrics#developing-metrics-with-metricsdevelop
Thinking it was accidentally left in the project and needs to be deleted?  Since there's an actual revenue metric at marts/revenue.yml
https://github.com/dbt-labs/jaffle_shop_metrics/blob/3c9c1cad8c2dcf4437a08285c0d6d1e190301f5a/models/marts/revenue_metric.sql#L1-L22

See ^ #7


Addition of File Manager

Fixes (Part of) #782
This PR is ready for review.
Risk
This PR makes minor API changes.
Testing Plan
Unit tests in this PR should cover all new code.
Summary

Uncomment FileManager sections in SdlManager
Implements agreed upon API for FileManager
Implements SdlFile, SdlArtwork

CLA

[ X ] I have signed the CLA

Codecov Report

Merging #816 into feature/issue_782_Android_Managers will decrease coverage by 0.26%.
The diff coverage is 82.67%.

@@                           Coverage Diff                           @@
##             feature/issue_782_Android_Managers    #816      +/-   ##
=======================================================================
- Coverage                                 44.27%     44%   -0.27%
+ Complexity                                 3205    3015     -190
=======================================================================
  Files                                       373     375       +2
  Lines                                     18011   17646     -365
  Branches                                   1755    1770      +15
=======================================================================
- Hits                                       7975    7766     -209
+ Misses                                     9705    9504     -201
- Partials                                    331     376      +45

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/com/smartdevicelink/api/SdlManager.java
43.6% <100%> (+5.91%)
16 <0> (+3)
:arrow_up:

.../main/java/com/smartdevicelink/api/SdlArtwork.java
100% <100%> (ø)
5 <5> (?)

...src/main/java/com/smartdevicelink/api/SdlFile.java
100% <100%> (ø)
13 <13> (?)

...main/java/com/smartdevicelink/api/FileManager.java
78% <78%> (ø)
21 <21> (?)

...a/com/smartdevicelink/proxy/rpc/OnVehicleData.java
94.94% <0%> (-0.71%)
57% <0%> (-59%)

...celink/proxy/rpc/SubscribeVehicleDataResponse.java
96.73% <0%> (-0.02%)
57% <0%> (-29%)

...link/proxy/rpc/UnsubscribeVehicleDataResponse.java
96.73% <0%> (-0.02%)
57% <0%> (-29%)

...artdevicelink/proxy/rpc/enums/VehicleDataType.java
100% <0%> (ø)
2% <0%> (-1%)
:arrow_down:

...ink/proxy/rpc/enums/ElectronicParkBrakeStatus.java

... and 21 more

Continue to review full report at Codecov.

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

@askirk SdlFile and SdlArtwork should probably go in the same package as the other structs and have tests like the other structs

Please pull in the feature/issue_782_Android_Managers to resolve conflicts


WithPermission around EditButton removes {id} in its url and makes it undefined

This was an issue in aor-permissions that was automatically closed :(
https://github.com/marmelab/aor-permissions/issues/31
demonstrated in CategoryList if you navigate to: http://localhost:3000/#/categories and you try to edit one of the categories in the list. Here is the repo with the commit causing it: https://github.com/zifnab87/admin-on-rest-demo
Check the address at the bottom when I hover:

Thank you!

As specified in the Authorization documentation, that's because:

Do not use the WithPermission component inside the others admin-on-rest components. It is only meant to be used in custom pages or components.

In all admin-on-rest components, you must use the function as a child pattern as explained in the Restricting Access To Fields And Inputs.

@djhi Thanks! My bad


yt-dlp api Options [bug] I get incompleted video with unmerged video fragments

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2023.09.24 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Hello,
I did find a bug with yt-dlp api options.
See I use VPS with a mounted disk(Buyvm RYZEN KVM 1GB and I have extra Block Storage Slab - 256GB mounted) I use this repo mirror-leech-telegram-bot which uses yt-dlp api options to execute cmds.
When I use yt-dlp from bot(I execute /yl) to get download/upload youtube video I get an incompleted uploaded video when I this YT-DLP api Options!
The video file gets incompleted uploaded and unmerged!
Example frag.16.part and incompleted.youtube.video.mkv
I get in total two different files uploaded to Telegram! LOG.txt

Used cmd:
/yl -opt verbose:True|concurrent_fragment_downloads:^9|format:bv*[height<=1080][ext=mp4]+ba[ext=m4a]/bv*|fragment_retries:^inf|final_ext:mkv|writesubtitles:True|subtitleslangs:['tur.*', 'en.*', '.*-orig']|writeautomaticsub:True|postprocessors:[{'key': 'FFmpegVideoRemuxer', 'preferedformat': 'mkv'}, {'format': 'ass', 'key': 'FFmpegSubtitlesConvertor', 'when': 'before_dl'}, {'key': 'FFmpegEmbedSubtitle', 'already_have_subtitle': False}, {'api': 'https://sponsor.ajay.app', 'categories': {'chapter', 'filler', 'interaction', 'intro', 'music_offtopic', 'outro', 'poi_highlight', 'preview', 'selfpromo', 'sponsor'}, 'key': 'SponsorBlock', 'when': 'after_filter'}, {'force_keyframes': False, 'key': 'ModifyChapters', 'remove_chapters_patterns': [], 'remove_ranges': [], 'remove_sponsor_segments': set(), 'sponsorblock_chapter_title': '[SponsorBlock]: ' '%(category_names)l'}, {'add_chapters': True, 'add_infojson': 'if_exists', 'add_metadata': True, 'key': 'FFmpegMetadata'}]

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output

[debug] Command-line config: ['-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8 (No ANSI), error utf-8 (No ANSI), screen utf-8 (No ANSI)
[debug] yt-dlp version<EMAIL_ADDRESS>[088add956] (pip)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-5.4.0-135-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, sqlite3-3.37.2, websockets-11.0.3, xattr-0.10.1
[debug] Proxy map: {}
[debug] Loaded 1886 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest

Reference:
https://github.com/anasty17/mirror-leech-telegram-bot/issues/1562

This incomplete download error happens with youtube videos!
So I have two VPS both are from Buyvm RYZEN KVM 1GB
but one VPS does have extra Block Storage Slab - 256GB disk mounted and this error always occurs with the mounted Storage vps!!!
I don't know why? Any help is appreciate please?

copy the verbose output from running your actual command/script


ReadBytesAsPointer error in std::collections::BTreeSet

Consider the following program:
// debug_miri.rs

#[derive(PartialEq, Eq, PartialOrd, Ord)]
pub enum Foo {
    A(&'static str),
    _B,
    _C,
}

pub fn main() {
    let mut b = std::collections::BTreeSet::new();
    b.insert(Foo::A("\'"));
    b.insert(Foo::A("/="));
    b.insert(Foo::A("#"));
    b.insert(Foo::A("0o"));
}

When I compile this program with rustc, the resulting executable runs and exits successfully. When I run the executable under valgrind, no errors are reported. When I run it with miri, I expect it to succeed, but it does not. Instead, it reports a ReadBytesAsPointer error:
$ RUST_BACKTRACE=1 cargo run --bin miri -- debug_miri.rs
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
     Running `target/debug/miri debug_miri.rs`
ERROR:rustc_miri::interpret::eval_context:

An error occurred in miri:
2: rustc_miri::interpret::error::{{impl}}::from
        at src/librustc_mir/interpret/error.rs:24
3: core::convert::{{impl}}::into<rustc_miri::interpret::error::EvalErrorKind,rustc_miri::interpret::error::EvalError>
        at /checkout/src/libcore/convert.rs:398
4: rustc_miri::interpret::value::{{impl}}::to_ptr
        at src/librustc_mir/interpret/value.rs:278
5: rustc_miri::interpret::value::{{impl}}::to_ptr
        at src/librustc_mir/interpret/value.rs:56
6: rustc_miri::interpret::memory::{{impl}}::read_bytes<miri::Evaluator>
        at /home/dwrensha/src/miri/src/librustc_mir/interpret/memory.rs:1146
7: miri::fn_call::{{impl}}::call_c_abi
        at miri/fn_call.rs:207
8: miri::fn_call::{{impl}}::call_missing_fn
        at miri/fn_call.rs:538
9: miri::fn_call::{{impl}}::eval_fn_call
        at miri/fn_call.rs:63
10: miri::{{impl}}::eval_fn_call
        at miri/lib.rs:188
11: rustc_miri::interpret::terminator::{{impl}}::eval_fn_call<miri::Evaluator>
        at /home/dwrensha/src/miri/src/librustc_mir/interpret/terminator/mod.rs:300
12: rustc_miri::interpret::terminator::{{impl}}::eval_terminator<miri::Evaluator>
        at /home/dwrensha/src/miri/src/librustc_mir/interpret/terminator/mod.rs:101
13: rustc_miri::interpret::step::{{impl}}::terminator<miri::Evaluator>
        at /home/dwrensha/src/miri/src/librustc_mir/interpret/step.rs:200
14: rustc_miri::interpret::step::{{impl}}::step<miri::Evaluator>
        at /home/dwrensha/src/miri/src/librustc_mir/interpret/step.rs:85
15: miri::eval_main::run_main
        at miri/lib.rs:123
16: miri::eval_main
        at miri/lib.rs:136

error: a memory access tried to interpret some bytes as a pointer
     |
note: inside call to <[u8] as core::slice::SliceOrd<u8>>::compare
note: inside call to core::slice::<impl std::cmp::Ord for [T]><u8>::cmp
note: inside call to core::str::traits::<impl std::cmp::Ord for str>::cmp
note: inside call to std::cmp::impls::<impl std::cmp::Ord for &'a A><str>::cmp
    --> debug_miri.rs:5:7
     |
5    |     A(&'static str),
     |       ^^^^^^^^^^^^^
note: inside call to <Foo as std::cmp::Ord>::cmp
note: inside call to alloc::btree::search::search_linear::<alloc::btree::node::marker::Mut, Foo, (), alloc::btree::node::marker::LeafOrInternal, Foo>
note: inside call to alloc::btree::search::search_node::<alloc::btree::node::marker::Mut, Foo, (), alloc::btree::node::marker::LeafOrInternal, Foo>
note: inside call to alloc::btree::search::search_tree::<alloc::btree::node::marker::Mut, Foo, (), Foo>
note: inside call to <std::collections::BTreeMap<K, V>><Foo, ()>::entry
note: inside call to <std::collections::BTreeMap<K, V>><Foo, ()>::insert
note: inside call to <std::collections::BTreeSet<T>><Foo>::insert
    --> debug_miri.rs:15:5
     |
15   |     b.insert(Foo::A("0o"));
     |     ^^^^^^^^^^^^^^^^^^^^^^
note: inside call to main
    --> debug_miri.rs:10:1
     |
10   | / pub fn main() {
11   | |     let mut b = std::collections::BTreeSet::new();
12   | |     b.insert(Foo::A("\'"));
13   | |     b.insert(Foo::A("/="));
14   | |     b.insert(Foo::A("#"));
15   | |     b.insert(Foo::A("0o"));
16   | | }
     | |_^

error: aborting due to previous error

thread 'main' panicked at 'Box<Any>', /checkout/src/librustc_errors/lib.rs:529:8
stack backtrace:
   0: std::sys::imp::backtrace::tracing::imp::unwind_backtrace
             at /checkout/src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::_print
             at /checkout/src/libstd/sys_common/backtrace.rs:71
   2: std::panicking::default_hook::{{closure}}
             at /checkout/src/libstd/sys_common/backtrace.rs:60
             at /checkout/src/libstd/panicking.rs:381
   3: std::panicking::default_hook
             at /checkout/src/libstd/panicking.rs:397
   4: std::panicking::rust_panic_with_hook
             at /checkout/src/libstd/panicking.rs:611
   5: std::panicking::begin_panic
   6: rustc_errors::Handler::abort_if_errors
   7: miri::after_analysis
             at miri/bin/miri.rs:139
   8: core::ops::function::Fn::call
             at /checkout/src/libcore/ops/function.rs:73
   9: rustc_driver::driver::compile_input::{{closure}}
  10: rustc::ty::context::TyCtxt::create_and_enter
  11: rustc_driver::driver::compile_input
  12: rustc_driver::run_compiler
  13: miri::main
             at miri/bin/miri.rs:258
  14: __rust_maybe_catch_panic
             at /checkout/src/libpanic_unwind/lib.rs:98
  15: std::rt::lang_start
             at /checkout/src/libstd/panicking.rs:459
             at /checkout/src/libstd/panic.rs:361
             at /checkout/src/libstd/rt.rs:61
  16: main
  17: __libc_start_main
  18: _start

It is a known problem that miri's handling of pointers is incomplete and cannot handle everything that is safe in Rust. (Some of the things that are safe IMHO shouldn't be because this is not just a superficial problem, but that ship has sailed.)
See https://github.com/solson/miri/issues/224 and in particular https://github.com/solson/miri/issues/224#issuecomment-320289999 for some more information about this. I had hoped to write the first of two blog posts on this by now, but that didn't happen -- but I still plan to come back to this, after my vacation.

That said, this incompleteness usually results in the "opposite" error (cannot turn pointer into bytes). This one looks different. Not sure what is going on.

The error gets thrown when memcmp(left, right, n) gets called with right equal to ByVal(Bytes(0)). To me, this seems like a symptom of corrupted memory. My wild guess is that there are some stray writes in glue code somewhere.

I've reduced to a much smaller test case, updating my first comment above.

I think I found the bug. Memory::copy doesn't copy relocations correctly if the copy memory is overlapping.
The issue is that we erase the relocations while we copy bytes. So when we try to copy the relocations in the next step, they are already gone in the overlapping memory.

Awesome!


Project Presentation Materials (5%)

[ ] Slide deck, as a PDF.
[ ] A link to the video of your presentation (when it becomes available).

[ ] Slide deck, as a PDF.
[ ] A link to the video of your presentation (when it becomes available).

I created a background/logo, input a few slides, and modified template


Add Line Support for Bar Chart like a Combo Chart

Hi, this is my first time creating an issue/feature request so kindly bear with me. 😅
Context
We have a feature in our company project that wants to have a stackable bar chart along with a line but unfortunately, current BarChart does not have yet support to render simple lines like how LineChart does them.
Kinda something like this:

It would be wonderful if we could have an added feature/enhancement for that.
So I tried to forked the project and implemented a simple line like how the LineChart does but I don't know if it is by far the best solution. Tho local testing seems it worked, I just need to add unit testing for that.
Suggested Solution
What I did was the ff:

move the barPaint instance from the LineChartPaint class to AxisChartPaint class as well as the methods needed to draw the lines. (i.e. drawBarLine, generateBarPath, etc. )
move the LineChartBarData from LineChartData to AxisChartData to be able to use globally.
then call the drawLine method in BarChart for every lineBarsData

Here is the link for the forked sample branch: Line Chart support for Bar Chart
Expected Result
Expected result usage could be like this, an added property in the BarChart for LineChartBarData list to render some simple lines. Ideally it would be awesome if it behaves like how it is in the line chart with the added area bars and touch interaction but for now we just need this.
return BarChart(
      BarChartData(
        ...
        lineBarsData: [
             LineChartBarData(
              spots: _generateSpots(chartBarData.dataPoints),
              barWidth: chartBarData.barWidth,
              color: chartBarData.solidColor,
              isCurved: chartBarData.isCurved,
              preventCurveOverShooting: chartBarData.preventCurveOverShooting,
              dotData: FlDotData(show: false),
              isStepLineChart: chartBarData.isStepLineChart,
            ),
          )
        ],
      ),

      /// set to: no animation
      swapAnimationDuration: Duration(),
    );

Looking forward for your response and great works as always for creating this awesome package. 💪

Hi. Thanks for creating this issue and sharing your experience.
We already have an open issue. You can follow it here: #764. Don't forget to thumb it up.
Also, I appreciate it if you write your experience there.


src/components/form/_form.scss has unnecessary margin

This CSS in /src/components/form/_form.scss;
.bx--form-item {
font-family: "ibm-plex-sans", Helvetica Neue, Arial, sans-serif;
flex: 1;
margin-right: 1rem;
margin-bottom: 1.5rem; }
.bx--form-item:last-of-type {
margin-right: 0;
margin-bottom: 0; }
Seems unnecessary and is actually counter-productive. I have to remove the margin, is there any use case which actually needs this CSS?

These margins were added to ensure there is correct spacing between form elements. You should be able to remove the class where you don't need the spacing or override the css as needed.


Personal openfarm.cc instance

Hi
For localisation purposes, I will have to create my own instance of openfarm.cc to localise the drop details.
I guess I would have to build my own version of the farmbot web app as well, is there a setting I can change to make it connect to my own openfarm.cc?

@tuananh2311 You bring up a good point, and it's something that we have not yet started.
I will keep this issue open for others who have a similar issue. Details to follow.

Studying the issue myself., ... +1 on the need ...
Will follow this one closely.
Regards.

Hi,
in the need of a local instance too, I've created a workaround to use an http ip address:
from the web app folder:
egrep -lRZ 'https://openfarm.cc'. | xargs -0 sed -i.bak -e 's/https:\/\/openfarm.cc/http:\/\/RYOU_IP_ADDRESS/'
It's hardcoded but it works :o)
(An alternative would be to add a fake name  record entry)
If someone can tell me how to make .env file entries available (or a documentation) in the typescript code portion of the webapp, I can modify it

@nicmel Glad you were able to make it work. As a side note, when you want to upgrade your versions of the FarmBot API, make sure you run git stash before the upgrade. You will need to run the egrep ... command again locally after the upgrade.

@RickCarlino is there a good place to start  to learn how to use env variables within the web code? (typescript mostly)?

@nicmel
That's a good question and it probably belongs in the developer docs as well.
The short answer is, GlobalConfig.create!(key: "FOO", value: "BAR") from within the Rails console.
The long answer is stated below. I'm happy to answer any additional questions you have. This document will be published into the developer docs to help other developers with the same question.
Why?
FarmBot is an Open Source project that can be self hosted by third party developers. The way that FarmBot, inc. configures its servers for public use might not meet the needs of self hosting users. There are also security implications to configuration management in an Open Source project- we would not want to leak our database password into source control, for example.
Farmbot attempts to decouple configuration from code as much as possible, in keeping with the 12 Factor Methodology:

The twelve-factor app stores config in environment variables (often shortened to env vars or env). Env vars are easy to change between deploys without changing any code; unlike config files, there is little chance of them being checked into the code repo accidentally; and unlike custom config files, or other config mechanisms such as Java System Properties, they are a language- and OS-agnostic standard.

-- 12 Factor Methodology Homepage
By using ENV vars as the first choice for server configuration, we can:

Allow for server customization
Safely share the application source code with the public
Reap all the other benefits that the 12 Factor Methodology provides.

How?
For most parts of the app, environment variables are loaded into a .env file.

Information about the .env file format can be found here
Documentation on legal values can be found here.

Important Note: The app also exposes a secondary configuration system called GlobalConfig. This secondary system addresses use cases where ENV variables are not appropriate. The system is discussed later in this document.
Real World Examples
FarmBot, Inc. requires validation of user email addresses. Most self hosting users do not want this behavior, however. To allow server admins to choose between verifying emails or not, the application exposes a ENV["NO_EMAILS"] variable. When present, this ENV var will disable email verification.
Another example is the EXTRA_DOMAINS variable. It can be set to a comma separated list of alternative domain names that a server may control. This allows the same server to be hosted on more than one domain name (my.farm.bot, my.farmbot.io, etc..).
These examples are for illustrative purposes. The full list of variables is available here.
Problems
ENV vars work for most configuration use cases, but there are some caveats:

Changing ENV vars requires a server restart. In many cases, this creates an unacceptable downtime period, particularly for servers with high uptime requirements.
Environment variables are a server-side concern. When a client loads the FarmBot Web App in their browser, it is not possible to see the server's ENV vars. If this were not the case, it would be possible for users to steal database credentials quite easily.
Not all customization happens on the server side ("back end"). There are many use cases that require ENV passing information from the server to the client.

Solutions
Solution: Runtime Reloading of ENV Vars
As stated previously, ENV variables require a process restart to take effect. For some values, a restart is not practical.
For values that must change after the server has started, the FarmBot Web App uses a model (database table) known as GlobalConfig. The GlobalConfig source code can be found here.
Some GlobalConfig values (such as "TOS_URL") are bootstraped using ENV vars if no value can be found in the global_configs table.
A GlobalConfigs value can be changed at runtime from the Rails console:
GlobalConfig.create!(key: "FAVORITE_VEGGIE", value: "Carrots")

WARNING: Unlike traditional ENV vars, GlobalConfig values are not hidden from the public. Do not store sensitive data in GlobalConfig!
Solution: Sharing ENV Vars with Clients
NOTE: This section requires knowledge from the previous section.
As stated in the "Problems" section, exposing all ENV vars to a browser would be an extreme security risk. We want to expose some ENV vars to clients, but not all of them.
To get around this problem the Web App's UI exposes a global variable to the browser: window.globalConfig. This variable contains all GlobalConfig items, but not necessarily all ENV vars. For this reason it is important that you do not store sensitive data in the GlobalConfig table.
We can inspect the value of globalConfig in our browser:
console.log(window.globalConfig);

On production, globalConfig contains the following values:
{
  // Current application environment:
  "NODE_ENV":"production",

  // Terms of Service URL:
  "TOS_URL":"https://farm.bot/tos/",

  // Privacy Policy URL:
  "PRIV_URL":"https://farm.bot/privacy/",

  // Lowest FarmBot OS versions the server supports:
  "MINIMUM_FBOS_VERSION":"7.0.0",
  "FBOS_END_OF_LIFE_VERSION":"7.0.1",

  // Web App version the server runs:
  "LONG_REVISION":"191b0ead3dfe1cc43ee416a5ff27a064af556192",
  "SHORT_REVISION":"191b0ead"
}

As stated, the server's environment variables are not exposed to the public. We also mentioned that we had a second configuration system that complements traditional server ENV vars- the GlobalConfig database table.
GlobalConfig values are transferred from the server to the client using server-side templating.
When the Web App loads a web page that needs ENV vars, it injects a a <script> tag that declares the globalConfig variable.
app/views/dashboard/_common_assets.html.erb:
<script>
  // ---SNIP!---

  // THIS IS WHERE ENV VARIABLES CROSS OVER
  // From the server to the client:

  window.globalConfig = <%= raw(@global_config) %>

  // The @global_config contains all database entries
  // for the `GlobalConfig` table (but not all ENV
  // vars).

  // ---SNIP!---

</script>

Please note that the <%= %> is Ruby ERB syntax
The value of @global_config is set in app/controllers/dashboard_controller.rb:
class DashboardController < ApplicationController
  before_action :set_global_config
  layout "dashboard"

# --- SNIP! ---

private

  def set_global_config
    # THIS IS WHERE @global_config is created:
    @global_config = GlobalConfig.dump.to_json
  end
end

With these systems in place, we can now add and remove ENV variables from the Rails console via:
GlobalConfig.create!(key: "FAVORITE_PLANT", value: "cabbage")
=> #<GlobalConfig:0x0000 key: "FAVORITE_PLANT", value: "cabbage">

To view the variable from the browser, we can simply type:
globalConfig.FAVORITE_PLANT;
// => "cabbage"

Hi,
in the need of a local instance too, I've created a workaround to use an http ip address:
from the web app folder:
egrep -lRZ 'https://openfarm.cc'. | xargs -0 sed -i.bak -e 's/https:\/\/openfarm.cc/http:\/\/RYOU_IP_ADDRESS/'
It's hardcoded but it works :o)
(An alternative would be to add a fake name record entry)
If someone can tell me how to make .env file entries available (or a documentation) in the typescript code portion of the webapp, I can modify it

Hi
I run your command on the web app folder (after putting in my openfarm IP Address) and then restarted the app.
I tried to search for a crop I curated on my openfarm instance but nothing returned. The links to create new crop are all updated and working though.
egrep -lRZ 'https://openfarm.cc'. | xargs -0 sed -i.bak -e 's/https:\/\/openfarm.cc/http:\/\/IP:PORT/'

Did you try to create a new crop and check if it's created?
Search the code for any https://openfarm.cc still left (not on the .bak files of course)

Did you try to create a new crop and check if it's created?
Search the code for any https://openfarm.cc still left (not on the .bak files of course)

Thanks for the suggestion, I got it working.


Fixes and improvements to history views

Allow refreshing while requests are still happening by
using a list of dict.items()
Fixed issue with history_refresh, it now returns SignedDataForm
to allow Switch requests to work after refresh.
Clean up old rows in FE from that have now expired when refresh
happens.
Fixed history_sidebar view to not raise 500 error when no toolbar
is found for an expired store_id.
Old rows that have now expired, display [EXPIRED] next to the
Switch text.

Fixes https://github.com/jazzband/django-debug-toolbar/issues/1483

I think this looks good. I'm not sure about the change from HistoryStoreForm to SignedDataForm. Shouldn't it be possible to remove the former now?

I'm a bit swamped this week, but I will try to review this on Thu/Fri.

@tim-schilling Thanks for your review.
With the above changes the switch URLs(even after refresh) look like this:

If we use HistoryStoreForm, the switch buttons loaded by the initial render of History have similar signed URLs, but after hitting refresh we end up with 400 errors for any switch: http://localhost:8000/__debug__/history_sidebar/?store_id=3be1d8503a0b46999cb4cb54efc1f2a1 as this results in invalid signed_form in signed_data_view decorator: {'signed': ['This field is required.']}
Dict mutation test: It's quite tricky to prove in a test. But here's an attempt in my fork https://github.com/ashwch/django-debug-toolbar/tree/dict_mutation_test: https://github.com/ashwch/django-debug-toolbar/blob/c0454b6c20e09b0db8d1d3eae67429cd7eb07bb3/tests/panels/test_history.py#L138.
The sleep time of 0.002 could be different based on the system, hence I personally don't like the test much.
make test TEST_ARGS='tests.panels.test_history.HistoryViewsTestCase.test_parallel_history_refresh'

Thanks, both of you! I think this looks good. Let's merge this.


[C to host error: returning null] Error: QuickJSContext had no callback with id -xxxx

When repeating calling a function, an error occurs from quickjs-emscripten.
import { getQuickJS } from "quickjs-emscripten";
import { expect, test } from "vitest";
import { Arena } from ".";

test(
  "repeated function",
  async () => {
    const rt = (await getQuickJS()).newRuntime();
    const ctx = rt.newContext();
    const arena = new Arena(ctx, { isMarshalable: true });

    vm.expose({ hoge: () => {} });
    // error happens from 3926 times on my machine
    for (let i = 0; i < 3926; i++) {
      // arg should be an object
      arena.evalCode(`hoge({})`);
    }

    arena.dispose();
    ctx.dispose();
    rt.dispose();
  },
  1000 * 60,
);

reported at https://github.com/justjake/quickjs-emscripten/issues/98

This issue is already fixed. I forgot closing it


ic-push-url with custom location

Hi,
as stated in https://intercoolerjs.org/attributes/ic-push-url.html ic-push-url takes the location of the ajax-request and pushes it into the location bar.
In some cases the URL of the ajax request does not represent the whole page, but only a specific element on the page. Is there any way to push either a custom URL or the URL of the href-attribute, which represents the complete page, to the location bar?
Best
Chris

Hi Bart,
It isn't perfect, but you can use the X-IC-PushURL response header in that case to push a new location.
ic-push-url should probably take either "true" of the actual URL you wish to push as a potential value.  Any interest in taking a crack at that?  :)

Hi Carson,
i will take a look, if i find the time and if this does not exceed my capacity. It would be a very useful feature, so i would be happy if somebody implements this. Thanks for Intercooler btw. ;)

No prob.  I'll close out the issue.


show progress in MultiNest feedback

In light of #78, I have tried to make the posterior sampling phase of Lensed more informative.
To get a better idea of how far along the algorithm is, I approximate the tol stopping criterion of MultiNest and calculate after how many samples it is going to be fulfilled, assuming no higher maximum likelihood will be found.
And to check whether or not the algorithm hit a roadblock, the time and number of samples of the last update are displayed.
This is what the output looks like:
find posterior

 *****************************************************
 MultiNest v3.9
 Copyright Farhan Feroz & Mike Hobson
 Release Oct 2014

 no. of live points =  100
 dimensionality =   12
 running in constant efficiency mode
 *****************************************************
  15:32:41 [#############       ] 65% | S: 1100

cc @fabiobg83 @rbmetcalf Let me know what you think. Maybe some other data would be more useful to you?


[Registration] 500 Internal Server Error appears during trying to 'Sign In' into the application

Date Time
25-05-2021 13:40
Priority
High
Environment:
OS: [Windows 10]
Browser [Google Chrome]
Version [ 90.0.4430.212]
Reproducible: always
Build found: 20210520.1
Pre-conditions
A page https://eventsexpress-test.azurewebsites.net/home/events is opened. A 'Sign in/up' pop-up is focused
Steps to Reproduce
Steps to reproduce the behavior:

Switch to the 'Log in' tab
Enter an e-mail address of an already registered user into the 'E-mail' field<EMAIL_ADDRESS>Enter a password of the already registered user into a 'Password' field (e.g. qwerty2001)
Right-click and select 'Перевірити' from the drop-down list --> Click on 'Network' tab
Click on the 'Sign in' button
Look at the errors with red color in the 'Network' tab and 'Console' tab

Actual result
The errors with red color in 'Network' tab ("550 error")  and in 'Console' tab ("SyntaxError: Unexpected end of JSON input")
Expected result
The user can Login into application after pressing on the 'Sign in' button
Screenshots

Postconditions
A clear and concise description of actions to be done after bug reproducing. E.g.: "Delete created item from database using SQL query: DELETE FROM table1 WHERE id=<item1_id>;"
User story and test case links E.g.: "User story #181"
Labels to be added "Bug", Priority ("pri: High"), Severity ("severity:"Critical), Type ( "Functional")

You have confused field descriptions "Actual" and "Expected"

You have confused field descriptions "Actual" and "Expected"

Sorry, I corrected them


Add timestamp for start of turn

This PR sets a timestamp start_of_turn in the result of the first card.
First when the board has been loaded, then on every third click of a turn.
Closes: #1298

In terms of analyzing data: I think it would be better to not set the start_of_turn variable on each card, but instead, send a start_of_turn variable within the result object. This will be saved to the results' json data. The response_interval_ms can probably be removed (check with @Lia-JX-Li ), as it can be derived from the timestamps.

start_of_turn is indeed more clear, so I changed it back.  I removed the response_interval_ms after consulting with Lia.


Create qodana.yml

:white_check_mark: Build GuiStracini.Mandae 8.0.23 completed (commit https://github.com/guibranco/GuiStracini.Mandae/commit/86c79adfbb by @guibranco)


Add support for limiting the branches for GitHub webhook notifications

We probably will want to control this by the user specifying &branches=master,development in the URL for now.

Yeah, ideally we'd do it for all the Git hosting services.

@zulipbot claim
Seems like a good little project to learn more about integrations! Hope to work on this over the weekend! Thanks! :-)

@TomaszKolek you mean we should think about bitbucket or gitlab too.

Yep I'm prettys ure that's what Tomasz means.

Exactly ;-)

Hey guys! Plan on working on this soon! Prepping for a Math midterm exam at the moment. Hope to submit a PR for this tomorrow night! Thanks! :-)

cool @eeshangarg no issue :)

Hi everyone! I would love to work on implementing this for all of our Git integrations. I think its a good way to dive into some more of our integrations codebase. Should I file a separate issue for each one, or should this one be enough? :-) Thanks!

I think you could just do them as a series of commits, one per integration, in the same PR.  Make sure each has tests, though!

Thanks, Tim! Yes, I am working on the tests right now for the current commit and I'll make sure I do the same for future related commits. :)

Hello @eeshangarg, you claimed this issue to work on it, but this issue and any referenced pull requests haven't been updated for a week. Are you still working on this issue?
If so, please update this issue by leaving a comment on this issue to let me know that you're still working on it. Otherwise, I'll automatically remove you from this issue in 3 days.
If you've decided to work on something else, simply comment @zulipbot abandon so that someone else can claim it and continue from where you left off.
Thank you for your valuable contributions to Zulip!


[BUG][Native/Android] The endless loading screen is displayed on the device after login with the google account

Description: The endless loading screen is displayed on the device after login with the google account
Steps to reproduce:

Launch the Gooddollar app
Press on the "Already have account? Log in" button
Press on the Log in with  Google button
Enter the email and password
Press Next
Pay attention to the endless loading screen

Expected result:
The user is logged in to the account on the app
Actual result:
The endless loading screen is displayed after login with the google account
The user is new: No
Environment:  App Version 1.13.0-0 (983)
Devices list: Xiaomi Mi 8 // Android 8.1.0
Additional info:
Email<EMAIL_ADDRESS>pass: Tester_11111
Notes: The same issue is reproduced when the user login or sign up  with Google, Facebook, and Passwordless
Video: https://www.dropbox.com/s/ymgf1ovz9n9nl8n/The endless loading after log in the app.mp4?dl=0

@sirpy @serdiukov-o-nordwhale
Reproduced the error with the apk, the problem is that the redirect link to our scripts.html (generated by the android sdk) has some data missing on the hash params:

When opening it on the browser and executing the scripts.html script step by step decoding the url, we can see that the state param contains just an empty JSON:

And thus, it doesn't know where to redirect (& redirection code doesn't execute), here's that part of the script:

@sirpy @serdiukov-o-nordwhale
Running locally works fine, here's what instanceParams (decoded state param) should look like:

@MarianoAmado

does this happen always? both on signup and signin?
is this an android only  issue? and only in production and  not in dev env? or could it be that we simply dont pass a correct  redirect  url  because we are missing some env in production?

@MarianoAmado

does this happen always? both on signup and signin?
is this an android only  issue? and only in production and  not in dev env? or could it be that we simply dont pass a correct  redirect  url  because we are missing some env in production?

@sirpy The issue is still reproduced.
Checked on the Android App 1.22.13-1 (1204)
Device: Xiaomi Mi 8 // Android 8.1.0
Attachment:
https://www.dropbox.com/s/cts335ul2n2209c/Login On the Android app.mp4?dl=0


SnpSift container not available

Hello,
While trying to run PTATO 1.3.3 I had an issue with pulling the SnpSift container:
https://github.com/ToolsVanBox/PTATO/blob/eafc29c4d887b4ba4795cb9cbff401e9bc621ef5/NextflowModules/SnpSift/4.3.1t--1/SnpSift.nf#L4
Apparently, it is not available anymore, and pulling the singularity image from another source is required.

Sorry for the late response,. For now I have added the container here:
https://surfdrive.surf.nl/files/index.php/s/l3FX6eLnTtuVK1g?path=%2Fcontainers


Fetch a reasonable size thumbnail for events (1600px)

Required to solve Issue #52

Codecov Report

Merging #161 (d5925de) into main (f6eaa4a) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #161      +/-   ##
==========================================
+ Coverage   92.01%   92.06%   +0.05%
==========================================
  Files          16       16
  Lines        1778     1778
==========================================
+ Hits         1636     1637       +1
+ Misses        142      141       -1

Impacted Files
Coverage Δ

google_nest_sdm/event_media.py
95.92% <100.00%> (+<0.01%)
:arrow_up:

google_nest_sdm/google_nest_subscriber.py
62.33% <0.00%> (-0.17%)
:arrow_down:

google_nest_sdm/camera_traits.py
95.50% <0.00%> (+0.37%)
:arrow_up:

Continue to review full report at Codecov.

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


Created AbstractStore without coreOpHandlers.

Implemented Store to extend AbstractStore and add coreOpHandlers.
ProxyStore implements AbstractStore only.

Codecov Report

Merging #1940 into develop will decrease coverage by 1.02%.
The diff coverage is 63.6%.

@@              Coverage Diff              @@
##             develop    #1940      +/-   ##
=============================================
- Coverage      64.73%   63.71%   -1.03%
+ Complexity      2228     2178      -50
=============================================
  Files            882      790      -92
  Lines          28950    26079    -2871
  Branches        3616     3346     -270
=============================================
- Hits           18741    16615    -2126
+ Misses          8738     8115     -623
+ Partials        1471     1349     -122

Impacted Files
Coverage Δ
Complexity Δ

...on/hdfs/handler/SplitStoreFromIterableHandler.java
70.58% <ø> (ø)
0 <0> (ø)
:arrow_down:

...eration/handler/SplitStoreFromIterableHandler.java
8.69% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ulostore/operation/handler/GetElementsHandler.java
77.77% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ulostore/operation/handler/AddElementsHandler.java
81.81% <ø> (ø)
0 <0> (ø)
:arrow_down:

...store/operation/handler/GetAdjacentIdsHandler.java
60% <ø> (ø)
0 <0> (ø)
:arrow_down:

...tore/operation/handler/function/FilterHandler.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...store/operation/handler/GetAllElementsHandler.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ation/hdfs/handler/AddElementsFromHdfsHandler.java
60% <ø> (ø)
0 <0> (ø)
:arrow_down:

...affer/store/operation/handler/GetWalksHandler.java
6.83% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ation/handler/utilities/AggregateDataForGroup.java
89.36% <ø> (ø)
0 <0> (ø)
:arrow_down:

... and 190 more

Continue to review full report at Codecov.

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

Switched/renamed to https://github.com/gchq/Gaffer/pull/1949

switched/renamed to #1949


Implement weighted hamming distance kernel for categorical inputs

Section 4.1.2 in http://arxiv.org/pdf/1211.0906v2.pdf

This should be rebased on top of master. Then I would assume that that special can be used when all dimensions are of types Categorical, right?


[CGPROD-3075] Phaser and RexUI upgrade

@AdamXA - I updated the Phaser plugin and the files work as is. Then noticed an npm audit issue with webpack. Got version 5 which broke everything. Looks working now - if you are ok with this just merge it. Need to do a draft release to check Stories module. If there are further problems we can fix in another commit maybe?


I would like to ask about the meaning of the script folder file

What are the extract_enlarge_patches capable of?

I also want to ask where the epoch was set

Did you mean the total training epoch, or others?

Training of the epoch


Insidious accidental lambda conversion

Can you spot the bug?
From Ben Deane
const bool proceed = [&](){ return complexcheck; };

if (proceed) {
  // do things
}

That does not even compile.
gcc claims that:
<source>:3:26: error: cannot convert 'main()::<lambda()>' to 'const bool' in initialization

Besides that, is there something else are we supposed to spot?

It probably should have been auto instead of bool, so that the conversion is attempted at the if: https://compiler-explorer.com/z/9Kxf3doqc.
bool complexcheck;
int main() {
  const auto proceed = [&](){ return complexcheck; };

  if (proceed) {
    // do things
  }
}

This does convert:
int main() {
  auto _ = [](){ };
  if (_) { }
}

Thanks everyone, you made me double check this.
The actual problem is the & capture. We need it to be a captureless lambda so that the conversion to function * is found, which implicit converts to bool

@elbeno if you can provide more context that would be great.

It was a variable template - something like this:
template <typename T>
constexpr bool thing_enabled = [] {
  if constexpr (requires { T::thing_enabled; }) {
    return T::thing_enabled;
  } else {
    return false;
  }
}; // missing immediate invocation

The intent is to immediately invoke the lambda to determine whether thing_enabled is true for a type. But we are forgetting the invocation. And the lambda doesn't capture, so it has an implicit conversion to function pointer which is always true.
For very simple cases, a conditional expression rather than an IILE could perhaps be used, but in general the logic inside the lambda doesn't have to be very complex to make this sort of thing awkward -- or impossible: if constexpr can result in different return types on different control paths -- to write as a conditional expression.
Of course adequate test coverage also makes this bug easier to find, but nevertheless it's a real-world bug formulation.


Service keeps Crashing (2.3)

Win Sever 2016. Just ran the installer.
Any way to find the problem ? Could it be Windows Defender ?
The following are event log entries.
Faulting application name: Wexflow.Clients.WindowsService.exe, version: <IP_ADDRESS>, time stamp: 0x5a09c867
Faulting module name: KERNELBASE.dll, version: 10.0.14393.1770, time stamp: 0x59bf2bcf
Exception code: 0xe0434352
Fault offset: 0x000daa12
Faulting process id: 0x950
Faulting application start time: 0x01d360f688a119fa
Faulting application path: C:\Program Files (x86)\Wexflow\Wexflow.Clients.WindowsService.exe
Faulting module path: C:\Windows\System32\KERNELBASE.dll
Report Id: bb0e5a6c-4f3a-4154-8532-305da5f30d1b
Faulting package full name:
Faulting package-relative application ID:
Application: Wexflow.Clients.WindowsService.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.ArgumentException
at System.ThrowHelper.ThrowArgumentException(System.ExceptionResource)
at System.Collections.Generic.Dictionary2[[System.Int32, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089],[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]].Insert(Int32, System.__Canon, Boolean)    at System.Collections.Generic.Dictionary2[[System.Int32, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089],[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]].Add(Int32, System.__Canon)
at Wexflow.Core.WexflowEngine.ScheduleWorkflow(Wexflow.Core.Workflow)
at Wexflow.Core.WexflowEngine.b__36_3(System.Object, System.IO.FileSystemEventArgs)
at System.IO.FileSystemWatcher.OnChanged(System.IO.FileSystemEventArgs)
at System.IO.FileSystemWatcher.NotifyFileSystemEventArgs(Int32, System.String)
at System.IO.FileSystemWatcher.CompletionStatusChanged(UInt32, UInt32, System.Threading.NativeOverlapped*)
at System.Threading._IOCompletionCallback.PerformIOCompletionCallback(UInt32, UInt32, System.Threading.NativeOverlapped*)

To provide a bit more context,

This was a machine that never had wexflow previously installed
Uninstalled 2.3, installed 2.2 - no issues
Reinstalled 2.3 and the problem is there again

I am now back at 2.2

Maybe you have a duplicated workflow id. Try to proceed as follows:

Backup your folder C:\Wexflow
Delete your folder C:\Wexflow
Install the version 2.3
Check that you don't have a duplicated workflow id in your custom workflows

I think I have isolated the problem to the trigger type.
When type=period, there is an inconsistency between the XML and the designer UI
i.e this is what I want, and setting directly in  XML works OK

but designer UI loads it as  24.00:00:00
If I set in the designer as 24:00:00 it results in the XML storing
which I think maybe causing the issue ? The workaround - editing XML and not changing in designer - so far it has not crashed

I fixed the issue. You can download the installer from here (release v2.3).


Are the available statuses listed on the page accurate?

[Enter feedback here]
This page indicates that the possible statuses for an operation include 'skipped', and indicates these statuses are inherited from longRunningOperation, which doesn't list 'skipped' as a possible status:
https://learn.microsoft.com/en-us/graph/api/resources/longrunningoperation?view=graph-rest-beta#properties

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

ID: 7c3f6cfe-ea58-5082-76e4-6d732d1acbc5
Version Independent ID: a52a1d44-7e9f-88d3-9a68-13bf4aa154bb
Content: engagementAsyncOperation resource type - Microsoft Graph beta
Content Source: api-reference/beta/resources/engagementasyncoperation.md
Product: employee-engagement
Technology: microsoft-graph
GitHub Login: @aditijha4
Microsoft Alias: MSGraphDocsVteam

@timward60 do you know about this? It could just be that the documentation for https://learn.microsoft.com/en-us/graph/api/resources/longrunningoperation?view=graph-rest-beta#properties is out of date.

I checked the metadata and skipped is not part of this list. We should update our documentation.
<EnumType Name="longRunningOperationStatus">
  <Member Name="notStarted" Value="0"/>
  <Member Name="running" Value="1"/>
  <Member Name="succeeded" Value="2"/>
  <Member Name="failed" Value="3"/>
  <Member Name="unknownFutureValue" Value="4"/>
</EnumType>

Created a PR: https://github.com/microsoftgraph/microsoft-graph-docs/pull/23282

Abandoned the PR - the skipped status is included for longRunningOperationStatus in beta, and the doc https://learn.microsoft.com/en-us/graph/api/resources/longrunningoperation?view=graph-rest-beta has been updated to reflect this.


Add app arsDrive

IPFS network disk
An Extension of ARS pServer Based on IPFS

PR is 4 years old, repo https://github.com/arsyun/arsDrive is 4 years old with each 1 star. closing


Reversed weather icons

Love the app - thanks for keeping it alive! Just a small bug - the night and day icons seem to be switched in the forecasts (it shows the moon during the day and the sun at night). I've tried toggling the 24hr/12hr and it doesn't fix it

For some context, it only seems to flip in Australia, the other places have correct icons

Mhm... are you from Australia? I checked New Zealand and it is the same for me. But from my point of view it makes sense. So checking the weather in Sydney right now (3:40 PM) from Berlin means that it is night time in Sydney (01:41 AM). So the moon will fit, wouldn't it?
If this is not the case I am happy to try some more debugging, just leave me a hint :)

Yep I'm from AU - I think the icons are main switched in the weekly forecast section:
So in the 4 day forecast on the bottom, the moon (moon icon or moon with clouds) will show regardless of what time it is. In the next view with the time scrubber, the icons seem jumbled, with the moon appearing at 3pm and the sun at 3am

Alright that really seems off. So thank you for reporting this. I will try to figure out where this is coming from and then of course fix it, if possible :p

Sorry for the late response. Till now I couldn't find a solution for this. For me it seems connected to the openweather response. And I am not entirely sure how to handle this...
Are you using your own API key from openweatehrmap?
Ah end do you know since when this is happening? I had to make a change according to the timezone stuff, so maybe this is connected...


Block and unblock user methods

Tested on both "profile" and "direct_thread_info" surfaces

@Yessirskiy thanks!


test.aria.widgets.form.multiautocomplete.testDMOnFreetext.MultiAutoDataModelTest fails for Chrome

I have just checked and this test does not fail on Chrome anymore.


feat: common command aliases

:gift: commands which refer to sets ignore plurality
:gift: aliases added for linux command synonyms

Ensures commands which have subcommands to operate on individual members (list, add, remove etc) have singular and plural as aliases.
Ensures consistent usage of linux-command-style aliases for list,remove and rename operations: "ls", "rm", "mv"

/kind enhancement
Commands such as `envs` can be referred to by their singular form `env` and vice-versa.
Commands with well-known command synonyms were added as aliases, such as`labels delete <name>` now supports `labels rm <name>`

@lkingland sorry, I mistaken this PR with mine 😄 🤦♂


Warning: Cannot read property 'pot-creation-date' of undefined Use --force to continue.

I'm trying to generate a .pot file from my plugin but I it keeps failing.
Here is the error message .
D:\wamp\www\wordpress-clean\wp-content\plugins\drivers-handbook-guideline>grunt i18n
Running "addtextdomain:update_all_domains" (addtextdomain) task

Running "makepot:target" (makepot) task
Warning: Cannot read property 'pot-creation-date' of undefined Use --force to continue.

Aborted due to warnings.

Other Info :
grunt-cli v1.2.0
grunt v0.4.5
nodejs v9.11.1

If anyone would ever encounter this error from grunt.
The solution is to change the potFilename inside your GruntFile.js
"test-plugin.pot"  to "test-plugin-translation.pot"


fix: http webhook header when no secret token

Using String here makes the function return 'undefined' instead of
undefined, thus breaking the verification process
Fixes #239

Codecov Report

Merging #240 (94ee512) into main (1402e2a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #240   +/-   ##
=======================================
  Coverage   22.21%   22.21%
=======================================
  Files           9        9
  Lines        2813     2813
  Branches       90       90
=======================================
  Hits          625      625
  Misses       2187     2187
  Partials        1        1

Continue to review full report at Codecov.

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

@roziscoding however, CI tells me I'm wrong. There could be an array of headers, rather than a single header. Could you fix this by accounting for that case?
You can check if your code compiles by running npm install && npm run backport, or deno task backport.

Sure! I didn't know to run backport. Should this be added to the contributing guidelines? (if it's already there and I've missed, I'm sorry lol)

Good point, I'll make sure that this will make its way into the guide, thanks!

We usually require all commits to be signed by contributors. I will overrule now and merge anyway, but I recommend you to look into this in the future :)

@all-contributors add @roziscoding  for bug and code

We usually require all commits to be signed by contributors. I will overrule now and merge anyway, but I recommend you to look into this in the future :)

Oh boy. I usually sign all my commits. I must've forgotten to set it up in my newest setup. Thanks for the heads up. I'll do it right now :)


Update packer install to 0.12.2

Without it a bake fails now that rosco uses the new source_image_family option.
https://github.com/spinnaker/rosco/pull/166

I signed it!
On Jan 27, 2017 4:53 PM, "googlebot"<EMAIL_ADDRESS>wrote:
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify.
Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/spinnaker-deploymentmanager/pull/2#issuecomment-275813096,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABV03CyvkDv5B2VHGt6c0wB6N9NQU4CBks5rWpGLgaJpZM4LwXW_
.

Thanks @taylorludwig!!!


Not compatible latest wagmi

Hi, I notice that Beta.4 not compatible with<EMAIL_ADDRESS>

@callmejm Yeh, beta.4 had an issue, there is already beta.5 😅 that solves this.


has vtable analysis

Fix #765 r? @fitzgen

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @fitzgen (or someone else) soon.

@bors-servo r+

@bors r+

Ok, manually merging for now.


@-moz-document (and universal support for other unsupported attributes)

@-moz-document url-prefix() {
    h1 {
        color: red;
    }
}

dotless totally does not like this selector, which is valid in firefox. To make things easier on yourself, so you don't have to support every obscure vendor definition that comes out, can't you just add a way to escape an attribute name so that it gets parsed "as is"?

Should be fixed, please re-open if this is still happening for you in main / 1.2.2


UI.Win: allow to use IUIAutomation2 interface

For now UI.Win uses IUIAutomation interface and CUIAutomation as result.
But starting form Windows 8 IUIAutomation2 interface is available along with CUIAutomation8. They have some improvements in terms of UIA timeouts handling and updates in some interfaces

done in 091b40d


container build is breaking

46.99 ERROR:  Error installing r10k:
46.99 	The last version of faraday-net_http (>= 2.0, < 3.2) to support your Ruby & RubyGems was 3.0.2. Try installing it with `gem install faraday-net_http -v 3.0.2` and then running the current command again
46.99 	faraday-net_http requires Ruby version >= 3.0.0. The current ruby version is <IP_ADDRESS>.

see https://github.com/voxpupuli/container-puppetserver/actions/runs/7478099037/job/20352284695?pr=46

r10k dependency is breaking. gem command is automatically used from the puppet-agent environment.

we could switch to the packaged r10k. but this would mean going back to version 3.7.x
we could try to install system ruby which is >=3.x.y

@tuxmea @bastelfreak whats you opinion?

Would it be possible to first install faraday-net_http in Version 3.0.2. and the install r10k?
Or maybe make Gemfile.lock part of the repo?


Implement asynchronous shutdown

Motivation:
Most NIO-based client implementation provide asynchronous shutdown method, we need to provide one as well. Closes #126
Modifications:

Converted all close and prepareClose methods in pool and connection providers to be asynchronous.
Removed checks for not running close and prepareClose in pool and connection providers on event loop.
Converted shutdown to use NIO asynchronous event loop shutdown method.
Implemented synchronous shutdown methods on top of asynchronous ones.

Result:
Client of the library can now use asynchronous shutdown when required.

CC @adtrevor , mind taking a look at the new async shutdown?

@swift-server-bot test this please

@swift-server-bot test this please


Bug report: Sketch sha256 mismatch

Description of issue
brew cask install sketch complaining about sha256 mismatch.
I followed the steps on https://github.com/caskroom/homebrew-cask#reporting-bugs and it still appears to occur.
Output of brew cask <command> --verbose
brettrogerson:~ $ brew cask install sketch --verbose
==> Downloading http://www.sketchapp.com/static/download/sketch.zip
/usr/bin/curl -fLA Homebrew-cask v0.51+ (Ruby 2.0.0-645) http://www.sketchapp.com/static/download/sketch.zip -C 0 -o /Library/Caches/Homebrew/sketch-3.7.1.zip.incomplete
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   307    0   307    0     0    536      0 --:--:-- --:--:-- --:--:--   535
100 23.2M  100 23.2M    0     0  1551k      0  0:00:15  0:00:15 --:--:--  790k
==> Verifying checksum for Cask sketch
==> Note: running "brew update" may fix sha256 checksum errors
Error: sha256 mismatch
Expected: a6aae446bb5b0a1ac883f55ef01a5e32ef7dc21000b861f114200025cfc96e8d
Actual: cf07466d5c8aaffb3e0c6d6242c3f2367b4f84d293d01064e1a7ee1a503c3429
File: /Library/Caches/Homebrew/sketch-3.7.1.zip
To retry an incomplete download, remove the file above.
Error: Kernel.exit

Output of brew doctor
brettrogerson:~ $ brew doctor
Your system is ready to brew.


Gigya returned error 403007, error when setting up component

Restarted Home Assistant now without any changes to the ZE component and got the following error:
Error while setting up renaultze platform for sensor Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 179, in _async_setup_platform     await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)   File "/usr/local/lib/python3.8/asyncio/tasks.py", line 483, in wait_for     return fut.result()   File "/config/custom_components/renaultze/sensor.py", line 102, in async_setup_platform     g.account_info()   File "/usr/local/lib/python3.8/site-packages/pyze/api/credentials.py", line 26, in inner     return func(*args, **kwargs)   File "/usr/local/lib/python3.8/site-packages/pyze/api/gigya.py", line 77, in account_info     raise_gigya_errors(response_body)   File "/usr/local/lib/python3.8/site-packages/pyze/api/gigya.py", line 123, in raise_gigya_errors     raise RuntimeError( RuntimeError: Gigya returned error 403007: Invalid namespace 'accounts' or method 'getAccountInfo' or you do not have the required permissions to call it.
Anyone experienced this?

403007 is an error from Gigya, so I assume the problem is on the Renault side.
You can also try to run the Pyze module directly in CLI mode. I expect it will be the same error there.

Note from Gigya documentation: https://developers.gigya.com/display/GD/Response+Codes+and+Errors
403007 | Permission denied | Returned  when a user lacks the necessary permissions to perform the requested  action, or when the user's credentials are not configured properly.

I got the same error starting around 5 hours ago.

I got the same error starting around 5 hours ago.

@age-creator did you fix it? I have not been able to find what is wrong. What's your location and car?
Mine is Sweden ZE50 R135

Same error since a couple of hours. Anybody checked the pyze module directly?

I got the same error starting around 5 hours ago.

@age-creator did you fix it? I have not been able to find what is wrong. What's your location and car?
Mine is Sweden ZE50 R135

No, I didn't have time to look into this further, but it looks again as an change to the authentication which is maybe implemented in phases. Car: NL ZE40 R110

Same error since a couple of hours. Anybody checked the pyze module directly?

@toby17780
Yes I get this error from pyze directly using the same values as before and working when logging out and of the app:
raise RuntimeError( RuntimeError: Gigya returned error 403042: invalid loginID or password

Fixed in pull #79. Waiting for a updated pipy package to resolve the issue with this repo.

Resolved in #44

Resolved in #44


Tweak permissions to allow for 7th/8th grade classroom sharing at WSNS

Feedback from Kathleen Seward, principal at the West Somerville Neighborhood School:
"7th HR teacher cannot see details on 8th grade students even though they teach them"

Set the following permissions for 7th / 8th grade teachers at WSNS using the Rails console:
grade_level_access = ["7", "8"]
Double-checked their views using the test account and all looked good.
Emailed Kathleen to let her know about the update. Closing for now, will re-open if Kathleen reports any new issues.


add test for oss-fuzz

It seems overkill to compile and run the fuzzers by default. I think we should have a separate parameter/command for that.
Currently this costs extra cpu-time on travis, and it would be better to have the option to control when it gets run.
Running the fuzzer with and without--zlib-compat might be enough. And always with the extra algorithms should be good, since that includes the stock ones as well.
So it might be enough to run the fuzzers twice per commit per arch, unless I am missing something.

We can reduce the amount of testing by selecting fewer files to run the fuzzers on.
Currently we are running the fuzzers on all . files in the top dir, and that includes all the build objects.
What about selecting the first 10 or so files in that list and only run the standalone fuzzers on these?

@sebpop How does that fix anything?
If we want to run the fuzzers, we probably want a full run. And if we don't want them, then we simply don't want them at all.

I amended the patch with b52698c that adds a configure flag --build-fuzzers.  The flag is off by default.  We now build the fuzzers when this flag is set at configure time, or when LIB_FUZZING_ENGINE is set in env.

./configure --with-sanitizers --with-fuzzers
make

Fails with:
gcc -L. -fsanitize=bool,address,float-divide-by-zero,integer-divide-by-zero,return,shift,signed-integer-overflow,undefined,vla-bound,vptr -o checksum_fuzzer standalone_fuzz_target_runner.o checksum_fuzzer.o libz-ng.a -lpthread
libz-ng.a(match.o):(.data+0x40): undefined reference to `__gcov_merge_add'
libz-ng.a(match.o): In function `_GLOBAL__sub_I_00100_0_longest_match':
match.c:(.text.startup+0x6): undefined reference to `__gcov_init'
libz-ng.a(match.o): In function `_GLOBAL__sub_D_00100_1_longest_match':
match.c:(.text.exit+0x1): undefined reference to `__gcov_exit'
collect2: error: ld returned 1 exit status
make: *** [Makefile:166: checksum_fuzzer] Error 1

./configure --with-fuzzers
make

Fails with:
gcc -L. -o checksum_fuzzer standalone_fuzz_target_runner.o checksum_fuzzer.o libz-ng.a -lpthread
libz-ng.a(match.o):(.data+0x40): undefined reference to `__gcov_merge_add'
libz-ng.a(match.o): In function `_GLOBAL__sub_I_00100_0_longest_match':
match.c:(.text.startup+0x6): undefined reference to `__gcov_init'
libz-ng.a(match.o): In function `_GLOBAL__sub_D_00100_1_longest_match':
match.c:(.text.exit+0x1): undefined reference to `__gcov_exit'
standalone_fuzz_target_runner.o: In function `main':
standalone_fuzz_target_runner.c:(.text+0x31): undefined reference to `__ubsan_handle_sub_overflow'
standalone_fuzz_target_runner.c:(.text+0x56): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x6c): undefined reference to `__ubsan_handle_nonnull_arg'
standalone_fuzz_target_runner.c:(.text+0xc6): undefined reference to `__ubsan_handle_pointer_overflow'
standalone_fuzz_target_runner.c:(.text+0xf4): undefined reference to `__ubsan_handle_type_mismatch_v1'
standalone_fuzz_target_runner.c:(.text+0x114): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x16c): undefined reference to `__ubsan_handle_pointer_overflow'
standalone_fuzz_target_runner.c:(.text+0x19a): undefined reference to `__ubsan_handle_type_mismatch_v1'
standalone_fuzz_target_runner.c:(.text+0x1ba): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x1df): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x1f5): undefined reference to `__ubsan_handle_nonnull_arg'
standalone_fuzz_target_runner.c:(.text+0x248): undefined reference to `__ubsan_handle_pointer_overflow'
standalone_fuzz_target_runner.c:(.text+0x276): undefined reference to `__ubsan_handle_type_mismatch_v1'
standalone_fuzz_target_runner.c:(.text+0x296): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x2bc): undefined reference to `__ubsan_handle_type_mismatch_v1'
standalone_fuzz_target_runner.c:(.text+0x2dd): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x306): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x31c): undefined reference to `__ubsan_handle_nonnull_arg'
standalone_fuzz_target_runner.c:(.text+0x3ac): undefined reference to `__asan_handle_no_return'
standalone_fuzz_target_runner.c:(.text+0x402): undefined reference to `__asan_handle_no_return'
standalone_fuzz_target_runner.c:(.text+0x45b): undefined reference to `__ubsan_handle_pointer_overflow'
standalone_fuzz_target_runner.c:(.text+0x489): undefined reference to `__ubsan_handle_type_mismatch_v1'
standalone_fuzz_target_runner.c:(.text+0x4a9): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x4ce): undefined reference to `__asan_report_load8'
standalone_fuzz_target_runner.c:(.text+0x4e4): undefined reference to `__ubsan_handle_nonnull_arg'
standalone_fuzz_target_runner.c:(.text+0x51d): undefined reference to `__ubsan_handle_add_overflow'
standalone_fuzz_target_runner.o: In function `_GLOBAL__sub_D_00099_0_main':
standalone_fuzz_target_runner.c:(.text+0x54f): undefined reference to `__asan_unregister_globals'
standalone_fuzz_target_runner.o: In function `_GLOBAL__sub_I_00099_1_main':
standalone_fuzz_target_runner.c:(.text+0x55a): undefined reference to `__asan_init'
standalone_fuzz_target_runner.c:(.text+0x55f): undefined reference to `__asan_version_mismatch_check_v8'
standalone_fuzz_target_runner.c:(.text+0x56e): undefined reference to `__asan_register_globals'
checksum_fuzzer.o: In function `LLVMFuzzerTestOneInput':
checksum_fuzzer.c:(.text+0x81): undefined reference to `__ubsan_handle_type_mismatch_v1'
checksum_fuzzer.c:(.text+0xb5): undefined reference to `__asan_report_load1'
checksum_fuzzer.c:(.text+0x122): undefined reference to `__ubsan_handle_pointer_overflow'
checksum_fuzzer.c:(.text+0x16f): undefined reference to `__ubsan_handle_divrem_overflow'
checksum_fuzzer.c:(.text+0x1a1): undefined reference to `__ubsan_handle_pointer_overflow'
checksum_fuzzer.c:(.text+0x1e1): undefined reference to `__asan_handle_no_return'
checksum_fuzzer.c:(.text+0x22b): undefined reference to `__asan_handle_no_return'
checksum_fuzzer.c:(.text+0x275): undefined reference to `__ubsan_handle_pointer_overflow'
checksum_fuzzer.c:(.text+0x2c2): undefined reference to `__ubsan_handle_divrem_overflow'
checksum_fuzzer.c:(.text+0x2f4): undefined reference to `__ubsan_handle_pointer_overflow'
checksum_fuzzer.c:(.text+0x334): undefined reference to `__asan_handle_no_return'
checksum_fuzzer.c:(.text+0x37e): undefined reference to `__asan_handle_no_return'
checksum_fuzzer.o: In function `_GLOBAL__sub_D_00099_0_LLVMFuzzerTestOneInput':
checksum_fuzzer.c:(.text+0x3b8): undefined reference to `__asan_unregister_globals'
checksum_fuzzer.o: In function `_GLOBAL__sub_I_00099_1_LLVMFuzzerTestOneInput':
checksum_fuzzer.c:(.text+0x3c3): undefined reference to `__asan_init'
checksum_fuzzer.c:(.text+0x3c8): undefined reference to `__asan_version_mismatch_check_v8'
checksum_fuzzer.c:(.text+0x3d7): undefined reference to `__asan_register_globals'
collect2: error: ld returned 1 exit status
make: *** [Makefile:166: checksum_fuzzer] Error 1

./configure --with-sanitizers
make
make test

Fails with:
make -C test
make[1]: Entering directory '/home/hansr/github/zlib/zlib-ng/test'
AddressSanitizer:DEADLYSIGNAL
=================================================================
==317032==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000080 (pc 0x7f6ff1e2a868 bp 0x7ffc66382ca0 sp 0x7ffc663823f0 T0)
==317032==The signal is caused by a READ memory access.
AddressSanitizer:DEADLYSIGNAL
=================================================================
==317032==Hint: address points to the zero page.
==317031==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000080 (pc 0x7fcd4be1ac8e bp 0x00000000000a sp 0x7ffe1f8e8ab0 T0)
==317031==The signal is caused by a READ memory access.
==317031==Hint: address points to the zero page.
    #0 0x7f6ff1e2a867 in __GI__IO_fread /usr/src/debug/glibc-2.27-78-g2b47bb9cba/libio/iofread.c:35
    #1 0x7f6ff2214d9e  (/lib64/libasan.so.5+0x9bd9e)
    #0 0x7fcd4be1ac8d in __GI__IO_fwrite /usr/src/debug/glibc-2.27-78-g2b47bb9cba/libio/iofwrite.c:35
    #2 0x402b23 in zng_gzread (/home/hansr/github/zlib/zlib-ng/minigzip+0x402b23)
    #1 0x7fcd4c2048ae  (/lib64/libasan.so.5+0x9b8ae)
    #3 0x403996 in gz_uncompress (/home/hansr/github/zlib/zlib-ng/minigzip+0x403996)
    #2 0x402628 in zng_gzwrite (/home/hansr/github/zlib/zlib-ng/minigzip+0x402628)
    #4 0x404cd2 in main (/home/hansr/github/zlib/zlib-ng/minigzip+0x404cd2)
    #3 0x403827 in gz_compress (/home/hansr/github/zlib/zlib-ng/minigzip+0x403827)
    #4 0x404d73 in main (/home/hansr/github/zlib/zlib-ng/minigzip+0x404d73)
    #5 0x7f6ff1ddd11a in __libc_start_main ../csu/libc-start.c:308
    #6 0x4014e9 in _start (/home/hansr/github/zlib/zlib-ng/minigzip+0x4014e9)
    #5 0x7fcd4bdcd11a in __libc_start_main ../csu/libc-start.c:308

    #6 0x4014e9 in _start (/home/hansr/github/zlib/zlib-ng/minigzip+0x4014e9)
AddressSanitizer can not provide additional info.

SUMMARY: AddressSanitizer: SEGV /usr/src/debug/glibc-2.27-78-g2b47bb9cba/libio/iofread.c:35 in __GI__IO_fread
AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /usr/src/debug/glibc-2.27-78-g2b47bb9cba/libio/iofwrite.c:35 in __GI__IO_fwrite
==317032==ABORTING
==317031==ABORTING
		*** zlib test FAILED ***
make[1]: *** [Makefile:49: teststatic] Error 1
make[1]: Leaving directory '/home/hansr/github/zlib/zlib-ng/test'
make: *** [Makefile:119: test] Error 2

cmake -DWITH_SANITIZERS=1
make
make test

Fails with
Running tests...
Test project /home/hansr/github/zlib/zlib-ng
    Start 1: example
1/2 Test #1: example ..........................***Failed    0.14 sec
    Start 2: example64
2/2 Test #2: example64 ........................***Failed    0.13 sec

0% tests passed, 2 tests failed out of 2

Total Test time (real) =   0.27 sec

The following tests FAILED:
	  1 - example (Failed)
	  2 - example64 (Failed)
Errors while running CTest
make: *** [Makefile:130: test] Error 8

This is on a Fedora 28 x86_64 system, using GCC 8.2.1

./configure --with-sanitizers --with-fuzzers
make
Fails with
libz-ng.a(match.o):(.data+0x40): undefined reference to `_gcov*'

I think match.o was previously compiled with -fprofile-* as part of a different build.
Please try make clean between builds with different configures:
./configure --with-sanitizers --with-fuzzers; make clean; make

ERROR: AddressSanitizer: SEGV on unknown address

we are missing some of the fixes to the test files test/example.c

Proposed fix for the error in test/example.c is in https://github.com/Dead2/zlib-ng/pull/213

Weird, I don't think I have ever run a build with a -fprofile-* parameter enabled.
But you are right about 'make clean'. I was told a while back that cmake had no 'make clean', I guess I should have verified that myself, because it obviously does.
That only leaves the bug in #213 then, when we get that one to pass Travis tests, I'll pull this PR too.

Proposed fix https://github.com/Dead2/zlib-ng/pull/214 for the error in test/minigzip.

The fix in #213 is not new, I have fixed the same problem some time ago in my develop branch.
The bug that #214 fixes is new: this bug appeared after the change to configure to unify the flags with cmake flags. Before, we used to build 4 different versions of zlib: static, shared, asan, and msan. minigzip used to be linked with the static zlib which was not containing sanitized code.  With the unified interface change to configure we now link minigzip against the static sanitized version of zlib, which exposes the new bug with address sanitizer on.

Once #213 is merged we should have a clean make test with sanitizers and with fuzzers.
Can we get the test/fuzz committed such that we can move oss-fuzz to pull from develop branch of https://github.com/Dead2/zlib-ng
Please let me know what needs to be done to merge this patch in.  Thanks!

Looks good to me, thanks!


Feature/updated docker testing docs

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Related issues
#15345

PR Updated.

Merged. Thank you!


New feature: Make a video file

I have just implemented this feature. It uses ffmpy/ffmpeg. In theory that should work for everyone since those are a dependency of gradio. But if you have problems with this, please let me know.

https://github.com/AlUlkesh/sd_save_intermediate_images/commit/441ec8ae3b13a48130ac31823fd4666615a7c9d5

I added a Smoothing / Interpolate feature. It's a bit on the slow side and gifs get pretty big. But it looks cool...
https://user-images.githubusercontent.com/99896447/212520744-bc2cfddd-98e7-4104-8609-3392f2aba532.mp4

Can you add some other options for the codec being used?

Are you thinking of anything in particular?

It was mainly h264 which I think it does already.. but something is wrong with it in that instagram won't accept it as a video some other setting with the codec it is using.. I've done previous videos about same  width/height/lenght/framerate etc and they've been fine... also converting some of them generated from your script with handbrake has worked.. so not sure specifically yet

You can try using ffmpeg's ffprobe on a file that works, and one that doesn't, maybe we can see what's different then.

A file that had worked  `ffprobe version N-109594-g6161eacc74-20230113 Copyright (c) 2007-2023 the FFmpeg developers
built with gcc 12.2.0 (crosstool-NG <IP_ADDRESS>_cf9beb1)
configuration: --prefix=/ffbuild/prefix --pkg-config-flags=--static --pkg-config=pkg-config --cross-prefix=x86_64-w64-mingw32- --arch=x86_64 --target-os=mingw32 --enable-gpl --enable-version3 --disable-debug --disable-w32threads --enable-pthreads --enable-iconv --enable-libxml2 --enable-zlib --enable-libfreetype --enable-libfribidi --enable-gmp --enable-lzma --enable-fontconfig --enable-libvorbis --enable-opencl --disable-libpulse --enable-libvmaf --disable-libxcb --disable-xlib --enable-amf --enable-libaom --enable-libaribb24 --enable-avisynth --enable-chromaprint --enable-libdav1d --enable-libdavs2 --disable-libfdk-aac --enable-ffnvcodec --enable-cuda-llvm --enable-frei0r --enable-libgme --enable-libkvazaar --enable-libass --enable-libbluray --enable-libjxl --enable-libmp3lame --enable-libopus --enable-librist --enable-libssh --enable-libtheora --enable-libvpx --enable-libwebp --enable-lv2 --disable-libmfx --enable-libvpl --enable-openal --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenh264 --enable-libopenjpeg --enable-libopenmpt --enable-librav1e --enable-librubberband --enable-schannel --enable-sdl2 --enable-libsoxr --enable-libsrt --enable-libsvtav1 --enable-libtwolame --enable-libuavs3d --disable-libdrm --disable-vaapi --enable-libvidstab --enable-vulkan --enable-libshaderc --enable-libplacebo --enable-libx264 --enable-libx265 --enable-libxavs2 --enable-libxvid --enable-libzimg --enable-libzvbi --extra-cflags=-DLIBTWOLAME_STATIC --extra-cxxflags= --extra-ldflags=-pthread --extra-ldexeflags= --extra-libs=-lgomp --extra-version=20230113
libavutil      57. 43.100 / 57. 43.100
libavcodec     59. 56.100 / 59. 56.100
libavformat    59. 35.100 / 59. 35.100
libavdevice    59.  8.101 / 59.  8.101
libavfilter     8. 53.100 /  8. 53.100
libswscale      6.  8.112 /  6.  8.112
libswresample   4.  9.100 /  4.  9.100
libpostproc    56.  7.100 / 56.  7.100
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from '01692-037-2-forest with city buildings like tokyo in the background high quality 3d sunset.mp4':
Metadata:
major_brand     : isom
minor_version   : 512
compatible_brands: isomiso2avc1mp41
encoder         : Lavf58.76.100
Duration: 00:00:07.70, start: 0.000000, bitrate: 992 kb/s
Stream #0:00x1: Video: h264 (High) (avc1 / 0x31637661), yuv420p(progressive), 768x768, 990 kb/s, 10 fps, 10 tbr, 10240 tbn (default)
Metadata:
handler_name    : VideoHandler
vendor_id       : [0][0][0][0]
C:\Projects\Stable_Diffusion\stable-diffusion-webui\outputs\txt2img-images\intermediates\01692>`

I changed the mp4 parameters to the ones that work for you. Let me know if that does the trick.
I'll add a couple of more options later.

Thanks I tested it on instagram it works nicely now


panic: pq: null value in column "is_relay" of relation "device_profile" violates not-null constraint

Hi
I'm trying to upgrade on different docker containers using last version of chirpstack-v3-to-v4. It doesn't end because this error Can somebody help me to fix this issue ?
debian@docker4:~$ ./chirpstack-v3-to-v4 --cs-config-file /home/debian/cs/configuration/chirpstack/chirpstack.toml --as-config-file /home/debian/eu868/configuration/chirpstack-application-server/chirpstack-application-server.toml --ns-config-file /home/debian/eu868/configuration/chirpstack-network-server/chirpstack-network-server.toml
2023/09/25 19:44:34 Reading ChirpStack configuration file: /home/debian/cs/configuration/chirpstack/chirpstack.toml
2023/09/25 19:44:34 Reading AS configuration file: /home/debian/eu868/configuration/chirpstack-application-server/chirpstack-application-server.toml
2023/09/25 19:44:34 Deleting users and tenants from target database
2023/09/25 19:44:34 Migrating users
2023/09/25 19:44:34 Migrating organizations
2023/09/25 19:44:34 Migrating organization users
2023/09/25 19:44:34 Migrating applications
2023/09/25 19:44:34 Migrating application integrations
2023/09/25 19:44:34 Reading NS configuration file: /home/debian/eu868/configuration/chirpstack-network-server/chirpstack-network-server.toml
2023/09/25 19:44:34 Migrating gateways
2023/09/25 19:44:35 Migrating device-profiles
panic: pq: null value in column "is_relay" of relation "device_profile" violates not-null constraint
goroutine 1 [running]:
main.migrateDeviceProfiles()
/chirpstack-v3-to-v4/main.go:1042 +0x97f
main.run(0xf32a80?, {0xa9afa7?, 0x6?, 0x6?})
/chirpstack-v3-to-v4/main.go:114 +0x57a
github.com/spf13/cobra.(*Command).execute(0xf32a80, {0xc00018a010, 0x6, 0x6})
/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:856 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xf32a80)
/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:974 +0x3b4
github.com/spf13/cobra.(*Command).Execute(...)
/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:902
main.main()
/chirpstack-v3-to-v4/main.go:79 +0x25

Please note that there is a strict relation between the v3 to v4 migration utility and the ChirpStack v4 version. It looks like you are using a migration version that didn't implement the relay feature in ChirpStack v4 yet.


fix: update ucc ui version to 1.8.4

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

v5.5.9
GitHub release

Your semantic-release bot :package::rocket:


The bot is able to move messages from bots to #help

If you use ++tohelp on a message from a bot it does actually get moved. Nothing breaks but embeds (i.e. from snipes) don't get moved and the thread title is a bit awkward. It has it's uses, such as moving code formatted by the bot (because that sometimes deletes the original message).


Remove hard coded sentry dsn

Currently the sentry dsn is hard coded and the sentry client cannot be disabled without recompiling the binary without the sentry feature flag. Leaving this in can result in unintentional data leakage and should generally be avoided.
Removing the value will let the sentry client fall back to using the SENTRY_DSN environment variable. This will enable the user to substitute it with their own dsn thus taking control of their own data. To retain the default behaviour that dsn can be moved to the Dockerfile.

Hey @Rio,
In the general idea I agree with you but I would say that it would be better to have a flag --no-sentry equivalent to the --no-analytic one that can be set to disable Sentry. I also looked at your PR (#813) and I am not convinced this is the best solution to put our Sentry DSN in the Dockerfile.
One thing that should be known here is that Sentry is here to help us find issue in the engine without leaking too much personal informations. It is important that crash reports are sent to our Sentry, always would be better.
A solution that can fix this issue would be to introduce to options to binary and not rely on any container system, people in the overall don't always use Docker:

the --no-sentry flag that can be used to disable Sentry at runtime.
and the --sentry-dsn option that can be set by the user but with a default value to our own current DSN.

What do you think about this solution?

No you're completely right. It's definitely a sub-par solution.
I understand that Sentry is there to help developers improve their software but having it default to a project that the operator doesn't have under control and that that behaviour isn't documented or logged is not really good practice.
Having a --sentry-dsn and using SENTRY_DSN to set them is better however if you're going the "default turned on and send everything to us` route at least it should be logged. Let me go over my PR to see what I can do.

@Kerollmops, I've introduced the --no-sentry and --sentry-dsn along with SENTRY_DSN and MEILI_NO_SENTRY environment variables.


todo: builtin support for language server protocol

link to https://github.com/autozimu/LanguageClient-neovim/issues/8

copied from https://github.com/neovim/neovim/issues/5522#issuecomment-284122091:
Since LanguageClient-neovim already has nice interface, I'm deprecating my previous nvim-cm-php-language-server
Here's a simple php language server package that could be easily setup to work with LanguageClient-neovim, and it is well integrated with nvim-completion-manager


Faster LENS calculation using FastNS

<oops, posted the previous pull request through my throwaway account, sorry>
Hello,
I'm an undergraduate researcher at Prof. Taras Pogorelov's lab at UIUC. I am using LENS for my research, and I found a faster way of calculating LENS parameters, so I thought I would share it. I didn't find any issues and the results were identical to your original code, can you please check as well? Thanks!

Hey @tulga-rdn , we will have a look t this soon, thank you for your interest and support!

Hey @tulga-rdn, this looks great, and like a simple 1:1 replacement. Thank you!
If you want me to merge this into the main, we will need to fix all the failed checks.

You can ignore the first ruff check, because that is something we need to change that has nothing to do with the changes you have made.
For the mypy and ruff-format checks, to reproduce those locally, you can do just check in the terminal at the repositories home if you have just installed. Otherwise you can open the justfile and find out the mypy and ruff-format commands that are used and run those.
For the pytests, there seems to be an issue with cut-offs that FastNS is stricter about than before? Feel free to modify the conftest.py to handle this issue.

Oops, it seems to still have errors. Will fix them soon

Hey @tulga-rdn, thanks for your interest, I took a look at your last commit and it seems that there is an issue with the argument _prepare_box which, if I understood correctly, is a private argument that read the box dimensions in order to choose the biggest cutoff radius possible. I cannot find the part of code for this argument, can you include that code so we can review it?

Hey @tulga-rdn  and @SimoneMartino98. I will close this for now, but feel free to reopen if either of you make progress.
Thank you


Clarify help message for '-H'?

I use qhs once in a blue moon, but I think when I used it, I often forget to specify -H because the description "skip the header row" doesn't seem like anything to me.
In my mind, nothing is being skipped -- the first row is used for either column names or row data.
Would it be clearer to change the help description from "Skip the header row" to "Treat first input line as header containing column names."? We can even change the long option from '--skip-header' to '--input-header', mirroring '--output-header'.
If it sounds good, I can send a PR.

Note that qhs command is designed to work as same as the original q command so I can't change the option name. But suggestion for changing the help description is welcome.
 % q -h | grep -A2 -- skip-header
    -H, --skip-header   Skip header row. This has been changed from earlier
                        version - Only one header row is supported, and the
                        header row is used for column naming

The original command says Skip header row.

I didn't realize qhs is a Haskell port/parallel of the Python q. I see the help message in the Python q is currently as you pointed out, and it alludes to "column names". I assume "earlier version" is irrelevant to qhs.
Maybe we can use "Skip header row for row input and use it for column names instead" to avoid changing things too much.

How about Use the first row as column names.? Assuming the row as the column names, skipping it as data is obvious.

Sure, but I thought some users might be left scratching their head about the meaning of 'skip' in --skip-header without further explanation. Personally, I didn't think of this option as skipping anything at all but just using the first row as a proper header and "skip the first line for row input" is how I made-sense-of/rationalized it.

Released as v0.3.2.


"None" in training labels

When I run the code with the command "--model_name dependency --epochs 10", there're errors like this:
Traceback (most recent call last):
  File "sentiment.py", line 228, in <module>
    main()
  File "sentiment.py", line 76, in main
    train_dataset = SSTDataset(train_dir, vocab, args.num_classes, args.fine_grain, args.model_name)
  File "/Users/runqi/Desktop/TreeLSTMSentiment/dataset.py", line 123, in __init__
    self.labels = torch.Tensor(self.labels) # let labels be tensor
TypeError: must be real number, not NoneType

Environment: python 3.6, Mac OS, pytorch 0.4.1
It seems like some root labels are None. Does anyone else have the same issue?

When I run the code with the command "--model_name constituency --epoch 10", there's another error:
File "sentiment.py", line 228, in <module>
    main()
  File "sentiment.py", line 76, in main
    train_dataset = SSTDataset(train_dir, vocab, args.num_classes, args.fine_grain, args.model_name)
  File "/Users/runqi/Desktop/TreeLSTMSentiment/dataset.py", line 102, in __init__
    temp_trees = self.read_trees(os.path.join(path, 'parents.txt'), os.path.join(path, 'labels.txt'))
  File "/Users/runqi/Desktop/TreeLSTMSentiment/dataset.py", line 155, in read_trees
    trees = [self.read_tree(p_line, l_line) for p_line, l_line in tqdm(pl)]
  File "/Users/runqi/Desktop/TreeLSTMSentiment/dataset.py", line 155, in <listcomp>
    trees = [self.read_tree(p_line, l_line) for p_line, l_line in tqdm(pl)]
  File "/Users/runqi/Desktop/TreeLSTMSentiment/dataset.py", line 183, in read_tree
    labels = list(map(self.parse_dlabel_token, label_line.split()))
  File "/Users/runqi/Desktop/TreeLSTMSentiment/dataset.py", line 166, in parse_dlabel_token
    tmp = int(x)
ValueError: invalid literal for int() with base 10: 'None'

The problems above occurred when I tried to translate the preprocessing code to python3 (by replacing "xrange" with "range" and adding "list" before "map"). I rerun the preprocessing code in python2.7 and the problems disappeared. I'll close this issue.


Teams bot not responding

Not sure why but the Teams bot is not responding to prompts. Thanks, Sam.

This is happening again. Is it something perhaps to do with fly.io ?


Fix typo in the word selection

Pending linting pass

Codecov Report

Merging #15309 (44abf5c) into main (790c909) will increase coverage by 0%.
The diff coverage is 74%.

@@           Coverage Diff           @@
##            main   #15309    +/-   ##
=======================================
  Coverage     65%      65%
=======================================
  Files        558      559     +1
  Lines      26646    26799   +153
  Branches    3840     3868    +28
=======================================
+ Hits       17322    17447   +125
- Misses      8617     8634    +17
- Partials     707      718    +11

Impacted Files
Coverage Δ

src/client/activation/jedi/analysisOptions.ts
55% <0%> (-16%)
:arrow_down:

...rc/client/activation/jedi/languageClientFactory.ts
46% <0%> (+6%)
:arrow_up:

src/client/activation/jedi/manager.ts
29% <ø> (ø)

src/client/common/application/commandManager.ts
25% <0%> (ø)

src/client/common/application/types.ts
100% <ø> (ø)

src/client/common/installer/productInstaller.ts
40% <ø> (ø)

src/client/common/platform/types.ts
100% <ø> (ø)

src/client/common/terminal/types.ts
100% <ø> (ø)

src/client/interpreter/interpreterService.ts
60% <0%> (ø)

src/client/providers/completionProvider.ts
30% <0%> (-3%)
:arrow_down:

... and 51 more


large assets are not constrained to page height

issue: when an asset is larger than the window, it spills down below the bottom of the window.  see screenshot below.

expected outcome:  when assets are taller than the page, they should be constrained to the height of the page.

@Invariant-Change would you be able to look into fixing this?


[Tensor] Support Read/Save Functionalities for Quantized Type

This PR enhances the existing functionality by supporting reading and saving operations on quantized type tensors.
This change can ensure that quantized tensors can accurately retrieve and store their data from binary files.
Self-evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test:   [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2799. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.