Autosave records?

Hi,
Is there a way to have cutelog autosave logs that it receives so that upon reopen, logs can come back up?

Not possible now, but that's not a bad idea.
Though I'm of the opinion that log persistence should be done by the log producer and not by the consumer. Currently this doesn't really work because JSON log formatters create incompatible output. But if I fixed that and also made it possible to pass a filename as an argument to cutelog so it would load the given log file when opened, would that be enough for your use case?

For me I'd want it to be the consumer. I have a few different things running at once. They would all send logs to cutelog.
Having cutelog be able to consume all the logs and then be able to in itself separate them out (already does that).. then being able to continue later would be amazing.
Otherwise I wind up having to hold onto json files from each producer and merging them or something to get the same effect.
If cutelog itself was updating a file with new consumed entries it would be perfect.
This winds up being sort of similar to something like ELK.. but much more lightweight for smaller use cases... Like a homelab.

Having cutelog be able to consume all the logs and then be able to in itself separate them out (already does that).. then being able to continue later would be amazing.

So you're using the single tab mode with multiple producers connected and you want everything to resume seamlessly after a restart? That's not as hard. Would be very tricky with multiple tabs.

If cutelog itself was updating a file with new consumed entries it would be perfect.

Yeah that's how it would work, but do you also want to specify the folder where these files would be created?

Yep exactly as you described. Setting a directory for it to work with (in case it needs to split or something) would be perfect.
Also have knobs to allow saving x days of logs with it auto removing older entries after it passes that line.
I'd personally probably save a couple months worth :)

I think this mechanism should also be able to be used to help speed up the ui. Right now if i log a bunch (like 10000 log messages) the ui becomes pretty likely to freeze up. Maybe if we only kept like 1000 logs in memory and load from disk as a user scrolls up, performance could improve.


Feat: add dnsConfig in helm chart

Description of your changes

🤖 Generated by Copilot at 8efadbe
Summary
:gear::globe_with_meridians::sparkles:

This pull request adds dnsConfig and dnsPolicy parameters to the KubeVela core helm chart to allow users to configure the DNS settings for the cluster. The parameters are applied to the cluster-gateway and kubevela-controller deployments in the cluster-gateway.yaml and kubevela-controller.yaml templates.

dnsConfig, dnsPolicy
New parameters for KubeVela
Cluster DNS settings

Walkthrough

Add dnsConfig and dnsPolicy parameters to KubeVela core chart to allow users to configure DNS settings for the cluster (link, link)
Pass dnsConfig and dnsPolicy values to cluster-gateway deployment template in cluster-gateway.yaml file (link)
Pass dnsConfig and dnsPolicy values to kubevela-controller deployment template in kubevela-controller.yaml file (link)

Fixes #
I have:

[ ] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

Why we need the dnsConfig in helm chart?


Support disabling src/dst checks on AWS ENIs.

