Add NP to allow comunication with sonataflow-infra

Fixes FLPATH-1828

Isn't https://github.com/parodos-dev/orchestrator-helm-operator/blob/main/helm-charts/orchestrator/templates/network-policies.yaml#L28 enough?

Isn't https://github.com/parodos-dev/orchestrator-helm-operator/blob/main/helm-charts/orchestrator/templates/network-policies.yaml#L28 enough?

that networkpolicy enables communication from the rhdh namespace to the sonataflow-infra one.
however, it turns out that there is a need to be explicit about enabling traffic within pods in the sonataflow-infra namespace which should be enabled by the added networkpolicy.

@gciavarrini pls make sure to update the document - https://github.com/parodos-dev/orchestrator-helm-operator/pull/353/files#diff-f9265e0705f55eef4885afe9d6cf0d4ee1861db4e03c9f4cfa0d01bbb3df945fR7
if your PR is going to be merged after.

that networkpolicy enables communication from the rhdh namespace to the sonataflow-infra one.
however, it turns out that there is a need to be explicit about enabling traffic within pods in the sonataflow-infra namespace which should be enabled by the added networkpolicy.

@masayag that's interesting, then the comment https://github.com/parodos-dev/orchestrator-helm-operator/blob/main/helm-charts/orchestrator/templates/network-policies.yaml#L26-L27 should be removed: it claims that adding this rule will allow any pods in namespace that has the label rhdh.redhat.com/workflow-namespace: "" to communicate with the pods in the NS "{{- include "get-workflow-namespace" . }}"
But sontaflow-infra itself has the label so its pods should be part of the allowed set, what am I missing? Maybe the namespace in which the NP is deployed cannot be targeted using namespaceSelector

@gabriel-farache I replied too quickly without viewing all of the aspects of that network policy.
The existing network policy enables traffic to the sonataflow-infra from

RHDH instance namespace
knative-serving namespace
Any namespace that has workflows deployed in (identified by rhdh.redhat.com/workflow-namespace label)

However, we don't have that label for our own sonataflow-infra namespace, and it will be good to be explicit regarding this part by creating a specific NP as this PR suggests since its role is different (internal vs external traffic).
The next comment is unrelated to this topic: we may evaluate 3 option and restrict the traffic only from the workflows pods and not from the entire namespace to be more strict.

However, we don't have that label for our own sonataflow-infra namespace,

@masayag But we do. As we checked together, it's the setup script that is setting this label, hence a user not using the setup script (i.e: existing RHDH instance) won't have the label set.
Hence this PR is needed indeed to make sure the infra communication will work, even on existing RHDH instance

@gabriel-farache had the same doubt, but @masayag explained that:
Adding the CR simplifies the process for the installation on existing RHDH instance.
We can't add the label because the resource is not under the helm operator. We usually don't edit existing resources by the helm chart, because they might be controlled by ArgoCD and then changes will make them go out of sync and be reconciled.

/retest


IPsec IPv4/IPv6 pools parameters. Issue #10296

[ ] Redmine Issue: https://redmine.pfsense.org/issues/10296
[ ] Ready for review

After last commit https://redmine.pfsense.org/projects/pfsense/repository/revisions/407a5c28093d46cb39cc1bba75740523a1ee97e6
The mobile-pool-v4 and mobile-pool-v6 pools are created as expected.
But seems that "addrs" is required for each pool, so the mobile-pool is not created (or referenced in connections).
Maybe the commons attrs should be included in the v4 pool or both for now?
This PR adds common attrs to each pool + description fix for vpn_ipsec_mobile.php, because RADIUS IP address priority applies to both ipv4 and ipv6

ok, I simply revert back that v4+v6 mobile pool code

What change?

Strange, it didn't post my review comment. I'll do it again.

Tested (EAP-TLS and EAP-RADIUS) and this works. But, the DNS info from mobile-pool is not passed.

If I revert back https://github.com/pfsense/pfsense/commit/fb0827f350e417cf29e0f3bb8d2f8ecefb2bb928 DNS is passed.
strongswan inherit bug?

I'm also curious to know if it's just DNS that isn't passed down, or if other fields like the default domain get omitted.
We could copy all of that into the other pools (v4, v6, per-user pools) but it seems broken to have to repeat it all when the references/templating are supposed to allow for that.

If mobile-pool doesn't have addresses then it's not being referenced at all as I understand it.  If mobile-pool does not have other utility (I have no clue) then seems that it would be better to apply the dns info to each of the v4/v6 pools.  That doesn't seem broken to me, since it would would work regardless of whether assignments were v4, v6 or both.

In the swanctl/strongswan.conf style syntax you are supposed to be able to setup templates with base settings and then make reference to them so others inherit the "defaults" from there and add their own options which differ. The mobile pool wouldn't be used directly in this case, it's a container for default values. Others reference it to obtain the common settings.
We can copy the defaults to every pool but that's inefficient and redundant. It should be working this way, and if it's not, there is likely a bug in strongSwan.

Aha.  Sorry I missed that.
Since the mobile-pool is defined as a real pool within the pools section, not just as a top-level default, it's reference should be "pools.mobile-pool", no?
For example:
1552c1552
<               $scconf['pools']['mobile-pool-v4 : mobile-pool'] = array();
---
>               $scconf['pools']['mobile-pool-v4 : pools.mobile-pool'] = array();

That's part of it, but it looks like per-user settings are also broken in other ways. I'm working on it now, it will supplant any changes made here, so I'm closing this out.

For those following along here, this should be OK on the next snapshot. See also: https://redmine.pfsense.org/issues/10314 and https://redmine.pfsense.org/issues/10296


Polio 1207 add more reasons for delay

Add more reasons for delay
Related JIRA tickets :  POLIO-1207
Self proofreading checklist

[X] Did I use eslint and black formatters
[X] Is my code clear enough and well documented
[ ] Are my typescript files well typed
[X] New translations have been added or updated if new strings have been introduced in the frontend
[X] My migrations file are included
[ ] Are there enough tests
[ ] Documentation has been included (for new feature)

Changes

Add in the model + migration
Add in the dropdown + translation

How to test
Go to polio> campaigns
Go to a round
Edit dates
check the dropdown
select the new reason
save
Print screen / video
https://github.com/BLSQ/iaso/assets/38907762/cb0d4a71-9fe7-438d-9c36-e2e2ff19ac97
Notes
merges in hotfix-reasons-for-delay-VRF

closing as I will make a cleaner version to avoid mixing up main with polio hotfixes


Update Version to 3.1.0

Should have been done as part of this PR: https://github.com/DFE-Digital/dfe-reference-data/pull/97

@ericaporter I forgot to update the version as part of the last PR. Could you check that this is correct? I've tagged the commit with the new version number as well.


Logarithmic volume scale for Spotify Connect

