[BUG] Pie chart is cut in half when responsive and inside a collapsed element 2.6.0

Expected Behavior

Currently while using a pie chart with responsive: true in a bootstrap modal and using Reactjs.net (or any collapsible element in html5 by my reckoning) I experienced a bug where the pie chart in a grid would become broken when re-rendered in a hidden modal.
Before I go any further yes I know reactjs has a charts wrapper called react-chartjs but using react components in Reactjs.net is hard and as far as I can tell charts.js works fine beyond this bug I found.
Here is the modal before I search my list of modals:

The canvas is in a <div class="col-sm-4"> wrapper element. The same goes for the bootstrap panels next to it.
I would expect this behavior to remain constant but when I used a reactjs search feature to filter out my list of modals and then reopen a modal the below behavior is encountered.
Current Behavior

After filtering out the modals based on their name, the pie chart appears like so:

The pie chart will flicker between the half-pie and the small pie depending on which one has had a mouseover event most recently.
Possible Solution

For now, the solution I found was to manually resize the canvas each time the modal is updated in Reactjs using componentWillUpdate(). Simply getting the canvas node and setting canvas.height=height and canvas.width=width works fine. I did this because the issue seems to be related to the responsive resizing returning a value of -30 for the dimensions of the canvas due to the canvas being in a collapsed modal element and then getting confused when it tries to resize from there. So simply changing this value manually before calling new Chart( ... ) works to fix this bug. However this does result in blurry canvas in some situations.
Steps to Reproduce (for bugs)

Bug would be hard to reproduce in a fiddle or something because it involves Reactjs. If someone can point me in the right direction for reproducing this bug in straight-forward way I would be glad to do that.
Environment

Chart.js version: 2.6.0
Browser name and version: Chrome Version 58.0.3029.110 (64-bit)
Reactjs v15.3.2
Bootstrap v3.3.7

I think this might be as simple as making sure the outputs of https://github.com/chartjs/Chart.js/blob/master/src/core/core.controller.js#L172-L173 are >= 0.
Do you have a fiddle or other equivalent that reproduces this that we could test with

I do not have a fiddle up and running. I can provide snippets of the code if that would be useful, however.

Ok, if you are able to test a local build with the idea I had above and it works, please submit a PR 😄

PR submitted. #4406

I had a similar problem (issue 4410). Strangely i solved with this:
if ((window.outerHeight > window.outerWidth) && window.devicePixelRatio > 1) {         alturaCanvas = parseInt(4 * anchoCanvas);         $('#contenedorGraficosTemp #grDescuentos').height(alturaCanvas);         $('#contenedorGraficosTemp #grDescuentos').width(anchoCanvas);     }     else {         $('#contenedorGraficosTemp #grDescuentos').height(alturaCanvas);     }


Fix for IDENTITY-3005

a temporary variable set to hold the recently changed userstore dto

Need to have a better fix. Because this will only solve the issue in one request. But if another request comes and the config file is still not persisted, again wrong values will be shown.


Global Console Design - Phase 1

The design had outgrown #2408 and the discussion unwieldy. I have pulled everything together into a proposal.

@lulf @vbusch updated to reflect the implementation as stands.  As I feel this design will need to change again for the next phase, I haven't polished the document.  I have just made the minimal changes.


Question: Luna as a USB-UART bridge

Hello!
I was wondering if Luna can be used out of the box as USB-UART bridge with an example. Do you think it's possible to generate a DTR and a RTS signals? Essencially I want to use the ECP5 to act like a USB-UART bridge to interface a ESP32 with a computer. It'd be amazing if it'd be possible to flash firmware via the USB-UART bridge to the ESP32 too! And the ESP32 can use DTR and RTS signals to put itself it bootloader mode, like so:

Do you think it's possible?
Thanks a lot ;)

It's definitely possible, but would take a little gateware work.
There is an example in the repository, in examples/usb/acm_serial.py, of creating a device that shows up as a USB serial port (with TX looped back to RX in the example). The example uses a partial implementation of the CDC ACM specification, that can be found in luna/gateware/usb/devices/acm.py.
To create a complete USB-serial adapter you would need to combine that with some further logic that implements the UART side of the adapter. There is a simple transmitter in luna/gateware/interface/uart.py but not a receiver. You would also need to implement further parts of the CDC ACM spec in order to make the DTR and RTS signals controllable by the host.


Update gamedata / plugin for TF2 version 8830351

Closes #10.
Important breaking change:  CTFPlayer::GiveDefaultItems has been inlined on Windows (caller xref is now CTFPlayer::InitClass). A hotfix was applied to make that SDKCall optional so the plugin would continue functioning without being able to regenerate players on whitelist reload.  It will raise a warning on startup.
Feel free to workshop other solutions here.

Thank you. Also, I am suffering.


(language name) Short description of issue...

Describe the issue
Pascal code has HEX notation: $ab, but github site don't support this.
Eg here
https://gitlab.com/freepascal.org/lazarus/lazarus/-/commit/f737b23e7c36fefb16b14600c8dc79979213d8e0
Which language seems to have the issue?
Pascal.
Are you using highlight or highlightAuto?
I use the GH site.
...
Sample Code to Reproduce
function UnicodeToISO_8859_7(Unicode: cardinal): integer;
begin
  case Unicode of
    0..$A0,
    $A3, $A6..$A9, $AB..$AE,
    $B0..$B3, $B7, $BB, $BD,
    $D2, $FF: Result:= Unicode;
    $2BD: Result:= $A1;
    $2BC: Result:= $A2;
    $20AC: Result:= $A4;
    $20AF: Result:= $A5;
    $37A: Result:= $AA;
    $2015: Result:= $AF;
    $384..$386, $388..$38A, $38C,
    $38E..$3A1, $3A3..$3CE: Result:= Unicode-($384-$B4);
    else Result:= -1;
  end;
end;

Expected behavior
Highlighted hex numbers.
Additional context
n/a.

GitLab does not use Highlight.js for highlighting.   Highlight.js does include highligthing $ literals already.


fix(curriculum): change the instruction text in step 31

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Avoid using word we in the instruction text owing to make it conform to how-to-work-on-coding-challenges document.
Since I'm not native English speaker, If any one could check and provide some advice if this is natural for instruction text, that would be a great help.
Affected page:
New Responsive Web Design > Learn HTML Forms by Building a Registration Form > Step 31
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-html-forms-by-building-a-registration-form/step-31

Hello,
In Japan, actually, It's been almost unbelievable I think, there is the woman whose age is 119 (She said aimming for age 120).
I believe freeCodeCamp acceps people of all ages. So the number 120 in instruction text may be able to change some more greater one.

Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 🎉


Errors when clicking copy for key generation commands.

During step 4 of the key generation, there are buttons to copy out the commands. When I click them I get a TypeError. Screen shot is attached.

Brave browser on MacOS

Closed by #74


shell support in groovy plugin

Expected Behavior
gradle shell would  launch groovysh with the classpath reflecting project dependencies
Current Behavior
An interactive shell would enable REPL-style development as a complement to the supported compile-test-package workflow
Context
REPL is invaluable for exploratory data science workflows as well as ETL development, where you need to experiment with transformations on data loaded in the memory. To achieve this I currently use an adhoc groovysh script that grabs the same dependencies declared in build.gradle.
There is an unmaintained groovysh plugin that almost worked for me except for invoking previous/next command (using arrow keys or Ctrl-N/Ctrl-P). Still I feel like groovysh should be part of the official groovy plugin rather than a 3rd party addon.
In the competitive maven camp GMavenPlus provides comprehensive Groovy support including the requested shell

I brought something similar up in https://github.com/gradle/gradle/issues/1251#issuecomment-297840054

Just wondering...is the Groovy web console not a viable option or do you necessarily need dependencies declared in your build script?
For experimenting with Gradle build code isn't running a build script good enough. Maybe you can describe your use case on a deeper level.

I think we should start by gather concrete use cases if you are still interested in the functionality. IMHO it's good enough for users to change a build script and run it to the see effect of a change.

I am pretty sure we'll work on this anytime soon. Please provide a concrete spec with use cases and an implementation approach if you'd like to see the feature. Maybe a good start might be to work on this as an external plugin. I am going to close the issue for now. Feel free to reopen the issue if you are ready to provide a spec in a PR.


In watch mode, adding VirtualHost doesn't install local SSL certificates

In watch mode, adding VirtualHost doesn't generate nor install local SSL certificates.

Sadly, I can't reproduce this one in a pytest integration test :(


In Certificates and Certificate Signing Requests, add hyperlink to CertificateSigningRequest API doc

This is a Feature Request

What would you like to be added

In the document Certificates and Certificate Signing Requests, it would be helpful to link to the CertificateSigningRequest API.
Why is this needed
I was studying for the CKA exam, and was practicing only using the https://kubernetes.io/docs pages to
search for info about CertificateSigningRequests.  The doc  Certificates and Certificate Signing Requests came up readily,
but the CertificateSigningRequest API was not so easily discoverable via a search using only the https://kubernetes.io/docs search interface.
Comments

/triage accepted


Make environment configuration shippable

Currently, during provisioning the VM, all answers on questions are storing in file inside of .vagrant directory, which is completely personal for every particular user.
The idea is to have this YAML file available within project directory with ability to commit it to VCS to automatically inform every team member about environment configuration changing.

Thanks @Mitsuroseba for the idea!

Done in https://github.com/BR0kEN-/cikit/pull/38


A better/more reliable way to set a global cooldown to commands?

I've recently been getting indo Node, and I'm curious if anybody has a better way to set a global cooldown on their bots commands. I feel like what I've written possibly can't be the most efficient way to go. Here's what I got so far.
var onCooldown = false;

client.on("chat", function(channel, user, message, self) {
    if (onCooldown) return;

    else if (self) {
        onCooldown = true;
        setTimeout(function() {
            onCooldown = false;
        }, 5000);
    }
)};

What it does: after every message received, it checks if bool onCooldown is true. If it is, it does nothing. If it isn't, it continues to listen for a self-call, and when it receives one (aka when the bot does something), it sets onCooldown to true for 5 seconds.
I think this is something more people are interested in - flood protection is always good.
What might be a good idea is to make the bot not listen to messages during the 5 seconds it's disabled to improve performance? I don't know, and I certainly wouldn't know how to do it.
Thanks in advance!

Realized this is the wrong place to ask. If anybody still wants to be kind and answer the question, please contact me at @ireliatv on twitter or nikolai#5434 on disc, thanks.

