Load logs in reverse order or only load the last N lines

Describe the problem/challenge you have
When opening the log view, all the logs from the start of time are streamed and displayed. This is slow, so if a pod has been running for a few days it can take half an hour for the latest logs, i.e. what I'm interested in, to appear.
Describe the solution you'd like
Two solutions I can see:

loading the last N lines first, and then backfilling the rest
a switch to load only the last N lines (plus any future lines, à la tail -fn N)

Environment:

Octant version (use octant version):
Version:  0.14.1
Git commit:  51ce7f6a7e87c9cfcabc9bf402dff7eda5d6986b
Built:  2020-07-20T23:52:37Z

Kubernetes version (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.5", GitCommit:"e6503f8d8f769ace2f338794c914a96fc335df0f", GitTreeState:"archive", BuildDate:"2020-07-01T16:28:46Z", GoVersion:"go1.14.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.5", GitCommit:"e6503f8d8f769ace2f338794c914a96fc335df0f", GitTreeState:"clean", BuildDate:"2020-06-26T03:39:24Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc):
Archlinux

Thank you for reporting this.
My thoughts:
Add since support to our current log streaming.
Default that to the last 15 or 30 minutes.
Provide a drop down in the log viewer with some pre-defined options, but also allow it to take custom in put.

15 minutes
30 minutes
1 hour
3 hours
5 hours

input would be something like: 5m, 12h, 1d, all

Oh, time is an interesting concept if it can be done, yeah ---- On Thu, 06 Aug 2020 01:21:48 -1200<EMAIL_ADDRESS>wrote ----
Thank you for reporting this.
My thoughts:
Add since support to our current log streaming.
Default that to the last 15 or 30 minutes.
Provide a drop down in the log viewer with some pre-defined options, but also allow it to take custom in put.
15 minutes
30 minutes
1 hour
3 hours
5 hours
input would be something like: 5m, 12h, 1d, all
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Yeah, I was thinking time mainly because that is usually a better measure of what I want to see when I'm dealing with random logs. When I'm debugging, I don't know if the last 20 or even 100 lines will have what I want, depending on verbosity, but I know that the error happened in the last 5 minutes.
But happy to entertain other approaches.

Another request for this from our slack channel:
https://kubernetes.slack.com/archives/CM37M9FCG/p1599204923030200

Add since support to our current log streaming.

Even better, we should implement more general filtering by time by defining the startTime and endTime to our logging interface. Quite often you want to see what exactly happened around specific time when an incident occurred in the past, e.g see logs for 5 minute interval from a week ago.
The default should probably be last 30 minutes.


Update coverage docs in advanced-usage.rst

I have updated here the first two bedtools coverage examples to reflect changes in the ordering of arguments for that command.
As I do not have experience with the -abam option, I did not change the command at https://github.com/arq5x/bedtools2/commit/91628aac2b171f54cacfc38fbe6da0f508450624#diff-0146c5f9d1549f0728ce970bf905d2d5R80 , but I suspect that may need be updating as well.

Thanks!


= javadoc

update JavaDoc
update spelling for IDE

Coverage increased (+0.04%) to 74.594% when pulling ebb6cd3acb039fdebcf4ff443103c812d29636a8 on zhaoxi1988:javadoc into a7e0d359b034aa9b50490baa55e48d5e85fc74b6 on brettwooldridge:master.


Marr/Mexican hat wavelet?

it match better pictures
http://www.phash.org/demo/

Interesting, I've had the Harr wavelet suggested as well. However I cannot base my implementation on pHash because its license is incompatible with img_hash's and could possibly cause conflicts. If you can find a high-level description of the algorithm somewhere, that would be very helpful.

Recently after some thought and research I've figured out how I want to implement this. The Marr wavelet is a continuous function and so needs to be discretely sampled for the transform. Clicking around relevant Wikipedia articles, I think pHash's implementation has something to do with the Difference of Gaussians operation which effectively performs edge detection on the image.
I plan on rewriting the API to allow the Discrete Cosine Transform as a preprocessing step for the mean, gradient and double-gradient hash types, and add Difference of Gaussians as another preprocessing step that can apply to the Blockhash algorithm as well as the others. The image crate provides a Gaussian blur function so all that's necessary is to implement the difference part which should be easy.

You are welcome to copy my implementation of the Marr wavelet from https://github.com/c-edw/perceptual-hash-poc.

I'm still struggling with basic vector algebra but I'm not sure if that implementation is implementing a Marr wavelet transform. It looks like it's just multiplying the pixel values with the scaled wavelet. My uneducated guess is that what you've implemented there is just a weighted average.
The image crate already provides a Gaussian blur function which I can used to implement Difference of Gaussians, though I don't actually know what its usefulness will be with the existing hash algorithms because a mostly white image with a handful of black pixels will scale down to a flat-grey image with few features. I imagine the Blockhash algorithm would be the only actually useful one to combine with DoG as a preprocessing step.

The code there is a translated of the formula seen here.

It's possible that I missed something, but the algorithm seems to generate correct hashes. As an example, the code compares two images at largely different resolutions. The Hamming distance when using the Marr wavelet is zero.

Yes but all that does is produce the value of the wavelet at that point, then you just multiply it with the corresponding luminance values before passing it to a normal averaging hash. As far as I can tell, you've basically created a weighted mean hash that pays more attention to the center and edges and less to the intermediate areas. It's not a wavelet transform that I can see.
If you're testing with images of the exact content and aspect ratio, just different resolutions, I don't see why this hash would perform much differently than a regular mean hash.

I don't see why this hash would perform much differently than a regular mean hash.

Well, it does. The average (aHash) algorithm yields a Hamming distance of 2 in that scenario.

I'll try it. Like I said I haven't looked at the pHash source in order to avoid any IP conflicts since it has an incompatible license (like how ReactOS can't allow anyone to contribute who has seen Windows source code). That and it's no fun just copying someone else's implementation.

If we are on the hash variant train: Take a look at these sample Python implementation https://github.com/thorn-oss/perception#supported-hashing-algorithms


kvm entries update command is not accepting true/false as the key value

In release v2.2.0 the kvm entries update command is not accepting true/false as the key value.
This new rule broke the logic of our automation pipeline.
Is there any way around it or would it be possible to accept true/false values in this command again?

@andreuji - to test this out, can you please share the command you are running?

@ssvaidyanathan
apigeecli kvms entries update -m  -k teste__kvm -l true -o  --env  -a ./sa-devops.json


SOLR-16106 ClusterState and BaseHttpClusterStateProvider should not depend on ZkStateReader

https://issues.apache.org/jira/browse/SOLR-16106

Some of this was already done by @haythemkh . I'm not sure it needs a dedicated JIRA issue (compared to an overall issue for the other things for separating SolrJ ZK).

Yea, the PR #584 touches 264 source files, I forgot about it for a second, so this is perhaps duplicated.
Problem with those huge PRs is that you don't get enough reviewers. Perhaps a central git feature branch is the way to go for this, and then open several small isolated PRs against that branch, and frequently merge in main branch?

@dsmiley I'm happy to defer this, not to conflict with @haythemkh 's work. Will revisit after merge of 584 if anything is left to do.

Abandoning


networkmanager-l2tp: 1.20.16 -> 1.20.20

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nm-l2tp/network-manager-l2tp/releases.
meta.description for networkmanager-l2tp is: L2TP plugin for NetworkManager
meta.homepage for networkmanager-l2tp is: https://github.com/nm-l2tp/network-manager-l2tp
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.20.18 with grep in /nix/store/x5zgxd7by8sa76ibsqc2xbfd5dmaqvhz-NetworkManager-l2tp-gnome-1.20.18
found 1.20.18 in filename of file in /nix/store/x5zgxd7by8sa76ibsqc2xbfd5dmaqvhz-NetworkManager-l2tp-gnome-1.20.18

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

networkmanager-l2tp

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/x5zgxd7by8sa76ibsqc2xbfd5dmaqvhz-NetworkManager-l2tp-gnome-1.20.18 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A networkmanager-l2tp https://github.com/r-ryantm/nixpkgs/archive/52c3bde1f6ffcf6ec9d88279a3410a5dcca747ec.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/52c3bde1f6ffcf6ec9d88279a3410a5dcca747ec#networkmanager-l2tp

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x5zgxd7by8sa76ibsqc2xbfd5dmaqvhz-NetworkManager-l2tp-gnome-1.20.18
ls -la /nix/store/x5zgxd7by8sa76ibsqc2xbfd5dmaqvhz-NetworkManager-l2tp-gnome-1.20.18/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

networkmanager-l2tp

Maintainer pings
cc @abbradar @obadz for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nm-l2tp/network-manager-l2tp/releases.
meta.description for networkmanager-l2tp is: L2TP plugin for NetworkManager
meta.homepage for networkmanager-l2tp is: https://github.com/nm-l2tp/network-manager-l2tp
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.20.20 with grep in /nix/store/d59xi9klszqcx1ibz14xn719naxq2g3p-NetworkManager-l2tp-gnome-1.20.20
found 1.20.20 in filename of file in /nix/store/d59xi9klszqcx1ibz14xn719naxq2g3p-NetworkManager-l2tp-gnome-1.20.20

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

networkmanager-l2tp

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/d59xi9klszqcx1ibz14xn719naxq2g3p-NetworkManager-l2tp-gnome-1.20.20 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A networkmanager-l2tp https://github.com/r-ryantm/nixpkgs/archive/31e2c06e110199cb81e5abe49c27341e5844695e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/31e2c06e110199cb81e5abe49c27341e5844695e#networkmanager-l2tp

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d59xi9klszqcx1ibz14xn719naxq2g3p-NetworkManager-l2tp-gnome-1.20.20
ls -la /nix/store/d59xi9klszqcx1ibz14xn719naxq2g3p-NetworkManager-l2tp-gnome-1.20.20/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

networkmanager-l2tp

Maintainer pings
cc @abbradar @obadz for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.


Null Object reference when playing certain songs

Describe your bug here.
Null Object reference when playing certain songs on PE 6.2
Command Prompt/Terminal logs (if existing)
PlayState.hx (line 2469)
PlayState.hx (line 2425)
PlayState.hx (line 1053)
flixel/FlxGame.hx (line 637)
flixel/FlxGame.hx (line 724)
flixel/FlxGame.hx (line 677)
flixel/FlxGame.hx (line 545)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)

Uncaught Error: Null Object Reference
Please report this error to the GitHub page: https://github.com/ShadowMario/FNF-PsychEngine

