Organize and cleanup the use of utils in test/

Description
There are some cleanup things that should to be done in the test/ folder:

[ ] EXPECT_MATRIX_NEAR macro is used a lot but defined in every file separately. Example: https://github.com/stan-dev/math/pull/1872#discussion_r425050016
[ ] gp_*_test.cpp files could replace the use of pull_msg with EXPECT_THROW_MSG
[ ] util.hpp are scattered all over the subfolders, reorganize them in /prim, /rev ,...
[ ] utils should be in stan::math::test or stan::test
[ ] functions/functors/... that are defined in the global space should be put in a file specific namespace Example: https://github.com/stan-dev/math/blob/develop/test/unit/math/mix/fun/col_test.cpp
f() there should be in the col_test namespace.
[ ] expect_matrix_eq should be replaced by EXPECT_MATRIX_NEAR
[ ] expect_std_vector_eq should be replaec by a macro

Current Version:
v3.2.0

It would also be nice to replace functions, such as expect_matrix_eq or expect_rel_near with macros, so they could print the line number, where they were called if the assertion fails.

@rok-cesnovar, I saw the PR and wanted to comment on the overall approach. Just a bunch of questions... you’ve probably already thought of these:

why macros instead of functions? (I’m hoping the answer is that it’s not possible to do with functions.)
naming and friendliness to devs... these look very close in name to Google Test’s naming convention. Is there a reason we want to do this?
how would a dev know that these are extensions past Google Test and are in our code?
do we have a dev-friendly doc for these? I’m thinking along the lines of Google Test’s markdown files, but it could be anything else as long as we can find it.

To be clear, I’m not opposed to the naming convention. But it’s good to know how devs are going to find info if they think it’s in Google Test instead of our code.

why macros instead of functions? (I’m hoping the answer is that it’s not possible to do with functions.)

It is certainly possible to do this with functions. However, the error thrown with a macro is a lot more friendly. If there is a mismatch it will say that there was a mismatch inside a function named "expect_matrix_eq" and not that it was throw in for example multiply_test, line 207.  It's true that you can probably guess easily where the error was if the tests only have a single call to expect_matrix_eq. But a lot of functions have more of them and its always nice if the error directly references the line.
Not a huge fan of macros otherwise, but this seems like a good use case.

these look very close in name to Google Test’s naming convention

One reason I used the naming convention is because Stan Math already extends the Google Test naming convention. See EXPECT_THROW_MSG_WITH_COUNT and EXPECT_THROW_MSG in https://github.com/stan-dev/math/blob/develop/test/unit/util.hpp#L9 Those are there for 5+ years.
The other reason is that I like following the naming convention of Google Test. If you think that is confusing, I am open to changing it. I just find it easier if we follow a naming convention. My line of thinking is: "Oh I see these tests used EXPECT_MATRIX_EQ. So there must be a EXPECT_MATRIX_FLOAT_EQ.". I might be in the minority, so if these names are confusing if theymatch Google test, I am open to other suggestions.
There is also a function in the Math library that has probably been there forever that is named expect_matrix_eq(). And the problem there is that it's not actually checking exact equality, it just used float_eq. I know the difference is not huge, but still.

how would a dev know that these are extensions past Google Test and are in our code
do we have a dev-friendly doc for these? I’m thinking along the lines of Google Test’s markdown files, but it could be anything else as long as we can find it.

That is what I am hoping goes in the getting started dev doc, which needs to be updated: https://github.com/stan-dev/math/issues/1883
That and reading tests, which is probably one of the better ways of familiarizing with a new codebase. At least to me.

Thanks, that all makes sense! Especially the reasoning for using the naming convention. I think the trade-off there is sound and hopefully we can make it easy for others to follow along.
I'd prefer if we set up the doc with the PR. Or else it won't get done... that happens way too much and it's just technical debt that keeps accumulating if we don't doc it at the time. I'll post on the PR itself asking for the doc; I'd prefer something that's imperfect than wait for #1883. The content still needs to be written.


feat: deploy push gateway

Fixes #930

Can we add the gateway version in as well ?

@teodor-pripoae Can we add the gateway version in as well ?


Impossible to get redstone

To achieve "Make quartz a bit rosy" you need redstone, but it's impossible to get redstone without nether wart (Nether wart impossible to get too)

And dont know how to get red mushrooms

Firstly, you need a Sniffer that digs up a Mooshroom Spawn Egg. This allows you to spawn a Mooshroom (these special cows) and shear it (preferably if you have several). You can haunt the (red) Mushrooms into Crimson Fungus, grow them, harvest the Nether Wart blocks and then throw them into a Millstone. And then you have your Nether Warts. Any questions?

Oh, ok, thanks


Error "value of 'count' cannot be computed"

So I'm mostly copied the code from cloudwatch-alerts-to-slack example.
And I'm getting error:
Error: Error refreshing state: 1 error(s) occurred:

* module.notify_slack.data.aws_iam_policy_document.lambda: data.aws_iam_policy_document.lambda: value of 'count' cannot be computed

Here's the copied code:
resource "aws_kms_key" "this" {
  description = "KMS key for notify-slack test"
}

resource "aws_kms_alias" "this" {
  name          = "alias/kms-test-key"
  target_key_id = "${aws_kms_key.this.id}"
}

data "aws_kms_ciphertext" "slack_url" {
  # i'm aware this is wrong :)
  plaintext = "https://hooks.slack.com/services/AAA/BBB/CCC"
  # or?
  # key_id    = "${aws_kms_key.this.key_id}"
  key_id    = "${aws_kms_key.this.arn}"
}

module "notify_slack" {
  source = "terraform-aws-modules/notify-slack/aws"
  version = "1.3.0"

  sns_topic_name = "slack-topic"

  slack_webhook_url = "${data.aws_kms_ciphertext.slack_url.ciphertext_blob}"
  slack_channel     = "aws-notification"
  slack_username    = "reporter"

  kms_key_arn = "${aws_kms_key.this.arn}"
}

resource "aws_cloudwatch_metric_alarm" "LambdaDuration" {
  alarm_name          = "NotifySlackDuration"
  comparison_operator = "GreaterThanOrEqualToThreshold"
  evaluation_periods  = "1"
  metric_name         = "Duration"
  namespace           = "AWS/Lambda"
  period              = "60"
  statistic           = "Average"
  threshold           = "5000"
  alarm_description   = "Duration of notifying slack exceeds threshold"

  alarm_actions = ["${module.notify_slack.this_slack_topic_arn}"]

  dimensions {
    FunctionName = "${module.notify_slack.notify_slack_lambda_function_name}"
  }
}

Am I missing something?

It's something with Option 3 setup:
resource "aws_kms_key" "this" {
  description = "KMS key for notify-slack test"
}

resource "aws_kms_alias" "this" {
  name          = "alias/kms-test-key"
  target_key_id = "${aws_kms_key.this.id}"
}

// kms_key_arn = "${aws_kms_key.this.arn}"

becasue Option 1 works:
variable "kms_key_arn" {
  default = "arn:aws:kms:eu-west-1:835367859851:key/054b4846-95fe-4537-94f2-1dfd255238cf"
}

// kms_key_arn = "${var.kms_key_arn}"

