Error after upgrading to 2.1.18 "MissingSerial"

I've been using this great integration for more than a year, but after upgrading 2.1.18 (and persisting in 2.1.19) I've been getting this error, and the device is no longer available in HA:
This error originated from a custom integration.

Logger: homeassistant.config_entries
Source: custom_components/openevse/__init__.py:71
Integration: OpenEVSE (documentation, issues)
First occurred: March 25, 2023 at 13:16:56 (2 occurrences)
Last logged: March 25, 2023 at 20:44:58

Error setting up entry OpenEVSE for openevse
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 383, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/openevse/__init__.py", line 71, in async_setup_entry
    data = await manager.test_and_get()
  File "/usr/local/lib/python3.10/site-packages/openevsehttp/__main__.py", line 194, in test_and_get
    raise MissingSerial
openevsehttp.exceptions.MissingSerial

EDIT: Using the openevse web interface works fine.

You're fast. Thanks a lot!


Creates a simulator for focusing an element on firefox.

This is for testing only.
Firefox does not fire focus event by element.focus() method. So that, we need to call both, element.focus() and metal.dom.triggerEvent(). The focus() method will update the document.activeElement and we need to trigger the event calling metal.dom.triggerEvent().

Just started reviewing :) :octocat: Sent from GH.


Inform minimum supported vim version

At the moment vim-prettier has been developed with vim8+ in mind.  We should however be more clear about it on our README and make sure we can identify what is our minimum supported vim version.
We should also investigate what is required to guarantee backwards support with older vim versions.

Added support for vim 7 by fix incompatibility bug when calling system() using list, as vim 7 only supports string
fixed by https://github.com/mitermayer/vim-prettier/pull/29


Ctrl-K (Mac) should delete to newline, not block end

Usually blocks don't have newlines in them, but when they do (shift-enter in a list item, for example), Ctrl-K should delete to the newline, not the end of the block.
Also, Ctrl-K should delete an empty block.

Unfortunately I don't have the energy right now to figure out the draft-js build + test system, but here's what I got working by editing SecondaryClipboard.js directly in my node_modules folder:
https://gist.github.com/danneu/89cbcf64bcb62995b91cb80acf8bf79b
(Main change was the addition of lines 13-30)

ctrl-k deletes til (but not including) next newline or it deletes til end of block
if next char is newline, then delete it and stop

I recall @sophiebits addressed this issue with 645549343e7248ae07bf8e274de971acfe1c53c0, so it can be closed

@claudiopro That's a different issue; can you reopen?
Repro for this one:

Open Workplace composer
Type * a opt-enter b then go to before the a
Ctrl-K

Expected: Only the "a" is deleted.
Actual: "a", the newline, and "b" is deleted

In the line selection.set('focusOffset', blockEnd);, instead of using blockEnd it should find the first newline whose position is greater than the current focusOffset and then cut up to (but not including) that char.

Thanks for pointing that out, I'll look into that 👍


Home Screen -> Search bar requires HTML encoded URLs

Hello,
Tested in Chrome
The serach engine requires HTML encoded URLs.
Reproduction steps:

In Wikipedia.org, search for Model-view-controller and you are sent to this URL: http://en.wikipedia.org/wiki/Model-view-controller
Copy this URL and place it into the Search Bar on the home screen and it encodes it to: http://en.wikipedia.org/wiki/ Model%E2%80%93view%E2%80%93controller

If you search using the encoded URL it works fine and you are directed to the correct page. However, if you search for http://en.wikipedia.org/wiki/Model-view-controller without the encoding you get:

If my assumption about this is correct, then to resolve this you may need to do the encoding in the controller instead of the view; because if you encode the URL in the view then you'll have to double encode if the user types in the url from memory.

The wikipedia article referenced above contains em dashes in its title (Model–view–controller). I believe what's occurring here is that URLs containing em dashes are encoded when pasted into the wikiwash searchbar. Currently, when a user attempts to search for an encoded URL (try http://en.wikipedia.org/wiki/%3F!_(album)), the associated page is not retrieved, although the article page's search bar correctly decoded the link.

Expected Results:

Any valid, encoded URL that is "washed" should successfully retrieve its associated page.

Tested on:

OSX Yosemite, Chrome 41.0, 1366x768 resolution


The behavior of numbervars/4, binding interval solutions

Hi!
Not sure if this a problem or not, but should the code below return the interval (1.21, 1.2100000000000006)? The predicate below instead fails.
test:-
    {X == 1.1 * 1.1},
    numbervars(X, 0, _, [attvar(bind)]),
    writeln(X).

I would expect this to fail. numbervars unifies the first argument with a term of the form $VAR(N). When X is an interval, it can only be unified with a numeric value or another interval. So when the option is attvar(bind) this causes a unification failure. attvar(skip) should work.
I assume you're trying to assign a more meaningful/simpler name to internal intervals.  This has crossed my mind, but couldn't think of any way where the costs outweigh the benefits. Let me know if you have any ideas.

I understand. My goal was to "pretty print" variable names, without the underscore format.
With this:
test1:-
    {X * X == 4},
    numbervars(X, 0, _, [attvar(skip)]),
    Y = someFunctor(this, X, that, _),
    writeln(Y).

I get the output:
someFunctor(this,_17060{real(-2,2)},that,_17628)
What i ended up doing to solve my problem was to strip the variable name for any attributed variables only keeping the interval:
test2:-
    {X * X == 4},
    Y = someFunctor(this, X, that, _),
    Y =.. L,
    maplist(parse_arg, L, L2),
    Y2 =.. L2,
    numbervars(Y2, 0, _),
    writeln(Y2).

parse_arg(I, O):- attvar(I), I::O.
parse_arg(I, O):- \+attvar(I), I = O.

Which yields:
someFunctor(this,real(-2,2),that,A)
Kind regards, JC

OK. One thing worth considering: by stripping the var you lose the ability to distinguish between two intervals with same domain, e.g., [X,Y]::real(1,10) would result in X and Y both being output as real(1,10) (which may or may not matter to you).

Yes, i would prefer to be able to have both the variable and interval. I just couldn't figure out how to keep the variable and pretty print its name (i.e. A instead of _58926395).

A hack that seems to work to get the variable is to convert to and from a string with term_string/2. The code below produces uninstantiated named variables (A, B, and so on) when the interval is too large, and a midpoint solution when the interval is small.
:-use_module(library(clpBNR)).

test:-
    {X * X == 4}, %here the resulting interval is to large (-2, 2) for a midpoint to say something meaningful
    {Y == 1.1 * 1.1}, %the interval is small enough to make a midpoint meaningful
    F = someFunctor(this, X, that, _, Y, X, nested(Y, X)), %a compound, where X and Y appear multiple times (to check if numbervars gives them the same name)
    F =.. L, %convert compound to list
    maplist(parse_arg, L, L1), %parse arg, see below
    F2 =.. L1, %convert parsed list back to compound
    term_string(F2, F3), %convert compound to string
    term_string(F4, F3), %convert string back to compound
    numbervars(F4, 0, _), %do numbervars on this compound
    writeln(F4).

parse_arg(I, O):- attvar(I), delta(I, D), {D < 0.0001}, midpoint(I, O). % replace with midpoint if the interval is reasonably small
parse_arg(I, O):- attvar(I), delta(I, D), {D >= 0.0001}, O = I. % continue using attvar if the interval is too large
parse_arg(I, O):- \+attvar(I), \+compound(I), I = O. %not attvar and not compound so just use the old argument
parse_arg(I, O):- compound(I), I =.. L, maplist(parse_arg, L, L1), O =.. L1. %term is a compound so apply parse_arg to each argument

Yields:
someFunctor(this,A,that,B,1.2100000000000004,A,nested(1.2100000000000004,A))


ag doing multi-line matching by default

I was looking for file in the FreeBSD ports tree, containing github.com but that did not start with a W.  So, I wrote:
# ag '^[^W].*github.com'
archivers/hs-zlib-enum/pkg-descr
3:
4:WWW: http://github.com/maltem/zlib-enum

archivers/nwreckdum/pkg-descr
3:
4:WWW: https://github.com/danfe/nwreckdum
....

And it seems .* means "any char" including new line.  Is there a way around that ?

Ag utilizes pcre(3) for regex matching, with the PCRE_MULTILINE feature enabled.  In multiline mode, \n is just another character.  Quoting from an online man page:

By  default,  for  the purposes of matching "start of line" and "end of line", PCRE treats the subject string as consisting of a single line of characters,  even if it actually contains newlines.....

When  PCRE_MULTILINE  is set, the "start of line" and "end of line" constructs match immediately following or immediately  before  internal newlines  in  the  subject string, respectively, as well as at the very start and end.

Nothing unexpected there, but from the point of view of traditional line-at-a-time grep'ing, your Ag search results are surprising.  Here's my theory of what's occurring:

The input stream is double-spaced (i.e., \n\nWWW: http://blahblah).
^ matches after the first \n  (so far, so good)
[^W] matches any char but "W" and \n is any char. So [^W] matches on the second newline in the input stream (double spaced text).
After that, .* matches all text up to "github.com" .

@nodakai in #459 has the solution. Change the regex like so:
^[^W\n].*github.com

Looking thru #459, it can be seen that a negated character class caused issues there as well.  The Ag man page probably needs to be amended to discuss the pitfalls of multiline matching and negated character classes.

A man page commit is now available that discusses multi-line matching.

I would love to see this multi-line feature -M built-in to ag

@ldong, +1

+1

There seems to be a --nomultiline option, but it does not appear to work as intended.  It definitely does something (ran a diff against the output of ag with and without the --nomultiline option and there were differences), however .* still matches against newlines:
ag --nomultiline 'class="modal.*(@keydown.esc){0}' returns this:

Add Thing

When it should have returned this:

Add Thing

https://github.com/ggreer/the_silver_searcher/issues/1215
https://github.com/ggreer/the_silver_searcher/issues/459
https://github.com/ggreer/the_silver_searcher/issues/560
https://github.com/ggreer/the_silver_searcher/issues/682
https://github.com/ggreer/the_silver_searcher/issues/713
https://github.com/ggreer/the_silver_searcher/issues/921


Ethane does currently not compile: "Multiple applicable items in scope"

There is currently an upstream issue with the funty package. Current workaround to make Ethane compile is to fix the version of the broken package:
cargo update -p funty --precise 1.1.0

This is the issue causing the problems: myrrlyn/funty#3

We have to take initiative here. A fix in v0.1.1 has been released to crates.io and cherry-picked into dev. So everything should work fine now.


Discern Activities of Interest Through Real-Time Analysis and Periodic Analysis - needs slides and videos posted

for Ops Server release
Deadline 9Oct
see email for post location
@nfeuerstein

Slides have been copied. For video do you mean the technical workshop video?
@dfoll.

I will not have time to make a video so I am closing this.


Cache image dimensions between different runs

#3 made me think of caching dimensions persistently to prevent them from being reloaded when the page is reloaded or the image is encountered again. Local storage can be used for that on a per-url basis.
Main challenges:

The entries in the cache will pile up and would have to be cleared periodically to prevent it from growing too large. Seems easy enough by including a timestamp and removing entries after x days.
Race conditions. If the script is opened on multiple tabs and both tabs write to local storage at the same time, something's going to clash. If two tabs try to load the same image which doesn't exist in the cache, both of them would load it. There would need to be some synchronisation between the tabs and/or locking to prevent this. OTOH, this is probably not that big of a deal...
Size limits. Local storage is limited in size, and I think it would be really easy to reach those limits given the number of images that one could see if intensively browsing MB.

Alternatively, we could use IndexedDB, but that might incur portability issues..

So I'm using an IndexedDB object store now to create a cache from image URL to its dimensions. That offers the great benefit of not being limited in size as much, overcoming problem 3, and solves most of the more dangerous issues with challenge 2 by using transactions. Issue 1 is solved by keeping a timestamp in local storage and clearing old (2 weeks) entries every day.
There's still some synchronisation problems between tabs though. If two tabs now try to store different images, everything should work fine, but if two tabs need to load the same image, they'll still both load the same image. I can't see a way to overcome this issue without going completely overkill in inter-tab synchronisation, and any attempt to do so would probably lead to more problems than it would solve. To be clear, this situation won't corrupt the cache in any way, but one of the two tabs would be doing duplicate work.


Help with adding ID (XYZWxxxx:) with every message

Hello @matth-x , I'm continuing to make use of this excellent repo for my project and was wondering if I can add an ID (eg: Serial No) with every message.
Current message goes as --> : receive message [2, "6964602", "StatusNotification", {"connectorId": 1, "errorCode": "NoError", "info": "null", "status": "Available", "vendorErrorCode": "0x00"}]
Requirement --> XYZW224121G2907: receive message [2, "6964602", "StatusNotification", {"connectorId": 1, "errorCode": "NoError", "info": "null", "status": "Available", "vendorErrorCode": "0x00"}]
I want to add that tag "XYZW******" to every message, Can you guide me with potential changes needed to achieve this?

Please Ignore this, I found a way without tweaking the messaging structure.


Refactor download module

Improve error handling when the MD5 file is not there (404).

Only skip existing file if MD5 matches (i.e. if file has been updated on server, redownload)
If MD5 of downloaded file is incorrect, retry once, before failing


Dynamic chart's timestamp should match with log and CSV timezones.

When using Naarad and looking at log or CSV files the timezones do not match and makes it harder to correlate. We should update the graphs to reflect the same time as the log or CSVs so it is easy to match up.
Logs and CSVs are generated based on the server time from raw logs. Dynamic chart, I believe, will generate based on the browser's timezone?

#328 is related as fixing the logs and dynamic chart can be done in same time.


Can't play mp4 online

let video = PhotosPickerItem.loadTransferable(type: Data.self) to get the video data and pass it to server.appendRoute return HTTPResponse(statusCode:.ok, body: video!),Can't play mp4 online

Hi, does the file download correctly using a downloader like wget or curl ?
Very large files should download correctly and most browsers will simply download the mp4 and play it.

Yes, server.appendRoute downloads files correctly, can FlayFox do static resource management similar to Jetty, I can use Jetty for online playback, I don't know if FlayFox needs to add headers to HTTPResponse for online playback


Rundown at end of profiling doesn't work for dynamic or collectible methods

From https://github.com/dotnet/coreclr/pull/27238 (comment: https://github.com/dotnet/coreclr/pull/27238#issuecomment-543421341)
Looks like PerfView currently only does a rundown when profiling is stopped, and relies on .NET Core to perform its own rundown on graceful process shutdown, in order to identify methods that were already jitted prior to when profiling was started.
This system does not work well for dynamic and collectible methods because their lifetime may expire prior to the rundown. On top of that, other dynamic or collectible methods may be allocated in the same memory thereafter, so the late rundown information may also lie to PerfView about where the samples actually went.
Proposal:

Do a rundown upon starting profiling
Don't do a rundown upon stopping profiling
Following the initial rundown, PerfView would get events to keep it up to date on changes occurring to the system
Probably some work would also be necessary to handle method unload events to track lifetime of limited-lifetime code ranges
Also theorizing that there may be some work involved in retaining the initial rundown information when using a circular buffer after it fills up completely
Offer a config option to change back to the previous mechanism (late rundown). There may be some overhead in rundown so users may in some specific cases prefer to perturb the system as little as possible when starting to profile.

The unload case here is interesting and I'm not sure if/how it is handled today.  There have been debates about whether rundown should occur at the beginning or the end of the trace.  So far, there has never been a strong argument for one over the other, given that the rundown will always have to be augmented with events during the trace to ensure completeness.  I would propose that if we need to make a change, from a cost/benefit perspective, we keep rundown at the end as is, but augment it with data from collectible assemblies at load/unload time so that PerfView knows about any code that may have run during the assembly lifetime.  The load/unload also represents the bounds during which samples can be mapped to that assembly.

That would work too and may be easier to implement. We may already be getting method unload events (not the DCStopVerbose event), but maybe not the IL to native map. If a mini-rundown could be down on module unload along with the necessary info, that would be great.


liar

The author is a scammer and the compilation did not pass

there is no code to compile anyways how could it fail

Instruction not clear, now I'm on Mars 🚀


Ask for an example for SNAPEdgeListPartitionReader

I downloaded the dataset from Stanford SNAP network repository. And I am trying to use SNAPEdgeListPartitionReader to read the *.edges files in the dataset. But I cannot understand the mapNodeIds of the three input arguments for read(std::string path, std::unordered_map<node,node>& mapNodeIds, Graph& G).
Would anybody help me with an example using SNAPEdgeListPartitionReader.read()?

Don't you want to read the graphs only? This can be done with the read method, see the user guide. Probably something like this:
G = readGraph("your.graph", Format.SNAP)

Some context: SNAPEdgeListPartitionReader also reads a partition of the vertex set, I believe.

If I understand correctly, SNAPEdgeListPartitionReader needs the input node ID -> internal node ID map that SNAPGraphReader emits. However, I'm not too familiar with the reader and I have not really looked at the code. Can someone confirm this?

@nie93 Did that solve your problem?

@avdgrinten Not really..

Did you try passing the map (it's a public member variable) from SNAPGraphReader to SNAPEdgeListPartitionReader?

SNAP dataset format comprises of 5 data formats for each partition, which are *.circles, *.edges, *.egofeat, *.feat, *.featnames. Based on my use case, I only need to use .edges files.
But what I don't get is, which kind of input data is designed for SNAPGraphReader. It returns RuntimeError: unable to read from file when I tried to read .edges files using it.


update Croatian kuna symbol

the proper way to write 100 Croatian Kunas is "100 kn" not "100kn"

A rebased version of this has been merged to master. Thanks!


feat: added saving session to localStorage

https://www.loom.com/share/7d14b87b1ada45069bfb5320fb2b1a8b

Thats looks like a great improvement @sjsakib 👌


support dockerfile-maven-plugin in TemporaryJobsBuilder.imageFromBuild()

Currently, the imageFromBuild() method in TemporaryJobsBuilder()
supports reading the image_info.json produced by docker-maven-plugin
to find the name of the image to use in the TemporaryJob. This file is
read from either the classpath or the filesystem if the former fails.
This change adds support for reading the target/docker/image-name file
produced by dockerfile-maven-plugin as well.

cc @gilles

The tests for this are pretty hacky. A much cleaner approach would be to add some sort of "FileReader" interface to TemporaryJobBuilder that hides the logic of reading from the classpath or from the target/ folder so I could swap that out in the test with a temporary file, but that felt like a more intrusive change.

@davidxia @negz @gimaker @rohansingh @dflemstr

nice.
For the test since you're reading via a Resource, you should be able to just put test files in test/resources/target/... and test/resources/docker/

Found a test which was relying on imageFromBuild() produced by some other module in the project; the test doesn't actually care what image is used - seems like it worked on accident before. Fixed in 7518d82

Current coverage is 49.04%

Merging #902 into master will increase coverage by +0.12% as of 7518d82

@@            master    #902   diff @@
======================================
  Files          271     271
  Stmts        12706   12719    +13
  Branches      1634    1636     +2
  Methods          0       0
======================================
+ Hit           6217    6238    +21
- Partial        468     472     +4
+ Missed        6021    6009    -12

Review entire Coverage Diff as of 7518d82

Powered by Codecov. Updated on successful CI builds.

👍


[a11y] Set aria-label explicitly

This PR would supersede #1590.
Currently the aria-label of the date picker input field is being set with the value of the placeholder. This seems to be almost always the wrong choice, as an aria-label is totally different from a placeholder. This PR makes it so you can explicitly set the aria-label of the input fields and the placeholder is no longer used for that attribute.

Coverage remained the same at 84.486% when pulling a969e8e977b02b5b721f4ec47cc51fc72a5e09cd on ryanmurakami:explicit-arialabel into 32b5f3dc4ccbd5484fe03b0d2b253c11089c700b on airbnb:master.

@backwardok @ljharb Thanks for the thorough review and good comments!


Return 204 on void return types irregardless of @Produces

Motivation:
Currently for annotated services, the return type of void methods can be 200 if a @Produces annotation is added.
@Get("/")
public void get() {
    // 204
}

@Get("/")
@ProducesJson
public void getJson() {
    // 200
}

We have decided to always return 204 (with the exception of @StatusCode) for void methods.

It may be more confusing for users under what conditions 200/204 is returned.
It is difficult to keep track and return valid values for each Content-Type. For instance, 200 with Content-Type: application/json returns null with the current implementation by default.
However, 200 OK with Content-Type: application/xml won't return a valid xml. (armeria returns an empty response)

Modifications:

Set the default return status code to 204 irregardless of @Produces annotation
Leave a warning log if @Produces is annotated on the method level for a void returning method.

Result:

Clearer behavior of annotated services.

:bow:

@ks-yim Forgot to tell you about this change. 😓
We have decided to return 204 regardless of produces annotation. 😉

@minwoox Thanks for the ping. Sounds reasonable to me 😄.
When I was writing https://github.com/line/armeria/pull/4398, Armeria was returning null (4-bytes string) for Mono<Void> return type combined with @ProducesJson annotation and I thought this is the intentional behavior and should be preserved, though I also doubted this behavior will ever be useful.
Thanks for the fix @jrhee17 @tobias-!
Maybe we should close this issue by now?

@ks-yim The other issue is "enough" fixed for me at least.


Supplementary material is not available

Thanks for your job!
As the title says, however, i cannot download the supplmentary material in your project page anymore.
Could you update it or realease a new link?

Sorry that there were some problems with our website. The website has been updated, and you can download the supplementary material directly from https://github.com/daooshee/BMVC2018website/blob/master/chen_bmvc18_sup.pdf.

Sorry that there were some problems with our website. The website has been updated, and you can download the supplementary material directly from Here.

Yet another question is, your released code doesn't appear to do an end-to-end training, is it missing?

As I can see, it seems that you train the DecomNet and the RelightNet separately.

Honestly, I am so curious because I want to re-implement your work in PyTorch, while the result I've got is not good enough.
It looks like this:


Is there a simple way to predict the relation of a single sentence as an input ?

Currently this is not possible.


trainning error on SSD-512

1.
When I try to train a model with input size 512, loss always be non/inf , I had tried to reduce the warmup.factor to 0.1 or change the learning rate to 0.00001, both of them seem not work.
It just work well when I  use smaller batch size ( <= 8)
I'm confused about that. Batch Size should decide by GPU-memory ( I run the program on 8*Tesla v100  32G memory, I think it is enougt for training ), why it cause such an error in loss function,
2.
I  think batch size should be related to the iteration step,  but the iteration is independent to batch size, so the small batch size will lead to shorter trainning time
Do you have any suggestions about the 2 questions ?
Thks a lot

1.
Maybe you can try SOLVER.WARMUP_FACTOR 0.01 SOLVER.WARMUP_ITERS 2000, see #51. When using 4 or 8 GPUs a longer warmup steps may help.
2.
Yes, batch size is related to the iteration step. For 512 input size, I use batch size 24 due to memory limit, which may cause performance drop. If you use V100, you can try batch size 32.


Upgrade to RAA v0.4.0

I wanted to move everything related to canReorder into my regexp-ast-analysis library for a long time now, and I finally got around to doing it.
This PR removes the code I added around canReorder and uses the RAA functions instead.
I also put some work into choosing a cleaner abstraction for getLongestPrefix as well as improving the function itself. As a result of these improvements, I found a false negative in our tests. /(?:js|jso?n?)$/ was listed as a valid regex for no-dupe-disjunctions even though the js alternative can be removed.
While looking through the no-dupe-disjunctions tests, I also found that the yml alternatives in both /(?:yml|ya?ml)$/ and /(?:yml|ya?ml)/ can be removed. I'm currently investigating this.

I also found that the yml alternatives in both /(?:yml|ya?ml)$/ and /(?:yml|ya?ml)/ can be removed. I'm currently investigating this.

I made a minor improvement to canReorder and it is not able to reorder /(?:yml|ya?ml)$/. I'll make a new RAA release tomorrow or the day after.
About /(?:yml|ya?ml)/: I have an idea that could work (RunDevelopment/regexp-ast-analysis#2), but I'll have to think about this some more.

I implemented my idea and make a new release (v0.4.1). RAA is now able to prove that /(?:yml|ya?ml)/ can be reordered. I adjusted the tests of no-dupe-disjunction accordingly.


Implementation of Minari callback for play

Implements #202
Still need to check:

Do reward, terminated, truncated etc. have to have a singleton dimension as in tutorials
How to handle non-numpy observations, actions
Are non-flat observations/actions handled correctly by Minari?
Can we save intermediate results to disk if the replay buffer grows too large?

"Can we save intermediate results to disk if the replay buffer grows too large?"
This is a feature I'd like to add to Minari - appending to MinariDatasets in batches. I don't believe this was supported in d3rlpy (which Minari is based off), so it would need to be added.
One potential issue with a replay buffer not fitting into memory is now it's more difficult to sample from earlier transitions (in case you'd want to do some wide-range off-policy learning)

Would it be possible that if we wrap the environment with the minari dataset collector then call play on it, this should record all of the actions, observations, etc from the environment?

@pseudo-rnd-thoughts that should be the case. We'll have to specify an event to call minari.create_dataset_from_collector_env() so that the data can be stored in a permanent location. This can be done with a custom callback in play and also adding the environment as one of the arguments.

@rodrigodelazcano Could we not make that call before or after calling play. My point is more that I don't think it is necessary to modify play to saving the data.

We are going to make this change in minari


Thanks for the quick fix!

I was literally just racking my brain about this ESM bug with remark-frontmatter and was writing an issue for it when you fixed it in https://github.com/kentcdodds/mdx-bundler/commit/f6bebc9e4763a91d5ec042ed669c5c726f455d7a. Working great now! 🎉
Thanks @kentcdodds

Yeah, sorry about that 😅 And you're welcome!


add metadata load time log

Changes proposed in this pull request:

add metadata load time log

Pull Request Test Coverage Report for Build 10116

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 59.033%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
1
0%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

Totals

Change from base Build 1021:
-0.02%

Covered Lines:
12113

Relevant Lines:
20519

💛  - Coveralls

Any different with #4295?


improvement skip cooling down tablets without cold data

Proposed changes
#29690
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Translate Conditional Rendering

Dear Maintainers, i propose PR in Translating Conditional Rendering. Please review this work and give me feedback. Thank you for giving me chance to join this project.

Hello there, @andriawan. Please check the discussion and commit suggestions, since others cannot make changes to your branch unless you allow edits from maintainers. Thanks in advance. 😄

Sorry for pending response for the feedback. Anyway, thanks for giving me the feedback. i will propose commit with suggestions. Could you give me clue in allowing edits my branch @grikomsn ?

Hello there, @andriawan. Please check the discussion and commit suggestions, since others cannot make changes to your branch unless you allow edits from maintainers. Thanks in advance. 😄

i think it has already done...

Yet somehow I cannot apply commit suggestions. Looks like you should apply revisions yourself for now. 😅

@resir014, any ideas about this?

Dear maintainers, could I have some feedback? @grikomsn

does it need any further improvement? @grikomsn


extend test-dbaLastBackup with -MaxTransferSize -BufferCount

I am using the function test-dbaLastBackup to test my backups 24/7 on a test server. And without these changes the server is slow.
These changes allow me to use -maxtransfersize 4128768 -buffercount = 100 settings for the restore action, and by this gain 50% in restore speed.
the parameters should only be for passtrough, and for testing of the parameters sanity the underlaying Restore-DbaDatabase is responsible

Are you asking for those parameters to be added to the command?

Actually I just wanted to ask if it would be a good idea...
But English is not my native language, so i think i forgot to ask.


Merge Release/2.7.0 on master

Codecov Report

Merging #1946 (324553d) into master (9d7e3f9) will decrease coverage by 0.20%.
Report is 21 commits behind head on master.
The diff coverage is 68.65%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1946      +/-   ##
==========================================
- Coverage   83.56%   83.36%   -0.20%
==========================================
  Files         481      484       +3
  Lines       17413    17555     +142
  Branches     2590     2596       +6
==========================================
+ Hits        14550    14634      +84
- Misses       2177     2232      +55
- Partials      686      689       +3

Files
Coverage Δ

...droid/core/configuration/BackPressureMitigation.kt
100.00% <100.00%> (ø)

...android/core/configuration/BackPressureStrategy.kt
100.00% <100.00%> (ø)

...atadog/android/core/configuration/Configuration.kt
97.00% <100.00%> (+0.30%)
:arrow_up:

.../datadog/android/core/internal/thread/ThreadExt.kt
60.47% <ø> (ø)

...rum/src/main/kotlin/com/datadog/android/rum/Rum.kt
91.84% <100.00%> (+0.35%)
:arrow_up:

.../main/kotlin/com/datadog/android/rum/RumMonitor.kt
28.57% <ø> (ø)

...lin/com/datadog/android/rum/internal/RumFeature.kt
93.29% <100.00%> (-0.91%)
:arrow_down:

...g/android/rum/internal/domain/scope/RumRawEvent.kt
100.00% <100.00%> (ø)

.../android/rum/internal/domain/scope/RumViewScope.kt
94.05% <100.00%> (+0.10%)
:arrow_up:

.../android/rum/internal/monitor/DatadogRumMonitor.kt
86.94% <100.00%> (+1.69%)
:arrow_up:

... and 10 more

... and 13 files with indirect coverage changes


Refactor CLI to support plugin overrides from wrapper

Changes proposed in this PR

Add an optional plugins argument to main. This will be the new preferred way to configure site-wide plugins rather than using the config file.
Refactor modules/plugins loading logic to first try the plugins map passed into main.
Add an environment variable for disabling the config-plugin integration tests in environments where we want to use only the plugins parameter.

Why are we making these changes?
Making site-wide changes to plugged-in logic is difficult/impossible if various users have old plugin overrides copied in .cs.json files in home and working directories. The new plugins argument to main allows creating a wrapper that overrides old behavior everywhere and takes precedence over any legacy config-file settings. Users still wanting to use the config file can invoke the unwrapped main function directly (i.e., circumvent the site-specific wrapper with the default values).

@dposada recommended that we just drop the current configuration-based plugins support altogether to simplify the code and minimize tech debt. Reworking that now.


Ignore library/file in Quantum

Hi,
I have a wasm library in my project. It's size almost 8mb. And Quantum crashes on the build with NodeJS memory exceptions.
Can I somehow ignore this library in the Quantum (it optimized already)?
Thanks.

I don't think you can ignore it, what you can is creating a separate bundle without running it through Quantum, and then lazy load it
Another option, of course is to increase nodejs memory limit (you can find it on Google)

Can I use the same fuse instance, or I should pack it into other, and then use packed JS?

@ypopovych what if you ignored the module completely? #1314

@ypopovych you can use the same instance, just create a different bundle

Can someone provide an example? Can't understand how to disable Quantum for the bundle. I can disable it only to all bundles in Fuse instance (by removing it from plugins).

Tried to import it as shim, but Quantum runs uglify on it :( And it crashes

@ypopovych you can't disable Quantum on one package. It breaks consistency. Allocate more memory
node --max-old-space-size=8192 server.js

Fixed for now by simply copying of this file with Sparky task, and importing it dynamically.


istioctl/cmd/remove-from-mesh:optimize the code

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[x] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@tanjunchen: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
0d6aaec32c350f35279ad17dc150789e96556e1a
link
/test release-notes_istio

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

@tanjunchen: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
0d6aaec32c350f35279ad17dc150789e96556e1a
link
/test release-notes_istio

lint_istio
0d6aaec32c350f35279ad17dc150789e96556e1a
link
/test lint_istio

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

@tanjunchen: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
0d6aaec32c350f35279ad17dc150789e96556e1a
link
/test lint_istio

release-notes_istio
f9714007d3a9047598299d1dcba5547d19ad5147
link
/test release-notes_istio

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

@esnible  Thanks for your review. There is no major problem in the function of the current code, but I think it is more versatile to not pass the array [here](There is no major problem in the function of the current code, but I think it is more versatile to not pass the array here.).

@tanjunchen: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-helm-tests_istio
f9714007d3a9047598299d1dcba5547d19ad5147
link
/test integ-helm-tests_istio

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

@tanjunchen: PR needs rebase.

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


fix!: Don't allow supplyWithOffset to go negative

What is the purpose of the change

Add a description of the overall background and high-level changes that this PR introduces

This PR prevents SupplyWithOffset to not go negative.
Not checking this is currently a bug, because in many places, the SupplyWithOffset (an sdk.Int) is cast to an sdk.Coin, which does not allow negative values.
Brief Changelog
(for example:)

Errors if SupplyOffset is trying to be set that makes Supply - SupplyOffset is negative.

Testing and Verifying
This change added unit tests.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Replaced by #295


Get rid of RawContent in favor of SourceLines

The puzzle 153-feff2f68 from #153 has to be resolved:
https://github.com/nergal-perm/java-decita/blob/456847e57b2548876e43034d54c4c0afdb265f44/src/main/java/ru/ewc/decisions/input/RawContent.java#L41-L41
The puzzle was created by Евгений Терехов on 10-Sep-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 153-feff2f68 has disappeared from the source code, that's why I closed this issue.


Fix existing unit tests and create new ones

Increase project coverage to more than 87%
42 unit test suites were created.
89 unit tests were developed.
Organize the folder structure for unit testing.

Before:

After:

@pierrotsmnrd The needed dependencies are added to the package.json. With that in mind, you just have to run yarn install to install them.

@jujogi I let you solve the conflict and then we can merge this PR.


Vulkan: Fix crash when opening a ShaderMaterial with code saved as an external .shader file

Fix for crashing vulkan on opening shader materials with code saves as external shader file.
Issue comes from a unsynced call of "shader_get_param_list" inside the rendering_server_default.h
Fixes #46053

Hey ho, there is a reviewer for it?

Thanks!

Welcome


add summer dog

🐕

Thanks

Thanks!🙂
Il giorno sab 19 mag 2018 alle 14:08 Indrashish Ghosh <
<EMAIL_ADDRESS>ha scritto:

Merged #413 https://github.com/ghosh/uiGradients/pull/413.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ghosh/uiGradients/pull/413#event-1635527581, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AWUaJyzBb0RLO3g_CO6ZOBU8YayQh51iks5t0AtWgaJpZM4TqsBA
.


In place restart hotfix

Fixes registry read error
Fixes infinite recursion when attempting to set a registry value

Coverage remained the same at 59.225% when pulling 0574fd2881a4a456ddc307c23354a39b0d32dfdd on in-place-restart-hotfix into 9f76862805246ea005eaa7dbe7abf89d0c85062b on master.


[FSDE-13059] Add function to open print dialog with the data table content only

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-342/1/. :white_check_mark:

You can check this PRs instance in https://96dc24224981fc3fcda009bdfe8e95b7e97d9802-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-342-ux-chi.kubeodc-test.corp.intranet/ (internal)

You can check this PRs instance in https://ba63a701c384f68feefa1438aca479145b73f052-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-342-ux-chi.kubeodc-test.corp.intranet/ (internal)

@josecarlosgonzalezv Awesome! Great work!

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-342/3/. :white_check_mark:

You can check this PRs instance in https://4b7aef75b97c5d18c33dabf8edfdd240dc8ac544-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-342-ux-chi.kubeodc-test.corp.intranet/ (internal)


DO NOT MERGE - gff is not a source file, but gff is

/!\ Do not merge

self merging, it'll break stuff momentarily, but I'm working on it


Logging: add an option to disable warning messages.

It would be nice for bpftrace to provide an option which hides warning messages.
This patch proposes using -w to diable warnings, just like in gcc.
Related to #1267 and #1411 .
Checklist

[ ] Language changes are updated in docs/reference_guide.md
[x] User-visible and non-trivial changes updated in CHANGELOG.md

Just realized that it relies on #1443 to actually take effects.

Ohhh, neat! LGTM!
Let's please add a runtime tests so we make sure the CLI (e.g. accidentally removing the argument parsing for this flag) won't regress!

Update: add runtime test

How about using log levels to control outputs? I guess many other logging libraries do that. (e.g., python's logging)

How about using log levels to control outputs? I guess many other logging libraries do that. (e.g., python's logging)

That would look nice. But I think having an option to disable warnings for now is good. We can introduce log levels when we have the granularity. Currently, most loggings are ERROR and the rest are WARNINGS.

Update: change the option from -w to --no-warnings


fixed wrong path for thumbnail request in field-image.tag, fixes #869

fixes #869

Wait with this one. I'm not sure, if I tested it enough.

After reading my changes, I don't understand, what I did there anymore. And I forgot, that image links can be full external paths, too.


Is it possible to filter a query?

Hi! I Is there a way to apply filter to a query?
I am requesting a quite big database and I am not getting back all the records present in the db, response code 200. I am not sure there is a limit to the size of the json. I have tried with another client and I am getting the same issue.
I just need few records from that database, maybe with a filter I can solve the problem.
On QueryDatabaseRequest I see there is a constructor with filter and sorting in the signature, not sure how to use them or if it has been implemented at all.
Thank you for your help.

Hey @enrico-laboratory, checking the following code examples and documents would be helpful to you!

https://github.com/seratch/notion-sdk-jvm/blob/v0.2.0/core/src/test/kotlin/integration_tests/DatabasesTest.kt#L131-L155
https://developers.notion.com/reference/post-database-query#compound-filters

For general questions about the Notion platform, you can ask their developer community too: https://notiondevs.slack.com/join/shared_invite/zt-vkinpzs0-hJSEAAxJzPMuiCNvZETF2A#/shared-invite/email

Indeed what I was looking for. Thank you!
(I should have checked the tests myself)
Thank you again for your great work!


add article for websocket

An article on WebSocket implementation using ws module in node.js.
I thought there should be an article for WebSocket implementation using ws.
Please let me know if any improvements needed.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@DarkByt31 Very well written and comprehensive article! Well done!!!
Congrats on your first-time contribution!


Set io_priority in all IO paths

Summary
There are a number of codepaths performing IO operations that do not set an io_priority flag, so they do not properly get labeled or prioritized by IOQ and default to other io_class [1]. Many moons ago I made a PR [2] that went through and properly tagged anywhere I could find making IO requests without an io_priority set. It looks like the main gaps around indexing and indexer compaction were fixed in [3], however there are still a number of places lacking an io_priority value.
I also think it would be worthwhile to introduce at least one, perhaps two, additional IO classes. I think we should add a system io_priority class for things like auth cache, loading mem3 shards, global changes, couch_per_user, etc. I think it might also be worthwhile to add a dedicated replication io_priority, although it kind of looks like the replicator no longer does local db replications? In [2], the replicator still allowed for doing a direct couch_changes fetch of the local database, so perhaps my original concern around the replicator doing direct changes reads of full databases going unflagged with io_priority is no longer relevant. We should probably still tag the replicator doc updates with either system or replication though.
Desired Behaviour
To minimize (or eliminate) the use of the fallback other io_priority class so that all IO is properly flagged with the appropriate IO class.
Possible Solution
The PR in [2] has a number of simple additions of where to put the io_priority process dictionary value when missing. I tracked those down by modifying the code to throw an exception when absent and then iterated through all the locations I could find. I found many, but the approach was not fully exhaustive so I don't think we should crash by default when there's no io_priority set.
Additional context
[1] https://github.com/apache/couchdb/blob/main/src/ioq/src/ioq.erl#L84-L85
[2] https://github.com/apache/couchdb/pull/1998/files
[3] https://github.com/apache/couchdb/commit/13bf0eb80813477bf3fbe79cffaf0faddffaaca9

@chewbranca good idea to have better priority classes!
Replicator db access from the replicator could be a system db access or a have its own replicator class. But I can see the user accessing db (user creating a replication doc) staying as an interactive request?

the replicator still allowed for doing a direct couch_changes fetch of the local database

I think that's just a left-over clause we didn't clean up when we removed the local access.

Replicator db access from the replicator could be a system db access or a have its own replicator class. But I can see the user accessing db (user creating a replication doc) staying as an interactive request?

Yeah I think system for the replicator db access would work fine.
I do like the idea of being able to prioritize replicator traffic at a different priority level than interactive, I think that would be a useful addition allowing for replication traffic to be deprioritized a bit relative to normal interactive client requests. This would be a bigger change than just properly setting io_priority values in places where it's currently absent, as I believe for that to work we would need to do something like extrapolate from the http request user agent that it's Couch-Replicator and then propagate the replication io_priority from the coordinator node to all the rpc nodes.

I think that's just a left-over clause we didn't clean up when we removed the local access.

Makes sense 👍

I've got two PRs out for this, one in the main CouchDB repo https://github.com/apache/couchdb/pull/4106 and a corresponding PR in the IOQ repo: https://github.com/apache/couchdb-ioq/pull/19

Can this be closed?


adding command line

adding command line to use iso20022's document files
adding testing

adding command line to use iso20022's document files
adding testing


Deep sorting of keys extractor result

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
After extracting keys by webpack plugin only first level keys are sorted in JSON file:
JSON:
{
    "login": {
        "password": "password",
        "sign in": "sign in"
        },
    "user": {
        "profile": "profile",
        "image": "image"
        }
}

Expected behavior
Deep sorting of second level keys - 'profile' and 'image' shoud be sorted as well as 'login' and 'user'
HTML:
<ng-container *transloco="let t; read: 'user'">
    {{ t('image') }}
    {{ t('profile') }}
</ng-container>

JSON:
{
    "login": {
        "password": "password",
        "sign in": "sign in"
        },
    "user": {
        "image": "image",
        "profile": "profile"
        }
}

Minimal reproduction of the problem with instructions
Starting app with --extra-webpack-config webpack-dev.config.js, then keys extractor generates keys which are sorted only on first level, nested keys are not sorted. This behavior causes merging problems, because nested keys are generated every time in different order.
Environment

Angular version: 8.2.3

For Tooling issues:
- Node version: v12.16.3
- Platform:  Windows 10

Others:
webpack-dev.config.js:
module.exports = {
  plugins: [new TranslocoExtractKeysWebpackPlugin({
    defaultValue: "{{key}}",
    langs: ["en"],
    unflat: true,
	sort: true
  })]
};

@bpiasecki Seems to work as expected with both your example and I added some keys in random order and they were sorted alphabetically.
Can you add a reproduction repo so I can further investigate?

@shaharkazaz Did you set 'unflat' option to true in webpack config file? Because i think that it might be a problem. Everything is sorted when this option is set to false.

@bpiasecki yes I did, if you don't use it there is no depth 😄
I tried several times and it seemed to work, try to create a small reproduction so I can check your use case.

@shaharkazaz I updated keys-manager version and now everything works fine, thanks for help 😄


Add SBE processing functions to the file.sbe module

The actual SBE algorithms are really well documented in the Seasoft V2:
SBE Data Processing Software Manual, Section 6: Data Processing Modules.
None of these algorithms are frighteningly complex, so I see no reason why we shouldn't code them up in kval and get rid of the woes with batch processing in windows. Doing this stuff stepwise in kval would give us much more control over what is going on.
Not something we have to do immediately, but it should definitely be a goal to do it eventually.
Maybe work on translating the functions here and there (the bin averaging is already implemented).
Note: Before coding anything up, I should check that SBE haven't already done it in the Sea-Bird Scientific (SBS) Community Toolkit. Could consider making that a dependency..

Had a quick look at the SBS Toolkit. Looks like a lot of these functions are there (/sbs/process/processing.py):
A list of available functions is found in the docstring:

    butterworth_filter (np.ndarray, float, float) -> np.ndarray
    low_pass_filter (np.ndarray, float, float) -> np.ndarray
    align_ctd (np.ndarray, float, float, float) -> np.ndarray
    cell_thermal_mass (np.ndarray, np.ndarray, float, float, float) -> np.ndarray
    loop_edit_pressure (np.ndarray, float, np.ndarray, float, MinVelocityType, float, float, float, bool, float, float, bool, bool,float) -> np.ndarray
    loop_edit_depth (np.ndarray, np.ndarray, float, MinVelocityType, float, float, float, bool, float, float, bool, bool,float) -> np.ndarray
    find_depth_peaks (np.ndarray, np.ndarray, bool, float, float, float) -> tuple[int, int]
    min_velocity_mask (np.ndarray, float, float, int, int, bool) -> np.ndarray
    mean_speed_percent_mask (np.ndarray, float, float, float, int, int, bool, int) -> np.ndarray
    flag_by_minima_maxima (np.ndarray, np.ndarray, int, int, float)
    bin_average (pd.DataFrame, str, float, bool, int, int, bool, bool)
    wild_edit (np.ndarray, np.ndarray, float, float, int, float, bool,float) -> np.ndarray
    flag_data (np.ndarray, np.ndarray, float, float, float, bool,float) -> np.ndarray
    window_filter (np.ndarray, np.ndarray, WindowFilterType, int, float, int, float, bool, float) -> np.ndarray
    calc_N2_buoyancy_modern (np.ndarray, np.ndarray, np.ndarray, float)
    calc_N2_buoyancy_retro (np.ndarray, np.ndarray, np.ndarray, float)
    buoyancy (np.ndarray, np.ndarray, np.ndarray, np.ndarray, np.ndarray, float, bool,float)
    Density (np.ndarray, np.ndarray, np.ndarray) -> np.ndarray
    PoTemp (np.ndarray, np.ndarray, np.ndarray, np.ndarray) -> np.ndarray
    ATG (np.ndarray, np.ndarray, np.ndarray) -> np.ndarray

So: It looks like what is missing are the oxygen functions; I should be able to code those up. Should also wrap this around xarray in kval format one way or another.
A question here is whether to grab source code from SBS or just make sbs a requirement of the kval library.  The package is under a MIT license, so there should be no real issue copying stuff.

@thinf


Semantic UI themeing doesn't work

The custom theming instructions are a little weak.
I ran:
ember install semantic-ui-ember
ember generate semantic-ui-ember.
Then I made modifications to my theme.config file in the semantic directory and did a gulp build, but the css in the app appears to be the same as the one from node_modules.
What do I need to do to make it use the newly built ones?

same for me

I just worked through this myself!
Two things:

It puts things in the default base directory of app_root/semantic and then will dynamically attempt to import them from the dist directory inside of them.

ember's build system disallows you from importing vendor files that aren't part of the vendor or node_modules directory.

So make the following changes to semantic.json and run gulp build
{
  "base": "semantic/",
  "paths": {
    "source": {
      "config": "src/theme.config",
      "definitions": "src/definitions/",
      "site": "src/site/",
      "themes": "src/themes/"
    },
    "output": {
      "packaged": "../vendor/semantic/dist/",
      "uncompressed": "../vendor/semantic/dist/components/",
      "compressed": "../vendor/semantic/dist/components/",
      "themes": "../vendor/semantic/dist/themes/"
    },
    "clean": "vendor/semantic/dist/"
  },
  "permission": false,
  "autoInstall": true,
  "rtl": false,
  "version": "2.2.14"
}

This will output your built CSS to vendor/semantic/dist instead of semantic/dist.
Then you need to add to ember-cli-build.js
'use strict';

const EmberApp = require('ember-cli/lib/broccoli/ember-app');

module.exports = function(defaults) {
  let app = new EmberApp(defaults, {
    SemanticUI: {
      source: {
        css: 'vendor/semantic/dist',
        javascript: 'vendor/semantic/dist',
        images: 'vendor/semantic/dist/themes/default/assets/images',
        fonts: 'vendor/semantic/dist/themes/default/assets/fonts'
      },
    }
  });
  return app.toTree();
};

This will tell ember where to look for the assets when building vendor.js and vendor.css

There is a logic mistake in the addon's JavaScript that causes it to never read the options passed in by the consuming app. See PR https://github.com/Semantic-Org/Semantic-UI-Ember/pull/242 with a fix.
@DVG I was not able to get your example working. Was there anything else you had to do to make this work? Thanks!


Test for Node 0.8 support

Allow node 0.8 to be used, and tested through Travis.
This is to allow mocha to adopt this package, see mochajs/mocha#1545
@sindresorhus you okay with extending support to 0.8?

@jbnicolai, I think this is more of an npm version error, but maybe I'm wrong.

@dasilvacontin am I wrong in my understanding this stacktrace is not due to node 0.8 not being supported by supports-color?
npm ERR! Error: No compatible version found<EMAIL_ADDRESS>npm ERR! Valid install targets:
npm ERR! ["0.2.0","1.0.0","1.1.0","1.2.0"]
npm ERR!     at installTargetsError (/home/travis/.nvm/v0.8.28/lib/node_modules/npm/lib/cache.js:719:10)
npm ERR!     at /home/travis/.nvm/v0.8.28/lib/node_modules/npm/lib/cache.js:641:10
npm ERR!     at RegClient.get_ (/home/travis/.nvm/v0.8.28/lib/node_modules/npm/node_modules/npm-registry-client/lib/get.js:101:14)
npm ERR!     at RegClient.<anonymous> (/home/travis/.nvm/v0.8.28/lib/node_modules/npm/node_modules/npm-registry-client/lib/get.js:37:12)
npm ERR!     at fs.readFile (fs.js:181:14)
npm ERR!     at Object.oncomplete (fs.js:302:15)
npm ERR! If you need help, you may report this log at:
npm ERR!     <http://github.com/isaacs/npm/issues>
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>

I think it's npm's change of behaviour for the ^ char some versions ago.

Sure, since Mocha needs it.
But Mocha should seriously consider dropping support for 0.8.
Build fixed in https://github.com/sindresorhus/supports-color/commit/4f4c583d6bca1af987804fb8c37e194ab4a46882

Thanks @sindresorhus! I'll talk with the team about dropping support for 0.8. Any other reason besides the release is over one year old?

Node 0.8 was released on June 25, 2012. That's 2 years 8 months. The few people still needing 0.8 support can just stay on the current Mocha version.


atomic: fix win32 atomic load const warnings

The load location was not const-qualified because _InterlockedCompareExchange64 could issue a store at that location, even though the stored value is unchanged. That is, you cannot perform a load from a read-only memory.

I see, but I think we should follow the C11 definition, which defines const:
https://en.cppreference.com/w/c/atomic/atomic_load
atomic_load( const volatile A* obj );

Maybe better to drop the windows x86 __int64 atomic load workaround (with a compiler warning or something similar).

It's for you to decide, of course, but I would rather keep the support for 64-bit atomics. IMHO, having those consistently across targets is more useful than being able to do atomic loads from read-only memory.
You can keep the const, of course, but I would add a note somewhere that this const does not mean support for read-only memory. It only means the operation doesn't change the value.


Update evil_attributes

evil_attributes remove javascript event handler but also remove data-on attributes for example: data-section: 5. This update remove the same attributes of w3schools: http://www.w3schools.com/jsref/dom_obj_event.asp

Neither 'data-section' nor 'data-onanything' is being removed. I don't know where you're getting this from ...


Issue983 npint

Main changes:
New strategy for int/ float type checks
# Useful scalar types
_NINT = (np.int32, np.int64)
_INT = (int,) + _NINT
_NFLOAT = (np.float32, np.float64)
_FLOAT = (float,) + _NFLOAT
_NUMB = _INT + _FLOAT

And always run isinstance()
Because the behaviour of np.int, np.int_ or np.integer is too dependent on the version of numpy and on the plateform
Also shows different results depending is using type() or dtype or isinstance()

isintance() allows multiple types and subclasses
Better to use tuple of all explicit numpy types that resort to generic types

Issues:
Fixes, in devel, issue #983

Hello @Didou09! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tofu/imas2tofu/_core.py:

Line 376:80: E501 line too long (83 > 79 characters)
Line 1500:71: E231 missing whitespace after ','
Line 1500:80: E501 line too long (83 > 79 characters)

In the file tofu/imas2tofu/_mat2ids2calc.py:

Line 208:16: W292 no newline at end of file


Add Android bindings

I've added a React Native Android module, all tests pass.
The module relies on my own fork of leveldb-android.
The initial idea was to use android-leveldb by litl but I've found that it lacked some functionalities required by the abstract-leveldown interface.
I've found another similar project: leveldb-android by hf. It was more complete but it still lacked support for the errorIfMissing configuration parameter.
I forked the library and added the parameter (actually there was an umerged pull request sitting around in the original project with the required feature, I copied the code over to the fork).
Resolves #1

Before merging, we should probably update instructions in the README about how to use this library.
Here are the instructions:

if you are cloning this project either clone with git clone --recurse-submodules git@github.com:andymatuschak/react-native-leveldown.git or, after cloning, run git submodule update --init --recursive

in android/settings.gradle, add at the end of the file:
include ':react-native-leveldown'
project(':react-native-leveldown').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-leveldown/android')
apply from: '../node_modules/react-native-leveldown/android/settings.gradle'
project(':leveldb-android').projectDir = new File(rootDir, '../node_modules/react-native-leveldown/android/modules/leveldb-
android/leveldb/')

You can use testapp/android/settings.gradle as a reference.

in android/app/build.gradle add:
dependencies {

    // ...

    implementation project(':react-native-leveldown')
}

in MainApplication.java add packages.add(new LeveldownPackage()); to the getPackages() method, like this:
import com.reactnativeleveldown.LeveldownPackage;

// ...

        @Override
        protected List<ReactPackage> getPackages() {
          @SuppressWarnings("UnnecessaryLocalVariable")
          List<ReactPackage> packages = new PackageList(this).getPackages();
          // Packages that cannot be autolinked yet can be added manually here, for example:
          packages.add(new LeveldownPackage());
          return packages;
        }

// ...

add a react-native.config.js file in the root of your project, add:
module.exports = {
  dependencies: {
    'react-native-leveldown': {
      platforms: {
        android: null, // disable Android platform, other platforms will still autolink if provided
      },
    },
  },
};

The testapp project serves as reference for the instructions.
The first instruction is required since I've added the leveldb-android dependency as a git submodule. If it is a problem, I can push the dependency in some repository and include it from there.

Hooray! I just ran the Android test suite, and everything looks great.
I'll merge this, then update the Readme, add the Android paths to package.json so it's included in NPM, then publish 1.0.
Thank you so much, @RAN3000!

Done: d07acd1. We're now at 1.0! Thanks again—looking forward to getting Orbit up and running on Android…


feat(storefront): dark mode support

resolves #2660
resolves #2770

Looks to work fine. I wouldn't mind an auto option but nothing dealbreaking for now.

yeah we can make an issue for this. I decided to omit this since we don't store the user choice anywhere


Show name for named router outles

For a named router outlet display its name in the Component Tree.
Picture below has two router outlets, the one at the bottom is named, would be good to display the name inside round brackets like we do for DOM nodes.

@tyllrb I think this one needs to go after #647, which addresses how or IF we should do this. The router outlet name is available in the properties tab when you select the component, so the user can see it. At any rate, this feature, if it goes ahead, will actually be a generic feature applying to potentially any element type and any piece of data within it's instance providers.


startup command opens a new window for the same command that was stopped and started again

startup command opens a new window for the same command that was stopped and started again,  the correct behavior is to write over the same pane by name...

This issue happend when :
1- start server
2- close jsng
3- open jsng
4- stop server
5- start server again

Fixed: https://github.com/threefoldtech/js-ng/commit/d382aadb764663e220470988805b468d4e9fe89f

verified


:Clap files :: Fuzzy search :: Matches only from left to right

Behaviour
Using fuzzy searching patterns upon initiating :Clap files allows me to match from left to right only. Using the same keywords in a different order, right to left, does not return any results.
Question
Is there a configuration setting that influences this behaviour ? I would like to specify any exact keyword regardless order and still get a match.
Example :: Left to right

Example :: Right to left

Custom settings
    " Option :: Plugin :: vim-clap
    autocmd User ClapOnEnter call ClapPreviewToggle()
    let g:ClapPrompt = function('ClapFormat')
    let g:clap_disable_bottom_top = 1
    let g:clap_layout = { 'width': '88%', 'height': '40%', 'row': '25%', 'col': '8%' }
    let g:clap_on_move_delay = 50
    let g:clap_popup_input_delay = 50
    let g:clap_provider_grep_executable = 'rg'
    let g:clap_provider_grep_opts = '--glob !.git/ --hidden --no-heading --smart-case --vimgrep'
    let g:clap_selected_sign = { 'text': 'ﱣ ', 'texthl': "ClapSelectedSign", "linehl": "ClapSelected" }
    let g:clap_theme = 'material_design_dark'

That works as expected for the substring algorithm used in clap https://github.com/liuchengxu/vim-clap/blob/master/pythonx/clap/fuzzymatch-rs/src/lib.rs#L16. I'm happy to accept PR that implements the order-insenitive substring filter, but I won't do it myself as the current one works great for me.

Hi @liuchengxu, thank you for the prompt reply. As i'm not a developer I wouldn't be able to help you with a PR. I'm happy to test or contribute in another way however if this feature does get more attention in the future. Cheers !


Update Developer Guide's Target User Profile

Convert Developer Guide (DG) from .docx to .md
Update Target User Profile to match content of current project iteration
Remove irrelevant section of the DG

Codecov Report

Merging #23 (c6bc672) into master (4eb741f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #23   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 4eb741f...c6bc672. Read the comment docs.


AttributeError: module 'numpy' has no attribute 'float'

I encounter the following Trackback when using ros_numpy:
Traceback (most recent call last):
  ...
  File "/home/kzheng/repo/robotdev/spot/ros_ws/src/genmos_object_search_ros/src/genmos_ros/ros_utils.py", line 13, in <module>
    import ros_numpy
  File "/opt/ros/noetic/lib/python3/dist-packages/ros_numpy/__init__.py", line 7, in <module>
    import ros_numpy
  File "/opt/ros/noetic/lib/python3/dist-packages/ros_numpy/__init__.py", line 7, in <module>
    from . import point_cloud2
  File "/opt/ros/noetic/lib/python3/dist-packages/ros_numpy/point_cloud2.py", line 224, in <module>
    from . import point_cloud2
  File "/opt/ros/noetic/lib/python3/dist-packages/ros_numpy/point_cloud2.py", line 224, in <module>
    def get_xyz_points(cloud_array, remove_nans=True, dtype=np.float):
  File "/home/kzheng/repo/robotdev/spot/venv/spot/lib/python3.8/site-packages/numpy/__init__.py", line 305, in __getattr__
    def get_xyz_points(cloud_array, remove_nans=True, dtype=np.float):
  File "/home/kzheng/repo/robotdev/spot/venv/spot/lib/python3.8/site-packages/numpy/__init__.py", line 305, in __getattr__
    raise AttributeError(__former_attrs__[attr])
AttributeError: module 'numpy' has no attribute 'float'.
`np.float` was a deprecated alias for the builtin `float`. To avoid this error in existing code, use `float` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.float64` here.
The aliases was originally deprecated in NumPy 1.20; for more details and guidance see the original release note at:
    https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
    raise AttributeError(__former_attrs__[attr])

According to this Stackoverflow, np.float causes an error with Numpy 1.24+. (I have 1.24.2 installed).
Is this package still being maintained to address this issue?

I have the same problem, both packages were installed through the ubuntu repositories

@eric-wieser -- any chance of getting one last release to the ROS debs pushing the fix to this? (#39)

Hey, I am currently overloaded with stuff, but I promise to he to it asap!

Eric, please give me the appropriate repository rights in order to add a secret (for a github action that releases the package)

It's not clear to me that I can do that without moving this to an organization repo. Can you either point me to instructions on how to generate the secret so I can generate add it myself, or email it to me at $last.$first @ gmail.com so that I can add the one you generated?

Click your profile picture -> Settings -> Developer Settings -> Personal Access Tokens -> Tokens (classic) -> Generate  new token -> Generate new token (classic) ->

Expiration 30/08/2025
(~3 months after the Noetic EOL, just to be sure we get the last sync, if needed)
Select scopes:
[x] workflow
(it should also automatically check the whole repo scope)

Copy the generated token

Go to the ros_numpy repo -> Settings -> Secrets and variables -> Actions -> New repository secret ->

Name
BLOOM_GITHUB_TOKEN
(important to be exactly like this, since I won'y be able to check)
Secret
Paste the previously generated token
That's all! If you need any more help, hit me up!

Thanks @gstavrinos! I'm not comfortable making a classic-style personal access token, as this grants permission to all of my repositories on github.
Are the fine-grained tokens an option? Can you share the script that will consume this token?

Hey, Eric. That's a totally valid concern. This is why I asked you to directly make it a secret, so that no one (not even you!) has access to it afterwards. I think a fine-grained token with access to this repo and its release one, would also suffice though. (I haven't tested it)
The action I am going to use is similar to the one I use here:
https://github.com/gstavrinos/odom_to_tf_ros2/blob/master/.github/workflows/bloom-release.yaml


New address encoding

This PR adds the new ark address encoding, replacing bare user public key by vtxo taproot key. It implies change on the APIs and the SDK.

Address encoding
An ark address is network prefix + VTXO taproot pubkey + ASP sweep pubkey encoded with bech32m.
ark1x0lm8hhr2wc6n6lyemtyh9rz8rg2ftpkfun46aca56kjg3ws0tsztfpuanaquxc6faedvjk3tax0575y6perapg3e95654pk8r4fjecs5fyd2

network prefix identifies testnet, mainnet, regtest...
VTXO taproot pubkey is the P2TR key that should be used as scriptpubkey in VTXO leaf output of the tree.
ASP sweep pubkey allows verifying the ASP of the address

API & server changes

remove descriptor from RegisterOutputsForNextRound RPC, only ark address is expected.
expect descriptor to be revealed while spending VTXO = RegisterInputsForNextRound RPC.
[db] VTXO table: drop descriptor field, replace by address field.

@altafan @tiero please review

[db] VTXO table: drop descriptor field, replace by address field.

@louisinger let's replace the descriptor with the taproot pubkey instead of the address:

we don't want to "depend" on network as address contains a network prefix
we don't want to persist the same server pubkey for all records

hello, @louisinger
Bech32 offers a robust and error-resistant format for encoding new address types in Go, leveraging its strong checksum and simplicity, making it an ideal choice for handling public key combinations.
`
package main
import (
"encoding/hex"
"fmt"
"github.com/btcsuite/btcutil/bech32"
)
// EncodeAddress encodes the new Ark address using Bech32.
func EncodeAddress(networkPrefix, vtxoPubkey, aspPubkey string) (string, error) {
// Combine the public keys into a single byte slice
combinedPubkey := vtxoPubkey + aspPubkey
pubkeyBytes, err := hex.DecodeString(combinedPubkey)
if err != nil {
return "", err
}
// Encode using Bech32
encodedAddress, err := bech32.Encode(networkPrefix, pubkeyBytes)
if err != nil {
    return "", err
}

return encodedAddress, nil

}
func main() {
networkPrefix := "ark1"
vtxoPubkey := "your_vtxo_pubkey_here"
aspPubkey := "your_asp_pubkey_here"
newAddress, err := EncodeAddress(networkPrefix, vtxoPubkey, aspPubkey)
if err != nil {
    fmt.Println("Error encoding address:", err)
    return
}

fmt.Println("Encoded Address:", newAddress)

}

package main
import (
"encoding/json"
"net/http"
)
func registerAddressHandler(w http.ResponseWriter, r *http.Request) {
var data struct {
NetworkPrefix string json:"network_prefix"
VtxoPubkey    string json:"vtxo_pubkey"
AspPubkey     string json:"asp_pubkey"
}
if err := json.NewDecoder(r.Body).Decode(&data); err != nil {
    http.Error(w, "Invalid request", http.StatusBadRequest)
    return
}

newAddress, err := EncodeAddress(data.NetworkPrefix, data.VtxoPubkey, data.AspPubkey)
if err != nil {
    http.Error(w, "Error encoding address", http.StatusInternalServerError)
    return
}

json.NewEncoder(w).Encode(map[string]string{"address": newAddress})

}

-- SQL command to update the VTXO table schema
ALTER TABLE vtxo_table
DROP COLUMN descriptor,  -- Remove the old descriptor field
ADD COLUMN pubkey VARCHAR(255);  -- Add a new pubkey field to store the VTXO public key
`


FindHasMany is limited to 25 (default) hits

When using ember-pouch (or relational-pouch), when following a hasMany link, it doesn't set an unlimited limit, so I only get the first 25 children
For instance when listing all managers, that have a managerFor relation to technicians (inverse via manager) results in a _find where manager = the id. However, since no limit is included, I only ever get the 25 first technicians for the manager.
The code in question is: https://github.com/pouchdb-community/relational-pouch/blob/master/lib/index.js#L425-L438

I am not aware there is a limit. Will investigate. On https://bloggr.exmer.com/ it will return more than 25 Posts. See https://bloggr.exmer.com/authors/D8F61FBC-A9C4-0318-851B-228507CDC032 after this record is loaded. https://github.com/broerse/ember-cli-blog/blob/master/app/models/author.js

Hmm... I wonder if this is only when running against CouchDB: http://docs.couchdb.org/en/2.1.0/api/database/find.html?highlight=limit#db-find
Perhaps PouchDB does not have a default limit?
My scenario is that I use Ember-Pouch running against a PouchDB syncing to CouchDB on the iPad, and in the browser running straight against the same CouchDB.

We run everything against Cloudant or CouchDB 1.6 and it does not have this limit. I see this limit in your CouchDB 2.1.0 doc but it does not state how to disable it. See also https://stackoverflow.com/questions/45632794/how-to-fetch-unlimited-documents-using-mango-query-couchdb-without-providing-li

@nolanlawson Is this fixed/addressed in the newest pouch. Can we now remove the dependency on pouch-find for relational-pouch?

@afinne Cloudant has this limit but we never saw it because if you use find on the local pouch it does not have this limit. So it is only the backend that has this limit. We stil need to fix this somewhere. I think this module is perhaps the correct place to fix this and not in pouchdb-find.

I've had this exact same issue using ember-pouch with CouchDB, and went through hours of exploration of messing with adapters and scouring through documentation to eventually come to a simple solution.
As mentioned above, CouchDB 2.1.0 added the default limit
By searching through the apache/couchdb source code I found there is an undocumented mango section in the CouchDB configuration: https://github.com/apache/couchdb/blob/master/rel/overlay/etc/default.ini#L280-L285
Therefore if you don't want a limit, you can just set a high limit under mango.default_limit through Fauxton or by modifying your config file.

@rhysdavies1994 I did't know that. Not sure this is what we shout advise but until we fix this for findHasMany it is a good workaround.

For a bit of clarification on @rhysdavies1994’s workaround: you can enlarge the find default in Fauxton by:

clicking on the gear on the left menu (Config), which takes you to http://<IP_ADDRESS>:5984/_utils/#_config/

choosing “Add Option” at the top

Filling in mango for “section,” default_limit for “name” and something large (50000) for “value.” On reload, it should look something like:

@muziejus solution working like a charm. Here's another pic:

(just put a bigger number in limit)


Show ascii-QR code of login url.

In addition to showing the URL to login in, also show the same URL as a (ascii-)QR code.
See eg https://github.com/fumiyas/qrc

https://github.com/fukuchi/libqrencode

of deze https://github.com/ricmoo/QRCode

https://github.com/SURFscz/SBS/commit/fcb0549061cc93ad1d4e3e4be4f645cac43d541c

Correct


Not working with newer toolchains

Starting with rustc version 1.52.0 till present, cargo_screeps panics with the following message
thread 'main' panicked at 'unknown name section chunk type: 7', /home/.cargo/registry/src/github.com/cargo-web-0.6.26/src/wasm_context.rs:642:41

@daboross are there any plans to fix this and https://github.com/rustyscreeps/cargo-screeps/issues/30 to make it work for newer rust versions, or is this project dead?

I'm not currently working on this project. If you're interested (or if anyone else is) in using this, I could definitely provide pointers or see how hard it would be to get it working on the latest version of rust, though! I think the starting point would be finishing PR - https://github.com/rustyscreeps/cargo-screeps/pull/24, since our dependency on cargo-web (which has been deprecated and is no longer updated) is the main thing broken here.
There are also a few other maintainers who I haven't been in contact with who may or may not be still using the project. I haven't been in contact with them for quite a while, but assuming everyone I knew switched over when the Screeps Slack community moved over to Discord, the best place to check in and contact them would be #rust in the Screeps Discord (https://discord.com/invite/RjSS5fQuFx).
I'm probably not going to start maintaining it actively, but I could definitely try to fix the build system and review PRs.

Resolved by #24


update architecture diagram

Update the application architecture diagram.

The text in app-arch.rst (line 14) also needs to be changed slightly as it explains the diagram in depth.

Good point! Thank you :) I'll look at the text.


[Chronicles of Darkness] Hide firefox spinner for derived inputs

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
Fixes #10707
Root Cause
The inputs are correctly populated, but the value is not seen because it is covered by the Firefox number input spinners (the up and down arrows for number inputs).
Fix
Hide the Firefox number input spinners for these "derived" inputs.
(Chromium spinners are already hidden with different css rules.)

Character Sheet Info Roll20 Internal Use only.


replace snake_case func names with camelCase in store

Part of #377

(test failed because node ran slow and broke a fragile test)


Handle empty packagemanifests

Fixes #82
Signed-off-by: Melvin Hillsman<EMAIL_ADDRESS>

/lgtm


Type information is lost during method resolution

From @slide on December 21, 2015 12:56
This was found when trying to fix #435
>python2
>>> import datetime
>>> class mydate(datetime.datetime): pass
...
>>> mydate.now()
mydate(2015, 12, 21, 5, 53, 2, 797939)
>>>

>ipy
>>> import datetime
>>> class mydate(datetime.datetime): pass
...
>>> mydate.now()
datetime.datetime(2015, 12, 21, 5, 53, 2, 797939)
>>>

This occurs because the type information is lost by the time you get to the repr method in datetime.cs. 'this" is seen as a datetime object, and nothing seems to allow you to get back to the mydate name.
Copied from original issue: IronLanguages/main#1267

From @rassilon on January 6, 2016 18:20
Isn't that what the PythonType properties from the return value from this code are for?
  DynamicHelpers.GetPythonTypeFromType(GetType())

cPickle.FindModuleForGlobal when given PythonType uses PythonType.Get__module__ or PythonOps.TryGetBoundAttr(context, obj, "__module__",...)
I would hope the PythonType case would be the most common occurrence when that function is called for pickling out object instances.

GetType() returns datetime, because that is what the method is resolved to, a member of datetime.

From @rassilon on January 6, 2016 22:11
Oh, I understand better I think. mydate.now() ends up calling the datetime.now() static method. Does PythonContext contain the current __class__? i.e. mydate

No, I don't think it does


Wire: fix setClock

clock source is fixed for RA4M1, so the rates will be wrong for C33.
This will be fixed in the next commit

@facchinm
Have a question on the settings that you are using for 400khz.  If I attach a Logic analyzer to SDA/SCL while running a sketch pulling data from a MPU-9250 I am seeing:

which I believe is showing a frequency of about 257khz not close to the 400khz.
However if I change your settings to:
          m_i2c_extend.clock_settings.brl_value = 12;
          m_i2c_extend.clock_settings.brh_value = 11;
          m_i2c_extend.clock_settings.cks_value = 0;

I get closer to 400khz (357khz):

@facchinm
These settings work better - gives 400khz with 60% duty cycle
        case I2C_MASTER_RATE_FAST:
          m_i2c_extend.clock_settings.brl_value = 16;
          m_i2c_extend.clock_settings.brh_value = 15;
          m_i2c_extend.clock_settings.cks_value = 0;

setClock should not only allow the three values, 100kHz, 400kHz, 1Mhz.  See  this issue.

@mjs513 @greiman I pushed a new version of the patch which should take care of the clocks for both UNO and C33.
In this patch I explicitly disabled FAST_MODE_PLUS for platforms not supporting it (eg. RA4M1) so we are in the specifications of the chip.
Let me know if it works for you, then we can merge it safely.

@facchinm
Got some strange things happening when testing the changes at 400khz.
Using a 5v liquidCrystal_i2c device on Wire with your current settings of 16/15 I am seeing 500khz using a LA:

If I change the settings for 400khz to 22/21 I am seeing 400khz, again for 5v device on Wire:

But if I use those settings 22/21 for a MPU-9250 running at 3.3v on Wire I am seeing something like 257khz

Using your settings, again on wire for a MPU-9250 I am getting 294khz:

Confused!!!!!!
And if I move the 9250 over to Wire1 using your settings of 16/15 I am seeing that 526khz again

and if I use the 22/21 settings I am seeing 416khz at 52% duty cycle.
So now I am confused.  Please note I do have the Micros propose PR incorporated as well as a couple of others.

@facchinm
out of curiosity I pulled out a T4.1 (had it handy) and check 100khz and 400khz.  At 100khz seeing 68khz with 50% while at 400khz I am seeing 357khz clock with 50% duty cycle for my MPU-9250.
Never one to leave enough alone:
Adafruit Metro M0 express: 97khz with 50% duty at 100Khz; 357khz with 50% duty at 400khz
Uno R3 Clone:  370khz with 52% duty at 400khz;  100khz with 49% duty at 100khz setclock.

Wire1 has level shifters and works correctly for 3V3 devices.  Wire is marginal for 3V3 devices.  For 3V3 devices on Wire, the speed depends on pull-up resistance, pull-up voltage, and VCC for the RA4M1.  See this.

@facchinm -  just wanted to know if there was anything else you wanted me to try with this?

@facchinm - maybe its a good idea to go ahead and incorporate the current fix (this pr) into the core.  Seems to be resolve some issues people are having: https://forum.arduino.cc/t/i2c-setclock-does-not-appear-to-be-standard-way-to-setclock/1147122/19

As @facchinm is currently out-of-office I'll move ahead and merge this one.


ng generate component - TypeError: Cannot read property '0' of undefined

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

@angular/cli: 1.0.0-rc.4
node: 6.9.5
os: win32 x64
@angular/common: 2.4.7
@angular/compiler: 2.4.7
@angular/compiler-cli: 2.4.7
@angular/core: 2.4.7
@angular/forms: 2.4.7
@angular/http: 2.4.7
@angular/material: 2.0.0-beta.2
@angular/platform-browser: 2.4.7
@angular/platform-browser-dynamic: 2.4.7
@angular/platform-server: 2.4.7
@angular/router: 3.4.7
@angular/cli: 1.0.0-rc.4
Windows 10
Repro steps.

This project was started with the seed project, not the cli. I was trying to just add a component via the cli.

cd to directory i wanted to create the component in
run ng generate component usersettings --flat --dry-run

The log given by the failure.

installing component
You specified the dry-run flag, so no changes will be written.
Cannot read property '0' of undefined
TypeError: Cannot read property '0' of undefined
at Object.getAppFromConfig (C:\source\lg-mbc\src\MBC.LG.Web\node_modules@angular\cli\utilities\app-utils.js:6:19)
at Class.normalizeEntityName<EMAIL_ADDRESS>s:124:39)
at Class.Blueprint._normalizeEntityName (C:\source\lg-mbc\src\MBC.LG.Web\node_modules@angular\cli\ember-cli\lib\mod
els\blueprint.js:454:24)
at Class.Blueprint.install (C:\source\lg-mbc\src\MBC.LG.Web\node_modules@angular\cli\ember-cli\lib\models\blueprint
.js:513:8)
at Class.run (C:\source\lg-mbc\src\MBC.LG.Web\node_modules@angular\cli\ember-cli\lib\tasks\generate-from-blueprint.
js:59:49)
at Class.run (C:\source\lg-mbc\src\MBC.LG.Web\node_modules@angular\cli\ember-cli\lib\commands\generate.js:71:17)
at Class. (C:\source\lg-mbc\src\MBC.LG.Web\node_modules@angular\cli\ember-cli\lib\models\command.js:134:
17)
at process._tickCallback (internal/process/next_tick.js:103:7)
Desired functionality.

generate component without errors
Mention any other details that might be useful.

I have no idea what the seed project is. CLI projects really need to be generated by the CLI to work.

Seed project is referenced in the docs.
https://github.com/angular/quickstart
The cli didn't exist when the project was started.
So the cli can't be used just to stub out a generic empty controller?

It won't let me edit my previous comment....
I meant component not controller. :)

You can probably use only the generators as long as you have a .angular-cli.json with the right configuration for your project. Otherwise there's no way for the CLI to know what's your app folder and such.
Using the CLI on non-CLI projects isn't something we test for or explicitly support though.

hello guys I believe  this is due to the feature  module
just specify  with  ng g c module_name/components_name
if you have multiple modules

I just stared having this problem using ng g component newfilename
worked one day now the next it stopped.
Error: Cannot read property '0' of null Cannot read property '0' of null

@Sevensnake Same here. Just this morning. Was working last week.

One more. This is a project that is only a few days old, with no old versions that I'm aware of. I deleted a bunch of unused components (5), and now when I try to add a new one, I get:
Cannot read property '0' of null
This was create with CLI, and still runs fine with ng serve.
These are the current versions:
Angular CLI: 6.0.8
Node: 8.9.4
OS: linux x64
Angular: 6.0.5
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.6.8
@angular-devkit/build-angular     0.6.8
@angular-devkit/build-optimizer   0.6.8
@angular-devkit/core              0.6.8
@angular-devkit/schematics        0.6.8
@angular/cdk                      6.2.1
@angular/cli                      6.0.8
@angular/material                 6.2.1
@ngtools/webpack                  6.0.8
@schematics/angular               0.6.8
@schematics/update                0.6.8
rxjs                              6.2.1
typescript                        2.7.2
webpack                           4.8.3

This creates the error:
git clone https://github.com/gknight4/postops-angular
cd postops-angular
npm i
ng serve (to verify that it runs OK)
ng g component newone
result is:
Cannot read property '0' of null

Same problem here, What can we do?

Is there no solution for this yet?  I'm getting the same issue.


ID3D12Resource obtained via IDXGISwapChain3::GetBuffer has wrong ref count

When RenderDoc v0.33 is loaded in D3D12 I get wrong refcount for swap chain ID3D12Resource.
Issue can be reproduced with latest bgfx (this check doesn't trigger without RenderDoc):
https://github.com/bkaradzic/bgfx/blob/master/src/renderer_d3d12.cpp#L1749

The refcount values aren't wrong, they're just different to the D3D12 runtime as it ships today - which could even be different from the D3D12 runtime in a year's time. The return values from AddRef and Release are for debugging only, they're not part of the interface contract and you shouldn't rely on their values being something specific.
I can change the behaviour to be similar to D3D11, where the internal reference on the swapchain image is hidden from the user so it will hit 0 when you release your last reference. That still fails your assert though because it seems to expect the refcount to be somehow shared between all the images.

It's useful for debugging resource leaks. I have mechanism to disable this functionality if RenderDoc is present, but I thought since only swap chain is wrong that might be easy fix.

With that change it should hit 0 instead of 1 when you've released all your references. That's about as good as I can make it, there's no way it's worthwhile to link up the separate refcounts to track each other.
I realise it's useful to have the return values for debugging, but I wouldn't suggest making it a fatal error that's always on. Like I say the refcount values could easily change in a future D3D runtime to be tracked individually, and then the code doesn't know which refcount to expect.

Closing this issue as the new behaviour is available in v0.34.


Imports from index.ts resolve as undefined

TypeScript Version:  2.0.3
VSCode Version:  1.6.0
Code
import { Rectangle } from '../../viewees/shapes';

Expected behavior:
Rectangle to be defined
Actual behavior:
Rectangle is undefined

My code based is full of imports using index.ts.
In the shapes folder index.ts is:
export { Rectangle } from './Rectangle';
export { Shape     } from './Shape';

I've just upgraded to typescript 2.0.3 - no problems (yet I had to remove quite a few .ts from imports).
But now when creating a new file and using:
import { Rectangle } from '../../viewees/shapes';

Rectangle resolves to undefined, whereas
import { Rectangle } from '../../viewees/shapes/Rectangle';

Resolves correctly (to a function/class).
This SO question may be related.

It seems that only Rectangle has this issue - all other index.ts seem to work fine. Very odd.

OK. So the problem was:

shapes/index.ts:
export { Rectangle } from './Rectangle';
export { Shape     } from './Shape';

shapes/Rectangle.ts:
import { Shape   } from './';

Which seems to mess things up (circular dependency?)
Changing shapes/Rectangle.ts to:
import { Shape   } from './Shape';

Solved the issue and now
import { Rectangle } from '../../viewees/shapes';

Works just fine.


How to run the file?

I need help on running it

What file do you mean? You can download the source and compile it on your computer. But the bot is work in progress.

Whenever I compile it it's looking for .exe file

Hmm i don't know what the problem could be. But make sure you compile the "src" and "Supreme-Bot-Plugins/src" folder separately, because the "src" folder is the source for the bot and the other is a test for plugins that I may finish in the future.

I have added a compiled version of the Bot, so you can run it directly. But i don't know if it's working, because maybe some extra files are missing/not created yet. But you can try it anyway.