It would be really cool to use this as a sort of queuing system for chat
messages too. Just a thought
Joseph Cellucci
<EMAIL_ADDRESS>On Nov 26, 2017 9:24 PM, "n-theien"<EMAIL_ADDRESS>wrote:

I've recently been getting indo Node, and I'm curious if anybody has a
better way to set a global cooldown on their bots commands. I feel like
what I've written possibly can't be the most efficient way to go. Here's
what I got so far.
var onCooldown = false;
client.on("chat", function(channel, user, message, self) {
if (onCooldown) return;
else if (self) {
    onCooldown = true;
    setTimeout(function() {
        onCooldown = false;
    }, 5000);
}

)};
What it does: after every message received, it checks if bool onCooldown
is true. If it is, it does nothing. If it isn't, it continues to listen for
a self-call, and when it receives one (aka when the bot does something), it
sets onCooldown to true for 5 seconds.
I think this is something more people are interested in - flood protection
is always good.
What might be a good idea is to make the bot not listen to messages
during the 5 seconds it's disabled to improve performance? I don't know,
and I certainly wouldn't know how to do it.
Thanks in advance!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tmijs/tmi.js/issues/265, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJyd1e6DzTFeQWE8br6NhQl1ntr6KDJWks5s6itugaJpZM4QrE0Z
.

@celluj34 I agree - it could be implemented to tmi.js itself, to simplify everything.
It could be as simple as a setCooldown(client.say, x) or (client.on, x), which stops the client from sending messages and stops client from recieving messages respectively.


Default values in owner ,RepoUrlPicker like workspace ,project and repository

Below is my template.yaml file where I am using Repourlpicker
owner:
title: Owner
type: string
description: Owner of the project
ui:field: OwnerPicker
ui:options:
allowedKinds:
- Group
- title: Choose a location
required:
- repoUrl
properties:
repoUrl:
title: Repository Location
type: string
ui:field: RepoUrlPicker
ui:options:
allowedHosts:
- bitbucket.org
I need to make default value to my workspace ,project and repository so that it doesnt need to be filled again and again in backstage.
Can someone help me in giving default value to workspace ,project and repository.

+1 to @benjdlambert we are also doing this in one of our template

I'm also facing issues like this, additionally i'm wondering if you cant already display a combobox with the current available options for organizations and owners.

You can provide allowedOwners or allowedOrganizations in ui:options too. There's some documentation here: https://backstage.io/docs/features/software-templates/writing-templates#the-repository-picker

Sounds like this is all covered already 👍

You can provide allowedOwners or allowedOrganizations in ui:options too. There's some documentation here: https://backstage.io/docs/features/software-templates/writing-templates#the-repository-picker

Even though the documentation states that we can set the allowed Organizations/Owners for that functionality I'm seeing a slightly different behavior.  First, the allowedOrganizations doesn't seem to work, even though I have it set up when rendering in backstage it always renders a text box without any predefined option, and secondly, the allowedOwners field seems to work even though it blocks the first owner as the only possible option. Let me know if you need any further information on my configuration to understand what's going on.

@luk-rs :wave:

First, the allowedOrganizations doesn't seem to work, even though I have it set up when rendering in backstage it always renders a text box without any predefined option

allowedOrganizations are only available on Azure, and was only added recently, so make sure that both of these are satisfied.

allowedOwners field seems to work even though it blocks the first owner as the only possible option

Are you providing more than one and it's still disabled and not allowing you to choose another one from the list that you've provided? Can you provide an simple reproduction template example?

Yeah, this is something we've spoken about and want to do. I started some work on this already, basically having an integrations-backend that would optionally take a userToken from the client and would be able to list teams and groups you had write access to for each provider. Should probably create a ticket for this to be honest as I don't think it's ticket yet, but it's on my radar! :pray:

@Rugvip 👋  I think this should be reopened as there is no way right now to pass a default workspace
For example, I have
https://gitlab.com/iptiq where iptiq is the default workspace, how can I setup this up now with the RepoUrlPicker? I have to input this always manual like the prefix of the owner which is not convenient

@artemptushkin have you tried following the above comment? https://github.com/backstage/backstage/issues/13665#issuecomment-1250807363

@benjdlambert I tried in default:

gitlab.com/my-space
gitlab.com?workspace=my-space
gitlab.com?project=my-space

neither works, it adds this things to the URL on the UI but it doesn't use - I get 404 later

Adding support for this


Generated types are broken

Looks like too old typescript version or old react types generates types incompatible with modern react.
Here NamedExoticComponent produces any with noLibCheck
https://unpkg.com/browse/<EMAIL_ADDRESS>I use<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Alright, I see NamedExoticComponent in latest react types.
But it doesn't work because of default import import React from 'react';
Not recommended by react team anymore. Better use import * as React from 'react';.

Another negative point of default react import is not manglable React.createElement(). Namespace import would be expanded by rollup (tsdx) into import { createElement } from 'react' and each call could be minified.

Oh, I got it now. esModuleInterop is required. Thought it doesn't affect types. But anyway named exports rock.

Hey @TrySound, thanks for the detailed report 👍  I'm planning to update the development and build tooling of this project in the coming months, will tackle this issue at the same time when I start working on this.
From the sounds of it, this isn't a blocker for you at the moment since there is a workaround.


NIFI-6569, NIFI-6570: Fixed bug that caused read timeouts not to occu…

…r with site-to-site. Fixed bug that caused site-to-site listener not to accept connections if there are no input/output ports on the root group - this used to be a valid check and was done to prevent spawning extra threads and doing extra work if no ports exist but now that we have site-to-site ports outside of the root group it's no longer a reasonable condition to check.
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Reviewing...

@markap14 Thanks for finding and fixing the issues! Change LGTM +1. Merging to master.


Priam on new ASG instance tells Cassandra to gossip with dead ASG EC2 instance

Setup:

running cassandra 3.11 and Priam 3.11
ASG with 3 nodes, one in each AZ 1a, 1b, 1c
Manually terminated the node in 1c via the AWS console.
New node comes up shortly after. Priam detects the old 1c node is dead in its log.
is_replace_token is true and the IP of the dead node is returned in get_replaced_ip
Old node marked as '-dead' in aws sdb and DN in nodetool status
New Node starts Priam, TRIES to start Cassandra, but keeps telling Cassandra that the old 1c node is still in gossip. Cassandra cannot connect to the downed node, and aborts on startup

Fix:

a service tomcat8 restart on the new node fixes the problem
on the restart, is_replace_token returns: false so no IP is replaced so no gossip with dead nodes occurs
upon restart, nodetool status on the other nodes replaces the 'DN' node with the new node

Questions:

Why is Cassandra not able to replace the dead node ?
Why on the Priam restart, is Cassandra able to restart successfully ignoring the dead node?

I think that the protocol used by priam is incorrect:
If is_replace = true, and it's attempting to replace a downed node - that node might be unavailable altogether.
Cassandra, when started with in replace mode, attempts to talk to the downed node and fails whenever the node doesn't exist in gossip. Hence the replace can never happen without manual intervention.
@arunagrawal84  thx for helping out in the past, could you comment on this?

@amr46 can you please confirm if other 2 nodes (in other AZ's), are marked as seed nodes as well?

I will have to replicate the environment and get back to you ASAP in the week of 9/3 @arunagrawal84


Prevent @apply from adding !important.

This PR addresses #299 and prevents @apply from adding !important to the output when you have using {options: {important: true}}.

Oh NOOOOOOOOOO, please not.
I love !important. Specitifity is still higher if i want to overwrite (and using !important.
This would be a huge BC-Break for me.
I KNOW WHAT I @apply

@psren Can you elaborate on use cases where you are using @apply to apply !important declarations? I can't think of any situation where I'd want to do that; I would always want to be able to override what's in my components with utilities.

Ah sorry @adamwathan
I just read the testcase and I have misunderstood hat.

@psren Hah! This all about being able to use !important :)

Merging but have a couple additional things to add related to this before tagging!


An ordinary operation causes the function SetReadDeadline to be invalid

What version of Go are you using (go version)?

go version go1.16.5 darwin/amd64

Does this issue reproduce with the latest release?
I don't know...
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
https://play.golang.org/p/fqZEVw7akq1
What did you expect to see?
I just thought about setting up the read buffer, and later found that "SetReadBuffer" was not as large as I designed, so I wanted to pass  syscall.GetsockoptInt(int(f.Fd()), syscall.SOL_SOCKET, syscall. SO_RCVBUF) Look at the actual value, but I found that after calling this function, the read timeout period set will no longer take effect
What did you see instead?
ReadFromUDP will never time out

Duplicate of #28165


a tentative hook for technoweenie to fill in secret endpoints for rebuilding pages

from conversation in irc. it's a service hook that will trigger pages builds.

What payload do you think is needed?  The builds action doesn't take anything really.

I just assumed the default payload would be passed along, but not actually used. You're right, no payload is really necessary. If it's hitting what will be a public API then whatever is necessary to make the build.


[Bug]: Pipeline steps pods report "permission denied" errors to "/.docker"

Is there an existing issue for this?

[X] I have searched the existing issues

Deploy type
Standalone DSPO (without ODH)
Version
v1.31
Environment

OCP Pipelines Version: 1.11
OCP Version: 4.13

Current Behavior
Pipelines step pods show warning messages:
2023/09/05 14:51:24 warning: unsuccessful cred copy: ".docker" from "/tekton/creds" to "/": unable to create destination directory: mkdir /.docker: permission denied

This is likely an openshift gotcha due to images being run with arbitrary ids. It doesn't seem to affect the workflow behavior/logic, but it is not the great seeing this in every log messages. We also should understand the full implications of this warning and if it's something we need to be concerned about.
This is kind of related to: https://github.com/opendatahub-io/data-science-pipelines-operator/issues/198
Where the question of should we support "any images may be relevant.
Expected Behavior
We expect these warning messages to not show.
Steps To Reproduce

Deploy DSPO/DSPA
Deploy flip coin
Inspect pipeline pod logs, they all should have this message

Workaround (if any)
Use pipeline images that give arbitrary ids permission to .docker (by addhing this directory to grp 0)
Anything else
No response

Migrated to Jira: https://issues.redhat.com/browse/RHOAIENG-1655


using flex:1 for View in Content does not have any impact.

react-native, react and native-base version
"native-base": "^0.5.19",
"react": "~15.4.0-rc.4",
"react-native": "0.38.0",

Expected behaviour
I want to divide  element and share among two Views. I can obtain the expected behaviour for example with:
            <View style={{flex: 1}}>
                <View style={{ flex:1, backgroundColor: 'yellow' }}>
                    <Text>window 1</Text>
                </View>
                <View style={{ flex:1, backgroundColor: 'green' }}>
                    <Text>window 2</Text>
                </View>
            </View>

this works as expected. But I want to use <Container> and <Content> elements for this component. I am trying the following:
<Container><Content>
                <View style={{ flex:1, backgroundColor: 'yellow' }}>
                    <Text>window 1</Text>
                </View>
                <View style={{ flex:1, backgroundColor: 'green' }}>
                    <Text>window 2</Text>
                </View>
</Content></Container>

Actual behaviour
With <Container> and <Content> elements, using flex: 1 has no impact. The upper view with yellow background is just shown as one liner encapsulating the <Text> element.
Steps to reproduce (code snippet or screenshot)
<Container><Content>
                <View style={{ flex:1, backgroundColor: 'yellow' }}>
                    <Text>window 1</Text>
                </View>
                <View style={{ flex:1, backgroundColor: 'green' }}>
                    <Text>window 2</Text>
                </View>
</Content></Container>

Is the bug present in both ios and android or in any one of them?
any of them.

fixed it by doing:
<Content style={{ flex: 1 }} contentContainerStyle={{ flex: 1 }}>

when I use flex in contentContainerStyle I keep getting an error saying 'you attempted to set the key padding with the value 0 on an object that is meant to be immutable and hes been frozen.
This triggers for ex when I push a route or pressing a Fab button, but only when setting flex: 1 on Content, did you encounter this issue?

I didn't get any such error.

I'm also seeing this issue. @eduedix workaround worked for me. Why the need for the workaround though?

The same problem happens to Tabs and Tab.
How to apply this workaround to Tabs and Tab?
Following code doesn't work.
<Container>
	<Headers></Headers>
 	<Tabs>
 		<Tab contentContainerStyle={{ flex: 1 }}>

            <View style={{ flex:1, backgroundColor: 'yellow' }}>
                <Text>window 1</Text>
            </View>

    	</Tab>
	</Tabs>
</Container>

@keslert That's because Content is wrapped around ScrollView and we need to add styles to contentContainerStyle for that.

This should be mentioned in the doc.


Improve docs for backend development

This expands the documentation, particularly the part about developing for the server.
Further improvements could be:

how to deal with oauth during local development?

creating owner-only credentials or is there a better way?

a bit more about Prisma: npx prisma studio, npx prisma migrate dev, ...

Closing this PR as we are deprecating the tool and archiving the repository.


Add RekognitionClient support

import {mockClient} from 'aws-sdk-client-mock';
import {RekognitionClient} from '@aws-sdk/client-rekognition';

const rekognitionMock = mockClient(RekognitionClient);

Error:
TS2345: Argument of type 'typeof RekognitionClient' is not assignable to parameter of type 'InstanceOrClassType<Client<ServiceInputTypes, MetadataBearer, any>>'.
Type 'typeof RekognitionClient' is not assignable to type 'ClassType<Client<ServiceInputTypes, MetadataBearer, any>>'. The types of 'prototype.middlewareStack.concat' are incompatible between these types.
Type '<InputType extends ServiceInputTypes, OutputType extends ServiceOutputTypes>(from: MiddlewareStack<InputType, OutputType>) => MiddlewareStack<...>' is not assignable to type '<InputType extends ServiceInputTypes, OutputType extends MetadataBearer>(from: MiddlewareStack<InputType, OutputType>) => MiddlewareStack<InputType, OutputType>'.
Types of parameters 'from' and 'from' are incompatible.
Type 'MiddlewareStack<InputType, OutputType>' is not assignable to type 'MiddlewareStack<InputType, ServiceOutputTypes>'.
Types of property 'addRelativeTo' are incompatible.
Type '(middleware: MiddlewareType<InputType, OutputType>, options: RelativeMiddlewareOptions) => void' is not assignable to type '(middleware: MiddlewareType<InputType, ServiceOutputTypes>, options: RelativeMiddlewareOptions) => void'.
Types of parameters 'middleware' and 'middleware' are incompatible.
Type 'MiddlewareType<InputType, ServiceOutputTypes>' is not assignable to type 'MiddlewareType<InputType, OutputType>'.
Type 'InitializeMiddleware<InputType, ServiceOutputTypes>' is not assignable to type 'MiddlewareType<InputType, OutputType>'.
Type 'InitializeMiddleware<InputType, ServiceOutputTypes>' is not assignable to type 'InitializeMiddleware<InputType, OutputType>'. Call signature return types 'InitializeHandler<InputType, ServiceOutputTypes>' and 'InitializeHandler<InputType, OutputType>' are incompatible.
Type 'Promise<InitializeHandlerOutput<ServiceOutputTypes>>' is not assignable to type 'Promise<InitializeHandlerOutput<OutputType>>'.
Type 'InitializeHandlerOutput<ServiceOutputTypes>' is not assignable to type 'InitializeHandlerOutput<OutputType>'.
Types of property 'output' are incompatible.
Type 'ServiceOutputTypes' is not assignable to type 'OutputType'. 'ServiceOutputTypes' is assignable to the constraint of type 'OutputType', but 'OutputType' could be instantiated with a different subtype of constraint 'MetadataBearer'.
Type 'CompareFacesCommandOutput' is not assignable to type 'OutputType'. 'CompareFacesCommandOutput' is assignable to the constraint of type 'OutputType', but 'OutputType' could be instantiated with a different subtype of constraint 'MetadataBearer'.

All the Clients are supported. Please see https://github.com/m-radzikowski/aws-sdk-client-mock#mixed-aws-sdktypes-versions and reopen the issue providing a reproducible example if that doesn't help.


TrajectoryExecutors should check that the trajectory starts at the right place

If the start of the trajectory doesn't match the current configuration, we should throw an exception.
I think RosTrajectoryExecutor already does this somehow (although I think the error might be raised by the action server).

This is important for simulated trajectory executors, e.g. the Instantaneous and Kinematic trajectory executors.
For RosTrajectoryExecutor, I think it's actually preferable to keep the logic in the action server. We need a check in the action server anyway for safety, since it is possible to command trajectory execution via ROS without going through the Aikido interface. Duplicating the check doesn't provie any more safety and gives us another set of threshold parameters to set. 😰


Add a 'period' variable to ModelCheckpoint to save every period epochs

Discussed in issue #4589 . Went with 'period' since I got no final sanction either way. Can change it to frequency if preferred.

LGTM


Typo "typescript"→"TypeScript"

https://docs.microsoft.com/en-us/azure/devops/extend/develop/add-build-task?view=azure-devops
https://github.com/MicrosoftDocs/azure-devops-docs/blob/main/docs/extend/develop/add-build-task.md
#PingMSFTDocs

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

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


consistent include / header file layout

Related to the point raised by @theopolis in #92: The include directives in some of the TSS code, namely the common test code, references headers in a way that requires that their location be on the gcc search path.
With the tpm2.0-tools referencing this code directly (not how they were intended to be used I imagine, but the code is useful so it's being used elsewhere) we run into a situation where the the tpm2.0-tools build must either use the headers in the TPM2.0-TSS repo (the current approach) or it will need to be told where to search for them on the build host. The later is preferable as it prevents us from referencing headers and compiled libraries that may be out of sync though it requires that the tpm2.0-tools know where to find the installed headers.
Neither of these two approaches are ideal though. The "right way to do it" is a bit more invasive and requires two things:

modify the code layout a bit in the TPM2.0-TSS repo so that all headers that are installed reside under $(srcdir)/include/ and in the same relative location when they're installed.
Update the common code to include headers relative to $(PREFIX)/include instead of $(PREFIX)/include/tpm2(sapi|tcti) like they do now. Using https://github.com/01org/TPM2.0-TSS/blob/master/test/common/sample/CatSizedByteBuffer.c#L28 as an example this would require changing #include <tpm20.h> to #include <tpm2sapi/tpm20.h>

The most disruptive change will be the first as it will effect both the Linux build as well as the Windows build. I doubt the changes will be significant but they will need to be coordinated. I'll put together a PR in the next few days to make this more clear.

These changes were trivial but I've held up the PR till I get my Windows system back from the IT shop for testing. If you're interested the Linux bits are up on my github mirror. I'll get the windows build tested when I get my laptop back.

Hey @flihp, I pulled your branch on my development machines. Looks and builds great!

thx for the extra set of eyes on this @theopolis. After being locked in mortal combat with Visual Studio for a day I've managed to update the VS build files as well. PR is #99
Also turns out that some of these headers shouldn't be installed in the first place (whoops!). Sorting that out today.

merged, missed trigger in commit message


Turn ON full monitoring experience for Java Function Apps by default

We recently did a GA release for Application Insights Distributed tracing support for Java Function Apps. Currently this is an opt-in feature for users. We’d like to turn this feature On-by-default for all new Java Function Apps that get created.
Business Impact: This will make it very easy for customers to observe their Functions and can benefit from day zero observability. Customers already have basic monitoring provided by the function host. With the Java agent they will gain full APM visibility with auto-collection and correlation of dependencies, logs, and metrics from popular libraries and Azure SDKs. We would like to have Java worker monitoring to be "on by default" for non-consumption SKUs as there is no cold start time implications for these SKUs. We need Java worker monitoring to be enabled regardless of how the function is created (i.e., AZ CLI, VSCode, portal, etc.).
Status: Azure portal will have this feature on-by-default in the next sprint.
Ask: We need this feature to be on-by-default on VSCode as well.

cc @abinetabate @ahmelsayed @nturinski

[like]  Abinet Abate reacted to your message:

From: Madhura Bharadwaj @.>
Sent: Wednesday, August 2, 2023 2:48:29 AM
To: microsoft/vscode-azurefunctions @.>
Cc: Mention @.***>
Subject: Re: [microsoft/vscode-azurefunctions] Turn ON full monitoring experience for Java Function Apps by default (Issue #3766)
cc @abinetabatehttps://github.com/abinetabate @ahmelsayedhttps://github.com/ahmelsayed @nturinskihttps://github.com/nturinski
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-azurefunctions/issues/3766#issuecomment-1661408195 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYLXLHOT7QZUBYHZ5Z5PZG3XTG5X5BFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVEYTANBUGAYDONRVQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRRHAYTENZXHE3DEMVHORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This has been released in v1.13.2 of the extension.


Fix URL Path for Booklist Swap in Dropdown Menu

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #3828
Description

Corrected the incorrect URL path for the Booklist for Swapping section in the "More" dropdown menu.
Users can now smoothly navigate to the Booklist Swap page without encountering broken links.
Ensured that the menu interaction remains consistent, with no disruption to the user experience.
This PR resolves the broken link for the Booklist Swap page, improving navigation within the dropdown menu.

Type of PR

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

Screenshots / videos (if applicable)
BEFORE
https://github.com/user-attachments/assets/472b2bc8-5f14-461d-8d37-189587b49928
AFTER
https://github.com/user-attachments/assets/b39f8003-6c44-4c10-98fc-3d11398474ff
Checklist:

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

resolve conflicts @SrijaVuppala295

@sailaja-adapa , please review it , its resolved !!!

create new PR @SrijaVuppala295


Tshradheya portfolio

Coverage remained the same at 90.497% when pulling f940ea2948bede4ab25f92d475d2cb8c2e073d73 on tshradheya:tshradheyaPortfolio into 93245c873ca49b789cb8f9d90cc392a7c2b986b8 on CS2103AUG2017-W14-B1:master.


Adds quoting to local version. Related to #15.

Checks local version arguments and quotes them if necessary if specified on command line.

Codecov Report

Merging #17 into master will decrease coverage by 5.54%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
- Coverage    37.3%   31.76%   -5.55%
==========================================
  Files          14       14
  Lines         890      891       +1
  Branches      121      121
==========================================
- Hits          332      283      -49
- Misses        529      599      +70
+ Partials       29        9      -20

Impacted Files
Coverage Δ

main.py
0% <0%> (ø)
:arrow_up:

modules/cdGetBing.py
13.51% <0%> (-29.73%)
:arrow_down:

modules/cdGetBitly.py
11.82% <0%> (-29.04%)
: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 f74dd8b...4c3bfaa. Read the comment docs.

This seem unnecessary to me. The quoting needs to be done when the argument is supplied on the terminal (when necessary) by the user so that intended argument reaches to the script. You cannot control what is being supplied on the terminal from the script because the script does not take effect until the shell processes the command.

On a second though, it does make sense because the script is making a system call from inside (there are certainly better ways to this though). My previous comment was based on an other assumption that I thought you were trying to solve.


validator error format refine

What is the issue about:
Validator should have a refined error format.
What happened:
Currently, error returned from validator.v9 is not clearly enough.
What you expected to happen:
A well formatted error for nirvana.
Anything else we need to know:

Can you give an example of what works and whatnot?  It seem they do process errors.

@walktall ping

duplicate with #126


Latest changes for multitree

Latest changes for https://github.com/paritytech/polkadot-sdk/pull/3386

@cheme  Could you take a look at the recent changes as well?

Following this, we should remove Extensions nodes and probably support for no_std

Changes

@cheme Could you take a look at the recent changes as well?

looks good
for no_std removal, I guess substrate approach should be followed (remove rstd but and just use alloc or core imports instead), I don't remember but maybe I added this to cumulus, but generally being able to add use this code in runtime sounds relevant to me.


WIP: Reconsider tool excludes

Use https://flake8.pycqa.org/en/latest/user/options.html#cmdoption-flake8-extend-exclude instead?


Allow colon in e.g. package fields script

I often do
"scripts": {
    "test:once": "...",
    "test:watch": "...",
  }

in my package script. Which are never completed properly.
joma@edison:~/entwicklung/nodews/angular-ultimate-fundamentals$ yarn run
build       code_frmt   dev         env         test:once   test:watch
joma@edison:~/entwicklung/nodews/angular-ultimate-fundamentals$ yarn run test:
config/       dist/         .git/         node_modules/ src/          types/        .vscode/

So i investigated that colon is a special, see http://tiswww.case.edu/php/chet/bash/FAQ  on the E13 case. Looked through your script and saw COMP_WORDBREAKS="\"'><=;|&(: ". Which explicitly includes :  as a completion word break character. Which might be kind of the opposite this issue is for.
However, if you accept a PR is see one viable solutions. If a am not missing why else colon should  explicitly be included as a wordbreak.
Solution A: Add the wordbreak exclusion parameter to the shell completion builtin calls
COMPREPLY=()
        if command -v _init_completion >/dev/null; then
                _init_completion -n :
        else
                if command -v _get_comp_words_by_ref >/dev/null; then
                        _get_comp_words_by_ref -n : cur prev words cword
                fi
        fi
...
__ltrim_colon_completions "$cur"

giving
joma@edison:~/entwicklung/nodews/angular-ultimate-fundamentals$ yarn run
build       code_frmt   dev         env         test:once   test:watch
joma@edison:~/entwicklung/nodews/angular-ultimate-fundamentals$ yarn run test:
once   watch
joma@edison:~/entwicklung/nodews/angular-ultimate-fundamentals$ yarn run test:once

P.S. I am right now of the opinion that changing COMP_WORDBREAKS as local var in you script never worked.

Hm, just saw that that having patched it, it does not complete in the not-run case
joma@edison:~/entwicklung/nodews/angular-ultimate-fundamentals$ yarn te
team        test:once   test:watch
joma@edison:~/entwicklung/nodews/angular-ultimate-fundamentals$ yarn test:

Hm, i don't get it.

Not sure why you're having these issues and unfortunately, I'm too pressed for time to look into it this week.
One thing I will say though is that I've never had issues with run scripts containing colons completing in any situation (using run, or not). So an easier fix might just be within your own dotfiles.

I confirm that I always had issues with completion for scripts like test:watch where colon is there (just upgraded to the latest version, nothing changed). It's interesting that if I type yarn test and press Tab I see all possible options, but once I have yarn test: typed then pressing Tab has no effect.

What's the output of the following command for you @ikornienko
$ echo "bash: ${BASH_VERSINFO[*]}" ::: "bash-completion: ${BASH_COMPLETION_VERSINFO[*]}"

Also, of note, I just started last night actually slowly putting together a basic test suite for this project. Should be a bit safer and more reliable when I get that finished.
End goal is to also fully support workspaces.

@dsifford first of all, thank you for looking into it!
$ echo "bash: ${BASH_VERSINFO[*]}" ::: "bash-completion: ${BASH_COMPLETION_VERSINFO[*]}"
bash: 4 4 19 1 release x86_64-apple-darwin17.3.0 ::: bash-completion:

I'm not an expert and cannot tell right away why ${BASH_COMPLETION_VERSINFO[*]} is empty. I have bash-completion version 1.3_3 installed via homebrew.
I have another entry point that may (or may not) help. In package.json scripts I have framerx:build and framerx:bootstrap. When I type yarn framerx completion shows options that include both of them, when I type yarn framerx: completion doesn't show any options (same as I described before), but when I type yarn framerx:b it completes it with yarn framerx:build even though I'd expect it to offer me both options, since it's a common prefix for both of them (but it selects build... maybe because it's defined before bootstrap in my package.json).

Noted, thanks for the info.
Can't offer any sound advice until I get a moment to check into this, but one thing I will mention right off the rip is that I'd recommend first checking to be sure you have the correct bash-completion installed from homebrew.
Using bash ^4.x, the correct one is bash-completion@2, not bash-completion.
Other than that, sit tight until maybe next weekend and I should be able to investigate further. :+1:

Thanks for letting me know about bash-completion2, @dsifford, somehow I overlooked it. Ok, now I have
$ echo "bash: ${BASH_VERSINFO[*]}" ::: "bash-completion: ${BASH_COMPLETION_VERSINFO[*]}"
bash: 4 4 23 1 release x86_64-apple-darwin17.5.0 ::: bash-completion: 2 8

Hm, you know, I re-read @joma74's comment which says

Ok, in this case this is happening. _yarn_test: should not be taken as next cmd. Ends in filedir completion which shoud not.

Since I have build dir, I nuked it... and now after typing yarn framerx:b completion doesn't offer anything. So the fact that it completed yarn framerx:build before was misleading, it actually was inserting dir name.

@ikornienko @joma74 Are you both still having issues with this? I have a little bit of time this weekend to maybe look into getting a fix out for this.
@ikornienko if you're still having issues, can you provide me with a minimal package.json to test out?
Thanks

@dsifford thanks for taking your time to look into it! The repo is here https://github.com/ikornienko/yarn-completion-bug (I've put a description of the behavior I observe on my two machines into the README.md file).

Awesome, thanks. I'll chime back with what I find when I get a chance to dig into it.

Fix should be out sometime later this week. Working on a big overhaul that tackles several longstanding issues

@mrmckeb @ikornienko Can you gentlemen give the updated completions a try and let me know if it now works for you? There's a lot more improvements peppered everywhere, but just focus on the colon thing for now.
https://github.com/dsifford/yarn-completion/blob/a1b02af98ac68296b3eaac0db892cf8812a66a0c/yarn-completion.bash

What system are you running? Can you go through the same steps as @ikornienko if you’re on MacOS?

Or better yet, just confirm that you have indeed done the steps outlined in the updated readme (if you're running mac os)

Hi @dsifford, I'm on Windows 10, using WSL. So it's Ubuntu 18.04 with Bash.
I removed all traces of the previous script, and then I ran this:
curl -o /etc/bash_completion.d/yarn-completion.bash https://raw.githubusercontent.com/dsifford/yarn-completion/a1b02af98ac68296b3eaac0db892cf8812a66a0c/yarn-completion.bash

It works the same as before, not better, not worse. I still get no autocomplete once I hit a colon.
I'm running<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Might be a silly question, but after downloading the file to your computer, you did either source it or log out and log back in, correct?

Same for me as for @mrmckeb , nothing really changed after I replaced the old file. And I did it in this way: killed old yarn completion, restarted the terminal and checked that yarn completion stopped working, then got https://raw.githubusercontent.com/dsifford/yarn-completion/master/yarn-completion.bash, restarted the terminal and checked that yarn completion works, but the bug is still there.
P.S. BTW, when I followed README instructions I got
$ curl -o "${BASH_COMPLETION_USER_DIR:-${XDG_DATA_HOME:-$HOME/.local/share/bash-completion}}/completions/yarn" https://raw.githubusercontent.com/dsifford/yarn-completion/master/yarn-completion.bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:03 --:--:--     0Warning: Failed to create the file
Warning: /Users/ikornienko/.local/share/bash-completion/completions/yarn: No
Warning: such file or directory
 12 15278   12  1916    0     0    499      0  0:00:30  0:00:03  0:00:27   500
curl: (23) Failed writing body (0 != 1916)

Which is true, I don't have the corresponding directory. So instead I just put it into /usr/local/share/bash-completion/completions.

Hm, well that's disappointing..
I can't for the life of me figure out why it works for me on both my desktop (Arch Linux) and my Laptop (Macbook Pro) and not for you guys.
I guess the only other thing I can point to is the following lines from the bash-completion README...

Q. Completion goes awry when I try to complete on something that contains a colon.
A. This is actually a 'feature' of bash. bash recognises a colon as starting a new completion token, which is often what you want when completing something like a PATH variable:
export PATH=/bin:/sbin:/usr
Without the special treatment of the colon, the above wouldn't work without programmable completion, so it has long been a feature of the shell.
Unfortunately, you don't want the colon to be treated as a special case when doing something like:
man File::B
Here, the colons make bash think that it's completing a new token that begins with 'B'.
Unfortunately, there's no way to turn this off. The only thing you can do is escape the colons with a backslash.

So I suppose the remedy in your cases would be to just stop using colons and instead use dashes or something.
Sorry I couldn't be more helpful.

@ikornienko @jasonkarns I was under the assumption that you guys had menu-complete enabled this entire time.
I'm able to reproduce this using the default complete method, but why would anybody prefer that over menu-complete!
Just enable menu-complete! That should fix it for you.

Add this to your ~/.inputrc
# Cycle through completions, rather than dumping all options
TAB:       menu-complete

# (Shift+Tab) Default complete to show all options
"\e[Z":    menu-complete-backward

Thanks for the investigation, @dsifford ! Ok, so it seems like completion after : won't work because of bash-completion's limitation / feature. It does work to some degree if one escapes colon (e.g. yarn test\: and then triggers the completion through TAB). And this limitation holds for both default and menu-complete method (i.e. even with menu-complete yarn test: and pressing TAB won't do anything).
At the same time with menu-complete one can type yarn test, hit TAB and iterate through all the possible options including options with colon, which almost entirely remediates the problem! Thank you for all the research and suggestions!

Yes, thanks for the effort @dsifford!


Desktop basic java

Have you read the Contributing Guidelines ?
YES
Description
Notepad is a word processing program which allows changing of text in computer file, Notepad is a simple text editor for basic text editing which enables computer users to create files.
Fixes #478
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

I have already removed all the unwanted files and folders from the project. The application contains the about us section where the windows image is used. That's why I added the windows image.


Upgrading the view of the page

Anyone can work in it.

**Just give a screen shot of your work
Do pull request
I will merge if accepted**

Thank You

Hi @Soumyajit2825 , I am Najeeba(Front End Developer) and i would like to work on this issue.

Yeah sure go with it.. Share a screen after your work and then do a PR

Anyone can work in it.

**Just give a screen shot of your work
Do pull request
I will merge if accepted**

Thank You

Hey @Soumyajit2825 , I have upgraded view for the page and opened a pull request.
Also I have attached screenshots of my work along with PR.
Please review it.


Missing information in main player HUD (cable ties, firemode, shaped charges)

There are several pieces of information missing from the main player's hud, notably:

Cable tie counter
Weapon firemode indicator
Shaped charges counter

These types of information are not visible to the player, neither on the floating ammo counter, nor the VR belt. This only affects the local player, teammate HUD is not relevant.
This forces the player to have to keep track of these things, which are otherwise clearly indicated in the desktop version of the game.
Seems like an oversight?
There is code for displaying firemode, but it doesn't appear to be working.
This will require adding new HUD elements. Maybe add a mod option to turn them off, if unwanted?

It looks like the firemode indicator was actually deliberately disabled by the developers.
Changing the visibility to true shows that it actually works, but it appears to be unfinished, and it clips into the ammo counter.
In its current state, it would be better to remake it, as it doesn't even support some of the newer firemodes like the Hailstorm Mk 5's volley mode.


Cannot import custom model

Tutorial Select
Prepare Custom Model
Feedback
I am working on OD and YOLOv5 tutorials. Now, I am customizing the YOLOv5 project. When I try to import my custom model using the "Import files" option, I am getting a "0 files imported, 1 failed to import" message. My model has a size of 28 Mb. I am not sure if there is a limitation on size. My questions:

Is it possible to import the custom model using the "Import files" options?
If not, how can I download my custom model to use on my project space? Maybe from Google Drive? How to code to download the model from Google?

@zenetio, models can be hosted on any http server as long as the model is available via a URL and your mobile device can access this URL. For example, I've been uploading my test models as GitHub release assets: https://github.com/raedle/test-some/releases
Uploading models to an Expo Snack is restricted to 25MB. The current recommended way to host models is via an http server (as suggested above)

For quick local development, a simple webserver and localtunnel can work well.
# assuming the files you want to make public live in models/
$ cd models && python -m http.server
$ lt --port 8000

This will give a URL that serves the files in models/ to anyone on the web.

@chrisklaiber , thanks for the tips. I tried to use your suggestion but got a format error and am not sure what is going on.
@raedle , I checked the example you pointed out but got no success because I could not find the correct way to call the URL and pass it to MobileModel.download() function. I am not familiar with Typescript. Anyway, I could manage the issue by uploading the model to my github repo and renaming it to yolov5s.ptl. I am guessing that this name is registered in some place that I could not find where. You can see the screenshot of my inference.
Another point is that I am able to run the app only 3x in sequence and then it crashes with no error message when it is in "Processing..." screen. Is there a way to log an error message and try to find out what is going on?

@zenetio, glad to see that the inference with the custom model produces outputs.

I am guessing that this name is registered in some place that I could not find where.
I am able to run the app only 3x in sequence and then it crashes with no error message when it is in "Processing..." screen.

Would you be able to share an Expo Snack with us to reproduce the two issues?

@raedle Yes, how can I do that? I send you my snack link or do I download the snack project and send you the zipped file?

@zenetio, you can post a link to a Snack here or send it to the email linked in my GitHub profile. If you take the latter route, please notify me here (in case your email lands in the spam filter)


spark.executor.instances par seems invalid when it is large

@mccheah When i set submit pars as below:
--conf spark.driver.memory=3G \
--conf spark.driver.cores=2 \
--conf spark.executor.instances=100 \
--conf spark.executor.memory=3G \
--conf spark.executor.cores=2 \

the spark task has been created successfully,but the problem is that only 50 executors has been produced during the whole life of the spark task(which keeps running all the time, never stopping)
so, i guess whether the spark.executor.instances par will become invalid when it is large?

@duyanghao possibly your cluster doesn't have enough space for 100 executor instances?  I would expect the current version of Spark to launch 100 executors and block until they're all ready, then begin launching tasks on the executors.

@ash211 I can make sure there are enough resources for this task to run!
Have you tested the scenario for more than spark.executors?

I haven't personally tested >100 executors but am currently unaware of any limits that would cause this to fail.
Are the pods not getting submitted and visible in kubectl, or are they not getting scheduled (stuck in pending state) ?

@ash211 another 50 pods are not visible  in kubectl as below:
kubectl get pods -n=xxx -a|grep app-kafka-streaming-1499486419238296266|wc -l
50

Can you please attach the logs from the spark-submit call in a .txt file here or on a pastebin-style website?

seen from the k8s log,there are only created records of app-kafka-streaming-1499486419238296266-(1-50) pods, nothing about 51-100 executor pods appears.

@mccheah It's strange that another task test show that it could produce 100 executors,maybe there is some unknown internal adjustments of driver for producing how many executors to run spark task.
So,i guess the driver doesn't decide produced executor nums only by par spark.executor.instances .

@duyanghao the spark driver log output will be very informative for localizing what is happening

@erikerlandson there are the last few lines for Requesting a new executor in driver logs:
[2017-08-10 12:52:14,077] INFO Requesting a new executor, total executors is now 42 (org.apache.spark.scheduler.cluster.kubernetes.Kube
rnetesClusterSchedulerBackend)
[2017-08-10 12:52:14,088] INFO Requesting a new executor, total executors is now 43 (org.apache.spark.scheduler.cluster.kubernetes.Kube
rnetesClusterSchedulerBackend)
[2017-08-10 12:52:14,097] INFO Requesting a new executor, total executors is now 44 (org.apache.spark.scheduler.cluster.kubernetes.Kube
rnetesClusterSchedulerBackend)
[2017-08-10 12:52:14,106] INFO Requesting a new executor, total executors is now 45 (org.apache.spark.scheduler.cluster.kubernetes.Kube
rnetesClusterSchedulerBackend)

@ash211 @mccheah @erikerlandson Can any one give some suggestion?

@duyanghao spark's dynamic allocation logic will never request more executors than there are data partitions.  So one possibility is; if there are 50 partitions in the data, then that will be the maximum the driver requests.

@erikerlandson i am not enabling spark's dynamic allocation.

i will close this issue since at the moment there is no better way to handle this situation than by avoiding it.


Unsure how to access cube transformation after grab

I am trying to use grabbable on an Actor and running into this issue where if I try to access the rotation properties on release, the values are not updated to the new transform.
cube.onGrab('end', () => {
	const c = this.context.actor(cube.id);
	console.log(c.transform.local.position.x);
	console.log(c.transform.local.position.y);
	console.log(c.transform.local.position.z);
	console.log(c.transform.local.rotation.w);
});

The printed values are all 0 even though the actor has been moved around.

Local transforms during grab are always 0, since grab is executed in app space. Try checking c.transform.app.position. If that's also incorrect, maybe you need to subscribe to transform updates? Though I don't think that's necessary.

Yes, trying c.transform.app.position also returns 0 for all values.
Is there documentation for how to subscribe to transform updates?

https://microsoft.github.io/mixed-reality-extension-sdk/classes/actor.html#subscribe

Ah, thank you very much! This worked, and it worked for retrieving both local and app properties.
The line I had to add was:
cube.subscribe("transform");


Upgrade stex, enable CORS restrictions

upgrades to latest stex master
re-shrinkwraps using latest npm-shrinkwrap

:+1:

Coverage remained the same at 94.02% when pulling 8f855ac2a71a2bdc0f013fefd621e8cd37a94dd3 on upgrade-stex into c9acf4661d62f4116e857e0899ff6b29d3be5896 on master.


(JS-0050) Avoid use of == and !=

Description
It is considered good practice to use the type-safe equality operators === and !== instead of their regular counterparts == and !=. If one of those occurs in an innocent-looking statement such as a == b the actual problem is very difficult to spot.
Occurrences
There are 5 occurrences of this issue in the repository.
See all occurrences on DeepSource → deepsource.io/gh/appsmithorg/appsmith/issue/JS-0050/occurrences/

If it is alright, I'd like to take this up.

can I pick this issue?

Hey👋🏼 @yashraut362 ,  thanks for showing interest in this issue.
Unfortunately, this issue is already taken. Please do check our other Good First issues
Looking forward to seeing your contributions :)

Umm, I see no actions on this issue from about a month, so if possible can I be assigned too for issue? @Rishabh-Rathod

Verified as part fo regression testing


M-RET in org mode no longer Insert a new heading or item

Observed behavior
M-RET in org mode no longer Insert a new heading or item.
Expected behavior
M-RET should be bound to org-meta-return
Steps to reproduce
Press M-RET in a [ ] list.

It should behave the same from normal mode. I'll look into a fix for this soon.

As of b461c99, command+RET should behave correctly and consistently across normal and insert mode in org-mode. Please update and let me know if this is what you expected.

Fixed. Thanks for your awesome work!


Problem with the async migrations

Environment

Elixir version (elixir -v):

Erlang/OTP 21 [erts-10.0] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [hipe]

Elixir 1.8.1 (compiled with Erlang/OTP 20)

Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): PostgreSQL 9.6
Ecto version (mix deps):

* ecto (Hex package) (mix)
  locked at 3.0.1 (ecto) a26605ee
* ecto_sql (Hex package) (mix)
  locked at 3.0.0 (ecto_sql) 8d188337

Database adapter and version (mix deps):

* postgrex (Hex package) (mix)
  locked at 0.14.0 (postgrex) f3d6ffea

Operating system: MacOS 10.14.3

Current behavior
So, we are trying to support ecto 3 on triplex (PR https://github.com/ateliware/triplex/pull/61), and we stumbled in a problem: we cannot run a CREATE SCHEMA "a" and the migrations for the prefix "a" inside the same transaction, because now the migrations are ran inside a Task.async, so the db connection will not be the same and then we get a ** (Postgrex.Error) ERROR 3F000 (invalid_schema_name) schema "a" does not exist for the migrations.
A sample code would be:
Repo.transaction fn ->
  Ecto.Adapters.SQL.query!(Repo, "CREATE SCHEMA \"a\"", [])
  Ecto.Migrator.run(Repo, :up, all: true, prefix: "a")
end

And the error you will get on ecto 3 is:
** (Postgrex.Error) ERROR 3F000 (invalid_schema_name) schema "a" does not exist
    (ecto_sql) lib/ecto/adapters/sql.ex:590: Ecto.Adapters.SQL.raise_sql_call_error/1
    (elixir) lib/enum.ex:1327: Enum."-map/2-lists^map/1-0-"/2
    (ecto_sql) lib/ecto/adapters/sql.ex:677: Ecto.Adapters.SQL.execute_ddl/4
    (ecto_sql) lib/ecto/migration/runner.ex:272: Ecto.Migration.Runner.log_and_execute_ddl/3
    (ecto_sql) lib/ecto/migration/runner.ex:105: anonymous fn/2 in Ecto.Migration.Runner.flush/0
    (elixir) lib/enum.ex:1940: Enum."-reduce/3-lists^foldl/2-0-"/3
    (ecto_sql) lib/ecto/migration/runner.ex:103: Ecto.Migration.Runner.flush/0
    (stdlib) timer.erl:181: :timer.tc/2

Expected behavior
So with ecto 2 the sample code runs smoothly, and migrates everything, because all the migrations are run inside of the same process of the transaction.
My suggestion for a solution would be an async option for Ecto.Migrator.up/4, Ecto.Migrator.down/4 and Ecto.Migrator.run/3,4, that could be true by default, but if false, it will not run the migrations on a Task.async.
I can work on that if you want.

The tasks are necessary, otherwise the migrations will run into a deadlock now that we lock the migrations table. So you would need to disable the migration lock too. Furthermore, the code is already complex, so I am a bit worried about adding more cases. I wonder if there are other things we could do? We have recently added a before_migration callback, maybe that can be used instead?

The thing about triplex is that every time we create a new tenant (which is a schema on postgres) we run the migrations from a priv/repo/tenant_migrations folder for that schema, otherwise new tenant schemas will not have the tables.
One solution maybe would be changing triplex to not run migrations when creating a new tenant at all, and instead use ecto.load. I would like do that sometime in the future anyway, since running migrations make the tenant creation quite slow and that make's testing it annoying. I will check how could I do that and get back to you if it didn't solve.
Closing for now. Thanks for the quick answer!


Static Drag and Drop PlaceHolder Feature Requesting

As the currently available placeholder is created only the item is dragged.
Where the requested is static and seperate placeholder element where items can be dragged!..
Stackoverflow

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.
(this is a canned response)


Fix filter functionality and performance

This PR fixes #211 and covers #577,
The filter functionality problem (on page > 1) has been solved by going to first page and load resources!
The performance issue has been solved by introducing debounce mechanism on loading resources.
This also covers Filter profiles!

in general fixed - just looks a little bit wobbly

Right, but the problem with datepicker in the filter was not covered by the initial issue #211,
Therefore, datepicker mispalcement needs to be fixed in a separate issue.

https://github.com/opencast/opencast-admin-interface/issues/528 here is the issue regarding datepicker UI

Works mostly, but not if I'm using the start date as filter. If I go to page 3 and enter a start date filter, filtering is not working. In the browser console I do get the following error:
MUI X: A component is changing the uncontrolled value of a picker to be controlled.
Elements should not switch from uncontrolled to controlled (or vice versa).
Decide between using a controlled or uncontrolled valuefor the lifetime of the component.
The nature of the state is determined during the first render. It's considered controlled if the value is not `undefined`.
More info: https://fb.me/react-controlled-components

Thanks for the testing and your feedback, I will fix it right away

As an info, this error also happens in the current main branch, therefore, it would say, it is separate issue!

Fix for MUI error is available in #652

Probably makes sense to merge https://github.com/opencast/opencast-admin-interface/pull/654 first

All other concerns seem to have been addressed (or moved), so I'll be merging this.


Error while running training bash command

Hi, I am getting following error while running the training bash command.
I am running
bash scripts/wikitabletext/train_had.sh data/wikitabletext bart.base/
custom_train.py: error: unrecognized arguments: data/wikitabletext/bins --source-lang text --target-lang data --truncate-source
ls: cannot access 'checkpoints//pt': No such file or directory
ls: cannot access 'checkpoints//checkpointbest_*pt': No such file or directory
2023-06-28 12:50:08.829246: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-06-28 12:50:09.695034: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
2023-06-28 12:50:11 | INFO | fairseq.tasks.text_to_speech | Please install tensorboardX: pip install tensorboardX
usage: average_checkpoints.py
[-h]
--inputs
INPUTS
[INPUTS ...]
--output
FILE
[--num-epoch-checkpoints NUM_EPOCH_CHECKPOINTS | --num-update-checkpoints NUM_UPDATE_CHECKPOINTS]
[--checkpoint-upper-bound CHECKPOINT_UPPER_BOUND]
average_checkpoints.py: error: argument --inputs: expected at least one argument

Hi, could you confirm your fairseq version is v0.10.2? Also could you check the file checkpoints/log to see if there's more error log in it? If there is, could you also post it here? Thanks!

It seems the argument parsing is incorrect. Since I directly import the parsing function from fairseq, it's not supposed to happen when the environment is correct.
Could you try re-installing fairseq==v0.10.2, or alternatively try downloading the source code, checkout to v0.10.2, and directly use the source code fairseq rather than your installed package?
Also, I'm suspecting your environment is somehow messed up, because v0.10.2 fairseq doesn't have the file distributed/utils.py (as in your log). The file distributed/utils.py is post v0.12; v0.10.2 only has the file distributed_utils.py

Hi, I have done some changes. Now I am getting this error-
Traceback (most recent call last):
File "/workspace/data/text_to_table/custom_train.py", line 463, in
cli_main()
File "/workspace/data/text_to_table/custom_train.py", line 452, in cli_main
parser = options.get_training_parser()
File "/home/gautam8/.local/lib/python3.6/site-packages/fairseq/options.py", line 34, in get_training_parser
parser = get_parser("Trainer", default_task)
File "/home/gautam8/.local/lib/python3.6/site-packages/fairseq/options.py", line 210, in get_parser
utils.import_user_module(usr_args)
File "/home/gautam8/.local/lib/python3.6/site-packages/fairseq/utils.py", line 446, in import_user_module
raise FileNotFoundError(module_path)
FileNotFoundError: /workspace/workspace/data/text_to_table/src
ls: cannot access '/workspace/data/text_to_table/checkpoints//pt': No such file or directory
ls: cannot access '/workspace/data/text_to_table/checkpoints//checkpointbest_*pt': No such file or directory
usage: average_checkpoints.py [-h] --inputs INPUTS [INPUTS ...] --output FILE
[--num-epoch-checkpoints NUM_EPOCH_CHECKPOINTS | --num-update-checkpoints NUM_UPDATE_CHECKPOINTS]
[--checkpoint-upper-bound CHECKPOINT_UPPER_BOUND]
average_checkpoints.py: error: argument --inputs: expected at least one argument
Can you tell me what is inputs here? and what is --user-id in train_had.sh

Hi, the script was trying to import the code under --user-dir argument but cannot find the path. --user-dir should be set as src/. That's where I implemented my custom task and model.
Have you properly set this argument? Are you running the script under the root directory of this repo? If not, you need to set the argument as the absolute path.
I don't see the --user-id argument. Are you referring to --user-dir?

Oh yes, sorry I mean --user-dir.
I am using the DGX2 server, where I need to give an absolute path, that's why I am changing it everywhere.


UI upgrade for ToDo-List App

Can I contribute to this project by upgrading its UI to Material 3?
This app is currently using Material 2 & I Would love to help migrate this app to Material 3.

Please be my guest! good luck :-)

Fixed in https://github.com/n1snt/Simple-ToDo-List-Android/pull/4


AfterUpdate hooks fails for nil uuid pointer

GORM Playground Link
https://github.com/go-gorm/playground/pull/747
Description
We use gorm hooks but we are seeing a problem when a field with type *uuid.UUID is updated with a nil. We expect the value passed to the AfterUpdate() hook to be nil but it is still populated with the original value. Note the database is updated with NULL as expected. It is the go struct in the AfterUpdate() hook that is incorrect.
This was working with gorm version v1.25.1 but is broken in gorm version 1.25.2. Specifically the breakage occurs with the commit 63534145fda9a2ac9ba703650b1a44da6a03e45e
The problem happens using Updates() with a pointer to uuid.UUID
var uuidPtr *uuid.UUID = nil
DB.Model(&p).Updates(map[string]interface{}{"unique_id": uuidPtr}).Error

The problem does not happen in the following scenarios

when a string pointer is used

var stringPtr *string = nil
DB.Model(&p).Updates(map[string]interface{}{"name": stringPtr}).Error

when the uuid.UUID pointer is a structure field

type Product6 struct {
	ID       uint `gorm:"primarykey"`
	Name     *string
	UniqueId *uuid.UUID
}

p = Product6{ID: p.ID, UniqueId: nil}
DB.Model(&p).Updates(map[string]interface{}{"unique_id": p.UniqueId}).Error

when nil interface is used

DB.Model(&p).Updates(map[string]interface{}{"unique_id": nil}).Error

Hey @neiledgar so I checked this out, and upon digging deeper I found that the issue here is that after the completion of Updates(), the model object (&p in your case above) doesn't reflect the new nil value of the *uuid.UUID field, which is why you're unable to see the new nil value in AfterUpdates as well.
So the root cause of this issue is the inconsistency of the model object and not just AfterUpdates. The reason why this is happening is because this condition gets satisfied which simply returns without assigning the nil value, thus leaving the model object with the old value.
I've raised this PR to fix this issue, along with few test cases around the Updates() with map and uuid.UUID. Hope this will be helpful. Thanks.

Hey @omkar-foss. Thank you for the pr/fix. This has fixed the my issue and the playground test now works. I will integrate and test with our system.

Hey @omkar-foss. I can confirm can confirm this fixes my issue having pulled the PR into our system.
Thanks again

@neiledgar Thank you so much for your confirmation that the fix is working well.
The PR is in review now (thanks to @a631807682) and when all concerns are resolved, it may get merged to master soon.


"Migration guide" link broken in Readme

New Issue Checklist
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[x] I have looked at the Documentation
[x] I have read the F.A.Q.
[x] I have filled out this issue template.

Issue Info

Info
Value

Platform
n/a

Platform Version
n/a

SnapKit Version
n/a

Integration Method
n/a

Issue Description
The "Migration guide" link in the readme doesn't lead anywhere. Wanted to look up what has changed between 4.x and 5.x ☺

I'm also having a hard time finding a CHANGELOG. Wanted to check why my UI was broken after I've changed to 5.0


Expose underlying chrome process

Hey all - quick question
I need access to the underlying chrome process that is launched by this tool. Is there a way to access it given the API? Would you be willing to expose it?

@ORESoftware would you mind sharing your scenario?

yeah np, I need to kill the process at will and collect the exit code. Can I get the exit code using this lib as it is?
To be more specific, I am creating a test runner. Imagine the user uses the --bail option. If a test fails, I need to kill the browser immediately.

yeah np, I need to kill the process at will and collect the exit code. Can I get the exit code using this lib as it is?

I guess chrome's exit code will be 0 most of the time (until it crashes?).

To be more specific, I am creating a test runner. Imagine the user uses the --bail option. If a test fails, I need to kill the browser immediately.

There's browser.close() that would close the browser. Would it work for you?

TBH, it helps to have access to the underlying Node.js child process instance, it will never hurt your users.

I'd rather expose less to avoid misuse. For example, killing chrome turned out to be quite a problem, check out #1047 and #615.

ok, np, I will try to work with it as it is, if I have problems will bring them up later


If the ffmpeg module fails, an invalid file entry is inserted and claim insertion is invalid (NaN for height)

Errors on ffmpeg go uncaught.
Uncaught errors create invalid artifacts.

Broke out into https://github.com/lbryio/spee.ch/issues/626


QC Ruleset Validation

Here, we add support for QC rulesets as pipeline inputs and validate them appropriately. The validation is performed via if/then logic in the schema, which I believe is new for our projects but works as expected per tests.

Do we need to add this verification logic to the MetaWorkflowRuns as well? We will probably need to overwrite the rulesets for different runs, i.e. it will be on the run item.


Remove invalid SPIR-V image decoration tests (#22)

The Volatile, Coherent, NonWritable and NonReadable decorations can only be applied to images if they are storage images, which require the Shader capability.
Note that in SPIR-V 1.4 and later, the NonWritable decoration can also be applied to objects in the Private and Function storarge classes (although Private also requires the Shader capability), but this was not covered by the existing test for the NonWritable decoration.
Signed-off-by: Stuart Brady<EMAIL_ADDRESS>

These tests are invalid so removing them seems like the right thing to do.
Do we know why they were added in the first place? Should we (eventually, not in this PR) add modified tests so we don't lose coverage? Or are these tests entirely unnecessary and therefore no action is required beyond removal?

The commit that added the tests is 05a11d8e499aca1efa1b91f07a9da5c74f6d1214 "Synchronise with Khronos-private Gitlab branch", so I can't say much here.
The original commit seems to be 26bd31d46ae66a6ec96cdf47da408ffd6605f902. I won't comment further on it, other than to say that it added these four image tests, and made no other changes, and there does not seem to have been any detailed rationale provided for the tests that I can see.


Custom DR and ER aren't recognized

Even when using the basic types (slashing, piercing, bludgeoning), the module does not recognize them.

Custom types are now recognized and handled properly in v0.6. Closing.


Grant privileges to public_annotations group if annotation is public

On creation for each mode grant privileges to public_annotations group if annotation is public

Implemented in this commit


Homepage

Note: 🔺Homepage modifications

Summary 🔺 Add a homepage banner with styling a functional project carousel

Issue Number 🔺 https://github.com/UofT-VEEP/VEEP-Website/issues/37

New Proposal Figure out how to fit the banner height to the height on any screen. Add css styles for banner and banner element positioning for small/medium screens

Thanks for the hard work. Looks great!


[SuperEditor][SuperTextField] - Show an underline for the composing region

When users add accents to characters, or compose asian language glyphs, the IME operates across one or more nearby glyphs, which is known as the composing region. Typically, text input systems show a thick underline beneath the glyphs/characters in the composing region.
Add an underline for the composing region in SuperEditor and SuperTextField. The underline should be the same color as the text. However, the underline shouldn't be a font underline, it should be a custom painted underline, because it should be thicker than the standard text underline.

@angelosilvestre I just noticed that this is still open. I thought I finished this. Do you know if there's anything remaining for this ticket that you're aware of?


Add license #8

adding licenses according to issue #8.

used same structure as orginal freecodecamp (https://github.com/freeCodeCamp/freeCodeCamp)
did not want to use a copyleft license due to restraints of marketability (verkehrsfreiheit/-fähigkeit)
divided the licenses between software and all other content
overall goal was to give as much freedom as possible since it is opensource, but to have at least credit established

LGTM +1. @42pre, do you want to chime in, in light of your numerous contributions?
Barring any major arguments against, I suggest we just got with this license, which we can always change in the future if the group decides to (but keeping in mind that everything contributed up until a license change will remain under this license).

@DDCreationStudios @lukastillmann I propose we merge this in 24 hours, unless @42pre replies with any objections.

@robeerob
I am writing on our contribution file at the moment. I thought of the same thing. Something like a default merge if one admin approves  after 1 week or so. However, now in this case it's not that urgent. Adding an additional file doesn't block further contribution 🚀

Merged because review timeframe is elapsed.


Meep material database does not have silicon at 1.3um

Meep material database does not have silicon refractive index at 1.3um
when running
import gdsfactory.simulation.gmeep as gm
import gdsfactory.simulation as sim
import gdsfactory as gf
import meep as mp

wavelength_start = 1.26
wavelength_stop = 1.36
sim_settings = dict(wavelength_start=wavelength_start, wavelength_stop=wavelength_stop)

c = gf.components.mmi1x2()
gm.write_sparameters_meep(c, **sim_settings)

we get
ValueError: User specified frequency 0.7633587786259541 is above the Medium's limit, 0.7352941176470588.

@smartalech
@thomasdorch
@SkandanC
@simbilod

Meep currently has four different kinds of Silicon models, none of which cover 1.3 $\mu$ m:
https://github.com/NanoComp/meep/blob/038a2e59866374aabe51d02d2a4de37a8045364c/python/materials.py#L1065-L1067
https://github.com/NanoComp/meep/blob/038a2e59866374aabe51d02d2a4de37a8045364c/python/materials.py#L13-L15
https://github.com/NanoComp/meep/blob/038a2e59866374aabe51d02d2a4de37a8045364c/python/materials.py#L38-L40
https://github.com/NanoComp/meep/blob/038a2e59866374aabe51d02d2a4de37a8045364c/python/materials.py#L55-L57
We can add another model, we just need a Lorentz-drude fit for it.
(I wouldn't call this a bug, but a feature enhancement).

Thank you Alec!


nsp check fails on 404 when gulp-zip is added to package.json

Hi there,
we're using NSP in our CI pipeline and nsp check started failing when we added gulp-zip dev dependency to our package.json. When I remove it, everything passes again.
(+) Debug output: "404 page not found\n"
(+) Error: Got an invalid response from Node Security, please email the above debug output to<EMAIL_ADDRESS>
I've sent an email as well. Is there anything else I can provide to help solve this issue?
We're using latest nsp 2.6.3.

not sure it's related to gulp-zip since other also gets this error. See #155

Thanks for noticing @gemal.
Closing with respect to #155.


Update Supported Databases to include Yugabyte

Yugabyte YSQL is wire-compatible with Postgres. There is no additional setup required to integrate with Prisma (other than changing the .env file to reflect the appropriate database connection URL - which is necessary for Postgres in the first place).
For an example app using Yugabyte & Prisma, view https://github.com/vvkgopalan/yugabyte-prisma2-examples/

We need to discuss internally if we are comfortable with promising full support if we are not running our full test suite on the database. "wire-compatible" is nice in theory, in practice there are a lot of possible gotchas. I will raise this internally.

We had this discussion, and we do not feel comfortable stating different implementations of the same protocol as supported without running our full test suite on them in CI. @janpio created issues about this for Yugabyte and Cockroach.

We have run the tests for both Yugabyte (https://github.com/prisma/prisma-engines/pull/1139) and Cockroach (https://github.com/prisma/prisma-engines/pull/1024) and unfortunately neither appear to work as-is with our Postgres connector. Even though they share a common network protocol with Postgres, the databases themselves differ in how they store and expose the data.
In order for Prisma to be compatible with Yugabyte and Cockroach we'll need to create dedicated connectors to support them. So unfortunately we can't claim we support either database quite yet. We may in the future though, but that's something that will need to be decided to be part of the roadmap.


Works in Chrome, Firefox but broken in Safari

Mac OS v10.13.6
Chrome Version 75.0.3770.90 (Official Build) (64-bit)
Firefox 67.0.2 (64-bit)
Safari Version 12.1.1 (136<IP_ADDRESS>.2)
Safari reports no errors yet it fails to render highlighted code blocks. I've included screen shots from all three browsers:

I no longer have access to Safari 12... can someone else possible look into this and at least provide a bit more of a pointer to where the problem might lay?

Works for me in Safari.

(Safari Version 13.0 (14608.1.49))

Well yeah... I have 13.0.1. :-)  I wonder if we should care about this so much since Safari is a free upgrade (lets say it's an actual issue with 12.x)... I wonder if Safari 13 isn't available on some versions of OS X?

Unfortunately, we don't have the resources to support legacy browsers :(

Safari 13 ships with iOS 13 and macOS 10.15. It’s also available for macOS 10.14.5 and 10.13.6.

At least it goes 3 versions back.
@jeffschwartz If you can identify the issue yourself and submit a PR I'm sure we'd be happy to review and merge it (I assume it's something simple once found), but otherwise we don't have the resources to track this down ourselves.

Awesome sauce.

@yyyc514 Awesome product :)


Clarification on how MJML supports the responsiveness of our product

Hi MJML,
My name is Jon and I'm reaching out on behalf of Bananatag, a customer of MJML, about the responsiveness of our Email Designer, which MJML supports. Can you assist with the below query?
One of our clients noticed that when they view a sent email, that was built using our email designer, on Outlook, the content of the email doesn't respond well with the native Outlook zoom feature. However, when they view a sent email that was created within the native Outlook compose window, the content of the email is able to respond to the resolution of the screen and adjust accordingly.
Can you please offer some insight on why this may be the case? I've attached to this support ticket both versions of the same email as well as two screenshots. The email titled "MJML Responsive Template" was created within the native Outlook compose window and the email titled "MJML Unresponsive Template" was created with our Email Designer.
We really appreciate you looking into this and please let me know if you require any additional information from us!

MJML - Unresponsive Template.txt
MJML - Responsive Template.txt
*I wasn't able to attach the .msg file for both emails so please let me know if there's a better place to send this to you. I also tried sending this as an email to<EMAIL_ADDRESS>but it couldn't be delivered.

then they view a sent email that was created within the native Outlook compose window

You shouldn't send email via Outlook send function as it alter the HTML ( if it's only for testing purpose use somehting as https://putsmail.com/ ).
There's no way to handle the use case of zoom level with Outlook limitation, responsive email have much more limitation than a responsive website.

Hi @iRyusa, thanks for getting back to me!
To confirm, the affected email (MJML - Unresponsive) that was built with the Email Designer (supported by MJML) was not sent via Outlook but from Bananatag directly. However, when viewed on Outlook, the email doesn't respond to the zoom feature, in the same way, a regular email, that was natively composed in Outlook, would.
Does MJML not support Outlook's zoom feature and if so, are there plans to support it in the near future?

Hi @iRyusa, any updates on this? Thanks

There's no way to handle the use case of zoom level with Outlook limitation, responsive email have much more limitation than a responsive website.

I've already reply to this in the 1st reply, try to send it with https://putsmail.com/ to see if the HTML wasn't altered by Bananatag.


Normalize DrugBank interaction types

The reimport of DrugBank has introduced a number of new interaction types. These need to be normalized or, for the ones we want to keep, we need to add a directionality, description, and reference:
"substrate",
"weak inhibitor",
"protector",
"downregulator",
"inactivator",
"degradation",
"oxidizer",
"aggregation inhibitor",
"coating agent",
"dilator",
"deoxidizer",
"diffusing substance",
"nucleotide exchange blocker",
"vesicant",
"inhibition of synthesis",
"stabilization",
"translocation inhibitor",
"gene replacement",
"inhibits downstream inflammation cascades"

The following types do not have any interactions (only ungrouped interaction claims):
"oxidizer",
"coating agent",
"dilator",
"deoxidizer",
"diffusing substance",
"nucleotide exchange blocker",
"vesicant",
"stabilization",
"translocation inhibitor",
"gene replacement"

weak inhibitor, substrate, and downregulator were also used by a number of other sources but are missing their directionality, description, and reference

Here's what I think is the best way to normalize these interaction types
"substrate": 'SUBSTRATE',
"weak inhibitor": 'INHIBITOR',
"protector": 'OTHER',
"downregulator": 'NEGATIVE MODULATOR',
"inactivator": 'INACTIVATOR',
"degradation": CLEAVAGE,
"oxidizer": 'OTHER',
"aggregation inhibitor": 'INHIBITOR',
"coating agent": 'OTHER',
"dilator": 'OTHER',
"deoxidizer": 'OTHER',
"diffusing substance": 'OTHER',
"nucleotide exchange blocker": 'BLOCKER',
"vesicant": 'OTHER',
"inhibition of synthesis": 'INHIBITOR',
"stabilization": 'CHAPERONE',
"translocation inhibitor": 'INHIBITOR',
"gene replacement": 'OTHER',
"inhibits downstream inflammation cascades": 'INHIBITOR'

That sounds good to me and I can work on making this change. It looks like this will add SUBSTRATE as a new interaction type so we need a description, a direction, and a reference for it.

Description: Substrates are drugs which are metabolized by the protein they are interacting with.
Direction: N/A
Reference: FDA - Table of Substrates, Inhibitors and Inducers
If we're not happy with that reference, we can pull pubmed ids

Just realized that INACTIVATOR will also be a new interaction type so we'll need a description, a direction, and a reference for it as well.

Let's normalize INACTIVATOR to INHIBITOR. I think that's a better way than having it be its own interaction type.


New version for sbt 1.0+

Hi,
Please consider releasing a version that supports sbt version 1.0+

Finally, after much frustration, I decided to push my own plugin.
https://github.com/ydubey/sbt-liquibase-plugin
Added:
Support for liquibase-update
Support to specify sql output file
Improve class loading by adding application classpath to plugin


Turn scene on is not possible.

Hi there, great project, that would fix a lot of my problems with scenes and HomeKit.
Unfortunately, I can't turn on a scene with the stateful_scene switch, only to turn it off is possible but the state switches if I'm turning it on via the scene directly.
I get the following error message if I try to turn it on.

I did the default setup for the integration without any errors.
Is this a bug or what did I wrong?
Thanks in advance.

Hi, thank you for your interest. Does this happen to all switches or one in particular?

That happens to all stateful_scene switches.

I think I have identified the issue. It may be related to the naming of your scenes. Stateful Scenes tries to find the scene's entity name by using the name you assigned to it. For example, your scene has the name "Daylight Lounge" then it will try to turn on an entity scene.daylight_lounge. However, this seems to fail when there is a mismatch between the entity_id and its name. I will have a look at how to resolve this.

That's a good point, changed the friendly name previously. I will test it tonight.

That was the problem, and now it's working.
Maybe you can find a solution for it in the future or add this information to the documentation.
Anyways, thank you very much for your quick support and great work.

Resolved in #31


Fix to work with non-text HTTP requests & responses

This PR addresses issue #23.

Thanks, it's really nice of you


Notifiactions are not getting in MI, Oneplus, Lenovo Mobiles

I am developing an android app, In that I have incorporated Push Notifications When some action performed in the web app I send a notification to the registered user(Push is incorporated using Amazon AWS). To configure these push notifications I have used cordova PushPlugin. If the application is in background, then I am getting the notifications. But If I close the app from recent apps try then Notifications are not firing. I want to how Facebook and Whatsapp apps are sending notifications if the app is terminated. I am using cordova and ionic framework to develop the hybrid mobile app. Anyone help is appreciated!

Are you using PushPlugin or phonegap-plugin-push? Those are two very different plugins.

I am using phonegap-plugin-push

@mohanchalla you might be running into this:
https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#huawei-and-xiaomi-phones

@macdonst If I use force-start: 1 then I am getting the notifications in MI mobile but app is loading in the background I don't want to do that. Even I have to tell all the users to enable autostart to my app. Which is some what risky process I have to call each and every one tell. Is there any workaround instead of this?

@mohanchalla nope, sorry. That's the way a lot of Chinese manufacturers of Android phones have decided to setup the Android OS. When you kill the app they also kill the services responsible for receiving push. You could put a first time warning in your app to direct people where to turn on auto-start.

But If I put the force-start option it is effecting even Non-China manufacturers also. Because I will have to keep this option to work on all kinds of mobiles. If I receive a notification Motorola mobiles with force-start option it is staring the app in the background.


Fix typo

People (like me) will think they should add a dot to the end of their label.

Good catch, thanks !


User-Agent header is removed for some requests since 18.0

Example:
17.x:
POST /v7/challenge HTTP/1.1
Host: dauth-lp1.ndas.srv.nintendo.net
User-Agent: libcurl (nnDauth; 16f4553f-9eee-4e39-9b61-59bc7c99b7c8; SDK <IP_ADDRESS>)
Accept: */*
X-Nintendo-PowerState: FA
Content-Length: 17
Content-Type: application/x-www-form-urlencoded

18.0.0 / 18.1.0:
POST /v7/challenge HTTP/1.1
Host: dauth-lp1.ndas.srv.nintendo.net
Accept: */*
Content-Type: application/x-www-form-urlencoded
X-Nintendo-PowerState: FA
Content-Length: 17

The position for Content-Type has also changed.
May need to update NintendoClients to remove the User-Agent for these requests.

Good catch. I'll do some investigation. It looks like they have rewritten their HTTP client.

It seems that only the dauth flow was affected. Other HTTP clients still have the same header order as before. I've pushed a fix. Thanks for notifying me.