Returning from Array#forEach

I noticed this code in the auth plugin:
https://github.com/genie-team/graphql-genie/blob/fa8f88ae40549a8afde5daaaa0594c200f98116c/plugins/authentication/src/authentication.ts#L291-L296
I don't think that's doing what you meant, is it? Returning from forEach  does nothing! Perhaps it's not actually a problem and I'm miss-interpreting it.
Anyway, thanks for working on this awesome GraphQL library!

hmm yea I'm sure I meant it to return false from the checkArgsFromResolver function but I think you are right that it won't work that way. I'll check it out, probably means I missing a test to cover something as well.

Thanks for the catch and compliment, hoping to share the library more widely soon and catching these sorts of things now is great.
Looks like it was missed by tests because most of my tests I was just throwing errors in the user code so it was never getting to that part of the plugin code on the failures I had. Added some tests to cover the boolean case.


http: fix the transfer-encoding header check in case of multiple values

For http chunked encoding, currently envoy needs the value of the TE header to be exactly chunked.
In our production env, it has led the following two unexpected stream resets.

Some legacy http servers respond duplicate headers. If two TE headers with the same value "chunked" are received,  envoy will concat them with a , into chunked,chunked,  while later this check fails on this value.
For TE headers with multiple values (e.g. 'gzip, chunked'), envoy also resets the stream because of this check.

We have also tested with nginx, which allows the above two cases. After this fix, now envoy acts properly as expected.
Commit Message: fix the transfer-encoding header check in case of multiple values
Additional Description:
Risk Level: Medium
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

+1 to @PiotrSikora . Having chunked twice is explicitly forbidden by RFC 7230 section 3.3.1 so Envoy has no choice but to reset these streams.
On the topic of unknown encodings, if we want to pass those through, we'll need to:

validate that the last encoding was chunked
validate that chunked only appears once
send the unknown encodings downstream
remove the chunked encoding if downstream is HTTP/2 or later
add tests to validate these points.

Thanks for the reviews. So we decide to disable this check in our production env. Let me close this PR.


Maven - ChangeDependencyGroupIdAndArtifactId should change  GAVs too

Presume we want to change org.acme:acme-foo to org.acme:acme-bar in this code:
<dependencies>
  <dependency>
    <groupId>org.acme</groupId>
    <artifactId>acme-foo</artifactId><!-- Change me to acme-bar -->
  </dependency>
</dependencies>
<dependencyManagement>
  <dependencies>
    <dependency>
      <groupId>org.acme</groupId>
      <artifactId>acme-foo</artifactId><!-- Change me to acme-bar -->
    </dependency>
    <dependency>
      <groupId>org.whatever</groupId>
      <artifactId>whatever</artifactId>
      <exclusions>
        <exclusion>
          <groupId>org.acme</groupId>
          <artifactId>acme-foo</artifactId><!-- Change me to acme-bar -->
        </exclusion>
      </exclusions>
    </dependency>
  </dependencies>
<dependencyManagement>

The first change can be done with ChangeDependencyGroupIdAndArtifactId.
The second change can be done with ChangeManagedDependencyGroupIdAndArtifactId
The third change can't be done.

Support changing the <exclusion> GAVs too.

Proposal A) Add a new recipe ChangeExclusionGroupIdAndArtifactId
Proposal B) Get ChangeDependencyGroupIdAndArtifactId to do all 3 changes always. Deprecate ChangeManagedDependencyGroupIdAndArtifactId.
Proposal C) Get ChangeDependencyGroupIdAndArtifactId to do all 3 changes if and only if the parameter managedToo=true and exclusionsToo=true. They are both false for Backwards Compatible reasons. Deprecate ChangeManagedDependencyGroupIdAndArtifactId.
Slack conversation: https://rewriteoss.slack.com/archives/C01A843MWG5/p1678119013546069

I personally think that proposal A is not user-friendly. It violates DRY for users.


Explicit linter config path removed to enable separate config for View

SwiftLint effectively reads configs for every nested directory when not provided with explicit path. Per dir configs emerged with #26

Superseded by https://github.com/flipperdevices/Flipper-iOS-App/pull/63


please add lenis.init() to reinitializes the scroll.

I am using barba.js but I need to reinitialize the code lenis.init() after lenis.destroy()

I'm using Lenis + barba.js and I encountered some problems with scrolltrigger animations. Wasn't smart but I used gsap.defaults() et Scrolltrigger.defaults() that messed up my animations.
Also try initialising Lenis once before barba even inits.

for now I init Lenis before barba init, and it works when switching pages without having to destroy, scrolltrigger is also no problem.
Previously I used Loco on my site, and I had a problem when switching pages the scrollbar doubled, I fixed it by destroying then reinstalling. So I was thinking is it ok to use Lenis + Barba without destroying and re-init again?

Indeed I do the same for locomotive scroll : I destroy it on afterLeave and init again on afterEnter. Fortunately you don't need to do that with Lenis.
I don't understand why you want to re-init again. You would creating another Lenis requestAnimationFrame function : not good for performances.

Hello guys,
I have such a strange problem with barbaJS.
I initiate lenis before barbaJS. But then even with .stop(), if I change pages, it adds class to the html but I can still scroll
Have you ever had this ?
Thanks Nicolas

Did you add the Lenis css to yours ?

Hello Oguilleux,
Yes of course :)

I explain more here : https://github.com/darkroomengineering/lenis/issues/310
Thanks !


Bridge: Fallback to NT if no fit with partner

Fix #181
This adjusts the case where no bid quite matches to fall back to NT if we don't have a fit with partner.
May still need some additional logic to rebid the original suit if sufficient length but too strong.

Superseded by #293


Added multi-schema support for DTF Sql-Server

Added multi-schema support for DTF Sql-Server as requested in #105.  Following the plan suggested by George, I've refactored the code to be able to use a custom schema name instead of the hardcoded "dt" as such:

Added a new property in SqlOrchestrationServiceSettings called SchemaName, defaulting to "dt"
Modified all the scripts under /Scripts by replacing "dt" with a placeholder called {{SchemaNamePlaceholder}} and also added explicit table accessing by specifying the schemaName for each access.
Added a new parameter to all xxxParameter methods from the SqlTypes files under /SqlTypes and from SqlUtils, called schemaName to be able to remove the hardcoded "dt".
Modified the SqlDbManager such that before executing the script commands, it will replace the schemaNamePlaceholder with the schema name from the settings. I found it to be the easiest way to replace the "dt" from the script files without rewriting the whole script. There is still room for improvement on this part and any suggestion is highly appreciated.
Added a new test file to test if the program is able to create and delete a schema with a custom name.
Modified the test files such that it will be able to test custom table names by adding a new parameter to the test method and replacing all the occurrences of "dt" with said parameter value.

As such, this version is backwards compatible as far as I know and is able to do all the operations with custom schema names.
Co-Authored-By: George Moldovan<EMAIL_ADDRESS>

Awesome! Looking forward to going through this. I’m out of the country on vacation now but will do a full review when I get back in a few weeks.

While writing the test @cgillum suggested, I encountered a problem with the LogAssert: After creating the first schema and validating the following steps,  the second schema validation fails with the following error

even though after manual testing, the logic is right and the program is executing the flow correctly. This can be avoided by not running the second validation, but this doesn't sit quite right. Do you have any suggestions in solving this?
On another note, I do not know how to edit the documentation. I am guessing you mean this one and not the README.md file.

On another note, I do not know how to edit the documentation. I am guessing you mean this one and not the README.md file.

Oh, sorry for not explaining this! Yes, I meant the published documentation. This documentation is authored via markdown files under the /docs directory. We're using GitHub Pages to have them published.

I'm not sure about the validation error. The log validation framework was created by another contributor @wsugarman, so I don't have as much insight into how it works internally to know why you're seeing these unexpected failures.

While writing the test @cgillum Chris Gillum FTE suggested, I encountered a problem with the LogAssert: After creating the first schema and validating the following steps,  the second schema validation fails with the following error  even though after manual testing, the logic is right and the program is executing the flow correctly. This can be avoided by not running the second validation, but this doesn't sit quite right. Do you have any suggestions in solving this?
On another note, I do not know how to edit the documentation. I am guessing you mean this one and not the README.md file.

I haven't looked at this in awhile haha, but it seems that there is a missing log entry that was expected: "secondTestSchemaName._UpdateVersion". In short, the tests record the log entries in a list, and the extensions validate the contents of the log through the Expects methods: https://github.com/microsoft/durabletask-mssql/blob/main/test/DurableTask.SqlServer.Tests/Logging/LogAssertExtensions.cs

Updated the code with @cgillum's suggestions.


Add telemetry helper

For Check, Eval add a helper that accepts ILogger and captures (As appropriate):

