Jacobian within Jacobian not compatible with np.dot()

Dear autograd guys,
First, thank you for your excellent library; I am finding it really useful. Below is an issue that I believe exists with the implementation of jacobian().
Context: I am computing a Jacobian matrix inside a function for which I am calculating a Jacobian. That is, a Jacobian within a Jacobian. Note: this is not a Hessian calculation.
Issue: The format of the inner Jacobian matrix does not appear to be compatible with that of other vectors in the function when using np.dot().
Suspected reason: I think that the current format of the Jacobian matrix is not correct. A revised format is suggested.
Below is the code where this issue arises. Sorry that it includes a lot of extraneous detail, but hopefully the essence is not too difficult to see. That is, that the formats of internal Jacobian matrix H and vector xnext are incompatible for np.dot(H, xnext).
Thanks for your time, and sorry if I have done something stupid....
Tim.

Removed code from here, added as attachment below

Sorry, the formatting of the code above came out badly. Will try an attachment here.
autograd issue.txt

Glad you like autograd!
Can you give us a minimal example that reproduces the error, or tell us what the error is and provide a traceback?
Usually when you see an ndarray of dtype object with a bunch of autograd-boxed FloatNodes inside it means that some unwrapped numpy function was called on an autograd ArrayNode. Is it possible that your code is calling numpy functions (rather than autograd.numpy functions) somewhere?

For easier reference, I'm pasting the contents of your code attachment here:
import autograd.numpy as np
from autograd import grad, jacobian

def test_jacobian_calculation():
    t, dt = 1., 0.01
    x = np.array([0.1, 0.1, 0.1])
    jac = jacobian(crank_nicolson_step, 2)
    J1 = jac(lorenz, t, x, dt, ())  # lorenz(t, x) is a differential equation model

# Broken; it appears that the format of H is incompatible with that of xnext
def crank_nicolson_step(model, t, x, dt, args, w=0.5):
    xnext = initial_guess(model, t, x, dt, args)  # simple function, no numpy operations
    jacmodel = jacobian(crank_nicolson_step_twosided, 3)  # another simple function
    for _ in range(2):  # improve xnext, by iterative least-squares refinement
        zs = crank_nicolson_step_twosided(model, t, x, xnext, dt, args, w)
        H = jacmodel(model, t, x, xnext, dt, args, w)
        v = np.dot(H, xnext) - zs  # Code breaks here; cannot perform dot-product!!
        y = np.dot(H.T, v)
        Y = np.dot(H.T, H)
        xnext = np.linalg.solve(Y, y)
    return xnext

Here is the format of H:
>>> H
Out[1]:
array([[<autograd.core.FloatNode object at 0x0000000004727BE0>,
       <autograd.core.FloatNode object at 0x0000000004727AC8>,
       <autograd.core.FloatNode object at 0x0000000004727B70>],
      [<autograd.core.FloatNode object at 0x0000000004727B00>,
       <autograd.core.FloatNode object at 0x0000000004727BA8>,
       <autograd.core.FloatNode object at 0x0000000004727C18>],
      [<autograd.core.FloatNode object at 0x0000000004727CC0>,
       <autograd.core.FloatNode object at 0x0000000004727C50>,
       <autograd.core.FloatNode object at 0x0000000004727A90>]], dtype=object)

Here is the format of xnext:
>>> xnext
Out[1]: <autograd.numpy.numpy_extra.ArrayNode at 0x471eac8>
>>> xnext[0]
Out[2]: <autograd.core.FloatNode at 0x5a2fe10>

The problem is resolved if, instead of jacobian(), we use grad() and loop over the rows of the Jacobian matrix to compose H.
def crank_nicolson_step(model, t, x, dt, args, w=0.5):
    def crank_nicolson_step_i(i, *args):  # returns i-th scalar element
        return crank_nicolson_step_twosided(*args)[i]
    xnext = initial_guess(model, t, x, dt, args)
    gradmodel = grad(crank_nicolson_step_i, 4)
    for _ in range(2):
        zs = crank_nicolson_step_twosided(model, t, x, xnext, dt, args, w)
        H = np.array([gradmodel(i, model, t, x, xnext, dt, args, w) for i in range(len(x))])  # loop over scalar function to compose H
        v = np.dot(H, xnext) - zs
        y = np.dot(H.T, v)
        Y = np.dot(H.T, H)
        xnext = np.linalg.solve(Y, y)
    return xnext

Here is the format of H that is compatible with xnext:
>>> H
Out[1]: <autograd.numpy.numpy_extra.ArrayNode at 0x575a3c8>
>>> H[0]
Out[2]: <autograd.numpy.numpy_extra.ArrayNode at 0x5ad7eb8>
>>> H[0,0]
Out[3]: <autograd.core.FloatNode at 0x5ad76d8>

Attached is a (semi-)working version of the code insofar as it runs and then fails at the stated point.
test_autograd_jacobian_issue.txt

Here is the traceback:
Traceback (most recent call last):
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\numpy\numpy_extra.py", line 85, in new_array_node
return array_dtype_mappings[value.dtype](value, tapes)
KeyError: dtype('O')
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 62, in forward_pass
try: end_node = fun(*args, **kwargs)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 42, in list_fun
val = fun(*args, **kwargs)
File "C:/Users/tbailey/Documents/GitHub/iterated-linearised-smoothing/iterated_linearised_smoothing/sandbox/test_autograd_jacobian_issue.py", line 32, in crank_nicolson_step_broken
v = np.dot(H, xnext) - zs  # Code breaks here; cannot perform dot-product!!
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 167, in call
result = new_node(result, tapes)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 188, in new_node
return Node.type_mappings[type(value)](value, tapes)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\numpy\numpy_extra.py", line 87, in new_array_node
raise TypeError("Can't differentiate wrt numpy arrays of dtype {0}".format(value.dtype))
TypeError: Can't differentiate wrt numpy arrays of dtype object
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:/Users/tbailey/Documents/GitHub/iterated-linearised-smoothing/iterated_linearised_smoothing/sandbox/test_autograd_jacobian_issue.py", line 72, in
test_jacobian_calculation()
File "C:/Users/tbailey/Documents/GitHub/iterated-linearised-smoothing/iterated_linearised_smoothing/sandbox/test_autograd_jacobian_issue.py", line 65, in test_jacobian_calculation
J = jac(lorenz, t, x, dt, ())
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 50, in jacfun
start_node, end_nodes, tape = forward_pass(list_fun, args, kwargs, argnum)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 63, in forward_pass
except Exception as e: add_extra_error_message(e)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 392, in add_extra_error_message
raise_(etype, value, traceback)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\future\utils_init_.py", line 414, in raise_
raise exc.with_traceback(tb)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 62, in forward_pass
try: end_node = fun(*args, **kwargs)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 42, in list_fun
val = fun(*args, **kwargs)
File "C:/Users/tbailey/Documents/GitHub/iterated-linearised-smoothing/iterated_linearised_smoothing/sandbox/test_autograd_jacobian_issue.py", line 32, in crank_nicolson_step_broken
v = np.dot(H, xnext) - zs  # Code breaks here; cannot perform dot-product!!
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 167, in call
result = new_node(result, tapes)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\core.py", line 188, in new_node
return Node.type_mappings[type(value)](value, tapes)
File "C:\Users\tbailey\AppData\Local\Continuum\Anaconda3\lib\site-packages\autograd\numpy\numpy_extra.py", line 87, in new_array_node
raise TypeError("Can't differentiate wrt numpy arrays of dtype {0}".format(value.dtype))
TypeError: Can't differentiate wrt numpy arrays of dtype object

I can run that code without error. I think you might be experiencing a bug from an old version of autograd.
Can you try upgrading autograd? I think the pip version has a fix for this bug, though you can always get the github version for the latest stuff.

I just verified that the current pip version indeed has the jacobian fix in it. It looks like we fixed it in 1.1.4.

Yep. Works for me now. Thanks a lot.
Tim.

Great, glad that helped!


Support deleting previous objects in kubectl apply

Description
When creating objects with kubectl apply, it is frequently desired to make changes to the config that remove objects and then re-apply and have those objects deleted.  This feature will support pruning which deletes objects previously applied that no longer exist in the source config.
Progress Tracker

[ ] Alpha

[ ] Write and maintain draft quality doc

[ ] During development keep a doc up-to-date about the desired experience of the feature and how someone can try the feature in its current state. Think of it as the README of your new feature and a skeleton for the docs to be written before the Kubernetes release. Paste link to Google Doc: DOC-LINK

[ ] Design Approval

[ ] Design Proposal.  https://github.com/kubernetes/kubernetes/pull/33075
[ ] Decide which repo this feature's code will be checked into. Not everything needs to land in the core kubernetes repo. REPO-NAME
[ ] Initial API review (if API).  Maybe same PR as design doc. PR-NUMBER

Any code that changes an API (/pkg/apis/...)
cc @kubernetes/api

[ ] Identify shepherd (your SIG lead and/or<EMAIL_ADDRESS>will be able to help you). My Shepherd is<EMAIL_ADDRESS>(and/or GH Handle)

A shepherd is an individual who will help acquaint you with the process of getting your feature into the repo, identify reviewers and provide feedback on the feature. They are not (necessarily) the code reviewer of the feature, or tech lead for the area.
The shepherd is not responsible for showing up to Kubernetes-PM meetings and/or communicating if the feature is on-track to make the release goals. That is still your responsibility.

[ ] Identify secondary/backup contact point. My Secondary Contact Point is<EMAIL_ADDRESS>(and/or GH Handle)

[ ] Write (code + tests + docs) then get them merged.  ALL-PR-NUMBERS

[ ] Code needs to be disabled by default.   Verified by code OWNERS
[ ] Minimal testing
[ ] Minimal docs

cc @kubernetes/docs on docs PR
cc @kubernetes/feature-reviewers on this issue to get approval before checking this off
New apis: Glossary Section Item in the docs repo: kubernetes/kubernetes.github.io

[ ] Update release notes

[ ] Beta