librespot recently got support for logarithmic volume scaling (see https://github.com/librespot-org/librespot/pull/10 ). It would be great if pimusicbox would support it as well :)

Sure, there's also support for repeat and shuffle. Have been playing with
the latest build for rc6 (aka v0.7.0)!
On 10 Feb 2018 11:28 am, "Fabian Henze"<EMAIL_ADDRESS>wrote:

librespot recently got support for logarithmic volume scaling (see
librespot-org/librespot#10
https://github.com/librespot-org/librespot/pull/10 ). It would be great
if pimusicbox would support it as well :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pimusicbox/pimusicbox/issues/465, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5DqP1tfalMvyn6ARGFMU7-83HI4i1iks5tTX1LgaJpZM4SA5aa
.

Oops, I forgot to enable this in rc6. rc7 is coming imminently to fix the mounting issues so I will sort this too.

And oops again I forgot it's enabled by default. That's twice I've made that mistake in as many weeks! So good news is that this is already available in https://github.com/pimusicbox/pimusicbox/releases/tag/v0.7.0RC6.


leaving more room for table, horiz scrollbars gone by default

See #64

Tested this on Chrome, Firefox, IE, and Safari

the function getScrollBarWidth() already exist in GigaGrid.tsx

Removed the large one and retested successfully


Peer count doesn't fully recover after network partition

Peer count doesn't fully recover after network partition
Description
We've noticed an issue where if we kill connectivity (specifically outbound connectivity) to a node, in some cases it does not fully recover to the total peer count
Your environment

OS and version Ubuntu 20
version of the Polygon SDK develop
branch that causes this issue develop

Steps to reproduce

Tell us how to reproduce this issue
Create a 4 node cluster (all bootnodes).  Simplest way is to make a 4 node cloud cluster and just toggle the security groups for outbound on AWS for example. You can try some variant of this script in node:

import EC2 from "aws-sdk/clients/ec2";

function randomIntFromInterval(min: number, max: number) {
  // min and max included
  return Math.floor(Math.random() * (max - min + 1) + min);
}

function notEmpty<TValue>(value: TValue | null | undefined): value is TValue {
  return value !== null && value !== undefined;
}

const fullyConnectedGroups = [
  "sg-a",
  "sg-b",
  "sg-c",
];
const disconnectedGroups = ["sg-a", "sg-b"];
// You can set the fullyConnectedGroups to be the security groups required for fully connected nodes, i.e., all nodes can community (full inbound/full outbound)
// You can set disconnectedGroups to be a subset (or an empty set) of the fullyConnected groups, for example, remove the one that allows for outbound access (sg-c in this case)

export async function main() {
  const ec2 = new EC2({ region: "us-east-1" });
  var params = {
    Filters: [
      {
        Name: "tag:network",
        Values: ["toronto"],
      },
    ],
  };
  const instances = await ec2.describeInstances(params).promise();
  const instanceIds = instances
    .Reservations!.map((reservation) =>
      reservation.Instances!.map((instance) => instance.InstanceId)
    )
    .flat();
  console.log(`Instance IDs ${instanceIds}`);
  const nonEmptyIds = instanceIds.filter((id) => id !== undefined);
  while (true) {
    // Shuffle array
    const shuffled = nonEmptyIds.sort(() => 0.5 - Math.random());

    // Get sub-array of first n elements after shuffled
    let selectedIds = shuffled
      .slice(0, randomIntFromInterval(0, instanceIds.length))
      .filter(notEmpty);
    console.log("Selected IDs to disconnect ", selectedIds);
    for (const id of selectedIds) {
      console.log(`Disconnecting ${id}`);
      await ec2
        .modifyInstanceAttribute({ InstanceId: id, Groups: disconnectedGroups })
        .promise();
    }
    const reconnectWaitTime = randomIntFromInterval(30, 60);
    console.log(`Waiting ${reconnectWaitTime} seconds to reconnect nodes`);
    // Wait random amount of time from 30 to 60s to reconnect
    await new Promise((r) => setTimeout(r, reconnectWaitTime * 1000));
    for (const id of selectedIds) {
      console.log(`Reconnecting ${id}`);
      await ec2
        .modifyInstanceAttribute({
          InstanceId: id,
          Groups: fullyConnectedGroups,
        })
        .promise();
    }
    const continueWaitTime = randomIntFromInterval(1, 20);
    console.log(`Waiting ${continueWaitTime} seconds to continue`);
    await new Promise((r) => setTimeout(r, continueWaitTime * 1000));
  }
}

main().then(() => console.log("Done"));

Where the issue is, if you know
Which commands triggered the issue, if any

Expected behaviour
Nodes should recover to the total peer count
Actual behaviour
Nodes only recover to a fraction of the total peers
Logs
See our Grafana screenshot with the new metrics from
https://github.com/0xPolygon/polygon-sdk/pull/302
You can see the behaviour prior to the script starting, and then afterwards when connectivity is restored. This was a 4 validator node cluster with 2 additional, non-sealing nodes online. Full connectivity was restored at 11:02

Here is another example with the 4 validator nodes and the additional 2 non-sealing nodes offline. Full connectivity was restored at 12:15:

In general it looks like these 2 errors come as a pair for each node:
[ERROR] polygon.network: failed to dial bootnode: id=16Uiu2HAmP2bpgr3sLUUzt61bEzAL7ptdNXnvsfnwcxyrfCMFjxDf err="rpc error: code = Unavailable desc = error reading from server: stream reset"

[ERROR] polygon.consensus.syncer: failed to notify: err="rpc error: code = Unavailable desc = error reading from server: stream reset"

Proposed solution
If you have an idea of how to fix this issue, please write it down here, so we can begin discussing it

Hello, @mrwillis. Thank you for raising the issue. I could reproduce this issue with docker.  Basically, this issue can be divided into 2 issues:
(1) Node can't reconnect to a peer if node failed to connect to the peer before. It's caused because peer will not be removed from routing table when node failed to connect. For that reason, node can't catch NewPeer event from routing table when node add peer info into routing table next time.
(2) Node won't connect to more bootnode once the node connects to one peer. The recovery process will happen and try to connect to one bootnode only when node doesn't connect to any peer. Let's say we have 4 bootnodes. And we let them connect to no peers firstly and fix the network issue after that. If A and B dial each other while C and D dials each other, we will have 2 sub-networks. In this case, each peer will not connect to new peer because they're connecting to one peer at least and recovery process doesn't happen.
I fixed (1) at https://github.com/0xPolygon/polygon-sdk/pull/318 and (2) will be fixed on another PR.

Hey @mrwillis,
Just wanted to give you a heads up that he 2nd PR @Kourin1996 mentioned is up now, so I'll link it 👍
PR #343


Appendix

Section 4.2 and 4.4 are missing the intro paragraph that describes the use case

Which document is this comment for? I am having difficulty finding a section 3.5. Please see https://github.com/CC-in-the-Cloud/General)/Offsite Review/ for this list of documents.

I found the location after cross-checking with the old version. Neither of these PP's have use cases for cloud. I can add the section and put 'none' if preferred.


[ci] Remove hardcoded test shards

This moves the sharding logic from being inlined in the Jenkinsfile to templated, so we can change just the number of shards and the test allocation in conftest.py and the Jenkinsfile will work to match. This also changes the test allocation from a manual balancing before to be random between shards. Each shard needs to know only its shard number and the total number of shards, then it hashes each test and skips it unless that hash falls within its allocated tests. This breaks up related tests across shards but has the downside that any change to the number of shards will shuffle around where the tests end up (but ideally this is rare as we settle on a good number of shards to use). Some tests are also manually allocated via round-robin to different shards to ensure that long-running tests run on different shards as much as possible.
This only does this for the GPU frontend tests but eventually we could expand it to more.
cc @areusch

I also verified the tests ran in the various shards vs the tests from a recent main run, the overall result of tests ran is the same so the sharding is working correctly. Code is here https://gist.github.com/f636700cd68b5717350c107a0eaaee4e for the curious.


Info: Reimplementation in Emacs Lisp

Hello,
I just want to inform you, that I started a reimplementation of fontpreview in Emacs Lisp,
because I often need fontnames for projects within Emacs. It's still in an early stage. Just
thought, you might want to know.
I gave you credits of course.
Sebastian Meisel

@SebastianMeisel Awesome! Let me know when you are finished so that I can take a look at it :grinning:

Oh, I forgot the link. It's already a working kind of prototype:
https://github.com/SebastianMeisel/fontpreview-mode.

@SebastianMeisel If I were you, I would put a nice demo gift or image in the start of the README so that when more people get attracted to your project ;)

I did that and also added a few features. I think it usable by now.

Looks neat! Post it on Reddit and get some users :)

Already done - well the posting part ;-)
Am Do., 30. Apr. 2020 um 21:29 Uhr schrieb Siddharth Dushantha <
<EMAIL_ADDRESS>
Looks neat! Post it on Reddit and get some users :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sdushantha/fontpreview/issues/37#issuecomment-622058041,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALURSNKX26GYDYHY4T3YG4TRPHGRPANCNFSM4MTDIMWA
.


Information about impact of maintenance

Hi,
The maintenance page has plenty of info about when maintenance might happen, but it doesn't mention the impact at all - are servers restarted? Is there downtime? How much? Or is it just a period of increased risk? For a service that offers High Availability I don't think anything can be assumed.

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

ID: 60c7f076-f889-895e-80b1-20b1d219957c
Version Independent ID: cae44e75-5096-766c-4e92-6324fea133c5
Content: Scheduled maintenance - Azure Database for PostgreSQL - Flexible server
Content Source: articles/postgresql/flexible-server/concepts-maintenance.md
Service: postgresql
Sub-service: flexible-server
GitHub Login: @GennadNY
Microsoft Alias: gennadyk

Hi @halfninja ,
Thanks for sharing this feedback. We will look into this and update as appropriate.
Hi @GennadNY,
Could you please look into this document request, thanks.

please assign to Varun Dharwan on my team

It does state the servers are restarted.
however, no mention of downtime.

Adding info to explain there is downtime for customers.
Article should be updated by the end of the week.
Thank you.

#please-close


Added support for absolute and relative URLs

Resolves #3111
Summary
URL page types now support absolute and relative URLs.  Furthermore, better handling has been added for URLs that begin with // or have not including a leading / (e.g., PageName, PageName#AnchorName).

Thank you sir and sorry for all the "edge cases" :)

No problem at all - it is a better PR now as a result.


Panasonic stop storing some values in API

I noticed that when the device is turned off, the values for the fanSpeed and ecoMode (Quiet and Powerful) parameters are always = 0.  Currently, for example, for Swing mode, the appropriate value is still maintained. But fanSpeed and ecoMode are zeroed every time. It looks like Panasonic store this directly in app?

@mkz212 they are probably stored on the device/server, if you send power on they should be restored.
The app disables all controls/info while the device is powered off and only shows a toggle.

@mkz212 they are probably stored on the device/server, if you send power on they should be restored. The app disables all controls/info while the device is powered off and only shows a toggle.

They probably changed it and the value is stored in the device. Therefore, after turning on the device, plug-in should also restore the value from the plugin and not from the api. Also, plugin should not check value for fanSpeed and ecoMode when device is off, cause from now it will always be 0.


allow setting default S3File's cache_type in S3FileSystem

Related to #213. The motivation is that sometimes we want to turn off caching by default (e.g. when reading parquet files with threads)
Since different fsspec implementations overrides _open() differently, I've chosen to add default_cache_type to s3fs instead of fsspec

Looks good, just the one nitpick remark

Thank you!


function name

Line306: https://github.com/nix-rust/nix/blob/master/src/sys/socket/addr.rs#L306
Line727: https://github.com/nix-rust/nix/blob/master/src/sys/socket/addr.rs#L727
change name to to_string is better ?

The ToString trait already provides a to_string function which does the same. It should already be implemented since those types implement Display, so I think to_str can just be removed.

I was actually looking through those types today and they could really use a refactoring. I believe we might be able to reuse types from std and get rid of some of our custom types. I was going to write an issue for it, but got lazy.


Sjölins gymnasium södermalm. Stockholm Sweden

Institution/School Name
Sjölings Gymnasium Södermalm
Instiution/School Website
https://www.sjolinsgymnasium.se/sodermalm/
Email Users
<EMAIL_ADDRESS>Please place an x between the square brackets if yes

