Edge Extension link not working

The README link to the Edge extension is: https://microsoftedge.microsoft.com/addons/detail/opendyslexic/
However, that gives a 404 error.

This issue also applies to the Firefox extension (https://addons.mozilla.org/en-US/firefox/addon/opendyslexic-for-firefox/). It seems like it has been removed from both the Firefox and Edge extension stores.

Oh! Will get these updated this weekend.

@coliff & @deadnamedimmer
Just waiting on them to approve both. Will close this for now and will update the readme once I get the approval links.

This issue was closed as completed back in July but the extension still isn't on the Edge store at the link on the README. :-(  microsoftedge.microsoft.com/addons/detail/opendyslexic
Any news?

Here you got https://microsoftedge.microsoft.com/addons/detail/opendyslexic-for-edge/foiljfndmiabdijmmagefnejggpfinej
Will update the readme, they changed the url links.
@coliff

Here you got microsoftedge.microsoft.com/addons/detail/opendyslexic-for-edge/foiljfndmiabdijmmagefnejggpfinej
Will update the readme, they changed the url links.
@coliff

thanks! Downloaded it and rated it 5-stars :-)


Make keybindings more flexible

So far, 1..10 was easy to bind programmatically, because it can be done in a for loop.
Allowing additional keys makes the tool more flexible, but also adds complexity.
Ideas:

the switches could allow a tokenized list of hotkeys, say, --switch-desktop-modifiers CTRL,WIN+0%CTRL,WIN+1
it could be possible to specify switches more than once and each would start binding at 0
"NUMBERS" could be used as an alias for 1..10
"FKEY" could be used as an alias for F1..F12

For much later, perhaps a followup issue:

"KP_NUM"  could be used as an alias for KP0 through KP9
"KP_NAV" could be used as an alias for KP_INS through KP_PgUp

These last two are trickier. Windows virtual keycodes don't distinguish keypad directions (e.g. Ins) from a regular Ins. It's probably possible to distinguish them via scancode (https://learn.microsoft.com/en-us/windows/win32/api/winuser/ns-winuser-kbdllhookstruct), but Win32HotkeyListener's architecture doesn't support this at this point. It would have to allow sub-actions for the same hotkey.

Overlaps with #4


Format of Time with SQLAlchemy (and daterangepicker)

Briefly speaking, I 'm try to use the '%M:%S' format of Time like 23:30.
What I have done:

Set the form_args and form_widget_args variable of ModelView like example here https://github.com/flask-admin/flask-admin/blob/master/flask_admin/model/base.py#L572
Make a new AdminModelConverter which override the method convert_time to return TimeField object with formats=('%H:%M:%S', '%M:%S') and default_formatt='%M:%S'.

What I got and need help with:

A edit page with '%M:%S' format, but a list page with '%H:%M:%S' format.
A pop up daterangepicker with hour selector & minute selector instead of minute selector & second selector in the edit page which I want.

If TimeField's formats=(''%M:%S') only, it says 'Invalid time format' all the time when I press save in edit page.
And I found that the widget_format parameter in constructor of TimeField may not be used anymore.
https://github.com/flask-admin/flask-admin/blob/master/flask_admin/form/fields.py#L48

Alright, I figure it out.
Leave my comment here for others who needs it.
I got my CustomModelView like the following:
class CustomModelView(ModelView):
    # changes how the input is parsed by strptime (12 hour time)
    form_args = dict(
        clock_start=dict(default_format='%M:%S', formats=('%H:%M:%S', '%M:%S')),
        clock_end=dict(default_format='%M:%S', formats=('%H:%M:%S', '%M:%S'))
    )
    # changes how the TimeField displays the time
    form_widget_args = dict(
        clock_start={'data-role': u'timepickersecs', 'data-date-format': u'mm:ss'},
        clock_end={'data-role': u'timepickersecs', 'data-date-format': u'mm:ss'}
    )
    column_formatters = dict(
        clock_start=lambda v, c, m, p:m.clock_start.strftime('%M:%S'),
        clock_end=lambda v, c, m, p:m.clock_end.strftime('%M:%S')
    )

And it shows right in both list and edit page.
(The 'data-role' part will be explained next)
I found a timePickerSeconds option here:
https://github.com/flask-admin/flask-admin/blob/master/flask_admin/static/vendor/bootstrap-daterangepicker/daterangepicker.js#L1281
So I add a new data-role 'timepickersecs' copy from here:
https://github.com/flask-admin/flask-admin/blob/master/flask_admin/static/admin/js/form.js#L398
with timePickerSeconds: true, and it's all good


Attributes should be loaded per product

What is the motivation for adding / enhancing this feature?

Currently, on the product page, we're loading all custom defined attributes. We're using attributes for the product options as well. IN some platforms like BigCommerce each product could have different attributes for options defined and this will end up with huge number of unique attributes.
We should be fetching the attributes meta per each product - getting the Object.keys(product) and then pre-fetching these attributes from Elastic.

probably it will also enable configurable options on product tiles

We need to małe sure we’re loading also attributes listed in product.clnfogurable_optoons
The issue mentioned on Slack is caused by this issue probably https://vuestorefront.slack.com/archives/CA68P1JUF/p1546270863075300

This line must be modified
https://github.com/DivanteLtd/vue-storefront/blob/6f187a646527b654bcfe1e79f558e258c2abf2cd/core/modules/catalog/store/product/actions.ts#L609
By adding the fields from configurable_options and/or switching off the “is_visible” “is_userdefined” filters. Maybe there is just an option like “is_configriable” in attribute collection?

The attribute:list probably uses “OR” in query so maybe just extending the list of productFields will be enough - this is to be checked


Repo isnt correctly initialised

I believe there is an issue with this repo as i cant seems to be able to open it with gitkraken to then make proper PR

Fork, commit then rebase then send a pr.


Consider supporting SymbolInformation

A frequently reported issue, which I've run into myself, is the "x does not support documentSymbols" error popping up every time an editor is opened (#2, #5, #7, #15, #22, #36). The standard recommendation is to not attach navic to problematic servers. While this is reasonable in some cases, there are a number of servers that navic would be useful with, like htmlls or cssls, that still use the SymbolInformation interface.
Showing an error every time a new file is opened is informative, but not very ergonomic. Not attaching navic, or silencing errors, means that I won't be able to easily tell when the language servers have updated to using documentSymbol.
Looking at navic's parsing code and the LSP spec, the key difference that navic cares about is that SymbolInformation puts the location on location.range, whereas DocumentSymbol has a top-level range. The other properties used by navic appear to be the same. Given that the SymbolInformation API is still in use by a number of existing servers, and that it's deprecated and therefore shouldn't change, would it be worthwhile to support it? In my (very limited) testing, that might be as simple as pulling the range from the proper location.

In my (very limited) testing, that might be as simple as pulling the range from the proper location.

Its not as easy as just using location.range instead of range. DocumentSymbol format arranges the nodes in a form of tree. Each node has a children field which makes it easy to traverse the tree and extract context easily and quickly. Whereas SymbolInformation is a flat list of all nodes with no information about node hierarchy.

here are a number of servers that navic would be useful with, like htmlls or cssls, that still use the SymbolInformation interface

I have tried looking at VScode's code to try to understand how does it work with htmlls. But due to my very limited knowledge of javascript, I am still clueless about it 😆
If anyone has any good ideas about how to work with SymbolInformation, please do share with me.

I see what you mean. If I had just read a little bit further in the source...
A hierarchy can be heuristically constructed by looking at overlapping symbol ranges; this appears to be what the VSCode outline view did when it was initially added.
The spec points out that symbol ranges can't reliably be used to reconstruct hierarchy, so I can see not wanting to support that because it may lead to a sub-optimal user experience in some cases, but it may still be worth considering.


Volto with shibboleth of SSO

Anyone, Have you used Volto with shibboleth of SSO? I need to use this.
I think very difficult to use shibboleth because Volto has server-side rendering. Do you have any solutions?
Or, Can I disable server-side rendering?
Or, Can Volto set different API URL from browser API call?

@terapyon there's RAZZLE_INTERNAL_API_PATH for the "SSR server needs to use a different URL backend".

@tiberiuichim Thank you for your information. I will try it.

