bpf: use ebpf.VariableSpec API for rewriting global constants

bpf: remove const inlining, use ebpf.VariableSpec API for rewriting globals

This commit removes compatibility with kernels 5.1 and older. From this point on,
Cilium relies on a read-only map to hold its datapath configuration data, and
config values will no longer be inlined into the bytecode.

This patch uses the new ebpf.VariableSpec abstraction for applying configuration
data to a CollectionSpec.

It also expands supported data types for runtime substitution to signed integers
and booleans, where before only unsigned integers were allowed.

bpf: inject THIS_INTERFACE_IFINDEX at runtime, replace NATIVE_DEV_IFINDEX

This is an example of converting a legacy preprocessor macro to a runtime configurable.

The default value of a configurable after declaring it with DECLARE_CONFIG() is zero,
so no need to assign dummy values.

Use ASSIGN_CONFIG to override the contents in tests after importing a declaration.

To avoid large refactors and churn, define e.g. THIS_INTERFACE_IFINDEX as a macro
around an accessor like CONFIG(interface_ifindex).

/test

/test

/test

/test

/test

/test

/test

/test

One more rebase to pick up https://github.com/cilium/cilium/pull/36235 should heal CI :crossed_fingers:

/test


minotaur --gs doesn't work in terminal on iPhone

I'm using terminus: https://itunes.apple.com/us/app/termius/id549039908?mt=8

try --gs --mobile   with <IP_ADDRESS>
(rotate into landscape mode first or ncurses goes a bit weird)


fix typos

My OCD finds them mildly distracting.
Checklist

[/] I have added tests
[/] I have updated the docs and cheatsheet
[/] I have not broken the cheatsheet

Maybe we should run codespell/cspell in pre-commit? 🤔

How did you find all these? Did you use a spellchecker?

About half or so I noticed while reading the code; I saved them up for a while to avoid lots of one-char PRs. (Note that s/the/that/ is not in the remit of a spellchecker!) And then before sending I spent a few minutes skimming the many false positives emitted by vale. It found several thousand "mistakes" of which maybe three or four were legit.
Spellcheckers in pre-commit are usually annoying due to the high rate of terms of art leading to lots of false positives, even with an (ever-growing) allowlist.


hop-node: prep for regenesis

DO NOT MERGE.
This is an example PR to prep for the Optimism regenesis. If the code makes sense, we can cherrypick commits into the bonder from develop.
Changes are:

Update bridgeDeployedBlockNumber to 0 in core
Turn off Optimism watchers
On the first sync, reset the Optimism latestBlockSynced to 0 for just the first run

This is meant to be removed the next time the bonder restarts

This PR was replaced by #277


Tiered control

Some thoughts:
<mbaxter>		One of the occasional feature requests I never got around to implementing with VanishNoPacket was a tiered system.
<mbaxter>		Let, say, the server owner see invisible staff, but staff can't see the owner (perhaps toggled)
<mbaxter>		Or even more tiers than that.
<mbaxter>		I'd like to investigate that concept for my Sponge port.
<mbaxter>		But, I have no idea how I'd implement it without creating my own grouping system or doing some nasty thing with perm nodes...
<mbaxter>		Thought: System defining perm nodes and parent/child status of those nodes?
<mbaxter>		So you could customize it to existing nodes? If they can ban players, they can see other invisible folks, if they can kick players they can not see those who can also ban?
<mbaxter>		Just throwing out thoughts in the hope someone will see this and have their own thoughts on it
<CaptainBern>	hmmm
<CaptainBern>	that seems complex
<lol768>		My concern is that it seems to make use of the node system for something it's not really suited for
<lol768>		and I know we'd be storing parent/child/arc information separately
<CaptainBern>	it's hard to come up with an alternative tho
<mbaxter>		Mmmhmm
<mbaxter>		thus my struggle! :3
<AdamQpzm>		I don't know how permission work on Sponge, but couldn't you just have it like "vanish.tier.number"?
<mbaxter>		For an infinite number of tiers...
<CaptainBern>	I would work with some kind of flag system
<AdamQpzm>		I don't see the problem, mbaxter
<CaptainBern>	I think he means that if you do it that way, there's a limit with the tiers
<AdamQpzm>		If someone has enough tiers to go past that limit they probably have a different sort of problem
<CaptainBern>	Doesn't Sponge provide a way to set metadata for players that perists through reboots?
<mbaxter>		AdamQpzm: What would you set for a tier count?
<mbaxter>		Or make it configurable
<mbaxter>		Because every time you check a player you have to query each of those tiers.
<AdamQpzm>		I would just get all their permissions and just check through that list for their tier
<CaptainBern>	that seems so uglyyy
<mbaxter>		Mmmhmm

Couldn't permission Contexts help you with this? @zml2008 pinging for advice.

You could check an OptionSubject for a numerical option such as "vanishTier"?
That's what i do in my /staff command to display which online players have the option "staff": 1".

@gabizou suggested a directed graph approach, which seems rather fun.

kittens:
  permission: kittens.in.mittens
cats:
  permission: older.kittens
puppies:
  permission: puppypuppy.puppy
  cansee: kittens
axolotls:
  permission: wtf.nature
  cansee: [puppies,cats]

@mbax What's the object oriented way of getting rich? Inheritance. I assume axolotls can see kittens then.

If Guava has something you can point me at, that'd be neat.

@mbax Haven't used it too specifically but wouldn't this just be a node system? You could traverse it to find the correct properties. Looking for a data storage to match this.
Say, for example, I have permission node A and B, where B inherits A. Getting object A would give you the parents of A, but getting B would give you the parents of B (including A).

Well, I thought of something, but I now realize Vanish has nothing to do with Packets. How would you enforce the visibility policy if you just use Invisibility keys?


carousel btns working in mobile

Related Issue

Issue number goes here: #139

Proposed Changes

fixed carousel buttons for mobiles

Checklist

[ ] Documentation of the code
[x] Add your name to the CONTRIBUTORS file
[x] Testing your code in the local machine

Screenshots

@AnkitaMalik22 Good job!. Thanks for contributing.


README update with usage details

I recently started using your library in a project of mine.  Thanks for putting it out there, it definitely does the trick.  I felt like the docs were severely lacking with implementation details so I added some updates and tried to stick to the format you use on the swift version.
Hope you find it helpful.

Thanks, @thegnartist.


Runtime Dependencies/Complex Example

I've been trying to create a binding for Intercom, but I'm struggling to run it on Android even though it's all building. Can there be a sample that has a deeper dependency tree and includes Android? Right now, the Facebook example is the only Android example but there are no run-time dependencies only compile-time.
Perhaps an example that packs up the work into a plugin? https://github.com/jfversluis/Plugin.Maui.Feature
Links:

https://mvnrepository.com/artifact/io.intercom.android/intercom-sdk-base/15.10.0
https://github.com/intercom/intercom-android
https://github.com/intercom/intercom-ios

Started trying to make sense of this here: https://github.com/kfrancis/Plugin.Maui.Intercom

Getting closer, able to call initialize but when trying to render the component I'm getting:
Java.Lang.NoClassDefFoundError: 'Failed resolution of: Landroidx/compose/runtime/internal/ComposableLambdaKt;'
Still a mess of dependencies, and the error it's mentioning isn't a direct one but rather transient from one of the material dependencies: https://mvnrepository.com/artifact/io.intercom.android/intercom-sdk-base/15.10.1

Hi, @kfrancis  - thanks for using Maui.NativeLibraryInterop and reporting this issue. It appears that you might be missing some dependencies that your binding library of interest requires. In case it is helpful, I have a sample that includes an Android binding with its dependencies at https://github.com/rachelkang/MauiCharts and more information about my process in a blog post .

@rachelkang I did check the blog post and your source, but I don't see where my issue lies. I mean, Intercom has many runtime dependencies whereas your project is single-dependency. In particular, the compose dependency seems to be more problematic than others - but I'm not sure where I'm going about it wrong.
https://github.com/kfrancis/Plugin.Maui.Intercom/blob/a5e09782ae77924e350290dba262aeda84a5d9cd/src/Plugin.Maui.Feature/Plugin.Maui.Intercom.csproj#L86
https://github.com/kfrancis/Plugin.Maui.Intercom/blob/a5e09782ae77924e350290dba262aeda84a5d9cd/src/android/native/mauiintercom/build.gradle.kts#L39
Any further suggestions? I put the source on github in the hopes that it might be useful to see what's going on in a far more dependency-complex scenario.

Hi, @kfrancis - I recommend double-checking the versions of your dependencies and ensuring they're all compatible with one another / there is no mismatch. If you continue to have issues, I recommend consulting on the issue with the library, as it may be a compatibility issue they have with Android.
Similar Java.Lang.NoClassDefFoundError: 'Failed resolution of: Landroidx/compose/runtime/internal/ type issues seem to commonly be due to version mismatches - e.g. https://stackoverflow.com/questions/66429770/jetpack-compose-fails-with-noclassdeffounderror-failed-resolution-of-landroidx

Hi @kfrancis I'm dealing with a similar situation for Android. I'm able to compile with no issue but when I try to initialize the native Android Binding library at runtime, I get a 'java.lang.ClassNotFoundException' exception. I'm thinking the native library has  run-time dependencies as well. Question: The dependencies you've listed on your Aug 7 post, are those all the dependencies that your binding needs or did you come up with that list some other way? I ask because the dependencies list at Maven show several more than that post. Thanks for your post!

I was mainly going from Intercom's android example, but yes - I agree, Maven lists far more.
This is rough, because I'm trying to migrate from Xamarin in which someone kindly had and relatively maintained bindings and it worked great but they aren't and it doesn't, and the binding story for Maui is in it's infancy and complex.


Smart alerts not working on New Mac UI with manifest

Testing below manifest file on Windows / Mac
https://github.com/OfficeDev/Office-Add-in-samples/blob/main/Samples/outlook-check-item-categories/manifest.xml
As per documentation, (https://learn.microsoft.com/en-us/office/dev/add-ins/outlook/smart-alerts-onmessagesend-walkthrough?tabs=xmlmanifest#differences-between-smart-alerts-and-the-on-send-feature), Smart Alerts is supported on New Mac UI.
But the above manifest is documented to work only in Windows, when would this work on New Mac UI or if supported, is there a way to customize the manifest to work on Mac ?
Tested on :
Windows Outlook version : Version 2209 Build 16.0.15629.20200) 64-bit
Mac New UI version : 16.61 (22050700)

