Restructure folders, deprecate some grid classes, update readme

What's this PR do?

Adds a 7-layouts folder to the mix
Fixes some inconsistent naming and removes "Experimental" designation
Adds markdown parser to CSS comment descriptions
Breaks typography classes into individual files, (but keeps the helpers together)
Adds a new layout to the homepage of of our docs site.
grid_container=> l-container
grid_separator => Already accounted for with has-<size>-btm-marg helper
Brain dump in .README

Why are we doing this? How does it help us?
Breaking up layouts and utilities allows us to better visually parse the classes of our toolbox. As we start to commit this library to memory, I think the breakdown creates a better connection as to what lives where and the helpers at our disposal.
The other changes were just general housekeeping
How should this be manually tested?
Spin it up locally and make sure it works. There's a lot to take in on this one so overall, just ensure the organization makes sense, nothing looks wonky on the docs site, and all the naming feels right.
There's also a preview link here.
Are there any smells or added technical debt to note?
Yes. We'll have to combine this with some class name and<EMAIL_ADDRESS>file name changes where we've used the new system already. That includes templates for the donor-wall, sponsor-wall, and podcasts. Also the UMP CSS might need some path/naming tweaks.
TODOs / next steps:

[x] Sort out these changes with where their instances on prod /texastribune This is ready on a branch, but I don't want to cause any more conflicts for need for speed. Letting this whole reorg wait until after that's in.

Love this! I say :shipit: 🚢. A few notes, including a few things from my ongoing "ask about after UMP" list that I figure I'll note now while you're refactoring:

It feels like we're on the brink of a layout revolution in texastribune. I'm totally, completely in favor of killing those dastardly grid_ classes. Before we go whole-hog on updating layout, though, let's have a sit-down and strategize about our grid future.
Random, but I'd argue this belongs in the components, not typography, folder
This is minor and almost certainly doesn't matter, but I notice we have a lil' bit of inconsistency with how we're doing BEM. With this notch class, you don't always have to include a modifier alongside the base because the base has margin-bottom, height and width in it. Including a modifier simply overrides those. Whereas with this loader, the base class is useless on its own and you have to include a modifier. Curious whether you have opinions on either strategy.
During UMP styling, I found myself adding one of these classes to pretty much all p tags in the app. I wonder whether the base p selector should have a line-height baked into it.

Phew.

Grid things: You say you want a revolution, well ya know...(had to) Right, I think a grid strategy as a byproduct of us meeting to chat is def needed. I feel like some simple containers and the margin helpers you already built are pretty straight forward, but I'm on-board with no further tweaks from this point forward until we lay out more groundwork

Headline list: Too funny, same minds. So for that I wanted to demo how that's just some combination of helpers and that it can be removed. Not sure why those bullets are so high and keep in mind those l-list classes are now c-list so we're in agreement in every way for that one.

BEMming with out a base: My take is has-notch more closely follows the BEM philosophy while c-loading steps out of bounds a bit. To a modifier is just that, a modification of the original, but technically c-loading can't stand alone as an original.

Fighting line-heights: Yeah line-heights are weird and that would make sense to me to make an opinion about how p tags should render. We can always try it on for size and see how it is in the wild.


os.MkdirAll(path) doesn't return nil when the path already exists on windows

Right here https://github.com/natefinch/lumberjack/blob/4b74a4dba98bba3444cce9a66dab2dd70d1c0639/lumberjack.go#L209, if the path already exists, the docs https://golang.org/pkg/os/#MkdirAll of os.MkdirAll says it expects to return nil.
However, I found that this is not the case on window 10/golang 1.12.7.
Wondering if we can have a defensive check here.

It turned out not to be true if i ran os.MkdirAll on powershell.
Mostly, the user permission issue on my setup.


Hide documentation of the Unpin implementation for !Unpin

Hide the documentation of the Unpin implementation that guarantees !Unpin to avoid confusion.
Before:

After:

Published in 1.1.3.


Refactor integration tests

After integration tests silently failed, I decided to figure out why, and during that process, I made some refactoring:

More human-readable integration tests code.
Use promises instead of callbacks: bulletproof execution flow and readability.
Show test output (currently output is hidden, it hides errors).
Fix STDIO-error (the actual cause of the problem).

/cc @markelog

Show test output (currently output is hidden, it hides errors).
Fix STDIO-error (the actual cause of the problem).

Those are fixed in 2.0 branch :)

Use promises instead of callbacks: bulletproof execution flow and readability.

Well, promise().then(fn) is basically the same as read(..., fn) if there was something like .then().then() i would all for it, otherwise i don't see the point. Besides there is still code that uses callbacks which makes the code heterogeneous, i.e. uses two different concepts.

@markelog Just see the code. I could not understand anything in the previous version — it was not properly abstracted — process routines messed up with control flow. With all respect, I was totally confused when I was trying to fix my build.
With this code you have structure and abstraction.

Those are fixed in 2.0 branch :)

Why not port this to 1.0 and fix PRs? Where can I read about 2.0 branch?

@markelog Just check out the code. I could not understand anything in the previous version — it was not properly abstracted — process routines messed up with control flow. With all respect, I was totally confused when I was trying to fix my build.

With this code you have structure and abstraction.

This is subjective, matter of taste even. And that is okay, we all see code that we don't like and have different opinions on.
For example i don't see how this code is better - additinal abstraction like promise
don't give me anything, except as what it is - an abstraction, which could be considered as needless
abstraction altogether since there is no advantages to use promises, at least, i don't see them, since promises usually dealing with different kind of code flow.
But vowfs and vow are not "popular" packages, without any real community, API that you have to know, know common issues too and bunch of other things.
Those might be obvious to you, but for anyone else this is a blackbox. Whereas i operated with standart API.
Yes, you introduced couple functions and couple constants, but that could be as seen as what i described above -  additional needless abstraction, like passthroughProgress helper, it just proxies to stdout does nothing else, why it need to be there? Because you might switch to console.log there? Or because it just feels right? Or both? You might see it as "proper abstraction" someone else could see this is "needless abstraction".
I mean phrase like

it was not properly abstracted — process routines messed up with control flow

is subjective and not productive, i can say the same thing about these changes, and mine phrase would mean nothing to you, just like yours statement means nothing to me. If you say something like that - prove it, or better yet, just lay the evidences, not the conclusion.
I'm not gonna call your code as "bad" or something else negative, no, but i'd propose you would give me the same courtesy and operate only on objective terms - expandability, cyclomatic complexity, modularity, etc.

Why not port this to 1.0 and fix PRs? Where can I read about 2.0 branch?

There is only roadmap. I'd propose landing only iterative changes for the 1.x, like new rules, bugfixes, etc. i.e. changes that wouldn't affect merging process with 2.0, everything else - 2.x that is my view though, didn't yet shared it with anyone else.

is subjective and not productive, i can say the same thing about these changes, and mine phrase would mean nothing to you, just like yours statement means nothing to me. If you say something like that - prove it, or better yet, just lay the evidences, not the conclusion.