@terapyon we implemented SSO with discourse at some point. I remember that it was all browser-based (redirect to the SSO server, then back with the required info, set it up, configure tokens/cookies etc.


TS can't automatically infer type from outer function in certain cases

🔎 Search Terms
infer function argument bug
🕗 Version & Regression Information

I think this is a bug
I could not find a change between versions

⏯ Playground Link
https://www.typescriptlang.org/play?#code/JYOwLgpgTgZghgYwgAgIILMA9iAPANTgBsBXCAPmQG8AoZZAfShJAApQAHEsALmUNIQAlHwFkA3DQC+NGjBYZsIZAFtQAGQggA5mAAWBYmWQQAHpBAATAM7JrYKKG3IAPsjggAngG0AuuVY6VVA+EBIVACNoGhE0RRxDQUpaeigIMBIoZRT6RmY2Tm4haiDc5GAYZHYQLjAAOiItXT1kXGCQYpyy+n0oLAB3ZBAIQYBRKD6oVgByNRBNHX1poUlu5Bk1tIys8pruVdypABogqUkZGgB6S-5oayVkAEYq-RQcFH7gfXdlaEmhWTyEDxZQcYAcCCPXBBAAq6EwOGeZgsNjiCLw9kcOnIJwCiHRjz4cJBj1i1VqfExTmKAF5KFSdCVUulMspyUVkHT3CS6kwWOywCtpLIEDh7MgAMoIPQQFRwZ405BgiGPVhzBbNViklbIa7IGESux6LAkIiWXYwaDIAAGDO01uQMD6KmQr2QJsgUEdCnRsj1+DuDwATFUvsh+lgoABrWz9GXKfpQOAcCHm0DuV0kDiNAFyH0PZUQIPQ+jEgkmcxaVHeeFKXB28i+XGsfFKQn62uIsmFXh2BzUzn0-uMnJbVmhvaCwfcgneAAMvl5+QFQouopA4qlMrlIcVhaDrG86qa+i1Ql8Or1AAloCgwxHo7Y4LYzBCMBBLEA
💻 Code
interface Action<Value> {
  _run(input: Value): Value;
}

function minLength<Value extends string | any[]>(
  min: number
): Action<Value> {
  return {
    _run(input) {
      if (input.length < min) {
        throw new Error('minLength');
      }
      return input;
    },
  };
}

// Version 1 (the one with an error)

function pipe1<
  TAction1 extends Action<string>,
>(action1: TAction1): (input: string) => string {
  return (input) => action1._run(input);
}

const Schema1 = pipe1(minLength(1)); // TS should infer `string` from the outer function

// Version 2 (it works when wrapped in a tuple)

function pipe2<
  TAction1 extends [Action<string>],
>(action1: TAction1): (input: string) => string {
  return (input) => action1[0]._run(input);
}

const Schema2 = pipe2([minLength(1)]); // Here it works as expected

🙁 Actual behavior
In certain cases, TS will not infer the type of the inner function based on the outer function automatically and will display an error instead. In other cases, such as when the type is wrapped in a tuple, it works. That's why I think the current behavior is inconsistent. Maybe there's a reason for this that I'm not aware of, otherwise it could be a bug.
🙂 Expected behavior
I would expect TS to infer the type and not show an error. TS should have all the information it needs to infer the type automatically.
Additional information about the issue
I discovered this problem while rewriting the source code of Valibot: https://valibot.dev/

Why isn't pipe1 declared this way?
function pipe1(action1: Action<string>): (input: string) => string {
  return (input) => action1._run(input);
}

Because in the actual source code, pipe is a dynamic function that assigns the data type of an action based on the previous action. Without generics, this would require an "infinite" number of overload signatures to support any possible case.

Because in the actual source code, pipe is a dynamic function that assigns the data type of an action based on the previous action. Without generics, this would require an "infinite" number of overload signatures to support any possible case.

duplicate of https://github.com/microsoft/TypeScript/issues/52864
https://github.com/microsoft/TypeScript/pull/52866 tries to address this and awaits further review

@RyanCavanaugh I looked at issue #52864. The problem is similar, but the code is completely different. Therefore, I would not consider this PR a "duplicate". There is also no evidence that PR #52866 actually fixes this issue. Could you re-evaluate it? Unfortunately, this problem blocks the entire rewrite of Valibot.

There is also no evidence that PR https://github.com/microsoft/TypeScript/pull/52866 actually fixes this issue.

I already added the test case from this issue to my PR - roughly at the same time when I commented here that it's a duplicate of that other issue. See the commit with the test: https://github.com/microsoft/TypeScript/pull/52866/commits/71421577603dc230ea63de4ce03781103872a47e

The problem is similar, but the code is completely different.

The root cause really seems to be the same so I think it's fair to classify this as a duplicate of the other issue. The fact that it doesn't work when ur inner call is directly at the argument position of the outer call while it works when it's deeper in the argument is a sign for me that it's the same issue (the other one suffered from exactly the same thing).

Thanks for the clarification and for creating a PR! I was wrong and agree with what you wrote.


Update rag instance creator to allow selection of embedding model

We might need to use a properties.schema or a schema that specifies how a embedding model or chat model can be configured.

Initial implementation in configure-embed-models branch. See fcc51f7f795de399e62cde18d5ac010b9d31302d for changes & current limitations


React 17 support

Can you add react 17 peer support? --legacy-peer-deps seems to work for now
npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^16.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/react-scroll-up-button
npm ERR<EMAIL_ADDRESS>from the root project```

+1 on the work around ... but I'm a bit concerned about doing this since there could be other reasons other than the react version restrictions
I may take a stab at testing my theory though


don't depend on binfmt_misc when used with recent toolchain

closes #118
cc @sfackler
One thing that's missing in this PR is some version check to fall back to the
old binfmt_misc logic when cross is used with a Rust toolchain that doesn't
support the Cargo runner feature.
Alternatively, we could release this as v0.2.0 and require a Rust toolchain
version equal or greater than 1.19.

@homunkulus r+

:pushpin: Commit acc37aa has been approved by japaric

:hourglass: Testing commit acc37aa with merge 3ecee92...

:pushpin: Commit 8a2bcaa has been approved by japaric

@homunkulus r+

:hourglass: Testing commit 8a2bcaa with merge 0f197f7...

:broken_heart: Test failed - status-travis

:pushpin: Commit 0ee902f has been approved by japaric

@homunkulus r+

:hourglass: Testing commit 0ee902f with merge 7daa8f0...

:broken_heart: Test failed - status-travis

Spurious MIPS failure but the runner seems to be working. The rest of tests passed so I'm going to merge manually.


Support ROCm managed memory

Close #5683.
This PR makes ROCm managed memory (HMM) support released in ROCm 4.3.

pfnCI, test this please.

Now ready to review.

Jenkins CI test (for commit 041f2cdf18672a990fb2dd57e12487e20cfc97d2, target branch master) succeeded!

pfnCI, test this please.

Jenkins CI test (for commit 1c4a884e59ea0b49aadb10c401d7e01c19e0ba96, target branch master) failed with status FAILURE.

pfnCI, test this please.

Jenkins CI test (for commit cee375b0b2ce1ed0462142676fcf2fe8ae0e751d, target branch master) succeeded!

pfnCI, test this please.

Jenkins CI test (for commit aac5aeca2cfac4f1547b46b5eb23c1bbee516a53, target branch master) failed with status FAILURE.


JS files ending inside single line comment cause code loss

What steps will reproduce the problem?
1. Combine two Javascript files
2. End file1 with a single line comment (no newline after)
3. Include no whitespace before code in file2

* What is the expected output?
The trailing comment in file1 is removed

* What do you see instead?
The first line of file2 is removed because Minify thought it was part of
the comment in file1

(Found by Bob Spryn)

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 6 Jan 2009 at 4:09

Fixed in R278

Original comment by<EMAIL_ADDRESS>on 6 Jan 2009 at 4:46

Changed state: Verified
Added labels: Release-2.1.1

Issue 88 has been merged into this issue.

Original comment by<EMAIL_ADDRESS>on 23 Feb 2009 at 1:46


Log to debug to make log.info production friendly

Fixes #77

@meteor-space/core This is an easy accept/merge so will leave it unassigned.


test: adding dumping of generated material for tests in attestation.jar

Added functionality to the test suite of attestation.jar to generate files with test keys and attestations, etc. which can be used for testing Authenticator.
See issue #282
This closes issue #280

I am merging this without review since it only concerns additional testing and does not touch upon functionality.

This closes #283


multi_transform is inconsistent with a custom optimizer

Hi! New to optax.
I wanted to implement the extra-gradient method (see e.g. here https://arxiv.org/abs/1901.08511v2), which is described mathematically by
$x_{k+1/2} = x_{k}-\eta \nabla f(x_k),  \quad x_{k+1} = x_{k}-\eta \nabla f(x_{k+1/2})$.
I'm not sure how to properly account for the midpoint step, here's how I did it.
    def extra_gradient_update(grads, params):

        # Midpoint
        mid_updates = jax.tree.map(lambda g: - learning_rate * g, grads)
        mid_params = optax.apply_updates(params, mid_updates)

        # Grads at midpoint
        mid_grads = jax.grad(func)(mid_params)

        # Final updates
        updates = jax.tree.map(lambda g: - learning_rate * g, mid_grads)

        return updates

The optimizer based on that update function works fine, but for some reason fails as a part of optax.multi_transform. Here is the full example attempting to performing a single update step for a function $f = x y$.
import jax
import jax.numpy as jnp
import optax

def extra_gradient_optimizer(func: callable, learning_rate: float):

    def extra_gradient_update(grads, params):

        # Midpoint
        mid_updates = jax.tree.map(lambda g: - learning_rate * g, grads)
        mid_params = optax.apply_updates(params, mid_updates)

        # Grads at midpoint
        mid_grads = jax.grad(func)(mid_params)

        # Final updates
        updates = jax.tree.map(lambda g: - learning_rate * g, mid_grads)

        return updates

    return optax.stateless(extra_gradient_update)

def f(params):
    return params['x'] * params['y']

params = {
    'x': jnp.array(1.0),
    'y': jnp.array(2.0)
}

opt = optax.multi_transform(
    {'xopt': extra_gradient_optimizer(f, 0.01),
     'yopt': extra_gradient_optimizer(f, -0.01)},
    {'x': 'xopt',
     'y': 'yopt'})

state = opt.init(params)
grads = jax.grad(f)(params)
opt.update(grads, state, params)

This results in an error that is traced back to the computation of $f$ itself TypeError: Only integer scalar arrays can be converted to a scalar index. If instead of the multi_transform I simply use opt=extra_gradient_optimizer(f, 0.01) the update works fine.
Is this a bug, or I'm not doing this the right way?

Hello @idnm,
GradientTransformations are generally not well suited to include the computation of the gradient inside them (as the name suggests it's a transformation of gradients not an optimization oracle).
So here you can

create a multitransform optimizer as you did
chain it with a transform that keeps a step counter and a copy of the params. When the counter is e.g. odd the function makes a step using the previously stored params.

@vroulet Understood! And thanks for the suggestion, using the parameters from one step back seems to do the trick.

Great, happy if it helped!


[Documentation] Programming Model, Kernel Programming guide

Adds new material to the kernel programming guide and the programming model sections.

@adarshyoga can you please give the Kernel programming guide a read once more. I have updated the all the section except the Advanced Concepts. I will fill in the Advanced Concepts in a separate PR.


Table not exported in anchor_trans_fit.html

In data/Britain_Cmplx/anchor_trans_fit.html there is a table which isn't being exported. It's in a div that has a note above it saying:

this is an example of a div of legitimate content that doesn't have an id, or fit into the BottomLayer/PageLayer model we see elsewhere

Do you know how often these occur? At the moment, we're excluding from processing any divs that don't have an ID. We could change that, but it may lead to selecting a load of content that we don't want selecting. If these only occur a few times then it might be easier to just give them an ID, or even wrap them in a BottomLayer/PageLayer.
What do you think?

That is quite an unusual type of file.  If the checker flags it up, I'm happy to fix it by hand.

Brilliant, I was hoping you'd say that!

Ok, there do appear to be just two occasions where this happens.  I'll fix them by hand, by wrapping with some BottomLayer goodness.  Then I'll fix this/these files to match the structure of the real content - and we can validate that processing.

Done.


Comfyui Import Failed

After installing PuLID from Comgyui manager and rerunning, I get the followig error:
0.0 seconds (IMPORT FAILED): C:\ComfyUI_cu121\ComfyUI\custom_nodes\PuLID_ComfyUI
Any idea what happened?

check this https://github.com/cubiq/PuLID_ComfyUI/issues/1


proposal: disallow duplicate entries

I'd suggest to disallow duplicate entries (within a certain timeframe) or rate limit post functionality heavily to make spamming more difficult.

Yep, I think this would be great to have.

One post every 6h should be enough for a single user in my opinion.

I think disallowing duplicate entries is sufficient. Let's see how we go just doing that first 👍🏾


RPC endpoint response Content-Type

RPC endpoint response Content-Type
Description
When I send a request to mainnet RPC, it responds with incorrect Content-Type header. It responds with "text/plain" instead of "application/json".
Expected behaviour
Response Content-Type header should be "application/json"
Actual behaviour
Response Content-Type header currently is "text/plain"
Mainnet

Testnet

Thanks for the head up.
We'll fix it ASAP.

thanks, it seems to be fixed already after the recent update, but a new issue with the endpoints popped up. I've created a new issue for that - #255


Using dependency variables in custom condition variable expression

Problem description
After setting a variable in a Powershell step using the following code:
Write-Host "##vso[task.setvariable variable=$(VariableName);isOutput=true]$(VariableValue)"

I'm trying to retrieve that variable in a subsequent job for use as a custom condition variable expression:
eq(dependencies.AgentJob1.outputs['SetVariable.ReadyToDeploy'], 'true')

Unfortunately I receive the following error when the agent attempts to start that job:
Exception Message: Unrecognized value: 'dependencies'. Located at position 4 within expression: eq(dependencies.AgentJob1.outputs['SetVariable.ReadyToDeploy'], 'true'). For more help, refer to https://go.microsoft.com/fwlink/?linkid=842996 (type ParseException)
Exception Stack Trace:    at Microsoft.TeamFoundation.DistributedTask.Expressions.ExpressionParser.HandleUnknownKeyword(ParseContext context) in d:\v2.0\P1\_work\7\s\DistributedTask\Client\WebApi\Expressions\ExpressionParser.cs:line 278
   at Microsoft.TeamFoundation.DistributedTask.Expressions.ExpressionParser.CreateTree(ParseContext context) in d:\v2.0\P1\_work\7\s\DistributedTask\Client\WebApi\Expressions\ExpressionParser.cs:line 85
   at Microsoft.TeamFoundation.DistributedTask.Expressions.ExpressionParser.CreateTree(String expression, ITraceWriter trace, IEnumerable`1 namedValues, IEnumerable`1 functions) in d:\v2.0\P1\_work\7\s\DistributedTask\Client\WebApi\Expressions\ExpressionParser.cs:line 18
   at Microsoft.VisualStudio.Services.ReleaseManagement.Server.Processors.PhaseConditionEvaluator.ShouldExecutePhase(IVssRequestContext requestContext, ReleaseEnvironment releaseEnvironment, DeployPhaseSnapshot deployPhaseSnapshot, AutomationEngineInput automationEngineInput) in d:\v2.0\P1\_work\7\s\ReleaseManagement\Service\ReleaseManagement2\Server\Processors\PhaseConditionEvaluator.cs:line 26
   at Microsoft.VisualStudio.Services.ReleaseManagement.Server.Processors.DeployPhaseRunner.Run(Release release, ReleaseEnvironment releaseEnvironment, ReleaseEnvironmentStep deployStep, DeployPhaseSnapshot snapshotToProcess) in d:\v2.0\P1\_work\7\s\ReleaseManagement\Service\ReleaseManagement2\Server\Processors\DeployPhaseRunner.cs:line 208

It appears from the docs that this available for YAML, and I had hoped it would also be supported in the GUI. Is this not the case or is my syntax just wrong?
Screenshots for context

The "SetVariable" task group is made up of the aforementioned powershell script to set the variable:

+1

Hi guys, any updates on this?

Any updates on this? im facing similar issue :)

Any updates?

Same issue over here, getting the same error:
Unrecognized value: 'dependencies'. Located at position 4 within expression
Any progress?

Same issue

Not supported.

Not supported.

Hi @vtbassmatt, what do you mean by "Not supported"?

Designer-based builds don't accept output variables from other jobs. That's a feature only in YAML.

It doesn't work in YAML, either - and yet the documents dated 8/2020 still show it.   I'd recommend this be re-opened.
##[error]Unrecognized value: 'dependencies'. Located at position 10 within expression: contains(dependencies.

@jason-e-gross Sorry for the confusion - cross-stage dependencies need a different syntax depending on whether you're consuming them at stage level or job level.
I see that this topic still has some outdated info, which I'll get corrected. This doc is up to date with respect to the various syntaxes involved.
Stage-level
stages:
- stage: A
  condition: false
  jobs:
  - job: A1
    steps:
    - script: echo Job A1
- stage: B
  condition: in(dependencies.A.result, 'Succeeded', 'SucceededWithIssues', 'Skipped')
  jobs:
  - job: B1
    steps:
    - script: echo Job B1

Job-level consumption
stages:
- stage: A
  jobs:
  - job: A1
    steps:
     - bash: echo "##vso[task.setvariable variable=shouldrun;isOutput=true]true"
     # or on Windows:
     # - script: echo ##vso[task.setvariable variable=shouldrun;isOutput=true]true
       name: printvar

- stage: B
  dependsOn: A
  jobs:
  - job: B1
    condition: in(stageDependencies.A.A1.result, 'Succeeded', 'SucceededWithIssues', 'Skipped')
    steps:
    - script: echo hello from Job B1
  - job: B2
    condition: eq(stageDependencies.A.A1.outputs['printvar.shouldrun'], 'true')
    steps:
     - script: echo hello from Job B2

@vtbassmatt - I'd suggest some clarification here on both this issue the error message that's shown is not at all helpful, and I'm not clear on how your comment resolves this for @jason-e-gross - I am getting a similar error when expanding conditions set as parameters in a template, but the syntax matches your example dependencies.<jobname>.outputs['stepname.outputvar']

@vtbassmatt - I'd suggest some clarification here on both this issue the error message that's shown is not at all helpful, and I'm not clear on how your comment resolves this for @jason-e-gross - I am getting a similar error when expanding conditions set as parameters in a template, but the syntax matches your example dependencies.<jobname>.outputs['stepname.outputvar']

I'm struggling with a similar issue. I wanted to combine if syntax with checking for the output of a previous job, but this doesn't seem to be supported unfortunately:
variables:
  - name: my_var
  ${{ if eq(dependencies.my_job.result, 'Succeeded') }}:
    value: true
  ${{ if in(dependencies.my_job, 'Skipped', 'Failed', 'Canceled') }}:
    value: false

This gives me Unrecognized value: 'dependencies'. Located at position 4 within expression.
I ended up with duplicating the job and its code except for changing the desired variable value.

eq(dependencies.xxtaskname.result, 'Succeeded') has the same error

any solution?
do releases have dependencies?
I have seen that this works well in the pipeline but not in the releases.

Seems that dependencies must be applied at a job condition and not at lower level (task). I noticed documentation only features samples where dependencies are used within condition only under stage or job https://learn.microsoft.com/en-us/azure/devops/pipelines/process/expressions?view=azure-devops
I'v tried apply it to a task condition and it failed. At job it doesn't fail (DevOps 2020 Server)
So this works:
  - job: PostBuild
    dependsOn: Build
    condition: |
      and(
        succeededOrFailed(),
        and(
          eq(dependencies.Build.outputs['npmfinished.yes'], 'true'),
          eq(dependencies.Build.outputs['restoreNpmCache.cacheRestored'], 'false')
        )
      )
    steps:
      - checkout: none
      - task: PowerShell@2
        ...

And this doesn't and gives said error:
  - job: PostBuild
    dependsOn: Build
    steps:
      - checkout: none
      # Saves cache even if Build failed, but npm ci step succeeded (cacheRestored variable successfully set.).
      - task: PowerShell@2
        condition: |
          and(
            succeededOrFailed(),
            and(
              eq(dependencies.Build.outputs['npmfinished.yes'], 'true'),
              eq(dependencies.Build.outputs['restoreNpmCache.cacheRestored'], 'false')
            )
          )
          ...

Seems that dependencies must be applied at a job condition and not at lower level (task). I noticed documentation only features samples where dependencies are used within condition only under stage or job https://learn.microsoft.com/en-us/azure/devops/pipelines/process/expressions?view=azure-devops
I'v tried apply it to a task condition and it failed. At job it doesn't fail (DevOps 2020 Server)
So this works:
  - job: PostBuild
    dependsOn: Build
    condition: |
      and(
        succeededOrFailed(),
        and(
          eq(dependencies.Build.outputs['npmfinished.yes'], 'true'),
          eq(dependencies.Build.outputs['restoreNpmCache.cacheRestored'], 'false')
        )
      )
    steps:
      - checkout: none
      - task: PowerShell@2
        ...

And this doesn't and gives said error:
  - job: PostBuild
    dependsOn: Build
    steps:
      - checkout: none
      # Saves cache even if Build failed, but npm ci step succeeded (cacheRestored variable successfully set.).
      - task: PowerShell@2
        condition: |
          and(
            succeededOrFailed(),
            and(
              eq(dependencies.Build.outputs['npmfinished.yes'], 'true'),
              eq(dependencies.Build.outputs['restoreNpmCache.cacheRestored'], 'false')
            )
          )
          ...

Workaround is to set the value as a variable:
- job: PostBuild
  dependsOn: Build
  variables:
    npmFinished: $[dependencies.Build.outputs['npmfinished.yes']]
    cacheRestored: $[dependencies.Build.outputs['restoreNpmCache.cacheRestored']]
  steps:
    - checkout: none
    # Saves cache even if Build failed, but npm ci step succeeded (cacheRestored variable successfully set.).
    - task: PowerShell@2
      condition: |
        and(
          succeededOrFailed(),
          and(
            eq(variables.npmFinished, 'true'),
            eq(variables.cacheRestored, 'false')
          )
        )
        ...

This feels like a bug and I think the ticket should be reopened.

Seems that dependencies must be applied at a job condition and not at lower level (task). I noticed documentation only features samples where dependencies are used within condition only under stage or job https://learn.microsoft.com/en-us/azure/devops/pipelines/process/expressions?view=azure-devops
I'v tried apply it to a task condition and it failed. At job it doesn't fail (DevOps 2020 Server)
So this works:
  - job: PostBuild
    dependsOn: Build
    condition: |
      and(
        succeededOrFailed(),
        and(
          eq(dependencies.Build.outputs['npmfinished.yes'], 'true'),
          eq(dependencies.Build.outputs['restoreNpmCache.cacheRestored'], 'false')
        )
      )
    steps:
      - checkout: none
      - task: PowerShell@2
        ...

And this doesn't and gives said error:
  - job: PostBuild
    dependsOn: Build
    steps:
      - checkout: none
      # Saves cache even if Build failed, but npm ci step succeeded (cacheRestored variable successfully set.).
      - task: PowerShell@2
        condition: |
          and(
            succeededOrFailed(),
            and(
              eq(dependencies.Build.outputs['npmfinished.yes'], 'true'),
              eq(dependencies.Build.outputs['restoreNpmCache.cacheRestored'], 'false')
            )
          )
          ...

Workaround is to set the value as a variable:
- job: PostBuild
  dependsOn: Build
  variables:
    npmFinished: $[dependencies.Build.outputs['npmfinished.yes']]
    cacheRestored: $[dependencies.Build.outputs['restoreNpmCache.cacheRestored']]
  steps:
    - checkout: none
    # Saves cache even if Build failed, but npm ci step succeeded (cacheRestored variable successfully set.).
    - task: PowerShell@2
      condition: |
        and(
          succeededOrFailed(),
          and(
            eq(variables.npmFinished, 'true'),
            eq(variables.cacheRestored, 'false')
          )
        )
        ...

This feels like a bug and I think the ticket should be reopened.

Y'know, I'd be really interested in these steps you have for caching the npm packages even on failed builds, if it's something you're willing to share.

Y'know, I'd be really interested in these steps you have for caching the npm packages even on failed builds, if it's something you're willing to share.

Azure Devops Online has this working task, which is not working on self hosted on-premises server: Pipeline Caching - Node.js/npm
For On-Premises I do it like this 🤯
jobs:
  - job: Build
  ...
      # Cache the npm cache folder https://learn.microsoft.com/en-us/azure/devops/pipelines/release/caching?view=azure-devops#nodejsnpm
    # Because npm ci deletes the node_modules folder to ensure that a consistent, repeatable set of modules is used, you should avoid caching node_modules when calling npm ci. I do use npm install - faster.
    # Cache@2 task doesn't work on prem
    - task: PowerShell@2
      name: restoreNpmCache
      displayName: Restore npm cache
      env:
        Agent_BuildDirectory: $(Agent.BuildDirectory)
      inputs:
        filePath: '$(System.DefaultWorkingDirectory)\Pipeline\RestoreFromCache.ps1'
        arguments: '-KeyFile "$(WebResourcesPath)\package-lock.json" -TargetFolder "$(npm_config_cache)"'
        errorActionPreference: 'continue'
        workingDirectory: '$(System.DefaultWorkingDirectory)'

    - task: PowerShell@2
      name: restoreNodeModulesCache
      displayName: Restore node_modules cache
      env:
        Agent_BuildDirectory: $(Agent.BuildDirectory)
      inputs:
        filePath: '$(System.DefaultWorkingDirectory)\Pipeline\RestoreFromCache.ps1'
        arguments: '-KeyFile "$(WebResourcesPath)\package-lock.json" -TargetFolder "$(WebResourcesPath)\node_modules"'
        errorActionPreference: 'continue'
        workingDirectory: '$(System.DefaultWorkingDirectory)'

    - task: Npm@1
      name: npm
      inputs:
        command: 'install'
        workingDir: $(WebResourcesPath)

    - task: PowerShell@2
      name: npmfinished
      inputs:
        targetType: 'inline'
        script: Write-Output "##vso[task.setvariable variable=yes;isOutput=true]true"
  ...
  - job: PostBuild
    dependsOn: Build
    condition: |
      and(
        succeededOrFailed(),
        and(
          eq(dependencies.Build.outputs['npmfinished.yes'], 'true'),
          eq(dependencies.Build.outputs['restoreNpmCache.cacheRestored'], 'false')
        )
      )
    steps:
      - checkout: none
      # Saves cache even if Build failed, but npm ci step succeeded (cacheRestored variable successfully set.).
      - task: PowerShell@2
        env:
          Agent_BuildDirectory: $(Agent.BuildDirectory)
        displayName: Cache npm
        inputs:
          filePath: '$(System.DefaultWorkingDirectory)\Pipeline\SaveToCache.ps1'
          arguments: '-KeyFile "$(WebResourcesPath)\package-lock.json" -TargetFolder "$(npm_config_cache)"'
          errorActionPreference: 'continue'
          workingDirectory: '$(System.DefaultWorkingDirectory)'

      - task: PowerShell@2
        env:
          Agent_BuildDirectory: $(Agent.BuildDirectory)
        displayName: Cache node_modules
        inputs:
          filePath: '$(System.DefaultWorkingDirectory)\Pipeline\SaveToCache.ps1'
          arguments: '-KeyFile "$(WebResourcesPath)\package-lock.json" -TargetFolder "$(WebResourcesPath)\node_modules"'
          errorActionPreference: 'continue'
          workingDirectory: '$(System.DefaultWorkingDirectory)'

SaveToCache.ps1:
param(
    [string]$KeyFile,
    [string]$TargetFolder,
    [string]$CacheRoot = "$env:Agent_BuildDirectory\azCache"
)

$CachePath = Join-Path $CacheRoot "$([System.IO.Path]::GetFileName($TargetFolder))/$((Get-FileHash $KeyFile -Algorithm SHA256).Hash)"

if (Test-Path $CachePath) {
    Write-Output "`"$CachePath`" exists - cache save not needed"
} else {
    Write-Output "Move-Item `"$TargetFolder\*`" -Destination `"$CachePath`""
    New-Item -ItemType Directory -Force -Path $CachePath
    Move-Item "$TargetFolder\*" -Destination $CachePath -Force
    Remove-Item "$TargetFolder"
}