Proposal / RFE
Hey everyone, While testing DSR on AWS we discovered that src/dst checks needed to be disabled in AWS ec2 instances for DSR to work (more info https://github.com/cilium/cilium/issues/13600). While the above solution worked, Internally in our team we had a discussion on how to disable this consistently across Auto scaling groups instances as AWS does not have an API to enable/disable DSR on a group of instances (src/dst API is only limited to an instance). To achieve this we thought about two solutions.

Disable SRC/DSR check during using user data. In this method, each instance will be responsible for disabling its own src/dst check.
Use some external component that has the knowledge of all instances and then can go and disable the check on those instances. In this case, one potential solution was to add this functionality in the Cilium CNI operator ( As it's already responsible for manipulating the ENI). Although one potential drawback of this is that, This setup might require some coordination between operator and cilium agents ( As we don't want agents to start working before the src/dst check is disabled)  otherwise it can break the networking setup in the cluster.
Let us know your thoughts on the above two approaches (or if there are some other ones that you think might be a better way). We are happy to make the contributions if the 2nd approach seems a better fit

Relevant discussion thread on slack: https://cilium.slack.com/archives/C1MATJ5U5/p1606914131477800

Since this is done per-ENI, it's best to consider making it part of ciliim-operator. There is already an implicit dependency on the operator in the agent, since IPs need to be allocated prior to agent starting up. On node reboot ENI settings should be persisted, since it's in the API and not a local state. However, currently operator manages secondary ENIs, but it will need make update  to primary ENIs for this, since that's where NodePort service traffic is routed.
NB: at present the agent has access to the same APIs as the operator, so it could theoretically make these API calls, however that is only due to the fact that instance roles are configured. In the future instance roles will get reduced and a specific role will be assigned to the operator and that should be the only component that intends to manage ENIs. The docs don't show this yet, but some folks are setting up their clusters this way already (see #13270).

Based on the above-mentioned points, It seems like integrating this in Cilium Operator makes sense. On the other hand, if the Eksctl or AWS API team are ready to make this an option for ASG then this requirement can be offloaded to the cluster provisioning tool and the Operator will not require to handle this.

It maybe plausible to still implement a check in the operator, so we can detect misconfiguration and warn the use.

Agree. It would be really helpful if there is some kind of check also, Otherwise tracking down the issue takes lots of effort.

@Buffer0x7cd / @errordeveloper -- I've put some thought into this and I think we should put this functionality into the operator given that is the central spot that handles AWS API comms. Until now the operator did not do any write modifications to the primary interface of the instance (it used it just for read operations to infer things like subnet, securitygroups, etc) -- which brings the question of managing that interface -- should Cilium take it under its purview.
I think the best course of action is to add a separate flag to the operator which defaults to true if the DSR is enabled. This gives users to not manage this setting via the operator if they do it somehow else (scripts at instance boot-up, etc).
I'm not that familiar with how DSR works but do we need to support this for the interfaces that the operator creates?
As to the tactical changes we can just add a new parameter to https://github.com/cilium/cilium/blob/master/pkg/aws/eni/instances.go#L41 and pipe that through all the way to the ec2 client internally.
Another question: I assume this is not working on Azure as well? I see that we can achieve the same result via https://docs.microsoft.com/en-us/azure/virtual-network/virtual-network-network-interface#enable-or-disable-ip-forwarding -- just need to pipe that through the Azure client inside the operator.

Regarding

I'm not that familiar with how DSR works but do we need to support this for the interfaces that the operator creates?

In most cases, Only the primary interface is used for the NodePort Services  (Unless the user has mentioned multiple interfaces for NodePort) due to this we only need this to be enabled for the primary interface. @brb Can you please confirm this?

Another question: I assume this is not working on Azure as well?

I haven't tested this on azure yet, So don’t have much information on it.

In most cases, Only the primary interface is used for the NodePort Services (Unless the user has mentioned multiple interfaces for NodePort) due to this we only need this to be enabled for the primary interface. @brb Can you please confirm this?

Why does this matter in the context of this issue? The src/dst check is per node, i.e. not per iface.

@brb it's actually per ENI and not per node -- https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-eni.html#modify-source-dest-check.

it's actually per ENI and not per node

Oh, I see. Thanks for the clarification.

In most cases, Only the primary interface is used for the NodePort Services (Unless the user has mentioned multiple interfaces for NodePort) due to this we only need this to be enabled for the primary interface. @brb Can you please confirm this?

I'm not very familiar with AWS, but it might be enough to disable the check for an interface which is used to communicate between k8s Nodes (aka --direct-routing-device). IIRC, the drop happened after the intermediate node had forwarded the request which was | src=client_from_outside_IP | dst=pod_IP |. However, the assumption needs validation, as I'm not sure whether the reply which is  | src=service_IP | dst=client_from_outside_IP | won't get dropped by the VPC if the outgoing interface is not the same as the one used for direct routing.

@brb -- after reading AWS docs I was able to confirm that this happens just for packets that are arriving on the machine, this check is not taken into consideration where the source is different.

I've been looking at implement this.
@brb  could you suggest how to pass the direct routing device's ENI id to the operator? From my understanding, the direct routing device detection is done by the agent and the operator has no knowledge of how to map device name to ENI interface.
An alternative is letting the operator to disable the SourceDestCheck flag for all the attached ENIs. But as being pointed out, flipping the flag on the direct routing device is suffice.
BTW, I've tested and can confirm that when SourceDestCheck is enabled, outgoing packets would be dropped, if the source address doesn't match any address of the instance.

Test details
This script (steal from here) sends an ICMP echo request to <IP_ADDRESS> host A with spoofing source address <IP_ADDRESS>. Only when sourceDestCheck on host A is disabled can the packet be received by the destination.
import socket
import time

s = socket.socket(socket.AF_INET, socket.SOCK_RAW, socket.IPPROTO_TCP)
s.setsockopt(socket.IPPROTO_IP, socket.IP_HDRINCL, 1)

ip_header = b'\x45\x00\x00\x1c' # Version, IHL, Type of Service | Total Length
ip_header += b'\xab\xcd\x00\x00' # Identification | Flags, Fragment Offset
ip_header += b'\x40\x01\x6b\xd8' # TTL, Protocol | Header Checksum
ip_header += b'\x08\x08\x08\x08' # Source Address <IP_ADDRESS>
ip_header += b'\x0a\x01\x9f\x42' # Destination Address <IP_ADDRESS>

icmp_header = b'\x08\x00\xe5\xca' # Type of message, Code | Checksum
icmp_header += b'\x12\x34\x00\x01' # Identifier | Sequence Number

packet = ip_header + icmp_header
s.sendto(packet, ('<IP_ADDRESS>', 0))

Disable sourceDestCheck on host A and run tcpdump on destination,
root@ip-10-1-159-42:~# tcpdump -n -i any icmp and host <IP_ADDRESS>
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on any, link-type LINUX_SLL (Linux cooked v1), capture size 262144 bytes
13:53:03.962753 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 4660, seq 1, length 8
13:53:03.962814 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 4660, seq 1, length 8

could you suggest how to pass the direct routing device's ENI id to the operator

We are trying to get rid of the "direct routing device" notion - https://github.com/cilium/cilium/issues/17441.


Does this work even if the app has not been started since boot?

Does this work even if the app has not been started since boot?

I would like to know as well!.

I would like to know as well!.


dconf-0.30.1.drv fails to build with linker error on macOS

Issue description
duplicate symbol _gvdb_table_new in:
    tests/libdconf-mock.a(dconf-mock-gvdb.c.o)
    gvdb/libgvdb.a(gvdb-reader.c.o)
duplicate symbol _gvdb_table_list in:
    tests/libdconf-mock.a(dconf-mock-gvdb.c.o)
    gvdb/libgvdb.a(gvdb-reader.c.o)

...

ld: 8 duplicate symbols for architecture x86_64
clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

Full log
Steps to reproduce
% nix-shell -p meld

Technical details

system: "x86_64-darwin"
host os: Darwin 18.2.0, macOS 10.14.2
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.1.3
channels(vadzim): ""
channels(root): "nixpkgs-19.03pre164964.b0f40b78513"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixpkgs

We probably just need to set doCheck = !stdenv.hostPlatform.isDarwin:
https://github.com/NixOS/nixpkgs/blob/3e950d584c8186239809e4d101e70e083ea9e9e0/pkgs/desktops/gnome-3/core/dconf/default.nix#L31

Does anyone have a good workaround for this (without needing a custom nixpkgs)?

This was a dependency of vim_configurable for me, I worked around it by turning off guiSupport as shown here: https://github.com/LnL7/nix-darwin/blob/master/modules/examples/lnl.nix#L404

We should be able to apply the following patch but what GSettings backend would be used on Darwin then:
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -420,7 +420,7 @@
   findXMLCatalogs = makeSetupHook { } ../build-support/setup-hooks/find-xml-catalogs.sh;

   wrapGAppsHook = makeSetupHook {
-    deps = [ gnome3.dconf.lib gnome3.gtk librsvg makeWrapper ];
+    deps = stdenv.optional (!stdenv.isDarwin) gnome3.dconf.lib ++ [ gnome3.gtk librsvg makeWrapper ];
   } ../build-support/setup-hooks/wrap-gapps-hook.sh;

   separateDebugInfo = makeSetupHook { } ../build-support/setup-hooks/separate-debug-info.sh;

Actually, I doubt even the current state works – dconf requires a daemon running to work.
Could someone please try the following commands on Darwin:
$ curl -L https://gist.github.com/jtojnar/bc099cd01b924e30fcd43de5d72b9afc/raw/0a780dacc378e58fc69c53c98fc6b8e4abd1caae/gsettings-test.nix > gsettings-test.nix
$ nix-build gsettings-test.nix
$ env G_MESSAGES_DEBUG=all result/test

@jtojnar
warning: Nix search path entry '/nix/var/nix/profiles/per-user/root/channels' does not exist, ignoring
these derivations will be built:
  /nix/store/sr9gz808nw514hnpz1b7zs0ai94d1jcw-test.c.drv
  /nix/store/9bx6g245bjw4grzsqr8f14x7c1dlmj9g-schema-test.drv
these paths will be fetched (37.26 MiB download, 155.29 MiB unpacked):
  /nix/store/0mxhkxc2w2jj1r7hhrq117y1mmlhzjps-gcc-wrapper-7.3.0
  /nix/store/8irll1a805hvbjd8q9hynz8x97afjmk3-libmpc-1.1.0
  /nix/store/lxjpk7f6hglaqp31lwhlm55abcdc6rm3-gcc-7.3.0
  /nix/store/nq1lwkgb7wzh0j2iz33dq03l2xqj1ngg-expand-response-params
  /nix/store/vvy9djpnkxslp8xi0sfxkka1vm92912z-gcc-7.3.0-lib
building '/nix/store/sr9gz808nw514hnpz1b7zs0ai94d1jcw-test.c.drv'...
copying path '/nix/store/nq1lwkgb7wzh0j2iz33dq03l2xqj1ngg-expand-response-params' from 'https://cache.nixos.org'...
copying path '/nix/store/vvy9djpnkxslp8xi0sfxkka1vm92912z-gcc-7.3.0-lib' from 'https://cache.nixos.org'...
copying path '/nix/store/8irll1a805hvbjd8q9hynz8x97afjmk3-libmpc-1.1.0' from 'https://cache.nixos.org'...
copying path '/nix/store/lxjpk7f6hglaqp31lwhlm55abcdc6rm3-gcc-7.3.0' from 'https://cache.nixos.org'...
copying path '/nix/store/0mxhkxc2w2jj1r7hhrq117y1mmlhzjps-gcc-wrapper-7.3.0' from 'https://cache.nixos.org'...
building '/nix/store/9bx6g245bjw4grzsqr8f14x7c1dlmj9g-schema-test.drv'...
/nix/store/0mxhkxc2w2jj1r7hhrq117y1mmlhzjps-gcc-wrapper-7.3.0/bin/as: assembler (clang) not installed
builder for '/nix/store/9bx6g245bjw4grzsqr8f14x7c1dlmj9g-schema-test.drv' failed with exit code 1
error: build of '/nix/store/9bx6g245bjw4grzsqr8f14x7c1dlmj9g-schema-test.drv' failed

Maybe changing gcc to clang will work better.

using clang, it built. Here is my output:
▿
𝝺 env G_MESSAGES_DEBUG=all /nix/store/vwvcw20323864pir35g11f74qnn0x13y-schema-test/test
GLib-GIO-Message: 22:00:01.269: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.
Theme: Adwaita


Update gettext catalog

@himdel

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Typo in settings

zulip-secrets.conf, and your cleitn ID here

The fix for this typo was merged recently.


Docker compose configuration

I prefer to manage infrastructure via declarative configuration as opposed to running a script, as it allows me to fully understand and track the desired state with source control and replicate a fully integrated deployment more reliably.
I would like to have a docker-compose.yml file that describes an example working terminusdb docker configuration.  Specifically, I would hope that this configuration be maintained here -- if not officially recommended, at least as a starting point.
I've tried to build this myself, but I ran into issues that prevented it from working.  I also tried the script and it works fine, but that just says to me that I don't understand what it takes to administer terminusdb and I don't have a grasp on what is important, which makes me even more leery of just relying on the script to manage everything.  No magic, please.
It's been a few weeks since I last approached the problem of defining a desired state configuration for terminusdb, but I would still like to tackle it if the team would be interested in such a contribution.  If we can get a docker compose-based configuration working, would the terminusdb team desire to keep it here, even unofficially?
I would provide more details about the specific problems I've encountered once I jump back into it.
TL;DR: I would like to docker-compose-ify terminusdb-quickstart; vibe check?

I have no objection to a docker compose file.
The main purpose of this script is:

To make it easy to run TerminusDB Server anywhere, including various Linux distros, MacOS, and windows, as a place holder for the "real distributions" that we will make in later stages of our development, like debs, rpms, flatpaks, exes, dmgs, etc

To be very configurable, but also have sane defaults that "work out of the box."

To allow developers working on TerminusDB's server and console components and SDKs, to run the server with various versions of its parts, primarily different versions the server and console components.

To use in our CI/CD testing pipelines for full end-to-end testing.

I think it would be very cool to have a a great example docker compose file for running TerminusDB, but I think it makes sense for such an example compose file to be as simple as possible as a starting point, which does not need to fulfill the goals of this script above.
Of course, this doesn't mean we couldn't have such a file in this repo tho, so look forward to seeing what you have in mind.
Thanks so much for your thoughtful comment and, of course, for trying TerminusDB!!

Thank you for your response! I especially appreciate that you outlined the motivation/purpose of the script as it exists today, your plans for how you hope to solve those problems as the project matures, and how my suggested addition could map onto that.
I'll update here when I have something more substantive.  Thank you!


fix: remove dark:hidden

Image is not displaying on the web, as advised by @Pierrci, removing this class attribute should fix.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


control-plane-operator: convert oauth endpoint to Route

Looking to minimize the number of NodePort based endpoints we need to expose for the guest cluster

/test e2e-aws

/lgtm


RHMAP-13988 SAML Cordova template has errors and doesn't work on iOS

Cherry pick
Removed the spec from platforms so it uses the pinned by the CLI
Added gap: to the CSP so Cordova Plugins work on iOS 10
Added a step to build instructions to add an access tag for the SAML_ENTRY_POINT, will also be documented on the SAML guide.

LGTM. Verified locally.


remove warningsFromTypeEnumerator null check

Looks like some tests are failing @Evangelink  are we strict with MSTest tests assertions or do we accept to update it if make sense?

I'd say yes just reading your message but I'll debug on Monday to understand exactly what's going on

@MarcoRossignoli I confirm that the test was not correctly setup.


llavaconfig error

File "/home/ma-user/anaconda3/envs/llava/lib/python3.10/site-packages/transformers/configuration_utils.py", line 264, in getattribute
return super().getattribute(key)
AttributeError: 'LlavaConfig' object has no attribute 'mlp_bias'
What is the cause of this problem?

I also have this problem when trying to test the inference with LLaVA-NeXT-Video-7B-DPO.

I also have this problem when trying to test the inference with LLaVA-NeXT-Video-7B-DPO.

It seems I have fixed this problem. We have to build the environment by pip install -e ".[train]".

@reachsak Hi, in the pyproject.toml, the train target require transformers version is :
"transformers@git+https://github.com/huggingface/transformers.git@1c39974a4c4036fd641bc1191cc32799f85715a4"

I uninstall transformers and reinstall by
pip install transformers@git+https://github.com/huggingface/transformers.git@1c39974a4c4036fd641bc1191cc32799f85715a4

solved the problem.
The werid thing is, I check the transformers version, the versions after 4.40.0 is conclude this commit, "1c39974a4c4036fd641bc1191cc32799f85715a4", I dont know this issue still happened.


Filename Makefile.inc for Makefile

This pull request adds the filename Makefile.inc for Makefile.
It fixes the issue reported by @cperciva in #2180 (comment).
In the wild examples.

Thanks!

:+1: thanks @pchaigno. @cperciva - this will be fixed in the next release of Linguist


Continue button doesn't appear when it didn't exist before

When you "rage quit" and the savefile is deleted the continue button doesn't appear. (normal)
After loading a savefile the continue button is still not there.

Quite frankly using the "Give up" button completely breaks everything :)

Fix, pushing change later, closing.


Unknown CPU: 12th Gen Intel(R) Core(TM) i7-1270P

CodeCarbon version: 2.2.3
Python version: 3.10.6
Operating System: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35