[ ] Testing is sufficient for beta
[ ] User docs with tutorials
- Updated walkthrough / tutorial in the docs repo: kubernetes/kubernetes.github.io
- cc @kubernetes/docs on docs PR
- cc @kubernetes/feature-reviewers on this issue to get approval before checking this off
[ ] Thorough API review

cc @kubernetes/api

[ ] Stable

[ ] docs/proposals/foo.md moved to docs/design/foo.md
- cc @kubernetes/feature-reviewers on this issue to get approval before checking this off
[ ] Soak, load testing
[ ] detailed user docs and examples

cc @kubernetes/docs
cc @kubernetes/feature-reviewers on this issue to get approval before checking this off

FEATURE_STATUS is used for feature tracking and to be updated by @kubernetes/feature-reviewers.
FEATURE_STATUS: IN_DEVELOPMENT
More advice:
Design

Once you get LGTM from a @kubernetes/feature-reviewers member, you can check this checkbox, and the reviewer will apply the "design-complete" label.

Coding

Use as many PRs as you need.  Write tests in the same or different PRs, as is convenient for you.
As each PR is merged, add a comment to this issue referencing the PRs.  Code goes in the http://github.com/kubernetes/kubernetes repository,
and sometimes http://github.com/kubernetes/contrib, or other repos.
When you are done with the code, apply the "code-complete" label.
When the feature has user docs, please add a comment mentioning @kubernetes/feature-reviewers and they will
check that the code matches the proposed feature and design, and that everything is done, and that there is adequate
testing.  They won't do detailed code review: that already happened when your PRs were reviewed.
When that is done, you can check this box and the reviewer will apply the "code-complete" label.

Docs

[ ] Write user docs and get them merged in.
User docs go into http://github.com/kubernetes/kubernetes.github.io.
When the feature has user docs, please add a comment mentioning @kubernetes/docs.
When you get LGTM, you can check this checkbox, and the reviewer will apply the "docs-complete" label.

I worried that we're starting to turn the features repo into another kubernetes/kubernetes issues repo, and it will become as undigestible.
I would argue this one doesn't meet the bar for going in the features repo.

@mikedanese Isn't this done?

@mikedanese can you update the checkboxes in the PR body with the parts that are finished.

Ping.  Is there a docs PR for this?

Sorry, I missed this one.
Original issue: https://github.com/kubernetes/kubernetes/issues/19805
Implementation: https://github.com/kubernetes/kubernetes/pull/33075
There is command documentation like all of our other commands. Examples are being added in https://github.com/kubernetes/kubernetes/pull/37397 at which point it will have as much documentation as any other kubectl flag. It will be labeled as alpha in the v1.5 kubectl help output, pending a sign off from @pwittrock .
cc @MrHohn

lgtm.

Ok. I'll mark this "no docs needed" Thanks!

@mikedanese @pwittrock can you confirm that this item targets beta in 1.6?


Fix typo

Current coverage is 100.00%

Merging #61 into master will not affect coverage as of 1a6cc44

@@            master     #61   diff @@
======================================
  Files            1       1
  Stmts           41      41
  Branches         0       0
  Methods          0       0
======================================
  Hit             41      41
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 1a6cc44

Powered by Codecov. Updated on successful CI builds.

Thank you :grin:


[broadcast][V4L] DVB-T tuning fixes

https://jenkins.code.rdkcentral.com/job/github-triggered/1304/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/1306/ started


Don't have listview  objects

like this

Hi,
there isn't a list view object directly like this. I thought an object like this wouldn't be "handy" on a small TFT display. However I see many people use LittlevGL on larger displays with powerful MCUs.
Can you tell what is your usecase?  Where you would like to apply a listview object?

My English is not very good, using the translation tool, I do not know if you can understand. I'm using TI's MCU AM3352, not using OS

I see. I agree on this object could be useful with larger displays.
I add this object type to the v5.2 TODO list

As there was no activity here for a while I close this issue.
If you have remarks about this topic feel free to comment here and reopen the issue.


revamp CoreCoord and logical/physical coordinate systems

Change CoreCoord logical/physical as follows:

Coordinates explicitly specify a core type
TBD: if explicit coord type is a part of the type or an enum contained by the type (probably the latter)
Each core type contains its own logical coordinates in a grid (0..n,0..m)
Active/Idle ethernet, for example, are separate coordinate types each with their own runtime-determined grid
(idle ethernet may not be exposed through the API)
Host API provides translation from logical space to "device" space, device space may be virtual or physical coordinates
Device API can translate from device space to physical coordinates within the API if needed
Open: should device coordinates be typed such that, eg, DPRINT converts to physical (or even logical) coordinates?
Expose a device side API to convert from logical space to device space, either an add for virtual or a table lookup for physical
Conversion table is provided by firmware.  The table will be loaded to the device rather than compiled into the firmware (as it is presently), breaking all builds' dependence on harvesting

Pending: performance testing w/ physical/logical coords for kernels highly dependent on DRAM traffic
See also #5783
@abhullar-tt @aliuTT @tt-asaigal @yan-zaretskiy @yugaoTT @davorchap @pgkeller