> Crash Handler written by: sqirra-rng

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No, I didnt even touch the source code

Not a bug
Remove dadbattle spotlight

not a bug
the following events are hardcoded to their songs

Philly Glow (Week 3)
Dadbattle Spotlight (Week 1)

they CANNOT be used anywhere else


fix: verify account contract call displayed as ae coin

it looks like an issue, FT have a fiat value. in product they don't
https://github.com/superhero-com/superhero-wallet/assets/69896204/ed24ff5a-c545-4bf1-869c-3dac5e9504d3

Fixed, LGTM


(PUP-1401) use execute instead of execpipe

Some package providers use execpipe which does not set LC_ALL or LANG.
With this commit the pip and rpm package provider use execute instead of execpipe.
If this does not address the issue, at least execute accepts a hash with 'override_locale' and 'custom_environment' which would address locale.

CLA signed by all contributors.

I have concerns about switching to locale "C" is safe in a 5.5.z release, because PUP-1401 and PUP-1191 are both valid reasons for and against changing the locale prior to executing commands.

Okay, I thought I would take a pass at converting, since I'm familiar with those providers.
Feel free to close it!

For posterity, another difference, execute handles commands with spaces (when given either a string or argv array):
irb(main):006:0> Puppet::Util::Execution.execute("C:/Program Files/Python27/python.exe --version")
=> "Python 2.7.16\n"
irb(main):007:0> Puppet::Util::Execution.execute(["C:/Program Files/Python27/python.exe", "--version"])
=> "Python 2.7.16\n"

But execpipe doesn't:
irb(main):008:0> Puppet::Util::Execution.execpipe("C:/Program Files/Python27/python.exe --version") { |proc| proc.collect { |line| puts line } }
'C:/Program' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
        3: from c:/Ruby25-x64/bin/irb.cmd:19:in `<main>'
        2: from (irb):11
        1: from C:/Users/Administrator/puppet/lib/puppet/util/execution.rb:86:in `execpipe'
Puppet::ExecutionFailure ([nil, nil])
irb(main):009:0> Puppet::Util::Execution.execpipe(["C:/Program Files/Python27/python.exe", "--version"]) { |proc| proc.collect { |line| puts line } }
'C:/Program' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
        3: from c:/Ruby25-x64/bin/irb.cmd:19:in `<main>'
        2: from (irb):9
        1: from C:/Users/Administrator/puppet/lib/puppet/util/execution.rb:86:in `execpipe'
Puppet::ExecutionFailure ([nil, nil])


Use scopes on a Feature::forBucket() call

Hi
I would like to scope the results of a bucket; So far, I'm putting Articles in a bucket, but in case I unpublish an article that is already in a bucket, the following still returns it :
    $articles = Feature::forBucket('home_articles')->all();

What I would like to do is something like this :
$articles = Feature::forBucket('home_articles')->scopes(['published', 'visible'])->all();

.. but it doesn't seem possible. Am I missing something here ?
Thanks a lot !
Best

Hi @tchapi!
You are right, it isn't possible. The forBucket scope returns all items without conditions. To add support for Eloquent scopes, we'd need to investigate Laravel support for scoped polymorphic queries, not sure that's a thing without reverting to manual joins.
You can filter the results by checking the models "manually" but, I totally hear you, you're loosing the power of scopes... Like, published is easy enough, but visible is much more complex to verify and you can't really share code in a way that feels right with scopes and a custom accessor for example.
To be honest I would love to see something possible in Laravel: using an Eloquent scope on an Eloquent model instance to verify its compliance to it. In your case you'd filter the articles like this:
$articles = Feature::forBucket('home_articles')->all()->filter(function ($article) {
    return $article->published()->visible();
});

I'm sure this is a problem a lot of Laravel developers are running into. Unless I'm missing something? I'd be curious to hear some thoughts here.

Hello @ifox
I've thought about it a bit, and the solution I've come up with at the moment is to add a scope in my model (Article):
public function scopeBucket($query, $bucketKey)
{
    return $query->select('articles.*')
                 ->leftJoin('features', 'features.featured_id', '=', 'articles.id')
                 ->where([
                    ['bucket_key', '=', $bucketKey],
                    ['featured_type', '=', 'articles'],
                 ])
                 ->orderBy('features.position');
}

that I can use with :
$articles = $repository->bucket('home_features')->scopes(['published', 'visible']);

instead of
$articles = Feature::forBucket('home_features')->all();

What do you think of that ?
Thanks

That's definitely a viable approach, thanks for sharing! Since then, https://github.com/beyondcode/laravel-scope-checks has been released which can help as well.


[http-plugin] Refactor tests once exporter available

Now, http plugin tests use several classes for auditing spans. Once this PR is merged https://github.com/open-telemetry/opentelemetry-js/pull/149 we should remove SpanAudit, SpanAuditProcessor and ProxyTracer in order to register a fake exporter.
This should reduce/simplify code.

@OlivierAlbertini I'd like to get your reviews on #234, this should help you to get ride of SpanAudit, SpanAuditProcessor and ProxyTracer in modules.

@OlivierAlbertini #234 is merged now, if you have the bandwidth feel free to take this one  :)


docker image arm64 compatibility

What happened:
the following docker images currently do not support arm64:
[ERROR] registry.cn-beijing.aliyuncs.com/yunionio/baremetal-agent:v3.7.5
[ERROR] registry.cn-beijing.aliyuncs.com/yunionio/climc:v3.7.5
[ERROR] registry.cn-beijing.aliyuncs.com/yunionio/web-ee:v3.7.5
please update the base images.

Thanks for your report. I'll check this.

Baremetal-agent not support arm64 currently.


Failed to run on M1 with anaconda?

I can successfully build the executable but it failed when run it. Error below occurs:
dyld[53265]: Library not loaded: @rpath/libpython3.12.dylib
  Referenced from: <29BA5D08-624D-3958-993E-175668870FA1> /Users/ucag/practice/call_python_from_rust/target/debug/call_python_from_rust
  Reason: no LC_RPATH's found
[1]    53265 abort      ./target/debug/call_python_from_rust

The code is copied from the doc.
main.rs
use pyo3::prelude::*;
use pyo3::types::IntoPyDict;

fn main() -> PyResult<()> {
    Python::with_gil(|py| {
        let sys = py.import_bound("sys")?;
        let version: String = sys.getattr("version")?.extract()?;

        let locals = [("os", py.import_bound("os")?)].into_py_dict_bound(py);
        let code = "os.getenv('USER') or os.getenv('USERNAME') or 'Unknown'";
        let user: String = py.eval_bound(code, None, Some(&locals))?.extract()?;

        println!("Hello {}, I'm Python {}", user, version);
        Ok(())
    })
}

Cargo.toml
[package]
name = "call_python_from_rust"
version = "0.1.0"
edition = "2021"

[dependencies.pyo3]
version = "0.21.2"
features = ["auto-initialize"]

I can build it without any error reported, but the error appears at runtime. For inspecting the built binary, use command below
otool -L target/debug/call_python_from_rust

target/debug/call_python_from_rust:
        @rpath/libpython3.12.dylib (compatibility version 3.12.0, current version 3.12.0)
        @rpath/libiconv.2.dylib (compatibility version 9.0.0, current version 9.1.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1345.100.2)

After really searching for hours, I finally solve this issue. Maybe this can be  a documentation PR? I'm not sure. For helping people meet the same issue. The solution is below:
TL;DR;
Add a build.rs to root of the project aligned to the Cargo.toml file.
use std::process::Command;
pub fn main(){
    let python_inline_script = "from distutils import sysconfig;print(sysconfig.get_config_var('LIBDIR'))";
    let output = Command::new("python")
        .arg("-c")
        .arg(python_inline_script)
        .output()
        .expect("Failed to execute command");
    if !output.status.success() {
        panic!("Python command failed");
    }
    let python_lib_path = String::from_utf8_lossy(&output.stdout);
    println!("cargo:rustc-link-arg=-Wl,-rpath,{}", python_lib_path);
}

Then remove the target directory and run cargo build.
For people who's interested in what's going on here:
The build script just simply add rpath to the linker. From the output of command
otool -L target/debug/call_python_from_rust

we can see that the built binary using @rpath to find used dynamic libraries. Here is the point, rpath differs from LD_LIBRARY_PATH. Value of rpath will be hard coded into binary header. Thus, we need to provide linker the right rpath.
Actually, rpath should be searched in system library path.  This problem won't happen if I use python lib shipped with macOS. It just happens with anaconda. I searched a lot find out that many people have the similar issue when use with anaconda.
There are some other ways to tell the system to search for the library, like use DYLD_FALLBACK_LIBRARY_PATH, but it's not easy to use. For details, see issue 1800.
Actually, this still does not totally solve the issue. Because the library path is hard coded, when publish the compiled binary to others, it will be a problem.  This is only a just fine to work solution.


Different schema generated in local vs prod

Hi, I've been using next-roots for all my projects without any problem (thanks for the work you have put on this package!) but recently after cloning a working project to build another site I found that the schema generated locally is different than the one in production (in Vercel).
I get 9 routes locally but only 5 when deployed so when trying to do get some urls from router I just get the root route '/'
I've been trying to debug but I'm not quite sure how to approach or what to look for in this case.
Any help or guidance on how to debug it or what I might be missing/or what might be the issue?
Routes has been generated properly since I can access them directly typing the url in the browser, is just when trying to getHref or printing the schema that there are some routes missing
Thank you in advance

Can you provide a log from vercel + i18n files for missing routes?


More ways of accessing the prefix

Instead of having to do @Erin. prefix we could also have @Erin. prefixes and even just @Erin.. If we try to do something else like @Erin. help then the bot should say something like Please use the available prefixes: <list of prefixes>.We can also have -prefix list them all and also tell the user to specify add or remove to change the list. And maybe have -prefixes do the same as -prefix.
If ya like this idea I can try to implement this via a PR.

Previously users could type @Erin to get the prefix list but it got annoying very quickly when users just wanted to mention the bot without getting the prefix list, hence the @Erin prefix system was implemented. I can add @Erin prefixes too.
About the -prefix and -prefixes i personally don't feel that's a good idea so i won't be implementing that. @Erin help This can be implemented but i'm not sure how, i know it can be done but i don't think it works when the bot is listening to an array of prefixes (mongo returns an array of string prefixes which are passed to the client object)... If you can submit a PR for @Erin help for now then that would be good.

So when I run @Erin. help it should show the help or tell the user to use the prefixes?

