MrXL: native JSON

Write a fud pass that automates banking. This will connect nicely with #1448, but is unrelated to it in a technical sense.
Consider a program of the form
input foo: int[4]
_ := map 2 (... foo ...) {...}

The key point, in English, is:

I'll do some map-ey thing with foo using banking factor 2

To do this, MrXL requires a data file of the form
{
  "foo_b0": {
    "data": [
      1,
      2
    ],
    "format": {
      "numeric_type": "bitnum",
      "is_signed": false,
      "width": 32
    }
  },
  "foo_b1": {
    "data": [
      3,
      4
    ],
    "format": {
      "numeric_type": "bitnum",
      "is_signed": false,
      "width": 32
    }
  }
  ...
}

That is, it requires manual banking of the logical foo into two chunks. It would be great if I could pass a data file of the form
{
  "foo": {
    "data": [
      1,
      2,
      3,
      4
    ],
    "format": {
      "numeric_type": "bitnum",
      "is_signed": false,
      "width": 32
    }
    "banking": 2
  }
  ...
}

and have a fud pass that converts the latter into the former.

If I can refine this proposal one step farther: the goal here is to make JSON data files that MrXL wants to take as input "natively," which are distinct from the Calyx-formatted JSON files. The new MrXL-native data files should:

Not be banked. That is, if the MrXL source code declares an array called a, then there is just one thing called a in the MrXL data file.
Use MrXL types (not Calyx types), if at all. Like, the declaration in MrXL looks like input a: int[4]; if we need to write the type down in the input file, let's just use int for that instead of the Calyx-style "format": { "numeric_type": "bitnum", "is_signed": false, "width": 32 }.