Something to consider, code such as this:
uint32_t CreateSemaphore(
Program &program,
const std::variant<CoreRange, CoreRangeSet> &core_spec,
uint32_t initial_value,
CoreType core_type) {
with a CoreCoord containing an enum-type can mix and match types through the API.  This may be ok for some APIs and not for others, we don't want to introduce a high burden of API validation

we currently denote dram channels as 0..n and then map to a single CoreCoord, but we may want to uplift this to expose multiple coordinates for a given dram channel

this might be a tricky thing to test out, but I think in some cases the virtual coord might not work for dram sharded matmul (or other dram read critical ops).
If we use virtual coords, then we will always have the mapping in the image. If the harvested rows are row 5 and 7, then the mapped cores (W2, W3, W10, W11) will be shifted down to rows 8 and 9. Then we will have two cores in one row and causing read request congestions. the only way to solve this is use virtual channels for different cores, but it cannot be determined from host side (no physical coord info).

this might be a tricky thing to test out, but I think in some cases the virtual coord might not work for dram sharded matmul (or other dram read critical ops).
If we use virtual coords, then we will always have the mapping in the image. If the harvested rows are row 5 and 7, then the mapped cores (W2, W3, W10, W11, W8, W7) will be shifted down (second image). Then we will have multiple cores in one column, while dram bank is not next to it, and causing congestions.

@yugaoTT ok, took me a bit to process that, but I see the problem.  And so you are saying that to get optimal placement in this scenario the host needs to understand the physical coordinates?

I had a thought last night, here's a proposal:

Use strongly typed coordinates for the API, eg, EthCoreCoord, TensixCoreCoord, etc (names up for debate)
APIs are templated for the different coordinate types
Internally set an enum in the base CoreCoord w/ the type info (either at creation time or when the API is called)
Internally use generic coordinates w/ type info

@pgkeller yeah that's right.

@pgkeller yeah that's right.

I'm thinking we expose physical coords behind a "detail::" API.  In other words, it isn't a first class API but provided as a semi-supported way of getting better perf, particularly since we don't think it'll be needed for BH and beyond

I had a thought last night, here's a proposal:

Use strongly typed coordinates for the API, eg, EthCoreCoord, TensixCoreCoord, etc (names up for debate)
APIs are templated for the different coordinate types
Internally set an enum in the base CoreCoord w/ the type info (either at creation time or when the API is called)
Internally use generic coordinates w/ type info

This makes sense to me, it doesn't seem like we would want a logical view of heterogeneous core types. I don't think we would support buffers being interleaved across different core types and wouldn't create kernels across different core types in one api


error with toaster component

Hello
I am using the toaster component and after updating some packages, i am getting error with the Toaster component.

This is the ss of the errors. When i comment out the Toaster call in main layout, the site is working fine. Because of the error, nothing is visible on the site, just plain white color.
It was working fine till yesterday only, then i updated all the packages using npm-check-update and since then the Toaster is giving errors. I think there is some issue of sonner with the version of other packages.
Any help?

I can't reproduce this problem, could you share a https://stackblitz.com/ link that reproduces it?

I get this error in next app/ dir when use Toaster without use client directive.

Yea, use client is required for it to work.


Allow building as a relocatable library

I have a library project .gpr file that with "sdlada". If sdlada is built as a static library then I get the following error: cannot import static library project "sdlada".
If I change Library_Kind in build/gnat/sdlada.gpr to "relocatable", then the error disappears.

Is this still an issue, I removed the relocatable build ages ago in favour of just having a static build to reduce the number of libs people would need to distribute if they were to distribute their app/game binary only.

No longer an issue


Extraction Engine Improvements

[x] I have read and agree to the contributing guidelines for submitting new pull requests.

Describe your changes
Changed

BREAKING: Split BaseExtractionEngine.extract into extract and extract_artifacts for consistency with BaseSummaryEngine.
BREAKING: BaseExtractionEngine no longer catches exceptions and returns ErrorArtifacts.
BREAKING: JsonExtractionEngine.template_schema is now required.
BREAKING: CsvExtractionEngine.column_names is now required.
JsonExtractionEngine.extract_artifacts now returns a ListArtifact[JsonArtifact].
CsvExtractionEngine.extract_artifacts now returns a ListArtifact[CsvRowArtifact].

Issue ticket number and link
NA

📚 Documentation preview 📚: https://griptape--1097.org.readthedocs.build//1097/

Gonna hold off on merging this until some of the artifact changes are settled.

@vachillo bump on this since it's an oldie


how to configure the storageBackend to use elasticsearch

Description
I want to aggregate multicluster event by kelemetry  as mentioned in #136
but I confused how to configure the storageBackend to use elasticsearch. I met errors when I set storageBackend.type to elasticsearch(I'm using kelemetry-chart-0.2.2):

Error: UPGRADE FAILED: template: kelemetry-chart/templates/frontend.deployment.yaml:57:15: executing "kelemetry-chart/templates/frontend.deployment.yaml" at <include "kelemetry.storage-plugin-options" .>: error calling include: template: kelemetry-chart/templates/_helpers.yaml:263:4: executing "kelemetry.storage-plugin-options" at <include "kelemetry.storage-plugin-options-raw" .>: error calling include: template: kelemetry-chart/templates/_helpers.yaml:276:27: executing "kelemetry.storage-plugin-options-raw" at <include "kelemetry.storage-options-raw" .>: error calling include: template: kelemetry-chart/templates/_helpers.yaml:299:4: executing "kelemetry.storage-options-raw" at <include "kelemetry.storage-options-raw-stateless" .>: error calling include: template: no template "kelemetry.storage-options-raw-stateless" associated with template "gotpl"

Is there any example about it?
User story
No response

Fixed in #132, please use v0.2.3 of the chart.


[Front] Fix prismJs highlight language

What does this PR do?
Remove prismJs from npm dependencies and add back custom prismJs lib in src folder in order to have a lighter bundle.min.js file and fix highlight for all languages we support.

 Snippets tests reports   SDK Version Tests overview      v6        v5                           :heavy_check_mark:                            v1        v1        v1


JuliaCon 2023 talk on OndaBatches

Not sure this is the best place for this but seems just as good as anywhere else!  not checking in the rendered slides since those should go on a gh-pages branch and I dunno how to do that in a way that places nicely with documenter
We may want to add a link to the actual talk recording once that's upon youtube.

@jrevels and @alexmchan , tagged you for review because this is basically a question of "is this repository an appropriate home for this content; otherwise I can push it to a separate repo like I did for https://github.com/kleinschmidt/juliacon2020

Okay, actually I think it's possible to put the rendered content into the gh-pages branch in a folder that documenter won't touch.


pass build information to extension

This PR is related to #115

Made changes to extension.ts to log results.

What are the results?

I'm worried that  process.env.BUILD_ENV and process.env.BUILD_NUMBER aren't reliable. Any idea how to make them have intended behavior? If not, you can just change them to strings ('development' and '1.0.0').
Can you remove vinyl-fs and fix formatting?

hi @andrewpareles , how do you want the formatting to be?


Please add evironment variable in CMS jobs for frontier debugging

At at least one point in the past I am quite sure WMAgent used to do this, but I just looked at some job logs where it is clearly not set.  Please set and export the environment variable FRONTIER_LOG_LEVEL=warning for CMS jobs in a future release of WMAgent.

@DrDaveD, I don't recall we have this environment variable set before. Anyway, we can add that. But I was wondering what is accessing this variable?

Must be Frontier being run inside CMSSW

In this case shouldn't CMSSW or Frontier set this then (which one is setting default value)? Or is it (warning level) only for the jobs run by Agents?

Yes it is the frontier client running inside CMSSW.  We don't want these messages to show up when people run cmsRun interactively, but they are helpful for debugging job failures.  There's a current issue that it would have helped with debugging, and the messages have helped with debugging many problems in the past.


please improve the documentation

Please improve the documentation of the list of integrations:

Elements Transformation (for markdown-it)
Inline Link Previewing (for VitePress)
Changelog & File history (for VitePress)

I want to try these integrations but found no docs to read😂😂

I'm so sorry!
Element Transformation
This markdown-it plugin is intended to be used as replacing all the <a> elements to <InlineLinkPreview> components.
This is the living example code where we use it as references:
https://github.com/nolebase/integrations/blob/08c4e47f43f4678e2138401f8b62a1966a48b6ba/docs/.vitepress/config.ts#L147-L169
However, such useage is a temporary workaround due to the fact that I haven't refactor Inline Link Previewing plugin to listen the mouse hovering event of all the <a> elements as singleton instance across all pages for both better accessibilities and compatibilities (since it replaces all the <a> elements and may break the markdown processing pipelines for advanced users).
Inline Link Previewing
This vitepress plugin is an implementation of Obsidian's link previewing functionalities.
To use it, you need to combine with Element Transformation:
https://github.com/nolebase/integrations/blob/08c4e47f43f4678e2138401f8b62a1966a48b6ba/docs/.vitepress/config.ts#L147-L169
And use the components as follows:
https://github.com/nolebase/integrations/blob/08c4e47f43f4678e2138401f8b62a1966a48b6ba/docs/.vitepress/theme/index.ts#L57
Don't forget to include the stylesheets as well:
https://github.com/nolebase/integrations/blob/08c4e47f43f4678e2138401f8b62a1966a48b6ba/docs/.vitepress/theme/index.ts#L28
Changelog & File history
This vitepress plugin read the git log as file history and renders through components with both Contributors section and Changelog section available in Markdown file.
The reason why we haven't made a documentation for it is because we have another pending issue #45 to improve the overall performance and user experience, we might change the API in the nearby future when refactoring.
This is roughly the most complex one to configure with.
First, you need a vite.config.ts or vite option in config property of vitepres, and then configure these two Vite plugins to read, prepare the git log data for components to render with, transform the Markdown file with the two new additional sections:
https://github.com/nolebase/integrations/blob/08c4e47f43f4678e2138401f8b62a1966a48b6ba/docs/vite.config.ts#L11-L35
And use the components as follows:
https://github.com/nolebase/integrations/blob/08c4e47f43f4678e2138401f8b62a1966a48b6ba/docs/.vitepress/theme/index.ts#L69-L84
Don't forget to include the stylesheets as well:
https://github.com/nolebase/integrations/blob/08c4e47f43f4678e2138401f8b62a1966a48b6ba/docs/.vitepress/theme/index.ts#L30

Sorry about the lack of documentations of our plugins. Feel free to ask any questions here and even provide us feedback about the not-well-designed plugins.
You are welcome to contribute documentations about these plugins if you want (either one language you comfort with)

Thx. I will try it latter


Support transposed weight instead of transpose input/output

This commit added the transposed flag about the input/output layout.
If it is set to true (By default is set to true) the input/output needs to be transposed.

Based on our face to face discussion, this fix is good. The problem is the softmax implmentatin so I'll merge this and create another fix for softmax.


ci: update golangci to v1.27.0

There are various improvements in golangci in window of v1.21.0
to v1.27.0
Reference# https://github.com/golangci/golangci-lint/releases
Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

Please also update scripts/Dockerfile.test with the same version

@nixpanic cant we make it as an argument for docker file? maybe a separate patch.

@nixpanic cant we make it as an argument for docker file? maybe a separate patch.

We could. But I am considering to store all environment variables in a separate file, so that we only have a single place where these are set. That definitely should be its own PR.
For now, when we change a version, it needs to be changed everywhere, unfortunately.

Wow, and there are many lint errors! These should be addressed cleanly with lots of separate commits.

OK.. So, I can update the DockerFile too. But again the CI is not going to pass with it.  The lint errors caught here would be handled in another PR itself or commits.. I can do both here. Let me know the choice @Madhu-1 @nixpanic

OK.. So, I can update the DockerFile too. But again the CI is not going to pass with it. The lint errors caught here would be handled in another PR itself or commits.. I can do both here. Let me know the choice @Madhu-1 @nixpanic

another commit in this PR makes sense.

This can easily be done once #1182 has been merged.

This can easily be done once #1182 has been merged.

Sure, once the other PR is merged, I can make the necessary changes here too!

Changes seem to work fine at #1187, Is upgrading golangci version causing it to fail?

`ERRO Running error: buildir: failed to load package rbd: could not load export data: no export data for "github.com/ceph/go-ceph/rbd"
INFO Memory: 1348 samples, avg is 280.5MB, max is 1018.4MB
INFO Execution took 2m39.824731991s
Makefile:90: recipe for target 'go-lint' failed
make: *** [go-lint] Error 3
The command "make go-lint" exited with 2.`

#1198 has been merged. Please rebase and move the change to scripts/golangci.yml to scripts/golangci.yml.in.

@nixpanic , I have resolved the merge conflicts and the static check seems to work fine now... My only concern is the susceptibility of this PR to merge conflicts as it contains large number of files... is there any way we can deal with this?

1 solution can be to address godot separately as it contains changes to ~ 50 files. Any thoughts?

@Madhu-1, Does this logging mechanism meet our expectations?

@Yuggupta27 still no luck  ? :)

@Yuggupta27 still no luck ? :)

Just a minor gosec issue remaining, Will get it resolved asap.

Addressed all the warnings :) Please take a look @humblec

LGTM.. Thanks @Yuggupta27 !

@Madhu-1 @nixpanic ptal

PTAL @humblec

Please take a look @humblec

LGTM.. Thanks ! @Madhu-1 and I can add ready-to-merge

@Madhu-1 @humblec , Should we add ready-to-merge label?

@Yuggupta27 added ready-to-merge but CI won't merge this one, as @nixpanic requested changes in the PR.

@Yuggupta27 added ready-to-merge but CI won't merge this one, as @nixpanic requested changes in the PR.

Oh Okay, Will wait for his re-review then. Thanks.

@nixpanic , I have added the reason for minLenToDecode here https://github.com/ceph/ceph-csi/pull/1157/commits/ce86405cb357993efe3ce1824e8c6f2cf14ff63c#diff-f85816b002d320114353e8823eb63e65R132 . Thanks.


MSON parsing bug on apiary.io

# Data Structures

## Session Object
- links
  - self: https://example.com/session (enum)
- data
  - id: abc123 (string)
  - type: session (enum)
  - Include Account Relationship

## Account Relationship Object
- relationships
  - account
    - links
      - related: https://example.com/session/account (enum)
    - data
      - id: abc1234
      - type: account (enum)

Parses to
{
  "links": {
    "self": "https://example.com/session"
  },
  "data": {
    "id": "abc123",
    "type": "session",
    "relationships": {
      "account": null,
      "links": {
        "related": "https://example.com/session/account"
      },
      "data": {
        "id": "abc1234",
        "type": "account"
      }
    }
  }
}

See the "account": null part, that’s odd. I expected
{
    "links": {
        "self": "https://example.com/session"
    },
    "data": {
        "id": "abc123",
        "type": "session",
        "relationships": {
            "account": {
                "links": {
                    "related": "https://example.com/session/account"
                },
                "data": {
                    "id": "abc1234",
                    "type": "account"
                }
            }
        }
    }
}

Could you check with 4space indenting ?
# Data Structures

## Session
- links
    - self: https://example.com/session (enum)
- data
    - id: abc123 (string)
    - type: session (enum)
    - Include Account Relationship