Thanks for the question @mkondadg.
@samantharamon, can you please confirm the accuracy of the Mac support and the timeline?

Any update @samantharamon ?

Hi @mkondadg,
Apologies for the delayed response. I've verified that Smart Alerts is supported on the new Mac UI starting in Version 16.65.827.0. When you get a chance, could you please update your Outlook on Mac client and try the sample again? Let me know if that resolves the issue.

Hi @samantharamon
I tested the same after upgrading to 16.66.2 Mac new UI on BigSur
The manifest file used is https://github.com/OfficeDev/Office-Add-in-samples/blob/main/Samples/outlook-check-item-categories/manifest-localhost.xml
The same addin is deployed on windows and gives smart alert as below.

But on Mac the same addin doesnt give any error.
Do you want me to collect diagnostic logs ?

Thanks for testing that and providing additional details, @mkondadg. Let me test this further this week to see if I can replicate the issue. Are you able to run the add-in created from the Smart Alerts walkthrough on a Mac by any chance?

A bit progress but not completely working.
Used below manifest file with most of the data from https://learn.microsoft.com/office/dev/add-ins/outlook/smart-alerts-onmessagesend-walkthrough?tabs=xmlmanifest
smartalerts-mac-manifest.xml.zip
Smart Alert seen

But mail not sent after clicking on "Send Anyway"

Thanks for sharing the manifest you're using, @mkondadg. I'll test this further and get back to you.

Thank you again for providing your test results, @mkondadg. I'm able to replicate the issue you're experiencing. I'll be working with the team to troubleshoot the sample on a Mac further. I forgot to request this in an earlier post. When you get a chance, may I request for your diagnostic logs?
I created an add-in from scratch using the Smart Alerts walkthrough and verified that it runs as expected on the Mac platform (this is without incorporating the GitHub sample aspects). If you'd like to test the Smart Alerts feature while we look into the GitHub sample further, I recommend completing the walkthrough and customizing your add-in from there. Apologies for the inconvenience.

@samantharamon could you provide the manifest xml generated using https://learn.microsoft.com/office/dev/add-ins/outlook/smart-alerts-onmessagesend-walkthrough?tabs=xmlmanifest ?
Could you let me know how to generate the diagnostic logs for this case which will be useful to you ?

We use the add-in project created from the Outlook quick start as a basis (see the 'Set up your environment' section of the walkthrough), then replace the <VersionOverrides> element in the manifest with the one in the walkthrough. This is the manifest used: manifest.zip. Let me know if that works for you.
To enable and generate logs, please refer to Runtime logging on Mac. Once you have logging set up, please run the GitHub sample add-in, then attach the .txt file to this post. Thanks!

@samantharamon Unable to reproduce the sendmail issue even with the same manifest xml I provided earlier. I tried redeploying multiple times and even restarted the machine and disabled runtime logging as well.
The outlook app version is still the same.
Could you try testing in your previously reproducible setup ?
I deployed the manifest you provided and it works as expected.

Thank you for the update, @mkondadg. I'm glad to hear you were able to create an add-in using the walkthrough. I tested configuring the sample's SendMode attribute to PromptUser and I'm also now able to send the email after selecting the Send Anyway option. However, I still receive the "add-in is unavailable" dialog. If possible, could you still provide the logs you get from running the sample?

@samantharamon Sorry for the delay.
Using the earlier manifest from sample (which was blocking sending anyway earlier, now sends the mail)
22_smartalerts_partialworking_backup.xml.zip
Enabled this log setting
defaults write com.microsoft.Outlook CEFRuntimeLoggingFile -string "runtime_logs3.txt"
Here is the log
runtime_logs3.txt

No worries, @mkondadg. Thank you for including the manifest and logs.

Thanks for your patience, @mkondadg. We've updated the sample to now support the Outlook on Mac client. When you get a chance, please try sideloading the sample's manifest.xml file and let me know if you're able to run it successfully.

@samantharamon thanks for the update.
Let me try the latest one and get back to you.


Theme global props nested classes not overwritten

Hi there,
we're trying to combine tailwindcss with material ui to overwrite the style and reuse functionality. That's why I started by overwriting classes in the theme global props object.  This works properly for root classes but not for nested classes.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When root classes are overwritten the matching classes in the html also disapear - when nested classes
get overwritten my classes get set but the old classes are still there.
See this example:
export const theme = createMuiTheme({
  props: {
    MuiTextField: {
    classes: {
      root: "some-root-class",  // setting this makes MuiTextField-root class gets removed
    },
    InputLabelProps: {
      classes: {
        root: "some-label-class", // setting these properties should remove the classes MuiInputLabel-root shrink etc but it doesn't
        shrink: "",
        animated: "",
      },
    },
  }
});

Expected Behavior 🤔
I would expect nested classes to work the same way as the root level classes do.

Your Environment 🌎

Tech
Version

Material-UI
v4.9.10

React
16.13.11

Browser
Chrome

TypeScript
3.8.3

@BraunreutherA Do you have a reproduction?

@oliviertassinari I've created a codesandbox with comments: https://codesandbox.io/s/interesting-moon-50wek?file=/src/Demo.js
I created the example for a textfield but the overwrites work the same way for every component.

@BraunreutherA I don't understand, it seems to behave correctly.

In this screen shot you can see, when the property props.MuiTextField.classes.root is not set the class MuiTextField-root is set on the Textfield root.

In this screenshot the props.MuiTextField.classes.root is set to an empty string and the class is removed from the html root. All fine until here.
In both screenshots you can see that props.MuiTextField.InputLabelProps.classes.root is set to an empty string. But even if this is set the class MuiInputLabel-root is still set on the html of the label. That's the issue - why is this still set? The class should be overwritten and therefore removed. You can test this with any nested prop in the theme.

I don't know if it's a problem or not but may I can help @oliviertassinari to understand.
For the TexFieldElement the root class is taken from method getThemeProps. The class then is passed as props to component replacing the default one.
The default class is in classes object while the new class is in more object. The more object contains also the InputLabelProps taken from the theme.
return <Component ref={innerRef || ref} classes={classes} {...more} />;
For the InputLabel instead the class is taken from props because is passed by TextFieldElement.
In this case the root class is merged with the default one.
Maybe a solution for @BraunreutherA is to add to theme the props MuiInputLabel.
const theme = createMuiTheme({
  props: {
    MuiTextField: {
      classes: {
        root: "rootClass"
      },
   },
   MuiInputLabel: {
      classes: {
        root: "labelClass"
      },
   }
 },
});

The problem here is that you will replace all root class for all label input

@marcosvega91 Thanks for the help! :)
Sadly the way you described it doesn't work. It was my first attempt until I had to use other Input elements. The idea is to have a smooth update path by using react-mui native components and upgrade screen as soon as we have the capacity to do it.

Yes I thought that you needed something different but I wrote this solution anyway maybe could help :)

This is now supported in v5. Developers can change the default value of the classes prop as long as it's not already defined. Proof: https://codesandbox.io/s/continuousslider-material-demo-forked-cbnbp?file=/demo.tsx
import * as React from "react";
import { ThemeProvider, createMuiTheme } from "@material-ui/core/styles";
import TextField from "@material-ui/core/TextField";

const theme = createMuiTheme({
  components: {
    MuiInputLabel: {
      defaultProps: {
        classes: {
          root: "some-label-class"
        }
      }
    }
  }
});

export default function Demo() {
  return (
    <ThemeProvider theme={theme}>
      <TextField label="issue #20640" />
    </ThemeProvider>
  );
}

Note that we don't encourage doing it. Prefer applying custom CSS targeting the existing class names.


Make JacocoCoverageVerification cacheable

This task is also expensive. Maybe it would be better to merge JacocoReport and JacocoCoverageVerification in the long run.

cc: @gradle/execution

Fixed via https://github.com/gradle/gradle/pull/16749.


Error Creating role assignments

the deployment completes successfully and all other tasks are successful too but this one fails, Any suggestion what I need to change? I am using an account that is Global admin and has all licenses and a mailbox
Creating role assignments
New-AzRoleAssignment : The client<EMAIL_ADDRESS>with object id '9071b6d0-d2d6-4be6-bc6f-1f4ed44d675e' does not have authorization to perform action
'Microsoft.Authorization/roleAssignments/write' over scope '/subscriptions/16c1d1b2-xxxxxxxxxxxxxxx-4090fe93472e/resourceGroups/weu-t-RequestATeam-rg/providers/Microsoft.Automation
/automationAccounts/teamsautomate-auto/providers/Microsoft.Authorization/roleAssignments/1e87c318-ad20-41dd-87da-cfb3803d1e01' or the scope is invalid. If access was recently
granted, please refresh your credentials.
At C:\Users\Foobar\Downloads\microsoft-teams-apps-requestateam-master\Deployment\Scripts\deploy.ps1:618 char:9

    New-AzRoleAssignment -ObjectId $global:appServicePrincipalId  ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : CloseError: (:) [New-AzRoleAssignment], CloudException
FullyQualifiedErrorId : Microsoft.Azure.Commands.Resources.NewAzureRoleAssignmentCommand

New-AzRoleAssignment : The client<EMAIL_ADDRESS>with object id '9071b6d0-d2d6-4be6-bc6f-1f4ed44d675e' does not have authorization to perform action
'Microsoft.Authorization/roleAssignments/write' over scope '/subscriptions/16c1d1b2-xxxxxxxxxxxxxx-4090fe93472e/resourceGroups/weu-t-RequestATeam-rg/providers/Microsoft.Automation
/automationAccounts/teamsautomate-auto/providers/Microsoft.Authorization/roleAssignments/897dd8d3-a6f3-4118-83d4-8f08f438d4b5' or the scope is invalid. If access was recently
granted, please refresh your credentials.
At C:\Users\Foobar\Downloads\microsoft-teams-apps-requestateam-master\Deployment\Scripts\deploy.ps1:619 char:9

    New-AzRoleAssignment -ObjectId $global:appServicePrincipalId  ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : CloseError: (:) [New-AzRoleAssignment], CloudException
FullyQualifiedErrorId : Microsoft.Azure.Commands.Resources.NewAzureRoleAssignmentCommand

My deployment fails on the same step, although with different error:
PS D:\DevProjects\PowerApps\RequestTeamApp\Deployment\Scripts> New-AzRoleAssignment -ObjectId $global:appServicePrincipalId -RoleDefinitionName "Automation Job Operator" -ResourceName $automationAccountName -ResourceType "Microsoft.Automation/automationAccounts" -ResourceGroupName $ResourceGroupName
New-AzRoleAssignment : Input string was not in a correct format.
At line:1 char:1

New-AzRoleAssignment -ObjectId $global:appServicePrincipalId -RoleDef ...

  + CategoryInfo          : CloseError: (:) [New-AzRoleAssignment], FormatException
  + FullyQualifiedErrorId : Microsoft.Azure.Commands.Resources.NewAzureRoleAssignmentCommand

However, error mentioned by @mcasaz shows up in the actual logic app:

I have the same issues, Input string was not in a correct format.

@mcasaz @Humpeklumpen can you let me know what permissions the account running the script/the account you are signing into at the popups has please?

@alexc-MSFT I mentioned in my post I am using a Global Admin. Would that answer your question?

@alexc-MSFT  The same, im using Global admin

I am getting the same error "not in a correct format" error.
Running the script as global admin with owner rights to the subscription.

Did some research myself and found a referance to the New-AzRoleAssignment cmdlet was failing with Az.Resources version 2.5.0.
By downgrading to 2.3.0 it worked without errors.

@alexc-MSFT why has this been labelled as "invalid"?

I have done all the configuration till the end and tested the app. The request in PowerApps and Flow go through but I do get the same error in Logic App when trying to create an Azure job

"The client 'b93ba0bf-98ad-4be1-8862-a83ebe741cb9' with object id 'b93ba0bf-98ad-4be1-8862-a83ebe741cb9'
does not have authorization to perform action 'Microsoft.Automation/automationAccounts/jobs/write' over scope
'/subscriptions/xxxxxxxxxxxxxxxxxxxxx/resourceGroups/weu-t-RequestATeam-rg/providers/Microsoft.Automation/automationAccounts/teamsautomate-auto/jobs/1e72313e-1004-4eb4-b656-4e96d63e70fc'
or the scope is invalid. If access was recently granted, please refresh your credentials."
@Lysakers how did you actually solve the issue? Did you redeploy?

@mcasaz I uninstalled the Az.Resources module then reinstalled version 2.3.0 of it.
Instead of redeploying i just picked apart the deployment script and ran the few lines of code needed to grant premissions.
Though i imagine if you go to your automation account and add the Logic app with the correct permissions that should solve the error you are seeing:

Thanks @Lysakers @marcinciezak for flagging this up, the runbook is shortly going to be removed and replaced with a Logic App so I'm going to hold off updating the deployment script for now. Will keep this issue open until the new code is commited.

Closing this issue as the implementation of a new logic app negates the need for automation and therefore this issue is no longer there.


Support for Ruby 3

Requesting support for Ruby 3.
Thanks.

https://github.com/oggy/looksee/pull/55

https://github.com/oggy/looksee/pull/55

@oggy any plans of merging #55 ?
@mathieujobin thanks much 🙏

@oggy any plans of merging #55 ?
@mathieujobin thanks much 🙏

https://github.com/oggy/looksee/pull/55#issuecomment-760108294

https://github.com/oggy/looksee/pull/55#issuecomment-760108294

Merged!

Merged!


[interoperator] Bind timeout during watch refresh

Fetch clusterID only if needed.
Ignore clusterID err on delete

Feature: NGPBUG-91418

Pull Request Test Coverage Report for Build 3063

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

Totals

Change from base Build 3058:
0.008%

Covered Lines:
9359

Relevant Lines:
10053

💛  - Coveralls

Can you please raise PR on rel-2019.T07a branch also


[REQUEST] Coloring email's in assignment if there is dot in between

When I do<EMAIL_ADDRESS>It prints as:

Would it be possible to detect string as whole where print<EMAIL_ADDRESS>as yellow.
If not how can I disable coloring for the<EMAIL_ADDRESS>and print is just white?

You could use highlight=False, to disable that behaviour.
You could also define your own highlighter like this

You could use highlight=False, to disable that behaviour.
You could also define your own highlighter like this
For your use case you could do:
from rich.console import Console
from rich.highlighter import RegexHighlighter
from rich.theme import Theme

class EmailHighlighter(RegexHighlighter):
    """Apply style to anything that looks like an email."""

    base_style = "example."
    highlights = [r"(?P<email>[\w-]+@([\w-]+\.)+[\w-]+)"]

theme = Theme({"example.email": "yellow"})
console = Console(highlighter=EmailHighlighter(), theme=theme)

