Failed to compile when starting server

Browser displays errors after starting the server with "npm start". Keeps saying "Failed to compile".
Module not found: Can't resolve 'assets/css/material-dashboard-react.css' in '/home/user/Desktop/projects/project-folder/src'
Node/npm versions:
npm version - 5.6.0
Node version - 9.5.0
OS - Linux Mint

Hello @garettschultz ,
Thank you for your interest in our product.
The problem is that it does not read the .env file. To do this, you can choose one of the following:

Change NODE_PATH=src to NODE_PATH=./src inside .env file;
Change "react-scripts start" to "cross-env NODE_PATH=./src react-scripts start" inside package.json ;
npm install -g cross-env ;

I would suggest that you do the third option, and that is because all our products will have the same configuration, and if you will work with other of our products, they will also get this error and you would have to make one of options 1 or 2 always. If you do the third option, the .env file will be taken in consideration on start.
Best,
Manu

Thanks for the response. So if I was going to use your components, which option would be best to use out of those 3?
-------- Original message --------
From: Nazare Emanuel-Ioan<EMAIL_ADDRESS>Date: 2/7/18 12:49 AM (GMT-06:00)
To: creativetimofficial/material-dashboard-react<EMAIL_ADDRESS>Cc: Garrett Schultz<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [creativetimofficial/material-dashboard-react] Failed to compile when starting server (#4)
Hello @garettschultzhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgarettschultz&data=02|01||0f5898b8e33a474211de08d56df6fdd5|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636535829935234492&sdata=E9YiNdB%2BTCgoutXOpUgS3fKmucUYzD5OkYFHZ2Iowgk%3D&reserved=0 ,
Thank you for your interest in our product.
The problem is that it does not read the .env file. To do this, you can choose one of the following:

Change NODE_PATH=src to NODE_PATH=./src inside .env file;
Change "react-scripts start" to "cross-env NODE_PATH=./src react-scripts start" inside package.json ;
npm install -g cross-env ;

I would suggest that you do the third option, and that is because all our products will have the same configuration, and if you will work with other of our products, they will also get this error and you would have to make one of options 1 or 2 always. If you do the third option, the .env file will be taken in consideration on start.
Best,
Manu
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcreativetimofficial%2Fmaterial-dashboard-react%2Fissues%2F4%23issuecomment-363674157&data=02|01||0f5898b8e33a474211de08d56df6fdd5|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636535829935234492&sdata=L%2Bghg5C6GjNNvHJb0OvmcWL31eRAD4qiFjCftVqSWfg%3D&reserved=0, or mute the threadhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAItePWpkKA37QwZSX6kiG58mfo2W5AyMks5tSUeQgaJpZM4R77W2&data=02|01||0f5898b8e33a474211de08d56df6fdd5|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636535829935234492&sdata=CvFtFXTSoM1zsxs063qqqiQEmM0XOrbErPujDucpHtk%3D&reserved=0.

You should go with the third one.
Manu

Okay, thanks for such a quick reply. Appreciate the support.
-------- Original message --------
From: Nazare Emanuel-Ioan<EMAIL_ADDRESS>Date: 2/7/18 12:56 AM (GMT-06:00)
To: creativetimofficial/material-dashboard-react<EMAIL_ADDRESS>Cc: Garrett Schultz<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [creativetimofficial/material-dashboard-react] Failed to compile when starting server (#4)
You should go with the third one.
Manu
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcreativetimofficial%2Fmaterial-dashboard-react%2Fissues%2F4%23issuecomment-363675281&data=02|01||22f206d94cbb41b1bfc808d56df7f273|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636535834042204296&sdata=Dt5awZv27OdIXaKzA21pbjg6Ln1RawF0EUYSx2JgVVs%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAItePcv62eNpkfysnqIdOJ-K8VbGygIZks5tSUkqgaJpZM4R77W2&data=02|01||22f206d94cbb41b1bfc808d56df7f273|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636535834042204296&sdata=P8GSPIpsMsuBaoIdcyKgIWWgiiUBPGvT5RmOarUa9BI%3D&reserved=0.

This info would have been really useful in the readme; I just wanted a super quick start but had to spend time figuring out that NODE_PATH=src SKIP_PREFLIGHT_CHECK=true npm run start is what you need to get stuff going asap


[FLINK-18300][sql-client] SQL Client doesn't support ALTER VIEW

What is the purpose of the change
Fix the issue that ALTER VIEW operation is not supported in SQL client
Brief change log

Support ALTER VIEW in SQL client
Add test

Verifying this change
Added test case
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? NA

Can you re-trigger azure testing?

My personal pipeline passed: https://dev.azure.com/lirui-apache/flink/_build/results?buildId=145&view=results


Flakyness in e2e test

While testing the coop settle with an updated PC, we got reports that the following error happens occasionally (~every third run):
FAIL tests/e2e/e2e.spec.ts (16.359 s)
  e2e
    ✕ scenario (61 ms)

  ● e2e › scenario

    thrown: Object {
      "reason": [ConnectionError: request failed: socket hang up],
    }

      139 |    *
      140 |    */
    > 141 |   test('scenario', async () => {
          |   ^
      142 |     await depositToUDC(raiden);
      143 |
      144 |     /*

      at new Spec (../node_modules/@jest/core/node_modules/jest-jasmine2/build/jasmine/Spec.js:116:22)
      at Suite.<anonymous> (tests/e2e/e2e.spec.ts:141:3)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 total
Snapshots:   0 total
Time:        19.689 s, estimated 158 s
Ran all test suites matching /tests\/e2e/i.

This wasn't seen in any of the tests runs on the last 3 PRs I created. I think it was some local flakyness which either got fixed in the environment, by one of the early PRs or by the auto-dependency upgrades. I think this can be closed.


EAGLE-140 Add eagle pipeline, single kafka monitor and hadoop metric monitor scripts

https://issues.apache.org/jira/browse/EAGLE-140

Merged into master branch.


Highlight.code not available

Highlight.code(text) (as referenced in the readme) doesn't seem to be available. I'm getting the following error:
Type 'Highlight' has no member 'code'

Am I missing something or is this an error in the documentation? Thanks 🙏

Ah, I'm assuming what is meant in the readme is Highlight.text(code). At least this seems to work.

Yes the readme is out-of-date, good spot. Opted to switch it to Highlight.text(code) but open to suggestions. I will update now. Thanks!


Pluralize the number of items in a folder

It's in the media manager. Here:

Also fixed icon for Return to the parent folder

@LukeTowers What is wrong with this PR?

@WebVPF hadn't had a chance to look yet. Do you have a screenshot of the end result?

This is what it looks like now:

After the change:

Icon changed to only link to parent folder

Endings in Russian can be seen well

Is it supposed to still say item(s)?

@LukeTowers

Great job @WebVPF, thanks! It's a small quality of life improvement but it's just one of those nice to have bits of polish that ties everything together more and makes the project as a whole feel more fleshed out and professional :)
Keep up the great work!


Update cost data FTT:P

Based on the conversation we had at the community meeting, we have figured out how to use the data we pay licenses for without displaying the original protected data: we build a method to combine multiple data sources. We have access to:

BNEF data (Exeter)
IEA
Enerdata (World Bank)

close as duplicate of #210


Nox(root mode) and samsung galaxy tab s7 can't work at selection of the friends list

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
TW
FGA build number
1836
Describe the bug
When I updated my FGA 1830 to 1836,it didn't work at friend list.Is that a bug?Thanks
Video
Nox: https://youtu.be/HqjAlSzmynE
Samsung galaxy tab s7: https://youtu.be/f313WzaM_7o
Device model
Nox
Android version
7
Screen size
1920x1080
RAM
4GB

https://github.com/Fate-Grand-Automata/FGA/wiki/Troubleshooting#where-should-i-place-the-play-button

For events I recommend using the ingame filters for ce instead of the preferred ce selection in fga


Add Catalina zsh support statement

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Closing this PR because of the above, but thanks again for contributing


Waiting on upload to Anaconda Cloud for version 0.8.0

cowplot 0.8.0 is not available on Anaconda Cloud after merging the updated recipe in PR https://github.com/conda-forge/r-cowplot-feedstock/pull/1. There is currently an issue with the conda-forge binstar token, and the core members are working on fixing this. For now we just have to wait until given further instructions.

It's a known issue with uploading the packages to anaconda.org. People are working on it and if this is fixed we need to trigger the build again.
Sorry.

The token here is expired. Please see the "Near term" plan on how to fix this.

Thanks to the quick work from the core team, we don't need to do anything. This was fixed by commits https://github.com/conda-forge/r-cowplot-feedstock/commit/bdb4a008ab14897711620952fa7b14d00e7c3bb2 and https://github.com/conda-forge/r-cowplot-feedstock/commit/cf3614e2654c1ef6399b7196fbaa5f91ba3b073d


get_token_information incorrectly handles memory allocation/deallocation

https://github.com/microsoft/wil/blob/7faa90d7185c8617f73694c5382b21cf4ed05825/include/wil/token_helpers.h#L163-L179
Essentially, the function directly calls operator new to allocate a buffer of memory:
https://github.com/microsoft/wil/blob/7faa90d7185c8617f73694c5382b21cf4ed05825/include/wil/token_helpers.h#L172-L173
and later casts the returned pointer to some structure type T (like TOKEN_USER):
https://github.com/microsoft/wil/blob/7faa90d7185c8617f73694c5382b21cf4ed05825/include/wil/token_helpers.h#L176
and finally calls delete ptr, where ptr has type T * (like TOKEN_USER *).
https://github.com/microsoft/wil/blob/7faa90d7185c8617f73694c5382b21cf4ed05825/include/wil/wistd_memory.h#L281
This does not seem to be legitimate (however "seems to work").
This issue is also reported by ASAN in Visual Studio 2022 17.4.0.

Agree on the solution. The tricky ones will be get_token_information_nothrow and get_token_information_failfast as the caller needs to explicitly declare the result's type as it is an output parameter. I.e. uses of these will need source changes. Searching for these two yields 76 files - let's call it 70 when we account for WIL itself and its tests.  There's also hits in 80 files for get_token_information and I'm unsure how many of them don't use auto, or store it as a member, or pass it as input to another function, or need to forward declare outside of lock scope, etc.
It's typically better and more preferrable if these types of changes are made directly in the OS source code.

Now I'm curious if I misunderstood the warning/error that led to me adding this when enabling the ASan/UBSan tests...

For folks playing along from home, http://task.ms/43619989 is the OS work item.


"in" parameter modifier: Contradiction in overload resolution

The page says:

The in, ref, and out keywords are not considered part of the method signature for the purpose of overload resolution

A few lines later, it says:

Overloading based on the presence of in is allowed

In my understanding these statements can't both be true. If the in keyword is not considered part of the signature, then overloads based on its presence shouldn't be allowed.

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

ID: 6b1434b0-5459-7569-47f7-46bf33fc3953
Version Independent ID: 8a4ae81b-1ae5-cbe6-d58b-df455414e245
Content: in parameter modifier - C# Reference
Content Source: docs/csharp/language-reference/keywords/in-parameter-modifier.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for pointing this out @neonb
We had the same discussion updating the C# standard (for C# 5.0).  We changed the pertinent text in the standard as follows:

Although out and ref parameter modifiers are considered part of a signature, members declared in a single type cannot differ in signature solely by ref and out. A compile-time error occurs if two members are declared in the same type with signatures that would be the same if all parameters in both methods with out modifiers were changed to ref modifiers. For other purposes of signature matching (e.g., hiding or overriding), ref and out are considered part of the signature and do not match each other.

Note: This restriction is to allow C# programs to be easily translated to run on the Common Language Infrastructure (CLI), which does not provide a way to define methods that differ solely in ref and out. end note

For later versions, when in is allowed, we should add something like the following:

Although in, out and ref parameter modifiers are considered part of a signature, members declared in a single type cannot differ in signature solely by in,  ref and out. A compile-time error occurs if two members are declared in the same type with signatures that would be the same if all parameters in both methods with out or in modifiers were changed to ref modifiers. For other purposes of signature matching (e.g., hiding or overriding), in, ref and out are considered part of the signature and do not match each other.

Note: This restriction is to allow C# programs to be easily translated to run on the Common Language Infrastructure (CLI), which does not provide a way to define methods that differ solely in in, ref and out. end note

I've added this to our backlog to update when we next update this article. I've also added the up-for-grabs label in case any community members want to submit a PR in the meantime.

@BillWagner , I recon this one should be closed as you've already added the changes you mention in the discussion, correct?

Thanks @rafcon-dev This has been addressed. I'll close it.


PSCore: Linux - Clear-Host injects data into function return value

Steps to reproduce
Function Get-SomeData {
    Clear-Host
    $somedata="somedata"
    return $somedata
}

$myData = Get-SomeData

$myData[0]     # will clear the screen
$myData[1]     # "somedata"
$myData.Count  # will be 2

Expected behavior
Function should return a single value ("somedata")

Actual behavior
Function returns an array with the Clear-Host command as the first object

Environment data
Verified in:

Name
Value

PSVersion
6.2.0

PSEdition
Core

GitCommitId
6.2.0

OS
Linux 4.15.0-47-generic #50-Ubuntu SMP Wed Mar 13 10:44:52 UTC 2019

Platform
Unix

PSCompatibleVersions
1.0 2.0 3.0 4.0 5.0 5.1.10032.0 6.2.0

PSRemotingProtocolVersion
2.3

SerializationVersion
<IP_ADDRESS>

WSManStackVersion
3.0

(BTW this works as expected in PSCore - Windows)

Clear-Host on Linux calls the native command /usr/bin/clear,
// Porting note: non-Windows platforms use `clear`
return @"
& (Get-Command -CommandType Application clear | Select-Object -First 1).Definition

which works by outputting a string with ANSI escape codes:
<esc>3;J<esc>H<esc>2J

They are: clear screen including scrollback, move cursor to top left corner, clear screen. Source.
#8554 and #8603 and #8606 and #8609 are related issues, and it looks like there's some debate about whether it's PowerShell or .Net's responsibility, what the default bahviour should be, and how it should interact with "clearing" remote sessions or newly implemented hosts.
As a Linux-specific workaround, you could write that string (or the output of clear) to the host yourself:
Function Get-SomeData {
    $Host.UI.Write("`e[3;J`e[H`e[2J")
    $somedata="somedata"
    return $somedata
}

$myData = Get-SomeData

Sure, I could also (as I've done) fork the logic:
If ($isLinux) { etc }
... but that kind of obviates the point of a cross-platform language, doesn't it?
I think what's more concerning to me is why Clear-Host is messing with my variable stack. Whatever the subtle differences (clear screen, clear buffer+screen, etc) it definitely shouldn't be returning a value.
But that seems to be what's happening here.

Actually the more elegant fix, in case anyone runs across this, is simply:
$null = Clear-Host
Which will clear the call stack so everything behaves as expected. But it's still code behaving badly.

When I try that fix on Ubuntu 16, the screen is not cleared?

why Clear-Host is messing with my variable stack. Whatever the subtle differences (clear screen, clear buffer+screen, etc) it definitely shouldn't be returning a value.

All command output or free values anywhere in a PowerShell function become part of the return value of the function (that decision was taken to copy sh/bash/ksh/etc. behaviour). Clear-Host particularly returns a value because it wraps /user/bin/clear a native command which works by returning a value.
clear doesn't clear the screen directly, it prints a string expecting the display terminal to clear itself. If you do something to block that, the screen doesn't clear and you get the string instead. The same thing happens in Bash; at a shell prompt:
clear     # screen is cleared
clear 1>/tmp/test    # screen is not cleared, reset string written to file

result=`clear`    #screen is not cleared, output captured into variable
echo $result     # screen is cleared.

function test { clear; echo "123"; }
test    # screen is cleared and 123 is output
result=`test`    # screen is not cleared, no visible output
echo $result | wc -c    # result variable has 16 characters, not 3
16

echo $result    # screen is cleared, 123 is output

that kind of obviates the point of a cross-platform language, doesn't it? :)

It seems to behaves like the platform it's running on. Whether that's badly or not, depends on what behaviour you want 🤔

I think it should be a bug because script behavior varies between platforms.
Clear command issues escapes and PowerShell intercepts this. Perhaps we could re-send by raw interface API.
/cc @mklement0

I agree, @iSazonov.
@HumanEquivalentUnit: True, but since PowerShell, unlike Bash, is able to distinguish between to-terminal output and success-stream output, the fix is to write directly to the terminal:
${function:Clear-Host} prints the function definition, and on Unix-like platforms you get:
& (Get-Command -CommandType Application clear | Select-Object -First 1).Definition

The above writes implicitly to the success output stream, so wrapping it in [Console]::Write() fixes the problem:
function Clear-Host {
  [Console]::Write((
    & (Get-Command -CommandType Application clear | Select-Object -First 1).Definition
  ))
}

Test:
& { Clear-Host; 'hi' } | Should -Be 'hi'

(Note: In PowerShell Core 7.0.0-preview.4 I get a spurious Suggestion [4,General]: The most similar commands are: New-Object, Tee-Object. message the first time Should is run in the session, but this is unrelated and can be ignored).

@mklement0 Many thanks! I pulled PR with your suggestion.

Thanks for the quick turnaround, @iSazonov.
Taking a step back: By unconditionally delegating to /usr/bin/clear on Unix-like platforms, we've actually already made Clear-Host terminal-specific, which means that it may malfunction / not do anything meaningful in other types of hosts.
To quote @BrucePay from one of the linked issues, at https://github.com/PowerShell/PowerShell/pull/8603#discussion_r246132597:

This isn't going to work for non-console hosts [...] and remoting. On those platforms, you still need to use the $RawUI portability APIs. A nice solution would be to actually add a new RawUI.Clear() API and then implement that using Console.Clear() in the console host but that may end up being a bunch of work...

Case in point: If you currently execute Clear-Host in a remoting session connected to a Unix machine, it prints the following message to stderr (and does nothing else).
TERM environment variable not set.

@mklement0 I see we have some issues with Clear-Host. but I still don't have the ability to test on Unix-s and can not grab this. (Although we could do RawUI.Clear() step by step).


Allow YouTube API Parameters

I'd like to be able to use the YouTube API parameters (available at https://developers.google.com/youtube/player_parameters?hl=en#Parameters) when using Lity. It seems like if I have a URL like https://www.youtube.com/watch?v=ngElkyQ6Rhs&autoplay=1&showinfo=0&rel=0&iv_load_policy=3 everything gets stripped except for the autoplay=1. I poked around in the source and saw where the regex takes place and figure there is probably a way to edit the code a bit to allow for more parameters. Thoughts?

Good idea! I've implemented this in d808314c035b03d30b3ad57af3b2bb472d0635c0. Could you try the files from master if it works as expected?

Works perfectly! I get this rendered into the DOM:

Thanks! If you add this as a new release I can just update by bower.json to reference this new version. Much appreciated!

As an aside, that "max-height" parameter could probably be something not hardcoded into the lity-iframe-container div. Perhaps moved to the css instead?

v1.4.2 tagged.
Regarding max-height: yes, i want to maintain support for IE8 at the moment.


URL have to be ends with "filename.vue"

Hi,
We needs to use your plugin to load vue components per http / ajax request.
In our usecases the URL to the external- or remote component looks like this:
http://service/path/to/my/component
or
http://service/path/to/my/component/
Currently this URL delivers the content of the related component as text/plain.
Is this a problem?
It looks as that your plugin changed our URL to this:
http://service/path/to/my/component/index.vue
Is there a option or whatever to configure the behavior for the url?
Greetings Mario

Another problem is:
How I have to import your package correctly?
If I try to import your package with import 'httpVueLoader' from 'http-vue-loader' the variable httpVueLoader is undefined.

Sorry, for my postings but I do not understanding how I have to use your plugin.
Can you help me with the http-vue-loader?
I try to use it in any components but when I place the component in my related template then an error occurse in vue because he do not found the component …

Best regards,
Mario


[ci skip] adding user @adrianinsaval

Hi! This is the friendly automated conda-forge-webservice.
I've added user @adrianinsaval as instructed in #94.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #94

@conda-forge/core please merge


API for "MarketOpeningsCalendar"

Please provide a API for something like "MarketOpeningsCalendar".
"MarketOpeningsCalendar" can inherit/extend from "HolidayCalendar" but with support of the opening times ("trading hours") of a stock market. For example, something like this:
StockMarketOpeningCalendarId id = StockMarketOpeningCalendarId.NYSE;
StockMarketOpeningCalendar  cal = id.resolve(ReferenceData.standard());
// returns true, if stock market is actually open
cal.isOpen(myLocalDateTime)
// returns the next open localDateTime
LocalDateTime myNewLocalDateTime = cal.next(myLocalDateTime);

Thanks.

Firstly, it is important to note that not all holiday calendars refer to stock markets, thus it is likely that this would be separate data, probably keyed by MIC code.
Secondly, the rest of the Strata codebase does not have a need for market opening data, so it is unlikely we would add it until there is a need in the analytics.

Okay, I understand. So feel free to close this issue or leave it open when there is a need for such analytics.


SW-15988 - Fixed backend order PDF download

Fix: Without the exit the server resets the connection for some reasons
In addition to #666

Not sure, but could this be a regression from 86e746db2f957870622f102736f338ac202461a2 ?
Before that revision, we prevented plugins from registering to that action for good reason - not sure, why this was removed in the first place.

I dont't think that it is related..
The Problem here was: The server send an RST-Package to  the client somewhere in the middle of the file download (and that only on the live server and not in my dev environment) -> The client stopped the doenload because of an "Network Error" (interesting: curl has no problem downloading it)

@dnoegel: I have tested it with an old version (without 86e746d), so it should not be reelated

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-16506.
Please use this issue to track the state of your pull request.


ExpandDerivedTypesNavigationProperties setting granularity support

Due to previous issues with the graph surface size seen in https://github.com/microsoft/OpenAPI.NET/issues/866 the ExpandDerivedTypesNavigationProperties setting is currently disabled in generating the openAPI from the CSDL metadata.
This however may lead to some missing paths like actions/functions bound on the derived types or convenient nav Properties just one level deep.

https://docs.microsoft.com/en-us/graph/api/ediscovery-caseexportoperation-getdownloadurl?view=graph-rest-beta
https://docs.microsoft.com/en-us/graph/api/intune-apps-mobileappcontentfile-commit?view=graph-rest-1.0

We could possibly consider adding a "depth control" setting to to limit how deep the paths would go to compromise on this rather than a yes/no situation.

The last time I tested this by generating derived types on navigation properties by going just one level deep I ended up with a significant number of paths generated.
Maybe we can figure out the possibility of having a custom annotation that we can attach to the derived types that require expansion?

Should we also consider having a list of entity types to expand in the settings?

Should we also consider having a list of entity types to expand in the settings?

That should probably be configured in the metadata as an annotation, as this would ideally be handled by the workload when they are updating their API.

For context, at the moment, we generate about 7K paths for v1.0. Expanding the navigation properties of derived types for the single type cast segment only generates about 126K paths.
@darrelmiller @baywet

Should we also consider having a list of entity types to expand in the settings?

I don't think this would be the best approach here, because I'd require the code that does the conversion (eg hidi, other tools) to know about the content of the CSDL in advance, which is not always the case (hidi, again).
I believe this problem of over expansion of methods has multiple facets:

type cast segments should only have the methods of the target cast type under them, not the ones from the base type as those can be accessed by simply not casting.
methods should only be added to singletons, entity sets, entity sets entries, single contained navigation properties, collection contained navigation properties and collection contained navigation properties entries.
I believe @darrelmiller introduced an annotation to support unbound methods, but I don't remember the specifics.

Then there's also the over expansion of navigation properties themselves:

Contains target is wrong at a bunch of places and we should either ask workloads to fix it or "fix it in post".
Only contained properties should be expanded, since the non-contained ones can be addressed via their canonical path.
I believe @darrelmiller introduced an annotation to force expansion or stop expansion in edge cases, but I don't remember the specifics.

Once we iron out those implementation aspects, we should be in a position where, when the metadata is correct, we generate an accurately expanded description for it.

I agree that we want to enable ExpandDerivedTypesNavigationProperties.  However, before we do that we must ensure the following things:

Where only a subset of derived types are supported on a navigation property we need to add the DerivedTypeConstraint annotation to limit the surface area generated to only valid types.  e.g. Manager is declared as directoryObject but can only be a user.
ContainsTargets that are wrong should be corrected to prevent auto expansion of navigation properties
Where we want non-contained navigation properties to have child paths, we must explicitly add annotations to describe the supported capabilities
As mentioned by Vincent, we must ensure that Operations bound to the base type are not regenerated for the path with the cast to the derived type.

@baywet There is no support for unbound operations.

@andrueastman to get operations bound on derived types of contained navigation properties, you just need to set AppendBoundOperationsOnDerivedTypeCastSegments = true
For the first example provided: https://docs.microsoft.com/en-us/graph/api/ediscovery-caseexportoperation-getdownloadurl?view=graph-rest-beta, the path can be generated as expected:

For context, at the moment, we generate about 7K paths for v1.0. Expanding the navigation properties of derived types for the single type cast segment only generates about 126K paths.

With the change made in https://github.com/microsoft/OpenAPI.NET.OData/pull/302, does this mean that this is no longer valid/correct as we were unnecessarily expanding path with ContainsTarget=False.

For context, at the moment, we generate about 7K paths for v1.0. Expanding the navigation properties of derived types for the single type cast segment only generates about 126K paths.

With the change made in #302, does this mean that this is no longer valid/correct as we were unnecessarily expanding path with ContainsTarget=False?.

Correct @andrueastman. Please check out this table for the appr. path figures:


Italian translation of README and FAQ

Thank you.
I rebased/squashed, fixed some internal links (since the titles are translated, the anchors automatically change), and merged into master.


Update bundle, cloudwatch, sdk-core, ses, sns, ... to 2.25.55

About this PR
📦 Updates

software.amazon.awssdk:bundle
software.amazon.awssdk:cloudwatch
software.amazon.awssdk:sdk-core
software.amazon.awssdk:ses
software.amazon.awssdk:sns
software.amazon.awssdk:sqs
software.amazon.awssdk:ssm

from 2.25.15 to 2.25.55
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #3209.


[📑 Docs]: Outdated tutorial guides https://www.asyncapi.com/docs/tools/generator/generator-template

What Dev Docs changes are you proposing?
In the tutorial it has this snippet
class TemperatureServiceClient:

  def __init__(self):
              self.client = mqtt.Client()
              self.client.connect(mqttBroker)

But this would throw an error because the python package in this case now requires a callback api version specification. I researched further on how to fix and found out the the python client packages offer two version for api call back and any can be used in this case
CallbackAPIVersion.VERSION2
or
CallbackAPIVersion.VERSION1

so the fixed code should look something like this
class TemperatureServiceClient:

  def __init__(self):
              self.client = mqtt.Client(CallbackAPIVersion.VERSION2)
              self.client.connect(mqttBroker)

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hey, Can I work on this?

Hey, Can I work on this?

I don't really mind you working on this but i was waiting for any of the mods to see this issue and talk on it if it's an important issue or relevant issue .


Error in bash update

Windows 10 Pro 1703 (64bits)
Use wired network

Sorry, the problem was Bitdefender.
Problem solved


Not able to assign static IP for multiple instances on single provision request.

Hi @Fryguy,
We are trying to provision more than two instances, and I need to assign a static IP address to each. However, there was only one column for configuring the static IP address, subnet mask, and gateway.
Could you please guide me in resolving this issue?
For your reference, I shared the screenshot below.

I don't think this is possible through the UI at the moment.  Instead you'd have to write some automate to set the values.

HI @Fryguy ,
Can you guide me on writing some automation scripts for now?

I'm sorry, but I can't help with that directly in issues.  You can ask questions in the Discussions forum. Additionally, there is a book we have in addition to the documentation.

I'm sorry, but I can't help with that directly in issues.  You can ask questions in the Discussions forum. In addition to the regular documentation, there is also a book which helps guide writing the automation.


Submission Events and Timeline

We need to enumerate when things will happen for the submission and review.

Week0 - June 22nd
Monday June 22 - Receive (over email) information for github uploads, reach out if you do not.
Thursday June 25 - Notify SWG Chairs if you will not submit.
Friday June 26 - 1PM PDT must have completed the submission.
Friday June 26 -  1:30PM PDT Submitters receive email from SWG with our understand of results.
Week 0.5 - June 29th
Holiday Week
Week 1 - July 6
Monday - Begin drafting neutral press release
Tuesday - (4:30PM-5:30PM PDT) Review committee meeting, discuss objections
Thursday - (8:30AM-9:30AM PDT) Review committee meeting
Friday - 5PM PDT Objections due in github
Week 2 - July 13
Monday - 5PM PDT response to objections due: see document
Tuesday - (4:30PM-5:30PM PDT) Review committee meeting, fix easy to resolve issues and discuss  more difficult
Wednesday -  5PM PDT written statements on disputed issues not resolved Tuesday
Thursday - (8:30AM-9:30AM PDT) Review committee meeting, voting to resolve issues if necessary.
Week 3 - July 20
Tuesday - (4:30PM-5:30PM PDT) Review committee meeting
Wednesday - 5PM PDT Final code due
Thursday - (8:30AM-9:30AM PDT) Review committee meeting, final voting decisions taking into account code fixes, approval of press release.
Friday - 1PM PDT Final logs due, 1:30PM final results distributed by SWG
Week 4 - July 27
Monday - Press and analyst pre-briefings allowed under embargo, all briefings to include neutral press release; 1PM PDT draft of results page available for comment
Tuesday - 1PM PDT corrects to result page due, 5PM mock of results available
Wednesday - 10AM PDT Results and PR public, press embargo ends

NOTICE TO ALL SUBMITTERS:
DO NOT DISCUSS RESULTS WITH PUBLIC UNTIL JULY 29th 10AM PDT.
All results are private until July 29th 10AM PDT.

@bitfort @petermattson the line about results withdraw is missing. should be the Friday of week 3.

I think there is a principle: once the submission deadline is passed, no one should be surprised by anyone's resubmissions.
Resubmissions can happen for 2 reasons:

To address a compliance issue, apply a fix, then rerun
To steal hyper parameters

All such resubmissions should be declared in advance so everyone knows what to expect.
So there should be a deadline for declaring intent to resubmit a result with stolen HPs.  I would propose Week 3 Monday (July 20).

SWG:
Approve week 3 Monday declaration of borrowing w/ specific hps.
Make withdraw deadline more obvious.

Just want to add that today we identified a valid 3rd reason for resubmit:

In case the rules were slightly wrong/ambiguous, there are special cases where we will allow resubmission.  I think these are (exhaustive list):

RN50 label smoothing - if submitted with 0, allowed to resubmit with 0.1
RN50 can switch from norm to trunc norm or vice versa
Transformer eval using the wrong dataset, can resubmit with the corrected dataset (which should improve score)


Fix QEMU-based Linux wheel jobs

Summary
There appears to be an issue with memory consumption under the hood when using QEMU with Rust cargo:
https://github.com/rust-lang/cargo/issues/10583.
This is causing the CI wheel workflow to run out of memory and fail for QEMU-based jobs, e.g.:
https://github.com/Qiskit/qiskit-terra/actions/runs/3244845622/jobs/5322060193#step:6:1964
Due to this issue the original release of Terra 0.22.0 never actually produced wheels for Linux aarch64, ppc64le, and s390x. A separate branch has been created to manually retrigger the 0.22.0 release wheel jobs, which have now completed successfully (also used in validating the changes in this PR).
https://github.com/Qiskit/qiskit-terra/compare/0.22.0...Qiskit:qiskit-terra:dnm-non-x86-64-wheels-retry?expand=1
Details and comments
From the discussion on the linked issue, the workaround is to tell cargo to use the Git executable directly, rather than libgit2.
Edit:
As noted by @mtreinish below, setting CARGO_NET_GIT_FETCH_WITH_CLI=true in the pyproject.toml changes the behavior to use the Git executable for all Linux configurations, including x64.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
28 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.04%) to 84.378%

Files with Coverage Reduction
New Missed Lines
%

src/results/marginalization.rs
14
80.46%

src/sampled_exp_val.rs
14
63.93%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
61678

Relevant Lines:
73097

💛  - Coveralls


Merge pull request #2

Merge from original repo to update my forked repository.

pull request 2

Wut

closing this because i dont think you meant to open this


[WIP] Add Mint finance sensor

Description:
This sensor collect balace from you Bank accounts from Mint service (www.mint.com)
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#2102
Example entry for configuration.yaml (if applicable):
 - platform: mint_finance
    username: USERNAME
    password: PASSWORT
    thx_guid: sat687340ad587a8
    ius_session: 8765sfhfsdh8
    accounts:
      - id: 123456789
        name: Check account
        currency: CAD
      - id: 987654321
        name: Credit card
        currency: CAD

Checklist:

[x] Documentation added/updated in home-assistant.github.io
[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Need more tests

I think it's good now, but I would let few days to confirm the stability...

Interesting component!

Travis was not triggered :/

@fabaff I don't get why I got an error. Could you help me ?

@fabaff Ready for review

Bueno 🐬


How? We have debugged it offline and the problem was the network.

We have debugged it offline and the problem was the network.
Running the Prometheus container with "--network=host" solved the issue.
Originally posted by @simonpasquier in https://github.com/prometheus/prometheus/issues/4419#issuecomment-408451914

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Writing annotations with Duration

Hello,
I try to de-identify edf files with highlevel.anonymize_edf(), and for some files, I get an error - please see below.
As I understand, there are two steps in highlevel.anonymize_edf():

writing the signals.
writing the annotations.

The first step performs fine, so I can open the de-identified edf in EDFbrowser.
But not all annotations are saved correctly.
My guess is that it has something to do with the parameter "Duration" of the annotations.
When Duration is not zero, the script generates an error.
I am attaching a screenshot, which shows annotations in EDFbrowser for the original and de-identified edf files.
Thank you.

File "C:\Users\Vasily\Anaconda3\lib\site-packages\pyedflib\highlevel.py", line 718, in anonymize_edf
write_edf(new_file, signals, signal_headers, header, digital=True)
File "C:\Users\Vasily\Anaconda3\lib\site-packages\pyedflib\highlevel.py", line 461, in write_edf
f.writeAnnotation(*annotation)
File "C:\Users\Vasily\Anaconda3\lib\site-packages\pyedflib\edfwriter.py", line 734, in writeAnnotation
if duration_in_seconds >= 0:
TypeError: '>=' not supported between instances of 'bytes' and 'int'

Thanks for your report!
this seems related to https://github.com/holgern/pyedflib/pull/126
can you install the master version from the repo and check if it's already fixed in the current version (which isn't released to pypi yet)? pip install git+https://github.com/holgern/pyedflib.git

Thank you for a quick reply!
I just updated the package.
And now I get a different error, even for the edf files which did not generate the error in the past.
File "C:\Users\Vasily\Anaconda3\lib\site-packages\pyedflib\highlevel.py", line 516, in write_edf
f.setDatarecordDuration(int(100000 * block_size))
File "C:\Users\Vasily\Anaconda3\lib\site-packages\pyedflib\edfwriter.py", line 401, in setDatarecordDuration
self.update_header()
File "C:\Users\Vasily\Anaconda3\lib\site-packages\pyedflib\edfwriter.py", line 179, in update_header
set_gender(self.handle, gender2int(self.gender))
File "pyedflib_extensions_pyedflib.pyx", line 528, in pyedflib._extensions._pyedflib.set_gender
TypeError: an integer is required

Can you supply me with a sample file?

Sorry for the confusion.
Just restated the environment, and highlevel.anonymize_edf() works fine now.
Thank you!

great to hear! so it works fine now?


wget: fix & re-enable tests

Description of changes
Mostly needed some patchShebangs work, but also refactored it to use the in-built conditional mechanisms of doCheck.
CVE in the pipeline for wget, getting this straightened out in preparation in case we need to patch.
Also tested pkgsi686Linux, pkgsMusl, variants.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] 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.11 Release Notes (or backporting 23.11 and 24.05 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 331146 run on aarch64-darwin 1

10 packages marked as broken and skipped:

apx
archivebox
archivebox.dist
cloud-init
cloud-init.dist
imsprog
mindustry
mindustry-wayland
tlaps
zpool-auto-expand-partitions

2 packages failed to build:

eggnog-mapper
eggnog-mapper.dist

37 packages built:

akkoma
apt-mirror
bochs
bozohttpd
bozohttpd-minimal
cloud-utils
cloud-utils.guest
dasht
datalad
datalad.dist
git-annex (haskellPackages.git-annex)
git-annex.doc (haskellPackages.git-annex.doc)
gtk-pipe-viewer
gtk-pipe-viewer.devdoc
ipfetch
navi
nf-test
openai-whisper-cpp
openrussian-cli
pcsc-tools
pipe-viewer
pipe-viewer.devdoc
pleroma
python311Packages.git-annex-adapter
python311Packages.git-annex-adapter.dist
python311Packages.heudiconv
python311Packages.heudiconv.dist
python312Packages.git-annex-adapter
python312Packages.git-annex-adapter.dist
python312Packages.heudiconv
python312Packages.heudiconv.dist
vimPlugins.vim-dasht
wget
wgetpaste
yle-dl
yle-dl.dist
znapzend


docs: update 1.5 manual

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev-1.5@5bde60e). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             dev-1.5     #232   +/-   ##
==========================================
  Coverage           ?   14.04%
==========================================
  Files              ?      284
  Lines              ?    19268
  Branches           ?        0
==========================================
  Hits               ?     2706
  Misses             ?    16562
  Partials           ?        0

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 5bde60e...81fe4df. Read the comment docs.


Add Mess Hall Utilities Plugin

This is missing a LICENSE file.
Also, I can't help but think this would be better suited as an addition to raiyni's Menu Swapper Extended plugin instead of a standalone plugin.

Since this has outstanding required changes and hasn't been updated for a while, I am closing it as stale. You can reopen it once you make the required changes.


No longer compatible with React < 18.

Issue
The changes introduced in 3.4.0 to add support for the new useId hook in React 18 while maintaining backwards compatibility don't seem to be working as intended. At least not when using Webpack. In an app using Create React App v5 (Webpack v5) and React 17, the following compilation error occurs:
Failed to compile.

Attempted import error: 'useId' is not exported from 'react' (imported as 'e').
ERROR in ./node_modules/react-collapsed/dist/react-collapsed.esm.js 49:6-13
export 'useId' (imported as 'e') was not found in 'react' (possible exports: Children, Component, Fragment, Profiler, PureComponent, StrictMode, Suspense, __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED, cloneElement, createContext, createElement, createFactory, createRef, forwardRef, isValidElement, lazy, memo, useCallback, useContext, useDebugValue, useEffect, useImperativeHandle, useLayoutEffect, useMemo, useReducer, useRef, useState, version)

webpack compiled with 1 error
No issues found.

Other folks are having the same issue. If you look at the compiled code in the dist/ directory (react-collapsed.esm.js as downloaded from npm), it starts with import*as e from"react"; and later calls y=e.useId.
Discussion
I don't know yet why the workaround mentioned in the 3.4.0 pull request works for material-ui and not in this project. Perhaps there is a key difference in how the two projects are distributed.

Huh, that's a bummer that it didn't work. Considering that there seems to be another breaking change between v17 and v18, I think it's best to revert the useId change on v3, and make v3 compatible with React@<=17. And plan to release a v4 that's compatible with React@^18


Slack invite link expired.

Slack invite link here: https://www.kubeflow.org/docs/about/community/
has expired.

@rtkshahwar https://github.com/kubeflow/website/issues/3252#issuecomment-1132410100

@rtkshahwar  Let's keep the discussion in one issue, find a temporary invite link here: https://github.com/kubeflow/website/issues/3252#issuecomment-1132410100
/close


jameica: add darwin support

Motivation for this change
Followup of #44943.
Tried adding darwin support for fun, and turns out, it wasn't that hard :-)
Things done

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

[x] NixOS
[x] 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)
[x] Fits CONTRIBUTING.md.

cc @andir

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

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

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

Success on x86_64-linux (full log)
Attempted: jameica
Partial log (click to expand)
glibPreInstallPhase
installing
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/lr7ij2wslhhzd6jyj0xxrajfim0p4y0f-jameica-2.8.1-449
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/lr7ij2wslhhzd6jyj0xxrajfim0p4y0f-jameica-2.8.1-449/lib  /nix/store/lr7ij2wslhhzd6jyj0xxrajfim0p4y0f-jameica-2.8.1-449/libexec  /nix/store/lr7ij2wslhhzd6jyj0xxrajfim0p4y0f-jameica-2.8.1-449/bin
patching script interpreter paths in /nix/store/lr7ij2wslhhzd6jyj0xxrajfim0p4y0f-jameica-2.8.1-449
checking for references to /build in /nix/store/lr7ij2wslhhzd6jyj0xxrajfim0p4y0f-jameica-2.8.1-449...
/nix/store/lr7ij2wslhhzd6jyj0xxrajfim0p4y0f-jameica-2.8.1-449

Success on x86_64-darwin (full log)
Attempted: jameica
Partial log (click to expand)
sub_jar_sign:

BUILD SUCCESSFUL
Total time: 3 seconds
installing
post-installation fixup
strip is /nix/store/251fvx4mx9q4zarbca5cciasmn66p668-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/2by7qs6an0kyrsydhsln1vxm57bx2hk6-jameica-2.8.1-449/lib  /nix/store/2by7qs6an0kyrsydhsln1vxm57bx2hk6-jameica-2.8.1-449/libexec  /nix/store/2by7qs6an0kyrsydhsln1vxm57bx2hk6-jameica-2.8.1-449/bin
patching script interpreter paths in /nix/store/2by7qs6an0kyrsydhsln1vxm57bx2hk6-jameica-2.8.1-449
/nix/store/2by7qs6an0kyrsydhsln1vxm57bx2hk6-jameica-2.8.1-449


Added the ability for plot to put the cube data on the x axis if plotting just a cube against a vertical or y coordinate.

Closes #612.
Would need a what's new.

Looks like a genuine failure. Old test: https://scitools.github.io/test-iris-imagehash/images/f15f832a5ee0492a36e8b9ed8fa4f2b629dace4921681e17a9807e7c89835ef0.png
New test:

That is an altltude vs temperature plot, the new one is correct for the change you made.

Others are either altitude vs temperature or latitude vs temperature, both of which should be affected by this PR, so all looks good to me.

Probably a small amount of effort needed to bring this up to speed.

I've rebased and fixed the pep8 issue. I'm unlikely to get the time to do the image hashes soon unfortunately.

Closing as this has gone stale. Implementation is solid if somebody want to take it on and get it over the finishing line.


docker-registry: support more s3 storage options

/assign viglesiasce

/assign @viglesiasce

/assign @viglesiasce

Thanks @unguiculus, added.

/assign @unguiculus

/unassign

cc @jpds

Please fix conflicts and this should be good to go.

Well, I messed this one up and can't reopen. Please review #4030. Thanks.


METRON-2261 Isolate Curator Dependencies

As part of the HDP 3.1 upgrade, we need to upgrade to Curator 4.x.  There was a discuss thread  covering the need for this.
Currently, Curator is being pulled in as a transitive of Hadoop, HBase and Storm.  At the current version levels of these dependencies this does not cause a problem.  But when upgrading Hadoop, different conflicting versions of Curator will get pulled in.  We need to ensure that only a single version of Curator is pulled in after upgrading Hadoop.
This change is preparing us for follow-on PRs including...

Upgrade to Curator 4.x
Upgrade to Hadoop 3.1.1.

Changes

This change will maintain the current versions in use including Curator 2.7.1 and Zookeeper 3.4.6 (see note 1).

Defines Zookeeper as a direct dependency instead of relying on it being pulled in as a transitive of Curator (see note 2).

Defines Curator as a direct dependency instead of relying on it being pulled in as a transitive.

Introduces a separate ${global_curator_test_version} to allow the Curator test dependencies to differ from the main Curator version ${global_curator_version}.  This will be needed when upgrading to Curator 4.x (see note 3).

Notes

In upgrading to Curator 4.x, there is a breaking change that causes the MaasIntegrationTest to fail.  The fix for that plus the actual upgrade to Curator 4.x will be performed under a separate Jira to aid the review process.

When upgrading to Curator 4.x, we will need to continue to run against Zookeeper 3.4.6.   By default Curator 4.x will pull in Zookeeper 3.5.x.

To run with Curator 4.x and Zookeeper 3.4.6, we need to use the 2.12.0 version of curator-test based on this information provided by the Curator community.

Acceptance Testing
This PR should be tested using the centos7 development environment.

Start up the centos7 dev environment.cd metron-deployment/development/centos7
vagrant destroy -f
vagrant up

Basics
Ensure that we can continue to parse, enrich, and index telemetry.  Verify data is flowing through the system, from parsing to indexing

Open Ambari and navigate to the Metron service http://node1:8080/#/main/services/METRON/summary

Open the Alerts UI.  Verify alerts show up in the main UI - click the search icon (you may need to wait a moment for them to appear)

Go to the Alerts UI and ensure that an ever increasing number of telemetry from Bro, Snort, and YAF are visible by watching the total alert count increase over time.

Ensure that geoip enrichment is occurring.  The telemetry should contain fields like enrichments:geo:ip_src_addr:location_point.

Head back to Ambari and select the Kibana service http://node1:8080/#/main/services/KIBANA/summary

Open the Kibana dashboard via the "Metron UI" option in the quick links

Verify the dashboard is populating

Pull Request Checklist
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[ ] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

[x] Have you ensured that any documentation diagrams have been updated, along with their source files, using draw.io? See Metron Development Guidelines for instructions.

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

My initial reaction to keeping our existing versions would be that we'd just have a couple pom dep additions here and there, but it looks like quite a lot of excludes across a lot of modules. Is that all related to us having previously leveraged transitive deps for curator and zookeeper?

Is that all related to us having previously leveraged transitive deps for curator and zookeeper?

Yes

I spun up full dev and telemetry is flowing through as expected. I'm seeing geo enrichments working as well.

Looked over the recent run of commits. Based on my initial tests, this looks pretty good. When you're confident in the manual testing I'm +1 on this.

All smoke tests passed.  I updated the PR description with the tests I went through.

This has been merged into the feature branch.


Update installation.rst

Minimum version is Python 3.8

see https://github.com/asreview/asreview/pull/1457

Hi @Rensvandeschoot , that one was the about.rst and this one is in installation.rst (which shows up in the docs)


How to attach "what is being modelled (ion channel, brain region,..." to a model (emodel, ...)"

In the modelinstance schema, there is a path pointing to modelOf.
It is not clear what values are expected for the modelOf property. The relation between model and modelinstance is not clear as well.
@apdavison, @jdcourcol can you please comment on this.

I think modelOf should point to ontology terms, e.g.
{
    "path": "nsg:brainRegion",
    "name": "Brain region",
    "description": "Brain region",
    "node": "{{base}}/schemas/bbp/core/typedlabeledontologyterm/v0.1.0/shapes/BrainRegionOntologyTermShape",
    "minCount": 1
},

{
    "path": "nsg:cellType",
    "name": "Cell type",
    "description": "The cell type.",
    "node": "{{base}}/schemas/bbp/core/typedlabeledontologyterm/v0.1.0/shapes/CellTypeOntologyTermShape",
    "maxCount": 1
}

The idea of distinguishing model and modelinstance is to have an easy way to conceptually and practically group together different versions and parameterizations of the same model, something like how a Github repository groups together different versions of some piece of software.
"Model Project" would be a possible alternative for what I have been calling "Model" (i.e. it would meet the functional need, although the concepts are not quite the same and I prefer "Model")

Thank you @apdavison .
I agree for taking values of nsg:modelOf from controlled vocabularies. If we can know the current known list of 'modelOf' values that would be great.
It would be great if we could have examples of a model (with the attributes examples) and a model instance.


add support for nested console messages

this very important pr will allow adding prefixes for nested console.log messages e.g.:
 console.log("hey", console.log("hi"))

here you go ✨


add-permanent-notation to object

Deploy preview for js-primer ready!
Built with commit 06b21d27eb7d308ebc7e6ba1084e25ff9eceee4b
https://deploy-preview-398--js-primer.netlify.com


[Errno -3] Temporary failure in name resolution

While trying to get access token from api I getting the follwoing error,
ConnectionError: HTTPSConnectionPool(host='', port=443): Max retries exceeded with url: /oauth2/access_token/ (Caused by <class 'socket.gaierror'>: [Errno -3] Temporary failure in name resolution)
Is there any way I can resolve this error?
Thanks.

This error means that a DNS problem occurred and we couldn't resolve the IP address. If you see this problem you should simply retry the request.

Already I have tried that, but it's not working for me. Shall I try with IP address Or is there any other solution for this?

Can you print the URL you're trying to reach?

I tried the same API URL in postman, its working fine. But  in my request method, it throws the above error.

So this strongly suggests you have a problem with your OS or Python configuration, or possibly your URL, because it seems like the Python code cannot resolve the domain name.

I am getting the right URL from requests. Following warning error message is displayed while running the script. Is there any issues?
/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/util/ssl_.py:315: SNIMissingWarning: An HTTPS request has been made, but the SNI (Subject Name Indication) extension to TLS is not available on this platform. This may cause the server to present an incorrect TLS certificate, which can cause validation failures. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning.
SNIMissingWarning
/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/util/ssl_.py:120: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
InsecurePlatformWarning

Those error messages aren't going to be relevant, no. Can you set allow_redirects=False to see if you're being redirected away?

it's working now. problem from dns system

how did u resolve dns problem???

I face the same problem, and solved by restart computer

I fixed this by disabling my VPN

You have issues with your DNS. Check on your system that public interface is configured to use correct DNS servers.

For me, it was the firewall that was blocking the connection. I'm using ufw .. the following command fixed it for me.
sudo ufw allow out to any port 53
don't forget to reload ufw
sudo ufw reload


Custom headers for columns.

closes #5
Add new parameter aliases: with query: to print custom headers for ALL fields.
Every field header must be in aliases array.

Let's make an KV dict, so alias only work for specific columns?


Landing Page Enhancement

You can Put landing page main files, index.html/style.css/script.js outside every folder and put every other things related to landing page inside assets.
for reference: https://github.com/Astrodevil/Awesome-JavaScript-Projects/blob/main/script.js
make a script.js file like this for landing page so that any new contributors can easily link there projects to landing page.
You Can drop your suggestions too.. If you can make it more simple to add projects.
Others Things You can Enhance:

Add logo
Add some documentation
Make Landing Page more Attractive

Assign me. I'll work on it

Are you sure, You can make it more better? You can take your own time. No need to hurry.

@o0aman0o Are you working on this?

yeah please assign someone else....i am in too much burden right now.....sorry for inconvience

@Astrodevil

Yes I'm working on it but I need 1 or 2 days to complete cause I was busy past days. I will update here for sure.

@ArijitAcharjee411 ??


chore: add firebase setup for iOS

Changes

add firebases set-up file and upgrade the dependency things.

UI

No visible changes.

Notes

https://flutter.dev/docs/development/ios-project-migration

Important to look at this migration documentation when upgrading the flutter for iOS.

This PR related to issues https://github.com/IndustryBear/MentorApp/issues/18


Edit case-sensitive for Task Description and Employee Name in UG

Codecov Report

Merging #321 (f6cbe6e) into master (8a9f542) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #321   +/-   ##
=========================================
  Coverage     73.23%   73.23%
  Complexity      850      850
=========================================
  Files           107      107
  Lines          2690     2690
  Branches        325      325
=========================================
  Hits           1970     1970
  Misses          609      609
  Partials        111      111

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 8a9f542...f6cbe6e. Read the comment docs.


[enhancement] Create libra_channel module which provides an mpsc channel

Summary
libra_channel provides an mpsc channel which has two ends libra_channel::Receiver
and libra_channel::Sender similar to existing mpsc data structures. What makes it different from existing mpsc channels is that we have full control over how the internal queueing in the channel happens and how we schedule messages to be sent out from this channel.
Note that libra_channel does not provide an implementation of this internal queueing mechanism, but provides a trait (MessageQueue) which the consumers of this channel can use to create libra_channels' according to their needs.
Related issue #1323
Related RFC PR : #1462
Test Plan
Added unit tests

@dmitri-perelman : I have stripped away all the internal queueing details into a trait and made libra_channel agnostic of its implementation details.
I'll add an implementation of the MessageQueue trait which implements per-validator round robin, configurable FIFO/LIFO Queueing, per-validator message limit in a separate PR.

Added. If the waker is removed, this new test will never complete.

:hourglass: Testing commit c36eb91a1c6461a6918b136867de6431f29f06b8 with merge ae372ef70cf74d9470db762d00c649f808d5b3fe...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: dmitri-perelman
Pushing ae372ef70cf74d9470db762d00c649f808d5b3fe to master...

I know this was already landed, but i would highly suggest taking a look at the channel implementations in the futures crate for improving this channel type.

@bmwill our use case is kinda different from the API of the regular channels (e.g., we don't need to support asynchronous send), hence I hope we can make it simpler.
Do you think we could come up with the list of the specific channel improvements and understand their importance?
From your comments:

A receiver might be stuck in case all the senders are gone. This is kinda what I'd expect to do anyway, but we could add some logic that would help with the debugging of these cases.

dumb question: why is this mpsc? from what I read, it looks like a mpmc channel? is there anything prevents cloning the receiver?

I am updating it to be spsc.
The Sender and Reciever ends don't implement Clone trait, so we should not be able to create copies of either. The change is in this PR : https://github.com/libra/libra/pull/1490

Tried cloning the Receiver and I get a compile error
% cargo check
    Checking consensus v0.1.0 (/Users/kush/git/libra/consensus)
error[E0599]: no method named `clone` found for type `channel::libra_channel::Receiver<channel::message_queues::PerValidatorQueue<_>>` in the current scope
   --> consensus/src/chained_bft/network.rs:294:44
    |
294 |         let proposal_rx_copy = proposal_rx.clone();
    |                                            ^^^^^ method not found in `channel::libra_channel::Receiver<channel::message_queues::PerValidatorQueue<_>>`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0599`.
error: could not compile `consensus`.

sorry I mean technically adding the Clone would not introduce any problem given we have mutex?
mpmc would be a more general data structure, although consensus probably doesn't need it to be

Yeah, it can be made mpmc because of the Mutex, but we would also need to rewrite the drop implementations for Sender and Receiver which assume that there is only one Sender and Receiver present.

We would also need to keep multiple wakers for multi-receiver?

Yes, that is true.


chore: add axe checks in all e2e tests that didnt have them

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.97%. Comparing base (a73d395) to head (eb5ce28).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #214   +/-   ##
=======================================
  Coverage   90.97%   90.97%
=======================================
  Files         226      226
  Lines        3324     3324
  Branches      527      527
=======================================
  Hits         3024     3024
  Misses        295      295
  Partials        5        5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


configurable host param

This allows a developer to expose their livereload server to other devices on the network.

Thanks.


Add option to hide window on launch

I typically launch yubioath-desktop when my window manager starts. Something I've always wanted to have was for the application to launch to the system tray. I was able to hack something up that works, though there are some side-effects I don't understand (I'm not very savvy when it comes to Qt).
Out of curiosity, is this something that the team might be interested in? If so, I have questions! These changes work with the exception of a bug when leaving the settings dialog if both Show in system tray and Hide on launched options are unchecked - the main window disappears. I suspect this is due to the change I made to the visible property, however I'm not sure of how to make this dynamic. Ideally we would inhibit showing the main window immediately after launch rather than when the option is checked.
Thanks in advance!
Obligatory screenshot below.

(Apologies for resubmitting the PR, I accidentally created this off of the master branch)

Thank you; at a glance this seems like a feature we would happily accept, but I'm going to need some time to look at this and there's a lot going on right now. I'll get back to you as soon as I can.

Sounds good! I'll do some work in the meantime to see if I can better work
out what's going on and apply a fix. Thanks for taking a look!
On Mon, Oct 23, 2017 at 1:36 PM, Emil Lundberg<EMAIL_ADDRESS>wrote:

Thank you; at a glance this seems like a feature we would happily accept,
but I'm going to need some time to look at this and there's a lot going on
right now. I'll get back to you as soon as I can.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yubico/yubioath-desktop/pull/197#issuecomment-338755679,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFtLiIaj4jkVNiqAmK4p306X6jzbTl9bks5svNy6gaJpZM4QDLYb
.

@emlun I've just pushed an update. I believe this has the correct behavior. Please let me know if there is anything I can do to help!

Looks good, thanks a lot!

My pleasure. Thanks!


Expanding the introduction and adding a FAQ doc

Signed-off-by: Lalatendu Mohanty<EMAIL_ADDRESS>

Currently the we have a single line explaining the project i.e. API driven OpenShift 4 cluster provisioning and management. which does not give much context to folks who are new to the project. I do not think this PR does justice in explaining what is Hive does. The idea is to give enough context to folks so that they will feel confident about the basic nature of Hive.

/lgtm

/lgtm cancel
still have a couple of nits

/lgtm


POC for the build job

An early POC for the build job which uses current otcdocstheme (surely to be adapted).
Usage:

git checkout master
tox -e docs

Prerequisites:

python3
python3-tox

Status:

currently job fails, due to the content of the repo (lot of absurd content)
only AntiDDoS and AutoScaling covered
job considers everything under doc/source and enforces that everything there is included (referred) in some form in the document tree.

It is not intended to be merged immediately, but opens place for discussion.
After executing: tox -e docsyou will have some content generated under doc/build/html and the index.html is the proper entry point.
Publishing of htmls is out of scope here, we need to create a build job first

After we do this and merge it - this job can be executed automatically by Zuul and thus enforcing quaility of the commits (PRs)

recheck

recheck

Result can be checked on http://hc2.eco.tsi-dev.otc-service.com
Let's merge it and do the following fixes, since otherwise change is "unmanageable".

vpcep is now structured in perhaps the most suitable way, I guess it would make sense to adapt all other services to follow same file structuring to avoid a mess like in CCE


[Tensorflow] package tensorflow wheel and deploy a runtime docker

@qiuxiafei @Yancey1989 @linearhit
I think we need to discuss whether we need to maintain a separated runtime docker image for tensorflow blade or should install tensorflow-blade in our current tf runtime images.
Since currently the ci action for tensorflow-blade does not build disc for tensorflow, maybe a separated docker image is good for now.

Intuitively, deploy dockers should named in devices (cpu/cu100/cu110). Make individual images for tf/torch increase complexity. I suggest to merge building/testing/deploy scripts into the device-specified images. But we dont have to do it single trial.

better to add an issue to introduce the target, the CI system deployed tf_disc, torch_disc, tf_blade runtime Dockers, and these Docker would make user confusion.
Do we have other Dockers on the road? And do we need to update the tutorials with these Docker?

better to add an issue to introduce the target, the CI system deployed tf_disc, torch_disc, tf_blade runtime Dockers, and these Docker would make user confusion. Do we have other Dockers on the road? And do we need to update the tutorials with these Docker?

Tutorials are on the way, this pr will create dockers for tutorials.

Shall we create an issue to describe the current status of blade_tf and its future roadmap? We are planning to bi-monthly or quarterly publish a release note on ZhiHu etc. Some clear description would help the readers to understand these major feature updates.

Issue regarding tensorflow-blade has been added. As for the first stage open-source, we have merged the cpp and python code. This pr main focus on deploy dockers.


[8.x] Remove unnecessary double MAC for AEAD ciphers

This PR does two things:

It resolves a small bug in the generateKey method (wrong result for AES-128-GCM, see comment in code below).
It removes the unnecessary hmac computation for AEAD ciphers, improving performance and reducing encrypted data amount.

These two problems were introduced recently when support for GCM ciphers was added, in https://github.com/laravel/framework/pull/38190
GCM is a so called Authenticated Encryption scheme (AEAD), which means that a Message Authentication Code (mac) is included in the algorithm and handled by openssl_encrypt/openssl_decrypt. This is called tag in the code, and serves the exact same purpose as the mac that is also present from before (and necessary in the CBC-case, which is not natively authenticated).
This PR removes the HMAC computation in the GCM case, since the tag already ensures data integrity (before decryption is attempted). This simplifies the code (very important for cryptographic implementations), improves performance and reduces the amount of data stored/transmitted. As an example, the XSRF and session id cookies are reduced by around 25 % (but the reduction for larger data is of course much smaller).
The changes are backwards compatible with https://github.com/laravel/framework/pull/38190, and of course with any data stored before that. In fact, the only change to the decryption part is skipping the MAC validation in the case of an AEAD-cipher being used.
This was actually all done correctly in a 2017 PR by bukka, https://github.com/laravel/framework/pull/21963, but that was never merged. I did use that PR as inspiration for this, but I tried to keep the changes to a minimum to increase chances of merging. One larger change is generalizing the list of supported ciphers, and adding the AEAD support as a property. This should simplify the process of adding additional ciphers in the future, and simplified other parts of the code.

Removing the $tag parameter from the mac-functions is technically a breaking change for extending code (not for data, since tag is always the empty string for non-aead ciphers), however only in relation to the commit a few weeks back, and this was determined acceptable, see https://github.com/laravel/framework/pull/38475#discussion_r692897571

Could the following be a problem?
Since the MAC is no longer included in the AEAD case with this PR (the whole point), downgrading Laravel after using this will cause session invalidation (since the unnecessary mac check will fail). I'm not sure how you usually consider BC for revertability?

No I don't think that's a high risk.


[BUG] examples/dev/trace-analytics-sample-app data-prepper failed to receive trace data

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Go to 'examples/dev/trace-analytics-sample-app', run docker-compose up -d
Open localhost:8089 in the browser, Click on 'buttons'
Go to Trace Analytics Dashboard, wait for a few minutes
No data showed up

Expected behavior
verified that examples/trace-analytics-sample-app still works.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: MacOS
Version: 12.6.1

Additional context
Add any other context about the problem here.

This was still working for Data Prepper 2.0.1. I remember digging into the issue and narrowed down the offending change to #1335. My guess is that it's something to do with the OTEL java version change in that PR.
So I think this issue is also related: #1963


Make Node metadata available to WASM.

Make Node metadata available to WASM.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Streamlined decode url hotspot. formatting

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Support for 2.13?

I'm quite in love with this plugin. However, without support for 2.13, it's hard for me to just go ahead and enable it on all my projects. Consider this a feature request. :-)

The 0.3.0-M4 is cross-compiled for 2.13.0-M5 now and is available on maven central. In theory that should be also compatible with previous milestone releases of 2.13, tho you won't be able to use %% in libraryDependencies

For the record, it isn't compatible with M4. :-)


How do I pass params to my devise_invitable controller?

I have created an invitation controller that is quite vanilla:
class InvitationsController < Devise::InvitationsController
  def new
    binding.pry
  end
end

And I created a link that triggers that request like so:
<%= link_to "Invite<EMAIL_ADDRESS>new_user_invitation_path(email: @profile.email), class: "btn btn-xs btn-primary" %>

The issue I am having is that when I get thrown into pry in that action, it doesn't show me that email parameter.
> params
=> <ActionController::Parameters {"controller"=>"invitations", "action"=>"new"} permitted: false>

How can I send params with that InvitationsControler#New action?

Can you post log for that request?

@scambra Sure. Here you go:
Started GET "/users/invitation/new" for ::1 at 2016-10-26 11:58:45 -0500
Processing by InvitationsController#new as HTML
  User Load (5.2ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 ORDER BY "users"."id" ASC LIMIT $2  [["id", 2], ["LIMIT", 1]]
  Rendering devise/invitations/new.html.erb within layouts/application
  Rendered devise/invitations/new.html.erb within layouts/application (721.1ms)
Completed 500 Internal Server Error in 801ms (ActiveRecord: 5.2ms)

DEPRECATION WARNING: #original_exception is deprecated. Use #cause instead. (called from initialize at /.rvm/gems/ruby-2.3.1@myapp-rails-5.0/gems/better_errors-2.1.1/lib/better_errors/raised_exception.rb:7)
DEPRECATION WARNING: #original_exception is deprecated. Use #cause instead. (called from initialize at /.rvm/gems/ruby-2.3.1@myapp-rails-5.0/gems/better_errors-2.1.1/lib/better_errors/raised_exception.rb:8)

ArgumentError - First argument in form cannot contain nil or be empty:
  actionview (<IP_ADDRESS>) lib/action_view/helpers/form_helper.rb:438:in `form_for'
  simple_form (3.3.1) lib/simple_form/action_view_extensions/form_helper.rb:26:in `block in simple_form_for'
  simple_form (3.3.1) lib/simple_form/action_view_extensions/form_helper.rb:45:in `with_simple_form_field_error_proc'
  simple_form (3.3.1) lib/simple_form/action_view_extensions/form_helper.rb:25:in `simple_form_for'
  app/views/devise/invitations/new.html.erb:10:in `_app_views_devise_invitations_new_html_erb__4202866183412601869_70355747934720'
  actionview (<IP_ADDRESS>) lib/action_view/template.rb:158:in `block in render'
  activesupport (<IP_ADDRESS>) lib/active_support/notifications.rb:166:in `instrument'
  actionview (<IP_ADDRESS>) lib/action_view/template.rb:348:in `instrument'

For this operation, I removed the binding.pry from the InvitationsController#New for illustration purposes.

I don't see your request include email in url (no ?email= in URL and no parameters line below "processing by" line)
Check href in generated html, does it include email parameter?

You are right, it doesn't include the email parameter in the link. Here is the HTML generated by the above link_to:
Invite Jeremy Verley

Isn't that a function of the routes for new_user_invitation_path?
Because if I create another link, say this one:
<%= link_to "Invite<EMAIL_ADDRESS>invite_path(@profile), method: :post, data: { confirm: "Are you SURE you are ready to invite #{@profile.name}?"}, class: "btn btn-xs btn-primary" %>

Which corresponds to the following path:
post 'invite/:id', to: 'profiles#invite_user', as: :invite

Then in my ProfilesController, I create an invite_user action that looks like this:
  def invite_user
    User.invite!(email: @profile.email, first_name: @profile.first_name, last_name: @profile.last_name, avatar: @profile.avatar)
  end

When I click that link, I get the following log event:
Started POST "/invite/jeremy-verley-st-george-s-college" for ::1 at 2016-10-26 23:09:01 -0500
Processing by ProfilesController#invite_user as HTML
  Parameters: {"authenticity_token"=>"rqibD/bh5km9crZv5/UEOwKqf/lAcaB6CkddZWl9HmQ==", "id"=>"jeremy-verley-st-george-s-college"}
  Profile Load (2.0ms)  SELECT  "profiles".* FROM "profiles" WHERE "profiles"."slug" = $1 ORDER BY "profiles"."id" ASC LIMIT $2  [["slug", "jeremy-verley-st-george-s-college"], ["LIMIT", 1]]
  Grade Load (2.2ms)  SELECT "grades".* FROM "grades" WHERE "grades"."profile_id" = 21
  HABTM_Positions Load (1.9ms)  SELECT "positions_profiles".* FROM "positions_profiles" WHERE "positions_profiles"."profile_id" = 21
  Position Load (1.5ms)  SELECT "positions".* FROM "positions" WHERE "positions"."id" IN (2, 3, 4)
  Achievement Load (1.0ms)  SELECT "achievements".* FROM "achievements" WHERE "achievements"."profile_id" = 21
  Video Load (1.8ms)  SELECT "videos".* FROM "videos" WHERE "videos"."profile_id" = 21
  Transcript Load (2.0ms)  SELECT "transcripts".* FROM "transcripts" WHERE "transcripts"."profile_id" = 21
No template found for ProfilesController#invite_user, rendering head :no_content
Completed 204 No Content in 469ms (ActiveRecord: 12.4ms)

Notice the params are now correct, as is the POST request and everything else that follows.
So it seems to me that the issue is in the routing for the new_user_invitation_path.

new_user_invitation_path is a route for new action in InvitationsController, new action in restful always is GET request to display form, and that form will make a post request to create action. So probably you wanted post request with email, as you did with invite_path(@profile), method: :post


Update github-actions-cli.md

Replacing the --docker-file-path parameter with --context-path

@pauldotyu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@craigshoemaker
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
#assign:@MicrosoftDocs/public-repo-pr-review-team

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team

@craigshoemaker
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged" #assign:@MicrosoftDocs/public-repo-pr-review-team

I asked Paul to open this PR. I am happy to review. Thanks @pauldotyu

I tested changes. It works when you select scope "workflow" Update Github actions workflows, when creating github Token.
@pauldotyu can you please add this information on the PR?
and also making sure that you already have a container apps created before running this command.
@craigshoemaker adding this information we should be good to merge.

@pauldotyu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@pauldotyu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jorgearteiro - I've added the info on requiring the "workflow" scope in the GitHub PAT

Thanks, @pauldotyu!
#sign-off

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Roll Engine from 3d1fd0cbb76f to f9a6fb1a5c33 (1 revision)

https://github.com/flutter/engine/compare/3d1fd0cbb76f...f9a6fb1a5c33
2021-08-28<EMAIL_ADDRESS>[web] Support raw straight RGBA format in Image.toByteData() in CanvasKit (flutter/engine#28339)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


http PUT can only upload 2GB max file

http PUT can only upload 2GB max file
when the file size is lager than 2GB :
http: error: Request timed out (30s).

can you put the exact command you running and the complete error message...Thanks

From: ambujmehra
Date: 2016-04-07 21:31
To: jkbrzt/httpie
CC: Jim Raynor
Subject: Re: [jkbrzt/httpie] http PUT can only upload 2GB max file (#460)
can you put the exact command you running and the complete error message...Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

------------------  success ------------------
root@ubuntu:/home/zxc# http  PUT "http://<IP_ADDRESS>:888/v1/admin/bucket-1/test" < /tmp/4M
HTTP/1.1 201 Created
Connection: keep-alive
Content-Type: text/plain;
Date: Fri, 08 Apr 2016 17:13:15 GMT
Server: openresty/<IP_ADDRESS>
Transfer-Encoding: chunked
------------------ error ------------------
root@ubuntu:/home/zxc# http --debug PUT "http://<IP_ADDRESS>:888/v1/admin/bucket-1/test" < /tmp/2.3GB
HTTPie 0.8.0
HTTPie data: /root/.httpie
Requests 2.2.1
Pygments 1.6
Python 2.7.6 (default, Mar 22 2014, 22:59:56)
[GCC 4.8.2] linux2
hang forever
Ctrl + C
^CTraceback (most recent call last):
File "/usr/bin/http", line 9, in
load_entry_point('httpie==0.8.0', 'console_scripts', 'http')()
File "/usr/lib/python2.7/dist-packages/httpie/core.py", line 95, in main
response = get_response(args, config_dir=env.config.directory)
File "/usr/lib/python2.7/dist-packages/httpie/client.py", line 24, in get_response
% pformat(requests_kwargs))
File "/usr/lib/python2.7/pprint.py", line 63, in pformat
return PrettyPrinter(indent=indent, width=width, depth=depth).pformat(object)
File "/usr/lib/python2.7/pprint.py", line 122, in pformat
self._format(object, sio, 0, 0, {}, 0)
File "/usr/lib/python2.7/pprint.py", line 140, in _format
rep = self._repr(object, context, level - 1)
File "/usr/lib/python2.7/pprint.py", line 226, in _repr
self._depth, level)
File "/usr/lib/python2.7/pprint.py", line 238, in format
return _safe_repr(object, context, maxlevels, level)
File "/usr/lib/python2.7/pprint.py", line 282, in _safe_repr
vrepr, vreadable, vrecur = saferepr(v, context, maxlevels, level)
File "/usr/lib/python2.7/pprint.py", line 261, in _safe_repr
write(qget(char, repr(char)[1:-1]))
KeyboardInterrupt

Thanks for the report. I'm currently working on support for streamed uploads so that files of any files could be uploaded.

Please follow #461 for progress.

http PUT can only upload 2GB max file
when the file size is lager than 2GB :
http: error: Request timed out (30s).


Getting error trying to check site

Hi,
I'm trying to track down a problem with one of my sites.   I used the online service and was getting a fail for my site demo.netbi.net.au.
In the browser console I get the message -
Command failed: depth=2 C = US, O = GeoTrust Inc., CN = GeoTrust Global CA
verify error:num=20:unable to get local issuer certificate
verify return:0
139950555604640:error:140790E5:SSL routines:SSL23_WRITE:ssl handshake failure:s23_lib.c:177:
I downloaded shaaaaaaaaaaaaa and tried running the command line version, to see what was going on.
I get this error -
hermes:shaaaaaaaaaaaaa-master rme$ ./bin/shaaaaaaaaaaaaa demo.netbi.net.au
/Users/rme/sha/shaaaaaaaaaaaaa-master/node_modules/x509/index.js:10
var ret = x509.parseCert(path);
^
TypeError: Certificate must be a string.
at TypeError (native)
at Object.exports.parseCert (/Users/rme/sha/shaaaaaaaaaaaaa-master/node_modules/x509/index.js:10:18)
at Object.Shaaa.cert (/Users/rme/sha/shaaaaaaaaaaaaa-master/shaaaaa.js:127:21)
at /Users/rme/sha/shaaaaaaaaaaaaa-master/shaaaaa.js:171:25
at /Users/rme/sha/shaaaaaaaaaaaaa-master/shaaaaa.js:115:7
at ChildProcess.exithandler (child_process.js:735:7)
at ChildProcess.emit (events.js:110:17)
at maybeClose (child_process.js:1008:16)
at Socket. (child_process.js:1176:11)
at Socket.emit (events.js:107:17)
Has anyone seen this before?
I have another site which works fine - apps.netbi.net.au.  It's on a different server, but running the same Geotrust SHA-2 wildcard certificate.
I've come to this site due to a problem I'm having with Google Chrome, where I am unable to access demo.netbi.net.au since the last Chrome update.  I'm thinking it has something to do with my SHA-2 certificate setup, but can't figure out what.
Any help would be greatly appreciated.

I can’t access apps.netbi.net.au at all. Even a ping times out. I can ping demo.netbi.net.au but my browser times out.

Thanks, but ping is disabled on apps.netbi.net.au.
I get NICE doing this - https://shaaaaaaaaaaaaa.com/check/apps.netbi.net.au
I get ARGH when doing https://shaaaaaaaaaaaaa.com/check/demo.netbi.net.au.

I'm not sure, but https://www.ssllabs.com/ssltest/analyze.html?d=demo.netbi.net.au has a pretty crazy set of configuration errors for you to work out.
It looks like my code is having trouble downloading your cert. It's difficult for me to debug, especially since the configuration seems so strange. Mind re-opening after you've narrowed the problem down a bit?

For reference this is what OpenSSL says when I try to connect:
➜  ~  brewssl s_client -connect demo.netbi.net.au:443 -servername demo.netbi.net.au -CAfile /usr/local/etc/openssl/cert.pem
CONNECTED(00000003)
depth=3 C = US, O = Equifax, OU = Equifax Secure Certificate Authority
verify return:1
depth=2 C = US, O = GeoTrust Inc., CN = GeoTrust Global CA
verify return:1
depth=1 C = US, O = GeoTrust Inc., CN = RapidSSL SHA256 CA - G3
verify return:1
depth=0 OU = GT38651010, OU = See www.rapidssl.com/resources/cps (c)14, OU = Domain Control Validated - RapidSSL(R), CN = *.netbi.net.au
verify return:1
140735164146512:error:140790E5:SSL routines:ssl23_write:ssl handshake failure:s23_lib.c:177:
---
Certificate chain
 0 s:/OU=GT38651010/OU=See www.rapidssl.com/resources/cps (c)14/OU=Domain Control Validated - RapidSSL(R)/CN=*.netbi.net.au
   i:/C=US/O=GeoTrust Inc./CN=RapidSSL SHA256 CA - G3
 1 s:/C=US/O=GeoTrust Inc./CN=RapidSSL SHA256 CA - G3
   i:/C=US/O=GeoTrust Inc./CN=GeoTrust Global CA
 2 s:/C=US/O=GeoTrust Inc./CN=GeoTrust Global CA
   i:/C=US/O=Equifax/OU=Equifax Secure Certificate Authority
---
Server certificate
-----BEGIN CERTIFICATE-----
MIIEsDCCA5igAwIBAgICBUUwDQYJKoZIhvcNAQELBQAwRzELMAkGA1UEBhMCVVMx
FjAUBgNVBAoTDUdlb1RydXN0IEluYy4xIDAeBgNVBAMTF1JhcGlkU1NMIFNIQTI1
NiBDQSAtIEczMB4XDTE0MDkxNzA4NDkwMVoXDTE2MDgwMjA2MTYxNFowgZIxEzAR
BgNVBAsTCkdUMzg2NTEwMTAxMTAvBgNVBAsTKFNlZSB3d3cucmFwaWRzc2wuY29t
L3Jlc291cmNlcy9jcHMgKGMpMTQxLzAtBgNVBAsTJkRvbWFpbiBDb250cm9sIFZh
bGlkYXRlZCAtIFJhcGlkU1NMKFIpMRcwFQYDVQQDDA4qLm5ldGJpLm5ldC5hdTCC
ASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALNEmMZszYz7PSTHD7/e/CcX
UFefwFf3yGe272RDjWcq/xE7Uc2cHIDZKyt8cV6rifJMHYV2te+WrSCqEQTJN//A
hjMFOjyLuZhyXO8AhAjmqn4HHiBAyMQldzYD4eZLq3ifLHrFuUKLoNIOZ9VBmCav
4Ni7AfmNLkK2lCbaQuBdZL/XmtF0JXxxbzC1kWgQP+Dix+ZcJcF0hQr9FbpYuxhF
rRzsTdMP+jnkjWkwQ6ryWSk8Jx+l9dg8vsFfau1Eqf7yLcnE5EeeuxuxNCj4PRXs
FlkLJDltjR3SUoLeS34FGRTTosegYRCiHOT94XmhMYkIw/SrqCT44s88uWWOTe0C
AwEAAaOCAVgwggFUMB8GA1UdIwQYMBaAFMOc8/zTRgg0u85Gf6B8W/PiCMtZMFcG
CCsGAQUFBwEBBEswSTAfBggrBgEFBQcwAYYTaHR0cDovL2d2LnN5bWNkLmNvbTAm
BggrBgEFBQcwAoYaaHR0cDovL2d2LnN5bWNiLmNvbS9ndi5jcnQwDgYDVR0PAQH/
BAQDAgWgMB0GA1UdJQQWMBQGCCsGAQUFBwMBBggrBgEFBQcDAjAnBgNVHREEIDAe
gg4qLm5ldGJpLm5ldC5hdYIMbmV0YmkubmV0LmF1MCsGA1UdHwQkMCIwIKAeoByG
Gmh0dHA6Ly9ndi5zeW1jYi5jb20vZ3YuY3JsMAwGA1UdEwEB/wQCMAAwRQYDVR0g
BD4wPDA6BgpghkgBhvhFAQc2MCwwKgYIKwYBBQUHAgEWHmh0dHBzOi8vd3d3LnJh
cGlkc3NsLmNvbS9sZWdhbDANBgkqhkiG9w0BAQsFAAOCAQEAICYeaSmuREWalLAq
m3UhkzWTxj6Rmv/lQSjd/w34qMWThAe6d9RzduikJ6ECDHvJJqV7w8bMvWOibrpN
QP92Kfr8972d5Q+22kg9kt39n68oufqIOB51FUUYCSNk7bGOSKJyGlQhfgjb+qoM
1dXOfpePcf29MUGBu2n/0hTRY9E1zMpJNBaKQwNHbGslHYZq9axoqOFVGX0NL3CP
MqFDyGOGG72tJnOwtA/4lJ8DvxrfEV2azdafNncsHI6/0w9oCbdbI8uW9sJBGYY6
R5BIMaZAc7p3Gdf8I24u9au+QTLxrAocrJteTsooNEN+BIRPD1tOIQ0GxBNqyefZ
quZxlw==
-----END CERTIFICATE-----
subject=/OU=GT38651010/OU=See www.rapidssl.com/resources/cps (c)14/OU=Domain Control Validated - RapidSSL(R)/CN=*.netbi.net.au
issuer=/C=US/O=GeoTrust Inc./CN=RapidSSL SHA256 CA - G3
---
No client certificate CA names sent
Server Temp Key: DH, 1024 bits
---
SSL handshake has read 3812 bytes and written 198 bytes
---
New, TLSv1/SSLv3, Cipher is DHE-RSA-AES256-SHA
Server public key is 2048 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1
    Cipher    : DHE-RSA-AES256-SHA
    Session-ID: E1CC14B4082900541AAB27C3556E144BEA444DDD2EB73C550C60971D6974AF5F
    Session-ID-ctx:
    Master-Key: 0D6FE09C87B90A8AB9740DC6D493D323B04092DE2C19B32B841DD0535B0E2F4D3188159D81C4F90BF024B8966BDFA2FA
    Key-Arg   : None
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 300 (sec)
    Verify return code: 0 (ok)
---


How to set default values for a drop down when the element is not visible yet?

Example:
2 elements
element_1
element_2
element_2 is only visible when element_1 is set to "show" using
const select = elements.find((element) => element.id === 'element_1');
if (select.value.includes('show')) {
      return true
}

How to set default value of element_2? As the element is not visible initially the initial request won't work.
Any pointers?

Hi, @ashish-y.
I created two dropdowns as example
1st:

2nd:

and add to initial request some logic

  if (response && response.ok) {
  elements.forEach(element => {
    if (element.id === "DropDown1") {
      element.value = element.options[2].value
    }

    if (element.id === "DropDown2") {
      element.value = element.options[1].value
    }
  }

  )
}

After that, when I change the value in 1 dropdown “show”, I see the pre-selected value on the second one

I guess the solution with еhe initial request works.
I think this is what will help you with your issue. Let us know if you have any questions
thanks