Quota master

This pull request is for quota service implementation, divided into following two commits:

Quota server side implementation
a. Quota Framework plugin containing Daos
b. Quota database plugin for quota APIs
c. Quota Usage server enhancements

Quota UI plugin

one big chunk of work Abhi, I have skimmed it and the structure looks good. I must have missed some minor issues in these 7000+ lines of code, though.
One major issue is that there is no single test in there, nor a test instruction/- description. I am sure I will break it if I poke at the other end of the system a bit.

@DaanHoogland I think the aim here is to bring the work out for everyone to review early on, Abhi tells me he's already working on unit and marvin tests.

@bhaisaab well it looks promising.

@DaanHoogland @kishankavala @jburwell The documentation to run and check the plugin are in this pull request: https://github.com/apache/cloudstack-docs-admin/pull/30

I saw @jburwell his remarks coming by. I trust him to be thorough. I will review when I see unit and integration tests.
try using more try with resource when possible. No beef just
try (Transaction trx =TransactionLegacy.open(TransactionLegacy.USAGE_DB)) {} is an often used pattern. maybe create a method/utility called switchDb(...)

Where ever there is a DB change the pattern is use. The plain switch is used where it is db readonly.

@DaanHoogland in addition to failing to convey the intention of the action, the open then close pattern is incredibly expensive due to rapid, unnecessary churn on the connection pool.  Please see my response above for my thoughts on how we can improve it and avoid the nasty side effects the TransactionLegacy imposes.

@jburwell  @DaanHoogland There is a underlying connection pool to save on expensive db connection operations. Moreover this cost should be similar to opening and closing transactions without switching DB.
I think we should divide the DB transaction issue into two:

The expense of the operation and how it can be avoided. My initial thoughts are that this is trivial and non issue.
The way a DB switch is affected, is there a better pattern ?

@abhi An connection pool makes acquiring a scarce resource less expensive, but it is not cost-free.  By taking a connection when the pool is under load, the following could occur:

This code may request a connection when none are available causing the connection pool to create another ad hoc connection or blocking the thread until a connection becomes available -- dependent on the pool configuration.
This code may exhaust the last connection in the pool causing other thread(s) to create ad hoc connection(s) or to block waiting for a connection to become available -- dependent on the pool configuration.

There are variety of patterns that can be employed to address this problem.   I lean towards a construct such as the following:
Transaction transaction = new Transaction(USAGE_DB).attempt(new TransactionWork() {

      public void execute(final Connection connection) throws SQLException{

             // Do database work ...
      }

}

The Transaction.attempt method wraps the call to the passed TransactionWork.execute method with the following boilerplate that appears throughout the code base:

Capturing the current database ID from TransactionLegacy
Open a connection to the requested database
Handling exceptions including translation and proper logging
Aborting/commiting/closing the transaction via TransactionLegacy as needed
If necessary, switching TransactionLegacy back to the previous database ID

These steps assume that TransactionLegacy has a switch method that changes the ThreadLocal tracking the current database ID without consuming a connection from the pool.
I prefer this approach because it allows DAO to declare transactional blocks in a side-effect free manner.  This form of code should be immune to a replacement of TransationLegacy because it only relies on the JDBC API and makes no assumptions about the call flow.

I think we should not be using TransactionLegacy anymore. It was used in first place just to be consistent with UsageServer and due to time constraints. I think if we should revamp this by using the new translation framework and do it like:
Transaction.execute(new TransactionCallback() {
@Override
public Object doInTransaction(TransactionStatus status) {
// ... do something ...
return myresult;
}
});
This is described here by Darren https://cwiki.apache.org/confluence/display/CLOUDSTACK/Database+Transactions.
I propose that we revamp the Quota with the new Transaction model and as well as the usage server as just the one without the other is still half work. Will work on that.

@jburwell was looking at com.cloud.utils.db.Transaction class the db is hardcoded in there:
execute()....
short databaseId = TransactionLegacy.CLOUD_DB;
I think whole revamping of transactions is a separate sub-project that we should work on and should not be clubbed with Quota changes.

The Transaction open and close causing churn in database connection is based on the notion that these methods somehow open and close db connections.
As far as I see from code they do not deal in DB connection and are only used to initialise the database structures when a DB switch needs to be undertaken or a new transaction context needs to be created.
The suggestion to change this model and the pros and cons should be discussed on mailing list and is clearly separate work than the Usage/Quota Service.

Changes included from the PR, branch rebased from latest master.
Please help review - @karuturi @jburwell @DaanHoogland @remibergsma @kishankavala and others

I will re-review later today (31 August 2015).

@bhaisaab lot of code :) .. I will review tomorrow.
Can you review 755 if possible? ;)

7.5kloc would justify 15klotc (kilo-lines of test code) I am not very worried about the quality of this code or the zeal of some bluely shaped programmers to maintain it but it will have to be maintainable by others! over 10 lines of code is an hazard to review if there is not twice as much test-code there, give or take.

New PR here: https://github.com/apache/cloudstack/pull/768


Special value definitions to drive visualization

core:r, core:g, core:b, core:a, core:radius, core:hide, core:stroke:width, core:stroke:r, core:stroke:g, core:stroke:b, border radius, etc.

How to do color, stroke width for edge types? Maybe value definitions that are based off of only self value types (properties on edge)

[ ] Have a shared library of [core] that is force-added to each imports statement

76306c366c2a28fdfa7435854e59fc1a9c9bdd51 should have also referenced this issue

Edge rendering is somewhat complex.
Currently we render an edge of fixed width and color if there is any edge connection from a result to a parent. (Skipping the 'implied' edges that go right to root).
Ideally we'd be able to decide whether to bundle all edges together into one of all types, or to split out with one edge per type, or some edge types having each individual edge rendered separately. Remember that this is only for edges from one parent to one result; if they have a different parent/result pair they are always bundled differently.
Ideally each property would have a three state for distinct: 'all', 'type', 'none', where 'all' is each and every edge renedered separately even ones from the same type, 'type' is 'render all of the edges of this type distinct  from all other types'.
Note that because edges of the same type to the same result, but with different parents, will not render together, the case of "multiple edges of the same type from the same parent to same result" is rare (it will get less rare when nodes can be combined into meta-nodes.). So it's sufficient for "should this type's edges show up in the default bundle of all edges or be distinct?" which is just a single boolean.
Combining edges of different types will be very useful once meta nodes exist'
The MapAdjacencyNode is responsible for calculating and keeping track of the rendering of edges. It reports an ExpandedEdgeValue for each parent, and the width, color, and opacity.
There is a set of ValueDefinitions for rendering edges. The MapDefinition has a display property that can define distinct, color, width defaults (if not provided they are replaced with deafult values). Each PropertyName can also define its own display section and override any of the ValueDefinitions.
After the values for a node are calculated, the engine calculates the distinct/color/width value definitions. Because they are all run after the main value calculation, they don't need to define any dependencies and may rely on any proeprty on result.
We compute each edge as a bundle. There is a default bundle and a bundle per edge. For each propertydefinition, first we calculate the display:distinct value. If it is truthy, the edges of this type go into their type's bundle. If not, they go into the default bundle. Then we calculate each bundle, calculating the color, opacity, and width for each. Note that this means that multiple types might go into the same bundle (how?).
Maybe for the default bundle there is a ValueDefinition for how to combine all of the various values into one?
(TODO: why is core:radius and core:opacity any different?)
PropertyDefinitions that set hide will not render edges (unless edges(true)) and their value calculations will be shortcut

Use cases:

All edges of any type merge together into a gray
All edges of any type merge together into blue if no engineering and red if an engineering is present
All edges merge together into the blended color they would have been if any edge had been present individually
Edges of all types bump outward to not overlap with one another, rendering separately
Edges of a given type bump separately out compared to other edges, all other edge types are together and operate according to any of the things above.

You can't use the same machinery for radius and opacity of node, because there is no way to override radius/opacity for a single node without introducing a property to rely on anyway.

How do you combine the values from multiple different edge types? They might have different constants they rely on.

Maybe there's a value definition type that is provided an input.  That basically pipes in ValueDefinitionCalculationArgs.input, and that value is [null] if in a context with no input, and otherwise the various numbers.

Actually, you could have radius, opacity be done in the same display system, because we're going to have override values anyway for #5 soon, and those need a mechanism of action anyway.

Nodes have a property called edges (currently called edges) that has named property edges. Ref is always required on an edge, although '' is legal
Nodes also have values, which is a NodeValues that partially sets values as the baseline value for the node.
There is a nodeDisplay block. This display block defines some subset of radius, opacity, color'  each of which is a ValueDefinition. A nodeDisplay block can be in MapDefinition (within a display block, like this: map.display.node, or on a specific node, as display`.
Similarly, there is a edgeDisplay block. This display block defines some subset of width, color, opacity, distinct.  An edgeDisplay block can be in MapDefinition (within a display block, like this: map.display.edge). It can also appear in a PropertyDefinition, as display. These ValueDefinitions will be fed the edges of their type. There is also a similarly named edgeDisplay block that may exist in map.display.edgeCombiner, but nowhere else, and omits the 'distinct' property.  There is also map.display.edgeTypeCombiner.  Those will be fed, via ValueDefinitionInput, the outputs of all of the different edge types being bundled together.
The ValueDefinitions in these display blocks are special in that they are outside of the value calculation pipeline; no other ValueDefinitions can rely on their output, which means they don't have to define any dependencies and can automatically rely on all properties in result.
Node calculation is straightforward. If the values for the various properties are 'derived' from other properties, then they are typically defined once, in MapDefinition, and not overriden. If they are constants, then they might be set as constants, that are set once in the root and then redefined at each node that wants a non-default.
Edges are more complex. For each output property, we do the following.

Each node reports its renderEdges, which is all of the edges that have it as the result node.  For each node, go through each node that is reffed as parent explicitly and then do:
Have the output edge set, which contains the final lines to render. Each one has a color, width, and opacity.
For each edge, run the various combinations, passing in only that edge as an edge ref. If its distinct value

[ ] refValue, rootValue, resultValue, and edgeConstant should all return null if undefined


Remove github.com/satori/go.uuid.

What does this PR do?

remove github.com/satori/go.uuid as direct dependency.
update sprig to remove transitive link to github.com/satori/go.uuid.

Motivation
Fixes #4717
More

[ ] ~Added/updated tests~
[ ] ~Added/updated documentation~

Additional Notes
Now Traefik don't use github.com/satori/go.uuid but the package is always a transitive dependency of:

https://github.com/exoscale/egoscale/blob/master/uuid.go

ping @greut @mcorbin

Thanks you for the notification @ldez, I will remove the github.com/satori/go.uuid package from egoscale.


Replies lose relation to original message on being edited (Matrix <-> Discord)

https://github.com/discord/discord-api-docs/issues/2251

https://github.com/discord/discord-api-docs/issues/2251

Messages seem to lose relation on Matrix too, but only on Element Android
Gomuks, element web, fluffychat work fine

Messages seem to lose relation on Matrix too, but only on Element Android
Gomuks, element web, fluffychat work fine

Does the reply feature actually reflect proper Discord replies? (and not a ping block of text?)

There's currently no way to send "proper" discord replies for webhooks, it's not allowed by the api. So it's sent in the plaintext format that you mentioned.
> <@Ping> content ...

Reply content

It could be made visually better by using embeds, but i just haven't gotten to it yet.


Multiple optional function arguments

declare fn(str?: string, num?: number): void

Test
fn(5);

Actual Result
fn(5);
   ^ number. This type is incompatible with
fn(str?: string, num?: number): void;
         ^^^^^^ string.

Expected Result
I assume it should not error for the following uses:
fn();
fn('str');
fn(5);
fn('str', 5);

I think it works as expected. You need to add an overload for this to work

Shouldn't this be documented somewhere though?

@vkurchatkin is right on both counts, this is working as designed and you might be looking for function overloads. You might also be looking for union types, i.e., function f(arg: number | string) { ... }.
Contributions to the docs are very welcome!


Add support for the Unitymedia Horizon HD Recorder

Description:
A new platform for the Unitymedia Horizon HD Recorder sold by Unitymedia in Germany. The channel list is hardcoded currently but I already reached out to Unitymedia to get a parseable list. As this list doesn't change often, I think that's not a problem. Also working on an async version of this but as this involves the underlying library (einder), this takes some time.
Related issue (if applicable): -
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#5303
Example entry for configuration.yaml (if applicable):
media_player:
  - platform: horizon
    host: <IP_ADDRESS>

Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

I will look at this in the next 1-2 weekends :) Thanks for the comments/tips!

@syssi & @MartinHjelmare thanks for your reviews, all mentioned things should be fixed now.

thanks for your reviews and your help @MartinHjelmare! I fixed all three mentioned things but...

Just one old comment left to fix. Looks good otherwise.

Can be merged when build passes.


No indication that the extension is running, error status

I've just installed the latest version from extensions.gnome.org, but it does not show up in the panel, and shows an error status.   That property was defined with 'let' or 'const' inside the module. This was previously supported, but is not correct according to the ES6 standard. Any symbols to be exported from a module must be defined with 'var'. The property access will work as previously for the time being, but please fix your code anyway.
Mar  9 10:46:12 xps gnome-shell[10256]: super Dbus:, true, <interface name="com.super.productivity.service">#012    <method name="markAsDone">#012    </method>#012    <method name="startTask">#012    </method>#012    <method name="pauseTask">#012    </method>#012    <method name="quitApp">#012    </method>#012    <method name="showApp">#012    </method>#012    <signal name='taskChanged'>#012        <arg type='s'/>#012        <arg type='s'/>#012    </signal>#012</interface>
Mar  9 10:46:12 xps gnome-shell[10256]: Some code accessed the property 'SuperProductivityIface' on the module 'dbus'. That property was defined with 'let' or 'const' inside the module. This was previously supported, but is not correct according to the ES6 standard. Any symbols to be exported from a module must be defined with 'var'. The property access will work as previously for the time being, but please fix your code anyway.
Mar  9 10:46:12 xps gnome-software[3078]: no app for changed<EMAIL_ADDRESS>Mar  9 10:46:12 xps gnome-software[3078]: no app for changed<EMAIL_ADDRESS>Mar  9 10:46:12 xps gnome-shell[10256]: super, DIS _ CONNECTED
Mar  9 10:47:22 xps gjs[6579]: g_object_unref: assertion 'G_IS_OBJECT (object)' failed
Mar  9 10:47:26 xps gnome-shell[10256]: Some code accessed the property 'Util' on the module 'main'. That property was defined with 'let' or 'const' inside the module. This was previously supported, but is not correct according to the ES6 standard. Any symbols to be exported from a module must be defined with 'var'. The property access will work as previously for the time being, but please fix your code anyway.
  Thanks!
Edit: I suspect this is related to the snap.  Looks like it needs to have a slot connected to the dbus interface.  https://forum.snapcraft.io/t/how-do-i-connect-a-snap-to-dbus/1533/2

Thanks for reporting. I'll look into it, as soon as I can.

Ok I think I found the issue. On the extensions page there isn't the latest version of the extension and the old one is not compatible any more as it seems. Could you try installing the extension directly from git?


Running ´npx prisma generate` in new Npm package/project leads to unhelpful output

C:\Users\Jan\Documents\throwaway>mkdir cliHelp

C:\Users\Jan\Documents\throwaway>cd cliHelp

C:\Users\Jan\Documents\throwaway\cliHelp>npm init -y
Wrote to C:\Users\Jan\Documents\throwaway\cliHelp\package.json:

{
  "name": "clihelp",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC"
}

C:\Users\Jan\Documents\throwaway\cliHelp>npm install prisma

added 2 packages, and audited 3 packages in 11s

C:\Users\Jan\Documents\throwaway\cliHelp>npx prisma generate
prisma:warn The postinstall script automatically ran `prisma generate` and did not find your `prisma/schema.prisma`.
If you have a Prisma schema file in a custom path, you will need to run
`prisma generate --schema=./path/to/your/schema.prisma` to generate Prisma Client.
If you do not have a Prisma schema file yet, you can ignore this message.

Current output:


React: add missing components

[ ] Navigation
[x] Local Navigation
[ ] Breadcrumb
[ ] Card
[ ] CardGroup
[x] Footer
[ ] Tabs

[ ] Navigation
[ ] Local Navigation
[ ] Breadcrumb
[ ] CardGroup
[ ] Footer
[ ] Tabs

@inesdgomes once you've added new categories, please specify where these components should go.

In Local Navigation, remove arrow from Main ILO Website Link. This needs to be done in twig as well

@wc-lei feel free to keep pushing up to the branch so we can review things bit by bit for now.

@wc-lei how about after the footer and localNav, we do the Cards next? The main navigation is really only for the main website which I don't see getting implemented in React any time soon.

hey @Shashika6, we can close this by merging #600. However, @bashlk had some feedback about the implementation that I think is right, we should adjust it as per his comments.

Noted @justintemps


Move Cluster tests to t.Run

refactor(cluster resource tests): Migrate the Cluster resource tests to use the t.run approach
PR checklist:

[ ] docs/ updated
[x] tests added
[ ] an example added to examples/ (useful to demo new field/resource)
[x] CHANGELOG.md updated with pending release information

@jradtilbrook I'll change it. I do feel as though that's a preference thing, but if it's going to block the PR then it's not a hill I'm going to die on.

@jradtilbrook I'll change it. I do feel as though that's a preference thing, but if it's going to block the PR then it's not a hill I'm going to die on.

Yeah because the tests are already slow, we should try to speed them up and parallelise them if we can


Global Forms Label

Description
With global forms, the ‘name’ property is used as the label in the global menu instead of 'label' as expected.
Steps to reproduce
Define a global form with a label but no name. No text will be rendered in the global forms list in the sidebar.

Expected result
I'd expect the 'label' property gets used to label the form.
Actual result
The label property is not used.

Is this still the case? It looks like GlobalFormPlugin sets name to the form label

Closing this — just tested in demo-gatsby and it's working as exptected. iow if you pass a label, the label is rendered in the global form list, if you don't pass a label, the name is rendered.


Copy & Paste support

On OS X at least, standard clipboard commands don't work, probably due to lack of menu bindings.
Should be able to use clipboard throughout the app.
An example of this:
https://pracucci.com/atom-electron-enable-copy-and-paste.html

we could evaluate using https://www.npmjs.com/package/electron-context-menu


luci-app-ocserv not setting up Firewall

Hello,
I used luci-app-ocserv to set up an openconnect server. Everything was set up correctly, except for the firewall. the Firewall Zone setting did not really do anything.
The only fix so far was to add the following to /etc/config/network:
config interface 'VPN'
option proto 'none'
option ifname 'vpns+'
and to add the VPN interface to the lan zone.
Am I missing something here?

The workaround is (unknown to me) documented here anyway: https://github.com/openwrt/packages/tree/master/net/ocserv
So it is just the luci app that does not do that as it seems.
btw, i am on trunk.

No action for years; closing


Better setup for android build version.

React Native now has its project set up with default build version of androids sdk. Set android build version to that of rootProject’s to prevent from build failing cause of version mismatch.

@thorbenprimke Could you please take a look at this PR? Thanks!

@maicki @thorbenprimke I just got notified and I updated code safer.

@maicki Someone already gave PR on this one. Thanks for the merge.


Unkown opcode error

With Rust 1.70, unsupported op codes appear when uploading the wasms on-chain. (See : https://github.com/CosmWasm/cosmwasm/issues/1727)
One fix is available on that issue. Would it be possible to implement that ?
Thanks

Im on Rust 1.73 and CosmWasm 1.5 but don't see this any issues. I did however see this error with cargo wasm.

updating to cosmwasm-std 1.5 doesn't seem to solve the issue :
Error calling the VM: Error during static Wasm validation: Wasm bytecode could not be deserialized. Deserialization error: \"Unknown opcode 192\"

Ok, this works with cosmwasm 1.5 indeed, thank you !


Fix join-contracts.py without --import-map

Before this commit, join-contracts.py without --import-map failed with
an error message:
  File "utils/join-contracts.py", line 66, in main
    import_map = json.loads(import_map)
  File "/usr/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

This issue fixes the problem by changing the default value from "" to
"{}".
I know this script is not used anywhere yet, so [skip ci].

That sounds like a cleaner way but forgive my laziness.
My current aim is to remove these concatenations: https://github.com/raiden-network/raiden-contracts/blob/fdede381d1f205d404c67af988e2574df69eb1d9/raiden_contracts/deploy/etherscan_verify.py#L94


Feature: Kielbasiago's Update coliseum flags to handle percentage random

https://github.com/AtmaTek/WorldsCollide/pull/24

Merged with 1.3.0 https://github.com/ff6wc/WorldsCollide/pull/64


[payment] - add manual instrumentation

Fixes #53

[x] Services extend automatic instrumentation.

[x] New attributes/events attached to existing spans.
[x] New spans are being created from existing spans.

Changes
Adds attributes to auto instrumented spans, and creates a new span with attributes.
If a bad card is used, span.recordException() is called from the auto-instrumented span, which ultimately creates a Span event.

Let’s make sure the feature table & service tracing issue reflect the changes


Do not initialize sgemm matrix C twice

Speedup on de_core_news_lg: 26924 -> 28515.

@danieldk Sweet! Nice catch.


Highlights from exluded field not showing up in Instantsearch hit (question)

Description
More of a question than an issue.
On the typesense end, I am using a scoped api key to remove fullText field from the search results but including the fullText highlight snippets.
This is what typesense returns for the search results, which is exactly what I need:

It excludes the full field value and only shows the snippet for the relevant content.
However this data/info seems to get lost with when it goes through instantsearch to the 'Hit' object. The highlighted snippet is not present in the _highlightResult or _snippetResult:

Is this a limitation of instantsearch, where it doesn't pick up a highlight field if it's not present in the search results? Is there anyway for me to use the highlight snippet without having to replace instantsearch with something custom? Is there a setting that will allow the highlight snippet from the typesense server to pass through to the instantsearch hit?
Thanks!

Could you share the code snippet that shows how you instantiate the adapter, and also the version of typesense-instantsearch-adapter and typesense server you're using?

I am using typesense cloud (v0.24.0) and typesense-instantsearch-adapter 2.6.0.
First I am creating a scoped api key like so:
  let scopedAPIKey = client
    .keys()
    .generateScopedSearchKey(keyWithSearchPermissions, {
      exclude_fields: "fullText",
      highlight_fields: "name,fullText,providerName,description,tags",
      highlight_full_fields: "name,providerName,description,tags",
      limit_hits: 20,
      //      limit_multi_searches: 20,
    });

Then I am using that scopedApiKey to instantiate the typesense adapter:
const researchSearchClient = new TypesenseInstantSearchAdapter({
  server: TYPESENSE_CONFIG,
  additionalSearchParameters: {
    query_by: "name,fullText",
    drop_tokens_threshold: 0,
    sort_by: "publishedAt:desc,_text_match:desc",
  },
}).searchClient;

I also tried including 'fullText' in the highlight_fields as part of the adapter parameters.

going through the adapter code, I think I've tracked down where the highlight gets lost. The adapter creates highlights by iterating over the document fields and finding corresponding entries in the highlight object. but because I have explicitly excluded the value of the 'fullText' field from the document it wont get added to the snippets, even though the snippet is present in the server response.
The solution is probably to do concat the document key/value entries with the highlight key/snippet entries where the key is not in the document.
What do you think?
Below is the code I am referring to:
  _adaptHighlightInObjectValue(objectValue, highlightObjectValue, snippetOrValue) {
    return Object.assign(
      {},
      **/// here it creates snippet results by mapping over the document, but we want to highlight a field
      /// which we intentionally did not include in the document.**
      ...Object.entries(objectValue).map(([attribute, value]) => {
        let adaptedValue;
        if (value == null) {
          adaptedValue = this._adaptHighlightNullValue();
        } else if (Array.isArray(value)) {
          adaptedValue = this._adaptHighlightInArrayValue(
            value,
            highlightObjectValue?.[attribute] ?? [],
            snippetOrValue
          );
        } else if (typeof value === "object") {
          adaptedValue = this._adaptHighlightInObjectValue(
            value,
            highlightObjectValue?.[attribute] ?? {},
            snippetOrValue
          );
        } else {
          adaptedValue = this._adaptHighlightInPrimitiveValue(value, highlightObjectValue?.[attribute], snippetOrValue);
        }

        return {
          [attribute]: adaptedValue,
        };
      })
    );
  }

I tried to fix the issue in the pull request but that didn't seem to fix it, so I will use the _rawTypesenseHit field to construct a custom highlight instead.

The solution is probably to do concat the document key/value entries with the highlight key/snippet entries where the key is not in the document.

Yup. I'll re-open this, so we can keep a track of this issue. I plan to address this in the next few weeks.


Update hint for advanced-typeshed

https://github.com/python/typeshed ->
https://github.com/python/typeshed/blob/main/stdlib/_typeshed/__init__.pyi

Resolve #86

Thanks!


columns overflow horizontally because of missing .row

It appears .row is not defined in the CSS.  Adding this:
.row{margin-right:-15px;margin-left:-15px}
helped solve the horizontal overflowing of a simple three- .col-md-4 layout.

Actually, I take it back: it did not solve the overflow.  I gave up at this point and switch back to standard bootstrap, which yielded no overflow (but reversed (i.e. LTR) order of columns, as expected).

this ( #117 ) seems to resolve the issue


refactor!: Len class refactoring

BREAKING CHANGES
Closes #47

@rultor merge.

@rultor merge.

@rafamizes OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge.

@rafamizes Done! FYI, the full log is here (took me 2min)


Please add support for external networks

It looks like dnsdock doesn't work when containers are attached to a custom network. Consider an example:
First. Create a custom network:
docker network create --subnet <IP_ADDRESS>/16 test

Second. Make containers use this network:
version: "2"

services:
  web1:
    image: nginx
    dns: <IP_ADDRESS>
    environment:
      - DNSDOCK_NAME=node1
      - DNSDOCK_IMAGE=test
    networks:
      test:
        ipv4_address: <IP_ADDRESS>

  dnsdock:
    image: tonistiigi/dnsdock
    volumes:
      - /var/run/docker.sock:/run/docker.sock
    ports:
      - 53/udp
      - 8080:80
    networks:
      test:
        ipv4_address: <IP_ADDRESS>

  tester:
    image: ubuntu
    command: bash -c "ping node1.test.docker -c3"
    links:
      - web1
      - dnsdock
    dns: <IP_ADDRESS>
    networks:
      test:
        ipv4_address: <IP_ADDRESS>

networks:
  test:
    external: true

When I run Docker Compose, the output I see is:
dnsdock_1  | 2016/07/31 17:24:18 Added service: c9a99bbf7783ba5d822504829ba91e762b8c472379e5e529933c13d014b20d09 {dnsdockexperiment_dnsdock_1 dnsdock <nil> -1 []}
dnsdock_1  | 2016/07/31 17:24:18 Added service: c9a99bbf7783ba5d822504829ba91e762b8c472379e5e529933c13d014b20d09 {dnsdockexperiment_dnsdock_1 dnsdock <nil> -1 []}
dnsdock_1  | 2016/07/31 17:24:18 Added service: 08bd9894a14abf223bbdfe87ea97d108cd51fe2ab52a141024f18e3654273302 {node1 test <nil> -1 []}
dnsdock_1  | 2016/07/31 17:24:18 Added service: 8e4a4f0ea0305c426e7518a987287c1f813861366f89be4ed07626ae597ebc10 {clever_blackwell ubuntu <nil> -1 []}
dnsdock_1  | 2016/07/31 17:24:18 Added service: a7b85cbde6b0166e285c48673c41b47c181e1a9922a1d340cd5f7a38b2003c6a {dnsdockexperiment_tester_1 ubuntu <nil> -1 []}
tester_1   | ping: unknown host node1.test.docker
dnsdock_1  | 2016/07/31 17:24:18 Stopped service: a7b85cbde6b0166e285c48673c41b47c181e1a9922a1d340cd5f7a38b2003c6a
dnsdockexperiment_tester_1 exited with code 2

Here's what dig says:
dig @<IP_ADDRESS> node1.test.docker
;; Warning: Message parser reports malformed message packet.

; <<>> DiG 9.10.3-P4-Ubuntu <<>> @<IP_ADDRESS> node1.test.docker
; (1 server found)
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 55868
;; flags: qr rd; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 0
;; WARNING: recursion requested but not available

;; QUESTION SECTION:
;node1.test.docker.		IN	A

;; Query time: 1 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Sun Jul 31 13:28:15 EDT 2016
;; MSG SIZE  rcvd: 64

Here's what curl says:
curl http://<IP_ADDRESS>/services
{
	"08bd9894a14abf223bbdfe87ea97d108cd51fe2ab52a141024f18e3654273302": {
		"Name": "node1",
		"Image": "test",
		"Ip": "",
		"Ttl": -1,
		"Aliases": []
	},
	"c9a99bbf7783ba5d822504829ba91e762b8c472379e5e529933c13d014b20d09": {
		"Name": "dnsdockexperiment_dnsdock_1",
		"Image": "dnsdock",
		"Ip": "",
		"Ttl": -1,
		"Aliases": []
	}
}

Here's what docker inspect says:
docker inspect 08bd
[
    {
        "Id": "08bd9894a14abf223bbdfe87ea97d108cd51fe2ab52a141024f18e3654273302",
        ...
        "NetworkSettings": {
            "Bridge": "",
            "SandboxID": "940e18a4793035e36674f4e29d5f1ec3f0fa97ed39d5d02be45a4f6f8b1bc395",
            "HairpinMode": false,
            "LinkLocalIPv6Address": "",
            "LinkLocalIPv6PrefixLen": 0,
            "Ports": {
                "443/tcp": null,
                "80/tcp": null
            },
            "SandboxKey": "/var/run/docker/netns/940e18a47930",
            "SecondaryIPAddresses": null,
            "SecondaryIPv6Addresses": null,
            "EndpointID": "",
            "Gateway": "",
            "GlobalIPv6Address": "",
            "GlobalIPv6PrefixLen": 0,
            "IPAddress": "",
            "IPPrefixLen": 0,
            "IPv6Gateway": "",
            "MacAddress": "",
            "Networks": {
                "test": {
                    "IPAMConfig": {
                        "IPv4Address": "<IP_ADDRESS>"
                    },
                    "Links": null,
                    "Aliases": [
                        "08bd9894a14a",
                        "web1"
                    ],
                    "NetworkID": "e8c5a53271330faf10d186bb73f1cc512c8153f69644a4350b12a6912dfe4cea",
                    "EndpointID": "2f9f6a82e79c401d0910526dbefd81317eab312d2ce80234043d48d434cfa19e",
                    "Gateway": "<IP_ADDRESS>",
                    "IPAddress": "<IP_ADDRESS>",
                    "IPPrefixLen": 16,
                    "IPv6Gateway": "",
                    "GlobalIPv6Address": "",
                    "GlobalIPv6PrefixLen": 0,
                    "MacAddress": "02:42:ac:19:00:0b"
                }
            }
        }
    }
]

I assume that dnsdock only looks at top level IPAddress and ignores the Networks collection. The top level IPAddress is OK when in bridge mode, but once you switch to external network, it's always empty.
Please consider adding support for this scenario.

Which version did you use?

Tried with both "latest" and "v1.10.0" (https://hub.docker.com/r/tonistiigi/dnsdock/tags/)

The issue has been solved with amd64-1.13.1

can you try ?

amd64-1.13.1 fails to get a list of containers:
dnsdock_1  | 2016/08/01 22:16:52 Handling DNS requests for docker.
dnsdock_1  | 2016/08/01 22:16:52 Error getting containers: Cannot connect to the Docker daemon. Is the docker daemon running on this host?

Tried my original "bridge" scenario, the only change is amd64-1.13.1 instead of latest.

In your compose file you map /var/run/docker.sock to /run/docker.sock
Can you change that and map it to /var/run/docker.sock?

This change helped. Both bridge and external network scenarios have succeeded. Thank you a lot!


fix(client_cli): Proxy fixes

A couple of minor issues I found:

Use the prisma user in the db proxy url, otherwise non elecrified tables are generated. Bad call, the prisma user is correctly being used for the introspection. The proxy url doesn't always need to use the prisma user. We can however log the correct proxy url used when generating the client, because the one in the config differs the one used for the introspection.
https://github.com/electric-sql/electric/blob/7fbf292ef1bcb07a71fd4e2a528d2cda965ac5ce/clients/typescript/src/cli/migrations/migrate.ts#L73
https://github.com/electric-sql/electric/blob/7fbf292ef1bcb07a71fd4e2a528d2cda965ac5ce/clients/typescript/src/cli/migrations/migrate.ts#L303-L309

Handle ssl mode when building the db url, so that when running locally, the db and proxy urls are built with ?sslmode=disable. Some postgres migration clients won't connect without that flag in the url when running without ssl.

Thanks @davidmartos96 for the changes, I've merged them into my branch of various fixes over here: https://github.com/electric-sql/electric/pull/834
I plan to finish this off, and do some more testing, tomorrow.

@davidmartos96

Some postgres migration clients won't connect without that flag in the url when running without ssl.

I'd like to know which clients have this.

@alco dbmate is an example, where it's required to provide ?sslmode=disable in the URL, and if I remember correctly, go-migrate too, which we used previously.

@alco dbmate is an example, where it's required to provide ?sslmode=disable in the URL, and if I remember correctly, go-migrate too, which we used previously.

Thanks!


opam install node not working

This project sounds good, I am trying to compile by git clone because opam install node not work. Would be very nice if you publish this project in opam packages

This project is just a prototype for now (I just pushed it yesterday), so I probably won't be releasing it in a while. If you want to use it regardless, I can always pin it: opam pin ocaml-node https://github.com/tmattio/ocaml-node.git

thanks @tmattio


Validating entities

Hey – I've been looking at api_struct with interest, to use in a Rails application where the models use an API for persistence, rather than a database. I'm interested in whether it's possible to rename attributes on-the-fly. For example, if I'm dealing with a messy API that, for /users/1 returns { "fNm": "Sam" }, then is there a nice way to have it so that when I do User.get(1) I have an entity with a first_name object? I'd also want to have a User.save method that I could pass User entity to, and have first_name map back to fNm when POSTing to the API. I hope that makes sense...
I think it's outside the scope of api_struct, which I'm enjoying using, but I wondered if you knew of any other gems that could do this. I know Hashie has some support for this, but it only goes in one direction.
Thanks! It's nice the the library is so small and focussed, because I can use other things for validation etc.

@kirillshevch Great, thanks for letting me know! I assume the alias only works in one direction, and that's when fetching data?

Yep, only in one direction


Brewery meetup Saturday

Looks like Saturday afternoon has been proposed as a date. Here is a Doodle with proposed times... https://doodle.com/poll/r7dhv2yuexwimxn5

1822 Garner Station Blvd, Raleigh, NC 27603

Does Brice's have Wifi? I can bring the Mevo.
On Mon, Aug 14, 2017 at 6:34 PM, Chris Mathews<EMAIL_ADDRESS>wrote:

1822 Garner Station Blvd, Raleigh, NC 27603
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/543#issuecomment-322328551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3Ie8BHO0aE9n97pv3vPhyAtMn3ECcOks5sYMuQgaJpZM4O2-pu
.

I believe they do and we also have a mifi that can be used.
Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------From: Leo Suarez<EMAIL_ADDRESS>Date: 8/16/17  9:32 AM  (GMT-05:00) To: codeforraleigh/NCOpenPass<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [codeforraleigh/NCOpenPass] Brewery meetup Saturday (#543)
Does Brice's have Wifi? I can bring the Mevo.
On Mon, Aug 14, 2017 at 6:34 PM, Chris Mathews<EMAIL_ADDRESS>wrote:

1822 Garner Station Blvd, Raleigh, NC 27603

—

You are receiving this because you were assigned.

Reply to this email directly, view it on GitHub

https://github.com/codeforraleigh/NCOpenPass/issues/543#issuecomment-322328551,

or mute the thread

https://github.com/notifications/unsubscribe-auth/AB3Ie8BHO0aE9n97pv3vPhyAtMn3ECcOks5sYMuQgaJpZM4O2-pu

.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/codeforraleigh/NCOpenPass","title":"codeforraleigh/NCOpenPass","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/codeforraleigh/NCOpenPass"}},"updates":{"snippets":[{"icon":"PERSON","message":"@dtraleigh in #543: Does Brice's have Wifi? I can bring the Mevo.\n\nOn Mon, Aug 14, 2017 at 6:34 PM, Chris Mathews \u003cnotifications@github.com\u003e\nwrote:\n\n\u003e 1822 Garner Station Blvd, Raleigh, NC 27603\n\u003e\n\u003e —\n\u003e You are receiving this because you were assigned.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/codeforraleigh/NCOpenPass/issues/543#issuecomment-322328551\u003e,\n\u003e or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/AB3Ie8BHO0aE9n97pv3vPhyAtMn3ECcOks5sYMuQgaJpZM4O2-pu\u003e\n\u003e .\n\u003e\n"}],"action":{"name":"View Issue","url":"https://github.com/codeforraleigh/NCOpenPass/issues/543#issuecomment-322771765"}}}

Going to call Doodle closed. We have three attending I say we meet at 2-2:30 on Saturday at Brices to sample the beer and check out the Mevo. All are still welcome to attend.


access-control: viewers limit

Remove rate limit and rate limit cache

@gioelecerati would you mind updating the PR description with a reason? Was this only needed for the duration of the fishtank season?

@gioelecerati gentle nudge to use PR descriptions explaining the what + why.


Added simple logic for reconnecting socket when titan-server restarts

I'll close to prevent a merge.


Local http://cocorico.dev/ is showing Symfony welcome page only

Hello,
How could I get a working local copy of cocorico? When I type http://cocorico.dev/ all I see is the Symfony welcome page that says "Your application is now ready. You can start working on it at: C:\wamp64\www\cocorico.dev"
What went wrong? I followed the steps shown in the installation information in github. Thank you.
This is my configuration, in case it helps.
(1) My cocorico folder

(2) My virtual host information

(3) The page I see at http://cocorico.dev/

Bonjour,
Je serais en congés jusqu'au 21/04/2017 inclus. Votre mail sera traité par Jeremy Bensidoun.
Cordialement,
Benjamin Perrault

Hi,
I think you miss some steps while reading installation instructions. I suggest you to install it by carefully reading each step of the installation guide.
PS: You vhosts seems to not be saved.

Hi,
I think you miss some steps while reading installation instructions. I suggest you to install it by carefully reading each step of the installation guide.
PS: You vhosts seems to not be saved.

Thank you for the reply. I am using WAMP and there is a section on ti that says "Virtual hosts", where my vhost "cocorico.dev" appears. I followed the steps and did it three times exactly thinking that I might have missed something. Is there a way in which the installation setup could be more explicit? I think it is missing some details that might be extremely helpful to users like me trying to install it.

Many peoples managed to install cocorico by reading the install documentation. I suppose that your problem is specific to your environment. This issue will help to be more explicit on how to resolve your specific problem.

I close this issue as it seems to depend more on a misunderstanding than an issue.

I agree with @soniacisneros the install should certainly be more prescriptive.  I have made 3 attempts to get this working to date and have been unsuccessful so far.  I get a developer who is familiar with Symfony certainly would be able to figure it out, but someone who is not would be challenged to get this to run.


Improve abstraction from RDFS/OWL

In model, there are only two CRM specific details, both very small:

[ ] The key order hashes
[ ] The trick to add p2 / classified_as at class init time

If they could be abstracted or moved, then the code should work for any rdfs (+ owl inverses) ontology.

Propose to move the key orders to the TSV to be loaded dynamically.
The non CRM properties would be either static (always put @context, then id, then type, then label) or added by the code that adds the property to the object.
And then move classified_as to the vocab module, as that's where it's used from.

Done in #4


Add framework plugin

As I mentioned, I add a framework plugin to mesos-cli and some new features for the already existing task plugin.
The features are:

show all framework
inspect frameworks and tasks
disable the limitation of tasks in the task list

Hey,
I'm on holiday so I'll have a look when I'm back, unless someone else beats me to it.

:joy: Take your time and enjoy your holiday.

I think it would be better to split the changes to the task plugin into its own MR.

Oh please don't push me to split it. :-( Thats very painfull. There will be more PR's for the mesos-cli in the future. Then I will keep an eye to separate it.

I think it would be better to split the changes to the task plugin into its own MR.

Oh please don't push me to split it. :-( Thats very painfull. There will be more PR's for the mesos-cli in the future. Then I will keep an eye to separate it.

Really?
The problem is that fundamentally this MR contains two unrelated changes, so it'd be nice to separate.
Either to review, commit, and revert if necessary...

I think it would be better to split the changes to the task plugin into its own MR.

Oh please don't push me to split it. :-( Thats very painfull. There will be more PR's for the mesos-cli in the future. Then I will keep an eye to separate it.

Really?
The problem is that fundamentally this MR contains two unrelated changes, so it'd be nice to separate.
Either to review, commit, and revert if necessary...

Charles u are killing me. :joy:  But I will do it.

Shell I open three PR's? Technically  there are three kind of changes.

Add Framework Plugins
Update Task Plugins
Update mesos_cli lib.

What do u think?

Sounds good!

@andreaspeters how's the refactoring going? :)

@andreaspeters how's the refactoring going? :)

Didn't found the time the last week. Hopefully the next two weeks.

@andreaspeters I'm closing this one since you've split it into separate MRs.


tcp: flipping the new nodelay code off

pinterest testing showed unexpected deltas, so flipping it off until we root cause.
Risk Level: Low (reinstating prior behavior)
Testing: n/a
Docs Changes: n/a
Release Notes: removed
Reported-by: @rgs1

Seems like you need to merge main to fix the format check.

cc @junr03
This PR reduced coverage for source/common/network/connection_impl.cc and is resulting in flaky coverage test failures.


where can I set display_github = true

I can copy whole layout.html to my local git repo and add
{%- set display_github = true -%}

but I think there might be a clean way, thanks

There may be a cleaner way but I've achieved it with the following in your conf.py as
html_context = {
  "display_github": True, # Add 'Edit on Github' link instead of 'View page source'
  "github_user": "user",
  "github_repo": project,
  "github_version": "master",
  "conf_py_path": "/doc/",
  "source_suffix": source_suffix,
}

cool, good to know, will give it a try, thanks

Closing this due to inactivity, it seems your question is answered. Feel free to reopen if not


Training/Finetuning over TETS

Hi! Do you have any script showing how to train TEMPO using "contextualized" TS such as your TETS dataset ?
Thanks !

Hi Thomzoy,
Thanks for your interest in our multimodal setting! We have added this part in the 'upcoming features' in our Readme file and are planning to publish this script very soon!
Best


Move service account tracking to incremental

https://github.com/istio/istio/blob/8ca6f0f2aa4be0c48a94ebd4f08a624ab057a48c/pilot/pkg/model/push_context.go#L87 does a full snapshot of all services which is costly.
Its used only in CDS: https://github.com/istio/istio/blob/0dc31434d2e75475e17f6e47350c7add240e78e5/pilot/pkg/networking/core/v1alpha3/cluster_builder.go#L71
We should use the incremental ones from EDS: https://github.com/istio/istio/blob/a1ca65553ce4078514b128b9e1e79d2342993b86/pilot/pkg/xds/eds.go#L217

cc @ramaraochavali @hzxuzhonghu would be useful but require some decent refactoring to get the info from into push context


How to change SKActivityIndicator background color?

Hi @SachK13, I couldn't find a way to change the same? Is there any?

Hey @SachK13, I would like to change the background color. Is there any option to do it?

@hemangshah @VinithNivvin current version doesn't have functionality to change background colour but will sure add it to next version, thanks for using it.


ci: Test up to python 3.12

Fixes #
Proposed Changes

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.82%. Comparing base (2e75771) to head (926d8f3).

Additional details and impacted files


double-beep: Watch heracloset.com

double-beep requests the watch of the watch_keyword heracloset\.com. See the MS search here and the Stack Exchange search here.

heracloset\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ

I wonder how this was approved and suddenly closed by SmokeDetector. Seems like it is an interesting case.

Approved by tripleee in Charcoal HQ


Updating to nodejs8.10 as nodejs4.3 is not supported by AWS Lambda.

Issue #, if available:
nodejs4.3 is not supported by AWS Lambda
Description of changes:
updated to nodejs8.10
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This never made it into master?

@kwwaikar @MarkAtwood @hyandell @jamesiri as contributors to this repo, can you please merge?

Hi @jdyke
Thank you so much for bringing this to our notice. The change has been tested and incorporated into the master. Unfortunately this repository is not open for PR and i recommend creating issue so that we can review and validate the pipeline before releasing the change.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

I signed it.
On Wed, Feb 3, 2021, 8:52 PM google-cla[bot]<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot
I signed it! and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fgoogle%2Fit-cert-automation-practice%2Fpull%2F11669
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/it-cert-automation-practice/pull/11669#issuecomment-772778703,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASN6S7Y5MRPXF4QY2YBGQATS5GSQ3ANCNFSM4XBNQ5TQ
.


Add LiveKit as example

Finally adding the so-awaited LiveKit demo.
Issuing: #22
Features:

[x] Create an integration demo between LiveKit and STUNner
[x] LiveKit is not running in the host network (hostNetwork: true) anymore
[x] LiveKit uses STUNner as a STUN server instead of a public one
[ ] Create a simple scaling demo with HPA  using the load simulation

This is fantastic, thanks! Couple of minor comments in closing:

the use of sed in this context is terrible, can't we use kubectl patch?
it'd be nice to jump from the obsolete Ingress API to the fnacy new Gateway API: after all, this is what we are using in STUNner!


How i can navigate to a spacific fragment when user click on notification while application in background

Hello kejunxia,
My application is receiving a push notifications from server while my application is in background and i suppose to open the target fragment when user click on the notification. How i can do this using your amazing library ?

Hi ibrahim,
What I did was to use a application wide controller say AppController to
manage intent routed from notification. 2 scenarios below,

When app is present when notifications come, parse the intent and
broadcast it as a POJO object to other controllers. Or directly reference
the controller to deal with the intent. For example, when a message comes,
in AppController, I "use" ChatControlller and invoke
chatControlller.showMessage(ParsedIntent)
When the app is not present, save the parsed intent somewhere e.g.
Preferences and in the app lifecycle check if there are pending intent
saved and handle it the app is brought back.

Hope it helps.
Regards,
Kejun
On Sun, Mar 12, 2017 at 12:58 AM, ibrahim132<EMAIL_ADDRESS>wrote:

Hello kejunxia,
My application is receiving a push notifications from server while my
application is in background and i suppose to open the target fragment when
user click on the notification. How i can do this using your amazing
library ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kejunxia/AndroidMvc/issues/68, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVrDhTHInqdK4qHiIpC52qWuF929MWgks5rkqhxgaJpZM4MaNeU
.

Hi Kejun,
Thanks man it helps me alot.


MB-6571 Update relevant Office & Admin statuses when User status is updated

Description
This PR keeps the office_users and admin_users tables in sync with users. If a user is deactivated that is also an office or admin user, both Active statuses will update.

user_updater service object now uses admin_user_updater and office_user_updater services objects to update as needed. By using these updaters, we can keep our error messages consistent.
If there is an issue updating the admin_users or office_users tables, an error is logged. I decided to not use a transaction, because if the code gets to this point, it means the user was successfully deactivated. Since one primary use case is security-related, we should always deactivate the user if we can.
Two test functions added to create a new admin or office user that is active, and new tests added.

Reviewer Notes

Is the logging of errors around admin/office not updating sufficient?

Setup
Run tests
go test ./pkg/services/user --testify.m "TestUserUpdater" -v
Use UI to manuall test

Run the server and client, and go to http://adminlocal:3000
Log in as an admin user. Go to the Office Users page and pick an active office user. Grab their email address.
Go to the Users index page and search for the user by email. Edit the user and set Active to No. Save and confirm they are no longer active.
Go back to the Office Users page and find the same user. Confirm their Active status is now false.
Repeat this process for an Admin User.

Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Messages

:book:
:link: MB-6571

  Generated by :no_entry_sign: dangerJS against d0854367679bb8d551407d835df33459b7b6e673


Add list:view command

fixes #12
cleaned this mess hopefully

I'll get to this either today or tomorrow :)

What's the status of this?


Controller doesn't compile chaincode model well (pascal case)

Property 'testchaincode' does not exist on type 'typeof Models'. Did you mean 'Testchaincode'
It's because it's using a lowercase but the rest of the project uses PascalCase.
Replicate
conv new tetproject -c testchaincode
cd tetproject
# Run convector-rest-api there

I cannot replicate it,
I just started from scratch new project and the generation/compilation works..

I just verified that method created in my example (from scratch)
async create(req: Request, res: Response) {
try {
let cntrl = await MytestaController.init();
let modelRaw = req.body;
let model = new Models.Mytesta(modelRaw);
await cntrl.create(model);
res.send(201);
} catch (ex) {
console.log(ex.message, ex.stack);
res.status(500).send(ex);
}
}
So it seems to work I am closing it by now, if you keep encountering it we investigate better


Set default androidInstallTimeout

A default value for this timeout is set within android-helpers.js and also exists downstream in android-adb if the cap is not provided.
If the cap is not provided this message...
 logger.info(Pushing ${appPackage} to device. Will wait up to ${androidInstallTimeout} +                  milliseconds before aborting);
...will print a timeout value of 'undefined' yet still continue with the correct default value of 90000
This PR will fix the misleading message provided to the upstream appium consumer.
Apologizes for the stacked PRs around this single change

bump

Whoops!
Thanks for this. Makes sense!

Published as<EMAIL_ADDRESS>


Update HelloWorld.c

Without main method printing Hello world

Description

Types of changes

[x] Bug fix or improving code quality
[ ] New language
[ ] New feature

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.

Update with new and fast methodologies


Update spur to 0.3.22 and use close() method

I also put a try: finally: in the close() method to still close sftp if there would be an error when closing the spur connection.

Thanks, @iwanb!


[MINOR] Mark a few new configs advanced and tag since version

Change Logs
As above.
Impact
Mark advanced configs and add since version of "0.14.0".
Risk level
None
Documentation Update
Make sure that a few configs that should be advanced do not show up in the basic configuration page and the configs have the right since version.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

CI is green.

As this does not change any core logic and only affects doc generation, will merge this PR now.


Files in ~/.config/jumpscale won't be backed up correctly.

All files saved to ~/.config/jumpscale are actually written to /data/jsngmodel.
~/.config/jumpscale is a symbolic link.
https://github.com/threefoldtech/owncloud_deployer/blob/2e643b5b7f77b929944dcffb889602e9feb410ba/docker/init.sh#L16
I'm not sure what is the added benefit of doing this, but I can see this being done too in TFTshop.
The downside is that the configured backup job will archive all Symlinks as symlinks, we don't back up the actual data.
From Restic Docs

Symlinks are archived as symlinks, Restic does not follow them. When you restore, you get the same symlink again, with the same link target and the same timestamps.

Believe that's mounted volume on the kubernetes cluster where it runs.

Verified
the symlinking no more happens when the container started.


Sorting patients by family name then given name does not seem to work.

Hi, I am using this query:
/Patient/_search?name=&_count=25&_sort%3Aasc=family&_sort%3Aasc=given
To attempt to get all patients sorted by family name, then given name.  I get a bundle back in an order I can't determine, but definitely isn't the right order.  For example:
Aaron Alexis
Daniel Adams
Frank Taylor
Lisa Coleman
Anthony Coleman
Patrick Taylor
And am expecting:
Daniel Adams
Aaron Alexis
Anthony Coleman
Lisa Coleman
Frank Taylor
Patrick Taylor
Thanks!

Thanks for reporting! Committing a fix shortly!
This was a pretty big bug in the way sorting worked, the sort was searching by all indexes of the same type as the parameter you used as opposed to just the specific one you selected. This has been fixed.

Hi James,
Thanks for the quick response!  Is this fix in 1.2-SNAPSHOT?  I tried to install and use that version but get an error saying:
Could not find the HAPI-FHIR structure JAR on the classpath for version DSTU2_HL7ORG. Note that as of HAPI-FHIR v0.8, a separate FHIR strcture JAR must be added to your classpath (or project pom.xml if you a
/DSTU2
Another question for you, we have a client that is needing very much to have java.io.Serializable functionality in the FHIR resource implementation.  I created a forum post for this:
https://groups.google.com/forum/#!topic/hapi-fhir/KXZ4n5zyIxc
Is this something you would be able to support quickly?  Alternatively, we would branch from your 1.1 release and add this functionality ourselves to get our client unblocked.
Thank you,
Travis Cummings
<EMAIL_ADDRESS>On Jul 18, 2015, at 3:05 PM, James Agnew<EMAIL_ADDRESS>wrote:

Closed #198 via 626f467.
—
Reply to this email directly or view it on GitHub.

Hi Travis,
A recent change in the JPA server accidentally introduced a dependency on the RI structures project. Temporarily, if you want to use the 1.2-SNAPSHOT version of the server (where this fix got deployed) you need to also add that dependency on your project:
<dependency>
   <groupId>ca.uhn.hapi.fhir</groupId>
   <artifactId>hapi-fhir-structures-hl7org-dstu2</artifactId>
   <version>1.2-SNAPSHOT</version>
</dependency>

Hoping to remove that dependency shortly, so this won't be a problem for long. Will reply to the Google Group post separately.

I believe I am using the fix and it doesn't seem to work on the secondary sort of given name.  I am using 1.2-SNAPSHOT and seeing these results now.  As you can see, Lisa Coleman and Sarah Graham are out of order.
Daniel Adams
Aaron Alexis
Carol Allen
Ruth Black
Brian Brooks
Susan Clark
Amy Clark
Anthony Coleman
Steven Coleman
Lisa Coleman
Ruth Cook
Betty Davis
Joshua Diaz
Brian Gracia
Stephan Graham
Sarah Graham

Hi Travis,
Unfortunately I'm not able to reproduce this, even in a test using your exact data. A few questions:

Is this using Derby or another DB?
Do searches with only a sort for family and only a sort for given both return the expected results?

Closing old bugs. Please feel free to reopen if this is still happening.

It seems like this is still broken. I am trying it on (STU3) https://sb-fhir-stu3.smarthealthit.org/smartstu3/open/Patient and I am not able to get sorted results no mather what query I use (_sort=name, _sort=given or _sort=family).

Hmm. @travistcummings Any idea what version of HAPI that server is based on?
This definitely works for me on our reference server at least: http://fhirtest.uhn.ca/baseDstu3/Patient?_sort=name

According to the conformance statement it should be 2.4 while the reference server is 2.5-SNAPSHOT.

So, HAPI 2.5 has been released, and it has fairly significant rewrites to the search code, including sorting. Probably the next thing to do is to test once the SMART server has been upgraded.
I'm not seeing this issue locally, so I'm going to assume whatever caused this has been fixed. Please feel free to comment or re-open if this continues.

Hi James,
Can you tell me if v2.5 is compatible with a schema and data from a 2.4 install?
Thanks for your excellent FHIR server!
Travis
Sent from my iPhone

On Jun 9, 2017, at 11:00 AM, James Agnew<EMAIL_ADDRESS>wrote:
Closed #198.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Consul timeouts in v2.20.0

level=error ts=2020-07-26T15:26:29.612Z caller=consul.go:487 component="discovery manager scrape" discovery=consul msg="Error refreshing service" service=consul tags= err="Get \"http://<IP_ADDRESS>:8500/v1/health/service/consul?index=170130&stale=&wait=600000ms\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)",
level=error ts=2020-07-26T15:26:11.690Z caller=consul.go:487 component="discovery manager scrape" discovery=consul msg="Error refreshing service" service=mongo-primary tags= err="Get \"http://<IP_ADDRESS>:8500/v1/health/service/mongo-primary?index=170347&stale=&wait=600000ms\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)",
level=error ts=2020-07-26T15:26:11.679Z caller=consul.go:370 component="discovery manager scrape" discovery=consul msg="Error refreshing service list" err="Get \"http://<IP_ADDRESS>:8500/v1/catalog/services?index=170345&stale=&wait=600000ms\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)",
level=error ts=2020-07-26T15:25:39.607Z caller=consul.go:487 component="discovery manager scrape" discovery=consul msg="Error refreshing service" service=consul tags= err="Get \"http://<IP_ADDRESS>:8500/v1/health/service/consul?index=170130&stale=&wait=600000ms\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)",
level=error ts=2020-07-26T15:25:21.688Z caller=consul.go:487 component="discovery manager scrape" discovery=consul msg="Error refreshing service" service=mongo-primary tags= err="Get \"http://<IP_ADDRESS>:8500/v1/health/service/mongo-primary?index=170347&stale=&wait=600000ms\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)",
level=error ts=2020-07-26T15:25:21.677Z caller=consul.go:370 component="discovery manager scrape" discovery=consul msg="Error refreshing service list" err="Get \"http://<IP_ADDRESS>:8500/v1/catalog/services?index=170345&stale=&wait=600000ms\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)"

With v2.19.3 all clean since started.
Before going from v2.20.0 to v2.19.3 I also upped the limits for Consul:
limits {
    http_max_conns_per_client = 500
}

Originally posted by @hbokh in https://github.com/prometheus/prometheus/issues/7626#issuecomment-664013648

Fixed in #7724 / v2.20.1


Pypy3 issue with  hash160 address

Hi there:
Quick question.
I am using  this library with pypy3 and seems to work fine with compress and uncompress btc address.
But  when try something like that:
mykey = int(key,16)
hash160 = ice.privatekey_to_h160(0, True, mykey).hex()
print(hash160)
I just got   only 40 zeros =<PHONE_NUMBER>000000000000000000000000000000
Can you  spot a solution, thanks!

Can you tell me what was the key for which it is happening. Or is it a secret ??

hi,
The key is just a decimal number like :18446744073709551616.
Again, works fine with pure python, but with pypy3  just got a bunch of zeros.

pypy and pypy3 doesn't support ctypes and finite fields operations. And pure python with ctypes is faster, than pypy.

Hi there: Quick question. I am using this library with pypy3 and seems to work fine with compress and uncompress btc address. But when try something like that: mykey = int(key,16) hash160 = ice.privatekey_to_h160(0, True, mykey).hex() print(hash160) I just got only 40 zeros =<PHONE_NUMBER>000000000000000000000000000000 Can you spot a solution, thanks!

No work with DLL !


add materialize jac option - results in significant speed up for small problems

sounds good thank you. I resolved the conflicts, also updated printing cost and a few other state parameters for verbose=True.


Intermediate-level guide in Time Series Forecasting example

URL(s) with the issue:
https://github.com/tensorflow/docs/blob/master/site/en/tutorials/structured_data/time_series.ipynb
Description of issue (what needs changing):
Clear description
I believe this guide is overly complex for most basic time-series forecasting measures. Due to the introduction of a class for windowing, while this makes it easy to re-use code between examples, it doesn't do a very good job of creating basic windows that users might benefit from for a basic use case.
For example, I only want some basic windowing, I found more help setting this up in the keras preprocessing docs here https://www.tensorflow.org/api_docs/python/tf/keras/preprocessing/timeseries_dataset_from_array than I did in this help page.
While this page demonstrates many potential use cases quite simply, I believe another page is required which outlines the very basic steps just to get one of these working with an intro to windowing as well.

Perhaps this lecture can help to get more clarity on setting up window sizes for time series data.

Thanks, looks like it will be useful, I will review it in detail. However I feel the point still stands, it would be useful to have a simple-case code example in the docs. From my position, of having some experience with ML / neural nets but being new to tensorflow, it's more about understanding and adapting to the syntax and terminology rather than trying to learn the fundamentals, which I think this would help with.
Your call though :)

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Revert "#1: add support for horses in database initialization"

/paulo - can you please review this pull request?

/paulo - can you please review this pull request?

/paulo - can you check now?


fix(dropdown): maxHeight for slot and popup event

🤔 这个 PR 的性质是？

日常 bug 修复

🔗 相关 Issue

https://github.com/Tencent/tdesign-vue/issues/840

💡 需求背景和解决方案

📝 更新日志

fix(dropdown): 插槽模式下maxHeight失效的问题

fix(dropdown): 透传popup事件问题

[ ] 本条 PR 不需要纳入 Changelog

☑ 请求合并前的自查清单
⚠ 请自检并全部勾选全部选项。⚠

[x] 文档已补充或无须补充
[x] 代码演示已提供或无须提供
[x] TypeScript 定义已补充或无须补充
[x] Changelog 已提供或无须提供

Codecov Report

Merging #857 (169ca28) into develop (a0db0df) will increase coverage by 3.84%.
The diff coverage is 69.31%.

@@             Coverage Diff             @@
##           develop     #857      +/-   ##
===========================================
+ Coverage    57.56%   61.40%   +3.84%
===========================================
  Files          346      355       +9
  Lines        12828    13449     +621
  Branches      2405     2552     +147
===========================================
+ Hits          7384     8259     +875
+ Misses        5200     4931     -269
- Partials       244      259      +15

Impacted Files
Coverage Δ

src/avatar/group.tsx
97.14% <ø> (ø)

src/breadcrumb/breadcrumb-item-props.ts
100.00% <ø> (ø)

src/calendar/props.ts
70.00% <0.00%> (-30.00%)
:arrow_down:

src/cascader/props.ts
91.66% <ø> (ø)

src/cascader/utils/inputContent.ts
59.45% <0.00%> (+6.68%)
:arrow_up:

src/date-picker/date-picker.tsx
46.57% <0.00%> (+0.72%)
:arrow_up:

src/date-picker/panel/date-range.tsx
7.48% <0.00%> (+0.29%)
:arrow_up:

src/dialog/props.ts
70.00% <0.00%> (-30.00%)
:arrow_down:

src/dropdown/dropdown-menu.tsx
18.42% <ø> (ø)

src/dropdown/dropdown.tsx
70.00% <0.00%> (+5.00%)
:arrow_up:

... and 128 more

Continue to review full report at Codecov.

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


in 2.1.4, strictNullChecks make index types of nullable fields be never

TypeScript Version:  2.1.4
Code
function get<T, K extends keyof T, K2 extends keyof T[K]>(obj: T, propertyName: K, propertyName2: K2): T[K][K2] | undefined {
    const step1 = obj ? obj[propertyName] : undefined;
    return step1 ? step1[propertyName2] : undefined;
}

interface Org {
    admin?: User;
}
interface User {
    email?: string;
}

const o: Org = { admin: { email: 'foo' } };
const o2: Org = {};

console.log(get(o, 'admin', 'email'))
console.log(get(o2, 'admin', 'email'))

Expected behavior:
Expect to output:
foo
undefined

Actual behavior:
with strictNullChecks, fails to compile:
foo.ts(17,29): error TS2345: Argument of type '"email"' is not assignable to parameter of type 'never'.
foo.ts(18,30): error TS2345: Argument of type '"email"' is not assignable to parameter of type 'never'.

works without strictNullChecks:
foo
undefined

Duplicate of #12651, #12723, and #12744.
fixed by https://github.com/Microsoft/TypeScript/pull/12770

Actually, this isn't a duplicate, but more like a suggestion that we somehow permit indexed access types to work on types that include undefined and/or null. It would probably require a new type operator, e.g. T!, which removes undefined and null from the type of T. We could then say that when a truthy, undefined, or null check is applied to a type parameter T, we produce the type T! in order to record the knowledge that the type is not nullable.
Anyway, I'm going to reopen and change to a suggestion.

Ah yeah i was wondering if this was expected behavior.
T! or somesuch would be great.

Closing due to low feedback volume.


Test test_mainwindow.py hangs

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
This is a somewhat bizarre one; I'm asking just in case you have any idea.  As you may know, we run our CI tests on Debian using lxc.  Because the test_mainwindow.py tests are sometimes flaky, I run them on their own (so I run the main tests with --deselect=spyder/app/tests/test_mainwindow.py and then test test_mainwindow.py separately, once with --run-slow and once without).
This worked fine with 5.4.2, but now that I'm trying to package 5.4.4, I'm finding that test_mainwindow.py with --run-slow typically gets to 99% and then just hangs.  Running it a second time (using the pytest_log.txt to skip previously passed tests) then goes fine - every individual test passes.  But it seems that something changed between 5.4.2 and 5.4.4 that means that something is not finishing properly.  (5.4.3 was released during the Debian freeze, so I didn't try packaging that.)  I tried excluding the test where the process hung, but that didn't help - it just hung on a different test instead.
Do you have any idea what might be causing this?  I am trying to work around it using timeout (though frustratingly that doesn't seem to be working inside the lxc container at the moment for some reason I don't understand), but solving the root cause would obviously be preferable.
Versions

Spyder version: 5.4.4
Python version: 3.11.4
Qt version: 5.15.9
PyQt version: 5.15.9
Operating System name/version: Debian testing

(Edit: BTW, test_mainwindow.py without --run-slow of course does nothing!)

Our 5.x tests have been running fine for several months and I don't see any problems when looking at the Linux Github actions logs:
https://github.com/spyder-ide/spyder/actions/runs/5661429629/job/15339300204
But I did some changes to the way we run our tests on PR https://github.com/spyder-ide/spyder/pull/20425 (included in 5.4.3), which removed the slow marker. Instead, we break the test suite as described here:
https://github.com/spyder-ide/spyder/pull/20425/files#diff-a31c7ed5d35f5ed8233994868c54d625b18e6bacb6794344c4531e62bd9dde59R70
But that means that in your case you don't need to

then test test_mainwindow.py separately, once with --run-slow and once without

If the main window tests are passing without --run-slow, then they are passing as expected.

(Not quite; the --run-slow option is still needed to run the main window tests; they're all marked as "slow", but in conftest.py rather than individually in test_mainwindow.py.)
I had guessed it was some sort of local problem, as I'd seen the actions were passing on GitHub.  I just thought that maybe you would have a clue what might be causing it.  But if not, feel free to close this.

(Not quite; the --run-slow option is still needed to run the main window tests; they're all marked as "slow", but in conftest.py rather than individually in test_mainwindow.py.)

I meant that you can run them separately with
pytest spyder/app/tests/test_mainwindow.py -vv

In that case, --run-slow is not necessary (it does nothing because there are no tests marked as slow, as I said).
How are you running them?

Oh.  We're running the following command:
    export CI=true
    export QTCONSOLE_TESTING=true

    for slow in "" "--run-slow"
    do
        debian/tests/run_pytest \
          xvfb-run -a -s "-screen 0 1024x768x24 +extension GLX" \
          $timeout_cmd $py runtests.py $EXCLUDE $slow spyder/app/tests/test_mainwindow.py
    done

where debian/tests/run_pytest is a wrapper script that runs the test up to five times, using the pytest_log.txt mechanism to record previously successful tests.
Interestingly though, when the test is run without --run-slow, nothing is tested, hence my comment above:
*** STARTING RUN 1 at 10:37:55: xvfb-run -a -s -screen 0 1024x768x24 +extension GLX timeout -k 5 1200 python3.11 runtests.py --deselect=spyder/plugins/editor/widgets/tests/test_hints_and_calltips.py::test_get_hints --deselect=spyder/app/tests/test_mainwindow.py::test_run_cython_code --deselect=spyder/plugins/findinfiles/widgets/tests/test_widgets.py::test_no_empty_file_items --deselect=spyder/plugins/ipythonconsole/tests/test_ipythonconsole.py::test_tk_backend spyder/app/tests/test_mainwindow.py
Pytest Arguments: ['-vv', '-rw', '--durations=10', '-W ignore::UserWarning', '--timeout=120', '--deselect=spyder/plugins/editor/widgets/tests/test_hints_and_calltips.py::test_get_hints', '--deselect=spyder/app/tests/test_mainwindow.py::test_run_cython_code', '--deselect=spyder/plugins/findinfiles/widgets/tests/test_widgets.py::test_no_empty_file_items', '--deselect=spyder/plugins/ipythonconsole/tests/test_ipythonconsole.py::test_tk_backend', 'spyder/app/tests/test_mainwindow.py']
============================= test session starts ==============================
platform linux -- Python 3.11.4, pytest-7.4.0, pluggy-1.2.0 -- /usr/bin/python3.11
cachedir: .pytest_cache
PyQt5 5.15.9 -- Qt runtime 5.15.8 -- Qt compiled 5.15.8
rootdir: /tmp/autopkgtest-lxc.khxxpya5/downtmp/build.B2K/src
configfile: pytest.ini
plugins: cov-4.1.0, lazy-fixture-0.6.3, order-1.0.1, qt-4.2.0+repack, timeout-2.1.0, mock-3.11.1, flaky-3.7.0
timeout: 120.0s
timeout method: signal
timeout func_only: False
collecting ... collected 120 items / 1 deselected / 119 selected

spyder/app/tests/test_mainwindow.py::test_single_instance_and_edit_magic SKIPPED [  0%]
spyder/app/tests/test_mainwindow.py::test_default_plugin_actions SKIPPED [  1%]
spyder/app/tests/test_mainwindow.py::test_dedicated_consoles SKIPPED     [  2%]
spyder/app/tests/test_mainwindow.py::test_change_types_in_varexp SKIPPED [  3%]
spyder/app/tests/test_mainwindow.py::test_preferences_change_interpreter SKIPPED
 [  4%]
spyder/app/tests/test_mainwindow.py::test_no_update_outline SKIPPED      [  5%]
spyder/app/tests/test_mainwindow.py::test_leaks SKIPPED (Need --run-slow
option to run)                                                           [  5%]
spyder/app/tests/test_mainwindow.py::test_lock_action SKIPPED (Need
--run-slow option to run)                                                [  6%]
[...]
================ 119 skipped, 1 deselected, 8 warnings in 0.63s ================
*** END OF RUN 1: ALL TESTS RUN HAVE NOW PASSED/XFAILED ***

Ah, I think it's because we've set CI=true, so conftest.py marks all of the tests as slow!

Yep, that's what I was thinking too.

Well, I did a little more investigation and the results are bizarre, to say the least. The test fails on the first attempt at test_clickable_ipython_tracebacks at the line
    assert QApplication.overrideCursor().shape() == Qt.PointingHandCursor

so it attempted to try again (because the test is marked flaky).  Unfortunately, it then hangs right at the start, specifically at the line:
    qtbot.waitUntil(lambda: shell._prompt_html is not None,
                    timeout=SHELL_TIMEOUT)

Running just this one test on its own works fine.  Removing this test makes the whole test file succeed without hanging.
But adding an extra test at the end of the file also works:
@flaky(max_runs=10)
def test_null_test(main_window, qtbot, tmp_path):
    pass

(The @flaky and fixtures are there to make the signature of the test look similar to that of the clickable test.)   In this case, the first run through test_clickable_ipython_tracebacks still fails in exactly the same way, the second attempt (controlled by flaky) works without a problem.
So there's some really strange interaction going on somewhere.  No expectation that you'll be able to solve it, and I don't think it's worth more effort, but I just wanted to say what I had discovered.

No expectation that you'll be able to solve it, and I don't think it's worth more effort

Ok, closing then.

but I just wanted to say what I had discovered.

Ok, thanks for sharing.


Added smooth scrolling to TextEdit

The PR adds smooth scrolling to TextEdit, includes a setting to toggle and exposed to the user.

closes #8953

NICE this is really really cool Thank you!
I checked your code and Couldn't find if it is also changing the way horizontal scrolling works. It needed a huge value to scroll as fast as the vertical scrolling? is that fixed?

Yeah this only changes the vertical scrolling. Dragging felt smooth with horizontal and I can't test the mouse wheel left and right buttons.


Set a Resource cap of 8Gb RAM for deployments

Do not let platform users use more than 8Gb RAM for one deployment. Open-source users should be able to request as much memory as they want.

The line break didn't work in the error message

Cut the check "requested memory"

TESTED - OK


200 | Number Of Islands | C++

Title - Number Of Islands
What will change - Adding a new file that is Q 200. Number Of Islands on LeetCode in C++.
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[] Improving Code
[] Improving Documentation
[] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[] Python
[x] C++
[] Java
[] SQL

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

hey @geetanshatrey can you assign this to me?

Hey @geetanshatrey can you assign this issue to me?

plz assign me it will be a good help

Please assign this to me for Python


Port /api/v0/ to graphql on /api/graphql

This PR is a backbone for a GraphQL API which I think Manifold would really benefit using.
For now only markets, slug and market are implemented as proof of concept. Changing what gets exposed and the API is very simple, most of the time only the graphql files will need to change.
To test: /api/graphql
Example query:
{
  slug(url: "can-i-please-have-a-free-answer-mar")
  {
    bets
    {
      amount
      createdTime
      id
      isAnte
      isSold
      market
      {
        id
      }
      outcome
      probAfter
      probBefore
      sale
      {
        amount
      }
      shares
      user
      {
        avatarUrl
        id
        name
        username
      }
    }
    closeTime
    comments
    {
      answerOutcome
      answers
      {
        id
        text
      }
      createdTime
      id
      market
      {
        id
      }
      replyTo
      {
        id
      }
      text
      user
      {
        avatarUrl
        id
        name
        username
      }
    }
    createdTime
    creator
    {
      avatarUrl
      id
      name
      username
    }
    description
    id
    isResolved
    mechanism
    outcome
    {
      __typename
      ...on MarketOutcomeBinary
      {
        probability
      }
      ...on MarketOutcomeFreeResponse
      {
        answers
        {
          createdTime
          creator
          {
            avatarUrl
            id
            name
            username
          }
          id
          number
          probability
          text
        }
      }
      ...on MarketOutcomeNumeric
      {
        max
        min
      }
    }
    p
    pool
    question
    resolution
    resolutionTime
    tags
    totalLiquidity
    volume
    volume7Days
    volume24Hours
  }
}```

Caching is implemented, as well as lazy responses (only what is queried will be fetched from the database). Other benefits include being able to keep track of what field gets used (for deprecation rollout), ease of dynamic use for programmers, and higher versatility compared to a REST api.

The Vercel logs say that yarn build failed like so:

You should try yarn build on your box and see if it works.

Thanks, is there a way to run all workflow approval locally? I'm running yarn build and yarn verify before pushing. The typescript error on vercel is due to it not running yarn generate before typecheck, maybe I should add it to yarn verify?
There is still an error on my end, which is due to database initialization I've missed. I'm trying to solve it now!

Actually, it seems the problem I have is also present on the main branch (it says
Error [FirebaseError]: Firebase: No Firebase App '[DEFAULT]' has been created - call Firebase App.initializeApp() (app/no-app). ).

Okay, I've added yarn generate to every scripts, which should fix the pipeline. I can rebase the branch and reopen a PR if you'd like

This is amazing! Being able to specify exactly what I need in a query would be so useful, rather than getting a dump of the entire database and parsing out what I want.

Just to manage expectations -- I am spinning some other plates right now so it might be a few days before I provide substantial feedback on this.

@joy-void-joy sorry for us ghosting you on this. I think it's very unlikely at this point that we'll do anything with this particular branch. There's been so much drift in the api code by now and will be more when we move the whole thing off vercel pretty soon. So unfortunately I'll be closing this.


Added support for .js.gz and .css.gz files

As discussed in #74 I made a simple addition to the render_bundle template tag to handle .js.gz and .css.gz files.
I also added some missing requirements to requirements_dev.txt for any future contributors.

Coverage remained the same at 97.945% when pulling d76ac95715cc7c847a1c00e639683cda0f4ec3e6 on mikeengland:master into 9e517583d19ccc2c72e846fdd267c8aa25d72507 on owais:master.

Coverage remained the same at 97.945% when pulling d76ac95715cc7c847a1c00e639683cda0f4ec3e6 on mikeengland:master into 9e517583d19ccc2c72e846fdd267c8aa25d72507 on owais:master.


Add initial bower configuration

This PR adds a package configuration for bower.
Please see https://bower.io/docs/creating-packages/ for more information.

alright, it's up on the bower-registry :bird:
I saw you added yourself to the authors (which is great, considering all the code you contributed).  Are you be okay fielding/forwarding any emails that come your way?
I only get them rarely, but if you're not itching for emails it's fine if you want your email off.

alright, it's up on the bower-registry :bird:

Great, thanks!

I saw you added yourself to the authors which is great considering all you contributed.
Are you be okay fielding/forwarding emails that come your way? (it'll be infrequent at most)
If you're not itching for emails it's just fine if you want your email off.

Oh, I didn't really intend to add my email address there.


LocalScheduler work stealing bug

I discovered a bug in the LocalScheduler which is affecting one of our applications. The app makes use of ThreadPoolScheduler and DispatcherScheduler, which both inherit from the LocalScheduler base class. I have attached a minimal source file which duplicates the problem. If everything worked as expected, the program should print "GUI" and "THREADPOOL" forever. Instead, it will eventually print the "FAIL!" message.
The "FAIL!" case arises when LocalScheduler.ExecuteNextShortTermWorkItem runs on the ThreadPoolScheduler, but 'next' (the result of s_shortTerm.Dequeue) is a DispatcherScheduler WorkItem. In other words, the running thread is a ThreadPool thread (and 'scheduler' is a ThreadPoolScheduler), but 'next'._scheduler is a DispatcherScheduler. The appearance of the bug is also sensitive to the size of the application's delays relative to to the size of LocalScheduler.RETRYSHORT (which is currently 50 ms).
I'm not sure what is the best way to fix this. One way is to convert s_shortTerm and the related data structures into instance variables rather than static variables. That will keep the various workItems from mixing together. Another possibility is to store a separate short term set per subclass (one for DispatcherScheduler, one for ThreadPoolScheduler, etc). A third possibility is to make ExecuteNextShortTermWorkItem use 'next'._scheduler to run the work, but I'm not sure if that causes other problems (excessive delays, ordering issues, etc).
Program.cs
using System;
using System.Diagnostics;
using System.Reactive.Concurrency;
using System.Windows;
using System.Windows.Threading;

namespace RxSandbox4 {
  public class ModelObject : DependencyObject {
    public void Start() {
      Schedule();
    }

    private void Schedule() {
      var now=DateTimeOffset.Now;
      const int dispatcherDelay=55;
      const int threadPoolDelay=60;
      new DispatcherScheduler(Dispatcher, DispatcherPriority.SystemIdle)
        .Schedule(now.Add(TimeSpan.FromMilliseconds(dispatcherDelay)), RunMeOnGui);
      ThreadPoolScheduler.Instance
        .Schedule(now.Add(TimeSpan.FromMilliseconds(threadPoolDelay)), RunMeOnThreadPool);
    }

    private void RunMeOnThreadPool() {
      Debug.Print("THREADPOOl");
    }

    private void RunMeOnGui() {
      if(!CheckAccess()) {
        Debug.Print("FAIL!");
        return;
      }
      Debug.Print("GUI");
      Schedule();
    }
  }

  internal class Program {
    private static void Main(string[] args) {
      Dispatcher.CurrentDispatcher.InvokeAsync(() => {
        var model=new ModelObject();
        model.Start();
      });
      Dispatcher.Run();
    }
  }
}

Please let me know if you need any more information. Thanks.

Great catch! We have a candidate fix for this, which will be verified and tested for the next release of Rx.NET.

There appears to be an issue with your fix, namely LocalScheduler.SystemClockChanged() is now an instance-level handler that gets subscribed to a static event, but it is never unsubscribed.


Unable to find fallback package folder 'C:\Program Files\dotnet\sdk\NuGetFallbackFolder'.

Expected Behavior
Solution should open gracefully in vscode.
Actual Behavior
After following instructions for Lean development container, i get the following errors :
[Error - 5:21:44 AM] [LanguageServerProjectSystem] Error while loading /Lean/DownloaderDataProvider/QuantConnect.DownloaderDataProvider.Launcher.csproj: The "ResolvePackageAssets" task failed unexpectedly.
NuGet.Packaging.Core.PackagingException: Unable to find fallback package folder 'C:\Program Files\dotnet\sdk\NuGetFallbackFolder'.
   at NuGet.Packaging.FallbackPackagePathResolver..ctor(String userPackageFolder, IEnumerable`1 fallbackPackageFolders)
   at Microsoft.NET.Build.Tasks.NuGetPackageResolver.CreateResolver(IEnumerable`1 packageFolders)
   at Microsoft.NET.Build.Tasks.NuGetPackageResolver.CreateResolver(LockFile lockFile)
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheWriter..ctor(ResolvePackageAssets task)
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.CreateReaderFromDisk(ResolvePackageAssets task, Byte[] settingsHash)
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader..ctor(ResolvePackageAssets task)
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ReadItemGroups()
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ExecuteCore()
   at Microsoft.NET.Build.Tasks.TaskBase.Execute()
   at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
   at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask)

Reproducing the Problem

I followed the documentation here : https://github.com/QuantConnect/Lean/tree/master/.vscode
More specifically, option 2 "Lean Development Container".

got vs code
got docker
ran docker pull quantconnect/research:latest
ran git clone https://github.com/QuantConnect/Lean
opened folder with vs code
clicked "Reopen in Container"

After some time, I get error messages. They lead to the output tab where errors are displayed.
NuGet.Packaging.Core.PackagingException: Unable to find fallback package folder 'C:\Program Files\dotnet\sdk\NuGetFallbackFolder'.
System Information
Windows 11
Docker
VS code
Checklist

[X] I have completely filled out this template
[X] I have confirmed that this issue exists on the current master branch
[X] I have confirmed that this is not a duplicate issue by searching issues

[X] I have provided detailed steps to reproduce the issue

Hey @alexandre-lecoq! Sorry not sure what's wrong here, we don't give support for local direct installations because many things can go wrong from one host to the other. To get it going python3.11, dotnet 6 & setting the pythonnet env would be enough. Suggest looking at our github actions and dockerfiles for more details.