Either way, I can just copy and paste the code that prints the prefix when you do @Erin. prefix and change the message and the command the bot is listening to.

So when I run @Erin. help it should show the help or tell the user to use the prefixes?

@Erin <commandname> could be any command, would be nice if that would get executed
if this is out of your skill set then implement the prefix idea you're talking about (idk how to get this done cuz as i said i think arrays disable the ping functionality)

Either way, I can just copy and paste the code that prints the prefix when you do @Erin. prefix and change the message and the command the bot is listening to.

Well, do what you must. That's all i'll say, this is an open source project sooooo it's pretty much whatever you want to make it unless someone comes up with a better / more efficient idea 😎


update SPARK_WORKER_MEMORY to 1g to fix resource issues on both Win11 and Linux

Introduction
Hi there, thanks for helping the project! We are doing our best to help the community to learn and practice
parallel computing in distributed environments through our projects. :sparkles:
Pull Request
Issue

*Issue number with link, e.g.: #22

Changes

High level description of change 1
High level description of change 2
...

Comments (optional)
Add some comments, if any
Checklist
Please make sure to check the following:

[ ] I have followed the steps in the CONTRIBUTING.md file.
[ ] I am aware that pull requests that do not follow the rules will be automatically rejected.

fixes #93


nativewind@next requires outdated package<EMAIL_ADDRESS>

Describe the bug
For the most part, nativewind seems to be working quite well with expo 49. However, I'm having issues with my react-native-web NextJs port (using solito).  This issue is coming from being unable to use the latest version of @expo/config-plugins.
Currently, nativewind@next seems to require<EMAIL_ADDRESS>which requires me to use an outdated version of @expo/next-adapter, which in turn requires me to use an outdated version of NextJS.
Expo 49 is looking for<EMAIL_ADDRESS>so I suppose I am asking for nativewind to either support this package or to provide some workaround.
Again, Expo app is working fine on Android and iOS with<EMAIL_ADDRESS>(at least in expo go on my macbook),
the issue is with using NextJS with react-native-web.
To Reproduce

Create Expo@49 project with nativewind@next
npx expo install --fix
yarn why @expo/config-plugins => 5.0.4 instead of 7.2.2
Create<EMAIL_ADDRESS>project with react-native-web and nativewind@next
Add<EMAIL_ADDRESS>Next app runs into several issues, mainly being that plugins are not added properly from next.config.js

Expected behavior
nativewind@next should be compatible with the latest @expo/config-plugins

Realizing this also may just be an issue of not knowing how to setup my next.config.js in order to properly transpile the myriad expo packages in my project which say they are web compatible, but are just not ready for next 13.2
Any insight from folks using<EMAIL_ADDRESS>is appreciated 🙌

This is resolved in v4
It will be releasing soon. The documentation is here if you wish to update early

Great! Looks like this issue is in fact solved with v4, just tried it on a new branch. Excited for the release!


flutter deploy does not work - empty page displayed, main.dart.js is missing

to deploy a flutter app to firebase hosting I used command : firebase deploy --only hosting
It finished succcessfully : firebase deploy --only hosting
And I got my hosting url.
But when I open the url, empty page is displayed.
Using developer console I can see that /main.dart.js is missing.
I could find the file localy in ./build/web/main.dart.js
why it was not deployed ?
`
flutter doctor -v
[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on Mac OS X 10.15.7 19H114 darwin-x64, locale sk-SK)
• Flutter version 1.24.0-10.2.pre at /Library/flutter
• Framework revision 022b333a08 (7 weeks ago), 2020-11-18 11:35:09 -0800
• Engine revision 07c1eed46b
• Dart version 2.12.0 (build 2.12.0-29.10.beta)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /Users/roman/Library/Android/sdk
• Platform android-30, build-tools 30.0.2
• Java binary at: /Users/roman/Library/Java/JavaVirtualMachines/corretto-1.8.0_275/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment Corretto-8.275.01.1 (build 1.8.0_275-b01)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.2, Build version 12B45b
• CocoaPods version 1.10.0
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
[✓] IntelliJ IDEA Community Edition (version 2020.3)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 52.0.5
• Dart plugin version 203.5981.152
[✓] Connected device (2 available)
• Web Server (web) • web-server • web-javascript • Flutter Tools
• Chrome (web)     • chrome     • web-javascript • Google Chrome 87.0.4280.88
! Doctor found issues in 1 category.
`

Hey @RomanGajdosSk ,
This issue tracker is used for tracking bugs and feature requests for the flutter framework, engine and the flutter tool.
Please see https://flutter.dev/community for resources and asking queries doubts and assistance on personal code. You may get some help if you post it on StackOverflow and if you need help with your code, please check https://www.reddit.com/r/flutterhelp/.
Also, make sure that you have deployed the build/web folder only.
Thanks

Hey @RomanGajdosSk ,
This issue tracker is used for tracking bugs and feature requests for the flutter framework, engine and the flutter tool.
Please see https://flutter.dev/community for resources and asking queries doubts and assistance on personal code. You may get some help if you post it on StackOverflow and if you need help with your code, please check https://www.reddit.com/r/flutterhelp/.
Also, make sure that you have deployed the build/web folder only.
Thanks

Hi @RomanGajdosSk
Your probably deployed app/web folder instead of app/build/web folder
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

Hi @RomanGajdosSk
Your probably deployed app/web folder instead of app/build/web folder
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


In chart.js added bgimage in barchart but the img.src is working only in repeat not working in  repeat-y

var img = new Image();
img.src = 'assets/images/graphpattern.png';
img.onload = function() {
var ctx = document.getElementById('myChart').getContext('2d');
var fillPattern=ctx.createPattern(img, 'repeat');repeat-y not working==>ctx.createPattern(img,'repeat-y');
var chart = new Chart(ctx, {
type: 'bar',
data: {
labels: ['Item 1', 'Item 2', 'Item 3', 'Item 4'],
datasets: [{
data: [5,20, 35, 45],
backgroundColor: fillPattern
}]
}
})
}
http://codepen.io/shan31/pen/OpYRmN

Hey @shan31 I don't think this is a chart.js issue since we don't touch anything specific about CanvasPattern objects. I think you'll get a better answer on StackOverflow


Camera chuangmi.camera.029a02 Video streaming

I managed to add the camera, but only the options to turn it on or the led appear. But the video does not appears. Is that any extra config to enabling streaming?
Thanks for this wonderful job!

Hi,
i have already briefly explained here why at this point it is not possible: https://github.com/merdok/homebridge-miot/issues/365#issuecomment-1288211712
Maybe if someone has time to implement that then it would be nice, other then that i might someday come back to it and try again...


Fixed error reporting showing 'file is not defined'

Hi!
The file variable should be set outside of the try statement because it is used in the catch error reporting. So when there's an error parsing a file I will only get:
ERROR in ./~/react-styleguidist/loaders/props.loader.js!./src/components/modal/prompt/Prompt.js
Module build failed: ReferenceError: file is not defined
    at Object.module.exports (/Users/adrien/Projects/dev-setup/tray/src/github.com/trayio/react-ui/node_modules/react-styleguidist/loaders/props.loader.js:41:66)
 @ ./~/react-styleguidist/loaders/styleguide.loader.js! 4:19905-20030
 @ ./~/react-styleguidist/src/index.js
 @ multi main

and after this fix I get the right error message:
Error when parsing src/components/modal/prompt/Prompt.js
Error: No suitable component definition found.

Thanks!


Fix serialization of non-int-based enum instances

Also applies to pickling.

There is. It's type.IsPrimitive 😄

Well, it turns out the test !type.IsPrimitive is not right by itself. decimal and DBNull are not primitive. I'll submita PR soon.

Ah right, didn't notice those were the list.


Error when adding duplicate identifiers in CSS objects

Bug report
Describe the bug
Doing a 100vh fix based on this article, https://css-tricks.com/css-fix-for-100vh-in-mobile-webkit/ , and trying to put two rules , as instructed, I get an error.
An object literal cannot have multiple properties with the same name in strict mode.ts(1117)
Duplicate identifier 'minHeight'.ts(2300)

To Reproduce
stitchesConfig.globalCss({
  body: {
    margin: 0,
    minHeight: '100vh',
    minHeight: '-webkit-fill-available',
  },
  html: {
    height: '-webkit-fill-available',
  },
});

Expected behaviour
I'm not sure why this is required, but in this scenario I'd expect to be able to pass an array in order to set multiple rules for a single CSS property. Similar to: '@font-face': [{}, {}, {}], I'd expect to be able to do :
body: {
  minHeight: ['100vh', '-webkit-fill-available']
}

Screenshots

Thanks for raising this issue. Unfortunately, that's a limitation of the JS object syntax.
With the styled/css APIs, we allow n Style Objects arguments. That way, you can pass them as separate args and Stitches will merge them for you:
const Button = styled('button', { minHeight: '100vh' }, { minHeight: '-webkit-fill-available' })

But the globalCss function only accepts one. I'm thinking whether it's a good idea to allow n as well.
Would that be a good solution for you?
Meanwhile, if you need this to work today, you can do:
globalCss({
  body: {
    // hack into the object string as raw CSS
    minHeight: '100vh; min-height: -webkit-fill-available;'
  }
})

Note: styled-components allow infinite args for their global css API https://codesandbox.io/s/funny-frog-txtcm?file=/src/App.tsx
/cc @jonathantneal

const Button = styled('button', { minHeight: '100vh' }, { minHeight: '-webkit-fill-available' })

globalCss({
  body: {
    // hack into the object string as raw CSS
    minHeight: '100vh; min-height: -webkit-fill-available;'
  }
})

These make sense and work for me, thanks!

I’m open to supporting n styles in globalCss. I’ve made a PR here for folks to review, should we want this. https://github.com/modulz/stitches/pull/762

Hi! This has been released to Stitches v1.1.0
yarn add @stitches/react@latest
# or
npm install @stitches/react@latest


Completion needs option for case sensitivity

Currently case sensitive. Default should probably be insensitive.

closing - duplicate of 123


Issue176 support python versions

Summary: Issue176 support python versions
Related issue: Closes #176
Suggested reviewer(s): Johannes
Reviewer checklist:
Ensure that we now support Python versions 3.6..3.10 for both unit testing and CI.

Think we actually have to delete and remake the pull request to fix the hanging pyTestCov? Or @mortenwh could force merge this?

If it works locally, maybe it's ok to just merge it...
ons. 16. nov. 2022 kl. 09:15 skrev Johannes Tobiassen Langvatn <
@.***>:

Think we actually have to delete and remake the pull request to fix the
hanging pyTestCov? Or @mortenwh https://github.com/mortenwh could force
merge this?
—
Reply to this email directly, view it on GitHub
https://github.com/metno/py-mmd-tools/pull/193#issuecomment-1316570591,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA2UBOREURPO5ZZH54WEOTWISJ27ANCNFSM6AAAAAAR3OSHX4
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Morten Wergeland Hansen, PhD
Senior scientist / Data Manager
Meteorologisk Institutt / Norwegian Meteorological Institute
T.: (+47) 915 47 844


CreateSessionRequest and CreateSessionResponse don't use message header / chunking mechanism

Attached is a PCAP file raising two exceptions for frame numbers 10 and 13 which cause protocol violations.
After the protocol violations no more OPC UA Logs are generated.
It looks like the OPC UA Parser is expecting an OPC UA Message Header in each frame.
This is not always true. In the attached PCAP, you can see an OPC UA CreateSessionRequest (frame number 9 and 10) with a message size of 1578 Bytes.
The same problem can be observed for the CreateSessionResponse Message (frame number, 12, 13, 14, 17, 18, 23, 26, 27, 30) with a message size of 133900 Bytes.
In both cases, the chunking mechanism of OPC UA is not used (Chunk Type: F).
PS: There is an "unusal" tcp handshake at the beginning of the PCAP causing a “possible_split_routing” message. This is an artefact from our environment, but it has no effect on the described problem.
opcua.pcap.zip

The initial implementation of the parser used a binpac datagram flow definition which results in faster parsing, but does not support incremental input or buffering.  Given the OPC UA data in the provided pcap has data fragmented over multiple frames, the parser has been updated to use a binpac flowunit flow which buffers data when there is not enough to evaluate the record and dispatches data for evaluation when the threshold is reached.
Testing:
Testing of the updated parser revealed an issue with the provided pcap that limited verification of the changes.  The updated parser successfully parsed the CreateSessionRequest (frames #9 and #10), but due to a TCP Spurious Retransmission at frame #26, parsing of the CreateSessionResponse caused the data to be undelivered (OPCUA_Binary_Analyzer::Undelivered(uint64_t seq, int len, bool orig)) which sets the had_gap flag.  Once this flag is set, further processing of data is disabled when new streams are delivered (OPCUA_Binary_Analyzer::DeliverStream(int len, const u_char* data, bool orig)).
Researching Zeek and TCP Spurious Retransmission turned up Zeek issue #1564 which explained that "Zeek assumes that the traffic that it parses is in the same order as it happened."  See the References section below for further details.  Additionally, reviewing the source code for some of the built-in parsers (RDP, SSH, SSL) showed the same code path when the had_gap flag is set.  It appears this behavior is normal for Zeek parsers
In an effort to verify the CreateSessionResponse parsing up to frame #26, temporary debug statements were put in place that showed the data being processed and buffered.
Implementation:
Implementing the binpac flowunit flow was not as simple as changing from a datagram reference to a flowunit reference in the opcua_binary.pac code base.  This small change would cause the binpac compiler to issue various errors such as "cannot determine initial buffer length for type", etc.  After much digging and gnashing of teeth, three areas in the binpac definitions were causing the compiler errors.

The compiler did not like the bytestring definitions of the msg_type and is_final fields in the Msg_Header record.  These fields were changed to be uint8[3] and uint8 which caused additional ripple effects throughout the code base.
The Msg_Header record required an &length definition based on the msg_size field from the header.
The compiler did not like specifying a &length definition on the OPCUA_Binary_PDU record.  This record has been removed and the Msg_Header record updated to take an is_orig: bool parameter.

For more details on the code base changes, see the commit history of the repository.
References:

https://github.com/zeek/binpac#flow
https://github.com/cisagov/icsnpp-opcua-binary/blob/main/src/OPCUA_Binary.cc#L83-L88
https://github.com/cisagov/icsnpp-opcua-binary/blob/main/src/OPCUA_Binary.cc#L60-L63
https://github.com/zeek/zeek/issues/1564
https://github.com/cisagov/icsnpp-opcua-binary/blob/main/src/opcua_binary.pac#L32


Fixed the Dark mode (issue #89)

Related Issue
The toggle button was not working and the other pages doesn't have the dark mode button.
Closes: #89
Are you creating this PR under Hacktoberfest
Yes, I'm opening this PR under hacktoberfest.(Please remove this section if you are not a hacktoberfest participant)
Describe the changes you've made
I've fixed the dark mode not working in the landing page. I also added the dark mode toggle to all pages and made it functional.
(NOTE : In previous PR I have added the individual dark mode event listener scripts into their respective htmls. I've tried to add them into main.js but it was showing errors so I put them into individual pages . But there was many redundant scripts so  I closed the PR and created a new branch and only added the contact page's event listener into the html and now other pages can access the event listeners via main.js )
Describe if there is any unusual behavior of your code(Write NA if there isn't)
A clear and concise description of it.
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.

Screenshots
Put any screenshot(s) of the project here.

@front-runner-sd Thank you for contributing : )


boltsync only shows 30 projects in dropdown

Describe the bug
I went to https://boltsync.mystify.tech/ to sync my bolt project with github. My bolt project is public so I expected to see it. But is seems like the drop down for repository only shows a certain number of projects, there is no way for me to scroll further down (yes I have quite a few projects) The dropdown only shows 30 projects.
Link to the Bolt URL that caused the error
boltsync.mystify.tech
Steps to reproduce
Set up a project in bolt
open stackblitz and upload it it to github
Go back to bolt and make more changes
Open boltsync.mystify.tech to upload latest changes.
Only 30 projects are visible in dropdown
Expected behavior
I expected all public projects to be visible in the dropdown and not just 30
And as future enhancements it would be nice to have access to private projects too
Screen Recording / Screenshot
No response
Platform
Chrome
Additional context
No response

Appreciate the feedback! It looks like that application was developed by a member of the community, which is awesome! I would reach out the them for feedback regarding their project. The team is hard at work improving GitHub integration, which is being tracked here: #2367.

Okay thanks, I had no clue., then just please close this one, looking forward to a more official integration one day!

FYI: I found the person that created him and sent him an email with the issue

Sounds great!

Update 2: He responded and fixed the issue, the api limited the response to 30... He fixed the issue and I've verified it with my repo :)

Awesome, glad to hear it!


Feature request - Make angularfire2 compatible with Typescript 2.9.2 or later

Version info
Angular: 6.1.7
Firebase: 4.2.1
AngularFire: 5.0.0-rc.11
** Node:** 9.5.0
** OS**: Windows 10
** Screenshots **
Expected behavior
Changing my package.json file to angularefire2 to 5.0.0 version and doing npm install, whereas my Typescript version is 2.9.2, shows a warning that no package was installed because angularfire2 is ok with
typescript >= 2.7.2 & < 2.8.0.
Actual behavior
I have an app that runs with Typescript 2.9.2 and angularfire 5.0.0-rc.11 and that works fine.  I installed first angularfire2, then upgraded my typescript version. The app is ok. The problem is when I try to upgrade to version 5.0.0.

I'll take a peak, but for right now we're keeping it inline with Angular.

Thanks @jamesdaniels, but isn't Angular compatible right now with typescript 3 ?
See
https://github.com/angular/angular/blob/7f1cace2a204aa2e47f9441eb8745a500bdc505a/package.json#L128
Am i misunderstanding something ?

@johanchouquet I believe the angular master branch is ahead of the releases. I don't think angular currently supports typescript 3, nor will it until angular 7.

@thefliik , maybe. But again, I still have my Angular 6.1.x application running with typescript 2.9.2 without any problem with angularfire2 (v5.0.0-rc.11).
And for Angular CLI, it'll run v7 with support for typescript 3.1 at the very least:
Angular CLI v7
I should be able to upgrade to v5 final & latests, not just because of my typescript version.

@johanchouquet Angular CLI v7 also is not released. As of this posting, they recently released v7 rc0. Angular CLI's major versions are released in lock step with Angular's major verson's (same goes for Angular material's major versions). So none of the official, stable releases support typescript 3 (so far as I know). I also run angular 6.x with typescript 2.9.2, but I'm actually not sure if 6.x officially supports typescript 2.9 even. I think the latest currently supported typescript is 2.7.x (I see a console warning when I install Angular 6.x in a project using typescript > 2.7). Typescript 2.9.x seems to work anyway though (I haven't had any problems).
I'd like to upgrade to typescript 3, so I've been trying to keep an eye on the angular releases to see if they added support. Haven't seen anything yet though.

Angular v7 RC.0 stepped up to TypeScript 3.1.1. It would be nice to support that in this library so that users of @angular/fire can test the Angular RCs.
It would also be nice to release a version that supported the Angular v7 RCs. Currently only ^6.0.0 is supported in 5.0.2.

Note that the current 5.0.2 release does seem to work with Angular 7.0.0-rc.1. It gives peer dependency warnings but the app still builds and runs.

Angular Version 7.0.0 is dependent on Typescript 3.1.
See blog: https://blog.angular.io/version-7-of-angular-cli-prompts-virtual-scroll-drag-and-drop-and-more-c594e22e7b8c
Or Release notes: https://github.com/angular/angular/blob/master/CHANGELOG.md#700-2018-10-18

Great! I'll give this a try anytime soon ! Thanks for this update @jamesdaniels .


Added rudimentary +,-, >, <, == operators for DateTime objects and tests

I have added <, >, and == operator overloading for the DateTime class.
These comparison operators compare the year, month, day, hour, minute and seconds fields directly, because unixtime only matches if the timezone is set to UTC.
I have also added + and - operators with the following behaviors:

operator takes a DateTime (LHS) and a number of seconds (RHS) and returns a new DateTime that many seconds greater than the LHS. The RHS must be >= 0, because adding negatives causes underflow with the underlying UnixTime variable.

operator returns the difference in seconds between two DateTimes. This value can be negative, meaning LHS is chronologically before the RHS.

@namanvs , Thank you for contributing! I look forward to reviewing your code.
To you and @awickert : Please allow me a few days to complete work on an initial set of documentation for existing functions before reviewing these DateTime enhancements. It will not take long. The writing is basically finished. I am in that pre-push phase of self-criticism and fine-polishing.  I might include a couple of example programs for working with the timer outputs on the 32K and the INT/SQW pins.
By the way, I have already uploaded and merged some documentation on the DateTime class. Based on testing after some recent PRs, it seems to me that the issues raised in the past about this class have been addressed and resolved. Please take a look at it in the "Documentation" folder. I will be grateful for the benefit of your thoughts.
David