## Account Relationship
- relationships
    - account
        - links
            - related: https://example.com/session/account (enum)
        - data
            - id: abc1234
            - type: account (enum)

@gr2m seems to work fine with 4 spaces indent for lists and with the latest parser (https://github.com/apiaryio/drafter)
Note the new parser is being rolled out to the Apiary but it is still not live.

yes, 4 spaces did the trick, thanks @zdne :)
Related question for apiary / api blueprint: Do you plan to allow to combine + Model with + Attributes? At least on apiary.io it currently doesn’t work. + Model seems to only accept + Body
Shall I ask the question somewhere else?

@gr2m I would like to deprecate and remove resource model once we have MSON-headers https://github.com/apiaryio/api-blueprint-rfcs/pull/3 (I will write more on this later in a blog post)

@gr2m and as always any feedback appreciated!

Nice! On what blog will you write about?
Do you give you more context: I'm working on Hoodie, which is an open source server/backend for front-end apps. It comes things like data persistance, user management and background tasks out of the box, so people can build full stack apps but nether touch a back-end, kinda like Firebase or Parse, but open source.
We are working on a new account module which will have a JSON API, and we use Apiary for the specs:
http://docs.accountrestjsonapi.apiary.io/
I would love to make the spec as simple as possible by using all the features you got, to make collaboration on it simpler, but also because we plan a blog series on Tools we love & use at Hoodie, and it would be nice to have a good example to share once we write about API Blueprint / Apiary. I'm still learning about how to simplify the *.apib file, and any kind of welcome would be more than welcome :) It's also a nice showcase of a JSON API-compatible REST API, of which there are not many right now

First of all, thank you for shoutout @gr2m !
Note I like and I am more or less following Hoodie since you guys have started!
I am also super excited about you guys using the JSON API! We need more Hypermedia API role models! Are you familiar with FoxyCart and their API? Check their take on API Blueprint at http://docs.foxyhapi.apiary.io ! Maybe you can find some inspiration there? Either way let me know should you have questions or ideas. I will be happy to hangout and chat!

Nice! On what blog will you write about it?

It will be at http://blog.apiary.io


Username is not being checked in an case insensitive manner

Website currently allows two users to register using the same username and same email, as long as the case is different. If not fixed, neither user will be able to login, since the authentication module expects only one entry to match the query.

For usernames, we'll force it to lowercase always. This way, the unique constraint of the database is enforced.
For emails, we'll store the data into the database in the original case, since people have preferences on the case of their emails. Plus, if a second user tries to use an existing user's email, that new user won't be able to confirm ownership of it anyway.


Fix doc for ~> to mention Mappable.

Wow! Thank you very much!
:heart: :yellow_heart: :green_heart:  :blue_heart:


Create E2E test framework

If we are generating cloud-init then a KVM/QEMU based framework should work - AppVeyor supports nested virtualization for CI, and we could potentially work with test-infra to get some nested virtualization capable runners on GCP.

Testing the logic of the controller could be done with the standard kube-builder testing framework
Testing the output of this provider would use KVM, e.g.:

Generate cloud-init for controller
Spin up a KVM instance with cloud-int and wait for ip
Generate cloud-init for worker
Spin up second KVM instance
Test that both nodes have registered/joined.

please outline the details on how this framework will be implemented.

/milestone Next
/priority awaiting-more-evidence

