[Bug Report]: Problems adding an ip address into the network acl

Describe the bug
If i try to add an ip address into the network acl part like so
networkAcls: {
defaultAction: 'Deny'
ipRules: [
{
value: 'xxx.xx.xxx.xx'
action: 'Allow'
}
]
}
It doesn't work using the storage bicep file.
If I create my own then it works using:-
resource stor 'Microsoft.Storage/storageAccounts@2022-09-01' = {
name: 'gregstoragetest'
location: 'westeurope'
sku: {
name: 'Standard_ZRS'
}
kind: 'StorageV2'
properties: {
networkAcls: {
defaultAction: 'Deny'
ipRules: [
{
value: 'xxx.xx.xxx.xx'
action: 'Allow'
}
]
}
}
}
Outcome: Enabled from selected virtual networks and IP addresses - this should be ticked
To reproduce
use the storage account bicep and try to create a new entry for an IP address to give it access so that :-
Outcome: Enabled from selected virtual networks and IP addresses - this should be ticked
Code snippet
see above

Relevant log output
No response

Tested using the current version of the module with the following params:
targetScope = 'resourceGroup'

module storageAccount '../ResourceModules/modules/Microsoft.Storage/storageAccounts/deploy.bicep' = {
  name: 'storageDeploy'
  params: {
    name: 'cs2469issuetest'
    enableDefaultTelemetry: true
    allowBlobPublicAccess: false
    location: 'uksouth'
    storageAccountSku: 'Standard_LRS'
    storageAccountKind: 'StorageV2'
    networkAcls: {
      bypass: 'AzureServices'
      defaultAction: 'Deny'
      ipRules: [
        {
          action: 'Allow'
          value: '<IP_ADDRESS>'
        }
        {
          action: 'Allow'
          value: '<IP_ADDRESS>'
        }
      ]
    }
  }
}

Deployment was successful as shown below

Networking Details shown below

Are you able to provide further details to the error you have received?

@gsuttie - Are you able to provide the team further details for this error you have received?

@gsuttie - If there any other follow up required here? if not we will close this issue out over the coming week

Closing issue as tests incorporate IpAllowList


Controlling Access to the Remote Cluster's Kubernetes API

Is your feature request related to a problem? Please describe.
Users access the API using kubectl, client libraries, or by making REST requests. Both human users and Kubernetes service accounts can be authorized for API access. How do I restrict/manage that once the K8s cluster is a registered Arc cluster.
A clear and concise description of what you want to happen.
Documentation to illustrate how this can be enabled and a demo scenario that showcases this happening.
Describe alternatives you've considered
Can this be done via an AAD integrated cluster? If so then that can be an alternative if the K8s identities are managed by AAD and the enforcing agent against K8s API is Azure RBAC policy on AAD identities.
Implementation Requirements
Template/script that sets up remote Arc for K8s cluster to adhere to this requirement.
Sample App to illustrate RBAC enforcing functionality.
Template/script to deploy sample app and its configuration.
Additional context
Add any other context or screenshots about the feature request here.

Archiving for now


Problem

Hi, am having issue with tsparticles script i my project.

Can u help me?

particlesInit is not part of window type, you need to declare a new property there like this: https://stackoverflow.com/questions/12709074/how-do-you-explicitly-set-a-new-property-on-window-in-typescript
about the other error, what tsparticles packages are you using and which version? astro-particles doesn't work with tsparticles v3 yet.

I solved the first problem, So how can i use tsParticles in Astro? With React?

I solved the first problem, So how can i use tsParticles in Astro? With React?

Use version 2, if you don’t need v3 features it’s fine.

I uninstall all modules and installed astro-particles. Now i don´t know how to install tsparticles v2.

npm install tsparticles@2

I tried everything. Now i have astro-particles and tsparticles@2. Another problem showed up.

I solved it. Solution was tsParticles.

I solved. Ty for your help. I am do happy after four days of trying. 😆


Add conda repo into readmes

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-messaging-webpubsubservice


iPhone 12 | A14 / iOS 14.1 (Unc0ver JB) | UTM broken

Hi
When you wanna create a new VM, the + and the edit button won‘t do anything. It‘s a fresh install:
Configuration

UTM Version: 2.0.22
OS Version: iOS 14.1
Device Model: iPhone 12
Is it jailbroken?: Yes, Unc0ver 6.0.0
How did you install UTM?: by Cydia Repository

Crash log:
Not available...
Debug log:
Not available...
Upload VM:
Not necessarily
Others:
https://user-images.githubusercontent.com/27576311/109423025-41e29100-79de-11eb-99b9-b610f74bc989.MP4
@

Is this the .deb?

Try installing the ipa via filza