Hi @namanvs
cc: @awickert
I had some time on Sept 17 to look at the operator overloadings in this PR.
Must confess I might not be the best person to assess operator overloading.
Looking at the implementations, I found a lot in this PR that might benefit from further study. Maybe best to take it one thought at a time.
For example...
The range of dates supported by DateTime may warrant consideration for the "+" overload function. See the 2106 Problem. Perhaps some added code might be warranted to constrain the valid range of the return value?
Thanks,  David

I am trying to understand the logic in the ">" and "<" overloads.
We have expressions such as the following
this->year() > rhs.year() || this->month() > rhs.month() || ...

which could be abstracted as boolYear || boolMonth || ....
Each of the six comparisons can evaluate to (true) or (false). The six values are consolidated into a single boolean value by applying successive logical OR operators to them, one after the other.
I ran a simulation assigning (true) and (false) randomly to six booleans then performing the same calculation. A series of trials of six booleans and the result of sequential "OR" operations on them is shown below. Is this the desired outcome?
If not, what changes to the code for these two overloaded operators could ensure correct comparisons of two DateTime instance objects?

Six boolean values       result
---------------------    ------
0 : 0 : 1 : 0 : 1 : 0       1
0 : 0 : 1 : 1 : 0 : 0       1
1 : 1 : 0 : 1 : 1 : 1       1
0 : 1 : 1 : 0 : 0 : 0       1
0 : 0 : 0 : 0 : 0 : 0       0
1 : 0 : 1 : 1 : 1 : 0       1
1 : 0 : 0 : 0 : 1 : 0       1
0 : 1 : 1 : 0 : 0 : 0       1
1 : 0 : 1 : 1 : 1 : 0       1
0 : 0 : 0 : 1 : 0 : 0       1
1 : 1 : 0 : 1 : 0 : 0       1
1 : 1 : 0 : 1 : 1 : 0       1
0 : 0 : 0 : 1 : 0 : 0       1
0 : 1 : 0 : 1 : 0 : 1       1
0 : 1 : 1 : 0 : 0 : 0       1
0 : 1 : 0 : 1 : 1 : 0       1
1 : 1 : 1 : 1 : 0 : 0       1
1 : 1 : 1 : 1 : 0 : 1       1
1 : 0 : 1 : 1 : 0 : 1       1
0 : 1 : 0 : 0 : 1 : 1       1
0 : 0 : 1 : 0 : 1 : 1       1

By the way, the unixtime() method of the DateTime class does not take into account the setting of a "system clock" nor does it access any information about UTC or any other time zone.
I believe that the only necessary condition for comparison to be valid between two DateTime objects is that the program writer ensures the values they contain reflect the same time zone, which need not be UTC.
If my belief is correct, then might these comparison overloads simply examine the unixtime() results of the two DateTime objects?

My initial goal with this PR was to add richer comparison operations for DateTime objects that let the programmer think in terms of calendar units (minutes, hours, days, months and years) instead of always having to drill down to the unix time uint32s, but it seems like I bit off more than I can chew, especially when considering issues like how months have differing numbers of days. Perhaps directly comparing unix time is the easier approach, especially if we include timezone handling, which I am personally all for, but otherwise, the comparators as they stand basically provide a deeper comparison of DateTime classes.
My intention with the '>' and '<' was to use the short-circuiting behavior of chained OR checks. Each expression is evaluated from left to right, so the first OR that evaluates as TRUE stops the evaluation of subsequent OR checks. Therefore setting the precedence of each check in order of largest to smallest unit of time should result in the expected behavior.
For example, A>B?
First we check A.year() > B.year():
If A.year() > B.year(), then A > B, so the evaluation stops and returns TRUE. Otherwise, we are still not sure of the result of the A>B, so we check the next OR condition, which is A.month() > B.month(), and so on...
In your test of 6 random booleans, because of short-circuiting, only the first boolean that evaluates to TRUE causes the result to evaluate as TRUE. The rest are actually of no consequence. However, in any case your table of results is the desired outcome.
I was mulling over creating a Duration class to facilitate much more natural DateTime operations, e.g:
Duration has Years, Months, Days, Hours, Minutes, Seconds, and
DateTime - DateTime = Duration
DateTime + Duration = DateTime
DateTime % Duration = Duration
But all this seemed to get too hairy and too complex for something running on a microcontroller. At least the current implementations of +. -, >, <, == should provide some utility for people creating software timers ( + can be used to set the next time an event should fire), measuring time elapsed ( -  operator), or checking if a particular DateTime has been reached or exceeded with the comparators (<,>,==).
I could also easily extend the set of operators by composing:
A >= B can be written as A > B || A ==B
and  A != B as !(A==B)
but I'll leave this decision to you.

This is a good and thoughtful conversation. Thank you! Let's keep it going.
I appreciate your point that the absence of time zone information somewhat limits the DateTime class. Well, that might be the way things are at the moment.
Given the limitation we would need to emphasize to users that operations on two DateTimes are valid only in the situation where both operands represent time in the same time zone.
I came in here to add some thoughts about two more operators.
Firstly, my thought on the "==" overload would take me back to suggesting comparison of the two unixtime() method outputs, given the lack of access to time zone information.
Also, the "-" overload looks fine to me. Again, the return value embeds the assumption of identical time zone.
I really want to understand the logic better in the ">" and "<" overload functions. It appears that you and I might be interpreting the order of computation differently. My understanding is that a calculation written thus:
return A || B || C || D || E || F; will always evaluate all six values. It must do so, because logical OR is commutative, meaning that changing the order of operation does not change the result. It might even be possible that the compiler would not guarantee to perform the calculations in the same order as written in a long sequence like that.
But I do see that you interpret it differently.
I believe you and I conceive the the intended logic of the matter similarly, even if we might elect different syntax to encode it. Will you allow me to attempt your idea in my words, so we can look at it together?
Something along the following line of pseudocode would illustrate my understanding of the logic result you described for two DateTime instance objects A and B:
if (A.year > B.year) return true; // end of comparison because A > B for sure
if (A.year < B.year) return false; // another, definite end of comparison because A < B for sure
// otherwise, year == year and  we don't know yet, need to check month next.

if (A.month > B.month) return true; // done!
if (A.month < B.month) return false; // likewise done!
// otherwise, year == year and month == month, we still don't know, check day...

if (A.day > B.day) return true;
if (A.day < B.day) return false;
// otherwise, year == year and month == month and day == day
... and so onward to hour, minute and second

All that lovely logic having been exercised, would the program writer using the comparison operator even know whether the implementation was performed with timestamps, rather than with six successive sets of comparisons?

I hasten to add, regarding the previous post, that the pseudocode listing at the end of it was for the ">" overload. The same logic would work for the "<" overload function, but reversing the return values.

I compile the following code and receive a result that is contrary to what @namanvs was hoping.
Notice that only the "month" value of A is greater than its counterpart in B. All other values in A are either less or equal to those in B. In particular, the "year" in A is less than that in B.
Yet, alas...
#include <DS3231.h>
DateTime A(2021, 5, 25, 6, 0, 0);
DateTime B(2022, 4, 26, 7, 0, 0);
void setup() {
  Serial.begin(9600);
  bool C = A > B;
  if (C) {
    Serial.println("A is greater than B");
  } else {
    Serial.println("A is NOT greater than B");
  }
}

Result: "A is greater than B".
This result appears to support a contention that Logical OR is commutative and the ">" overload function as written evaluates all six date and time element comparisons.

Well, I feel like a dunce for misunderstanding how short-circuit behavior works...
How about this? I will re-work this code so that comparators simply look at UNIX time. I will add support for >=, <=, and != as well.
Regarding timezone support. It might just be easiest to add an enum to the DateTime class with all the UTC offsets, and have it as an optional parameter in all the constructors (defaults to 0, i.e. UTC time). Calling the unixtime() function will report the current UNIX time in UTC. I.e. if your timezone was UTC +4, unixtime() would subtract 4 hours from the value obtained after adding SECONDS_FROM_1970_TO_2000.

@namanvs Oh! I know how you feel. Want to see a really big collection of dunce hats? Take a look at mine!
Iteration is how we make open source software. It is also how we all learn. Who never errs, has stopped trying. Who never loses the path, gains no knowledge of the world it passes through.
Let's take things a step at a time. A robust set of operator overloads on the existing data structure of the class could add some value. Please do complete those, as you propose.
The idea of altering the data structure to introduce time zones best belongs in a separate PR.
Many thanks,
David

@namanvs : Please also strengthen the "+" operator overload function to guard against sums exceeding the design range of the DateTime class.
Firstly, it is necessary, but not sufficient, to guard against overflowing the 32-bit unsigned integer return type.
Secondly, the sum of this->unixtime() + rhs should be protected against overflowing the 8-bit year-offset (yOff) property of the class.  This means the maximum sum must not exceed<PHONE_NUMBER> = 23:59:59 (p.m.) December 31, 2099.
Note that a sum of<PHONE_NUMBER> would overflow the yOff property to 00 from 99. A new DateTime instance constructed on such a sum would have no meaningful value in relation to the two parameters fed into the "+" overload function.
The "+" operator overload should implement appropriate safeguards against sums exceeding the value<PHONE_NUMBER>.

@namanvs Today it is my face that turns red with embarrassment. Please allow me to modify my request.
In the previous comment I remarked that a Unix-style timestamp value of<PHONE_NUMBER> would overflow the yOff property of the DateTime class to 00 from 99. That is not correct.
The yOff value will be set to 100, not zero.  Note that the DateTime year() method will return 2100 because it adds 2000 to the yOff value.
Likewise, the getYear() method of the DS3231 class will return 100 after setting the DS3231 hardware with setEpoch(4102444800).
It might not be strictly necessary to constrain timestamp sums to values less than<PHONE_NUMBER>. Even so, I believe it still makes sense to prevent overflowing the 32-bit integer. The maximum timestamp comparable to that of the present day would be 232 – 1 =<PHONE_NUMBER> ( = 06:28:15 Feb 7 2106).
Thanks,
David