This will be covered, at least at a cluster-api level by the testing CAEP (as of this comment this document is still early and hasn't been edited yet) https://docs.google.com/document/d/1SZkKoDok62eWHg5UVOmc96gW8KET69uNHZ4nWV0yyBs/edit

/remove-lifecycle stale
/milestone Next
/priority important-longterm

/remove-priority awaiting-more-evidence


qt52 failed to build on OS X 10.10.2

I'm trying to install qt 5.2.x for compatibility with a project, and the build is failing for me.
See this gist for more information

This is very likely an environment error on my part.


Regent/Legion Pruning Use of Empty Regions

Sometimes empty logical regions get made. It happens (especially with static control replication). We should do better at avoiding the costs of them as they add to runtime overhead.
Two proposed changes:

The runtime re-writes region requirements for empty logical regions to NO_ACCESS
Have the compiler recognize when regions have been rewritten to NO_ACCESS to avoid using them.

Will switching the region requirements to NO_ACCESS cause problems in the task body when it makes an accessor with read/write/reduce privileges?

The C API accessors don't use privileges:
https://github.com/StanfordLegion/legion/blob/master/runtime/legion/legion_c.h#L3266-L3269
I think we have a guard so that we should be ok accepting null pointers if the volume of the region is 0. At the moment the compiler will still request an accessor, and I don't think that should change because various things in the compiler assume that the variables are set up unconditionally.

One thing to be careful about if you do this is that you will need to modify the runtime so that a task can call another task with READ_WRITE privilege if the region is empty, even if the parent has NO_ACCESS. Otherwise you will break any transitive task calls in Regent.

@elliottslaughter The only regions that can inherit privileges from an empty sub-region must also be empty and therefore will also be re-written to NO_ACCESS.

That's fine, as long as you do your rewrite to NO_ACCESS before checking privileges.

Yes, that is when it would have to be done.

I think option (2) isn't really feasible under DCR since all launches are (or should be) index launches. Is this still an issue and is (1) a viable approach?

Yes, this is still potentially an issue under DCR. It's the emptiness of the index spaces for the logical regions themselves, not the index space launches that is the issue.


Raise error when not association passed to Ecto.assoc_loaded?/1

I noticed that Ecto.assoc_loaded?/1 returns true on any input except %Ecto.Association.NotLoaded{}. I think it can be source of errors when somebody will try to put atom instead of entry Ecto.assoc_loaded?(:assoc) or something else.

Thank you @Apelsinka223! I have just removed the catch-all clause in favor of a FunctionClauseError, which are quite reliable from Elixir v1.5 and OTP 20+.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


ipa-replica-install / Feature Request: Replicas

I have an existing IPA installation (all 3.3.3, CentOS 7).
I'd be interested in trying out a container for the next cell.
So, feature/container request: Ability to run an IPA server in a container that is a replica of an existing server.

So, feature/container request: Ability to run an IPA server in a container that is a replica of an existing server.

It is now available in master and derived branches, as well as in the images in the Docker registry.
Use volume and put ipa-replica-install-options and the GPG-encrypted replica information file to the directory that you bind with -v before starting the container. The existence of ipa-replica-install-options will trigger ipa-repliace-install.


Check if the folder already exists

How to check if the folder already exists. Thanks

`try
{
$dropbox->createFolder($path);
}
catch (Exception $e)
{
}`

I'm working in the symfony framework and can't do a simple try/catch, so I wrote a function to handle this:
`	function doesItemExist( $path, $depth = 0 ){
$dropbox = new Dropbox($app); // this isn't actually in my code, as I'm using a an object based approach, but you could define your dropbox object here if needed
	$depth++;
	if( !is_array( $path ) ){
		$path = explode( '/', $path );
	}

	$pathDepth = count( $path ) - 1; // the max depth of the exploded path
	$currentPath = join( '/', array_slice( $path, 0, $depth ) ); // the joined path at the current depth

	if( $depth <= $pathDepth ){
		// get the dropbox items
		$listFolder = $dropbox->listFolder( $currentPath, ['recursive' => false] );
		$items = $listFolder->getItems()->all();

		// loop through the items looking for the current path
		foreach( $items as $item ){
			// match case insensitive
			if(
				"$currentPath/$path[$depth]" === $item->getDataProperty( 'path_display' ) ||
				"$currentPath/$path[$depth]" === $item->getDataProperty( 'path_lower' )
		 	){
				// found our current depth item
				if( $depth === $pathDepth ){
					// if we are at the max depth then we are done
					return true;
					break;
				} else {
					// otherwise dive deeper
					return doesItemExist( $path, $depth );
					break;
				}
			}
		}

		// our item wasn't found
		return false;
	}
}`

After doing A/B testing, dropbox's search function is faster than my function.


Missing Japanese resrouces in React

I noticed the repo didn't have a lot of good Japanese React resources. I'd like to add one here.

Closing this issue as the PR was merged. In future if you want to add any resource, feel free to raise a PR directly. No need for raising an issue


Update docs

lldb on Windows fails to build with cl due to a template instantiation
error. Choose clang-cl

I think that we should fix lldb instead.  It should build with cl.


Docs should explain downloading and updating glue stack

Description
Installation instructions, make mention of installing, but not updating. When installing, it seems to download many directories of components, while also installing packages. It is unclear how these get updated, or why they download them this way instead of just using packages installed. Is it so I can modify them manually if I want to? And wouldn’t doing so make them difficult to download/install updates if I’ve modified them? I don’t really see any explanation about this.
Problem Statement
See description
Proposed Solution or API
Have explainers about why they download in this way, and make mention on installation, instructions regarding updating instructions
Alternatives
No response
Additional Information
Individual components seem to make mention of individually updating them. More globally, applicable update reference might make sense on other installation pages like expo.
mentions individually updating component https://ui.gluestack.io/docs/components/typography/text
Doesn’t mention globally updating at all https://ui.gluestack.io/docs/getting-started/install-expo

@Seanmclem Thanks for the feedback. We are working on the smooth onboarding of an app using gluestack-ui.
To help you a bit, the commands to update all the components and few others are listed on https://ui.gluestack.io/docs/advanced/gluestack-ui-cli.

@Seanmclem Thanks for the feedback. We are working on the smooth onboarding of an app using gluestack-ui.
To help you a bit, the commands to update all the components and few others are listed on https://ui.gluestack.io/docs/advanced/gluestack-ui-cli.

Thanks! I should have, but never really thought to look at the CLI documentation

this url its down (404) https://ui.gluestack.io/docs/advanced/gluestack-ui-cli

este url está desativado (404) https://ui.gluestack.io/docs/advanced/gluestack-ui-cli

@SeanmclemObrigado pelo feedback. Estamos trabalhando na integração suave de um aplicativo usando gluestack-ui. Para te ajudar um pouco, os comandos para atualizar todos os componentes e alguns outros estão listados em https://ui.gluestack.io/docs/advanced/gluestack-ui-cli .

este url está desativado (404)


Conso live error : energy dashboard consumption is bigger than expected (sometimes)

Hello,
i'm very surprised to find my daily consumption, totaly different than the official consumption,
could you help me to find the problem ?

Thanks

Please can you look, if the live data is correct ?

No news ?

Hello, I have also this problem. It happened sometimes, here some data that seems weird as the sum should be linear.

strange.
I will look in sensor attribute option if nothing strange.
Please can you take a picture of live data curve and daily curve.
Thanks in advance

Should be linear!

Agree!
In the logs do you have trace that seems strange?

I have this error multiples times:
2022-03-23 02:12:45 WARNING (SyncWorker_8) [custom_components.keyatome.sensor] Live Data : Missing last value in values: {'isConnected': True, 'time': '2022-03-23T02:12:45+01:00', 'timeLimitBeforePowerFailure': -1}
2022-03-23 02:12:46 ERROR (SyncWorker_8) [custom_components.keyatome.sensor] Live Data : Missing last value in values: {'isConnected': True, 'time': '2022-03-23T02:12:46+01:00', 'timeLimitBeforePowerFailure': -1}

Hello,
same problem today :
Logger: custom_components.keyatome.sensor
Source: custom_components/keyatome/sensor.py:399
Integration: KeyAtome Linky (documentation, issues)
First occurred: 22 mars 2022, 19:48:22 (26 occurrences)
Last logged: 19:48:59
Live Data : Missing last value in values: {'isConnected': True, 'time': '2022-03-23T02:19:12+01:00', 'timeLimitBeforePowerFailure': -1}
Live Data : Missing last value in values: {'isConnected': True, 'time': '2022-03-23T02:19:43+01:00', 'timeLimitBeforePowerFailure': -1}
Live Data : Missing last value in values: {'isConnected': True, 'time': '2022-03-23T02:20:44+01:00', 'timeLimitBeforePowerFailure': -1}
Live Data : Missing last value in values: {'isConnected': True, 'time': '2022-03-23T02:21:15+01:00', 'timeLimitBeforePowerFailure': -1}
Live Data : Missing last value in values: None

Since I restarted HA, I don't have the logs anymore :'(
I do not use templating and I have only one subscription.
today, everything seems ok so far!
screenshot of yesterday (the first hours between 0h and 6h are a bit messy as I tried to correct the mysql tables)

Thanks, I suspect that when there is no value from Total Energies, the None value gives to HA is not the expected one.
I mean if None is used as 0 (zero) then I have to change something

Longterm statistics automatically drops unavailable/none values. If "0" is pass instead of none or unavailable, it automatically increases value as it's a new "reset".

Thanks !

I think the consumption of 23r march is false.
If you have an appart or house the consumption is 40kw per day not not per hour.
Can you check the daily sensor.
Mine is messy

The back and force value is interpreted by energydash board as peak consumption

I barely use 15kw max per day.
mine also is messy

Great! Sorry, I can just help point out what's wrong, I'm not good enough to help you with coding!

I mean, thanks we have found the root cause, i.e. data received from server.
Now I wonder what i can do to remove wrong data or warn the user

I can see the logic:
IF data Between 00:00 and 23:55 is less Than previous Mesure and > 0 == DROP

I think , I will create a "protected" value and "raw" value. In case ;-)

Thanks, let me know if you need more data or feedback.

It's ok for time being :-)

Please can you test V5.2.1 ?

Done! thanks for the quick update! Should I monitor some logs to see if it's working (dropping unusual values)?

If something wrong, in the log, I've put the fllowing error :  Period are strictly increasing except reset to zero

Thanks for your quick update (i use the version 5.2.2).
I hope it will work better in the next days, and the TotalEnergies server too.

Hello
I will close the issue as there is no regression for time being.
If one day, the total energies sends data that are not increasing, then the 5.2.1/5.2.2 version will prevent from having strange consumption.

Working great!

Thanks !


Spirv-fuzz: new various transformations and fuzzer passes for some atomic operations

Add new transformations and fuzzer passes for some atomic operations like OpAtomicIAdd, OpAtomicSMax, OpAtomicAnd, OpAtomicOr, OpAtomicXor, and others. The full description will update soon, will mention each new transformation PR to this issue.

Let's use #4419 to track this instead.


Latest sales for Basilsik

Same as we have latest sales on RMRK, we should have for Basilisk on landing

👋

snek#42
Made a PR for these two issues(#3570 #3571), @vikiival pls have a look when you are free:)

@zhengow seems we can continue on this?:)

Btw, Do I need to show popular collection list for bsx? Currently I hide it because there is no issue for it.

🔜 #3694


Convert custom TinyLlama (custom config) to Pytorch or HuggingFace format

Hello everyone,
Firstly, thanks for everyone who has contributed to this amazing repo. It is life saving.
My problem is that, I have pretrained custom TinyLlama with different configs (n_layer, n_head, n_embd.) on my custom dataset. I have seen ways to convert the .pth checkpoint to Pytorch module or HuggingFace transformers. But how to do that for my custom model which has different layers and params from the original version.
Any help or guidance is appreciated as it is important for me. As you know pretraining is difficult and takes time.

I have changed following hyperparameters when pretraining.

n_layer=6

n_head=4

n_embd=128

@carmocca could you please take a look at that?

Hey @eljanmahammadli
As @carmocca pointed out, you can specify all the changed parameters in HF config (or provide them as kwargs during loading).
That means that after you converted a model, as specified in the tutorial, you need to load it, but, compared to the tutorial, your code should be something like:
import torch
from transformers import AutoModel

state_dict = torch.load('out/hf-tinyllama/converted_model.pth')
model = AutoModel.from_pretrained(
    "TinyLlama/TinyLlama-1.1B-intermediate-step-1431k-3T",
    num_hidden_layers=6,
    num_attention_heads=4,
    hidden_size=128,
    state_dict=state_dict,
)

@carmocca @Andrei-Aksionov
Thanks a lot, I was able to use both ways. I didn’t know we are able to initialize model like this.


Linux 6.11-rc1 (and misc fixes)

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 330862 run on x86_64-linux 1

7 packages built:

linux-rt_latest (linuxKernel.kernels.linux_rt_6_6)
linux-rt_latest.dev (linuxKernel.kernels.linux_rt_6_6.dev)
linuxKernel.kernels.linux_rt_6_1
linuxKernel.kernels.linux_rt_6_1.dev
linux_testing (linuxKernel.kernels.linux_testing)
linux_testing.dev (linuxKernel.kernels.linux_testing.dev)
nixpkgs-manual


CSGO crash

Hi,
I have a problem that when I try to inject with Sazinjector via "Manual Map" CSGO just crashes :(

I tried latest code, csgo crashed with other injectors too

Yea, im trying for MONTHS, to inject with MetroLoader but doesn't work, it instant crash the csgo... but with process hacker injects just fine.
I already opened an Issue here but never got reply back :D

I had switched to AVX512 mentioned in Readme, after I switch back to SS2 it works fine, no crashes if you have changed optimization settings

Further optimizations
If your CPU supports AVX / AVX2 / AVX-512 instruction set, you can enable it in project settings. This should result in more performant code, optimized for your CPU. Currently SSE2 instructions are selected in project settings.

afaik osiris doesnt work with manualmap


Search without a query

Hey Pat, I'm trying to upgrade to TS 3 and Sphinx 2.
In TS 2, the later would consider all rows as matched.

I would certainly expect Foo.search(options) to work - does it work with no options? Does it seem like there's a specific option that could be part of the problem?

I'm getting...
>> Foo.search
  Sphinx Query (3.1ms)  SELECT * FROM `foo_core` WHERE MATCH('[]') AND `sphinx_deleted` = 0 LIMIT 0, 20 OPTION max_matches=100000
  Sphinx  Found 0 results

I think the MATCH('[]') is the culprit...but not sure how to remove it.

Hmm, maybe it is a bug in TS! Which version are you using?
Also, as a workaround, try passing through nil as the query: Foo.search(nil, options)

thinking-sphinx (3.1.3)

nil doesn't work either...
>> Foo.search(nil)
  Sphinx Query (3.2ms)  SELECT * FROM `foo_core` WHERE MATCH('[nil]') AND `sphinx_deleted` = 0 LIMIT 0, 20 OPTION max_matches=100000
  Sphinx  Found 0 results

I'm not seeing this issue locally (using TS v3.1.3 as well). You're not overwriting the search method, or perhaps using a default sphinx scope or something else that may be different?

Argh! we were overriding.  Thanks Pat. As always, you're the BEST!

No worries :)

One more quick question :-D  We have 2 indices for 1 model.   When doing Foo.search, is there a way to default to just one of the indexes (and not search both unless specified)?

I think the following should work, within your model:
sphinx_scope(:main_index_only) { {:indices => ['foo_core']} }
default_sphinx_scope :main_index_only

Oh, and include this module into your model: include ThinkingSphinx::Scopes

:heart:


converter: enhance nydus tar unpack

Unpacking file from nydus blob should seek until encountering the head
of blob stream, this patch ensures this to improve unpack compatibility
and refine the test codes.
Signed-off-by: Yan Song<EMAIL_ADDRESS>

Smoke failed


.Net Core 3.0

Hi,
I tried migrate my project from .net core 2.2 to 3.0 and I get a issue on Rotativa Configuration.Setup
it is returning the message below.
Argument 1: cannot convert from 'Microsoft.AspNetCore.Hosting.IWebHostEnvironment' to 'Microsoft.AspNetCore.Hosting.IHostingEnvironment' {my project name}
\src{my project name}\Startup.cs
any plan to migrate it to .net core 3.0 ?

I have the same issue, is this repository alive?

The last PR should have solved the problem. Take a look here. https://github.com/webgio/Rotativa.AspNetCore/issues/42


LFS-1087: Implement test run mode

This pull request adds the test runmode that, when used, makes available all previously used test Questionnaires.
This pull request makes available the following test Questionnaires from the following tasks:

Date Conditional Test (LFS-273, LFS-275, LFS-484)

Decimal Conditional Test (LFS-273, LFS-275, LFS-484)

Double Conditional Test (LFS-273, LFS-275, LFS-484)

Long Conditional Test (LFS-273, LFS-275, LFS-484)

String Conditional Test (LFS-273, LFS-275, LFS-484)

Date Formats Test (LFS-975, LFS-366)

Radio Button / Checkbox Compact Test (LFS-1077)

Vocabulary Multiple Choice Test (LFS-1073)

Although out-of-scope for this PR, the following Questionnaires expose the following bugs:

Date Conditional Test (LFS-273, LFS-275, LFS-484)

Must be equal to 2020-06-30 is broken
Must be empty if you set it then clear it, conditional section does not re-appear
Must not be empty if you set it then clear it, conditional section does not disappear

Date Formats Test (LFS-975, LFS-366)

Year Month Day Hour Minute does not allow for input
Year Month Day Hour Minute Second does not allow for input
Year Month Day Hour Minute Interval does not allow for input
Year Month Day Hour Minute Second Interval does not allow for input

allow tab on Date Conditional Test form fields


Some box config, custom/apex presets redone, some classname changes a…

…nd some very basic stringtable changes.

Will aim to complete these truck configs next:
		rhsusf_M977A4_usarmy_d
		rhsusf_M977A4_usarmy_wd
		RHS_Ural_Open_MSV_01
		RHS_Ural_MSV_01
		rhs_kamaz5350_open_msv
		rhs_kamaz5350_msv
		LOP_TKA_Ural_open
		LOP_TKA_Ural

As well as add them suitably to the remaining presets that need finishing (civ trucks too). Also Altis map will be in later, completely redone with some improvements. :))
Any feedback on pricing would be greatly appreciated!