exceptions thrown - ToString(), type, callstack
Expression Length
Anonymized Expression (see https://github.com/microsoft/Power-Fx/blob/9178b974eced29fd06030d0f3ec563fa0048cf74/src/libraries/Microsoft.PowerFx.Core/Public/CheckResult.cs#L519 ).  Probably allow host to customize this.
Duration MS of check
Duration MS of Eval

Some optional additional information provided by host:

anonymized user
"document id"

Also could apply this to LSP.  Notably, we want:

duration
message kind
any exceptions thrown

See Fx test site for examples.

TBD whether we depend on ILogger directly, or introduce an interface. ILogger would add another dependency; so may be better to just define our own interface that is similar to ILogger.

Here's a proposal:

Create a new class, CheckDiagnostics, with properties corresponding to items above.
CheckResult then hold a single instance of this object.  As Check does things (parse, eval, etc), if the object is available, fill out the properties on the object.
The host can then log that object.


Cursive, evaluating cljs in nRepl connection

This perhaps should have been obvious to me yet I think documenting it might be helpful?
The file watcher re-compiles /target/resources/public/cljs/app/main.js whenever .cljs files change. However, at least one user must log into the app before shadow-cljs spins up a CLJS runtime.
Prior to evaluating .cljs code in an nRepl connection on port 7888:

log a user into the app
eval (shadow.cljs.devtools.api/repl :app) to transform the CLJ repl into a CLJS repl.

It is a good idea. I'll probably add this soon. Perhaps create another section with development workflow tips, and move some of the current stuff in the getting started section there (e.g. the explanation about evaling and then saving files).

No longer needed since new version of Biff doesn't use CLJS.


Set manegement bond network during installation

What
We can set management bond network during installation.

ISO: adding two action items in network connection page

Management Bond: Former "Management NIC". Now user can selection multiple NIC to create a NIC bonding
Bond Mode: Select bond mode

PXE booting: Since update the harvester config install.networks has been updated. User can specify install.networks.intefaces to create a bond.

Got "is attempting to grant RBAC permissions not currently held" for following rules:
- apiGroups:
    - subresources.kubevirt.io
  resources:
    - virtualmachineinstances/guestosinfo
    - virtualmachineinstances/filesystemlist
    - virtualmachineinstances/userlist
  verbs:
    - get

Need to figure out why.

@weihanglo can we please prepare PRs for both the docs https://github.com/harvester/docs/tree/v0.3-dev and ipxe-example of this change, thanks

If I have two interfaces on my system: ens5 and ens6

Create a bond with ens5 and ens6 and DHCP it.
Go back to the panel again, but this time only chose ens5.
Configuration file of ens6 still exist.

This is not introduced in this PR, but we need a way to reset the config files to the initial state.

What about adding the first one NIC as the default slave?

What about adding the first one NIC as the default slave?

@yaocw2020 Fair! I'll add that. Thanks

This is not introduced in this PR, but we need a way to reset the config files to the initial state.

@bk201. On 79769e7,  I've just added a mechanism that will restore the original network config before each time applying network to live OS. Please check it thanks. 😃

What about adding the first one NIC as the default slave?

@yaocw2020. I found that it's not convenient for current widget to provide a default value 😞 . If you think this is necessary in RC release, please tell me. I'll do a workaround. 🙇🏾♂

Is it convenient to add the miimon option for the bond device?

Is it convenient to add the miimon option for the bond device?

considering doing it in the new console UI.

Does this also cover bond for VLAN network or is that a separate implementation?  https://github.com/harvester/harvester/issues/1048

Does this also cover bond for VLAN network or is that a separate implementation? harvester/harvester#1048

this is for the management bond, the VLAN bond will be separated, and probably will need to do it in 1.0.0 version


Proposed documentation and unit tests for #2396

#2396

@CrossEye Shall we close this for now?

@CrossEye I'm tempted to reopen as we have docs and test cases. Perhaps we can simply drop the type signature for now?

Reopened, but with the focus on the signature, I don't remember ever really looking over those examples and docs.  I like the [dec, identity, inc] one, but I'm less thrilled with the always('add') one.  In all of these cases, I'd rather use simple arrow functions than require someone to understand other Ramda functions to follow the docs for this one.
I won't have time this evening, but I'll try soon to make suggested changes.


Update four digit year moment token

YYYY seems to be the correct token when I was testing it locally

Coverage remained the same at 96.401% when pulling dfa23dbcf2cec73c0c442da9f367ee345e590ec8 on ClemMakesApps:patch-1 into 03c18c50dfe7ca636c5665485fad80eb83d60d9e on charliekassel:master.

Seems like yyyy works when you do the format prop but not when you are using the method to do formatting.


Week 5 Review Ticket

CB Create Task completed and documented on GH Pages - Prep to submit to CB on or before May, 2nd. (2 Points - Individual)
CB Create Task Coding requirements documented
Runtime of Code(with comments) on Replit/Intellij
Accounts and Logins Challenges/Hacks ( 3 Points - Pair/Trio)
Start or add to existing project:  init.py, main.py, requirements.txt and cruddy blueprint directory are things you will need to add to your existing project.  Additionally, you should start with some of the content in templates and static folders.  Find a pair and work on this together.  Ask questions.
Analysis: the csp.nighthawkcodingsociety.com implementation is for educational purposes.  It has some obvious flaws in its security.   Discuss some of the flaws that you see in its security as implemented.
Hack #1 Add Phone Number to Sign Up / Authorization screen
Hack #2 Add logout to CRUD screen
Hack #3 Add login_required to another portion of the project
Extra Credit:  A quality design would have login and logout on Navbar with user information.   Be sure to have login protecting something on your site, however, do NOT make entire site under login_required.

CB Create Task Coding requirements documented
Comments: Incomplete +0.5
Runtime of Code(with comments) on Replit/Intellij
Comments: He has code but no comments and no runtime. +0.6
Hack 1 Add Phone Number to Sign Up / Authorization screen
Comments: Hack 1 is present and is clearly shown +1
Hack 2 Add logout to CRUD screen
Comments: Hack 2 works +1
Hack 3 Add login_required to another portion of the project
Comments: Code is there but doesn't work on website +0.5
Extra Credit: A quality design would have login and logout on Navbar with user information. Be sure to have login protecting something on your site, however, do NOT make entire site under login_required.
Comments: Did not do extra credit
Final Score: 3.6/5


Server doesn't start due to anyio issue

Well, Chat GPT helps me to run this server.
Chat GPT suggests the anyio library doesn't work or something.
It says I need to execute command:
mcp-pandoc server
This command hangs on execution and returns this log on stopping:
PS C:\Users\John_Doe> mcp-pandoc server
Traceback (most recent call last):
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\anyio\streams\memory.py", line 111, in receive
    return self.receive_nowait()
           ~~~~~~~~~~~~~~~~~~~^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\anyio\streams\memory.py", line 106, in receive_nowait
    raise WouldBlock
anyio.WouldBlock

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\asyncio\runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\asyncio\base_events.py", line 720, in run_until_complete
    return future.result()
           ~~~~~~~~~~~~~^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\mcp_pandoc\server.py", line 290, in main
    await server.run(
    ...<10 lines>...
    )
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\mcp\server\__init__.py", line 423, in run
    async with ServerSession(
               ~~~~~~~~~~~~~^
        read_stream, write_stream, initialization_options
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    ) as session:
    ^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\mcp\shared\session.py", line 122, in __aexit__
    return await self._task_group.__aexit__(exc_type, exc_val, exc_tb)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\anyio\_backends\_asyncio.py", line 819, in __aexit__
    raise exc_val
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\mcp\server\__init__.py", line 426, in run
    async for message in session.incoming_messages:
    ...<69 lines>...
            )
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\anyio\abc\_streams.py", line 35, in __anext__
    return await self.receive()
           ^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\anyio\streams\memory.py", line 119, in receive
    await receive_event.wait()
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\anyio\_backends\_asyncio.py", line 1809, in wait
    await self._event.wait()
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\asyncio\locks.py", line 213, in wait
    await fut
asyncio.exceptions.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Scripts\mcp-pandoc.exe\__main__.py", line 7, in <module>
    sys.exit(main())
             ~~~~^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\site-packages\mcp_pandoc\__init__.py", line 6, in main
    asyncio.run(server.main())
    ~~~~~~~~~~~^^^^^^^^^^^^^^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\asyncio\runners.py", line 194, in run
    return runner.run(main)
           ~~~~~~~~~~^^^^^^
  File "C:\Users\John_Doe\AppData\Local\Programs\Python\Python313\Lib\asyncio\runners.py", line 123, in run
    raise KeyboardInterrupt()
KeyboardInterrupt
PS C:\Users\John_Doe>

And there is the log from Claude:
2024-12-10T11:30:45.772Z [info] Attempting to connect to MCP server mcp-pandoc...
2024-12-10T11:30:45.800Z [error] Error in MCP connection to server mcp-pandoc: Error: spawn uvx ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:286:19)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn uvx',
  path: 'uvx',
  spawnargs: [Array]
}
2024-12-10T11:30:45.800Z [error] Could not start MCP server mcp-pandoc: Error: spawn uvx ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:286:19)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn uvx',
  path: 'uvx',
  spawnargs: [Array]
}

@koreyba  Could you please share the replication steps?
btw, recently added .docx & .pdf support

@koreyba Closing this issue. Please reopen or comment with replication steps if you still face the issue.
Also, added more file conversions support recently.
https://github.com/vivekVells/mcp-pandoc?tab=readme-ov-file#usage--configuration -> refer this to consume it.


Streamline interface between models and estimators

Currently, for every new model we create, we also need to create a corresponding ModelFilterInterface type class. In the future, when we introduce new models, we should revisit this, and considering writing a universal ModelFilterInterface that takes the model as input, and does all the same basic functionality regardless of the model used, or of the online estimator used.

We can also reduce the need to create as many subclasses more generally. There are a fair number of subclasses which redefine the docstring but don't provide new funcitonality (e.g., the KalmanHiddenState) that would be a good target for eliminating as we prune subclasses of ModelFilterInterface

Just going to point it out here based on the Slack discussion that the KalmanHiddenState is a child of two independent classes, so likely needs to exist on its own. However, I understand and agree with that sentiment, there are definitely a fair number of classes that are sort of like flair.

The ModelFilterInterface was eliminated in #38


Swipe down to dismiss Entry