@namanvs
Couple of comments...
UINT32_MAX is already defined, for AVR architecture at least, in the core library stdint.h.
It surprised me to observe that the compiler would not object to assigning an integer constant, e.g.,<PHONE_NUMBER>, to a DateTime class instance. The following is essentially what would happen in your proposed code:
DateTime myDT =<PHONE_NUMBER>;
... and it compiles without so much as a warning! That strikes me as very odd.
Even more puzzling to me, that instruction executed quietly when I tested it on a Nano.
Wearing my documentation-writer hat, let me ask you two questions:
What time, date, and unixtime() values did you expect to result in the DateTime instance following this assignment, and what values did you actually find?
How should users of this overload function formulate their code to detect the "overflow" result?

@namanvs
It seems likely that the compiler behavior described in the previous post results from having a constructor that accepts a single integer argument.
Consideration is being given to marking that constructor "explicit", which may prompt the compiler to regard assigning an integer as an error.
To prepare for this possibility, and for compatibility with the function's declared return type, please consider modifying your "+" overload function to return a datetime object rather than an integer.  For example, using the value you propose,
"return DateTime(UINT32_MAX);.
It would then be possible to tell users what date and time values to expect in the event of an overflow. The community can decide later whether such a result is what they want.
Thanks!

@awickert @jnuernberg @hasenradball
I am asking others to step in and take over the consideration of this PR #71 for two reasons.
Firstly, while it remains under development, unfortunately I need to reduce the amount of time I devote to this repo. The PR will move forward more quickly if someone else steps in.
Secondly, I don't feel qualified to judge how much value the PR adds to the library. The "+" operator as written looks problematic to me. The other operator overload functions boil down to one-line code statements that a library user could as easily write directly.
Thanks,  David

Hi David,
I will have a look in the PR the check what the Topic is abaout.


Django docker isn't working

Hey, I've followed the steps listed under the installation process, and when I ran the "docker-compose up" everything worked fine other than the Django docker. It exited immediately.
I ran "docker logs orochi_django" to check why might it exited, and I saw this:

Can you help me understand why it happened and fix this issue?

Sorry, was my fault.
We are working to add automatic symbol download (actually in develop under banner_support branch) and I added new field in admin.
The latest pull should resolve the issue.

Confirm now is working


[Issue] Install AZD pipeline task depends on a version of node that is end of life

The setup-azd@0 task now causes this warning to appear in Azure DevOps pipelines, advising that it relies a version of Node that is end of life:
##[warning]Task 'Setup Azure Developer CLI' version 0 (setup-azd@0) is dependent on a Node version (10) that is end-of-life. Contact the extension owner for an updated version of the task. Task maintainers should review Node upgrade guidance: https://aka.ms/node-runner-guidance
Can the task please be updated.

@markwragg , I have released version 0.14 https://marketplace.visualstudio.com/items?itemName=ms-azuretools.azd
Can you please check if the Warning is gone?  (It might take a little longer for me to run a e2e test and validate)


Remove time-series hacks to support histogram

They were necessary to move forward with #786
Once https://github.com/CartoDB/Windshaft/issues/405 is resolved they can be removed

merged in #901

:smile: thanks again!


Feature/multiple trip dates

Add the possibility to have multiple dates on one trip

We'll be merging this branch even though building is broken.
We'll fix it in a separate branch.


Server won't recognize user

I did a little testing with connecting to dwarftowers.com using the HTML client currently in the lobby branch.
First, I opened up the Java client, and logged in. Success.
Then, I opened up the HTML client, and logged in. Nothing.
The server chat did not log a message when I logged in via the HTML client. However, when I logged off, it said something along the lines of:

has logged off

Without the username. Perhaps this has something to do with the current issue in the client where you have to rapidly double click the login button in order for the the page to successfully change, which in the end sends at least two login messages.

This is sort-of a double bug issue.

close #22

resolves #22


Replace test image URL; original is no longer available

The wisc.edu baboon image URL, used in some unit tests, is no longer available. This PR switches the URL to a Wikimedia image of roughly similar size.
@beijbom : I was working on a small change to spacer, in hopes of stopping error emails from deploy-requests of unreachable image URLs, but I ran into this issue first. Do you think you'll be able to review infrequent spacer PRs like this for the foreseeable future? (This repo requires 1 approving review to merge PRs.)

Yeah. No problems! Approved.

I'm actually going to make this change part of PR #48 instead, so that it's just one PR which fixes the spacer build. Apologies for the mess.


[Improvement] Add landscape layout for SimpleSlide

This landscape layout would have the text and description on the left and the image on the right, which would also partially fix #11...

Great idea!

Thanks :)


from torch-cluster import fps报错

from torch-cluster import fps报错，其他模块均按照readme要求安装
pip install torch-cluster==latest+cu101 -f https://pytorch-geometric.com/whl/torch-1.5.0.html安装失败，无法找到对应包，所以直接使用的对应版本链接whel安装 pip install https://data.pyg.org/whl/torch-1.5.0/torch_cluster-latest%2Bcu101-cp36-cp36m-linux_x86_64.whl
但是运行时报错找不到模块
(torch1.4.0) ubuntu@ubuntu-System-Product-Name:/data/NZH/attMPTI-main$ bash scripts/train_attMPTI.sh
Traceback (most recent call last):
File "main.py", line 100, in
from runs.mpti_train import train
File "/data/NZH/attMPTI-main/runs/mpti_train.py", line 10, in
from runs.eval import test_few_shot
File "/data/NZH/attMPTI-main/runs/eval.py", line 14, in
from models.mpti_learner import MPTILearner
File "/data/NZH/attMPTI-main/models/mpti_learner.py", line 10, in
from models.mpti import MultiPrototypeTransductiveInference
File "/data/NZH/attMPTI-main/models/mpti.py", line 11, in
from torch_cluster import fps
File "/home/ubuntu/anaconda3/envs/torch1.4.0/lib/python3.6/site-packages/torch_cluster/init.py", line 13, in
library, [osp.dirname(file)]).origin)
File "/home/ubuntu/anaconda3/envs/torch1.4.0/lib/python3.6/site-packages/torch/_ops.py", line 106, in load_library
ctypes.CDLL(path)
File "/home/ubuntu/anaconda3/envs/torch1.4.0/lib/python3.6/ctypes/init.py", line 348, in init
self._handle = _dlopen(self._name, mode)
OSError: libtorch_cpu.so: cannot open shared object file: No such file or directory

Following are work
pytorch 1.13
conda install pytorch-cluster -c pyg


Questions about "soft labels"

https://github.com/cliang1453/BOND/blob/3651a92adedbd53bbc6210fade7f688b171d6434/run_self_training_ner.py#L240
Hello, a few questions came to mind when I read the code dealing with "soft labels", and I wonder if you could kindly help:

What's the difference between "label_mask" and "attention_mask" here? And since BERT forward doesn't take "label_mask" as input, including the keyword "label_mask" seems to cause an unexpected keyword argument error when inputs is passed to model

It seems to me that pred_labels is of shape (batch_size , sequence_length , num_labels), which is different than what is accepted by BERT labels, i.e. (batch_size, sequence_length)(thus potential source of size does not match error when passed to BERT forward). And accoding to the paper, losses associated with soft labels are calculate in a different way than BERT loss, but in the code, soft label loss is also calculated by passing inputs to BERT model, so I'm a bit confused.

It would be grateful if you could kindly help, and have a nice day!

I have meet the same problem.

Hi,

''label_mask'' is a binary mask of selected tokens with high prediction confidence. We optimize losses only on such tokens (See Paper Section 3.2.2). Unfortunately, our codes only support RoBERTa, where we implement a new classification head in RobertaForTokenClassification_v2. To use BERT, You'll need to implement new classification head instead of using BertForTokenClassification.
The soft label implementation is in https://github.com/cliang1453/BOND/blob/3651a92adedbd53bbc6210fade7f688b171d6434/modeling_roberta.py#L92.

Hope this helps!


Conflict with docker-compose 1.24.0

The latest version of docker-compose (1.24.0) errors instead of silently fails when the .env file has whitespaces in it, such as with export VAR=FOO. As such, users using the latest version of docker-compose with the export syntax will not be able to run docker-compose. The prior behavior was that any lines with whitespaces was silently ignored, now it errors out and fails.
Although I do not believe it should be dotenv responsibility to fix this issue, I wanted to bring it up here so that the dev team can be aware of it. If docker-compose won't change, you may have to at least add something into a FAQ or something.
Related issues:
https://github.com/docker/compose/issues/6511
https://github.com/docker/compose/issues/6741

ruby-dotenv per README works with and without export keyword. Using without export is compatible with docker-compose.
If using export keyword latest docker-compose falis with error whereas previously value was ignored.

export keyword is useful for being compatible with bash/sh source .env. Not sure if we're trying to juggle with too many areas of compatibility here.

I agree. However at least docker-compose will ignore lines starting with export.

that is why i'm reporting the issue - docker-compose 1.24.0 no longer ignores the lines starting with export - it is a hard fail in docker-compose at this point.


[backport] Package tests in sdist

Backport of #2563

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 161161f0afe416fc414ab6ddf89321e5d0e4d553, target branch v6) succeeded!

Jenkins, test this please.

Jenkins CI test (for commit 161161f0afe416fc414ab6ddf89321e5d0e4d553, target branch v6) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 161161f0afe416fc414ab6ddf89321e5d0e4d553, target branch v6) succeeded!

Thanks for doing this! 😄

/test

Jenkins, test this please.

Jenkins CI test (for commit 161161f0afe416fc414ab6ddf89321e5d0e4d553, target branch v6) succeeded!

Sorry for the delay, LGTM.


x/tools/imports: Process do not remove used but duplicated imports

What version of Go are you using (go version)?

$ go version
go version go1.19.1 darwin/arm64

Does this issue reproduce with the latest release?
Yes
What did you do?
https://go.dev/play/p/QEVFgl-HPhb
What did you expect to see?
import (
    "fmt"
)

For all cases.
What did you see instead?
import (
    "fmt"
)

import (
    "fmt"

    "fmt"
)

import (
    "fmt"

    "fmt"

    "fmt"
)

import (
    "fmt"

    "fmt"

    "fmt"
)

P.S. Related to https://github.com/golang/go/issues/37689, because fix for my issue is to add import aliases.

This looks related to but not the same as #45398.
CC @golang/tools-team.

I don't think this is related to #45398. The issue here is that these are in multiple import groups, and goimports (as a principle) processes import groups separately.
Note that if the duplicate imports are in the same group (as below) they are cleaned up as expected (as below):
import (
	"fmt"
	"fmt"
)

CC @heschi to confirm that this is expected behavior.

Correct AFAIK.

Thanks. Closing this as WAI.


Working around message contents which is now a privileged intent