test(docker): Decompose duplicate asserts

Group duplicated asserts in a helper function.

I updated the pull request title to match the new commit message. Good choice of positioning within the file for the helper function!


[linmath] add new port

Describe the pull request
add port for linmath.h

What does your PR fix?
Fixes #...
It make the header only library linmath.h available via vcpkg

Which triplets are supported/not supported? Have you updated the CI baseline?
<all / linux, windows, ...>, <Yes/No>
No

Does your PR follow the maintainer guide?

Yes I believe so.

It is a new port with only one version so no update is required.
If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/
It is complete.
Fixes https://github.com/microsoft/vcpkg/issues/24657

This pull request solves the following:
https://github.com/microsoft/vcpkg/issues/24657

Should this PR be closed?

Should this PR be closed?

@RyanPaulMcKenna, No, Please revise according to this review

Should this PR be closed?

No, I have approved this pr, Thanks!

@RyanPaulMcKenna, Ping for response

Any chances this may get any traction?

Any chances this may get any traction?

Since the author didn't respond to the requested changes and people actually want this port I finished this PR in #26069.

Duplicate of #26069


first example does not seams to work.

hi, this tool looks very great. made by fellow Montrealers?
I was trying draggable with the simple implementation I could. when running it, I get the Draggable object defined, but <li> element are not draggable and, more importantly, I do not get any form of feedback like an error.
See sample implementation
https://github.com/pywebdesign/shopify-draggable-sample
Hope I didn't just miss something!
<3

Hi 👋 thanks for posting some feedback! In my case from Ottawa actually 😄
I may know why this is the case, and I should probably update the README to make this clearer, but draggable will look for .draggable-source by default unless you define your own selector (see the draggable options here), e.g.
const draggable = new Draggable({
  draggable: 'custom-css-selector',
});

Please let me know if that fixes the issue

Hi, glad I can help!
So after changing it, I get.
Uncaught TypeError: this.containers[Symbol.iterator] is not a function.
I feel like there should be some distribution for current browsers or transpiling information on the readme.
I think babel preset:  presets: ["shopify/web"] is not the best option for external open source project.

Draggable will be primarily targeting modern browsers by default, but the plan is to provide legacy/fully ES5 transpiled bundles that can be used for better browser support.
Furthermore we want to provide modules separately, so you can import only what you need from the library, e.g. import { Sortable } from ‘@shopify/draggable/sortable’

@pywebdesign I hope the original issue is resolved with this, I will close for now and add more information on the readme as you mentioned


feat(minio): Add option to use hostPath for minio storage

This adds an option to minio to set up a hostPath volume. Running deis in minikube is a use case.

You'll want to do those changes against workflow-dev. That's the chart we use as the basis for releases

This will work with single-node clusters where there is only one worker node available. As long as we're abundantly clear about that in the docs I'm OK with this.

the option of nodeSelector should be there as well if the k8s cluster is not the single-node one.

we've stopped development on helm-classic. Please re-open another PR on deis/minio against that chart. Thanks!


Fix for sticky contents link overlap issue

At foot of page was possible to get 'Contents' fixed position link to briefly overlap content below it
Problem was due to height of parent element including height of Contents link, then changing after it was set to position fixed
Fixed by defaulting Contents link to position absolute so it never adds any height to the parent

Relates to https://github.com/alphagov/government-frontend/pull/367

When the JS fails on mobile it looks broken:
Old

New

Have removed unnecessary CSS and fixed when JS disabled.

Link behaves nicer now, top work :100: 👍


How to access a key in a json object?

For example:
{
    response =     {
        docs =         (
                        {
                "_version_" = 15352wqwd18967678812160;
                active =                 (
                    1
                );
                email =                 (
<EMAIL_ADDRESS>                );
                firstName =                 (
                    Nehlo
                );
                id = NehloTestUser1464098899306;
                lastName =                 (
                    Test
                );
                mode =                 (
                    User
                );
                password =                 (
                    wear21zero
                );
            }
        );
        numFound = 1;
        start = 0;
    };
    responseHeader =     {
        QTime = 1;
        params =         {
            indent = true;
            q = "mode:* AND active:true AND<EMAIL_ADDRESS>AND password:(wear21zero)";
            rows = 10;
            wt = json;
        };
        status = 0;
    };
}

is a json I am receiving from the server. How do I check for the "numFound" value? Like if it is 1, i should push the user to the main screen else ask him to input correct username/password. Help please??

The responseJSON parameter use the iOS native JSON parser, which returns a NSDictionary or NSArray, you need to check the object type. For your case, you can try the code below(although you need to check the object type step by step in practice or use 3rd-party library like SwiftyJSON):

object["response"]["numFound"].intValue

Thank you so much. But is it intValue or just int?

@Dershowitz011 ,as you see from the document, the return value is id, which may be NSDictionary,NSArray,NSNumber,NSString. In your case, you will get a NSNumber, if you want to get the scalar int, you can use -[NSNumber intValue] to obtain it.

Hi @Dershowitz011,
In the future, I'd suggest you open this type of question on Stack Overflow. We use our GitHub project for bug reports and feature requests.
Cheers. 🍻

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


backend not actually used?

As far as I can tell, the bucket and dynamodb table set up by ./modules/backend isn't used anywhere: I can't see how the table and d3 bucket are then configured and used in the project...assuming that indeed they are supposed to be?
Or do I misunderstand, and these settings are to be used in the "child" organization terraforms themselves to set up their own resources?

...and it's right there in the blog post:
# part of the application infrastracture codebase

terraform {
  backend "s3" {
    bucket         = "acme-corp-terraform-state-staging"
    region         = "eu-central-1"
    key            = "terraform.tfstate"
    dynamodb_table = "terraform-lock"
  }
}

so...I should RT(f)M :)


log4j:WARN

Hello
I am having a problem when try to run (spark-notebook-0.6.3-scala-2.10.4-spark-1.4.1-hadoop-2.7.1). After unzipping, I logged in to my Hadoop cluster, changed to the root directory of the unzipped distribution and run the following command:
./bin/spark-notebook
But it produced the following warnings and then stops:
Play server process ID is 22128
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/s/ishams/spark-notebook-0.6.3-scala-2.10.4-spark-1.4.1-hadoop-2.7.1/lib/org.slf4j.slf4j-log4j12-1.7.10.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/s/ishams/spark-notebook-0.6.3-scala-2.10.4-spark-1.4.1-hadoop-2.7.1/lib/ch.qos.logback.logback-classic-1.1.1.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.slf4j.impl.Log4jLoggerFactory]
log4j:WARN No appenders could be found for logger (net.sf.ehcache.config.ConfigurationFactory).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
Appreciate any helpful comments!

Hey guys,
I am facing exactly the same issue trying to get it runnin for Spark 1.6.1 on Ubuntu 16.04 LTS server.
bin/spark-notebook
Play server process ID is 14937 SLF4J: Class path contains multiple SLF4J bindings. SLF4J: Found binding in [jar:file:/opt/spark-notebook/lib/org.slf4j.slf4j-log4j12-1.7.10.jar!/org/slf4j/impl/StaticLoggerBinder.class] SLF4J: Found binding in [jar:file:/opt/spark-notebook/lib/ch.qos.logback.logback-classic-1.1.1.jar!/org/slf4j/impl/StaticLoggerBinder.class] SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation. SLF4J: Actual binding is of type [org.slf4j.impl.Log4jLoggerFactory] log4j:WARN No appenders could be found for logger (net.sf.ehcache.config.ConfigurationFactory). log4j:WARN Please initialize the log4j system properly. log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
Two days of googling and experimenting didn't help. So, I am beggin the awesome @andypetrella for a lil support. Thank you in advance !!!

Hey @Isaacsh ,
I had a nice chat with @andypetrella and Adam about the issue. It turns out, we have to blame our proxy-server or the firewall. So, it is no Spark-Notebook issue. When I "tear down the wall", I can catch the Spark-Notebook on port 9000 (default).
I am currently working on the proxy the have a password protection enabled.
Thanks again to Andy and Adam, awesome support and really, really friendly !

@dengelha
Thanks for your helpful comment. Never have thought about firewall being the issue. Please post in case you nail it.

Hey @Isaacsh ,
I desperately worked on that and tested all the NGINX configurations. No solution on the SSL + password protection yet, as neither the awesome Spark-Notebook nor NGINX can find their way to the assets. Therefore, I get lots of 404 on css, png ... analyzing the browser behavior when accessing the https://...:9000.
But are you dependent on the password protection? You can also build an open tunnel for 9000 to localhost:9000 via putty. Then, you can access SNB on your server via the browser of your client PC.
What do you thing?
Enjoy coding :-)
David