How is it subjective?
Look at this code:
        // Execute jscs -x on jscs source code for the given presets
        var child = spawn('node', args, { timeout: Infinity });

        process.stdout.write(messages.start + '\n\n');
        process.stdout.write(chalk.magenta(' - ') + messages.fixStart + '\n');

        // For some reason this makes subprocess to be more effective
        child.stdout.on('data', function() {});

        // Wait until autofix process is done
        child.on('close', function executeTests(code) {

You have low-level routines messed (controlling child processes) up with control routines and user interface (console).
And look at this:
        console.log('\nPreset "' + chalk.green(presetName) + '"');
        logStep('Autofix execution');
        return applyPreset(presetName).then(
            function() {
                logStep('Unit tests');
                console.log('');
                return runTests().then(
                    function() {
                        console.log('');
                        logStep('Unit test on "' + chalk.green(presetName) + '" preset are finished');
                    },
                    logErrorHandler('Unit tests failure'),
                    passthroughProgress
                );
            },
            logErrorHandler('Autofix failure')
        );

As you can see, this is pure high-level part: only control flow and user interface. All low-level code is encapsulated into functions with names which are abstracted over implementation.
So I conclude that this is not the matter of tastes.

You have low-level routines (controlling child processes) messed up with control routines and user interface (console).

I don't see who they "messed up" with it, but i see dramatic increase of cyclomatic complexity, which is has direct correlation with code perception.
Encapsulation by itself is not always great thing, if you put something in function it doesn't mean it is became encapluslated. "high-level" code is not always great too, especially, with lack of definition of it and context dependency.
Also this kind be interpreted as half way still, since constructions like this -
console.log('');
logStep(...);

or console.log('\n' + chalk.green('> ') + 'Autofix ingeration tests');, while in logStep you "encapsulated" console.log(chalk.magenta(' - ') + step);.
Do not strike me as properly encapsulated.
Those things are needed only if you need them, while you increased the complexity, level of encapsulation, added new interfaces, new APIs, for what?
You had the problem - you didn't see error output of failed tests, in 2.0 branch you can now do that. Whereas here, you don't introduce new features or improvements, you exercising your level of "rightness", you refactor because of the refactoring, uroborus of some sort.
This discussion could go on for the long time, we will start to define what is a "low-level" and why cyclomatic complexity not the right metric and so on.
That would be controproductive. You want to make this "right", fine, i'm not oppose to it, i don't see the point, i don't see who this would be better or why you paradigms are better then mine. But fine, this is your call anyhow.
There is only one admonition from me, i would prefer if you don't use promises.

I don't understand what are we discussing here. In the initial version you cannot understand what is going on reading the code. We had several process instances, console writes all over around. In the new version you can easily understand the flow just reading the code from top to bottom.
Again, it took me 1 hour to debug this file. Mainly because I could not understand the flow and what's going on. That's the only reason I rewrote this file.

There is only one admonition from me - i would prefer if you don't use promises.

I don't care about "promises vs callbacks". We already use promises in JSCS, so consistency is more important than preferences.

Coverage remained the same at 99.17% when pulling 6142e4f2f41a1aaa1d166c872c5f7ce87888bb7a on mdevils.integration-tests into f963cd496d4e5d89bb8ffee9abc1c8114b69f41f on master.

Coverage remained the same at 99.17% when pulling 6142e4f2f41a1aaa1d166c872c5f7ce87888bb7a on mdevils.integration-tests into f963cd496d4e5d89bb8ffee9abc1c8114b69f41f on master.

We finished out discussion in IM.


Changed deprecated method

I was recieving annoyng deprecation warnings while running my tests, so I decided to fork and update the method

Hello! Can you rebase that against master?

Apparently, we can't just change because we support some old versions as well.
You probably need to check if the object responds to the method before calling it.
Can you do that?
The CI is showing the error.

I think this pull request is unnecessary
I've tested this with 3 versions of rails,  5.2.3, 5.2.2 and 5.2.1, and they didn't produce any warning about it.
I guess perhaps this happened due to of this issue on Rails.
This method "will_change!" in ActiveRecord::Dirty was showing a deprecation warning because of its call "attribute_changed?" inside of the method, and now it seems to be fixed
Then I think we can close this pull request.

Thanks!


Line Numbers Overlapping

Every 7th line number ends up underneath every 6th line number. Playing around right now with the theme's .less files to see if I can fix the issue.
To be sure it wasn't the Syntax theme, I did try switching the UI theme. The issue was still there though. Also, I listed the current version of Atom I'm using, but I only just updated to that version after installing this theme. Previous version was 1.2.1.
Atom version: 1.2.2
Theme version: 0.5.0
OS: Windows 10
Also, after looking at your repo, I see changes you already made to fix this (b02b563). I can confirm they work.

Published new version.

Sorry I never replied to this. Issue is fixed. Thanks.


Debugging | Running F5 errors out

Repro steps:

Click on F5
See the below error under task -clean
1>C:\Program Files\dotnet\sdk\3.1.101\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(234,5): error MSB4018: The "ResolvePackageAssets" task failed unexpectedly. [C:\Users\xxxxx\VSProjects\TestAzureFunc\TestAzureFunc\TestAzureFunc.csproj]

Action: azureFunctions.pickProcess
Error type: Error
Error Message: Failed to start debugging. Task "clean" failed with exit code "1".
Version: 0.20.2
OS: win32
Product: Visual Studio Code - Insiders
Product Version: 1.43.0-insider
Language: en

Call Stack
extension.bundle.js:353:86654extension.bundle.js:353:86654
l.fire extensionHostProcess.js:48:845
y.$onDidEndTaskProcess extensionHostProcess.js:758:693

This "intractable" problem was solved by deleting bin and obj folders 😒. Revisited C#/.Net after 10 years and somethings just don't change


Downloads fail on Windows due to https support in libcurl

Our download tests are failing on Jenkins (Windows) with this message:
libcurl: (1) Protocol "https" not supported or disabled in libcurl
I didn't dig too deep and will open a PR disabling the tests for now.

This was fixed with this two PRs

https://github.com/ignition-tooling/release-tools/pull/434
https://github.com/ignition-tooling/release-tools/pull/430


High CPU usage when JDBC driver not found

Almost similar with #230, but In this case symptom is high CPU usage(smashing InspectContainerCmdExec).
2019-04-10 16:30:12.006 DEBUG   --- [     ducttape-0] c.g.d.core.exec.InspectContainerCmdExec  : GET: OkHttpWebTarget(okHttpClient=org.testcontainers.shaded.okhttp3.OkHttpClient@53d102a2, baseUrl=http://docker.socket/, path=[/containers/f949e1eaf71732d083b7ac8751210643c379ec434f77fde806518aeda2eb8a35/json], queryParams={})
2019-04-10 16:30:12.010 TRACE   --- [     ducttape-0] o.r.ducttape.unreliables.Unreliables     : Retrying lambda call on attempt 1159
2019-04-10 16:30:12.010 DEBUG   --- [     ducttape-0] c.g.d.core.command.AbstrDockerCmd        : Cmd: f949e1eaf71732d083b7ac8751210643c379ec434f77fde806518aeda2eb8a35,false,com.github.dockerjava.core.exec.InspectContainerCmdExec@2cd79a32
2019-04-10 16:30:12.010 DEBUG   --- [     ducttape-0] c.g.d.core.exec.InspectContainerCmdExec  : GET: OkHttpWebTarget(okHttpClient=org.testcontainers.shaded.okhttp3.OkHttpClient@53d102a2, baseUrl=http://docker.socket/, path=[/containers/f949e1eaf71732d083b7ac8751210643c379ec434f77fde806518aeda2eb8a35/json], queryParams={})
2019-04-10 16:30:12.014 TRACE   --- [     ducttape-0] o.r.ducttape.unreliables.Unreliables     : Retrying lambda call on attempt 1160
2019-04-10 16:30:12.014 DEBUG   --- [     ducttape-0] c.g.d.core.command.AbstrDockerCmd        : Cmd: f949e1eaf71732d083b7ac8751210643c379ec434f77fde806518aeda2eb8a35,false,com.github.dockerjava.core.exec.InspectContainerCmdExec@598e1db8
2019-04-10 16:30:12.015 DEBUG   --- [     ducttape-0] c.g.d.core.exec.InspectContainerCmdExec  : GET: OkHttpWebTarget(okHttpClient=org.testcontainers.shaded.okhttp3.OkHttpClient@53d102a2, baseUrl=http://docker.socket/, path=[/containers/f949e1eaf71732d083b7ac8751210643c379ec434f77fde806518aeda2eb8a35/json], queryParams={})
2019-04-10 16:30:12.019 TRACE   --- [     ducttape-0] o.r.ducttape.unreliables.Unreliables     : Retrying lambda call on attempt 1161
2019-04-10 16:30:12.019 DEBUG   --- [     ducttape-0] c.g.d.core.command.AbstrDockerCmd        : Cmd: f949e1eaf71732d083b7ac8751210643c379ec434f77fde806518aeda2eb8a35,false,com.github.dockerjava.core.exec.InspectContainerCmdExec@53f76db3
2019-04-10 16:30:12.019 DEBUG   --- [     ducttape-0] c.g.d.core.exec.InspectContainerCmdExec  : GET: OkHttpWebTarget(okHttpClient=org.testcontainers.shaded.okhttp3.OkHttpClient@53d102a2, baseUrl=http://docker.socket/, path=[/containers/f949e1eaf71732d083b7ac8751210643c379ec434f77fde806518aeda2eb8a35/json], queryParams={})

Steps to reproduce

make project WITHOUT 'com.mysql.jdbc.Driver'
use MySQLContainer
run

Symptom

CPU usage is up to 30~40%
o.r.ducttape.unreliables.Unreliables     : Retrying lambda call on attempt 1161 is printed inwhile(true)`

https://github.com/testcontainers/testcontainers-java/blob/2da17b9855f855b3ae6ea5b9f07f8cb6080dc2fa/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java#L125-L146
If createConnection() raises Exception, isRunning() is called in succession by retryUntilSuccess.

The cause seems to be that Unreliables#retryUntilSuccess only expect an Exception to be thrown by the poll-lambda whereas a missing driver probably throws an Error. This gets propagated to the executor where most probably it's swallowed leading to an infinite busy-loop somewhere.

This should be solved by #1434, so I shall close.


Toolbar documentation is wrong

The example in the current documentation for toolbar appears to be incorrect.
It lists properties in an attributes object when they should be outside.
let myBar = new Toolbar({
  attributes: {
    id: 'bar-1',
    classNames: ['my-class'],
    title: 'My Toolbar',
    type: 'header'
  }
}, [child-component, child-component]);

I believe it should be:
let myBar = new Toolbar({
  attributes: {
    id: 'bar-1',
  },
  classNames: ['my-class'],
  title: 'My Toolbar',
  type: 'header'
}, [child-component, child-component]);

For reference, the code for this feature is in src/toolbar.js.

This was fixed in #4.  Thanks again @jose4125!  ✨  :balloon: :tada:

If it is fixed, please close this issue. Thanks :beers:


Setting global gain does not work with SoapySDR API

while setting individual element gains does work as expected.
Did anybody notice, can confirm and/or correct ?
Pawel.

Whenever probing a receive channel's overall gain, I only ever get 12 dB when I set it to all manner of different values. The XTRX logs themselves confirm the requested gain was correctly interpreted.

...
08:34:17.272022 INFO:   [LSM7] USB3_2/1/4: SXR: VCO2 [199;218] -> 208
08:34:17.272669 INFO:   [LMSF] USB3_2/1/4: NCO ch=2 type=3 freq=0
[SDR: XTRX (R)] [CHAN 1] Actual Frequency:<PHONE_NUMBER>.000000
[SDR: XTRX (R)] [CHAN 1] Setting Gain Mode (Automatic): 0
[SDR: XTRX (R)] [CHAN 1] Actual Gain Mode (Automatic): 0
[SDR: XTRX (R)] [CHAN 1] Setting Gain: 40.000000
08:34:17.272883 INFO:   [LMSF] USB3_2/1/4: Set gain 40.0 to 0 on 2 channel
08:34:17.273172 INFO:   [LSM7] USB3_2/1/4: RFE: set_lna(0 -> 0) => 15
[SDR: XTRX (R)] [CHAN 1] Actual Gain: 12.000000
08:34:17.274719 INFO:   [CTRL] USB3_2/1/4: RFIC_GPIO 0x000300
08:34:17.276935 INFO:   [USB3] USB3_2/1/4: Device closing
08:34:17.277776 INFO:   [3380] IO thread termitaed with result 0

When I requested LNA only gain, this returned as 0 dB.
I've been looking at the following sections from XTRX and Soapy's libraries...
https://github.com/xtrx-sdr/libxtrx/blob/2443a10621039049c3ced7c295d4959e74a1dcef/soapy/SoapyXTRX.cpp#L298-L324
https://github.com/pothosware/SoapySDR/blob/23e5951d8d2b4f47e235b2f6025743d9b122d55b/lib/Device.cpp#L307-L317
Was wondering what would cause this to occur?


Source code link for stub workflows contain "null" and doesn't work

Describe the bug
On staging and QA, the source code link for stub workflows contain a null value and clicking it leads to a 404 on GitHub

To Reproduce
Steps to reproduce the behavior:

Go to My Workflows and quick register a workflow to create a stub workflow
See that the Source Code link contains null and clicking it leads to a 404

Expected behavior
On prod, the source code link for a stub workflow is just a link to the repository. In the case of a stub workflow which has no descriptor file defined (yet), we should also link to the repository instead of to the primary descriptor file.
Screenshots
See above
Additional context
[Domain] - https://staging.dockstore.org and https://qa.dockstore.org
Webservice - 1.14.0-beta.0
UI - 2.11.0-beta.0
Compose Setup - 1.14.0-beta.0
Deploy - 1.14.0-beta.0
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.14
┆Issue Number: DOCK-2395
┆Sprint: 113 - Liao
┆Issue Type: Story

There's a case where this still happens:

In My Workflows, go to a non-stub workflow
Then navigate to a stub workflow and see the null.

It looks like it's because we're using the same component instance for both workflows, but the ngOnChanges() is assuming that sourceCodeFile and other fields are in their initial state. See here -- looks like there are other fields that are also wrong.


Batch operations

Hi @tkipf , excellent work on pygcn! Really nice engineering setting up sparse adjacency multiplications and super clean code. I'm curious to hear how you suggest dealing with batch operations? Unless I am misunderstanding, it looks in train.py that each epoch operates on a single large graph, and the labels are per-node labels. If this interpretation is correct, do you have any suggestions for datasets consisting of many graphs (a series of sparse matrices) each mapped to a graph-level output/label? this would be solved if PyTorch could accept a list of tensors as an input but that does not seem (easily) supported right now. Thanks for any advice!
Cheers,
Evan
PS Great to meet you at Stanford a few weeks ago!

Hi @evanfeinberg, thanks! It was a pleasure meeting you as well. I posted a solution for this problem some time ago in this thread (from the gun repository): https://github.com/tkipf/gcn/issues/4 . I had an implementation of this some time earlier last year already and if I remember correctly, results were comparable (or sometimes better) in relation to the method by Niepert et al. (ICML 2016) for batch-wise graph classification. So inductive learning with this type of model (contrary to common belief) is indeed very much possible.
To re-state the solution over here: multiple graphs can be fed as a single sparse block-diagonal adjacency matrix, and graph-level classification can be done by adding a global "hub" node that is connected to all other nodes (this was suggested already back in 2009 by Scarselli et al.).

Thank you for the speedy response, @tkipf ! The workaround of a block diagonal sparse matrix consisting of individual adjacency matrices makes a lot of sense. I will try and report back!

@tkipf thanks for the package, it is really useful.
Will it be possible to share your code that switches a batch of graphs into a diagonal blocks matrix. I found a scipy implementation. But maybe you managed to use a pytorch built-in functions, which will be much more efficient.

I don’t have an implementation for this, unfortunately.
On Sat 21. Apr 2018 at 12:06 idansc<EMAIL_ADDRESS>wrote:

@tkipf https://github.com/tkipf thanks for the package, it is really
useful.
Will it be possible to share your code that switches a batch of graphs
into a diagonal blocks matrix. I found a scipy implementation. But maybe
you managed to use a pytorch built-in functions, which will be much more
efficient.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tkipf/pygcn/issues/1#issuecomment-383282818, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHAcYKInXPnq6YoLJ13UYz2os79dbeFAks5tqwSggaJpZM4QNhT7
.

@idansc Hi,  can you provide a link address code for the  scipy  implementation ?

@malizheng  see scipy.linalg.block_diag


how can I move a grid in static grid

I want to set a big static grid other grid can move in the static grid，eg:

You need to do event. stopPropagation() on onDrag (and/or onDragStart/onDragStop on the grid), so it doesn't propagate to the current event.
This is more or less what I do:
      const onDragEvt = React.useCallback<ItemCallback>((_l, _o, _n, _p, evt, _elem) => {
        evt.preventDefault()
        evt.stopPropagation()
      }, [])


Log and quit upon being transferred

Awesome!

Wait, so specters can be transferred now?

They will leave the game, but they of course won't transfer. Just read the code :P

Obviously I read the code, I wanted to make sure they couldn't be transferred, people could slam players to other servers if specters could be transferred.

@Gamecrafter specters are just "players connected using console through PHP" (just like BigBrother connected "using MCPC through TCP" and normal players connected "using MCPE through UDP". Specter basically does no external networking. How can they join other servers? :grin:
And it would be great if we can bot them using three lines of code :smirk:


This page doesn't show how to use existing Identity

I recently worked with NVIDIA to enable MacSec on 100G ER Direct circuit. They wanted to use Existing Key vault, MacSec secrets and identity.
This document is missing information around, how to retrieve user identity and reuse them

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

ID: 1d55ccda-fe86-6732-41d3-abea758842c8
Version Independent ID: 4e043350-9d88-a57f-e743-a169d73d86be
Content: Azure ExpressRoute: Configure MACsec
Content Source: articles/expressroute/expressroute-howto-macsec.md
Service: expressroute
GitHub Login: @duongau
Microsoft Alias: duau

@kiritbhamre
Thanks for your feedback! We will investigate and update as appropriate.

Hello @kiritbhamre, thank you for your suggestion and feedback on this article. I added the command for using an existing user identity into the article. Please feel free to reach out if you see anything else missing. Thank you! #please-close


Debug service

Hi,
first of all debug is wonderful. Thank you!!!
I don't know what I'm doing wrong, but when debug start, looking at joblog I see this:

But the default of my strdbg command in qsys is:
Updprod(*YES)
I try both from local and from object browser.
In local the debug service create a file launch.json:
{   // Use IntelliSense to learn about possible attributes.   // Hover to view descriptions of existing attributes.   // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387   "version": "0.2.0",   "configurations": [     {       "type": "IBMiDebug",       "request": "launch",       "name": "IBM i Debug: Launch a batch debug session",       "user": "",       "password": "",       "host": "",       "port": 8005,       "library": "",       "program": "",       "startBatchJobCommand": "",       "updateProductionFiles": true     }   ] }
But when my sqlrpgle make an insert the sqlstate is equal to: 55019 .
Many thanks for your help.

Do you have 'update production files' enabled in the connection settings? After you've connected to the system, those options appear at the bottom.

Hi Liam,
Thank you for your response. now it works.
Many thanks


docs: change card-title headers to ion-card-title elements

Refactor done in this issue: https://github.com/driftyco/ionic/issues/5303

http://ionicframework.com/docs/v2/components/#card-list
Use the ionic element, not classes


Add support for adding scrollview with buttons

Would be nice to be able to add a series of buttons through which the user can scroll. See: https://stackoverflow.com/questions/42123495/touch-bar-how-to-add-a-scrollable-list-of-buttons. For an example see the 'Calendar' application that allows selection of months from a horizontally scrolling list.

Adding UIScrollView support to the touchbar isn't really supported in a nice way by the OS.
The Calendar app uses the NSScrubber for doing this which is currently already supported in JTouchBar.


fix(pingcap/tidb-dashboard): forward event to cherrypicker external plugin

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
What's Changed:
How it Works:

/approve


AzureWebAppV1 Gives False Error when Deploying Web App Files

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: AzureWebAppV1
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
Account Name: NetSupportSoftware
Team Project Name: NetSupport_DevOps
build number: buildId=2202

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: ubuntu-latest also fails with windows_latest

Issue Description
[Include task name(s), screenshots and any other relevant details]
AzureWebAppV1 task gives a wrong negative error when completing. Nothing I do can make it go away. It comes back with the error:
##[warning]Failed to update App Service configuration details. Error: Error: Failed to update App service 'nsclassroom-dgyn27h2dfoyojc' configuration. Error: BadRequest - Required parameter AccessKey is missing. (CODE: 400)
Yet it successfully downloads the artifact from the job and puts it into place yet shows this error. Please see screenshot.

The job even finishes and lets the pipeline go on. When checking the website the files are in place and the website is working as expected.
Task logs
See Bellow
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Troubleshooting Steps:
3 Days work:
Changing webapp settings removing one after the other,
Changing subscriptions
Changing service connections.
Building another pipeline.
Changing pre download and publish tasks to newer pipeline tasks.
Changing Artifact files deploying from another repo.
Asking on Stack Overflow. Question Ref: https://stackoverflow.com/questions/70800480/azure-devops-failed-to-update-app-service-configuration-details
Error logs
##[warning]Failed to update App Service configuration details. Error: Error: Failed to update App service 'nsclassroom-dgyn27h2dfoyojc' configuration. Error: BadRequest - Required parameter AccessKey is missing. (CODE: 400)
Debug Log File:
AzureWebAppV1Debuglog.zip
Project Files can not upload contact direct for them.

We are also having this exact warning when deploying azure functions using the AzureFunctionAppV1 task. The package gets deployed correctly, and we can trigger the function without any problems, but we still get this warning in our logs. Nothing we do manages to get this warning away.
More info about the function: we deploy python functions to an app service that mounts a file share from a storage account.
On another note: isn't the warning message a little bin unclear? What access key is the task looking for?


[BUG] LightGCN: Training time discrepancy in Benchmark MovieLens notebook

Description

LightGCN algorithm is reported to have a larger Training time than expected in the microsoft/recommenders/blob/main/examples/06_benchmarks/movielens.ipynb notebook
In which platform does it happen?

How do we replicate the issue?

Expected behavior (i.e. solution)
 Current Training time: 6129.2162s
Expected Training time: Training time: 417.1986s

Other Comments
This issue resolves #1758

Tracking a similar issue https://github.com/microsoft/recommenders/issues/1825


Fixes 2218

GIF showing the changes:

Codecov Report

Merging #2422 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2422   +/-   ##
=======================================
  Coverage   93.87%   93.87%
=======================================
  Files         161      161
  Lines       19177    19177
=======================================
  Hits        18003    18003
  Misses       1174     1174

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 7aa891b...f3e5f8e. Read the comment docs.


Make CustomerManager#get_or_create_from_request idempotent

The method name get_or_create_from_request implies that if a customer already exists, it just returns the existing customer. This fails however, if called multiple times, as the user of the customer is always created.
Maybe the preferred way to avoid that is:
if request.customer.is_visitor():
    request.customer = CustomerModel.objects.get_or_create_from_request(request)

But it should still be possible to just call the method multiple times.

Very well noticed, thanks!
Just for curiosity, does your project code invoke that method directly? I reviewed my projects, and there this method is invoked exclusively by Cart.get_or_create_from_request(), hence I never encountered the problem of having different Users/Customers for the same session key.

@rfleschenberg could you also review and then merge this?

@rfleschenberg Should I simply change to comment to create or get a previously created inactive intermediate user..., because getting the user (and customer) would still return the mentioned intermediate user. Moving the comment might leave questions what the get returns?
@jrief Yes, I call that directly in my allauth adapter for returning a new user:
class CustomerAccountAdapter(DefaultAccountAdapter):

    def new_user(self, request):
        customer = models.Customer.objects.get_or_create_from_request(request)

        user = customer.user
        user.is_active = True

        return user

@rubengrill Sounds good.
I wonder about the active flag. Is it guarantueed to be False for existing users? Or should we maybe limit the query to users with active == False? Not sure, I'd have to dig into the code more. But it's not central to this PR.

@rfleschenberg Good point, I could imagine that this might be too restrictive in case a customer gets converted:
request.customer = Customer.objects.get_or_create_from_request(request)

# Later e.g. in a register form
request.customer.recognize_as_registered()
request.customer.user.is_active = True
request.customer.user.save()
login(request, request.customer.user)

# Later e.g. in a login signal handler
customer = Customer.objects.get_or_create_from_request(request)

Maybe its safe enough to rely on the fact that the username is the session key and if that user exists, it most probably exists because it was created in this method.

Thanks!


iPhone X support

what about iPhone X support ?
i have tried playing with constraints and frame , but it does not work , any ideas ?

Hi @AbdallahJaja,
Thanks for opening issue 👍 .
This issue has bean addressed. Please check the latest version 2.3.5,
Thanks.


docs(*): Change doc generation to keep them out of master

Ok, so docs being in master were really annoying and they often became out of date, this PR shunts them into the gh-pages branch and will automaticaly update them as a part of a release cycle

    oK, so dOcS beInG In mAStEr wErE REaLLy aNnoyINg and tHEy OftEn becAmE OUt Of daTe, thIS Pr ShUNTS theM iNTo ThE gh-PageS BRANch and Will auTOmATICAlY UpDAte them aS A parT of a ReLEAse cYCLe

Oh and typescript 2.4 is kinda mean: http://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-4.html

@ProbablePrime,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Correctly format nested lists

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
This PR fixes the formatting of nested lists in the README under the "API Endpoints" section.
Description
GitHub flavored markdown requires two spaces to nest list, not just one.

I signed it!


possible rename: embryonic precursor cell of adult midgut

'embryonic precursor cell of adult midgut' FBbt:00000449 has definition: 'A cell that is a progenitor of the adult midgut and is part of the embryonic/larval midgut. These cells occupy a basal position in the midgut epithelium and divide twice during larval stages to form clusters of four cells called midgut imaginal islands.'
As little as I like the 'embryonic/larval' construction, I think this term might deserve it, given the definition. It already has appropriate relationships.
I'll note this term has already had a rename; the original term was "adult midgut precursor cell" (or something very similar); the name ambiguously made it appear that it was an adult stage term, not an embryonic/larval term.

Just peeved the file I used the term in, and see you've already renamed this term to adult midgut progenitor cell; the original rename had been intended to remove the ambiguity of a term starting "adult" that is inappropriate to use with adult stage.


Absolute paths to configuration etc. files

Please notify users that files must have absolute path.
E.g. -c, --configuration=CONFIGURATION  Path to project configuration file could be -c, --configuration=CONFIGURATION  Absolute path to project configuration file
Also for autoload-file.

Hey @szepeviktor 👋
Those options aren't documented yet. Shouldn't we ignore/remove them from the command CodeAnalyseCommand forcing people to use PHPStan like normal on those cases?
I need some time to think about this.
Thanks for creating this issue! ☺

Hello 👋
Setting a configuration file pulls the plug on Larastan as extension.neon will be ignored.

Those options aren't documented yet.

It's documented now, at least via the Command; but it doesn't state the requirement about absolute paths:
Options:
  -c, --configuration=CONFIGURATION  Path to project configuration file [default: "/vagrant/project/vendor/nunomaduro/larastan/src/Console/../../extension.neon"]

I stumbled across this too.
In my case:

I'm already using phpstan
I already have a phpstan.neon in the root of the project

I would have expected either of the two things:

./artisan code:analyse would pick up the mine automatically
or
./artisan code:analyse --configuration=phpstan.neon

But neither does (nowhere it's started the first does, just a tale of experience here).
I would like to see the file resolution being improved so that picks up relative files from the CWD where ./artisan is ran; I think that's sensible.

I usually use ./artisan code:analyse --configuration=$(pwd)/phpstan.neon

Yeah, I know, it's totally fixable. But in the hopes of unsurprising DX I wrote this ;-)

This PR should more or less address this issue. The config file will show it is an absolute path.
https://github.com/nunomaduro/larastan/pull/178/files

This issue seems no longer be relevant.


anki-bin: add commandLineArgs to package arguments

anki-bin is built with buildFHSUserEnv on Linux, which makes using overrides
to pass in command line arguments impossible. This commit adds the
commandLineArgs argument to the package inputs (similar to google-chrome) and
appends the string to the runscript, allowing pkgs.override to add flags to pass to anki
when the derivation is built.
This is useful when anki's desired directory isn't the default, allowing
users to specify the directory in their overlays instead of adding a
flag everytime they call the program.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] 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
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[X] Fits CONTRIBUTING.md.

I don't really see much of a point in this since you could just override to add a wrapper or make your own wrapper but it doesn't hurt either.

I can't agree more. I have approved, as it doesn't hurt.


Confusing typo?

ooops, yea however its not working atm so dont use, will fix when i have time

The typo will be fixed when I get the chance.

This is resolved. Download the new builder, it should be fixed.


Make hyperlinks right clickable

Hyperlinks in markdown and other places in the app should be right clickable so you can open in a new tab / copy URL.

The believe the fix for this is to conditionally render an <a> or <Button /> here
https://github.com/lbryio/lbry-desktop/blob/master/src/ui/component/externalLink/view.jsx#L34

This should also fix being able to right-click a video from the trending page.


replace deprecated logger methods

Overview
Just replace deprecated logger methods with new ones
What this PR does / why we need it
Fixed: #685
Special notes for your reviewer
nope

@juev sorry to bother you, but we've got some conflicts to resolve :|

@juev sorry to bother you, but we've got some conflicts to resolve :|

The rebase is complete. All conflicts have been resolved.


Opacity settings

I would like to suggest the addition of opacity settings for the trade windows.
It's great that you've modernized a new version of trade overlay, but the lack of opacity filters makes the play experience a bit scuffed as you can't see your window once all the trades come in, especially if theres 3+ trades. The addition of opacity settings on a timer setting would free up the screen space to better deal with content when not actively on the window thus allowing players to react to the game far better.

I've seen this feature on other trade macros, but never used it myself. I assume the messages are opaque when they appear, then turn transparent after a set delay? I'd also imagine this is per message, so each message fades out separately?
I've had some drawing bugs when using transparent windows in the past, but assuming I can work those out I think this is a very solid idea.

Awesome! Its a real QoL booster as you rarely need the notification fully visible the whole time when you get a message and a life saver in HC so you don't get tapped by an off screen effect. GL man, I like what I see so far!

Added in v0.2.10, as well as the ability to collapse trade messages.


There is no API to get a request's socket connection's server side ip, only a request.ip

When do app.listen(PORT);, that means express will bind to any available IP on server side, but according to API docs, request.ip returns request's ip, so there is no API to get request's server-side ip.

The use case behind this bug scene is:
I build a test web server and a fake CDN server on my notebook, use the WLAN local address, i need to detect the client's request's server-side ip, then i can program it into a 302 redirect Location url.
If no this API, i have to hard-encoded it in code, which is not flexible.

It seems i can work around this:
app.use(function(req, res, next){     console.log("["+req.request_id+"] logReqHeaders: req.ip=" + req.ip+" req.socket.address().address="+req.socket.address().address);     console.log("["+req.request_id+"] req.headers: "+JSON.stringify(req.headers, null, 2));     next(); })
There is just a little problem: the ip address is IPv6 first on Windows 10, this may cause the full url format not valid...

Hi @chenzx there are some APIs provided by Node.js core and some by us (which just sugar coat the underlying Node.js core APIs). I'm not clear on what you're trying to do exactly, but wouldn't req.hostname work for you use-case? If not, then I'm not understanding your request, so can you explain more?

@dougwilson Problem has been solved. use req.socket.localAddress.replace("::ffff:","")
The background is this: i deploy a test REST app server on my local machine with a LAN address, & i need to mock another CDN file server which is, in fact, still on my local machine, so i need to know when i do a 302 redirect by absolute URL, which hostname should i use.
Since the default express app binds to ‘’, which means any address available, i need to know the exact address that current request client uses.

Hi @chenzx Express.js doesn't do any server binding directly. If you are using app.listen that is a straight proxy to the Node.js core server.listen method.
You can provide a specific IP address to listen on, specify to only listen in IPv4 mode (instead of the default dual stack), and more. Just refer to the documentation for your version of Node.js to see what is possible.

@dougwilson Thanks! I thought express community is really very kind. :-)


Behaviour of the Flux when replicas field is not mentioned in the deployment

Scenario 1:
Create a simple deployment in a repository that is under flux control
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  namespace: test
  labels:
    app: nginx
spec:
  replicas: 1
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:1.14.2
        ports:
        - containerPort: 80

Reconcile the customization and once the pod is up, scale replicas to zero by running kubectl scale deployment -n test nginx-deployment --replicas=0
After the deployments is scaled to zero reconcile the customization again by running flux reconcile kustomization <name>
Expected Result:
Flux should revert the changes made to the replicas and deployment should have replicas=1 as mentioned in the yaml file
Actual Result:
Flux reverted the changes made to the replicas and deployment had replicas=1  as mentioned in the yaml file
Scenario 2:
Repeat the steps mentioned in Scenario 1 but this time do not specify the replicas field in deployment
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  namespace: test
  labels:
    app: nginx
spec:
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:1.14.2
        ports:
        - containerPort: 80

Expected Result:
Flux should revert the changes made to the replicas and deployment should have replicas=1 as mentioned in the yaml file
Actual Result:
Flux did not revert changes made to the replicas even after reconciling customization
I am not sure if this is expected behavior or an issue
flux: v0.24.0
helm-controller: v0.14.0
image-automation-controller: v0.18.0
image-reflector-controller: v0.14.0
kustomize-controller: v0.18.1
notification-controller: v0.19.0
source-controller: v0.19.0

closing this and raising it in correct repo


[CHALLENGE SUBMISSION] OmegaMadLab - Challenge 2

What is your name? (First, Last)
Marco Obinu
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/OmegaMadLab/TerraformOnAzure-Challenge2
webapp URL: https://tfonazure-challenge2-webapp.azurewebsites.net/
What Challenge is this for? (Challenge number between 1 and 5)
Challenge 2
(Optional) Anything else we should know? e.g., language used, location, blog post?
Added some structure to the TF code used in challenge 1.

Thanks for participating in the #TerraformOnAzure coding challenges!
We have attached your very special Astronaut Badger background, for you to use in all your upcoming video calls!
And to recognize you for your learning, you can collect your digital Azure Heroes badge on your mobile (you may need to install the Enjin wallet to collect it). Make sure to collect it today!
https://beam.enjin.io/claim/d32599695c8c28cfc24820492080fac0
Keep on learning and sharing!
Greetings from the #TerraformOnAzure team,
Suzanne, Floor, Erik & Nick


change vision angle without rotating the phone physically ?

Hello Viro team, I am wondering if there is any way to rotate the Vironode, video360 or image360 view by scrolling/swiping on the touchscreen without rotating the smartphone ? useful for mono mode!

Hi @aminesig,
We support the onRotate gesture which is a two finger rotation. Unfortunately we don't support other gestures for rotation.
An example of how to use the onRotate gesture can be found in our ViroARSampleApp project:
https://github.com/viromedia/ViroARSampleApp/blob/master/js/ARHitTestSample.js
In that sample, the onRotate callback is added to the Viro3DObject and then the returned rotationFactor is used to update the rotation of the ViroNode containing the object, lights, shadow planes.
Hope that helps!
Andy

Closing due to inactivity.
If you still have issues, feel free to re-open this issue or create a new one.
The latest releases are here:
React Viro - https://docs.viromedia.com/docs/releases
ViroCore - https://virocore.viromedia.com/docs/releases
Thanks for using Viro!


Update discipline-munit to 1.0.9

Updates org.typelevel:discipline-munit from 1.0.8 to 1.0.9.
GitHub Release Notes - Version Diff
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "discipline-munit" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #493 (d2b1ccb) into master (7eab560) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #493   +/-   ##
=======================================
  Coverage   62.37%   62.37%
=======================================
  Files          38       38
  Lines        1289     1289
  Branches        7        8    +1
=======================================
  Hits          804      804
  Misses        485      485

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 7eab560...d2b1ccb. Read the comment docs.


Question: Thumbor options such as fit-in

Hi thank you for a nice library, the example works perfectly. Are there other config options?
Thumbor and (and ruby-thumbor as far as I can see) allow a wide range of options. E.g. fit-in and filters:format(webp). Is that possible? I've tried variations of below without success:

UPDATE: Completely new to Ruby I looked at the source - it seems it isn't possible.
I did an attempt here, but honestly I can see it is ugly, but I have no idea how to improve it
usage

I'm going to delete above repo, but here is the my in case anyone has an interest
# It feels like it would be more natural to do this as a filter, but due to the way Liquid works that would mean awful syntax for more complex filters like watermarks - at least I couldn't come up with a reasonable signature for a filter
require "jekyll"
require "liquid"
require 'yaml'
require "ruby-thumbor"
require 'uri'

module Jekyll
  module ThumborTag
    class ThumborTag < Liquid::Tag

      def initialize(tag_name, text, tokens)
        super

        @url, @paramString = text.split(', ', 2) # extract the first string before the comma as the image url
        @parameters = {}

        if<EMAIL_ADDRESS>          @parameters = YAML.load('{' + @paramString + '}') # parse the string as yaml, this to allow the user to provide use complex filters with many params
          @parameters.each do |key, val|
            if val.nil? # if no value provided, then assume boolean true instead of nil
              @parameters[key] = true
            end
          end
        end
      end

      def render(context)
        url = Liquid::Template.parse(@url).render context
        url = URI.encode_www_form_component(url)

        @config = context.registers[:site].config['thumbor']
        image = Thumbor::Cascade.new(@config['key'], url)

        @parameters.each do |key, val|
          image.send(key, *val)
        end

        image_url = image.generate

        "#{@config['url']}#{image_url}"
      end
    end
  end
end

Liquid::Template.register_tag('thumbor', Jekyll::ThumborTag::ThumborTag)


Update to Next 13

It's still in beta as of current and suffers from some issues (most notably that client transitions don't update the page title yet) but it brings a lot of cool features worth investigating.
https://beta.nextjs.org/docs/rendering/fundamentals
https://beta.nextjs.org/docs/rendering/server-and-client-components
https://beta.nextjs.org/docs/routing/fundamentals
https://beta.nextjs.org/docs/routing/defining-routes
https://beta.nextjs.org/docs/routing/pages-and-layouts
https://beta.nextjs.org/docs/routing/loading-ui
https://beta.nextjs.org/docs/api-reference/components/link (no longer needs to wrap an <a>)
https://beta.nextjs.org/docs/api-reference/file-conventions/head
https://beta.nextjs.org/docs/api-reference/file-conventions/not-found
https://beta.nextjs.org/docs/data-fetching/fundamentals
https://beta.nextjs.org/docs/data-fetching/fetching
I don't know how well it plays with static HTML export though.

Currently, this PR is waiting for Next 13 to support

global "error page" (404.tsx)
static HTML export (next export)

before it can be merged. Also, the new data-fetching model does not seem to dedupe fetch requests on dev, which causes /concepts/codeforces pages to run into the codeforces api limit issue encountered before (at least, I think that's what all the "unexpected character > in JSON" errors are when compiling pages).


Added two examples for getting tickets and emailing their assignees

The built-in alerts function in SecurityCenter wasn't sufficient for my needs, so I utilized your code to come up with a solution. I felt it would be helpful for future searchers to have an example of using your code for tickets.

@maravedi  Would be nice to see examples of these emails :)


Add support for x-ms-parameter-grouping

Create parameter group structs as specified by the extension.
The grouping of optional args into an 'options' struct was reworked to
build on this implementation.
Added paramgroupinggroup tests and fixed a few tests that were impacted.

At first I was a bit annoyed about the changing of the optional params arg name, however after thinking about it a bit, it's consistent with explicitly defined param groups so I'm ok with it now (let me know if you disagree).

I'm still thinking about it. Not sure I'm where you are about it yet. But I'm also not strongly opposed at the moment. You're saying that you don't mind it because the name corresponds to the param group name?

Yes and is consistent with explicitly defined parameter groups.

Fixed https://github.com/Azure/autorest.go/issues/348


compile error for else if

This code will produce a duplicate else branch found:
{% if 3 == 4 %}
  3 is not 4
{% else if 4 == 5 %}
  bug
{% else %}
  compile error
{% endif %}

But this code will compile and output bug:
{% if 3 == 4 %}
  3 is not 4
{% else if 4 == 5 %}
  bug
{% endif %}

I got confused by this today, as syntax is so close to go.
This PR will produce a compile error in this case.

Thanks, @stereosteve !


igraph: 0.8.3 -> 0.8.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/igraph/igraph/releases.
meta.description for igraph is: "The network analysis package"
meta.homepage for igraph is: "https://igraph.org/"
meta.changelog for igraph is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.8.4 with grep in /nix/store/5z7fazffwxm6zhf5rz61zs97al0dprxs-igraph-0.8.4

directory tree listing: https://gist.github.com/cf02e366890ebc1d2d433fe2142da884

du listing: https://gist.github.com/08ad40e02239716876c986c5d8b063fe

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

34 total rebuild path(s)

10 package rebuild(s)

10 x86_64-linux rebuild(s)
5 i686-linux rebuild(s)
10 x86_64-darwin rebuild(s)
9 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
hal-hardware-analyzer
igraph
python37Packages.cozy
python37Packages.kmapper
python37Packages.python-igraph
python37Packages.scikit-tda
python38Packages.cozy
python38Packages.kmapper
python38Packages.python-igraph
python38Packages.scikit-tda

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5z7fazffwxm6zhf5rz61zs97al0dprxs-igraph-0.8.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5z7fazffwxm6zhf5rz61zs97al0dprxs-igraph-0.8.4
ls -la /nix/store/5z7fazffwxm6zhf5rz61zs97al0dprxs-igraph-0.8.4/bin

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

10 packages built:

hal-hardware-analyzer
igraph
python37Packages.cozy
python37Packages.kmapper
python37Packages.python-igraph
python37Packages.scikit-tda
python38Packages.cozy
python38Packages.kmapper
python38Packages.python-igraph
python38Packages.scikit-tda

Maintainer pings
cc @MostAwesomeDude for testing.

Result of nixpkgs-review pr 104990 run on x86_64-darwin 1

10 packages built:

hal-hardware-analyzer
igraph
python37Packages.cozy
python37Packages.kmapper
python37Packages.python-igraph
python37Packages.scikit-tda
python38Packages.cozy
python38Packages.kmapper
python38Packages.python-igraph
python38Packages.scikit-tda


feature(loki): make Resultset and DynamicView sortable by full-text-s…

…earch scoring

refactor sort priority of dynamic view as enum

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/LokiJS-Forge/LokiJS2/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

Coverage increased (+0.05%) to 92.029% when pulling 54179b062d36f0e0451875439230a28c37a07ccd on sort_full_text_search into 2f3b718774766ce4323c24998172931054c8cf6f on master.


Feat/automatic detach from debugger

Branched off from https://github.com/whitecolor/ts-node-dev/pull/93
Specifically I had to revert/alter the undocumented newly added exitChild option. It was wrongly spelled in options, not documented, and simply went contrary to expect behavior. It least IMO. Please enlighten me @whitecolor

ping @whitecolor - separated this out now

This sounds awesome. Does it work? If so I can probably create a patch-package with the changes cause it doesn't look like this'll ever get merged


RuleForEach.SetValidator ignores when clause

System Details

FluentValidation version: 8.4.0

Issue Description
Steps to reproduce
Use the following rule in an AbstractValidator
RuleForEach(p => p)
    .SetValidator(c => new CustomValidator())
    .When(p => false);

Expected behaviour
The rule does not run, as the when will always evaluate to false.
Actual behaviour
The rule runs.

Complete Example
public class ClassA
{
    public ClassB OptionalProperty { get; set; }
}

public class ClassB
{
    public List<ClassC> Collection { get; set; }
}

public class ClassC
{
    public string Name { get; set; }
}

public class ClassAValidator : AbstractValidator<ClassA>
{
    public ClassAValidator()
    {
        // Correct behaviour. Generates a deprecated warning
        RuleFor(p => p.OptionalProperty.Collection)
            .SetCollectionValidator(c => new ClassCValidator())
            .When(p => p.OptionalProperty != null);

        // Blows up- NullReferenceException
        RuleForEach(p => p.OptionalProperty.Collection)
            .SetValidator(c => new ClassCValidator())
            .When(p => p.OptionalProperty != null);
    }
}

public class ClassCValidator : AbstractValidator<ClassC>
{
    public ClassCValidator()
    {
        RuleFor(c => c.Name).MaximumLength(20);
    }
}

Hi there, I think maybe you're misunderstanding how When interacts with RuleForEach. If you think of the two examples procedurally, this is what they look like:
// first example. No NRE, as you're checking the collection. This is what you're trying to do.
if (OptionalProperty != null) {
   foreach (var item in OptionalProperty.Collection) {
       if (item.Name.Length > 20) {
         // validation failed.
       }
   }
}

// Second example, this is the equivalent of what you're doing with RuleForEach.
foreach (var item in OptionalProperty.Collection) {
    // This doesn't make sense - running the null check inside the loop
    if (OptionalProperty != null) {
       if (item.Name.Length > 20) {
          // Validation fails
       }
    }
}

Hopefully that illustrates the issue. If you think of a call to RuleForEach as a foreach loop anything after the call to RuleForEach is inside the loop and run once for each element in the collection, so you're essentially attempting to check the nullability of the parent property inside the loop, rather than before the loop. You need to ensure your condition is outside the loop. If you want to keep using RuleForEach then the simplest option is just to wrap it inside a call to When:
// 1. Wrap the RuleForEach in a call to When
When(x => x.OptionalProperty != null, () => {
  RuleForEach(p => p.OptionalProperty.Collection).SetValidator(c => new ClassCValidator());
});

It should also be possible to use RuleFor and then drop down to ForEach behaviour like this:
RuleFor(x => x.OptionalProperty.Collection)
	.ForEach(element => element.SetValidator(new ClassCValidator()))
    .When(x => x.OptionalProperty != null)

...but I've realised that the return type for ForEach is incorrect so this won't actually compile. I'll get this fixed for the next version.
Hope that makes sense.

Also, I think that the behaviour isn't particularly intutive, so I'll think about changing this so that calls to When do run outside the loop (we already have the Where method for filtering elements inside the loop), but I need to do some testing to see if this would be a breaking change.

Initial implementation to change this behaviour is here: https://github.com/JeremySkinner/FluentValidation/pull/1196 Technically this would be a breaking change, so might be better for the 9.0 release rather than 8.5

Hey @JeremySkinner thanks for the quick response and proposed change!
Like you said, it does seem like the per-collection item filtering feature is provided by the Where method.
Whereas the When method controls if a Rule should run (which I assumed would also control if the whole RuleForEach should run).

I shall run with your suggested fix for now
// 1. Wrap the RuleForEach in a call to When
When(x => x.OptionalProperty != null, () => {
  RuleForEach(p => p.OptionalProperty.Collection).SetValidator(c => new ClassCValidator());
});

Decided that I will include it in 8.5 - aiming to push out a 8.5-preview4 later this week.

8.5-preview4 is now on nuget with this change.

Excellent. Thank you!


feat: Add possibility to pass envs to shared hooks

Implement in .shared.yaml:
envs:
    hooksA:
      - "OPTION=A"
    hooksB:
      - "OPTION=G"

which would inject env variables before running all hooks in namespace hooksAand hooksB.
With that, the user can customize some shared hooks when using them.

Solved in #96.


Fix replace deprecated count_scalar with count

https://prometheus.io/docs/prometheus/2.0/migration/#promql

Thanks eeddaann!


Sort tests into single and double backends

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

Project coverage is 95.01%. Comparing base (c8e6bd5) to head (d43a741).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
- Coverage   96.29%   95.01%   -1.28%
==========================================
  Files          74       15      -59
  Lines        3937     1787    -2150
==========================================
- Hits         3791     1698    -2093
+ Misses        146       89      -57

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


UploadedFile: Handle content being a Pathname

Dynamic languages... Can't live with them, can't live without them.
In #149, we added support for UploadedFile being an StringIO object, by means of the if content.respond_to?(:read) logic. However, that had the unintended consequence of breaking the use case when a Pathname is provided instead of a string as the content paramer, since a Pathname also responds to the read message...
This PR works around that, by adding some extra checking. As an added bonus, I also added some YARD comments to make it more clear what parameter types this method expects/accepts.
Fixes #207.

Great job @perlun !
@scepticulous could you help to review?

(For reference, two people have written in #207 that the fix works for them. 🎉)

Shall we add test cases for IO, Pathname and String? As this PR is urgent, it's up to you whether we add those later.

I think we go with the "later" approach for now. But feel free to add them if you like, and have time & energy for it.

Thanks for the releasing! Of course I am fine for the "later" approach.
Sorry I have no energy to add them right now.

Of course I am fine for the "later" approach.
Sorry I have no energy to add them right now.

That's perfectly fine with me. We're all humans.


[Bug] search filter not showing

https://user-images.githubusercontent.com/75091899/162565406-db5cf1cb-6879-422c-9086-7c6f366d61e0.mp4
Android 8
Huawei p8 lite 2017

Can you provide android version?

Can you provide android version?

8


Framework: Add support for hover in GVRViewSceneObject

Add support for hover events in GVRViewSceneObject based on GVRPicker events. It dispatches hover events on three picker events: onInside (Hover_Move), onExit (Hover_Exit) and onEnter(Hover_Enter).
To test this support, a GVRf-Demo (gvr-events) has been updated. See https://github.com/gearvrf/GearVRf-Demos/pull/620

Hi @NolaDonato, yes it is working fine with the controller. It was tested using the updated demo (proposed in https://github.com/gearvrf/GearVRf-Demos/pull/620).

Works with controller too. Planning to merge on Wednesday.


Hexadecimal integer literals get corrupted

Describe the bug
When returning 0x1 cast to Int from a Script, the Playground shows the result value as 5.
When Cadence code contains a hexadecimal integer literal like 0x1, it gets corrupted and evaluates to the number + 4 (number of accounts).
To Reproduce
Steps to reproduce the behavior:

Run pub fun main(): Int { return 0x1 } as a script
The returned result is:

{"value":"5","type":"Int"}

Expected behavior

The result should be:
{"value":"1","type":"Int"}

Hexadecimal integer literals do not get corrupted

Screenshots

Environment

OS: macOS Big Sur, 11.6.6
Browser: Chrome, 109.0.5414.119 (Official Build) (x86_64)

Reason
Address replacement logic in https://github.com/onflow/flow-playground-api/blob/f4b28c178641940b2294391eda1637d3d17c8622/adapter/address.go#L50-L70 should only apply to imports, not integer literals

Related:

onflow/flow-go#2812
#125

@DylanTinianov can you take a look at this, it probably is coming form the address adapters.

@DylanTinianov can you take a look at this, it probably is coming form the address adapters.

I'll look into this asap.

Happy to help here. Is the address adjustment logic trying to only adjust addresses in imports, or any address in the code?
import 0x1 // address in import

pub fun main(): Address {
    let number = 0x1 // integer literal, not an address

    return 0x1 // address value
}

The translation happens here https://github.com/onflow/flow-playground-api/blob/master/adapter/address.go
If I remember correctly it should do that in many different places. Probably there is a miss tho.

Currently this is an edge case we don’t plan on supporting in this version of Playground, but will be fixed in Playground v2. Sorry for any inconvenience this has caused!

@DylanTinianov If v1 has some more time to live, I can make a quick PR to emulator to skip N addresses when bootstrapping.


[CI] use obltGitHubComments

What does this PR do?
Use https://github.com/elastic/apm-pipeline-library/blob/master/vars/obltGitHubComments.txt
Why is it important?
Avoid hardcoded strings and easy to normalise the GitHub comments in all the oblt projects
You can also see in the GitHub comment that there is a specific section for some of those GitHub comments called 🤖 GitHub comments

:grey_exclamation: Build Aborted

The PR is not allowed to run in the CI yet

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Reason: The PR is not allowed to run in the CI yet

Start Time: 2021-09-22T09:29:02.785+0000

Duration: 3 min 10 sec

Commit: f61cab1e9b150b9fe1216efb8431849fa6224099

Steps errors
Expand to view the steps failures

Load a resource file from a shared library

Took 0 min 0 sec . View more details here
Description: approval-list/elastic/ecs-logging-nodejs.yml

Error signal

Took 0 min 0 sec . View more details here
Description: githubPrCheckApproved: The PR is not allowed to run in the CI yet. (Only users with write permissions can do so.)

/test

I've given observablt-robots admin access to this repo. Perhaps that is what is holding this up.
@v1v Do you know what "approval-list/elastic/ecs-logging-nodejs.yml" is? It is mentioned in the details of the "Steps errors" above. I see https://github.com/elastic/apm-pipeline-library/tree/master/resources/approval-list/ has a list of some repos, but not this one. However, are those files just early tests of something that isn't used, because all those files only list your account.

@v1v Do you know what "approval-list/elastic/ecs-logging-nodejs.yml" is? It is mentioned in the details of the "Steps errors" above. I see https://github.com/elastic/apm-pipeline-library/tree/master/resources/approval-list/ has a list of some repos, but not this one. However, are those files just early tests of something that isn't used, because all those files only list your account.

That's some unused internal approval list implementation to give granted access to run in the CI to certain users or a project based. It was implemented but never activated, likely there is a missing condition if the file exists, good catch, it should not be shown in the PR comment at all. I'll add the fix to avoid those misleading comments, that only happens when the contributor doesn't have write access to the project.

/test

/test


micro: prepare to port operator ADD_N kernel from lite with test

Implement skeleton (non-working) code for operator and test.
Header files changed.
Namespaces changed.
Some original code deleted.
Some original code modified.
PR step 4 of the work to port operator ADD_N as tracked in Issue #46162

@ddavis-2015  This PR is in draft, any update on this? Please. Thanks!


ModuleParseError when referencing external repo

Getting this when I try to reference a separate project (repo/b/components/layout.js):
ModuleParseError: Module parse failed: /path/to/repo/a/node_modules/react-hot-loader/webpack.js!/path/to/repo/b/components/layout.js Unexpected token (65:17)
You may need an appropriate loader to handle this file type.
|     return

...

The paths are like:
/repo/a
/repo/a/pages/page1.js
/repo/b
/repo/b/components/layout.js

And in page1.js it is doing this:
import Layout from '../../b/components/layout'

The next server is running from within repo a:
cd repo/a
npm run dev # which is this:
babel-node --presets=es2015,stage-0 --plugins=transform-react-jsx,styled-jsx/babel server.js

Wondering how to make it so this external repo works. Not sure if it is related to https://github.com/zeit/next.js/issues/1706.

[ ] I have searched the issues of this repository and believe that this is not a duplicate. Closest is https://github.com/zeit/next.js/issues/1706.

Expected Behavior
Expected behavior is that I can load this JSX module from a separate repo outside of the main project.
Current Behavior
Current behavior is that I can't abstract out code into a separate repo if it has JSX in it.
Context
I can't refactor code into a shared lib.
Your Environment

Tech
Version

next
4beta2

node
8.4

I also get this error if I npm link repo b into repo a:
cd repo/b
npm link
cd ../..
cd repo/a
npm link repo-b

Then import this instead:
import Layout from 'repo-b/components/layout'

Same error:
ModuleParseError: Module parse failed: /path/to/repo/a/node_modules/react-hot-loader/webpack.js!/path/to/repo/b/components/layout.js Unexpected token (65:17)
You may need an appropriate loader to handle this file type.
|     return

...

However there is one small difference. It renders one time without the error, then on refresh is when it shows the error. Not sure. Either way its not working.

Duplicate of #706


need batch mode

I need a way to run it without the human interface. I search for the best word to narrow my search. I see that for every word there should be 3**5 = 243 possible outcomes. I want to run all 2315 words for all 243 possible outcomes. For each word and outcome, I want to know how many words will remain. Then I can use this information to help select the best starting word. So something like guess_word = None, is_correct_place = None, is_used = None as default as a default arguments to ask_guess and then it only ask for input if not == None.

Solved by calc_remaining_words


Allow to ensure the presence of a primary key

If ensure_primary_key is True and no primary key is found on the table,
datagrid will create one for it.
This is necessary for some things to work, like to select a single
row to call the callbacks when a row is selected/activated/etc,
to update the 'selected' column when it is present on the view
and other things.

:+1:
Assigned to @bosonogi for code review

:+1:


Can not move Item from Chest

Hello ;)
Problem: I have in one of my cabinets, Generic Frog Eggs inside, and I can not take them out of the cabinet. And I can't dismantle the cabinet because of this. I use your StackItem mod. I can add to the stack eggs, more eggs. But as soon as I want to take them all out again, there are always 2 left that I can't pick up.
Normally I would have used the autocrafter to get the eggs out of the cabinet, but the recipe is missing in the autocrafter. Now I don't know if it's the stack mod or the game itself.
I use the latest game version and the latest game mods from today.
What do you need to find the error?
Player log or dev log? Or Save Game? Or everything?
Active Mods:

AutoConsume
Minimap
InventoryStacking
FixUnofficialPatch
AutoBackup
QuickSave
PinRecipe
SaveonQuit
SortSaves

There is a vanilla respawn bug that creates duplicates of items which can not be in the same inventory. You have to use up, shred or lose the duplicate(s).

Ok. Is there a way to remove the cabinet inclusive items? Any mod or something?

What do you mean by "cabinet inclusive items"?

Problem has probably just been solved. I walked around the map less, and just went back to the cabinet. And suddenly I was able to pick up the items after all.
I meant that I can remove the cabinet, including the items that are in it. The items are not stuck in my inventory, but in the cabinet.
But has now settled for the time being. Somehow the game has regulated the itself. However^^I


Skip empty or commented lines when reading python requirements/dependencies

Fixes https://github.com/flet-dev/serious-python/issues/102

serious_package's package command does not process requirements.txt anymore.


Strip problematic private escape sequences

Type of changes

[x] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[x] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
"Private escape sequences" are escape sequences that are reserved for private use, though \x1b7 and \x1b8 are commonly used for storing and restoring the current cursor position. When those escape codes are not stripped the cursor jumps around and causes Rich to write garbage output. An example of a program that uses this cursor store/restore functionality is the APK package manager in Alpine Linux:
https://gitlab.alpinelinux.org/alpine/apk-tools/-/blob/48d91f482eb48a0a107b714ee183bb7e07782e14/src/print.c#L232-240
This commit updates the ANSI parser to ignore the \x1b0-\x1b? escape sequences, thus preventing them from being printed and causing havoc.

Thanks


Flutter UIAlertView is deprecated

Issue running on ios simulator after flutter upgrade 2 days ago, was working fine before:
Last login: Wed Sep 28 19:17:08 on ttys117
rohan@Rohans-Air jakstatcourier % flutter clean && flutter run -d 3D3
Cleaning Xcode workspace...                                        14.3s
Cleaning Xcode workspace...                                         6.6s
Deleting build...                                                  167ms
Deleting .dart_tool...                                               2ms
Deleting Generated.xcconfig...                                       0ms
Deleting flutter_export_environment.sh...                            0ms
Deleting Flutter.podspec...                                          0ms
Deleting ephemeral...                                                0ms
Deleting ephemeral...                                                0ms
Deleting ephemeral...                                                0ms
Deleting .flutter-plugins-dependencies...                            0ms
Deleting .flutter-plugins...                                         0ms
Running "flutter pub get" in jakstatcourier...                   1,521ms
Launching lib/main.dart on iPhone 13 in debug mode...
Running pod install...                                             10.6s
Running Xcode build...
Xcode build done.                                           351.7s
Failed to build iOS app
Error output from Xcode build:
↳
objc[2932]: Class AMSupportURLConnectionDelegate is implemented in both
/usr/lib/libauthinstall.dylib (0x2051e7ad8) and
/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDe
vice (0x10e52c2c8). One of the two will be used. Which one is undefined.
objc[2932]: Class AMSupportURLSession is implemented in both
/usr/lib/libauthinstall.dylib (0x2051e7b28) and
/System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDe
vice (0x10e52c318). One of the two will be used. Which one is undefined.
** BUILD FAILED **
Xcode's output:
↳
Writing result bundle at path:
/var/folders/dt/ymx6cbj16xq_flrs8l2xzc9r0000gn/T/flutter_tools.rzO49o/flutt
er_ios_build_temp_dirHHxj9c/temporary_xcresult_bundle
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/location-4.4.0/ios/Classes/L
ocationPlugin.m:176:13: warning: 'UIAlertView' is deprecated: first
deprecated in iOS 9.0 - UIAlertView is deprecated. Use UIAlertController
with a preferredStyle of UIAlertControllerStyleAlert instead
[-Wdeprecated-declarations]
            UIAlertView *alert = [[UIAlertView alloc]
            initWithTitle:@"Location is Disabled"
            ^
In module 'UIKit' imported from
/Users/rohan/Downloads/jakstatcourier/ios/Pods/Target Support
Files/location/location-prefix.pch:2:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/UIKit.fra
mework/Headers/UIAlertView.h:27:12: note: 'UIAlertView' has been explicitly
marked deprecated here
@interface UIAlertView : UIView
           ^
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/location-4.4.0/ios/Classes/L
ocationPlugin.m:176:48: warning: 'UIAlertView' is deprecated: first
deprecated in iOS 9.0 - UIAlertView is deprecated. Use UIAlertController
with a preferredStyle of UIAlertControllerStyleAlert instead
[-Wdeprecated-declarations]
            UIAlertView *alert = [[UIAlertView alloc]
            initWithTitle:@"Location is Disabled"
                                               ^
In module 'UIKit' imported from
/Users/rohan/Downloads/jakstatcourier/ios/Pods/Target Support
Files/location/location-prefix.pch:2:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/UIKit.fra
mework/Headers/UIAlertView.h:27:12: note: 'UIAlertView' has been explicitly
marked deprecated here
@interface UIAlertView : UIView
           ^
2 warnings generated.
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/location-4.4.0/ios/Classes/L
ocationPlugin.m:176:13: warning: 'UIAlertView' is deprecated: first
deprecated in iOS 9.0 - UIAlertView is deprecated. Use UIAlertController
with a preferredStyle of UIAlertControllerStyleAlert instead
[-Wdeprecated-declarations]
            UIAlertView *alert = [[UIAlertView alloc]
            initWithTitle:@"Location is Disabled"
            ^
In module 'UIKit' imported from
/Users/rohan/Downloads/jakstatcourier/ios/Pods/Target Support
Files/location/location-prefix.pch:2:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/UIKit.fra
mework/Headers/UIAlertView.h:27:12: note: 'UIAlertView' has been explicitly
marked deprecated here
@interface UIAlertView : UIView
           ^
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/location-4.4.0/ios/Classes/L
ocationPlugin.m:176:48: warning: 'UIAlertView' is deprecated: first
deprecated in iOS 9.0 - UIAlertView is deprecated. Use UIAlertController
with a preferredStyle of UIAlertControllerStyleAlert instead
[-Wdeprecated-declarations]
            UIAlertView *alert = [[UIAlertView alloc]
            initWithTitle:@"Location is Disabled"
                                               ^
In module 'UIKit' imported from
/Users/rohan/Downloads/jakstatcourier/ios/Pods/Target Support
Files/location/location-prefix.pch:2:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/UIKit.fra
mework/Headers/UIAlertView.h:27:12: note: 'UIAlertView' has been explicitly
marked deprecated here
@interface UIAlertView : UIView
           ^
2 warnings generated.
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:22:18: warning: 'addressDictionary' is
deprecated: first deprecated in iOS 11.0 - Use @properties
[-Wdeprecated-declarations]
        if (self.addressDictionary != nil) {
                 ^
In module 'CoreLocation' imported from
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.h:10:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/CoreLocat
ion.framework/Headers/CLPlacemark.h:73:63: note: 'addressDictionary' has
been explicitly marked deprecated here
@property (nonatomic, readonly, copy, nullable) NSDictionary
*addressDictionary API_DEPRECATED("Use @properties", macos(10.8, 10.13),
ios(5.0, 11.0), watchos(1.0, 4.0));
                                                              ^
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:23:29: warning: 'addressDictionary' is
deprecated: first deprecated in iOS 11.0 - Use @properties
[-Wdeprecated-declarations]
            street = [[self addressDictionary] objectForKey:(NSString
            *)kABPersonAddressStreetKey];
                            ^
In module 'CoreLocation' imported from
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.h:10:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/CoreLocat
ion.framework/Headers/CLPlacemark.h:73:63: note: property
'addressDictionary' is declared deprecated here
@property (nonatomic, readonly, copy, nullable) NSDictionary
*addressDictionary API_DEPRECATED("Use @properties", macos(10.8, 10.13),
ios(5.0, 11.0), watchos(1.0, 4.0));
                                                              ^
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/CoreLocat
ion.framework/Headers/CLPlacemark.h:73:63: note: 'addressDictionary' has
been explicitly marked deprecated here
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:23:73: warning:
'kABPersonAddressStreetKey' is deprecated: first deprecated in iOS 9.0 - use
CNPostalAddress.street [-Wdeprecated-declarations]
            street = [[self addressDictionary] objectForKey:(NSString
            *)kABPersonAddressStreetKey];
                                                                  ^
In module 'AddressBook' imported from
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:8:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/AddressBo
ok.framework/Headers/ABPerson.h:113:29: note: 'kABPersonAddressStreetKey'
has been explicitly marked deprecated here
AB_EXTERN const CFStringRef kABPersonAddressStreetKey AB_DEPRECATED("use
CNPostalAddress.street");
                            ^
3 warnings generated.
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:22:18: warning: 'addressDictionary' is
deprecated: first deprecated in iOS 11.0 - Use @properties
[-Wdeprecated-declarations]
        if (self.addressDictionary != nil) {
                 ^
In module 'CoreLocation' imported from
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.h:10:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/CoreLocat
ion.framework/Headers/CLPlacemark.h:73:63: note: 'addressDictionary' has
been explicitly marked deprecated here
@property (nonatomic, readonly, copy, nullable) NSDictionary
*addressDictionary API_DEPRECATED("Use @properties", macos(10.8, 10.13),
ios(5.0, 11.0), watchos(1.0, 4.0));
                                                              ^
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:23:29: warning: 'addressDictionary' is
deprecated: first deprecated in iOS 11.0 - Use @properties
[-Wdeprecated-declarations]
            street = [[self addressDictionary] objectForKey:(NSString
            *)kABPersonAddressStreetKey];
                            ^
In module 'CoreLocation' imported from
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.h:10:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/CoreLocat
ion.framework/Headers/CLPlacemark.h:73:63: note: property
'addressDictionary' is declared deprecated here
@property (nonatomic, readonly, copy, nullable) NSDictionary
*addressDictionary API_DEPRECATED("Use @properties", macos(10.8, 10.13),
ios(5.0, 11.0), watchos(1.0, 4.0));
                                                              ^
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/CoreLocat
ion.framework/Headers/CLPlacemark.h:73:63: note: 'addressDictionary' has
been explicitly marked deprecated here
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:23:73: warning:
'kABPersonAddressStreetKey' is deprecated: first deprecated in iOS 9.0 - use
CNPostalAddress.street [-Wdeprecated-declarations]
            street = [[self addressDictionary] objectForKey:(NSString
            *)kABPersonAddressStreetKey];
                                                                  ^
In module 'AddressBook' imported from
/Users/rohan/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.5/ios/Classes/
Extensions/CLPlacemarkExtensions.m:8:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platfor
m/Developer/SDKs/iPhoneSimulator15.2.sdk/System/Library/Frameworks/AddressBo
ok.framework/Headers/ABPerson.h:113:29: note: 'kABPersonAddressStreetKey'
has been explicitly marked deprecated here
AB_EXTERN const CFStringRef kABPersonAddressStreetKey AB_DEPRECATED("use
CNPostalAddress.street");
                            ^
3 warnings generated.
1 warning generated.
1 warning generated.
../../.pub-cache/hosted/pub.dartlang.org/file-6.1.2/lib/src/interface/file.d
art:15:16: Error: The method 'File.create' has fewer named arguments than
those of overridden method 'File.create'.
  Future<File> create({bool recursive = false});
               ^
org-dartlang-sdk:///third_party/dart/sdk/lib/io/file.dart:232:16: Context:
This is the overridden method ('create').
  Future<File> create({bool recursive = false, bool exclusive = false});
               ^
../../.pub-cache/hosted/pub.dartlang.org/file-6.1.2/lib/src/interface/file.d
art:15:16: Error: The method 'File.create' doesn't have the named parameter
'exclusive' of overridden method 'File.create'.
  Future<File> create({bool recursive = false});
               ^
org-dartlang-sdk:///third_party/dart/sdk/lib/io/file.dart:232:16: Context:
This is the overridden method ('create').
  Future<File> create({bool recursive = false, bool exclusive = false});
               ^
../../.pub-cache/hosted/pub.dartlang.org/file-6.1.2/lib/src/forwarding/forwa
rding_file.dart:22:8: Error: The method 'ForwardingFile.createSync' has
fewer named arguments than those of overridden method 'File.createSync'.
  void createSync({bool recursive = false}) =>
       ^
org-dartlang-sdk:///third_party/dart/sdk/lib/io/file.dart:248:8: Context:
This is the overridden method ('createSync').
  void createSync({bool recursive = false, bool exclusive = false});
       ^
../../.pub-cache/hosted/pub.dartlang.org/file-6.1.2/lib/src/forwarding/forwa
rding_file.dart:22:8: Error: The method 'ForwardingFile.createSync' doesn't
have the named parameter 'exclusive' of overridden method 'File.createSync'.
  void createSync({bool recursive = false}) =>
       ^
org-dartlang-sdk:///third_party/dart/sdk/lib/io/file.dart:248:8: Context:
This is the overridden method ('createSync').
  void createSync({bool recursive = false, bool exclusive = false});
       ^
Failed to package /Users/rohan/Downloads/jakstatcourier.
Command PhaseScriptExecution failed with a nonzero exit code
note: Using new build system
note: Planning
note: Build preparation complete
note: Building targets in dependency order
warning: Capabilities for Signing & Capabilities may not function correctly
because its entitlements use a placeholder team ID. To resolve this, select
a development team in the Runner editor. (in target 'Runner' from project
'Runner')
/Users/rohan/Downloads/jakstatcourier/ios/Pods/Pods.xcodeproj: warning: The
iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0,
but the range of supported deployment target versions is 9.0 to 15.2.99. (in
target 'leveldb-library' from project 'Pods')

Result bundle written to path:
	/var/folders/dt/ymx6cbj16xq_flrs8l2xzc9r0000gn/T/flutter_tools.rzO49o/flutt
	er_ios_build_temp_dirHHxj9c/temporary_xcresult_bundle

Could not build the application for the simulator.
Error launching application on iPhone 13.
rohan@Rohans-Air jakstatcourier %

Hi @im-pch, thank you for filing a bug report!
Apologies for the late reply. I was wondering if you are still experiencing this issue on the latest version (2.1.0) of the geocoding plugin? I just tested the example app on an iOS 14 Pro simulator and did not encounter the error.


Create REST interface for endpoint

Steve Martin from Cray requested that there be a JSON based RESTful interface for updating the endpoint.  Our support for dynamic control is limited to the C and command line interfaces:
https://geopm.github.io/man/geopmendpoint.1.html
https://geopm.github.io/man/geopm_endpoint_c.3.html
We currently support static policies in JSON files on disk read at job start time:
https://geopm.github.io/man/geopmagent.1.html
https://geopm.github.io/man/geopm_agent_c.3.html
but a REST interface with JSON back end that can be dynamically modified at runtime would be nice to have and would allow easy integration with CAPMC.

To be clear, we want to be able to read samples through this interface, not just write policies.


More specific  description

There seems to be some confusion around this.
I think that the link will work (it works within GitHub), but at least for me, none of the single-dot relative paths work when running the documentation site locally.

👍


Packaging: Auto-generated modules are missing a setup.cfg

Was #4307.

This issue was moved to googleapis/toolkit#1676

Thanks Luke!


cz-cli: 4.2.4 -> 4.3.0

bump and repackage with buildNpmPackage
Description of changes
related #229910
nodejs_14 -> nodejs_18

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

cz-cli