Sub-type type

I have two use cases that I cannot seem to type correctly in TypeScript. Both use cases involve functions that take a parameter of type "an object with any subset of the members of interface X".
Use case 1: options merging
interface Options {
  a: string;
  b: string;
}

const defaults: Options = {
  a: "a",
  b: "b"
}

function merge<T>(base: T, additional: <subtype of T>): T {
  // clone all members of 'additional' onto 'base'
}

let opts: Options = merge<Options>(defaults, { b: "some other b" });

Use case 2: React state
See also https://facebook.github.io/react/docs/component-api.html
A React component is a class with three generic parameters Props, State and Context. The base class defines a setState(newState: State) method. In reality though, the setState() method accepts any sub-type of State.
This becomes particularly painful when having multiple layers of inheritance in React components:
BaseWidget.tsx
// one React component
class BaseWidget<P extends BaseWidget.Props, S extends BaseWidget.State, C extends BaseWidget.Context> extends ReactComponent<P, S, C> {

	public componentWillMount()
                 // valid React call, TypeScript error 'argument of type foo is not assignable to parameter bar'
		this.setState({ a: "string" });
	}
}

module BaseWidget {
	"use strict";

        export interface Props extends React.Props<BaseWidget<Props, State, Context>> {
	}

        // note that all members must have ? on them to work
	export interface State {
            a?: string;
	}

	export interface Context {
	}
}

export = BaseWidget

I think you mean a supertype, not a subtype. { b: string } is a supertype of Options.
In TypeScript 1.8, you'll be able to write
interface Options {
    a: string;
    b: string;
}

const defaults: Options = {
    a: "a",
    b: "b"
}

function merge<T extends U, U>(base: T, additional: U): T {
    return base;
}

let opts = merge(defaults, { b: "some other b" })

Try it out by running npm install -g typescript@next.
Does this address your use cases?

Hi @DanielRosenwasser thanks for your reply. It definitely addresses the first use case but not the second. This is because it is not possible to explicitly mention the supertype (and you were of course right about it being a supertype) and therefore you cannot make a proper signature for the setState method.
Using public setState(newState: S): void would not work; the React library allows any supertype of S, and you have to be able to state that somehow.

@rogierschouten What about this? (didn't try)
public setState<S extends T>(newState: T): void

@rogierschouten what if you just made all the properties optional? I think that's the usual approach.

@lbguilherme do you mean this? It generates the compile errors below.

/**
 * This class is a given by the React framework
 */
class ReactComponent<S> {
    protected _state: S;

    /**
     * This method actually accepts any supertype of S
     */
    protected setState<S extends T>(newState: T): void {
        for (let name in newState) {
            if (newState.hasOwnProperty(name)) {
                this._state[name] = newState[name];
            }
        }
    }

    protected componentWillMount(): void {
        // abstract
    }

}

/**
 * Some state interface declaration. Note all members are optional to allow setState to
 * be called with supertypes of BaseState
 */
interface BaseState {
    a?: string;
}

/**
 * My own base class for certain React widgets
 */
class BaseWidget<S extends BaseState> extends ReactComponent<S> {

    constructor() {
        super();
        this._state = {};
    }

    protected componentWillMount(): void {
        this.setState({ a: "boo" });
    }
}

$ tsc v2.ts
v2.ts(11,34): error TS2304: Cannot find name 'T'.
v2.ts(11,47): error TS2304: Cannot find name 'T'.
v2.ts(40,9): error TS2322: Type '{}' is not assignable to type 'S'.

@DanielRosenwasser your solution only works when you don't have an intermediary base class:

/**
 * This class is a given by the React framework
 */
class ReactComponent<S> {
    protected _state: S;

    /**
     * This method actually accepts any supertype of S
     */
    protected setState(newState: S): void {
        for (let name in newState) {
            if (newState.hasOwnProperty(name)) {
                this._state[name] = newState[name];
            }
        }
    }

    protected componentWillMount(): void {
        // abstract
    }
}

/**
 * Some state interface declaration. Note all members are optional to allow setState to
 * be called with supertypes of BaseState
 */
interface BaseState {
    a?: string;
}

/**
 * My own base class for certain React widgets
 */
class BaseWidget<S extends BaseState> extends ReactComponent<S> {

    constructor() {
        super();
        this._state = {};
    }

    protected componentWillMount(): void {
        this.setState({ a: "boo" });
    }
}

$ tsc v1.ts
v1.ts(39,9): error TS2322: Type '{}' is not assignable to type 'S'.
v1.ts(43,23): error TS2345: Argument of type '{ a: string; }' is not assignable to parameter of type 'S'.

@rogierschouten Specifying a T will work:
protected setState<T, S extends T>(newState: T): void

But... this is too unrestricted. When S is {a: string}, setState({a: 4}) will work just fine. And for setState adding new fields is also not acceptable. Ideally there must be some language construct to turn all fields into optionals, I think flow has something about this.

@lbguilherme Yes I agree, there should be another language construct. Can somebody maybe remove the Question label from this issue and put something in that reflects this?

At this point I'd prefer we have a new issue putting forth a concrete proposal (hopefully specifically addressing typing of setState, which is indeed problematic) - there's a lot going on here and it would be confusing to read top-down at this point.

same issue here. If you use React + Typescript and you have some React components extending other components and you try to properly define the states with interfaces (with all properties being optional) it leads exactly to the problem described as usecase 2 above.
In other words, even though you provide a proper state object {someStateProp:true}, and all properties are optional still leads to the error:
Argument of type '{ someStateProp: boolean; }' is not assignable to parameter of type 'S'.
Sadly the only solution that I could find to make it work for now is to turn off the typecheck by manually editing react.d.ts and replacing S with any: setState(state: any)

FYI: Looks like this has been resolved with #12114.


Change booleans into TriStates

Using TriState allows for nicer denying behavior. Also updated the mod to 1.15.1 and bumped the version to 2.0.0 (breaking changes).

I've actually got this already set up on my local lol, for some reason it didn't succeed when I tried to push it last night. In my version, any true return will override any false returns, so we might wanna discuss whether we want to do that or have it go your way.

After discussion, we're gonna use the true-overrides-false behavior.


System.Collections.Generic.List Sort(Comparison comparison) doesn't invalidate the enumerator

Incrementing the _version field is probably missing from one overload of the Sort method in List after calling Array.Sort
https://github.com/microsoft/referencesource/blob/5697c29004a34d80acdaf5742d7e699022c64ecd/mscorlib/system/collections/generic/list.cs#L992

This repository is not the right place to report issues with .Net.
For .Net Core/.Net 5+, this has already been fixed in https://github.com/dotnet/coreclr/pull/8944.
For .Net Framework, you could report this issue on Developer Community, but I'm not convinced it would be fixed, considering the status of .Net Framework.


Elaborate a plan for updating the CONTRIBUTING and README documents based on Onboarding.md contents

As a preliminary step towards https://github.com/input-output-hk/ouroboros-network/issues/4061 we need to come up with a plan for updating the CONTRIBUTING and README documents based on Onboarding.md contents.

These documents were updated and moved to https://github.com/input-output-hk/ouroboros-consensus/


Update build script

This improves but doesn't completely fix the issue of dynamic libraries on OS X. I'm throwing in the towel at this point though, because I think we just need to get ghc/cabal to stop ignoring -rpath. The current approach is:

Use install_name_tool as a post-build step to explicitly embed the LC_RPATH to the location of the CUDA library. This enables this package to be loaded in GHCi correctly and for other libraries to link against it without the need for DYLD_LIBRARY_PATH (which won't work anymore due to SIP).

Executables that use this library won't have the necessary LC_RPATH of course, and thus fail in the usual way. However, DYLD_LIBRARY_PATH seems to still work for those (at least, when called directly, which is why it doesn't work with ghc/ghci.).

Thanks for this. You're doing god's work.
It's a shame that we have to go to some much effort for linking on OSX. Have you filed a bug with the cabal/ghc people?

I did not file a bug... :


Instead of interactive map, is it possible to insert static map in DOm?

This feature will be added in next release.

Geolocator v2.1.0 is released. This feature is added.
Pls see the change-log.


Help me  im holding multiple assets what should I do?

Is your proposal related to a problem?

(Write your answer here.)
Describe the solution you'd like

(Describe your proposed solution here.)
Describe alternatives you've considered

(Write your answer here.)
Additional context

(Write your answer here.)


Executing on windows with TensorRT

Hi,
Are the steps for tensorRT verified on a windows PC as well?
I could not follow the steps on windows.
Unfortunately, the python bindings for tensorRT on windows are not available as well.
Bests,
Sid

Hi! Since TensorRT does not provide Python support on Windows, it can not be ran on Windows through Python.

Hope, it is clear now.


CLI needlessly depends on git

in order to install the pickle dependency, Git must be installed. This turns out to be a requirement for using our CLI. pickle should be published to NPM and this dependency severed.

Ok will handle our own npm publishing and this today.


Unable to install the platformio-atom-ide-terminal

[ ] I have reset Atom to defaults prior to submitting report.
[ ] I have not reset Atom to defaults prior to submitting report.

Description

Steps to reproduce

Expected behavior:
Actual behavior:
Reproduces how often:
Versions

> atom --version

> apm --version

OS name and version:
Platformio-ide-terminal version:
Additional Information

Me neigther

@Ikram24817 thank you for reaching ou to use with this issue report.
Please do not ignore the issue template fields in future, this sort of disregard isnt helpful in any case, in this instance the issue is well know, and, many duplicates to #700 have been filed including yours.
There is a workaround, its been posted in this issue https://github.com/platformio/platformio-atom-ide-terminal/issues/707#issuecomment-514634895 < click that.
Thanks for your understanding.


Problem with tags

What version of the program are you using? On what operating system?
I use 5.5.2 (S)table and 5.5.2 (N)ightly
Windows10
What steps will reproduce the problem?
Search image and click or download image:
Derpibooru - (S N) miss/doesn't show all tags in preview and downloading except %character, %artist
Konachan - (N) when downloading/saving image it miss all tags except %rating
Rule34.paheal - (S N) doesn't recognize tags (missing colors and doesn't download them "save") %character, %artist, %copyright
yande.re - (N) when downloading/saving image it miss tags %character, %artist, %copyright
zerochan - (S N) doesn't recognize tags (missing colors and doesn't download them "save") %character, %artist, %copyright, %rating
Also in Nightly "open in browser" doesn't work.
What is the expected behavior? What do you get instead?
I would like working tags. But don't know how much is program and how much is a website problem.
How often does this problem occur? On which sources?
all the time.
Please provide any additional information below
Trying Nightly from #1246 to be up to date

Thanks for the info!
 Feel free to try the latest nightly and tell me if they are properly fixed.

Rule34.paheal - (S N) doesn't recognize tags (missing colors and doesn't download them "save") %character, %artist, %copyright

It's a Shimmie board, so tags are not typed.
Therefore, %character%, %artist%, and %copyright` will never work.

It's a Shimmie board, so tags are not typed.
Therefore, %character%, %artist%, and%copyright` will never work.
OK understand. Bug was on many sites so I just tested all of them.

I check f547bd6 and everything except derpibooru works perf now.


Could I please join this organization

I have contributed to this organization on the unique-github-profiles and I would like to become a member of this organization. Thanks (:

Now you become owner plz do something add members

Congratulations 🥳

@VasudevJaiswal Thanks, I can't seem to add members?

Now check

@VasudevJaiswal Thanks. Just to clarify, do I just invite anyone I know or do I invite only people to contribute to the org?

@VasudevJaiswal, just to clarify, do I only invite people who contribute to the org?

Don't just try


"ansible-bender clean" throws an exception

21:54 $ ansible-bender --debug clean
21:56:25.572 db.py             DEBUG  search for runtime dir
21:56:25.573 db.py             DEBUG  trying ~/.cache
21:56:25.574 db.py             DEBUG  runtime dir is /home/bmillemathias/.cache
Cleaning images from database which are no longer present on the disk...
Traceback (most recent call last):
  File "/home/bmillemathias/Code/venv/ansible/lib/python3.7/site-packages/ansible_bender/cli.py", line 371, in _clean
    run_cmd(["podman", "inspect", b.target_image],
NameError: name 'run_cmd' is not defined

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/bmillemathias/Code/venv/ansible/bin/ansible-bender", line 8, in <module>
    sys.exit(main())
  File "/home/bmillemathias/Code/venv/ansible/lib/python3.7/site-packages/ansible_bender/cli.py", line 436, in main
    sys.exit(CLI().run())
  File "/home/bmillemathias/Code/venv/ansible/lib/python3.7/site-packages/ansible_bender/cli.py", line 415, in run
    self._clean()
  File "/home/bmillemathias/Code/venv/ansible/lib/python3.7/site-packages/ansible_bender/cli.py", line 373, in _clean
    except subprocess.CalledProcessError:
NameError: name 'subprocess' is not defined

21:56 $ ansible-bender --version
0.8.0

@bmillemathias-1a
Hi, this is fixed and merged in to master a couple of days ago. Would be reflected in the coming minor release of ansible-bender.
What is the workaround till then?
You can run ansible-bender directly from your git clone (please make sure that your clone is in sync with upstream with all the latest changes pulled) and run the CLI module
 python3 -m ansible-bender.cli
Sorry for the trouble.

Sorry for the trouble.

No problem, I've open the issue in order to help, I forgot to check commit logs.
Best.

I'm planning on releasing a new version this week, before I disappear for the holidays.


Debug node does not fire when using msg.payload and contents is 'NaN'

Before you hit that Submit button....
This issue tracker is for problems with the Node-RED runtime, the editor or the core nodes.
If your issue is:

a general 'how-to' type question,
a feature request or suggestion for a change,
or problems with 3rd party (node-red-contrib-) nodes

please use the mailing list, slack team or ask a question on Stack Overflow and tag it node-red.
That way the whole Node-RED user community can help, rather than rely on the core development team.
So you have a real issue to raise...
To help us understand the issue, please fill-in as much of the following information as you can:
What are the steps to reproduce?
Import the following flow, deploy. press the three inject nodes and view the debug log
[{"id":"e5532397.93a28","type":"function","z":"50cbb665.b57728","name":"set payload to Number.NaN","func":"//var x = msg.payload --- "<5";\nmsg.payload = Number.NaN;\nreturn msg;","outputs":1,"noerr":0,"x":400,"y":640,"wires":[["5039920d.34720c","28960b1e.dd1fc4"]]},{"id":"5039920d.34720c","type":"debug","z":"50cbb665.b57728","name":"","active":true,"console":"false","complete":"true","x":730,"y":680,"wires":[]},{"id":"7f7f41fe.a700b8","type":"inject","z":"50cbb665.b57728","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"x":180,"y":640,"wires":[["e5532397.93a28"]]},{"id":"28960b1e.dd1fc4","type":"debug","z":"50cbb665.b57728","name":"","active":true,"console":"false","complete":"payload","x":730,"y":600,"wires":[]},{"id":"28c9f769.6c734","type":"function","z":"50cbb665.b57728","name":"set payload to null","func":"msg.payload = null;\nreturn msg;","outputs":1,"noerr":0,"x":370,"y":560,"wires":[["5039920d.34720c","28960b1e.dd1fc4"]]},{"id":"9f2c8153.65a128","type":"inject","z":"50cbb665.b57728","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"x":180,"y":560,"wires":[["28c9f769.6c734"]]},{"id":"d68e90c5.9def6","type":"function","z":"50cbb665.b57728","name":"set payload to NaN","func":"msg.payload = NaN;\nreturn msg;","outputs":1,"noerr":0,"x":370,"y":720,"wires":[["28960b1e.dd1fc4","5039920d.34720c"]]},{"id":"212aab30.9b6e5c","type":"inject","z":"50cbb665.b57728","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"x":180,"y":720,"wires":[["d68e90c5.9def6"]]}]

What happens?
In two of the case, when the msg.payload is NaN, the debug node displaying only the msg.paylod does not show anything in the debug log while the debug node displaying the complete msg object does display a result
What do you expect to happen?
The debug node should display in the debug log even when the payloas is NaN - it does when the payload is 'null' (first example)
Please tell us about your environment:

[ ] Node-RED version: v0.17.5
[ ] node.js version: v6.11.0
[ ] npm version: 3.10.10
[ ] Platform/OS: OSX 10.12.6
[ ] Browser: Safari

yes - we aren't handling NaN correctly - we try and JSON parse it and it fails as it's not a number... so it stops at that point and displays nothing.


Is this install fighting with my previous Anaconda install?

I've installed IHaskell as instructed, but I do have a previous Anaconda environment on my Ubuntu 20.04. Are these installs fighting? For example, what is the difference between starting with
jupyter notebook
or
stack exec jupyter -- notebook
Either way, I can only see the home root directory, i.e., files are being created in my home directory. I tried to change the working directory, i.e.
jupyter notebook --generate-config
then changing in ~/.jupyter/jupyter_notebook_config.py
...
c.NotebookApp.notebook_dir = '/home/lawrence/Notebooks/Haskell'
This has no effect. Are there any tutorials?

I don't have enough information to say whether this install is clobbering your Anaconda install, but have you tried running stack exec jupyter -- notebook from the directory you want files to be created in, such as /home/lawrence/Notebooks/Haskell?

I don't have enough information to say whether this install is clobbering your Anaconda install, but have you tried running stack exec jupyter -- notebook from the directory you want files to be created in, such as /home/lawrence/Notebooks/Haskell?

I'm fairly certain now they are not in conflict, because it would seem a IHaskell is a completely separate, stand-alone world from Anaconda when cloned from here. My problem is none of the help for normal Jupyter Notebook seems to apply to this IHaskell version. For example, footnotes don't seem to work, and what about mono-spaced font for code? I can look up answers, but they don't seem to apply to IHaskell. Some tutorials would help on what's different and what's the same.

I'm fairly certain now they are not in conflict, because it would seem a IHaskell is a completely separate, stand-alone world from Anaconda when cloned from here. My problem is none of the help for normal Jupyter Notebook seems to apply to this IHaskell version. For example, footnotes don't seem to work, and what about mono-spaced font for code? I can look up answers, but they don't seem to apply to IHaskell. Some tutorials would help on what's different and what's the same.

We have https://github.com/gibiansky/IHaskell/blob/master/notebooks/IHaskell.ipynb, which is reasonably comprehensive.

We have https://github.com/gibiansky/IHaskell/blob/master/notebooks/IHaskell.ipynb, which is reasonably comprehensive.

Closing because there doesn't really seem to be an issue to be resolved here.

Closing because there doesn't really seem to be an issue to be resolved here.


[Feature] Make terms and conditions a mandatory requirement while sign up.

Description
Make terms and conditions a mandatory requirement while sign up by inserting a checkbox.

Screenshots
No response
Checklist

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

Congratulations, @AkhilaSunesh! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

No need as of now.


show error message when push fails

This change will show the error message from the request in the traceback, and doesn't assume a status code is available (which seems it may be the case with new jobq).
Before the change, I got the following error:
  File "hubstorage/jobq.py", line 29, in push
    if exc.response.status_code == 409:
AttributeError: 'NoneType' object has no attribute 'status_code'

After:
  File "hubstorage/jobq.py", line 26, in push
    raise HTTPError(o['error'])
requests.exceptions.HTTPError: job must contain a spider

An HTTPError without response is a bit rare, how can I reproduce?

Right now, that's what I get if I try to schedule a script with hub-crontask.

what is hub-crontask? the old hub-crontask should be dead by now.

@dangra fair point.
we have a copy of it in a project, we use it to schedule Python scripts in Scrapy Cloud.


PDF export of slides using Split are empty

I could reproduce with mdx-deck 1.7.1 and 1.7.7.
Create a deck.mdx file, copy paste this:
import { Split } from 'mdx-deck/layouts'

# LEFT

# RIGHT

export default Split

Then run mdx-deck pdf deck.mdx. The PDF will be empty however the web version looks fine.

it seems similar to #121, I can have a look at it.

I can confirm that bug.

I think this should work now in v2. Feel free to reopen if you're still seeing issues


Have a command to redo port forwarding (ssh tunnels)

It would be nice to have something like vagrant forwarding reload or something.
When connecting to libvirt via ssh on a remote machine the ssh tunnels created by vagrant-libvirt (the vagrant forwarded_port feature) often break and recreating them by hand is painful.
I know that vagrant reload recreates the tunnels but since it halts and restarts the VM it is less than ideal. For example I lose my various terminals running inside tmux tailing all the logs I need to look at.
I would be ready to look at the code and try to contribute it but wanted to know if there is interest and maybe what name should I give the new command?

I don't see any command in vagrant 1.8.1 to do this, and given the workaround mentioned above, I'm closing this.
If upstream vagrant adds a command to do this, we can hook onto it, but as a plugin we are relying on upstream to provide such commands. I see no way of fixing this today.


Named bot

Add the possibility to name the bot in terminal and discord.
When using several bots, it is convenient to know which one is stuck, or has dropped an item, when looking at the discord channel.
This (trivial) PR implement a very basic way to do it: the user can register in custom.ini a name field which is then used in send_discord_thread
Example
Setting in custom.ini
name=My favorite bot
will lead to message such as

My favorite bot just found: uniq_armor_ormus_robes (v0.2.2-dev)

I just saw the PR #16 ... But I think mine is a bit more general if in the future there is more advanced logging / different hooks / webserver interface to it, instead of being tied to discord.

Also I think #16 is dead. Doesnt seem like there will be any more updates. I like your approach.


Is there any elegant way to preserve decorated function name ?

Without surprise,
@mydecorator
def myfunction():
   ...

is transpiled to
var myfunction = mydecorator(myfunction(){
    ....

As a result myfunction.name is "mydecorator". This is a problem for me because I rely on function names in the rest of my application. This is a classic problem and python provides utilities for that like the functools.wraps decorator (link) but is there a similar utility in transcrypt allowing that ? A pragma maybe ?
The only trick I found is to add
__pragma__("js", {},'Object.defineProperty(myfunction, "name", { value: "myfunction" });')

after the declaration of my function (func.name being read-only in js, that's the trick I have found) but it's rather inelegant.

I had a need for this same thing.  I worked around it for now by using adding an argument to my decorator that lets me explicitly pass in the function name.  But having the original function name available inside the decorator as __name__ would have made for a bit cleaner code.


feat: genesis config presets

[ ] implement genesis config presets, move genesis configs from node/chain_spec.rs to the relevant runtimes.
[ ] remove the redundant type aliases at https://github.com/r0gue-io/pop-node/blob/71d81b94e33196092cfa29910b399b84437bd1f6/node/src/chain_spec.rs#L10-L13

https://github.com/r0gue-io/pop-node/issues/197


chore: update renovate config

What this PR does / why we need it:
chore: update renovate config
Add sign off of commit and no release note label
Release note:
NONE

/cherrypick release-v0.17


A support for ZhipuAI(bigmodel.cn)

A support for ZhipuAI(bigmodel.cn),the only dependency is zhipuai` package.I've tested it and passed,so I open a new PR.

我们不是已经有ChatZipuAI了吗？似乎底层界面是聊天模型，所以也不需要创建法学硕士？

Yes,you're right.It's an old PR,when I first open it,there was not any support for Zhipu.Now I will close this PR.Thanks.


[Bug]: Not understanding warning message from autogen during ConverseableAgent creation.

Describe the bug
The following warning is issued by autogen during the ConverseableAgent creation even though the code functions as expected.
Sample response from the two agent conversation relying upon a local function to return weather in order to determine tour suggestions.
[autogen.oai.client: 07-17 14:33:30] {164} WARNING - The API key specified is not a valid OpenAI format; it won't work with the OpenAI-hosted model.
user_proxy (to assistant2):
What are some interesting things to do while in Washington DC today?

USING AUTO REPLY...
assistant2 (to user_proxy):
***** Suggested tool call (call_GZJyzpI9Rf9KtNvEEL0slu16): get_weather *****
Arguments:
{"location":"Washington DC"}

user_proxy (to assistant2):
***** Response from calling tool (call_GZJyzpI9Rf9KtNvEEL0slu16) *****
Heavy percipitation is expected throughout the day in Washington DC. T-Storms are very possible.

USING AUTO REPLY...
assistant2 (to user_proxy):

Here are five suggested indoor activities to enjoy in Washington DC due to the heavy precipitation and possible thunderstorms:

Visit the Smithsonian museums.
Explore the National Gallery of Art.
Tour the U.S. Capitol Visitor Center.
Discover the Library of Congress.
Experience the International Spy Museum.

Feel free to let me know if you need more recommendations or information!
Steps to reproduce
The llm_config contains the model and api_key in this format:
model = os.getenv('OPENAI_MODEL')
api_key = os.getenv('OPENAI_API_KEY')
config_list = [
{
"model": model,
"api_key": api_key,
}
]
llm_config = { "config_list": config_list}
assistant2 = ConversableAgent(
name="assistant2",
system_message="""You are a helpful tour guide assistant that can recommend points of interest based upon
......
Return 'TERMINATE' when the task is done.""",
llm_config=llm_config,
)
print(assistant2.print_usage_summary)
print(assistant2.llm_config)
Model Used
gpt-3.5-turbo
Expected Behavior
The behavior is correct, I can't figure out how to clear the warning.
Screenshots and logs
No response
Additional Information
AutoGen Version: 0.2.32
Operating System: macOS Montery 12.6.2
Python Version: 3.12.4

Check your API_KEY.
It probably starts with sk-None..., which is probably because it's supposed to have a specific value, like sk-proj..., so creating an API_KEY that belongs to a project rather than a legacy key should fix it.
Of course, this is a bug! :D

Create a project key and I still get the same warning message. Behavior remains as expected and operational.

Hello I am having the same issue, I am trying both with "User API keys (Legacy)" and also with the new Project API keys ("sk-proj-...)


Build a twitter bot that tweets whenever a new repository is added

Twitter handle: @gfidev

Hi @jaipradeesh!
How are we taking care of storing the repos that have already been tweeted? Will need to maintain some sort of a DB or any other type of persistent storage in order to avoid duplicate tweets. Let me know your thoughts so I can help by picking this up.

@sam95 Sure. The bot should tweet the repository only once (when it is added) -- to start with. And maybe down the line, we can figure out how to randomly tweet a repository every n days (without repeating).

@jaipradeesh how do we maintain persistent repository records in this case (storing whether we tweeted only once inspite of multiple deployments)? I was thinking we can try persistent redis instead of setting up a DB only for this case.
Let me know your thoughts.

@sam95 maybe, listen to changes on data/repositories.toml on master branch.
Basically, push a tweet whenever a new line is added to the file.
We can safely assume that commits won't be rewritten (protected branch) and we already have checks in place to ensure that entries aren't duplicated.

I am also looking into this. Cheers!

I am planning to create and test the bot on my twitter account and then give you the docker image for your twitter handle. @jaipradeesh what do you say?

@sam95 That works. Apologies for the delay.
Also, can you help me understand the workflow?

@jaipradeesh can i work on this?

Wow. This has been going on for a very long time now. @sam95 is working on it with @RJ722.
Let's wait for a couple more days. @aneeshdua Let me try to open up a few more issues in this repo.

Wow. This has been going on for a very long time now. @sam95 is working on it with @RJ722.
Let's wait for a couple more days. @aneeshdua Let me try to open up a few more issues in this repo.


docker_org_member: revamp the model

the invite API lets you send invites by username and email
before this PR, docker_org_member was designed to match
the invite API
In practice, this made it impossible to implement
import behavior.
now, we model docker_org_member closer to how
the list org members api represents things

Signed-off-by: Nick Santos<EMAIL_ADDRESS>

oops, i think a bunch of unrelated docs changes got spun up here. opened that as a separate PR - https://github.com/docker/terraform-provider-docker/pull/74

@nicks doc changes are still showing here, once removed can ✅

updated!


Generate different types of operations

Feature description
Currently, all generated records have the operation set to created.
It would be good if we had the option to specify an operation or if the generator chooses a random operation.

@AdamHaffar might be a good one


Use beta api to get additional firewall data.

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


SAM crashes with TOU rates

Describe the bug
PVWatts-Residential case with tiered TOU energy and demand rates crashes.
To Reproduce

Open the tou-outputs.sam file tou-outputs.zip.

Run a simulation.

Operating System and Version

OS: Windows
Version: SAM 2020.11.29 r1

It looks like the issue here is that the TOU periods have different tiers, all periods need to have the same tier. Fix for the crash will be to catch the exception or index error and provide an error message to the user.

Still not fixed, in 2020.11.29r2 and 2021.12.2. Using a Time-of-Day rate from Public Service company of Oklahoma
GUID: 5dae1b2f5457a3cc62d2ae4f
This crashes both versions.

@drevastation thanks for reporting! The root cause of your issue is different from what was reported and fixed here, so I've opened a new issue: https://github.com/NREL/SAM/issues/857 and put it on the list for patch 1. Thanks for helping us expand the number of rates that SAM supports!


ETOR Retreat Prep

Backlog Task
Add a title and description here
Completion Criteria

[ ] Recorded demo
[ ] presentation slides by end of day January 28
[ ] any notes collated for the different sessions to help with the discussions
[ ]

Tasks

[ ] What tasks need to be done to fulfill the completion criteria?

Other Notes

Any other notes to help clarify this task for the team

ETOR Retreat 30Jan 2024 Draft Agenda.docx

Bookmark for myself to share out the link to the revised agenda timeblocks! (Content remains unchanged.)

Live agendas for both days available at: https://cdc-my.sharepoint.com/:f:/r/personal/uwx1_cdc_gov/Documents/ETOR/2. Cross-Intermediary Coordination/Jan 2024 Retreat Planning/Agendas?csf=1&web=1&e=sxYmHh

Work-in-progress Day 1 slides at: https://cdc-my.sharepoint.com/:f:/g/personal/uwx1_cdc_gov/EsatTf7mrrNCi6187oxsQSMBHHg2PsLWd_INl4rQn2vhfg?e=FSXcpW.  (Check out 18-23 and 26 as a starting point.) Input welcomed throughout on remaining slides (please use comment feature).TY!
Comments throughout on

Slides updated, video posted to sharepoint: https://cdc.sharepoint.com/:v:/r/teams/OLSS_ETOR_ReportStream/Shared Documents/General/ETOR for Newborn Screening/99. Reference and Landscape/CDC-ReportStream-Intermediary-Demo-20240130.mp4?csf=1&web=1&e=REwMVI

Slides updated, video posted to sharepoint: https://cdc.sharepoint.com/:v:/r/teams/OLSS_ETOR_ReportStream/Shared Documents/General/ETOR for Newborn Screening/99. Reference and Landscape/CDC-ReportStream-Intermediary-Demo-20240130.mp4?csf=1&web=1&e=REwMVI


Create a single conversion script for generating html

The original project had a series of shell and python scripts to generate html files from the notebooks. Some of that was really hacky, because I didn't know how Jupyter/ IPython notebooks were generated in enough detail. In this issue, I'll pull this into one Python script, that uses nbconvert and templates as much as possible. All remaining conversion work will be done in Python.

[x] Create base Python conversion script.
[x] Base conversion script in root project folder. /notebooks folder should only contain notebooks. Other resources should be in folders under project root.

Incorporate the functionality of all the individual .sh and .py files in scripts/ into build_html_site.py, and templates. When all of this functionality has been updated and simplified, these scripts can be deleted.

[x] build_html_pages.sh

calls all of the other scripts, this should be checked off last

[x] sed section of build_html_pages.sh to make output display initially on Mapping Global Earthquake Activity

Add a tag that allows specific code cells to show output by default?
Add a tag that allows specific notebooks to show all output initially by default?

[x] (keeping this) build_all_exercises_page.py

May just keep this as a separate issue for completion.

[x] create_common_hmtl.sh

The default nbconvert behavior is to link to other notebooks on nbviewer. This script modifies the html output to link to other html notebooks. (Make sure links between notebooks work.)

[x] highlight_code.py

highlight specified lines of code

[x] highlight_inline_code.sh

highlight code snippets in bodies of text

[x] show_hide_output.py

This is being taken care of in the template now. But the script only included show/hide all buttons if there was an output div at some point in the notebook. That might be good to implement. (see #148)

[x] convert_home_links.py

This script converts links from "index.html" to "/". The server responds by serving the index page. Difference between requesting introtopython.org/index.html and introtopython.org.

[x] remove_input_references.py
[x] build_index.py

look at actual code before checking this off

This is a great issue to finally close!


Individual Address balance

How do I get the Wallet balance for a particular address receiving address.

Hi, you can't get the wallet balance for a particular receiving address.


Added Skill Messaging API

Official documentation here:
https://developer.amazon.com/docs/smapi/skill-messaging-api-reference.html
The methods listed in this class are meant to work out-of-session. That's why I didn't make it extend from the ApiBase class like the other api classes. The types for the response for both methods are not listed in the latest version of the Alexa typescript package.

@armonge done!


Is supported select range on touch device

Select range is not woking on touch deivce. Can you support it?
Thanks.
Versions
Angular: 1.3
Calendar library: 0.30
Browser name and version: chrome on ipad

Sorry but due to the way that touch events work, all touch events fire on the initial starting segment so its non trivial to add support for this.


Adds authorization for sidecar-promclient

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Added new flag prometheus.http-client to sidecar to pass http client config via file or string(similar to alertmanager). Using this flag an instance of thanoshttp.Client is created and is passed to promclient.NewClient  method which uses this client as base client instead of whole new client which then makes authorized api calls.
The same client is passed to reloader config which then makes authorized api calls to prom api.
There are some code duplication which has been duplicated from prometheus which can be easily removed by exposing one of the functions in prometheus http_config
:raised_hands:
Close: #3975
Verification
Added a e2e test:
Checks if query is able to query when prom uses auth

This implementation helps in providing a generic way to create http clients used across. There are still many instances where directly http library is used to create client which can be replaced with thanos http client instance and can be configured flexibly.
Also solves the issue mentined by @wiardvanrij  is here.

Any update on this?

Any update on this?

Its ready for a review, I guess I forgot to put a review request :smile:

@someshkoli Can you rebase on main and fix the conflicts? Thank you!

Hi, @someshkoli, sorry for the late reply. Can you fix the conflict and get CI passing? Your work is really amazing and we would like to have this feature!

Hi, @someshkoli, sorry for the late reply. Can you fix the conflict and get CI passing? Your work is really amazing and we really want to have this feature!

will update it soon, have a lot to rebase :")

Moving this to #4612

@yeya24 can you please take a look at #4612 ?

Close in favor of #4612


fix: Binary location in the container

Changes:

Binary location for git-cloner, waiter, image-processing and image-bundler are updated to /ko-app/ since this is hardcoded in the upstream source code.

Merging since the images are confirmed to work.
https://redhat-internal.slack.com/archives/C02258G4S79/p1722331771883679?thread_ts=1722329911.856279&cid=C02258G4S79


MediaPipe SelfieSegmentation Script Version Upgrade

Hi @akhil-rana  for the short term we can upgrade it like this. In meantime you can do the optimization when you get chance. Ideally, this way for production is not ideal due to CSP.
resolves #2

This seems okay. I'll just test once and approve asap.
Thanks for the contribution.


CB-19879 Fixed cdp-prometheus not being able to start on Azure base images

The change will be implemented in cloudbreak-images. Closing this PR.


Add a video of using codespaces for software development

Summary
Need to add a video to Codespaces that illustrates how to use codespaces to contribute software, especially running the tests locally.  The existing video targets doc contributions using the doc tools.  The Software development environment page describes what is required for software development , including the tests that should be run locally before creating the PR.
Should we have two codespace videos or combine this issue with https://github.com/keptn/lifecycle-toolkit/issues/2820 to create one video that does both docs and software?
DoD

Video for contributing a software fix using codespace

Closed in favor of #2820


Adding drag and drop integration test helpers [WIP]

This is a work in progress PR intended to add test helpers for integration tests. The API is up for debate, but is currently implemented as:
drag('.drag-handle-css-selector', {
  beforeDrop: function() {
    // here would be an opportunity to assert certain things while drop targets are visible
  },
  drop: '.drop-target-selector'
});

Once the API is solidified, then documentation and possibly tests would follow

This will be useful, thanks for this. I'll need to test this a bit more and make sure we have some test examples and an updated readme before I can release it though.

Sounds good, I can help provide that. I just wanted to make sure that the approach was okay, before I dug in and worked on it some more. I'll push a few commits later this week

@dgavey What can I do to help push this PR through? Add a test that uses the helper and update the README?

Sorry, kind of dropped the ball on this.  We will need some tests that use the helpers to confirm they are working, and some info in the readme under a new tests section would be fantastic.

@dgavey I just pushed a commit that has tests for the test helpers and documentation in the README. Please take a look when you get a chance and let me know if anything else needs to be added/fixed.

This is great, thanks so much for this!


Rename/alias R.of --> R.listOf

For me personally R.of has a confusing name. Every time I meet it in code I spend extra cognitive payload on it. Unlike R.objOf that is rather clear.
What do you think about renaming/aliasing it to R.listOf / R.toList?
I've read that ramda should be 100% without aliases. I understand that this change can break existing codebase. But, from the other side, I believe that clear function naming is an important point. Especially for newcomers.

The chief reason for this function is to work with the [Fantasyland] Applicative specification, which gives us for instance, Maybe.of(42) //=> Maybe.Just(42).  So we can't really change the name.  And as you note, we have accepted a strong no-alias policy.
I agree that the name is not wonderful.  But I don't see that there's much to do.

Apologies for being a bit off-topic, but a quick question... @CrossEye I see your reference to Maybe in your comment above.  I'm confused.  Does Ramda have a "Maybe" construct?  I'm just learning, but it seemed to be one thing that was lacking.  What did I miss?
Thanks.

@CrossEye I was considering writing something similar to what you put, but then I thought about it, and our of can't handle anything other than Arrays. It doesn't dispatch, and of course it can't dispatch given its current signature.
It would be nice to have an of function that actually did do some dispatching.

@Bradcomp: I forgot that we had this issue.  I agree that we need to get to that and pretty soon.  I have a few Ramda things on my plate, but will move this up the list.  I agree that we should do something like that formulation.  (And then @vitalets could get his wish too!)

@GenCodeDev: No, Ramda does not include Maybe or other similar types.  There was a now-dead Ramda-fantasy project for such things.  But we didn't put the effort in to keep it updated.  And there were many good libraries out there for this.
I've described Ramda as not supplying new types, simply operating on built-in, and often custom, types.  But it works well with (a subset) of the FantasyLand specification, so you can use it with many types, such as most Maybe implementations.

Is the arrayOf function actually needed at all? It can be easily implemented as unary(list),
or just x => [x].
Similary, pair = curry(binary(list)).

It needs to be fixed to dispatch properly.  I don't have the time right now like digging to find why we had problems doing so.  But we need to address them, so that we can work properly with Applicatives.

@CrossEye I was considering writing something similar to what you put, but then I thought about it, and our of can't handle anything other than Arrays. It doesn't dispatch, and of course it can't dispatch given its current signature.
It would be nice to have an of function that actually did do some dispatching.
of(Array, 42) //> [42]
of(Maybe, 42) //> Just(42)
of(Either, 42) //> Right(42)

and so on.

I quite like this. @CrossEye should be simply rewrite R.of to work as @Bradcomp suggested for v1.0. Quite a substantial breaking change but I reckon we should try to ship as many of them in v1.0 since that would be our first semver compliant release.

@customcommander: Agreed.  And then we can get the OP's listOf as a trivial of(Array).


Add minimised version of dist CSS

issue goal: have a minimised version of all CSS files we distribute, for use on production.
The current generated main Techne 2.0 CSS is not ready to use on production.
https://github.com/SAP/techne/blob/v2.0/dist/css/techne-all.css
To be production ready is still needs:

[ ] Be minimised and trimmed of whitespace and formatting.
[ ] Comments should be stripped down

The minimised files will have *.min.css suffix and be located along with the non-minimised files.

This is already captured in #643

Closing this as dupe of #643


[GHSA-22wj-vf5f-wrvj] Password exposure in H2 Database

Updates

Affected products

Comments
Removing the affected product so dependabot doesn't flag it - given that it doesn't seem like it should be a CVE - see https://github.com/h2database/h2database/issues/3686#issuecomment-1333917904

Gah I'm sorry this is happening. I see in that thread that the maintainer may be revoking the CVE. Would you mind waiting until that takes its course and we can pull our record after that?

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.


Use an optional chaining operator when calling preUpdate

Please do not update the README or Change Log, we will do this when we merge your PR.
This PR (delete as applicable)

Fixes a bug

Describe the changes below:
This simply ensures gameObject.preUpdate exists before attempting to call it. Doing so allows developers to forego the preUpdate method when creating custom GameObjects.
If we would prefer to avoid using an optional chaining operator, I can submit another pull request that instead checks for the existence of the method in the if (gameObject.active) conditional.

But game objects without preUpdate() shouldn't be on the Update List, since its only job is to call preUpdate(). 🙂
add.existing() and addToUpdateList() do check for preUpdate().


Can the v1.0 support to deploy on an existing k8s cluster

Is there a guide for openpai v1.0 deployment with an existing k8s cluster?

related to https://github.com/microsoft/pai/issues/4030.
We do not recommend to deploy v1.0 to an existing k8s cluster, unless you know very well about OpenPAI and what it requires.
Here is an example showing how to deploy OpenPAI on an existing (clean) k8s cluster using AKS-engine. #4735

thanks @fanyangCS


Fix extract fullstack test

What problem does this PR solve?
Issue Number: close #5421
Problem Summary:
After TiDB PR https://github.com/pingcap/tidb/pull/36297, extract(day_microsecond/second/minute/hour) behavior changed to evaluate the second argument to time, thus the day portion wouldn't be showing up in the result.
What is changed and how it works?
Update test reference.
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/check-issue-triage-complete

/run-check-issue-triage-complete

/rebuild

/merge


Navigation - BE

Link to Ticket
Description of Changes Made
This updates the primary navigation setup so it works according to the new design, which allows for a multi-level menu (up to 3 levels).
The initial approach was to modify the primary nav settings to allow for a nested structure. However, it was observed that this could become complicated and may in turn be confusing for content editors (please see ticket overview for link to Slack thread). Therefore, the existing flat structure of the primary nav settings is maintained, and changes are instead made to the templates to control the rendering of the primary navigation.
How to Test

Add some links to the primary navigation in the Navigation Settings. These should be displayed on the frontend (without the final styling)
For any of the pages added above with children and grandchildren, edit a few of the children/granchildren by checking the Show in Menu box in the promote tab. These should appear on the frontend. (please see screenshot below for reference)

Screenshots

Expand to see more

MR Checklist

[x] Add a description of your pull request and instructions for the reviewer to verify your work.
[x] If your pull request is for a specific ticket, link to it in the description.
[x] Stay on point and keep it small so the merge request can be easily reviewed.
[x] Tests and linting passes.

Unit tests

[ ] Added
[ ] Not required

Documentation

[ ] Updated build docs
[ ] Updated editor guidelines (https://docs.google.com/document/d/1PAWccdQ4tfaZsrEWmpDhvP3GH5RRmBOARFVp4b-kje8/edit?usp=sharing)
[x] Not required

Browser testing

[x] I have tested in the following browsers and environments (edit the list as required)

Latest version of Chrome on Linux
Latest version of Firefox on Linux

[ ] Not required

Data protection

[ ] Not relevant
[x] This adds new sources of PII and documents it and modifies Birdbath processors accordingly

Accessibility

[ ] Automated WCAG 2.1 tests pass
[ ] Manual WCAG 2.1 tests completed
[ ] I have tested in a screen reader
[ ] I have tested in high-contrast mode
[ ] Any animations removed for prefers-reduced-motion
[x] Not required

Sustainability

[ ] Images are optimised and lazy-loading used where appropriate
[ ] SVGs have been optimised
[x] Perfomance and transfer of data considered
[ ] If JavaScript is needed alternatives have been considered
[ ] Not required

Pattern library

[ ] The pattern library component for this template displays correctly, and does not break parent templates
[ ] The styleguide is updated if relevant
[ ] ~Changes are not relevant the pattern library~ The pattern library will be updated on the FE ticket, as the templates will change significantly from the current version.

Thanks @engineervix I have been doing some testing of this on my local build.
At the moment, it seems like child pages are appearing even if the 'show in menu' option is not ticked - for example all the individual work and historical work pages are appearing.

I am just checking in with Anna about how we handle any menu items that don't have a drop-down - as I think there may have been some that we agreed should not, but we didn't discuss that at sprint planning.

Thanks @engineervix I have been doing some testing of this on my local build.
At the moment, it seems like child pages are appearing even if the 'show in menu' option is not ticked - for example all the individual work and historical work pages are appearing.

I am just checking in with Anna about how we handle any menu items that don't have a drop-down - as I think there may have been some that we agreed should not, but we didn't discuss that at sprint planning.

Thanks @helenb I noticed this when testing. This is happening because all historical work pages and blog pages have their show in menus ticked.
>>> WorkPage.objects.in_menu()
<PageQuerySet []>
>>> HistoricalWorkPage.objects.in_menu()
<PageQuerySet [<HistoricalWorkPage: Spotify Listeners Tune Into Samaritans' Big Listen Campaign>, <HistoricalWorkPage: Raising £1.3 million for the DEC>, <HistoricalWorkPage: Personalising experiences for vulnerable audiences>, <HistoricalWorkPage: A SMS and call centre application for Kenyan farmers>, <HistoricalWorkPage: A tool to organise sports competitions for 3m young people>, <HistoricalWorkPage: PPC Provides the Missing Link for Mencap>, <HistoricalWorkPage: Afternoon Tea with Breast Cancer Care>, <HistoricalWorkPage: 64% increase in donation income from Mind's Google Ad Grants activity>, <HistoricalWorkPage: Breast Cancer Now brand launch reaches 1.2 million people>, <HistoricalWorkPage: A successful remote GDS alpha phase for the Department for Transport>, <HistoricalWorkPage: An Extra 3 Million Organic Clicks a Month for the NHS Website>, <HistoricalWorkPage: Connecting Greenpeace UK with supporters via SEO>, <HistoricalWorkPage: Email Marketing in the Time of COVID-19>, <HistoricalWorkPage: Wagtail powers Buckinghamshire Council’s digital transformation>, <HistoricalWorkPage: Delivering volunteers for Girlguiding's InspireGirls campaign>, <HistoricalWorkPage: Brand awareness campaign delivers a 63% increase in donations>, <HistoricalWorkPage: Usability testing for the National Deaf Children’s Society>, <HistoricalWorkPage: Joining the battle to protect our oceans>, <HistoricalWorkPage: A strategy for improvement>, <HistoricalWorkPage: Headless Wagtail powers the digital home of Hong Kong’s museum of culture>, '...(remaining elements truncated)...']>
>>> HistoricalWorkPage.objects.in_menu().count()
41
>>>

>>> BlogPage.objects.in_menu()
<PageQuerySet [<BlogPage: Increasing Samaritans online donations revenue by 200%>, <BlogPage: Gold panning in Google Analytics>, <BlogPage: Accessibility: interesting links>, <BlogPage: Meet Wagtail, our new CMS>, <BlogPage: Wagtail in one line>, <BlogPage: Icon gonts, SVG and modern web graphics>, <BlogPage: A choose your own adventure video>, <BlogPage: Royal College of Art website launched>, <BlogPage: Hackference Birmingham>, <BlogPage: Mezzanine for data geeks>, <BlogPage: Record & ride>, <BlogPage: Frontend United: when hard work pays off>, <BlogPage: Display Suite for Drupal>, <BlogPage: Adios, Manuel>, <BlogPage: Responsive email design for Nuffield Trust>, <BlogPage: Guerrilla user testing>, <BlogPage: User research techniques>, <BlogPage: Jekyll for front end development>, <BlogPage: Torchbox University: Advanced Google Analytics>, <BlogPage: Are Content Management Systems Dead?>, '...(remaining elements truncated)...']>
>>> BlogPage.objects.in_menu().count()
278
>>>

So I had to set each historical work page and blog page to page.show_in_menus = False. We might have to do that on staging & prod, or I could do a data migration so this is done automatically when this PR is merged.

Update
@nimasmi following a small change in the requirements:

We'll add a checkbox to navigation settings 'Don't show child pages in menu'. This will be used for the /work/ and /blog/ sections and will override any 'show in menu' settings on the promote tab lower down the tree.
This box should be unchecked by default.
If unchecked, child pages will be shown in the menu and will respect the 'show in menu' settings in the promote tab.

I have made some changes via 05655a1, 6962f1b and e4c5ccb to address the above. Here's an illustration
Illustration
https://github.com/torchbox/torchbox.com/assets/7713776/ff6b0616-75ae-44bb-902a-1f27f988fe5c

It's really confusing and makes incremental review harder if you rebase and push an entirely new set of commits to the branch. If you do want to do this for tidiness reasons, a good way is to push fixup commits, and then perform an autosquash rebase once code review is approved, and you're ready to merge the branch. If you must rebase beforehand, then it's helpful to add a comment to that effect.


fix: comments and style in greedy_algorithms/dijkstra.cpp

Description of Change
As described in #895, I've updated the comments and style in greedy_algorithms/dijkstra.cpp to pass the cpplint check.

Checklist

[x ] Added description of change
[x ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x ] PR title follows semantic commit guidelines
[ x] I acknowledge that all my contributions will be made under the project's license.

Notes:

I addressed the latest comments on the PR. I had a clarification - the commits will be squashed prior to committing, yes? I'm just concerned about the last commit I made, because the message wasn't that descriptive.

please see comment above
code doesn't compile:


Logs agent dont filter docker containers by name

Output of the info page (if this is a bug)
Getting the status from the agent.

==============
Agent (v6.2.1)
==============

  Status date: 2018-05-28 16:10:13.072448 UTC
  Pid: 4600
  Python Version: 2.7.5
  Logs: Log Level: info

  Paths
  =====
    Config File: /etc/datadog-agent/datadog.yaml
    conf.d: /etc/datadog-agent/conf.d
    checks.d: /etc/datadog-agent/checks.d

  Clocks
  ======
    System UTC time: 2018-05-28 16:10:13.072448 UTC

  Host Info
  =========
    bootTime: 2017-07-11 18:16:16.000000 UTC
    kernelVersion: 3.10.0-514.21.1.el7.x86_64
    os: linux
    platform: centos
    platformFamily: rhel
    platformVersion: 7.3.1611
    procs: 295
    uptime: 2.7726836e+07
    virtualizationRole: host
    virtualizationSystem: kvm

  Hostnames
  =========
    hostname: backend2
    socket-fqdn: back2.server.local
    socket-hostname: back2.server.local

=========
Forwarder
=========

  RetryQueueSize: 0
  Success: 0

  API Keys status
  ===============
    https://6-2-1-app.agent.datadoghq.com,*************************24fb5: API Key valid

==========
Logs Agent
==========

  custom
  ------
    Type: docker
    Status: OK
    Inputs: 3c32a33f06070adfc3340b40a008214176b8f9854929ff42fe899d2bbb9ee691 179b8902dad3dfac9d4e5f061696b8f94d1168ffd27d6e1122ecc5fc5d051574

=========
DogStatsD
=========

  Event: 1

Describe what happened:
Datadog ignores name: directive for logs while filtering containers.
Describe what you expected:
I've expected to get one log source (in datadog stats and in UI on site) while using name in logs config.
Steps to reproduce the issue:
I've configured agent to collect logs from containers filtered by name
file /etc/datadog-agent/conf.d/custom.yaml
logs:
  - type: docker
    name: landstorage
    source: nginx
    service: docker

but as you see from stats I have two connected containers:
  Inputs: 3c32a33f06070adfc3340b40a008214176b8f9854929ff42fe899d2bbb9ee691 179b8902dad3dfac9d4e5f061696b8f94d1168ffd27d6e1122ecc5fc5d051574

P/s image: is working as expected
  custom
  ------
    Type: docker
    Image: test/templater_storage:latest
    Status: OK
    Inputs: 3c32a33f06070adfc3340b40a008214176b8f9854929ff42fe899d2bbb9ee691

Additional environment details (Operating System, Cloud provider, etc):
Centos 7.3.1611
DataDog agent v6.2.1

Maybe this bug is one of cases described here. Datadog read self container logs too because cant filter containers by name.
Also I've noticed that image: config for logs should include full tag, so docker/image:latest != docker/image

Hello @undiabler ,
Thanks for reaching out.
The filtering by name and short image name should be working on the version of the agent that you have.
You can find more information about the short image name filtering in our documentation.
For the image filtering you should be able to use:

templater_storage
test/templater_storage
templater_storage:latest
test/templater_storage:latest

Therefore I would expect your configuration to work fine.
Would it be possible to have the output of the docker ps -a command ?
The issue you linked should not be related as it is about picking up configuration from container labels when the environment variable is not set.
Thanks a lot,
Nils

Thanks @undiabler for all those details.
It seems indeed strange as it should work with your configuration.
I'll try to replicate and let you know of my findings.
Sorry for the trouble.
Nils

Can you please run two containers with different names and try to filter one.
According to you test everything is ok because its only one working container.

Sure, i'll start the nginx container and do the same test.
I'll come back to you with the outcome.

Hello @undiabler ,
I just wanted to come back to you on my latest tests.
For me it seems to work fine no matter how many containers I have running.
Here is an example with two running containers
Docker containers
CONTAINER ID        IMAGE                          COMMAND                  CREATED              STATUS              PORTS                NAMES
e9419023f174        nginx                          "nginx -g 'daemon of…"   About a minute ago   Up About a minute   <IP_ADDRESS>:80->80/tcp   docker-nginx
95013288e6d2        chentex/random-logger:latest   "/entrypoint.sh myte…"   3 days ago           Up 5 minutes                             gallant_shannon

Datadog Agent configuration
logs:
    - type: docker
    service: docker
    source: docker
    name: gallant_shannon

Datadog Agent status
    Type: docker
    Status: OK
    Inputs: 95013288e6d2d32b807e42ffed264c437b88429fff5af84136f4d0ecf595d1a4

So there might be an issue with your configuration. To make sure we find the issue would it be possible to send us an email at<EMAIL_ADDRESS>or answer to this tread with the following information:

an Agent Flare
the result of the docker inspect command for both your containers

Thanks a lot.

Hello,
We have not heard back from you so we are going to close this issue. But feel free to reach out to<EMAIL_ADDRESS>if you still observe a problem with the filtering by container name.


laravel 5.2 session/cookie issue

Hello,
i'm having a problem with a fresh laravel 5.2 install. When implementing the authentication system (not using Route::auth() - i'm implementing my own Auth controller) i always get a CSRF token mismatch exception.
I disabled the VerifyCsrfToken middleware so i can debug. The error that i'm facing is that after each login POST request (normal HTTP post not AJAX) the cookie data from the request object that contains the session ID is set to null. This causes laravel to start a new session. I am using the database driver for both session and cache.
It seems that the problem lies in the decrypt method of the EncryptCookies middleware. When looping through the request cookies, somehow there are 2 cookies setup on the request object with the same name - both representing the session id cookie) - first one has the correct value, the second one does not and generates this error.
PS: before performing the test i've always cleared the browser cache/cookies, truncated the session and cookie tables in mysql.
Here is the debug sequence and output: http://kopy.io/A4036

Thanks but this probably isn't a core bug. Feel free to discuss on the forums.

Yeap, user problem...
Figured out what lead to this: the web middleware is included by default for all routes via the RouteServiceProvider and my routes were declared under the web middleware. So the decrypt method was getting called twice, of course the 2nd call failed since the values were already decrypted.


Pkg integration

Description
This PR is to test using vercel/pkg with native modules.
It imports the following node modules:

level
utp-native
threads.js

It should be able to build via pkg and nix-build method.
Issues Fixed

Fixes #27

Tasks

import, build and test level
import, build and test utp-native
import, build and test threads

Final checklist

import and create hello world example for:

[x] level
[x] utp-native
[x] threads

build and test via pkg

[x] level
[x] utp-native
[x] threads

build and test via nix-build

[x] level
[x] utp-native
[x] threads

[x] modify release.nix to include the required prebuilds in the result.
[x] final tests of the nix builds. see if they include prebuilds properly.
[ ] test if GH auto release still works.

Completed most of the checklist, The last thing to do is check of the GH releases works properly with the changes.
Depending on the output we may have to zip the builds now since they are directories instead of just the executable.

Is there a way for you test the CI/CD execution of your changes.
You just need to go into the gitlab yaml file and enable this for your branch as well.
Then go to https://gitlab.com/MatrixAI/open-source/typescript-demo-lib/-/pipelines to manually trigger it. Or https://gitlab.com/MatrixAI/open-source/typescript-demo-lib/-/jobs.

Is there a way for you test the CI/CD execution of your changes.
You just need to go into the gitlab yaml file and enable this for your branch as well.
Then go to https://gitlab.com/MatrixAI/open-source/typescript-demo-lib/-/pipelines to manually trigger it. Or https://gitlab.com/MatrixAI/open-source/typescript-demo-lib/-/jobs.

Oh if you do this, make sure to switch it back later, we only want builds to occur on master, not on every branch.

We need --no-bytecode because without this flag it doesn't work inside
Nix and it isn't deterministic. So up to your decision. As I said if you
really need a directory, try pkg as the directory name to avoid confusion.
On 7/22/21 2:28 PM, Brian Botha wrote:

@.**** commented on this pull request.

In release.nix
https://github.com/MatrixAI/TypeScript-Demo-Lib/pull/29#discussion_r674493171:

@@ -59,13 +65,16 @@ let
PKG_IGNORE_TAG = 1;
buildPhase = ''
cp ${./package.json} package.json

   mkdir -p output/prebuilds

Ok, I can change it to pkg.
as for |--public-packages "*" | is just flagging everything is a
public package since non public packages are included as bytecode by
default.. This breaks with the |--no-bytecode| flag.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MatrixAI/TypeScript-Demo-Lib/pull/29#discussion_r674493171,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE4OHJ6LPSIJSE7VDU7XRLTY6M7PANCNFSM5AXJKZ6Q.

You mean nixpkgs commit from master? Does this all work with just the
revision provided by nixpkgs?
https://github.com/NixOS/nixpkgs/commit/3d64b6067f1c457f889e91c59ced273be1d1e4f1
The above is the just the one that brought in pkg.
On 7/22/21 4:28 PM, Brian Botha wrote:

@.**** commented on this pull request.

In pkgs.nix
https://github.com/MatrixAI/TypeScript-Demo-Lib/pull/29#discussion_r674530748:

@@ -1,5 +1,6 @@
import (
# let rev = "60cce7e5e1fdf62421ef6d4184ee399b46209366"; in

let rev = "04a48a2fcd2370f4e8a8c9fd3394531ccdb30f1e"; in

let rev = "04a48a2fcd2370f4e8a8c9fd3394531ccdb30f1e"; in

let rev = "53caacaf56640d04180775aee016d2f16d6f083c"; in

I picked the latest pkgs commit when I started on this issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MatrixAI/TypeScript-Demo-Lib/pull/29#discussion_r674530748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE4OHITJMVXE5N4SW42KT3TY63A5ANCNFSM5AXJKZ6Q.

I have created an upstram PR addressing the issue with the leveldown warnings.
vercel/pkg#1273

When this is done, the changes for js-polykey are not far behind. the only complication would be the jose/jwt throwing warnings. But I've added them as an asset and it seems to be working with manual testing e.g. starting agent + starting a session.

I'm checking out your change to vercel/pkg now.
Firstly:
git fetch origin pull/1273/head:pr-1273
git checkout pr-1273
# setup nix-shell here
npm install
npm run build
# the build outputs to ./lib-es5, however the executable requires the node interpreter
node ./lib-es5/bin.js  --help

The output:

  pkg [options] <input>

  Options:

    -h, --help           output usage information
    -v, --version        output pkg version
    -t, --targets        comma-separated list of targets (see examples)
    -c, --config         package.json or any json file with top-level config
    --options            bake v8 options into executable to run with them on
    -o, --output         output file name or template for several files
    --out-path           path to save output one or more executables
    -d, --debug          show more information during packaging process [off]
    -b, --build          don't download prebuilt base binaries, build them
    --public             speed up and disclose the sources of top-level project
    --public-packages    force specified packages to be considered public
    --no-bytecode        skip bytecode generation and include source files as plain js
    -C, --compress       [default=None] compression algorithm = Brotli or GZip

  Examples:

  – Makes executables for Linux, macOS and Windows
    $ pkg index.js
  – Takes package.json from cwd and follows 'bin' entry
    $ pkg .
  – Makes executable for particular target machine
    $ pkg -t node14-win-arm64 index.js
  – Makes executables for target machines of your choice
    $ pkg -t node12-linux,node14-linux,node14-win index.js
  – Bakes '--expose-gc' and '--max-heap-size=34' into executable
    $ pkg --options "expose-gc,max-heap-size=34" index.js
  – Consider packageA and packageB to be public
    $ pkg --public-packages "packageA,packageB" index.js
  – Consider all packages to be public
    $ pkg --public-packages "*" index.js
  – Bakes '--expose-gc' into executable
    $ pkg --options expose-gc index.js
  – reduce size of the data packed inside the executable with GZip
    $ pkg --compress GZip index.js

I'm going to assume you manage to make it work.. so he main thing is that we need to get node2nix to work against your fork's branch.

@tegefaulkes In the future for this sort of work you would want to create the forks under MatrixAI group in github rather than your personal account: https://github.com/tegefaulkes/pkg
It makes it easier for us to maintain forks and maintain overrides in our nixpkgs overlay.

The other really important thing is to always branch off from main or master when creating a PR. This way you can have a main that syncs up with upstream, while creating a feature branch to be PRed into upstream main.
So for now I'll just create a feature branch off main and do the same as what you did.

This is now done here: https://github.com/MatrixAI/pkg/tree/leveldown
The steps were:
git remote add matrixai git@github.com:MatrixAI/pkg.git
git checkout main
git checkout -b leveldown
# make the change
git add -A
git commit -m 'Removes dictionary override for bundling leveldown - Fixes vercel#1264'
git push matrixai

The nix override work in our overlay is located here: https://gitlab.com/MatrixAI/Engineering/nixpkgs-overlay/-/merge_requests/15

Ok, should be working now. Last thing is to check if the github CI still works.

I don't think anything needs to change in our .gitlab-ci.yml file. It should work fine now.
So please fix up my comments, ensure any linting is done, and also clean up the README and format it and squash all the commits.

See comment here https://gitlab.com/MatrixAI/Engineering/nixpkgs-overlay/-/merge_requests/15/diffs#note_642632986 regarding our policy of using our private overlay in public packages.

K, fixing up now.

After everything, don't forget to test out the Gitlab CI/CD with this method: https://github.com/MatrixAI/TypeScript-Demo-Lib/pull/29#issuecomment-884637611
Then finally we can merge.

I switched back to the public pkgs and added in the patch.
I've tested it in wsl and windows. It seems to be working. I'll try to run a pipeline in a second.

Misclicked closed. My bad.

Failing on tests. I'll look it over and fix it up tomorrow.

Checks are passing, looks like it's working. this should be good for a merge after a review. @CMCDragonkai

Squashed.

That should be the last of it.

[x] rebase
[x] small fixes
[x] squash
[ ] review
[ ] merge

Just fixed up a few things:

Removed useless comments from pkgs.nix
Formatted README.md with spacing between the code blocks and text
Restoring the mocks should happen at the end of each test
Indentation issues in release.nix

I noticed you also enabled source map for this repo. This is a good idea for a library. For a end-user application, not sure if it is relevant.

I noticed you also enabled source map for this repo. This is a good idea for a library. For a end-user application, not sure if it is relevant.

Ah, jest was throwing some minor errors without it. It's not needed I think.

You should also always test: nix-build ./release.nix --attr application.
Also we now have a custom override for our pkg here, this would have to done for all the repos using pkg.
Until upstream merges. Our overlay can also have this change applied too in the mean time.

The application build is currently failing due to the missing node-gyp-build command. It needs to be available to the derivation. We've dealt with this before.

So it turns out that what's required is that:
     node2nixProd = (
  -    (import (node2nixDrv false) { inherit pkgs nodejs; }).shell.override {
  +    (import (node2nixDrv false) { inherit pkgs nodejs; }).shell.override (attrs: {
  +      buildInputs = attrs.buildInputs ++ [ nodePackages.node-gyp-build ];
         dontNpmInstall = true;
  -    }
  +    })

The override can also take a function which takes the attrs, originally this was not the case. And there was a difference between override and overrideAttrs.
Interestingly you cannot make this work if you use overrideAttrs.

Now I find that you need node-gyp-build for both node2nixProd and node2nixDev.
This makes sense as the node2nixProd will only install production dependencies, whereas node2nixDev will install both production and development dependencies. So if there's an override, both require the node-gyp-build to be available.
Now that override and overrideAttrs both take a function with attrs, I wonder if this is because override delegates to overrideAttrs if it's an attribute function. Because nixpkgs upstream says that override is meant to override the function parameters. But this doesn't make any sense if there is access to existing attributes.

Yea I checked override being able to take attrs and applying something like buildInputs is specific to node2nix.

Regarding override issues: https://github.com/svanderburg/node2nix/issues/178
I've changed to using override instead of overrideAttrs for both node2nixDev and node2nixProd.
Regarding the build warnings of creating a homeless shelter that's cause pkg is attempting to download prebuilt-binaries but instead building from source should be possible.
This is solved with this environment variable in the release builds:
      # ensure that native modules are built from source
      npm_config_build_from_source = "true";

Regarding the application build, both dev and prod require the node-gyp-build due to leveldown. So this will be important for any future native modules we are adding in, the build system will become ever more complex. So it's ideal we document what each thing is for.

@tegefaulkes still has to fix up the executable itself, the arguments to ./result/bin/typescript-demo-lib should be all optional. I think some arguments were introduced by @emmacasolin before where there was no defaulting value for them. Right now not passing it the right set of args results in weird errors.
This is also important for the docker output which doesn't pass any parameters to the command. It will look like something is broken without this being fixed.

My last commit ensures parity between nix-shell and nix-build. This should ensure that pkg being used inside nix-shell is the same as that which is used for the release. These changes will need to be applied to js-polykey too.
At the final point make sure to test all builds, application, docker, and the executables. And then merging time!


cannot enable peer access from device ordinal 0 to device ordinal 0

Hey guys. I just tried latest precompiled version of tf-coriander and it's failing with following call-stack for helloworld app
OpenCL platform: Intel(R) OpenCL
OpenCL device: Intel(R) HD Graphics
I tensorflow/core/common_runtime/gpu/gpu_device.cc:989] Found device 0 with properties:
name: Intel(R) HD Graphics
major: -1 minor: -1 memoryClockRate (GHz) 1150
pciBusID 0000.0000
Total memory: 12.33GiB
Free memory: 4.00GiB
I tensorflow/core/common_runtime/gpu/gpu_device.cc:877] cannot enable peer access from device ordinal 0 to device ordinal 0
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1011] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1021] 0:   N
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1083] Creating TensorFlow device (/gpu:0) -> (device: 0, name: Intel(R) HD Graphics, pci bus id: 0000.0000)
cl_driver DeviceAllocate<PHONE_NUMBER>
b'Hello, TensorFlow!'

Output of clinfo is this
Number of platforms                               1
  Platform Name                                   Intel(R) OpenCL
  Platform Vendor                                 Intel(R) Corporation
  Platform Version                                OpenCL 2.0
  Platform Profile                                FULL_PROFILE
  Platform Extensions                             cl_intel_accelerator cl_intel_advanced_motion_estimation cl_intel_device_side_avc_motion_estimation cl_intel_driver_diagnostics cl_intel_media_block_io cl_intel_motion_estimation cl_intel_planar_yuv cl_intel_packed_yuv cl_intel_required_subgroup_size cl_intel_subgroups cl_intel_subgroups_short cl_intel_va_api_media_sharing cl_khr_3d_image_writes cl_khr_byte_addressable_store cl_khr_depth_images cl_khr_fp16 cl_khr_fp64 cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_icd cl_khr_image2d_from_buffer cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_mipmap_image cl_khr_mipmap_image_writes cl_khr_spir cl_khr_subgroups
  Platform Extensions function suffix             INTEL

  Platform Name                                   Intel(R) OpenCL
Number of devices                                 1
  Device Name                                     Intel(R) HD Graphics
  Device Vendor                                   Intel(R) Corporation
  Device Vendor ID                                0x8086
  Device Version                                  OpenCL 2.0
  Driver Version                                  r5.0.63503
  Device OpenCL C Version                         OpenCL C 2.0
  Device Type                                     GPU
  Device Profile                                  FULL_PROFILE
  Max compute units                               23
  Max clock frequency                             1150MHz
  Device Partition                                (core)
    Max number of sub-devices                     0
    Supported partition types                     by <unknown> (0x7F5900000000)
  Max work item dimensions                        3
  Max work item sizes                             256x256x256
  Max work group size                             256
  Preferred work group size multiple              32
  Preferred / native vector sizes
    char                                                16 / 16
    short                                                8 / 8
    int                                                  4 / 4
    long                                                 1 / 1
    half                                                 8 / 8        (cl_khr_fp16)
    float                                                1 / 1
    double                                               1 / 1        (cl_khr_fp64)
  Half-precision Floating-point support           (cl_khr_fp16)
    Denormals                                     Yes
    Infinity and NANs                             Yes
    Round to nearest                              Yes
    Round to zero                                 Yes
    Round to infinity                             Yes
    IEEE754-2008 fused multiply-add               Yes
    Support is emulated in software               No
    Correctly-rounded divide and sqrt operations  No
  Single-precision Floating-point support         (core)
    Denormals                                     Yes
    Infinity and NANs                             Yes
    Round to nearest                              Yes
    Round to zero                                 Yes
    Round to infinity                             Yes
    IEEE754-2008 fused multiply-add               Yes
    Support is emulated in software               No
    Correctly-rounded divide and sqrt operations  Yes
  Double-precision Floating-point support         (cl_khr_fp64)
    Denormals                                     Yes
    Infinity and NANs                             Yes
    Round to nearest                              Yes
    Round to zero                                 Yes
    Round to infinity                             Yes
    IEEE754-2008 fused multiply-add               Yes
    Support is emulated in software               No
    Correctly-rounded divide and sqrt operations  No
  Address bits                                    64, Little-Endian
  Global memory size<PHONE_NUMBER>7 (12.33GiB)
  Error Correction support                        No
  Max memory allocation<PHONE_NUMBER> (4GiB)
  Unified memory for Host and Device              Yes
  Shared Virtual Memory (SVM) capabilities        (core)
    Coarse-grained buffer sharing                 Yes
    Fine-grained buffer sharing                   No
    Fine-grained system sharing                   No
    Atomics                                       No
  Minimum alignment for any data type             128 bytes
  Alignment of base address                       1024 bits (128 bytes)
  Preferred alignment for atomics
    SVM                                           64 bytes
    Global                                        64 bytes
    Local                                         64 bytes
  Max size for global variable                    65536 (64KiB)
  Preferred total size of global vars<PHONE_NUMBER> (4GiB)
  Global Memory cache type                        Read/Write
  Global Memory cache size                        524288
  Global Memory cache line                        64 bytes
  Image support                                   Yes
    Max number of samplers per kernel             16
    Max size for 1D images from buffer            268434943 pixels
    Max 1D or 2D image array size                 2048 images
    Base address alignment for 2D image buffers   4 bytes
    Pitch alignment for 2D image buffers          4 bytes
    Max 2D image size                             16384x16384 pixels
    Max 3D image size                             16384x16384x2048 pixels
    Max number of read image args                 128
    Max number of write image args                128
    Max number of read/write image args           128
  Max number of pipe args                         16
  Max active pipe reservations                    1
  Max pipe packet size                            1024
  Local memory type                               Local
  Local memory size                               65536 (64KiB)
  Max constant buffer size<PHONE_NUMBER> (4GiB)
  Max number of constant args                     8
  Max size of kernel argument                     1024
  Queue properties (on host)
    Out-of-order execution                        Yes
    Profiling                                     Yes
  Queue properties (on device)
    Out-of-order execution                        Yes
    Profiling                                     Yes
    Preferred size                                131072 (128KiB)
    Max size                                      67108864 (64MiB)
  Max queues on device                            1
  Max events on device                            1024
  Prefer user sync for interop                    Yes
  Profiling timer resolution                      83ns
  Execution capabilities
    Run OpenCL kernels                            Yes
    Run native kernels                            No
    SPIR versions                                 1.2
  printf() buffer size                            4194304 (4MiB)
  Built-in kernels                                block_motion_estimate_intel;block_advanced_motion_estimate_check_intel;block_advanced_motion_estimate_bidirectional_check_intel
  Motion Estimation accelerator version	(Intel)   2
  Device Available                                Yes
  Compiler Available                              Yes
  Linker Available                                Yes
  Device Extensions                               cl_intel_accelerator cl_intel_advanced_motion_estimation cl_intel_device_side_avc_motion_estimation cl_intel_driver_diagnostics cl_intel_media_block_io cl_intel_motion_estimation cl_intel_planar_yuv cl_intel_packed_yuv cl_intel_required_subgroup_size cl_intel_subgroups cl_intel_subgroups_short cl_intel_va_api_media_sharing cl_khr_3d_image_writes cl_khr_byte_addressable_store cl_khr_depth_images cl_khr_fp16 cl_khr_fp64 cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_icd cl_khr_image2d_from_buffer cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_mipmap_image cl_khr_mipmap_image_writes cl_khr_spir cl_khr_subgroups

NULL platform behavior
  clGetPlatformInfo(NULL, CL_PLATFORM_NAME, ...)  No platform
  clGetDeviceIDs(NULL, CL_DEVICE_TYPE_ALL, ...)   No platform
  clCreateContext(NULL, ...) [default]            No platform
  clCreateContext(NULL, ...) [other]              Success [INTEL]
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_CPU)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_GPU)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_ACCELERATOR)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_CUSTOM)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_ALL)  No platform

Is there anything I can do to make it work ?
Thanks!

And looks like this bug is related https://github.com/hughperkins/tf-coriander/issues/52

@unoexperto why do you think helloworld fails?

I get the same, it there any update? Could it be some env setup I need?


Keyword gymnastics with diffuse model

I attempted to have the new polarized diffuse model only be sutracted, and not calibrated on. The set of keywords I used to do this was:
diffuse_model=filepath('EoR0_polarized_diffuse.sav',root=rootdir('FHD'),subdir='catalog_data')
model_visibilities=1
undefine, diffuse_calibrate
I ran into error in fhd_struct_combine_skymodel.pro about the lack of definition of the catalog_path keyword.
Logs are below, along with settings file. Was run on calibration_sim branch in "normal" firstpass mode (no simulation), which was last merged with main branch 1e8ed60.
1061316296_settings.txt
firstpass.e64612.1.txt
firstpass.o64612.1.txt

This bug was fixed in v3.5-56-gc3af389. After a fresh pull it should be fixed.


Fix jsr badge.

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

Project coverage is 92.00%. Comparing base (ec56c0d) to head (416c8a0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #379   +/-   ##
=======================================
  Coverage   92.00%   92.00%
=======================================
  Files          53       53
  Lines        3792     3792
  Branches      352      352
=======================================
  Hits         3489     3489
  Misses        302      302
  Partials        1        1

Flag
Coverage Δ

unittests
92.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Can't type numbers in chat?

Hello!
I'm not sure if this is a bug or things working as intended, but trying to use the number row while in the chatbox still uses the hotbar buttons. The numpad still works but that's kind of an awkward workaround when typing.
I love this addon though! Making macros was making my head spin, but having an actual interface for it helps me immensely.
Cheers!

same issue as here:
https://github.com/ThornyFFXI/tHotBar/issues/10
change the ashita setting

That was indeed it! Thank you for the help!


Disabling changelog uploading

Thank you for this PR.
Wouldn't it be better if the changelog upload was parametrized ?

Yes it's would be better

Who could do it ?

Nope since I will re-implment mercure in fastlane

I will do it in Fastlane directly

OK.
I will have a deeper look at fastlane when I'll do my next iOS job


Add orthography profile

Ah, just saw the issue. Ignore the release commit, sorry. :)

@SimonGreenhill this is not going to happen for CLICS 3, right?

unlikely, unless @tresoldi has time?

Will release now, we can still provide the profile later, ok? :)

I've started to work on it: https://github.com/lexibank/transnewguineaorg/tree/profile
Far from finished, I'll open a PR when I need you guys involved. :smiley:


Does marl support switching context while unwinding stack? i.e. is it exception safe?

I know windows fiber is exception safe with proper compiler flags. But gcc/llvm based implementations typically require save/restore cxa global states while switching context to assure correct exception behaviour, which I didn't find in the source.

Probably not. I'm not aware of marl being used in any project with exceptions enabled, and I certainly didn't consider exceptions during implementation. Sorry.


design trade history & skew panel

design trade history / skew panel
find a good place to display the info

https://www.figma.com/file/YWU3PMsnqZaqr1gABgAFJK/UI-Refresh?node-id=2221%3A28166

individual cells in the trade history table should:

change cursor on hover
change background on hover
on click should take you to the tx on optiscan


Add Rescience paper as unit test for Unitary Events

The unitary events implementation in Elephant was validated against the original publication of 1997 in
https://github.com/ReScience-Archives/Rostami-Ito-Denker-Gruen-2017
Parts of this validation should be integrated as unit test into Elephant. One possibility would be to load the original UE time stamps from a file, and compare them to the output of the unitary_events function.

@Vahidrostami Hi, have you had the chance to make any progress on this issue? Could I be of help?

Done : https://github.com/NeuralEnsemble/elephant/pull/152


Replace all os.system() calls with suprocess.run()

Modernize code by replacing os.system with subprocess.
Also allows trapping more kinds of potential errors with cloning and syncing.
Fixes #73
Addresses some of the issues in #75.
Tag @mscherer @kalkayan

Updated Docker image to Python 3.7-stretch.


vrscene creator Deadline webservice URL handling

Fix
This is fixing obsolete use of API to determine Deadline URL that was causing creator crash.
⚠ WARNING: it seems that vrscene publishing is still broken somewhere because.

Task linked: OP-2478 vrscene creator in Maya and Deadline URL


[feature-1] staircase_search.cpp

Closes: #385
Implemented the StairCase Search in C++

[x] Code is well-formatted
[x] Code is well commented
[x] Code is not plagiarised
[x] Code is placed in the proper folder
[x] readme.md of 2D_Array is updated with the code link

@tarun26091999 Please review this and suggest me any changes if required.

@tarun26091999 I have added comments to my code. And, I am not sure about which searching folder you are saying. Is it the one inside the Algorithms folder? If so, can you please explain to me in detail what changes do I have to make in that folder?

@tarun26091999 Can you please guide me regarding the comment I gave above

@tarun26091999 Please review this

@tarun26091999 Please review this

@tarun26091999 Please review this

@rudrakshi99 Can you please review this

@tarun26091999 I have added comments to my code. And, I am not sure about which searching folder you are saying. Is it the one inside the Algorithms folder? If so, can you please explain to me in detail what changes do I have to make in that folder?

this one - https://github.com/Algo-Phantoms/Algo-Tree/tree/main/Algorithm/Searching_Sorting

@rudrakshi99 I made the changes. Suggest me any more changes if required

@OjusWiZard Please recheck it again. It has been more than 20 days waiting for this PR to get reviewed


GithubRelease task: clarify the action

First off, the action must be case sensitive as of today.
Secondly, it is strange that I have to use the Edit action if I would like to create the tag as part of creating the GH release.
See here:
https://github.com/Microsoft/azure-pipelines-tasks/blob/d9cfbdfcfcd98bf8dfb162128123075d3035cc40/Tasks/GithubReleaseV0/main.ts#L56-L66

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

ID: 3b12d2c4-f871-1d14-0c16-91e58fb1130b
Version Independent ID: 6b9e1d67-f303-7445-e573-830309ba49a4
Content: GitHub Release task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/github-release.md
Product: devops
GitHub Login: @erickson-doug
Microsoft Alias: dastahel

@Raiyan<EMAIL_ADDRESS>– can you please take a look?
Thanks
Roopesh Nair
From: Steve Danielson<EMAIL_ADDRESS>Reply-To: MicrosoftDocs/vsts-docs<EMAIL_ADDRESS>Date: Friday, 14 December 2018 at 8:50 PM
Subject: Re: [MicrosoftDocs/vsts-docs] GithubRelease task: clarify the action (#2755)
Assigned #2755https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fvsts-docs%2Fissues%2F2755&data=02|01|roopesh.nair%40microsoft.com|7131af9cc98441caed7f08d661d7b837|72f988bf86f141af91ab2d7cd011db47|1|0|636803976471511457&sdata=6a1tT2wNnFOlO0Oqd8t8YrtXp%2F6zMT1ZtfTaavxizv0%3D&reserved=0 to @RoopeshNairhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRoopeshNair&data=02|01|roopesh.nair%40microsoft.com|7131af9cc98441caed7f08d661d7b837|72f988bf86f141af91ab2d7cd011db47|1|0|636803976471521470&sdata=gaTMavqbxblN5ghEVYnwBbs3AkKmYLAyVHa%2FFaKNZBk%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fvsts-docs%2Fissues%2F2755%23event-2027084038&data=02|01|roopesh.nair%40microsoft.com|7131af9cc98441caed7f08d661d7b837|72f988bf86f141af91ab2d7cd011db47|1|0|636803976471521470&sdata=utMsrUpzJTUg1qQhxX7MQrsifKQ4qDg%2Bk6eExp7tVvE%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABTpcHFbkOnYxg3G7i1LETwlsTyw4oMhks5u48HMgaJpZM4ZRF3s&data=02|01|roopesh.nair%40microsoft.com|7131af9cc98441caed7f08d661d7b837|72f988bf86f141af91ab2d7cd011db47|1|0|636803976471531471&sdata=1o85lAxPq%2FoV6VUVeHa6pUXpTctbzKqw5ogCn0qcfZM%3D&reserved=0.

@kittaakos Apologies for the inconvenience.
The task can create tags even when create  action is used. However, you need to provide us what tag you want us to create. For doing this we have a parameter called 'tagSource'. Set it to 'manual' and in the 'tag' parameter mention the tag you want to create. Here is the yaml that would might be useful to you:
jobs:
- job: release_macOS
  displayName: 'Release Build (macOS)'
  pool:
    vmImage: 'macOS-10.13'
  steps:
  - script: rm -rf dist && mkdir dist && echo `date` > dist/my-release.txt && zip -jr ./dist/my-release.zip ./dist/my-release.txt
    displayName: 'Create Release'
  - task: PublishBuildArtifacts@1
    inputs:
      pathtoPublish: dist/my-release.zip
      artifactName: 'my-release.zip'
    displayName: 'Publish Release'
  - task: GithubRelease@0
    inputs:
      gitHubConnection: kittaakos
      repositoryName: kittaakos/azure-sandbox
      assets: $(build.artifactstagingdirectory)/*.zip
      action: 'create'
      target: '$(build.sourceVersion)'
      tagSource: 'manual'
      tag: '$(myVariableForUserDefinedTag)'
      title: 'Tag Title'
      assetUploadMode: 'delete'
      isDraft: true