Feature Request: Always discard project state when "Restore Previous Windows On Start" is false.

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Hey Atomizers! I wanted to kick a little feature request your way, if it's not too much trouble. I've noticed that in the beta build of Atom, reopening a project in a window with a preexisting tab gives one the option to discard Atom's autosaved project state. I was wondering if it wouldn't be more consistent with the "Restore Previous Windows on Start" setting if, when "restore windows" was flagged false, opening any project---from the menu, via drag-and-drop, from the CLI, from a project management package---automatically dropped its state on the floor.
I'm ultimately trying to get back to a world like the one I enjoyed when setting hot_exit to false in Sublime Text, wherein projects never remember expanded directories or opened tabs. (Unsaved buffers are another story: losing those was a part of the deal with Sublime, but Atom's behavior could in theory be gentler there.) Nineish times out of ten, rather than dropping me helpfully where I was working six minutes ago, the state restoration drops me annoyingly where I was working six weeks ago; it's easier just to get back to where I was on my own.
The behavior is so close to what I'm looking for in 1.17.0-beta3, except that reopening a project in a clean window always restores its tabs (which focus themselves in the Tree View as I close them, ack):

Can't imagine this being super-high-priority by any stretch, but I also don't quite have the Atom-fu to tackle it on my own. Would be grateful if someone looking for an easy PR would consider it. :pray:
Steps to Reproduce

Open a whole mess'a tabs in an Atom project window.
Close the project without explicitly closing your tabs.
Open a new window and reopen that selfsame project.
Tabs. Why did it have to be tabs.

Suggested behavior: When "Restore Previous Windows on Start" is disabled, Atom should discard all preexisting project state---including opened tabs.
Actual behavior: Atom selectively remembers some preexisting project state regardless of the setting---including opened tabs.
Reproduces how often: 100%
Versions
❯ atom --version
Atom    : 1.17.0-beta3
Electron: 1.3.14
Chrome  : 52.0.2743.82
Node    : 6.5.0

❯ apm --version
apm  1.17.0
npm  3.10.5
node 6.9.5 x64
python 2.7.13
git 2.12.2

❯ system_profiler SPSoftwareDataType
Software:

    System Software Overview:

      System Version: macOS 10.12.4 (16E195)
      Kernel Version: Darwin 16.5.0
      Boot Volume: Macintosh HD
      Boot Mode: Normal
      Computer Name: Zbornak
      User Name: Daniel P. Shannon (daniel)
      Secure Virtual Memory: Enabled
      System Integrity Protection: Disabled
      Time since boot: 1 day 12:19

Additional Information
💖

Open a whole mess'a tabs in an Atom project window.
Close the project without explicitly closing your tabs.

Hey @phyllisstein :wave:  Just to clarify step 2 (I couldn't quite follow it in the GIF),  how did you close the project with closing the tabs opened in step 1?

Sure! I mashed ⌘+Shift+W. It looks like the behavior is similar when using the native red "Close" bubble: project reopens with a collapsed Tree View and all its old tabs.

I'm ultimately trying to get back to a world like the one I enjoyed when setting hot_exit to false in Sublime Text

I've been googling this exactly, trying to figure out how to accomplish this in Atom. Many editors still use this as the default behavior, but I've never seen one that didn't least least provide the option in some form. Sublime and VSC both have similar hot exit JSON settings to tweak.
If I have Atom set to open to an empty editor, why would I want it to save edits internally? Opening files later prompts a load/discard confirmation for edits which I don't even recall because I never intended them to be saved internally. It saves it as  a "project", so other tabs that were randomly open are reopened.
I get that you guys have put a lot of effort into the hot exit, so edits save internally as projects. It seems to work pretty well, and I understand it being the default as it is in many modern editors, but every editor should be capable of switching to a traditional save/discard prompt for all open, edited files.

@rsese This is still an issue. Is this on any road map? Thanks for you great work!

Loved Atom for it's customizability, but quit using it because pf the inability to set a classic save prompt and do away with random unintentional unsaved changes, not to mention random "projects" opening when opening a simple file, because those files happened to be open at the same time and are now forever associated. Would love to go back to Atom, but not until there's a simple hot-exit option.

@narcolepticinsomniac Yeah its very annoying. But I still use Atom anyway, lol.

Yes, I agree with the above, please provide a simple tick box in the config to close tabs on exit. Obviously it would need to provide the standard prompt to save any files with changes, and close the others automatically). This is a deal breaker for me, it's very annoying.


Made some small changes

All endpoints are now.
And some other stuff

Have you tested it? No issues that appeared?

It builds


Suggest simulator when no devices

When flutter run is used on a Mac and no devices are specified or attached,
suggest launching a simulator first.
Fixes https://github.com/flutter/flutter/issues/5674
@pq

LGTM but I'd defer to @chinmaygarde on the suggestion details.

lgtm


Update first-timers.md

Description of the change:
Reason for the change:
Link to original source:

Closes #

Please see our issues tab for ideas if you're stuck for things to do. Thanks!


Theme colors: support RGBA

enables transparency

I'll note that rgba is parsed, but it doesn't appear to apply always. It 'works' for background ui colors, though I don't think we do whatever is needed to allow transparent windows so it just is transparent over a white window (presumably how the screen is filled initially).
However, for text, where you'd probably want to use it: it doesn't appear to have any effect. The colors get passed into the range_attribute functions but whatever part of the code in piet/parley that handles colors doesn't appear to properly handle transparency.


AttributeError: 'function' object has no attribute 'write'

subprocess_tee 0.3.1 introduces an attribute error that was not present in 0.2.0:
Task exception was never retrieved
future: <Task finished coro=<_read_stream() done, defined at /root/venv/lib/python3.7/site-packages/subprocess_tee/__init__.py:21> exception=AttributeError("'function' object has no attribute 'write'")>
Traceback (most recent call last):
  File "/root/venv/lib/python3.7/site-packages/subprocess_tee/__init__.py", line 25, in _read_stream
    callback(line)
  File "/root/venv/lib/python3.7/site-packages/subprocess_tee/__init__.py", line 86, in <lambda>
    _read_stream(process.stderr, lambda l: tee_func(l, err, stderr))
  File "/root/venv/lib/python3.7/site-packages/subprocess_tee/__init__.py", line 73, in tee_func
    print(line_str, file=pipe)
AttributeError: 'function' object has no attribute 'write'

Full molecule log: https://cirrus-ci.com/task/4613075711033344?logs=test#L247

I can not reproduce this anymore. I think it was some incompatibility issue with an older molecule version. Latest molecule works fine.


Update Turkish Name Provider

some update for turkish name proivder

Do the new names come from the same www.guzelisimler.com source?
Also, please keep the list alphabetically sorted, it eases spotting the differences.

@fzaninotto Source not same. I will do alphabetically order as soon as possible.

Bump

@fzaninotto I totally forgot that. Right now alphabetical order done 💃

Read the CI report: you have code style issues in your code.

@fatihkahveci bump


Suggestion: Type narrowing also narrows conditional types

TypeScript Version: 2.8.0-dev.20180204
Code
declare function broke(impossible: never): never; // used to ensure full case coverage

interface Foo { kind: 'foo'; }
declare function isFoo(foobar: Foo | Bar): foobar is Foo;
interface Bar { kind: 'bar'; }
declare function isBar(foobar: Foo | Bar): foobar is Bar;

function mapFooOrBar<FooBar extends Foo | Bar, R>(
    foobar: FooBar,
    mapFoo: FooBar extends Foo ? ((value: Foo) => R) : ((impossible: never) => never),
    mapBar: FooBar extends Bar ? ((value: Bar) => R) : ((impossible: never) => never),
): R {
    if (isFoo(foobar)) {
        return mapFoo(foobar);
/*             ^^^^^^^^^^^^^^
Cannot invoke an expression whose type lacks a call signature. Type '((value: Foo) => R) | ((impossible: never) => never)' has no compatible call signatures. */
    }
    else if (isBar(foobar)) {
        return mapBar(foobar);
/*             ^^^^^^^^^^^^^^
Cannot invoke an expression whose type lacks a call signature. Type '((value: Bar) => R) | ((impossible: never) => never)' has no compatible call signatures. */
    }
    else {
        return broke(foobar as never);
    }
}

(the casting foobar as never for the broke function is a workaround for #20375)
Expected Behavior
Compile without errors.
Actual Behavior
The types of mapFoo and mapBar are not reconsidered in light of passing the typeguard:
Cannot invoke an expression whose type lacks a call signature. Type '((value: Foo) => R) | ((impossible: never) => never)' has no compatible call signatures.
Cannot invoke an expression whose type lacks a call signature. Type '((value: Bar) => R) | ((impossible: never) => never)' has no compatible call signatures.
My use case for this is a situation where I have a union of three types, but some cases will only ever use a union of two of those types. I want this kind of utility function to be re-usable for those situations, but indicate that the handlers for unincluded cases will never be called (and allow us to type-safely stub those branches, e.g. with that broke function).
Our project uses these kinds of unions and utility functions a lot and this would significantly improve them.

I think this might be related, but it's reversed -- using a conditional mapped type to narrow the original.
declare function log(s: string): void;

type Transforms<T, S> = {
  [K in keyof T]: T[K] extends S ? (undefined | ((d: T[K]) => S)) : ((d: T[K]) => S);
}

function logData<T>(d: T, keys: (keyof T)[], transforms: Transforms<T, string>): void {
  keys.forEach((k) => {
    const t = transforms[k];
    const dk = d[k];
    if (t) log(t(dk));
    else log(dk); // Argument of type 'T[keyof T]' is not assignable to parameter of type 'string'.
  });
}

Updated this quite a bit: the example I had was poor, the typeof "workaround" wasn't and was never going to work, but unfortunately after addressing all of that, the problem remains. It has a workaround, but type inference doesn't seem to work for the workaround.

If I've read correctly, I think this is working as intended; in the general case, the type of foobar itself doesn't necessarily reflect that FooBar (the type variable) will describe identical types of a given instantiation. For example:
function compare<T>(x: T, y: T) {
  if (typeof x === "string") {
    y.toLowerCase() // appropriately errors; 'y' isn't suddenly also a 'string'
  }
  // ...
}

// why not?
compare<string | number>("hello", 100);

Ah right, I see what you mean. But this is a shortcoming of TypeScript as well, because the actual definitions I've used cannot cause that problem The issue is that TS doesn't recognize that Foo & Bar is actually never since the kind property cannot possibly be 'foo' and 'bar' simultaneously. If it recognized that, it could know that you will never get something that simultaneously isFoo and isBar. That means that if you try to explicitly force the generic parameter FooBar to be Foo, either foobar is a Foo (and thus not Bar) and the never in mapBar is perfectly accurate, or foobar is a Bar which would make the never in mapBar wrong, except that foobar wouldn't then be a valid argument to the function in the first place.
Some notion of that sort of mutual exclusivity also seems to be the primary impediment to #20375, which also came up in this example. If we had that, both of these problems would be tractable.
Furthermore, in this case, I don't even want FooBar as a declared/declarable generic parameter, I would gladly use typeof foobar—except typeof foobar is resolved with the declaration of the function, rather than based on the particular foobar passed in with any given invocation.
For example, what I would have liked was
function mapFooOrBar<R>(
    foobar: Foo | Bar,
    mapFoo: typeof foobar ? ((value: Foo) => R) : ((impossible: never) => never),
    mapBar: typeof foobar ? ((value: Bar) => R) : ((impossible: never) => never),
): R {

but of course, typeof foobar is evaluated immediately as Foo | Bar and so by the time I call mapFooOrBar(foo, it's already too late.
Would suggestions of this mutual exclusivity notion and/or this delayed typeof idea be worthwhile? If so, should they be separate issues, or is post itself a suggestion for them?

@krryan see #14094 for mutually exclusive types


Use postcss and caniuse to remove vendor prefix

Hi,here a little pull request to remove vendor prefix and let caniuse and autoprefixer add them at compile time.
It removes a lot of lines

hi @obsidienne this removes -webkit-transition-property: -webkit-transform; which will kill related css transitions in safari

I think i have been to restrective on the targeted browser (in the grunt file).
Do you have a list of targeted browser ?


Errors when running flatzinc command using the ortools.msc file from the MiniZinc IDE

What version of OR-Tools and what language are you using?
Version: main and 9.3.10470
Language: FlatZinc
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
What operating system (Linux, Windows, ...) and version?
MacOS 12.3.1 on M1 Max
What did you do?
Steps to reproduce the behavior:

Download and install the FlatZinc distribution of or-tools (or build and package from main)
Install the solver into the latest MiniZinc IDE (currently 2.6.3) by adding the directory with the ortools.msc file as a solver search path in the MiniZinc IDE preferences.
Solve any model (for example the simplest with just solve satisfy;)

What did you expect to see
A solved model
What did you see instead?
=== ERROR ===

There where three different things that I needed to fix to get it running

Comment out the two warm_start definitions at the end of the redefinitions-2.0.mznfile (not sure why this is a problem, but also discussed in as a problem in the MiniZinc discussion forums)
Adjust the ortools.msc executable path. It specifies fz as the executable to run which does not exist, not the existing fzn-or-tools
Make sure to not use the "Verbose solving" flag in the Configuration editor, as that sends the -v flag, which the OR-Tools flatzinc runner does not handle (although it is specified as handled in the ortools.msc file)

Known issue. Minizinc before 2.6.1 was missing some definitions for warm starts.
They were added to or-tools, then minizing 2.6.1 readded them, and now they conflict.
It is fixed on the main branch.

I checked in the main branch also and built form there, and all three issues were still present. Links below are to the main branch as of this comment.

Redefinitions file: https://github.com/google/or-tools/blob/5ad0feaa0c92232c33249802b647b90b99108e09/ortools/flatzinc/mznlib/redefinitions-2.0.mzn#L101
The wrong flatzinc binary name for the flatzinc executable: https://github.com/google/or-tools/blob/49b6301e1e1e231d654d79b6032e79809868a70e/cmake/flatzinc.cmake#L163
Indicating that -v is a supported flag: https://github.com/google/or-tools/blob/fa84bc05e72641dddfbb98164d81b8bc9bef6ea5/ortools/flatzinc/ortools.msc.in#L9

Just pushed back on main. The change was overwritten 11 days ago :-(
we will have a look
is ok. I added the -v flags to the binary.

should be fixed on main.
note: I tested it using https://github.com/MiniZinc/MiniZincIDE/releases/download/2.6.3/MiniZincIDE-2.6.3-bundle-linux-x86_64.tgz


Pin the Fedora version in Dockerfile

Pin the fedora base image version to 38 in the Dockerfile, to ensure that CI builds which use the gtk-rs Docker images are reproducible.

We might need the most recent build tools in certain cases as we depend on bleeding edge everything to test the newest additions to the libraries we provide bindings for. So, I don't think this change makes sense.

The image is only meant for bindings CI, using it for anything else is your own responsibility.

Thanks for clarifying that. The idea was to use the gtk-rs image as the basis for Relm4 app builds on GitHub Actions; that way, these apps would not be limited to using the GTK / GNOME versions present on the current Ubuntu LTS release. I believe the same thing could work for building apps that use gtk-rs directly.
Would you be willing to support these use cases, either by enabling the gtk-rs image to be generally used for downstream app builds, or with a different solution e.g. by creating a parallel image with stable versions on it?

Thanks for clarifying that. The idea was to use the gtk-rs image as the basis for Relm4 app builds on GitHub Actions; that way, these apps would not be limited to using the GTK / GNOME versions present on the current Ubuntu LTS release. I believe the same thing could work for building apps that use gtk-rs directly.
Would you be willing to support these use cases, either by enabling the gtk-rs image to be generally used for downstream app builds, or with a different solution e.g. by creating a parallel image with stable versions on it?

You are free to build an image yourself, the dockerfile & the CI file are available for people to copy from but we have no power to get involved in such things.

Let us close this; feel free to re-open if you have stronger motivations


Downloading extensions/themes outside of the NuGet Package Manager

I was wondering if it would be possible to use the NuGet Package Manger outside of VSCode.
Right now @codesandbox has to manually install the extension/theme locally (from the Marketplace) and copy it over to the codesandbox/codesandbox-client repo (like you can see in codesandbox/codesandbox-client#3564 and a lot of other PRs/commits) so they appear in the standalone-packages/vscode-extensions/out/extensions folder.
It would be awesome to just have some sort of package.json file so the dependencies don't have to be installed locally and copied over.
This way there could also a better way to automatically check whether or not an extension/theme has a new version, because right now some extensions are outdated (f.i. CodeSandbox has version 4.10 of Shades of Purple, but it's already at 6.3.0) and we can't know except if we check it manually.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Fix and improve GitLab integration docs

Summary
I've fixed and improved the GitLab integration docs:

Some syntax fixes (especially use of a relative path for the cache directory as required according to GitLab CI cache path docs)
Some more idiomatic syntax and naming

I've split the changes across several commits, intended to be rebase-merged and not squashed, to avoid mixing concerns.
Test Plan
I've created a demo project: https://gitlab.com/sisp/gitlab-uv-demo

Thanks!


[10.x] Adds an app setter to ViewServiceProvider.php

This is the PR from  #51443  but instead into the 10x branch

Please also add a description so Taylor knows what this is about.

@nunomaduro is working on this.


Can't get height of content

I'd love to get the height of dynamic content. I am passing in the text as a prop:
<ReactModal style={this.props.style} isOpen={this.props.open}>
  // Setting inner HTML for some arbitrary bold word

    <input ref="yes" type="button" value="Yes" />
    <input ref="no" type="button" value="No" />

</ReactModal>

Setting styles like:
{
  width: '250px',
  marginLeft: '50%',
  left: '-125px'
}

I want to be able to measure the height so I can apply similar rules for vertical centering.

Hi @DaveStein. Now we have a onAfterOpen() callback. Maybe it can helps with this.

@diasbruno is there a strong reason to, or not to, have an option to allow auto centering? Seems like a common thing people would want and is usually default in other implementations. I can look into using the callback in the meantime

@DaveStein I think css is the best option, but maybe we can have some 'presets' for animations and positions.
import { Style1, ... } from react-modal-presets;

It's been a while since there was any discussion on this issue.  I'm assuming that it isn't a problem anymore.  If it is, feel free to re-open the issue.


Fix export as HTML

This pull request fixes 2 bugs that may occur when trying to export the API documentation as HTML (or MediaWiki, or TracMarkup).

variable [ENTITY] doesn't exist
How to reproduce:
Create a new ColdBox app with the rest template (Relax should come installed with this template):
coldbox create app skeleton=rest

Go to the Relax Petstore page (/relax/api/petstore) and click "Export as HTML" on the right sidebar. You will see the error:

The function [getNormalizedDocument] does not exist
How to reproduce
Create a new ColdBox app with the rest template (like the 1st step).
Go to the Relax cbswagger page (/relax/api/cbswagger) and click "Export as HTML" on the right sidebar. You will see the error:

Thanks so much for this!

@lmajano :  We will need to take a look at the downstream dependencies and, possibly, pin them, now that you've merged this.  The fact that .createDocFromRoutes() in the SwaggerSDK is no longer returning the document object, but is returning a struct has ramfications for other places we use CBSwagger documents. createDocFromRoutes used to return a Document object: https://github.com/coldbox-modules/swagger-sdk/blob/development/models/OpenAPI/Document.cfc

Good point, I marked it to test locally


Null value in externals ID

External Ids return null for IMDB and TVDB.
Firebase and TVRage can get the values.

ID in web:

This error is due to the TMDB API, since they have a cache of more than 8 hours, for this reason, although the information is displayed on the page, the API does not provide the same information.
I had to create a system to connect to the API and review the information in real time.


Integrate ideas to improve performance

https://github.com/cmuratori/refterm/blob/main/faq.md
video of unoptimized terminal being orders of magnitude faster than windows terminal.
https://youtu.be/hxM8QmyZXtg

This is already being handled in #10462 and #10461

Thank you.
We are tracking this feedback and any improvements we make in /dup #10462.


remaining days is showing positive values after expiration

Hi Klaus - thanks for reporting this.
It should be fixed already with version 1.3.1 which was just published to NPM:
https://www.npmjs.com/package/openssl-cert-tools/v/1.3.
Please let me know in case you find any other problem with this lib.


Fixing overflow value in the input component in theme

** Before reporting this bug, please...**

Check the latest documentation: https://docs.nativebase.io/
Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
For discussion purpose make use of NativeBase discord: https://discord.gg/TSgCw2UPmb

Describe the bug
When installing the latest RC branch, we had a problem with the value of the overflow:
https://github.com/GeekyAnts/NativeBase/blob/98ddc36b83c40148a6a18d54eda7fe03a1136a41/src/theme/components/input.ts#L34
We had to switch it to scroll for the time being so we could start using the RC branch.
Can we release a new RC with that value fixed?
To Reproduce
Steps to reproduce the behaviour:

Install the latest of master
Run the server
The issue should show up.

Hi @vytch, Thanks for reporting a issue. We have made changes and they are merged into master.
https://github.com/GeekyAnts/NativeBase/blob/9d1618ed0844addfb4042d6a85a219ca93e27083/src/theme/components/input.ts#L34


create_changelog_release_v010

What type of PR is this?

subproject management

What this PR does / why we need it:
Update of CHANGELOG.md for the release v0.1.0
The merge commit of this PR will be tagged as release v0.1.0
Which issue(s) this PR fixes:

Fixes #22
Special notes for reviewers:
N/A

Seb, Georgios just to kindly ask the approval again as codeowners

Hello @PedroDiez,
the Changelog link https://github.com/camaraproject/BlockchainPublicAddress/commits/v0.1.0
returns a 404 to me

Hello @PedroDiez, the Changelog link https://github.com/camaraproject/BlockchainPublicAddress/commits/v0.1.0 returns a 404 to me

I think it will work after merging this PR and afterwards generating the tag/release, isn't it @rartych?

Thanks guys!!!! Let's go then


Allow filtering of services in discovery client route locator

Do we have implementation for ignoredServices and ignoredPatterns in gateway as in zuul?

The gateway isnt proactive like Zuul is and automatically creates routes based on service discovery they have to be explicitly configured (as far as I know).  Can you provide an example of where this would be necessary?

I'd rather have some metadata that is set that would tell the integration to setup a route (edge=true) or something. @ryanjbaxter internal only services come to mind.

The gateway isnt proactive like Zuul is and automatically creates routes based on service discovery they have to be explicitly configured (as far as I know). Can you provide an example of where this would be necessary?

In myZooi have some services that should be gatewayed' and others that don't and shoud be ingnored-services as in zuul. How to create a functionality like this?

You can now specify an expression that works on the ServiceInstace to test if the particular service should be included in as a route.

Setting spring.cloud.gateway.discovery.locator.include-expression=metadata['edge'] == 'true' will test for a metadata field named edge whose value is true.

You can now specify an expression that works on the ServiceInstace to test if the particular service should be included in as a route.

Setting spring.cloud.gateway.discovery.locator.include-expression=metadata['edge'] == 'true' will test for a metadata field named edge whose value is true.

im sorry, i don't get it.
e.g. if i have 3 services, eureka name: "backend", "stream", "oldbackend" and i want to include discovery of backend and stream, but not of oldbackend.. i would set:
spring.cloud.gateway.discovery.locator.include-expression=metadata['backend','stream'] == 'true'
?

No, you would include a bit of metadata in the discovery configuration of each service that you want in gateway such as 'edge=true'. Then gateway will only include those.

No, you would include a bit of metadata in the discovery configuration of each service that you want in gateway such as 'edge=true'. Then gateway will only include those.

i see.
.. so for my "backend" and "stream" service i'd do
spring:
  cloud:
    discovery:
      client:
        simple:
          local:
            metadata:
              isongateway: true

in the bootstrap.yml of the backend and stream service and then check on the gateway via spring.cloud.gateway.discovery.locator.include-expression=metadata['isongateway']=='true' --- and all other services, though still discovered by eureka, are not reachable via gateway.

@spencergibb @ryanjbaxter this works nice, thanks for support. keep up the good work!

@spencergibb @ryanjbaxter  What is about the IgnoredPatterns? There is some rest interface is internal in some service.

@xiaoshao if you have read thru this issue, you'll see that is not necessary and there is no ignore services in gateway

@spencergibb @ryanjbaxter  do you know if the spring.cloud.gateway.discovery.locator.include-expression works for all DiscoveryClient implementations? i.e Would Spring Cloud Kubernetes discovery work with this?

It works on the API but the implementation, so yes, it should work.

Please open a new issue and we will continue it there


interconnect fails with new state-space systems

Example code:
import control as ct
import control.iosys as io
sys1 = ct.ss(1,1,1,1, inputs='a', outputs='b')
sys2 = ct.ss(1,1,1,1, inputs='b', outputs='c')
interconnected = io.interconnect((sys1, sys2), inputs='a', outputs='c')

gives the error:
File ~/Documents/src/python-control-dev/python-control/control/iosys.py:2507, in interconnect(syslist, connections, inplist, outlist, inputs, outputs, states, params, dt, name, check_unused, ignore_inputs, ignore_outputs, **kwargs)
   2505 connections = []
   2506 for input_sys in syslist:
-> 2507     for input_name in input_sys.input_index.keys():
   2508         connect = [input_sys.name + "." + input_name]
   2509         for output_sys in syslist:

AttributeError: 'StateSpace' object has no attribute 'input_index'

Maybe an attribute is not correctly set if ct.ss is used to instantiate a state-space system?
If instead the systems are explicitly turned into io systems the following works:
import control as ct
import control.iosys as io
sys1 = io.ss2io(ct.ss(1,1,1,1), inputs='a', outputs='b')
sys2 = io.ss2io(ct.ss(1,1,1,1), inputs='b', outputs='c')
interconnected = io.interconnect((sys1, sys2), inputs='a', outputs='c')

Which version of python-control are you using?  I tested on 0.9.2 (latest release) and HEAD of main and it worked OK.

Ok you're right it does work for me on 0.9.2. But not on what I thought is HEAD of main. Maybe I haven't pulled it correctly. Checking.

OK never mind, looks like i am not pulling correctly, apologies!


Please review changes

Whoops, I just merged another PR that changes the Haskell references to use roles, so this has conflicts now. I recommend rebasing. While you're at it, can you squash your changes down into a single commit?

Also, you can review what RTD generated from the link in the PR statuses - that's how I checked that the admonitions were indeed broken!

The tutorial was reviewed from a grammar & text flow perspective. Code lines remains intact.

Merged in https://github.com/input-output-hk/plutus/pull/2160


Features/backend Refactor and prepare for microservices

Microservices and Go can mostly be found united. As we are going to implement a new functionality, it makes sense to learn this concept and implement chat as a separate microservice.
This pull request makes migrate package independent of the forum package, so now we can define migrations for several services at the same time.
Our main forum service also was refactored, I've replaced boilerplate route declarations with the simple route table (inspired by https://benhoyt.com/writings/go-routing/). Now the backend routing became much easier to read and maintain.
No core logic has changed, but as the tests coverage is not too good yet, it makes sense to test it locally. Works great in docker compose and dev environment for me.

Deployment was successful. Thank you for the review! @bomanviktor


How to use custom handlebars to make a counter

Hello!
First of all, thanks for this very usefull task GenerateReleaseNotes Node based Cross Platform Task YAML!
It's really helps a lot, to make powerfull and complete release notes, with workitems in Azure Devops!
Please I need some help, with my script.
I need to make a counter, for example, I need to write the quantity of tasks per type, like 300 tasks, 32 bugs, 30 Product Work Itens and etc
I try to make a function with javascript to make do the count, but I'm not get the way to use this, I follow the example in your wiki, but the Azure Devops says that my script is wrong.
Here my script:
`
{{#forEach this.workItems}}
{{#if (contains (lookup this.fields 'System.Tags') 'RN-BUSINESS')}} {{#if isFirst}}# What's New {{/if}}
{{{lookup this.fields 'System.Description'}}}
{{/if}} {{/forEach}}
      # Task

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Task') }}
      * Fix [**{{this.id}}**](https://xxxxxxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}

      # Improvement

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Improvement') }}
      * Fix [**{{this.id}}**](https://xxxxxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}

      # Product Backlog Item

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Product Backlog Item') }}
      * Fix [**{{this.id}}**](https://xxxxxxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}

      # Development Bug

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Development Bug') }}
      * Fix [**{{this.id}}**](https://xxxxxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}

      # Bugs

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Bug') }}
      * Fix [**{{this.id}}**](https://xxxxxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}

      # Issue

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Issue') }}
      * Fix [**{{this.id}}**](https://xxxxxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}

      # Technical Debt

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Technical Debt') }}
      * Fix [**{{this.id}}**](https://xxxxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}

      # Vulnerability

      {{#forEach this.workItems~}}
      {{#if (contains (lookup this.fields 'System.WorkItemType') 'Vulnerability') }}
      * Fix [**{{this.id}}**](https://xxx/Projetos/_workitems/edit/{{this.id}}): {{{lookup this.fields 'System.Title'}}}
      Total : HERE I need to put the quantity
      {{/if}}{{~/forEach~}}
    delimiter: ':'
    fieldEquality: '='
    anyFieldContent: '*'
    dumpPayloadToConsole: false
    dumpPayloadToFile: true
    replaceFile: true
    getParentsAndChildren: false
    customHandlebarsExtensionCode: module.exports = {foo: function () {return 'Returns foo';}};`

I really don't understand how to use customHandlebarsExtensionCode, note that in this example, I put the code that you mention on Wiki, but when I save, the Azure Devops broke the execution with this message:
/azure-pipelines.yml: (Line: 144, Col: 61, Idx: 5901) - (Line: 144, Col: 61, Idx: 5901): Mapping values are not allowed in this context.
Sorry, if I doing a silly question, but I'm already a few days trying to understand how to do this.
If you need some more information, please let me know!
Thanks
Guilherme

I am not sure that is possible as  in your template you need to store the counts across loops
If you had a simple loop you can use
Total WI {{this.workItems.length}}
{{#forEach this.workItems}}
{{#if isFirst}}### WorkItems with 'Tag 1'{{/if}}
{{#if (contains (lookup this.fields 'System.Tags') 'Tag 1')}}
*  **{{this.id}}**  {{lookup this.fields 'System.Title'}}
   - **WIT** {{lookup this.fields 'System.WorkItemType'}}
   - **Tags** {{lookup this.fields 'System.Tags'}}
   - Index in the loop {{@index}}
{{/if}}
{{/forEach}}

The problem is that the index does not know about the if filter. so you see jumps in the numbers.
Total WI 6
### WorkItems with 'Tag 1'

*  **14**  Bug with 'Tag 1'
   - **WIT** Bug
   - **Tags** Tag 1
   - Index in the loop 1

*  **64**  PBI with 'Tag 1' and "TAG2"
   - **WIT** Product Backlog Item
   - **Tags** Tag 1; TAG2
   - Index in the loop 2

*  **66**  PBI with 'TAG2' & 'Tag 1'
   - **WIT** Product Backlog Item
   - **Tags** Tag 1; TAG2
   - Index in the loop 4

*  **67**  Bug with 'Tag 1' & 'TAG2' & "TAG3"
   - **WIT** Bug
   - **Tags** Tag 1; TAG2; TAG3
   - Index in the loop 5

You need your count to be outside of any loop do you not?
So I guess the custom module will be something like
module.exports = {countItems: function (array) {return array. {count where correct type }};
The problem is going to be the lookup of the field in the array.
I don't have time to experiment at present, but I hope that helps

Yeah, if I have a simple loop I can use the code that you post, even the {{this.workItems.length}} will be perfect for me.
But, in this case, I will have the total of all types of work itens, but I need the total of each kind of work itens, so I think to make a counter inside of each loop, that I previsiouly check the type with de if sentence.
About your question, at the end, I want the value of the count variable outside of each loop, in the end, I think to prepare a table with the values, but as I wrote above, make the count inside the loop, for me was the first option to have the number.
I can use this custom module, but here is the second question that I make in the first post, I really don't understand how to use the custom module.
When I put the code, that you post in your Wiki, in the parameter customHandlebarsExtensionCode the Azure Devops respond with an error like this: (/azure-pipelines.yml: (Line: 144, Col: 61, Idx: 5901) - (Line: 144, Col: 61, Idx: 5901): Mapping values are not allowed in this context.).
I think that I miss something in how to use correctly this feature, if you could detail a little more for me please
Thanks

I discover that the error that Azure Devops respond, with the code that is in the wiki is because of the :
I attached two file, one with the : and other with not.
If I save with the : the execution of the script returns an error, with not the execution is ok, but the script doesn't work

I think the answer is to write an extension that takes the whole workitem array data set and then calculates the number  work items that meet a given criteria so the usage would be similar to the follow
{{countWiByType this.workitem "Bug2}}
Finding all the bugs.
I will write a sample when I get time

How is this
In the template (outside the loops)
- Total WI {{this.workItems.length}}
- Bugs {{countWorkItemsByType this.workItems "Bug"}}
- Product Backlog Item {{countWorkItemsByType this.workItems "Product Backlog Item"}}

And the custom script text is
module.exports = {countWorkItemsByType: function (array, typeName) {return array.filter(wi => wi.fields['System.WorkItemType'] === typeName).length;}};

You could of course alter the filter to check any field

I have started to curate sample custom Handlebars modules with the sample templates

@rfennell Hi!!!
It works perfectly!!!! 🥇
But I need to do little change, like this:
module.exports = {countWorkItemsByType(array, typeName) {return array.filter(wi => wi.fields['System.WorkItemType'] === typeName).length;}};
It not works if you declares countWorkItemsByType: function (array, typeName)..., only works if the function declaration in this form countWorkItemsByType(array, typeName)
Anyway, thank you very much for your attention and help!

I will update the sample to make sure the format is right there too.
Glad we hot you sorted


Implicit evaluation of arguments

In Clojure, arrow macros operate purely on syntax and do not evaluate arguments passed to them.
This is not the case with arrow macros, where me and @digikar99 have found a case of a part of being evaluated, which prevents arrow macros from being used with data that is not supposed to be evaluated.
For example, in Clojure, the below is valid code:
(-> foo (defn [x] (+ x 42)))
(println (foo 24))
;; => 66

arrows do a valid expansion of this example:
CL-USER> (macroexpand-1 '(arrows:-<> foo (defun <> (bar) (1+ bar))))
(DEFUN FOO (BAR) (1+ BAR))
T

arrow-macros perform an invalid expansion:
CL-USER> (macroexpand-1 '(m:-<> foo (defun m:<> (bar) (1+ bar))))
(LET ((M:<> FOO))
  M:<>
  (M:-> M:<>
    (DEFUN M:<> (BAR) (1+ BAR))))
T

Thank you for your helpful report. I probably have fixed the problem for now.


Add const qualifier support for function arguments in shaders (3.x)

3.x version of https://github.com/godotengine/godot/pull/40010. I found out about this while skimming over commits for writing the 4.0 alpha changelog :slightly_smiling_face:
Example:
void test(const int t) {}

@Chaosus The following code gives me an error on both master and 3.x, is this expected?
shader_type spatial;

vec3 test_function_mutable(float red) {
	red = 0.0;
	return vec3(red, 0.3, 0.3);
}

vec3 test_function_const(const float red) {
	//red = 0.0; // This causes an error if uncommented.
	return vec3(red, 0.3, 0.3);
}

void fragment() {
	ALBEDO = test_function_const(0.9);
}

@Chaosus The following code gives me an error on both master and 3.x, is this expected?

Seems like it's bugged and it's allowed in GLSL.

@Calinou You can add the changes from https://github.com/godotengine/godot/pull/55113 to fix that.

@Calinou You can add the changes from #55113 (shader_language.cpp) to fix that.

Done :slightly_smiling_face:

Thanks!


Reduce Allocations

Especially when parsing IRC messages, a lot of Strings are created that are really just substrings of the original message, so a &str would be more efficient. This would make the message struct tied to a lifetime (of the backing String) making it hard to move the struct around. However, there are crates out there that would make this simpler, like yoke.
I think it's worth at least discussing this.
Also see this article on yoke: Not a Yoking Matter (Zero-Copy #1).

I think you're right, that would make a lot of sense. I hadn't heard of yoke, actually.

It may be possible to partially get around this by using inline strings. Something like [SmartString] or [compact_str] may work. It wont be a lot (24 bytes at most) but it's better than nothing.
[SmartString]: https://lib.rs/crates/smartstring
[compact_str]: https://lib.rs/crates/compact_str

Mentioning fast-str here as well.


Request for Multi-Processing Example

Thank you for this amazing work! I've been using this project for a few days now, and it has been incredibly helpful in my work.
I'm currently working on a project where I need to search a large amount of keywords in parallel. However, I'm new to multi-processing and await function. I would like to see a simple example that I can build upon.
Could you please provide some examples that demonstrates how to use multi-processing to search a list of keywords in parallel?Thank you for your help!

Hi, @XWang20.
Added few examples: https://github.com/vladkens/twscrape/tree/main/examples


Set up 2e2 test framework

Fixes #TODO
Modifications
The e2e test for the CRUD operation for PipelineRollout. will have a separate pr for other CR's e2e test

Verification

Working on fixing the failing test. but ran make test to run the test locally

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@3f7eaaf). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage        ?   49.81%
=======================================
  Files           ?       12
  Lines           ?     1371
  Branches        ?        0
=======================================
  Hits            ?      683
  Misses          ?      582
  Partials        ?      106

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


[Core] Enrich Vector with new methods

Methods to be added that support the following:

Remove duplicate values from an array
Find multiplicity of duplicate values
Find intersection of arrays
Find union of arrays
Ascending sort of array values

I would suggest to add the appropriate methods to the IVector interface and implement accordingly to the classes that implement the interface.


A few suggestions

I have created the following fork -
https://github.com/IeuanWalker/Xamarin.Forms.FlexButton
I created this because our designers needed different features, which where -

Use font icons instead of images (do want to look into using FontImageSource , to allow FontImageSource, embedded resources, files, and URIs. But wasn't to sure how to implement this - https://devblogs.microsoft.com/xamarin/introducing-xamarin-forms-4-0-the-era-of-shell/)
Icons either side of the text
Be able to move the icons and text as they wish

In doing so I noticed a few things that could be changed -

The IconOrientation could be removed and use FlexDirection directly
Could also remove icon padding logic based on IconOrientation, allow the user to add there own logic
No need to handle the icon and text using a grid, I ended up replacing this with FlexBasis

FlexLayout.SetBasis(ButtonText, string.IsNullOrEmpty(Text) ?
    new FlexBasis(0, false) :
    FlexBasis.Auto);

Instead of using the Xamarin.Forms frame as the base for the GestureFrame, I used PancakeView and then exposed a few of the pancakeview properties
Added a property to allow the user to control the JustifyContent of the flexlayout
Added FlexLayout.Grow and HorizontalTextAlignment to allow complete control of the placement of the different items within the button

Here is what I am now able to do with FlexButton -

Most of the changes I made are braking changes that's why I havnt created a pull request.
But if you do want me to add or update anything let me know.
Hope this was useful :)

I have a case where the HorizontalTextAlignment property would be very useful.

@magnusblb I am in the middle of creating my own control that gives complete control to the content of the button, ill update you once a beta is ready. Not sure when that will be though.

Thanks for all the suggestions. Some of them like the Font icons (#1) have found their way into new and existing issues. For the other suggestions, I would highly welcome contributions and PRs!

@magnusblb should check out my new control - https://github.com/IeuanWalker/Xamarin.Forms.StateButton


Automatically check for phar updates

Q
A

Issue Type
Feature Request

Deployer Version
v3.1.0

Description
Implement checking for updates on dep command run.
$ dep deploy
> There is a new version of Deployer: 3.0.12 (you are using 3.0.11)

What are you think?

Maybe the deployer should remind you to update the tool from time to time and may provide a self-update method like composer:
$ composer update
> Warning: This development build of composer is over 30 days old. It is recommended to update it by running "composer.phar self-update" to get the latest version.

Now deployer has dep self-update. I thinking of how to remind user to upgrade.


Grade tags not showing

The grade tags only work for one of my classes, but the rest show "undefined"

Could you post the output of the developer console when the issue arises? That would go a long way to helping solve this issue.


[HOLD] [Polish] [OldDotRules Migration] Clean up various amount form logic

Problem
Coming from this comment it seems there is a lot of duplicate logic pertaining to the various "amount forms" in the codebase. With each new feature we have been copying logic related to setting the selection, formatting the amount, etc. Instead of creating a re-usable component.
Solution
Explore refactoring these components so that there is a standard "amount form" that can be used throughout the app.

@marcaaron is there anything that I need to do on this yet?

Nah I'm gonna remove the New Feature label for now sorry for the distraction. I'll re-add it if we end up doing this.

Ok, gonna close this out for now. This was an optional thing that we probably don't need to spend time on right now.


Keep a consistent coding style

GetLeafCerts and GetIntermediaCerts are similiar and a consistent
implementation will be more friendly to those people who wants to read
the code.
Signed-off-by: Hu Keping<EMAIL_ADDRESS>

LGTM. Thanks!

LGTM!


Highlight changes

I'm sure if this has been suggested before but it would be great if the current file could show what has changed since the commit that is shown. This would make code reviews a lot easier ;)

I support this request !
You made multiple good packages in this direction already, please continue to add to the "git colorify the world with my commits and diffs"  ;)

I have looked into adding this in the future.. A previous release included a rudimentary implementation of diffing, however, I removed it (temporarily?) in favor of other plugins that solely handle diff visualization.

So I've looked around for plugins that do this. One of the plugins I found is Atom compare files. But none of them seem to support diffing commits.
I don't know what the best approach is to implement this feature, but I like what you have and adding a similar diffing view to what for example Kaleidoscope has would be pretty amazing.
I'm fully aware of what I'm asking for, and realize this is not a simple feature. But if you have an idea on how to do this I'm more than happy to help you out (with the limited amount of time I have) if necessary.

Not sure if useful, but maybe Mergely could make implementing this feature a whole lot easier?

I released a new version today that includes an initial implementation of diffing. In the git-history settings, just enable 'Show Diff' if it isn't already enabled. it will show you what has changed in your current version of the file vs. the commit you selected.

Awesome, thanks heaps for implementing this so quickly.

Yeah, this is super awesome!

Great! Thanks for the feedback. Enjoy! :+1:


Switched to SLF4J for logging

This change will bump the version to 3.3.1

Is it possible to add some more methods that accept format arguments?
So when TRACE is not enabled, there is no string concatenation done?
(When all events come in, the message is concatenated, but not actually printed, it will save resources and be faster to only do it on demand)
Also, (Simple)DateFormat isn't thread safe.
(I know it might not be possible to change it in this PR, but I'd recommend fixing that)

This is the first iteration of completely replacing logging.
In this step we are adding first support for SLF4J and in the second step we will be removing SimpleLog completely in favor of the Logger interface. In that step we can make use of the SLF4J formatting methods.

👍 Just wanted to make sure, carry on.

This will be merged once categories have been released on the stable discord client to be in sync with the stable category release when bumping the version.


Orchard.Resources module override "bootstrap.css"

I have developed a theme in which in Layout.cshtml i include my customized bootstrap.css with
Style.Include(themeRoot + "Scripts/bootstrap/css/bootstrap.css");
but when i run the page and view page source this is what happened
<link href="/MyProject/Modules/Orchard.Resources/Styles/bootstrap.css" rel="stylesheet" type="text/css" />
Orchard picks Orchard.Resources bootstrap.css instead of my custom bootstrap.css in my custom folder.
I use orchard 1.10. Is this possible?
P.S. Orchard.Resources module is enabled from a dependency (cannot uninstall) and i gave it priority:5 because i had issues with datetime pickers in admin !!!!

Strange, I also use my own bootstrap in dedicated theme but I use less sources modified (from bootswatch + adaptations) it is included in a lib.css so I never load bootstrap.css. Could be a solution.
have you any module requiring bootstrap.css ?

if i rename bootstrap.css ->bootstrap1.css then all working fine. Orchard includes my custom bootstrap1.css.
If i leave the default name bootstrap.css then it picks Orchard.Resources bootstrap.css.
Is orchard performs some weird code for merging duplicate files (css or js) of external assets ?

Try to make your theme to depend on Orchard.Resources.

As @sebastienros mentioned above you need to make your theme to depend on Orchard.Resources.
Also thanks to Orchard, you can override predefined resources such as Bootstrap (by specifying an upper version number).
The later method should be done carefully.

The problem still remain!!!
This is my theme.txt
Name: MyTheme
Author: pda
Website:
Description: MyTheme Bootstrap theme based on PJS.bootstrap
Version: <IP_ADDRESS>
Tags:
Zones: Header, Navigation, Featured, BeforeMain, AsideFirst, Messages, BeforeContent, Content, AfterContent, AsideSecond, AfterMain, TripelFirst, TripelSecond, TripelThird, FooterQuadFirst, FooterQuadSecond, FooterQuadThird, FooterQuadFourth, Footer
Dependencies: Orchard.Resources

**This is my Layout.cshtml**
![2016-04-11_1512](https://cloud.githubusercontent.com/assets/10973438/14426579/a6294b32-fff8-11e5-81ab-fe9a521781dd.png)

**Replaced Bootstrap.css and font-awsome**
![2016-04-11_1513](https://cloud.githubusercontent.com/assets/10973438/14426580/a629f924-fff8-11e5-9245-1e60ee8eff37.png)

@pda-code Check the PJS.bootstrap theme, I assume this is where the selection is made.... Why do you need PJS.bootstrap theme?
Another hint is to turn off shape designer in order to find out exactly where the issues happens.

@urbanit Its not a matter of using an already created theme (the name PJS.bootstrap left as legacy, my theme is created from scratch) nor a shape issue.
With Orchard 1.10, Orchard.Resources caused me problems with Datetime pickers in admin (javascript jquery failures - already reported as an issue by some other user https://github.com/OrchardCMS/Orchard/issues/6674) and now with "magically" replace my version of bootstrap.css with the one that lives in Orchard.Resources module

Please try to create a new theme, create a bootstrap.css, and reference it. If we can make this work, then there is an issue with PJS Bootstrap, nothing we can do, otherwise we'll be able to understand where the issue lies.

Sorry for the inconvenience but i think it is a difficult bug and i try to investigate.
First of all, do not get confused with PJS Bootstrap theme. I made my custom theme from scratch and only copy views from PJS Bootstrap like layout.cshtml, pager.cshtml e.t.c because they already use bootstrap html markup.
No hidden css includes, no hidden cs code. In my theme there is no cs code excepts cshtml views!!!!
Orchard.DashBoards, Orchard.Resources is enabled.
I thing there is a bug in Orchard.UI.Resources.ResourceManager that perform some linq to find resources.
I tried to debug step by step my code and the below pictures speaks for themselves.
For the same function call  WriteResource look the difference between the 2 parameters
ResourceDefinition resource and Url

Looks like a feature to me ;) Just that you don't know this feature. Not saying there is no bug but FYI a css is seen as a shape in Orchard, so that a theme can overwrite the css from another module, even if the module references explicitly a local css. If this is the case then you need to add Orchard.Resources or any module that the css is actually coming from to your dependencies to make it obvious to Orchard that you are rewriting the css. But even though you say you are not using PJS module, you actually are, all the resource manager instructions from your template might be the issue.
I could see a way to fix it, by putting your css file in a different directory such that the full filename doesn't match the ones from the other modules. I will try to do a simple repro locally to identify how to fix it in the Resources module directly, to prevent any obvious conflict.

Thank you...no need to write code.I will put it in a diferrent folder. I am sure it will work fine.
I was trying to prove that maybe this is was a bug. If this is the logic of the code, no problem with that

It might still be a bug, why I said I would investigate.

I'm in the same situation with Orchard v.<IP_ADDRESS>.
In my custom theme I've defined links to bootstrap.css and font-awesome.css with paths to the corresponding files in the styles folder of the theme.
But Orchard still uses the files from Orchard.Resources:

How can I force Orchard to use the files of my theme instead of its own files?

I've seen this behaviour since Orchard 1.8, i'm also cursios what is causing this.

Can you try add a dependency on Orchard.Resources from your Theme manifest? Maybe it will then use the style shape from it.

I keep bumping into this issue as well, ever since Orchard 1.10. I reference custom Bootstrap and FontAwesome in my custom theme. Both my custom module and custom theme have a dependency on Orchard.Resources and the folder structure is different but it keeps using the Orchard.Resources versions simply because the filename is "bootstrap.css" and "font-awesome.css".

Now I finally had time to deal with the issue(s), but it took some time ...
A) Styles...
I was able to solve the issue by locating bootstrap and font-awesome with exactly the same naming and at the same location within my theme (myTheme\Styles<files>), as they are located in Orchard.Resources (Orchard.Resources\Styles<files>).
In case they are located at another location (i.e. myTheme\Styles\bootstrap<files>), the path to Orchard.Resources\Styles... is taken.
Strangely this also happens in case I tried to set the style-path by manifest.DefineStyle("Bootstrap").SetUrl("bootstrap/bootstrap.min.css", "bootstrap/bootstrap.css").SetVersion("3.3.5");
In case they are not located at the same location, Orchard takes the files from Orchard.Resources.
B) Scripts...
In addition, I found out that a bit more strange it does work with the script-files:
I had to change the naming from manifest.DefineScript("jQuery")... to i.e. manifest.DefineScript("jQuery_2.1.4").SetUrl("jquery.min.js", "jquery.js") and then set the dependencies to exactly this name: .SetDependencies("jQuery_2.1.4", ...). By this Orchard takes the files from the theme.
As long as I used for manifest.DefineScript & .SetDependencies the original naming from Orchard.Resources (i.e. "jQuery"), the path to Orchard.Resources/Scripts/... was used.
So for now, at my project it works and I hope you are able to reproduce the behavior also at your side.

You can override a resource using the trick: set version more that already registered. For example SetVersion("1000.1000.1000")
If check the source code of ResourceManager, there is a statement "OrderBy version descending".
private ResourceDefinition FindResource(RequireSettings settings, bool resolveInlineDefinitions) {
            // find the resource with the given type and name
            // that has at least the given version number. If multiple,
            // return the resource with the greatest version number.
            // If not found and an inlineDefinition is given, define the resource on the fly
            // using the action.
            var name = settings.Name ?? "";
            var type = settings.Type;
            var resource = (from p in ResourceProviders
                            from r in p.GetResources(type)
                            where name.Equals(r.Key, StringComparison.OrdinalIgnoreCase)
                            orderby r.Value.Version descending
                            select r.Value).FirstOrDefault();
...

after trying the suggested solutions with no luck, my work around this was to remove the reference to the bootstrap file from YourTheme\Views\Document.cshtml then either replace the bootstrap file you want to be reference there, or you reference it from your razor view Style.Require("BootstrapVersionYouWant")


Identity file being generated by accident

I am getting a spurious -i being generated. seems that expand user ret
on  fedora core python-2.7.5-15.fc20.x86_64

os.path.abspath(os.path.expanduser(""))
'/home/mdupont/experiments/bgtunnel'

@jmagnusson have you had time to look at this?

Sorry for the delay @jmdcal. I've merged your fix now and uploaded version 0.2.0 to PyPI. Thanks for the find!


Remove version: NSIS.NSIS version 3.07

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


Please share the demo's YAML that you tease us with

Please share the YAML for this demo:

It is in the .devcontainer folder

https://community.home-assistant.io/t/custom-header/155399/298


Add Geolocation from behind feature flag

Purpose

[x] Add Geolocation
[x] closes #1694
[x] add text as "Geolocating..." in black text with white background
[x] change icon color to black
[x] change beacon color to black
[x] refactored most failed component due to USWDS upgrade
~still working on getting the beacon to turn off when off screen. One idea is to look for the user.dot class in the DOM during onClick and either remove or hide it, depending on what happens normally.~
[ ] still working on getting external links to have the proper vertical alignment
[ ] change disabled tool tip to "disabled"
[ ] add directional arrow during mobile [options.showUserHeading]
[ ] increase radius of beacon's inner border to X pixels
[ ] waiting on Kameron - new SVGs for waiting, active and background (if possible, checking contrast...)
[ ] get SSL on staging working

QA
In order to get Geolocation working in Chrome on staging we need to allow insecure content (history on why this is can be found here). This can be enabled by

Open a new tab in Chrome and navigate here:

chrome://flags/#unsafely-treat-insecure-origin-as-secure

This will open a page of experimental features:

Enter the following URL in the box as shown above in the

http://usds-geoplatform-justice40-website.s3-website-us-east-1.amazonaws.com/

From the blue drop down, select Enable
Once enabled, a prompt will slide up in the browser, requesting to Relaunch. Click Relaunch

The QA link below should allow Geolocating now🤞

QA round 2 link here
QA spec

[x] Does the Geolocate button work?
[ ] Does the Geolocate button look black in all 4 states? (off, geolocating, locked, and off-center)
[ ] is it clear when the button is "geo-locating"?
[ ] is it clear when the button is finished "geo-locating"?

Note: Once QA is completed, you can return to the tab from Step 1 and disable this feature (returning to the original state of the browser).

Worked on personal Macbook w/ Chrome

it did not work on safari (desktop), chrome (desktop), or chrome (mobile) for me. I got the circle can't use on all three.

maybe I was too late? @vim-usds ?

it did not work on safari (desktop), chrome (desktop), or chrome (mobile) for me. I got the circle can't use on all three.  maybe I was too late? @vim-usds ?

@KameronKerger - I have a feeling we can get it work maybe. Let me know if you have some time to zoom together. I can drop a link

there are still a couple of things
if you are geolocating and you press the button you get stuck in this nothing is happening state?

same thing if you are geolocating in another area and you try to turn it off

do we have to have the blue outline on the icon?
I think the states may need to be updated to: off, in process (geolocating), on - centered, on-not centered

@vim-usds
icon default = leave as is
beacon = 0076d6 with a 4 pt stroke
icon waiting = 0076d6 spinning
icon active = 0076d6
icon panned = 0076d6

@KameronKerger - Thank you!

@KameronKerger
QA round 3 link here
If the icon is disabled, please see the instructions at the start of this thread.
QA spec (desktop)

[ ] When you click on the Geolocate button, does it show you your location?
[ ] is the button's styling correct?
[ ] is the beacon's styling correct?
[ ] is the message during 'geolocating" correct?

the behavior on desktop still seems a little off / funky. if you press the button after you are located it gets stuck in the "locating" step and there's no way to turn it off.

I'd also like to move the button on mobile and desktop as shown in these two screenshots (move closer to the search bar and up on mobile)

I'll have to test the following moving on mobile next time I take odin out for a walk.

mobile tracking is good. it doesn't have direction though... is that easy to add? supported by Mapbox?

@KameronKerger  - I think we will have to be on a newer version of MapBox to get that directional feature on mobile. Also I'm unable to recreate the "2nd-click-on-icon-makes-it-freeze" issue. Maybe we can do a quick screen share today.


add an opt-out config settings for this addon

It looks like it doesn't make sense to add this addon to a volto project and then opt-out from it, but we have this addon through volto-eea-kitkat and this way we can disable it easily, the same way that it can be disabled in volto master

Also we still have sluggified id of headings after this feature is disabled. Hope it shouldn't bother you.

not at all.


improve field component

bump dependencies
fix spelling
fix attribute types
remove stray code
assign class
assign describedby, errormessage
remove required_title attribute
add variation group for required inputs
apply base class to required mark class
apply base class consistently
split test module
set required text as compile-time option, add optional_text option
add missing classes
improve test coverage

Pull Request Test Coverage Report for Build fc5d4463ffaafae5a9c6a20fe4ff0eb07328d64a-PR-399
Details

97 of 109 (88.99%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 81.579%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/doggo/components/field.ex
97
109
88.99%

Totals

Change from base Build cce0094972636e539e4b0b04eab96904e70db040:
0.6%

Covered Lines:
1674

Relevant Lines:
2052

💛  - Coveralls


Font modified to PingFang SC, the console font becomes ugly!

- VSCode Version: Code 1.17.0 (be377c0faf7574a59f84940f593a6849f12e4de7, 2017-10-05T06:07:51.906Z)

OS Version: Darwin x64 17.0.0
Extensions:

Extension
Author (truncated)
Version

react-native-react-redux
EQu
1.4.1

vsc-material-theme
Equ
1.0.4

auto-close-tag
for
0.5.1

auto-rename-tag
for
0.0.14

path-autocomplete
ion
1.6.1

ayu
tea
0.8.0

(5 theme extensions excluded)

Steps to Reproduce:

Reproduces without extensions: Yes/No

Duplicate https://github.com/Microsoft/vscode/issues/35681
Please use a monospace font in the meantime.


HWKAPM-682 Don't show hostname filter

@ammendonca Hi Alex, could you take a look - it doesn't seem to have stopped the hostname filter being displayed.

@objectiser as per https://github.com/hawkular/hawkular-apm/blob/master/ui/src/main/scripts/plugins/directives/filter-sidebar/ts/filterSidebarDirective.ts#L85 it should be no-hosts instead of no-host.

@ammendonca Thanks Alex - that worked. I'll merge once CI finishes.


💄(courses) segment course runs on published course detail page

Purpose
Segment course runs according to status on published course detail page. Show less details for ongoing and archived course run. Sections with no course run are not shown. The draft page segments the runs but shows everything.
Proposal
Separate the course runs according to their state.

open sub and running
open/will open sub and upcoming
closed sub and upcoming/running (less details)
archived (less details)
to be scheduled (not shown unless on draft)
The draft page segments the runs but shows everything.

Fixes #574

@2018D I wasn't sure about some issues so pulled the branch and played with it. I proposed some modifications in a fixup. The result is as follows:

Among the changes I propose:

Rename get_course_runs_state to get_course_runs_dict,
Make sure that get_course_runs_dict is called only once as it is expensive to compute (it was the whole point of computing it as a dict),
Refactor course run fragement templates to make things DRY and self contained,
Only show course run details on the course page when they are open for enrollment, otherwise display as a list,
Course runs displayed as a list must also have a link so that we can visit them,
The "Course runs" title remains a H2 but particular states get a H3,
Future course runs that are open for enrollment go to the main bucket and get an "Enroll now" button,
Future course runs that are not yet open for enrollment go the "Upcoming" bucket,
Future course runs that are no more open for enrollment go to the "Ongoing" bucket (teasing them as upcoming is counter productive...).

All good for me. Thank you for the upgrades to my initial commit.


remove myst_nb

Prevent tutorial from having cell outputs by removing myst_nb

Would you like this to be part of 0.8.0?

The readthedoc passes but does render


Fix compilation under Windows/MinGW64

Dear @arut,
Please merge this PR to compile properly under Windows / MinGW64
This PR also cover #1340
Regards.

Thanks. After hours of stress trying to figure out why it wouldn't build, merging this locally fixed the problem.

hello,
i'm got issues when compiling w64
objs/lib/nginx-rtmp-module-arut-modded-HerveAncher/ngx_rtmp_flv_module.c(508): error C2220: l'avertissement suivant est traité comme une erreur
objs/lib/nginx-rtmp-module-arut-modded-HerveAncher/ngx_rtmp_flv_module.c(508): warning C4334: '<<' : résultat du décalage 32 bits converti implicitement en 64 bits (le décalage 64 bits est-il intentionnel ?)
objs/lib/nginx-rtmp-module-arut-modded-HerveAncher/ngx_rtmp_flv_module.c(521): warning C4334: '<<' : résultat du décalage 32 bits converti implicitement en 64 bits (le décalage 64 bits est-il intentionnel ?)
NMAKE : fatal error U1077: '"C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.33.31629\bin\HostX64\x64\cl.EXE"' : code retour '0x2'
Stop.
any advices ?
thanks.

Hi,
In fact, I never try to compile Nginx with MSVC, I am using MSYS2.
The error MSVC returns telling you that the left shift imply that the temporary created variable potentially become a 64 bits, and to fix that I think you just have to convert:
(1 << h.type)
to:
(ngx_uint_t)(1 << h.type)

Another option is to tell MSVC to do not treat warning as error by removing the flag /WX in compiler option:
https://docs.microsoft.com/en-us/cpp/error-messages/compiler-errors-1/compiler-error-c2220?view=msvc-170

thanks answering so quiclky.
in nginx_rtmp_flv_module.c
i found line 521 this
ctx->msg_mask |= (1 << h.type);
so i need to change by this ?
(ngx_uint_t)(1 << h.type)
thanks.
i will try msys2 asap.

Another option is to tell MSVC to do not treat warning as error by removing the flag /WX in compiler option:
https://docs.microsoft.com/en-us/cpp/error-messages/compiler-errors-1/compiler-error-c2220?view=msvc-170

thanks
i already use w3 and have the issue mentioned.

Hi, In fact, I never try to compile Nginx with MSVC, I am using MSYS2.
The error MSVC returns telling you that the left shift imply that the temporary created variable potentially become a 64 bits, and to fix that I think you just have to convert:
(1 << h.type)
to: (ngx_uint_t)(1 << h.type)

hi !
i modified lines 508 and 521 like you said in ngx_rtmp_flv_module.c
and it's now working like expected (compiling nginx with your RTMP module in 64 bits with MSVC).
Thanks so much


sqlx.In

I want to try addressing one of the elephants in the room in database/sql land, which is dealing with IN type queries.
The problem arises with queries like this:
SELECT * FROM people WHERE id IN (1,2,3,4,5);

In code, you often have this scenario:
ids := []int{}
for _, user := range userGroup {
    ids = append(ids, user.Id)
}
q := `SELECT * FROM people WHERE id IN (?)`

Now we have a problem;  the driver will not handle an []int{} in the way we want here;  our query actually has to look like .. WHERE id IN (?, ?, ...), with a number of bindVars equal to len(ids).  This is incredibly cumbersome.
The proposed solution to this will work similarly to some of the internal-only named query support functions.  This will have the same limitations with its parsing as well:

the ? character will be used as the bindVar, only, regardless of database.
all ? characters are considered bindVars.  If you need to use a literal ?, you cannot use this function

The new function will look like this:
func (d *DB) In(query string, args ...interface{}) (query string, args []interface{}) {
    // each ? corresponding to "slice kind" var gets expanded
}

And be used like this:
// generate an In query with appropriate args []interface{}
q, args := db.In("SELECT * FROM people WHERE id IN (?)", ids)

This would be appreciated. I've actually been handling this by expanding the named parameters and breaking the slice elements out and putting them in the map. Hacky, but effective. Little snippet....
if reflect.TypeOf(val).Kind() == reflect.Slice {
	slc := reflect.ValueOf(val)
	subs := []string{}
	for i := 0; i < slc.Len(); i++ {
		element := fmt.Sprintf("%v", slc.Index(i).Interface())
		if !IsType(element, t) {
		        writeError(w, http.StatusBadRequest,
			"%v is invalid for type %v", val, t)
			return
		}
                newKey := fmt.Sprintf("%v%d", param, i)
			subs = append(subs, newKey)
			params[newKey] = element
		}
		replacement := ":" + strings.Join(subs, ", :")
		thisQuery = strings.Replace(thisQuery, ":"+param, replacement, -1)
        }
}

The current hangup is whether or not to include the binding in the call to In.  I like the interface defined above with sqlx.In(q string, args ...interface{}), but it won't know the driver type and it will mean the extra step of rebinding yourself each time.  Possible interfaces:

sqlx.In(q string, args ...interface{}) - does not do query param binding;  extra step for users of non-QUESTION bindvar databases
sqlx.In(b binder, q string, args ...interface{}) - exposes the Binder interface which might already be exposed, but it's not immediately clear what it is.
sqlx.DB.In(q string, args ...interface{}) - nicer interface, but it's actually a bit unclear why the DB is necessary to outsiders, and I'd probably have to add it to Tx as well for congruence.

Either binding is left to the caller:
q, args := sqlx.In("select * from bar where x in (?)", someSlice)
q = db.Rebind(q)

Or an alternate InBind function is written that takes the binder.
Or it's just hung off the DB & Tx objects.

How would this support NamedExec?
I do an approach similar to @tristanwietsma that converts something like this:
SELECT * FROM people WHERE id IN (:id);

to something like this:
SELECT * FROM people WHERE id IN (:id, :id_1, :id_2, :id_3, :id_4);

which also requires that the arg parameter for NamedExec is a map and that I dynamically add id_1, id_2, etc. to the map.
Your current sqlx.In proposal seems like it would not work with a NamedExec. Or does NamedExec now use sqlx.In under-the-hood when it encounters a slice in its arg parameter?

That's a good question.
The current proposed solutions are to have sqlx.In be a separate manner of pre-processing a query/args list.  With sqlx.BindNamed, you can get from (namedQuery string, arg interface{}) -> (query string, args []interface{}) already, so from there you could use sqlx.In to get the real end result.
I did not think the implementation of BindNamed through, and because of that it's interface is a bit wonky from a user perspective.  It makes a bit more sense from a "I want people to be able to use sqlx to build their database libraries" perspective, but it's still not great.
Currently, it'd just be a small utility to combine them:

func CompileNamedQuery(query string, arg interface{}) (string, []interface{}, error) {
    q, args, err := sqlx.BindNamed(sqlx.QUESTION, query, arg)
    if err != nil {
        return q, args, err
    }
    return sqlx.In(q, args...)
}

I could see myself changing the BindNamed interface in the future;  it exposes bindTypes which most people don't care about (again this was for library-writers not application-writers) and it isn't likely to be widely used.  It's also making me realize that all sqlx query parsing should just standardize on the simpler question bind type for composabilities sake and then let the user rebind at the end when they need to;  if BindNamed used the a binder to bind the query, and you passed it a postgresql connected DB object, you wouldn't be able to compose it with In anymore.

This has been released as sqlx.In


On RBAC failure - no error message is shown

Describe the bug
When RBAC request failed or timeout-ed no error indication is shown or displayed on the screen.
To Reproduce
Steps to reproduce the behavior:

Go to 'Cost models'
Wait for a timeout
See error -> the create cost model button is disabled but the user permissions has write permissions. In addition look at the Network tab in Dev-tools and see that the request was timeout-ed.

Expected behavior
After talking with Nataly and Sergio - Show a toast alert message.

verified


kong.Response.Exit not picking up response header map

I've successfully set up a Kong Go custom plugin, but am finding that my kong.Response.Exit responses don't have any headers. Take a simple Go plugin, called go-plugin-test.go:
package main

import (
	"fmt"

	"github.com/Kong/go-pdk"
	"github.com/Kong/go-pdk/server"
)

type Config struct {
	Message string
}

func New() interface{} {
	return &Config{}
}

const Version = "1.0.0"
const Priority = 1

func main() {
	server.StartServer(New, Version, Priority)
}

func (conf Config) Access(kong *pdk.PDK) {
	body := fmt.Sprintf("{\"message\": \"%s\"}", conf.Message)
	headers := make(map[string][]string)
	headers["Content-Type"] = append(headers["Content-Type"], "application/json")
	headers["X-Hello-From-Go"] = append(headers["X-Hello-From-Go"], "hello")
	kong.Response.Exit(200, body, headers)
}

I've been running it with kong.yaml:
_format_version: "2.1"
_transform: true

services:
- name: test-service
  url: https://httpbin.org/get
  routes:
  - name: test-route
    paths:
    - /test-service
plugins:
  - name: go-plugin-test
    service: test-service
    config:
      message: "hello world"

A simple kong.conf:
plugins = bundled,go-plugin-test
pluginserver_names = go-plugin-test
pluginserver_go_plugin_test_query_cmd = /usr/local/bin/go-plugin-test -dump
admin_listen = <IP_ADDRESS>:8001 reuseport backlog=16384, <IP_ADDRESS>:8444 http2 ssl reuseport backlog=16384

And a Dockerfile:
FROM kong:latest

USER root

ENV KONG_DATABASE=off
ENV KONG_DECLARATIVE_CONFIG=kong.yml
ENV KONG_ADMIN_LISTEN="<IP_ADDRESS>:8001, <IP_ADDRESS>:8444 ssl"

COPY go-plugin-test /usr/local/bin/go-plugin-test
COPY kong.yml /
COPY kong.conf /etc/kong/kong.conf

EXPOSE 8000
EXPOSE 8001

ENTRYPOINT [ "/docker-entrypoint.sh" ]
CMD [ "kong", "docker-start" ]

I've built & run with:
> GOOS=linux GOARCH=amd64 go build .
> docker build -t kong-test .
> docker run -p 8000:8000 kong-test

And if I curl the endpoint, I get the 200 response I want, but no Content-Type, or X-Hello-From-Go header:
> curl -v localhost:8000/test-service
*   Trying ::1:8000...
* Connected to localhost (::1) port 8000 (#0)
> GET /test-service HTTP/1.1
> Host: localhost:8000
> User-Agent: curl/7.77.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Date: Fri, 28 Jan 2022 17:46:10 GMT
< Connection: keep-alive
< Content-Length: 26
< X-Kong-Response-Latency: 54
< Server: kong/2.7.0
<
* Connection #0 to host localhost left intact
{"message": "hello world"}

seems related to #83, which was fixed by Kong/kong#8267

Closing this due to the lack of activity since Javier's reply. Please reopen if the issue persists in newer Kong versions.


Word-Add-in-JavaScript-InvoiceManager control binding and table fillin problem

I'm having a problem following the OfficeDev/Word-Add-in-JavaScript-InvoiceManager found in github. can somebody please direct me to a forum or dev center office where I can ask questions.
the specific problems I'm having are:

how to bind the controls in the word document to a SharePoint list.
how to create a dynamic table control collections so it can be filled from list items.

these are critical as I have a customer waiting for a solution and I have not been able to find anything on the subject anywhere!
Thanks for your help

@dcu-sharepoint
If I understand the issue that you have raised, you are, in effect, requesting a new sample that illustrates a solution your two problems. I cannot give you a timeline, but I will investigate whether this sample can be changed to do this without interfering with its basic purpose. Alternatively, perhaps a new sample can be developed that will help.
Please note that creating an Issue on GitHub is mainly intended as way to report bugs in the sample. To request a new sample or significantly enhanced sample, please use Office User Voice. Here's an example: error-code-message-documentation

I have found another sample that shows table binding with variable data: Excel-Add-in-Bind-To-Table. So, it doesn't look like there's much point to updating this sample, so I will close this issue.


Regarding getting TFS WorkItems using Ids

Regarding getting TFS WorkItems using Ids
I was using rest api for tfs to get workitems details by ids but when i am giving all the ids the api is turning to be very large and hence its not processing and returning internal server error message.
My concern is that can you people make this api as post request as well so that i can pass as many ids as we can.
Thanks,
Sangam

@Powerhelmsman   The doc link i used to get the api is https://docs.microsoft.com/en-us/rest/api/vsts/wit/work items/list
In the api  GET https://{accountName}.visualstudio.com/{project}/_apis/wit/workitems?ids={ids}&api-version=4.1-preview    if i am giving lots of ids comma seperated then its failing because of url length.
Please help me out with some alternate solution

It will be good if api is like getting all the work item details of specific project without specifying one one ids of each work items. We are having python based addon to pull data from tfs account and index it in splunk. This i want basically for Splunk addon.

@erickson-doug Can you help here?

@erickson-doug Any update?

Sent a email to @erickson-doug to follow up

@wnjenkin @elbatk Can you help assign to the right person?

@sangam-sgm Your issue is transferred to vsts-rest-api team. They will help you there. I will close as transferred. If you have other question related to this issue, we can reopen it and continue discussion.


Deploy to heroku failed

First of all, thanks for your work. Help me so much.
When I try to deploy your app to heroku, i get this error.
ERROR in ./app/components/Header/images/banner.jpg
remote: Module build failed (from ./node_modules/image-webpack-loader/index.js):
remote: Error: write EPIPE
remote:     at WriteWrap.afterWrite (net.js:835:14)
remote:  @ ./app/components/Header/Header.js 13:0-41 21:7-13
remote:  @ ./app/components/Header/index.js
remote:  @ ./app/containers/App/App.js
remote:  @ ./app/containers/App/index.js
remote:  @ ./app/app.js
remote:  @ multi ./app/app.js
remote: Child html-webpack-plugin for "index.html":
remote:      1 asset
remote: npm ERR! code ELIFECYCLE
remote: npm ERR! errno 2
remote: npm ERR<EMAIL_ADDRESS>build: `cross-env NODE_ENV=production webpack --config config/webpack.prod.babel.js --color -p --progress --hide-modules --display-optimization-bailout`
remote: npm ERR! Exit status 2
remote: npm ERR!
remote: npm ERR! Failed at the<EMAIL_ADDRESS>build script.
remote: npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
remote:
remote: npm ERR!      Push rejected, failed to compile React.js (create-react-app) multi app.
remote:
remote:  !     Push failed
remote: Verifying deploy...
remote:
remote: !	Push rejected to bm-migrator.
remote:

Do you know what could be?
Only worked when I removed Header component.

@manoelneto This should be related to libpng16-dev OS dependency required for image-webpack-loader v4 and above. If you could figure out a way to install that in Heroku through a script or any other way before running npm install, this error won't happen probably.

This won't happen again. We have now removed the buggy image-webpack-loader altogether from this repo. 55fad532b589612bd638eae27d919f7fa3662fcb


x/tools/gopls: spurious 'unused write' in package reflect

I opened reflect/type.go and gopls started telling me:
/Users/rsc/go/src/reflect/type.go:1795.5,1795.5: unused write to field Dir
/Users/rsc/go/src/reflect/type.go:1798.5,1798.5: unused write to field Elem
/Users/rsc/go/src/reflect/type.go:2816.8,2816.8: unused write to field Elem
/Users/rsc/go/src/reflect/type.go:2831.8,2831.8: unused write to field Slice

This is false. The code for the first two errors is:
// Make a channel type.
var ichan any = (chan unsafe.Pointer)(nil)
prototype := *(**chanType)(unsafe.Pointer(&ichan))
ch := *prototype
ch.TFlag = abi.TFlagRegularMemory
ch.Dir = abi.ChanDir(dir)
ch.Str = resolveReflectName(newName(s, "", false, false))
ch.Hash = fnv1(typ.Hash, 'c', byte(dir))
ch.Elem = typ

ti, _ := lookupCache.LoadOrStore(ckey, toRType(&ch.Type))
return ti.(Type)

What the analyzer doesn't see is that &ch.Type provides access to all of ch.
/cc @rfindley @adonovan

CC @timothy-king
I suspect it would be enough for the analyzer to differentiate between implicit field access and explicit field references, bailing out for the latter. That would preserve most of the usefulness and avoid these types of false positives.
This looks (naively) like a pretty straightforward fix. Putting in the<EMAIL_ADDRESS>milestone.

What the analyzer doesn't see is that &ch.Type provides access to all of ch.

toRType involves a C-stylt case to coerce a *T a pointer to a *struct{T; ...} pointer. I don't expect the analysis to be able to handle that, nor do I imagine this is a common thing to do in Go code outside the runtime.
Perhaps the analyzer should just be silent in packages that import "unsafe"?

Perhaps the analyzer should just be silent in packages that import "unsafe"?

That was my first thought as well, but there's no reason the cast must be in the same package as the field addr.

there's no reason the cast must be in the same package as the field addr.

Well, there kind of is: it is very bad form to expose unsafe public API.
There could be a constellation of internal packages working together this way, but that may be rare enough to justify the heuristic.

What the analyzer doesn't see is that &ch.Type provides access to all of ch.

toRType involves a C-style cast to coerce a *T pointer to a *struct{T; ...} pointer. I don't expect the analysis to be able to handle that, nor do I imagine this is a common thing to do in Go code outside the runtime.
Perhaps the analyzer should just be silent in packages that import "unsafe"?

According to the information available here a significant number of  packages import unsafe

@MikeMitchellWebDev pkg.go.dev reports over 4m importers of fmt, so the number of packages importing unsafe is at most 2-3%, probably more like 1-2%. Skipping this specific heuristic in the presence of unsafe seems acceptable.

Circling back, I'm rethinking this after making the above CL.
Why don't we just mark all fields in the value as live if any of them are address-taken? That seems like a more principled fix, and I still think the analyzer would catch most of the bugs it currently catches.

Circling back, I'm rethinking this after making the above CL.
Why don't we just mark all fields in the value as live if any of them are address-taken? That seems like a more principled fix, and I still think the analyzer would catch most of the bugs it currently catches.

In the course of reviewing your CL I have been studying this analyzer, which is sorely lacking in documentation. Its approach is to work back from stores of the form x.f=v and x[i]=v to find all references to x. If it finds a non-storing use of x it considers the original store x to be live. Unsafe pointer conversions u.P(&x) are not considered to make x.f live, nor are unsafe.Pointer conversions of the first field or element, u.P(&x.field0) or u.P(&x[0]), which can be coerced to &x.
The analyzer only works on simple local control flow because it has no alias analysis. So, if cond { ptr=new(T) } else { ptr=nil } would confuse it because the value of ptr after the if statement is a phi, which is distinct from new(T).
I think the analyzer would be clearer expressed as a forwards analysis (instead of using SSA referrers) that marked every storing and non-storing use of each object and field, as this would allow it to see unsafe.Pointer conversions easily, and to traverse through phi nodes conservatively.
Back to your question: I suspect marking the all fields live if any is address-taken would be far too conservative. In any case it would require a similar restructuring to what I just described.


Add custom User-Agent to app's axios calls

I have my own node I run ReStake against, and when analyzing node's logs, I see entries like these in nginx logs:
<source-ip> - - [02/May/2024:01:19:35 +0200] "GET /cosmos/distribution/v1beta1/delegators/<address>/rewards HTTP/1.1" 200 335 "-" "axios/1.3.4"

I know that it's ReStake doing these, but having ReStake somehow identifying itself and its headers via User-Agent header would be nice, to differentiate these from other queries, as if some other JS-based tool using axios as a library to fetch data from would also query my node, it would be impossible to distinguish the two.
(Feel free to assign me if you want, I can make a PR adding it.)

Thanks @freak12techno - I've been meaning to do this for ages! Thanks for raising the issue, I'll get to it ASAP but won't be for a week or so, so please do have a go at a PR if you would like it sooner


up_squared: samples/boards/up_squared/gpio_counter run failed

Describe the bug
Test case in samples/boards/up_squared/gpio_counter runs failed.
To Reproduce
Steps to reproduce the behavior:

Change BIOS settings as samples/boards/up_squared/gpio_counter/src/main.c describes.
make
Boot up_squared board over network
See error

Checking Media Presence......
Media Present......
Downloading NBP file...
Succeed to download NBP file.
*** Zephyr EFI Loader ***
RSDP found at 0x79c34000
Zeroing 4816288 bytes of memory at 0x106000
Copying 32768 data bytes to 0x1000 from image offset
Copying 24576 data bytes to 0x100000 from image offset 32768
Copying 2110048 data bytes to 0x59dda0 from image offset 57344
Jumping to Entry Point: 0x1137 (48 31 c0 48 31 d2 48)
*** Booting Zephyr OS build zephyr-v3.2.0-412-g3d85ae37b6ee  ***
counter: 0x0

Environment (please complete the following information):

OS: (Linux)
Toolchain (Zephyr SDK 0.15.0)
Commit SHA or Version used 3d85ae37b6ee

After given the HAT pin16 an input signal, I saw following result:
>>Checking Media Presence......
>>Media Present......
 Downloading NBP file...

  Succeed to download NBP file.
*** Zephyr EFI Loader ***
RSDP found at 0x79c34000
Zeroing 4816288 bytes of memory at 0x106000
Copying 32768 data bytes to 0x1000 from image offset
Copying 24576 data bytes to 0x100000 from image offset 32768
Copying 2110048 data bytes to 0x59dda0 from image offset 57344
Jumping to Entry Point: 0x1137 (48 31 c0 48 31 d2 48)
*** Booting Zephyr OS build zephyr-v3.2.0-1233-g39bb5cd06a08 ***
counter: 0x0
counter: 0x1
counter: 0x2
counter: 0x3
counter: 0x4
counter: 0x5
counter: 0x6
counter: 0x7
counter: 0x8
counter: 0x9
counter: 0xa
counter: 0xb
counter: 0xc
counter: 0xd
counter: 0xe
counter: 0xf
counter: 0x0
counter: 0x1
counter: 0x2
counter: 0x3
counter: 0x4
counter: 0x5
counter: 0x6
counter: 0x7
counter: 0x8
counter: 0x9
counter: 0xa
counter: 0xb
counter: 0xc
counter: 0xd
counter: 0xe
counter: 0xf
counter: 0x0
counter: 0x1
counter: 0x2

It works when I manually trigger the input. I think this is a expected result, @dcpleung , could you please help to confirm? Thank you very much!

I don't have an UP2 board to test this. But the output looks correct.

Thanks for comments from @dcpleung . A manually input triggering works. So I think this is not a bug. Double checking with a output loopback.

Close it because this is triggered successfully manually on UP2.


Having trouble generating speech recognition adversarial samples.

I am attempting to run the following code snippet. I am getting a size error with the tensor for attack generation. However, the X I query works for making predictions on the model. Am I misunderstanding the formatting of arguments for the attack generation function?
test_dataset = torchaudio.datasets.LIBRISPEECH("./", url="test-clean", download=True)
X = np.array([x[0].cpu().detach().numpy().flatten() for x in test_dataset])
y = [x[2] for x in test_dataset]

# load model
model = PyTorchDeepSpeech(pretrained_model='librispeech')

# measure accuracy on test data
predictions = model.predict(X,transcription_output=True)

# Load Attack
attack = ImperceptibleASRPyTorch(estimator=model)
y_tar = np.array(['HELLO' for x in range(len(X))])

# Generate adversarial samples
adv_xs = attack.generate(X,y)

Error:
/usr/local/lib/python3.6/dist-packages/art/attacks/evasion/imperceptible_asr/imperceptible_asr_pytorch.py in _forward_1st_stage(self, original_input, original_output, local_batch_size, local_max_length, rescale, input_mask, real_lengths)
    446         local_delta = self.global_optimal_delta[:local_batch_size, :local_max_length]
    447         local_delta_rescale = torch.clamp(local_delta, -self.initial_eps, self.initial_eps).to(self.estimator.device)
--> 448         local_delta_rescale *= torch.tensor(rescale).to(self.estimator.device)
    449         adv_input = local_delta_rescale + torch.tensor(original_input).to(self.estimator.device)
    450         masked_adv_input = adv_input * torch.tensor(input_mask).to(self.estimator.device)

RuntimeError: The size of tensor a (10000) must match the size of tensor b (166960) at non-singleton dimension 1

Hi @aaa6237 Thank you for using ART!
I think the origin and preprocessing of the data is important for this model. Have you already looked at our ASR notebook demonstrating the attack for this estimator: https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/notebooks/asr_deepspeech_examples.ipynb
Let us know if it works for you!


[Bug] Restore jobs fails to restore backup on NFS

Documentation

[x] I acknowledge that I have read the relevant documentation.

Describe the bug
When running CRD as below
apiVersion: k8s.mariadb.com/v1alpha1
kind: Restore
metadata:
  name: db1-mariadb-restore
  namespace: default
spec:
  mariaDbRef:
    name: db1-mariadb
  args:
  - --verbose
  resources:
    requests:
      cpu: 100m
      memory: 128Mi
  targetRecoveryTime: "2024-12-27T10:00:01Z"
  volume:
    nfs:
      path: /volume1/db1_backups
      server: nas.local

During processing creating intermediate file 0-backup-target.txt  referencing backup SQL file to be imported e.g. backup.2024-12-27T10:00:01Z.sql
Second container in restore job pod use that file via "cat" to extract actual file name. Issue is it however report an problem
mariadb bash: line 1: backup.2024-12-27T10:00:01Z.sql: No such file or directory
Issue is that default container path is not /backup mount point, but I guess root, so exact path should be specified in 0-backup-target.txt file e.g.:
/backup/backup.2024-12-27T10:00:01Z.sql
Expected behaviour
exact path provided in 0-backup-target.txt file .e.g.
/backup/backup.2024-12-27T10:00:01Z.sql
Steps to reproduce the bug

Try Restore CRD with NFS mount point

apiVersion: k8s.mariadb.com/v1alpha1
kind: Restore
metadata:
  name: db1-mariadb-restore
  namespace: default
spec:
  mariaDbRef:
    name: db1-mariadb
  args:
  - --verbose
  resources:
    requests:
      cpu: 100m
      memory: 128Mi
  targetRecoveryTime: "2024-12-27T10:00:01Z"
  volume:
    nfs:
      path: /volume1/db1_backups
      server: nas.local

Was not on latest version


C++ examples README cleanup

Contributes to https://github.com/grpc/grpc.io/issues/180.
cc @ejona86 @jtattermusch @srini100

@jtattermusch thanks for the review.
All: will one of you be merging this? (I can't merge over this repo)


interpolate does not exist on property x

Hi everyone,
in a typescript setup on this line from the Trails example with renderprops
transform: x.interpolate(x => translate3d(${x}%,0,0)),
typescript screams that interpolate doesn't exist on property x.
pretty sure i am making a rookie mistake here.
thanks,
Cezar

interpolate is part of it, but maybe something in the types broke. do you understand any of this enough to check? https://github.com/react-spring/react-spring/blob/master/types/renderprops-universal.d.ts
@jacobrask @salguerooo

not sure. the definitions are quite deep. i am both not familiar with the entire react-spring api and typescript to make a judgement. :)

Unfortunately I'm not sure I can follow the Trail typings completely yet either, but we're in the process of rewriting the library in TypeScript, which means TypeScript can do more of it's own inference and the types will be a bit simpler. Until then you might occasionally need to add some manual type annotations to your code.

Thanks @jacobrask
All the best in the rewrite. I am sure these will be the toughest types you’ve written :)