support models of generic types

As a baby step toward support of generics, we should support writing models of generic library types and checking uses of those types against the models.  E.g., for a Pair<S, T> type, we should be able to write a model indicating the type is really Pair<@PolyNull S, @PolyNull T> and then allowing app code to write, e.g., Pair<@Nullable String, String>.  We would not initially check that the implementation of the Pair class respects the model.  Depending on implementation difficulty of figuring out how generic types are instantiated, we may require the models to be more or less explicit.

Couple notes based on some initial tinkering:

We'll need to handle assignments, e.g., you can't assign a Pair<@Nullable String, String> to Pair<String, String> (with both @NonNull).  Probably just want to support invariance.
Without inference, ergonomics will be a bit rough.  E.g., if you write Pair<@Nullable String, String> p = new Pair<>(null, "hello"); I don't think javac's inference will tell us that the first type parameter for the new expression is @Nullable.  Would be a bit unfortunate to have to write Pair<@Nullable String, String> p = new Pair<@Nullable String, String>(null, "hello"); but supporting inference could be a lot more work.

Overall I think adding basic support (even without inference) will be a moderate amount of work to get right.

FWIW Error Prone has some support for generics and null checking now, implemented by @bennostein.  See the inference code and this commit.  Perhaps we could do something similar in NullAway?


Update pt.json

Update portuguese from Portugal translation.

Thanks a lot. Ignore the failing Test job. But hassfest validation indicates the JSON syntax is invalid. probably a superfluous or missing comma somewhere.

Thanks a lot. Ignore the failing Test job. But hassfest validation indicates the JSON syntax is invalid. probably a superfluous or missing comma somewhere.

Think I got it!


Ensure meal exists in menu

What does this PR do?
Ensure meal exists in menu
Description of Task to be completed?
N/A
How should this be manually tested?
N/A
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?(if applicable)
N/A

Pull Request Test Coverage Report for Build 195

74 of 74 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 98.815%

Totals

Change from base Build 193:
0.1%

Covered Lines:
1339

Relevant Lines:
1345

💛  - Coveralls


Update deps to add support for ESLint 6

Hi @iamturns! what is the release cycle? When could we expect this to be merged and released?

Hey @efoken - thanks for this PR! And apologies for taking a few weeks to get back (CC @bailer  @danm)
It seems a couple of rules changed (very slightly) with the latest Airbnb version.
There are also many new rules within the latest @typescript-eslint version.
This required some additional work beyond updating dependencies. I've just pushed the changes into #17 and released v5.0.0.
You will also need to update your local app dependencies for these changes.


Adds labelDummies parameter to the Recap implementation

The original RECAP paper (https://doi.org/10.1021/ci970429i) describes the labelling of dummy atoms in correspondence to each cleaved bond with the index of the reaction that matched and cleaved the bond.
This PR adds a labelDummies parameter to the RDKit RECAP implementation to add this functionality.
While preparing this PR I realized that some of the SMARTS pattern are not sufficiently specific (e.g., the amine pattern also matches amides, the cyclic amines also matches aromatic amines, the cyclic amine pattern also matches lactams, etc.), so I tweaked them to only match what they are supposed to match.

After looking more carefully at the RDKit implementation and at the original paper, I have noticed that the RDKit implementation significantly deviates from the one described in the original paper: the reaction definitions are different and also in a different order, so it probably does not make sense to change it now, given that adhering more closely to the implementation described in the paper would change results.
@greglandrum Any thoughts about this? I think that having the dummies labelled by cleavage reaction would be useful in workflows where one wants to track down where the fragments come from. However, I think it would be better to submit a new Python script (e.g., RecapOriginalImplementation.py) where the reaction definitions and numbering match those in the original publication, and labelling of dummies would be available. This would also guarantee that results carried with the historical RDKit implementation would not change.

@ptosco : I did the original implementation long enough ago that I no longer remember the reason for the deviation from what's in the original RECAP paper.
Rather than adding another script/function, how about just adding an additional argument useLegacyReactions to RecapDecompose() which defaults to true? When it's false then your updated definitions would be used.

@greglandrum, that's a good idea, I will do that :-)


[v0.3-DeNas]common trainer

Completed the common trainer refactor. The test works well for CNN trainer. Please help to review and merge. @xuechendi

@csdingbin , I have went through this PR, approved.
But there are two small conflict, I tried to resolve today, but in that way, I will need to push to your Repo, so please help to resolve then I'll merge this PR.

@csdingbin , I have went through this PR, approved. But there are two small conflict, I tried to resolve today, but in that way, I will need to push to your Repo, so please help to resolve then I'll merge this PR.

The conflicts resolved.


@craigspaeth => [Venice] Cleaner initial load on episodes 2+

Since the carousel dictates routing on client-side, going to https://www.artsy.net/venice-biennale/episode-2 (Episode 2) feels a little clunky since Flickity always "shows" the first item before going to the right index. This problem is better understood visually:
Problem:

Solution:

Nice. Simple + clever fix! Feel free to merge when 💚


Need a more robust solution for name collisions within dart:html and JS macros implementing it

See https://codereview.chromium.org/2587203002/
which works around the issue for the Blob class by prefixing window. for a couple constructors in
dart:html that reference types with names that conflict with dart:html class names.

Obsolete. Focus going forward should be on a robust publicly available JS interop scheme versus improving JS macros in the SDK.


Feat: Upload multiple files, upload directory

Hey! I love this uploader, and now that I've been learning Go I thought I'd give adding some functionality a shot. I'm sorry if I'm doing something wrong - I'm new to contributing. Please let me know if there's any issues and I'll handle them :)
If you do decide you'd like to merge the request you'll have to let me know how you generate the CLI usage screenshot! I couldn't figure out what tool you used to update the README.

New Features

Added a -dir flag that accepts a directory then uploads all files inside the directory (non-recursive... maybe later haha)
Using the same update, it was easy now to read multiple args instead of a single one and plug it into the directory upload code

To make the -dir flag work, the script now loops all args/directory files first and for each of those it runs the upload code and stores them in a slice for later. Once done, if there's only one file uploaded it'll print the upload URL and copy it to the clipboard, but if there's multiple files it'll print a table with the file name and the uploaded URL and store them in a log file.
I've done some small refactors where necessary, like updating deprecated function calls and moving some code to separate functions to make reading the main loop easier.

That's it I think. Code should be pretty self-explanatory but, like I said, I'm new to Go so if you have any questions or suggestions etc please let me know. This is as much a learning experience for me as it is improving a dope tool.

Hey, first I wanted to thank you for this pull request.
Currently I am off for holidays, but I'll be free to review the changes next week.
Wish you a Happy New Year!


Fixed naming convention for src/bebone/core/input files

will look at this, tomorrow

As Iremember we decided to use Pascal case for enums


Update doc Jamfile

OOHHH... so, the Jamfile when the repo is in CPPAlliance is different from when it becomes integrated into Boost? Because JSON and URL don't have the "broot" thingy

Thanks!


Refactor Ratio Object + Improve Cross Browser Support

Re-enabled including the ratio object JavaScript in the Bolt and Pega.com D8 JS bundles

Simplifies and cleans up the ratio object Sass partial

Fixes some gaps in the original JS logic + significantly improves cross browser support in browsers that don't fully support Shadow Dom (ie. IE 11) -- now visually much more consistant across the board!

Adds new aspectRatioHeight and aspectRatioWidth parameters that can get passed into the ratio object for configuring the dimensions directly (which the original didn't have) -- updates the Image component to now take advantage of this these new params

CC @theSadowski @EvanLovely @mikemai2awesome

** I totally forgot to mention it but this update also addresses a sizable number of reported issues relating to missing images / videos showing up across a handful of devices / browsers due to not having a fallback method of calculating the aspect ratio dimensions + not polyfilling the rendering of this.


[FEATURE] Installation guide in README

🚀 Feature request
Is your feature request related to a problem?

 What do you think about adding an installation section in README file? It would speed up the installation :) (Yeah, I prefer to copy these things)
Describe the solution you'd like

npm: npm install @ng-web-apis/common --save
yarn: yarn add @ng-web-apis/common
ng add: ng add @ng-web-apis/common@latest

Thanks for this library. I would like to also see a small example how to work with it in unit tests. I am not very familiar with angular


Feature/501 dependency tree for dotnet

Closes #501
Adds parsing of dependency info from package.assets.json targets, including target dependency dependencies.

Hi. Could you rebase your branch and squash into a single commit?
git rebase -i `git merge-base origin master`

# change pick from second line onwards to s
# Fix commit message
git push origin feature/501-dependency-tree-for-dotnet -f

Hi, I was just trying this out and wanted to know if this is expected behaviour.
My .csproj file
<Project Sdk="Microsoft.NET.Sdk.Web">
  <PropertyGroup>
    <TargetFramework>net7.0</TargetFramework>
    <Nullable>enable</Nullable>
    <ImplicitUsings>enable</ImplicitUsings>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Server.IIS" Version="2.2.6" />
  </ItemGroup>
</Project>

but the bom generated looks like this MyWebApp (top level pkg)
{
      "ref"<EMAIL_ADDRESS>      "dependsOn": [
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>      ]
    }

I would have assumed that the top level package to look like this so that you can understand which are direct and which are the transitive deps.
    {
      "ref"<EMAIL_ADDRESS>      "dependsOn": [
<EMAIL_ADDRESS>      ]
    }

Thank you!

@noqcks We parse the projects.assets.json file to construct the tree. Could you check the dependencies section in that file? Also, would you be able to check if another independent tool, such as
https://github.com/bjorkstromm/depends

Looking at the projects.assets.json file for my project, I see we can extract Microsoft.AspNetCore.Server.IIS as a direct package dependency.
    "frameworks": {
      "net7.0": {
        "targetAlias": "net7.0",
        "dependencies": {
          "Microsoft.AspNetCore.Server.IIS": {
            "target": "Package",
            "version": "[2.2.6, )"
          }
        },
        "imports": [
          "net461",
          "net462",
          "net47",
          "net471",
          "net472",
          "net48",
          "net481"
        ],
        "assetTargetFallback": true,
        "warn": true,
        "frameworkReferences": {
          "Microsoft.AspNetCore.App": {
            "privateAssets": "none"
          },
          "Microsoft.NETCore.App": {
            "privateAssets": "all"
          }
        },
        "runtimeIdentifierGraphPath": "/usr/local/share/dotnet/sdk/7.0.400/RuntimeIdentifierGraph.json"
      }
    }
  }

Looking at dotnet depends will surface them as well (but through the .csproj file I suppose)

I suppose what im looking for is a way to have a distinction between direct dependencies, and all dependencies for a .NET application. This PR only support the latter from what I can see.
Would be happy to create an issue and PR support for the former

@noqcks, thank you. Sounds like a good proposal. @cerrussell, any thoughts on whether this is achievable?

@prabhu I added a PR here for @cerrussell to review


Add redirects to new documentation?

Not sure when the move was made, but I see now a lot of 404s when referencing QCoDeS pages.
e.g. https://qcodes.github.io/Qcodes/api/instrument/index.html gives a 404

and it looks like everything is moved to microsoft.github.io/...
Could we maybe add redirects from qcodes.github.io/Qcodes to microsoft.github.io/Qcodes?

Unfortunately a straightforward redirect based on CNAME etc is not possible with github pages however each html page should now be redirected using a Jekyll layout. The code doing the redirect lives in
https://github.com/QCoDeS/qcodes.github.io/blob/master/_layouts/redirect.html
https://github.com/QCoDeS/qcodes.github.io/blob/master/_config.yml
and
a copy of the qcodes source where each page html page is replaced by a blank page containing only a reference to the redirect layout. This will unfortunately not handle intersphinx objects.inv files but I think this is the bese we can do given the limitations of ghpages
For reference the blank pages were generated with
from pathlib import Path

p = Path("original")
myglob = p.glob("**/*.html")

for file in myglob:
    print(file)

    new_parts = ("blank_docs",) + file.parts[1:]

    new_file = Path("blank_docs", *file.parts[1:])
    print(new_file)
    new_file.parent.mkdir(exist_ok=True, parents=True)
    with new_file.open("w") as myf:
        myf.write(
            """---
layout: redirect
---
"""

from a copy of the existing qcodes docs

amazing. thanks! :)


Bugfix: broken save as template link

The change adds a special jquery selector and function to stop event propagation.  Using it on the app settings menu link to save app as template fixes the bug, allowing the browser to load the save as template view.

:smile_cat: :+1:

:+1:


Tool: ACGTraversal won't let me select source

Make selection the source does not work.
The field 'Id' is not shown as well.

Yes, it will also be put on the backlog.
There were some regression issues when I migrated from W2UI 1 to 2.
I will very quickly address this

I made the correction on the tool pane in order to have the source field displayed and useable?


Need climate log file

We have an excessive number of output statements in the landis-log file related to climate so sometimes you have to scroll down a long way, just to get to your error message.  We should create a new landis-climate-log file that stores all the climate-related statements.

Moved issue to the Climate Library repository.


More Features to WeatherData(Vec/Line)

I included an option to use the indexing operator, and to get by regex as well as to declare directly from filename

My kind request


Feature Request: Keep track of weight of the bag's content on the bag's owner's sheet

The module seems to be really useful so far, but I would like to see it properly tracking the character's carrying capacity.
If a container does not have weightless contents check, it's weight, or something that would represent the weight of the items inside  should update to keep track of the weight of everything in it.
And when the container has Weightless Contents checked, it should return to just the container's weight, ignoring the weight of the items in it.

This SOOOOO much!

The module seems to be really useful so far, but I would like to see it properly tracking the character's carrying capacity. If a container does not have weightless contents check, it's weight, or something that would represent the weight of the items inside should update to keep track of the weight of everything in it. And when the container has Weightless Contents checked, it should return to just the container's weight, ignoring the weight of the items in it.

I've not come up with an idiot-proof and clean solution for this. If you have any thoughts on how it would be implemented, I won't mind hearing it. Libwrapper won't be used.

Here is just my 2 cp on this cause I'd love to see it implemented also.
Add an item that you don't display in your ui that weighs the same as the initial weight, ie 5 lbs for a backpack.  Then when ever your UI changes change the item in the players inventory to the total weight.  If weightless is checked you just skip the update or for safety sake update the item weight in inventory to the base weight you set prior.  Oh and obviously ignore your hidden item when totaling the weight of the bag contents in your UI but include it the the total when bag is updated in the player inventory.
Along these lines I'd also like to see the weight limit of the bag be enforced some way.  Perhaps not allowing the transfer to the bag if it pushed the bag beyond it's limit or turn the background of the bag contents red to show its overstuffed.
Bob

Just came here to request this same thing and saw it was already raised. I'll immediately hop over from using the Item Collection module if this is ever implemented.

 I wanted to just ask if there is any news with this. I usually used @p4535992 Variant Encumbrance+Midi module, but I got a new computer and I migrated everything to the newer versions when I downloaded Foundry, so I cannot use their module now and the backpacks not tracking their total weight on the owner sheet is a pain for me now, since I was used to it. Thank you beforehand!

The module seems to be really useful so far, but I would like to see it properly tracking the character's carrying capacity. If a container does not have weightless contents check, it's weight, or something that would represent the weight of the items inside should update to keep track of the weight of everything in it. And when the container has Weightless Contents checked, it should return to just the container's weight, ignoring the weight of the items in it.

I've not come up with an idiot-proof and clean solution for this. If you have any thoughts on how it would be implemented, I won't mind hearing it. Libwrapper won't be used.

Actor Update Hook on the backpack actor updating weight of backpack item on the owner actor?


Update build-first-sharepoint-adaptive-card-extension.md

Category

[ x] Content fix
[ ] New article
[ ]

What's in this Pull Request?
Adding --plusBeta to scaffolding instructions.

Docs Build status updates of commit a8d0e33:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/viva/get-started/build-first-sharepoint-adaptive-card-extension.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Turn off fail fast

When fail fast is on, any errors that occur when deploying a flow result in all other flows being deployed cancelling.
When a second commit is pushed fixing the previous error, the cancelled flows aren't deployed as they no longer have “new” code in the recent commit.
By setting fail-fast: false it will stop the auto cancelling behaviour and allow flows to be deployed irrespective of another failing.

On a side note, had not used matrix before in a GitHub action until using your template. Super helpful!!! Thank you for the tip :)

thanks for the PR and sorry for getting late to you on that, I didn't get a notification somehow

fail-fast: false it will stop the auto-cancelling behaviour and allow flows to be deployed irrespective of another failing

I very intentionally disabled that default behavior so that this one failed step doesn't hold other jobs. You can change that in your setup, though - for tests fail-fast: true is desirable bc if one test fails, no need to run other ones, but in my case, I don't want that one bad flow to destroy the build pipeline for my entire data platform - tradeoffs


Pressing enter in the search box should do the appropriate search!

Right now it redirects to?q=searchterm. Frankly, this is the most appropriate way to do things on the web -- the /search/searchterm is kind of a weird way to do things, but either way, the ?q=searchterm should, in fact, execute the appropriate search. Related to #210.

@eddieantonio It looks like:

pressing the enter key, after inputting a string in the search box, such as mici, does result in that string being searched (and matched), or
one ends up on the landing page, if there is no match.
also, matching is done with every input character even without pressing the enter key, and
if one presses the enter key after a string that isn't a wordform, like mi or mic (even taking into account the spelling relaxation), one ends up on the landing page.

So, isn't this working as it should?

But now I notice that pressing enter with a legitimate search string results in a match, but so that the (i) info-button no longer produces a pop-up window for the linguistic breakdown nor the preverbs, if apparent. So, there is a vestigial bug.

I've changed this issue to the following:
Search should follow the conventions of other websites, like

Google: https://www.google.com/search?q=foo
GitHub: https://github.com/search?q=foo
mdn https://developer.mozilla.org/en-US/search?q=foo
DuckDuckGo https://duckduckgo.com/?q=foo

etc.
Why? Mostly because my browser picks up on it automatically:

Except my browser thinks the search page is /search/herp?q=! 😩


[NT-1039, NT-1042] ProjectSummary query and experiment activation

📲 What

Adds the native_me_project_summary experiment and activation.
Adds the GraphQL query and de-serialization for the fetching of project summary items.

🤔 Why
A continuation of the work in #1135, we are planning to run an experiment to display horizontally scrollable project summary cells on the project page.
🛠 How

Added deserialization for ProjectSummaryEnvelope which contains an array of ProjectSummaryItems.
Added some safe-guards to ignore ProjectSummaryItems that might have unsupported questions. These will just be discarded during de-serialization.
Added activation of the native_me_project_summary experiment. This behaves similarly to #1093 in that we first attempt to load the project summary items and if that request fails we don't activate the experiment.

✅ Acceptance criteria
This is a little tricky to test because the UI is not completely wired up. I will do that in a subsequent PR to keep this one from growing.
Best bet is to create a project on staging that has project summary items. Navigate to that project and observe using logEvents statements that the following are true.

[ ] If you are in the experimental variant and project summary items are being returned, these are returned from loadProjectPamphletContentDataIntoDataSource in ProjectPamphletContentViewModel. The experiment is activated.
[ ] If you are in the control group and project summary items are being returned, loadProjectPamphletContentDataIntoDataSource emits and empty array for the project summary items. The experiment is activated.
[ ] If you are in the experimental variant and project summary items are not being returned, loadProjectPamphletContentDataIntoDataSource emits and empty array for the project summary items. The experiment is not activated.
[ ] If you are in the experimental variant and the project summary call errors, loadProjectPamphletContentDataIntoDataSource emits and empty array for the project summary items. The experiment is not activated.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger


Had to run with sudo

I assume it's an issue because the Readme does not mention execution with SUDO. This is what happens when running without SUDO using PHP7 in Ubuntu 16.04:

The script works when running with SUDO

Did you check to make sure the user has write access to the folder you have specified under "local_path" in the config.ini? It looks like your just using the laracasts-downloader folder so you should make sure it is writable or change the user:group to match the user launching the script.
Cheers.

That was the issue! My fault. Thanks!


Crash in function Aws::Monitoring::GetHttpClientMetricNameByType

What platform/OS are you using?
Mac OS X 10.13.3
What compiler are you using? what version?
clang
What's your CMake arguments?
Can you provide a TRACE level log? (sanitize any sensitive information)
trace_sync1.txt
We are facing with crash issue. Please take a look in the stack trace. I'm not sure the bug from your side or something wrong when we implement aws-sdk.
We are using version 1.7.71

from stack trace. it crash in this function
auto it = metricsTypeToName.find(static_cast(type));
I'm thinking function find() of static map is not thread safe.

STL map spec doesn't guarantee thread safety of find. But the general sense of find is just reading information for the tree. So I think most  probably it's thread safe for concurrent readers. The only problem I see from here is concurrent calls to that function. One thread is initializing the map during first call and another thread call that function during the initialization. Could you please give more information about your use case or your sample code? Or could you please verify this is the root cause?

Yes, I tried to move static map outside the scope of functions and seems it's not crash. I'm waiting response from our QA and will send you the result soon.

Update, still get crash with solution above. I need to use local variable in the function Aws::String GetHttpClientMetricNameByType. and after pass previous crashing, we get another crash
CortexSync_2019-03-27-164258_Cuongs-MacBook-Pro-2.crash.zip
assertion failed: (s_monitors), function OnRequestSucceeded
@singku do you have any idea ?

another crash , now from Assertion failed: (s_HttpClientFactory), function CreateHttpRequest
CortexSync_2019-03-27-165656_Cuongs-MacBook-Pro-2.crash.zip

Ah, I found the root cause. You have a static unique pointer in MonitoringManager.cpp. and this pointer is used in multithreading.
when call function Init . this pointer is created in thread 0x10cedf5c0
but in function callback OnRequestSucceeded or OnRequestFailed
you check this pointer on another thread "Monitors check  on thread 0x7000094a0000"
I think amazon already support multithread request ?
and this is a bug on your side ?

After look at in the document. I don't see anyway to stop thread download before call ShutdownAPI().
except call requestPtr->WaitUntilFinished(); for each request
Do you have any idea ?

In the logical level, you have to make sure all threads finished their job before calling ShutdownAPI. For this case, you can call requestptr->Cancel() to stop threads from further downloading. Then to call requestptr->WaitUntilFinished() to make sure threads are done. Then to call ShutdownAPI

okay, look like previous crash and new crash have same the cause. I tried to log size of map before call function metricsTypeToName.find(static_cast(type)); and it always output size = 9. that mean crash happens in destroy time, not in initial time.
I think should not use static variable in multithreading. it's not thread safe and have many problems behind.

Global (static) variables are initialized before program starts. Local static variables are initialized when program first hits the declaration and concurrent call should wait until initialization finished. In either case, it's thread safe.
Since you have found the root cause, and it's caused by lingering thread accessing aws related memory after shutdownAPI. I am going to close this issue.


ERROR - non-monotonic data-flow analysis

The following code:
/**

@private
@param {org.apache.flex.textLayout.elements.SpanElement} firstSpan
@param {org.apache.flex.textLayout.elements.SpanElement} lastSpan
@param {string} styleType
@return {boolean}
/
test = function(firstSpan, lastSpan, styleType) {
if (firstSpan == null || lastSpan == null)
return false;
if (firstSpan == lastSpan)
return false;
var /* @type {org.apache.flex.textLayout.elements.SpanElement} */ curSpan = firstSpan.getNextLeaf();
while (curSpan) {
if (firstSpan.getStyle(styleType) != curSpan.getStyle(styleType))
return true;
if (curSpan == lastSpan)
break;
curSpan = curSpan.getNextLeaf();
}
return false;
};

results in "non-monotonic data-flow analysis" error.  In stepping through it in the debugger, the flow() call in DataFlowAnalysis.java:analyze() keeps thinking that the last line in the while loop is affecting the top of the while loop so it adds the while line to the orderedWorkSet and you keep looping until you hit the MaxIterationsExceededException
I've gotten around it by specifying the type of curSpan to be ISpanElement instead.  But is there something wrong with our code that the DataFlow can't get out of the loop?
TLFElements.zip

There is not enough information here. Please provide exact steps how to build this code to reproduce the warning.


arch: arm: re-program MPU stack guard upon userspace enter

Because before entering userspace, the MPU stack guard is
configured to guard the default (i.e. afterwards, the "user")
thread stack, as the privileged stack has not yet been set. Upon
entering userspace the MPU stack guard needs to be re-programmed
to the privileged stack, otherwise, there is a risk that the
first system call might be serviced without privileged stack
protection, if it occurs before any context-switch. This commit
fixes this issue, by calling configure_mpu_stack_guard(..) upon
userspace entering.
Signed-off-by: Ioannis Glaropoulos<EMAIL_ADDRESS>

Codecov Report

Merging #10467 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10467   +/-   ##
=======================================
  Coverage   53.23%   53.23%
=======================================
  Files         210      210
  Lines       25800    25800
  Branches     5677     5677
=======================================
  Hits        13735    13735
  Misses       9755     9755
  Partials     2310     2310

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 ed5139a...b86bf9a. Read the comment docs.

@AdithyaBaglody updated. Pls, re-review.

Temporarily adding a DNM, to do some additional testing.

Rebased (and removed the DNM)


Setting the user-agent

Is it possible to set the user agent string when using this wrapper? This is encouraged by Mailchimp so they can identify our integration in their logs.
Thanks in advance!

Currently the user agent string is set in the MailchimpConnection class when preparing the file handler. You right, having that set is helpful to troubleshoot issues when communicating with Mailchimp support, as well it allows Mailchimp to keep metrics around use of this library. That is why we chose to include this repositories URL in the user agent string. I'm hesitant to remove that or allow it to be altered easily when using versions of this library, as we might loose those benefits.
That said if you are looking to alter that user agent string when calling Mailchimp's API through this library you could fork this repository and make edits in your fork.

But the user is losing the benefits of troubleshooting their specific connection with Mailchimp. How are those metrics more important than that? I feel like this should really be a user set attribute, should it not?

Apologies. I had another comment, but it came across snarky so I deleted it. It's early in the morning and I'm still sleepy

😄 No worries, I know how that goes. Since I wrote that last response I did have the thought that we could add a function that appends to the user agent string after a delimiter like a ;. That way string matching can be done for both on the Mailchimp end.

That would be helpful. You could possibly look at having a second parameter in the constructor to pass through additional options?

Hi @Jhut89, any progress on this? Or any final ideas on how you'd like to achieve this? I could "try" do a PR for it?

Hey!
So sorry it’s taken a while! Work got busy. 😆I have some time off later
this week for a holiday and will likely be able to add this then.
If you want to give it a try please feel free! No pressure though.  I don’t
want to be super prescriptive about how to make it happen, but I would ask
we limit it to additive changes.
Best wishes,
Hutch
On Sun, Nov 24, 2019 at 8:31 PM Lee<EMAIL_ADDRESS>wrote:

Hi @Jhut89 https://github.com/Jhut89, any progress on this? Or any
final ideas on how you'd like to achieve this? I could "try" do a PR for it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Jhut89/Mailchimp-API-3.0-PHP/issues/69?email_source=notifications&email_token=ADE7NUERH2KVO6FM3ALZMZTQVMTG7A5CNFSM4JINBX7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFA3FKQ#issuecomment-557953706,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADE7NUCRQDPYHOAAU7I3ZLLQVMTG7ANCNFSM4JINBX7A
.

Hi @Jhut89!
Created a PR for you now. The changes are pretty minor so hopefully, you're happy with the implementation.
Lee

Just to catch anyone up reading this thread, we resolved this in #70


test: add typescript e2e test

Add a small TypeScript example that will also be used in e2e tests - so we know the published package does not have any export breakages.

Codecov Report
Patch and project coverage have no change.

Comparison is base (46a3e24) 86.10% compared to head (70b0fde) 86.10%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #800   +/-   ##
=======================================
  Coverage   86.10%   86.10%
=======================================
  Files          19       19
  Lines         662      662
  Branches      151      151
=======================================
  Hits          570      570
  Misses         92       92

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


Add support for oracle

Related to the bugfix in train (used by inspec):
https://github.com/chef/train/commit/6b5d58250c7fd5d9f8e21ca7e27ca400ffed6b5a

@atomic111 @chris-rock it would be nice to review this one:) it would fix the broken master of chef-ssh-hardening

Thank you @artem-sidorenko


conf http(s) proxy load failure & ariang always use http://<IP_ADDRESS>:10809/ as http(s) proxy

I used the chrome extension Aria2 for Chrome version 1.5.6$ aria2c -v
aria2 version 1.36.0

my aria.conf...
http-proxy=http://<IP_ADDRESS>:10889
https-proxy=https://<IP_ADDRESS>:10889
all-proxy=http://<IP_ADDRESS>:10889
...

load in bash of mingw-64 byaria2c --conf-path=/c/Users/czg/Desktop/automatic_command/aria2.conf

when use http-proxy=http://<IP_ADDRESS>:10889 , it will throw error  Parse error in C:/Users/czg/Desktop/automatic_command/aria2.conf
  Exception: [AbstractOptionHandler.cc:69] errorCode=28 We encountered a problem while processing the option '--http-proxy'.
    -> [OptionHandlerImpl.cc:520] errorCode=1 unrecognized proxy format

in AriaNG
before changing proxy manually in the browser

http proxy unchanged with the default value maybe

throw error  Exception: [AbstractCommand.cc:312] errorCode=1 Network problem has occurred. cause:No connection could be made because the target machine actively refused it.

  04/22 20:16:57 [NOTICE] Download GID#083aee5962ecfbc7 not complete:

all-proxy changed

after changing

bash output with success

04/22 20:19:19 [NOTICE] CUID#14 - Redirecting to https://objects.githubusercontent.com/github-production-release-asset-2e65be/13021798/c192f24f-9069-4705-a12f-ab90dcb7a89c?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20220422%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220422T121919Z&X-Amz-Expires=300&X-Amz-Signature=ae8ff7ce88be00f25704ec51b6cc65d3adb3d60510a66860dda5b8fff40c07ad&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=13021798&response-content-disposition=attachment%3B%20filename%3Dgit-lfs-windows-v3.1.4.exe&response-content-type=application%2Foctet-stream
 *** Download Progress Summary as of Fri Apr 22 20:19:19 2022 ***
======================================================================================================================
[#32ae00 0B/0B CN:1 DL:0B]
FILE: https://github.com/git-lfs/git-lfs/releases/download/v3.1.4/git-lfs-windows-v3.1.4.exe
----------------------------------------------------------------------------------------------------------------------

[#32ae00 0B/0B CN:1 DL:0B]
04/22 20:19:20 [WARN] Gaining privilege SeManageVolumePrivilege failed.

04/22 20:19:20 [WARN] --file-allocation=falloc will not work properly.

04/22 20:19:20 [NOTICE] GID#32ae00049a5a81fd - Download has already completed: D:/aria_config/git-lfs-windows-v3.1.4.exe

04/22 20:19:20 [NOTICE] Download complete: D:/aria_config/git-lfs-windows-v3.1.4.exe

04/22 20:19:23 [NOTICE] Serialized session to 'D:\aria_config\aria2.session' successfully.

questions

So anyone can tell me how change the default http(s) proxy by conf

although it is valid by passing argument in command linearia2c --enable-rpc --rpc-listen-all --http-proxy="http://<IP_ADDRESS>:10889"

If you use Aria2 for Chrome instead of AriaNg, please go to the support page of Aria2 for Chrome for help


Add tests that show #409

Apparently, both the pickler and unpickler that are previously returned from lookup{Un}Pickler are created by a generator stored in genPicklerMap. I'm actually debugging why this is happening in the first place and where we put that generator (which is a generator of List). Also, I guess that these generators are used only because of reflection and we do use them also for generating static picklers, but we shouldn't. And if we do, we should cache it correctly and make it visible so that implicitly[Pickler[List[String]]] doesn't trigger the standard generation. I'm trying to find a slick way to solve this issue.

My diagnosis: when we look a pickler up, it doesn't only look it up in the picklerMap but the genPicklerMap. Before querying the last one, we extract the top level type and use it as the key, e.g. from List[String] we get List. Then, the lookup succeeds and returns the generator for List, which doesn't correspond to Pickler[List[String]]].
To solve this problem it's necessary to register the picklers/unpicklers that are created from the generators. Therefore, next time someone looks them up they will be in picklerMap. Also, I've added two new methods in DefaultPicklerRegistry that only query picklerMap/unpicklerMap, ignoring the generators.

Tests passing thanks to #411. Review by @jsuereth.


merge from template-sync/template-arlac77-github

LICENSE

chore(license): update from template

README.md

docs(README): update from template

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.185%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
298

Relevant Lines:
348

💛  - Coveralls


ci: auto update readme

Description
This pull request changes the following:

Auto update example output part of Readme
Creating a template file README.md.template from existing README.md, and  adding some environment variables to replace previous example outputs sections
Saving command line outputs from solo/kind to environment variables.
Then using linux utility envsubst to replace these pre-defined environment variables in README.md.template
Only changing README.md if the number of changed lines greater than a certain threshold
Only trigger trigger the workflow if source code changes

Related Issues

Closes #121

I was able to remove lines contain these unwanted special character as well as escape color code,
but still the output no longer have indent to showing subtask status.
Example, the line started with Check dependency: helm
Before
✔ Setup home directory and cache
✔ Check dependencies
  ✔ Check dependency: helm [OS: darwin, Release: 23.3.0, Arch: arm64]
✔ Setup chart manager [0.9s]
✔ Copy configuration file templates

After
❯ Setup home directory and cache
✔ Setup home directory and cache
❯ Check dependencies
❯ Check dependency: helm [OS: darwin, Release: 23.3.0, Arch: arm64]
✔ Check dependency: helm [OS: darwin, Release: 23.3.0, Arch: arm64]
✔ Check dependencies
❯ Setup chart manager
✔ Setup chart manager
❯ Copy configuration file templates

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@release-0.24.x dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Export writeQueryToStore and writeFragmentToStore.

TODO:

[x] Update CHANGELOG.md with your change
[ ] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass

It'd be nice to import writeQueryToStore and writeFragmentToStore directly.

Great idea!

Released in 0.3.20!


"(0 , P.randomUUID) is not a function" error when doing almost anything

I'm receiving "(0 , P.randomUUID) is not a function" when I do almost anything with this extension.
Some googling suggests it is related to nodejs crypto and old node versions so I tried updating node, I tried deinstalling and reinstalling the Azure Tools pack, but nothing is working.
Have you seen this before?

What version of VS Code are you using? This may be fixed by updating VS Code.

 I had stopped VS Code from auto updating. Once I did it is fixed. Thanks!


Investigate Azure container options

Github directly supports deployment to

Azure Web App
Azure Functions App.

Via third party Github Actions (Microsoft) deployment to Azure Container Instance is also supported (https://learn.microsoft.com/en-us/azure/container-instances/container-instances-github-action?tabs=userlevel)
See this for comparison between ACS and Azure web apps

https://stackoverflow.com/questions/47232013/difference-between-azure-container-service-and-web-app-for-containers#47233235

See this for comparison between ACI and ACA

https://serverfault.com/questions/1083358/difference-between-azure-container-instances-and-azure-container-apps

Broader MS comparison of Azure container based services

https://learn.microsoft.com/en-us/azure/container-apps/compare-options
From this, it appears Azure App Service and Azure Container Instance both suit our needs.

As always, the Azure naming is ambigous and it is not immidiately clear whether Azure Web App and Azure App Service is the same thing. Looking into it, they are in fact the same thing (https://stackoverflow.com/questions/66460146/what-is-the-difference-between-azure-web-app-and-azure-app-service).
The immidiate conclusion is:

we can use Azure Web App/Azure App Service or ACI for hosting Aggregator
Github supports deploying to both

Hi. I will re-open this ticket because I think the scope is wider than Github and Azure possibilities. I think the scope should also include what is possible in the target enterprise system. I will report on this in the morning meetings when I have made progress.

I have investigated the following:

Deployment to Azure Container Instance
Deployment to Azure Web App
Deployment to Azure Virtual Machine

I think option 1 is optimal for testing and development and that option 3 is optimal for production deployment.
Currently, an Azure Container Instance is running at http://github.openpectus.org/ for development purposes. A virtual machine on the closed enterprise network is also operational for "production" use.
Br Eskild


Chef 16: Remove support for writing encrypted data bags version 1 and 2

In Chef version 11.6 we introduced encrypted data bag format version 3.
Chef client can read all formats and writes out version 3 by default. Support for writing to version 1 and 2 remains and can be used if the config.rb/knife.rb is set to the older versions. We should remove support for encrypting (not decrypting) version 1 and 2. This would mean that a knife user would only be able to write encrypted data bags that could be read by Chef 11.6 or later.
The downside as far as testing goes it we test the decryption method by first encoding data. We'd lose the specs for reading versions 1 & 2, just like we did with version 0 when we removed that support.

https://github.com/chef/chef/pull/8865

I think we determined this wasn't worth the effort. We'd need to make a system to migrate to the new format and that was a pretty large undertaking. Closing this out for now. If someone wants to make a magical setup that reads out legacy data bags and writes them out at v3 you'd be the coolest person ever.


[test] add Kamelet errorHandler test

Release Note
NONE

@squakez, @astefanutti  the CI fails with
 logs:
Error:     dump.go:142:     integration
Error:     dump.go:174:     > 	at org.apache.camel.reifier.RouteReifier.doCreateRoute(RouteReifier.java:236)
Error:     dump.go:174:     > 	... 31 more
Error:     dump.go:174:     >
Error:     dump.go:174:     > 2021-07-23 11:25:32,186 ERROR [io.qua.run.Application] (main) Failed to start application (with profile prod): org.apache.camel.NoSuchBeanException: No bean could be found in the registry for:  defaultErrorHandler of type: org.apache.camel.ErrorHandlerFactory
Error:     dump.go:174:     > 	at org.apache.camel.support.CamelContextHelper.mandatoryLookup(CamelContextHelper.java:241)
Error:     dump.go:174:     > 	at org.apache.camel.model.errorhandler.ErrorHandlerHelper.lookupErrorHandlerFactory(ErrorHandlerHelper.java:82)
Error:     dump.go:174:     > 	at org.apache.camel.reifier.errorhandler.ErrorHandlerRefReifier.lookupErrorHandler(ErrorHandlerRefReifier.java:41)
Error:     dump.go:174:     > 	at org.apache.camel.reifier.errorhandler.ErrorHandlerRefReifier.createErrorHandler(ErrorHandlerRefReifier.java:35)

the generated KameletBinding is
spec:
  errorHandler:
    dead-letter-channel:
      endpoint:
        properties:
          loggerName: kameletErrorHandler
        ref:
          apiVersion: camel.apache.org/v1alpha1
          kind: Kamelet
          name: my-own-log-sink
  sink:
    properties:
      loggerName: integrationLogger
    ref:
      apiVersion: camel.apache.org/v1alpha1
      kind: Kamelet
      name: my-own-log-sink
  source:
    properties:
      message: throw Error
    ref:
      apiVersion: camel.apache.org/v1alpha1
      kind: Kamelet
      name: my-own-error-producer-source

   spec:
          configuration:
          - type: property
            value: 'camel.beans.defaultErrorHandler = #class:org.apache.camel.builder.DeadLetterChannelBuilder'
          - type: property
            value: camel.beans.defaultErrorHandler.deadLetterUri = kamelet:my-own-log-sink/errorHandler
          - type: property
            value: camel.k.errorHandler.ref = defaultErrorHandler
          - type: property
            value: camel.kamelet.my-own-error-producer-source.source.message = throw Error
          - type: property
            value: camel.kamelet.my-own-log-sink.errorHandler.loggerName = kameletErrorHandler
          - type: property
            value: camel.kamelet.my-own-log-sink.sink.loggerName = integrationLogger
          flows:
          - from:
              steps:
              - to: kamelet:my-own-log-sink/sink
              uri: kamelet:my-own-error-producer-source/source
          profile: OpenShift

The most important message is that the test is working on 1.4.0. Do you know what could be wrong?

Yeah, I bumped into that issue as well while working on something different. I am investigating and will report more details shortly.

Tracked here #2525

Let's wait to merge #2564 and we can rebase this.

#2564 is merged - can we proceed with the rebase?

#2564 is merged - can we proceed with the rebase?

Yes please.

@squakez Two CI workflows failed but it seems that it's not my fault. Can you rerun them?


[BUG] auto-restore-last-session not working

Describe the bug
When I open nvim with the auto-session plugin enabled and the config option auto-restore-last-session, the last session is not restored.
To Reproduce
Steps to reproduce the behavior:

Ensure there is no session file in ~/.local/share/nvim/sessions corresponding to /home/user.
Ensure that the last session opened was in a different directory to /home/user.
cd ~
Start nvim with the auto-session plugin enabled and the config option auto-restore-last-session.

Expected behavior
The last session is restored.
Screenshots

Checkhealth
  1
    1 ──────────────────────────────────────────────────────────────────────────────
    2 auto-session: require("auto-session.health").check()
    3
    4 vim options
    5 - OK vim.o.sessionoptions
    6
    7 Lazy.nvim settings
    8 - OK Lazy.nvim support is enabled
    9 - OK auto-session is not lazy loaded
   10
   11 Config
   12 - OK·
   13   {
   14     auto_restore_last_session = true,
   15     log_level = "debug"
   16   }
   17
   18 General Info
   19 - Session directory: /home/jackus/.local/share/nvim/sessions/
   20 - Current session:·
   21 - Current session file:·

Baseline (please complete the following information):

Result of set sessionoptions?:   sessionoptions=blank,buffers,curdir,folds,help,tabpages,winsize,winpos,terminal,localoptions
OS. e.g uname -a:  Linux pcname 6.10.10-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 12 Sep 2024 17:21:02 +0000 x86_64 GNU/Linux
Neovim version nvim --version:

Build type: Release
LuaJIT 2.1.1725453128

URL to your current config (if public): https://github.com/jack-obrien/kickstart.nvim

Additional context
Thanks for any help! I love your work.

Thanks for the report! A trivial test case works for me so there must be more going on. Can you do the following:

add log_level = 'debug', to your auto-session config
start nvim where it doesn't restore the last session as expected
Post the logs as reported by either :messages or :Notifications (depending on if you have nvim-notify or not)?

Yes, here are the logs! Thank you for a quick response :)
For this example, my previous session i want to restore is my neovim config with cwd .local/nvim/. My workflow to reproduce the bug is:

cd .local/nvim
nvim
:e init.lua
:q
cd ~
nvim

And after doing that, here is the debug output from auto-session as reported by :messages:
auto-session DEBUG: Config at start of setup {
  args_allow_files_auto_save = false,
  args_allow_single_directory = true,
  auto_create = true,
  auto_restore = true,
  auto_restore_last_session = true,
  auto_save = true,
  close_unsupported_windows = true,
  continue_restore_on_error = true,
  cwd_change_handling = false,
  enabled = true,
  lazy_support = true,
  log_level = "debug",
  root_dir = "/home/jackus/.local/share/nvim/sessions/",
  session_lens = {
    load_on_setup = true,
    mappings = {
      alternate_session = { "i", "<C-S>" },
      delete_session = { "i", "<C-D>" }
    },
    previewer = false,
    session_control = {
      control_dir = "/home/jackus/.local/share/nvim/auto_session/",
      control_filename = "session_control.json"
    },
    theme_conf = {}
  },
  use_git_branch = false
}
auto-session DEBUG: Root dir set to: /home/jackus/.local/share/nvim/sessions/
auto-session DEBUG: cwd_change_handling is disabled, skipping setting DirChangedPre and DirChanged autocmd handling
auto-session DEBUG: Loading session lens
auto-session DEBUG: Lazy is loaded, but not visible, will try to restore session
auto-session DEBUG: enabled_for_command_line_argv, launch_argv: {}
auto-session DEBUG: No arguments, saving/restoring enabled
auto-session DEBUG: find_matching_directory {
  dirToFind = "/home/jackus",
  dirsToCheck = {}
}
auto-session DEBUG: suppress_session didn't find a match, returning false
auto-session DEBUG: RestoreSessionFromDir start { "/home/jackus/.local/share/nvim/sessions/" }
auto-session DEBUG: RestoreSessionFromDir validated session_dir:  /home/jackus/.local/share/nvim/sessions/
auto-session DEBUG: get_session_file_name no session_name, using cwd: /home/jackus
auto-session DEBUG: RestoreSessionFromDir escaped session name: %2Fhome%2Fjackus.vim
auto-session DEBUG: RestoreSessionFromDir session does not exist: /home/jackus/.local/share/nvim/sessions/%2Fhome%2Fjackus.vim
auto-session DEBUG: get_session_file_name no session_name, using cwd: /home/jackus
auto-session DEBUG: No session restored, call no_restore hooks

Ah, it's a typo in the recent config name migration I did:
https://github.com/rmagatti/auto-session/blob/aa01054d478c6d3efc0188cb2ed4850e9f475664/lua/auto-session/init.lua#L475-L475
It wasn't caught by the unit tests because they use the old config names. I'll get a PR submitted shortly. In the meantime, you can set the old config name and it should work:
auto_session_enable_last_session = true

Perfect! Thank you, that fixes it. still better than vscode

@jack-obrien the new config name auto_restore_last_session should now work


Angular DevTools install suggestion pop-up

🚀 Feature request
Angular CLI should show install devtools suggestion pop-up when running ng serve
Command (mark with an x)

[ ] new
[ ] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[x] config
[ ] help
[ ] version
[ ] doc

Description
When a user executes ng serve on their application running in development mode, the CLI should inject a script which checks if Angular DevTools is available. If it’s not available, the script should show a pop up asking if the user wants to install the extension or never see the pop up again.
Clicking on “Install” should redirect the user to the Chrome Web Store page of Angular DevTools.
New configuration option disableDevtoolsPopup should be added to disable the functionality.
Describe the solution you'd like
Document describing the proposed solution.
https://docs.google.com/document/d/e/2PACX-1vQiWzu6eqD_Ep4-Yt0yGjfY8o7n3jmYz0pC6lwceO6SR3Jz7EDLxjGkrSbCinav7ceWyk13qguGRA9Z/pub

Browser sniffing will be needed to not spam Firefox and Safari users that can't install the extension:

https://github.com/rangle/angular-devtools/issues/785
https://github.com/rangle/angular-devtools/issues/895

@sorin-davidoi good point! We should have Firefox support soon.

Some points to consider:

As mentioned above this should only be done for Chrome desktop, therefore there needs to be some browser detection logic.
When the Close button is clicked, likely a state needs to be persisted in local storage to avoid the pop-up to re-appear when the page is refresh when there are code changes due to live-reloading.
For browser to node communication we'd can probably add a dev-server middleware to modify the angular.json. (This is actually quite involved).
From the builder there is no easy way to access the global configuration, therefore a builder level option would be a better alternative.

@mgechev is this something that we want to do and keep tracking on Github?

Closing as this is tracked internally.


📦 NEW: Shades of Purple Theme

Thank you for the excellent project of Prism.js.
This PR adds 🦄 Shades of Purple theme to the mix:

[x] Shades of Purple CSS
[x] Shades of Purple Screenshot with 250px x 180px
[x] Addition to the README.md file as per the format

Context
🦄 Shades of Purple theme is used by over 2,000,000 developers. I have built other Shades of Purple themes for different software. Here's a list.

🦄 VSCode theme — Shades of Purple
🦄 iTerm2 theme — Shades of Purple
🦄 Hyper™ theme — Shades of Purple
🦄 HighlightJS theme — Shades of Purple
🦄 Konsole theme — Shades of Purple
🦄 Slack theme — Shades of Purple
🦄 Alfred theme — Shades of Purple
🦄 Cygwin theme — Shades of Purple
🦄 WordPress theme — Shades of Purple
🦄 Prism JS + Prism CSS + Code-Surfer theme — Shades of Purple

SOP's Syntax Colors
Shades of purple theme is built with several shades of purple and a few contrast colors to make things pop. This also makes SOP a very good theme for teaching, presenting, and using on your site via Prisma or HighlightJS. A rough collection of important colors in SOP theme is listed below.

USAGE
HEX CODES

Background
 #2D2B55

Background Dark
 #1E1E3F

Foreground
 #A599E9

Hover Background
 #4D21FC

Contrast
 #FAD000

Contrast Lite
 #FFEE80

Contrast Lite II
 #FAEFA5

Highlight
 #FF7200

Comment
 #B362FF

Constants
 #FF628C

Keywords
 #FF9D00

Other
 #9EFFFF

Strings
 #A5FF90

Templates
 #3AD900

Definitions
 #FB94FF

Invalid
 #EC3A37F5

Diff Added
 #00FF009A

Diff Removed
 #FF000D81

Looking forward, peace! ✌

Fixed the file names. Looking forward, peace! ✌

Also need resolve the conflict.
Thank you for contributing!

@mAAdhaTTah thanks for the review. I have improved the theme and removed !important tags by making them more specific.
Looking forward, peace! ✌

Looks good! Thank you!

Woohoo! 🥳🥳🥳


reflect: should DeepEqual handle cyclic references?

Hi!
Here is a test case:
package main

import "reflect"

type Test *Test

func makeTest() Test {
	var t1, t2 Test
	t1 = new(Test)
	t2 = &t1
	*t1 = &t2
	return t1
}

func main() {
	a := makeTest()
	b := makeTest()

	reflect.DeepEqual(a, b)
}

I'm using go 1.6.1 on Linux and this test fails with a stack overflow.
It seems that reflect.DeepEqual could theoretically handle cyclic references, but it doesn't check visited map for pointers.
Is this a bug (probably regression) of feature? If it's a feature, it will be nice if it would be documented.

I suspect we're done making changes to DeepEqual, but I'll keep this bug open for us to decide later, during the Go 1.8 dev cycle.

This seems like a legit, albeit niche, inconsistency to me.  If Test's declaration is changed from type Test *Test to type Test struct { p *Test } or type Test [1]*Test and makeTest is updated accordingly, then DeepEqual returns immediately.  There's nothing in DeepEqual's godocs to suggest this inconsistency is / should be present.
The obvious fix is to simply relax the deepEqualValue's "hard" predicate to include Ptr-Kinded Values. Downside is this would probably grow the number of entries needed in the visited map for typical programs that don't create cycles using only pointer types.
I suspect we can fix it somehow without regressing typical programs though.
@gavv How did you discover this problem? Do you have a real-world application that's negatively affected by it?

The obvious fix is to simply relax the deepEqualValue's "hard" predicate to include Ptr-Kinded Values.

Yes, this does work, just checked.

How did you discover this problem? Do you have a real-world application that's negatively affected by it?

It was a failing test for deepcopy module found on github. Not a real-world application though, but I assume this test worked before so it's probably a regression.

Yes, this is a bug, since type Test struct { X *Test } works fine. It's a bug that's been there since the initial Go release, but still a bug.
Another failing case:
package main

import "reflect"

type Test interface{}

func makeTest() Test {
	var t1, t2 Test
	t1 = new(Test)
	t2 = &t1
	*t1.(*Test) = &t2
	return t1
}

func main() {
	a := makeTest()
	b := makeTest()
	reflect.DeepEqual(a, b)
}

In the implementation, the goal of the 'hard' predicate is to avoid making visited too big. I think it should work to change hard to be
hard := func(t Type) bool {
	switch t.Kind() {
	case Array, Map, Slice, Struct:
		return true
	case Ptr:
		t = t.Elem()
		return t.Kind() == Ptr || t.Kind() == Interface
	}
	return false
}


Lab 28 Issue

Lab/Demo: 28
Task: Import the multiple missing libraries into the .NET script
Step: 4. Add the [Microsoft.Azure.Services.AppAuthentication][nuget.org/packages/Microsoft.Azure.Services.AppAuthentication] package from NuGet using the following command:
dotnet add package Microsoft.Azure.Services.AppAuthentication
Description of issue
Error : There are no versions available for the package ´Microsoft.Azure.Services.AppAuthentication´.

After that, in the Deploy your application to Azure App Services, on step 17,selecting the folder to deploy, this happens:

Selecting Deploy anyway will result in an error :

After some research, it appears like we have depricated the Microsoft.Azure.Services.AppAuthentication library and replaced it with the Azure.Identity one and it appears like NuGet is no longer providing the deprecated one.  I am in the process of re-writing this script to use the Azure.Identity library.

The fix is simple in the short-term. Simply add the version on to the instruction.
dotnet add package Microsoft.Azure.Services.AppAuthentication --version 1.6.2

Thank you @WigF1, the fixes are scheduled to go in Monday.

Fix was done a while back


Refactored SelectInput and MultiSelectInput to use redux

#MINOR#
Refactored the frost-select and frost-multi-select to rely on redux for state management. This should simplify the state handled directly by ember as well as provide useful information about state transitions when debugging components.
CHANGELOG
The select input widgets now use redux.

You'll need to add all the new dependencies into the blueprint as well

:+1:

:+1:

:+1:


StockCurve[A].item returns a collection of items

So, according to the documentation, Stock.curves returns a collection of curves, "each having the properties line, area, item and index. [...] Under item one finds the original element in the data."
The problem is that this implies that each curve is associated with a singular item, despite the fact that the curve is dependent on a series of items. This is formalized in the explicitly typed scala.js documentation. Where the type of StockCurve[A].item is A rather than Array[A]
This is inspite of the fact the .item actually returns an array, leading to a runtime error when treating item like an A.

Thank you, I have fixed the documentation and accepted your PR on paths-scala-js

Thanks! I'm loving this library, by the way. The level of control I have over the final output is great, yet my code remains small and clean.


boa --output does not cover expected flow

Apologies for the lack of code / stack-trace, my workplace is rather strict in exporting code and i don't have a working environment in my local machine for WFH :/
Steps to recreate:

Setup any recipe.yaml in any $RECIPE_PATH folder
Run conda mambabuild $RECIPE_PATH --output
conda build throws error for not having the recipe in expected format

the code from conda-build
Note that boa build --output $RECIPE_PATH does not work as intended neither
Suggestion:
Suspected culprit: in boa
Boa delegates call to conda-build's build arg-parse, which does not accept --output as argument.
It appears that the --output flag is for render only.

Proposing patch
#364__split_render___build_argument_handling.patch
Had to split build / render handling because not all build-flags are supported in render mode, e.g. in my use case I need --output-directory

The PR is failing because of ‘—output’ and ‘—output-folder’ shares the same prefix


Clarify Order Flow on on Account Page

"Order Now" button on account page may be ambiguous to new users looking to update an existing order or subscription.
Suggest that "Order Now" be renamed "New Order" and additional button to help direct new users to Order History.

Implemented in https://github.com/MRKTBOX/mrktbox-web/commit/c8243eb35a8bccd1e42bbba5066c024fc782f2e4 and deployed in https://github.com/MRKTBOX/mrktbox-web/commit/b644f9e64d1984bc67b65c15a4df31f4dfdb9af7 (v0.1.3).


Animation stop, when we change tabs

Hi,
I don't know why, but I have an animation who is running perfectly (thanks Raphaël), but when I switch tab on the browser, the animation is stopped. Any one have an idea why ? And how can I disable this behaviour ?
Thanks

Raphael uses the requestAnimationFrame API for animations, is the most performant (and correct) way of doing it.
If you really need that you could do it the old way with setInterval...


Add custom_object argument for keras model

Fix load_keras_model() cannot load custom classes and functions.

I don't think we can accept this PR; 1.13 has already been released, and we're only doing patch releases for it. Contrib is no longer packaged in newer releases.
Please try the new SavedModel format support in tf.keras.Model.save.


URSA + ed25519-dalek v1.0.x crate for compatibility with Rust-Crypto stack

Direct reference to github of ursa crate as there was no publish of it to crates.io in past 4 month.
This should be updated to crates.io reference when published eventually.

@mitfik do HCF depend on the dalek-ed25519 crate outside of KERI? this PR bumps it to 1.0, which is incompatible with versions <1.0

@chunningham we do not depend upon KERI yet so feel free to merge.


Permission denied for this action saving object through cloud code

Check out this issue for an ideal bug report. The closer your issue report is to that one, the more likely we are to be able to help, and the more likely we will be to fix the issue quickly!
For implementation related questions or technical support, please refer to the Stack Overflow and Server Fault communities.
Make sure these boxes are checked before submitting your issue -- thanks for reporting issues back to Parse Server!

[x] You've met the prerequisites.

[x] You're running the latest version of Parse Server.

[x] You've searched through existing issues. Chances are that your issue has been reported or resolved before.

Environment Setup
stand alone installation using express on CentOS behind nginx set-up as reversed proxy.
Parse-server version : 2.2.7
parse-dashboard version: 1.0.8
Mongo-db version: 3.2.3
Steps to reproduce

Create new class via dashboard (Test)
-> class level permissions are blank at this time

Set all class level permissions (read, write, add) public
-> mongo db document:
{
"_id" : "Test",
"objectId" : "string",
"updatedAt" : "string",
"createdAt" : "string",
"_metadata" : {
"class_permissions" : {
"get" : {
"" : true
},
"find" : {
"" : true
},
"create" : {
"" : true
},
"update" : {
"" : true
},
"delete" : {
"" : true
},
"addField" : {
"" : true
}
}
}
}

try to add new document via cloud function
Parse.Cloud.define("sendTest", function(request, response) {
var Test = Parse.Object.extend('Test');
var newTest = new Test();
newTest.save().then( .....
curl:
curl -X POST
-H "X-Parse-Application-Id: XXXXXXXXXXXXXXXXXXXXXXXXXXX"
-H "X-Parse-REST-API-Key: undefined"
-H "X-Parse-Session-Token: 10LRXNxgfxDjXxy6zN5ioSXVp"
--data-urlencode "{
"test": "hWItYo37U0"
}"
https://www.testserver.com/functions/sendTest

error :
verbose: error: code=101, message=Permission denied for this action.
parse-server-2 Error
parse-server-2 verbose: error: code=141, message=Permission denied for this action.

if I use the master key in the cloud function the object is save successfully. Creating a new object via the parse-dashboard API console also works fine: curl:
curl -X POST
-H "X-Parse-Application-Id: XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"
-H "X-Parse-REST-API-Key: undefined"
-H "X-Parse-Session-Token: 10LRXNxgfxDjXxy6zN5ioSXVp"
--data-urlencode "{
"test": "hWItYo37U0"
}"
https://www.testserver.com/functions/sendTest
Logs/Trace

We have lots of passing test cases that cover these features. Can you post some code that reliably reproduces your issue? Ideally as a test case we can add to our test suite.

ok, I'll see what I can cook up. I am working on this project after hours so need to find some time. One thing I noticed though today is that the class I created through the parse-dashboard did not had 'readUserField' and 'writeUserField' in the schema definition:
{
"_id" : "Test",
"objectId" : "string",
"updatedAt" : "string",
"createdAt" : "string",
"_metadata" : {
"class_permissions" : {
"get" : {
"" : true
},
"find" : {
"" : true
},
"create" : {
"" : true
},
"update" : {
"" : true
},
"delete" : {
"" : true
},
"addField" : {
"" : true
}
}
},
"test" : "string"
}
vs
{
"_id" : "MessageLog",
"_metadata" : {
"class_permissions" : {
"get" : {
"role:administrator" : true
},
"find" : {
"role:administrator" : true
},
"update" : {
"role:administrator" : true
},
"create" : {
"role:administrator" : true
},
"delete" : {
"role:administrator" : true
},
"addField" : {
"role:administrator" : true,
"*" : true
},
"readUserFields" : [],
"writeUserFields" : []
}
},
"referenceId" : "string",
"sendTime" : "date",
"sent" : "boolean",
"to" : "string",
"type" : "string",
"extra" : "string",
"from" : "string",
"message" : "string"
}
for an imported class from Parse.com

You should set sessionToken
Read this https://github.com/ParsePlatform/parse-server/wiki/Compatibility-with-Hosted-Parse
...
test.save({sessionToken: request.user.getSessionToken()}).then ...

it didn't worked as @mbilling sample code
test.save(null, {sessionToken: request.user.getSessionToken()}).then...
notice the null before setting the sessionToken.

how about putting such an important thing in the docs ?

It is in the docs and the api reference

I still can not find it in the docs, anyway at least one can say it is not bold enough ;)

http://parseplatform.org/Parse-SDK-JS/api/classes/Parse.Object.html#methods_save

it is your call bro, but it still is not in the docs ;)

Feel free to open a Pr to make it more clear if you feel this should be specified more clearly.


Add missing scopes to GitLab auth provider docs

closes #17189
The auth docs are missing some scopes required by the GitLab provider. Without these scopes the RepoUrlPicker in software templates is unable to retrieve oauth tokens for the user.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

not stale

not stale


add hover_cols example

I learnt how to do this here https://discourse.holoviz.org/t/add-an-extra-field-when-hovering-in-hvplot-scatter/2331 and wanted to contribute back the docs as my reference. I would like to think others would benefit from this example as well

Great, thanks @raybellwaves!


"TODO add the encoder here" - add platypus image encoder

It has to exist somewhere, if only to test the decoder:
https://github.com/raspberrypi/pico-extras/blob/77eae2836638baf2f61b321eb61125da99bb4445/README.md?plain=1#L23

This looks like the encoder (with some additional stuff for compressing several images in a row): https://github.com/raspberrypi/pico-playground/blob/eb6658e82069c603244b42d2bdd858a3fa270c63/apps/popcorn/converter/src/convert.cpp#L172


[Master]Add time stamp in image query

@jichenjc Please note this change will change image_query outputs, potential in-compatible problem.

@jichenjc Please note this change will change image_query outputs, potential in-compatible problem.

I updated document, so let's do full test when FP1 be built.

@bjhuangr I don't know the detail of this PR , need check
@SharpRazor we have no other way to do because we have to query the info, correct?
And this will only add additional field into output, nothing changed, correct?

@jichenjc @bjhuangr yes, we must know the timestamp from zcc. So I have updated the document to add the description of timestamp.

@bjhuangr I don't know the detail of this PR , need check
@SharpRazor we have no other way to do because we have to query the info, correct?
And this will only add additional field into output, nothing changed, correct?

And this fix imports a new potential exception while doing the image query, which might be in-compatible. So, @SharpRazor I think we should mute this exception in Line: https://github.com/openmainframeproject/python-zvm-sdk/pull/100/files#diff-00999ea921397098b8dc799ec59174cfR2268

@qianxi2017 you are right.

lgtm


Fixes binary data issue in parse_log

Originally from Jason

coverage: 93.51%. remained the same
when pulling 0bbd07cf78e9935422a41a89a1e33982382a4c01 on njw-parse-log
into b201075160e1222b1baa9164c9159b4e96dfc02d on main.


Add support for Zig Language Server

this one is for you @bfredl :p

any plans on getting this merged?

Hey guys, can we maybe open a new PR of this, since this is being hanging for a while? i can provide the changes if nobody wants to do it

@ellisonleao there is https://github.com/neovim/nvim-lspconfig/pull/433 it seems could you test/comment on that one ? I dont use zig so it helps to know it worked for at least 2 people before merging

@teto testing here, and so far so good. I think we can already merge it

hey @teto i though we were merging #433  instead of this one

well you commented here with an ambiguous "testing here" :s

sorry for the confusion, the other one has the most updated code. Can we revert this and merge the other one?


Fix docs for interval score

Please work through the following checklists. Delete anything that isn't relevant.
Development for new xarray-based metrics

[x] Works with n-dimensional data and includes reduce_dims, preserve_dims, and weights args.
[x] Typehints added
[x] Add error handling
[x] Imported into the API
[ ] Works with both xr.DataArrays and xr.Datasets if possible

Docstrings

[x] Docstrings complete and follow Napoleon (google) style
[x] Maths equation added
[x] Reference to paper/webpage is in docstring. The preferred referencing style for journal articles is APA (7th edition)
[x] Code example added

Testing of new xarray-based metrics

[x] 100% unit test coverage
[x] Test that metric is compatible with dask.
[x] Test that metrics work with inputs that contain NaNs
[x] Test that broadcasting with xarray works
[x] Test both reduce and preserve dims arguments work
[x] Test that errors are raised as expected
[ ] Test that it works with both xr.DataArrays and xr.Datasets

Tutorial notebook

[x] Short introduction to why you would use that metric and what it tells you
[x] A link to a reference
[x] A "things to try next" section at the end
[x] Add notebook to Tutorial_Gallery.ipynb
[ ] Optional - a detailed discussion of how the metric works at the end of the notebook

Documentation

[x] Add the score to the API documentation
[x] Add the score to the included list of metrics and tools

Thanks. I have built the docs and pushed them up - see  https://scores.readthedocs.io/en/228-documentation-testing-branch/api.html#scores.continuous.quantile_interval_score and https://scores.readthedocs.io/en/228-documentation-testing-branch/api.html#scores.continuous.interval_score . @reza-armuei @Steph-Chong you might want to take a look at that and make sure all is as intended


[QUESTION] kubectl error: You must be logged in to the server (Unauthorized)

Hi @vitobotta,
almost exactly 1 year ago I created my cluster using the docker image of version v0.5.3 and today we experienced issues with our cluster, mainly with the LoadBalancer as the targets of the ingress were gone.
I recognized I cannot access my cluster from outside anymore and I assume some certificate expired:
kubectl get pods -n prod
error: You must be logged in to the server (Unauthorized)

I can access the master node directly and use kubectl there. As far as I know kubeadmin has not been installed, right?
Can you somehow help me out or at least tell me if my assumption is correct and maybe also give a hint how to solve the issue?
I know this is not an issue of this tool, but help is really appreciated. I want to get the setup healthy again.
Thanks
Marco

meanwhile I updated k3s using the old v0.5.3 version and added a new worker node. the loadbalancing issue is now resolved. but I still cannot access the cluster from outside (same error as above)
wondering how to solve that 🤔

ssh to worker and check if certificates have expired: openssl s_client -connect localhost:6443 -showcerts < /dev/null 2>&1 | openssl x509 -noout -enddate

If yes:

# Stop K3s
systemctl stop k3s

# Rotate certificates
k3s certificate rotate

# Start K3s
systemctl start k3s

However, according to this page certificates automatically should be renewed:

K3s client and server certificates are valid for 365 days from their date of issuance. Any certificates that are expired, or within 90 days of expiring, are automatically renewed every time K3s starts.

scp root@WORKER_IP:/etc/rancher/k3s/k3s.yaml ./
export KUBECONFIG=./kubeconfig

Now everything should be ok.


Improvement suggestion re Customer.getTokenCustomerID()

Was just looking through the recent commit: I like the hasFraudCode and getResponseMessages.  But the Customer.getTokenCustomerID() seems flawed.  public long? TokenCustomerID is a nullable long, so calling TokenCustomerID.ToString() without a null check is just asking for trouble.  Suggest you change it to return (TokenCustomerID == null) ? null : TokenCustomerID.ToString();.  Or return "", either is better than a NullReferenceException.
Of course, the method itself seems redundant to me, but perhaps you have longer-term plans for it.

Good pickup thanks Ian, I've added that improvement for the next release. This method may be redundant, I was thinking of the Java SDK where the types aren't as consistent as they could be.


Introduce hardware inventory functionality

I'm interested in extending NAPALM to include a method for retrieving hardware information from devices. Currently, only the chassis serial number can be read (via get_facts()). I'd like to add the ability to return the equivalent of show chassis hardware (Junos) or show inventory (IOS). Here's an example CLI output for Junos:
demo> show chassis hardware
Hardware inventory:
Item             Version  Part number  Serial number     Description
Chassis                                VG371429XXXX      QFX5100-24Q-2P
Pseudo CB 0
Routing Engine 0          BUILTIN      BUILTIN           QFX Routing Engine
FPC 0            REV 06   650-056263   VG371429XXXX      QFX5100-24Q-2P
  CPU                     BUILTIN      BUILTIN           FPC CPU
  PIC 0                   BUILTIN      BUILTIN           24x 40G-QSFP
    Xcvr 0       2M DAC   740-038623   F786SNG-A         QSFP+-40G-CU1M
    Xcvr 1       7M AOC   740-032986   F8205J4-B         QSFP+-40G-SR4
    Xcvr 2       7M AOC   740-032986   F8205TG-B         QSFP+-40G-SR4
    Xcvr 3       2M DAC   740-038623   F786SNL-A         QSFP+-40G-CU1M
    Xcvr 4       2M DAC   740-038623   F786SNP-A         QSFP+-40G-CU1M
  PIC 1          REV 03   611-049555   RR371408XXXX      QFX-EM-4Q
  PIC 2          REV 03   611-049555   RR371408XXXX      QFX-EM-4Q
Power Supply 0   REV 03   740-044332   1GA1413XXXX       JPSU-650W-AC-AFI
Power Supply 1   REV 03   740-044332   1GA1413XXXX       JPSU-650W-AC-AFI
Fan Tray 0                                               QFX5100 Fan Tray 0, Back to Front Airflow - AFI
Fan Tray 1                                               QFX5100 Fan Tray 1, Back to Front Airflow - AFI

And for IOS:
demo# show inventory
NAME: "1", DESCR: "WS-C3560X-48"
PID: WS-C3560X-48T-L   , VID: V05  , SN: FDO1720XXXX

NAME: "Power Supply 0", DESCR: "FRU Power Supply"
PID: C3KX-PWR-350WAC   , VID: V03L , SN: LIT1731XXXX

NAME: "Power Supply 1", DESCR: "FRU Power Supply"
PID: C3KX-PWR-350WAC   , VID: V03L , SN: LIT1851XXXX

The proposed method would be read-only, similar to get_environment(). It would return a list of objects, each representing a hardware component. The format of each object will be largely arbitrary, as it would probably not be practical to attempt to standardize among the platforms, but each component would include at least a name, model/part number, and serial number. I'm not too familiar with OpenConfig yet, but I believe this object would correspond to the platform model.
My primary interest in adding this capability is to extend an IPAM/DCIM application I maintain called NetBox. It would be great to replace the custom RPC functionality I've written with NAPALM; I just need to implement the hardware inventory piece. Beyond that, of course, there's inherent value in being able to retrieve a list of all the components installed within a device.
If people think this would be a valuable addition to NAPALM, I'd love to volunteer my time.

This sounds great to me. Give me a few days to investigate which OC models we need and then I can guide you on how to get it done with the OC framework.

@dbarrosop Just checking in. Happy to give this a shot on my own if you can't spare the time.

I haven't forgotten about this :) I am almost done but there is a severe lack of documentation on how to use the framework. If you have the energy you can look at this:
https://github.com/napalm-automation/napalm-yang/blob/augment/napalm_yang/mappings/eos/extractors/oc-if/interfaces.yaml
https://github.com/napalm-automation/napalm-yang/blob/augment/napalm_yang/mappings/eos/extractors/oc-ip/ipv4.yaml
Basically, to extract data from the device into the OC framework you just have to define the rules to do so. So far I have implemented an "extractor" for plain text and for xml (I think you only care about the text one right now). Rules are not documented, sorry about that but I guess it shouldn't be difficult to start playing with it and figure out how it works.

@jeremystretch You can start looking at this:
https://github.com/napalm-automation/napalm-yang/blob/augment/interactive_demo/tutorial.ipynb
There are some things to be refactored that will happen soon but you can start playing with the model if you want to take a look. You could do the folllowing:
import napalm_yang
import pprint

config = napalm_yang.BaseBinding()
config.add_model(napalm_yang.oc_platform.platform())

pretty_print(config.model_to_dict())

And start seeing how to map what you want with that structure. I think the platform model is quite "open to interpretation".

Just a quick keepalive. I am almost done, paperwork (aka docs, testing and cleaning) always takes more than anticipated. You can start looking at the docs if you want (WIP but the parts that are relevant for you are probably all there already):
https://napalm.readthedocs.io/en/yang_doc/YANG/index.html
This weekend I will finish the docs and next week I will start merging and releasing stuff so you can start working on this in 1-2 weeks tops.

Excellent! Thanks for the update. Really hoping to dig into the code and familiarize myself more this week.

Ok, this is all out, documentation, libraries, etc... You can find a quick summary here:
https://www.dravetech.com/blog/2017/04/09/napalm-yang.html
Note that @bewing is also working on this. Turns out that the components model is a nightmare but we are discussing how to address it here:
https://github.com/napalm-automation/napalm-yang/issues/39

What's the state of this? Is this something you still need or that you are working on?

@dbarrosop Sorry, NetBox has been sucking up all my dev time lately. I've been following what @bewing has been working on though. How crucial is it that we utilize the OpenConfig platform model for inventory data in the first place, given that it's essentially arbitrary?

I think it makes sense to use the OC models even if use a regular getter. It doesn't make sense to bother reinventing the wheel and it will help migrating code to napalm-yang later on. If the model doesn't make sense at all we should bring it to the OC group.

FYI, there's code now (napalm-automation/napalm-eos#165), I just need to write tests.

Did this ever make it live?

This repo is deprecated. Head to main napalm repo.


Fixed issue with multiple calls to favicons()

Unexpected results are occurring when favicons() is called multiple times with certain configurations. In my case, when path is different.
The unexpected result is that when "files" (manifest.json, browserconfig.xml, etc.) are being created the paths (i.e. "path/android-chrome-36x36.png") are incorrect for any favicons() call that's made after already calling favicons().
// Forget the fact that favicons() is asynchronous in this example.

favicons('./favicon.png', {
  path: '/one'
});
// File path inside the created manifest.json:
// "\\one\\android-chrome-36x36.png"
// CORRECT

favicons('./favicon.png', {
  path: '/two'
});
// File path inside the created manifest.json:
// "\\two\\one\\android-chrome-36x36.png"
// INCORRECT - "\\one" was included in the path

This is happening because config is the same object every time favicons() is called, which normally isn't an issue if it weren't for the fact that properties of config are being overridden with each call.
The fix in this PR makes a deep copy of config so that each time favicons is called config is an accurate representation of what it's intended to be; the content of the files in /config.

Hey @VinceMalone, awesome PR but clone needs to be in dependencies rather than devDependencies :)


Sonic the Inclusive Werehog: Watch tanishatray4Saveyra

Sonic the Inclusive Werehog requests the watch of the watch_keyword tanishatray4Saveyra. See the MS search here and the Stack Exchange search here.

tanishatray4Saveyra has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Coverage remained the same at 64.713% when pulling 90fcf4a9d992d69ea780a5d6c5b64e3b3575b5ef on auto-blacklist-1538746313 into 728cff33d4cbdbabd34fe9c984f35be00cfb9658 on master.

!!/approve


Not working with https?

I can't seem to get this to work over a https connection... any thoughts?
The error I get is:
undefined:1

503 Service Unavailable

^
SyntaxError: Unexpected token <
  at Object.parse (native)
  at Request._callback (/Users/mike/.nvm/versions/node/v0.12.0/lib/node_modules/meteor-down/node_modules/ddp/lib/ddp-client.js:336:23)
  at Request.self.callback (/Users/mike/.nvm/versions/node/v0.12.0/lib/node_modules/meteor-down/node_modules/ddp/node_modules/request/request.js:344:22)
  at Request.emit (events.js:110:17)
  at Request.<anonymous> (/Users/mike/.nvm/versions/node/v0.12.0/lib/node_modules/meteor-down/node_modules/ddp/node_modules/request/request.js:1239:14)
  at Request.emit (events.js:129:20)
  at IncomingMessage.<anonymous> (/Users/mike/.nvm/versions/node/v0.12.0/lib/node_modules/meteor-down/node_modules/ddp/node_modules/request/request.js:1187:12)
  at IncomingMessage.emit (events.js:129:20)
  at _stream_readable.js:908:16
  at process._tickCallback (node.js:355:11)

@arunoda any ideas?

+1

Try this PR : https://github.com/meteorhacks/meteor-down/pull/20
Working for me.

hi,
i was able to test over https with this bit
meteorDown.init(function (Meteor) {
  Meteor.call('addClient', { id: 'newClient', apiKey: '7schWTAcWZQ5NkHt' }, function (error, result) {
    Meteor.kill();
  });
});
meteorDown.run({
  concurrency: 600,
  url: 'https://domain.com',
  ssl: true,
});


Reduce type piracy by shifting methods to AbstractFFTs

https://github.com/JuliaApproximation/GenericFFT.jl/blob/2e55016b642f5bdc7324a064b81f16dca6b967e4/src/GenericFFT.jl#L24-L33
These should ideally be moved to AbstractFFTs to reduce the type-piracy in this package.

Yes, that would be more appropriate. Although it may have side-effects, but even then it would be worth fleshing them out.


Dealing with list of values (words)

Missing functionality
When dealing with NLP tasks I always find myself doing the same EDA on words.
Sentence length, Word length, lemmas, character length, etc...
I don't mind pandas profiling not being able to do NLP tasks but I'm frustrated not being able to get insights on lists of words.
Proposed feature
When a type list of strings is found it could be nice to analyze automatically the most commons words, the mean length, max length,
The sentence length etc.... all the boilerplate code/analysis that we always do in NLP.
Alternatives considered
I do it by hand, adding myself the needed boilerplate code, with each column like var_1 / var_1_len / var_1_len_max etc...
Additional context
This is done for corpus analysis so no labels only text processing.
I don't know if there is any objective in this direction, but I tried to ask anyway.
Have a great day

@Ierezell this should already be possible (word counts, length distributions and even character level statistics are all included)! explorative=True enables these features.
The data should be in string format (you mention list of strings). Happy to take a PR to support that format if it's a common corpus format.

Hi @sbrugman,
Indeed it works with raw text :smiley:
I found a hackish way to explore NER/POS/lemmas by just concatenating them with spaces (as I guess you do the reverse operation to get metrics).
However I cannot get character based metrics even with
 profile = ProfileReport(
        df,
        title="Pandas Profiling Report",
        explorative=True,
        dark_mode=True,
        vars={
            "cat": {
                "length": True,
                "characters": True,
                "words": True,
                "n_obs": 5,
            },
        },
        html={"style": {"theme": "flatly"}},
    )

Thanks for your help,
Have a great day

That's a bug, worked neatly before. Will look into it

@Ierezell Fix is upcoming in https://github.com/pandas-profiling/pandas-profiling/pull/850

That's perfect!
Since pandas-profiling is able to deal with sequences internally could it be nice to be able to have a list of strings as a value instead of concatenating them?
Thanks a lot for the fix! However, I modified the same file as you and I'm still not able to get the character infos :

Thanks for your help

Which version of pandas/pandas-profiling are you on?

I am on 3.1.0 installed with poetry.

And pandas?

Also latest, 1.3.3

The fix seems to work. I'm creating a release (v3.1.1) and will rerun the examples.
If upgrading doesn't resolve the issue then I'll look into it again.


How can I have some padding around the images in a small circle menu?

For my app, I want to add a smaller circle menu, not as big as the one shown in the demo. Its width and height are 32x32.
When i resize the menu button in the storyboard, the menu buttons that pops up is smaller as well, which is what I expected. However, the images on the button are still as big. This means that he nice padding around the images is gone, and the button looks like this:

I tried to manually add some transparent padding on the image, but the result stayed the same.

in version 3.0.6 I made properties customNormalIconView, customSelectedIconView public
and now you can set icon size:
        CIRCLE_BUTTON.customNormalIconView?.frame.size = SIZE
        CIRCLE_BUTTON.customNormalIconView?.contentMode = .scaleAspectFit
        CIRCLE_BUTTON.customSelectedIconView?.frame.size = SIZE
        CIRCLE_BUTTON.customSelectedIconView?.contentMode = .scaleAspectFit

Is the CIRCLE_BUTTON referring to the CircleMenu object that I am using or the UIButton object passed to the delegate method willDisplayButton?

Yes CIRCLE_BUTTON is a referring to the CircleMenu, and you can set size in viewdidload


Document format is very slow | CPU | Any extention

VSCode Version: 1.5-1.7.1 (didn't tested <1.5)
OS Version: Windows 7

Redirect from:
https://github.com/Microsoft/vscode/issues/15221#issuecomment-260099727
&
https://github.com/Microsoft/vscode-cpptools/issues/345
I can't reopen original issue
Steps to Reproduce:

Check Format Document on this file - 40k lines
www.gta-samp.com/size.cpp

clang-format.exe works 1 second
but Code.exe stuck at CPU 100% for a minute
Visual Studio 2013 format this file in second
If you delete ms-vscode.cpptools and install
marketplace.visualstudio.com/items?itemName=ryannaddy.vscode-format
or change filename to .css Code.exe will stuck at CPU 100% too
Also run
extention dir\ms-vscode.cpptools-0.9.2\LLVM\bin
clang-format.exe size.cpp
it will format in 3 seconds too
It's not problem of extension

duplicate of #15221


Add TweakScale-Redist as dependency to PhotonSailor

This mod breaks without TweakScale-Redist (it's also bundled in the download)

https://spacedock.info/mod/1895/Photon Sailor

BTW @JonnyOThan, I think that your invitation to the metadata wranglers team was sent at some point. If it's still there, you can accept it to merge stuff like this yourself if you like. I don't know whether those invites expire, so just let me know if you want it re-sent.


Update Site settings “info”

Automatically generated by Netlify CMS

👷 Deploy Preview for distracted-hoover-7b89cc processing.
🔨 Explore the source changes: 0573a8a9064f741756916354815e865dfdd51301
🔍 Inspect the deploy log: https://app.netlify.com/sites/distracted-hoover-7b89cc/deploys/60f88a84647c1800076d82c7


CPU Usage is bursting when stream of logs

We have an nginx container that accepts hundreds of request per second and logspout is causing our CPU to burst with the stream of logs it produces. We already applied:
TAIL=0
BACKLOG=true

But still the issue exists. Are there any other solutions?
Take note:

We are combining logspout with logstash using this image.. bekt/logspout-logstash:master
We are using Docker Swarm Architecture

@Dean-Christian-Armada what transport are you using? tcp or udp? are you also using tls? I have also noticed high cpu usage when sending a ton of logs. logspout may not be the ideal solution when a dealing with an excessive amount of logs. In this case, it may be best to use the dockerd logging drivers to send to a remote host.


git_resource: handle branch names with #

alternative to https://github.com/tilt-dev/tilt-extensions/pull/496

actually, i changed my mind, i like the other pr better


simplify 'getting started'

addresses #48.
curious to hear everyone's thoughts (especially @JimBarry) before we merge.

i think this is ready to be merged, but i'm happy to make additional tweaks if anyone has additional feedback.


Update Pure prompt to 1.20.0

Please be sure to check out our contributing guidelines
before submitting your pull request.
Fixes #1641 #1677 #1862
Proposed Changes

Update Pure prompt to 1.20.0

I see no evidence that the vi mode bug exists in Pure 1.20.0, so I think we can finally update Prezto's Pure prompt. Tested on MacOS 12.2, zsh 5.8 (x86_64-apple-darwin20.1.0).

We'll have to watch for any vi regressions, but I'm happy to merge this. Thanks!


renamespace to tabula.technlogy, closes #7

I don't know if I did it right :)
If I did, let's merge this; it'll close #7

The paths to the PDFs in the tests needed to be adjusted. Done in 412ea35


PR: Restore buttons that control the debugger to the main toolbar

Description of Changes

That's necessary because many people are accustomed to the Spyder 5 workflow, where they could control the debugger and see the Variable Explorer and IPython console at the same time.
Since we needed to introduce this toolbar next to the Debugger one and not visible by default (to not show repeated functionality in the main toolbar and Debugger pane), it was not possible to rely on the window state to restore the toolbars positioning anymore. Instead, several changes were required to the Toolbar plugin:

Declare a default order for toolbars in our config system.
Save toolbars order when the application is closed.
Restore that order in the next session.
Always leave the Working Directory toolbar to the right of all others to show a good UI to users.
Fix loading last visible toolbars.
Rename/improve a couple of methods for clarity.

Visual changes

New toolbar entry in the Toolbars menu.

New toolbar to control the debugger.

Issue(s) Resolved

Fixes #22434
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

@dalthviz, could you manually check that the following things are working for you? Thanks!

The Control debugger toolbar is hidden for you when starting Spyder from this PR.
When enabled, it should be shown next to the Debug toolbar.
Change toolbars position, close and open Spyder and check the order is restored as expected.
Move a toolbar to the right of the Working Directory, close and open Spyder, and check the toolbar is moved to the left of it. That means the Working Directory will always be shown to the right of all other toolbars.

Gave this a manual check and seems like the elements described above are working :+1: However, with the changes here, now is not possible to fully save toolbars state. So things like having more than one row of toolbars will not be possible, so something like:

Will not persist between sessions

Thanks @dalthviz for the review!

Gave this a manual check and seems like the elements described above are working 👍 However, with the changes here, now is not possible to fully save toolbars state.

This should be fixed now, so please check again.
Also, I changed the implementation according to the UX team feedback. Now the buttons that control the debugger are shown in/hidden from the Debug toolbar when a debugging session is started/stopped (see the OP), or if the current console is in a debugging session or not. That avoids a serious issue with my previous approach: users would need to know a new toolbar is available and enable it to get the Spyder 5 UX. In addition, this approach prevents cluttering the interface because the new buttons will be hidden when not needed.
So, could you check if that's working as expected for you?

@meeseeksdev please backport to 6.x


Enhance UI of the 8 puzzle solver.

Match the colour palette of the solver to match the other solvers.

Maintainer Assign this to me.

@HARSH00M Go for it!


Some tests (#29)

Added some specs but not much. Chucked Guard in for good measure.

Great work as usual! I'll deploy the path changes you made later this evening.