[x ] Students
[ ] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[x] High School or equivalent
[ ] Elementary School or equivalent

@thicbruh Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Dev

Make HorizontalStackScrollView public to make possible creating
subclass of ChatInputBar with own xib

[Current coverage][cc-pull] is 64.75%

Merging [#153][cc-pull] into [dev][cc-base-branch] will not change coverage

@@                dev       #153   diff @@
==========================================
  Files            60         60
  Lines          3266       3266
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2115       2115
  Misses         1151       1151
  Partials          0          0

Powered by Codecov. Last updated by [f442111...ec61625][cc-compare]
[cc-base-branch]: https://codecov.io/gh/badoo/Chatto/branch/dev?src=pr
[cc-compare]: https://codecov.io/gh/badoo/Chatto/compare/f44211193704019aa6c74ab9e573aeef7458171d...ec6162581834f5d6f549da5662442b83177a23f7
[cc-pull]: https://codecov.io/gh/badoo/Chatto/pull/153?src=pr


CMake CUDA architecture flags should add PTX for All

"Auto" is okay, but "All" does never add PTX, therefore not having future-compatibility

This will likely also fix #87.

done

Now CMake returns "-gencode;arch=compute_20,code=sm_20;-gencode;arch=compute_20,code=sm_21;-gencode;arch=compute_30,code=sm_30;-gencode;arch=compute_35,code=sm_35;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_52,code=sm_52;-gencode;arch=compute_32,code=sm_32;-gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_53,code=sm_53;-gencode;arch=compute_52,code=compute_52" for CUDA 7.5 and "All".
Why is there no arch=compute_52,code=sm_52?

another CMake bug (unfortunately)

That wasn't the problem. The problem was the wrong "code=compute_52" instead of "code=compute_53". With 27f0d3123d84197160a330464c8821f6a611666d it's fixed. I'll add some comments explaining that "All" does not contain the latest PTX although in my opinion "All" should actually contain it or there should be an option like "All+PTX".

'All' now includes the latest PTX


home-assistant: pin plugwise at 0.37.3

Description of changes
#312235 was merged too soon.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 312428 run on x86_64-linux 1

1 package built:

home-assistant-component-tests.plugwise


Offer a more conventional default package name in build-init questionnaire

When the user runs the init task, the default value for the package name is the name of the project.
The project name, however, has no formatting enforced and can start with an upper-case character.
Upper-case package names are unconventional in Java, and even cause problems with test filtering,
as reported at https://github.com/gradle/gradle/issues/20350.
This change improves the situation by offering the project name lower-cased as the default value for
the package name in the init task command input questionnaire.
Fixes #20350

@bot-gradle test RFM

OK, I've already triggered the following builds for you:

PullRequestFeedback build

Thanks for the review @tresat!

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


Random "Timeout" or "Hang" issues when loading new subject image (aka the Empty Subject Sets issue)

Problem:

Occasionally, attempting to annotate any ship in Old Weather will cause a "timeout"/"hang" error - the loading sign keeps spinning with no images loaded.
Occurs randomly, on multiple ships (e.g. Eastwind 1946, Northland 1930), on all browsers, regardless of signed in or not.

Analysis:

Calls to /api/subjects will randomly return empty subject sets.
Empty subject sets results in the app "hanging", not knowing what to do.
It is unknown WHY /api/subjects is returning empty subject sets, however.
It is noted that if a user is logged in, the API remembers the "last served" subject sets (even empty ones), which will explain why logged in users can be "permanently locked out" of certain ships.

Notes:

Tested on Surface Pro 3 Win 10 - Chrome 46 (primary), Edge 20, Opera 33, Firefox 42
Tested with non-logged in user account to confirm random nature of empty subject sets.
Tested with logged in user account to confirm logged-in users getting "locked out" of ships they were annotating.
Sample record of notable attempts, with request headers & responses, attached.

zooniverse-oldweather-20151120-bug-report.txt

pretty sure Roger debugged this that the default non-logged in queues weren't built due to the way we imported the data...

I did, and I edited the subject sets (tweaked set names and reverted), but it looks like it's still an issue

did the names actually save?

yeah

then it should have redone the q's, https://github.com/zooniverse/Panoptes/blob/69a96b3d4144adf9d162caab07169740b35a5e70/app/controllers/api/v1/subject_sets_controller.rb#L15

Is your workflow grouped? If not then it won't....

Not as far as I know, it's just grabbing subjects from a set

your workflow has to be grouped if you're using subject set groups. same as annotate..without this the grouping configs don't work.

They're not groups of sets, just sets

ok - each subject set is considered a group for selection. annotate works this way, check the grouped setting on the workflow. if you want grouped (subject_set) selection to work correctly (including building the non-logged in queues) this has to be set.

Okay - I've set this, and and edited the sets again, but I'm still getting empty sets. Would you mind having a look @camallen?

Please let me know if I can help look into the sets or etc - I'm currently running some tests to see the % of getting an empty set in each of the ships, just in case that data might help the debug.

Update: whatever's been happening in the past minute, the subject sets have gone from empty 80% of the time to the full 20 randomly-picked subjects in a set, all the time.
Tested for Eastwind 1946. (https://panoptes.zooniverse.org/api/subjects?sort=queued&workflow_id=886&page=1&page_size=20&subject_set_id=2529) Nice!

not sure what's up here..probably won't have time to look today..when is it needed?

please close if this isn't an issue anymore

Okay, I've checked all of the sets - they seem to be okay, apart from Bear 1899 (subject set id 2493). Doesn't need to be today, but can you take a look on Monday?

I take it back, I think it's okay :D Thanks all


Fix the suffix of dependency urls in Package.swift

Bug/issue #, if applicable:
Summary
I unified the URLs of external packages defined in Package.swift's dependencies to have suffix .git.
In the Swift Package Manager ecosystem, it is often better to have .git at the end of the repository URL.
Dependencies
n/a
Testing
n/a
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[n/a] Added tests
[x] Ran the ./bin/test script and it succeeded
[n/a] Updated documentation if necessary

These two reasons from the linked PR make this worthwhile in my opinion, even if it sounds like SwiftPM successfully resolves the dependencies regardless.

All this does is add the .git suffix to the package URL for swift-numerics. Why do this?

Consistency. A majority of the other Apple-published Swift ecosystem packages (swift-logging, swift-crypto, swift-certificates, swift-asn1, swift-collections, swift-docc, swift-nio, swift-argument-parser, swift-atomics, and so on) use the .git-suffixed form, if not exclusively, then more often than not.
It makes a difference. If, for example, the same dependency is specified more than once in a given dependency graph and the use of the suffix is inconsistent, SwiftPM does take notice (although the message is only shown when -v is in use):

info: 'swift-algorithms': dependency on 'swift-numerics' is represented by similar locations
('https://github.com/apple/swift-numerics.git' and 'https://github.com/apple/swift-numerics')
which are treated as the same canonical location 'github.com/apple/swift-numerics'.

@swift-ci please test


Broken icons when open project from terminal

VSCode Version: 1.12.1
OS Version: macOS Sierra

Steps to Reproduce:

Open terminal an run code path_to_folder

The icons look awful (after update to 1.12.1)

I'm seeing the same thing. Opening from terminal with code /path/to/project draws all icons in the UI as strange artifacts. Opening the same project by first opening vscode, then using file->open recent draws the UI as expected.
macOS sierra 10.12.4
vscode: 1.12.1

Same here.
OSX El Capitan 10.11.6
VSCode 1.12.1

This is https://github.com/Microsoft/vscode/issues/24820 but I wonder why my workaround is not working as expected in main.js
@redVi @jenswegar @hannupekka can you send me the contents of your shell environment? You can quickly get there by running process.env from the console when developer tools are open (from the help menu):

@bpasero yes, this is the same reason. I removed locale settings export LC_CTYPE and export LC_ALL and now the artifacts are gone.

@redVi and so what are the values you have for LC_CTYPE and LC_ALL?

@bpasero
export LC_CTYPE="ru_RU.UTF-8"
export LC_ALL="ru_RU.UTF-8"

@bpasero from my process.env output:
LC_ALL: "fi_FI.UTF-8"
LC_COLLATE: "fi_FI.UTF-8"
LC_CTYPE: "fi_FI.UTF-8"
LC_MESSAGES: "fi_FI.UTF-8"
LC_MONETARY: "fi_FI.UTF-8"
LC_NUMERIC: "C"
LC_TIME: "fi_FI.UTF-8"

Closing as duplicate of https://github.com/Microsoft/vscode/issues/24820. It looks like having an environment variable LC_ALL with certain languages is enough to trigger this issue. In #24820 I was pushing a workaround when LC_NUMERIC  is defined but this does not seem to be enough.
There are three options for people hitting this:

start VS code from the dock instead of terminal
make sure LC_ALL is not set when running from the command line
use our insiders build where I just pushed a fix (http://code.visualstudio.com/Download#insiders)


Virtual system instruction set specification

Specify the virtual system instruction set.

An instruction set has been specified.


feat: ability to set svc annotations

Adds the ability to set additional annotations on the Service resource.

Thanks @shanehull  I'll try and give this a look so we can get it merged in. I may have to add CI tests into the Actions flow before we do merge this PR though just so we have as much assurance as possible.

@ChrisJBurns let me know if there's anything we can do to get this merged. Currently we're blocked on using the chart, as we need annotations on the Service resource to be able to expose it.

@shanehull Yep apologies, just had a chance to go through my unreads. Have approved and run pipeline, and seems to be ok, however are you able to sign the commit?

Done @ChrisJBurns.

@shanehull I don't think the commit itself has been signed, you've added a signature to the commit message but the commit itself isn't signed with a key for more info: https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

Sorry @ChrisJBurns, should be verified now.


Add more taps from the hub as discoverable extractors in Meltano

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/57
Originally created by @tayloramurphy on 2021-05-24 22:22:10

We should have a target to add one or so a week.

View 4 previous comments from the original issue on GitLab


Powershell path parsing problem (Windows)

When the -s flag is used to pass in a shell to use, oneshot parses the path as having double slashes. This then leads to a 'file not found' error.
example: C:\Windows\System32\WindowsPowerShell\v1\powershell.exe -> C:\Windows\System32\WindowsPowerShell\v1\powershell.exe

This was somewhat resolved by making PowerShell the default shell on Windows.


style right and left handles of region plugin independently

Short description of changes:
In 804334d63e36f88b5340aa8faac2d0db5adf5acf, the css style of the right region resize handle was changed to right: 0, but the element in question already had a left: 0 style.  This means that the drag handles for the region were basically in the same place.

Coverage remained the same at 80.984% when pulling e493824129e28c0de69aef1b0f0f1035b7340ad1 on osheroff:master into 8f984b5d0b7691a9b231daf2445ad8b85bb45e59 on katspaugh:master.

thanks!


Change User-Agent strings

We use the string Jetstack cert-manager in several places. With the CNCF donation we should change these to be cert-manager.
This should also come with a clear warning in the release notes and a check on what it might break.
Places where this is used:
https://github.com/jetstack/cert-manager/blob/4f894a7d994e79e2664496f5c9aa82234e4f7aa8/pkg/util/useragent.go#L20
https://github.com/jetstack/cert-manager/blob/c98415062ae3273af440ecee4b5fd5e1df4c0e79/pkg/issuer/venafi/client/request.go#L168

xref: https://github.com/jetstack/cert-manager/issues/3451 https://github.com/cncf/toc/issues/557

I know Let's Encrypt have used our user agent in the past to identify traffic coming from cert-manager cc @jsha
There could well be non-public uses of this user agent too.. i.e. for metrics etc

Changing user-agent strings is fine by us at Let's Encrypt. Our queries are pretty ad-hoc (log analysis, not live metrics), and we can easily adapt them.


New Version of Climsight with Modular Structure and Terminal Option

This pull request introduces a new version of Climsight with the following updates:
Modular Structure:

climsight.py: Reads the configuration and determines whether to run the Streamlit app or the terminal interface.
streamlit_interface.py: Handles the Streamlit interface.
terminal_interface.py: Handles the terminal interface.
climsight_engine.py: Contains the core functions for preparing and requesting data, and sending requests to the LLM. Both functions return and yield results to the respective interfaces, which then handle the subsequent processes.

Terminal Option:
In addition to running Climsight with Streamlit, you can now run it directly in the terminal. To run Climsight in the terminal, use the following command:
  python src/climsight/climsight.py terminal

Logging:
This version includes the initial implementation of logging to help with debugging and tracking the application's behavior.
Running Climsight with Streamlit remains unchanged:
streamlit run src/climsight/climsight.py

Sorry @kuivi had to change the colors a bit :)


AVS 2021-01-01-preview

Replacing #12762 to make it easier to review. We are wanting to go live by Monday, 2/15.
This is a new version 2021-01-01-preview for AVS (Azure VMware Solution).  It adds these operations:

Datastores_Create
Datastores_Delete
Datastores_Get
Datastores_List

Pleases see this private pull request for previous reviews.
There are no breaking changes in this API version. The CI labels are from breaking changes that were already approved for both Go & Python in https://github.com/Azure/azure-rest-api-specs/pull/12290. These Go changes are from #12290.
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? 2/15/2021
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Thanks for the review yesterday @varunkumta. I made the requested ARM changes to the examples.

/azp run

Thank for the ARM approval @varunkumta. Do you or @zhenglaizhang know who this should be assigned to in order to get merged?

Hi @ctaggart I just assigned the PR to myself. And you need get review approval from the requested reviewers,

@zhenglaizhang, who do I need to get approval from? It has been approved by ARM.


Move the help icon to the individual Cards.

Danny Edwards had a great point about the help icon. It doesn't make sense on the ActionBar. It should be one the Card. It would make the code waaay cleaner too.

Dupe of #114.


Add annotation access control at the folder level

Adds the ability to change the access list for all annotations in a folder (and optionally all annotations in the subfolders of the folder).  The control can be found as an entry in the 'Folder actions' dropdown menu.
Closes #27

In retrospect, this really should be in the large_image repo in the girder_annotation module.

In retrospect, this really should be in the large_image repo in the girder_annotation module.

Would you like me to move it there?

Yes, please.

With #905 in large image, change is no longer needed here.


notFoundMode: 'root' ignored when throw notFound

Which project does this relate to?
Router
Describe the bug
In doc I see (https://tanstack.com/router/latest/docs/framework/react/guide/not-found-errors#notfoundmode-root) the example:
If provided the path of `/posts/1/edit`, the following component structure will be rendered:

- <Root>
- - <Root.notFoundComponent>

The notFoundComponent of the __root__ route will be rendered because the `notFoundMode` is set to `root`.

But I see another behaviour if I generate notFound error myself.
Your Example Website or App
https://stackblitz.com/edit/github-ovpjyk-3gh8gb?file=src%2Fmain.tsx
Steps to Reproduce the Bug or Issue
Try to open "Not found" or "Sub Not found" - they will be work as expected.
Try to open "Generate Not found" page - it works in another way
Expected behavior
I expect that it will be works the same as described in documentation.
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Ok, I found in examples that notFound has not described option - global - https://tanstack.com/router/latest/docs/framework/react/api/router/notFoundFunction#examples
Throwing not found error like throw notFound({ global: true }) fixing my issue.
Please describe it better in documentation.


fix(sequencer): provide context in checktx response log

Summary
We were using e.to_string to provide error messages in CheckTx responses. This results in dropping the context that the error objects carry around, only providing the top level error. This uses anyhow's format!("{e:#}") to format the error with its context.
Changes

changed all instances of e.to_string() to `format("{e:#}") in the sequencer's mempool service

Related Issues
Link any issues that are related, prefer full github links.
closes #1464

Can you check if this still works with the recent move from anyhow to eyre?

Can you check if this still works with the recent move from anyhow to eyre?
Checked and provided the code snippet + output.


Regenerate controller with code-generator 3a50ed4

Issue #, if available:
Description of changes:

Regenerate controller with code-generator 3a50ed4

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Introduce an abstract filesystem for all output

https://github.com/spf13/afero seems a likely contender.

#289


apache-airflow resolution

When running
cargo r apache-airflow

we get the following error message.
thread 'main' panicked at 'internal error: entered unreachable code: when we get here it means that all candidates have been assigned false. This should not be able to happen at this point because during propagation the solvable should have been assigned false as well.', /Users/tdejager/.cargo/registry/src/index.crates.io-6f17d22bba15001f/resolvo-0.2.0/src/solver/mod.rs:584:53
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Some impossible statement is being hit, is being hit inside of resolvo.

@baszalmstra can you update with any progress made in resolvo :)?

I spent half a day debugging the issue. It looks like at some point something is not properly propagated but I cant figure out what causes it. I'll have to get back to it later.

I think we should:

write something in resolvo so that we can store all the rules, and recreate the state
then debug this

I also looked into it and could not yet figure out the root cause.

Not sure if this helps,
cargo run openmetadata-managed-apis
which has dependency of apache-airflow>=2.2 works and installs apache-airflow version 2.6.3 which is valid since all the newer versions need opentelemetry-api and opentelemetry-exporter-otlp which cannot be installed as one of their dependency (opentelemetry-semantic-conventions) has only pre-release versions available

From the logs on the cargo run apache-airflow, I see rip

Trying to install the latest 2.8.0 version of airflow
Couldn't because of opentelemetry-api and opentelemetry-exporter-otlp deps
Backtracks to prev version 2.7.3 which exhausts all the stable variants of both opentelemetry deps and thereby panicking.

This should ideally backtracks to next version of 2.7.2 and so on.
Wish I could dig more on this, still at early days of Rust and understanding SAT solvers.

I added an integration test on resolvo which replicates this,
https://github.com/sumanth-manchala/resolvo/blob/airflow-int-test/tests/solver.rs#L462

Hi @sumanth-manchala this is amazing. I also tried to make a reproducer the other day, but didn't manage so I am really happy to see that you managed to do it!
Did you – perhaps – also work on a fix? This issue is certainly one that we would like to fix sooner rather than later :)

Thanks @wolfv. Have added the integration test to expect panic, will ping here if I could figure out the fix

Yeah that's absolutely great work, @sumanth-manchala :) Jumping head first into such a gnarly problem couldn't have been easy! Thanks a lot 😄

Just in case anyone out there is trying to debug this further, I'm pretty sure I've found the root cause of the issue and hope to have a fix soon.


Standardize the data

Files within 'dataset' file are organized differently (columns number, the order of columns). Will it better to standardize the format? I mean all the output file have the same columns (maybe just some common columns which all files should have) and in the same order. That will be easier for the user to use and follow.

Thanks @jinshijian . The number of columns is going to differ between datasets; there's no way around that, as they measure different ancillary data (T5 versus T10, etc) that we don't want to just toss out. In addition, it's easy in R to combine data frames with different columns (dplyr::bind_rows(), plyr::rbind.fill(), data.table::rbindlist(), etc).
But having a consistent ordering to the most common columns seems like a good idea. For example, CSR_TIMESTAMP, CSR_FLUX, and CSR_PORT are all guaranteed to be present, and could always appear in that order; anything after that could be alphabetical. Would that partly address this?

Another possibility would be separate the current data table into two:

data_core (guaranteed to have constant fields) and
data_other (variable fields like CSR_T5, CSR_SM10, etc).

Thoughts @jinshijian ?


BluetoothGattDescriptor.hpp added functional header (fix for issue 100)

Signed-off-by: Vadim Panov<EMAIL_ADDRESS>

Thank you for your contribution. I forgot about this issue.


How to run properly the script?

Hi,
I've installed Python for windows and all the required modules but can't run the script without getting error after the initial check timeout request.
Error:
Traceback (most recent call last):
File "C:\Unieuro\main.py", line 10, in
f = open("credenziali.txt", "r")
FileNotFoundError: [Errno 2] No such file or directory: 'credenziali.txt'
Thanks

Hi,
make sure you have put the file credenziali.txt in the same folder of Main.py  and changed it appropriately with your data

Yep, the file "credenziali" and the other txt are into the same folder and properly configured. What command I have to use to run the script? Simply python.exe main.py?

On windows i think you have to put the absolute path of python.exe and then Main.py, so it would be something like this
C:\Users\user\Programs\Python\Python38\python.exe Main.py and you have to run this command from the folder where there is Main.py

Thanks so much I got the problem, was some spaces into the txt file, and now is running, there is any method to let the script continuosly check a not available product until it get on and then add it to the cart?

Good job! My script when the product it's not available runs anyway
Try it putting an higher price of the real and let's see ;)

If an higher price is set the script tries to add to cart and returns an error stopping. Is there a method to put it in loops without making it stop?

Sorry i mean lower, the price you insert in file unieuro.txt is the budget you have, you have to insert a lower value there
You can also try with a product that is not available

ok, tryed now.
If a lower value is set the script goes in loop with "non è sceso di prezzo" but if the product is available it does not add it to the cart and repeat "non è sceso di prezzo" always.

Yes it goes in the loop because it searches endlessly if it drops in price, if the product is not available or is priced higher than your budget then the print is the same, in the future I will change it with a distinction between them

It would be very useful to check an unavailable product while waiting for it to become purchasable again. Like the damn RTX series I'm trying to buy :(

you can still put a very high price in the file and then let it run, when it becomes available it adds it directly to the cart

Unfortunately unieuro always shows the price of the product (even if not available) so the script always tries to add it then fails and stops.

I fixed it, copy the new file Check.py and let's see.
Hope you enjoy ;)

Awesome! great job indeed mate, works like a charm. I was able to find a "work around" for the previous problem and run it anyway, but this works much better now ;) So much clean

Thanks I appreciate it!


import Multi-line curl error

Expected Behavior
import edge or postman's curl
Actual Behavior
Error while scanning for resources to improt: cannot read properties of undefined (reading 'trim') ??
Reproduction Steps
No response
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
latest
What operating system are you using?
macOS
Operating System Version
macos 13.1
Installation method
insomnia
Last Known Working Insomnia version
No response

Thanks for the report, could you provide the import file? or more information about how you created it.

sure, such as
curl --location --request POST 'http://localhost:8900/fbi/contract/list' \
--header 'x-rpc-fake_user: miyin' \
--header 'x-rpc-god;' \
--header 'User-Agent: Apifox/1.0.0 (https://www.apifox.cn)' \
--header 'Content-Type: application/json' \
--header 'Accept: */*' \
--header 'Host: localhost:8900' \
--header 'Connection: keep-alive' \
--data-raw '{
    "page": 1,
    "page_size": 100,
    "name": "",
    "company_code_list": [],
    "supplier_code_list": [],
    "currency_list": [],
    "usable": false
}'

I'm not sure if it was my mishandling

ah you can paste that directly into the url bar of an empty request and it will be parsed. You are correct though, it does list cURL as one of the import types. Thats our bad. Thanks again.

ah you can paste that directly into the url bar of an empty request and it will be parsed. You are correct though, it does list cURL as one of the import types. Thats our bad. Thanks again.

You mean this...? Seems to be unsupportive

ah its because of this line,
--header 'x-rpc-god;' \
it works if you remove it.
although the code should support valueless headers, so this is a bug

ah its because of this line, --header 'x-rpc-god;' \ it works if you remove it. although the code should support valueless headers, so this is a bug

ah. Thank you very much. I got it!

Fixed in #6069


Features returned from the Captum package does not match the Captum output

Describe the bug
The attribution returned by Saliency_PTY did not make any sense to me. So, I checked with the original Captum package and I found out that the Captum output is different than the attribution returned by TSInterpret.
Here is the output of TSinterpret

Here is the output of Captum.

The dataset is synthetic. Just a portion of Sine wave (distinguishable feature) added to a random noise. So, it is clear that the attribution of the TSInterpret is incorrect. In this example I used IG. But, this is the case for others that I tried as well.
I hope it helps you fix the issue.

Hi @shrezaei,
would you might sharing your parametrization for both captum and TSInterpret?
A reason can be the different baslining behavior implemented as default.
captum uses for IG zeros as baseline default. We use random noise, as zeros are often contain information in time series and therefore do not provide masking. (More details here: https://github.com/fzi-forschungszentrum-informatik/TSInterpret/issues/33)
If you want to have a similar behavior of TSInterpet to captum you can provide the baseline in the **kwargs of the explain call:
int_mod.explain(item,labels=label,baseline_Single= np.zeros(item.shape), assignment=0)
I Hope this helps to solve the issue.

Thanks for your quick response.
For other to use: Note that "baseline_Single" should be lower case, other wise it doesn't take the baseline into account. Use "baseline_single=".
Unfortunately, it didn't solve the issue. I still get a meaningless attribution which is very different from Captum.
Here is how I call the function.
exp = xia_method.explain(x_test[i,:,:], labels=label, TSR=True, attribution=0.0, baseline_single=np.zeros(x_test.shape[1:]), assignment=0)

By the way, because the dataset is synthetic, I know the ground truth. Dataset is just a random noise plus a portion of sine wave.

Hi @shrezaei,
my guess is that the problem is that we return the normalized attributions to range of [0,1] (leaned onto [1]).
In such a case the heatmap can lead to misleading visualizations.
I opened up a parameter that takes away the normalization here:
pip install https://github.com/fzi-forschungszentrum-informatik/TSInterpret/archive/refs/heads/LEFTIST.zip
Simply set normalization = False when instantiating the method.
int_mod=TSR(model, train_x.shape[-2],train_x.shape[-1], method='IG', mode='time', normalize=False)
Hope the results are now identical.
[1] Ismail, Aya Abdelsalam, et al. "Benchmarking deep learning interpretability in time series predictions." Advances in neural information processing systems 33 (2020): 6441-6452.

That is a good feature to have. I appreciate it.
But, I don't think that is the issue. In the Captum output, the attribution is centered around the sine wave, although values are relatively small. So, if one normalizes it, it should be still centered around the sine wave. But, the output of TSInterpret capturing majority of the signal, but not the sine wave at all. It doesn't matter if you normalize it or not. If it's zero (assuming all attribution values are non-negative), it will remain zero, unless there is a bug in the normalization function.

Hi @shrezaei,
I just realized that TSR = True (So temporal Saliency Rescaling is turned on).
If you want to obtain the same results as with captum: TSR = False, normalization=False and the baseline setting of above.
For TSR = True, I could find a problem for univariate data:
For univariate data the calculation of the time series feature importance leads to many time step being attributed with 0.1 as this is the fallback if the time contribution is not larger than the mean of all contributions. If the mean of all contribution is smaller that the time contribution at time step t, the feature contribution x time contribution leads to squaring (in case of attribution <1, the attribution gets smaller and smaller). In such a inconvenient setting, the default setting can lead to a higher attribution than the original attribution. If we only have one time series feature, it is supposed to be one (only feature = always most important feature).
It is fixed here :
pip install https://github.com/fzi-forschungszentrum-informatik/TSInterpret/archive/refs/heads/LEFTIST.zip
Thanks for pinpointing the issue.

Closed due to inactivity.


built face down over bedrock

crash-2024-02-03_09.15.52-server.txt
Building the gate face down in Y2(3) cause a crash this crash occurs again on reload

Should be fixed


redis.exceptions.DataError: Invalid input of type: 'NoneType'. Convert to a byte, string or number first

i can't connect to bancho in osu and i get this error in server console

change https://github.com/osuAkatsuki/bancho.py/blob/25135d859cd7ae75549a90834f1258fe05814dc6/app/settings.py#L33 to REDIS_DSN = "redis://localhost"

i should just fix this properly already


Issues-1456: Exclude hamcrest from parent

Related to: https://github.com/strongbox/strongbox/pull/1485

Thanks! Please, let me know when you think it should be merged.

@carlspring I updated PR comments on strongbox main project, so I think @fuss86 got final decision here :)


Timeouts when table gets too big

Hi
I was having a problem with this plugin, the following query was taking too long and causing a timeout:
select max(entityaudi0_.commit_version) as col_0_0_ from jhi_entity_audit_event entityaudi0_ where entityaudi0_.entity_type='xxxxxxxxxx' and entityaudi0_.entity_id=128200;

Doing an explain the result is:

id
select_type
table
type
possible_keys
key
key_len
ref
rows  Extra

1
SIMPLE
entityaudi0_
ALL
NULL
NULL
NULL
NULL
504808

So what I did to fix was adding an index on the where fields (sorry, no command as I used a visual tool and too lazy).
That changed my explain to:

id
select_type
table
type
possible_keys
key
key_len
ref
rows  Extra

1
SIMPLE
entityaudi0_
ref
entity_id_type
entity_id_type
265
const,const
1

And that speed up the query almost 99% (was taking 40s now taking less than 1 second).
I think it would be nice if you'd automatically add it to the liquibase table creation.

It's worth noting that the table has over 500k rows and almost 1GB

would you consider doing a PR for this?

will be released after next jhipster version


Issue 102 Refactor purchase endpoint

I was able to refactor the purchase endpoint to match all the api calls to the shopcart endpoint.

IBM Cloud toolchain: Delivery Pipeline deployed products to dev, including commit a6916a8ffdceb9a89890826f17e1883edd754a27

IBM Cloud toolchain: Delivery Pipeline deployed products to prod, including commit a6916a8ffdceb9a89890826f17e1883edd754a27


[193] SPIKE: Validations for trainee sections

Context

https://trello.com/c/vtzaPbHw/193-spike-validations-for-trainee-sections

Changes proposed in this pull request

Refactor the personal detail routes
Handle personal detail editing in its own controller (treat it as a separate resource)
Introduce an ActiveModel object to validate personal detail fields in isolation to other sections

Guidance to review

Visit the review app https://dfe-twd-rttd-pr-64.herokuapp.com/
Skip filling out trainee details initially
Navigate to the personal details section and try to continue without filling out any details
Assert validations are triggered

Depends on https://github.com/DFE-Digital/register-trainee-teacher-data/pull/60

 Spike: Validations for trainee sections

We don't seem to have an example of customising the default rails error message eg. "can't be blank" messages

I think this is a good approach. Having the validation tied to the form in this layer will mean we can't check that the trainee record is valid outside of the context of the form submission though. That's not necessarily a problem at the moment but might paint us into a corner in the future. For example if the validation rules change we won't easily be able to check that an existing record is valid with the new rules.

Yeah my thoughts exactly. I think we should separate the two. But we may need another directory to store the validation rules/classes. If others are happy with that, then we could go down that road 👍

@aliuk2012 I've provided examples on customising the default activemodel error messages.
Also, I've tweaked this slightly so we just leverage PORO (plain old ruby objects) with activemodel to hold the validation rules for different sections. These objects can be re-used by other classes/services which will give us more flexibility down the road.
Can you re-review @gpeng, @misaka, @aliuk2012, @JonF1 🙏

I think the error summary position has moved? could we update the screenshot?

I think the error summary position has moved? could we update the screenshot?

Updated the screenshot


Tests for $.unbind()

See here: #156
I added docs too. Anyone willing to write tests?

I'll take a shot at this.
I can register various event handlers (sinnon spies), then remove then using unbind, trigger an event and verify that the spies did not get called.

Thanks!!


Database Migration : on wallet restore ERC20 tokens show balances from previous wallet

It looks like UNLINK FROM THIS DEVICE doesn't work correctly for ERC20 tokens
To reproduce:

download a new build and restore some wallet that has some ERC20 tokens with non-zero balance
unlink the wallet from device
restore some other wallet.
you will notice that balance s for erc20 tokens would be shown from previous wallet.

Moved this ticket to Ethereum Kit Project


KOGITO-9410 - Add exposed Data Index container port

(tests) - kogito-runtimes job #2765 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
export BUILD_MVN_OPTS_CURRENT=-T 1C -Dvalidate-formatting -Prun-code-coverage
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-runtimes -u https://github.com/kiegroup/kogito-runtimes/pull/3069 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.kogito-runtimes/2765/display/redirect
Test results:

PASSED: 3313
FAILED: 1

Those are the test failures:

org.kie.kogito.persistence.kafka.KafkaProcessInstancesIT.testFindByIdReadMode
expected: 5 but was: 0

Jenkins rerun kogito-runtimes tests

(tests) - kogito-runtimes job #2768 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
export BUILD_MVN_OPTS_CURRENT=-T 1C -Dvalidate-formatting -Prun-code-coverage
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-runtimes -u https://github.com/kiegroup/kogito-runtimes/pull/3069 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.kogito-runtimes/2768/display/redirect
Test results:

PASSED: 3348
FAILED: 1

Those are the test failures:

org.kie.kogito.persistence.kafka.KafkaProcessInstancesIT.testFindByIdReadMode
expected: 5 but was: 0

Jenkins rerun kogito-runtimes tests


better error message for missing snaps

Related Issue/Discussion: https://github.com/aaronabramov/k9/issues/84

are there no tests for missing snapshot? can you add some?
also probably make that string green. can you post some screenshots?but please don't do screnshots of snapshots that also have colors in them :)
also you need to link your email to your github account :)

Will do these things today. 👍

fixed in https://github.com/aaronabramov/k9/pull/117 for snapshot!() macro


CSS Framework for web dashboard

[ ] Switch to bootstrap/foundation/etc. (Preferably bootstrap)
[ ] Add feedback messages after config edit (i.e: "Config updated!")

I finished that. Could you detail the use of layout_options ? I can't seem to find any explanation about what it is used for. I first guessed that it holds the path to layouts but it apparently doesn't work that way.


Signup part 1: engine changes. added waiting change action

[x] Changed several engine methods from taking a bunch of params to an object so its more clear what's what and avoids passing a bunch of null,null,null type stuff
[x] Promise rpcs and engine sagas take a waiting key instead of a waiting callback or waiting action creator
[x] Added a changeWaiting action so you can just pass a boolean if you want an increment or not

This is part of a 4 part pr. It won't work until the 4th part:

https://github.com/keybase/client/pull/12524
https://github.com/keybase/client/pull/12525
https://github.com/keybase/client/pull/12526
https://github.com/keybase/client/pull/12527

@keybase/react-hackers this is ready

[ ] merge other 3 first

@keybase/react-hackers bump

@keybase/react-hackers this is the merged down parts 1-5 of the signup pr. still looking for a second thumb (first was @akalin s)


Fix query limits on Astarte datastream interfaces

Astarte Client expects a query key to pass query parameters, that was missing, so the limit was not actually passed. This went unnoticed because most of the interfaces involved here were not actually used on physical devices, and tests mock the Astarte response so always returned a single value.

Pull Request Test Coverage Report for Build 08b9d66956522350decd19a5267c876b005c6bb7-PR-479
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+85.0%) to 85.03%

Totals

Change from base Build<PHONE_NUMBER>:
85.0%

Covered Lines:
1721

Relevant Lines:
2024

💛  - Coveralls


feat: remove requirement for halt

Removes the requirement for a state read program to include halt at the end. This is possible because the program can only move forward or loop a fixed amount of times.

Just noting that I think this means a JumpForwardIf that jumps beyond the last program counter can now silently end the program too. We may want to update its entry in the spec to remove this panic condition. I don't think there's a way to do this check local to the JumpForwardIf operation as we don't know the total number of operations (not without adding a len fn or similar to OpAccess, which would make the lazy implementations less useful).

Yeh good point. Kinda sucks we can't check it but now that I think about it this is already the case for the constraint VM so maybe I should just remove that panic and add to the docs that jumping beyond the program will end the program without panicking.
We could also check the op_access to return an error if you call it with ops.len() + 1. But that's not perfect because a jump could just jump to ops.len() and it would still silently end. 🤔


[stable/airflow] Support for custom operators

Airflow provides a way to add custom operators which can be added to specific folder location. Currently there is a way to mount a volume into airflow where DAGs can be added which are picked up by airflow. Similar behavior is not supported for custom operators or hooks
Is your feature request related to a problem? Please describe.
There is no support to add custom operators which are picked up by airflow
Describe the solution you'd like
Provide a way to mount a volume where custom operators can be added, and airflow can be used in DAGs.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@kishore25kumar If this is still an issue, please raise it on the new repo: https://github.com/airflow-helm/charts/tree/main/charts/airflow
The helm/charts repo is being archived soon.


UI to setup FIDO keys is not available until after setting up TOTP

Gitea Version
1.15
Git Version
No response
Operating System
No response
How are you running Gitea?
https://codeberg.org
Database
No response
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Description
Configuring an account to use FIDO keys to log in first requires configuring TOTP. This is problematic because TOTP is harder to user and vulnerable to phishing.
Screenshots

Ref. my original comment over at Mastodon (same issue):
https://github.com/mastodon/mastodon/issues/16693

Just for the clarity: WebAuthn, also referred to as "passwordLESS", is actually replacing the 1st factor - namely the password - not the second factor. It is not a 2FA alternative.
Policywise, any site should have the option to choose whether or not to use 2FA, but still be able to choose to offer WebAuthn instead of passwords, and some sites might even want to say to all users that do NOT want to use 2FA, that then they must use WebAuthn instead of "just password", because passwords alone is worse...
(allthough that is not very good, as someone who gets hold of the usb key can then get in without any other information, so it is always recommended to "enforce" 2FA anyway, IMO)
regardless, this issue should move ahead anyway.

Plus my second comment there:
https://github.com/mastodon/mastodon/issues/16693#issuecomment-955594924

https://github.com/go-gitea/gitea/pull/11573

The accepted pull request does not really resolve the issue fully.
WebAuthn / FIDO2 is both factors in one, it is not a second factor only. The user has 'something they own' (the hardware key) and 'something they know' (the PIN for the device) and possibly also 'something they are' (biometric data).
The WebAuthn standard is meant to allow users to log in passwordless. E.g. with a Yubikey Bio, no password is needed, and no PIN is needed unless you mess up the fingerprint scan a couple of times. This is still two factors of authentication.
Also note this warning regarding the current protocol Gitea is using:


feat(specs): convert nested specs to dictionnary of dm_env specs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.7 out of 8 committers have signed the CLA.:white_check_mark: dluo96:white_check_mark: PDuckworth:white_check_mark: OmaymaMahjoub:white_check_mark: coyettev:white_check_mark: TristanKalloniatis:white_check_mark: clement-bonnet:white_check_mark: cemlyn007:x: sash-aYou have signed the CLA already but the status is still pending? Let us recheck it.


add pybullet env

add pybullet env. pybullet-gym
add td3/ddpg/ppo/sac configs for pybullet.

Codecov Report

Merging #16 (7d9aec9) into main (097ee4d) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   89.92%   89.88%   -0.04%
==========================================
  Files         328      328
  Lines       23632    23597      -35
==========================================
- Hits        21250    21211      -39
- Misses       2382     2386       +4

Flag
Coverage Δ

unittests
89.88% <ø> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ding/interaction/base/network.py
96.29% <0.00%> (-1.27%)
:arrow_down:

ding/worker/coordinator/coordinator.py
85.65% <0.00%> (-0.99%)
:arrow_down:

ding/interaction/slave/slave.py
85.08% <0.00%> (-0.51%)
:arrow_down:

ding/interaction/master/master.py
90.16% <0.00%> (-0.45%)
:arrow_down:

ding/envs/env_manager/subprocess_env_manager.py
82.23% <0.00%> (-0.38%)
:arrow_down:

ding/worker/replay_buffer/advanced_buffer.py
92.89% <0.00%> (-0.33%)
:arrow_down:

ding/worker/collector/comm/flask_fs_collector.py
90.00% <0.00%> (-0.18%)
:arrow_down:

...s/env_manager/tests/test_subprocess_env_manager.py
71.97% <0.00%> (-0.18%)
:arrow_down:

...ctor/comm/tests/test_collector_with_coordinator.py
95.45% <0.00%> (-0.14%)
:arrow_down:

ding/worker/coordinator/tests/test_coordinator.py
93.22% <0.00%> (-0.12%)
:arrow_down:

... and 17 more

Continue to review full report at Codecov.

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


Incorrect documentation about using environment variables inside functions

What are the steps to reproduce?
Follow documentation on how to get an environment variable inside a function node: get(..).
What happens?
Error stating that get is not defined.
What do you expect to happen?
Fix documentation to properly explain how to use environment variables in function nodes: env.get(..).
Please tell us about your environment:

[ ] Node-RED version:
[ ] Node.js version:
[ ] npm version:
[ ] Platform/OS:
[ ] Browser:

Moving this over to the documentation repo.


Output Binding: Switch to MERGE + JSON query model

SqlDataAdapter requires a lot of client-side work. It has to pull the whole SQL table into memory in order to distinguish between INSERTs and UPDATEs. As a result, we've now switched to generating MERGE T-SQL queries. This allows us to offload the work to SQL, who can then optimize for upserts. Thanks @yorek for the suggestion & assistance!
Assumptions made:

Column Names in SQL are Case-Insensitive (cannot have 'Cost' and 'cost')
The set of C# class field/property names must be equivalent to the set of your SQL column names (case insensitive)
I cache the PKs and column names and types for 10 minutes. If a user somehow manages to change their primary keys or column types without needing to recompile (and thus restart) their Functions code+host, then they may experience issues for up to 10 minutes.

TODOs/Open Questions:

Is assumption 2 above an acceptable restriction? Do we want to allow extra SQL/C# class columns/properties?
If we cannot find the primary keys/column names, we fail the operation. Do we need a backoff period?

If we cannot find the primary keys/column names, we fail the operation. Do we need a backoff period?

Primary Key are definitely needed otherwise updating the correct row will be challenging. (Technical any UNIQUE index will be enough, bet let's start by assuming a PRIMARY KEY needs to be there. Let's see the feedback on this)
For what concern properties and columns: I will not raise any error if a property of the POCO object is not mappable to any columns: simply that property will not be persisted. This approach is ignoring additional properties is used also when deserializing JSON into an object and I think it is coherent to follow the same approach also for "serializing" and object into the database.


Checkmark in checkboxes behave inconsistently

Sometimes the checkmark is transparent (mask), sometimes it is white. This makes white checkboxes impossible.

Chrome:

FF:

Safari:

IE10:

IE11:

Safari Mobile:

They are meant to be transparent, with the white as a fallback. Is there a better, cross-browser solution to this?

I spent hours trying out different combinations. Couldn’t get it work consistently back then. I don’t think anything’s changed.

Wouldn't background-color: white; applied to .mdl-checkbox:checked fix the issue? That should provide a set background color for the transparency to hit when present.

Since this issue was open, I wasn't sure to create another one, but I've also experienced inconsistent behavior regarding the checkbox ripple. Sometimes, when I click on the checkbox, it would not always trigger the ripple.
I'm on the latest Chrome on the latest Mac OS X.

Not feasible to achieve consistency in v1. Checkboxes will be rewritten for v2 through #4471.


No backends are associated with TCP frontend when targets are added with host routing rules.

Server version - V0.32.0-rc2
Steps to reproduce the problem:
Create Lb rule with TCP listeners and add targets having hots routing rules.

No backends are associated with the TCP frontend in this case.

We are not able to access the LB rules on this tcp port.
Sangeethas-MacBook-Pro:testwget sangeethahariharan1$ wget http://<IP_ADDRESS>:9090/name.html
--2015-08-07 17:00:37--  http://<IP_ADDRESS>:9090/name.html
Connecting to <IP_ADDRESS>:9090... connected.
HTTP request sent, awaiting response... No data received.
Retrying.

--2015-08-07 17:00:38--  (try: 2)  http://<IP_ADDRESS>:9090/name.html
Connecting to <IP_ADDRESS>:9090... connected.
HTTP request sent, awaiting response... Read error (Connection reset by peer) in headers.
Retrying.

Expected Behavior:
Since we do not support for host routing rules for tcp protocol , when  host routing rules are specified , we should be able to do 1 of the 2 things:

Error out at the API level so that we do not accept host routing rules for tcp protocols.
Ignore the host routing set and create default backends with no acls , so the LB rules are accessible without honoring the hostname routing settings.

hostname routing rules will be ignored for tcp listeners now, and you should see them all merged to the same backend assigned to the source port

Tested with build from master:
hostname routing rules are ignored for tcp listeners  and there is 1 backend created with all the targets.


Feature Request: MultiSegmentPlayer Recipe

The new MultiSegmentPlayer seems cool; it would be nice with recipe on that one

Agreed!

Sorry I'm slacking! Will try to get around to this soon. I'll have to think of a simple "track with segments" view to represent this

I made very small demonstration "Section Player". It features 4 sections that can be played back in random order.
I did not use the MultiSegmentPlayer for this, because I couldn't grasp how it worked, but I'm sure you can use my player as an inspiration to show the segments.
You are welcome to use whatever from my source code, including the audio, which I did myself and therefore is free of any musical rights.
Compiled for macOS:
SectionPlayer.zip
Source, Xcode 13.2.1
SectionPlayer Source.zip

Closing this is with addition of #86


[tool_crash] FileSystemException: Cannot open file, OS Error: No such file or directory, errno = 2

Command
flutter --no-color pub get

Steps to Reproduce
when I try to run flutter pub get with the image_cropper package and some other packages I get the following error message:
Oops; flutter has exited unexpectedly: "FileSystemException: Cannot open file, path = '/Users/usr/flutter/packages/flutter_tools/templates/cocoapods/Podfile-ios-swift' (OS Error: No such file or directory, errno = 2)".
I only run into this issue on my Mac m1. I looked into the specific path and there is no folder cocoapods, why is this and how do I install this folder and the needed file?
Logs
FileSystemException: Cannot open file, OS Error: No such file or directory, errno = 2
#0      _File.throwIfError (dart:io/file_impl.dart:635:7)
#1      _File.openSync (dart:io/file_impl.dart:479:5)
#2      ForwardingFile.openSync (package:file/src/forwarding/forwarding_file.dart:71:16)
#3      ErrorHandlingFile.copySync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:315:22)
#4      _runSync (package:flutter_tools/src/base/error_handling_io.dart:597:14)
#5      ErrorHandlingFile.copySync (package:flutter_tools/src/base/error_handling_io.dart:314:5)
#6      CocoaPods.setupPodfile (package:flutter_tools/src/macos/cocoapods.dart:260:21)
<asynchronous suspension>
#7      injectPlugins (package:flutter_tools/src/flutter_plugins.dart:1089:9)
<asynchronous suspension>
#8      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:338:5)
<asynchronous suspension>
#9      PackagesGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:173:5)
<asynchronous suspension>
#10     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1125:27)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)

[✓] Flutter (Channel stable, 2.5.2, on macOS 11.6 20G165 darwin-arm, locale de-DE)
    • Flutter version 2.5.2 at /Users/timschauffert/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3595343e20 (3 months ago), 2021-09-30 12:58:18 -0700
    • Engine revision 6ac856380f
    • Dart version 2.14.3

[!] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/timschauffert/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.11.2

[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.60.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.27.0

[!] Connected device
    ! No devices available

! Doctor found issues in 3 categories.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 3595343e20a61ff16d14e8ecc25f364276bb1b8b
Plugins
image_cropper-1.4.1

Closing this as duplicate of https://github.com/flutter/flutter/issues/91559


wokwi-text missing as standard-part

In some projects i found the part "wokwi-text" for labeling parts. Why isn't such a basic thing in the parts-list? If there is a hotkey to input texts - i haven't found it / won't remerber it long ;-)
At thet time, i copy/paste this part from old projects an edit it in the .json...
best regards
Bernhard

Yeah, it's not yet completed. See wokwi/wokwi-features#43. Once it's completed, it'll also be documented.


Page reordering interface displays "Page of ." in the footer

Hi @gasman -  can you close the issue? The problem should be resolved with #1496.

@Vorlif Ah, yes - thanks for pointing that out!


Errors

Does not work on my machine.
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

C:\Users\robin\OneDrive\Bureaublad\kali.exe run "echo '' | sudo -S dpkg -l | grep ^ii | sed 's_  _\t_g' | cut -f 2 | grep openssh-server 1> /dev/null && echo '' | sudo -S cp /etc/ssh/sshd_config /etc/ssh/sshd_config.winsshterm.bak && echo '***' | sudo -S apt-get purge -y openssh-server 2>/dev/null"
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

C:\Users\robin\OneDrive\Bureaublad\kali.exe run "echo '' | sudo -S apt-get -y update && echo '' | sudo -S DEBIAN_FRONTEND=noninteractive apt-get install -y openssh-server"
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

C:\Users\robin\OneDrive\Bureaublad\kali.exe run "echo '***' | sudo -S ssh-keygen -A"
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

C:\Users\robin\OneDrive\Bureaublad\kali.exe run "echo '' | sudo -S sed -i -e '/^#[[:space:]][[:space:]]Port[[:space:]]./ s/^#[[:space:]][[:space:]](Port)[[:space:]](.*)/\1 2222/i' -- /etc/ssh/sshd_config"
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

C:\Users\robin\OneDrive\Bureaublad\kali.exe run "echo '*' | sudo -S sed -i -e '/^#[[:space:]][[:space:]]ListenAddress[[:space:]].<IP_ADDRESS>/ s/^#[[:space:]][[:space:]](ListenAddress)[[:space:]](.)/\1 <IP_ADDRESS>/i' -- /etc/ssh/sshd_config"
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

C:\Users\robin\OneDrive\Bureaublad\kali.exe run "echo '*' | sudo -S sed -i -e '/^[[:space:]]PasswordAuthentication[[:space:]]./ s/^[[:space:]](PasswordAuthentication)[[:space:]](.)/\1 yes/i' -- /etc/ssh/sshd_config"
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

C:\Users\robin\OneDrive\Bureaublad\kali.exe run "echo '' | sudo -S diff /etc/ssh/sshd_config /etc/ssh/sshd_config.winsshterm.bak &> /dev/null; if [ $? -ne 0 ]; then echo '' | sudo -S mv /etc/ssh/sshd_config.winsshterm.bak /etc/ssh/sshd_config.$(date +%Y%m%d_%H%M%S).bak; else echo '***' | sudo -S rm /etc/ssh/sshd_config.winsshterm.bak; fi
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0xffffffff Error: 0xffffffff (null) > Finished

It looks like the distribution was not correctly installed. Probably there is some other process listening on Port 53 (see https://github.com/microsoft/WSL/issues/4364)


Graceful stop from CLI

Many times it's useful to be able to gracefully shutdown agents from the CLI when ssh'd into a box
@keithpitt

As an example: Nginx use the QUIT signal for graceful shutdown and TERM / INT for quick shuftdown: http://wiki.nginx.org/CommandLine#Stopping_or_Restarting_Nginx

It would be great to be able to either shutdown after the current job has finished, or alternately shutdown immediately and have the job sent to another agent without generating an error.

@es just looking at this again, and I forgot what it was :) It's essentially to send a signal to the agent, so when it's finished it's current job, it'll exit?

Yup, for example for auto-scaling. Even better would be a way to stop it immediately and re-send the job to another agent ;)

Cool! Graceful shutdown is back! https://github.com/buildkite/agent/commit/62d2feb28dcab5b15e335395943c2cf3c9c90c4b#diff-ce9d59e4030c50e8a42c94202dbb9c82R69 This will be available in the next beta that I'll push out in the next hour or so.
Because there are 2 types of graceful shutdown (kill job, or wait for job) it was hard choosing which signals should do what. So TERM and INT will wait for the current job to finish before disconnecting. QUIT will cancel the current job, then disconnect. Hopefully that's not too confusing :\ If it is, or you disagree, I can change :)
For your case @lox, the resend the job to another agent gear is a little tricky and work will need to be done on the Buildkite side of things, but I've come up with a short term solution for the "one shot" jobs. I've added the agents PID as an environment variable to the agent: https://github.com/buildkite/agent/commit/62d2feb28dcab5b15e335395943c2cf3c9c90c4b#diff-fb08d3fc4fc6480db9e28ae0e8e21614R157
So you could have a post-command hook that does this:
kill -s INT $BUILDKITE_AGENT_PID

Which will cause the agent to disconnect after finishing the current job.


Windows: cannot use absolut path in layout-option of response.render

Following call:
   router.get('/hello', function(req, res) {
       res.render(__dirname + '/views/home.handlebars', {
           layout: __dirname + '/views/layouts/main.handlebars',
           markup: markup // Pass rendered react markup
       });
    });

leads to following error message:
Error: ENOENT, open D:\ZHAW\BA\IoT_Base\lib\views\layouts\D:\ZHAW\BA\IoT_Base\lib\plugins\IoT_HelloWorld\views\layouts\main.handlebars'
Seems to me, as if the path is handled relative even though it's obviously absolute.

@Tineler Yeah, this would need to change so that it works with Windows paths and not just Unix-style ones: https://github.com/ericf/express-handlebars/blob/master/lib/express-handlebars.js#L331-333

Seems legit. Do you mind if I fork the repo and create a pull-request after fixing it? Because I need it pretty soon. :-)


[Stitch] Update call from renderLayout to stitch

Not sure why it wasn't this way before /shrug

Merging per slack convo


Fix Application Name

This PR fixes a bug where the application name was not determined correctly.
The code now uses NSWorkspace to get the name of the application which is frontmost.
Previously, the application name was not correctly determided and rather random.

This probably also fixes #15

@seletz thank you!

@jasonjmcghee thanks for merging!


<EMAIL_ADDRESS>hash check failed

scoop install eagleget-portable
Installing 'eagleget-portable' (<IP_ADDRESS>) [64bit]
EagleGetProtable_<IP_ADDRESS>.zip (13.8 MB) [>                                   ]   0%
Checking hash of EagleGetProtable_<IP_ADDRESS>.zip ... ERROR Hash check failed!
App:         extras/eagleget-portable
URL:         http://dl.eagleget.com/latest/EagleGetProtable_<IP_ADDRESS>.zip
First bytes: 50 4B 03 04 14 00 00 00
Expected:    77c1c1aa178c06dd9a56a6974be6e211aecef5f0
Actual:      c4dc13a74e2f93c648314ce089a8bcb278af1a98

[> ] 0%

You download failed. scoop uninstall eagleget-portable; scoop install eagleget-portable


Raise when empty list is given to values/2

Closes https://github.com/elixir-ecto/ecto/issues/4522

I think it would be great to mention this in docs as well. It is not obvious that empty list must be handled explicitly. I caught this error in production.

@fuelen Thank you that's a good callout. I will update the docs.
Out of curiosity do you know of any other standard way that this is handled?

Out of curiosity do you know of any other standard way that this is handled?

If we talk about elegant ways, then I don't know any

I have asked chatgpt and it suggested generating (SELECT NULL WHERE false), which I think is better than failing. WDYT?

It will work, but it is no longer a VALUES statement :slightly_smiling_face: Don't know whether this is a problem.
Columns and types should be defined as well, so in a more complex scenario it will look like this
SELECT * FROM (SELECT NULL::uuid, NULL::text, NULL::int WHERE false) AS v1 ("id", "name", "age")

I don't think it is a problem in this case, but maybe @greg-rychlewski disagrees. :)

I think it could be a bit jarring to see such a different query in the logs if you're investigating unexpected input. But I also agree it is not good that it raises.
Since I can't think of anything better then I would go with it because I agree raising is the worst option.

@josevalim is it really wise to send that along the connection and wait for the db to respond though?

It could be part of a left join like this
select * from test left join (select null where false) as n  on true;

I am still a bit unsure but after thinking about it more, it might not be so bad to let the query go through and then raise at the driver level. My reasoning:

It keeps the query the same as the user wrote it, so that it is not surprising to see something different in your logs.
It is closer to the behaviour you will get in other systems like psql. Where you will get an error if you try to use an empty values list.
It leaves the option open for databases that want to support empty values lists. I don't know of any but I know in general some databases introduce nonstandard behaviour for convenience.

Probably point 2 is the most contentious because sometimes Ecto helps and sometimes it doesn't. In this case I could see a case for saying "ok you asked for an empty list, you got it". Similar to if you ask not a non-existent table.
If we go this route then I still to clean up the SQL because we have some issues like missing types.

Any route is fine by me. :+1:


feat(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1493

Questions ?
Please refer to the Backport tool documentation


Window size is wrong after exiting from full screen

In image viewer, if you go full screen and browse images using the arrow keys and exit full screen the window size will be the size of the original image before entering full screen. It should fit the current image instead.

Fixed