console.print("Send funds to<EMAIL_ADDRESS>

Thanks but it does not work for<EMAIL_ADDRESS>I am seeing as:

@avatar-lavventura You could simply modify the regex
from rich.console import Console
from rich.highlighter import RegexHighlighter
from rich.theme import Theme

class EmailHighlighter(RegexHighlighter):
    """Apply style to anything that looks like an email."""

    base_style = "example."
    highlights = [r"(?P<email>[-\w\.]+@([\w-]+\.)+[\w-]+)"]

theme = Theme({"example.email": "bold u yellow"})
console = Console(highlighter=EmailHighlighter(), theme=theme)

<EMAIL_ADDRESS>
You could also just use something like:
console.print('provider =<EMAIL_ADDRESS>


inconsistent punctuation in suggestion

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


fixed grammar and phrasing

Okay I'll merge it when I back home lol. Thanks for helping me to check my grammar issue :>

Aww man I was going to do this.


Improve styling of the game trivia.

I think the trivia game needs an improvement in the styling and I can make it a little better. I can also add some more questions in the game and make it a little more appealing. Please assign this issue to me. Under the tag gssoc'23.

Assigned this issue to you kindly get it done within a week.

Alright I will try to do it in a given time....


Add outDir to default scully.config.js

🧩 Feature request
When I run the schematic to add scully, and it generates a scully.config.js, we need to add the outDir to the scully.config.js when it is generated.
Also, we need to warn them if the provide a config that doesn't have an outDir and the warning needs to tell them where we will put the output.

The error message seems to indicate that the option should be outFolder?

The option outFolder isn't configures, we are using "/Users/splaktar/Git/.../dist/static/ by default now"
Cleaned up /Users/splaktar/Git/.../dist/static folder.

Adding that to my config, seems to have stopped the message:
exports.config = {
  projectRoot: "./src/app",
  outFolder: "./dist/static",
  routes: {}
}

Cleaned up /Users/splaktar/Git/.../dist/static folder.

@Splaktar I will update the wording to be a bit more descriptive.
Im not sure what happened, but this issue is dealt with, so I'm going to close it now


Getting started guide only explains how to do things with brew but doesn't install brew

We should either say that "brew" is a recommend prerequisite and then show you how to install it, or we should give instructions for how to install things without it. For Dart we almost do the latter (see #1884), but later we just assume you have brew available.

At this point the only thing we use homebrew for is ideviceinstaller. I think we'd be better off just internalising that dependency somehow. Filed that as https://github.com/flutter/flutter/issues/1907 which would make this bug moot.

We could also consider have a brew recipe to install flutter. That would be an easy install process for mac users. We could also use the idea from here: https://github.com/flutter/flutter/issues/1805, and have brew install other needed deps for us in one go.

that sounds good to be. not sure if brew supports the "check out a github repo in a subdirectory of the user's home directory" model though.

Will be addressed by https://docs.google.com/document/d/1wL_7BvpExxyvLqtf5am3_iMQThWBZ0Ak1uPVFiI3KN8/edit?usp=sharing

Made this fix for the iOS setup instructions in the Flutter Setup page, based on doc shared by<EMAIL_ADDRESS>

Marking as closed.


[TSK-227] Repository initiated

@Rahul-7131 is this ready for merging or you plan to do some more stuff?

@maany this is ready for merging

Hi @Rahul-7131 ! I'm taking a look at this scraper. First small remark, are you using the following libraries?

torch (pytorch) (~1GB)
and nvidia_cublas (>300MB)

Asking because they are in requirements.txt and are very heavy, so the setup of this project is slow. If you're not using them, I'll just remove them from the requirements file

Yeah @alebg ,please do remove them, I was testing them for an image recognition model which I have shifted to climate augmentation repo. They aren't necessary for this scraper here.

@Rahul-7131 I cleaned up the setup a bit. Great job! Everything works. Will be working on some fixes to integrate it with the new websat page, but in a different branch. For now I'll just merge this.


Fix enum naming for avoid collision

Related: https://github.com/micro-ROS/micro_ros_arduino/runs/5107455710?check_suite_focus=true

@mergify backport foxy galactic

Codecov Report

Merging #242 (8931eeb) into master (9c844f1) will decrease coverage by 0.18%.
The diff coverage is 86.44%.

@@            Coverage Diff             @@
##           master     #242      +/-   ##
==========================================
- Coverage   63.61%   63.42%   -0.19%
==========================================
  Files          16       16
  Lines        2149     2149
  Branches      641      641
==========================================
- Hits         1367     1363       -4
- Misses        470      472       +2
- Partials      312      314       +2

Impacted Files
Coverage Δ

rclc/src/rclc/executor_handle.c
69.86% <53.84%> (ø)

rclc/src/rclc/executor.c
61.90% <95.65%> (ø)

rclc/src/rclc/action_goal_handle.c
59.87% <0.00%> (-2.55%)
:arrow_down:

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 9c844f1...8931eeb. Read the comment docs.


Show current page on menu

Description
The menu doesn't show the proper current page when not navigating directly through the menu.

The menu should be updated depending on the actual current page I am in.
Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%.
[ ] Propagate any change on the demos and run all of them to ensure there is no breaking change.
[ ] Ensure any change is well documented.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

Hi, I would like to work on this.

hey @heysolomon , are you still working on this issue nuh! ?
'cause there no assignee on this issue and also i am a beginner contributor so if you like to i would love to work on this issue with you and i also get a context of the project

@NILAY1556 I haven’t started working on this issue yet as I haven’t been assigned to it yet. I have no problem collaborating with you on the issue. @FlorianJacta @FredLL-Avaiga is it possible for two contributors to be assigned on an issue?

We can do with two contributors.

@namnguyen20999 is also working on it inside of our R&D.

I don't think there's enough meat on this issue for more than one especially if it is @namnguyen20999 ;-)

This current issue has not been reviewed by the maintainer team and is not open to external contribution.
Please look for issues labeled "good first issue" or "help wanted", as they are opened for an external contribution.
Thank you.


Add Branches Whitelist article

@jpreynat Good first try :+1:
It was a bit confusing, and sounded very "marketing", so I reworded parts of it with examples so that's easier to understand and more to the point.

@jpreynat If that's good for you, merge it and let me know once it's done; otherwise comment the content.

Thanks Aaron for your comments and these changes, this is indeed a lot
clearer for users =)
I just merged the PR.
Le ven. 13 nov. 2015 à 12:25, Samy Pessé<EMAIL_ADDRESS>a
écrit :

@jpreynat https://github.com/jpreynat If that's good for you, merge it
and let me know once it's done; otherwise comment the content.
—
Reply to this email directly or view it on GitHub
https://github.com/GitbookIO/blog/pull/42#issuecomment-156402417.


Particles start freezing after 3min

Hi all,
I am trying to use the GVRPlaneEmitter but after 3min the particles start to freeze!
The same behavior you can see at gvr-particles demo.
Does anybody could help me about this?

I will take a look.

@ragner how many particles before they freeze in place? Just the particle freezes not the app right?

@ragner Is it always reproducible? Could you attach logs? It is yet to reproduce for me (S8).

Hi @all,
yes it is always reproducible. You can see this issue at gvr-particles demo from GVRf-Demos repository.
@bohuang3d,
yes, just the particles freezes ... the other scene objects continue rendering well. The app does not freeze.
It look like a shader issue but I did not find the cause ;(

I am not able to reproduce.

@liaxim,
if you do any touch it resets the particle system and the bug is not reproducible ...
You should launch the particles demo  and wait for +- 3min to see this issue

Will try again

Does it gradually get slower and slower? This is what I am noticing on S7 although it hasn't completely stopped yet for me. I guess on S8 it would just take a lot longer than 3 min before the "Big Freeze".

@liaxim  yes,
it gets slower and slower ... It does not freeze completely.
But the applications continues rendering with 60 fps ...

Something about the total time. Once it gets big enough everything seems to slow down.

currentTime is long, but onDrawFrame time parameter is float. float has low precision (10e-7). For accurate callbacks, precision must be above 10 ms; float frameTime must be less than 10000s, or about 3 hours since boot.

Easy fix, breaking a lot of code: onDrawFrame(long)
Compatible fix: deprecate GVRDrawFrameListener, create new interface with new name declaring onDrawFrame(long), create new registerDrawFrameListener() methods (deprecate old ones), etc.etc.etc.

@ragner Another easy, but not cheap fix: https://github.com/Samsung/GearVRf/pull/1496. Works for me pretty well on S7. Tried with some pretty large totalTimes though probably there is a point it will break too.

@liaxim, thank you. I will try it.

Fixed by #1496


Feature: Provide a way to switch to the default version of an SDK

[X] I have read the CONTRIBUTING guidelines
[ ] I've had a conversation on the #cli-development Slack channel about this request.

Feature request

I'm trying to write a script which uses sdkman to switch to a specific version of java, then switches back to the user's default version, but it doesn't seem there's any way to accomplish the latter. One possible solution is to extend the sdk u command such that if a version isn't provided, the default version is used, e.g. sdk u java would switch to the default java version.

Do you mean "current" version by default version?

Hi @donalmurtagh,
you can do something like that using sdk env.
Add an .sdkmanrc where you put your in your SDK version of choice, e.g.  java=22.2.r17-grl.
Then type sdk config to open SDKMAN!'s config and set sdkman_auto_env=true.
Now, whenever you enter the directory, SDKMAN! automatically switched to the version set in the .sdkmanrc file, and switches back to the default as soon as you exit that directory. Works both for Bash and zsh.

Closing this one as there has been no more feedback.


what-if seems to be ignoring some properties

Related command
az deployment group what-if
Describe the bug
The what-if analysis seems to ignore some properties or incorrectly report they not yet set. For example, I'll create an app service with a siteConfig via bicep template and when I run the what-if analysis afterwards, it says the siteConfig values are not yet set. Am I missing something or is this by design and there's a limitation to be accepted with the what-if analysis? I'm looking to utilize bicep for azure resource creation and am starting with existing resources.
To Reproduce
Using bicep with a single module to create an app service plan and app service.
resource appServicePlan 'Microsoft.Web/serverFarms@2022-03-01' = {
  name: 'myAppServicePlan'
  location: location
  sku: {
    name: 'F1'
  }
}

resource appServiceApp 'Microsoft.Web/sites@2022-03-01' = {
  name: 'myAppService'
  location: location
  properties: {
    serverFarmId: appServicePlan.id
    httpsOnly: true
    siteConfig:{
      cors: {
        allowedOrigins: ['https://{some-site.domain}.com']
      }
    }
  }
}

Run az deployment group create --template-file .\main.bicep -g {rg-yourrg} --parameters environmentName=test location=centralus
Observe that the asp and app have been created, including the cors allowed origin

Run az deployment group create --template-file .\main.bicep -g {rg-yourrg} --parameters environmentName=test location=centralus -c
Observe that the what-if analysis says the cors allowed origin is new and will be added, even though it already exists

Expected behavior

Run az deployment group create --template-file .\main.bicep -g {rg-yourrg} --parameters environmentName=test location=centralus
Observe that the asp and app have been created, including the cors allowed origin
Run az deployment group create --template-file .\main.bicep -g {rg-yourrg} --parameters environmentName=test location=centralus -c
Observe that the what-if analysis says there are no modifications to be made

Environment summary

azure-cli 2.40.0
OS Windows 10 19042
Shell Type PowerShell

route to CXP team

@shenglol Could you please help take a look at the what-if related issue?

Hello, any update on this? Thanks!

Adding Service team to look into this.
@shenglol @josephkwchan, @jennyhunter-msft Could you please assist on this ? Thanks in advance.

This is due the web RP not returning siteConfig in GET responses because. It is a known service provider issue and is tracked by https://github.com/Azure/arm-template-whatif/issues/265. Unfortunately, there is no workaround we can do on the What-If side. @zhoxing-ms Feel free to close this since we are tracking it in the What-If issues repo.

@tholmesdev Tracking this here https://github.com/Azure/arm-template-whatif/issues/265


Sync eng/common directory with azure-sdk-tools for PR 9328

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/9328  See eng/common workflow

This is a new build failure occuring in main caused by changes to the mac agents.

/check-enforcer override


WIP: try out karmatic

Closing in favor to plain Karma in https://github.com/google/web-stories-wp/pull/1695


Add blocks for footer

Codecov Report

Merging #138 (df89263) into prep-release (b24191c) will decrease coverage by 0.07%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@               Coverage Diff                @@
##           prep-release     #138      +/-   ##
================================================
- Coverage         48.37%   48.31%   -0.07%
================================================
  Files               131      131
  Lines              4690     4740      +50
================================================
+ Hits               2269     2290      +21
- Misses             2421     2450      +29

see 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Should allow for BCC field to be parsed.

I'm building out a CRM system right now and I have a use case for using the BCC field. If a user responds to a sales lead, they may want to respond to the lead and BCC our<EMAIL_ADDRESS>receiving address to log the email conversation.
I know there are other alternatives such as email syncing (POP/IMAP) and forwarding sent emails to<EMAIL_ADDRESS>but I think having the BCC could allow more flexibility.
The introduction of BCC was first discussed here: https://github.com/thoughtbot/griddler/pull/95

I've added a PR to handle BCCed emails in the Cloudmailin adapter. I don't think anything needs to change in Griddler itself https://github.com/thoughtbot/griddler-cloudmailin/pull/3


v.2.0.0 not displaying load for ES 1.3.1

HI -
I am using kopf v2.0.0 which was the latest for an 1.3.1 cluster and notice that load average is not displaying in either the display bar or node tab. Let me know if you need more information or screenshots.

Oh just realized its not supported. My bad!


dev-cmd: run Homebrew.install_bundler_gems! earlier.

Run Homebrew.install_bundler_gems! a bit earlier for some developer commands to avoid printing gem installation output in the middle of normal developer command output.

Review period will end on 2021-04-30 at 10:23:30 UTC.

Review period ended.


Changed AustriaLocale

This fixes #459:

renamed AustriaLocale to AustrianLocale
changed name of first month to "Jänner"

Codecov Report

Merging #461 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #461   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          14     14
  Lines        3111   3112    +1
  Branches      226    226
=====================================
+ Hits         3111   3112    +1

Impacted Files
Coverage Δ

arrow/locales.py
100% <100%> (ø)
:arrow_up:

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 9186a2a...9c2ca43. Read the comment docs.


Locust fails with unknown flag

Running the load test gives an error: no such option: --num-request message and exits.
Reproduction

Install locust as per instructions pip install locustio
Execute runLocust.sh with arguments

or, using Docker

Build docker image as per instructions docker build -t load-test .
Execute docker run load-test with arguments

Expected Behavior
Locust executes.
Actual Behavior
The following error:
$ ./runLocust.sh -h sock-shop.example.com -c 5 -r 10
Default Locust file: locustfile.py
Will run locustfile.py against sock-shop.example.com. Spawning 5 clients and 10 total requests.
Usage: locust [options] [LocustClass [LocustClass2 ... ]]

locust: error: no such option: --num-request
done

or, using Docker
$ docker run load-test -h sock-shop.example.com -c 5 -r 10
Locust file: /config/locustfile.py
Will run /config/locustfile.py against sock-shop.example.com. Spawning 5 clients and 10 total requests.
Usage: locust [options] [LocustClass [LocustClass2 ... ]]

locust: error: no such option: --num-request
done

Environment

Tool
Version

Python
Python 2.7.15rc1

Locust
Locust 0.11.0

Docker
18.09.2

A couple of workarounds are possible. The https://hub.docker.com/r/weaveworksdemos/load-test image is available and appears to work.
For example:
docker run weaveworksdemos/load-test:0.1.1 -h sock-shop.example.com -c 5 -r 10


Add Winston for Logging Application Details in Backend

Implement Winston for structured logging to replace console.log() statements. Ensure different log levels (info, error, etc.) are used appropriately and logging works in both development (console) and production (file).

@rishyym0927 ,
Great Idea,
go ahead.
The level of this issue would depend on the work you do.


testNotificationsForChangesWhileSuspended occasionally fails

See https://ci.realm.io/job/objc_pr/1031/configuration=Debug,target=osx/console

This only seems to occasionally fail on Debug OSX builds.

I was unable to reproduce this test failure, even when running ~40 times on the workers, even when their hosts were under heavy load.

@tgoyne observed this this morning on his machine -- appeared to be waiting for the process to finish when it was no longer alive


solved #4938 #4939 #4940

solved #4938 #4939 #4940
Description
Fantasy books:
Added quiz
Added solutions to quiz
Added approach

Type of PR

[ ] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[ ] I have made this change from my own.
[ ] I have taken help from some online resources.
[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers and screenshots after making the changes.

@atmajaa please merge this


Model include added in integrations layout

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 32f49c121e976c7064fdc97c5cf29d1c6fe0ca8a at: https://6627bb52374b210093328bee--meshery-docs-preview.netlify.app

🚀 Preview for commit a76e60e3024cd76a6cc5ebc9a6ee38198ace4eea at: https://662f7ea58ab10800a7e652bc--meshery-docs-preview.netlify.app

🚀 Preview for commit d9088b625c92e42ba3e2ffd2100b7f9086882575 at: https://66364d4d2788c4059d1f410c--meshery-docs-preview.netlify.app


Set default Linux path

In this PR I set default Neovim path to /usr/bin/nvim for Linux. I think this can be reasonable default.

I also noticed that just nvim as a path works for me. Is it expected?

Oh, I think I got why you not set this path as default. If the path is wrong, then the user will not receive a message that they need to specify the path to Neovim, right?

I also noticed that just nvim as a path works for me. Is it expected?

Looks like spawn() uses system path variable. But extension host may override path, not load the one from a user profile so it's easier to just tell specify full path

Oh, I think I got why you not set this path as default. If the path is wrong, then the user will not receive a message that they need to specify the path to Neovim, right?

Correct. However this can be fixed, move https://github.com/asvetliakov/vscode-neovim/blob/f55e2374864b6b62f890a170ad3cddc481661639/src/controller.ts#L304 to top of the constructor (so we won't bind anything if there is an error), check spawn result and throw error if not spawned. Catch in extension.ts and display error message

Correct. However this can be fixed

Sorry, but I do not have any knowledge in TS :(
But I like the idea. This will notify user if Neovim executable does not exist.

Looks like spawn() uses system path variable.

What if try to execute nvim from PATH if the previous check if failed?

I will close this PR and open a new one.


Feature/#1015

Types of Changes
Prerequisites
Please make sure you can check the following two boxes:

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project

Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[ ] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] All new and existing tests passed

Description
Fix #1015
Fixed behavior of Url.SearchParams with empty query
old behavior
var url = new AngleSharp.Dom.Url("http://example.com/foo/bar");
Console.WriteLine(url.SearchParams.ToString()); // "="

new behavior
var url = new AngleSharp.Dom.Url("http://example.com/foo/bar");
Console.WriteLine(url.SearchParams.ToString()); // ""

Assigning null to Url.Query
old behavior
var url = new AngleSharp.Dom.Url("http://example.com/foo/bar");
Console.WriteLine(url.ToString());      // "http://example.com/foo/bar"
Console.WriteLine(url.Query ?? "null"); // null
url.Query = null;
Console.WriteLine(url.ToString());      // "http://example.com/foo/bar?"
Console.WriteLine(url.Query ?? "null"); // ""

new behavior
var url = new AngleSharp.Dom.Url("http://example.com/foo/bar");
Console.WriteLine(url.ToString());      // "http://example.com/foo/bar"
Console.WriteLine(url.Query ?? "null"); // null
url.Query = null;
Console.WriteLine(url.ToString());      // "http://example.com/foo/bar"
Console.WriteLine(url.Query ?? "null"); // null

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


Make variable PBC optional

Because it is not part of H5MD this should be a kwarg and log a warning that the file might not be readable!

Also store as a different key?


Add cisco.ios inventory file

This will be used by cisco.ios network-ee-integration-tests jobs.
Signed-off-by: Paul Belanger<EMAIL_ADDRESS>

Pull-request updated, HEAD is now 9f9b92120bcbdb16dac7f3b04e64a519ce4e8202

Pull-request updated, HEAD is now 37baad6671ab924eeb43355124ea33b66145f1af

recheck

recheck

Pull-request updated, HEAD is now 273e21dbe3e1ed67e3166c55e385121953dc1d52


Error with install

I found some errors trying to configure cmake with install option: cmake .. -DRSID_INSTALL=ON
-- Build type: Release
-- Linux platform
-- CMAKE_BUILD_TYPE = Release
-- VERSION = 2.1.0, BUILD = 20210707
-- 64-bit build (x86_64)
-- CMAKE_INSTALL_PREFIX = /opt/RealSenseID
-- CMAKE_INSTALL_BINDIR = bin (/opt/RealSenseID/bin)
-- CMAKE_INSTALL_DATAROOTDIR =  (/opt/RealSenseID)
-- CMAKE_INSTALL_DOCDIR = doc (/opt/RealSenseID/doc)
-- CMAKE_INSTALL_INCLUDEDIR = include (/opt/RealSenseID/include)
-- CMAKE_INSTALL_LIBDIR = lib64 (/opt/RealSenseID/lib64)
-- CMAKE_INSTALL_MANDIR = man (/opt/RealSenseID/man)
-- Shared libraries disabled (ENABLE_SHARED = 0)
-- Static libraries enabled (ENABLE_STATIC = 1)
-- 12-bit JPEG support disabled (WITH_12BIT = 0)
-- Arithmetic decoding support enabled (WITH_ARITH_DEC = 1)
-- Arithmetic encoding support enabled (WITH_ARITH_ENC = 1)
-- TurboJPEG API library disabled (WITH_TURBOJPEG = 0)
-- TurboJPEG Java wrapper disabled (WITH_JAVA = 0)
-- In-memory source/destination managers enabled (WITH_MEM_SRCDST = 1)
-- Emulating libjpeg API/ABI v6.2 (WITH_JPEG7 = 0, WITH_JPEG8 = 0)
-- libjpeg API shared library version = 62.3.0
-- Compiler flags =  -O3 -DNDEBUG
-- Linker flags =
-- Compiler supports pointers to undefined structures.
-- INLINE = __inline__ __attribute__((always_inline)) (FORCE_INLINE = 1)
-- Linker supports GNU-style version scripts
-- CMAKE_EXECUTABLE_SUFFIX =
-- SIMD extensions: None (WITH_SIMD = 0)
-- FLOATTEST = no-fp-contract
-- RPM architecture = x86_64, DEB architecture = amd64
-- Generating install..
CMake Error at cmake/Install.cmake:19 (install):
  install TARGETS given target "rsid-cli" which does not exist in this
  directory.
Call Stack (most recent call first):
  CMakeLists.txt:119 (include)

CMake Error at cmake/Install.cmake:26 (install):
  install TARGETS given target "rsid" which does not exist in this directory.
Call Stack (most recent call first):
  CMakeLists.txt:119 (include)

-- Configuring incomplete, errors occurred!

To me it looks like you tried to install to /opt/ without sufficient permissions...

I just did cmake ... not make install

-- CMAKE_INSTALL_DATAROOTDIR =  (/opt/RealSenseID)

My understanding is it's a configuration problem with your cmake install.  It looks like you are running a version of cmake which is trying to populate the build files in the /opt directory.  When you run cmake, it makes files for make to use.  /opt/ is a great place to keep 3rd party programs that are installed outside of your package manager.


The "Foundation" way to vertical align...?

I emailed Zurb about this yesterday and the Lead Design, Jonathan, kindly emailed me back, but I wanted to open it up for other people in case you have any ideas.
I am a Foundation newbie but I like the idea of a cross-platform framework that really works.
The challenge I have found is with a fairly simple header - logo in the top left with a row or social media icons to the right.
Pretty standard stuff, but the logo is much bigger than the icons, and I need the icons to align to the bottom of the logo.
In Foundation, we're working mostly with blocks of content, so vertical-align won't work.
I could fix the relative position of the icons, but again that's not going to work well with Foundation.
But making the images inline means that you're not really using Foundation at all.
Also, if the images are inline, I get a rather unattractive rearrangement of the icons when I squeeze the browser to a certain width. Obviously in an ideal world, you'd want the three images to stay grouped but because they're inline, the last icon in the line drops down the page as you narrow the browser width, then the second-to-last icon drops down as you narrow it further.
Here's a simple version of the code with the images inline - how would you do it the "Foundation" way so that the images align to the bottom and still look great if the icons are wrapped underneath the logo...?

NO! inline CSS is bad bad bad.

Split your header into 2 columns may be a solution.

My original approach was to use two columns. That's when I noticed the problem. The two columns are blocks, so I can't use vertical-align. And if I start fixing the height or position, I'm, not really using Foundation properly.

can you not wrap the social icons in a  with their own unique ID and then push them to the bottom with padding-top? and don't forget to use the mobile class three-up.

oops, guess I can't put code in here. It should read:
can you not wrap the social icons in a ul tag with their own unique ID and then push them to the bottom with padding-top? and don't forget to use the mobile class three-up.

Thanks Jason. I can see how that would work, but it feels like vanilla CSS rather than Foundation CSS, if you get my meaning. It's much the same as using absolute or relative positioning.
Say I add 100 pixels of padding-top to an icon. That's fine when I'm viewing it in a web browser, because that icon is on the right-hand-side of the page and it will align neatly with the logo to its left. However, on a mobile 'phone browser, there will still be 100 pixels of padding-top, but this time the icon is underneath the logo, so there's a needless 100 pixels of vertical space between them.
If that's the case, then I need to have two slightly different versions of the CSS. Which is fine. But can't we do better!?

Just add a padding-top: 0; to your mobile media query and done.

Why not use a combination of position: relative; on the parent and position: absolute; bottom: 0; on the smaller elements?

I was involved in similar problem. Trying to get my login input field to align to bottom of its column div.  This did the trick for me and is part of Foundation...


upload by other sub-domain with CKEditor

hi , i want to ask , how to config laravel-filemanager if my image server using other sub-domain?
ex. myadmin panel : http://backend.xxx.com/
my image : http://images.xxx.com

Try using <iframe> to embed filemanager in your admin panel.

We should add some example code to document about this case.


Fix solc-nightly job

Fixes #1605.
Also had to update Truffle to get this to work.

I also moved scripts/build.sh to scripts/prepack.sh because it was very ambiguous. That (nasty) script is only used to prepare the package.

Did you chmod +x the compile script?

Yeah, sorry about that.


pscale: 0.130.0 -> 0.133.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/planetscale/cli/releases.
meta.description for pscale is: The CLI for PlanetScale Database
meta.homepage for pscale is: https://www.planetscale.com/
meta.changelog for pscale is: https://github.com/planetscale/cli/releases/tag/v0.131.0
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.131.0 with grep in /nix/store/z5l8y3dicp808lmlj3sksbl3144nb32p-pscale-0.131.0
found 0.131.0 in filename of file in /nix/store/z5l8y3dicp808lmlj3sksbl3144nb32p-pscale-0.131.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pscale

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

pscale

Maintainer pings
cc @pimeys for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/planetscale/cli/releases.
meta.description for pscale is: The CLI for PlanetScale Database
meta.homepage for pscale is: https://www.planetscale.com/
meta.changelog for pscale is: https://github.com/planetscale/cli/releases/tag/v0.133.0
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.133.0 with grep in /nix/store/lw84qy60zbcnq46vqsg91a5xa3kfnhns-pscale-0.133.0
found 0.133.0 in filename of file in /nix/store/lw84qy60zbcnq46vqsg91a5xa3kfnhns-pscale-0.133.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pscale

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

pscale

Maintainer pings
cc @pimeys for testing.


Update request for lz4 submodule

Hi T-mat,
First off, thanks for the great C++ port for lz4. The cross-platform ability is a huge benefit.
I was wondering, might you have an opportunity to update the lz4 submodule? The commit that this repo links to is quite old and it has been updated a fair bit since then. Unfortunately, they have moved files around and re-named some functions in the lz4 repo so I couldn't just easily update the submodule to the head myself. Some of your code will need to be modified a bit as well.
Thanks!

:+1 Please update lz4 to r131 when you have a chance.

Has there been any update on this?
Is lz4mt abandoned or will it be upgraded to r131 or newer?


fix(qp_interface): fix unreadVariable

Description
This is a fix based on cppcheck unreadVariable warnings.
common/qp_interface/src/osqp_interface.cpp:359:46: style: Variable 'sol_lagrange_multiplier' is assigned a value that is never used. [unreadVariable]
  std::vector<double> sol_lagrange_multiplier(sol_y, sol_y + data_->m);
                                             ^

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

Currently the warning is suppressed with a comment, but if it is not going to be used in the future, it can be removed.

Variables were removed because clearly not used.

@rej55 @maxime-clem @satoshi-ota @takayuki5168


option for dashes at top of file

Our convention for YAML files is to begin with three dashes, and then to begin at a single indentation level.
---
  a: a
  b: b

When we dump an object literal like {a: 'a', b: 'b'}, the resulting file does not have the three dashes we expect. Is there an option to enable that type of behavior? Our workaround is to simply add that line, and indent every subsequent line, after dumping the object as YAML.
Thank you for your advice!

Hello its a same code

The default configuration of yamllint will warn if the three dashes aren't present at the start. That does not expect that all content will be indented.

Is there an option to enable that type of behavior?

No such option is available as far as I remember.
I'd suggest to add it manually, since the task is rather trivial:
let obj = { a: 'a', b: 'b' };
console.log(yaml.safeDump(obj).replace(/^(?!$)/mg, '  ').replace(/^/, '---\n'));


[zh] Fix some typos

Fix some typos:

Guaranted -> Guaranteed
runing -> running
Registery -> Registry

I signed it

I signed it

/lgtm
/approve


Increase the usage of augmented assignment statements

:eyes: Some source code analysis tools can help to find opportunities for improving software components.
:thought_balloon: I propose to increase the usage of augmented assignment statements accordingly.
Would you like to integrate anything from a transformation result which can be generated by a command like the following?
(:point_right: Please check also for questionable change suggestions because of an evolving search pattern.)
[Markus_Elfring@fedora lokal]$ perl -p -i.orig -0777 -e 's/^(?<indentation>\s+)(?<target>\S+)\s*=\s*\k<target>[ \t]*(?<operator>[+\-%&|^@]|\*\*?|\/\/?|<<|>>)/$+{indentation}$+{target} $+{operator}=/gm' $(find ~/Projekte/pvlib-python/lokal -name '*.py')

Hi @elfring, I think we're generally open to improving our codebase, but I'm not sure perl regexes would be the tool of choice to do it.  Seems like a proper linter, or something like black, would make more sense.  What do you think?
Also, any reason why you're interested in augmented assignment statements in particular?
Probably worth pointing out that there is a slim chance that blindly replacing x = x+y with x += y could break something since += generally edits in-place while + creates a new object (though of course it depends on how each object defines __add__ and __iadd__).

…, but I'm not sure perl regexes would be the tool of choice to do it.

The programming language “Perl” provides special support for the application of regular expressions.
Thus it can look convenient to use it for some data processing approaches.

Seems like a proper linter, …, would make more sense.

Such software development tools are evolving in some directions.

Also, any reason why you're interested in augmented assignment statements in particular?

:thought_balloon: I hope that some information (which was published also together with release notes for Python 2) can be taken better into account.
How do you think about to support another bit of collateral evolution? :thinking:

… could break something since += generally edits in-place while + creates a new object

:thought_balloon: This technical detail influences run time characteristics in potentially desirable ways, doesn't it?

(though of course it depends on how each object defines __add__ and __iadd__)

Would you like to clarify your development concerns in more detail? :thinking:


Convert to use MarkedString

MarkedString is deprecated and causes confusion. See https://github.com/Microsoft/vscode/issues/62332

Implementation note:
MarkedString might not be supported by every client of the service. To let clients specify what capabilities they have, I started in the JSON-languageservice that the client can pass in a ClientCapabilty to the service: See https://github.com/Microsoft/vscode-json-languageservice/blob/7b0bc0dcf45ab6411429e986716f969f92773850/src/jsonLanguageTypes.ts#L207

Will do this together with HTML.
Also, https://github.com/Microsoft/vscode-html-languageservice/issues/12 needs to be handled as well. Per the policy we need to show warning when linking to non-Microsoft websites.


Flyway compatibility with Java 8 in JDK 8.0 compatibility mode

What version of Flyway are you using?
Flyway 4.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Command-line
What database are you using (type & version)?
Oracle 11g XE
What operating system are you using?
Centos, Ubuntu 16
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)

Created Java based Migration JAR with JDK 8 compatibility mode to use some streaming features by extending JdbcMigration.
Tried to apply migration

What did you expect to see?

Migration should have been successful

What did you see instead?

Migration failed with "Java unsupported major minor version 52.0".

This is caused by the JRE you use to run Flyway. To make things easy, simply pick the Flyway download with a bundled JRE and all Java 8 features should work.


Can you share the VQ-VAE autoencoder model for the high resolution?

Hi, thanks for your sharing. This is a wonderful work. But this repo only contains the VAE model for the cifar-10 with the low resolution. If possible, can you share the VQ-VAE model for datasets with the high resolution?
And, the "mixup_data" operation in training stage seems to make model more robust.  Have you done some ablation experiments for evaluating this operation?

Thanks for your interest. I have provided the VQ-VAE model for imagenet in CVAE_imagenet in vae.py https://github.com/kai-wen-yang/CD-VAE/blob/fa614cc930cf7c0ae2f19251678f247638ce197b/networks/vae.py#L230
I use it as:
CVAE_imagenet(d=64, k=128, num_classes=num_classes)

The training for imagenet is very similar to cifar.
I find mixup_data can lead to better disentanglement, especially the classification result for R(X). So I use it as the default setting and haven't done ablation experiments for it.

The situation is still the same when the classification network is removed, but my Vqvae is completely based on your code, for binary classification, k=64, D =128 parameters should not be changed？


Move babel config to .babelrc file

Fixes issue #19

please merge - breaks https://github.com/erikras/redux-form mocking with https://facebook.github.io/jest/ as well as react-native

actually, I just tried this and it doesn't work for me. I still get
ReferenceError: Unknown plugin "babel-plugin-transform-builtin-extend" specified in "node_modules/es6-error/.babelrc" at 1, attempted to resolve relative to "node_modules/es6-error"

Updated to not publish .babelrc.

alright, i'll just close my PR as you fixed it the same way. cheers!

Oh, I didn't even see that you'd opened a PR. oops

:D np, it was redundant anyway

Thanks! Published v3.0.1 to npm.


Fix for #689 and improve blank line handling for the ip/ap text-objects

Fixes #689. Also makes the handling of blank lines, and lines that only contain whitespace more like in vim, e.g. you can now select the lines between two paragraphs by doing 'vip' .

Any feedback on this?

I think it looks great.  Definitely works better now.
I notice that hitting undo after 'dap' or 'dip' leaves the cursor at the end of the deleted region.  Vim leaves the cursor at the beginning of the deleted region.  Might be worth opening a new issue for this though, not sure how deep this problem runs.
Ideally you'd combine some of these commits so the changes are more obvious.

Seems to need rebasing please.

Refactored and rebased.

please remove .swp from the commit :sunglasses:

Sneaky .swp removed :)

Great work @MarkusSN; thanks for fixing this!


Fixes for Decodable protocol in DateInRegion/Region on Swift 4.2

Just did a pod update and it broke my project.

Same here

@malcommac Well, it still seems to fail for me in the same way.
Running Sierra and Xcode 9.2.
I'll monitor it.

Same here
MacOS Sierra - Version 10.12.6 (16G1036)
XCode - Version 9.2 (9C40b)

Hi, I'm using 5.0.7 and this bug is still there.
MacOS Sierra - Version 10.12.6
XCode - Version 9.0

Sure it still here, you need to update to 5.0.9, otherwise you still get the same problem :)