Then, the MrXL tooling (whether it's a fud stage or not!) can be responsible for translating this MrXL format into the Calyx-native format.
There is one choice to make:

The banking and type information could be contained within the MrXL JSON data file, as @anshumanmohan proposes above.
The banking and type information could come from the MrXL source code. Like, the data file could just look like {"a": [1,2,34], "b": [...]} with no banking or type info at all. When we do the JSON-to-JSON translation, then, we also have to look at the MrXL source code to know how many banks to produce and which Calyx types to generate.

I would suggest that we want option 2 if it doesn't seem like too much effort.

#1448 is now absorbed into this. And your option 2 sounds like the ticket indeed; thanks for that!

I'll add another nice-to-have to this issue: would be great if MrXL-native JSONs could just punt on the output registers/arrays.
For the MrXL program
input avec: int[4]
output sos: int
squares := map 2 (a <- avec) { a * a }
sos := reduce 1 (acc, i <- squares) 0 { acc + i }

a reasonable MrXL-native .data file would be
{ "avec": [1,2,3,4] }

The MrXL source code has enough info to infer that the "real" (i.e. Calyx-native) .data file needs

banking
the addition of format blocks
the addition of a register called "sos".

Summarizing after the meeting we just had:

[ ] Reduce format-based boilerplate.
[ ] Autobank, reading the necessary info from the MrXL source program.
[ ] Generate the memories required for output, again by reading the MrXL source program.


[VP9e-plugin] Segmentation corner cases processing

Taking care of cases when segmentation is set but doesn't make sense
because nothing is changed or can be applied directly to frame params.
Managed on the MSDK side after agreement with the driver team as it
requires more efforts to do such checkings in the driver

@stasefremov , can you please review?


support for debian or windows x32

Hello, plans for supporting debian or windows x32 ?

Once this PR https://github.com/begriffs/postgrest/pull/923 is merged next week, it will be relatively trivial to add other linux distros as long as there are base docker images for them and they are supported by haskell/stack.
You can check the status of the PR in a week or so, then submit a new PR with additional changes to those scripts to build the platforms you want (it's a matter of a few bash/docker files), i don't mean anyone will mind distributions for additional platforms :).

Resurrect: Will there be a debian image, if not is ubuntu binary debian suitable. If not are source builds transferable from eg deb10 to deb10 ?


Courses search resets on go back

When clicking on a course in courses search and then going back to the search page, the search is reset, not sustaining the fetched search result.

Merged #38


tf.raw_ops.SparseConcat: Overflow bug

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
tf 2.15
Custom code
Yes
OS platform and distribution
Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Under specific input, tf.raw_ops.SparseConcat encounters overflow bug.
Standalone code to reproduce the issue
import tensorflow as tf

# Generate input data
indices1 = tf.constant([[0, 0], [1, 2]], dtype=tf.int64)
values1 = tf.constant([1, 2], dtype=tf.float32)
shape1 = tf.constant([3, 4], dtype=tf.int64)

indices2 = tf.constant([[0, 1], [2, 3]], dtype=tf.int64)
values2 = tf.constant([3, 4], dtype=tf.float32)
shape2 = tf.constant([-1, 4], dtype=tf.int64)  # Mutated shape with the negative bit set

# Invoke tf.raw_ops.SparseConcat
concatenated_sparse = tf.raw_ops.SparseConcat(
    indices=[indices1, indices2],
    values=[values1, values2],
    shapes=[shape1, shape2],
    concat_dim=0
)

print(concatenated_sparse)

Relevant log output
tensorflow.python.framework.errors_impl.InternalError: {{function_node __wrapped__SparseConcat_N_2_device_/job:localhost/replica:0/task:0/device:CPU:0}} Encountered overflow from large input shape. [Op:SparseConcat] name:

Hi @nonenone12135 ,
The error message indicates an overflow from a large input shape. The issue lies in the shape2 variable. You have set the second dimension of shape2 to -1, which is a negative value. This negative value is interpreted as a mutable dimension, meaning that the size of that dimension can change. However, the SparseConcat operation requires all input shapes to be fully defined and non-mutable.
To fix this issue, you need to set the second dimension of shape2 to a positive value. In this case, since the second dimension represents the number of columns in the sparse matrix, you can set it to the same value as shape1, which is 4.
shape2 = tf.constant([3, 4], dtype=tf.int64)  # Fixed shape

I am providing gist here for your reference.
Thank you!

Thank you for your response. So, is this error message expected behavior rather than a bug?

Hi @vraebfdsb ,

Looks like it is working as intended, Meanwhile i will check with developer team on the same issue and will update the same.

Thank you!


Swish activation support

Hi,
Changes discussed here are introducing support to SWISH activation (https://arxiv.org/pdf/1710.05941.pdf)
as part of a eltwise primitve
Scope of changes:

CPU only (no GPU support)
Reference and unified jitted FWD
Reference BWD
unit tests
benchDNN extented for swish eltwise

@emfomenk , @akharito I have run clang_format on changes  and manually adapted some code.

Please also extend the documentation in doc/primitives/eltwise.md:

Add a formula for swish
In Implementation Limitations please mention that GPU doesn't support swish

Hi @jczaja,
Thanks a lot for the PR! Overall it looks really good 👍
Let's resolve the remaining issues and promote it.

@emfomenk , @akharito I have added entry in limitation section on CPU only support  & some style fixes were added as well.

Hi @jczaja,
Thanks, I think everything looks good.
Let me run CI on your changes and everything is good, promote them to master.
P.S. With the upcoming US holidays and some maintenance on a site the promotion might be delayed by Mon/Tue next week. But I hope to do that ASAP.

Thanks @emfomenk and @akharito for taking time and reviewing those changes

Finally merged!
Sorry for delay and great thanks for the contribution! 🎉


Set up some kind of system to build against the latest nightly regularly.

In times when I'm not working on the project daily, it can take me some time to notice a nightly breaks the build. This needs fixing (not necessarily in this repo, however).

You could just enable a cron job (daily/weekly/monthly) in Travis CI settings.

Done! I didn't know that's an option, thanks for pointing it out @cuviper.


Multi-line captions

By using line-height and box-decoration-break: clone, captions can look better when they break to multiple lines (on small screens, for example). Box-decoration-break ensures the same padding is applied to each broken line (limited support, other approaches here).
Happy to contribute a fix for this when I have time but figured I would document it for now.
Code:
.plyr__captions span {
  box-decoration-break: clone;
  line-height: 150%;
}

Before

After

Awesome. Thanks for this. I can add it in the next release and give you the credit for it of course or a PR (remember LESS + SCSS versions), whichever. 👍


Add layer finaltypetable.sexp

I re-ran the CI for this after merging #440, and it's still failing. Other than that, this looks broadly reasonable. Once you get this passing, I think a good next step would be to update the parser in the haskell implementation in accordance with this change.

@zenhack After rebase, CI passes.


jpeg.openRAM support

Hi, can you provide an example for esp32 that reads a jpg file from sd, save it internal ram and calls jpeg.openRAM function to render the image?

What you're asking for is possible by looking at the existing examples, but is only applicable to you. I guess a simpler issue to open would be to ask me what part of the API you need help understanding.


SASS-3308 fix link for payments into overseas pensions to route to the paymentIntoPensionScheme

Checklist
Jira subtask: SASS-3327
Developer: Before Asking Others To Review Your Code
Have you ...

[x]  Had a QA verify that all acceptance criteria have been met, with comment(s) in the ticket?
[x]  Hosted a group review of your implementation?
[x]  Rebased against main?
[x]  Ensured that your commit(s) contain the ticket key?
[x]  Ensured that your branch is named after the ticket key?
[x]  Successfully run the unit tests?
[x]  Successfully run the integration tests?
[x]  Successfully run the journey tests? If not, please add a comment explaining why it wasn't necessary.
[x]  Ensured that the PR job has passed.

Reviewer: Before Reviewing
Have you ...

[ ]  Pulled the branch down?
[x]  Assigned yourself to this PR?
[x]  Moved the Jira subtask to “in review”?
[x]  Ensured that it is rebased against main?

Developer: After Approval
Have you ...

[ ]  Rebased against main?
[ ]  Moved the Jira subtask to “in pipeline”?

Developer: After Merging
Have you ...

[ ]  Deleted the branch?
[ ]  Scheduled a demo? If not, please add a comment.

Pulled the branch down?

I didn't, as the changes are minor so assessing them here is fine.


Discord link expired

The link for Discord in the footer is expired. I must have put a 24-hour link in the JSON file by accident. Here is an updated link that should never expire: https://discord.gg/Wx63jaYa2Z

in which Garden is this happening?

BrightDAO

closed by https://github.com/1Hive/dao-list/pull/32


Integrate FlashInfer into llama model

This PR integrates FlashInfer into llama model.

I'm taking over this PR along with SourceModule redesign: https://github.com/apache/tvm/pull/16247


Bugfix/yvikhlya/ogcm param fix

Fixes two issues with coupled model:

Ocean model parameters section in AGCM.rc
CICE_N_ICE_CATEGORIES and CICE_N_ICE_LAYER where previously set default values in GEOSgcs compinent. Since GEOSgcs component is gone, we need these options be in AGCM.rc

@yvikhlya This request will (for now) have to wait until all of CVS is replayed into git. Once that is done, we will pull it (after testing).

This change seems zero-diff. I made a test run with the code from GEOS-ESM/MAPL#8 and it is zero-diff.
One caveat. If, like me, you have scripts that were looking for the old OGCM_IM say, you will have to change them. @yvikhlya it looks like some in coupled_diagnostics might, so you might want to take a look.

This change will need to wait until we have a formal GEOSgcm reviewer.  (Nominally Bill Putman until his decides to share such responsibilities.)

This change seems zero-diff. I made a test run with the code from GEOS-ESM/MAPL#8 and it is zero-diff.
One caveat. If, like me, you have scripts that were looking for the old OGCM_IM say, you will have to change them. @yvikhlya it looks like some in coupled_diagnostics might, so you might want to take a look.

Yes, I need to update a number of things in coupled_diagnostics. Don't warry about this, I'll handle this.


chore(main): release 1.4.5

:robot: I have created a release beep boop
1.4.5 (2023-04-15)
update

添加服务器端口 (c768636)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/future-front-end/releases/tag/v1.4.5 :sunflower:


Update to 1.20

Please, update mod to new version minecraft

Resolved in https://github.com/Prismwork/EMITrades/commit/6b39b8209239658ca1330bec771716d6397fc663

@Flamarine are you planning on uploading it to modrinth? The build from here seems to work.
And for EMIEffect as well.


chore: ccutil to use the article doctype

Ccutil and Pantheon are not using the :doctype: attribute.
Ccutil doctype defaults to book.
To change the doctype with ccutil, use the CLI:

  --doctype DOCTYPE           The document type. If no type is passed, the
                              type is assumed to be a book.

/cherry-pick release-1.3

/cherry-pick release-1.4


sqlx migrate run got broken after update

I get this when I run sqlx migrate run after update to 51aeee2:
error: while resolving migrations: invalid digit found in string

It worked before the update (e3b3806).
It doesn't seem like it's a problem with specific migration since it doesn't work with any migration I tried.

Figured. The new migration file name format has changed and all -_ must be removed from datetime prefix.

$ cargo sqlx migrate run
error: while resolving migrations: invalid digit found in string

$ ls -l migrations/
total 8
-rw-r--r--  1 brandon  staff  1050 Feb 13 15:14 001-quote_snapshots.sql

confirmed as well + just adding for clarity. 001-snapshots.sql is fine, 001-quote_snapshots.sql is not


EOFError when the connection was closed by broker because of idle client

client connections are being closed by broker after some time, defined by connections.max.idle.ms config on the broker (default to 9 min).
I noticed that such closed connections are still being used by ruby-kafka when producing message, and this cause EOFError exceptions, and sync retry after backoff time.
I saw that an attempt to fix this had already been added here https://github.com/zendesk/ruby-kafka/pull/242, but it seems this fix don't work, since the closed?return false for these half closed socket.
Maybe the client also need to check this socket expiration and implicitly close the connection when time is expired

@Elyahou the problem is that the timeout is configurable on the broker side, but we won't know the actual value. I've thought about sending some kind of keep-alive request periodically, but I'm not sure if that's the best solution, as it would add unnecessary traffic.

I know Java client don't have this issue since 0.9 version. Maybe we should
mimic the same behavior as this client ?
On Aug 16, 2017 12:27 PM, "Daniel Schierbeck"<EMAIL_ADDRESS>wrote:

@Elyahou https://github.com/elyahou the problem is that the timeout is
configurable on the broker side, but we won't know the actual value. I've
thought about sending some kind of keep-alive request periodically, but I'm
not sure if that's the best solution, as it would add unnecessary traffic.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendesk/ruby-kafka/issues/395#issuecomment-322716106,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE5A33SNLbZ0Hy-Aik3kzcLSdFR9M3vXks5sYrXvgaJpZM4O4HbD
.

Can you investigate what they're doing in order to avoid the problem? They may do client-side socket cleanup with a lower timeout than the brokers, but I'm not sure if it's bullet-proof.

Fixed by #399!


Convert install docs to use bash code blocks for commands

We have a crazy idea (#91) which will allow our documentation to be "executed" by users (and by our CI systems. That ensures the docs are always correct. We need to modify the install docs to "flag" those code blocks that are executable by changing the basic code blocks:
    ```
    stuff
    ```

into a bash code block:
    ```bash
    bash commands!
    ```

@bergwolf @gnawux  - PTAL.  James has gone off the deep end :)

哈哈 :smile:
We'll obviously have to be careful where we run the scripts we generate, but we can sanity check the install docs and also use the script "in anger" for https://github.com/kata-containers/osbuilder/issues/97 for example.


Auto fix for jsx-max-props-per-line not in npm release

Looking at a fresh install of this library (7.0.1) but jsx-max-props-per-line does not contain an auto fix. The source on github does. I suspect something wonky happened in the last publish. Copy and pasting the rule into my node_modules works fine.

NVM, misread the blame, thought this was supposed to be released.


Added aria attributes to Login and Signup Forms

Description
Added aria attributes like aria-label, aria-invalid, aria-describedby, role, etc to login and signup forms of both customer and tailor.
Fixes #455
Type of change
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Hello @Avinash905!
Thank you for submitting this pull request Your pull request will be reviewed by maintainers shortly. Meanwhile make sure you have referenced the respected issue in this pr Your pr will not be merged if it does not have reference to respected issue
please read our Contributing Guidelines https://github.com/UBA-GCOEN/StichHub/blob/main/CONTRIBUTING.md And Code of Conduct https://github.com/UBA-GCOEN/StichHub/blob/main/CODE_OF_CONDUCT.md

@Naresh-chandanbatve Could you please add the labels?


Rework streaming interceptors

Streaming interceptors aren't very useful in their current state. Rather
than wrapping user code, they wrap the the networking primitives used by
the user's code. This is a subtle distinction, but it makes it
impossible to write a panic-recovering interceptor. It also makes common
server-side interceptors (e.g., authentication) needlessly difficult to
write.
This commit reworks the Sender and Receiver interfaces into a ClientConn
and HandlerConn. Just as unary interceptors wrap a UnaryFunc, streaming
interceptors now wrap ClientConnFuncs and HandlerConnFuncs. This ends up
being quite a bit easier to understand.
Connect's internals change a lot, but the user-visible changes are
modest. Today, we have:
type Sender interface {
  Spec() Spec
  Send(any) error
  Header() http.Header
  Trailer() (http.Header, bool)
  Close(error) error
}

type Receiver interface {
  Spec() Spec
  Receive(any) error
  Header() http.Header
  Trailer() (http.Header, bool)
  Close() error
}

type Interceptor interface {
  WrapUnary(UnaryFunc) UnaryFunc
  WrapStreamContext(context.Context) context.Context
  WrapStreamSender(context.Context, Sender) Sender
  WrapStreamReceiver(context.Context, Receiver) Receiver
}

With this commit, we change to:
type ClientConn interface {
  Spec() Spec

  Send(any) error
  RequestHeader() http.Header
  CloseRequest() error

  Receive(any) error
  ResponseHeader() http.Header
  ResponseTrailer() http.Header
  CloseResponse() error
}

type HandlerConn interface {
  Spec() Spec

  Receive(any) error
  RequestHeader() http.Header

  Send(any) error
  ResponseHeader() http.Header
  ResponseTrailer() http.Header
  Close(error) error
}

type ClientConnFunc func(Spec, http.Header) ClientConn

type HandlerConnFunc func(context.Context, HandlerConn) error

type Interceptor interface {
  WrapUnary(UnaryFunc) UnaryFunc
  WrapStreamingClient(ClientConnFunc) ClientConnFunc
  WrapStreamingHandler(HandlerConnFunc) HandlerConnFunc
}

Fixes #296.

I'm not going to go through all the code, but can we talk briefly offline about interfaces? Want to make sure I understand this.


Illuminate/Testing/TestResponse.php not returning decoded response

Laravel Version
11.x
PHP Version
8.1
Database Driver & Version
No response
Description
decodeResponseJson() here: https://github.com/illuminate/testing/blob/600e3d55288fec5dd00ebe330ce3766952611556/TestResponse.php#L998
This method called decodeResponseJson and with the comment "Validate and return the decoded response JSON." Returns the $testJson (which is asserted as vaid JSON) and not the $decodedResponse, which would seem like the correct data to return.
Maybe the comment is wrong and the name spurious. Otherwise I think that this should return the decoded JSON as implied.
I notice that this behaviour changed since 10.x
Steps To Reproduce
Run a test that requests decodeResponseJson() on a json output.
The returned data is the output of AssertableJsonString() and not the decoded response $decodedResponse, as stated.

Seems like a copy paste mistake indeed from the method below it. Would welcome a PR to adjust the comment! Thanks


🚀 Add link to LCC Noticeboard on WhatsApp, and make Instagram link more prominent

Resolved in v3.2.0


Change shebang line to be more universal (as per PEP 394)

PEP 394 states that distributions should bind the python command to python2, but this is not always the case (see Arch Linux for example), and this script assumes it points to python3.
Moreover, the Python executable very well might be located somewhere else than /usr/bin/python, replacing it with an /usr/bin/env call ensures Python is run regardless of where it is in the PATH.

Merged. Thanks


Frontend synk vulnerabilities

Closes: RHOAIENG-4843
Description
Upgrades frontend packages to remove vulnerabilities reported on snyk.
How Has This Been Tested?
odh-dashboard/frontend$ npm audit
found 0 vulnerabilities

Test Impact

No test added, just a package upgrade.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/retest


Updates and Fixes for Library Interface

UI fixes for /library page to make cards have same height and make only the cards list scrollable
Used actual uploaded media content data to display on library page
Added page for viewing a specific media content file
Fixed bug in media content admin interface where same image was being uploaded to consecutively uploaded media content

Let's see if we can merge this today 🙏


Adding normal user account

Hello, I'm curious why the script doesn't ask for a normal user and only leaves root as the initial user.
My extensive fork here has that feature. I wonder if it is a good idea if I create a pull request for that.
Thank you for your script.

Actually your fork is cool but it's like an entire new script (and you seem to know bash way better than I do, on a sidenote i literally wrote this script half assed and lazy).


[WIP] Add CombinedCertificateValidationContext.

Signed-off-by: JimmyCYJ<EMAIL_ADDRESS>Description: Implement a new certificate validation context type CombinedCertificateValidationContext, which has a default CertificateValidationContextoption and SDS config. This default CertificateValidationContext will be merged with dynamic CertificateValidationContext into a new secret to serve. This is option 4 in https://docs.google.com/document/d/12gdjGN5m3v4vxUnDAglCP6pyyMoeuVGAGo7D_jc27jw/edit?usp=sharing
Risk Level: Low
Testing:
Docs Changes: NONE
Release Notes: NONE
Fixes: #4763

LGTM

@lizan @htuch This is ready for review. Please take a look. Thanks.

@htuch This is ready to review. Please take a look. Thanks.


subscriptions array is empty when calling getSubscriptions with non-renewable skus on ios

Description
When retrieving non-renewable subscriptions on ios using getSubscriptions method, the subscriptions array is always empty. But if I were to call requestSubscription for that same sku, then I am able to purchase that subscription.

Expected Behavior
subscriptions shouldn't be empty

Environment:

react-native-iap: "^12.4.8"
react-native: "0.69.5"
Platforms: iOS simulator & device.

To Reproduce
Steps to reproduce the behavior:

call getSubscriptions({skus: ["some_non_renewable_sku"]});
Log subscriptions array.
Array is empty.

[Optional] Additional Context

Note that this issue does not occur on renewable subscriptions, I can call getSubscriptions with it, and subscriptions array will get updated with the data.

Fix: https://github.com/dooboolab/react-native-iap/pull/2180

Released in 12.4.10 please lt me know if any other issues

Released in 12.4.10 please lt me know if any other issues

So it's being returned by the getProducts function, shouldn't it be returned from getSubscriptions(since it's a non-renewable subscription)?.

I haven't found a more reliable way to check if a product is a subscription. So I have removed the filter. The fix was released on 12.4.12


Implement a basic transaction forwarder using TPU client

Create a basic transaction forwarder using TPUClient class of solana.
The transaction forwarder will recieve a transaction from and RPC port and forward the transaction to the current leader.
The transaction forward should be able to use both UDP protocol and QUIC protocol.
After successfully sending transaction LiteRPC should respond the sender with the signature.
Depends on : https://github.com/blockworks-foundation/solana/issues/1

This should basically work like sendTransaction if I am not wrong?


⚠ Bofrost.lu has degraded performance

In 283c13e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1139 ms

Resolved: Bofrost.lu performance has improved in 28cd66a after 6 minutes.


fix version_info if version can not be determined

The current implementation assumes that the version info can be split
into at least three parts using the period as delimiter. In certain
situations, such as running from a dowloaded release archive, or when
building from a git checkout that does not have any tags, this
assumption does not hold.
Fixes: #6125

I considered some tests for this function, but then was discouraged, as _gen_v_info is deleted again after having been defined and used and because the function is in numba/__init__.py I am not sure it is worth the effort to mock the del in that module during testing. I was unsure what the intent of this is, @stuartarchibald can you clarify? I think I would prefer to keep the function as a utility and additionally add something like myversion as a parameter so that it can operate on arbitrary versions rather than always getting the __version__ from numba. That would also significantly ease testing as nothing would need to be mocked.

I considered some tests for this function, but then was discouraged, as _gen_v_info is deleted again after having been defined and used and because the function is in numba/__init__.py I am not sure it is worth the effort to mock the del in that module during testing. I was unsure what the intent of this is, @stuartarchibald can you clarify?

Didn't want to pollute the top level namespace.

I think I would prefer to keep the function as a utility and additionally add something like myversion as a parameter so that it can operate on arbitrary versions rather than always getting the __version__ from numba. That would also significantly ease testing as nothing would need to be mocked.

Could the function be moved into numba.misc.init or something, imported in __init__, run and results stored, then deleted. This keeps the top level cleaner and also gives opportunity to test? It'd also present an opportunity to get a load of the other functions out of __init__ and somewhere testable.

This has now been updated and is ready for the first round of reviews.

https://github.com/numba/numba/issues/6167 is probably the cause of CI failure.

Changes have been made and the PR is ready for the next round of reviews.

/AzurePipelines run

CI fail is unrelated: https://github.com/numba/numba/issues/5973


[DOC] need comprehensive guide on Rust contracts compiled to Wasm

we need a comprehensive guide on rust contracts that would explain all nitty gritty details on what happens with Wasm compilation
related notes from recent thread between @nearmax and @mikedotexe

try running it on the node and check that everything works correctly, including cross-contract calls. run through steps in here and verify that it works correctly. make sure you remove old wasm file and target directory before compiling.
make sure you are using stable Rust in case they are experimenting with Wasm in nightly.
Wasm files are libraries and that specific lib annotation is designed to make it work correctly,
walk through https://rustwasm.github.io/book/ to get a better understanding what is the difference between Rust compiled to Wasm and Rust contract compiled to Wasm.
unit tests perform a non-trivial fixture needed for the contract to function.
when unit tests are compiled they are compiled for non-wasm platform and their code is slightly different (does not contain JSON/Borsh wrappers).

@amgando thank you so much for taking care for scoping it and creating an issue!
In addition to https://github.com/nearprotocol/near-bindgen/tree/master/examples/cross-contract-high-level here is piece of documentation that I send to people: https://github.com/nearprotocol/workshop

This looks like too deep for our documentation. Maybe for nomicon.io? We could add a link in our docs to https://rustwasm.github.io/book/ ?


ovsdb: Break up ovsdb mock apply method

Simplifies things a bit, main reason is the report card though.

Addressed comments, will merge once travis is happy


display the image from the debug console?

thanks for this extension this is amazing.
Would it be also possible to make it to be able to display the image when I do it in debug console.
I mean, lets say i have:
import torch as np
img = np.random.rand(100,100,3)

now I just can display the image (img) in the main file. would it be possible that I copy and paste np.random.rand(100,100,3) in the debug console like simple_view(np.random.rand(100,100,3)) and it display the image. simple_view is gonna be a function from your code that you tell us to run to see the image.
Thanks for this extension anyways :)

Thanks for using this extension.  But this extension is not a debugger extension, so it cannot read any information from the debug console. It just provides a standard extension command that you can press F1 to invoke. If you need to view an image with the debug console, I suggest you can create an image exporting method such as "simple_view(img)"  in your main.py, suppose this method will save the image as a certain filename, then you can call this method in the debug console, after all, keep the saved file open to view the image.


Improve SMTP logging output

It is currently impossible to debug why a custom SMTP connection fails as the only log entry related to this is a error 500 when requesting an email e.g:
2023-01-04 16:26:56 hasura-auth {"latencyInNs":32000000,"level":"error","message":"POST /user/password/reset 500 32ms","method":"POST","statusCode":500,"url":"/user/password/reset"}

It would be good to get the actual reason why e.g sending the email failed.
Additionally, it would be awesome to be able to send a testing email from the NHOST dashboard when setting the SMTP settings.

Hello Mirko, thanks for making this issue; I've replied to you in Discord (I've cheked the app states and deployment logs and haven't found anything amiss, plus the SMTP columns are filled):
- We have moved this to the infra team in order to check the services logs. In the meantime, could you double-check that the SMTP settings you've entered are correct? Both the host, user, and auth method?
I agree with to be able to send a testing email from the NHOST dashboard when setting the SMTP settings. I think it's a great suggestion, we'll definitely add :+1:

Hey @guicurcio, tried the SMTP settings with the nhost-cli and it worked, so it should work in Prod as well!

https://discord.com/channels/552499021260914688/1060038268252594186/1061349935213183047
> sheena — Yesterday at 3:25 PM

i had the same problem (and unfortunately still haven't found a combination of settings that lets me set secure = true) and agree that it would be good to add better logging and debugging for the SMTP settings. otherwise it's just trial and error and pretty hard to do on production. one suggestion @mirko is that we found it was easier to test the settings with the local nhost client until we found a combination that worked. much easier to try a bunch of values locally than to keep updating the settings in the nhost dashboard (especially since you have to make a commit every time since there is no way to force a deploy or restart services with new env)

one suggestion @mirko

wrong mirko :)
-> @mirko314

Fixed here: https://github.com/nhost/hasura-auth/pull/322
We'll try to release this on Nhost ASAP.


is lsp for all editor to implement or just for vscode

the elixir lsp could not do any thing outside the vscode world it just broken on emacs and not even work on native lsp on nvim.

Please use forum for support


neon-sso-session-proxy should encrypt cached objects

Use cookie cipher key to encrypt objects in Redis.
Also set objects to expire after 15 minutes.

DONE


Should support over 4.2 Giga adding files

Old version Zipstorer did not support big files if added file size was over uint range :). I have not tested this new version but I had feeling, when I looked source code, that there is same problem. You should check this out.

I will evaluate this feature. I don't have any explicit management of 4GB+ files.

I confirm that Version: 3.4.0 (August 4, 2017) contains the same "bug": trying to extract a 6GB+ file results in file truncation (at about 4GB). It would be really nice to fix this limitation.

Try switching the type from a uint to long. It should allow a greater size in bytes.

Let me know if that works for doing the change

I have reviewed the documentation for ZIP64. This is not a trivial task. Many things to consider.
See: https://www.loc.gov/preservation/digital/formats/digformatspecs/APPNOTE(20120901)_Version_6.3.3.txt

@ramihoo @stribizhev, Check out my ZipStorer64 fork: https://github.com/usr-sse2/zipstorer

@usr-sse2 you can always create pull request so we consolidate more features in a single project.

Added support for Zip64 in version 3.6.0
Please let me know if there is any issue


begin: require first char to be / or . when starting from local file

So as to provide better error messages when a user attempts to start a
build from a local file and this file doesn't exist, the user must
provide a path to the file starting with / or ..
Additionally, begin now properly cleans up whatever context it started
to make if an error is returned.

If this gets merged before the docs PR, I'll update that to reflect the new semantics of begin. If the docs PR gets merged first, I'll rebase and add those changes here.

I just updated the docs to reflect this change, so please also make sure the phrasing/spelling there is sane.

I guess I missed the reason why this is better than the old behavior (which check if the image exists on local first, if not treat it as a remote image. Also this is consistent with rkt fetch).

Shouldn't ~ get interpreted by the shell before it gets to rkt?

Either way the docs should state that it's allowed.

On Nov 16, 2015, at 18:37, Chance Zibolski<EMAIL_ADDRESS>wrote:
Shouldn't ~ get interpreted by the shell before it gets to rkt?
―
Reply to this email directly or view it on GitHub.

Just tweaked the docs to also mention ~. Once this gets merged I'd like to release v0.2.0. Anything else you'd like to see done to this @jonboulle?

Oh, and to answer Yifan's "I missed the reason why this is better than the old behavior", this is a UX thing. I can't provide a good error message when you typo a local file name, since if the file isn't local it'll do meta discovery, and then you'd get "no such host" when you typo a file name.

:sheep:


[Feature Request] Populate and activate popover dynamically

Existing Component
Yes
Component Name
Popover
Description
Is it possible to populate and activate popover dynamically?
In my case I have fullcalendar and want to activate popover on event hover but since there is only this code for calendar...
<full-calendar :events="events" :config="config" @event-render="eventRender" @event-selected="eventSelected" ref="calendar"></fullcalendar>
...then I can't wrap event in popover element.
Is it possible to trigger it from method like notifications for example or like with bootstrap where we can do something like this:
element.popover({
  animation: true,
  delay: 1000,
  content: '
some content
',
  trigger: 'hover',
  placement: 'auto',
  container: 'body',
  html: true
})

Maybe you need vue-popper mixin.


Account: Change the field level name for the Primary_Contact__c field to something that is more user friendly like "Primary Contact" (which would match NPSP)

Critical Changes
Changes

We've updated the label on the Primary_Contact__c field on the Account object from "_SYSTEM: One2OneContact" to "Primary Contact."

Issues Closed
Closes #559
New Metadata
Deleted Metadata
Testing Notes

Reverted - should be back to pre-robot change
On Wed, Oct 16, 2019 at 1:07 PM Carlos<EMAIL_ADDRESS>wrote:

@ceiroa requested changes on this pull request.
I've done some research on this, and everything looks good. Yes, we want
to rename it, and yes, we want it in the layout. The only thing to remove
is that robot test change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SalesforceFoundation/EDA/pull/965?email_source=notifications&email_token=AH77T2AV7YPF3LMDYBJ4TGDQO5Q6JA5CNFSM4JASOSA2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCIGIGDY#pullrequestreview-302809871,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH77T2FV3EUR62ODG2SCORTQO5Q6JANCNFSM4JASOSAQ
.

--
Doug Shattuck
Test Automation Engineer | Salesforce
Mobile:<PHONE_NUMBER> <(385)%20242-9732>
http://smart.salesforce.com/sig/dshattuck//us_mb_kb/default/link.html

As per discussion, will not include in Page Layout for now, and will revisit adding after re-testing the logic. Tested / Passes QE. Confirmed field is not on Page Layout. Confirmed API name did not change.


Ability to create overlapping blanket impls

I want the following code to become valid Rust:
trait T{
    fn foo(&self);
}
trait A{}
trait B{}

impl<S> T for S where S: A {
    fn foo(&self){
        println!("foo");
    }
}

impl<S> T for S where S: B {
    fn foo(&self){
        println!("bar");
    }
}

struct X {
}

impl A for X {
}

impl B for X {
}

fn main() {
    let x = X{};
    <x as A>.foo();
    <x as B>.foo();
}

This has some uses in game development. We can already put multiple traits with the same method on the same object, so why not bump that up a notch? Methods that take an A would call the A's variant of foo, methods that take a B would call the B's variant of foo, methods that take a both would have to pick the one they wanna call.
(I do not have a name for this feature.)

You could take a look at https://github.com/rust-lang/rfcs/blob/master/text/1210-impl-specialization.md

This has some uses in game development.

You'll need to clarify that.
Bear in mind that just because a pattern works in a different language doesn't mean it must work in Rust as well. Inheritance based OOP is a common pattern that doesn't work in Rust, but there are patterns that fill the same use cases.

Methods that take an A would call the A's variant of foo, methods that take a B would call the B's variant of foo, methods that take a both would have to pick the one they wanna call.

For anything like this to work you'll also have to figure out how to deal with the fact that everything in the language assumes that a type can only implement a trait once.

trait HasFullness {
  fn fullness(&self) -> f32;
}

trait Inventory {
  fn slots(&self) -> usize;
  fn size(&self, usize) -> usize;
  fn get_item(&self, usize) -> usize;
}

trait Tank {
  fn size(&self) -> usize;
  fn get_fluid(&self) -> usize;
}

// because rust traits are way closer to composition than inheritance,
// and in fact I've described them as "glorified compile-time ECS" multiple times,
// it makes sense for some "entity" to be both a tank and an inventory.
// however, tanks and inventories have different concepts of fullness.
// it would be nice to be able to have different implementations of fullness for each --
// even if they had to be blanket impls.

There should be distinct traits for distinct concepts. What about parameterizing the trait?
trait Fullness<C: ?Sized> {
    fn fullness(&self) -> f32;
}

trait Inventory {}
trait Tank {}

struct Foo;
impl Inventory for Foo {}
impl Tank for Foo {}

impl<T> Fullness<Inventory> for T where T: Inventory {
    fn fullness(&self) -> f32 { ... }
}

impl<T> Fullness<Tank> for T where T: Tank {
    fn fullness(&self) -> f32 { ... }
}

fn main() {
    let foo = Foo;
    Fullness::<Inventory>::fullness(&foo);
    Fullness::<Tank>::fullness(&foo);
}

Wow, I never knew traits could take traits as type parameters.  Is that used anywhere currently?
In fact, I suppose <Inventory> denotes the trait object type for Inventory, so it works for the purpose of distinguishing, but maybe you'd need <dyn Inventory> in the distant future.

Yeah I know, just thought up using it for a handy marker type. I don't know if it's used anywhere...

In fact Inventory and Tank should just be structs in your example.

Actually no they shouldn't
struct GlassJar {
}

impl Inventory for GlassJar {
  // one slot, holds crackers
}

impl Tank for GlassJar {
  // holds milk
}

// you can put milk and crackers in the same glass jar to make soggy crackers.

That's one way to do it; I'm saying that if you want something like HasFullness you can get it to work by having Inventory and Tank be minimal structs, and have the jar implement HasInventory which has a function which returns the Inventory (on which you can do whatever)

This sort of overlap would violate coherence which is required for the soundness of the type system. As it is not possible to do that, we cannot do this so I'll close this as non-actionable.

It doesn't violate coherence, it just tweaks coherence a little.
It's like implicit newtypes/named impls/etc in many ways.

Come up with a detailed proposal that tweaks coherence in a backwards compatible way. Discuss it on the internals forum, not here. We've been over this.


Library doesn't recognize leader change

HI @asher, hey I noticed that the library doesn't recognize when a leader changes and even if you include all servers in the list during initialization it still tries to communicate only to the first one in the list replying with:
\n\n<meta http-equiv="Content-Type" content="text/html;charset=ISO-8859-1"/>\n<title>Error 500 </title>\n\n\n
HTTP ERROR: 500
\n
Problem accessing /v1/scheduler/graph/csv. Reason:\n
    Error proxying request to leader (maybe the leadership just changed?)\n\nError:\njava.net.ConnectException: Connection timed out (Connection timed out)\n\tat java.net.PlainSocketImpl.socket

is there a workaround for this or am I missing something, thank you.

That is surprising, leader election works for me.
However this is not a problem with the library. Any HTTP request sent to chronos like that will return that same error.
Are you running the different chronos processes in a restricted network mode? (in docker in NAT) so they can't talk to each other?


Updated the update model function so that it also works for algebraic equations

[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #166 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
+ Coverage   96.44%   96.45%   +<.01%
==========================================
  Files          37       37
  Lines        1436     1440       +4
==========================================
+ Hits         1385     1389       +4
  Misses         51       51

Impacted Files
Coverage Δ

pybamm/models/base_model.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


whatsnew for #3681

🚀 Pull Request
Description
"What's New" entry for #3681
Just checking I've got the hang of this 😁.

Consult Iris pull request check list

Thanks @rcomer!


unusable parameter - [Event Request] T77 "Report Selections" - procedure GetPurchaseOrderEmailAddress() - OrderAddress

https://github.com/microsoft/ALAppExtensions/issues/10804
The publisher added was
local procedure OnBeforeGetPurchaseOrderEmailAddress(BuyFromVendorNo: Code[20]; RecVar: Variant; ReportUsage: Enum "Report Selection Usage"; var OrderAddress: Record "Order Address"; var EmailAddress: Text[250]; var IsHandled: Boolean)

The var OrderAddress there is pointless, since (a) the subscriber just gets a brand-new uninitialised record, and (b) if they did anything to change it via the var that would just get trampled anyway by the OrderAddress.Get() later in GetPurchaseOrderEmailAddress(). I see no reason to pass OrderAddress, var or not.

We cannot remove an existing parameter. What would be a correction to make OrderAddress useful?

I have no idea; that's why it should've been thought about properly while adding the event!
The original requestor presumably included the parameter because their suggestion was to if IsHandled then exit(OrderAddress."E-Mail"). But MS's code does if IsHandled then exit(EmailAddress). Which makes more sense; there'd be no point passing a record just to set/get one of its fields' values. But it seems that wasn't thought about, and it got left in.
Passed OrderAddress is not used again in the function until many lines later, after many other possible exit points, and then its 'use' is only to overwrite all its values via a .Get().
So I don't see how it can possibly be of any use to anyone, and anyone writing to it via var or expecting you to exit(OrderAddress."E-Mail") is going to be disappointed, as it won't be used.

Full current code from 17.5 for reference:
local procedure GetPurchaseOrderEmailAddress(BuyFromVendorNo: Code[20]; RecVar: Variant; ReportUsage: Enum "Report Selection Usage") EmailAddress: Text[250]
var
    PurchaseHeader: Record "Purchase Header";
    OrderAddress: Record "Order Address";
    RecRef: RecordRef;
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeGetPurchaseOrderEmailAddress(BuyFromVendorNo, RecVar, ReportUsage, OrderAddress, EmailAddress, IsHandled);
    if IsHandled then
        exit(EmailAddress);

    if BuyFromVendorNo = '' then
        exit('');

    if ReportUsage <> Usage::"P.Order" then
        exit('');

    RecRef.GetTable(RecVar);
    if RecRef.Number <> DATABASE::"Purchase Header" then
        exit('');

    PurchaseHeader := RecVar;
    if PurchaseHeader."Order Address Code" = '' then
        exit('');

    if not OrderAddress.Get(BuyFromVendorNo, PurchaseHeader."Order Address Code") then
        exit('');

    exit(OrderAddress."E-Mail");
end;

You are absolutely right.
Though there is only one way to get rid of this parameter is to obsolete this event and create another one without this parameter. But it will lead to the code like this:
    IsHandled := false;
    OnBeforeGetPurchaseOrderEmailAddress(BuyFromVendorNo, RecVar, ReportUsage, OrderAddress, EmailAddress, IsHandled);
    **NewOnBeforeGetPurchaseOrderEmailAddress(BuyFromVendorNo, RecVar, ReportUsage, EmailAddress, IsHandled);**
    if IsHandled then
        exit(EmailAddress);

and we will be able to remove OnBeforeGetPurchaseOrderEmailAddress in 3 versions later (18+ months from now).
As this parameter may cause some confusion as you mentioned, but having two twin events seems to look even worse.
I prefer to keep that unnecessary parameter as removing it brings more noise than help. Would you agree?

Yes, I think right now the best thing is just to leave it, and have this issue as a reference for how we know it doesn't do anything.
Could there be a way to deprecate individual arguments, at least for events? That way we could signal clearly that they're not used/useful, omit them by default when looking-up events, warn if still present in old code or manually added, etc.

As I know there is no such an "obsolete parameter" feature in the backlog.
You could submit a product suggestion on https://aka.ms/bcideas

done: https://experience.dynamics.com/ideas/idea/?ideaid=9243395f-769b-eb11-89ee-0003ff45b7ac

Great! Thank you.


Cannot currently create HexagonDelegate, but was working in the past

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
org.tensorflow:tensorflow-lite:0.0.0-nightly-SNAPSHOT and org.tensorflow:tensorflow-lite-hexagon:0.0.0-nightly-SNAPSHOT with hexagon libraries v<IP_ADDRESS>
Custom Code
Yes
OS Platform and Distribution
Android 13
Mobile device
Samsung Galaxy A71
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
The HexagonDelegate was working fine via a Java Android app on my device (Samsung Galaxy A71 with Snapdragon 730), but then i changed some things on the project (libraries versions, manifest) because i had issues with running some transformer models on the GPU and since then it is not working and produces the error:
Failed to load libhexagon_interface.so, Error: dlopen failed: library "libadsprpc.so" not found: needed by data/app/~~o6sPBobgR5g-DMgV2fEROg==/com.example.-8QtHh6qQZAELc86wE5E8IA==/lib/arm64/libhexagon_interface.so in namespace classloader-namespace
I have also tried stable versions from 2.9.0 to 2.12.0 instead of the nightly snapshots and the error persists.
I can share my project if someone could look into it.
Standalone code to reproduce the issue
On a Java project in Android Studio:

Follow the instructions in https://www.tensorflow.org/lite/android/delegates/hexagon#hexagon_delegate_java_api

build.gradle (:app):
def version = "0.0.0-nightly-SNAPSHOT"
implementation "org.tensorflow:tensorflow-lite:$version"
implementation "org.tensorflow:tensorflow-lite-hexagon:$version"
implementation "org.tensorflow:tensorflow-lite-select-tf-ops:$version"

After running the application on the device:

> adb shell ls /data/app/~~o6sPBobgR5g-DMgV2fEROg==/com.example.<project>-8QtHh6qQZAELc86wE5E8IA==/lib/arm64
libhexagon_interface.so
libhexagon_nn_skel.so
libhexagon_nn_skel_v65.so
libhexagon_nn_skel_v66.so
libtensorflowlite_flex_jni.so
libtensorflowlite_hexagon_jni.so
libtensorflowlite_jni.so

> adb shell cat /proc/cpuinfo | grep Hardware
Hardware        : Qualcomm Technologies, Inc SDMMAGPIE

> adb shell cat /sys/devices/soc0/soc_id
365

> adb shell getprop ro.product.device
a71

> adb shell getprop ro.board.platform
sm6150

Library "libadsprpc.so" is present in /system/vendor/lib

Code to create a HexagonDelegate:
Interpreter.Options finalOptions = new Interpreter.Options();
try {
    mHexagonDelegate = new HexagonDelegate(this);
    finalOptions.addDelegate(mHexagonDelegate);
    finalOptions.setUseXNNPACK(mOptions.useXNNPACK);
    mInterpreter = new Interpreter(mModel, finalOptions);
} catch (UnsupportedOperationException e) {
    // Hexagon delegate is not supported on this device.
    Log.e(TAG, "Hexagon delegate is not supported on this device.");
    return;
}

Relevant log output
Android Studio log:

2023-04-24 09:14:43.588 19191-19449 tflite                  com.example.<project>             E  Failed to load libhexagon_interface.so, Error: dlopen failed: library "libadsprpc.so" not found: needed by /data/app/~~o6sPBobgR5g-DMgV2fEROg==/com.example.<project>-8QtHh6qQZAELc86wE5E8IA==/lib/arm64/libhexagon_interface.so in namespace classloader-namespace
2023-04-24 09:14:43.588 19191-19449 tflite                  com.example.<project>             I  Hexagon Delegate is not supported.

@ioannispan,
Could you you make sure that you're using the latest nightlies - may be clear the gradle cash ?
We just cloned tensorflow examples and updated image classification example to include hexagon. I followed the guide
by adding the gradle dep in build.gradle
and created directory jniLibs/arm64-v8a and included the files i got from the libhexagon_nn_skel extraction. as explained here
Also please have a look at this issue https://github.com/tensorflow/tensorflow/issues/55364 and https://github.com/tensorflow/tensorflow/issues/53380 for the reference. Thank you!

@tilakrayal
I've cleared the gradle cache and done all the steps in the guide, but the error persists. The issues you stated did not help.
UPDATE
I've tried using the TFLite Model Benchmark Tool with Android Apk (https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/tools/benchmark/android) to see if i'll have the same issue with Hexagon. In order to be able to use the Hexagon delegate i do the following:
adb push libhexagon_interface.so /data/local/tmp
adb push libhexagon_nn_skel.so /data/local/tmp
adb push libhexagon_nn_skel_v65.so /data/local/tmp
adb push libhexagon_nn_skel_v66.so /data/local/tmp

where libhexagon_interface.so comes from this link from this guide and libhexagon_nn_skel(_v65/6).so come from this link (v<IP_ADDRESS>) from this guide.
When i set use_hexagon=true and don't specify the hexagon_lib_path, then the path is set to:
I tflite  : Hexagon lib path: [/data/app/~~K6aeHsxXcgmL2FRThR7mBQ==/org.tensorflow.lite.benchmark-tPmERhUnFAHAenBJZq4OQw==/lib/arm64]
and i get the error:
W tflite  : Failed to fetch Hexagon NN version. This might be because you're using incompatible versions of libhexagon_interface and libhexagon_nn_skel. You must use compatible versions. Refer to Tensorflow Lite Hexagon Delegate Guide.
I tflite  : Hexagon Delegate is not supported.
W tflite  : Could not create Hexagon delegate: platform may not support delegate or required libraries are missing

But when i specify hexagon_lib_path=/data/local/tmp/, then the delegate is applied succesfully:
I tflite  : Hexagon delegate created.
I tflite  : TfLiteHexagonDelegate delegate: 61 nodes delegated out of 61 nodes with 1 partitions.
I tflite  : Replacing 61 out of 61 node(s) with delegate (TfLiteHexagonDelegate) node, yielding 1 partitions for the whole graph.

Does this help with the problem in my Android app in any way?
Thanks

Hi @ioannispan
Although the documentation suggests to use v<IP_ADDRESS> , the config has v<IP_ADDRESS> as latest version to use for nightly snapshots which can be downloaded from here.
Could you please try with libhexagon_interface.so using v<IP_ADDRESS> and see if it resolves the issue?
Thanks.

Hello @pjpratik
I think i am missing something. The link you provided for v<IP_ADDRESS> does not contain libhexagon_nn_skel(_v65/6).so files.
I copy the new libhexagon_interface.so to my Android jniLibs/arm64-v8a/ directory, but the error persists:
Failed to load libhexagon_interface.so, Error: dlopen failed: library "libadsprpc.so" not found: needed by /data/app/~~3WH2wsowpMNgALxjaSkcCw==/com.example.<project>--XTLMj82qPraRQM25waMKg==/lib/arm64/libhexagon_interface.so in namespace classloader-namespace
Should i put libhexagon_interface.so somewhere else?
Thank you

Hi @ioannispan
I have observed that the  path for hexagon_lib_path has to be /data/local/tmp/ as per the build file for the benchmark tool.
https://github.com/tensorflow/tensorflow/blob/785f7f02f78c6af70ee79729ce95724e89a221bd/tensorflow/lite/tools/benchmark/BUILD#L38
Are per the hexagon delegate documentation,

'libhexagon_interface.so' which holds the interface that the delegate uses. It must be available if you linked the hexagon_delegate library to TFLite. You can load it either from shell by overriding LD_LIBRARY_PATH=$LD_LIBRARY_PATH:"path to the so", or add it inside your apk in a way it is available.

'libhexagon_nn_skel(_v65/_v66).so' which holds the DSP code. Use TfLiteHexagonInitWithPath(..) and provide the path to the directory which holds the shared libraries for the Hexagon NN on device. If you're using TfLiteHexagonInit() then You will need to set environment variable "ADSP_LIBRARY_PATH" to "path_to_the_lib";/system/lib/rfsa/adsp;/system/vendor/lib/rfsa/adsp;/dsp Note that separator here is ';' not ':' You can push all 3 files, and the library will pick the one needed based on the runtime. Or if you are sure of what you will use on the device then push only one of them.

Could you please confirm if you are following this instructions to set .so files?
Thanks.


Migrate odometry display

Closes #80

also ports covariance visual and covariance property as it is used in odometry display
makes covariance property and visual work like other visuals and properties
further improve Visual Testing Framework: can now easily search for subproperties

CI:

Linux
Linux-aarch64
macOS
Windows

I have no idea why this always fails - it's not rviz. I updated my ros2.repos file, but that's apparently not it.

It looks like the repos file used in the CI above is not exactly up to date, as it doesnt contain: https://github.com/ros2/ros2/pull/491. That's why CI is failing complaining about a missing libyaml_vendor package

--- stderr: rcl_yaml_param_parser
00:07:40.622 CMake Error at CMakeLists.txt:8 (find_package):
00:07:40.622   By not providing "Findlibyaml_vendor.cmake" in CMAKE_MODULE_PATH this
00:07:40.622   project has asked CMake to find a package configuration file provided by
00:07:40.622   "libyaml_vendor", but CMake did not find one.
00:07:40.622
00:07:40.622   Could not find a package configuration file provided by "libyaml_vendor"
00:07:40.622   with any of the following names:
00:07:40.622
00:07:40.622     libyaml_vendorConfig.cmake
00:07:40.622     libyaml_vendor-config.cmake
00:07:40.622
00:07:40.622   Add the installation prefix of "libyaml_vendor" to CMAKE_PREFIX_PATH or set
00:07:40.622   "libyaml_vendor_DIR" to a directory containing one of the above files.  If
00:07:40.623   "libyaml_vendor" provides a separate development package or SDK, be sure it
00:07:40.623   has been installed.
00:07:40.623
00:07:40.623
00:07:40.623 ---

Weird, because I did update it just today... I guess I must've forgotten to enter the new URL before rebuilding... Thanks for looking into this!
Here's hopefully better CI:

Linux
Linux-aarch64
macOS
Windows

New CI after rebase:

Linux
Linux-aarch64
macOS
Windows


Improvement/fix 3700 platfrom flash issues

changes include:

Removing the debug config at the direction of @esarver
Adding the 37xx platform
A. Changing the flash portion of the code to match what has previously been done
B. Adding the newly created instrument to mod.rs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


iOS app submission

"Your app is using the Advertising Identifier (IDFA). You must either provide details about the IDFA usage or remove it from the app and submit your binary again."
Getting this message from apple when submitting app for review.
Using Mixpanel/Mixpanel (2.9.9)
No other 3rd party include this framework.
Also no AdSupport.framework is added to build settings Linked Frameworks and Libraries.
Please advice.

Using Cocoa Pods

Hi @neerajgoyal12 ! If you are not using AdSupport.framework at all, then you need to include MIXPANEL_NO_IFA preprocessor macro in your build settings. You can see https://github.com/mixpanel/mixpanel-iphone/blob/v2.9.9/Mixpanel/Mixpanel.h#L373 for more details.
Let me know if you are not able to do it and I'll walk you through the steps!
Related: https://github.com/mixpanel/mixpanel-iphone/issues/112 https://github.com/mixpanel/mixpanel-iphone/issues/109

Hi @patedit I did submit without setting it in build settings to the app store and said yes to using adFramework, the app might get rejected and will post it here for an update.
Thanks was able to set it, though confused to set it at the app target level or pod target level, please clarify, am doing it at both the places for now, and submit a new binary if the earlier one gets rejected, so please suggest urgently.

@neerajgoyal12 Setting the preprocessor macro at Pods-Mixpanel target level should be enough. If you want to test it, you could run the app on release mode and check if https://github.com/mixpanel/mixpanel-iphone/blob/master/Mixpanel/Mixpanel.m#L1012 is executed. It should not 👍

@patedit will submit with just the pod level preprocessor and let you know about the rejection, if not will close the issue ...
Thanks for your help and quick reply

@patedit  Even after adding the macro at Mixpanel level both at debug and release getting the same message
"Your app is using the Advertising Identifier (IDFA). You must either provide details about the IDFA usage or remove it from the app and submit your binary again." on app submission.
Please advice

@patedit sending you the screen shot for running in release mode on a 4s simulator
the ifa is not nil, in fact it is nil for debug mode, though setting the same preprocessor macro for both debug and release setting at Mixpanel pod target

Make sure you don't have AdSupport.framework linked in your project and not other library is linking to that framework. If ifa is not nil it means that AdSupport was linked somehow. Also, for both debug and release preprocessor macros check that your Resolved column contains MIXPANEL_NO_IFA.

hi @patedit This time I just updated cocapods to version 1 and did pod update, set the preprocessor macro MIXPANEL_NO_IFA at pod mixpanel level only for release and didn't get the message
"Your app is using the Advertising Identifier (IDFA). .... "
Also using
[Mixpanel sharedInstanceWithToken:kMixpanelToken launchOptions:launchOptions];

instead of
[Mixpanel sharedInstanceWithToken:]

in
- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions

i don't know if that changed the things, also preprocessed Mixpanel.m and made sure IFA returns nil always. I think, I will close the issue, described it here in full for further assistance to anyone else facing a similar problem.
Thanks

Thanks @neerajgoyal12 ! Happy it worked.


Add IT for PRs

This is testing already a lot of things:

Maven and Gradle udpate
create standard project, update, build and check version in pom.xml or gradle.properties
this is able to check different versions udpates
we should add new versions to check when new versions arrives

There is update-tests.csv where we can add specific versions tests.
This is a good start, there is everything here to test other scenarios (with specific extensions..., with pre-existing projects, ...).
I also renamed the module unit-tests to tests and excluded it from the release.

We need to investigate why the unit tests are not passing on Windows

I am merging this, it's important to have it for new PRs


xDisk needs unit tests

I added unit tests for issues #23, in PR #21, but it still needs tests to cover existing functionality.

Working on this now but need #38 merged before I can complete it.

Have completed full unit test coverage on this as part of #53.


Update Readme

A PR for test purpose
curl \
  -X POST \
  -H "Accept: application/vnd.github.v3+json" \
  -H "Authorization: token ghp_************************************" \
  https://api.github.com/repos/nsavelyeva/go-shopping/pulls/3/comments \
  -d '{"body":"dummy comment to test PR Comments API"}'

Responds with:
{
  "message": "Invalid request.\n\nNo subschema in \"oneOf\" matched.\n\"commit_id\", \"path\", \"position\" weren't supplied.\n\"in_reply_to\" wasn't supplied.\n\"commit_id\", \"line\", \"path\" weren't supplied.",
  "documentation_url": "https://docs.github.com/rest"
}

Maybe this will help: https://github.com/sigmavirus24/github3.py/issues/374
Links:

https://docs.github.com/en/actions/creating-actions/creating-a-docker-container-action
https://docs.github.com/en/rest/reference/pulls#create-a-review-comment-for-a-pull-request

This PR was not intended to be merged, it was created for debug purpose


dnsdbq: 2.6.7 -> 2.6.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for dnsdbq is: C99 program that accesses passive DNS database systems
meta.homepage for dnsdbq is: https://github.com/dnsdb/dnsdbq
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.6.8 with grep in /nix/store/vw52a6m9f0fcfl3cff2bk8f38gj2lm47-dnsdbq-2.6.8
found 2.6.8 in filename of file in /nix/store/vw52a6m9f0fcfl3cff2bk8f38gj2lm47-dnsdbq-2.6.8

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

dnsdbq

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/vw52a6m9f0fcfl3cff2bk8f38gj2lm47-dnsdbq-2.6.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A dnsdbq https://github.com/r-ryantm/nixpkgs/archive/70aeff5814a04f7a1a7eec659525cd129f9ada57.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/70aeff5814a04f7a1a7eec659525cd129f9ada57#dnsdbq

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vw52a6m9f0fcfl3cff2bk8f38gj2lm47-dnsdbq-2.6.8
ls -la /nix/store/vw52a6m9f0fcfl3cff2bk8f38gj2lm47-dnsdbq-2.6.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

dnsdbq

Maintainer pings
cc @x123 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Looks good here.
@NixOS/nixpkgs-merge-bot merge


LabelFile.. no attribute 'to_training_data'

from mlimages.model import LabelFile, ImageProperty
lf = LabelFile("path/to/training_data.txt", img_root="path_to_your_image_folder")
prop = ImageProperty(width=32, gray_scale=True)
td = lf.to_training_data(prop)
td.make_mean_image("path/to/mean_image")  # store mean image to normalize
for d in td.generate():
# d is numpy array that adjusted according to ImageProperty, and normalized by mean_image!
# only you have to do is train the model by it!
print(d)
the error ..........................................................................................................................
AttributeError                            Traceback (most recent call last)
 in ()
5 prop = ImageProperty(width=6000, gray_scale=False)
6
----> 7 td = lf.to_training_data(prop)
8 td.make_mean_image("/home/yared/Documents/RELEASE_FOLDER/Data/")  # store mean image to normalize
9
AttributeError: 'LabelFile' object has no attribute 'to_training_data'

LabelFile fuction calls to_training_data attribute in model.py from but has no such thing.

Sorry for the un-updating the document.
You can get training data by list(lf.fetch())

It's OK. Thanks. I am trying to generate "mean_image.png" "batch ".. but i got error .... i have to do some pre-processing to train my model before send it  " list(lf.fetch())" .i have images and "training_data.txt. "
""""
Training
Now, you have images and training_data.txt. But you have to do some pre-processing to train your model. For example...
resize image
normalize image
sometimes change color...  """

Am trying to do the above part ... but got error. The error is the first post that I made. If you have time, could you tell me how can i solve the error.
AttributeError: 'LabelFile' object has no attribute 'to_training_data'

Hello, I have the same issue when I try to call
td = lf.to_training_data(prop)
If we just call list(lf.fetch()) to obtain training data, how can we incorporate prop into lf? Thank you very much!


End has overworld biomes

The end now has overworld biomes:

I have the follow installed:
Biomes o Plenty
Traverse
Hexlands
Stygian End

Ok, I've tried just about everything I can think of and cannot get the bug to show up for me. Tried SE with each of the other 3 mods individually and it all looked normal. Tried SE with all 3 mods combined on normal, BoP, hexlands, and BoP hexlands world types and all of them looked normal. Tried using Natures compass to find a forest biome in the end and still nada.
So I'll need more info here. Does this bug happen with only those 4 mods installed? What world type did you use? Are the overworld biomes rare, or is the entire end dim converted to overworld biomes?

I'll start separating mods in a testing instance and I'll report back as I get the information.

I can confirm that Stygian End did cause this, also updated responses:
What world type did you use?:
Hexlands, BOP Hexlands, Vanilla, Quark.
Are the overworld biomes rare, or is the entire end dim converted to overworld biomes:
The entire end is converted (yes the 1000+ block outskirt end islands with the end cities is also converted).
Does this bug happen with only those 4 mods installed? (UPDATED MOD LIST):
I compiled all the mods I has that deal with terrain generation and dimensions:
BiomesOPlenty-1.12.2-<IP_ADDRESS>9-universal
hexlands-1.8.1
AutoRegLib-1.3-20 (Quark Library)
RebornCore-1.12.2-<IP_ADDRESS>7-universal
Optifine_1.12.2_HD_U_E2
QuarkOddities-1.12.2
Quark-r1.5-130
Simple-Void-World-1.12-<IP_ADDRESS>-universal
Traverse-1.12.2-1.6.0-69
ActuallyAdditions-1.12.2-r143
SimpleDimensions-MC1.12-1.3.1
somelikeitdry-1.1.0
worleycaves-1.3.0
twilightforest-1.12.2-3.8.689-universal
stygian-1.0.1
When I had Stygian End the biomes were overworld like, when I removed it, it went back to normal in a new word.
(Also side note, I made a post a while ago about BOP biomes not generating when Traverse was installed even when I had the generator set to BOP Hexlands, that is still happening to me even in this condensed list of mods I put together for testing this original overworld end bug)

Think I've solved the issue. Was a bad interaction between Stygian End and Some Like It Dry.
Also for your other issue about BoP biomes not generating with Traverse, I believe Some Like It Dry might be causing that as well. Some Like It Dry and BoP are not compatible, but I believe BoP has its own settings for reducing the number of oceans. Feel free to reopen or post a new issue if that issue still happens with SLID.


APIs for dynamically creating thread stacks

Is your enhancement proposal related to a problem? Please describe.
There are use-cases for creating thread stacks at runtime from a heap, rather than statically allocate them. Doing this has some special requirements:

At creation time it must be decided whether the created stack will have the capability to host a user thread, or only a supervisor thread.
To create a stack with requested size N, requires reserving a memory region of size X with alignment Y, where both X and Y are functions of N that vary per-arch. We now have macros to help with this:

For size, X = Z_THREAD_STACK_SIZE_ADJUST(N) or X = Z_KERNEL_STACK_SIZE_ADJUST(N), for user and supervisor-only stacks respectively
For alignment, Y = Z_THREAD_STACK_OBJ_ALIGN(N) or Y = Z_KERNEL_STACK_OBJ_ALIGN, for user and supervisor-only stacks respectively. In the latter case, the alignment is not a function of N.

If the stack can host user thread, kernel object metadata must be set up properly

A struct z_object metadata must be created for the stack object and placed within the dynamic tables, this is handled by z_dynamic_object_create()
If CONFIG_GEN_PRIV_STACKS=y, then the privilege mode stack memory must also be allocated, this is a fixed-size kernel-only stack. A struct z_stack_data must also be allocated and properly configured.

Describe the solution you'd like
A public API for doing this so the end user doesn't have to figure all of this out. I think there needs to be flexibility on where the memory comes from, although in a lot of cases thread resource pool is appropriate.

duplicate, closing


Raising Events

Great little plugin works fantastically!
A great addition to this would be assigning some kind of callback to the directive which is called when the onSuggestedItemsClick() event is raised.
In my case I want to do additional work each time my ng-model passed to the directive changes.
Have you thought about making this into a .component() also?

Thanks sicollins for feedback.
I have already developed 3 major callback addition to this plugin & would be releasing new version in couple of days. In new release you will get follow things :

beforeAddItem : type - function, which will be called just before user select/add an item to ng-model.
afterAddItem : type - function, which will be called after user add an item to ng-model.
beforeRemoveItem : type - function, which will be called just before user delete/remove an item from ng-model.
afterRemoveItem : type - function, which will be called after user delete/remove an item from ng-model.

Apart from these, if you need any more functionality then please let me know.
Thanks

Great work! Fantastic - will update when you release.

Hi,
I have added event listeners, you can find its docs on readme/home page.
So, i am closing this issue. If you still need anything you can raise issue and for more ideas & suggestion you are most welcome.


extend README on usage of registry w/ k3d on podman

rendered markdown is at https://github.com/leelavg/k3d-registry-dockerd/tree/podman-docs

Even though talk is cheap, I want to mention that I also planned to create a tool like this from some time, thanks a lot for making it happen!
This was the notes I prepared, use https://github.com/containers/image and implement some of the interfaces mentioned in https://github.com/containers/image/blob/main/types/types.go by following https://github.com/containers/image/tree/main/openshift as an example, however your implementation just gets the work done in a straight forward way.
I observed in the container logs Not Found statements even though everything should be there in the cache while creating a second identical cluster. Will relook at it.

Thanks for the contribution @leelavg. What's the difference from running it against Docker? Is it just creating the registry ahead of time to get around the missing bridge network?

Mostly yes, as a side effect of creating registry before creating cluster we need to supply registry details explicitly.
I also believe creating registry ahead of time helps for reusing the same for multiple clusters.
Remaining part of doc just shows how to run k3d with rootless podman.

Thanks @leelavg . I'd rather stick to just the information about this project, and not duplicate k3d's instructions for running with podman (I also can't replicate it... apparently podman has issues with k3d on macOS).
Do you think an added section on "running a registry separate from a specific cluster" would be helpful? Like:
Or, to use a separate-but-still-k3d-managed registry:

```sh
k3d registry create -i ligfx/k3d-registry-dockerd:v0.4 \
  -v /var/run/docker.sock:/var/run/docker.sock \
  myregistry

configfile=$(mktemp)
cat << HERE > "$configfile"
apiVersion: k3d.io/v1alpha5
kind: Simple
registries:
  use:
    - k3d-myregistry:5000
  config: |
    mirrors:
      "*":
        endpoint:
          - "http://k3d-myregistry:5000"
HERE
k3d cluster create mytest --config "$configfile"
```.

I'd rather stick to just the information about this project

ack, current PR has specific about k3d with podman and podman with the project. Yes, the former can be omitted, however the suggestion doesn't reflect the usage w/ latter, like we need to supply a flag to mention the network for podman
Nevertheless, it's still fine not to take this PR for not adding podman, I can close it.

I've added another section in the readme about running as a standalone k3d registry in 18f59ca


cocoaspod 0.39 with awaitkit

i'm use cocoaspod 0.39, when i'm run pod update, there some errors below:
[!] Unable to find a specification for AwaitKit (~> 1.0.0)
the awaitkit must to use cocoapod 1.0.0?

Hi arden,
This is normal, for the moment the AwaitKit is still under dev and I didn't published it to cocoapods yet.
I push a modification to the PromiseKit and I need to wait for merge and new release before...
For the moment, if you want test or use it in your project you will you to copy the files manually and add this line to your Podfile: pod 'PromiseKit', :git => 'https://github.com/yannickl/PromiseKit'

@yannickl  i see, thanks.

FYI I just published the library. So you can now use it via CocoaPods.


Use Timeout.timeout instead of Object#timeout

Fix #263.
This change suppresses the following warning on Ruby 2.7:
warning: Object#timeout is deprecated, use Timeout.timeout instead.

Object#timeouts is removed from Rails 3.0, so god cannot be used with Rails 3.0.
$ irb
irb(main):001:0> RUBY_VERSION
=> "2.7.0"
irb(main):002:0> timeout(1) {}
(irb):2: warning: Object#timeout is deprecated, use Timeout.timeout instead.
=> nil

$ irb
irb(main):001:0> RUBY_VERSION
=> "3.0.2"
irb(main):002:0> timeout(1) {}
(irb):2:in `<main>': undefined method `timeout' for main:Object (NoMethodError)
        from /home/owner/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/irb-1.3.5/exe/irb:11:in `<top (required)>'
        from /home/owner/.rbenv/versions/3.0.2/bin/irb:23:in `load'
        from /home/owner/.rbenv/versions/3.0.2/bin/irb:23:in `<main>'

o god... i have company here... i am facing that exact problem

@mishina2228
Object#timeout is deprecated but still available in ruby 3.0.2
https://github.com/ruby/ruby/blob/v3_0_2/lib/timeout.rb#L124-L134
% docker run --rm -ti ruby:3.0.2
irb(main):001:0> RUBY_VERSION
=> "3.0.2"
irb(main):002:0> timeout(1) {}
(irb):2: warning: Object#timeout is deprecated, use Timeout.timeout instead.
=> nil
irb(main):003:0>

But if timeout gem is updated to 0.2.0 or later
by gem install timeout or bundler, Object#timeout is removed.
https://github.com/ruby/timeout/commit/14d421252b54027108da183783a7a26d0cf5aac1
% docker run --rm -ti ruby:3.0.2 /bin/bash
root@f74035cc99bc:/# gem install timeout
Fetching timeout-0.3.0.gem
Successfully installed timeout-0.3.0
1 gem installed
root@f74035cc99bc:/# irb
irb(main):001:0> RUBY_VERSION
=> "3.0.2"
irb(main):002:0> timeout(1) {}
(irb):2:in `<main>': undefined method `timeout' for main:Object (NoMethodError)
	from /usr/local/lib/ruby/gems/3.0.0/gems/irb-1.3.5/exe/irb:11:in `<top (required)>'
	from /usr/local/bin/irb:23:in `load'
	from /usr/local/bin/irb:23:in `<main>'
irb(main):003:0>

@mishina2228 Did you update timeout gem?
Anyway, to run god with ruby 3.1+, this PR is necessary.

@minimum2scp Ah, you're right! I was using timeout v0.2.0.
$ ruby -v
ruby 3.0.2p107 (2021-07-07 revision 0db68f0233) [x86_64-linux]
$ gem list timeout

*** LOCAL GEMS ***

timeout (0.2.0, default: 0.1.1)

Thanks for pointing out and explaining in detail!
I will edit comments...

+1 on needing this

+1 as its a need now

+1 as it's needed now

A forked version of this gem has been released, resurrected_god, with homepage https://github.com/mishina2228/resurrected_god. It's a drop-in replacement for god which includes this fix and supports ruby 3.1.


Add a long description to setup

Include the readme as the long_description argument to setup and set
its content type appropriately. This allows PyPI to render the content
correctly.

Coverage decreased (-0.07%) to 92.476% when pulling b4c44a820f48d22090e131e27ae8f58738649ecb on jonafato:pypi-long-description into 71a054af92a45e2a8b5d7c4fb435841949e1895c on spulec:master.

Coverage decreased (-0.07%) to 92.476% when pulling b4c44a820f48d22090e131e27ae8f58738649ecb on jonafato:pypi-long-description into 71a054af92a45e2a8b5d7c4fb435841949e1895c on spulec:master.

Coverage decreased (-0.07%) to 92.476% when pulling b4c44a820f48d22090e131e27ae8f58738649ecb on jonafato:pypi-long-description into 71a054af92a45e2a8b5d7c4fb435841949e1895c on spulec:master.


Android Basics: change-app-theme

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-change-app-theme?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-change-app-theme#8
In which task and step of the codelab can this issue be found?
Step 9. Challenge task
Describe the problem
It says we have to adjust the theme attributes based on the https://github.com/material-components/material-components-android-examples project. And then it provides the link https://github.com/material-components/material-components-android-examples/blob/develop/MaterialThemeBuilder/README.md for the adjustments. The problem is that the link is not found and I don't have the guide for this challenge. Please make it more clear so that we can practice on this fun challenge :). Well pardon me I just checked that it appeared only if the language page is changed to my native language "Indonesian", when I changed to English there is no challenge provided but learn more instead. So I hope it helps you to make the translation feature even better.
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Duplicate of #241 , #222 , #219

@rahmadanu
Thank you for reaching out to us and reporting this localization issue. The actions for fixing it will be taken soon.


Exception with bleeding edge SDK

Analyzer Feedback from IntelliJ
Version information

IDEA IC-173.4674.33
<NO SDK>
IC-173.4674.33, JRE 1.8.0_151-b12x64 Oracle Corporation, OS Mac OS X(x86_64) v10.13.3 unknown, screens 1680x1050  Retina

Exception
Please append the contents of:
file:///private/var/folders/cp/32g44l548q8d0bfj4_l9s3m003ky/T/report.txt
Dart analysis server, SDK version 2.0.0-edge.0d5cf900b021bf5c9fa593ffa12b15bcd1cc5fe0, server version , FATAL error: NoSuchMethodError: The getter 'length' was called on null.
Receiver: null
Tried calling: length
#0      Object.noSuchMethod (dart:core-patch/dart:core/object_patch.dart:46)
#1      FolderBasedDartSdk.getSdkProperty (package:analyzer/src/dart/sdk/sdk.dart:690:14)
#2      FolderBasedDartSdk.defaultSdkDirectory (package:analyzer/src/dart/sdk/sdk.dart:677:26)
#3      Driver.start (package:analysis_server/src/server/driver.dart:389:12)
#4      main (file:///Users/scheglov/Source/Dart/sdk.git/sdk/pkg/analysis_server/bin/server.dart:12:11)
#5      _startIsolate.<anonymous closure> (dart:isolate-patch/dart:isolate/isolate_patch.dart:277)

We can test for null, but that would only move the failure to somewhere else. I'd like to understand when we are unable to find the path to the script being run and how to reliably find the location of the SDK.

@scheglov Should we close this as being stale, or do we still want to try to look into it?


Update

Fix README
Change stat_periodic to stats_periodic
Change stat_sum to stats_sum
Add description to all properties
header: Remove time_units_per_sec from required
stats:

Rename requests to queries
Rename answers to responses
Rename responses to response_rcodes
Rename answer_latency to response_latency

Merging for now so other people can access the generated HTML. We will revisit this later.

I'll merge now and refer the comments into new issues

Header:
time_units_per_sec ... remove! Always use microseconds.
timestamp_start    ... usec-precision timestamp of measurement start (can be zero, or unixtime, or anything random)
Stats:
since         ... usec-precision timestamp of periodic stat begin
until         ... dtto
period_number ... sequential number of this periodic stats
Event-based counters:
conn_init               ... sent SYN packets (TCP) or Initials (QUIC, where it's equal to session_init + resumption_init)
conn_established        ... TCP handshakes completed (in DoQ, this is equal to session_established + resumption_established)
session_init            ... attempted initiating TLS session
session_established     ... TLS/QUIC handshakes completed
resumption_init         ... attempted resumpting TLS session
resumption_established  ... TLS/QUIC handshakes completed by resumption
resumption_fallbacks    ... attempt to session resumption denied by counterpart, continue to session_init (NOTE: in case of DoQ, this can also lead to "lost" DNS queries)
resumptions_try         ... TLS/QUIC session resumptions attempted
resumptions             ... TLS/QUIC session resumptions successful (NOTE: this is ALSO counted as "established") [TODO right now we are assuming that 0-RTT DNS query is sent on every successful resumption on DoQ and never on DoT]
queries         ... sent DNS queries
replies         ... received DNS replies
close_sent      ... initiated graceful connection close (FIN sent in TCP)
closed          ... gracefully closed connections
reset_send      ... abruptly closed by us
reset_recv      ... abruptly closed connections by counterpart (only those RSTs that belong to existing conns?)
unexpected_DNS  ... received unexpected DNS packet
unexpected_raw  ... received unexpected packet that is NOT(!) DNS (e.g. TCP SYN+ACK without matching SYN), any ballast on wire
discarded_DNS   ... DNS queries from given input that were not sent (e.g. excessive size of DNS query on input)
discarded_raw   ... any packet that was not able to send (BESIDES! discarded_DNS)
State-based periodic timers (also in the case when measurement canceled prematurely, this will contain non-zeros at the end):
establishing  ... handshakes pending
connected     ... established connections alive
ongoing       ... DNS queries waiting for reply (not timeouted)
Size counters (in bytes, cumulative, not average!):
DNS_query_size  ... DNS payload ONLY (even without 2-byte prefix in TCP)
DNS_reply_size
wire_size_out
wire_size_in
Latency timers (in usecs, avg/min/max/stddev/buckets):
initial_latency       ... within handshake, from initial packet sent to first reaction recv
handshake_duration    ... from initial packet to handshake (incl. TLS/QUIC) completed (possibly multiple round-trips)
response_latency      ... from DNS query sent to DNS response recv
user_latency          ... from initial packet sent to DNS response recv (including handshake ONLY if it was needed)
connection_duration   ... from initial packet sent to connection closed (or timeouted, failed, abandoned...)


[Deleted] As a new user, I should receive an email inviting me to join the app to access my class

[ ] Create sign up UI
[ ] Create unique registration link using AES-256 encryption that points to sign up page for student
[ ] Validation of input (password and name)
[ ] add new student user to the database (name, email, hashed password and class)

Validation: Create a new user with an existing email. After sign-up, check existing email to see if if an email was received.
Story point: 3
Time estimation: 8hr

This will be done in milestone 3 because we need the email campaign first #65

This issue has been deleted since it does not reflect the new on-boarding for students. View Scrum notes for more details


docs: add revision in glossary.rst

Pull Request Test Coverage Report for Build 837660950

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.97%

Totals

Change from base Build 835879194:
0.0%

Covered Lines:
4276

Relevant Lines:
5347

💛  - Coveralls


[BUG]  after 9.2 unable to load

Describe the bug
A clear and concise description of what the bug is.
Account type

[ x] Installer
[ ] DIY / DHZ
[x ] Home Owner (This integration is not (fully) functional with a home owner account!)

Envoy

[ x] Metered with CTs installed

[ ] Metered without CTs

[ ] Standard

FW version: D7.xxx

Amount of micro inverters connected: 12

Home Assistant

Version: 2024.9.2

Additional context
Envoy *************

Geen apparaten of entiteiten

Instellen mislukt, wordt opnieuw geprobeerd: 'NoneType' object has no attribute 'status_code

Relevant snippet of Home Assistant error log.

I have the exact same issue

I manage a Standard installation, but it shows the same error.
Reverting to version 0.4.5 seems to fix the issue ...

Cam confirm 0.45 solves isseu


Update the packaged build process to run the client in non-dev mode

The client should be run via npm run start-app as opposed to npm start. This will launch purely the Electron app without any helper dev processes.

Nevermind, the bundled app is already being started via electron . which bypasses any dev processes.


Max Digest Iteration loop triggered by using ng-bind with a filter that returns an Array

When ng-bind is used, and the expression includes a filter that returns an array, angular enters an infinite digest loop and eventually cuts out after the 10 digest limit.  I looked at the source and the issue is caused by the fact that ng-bind never uses object equality (angular.equals) to compare the new value and the old value in the watcher, rather since the filter is returning a new array, the old array is never equal and an infinite digest loop is caused.
The same expression will work if you use double curlies for binding rather than ng-bind
See plnkr which shows the bug in action.
http://plnkr.co/edit/t6B3uz

Instead of binding the array directly, convert it to a string with a filter first.


Smaller version?

File size is getting substantially large. Could we have a smaller version for client side use? My usecase is 2 views a "chat" view that disallows HTML and an "article view" that allows formatting.

Probably would shrink a lot if the CSS support were an optional module you
had to import separately, which is something that could be done in a new
major release, if others were actively involved in supporting the project.
We don't use it client side.
On Sat, Feb 22, 2020, 3:01 PM Charles<EMAIL_ADDRESS>wrote:

File size is getting substantially large. Could we have a smaller version
for client side use? My usecase is 2 views a "chat" view that disallows
HTML and an "article view" that allows formatting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/sanitize-html/issues/330?email_source=notifications&email_token=AAAH27JTO26XMU46VRQBXQ3REGACPA5CNFSM4KZUF3R2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IPQENVA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27NHLT2R23IXFKXYWBLREGACPANCNFSM4KZUF3RQ
.

@boutell Is bundling with including dozens of browser polyfills necessary if the project is targeted to node?

The plan for the next major version is that the "main" will not involve any
transpilation or bundling and if there is a prebuilt browser version —
which I'm thinking no, because people know how to use webpack these days —
it won't be "main".
On Sat, Jul 4, 2020 at 4:12 AM Bogdan Chadkin<EMAIL_ADDRESS>wrote:

@boutell https://github.com/boutell Is bundling with including dozens
of browser polyfills necessary if the project is targeted to node?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/sanitize-html/issues/330#issuecomment-653737166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27N6AXH6TSY4WVUFMOLRZ3P6XANCNFSM4KZUF3RQ
.

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

Please see the newly published<EMAIL_ADDRESS>There is no build, so it can be run through your build as needed. We'll welcome notes on how to trim any more fat before the stable version.


Replace vouch cookie checks with calls to UIS

Portal - replace vouch cookie checks with calls to UIS to ensure user is properly logged in (current method relies on setting cookie with same name - may not be future-proof)

https://github.com/fabric-testbed/software-planning/issues/30
UIS

GET /whoami returns 401 unauthorized when no auth cookie exists;
GET /whoami returns 403 forbidden when the user has logged in but hasn't completed self-signup yet;
GET /whoami returns 200 OK when the enrolled user has logged in (with valid cookie).

Portal

Call UIS /whoami when the user first opens a portal page by checking if the sessionStorage mentioned below exists;
Store 'hasValidCookie' boolean value in browser using sessionStorage to remember that the user is authorized and authenticated;
The 'hasValidCookie' sessionStorage value is going to be removed when the user clicks 'Log out' or the window or tab is closed or (a timeout we set in config based on Vouch Proxy/ CILogon auth expiration time?)


Fix a sockets bug and add a note about GCHandle recycling.

The note re: recycling will turn into an issue once this is merged into corefx.

@stephentoub PTAL

LGTM

LGTM


[BEAM-7501] 2.7.1 LTS backport: fixes concatenated compressed files bug with python sdk

This cherrypicks #8187 and #8784 to 2.7.1 LTS release

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@dlesco FYI in case you are interested

LGTM. Thanks.

All test suites that exist for 2.7 are healthy. Merging.


Added Signal version 5.54.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

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

My educated guess: The merge + publishing is triggered by setting the Validation-Completed. Sometimes this fails or gets stuck so the bot tries to trigger it again by resetting the label.


Add new Accordion designs

🤔 Questions for the reviewer

Do you guys are happy with the icons in CSS?
Do I have to remove the Arrow icon as it's not longer used?

🖌 Designs
Designs are not defined as 100% as I don't have a reference for the open state. Ask the design team is they are happy. I used this reference: https://marvelapp.com/e14bhc4/screen/67764982/handoff

Mine looks like this:

Do you guys are happy with the icons in CSS?
I see no issue in fact I would say it's better because you don't have extra elements to load. 👍
Do I have to remove the Arrow icon as it's not longer used?
Yes please.

@moshie I need to open a PR removing the existing icons and using font-awesome ones instead ( and add the font-awesome import to Styleguidist ), is the last thing I plan to do on ZRC before my furlough 📐

@moshie I need to open a PR removing the existing icons and using font-awesome ones instead ( and add the font-awesome import to Styleguidist ), is the last thing I plan to do on ZRC before my furlough 📐

No problem, the Arrow can still be removed

I will leave the deprecated button to @lluia PR as he's going to kill all the folder

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: