Merge tailwind.config.ts

Version
@nuxtjs/tailwindcss: ^6.8.0
nuxt: 3.6.0
Steps to reproduce

Create a projectOne with a tailwind.config.ts
Create a projectTwo that extends projectOne
Create a tailwind.config.ts in projectTwo

What is Expected?
The configs should be merged
What is actually happening?
The second config is overriding the first one
Using the config inside nuxt.config it get's merged as expected.

Hi, this is expected behaviour. projectOne is a layer while projectTwo is your main Nuxt project - that is given precedence and then missing defaults are filled in from projectOne (using defu). Did you expect overriding/reading the other way around? (it could mean that filling values in projectTwo - your main app - would be useless)

Sorry, didn't described the actual behaviour very precisely.

missing defaults are filled in from projectOne (using defu)

That's excactly what I'm expecting to happen. When using the nuxt.config it works and gets merged.
But when I do create a dedicated tailwind.config.ts file those doesn't get merged. Example:
// projectOne/tailwind.config.ts
export default <Partial<Config>>{
   prefix: 'tw-',
   theme: {
     extend: {
       colors: {
         primary: '#4e67aa',
       }
     }
   }
 }

// projectTwo/tailwind.config.ts
export default <Partial<Config>>{
  prefix: 'tw-', // NOTE: already need to re-add the prefix here, as it doesn't get merged from projectOne
  theme: {
    extend: {
      colors: {
        secondary: '#FFC0CB',
      }
    }
  }
}

In this case only projectTwo/tailwind.config.ts will be used. Inside of projectTwo the primary color will be unknown and I also need to re-add the prefix.

I'm unable to replicate this, are you sure you're configuring properly?
Using the repository playground - https://stackblitz.com/github/nuxt-modules/tailwindcss?file=playground%2Ftheme%2Ftailwind.config.cjs
I copy the config of projectOne/tailwind.config.ts into playground/theme/tailwind.config.cjs and the config from projectTwo/tailwind.config.ts into playground/tailwind.config.ts and in the playground viewer, the prefix and primary & secondary colors merge. If that's not the case with yours, would you be able to provide a reproduction?

Thanks for checking!
I haven't been able to replicate it in a clean project. After further debugging it turned out that projectOne included an unresolved config-path what has caused the issue:
export default defineNuxtConfig({
  tailwindcss: {
    configPath: './tailwind.config.ts',
  }
}

After adding a resolver everything works as expected:
import { createResolver } from '@nuxt/kit';
const { resolve } = createResolver(import.meta.url);

export default defineNuxtConfig({
  tailwindcss: {
      configPath: resolve('./tailwind.config.ts'),
  }
}


Slic3r gcode textareas show \n instead of newline

When I use the Slic3r engine, the textareas for the gcode show \n characters instead of actual newlines. This is kind of ugly. If I edit them and add real newlines, it breaks the slicer.
Screenshot showing the problem:

good catch. And I agree it's ugly! Thanks for reporting it @clee


Unknown provider: hSweetAlertProvider <- hSweetAlert

I get unknown provider when I inject as dependency into my controller.
NB: My controller.js and app.js files are separate. Also I'm using with the ionic framework that uses angularJS 1.3
What could be the problem, thanks!

Possible to provide code snippet? A plunker is good to help to locate where the issue is.

I can give you the snippets over here if that's okay, cause it will be hard
to recreate on plunker cause all my files are on local machine e.g ionic
and likewise.
On Friday, 6 March 2015, Howard.Zuo<EMAIL_ADDRESS>wrote:

Reopened #1 https://github.com/leftstick/angular-sweetalert/issues/1.
—
Reply to this email directly or view it on GitHub
https://github.com/leftstick/angular-sweetalert/issues/1#event-247449958
.

OK, code snippet is fine. Let's see how you import those dependencies, and how you used them?

Cool! First thing to note is that I’m using it in Ionic framework which comes bundled with angular 1.3.x or something like that.
In my app.js file I have this
app.js:
angular.module(‘myapp', ['ionic', 'ngCordova’])
.run(function($ionicPlatform, $cordovaToast) {
$ionicPlatform.ready(function() {
// Hide the accessory bar by default (remove this to show the accessory bar above the keyboard
// for form inputs)
if(window.cordova && window.cordova.plugins.Keyboard) {
cordova.plugins.Keyboard.hideKeyboardAccessoryBar(true);
}
if(window.StatusBar) {
// org.apache.cordova.statusbar required
StatusBar.styleDefault();
}
if(window.Connection) {
if(navigator.connection.type == Connection.NONE) {
$cordovaToast.show('Can't connect to network. Please re-try', 'long', 'bottom').then(function(success) {
console.log("The toast was shown");
}, function(error) {
console.log("The toast was not shown due to " + error);
});
}
}
});
})
then I have a controller
popUpCtrl.js:
angular.module(‘myapp').controller('popUpCtrl', ['$scope','hSweetAlert', popUpCtrl]);
function popUpCtrl($scope, sweet) {
'use strict';
// A confirm dialog box to get the GPS location.
$scope.showConfirm = function() {
	sweet.show({
		title: ‘Hello',
		text: ‘Howdy there?',
		type: 'Warning',
		showCancelButton: true,
		confirmButtonColor: '#DD6B55',
		confirmButtonText: 'Yes',
		cancelButtonText: 'No',
		closeOnConfirm: false,
		closeOnCancel: false },

		function(isConfirm) {
			if(isConfirm) {
				sweet.show(‘Hello', ’SweetAlert is nice.', 'success');
			} else {
				sweet.show('Requirement!', ‘You should try using SweetAlertJS.', 'error');
			}
		});
	};
}

On Mar 6, 2015, at 12:14 PM, Howard.Zuo<EMAIL_ADDRESS>wrote:
OK, code snippet is fine. Let's see how you import those dependencies, and how you used them?
—
Reply to this email directly or view it on GitHub https://github.com/leftstick/angular-sweetalert/issues/1#issuecomment-77550289.

You missed dependency of angular-sweetalert.
Something should be modified as following:
angular.module(‘myapp', ['ionic', 'ngCordova’])  =>   angular.module(‘myapp', ['ionic', 'ngCordova’, 'hSweetAlert'])

angular.module(‘myapp').controller('popUpCtrl', ['$scope','hSweetAlert', popUpCtrl]);  =>  angular.module(‘myapp').controller('popUpCtrl', ['$scope','sweet', popUpCtrl]);

Excellent!…… It works fine now, thanks a lot. Noobie mistake I made there with the controller bit. Thanks a lot again. One final thing, I can put an ngClick attribute in the Sweet-Alert.html file right? I need a function to be called when I press the OK button.
Cheers!

On Mar 6, 2015, at 1:04 PM, Howard.Zuo<EMAIL_ADDRESS>wrote:
You missed dependency of angular-sweetalert.
Something should be modified as following:
angular.module(‘myapp', ['ionic', 'ngCordova’])  =>   angular.module(‘myapp', ['ionic', 'ngCordova’, 'hSweetAlert'])
angular.module(‘myapp').controller('popUpCtrl', ['$scope','hSweetAlert', popUpCtrl]);  =>  angular.module(‘myapp').controller('popUpCtrl', ['$scope','sweet', popUpCtrl]);
—
Reply to this email directly or view it on GitHub https://github.com/leftstick/angular-sweetalert/issues/1#issuecomment-77555400.

You mean sweetalert/lib/ Sweet-Alert.html? This file is only used in development phase. You don't need it.

hmmmm! Okay, so how do I call a function when I click on the "Okay" or "Cancel" button?

Check following code:
 sweet.show({
        title: 'Confirm',
        text: 'Delete this file?',
        type: 'warning',
        showCancelButton: true
    }, function(isOk) {
        if (isOk) {
            alert('OK is clicked');
        } else {
            alert('Cancel is clicked');
        }
    });

Thanks!

On Mar 6, 2015, at 11:29 PM, Howard.Zuo<EMAIL_ADDRESS>wrote:
Check following code:
sweet.show({
title: 'Confirm',
text: 'Delete this file?',
type: 'warning',
showCancelButton: true
}, function(isOk) {
if (isOk) {
alert('OK is clicked');
} else {
alert('Cancel is clicked');
}
});
—
Reply to this email directly or view it on GitHub https://github.com/leftstick/angular-sweetalert/issues/1#issuecomment-77655443.


fix: update websockets dependency to be compatible with pytket-quantinuum

pytket-quantinuum updated in version 0.38.1 (October 2024)
See https://github.com/CQCL/pytket-quantinuum/commit/6828486190c3b335756412774468649a8cd02e5d
Our only dependency is qnexus itself, and we only use the imported code in a very "vanilla" fashion, so I've kept >11 for maximum compatibility with other libraries that might be installed alongside qnexus.
What I'm not sure about is whether to just remove the upper bound on the constraint and go with an open-ended >11.0 (similar to pytket-quantinuum which now has >=13.1).  For correctness I've left it in, because websockets might release a breaking change in their version 14.x.  But we could discuss.

the integration test failure isn't this PR, I fix the issue here: https://github.com/CQCL/qnexus/pull/92


[fix] Also auto formatter workflow instance if location is null

In #11535(096fae77) and #11681(4dca488c), we already formatted the workflow definition, but I found out we forgot formatted the workflow instance with the definition's location is null, this patch also formatted the workflow instance.

Codecov Report

Merging #12080 (f657cb4) into dev (cf522e2) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #12080      +/-   ##
============================================
- Coverage     38.74%   38.73%   -0.02%
+ Complexity     4018     4016       -2
============================================
  Files          1004     1004
  Lines         37390    37359      -31
  Branches       4263     4255       -8
============================================
- Hits          14488    14472      -16
+ Misses        21258    21242      -16
- Partials       1644     1645       +1

Impacted Files
Coverage Δ

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
55.12% <0.00%> (-1.29%)
:arrow_down:

...pache/dolphinscheduler/common/utils/DateUtils.java
74.37% <0.00%> (-0.63%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...heduler/api/service/impl/ResourcesServiceImpl.java
51.68% <0.00%> (-0.47%)
:arrow_down:

...eduler/api/service/impl/DataSourceServiceImpl.java
30.06% <0.00%> (ø)

...cheduler/server/master/utils/DependentExecute.java
0.00% <0.00%> (ø)

.../dolphinscheduler/plugin/task/datax/DataxTask.java
0.00% <0.00%> (ø)

...phinscheduler/plugin/task/chunjun/ChunJunTask.java
0.00% <0.00%> (ø)

...cheduler/plugin/task/api/ShellCommandExecutor.java
0.00% <0.00%> (ø)

... and 9 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I've already tested it, LGTM

Thanks


Ccasin/user func spec modified vars fix

Changes how we calculate what system state is modified when using a user function spec.
The old story was that we analyzed the actual implementation of the function to see what state it touched, and treated everything it touched as both an input an an output to the function.  This had a few issues.  Most pressingly, it doesn't work at all for functions where the implementation is unavailable or can't be analyzed.  It also meant we overestimated what state is changed by a function.
The new system is to look at the variables in the user-provided function spec.  Uses of variables corresponding to the initial values of registers/memory mean those are inputs, and uses of variables corresponding to the final values of registers/memory mean those are updated by the function.  This works well enough - the tests pass (I haven't added new tests since it is exercised by all our existing user function spec tests).

For the implementation, there are two hurdles: 1) get the variables, as strings, from the user-provided spec, and 2) find the corresponding Var.ts (since that is what is needed by the function summary code).
For 1: The user specs are parsed by mk_smtlib2_single.  This function already finds the variables we care about, so it seems easiest to just have it return them in addition to the constraint.  So I made a new version of it, mk_smtlib2_single_with_vars that does.  (I changed mk_smtlib2_single to be defined in terms of it, to avoid code duplication).
For 2: It wasn't immediately obvious to me the best way to go from the variable strings to the corresponding Var.ts.  I ended up just getting the set of Var.ts for all registers and memory from the env, and then filtering out anything that's not mentioned in the user spec.
Let me know if there's a better way to do either of these.

For the implementation, there are two hurdles: 1) get the variables, as strings, from the user-provided spec, and 2) find the corresponding Var.ts (since that is what is needed by the function summary code).
For 1: The user specs are parsed by mk_smtlib2_single. This function already finds the variables we care about, so it seems easiest to just have it return them in addition to the constraint. So I made a new version of it, mk_smtlib2_single_with_vars that does. (I changed mk_smtlib2_single to be defined in terms of it, to avoid code duplication).
For 2: It wasn't immediately obvious to me the best way to go from the variable strings to the corresponding Var.ts. I ended up just getting the set of Var.ts for all registers and memory from the env, and then filtering out anything that's not mentioned in the user spec.
Let me know if there's a better way to do either of these.

I can't think of a better solution for those issues. One pressing question I have would be if there is ever a case where the user has a virtual variable in their spec. If there is a case, are we able to handle this?

Your last question is a good one, but I think we're fine.  The way sub_inputs and sub_outputs are calculated is that we start from the set of register/memory names we got out of the environment, and we then keep only the ones that match a name in the user spec.  We are never adding random things from the user spec to this set.


chore: Use group constants for launch template naming

Fixes #N/A
Description
Use v1beta1.Group constants for launch template naming and keys
How was this change tested?
make presubmit
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/launchtemplate/launchtemplate.go
25
83.22%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5333

Relevant Lines:
6483

💛  - Coveralls


python3.pkgs.heisenbridge: Move to python-packages.nix

Motivation for this change
No need for a top-level attribute since this does not provide any binaries.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Maybe we should add a wrapper script in bin or something along those lines?

I'm working on a patch to fix this upstream.

I don't see any problem with sumnerevans's solution.

❯ result/bin/heisenbridge
usage: python3.8 -m heisenbridge [-h] [-v] -c CONFIG [-l LISTEN_ADDRESS] [-p LISTEN_PORT] [-u UID] [-g GID] [-i] [--generate] [--reset] [-o OWNER] [homeserver]
python3.8 -m heisenbridge: error: the following arguments are required: -c/--config

Result of nixpkgs-review pr 124889 at a47ada38 run on x86_64-linux 1

2 packages built successfully:

python38Packages.heisenbridge
python39Packages.heisenbridge


WLValidationError obscures the original error

This is confusing, because for any other type of error, you can get the useful information by reading err.originalError, except certain types of uniqueness constraints, which don't persist this information.

Hi @kevinburkeshyp, is this the same as balderdashy/sails-postgresql#179 / PR balderdashy/sails-postgresql#180 or does this problem requires changes in waterline core?

It depends on the library I believe... at least for sails-postgresql the original error never makes it to Waterline so nothing could be done about it in core

OK, should we close this issue and focus on balderdashy/sails-postgresql#179 / other adapters then?

Sure

Thanks


Edit: Add full hooks API reference for integrations

Motivation: Hackathon participants were confused how integrations should be built. This PR introduces detailed docs and examples for each API hook!

It also occurs to me that a nice follow up to this PR would be an example of settings up a dev env for local testing of said integration. For example, should I create a /integration_test that's a literal astro project? Then do I do npm pack and point to that tarball?  Where exactly should the integration files go and what are the conventions there?

This is great! The only big feedback that I'd have is to add an example below each hook that just shows all of the options, so that the reader understands what “config” option means.
Example:
## astro:config:setup

When it’s run: ...
Use case: ...

'''
'astro:config:setup': ({
  config,
  command,
  updateConfig,
  addRenderer,
  injectScript,
}) => {
  // ...
},
'''

Open to other ideas here as well, as long as it solves the "what does “config” option mean?"  I'd suggest moving this to a docs issue so we can follow-up here. I'll look into this myself for some best practices with pnpm 😁


add unit tests for ignoring LongParameterList for function/constructo…

add unit tests for ignoring LongParameterList for functions/constructors with configured annotations

I think that's it. We're not adding a brand new rule so there's no more admin work i think


Configurable PWA Icon

Given https://github.com/FlowFuse/node-red-dashboard/issues/1326, the configured icon shows when Dashboard is installed locally to a mobile device or laptop.
Configuring PWA

Open Browser on Mobile
Navigate to ipaddress:port/dashboard
Settings > Install (Exact phrasing depends on which phone you've got)

@joepavitt
I’ve tried making the PWA icon dynamic, but unfortunately, it's not something that can be set at runtime like a favicon. PWA icons must be specified at build time, and they only support files in the local directory. External image URLs are not supported.
Also I've this way which was mentioned in the main issue #1144
We’ll need to generate the icons during the build process and ensure they’re bundled within the application.

What happened to the option of serving up manifest file via the Express server? e.g. https://stackoverflow.com/questions/50086761/dynamic-icon-pwa-manifest

Yes, it would need to be a local file, but I think that's still an option here, as we can upload local files to Node-RED?

What happened to the option of serving up manifest file via the Express server? e.g. https://stackoverflow.com/questions/50086761/dynamic-icon-pwa-manifest

This works when we have the icons in the public directory itself.

Yes, it would need to be a local file, but I think that's still an option here, as we can upload local files to Node-RED?

I will try in this way as well


Virtual users of load test

Description :sparkles:
Virtual users of load test is adjusted to match reality
Issues :bug:
Closes :no_good_woman:
DEVOPS-428:
Related :handshake:
Testing :alembic:
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #245 (8c11b8e) into master (e3dc01d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #245   +/-   ##
=======================================
  Coverage   88.50%   88.50%
=======================================
  Files         120      120
  Lines        3654     3654
  Branches      847      804   -43
=======================================
  Hits         3234     3234
  Misses        412      412
  Partials        8        8

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Subsonic Error (code 70) / Requested action getNowPlaying is not supported

Describe the bug
When I click on an artist and the albums of that artist are displayed, the error message is displayed for each album:
Subsonic Error (code 70)
Requested action getNowPlaying is not supported
To Reproduce
Steps to reproduce the behavior:

Go to 'Added Subsonic Server'
Click on 'Artists'
See the error message per album

Expected behavior
No Error PopUp-Message
Screenshots

Versions:

macOS [e.g. 13.5]: 13.4.1 (c)
Subsonic Server and Version [e.g. Navidrome 0.44, Subsonic 6.1.6]: Nextcloud 27.0.1, With Music v1.8.4 Plugin:  (https://github.com/owncloud/music/releases/tag/v1.8.4)
Submariner Version [e.g. 2.2]: 2.3.1
Download Source [e.g. Mac App Store]: Mac App Store

Additional context
Add any other context about the problem here.

Looks like Nextcloud Music doesn't support the "now playing" functionality (that shows what other users are listening to). It's not a big deal since it should only be touching that endpoint if the now playing sidebar is activated (even if hidden) though. However, there is an auto-refresh timer for it. Disabling that timer in the settings might help with this.
Unfortunately, there is no way to check for featureset beyond Subsonic API version numbers. It's a bit annoying to support multiple servers as a result.

Thanks for your reply. I see, unfortunately it doesn't help to disable the Auto Refresh from Now Playing. The now playing sidebar is closed. Is there any way not to display the error message in this case? Or e.g. display "not availible with the curret server" in the sidebar? Submariner is really a great app and I use it despite the error message, although it is annoying. :-)

Looking at the callsites, it does call now playing on each refresh (i.e. via Cmd+R).
Annoyingly, code 70 is also used for when the API exists but the requested object wasn't found. We'd have to keep track of what APIs returned code 70, and check what didn't work at each part that calls it, basically. This is more annoying than being able to just check if it wouldn't work from the beginning, but it'd work.

When I press Cmd+R, the error message 70 appears.

On a newly installed submariner + clean subsonic installation, I have the same issue.

I just got back from vacation, so I can take a look at this again.

I have just tried a few old versions. The problem appears from version 2.2 onwards. Version 2.1.1 runs without the error message code "Subsonic Error (code 70) / Requested action getNowPlaying is not supported". Might this help?

2.2 introduced refreshing the now playing view as part of refreshing normally.
FWIW, I've filed owncloud/music#1079 as a server-side solution for OC Music at least. On the Submariner side, it's a little bit annoying because servers can return different things for unimplemented routes - i.e. OC Music returns code 70 Subsonic responses, whereas Navidrome returns HTTP 404s.

Fixed this, will be available in the next release

Thanks for the fix! I'm already looking forward to the new release :)


fix integration-test TestBadWasmRemoteLoad for default deny-all policy

fix integration-test TestBadWasmRemoteLoad for default deny-all policy when wasmplugin  FAIL_CLOSE
Please provide a description of this PR:
#53363 broke TestBadWasmRemoteLoad and this PR is a fix for it.

Hi @MaYuan-02. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

/ok-to-test

@MaYuan-02: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-discovery_istio
a60d8330d8a7fa90eb55a3acb142cf1b1d82705e
link
true
/test integ-telemetry-discovery

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@MaYuan-02: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-discovery_istio
a60d8330d8a7fa90eb55a3acb142cf1b1d82705e
link
true
/test integ-telemetry-discovery

integ-telemetry-istiodremote_istio
a60d8330d8a7fa90eb55a3acb142cf1b1d82705e
link
true
/test integ-telemetry-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@MaYuan-02: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-discovery_istio
a60d8330d8a7fa90eb55a3acb142cf1b1d82705e
link
true
/test integ-telemetry-discovery

integ-telemetry-istiodremote_istio
a60d8330d8a7fa90eb55a3acb142cf1b1d82705e
link
true
/test integ-telemetry-istiodremote

integ-telemetry-mc_istio
a60d8330d8a7fa90eb55a3acb142cf1b1d82705e
link
true
/test integ-telemetry-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

seems not complete

@zirain @hzxuzhonghu Thanks for your review.
It seems there is some delay in the xDS configuration taking effect. Due to the fast execution of automated tests, occasional test failures may occur. I have added a retry logic, and now the tests behave as expected.


@estudante.sc.senai.br domain

Official website URL: https://sc.senai.br
Proof domain exists: https://imgur.com/a/ovUxDTF

@vXultz I am sorry, the domain estudante.sc.senai.br is removed from our trusted domains list because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).

@philipto sorry for the late awnser but they doens't let you register a student e-mail, even if you are a student you can't, they give you these e-mails.


[QUESTION] Is it possible to have only choosen field in output file?

Question
When I use a twint as a module in python script, is it possible to do not extract some fileds from post? For example I don't need "id" in output file, is there any parameter I can add to script to don't have this in final file?

So, I would like to say thank you for your work.
That's so helpful :)


Using ES syntax with Netlify Functions and Netlify Dev?

- Do you want to request a feature or report a bug?
Feature.
- What is the current behavior?
Unable to get the ES syntax working with Netlify Functions when deployed using Netlify Dev. Netlify will either complain about unsupported import statement, or when I change my functions to use the .mjs extension and add type: "module" to package.json, it will say the function cannot be found.
- If the current behavior is a bug, please provide the steps to reproduce.
N/A
- What is the expected behavior?
I should be able to use the same ES syntax in both client code and serverless functions to increase code reusability.
- Local Environment Information
Paste the results of netlify status --verbose here
System:
    OS: macOS 10.15.3
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
  Binaries:
    Node: 12.15.0 - ~/.nvm/versions/node/v12.15.0/bin/node
    Yarn: 1.15.2 - /usr/local/bin/yarn
    npm: 6.13.4 - ~/.nvm/versions/node/v12.15.0/bin/npm
  Browsers:
    Chrome: 79.0.3945.130
    Firefox: 69.0.1
    Safari: 13.0.5

Hey @mrlubos I believe you cannot use import statements within Netlify functions without compile them with webpack! I learned however, you can do that using netlify-lambda! I am writing a blog post about this very thing but short synopsis:

Place your functions in a folder you can later find
In your build step, you need to do something like netlify build <folder full of lambdas you want to compile>
In your netlify.toml declare where your functions will be built and served from such as

So for instance my netlify.toml looks something like:
[build]
  command = "npm run build" # This commands maps to building both my gatsby blog and my functions
  functions = "functions-build" # where my functions will be located and I also link this on Netlify

The reason I figured this out was through this documentation here on When to use Netlify Dev, netlify-lambda, or both

Hi @maxcell, you're right! That's why I am requesting this as a new feature, one of my requirements is to use Netlify Dev as opposed to Netlify Lambda.

My bad about that one 😅, I'm not reading enough!

Thanks for looking into this issue! Hopefully somebody from Netlify will respond to it soon 🤧

We now have built-in support for this using our esbuild bundler, see more here.
If the new bundler doesn't satisfy your use cases, please open a new issue on this repo.

Nice, thank you @erezrokah


Expose some additional nixosModules

These are “general” modules that provide options without config and are already relied on by other projects (such as home-manager). Exposing them this way makes them an explicit part of the nixpkgs flake interface and can allow other projects to use, e.g.,
nixpkgs.nixosModules.assertions

rather than something like
(pkgs.path + "/nixos/modules/misc/assertions.nix")

which is bad not only because it treats the directory structure of nixpkgs as an interface, but also because it shouldn’t be necessary to have pkgs at the point modules are loaded (as opposed to when they are applied).

I like the idea, but we have some work to do before we make these modules official.

Thanks! This is exactly what I was hoping for. I’ll work on changes in this vein.


place movie/tvshow name in a small overlay at the start/ after a commercial break

just like a second watermark that automatically disappears after a few seconds

This one would still be great to have


SDN Missing properties on multiple commands & column order incorrect

Missing properties on all SDN commands
Attached is the analysis made comparing Python CLI with PS CLI
az_network_table_output.txt
azure_network_table_output.txt
Files are Sync for easy comparison with lock scroll, same lines for same commands.
The following format after each command
#+ExtraProperty1, +ExtraPropertyN #*ChangedProperty  # -MissingProperty1, -MissingProperty2
Extra properties are ok, but missing must be included.
Additional with low priority, sync property order since some of those cases are not easy to read due the random column order.

[ ] vnet: Add type, address prefixes, subnets
[ ] vnet peering show: Add 'allow_virtual_network_access', 'allow_forwarded_traffic', 'allow_gateway_transit', 'use_remote_gateways', 'remove_virtual_network'
[ ] lb list: Add probe, FIP, backend pool, rule, inbound NAT rule, inbound NAT pool, outbound nat rule
[ ] lb show: Add type, frontend IP configs, probes, BAP, LB rules, inbound NAT rules
[ ] lb frontend-ip list: Add private IP, subnet, public IP
[ ] lb proble list: Add path
[ ] lb rule list: Add "enable_floating_ip"
[ ] lb inbound-nat-rule list: Add enable_floating_ip
[ ] public-ip list: Add FQDN
[ ] public-ip show: Add type, IP config ID
[ ] nic list: Add IP forwarding, internal DNS name, internal FQDN, internal domain name suffix
[ ] nic show: Add type, internal domain name suffix, enable IP forwarding, IP configs
[ ] nic list-effective-nsg, show-effective-route-table: not supported
[ ] nic ip-config list: primary, subnet, public-ip
[ ] nic ip-config show: primary, subnet
[ ] nsg list: rules number
[ ] nsg show: type, security rules
[ ] nsg rule show: type

@malopMSFT Hello. Is this issue still used? or will us use the CLI Gap V2?

@malopMSFT any update for this issue?

@malopMSFT Hello. Since this one is quite old and this issue is quite hard for us to track, I closed this one first. If necessary, could we open separated issue?


Add option to mark paste as private

Let's add an option with every new paste with the ability to mark it as private, effectively removing it from the list of all pastes.
A feature like this will conflict with the paste deduplication method we currently use since a new paste can override the previous paste's status, however, one could argue that if someone discovers the paste ID and then posts it again, there's no point in marking the previous paste as private again.
The ability for users to mark pastes as private can be configurable via the configuration file.

So a private paste would not appear in the list of pastes, but if one guessed the id and entered the correct URL, they would be able to see the paste? That is what Pastebin calls "unlisted" pastes. How about implementing both unlisted and private pastes?
A simple solution for the filesystem backend is to simply use a new metadata, let's call it "visibility", which contains public/unlisted/private, and an additional metadata, "password", which only exists for private pastes.
Private pastes could not be submitted with an empty password, and the password would be ignored for non-private pastes. The value we save for passwords would be the sha256 hash of the submitted password.
The listing page would only show public pastes, which is a simple map to do on the PasteList variable in the list() method.

First of all, let me say that I had Unlisted Pastes in mind and not Private. Thanks for making this distinction.
However, in any case, we have a tiny problem we need to solve if we are to add this functionality. The current Paste ID is calculated as sha256( P ) where P is the content of the paste. That means that if User A posts a paste with the content Hello World and then marks it as Unlisted, User B can post a paste with the same content, Hello World, and mark it as public.
Of course, one could argue that if a user already knows the paste contents, it is essentially public.
The current method of generating Paste IDs is used for easy deduplication: a paste with the same content will always overwrite the previous one.
So I would recommend we first implement Unlisted pastes, and allow users to overwrite the status of a previous paste. That means that if they're able to do it, they already know the contents.
For the Private Pastes, I will open a new GitHub Issue and we can discuss it there.

Agreed.

FYI: this feature is now working on my local version and I will send the PR soon. I'm not gone! :)


Changes to improve readability of service-fabric-reliable-services-communication-webapi.md

Changes to improve readability of document.  When code blocks don't contain file names it can be confusing switching back and forth.

Hi @jgardner04, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

@jgardner04, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;

@jgardner04 : Thanks for your contribution to the Azure documentation! The author, @vturecek, has been notified to review your proposed change.


⚖ Inconsistency: Install Dependencies

Summary
In the install_dependencies() we don't wan't the npm bar to be shown to the user but the script that lives on the user's device shows that.
How Found
By using the local script.
Steps to Replicate
N/A
Description
The cloud script implements the subprocess in install_dependencies() in a way that the normal npm output is not shown to user.

Proposed Solution
The cloud script's implementation should be carried over to the local script.

This issue is no longer valid for more info refer #15 .


Add Alpine Linux support

I'm not familiar with Alpine packaging. Is GNU libc not supported by Alpine?


feat: optimized postgres dataWriter.go by Implementing Batch Insert and Update operations

What does this PR do? it does not solve anything

What does this PR do? it does not solve anything
cc @snigdha510

wait just give me few minutes

wait just give me few minutes
FYI #1271 is already there solving the issue, i just need to update the test


Fix the signatures of matrix functions

And cleanup matrix a bit

@marek-safar hey do you mind merging some of the open PRs? The easiest ones are
#276
#366
#312
#327
#328
#334
#362
#365
#369


Adds minDuration and maxDuration to the query api, implemented by SQL

This adds minDuration and maxDuration queries that zipkin-web can use to
search for long traces. This is initially supported by the in-memory and
anorm (SQL) span stores.
This adds very basic UI support, which someone better at javascript can
polish up:

@yurishkuro let me know, if you need a hand on cassandra support.

has something changed with the styling of UI elements recently?  I just upgraded, and the dropdowns/textboxes look taller / bigger.
Circa 1.20.x:

Now:

@yurishkuro I am not aware of any intentional change like that.

Between 1.19 and now, mustache changed.. that's the template library. I
don't see any CSS changes, but maybe viewing source on both could identify
what's up
git diff 1.19.0 1.22.1 zipkin-web/build.gradle

Looking @ Cassandra. One difficulty there is that the query becomes a composite query, ts <= X and duration >= Y, which cannot be satisfied directly by Cassandra.
cqlsh:zipkin> select * from duration_index where service_name='mezzanine' and time_bucket=1 and ts > '2015-11-07T11:00:00' and duration > 100000;
InvalidRequest: code=2200 [Invalid query] message="PRIMARY KEY column "duration" cannot be restricted (preceding column "ts" is restricted by a non-EQ relation)"

The only efficient way I can think of doing it is by bucketing the timestamps, say hourly, so that the schema will be
CREATE TABLE IF NOT EXISTS zipkin.duration_index (
    service_name      text,
    time_bucket       int,
    duration          bigint,
    ts                timestamp,
    trace_id          bigint,
    PRIMARY KEY ((service_name, time_bucket), duration, ts)
) WITH CLUSTERING ORDER BY (duration DESC, ts DESC)

The drawback of this is that if the time buckets are small, the distribution of returned traces will be skewed. If the buckets are large, like 1d, the partition/row might become too large.
Is there another solution I am missing?

small good news, Cassandra allows multi-column restrictions on clustering columns, it just requires special syntax like (duration, ts) >= (20, '2015-11-07T11:00:00'). With the table above, and the following data
 service_name | time_bucket | duration | ts                       | trace_id
--------------+-------------+----------+--------------------------+----------
            x |           1 |       20 | 2015-11-07 12:00:00-0500 |        1
            x |           1 |       20 | 2015-11-07 11:00:00-0500 |        1
            x |           1 |       10 | 2015-11-07 12:00:00-0500 |        1
            x |           1 |       10 | 2015-11-07 11:00:00-0500 |        1

the following query finds both D=20 records at different timestamps:
cqlsh:zipkin> select * from duration_index where service_name='x' and time_bucket=1 and (duration, ts) >= (20, '2015-11-07T11:00:00');

 service_name | time_bucket | duration | ts                       | trace_id
--------------+-------------+----------+--------------------------+----------
            x |           1 |       20 | 2015-11-07 12:00:00-0500 |        1
            x |           1 |       20 | 2015-11-07 11:00:00-0500 |        1

The question of buckets skewing the distribution still remains, but it could be addressed by asking the user to specify both ends of the timeframe, not just endTs, and searching all relevant buckets. Then the bucket size can be tuned depending on the traffic volume in a given deployment.

Great progress!
I think we'll eventually hit a point where we need to specify startTs. We
already need this for the dependency query. Should we add this now?

Q1 - I see that it only does root spans, due to  |WHERE trace_id = id. Which is probably a bad assumption anyway, nowhere do we say that root span ID must be equal to trace_id (a more accurate check would've been parent_id == nil or 0).
Q2 - the sql duration query checks for service_name == query.sn. In combination with the above, I conclude it's only limited to the services mentioned in the root span.
Q3 - I remember the slice query, iirc it's one of the reasons the limit param doesn't work correctly in combination queries. This is very hard to support in Cassandra, actually. I would go as far as forcing the UI to make the use choose between annotations query and duration queries. If we want to support both, then at least in Cassandra implementation it requires in-memory join of trace IDs from different index queries, and it's fairly hard to make that join work correctly with LIMIT.
It's fairly easy to implement correct LIMIT for serviceName {+span name} + duration combination in Cassandra, as it can all be stored in the duration index I showed above. As for the semantics of such query, I would go for span-level query, i.e. where both duration and service-name (and span name) must apply to the same span in order for the trace ID to qualify. It seems like it would satisfy most of the queries. The one that it would not satisfy is finding long traces by service name whose span was not long.


Fix for #34

Fix for #34

Hi @mrfelton and thanks for the PR!
One question though, the pickadate tarball that you changed to in bower.json, that's the fix for #34?
It that a temporary version that we should change later?
btw, if you can redo it without the built files it would be awesome since that would make it easier to merge :)

Hi,
this PR is open for quite a while and I am looking to configure the years and months dropdown from outside. I hope you don't mind that I created another PR https://github.com/Textalk/angular-schema-form-datepicker/pull/39 The main difference is that the options are optional and not hardcoded.


CLI progress monitor broken

The progress monitor displayed by the CLI is broken. Instead of staying in one line, it prints a new line on every update, which is extremely annoying.

This seems to be a window problem. On Linux the progress monitor stays on the same line.

Thanks for testing, I was the last who touched it.. and worked/tested it on Linux :)

Fix tested on Windows and worked fine.


Update the broken txt url link

https://github.intel.com/Intel-MLPerf/inference/blob/master/gnmt/nmt/train_gnmt.txt is deadlink.
Signed-off-by: Insop Song<EMAIL_ADDRESS>

merge?


SOLR-16812: update to cbor 2.15.2

I think there's a solrbot version of this same PR. We should close one of them.
https://github.com/apache/solr/pull/1599


Add file mode for crontab and cron.

Added file mode permissions for
/etc/crontab
/etc/cron.

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Spec tests has not been updated. It would probably fail dye to the changed permissions of these files.
Add tests for these parameters.

Updated parameters in Spec

Can you combine all 10 commits to one commit?


Add a JTE reporter

Add a JTE template reporter as an alternative to the Freemarker template reporter.
In constrast to Freemarker, JTE supports Kotlin expressions in templates which would make it easier to call Kotlin code from the templates without having to learn another syntax.

It would be awesome of this allows us to get rid of the overly complex FreemarkerTemplateProcessor which basically just makes the Kotlin models more accessible from Freemarker.

make it easier to call Kotlin code from the templates without having to learn another syntax.

But isn't it so that e.g. control structures still requite JTE syntax, like @for etc.? Because of that, just do document the idea, I was wondering whether we could instead simply create a Kotlin script reporter that returns a (huge) string which gets expanded / interpolated by standard Kotlin functionality.

But isn't it so that e.g. control structures still requite JTE syntax, like @for etc.?

Yes.

Because of that, just do document the idea, I was wondering whether we could instead simply create a Kotlin script reporter that returns a (huge) string which gets expanded / interpolated by standard Kotlin functionality.

This is definitely also an option. It's somehow the opposite approach to template engines. With template engines you write the template in the native output format like Markdown and inject some code here and there, so the "main" syntax of the template file is that of the output format. With the Kotlin script approach the code part becomes easier, but it becomes harder to ensure that the syntax of the output format is followed (expect when a DSL for the format is available).


Display a warning when Bash is run in POSIX mode

When SDKMAN! is run in POSIX mode, a warning should be displayed and Bash completion turned off.

Closed in favor of #978.


@W-16255012 Fix added for combobox on enter key pressed

Fix added for combobox on enter key pressed same as down key pressed

@Ryoji Osawa I'm unable to add reviewers here .Can you please review this code ?
This is a change is combobox.jsx where when enter key is pressed, the dialog should be opened and closed .
No other functionality changes.

@aarthim1111 Please see if you can sign the CLA

@interactivellama I have signed the CLA today


Warnings on install

System Info

window 7
node v6.6.0
npm 3.10.3
git version 2.7.2.windows.1

followed instructions in README.md and got the following errors

just text
npm WARN deprecated<EMAIL_ADDRESS>Babel 5 is no longer being maintained. Upgrade to Babel 6.
npm WARN deprecated<EMAIL_ADDRESS>Copy its ~20 LOC directly into your code instead.
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>cross-spawn no longer requires a build toolchain, use it instead!
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.

and at the end after the dependency tree (probably from running windows not mac)
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>

Hi, know about these. So far there has been no reason to update dependencies and potentially introduce bugs. Is the registry working for you?

Thanks for the quick reply. Everything seems to be working fine. Just wanted to make sure that you knew about the deprecations.
Also sorry for not being friendly in my initial post. I have got in a habit of being as quick as possible with issues for my own projects and didn't stop and think to be human.
Thank you for all of your great work with Brackets!
Josh


Openlane at rcx_mcsta.min stage crasses with segmentation fault

Description
With latest openlane at rcx_mcsta.min stage  fails with segmentation fault.
[INFO]: Checking Wire Lengths (log: wb_host/runs/wb_host/logs/routing/20-wire_lengths.log)...
[STEP 21]
[INFO]: Running SPEF Extraction at the min process corner (log: wb_host/runs/wb_host/logs/signoff/21-parasitics_extraction.min.log)...
[STEP 22]
[INFO]: Running Multi-Corner Static Timing Analysis at the min process corner (log: wb_host/runs/wb_host/logs/signoff/22-rcx_mcsta.min.log)...
[ERROR]: during executing openroad script /openlane/scripts/openroad/sta_multi_corner.tcl
[ERROR]: Log: wb_host/runs/wb_host/logs/signoff/22-rcx_mcsta.min.log
[ERROR]: Last 10 lines:
39# 0x00007F58169B4F1E in /lib64/libtcl8.5.so
40# Tcl_EvalEx in /lib64/libtcl8.5.so
41# Tcl_Eval in /lib64/libtcl8.5.so
42# sta::sourceTclFile(char const*, bool, bool, Tcl_Interp*) in openroad
43# ord::tclAppInit(Tcl_Interp*) in openroad
44# Tcl_Main in /lib64/libtcl8.5.so
45# main in openroad
46# __libc_start_main in /lib64/libc.so.6
47# 0x0000000000CA05E7 in openroad
child killed: segmentation violation

[ERROR]: Creating issue reproducible...
[INFO]: Saving runtime environment...
OpenLane TCL Issue Packager

Expected Behavior
Expected clean openlane flow.
Environment report

dinesha@lenovo-i3-10100-07IMB05:~/workarea/efabless/MPW-7/OpenLane$ python3 ./env.py issue-survey
Kernel: Linux v5.15.0-50-generic
Distribution: ubuntu 20.04
Python: v3.8.10 (OK)
Container Engine: docker v20.10.17 (OK)
OpenLane Git Version: a0b19fa68b79eff3e29b674106b06d20aab637f1
pip: INSTALLED
python-venv: INSTALLED
---
PDK Version Verification Status: OK
---
Git Log (Last 3 Commits)

a0b19fa 2022-11-01T19:09:25+02:00 Add Wire Length Checker (#1463) - Mohamed Gaber -  (HEAD -> master)
1f923eb 2022-11-01T12:07:33+02:00 [BOT] Update openroad_app (#1435) - Openlane Bot -  ()
41ed003 2022-10-25T20:04:52+02:00 Fix get_design_path (#1450) - Liu Yihua -  ()
---
Git Remotes

origin	https://github.com/efabless/OpenLane.git (fetch)
origin	https://github.com/efabless/OpenLane.git (push)

Reproduction material
issue_reproducible.tar.gz
22-rcx_mcsta.min.log.gz
Relevant log output

[ERROR]: during executing openroad script /openlane/scripts/openroad/sta_multi_corner.tcl
[ERROR]: Log: wb_host/runs/wb_host/logs/signoff/22-rcx_mcsta.min.log
[ERROR]: Last 10 lines:
39# 0x00007F58169B4F1E in /lib64/libtcl8.5.so
40# Tcl_EvalEx in /lib64/libtcl8.5.so
41# Tcl_Eval in /lib64/libtcl8.5.so
42# sta::sourceTclFile(char const*, bool, bool, Tcl_Interp*) in openroad
43# ord::tclAppInit(Tcl_Interp*) in openroad
44# Tcl_Main in /lib64/libtcl8.5.so
45# main in openroad
46# __libc_start_main in /lib64/libc.so.6
47# 0x0000000000CA05E7 in openroad
child killed: segmentation violation

[ERROR]: Creating issue reproducible...
[INFO]: Saving runtime environment...
OpenLane TCL Issue Packager

EFABLESS CORPORATION AND ALL AUTHORS OF THE OPENLANE PROJECT SHALL NOT BE HELD
LIABLE FOR ANY LEAKS THAT MAY OCCUR TO ANY PROPRIETARY DATA AS A RESULT OF USING
THIS SCRIPT.

Parsing config file(s)…
Setting up /home/dinesha/workarea/opencore/git/riscduino/openlane/wb_host/runs/wb_host/issue_reproducible…
Done.
[INFO]: Reproducible packaged: Please tarball and upload 'wb_host/runs/wb_host/issue_reproducible' if you're going to submit an issue.
[INFO]: Saving current set of views in 'wb_host/runs/wb_host/results/final'...
[INFO]: Generating final set of reports...
[INFO]: Created manufacturability report at 'wb_host/runs/wb_host/reports/manufacturability.rpt'.
[INFO]: Created metrics report at 'wb_host/runs/wb_host/reports/metrics.csv'.
[INFO]: Saving runtime environment...
[ERROR]: Flow failed.

@dineshannayya
Duplicate of #1467.
Its got resolved it with this PR: https://github.com/The-OpenROAD-Project/OpenLane/pull/1466
Please update commit to latest to resolve above issue.


Could you add the Bluetooth Headset detect?

Hi,
I'm using Android and it works. But it only detects line-in headset, Bluetooth headset is ignored.
I'm not a programmer, so I hope you could add the Bluetooth detection.
It surprised me that YOU made "A Blind Legend"!  What a coincidence : ) I like it very much. Good work indeed!

Just created a pull request with bluetooth headset detection on IOS. -> #3
Added the code for Android too, but it's untested and you'll need to build the .jar to test it.

Also not familiar with building jar files. Are you able to push an updated version of the .jar for us?

@alamboley Same here. Any chance you could recompile the JAR with a more recent version of Android please?

Done via #7!


Escape variable name and value

Before this change, the generated code could contain syntax errors, if the variable name or value contained a single quote character, or backslash character.
This commit escapes both strings to fix this issue.

Thank you!


fix(docs): update migration guide for Ts.ED dependencies

Information

Type
Breaking change

Doc
No

Todos

[x] Documentation

Solves #2369
Before

After

@Romakita

HI @geekyharsh05
please just change the message commit by:
docs: update migration guide for Ts.ED dependencies

HI @geekyharsh05
please just change the message commit by:
docs: update migration guide for Ts.ED dependencies

Done

No.  Not using the github ^^
fix will create a release. Here it’s just a doc improvment. I don’t to publish a release for that on npm.

@Romakita

:tada: This PR is included in version 8.0.0-beta.1 :tada:
The release is available on:

v8.0.0-beta.1
GitHub release

Your semantic-release bot :package::rocket:


Naive Bayes learners

I am trying to set up naive Bayes learners(bernoulli and multinomial), how do i set up the vectorizers for two separate situations?

presence or non presence in feature vector
the count of words stored in feature vector

I believe the load_dataset function is doing the 2nd situation currently by default, is this right?

is it just the change in the binary parameter in the initialization of the EmailDataset?


There is preview support?

How does one activate it?

Yes, there is preview support. Click on the icon next to the component definition.

Thank you, I was looking in menus!


fix: Modal Accessibility and copy paste.

Description
In this Pr we are tackling a few issues.

disallow copy pasting modals into other widgets instead always paste them on the main canvas.
disable modal header and footer accessibility in edit mode.
make sure edit mode based wrappers are not wrapped for skeleton widget.
make sure widget name is not shown for skeleton widget.
fixed random errors in the withWidgetProps flow
removed unwanted invocation of setting highlights.

Fixes #33382
Fixes #32923
or
Fixes Issue URL

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!TIP]
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9107042945
Commit: 35fafa6eb95547046dc6bfac9b1936b95864a04f
Cypress dashboard url: Click here!

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

/build-deploy-preview skip-tests=true


Investigate phantomjs for test execution

jsdom is pretty flaky - let's see if we can make the tests more sane by moving to phantom.

The maintainer of PhantomJS is stepping down so another choice might be better for the long term.


Problem with the depricated ActionBarActivity

viewAnimator = new ViewAnimator<>(this, list, contentFragment, drawerLayout, this);
The first Parameter must be of type "ActionBarActivity", but its depricated and not available anymore.
And I cant use AppCompatActivity, cause that leads to an error

Hi getting the same problem with parameter type in
viewAnimator = new ViewAnimator<>(this, list, contentFragment, drawerLayout this) . I tried replacing it by : ActionBarActivity.this but this time Android studio shows error : cannot resolve symbol 'ActionbarActivity'.

Hello. @dodoweder97  , @Dhrishta  seems like you, guys, have outdated dependency in your build.gradle
Actual version is:
dependencies { 	    compile 'com.github.yalantis:Side-Menu.Android:1.0.1' 	}
Because actual version contains merge request, which fix this #34 , #57

Just updated the dependencies. Thanks.
There seems to be no error now.

Thanks I did the it works

hey can you tell me how its work. i stuck in it

Hello,
Can you be more precise? Are you talking about the depency or the yalantis
side menu bar as a whole?
On Tue 3 Jul 2018 at 01:06, Imran-Zulfi6121<EMAIL_ADDRESS>wrote:

hey can you tell me how its work. i stuck in it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Yalantis/Side-Menu.Android/issues/77#issuecomment-401963769,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARoHzdBEv3ATrYK2ENR_qBQPeXjaJxFQks5uCqeIgaJpZM4SbfeP
.

--
Dhrishta SEWRAJ
Université Paul Sabatier Toulouse III


Bug: You can create site with empty fields "Git Repo Source"

Description

The backend returns a phrase where there are two spaces after "There is"

Preconditions:
If necessary
Reproduction steps:

Create new Static Site with empty fields "Git Repo Source"

Expected Result:
Error
What we really wanted to see, how it should work
Actual result:
Created successfully , but new site get Current status:  error_creating
What we see after we reproduced the bug
Environment:

OS:
Выпуск	Windows 11 Pro
Версия	22H2
Дата установки	16.10.2023
Сборка ОС	22621.3007
Взаимодействие	Windows Feature Experience Pack 1000.22681.1000.0

Priority(High, Medium, Low)

Low

Severity(Blocker, Critical, Major, Minor, Trivial)

Minor

Application:
Link to Allure report:
https://areso.github.io/Oblache/
Additional context

Yeap, I should check it.

Fixed


remove the use of the --package-warnings flag in flutter analyze

I don't think we want this flag - this causes all direct and transitively imported packages to be analyzed. We should remove it (and verify that all our packages in the flutter repo are still properly analzyed).
Related to https://github.com/dart-lang/sdk/issues/25722. /cc @Hixie. Happy to experiment w/ this myself (after closing out a few other PRs).

Yeah I had turned this on because I wanted all our direct and transitively imported packages to be analyzed. What I really want to be able to express is "analyze anything that didn't get pulled down by pub". For example, say we analyze a test in the stocks package. I want it to analyze all the stocks files, and all the flutter files that that pulls in, and all the files in the newton package that that pulls in. I'm worried that if we just rely on a test in the newton package pulling in the newton package, it'll actually only be bringing in the few files we happened to have tests for.
This gets less and less problematic as we add more and more tests, of course.
cc @pq

After talking to @devoncarew and @bwilkerson, I think what we really want longer-term is CLI support for directory-based analysis (https://github.com/dart-lang/sdk/issues/25129).

Closing this since I think supporting project directories (and directories of project directories) in the dartanalyzer will better support Ian's use case.


Remove Guava from API?

Please answer these questions before submitting a bug report.
What version of OpenCensus are you using?
0.12.x
What did you expect to see?
opencensus-api has dependency on Guava. I don't know how Guava is actually used by the API, but it's strange for an API module to have explicit dependency on Guava that may actually introduce dependency conflicts for users.
In fact, if Guava is not used extensively, it's probably better not to use it at all, or, shade it to avoid issues.

Also, here is a dependency tree on using OpenCensus. I wonder if they are all necessary to expose to the end user.
[INFO] com.example:census-example:jar:1.0-SNAPSHOT
[INFO] +- io.opencensus:opencensus-api:jar:0.12.2:compile
[INFO] |  +- com.google.errorprone:error_prone_annotations:jar:2.2.0:compile
[INFO] |  +- com.google.code.findbugs:jsr305:jar:3.0.1:compile
[INFO] |  +- io.grpc:grpc-context:jar:1.9.0:compile
[INFO] |  \- com.google.guava:guava:jar:19.0:compile
[INFO] +- io.opencensus:opencensus-impl:jar:0.12.2:runtime
[INFO] |  +- io.opencensus:opencensus-impl-core:jar:0.12.2:runtime
[INFO] |  \- com.lmax:disruptor:jar:3.3.9:runtime
[INFO] +- io.opencensus:opencensus-exporter-stats-prometheus:jar:0.12.2:compile
[INFO] +- io.prometheus:simpleclient:jar:0.3.0:compile
[INFO] +- io.prometheus:simpleclient_hotspot:jar:0.3.0:compile
[INFO] +- io.prometheus:simpleclient_httpserver:jar:0.3.0:compile
[INFO] |  \- io.prometheus:simpleclient_common:jar:0.3.0:compile
[INFO] \- io.opencensus:opencensus-exporter-trace-zipkin:jar:0.12.2:compile
[INFO]    +- io.zipkin.reporter2:zipkin-reporter:jar:2.3.2:compile
[INFO]    |  \- io.zipkin.zipkin2:zipkin:jar:2.4.6:compile
[INFO]    \- io.zipkin.reporter2:zipkin-sender-urlconnection:jar:2.3.2:compile

@saturnism Which dependencies are you referring to in the graph above?  It looks like it includes an OpenCensus implementation and exporters in addition to the API library.
I think that it wouldn't be too difficult to remove Guava from opencensus-api.  We already removed the dependency on Guava collections in #1068.

you didn't ask me but io.prometheus:simpleclient is a liability as it is a pre-release api version
com.lmax:disruptor should probably be shaded (assuming it works ok shaded)

io.prometheus:simpleclient liability because it could conflict with other versions of this, and assuming that 0.3 means api changes are likely

@adriancole I tried to upgrade to io.prometheus:simpleclient v0.3.0 before and I think it is backwards compatible, though it has some issues on our CI build on JDK 8 and 9 (https://github.com/census-instrumentation/opencensus-java/pull/1051).

@sebright i simply created a new project that uses opencensus, and generated a dependency tree from there.
opencensus-api has dependency on guava, and also exports a few others like errorprone and findbugs annotations. I'm not sure if those need to be exported to user runtime?
+1 to @adriancole for shading disruptor.

+1 for the shading of the disruptor and probably all dependencies that can. One downside of doing this is the increase in the application binary size but probably should be fine.

If I understand correctly, we should be able to make the Error Prone and FindBugs dependencies compileOnly.

I made a PR to make the annotation libraries compileOnly: #1098

There is another suggestion related to this issue in https://github.com/census-instrumentation/opencensus-java/pull/1111#issuecomment-378439633.

There is another suggestion related to this issue in #1111 (comment)
https://github.com/census-instrumentation/opencensus-java/pull/1111#issuecomment-378439633
.

yep TL;DR; base16 is not a good reason to keep a dep on guava. Implementing
base16 is trivial and it doesn't change. just do it (as nike likes to say)

Unassigning myself, since I'm switching projects.


[TF 2.0 API Docs] Added Documentation to tf.keras.activations.selu

Fix #27657 by adding required information to tf.keras.activations.selu here and added examples of FNN instead of ConvNets because SELU finds better use there. Also added an image of the function, hoping the markdown renders properly.

@kyscg @rthadur @googlebot any ways to confirm the CLA if all co-authors of the commit have already signed it (Even though individual commits are made at later stages)? Is this level of granularity necessary (for each commit?) just asking out of curiosity

I think it is necessary, there is no other way of knowing whether all the co-authors are aware of a proposed merge. It's not that complex either.

@rthadur, gentle ping to review and set CLA's to yes

Sorry again @kyscg but as @ekerazha pointed out, the kernel_initializer='lecun_normal' has been used. Please re-review

@Bharat123rox thank you for your contribution , please make sure all the contributors have signed CLA

Yes, all contributors have signed the CLA @Bharat123rox and @kyscg and are
willing to merge contributions in source code
On Mon, 15 Apr 2019, 10:47 pm rthadur<EMAIL_ADDRESS>wrote:

@Bharat123rox https://github.com/Bharat123rox thank you for your
contribution , please make sure all the contributors have signed CLA
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/27837#issuecomment-483339145,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMwu8bRdkJk3ghjI5SoIE8CFeLyjjBnQks5vhLQUgaJpZM4cuvp2
.

@gbaned @rthadur please change CLAs to yes

@rthadur @gbaned please add the cla:yes labels and @tanzhenyu please re-review


SequenceMatcher $junkCallback

the $junkCallback parameter is never used in the SequenceMatcher  class.
So the lines 193 - 207 (chainB function)
Maybe that can be removed for good.

See PR https://github.com/JBlond/php-diff/pull/41

It can be removed, because Diff currently only calls the SequenceMatchter without a 4th parameter.
In other words, the callback functionality isn't implemented into php-diff.
However, the callback functionality of SequenceMatcher::chainB() might be a nice feature...
SequenceMatcher::chainB() evaluates which lines of the second sequence are repeated enough to mark them as "popular".
These lines are stored in a "popular" dictionary or else in a "nonJunk" dictionary.
The callback allows a user to define a custom function which does the opposite.
SequenceMatcher::chainB() passes the same lines as above to this callback function which in turn can also evaluate them.
When the line is considered to be "junk", the callback function should return a thruthy or otherwise a falsy.
If the callback function returns a thruthy (line is junk), SequenceMatcher::chainB will move the line from the "popular" dictionary and from the "nonJunk" dictionary to the "junk" dictionary.
Later on SequenceMatcher::findLongestMatch() uses this "junk" dictionary.
But here it's getting tricky and I'm not (yet) really sure what happens. 😅
This because of the sequenceMatcher originally being designed for character sequences and not for line sequences as php-diff uses it.
My advice is not to remove code related to this callback from SequenceMatcher.
First find out what SequenceMatcher::findLongestMatch() exactly does with it.
If it adds value to php-diff, implement the callback feature into  Diff.
E.g. Add a new property in which a function reference or closure can be defined and pass this property to SequenceMatcher as 4th parameter at its construction.
If it doesn't add value, it can be removed later on.
Please feel free to contact me when you have any questions or comments about above.

@DigiLive Maybe to understand this better it might help to read the the original https://github.com/python/cpython/blob/3.8/Lib/difflib.py


RENDER BUG

I have no other mods installed...
its running on forge <IP_ADDRESS>3...
help

what did you do bryan?
Sorry that was a joke

Old issue, closed.


chore(main): release iso-signatures 0.1.9

:robot: I have created a release beep boop
0.1.9 (2023-07-25)
Bug Fixes

add npm provenance (ea8a4f3)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugomrdias/iso-repo/releases/tag/iso-signatures-v0.1.9 :sunflower:


Consistent project name

Is it releaser, release, Goreleaser, goreleaser, GoReleaser,... ?
Also, the released binary is called release and the binary I get via go get is called releaser.
(Personally, I would prefer a goreleaser binary)
What do you think?

I agree!
Let's rename the binary to goreleaser and in other places let's let it be GoReleaser.
What do you think?

As long as it is consistent I think its a good choice :)
To get binary to be goreleaser in the go get way you would need to rename the whole repository though.

indeed, I need to go AFK now, I'll do it later =)

No hurry :)

Done, let's wait for what will break 😆


add find_*_licenses

implements https://github.com/ericphanson/LicenseCheck.jl/issues/2#issuecomment-777147627
I'm not totally sure it's the best approach but I kind of like it. I think the size check I added makes it not too crazy.
Still needs

[ ] README update
[ ] tests

I think to be confident this can be used on unknown directories, I should do a check and if there are more than 1k files in the directory, then fallback to just looking for LICENSE, COPYING etc. Then we can be sure there are < 1k licensecheck calls and each call is on a small-enough file, so that we can be fairly sure it will complete in a reasonable amount of time (a couple of seconds), although I should do more benchmarking to confirm.

Yeah, the brute-force approach is interesting (and capping the size of the files to check is a good idea!), but you can really get anything!  Looking for sensible file names is faster, but of course can miss files with wild names

there's probably not one best way to do it for all cases, so I figured I'd just implement a few different ways for the user to choose. I saved the name find_license and find_licences for my favorite approach, which is bruteforce for < 100 files, and a list for larger directories.
I also added a find_licenses_by_list which in some ways is the safest, because it avoids readdir so you could theoretically use it with infinitely large directories (which obviously doesn't make sense usually, but maybe you're using AWSS3.S3Paths and looking at giant s3 buckets or something...).


Fix @batch with @parallel and local metadata and refactor task ids

Pass parallel_task_prefix from decorator to client so the task id logic is in one place only.
Now works with local metadata. The problem was _wait_for_mapper_tasks. There is now special version for local metadata, which uses S3 files to communicate finishing of the tasks. I could not make it work with the metadata object itself.
some small logging message changes for clarity.

Closing as leaving the project. Can be resurrected.


CSS Selectors Need Update

It's that time again: TypeError: Cannot read property 'getElementsByClassName' of null

Thanks for letting me know, I'll get a fix out as soon as possible.

Luckily it wasn't too many things that had broken, it was just contained to the navigation bar and some of the elements in it.
I have updated the class names and selectors in 1e8e5ce.
This has been released as v1.3.37 and should now be available in all browsers.

Thank you! Great work as always.


The example is outdated and doesn't work with azure flow for registering apps

I'm trying to follow along on this but all of the instructions are for an outdated flow. App registration now occurs at azure.com. Unfortunately the words and screenshots are out of date. Especially chapters 1 & 2. I'm unable to complete the example.
I'm really stuck on the details of what I'm supposed to enter for my "Package SID" & "Security Key". It's very confusing that I can't find those specific words anywhere in the portal, except for these fields when trying to fill out the form for "Windows (WNS)". Going to my "App registrations" shows me all kinds of IDs for my app, but nothing that says "Package SID" for instance.

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

ID: 5b48be73-2f29-6b2a-c837-751d3862bcdb
Version Independent ID: efd00d73-1ff5-8689-8415-fe1630795123
Content: Send notifications to Universal Windows Platform apps using Azure Notification Hubs
Content Source: articles/notification-hubs/notification-hubs-windows-store-dotnet-get-started-wns-push-notification.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@pinkwerks Thank you for your feedback . We will investigate and update the thread .

Apologies for any inconvenience or frustration with this issue. Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review and update the document as appropriate.

As we are moving to a new feedback system, please reopen this issue or contact Microsoft Support if you still want to provide feedback on the documentation. Thank you! #please-close


Step 6 and Licensing

In order to enable a CSMeetingRoom, you need to have enabled licensing.  You also have to wait.  In my case, it was about 15 minutes.
In the steps above, step 6 adds licensing.  That is fine.  But then below it has another section on adding licensing.  Licensing can't come after the room setup, it has to happen during the account setup.  So this statement at the end need to be updated:
After you've completed the preceding steps to enable your Microsoft Teams Rooms account in Microsoft Teams or Skype for Business Online, you need to assign a license to Microsoft Teams Rooms device. Using the Office 365 administrative portal, assign either a Skype for Business Online (Plan 2) or a Skype for Business Online (Plan 3) license to the device.

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

ID: 05282aa8-ae14-117b-5885-7d851ee66b89
Version Independent ID: 85abe13c-d34c-f42c-e273-a69be8cd991e
Content: Deploy Microsoft Teams Rooms with Microsoft 365 or Office 365 - Microsoft Teams
Content Source: Teams/rooms/with-office-365.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@danmyhre Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@danmyhre thanks again for your help improve MS docs. Would you recommend changing the statement to read as follows:  "You need to assign a license to Microsoft Teams Rooms device before completing the preceding steps to enable your Microsoft Teams Rooms account in Microsoft Teams or Skype for Business Online. Using the Office 365 administrative portal, assign either a Skype for Business Online (Plan 2) or a Skype for Business Online (Plan 3) license to the device."?

Yes, but instead of SfBO P2 or P3, I think you should reference the new SKU for a Teams Room:
https://docs.microsoft.com/en-us/MicrosoftTeams/rooms/rooms-licensing

@danmyhre I have updated the wording. I see your suggested link is showing in step 5 under "Assign a license to your account".

@danmyhre  Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Which is the yii2-oauth2-server version used?

Hi, I've just installed the template, but seems the yii2-oauth2-server was updated to latest (composer is marked as *) and broke compatibility with this application. What is the original version number? Thanks!

Hi, try to use version 2.0. Could you me your error?

Fixed, thanks... Updating filsh/yii2-oauth2-server (dev-master 0b8e535 => v2.0.0)
Do not update to 2.0.1, there are changes on Module attributes (removed $options, etc.)

Made changes: https://github.com/ikaras/yii2-oauth2-rest-template/commit/5d0888748ad60455f5f4d2789c0d19d487a2d340
Thanks


remove FDP_SMIME_EXT.1.2

This is covered in the App PP by FCS_STO_EXT.
http://common-criteria.rhcloud.com/emailclient/output/emailclient.html#FDP_SMIME_EXT.1.2

Commented out FDP_SMIME_EXT.1.2.  Update to be pushed later today.


C++ User Snippet Problem

VSCode Version: 1.5.3
OS Version: WinDows 7

Steps to Reproduce:

I think there is a problem on adding more than one snippet on "C++ User Snippet".
It gives a red underline with an error "End of File Expected" even if the file is ended.
Please solve this issue if there is any if not Sorry for causing inconvenience to you.

Thank You,
Your Happy User.

C++ snippets are contributed by an extension you have installed. Please open a bug report for that extension. Thanks.


Convert website from php to a static site generator

Either in node.js, ruby, or go.
Must include full setup, develop, and run instructions to be merged in.

Note: The current live website exists in the "just-get-it-done-php" branch, not master.

I could help with this!

@infernocloud The website is ready here: https://github.com/jameskolce/lake-afton-website/tree/static-site how do you want to proceed? Maybe creating a gh-pages branch would be ideal, then I can make a pull request there, what do you think?

Very awesome, thank you! I'll make that branch and then this weekend I'll test your pull request and merge it in if it goes smoothly on my local environment.

https://github.com/openwichita/lake-afton-website/tree/gh-pages

Alright! Pull request sent. Let me know how that works or if you have any issue.


[SIMPLIFY-2] Predict id optimization

Add predict_id as parameter to Listener. @jieguangzhou to append design to this issue. @kartik4949 and @blythed to sign off.

Add a predict_id field to the Listener, with a default value of “”. If it is empty, reset it in post_init.
Setting rules:

If version is None, default to listener.identifier.
If version is not empty, default to {listener.identifier}::{listener.version}

Reasoning:
1.	If the user specifies a version, it indicates an explicit need for versioning.
2.	If the user does not specify a version, it suggests no explicit need for versioning.
At the same time, predict_id can be flexibly set:

extraction
extraction::0
extraction::20240628
extraction::tiny-bert / extraction::bert


complexLinks not working correctly

I tried to enable the complexLinks functionality but the site scrolls to the wrong position. However, the anchor navigation does work as expected. see http://feldenkrais-praxis-graz.at/#teachers for example.
does this option depend on any other skrollr options?

Does it work correctly when complexLinks is disabled?
Please include the unminifed skrollr and skrollr-menu sources, otherwise I can't debug.


chore: minify prod CLI

Cuts the bundle size by 50%

@abvthecity there's no diff here -- did you forget to commit the changes?


nvALT crashing with tags

nvALT is reliable crashing when I try to enter a new tag. Right the moment when I enter cmd-shift-T, nvALT is terminated. After restarting my MacBook Pro with High Sierra installed, it worked for about ten or twenty tags and then it returned to crashing. Any idea?
Maybe this is related to #478

Same issue here

Same here. Seems new, as I know I've added tags in the last few months.

Weirdly, this issue seems to have resolved itself. Without any obvious intervention on my part, tags are now working without any crashes. Maybe it was due to an OS update? I am running OS X 10.13.3 but when I first posted I was running a previous point OS X version.

Still happening here, looks like a bad memory allocation or similar
Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_PROTECTION_FAILURE at 0x00007ffeeaa33dc8
Exception Note:        EXC_CORPSE_NOTIFY

Termination Signal:    Segmentation fault: 11
Termination Reason:    Namespace SIGNAL, Code 0xb
Terminating Process:   exc handler [0]

I've been running into this quite a bit lately.


DNS and Routing App settings should be explained

We have a few App Settings controlling DNS resolution in App Service, briefly alluded to in this document.
The following items:

WEBSITE_DNS_SERVER
WEBSITE_DNS_ALT_SERVER
WEBSITE_VNET_ROUTE_ALL

The naming is pretty self explanatory, but surely it wouldn't hurt to have a standalone table explaining what each value does?
I'd propose the Functions App Settings Reference document as an example format:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-app-settings

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@vlkuznet Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@vlkuznet An App Settings reference has been added to the documentation. This also highlights the settings you mention. https://docs.microsoft.com/en-us/azure/app-service/reference-app-settings
Let us know if you are missing other settings in the list. Thanks.

Very nice documentation! Happy to see that!


Error "Cannot read property 'some' of null" in multiple mode

Hi @Sharlaan ,
Just to forward a issue I got over React Most Wanted.
First I was thinking that the issue was in our implementation of SuperSelectField to ReduxForm but it seems that it is a SuperSelectField issue. When using the multiple mode we get the error message:
"Cannot read property 'some' of null" in this part of the code
    _this.handleMenuSelection = function (selectedItem) {
      return function (event) {
        event.preventDefault();
        var selectedItems = _this.state.selectedItems;

        if (_this.props.multiple) {
          var selectedItemExists = selectedItems.some(function (obj) {
            return areEqual(obj.value, selectedItem.value);
          });

It seems that selectedItems are null.
To be sure that the error is not form our implementation to ReduxForm I copied the example from "example2" into our code and the same error appears.
If I find time I will take a look on it myself but if you have it would be great to fix thiss issue. It seems like a issue that could affect everyone who is using the multiple mode.

I think the erro might come from :
componentWillReceiveProps (nextProps) {
    if (!areEqual(nextProps.value, this.state.selectedItems)) {
      this.setState({ selectedItems: nextProps.value })
    }

... and here the proptype definition for value:
const checkFormat = (value) => value.findIndex((v) => typeof v !== 'object' || !('value' in v))
value: (props, propName, componentName, location, propFullName) => {
    const { multiple, value } = props
    if (multiple) {
      const index = checkFormat(value)
      if (!Array.isArray(value)) {
        return new Error(`
          When using 'multiple' mode, 'value' of '${componentName} ${props.name}' must be an array.
          Validation failed.`)
      } else if (index !== -1) {
        return new Error(`
          'value[${index}]' of '${componentName} ${props.name}' must be an object including a 'value' property.
          Validation failed.`)
      }

so if value is set to wrong type (Array when multiple, primitive if not multiple), the above snippet might lead to error if value is set to null for any reason.
Can you give an example to repoduce this issue please, as my examples don't throw ?

Hi,
Just change the SSF in this file to multiple.
Even if you don't use the ReduxForm wrapper it happens.

Ok i reproduced the issue and found out that my initial guess were correct:

the proptype check for value is broken somehow, it should have yelled at me when i put multiple prop in CodeExample1 where only single values are set.
you did not set value type properly depending on multiple mode. More specifically Field seems to be uncontrolled, while SSF is a controlled-only component. This means it requires value and onChange handler. I did not find these props in RMW Task.js

Hi,
Can you try with v1.8.9 : it should spit an error about invalid value prop ?


Trailing delimiter

Looks like a trailing delimiter considered as an invalid response.
For example, if space is a delimiter, such response will not be accepted:
0_1_
// where underscore _ is a space for clarity

Will actually need to look into a different package to trim before split. Thanks

https://github.com/dixonwille/wlog I would like to reference this package as well that I am using for getting answers and printing to string. I may open an issue there to add an enhancement to that package to be able to trim the response by certain characters (spaces, ,, or whatever delimiter you use).


Add a FIR game

[x] Project URL:
[x] One project per pull request
[ ] Insert the project in alphabetical order
[x] Avoid iOS or open-source in description as it is assumed
[x] Use this commit title format if applicable: Add app-name by @github-username
[x] Use approved format for your entry (use 🔶 for Swift projects)

We added this here https://github.com/dkhamsing/open-source-ios-apps/pull/270
Thx!


Yosys Parsing Error in RTL Design File (rtl.v) During Synthesis

Version
Yosys 0.47+86
On which OS did this happen?
Linux
Reproduction Steps
I am encountering a parsing error while using Yosys to synthesize my RTL design file rtl.v. Below is the description of the problem and the steps I followed:
read_verilog rtl.v
synth
write_verilog syn_yosys.v
When running the above commands, Yosys fails to parse the rtl.v file and reports an error. I have attached a screenshot of the error message for reference.

Steps Taken:
I used the command read_verilog rtl.v to load the Verilog RTL file.
The synth command was executed to perform synthesis.
The write_verilog syn_yosys.v command was issued to output the synthesized Verilog file.
However, the parsing error occurs during the read_verilog step, preventing the synthesis process from completing.
Files Attached:
The RTL design file rtl.v (the file where the issue occurs).
System Information:
Yosys version: [0.47+86]
Operating System: [Ubuntu 20.04]
yosys_design.zip
Could anyone please help me understand the cause of this parsing error? Any suggestions on how to resolve this issue would be greatly appreciated. Thank you for your assistance!
Expected Behavior
Successfully parse and complete synthesis, and finally generate a netlist
Actual Behavior
Parsing fail

This is a duplicate of #4427 (compare https://github.com/YosysHQ/yosys/blob/855ac285f49/frontends/ast/ast.cc#L855 vs https://github.com/YosysHQ/yosys/blob/b89bd027a/frontends/ast/ast.cc#L873) which is an issue you are already aware of.
For the future, please check whether you or someone else already reported the same issue before and clearly state when an issue was found using fuzzing.


First draft of a simplification tute.

Not yet finished, but will close #52 when it is. Just making this PR now so that people can look and comment.

Who putting together my ARG poster (https://docs.google.com/presentation/d/1_IcShje2CFbEPbUzodNkgNBv2m37xGx32np_YK7LvfE), I realised that there's a difference between the sort of "simplification" that keeps all nodes present but simply changes edge intervals (i.e. converting a "direct inheritance" graph into a "sample resolved inheritance" graph, and the sort of simplification that we normally mean in tskit, in which non-ancestral nodes are dropped (and potentially, ancestral unary nodes of various sorts are dropped too.
I wonder if we should have a separate word for the first thing, with the assumption that the first stage of running "true" simplification is to ensure that the edges are sample resolved, so that you can identify which nodes can be dropped, etc?

Sorry not have merged this yet @gtsambos - I thought maybe it was worth waiting until @GertjanBisschop had finished thinking about the logical arrangement of the simplify flags.


[release-0.53] [Bugfix]: HyperV Reenlightenment VMIs should be able to start when TSC Frequency is not exposed

This is an manual cherry-pick of https://github.com/kubevirt/kubevirt/pull/8359
Reason for manual cherry-pick
The main conflict was with node labeller test helper functions that did not exist in this release. Therefore, in commit https://github.com/kubevirt/kubevirt/pull/8452/commits/57a0d98d541109f5f68af21c4c8635f0a5ee45df I've added them into the test's local scope. Aside from that, nothing significant is changed.
Release note:

[Bugfix]: HyperV Reenlightenment VMIs should be able to start when TSC Frequency is not exposed

/cc @enp0s3 @Barakmor1

Thanks.
/lgtm

/retest-required

/lgtm

/lgtm

/test pull-kubevirt-e2e-windows2016

/test pull-kubevirt-e2e-windows2016-0.53

/test pull-kubevirt-e2e-windows2016-0.53

/lgtm

/retest-required


Improve the logic of MigrationRuleListener#destroy method

What is the purpose of the change
1.Improve the logic of MigrationRuleListener#destroy method
2.Fix typo

Codecov Report

Merging #8853 (b67d805) into 3.0 (acd49fe) will decrease coverage by 0.15%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##                3.0    #8853      +/-   ##
============================================
- Coverage     63.75%   63.60%   -0.16%
+ Complexity      315      313       -2
============================================
  Files          1169     1169
  Lines         49226    49191      -35
  Branches       7361     7333      -28
============================================
- Hits          31384    31287      -97
- Misses        14418    14466      +48
- Partials       3424     3438      +14

Impacted Files
Coverage Δ

...gistry/client/migration/MigrationRuleListener.java
72.32% <75.00%> (-0.16%)
:arrow_down:

...t/migration/DefaultMigrationAddressComparator.java
78.04% <100.00%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
47.82% <0.00%> (-15.22%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...ubbo/registry/client/AbstractServiceDiscovery.java
87.50% <0.00%> (-12.50%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

...in/java/org/apache/dubbo/common/utils/JVMUtil.java
81.13% <0.00%> (-11.33%)
:arrow_down:

...moting/transport/netty4/NettyEventLoopFactory.java
22.22% <0.00%> (-7.78%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

... and 41 more

Continue to review full report at Codecov.

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


Using the kamstrup module

Hi,
Have you had any success with getting the necessary information from the ESB to install https://doc.smart-me.com/products/kamstrup-module
Thanks,
Ian.

thanks, I'd be interested to access this, too.


set lateral menu to always open in reduced mode

Every time I open files, I have to click on the upper left for it to go from expanded lateral menu to small, it would be nice to configure it to always open the way I prefer it.
this is what e mean to expanded

and this is small

@nvi9 Can you see if this issue is resolved in main?

@nvi9 Can you see if this issue is resolved in main?

@yaichenbaum I can reproduce the same in main.
It seems to be set by the NavigationView control when loading, either the setting what stores the open state was true or false:

this property is set to true by the binding on the control https://github.com/files-community/Files/blob/abde6e2adc71abdc735e80f1d7bb3f4ff2985b17/Files/UserControls/SidebarControl.xaml.cs#L55-L65
what updates the setting here https://github.com/files-community/Files/blob/abde6e2adc71abdc735e80f1d7bb3f4ff2985b17/Files/ViewModels/AdaptiveSidebarViewModel.cs#L42-L58

This issue appears to be resolved in main.


Fix the public path handling

@gnestor, with this change I was able to get https://github.com/gnestor/jupyterlab_table/tree/labextension-build to limp along to this point:

I had to npm install -g css-loader and change application/table-schema+json -> application/json.

I made the change manually in<EMAIL_ADDRESS> Any ideas on what the react error means or why I had to change the mime type?

@gnestor, v0.10.0 has this patch (and moves the loader to JupyterLab per #46).

v0.10.0

v0.11.2?

Correct, v0.10.0 of this repo.

@blink1073 Awesome, thanks for looking into this! Regarding the new React error, I've been running into it with other React components. I assume that it's still related to the webpack config because the jupyter_table extension in notebook doesn't throw this error. In this context it can't be the latter, so it must be the former You are trying to add a ref to an element that is being created outside of a component's render() function. It's very cryptic but I will figure it out 👍

I am seeing this in jupyterlab_table/labextension:
$ npm ls react
<EMAIL_ADDRESS>/Users/ssilvester/workspace/jupyter/jupyterlab_table/labextension
├──<EMAIL_ADDRESS>peer invalid
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
That could be the source of the problem.

Oh no way! Ok, do you know if there is a way to resolve peerDepedency conflicts other than npm shrinkwrap?

  I'm not aware of an automatic way to handle this other than shrinkwrap or yarn.


Fix debug assertion failure on Visual Studio

When compiling and running texrecon in debug mode on Visual Studio, it exits
due to a failed debug assertion. Specifically, "vector subscript out of range".
This is because BVHTree::ssplit() in libs/acc/bvh_tree.h passes a pointer as the
second argument to std::sort() which can point one element past the end of the
vector like an iterator would. Using iterators instead of pointers fixes the
failed assertion.

Thank you :-)


Login dialog - excessive invocation of checkCredentials function

On the login dialog, the password field ng-change binds to checkCredentials(). This means for every character entered, a request is sent to the target Jolokia instance. If a long password is entered, this results in many requests being sent in a potentially very short period of time.
Maybe it's best to just call checkCredentials when the login button is clicked.

I believe @akieling can explain this.

@jamesnetherton @tadayosi @astefanutti
This is not a bug but it's something that could be improved. If I invoke checkCredentials() when the user clicks on the Connect button then the browser blocks the window.open() call. The code in the Promise then function doesn't seem to be in the same context as the code invoked when the button is clicked. To avoid that, I check the credentials when the user is typing the password. When the user clicks on Connect, I already know if the credentials are correct. The problem is that the Jolokia API returns status 403, which gets logged as an error in the console. We could implement a new endpoint in hawtio-war to check the credentials and use different status codes. What do you think? Any other ideas?

Yes, the warnings output to the console log aren't great. When there have been lots of keystrokes, it almost looks like someone's been trying a brute force password hack! For WildFly / EAP I see 2 log entries per typed character.
I still think we should avoid making network calls on every keystroke if we can.

@jamesnetherton I've made some changes. Please test and let me know what you think.

It works (sort of). But the browser login dialog seems to have made a return:

Please make sure you have the latest version of @hawtio/node-backend (3.0.4). Besides, when you run an angularjs app (hawtio-jmx, hawtio-integration, ...) without the hawtio server-side stuff, there is no login page, so we have to use the browser popup in those cases.

Ok... retested. Looks much better. Thanks!


RDCC-320 ensuring emails are made lowercase

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RDCC-320
Change description
Making email addresses lowercase
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

Removing old prs which is not required .


Fixed typo in add moderator.

While going through the docs noticed that there was a typo currrent which should have been current.

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


tools/present: Documentation link to sam documentation is dead

The documentation for the tools/present tool includes a link to http://plan9.bell-labs.com/sys/doc/sam/sam.html, which according to me and according to http://downforeveryoneorjustme.com/plan9.bell-labs.com cannot be reached.
This means it's hard to find out how the .code address syntax works.

Current location https://plan9.io/sys/doc/sam/sam.html

I'll take a stab at fixing this.


Should LRU cache apply to arbitrary objects, not only strings?

One question that immediately comes to mind: during deserialization, should cached objects be (deep)cloned, or should it assign multiple references to the same object?
I'm guessing cloning has the most consistent and predictable behavior when crossing language boundaries (not all languages have references), and it would also match how multiple references to the same object get turned into separate objects after a roundtrip through JSON.stringify/JSON.parse:
    const t = {foo: "bar"}, t2 = {a: t, b: t};
    console.log(JSON.stringify(t2));
   //  ==> {"a":{"foo":"bar"},"b":{"foo":"bar"}}

(sorry if I come across as asking the obvious, but it pays to be explicit and pedantic with these things, right?)

After thinking more about it, I don't think it's worth mixing string references and generic references.
If it becomes absolutely necessary in future, a new tag can be added for that specific purpose


"invalid character '/' looking for beginning of object key string" on open

Installed as instructed, made executable, added slack token; terminal displays:
2018/03/16 13:07:12 invalid character '/' looking for beginning of object key string
May be user error.

@joniler If you copied the config from the README you'll need to remove the comments to make sure the JSON is valid.

Omg, duh. Thanks duder.


edits

[X] Content fix
alert formatting.

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

File
Status
Preview URL
Details

Community/versioning-basics-best-practices.md
:white_check_mark:Succeeded
View

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