Sure it still here, you need to update to 5.0.9, otherwise you still get the same problem :)

Hi, i'm  using 6.0.1 and this bug is still here


fix(crypto-square): put test name on single line

Just to be sure, is the issue the "spaces" or that the whole TEST_CASE line is over two lines?
TEST_CASE(
    "54 character plaintext results in 7 chunks, the last two with trailing "
    "spaces",
    // clang-format off
    "54 character plaintext results in 7 chunks, the last two with trailing spaces",
    // clang-format on
    "[fbcb0c6d-4c39-4a31-83f6-c473baa6af80]")

Here's the relevant comment from the original review: https://github.com/exercism/cpp/pull/827#discussion_r1510922224

@iHiD The test runner searches the name of the tests in the _tests.cpp file.
https://github.com/exercism/cpp-test-runner/blob/main/src/xml2json.cpp#L192
But if the string with the name is split it will not be found.

@ahans Is that second commit really necessary, with all those // clang-format lines? Shouldn't the first one that joins the test name be enough?

That was in response to @iHiD's comment. Just to be on the safe side. I'm still not fully following what the parser is doing. If test_case.name comes from elsewhere (tests.toml?) then it should not be necessary. I can revert it if you prefer. But I think it doesn't matter that much, since it's a temporary thing anyways, isn't it?