Is your feature request related to a problem? Please describe.
The current version of the library although it supports custom exit animations does not seem to support swiping down to dismiss an entry.
Describe the solution you'd like
If possible, create another flag in .enable called something like direction. I want to present an Entry up from the bottom of the screen and then dismiss it by dragging down and then have it also exit from the bottom of the screen.
Describe alternatives you've considered
I've been looking through the code, especially EKContentView and EKAttributes + Scroll

Hi @asikand,
Perhaps I didn't quite understand your problem (If so, I'd appreciate if you could provide further detail), but I should state thatSwiftEntryKit does support a swipe-to-dismiss gesture. Have you set up the example project yet? ^^

There are only two options in attributes.scroll.
.enabled lets you scroll the Entry in both directions (pullback animation when user swipes toward bottom of screen), swipe toward top of screen to dismiss.
.edgeCrossingDisabled which lets you swipe toward the toward the top of the screen to dismiss and does not allow scrolling toward the bottom of the screen.
I need to be able to specify whether the dismissal should result from a swipe toward the bottom of the screen instead of a swipe toward the top of the screen

Have you tried the example project from this Github repository? All of the ones presented from the bottom are dismissable by scrolling downward.

Let me check the code and get back to you, perhaps I'm missing something.

I'm presenting a View Controller and not a view? Does this make any difference?

Issue was fixed by initializing attributes with EKAttributes.bottomFloat instead of just EKAttributes()

So there is no way to specify which direction you wanna allow to dismiss ?
regardless of which .position property you set ?
ex.
var attributes: EKAttributes = .topFloat
but allow the user to dismiss either swiping up or swiping down. (both)

Nope. Not at the moment.


build: ensure python3 is used for all python scripts

As OVN is Python 3 only ensure that the build process only
uses PYTHON3 and not PYTHON.
Signed-off-by: James Page<EMAIL_ADDRESS>

Thanks for the PR. I'll test this out tomorrow.

Closing the PR. I sent the patch to ovs-dev mailing list and then applied to master - https://github.com/ovn-org/ovn/commit/ff39e6c54a022ab8824d481ee26920e137b938a5


Refactor api

Some preliminaries before trying to make receive_frame thinner by moving the FrameInvariants generation on the send_frame side.

Coverage decreased (-0.7%) to 79.626% when pulling cce7d4480a5215e349c07718cb7a0a1e63c2f467 on rust-av:refactor_api into 35b4a23b2c10639db3d666e597552c87d432e85a on xiph:master.


Add nbytes to repr?

Is your feature request related to a problem?
Would having the nbytes value in the Dataset repr be reasonable?
I frequently find myself logging this separately. For example:
<xarray.Dataset>
Dimensions:  (lat: 25, time: 2920, lon: 53)
Coordinates:
  * lat      (lat) float32 75.0 72.5 70.0 67.5 65.0 ... 25.0 22.5 20.0 17.5 15.0
  * lon      (lon) float32 200.0 202.5 205.0 207.5 ... 322.5 325.0 327.5 330.0
  * time     (time) datetime64[ns] 2013-01-01 ... 2014-12-31T18:00:00
Data variables:
-    air      (time, lat, lon) float32 dask.array<chunksize=(2920, 25, 53), meta=np.ndarray>
+    air      (time, lat, lon) float32 15MB dask.array<chunksize=(2920, 25, 53), meta=np.ndarray>
Attributes:
    Conventions:  COARDS
    title:        4x daily NMC reanalysis (1948)
    description:  Data is from NMC initialized reanalysis\n(4x/day).  These a...
    platform:     Model
    references:   http://www.esrl.noaa.gov/psd/data/gridded/data.ncep.reanaly...

Describe the solution you'd like
No response
Describe alternatives you've considered
Status quo :)
Additional context
No response

I agree - I'm constantly checking this attribute. It would be nice to also quickly see the total nbytes of the whole dataset, but I'm not sure where that would go in the repr.

It would be nice to also quickly see the total nbytes of the whole dataset,

Yes very much agree. Maaaybe after Dimensions:  (lat: 25, time: 2920, lon: 53); 16MB??
Or if there's some consensus about adding to data vars, we could start with that. Though arguably it's more useful to have for the whole object...

This would be a really nice addition!

Hello,
I can suggest the following:

Use "natural human units" (multiples of 1000 like "MB"), not binary units (1024) ;
Max unit is "YB" (Yotta, 10**24) (this is arbitrary; is there real-life use cases of required larger units :question: ) ;
Put the nbytes representation in the "Data variables" section like suggested in the initial post ;
Do not print decimal part, as suggested in all posts above. Helps conciseness ;
For DataArray only representation and total size of a Dataset, put the rendered size into the header of the repr. There is room for short string content like a size. DataArrays representation already uses this place to put the name and dimensions. Datasets don't make use of this space yet and there is plenty of room.

If more customization capabilities are needed, eg choosing between "human" and "binary" prefixes, there exists a library under MIT license, humanize, that specializes into rendering various numbers, including file sizes. Some of its code could potentially be extracted and integrated into xarray.
Examples
<xarray.Dataset 10kB>
Dimensions:  (foo: 1200, bar: 111)
Coordinates:
  * foo      (foo) int64  10kB 0 1 2 3 4 5 6 ... 1194 1195 1196 1197 1198 1199
  * bar      (bar) int64 888B  0 1 2 3 4 5 6 7 ... 104 105 106 107 108 109 110
Data variables:
    *empty*

<xarray.DataArray 'foo' (foo: 1200) 10kB>
array([   0,    1,    2, ..., 1197, 1198, 1199])
Coordinates:
  * foo      (foo) int64  10kB 0 1 2 3 4 5 6 ... 1194 1195 1196 1197 1198 1199

Updated examples after update on the PR. The update is: the size in the header is outside of the <>-delimited header string, and prefixed with Size: :
<xarray.Dataset> Size: 3kB
Dimensions:  (foo: 1200, bar: 111)
Coordinates:
  * foo      (foo) int16   2kB 0 1 2 3 4 5 6 ... 1194 1195 1196 1197 1198 1199
  * bar      (bar) int16 222B  0 1 2 3 4 5 6 7 ... 104 105 106 107 108 109 110
Data variables:
    *empty*

<xarray.DataArray 'foo' (foo: 1200)> Size: 2kB
array([   0,    1,    2, ..., 1197, 1198, 1199], dtype=int16)
Coordinates:
  * foo      (foo) int16   2kB 0 1 2 3 4 5 6 ... 1194 1195 1196 1197 1198 1199

As there are many representations scattered accross the code, both in tests and doctests (800+ occurences when looking for the string     <xarray. in the codebase`, I would like to get your feedback on this representation before updating all of them.
Summary of the current repr:

For both Dataset and DataArray, the header is appended with a Size: {size} string
There should be at most 3 digits for size as long as we do not exceed  "YB" (Yotta, 10**24)
For the inline representation of a Variable of a Dataset, the size occupies a fixed 5-char width (as long as we do not exceed  "YB" (Yotta, 10**24) ): Examples:

[999kB]
[  8B ]

This padding may be irrelevant, as it only preserves layout in some specific case (same dimension tuple width and same dtype width). It could make sense to move the size before the dimension tuple and dtype and keeping its fixed width, or keep its current position but remove the padding.

Just a quick comment: @max-sixty wrote pytest-accept for this kind of thing. It's pretty great :)
I prefer (2) because dimension names are usually what i look for first in the variable repr.

Hello,
Thanks for the recommendation!
I pip install pytest-accept and then ensured my git working tree was clean.
I first tried using it on a single file with few doctests:
pytest --doctest-modules xarray/core/accessor_dt.py --accept

It correctly updated the doctests: git diff prints:

I really wish I would have read your comment @dcherian after getting hit with this change in my own CI and spending an hour tracking down each new difference (and only in 4 small example documents). Sphinx apparently really doesn't want to render docs in a consistent order...or maybe doctest isn't running the tests in the same order.
As someone who has never checked the raw bytes size of an array, I was surprised when I tracked down this change and saw so many people eager to have it. I guess that just shows how many use cases there are for something so "simple" (the repr).

Maybe we should add an option to opt-out? Or is it better to have a canonical repr?

Probably just one repr since it seems like enough people want this. If more people come here to complain then maybe revisit the idea?

FWIW I find the size per variable quite helpful. I'm often working with datasets with very differently sized variables.
We should do the utilitarian thing, very possibly I'm less representative

My main complaint is that the repr says something like "Size: 24B" but .size on xarray.DataArray refers to the number of elements, not the size in memory!

We don't technically need the term Size there; it's self-explanatory if we have the "15GB", no objection to removing that. Though I also don't think we need nbytes for the same reason...

Hello,
About display of individual variables' nbytes
Two different feedbacks here:

I also think this information is generally less relevant on individual coordinates/variables inside the repr, and would prefer not to include it there in favor of showing more data values (unless that data values are hidden, e.g., for a lazy array, in which case we could show memory usage instead).

and

FWIW I find the size per variable quite helpful. I'm often working with datasets with very differently sized variables.

I can add mine: I also find the size per variable helpful
Could a global option control this preference maybe? I see on the Configuration doc page that other options already exist to control the repr. A display_size_per_variable option or display_variable_nbytes could be added to suit everyone's needs.
:grey_question: What should be the default for display_variable_nbytes if such a global option is introduced?
:arrow_right: I would suggest True to keep the existing behaviour.
:grey_question: Regarding a different display behaviour according to the laziness of a variable, do you know a reliable attribute in xarray that can answer the question: is the variable lazy?
I had in mind that the current way was to check for the value of the chunks attribute, for Dask arrays at least
https://docs.xarray.dev/en/stable/generated/xarray.Dataset.chunks.html

Mapping from dimension names to block lengths for this dataset’s data, or None if the underlying data is not a dask array.

About the removal of the term Size

We don't technically need the term Size there; it's self-explanatory if we have the "15GB", no objection to removing that. Though I also don't think we need nbytes for the same reason...

I agree with that. For instance, dtypes are not prefixed, and the "Size: " prefix is absent from the variables' representations to gain space. We can assume that the suffix solely (B, kB, MB...) is a well-known hint that the number represents a memory footprint.
Proposal of new reprs
What do you think of this? :arrow_down:
For non-lazy arrays:
with xr.set_options(display_variable_nbytes=True):
    print(xds) # Existing behaviour, without the Size: prefix

<xarray.Dataset> 27B
Dimensions:  (x: 3)
Coordinates:
  * x        (x) float64 24B 10.0 20.0 30.0
Data variables:
    myvar    (x) uint8 3B 11 22 33

with xr.set_options(display_variable_nbytes=False):
    print(xds) # Only top-level nbytes is represented

<xarray.Dataset> 27B
Dimensions:  (x: 3)
Coordinates:
  * x        (x) float64 10.0 20.0 30.0
Data variables:
    myvar    (x) uint8 11 22 33

For lazy arrays:
with xr.set_options(display_variable_nbytes=True):
    print(xds) # Existing behaviour, without the Size: prefix

<xarray.Dataset> 27B
Dimensions:  (x: 3)
Coordinates:
  * x        (x) float64 24B 10.0 20.0 30.0
Data variables:
    myvar    (x) uint8 3B dask.array<chunksize=(1,), meta=np.ndarray>

with xr.set_options(display_variable_nbytes=False):
    print(xds) # The `myvar` nbytes is still shown because it is a dask array.

<xarray.Dataset> 27B
Dimensions:  (x: 3)
Coordinates:
  * x        (x) float64 10.0 20.0 30.0
Data variables:
    myvar    (x) uint8 3B dask.array<chunksize=(1,), meta=np.ndarray>

:grey_question:  Should the nbytes be integrated into the "class name tag" ? <xarray.Dataset 27B> for Dataset  and <xarray.DataArray (x: 3) 3B> (size last) or <xarray.DataArray 3B (x: 3)> (size first) for DataArray
In the examples I kept the nbytes out of the class name enclosed in chevrons (<xarray.Dataset> 27B and not <xarray.Dataset 27B>), as in the existing behaviour.
Also, I note that the fact that the nbytes is shown for a variable that is a dask array even when display_variable_nbytes=False could be a bit surprising. It would be less surprising if there was no global option in the first place. However, some might find the printing of nbytes per variable helpful even for non-lazy arrays and this would penalize them 🤔

This nbytes representation can be quite useful,
I'm curious is there a reason as to why it is not included in the html repr?
While dask arrays show their size, pure numpy arrays do not, and both have no top level complete size representation.


"typ" header parameter value is optional according to RFC7519 but is required in this library

What happened?
AWS Cognito access tokens don't have the "typ" header parameter value which seems to be compliant with the RFC7519 spec but this library errors if it is not present. From the validateJwtAccessToken() function we have the following logic:
function checkJwtType(expected, result) {
    if (typeof result.header.typ !== 'string' || normalizeTyp(result.header.typ) !== expected) {
        throw new OPE('unexpected JWT "typ" header parameter value');
    }
    return result;
}

Version
v2.10.3
Runtime
Node.js
Runtime Details
v20.11.1
Code to reproduce
{}

Required

[X] I have searched the issues tracker and discussions for similar topics and couldn't find anything related.
[X] I agree to follow this project's Code of Conduct

Absolutely same reasoning as in https://github.com/panva/oauth4webapi/issues/117#issuecomment-2027456482


Treat buttons differently than security alerts

When I hooked my X10 to Home Assistant with mochad_dispatch, I found that whenever I restarted, HA, it ran through the most recent button press of every button because the messages were sent with the retain flag.  I use some of my remote buttons to select sources on my Sonos, so there was chaos before things settled down!
This patch does two things:

Changes the topic for button presses to 'X10/host/button/device'
Sends button presses with QoS=0, retain=false

This agrees with Andy Stanford-Clark's suggestion in the MQTT Google Group at https://groups.google.com/d/msg/mqtt/rIp1uJsT9Nk/7YOWNCQO3ZEJ

Awesome!

very cool feature, elegant implementation.  Thanks!


Pass data-mh to line-height

Hi.
Is it possible to pass the calculated height to other style attributes like line height?
(This way it would be peace of cake to vertically align text).

You could probably do it using the callback event:
$.fn.matchHeight._afterUpdate = function(event, groups) {
    // do something after all updates are applied
}

You could loop through the elements that you need to apply this to, set their line-height to their height value?

I found another solution: http://zerosixthree.se/vertical-align-anything-with-just-3-lines-of-css/
Together with matchHeight.js this works great.

Hey awesome, I'll close this one then thanks.


Add Helm Support in tty image

Helm is a very useful software.
It may be wonderful to add it in the tty image.
Refer to https://github.com/dtzar/helm-kubectl

fixed in https://github.com/cloudtty/cloudtty/pull/59 by @yankay


[ODS-5612] Add support for reading API Profile definitions from the EdFi_Admin database

--Insert script used in my testing
INSERT INTO dbo.Profiles
SELECT 'Sample-Profile-Resource-ExcludeOnly',
'

'
INSERT INTO dbo.Profiles
SELECT 'Sample-Profile-Resource-IncludeOnly',
'

'
INSERT INTO dbo.Profiles
SELECT 'Sample-Profile-Resource-ReadOnly','

'
INSERT INTO dbo.Profiles
SELECT 'Sample-Profile-Resource-WriteOnly','

'

After a discussion with @vimayya , we need to refactor the implementation a bit here. The ProfilesMetadataProvider class implements two distinct interfaces. The only reason these two interfaces were implemented by a single class is because we only had one implementation of IProfileMetadataProvider. However, now that we have 2 sources for this metadata, the abstractions need additional cleanup.
Here is how we'd like the functionality to split up:

Functionality related to providing the Profile/Resource names should be separated out into a ProfileResourceNamesProvider class which is the only class to implement IProfileResourceNamesProvider. That class will take a dependency on IProfileMetadataProvider which will need to expose an additional property (to the currently private dictionary) to make this separation possible. See the diagram above for the property signature.
The functionality that is core to the IProfileMetadataProvider interface and ProfileMetadataProvider implementation will remain in that class only. We will introduce a new IProfileDefinitionsProvider (in the same style of the existing IDomainModelDefinitionsProvider interface and implementations).
Behavior that was in ProfileMedataProvider related to locating profile definitions in embedded resources in assemblies will be moved to a new class EmbeddedResourceProfileDefinitionsProvider class that implements IProfileDefinitionsProvider, and the class for obtaining it from the Admin database will be called AdminDatabaseProfileDefinitionsProvider.
Then, both IProfileDefinitionsProvider implementations will be registered with Autofac and injected into the ProfileMetadataProvider constructor, removing the LoadProfilesFromDatabase configuration setting that was added with this PR.

I tested with 4 different profile for each one here in this file https://github.com/Ed-Fi-Alliance-OSS/Ed-Fi-Extensions/blob/main/Extensions/EdFi.Ods.Profiles.Sample/Profiles.xml only 2 profile added in UI and other profile row is failing with validation error
I was working with this today, and the issue occurs because the validation code hasn't encountered  profile definition documents that don't have at least 1 readable or 1 writable profile definition present. With each entry in the Admin database containing a single profile definition, any readable-only or writable only profile definition will result in an exception, preventing the profile document from being processed, thus the Profile definition is dropped from the available metadata. I have fixed this in additional commits added to the PR.


Add /windows endpoint for Windows configuration + /apple endpoint cleanup

[x] read the CONTRIBUTING guidelines
[x] raised a GitHub issue or discussed it on the projects chat beforehand
[ ] added unit tests
[ ] added integration tests
[ ] updated documentation if needed
[x] updated CHANGELOG.md

Closes #340.
/windows:

Allows download of pre-filled .reg file /windows/tailscale.reg
Includes CLI/command-prompt instructions as well

/apple changes:

Renamed the function that prints HTML to AppleConfigMessage to match WindowsConfigMessage.
The template for /apple used .Url, when the correct variable needed to be .URL. This has been fixed.

Looks sensible, just need the golangcilint to pass :)

@kradalby @juanfont latest commit fixed comments causing golangci-lint to fail 👍


Tab indicator incorrectly positioned if width changes (e.g. scrollbars)

Imagine a tab filled with content that needs a scrollbar to view it, then add a second short tab without much content and therefore no scrollbar.
#5803 might be another example of this bug.
Expected Behavior
Clicking tabs should update the tab indicator correctly, underlining the current one.
Current Behavior
Changing between tabs that cause the width to change (for example adding or removing a scrollbar) the indicator is incorrectly positioned. Triggering tabs.updateTabIndicator() will not fix the issue, but clicking the current tab again yes.
Possible Solution
Call the code responsible of the indicator after the tabs have been switched.
Steps to Reproduce

Access this CodePen example.
Switch between tabs, indicator will be placed incorrectly.

Context
While filling up information in my website and testing it I notice the bug. Which only occured on tabs with different amounts of content, being trigger by scrollbars appearing and disappearing.
Environment

Version used: v1.0.0-beta
Browser Name and version: Chrome 66.0.3359.139 (Build oficial) (64 bits)
Operating System and version: Windows 10 Home, 1803

Duplicate of #5868


Feat: Web Provider

Describe changes
Pallad needs to be able to connect to zkApps. Introducing a new provider that abstracts over the wallet and receive requests.

Merging.


hard guarantees about not downloading new JVM or Scala version

At the moment, specifying the JVM arg of corretto:11 will actually download corretto:<IP_ADDRESS>.1-- which is fine, but what happens if AWS releases corretto:<IP_ADDRESS>.1.1?  Will scala-cli check for and download it?  If so, it makes scala-cli unsuitable for use in containers, as while I can specify the version in the Dockerfile and make sure it downloads into the image, I cannot be sure a new sub-version will be released and scala-cli will download it on every container instance run.  The same for the scala version.
There needs to be a way to specify that scala-cli should not check for any new versions of the JVM or scala-- just make a best-effort to use what is specified and fail if not found.

There is a way to specify the JVM used with --java-home option and using javaHome directive.
E.g: //> using javaHome /Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home
or --java-home /Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home
JVM 17 is mandatory when using Bloop and it will be installed if the one provided has a lower version. You can opt for using scalac instead of Bloop with option server=false. Note that the bsp command always uses Bloop.


[Doc Improvement][RSC permissions needed to enable Meeting Stage sharing]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
96
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 33106a6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
96
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 044b530:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Don't print to console when dumping state for ViewStore.debug

While debugging something I realized that ViewStore.debug was printing too much to the console.

I would like to add that this will fix the problem that I introduced in version 0.28.0.
When you have an NSManagedObjectID in your state, the app will just crash when using .debug(), however, it seems like a problem inside the CustomDump package, you will have problems when trying to dump an NSManagedObjectID value.

@hadiidbouk Interesting. We're not sure why it would crash, but also in general it's probably not a good idea to store core data reference types in state. Instead you could hold onto the uriRepresentation() of the object id, and then convert it back into an object id using managedObjectID(forURIRepresentation:).
But, if you want to open a discussion with more details on the crash there may be some people in the community who can help.

Yes you are definitely right


Update CMMCPolicyMapping.csv

Fixes #
Proposed Changes

Has some merge conflicts that I am working offline to resolve.

@Ajeet Prakash @.***>,
Please let me know if there's anything I can help with. The CMMC Mapping passes GH checks and the current mapping is parsing as expected.
let CMMCMapping = externaldata(RecommendationDisplayName:string,ControlFamily:string,ControlNumber:string,MaturityLevelr:string) @.***://raw.githubusercontent.com/Azure/Azure-Sentinel/master/Sample%20Data/Feeds/CMMCPolicyMapping.csv] with (format="csv", ignoreFirstRecord=True);
CMMCMapping
@.***
Very Respectfully,
TJ Banasik
CISSP-ISSEP,ISSAP,ISSMP
Senior Program Manager
C&AI, Sovereign Cloud Security Group
<EMAIL_ADDRESS>(202) 658-8148
@.***
LinkedIn: In/tjbanasikhttp://www.linkedin.com/in/tjbanasik
From: aprakash13 @.>
Sent: Sunday, October 3, 2021 9:16 PM
To: Azure/Azure-Sentinel @.>
Cc: TJ Banasik @.>; Author @.>
Subject: Re: [Azure/Azure-Sentinel] Update CMMCPolicyMapping.csv (#3103)
Has some merge conflicts that I am working offline to resolve.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2FAzure-Sentinel%2Fpull%2F3103%23issuecomment-933070893&data=04|01|Thomas.Banasik%40microsoft.com|0dea81507aac495f1fdd08d986d48336|72f988bf86f141af91ab2d7cd011db47|1|0|637689069585721822|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RTCGhWrtvthO3RU2H7K5y9JXXkVkEcwbtqhnSMmeoyA%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAM6PREQ6OKVCLTRXVIZQUA3UFD54TANCNFSM5EV5YLMA&data=04|01|Thomas.Banasik%40microsoft.com|0dea81507aac495f1fdd08d986d48336|72f988bf86f141af91ab2d7cd011db47|1|0|637689069585731816|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=aqbHyUB16sp%2FtrP5f2GXY%2FqdTgSVYLfoe%2BMXXH%2ByGHY%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Thomas.Banasik%40microsoft.com|0dea81507aac495f1fdd08d986d48336|72f988bf86f141af91ab2d7cd011db47|1|0|637689069585731816|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NE4FplLbsANX%2FfjLddUp8bb79otQKSpBXLkZMX5%2FRUc%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Thomas.Banasik%40microsoft.com|0dea81507aac495f1fdd08d986d48336|72f988bf86f141af91ab2d7cd011db47|1|0|637689069585741807|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=awhUp2X1IcosWvfQ9M%2BMwdFifKlp1ICwDsI%2FWxtezn0%3D&reserved=0.


Import module only if is browser

I'm submitting a ...  (check one with "x")
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  <!-- Please check the repository for a similar issue or PR before submitting -->
[x] Support request => <!-- Please check the repository for a similar issue or PR before submitting -->
[ ] Feature request
[x] Documentation issue or request

If I have a module that imports module that is not compatible with server side rendering (for example uses window) how I can check if is browser before I declare @NgModule?

@vlados please have a look at https://github.com/ng-seed/universal/blob/master/src/client/app/components/layout/main.component.ts#L13.
You will find the implementation of isBrowser flag on the component code. Then go to the tempate, and you will notice the isBrowser flag in a ngIf statement. This way, you might avoid use of non-universal compliant statements.


Ansible ZipShip Teams and Repository

Create the following assets for the Ansible ZipShip effort
Create Teams
Each of the following teams will include the following users as maintainers

@A-Beck
@sabre1041
@bontreger
@brady-spiva
@jeffcpullen
@kyetter

Top Level Team

ansible-zipship

Subteams

ansible-zipship-mergers
ansible-zipship-admins

Organization or repository
Create new ansible-zipship-images repository
Associate  ansible-zipship with read access, ansible-zipship-admins with admin access and ansible-mergers with write access

Review request:
isr
cc: @redhat-cop/container-cop-core

Done with #607 - closing


Cleanup dependency cruft

A few things from #80 that had been languishing for a while.
aioredis & redis-py got removed upstream even before 1.13.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

As this passed CI in #80 already, I'm putting this one in.


[stable/kibana] Fixed issue in introduced in merge

Signed-off-by: towmeykaw<EMAIL_ADDRESS>

/assign @compleatang I hate merges :D

/assign @compleatang

I hate merging :D

Yeah. A few of these PRs were stepping on each other's toes and I don't have time to deeply test unfortunately :-(
/ok-to-test

/lgtm
/approve


Feat: Add dual sorting feature in resource page "Sort by" dropdown

Describe the feature
Currently we have only one type of sorting, for Name we sort from A to Z and for Date we sort it from latest to last
I want to add a feature where user can sort from Z-to-A also and last to latest also
Implementation:
When user clicks on the Name it sorts by default from A to Z, now user can again click on Name to sort it in the other way that is Z-to-A
Similar for Date feature also.
Add ScreenShots
BEFORE

AFTER

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSoC'24 contributor
[X] I want to work on this issue

@vansh-codes go ahead


Update the Definition file of D3.js to v3.5.3

D3.js has been updated to v3.5 “Mount Sharp” 2 months ago. There are some new methods in the  latest version. So I try to write the definition of these new methods:

Add signature of named transitions for concurrent animations on the same element.
Add signature of d3.layout.pie padAngle, which preserves approximate relative area of arcs.
(ps. I think there maybe something wrong with the definition of startAngle,endAngle and padAngle. Based on the source code of d3.layout.pie, the param i:number is useless and should not be pass in. I not quite sure about it. Need more discussion.)
Add signature of d3.svg.arc cornerRadius for arcs with rounded corners.
Add signature of d3.behavior.zoom duration for customizing or disabling double-click transitions.
Add signature of d3.map(array, accessor) for constructing a map by key from an array.
Add signature of d3.scale.ordinal rangeRoundPoints, a rounded version of rangePoints.
Add signature of d3.geom.quadtree find for efficient 2D search.
Add signature of d3.variance and d3.deviation.
Add signature of d3.shuffle(array, lo, hi) for shuffling a subset of an array.

Besides, I add and fix some signature in the previous version:

Add the signature of InterpolateZoom
Add a overload definition of Transition.each(..)
Add a overload definition of quadtree(Array<number[]>)
Fix some signature in D3.Base

@borisyankov @bluong @nestalk could you review this PR?

Clear and remove some redundant signature by change the parameter index to optional.
Fix the signature of startAngle, endAngle and paddingAngle in PieLayout, parameter index should not be passed in.

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


Fix setupToolbar crash

java.lang.RuntimeException:
at android.app.ActivityThread.performLaunchActivity (ActivityThread.java:2984)
at android.app.ActivityThread.handleLaunchActivity (ActivityThread.java:3045)
at android.app.ActivityThread.handleRelaunchActivity (ActivityThread.java:4978)
at android.app.ActivityThread.-wrap21 (ActivityThread.java)
at android.app.ActivityThread$H.handleMessage (ActivityThread.java:1648)
at android.os.Handler.dispatchMessage (Handler.java:102)
at android.os.Looper.loop (Looper.java:154)
at android.app.ActivityThread.main (ActivityThread.java:6776)
at java.lang.reflect.Method.invoke (Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run (ZygoteInit.java:1520)
at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1410)
Caused by: java.lang.IllegalStateException:
at android.app.Activity.setActionBar (Activity.java:2489)
at com.veniosg.dir.activity.BaseActivity.setupToolbar (BaseActivity.java:71)
at com.veniosg.dir.activity.SearchableActivity.onCreate (SearchableActivity.java:48)
at android.app.Activity.performCreate (Activity.java:6955)
at android.app.Instrumentation.callActivityOnCreate (Instrumentation.java:1126)
at android.app.ActivityThread.performLaunchActivity (ActivityThread.java:2927)

is this related?

please don't release a new version until this is fixed... new versions of something that is broken are nonsense.

This was a very low occurrence issue and according to Google play it has
not happened in 1.5.0 at all. It is scheduled for 1.5.1 (next version)
On Mon, 1 Jan 2018, 9:03 am Luraktinus<EMAIL_ADDRESS>wrote:

please don't release a new version until this is fixed... new versions of
something that is broken are nonsense.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/veniosg/Dir/issues/80#issuecomment-354639622, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABmcZAhPNh6Ut2aoSMAjJQF2GCiyAaLGks5tGINAgaJpZM4Q35rg
.

Closing as this does no longer apply. It only affected the old search, which has been completely rewritten in 1.5.0. No related crashes reported since release.


Disable expand for empty cells

Resolves: #147

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

Project coverage is 99.77%. Comparing base (b6bda16) to head (c09f242).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #169   +/-   ##
=======================================
  Coverage   99.76%   99.77%
=======================================
  Files         163      163
  Lines        2592     2616   +24
  Branches      588      593    +5
=======================================
+ Hits         2586     2610   +24
  Misses          6        6

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Implemented onBufferEnd cb prop for FilePlayer, YouTube and Facebook

One suggested tweak to the readme, and onBufferEnd needs to be added to index.d.ts, but otherwise I think we are good 👍

done :)

Nice work 👍 Apologies for the delay.


How to create fusion_pos file?

I want to detect gene fusion in my single cell sample,so can I use the fusion function in the celescope?And how can I create or get the fusion_pos file when I use the mk_ref function in celescope fusion?

You can get the fusion_pos file of the pre-defined blood panel at: https://github.com/singleron-RD/CeleScope/tree/dev/celescope/data/fusion/blood_1
--fusion_pos fusion position file. A two column tab-delimited text file with header.
"pos" is the end postion of the first gene(1-based).
e.g.
name	pos
PML_3	183
PML_4	254
PML_5	326
PML_6	204
https://github.com/singleron-RD/CeleScope/blob/dev/docs/fusion/mkref.md


Apple Reminders: Improve NLP for the end date of recurring reminders

Description
Improve NLP parsing in the Quick Add Reminder command by adding an endDate for recurring reminders.
Fixes #9517
Screencast
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Has the new prompt yielded good results for you? I usually get a due date set when all I wanted was the end date. Here's an example with Cook meals until this thursday.

Hi @thomaslombart,
I had good results when an interval was included. However, your query is missing the interval, and the prompt specifies "this Thursday" as the due date. For example, you can say "Cook meals every day until Thursday." The interval will be recognized and the end date will be determined.
Should I include additional examples for the prompt?

Yeah maybe adding one or two example should be enough.


Reaction is changed when app goes from background to foreground.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a conversation and give reaction to a message
Remove the reaction and make the app goes to background.
Open the app again and add react to that message. Repeat step 2.
Notice that reaction is changed to previous reaction and sometimes reaction disappears when the app goes from background to foreground.

Expected Result:
Reaction shouldn’t change when the app goes from background to foreground
Actual Result:
Reaction is changed to previous reaction and sometimes reaction disappears when the app goes from background to foreground.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.86-1
Reproducible in staging?: Need reproduction
Reproducible in production?: Need reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/227057464-d0256716-2aa1-4685-b21e-d9732ce1507b.MP4
Expensify/Expensify Issue URL:
Issue reported by: @hungvu193
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1679366712835639
View all open jobs on GitHub

Triggered auto assignment to @arielgreen (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Cannot reproduce.


Minimize to tray bug

Windows 10 64-bit
Each time when the application is minimized to tray, the tray receive a new icon

Related to #44, will be fixed together


Mission feasability check status

Currently, the mission feasibility check is made when assigning a mission to the UAV. However, if the check fails we still get a success status on the upload_mission call even though the check might have failed. The results of the check are printed to the status text telemetry, but this is difficult to parse.
There should be a way to query the last mission feasibility check results (possibly just pass/fail, but preferably more detail), and a way to explicitly run the checker. In the case that a mission parameter has changed (such as a geofence), the user may want to re-check the mission to ensure it is still feasible. We may also want to check it to verify before calling start_mission.
@hamishwillee  https://discord.com/channels/1022170275984457759/1129508414414925994

Discussed this in the MAVLink call.
Proposal is a message that provides information about a feasibility check. The message is emitted whenever the feasibility checker is run - after an upload or when the user decides they want to start a mission. We might also have a command to explicitly run the feasibility checker.
@jperez-droneup Would you be interested in defining such a message in development.xml - and possibly providing an implementation?
Note, an alternative would be to standardize the format of STATUSTEXT or event messages for this purpose. That is a valid approach too, but given it would have to happen in two places is more problematic.

@bkueng could this be covered by events already?

Yes as part of the arming reporting. You can check generically if you can run the mission mode. If it fails you get a list of reasons why you cannot run the mission. One of them is a missing valid mission: https://github.com/PX4/PX4-Autopilot/blob/main/src/modules/commander/HealthAndArmingChecks/checks/modeCheck.cpp#L120
If I recall correctly there was also a push in PX4 to directly run feasibility checks on upload and fail the upload accordingly.

fail the upload accordingly.

Interesting. but we don't have an error to report that, see: https://github.com/mavlink/mavlink/pull/1710

@bkueng could this be covered by events already?

ArduPilot don't use events yet - they use STATUSTEXT of some format for most things. While I'd be very happy to use events for this kind of thing if it was standardized, until events is widely supported AND the particular events for feasibility are standardized in some way, not sure how we can do so.
So I'm still keen on a message, and possibly a command to trigger feasibility checks on request. Happy for some kind of counter proposal.

It seems like a message would be the easiest way to implement this given the current infrastructure and taking into account Ardupilot. Is this something you would be able to put together?

I'm not opposed but I'm busy on other work. If you need to move this up my priority list, you can always consider sponsoring me.

@hamishwillee and I just talked about this on the MAVLink call and found that the events system would be the right way to go about this. We might need @bkueng's help to get this drafted properly though.

Re the comment above, the idea is inspired by @bkueng implementation of prearm checks in PX4 using Events. From the docs:

Note that internally PX4 runs arming checks at 10Hz. A list of the failed checks is kept, and if the list changes PX4 emits the current list using the Events interface. The list is also sent out when the GCS connects. Effectively the GCS knows the status of prearm checks immediately, both when disarmed and armed.

I would say might be the right way to address this.
What events give us is a way to define both standardized and custom feasibility checks. The events can also be translated and shared with the ground station in a standard way.
What continuously running the checks does is give us a way to clear the failed checks as needed.
It's cool, but only PX4 has implemented Component Information and EVENTs. I'm also not sure that the mission checks can be run continuously at high enough rate.
Instead of events you could have a message with bitmask fields for standard and custom checks failing and stream this, or emit on change.
Instead of continuously running checks you could have a command  that triggers the check to be run and emits either events or above message.
I think events are a really good way to go for this, but not sure what it means to "standardize" this behaviour so flight stacks and GCS can implement to it without having to know about each other. Perhaps it is as simply as defining an event category.

but not sure what it means to "standardize" this behaviour so flight stacks and GCS can implement to it without having to know about each other

There's a common event namespace intended for that: https://github.com/mavlink/libevents/blob/main/events/common.json#L52


OpenTelemetry Metrics: Adds support to collect request level metrics

Description

Added new flag in CosmosClientTelemetryOptions i.e. IsClientMetricsEnabled, to enable/disable metrics. By default, it would be disabled. (inspired from Java SDK https://github.com/Azure/azure-sdk-for-java/blob/5bc07ca75c7c0520c1098b5a6264258b6e043435/sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/models/CosmosClientTelemetryConfig.java#L61)
If enabled, collecting below metrics ref. https://github.com/open-telemetry/semantic-conventions/pull/1438

ref. Java Metric Doumentation, https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/cosmos/azure-cosmos/docs/Metrics.md
ref. Discussion with open telemetry community https://github.com/open-telemetry/semantic-conventions/pull/1438
Type of change

[] New feature (non-breaking change which adds functionality)

What's the performance impact with disabled and enabled E2E?


Impossible to start kubernetes cluster on azure

Rancher Version:
Rancher v1.1.4
User Interface  v1.1.14
Rancher Compose v0.8.6
Docker Version:
Docker version 1.12.3, build 6b644ec
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 16.04 - Azure A2 machine - Resource Group
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single Node Rancher (used for testing purpose)
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:

Add a new Kubernetes environment in Rancher
Add a new VM in azure resource group ( same resource group where rancher machine is installed)
In rancher, go to Add Host -> Custom and copy and paste the command in the terminal of the new VM created
Service begin to be created but stuck at 7/9

Results:
No logs in kubelet and proxy. So it's impossible to get any information about the origin of the problem.

Expected:
kubelet and proxy containers should be able to start to create an available Kubernetes Cluster on Azure.

Same here, although not on Azure. I try to build a kubernetes cluster on KVM and RancherOS 0.70, but the kubernetes-proxy- and kubelet-containers are stuck. Since there are no logs it's very hard to debug.

@AlexGrs can you share the output from the etcd logs?

@timbordemann ROS 0.7.0 uses a more modern version of docker than what rancher 1.1.4's version of k8s (1.2.4) supports. You should be able to either downgrade user docker in ROS or go with ROS 0.4.5... let me know if that gets you up and running. I'll close this out for now, if you guys have further issues after aligning versions, feel free to reopen or open a new issue.

In ROS 0.7.0 you can do:
#cloud-config
rancher:
  ...
  docker:
     engine: docker-1.10.3
 ...

To swap User Docker versions.


[Thruster plugin] Flip sign direction.

Flip the sign direction of the thruster plugin/XML of joint axis to make it move forward when we give a positive thrust command.

Has this been fixed / is the failure tracked elsewhere / can this be closed?

I'm working on it as part of #48 and #37

See https://github.com/osrf/lrauv/pull/76


Snap 0.3.13 missing libcudart.so.12

What is the issue?
I am trying ollama for the first time.  Since there was a snap available, I tried that first.  I can download llama 3.2, but when I attempt to run it, I get:
(venv) ed@banana ~/P/re-copilot (dev)> ollama run llama3.2
Error: llama runner process has terminated: exit status 127

Here are the logs from snap logs ollama:
2024-11-05T10:38:44-05:00 ollama.listener[631587]: time=2024-11-05T10:38:44.915-05:00 level=INFO source=server.go:399 msg="starting llama server" cmd="/tmp/ollama2989640063/runners/cuda_v12/ollama_llama_server --model /var/snap/ollama/common/models/blobs/sha256-dde5aa3fc5ffc17176b5e8bdc82f587b24b2678c6c66101bf7da77af9f7ccdff --ctx-size 8192 --batch-size 512 --embedding --log-disable --n-gpu-layers 29 --parallel 4 --port 34001"
2024-11-05T10:38:44-05:00 ollama.listener[631587]: time=2024-11-05T10:38:44.915-05:00 level=INFO source=sched.go:449 msg="loaded runners" count=1
2024-11-05T10:38:44-05:00 ollama.listener[631587]: time=2024-11-05T10:38:44.915-05:00 level=INFO source=server.go:598 msg="waiting for llama runner to start responding"
2024-11-05T10:38:44-05:00 ollama.listener[631587]: time=2024-11-05T10:38:44.916-05:00 level=INFO source=server.go:632 msg="waiting for server to become available" status="llm server error"
2024-11-05T10:38:44-05:00 ollama.listener[631587]: /tmp/ollama2989640063/runners/cuda_v12/ollama_llama_server: error while loading shared libraries: libcudart.so.12: cannot open shared object file: No such file or directory
2024-11-05T10:38:45-05:00 ollama.listener[631587]: time=2024-11-05T10:38:45.166-05:00 level=ERROR source=sched.go:455 msg="error loading llama server" error="llama runner process has terminated: exit status 127"
2024-11-05T10:38:45-05:00 ollama.listener[631587]: [GIN] 2024/11/05 - 10:38:45 | 500 |  400.927983ms |       <IP_ADDRESS> | POST     "/api/generate"
2024-11-05T10:38:50-05:00 ollama.listener[631587]: time=2024-11-05T10:38:50.312-05:00 level=WARN source=sched.go:646 msg="gpu VRAM usage didn't recover within timeout" seconds=5.145569241 model=/var/snap/ollama/common/models/blobs/sha256-dde5aa3fc5ffc17176b5e8bdc82f587b24b2678c6c66101bf7da77af9f7ccdff
2024-11-05T10:38:50-05:00 ollama.listener[631587]: time=2024-11-05T10:38:50.561-05:00 level=WARN source=sched.go:646 msg="gpu VRAM usage didn't recover within timeout" seconds=5.39517205 model=/var/snap/ollama/common/models/blobs/sha256-dde5aa3fc5ffc17176b5e8bdc82f587b24b2678c6c66101bf7da77af9f7ccdff
2024-11-05T10:38:50-05:00 ollama.listener[631587]: time=2024-11-05T10:38:50.811-05:00 level=WARN source=sched.go:646 msg="gpu VRAM usage didn't recover within timeout" seconds=5.644657486 model=/var/snap/ollama/common/models/blobs/sha256-dde5aa3fc5ffc17176b5e8bdc82f587b24b2678c6c66101bf7da77af9f7ccdff

It seems like the problem is that libcudart.so.12 is missing.  Should it be getting that from the host, or the snap?
OS
Linux
GPU
Nvidia
CPU
Intel
Ollama version
0.3.13 snap

I do seem to have cudart12.so on my host:
ed@banana ~/P/re-copilot (dev)> sudo ldconfig -p | fgrep cudart
        libcudart.so.12 (libc6,x86-64) => /usr/local/cuda/targets/x86_64-linux/lib/libcudart.so.12
        libcudart.so (libc6,x86-64) => /usr/local/cuda/targets/x86_64-linux/lib/libcudart.so

It's normally bundled with the application to prevent library version skew.  snap is not the usual way to install ollama on linux, you may have more success with the caonlical way: curl -fsSL https://ollama.com/install.sh | sh.

The snap packaging isn't authored or maintained by the Ollama maintainers so unfortunately we wont be able to fix this.  Hopefully the package identifies who created it and you can file an issue with them to update their packaging.  Until then, as Rick suggested, you can uninstall the snap package and use our official curl install script.

Thanks for clarifying.  The snap's page linked to the ollama's issue page, so I thought that meant it was an official project.


Fix APP reference in the app.yaml

:+1:

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Update Dutch (nl) translation

Includes new strings from latest versions.
"Undo" is really long in Dutch ("ongedaan maken"), so it has been translated similar to the official Google apps ("onged. maken").

@romannurik Makes sense ;). Removed the strings.


Dependency update: use the latest TypeScript 1.8

Current coverage is 100.00%

Merging #1474 into master will not affect coverage as of db443fa

@@            master   #1474   diff @@
======================================
  Files            1       1
  Stmts         3734    3734
  Branches       628     628
  Methods          0       0
======================================
  Hit           3734    3734
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of db443fa

Powered by Codecov. Updated on successful CI builds.


Add school details

Add liberty.txt with school details. This PR adds liberty.txt  file with the necessary school details for the JetBrains student license request.

@Alcain-jje I am sorry, the domain liberty.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Proxy ?

Hello I just have a quick question how to set up a proxy while using the bot ... i noticed that this is in init : proxy=None,
and then i found this part of a code :
def login(self, **args):
if self.proxy:
args['proxy'] = self.proxy
if self.api.login(**args) is False:
return False
self.prepare()
signal.signal(signal.SIGTERM, self.logout)
atexit.register(self.logout)
return True
I am kinda newbie in proxies etc. so I would like to ask how to set it up, thank you

try this
b.login(username ="username", password ="pass", proxy = "http://login:password@ip:port")


Update Cart.total_price minus: 5000 in custom code

Hi!
Loving cartjs, been using it for a long time on my stores. I am working on a count down to free shipping based on the cart.total.price. It should be pretty simple, but I can't get the cart.total_price to be updated based on cart update.
{% assign some_number = 50000 | minus: cart.total_price %}

You qualify for free delivery!
If you spend {{ 50000 | minus: cart.total_price | money }} more you will qualify for free delivery.

How may I update this so I don't have to refresh to see changes? I have tried something like this:

Hey I think you need to remove the colon after minus, which isn't intuitive since that's how it is in liquid
I'm doing the same thing and it's like this and it works
Congratulations. Your shipping is free.
Free US shipping over $50 — ${5000 | minus cart.total_price} to go


Postgresql Class path contains multiple SLF4J bindings.

Getting an error while trying to connect postgresql docker host to logstash container
logstash    | 04:33:18.805 [[main]-pipeline-manager] INFO  logstash.outputs.jdbc - JDBC - Starting up logstash    | SLF4J: Class path contains multiple SLF4J bindings. logstash    | SLF4J: Found binding in [jar:file:/usr/share/logstash/vendor/bundle/jruby/1.9/gems/logstash-output-jdbc-5.4.0/lib/org/apache/logging/log4j/log4j-slf4j-impl/2.6.2/log4j-slf4j-impl-2.6.2.jar!/org/slf4j/impl/StaticLoggerBinder.class] logstash    | SLF4J: Found binding in [jar:file:/usr/share/logstash/vendor/bundle/jruby/1.9/gems/logstash-output-jdbc-5.4.0/vendor/jar-dependencies/runtime-jars/log4j-slf4j-impl-2.6.2.jar!/org/slf4j/impl/StaticLoggerBinder.class] logstash    | SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation. logstash    | SLF4J: Actual binding is of type [org.apache.logging.slf4j.Log4jLoggerFactory] logstash    | 04:33:18.861 [[main]-pipeline-manager] INFO  com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Starting... logstash    | 04:33:19.015 [[main]-pipeline-manager] INFO  com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Start completed. logstash    | 04:33:19.039 [[main]-pipeline-manager] INFO  logstash.pipeline - Starting pipeline {"id"=>"main", "pipeline.workers"=>4, "pipeline.batch.size"=>125, "pipeline.batch.delay"=>5, "pipeline.max_inflight"=>500} logstash    | 04:33:19.075 [[main]-pipeline-manager] INFO  logstash.pipeline - Pipeline main started logstash    | 04:33:19.175 [Api Webserver] INFO  logstash.agent - Successfully started Logstash API endpoint {:port=>9600} logstash    | 04:33:19.227 [[main]-pipeline-manager] INFO  com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Shutdown initiated... logstash    | 04:33:19.273 [[main]-pipeline-manager] INFO  com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Shutdown completed. logstash    | 04:33:22.105 [LogStash::Runner] WARN  logstash.agent - stopping pipeline {:id=>"main"}
Environment

Version of plugin used: logstash-output-jdbc 5.4.0
Version of Logstash used:  logstash:5-alpine
Database engine & version you're connecting to: postgres docker image
config with loaded jar version-

`input {
stdin { }
}
filter {
}
output {
jdbc {
    driver_jar_path => '/usr/share/logstash/vendor/jar/jdbc/postgresql-42.2.5.jre7.jar'

	connection_string => 'jdbc:postgresql://db:5432/postgres?user=postgres&password=postgres'

    statement => [ "INSERT INTO log (host, timestamp, message) VALUES(?, CAST (? AS timestamp), ?)", "host", "@timestamp", "message" ]
}

}`

The double binding isn’t ideal but it shouldn’t shut down the app from my understanding. Put logstash into debug mode to see if you any better errors.
Sent from my iPhone

On 13 Mar 2019, at 04:41, Jaswinder Singh<EMAIL_ADDRESS>wrote:
Getting an error while trying to connect postgresql docker host to logstash container
logstash | 04:33:18.805 [[main]-pipeline-manager] INFO logstash.outputs.jdbc - JDBC - Starting up logstash | SLF4J: Class path contains multiple SLF4J bindings. logstash | SLF4J: Found binding in [jar:file:/usr/share/logstash/vendor/bundle/jruby/1.9/gems/logstash-output-jdbc-5.4.0/lib/org/apache/logging/log4j/log4j-slf4j-impl/2.6.2/log4j-slf4j-impl-2.6.2.jar!/org/slf4j/impl/StaticLoggerBinder.class] logstash | SLF4J: Found binding in [jar:file:/usr/share/logstash/vendor/bundle/jruby/1.9/gems/logstash-output-jdbc-5.4.0/vendor/jar-dependencies/runtime-jars/log4j-slf4j-impl-2.6.2.jar!/org/slf4j/impl/StaticLoggerBinder.class] logstash | SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation. logstash | SLF4J: Actual binding is of type [org.apache.logging.slf4j.Log4jLoggerFactory] logstash | 04:33:18.861 [[main]-pipeline-manager] INFO com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Starting... logstash | 04:33:19.015 [[main]-pipeline-manager] INFO com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Start completed. logstash | 04:33:19.039 [[main]-pipeline-manager] INFO logstash.pipeline - Starting pipeline {"id"=>"main", "pipeline.workers"=>4, "pipeline.batch.size"=>125, "pipeline.batch.delay"=>5, "pipeline.max_inflight"=>500} logstash | 04:33:19.075 [[main]-pipeline-manager] INFO logstash.pipeline - Pipeline main started logstash | 04:33:19.175 [Api Webserver] INFO logstash.agent - Successfully started Logstash API endpoint {:port=>9600} logstash | 04:33:19.227 [[main]-pipeline-manager] INFO com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Shutdown initiated... logstash | 04:33:19.273 [[main]-pipeline-manager] INFO com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Shutdown completed. logstash | 04:33:22.105 [LogStash::Runner] WARN logstash.agent - stopping pipeline {:id=>"main"}
Environment
Version of plugin used: logstash-output-jdbc 5.4.0
Version of Logstash used: logstash:5-alpine
Database engine & version you're connecting to: postgres docker image
config with loaded jar version-
`input {
stdin { }
}
filter {
}
output {
jdbc {
driver_jar_path => '/usr/share/logstash/vendor/jar/jdbc/postgresql-42.2.5.jre7.jar'

connection_string => 'jdbc:postgresql://db:5432/postgres?user=postgres&password=postgres'
statement => [ "INSERT INTO log (host, timestamp, message) VALUES(?, CAST (? AS timestamp), ?)", "host", "@timestamp", "message" ]

}
}`
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Support for Google Cast

What think about implement this?
My idea is to user can use your device to cast Pokémon Showdown battles to a TV and use smartphone to control. If approves, I will send a pull request with this feature.

An actual pull request for Google Cast would interest me, but there's no need to keep this issue open while you make such a pull request.


Fixing OSX El Capitan compilation problem

details please see commit message.
in short, I want to compile the code on my mac and faced some hurdles.
here is the issue link: https://github.com/Hopson97/HopsonCraft/issues/48
Nevertheless, I can compile but can't start the program.

Looks good, but how come it won't start? (

Got this messages when started. A blue screen appeared but nothing more.
I think there are multiple problems.

unable to create context
I've checked there's no 9.png in the folder

Any hint? :)
Gabriels-MacBook-Pro:HopsonCraft gabriel$ ./game
The requested video mode is not available, switching to a valid mode
Warning. Compatibility profile not supported on this platform.
Error. Unable to create the context. Retrying without shared context.
Warning. New context created without shared context.
Warning: The created OpenGL context does not fully meet the settings that were requested
Requested: version = 3.3 ; depth bits = 24 ; stencil bits = 8 ; AA level = 0 ; core = false ; debug = false ; sRGB = false
Created: version = 4.1 ; depth bits = 24 ; stencil bits = 8 ; AA level = 0 ; core = true ; debug = false ; sRGB = false
Failed to load image "Data/Textures/Menu_BG/9.png". Reason: Unable to open file

Aparently it can be fixed if you make the window non-fullscreen
Check the window creation in Display.cpp :)


Render array of objects vertically in a react bootstrap table

Hi Allen,
I have array of objects if I want to render it vertically like as below
my array is[
{
"month": "Jan",
"amount":"10.00"
},
{
"month": "Feb",
"amount":"20.00"
},
{
"month": "mar",
"amount":"30.00"
},
{
"month": "apr",
"amount":"30.00"
},
{
"month": "may",
"amount":"60.00"
},
]
I want as table like below month comes as  headers and amount comes as row
jan feb mar apr may
10   20  30   30   60

react-bootstrap-table already stop implement new feature, and start to forward to react-bootstrap-table2, we will consider to implement this feature on react-bootstrap-able2 > https://github.com/react-bootstrap-table/react-bootstrap-table2/issues/79


cant find the discord's user ID

I'm getting the following error while adding my account user ID, not sure what's wrong

help needed

Just pushed a fix to catch this error better.
This error is usually thrown when Lanyard failed to fetch your user data.
The 2 obvious checks I need to ask are this:

Check to make sure your Discord ID is correct (EG: NAME#0000)
Make sure you're in the Lanyard Discord server as that will actually allow for Discord presence detection.


Decay Function Showing Incorrect Results Where Length Greater Than 1

Which version are you running? The lastest version is on Github. Pip is for major releases.
3.99b0
Do you have TA Lib also installed in your environment?
Yes
Have you tried the development version? Did it resolve the issue?
Tested in development version.
Describe the bug
According to the documentation provided at https://tulipindicators.org/decay if length is 4 and signals are:
[0, 0, 0, 1, 0, 0, 0, 1]
the output of the decay indicator should be:
[0, 0, 0, 1, .75, 0.5, 0.25, 1]
instead the output is:
[0, 0, 0, 1, .75, 0, 0, 1]
Expected behavior
Decay should work for lengths greater than 1 as described in documentation.
Thanks for using Pandas TA!

Hello @PVGibson,
Thanks for finding this and letting me know. 😎
Kind Regards,
KJ

HI @twopirllc @PVGibson
I have fixed the issue and tried to optimise it too.
Data Used for benchmark: [81.59, 81.06, 82.87, 83.00, 83.61, 83.15, 82.84, 83.99, 84.55, 84.36, 85.53, 86.54, 86.89, 87.77, 87.29]
Perf Counter (Original Implementation): 0.003188986040186137
Perf Counter (Optimised Implementation: 0.0003332220367155969
Hope you find this helpful
Thanks

Hi @PVGibson & @sampanacharya,
I also converted them to numpy/numba to help with speed. It is available on the development branch.
Thanks for helping!
KJ


Is storage value of Option<()> supported?

We have few storage value with double map X, Y => Option<()> type to simulate an unordered set.
However I found out the .iter(key1) is not able to iterate all the values for key1.
Using getStorage RPC is able to read that storage did exists. getKeys is not able to iterate as the key wasn't exists.
Seems like the trie db is not able to distinguish between a non-exist key and an exist key with no value in some cases?
Change the type to Option<bool> works around the issue for me.
Example code:
https://github.com/laminar-protocol/laminar-chain/blob/f53fb6044ffa431993e4467f320f93af1b042ce2/modules/margin-protocol/src/lib.rs#L79
https://github.com/laminar-protocol/laminar-chain/blob/f53fb6044ffa431993e4467f320f93af1b042ce2/modules/margin-protocol/src/lib.rs#L667
This is not able to iterate.
The interesting thing is it works in unit test so we wasn't able to catch this issue before. Maybe something to do with low level db? The in memory db is able to handle but the real rocks db is not?

cc @thiolliere.


Twister exclude directories from coverage report

Is your feature request related to a problem? Please describe.
When I run unit tests with twister and generate a coverage report, this report contains files that are not part of the code under test like files in twister-out/, test/cmock, files from zephyr core, and mbedtls.
I would like to have the ability to filter out those directories from the coverage report. I found that something like that is already done inside twister script, but there are only few directories filter out, and there is no way to add custom filters.
https://github.com/zephyrproject-rtos/zephyr/blob/3f8d5c49b3fb54878504f50e87902eab113fc5e2/scripts/twister#L1350-L1357
I would like to see an additional command line argument that would ignore directories passed to it.
Describe alternatives you've considered
Currently, I use lcov -q --remove to remove those not wanted directories from coverage.info, but this requires regeneration of HTML.
I also tried to set --coverage-basedir to the dir with the source code that I'm interested, but this did nothing.

One thing I like to add: it seems, that filtering works well when using gcovr (--coverage-tool gcovr --coverage-basedir <out-of-tree-code-path). The feature seems to be missing only for lcov.