RestoreFromCache.ps1
param(
    [string]$KeyFile,
    [string]$TargetFolder,
    [string]$CacheRoot = "$env:Agent_BuildDirectory\azCache"
)

$CachePath = Join-Path $CacheRoot "$([System.IO.Path]::GetFileName($TargetFolder))/$((Get-FileHash $KeyFile -Algorithm SHA256).Hash)"

#Cache does not exist for this key
if (!(Test-Path $CachePath -PathType Container)) {
    Write-Output("Cache Miss: `"$CachePath`"")

    #Set Pipeline Varaible
    Write-Host "##vso[task.setvariable variable=cacheRestored;isOutput=true]false"
} else{
   Write-Output("Cache Hit. `"$CachePath`"")

   If(!(test-path $TargetFolder)){
       #Create Symbolic link to cache
       New-Item -ItemType SymbolicLink -Path "$TargetFolder" -Target "$CachePath\$target_folder" -Force
   }

    #Set Pipeline Varaible
    Write-Host "##vso[task.setvariable variable=cacheRestored;isOutput=true]true"
}

I found out that what could matter is using ""
condition: ne(dependencies.SettingVariablesLegacy.outputs['SetPOCustSecretVariables.PO_EnvironmentNamesJson_0'], '{}')

returns Unrecognized value: 'dependencies'
condition: and(succeeded(), ne(dependencies.SettingVariablesLegacy.outputs['SetPOCustSecretVariables.PO_EnvironmentNamesJson_0'], '{}'))

works just fine


Fix for delayed archive extraction issue

@alanhoff, here's hoping this one finally gets through (fix for issue #20).

LGTM


Fix function comments based on best practices from Effective Go

Signed-off-by: CodeLingo Bot<EMAIL_ADDRESS>

Every exported function in a program should have a doc comment. The first sentence should be a summary that starts with the name being declared.
From effective go.
I generated this with CodeLingo and I'm keen to get some feedback, but this is automated so feel free to close it and just say "opt out" to opt out of future CodeLingo outreach PRs.

I have triple checked this PR to make sure the bot has done its job of correctly fixing incorrect doc comments without introducing any false positives.
If this service is useful to you, please let me know :smile:
Note: I've only updated functions that I know CodeLingo can rewrite nicely, but you can install it from the GitHub marketplace to review your PRs automatically and find all issues.

Thanks!


[ST] User sent NEXT to the token contract

Describe the bug
A user mistakenly sent NEXT tokens to the NEXT contract address on Arbitrum https://arbiscan.io/tx/0x9b4cad6adff959d2d9e38de07ed8d06d7158de9fa98046b563c283e6cb7d4108
Is it possible to sweep them from the contract and send it back?
Link to ticket in discord
https://discord.com/channels/454734546869551114/1182072006330556528/1182073513692115026

There is no sweep functionality on the NEXT erc20 token contract. We'd need the security council to sign a multisig transaction to return this.

not acceptable to use the security council this way :(


Problem with low res monitors

Describe your bug here.
Basically, i'm in this old AOC monitor, then, the size is wrong then i cant see the score, Accuracy, Rating, Right Note. (except for healthbar) This happens everytime! Please fix this bug, Thank you.
~ Reflexx#1232

This is proof, More size in the file to make the proof.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No, I don't know anything about programming. It's just a mod that is in gamebanana.

Haxe bug.


alias links to log files appear to be wrong

The refgenie alias dir uses symlinks to point to the data dir. These links are correct on our main rivanna genomes directory:

But when I build for a new server, I see this:

The links point to the wrong level, I think they need 5 ../ levels rather than 4.

did you try running some tests again? The assets I built for rg.databio.org a couple of days ago have functional symlinks: $PROJECT/deploy/rg.databio.org/genomes/alias/hg38/fasta/default.
Is it possible that this is somehow specific to your setup?

ah ok, I see the issue. It's about files in _refgenie_build only


Add ability to switch to manual mode for only brightness

First to help understand my configuration, I have set up an adaptive-lighting entity for each light fixture.  This way I can set one fixture to manual while leaving the others alone.
During the day, I like to have the color temperature change the whole day.  Sometimes I like to decrease the brightness level (especially a lamp near my desk).  I still want the color temperature to change.
So the ability to switch brightness control to manual, while leaving the rest of the system running would be helpful.
I'm thinking something along the lines of
if brightness is changed:
if brightness_only_switch:
set adaptive_lighting_brightness "off"
else:
set adaptive_lighting_to_manual: "on"
With the idea being if I manually change the color value it'll turn the whole thing off, while if it's only the brightness I changed it'll only turn off the brightness control.

Hey @gderber,
thanks for your input, I think we have this already on our todo list:
https://github.com/basnijholt/adaptive-lighting/issues/132
If you agree I would like to mark this as a duplicate. If not, please elaborate why this won't cover your usecase :)

It might be covered by #132, but I am not certain.
I am not sure how simply turning off the brightness adjustment if I manually adjust the brightness while still leaving the color temperature adjustment active relates to multiple light "modes"

I would like this feature as well since we often turn down the brightness while watching a movie but I'd still like the color temp to be adjusted.  I'm not sure that #132 covers this either.

It be nice to have the flip where you can set the colors to whatever you like but have you stuff so that the brightens is that only thing that changes not the light tone or color.

Well that the thing some lights don't play nice with that at all. like the Govee ones that you can pick up very easily at Lowes stores or even online and they do a great job but they don't play nice with HA as someone had to come up with a 3rd party way to get the lights and other things to they make to be able to be used without the need for Bluetooth that is if your setup would even let you use Bluetooth to begin with. So all I'm asking is for a way to be able to not have to set any color as a thing for those of us that can't go whole hog with are setups.

Hey @michaelcrossland1979,
I don't think I can follow your setup and train of thoughts here fully...
Would you mind creating a new ticket for this?
The exact type devices itself doesn't matter much, it's just important how HA "sees" it.
So just download the diagnostic data for the device and add it to the new ticket:

Well I'll just have to find some other way to deal with it my self. Because in the case of the 2 lights that I don't want it to change colors on those are outside lights and if left with the warmer colors you can guess what happens not to long after sun goes down all the bugs come right for those lights unless they are left in a very dark red color. But otherwise have no problems with it for the  main house lights.

See there is a very good use case for a advacted setting like that where the user can disable the color part but leave the dimming part alone. That or say when my plex server gets done playing something and I would just like my lights that where dimmed using webhoocks to stay that nice deep red but to just come back up to the bightens of what the rest are set at. Because don't know about you but have you sat in almost dark for 2 or 3 hours with just the TV and some low lights running at dark red and then have pop right up to a warm whitgh light that will blind you? I know I have and it's no fun what so ever.


Translate stakedLp contract to Java

ICON will only accept new contracts in Java going forward so we have to rewrite the stakedLp contract in order to deploy it.

Waiting for continuous rewards to go live.


SPARKC 492: Protect against Size Estimate Overflows

val tokenRangeSizeInBytes = (totalDataSizeInBytes /
ringFraction).toLong

In datasize estimates has the possibility of overflowing when
getting the size_estimates from a datacenter whose primary range
contains a very tiny range BUT still also contains data. In these
cases a tiny size in bytes will be divided by a miniscule range
fraction leading to a huge value. This when converted toLong becomes
Long.MaxValue.
When these situations arise our size_estimates are no longer
reliable so we need to fall back to a safer estimate. In 2.0 we can
use the min Spark Nodes * 2 + 1 but in 1.6 we don't have that
information. To keep our changes small, here we will just use the
endpoints present in the target datacenter * 2 + 1.

Build against Scala 2.10 finished with success

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with success

Build against Scala 2.11 finished with success

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with failure

Build against Scala 2.10 finished with failure


[T4A3][T15-B3]Bartłomiej Woźniak

Extract commonalities from Phone, Email and Address classes into a parent class called Contact.

Ready for review

@bartekwozniak93 Good work overall. Some minor issues.


Don’t modify Array.prototype

I ran into an issue trying to use https://github.com/rtfeldman/seamless-immutable (https://github.com/rtfeldman/seamless-immutable/issues/39) as you are extending Array.prototype.
This removes the modification to Array.prototype.

Thanks @wvl - This was fixed with 9b82b5624c59948e6c9648d4253f15051e751fcd

Even better, thanks @hai-cea


node16 actions are deprecated

This needs to be moved to a newer engine version.

Oh. People use this action?
Will do sometime this weekend

pushed to v3.2
Will override v3 if v3.2 works well for you

Ha, maybe I'm on the only user? Is there a better choice these days? When I added it, it seemed to clearly be the best choice to solve a problem for self-hosted runners.

I use it for some repos still. But then I moved to https://github.com/kevincobain2000/action-coveritup
A slightly different metric to calculate and track real time taken. You might like it.


Live server information on the admin panel

Is your feature request related to a problem? Please describe.
This isn't really related to a problem, it's more so that you can just get some live information about the FiveM server(s) directly on the admin panel.
Describe the solution you'd like
I'd like to see a new page called 'Servers' where you can view some information such as framework version, uptime, players and more. Being able to see who's currently online would also be great, and perhaps having a redirect to their user profile on the admin panel.
Additional context
This can be achieved by using some of the routes that I have created for the framework. Currently there are only two that would be relevant to this, but I'd be open to add far more for whatever you'd want and/or need.
The two (GET) routes are:
/api.json
/connections.json
You can reach them by going to this 'example' URL.
Some clusters have multiple servers, though, so if this were to be implemented, it would be great to have a way of viewing various servers that have been added to the admin panel.

You can now add the servers (see GIF). Will see if I can get a PR in soon to also view the online players.

This page now also shows all online players and allows deleting (removing from tracking) servers. I think this issue can be closed @ExpDev07


Bumped MailChimp API to v3 as v2 is no longer supported by MailChimp.

MailChimp dropped support for v2 API last year, and it's no longer documented, and prone to issues. This PR changes the request for account details from v2 to v3.
This ensures better future support and also fixes a few permissions related issues authorizing non-Owner/Admin accounts using the strategy.

Thanks for merging this in, really appreciate it. 👍


Download page: remove link to laanwj's key; mention --refresh-keys

Recently, @laanwj's release signing key expired.  He has uploaded to key servers an updated key with a future expiration date, but the copy of the key hosted on this server is still the expired version.  Instead of updating it, this drops the link to it on the Download page and also adds a tiny note about gpg --refresh-keys  Preview (bottom text):

The verification instructions on the page all mention getting the key with --recv-keys and there are no other links to the key on this site, so I think this is an acceptable solution.  However, this PR leaves the actual expired key on the site in case any third party scripts are grabbing it automatically and just ignoring its expired status.

ACK


WIP

Summary

Add support for memory to an Assistant without leveraging LangChain.
Checklist

[x] My code follows the code style of this project and passes make format.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate unit test coverage.

@homanp Thank you for the pull request! Overall, the changes look good. I have a few suggestions:

In the superagent.py file, you have removed the _get_prompt method. Is this intentional? If not, please add the implementation for this method.

In the superagent.py file, you have removed the _run_completion and arun methods. Is this intentional? If not, please add the implementation for these methods.

In the completion/base.py file, you have removed the prompt attribute from the Completion class. Is this intentional? If not, please add the prompt attribute back.

In the completion/base.py file, you have removed the prompt parameter from the get_agent method. Is this intentional? If not, please add the prompt parameter back.

In the memory/base.py file, you have removed the inputs and outputs parameters from the save_context method. Is this intentional? If not, please add the inputs and outputs parameters back.

In the tools/chitchat.py file, you have removed the memory parameter from the ChitChatTool class. Is this intentional? If not, please add the memory parameter back.

In the tools/prompts.py file, you have added a new create_memory_prompt function. Could you provide more context on how this function is used and what it does?

Let me know if you have any questions or need further clarification on any of the suggestions.


[Exposure Request] codeunit 550 "VAT Rate Change Conversion" - InitVATRateChangeLogEntry

Please make function InitVATRateChangeLogEntry in codeunit 550 "VAT Rate Change Conversion" externally available.
procedure InitVATRateChangeLogEntryFromConfigTemplateLine(var VATRateChangeLogEntry: Record "VAT Rate Change Log Entry"; ConfigTemplateLine: Record "Config. Template Line")

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: .


Real time updates on parameter optimization

If BaSiC.fit() could provide a way to peak at its parameters throughout optimization, we can do some cool stuff in the napari plugin. For example, there could be a preview layer that is updated with the flatfield estimate every n iterations. This would probably slow down the computation but would provide a cool way of watching the process unfold. We could also use the parameters to plot error, etc.
napari provides a thread_worker decorator to simplify adding plugin functions as to not block using the gui. The worker can yield values during its call, which can be passed onto other functions.
It is generally bad design to change output type of a function based on argument, but ignoring that, here is one way this could be implemented.
class BaSiC:
  ...
  def fit(self, images, ..., updates=False):
    ...
    while not CONVERGED:
      ...
      if updates:
        yield self.params
      ...

# napari plugin

def update_layer(image):
  # update preview layer
  ...

@thread_worker(start_thread=False, connect={"yielded": update_layer})
def call_basic(images):
  basic = BaSiC()
  for update in basic.fit(images, updates=True):
    yield update
  ...

This is a good idea and implementation.

Unfortunately, I believe this is not compatible with jit-compiled functions. Closing this for now but please feel free to re-open.


Fixed the Game ending bug in Tic Tac Toe by Updating main.js

issue #32 is resolved

This issue has already been resolved.


Consider replacing iOS WKWebView with react-native-wbkwebview

It doesn't seem like a good idea to redo all the work that has already been done here:
https://github.com/CRAlpha/react-native-wkwebview
We should probably get in touch with the author and see if they want to merge their efforts into this repository (or provide it as a submodule).
They have already implemented some of the changes I proposed in another PR (notably removing window.postMessage integration) as well: https://github.com/CRAlpha/react-native-wkwebview/commit/29588d4144940864126c689c4c400eb215f72adf
PR: https://github.com/react-native-community/react-native-webview/pull/104
@insraq @ruoyu-unity

We already got in touch with them:
https://github.com/CRAlpha/react-native-wkwebview/issues/191
I think it is also good to note this:
https://facebook.github.io/react-native/blog/2018/08/27/wkwebview
I think it's better to keep the whole code in one place. This package has been designed with the community participating in mind.
However, I would not want to force everyone to use a new version of the WKWebview, as it is supposed to be react-native's webview extracted on which we will iterate, and probably take back some parts from all the existing repository.
As a quick example, the solution to the bridge CRAlpha/react-native-wkwebview@29588d4 is not necessarly what we would implement over here.
I'm open to discuss this ! 😄

@Titozzz good to see! I didn't catch that.
I agree it would ideally be kept in one place, and it seems the author of the old package is willing to make that happen, so that's great!
With regard to the new iOS WebView; I assumed this repository would already be up to date with the new WKWebView from React Native?
It definitely seems sensible to extract the WKWebView from React Native first and then apply a selection of the fixes from react-native-wkwebview on top of that; I just noticed that some issues in this repository could already be fixed by changes made in that package, so thought I'd open up a wider discussion.
Maybe we can include a note about checking pre-existing WebView packages for fixes to issues in the issue reporting guidelines (which aren't setup yet)?

Okay sorry I was not clear, webview HAS BEEN extracted! Nothing will probably be merged on react-native side. (or would need to be it critical and would get pulled here too). I like the idea of adding the comment to check other packages to our non existing template (we really need one) ❤ .
I'll keep this issue opened as a reminder !

:tada: This issue has been resolved in version 4.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 5.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Hello. I'm using react-native-wkwebview in my project and once thing because I can't replace it with react-native-webview is not possible to share cookies between fetch/react-native-cookies and react-native-webview (look at sendCookies property). Was it done? Probably I do something wrong?

@indapublic We are working on it in https://github.com/react-native-community/react-native-webview/pull/175 -- once we release that, you should be able to share cookies between the two.

@jamonholmgren Oh, glad to hear it! Thanks


Fixed the overflow-hidden on the modal

This PR fixed the modal overflow. In my case, i got a select component in the modal that doesnt show all the content.
Before

After

Thanks for the PR; I expect these changes to affect the mobile screen size. Have you tested this by any chance?

Thanks for the PR; I expect these changes to affect the mobile screen size. Have you tested this by any chance?

Yes. i tried. for me it is working as expected.

Hi @putera
Thanks for the feedback. Currently, the modal will jump to the bottom of the screen on mobile to make it easier to be accessed. This seems to be caused by the change of items-end to items-center.


EPP Starter Kit v1.0 views

Views and readme for EPP Starter Kit's v1.0

Views seems to be correct and no conflicts found in reviewing

It seems that the first commit on this PR is not signed.
Merging blocked due this.

Canceling this PR, will resubmit with a signed commit


📃 docs(examples): fix KnowledgeGraphDemo link error

Description

fix docs\docs\examples\index_structs\knowledge_graph\KnowledgeGraphDemo.ipynb example.html link error

New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ ] Yes
[x] No

Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

It works in my environment, you can try this link.
@nerdai


Multiple ifDefined() directives don't work properly

Description
Only the last ifDefined() instance is respected. If the last value is defined, the committer is dirtied and setAttribtue() is called, causing the attribute to be rendered with the ToString value of the directive rendered to the attribute.
Steps to Reproduce

Write this code

html`<div foo="${ifDefined('a')}${ifDefined(undefined)}"></div>">`

See this output...

😬

Repro showing the general result of having noChange at different positions in a multi-part attribute binding: https://jsbin.com/mivoriz/edit?html,output

This is fixed in 2.0


update the sw.yml

change the name and change the os

Because it is bulid the new program name and it will be necessary for the
program.
On Mon, Oct 12, 2020, 11:53 AM Stefan Weil<EMAIL_ADDRESS>wrote:

@stweil commented on this pull request.
In .github/workflows/sw.yml
https://github.com/tesseract-ocr/tesseract/pull/3121#discussion_r503062311
:

@@ -1,4 +1,4 @@
-name: sw
+name: as

What is the reason for this change?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/pull/3121#pullrequestreview-506321084,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARHIUVP4F3BVYLXE3P3QTWLSKKOH3ANCNFSM4SMJIUYQ
.

@egorpugin : any idea about this?

Maybe @Anshulverma-prg tried to do changes only in his fork?

Name is fine.
We'll use Ubuntu LTS, no need to update.


Disable HDR

There have been many issues related to HDR causing imagery to appear darker in Cesium:

https://github.com/AnalyticalGraphicsInc/cesium/issues/7962
https://github.com/AnalyticalGraphicsInc/cesium/issues/7957
https://github.com/AnalyticalGraphicsInc/cesium/issues/7815
https://github.com/AnalyticalGraphicsInc/cesium/issues/7417

Personally I do not think HDR provides enough (or any) of a visual quality improvement to justify being on by default. The problem is very little of the Cesium rendering engine outputs high dynamic range values so tone-mapping is ineffective and only darkens the scene.
We should consider disabling it on everything but PBR glTF 2.0 models, where the tonemapping would just be part of the PBR shader.

+1

I agree as well, I feel like HDR went in prematurely and it needs a lot more thought to be enabled by default.  It affects too many things in unintended ways.

Agree to turn it off by default.  I would think this includes glTF PBR models but that is up to @lilleyse.

I made a quick sandcastle example to see the differences between HDR and non-HDR cesium.
It might be worthwhile to:

Enable HDR on glTF models or
Leave HDR enable but create a sand castle to show how to enable/disable

Here is a

Just seeing how washed out the milk truck looks with HDR disabled I think it will be a good idea to add tonemapping to the PBR shader when HDR is disabled.

And both the atmsophere and shadows can be tweaked when HDR is disabled.

The reason it looks washed out is because of our PBR shader is trying to output HDR values right? These equations were created with HDR in mind for example: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Scene/processPbrMaterials.js#L787-L790

HDR was disabled by default in https://github.com/AnalyticalGraphicsInc/cesium/pull/8055


bug: make section items center

bug fix for issue #10

@bangadam, I appreciate your eagerness to contribute, but this specific task has already been assigned to @prince5638. To avoid any confusion, I will only be merging pull requests from contributors to whom the issue has been assigned.


Samsung.TizenStudio version 5.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @upintheairsheep,
The installation isn't unattended - the installer requires a keypress to exit:

Is there an installer switch to bypass this and have it exit automatically?

Hi @upintheairsheep,
The installation isn't unattended - the installer requires a keypress to exit:
Is there an installer switch to bypass this and have it exit automatically?

I'm not sure, try /s or /S

This package doesn't appear to have a silent install option. Examining the suggested switches only gives 3 options, 2 related to the license and 1 related to Java:
Loading... Please wait.
Setting up the jdk in temp folder
Invalid options : -s

Usage: tizen-web-cli_XXX [option]... [<installation path>]
       tizen-web-cli_XXX [--show-license] [--accept-license] [--no-java-check] [
<installation path>]

Options:
  --show-license         display Tizen Studio software license agreement.
  --accept-license       accept license agreement.
...

Pending:

https://github.com/microsoft/winget-cli/issues/910

[Policy] Blocking-Issue


Mobile View Navbar Inoperable, Content Unreachable

The navbar on our website fails to open on mobile devices, rendering the content within it inaccessible. This affects the user experience, preventing users from accessing essential navigation elements such as login, sign-in, home, and about us sections.
Expected Behavior

Navbar should expand on mobile devices.
All menu items should be accessible.

Current Behavior

Navbar remains unresponsive on mobile.
Content within the navbar is unreachable.

Proposed Solution

Investigate code for responsiveness.
Test on various mobile devices.
Optimize content for smaller screens.

screenshots :

@Muskansahuincredible please assign it to me

Thanks @shivamgaur99 for raising an issue
Okay assigned to you


BLE token appears to increment with each broadcast

I re-scanned my remote for its BLE advertising token today, and the token it advertised is different to the one when I first did this a few months ago. I ran a few more tests and it appears the BLE advertising token of my remote is incrementing roughly with each broadcast.
I wrote a script for scanning tokens using python, was originally looking ways to sniff the token directly on a bluetooth-capable host. I ran the script whilst pressing the power button (projector unplugged ofc).  The first token bytes are somewhat sequential, which I could see happening over a few attempts at capturing with EFR. Interestingly, for me both the original token and an 'incremented' token spoofed with EFR Connect worked to turn on my projector, which it doesn't for the linked comment.
Also, please add a Bluetooth label for issues so we can tell which are and are not bluetooth related :)

I think I can provide a bit more potential background although I also don't have an answer yet to why my device (Elfin projector) behaves like this:
When I turn my projector on using the BLE advertising data that I'm using every time to turn on the projector this works regardless that the original XGIMI remote also changes the first byte every time the device is being turned on again. It's also counting up, so I assume it will count up to FF and then (hopefully) start with 00 again. I ran an automation to repeating on / off cycles of the projector 30 times (with delays of 30 seconds between turning on and off the projector) and my same advertising code always worked despite the XGIMI remote using slightly different advertising data.
But now there's an interesting exception to this pattern: By coincidence I found out that when I turn off the projector (I used the XGIMI remote, I haven't tested the following scenario yet using a software based remote command over WiFi to turn off the projector) and I send my own advertising data shortly after the projector led was turned off (e.g. within about 3-5 seconds) the projector would not turn on again + the projector would from this moment onwards not accept my own advertising data anymore. Even after 1 hour my advertising data would not turn the projector on anymore.
Nevertheless the projector would turn on again using the XGIMI original remote. So obviously this hex byte changes both in the remote and the projector every time you turn it on (or off).
When I turned the projector on and off again using the XGIMI original remote and I waited again about 30 seconds at least before I tried my own advertising again the projector would again react to my own advertising data.
Since I successfully ran the simulation of turning on/off the projector 30 times with the same advertising data with some delay between each turn on/off cycle I'm confident that I can rely on the same advertising data. Nevertheless it would of course be interesting to better understand the exact mechanism in the device and the XGIMI remote control. I'm definitely curious to see what will happen when the projector and the XGIMI remote have reached FF and if the HEX byte will start with 00 again and not change any other part of the advertising data.
One more thought: In case the advertising data should remain the same forever except the first HEX byte then one could of course think of additional tweaks to achieve maximum reliability for the projector wake-up :