I'm still not fully following what the parser is doing. If test_case.name comes from elsewhere (tests.toml?) then it should not be necessary.

The website should be able to display the test code. Therefore the xml2json.cpp extracts the code of each test from the slug_test.cpp.
@vaeng will be able to explain that better than me, he wrote the test runner.

But I think it doesn't matter that much, since it's a temporary thing anyways, isn't it?

I'm not following, why is it temporary?

@ahans Is that second commit really necessary, with all those // clang-format lines? Shouldn't the first one that joins the test name be enough?

@siebenschlaefer  I reverted that now and confirmed with the test runner locally that it still works.
With the crypto-square test from main, this is the error message:
Error: basic_string::substr: __pos (which is<PHONE_NUMBER>3709551615) > this->size() (which is 1966)
/opt/test-runner
crypto-square: done

With the crypto_square_test.cpp from this PR, the error message goes away and the produced JSON file looks reasonable.

But I think it doesn't matter that much, since it's a temporary thing anyways, isn't it?

I'm not following, why is it temporary?

Because it's a known limitation of the test runner that I will look into fixing. I can think of a quite simple fix for the multi-line string issue that we're facing here, but ideally, the parser would be able to handle any valid C++ code. Anyway, I will add some thoughts over at this issue.

The test runner in v3 had to print the code that is actually run for each test.
I wrote the runner to search for the "name" lines in the $slug_test.cpp to parse the content for each test case.
If the name is split, it cannot find the positions and fails.