The problem is with this line:
https://github.com/terraform-aws-modules/terraform-aws-notify-slack/blob/efac652a08a1a8f9ccb99dc660dc0cda47d36f25/iam.tf#L31
because when I fix the count to 0 or 1 (and use this module as sources) like
data "aws_iam_policy_document" "lambda" {
  #count = "${var.kms_key_arn == "" ? 0 : 1}"
  count = 1 # or 0
  source_json = "${data.aws_iam_policy_document.lambda_basic.json}"

  statement {
..

Everything works
Related issue: https://github.com/hashicorp/terraform/issues/17421

@xliiv sorry for the delay in my response.
Yes, HCL2 will have it fixed. Meanwhile, you have to specify key using any of available options (static var, data-source, or pass from tfvars, or from CLI). Everything will work except computed values (${aws_kms_key.this.arn}).
There is not much this module can do about it to keep in an easy-maintainable condition.

Is this mean that example doesn't work?
There is used ${aws_kms_key.this.arn}.

You are right, it does not work as it is now in that example. My bad.
I will make a PR now which will fix this.

v1.9.0 has been released. I've updated examples there.
Thanks a lot for opening this issue!


list[File] not handled correctly when there is exactly one file is provided

Validation fails when one file is provided in file.
from pydantic import BaseModel
from quart import Quart
from quart_schema import QuartSchema
from quart_schema.pydantic import File

app = Quart(__name__)
app.config["MAX_CONTENT_LENGTH"] = 150 * 1024 * 1024  # 150MB limit for rag file upload
QuartSchema(app)

class SomeOtherArguments(BaseModel):
    name: str

class Upload(SomeOtherArguments):
    file: list[File]  # Work around: `list[File] | File`

@rag_blueprint.post("/upload")
@validate_request(Upload, source=DataSource.FORM_MULTIPART)
async def rag_create(data: Upload):
    return {}, 200

Error:
{
  "errors": "74956 validation errors for Upload\nfile.0\n  Value error, Expected FileStorage, received: <class 'bytes'> ...
}

When the work around list[File] | File is used, the docs no longer handled the filed as a file, but just as a string:

Version:
Quart = "0.19.8"
"quart-schema[pydantic]" = "==0.20.0"
pydantic = "2.5.3"


[FEATURE] Import format for importing large amount of events - particularly holidays

Support for external file format that allows to quickly manage and import large amount of imported events. The option to mark them as imported (can be internally) so they can also be erased separately from the manually entered one. This will allow to make, erase and reimport modified holidays, yearly events, planned events much faster than manually entering them.

Is there any specific format you were thinking for this? My initial thoughts would be for a JSON file.
I think this would also be similar to having the option to export/backup notes like requested in #27 and #17

I would go with text format using a delimiter, perhaps even CVS, with an example file with some placeholder holidays examples to demonstrate how such a file is made.
The problem with JSON file is that GMs may not have the tools available to make them, and thus the number of people using them would be limited. This means that most people would rely on available Forgotten Realms, Golarion, etc. files for the holiday import.

BTW, if you need help, I could get to produce such files for FR and Golarion, as those are the main worlds I DM in.

I second the vote for CSV format.
proposed format:
year nr, month nr, day nr, note name, html in quotes
Of course, intercalenary stuff screws with that format. Perhaps it's better to use month name instead of number. Number works for my needs which I why I suggested it. You get all the fun of taking care of all the special cases. 😆


Crash exit Code -1 when launchign 1.20.4

Whenever I try to launch 1.20.4 I get exit code -1, and I cant find anything a bout it online. It also launches fine whenever I don't have CIT resewn in the mod folder

The mod isn't updated for 1.20.4.

As dicedpixels says, the mod is not out on 1.20.4 yet. Also exit codes are not useful without the logs.


Null in Java SDK Causes Application Crash

Hello team,
I've provided an example of Java code using the Java SDK:
public static void main(String[] args) {
    FliptEvaluationClient fliptClient = null;
    try {
        fliptClient = FliptEvaluationClient.builder()
            .url("https://api.feature-flag.test.mfw.work")
            .authentication(new ClientTokenAuthentication("Ig4MkhZzbnXMhhKQpHA5HoTpKU_L3HLXBenMid6L-Bc="))
            .build();

        Map<String, String> context = new HashMap<>();
        context.put("fizz", "buzz");

        VariantEvaluationResponse response = fliptClient.evaluateVariant("flag1", null, context);
    } catch (Exception e) {
        e.printStackTrace();
    } finally {
        // Important: always close the client to release resources
        if (fliptClient != null) {
            fliptClient.close();
        }
    }
}

As demonstrated, I passed a null value to fliptClient.evaluateVariant, resulting in the following error:
thread '<unnamed>' panicked at flipt-engine-ffi/src/lib.rs:344:90:
called `Result::unwrap()` on an `Err` value: Error("invalid type: null, expected a string", line: 1, column: 36)
stack backtrace:
   0:        0x122157578 - <unknown>
   1:        0x12219a9e8 - <unknown>
   2:        0x12214c9fc - <unknown>
   3:        0x122159b38 - <unknown>
   4:        0x1221596e8 - <unknown>
   5:        0x12215a6cc - <unknown>
   6:        0x12215a0f8 - <unknown>
   7:        0x122157a3c - <unknown>
   8:        0x122159dc0 - <unknown>
   9:        0x1221cd630 - <unknown>
  10:        0x1221cda4c - <unknown>
  11:        0x121ed85e0 - <unknown>
  12:        0x121ed7c38 - <unknown>
  13:        0x10143804c - <unknown>
thread '<unnamed>' panicked at core/src/panicking.rs:221:5:
panic in a function that cannot unwind
stack backtrace:
   0:        0x122157578 - <unknown>
   1:        0x12219a9e8 - <unknown>
   2:        0x12214c9fc - <unknown>
   3:        0x122159b38 - <unknown>
   4:        0x1221596e8 - <unknown>
   5:        0x12215a6cc - <unknown>
   6:        0x12215a0d0 - <unknown>
   7:        0x122157a3c - <unknown>
   8:        0x122159dc0 - <unknown>
   9:        0x1221cd660 - <unknown>
  10:        0x1221cd6d8 - <unknown>
  11:        0x1221cd850 - <unknown>
  12:        0x121ed7d64 - <unknown>
  13:        0x10143804c - <unknown>
thread caused non-unwinding panic. aborting.

Process finished with exit code 134 (interrupted by signal 6:SIGABRT)

I understand that Rust does not have a null value, but in this scenario, I expected an exception or some form of null handling. Unfortunately, my application crashed. I have created an issue on GitHub to address this, and I kindly request an update to the convention and language to handle such cases more gracefully.

@baonguyen-bm thanks for reporting. I will create a new release now of the java sdks (and others) with this fixed

@baonguyen-bm I have just released new versions of the Java SDK that should resolve this. please let me know if this fixes your issue after upgraded

I got it. The issue have been fixed after upgrading to 0.12.0
Thanks @markphelps


Prefer serde_json::from_str over serde_json::from_reader

serde_json is optimized for &str deserialization. So much so, that it is faster to read everything into memory once, and then deserialize from &str.
This commit replaces all from_reader with from_str.

thanks! merged


Validate educators assigned to correct homerooms

What problem does this PR fix?

Educators at our new school district deployment assigned to incorrect homerooms.

What does this PR do?

Changes the import process to select the correct homeroom for each educator based on {name,school}.

Adds a new index with unique: true to verify that all homerooms in the database are unique on {name,school}, so that there can only be one per combination.

I verified that there are no duplicate homerooms on {name,school} in any of our databases, so this is a guard against data quality regression in the future.

Deployment

The migration should deploy without errors on all of our instances, since there are no homeroom duplicates on {name,school} in our instances.

The import process should heal all incorrect homeroom assignments.

# educators_importer.rb

# this line will now select the correct homeroom:
[92] homeroom = Homeroom.find_by(name: row[:homeroom], school: educator.school)
...
# this line will heal the mis-assigned educator ids:
[97] homeroom.update!(educator: educator)

@kevinrobinson: This is the first PR improving how homerooms are handled in the import process. This fixes the immediate issue with homeroom mis-assignment to the wrong educators. I tested both the migration and the import code locally and saw that it healed the bug; see "Deployment" above. I'm going to 🚢 this one as a selfie.


Question: about cdk8s use case

Can cdk8s be used inside bigger application code base to synth YAML (using app.synthYaml()) to manually pass YAML to kubectl.
Basically I want to synth YAML during runtime on request, not pre-synth it.
Is this a valid use case or is it not supposed to be used like that?

I'm also interested in this. Thanks!

@ulutinquity appologies for the (very) delayed response. This is a perfectly valid and supported use case. If you're having any issues with it let us know 👍


Create auto-apply-retention-labels-in-office-365-using-content-types-…

…and-metadata

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: joannecklein sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


feat: input: add underline variant

SUMMARY:
Adds underline Input shape and disabled styles, updates theme 1 styles to the latest.
JIRA TASK (Eightfold Employees Only):
ENG-24330
CHANGE TYPE:

[X] Bugfix Pull Request
[X] Feature Pull Request

TEST COVERAGE:
TODO: Need to add UTs for Inputs in future change
TEST PLAN:
Pull the pr branch, run yarn storybook and verify the prop changes:

allowDisabledFocus
disabled
shape: Underline

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@89e7590). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #237   +/-   ##
=======================================
  Coverage        ?   88.41%
=======================================
  Files           ?      384
  Lines           ?    10250
  Branches        ?     3408
=======================================
  Hits            ?     9063
  Misses          ?     1167
  Partials        ?       20

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 89e7590...3a70736. Read the comment docs.

:shipit:


OOPS - Method Overloading in java

Can you please assign me this issue under the DevIncept program

Sure @KeerthanaPravallika, you can go ahead!

I have done pull request having 3 issues, so I was creating different pull requests for each one, due to our exams there was a delay in creating PR, can you please reopen this issue.


Added the first pass of documentation for the CREST fitting resources.

Will wait until this hits TQ to merge it.

giddy
Can we also get expansion on docs/sso/nonbrowserapps.md? From what I've seen and discussed with others, including EVEHQ devs, it seems the only way to do this at the moment with the current OAUTH implementation is to use a proxy that the app developers control so that we don't give away our app auth keys. While this is a slight problem in that it requires an app to have a dedicated server for this, it's not outside the realm of possibility; but it would be helpful to see this page expand and maybe have examples on how to manage the connection in between.

Feel free to open an issue for that and we can discuss it there. It's a bit off topic for this PR.


USB-Serial Freezes OS when buffer is full (Blue Screen)

Using Hardware or Software Handshaking on an external USB-Serial (FTDI) device:
When the receiving devices holds it's RTS low, the StoreAsync() should time out based on the .WriteTimeout parameter in the SerialDevice, return a 0 and allow me to react appropriately.
In IOT Core however, the StoreAsync() never times out, and will even lock up the entire OS and cause a blue screen watchdog violation after a while.
I believe there is a serious issue with the Microsoft-provided driver for USB-Serial devices.
I've detailed the issue further and supplied sample code here:
https://github.com/MyersMade/PiSerialCrashTest
I would really appreciate it if someone could verify this as well.

Hi @MyersMade ,
I will ask someone to review this issue but please be aware that there are a number of issues with the USB hardware Stack in the RPi.  Have you seen this issue on any other platform?
Sincerely,
IoTGirl

No, Pi and Windows Desktop is all I have available.

Jordan, have you had a chance to look at this?

This issue has been cross posted - https://social.msdn.microsoft.com/Forums/en-US/a299cdbf-6087-4f37-baa9-43e9f8cea2af/serial-rs232-datawriterstoreasync-causes-blue-screen-in-windows-iot-core?forum=WindowsIoT#6b167353-a856-4ab2-b372-31eea9d12cc6

@MyersMade,
There are known issues with certain FTDI USB-to-RS232 chip (I don't remember the details.).
A while back, I was using a batch of USB-to-RS232 adapter from FTDI for a community project using Windows 10 as the dev machine.  I got report from some participants the FTDI adapters had problems.
I reached out to my contact at FTDI at the time and was told there was known issue with the USB-to-RS232 adapters we were using.
Sorry, I don't have the affected FTDI) part number.  You should check to make sure you are not using a problematic device.
If the USB-to-Serial device is external to the controller, I would try a different chip to see whether the same problem occur.

Thanks for your response.  My hardware is an external USB-RS232 adapter
I've tried two different models:
GEARMO FTDI-LED
Chipset FTDI FT232RL
(tried 2 different units)
Unknown Mfg
Chipset FTDI FT232RL
Both devices I got the same results.

I guess I had assumed the hardware was okay, since they work fine in Windows Desktop.
My gut tells me that the issue is coming from whatever driver that IOT ships with for the FTDI chipsets.
Does anything stand out from what I've posted here?


Don't warn of unrecognized options when customizing options

Breaks the check for unrecognized options out into its own function. Previously it closed over the local 'options' variable and so didn't recognize options added by runners other than the included one.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


Remote session on local VM unable to read shared folder on host

So, I've (finally!) managed to get file sharing working between a Hyper-V VM and the host.
In Explorer on the VM, I can navigate to a shared folder on the host, and see that the files are there. I have to use an IP address because I'm a network admin noob.

If I run the following in rgui.exe inside the VM, I get a similar listing of the files:
> dir("\\\\<IP_ADDRESS>\\misc")
 [1] "cases.R"
 [2] "clmSamp.xdf"
 [3] "dsvm"
 [....]

However, if I run the same command in a remote RTVS session, I get nothing:
⚹> dir("\\\\<IP_ADDRESS>\\misc")
character(0)

Both the VM and host are running Win 10 Pro build 1703.

Since the code behind dir is the same in both cases (we don't do anything to handle it specially), it is likely that the difference is due to configuration. Specifically, I'm thinking it might be a firewall rule issue. While both things should use the same R.dll binary, firewall rules are applied to applications (i.e. .exe files). So if RGui.exe has a firewall rule for it, but Microsoft.R.Host.exe does not, it might be that the latter is simply not allowed to make the remote connection necessary to access files?

I couldn't find a firewall rule on the VM for rgui.exe. I turned it off anyway just to make sure, but the problem still occurs.

Were you logged-in on the VM when you tried dir with RTVS Remote? If so, can you log out of the VM and (reset the R session) trying the same? Let me know the results. This will help narrow the issue.

Yes, I was logged in. I logged out and tried it again, after restarting VS. Still happens.

Another data point: if I use a nonsense location like dir("\\\\<IP_ADDRESS>\\foo"), there's a long delay before R returns. Whereas with this, it's immediate.

I had to mess around with the network config in the VM before I could get everything to work. I've no idea if this is the right way to do it.

There's an external switch hooked up to my Wifi, for internet access
An internal switch is what I use for host file sharing. I use the IP address because the computer name is taken by the external switch.

IP configuration in the VM:
C:\Users\hong>ipconfig

Windows IP Configuration

Ethernet adapter Ethernet 2:

   Connection-specific DNS Suffix  .  :

Ethernet adapter Ethernet:

   Connection-specific DNS Suffix  .   :

Ethernet adapter vEthernet (Wifi Ex):

   Connection-specific DNS Suffix  .


[1.5] Base table or view not found on MySql 5.7

Hey Mike,
I wanted to try your module, but can't install it correctly. The SQL Queries that get executed fail for me (MySQL 5.7.13) because of the following error:
A symbol name was expected! (near "source" at position 98)
This is because "source" is a keyword, probably starting from MySql 5.7. https://dev.mysql.com/doc/refman/5.7/en/keywords.html
After installation I see the following error in the backend
Modules: Failed to init module: ProcessJumplinks - SQLSTATE[42S02]: Base table or view not found: 1146 Table 'stromvergleich.process_jumplinks' doesn't exist
A quick fix for the issue would be to escape the field names in the sql files.
Thank you for your great work!

Hi @MrSnoozles, and thanks for reporting that. I develop with MariaDB and haven't encountered the issue there. Seems somewhat strange as if it is a keyword in MySQL 5.7, then it should be a keyword in MDB as well, for the purposes of keeping features between the two.
So you reckon if I change the schemas like this, it'll be okay?
CREATE TABLE IF NOT EXISTS <table-name> (
    `id` INT UNSIGNED NOT NULL AUTO_INCREMENT,
    `source` VARCHAR(512) NOT NULL,
    `destination` VARCHAR(512) NOT NULL,
    `hits` INT UNSIGNED DEFAULT 0,
    `user_created` INT UNSIGNED DEFAULT 0,
    `user_updated` INT UNSIGNED DEFAULT 0,
    `date_start` TIMESTAMP NULL,
    `date_end` TIMESTAMP NULL,
    `created_at` TIMESTAMP,
    `updated_at` TIMESTAMP,
    PRIMARY KEY(id)
) ENGINE = MyISAM DEFAULT CHARSET=utf8;

Fixed in the above commit - please confirm that it works for you. Thanks.

@MrSnoozles - please let me know if the fix does it for you so we can close. Thanks.

Going to close this now - if you are still experiencing the issue, please re-open. Thanks.

I'm having the same issue and the latest commit did not resolve it.

@thetuningspoon what exceptions are you getting?

Module reported error during install (ProcessJumplinks): SQLSTATE[42000]: Syntax error or access violation: 1067 Invalid default value for 'created_at'
Failed to init module: ProcessJumplinks - SQLSTATE[42S02]: Base table or view not found: 1146 Table 'icare.process_jumplinks' doesn't exist

Had the same problem with mysql 5.7. Will try to submit a PR this weekend. Sorry it took so long. :)

There seems to be a timestamp issue between versions - there was a reason I changed it, just can't remember exactly what it was.
@MrSnoozles I'd appreciate that very much as I'm actually not sure what the correct approach here is. I left the default out so the timestamp fields could be populated by the insert statement. I'm not using MySQL though, so didn't pick up on the issue again.

@thetuningspoon Please see if #17 fixes this for you.

Fixed, thanks!

Excellent, closing. 👍


semantic_analyser: fix gcc build error on xenial

The following error is happening on xenial (with gcc 5.4.0):
./src/ast/semantic_analyser.cpp:1003:3: error: converting to ‘const std::tuple<long unsigned int, bool>’ from initializer list would use explicit constructor ‘constexpr std::tuple<_T1, _T2>::tuple(_U1&&, _U2&&) [with _U1 = int; _U2 = bool;  = void; _T1 = long unsigned int; _T2 = bool]’
};
^
Using Uniform initialization instead of implicit initializer_list to
tuple casting fixes the issue.

LGTM


azure lb disconnect from aks vmss

Extension name (the extension in question)
aks-preview
Description of issue (in as much detail as possible)
When creating a aks with --enable-vmss and you expose your cluster with a loadbalancer the loadbalancer gets connected to the vmsscluster.
If you need to disconnect, lets say you by misstake didn't use the correct public ip, this is not posible as vmss needs lb once it got is.

Love the feature and scaleability

aks-preview

@tehho Apologies for the late reply. Refer this article and check if that helps. This issue is open for quite sometime. Please let us know if you need any further assistance on this. Awaiting your reply.

Thank you for the response and the walk down memory lane.
This was an issue back when we used azure cli to create our aks cluster.
We have moved on to terraform as this was a better fit for us.


Refactor getVariantAnnotations code path

Assumes #1166 has been merged.
Currently the code path that is taken for a given annotation type in the getVariantAnnotations method is quite opaque. We should refactor this so that it's clear where the different annotation types share code and where they don't. We should not use the identity of the transcriptConverter function for control flow.

Working on this in conjunction with #1334

Closing based on https://github.com/ga4gh/ga4gh-server/pull/1562


User friendly version of Operations requiring idmap will not be available

If /etc/subuid is not available, LXD server gives warnings:
WARN[09-28|12:32:53] apparmor_parser binary not found or apparmor fs not mounted. AppArmor disabled.
INFO[09-28|12:32:53] LXD is starting                          path=/var/lib/lxd
WARN[09-28|12:32:53] Error reading idmap                      err="open /etc/subuid: no such file or directory"
WARN[09-28|12:32:53] Operations requiring idmap will not be available

It would be nice to know what specific operations will not be available to understand if I need this file or not.

Awesome.


feat: make checkbox labels clickable

Summary of changes

improve and simplify Checkbox component to include the checkbox label as clickable element
moved "Source" to new line because on smaller screens there never was enough space to display it properly

Context and reason for change
Closes #2349 .
How can the changes be tested
Check checkboxes and their labels in the following components:

auditing sub-panel ("1st party" etc.)
edit attribution popup (report view)
locator popup
"needs review" button in attribution details

The source field looks like a comment field, that might be confusing.
Can we maybe make the source field shorter, according to the content length? I would also prefer to see it still next to the confidence, if it fits in there. Maybe keep it there as it is now, but add onHover text with a full name of the source.

As discussed: we moved the confidence field into the same line as the source to make the two belong together and not make the source field look like a comment field.

looks nice. Just not sure if we already want to show a source of attribution (it seems not to be ready)

Discussed: we'll keep it for now and make a release only when the current story to show an attribution's source is complete.


Add state handlers for OnChange, OnEvent, etc.

Fixes #109
This will allow for state objects to be used for Change and Event callbacks.

Is this designed enough yet?
The original issue mentions the gain to be "avoiding unnecessary memory from an unused connection", but this implementation does not accomplish that.
This implementation also breaks when the state result is nil, as was given as a usecase in the original issue.

This implementation also breaks when the state result is nil, as was given as a usecase in the original issue.

Didn't realise this! Will update :)


Dynamically constructed []string has type of []interface

I want to implement an expression that returns slice of string. expr properly gives []string when it's immediate value eg ["a"] but when I do something like [str] it gives type []interface even though str is string. Below is code to reproduce behaviour.  In case of using ["a"] there is optimisation which replaces it with precomputed constant and alter type.
Maybe this. An be extended to alter type in case of string vars too.

Fixed! Now always []any!


[Idea] Separated Row & Column widgets

I'd like to know if it would be interesting to implement the idea of separated Row & Column widgets, using Gutter.
This will allow developers to create Column widgets, where the children are separated according the specific space value.
This is how it wold be used:
 GuttedColumn(
  space: BreakpointValue(xs: 0.0, sm: 10.0).resolve(context),
  children: <Widget>[
    // ...
  ],
),

This is just an idea I had, and I'd like to know if this worth including within this package.

Hey! Thanks for submitting this issue.
It looks interesting enough to be considered. But I don't think it would provide enough value to create two new Widgets that people would need to learn, and with the same properties that Column and Row.
These two widgets are the most used by lots of developers and I think we can find a solution that doesn't make them to replace them.
For example, in my projects, I usually use this extension to add a Gutter in between.
extension ListUtils<T> on List<T> {
  List<T> addInBetween(T item) =>
      length <= 1 ? this : sublist(1).fold(<T>[first], (r, element) => [...r, item, element],);
}

That would look like this
Column(
    children: <Widget>[
        Item1(),
        Item2(),
        Item3(),
    ].addInBetween(Gutter()),
);

Another interesting solution would be to add a static method inside Gutter to help doing this. Similar on how ListTile.divideTiles works.
Something like:
Column(
    children: Gutter.addInBetween([
        Item1(),
        Item2(),
        Item3(),
    ], space: 20),
);

Let me know what you think about this

I see your point about not reimplementing the Column and Row widgets using Gutter between the child.
I think the second option would work and it would be very beneficial for everybody.
One suggestion I'd make regarding your sample implementation would be to make the list argument inside the Gutter.addInBetween named. This way, you could check the spacing between each element, at the top of the function, not at the final, which would be optimal for large lists.
An example here:
Column(
    children: Gutter.addInBetween(
        list: [
          Item1(),
          Item2(),
          Item3(),
        ],
        space: 20,
    ),
);

What do you think about this @jamesblasco ?

Sounds great to me! Maybe calling it children instead of list?

Sounds great to me! Maybe calling it children instead of list?

Sure! I will open a PR soon related to the implementation & testing of this feature!

With PR #12 can close this issue?


Compute jacobian for tf.image.rot90 throws error "Encountered an exception while vectorizing the jacobian computation"

Click to expand!
Issue Type
Bug
Source
source
Tensorflow Version
tf-nightly
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Compute jacobian for `tf.image.rot90` fails and throws error "Encountered an exception while vectorizing the jacobian computation". This only happens when I set jit_compile=True.

Standalone code to reproduce the issue
import tensorflow as tf
image = tf.random.uniform([2, 2, 1, 1], minval=0, maxval=1, dtype=tf.float64)
@tf.function(jit_compile=True)
def rot_func(image):
    return tf.image.rot90(image, k=3)
with tf.GradientTape(persistent=True) as tape:
    tape.watch(image)
    outputs = rot_func(image)
print(outputs.shape)
gradient = tape.jacobian(outputs, [image])
print(gradient)

Relevant log output
(2, 1, 2, 1)

......

ValueError: in user code:
ValueError: Dimension 2 in both shapes must be equal, but are 1 and 2. Shapes are [4,2,1,2,1] and [?,2,2,1,1].
Encountered an exception while vectorizing the jacobian computation. Vectorization can be disabled by setting experimental_use_pfor to False.

Can I work on this issue.

Hi @ganeshbagal072 , I unassigned myself since I'm not working on it. Feel free to take it, but I don't know whether you need to get someone to assign it to you. I'm not familiar with the contribution workflow.

Hey! If this project is not being worked on, allow me to give it a shot.

Hi @ganeshbagal072 what's the status on the issue? I was thinking of starting to work on it if it's okay?

Hi @MThalberg,
Please make vectorization disabled by setting experimental_use_pfor=False in the Jacobian function. Please refer to the gist.
Thank You


持久化状态可以自定义时间

Clear and concise description of the problem
持久化的时间可以自定义设定 根据设定的时间选择是否失效  默认一直持久
Suggested solution
存取到本地缓存中加一个时间戳
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Hi, the option was asked in #21 and we didnt consider it. It is the responsibility of the storage and not the plugin.
You can use the storage option and create one that will expire when you want it to :)


request for new system variable of ${worskpaceRootFolderName}

VSCode Version: 1.4.0
OS Version: Windows 7

currently there is the system variable ${workspaceRoot}. This returns the full path to the workspace root so if your open folder is:
c:\folder1\folder2 then that is what ${workspaceRoot} returns.
that is fine if you want the full path returned but if you are developing a website and want to use tasks to intiate running the open file in a browser on local webserver, then you don't want the full path returned. You want the website root foldername returned so in above example you want just the foldername returned which would be "folder2". This has the benefit of greatly simplifying building a url to run in browser without resorting to coding.
For example you could build a url as
${workspaceRootFolderName}/${relativeFile}
and pass that to your web browser with a task or key definition.
currently you would need to strip the foldername from ${workspaceRoot} and you can't do that in tasks.json file.
So please, please could we have an additional system variable with just the workspaceRoot folder name in it ( NOT the full path to it.)
I'm suggesting calling it ${workspaceRootFolderName}
(unless there is already some system variable I don't know about which could be used without needing to edit it).
Thanks

@visualperception a good opportunity for a PR.

Thanks muchly.
And I would its just the folder name without any slashes. Slashes are easy to add in arguments but not easy to take out of variables in json

just installed the insider daily build and it seems the new system varaible ${workspaceRootFolderName} is there and is working exactly as I had hoped. I'll check again when next release comes out.
So thanks all for implementing this.

Great. Closing as verified.

I apologise for bit off-topic comment, but I'm wondering if there is a list of all the implemented system variables?

https://code.visualstudio.com/docs/editor/tasks#_variable-substitution

Thank you very much.


Consider defaulting required 'location' to 'resourceGroup().location'

In a lot of simpler deployment scenarios, customers would want to deploy resource groups and all the resources to be in the same location. It also creates friction for new customers that need to figure out why each resource needs to have its own location and what to set it to.
Proposal
Default the required 'location' property of resources to 'resourceGroup().location'
Before
resource singleResource 'Microsoft.Storage/storageAccounts@2019-06-01' = {
  name: '${name}single-resource-name'
  location: resourceGroup().location
}

After
resource singleResource 'Microsoft.Storage/storageAccounts@2019-06-01' = {
  name: '${name}single-resource-name'
}

Analyzer
We can also add an analyzer that prompts users to remove the redundant qualification.
Happy to do the work if you decide this feature is something you are interested in.

In the past, we've discussed the ability to opt-in to specific defaults. I'm not sure it's appropriate to make the default behavior of bicep include a default like this.
It would be interesting if we could have this as a provider setting once we have the ability to configure a provider:
import az {
  defaults: {
    location: resourceGroup().location
  }
}

import kubernetes {
  defaults: {
    meta: {
      foo: 'bar'
    }
  }
}

Just wanted to point out - there are a few other pieces of underlying complexity to be aware of in particular with location:

Not all resources support all ARM regions, so we can't generically choose resourceGroup().location and have it work 100%.
Some resources support special region strings such as global or UnitedStates.
We don't necessarily have accurate data on which resource types do/don't support location as a property.

To add another data point, Azure CLI does default the location: https://docs.microsoft.com/en-us/cli/azure/storage/account?view=azure-cli-latest#az_storage_account_create

Throwing another wrench in this, in what I hope is a very limited exception... BackendAddressPools (under Load Balancers) don't list 'location' on the root and hide it within properties.

Note that I'm implementing on of the TTK rules that resources should use a 'location' parameter instead of resourceGroup().location directly, as a best practice (location: location). All the resource snippets will also now use "location: location" by default.  See #2776.


changing all the requiresafe things to nsp things

The https://www.npmjs.com/package/requiresafe is still up and requiresafe.com seems not to be responding anymore.
Would it be relevant to turn the package down, or to mark it as deprecated? (unless if it already is, npmwww does not mark it as such sadly)

probably a good idea yeah. We'll add that to the list of things to do :)
On Fri, Feb 17, 2017 at 10:48 AM, Thomas Parisot<EMAIL_ADDRESS>wrote:

The https://www.npmjs.com/package/requiresafe is still up and
requiresafe.com seems not to be responding anymore.
Would it be relevant to turn the package down, or to mark it as
deprecated? (unless if it already is, npmwww does not mark it as such sadly)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/nodesecurity/nsp/pull/39#issuecomment-280733636, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHEOXct1g4ljJ8lwLboj-HiPqP9tOoSks5rdetngaJpZM4GZPAj
.


fix: InfoQuery coordinate transformation is inverted for y axis

Short version:
Looks like the coordinate transformation being done for InfoQuery.coord has the y bounds inverted and is getting errors in the results for the Y coordinate transformation.
Longer version:
For WMS GetFeatureInfo requests, position is queried as coordinates I and J in image coordinates based on a bbox taken from the bounds of the viewing window. The bbox is in geographic coordinates, but the position is in image coordinates where the origin (0, 0) is upper left and (WIDTH, HEIGHT) are lower right. The srs.make_lin_transf() function used to transform between cartesian and image coordinates expects image bounds to be defined as (0, 0, WIDTH, HEIGHT) but when called for InfoQuery.coord, it is being defined as (0, HEIGHT, WIDTH, 0) so the Y coordinate ends up being off.

Please consider merging this in order to extend the WMS features referenced at the bottom. We can really use this.


vSphere: Get static IP addresses

Previously, we assumed all disks of types VirtualDiskFlatVer1BackingInfo, VirtualDiskFlatVer2BackingInfo and VirtualDiskRawDiskMappingVer1BackingInfo are located on a datastore. The only exception was disks of type VirtualDiskRawDiskVer2BackingInfo for which we didn't assume they are located on a datastore but for them RDM is set to true and so we have a critical concern saying VMs which such disks cannot be migrated.
However, the 'Datastore' property is optional and according to the vSphere documentation: "If the file is not located on a datastore, then this reference is null.". We noticed that this could actually happen, even though we were not able to reproduce it on our development environments, and once it happens forklift-controller crashes while creating the inventory.
Therefore this PR introduces the following changes:

Make the code that builds the inventory more robust in terms of handling VMs with disks that are not located on a datastore
Add a critical concern for such VMs with disks that are not located on a datastore, saying such VMs cannot be migrated (later on we will anyway fail the validation of a plan due to unmapped storage in this case, even if we don't block migration of VMs with critical concerns)

@liranr23 FYI

Based on the commit - each type of disk can be without datastore?

Based on the commit - each type of disk can be without datastore?

yes


Not pulling TABU content

The constant
zif_abapgit_definitions=>c_yes

is an upper case 'Y'
https://github.com/abapGit/abapGit/blob/393b1aad8c46d54b380479c067499c771e782526/src/zif_abapgit_definitions.intf.abap#LL447C7-L447C7
and is checked in the deserialize here:
https://github.com/abapGit/abapGit/blob/393b1aad8c46d54b380479c067499c771e782526/src/data/zcl_abapgit_data_deserializer.clas.abap#L227
however the zabapgit_api_rfc_pull func module fills the decision with a lower case 'y'
https://github.com/abapGit/abapgit-api-rfc/blob/694d82e9be240bed980cab10c0f7e111bb62d7f2/src/zabapgit_api_rfc.fugr.zabapgit_api_rfc_pull.abap#LL40C23-L40C23
leading to a filter that won't import any tabu.
It should be changed to an uppercase 'Y' in all "decisions".

same goes for the requirements-met = 'n' that should be uppercase


SBC may produce old format data

From Discord:

Damuth: Question.  I updated everything to V10 (Foundry, Game System, Mods and World), migrated the system and it seems to work perfectly fine.  However: today I imported an NPC through SBC and several thing on the sheet gave me a 'Migrate Document' warning.  Will I have to do this with every new NPC I make?
[12:40 PM]Hevka: Those are added by Little Helper module i think. It have settings to turn them off.
[1:11 PM]Adenrius: Pull Request, proposing a change to the code
[1:12 PM]Damuth: Settings in Little Helper, you mean?  I'll go look for them.
Does it mean there's something actually wrong with the sheet though?
[2:04 PM]Mana ᵘᵗᶜ⁺² [they/them/it]: Yes, you can disable them, and yes, there is something wrong.
In most cases those probably don't cause issues, but they can.
[2:05 PM]Mana ᵘᵗᶜ⁺² [they/them/it]: Please report that to SBC that they're producing old format data in some aspects.
[2:12 PM]Mana ᵘᵗᶜ⁺² [they/them/it]: That feature will likely be removed from LH once PF1 implements datamodels, since that will either start making them break hard  and requiring actual repair, or they get better automigration/initialization with bad data.
[2:16 PM]Damuth: Alright, I will try to reproduce the issue (Migrated in the meanwhile) and put it up on the Git

Ogre Mage      CR 8
XP 4,800
Ogre mage (Pathfinder RPG Bestiary 221)
LE Large outsider (native, oni, shapechanger)
Init +7; Senses darkvision 60 ft., low-light vision; Perception +13
--------------------
Defense
--------------------
AC 21, touch 12, flat-footed 18 (+4 armor, +3 Dex, +5 natural, -1 size)
hp 128 (8d10+48); regeneration 5 (fire or acid)
Fort +12, Ref +5, Will +10
SR 19
--------------------
Offense
--------------------
Speed 40 ft., fly 60 ft. (good)
Melee mwk greatsword +15/+10 (3d6+10/19-20)
Ranged mwk composite longbow +11/+6 (2d6+7/×3)
Space 10 ft.; Reach 10 ft.
Spell-Like Abilities (CL 9th; concentration +12)
Constant—fly
At will—darkness, invisibility
1/day—charm monster (DC 17), cone of cold (DC 18), deep slumber (DC 16), gaseous form
--------------------
Statistics
--------------------
Str 24, Dex 17, Con 23, Int 14, Wis 14, Cha 17
Base Atk +8; CMB +16; CMD 29
Feats Combat Expertise, Combat Reflexes, Deadly Aim, Improved Initiative, Iron Will, Power Attack
Skills Acrobatics +2 (+6 to jump), Bluff +14, Disguise +14, Fly +4, Intimidate +14, Knowledge (arcana) +13, Perception +13, Sense Motive +13, Spellcraft +13, Use Magic Device +14
Languages Common, Giant, Thassilonian
SQ change shape (small, medium, or large humanoid; alter self or giant form I), finesse weapon attack attribute
Other Gear mwk lamellar (leather) armor[UC], mwk composite longbow (+7 Str), mwk greatsword

Special Abilities

Change Shape (Small, Medium, or Large Humanoid; alter self or giant form I) (Su) You can change your form.


'Access Violation' Crash

Bug
Game crash
nslog2022-01-11 16-38-41.txt
Steps to reproduce
Seemingly random. The same crash has occurred when joining a server and while in a game.
Specifications

Northstar version: 1.4.0
Platform: Origin

have you tried to see if your game files are corrupted?

I had this same problem, I tried to reinstall the game several times, as well as Northstar itself, but the game always crashes in the middle of the matches and every time I tried I couldn't play a full match

I'm having this same issue.

Im also having the same issue

This is happening to me too. I cannot run Northstar because of this. as soon as I launch it, this error appears and the game shuts down.

If this happens on Linux, u must use this proton Version: https://github.com/GloriousEggroll/proton-ge-custom/

If this happens on Linux, u must use this proton Version: https://github.com/GloriousEggroll/proton-ge-custom/

the thing is, i'm using Windows

I discovered that the access violation crash occurs for me if I press the melee button during a killcam. Avoiding this case helped me get through games without crashing.

As mentioned in #180 none of your crashes are actually caused by access violations unless the memory trying to be read is not 0x000000.

I get this every game. Never managed to play for more then 3 minutes, most of the time I can't even get into a session.

@Dunrar could open a new issue with information about your setup and current Northstar install?


Recording fails silently if the microphone is removed. (Chrome and Firefox)

Hey Muaz, Great work! RecordRTC is a great library which works seamlessly across browsers for recording audio.
There is an important case as listed.

We click "Allow" when the browser prompts us for microphone access.
Audio recording works fine, except until it fails silently when the microphone is removed either while recording is in progress, or after recording is stopped and we try to record again.

Now Chrome stops recording when any of the track is stopped or removed: StereoAudioRecorder.js#L485-L491
if (isMediaStreamActive() === false) {
    if (!config.disableLogs) {
        console.error('MediaStream seems stopped.');
    }
    jsAudioNode.disconnect();
    recording = false;
}

Same solution can be added in MediaStreamRecorder.js (inside a timer) to auto-invoke recorder.stop as soon as track is disabled or inactive.

Maybe fixed. Closing this old thread.


Prettier file contents

This change uses bat, less, or cat, depending on what is available on the users system to display the contents of new/changed env files. (Configurable via AUTOENV_VIEWER.)
Additionally, added an autoenv_message() function for colored messages.

Thank you for your contribution ^w^. This is great as I use bat as well and customizing the file output would be very useful


About the reduce ratio of step learning rate

I wonder how to set the reduce ratio of learning rate in step learning style? It seems default config is reducing learning rate with 0.1. What should I do if I want to reduce learning rate by 0.5 or other values?
optimizer = dict(type='SGD', lr=0.01, weight_decay=0.0001)
optimizer_config = dict(grad_clip=None)
lr_config = dict(
policy='step',
warmup='linear',
warmup_iters=500,
warmup_ratio=0.001,
step=[150])

Add gamma=0.5 to lr_config. You can refer to https://mmcv.readthedocs.io/en/latest/_modules/mmcv/runner/hooks/lr_updater.html?highlight=StepLrUpdaterHook# to find more details.

Add gamma=0.5 to lr_config. You can refer to https://mmcv.readthedocs.io/en/latest/_modules/mmcv/runner/hooks/lr_updater.html?highlight=StepLrUpdaterHook# to find more details.

thanks!!!!!!!!!


Please add a redirect urls for applications to support regular matching support

thks

Fixed by: https://github.com/casdoor/casdoor/commit/c661a57cb2290009c9f626f2fe9af25422a3e932


Display same color

The autocomplete doesn't have a different color than something that was typed into the shell. I tried installing as a plugin as well as manually, still no difference.

See #174, #172, #171. Probably it's an issue with your color scheme.

The color in the suggestion is same the default text color. I tried the suggestion but it doesn't it.

Have you tried setting the ZSH_AUTOSUGGEST_HIGHLIGHT_STYLE variable? See the section on configuration in the readme: https://github.com/zsh-users/zsh-autosuggestions#configuration


sdlog2: move from STAT.MainState to COMM.MainState

This fixes a bug where vehicle_system_status and the commander internal
state were conflicting.
This will need adaptations in logging tools.
The change for FlightPlot is here: https://github.com/DrTon/FlightPlot/pull/62

I don't see why we need to change the output format - can't you buffer the commander state in a local variable and just write it into the main logging thing? I certainly don't want to break all downstream tools for a trivial workaround.

Right, technically feasible. Two points against it:

I don't want tools to look at the commander internal state, otherwise commander state machine changes will be hard.
The tools are sort of outdated in that respect anyway, see:
https://github.com/DrTon/FlightPlot/blob/master/src/me/drton/flightplot/export/PX4TrackReader.java
https://github.com/mavlink/mavlink/blob/master/pymavlink/tools/mavgraph.py#L41

Maybe we should publish to both for some transition time?

Removing it is a no-no. This is what we use to understand in what flight mode the system is, there is nothing "internal" about it. And all the tools rely on it and people have written deployed scripts. The value of the new message is marginal as we will likely move on to the new logger entirely. So I would prefer to just patch it up internally and update the tools so we have local stability. Then do all the cleanup using the new format.

@LorenzMeier ok makes sense. Done and tested on Pixhawk.

Awesome, thanks!


StorageException: 404 Not Found

We make use of Google's JAVA client library to create storage client using which we get a particular blob by: storageClient.get(blobId).
We are facing an issue - calling the above method intermittently leads to "StorageException: 404 Not Found". Even when the objects exists and can be seen from console.
Below is the stack-trace:
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT **Caused by: com.google.cloud.storage.StorageException: 404 Not Found**
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT No such object: <our object name - we would not want to disclose>
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.spi.v1.HttpStorageRpc.translate(HttpStorageRpc.java:220) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.spi.v1.HttpStorageRpc.load(HttpStorageRpc.java:588) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.StorageImpl$16.call(StorageImpl.java:464) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.StorageImpl$16.call(StorageImpl.java:461) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:89) ~[gax-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.RetryHelper.run(RetryHelper.java:74) ~[google-cloud-core-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:51) ~[google-cloud-core-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.StorageImpl.readAllBytes(StorageImpl.java:461) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.Blob.getContent(Blob.java:455) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.sap.hcp.osaas.cfbroker.utils.GCPUtils.getBlob(GCPUtils.java:60) ~[classes/:?]

   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	... 89 more
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT **Caused by: com.google.api.client.googleapis.json.GoogleJsonResponseException: 404 Not Found**
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT No such object: <our object name we would not want to disclose>
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.googleapis.json.GoogleJsonResponseException.from(GoogleJsonResponseException.java:146) ~[google-api-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.googleapis.services.json.AbstractGoogleJsonClientRequest.newExceptionOnError(AbstractGoogleJsonClientRequest.java:113) ~[google-api-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.googleapis.services.json.AbstractGoogleJsonClientRequest.newExceptionOnError(AbstractGoogleJsonClientRequest.java:40) ~[google-api-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest$1.interceptResponse(AbstractGoogleClientRequest.java:321) ~[google-api-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:1065) ~[google-http-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(AbstractGoogleClientRequest.java:419) ~[google-api-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(AbstractGoogleClientRequest.java:352) ~[google-api-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeMedia(AbstractGoogleClientRequest.java:380) ~[google-api-client-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.services.storage.Storage$Objects$Get.executeMedia(Storage.java:6189) ~[google-api-services-storage-v1-rev114-1.23.0.jar!/:?]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.spi.v1.HttpStorageRpc.load(HttpStorageRpc.java:584) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.StorageImpl$16.call(StorageImpl.java:464) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.StorageImpl$16.call(StorageImpl.java:461) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:89) ~[gax-1.23.0.jar!/:1.23.0]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.RetryHelper.run(RetryHelper.java:74) ~[google-cloud-core-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:51) ~[google-cloud-core-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at com.google.cloud.storage.StorageImpl.readAllBytes(StorageImpl.java:461) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]
   2018-06-25T15:25:38.58+0530 [APP/PROC/WEB/1] OUT 	at **com.google.cloud.storage.Blob.getContent**(Blob.java:455) ~[google-cloud-storage-1.24.1.jar!/:1.24.1]

Can there be anything that we might be missing? or is it an issue on the library side?
Thanks,
Swati Jain

Hello @samir
Sorry for the long delay.
According to  the storage consistency spec read-after-write should never cause 404 Not Found.
After 5 day running of the program similar to you I'm not able to reproduce your 404 .
One can only guess about the reasons which lead to the 404 error. Some of them:

someone starts the same service on the same bucket (might be nightly build test)
someone else who has an access to your bucket may delete/move/cleanup your data
name collision in multithreading environment:
String eventsPath = element.getKey() + "/" + date.get() + "/events.csv";
 String tempPath = element.getKey() + "/" + date.get() + "/temp_events.csv";
Such names might be not unique,  I quite admit that another thread may perform operations on an object with the same name
your call: createGcsFile(bucketName, eventsPath); does not really create a blob. It could:

ignore throwing exceptions
try create a blob in another thread
by mistake do not create an object

There could be many other reasons, but I'm pretty sure your application is doing something wrong.

I'm facing same issue. I'm using spring batch to do validations(partitioning) then write the files to GCS. If there are validation error files then merge those files into one consolidated file in next step. I'm getting below error though file exist in the folder. "com.google.cloud.storage.StorageException: Object 6_Error_permuploadinput.csv (generation: 0) not found." Any help is really appreciative.
request.setTarget(
BlobInfo.newBuilder(bucketName, filePath + VALIDATION_CONSOLIDATED_ERROR_FILE_PREFIX+fileName).build());
List filesToMerge = getFileNamesToMerge(jobExecution,fileKey,fileName);
for (String name : filesToMerge) {
request.addSource(name);
}
Blob compositeObject = gcsStorage.compose(request.build());

FYI. Every-time it reports different file is missing.
{
"code" : 404,
"errors" : [ {
"domain" : "global",
"message" : "Object 4_Error_permuploadinput.csv (generation: 0) not found.",
"reason" : "notFound"
} ],
"message" : "Object 4_Error_permuploadinput.csv (generation: 0) not found."

@dmitry-fa - We identified the root cause. This is an application issue. Apologies for inconvenience.


How to 'microk8s enable gpu' in a mixed cluster?

I have a microk8s cluster in which the machines are mixed: some have GPUs and others don't. The master node does NOT.
I am trying to enable GPU usage by running microk8s enable gpu. But I get this:
Nothing to do for gpu.
I've also tried removing the GPU nodes from the cluster, enabling microk8s gpu, then rejoining the cluster. Then tried to run nvidia/samples:vectoradd-cuda10.2 like this
apiVersion: v1
kind: Pod
metadata:
  name: gpu-operator-test
spec:
  restartPolicy: OnFailure
  nodeSelector:
    hasgpu: "true"
  containers:
  - name: cuda-vector-add
    image: "nvidia/samples:vectoradd-cuda10.2"
    resources:
      limits:
         nvidia.com/gpu: 1

But this led to the pod being stuck in Pending and with events saying that the pod cannot be scheduled.
Warning   FailedScheduling          pod/gpu-operator-test   0/4 nodes are available: 4 Insufficient nvidia.com/gpu.
Is there a possibility to enable the GPU even if not all the nodes in the cluster have a GPU?

Interestingly enough, running a single-node cluster on a GPU enabled machine still doesn't allow me to run the gpu sample above.
inspection-report-20211115_225215.tar.gz


unixODBCDrivers.msodbcsql17: init at <IP_ADDRESS>

Motivation for this change
This patch adds Microsoft's ODBC driver for SQL Server.  I have tested it on NixOS against SQL Server 2008 and it has worked quite reliably in that scenario.
I'm new to packaging binaries with Nix and would appreciate feedback.  The 'libshim' symlinks in particular I'm not sure about -- the driver is linked against specific names of libssl and libcrypto and I couldn't find another way to provide or fake those names.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build unixODBCDrivers.msodbcsql17

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: unixODBCDrivers.msodbcsql17
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: unixODBCDrivers.msodbcsql17
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: unixODBCDrivers.msodbcsql17
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.


I need help with adding new stuff

Hey, i would like to include the "fake unbox" into my osiris source.
Yeah, i know that there's the same issue there: https://github.com/danielkrupinski/Osiris/issues/512 ,
but i have a problem with adding that. i tried it meny times to make it working.
Still doesnt works. could someone help me or make a tutorial?
thank you in advance.
I also want to include some stuff from "lua script" into my osiris source. I got some but the lua is new for me, and also i don't know how to code the same thing into c ++, but it's not importent for now, i mean if someone would like to help me out, pls give me your Discord contact below.
But at first, i would like to make my "fake unbox" working.
Thanks

bump

bump

#1483 close this issue I made a pull


Fix handling of service provider attributes when parsing an XML file

When #5 added service provider attributes, it didn't update ServiceProvider#from_xml method accordingly.
This PR fixes the issue in the same way IdentityProvider#from_xml handles such cases.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Gabriel Prado seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Moment.js error

I'm getting Cannot read property '_weekdaysMin' of undefined error with the latest version of MomentJS plugin.
The error is produced by the following code:
daysOfWeek: moment()._lang._weekdaysMin.slice(),
monthNames: moment()._lang._monthsShort.slice(),

In the original repository this is changed to
daysOfWeek: moment.weekdaysMin(),
monthNames: moment.monthsShort(),

Also, the fork is not in sync with the original plugin, any chance to update it?

I'd also vote for a new version ;-)


How to generate cams along with the predictions themselves?

In my use case, I only need to generate cams for one specific class (I am classifying hole-free and hole-present images). I am wondering if it is possible to generate cams along with actual model output? In an ideal scenario I would like to save inference time because I don't need cams for hole-free images. Thank you for your help.

Hi, were you able to get the model predictions along with the generated CAMs?

Hi @jacobgil !
I created a PR to implement this request. Please take a look at your convenience:
https://github.com/jacobgil/pytorch-grad-cam/pull/419


IDE Download -- Atom editor opens new window for each lesson

I am having three problems with atom editor and learn co package
1/ When finishing a lesson and entering learn submit my learn co profile does not update without refreshing the browser page.
2/ When moving on to the next lesson learn next, my editor opens a whole new atom window. Is there anyway that I can disable that from happening?
3/ Lastly, when opening a new lesson with learn open (after completing lessons in the browser that do not have IDE components) I get this error -- "WARNING: You are attempting to open a lesson that is beyond your current lesson.  Are you sure you want to continue? Yn?"
How do I get my atom editor and my learn co profile to communicate properly??
Thanks in advance.

PS: I have three atom packages installed for the the learn co curriculum:
learn-ide-3, learn-ide-3-notifications, learn-ide-3-tree
Is there a setting I can change within one of the preferences for a specific package?


[embree3] Add macOS and ARM support, plus various options

If this PR updates an existing port, please uncomment and fill out this checklist:

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Looking at existing PR/issue there may be some redundancy with #30548
Something to double check before removing the draft status

All feature tested on the following triplets:

x64-windows
x64-windows-static
x86-windows

Tagged vcpkg-team-review to answer this question:
Today, we would refuse to merge features that try to change ISA like this, under the 'do not implement alternatives as features' rule. However, embree3 already has other ISA features, should we make an exception in that this adds one like the others? See also, last time the maintainers weighed in on this topic: https://github.com/microsoft/vcpkg/pull/29093#issuecomment-1465969155
Alternatively, should we remove all ISA features entirely, because they are broken?

I'll vote for removal. Stuff like this can be set via VCPKG_CMAKE_COINFIGURE_OPTIONS or the port needs to inspect compiler flags to enable those.

@BillyONeal @Neumann-A great discussion !
I took the liberty to implement an embree3 port where we get rid of ISA features and handle it as best we can in the portfile.
I gave it a quick test on an internal project which target both native and emscripten platforms.
For Emscripten, I had to hijack the ISA selection because it was incorrect otherwise letting Embree select the default one by itself seems to work fine.
Let me know what you think of 9d47b0a

Hey @BillyONeal @Neumann-A, did you have the time to look at my latest revision which should take into account to your feedbacks ?

Gentle reminder, @BillyONeal @Neumann-A, did you have the time to look at my latest revision which should take into account to your feedbacks ?

The title of the PR should be changed to reflect the new changes.

Indeed, I'll find something more appropriate before the end of the week.


Added twitch color

The css compressor I used fucked up css/colours.min.css

I do not write directly into the CSS file.. use SCSS

Done i think

I can make a new commit about it, will close it and do it.

#11

Thank you! ^^


Improve the README

PR Checklist

[ ] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] Documentation in docs is updated

Description of changes

Codecov Report

Merging #75 (6f0a112) into main (571b6eb) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@          Coverage Diff          @@
##            main     #75   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          1       1
  Lines          5       5
=====================================
  Misses         5       5

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Pantheon: screenshots of windows missing window decoration

Describe the bug
this is an issue that exists for a long time. i'm not sure in which case it happens. let's collect situations
Screenshot has window decoration

Chromium
Steam
elementary Files
elementary Videos
elementary Music
elementary Calculator
elementary Terminal
Pantheon System Settings
Remmina
elementary Web
Gnumeric with cells selected
Mahjongg
Meld
Notes-Up
Gajim
Dino
Firefox
darktable
Fractal
GNOME Disks

window decoration missing

Atom (Electron 5.0.13)
KeePassXC (Qt 5.15.0)
Signal (Electron/8.3.4)
XTerm
Gimp
HexChat
Kdenlive (Qt 5.14.2, xcb windowing system)
Mumble (Qt 5.14.2.)
Audacity with file open (wxWidgets 3.0.4)

just black image (parts missing)

OBS
Gimp without image opened
Gnumeric without cells selected
Audacity without file opened

To Reproduce
Steps to reproduce the behavior:

do screenshot with ALT + PRINT when a window is active

Expected behavior
all windows should have window decoration
Screenshots

should look like this

OBS

should look like this :smile:

Gnumeric without cells selected

Audacity without file opened

Additional context
Notify maintainers
@worldofpeace
Metadata

system: "x86_64-linux"
host os: Linux 5.8.13, NixOS, 20.09beta992.7badbf18c45 (Nightingale)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.7
channels(root): "home-manager-20.09, nixos-20.09beta992.7badbf18c45, nixos-hardware, nixos-unstable-21.03pre246062.420f89ceb26"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

So this likely is just an issue with gala and maybe the version of mutter we're using. It would be interesting to see if you could reproduce on gnome-shell xorg these scenarios

Still an issue with Pantheon 6.


josm: 17560 -> 17580

Motivation for this change
https://josm.openstreetmap.de/wiki/Changelog#stable-release-21.02

First version 17560 was released as stable version and due to major bug version 17580 was released as hotfix.

Note that the macosx download url was apparently changed to no longer include a version.. :confused:
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117398 at f15ae741 run on x86_64-linux 1

1 package built successfully:

josm

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/misc/josm/default.nix:29:3:
   |
29 |   installPhase =
   |   ^


Site logo link not generated correctly

Describe the bug
Site logo link not generated correctly at page header.
To Reproduce
Steps to reproduce the behavior:

set baseURL = "https://example.com/" to config.toml
hugo or hugo server -D
check the html source of site-logo part
incorrect link https://example.com//images/site-logo.svg

Expected behavior
Fix site-logo path in layout files.

layouts/404.html
layouts/partials/header.html

Screenshots

Desktop (please complete the following information):

OS: macOS 10.12.6
Browser Chrome
Version 70.0.3538.67（Official Build）

Smartphone (please complete the following information):
N/A
Additional context
Add any other context about the problem here.

Good catch. Other assets are set up as `{{ .Site.BaseURL }}directory/asset-name.extension``; the site logo should be retrieved the same way as all other assets.

Thank you for your comment.
I have already prepared PR for solving this issue. May I create a PR?

@michimani Yes, that would be wonderful! Thanks. 😊


Improve table markup in Layout / Containers

Drop useless .text-left on table
Use scope="col" on th, and replace empty th with td
Use th scope="row" on class names' column.

This is all for accessibility purpose —since class names being headers makes sense here. See WAI tutorials about tables.
@patrickhlauke Feel free to tell me if it's not helpful :) I made separate commits to ease reverting some.

@patrickhlauke LGTY?

Arguably, we could probably switch this table to Markdown? But that's for another PR.

@patrickhlauke I think this looks good, I'm going to merge it later today.

Arguably, we could probably switch this table to Markdown? But that's for another PR.

sorry coming in late, but just to touch on this point: I believe (unless I'm missing something obvious) that markdown tables still don't allow to define a row header. while not absolutely critical, it's nonetheless nice from a structural point of view to explicitly be able to say that the cells in the first column are the headers for their respective rows.
(though if I'm mistaken, and there IS a way to get markdown to explicitly mark row headers...I'd be super interested to hear it as I've recently switched to using markdown tables for some of my resources, and feel dirty not having row headers)

It's still not possible AFAICT.
But currently we seem to use a mix of HTML and Markdown tables across the codebase. We should streamline this stuff and make sure what we do is right. :)
Do note that we can pass extra attributes in Markdown, since kramdown is used in v4-dev, but for master I think we can't, so better not touch this.


Improvements to DE translation

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

Inconsistency and an error in the german translation.
Gitter -> Raster (it was called Raster in every other section and is, as far as I can tell, called raster in other applications)
Festlegen - bis -> Setzen - auf
As "festlegen bis" makes no sense at all in the context of a change node.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 76.374% when pulling 62da0a0de04ead5d52fb5b74df2c0bba51fb2e0f on ketzu:de_improvements into e0111d3fe6b54bdc3d30e5f4075dadeb5bcf0177 on node-red:master.

@hobbyquaker @jwende please review these proposed changes to the .de translations.

@ketzu Glad that you are going to contribute - but can we discuss this on slack (#2193 i18n-german) or our translation issues github (https://github.com/jwende/nodered-german)  before (so we could switch to german ;-))

Gitter->Raster - I see this too
not clear (need a pointer to find this in the UI)

Cheers
J.

Can I check where this has got to? Have there been further discussions on the translations and are there any updates needed?

@hobbyquaker @jwende ding ding - heads up guys. We are trying to finalise the 1.0 release of Node-RED - so would really appreciate some input on these translations - or they will miss the boat and be in a fix later on.
Thanks

@jwende @knolleary You can find the second fix in the properties menu of the change node:

I am in favor of this change since this confused me as well until I looked at the English translation. "Setzen ... auf ..." would clear it up.


fix wrong promql in dashboard

https://github.com/open-cluster-management/backlog/issues/17320
Signed-off-by: Marco<EMAIL_ADDRESS>

/cherry-pick release-2.3


Intention to use wagtail in the near future as a CMS

Please describe the problem you're fixing here. Include the issue number, if applicable.

Hi, I think you've created this pull request in error - pull requests here are for contributing to the Wagtail project itself.


What should be value of YOUR_DOMAIN_GOES_HERE ?

This might feel like very basic question but I'm confused on what exactly the value of this variable would be. Will it be azure app service or some Sprint boot localhost ? According to the documentation here I used azure endpoint of bot service in AADv2 apps.dev portal as well as in apps.js but I get following error.
{"code":"ResourceNotFound","message":"/botauth/aadv2 does not exist"}

Also, on  apps.dev.microsoft.com in need following url's
Redirect URLs = https://{???????????}/botauth/aadv2/callback
Home page URL = https://{???????}/botauth/aadv2

I tried azure bot service endpoint but it didn't work.
I ran localhost:port using sprint boot and used localhost:port as MY_domain but did not work.
My ngrok have same issue

I doubt if it has to be host part of Bot endpoint http://{?????}:3979/api/messages

@EricDahlvang Please let me know if you could answer. I would appreciate.

I'm getting following error in the browser.
Sign in
Sorry, but we’re having trouble signing you in.
AADSTS50011: The reply address 'https://localhost:3979/botauth/aadv2/callback' does not match the reply addresses configured for the application: '6a582*********e449b613f0'. More details: not specified

@rjgmail88 you can't use a localhost domain for Azure AD authentication. It has to be a public facing URL, like a webapp you have deployed on Azure. If you want to expose your localhost for development, have a look at ngrok.

I hosted my bot in azure so I have https public endpoint as my domain. I am able to see Microsoft login page, once entered my AAD account and password it takes me to my Corp specific login page. Once I re-enter credentials  I get 403 error.
This is weird because I'm using same app and same user to generate token to hit MS graph API's from Postman.
But, I validated my credentials to be correct 100% correct. In fact use same corp account through CORTANA skill where I use AAdv1 for authentication.

Sometimes error says
"failed to get bot state"


Remove pattern flags and disable failing tests

Codecov Report

Merging #129 (437cd65) into refactor-patterns (57021b1) will decrease coverage by 0.73%.
The diff coverage is 100.00%.

@@                  Coverage Diff                  @@
##           refactor-patterns     #129      +/-   ##
=====================================================
- Coverage              91.31%   90.57%   -0.74%
=====================================================
  Files                     29       29
  Lines                   3039     2981      -58
=====================================================
- Hits                    2775     2700      -75
- Misses                   264      281      +17

Impacted Files
Coverage Δ

src/infer/infer_expr.rs
99.31% <ø> (-0.01%)
:arrow_down:

src/infer/infer_pattern.rs
67.56% <ø> (-11.38%)
:arrow_down:

src/infer/infer_prog.rs
93.75% <ø> (-0.19%)
:arrow_down:

src/types.rs
75.00% <ø> (-0.74%)
:arrow_down:

tests/integration_test.rs
98.34% <ø> (-0.09%)
:arrow_down:

src/infer/constraint_solver.rs
90.34% <100.00%> (-0.44%)
:arrow_down:

src/codegen/js.rs
88.14% <0.00%> (-2.16%)
:arrow_down:

src/codegen/d_ts.rs
83.23% <0.00%> (-0.60%)
:arrow_down:

... and 1 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 57021b1...437cd65. Read the comment docs.


[FEATURE] builtin file watcher

The current one has a lots of problems, starting a list:

Doesn't ignore changes to *.swp files.
Sometimes fails to restart after errors.
Some graceful shutdown problem. e.g. go run leaves behind binaries (h/t @BenElgar)

Perhaps more importantly, builtin file watcher support would be a welcome feature! Once we have custom exo manifests, we could add watch configuration for each service/process/container. Exo would then automatically restart processes via the control interface when things change.

cc @finnigantime

Some foundational WIP here: https://github.com/deref/fsw