Same effect ;(

@0n1cOn3 Can you uninstall the UTM DEB and then install https://cydia.akemi.ai/?page/net.angelxwind.appsyncunified With AppSync Unified installed, AirDrop the UTM IPA and it should be installed (no altstore needed). Then see if that works?

IPhone 11 iOS 14.2 (unc0ver jailbroken) the app no work

I have had this problem on both the IPA and the DEB; something about the corners of the screen is weird; I had to spam on the + button to make it work, or I had to put it in landscape mode and click the big + button.

I tried UTM on a fresh iOS 14.3 system and don’t see this. I think some tweak is messing up the UI. Try disabling all tweaks and try again.


Speed up DBSCAN post-processing

I'm still not sure if this is the best way to do, but it speeds up postproccessing on V100 for halo finding from 0.050 to 0.027. 3% overall time improvement for the win :) The improvements are better on AMD.
A side note: --verify option does not test this. The only way right now to validate this patch is to run with --output-sizes-and-centers and compare the output to master.

@dalg24 Added more comments.

PGI failed in the distributed tree, but it has nothing to do with this PR.

retest this please


Demo links on readme are not working

Giving Cannot proxy the given URL

Well I accidentally deleted the source somewhere, so for now I... decided to remove it. Sorry about that :D


Update deployments.md regarding force-updating.

Clarify what constitutes an inconsistent state.
Describe the circumstances under which such a state will be reached.

/cc @drexin
References #1346.

:100: Thanks!


Avoid false FileDependnecies among Implicitly-build Swift and Clang modules

InterfaceSubContextDelegateImpl causes sub-instances to inherit -fmodule-map-file= options.
Those Module Maps become file dependencies of all downstream PCMs and their depending Swift modules, even though they really aren't.
This causes frequent re-builds of the Module Cache contents when seemingly-unrelated files are touched.
Explicit Module Builds rely on these options for building Swift Interface files, so for now we just disable inheritance of these options in Implicit Module builds.
Part of: rdar://73723322

@swift-ci please test

Why do we exclude all extra arguments rather than just those -fmodule-map-file= ones?

Should things like linker flags, warning flags, etc. matter to the sub-module interface builder? If I were to, for example, turn on -Xcc -Werror in my client module, should that blow up when there are warnings in any clang module downstream of the interface builder's implicit build logic? My gut says no - that those modules should build atomically. But I'm not sure I have a full picture of what the intended behavior is.

Good catch! Why do we exclude all extra arguments rather than just those -fmodule-map-file= ones?

That's a good question. I thought about doing so, but it seems to me that we do not need any of these arguments for correctness so it seemed simpler to exclude them all. One benefit we may get from more-strict filtering is that it increases the likelihood of matching Clang invocation commands across various subInvocations we generate downstream, which may increase the likelihood that we get sharing of PCMs built for ClangInvocations with the same command line.

@swift-ci please test

@artemcm That's fair. It just seems to me excluding extra arguments doesn't logically connected with the underlying issue of unnecessary -fmodule-map. Though I agree we should simplify the clang commands as much as we can.

Build failed
Swift Test Linux Platform
Git Sha - 6f3fda136983ec396281240521b71580adbe487f

@shahmishal, FYI, https://ci.swift.org/job/swift-PR-Linux/25500/console failed with:
java.io.EOFException

In case we keep track or if there's something actionable.

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - 6f3fda136983ec396281240521b71580adbe487f

@swift-ci please test Linux platform


[Design][Perf] Hoist simple Expression<Func<T, U>> into fields in Razor

This change uses Roslyn to transform the syntax tree of a Razor page and hoist all simple expressions into static readonly fields.
For code like:
@Html.EditorFor(m => m.Occupation)

The c# compiler inserts a bunch of code calling into System.Linq.Expressions to construct and verify an Expression<Func<T, U>> matching the lambda being passed here. This runs every time the page is rendered, and allocates a lot of collections to verify that the types are valid.
The transformation done here looks like this:
public class __Generated_RazorPage
{
    private static readonly Expression<Func<Person, string>> __Hoist_1; = m => m.Occuptation;

    public Task RenderAsync()
    {
        ...
        Write(Html.EditorFor(__Hoist_1));
        ...
    }
}

When we encounter a case that we can't hoist, we simply ignore it. The 90% case in Razor is a property accessor or identity func. The only common case we can't handle is indexers, as usually the index is captured.

Perf numbers - allocations for 3001 requests on a basic site with model binding 3 fields and rendering a view. The view uses 3 LabelFors and 3 EditorFors.
Before:
Filtered to types in System.Linq.Expressions

Total Allocations:

After:
No allocations in System.Linq.Expressions
Total Allocations:

Overall, there's a reduction of 8.2mb of allocations from this change on a page with 6 uses of HtmlHelpers - that's about 2.7kb/request.
It's not easy to show where the savings come from in a simple graph as dotMemory doesn't show inclusive bytes on allocations - the savings is coming from a variety of collection types used in expression compilation and type checking.

Walked through in person all my concerns. Looks solid.

Closing this for now. We'll consider this change for RC.


[SPARK-28617][SQL][TEST] Fix misplacement when comment is at the end of the query

What changes were proposed in this pull request?
This PR fixes the issue of misplacement when the comment at the end of the query. Example:
Comment for  SELECT date '5874898-01-01':
https://github.com/apache/spark/blob/2d74f14d74e7b24109f347822600ebf9819b04c1/sql/core/src/test/resources/sql-tests/inputs/pgSQL/date.sql#L200
But the golden file is:
https://github.com/apache/spark/blob/a5a5da78cfbb1b439c139fd2ffd5744fc60ebafe/sql/core/src/test/resources/sql-tests/results/pgSQL/date.sql.out#L484-L507
After this PR:
https://github.com/apache/spark/blob/eeb7405ad0c7cc1004e2cad36929d20d95ab2726/sql/core/src/test/resources/sql-tests/results/pgSQL/date.sql.out#L482-L501
How was this patch tested?
N/A

Test build #108639 has started for PR 25357 at commit 523c426.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13726/
Test PASSed.

This PR seems to have a bigger positive side than a negative side. I'm supporting this PR, @wangyum .
cc @gatorsmile , @maropu

+1 too

Test build #108639 has finished for PR 25357 at commit 523c426.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108639/
Test PASSed.

+1, too

Test build #108695 has started for PR 25357 at commit eeb7405.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13782/
Test PASSed.

Test build #108695 has finished for PR 25357 at commit eeb7405.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108695/
Test FAILed.

Test build #108710 has started for PR 25357 at commit a9abe00.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13795/
Test PASSed.

Test build #108710 has finished for PR 25357 at commit a9abe00.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108710/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13835/
Test PASSed.

Test build #108754 has started for PR 25357 at commit 60e38af.

Test build #108754 has finished for PR 25357 at commit 60e38af.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108754/
Test PASSed.


ability to advance time in a test

Problem to be solved
Upgrade16 had some auction timing bugs with timing that could have been detected in an A3P test if the test could time travel forward.
Unit tests in agoric-sdk have this ability through the ManualTimer service replacing the real one in bootstrap, but A3P tests are too high fidelity for that.
We need some other way to advance time.
Design
The test tooling could provide a way to advance the Linux system time.
Some other designs may need changes in agoric-sdk. This ticket tracks having the ability in the @agoric/synthetic-chain support package.

The main problem is how to advance time.
The system clock is global for a whole physical machine: VMs usually share it with the physical machine, and containers definitely share it with the host kernel.
On the opposite side we have time observed by vats, which is controlled by vat-timer. There are currently no specific plans to upgrade that vat. Furthermore having a time observed by vats being different from the chain time might cause confusion for tools sitting outside the chain and observing values written by vats that are disconnected from the chain time.
In production, the time observed by vats is derived from the chain time, so the least disruptive would be to tweak the chain time instead. I am not sure whether cosmos-sdk has any affordances to manually derive the chain time.

There seem to be some tricks with LD_PRELOAD=/lib/faketime.so.

Change Date Inside a Docker Container | Baeldung on Ops
Set System Time Dynamically in a Docker Container | Baeldung on Linux

@anilhelvaci says he has experience providing an alternative chain timer service using a contract.


Isolate client dependencies in upstream

Working on Kubeless integration, I ran into dependency conflicts. One way out I found was to isolate each upstream with dependencies into its own go module, and manage the dependencies there.
This PR accomplishes that for the existing code base.

Codecov Report

Merging #216 (f2ec945) into master (5c2f1be) will decrease coverage by 0.08%.
The diff coverage is 41.28%.

@@            Coverage Diff             @@
##           master     #216      +/-   ##
==========================================
- Coverage   25.91%   25.82%   -0.09%
==========================================
  Files          25       25
  Lines        1536     1568      +32
==========================================
+ Hits          398      405       +7
- Misses       1107     1125      +18
- Partials       31       38       +7

Impacted Files
Coverage Δ

server/proxy/install.go
0.00% <0.00%> (ø)

server/proxy/service.go
0.00% <0.00%> (ø)

server/proxy/proxy.go
21.79% <33.33%> (-7.59%)
:arrow_down:

server/store/manifest.go
7.07% <55.55%> (+7.07%)
:arrow_up:

server/store/apps.go
8.00% <75.00%> (+8.00%)
:arrow_up:

server/store/service.go
85.48% <75.00%> (-6.46%)
: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 5c2f1be...f2ec945. Read the comment docs.

@DHaussermann this is now merged with master and ready to be tested. I still need to change the autolink example to return PluginID in its Manifest (now a requirement), but the rest (HTTP, AWS) is testable.
@hanzei @catalintomai @mickmister - somewhat of a straight merge, but for plugin Apps I moved PluginID into the Manifest and the plugin is required to set it in its manifest.json; 3/5 it's cleaner this way.

@levb When trying to update the apps plugin dependency in the autolink plugin (to use the new Manifest struct) I get this error:
> go get github.com/mattermost/mattermost-plugin-apps@f21ce39b77678d71f912e966c9fb103a46e53bcd

go get<EMAIL_ADDRESS>updating to
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: unknown revision<PHONE_NUMBER>00

You can try it out using the autolink PR https://github.com/mattermost/mattermost-plugin-autolink/pull/164

Going to close this PR in favor of https://github.com/mattermost/mattermost-plugin-apps/pull/222


Meta tag in top.jsp missing proper closing tag

Confirmed missing in 4.1.x and 4.2.x as of 3/31/16. Found on 4.1.4
Tomcat 8
Vendor provided that error messages were thrown from CAS client for inproper meta tag closing.

Meta tag current:
Meta tag correct:
Exception Details:

Error Description: The 'meta' start tag on line 15 position 4 does not match the end tag of 'head'. Line 26, position 3.
Error Type: System.Xml.XmlException
Source: System.Xml
Stack Trace:
at System.Xml.XmlTextReaderImpl.Throw(String res, String[] args)
at System.Xml.XmlTextReaderImpl.ThrowTagMismatch(NodeData startTag)
at System.Xml.XmlTextReaderImpl.ParseEndElement()

Please apply to 4.1.x and 4.2.x for future compatibility.

Happy to. Do you want to submit a pull request?

Been a long time since I have worked with Git. Think I created a pull request with the change.  Not sure if you or I merge it into the next builds of 4.1.7 and 4.2.x.
Thanks for the quick response!
-Jeff

I will merge. You want to:

Fork the repo here and clone your fork
Create a branch off of 4.1.x or 4.2.x
Apply the change
Commit and push the change to a remote branch of your fork
Submit a PR, targeting 4.1.x OR 4.2.x

See http://jasig.github.io/cas/developer/Contributor-Guidelines.html

Thanks for the patch! I merged the change back to all 4.1 and 4.2 releases. Releases should be out in a few weeks. Great work!


Feature Request: Add support for Managed Media Source API

Is there an existing issue for this?

[X] I have searched the existing issues

Which Mux Elements/Packages does this apply to? Select all that apply
mux-video, mux-video-react, playback-core
Description
Hi!
Last year, Apple introduced the Managed Media Source API, which is supposed to have the capabilities of MSE, without any of the drawbacks, and also works on iOS!
Could this be implemented into elements, and added to prefer-playback in mux-video?
Related issues/resources:
https://github.com/Dash-Industry-Forum/dash.js/issues/4202
https://webkit.org/blog/14735/webkit-features-in-safari-17-1/
https://developer.apple.com/videos/play/wwdc2023/10122/
Expected Behavior
MMS support would be implemented throughout the package, and prefer-playback='mms' would be an option in mux-video.

Hey @jakubslambor this is definitely on our radar! Since we rely on hls.js "under the hood" for our "playback engine," which already supports ManagedMediaSource in the latest version (https://github.com/video-dev/hls.js/blob/master/docs/API.md#prefermanagedmediasource), we're hoping this will be fairly straightforward to support once we get the bandwidth to do the relevant upgrades and tweaks. From a "devex"/"developer ergonomics" perspective, do you foresee any issues with treating this as just a variation on "mse" for preferred playback? aka, would you foresee any issues if prefer-playback="mse" would simply use ManagedMediaSource in environments like iOS, or do you think that level of granularity/specificity will be fine?

That's great to hear!

From a "devex"/"developer ergonomics" perspective, do you foresee any issues with treating this as just a variation on "mse" for preferred playback?

I can't speak for everyone, but for us no, if prefer-playback="mse" would simply use ManagedMediaSource where available that would definitely work!

Just to keep you in the loop: Step 1 of getting support for this just occurred and should be released soon (upgrading to the latest version of hls.js). I'll keep you posted on our progress for step 2 (updating our codebase to take advantage of this).


Method on the exchange Object to get the url decoded QueryString

Is your feature request related to a problem? Please describe.
When using the raw query encodings like %25 for % is not decoded and needs to be manually decoded back. It would be convenient to get the whole query decoded as I use the query parameter names in a variable name and don't want to figure their name out first before getting the the decoded value.
Describe the solution you'd like
A method on the exchange object called getQueryString() which does the above.
Describe alternatives you've considered
None
Additional context
None

Won't be supported as it will return an AbstractObject instead.
If the raw decoded query String is needed (which sometimes can be easier to handle) the following is would be the solution:
URLDecoder.decode(exchange.rawRequest().getQueryString(), StandardCharsets.UTF_8)


Uncaught ReferenceError: Cannot access before initialization

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, srcDir
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-hqc4nd
Describe the bug
I'm using composition API in using nuxt 3, then as what this section describe.

Each *.vue file can contain at most one

Hi @SedanaYoga!
You can move everything from

<template>
  <button @click="clickGreeting">Click Here</button>
</template>

Is there reason why you don't use props for this?
detail.vue could look like this

<template>

{{ nameRef }}

  <SubDetail :name="nameRef" />
</template>

Cheers!

Hi @SedanaYoga!
You can move everything from

<template>
  <button @click="clickGreeting">Click Here</button>
</template>

Is there a reason why you don't use props for this?
detail.vue could look like this

<template>

{{ nameRef }}

  <SubDetail :name="nameRef" />
</template>

Cheers!

Hey, @darioferderber
Thanks for the reply, but thats not what I want. I intended to have exported variable on detail.vue cause I simplify the use case a lot here. I'm actually trying to store gql.tada's fragment (which is a variable) inside vue component. And that gql.tada fragment needs to be used/imported by another component. Once, I tried this method (

No problem 🙂
You can still export variable in detail.vue, but when you're importing it in sub-detail.vue you can just move your import from


Cannot install on Ubuntu 16.04

This looks really interesting, but fails to install on my system.
Output:
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1341  100  1341    0     0   5728      0 --:--:-- --:--:-- --:--:--  5730
Download Dnote binary from curl https://github.com/dnote-io/cli/releases/download/v0.0.2/dnote-linux-amd64 to /usr/local/bin/dnote
Installation failed

Tell me how I can provide you with more info.

I think this happens because the curl command is trying to write to /usr/local/bin/dnote which in Ubuntu is restricted for the non root users.
An easy fix could be to run the whole script with sudo however I don't think this is a great idea. Probably trying to create a file in that dir and if it fails, ask for sudo could be a better alternative?

Actually, even when running with sudo rights did not work either.

I was able to reproduce it on Ubuntu 16.04. Will look into it.

The following worked for me (note sudo after the pipe rather than in front of curl):
curl https://raw.githubusercontent.com/dnote-io/cli/master/install.sh | sudo sh

It turns out that we need to either add sudo or run chown /usr/local in this case.
Some ideas:

specify the above solution in the README
there might be a way to download the binary, and copy it to /usr/local using sudo inside install.sh.
If download fails, suggest using sudo

How do you feel about these solutions?

 :)
As for the solution, the best one would be to ask the user for sudo password if needed in the process of running the script. I don't think asking users to put sudo in the middle of the command line is very user-friendly. At least maybe the script can be restructured in a way, that it starts with sh, so that sudo could go right before it in the beginning of the command line.

In that case, let's suppress the output for curl with -s flag. Updated the README.
curl -s https://raw.githubusercontent.com/dnote-io/cli/master/install.sh | sudo sh

Yes, I agree that asking users to execute some shell script using sudo is not user-friendly.
Maybe install Dnote system-wide is unnecessary? Feedback welcomed. I will close this in the meantime. Please reopen if issue persists.


Slight changes to ubuntu_example.sh

See title.

Coverage remained the same at 88.486% when pulling 630a27d19a6594dd1e10c1d67b4ad75298d275f6 on RickCarlino:shell_script_updates into 25d20208cce1b54cf083dee0fedb528051bd4f3a on FarmBot:staging.


Dribbble Updates Break Booom

Dribbble recently launched some updates that change the way thumbnails flow on larger screens. Shots also now open in a modal. More information available on this tweet from today.
So far, the main breaking point is that thumbnails only show 2 x 2 with Booom enabled.

Fixed and published!


Random sprite state / color needs to be more robust

I have this yml:
- type: Sprite
    drawdepth: Mobs
    sprite: Mobs/Aliens/Guardians/guardians.rsi
    layers:
    - state: tech_basic
    - state: tech_flare
      color: "#40a7d7"
      shader: unshaded
    noRot: true

Ideally:

I could set the first layer (tech_basic) to anything out of a list of options
The 2nd layer (tech_flare) could be randomly coloured and have its state also updated to correspond with the above
AsteroidRock is finally renamed

related issue: #6227


add a .NET example for PubSub routing

What content needs to be created or modified?
When the PubSub routing docs were originally added, there was a final comment in the PR#1786 to add a .NET example but this was not done before the PR got merged. Thus, this issue is to create that and add it to this documentation
Describe the solution you'd like
.NET example for pubsub routing added to docs
Where should the new material be placed?
daprdocs/content/en/developing-applications/building-blocks/pubsub/howto-route-messages.md
The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos
N/A
Additional context
None

@greenie-msft - Is this address by the quickstarts now?

This scenario is not included in the quickstarts. Should this remain a how-to only?

Closing as a C# example has been added to the how-to


feat: Add Netmotion reporting

:tada: This PR is included in version 1.53.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


 This results in an ActionController::BadRequest exception being raised. Rails handles this exception and responds with a 400 Bad Request which seems correct. However, the exception is unnecessarily reported to Errbit.
I did consider only reporting these exceptions when they had a specific exception message, but it turns out that in this case there is no exception message, so it's difficult to be more specific.
However, I'm pretty convinced that ignoring all such exceptions is safe to do given that a bunch of other exceptions that Rails handles by default are ignored by default by Airbrake. Indeed there is a case to be made that we should bring the Airbrake list into line with the Rails list.
Closes #121.

@chrisroos: Thanks for reviewing.
Rebased against master and force-pushed in preparation for merging.


About Objects 365

Hello, thanks for your inspiring work! I am trying to reproduce the training but I have no idea how to prepare the Objects365 datasets based on the information in readme. Did you use all the images of objects365 (2million) or only the train split(1.7million)?

Hi. Did you solve the problem?
I cant figure out which object365 dataset I should downlowd..


feat: port Analysis.Normed.Group.AddTorsor

[x] depends on: #2866
[x] depends on: #2923

This PR/issue depends on:

leanprover-community/mathlib4#2866
leanprover-community/mathlib4#2923
By Dependent Issues (🤖). Happy coding!

Potentially another victim of https://github.com/leanprover/lean4/issues/2055?

Potentially another victim of leanprover/lean4#2055?

It's not! :smile_cat:

bors merge


Implement python bindings for the classes that handle the clock

This PR implements:

The python bindings for the Clock machinery and for the yarp clock

Furthermore, since the clock is implemented as a singleton, an error it will be returned if the user tries to change the clock type once the clock() has been already called once. Notice that BipedalLocomotion::clock() instantiate the singleton the first time is called (See Meyers' singleton implementation).
For instance, in python we will have
In [1]: import bipedal_locomotion_framework.bindings as blf

In [2]: blf.clock().now()
Out[2]: datetime.timedelta(days=19010, seconds=63353, microseconds=503687)

In [3]: blf.system.ClockBuilder.set_factory(blf.system.YarpClockFactory())
[2022-01-18 18:35:55.945] [thread: 66773] [blf] [error] [ClockBuilder::setFactory] The clock has been already called. Since the clock() returns a singleton it is not possible to set the factory anymore. Please call 'setFactory()' at the beginning of your application to avoid this problem.
Out[3]: False

In [1]: import bipedal_locomotion_framework.bindings as blf

In [2]: blf.system.ClockBuilder.set_factory(blf.system.YarpClockFactory())
Out[2]: True

In [3]: blf.system.ClockBuilder.set_factory(blf.system.YarpClockFactory())
Out[3]: True

In [4]: blf.clock().now()
Out[4]: datetime.timedelta(days=19010, seconds=63392, microseconds=416123)

Thanks to this PR it will be possible to easily synchronize yarp process also in python.
cc @Giulero

The PR is ready to be reviewed


WebExt bookmarks API docs tasks for Fx 68 (Irene)

See https://trello.com/c/n7bNG8wh/210-bookmarks-enhancements-fx-68 for original trello card.
Acceptance criteria

[x] provide better error message when attempt is made to add bookmark folder to root folder — documented appropriately.

[x] Information added to 68 release notes

Here are the updates I made. I'm not entirely sure it needs to be formatted as a warning. Let me know if you would like to change that or make it a note instead. Also, please review my changes and let me know if I missed anything!

Added the following warning to Bookmarks API:
Extensions cannot create, modify, or delete bookmarks in the root node of the bookmarks tree. Doing so causes an error with the message: "The bookmark root cannot be modified"

Added the following warning to the bookmarks.create method:
If your extension tries to create a new bookmark in the bookmark tree's root node, it raises an error: "The bookmark root cannot be modified" and the bookmark won't be created.

Added the following warning to bookmarks.move method:
If your extension attempts to move a bookmark into the bookmarks tree root node, the call will raise an error with the message: "The bookmark root cannot be modified" and the move won't be completed.

Added the following to warning bookmarks.remove:
If your extension attempts to remove a bookmark from the bookmarks tree root node, the call will raise an error with the message: "The bookmark root cannot be modified" and the bookmark won't be removed.

Added the following message to bookmarks.removeTree:
If your extension attempts to remove a bookmark tree from the bookmarks tree root node, the call will raise an error with the message: "The bookmark root cannot be modified" and the bookmark won't be removed.

Added the following message to bookmarks.update:
If your extension attempts to update a bookmark in the bookmarks tree root node, the call will raise an error with the message: "The bookmark root cannot be modified" and the bookmark won't be updated.

LGTM from Rob Wu on Bugzilla 1512171


This was a mistake -- multiple threads can't wait on the same latch,
because the inner CoreLatch tracks thread-specific sleep state.

bors r+


Proxify

Send to burp final results with proxify

Done https://github.com/six2dez/reconftw/commit/fee2ed6912a943e82f5b7708d3a856d5a69d5f35


f77blas.h: unknown type name bfloat16

When compiling OpenBLAS 0.3.10 and include the f77blas.h header in my program, I receive the following error.
/usr/include/f77blas.h:476:5: error: unknown type name 'bfloat16'

This seems related to the change
https://github.com/xianyi/OpenBLAS/commit/7eb55504b1727eebcb0f451fa5b148dbea303b69#diff-7a719af21b4ed2f340e10df2dc8da843R40-R476
followed by this change
https://github.com/xianyi/OpenBLAS/commit/67cc4b9e16d2e8c017731d2b9eabb5c6b45a9ad5#diff-7a719af21b4ed2f340e10df2dc8da843L40-L42
f77blas.h is generated from common_interface.h and the symbol bfloat16 is not defined.
Thank you for your time and help.

Can you try if adding
#ifndef BFLOAT16
typedef unsigned short bfloat16;
#define HALFCONVERSION 1
#endif

to openblas_config.h (which gets included by f77blas.h et al.) solves the problem ?

Yes, thanks, this does solve my problem.  I don't know if #define HALFCONVERSION 1 was necessary.  For my case it is not.

Probably not, I just copypasted it from common.h (internal to OpenBLAS), but the other part needs to be added to openblas_config_template.h - I'll do a PR for this later today.

Thanks a lot :+1:


some chinese character can not be searched

to produce, copy some texts in chinese character to notes, like this one: https://m.bjnews.com.cn/detail/154105774414080.html
search for 感冒 returns no result. but 订阅 is ok。
version: docker:latest (0c7b3e7aaec0)
search for 感冒

search for 订阅

I think the problem here is that the search index is currently very targeted to the English language. When text is added to the index is goes through a number of steps:
Tokenisation - This breaks up the words into "tokens" (words to be indexed). Currently flatnotes uses a regex tokeniser (\w+(\.?\w+)*) which is fairly generic but may/may not work well with Chinese text.
Lower Case Filtering - I can imagine this would be ok with any language.
Accent Folding - This ensures that "café" is indexed as "cafe". Again, should work ok with any language.
Stop Word Removal - This currently filters out common English words that are not useful for searches e.g. 'for', 'from' and 'have'. This should work ok with non-English languages but they wouldn't get the benefit this step is designed for.
Stemming - This process tries to "normalise" related words. For exmaple, a note containing one of “render”, “rendered”, “renders” or “rendering” could be found by searching for any of those words. flatnotes currently uses the "Porter" stemming algorithm which is designed to remove suffixes from English words.
I'd love to be able to target other languages but I'd likely need to make some changes to the index and possible add options to it.

I, too, would like other languages to be added. Searching in Korean does not bring up any results.

Same issue

Same issue


Remove revoke current device from account deletion screen

:eyeglasses: @keybase/react-hackers

The commits d673606df51e...8a389629df40 introduce visual changes on linux. :mag_right: 2 changed

Changed: Delete_Me-Normal (before) (after)

Changed: Chat_Input-Emoji_Open (before) (after)


HARP-10163: sunny coffee example

The use case of this example is to show shadows for a location at a certain time of the day, so that a user can search around for a sunny place.

@harpgl-bot ok to test

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1515   +/-   ##
=======================================
  Coverage   60.07%   60.07%
=======================================
  Files         275      275
  Lines       24147    24147
  Branches     5422     5422
=======================================
  Hits        14506    14506
  Misses       9641     9641

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 149109e...9725225. Read the comment docs.

Codecov Report

Merging #1515 into master will decrease coverage by 0.53%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1515      +/-   ##
==========================================
- Coverage   60.61%   60.08%   -0.54%
==========================================
  Files         278      275       -3
  Lines       24537    24153     -384
  Branches     5524     5424     -100
==========================================
- Hits        14874    14512     -362
+ Misses       9663     9641      -22

Impacted Files
Coverage Δ

@here/harp-mapview/lib/MapView.ts
63.71% <0.00%> (-2.13%)
:arrow_down:

@here/harp-mapview/lib/CameraMovementDetector.ts
61.97% <0.00%> (-28.17%)
:arrow_down:

@here/harp-mapview/lib/text/LayoutState.ts
61.11% <0.00%> (-16.67%)
:arrow_down:

...re/harp-datasource-protocol/lib/TechniqueParams.ts
59.52% <0.00%> (-12.15%)
:arrow_down:

@here/harp-text-canvas/lib/rendering/TextStyle.ts
84.61% <0.00%> (-6.09%)
:arrow_down:

@here/harp-datasource-protocol/lib/DecodedTile.ts
37.86% <0.00%> (-5.83%)
:arrow_down:

...datasource-protocol/lib/operators/MathOperators.ts
19.83% <0.00%> (-3.31%)
:arrow_down:

...e/harp-mapview/lib/geometry/TileGeometryCreator.ts
55.32% <0.00%> (-3.27%)
:arrow_down:

...tasource-protocol/lib/operators/ObjectOperators.ts
91.42% <0.00%> (-2.86%)
:arrow_down:

...e/harp-datasource-protocol/lib/ExprInstantiator.ts
73.68% <0.00%> (-2.79%)
:arrow_down:

... and 67 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8b3dfe...77b0398. Read the comment docs.


Update buildkite template and pipeline configurations

This example repository is partnered with Elastic CI Stack tutorial.
This PR:

Remove pipeline.sh
Fixed the label of command step and
Add missing description in template
agent queue defaults to default

@yob yep, that was it from memory!
This is so great @JuanitoFatas ❤ So good to see this getting some love.


Error: could not automatically setup MSVC

Describe your problem here.
I have trouble compiling engine. I don't know what version of  VS to use.

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
I only edited some values in Discord.hx to match my mod (i did not rewrite any of the code)

Simplest way to install the correct VS stuff is by entering these commands (in order) on command prompt.
This downloads Visual Studio Installer
curl -# -O https://download.visualstudio.microsoft.com/download/pr/3105fcfe-e771-41d6-9a1c-fc971e7d03a7/8eb13958dc429a6e6f7e0d6704d43a55f18d02a253608351b6bf6723ffdaf24e/vs_Community.exe

Starts the installer and selects the packages you need to compile.
vs_Community.exe --add Microsoft.VisualStudio.Component.VC.Tools.x86.x64 --add Microsoft.VisualStudio.Component.Windows10SDK.19041 -p --norestart

Deletes the setup file after you've installed it
del vs_Community.exe


Possibilty to hide a template

It would be a nice feature to hide my daily template (which includes everyday todo's) from the checklist.

Hmm, i didn't even think about that, good find. i'll come up with something for this

Hmm, i didn't even think about that, good find. i'll come up with something for this


libfs: MountInterrupter should be done after trying to unmount fails

Currently if unmounting fails the MountInterrupter is not marked as done.
This means that shutdown can get blocked when unmounting fails
again and again.

@songgao could be the right person for review.

If I remember correctly, that was intentional, so that one can keep hitting Ctrl-C to call unmount multiple times.

Perhaps we could do something like making it close if we get two signals within a second, but not if they are more than one second apart?

Or having a counter there. Because now if unmount fails each time then it will keep spinning forever.

Hmm wdym keep spinning forever?
Re counter, the one second thing I mentioned above was basically a counter, but with a timeout. It seems just having a counter without ever reset would generate some surprises as over a long time it just seems inconsistent.


Repurpose ipfs-block as a container for { data, cid }

currently it is a wrapper around the data that adds a utility method for generating the multihash.
block.key(hashFn, cb) is not much better than hash(hashFn, block.data, cb).
Instead I propose we repurpose this as a container around { data, cid } since that is not captured in a standard interface yet but use it in a number of places.

Happening here: https://github.com/ipfs/js-ipfs-block/pull/31


Update index.js

added optional click navigation which will only show up if set:
[params.gallerydeluxe]
enable_click_navigation = true

you may view the change with that configuration value set here - https://benstraw.com/categories/photography

Ooops, I just realized I reversed the swipe direction calls on my link buttons. It just needs to be:
leftNav.addEventListener('click', function () {
				swipe('right');
			});

and
rightNav.addEventListener('click', function () {
				swipe('left');
			});

I'll have a look at this later, but a little note: If this was my site I would add compress/reduce the size of the full size images. 5MB JPEGs seems a little on the big side for the web.


abnormal gastric inhibitory polypeptide

Requested term: abnormal gastric inhibitory polypeptide
Proposed parent: abnormal circulating hormone level
Draft definition: any anomaly in the blood concentration of
this endocrine hormone, secreted by the enteroendocrine K-cells in the mucosa
of the duodenum and jejunum in response to feeding, that stimulates insulin
release to regulate blood glucose and nutrient homeostasis.
Parallel VT term:
VT:0010916 | blood   gastric inhibitory polypeptide amount

Reopened issue to add 'glucose-dependent insulinotropic polypeptide' as an exact synonym to 'gastric inhibitory polypeptide' to all relevant terms.
abnormal circulating gastric inhibitory polypeptide level MP:0014581
decreased circulating gastric inhibitory polypeptide level MP:0014583
increased circulating gastric inhibitory polypeptide level MP:0014582
SYNONYM SOURCE: PR:000007986, https://en.wikipedia.org/wiki/Gastric_inhibitory_polypeptide


Resolve typos for Bucket Migration StackStorm Workflow

What this PR does / why we need it:
This PR is for correcting the typo for bucket migration workflow and remove the unused code from the python runner.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   71.86%   71.86%
=======================================
  Files           6        6
  Lines         231      231
  Branches       18       18
=======================================
  Hits          166      166
  Misses         57       57
  Partials        8        8


Add a tutorial.

Feel free to close this if you're not interested, or if it's wildly wrong.  I was just thinking it might be helpful for people to learn how to use this very useful tool!


Build Failure

Expected results:       i want App Running
Actual results:         i See Build Failure

Logs

[√] Flutter (Channel stable, 1.20.4, on Microsoft Windows [Version 10.0.18362.1082], locale en-US)
    • Flutter version 1.20.4 at C:\Users\adil rehman\Documents\flutterSdk\flutter
    • Framework revision fba99f6cf9 (12 days ago), 2020-09-14 15:32:52 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\adil rehman\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Android Studio (version 4.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.42.1)
    • VS Code at C:\Users\adil rehman\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.8.1

[√] Connected device (1 available)
    • CPH1809 (mobile) • 25cfa711 • android-arm64 • Android 8.1.0 (API 27)

• No issues found!
Process finished with exit code 0

* What went wrong:
Execution failed for task ':app:mergeDebugResources'.
> Multiple task action failures occurred:
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #1: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #3: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #0: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #2: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #6: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #7: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #5: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.
   > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
      > AAPT2 aapt2-3.5.0-5435860-windows Daemon #4: Daemon startup failed
        This should not happen under normal circumstances, please file an issue if it does.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 18m 41s
Exception: Gradle task assembleDebug failed with exit code 1

@adilwahla can you please include output of flutter doctor -v

Hi @adilwahla
Make sure you're using classpath 'com.android.tools.build:gradle:3.5.0' in your build.grdle, try again
If the problem persists, Can you please provide your flutter run --verbose and a complete reproducible minimal code sample
Thank you

thank you issue resolved


[release/3.0] Update dependencies from dotnet/source-build-reference-packages

This pull request updates the following dependencies
From https://github.com/dotnet/source-build-reference-packages

Build: 20190717.1
Date Produced: 7/17/2019 2:21 PM
Commit: 3b5b0e599782e3ccdeed47e67dd2d384e09fba9d
Branch: refs/heads/master
Updates:

Private.SourceBuild.ReferencePackages -> 1.0.0-beta.19367.1

CI completed successfully.  Reporting in GitHub is incorrect.


Feature Request: Allow naming of source windows

Currently the window source in actions are simply window x (where x is the BrowserWindow id (integer)).
What do you think of the idea of allowing redux-electron-store to accept a custom source option? It would make it easier to identify which window the action originated from.
Generally, :+1: for this lib by the way it works very well!

Great Idea!  I'll try to get to it over the next few days.  Any ideas for the name of the option?

I guess windowName or windowTitle or something like that. Another possibility is that you could read document.title from the DOM.

Maybe have a fallback if the windowTitle (or whatever) option isn't specified. Pseudocode:
const windowSourceTitle = options.windowTitle || document.title || rendererId;

You would probably need an additional step to ensure that window source titles are unique (I guess you could just postfix the rendererId).

Added in https://github.com/samiskin/redux-electron-store/pull/14 under the option sourceName and published in v0.3.5 :+1:
I decided to just leave the default as what it currently is (window x or webview x), since providing sourceName: document.title isn't much for a user to add themselves if they want it.

Awesome. Thanks!


Split ERC20PositionWithdrawer V1 and V2

Pull Request Test Coverage Report for Build 8367

19 of 19 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 8349:
0.0%

Covered Lines:
1474

Relevant Lines:
1474

💛  - Coveralls


Join Operations

Implement merge and join with current broadcast algorithm.

I think I have two comments.

Target the microbenchmark designed by MODIN developers.
Be more careful about code refactoring and code reviewing this time.

I think I have two comments.

Target the microbenchmark designed by MODIN developers.
Be more careful about code refactoring and code reviewing this time.

Could we assign this issue to you, Luis? @pastranakike

Could we assign this issue to you, Luis? @pastranakike

Yes! I assigned. I agree with your comments. All this issues must not only focus on implementation but also refactoring of previous code.

Yes! I assigned. I agree with your comments. All this issues must not only focus on implementation but also refactoring of previous code.


Mac address is always null

Whenever i scan or try to find the mac address it always returns null

Can you provide your calling code so I can run some tests on it? The local device's mac address will be hidden as it is a security feature in android.

android 10 can't fetch subnet devices mac address?

I want to work on this bug !!!

Hey sir please see my update . I have resolved the bug

@stealthcopter please see it

@Kushagrasaxena-13 did you get this working? would love to finally be able to use this library in my latest build but mac is always null!

@TaylorStyles did you find solution i have same issue

This happens because Android hides local network MAC addresses for privacy reasons. There are some work arounds but nothing with the latest versions of Android :(


Warn when changing the keybind for 'Undo changes' to Ctrl-Z

I was fiddling around with the settings when I saw the default hotkey for undo is Ctrl-U, and I thought to myself "Oh, and why would any hotkey that undo stuff not be Ctrl-Z?" and properly decided to correct this oversight. Later when I was writing lines and lines of instruction in the assembler, I absentmindedly hit Ctrl-Z AND ALL THE CHANGES I DID WERE GONE IN AN INSTANT -
…
Please show a warning when the keybind is changed to Ctrl-Z. Or maybe clarify just what exactly does "most recent changes" in the phrase "undo the most recent changes in the current editor" mean. Or maybe add a redo button.

😂 yeah, thats exactly why that is bound that way


Update README.md of the powershell repo to let people know they need to install  v1.11.0-beta4 of C# extension to work with the latest .NET Core 2.0-preview2 packages

Update README.md of the powershell repo to let people know they need to install  v1.11.0-beta4 of C# extension to work with the latest .NET Core 2.0-preview2 packages

OmniSharp was fixed https://github.com/OmniSharp/omnisharp-vscode/issues/1495 - Beta4 contains the fix.

Yes, but they will not make a public release of v1.11.0 for a while, see https://github.com/OmniSharp/omnisharp-vscode/issues/1495#issuecomment-307863132
So we need to let people know they need to manually install v1.11.0-beta4 release of C# extension on OSX and Linux.


Need BNHA S2 OP's

Please add boku no hero academia, season - 2, openings.

[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate.
[ ] This issue is for enhancement/bug fixing.
[x] This issue is an addition to JSON database.
[x] I have gone through Contribution Guidelines.
[ ] Assign this issue to me.

@VaaibhaviSingh please add. :smile:


Exercise - Connect an ASP.NET application to Azure SQL Database

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
'NoneType' object has no attribute 'upper'
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 603, in execute
    raise ex
cli/core/commands/__init__.py, ln 661, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 652, in _run_job
    cmd_copy.exception_handler(ex)
...
cli/command_modules/appservice/custom.py, ln 2993, in webapp_up
    create_app_service_plan(cmd, rg_name, plan, _is_linux, False, sku, 1 if _is_linux else None, location)
cli/command_modules/appservice/custom.py, ln 1390, in create_app_service_plan
    sku = _normalize_sku(sku)
cli/command_modules/appservice/utils.py, ln 20, in _normalize_sku
    sku = sku.upper()
AttributeError: 'NoneType' object has no attribute 'upper'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp up --resource-group {} --sku {} --name {}

Expected Behavior
Environment Summary
Linux-4.15.0-1063-azure-x86_64-with-debian-stretch-sid
Python 3.6.5
Shell: bash

azure-cli 2.0.76

Additional Context

https://github.com/Azure/azure-cli/issues/11100

Please use latest version of CLI. 2.0.77 has the fix.


Add backface-visibility to .hover-grow

Fixes https://github.com/github/education-web/issues/1672
This PR adds one line of CSS to the .hover-grow utility class: backface-visibility: hidden;
That line addresses a visual bug where any images contained within the element would flicker or distort briefly during the animation.
/cc @primer/ds-core

@brandonrosage @sophshep Do you need us to ship this quickly? Just checking as we didn't have a release planned next week but happy to do a patch release if needed.

I'd categorize this as a "nice-to-have" since it's adding polish vs. fixing something that's broken. I think if you've got a release in the next couple of weeks that would be fine. Thanks for checking though @broccolini!


Circular dependency detected

I have Circular dependency checking in my project and when I include this library, I get this error:

This makes my builds fails.

Fixed in v3.17.1

Pulled it down and no longer seeing those errors, thanks for the fix

Seems issue was reintroduced in v3.17.6.
Installed and tested v3.17.2 - v3.17.5 and no warning. v3.17.6 and up warning is present. I would suspect that jsonStringifyReplacer in "./helpers/parseUtil.ts" could potentially also be moved to "./helpers/util.ts"(https://github.com/colinhacks/zod/compare/v3.17.5...v3.17.6#diff-b89cb81c2ff2ce8f5647cfb7d9fcd505bd7271abebfb8de24230a0ebb022f1daR2)
@colinhacks can we reopen please?


Structure update

Structure update

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Debug log not showing in console

Hi,
I'm using Firebase Analytics, version v.40005000 (installed today via cocoapods), running on an iPhone 6S under iOS 11.2.1.
In XCode (9.2) I have added both -FIRDebugEnabled and -FIRAnalyticsDebugEnabled as Run parameters to my Debug scheme.
I have the following code in my AppDelegate.m 👍
In imports:
@import Firebase;

At the end of didFinishLaunchingWithOptions, just before return YES;
[FIRApp configure];
[FIRAnalytics logEventWithName:kFIREventAppOpen parameters:nil];

When running the app on my device I see the following in the console:
2018-03-02 20:09:53.529142+0100 xxxxxxx[1796:898990] [Firebase/InstanceID][I-IID013010] InstanceID library version 2.0.8
2018-03-02 20:09:53.546928+0100 xxxxxxx[1796:898975] [Firebase/Analytics][I-ACS023007] Firebase Analytics v.40005000 started
2018-03-02 20:09:53.548088+0100 xxxxxxx[1796:898975] [Firebase/Analytics][I-ACS023008] To disable debug logging set the following application argument: -noFIRAnalyticsDebugEnabled (see http://goo.gl/RfcP7r)

... and that's it. No further logging is shown. There is no debug output for the App Open event - or for any other custom events I've logged with logEventWithName.
Nothing shows in Analytics or the stream.
In the GoogleService-Info.plist I have also set IS_ANALYTICS_ENABLED to YES.
What am I missing here?
Thank you in advance for any tips!

UPDATE: Deselecting the following resulting in my console log being flooded with a veritable plethora of Firebase/Analytics debug traces:

It looks like this was the issue.


Simplify so whole dir gets copied to /usr/src/app

before bundle install.
This is:

simpler

solves problem that some Gemfiles reference a gemspec (See Yehuda Katz post) and this wasn't
including the gemspec before running bundle install, resulting in:
Trigger 2, RUN bundle install --system
Step 0 : RUN bundle install --system
 ---> Running in 2cb8961c7e75
Don't run Bundler as root. Bundler can ask for sudo if it is needed, and
installing your bundle as root will break this application for all non-root
users on this machine.
There are no gemspecs at /usr/src/app.
2015/02/02 20:53:06 The command [/bin/sh -c bundle install --system] returned a non-zero code: 15

Sample Gemfile:
source 'http://rubygems.org'

ruby '1.9.3', engine: 'jruby', engine_version: '1.7.19'

gemspec

This pattern is taken from the official ruby-onbuild image.
The reason it is this way is to make the most effective use of the build cache.
In your version, in cases where something in "." has changed (anything at all), all gems will have to be re-installed.  This is undesirable in almost all scenarios.
It does make since to add the gempsec in before running bundle install.  If you want to change your PR to that I'd be happy to merge.
cc @tianon @yosifkit

Ah, right. Forgot about the build cache. I'll update shortly.
Is there a similar reason for putting ADDs on separate lines -- e.g.:
ONBUILD ADD Gemfile /usr/src/app/
ONBUILD ADD Gemfile.lock /usr/src/app/

instead of say:
ONBUILD ADD Gemfile Gemfile.lock *.gemspec /usr/src/app/

?

Because it wasn't possible before.
Let's keep it separate for now.

Ok, I'll update this in a bit.

Thanks!

Updated. 7faa356 just adds the one line with the gemspecs and 2fdb681 adds a comment that explains the ordering of steps to optimize use of the build cache.

Gah.
$ docker build -t msabramo/docverter .
Sending build context to Docker daemon 4.708 MB
Sending build context to Docker daemon
Step 0 : FROM msabramo/jruby:1.7-onbuild
# Executing 5 build triggers
Trigger 0, ADD Gemfile /usr/src/app/
Step 0 : ADD Gemfile /usr/src/app/
Trigger 1, ADD Gemfile.lock /usr/src/app/
Step 0 : ADD Gemfile.lock /usr/src/app/
Trigger 2, ADD *.gemspec .gemspec /usr/src/app/
Step 0 : ADD *.gemspec .gemspec /usr/src/app/
2015/02/03 07:44:45 .gemspec: no such file or directory

How do I add files that may or may not be there?

It looks like changing .gemspec to .gemspec* solves that problem.

Updated. 63d5d4077645a54947628d9a442f781169d416b4 just adds the one line with the gemspecs and de90ceb7ee22d4825deeeed7ba2d3f636b010ed6 adds a comment that explains the ordering of steps to optimize use of the build cache.
$ docker build -t msabramo/jruby:1.7-onbuild .
Sending build context to Docker daemon 3.072 kB
Sending build context to Docker daemon
Step 0 : FROM jruby:1.7-jdk
 ---> 6a31a0c9b2b2
Step 1 : RUN mkdir -p /usr/src/app
 ---> Using cache
 ---> 8a45d7ddcc4f
Step 2 : WORKDIR /usr/src/app
 ---> Using cache
 ---> 58e75d850ff4
Step 3 : ONBUILD add Gemfile /usr/src/app/
 ---> Using cache
 ---> debd9c9655c3
Step 4 : ONBUILD add Gemfile.lock /usr/src/app/
 ---> Using cache
 ---> e7de343150bf
Step 5 : ONBUILD add *.gemspec .gemspec* /usr/src/app/
 ---> Running in 8d86a6f3eb11
 ---> e4e68c069b58
Removing intermediate container 8d86a6f3eb11
Step 6 : ONBUILD run bundle install --system
 ---> Running in 8473cdd4f757
 ---> 6cf25bb788de
Removing intermediate container 8473cdd4f757
Step 7 : ONBUILD add . /usr/src/app
 ---> Running in 94d183502d94
 ---> fd20142d559f
Removing intermediate container 94d183502d94
Successfully built fd20142d559f

I would be -1 here.  If a user has no gemspec files (only Gemfile and lock) then it will fail to build for them.

@yosifkit It seems to work if you use the COPY foo* method instead of directly specifying the file.
However as was noticed above, most people refer to a VERSION const in their gemspecs, and there is no way for us to programmatically add that here.

On a phone so can't check now, but I think it will work if the ADD is done in one command because as long as there is at least one match I think it's ok.
Should I change it to do that?

Closing as this will have undesirable consequences for users missing a gemspec, or using a version of docker that doesn't support wildcards.


Change compilation targets

@pranavkm,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@PinpointTownes I'll let @DamianEdwards follow up separately. Merging this in for now to unblock my work.


Owning organisation(s) for each problem report now updated each time one comes in

Before this change, the owning organisation for each piece of content was only determined when the content was first seen. This meant that for content where the owning org was updated, the changes were never picked up.
https://trello.com/c/WK0EqQUl/100-content-in-feedex-incorrectly-appearing-for-dvsa-filter-medium
/cc @jamiecobbett @boffbowsh @binaryberry

:top:


Slash Commands with ViewMenu showing error on Discord Client

Describe the bug
Whenever I run a ViewMenu command as a Slash Command, the bot sends an Ephemeral Message saying
The application did not respond
I'm following the basic format. The command is a hybrid_command, and while the Slash Command raises an Error, the prefix command works without any issue.
Minimal Reproducible Code
from reactionmenu import ViewMenu, ViewSelect, Page, ViewButton
from datetime import datetime, timedelta
import discord
from discord import app_commands
from discord.ext import commands
from discord.ext.commands import Context

class General(commands.Cog, name="general"):
	def __init__(self, bot):
		self.bot = bot

        @commands.hybrid_command(name = "uwu", description = "...")
	@checks.not_blacklisted()
	async def uwu(self, context: Context) -> None:

		menu = ViewMenu(context, menu_type = ViewMenu.TypeEmbed, disable_items_on_timeout = False, remove_items_on_timeout = True, timeout = 600.0)

		page1 = discord.Embed(
			title="uwu1",
			description="...",
			color=0x752C18,
			timestamp=datetime.now(),
			)

		page2 = discord.Embed(
			title="uwu2",
			description="...",
			color=0x752C18,
			timestamp=datetime.now(),
			)

		menu.add_page(page1)
		menu.add_page(page2)

		menu.add_button(ViewButton(style=discord.ButtonStyle.primary, emoji='👑', label='x', custom_id = ViewButton.ID_GO_TO_FIRST_PAGE))
		menu.add_button(ViewButton(style=discord.ButtonStyle.primary, emoji='💎', label='y', custom_id = ViewButton.ID_GO_TO_LAST_PAGE))
		menu.add_button(ViewButton(style=discord.ButtonStyle.danger, label = "Close", custom_id = ViewButton.ID_END_SESSION))

		await menu.start()

Traceback
None

Library Version

Python v3.10.0-final
discord.py v2.3.1-final
aiohttp v3.8.4
system info: Windows 10 10.0.19044

Required Checklist

[X] I am using discord.py 2.0 or higher
[X] I am using reactionmenu 3.1.0 or higher
[X] I have the required intents enabled

Additional Comments
No response

This has been fixed as of v3.1.4. It has not been released to PyPI. It's planned to officially release in a few weeks, maybe the end of the month. In the mean time, you can install the development version of v3.1.4 by checking out the how to install section of the readme.
Thank you for reporting the issue! 😎


Header version/implementation selection logic update

Is your feature request related to a problem? Please describe.
#726 separates the header implementation logic and paves the way to use source selection for msgid implementation and CCSDS extended header use.  It partially uses new cmake flags to implement, but still relies on MESSAGE_FORMAT_IS_CCSDS_VER_2 being set correctly (but does not do it by default).
Describe the solution you'd like
Remove dependencies on MESSAGE_FORMAT_IS_CCSDS_VER_2 (see proposal below)
Describe alternatives you've considered
Consider changing cfe configs to cmake options (like the osconfig update), and use the cmake options to set the define if still required.  Or could just set the version 2 define if either cmake option (from #726) is set.
Additional context
#726
Requester Info
Jacob Hageman - NASA/GSFC

Proposal:

cfe_sb_extern_typedefs.h:CFE_SB_MsgId_Atom_t always uint32 (always big enough to hold either msgid v1 or msgid v2
cfe_sb_init.c:Remove CFE_SB_InitMsgMap version reporting (gets reported at prep)
cfe_sb_verify.h: Error if VER_2 is defined (since it won't do anything and will change behavior underneath user if they don't use the new method)
cfe_sb_msg_id_util.c: clean documentation (now documented in msg implementation)
sb_UT.c: Remove explict checksum assert
ut_support.c: get actual command code from the structure, not a byte offset
cfe_msg_defaults.h: CCSDS Version in header... not really standards conforming anyways to change this number, just default to 0?

VER_2 define is also used to set the MID_BASE and MID_GLOB values, which has it's own problems and ticket for discussion at #732.

Proposal:

cfe_sb_extern_typedefs.h:CFE_SB_MsgId_Atom_t always uint32 (always big enough to hold either msgid v1 or msgid v2

why Atom? Would there be a CFE_SB_MsgId_t or CFE_SB_MsgId_Molecule_t? Or are operations on it atomic? Kidding aside, I say skip the addl 5 chars.

There actually is a reason - this typedef is the "internal value" and not the one passed around by apps.  The value used in the external API is CFE_SB_MsgId_t and it is defined separately, and it needs a distinct name.
The term "Atom" originates from the circa 3-4 years ago when the GHAPS project was developing the EDS toolchain and it looked like the CFE infrastructure would move potentially toward an EDS-based solution.  This tool puts an "_Atom_t" suffix on all the fundamental data types (integers, etc) that are generated from EDS definitions.
All hardcoded data types in the "*_extern_typedefs.h" files were separated from the normal API/msg header file in anticipation of transitioning to a generated header file in some future version.
I'm not necessarily tied to the "Atom" term - as it does imply atomicness but that is not necessarily so - but I do think these basic/fundamental data types used for interoperability should share a common suffix that is different than just _t to help differentiate from other local-only typedefs.

There actually is a reason - this typedef is the "internal value" and not the one passed around by apps. The value used in the external API is CFE_SB_MsgId_t and it is defined separately, and it needs a distinct name.

how about CFE_SB_MsgId_Internal_t (or some abbrev.?)

Not an internal/external thing -- it is a question of if we want to name typedefs of basic fundamental values (e.g. int, float) differently than compound values (struct, union, etc).
The EDS tool (currently) generates typedefs with the following pattern:
basic fundamental types -> _Atom_t
enums -> _Enum_t
strings -> _String_t
array -> _Array_t
everything else (incl structs) ->_t  (so the names would end up being compatible with existing code).
While I do like having some indicator in the typedef name to indicate its nature, I'm not bound to it.  Could be replaced by another layer of typedef or updating the tool to output typedefs without the extra suffix.
It was a CCB topic of conversation years ago when these *_extern_typedefs.h files were forked off from the others, and at the time the CCB decided to keep the suffixes, but this could always be changed again.
As it stands, it was only types/structs that were anticipated being part of the inter-processor communication (message, tables) that ever had this convention applied anyway - types for internal use were never made to match this pattern.

Not an internal/external thing...

Thanks Joe, makes sense.
<joke>Or I suggest we have a base64 encoding of a bitmap that maps into a semantic definition of types.
CFE_aF3#_t -> SB, Internal, Atom, Message, ID...</joke>


Better /users/me/ architecture

The current implementation of /users/me/ is difficult to customize because it depends on overrides of the UserViewSet in conjunction with a custom route on that viewset.
It would be nice to explore either:

Splitting the current user endpoints into another viewset, which can be nested on top of /users/.
Revising the architecture of UserViewSet so that "me" is not a separate action, but just a special keyword that is transformed internally into the current user's pk.

2 is much simpler, but 1 is more customizable. 2 might also pose some issues with regards to logic that should be present on the current user, but not public users or vice versa.

I've been using djoser.views.UserView as the /me/ endpoint for ages, I wasn't even aware there was another way of getting this information (I don't even use UserViewSet).

I've been using djoser.views.UserView as the /me/ endpoint for ages, I wasn't even aware there was another way of getting this information (I don't even use UserViewSet).

In commit #1888efcbb9f03f4657433313343fcd9ed29dff5c change from UserView to UserViewSet to allow use custom @action by @pmlynarek


Cannot initialize replication

I am unable to transfer any data without getting this issue. I made sure to configure MySQL's my.cnf and restart it as well.
table_list = [table["table_name"] for table in self.cursor_buffered.fetchall()] KeyError:
'table_name'

I've tried variations of the database names in the configuration nothing seems to work. My localhost Postgres desired table is to output is public.testing and the MySQL schema is testing
Which I believe should look like this:
pg_conn:
  host: "localhost"
  port: "5432"
  user: "usr_replica"
  password: ""
  database: "testing"
  charset: "utf8"

sources:
  mysql:
    db_conn:
      host: "localhost"
      port: "3306"
      user: "usr_replica"
      password: ""
      charset: 'utf8'
      connect_timeout: 10
    schema_mappings:
      testing: public

Here is the full stacktrace.
Traceback (most recent call last):
  File "/Users/tylerguo/Documents/HTRepos/ht-processdb/ve/bin/chameleon", line 5, in     <module>
    exec(compile(open(__file__).read(), __file__, 'exec'))
  File "/Users/tylerguo/Documents/HTRepos/ht-processdb/ve/bin/chameleon.py", line 58, in     <module>
    getattr(replica, args.command)()
  File "/Users/tylerguo/Documents/HTRepos/ht-processdb/ve/lib/python3.6/site-    packages/pg_chameleon/lib/global_lib.py", line 327, in init_replica
    self.__init_mysql_replica()
  File "/Users/tylerguo/Documents/HTRepos/ht-processdb/ve/lib/python3.6/site-    packages/pg_chameleon/lib/global_lib.py", line 337, in __init_mysql_replica
    self.mysql_source.init_replica()
  File "/Users/tylerguo/Documents/HTRepos/ht-processdb/ve/lib/python3.6/site-    packages/pg_chameleon/lib/mysql_lib.py", line 1374, in init_replica
    self.get_table_list()
  File "/Users/tylerguo/Documents/HTRepos/ht-processdb/ve/lib/python3.6/site-    packages/pg_chameleon/lib/mysql_lib.py", line 261, in get_table_list
    table_list = [table["table_name"] for table in self.cursor_buffered.fetchall()]
  File "/Users/tylerguo/Documents/HTRepos/ht-processdb/ve/lib/python3.6/site-    packages/pg_chameleon/lib/mysql_lib.py", line 261, in <listcomp>
    table_list = [table["table_name"] for table in self.cursor_buffered.fetchall()]
KeyError: 'table_name'

Mmm does the user have access to the tables? There are specify grants to give before being able to use pgchameleon

Schema mappings seems wrong to me. What's the database on mysql you are reading from?

Mmm does the user have access to the tables? There are specify grants to give before being able to use pgchameleon

Yes, I made sure to log in and have even gotten a stacktrace when an incorrect password was given.

Schema mappings seems wrong to me. What's the database on mysql you are reading from?

I tried using my own schema/database at first as well as my own credentials after that didn't work I followed one of your talks on YouTube and used the Sakila database and your example credentials.
MySQL version 8.0.12 and Postgres version 11.2 on macOS High Sierra 10.13.3

i got the same issue with PostgreSQL 11.3, MySQL 8.0.16 (community version), pg-chameleon 2.0.10 on CentOS Linux release 7.6.1810. Here is my my.cnf and defaults.yml. I have followed the online documentation with some addition references from Google.
cat /etc/my.cnf
[mysqld]
datadir=/var/lib/mysql
socket=/var/lib/mysql/mysql.sock

log-error=/var/log/mysqld.log
pid-file=/var/run/mysqld/mysqld.pid

#system timestamp
log_timestamps=SYSTEM
binlog_format= ROW
binlog_row_image=FULL
log-bin = mysql-bin
server-id = 1
cat default.yml
#global settings
pid_dir: '~/.pg_chameleon/pid/'
log_dir: '~/.pg_chameleon/logs/'
log_dest: file
log_level: info
log_days_keep: 10
rollbar_key: ''
rollbar_env: ''
type_override allows the user to override the default type conversion into a different one.
type_override:
"tinyint(1)":
override_to: boolean
override_tables:
- "*"
#postgres  destination connection
pg_conn:
host: "localhost"
port: "5432"
user: "usr_replica"
password: "replica"
database: "db_replica"
charset: "utf8"
sources:
mysql:
db_conn:
host: "localhost"
port: "3306"
user: "usr_replica"
password: "replica123"
charset: 'utf8'
connect_timeout: 10
schema_mappings:
world_x: pgworld_x
limit_tables:
- delphis_mediterranea.foo
skip_tables:

- delphis_mediterranea.bar
grant_select_to:
  - usr_replica
lock_timeout: "120s"
my_server_id: 100
replica_batch_size: 10000
replay_max_rows: 10000
batch_retention: '1 day'
copy_max_memory: "300M"
copy_mode: 'file'
out_dir: /tmp
sleep_loop: 1
on_error_replay: continue
on_error_read: continue
auto_maintenance: "disabled"
gtid_enable: No
type: mysql
skip_events:
  insert:

- delphis_mediterranea.foo #skips inserts on the table delphis_mediterranea.foo
  delete:

- delphis_mediterranea #skips deletes on schema delphis_mediterranea
  update:

pgsql:
db_conn:
host: "localhost"
port: "5432"
user: "usr_replica"
password: "replica"
database:  "db_replica"
charset: 'utf8'
connect_timeout: 10
schema_mappings:
world_x: pg_world_x
limit_tables:
- loxodonta_africana.foo
skip_tables:
- loxodonta_africana.bar
copy_max_memory: "300M"
grant_select_to:
- usr_readonly
lock_timeout: "10s"
my_server_id: 100
replica_batch_size: 3000
replay_max_rows : 10000
sleep_loop: 5
batch_retention: '1 day'
copy_mode: 'file'
out_dir: /tmp
type: pgsql

i got the same issue with PostgreSQL 11.3, MySQL 8.0.16 (community version), pg-chameleon 2.0.10 on CentOS Linux release 7.6.1810. Here is my my.cnf and defaults.yml. I have followed the online documentation with some addition references from Google.
 cat default.yml
---
#global settings
pid_dir: '~/.pg_chameleon/pid/'
log_dir: '~/.pg_chameleon/logs/'
log_dest: file
log_level: info
log_days_keep: 10
rollbar_key: ''
rollbar_env: ''

# type_override allows the user to override the default type conversion into a different one.
type_override:
  "tinyint(1)":
    override_to: boolean
    override_tables:
      - "*"

#postgres  destination connection
pg_conn:
  host: "localhost"
  port: "5432"
  user: "usr_replica"
  password: "replica"
  database: "db_replica"
  charset: "utf8"

sources:
  mysql:
    db_conn:
      host: "localhost"
      port: "3306"
      user: "usr_replica"
      password: "replica123"
      charset: 'utf8'
      connect_timeout: 10
    schema_mappings:
      world_x: pgworld_x
    limit_tables:
#      - delphis_mediterranea.foo
    skip_tables:
#      - delphis_mediterranea.bar
    grant_select_to:
      - usr_replica
    lock_timeout: "120s"
    my_server_id: 100
    replica_batch_size: 10000
    replay_max_rows: 10000
    batch_retention: '1 day'
    copy_max_memory: "300M"
    copy_mode: 'file'
    out_dir: /tmp
    sleep_loop: 1
    on_error_replay: continue
    on_error_read: continue
    auto_maintenance: "disabled"
    gtid_enable: No
    type: mysql
    skip_events:
      insert:
#        - delphis_mediterranea.foo #skips inserts on the table delphis_mediterranea.foo
      delete:
#        - delphis_mediterranea #skips deletes on schema delphis_mediterranea
      update:

  pgsql:
  db_conn:
    host: "localhost"
    port: "5432"
    user: "usr_replica"
    password: "replica"
    database:  "db_replica"
    charset: 'utf8'
    connect_timeout: 10
#      schema_mappings:
#        world_x: pg_world_x
#      limit_tables:
#        - loxodonta_africana.foo
#      skip_tables:
#        - loxodonta_africana.bar
#      copy_max_memory: "300M"
#      grant_select_to:
#        - usr_readonly
#      lock_timeout: "10s"
#      my_server_id: 100
#      replica_batch_size: 3000
#      replay_max_rows : 10000
#      sleep_loop: 5
#      batch_retention: '1 day'
#      copy_mode: 'file'
#      out_dir: /tmp
#      type: pgsql

cat /etc/my.cnf
# For advice on how to change settings please see
# http://dev.mysql.com/doc/refman/8.0/en/server-configuration-defaults.html

[mysqld]
#
datadir=/var/lib/mysql
socket=/var/lib/mysql/mysql.sock
#
log-error=/var/log/mysqld.log
pid-file=/var/run/mysqld/mysqld.pid
#
#system timestamp
log_timestamps=SYSTEM
binlog_format= ROW
binlog_row_image=FULL
log-bin = mysql-bin
server-id = 1

I think i kind of figured out what's happening but i need the developer to confirm if my analysis was right and the fix for that.
I could see the below code in the mysql_lib.py, but just a few line above that code is a sql query (pasted further below), which gets a list of tables that belong to the intended schema. I tried running that plain sql against the MySQL database and i get the results just fine but there is a header row "TABLE_NAME", which i assume is getting processed through the code of the for loop "for schema in self.schema_list:".
I tried to include that header as a table name in the "skip tables" section but to no avail. I still get the same error.
"table_list = [table["table_name"] for table in self.cursor_buffered.fetchall()]"
sql_tables="""
			SELECT
				table_name
			FROM
				information_schema.TABLES
			WHERE
					table_type='BASE TABLE'
				AND table_schema=%s

sorry for the long delay, I've been around. I'll pick up all the stuff and the project's development asap.

... There is a header row "TABLE_NAME", which i assume is getting processed through the code of the for loop "for schema in

As pointed in the above comment, the cursor returns a dict in which the key is "TABLE_NAME" (all upper case), so change the key 'table_name' in line to 'TABLE_NAME' will be an immediate fix.

Fine, there are similar cases all over mysql_lib.py. Changing the lower case "table_name" (and alike) to "TABLE_NAME" may have compatibility issues with previous mysql version.
I think maybe we can change the query statement:
SELECT
	table_name
FROM
	information_schema.TABLES
WHERE
	table_type='BASE TABLE'
	AND table_schema=%s
;

to something like:
SELECT
	table_name AS table_name
FROM
	information_schema.TABLES
WHERE
	table_type='BASE TABLE'
	AND table_schema=%s
;

Works for me (MySQL 8.0)

thanks, I'll have a look

Hi, i got the same issue, version :
mysql  Ver 8.0.18 for Linux on x86_64 (MySQL Community Server - GPL)
psql (PostgreSQL) 9.6.15
CentOS Linux release 7.7.1908 (Core)

content of default.yml :
# global settings
pid_dir: '~/.pg_chameleon/pid/'
log_dir: '~/.pg_chameleon/logs/'
log_dest: file
log_level: info
log_days_keep: 10
rollbar_key: ''
rollbar_env: ''# type_override allows the user to override the default type conversion
# into a different one.type_override:
 "tinyint(1)":
   override_to: boolean
   override_tables:
     - "*"# postgres  destination connection
pg_conn:
 host: "localhost"
 port: "5432"
 user: "usr_replica"
 password: "pass"
 database: "db_replica"
 charset: "utf8"sources:
 mysql:
   db_conn:
     host: "localhost"
     port: "3306"
     user: "usr_replica"
     password: "pass"
     charset: 'utf8'
     connect_timeout: 10
   schema_mappings:
     database: pg_database
   limit_tables:
#      - delphis_mediterranea.foo
   skip_tables:
#      - delphis_mediterranea.bar
   grant_select_to:
     - usr_replica
   lock_timeout: "120s"
   my_server_id: 100
   replica_batch_size: 10000
   replay_max_rows: 10000
   batch_retention: '1 day'
   copy_max_memory: "300M"
   copy_mode: 'file'
   out_dir: /tmp
   sleep_loop: 1
   on_error_replay: continue
   on_error_read: continue
   auto_maintenance: "disabled"
        gtid_enable: false
   type: mysql
   skip_events:
     insert:
#        - delphis_mediterranea.foo  # skips inserts on delphis_mediterranea.foo
     delete:
#        - delphis_mediterranea  # skips deletes on schema delphis_mediterranea
     update:#  pgsql:
#    db_conn:
#      host: "localhost"
#      port: "5432"
#      user: "usr_replica"
#      password: "never_commit_passwords"
#      database: "db_replica"
#      charset: 'utf8'
#      connect_timeout: 10
#      schema_mappings:
#        loxodonta_africana: elephas_maximus
#      limit_tables:
#        - loxodonta_africana.foo
#      skip_tables:
#        - loxodonta_africana.bar
#      copy_max_memory: "300M"
#      grant_select_to:
#        - usr_readonly
#      lock_timeout: "10s"
#      my_server_id: 100
#      replica_batch_size: 3000
#      replay_max_rows: 10000
#      sleep_loop: 5
#      batch_retention: '1 day'
#      copy_mode: 'file'
#      out_dir: /tmp
#      type: pgsql

# chameleon init_replica --config default --debug --source mysql
2019-11-07 09:56:39 MainProcess DEBUG pg_lib.py (620): Changing the autocommit flag to True
2019-11-07 09:56:39 MainProcess DEBUG pg_lib.py (620): Changing the autocommit flag to True
2019-11-07 09:56:39 MainProcess DEBUG mysql_lib.py (1366): starting init replica for source mysql
2019-11-07 09:56:39 MainProcess DEBUG pg_lib.py (620): Changing the autocommit flag to True
2019-11-07 09:56:39 MainProcess DEBUG pg_lib.py (3070): Collecting schema mappings for source mysql
2019-11-07 09:56:39 MainProcess INFO pg_lib.py (1846): deleting all the table references from the replica catalog for source mysql
Traceback (most recent call last):
 File "/root/migration/bin/chameleon", line 5, in <module>
   exec(compile(open(__file__).read(), __file__, 'exec'))
 File "/root/migration/bin/chameleon.py", line 58, in <module>
   getattr(replica, args.command)()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/global_lib.py", line 327, in init_replica
   self.__init_mysql_replica()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/global_lib.py", line 337, in __init_mysql_replica
   self.mysql_source.init_replica()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/mysql_lib.py", line 1374, in init_replica
   self.get_table_list()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/mysql_lib.py", line 261, in get_table_list
   table_list = [table["table_name"] for table in self.cursor_buffered.fetchall()]
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/mysql_lib.py", line 261, in <listcomp>
   table_list = [table["table_name"] for table in self.cursor_buffered.fetchall()]
KeyError: 'table_name'

After doing the solution from sunbing81 's answer we got this :
# chameleon init_replica --config default --debug --source mysql
2019-11-07 10:05:22 MainProcess DEBUG pg_lib.py (620): Changing the autocommit flag to True
2019-11-07 10:05:22 MainProcess DEBUG pg_lib.py (620): Changing the autocommit flag to True
2019-11-07 10:05:22 MainProcess DEBUG mysql_lib.py (1366): starting init replica for source mysql
2019-11-07 10:05:22 MainProcess DEBUG pg_lib.py (620): Changing the autocommit flag to True
2019-11-07 10:05:22 MainProcess DEBUG pg_lib.py (3070): Collecting schema mappings for source mysql
2019-11-07 10:05:22 MainProcess INFO pg_lib.py (1846): deleting all the table references from the replica catalog for source mysql
2019-11-07 10:05:22 MainProcess DEBUG mysql_lib.py (291): Creating the schema _redmine_adm_tmp.
2019-11-07 10:05:22 MainProcess DEBUG mysql_lib.py (293): Creating the schema redmine_adm.
2019-11-07 10:05:22 MainProcess DEBUG mysql_lib.py (304): Dropping the schema _redmine_adm_tmp.
2019-11-07 10:05:22 MainProcess DEBUG pg_lib.py (658): Changing the lock timeout for the session to 120s.
2019-11-07 10:05:22 MainProcess CRITICAL mysql_lib.py (1400): init replica for source mysql failed
Traceback (most recent call last):
 File "/root/migration/bin/chameleon", line 5, in <module>
   exec(compile(open(__file__).read(), __file__, 'exec'))
 File "/root/migration/bin/chameleon.py", line 58, in <module>
   getattr(replica, args.command)()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/global_lib.py", line 327, in init_replica
   self.__init_mysql_replica()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/global_lib.py", line 337, in __init_mysql_replica
   self.mysql_source.init_replica()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/mysql_lib.py", line 1379, in init_replica
   self.create_destination_tables()
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/mysql_lib.py", line 394, in create_destination_tables
   self.pg_engine.create_table(table_metadata, table, schema, 'mysql')
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/pg_lib.py", line 2972, in create_table
   table_ddl = self.__build_create_table_mysql( table_metadata,table_name,  schema)
 File "/root/migration/lib64/python3.6/site-packages/pg_chameleon/lib/pg_lib.py", line 2378, in __build_create_table_mysql
   if column["is_nullable"]=="NO":
KeyError: 'is_nullable'

Thanks for the help

hi, can you please check if the last PR merged fixes your issue? thanks.

I will check when I get some time, thanks

Hi Federico, the last pull request solved the problems we had, the initialization is done correctly and we can continue the conversion, thank you

hi, can you please check if the last PR merged fixes your issue? thanks.

Hi Federico, the last pull request solved the problems we had, the initialization is done correctly and we can continue the conversion, thank you

hi, can you please check if the last PR merged fixes your issue? thanks.

Hi Federico, the last pull request solved the problems we had, the initialization is done correctly and we can continue the conversion, thank you

Amazing. I will pack a new release soon. Thanks

Hi Federico,
I lost that setup which caused the issue but I am happy for you to close this ticket as it is confirmed by others to have fixed the replication initialization issue. Thanks for fixing it.
Regards, Nawaz Ahmed +61 414 513 643
On Thursday, 28 November, 2019, 12:02:41 am GMT+10, Federico Campoli<EMAIL_ADDRESS>wrote:

hi, can you please check if the last PR merged fixes your issue? thanks.
Hi Federico, the last pull request solved the problems we had, the initialization is done correctly and we can continue the conversion, thank you
Amazing. I will pack a new release soon. Thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

many thanks to @daniel-qcode who fixed it. closing the ticket


[opentitantool,hyperdebug] enable activating hyperdebug PWM to drive OT external clock

Background
For manufacturing tests, we need to be able to drive LC transitions via the LC JTAG TAP.
Problem
Driving LC transitions via the LC JTAG TAP may require using the external clock. However there is no way to actuate this on the FPGA at the moment.
(Potential) Solution
Let HyperDebug board drive external clk with PWM signal, and/or, potenially the SAM3X may be able to as well (investigation needed here) in the case where there is no HyperDebug board attached to the CW310 board.
Requirements

[ ] External clock should run at frequency fast enough to drive PLL on FPGA and at either 48 or 96 MHz, so hyperdebug PWM signal should support that speed
[ ] hyperdebug firmware must be updated to allow toggling on / off a PWM signal on a specific pin (preferrably what ever pin is connected to IOC6 as this is the pin used for the external clock on the OT ASIC)
[ ] opentitantool should expose a mechanism to toggle on/off "external clock"

HyperDebug pin CN10_7 is connected to IOC6 on the swizzle board, at least according to hyperdebug_cw310.json in our repository.  That pin happens to be capable of PWM on the STM32L5 microcontroller.
The fastest signal that HyperDebug could possibly generate would be half its internal clock speed, which is currently 104MHz.  The internal clock could be lowered to 96MHz, possibly allowing a 48MHz signal to be generated on CN10_7.
If we insist on a 96MHz signal, then there is only CN10_31, which has a special "clock output" mode, which present the unaltered STM32L5 clock, if enabled.
I am not an electrical engineer, but I am somewhat worried that signals this fast will experience quality issues considering the many connector and meandering traces on the Nucleo board, swizzle board and ribbon cable to the CW310 board.  Are we sure that the FPGA implementation can and need to run at this fast speed, or could the testing be done at a lower speed?
I will work some more to verify the exact capabilities of HyperDebug pin CN10_7.

Thanks @jesultra! Perhaps @a-will could chime in here too?

On my scope, I have seen a 48MHz signal on CN10_7, as well as a 96MHz signal on CN10_31.  They had significant distortion, possibly in part due to the limitation of the ordinary 10x probes available.  I do not know if these high-frequency signals would be usable on the "far end" of the swizzle board and ribbon cable.
I want to allow OpenTitan hardware developers to try out the signals, but I do not want to spend too much time adding fancy abstractions to the OpenTitanTool traits.  Instead am thinking on initially piggy backing on the existing support for an "alternate" functions.
Currently, each particular HyperDebug pin has zero or one "alternate" function, that is, some pins become UART signals when put in this mode, others become SPI or I2C signals.  Since it happens that neither CN10_7 nor CN10_31 currently have any alternate function, I propose that we declare their alternate function to be 48MHz and 96Mhz signal generation respectively.  It will take me a few days to polish my code, and get it through ChromeOS EC review.  I could produce a binary firmware image before then, if required.
If it turns out that the feature is useful, and that we maybe want more control of the frequency, then we can at that point add proper features to OpenTitanTool.

That SG to me, thanks for looking Jes! (as @a-will pointed out) the PWM signal will drive a PLL on the FPGA, so the signal could be slower, maybe ~10MHz, but for ASIC we will need 48 or 96 MHz I think.

I have a demonstration HyperDebug firmware, which can be used for testing frequency generation.  I do not want to submit it into the actual repository, until we know that it is useful.
It can be obtained by applying this patch: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4329118
while following the building instructions here: https://docs.google.com/document/d/1ZEH7L5j9-wMw4tkW28-xt6JU5B6hTX0RdZD4h4OZzDo/edit#heading=h.her82ektz472
Once running the new firmware, OpenTitanTool can enable the feature without patches, by running any of these commands:
48 MHz output on CN10_7 by: opentitantool gpio set-mode CN10_7 alternate
96 MHz output on CN10_31 by: opentitantool  gpio set-mode CN10_31 alternate
The code can easily be modified to generate any fraction of 48 MHz on pin CN10_7 instead (e.g. 24 MHz, 16 Mhz, 12 Mhz, 9.6 Mhz, 8 Mhz, ...).  Doing so may work better with the long traces and ribbon cables to the FPGA, let me know if you prefer any of the lower frequencies.

wonderful, thanks @jesultra !

After some further discussion, for the time being, I think we are going to just route the ext_clk on the OT FPGA top-level to an internal PLL (the same / a similar one that already drives the main system clk) so the OT FPGA instance doesn't freeze up when we select the external clk source (since it is currently tied to 0).
This will allow us to still develop the manufacturing tests on the FPGA, but means we won't need hyp to drive an ext_clk on any FPGA pin for the time being.
I could still see a potential case in the future where we want to use hyp + ASIC together and drive ext_clk. However, in that case, might be better to have ext_clk driven by something other than hyp, e.g. something with tigher tolerances like a function generator. Though that is still TBD.
Either way, I think we should keep the Hyp firmware patch available (since you already made it @jesultra) and we can evaluate it in the future if need be, but I will mark this issue as a future item, and change the priority.

For the record, STM has this to say about the RC oscillator found on HyperDebug: "each device is factory calibrated by ST for 1 % accuracy at TA = 25°C."  Such a precision is far from fantastic.  It could possibly be improved by HyperDebug software continuously calibrating based on the USB bus, but it will probably still not be exactly worthy of being a frequency reference.

going to close this issue for now, as we have gone a different route for ext clk on FPGA, we can open this again later if needed.


High Contrast: No border around toasts and tooltips

Description
There is no border around toasts and tooltips when running in windows high contrast mode
Steps to reproduce

Go to http://opensource.adobe.com/spectrum-css/
Turn on Windows High Contrast mode
Go to Toasts
Observe no border
Go to tooltip
Observe no border

Expected behavior
I expect there to be a border to separate the content from the rest of the page
Screenshots

Environment

Spectrum CSS version:  3.0.0.beta
Browser(s) and OS(s):  Edge 86.0 on Win 10

Additional context

I want to work on this bug

This has been taken care of with the migration to @adobe/spectrum-tokens. Please reopen if necessary


Xunit conversion

Migrated unit tests to Xunit.  Fixes #15

Required upgrading the test project to Framework 4.5 (minimum supported by Xunit).
Required removing some mock setups which were previously failing, but being hidden by the ExpectedException attribute which is not used in Xunit (fixes #36).

@richardlawley good work, thanks.
I've checked your branch locally and catch that some tests periodically fail. I think this happening because xunit 2.0 by default runs tests in parallel and Rhino mocks turned up not thread safe :smile:
So until we couldn't resolve this we should disable parallelism. You can do this by adding the following line to AssemblyInfo.cs file.
[assembly: CollectionBehavior(CollectionBehavior.CollectionPerAssembly, DisableTestParallelization = true, MaxParallelThreads = 1)]

About bb743c9. Could you revert it and tests which fail mark as
[Fact(Skip="https://github.com/NModbus4/NModbus4/issues/36")]

I don't believe #36 is actually a problem - it's the mocks.VerifyAll() which failed - from what I can see the thing being tested is still fine.  I suspect the setup was copied from another test, but the use of ExpectedException prevented mocks.VerifyAll from ever being called so it was never noticed.
Let me know what you think before I do any more tidying up.

I don't believe #36 is actually a problem - it's the mocks.VerifyAll() which failed - from what I can see the thing being tested is still fine.

:ok: will mark your PR as fix for #36

LGTM. Thanks for taking a look at this. I will merge the PR once the CI completes.


More flexible index page templating

Issue: https://github.com/plotly/dash/issues/265
PR: https://github.com/plotly/dash/pull/286

🎉The index customization feature has been merged into the latest release; closing this issue now.