Refactor project into a more useful seed

Right now, to use this project, you have to clone it and then rip out the parts you don't need. You also have to come up with a good understanding of how all the parts fit together and how the gulpfile interacts with them. Sure, it is pretty small (right now), but it is kind of intimidating for a new user. This project isn't the only one with this issue, all seed projects kind of suck in this same way.
I've been thinking about this a lot. There is other projects like Yeoman, which give you the Rails type experience of being able to 'new' a project with a generator model and create templates for things like controllers, etc. This is great for just starting off, but doesn't solve the issue of when the base project changes, updating your own project, most of that code lives in the gulpfile.
Sorry this is so long winded.
What I was thinking as a first step to providing a better experience would be to extract out all of the gulp tasks that do stuff into functions and put them into their own npm module. This way, the gulpfile just loads a bunch of functions that can be more easily re-used (and therefore updated).
Once we have that, I think that building a yeoman experience might be the next step. Then we won't have to rip out all the existing example code as yeoman can generate that for us.
Thoughts?

We have a blog article we are going to do soon which will help with this.

Where did you see it done?
The angular 2 gulpfile (tools/build/*) is what got me thinking about it.

This is one of them i've seen; but i seen a better one i know https://github.com/gocardless/es6-angularjs/tree/master/script/lib

:+1: (i even like the one you had in your pre-edited comment)

Ya both are good, that one wasn't gulp per say just one off scripts which is why i switched to that one.

Ok, I've taken a stab at this and so far I'm pretty happy with the initial result. It can certainly get better, but it is a good first pass. It is called gulp-helpers. You can see an example of a full working gulpfile in my seed project. As you can see, the file is now mostly just configuration and very little code. Sure makes working with seed projects as a basis now so much easier.
gulp-helpers is written in ES6 too. Seemed like a good way to start to get my hands dirty with it.
Enjoy. =)

I like your update, wanna un-coffee ;) it and do a PR?

reference @lookfirst gulp file: https://github.com/lookfirst/systemjs-seed/blob/master/gulpfile.js  ...

For what it's worth, I'm joining this project (using it for a huge, just-now-starting-development production app) from a previous seed of https://github.com/DaftMonk/generator-angular-fullstack.  Personally, I would love to see this project move in that direction.  For me, I had to refactor it all to accommodate my Node backend living in the same project root.  One nice thing about generator-angular-fullstack is that it doesn't force it's opinions on you, so you just run the generator and and choose if you want a Node backend or not and then the generated code (from the yeoman generator) accommodates your choice.  I don't think we should bother with supporting every configuration under the sun, but if the project became a yeoman generator, then the burden could be placed on individuals wanting ____ (tech thing) to PR that and it would not hurt those who don't want to use (LESS, SASS, CoffeeScript, Node Backend).

@brianfeister you may want to look at what i've done with this project. i've got a generator started too.

today, i'm going to split out the routing.js file into its own project. that is the last thing that is really specific.

Until we can remove it totally :)

@ocombe Hurry up! ;-)

Are you separating your front and backend @lookfirst ?  Also, make sure to steal liberally from https://github.com/DaftMonk/generator-angular-fullstack/blob/master/app/index.js for examples of giving users prompts in yeoman like "Do you want a Node backend?" and "Do you want to use React?"

Also, have a look at https://github.com/Swimlane/systemjs-route-bundler/issues/15 for a problem I ran into when migrating the JS into /client/src rather than project root

@brianfeister I see this as a strictly frontend project. I see the future as being that the frontend and backend are really two different beasts communicating through JSON. (Don't forget about my other project JSOG).
If you want to throw everything into a client and server folder, that's up to you, but I think they should have separate build processes.
In terms of using React (ngReact) or not, that is just a) a dependency in your package.json / system.config.js b) it doesn't affect anything. The fewer questions we have to ask, the easier everything is.

+1 to hurry up @ocombe  ;)

:D
According to my plans, I should get v1.0 out this week, then I'll start working on 1.1 that should be finished by the end of the month.
After that it's 1.2 with the plugins for SystemJS & Webpack.

@ocombe I see webpack getting a lot of attention.  Why would you choose that over System since System is ES6.

Because of jspm ?

Wait, you're asking why someone would choose webpack instead of systemjs ?

I'm seriously not a fan of webpack. I tried it and found it to be a giant mess with terrible documentation. SystemJS + SystemJS builder is the future.

Well I've watched the videos on egghead.io by @kentcdodds and it looks freakingly cool :)
It's really smart about the module building part and will optimize the bundles so that you will load the minimum number of files (instead of one file per resource like it is the case with systemjs).
And you can use ES6 with Webpack as well.

Also a lot of people that I know recommended webpack to me, and they are really good at what they do, so I'm inclined to trust their judgment. I haven't tried it myself yet, but I'm willing to give it its chance.

Great to hear @ocombe! :D

By the way @kentcdodds, why do you use require instead of import in your videos with webpack? Since your code is transpiled by babel, it should work, right?

@ocombe, yes, I use them interchangeably actually... Weird. I should probably just conform to the ES6 style I think... Haha.

@ocombe I gave it a stab but had no luck.  I've got some articles of @kentcdodds to go through after we chatted at ngconf.
@ocombe I wrote a bundle system that utilizes systemjs's build system to do that kind of bundling, is that similar to what webpack is doing?

@ocombe I found this idea by @amcdnl and his team to be more in line with what I'd want to see... route based bundling! https://github.com/Swimlane/systemjs-route-bundler

git clone https://github.com/lookfirst/systemjs-seed.git
npm install
SITUATION=production gulp run
http://localhost:9000/todo
http://localhost:9000/react

Now, look at what is loaded by the browser when you visit those urls. =)

Anyway it doesn't really change what I'll be working on, which is a hook on require to register angular components when their files are loaded. Since require is used both by systemjs & webpack, and pure ES6 code transpiled to ES5 by babel, it should work!
:)

@ocombe ya, I think thats the most painful thing at the moment.  having to re-write the modules is the most annoying thing ever - http://www.screencast.com/t/VWonfVKNcp

well, get used to it because you'll probably use it more and more :)
Unless we can imagine a tool that would add the import in your files at build time

if you use my version of ng-html2js, you get the strings for you...
import fooTpl from './views/foo.tpl'
angular.module(module.id, [fooTpl.name])

(note the use of module.id too!)

@ocombe no I was meaning having to write the import and then declare the dependency in the angular module.

Ah yes, @kentcdodds has a trick for that, see https://github.com/kentcdodds/webpack-angular/blob/master/app/kcdModule/index.js

@ocombe you can't nest ES6 syntax like that though. :(

yes, it only works with require :)

@ocombe that's the reason that I sometimes mix es6 and require

@ocombe @kentcdodds Ya, I had to do that here: https://github.com/Swimlane/angular-systemjs-seed/blob/master/src/common/utils/routing.js#L13 with system.  I hate it.

@kentcdodds I'd like to see how you mix require with es6 and writing unit tests. ;-)

Writing unit tests with ES6 is actually quite awesome. Man I wish I had time to do all the egghead.io lessons I could think of!


nixos/gpaste: also add to cinnamon session path + fix sessionPath

cinnamon.cinnamon-session: add patch for login shell
nixos/gpaste: also add to cinnamon session path - fixes #276028

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Still wish to get an upstream review if possible, let's hope this can get reviewed before Mint 21.3
To do this in nixpkgs, we probably want to double check with gnome-session packaging, specifically:

They doesn't seem to wrap $out/bin/gnome-session: https://github.com/NixOS/nixpkgs/blob/0b658a0ac7fac55e362c4e219bda06d9aa307417/pkgs/desktops/gnome/core/gnome-session/default.nix#L114
There is another fixup: https://github.com/NixOS/nixpkgs/blob/0b658a0ac7fac55e362c4e219bda06d9aa307417/pkgs/desktops/gnome/core/gnome-session/fix-paths.patch#L11-L12


FYI: New robospock plugin for Gradle

For your information:
I've created a new Gradle plugin for robospock which actually works and makes life ridiculously simple...
Here it is: https://github.com/Centril/gradle-plugin-robospock
You can apply it in a host of different ways, but the most simple is:
buildscript {
    repositories {
        jcenter()
    }
    dependencies {
        classpath 'se.centril.robospock:gradle-plugin-robospock:0.2.0'
    }
}

['android-sdk-manager', 'com.android.application', 'robospock']
	.each { apply plugin: it }

android {
	buildTypes {
		defaultConfig {
			minSdkVersion 9
			targetSdkVersion 20
		}
	}
	compileSdkVersion 20
	buildToolsVersion "20"

	// more exciting stuff...
}

Then you can put all your testing classes in {android-project-root}/src/test/.
Thats it =)

I think it's not valid since google add support to their official plugin


[WIP] Integrate permissions into core

Proposal for resources permissions check:
<Admin
    restClient={restClient}
    authClient={authClient}
>
    {permissions => [
        <Resource name="customers" list={permissions === 'admin' ? VisitorList : null} edit={VisitorEdit} remove={VisitorDelete} icon={VisitorIcon} />,
        <Resource name="commands" list={CommandList} edit={CommandEdit} remove={Delete} icon={CommandIcon} options={{ label: 'Orders' }} />,
        <Resource name="products" list={ProductList} create={ProductCreate} edit={ProductEdit} remove={Delete} icon={ProductIcon} />,
        <Resource name="categories" list={CategoryList} edit={CategoryEdit} remove={Delete} icon={CategoryIcon} />,
        <Resource name="reviews" list={ReviewList} edit={ReviewEdit} icon={ReviewIcon} />,
    ]}
</Admin>

In a nutshell, specify a function instead of nodes as the Admin children. This function will be called with the results of the call to the authClient with type AUTH_GET_PERMISSIONS. Then do whatever you want with it. The authClient has to return a promise as usual but it can resolves to whatever you need: single role, multiple roles or even the user itself.
Note that in the example, we return an array of components to avoid React warning about having to wrap them in a container element.
I updated the List, Create, Edit, Show and Filter components to allow this too. For example:

List

export const PostList = ({ ...props }) =>
    <List
        {...props}
        filters={<PostFilter />}
        sort={{ field: 'published_at', order: 'DESC' }}
    >
        {permissions =>
            <Responsive
                small={
                    <SimpleList
                        primaryText={record => record.title}
                        secondaryText={
                            permissions === 'admin'
                                ? record => `${record.views} views`
                                : null
                        }
                    />
                }
                medium={
                    <Datagrid>
                        {permissions === 'admin'
                            ? <TextField source="id" />
                            : null}
                        {permissions === 'admin' && <TextField source="title" style={titleFieldStyle} />}
                        <DateField
                            source="published_at"
                            style={{ fontStyle: 'italic' }}
                        />
                    </Datagrid>
                }
            />}
    </List>;

Filter

const CommandFilter = props => (
    <Filter {...props}>
        {permissions => [
            <TextInput label="pos.search" source="q" alwaysOn />,
            permissions === 'admin' && <NullableBooleanInput source="returned" />,
        ]}
    </Filter>
);

I tested with the demo app and it works fine :)
I also kept the other components as well (WithPermissions and SwitchPermissions) for usage inside custom menu, custom pages and dashboard.
TODO:

[x] Ability to apply permissions on resources
[x] Use context from Admin to inject the authClient
[x] Integration of aor-permissions components for other use cases (WithPermissions, SwitchPermissions
[x] Updated List, Create, Edit, Show and Filter so they accept either nodes or a function as their children like Admin does
[x] More tests
[x] Documentation

About the usage inside List component, I think the best way to handle it would be to apply the same logic as in Admin. Either provide node as children or a function, which will be called with the permissions.
If we choose this route, I would even apply this to all views (Create, Edit and Show). This might do the job for Filter too.
However, it would nice to keep the other components as well (WithPermissions and SwitchPermissions) for usage inside custom menu, custom pages and dashboard.
export const PostList = ({ ...props }) =>
    <List
        {...props}
        filters={<PostFilter />}
        sort={{ field: 'published_at', order: 'DESC' }}
    >
        {permissions =>
            <Responsive
                small={
                    <SimpleList
                        primaryText={record => record.title}
                        secondaryText={
                            permissions === 'admin'
                                ? record => `${record.views} views`
                                : null
                        }
                    />
                }
                medium={
                    <Datagrid>
                        {permissions === 'admin'
                            ? <TextField source="id" />
                            : null}
                        <TextField source="title" style={titleFieldStyle} />
                        <DateField
                            source="published_at"
                            style={{ fontStyle: 'italic' }}
                        />
                    </Datagrid>
                }
            />}
    </List>;

ping @fzaninotto

I'm not sure about the SwitchPermissions component. I can't figure out a use case it would handle better than multiple WithPermissions components. Thoughts ?


batch join wont compile

i got an issue trying to use join. kapacitor v 1.0 beta, influxdb v 0.13.0
my tick:
var one = batch
    |query(''' SELECT 100*mean("value") FROM "metrics"."default"."134" WHERE "measure" == 'impressions' AND "relative" == '0' AND "env" == 'exchange' AND "type" == 'diff_percent' ''')
        .period(60m)
        .every(30s)
        .groupBy('measure', 'relative', 'env', 'type', time(10s))
        .fill(0)
    |eval(lambda: int("mean"))
        .as('mean')

var two = batch
    |query(''' SELECT min("value") FROM "metrics"."default"."134" WHERE "relative" == '0' AND "source" == 'adk2' AND "env" == 'exchange' AND "measure" == 'impressions' ''')
        .period(60m)
        .every(30s)
        .groupBy('measure', 'relative', 'env', 'source', time(10s))
        .fill(0)
    |eval(lambda: "min"/"mean")
        .as('bla')

one
    |join(two)
      .as('one', 'two')

    |eval(lambda: "two"/"one")
        .as('bla')

    |alert()
        .id('{{ .Name }}@{{ index .Tags "measure" }}')
        .message('{{ .ID }} {{ .Level }} measure: {{ index .Fields "bla" }} (threshold: below 0)')
        .crit(lambda: "bla" > 0)
        .log('/adk2/var/kapacitor/alerts.log')
        .slack()
        .channel('#alerts-test')

my error when defining:
found =, expected identifier, string, number, bool at line 1, char 71
well - there is no char 71 in line 1...
help pls

@nirpodo I copy pasted your script and defined it without issue. Is it possible you have non utf-8 data in the original file?

This issue was a misleading error. The error is from parsing the query not parsing the TICKscript.
The specific error is that queries use the = comparison operator instead of ==.


clusterupdate output not consistent with cluster/linclust output

clusterupdate only generates one main database file output (eg., clusters_db), regardless of --threads, while cluster and linclust generate one file per thread (eg., --threads=4 generates clusters_db.0, clusters_db.1, clusters_db.2, clusters_db.3). This leads to pipeline complications, given that downstream processing of the clusters_db may require multiple inputs (clusters_db.*) or just one input (cluster_db). It would help if clusterupdate and cluster/linclust were consistent. It would be best if cluster/linclust just produced one database file per thread.
I'm running mmseqs2  11.e1a1c

I think MMseqs2 12 should now deal with this better. Could you please update? We resolved many issues with cluster updating in the last release.

I've been using v12, and clusterupdate just generates 1 main database file, regardless of the number of threads.

Just to check: is there a command for merging all of the cluster_db.* files generated by linclust and cluster? I didn't see one in the list of subcommands, but it's a long list.

Currently still no, however I just added an environment variable to prohibit MMseqs2 to create split databases.
If you export MMSEQS_FORCE_MERGE=1 split databases will not be produced anymore. This might slow down some intermediate steps somewhat though. I might also build a module to merge databases when I have time.


HTTP/2 Client Example TLS Pipeline Initialization

Motivation:
The HTTP/2 client example is not validating the results of ALPN if TLS is enabled.
Modifications:

Use ApplicationProtocolNegotiationHandler to validate ALPN results.

Result:
Client example validates ALPN results.

@nmittler @normanmaurer - FYI

@normanmaurer - Updated and verified the example runs

Ship it

Cherry-picked 4.1 (035053b) master (81de3e4)


add windows support

use
arpfox  -t <IP_ADDRESS> -i "WLAN"

Interesting! thanks @sxueck, gonna take a look

I cleaned this up a bit, tested and merged: https://github.com/malfunkt/arpfox/pull/10. Good stuff! thanks.


Increase readability of binding docs

Transforms some of the GDScript documentation markup to Rustdoc, using simple de-indentation and replacements. This makes bindings documentation, which has so far been a single huge code block, much more readable.
Adds the unindent crate, which has no further dependencies.
Before:

After:

This would be a first step. We could pull in the regex crate, to do smarter replacements such as [constant HTTPClient.METHOD_GET], and possibly even link them using Rust 1.48's new feature (doc, announcement).

Since now the doc looks almost like proper Rustdoc, but will have GDScript references and code blocks, should each doc be prefixed with a comment

Generated from GDScript

for clarity?

This looks pretty nice! Even if the "parsing" is barely a hack, it's still much better than unparsed text.

Since now the doc looks almost like proper Rustdoc, but will have GDScript references and code blocks, should each doc be prefixed with a comment

Generated from GDScript

for clarity?

Yes, I think that would be a good idea. I think the message can be a bit clearer about the implication that any sample code is probably unusable, though. Perhaps something like the following?

Generated from official XML docs. Sample code is GDScript unless otherwise noted.

Yes, it's a total hack and quite inefficient too 🙂
Would you be OK if at some point in the future we would use the regex crate, and also try to resolve the links? I don't know about the dependency policy, but since it's just the generator and not the end-user library, should probably be OK?
I tried to add the line you suggested, but in my opinion it looks quite busy:

50% of the information is just this line. Maybe we should keep it very short, or insert it only for the documentation strings that contain [codeblock]?

Would you be OK if at some point in the future we would use the regex crate, and also try to resolve the links?

Yes, although nested tags can't be parsed using regular expressions (unless you manually write a recursive descent parser on top of that). For easier "proper" parsing you might want to look into parser combinator libraries, like nom or pest.

I don't know about the dependency policy, but since it's just the generator and not the end-user library, should probably be OK?

Adding dependencies to the generator should be fine in general, I think.

50% of the information is just this line. Maybe we should keep it very short, or insert it only for the documentation strings that contain [codeblock]? Or at the beginning of the file?
I see it a bit like this: it's useful to learn the first time you come across the doc, but from then on you know it. Now every time you read it, it's just noise, so let's keep that to a minimum

Yes, I think it should be best to put the information at the beginning of the file, considering methods with shorter descriptions. Detecting [codeblock]s should also be great if it doesn't take too much effort to do. With the current implementation you can store the string before replacing [codeblock] and compare it with the replacement result, I suppose?

Changes:

Add a line to the beginning of each file
Add a line to beginning of method docs containing [codeblock]. A simple contains() check should do -- not efficient, but string equality should cost roughly the same (at least for equal strings).

I don't think a full-fledged parser is even necessary for 99% of the cases. I didn't suggest regex for nested statements, but rather for expressions of the form [method poll] which could easily be disected into at least `poll`, or possibly even [`poll`](link/to/poll).

Fixed formatting.

bors retry


Login Even With Wrong Password

I checked Auth Functionality and got a bug that i am able to login even with wrong password that i have used while registration this is totally false authentication system, need to work on it.
I want to resolve this bug.

Broo Whole authentication process is messed up i am rewriting script1.js basically two functions login and signup eventListeners is it okay ?

ok


Raises ValueError: Missing some input keys: {'query'} everytime I invoke 'GraphCypherQAChain.from_llm' chain with query present as input keys

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
"""
python
CYPHER_GENERATION_TEMPLATE = '''Task:Generate Cypher statement to query a graph database.
Instructions:
Use only the provided relationship types and properties in the schema.
Do not use any other relationship types or properties that are not provided.
Schema:
{schema}
Note: Do not include any explanations or apologies in your responses.
Do not respond to any questions that might ask anything else than for you to construct a Cypher statement.
Do not include any text except the generated Cypher statement.
Examples: Here are a few examples of generated Cypher statements for particular questions:
Which sections talk about medication?
MATCH (n2:ner_entity)-[new_present_in:PRESENT_IN]->(n1:Chunk)
WHERE n2.name = 'medication'
RETURN DISTINCT n1.section as section_list
The question is:
{query} '''
CYPHER_GENERATION_PROMPT = PromptTemplate(
input_variables=['schema', 'query'],
template=CYPHER_GENERATION_TEMPLATE
)
QA_CHAIN = GraphCypherQAChain.from_llm(
graph=return_graph(), # This is an internal method that returns a graph object
llm=READER_LLM,
verbose=True,
cypher_prompt=CYPHER_GENERATION_PROMPT,
)
QA_CHAIN.invoke(
input = {'query': query},
return_only_outputs=True,
)
"""
query = 'Sample question?'
Error Message and Stack Trace (if applicable)

Entering new GraphCypherQAChain chain...

ValueError                                Traceback (most recent call last)
 in <cell line: 1>()
----> 1 responce = rag.query(
2     query = 'Which sections talk about medication?'
3 )
4 # self.input_key
10 frames
/content/RAG/KG_for_RAG/src/execute_rag.py in query(self, query)
336
337     if (self.query_type is not None) and (self.query_type.lower() in ['self_genrate','prompt']):
--> 338       answer = self.QA_CHAIN.invoke(
339           input = {'query': query},
340           return_only_outputs=True,
/usr/local/lib/python3.10/dist-packages/langchain/chains/base.py in invoke(self, input, config, **kwargs)
164         except BaseException as e:
165             run_manager.on_chain_error(e)
--> 166             raise e
167         run_manager.on_chain_end(outputs)
168
/usr/local/lib/python3.10/dist-packages/langchain/chains/base.py in invoke(self, input, config, **kwargs)
154             self._validate_inputs(inputs)
155             outputs = (
--> 156                 self._call(inputs, run_manager=run_manager)
157                 if new_arg_supported
158                 else self._call(inputs)
/usr/local/lib/python3.10/dist-packages/langchain_community/chains/graph_qa/cypher.py in _call(self, inputs, run_manager)
251         intermediate_steps: List = []
252
--> 253         generated_cypher = self.cypher_generation_chain.run(
254             {"question": question, "schema": self.graph_schema}, callbacks=callbacks
255         )
/usr/local/lib/python3.10/dist-packages/langchain_core/_api/deprecation.py in warning_emitting_wrapper(*args, **kwargs)
168                 warned = True
169                 emit_warning()
--> 170             return wrapped(*args, **kwargs)
171
172         async def awarning_emitting_wrapper(*args: Any, **kwargs: Any) -> Any:
/usr/local/lib/python3.10/dist-packages/langchain/chains/base.py in run(self, callbacks, tags, metadata, *args, **kwargs)
598             if len(args) != 1:
599                 raise ValueError("run supports only one positional argument.")
--> 600             return self(args[0], callbacks=callbacks, tags=tags, metadata=metadata)[
601                 _output_key
602             ]
/usr/local/lib/python3.10/dist-packages/langchain_core/_api/deprecation.py in warning_emitting_wrapper(*args, **kwargs)
168                 warned = True
169                 emit_warning()
--> 170             return wrapped(*args, **kwargs)
171
172         async def awarning_emitting_wrapper(*args: Any, **kwargs: Any) -> Any:
/usr/local/lib/python3.10/dist-packages/langchain/chains/base.py in call(self, inputs, return_only_outputs, callbacks, tags, metadata, run_name, include_run_info)
381         }
382
--> 383         return self.invoke(
384             inputs,
385             cast(RunnableConfig, {k: v for k, v in config.items() if v is not None}),
/usr/local/lib/python3.10/dist-packages/langchain/chains/base.py in invoke(self, input, config, **kwargs)
164         except BaseException as e:
165             run_manager.on_chain_error(e)
--> 166             raise e
167         run_manager.on_chain_end(outputs)
168
/usr/local/lib/python3.10/dist-packages/langchain/chains/base.py in invoke(self, input, config, **kwargs)
152         )
153         try:
--> 154             self._validate_inputs(inputs)
155             outputs = (
156                 self._call(inputs, run_manager=run_manager)
/usr/local/lib/python3.10/dist-packages/langchain/chains/base.py in _validate_inputs(self, inputs)
282         missing_keys = set(self.input_keys).difference(inputs)
283         if missing_keys:
--> 284             raise ValueError(f"Missing some input keys: {missing_keys}")
285
286     def _validate_outputs(self, outputs: Dict[str, Any]) -> None:
ValueError: Missing some input keys: {'query'}
Description
As seen I am passing my input question with the dictionary key 'query' to the GraphCypherQAChain. It still keeps poping the error "Missing some input keys: {'query'} "
System Info
System Information

OS:  Linux
OS Version:  #1 SMP PREEMPT_DYNAMIC Thu Jun 27 21:05:47 UTC 2024
Python Version:  3.10.12 (main, Jul 29 2024, 16:56:48) [GCC 11.4.0]

Package Information

langchain_core: 0.2.32
langchain: 0.2.11
langchain_community: 0.2.0
langsmith: 0.1.99
langchain_google_genai: 1.0.8
langchain_google_vertexai: 1.0.8
langchain_openai: 0.1.7
langchain_text_splitters: 0.2.2

Optional packages not installed

langgraph
langserve

Other Dependencies

aiohttp: 3.10.2
aiosqlite: Installed. No version info available.
aleph-alpha-client: Installed. No version info available.
anthropic: Installed. No version info available.
anthropic[vertexai]: Installed. No version info available.
arxiv: Installed. No version info available.
assemblyai: Installed. No version info available.
async-timeout: 4.0.3
atlassian-python-api: Installed. No version info available.
azure-ai-documentintelligence: Installed. No version info available.
azure-identity: Installed. No version info available.
azure-search-documents: Installed. No version info available.
beautifulsoup4: 4.12.3
bibtexparser: Installed. No version info available.
cassio: Installed. No version info available.
chardet: 5.2.0
cloudpickle: 2.2.1
cohere: Installed. No version info available.
databricks-vectorsearch: Installed. No version info available.
dataclasses-json: 0.6.7
datasets: 2.20.0
dgml-utils: Installed. No version info available.
elasticsearch: Installed. No version info available.
esprima: Installed. No version info available.
faiss-cpu: Installed. No version info available.
feedparser: Installed. No version info available.
fireworks-ai: Installed. No version info available.
friendli-client: Installed. No version info available.
geopandas: 0.14.4
gitpython: Installed. No version info available.
google-cloud-aiplatform: 1.59.0
google-cloud-documentai: Installed. No version info available.
google-cloud-storage: 2.18.2
google-generativeai: 0.7.2
gql: Installed. No version info available.
gradientai: Installed. No version info available.
hdbcli: Installed. No version info available.
hologres-vector: Installed. No version info available.
html2text: Installed. No version info available.
httpx: 0.27.0
httpx-sse: Installed. No version info available.
javelin-sdk: Installed. No version info available.
jinja2: 3.1.4
jq: Installed. No version info available.
jsonpatch: 1.33
jsonschema: 4.23.0
lxml: 4.9.4
markdownify: Installed. No version info available.
motor: Installed. No version info available.
msal: Installed. No version info available.
mwparserfromhell: Installed. No version info available.
mwxml: Installed. No version info available.
newspaper3k: Installed. No version info available.
numexpr: 2.10.1
numpy: 1.26.4
nvidia-riva-client: Installed. No version info available.
oci: Installed. No version info available.
openai: 1.40.8
openapi-pydantic: Installed. No version info available.
oracle-ads: Installed. No version info available.
oracledb: Installed. No version info available.
orjson: 3.10.7
packaging: 24.1
pandas: 2.1.4
pdfminer-six: 20240706
pgvector: Installed. No version info available.
pillow: 9.4.0
praw: Installed. No version info available.
premai: Installed. No version info available.
psychicapi: Installed. No version info available.
py-trello: Installed. No version info available.
pydantic: 2.8.2
pyjwt: 2.9.0
pymupdf: Installed. No version info available.
pypdf: 4.2.0
pypdfium2: Installed. No version info available.
pyspark: Installed. No version info available.
PyYAML: 6.0.2
rank-bm25: Installed. No version info available.
rapidfuzz: Installed. No version info available.
rapidocr-onnxruntime: Installed. No version info available.
rdflib: Installed. No version info available.
requests: 2.32.3
requests-toolbelt: Installed. No version info available.
rspace_client: Installed. No version info available.
scikit-learn: 1.3.2
SQLAlchemy: 2.0.32
sqlite-vss: Installed. No version info available.
streamlit: Installed. No version info available.
sympy: 1.13.1
telethon: Installed. No version info available.
tenacity: 8.5.0
tidb-vector: Installed. No version info available.
tiktoken: 0.7.0
timescale-vector: Installed. No version info available.
tqdm: 4.66.5
tree-sitter: Installed. No version info available.
tree-sitter-languages: Installed. No version info available.
typer: 0.12.3
typing-extensions: 4.12.2
upstash-redis: Installed. No version info available.
vdms: Installed. No version info available.
xata: Installed. No version info available.
xmltodict: Installed. No version info available.

@AkashBais in the Prompt do not use query but use question and it should work:

CYPHER_GENERATION_TEMPLATE = '''Task:Generate Cypher statement to query a graph database.
Instructions:
Use only the provided relationship types and properties in the schema.
Do not use any other relationship types or properties that are not provided.
Schema:
{schema}
Note: Do not include any explanations or apologies in your responses.
Do not respond to any questions that might ask anything else than for you to construct a Cypher statement.
Do not include any text except the generated Cypher statement.
Examples: Here are a few examples of generated Cypher statements for particular questions:

Which sections talk about medication?
MATCH (n2:ner_entity)-[new_present_in:PRESENT_IN]->(n1:Chunk)
WHERE n2.name = 'medication'
RETURN DISTINCT n1.section as section_list

The question is:
{question} '''

CYPHER_GENERATION_PROMPT = PromptTemplate(
input_variables=['schema', 'question'],
template=CYPHER_GENERATION_TEMPLATE
)
# print(CYPHER_GENERATION_PROMPT.input_variables)
QA_CHAIN = GraphCypherQAChain.from_llm(
graph=return_graph(), # This is an internal method that returns a graph object
llm=llm,
verbose=True,
cypher_prompt=CYPHER_GENERATION_PROMPT,
)
QA_CHAIN.invoke(
input = {'question': query},
return_only_outputs=True,
)

@keenborder786 I did try that but i get the same error.
I noticed that the implimentation of GraphCypherQAChain has query hardcoded in it as input key.
"input_key: str = "query"  #: :meta private:"
This key is used to populate question variable in _call method that gets passed to the cypher_generation_chain
question = inputs[self.input_key]
'''some other code'''
generated_cypher = self.cypher_generation_chain.run(
{"question": question, "schema": self.graph_schema}, callbacks=callbacks
)

@keenborder786 Would appriciate any help or direction on this. Or just a list of plausible causes for this error or something to start looking into . Let me know .Thanks

For anyone looking for a solution.
Step 1: Set the key for question in QA_CHAIN.invoke as query because thats the key that is hard coded in the GraphCypherQAChain and it checks for it's existance. Hence the error [Check the comment above]
Step 2: Set the input variable for the prompt and PromptTemplae as question because when  QA_CHAIN.invoke calls it's internal _call method it dose so using
generated_cypher = self.cypher_generation_chain.run(
{"question": question, "schema": self.graph_schema}, callbacks=callbacks
)

And so the question passed along with the key query is passed down further as question


Tests does not run out of the box

When cloning this repository, installing with dev dependencies, running pytest, I get the following:
ImportError: urllib3 v2 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'OpenSSL 1.0.2k-fips  26 Jan 2017'. See: https://github.com/urllib3/urllib3/issues/2168

Simply importing requests will reproduce this:
import requests
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/private/peesv/bin/sumo/fmu-sumo/_venv/lib64/python3.8/site-packages/requests/__init__.py", line 43, in <module>
    import urllib3
  File "/private/peesv/bin/sumo/fmu-sumo/_venv/lib64/python3.8/site-packages/urllib3/__init__.py", line 41, in <module>
    raise ImportError(
ImportError: urllib3 v2 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'OpenSSL 1.0.2k-fips  26 Jan 2017'. See: https://github.com/urllib3/urllib3/issues/2168

Using latest version (2.31.0) of requests.

There is a known OpenSSL related issue on urllib3 v2 on RHEL 7, so komodo team ensures we are on older urllib3 until RHEL8: see their details here: https://github.com/equinor/komodo-releases/blob/1f89d125f0ab27c6629102c7ecf2a508f95c5847/releases/matrices/bleeding.yml#L351
I am not able to recreate this issue on my Windows laptop, PY3.9, create new venv:

pip install .[dev]
pytest
Runs as expected.
pip list gives: requests  2.31.0 and urllib3  2.2.1

RGS, komodo stable, komodoenv (m3 + 2024.02.12-py38-rhel7):

fmu-sumo already installed by default
pip install pytest
pytest
Runs as expected
pip list gives: requests 2.31.0 and urllib3 1.26.18

Not sure how to set up an environment that triggers this problem: python3 venv on RHEL7 but not komodo?
We could demand a lower version for fmu-sumo in the toml-file, but I assume that could give other problems on other platforms, and also we would have to remember to edit the toml.file again when RHEL7 is out.
Propose to not leave as is.


Fix settings won't sync on core v20.8.1

As reported in https://github.com/MycroftAI/mycroft-core/issues/2842
The Skill settings stopped syncing due to the Skill storing information on currently and recently played items in the Skill settings object.
This moves all values that are not Skill settings to a local json database stored in the Skills namespaced filesystem.
An auto-format and small import clean up was also done.
FuzzyWuzzy has been updated to incorporate fixes for Python3.7.
To Test

Ensure you are running mycroft-core 20.8.1 (or latest dev branch)
Install new version of Skill - if switching branches ensure you get the new dependenciesmycroft-pip install -r /opt/mycroft/skills/mycroft-pandora.mycroftai/requirements.txt

Delete any existing settings.json
Reboot Mycroft.

skills.log
Same result

consider using https://github.com/maxbachmann/rapidfuzz
fuzzywuzzy is GPL


meta-lxatac-bsp: linux-lxatac: upgrade to 6.11

umpf tag is now: 6.11/customers/lxa/lxatac/20240830-1 The tag includes the following topic branches:

v6.11/topic/reproducible-build
v6.8/customers/lxa/lxatac

TODO before merging:

[x] Test on hardware
[x] Wait for Kernel 6.11 to be released

I will have to investigate whether the kernel update is actually the root cause here, but the in the bundle I just tested (which includes this kernel update) the tacd crashed with a SIGBUS error in this piece of code:
https://github.com/linux-automation/tacd/blob/a25b8a72e8b7c5d10c2de39190c80b8f77d709f6/src/ui/display.rs#L99-L101

I've just double-checked. The SIGBUS does not occur in Kernel 6.10. It is very unfortunate that 6.11 is already at rc5, because I will be on vacation next week and will not be able to investigate this before the release.

I think I have found the offending commit: https://github.com/torvalds/linux/commit/86634fa4e6aeff2e190616c304156899beb4d76b the web version of the diff is not really helpful, but the command line tool shows that the merge commit contains changes which were likely introduced to fix a merge conflict.
One of these changes affects drivers/gpu/drm/drm_fbdev_dma.c, which (at least by name) matches the error we see.
Both parents of the merge commit compile and work fine, but the merged version does not (resulting in the previously mentioned SIGBUS error).

Thanks to @pH5 we now know why https://github.com/torvalds/linux/commit/86634fa4e6aeff2e190616c304156899beb4d76b is the first commit to expose the problem.
It brings together https://github.com/torvalds/linux/commit/9317ff69170d and https://github.com/torvalds/linux/commit/d92a7580392a for the first time.
The first one changes panel-mipi-dbi from being a drm_fbdev_generic device to being a drm_fbdev_dma device.
The second one prevents drm_fbdev_dma devices from leaking the memory address of the framebuffer to userspace unless the drm_kms_helper.drm_leak_fbdev_smem=1 kernel commandline option is set.
This being an issue is likely a bug in the panel-mipi-dbi driver that should be fixed there, but this workaround makes sure we can use 6.11 until the driver is fixed.
Thanks again to @pH5 for finding the two commits and the workaround.

Hi @pH5, would you be interested reviewing for this PR?

Looks fine to me. Maybe add a note that when the drm_kms_helper.drm_leak_fbdev_smem=1 workaround is removed, CONFIG_DRM_FBDEV_LEAK_PHYS_SMEM should be disabled again as well?

Good idea! I've added a comment  above the BOOTSPEC_OPTIONS += line:
# Once drm_leak_fbdev_smem is removed the kernel config option
# CONFIG_DRM_FBDEV_LEAK_PHYS_SMEM should be disabled as well.


JavaScript running on Photino will not get any keyboard events until window is clicked

Hello,
As the title says - no matter what I do, until clicking the Photino window, no keyboard events will be fired. I even tried to force windows to focus on the window (using the window handle) but it didn't work.
Any way around this?
Thanks! :)

@RonenNess Apparently, focus is set to the web control on Mac and Linux automatically, but not in Windows. We seem to have fixed that just in time for our next release (in the next few days).

That's great, thanks!
Can you please add a method to focus on the WebView window while at it? Its a very useful functionality and sounds like something you could do from your end.
Thanks :)

@RonenNess That is actually what fixed the keyboard event issue. On Mac and Linux, the control gets focus by default since it's the only control on the form. On Windows, we had to explicitly set focus on it.


Expose Rviz Viewpoint Transformation

It would be a nice feature to be able to get the transformation of the Rviz viewpoint w.r.t. the fixed frame. Teleoperation is a good use case for this, as it can be difficult/unintuitive to control a robot w.r.t. a fixed frame when changing to viewpoint.
Thoughts?

Sounds reasonable to me, only question is in which ways:

programmatically (via a plugin)
a ROS topic
tf broadcaster

Personally, I'd like to see a new kind of plugin (maybe a "ViewControllerObserver" plugin) which has access to this information, but that would just be "nice to have" (following the dog fooding principal). That plugin interface could then be used to implement a plugin for that interface which publishes to a topic and/or does a tf broadcast. I think the ros topic could be "always on", but for the tf broadcaster you would definitely want to have that opt-in only.
Original ROS answers question: https://answers.ros.org/question/283462/rviz-viewpoint-transform/

Is this feature implemented? Or is there any other way to get the transformation of rviz viewpoint?
Thanks!

No, as far as I know, this feature is not yet implemented. But you are invited to so.

Hi rhaschke,
Thanks for your information. I would love to help implementing this feature, but I have not read any rviz code before. If anyone can provide some hints on where I should start looking into, or maybe a basic outline of steps to do this implementation, that would be very helpful.
Muqing

For those interested I did a quick modification of the rviz_animated_view_controller so that it publishes the rviz view position and orientation as a pose message at 10Hz. I tested it under ROS Kinetic. Not a very neat solution but sufficient for people who just want this piece of information.
You are welcome to try it here.
Best Regards,
Muqing


Deserialize with additional data

I have a medium-complexity AST tree structure that I'd like to serialize.
The issue is that the AST stores only Range<usize> pointers to a String, and I'd like to serialize to JSON the strings, not usize.
Would it be possible to somehow specify additional data to be passed down through the serde derive to the place where the additional data can be used to turn a Range<usize> into  &str`?
Minimized example:
struct Resource(Vec<Entry>);

enum Entry {
    Message(Message),
}

struct Message {
    id: Identifier,
    value: Option<Value>
}

struct Identifier {
    name: Range<usize>
}

struct Value {
    content: Range<usize>
}

Now, if I produce this AST using a string:
key = Value
key2 = Value 2

how could I pass that string to the serializer so that Identifier and Value can be serialized to &str?

Hi @zbraniecki, sorry that no one was able to provide help here. If this is still an issue, you could try taking this question to any of the resources shown in https://github.com/serde-rs/serde#getting-help.


Bulk query observations and issues

Hi,
I'm using the bulk query to download large amounts of data (I'm testing with 500,000 Salesforce records) to a CSV.  I've noticed that this uses a huge amount of memory (> 4GB) - UNLESS I comment out the on('record') handler.  If I comment out the on('record') handler then the call uses about 500MB of memory.
However, and this is a big however, if I comment out the on('record') handler then the on('end') handler is never triggered.
I don't really need to do anything with the on('record') handler as that would slow the download process, but why isn't the on('end') triggered if the on('record') doesn't exist?
Also, with the on('record') active, node never releases the memory (4GB in my test) that was used.
I had to set the --max_old_space_size=8192 to be able to download these rows.  I am using the .stream().pipe(fsStream) technique.  I thought that streaming didn't use large amounts of memory as each record was simply piped to the output file as data rows were read from Salesforce.  If I don't set the heap size, then I get an ECONNRESET error.  I know other folks have experienced similar issues.  Here is my code:
		salesForceConnection.bulk.pollTimeout = Number.MAX_VALUE;
		salesForceConnection.bulk.pollInterval = Number.MAX_VALUE;
		salesForceConnection.bulk.query(soqlStatement)
                            // Commenting the .on('record') uses very little memory but doesn't trigger the on('end')!!
			.on('record', function () {
			})
			.on('end', () => {
				console.log('end');
				return resolve(true);
			})
			.on('error', function (err) {
				console.error('error ' + err);
				return reject(err);
			})
			.stream().pipe(fsStream)
		;

@Rob-ModernAnalytics Any luck with this? I think I have had similar issues before, will report back if I find a solution.

Henry,
I ended up piping the records through a transform stream and putting my record and end methods in the transform stream instead of the jsforce stream.  Something like this:
		const stream = require('stream');

		// Transform stream...
		var transform = new stream.Transform();
		//transform.writable = true;
		transform.write = function(data) {
			// Send the buffer along to next pipeline item...
			this.push(data);
                            ...
                   }
		transform.end = function() {
			this.push(null);
		};
		salesForceConnection.bulk.query(soqlStatement)
			.stream().pipe(transform).pipe(fsStream)
		;


Bulk-edit form for Status (and Role, etc.?) should have content_types_add and content_types_remove fields

Environment

Nautobot version (Docker tag too if applicable): 1.5.7

Steps to Reproduce

Bulk-edit a group of Statuses with the intent to add a content-type to each status
Select a content-type in the bulk-edit form and submit the form

Expected Behavior
The selected content type to be added to each status
Observed Behavior
The selected content-type is added to each status and all other content-types are removed from these statuses.
Probably the UI pattern we should have here is similar to bulk-editing Tag associations to a group of models, where we have separate "Add Tags" and "Remove Tags" fields in the form to make the user's intent super clear.
There's probably a similar issue with some or all of our other models that have a ManyToManyField with ContentType (Role, CustomField, etc.)

+1

Relates to #1943


[NFC] Mark CrossImport test unsupported without assertions.

The test is relying on output from LLVM_DEBUG, so we won't get any output when NDEBUG is set.

@swift-ci please smoke test and merge

Fixes rdar://problem/59812989


Unable to import relationship between two content models

I have been unable to get a relationship to import the --prepare-json step fails with one of these two messages:
Missing link field for SQLBlogPost in SQLBlogComment in contentful structure JSON file!
OR
undefined method []' for nil:NilClass`
I'm sure I don't have it configured properly. The docs are not really clear about what values to use when defining a relationship. I tried the following, but it didn't work.
"SQLBlogComment": {
	...
	"links": {
		"belongs_to": [
			{
				"relation_to": "SQLBlogPost",
				"foreign_id": "sql_blog_post_id"
			}
		]
	}
}

I made a bit more progress by changing the case of the Blog Post field in the Blog Comment object in the contentful_structure.json file.
The P in Blog Post was lower case instead of upper case. I changed to this:
"Blog Comment": {
	...
	"fields": {
		...
		"Blog Post": {
			"id": "blogPost",
			"type": "Entry",
			"link": "Link"
		}
	}
}

This allowed the --prepare-json step to run, but it only processed 2 records and then quits without an error message. There are over 1200 records it should have processed.

Hey @keithws,
Sorry to hear that problem, we wrote a new Database Importer tool https://github.com/contentful/contentful-database-importer.rb that solves all the issues that this current one had.
Hope this helps,
Cheers


Remove duplicate testset

This PR removes duplicate lines already in the tests above:
https://github.com/rafaqz/ModelParameters.jl/blob/005bebb59a1a11fd3b7719169f01a36d8b2ede85/test/runtests.jl#L10-L16

Codecov Report

Merging #35 (5903f3f) into master (005bebb) will decrease coverage by 7.82%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   72.25%   64.42%   -7.83%
==========================================
  Files           4        4
  Lines         173      149      -24
==========================================
- Hits          125       96      -29
- Misses         48       53       +5

Impacted Files
Coverage Δ

src/param.jl
59.37% <0.00%> (-26.34%)
:arrow_down:

src/tables.jl
45.45% <0.00%> (-18.19%)
:arrow_down:

src/model.jl
67.61% <0.00%> (-1.43%)
:arrow_down:

Continue to review full report at Codecov.

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


Update Azure SQL .NET SDK for encryption protector and server key for…

… updated API.

Description
The Azure SQL DB team has updated the APIs for encryption protector and server key properties to rename "type" to "serverKeyType". This change updates the .NET SDK to account for this.
The corresponding hydra-spec changes can be viewed in pull request https://github.com/Azure/hydra-specs-pr/pull/1371

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as a link to the swagger spec, used to generate the code.
[x] The project.json and AssemblyInfo.cs files have been updated with the new version of the SDK.

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

It looks like you're working at Microsoft (mihymel). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@azuresdkci add to whitelist


Clicking on Copilot Edit Window in Secondary Panel makes Outline Panel disappear

Type: Bug

Have outline view docked in the secondary bar
Click on the new Copilot Edit panel

Expected Result: outline view stays as an option in the secondary bar
Actual Result: outline view is gone from the secondary bar
Repro Video:
https://github.com/user-attachments/assets/58589a79-dfd1-429f-852f-6cf5add29608
Furthermore, the commands to focus on the outline view look weird?

VS Code version: Code 1.95.1 (Universal) (65edc4939843c90c34d61f4ce11704f09d3e5cb6, 2024-10-31T05:14:54.222Z)
OS version: Darwin arm64 24.1.0
Modes:

System Info

Item
Value

CPUs
Apple M1 (8 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
3, 3, 3

Memory (System)
16.00GB (0.07GB free)

Process Argv
--crash-reporter-id ebd6b185-e8ed-432f-8844-835eaab863ed

Screen Reader
no

VM
0%

Extensions (64)

Extension
Author (truncated)
Version

vscode-sql-formatter
adp
1.4.4

project-manager
ale
12.8.0

tsl-problem-matcher
amo
0.6.2

vscode-zipfs
arc
3.0.0

quitcontrol-vscode
art
4.0.0

catppuccin-vsc
Cat
3.16.0

css-theme-completions
con
0.0.5

vscode-eslint
dba
3.0.10

dendron-paste-image
den
1.1.1

nightly
den
0.124.77

gitlens
eam
15.6.3

EditorConfig
Edi
0.16.4

prettier-vscode
esb
11.0.0

copilot
Git
1.245.0

copilot-chat
Git
0.22.2

remotehub
Git
0.64.0

vscode-github-actions
git
0.27.0

vscode-pull-request-github
Git
0.100.1

rest-client
hum
0.25.1

vscode-peacock
joh
4.2.2

mark-sharp
jon
1.6.1

prettify-json
moh
0.0.3

vscode-docker
ms-
1.29.3

vscode-language-pack-qps-ploc
MS-
1.95.2024103009

vscode-language-pack-zh-hans
MS-
1.95.2024103009

vscode-language-pack-zh-hant
MS-
1.95.2024103009

csdevkit
ms-
1.11.14

csharp
ms-
2.50.27

vscode-dotnet-runtime
ms-
2.2.2

vscodeintellicode-csharp
ms-
2.1.11

vscode-edge-devtools
ms-
2.1.6

playwright
ms-
1.1.11

debugpy
ms-
2024.12.0

isort
ms-
2023.10.1

python
ms-
2024.18.1

vscode-pylance
ms-
2024.11.1

datawrangler
ms-
1.12.1

jupyter
ms-
2024.10.0

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.21

vscode-jupyter-cell-tags
ms-
0.1.9

vscode-jupyter-slideshow
ms-
0.1.6

remote-containers
ms-
0.388.0

remote-ssh
ms-
0.115.0

remote-ssh-edit
ms-
0.87.0

remote-wsl
ms-
0.88.5

vscode-remote-extensionpack
ms-
0.26.0

azure-repos
ms-
0.40.0

extension-test-runner
ms-
0.0.12

live-server
ms-
0.4.15

remote-explorer
ms-
0.4.3

remote-repositories
ms-
0.42.0

remote-server
ms-
1.5.2

wordcount
ms-
0.1.0

vsliveshare
ms-
1.0.5941

vsliveshare-pack
ms-
0.4.0

angular-console
nrw
18.29.1

vscode-versionlens
pfl
1.14.2

excalidraw-editor
pom
3.7.4

run-on-save
puc
1.9.0

timestamp-converter
Sta
0.0.1

rewrap
stk
1.16.3

code-spell-checker
str
3.0.1

material-theme
zhu
3.17.6

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyonecf:30548226
962ge761:30959799
9b8hh234:30694863
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
724cj586:31013169
dvdeprecation:31068756
dwnewjupytercf:31046870
newcmakeconfigv2:31071590
impr_priority:31102340
nativerepl2:31139839
refactort:31108082
pythonrstrctxt:31112756
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013
j44ff735:31177056

@jonathanyeung Can you please right click on the empty space next to the two icons and see if there is a "Copilot Edits" along with a another view name (possibly an id as I see those surfacing for you). I expect the "Copilot Edits" to have a check mark and the other to not have a check mark. Clicking the other should bring it back to visible even when you select something else.

Hi @sbatten - I toggled some settings in the right click context menu and now I can't get it to repro anymore... is this just a configuration error on my end?  This is what I see now, but when I click between the 'Copilot Edits' icon and the 'outline' icon, it works as expected and nothing disappears anymore:

Maybe my editor was just in a weird state after the latest upgrade that added the 'Copilots Edit' window?

You now have the ✅ next to Outline which means it will not disappear when you open copilot edits. If you select Outline in the context menu (unchecking it). You will be able to reproduce your issue again, but this is not a bug, just the design of the product.


ARIMA/NAB updates

NAB should now actually work again! Running a final evaluation now to see how well it actually performs. Adding timeColumn to targetLowMetrics seems a little iffy, but fixes ingest for now.

Also changed some of @mamikonyana's code. I don't think I broke anything (unless you were using ScoreDumpDiagnostic somewhere).

Coverage decreased (-0.1%) to 93.288% when pulling 94440a0713b6d82dfc785797249b8af41b9c4dee on arima into 8377e82bc5716e631f3bcc27f4a9430293d9495c on master.

Could you rebase off master?

Hmm was there a force push? I'm traveling for a month and may not have time to look at this, although I will try.
@pbailis did you decide if you're OK with hardcoding timeColumn to be at index 0 of metrics (enforced by the loaders)? Or is there a better way to do it?

Closing for now due to inactivity.


Extract History, Select/Unselect broken

Describe the bug
When extracting the JSON from a project, I should be able to select only part of the operation using the select option. This option does not work at least since OpenRefine 2.8
No error shown in the terminal  (in the log I create a project and apply two operations)
21:56:27.155 [                   refine] POST /command/core/importing-controller (21361ms)
21:56:28.170 [                   refine] POST /command/core/get-importing-job-status (1015ms)
21:56:28.176 [                   refine] POST /command/core/cancel-importing-job (6ms)
21:56:29.270 [                   refine] POST /command/core/load-language (1094ms)
21:56:29.304 [                   refine] GET /command/core/get-preference (34ms)
21:56:29.316 [                   refine] POST /command/core/load-language (12ms)
21:56:29.330 [                   refine] POST /command/core/load-language (14ms)
21:56:29.343 [                   refine] POST /command/core/load-language (13ms)
21:56:29.534 [                   refine] GET /command/core/get-project-metadata (191ms)
21:56:29.563 [                   refine] GET /command/core/get-models (29ms)
21:56:29.780 [                   refine] GET /command/core/get-history (217ms)
21:56:29.781 [                   refine] POST /command/core/get-rows (1ms)
21:56:30.029 [                   refine] GET /command/core/get-history (248ms)
21:56:33.728 [                   refine] POST /command/core/apply-operations (3699ms)
21:56:33.830 [                   refine] GET /command/core/get-history (102ms)
21:56:33.860 [                   refine] GET /command/core/get-project-metadata (30ms)
21:56:33.876 [                   refine] GET /command/core/get-models (16ms)
21:56:33.891 [                   refine] POST /command/core/get-rows (15ms)
21:56:34.032 [                   refine] POST /command/core/compute-facets (141ms)
21:56:35.311 [                   refine] GET /command/core/get-operations (1279ms)
21:58:19.139 [           ProjectManager] Saving some modified projects ... (103828ms)
21:58:19.147 [        project_utilities] Saved project '1898382528686' (8ms)

To Reproduce
Steps to reproduce the behavior:

Create a project
Perform a couple of transforms operation
Go to Undo/Redo and then select Extract
Click to select, unselect some steps
The Json on the right panel should update based on the steps (un)selected. It does not

Screenshots

Desktop:

OS: ubuntu 16.04
Browser Chromium

OpenRefine :

Version OpenRefine2.8 and 3.0RC1

@magdmartin any errors in javascript ?  Can you check the Browser console ?

@magdmartin I can't reproduce this either (with chromium on linux)… maybe a sample project would help indeed?

Closing as not reproducible.


Implement mutation SetRumor and SetAnswer

Implement these:
https://github.com/MrOrz/rumors-api/tree/master/src/graphql/mutations
The expected behavior is written in the comments. They are not hard requirements and can be changed.
Please also add test cases for SetRumor and SetAnswer. Please refer to https://github.com/MrOrz/rumors-api/tree/master/src/graphql/queries on how to structure and write test cases in this project, and see README for running the test.

Implemented in f5ab734, d7c2456,  91434a6


Performance: much slower than serde-json.

I was trying to port the polars ndjson reader over to this crate instead of serde_json, and noticed a huge degradation in performance after the refactor.
On a dataset of 500_000 records. containing data similar to this: (more than happy to share the actual file if needed)
{
  "number": 300,
  "hash": "0xb3e37f7c14742bc54d08163792d38ada69c3951817b8dde6ef96776aa5c0f00c",
  "parent_hash": "0x989b8bf2af0be6c18c9c95bfde81492e0b47bcc1c26d555bb7cea2d09e92c6c3",
  "nonce": "0x424b554fa4a7a04f",
  "sha3_uncles": "0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347",
  "logs_bloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
  "transactions_root": "0x56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421",
  "state_root": "0x34e5b52497408cd2bbcb6992dee0292498a235ec7aca1b34f6cbccb396f85105",
  "receipts_root": "0x56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421",
  "miner": "0xbb7b8287f3f0a933474a79eae42cbca977791171",
  "difficulty":<PHONE_NUMBER>9,
  "total_difficulty":<PHONE_NUMBER>386,
  "size": 544,
  "extra_data": "0x476574682f4c5649562f76312e302e302f6c696e75782f676f312e342e32",
  "gas_limit": 5000,
  "gas_used": 0,
  "timestamp":<PHONE_NUMBER>,
  "transaction_count": 0,
  "base_fee_per_gas": null
}

An unoptimized build using serde_json will load the json file in ~35s on my device.
The unoptimized build using json_serializer loads in ~55s
The code changes are available here just in case I did something wrong on my end in using json_serializer
https://github.com/pola-rs/polars/compare/master...universalmind303:polars:ndjson-perf-enhancements?expand=1.
fn parse_lines<'a>(bytes: &[u8], buffers: &mut PlIndexMap<String, Buffer<'a>>) -> Result<usize> {
    let lines: SplitLines = SplitLines::new(bytes, NEWLINE);

    for mut line in lines {
        let value = json_deserializer::parse(&mut line).map_err(|e| {
            PolarsError::ComputeError(format!("unable to parse line: {}", e).into())
        })?;
        match value {
            Value::Object(value) => {
                buffers
                    .iter_mut()
                    .for_each(|(s, inner)| match value.get(s) {
                        Some(v) => inner.add(v).expect("inner.add(v)"),
                        None => inner.add_null(),
                    });
            }
            _ => {
                buffers.iter_mut().for_each(|(_, inner)| inner.add_null());
            }
        }
    }
    let bytes_read = bytes.len();

    Ok(bytes_read)
}

The parser doesn't expose a way to get the number of bytes read back so the SplitLines does cause some overhead. I copied over the code to call the parse_value function directly, and created an optimized loop that doesnt use the custom SplitLines iterator. I saw some perf gains ~5s, but it is still very far off from the serde implementation.
the more optimized version:
// it's ndjson, so we know it is always should be {obj}\n{obj}\n
fn parse_lines<'a>(
    bytes: &mut &[u8],
    buffers: &mut PlIndexMap<String, Buffer<'a>>,
) -> Result<usize> {
    use json_deserialize::Value;
    let total_bytes = bytes.len();
    let mut read = 0;

    loop {
        if read == total_bytes {
            break;
        }
       // its a newline
        if json_deserialize::current_token(bytes)
            .map_err(|e| PolarsError::ComputeError(format!("EOF error: {}", e).into()))?
            == NEWLINE
        {
            read += 1;
            *bytes = &bytes[1..];
        }
        // its the object
        else {
            let byte_size = bytes.len();
            let value = json_deserialize::parse_value(bytes).map_err(|e| {
                PolarsError::ComputeError(format!("unable to parse line: {}", e).into())
            })?;
            read += byte_size - bytes.len();

            match value {
                Value::Object(value) => {
                    buffers
                        .iter_mut()
                        .for_each(|(s, inner)| match value.get(s) {
                            Some(v) => inner.add(v).expect("inner.add(v)"),
                            None => inner.add_null(),
                        });
                }
                _ => {
                    buffers.iter_mut().for_each(|(_, inner)| inner.add_null());
                }
            };
        }
    }
    Ok(total_bytes)
}

I am unable to reproduce this on a bench - created a bench for it here: https://github.com/jorgecarleitao/json-deserializer/pull/12:
object_complex json_deserializer 2^12
                        time:   [24.209 ms 24.281 ms 24.355 ms]
object_complex serde_json 2^12
                        time:   [29.140 ms 29.216 ms 29.295 ms]
object_complex serde_json_custom 2^12
                        time:   [23.982 ms 24.053 ms 24.130 ms]
object_complex simd_json 2^12
                        time:   [10.095 ms 10.117 ms 10.139 ms]

with that said, maybe we should use simd_json - it is 2x faster 🚀

@jorgecarleitao  would it be within scope to add a bench that includes serde_json::StreamDeserializer? That is what is being used within the current polars implementation.

Of course! - everything counts :)

closing this as it mostly pertains to ndjson parsing, which is not directly supported in this crate yet.


Epic: Bugs for Alpha

[ ] https://github.com/quantified-uncertainty/squiggle/issues/256
[ ] https://github.com/quantified-uncertainty/squiggle/issues/255
[ ] https://github.com/quantified-uncertainty/squiggle/issues/243
[ ] https://github.com/quantified-uncertainty/squiggle/issues/149
[ ] https://github.com/quantified-uncertainty/squiggle/issues/40
[ ] https://github.com/quantified-uncertainty/squiggle/issues/254
[ ] https://github.com/quantified-uncertainty/squiggle/issues/260
[ ] https://github.com/quantified-uncertainty/squiggle/issues/8
[x] https://github.com/quantified-uncertainty/squiggle/issues/16
[x] https://github.com/quantified-uncertainty/squiggle/issues/54
[x] https://github.com/quantified-uncertainty/squiggle/issues/247
[ ] https://github.com/quantified-uncertainty/squiggle/issues/120

It's not allowing me to save this. Hopefully this comment works

[ ] https://github.com/quantified-uncertainty/squiggle/issues/304
[ ] https://github.com/quantified-uncertainty/squiggle/issues/293
[ ] https://github.com/quantified-uncertainty/squiggle/issues/256
[ ] https://github.com/quantified-uncertainty/squiggle/issues/255
[ ] https://github.com/quantified-uncertainty/squiggle/issues/260
[ ] https://github.com/quantified-uncertainty/squiggle/issues/254
[x] https://github.com/quantified-uncertainty/squiggle/issues/247
[ ] https://github.com/quantified-uncertainty/squiggle/issues/243
[ ] https://github.com/quantified-uncertainty/squiggle/issues/149
[x] https://github.com/quantified-uncertainty/squiggle/issues/120
[x] https://github.com/quantified-uncertainty/squiggle/issues/54
[ ] https://github.com/quantified-uncertainty/squiggle/issues/40
[ ] https://github.com/quantified-uncertainty/squiggle/issues/16
[ ] https://github.com/quantified-uncertainty/squiggle/issues/8

No reason to keep this open anymore, Alpha milestone is closed.


Shared context vs application

Currently, the direction of exponentials is to interpret them as applicative expressions. This direction was chosen to make them more different from implication. It has also something to do with the plans I have for unifying lambdas and product/sum types as exponentials. At the moment, the effort put into Hooo only is concerned type level theorem proving (without proofs of transformation). I also need to write down what the difference is between type level theorem proving and theorem proving with proofs.
Application
The rule for application substitution is the following for exponentials (app_subst):
(((A ^ B) ↞ X) → A[B := X])

In order to accommodate this direction, I removed a rule ((A ^ B) -> (A -> B)) in the "hooo" tactic, since it would leak context into the environment.
The difference between an applicative expression and implication is the following:

The context (the exponent) of an applicative expression is local and meant to be substituted in the environment
The assumption of implication is part of the environment

When a variable is bound to some environment, it loses the ability of meaning "for all". Instead, it represents a concrete case.
The context of an applicative expression on the other hand, preserved the meaning "for all" precisely because the variable in the context is local and meant to be substituted.
Shared Context
Now, the issue is whether one should allow shared context or not. A shared context means e.g. A1 ^ B and A2 ^ B shares B as context. They are treated by the solver as part of the same set of facts that are valid in the context B. Instead of being treated as separate, "isolated singleverses", they are treated as "portals into parallel universes" where one can reason about what-if scenarios given introducing of some new, unknown context.
Shared context works, because the context is not leaked up, but sideways.

One problem is when (a^b)^b, the b is referred to in a as the second argument. However, when swapping the two arguments, the references in a still point to the second argument, while for (a^b)^c it becomes (a^c)^b which now points to the first argument.

Due to problems like swapping, I am becoming more skeptical toward using application. It might be that ∏(a^b) or ∏ b { a } could use application, but not a^b. The reason I am skeptical is because there are good indications that b in a^b should be a concrete known type. Thus, it is treated as part of the environment.


The custom role accessibility is not working as expected.

Using swa cli version 1.0.6
Describe the bug
The custom role accessibility is not working as expected.
I run swa cli from my local machine and publish to a static webabb.
I have login to azure using Service principle and The command I use is,
swa deploy --verbose silly --app-location [appLocation] --deployment-token [deploymentToken] --env [envName] --api-location [apilocation] --swa-config-location [configFileLocation]
The custom role access restriction is not working. The restricted page is accessible to all.
Expected behavior
The restricted page is not accessible to all except to the respective members of active directory group.

There is not enough information to diagnose the problem here.
What does your staticwebapp.config.json look like?
How is your Entra ID enterprise app set up?
How is your role function set up (if you have one)?
Do you have a repo we can look at?


How to use the legacy react plugins?

Your question.

How do I configure key and register the plugin?

I am importing ChordChartPlugin and configuring it as shown below. Then I am using the plugin with SuperChart as mentioned in the README of the plugin here.
import ChordChartPlugin from '@superset-ui/legacy-plugin-chart-chord';

new ChordChartPlugin()
            .configure({ key: 'chord' })
            .register();

But I am getting the below error in the browser and I am not able to see any chart. Please help me. Am I missing something?
ERROR chartType="chord" —Error: Item with key "chord" is not registered.

Your code looks right. There is a known issue with multiple copies of @superset-ui/chart. Make sure your node_modules directory has only one @superset-ui/chart.

Try to remove yarn.lock or package-lock.json and yarn install or npm install again.
If 1) does not work, try deleting node_modules directory and try yarn install or npm install again.

@kristw
I tried both the steps you mentioned and installed the dependencies again, but still I am getting the same error. Also, I have only one version of @superset-ui/chart in node_modules.

Please find below the code where I am using superset plugins. Please let me know if I am missing something. Thanks in advance.
import React, { Component } from 'react';
import './App.css';
import { SuperChart } from '../node_modules/@superset-ui/chart/lib';
import ChordChartPlugin from '@superset-ui/legacy-plugin-chart-chord';

new ChordChartPlugin()
    .configure({ key: 'chord' })
    .register();

class App extends Component {
  render() {
    // new ChordChartPlugin()
    //   .configure({ key: 'chord' })
    //   .register();
    return (

 Sample chart

            <SuperChart
            chartProps={{
              formData: {
                colorScheme: 'd3Category10',
                yAxisFormat: '.2f'
              },
              height: 400,
              payload: {
                data: {
                  matrix: [
                    [ 381, 16, 14, 5, 4, 6 ],
                    [ 29, 535, 132, 20, 21, 16 ],
                    [ 22, 18, 1, 3, 0, 214 ],
                    [ 3, 4, 462, 152, 132, 0 ],
                    [ 16, 32, 299, 123, 146, 1 ],
                    [ 22, 83, 53, 21, 22, 6 ]
                  ], nodes: [
                    'Hong Kong', 'Tokyo', 'Taipei', 'Beijing', 'Bangkok', 'Jakarta', 'Singapore'
                  ]
                }
              },
              width: 400
            }}
            chartType="chord"
            className=""
            id=""
            onRenderFailure={function(){}}
            onRenderSuccess={function(){}}
            overrideTransformProps={undefined}
            postTransformProps={function(a){return a}}
            preTransformProps={function(a){return a}}
          />

    );
  }
}

export default App;

import { SuperChart } from '@superset-ui/chart';

Thanks! It's working now.

@madhu-aithal which file did you add the key and the Superchart code?


[stable/nginx-ingress] cluster-wide rbac resources are created even when controller.scope.enabled is set to true

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT

Version of Helm and Kubernetes:
Which chart: stable/nginx-ingress
What happened: RBAC resources are created at a cluster level (ClusterRole and ClusterRoleBinding) even when controller.scope.enabled is true. Role and RoleBinding should be created instead containing all the RBAC rules required by the chart to function.

Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT
Version of Helm and Kubernetes:
Which chart: stable/nginx-ingress
What happened: RBAC resources are created at a cluster level (ClusterRole and ClusterRoleBinding) even when controller.scope.enabled is true. Role and RoleBinding should be created instead containing all the RBAC rules required by the chart to function.


Use appropriate log level for exceptions from event listeners

From @SebastiaanZ:

Finally, discord.py currently "hides" errors/tracebacks that happen in event listeners as we only have a custom error handler for commands. This isn't too bad locally, since d.py prints those exceptions to stderr, but it obviously means they'll never show up in Sentry, as they are not actually logged with the appropriate level.

Right, thanks. I should have added this myself.


Not working in xcode 9

Not working in xcode 9

@sahabe1
We're already using SwifterSwift in Xcode 9, installation guide.
However if it didn't work for you, please specify more details to let us assist you:

what exactly is not working?
what version of SwifterSwift are you using?
what method are you using to integrate SwifterSwift into your project?
Attach an example if possible.

please see attachment

@sahabe1 It looks like you are still using v3 please update to v4.
You can update your pods by typing
pod update

I'm using Xcode Version 9.0 (9A235) with SwifterSwift version 4.0.1
Here's the list of errors Xcode throws:

@agfa555 What version of Swift are you using? You need Swift 4 for SwifterSwift 4.0

I had forgotten about that minor little detail. 👯♂
All solved now.
Sorrrry...

Hi, @agfa555
I have the same problem
How do you solve it?


Decrease

✏ Edit curr abbr
Which abbrs do you want to modify?
-> dec
What do you want to change?
Find another one with {} without spaces

[x] I can handle the following pr related to this issue to edit the abbrs.

Thanks, fixed.
I've checked that with regex that there are no more issues like this


Set XDG_CACHE_HOME to /tmp

Running builds as in CI is failing with failed to initialize build cache at /go/.cache: open /go/.cache/log.txt: permission denied

CI doesn't use Dockerfile.src from PRs, so I'm merging to see if that fixes it.


importccl: add deprecation warning to IMPORT TABLE

PR: https://github.com/cockroachdb/cockroach/pull/67275
From release notes:

IMPORT TABLE will be deprecated in v21.2 and removed in a future release. Users should create a table using CREATE TABLE and then IMPORT INTO the newly created table. [#67275][#67275] {% comment %}doc{% endcomment %}

A deprecation warning for this has already been added to 21.2 and 21.1 re the deprecation. Feel free to reopen if further docs are needed


E712 "true-false-comparison" incorrectly raises on array like

Consider
import pandas as pd
s = pd.Series([True, True, False, False, None, None])
filter = s != False # [True, True, False, False, True, True]
notAFilter = s is not False # True

or similar operations on array-likes. Here, E712 is very much not equivalent, and in fact, wrong.
This may not be tractable as it requires type analysis.

Thanks! I think this is the same as this open issue: https://github.com/astral-sh/ruff/issues/4560.

(Closing to merge / consolidate into #4560.)


Handling multiple values of context variables

Hello Everyone,
I am developing a pizza ordering chatbot. I have a context variable: 'pizza_type' which takes the value of type of pizza the user wants to order. It works fine if the user only orders 1 type of pizza. If the user works 2 or more pizzas, the chatbot only considers 1st pizza type and ignores rest of the entries. How do I solve this problem?

No definite answer for you, but it would probably take some changes to the context passed between the app and Assistant, and also to the Assistant dialog to ask user if they want to order multiple pizzas. The context could manage which pizza (in the array of pizzas) is currently being ordered.
Please share your solution if you decide to pursue this.

Thank you @rhagarty . Is there a method to run something like a "for loop" in assitant dialog? For example if user order salami pizza with cheese topping and vegetarian pizza with olives topping. Then currently, chatbot responds that user has ordered ONLY salami pizza with cheese and olive toppings.  I cannot find a way to assign correct toppings to correct pizza and store the data for further usage.

@kraken24 There is a "jump to" option when creating your Assistant dialog nodes. You could build a loop around ordering a single pizza. Then ask "Would you like to order another pizza?", and jump back to the top node and do it again.

@rhagarty it worked. Thank you. I will try to improve the dialog flow structure more


Whitney font on openvault.wgbh.org

Make sure that the font referrer check is turned off for openvault.wgbh.org

Email sent to Tim Kinnel.. the typography.com gatekeeper.

I'm not on the email chain, but when I changed my /etc/hosts so that my local server would spoof that name, fonts worked, so I believe they'll work on the real thing. Closing.

Yay. Did not receive any confirm in my inbox (will double check), but I guess it has happened.


bug(select): Search is not filtering results in multi-select

Bug description:
When searching options in sui-multi-select, the options are not filtered at all.
Only part that seems to behave correctly is if the search param doesn't match any options, it will still display 'No results'.
Please let me know if I'm doing something wrong.
Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/edit/fbrOoP?p=preview
Version of Angular, ng2-semantic-ui, and Semantic UI:
ng2-semantic-ui: 0.8.4

See my answer to #148


Add run on repl.it badge to README

This pull request configures this repository to be run on Repl.it.      It adds a .replit configuration file and a Repl.it badge to the README.
You can read more about running repos on Repl.it here, or view the Repl here.

Thanks, I'll give it a shot!


Add CI run on Microsoft / MacOS

This PR includes installation / test / type analysis on both Windows and MacOS. Because these are more costly than Linux (resp. x2 and x10), I tried to make the workflow run only when everything else succeeded.
Closes #311

As rightly noted by @KabirDubey in a mail, this PR does not include the change proposed in #311 but still succeeds to install the tqec library. That means that we are likely not able to catch everything, and other errors like in #311 will eventually appear. These are errors due to misconfigured user spaces (be it the fault of the OS or not), and it is not our responsibility to be sure that a misconfigured system is still able to install the library. Nevertheless, I will review the installation process in another PR.

The workflow_run only runs when the file defining the jobs triggered by the workflow_run event are on the default (main for us) branch. So I am merging this PR in the hope that the workflow_run will correctly trigger.


[ISSUE #4847] HTTPS/(m)TLSv1.3 support for HTTP Admin Server

Fixes #4847
Motivation
Admin endpoints may need secure transmission.
Modifications

Refactor AdminServer to have its own independent configuration class.
Start AdminServer even if all three protocols are not enabled.
Fix NPE caused by AbstractRemotingServer shutdown() method calling the wrong method.
Add TLS support for AdminServer (Document: https://github.com/apache/eventmesh-site/pull/214/files)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

Please fix the pr code conflict problem, and by the way, why do you need to mention pr that supports tls v1.3? jdk8 should not support it, and it needs jdk11 to support it, right? @Pil0tXia

@xwm1992
I've added a "JDK/JRE 8 Compatibility" chapter in https://github.com/apache/eventmesh-site/pull/214/commits/5ccbf9be142fd0c22f5068b466f4e165140a7a33.

@pchengma Great

@xwm1992 PTAL~


testnet network support - g9qjfxwljdyyyd6 - validator,rpc,api

testnet network support - g9qjfxwljdyyyd6- rpc,api

@vkraig, Your Moniker and validator addresses are required, please update the PR with those details.

@vkraig, Your Moniker and validator addresses are required, please update the PR with those details.


Add new heroku mirror

@alexlincoln Nice!
Are you running this mirror? If you sign up for Twilio, you can add your credentials there and the "Could not get WebRTC config from server. Using default (without TURN)." warning will go away.
TURN helps peers connect when they're unable to do so because both of them are behind super restrictive firewalls. It's not essential, but it's nice to have. It will cost you only a few cents per month.
Alternatively, I could change the code so that mirrors like this one will use https://instant.io/rtcConfig if no local Twilio credentials are set.

Yeah I'm running it right now, I can create a follow up PR with the ability for anyone to host a mirror from heroku if you'd like. As far as Twilio is concerned, I didn't get a chance to look at their pricing when setting it up yesterday so if it's just a few cents a month I'd be happy to contribute.

Just a heads up I found http://numb.viagenie.ca/, they offer free TURN access

Twilio just costs a few cents a month:

I'll update the codebase to fallback to using https://instant.io/rtcConfig when the local /rtcConfig returns nothing, which happens when the twilio credentials haven't been set up.

I can create a follow up PR with the ability for anyone to host a mirror from heroku if you'd like

This sounds awesome. Can you do that!

@alexlincoln Okay, I just updated the client code so it tries /rtcConfig followed by https://instant.io/rtcConfig. Mirrors should get TURN automatically now, without Twilio credentials. Though, obviously, the mirror will be more resilient with it's own Twilio credentials.

@alexlincoln Also, your mirror seems down to me: https://instant-io.herokuapp.com/

@feross yeah that'll happen when there's no traffic for a period of time, it takes a few seconds for it to spin up but it is up. See https://devcenter.heroku.com/articles/dyno-sleeping.

@feross Just signed up and purchased credit, it is now live

@alexlincoln Nice - looks like it's working!

I might register to twilio and all later, but anyway for now I just updated my instant.io instance so it uses https://instant.io/rtcConfig , seems to work, no error message anymore !

@rom1504 Nice! I was hoping you'd do that!


Google cloud policy

Codecov Report

Merging #37 (fd84aa7) into main (9142267) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   91.12%   91.12%
=======================================
  Files          24       24
  Lines         755      755
=======================================
  Hits          688      688
  Misses         45       45
  Partials       22       22

Continue to review full report at Codecov.

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

Fixes #18


[Docs] Blockwise example

Including an example of blockwise for the client and server or a brief explanation on how to use it in the README would be beneficial to get started, since it is a feature that will be used a lot in constrained networks.

Hello,
Does anyone have a simple example how to use blockwise transfer on the server side? I have a RTOS based client that is able to communicate with the server using normal CoAP packets, but I struggle to understand how the server handle function can process a blockwise transfer request and send the appropriate amount of data.
Thanks!

Does someone have an example for block-wise?


Simple self-contained correct example missing

Unfortunately I cannot get this review rule working. It always results in "I found no errors!". Here's my example code:
Main.elm
module Main exposing (Model, main)

import Browser
import Html exposing (Html, div, text)

type alias Model =
    { x : Int
    }

main : Program () Model Msg
main =
    Browser.sandbox { init = { x = 0 }, update = update, view = view }

type Msg
    = Increment
    | Decrement

update : Msg -> Model -> Model
update msg model =
    case msg of
        Increment ->
            { model | x = model.x + 1 }

        Decrement ->
            { model | x = model.x - 1 }

view : Model -> Html Msg
view model =
    div []
        [ text <| String.fromInt model.x
        ]

review/src/ReviewConfig.elm
module ReviewConfig exposing (config)

import RecordFieldHelper.GenerateUsed
import Review.Rule as Rule exposing (Rule)

config : List Rule
config =
    [ RecordFieldHelper.GenerateUsed.rule
        { generator = RecordFieldHelper.GenerateUsed.accessors
        , generateIn = ( "Lense", [] )
        }
    ]

Then I run npx elm-review --debug:
I found no errors!

I had expected that something gets generated in Lense.elm.
I also tried to create Lense.elm as an empty module beforehand, but didn't work either.
Could someone provide a simple self-contained correct example?

Whenever you add Lense.yourField somewhere in your code, the lens should automatically be generated.

Ah, alright! Got it. Thanks for the quick response! However, a SSCCE would be a helpful addition to the package's docs.


Please update the README?

This seems like it would be really cool to use but unfortunately the example code in the readme is no longer valid, and I've been struggling for a long time to get this thing to work. It seems "NLITopicClassifier" now requires an additional argument to specify which pretrained model to use, but not all of them work.
Provide further explanation or documentation?

For example, here is the stack trace when I try to run NLITopicClassifier using 'gpt2' as the first argument of its contructor in exactly the usage provided in the README:

Hi! You are right, I should give more information about which are the pretrained models expected to be used with the NLITopicClassifier class. I will push a hotfix asap with the indications.
But for now, the only models that are expected to work are the ones that are already fine-tuned on the MNLI task. For instance you can try the following ones: "roberta-large-mnli", "facebook/bart-large-mnli", "microsoft/deberta-v2-xlarge-mnli", "joeddav/xlm-roberta-large-xnli", etc ... Remember that to use some of them you will need to update your Transformers version.

I tried with microsoft/deberta-v2-xlarge-mnli and so far its working well. Outside of lack of documentation it is a very cool project 👍.


remove forced height and hide horizontal scrollbars

Fixes issue #55 by using the inherited height and just remove the horizontal scroll.

Not needed; update in Jenkins 1.598 fixed this particular issue


When entity searched, added and chosen as another classification entity, the original chosen is not among chosen in the next run

From @brodecva on November 16, 2016 16:8
The feedback sent by the client seems to be OK, so it must be the algorithms (meaning the constraints processing) fault.
True, I have set a resource (Hugo Award for Best Novel@en (http://dbpedia.org/resource/Hugo_Award_for_Best_Novel)) as the next candidate, and the original classification Item (http://www.wikidata.org/ontology#Item) is a class. Could it affect that?
Copied from original issue: odalic/odalic-ui#133

@KataBoku  also reports, that even the candidates are somehow sometimes messed up.
@josef-janousek You told me on Skype, that the algorithm is unable to return more than one class as the chosen. This makes sense, unfortunately it effectively prohibits us from sending more that one chosen classification per column as a feedback. It also hard to do it in such way, that we would pick one of the chosen classification as the primary one, because there are no clear criteria how to choose one. Ah, I am kinda mad that I did not expect this.
Assigning to Tomas, I will explain the situation to him.

S tím ale netuším, jak si poradit zatím, proto jsem přiřadil Tomášovi k vyjádření. V zásadě jde o to, že TableMiner+ sám nevrací více než jednu vybranou klasifikaci na sloupec, a tím pádem není jen tak možné mu nakázat feedbackem, aby jich vzal více.
Nechápu, jak jsem si to nemohl všimnout dříve.

As discussed, proposed solution: Just please document in the wiki - in the https://grips.semantic-web.at/display/ADEQ/Odalic+Core subtree - that there is such a limitation of the algorithm.
For now, we will go only for one classification per KB.

@tomas-knap Closing. The documentation is dealt with in #222 .


Midje should work with lazytest

It'd be useful to have lazytest's reporting and Midje's checkers in the same tests. Currently I can't get lazytest to run with Midje checkers:
#<CompilerException java.lang.Exception: Unable to resolve symbol: print-doc in this context, compiling:(clojure/contrib/ns_utils.clj:74)>

This is with SNAPSHOT versions of clojure, clojure-contrib, Midje and lazytest.

You can use lazytest with lein-midje now like this: lein midje --lazytest