Thanks all!

One question - why did the CI not detect that the previous test broke things? Maybe something we can add?

One question - why did the CI not detect that the previous test broke things? Maybe something we can add?

We would have to compile the same program that the test runner is using and use that binary with the CI, then check the validity of the resulting json

We would have to compile the same program that the test runner is using and use that binary with the CI, then check the validity of the resulting json

Many test-runners spin up the Dockerfile that's built for production and run some golden tests in there. I'm guessing that's not the case here then? I think @ErikSchierboom has a pretty quick workflow to add that to a repo, so maybe he does that when he's back?

The test runners golden tests are in place. This exercise had a "malformed" test file, that compiles fine, but will not parse the JSON parsing in the test runner during production.
The only way I see to detect this beforehand is to run every commit through the real thing

The only way I see to detect this beforehand is to run every commit through the real thing

Having a full-blown integration test that runs each example implementation through an instance of the Docker image would probably be the most complete solution. But the critical part is the parsing. Wouldn't it be possible to combine this with the regular tests using the example implementations? The executables are built anyway. Just run them again with -r xml, pass the result along with the test source file through exercism_parser. Make sure that it exits successfully and the json file looks reasonable. I think this would have caught this bug.


mapping checklist

Mapping checklist for medical_claim and pharmacy_claim.

Some feedback on this PR...

Pretty sure data_source HAS TO be part of the PK.
Need to explain the difference between patient_id and member_id, otherwise I have no idea how to map the data
Need to explain the difference between payer and plan, otherwise I have no idea how to map the data
admit source and type aren't really used for any analytics and therefore we should remove from audit
the audit should check the following fields for valid values but the explanation doesn't say that it does...

discharge disposition
place of service
bill type
drg
rev center
diag codes
proc codes
Instead the instructions just say something really generic like "The mapping audit checks that the value of this field is a three-character string and that it is consistent across all lines for a given claim_id."

Also let's remove the unioning, customization, and upgrades sections for now.


[JSPM] router-link: unknown custom element warning when used inside child component

Getting the known warning:
Unknown custom element:  - did you register the component correctly?
when using the router-link component inside a child component.
I am using JSPM / System.js and ES6 to import modules.
I use the full version of VueJS (with the compile), importing it:
import Vue from 'vue/dist/vue.js';
I realise the solution to Webpack or Browserify is to not include Vue  like this, but define an alias to the full version. I have done so in SystemJS config, yet still receiving the same warning.
Checked and Vue is being loaded only once, as is VueRouter.
Any help would be greatly appreciated.

Did you call Vue.use(VueRouter)?

@yyx990803 yes, Vue.use(VueRouter) is called.
The router-link components seems to work normally, until it is used in a child component.
This is a general representation of the app structure:
[App] - Uses router-view.

    [MainControllerComponent] - Uses router-view

        [SectionControllerComponent] - Extended MainControllerComponent. Uses router-view.

            [SectionPageControllerComponent] - Extended SectionControllerComponent. Uses router-view. Router-link works here.

                [ListComponent] - Locally registered component of SectionPageControllerComponent. Router-link stops working here.

                    [ListItemComponent] - Router-link doesn't work.

@yyx990803
It does seem that JSPM reloads the Vue library when it is imported in a different file, though that is not seen in the network tab of devtools.
I was able to reproduce and resolve the behaviour:
// === main.js ===

import Vue from 'vue-full';
import VueRouter from 'vue-router';

import LocalComponent from './local-comp.js';

Vue.use(VueRouter);

const app = new Vue({...});

let View1 = Vue.extend({
    components: {
        'local-comp': LocalComponent
    }
});

// === local-comp.js ==

import Vue from 'vue-full';

export Vue.extend({...});

This will cause the warning if router-link is used in the LocalComp component. Interestingly the LocalComp component does have the $route and $router properties available in its instance.
In order to fix this the VueRouter needs to be reapplied whenever Vue is imported in a different file:
// === main.js ===

import Vue from 'vue-full';
import VueRouter from 'vue-router';

import LocalComponent from './local-comp.js';

Vue.use(VueRouter);

const app = new Vue({...});

let View1 = Vue.extend({
    components: {
        'local-comp': LocalComponent
    }
});

// === local-comp.js ==

import Vue from 'vue-full';
import VueRouter from 'vue-router';

Vue.use(VueRouter);

export Vue.extend({...});

See the Vue.use(VueRouter); in the LocalComp file.
To work around this, I made a sort of decorator for vue in a separate file:
// === vue-decorator.js ===

import Vue from 'vue-full';
import VueRouter from 'vue-router';

Vue.use(VueRouter);

export default Vue;

// === other file(s) ===
import Vue from 'vue-decorator.js';

I'm not sure if this is good practice or not, but solves the issue with JSPM.
Sidenote:
If you're using a vue instance as an event hub, you will have to import the default vue library and use the constructor.

@beno-dlabs You should never use Vue.extend (or Vue.component) when exporting components, simply export a plain object instead.
// === local-comp.js ==
export {...}

@fnlctrl Thank you for the information. Can I ask why this shouldn't be used?

When you use Vue.extend() you get a fresh new Vue constructor, you can think of it as a fork of the global Vue. In the future, not all modifications to the global Vue by plugins will be applied to it.
By the way, there is no practical benefit of doing it in the first place - duplicating import Vue and Vue.extend() in every component.

@fnlctrl Ok, understood. Thank you for the extra explanation!


Multiple optimizer passes

Which issue does this PR close?

Closes https://github.com/apache/arrow-datafusion/issues/3879 and https://github.com/apache/arrow-datafusion/issues/3881
Possibly helps with https://github.com/apache/arrow-datafusion/issues/3864 but I have not tested that yet
Rationale for this change

We miss optimizations if we only run the rules once
What changes are included in this PR?

Are there any user-facing changes?

It probably makes sense to have some information now how long the optimization takes & maybe provide a conservative number for the maximum optimization passes as @alamb hinted in an earlier PR

It probably makes sense to have some information now how long the optimization takes & maybe provide a conservative number for the maximum optimization passes as @alamb hinted in an earlier PR

I reduced the default max_passes to 3 and added debug logging for optimizer elapsed time


TextInput behavior on programmatic value change differs between iOS/Android

[x] I have reviewed the documentation
[x] I have searched existing issues
[x] I am using the latest React Native version

If one has a TextInput and changes its value prop, selection behavior differs between iOS and Android. On iOS, the cursor jumps to the end. On Android, the cursor moves to the right by some varying amount.
Also, if the TextInput has focus, on iOS no onSelectionChange even is fired, but one is on Android.
Finally, it would be nice if it were documented somewhere that the initial selection is at the end of the initial value.
Environment
Environment:
OS:  macOS High Sierra 10.13.3
Node:  9.9.0
Yarn:  1.5.1
npm:  5.8.0
Watchman:  4.9.0
Xcode:  Xcode 9.2 Build version 9C40b
Android Studio:  Not Found
Packages: (wanted => installed)
react: 16.3.0-alpha.1 => 16.3.0-alpha.1
react-native: 0.54.2 => 0.54.2
Steps to Reproduce
See https://snack.expo.io/BJUeRfw5M
Expected Behavior
Consistent behavior between platforms. Jumping to the end of the line/value, perhaps, and firing an onSelectionChange event.
Documentation stating that the initial selection is at the end of the initial value.
Actual Behavior
On iOS, the cursor jumps to the end of the line, or the end of the replace text, sometimes. On Android, the cursor moves to the right by some varying amount.
Also, if the TextInput has focus, on iOS no onSelectionChange even is fired, but one is on Android.