Description
When trying to initialise an EmissionsTracker I got a similar result to #300. I thought I'd report it so you can update the list if needed.
What I Did
Python 3.10.6 (main, May 29 2023, 11:10:38) [GCC 11.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from codecarbon import EmissionsTracker
>>> a = EmissionsTracker()
[codecarbon INFO @ 16:44:06] [setup] RAM Tracking...
[codecarbon INFO @ 16:44:06] [setup] GPU Tracking...
[codecarbon INFO @ 16:44:06] No GPU found.
[codecarbon INFO @ 16:44:06] [setup] CPU Tracking...
[codecarbon WARNING @ 16:44:06] No CPU tracking mode found. Falling back on CPU constant mode.
[codecarbon WARNING @ 16:44:07] We saw that you have a 12th Gen Intel(R) Core(TM) i7-1270P but we don't know it. Please contact us.
[codecarbon INFO @ 16:44:07] CPU Model on constant consumption mode: 12th Gen Intel(R) Core(TM) i7-1270P
[codecarbon INFO @ 16:44:07] >>> Tracker's metadata:
[codecarbon INFO @ 16:44:07]   Platform system: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35
[codecarbon INFO @ 16:44:07]   Python version: 3.10.6
[codecarbon INFO @ 16:44:07]   CodeCarbon version: 2.2.3
[codecarbon INFO @ 16:44:07]   Available RAM : 15.344 GB
[codecarbon INFO @ 16:44:07]   CPU count: 16
[codecarbon INFO @ 16:44:07]   CPU model: 12th Gen Intel(R) Core(TM) i7-1270P
[codecarbon INFO @ 16:44:07]   GPU count: None
[codecarbon INFO @ 16:44:07]   GPU model: None

Thanks for reporting this.
According to the docs of your processor (https://www.intel.com/content/www/us/en/products/sku/226255/intel-core-i71270p-processor-18m-cache-up-to-4-80-ghz/specifications.html) the TDP is 64 W.
Someone will add it to https://github.com/mlco2/codecarbon/blob/master/codecarbon/data/hardware/cpu_power.csv

I've added it, it will be in a futur release.


Entity config conditional actions (if)

If action
An If action triggers actions if a condition succeeds.
IfElse action
Same as If, but has a fallback action (else).
Condition
A "condition" returns true or false.
Conditions can be logical comparisons like equal, larger-than, etc.,
which take expressions as arguments, or conditions can be operations
like and, or, etc., which would take more conditions as arguments.
Expression
An expression returns a value (primarily numbers).
Possible expressions could include expressions that return the entity's health,
their position, velocity, lifecycle state, current variant, collision, etc.
(and in the future maybe entity config variables, if I ever add those).

Syntax design ideas:
events: {
    // Simple `if` conditions that add actions if condition passes.
    Init: If(
        if: True,
        then: Echo("True!"),
    ),
    Init: If(
        if: Not(False),
        then: Echo("Not false!"),
    ),

    OnInteract: PlayerAction(If(
        if: LessThan(Health, Num(10.0)),
        then: HealthAction(Gain(5.0)),
    )),

    // If condition with fallback `else` action.
    OnCollision(/* ... */): If(
        if: LargerThan(Velocity(X), 10.0)),
        then: MoveAction(AddVelocity(x: -10.0)),
        else: MoveAction(SetVelocity(x: 0.0)),
    ),

    /*
     * DESIGNS I DON'T LIKE AND THAT SHOULDN'T BE IMPLEMENTED.
     * (At least not exactly like this.)
     */

    // With `else_if` field.
    // I don't like this design.
    Interval(500): If(
        if: LargerThan(Velocity(X), 0.0),
        then: Echo("Moving RIGHT!"),
        else_if: [
            (
                if: LessThan(Velocity(X), 0.0),
                then: Echo("Moving LEFT!"),
            ),
            (
                if: Equal(Velocity(X), 0.0),
                then: Echo("Standing still."),
            ),
        ],
        else: Echo("Default, shouldn't get here."),
    ),

    // If/else chain as its own conditional action.
    // I still don't like this. Isn't obvious enough that
    // this is an if/else-if chain, and not just multiple ifs.
    // Default/fallback `else` also doesn't really work here.
    Interval(1000): IfElseChain([
        (
            if: LargerThan(Velocity(X), 0.0),
            then: Echo("Moving RIGHT!"),
        ),
        (
            if: LessThan(Velocity(X), 0.0),
            then: Echo("Moving LEFT!"),
        ),
        (
            if: Equal(Velocity(X), 0.0),
            then: Echo("Standing still."),
        ),
    ]),

    // Conditional switch action.
    // This is probably not needed for a while.
    Interval(750): Switch(
        value: Health,
        cases: [
            (0.0, Echo("No health, dead.")),
            (10.0, Echo("Full health, definitely not dead.")),
        ],
    ),
},

Expressions:

[X] Velocity(Axis)
[X] Health
[ ] Position(Axis)
[ ] lifecycle state
[ ] active variant
[ ] collision query

This is good enough and working, closing for now.
I'll add more condition expressions later when we need them.


Add when, which allows to modify only a given subtype

Example:
trait Animal
case class Dog(age: Int) extends Animal
case class Cat(ages: List[Int]) extends Animal

case class Zoo(animals: List[Animal])

val zoo = Zoo(List(Dog(4), Cat(List(3, 12, 13))))

val olderZoo = zoo.modifyAll(
  _.animals.each.when[Dog].age,
  _.animals.each.when[Cat].ages.at(0)
).using(_ + 1)

By the way, this is also known as a prism (http://julien-truffaut.github.io/Monocle//tut/prism.html).

Thanks! :) Looks great

Released as 1.4.7

Awesome, thanks for the fast turnaround (as usual) :)

One of these days, I'll got for vacation ;-)


Printing to real screen

I was trying to implement printing to real screen using the already made code found on this repo: https://jsandler18.github.io/tutorial/hdmi.html maybe someone can help me with this!

I played around with this same thing, here's a repo i have which is mostly based on what I learned from this repo.  I added a video.c which would write things to the screen using the HDMI output:  https://github.com/rockytriton/DSOS/tree/master/src
The problem here is that this uses the CPU to draw on the screen, which is ungodly slow.  If you have a whole screen of text it takes several seconds to draw it.  The only way to fast write to the screen is using the GPU, which I couldn't find any documentation on.

Hi, maybe you can get some information from here : https://www.cl.cam.ac.uk/projects/raspberrypi/tutorials/os/screen01.html

hi, @robitec97 i have the same problem with the jsanders OS. Did u get it to work?

hi, @robitec97 i have the same problem with the jsanders OS. Did u get it to work?

@Jesus8000 Unfortunately, no.

@Jesus8000 Unfortunately, no.


Enable @truffle/dashboard-hardhat-plugin

Once the fix is confirmed here, enable the plugin.

Resolved via https://github.com/pcaversaccio/batch-distributor/commit/a4178ec920c8738f02b768fcace26fd0c26ffb1b.


[Ionic v4.0.0-beta.19]animation Input on ion-router-outlet

Ionic version:
[x][4.0.0-beta.19]
In @ionic/core in the router-outlet we can set a "animation" property to set the animation between pages, in @ionic/angular ion-router-outlet directive the property doesn't exist, so we cant change the animation when using ion-router-outlet or i'm missing something ?

fixed using the solution provied by @duyetnk in #16829


Support creating EcsClientConfig from taskConfig

What does this PR change?
This PR introduces

a support to create an EcsClientConfig instance from taskConfig
new properties which can be specified from EcsClientConfig

a support to specify capacityProvider on a RunTaskRequest
a support to specify cpu on a ContainerOverride
a support to specify memory on a ContainerOverride

Background
In the current implementation, we can not create EcsClientConfig instances from taskConfig (we can create them only from systemConfig).
So it is difficult to execute Ecs tasks with different configurations for each Ecs task.

@myui

Revising of EcsClientConfig of was done  at 4faf4b0
startedBy option was added at c47e4f0

Regarding getAwsLogsConfiguration is out of this PR, isn't it?

LGTM excepting public IP assignment. Make it ECSClientConfig option.


GIF loads rotated if imageview was previously used to load JPG

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
10+

SDWebImage Version
4.4.2

Integration Method
cocoapods

Xcode Version
Xcode 9.4

Repro rate
100% given the steps

Repro with our demo prj
- -

Demo project link
- -

Issue Description and Steps
When I load a Gif into the same FLAnimatedImageView that was previously used to load a JPG(which was taken with the camera), the gif appears to be rotated. But if I load that same cached Gif without it previously loading the JPG, the Gif isn't rotated.
This is happening in a tableview which has tableviewcells that have FLAnimatedImageView's on them.

Have a table view with cells that are half the screen tall ( makes the cell reloading happen quicker )
Have a data source provide 5 gif urls and then 5 Jpg urls
Open the VC, see the Gifs load and play properly
Scroll to the Jpgs, see them load and display properly
Scroll back to the Gifs, any cell that was previously displaying a Jpg will now be displaying their Gif rotated

Similar Issue: https://github.com/rs/SDWebImage/issues/1317
From what I've found in my searches this has something to with the JPG having EXIF metadata and the Gif not, but I'm unsure what I can do to fix the issue.
One of the JPG's causing the issue: https://skrumble-dev-assets.s3.amazonaws.com/chats/5b4cadeaf1ad970f5063416b/4744214435480065278.jpg

Even easier way to reproduce it is,
I select a JPG from my Photo library & load it into an image view, then I select a gif from my photo library and load it into the same image view, the displayed Gif is rotated.

It seems a FLAnimatedImageView bug. UIImageView and our decoder always process the EXIF to image orientation correctly.
Could you please help to provide a demo that can trigger this issue ? Maybe we can figure it out quickly.

Here's a test repo link:
https://github.com/ericeddy/FLAnimatedImage-RotationIssue
I've noticed that using just any jpg doesn't cause it, like random jpgs from the internet cause it to occur sporadically,
But using images taken with the phone's camera will cause it 100% of the time, and using a landscape photo vs using a portrait photo will add an extra 90 degrees to the rotation

@ericeddy After some test and my experiment, I can confirm it's a BUG in FLAnimatedImage third party library itself. I've create a MR to the FLAnimatedImage repo. You can push them to merge it and solve this problem.
Now I can talk about the reason that cause this problem. At first, you should have basic knowledge of UIView-CALayer, if you don't know them, you should watch Understanding UIKit Rendering before reading the below.
What cause this problem
FLAnimatedImageView, it's a UIImageView subclass, which manually specify CALayer.contents with the current rendering frame's CGImage. And they use a CADisplayLink as a timer, to refresh current frame, finally produce the visual animation.
However, CALayer itself, contains a private property, called contentsTransform. You can check the private header here. CALayer will calculate current UIImage.imageOrientation and translate to CGAffineTransform to render the bitmap. Why they to do this it's because CGImage does not actually take rotation when you create UIImage with -[UIImage imageWithCGImage:scale:orientation:], The both scale and imageOrientation are just hint to the CALayer's property contentsScale and contentsTransform. So that CALayer use the contentsTransform to actually do rotation about the bitmap. What to say, actually, CALayer draw its contents, based on the both publich CALayer.transform and private CALayer.contentsTransform.
See the screenshot during View Debug and print out the CALayer information. The actually CALayer.contents bitmap show the correct orientation, but the rendering result cause a rotation.

After I use KVO to observe this contentsTransform changes, I found that the only write on this property was happend on -[UIImage setImage:] function call with a non-nil UIImage. The non-nil is really important. Because FLAnimatedImage original code only trigger it with a nil object when -[FLAnimatedImageView setAnimatedImage:] was triggered.