Thanks @dengelha It works on localhost:9000 via putty


'Initial deposit' should look at your current balance and show error if trying to set higher than current ballance

This isn't so simple, it requires calculating how much deposit, and also gas cost + gas price (give reasonable value from eth-gas-station or something similar)

Migrated to trello


Question: How do I change the shape of the selected/hovered calendar days from square to round?

I'm trying to change the selected and hovered calendar days from square to round but it seems that since the date is directly in a td element it isn't possible do without the whole calendar day becoming round, which doesn't work for spanned selections (see screen shot). It seems that to make this work correctly the date would need to be in a div within the td. Any thoughts or suggestions on how to make this work?

CalendarDay takes a renderDayContents prop that may be helpful.

this is exactly what I need, thanks!


Adds a flag to specify Acme Solver Namespace

Signed-off-by: Ricardo Pchevuzinske Katz<EMAIL_ADDRESS>
What this PR does / why we need it: This PR adds a flag to specify a common namespace for Solver.
This behavior is the same as existed in kube-lego and is necessary in cases where a user namespace doesn't have enough access to resources, as the following example:

The environment have restrictive Network Policies, not allowing the user namespace to receive external/ingress connections
The environment have restrictive RBAC, not allowing cert-manager to create and delete objects as Pods, Services and Ingress in other users namespaces

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
There are some TODOs in the code, I hope in the review of this PR we can improve the cases with some better implementation
Release note:

New flag --acme-http01-solver-namespace allowing the Cluster Admin to specify if there is a namespace to create the solver resources