A workaround is just to set selection explicitly. But doing so runs into https://github.com/facebook/react-native/issues/18316 , which also has its own workaround. https://github.com/akalin-keybase/rn-text-input-bugs has a demo that implements these workarounds.

Not fixed AFAIK

any news on this issue?
I also ran into this problem, at least in part "if the TextInput has focus, on iOS no onSelectionChange even is fired" when programaticaly change value prop.


feat: Update docs about recent changes related to kyma-dashboard

Description
Changes proposed in this pull request:

Adjust docs to new configuration management

Related issue(s)
https://github.com/kyma-project/busola/issues/3298
Definition of done

[x] The PR's title starts with one of the following prefixes:

feat: A new feature
fix: A bug fix
docs: Documentation only changes
refactor: A code change that neither fixes a bug nor adds a feature
test: Adding tests
chore: Maintainance changes to the build process or auxiliary tools, libraries, workflows, etc.

[x] Related issues are linked. To link internal trackers, use the issue IDs like backlog#4567
[x] Explain clearly why you created the PR and what changes it introduces
[x] All necessary steps are delivered, for example, tests, documentation, merging

/override tide


KDTrees

GPU, N=1_000_00 (ht @polvalente)
Operating System: Linux
CPU Information: 13th Gen Intel(R) Core(TM) i9-13900KS
Number of Available Cores: 32
Available memory: 125.53 GB
Elixir 1.15.1
Erlang 26.0.2

Benchmark suite executing with the following configuration:
warmup: 2 s
time: 10 s
memory time: 2 s
reduction time: 0 ns
parallel: 1
inputs: none specified
Estimated total run time: 28 s

Benchmarking banded 1M ...
Benchmarking unbanded 1M ...

Name                  ips        average  deviation         median         99th %
banded 1M           55.01    0.00030 min     ±2.15%    0.00030 min    0.00032 min
unbanded 1M        0.0167       1.00 min     ±0.00%       1.00 min       1.00 min

Comparison:
banded 1M           55.01
unbanded 1M        0.0167 - 3301.18x slower +1.00 min

Memory usage statistics:

Name           Memory usage
banded 1M        0.00001 GB
unbanded 1M         4.44 GB - 825455.53x memory usage +4.44 GB

**All measurements for memory usage were the same**

What is the dimension of the data you used for benchmarks?

To add to your question/answer, the dimension should not impact the results, as we don't store the tensors and we only cycle through them.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


[Issue-3752] Extension - Update address for TON testnet in the token detail screen on All accounts mode

Related issue(s)

#3752

Is your feature request related to a problem(s)? Please describe.

Update logic show address on balance detail modal

Describe the solution you'd like

Update logic show address on balance detail modal

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-3763--sw-web-runner.netlify.app


Azure SDK Review - [Device Update for IoT Hub]

New SDK Review meeting has been requested.
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 12/13/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 12/04/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 12/04/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 12/04/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here

Meeting updated by Leo Lie
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 12/04/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here

Meeting updated by Leo Lie
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 12/04/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 11/27/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here

Meeting updated by Leo Lie
Service Name: Device Update for IoT Hub
Review Created By: Leo Lie
Review Date: 11/27/2023 02:05 PM PT
Hero Scenarios Link: here
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java, Javascript, .NET, Python,
Description: Adding Limits (stats) API to ADU data plane
Detailed meeting information and documents provided can be accessed here


Server side validation of different type Operators

Currently, only validates operators as an abstract BaseOp. Needs to validate the request body model as a concrete type of operator. This will likely require Weaver changes.

More planning will be needed for a better server side validation regime (Spring?) Size this as database constraints to keep bad objects out.


Convert Vue TS files to JS files

Since many people in the team are not proficient in TS, the TS code written in vue wants to be converted into JS. Is there any good way to do this?

I'm guessing there are some TS to JS conversion tools, but I don't know of any, and that's outside the scope of the project.


non-viral capsid

PMID:37326306
How to describe a non-viral capsid?

From https://www.mdpi.com/1422-0067/24/2/1036
"Ancestral retrotransposon Arc protein recruits ALIX and is secreted. Arc encodes the capsid GAG domain, which self-assembles into a virus-like-capsid containing Arc mRNA. The capsid might recruit ALIX, enhancing the budding of the plasma membrane and the secretion of EVs containing Arc capsid and mRNA"...."For instance, the mammalian gene PEG10 and the neuronal gene ARC are both LTR retrotransposons encoding for GAG analogs. They self-assemble and encapsulate their own mRNA before being secreted in EVs. The PEG10 gene encodes the capsid, nucleocapsid, protease and retro-transcriptase GAG domains."

Made NTR: virus-like capsid


Update default ordering for table when filters are selected

Should order by default:

By grade
By last name (within the same grade)

@travisbloom-cb can you confirm if this is done already? The specific feedback didn't take into account that there are 'promoted' politicians on the landing page

@axelmarko-cb we sort by promoted positioning, then fall back to stance score now. Closing this out.

@travisbloom-cb I think it would make more sense to have:

Promoted
Grade
Last name

Right now the sorting looks pretty random to the user as we don't expose the actual underlying score that leads to a specific grade.


After a long time running, the cpu up to 50%, and the interface was stuck.

OS Version: windows 10
VERSION: 1.28.2 (system setup)
SUBMIT: 7f3ce96ff4729c91352ae6def877e59c561f4850
DATE: 2018-10-17T00:23:51.859Z
Electron: 2.0.9
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
x64

I didn’t have this exception before, the latest ones have appeared.
Restart the system for a few hours or after a dozen hours, it will appear.
What should I provide debugging data for you to test and analyze?
SETP:

code in VSCODE
switch to other VistualDesktop(Win10 Feature), use other application (such as Chrome)
switch back.
as long as I click on VSCODE for editing code, it will Stuck.
wait 30 seconds (maybe), It becomes normal.
4,5,4,5,4,5,4,5.....loop.

Does this issue occur when all extensions are disabled?: No

I just restarted, when there is a problem, I will cut off a video for your reference.

Please follow this on how to file a proper performance issue https://github.com/Microsoft/vscode/wiki/Performance-Issues

I followed the help documentation above to confirm that it should be an extension, specifically which extension, I will give feedback to the corresponding developer.


Weird feathery wing thing near/on the hotbar

https://github.com/RootsTeam/Goetia/issues/8

Fixed in 5.1


fix: CREATED_AFTER_DATE datetime parsing

Pull Request

Proposed Changes

Readiness Checklist
Author/Contributor

[x] If documentation is needed for this change, has that been included in this pull request
[x] run make lint and fix any issues that you have introduced
[x] run make test and ensure you have test coverage for the lines you are introducing

Reviewer

[x] Label as either bug, documentation, enhancement, infrastructure, or breaking

Alright, I'm actually done with it now. Sorry for all of the post PR commits, I'm not used to working on forks so my git commands are getting really sloppy.

The check done with super-linter seems to be at odds with the output of running make lint in this repo.
When I add the suggestions from the Lint Code Base action and then run make lint locally, I get:
...
evergreen.py:7:0: C0411: third party import "import github3" should be placed before "import auth" (wrong-import-order)
evergreen.py:8:0: C0411: third party import "import requests" should be placed before "import auth" (wrong-import-order)

Which of these should I be prioritizing? I'm guessing super-linter since it is the one with the big :x: on my PR?

Let me see If I can get them to agree on style since that would be ideal...


Review data structs for agent/services metrics

These are the metrics we upload to wmstats (agentInfo), here:
https://cmsweb.cern.ch/couchdb/_utils/database.html?wmstats/_design/WMStatsErl/_view/agentInfo
Benjamin discussed with Alberto (CERN IT) and we have to improve some of these metrics in order to have an easier data lookup and aggregation.
BTW, dots '.' are not allowed in key names. @stiegerb can you please:
*) find out which regex is used to validate these key names
*) and comment here on a few cases - with examples - that should be changed from/to
?
As we discussed, I'm assigning this issue to you :) but if it becomes urgent and you don't manage to check it, we can always reassign it.

This is one (big) api used to collect global workqueue information:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WorkQueue/WorkQueue.py#L1153
and then some of these APIs are again used on the agent side:
https://github.com/dmwm/WMCore/blob/master/src/python/WMComponent/AgentStatusWatcher/AgentStatusPoller.py#L133

About allowed key names, it seems only dots are a problem.
One example is the workByAgentAndPriority for the global_workqueue agent, where we use the agent hostname as field names:
  "workByAgentAndPriority": {
       "http://vocms0130.cern.ch:5984": {
           "900000": {
               "count": 7,
               "min": 1,
               "max": 112,
               "sum": 169,
               "sumsqr": 13387
           }
       }, ...

This falls under the restructuring anyway, so we should fix both in one go.
One other thing, we have a top level type field, which is always agent_info. Shall we change that to something else for the reqmgr and global_workqueue documents?


Modify for GroupMappingService extension

What changes are proposed in this pull request?
Modify for GroupMappingService extension
Why are the changes needed?
Modify for GroupMappingService extension
Does this PR introduce any user facing changes?
NA

alluxio-bot, merge this please


Headers in link is not considered in resolve fully

Hi team,
Links is considered in resolve fully option and headers field in link is not considered in resolve fully.
issue_884.txt
Thanks,
Mohammed

So I am having similar issues where a valid spec does not contain any headers whatsoever, but what you attached here is not a valid spec.
Please paste this in an editor like [editor.swagger.io](swagger editor) and you will see that the UserRepository schema is not correct:
    Schema error at components.links.userRepository
    should NOT have additional properties
    additionalProperty: operationId, headers

Can you specify why this spec should be working and how it is used? Maybe we can help eachother out.

Fixed by #937