One could send the advertising data with all HEX byte variants (although I'm not sure to how many the projector would listen too if all variants would be broadcasted within a too short period of time.
One could look into potential ways to track the current HEX byte that was exchanged between the projector and the XGIMI remote (I'd assume this might even happen if you don't use the XGIMI remote at all to turn the projector on or off, but of course I don't know what would happen if not even a remote is paired to the projector).


Remove qml.Rot (and CRot) from merge rotation pass patterns

Context:
In #1162 the mlir pass for merging rotation was added, and the pattern included qml.Rot (and CRot).
However, it was then realized that these two kinds of rotations should not be merged, as the rotation is specified via
rot(a, b, c) = rz(c) ry(b) rz(a)
This means
rot(a, b, c) rot(d, e, f) != rot(a+d, b+e, c+f)
since y and z rotations do not commute.
Description of the Change:
Remove qml.Rot (and CRot) from merge rotation pass patterns
Benefits:
No numerical errors

Do not just exclude Rot from the optimization pass, different gates may have different merging rules. See here for a Rot reference:
https://docs.pennylane.ai/en/stable/code/api/pennylane.transforms.merge_rotations.html#details-on-rot
If we determine that the method used in PL is not suitable for Catalyst, we could consider excluding it from the pass.


Requirements For Compliance App

Abstract:
Going forward, we would like for this repository to control the source of truth for the vendors listed in the vendor portion of the compliance app.
Context:
Currently, vendors are loaded into the app via J1QL. We would like to propose that this list of vendors instead comes from this repository.
Technical Details
The vendor stack repository will be responsible for managing the state of a vendor's information. When state changes in the vendors folder in this repository on the main branch with a new package.json version, we will deploy a new Vendors.json file to an S3 location. This JSON will include all the vendors in this file in a format like so:
 [
    {vendor_name}: {
      [key]: value,
      ....
    },
    ....
 ]

This file will be encrypted and this location will only be accessible from the compliance app.
In the compliance app, we can make a GET request to this S3 location to retrieve the JSON and populate the data of the app.

Moved to Confluence: https://jupiterone.atlassian.net/wiki/spaces/SEC/pages/11599873/Compliance+App+Vendor+Stack+Repository+Requirements


Copying image not working

Describe the bug
When I tried to use the "Copy image" button, it only shows the following message in the output panel
[CaptureImageFailure]
undefined

without further info. This is not specific to particular diagram; I've tried many different diagrams but all failed.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
VSCode version 1.77.3
Extension version 0.18.1

I have the same scenario, the copy image button and copy image command doesn't work

Thank you for reporting the bug. I found the cause. I will fix it in the next release. For short term, you can avoid the error by changing "MermaidEditor>Generate:Type" to png from the other value.

Recently I updated my default browser and now it seems to work correctly, probably because the Diagram visualization is based on browser

@daniecas I published v0.19.1 with this bug fix. Thank you for your reporting again.


Added examples

Yesterday I've created two examples to help a guy and think that would be useful add them in the repo.
I used the gitcdn to host the live demos. The links in the README doesn't work yet because they point to jackedgson/crunker , wich doesn't have the examples. They do work from my fork:

Audio from Client
Audio from Server

@Alynva thanks bro, wondering if we should expose something like a doTheWork method in the core api as a more abstract method which is like the simplest entry point to the API to make dimple

I think that will be cool, with a well defined schema. Things like 'autoDownload' may not what some users want by default.
But aside as that, the little line that I did the get the buffers from the audio files, that would be awesome have abstracted.

Cool, if you want to take this on let me know. Pretty packed at the moment so if you'd like to I'd appreciate it.
If not all good, I'll try and implement within the next few weeks. Thanks for all your contributions thusfar. If you'd like to add a contributors section to the readme with your profile, feel free to do so 😎

I probablly can take some effort on that in this week, I'll "doSomeWork" in a branch and we talk about it later


iOS 13

Hello everyone,
So iOS 13 is now in the hands of developers and there's been some changed to the Home App.
Fans now group accessories with them, so the Fan and Light Accessory and now 1 (Still 2 accessories just show as 1 under the same icon). Now it also added the Fan Reset switch in it so you see the Fan Reset and Fan Light together. I was wondering if anyone could make a PR or a branch based off #19 to be able to have the "Fan Reset" switch not pair with the fan.
If this was confusing here's a tldr:
This plugin adds 3 accessories per fan (Fan, Light, and Fan Reset). iOS 13 now lets you tap on the fan accessory to control the Fan and Light in the same field. Since this plugin adds 3 accessories, it groups all 3 since they are linked (individual controls still work). I was just asking if there can be a way to have the "Fan Reset" accessory not group together with the new fan in iOS 13.

So then it should just look like the Fan and just a Light on the side and have the fan reset be separate.
Paging @plainsane

ill take a look at some point but im wrapped up at the moment.

I don’t currently have a device I’m willing to sacrifice to beta1, but I’ll try to take a look at it once the public beta is released.

@vdotn I made a fork that removes the Fan Reset device, which we still need, but I'm not experienced enough to separate the accessory from being merged.

I posted this in a couple other issues on this repo, but I just finished writing the initial version of the plugin using the v2 API: https://github.com/aarons22/homebridge-bond-v2
In doing so, I ended up dropping the Reset switch all together since the API exposes the current state of the device from the Bond. When HomeKit requests the current value (either on startup or when it's Updating..., I'm now querying the API to get the state. Now the Bond device is the source of truth, and I'm not really seeing a need for the Reset switch at this point!

Anyone having issues should use: https://github.com/aarons22/homebridge-bond-v2. Fixed all my issues and the project is still active.


Knetbuilder is working not working for 3.0 while it is working for 1.2

I am using following command to build in ondex-mini folder.
./runme.sh ../tutorial_files/Workflow/HumanDiseaseKNET.xml. It is not working for 3.0 while it is working for 1.2. May i know know what is the issue?

Could you provide us with details? What do you have in the XML? What output do you get from the command? Could you send us the latter (using output redirection) and also the ondex.log file?

I updated java in my system.
openjdk version "11.0.5" 2019-10-15
OpenJDK Runtime Environment (build 11.0.5+10-post-Ubuntu-2ubuntu116.04)
OpenJDK 64-Bit Server VM (build 11.0.5+10-post-Ubuntu-2ubuntu116.04, mixed mode, sharing)
And ran the same command as above but still getting issue.
java11-ondex.log
java11-output.txt

Version 3.0 doesn't work with Java > 8. JAXBContext not found means it isn't included with its dependencies cause it used to be available from the JDK, while it was removed in recent Java versions.
If you want to use Knetbuilder with J11, you should download the latest snapshoot, see here.
The original error about the mapping plugin yielding a null pointer exception from Lucene, it's due to #29: certain accession formats (eg, '-') aren't accepted by Ondex. You should review your input files, almost certainly you have wrong accessions.

I am just trying to recreate the tutorial example described here https://github.com/Rothamsted/knetbuilder/blob/master/README.md    So none of it is my data. The tutorial files are here https://rrescloud.rothamsted.ac.uk/index.php/s/H6sl0RIT9CoMaUI/download
How do I fix this?

Sorry, we will need to update the README as some of the tutorial datasets are not compatible with version 3.0.
The latest knetbuilder tutorial 3.0 with tested datasets and workflows can be found in our wiki:
https://github.com/Rothamsted/knetbuilder/wiki/Building-Knowledge-Networks
Hope this helps.

Thanks i will play with and report back. Is there any build script for Arabidopsis KG v45 (https://knetminer.com/tutorial/knetbuilder/arabidopsis_45.oxl) file?

Yes there is but it's not open-access. The arabidopis_45.oxl contains datasets that are a few years old. Newer Arabidopsis KGs and build scripts can be licensed from us. Drop me a DM or an email if you would like to discuss the licensing terms and pricing plans.

Thanks for prompt reply. I was just curious to know about the scripts for generating arabidopis_45.oxl and currently focussing on other methods for our analysis.


Adds energy monitoring capabilities to the TP-Link HS110

Description:
Adds energy monitoring to the TP-Link HS110 Smart switches
Related issue (if applicable):
Pull request in home-assistant.github.io with documentation (if applicable):
Example entry for configuration.yaml (if applicable):
switch:
  - platform: tplink
    host: IP_ADRRESS

No changes to the configuration are needed. The updated module should automatically detect whether the switch's model and determine whether energy monitoring is available (HS110) or not (HS100)
Checklist:
If user exposed functionality or configuration variables are added/changed:

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

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

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[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.
[-] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Energy monitoring works only on the HS110 model

Two notes to make:

I haven't heard from the original developer of the pyHS100 module, so I created my own fork and created version 2.0
Currently the device_state_attributes will be polled irrespective of whether the switch supports energy monitoring or not. I couldn't figure out a way how to conditionally define device_state_attributes so it'd run only for the model that actually support energy monitoring, and not run for the model that doesn't support energy monitoring. I'm open to suggestions on how to optimize this.

Your fork introduces alot of mixed indentation.
pyHS100/__init__.py:1:1: W391 blank line at end of file
pyHS100/pyHS100.py:1:80: E501 line too long (87 > 79 characters)
pyHS100/pyHS100.py:1:88: W291 trailing whitespace
pyHS100/pyHS100.py:3:80: E501 line too long (87 > 79 characters)
pyHS100/pyHS100.py:4:80: E501 line too long (94 > 79 characters)
pyHS100/pyHS100.py:58:11: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:61:11: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:97:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:98:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:101:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:107:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:107:80: E501 line too long (81 > 79 characters)
pyHS100/pyHS100.py:109:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:112:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:115:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:121:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:121:80: E501 line too long (81 > 79 characters)
pyHS100/pyHS100.py:123:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:126:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:127:1: W293 blank line contains whitespace
pyHS100/pyHS100.py:129:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:132:80: E501 line too long (90 > 79 characters)
pyHS100/pyHS100.py:135:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:138:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:140:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:143:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:144:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:146:36: E251 unexpected spaces around keyword / parameter equals
pyHS100/pyHS100.py:146:38: E251 unexpected spaces around keyword / parameter equals
pyHS100/pyHS100.py:146:74: E251 unexpected spaces around keyword / parameter equals
pyHS100/pyHS100.py:146:76: E251 unexpected spaces around keyword / parameter equals
pyHS100/pyHS100.py:146:80: E501 line too long (107 > 79 characters)
pyHS100/pyHS100.py:147:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:148:1: W293 blank line contains whitespace
pyHS100/pyHS100.py:150:80: E501 line too long (94 > 79 characters)
pyHS100/pyHS100.py:151:80: E501 line too long (96 > 79 characters)
pyHS100/pyHS100.py:154:80: E501 line too long (90 > 79 characters)
pyHS100/pyHS100.py:155:80: E501 line too long (94 > 79 characters)
pyHS100/pyHS100.py:157:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:160:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:160:80: E501 line too long (119 > 79 characters)
pyHS100/pyHS100.py:162:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:165:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:167:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:169:11: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:171:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:173:38: E251 unexpected spaces around keyword / parameter equals
pyHS100/pyHS100.py:173:40: E251 unexpected spaces around keyword / parameter equals
pyHS100/pyHS100.py:174:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:177:80: E501 line too long (94 > 79 characters)
pyHS100/pyHS100.py:180:80: E501 line too long (90 > 79 characters)
pyHS100/pyHS100.py:181:80: E501 line too long (88 > 79 characters)
pyHS100/pyHS100.py:183:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:186:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:186:80: E501 line too long (94 > 79 characters)
pyHS100/pyHS100.py:188:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:191:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:193:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:195:11: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:197:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:198:1: W293 blank line contains whitespace
pyHS100/pyHS100.py:200:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:207:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:210:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:212:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:214:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:218:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:220:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:222:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:225:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:227:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:228:80: E501 line too long (105 > 79 characters)
pyHS100/pyHS100.py:231:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:232:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:233:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:233:23: W291 trailing whitespace
pyHS100/pyHS100.py:238:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:241:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:244:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:246:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:247:80: E501 line too long (105 > 79 characters)
pyHS100/pyHS100.py:248:80: E501 line too long (81 > 79 characters)
pyHS100/pyHS100.py:250:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:253:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:253:16: W291 trailing whitespace
pyHS100/pyHS100.py:254:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:255:7: E111 indentation is not a multiple of four
pyHS100/pyHS100.py:255:23: W291 trailing whitespace
pyHS100/pyHS100.py:257:21: W291 trailing whitespace

It looks like the original author has responded to your PR. Let's await his merging of your PR before we continue with this PR.
@GadgetReactor, since you are familiar with this code, would you mind reviewing this PR?

My updates were merged into GadgetReactor's and most of the styling errors in the module should be resolved.
@mweinelt can you tell me what lint settings you're using, because I got different output when using the HASS pylintrc against the pyHS100/pyHS100.py codebase.

From my end, the code looks good and tested with the HS100 with no issues.

@kirichkov thats just the flake8 default.

@balloob any possibility of getting this in for 0.31?


timestamp format issue

https://github.com/GENIVI/iot-event-analytics/issues/105#issuecomment-867438435 reported that in kuksa.val the timestamp format is YYYY-MM-DDTHH:mm:ss.SSSZXXX, whereas XXX is some numeric value and which leads to problems when parsing that value.

ok, expected format is YYYY-MM-DDTHH:MM:SS.ssssssZ    (ending after "Z" whioch is the UTC timezone designator) , https://www.w3.org/TR/NOTE-datetime
need to check

Quick test showed no problem. But maybe some memory corruption when the fractional part is smaller?
Test Client> authorize ~/kuksa.val/kuksa_certificates/jwt/super-admin.json.token {
    "TTL":<PHONE_NUMBER>,
    "action": "authorize",
    "requestId": "916e837f-10b8-437b-8110-9eb76b72ea63",
    "timestamp": "2021-06-24T09:22:42.1624522962Z"
}

Test Client> setValue Vehicle.Speed 200
{
    "action": "set",
    "requestId": "2f00a6e8-ed9b-497a-80ae-7f150609f9e9",
    "timestamp": "2021-06-24T09:22:58.1624522978Z"
}

Test Client> getValue Vehicle.Speed
{
    "action": "get",
    "path": "Vehicle.Speed",
    "requestId": "dd5922e1-8d31-4f69-86c2-00bd535d1ed5",
    "timestamp": "2021-06-24T09:22:58.1624522978Z",
    "value": "200"
}

Or a problem with  the alpine image... Need to check....

The numeric value is gone in the meantime. The whole adapter was broken, since we expected actually a Unix-Timestamp in Milliseconds. So we compared the given number with a threshold. If the threshold was succeeded (Which the given ISO timestamp literally does) we appended the Milliseconds to it --> Since we added something to the given string, it seemed to us, that Kuksa.Val added the stuff at the end. But in the end it was our adapter. One question still remains: Did you actually sent a Unix-Timestamp and changed it to be an ISO timestamp?

Yes, because that was a change upgrading the VISS protcol: V1 used Unix timestamps, V2 switched to the W3C convention of using ISO timestamps


Rule test updates for PR #3865 v1

This PR has the test updates to account for the changes made in https://github.com/OISF/suricata/pull/3865

I will do it when merging, but I think in a case like this a single commit is sufficient.

I will do it when merging, but I think in a case like this a single commit is sufficient.

Okay, I will keep in mind for the next set. Thank you!

Merged in #72, thanks Jason. I did squash all the commits together.


Implemented FromStr for NumberPrefix

Now with this PR we can parse NumberPrefix structs from strings. I've also added a case for units of measurement (as for now meters, bits and bytes are available).
cc @ogham

Thanks, this is a good idea!


Logging sync=trace is way too verbose

sync=trace is extremely spammy, because we print log messages in getters that are called every time the network future is getting polled.
This should be reworked a bit.

Note that by "rework this a bit", it doesn't necessarily mean removing some log messages. It can also mean adding some cache if the calculation is expensive.


ChatGPT endpoint?

How would I go about setting that up? It's even better than DeepL for translating Japanese in my testing

Copy one of the existing endpoints in the Translators folder and modify it as needed I suppose.

Copy one of the existing endpoints in the Translators folder and modify it as needed I suppose.

How would one go about doing this? Would it be possible to provide a step-by-step guide? It'd be greatly appreciated

Here's one I did for Chinese, you could probably mod it for yourself:
https://github.com/joshfreitas1984/XUnity.AutoChatGptTranslator


Fixing smogintro link

Fix link of "Introduce yourself to Smogon!".

The others links are bad too, but I don't find them the correct link on smogon. I just found the correct for  "Introduce yourself to Smogon!".

The mentorship program has been inactive for a while

Ohh.

Thanks for fixing a link, but I think this commands needs a bigger improvement. Feel free to check http://www.smogon.com/forums/forums/smogon-info-intro-hub.264/ (Smogon Info / Intro Hub), which replaced old Smogon Mentorship to check what links may be useful.
Also, don't make links with thread titles. Just do https://www.smogon.com/forums/threads/3522706. It will still work, and it's the style used in the code. Make sure to also use HTTPS, despite Smogon not supporting HTTPS (eases maintenance when Smogon will finally support HTTPS).

Ok thanks, i will fix and search for links.

I think it should just be:

Welcome to Smogon's official simulator!

Smogon Info / Intro Hub

There's no point in linking just one thread in that forum, and linking all the threads is too much. Just link the forum.

I will wait @Joimer.

@TheImmortal, I think that is good to link some specific threads, because with that user can have a better guide.

I'd go with Useful Smogon Info and Tiering FAQ.

Agreeing with @TheImmortal here, that would be neat for this update.


Publish

[x] Add zip to release
[ ] FireFox
[ ] Chrome
[ ] Safari

Running safari extensions

Moz Addons page


Add link to homepage for 500 error page

Add link to homepage for general 500 error page (agreed with design)
Refactor some props and repetitive styling across error templates

:tada: This PR is included in version 4.47.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Check that we are doing the right thing with images with GUS contributor code

There are 23  images with GUS contributor code that at some point ended up in the API.

9 of these we were asked to remove by Hannah:

B0009891
B0009897
B0009886
B0009893
B0009887
B0009895
B0009884
B0009890
B0009888

Another 14 were in the API until #2212 was merged, which prevents all images with GUS contributor code to appear in the API:

B0009883
B0009885
B0009889
B0009892
B0009894
B0009896
B0010007
B0010008
B0010009
B0010010
B0010011
B0010012
B0010013
B0010014

I seem to understand that Hannah request came directly from the contributor who asked to remove the top 9, but not the others.
Where has the decision to not show all GUS images come from? Is that the correct behaviour?

I’ve asked Hannah to dig out Karen’s original email, so hopefully we'll get some answers that way.

Looking at the original emails, we should have excluded just those nine records. 🙈
I’ll fix.


feat: fall back to lua version from installed version

Closes #213.

Something seems broken

The pull_manifest tests are flaky again.

Removed some entries from our test manifest

Looks like there's a failure with the mock server.


remove replConfig in pipeline

What problem does this PR solve?

Issue Number: close #6138
What is changed and how it works?
remove replConfig in pipeline
Check List
Tests

Unit test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/cc asddongmen

Codecov Report

Merging #6139 (33993d3) into master (792faf7) will decrease coverage by 1.4778%.
The diff coverage is 65.7303%.

Flag
Coverage Δ

cdc
63.4868% <76.6666%> (-1.7201%)
:arrow_down:

dm
51.9034% <43.1034%> (-0.0204%)
:arrow_down:

engine
?

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #6139        +/-   ##
================================================
- Coverage   58.6479%   57.1701%   -1.4779%
================================================
  Files           711        543       -168
  Lines         83933      72349     -11584
================================================
- Hits          49225      41362      -7863
+ Misses        30258      27191      -3067
+ Partials       4450       3796       -654

/run-all-tests

/run-integration-test

/merge


NPM installs pre released version by default

Expected behavior:
Expected npm install --save-dev gulp-typescript to install version 4 not 5.0.0-alpha.3
maybe this will work?
npm publish --tag next
as described in this post
Actual behavior:
installs 5.0.0-alpha.3
Your gulpfile:
N/A
Include your gulpfile, or only the related task (with ts.createProject).

tsconfig.json
N/A
Include your tsconfig, if related to this issue.

Code
N/A
Include your TypeScript code, if necessary.

Just wanted to add that, as you would expect, this is also an issue if installing via yarn.
If we run yarn outdated on our repos it complains that we are not using the latest version and suggests we upgrade to the alpha.

Four months later. Also got 5 alpha automatically. It's buggy, you really should not distribute this by default.

Sorry for the late response. It's annoying that npm marks alpha releases as latest and will install them by default. Anyway, I just published 5.0.0, see the release notes, so this can be closed. I'll keep this in mind for future alpha releases.


Changing the logo of twitter icon

the twitter logo can be changed to

Please assign me this task under GSSOC 24


Duplicate URN error when trying to create 2 GKE clusters in differents projects/regions

I'm trying to create several clusters with the same name in different regions within the same stack, but it's impossible (eg. identical clusters named prod in regions europe-west3 and us-east1).
I understand that the cluster name is used by terraform to generate the actual cluster name (with the random string at the end) if you don't specify name in the args. The problem is, the cluster name is used as-is to build the URN, so you can't have 2 clusters named the same, even though they're in different regions/projects and that it's perfectly valid (even in terraform).
Do you have plans to only have unique names by default so it's not up to the user to come up with one ? Here, you could use the combination of project id, region/zone and cluster name to ensure that it's unique.

You can create the cluster with:
let cluster = new gcp.container.Cluster("puluminame", {
    name: "googlename",
});

This allows the puluminame which must be unique for all resources in the Pulumi program to differ from the googlename which you want to be the same in each region.
The name: ... argument is optional, and will default to a randomly generated name with puluminame as a prefix if not provided - but you can override this by specifying an explicit name:.
I believe this should address your scenario - but let me know if it does not.


Buttons don't look right on macOS

Describe the bug
Buttons don't look right on macOS (there is a rounded border on them)
To Reproduce
Steps to reproduce the behavior:

Use macOS
Visit the challenge generator

Expected behavior
The buttons look right
Screenshots

Desktop (please complete the following information):

OS: macOS
Browser: Google Chrome
Version: 74.0.3729.169

Fixed by #21


Computed properties should observe changes to service by default

Similar to the helper, our computed property macros should observe changes to service by default

6.0.0