I submitted a similar PR a few months ago (https://github.com/jetstack/cert-manager/pull/1338) for this that. We have been running it internally since 0.5.
It was tied to #1272, which was tagged as priority/awaiting-more-evidence so hopefully this can give it more priority (merging either this or my PR would be equally good).

I submitted a similar PR a few months ago (#1338) for this same issue. We have been running it internally since 0.5.
It was tied to #1272, which was tagged as priority/awaiting-more-evidence so hopefully this can give it more priority (merging either this or my PR would be equally good).

Oh I've missed that.
Will close this PR and follow yours!
Thank you very much.

/close
/duplicate #1338


Don't send control messages when stopping the simulation/deployment

I fixed it.
It still could occur in rare cases when the robots get disconnected or deactivated from the hardware side just before the backend receive the STOP message from the frontend. So, I think you should handle it in the frontend too, just in case.


Update config.yaml

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

2fc93b4 Update config.yaml

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign lucperkins
You can assign the PR to them by writing /assign @lucperkins in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @Riyan-Rickson! It looks like this is your first PR to falcosecurity/falco-website 🎉

good work. I made changes just to notify you that you need to do some improvements. would love to see it improved!

We believe that this being opened by a bot and we are closing this PR.
We had many interactions on this repository coming from bots and we limited the interactions to existing contributors until this is solved (starting with one day)
Read more at: lists.cncf.io/g/cncf-falco-dev/message/164
/close

@poiana: Closed this PR.

In response to this:

We believe that this being opened by a bot and we are closing this PR.
We had many interactions on this repository coming from bots and we limited the interactions to existing contributors until this is solved (starting with one day)
Read more at: lists.cncf.io/g/cncf-falco-dev/message/164
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Tekken 7

For some reason it doesn't pick up any of the ipc info it could be cause the log gets flooded with tekken specific vars

Sorry just had to edit SteamPeerManager.cs I thought that attaching another process was enough lol

It should be enough. What bug(s) did you encounter that prevented the tool from working with Tekken? Since I'm trying to support all games, I'll fix it.

Just didn't show the SteamID at all until I built with that change I even used the same config file from when I was trying it idk it could just be a fluke though cause sometimes I know it didn't pick up specific people in Elden Ring
Like someone I play with never showed up but another would

Was just line 87 I assume it's redundant but at least with the small batch I was working with it started to actually show the steam p2p session
 if (!line.Contains("TekkenGame-Win64-Shipping.exe") || line.Contains("IClientMatchmaking::LeaveLobby"))

No you're definitely right, this was dumb typo on my part. Should definitely have tested with other games myself!


python310Packages.objax: 1.6.0 -> 1.7.0

Description of changes
Update objax.
I switched to using fetchPypi because the latest release is not present on the GitHub repo (no tag nor release).
cc @ndl
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 255310 run on x86_64-linux 1

4 packages built:

python310Packages.objax
python310Packages.objax.dist
python311Packages.objax
python311Packages.objax.dist


Median Filtering

Implementation of the same.

@geekquad Sir can you assign this issue to me as I have knowledge about it and I want to apply it by solving this issue.

Assigning it to you.
Please mention the issue number in the description while making a PR.


Update Readme.md to reflect correct arg list

As title indicated

Hi @JasonZhu1313,
Thank you for your suggestion. I've modified the arg name accordingly.  Will close this PR.


Possible to switch image gallery

Is it possible to dynamically change the images property to point to a different image array?

Of course. You need to change the array.
this.galleryImages = newGalleryImages;

Check out this Plunker: https://plnkr.co/edit/hszCyPPwRkh4P7yoJtxv?p=preview
Click the button below the gallery: It replaces the images array.

Thanks!  Working great…

Dave

From: Germán E. Más<EMAIL_ADDRESS>Sent: Wednesday, February 7, 2018 9:44 PM
To: lukasz-galka/ngx-gallery<EMAIL_ADDRESS>Cc: dmessier<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [lukasz-galka/ngx-gallery] Possible to switch image gallery (#115)
Of course. You need to change the array.
this.galleryImages = newGalleryImages;
Check out this Plunker: https://plnkr.co/edit/hszCyPPwRkh4P7yoJtxv?p=preview
Click the button below the gallery: It replaces the images array.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/lukasz-galka/ngx-gallery/issues/115#issuecomment-363984767 , or mute the thread https://github.com/notifications/unsubscribe-auth/AdnRmmmNRe6voE-IbUWO7GOv3qJsKIaeks5tSl9-gaJpZM4R9iJj .


Extended german translation

Hello Team,
I extended the german translation as required by a gov-client I am working with. Please note that this translation is not claimed to be complete; however, I have done my best to identify and translate most of the English terms. I am happy to contribute to this plugin and would like to express my gratitude for your efforts in developing it. I hope my contributions are helpful!
Cheers,
Jakob

Hi @jjarndt, thank you very much for this contribution! This sure was a lot of work and is very much appreciated.
I think there is one thing we should clarify before proceeding with this PR though: How do we want to name things in German? Do we prefer "Denglisch" (like "Lane hinzufügen", "End-Event erstellen") or plain German ("Bahn hinzufügen", "Endereignis erstellen")? What do you guys think, @dominikhorn93 @jjarndt?

Hey @alex-praschek
Thank you very much! The client for whom I continued the translation preferred the consistent German spelling, for example. they chose the word 'Schwimmbahn' instead of 'Lane'.

Hey @alex-praschek
Update:
I updated the translation to match the guidelines of the Praxishandbuch BPMN.


January Publish #3 (#342)

PsExec update

Docs Build status updates of commit d19fbe2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/downloads/index.md
:white_check_mark:Succeeded
View

sysinternals/downloads/psexec.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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit d19fbe2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/downloads/index.md
:white_check_mark:Succeeded
View

sysinternals/downloads/psexec.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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Testing Components with service in OnInit

Hi,
While is was testing with your beautiful library I ran into a exception at the mocks part.
In most of my components I subscribe to an observable, for example this.route.paramMap.subscribe(paramMap => {}); and it will crash with TypeError: Cannot read property 'subscribe' of null.
So I have created a Gist with minimal setup code as example for other people.
Maybe it's nice to make a way to configure the return value of the mocked functions. I saw the default return value was 'null' but with observables it would be nice to have some way to configure it.
Example (brain dump):
@Component({})
class MyComponent {
    constructor(private barService: BarService,
                private route: ActivatedRoute) {}

    ngOnInit(): void {
        // this fails without configuring the mocks:

        // ERROR: Cannot read property 'subscribe' of null
        this.route.paramMap.pipe(
                switchMap(() => this.barService.getFoo())
            ).subscribe();

        // and this fails too...
        this.barService.getBar();
    }
}

describe('', () => {
    // proposal, a way to configure mocks in the factory?
    // or just mock all Observables on the mock prototypes by default...

    const createComponent = createTestComponentFactory({
        component: MyComponent,
        mocks: [[BarService, { getFoo: of('lorem'), getBar: 'ipsum' }]]
    });

    it('', () => {
        const spectator = createComponent();

        // detectChanges should work without issues
    });
})

Closing this issue. If you have any further questions, let me know.


Allow multiple dot notation on child labels

Currently, if you want a child of a child, you either have to use full indentation of write something along the lines of ::.child1.child2
Using a multi dot notation at the beginning of labels would simplify this process evan more and that's kind of the point of the script builder. Might be worth finishing up issue #4 first
Example with multi dot notation:
::parent1
::.child1
::..child2

label parent1:
label parent1.child1:
label parent1.child1.child2:

Seems Ren'Py doesn't like any more than 1 sub label so this kind of thing is now irrelevant. On the plus side, it means it simplifies the label code a fair bit.


Hard refreshes page every time when route is changed.

Every time the route path is changed, the browser fully refreshes the app.

What's  your routes look like?

This is going to be a configuration error on your side, e.g. make sure you're using <Link>/&c.


add new section.

want to add the new section to tour.
Content of beaches and backwater are same.
pls assign this issue.

hey @harshvardhansb  can you assign this issue to me

created PR #307


Restore 1.09 logging_enabled check?

First off, awesome library.
Now, I have a project that uses the logging library itself, and while I could add code to turn off 'sh' logging I'd have to do it in a lot of places due to the project layout. Right now I get [INFO] sh.command <Command '...'> starting process littered about everywhere, including when I run shell scripts.
Any chance you can turn off all logging by default as before, or at least change the level to DEBUG since it's more debugging information than anything else?
Thanks,
- Alex

Hi Alex.  You can change the logging level of a logger by its namespace.  Take a look at this answer.  This is probably the easiest way.  There is also a way to completely disable logging for a namespace, but it is a little more involved:
logger = logging.getLogger("sh")
logger.propagate = False
logger.addHandler(logging.NullHandler())

Let me know if this works

The problem with that approach is as I said, I have a lot of entry points so I'd have to add that code in a lot of places.
Maybe I can word it this way:
Any reason why this line is info instead of debug like it is here, or even why that first one exists since you're debugging the same message in the OProc.__init__?

Yes, OProc is much lower level than Command, so it gets the debug level.  I don't think you should depend on a package's logging levels being static.  Your best bet is to handle sh's logging propagation where you set up your logging.  This is pretty standard for a 3rd party packge.  Having a logging_enabled flag is pretty non-standard, which is why it was removed.  If you are setting up logging in many entry points in your project, then you will also need to add the filtering/disabling code in all of those entry points, or refactor it so you only need to do it in one place.

I agree that 3rd party packages likely have configurable logging, but most libraries I've used are either use logging because it's part of the library's purpose (such as django, cherrypy, where logging is expected), only have debugging output with verbosity flags (pexpect, telnetlib), or have specific places they pipe debug output to instead of to stdout.
Having "starting process" print out is not useful in most situations.
I'll close this anyway since I don't think I'm gonna get my way :)
Cheers,

Alex


Formatting updates only

Jira link(s)
What
Formatting updates
Why
To increase consistency
How
Ran prettier to standardize CHANGELOG.md
Ran prettier-package-json to standardize package.json with correctly ordered properties and formatting.
Verification Steps
Review the formatting
Checklist:

[x] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Hi @danseethaler,
Really thank you for your contribution. Please, in the next feel free to ping me to review your PR. In this case, could you rebase it with the master for we are able to review and move forward to merge it?

Thanks @camilamacedo86 - I've rebase off master to resolve the conflicts. Let me know if you need anything further 👍🏼


setting number of thread per executor

I'm using version com.microsoft.ml.spark:mmlspark_2.11:1.0.0-rc2 on a k8s cluster.
Specifically the lightgbm classifier with binary response.
I'm setting num executors to 10 and allocating spark.executor.cores=80 and park.task.cpus=80 so that each machine runs exactly one task with 80 cores available for the task.
What I was expecting to see is full utilisation of 80 cores, instead only ~8 cores are utilised.
My best guess is that it is related to the num_threads parameter, which you have exposed as numThreads a long time ago, but currently is not present in the param set.
I have dataset of 30,000,000 samples with a large amount of features, each tree take around 10 sec to generate
rebagging takes a lot of time too and uses a single core.
Please advise

Same problem as in https://github.com/Azure/mmlspark/issues/292. Not fixed yet.
Try to play with the number of partitions, and partition key(s). This resulted in 20% utilisation instead of 10%. So still not there.

Same problem +1

@JWenBin have you tried the new single dataset mode parameter on latest master?
https://github.com/Azure/mmlspark/pull/1066
In our benchmarking it resolved the low CPU utilization issue.


Add more tests

Better examples, more edge cases, etc.

My goal is to add acceptance tests for a rule every couple of days.


[Feature request] Interactive rebase onto another branch

In some cases, it is useful to interactive rebase onto another branch (git rebase --onto parameter)
Currently it seems that sourcegit only allows interactive rebase for ancestor commits, so it would be useful to allow it everywhere

I've revert the commit 65bcffa because it need totally rewrite to support full-featured interactive rebase.

Recently, I tried to implement it, but finally failed. It's hard to calculate the commit list to be deal with between two branches before running rebase -i, which will invoke another sequencer process that I do NOT want in that time.
Please forgive my poor ability.
Therefore, the following two alternatives are recommended:

use cherry-pick
run rebase first, then interactive rebase

This issue will be closed and will not be planed in near future


Added restart tag to docker-compose for cvat_ui

Without the restart flag for the cvat_ui container, the app does not successfully load on the HOST machine rebooting

Thanks for the contribution.


GIX-1005: Improve Token Amount Class

Motivation
Improvements on TokenAmount API.
Changes

token param in TokenAmount factory methods is mandatory instead of using ICPToken as default.
TokenAmount.fromNumber does not return FromStringToTokenError, only TokenAmount.

Tests

Add token param to all tests.

@peterpeterparker please review

The token becomes mandatory following one of my comment on NNS-dapp side: https://github.com/dfinity/nns-dapp/pull/1314#discussion_r964445796
i.e. to prevent issues in large application that supports multiple tokens, we make the param mandatory


fix(NODE-3166): allowInvalidHostnames and allowInvalidCertificates flags are ignored

The insecure TLS flags were not being correctly translated to the equivalent nodejs options.

I'm not listed as a reviewer but LGTM


Fix issue 1000: Indexes created on WHERE (#1133)

This fixes the issue with applying indexes on the WHERE clause.
It does this by modifying the volatility flag for 3 functions -
_agtype_build_vertex
_agtype_build_edge
_label_name

All three were verified to meet the specifications for IMMUTABLE.

Looks good to me.


Merge Dev to UAT 21 Mei 2018

Coverage increased (+0.009%) to 78.419% when pulling 949e0c22fc4c9ee8fdbe328af207b00325e2bfed on dev into 4ba27ba9ff3a7b8f0d7ca30f66e9df522a38f85b on uat.


Customize the Wizard Generated Application

Tutorial URL: https://developers.sap.com/tutorials/cp-sdk-android-wizard-app-customize.html
Language: Kotlin
Step 6: Add a Top Products section with a CollectionView
13 .Add the following methods to the class:
...
private class CollectionViewAdapter(current_activity: FragmentActivity, product_list: List<Product>) : CollectionView.CollectionViewAdapter() {
    ...
    override fun onBindViewHolder(collectionViewItemHolder: CollectionViewItemHolder, i: Int) {
        val cvi: CollectionViewItem = collectionViewItemHolder.collectionViewItem
        ...
    }
    ...
}
...

Android Studio returns an error:
Cannot access 'CollectionViewAdapterInternal': it is package-private in 'com.sap.cloud.mobile.fiori.object'

Bruce, let me know if we can address this issue or if we need to take tutorials offline for a little while.

SDK for Android 3.1.0 does have such isuue. We fixed it in SDK for Android 3.1.2 and higher version.

Bruce, let me know if we can address this issue or if we need to take tutorials offline for a little while.

Tutorials don't have to go offline because I specify the version of SDK for Android (2.2.x) and Android Studio (3.5.x)  in the Prerequisites in the first tutorial which is good to the published tutorials. @VGordillo-Resulto is using SDK 3.1.0 and in this scenario, the tutorials are not fully applicable.

Up-to-date tutorials will be published soon.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Restore card removal

It turned out the removal method is in active use

I guess that answers it.

@zenangst We discussed this ones before, this one explains why the code is needed, not really what it does.


Limiting scipy integration

Context: The scipy package can be difficult to redistribute as it requires fortran and some other custom libraries.
Looking at the code, scipy is only used in two places:

The _align_bags func in utils.py for DROP: https://github.com/EleutherAI/lm-evaluation-harness/blob/main/lm_eval/tasks/drop/utils.py#L125
The calculate_z_value func in model_comparator.py script: https://github.com/EleutherAI/lm-evaluation-harness/blob/main/scripts/model_comparator.py#L30

Would the maintainers be amenable to making scipy a local import for these functions so other aspects of the library can be used without needing to install scipy?

This seems very reasonable!

So @nathan-weinberg scipy is installed by way of scikit-learn--while we don't use sklearn in too many places within lm_eval.api.metrics, it might be annoying for many users to have to manually install a scikit-learn extra for any task using F1.
If scipy being installed is a big dealbreaker though, can likely just make sklearn an optional dependency as well. But curious if there are any in-betweens commonly taken here or that could still be useful

Hi @haileyschoelkopf
I'm also the same team. We haven't considered the dependency on sklearn. I can see how the additional dependency makes the situation more complicated.
I have another idea. We are rebuilding all wheels from sources anyway. Our build pipeline has the capability to modify a package to e.g. drop a dependency. It would help us a lot of you could either consolidate import scipy, sklearn into one module or use function-level imports instead of module-level imports for scipy and sklearn. That would reduce the complexity of our downstream changes.
Example:
def cb_multi_fi(items):
    from sklearn.metrics import f1_score

    preds, golds = zip(*items)
    preds = np.array(preds)
    golds = np.array(golds)
    f11 = f1_score(y_true=golds == 0, y_pred=preds == 0)
    f12 = f1_score(y_true=golds == 1, y_pred=preds == 1)
    f13 = f1_score(y_true=golds == 2, y_pred=preds == 2)
    avg_f1 = np.mean([f11, f12, f13])
    return avg_f1

What do you think?

@haileyschoelkopf I'm also from the same team as well. I'd be happy to open up a PR consolidating the imports into one module or moving the imports around to being function-level as long as you'd be receptive to reviewing and accepting such a change.

Function-level imports seem like a good idea! Would be happy to accept that change : )


WarningReporting: config option support with new config too

looks just like the generic cmake config, so all seems good


Anonymous Consent Styling

To be merged on Nik's branch feature/GH-4427-dialog
Style as per mockups.

Hey @znikola, I've committed my last changes for the time being, pls check it out and let me know... these are the things I wasn't able to implement:

id/for dynamic relationship for the checkbox/toggle.
Configurable Icon for the accordion, not sure how to pull it in, as done in navigation menus for example.
Open/Close functionality as done with jQuery in my example, done with the outerHeight()  method, so don't know what's the equivalent for TS or how to re-write it, the rest is a class toggle. (if this is too complicated, maybe it's easier to recycle the collapse functionality used in the search results filter and change the icon. Let me know).

Once you finish pls send it back to me, I may have to move some generic styles to the cx-base for the toggles (and accordion) so they will be available for future implementations


⚠ Samsung has degraded performance

In 30e2eac, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2448 ms

Resolved: Samsung performance has improved in 726cdd8.