So now, you may know the reason. Right, the previous JPEG image which contains non-Up image orienation change the conrtentsTransform, but the new FLAnimatedImage does not clear that. I check the current FLAnimatedImage code and finally find the bug. The root case is that -[FLAnimatedImage setAnimatedImage:] does not do the correct thing, to reset the super UIImageView state about image.
- (void)setAnimatedImage:(FLAnimatedImage *)animatedImage
{
    if (![_animatedImage isEqual:animatedImage]) {
        if (animatedImage) {
            // Clear out the image.
            super.image = nil;   // -> This will not reset `contentsTransform`, not actually "Clear out the image."
            // Ensure disabled highlighting; it's not supported (see `-setHighlighted:`).
            super.highlighted = NO;
           //...
    }
}

Solution
The naive solution, it's quite simple, firstlly call super.image = non-nil image and then call super.image = nil. Only one line code and the issue disappear.
 {
     if (![_animatedImage isEqual:animatedImage]) {
         if (animatedImage) {
             // Reset `contentsTransform` state
+            super.image = animatedImage.posterImage;
             // Clear out the image.
             super.image = nil;
             // Ensure disabled highlighting; it's not supported (see `-setHighlighted:`).

From my personal opinion, I think they should always call [super setImage:] during [FLAnimatedImageView setAnimatedImage:]. If you don't call super, you may loss internal consistent behavior during subclass. But anyway, this fix is much simpler.
Current workaround ?
Yes. FLAnimatedImage seems no longer maintained, though I fire the PR, it may take long time to merge in (or may be forever :) ). For our FLAnimatedImage+WebCache, the simple way to fix this, it's just use the code above, call super.image = animatedImage.posterImage; before super.animatedImage = animatedImage. I will create another PR fix for SDWebImage/GIF subspec.

Thank you for such a thought out and fast response!
I think for the meantime I'm going to go ahead with your suggestion and use 5.0.0-beta, I tried out how it works in the demo and it seems to be working well,
Again, Thank you for all this work, I really appreciate it!


Community Exchange Introduction & Tracking

👋 Hi @Ammar-Raneez,
I am your GitHub mentor for the GitHub Education, Community Exchange (CX) project. I'll be collaborating with you on preparing your repo for CX.
You mentioned in your submission that you wanted to submit a Learn repo. This means that you would like to teach students, step by step, how to build this project. I will generate issues, which will provide guidance on how to prepare your repo for a Learn CX submission on June 1, 2022.
This issue will serve as a tracking issue to track all issues related to CX. I recommend creating a new branch for every issue and opening a pull request to track changes so we can effectively collaborate with each other and merge changes when you and I feel like those changes are ready to be merged on your primary branch.
If you have any questions or concerns, please feel free to leave a comment on this issue or any of the other issues that are generated.
I look forward to working with you! :octocat:
CX Issues:

[x] https://github.com/Ammar-Raneez/SDGP-ONCO/issues/55
[x] https://github.com/Ammar-Raneez/SDGP-ONCO/issues/56
[x] https://github.com/Ammar-Raneez/SDGP-ONCO/issues/57
[x] https://github.com/Ammar-Raneez/SDGP-ONCO/issues/58
[x] https://github.com/Ammar-Raneez/SDGP-ONCO/issues/59

Sounds like a plan 😁

Thanks for completing all the tasks! I'll leave this open until I can let you know when to submit to CX! ✨

Alrighty 😄


Fix typo

Description

Fix typo.
Motivation and Context

#1300

How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.
Fix typo in comments.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[x] Fix bugs
[ ] Add new feature
[ ] Update documentation

@you-n-g
This is a typo patch.
You can review it very quick.

@you-n-g
The errors are not related to my commit. How do I fix it?

Thanks for your contribution.
This CI error may be caused by an issue in windows.
I'll merge it first.


Make AzureIndexer compatible with new Stratis.Bitcoin packages

This PR updates the AzureIndexer to make it compatible with the latest Stratis.Bitcoin packages.

Is that the only changes that are required?

Those are the only changes required to achieve compatibility with the new version of the packages. The new code has been running for a few hours now on the server.

ok I've pushed it in, can you please have a look at my comment in a new PR?
Thanks Gustave!

@bokobza, please see PR https://github.com/stratisproject/BlockExplorer/pull/44.


Starts manually okay, but as service 'Error: Please configure a random SECRET_KEY'

Hi, hope you can help - it's probably something simple I'm missing.
This works perfectly from the command line -
/var/www/otterwiki/venv/bin/uwsgi --http <IP_ADDRESS>:8080 --enable-threads --die-on-term -w otterwiki.server:app
But when it is run from the service it errors -
Error: Please configure a random SECRET_KEY with a length of at least 16 characters.
otterwiki.service: Main process exited, code=exited, status=1/FAILURE
otterbox systemd[1]: otterwiki.service: Failed with result 'exit-code'.

Relevant part of - /etc/systemd/system/otterwiki.service -
[Service]
User=www-data
Group=www-data
WorkingDirectory=/var/www/otterwiki
ExecStart=/var/www/otterwiki/venv/bin/uwsgi --http <IP_ADDRESS>:8080 --master --enable-threads --die-on-term -w otterwiki.server:app
SyslogIdentifier=otterwiki

setting.cfg definitely has a working SECRET_Key too.
Any help would be appreciated

Hey @nateProjects, thank you for reporting this! The documentation was missing an important setting:
[Service]
...
Environment=OTTERWIKI_SETTINGS=/path/to/the/settings.cfg
...

I've just updated this on https://otterwiki.com/Installation and added a comment about changing the --http option if necessary.

Hey @nateProjects, thank you for reporting this! The documentation was missing an important setting:
[Service]
...
Environment=OTTERWIKI_SETTINGS=/path/to/the/settings.cfg
...

I've just updated this on https://otterwiki.com/Installation and added a comment about changing the --http option if necessary.

That's it! Perfect! Thanks


Add gtk-layer-shell to dependency list in docs

Currently, eww depends upon gtk-layer-shell when compiled with --features=wayland.  It might be helpful to list this in the documentation under "Packages".

Thanks! :tada:


Plugin is generic ?

Does this plugin require modification in source code for other data sources that not using Kubernetes API ? like arbitrary json log in loki

I actually have this question as well. I have a json tree that I want to show in this component, but I cannot find any way to do so. Would appreciate an answer.

It's a generic plugin, not only Kubernetes API output can be presented.
If you have trouble, could you attach a few JSON example response, please?

Thanks for the timely response. I opened a new Issue on it. Let's continue over there.


Getting fallback execution rejected exception some times in HystrixObservableCommand.

Hi Team,
We are using Hystrix in our application. We have HystrixObservaleCommand on service layer. When i start the application and access one hystrix command it will work fine, after some time we are getting below exception.
com.netflix.hystrix.exception.HystrixRuntimeException: getTicketsAsync fallback execution rejected.
        at com.netflix.hystrix.AbstractCommand.getFallbackOrThrowException(AbstractCommand.java:764) ~[hystrix-core-1.4.21.jar:1.4.21]
        at com.netflix.hystrix.AbstractCommand.access$300(AbstractCommand.java:64) ~[hystrix-core-1.4.21.jar:1.4.21]
        at com.netflix.hystrix.AbstractCommand$1.call(AbstractCommand.java:403) ~[hystrix-core-1.4.21.jar:1.4.21]
        at com.netflix.hystrix.AbstractCommand$1.call(AbstractCommand.java:363) ~[hystrix-core-1.4.21.jar:1.4.21]
        at rx.Observable$2.call(Observable.java:162) ~[rxjava-1.0.14.jar:1.0.14]
        at rx.Observable$2.call(Observable.java:154) ~[rxjava-1.0.14.jar:1.0.14]
        at rx.Observable$2.call(Observable.java:162) ~[rxjava-1.0.14.jar:1.0.14]
        at rx.Observable$2.call(Observable.java:154) ~[rxjava-1.0.14.jar:1.0.14]
        at rx.Observable$2.call(Observable.java:162) ~[rxjava-1.0.14.jar:1.0.14]
        at rx.Observable$2.call(Observable.java:154) ~[rxjava-1.0.14.jar:1.0.14]
        at rx.Observable.unsafeSubscribe(Observable.java:7710) ~[rxjava-1.0.14.jar:1.0.14]

I have set bellow property for the hystrix command.
hystrix.command.getTicketsAsync.execution.isolation.semaphore.maxConcurrentRequests=50
hystrix.command.getTicketsAsync.fallback.isolation.semaphore.maxConcurrentRequests=50

When, i start the application it is working fine and proccess some requests perfectly. But after some time will get above exception. I am not getting why hystrix is giving this error even if i am proccessing only one request.
Thanks in advance.

Try increasing the service timeout.
Sent from my iPhone
On Jun 1, 2016, at 7:42 AM, ankit8051 <notifications<EMAIL_ADDRESS>wrote:
Hi Team,
We are using Hystrix in our application. We have HystrixObservaleCommand on service layer. When i start the application and access one hystrix command it will work fine, after some time we are getting below exception.
com.netflix.hystrix.exception.HystrixRuntimeException: getTicketsAsync fallback execution rejected.
at com.netflix.hystrix.AbstractCommand.getFallbackOrThrowException(AbstractCommand.java:764) ~[hystrix-core-1.4.21.jar:1.4.21]
at com.netflix.hystrix.AbstractCommand.access$300(AbstractCommand.java:64) ~[hystrix-core-1.4.21.jar:1.4.21]
at com.netflix.hystrix.AbstractCommand$1.call(AbstractCommand.java:403) ~[hystrix-core-1.4.21.jar:1.4.21]
at com.netflix.hystrix.AbstractCommand$1.call(AbstractCommand.java:363) ~[hystrix-core-1.4.21.jar:1.4.21]
at rx.Observable$2.call(Observable.java:162) ~[rxjava-1.0.14.jar:1.0.14]
at rx.Observable$2.call(Observable.java:154) ~[rxjava-1.0.14.jar:1.0.14]
at rx.Observable$2.call(Observable.java:162) ~[rxjava-1.0.14.jar:1.0.14]
at rx.Observable$2.call(Observable.java:154) ~[rxjava-1.0.14.jar:1.0.14]
at rx.Observable$2.call(Observable.java:162) ~[rxjava-1.0.14.jar:1.0.14]
at rx.Observable$2.call(Observable.java:154) ~[rxjava-1.0.14.jar:1.0.14]
at rx.Observable.unsafeSubscribe(Observable.java:7710) ~[rxjava-1.0.14.jar:1.0.14]
I have set bellow property for the hystrix command.
hystrix.command.getTicketsAsync.execution.isolation.semaphore.maxConcurrentRequests=50
hystrix.command.getTicketsAsync.fallback.isolation.semaphore.maxConcurrentRequests=50
When, i start the application it is working fine and proccess some requests perfectly. But after some time will get above exception. I am not getting why hystrix is giving this error even if i am proccessing only one request.
Thanks in advance.

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/Netflix/Hystrix/issues/1230, or mute the threadhttps://github.com/notifications/unsubscribe/AKdlY3HykN2WvHL8u5txiuOYuyLzX4xjks5qHZm0gaJpZM4IrnmK.

I don't think timeouts are related.  This problem is specific to fallback semaphores being acquired and never released.
This sounds like a bug which got fixed in 1.5.3.  See #1204 for some more details.  Can you try upgrading to that release to see if that solves your issue?

Thanks, Matt,
Thanks for your help. This new version hystrix jar worked for me. Now there are no any hystrix command  execution rejections.

Excellent- thanks for the confirmation.  We've started using HystrixObservableCommands much more, and ran into this issue as well


Switch to buildFromSdist

Closes #220
buildFromCabalSdist has problems; see #220
So we are switching to the slightly less performant buildFromSdist.

@roberth I assume you are okay with this. I'll merge this shortly.


Verify that resource referenced in request field has pattern(s) present in google.api.http annotations

When an RPC has a resource name in its request, the name is often used as part of the HTTP binding. For example:
service FooService {
  rpc GetFoo(GetFooRequest) returns (Foo) {
    option (google.api.http) = {
      get: "v1/{name=projects/*/foos/*}"
    };
  };
}

message GetFooRequest {
  // Format: projects/{project}/foos/{foo}
  string name = 1 [(google.api.resource_reference).type = "foo.googleapis.com/Foo"];
}

message Foo {
  option (google.api.resource) = {
    type: "foo.googleapis.com/Foo"
    pattern: "projects/{project}/foos/{foo}"
  };

  string name = 1;
}

Here, GetFooRequest.name field is used to populate a section of the HTTP Request path. That field references a resource definition that defines a resource name pattern. The linter should enforce that the expected pattern(s) present in the google.api.http path can be found in the google.api.resource annotation for the referenced resource.

acamadeo@ all yours!

Fixed by #1045. Will be in the next release. Closing this as the other case of child_type references in the google.api.http annotation is covered by #1047. Thank you @acamadeo


The issue with the change in the method signature of initSDK in version 2.3.5 of the SDK

新版 initSDK 方法签名里多了几个参数。请问各参数作用是什么？文档里没说明

Detail,Please check https://docs.openim.io/sdks/class/init/config


Publish new patch to npm for the latest REAME.md

The README that currently is available at npm is out of date with the master on this repository.
Most notably the rule is incorrect leading to the user getting the following error:
  1:1  error  Definition for rule 'import-order' was not found  import-order

Publishing a new version will avoid this confusion.

I was waiting for (expecting) issues to pop in order to re-publish, but seems like it's working fine (except what you reported).
I just published 2.1.2. Sorry for the confusion


Make IL View Changes Consistent with Respect to Navigation History

Changing IL views in graph view saves the transition as part of the navigation history, whereas linear view does not.

Fixed in 2.2.2514-dev. IL view changes are no longer part of navigation history.


Can algebra.js file work in browser?

Hi @nicolewhite ,
We are cdnjs team. We want to host this library.
Because the algebra.js file includes files in src folder, we want to know if it can work in browser without files in src folder. Could you please tell us whether files in src folder need to be added in cdnjs?
Thank you.
https://github.com/cdnjs/cdnjs/issues/7971
https://github.com/cdnjs/cdnjs/pull/8152

Hello, sorry, I don't have a lot of free time to maintain this project anymore. Can you help me understand what's changed recently with browsers loading JavaScript files? My current setup worked when I built this project a few months ago. I simply loaded the minified js like so:
<script src="javascripts/algebra.min.js"></script>

Now this no longer works and I get undefined errors.

@nicolewhite exactly! This is the problem that I am facing! It shows Equation is not defined or Expression is not defined even though I load the minified js file properly.

I updated the dependencies and all seems to be well. A new minified file has been uploaded to http://algebra.js.org/ and works correctly in the browser (only tested on Chrome).


style: standardize whitespace and formatting

The two holy war topics this touches on is spaces vs tabs and if there
should be newlines around braces
omersiar suggested spaces in a github issue, and I chose no no newline
before open brace inside of functions, but not for functions.
I got this from the linux kernel style guide, and I like it because
functions set scope in C, while other blocks do not.
so for example:
void example(void)
{
    if (true) {
        doStuff();
    } else {
        otherStuff();
    }
}

Happy to change, but this is my preferences and it was inconsistant
so I felt I could choose.
Closes #518

Oh also I put the uncrustify config I used into the repo.
To run this what I did was
for f in src/*.*; do uncrustify -c uncrustify.cfg --no-backup -l cpp $f; done

I dunno how we want to document that or if we even want to commit the uncrustify config.

We could auto format on precommit, to enforce one style automatically. Also the editorconfig approach looks fine.
In any case if we merge this I would like to avoid having every now and then these kind of big commits. I don't care about the style so long it's one and it's automatically enforced.
If you add some automatic formatting we can merge this :)
Also because I have a couple branches that will be a nightmare to rebase and merge if we merge this first :)
Thank you!

I'm closing this as we are tracking the issue here: https://github.com/esprfid/esp-rfid/issues/518


SurveySelector - Only render the latest list of surveys

Fixes some UI jitter by restricting our rendering of surveys in the selector to the most recent list at the time the user opens the selector. This means the list isn't fully dynamic (if an organizer adds them to a survey while the selector is open, they'll need to close then re-open the selector). However, it does fix the rendering and UX issues that resulted from our previous dynamism, and this seems like an acceptable tradeoff.
We do re-render the list in the case that the user removes a survey from offline use.
https://github.com/google/ground-android/assets/11237600/dcef1e7a-674e-4e0a-bf7c-fb934879829d
Fixes #2109

Hi @scolsen! The reactive list was actually by design - it was updated from non-reactive, so this change would be a regression. Instead, can we unsubscribe to changes and show a spinner while loading the survey?

Ah sure thing — we can just cancel on survey selection instead, lemme make the change.

Thanks!

kk, @gino-m we now handle this via coroutine cancellation -- the effect is the same. On selection we briefly show a spinner then immediately take the user to the home screen without re-rendering the list.


tab: include installation date in string representation

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] N/A (no to_s test so far). Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

I often wonder when a formula was upgraded, but despite showing each installed version, brew info does not show dates. I have to dig into INSTALL_RECEIPT.json or log/blame a formula to find out the date.
With this patch, brew info will include the installation date for each installed version, e.g.,
> brew info git
git: stable 2.8.2 (bottled), HEAD
Distributed revision control system
https://git-scm.com
...
/usr/local/Cellar/git/2.8.0 (1,416 files, 30.5M)
  Poured from bottle on 2016-03-29
/usr/local/Cellar/git/2.8.1 (1,416 files, 30.4M)
  Built from source on 2016-04-10 with: --without-completions
/usr/local/Cellar/git/2.8.2 (1,417 files, 30.4M) *
  Built from source on 2016-04-30 with: --without-completions
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/git.rb
...

I'm using ISO 8601 date format here because it's most widely accepted across the world (hopefully), but this is debatable.

Update. Time is also included now:
> brew info git
git: stable 2.8.2 (bottled), HEAD
Distributed revision control system
https://git-scm.com
...
/usr/local/Cellar/git/2.7.4 (1,415 files, 30.5M)
  Poured from bottle on 2016-03-18 at 04:00:26
/usr/local/Cellar/git/2.8.0 (1,416 files, 30.5M)
  Poured from bottle on 2016-03-29 at 04:00:22
/usr/local/Cellar/git/2.8.1 (1,416 files, 30.4M)
  Built from source on 2016-04-10 at 22:58:14 with: --without-completions
/usr/local/Cellar/git/2.8.2 (1,417 files, 30.4M) *
  Built from source on 2016-04-30 at 09:16:56 with: --without-completions
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/git.rb
...

I think this is useful information worth including in brew info.

I'm using ISO 8601 date format here because it's most widely accepted across the world (hopefully), but this is debatable.

Thanks! Obviously a bikeshedding topic, but I hope most technically inclined folks will agree that this is the only sane (and reasonably unambiguous) date format. I certainly do!

Thanks! Obviously a bikeshedding topic, but I hope most technically inclined folks will agree that this is the only sane (and reasonably unambiguous) date format. I certainly do!

One more item for the Style Guide. Team 8601 for life!

Given this is producing no more lines of output than before (my initial objection) I'm 🆒 with it 👍

Merged in 1fa48234e5301f018dc93e62bc7e03d71729b296. Thank you for another contribution to Homebrew, @zmwangx! :tada:


Incorrect Disciplines

Submitter: Devin Burch
Brujah has incorrect disciplines automatically selected based on text at the top of the page.

The disciplines seem fine to me. The clan Brujah should select Celerity, Potence, and Presence. Those are their starting disciplines. Maybe you selected True Brujah? They have Temporis instead of Celerity. If you want to check, it's page 51 in the V20 rulebook.


[CI:DOCS] Update /_ping support

Add swagger support and documentation
Promote handler as there can only ever be one
Update swagger tags

Between verbs and paths there are four endpoints for /_ping.  I've
attempted to document them well without four copies of the details.
Time will tell.
Clients will use the headers Libpod-API-Version and
Libpod-Buildha-Version to determine if they are connecting to a Podman
service or some other compatible engine.
Client calls GET(/_ping):

VARLINK engines will fail with error message
Compatible engines will return 200 without Libpod headers
Podman engines will return 200 and Libpod headers

Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

yeah LGTM

/lgtm


Consolidate owner and requester in Workflows

Making changes to owners in workflow to help us with bz 1187777
Workflows only have requesters (assigned to @owner and @requester)
But @owner and @requester were then never assigned after the initializer.
This consolidates the two variables together.
Also, the @{requester,owner}_id is derived from the @requester.userid,
No need for these variables.
Finally, user was looked up from @requester.userid.
No need to look this up, already have a user object in @requester.
/cc @gmcculloug

This is great that we are clarifying the roles of the requester and owner within the request workflow as each has a separate responsibility which was not clear before.


⚠ Bofrost.lu has degraded performance

In aef79de, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 952 ms

Resolved: Bofrost.lu performance has improved in a8ea2ff after 20 minutes.


xgboost 0.7 support issue, TypeError: 'str' object is not callable

Hi,
apparently xgbfir does not support the recently released of xgboost 0.7:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-19-7de868c73240> in <module>()
      8
      9 # saving to file with proper feature names
---> 10 xgbfir.saveXgbFI(xgb_rmodel, feature_names=X.columns, OutputXlsxFile='FI.xlsx')
     11
     12

~/anaconda3/lib/python3.6/site-packages/xgbfir/main.py in saveXgbFI(booster, feature_names, OutputXlsxFile, MaxTrees, MaxInteractionDepth, MaxDeepening, TopK, MaxHistograms, SortBy)
    589     if not 'get_dump' in dir(booster):
    590         if 'booster' in dir(booster):
--> 591             booster = booster.booster()
    592         else:
    593             return -20

TypeError: 'str' object is not callable

the issue is solved with a new version.


Created contracts included in constructor object.

What was wrong?
fixes #284
How was it fixed?
Added the created_contracts vector to the constructor object.

The Yul optimizer does not currently remove these objects if they are unused. Meaning, created contracts will be included in the initialization code twice no matter what. This does not increase the contract size, but increases the initialization code size.
The cost of this extra tx data is 16 for non-zero bytes and 4 for zero bytes (see appendix g).
For example, say we write a contract that deploys an ERC-20 contract and has an __init__ that does not create an ERC-20 contract. A compiled Fe ERC-20 contract is about 2600 bytes. So, the extra cost would be slightly less than 42000, which is roughly equivalent to the cost of 2 sstore operations. This is a non-trivial expense and we should find a way to remove these unused data object. It might be the case that the Yul team plans on implementing this, so we don't have to worry about.
To-Do

[x] Create an issue to track the gas issue. #305
[x] Add a test where a contract is created in __init__.
[ ] OPTIONAL: Update Spec if applicable
[x] Add entry to the release notes (may forgo for trivial changes)
[x] Clean up commit history

Codecov Report

Merging #304 (b59d4ce) into master (ba7bb3a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #304   +/-   ##
=======================================
  Coverage   93.72%   93.72%
=======================================
  Files          56       56
  Lines        3986     3986
=======================================
  Hits         3736     3736
  Misses        250      250

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 ba7bb3a...b59d4ce. Read the comment docs.


Seeing issue with aws-iam-token missing for all pods

HI Im seeing the below issue when bringing up the new pods
Almost all the pods in namespace have this issue..I checked all teh resources oidc role policy etc..everything looks good
MountVolume.SetUp failed for volume "aws-iam-token" : failed to fetch token: the server could not find the requested resource
Could some one help

@saiharshitachava would be great if you can provide more details on what does your setup look like and how to reproduce it.

Delete the pod and check @saiharshitachava


'nil' requires a contextual type

Description
After installing adapty in my project, I got "'nil' requires a contextual type" when building ios. Does anyone know why this happens?
dependencies:
`{
"@adapty/react-native-ui": "^2.1.2",
"@gorhom/bottom-sheet": "^4",
"@react-native-firebase/app": "^19.2.2",
"@react-native-firebase/auth": "^19.2.2",
"@react-native-firebase/firestore": "^19.2.2",
"@react-native-google-signin/google-signin": "^11.0.1",
"@react-native-masked-view/masked-view": "^0.3.1",
"@react-navigation/native": "^6.1.17",
"@react-navigation/native-stack": "^6.9.26",
"@react-navigation/stack": "^6.3.29",
"react": "18.2.0",
"react-native": "0.73.6",
"react-native-adapty": "^2.10.1",
"react-native-device-info": "^10.13.1",
"react-native-gesture-handler": "^2.16.0",
"react-native-modal": "^13.0.1",
"react-native-reanimated": "^3.8.1",
"react-native-reanimated-carousel": "^3.5.1",
"react-native-safe-area-context": "^4.9.0",
"react-native-screens": "^3.31.1"

},`
Version
"react-native-adapty": "^2.10.1",
What platforms are you seeing the problem on?
iOS
System info

Hi @T30-H0
What's the XCode version?

Hi @vladd-g,
I'm using  Xcode 14.3.1. Also, I checked the release and it mentions "The minimal Xcode version is now 15.0.". So let me update my Xcode and will have an update later. thanks

Work fine on Xcode ver 15. closed.


General: Increase feedback length limit

Do NOT deploy this PR on a test server
Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Motivation and Context
Similar to #4796, we have some exercises that test programs via CLI interactions. Therefore, more text output is generated which sometimes is longer than 5.000 characters.
Description
In principle, there were several options on how to achieve this goal:

(a) change the varchar(5000) definition to something longer
(b) replace varchar with datatype text entirely.

Option (a)
MySQL theoretically allows varchars up to a maximum length of 65.535 bytes [1]. However, since each tuple/row in the database table is only allowed to have up to this same length [2] the actual length is lower in practice.
Additionally, Artemis uses a database in utf8mb4 character set. That means each character can use up to four bytes of storage [3]. Therefore, the actually allowed maximum length in our case would be $\frac{65535}{4} = 16383$. To then also allow for the other columns in the feedback table, the maximum length has to be reduced to about 15000 characters (see appendix at the end of the PR description).
Benefits:

No performance impact since unused bytes in varchars remain unallocated.

Disadvantages:

This hinders possible extension in the future, since the table does not have enough space to accommodate additional columns.

Option (b)
MySQL has a special data type for longer texts which allows the storage of up to 65.535 bytes in a single string regardless which other columns contains [4]. PostgreSQL also supports this data type without any upper byte limit and explicitly no performance difference compared to varchar [6].
One disadvantage of the text column type is the possible performance impact. Whenever a temporary table has to be created by MySQL that includes a text column, this table has to be written to disk. Nevertheless, for Artemis this should not have an impact, since temporary tables only need to be created for operations that are usually avoided in the Artemis codebase anyway (e.g. UNION, subqueries) [5].
Disadvantages:

Possible performance impact.

Benefits:

Future extensibility of the feedback table.
Better suited for PostgreSQL databases.

Conclusions
Based on those facts, we decided to change the datatype of the feedback.detail_text to text, for two reasons: a performance impact is unlikely, and the feedback table remains extensible in the future.
Allowing up to 15.000 characters in a varchar would be acceptable to us, too. However, we believe that this severely limits the future extensibility. Additionally, allowing the flexibility a text column provides, other instructors can freely decide which amount of feedback they find acceptable based only on the actual task/exercise, not based on database limitations.
A temporary compromise to investigate the possible performance impact would be to change the data type to text in the database, but keep the maximum length enforced in the Artemis code by FEEDBACK_DETAIL_TEXT_MAX_CHARACTERS to 5000.
With that the impact could be tested on a large database like at TUM, with the option to later revert the data type back to a varchar(n) with $n \geq 5000$ without data loss.
Additional Considerations
PostgreSQL supports arbitrary length texts, MySQL sadly does not. Therefore, a length check in the code has to remain to keep supporting both database types.
Since the limit for text objects is not counted as actual characters, but bytes, the actual length limit is less than 65.535. For FEEDBACK_DETAIL_TEXT_MAX_CHARACTERS we decided that 50.000 should be length where most feedbacks should safely fit even if they contain many unicode characters that take up more than one byte each.
COMPLAINT_RESPONSE_TEXT_LIMIT and COMPLAINT_TEXT_LIMIT face this same issue but have not been adapted, since this might be a breaking change: The @Size constraint on the corresponding attributes of the Complaint domain object might fail in case some elements already exist which exceed this limit.
[1] https://dev.mysql.com/doc/refman/8.0/en/char.html
[2] https://dev.mysql.com/doc/refman/8.0/en/column-count-limit.html
[3] https://dev.mysql.com/doc/refman/8.0/en/charset-unicode-sets.html
[4] https://dev.mysql.com/doc/refman/8.0/en/blob.html
[5] https://dev.mysql.com/doc/refman/8.0/en/internal-temporary-tables.html
[6] https://www.postgresql.org/docs/current/datatype-character.html
Steps for Testing
Do NOT deploy this PR on a test server
Prerequisites:

1 Instructor
1 Programming Exercise

Submit to the programming exercise.
The feedback should be shown as usual.

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage
unchanged
Appendix

Feedback table size

column
type
size (B)

id
bigint
8

detail_text
varchar(5000)
20000

text
varchar(500)
2000

result_id
bigint
8

type
varchar(255)
1020

positive
boolean
1

feedbacks_order
integer
4

credits
double
8

reference
varchar(255)
1020

grading_instruction_id
bigint
8

visibility
varchar(255)
1020

Σ

25097

⇒ remaining bytes: $65535 - 25097 = 40438$
⇒ divided by 4 bytes per character, 10.000 additional characters possible

We cannot merge this PR at the moment, because we have issues with the feedback table becoming too large at TUM. At the moment, it includes >100 Mio entries and uses >30GB disk space.
A migration for a datatype on this table alone would need a down time for more than 1 hour.
We also experienced other issues in the past with too long feedback and have a very good reason to limit the feedback length. I don't think it is appropriate to provide log statements as part of the feedback. I would suggest, we find a different solution here

Unfortunately, this is a major problem for us in Passau to go live with Artemis for the first time in our Programming II lecture in the summer term 2023, as Gordon may already have told you. We have worked nearly three years torwards that goal now. As you know, we use black box testing against the running command line programs of the students, where the feedbacks are full dialogs with inputs for and outputs of this program runs (enriched with expected values). For that 5000 characters is in many cases a (too) strong limitation.
So it seems the real problem is the feedback size in total of the so many submission/test runs. Would it be a solution to only store the feedback of the last submissions/pushes? Is it realy so much helpful to store all feedbacks of every push or wouldn't it be enough to permanently store only some of them?
For sure another problem is mysql, however, our test systems are already converted to postgresql and our production system will follow soon.
@Stephan: I would be happy if we could have a telephone/skype/zoom call about that topic.

Artemis already allows the students to download build artifacts. In most cases this is a .jar file or any other executable generated by the students' submission.
But in these cases, the artifact could also be the log file. The students then can download the log and view it in their local text editor.
Would this be a sufficient solution for your case?

Re: build artefacts
I was not aware of this feature since it seems to be implemented only for Bamboo setups. This could be solved, though.
However, tests ‘after due date‘ and/or ‘secret‘ tests would not really be possible in this case. I would take quite a bit of new code basically duplicating the logic for test case visibility to decide which artefacts should be visible at which point in time to which users.

As discussed yesterday, closing this in favour of #6296 outlining the proposed solution.


[Fix] Correct config.json content

Describe your changes
Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics?
[ ] Will this be part of a product update? If yes, please write one phrase about this update.

Not relevant due to PR-64


How to out put captured output on the UI？

Hi again. I have fixed running Tensorboard in the latest dev version. But it always starts it on port 6006, so no multiple tensorboards supported (yet) :(


Multiband Raster Support

@Kirill888 , is there support for loading mutliband cogs ind odc-stac ?

For example, the visual  band is multi band and only loads a single band.
Thanks in advance.

odc-stac does support multi-band images but it requires stac items to use eo extension and describe all the bands.
But this is mainly for "hyper-spectral" type of data stored within a single TIFF image, and not for "RGB(A)" assets, eo:bands should still work for those too.
Can you provide an example stac item from which screenshot above was generated.

@julianblue rgb(a) sources introduce a bit of a conundrum actually, as they can be seen as either

3 or 4 separate bands with common footprint and aligned pixels (t,y,x coords on 3/4 DataArrays)
As as single "visual band" (t,y,x,band coords on a single DataArray)

I'm guessing your expectation for the output would be option (2)? Currently only option (1) is supported and only if eo:bands are present on the asset being loaded. I'm not aware of any STAC extension mechanism to indicate that loading data in t,y,x,band format should be preferred as an asset contains rgb(a) imagery. Maybe @gadomski or @TomAugspurger can comment on the STAC side of things?

odc-stac does support multi-band images but it requires stac items to use eo extension and describe all the bands.
But this is mainly for "hyper-spectral" type of data stored within a single TIFF image, and not for "RGB(A)" assets, eo:bands should still work for those too.
Can you provide an example stac item from which screenshot above was generated.

The example I mentioned was for the true color image of a sentinel 2-l2a tile. https://stacindex.org/catalogs/earth-search#/item/43bjKKcJQfxYaT1ir3Ep6uENfjEoQrjkzhd2/NopKk2m24tJYiekqbLp72LWk57p1ZsViRLdV3HiFw13eHu73QTYZUKAB?t=2#9/-6.825703/133.001187
Ideally the returned xarray object would have the visual raster listed as a single band/data variable, although I see the complications of inconsistent band dimensions through all data variables.
To extend this question, I am especially looking to load multi band geotiffs though odc-stac. For example, RGBNIR Planet scope data. What would a stac item need to look like to load this data via odc-stac ?

After patching data and patching code I was able to load rgb
xx = stac_load(
    [item_patched],
    bands=("visual.1", "visual.2", "visual.3"),
    chunks={},
    resolution=160,
)
rgb = xx.to_array("band").transpose("time", "y", "x", "band")
display(rgb)
rgb.compute().plot.imshow(col='time');

@julianblue

Just for clarification, is it best practice to have both eo:bands and raster:bands as properties ? Does odc-stac require both to create the xarray object ?

This page describes what extensions are used by odc-stac when available:
https://odc-stac.readthedocs.io/en/latest/stac-best-practice.html
odc-stac aspires to be able to load data from any valid STAC collection with GeoTIFF assets, in practice that might not be always possible. STAC by itself guarantees very little beyond a link to the file, timestamp and "approximate" region covered. One can always inspect the image files to figure out everything that raster:bands and proj:* provide. When you are processing thousands of files together to form a data-cube, extracting that metadata from each asset ahead of time is not really feasible (slow, and in case of remote Dask cluster might not be even possible due to access rights), so moving that information into STAC is the best practice.
When extensions are not supplied, rather than inspecting all the assets to figure out things like dtype, nodata, native resolution, width/height and projection of the assets odc-stac either requires information from the user (desired CRS and resolution of the output), or defaults to something reasonable (dtype=float32, 1 single band per asset). There is also user configuration, which is basically an override for missing or incorrect metadata.
We could potentially introduce a mode where we inspect one of the items from a collection and then assume that all of them are like that, this should at least work for dtype, nodata and resolution, and maybe CRS (will need to detect "families" of projections, i.e. UTM).

@julianblue in STAC the same "common_name" can be attached to multiple bands of different assets so there is potential for clashing of output bands, so we default to asset name which is guaranteed to be unique. But this approach obviously breaks down for multi-band assets. You can supply bands = ["Red", "Green", "Blue", "NIR"] and it will pick the right band from the right assets and name output data vars properly, but this would break if there are other assets using the same common_name as then there is an ambiguity as to which "red" you mean.
I think logic should check if asset pointed by k has more bands than 1 and act differently
- if idx == 1:
+ if idx == 1 and self.band_count[asset] == 1:


Screen sizes

For the breakpoints, consider both device-size and screen size.
https://github.com/bcinarli/caffeine/blob/master/lib/mixins/_breakpoints.scss

For tablets device-size used.


BAEL-11: Craedel feign

Hi,
this is the Feign client without Lombok dependencies. Please pick the latest BAEL-11 commit.
Regards,
Christian

already merged


Command (grayscale) is not available for driver

I have tried using both GD and Imagick and both return the same error.
I have tested using PHP 5.5, 5.6 and 7 to see if that could have been the issue however it didn't change anything.
Here is the code I am using. I can use all the other api methods except this one, which is the one I need.
$img   = Image::make($image)->grayscale()->save($savePath,100);

It's greyscale()
http://image.intervention.io/api/greyscale

Am 11.05.2016 um 17:40 schrieb Michael Fairchild<EMAIL_ADDRESS>I have tried using both GD and Imagick and both return the same error.
I have tested using PHP 5.5, 5.6 and 7 to see if that could have been the issue however it didn't change anything.
Here is the code I am using. I can use all the other api methods except this one, which is the one I need.
$img = Image::make($image)->grayscale()->save($savePath,100);
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Thanks! I could have sworn I had copy/pasted from the docs. Simple mistake. Thanks!


When Logged out navigating to dataset produces AttributeError

When not logged in navigating to the dataset page produces the following error: AttributeError: 'NoneType' object has no attribute 'id'

This issue seems to be resolved and should be closed.

you can close an issue yourself, especially if you reported it


change remember checkbox to accept checkbox in sign-up page

#1
Hello, this commit is to fix the checkbox issue on sign-up page.
The remember checkbox was changed to accept.

#1
Hello, this commit is to fix the checkbox issue on sign-up page.
The remember checkbox was changed to accept.


🚀 Updated to .NET Standard 2.0.

Also cleaned up the error message to be nicer 👍 🍰

Fixes #30
Fixes #31


Add IP Allow List Events to Org Security Audit Log Review

Why:
I noticed that the Audit Log Review docs didn't contain information regarding ip_allow_list and ip_allow_list_entry in the Organization Security section, so I updated the information on the page with that data.

What's being changed:

Below integration_installation_request I've added:
### `ip_allow_list` category actions

| Action | Description
|------------------|-------------------
| `enabled` | Triggered when an IP allow list was enabled for an organization.
| `disable` | Triggered when an IP allow list was disabled for an organization.
| `enable_for_installed_apps` | Triggered when an IP allow list was enabled for installed GitHub Apps.
| `disable_for_installed_apps` | Triggered when an IP allow list was disabled for installed GitHub Apps.

### `ip_allow_list_entry` category actions

| Action | Description
|------------------|-------------------
| `create` | Triggered when an IP address was added to an IP allow list.
| `update` | Triggered when an IP address or its description was changed.
| `destroy` | Triggered when an IP address was deleted from an IP allow list.

Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@Hollywood
Thanks so much for opening a PR! I'll get this triaged for review :zap:

Any time @ramyaparimi!

Allow

Write

Leave a comment

Почему:
Я заметил, что документы обзора журнала аудита не содержат информации о разделе ip_allow_listи ip_allow_list_entryв нем Organization Security, поэтому я обновил информацию на странице этими данными.
Что меняется:
Ниже integration_installation_requestя добавил:
### `ip_allow_list` category actions

| Action | Description
|------------------|-------------------
| `enabled` | Triggered when an IP allow list was enabled for an organization.
| `disable` | Triggered when an IP allow list was disabled for an organization.
| `enable_for_installed_apps` | Triggered when an IP allow list was enabled for installed GitHub Apps.
| `disable_for_installed_apps` | Triggered when an IP allow list was disabled for installed GitHub Apps.

### `ip_allow_list_entry` category actions

| Action | Description
|------------------|-------------------
| `create` | Triggered when an IP address was added to an IP allow list.
| `update` | Triggered when an IP address or its description was changed.
| `destroy` | Triggered when an IP address was deleted from an IP allow list.

Отметьте следующее:

[ ] Я просмотрел свои изменения в постановке (найдите последнее событие развертывания на шкале времени вашего запроса на вытягивание, затем нажмите « Просмотреть развертывание» ).
[ ] Что касается изменений содержания, я заполнил контрольный список самопроверки .

Влияние писателя (этот раздел предназначен только для сотрудников GitHub):

[ ]  Этот запрос на вытягивание влияет на опыт участия

[ ]  Я добавил метку "влияние писателя"
[ ]  Я добавил описание и / или видео-демонстрацию изменений ниже (например, «видео до и после»)

Hi
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Ethan Palm @.>
Sent: Monday, October 4, 2021 5:17:59 PM
To: github/docs @.>
Cc: Subscribed @.***>
Subject: Re: [github/docs] Add IP Allow List Events to Org Security Audit Log Review (#10261)
Merged #10261https://github.com/github/docs/pull/10261 into main.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/github/docs/pull/10261#event-5406726041, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVXVWOBSQ3V7JTG3LZVCK6TUFIKYPANCNFSM5EUFYA7A.


Production deploy #35

10 gem updates after the install of Dependabot
Changes to the jobs listing view on mobile
Reporting to the performance platform
Hiring staff who land on the sign-in step when are already logged in will be automatically redirected to their schools page and will no longer see this page
Users of DfE Sign-in will see the DSI 'Choose organisation' page if they have more than one organisation
Users who do not have a DSI organisation configured will not be authorisable we now use a combination of email and organisation_id from DSI to check our authorisation service for approval
CloudFront caches all files under the /assets directory for efficiency improvements

...to avoid any confusion, until I get the rake task update in.

Sure. Thanks for the review.


The first page that I need transition from is a shell Item and hence not able to make it a SharedTransitionNavigationPage. Any workaround for this ?

I realized later that you are already looking into this and hence closing the issue.


Find new way to link to teams en-masse quickly

Currently, to link to a team's page, we use this format in the templates:
{{ team }}
This is incredibly slow on pages with lots of links, such as /e/, the events list page, where a tags are generated as so:
{% with event.1.teams.all as a %}

        {{ a.0 }}

        {{ a.1 }}

        {{ a.2 }}

{% endwith %}

where event.1 is simply accessing a tuple of the form ( ... , event.winning_alliance, ... ... ).
This bogs down page load time by a large margin and we should optimize pages like this.

Right now I'm just going to replace things with /t/{{ team.id }}/, but I'd rather have a better solution in place.

See https://github.com/FRC-RS/FRS/commit/655a6ad09e06fff7bcf80bd161af663a958bb4e7
and https://github.com/FRC-RS/FRS/commit/cdea22b94c94b273c0ef317f10189113829b55d2


add src/SYSCONFDIR.imp to gitignore

Hey this file keeps popping up in git console when I run a simple build - how about adding it to the ignore file?

Auto-merge toggled on

At some point we really ought to move generating stuff in the source directory and have a separate build folder.


Rename proto::SparsePoseGraph.

RFC=0001

@wally-the-cartographer merge

Merge requested by authorized user wohe. Merge queue now has a length of 1.


Monthly app mining schedules can incentivise "bad" UX choices.

What is the problem you are seeing? Please describe.
Bad UX choices are being forced an App developers due to app mining schedules.
How is this problem misaligned with goals of app mining?
Monthly app mining schedules incentivise bad UX choices, devs are incentivised to get the app published before they would normally. This can result in functionality as a priority and delayed UX/UI design.
What is the explicit recommendation you’re looking to propose?

Submitting an app for mining should be separated from submitting it to UX/UI review.
Founders need to be able to choose when they use the UX review service.
App mining schedule could be weekly instead of monthly, which reduces but doesn't eliminate the incentive to deploy without proper design.
Analyse apps at the end of the period, not at the beginning.

Describe your long term considerations in proposing this change. Please include the ways you can predict this recommendation could go wrong and possible ways mitigate.

It can go wrong because the App mining schedule should be daily not weekly or monthly but practical realities may dictate that.
It can go wrong because there is no organised human oversight.
It's just one change of several that are needed to re-align incentives and any one of them can have unintended consequences.
I am presuming that human-in-the-loop is scalable.

Perhaps proposal #230 can help by splitting app mining into multiple cohorts that can have staggered starting times.


Removing error handle and its dependency from all workbook

PR Checklist

[ ] Explain your changes, so people looking at the PR know what and why, the code changes are the how.

If adding or updating templates:

[x] post a screenshot of templates and/or gallery changes
[ ] ensure your template has a corresponding gallery entry in the gallery folder
[ ] If you are adding a new template, add your team and template/gallery file(s) to the CODEOWNERS file. CODEOWNERS entries should be teams, not individuals
[ ] ensure all steps have meaningful names
[ ] ensure all parameters and grid columns have display names set so they can be localized
[ ] ensure that parameters id values are unique or they will fail PR validation (parameter ids are used for localization)
[ ] ensure that steps names are unique or they will fail PR validation (step names are used for localization)
[ ] grep /subscription/ and ensure that your parameters don't have any hardcoded resourceIds or they will fail PR validation
[ ] remove fallbackResourceIds and fromTemplateId fields from your template workbook or they will fail PR validation

This PR includes changes for the Azure Backup Workbook.
So we have removed the ErrorHandle which was earlier preventing any exception that might occur if the log analytics workspace is empty. Since now the workbook team has improved the UI and we are not getting any exception for the empty LA workspaces, so we have removed the Error Handle and its dependencies to improve the overall efficiency and thus loading various tabs and sub-tabs faster.


Backup Possibility

I am running python server
How can backup server and client settings and transfer it to another server? is it possible?

Hello
Yes. At first install the proxy on new server, (with my script) and then copy all /opt/mtprotoproxy to the new server and overwrite anything.

I am using centos, is any problem occur if centos version will upgrade from
7.4 to 8.1
On Wed, Mar 18, 2020 at 5:51 PM Hirbod Behnam<EMAIL_ADDRESS>wrote:

Closed #23
https://github.com/HirbodBehnam/MTProtoProxyInstaller/issues/23.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/HirbodBehnam/MTProtoProxyInstaller/issues/23#event-3141671272,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ5HKJE32Z6IE64SKJGGKSTRIDKGDANCNFSM4LOSO4YQ
.

@A2116 This script might not work on Centos 8

if I want to upgrade to the new script that supports random padding and I copy all /opt/mtprotoproxy to new server then is the new script will work? or I should only transfer certain files to the new server?

Hello
No I will not add backup support to script. You can manually copy the files as said. Also the script supports random padding you have to enable it when it's installing (as I said in your previous issue)
Also it's a good idea to copy all of the files to new server.

when I copy all files mtprotoproxy.py also will be copied and I think it's the main file
is this lead to revert to the old version or not?

Please note that if you are going to use newer scripts with older MTproto installs, you might brick the config file.
So my best suggestion is to copy all of the files including the mtprotoproxy.py and script itself.
Now two things:

If you are going to upgrade the MTProtoProxy you can do it via script without any worries.
If you are going to upgrade the script itself, completely uninstall the MTproto proxy (without backups) and install a fresh one later. I do not grantee that the newer scripts are compatible with the older ones.

my main goal is to keep users and configs and upgrading server
so only thing that I need user keys and user limitations

As I said, It's a good idea to copy everything including the script.

I copy all /opt/mtprotoproxy content to another place
then I copy mtprotoproxy.service and MTProtoProxyInstall.sh to another place
then I uninstall it using the script and remove MTProtoProxyInstall.sh file and download the new script using curl
then I installed the new script and create a new user and add all limitation to it to reproduce config files
by comparison of the new and old file, I recognized that the main structure does not change and so I copy config.py and limits_bash.txt and limits_date.json and limits_quota.json content to new files and restart the server
everything works fine and all users can connect using their old configurations
so I think migration process becomes completely successful


Button inside tooltip component is not resizeable

Describe the bug
We should be able to manipulate button sizes as we please. E.g. making it full width
Screenshots
Current

Expected (if we want to make the button full width)

Hi maintainer, if you agree to this you can assign it to me so I can push a PR. Cheers 🍻

Issue is connected with the "w-fit" class set on button in Button.svelte.
This will disappear in one of upcoming PRs.

I close the issue for now since it it merged.


Very unhelpful startup traceback if the syslog is unavailable

If for some reason the system logging socket is unavailable (i.e. the syslog failed to start, or is not installed) all of the daemons and init tasks provided by this package die with a traceback for a socket error.  The errno returned is 2 (no such file or directory).
It would be nice to catch that exception when the logging is starting up and spit out a print statement that the logging socket is unavailable.  It would also be nice to have an option to gracefully degrade rather than just hitting a hard fault.

The guest environment relies upon syslog to present errors and other logging information to users outside of the VM. If syslog isn't available, catching the exception won't be very helpful since we won't be able to log it anywhere.
That said, the error is likely coming from logger.py which each daemon uses. If you'd like to add exception handling there, please make a pull request.

That makes sense.  Where catching the exception is very helpful is bootstrapping a new distro which does not have supported packages, logged in to a shell I was able to see the exception and would therefor also see the error that the scripts need syslog to be up.
I will prepare a PR shortly.

Hi, are you still planning to make the PR for this issue?

I have a patch, but I'm not really satisfied with how this works.  I'll go ahead and close this issue.


ARROW-1697: [GitHub] Add ISSUE_TEMPLATE.md

This will save us repeating ourselves when users report bugs or request features on GitHub

+1


Search facet bug

Getting weird behavior on the Search facet searches.  I'll start clicking on one, then another, then another (without unchecking any).  The second facet will cause an error, then unchecking it and checking it again - it works, third facet - same thing - broken, uncheck, recheck works - I'm getting the following 400 error associated with this when I'm first getting the checked facet error:
{
status: 400,
msg: "Invalid JSON filter: {"category":"experiment_context","name":"Cell","value":"stem cell".",
detail: "Unexpected end-of-input: expected close marker for OBJECT (from [Source: [B@352be7df; line: 1, column: 0]) at [Source: [B@352be7df; line: 1, column: 133]"
}

The 400 indicates invalid JSON was sent in a filter parameter.
The error indicates that the closing } was omitted.

Thanks!


.net 4.0 is not available

My project is developed by.net 4.0, now nuget cannot be used, install-package xffmpegnet-version 3.4.0 cannot be installed。
ERROR:
The severity code indicates that the project file line forbids display status
The warning failed to resolve the main reference "netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51", Because it is the framework assemblies "System. Diagnostics. Tracing, Version = <IP_ADDRESS>, Culture = neutral, PublicKeyToken = b03f5f7f11d50a3a" indirect dependencies, and in the frame of the current target failed to resolve the assembly. ". NETFramework, Version = v4.0 ". To resolve this issue, remove references to "netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51", Or the application of the goal to contain "System. Diagnostics. Tracing, Version = <IP_ADDRESS>, Culture = neutral, PublicKeyToken = b03f5f7f11d50a3a" framework Version. FFmpegDemo

Added support for .NET Framework 4.5
This should be available when #42 pull request is accepted

I decided to only maintain netstandard2.1 as a minimum which means that this library won't be available for .net4.0


Use InjectionToken instead of OpaqueToken

Thanks for publishing this library. Very simple and easy to use.
Angular 4 introduced InjectionToken and deprecated OpaqueToken.
Now with Angular 5, OpaqueToken is removed.
Could it be possible to update the library for angular ^4.0.0 by making the above change?
The change is required only in LogObserverService.ts. Something like the following.
import { Injectable, Inject, InjectionToken } from '@angular/core';

export let LOG_LISTENER = new InjectionToken<ILogListener[]>('LogListener');

I am also looking for a solution which is angular 5 compatible - will this move forward and is this project still active?

I will be updating to support angular 5 this week.

Will this issue be resolved soon? It's just what we need in our project, but we cannot use it because we already use Angular 5.

@Igor-G The Angular 5x version is here: https://github.com/unisoftapps/ng-log-service


Report API errors more clearly

Same approach as used in prison-visits-public, but for calls into NOMIS.

I'm closing this. We've already done some work around API errors + monitoring


Delete redundant gettext.test.js file

As per https://github.com/wagtail/wagtail/pull/8747#issuecomment-1294852488 - as far as I can see this test suite serves no purpose, as each test in it is simply mocking a function from gettext.ts, calling the mock, and asserting that the mock has been called, so no code from the real functions ever gets run.
If we do keep this, we should update the ngettext test to pass string variables for 'One bird' / 'Many birds', like the gettext test does, so that this line doesn't get picked up by extract-translatable-strings.js as part of the real translation strings sent to Transifex.

Yes this does seem odd. I'll take a look and try to see if we can make these tests useful.
I think originally it was to ensure that the imports called the window functions and this did actually uncover a bug with the functions when I wrote this.

An alternative approach to just deleting the test file - PR up https://github.com/wagtail/wagtail/pull/9617

Replaced with #9617


complete sync only on 204

Sync process should complete only when HTTP 204 is returned. Current version of client completes sync process if returned item collection is empty even though 204 has not been returned. This is incorrect behaviour as sync is still in progress and next request can still return items. This PR delivers a fix to this.

LGTM

Looks good.


Browserwindows opening as subwindows of vscode

Issue Type: Bug
-open a link from code in vscode with ctrl+click
-now open your standart browser outside of vscode
-the browser window will appear under the vs code section
-close all browser windows in the vscode section
-now open your standart browser outside of vscode
-the browser window will appear under the standart browser section
VS Code version: Code 1.29.0 (5f24c93878bd4bc645a4a17c620e2487b11005f9, 2018-11-12T07:42:27.562Z)
OS version: Linux x64 4.15.0-39-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 2481)

GPU Status
2d_canvas: unavailable_softwarechecker_imaging: disabled_offflash_3d: unavailable_softwareflash_stage3d: unavailable_softwareflash_stage3d_baseline: unavailable_softwaregpu_compositing: unavailable_softwaremultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: unavailable_softwarevideo_encode: unavailable_softwarewebgl: enabled_readbackwebgl2: unavailable_off

Load (avg)
2, 1, 0

Memory (System)
15.40GB (11.42GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

monokai-charcoal-high-contrast
74t
1.0.4

dart-code
Dar
2.20.0

flutter
Dar
2.20.0

bazel-code
Dev
0.1.9

jupyter
don
1.1.4

vsc-material-theme
Equ
2.5.1

nbpreviewer
jit
1.0.0

python
ms-
2018.10.1

cpptools
ms-
0.20.1

markdown-preview-enhanced
shd
0.3.11

cmake
twx
0.0.17

cmake-tools
vec
1.1.2

(3 theme extensions excluded)

-the browser window will appear under the vs code section

I'm not sure what you mean by "the vscode section"