While trying to test the discord.js v14 support added by PR #276, I noticed that the ?tttdeploy and ?tttdelete text commands were not working anymore. I learned that this is a limitation at the initiative of Discord.
Discord article: https://support-dev.discord.com/hc/en-us/articles/4404772028055
Quoting: "on August 31, 2022, access to message content will become a Privileged Intent".
Also:
The textual command to start a game is also affected by this limitation. Soon you will not be able to start a game using it. In order to fix that, I will work on new releases of versions 2 and 3 before this date. These versions will require you to add the privileged intent MESSAGE CONTENT to your bot to use the text command.
And for the future?
I understand the policy of Discord and I follow their principes. Messages should no longer be used to start a game, commands are the norm. Version 2 of the module will be deprecated as of August 31, 2022, and no more feature will be added on text commands as of this date on versions 3+ of the module.

Module version: 3.0.0
Discord.js version: 14.0.3

Do you have any idea on how to register slash commands without using messages? I do not want to force the usage of the privileged intent, only for admin commands. Thank you!

According to the rules of discord's new intent, when mentioning his bot or botrole, the message is loaded, I've been using this in mine as an alternative for a while, so try using that too.
Take a look at the code below which is from my bot.
https://github.com/Raccoons-Code/Konan/blob/main/src/events/MessageCreate.ts#L20

Super interesting idea! Thank you, I will work on this :+1:

Versions 2 and 3 of the module are using Discord API v6 and v9, so I do not have specific action to do in the module source code.
More info on the Discord documentation: https://discord.com/developers/docs/topics/gateway#gateway-intents
But I will update the wiki to explicit the need of the privilged intent MESSAGE_CONTENT and discourage using text commands.

Starting from v4 of the module, an error will be thrown with an explicit message when you configure a text command without the privileged intent enabled:
Error: You must enable Message Content intent to use the text command.
    at TicTacToe.<anonymous> (<anonymous>\discord-tictactoe\dist\src\index.js:48:27)
    at Generator.throw (<anonymous>)
    at rejected (<anonymous>\discord-tictactoe\dist\src\index.js:6:65)

Version 4.0.0 of the module is now out with the support of discord.js v14 👍
More info on the release here: https://github.com/utarwyn/discord-tictactoe/releases/tag/v4.0.0


Change type of local_disk_size attribute for InstanceType resource

Resource InstanceType currently has attribute local_disk_size set to string, but the actual json document is returning an int (not sure if this is cloud dependant or not). ex:

{
   "resource_uid":"1",
   "name":"small",
   "description":"canned type small",
   "memory":"1GB",
   "cpu_speed":"",
   "cpu_count":1,
   "cpu_architecture":null,
   "local_disks":0,
   "local_disk_size":0,
   "links":[  ],
   "actions":[  ]
},
{
   "resource_uid":"3",
   "name":"small (memory reserved)",
   "description":"canned type small",
   "memory":"1GB (Memory Reservation)",
   "cpu_speed":"1Ghz",
   "cpu_count":1,
   "cpu_architecture":null,
   "local_disks":1,
   "local_disk_size":10,
   "links":[  ],
   "actions":[  ]
}

with the attribute defined as string was getting "cannot unmarshal number into Go value of type string".

thanks!


refactor: move linarith Ineq datatype to standalone file

The linarith Ineq datatype is an enum with the constructors eq, le, and lt.  This is pretty general, and in fact I would like to re-use it in the new version of linear_combination which handles inequalities (#16841).  To enable this, this PR moves Linarith.Ineq and its API to a new file Mathlib/Tactic/Linarith/Ineq.  Previously this material was in Mathlib/Tactic/Linarith/Datatypes.
I also extend the API by adding two new MetaM functions, Lean.Expr.ineq? and Lean.Expr.ineqOrNotIneq?, which parse an expression to the Ineq it represents, if applicable.  These will be used in #16841, but I decided that at the same time I might as well golf some of the linarith code to use these functions rather than parsing inequality expressions from scratch each time.  The following functions from the linarith implementation get rewritten:
Linarith/Datatypes
Linarith.getRelSides (removed since entirely subsumed by Lean.Expr.ineq?)
Linarith.parseCompAndExpr
Linarith.mkSingleCompZeroOf
Linarith/Preprocessing
Linarith.filterComparisons
Linarith.isNatProp
Linarith.mkNonstrictIntProof
Linarith.rearrangeComparison
Linarith/Verification
Linarith.leftOfIneqProof
Linarith.typeOfIneqProof
Linarith/Frontend
Linarith.getContrLemma
I also streamlined several of these functions in ways I believe are harmless:

rather than separately handling LT.lt and GT.gt (resp. LE.le and GE.ge) I just include a whnfR in the parsing function
in Linarith.rearrangeComparison I do not special-case the LHS or RHS being zero
in the preprocessor Linarith.compWithZero I drop the handling of negation hypotheses; this is harmless when the Linarith.removeNegations preprocessor has been run before the compWithZero preprocessor (which is the default), and although in principle it is possible to run linarith in a custom configuration which doesn't include the removeNegations preprocessor, in that case the negation hypotheses would never be relevant, so it wouldn't matter that the compWithZero preprocessor doesn't handle them.

!bench

Here are the benchmark results for commit f94620d. There were no significant changes against commit 2fc69c7.

Not sure quite what to make of these benchmark results.  The overall effect is negligible and very few files are affected by more than 1%.  But nonetheless the files which seem most adversely affected are all analysis files (presumably containing lots of linarith), which suggests the changes are not just noise.
If indeed there's a slowdown, I'm not sure how to guess which part of the changes caused it.

bors r+


Charts not showing up in rancher UI

Using Rancher v2.8.4. No charts come up in list after adding repo.

Hey @johnwc, are you still seeing this issue? Have you tried with v2.8.5?

hey @johnwc, it is not supposed to show up under the Charts section of Rancher. It is supposed to show up in the Cluster Management screen when you're creating a cluster...


View revision history on web

Git has rich revision history. Using grit, we should be able to retrieve and display that on the web front-end, allowing people to view the history of a file.

Implemented first pass basic revision history in 0.4.10!


V2

Paging @lachlanjc @zanedb for thoughts on this concept 😄.

I like this idea a lot! My two concerns are:

Text readability (i.e. white-on-white for the Macintosh sticker)
Out of stock items should be at the bottom of the list (maybe a separate/hidden section?)
Background needs to be more interesting. Maybe something similar ish to the Bank landing page bg?

@merelinguist Are you planning on finishing this? Otherwise, let’s close it. Shop design isn’t a priority for HQ right now.


attempting to save user calls with null

Select user from list
view user
save
Argument 1 passed to Grav\Common\User\DataUser\User::filterUsername() must be of the type string, null given, called in /var/www/html/system/src/Grav/Common/User/DataUser/User.php on line 123.
I have just updated to latest on all modules
This is NOT related to the 2fa fix I have running as I have tested with and without and both fail.

https://github.com/getgrav/grav/commit/d34593928da30df8996f9def3d2ee3b5033adb47


Execution page: "Observe" module

The "Observe" module allows the user to search for an object to observe; give exposure instances and duration; choose the type of observation to make between "Dark", "Flat" and "ThAr" (Thorium Argon); and start and stop the observation.
This module should disallow the user, in some way, from interacting with the spectrograph while it is in operation. We need to implement a queue to accept waiting jobs. A stretch goal that we might consider when implementing this system is job scheduling. For example, assume that user A is using the spectrograph and user B wants to submit a job, or multiple jobs, for later. "Graying out" the interface disallows all other users from submitting jobs to the queue or scheduling system. We probably should allow users to interact with the job queue or scheduling system while the spectrograph is in use, but obviously disallow those jobs from interrupting the current job.

see b9cfc50ab3cecb54bf8f22f7053c6133d53034cc, 058043c63e727e8fcb02c5a04b7c2aa610794699, 692d537e619bd4c7242e9f27dfd9ad955e4ae284, f2d9e3717829ba1e96b91ea1f584b6e123d702fa, 3209f005abe6748f82a96830d4989293c5223664, #62, cd030fa7e556f183d0b06d7c2cb044edc0d4d55b, 7210baad967ca269bda23163f5da589001cac956


Fix typo in the description of instance_type var

what

Fix typo in the description of instance_type var

why

Avoids confusion

references

/rebuild-readme

/test all

@alexandrusavin please have a look at test output.
I think Terraform outputs need updating.

@alexandrusavin please have a look at test output.
I think Terraform outputs need updating.

Ups... I've updated the branch before checking the test output so I lost it. I don't know if the tests are scheduled automatically or you need to do it so I can see the output again.

/test all

@alexandrusavin they are run using chat commands but are restricted to contributors

@joe-niland I'm a bit confused by the test output. There were though some missed descriptions that needed update. I added 2 more commits. Is this what the tests caught?

This is the output of the 2 failing tests:
not ok 8 check if terraform providers are properly pinned
# (in test file test/.test-harness/test/terraform/provider-pinning.bats, line 30)
#   `grep -Eo '^\".*\|[^\~]*\"$' $TMPFILE | diff $TMPFILE -' failed
# --- /tmp/terraform-providers-2XiQMk7nRCM.txt
# +++ -
# @@ -1,3 +0,0 @@
# -"aws|~> 2.55"
# -"null|~> 2.1"
# -"template|~> 2.1"
not ok 9 check if terraform providers have explicit source locations for TF =>0.13
# (in test file test/.test-harness/test/terraform/provider-pinning.bats, line 41)
#   `grep -Ev '^\".*\|null"$' $TMPFILE | diff $TMPFILE -' failed
# --- /tmp/terraform-providers-x7ZvsE14FiX.txt
# +++ -
# @@ -1,3 +0,0 @@
# -"aws|null"
# -"null|null"
# -"template|null"

@alexandrusavin there is currently an issue with one or two of the tests sowe have to hold off until fixed. I will try again now though to see what happens.

/rebuild-readme

/test all

/test all

/test all

/test all

/test-readme

/test all

/test all


add bin alias for binary

there are already 3-letter aliases for hex -> hexadecimal and dec -> decimal, this adds one for binary
passes cargo fmt, clippy, and tests

Thanks!


Track with NodeJS LTS

Is your feature request related to a problem? Please describe.

There is no process in place to stay current with NodeJS LTS versions.
Describe the solution you'd like

NodeOS could try it's best to track with NodeJS LTS versions. If this is something that is doable, I'd be up for helping. I'd probably need a bit of coaching up front though on what processes to put in place around that.
Additional context

A big piece of this request is getting Worker support in Node v10+ and having some use cases I'd like to experiment with.

Some sort of upgrade system would be nice. Then again, since the entire OS runs on JS you can just replace the actual OS layer that handles everything low-level and npm will handle the rest.


Implement exponential backoff

It would be really awesome if Rack::Attack supported exponential backoff so as people continue to misbehave, their ban times increase (maybe up to some max time).
This article has some useful thoughts and tables about it:
https://devcentral.f5.com/articles/implementing-the-exponential-backoff-algorithm-to-thwart-dictionary-attacks

https://github.com/kickstarter/rack-attack/wiki/Advanced-Configuration :smile:

@reedloden - does the Advanced Configuration page solve your issue?

I haven't had time to test. I could close this out if it's a problem... I hope to pick this up later this month.

Haha, not a problem at all. Just wanted to check in since it had been a while :smiley:

Closing this since it's covered in the advanced config docs.
Happy throttling! :sunflower:

@reedloden - Was this ever tested?

@ACPK Not sure if we ever implemented it in this fashion. I will check and see.

@reedloden Thanks!

Can anyone confirm this approach actually works? To me, it seems like only the last rule is active.
In my example below, I only have a limit after the 10th request (level = 5). It should already limit after the 2nd request within 8 seconds etc.
(1..5).each do |level|
  throttle('req/ip', limit: (2 * level), period: (8 ** level).seconds) do |req|
    req.ip
  end
end

@em4nue1 You need to have unique names for each throttle. So, use throttle("req/ip/#{level}") or similar.


Could not find mathglConfig.cmake

I got the following errors when trying to use the package mathgl.
Selecting Windows SDK version 10.0.22000.0 to target Windows 10.0.19044.
CMake Error at D:/vcpkg/scripts/buildsystems/vcpkg.cmake:843 (_find_package):
  Could not find a package configuration file provided by "mathgl" with any
  of the following names:

    mathglConfig.cmake
    mathgl-config.cmake

  Add the installation prefix of "mathgl" to CMAKE_PREFIX_PATH or set
  "mathgl_DIR" to a directory containing one of the above files.  If "mathgl"
  provides a separate development package or SDK, be sure it has been
  installed.
Call Stack (most recent call first):
  CMakeLists.txt:16 (find_package)

But I really find the follow files:
MathGL2Config.cmake
MathGL2Targets.cmake

Is this a bug relate to the version?

MathGL2Config.cmake create a case-sensitive package name MathGL2, and MathGL should work, too.
(mathgl  would  work with mathgl2-config.cmake.)

Duplicates #22264.

Fixed by #23811.


Incorrect Transform Documentation

The documentation for the current version (0.25.3 at time of writing) has the following instructions for transform:

Whatever is returned from this transform method will end up being the resulting hash passed to serialization.

This is incorrect. The return value doesn't seem to be used at all, and any transformations need to be made on the hash argument directly. The example is set up correctly using merge!, but I assumed the focus was still on returning a the mutated hash and not on the mutation itself. This can be seen in the source code here.

Welcome to make a PR to update this doc


config files for ITER loco project

The project uses an MC4-PLUS with 2 DC motors and 2 solenoids

Hi @maggia80 , what is the status of this PR? Is it ready to be merged?
I am checking the open PR since on 31/05 we have the Distro release

Hi @maggia80 , what is the status of this PR? Is it ready to be merged?
I am checking the open PR since on 31/05 we have the Distro release

This PR is not blocking

to be checked on a real setup


correct mismatched enum

It's my first time contributing to open source, I did not know that I need to fork repo, instead of clone. haha...

It's my first time contributing to open source, I did not know that I need to fork repo, instead of clone. haha...

No worries, we all have been there 😉

Closes #66

@lyra95 would you also like to add a test to your PR? You just have to commit and push your changes and it will show up here.

Ok

I wanted to narrow the test target down to the method ValidateIssuer(), but I could not because  the method is protected in abstract class. Instead, I've added tests that generate tokens and verify them, which triggers the target method.

@lyra95 Thanks for adding the tests, and don’t worry, those are perfectly fine and should be testing the decoding of the token, there are already tests for the equality validator which the Issuer validator uses.

Thanks!


Custom_Args Feature

Added Custom_Args Feature to get the values in webhook.
How can I add unit test and run?

Codecov Report

Merging #53 into master will increase coverage by 1.72%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   60.22%   61.95%   +1.72%
==========================================
  Files           1        1
  Lines          88       92       +4
  Branches       22       25       +3
==========================================
+ Hits           53       57       +4
  Misses         29       29
  Partials        6        6

Impacted Files
Coverage Δ

sgbackend/mail.py
61.95% <57.14%> (+1.72%)
: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 476bfbe...8df68d5. Read the comment docs.


Add simple instructions to set up pkg-config for MSYS2

Adding simple instructions of how to install the UCRT and CLANG's toolchains for the MSYS2 on Windows.
Thanks to @michaelsbradleyjr.
Additional links:

#144

I think it makes sense to actually give step by step install instructions for owlkettle on windows. The aim is to have one supported way of installing that usually works. If somebody needs help debugging their install it is useful if their setup is as standardized as possible.
E.g. (note that I have never used MSYS2, so this might not be correct)
# Installing on Windows using MSYS2

1. Install Msys2 (link to install docs)
2. Install this Msys2 environment: `some command`
3. Install pkg-config: `...`
4. Install GTK: `some command`
5. ...

If you encounter any issues, please check out the troubleshooting steps below. Feel free to ask questions [in this discussion](...).

If there are specific troubleshooting steps those should be added to the troubleshooting section.

Now, I have a question @michaelsbradleyjr, in this case, as we're recommending the CLANG64/UCRT64 packages, so I should update the installation of gtk for the respective packages for these environments, right?

What about now, guys? @can-lehmann and @PhilippMDoerner.
@michaelsbradleyjr, if you have any suggestion, let me know.

What about now, guys?

Updated!

Sorry for not replying sooner, caught the flu/whatever and wasn't looking closely at email.
Glad to see this documented, nice work!
One thing that can be considered is adding a recommendation for pacman -S base-devel git regardless of those choice of toolchain; note those package names aren't prefixed with e.g. mingw-w64-clang-x86_64. That's because they're packages for the base MSYS environment.
https://www.msys2.org/docs/environments/

The MSYS environment contains the unix-like/cygwin based tools, lives under /usr and is special in that it is always active. All the other environments inherit from the MSYS environment and add various things on top of it.

Most devs working with Nim in CLANG64 (or UCRT64 or ...) will want git and the tools included in base-devel.


Debug console is now outputting tabs as spaces

Issue Type: Bug
Since the latest update, debug console has started outputting tabs as spaces.
I had a simple script that output tabbed data which I would paste into Excel and use Text to Columns delimited on tabs, which now does not work.
To reproduce I created a new Python script tab.py with the following code:
print("x\ty")
Copy and paste the output from the Debug Console into a text editor with white space showing (I used Notepad++) and you can see the tab has been substituted with a single space. This was not happening until the latest update.
VS Code version: Code 1.27.2 (f46c4c469d6e6d8c46f268d1553c5dc4b475840f, 2018-09-12T16:17:45.060Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2592)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.82GB (8.21GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe --disable-extensions

Screen Reader
no

VM
0%

Extensions disabled

File against the python extension please


Add MicroProfile Health extension

fixes #141

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/258/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/261/Failed Tests: 1camel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-paho: 1org.apache.camel.quarkus.component.paho.it.PahoIT.test


compile without IPv6 enabled (#10582)

does any branch include [fix(net): Allow to compile without IPv6 enabled (#10582) yet?

Yes, with the Hybrid compile branch. There you can modify sdkconfig settings and the Arduino libs are compiled with this changed settings before the compile run of the Arduino sketch is done with this libs. Used with:
platform = https://github.com/pioarduino/platform-espressif32.git#Arduino/IDF53_work_hybrid

[env:noipv6]
board                   = <your board>
custom_sdkconfig        =
                          '# CONFIG_LWIP_IPV6 is not set'
                          '# CONFIG_LWIP_IPV6_AUTOCONFIG is not set'
                          '# CONFIG_LWIP_PPP_SUPPORT is not set'

See some examples here: https://github.com/arendst/Tasmota/pull/22474/files

perfect - thx for your support!

Will be happy to compile Brautomat32 with when the source code is available ;-)


OPA executable path

/claim #676
feat(opal-client): Add configurable OPA executable path

Introduce INLINE_OPA_EXECUTABLE_PATH configuration option
Update OpaRunner to use configurable path for OPA executable
Default to 'opa' if no custom path is specified
Update setup_opa_runner to handle the new configuration

@danyi1212 review when free....

Hey @daveads can you please check why the test isn't pass before asking @danyi1212 for a review?

Hey @daveads can you please check why the test isn't pass before asking @danyi1212 for a review?

sure

 Cloning into 'opal-tests-policy-repo'...
Load key "/home/runner/work/opal/opal/app-tests/opal-tests-policy-repo-key": error in libcrypto
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

This suggests that the SSH key is not being recognized or doesn't have the correct permissions.

pre-commit / pre-commit (pull_request) FIX
https://github.com/permitio/opal/pull/683

@danyi1212 @gemanor
Good to go ?


Run nightly integration tests with the service branch

estimate 2

Integrate the new build process (service subdir first, then base dir) into the nightly scripting.
0.5. Run the unit tests with the new structure.
Run the integration tests on the service branch similar to how we always have (where the user has direct access to the IOGroups).
Prototype a second run where the user does not have direct access to the IOGroups, but the GEOPM service is installed (so the root user has access).  Then the expectation would be the service provides access for the desired signals if the batch interface was implemented.  Since the batch interface is not implemented, it is expected that these tests will fail.

Added 0 and 0.5 above at the end of Milestone #81 on YR21 WW34.


Menu doesn't close on button pressed and misalignment on open menus without HotKey.

After a menu is opening by a button pressed it can't be closed by a button pressed again.
If the menu bar is created without shortcut keys, the right menus are open misaligned.

If I read this correctly, there are two separate Issues here, right?

If I read this correctly, there are two separate Issues here, right?

Right. I'm almost finishing the fix.

Could you make them two issues and PRs? Not a huge deal, but it's just better to separate Issues if they are actually separate.

Could you make them two issues and PRs? Not a huge deal, but it's just better to separate Issues if they are actually separate.

Sorry @tig, but no, because the fix is only on the Menu.cs and is all mixed up. So it'll give me a lot of work to change the code to separate them and could corrupt some of the changes.

No problem then!