Protobuf error

I'm trying to make node-csgo compatible with node-steam>1.0. If steam installed separately everything works, but if it installed in node-csgo node_modules directory error occuring:
/home/gittest/node-csgo/node_modules/protobufjs/dist/ProtoBuf.js:2949
    clone[i] = cloneRaw(value, binaryAsBase64, longsAsStrings, field.type, fie
                                                                    ^
TypeError: Cannot read property 'type' of undefined
    at cloneRaw (/home/gittest/node-csgo/node_modules/protobufjs/dist/ProtoBuf.js:2949:97)
    at cloneRaw (/home/gittest/node-csgo/node_modules/protobufjs/dist/ProtoBuf.js:2949:44)
    at MessagePrototype.toRaw (/home/gittest/node-csgo/node_modules/protobufjs/dist/ProtoBuf.js:2964:28)
    at Object.Steam._processProto (/home/gittest/node-csgo/node_modules/steam/lib/steam_client.js:21:17)
    at SteamClient._netMsgReceived (/home/gittest/node-csgo/node_modules/steam/lib/steam_client.js:151:26)
    at SteamClient.handlers.(anonymous function) (/home/gittest/node-csgo/node_modules/steam/lib/steam_client.js:270:10)
    at SteamClient._netMsgReceived (/home/gittest/node-csgo/node_modules/steam/lib/steam_client.js:168:26)
    at Connection.emit (events.js:107:17)
    at Connection._readPacket (/home/gittest/node-csgo/node_modules/steam/lib/connection.js:50:8)
    at Connection.emit (events.js:104:17)

I tried removing steam folder, switch directory to /home/gittest/, run npm install node-steam and then run example.js, it works. Also I've tried to run command npm install protobufjs in /home/gittest/node-csgo/node_modules/steam/ and it also worked nice.

Also
npm install steam
npm install

works too.

Looks like dcodeIO/ProtoBuf.js#310. Why are you installing node-steam into the node-csgo directory though?

I'm not installing it directly, just running npm install. Dependencies:
dependencies": {
  	"steam": "1.1.0",
	"protobufjs": "*",
    "deferred": "*",
    "bignumber.js": "^2.0.7"
  }

Does node-csgo depend on node-steam just for the example? You shouldn't do that.

node-csgo depends node-steam for steam methods. Like logon. Also dependencies likes this works.
"dependencies": {
	"steam": "1.1.0",
    "protobufjs": "*",
    "deferred": "*",
    "bignumber.js": "^2.0.7",
	"bytebuffer": ">=3.5.5"
  }

node-csgo doesn't call any node-steam methods anywhere outside of example.js. If you want to specify that node-csgo is meant to be used with node-steam, then add it to peerDependencies.
Any dependency configuration will work as long as ByteBuffer used by node-steam is physically the same module as ByteBuffer used by ProtoBuf.js used by node-steam. Otherwise ProtoBuf.js doesn't recognize the Long object. I assume that's a bug.

Well node csgo using GC from steam
this._gc.send({msg:CSGO.ECSGOCMsg.k_EMsgGCCStrike15_v2_MatchmakingClient2GCHello, proto: {}},
+      payload.toBuffer());

All messages are send through node-steam. Current version of node-csgo have node-steam as peerDependency, but since it will not work without node-steam connection to steam and game coordinator I think it should be dependency.

It uses the GameCoordinator object provided by the user, it doesn't create it itself. It doesn't need steam inside its node_modules, so it shouldn't be a dependency.

Ok, done. Thank you for help.

I'm having this problem from within a project that uses node-steam.

I've linked node-steam in via npm link because I'm working on developing it, though.

@paralin, add
"bytebuffer": ">=3.5.5"

To your dependencies, it should help.

node-steam master and node-steam-resources now both use ByteBuffer.js 4.1, where this problem was fixed.


Add the following games and services:

Weather, using your IP address for location
gemini://tilde.cafe/~stack/weather
or a specific location, for instance:
gemini://tilde.cafe/~stack/weather?nyc
Spellbinding game -- make words with specified letters
gemini://tilde.cafe/~spellbinding/game/cgi?
A wordle-like game
gemini://tilde.cafe/~spellbinding/wordo/cgi?
A public wall - leave a message
gemini://tilde.cafe/~spellbinding/wall/cgi

@stacksmith Please be so kind and open a Pull Request because I do not have the time to transform all the Github Issues to Pull Requests. Thanks!


Add option to pass kernel_* and bias_* regularizers to Coral layer

CORN layer inherits fromDense, so it all usual **kwargs can be passed.
Coral layer inherits from Layer, so kernel_regualizer/bias_regularizer (and other kwargs that would make sense for Coral) dont work as expected (passing them fails init since Layer does not recognized those kwarggs)

Should the Coral layer inherit from Dense instead of Layer?
-ck.mobile

On Mar 2, 2022, at 7:11 PM, Georg @.***> wrote:

CORN layer inherits fromDense, so it all usual **kwargs can be passed.
Coral layer inherits from Layer, so kernel_regualizer/bias_regularizer (and other kwargs that would make sense for Coral) dont work as expected.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.

I don't think that will be as trivial, since the AFAIU the point of the CORAL layer is that it is a specific (constrained) structure of a fully connected dense layer (which is what CORN solves).
Looking at Dense() layer implementation, it should be straightforward to pass kernel_regularizer and bias_regularizer as arguments to the CoralOrdinalLayer init and then pass them to self.add_weight(...) for the kernel and bias respectively.
See https://github.com/keras-team/keras/blob/v2.8.0/keras/layers/core/dense.py#L147
This is mostly to support the same function signagure as a typical Dense layer that has bias/kernel weights.

Belated response but thanks for the explanation on this!


No implicit file packaging in packet.template

It is OK when paket add PackageName.dll to NuGet package when files section is not specified.
But when I write
files
	../../bin/DynamicsCRMProvider ==> lib/net40

i expect that all content of the folder will be packed inside 'lib/net40'. But in fact I get error
Paket failed with:
        An item with the same key has already been added.
Running build failed.
Error:
System.Exception: Error during packing ..
   at Fake.Paket.Pack@66.Invoke(String message) in C:\code\fake\src\app\FakeLib\PaketHelper.fs:line 66
   at Fake.Paket.Pack(FSharpFunc`2 setParams) in C:\code\fake\src\app\FakeLib\PaketHelper.fs:line 66
   at FSI_0001.Build.clo@180-16.Invoke(Unit _arg9) in d:\Personal\GitHub\DynamicsCRMProvider\build.fsx:line 181
   at Fake.TargetHelper.runTarget@329(String targetName) in C:\code\fake\src\app\FakeLib\TargetHelper.fs:line 343

So when I explicitly create section files I want to be sure that paket import only files mentioned there.

/cc @mavnn

@sergey-tihon this is what the file type templates are for. In general either you're going to want the default behaviour (a library nuget package based on the project) or you're doing something non-standard which needs specific files in specific places. At that point you can just create a template file with type file at the top and set everything up as needed.
If you've got a use case where you want to include elements of both, more details would be awesome.

Thanks @mavnn! I did not realize concept of configurable template type before.

But, I think that would be nice to have more granular control over dependencies, files, metadata inference. Like this:
type file
files
    ../../bin/DynamicsCRMProvider ==> lib/net40
dependencies
    auto

So I can specify what sections I want to infer.
@mavnn What do you think?


Update MD file on new branch 2017-19-13_22-11-07

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


New github labels

Levels like beginner, advanced, expert, etc would make it easier to pick the right issues to contribute. This would work well with the status/help-wanted label.

For the most part, issues with status/help-wanted should be more to the beginner side of things.
If you see a status/help-wanted issue, or any issue, that looks interesting, feel free to comment on it and one of the core contributors would be happy to give an assessment of difficulty, and pointers, if you want.
The other thing we suggest for new contributors is to just start reading code and you will probably find something that can be improved within a few hours.


Move operator package from internal to pkg directory

Description:
Integration tests needs APK defined kinds like API ..etc.
For example:

Wait until API is deployed before continue running tests

https://github.com/wso2/apk/pull/720 merged


Getting wrong raster image size from CUPS

I am from Lexmark driver development team. Recently we encountered with an issue regarding wrong raster image size from CUPS. We have an entry in our PPD file like "*cupsFilter: "application/vnd.cups-raster 0 ". In our filter we retrieve the raster image's header information through CUPS's API "CupsRasterReadHeader2()". We have found that the wrong image size of raster image for a Custom paper size input. Not sure for other input. The details is given below.
    Input                                       Raster Image size getting                             Expected Image size
 =======                                =====================                        ===================
110x220mm (Envelop DL)                    312x624                                                312x624    (Correct)
105x210mm                                           312x624                                                298x596   (Wrong)
105x209mm                                          298x592                                                298x592.  (Correct)

The problem we are getting from different applications like Document Viewer and Firefox as well as Command line printing. The command we have used is lp -d <queue name> -o PageSize=Custom.<page size> -o fit-to-page <file to print>
Need help urgently. Please look into it.

@skapas We really need to see the PPD file in question.

Also, a CUPS error_log file with debug logging (cupsctl --debug-logging) showing the jobs in question will tell us what is happening in the raster filters.

error_log.txt
Lexmark_ppd.txt
Hi Michael,
I have attached the log and ppd file.
Regards
Saptarshi

OK, could you try this on a newer version of CUPS? This looks like a media matching bug, but I know we made changes for CUPS 2.4 to allow for media sizes with such a small difference in size.

We have installed the CUPS version 2.4.2 by compiling the source code of CUPS in local machine. We have also checked the issue was not resolved. Attaching the CUPS error_log file.
error_log.txt

OK, it would be helpful if your filter logged the raster header it was getting. Nothing inside CUPS is going to be changing the media size, and I double-checked that the current comparison/matching code uses an epsilon of 0.5mm. So my guess is a Ghostscript issue, but I need to do some more digging...

@michaelrsweet have you got time to investigate Ghostscript issue?

@skapas no, have you had a chance to add some logging of the raster header you are getting?

@michaelrsweet in our last attached error_log.txt log already contains all the details which we get from the CupsRasterReadHeader2.

@skapas The log data you quoted in your comment shows the DL envelope size being used, with the correct raster dimensions.
Please provide separate log file attachments showing a working and non-working job - maybe then we can make some progress here but at the moment it doesn't look like anything in CUPS is broken, but it might be one of the filters that is shipped separate from CUPS...

No response from the reporter over one month, I'm going to close the issue. In case the issue persists, please let us know and provide the requested data.


LSM: Pacing, Pipelining & Performance

Turn back, traveler. Here be :dragon:s.
(Full write-up, rebase and production ready version coming soon™)

Some high level asserts I'd love to see:

to make sure we don't alias pooled blocks, it would help if each CompactionBlock stored its current state: which compaction owns the block, and which state of the pipeline owns the block. Then, in pipeline stages callback, asssert thate the owner and the stage are still valid
To make sure that we didn't skip over any work: let compaction maintain separate counters for

values_read_count (incremented by the read stage based on data block size)
values_merged_input / values_merged_output incremented by the merge step by comparing compaction state before/after merge_values (we could also maintain the counter interanlly in merge_values, but that's a hot loop, so I'd rather not burn extra registers there, and I am fairly certain that merge is corret)
values_write_count incrementdeb by the write step based on the size of the data block
at the end of compaction, assert that the values agree with each other, and match the manifest
If input table total value count = total values read by read stage = total values read by the merge stage, I think we guarantee that no water was spilled from the interconnected bathtubs!

no good ideas how to check that we don't have races! I am fairly certain that we can't race on acquiring reservations, but it's not obvious that we can't race on using reservations...

I'd love to iterate on this for a couple of weeks post merge though 🤣

:100: - so many performance improvements possible!


Allow InputStream as Parser input

Currently, the only input type is String. In case the input is read from a stream, this means it has to be read into a String first, and then the parser has another copy of the data as block content.
Accepting an InputStream (or Reader?) instead would allow to get rid of one of the copies. DocumentParser already processes the input line by line, so this should be trivial.

What would be the viability of passing in more focused type, e.g. Iterable<String> or a custom interface such as a LineSupplier?
(And yes, the viability of Iterable for this type of abstraction is debatable, too.)

That would make the common case of parsing a file or other stream of characters harder, because the caller would have to adapt a BufferedReader to an Iterable<String>. Also, the contract of an Iterable is that it can be iterated multiple times, which is not the case for a stream.
After some more thinking, I think we should add an overloaded parse with a Reader argument. If it's already a BufferedReader (e.g. from Files.newBufferedReader), we use it as-is, otherwise we wrap it in one. Then read line by line. The caller is responsible for closing it.
I think a Reader is better suited than InputStream, as we don't want to know about the Charset, the caller is responsible for that (also matches spec).

There may also be situations where Iterable<String> makes sense, but we can add that later if needed.

Thanks for working on this, @sentyaev. It would have been nice if you commented here first though, maybe @armstnp would have liked to implement this.

All's good. :+1:

Implemented with a new method parseReader on Parser, thanks @sentyaev

Released in 0.2.0.

Great! My first open source project commit.

Oh, congratulations! Hopefully there will be many more :).


Add pypi supported python version badge

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


Use folder "jats" instead of "paper.jats"

GitHub action updated: https://github.com/xuanxu/paper-action/pull/5


Add initial Cranelift interpreter

See https://github.com/bytecodealliance/cranelift/pull/1351.

@bnjbvr, I had to commit 1bed1db to get this back in a working state. If the barrier to merging is that only want to expose features that are complete, we could remove the clif-util interpret ... functionality; we have to start somewhere, though, so I recommend we merge this as-is--missing instructions and all--and add a log message to clif-util interpret ... like This is an unstable feature and a work in progress implementation.

@bnjbvr, I think is ready for another review but it would be best if we took a look at #1517 first since this uses some of the changes that PR made to RunCommand.

Sorry, I am swamped in reviews again, and I don't know when I'd be able to do this; forwarding to other people.

Stuff I'm going to make a note to work on later:

a Trap variant for unknown opcodes instead of using unimplemented!, from @fitzgen
convert the block() method to use a loop instead of recursing, from @bnjbvr
think about renaming Frame and Environment, from a comment to @fitzgen


Connections dialog Extra Call Argument menu is missing many types

Godot version
v4.0.alpha.custom_build [7b9fd0da6]
System information
Windows 10
Issue description
The connection dialog is missing many types. Other type selectors, including the array, dictionary, metadata, and project settings editors have all of the types.

Connection Dialog
Array Editor

Steps to reproduce

Select a node
Switch to the Signals tab in the Node dock
Double-click a signal to open the connection dialog
In the bottom-right corner, click Advanced
Click the dropdown button below "Add Extra Call Argument" in the top-right to see an incomplete list of types

Minimal reproduction project
No response

Related to https://github.com/godotengine/godot/pull/32624.


add code to check if a page should be shown in the navigation

Add a check to see if the page class have a method called showInNav, if it has this method, call it so see if the pages should be registered.
It is not a breaking change.

@rabol No code changes needed to achieve this. You can simply overwrite the navigationItems method on the class and return an empty array, [].
public static function navigationItems()
{
    return [];
}


How to work with reference styled hyperlinks?

We adopt the reference styled hyperlinks for certain reasons. I would like to know how mkdocs-badges plugin can be used with such referenced styled links.
See the example below.
|t:More Info|v:See more info|l:more_info|

[more_info]: https://www.example.com

Please provide the working example.
Thanks
Jayesh
P.S. The example does not work.

Hi Jayesh,
I don't currently support reference style links. A workaround may be to just use mkdocs-badges to render the badge and use the badge in a normal markdown link. It would liik something like this:
[
|t:More Info|v:See more info|
][more_info]

[more_info]: https://www.example.com

This looks pretty ugly and the badge is not underlined when it is hovered, but it should work.
However, I plan on doing a few (breaking) changes to the badge syntax to add support for CSS classes and make it more easy to understand. When doing these changes I will think about, if I can easily add reference style links.
BR
six-two

I would highly appreciate the update.
Thanks and best regards,
Jayesh

Reference link badges are now supported in version 0.3.0. See section "Reference link badges" on the badges page. However since the badge formats changed you may need to update existing badges manually. See the migration page.


Sensors Unavailable

Seems my sensors are down, I have one running hourly and one running daily. Been this way for about a week or so. Seen some updates come through which didn't fix the issue. Curious if I should just uninstall/reinstall through HACS?
EDIT: Note that I renamed the entities/sensors

Thanks for noticing this- I'm pretty good at catching things during the day, but overnight (for eastern time) is another issue!
It wasn't just your setup, so don't reinstall anything, and should be back now. The back end API was throwing an error for 6 hours because one of the input files was broken. So easy to diagnose the immediate cause here, root cause is trickier. Taking a look at the CloudWatch logs, and I'll try to address the main issue so it doesn't happen again (most likely due to something with the NOAA model).

Well, root cause is that the file system threw a "permission denied" error, which is bizarre, so I'm going to chalk this one up to cloud gremlins. There's a lot (lot!) of processing that goes on every 6 hours to make the input NetCDF files, so many places for something weird to happen. The code is designed not to erase the previous dataset unless it completes successfully, but apparently something slipped through here, which is an issue!
Thinking about this, I'm going to combine this issue with https://github.com/alexander0042/pirate-weather-ha/issues/4. Once I set things up to keep the past 48 hours worth of data on the filesystem, I'll add another block of error checks in the API retrieval code. This way, if something weird happens during the processing, the API can fall back on the previous forecast, which is slightly less accurate, but still more than adequate.

Thank you as always
For the record when you said "should be back on now", ive given it some time, rebooted HA, refreshed caches on browser and ran the entity update service on it (which to my knowledge is really all i can do to help this along) and still hasnt come back online.

Shoot, that shouldn't be happening, since it's back online for me. My best guess now would be something about the API key, so if you let me know the email address you're using, I can take a look and see if that's the issue?

emailed you

Saw your reply - moving the conversation back here
First of all thank you again - you didnt have to do that!
Secondly, I'm surprised that HA would reach out that many times a day. Is this something that should be limited in the integration so this isn't the case? Ive seen plenty of HACS integrations where you can specify the update interval.
I'm not using the API for anything other than the HA sensor. Even if this ran once every 5 minutes (weather doesn't change that frequently does it?!) it would result in a max of 8640 calls allowing the other 1400 or so to be used by the users without hitting the max.

Don't mention it! I'm thrilled that people are finding this useful, so too many calls is a good problem to have!
Thanks to this report, I realized the default weather card was updating every three minutes, which might have been the issue here (unless there's something overriding it on your config.yaml). I've pushed a new version https://github.com/alexander0042/pirate-weather-ha/commit/d040d53e8bdfa00eae6248a886f678545ad34f42 that changes it to a 15 minute interval, since that's roughly what the underlying models provide, so make much more sense as a default. The integration tries to calculate an dynamic minimum rate depending on the config, which is why this might not have been noticed yet.
Thanks again for your help here, and let me know if anything else crops up!


Support using array segments for serializers

Adjusts the primary serializer interface to return an ArraySegment-wrapped byte array in favor of the byte array itself. It then uses the resulting count and offset fields to be passed into the produce call. The new interface also supports a Release hook which can be used after the produce call has happened.
The primary use case is to allow for pooled arrays to be written to, or recycled streams' internal buffers. This helps a lot when writing data greater than 85K which would ordinary require LOH allocations and greatly impact the performance of the service.
I made the adjustment so there should be no breaking changes - the builder interfaces just "upgrade" the old variant serializers by wrapping them. The producer class internally uses the new interfaces only. The release methods are no-op on the wrapped serializers.
This may provide very similar functionality to https://github.com/confluentinc/confluent-kafka-dotnet/pull/1725 without an API break

Just noting I am not an official collaborator, just reading your issue and PR.
This adds capabilities without modifying any of the existing serializers to take advantage of this capability. I'd love to see how this would change the AvroSerializer or ProtobufSerializer etc to now be more efficient or better.


Fix typo in flag message

Proposed changes
This PR fixes type in flag message.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

Coverage remained the same at 100.0% when pulling fa382e3e1c579779e9f891e467ff3962790ad761 on fisuda:fix/typo into 6cf9e127318ff708ad0d1a67ca268fa004ef51f9 on lets-fiware:main.


bump html5gum to 0.5.7

more performance improvements, also i accidentally pushed a breaking
change in 0.5.7 and want to avoid 0.5.5 to circulate around too much
because of that

ah, https://github.com/lycheeverse/lychee/pull/1184


[SAI] Add support for dash configuration from sai interface

Adding this support will allow syncd to easy pass configuration parameters to dash libsai using service method table

for dash it should be transparent, main use case is for syncd, for example we load profile.ini file which configure many different aspects of libsai, in this case i allow to configure some dash api, like every variable in config.h and saidash.h keys, you can take a look of use case in syncd here: https://github.com/sonic-net/sonic-sairedis/blob/master/syncd/Syncd.cpp#L3165 when we load profile from file, if this config will be supproted in dash then we can modify easy those parameters from syncd pointing to right configuration files

can we have this merged ?

thanks!


Update nodejs version

Codecov Report

Merging #652 (49215eb) into main (b29505f) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 49215eb differs from pull request most recent head 01289c1. Consider uploading reports for the commit 01289c1 to get more accurate results

@@           Coverage Diff           @@
##             main     #652   +/-   ##
=======================================
  Coverage   67.89%   67.89%
=======================================
  Files         114      114
  Lines        6615     6615
  Branches     1164     1164
=======================================
  Hits         4491     4491
  Misses       2124     2124

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 b29505f...01289c1. Read the comment docs.


ServerInformationReceived Event called 3 times in a row

I have created one instance of IrcClient and added the ServerInformationReceived event. Calling the AsyncConnect method first fires my also added ConnectionComplete event and then ServerInformationReceived gets called 3 times in a row. Is this intended or is it a bug? The server ist chat.freenode.net on port 6667.

Can you capture RawMessageReceived and log it somewhere? I'd be interested to see what it shows.

I have logged RawMessageReceived and it says the following using chat.freenode.net 6667:
:adams.freenode.net NOTICE * :*** Looking up your hostname...
:adams.freenode.net NOTICE * :*** Checking Ident
:adams.freenode.net NOTICE * :*** Found your hostname
:adams.freenode.net NOTICE * :*** No Ident response
:adams.freenode.net 001 maxpa1n87 :Welcome to the freenode Internet Relay Chat Network maxpa1n87
:adams.freenode.net 002 maxpa1n87 :Your host is adams.freenode.net[<IP_ADDRESS>/6667], running version ircd-seven-1.1.3
:adams.freenode.net 003 maxpa1n87 :This server was created Mon Jul 20 2015 at 16:58:36 UTC
:adams.freenode.net 004 maxpa1n87 adams.freenode.net ircd-seven-1.1.3 DOQRSZaghilopswz CFILMPQSbcefgijklmnopqrstvz bkloveqjfI
:adams.freenode.net 005 maxpa1n87 CHANTYPES=# EXCEPTS INVEX CHANMODES=eIbq,k,flj,CFLMPQScgimnprstz CHANLIMIT=#:120 PREFIX=(ov)@+ MAXLIST=bqeI:100 MODES=4 NETWORK=freenode KNOCK STATUSMSG=@+ CALLERID=g :are supported by this server
:adams.freenode.net 005 maxpa1n87 CASEMAPPING=rfc1459 CHARSET=ascii NICKLEN=16 CHANNELLEN=50 TOPICLEN=390 ETRACE CPRIVMSG CNOTICE DEAF=D MONITOR=100 FNC TARGMAX=NAMES:1,LIST:1,KICK:1,WHOIS:1,PRIVMSG:4,NOTICE:4,ACCEPT:,MONITOR: :are supported by this server
:adams.freenode.net 005 maxpa1n87 EXTBAN=$,ajrxz WHOX CLIENTVER=3.0 SAFELIST ELIST=CTU :are supported by this server
:adams.freenode.net 251 maxpa1n87 :There are 162 users and 92674 invisible on 26 servers
:adams.freenode.net 252 maxpa1n87 25 :IRC Operators online
:adams.freenode.net 253 maxpa1n87 8 :unknown connection(s)
:adams.freenode.net 254 maxpa1n87 63428 :channels formed
:adams.freenode.net 255 maxpa1n87 :I have 5876 clients and 1 servers
:adams.freenode.net 265 maxpa1n87 5876 5904 :Current local users 5876, max 5904
:adams.freenode.net 266 maxpa1n87 92836 94396 :Current global users 92836, max 94396
:adams.freenode.net 250 maxpa1n87 :Highest connection count: 5905 (5904 clients) (673447 connections received)
:adams.freenode.net 375 maxpa1n87 :- adams.freenode.net Message of the Day -
:adams.freenode.net 372 maxpa1n87 :- Welcome to adams.freenode.net. Thanks to ATW Internet Kft
:adams.freenode.net 372 maxpa1n87 :- (http://www.atw.hu) for sponsoring this server!
:adams.freenode.net 372 maxpa1n87 :-
:adams.freenode.net 372 maxpa1n87 :- ADAMS, DOUGLAS (1952-2001).  Author of The Hitch Hikers Guide
:adams.freenode.net 372 maxpa1n87 :- to the Galaxy and many other witty and humourous books,
:adams.freenode.net 372 maxpa1n87 :- portrayed in his uniquely British irony. He is sorely missed
:adams.freenode.net 372 maxpa1n87 :- by many millions of devoted fans. "So long and thanks for all
:adams.freenode.net 372 maxpa1n87 :- the books!"
:adams.freenode.net 372 maxpa1n87 :-
:adams.freenode.net 372 maxpa1n87 :- Welcome to freenode - supporting the free and open source
:adams.freenode.net 372 maxpa1n87 :- software communities since 1998.
:adams.freenode.net 372 maxpa1n87 :-
:adams.freenode.net 372 maxpa1n87 :- By connecting to freenode you indicate that you have read and
:adams.freenode.net 372 maxpa1n87 :- accept our policies as set out on http://www.freenode.net
:adams.freenode.net 372 maxpa1n87 :- freenode runs an open proxy scanner. Please join #freenode for
:adams.freenode.net 372 maxpa1n87 :- any network-related questions or queries, where a number of
:adams.freenode.net 372 maxpa1n87 :- volunteer staff and helpful users will be happy to assist you.
:adams.freenode.net 372 maxpa1n87 :-
:adams.freenode.net 372 maxpa1n87 :- You can meet us at FOSSCON (http://www.fosscon.org) where we get
:adams.freenode.net 372 maxpa1n87 :- together with like-minded FOSS enthusiasts for talks and
:adams.freenode.net 372 maxpa1n87 :- real-life collaboration.
:adams.freenode.net 372 maxpa1n87 :-
:adams.freenode.net 372 maxpa1n87 :- We would like to thank Private Internet Access
:adams.freenode.net 372 maxpa1n87 :- (https://www.privateinternetaccess.com/) and the other
:adams.freenode.net 372 maxpa1n87 :- organisations that help keep freenode and our other projects
:adams.freenode.net 372 maxpa1n87 :- running for their sustained support.
:adams.freenode.net 372 maxpa1n87 :-
:adams.freenode.net 372 maxpa1n87 :- In particular we would like to thank the sponsor
:adams.freenode.net 372 maxpa1n87 :- of this server, details of which can be found above.
:adams.freenode.net 372 maxpa1n87 :-
:adams.freenode.net 376 maxpa1n87 :End of /MOTD command.
:maxpa1n87 MODE maxpa1n87 :+i
:NickServ!NickServ@services. NOTICE maxpa1n87 :maxpa1n87 is not a registered nickname.
:adams.freenode.net 311 maxpa1n87 maxpa1n87 ~maxpa1n87 ip5f5acc3e.dynamic.kabel-deutschland.de * :Max Hölzer
:adams.freenode.net 312 maxpa1n87 maxpa1n87 adams.freenode.net :Budapest, HU, EU
:adams.freenode.net 378 maxpa1n87 maxpa1n87 :is connecting from<EMAIL_ADDRESS><IP_ADDRESS>
:adams.freenode.net 317 maxpa1n87 maxpa1n87 6<PHONE_NUMBER> :seconds idle, signon time
:adams.freenode.net 318 maxpa1n87 maxpa1n87 :End of /WHOIS list.


Search by id is returning a deleted resource with a client generated id

Describe the bug
When searching for a deleted resource with a client generated id (i.e. added via PUT) using _id, the returned bundle includes the deleted resource in the total count.
To Reproduce

Create a Patient with a client-generated id via HTTP PUT (i.e. AAA)
Delete the Patient
Search for the deleted Patient using _id (issue persists when Cache-Control:  no-cache)
The returned bundle has a total count of 1 with no resources in the entry field

Expected behavior
The returned bundle should have a total value of 0 and no resources in the entry field
Environment

HAPI-FHIR: 6.0.1

Hi,
I am using v6.2.5 but still see that resources marked as deleted are included in the total of the bundle when executing a REST search using the Spring Boot JPA Server implementation, while the resulting bundle does only include the non-deleted ones. Does this fix not fix the issue I perceive?
Kind regards,
Jacob

Try to search using GET with headers Cache-Control: no-cache, in version 7 works.


Update Add-AzureADGroupMember.md

Clarifies that the RefObjectId is the member being added, and that the ObjectId is the group ID being updated.

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

File
Status
Preview URL
Details

azureadps-2.0/AzureAD/Add-AzureADGroupMember.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Hi @aharonhannan
Per the read me file:
Please be aware that if you have any updates to automatically generated reference content, you should create an issue and not a PR.  The issue will then be created into a bug and triaged accordingly.  We do this because the reference content must mirror the *dll-help.xml file and subsequent updates to the reference content may overwrite any changes made through the repo.  Capturing it in the source code ensures that this does not occur.  Any PR with changes to automatically generated reference content will be closed.
Automatically generated reference content is any of the content under this Reference node, such as Get-AzureADUser
This only applies to reference content and not conceptual content.  Pull requests for changes to conceptual content are accepted and encouraged.  An example of the conceptual content is the Overview or a scenario such as importing data.
Thank you!
Bill


Set schemes' build config to debug/release in iOS Xcode export

For the workflow where you export to Xcode project from a non-Mac platform and then switch to Mac to open the project in Xcode and build/archive manually, this just aligns the default build configuration chosen for the different schemes so it matches the debug enabled export option.
When you are using that workflow to get the final .ipa to submit to Apple, without this change you have to set the build config manually to Release to be sure you are getting the actually production-ready variant.
This makes sense because it matches what would happen when exporting from Mac, where the .ipa is built in Debug/Release depending on the aforementioned debug enabled export setting.

Thanks!

Cherry-picked for 3.4.


Add a Clear Method

Any chance we could get a clear method to clear the input and typeahead's results? Using jQuery's .val(''); on the input element clears it, but when you focus typeahead adds the last entered text back in. Having a clear method would be useful to create a clear button (similar to the iOS standard search fields) for small screen touch devices where selecting and clearing inputs is a bit more tedious.

This could help, I resolved this simulating the click on cancel button
$('.typeahead__cancel-button').trigger('mousedown');


Don't assume exit code 8 means missing linter binary

The problem
Currently the exit code 8 when running a linter is always assumed to mean that the linter binary couldn't be found.
These are the related lines (link goes to latest commit as of this writing to ensure correct line numbers):
if exitCode is 8
  # Exit code of node when the file you execute doesn't exist
  return @linterNotFound()

https://github.com/AtomLinter/Linter/blob/d4827d6e5d49c9e8e09fced27afe6dde703f6fad/lib/linter.coffee#L172-174
The exit code 8 seems to be the exit code that Node uses for any kind of crash though – not just crashes because the node-binary couldn't find the file to execute (which I think is what caused this code to be implemented in the first place) but also any other kind of crashes.
I just ran into the category of "other kind of crashes" when I used linter-jshint and the JSHint binary crashed due to what I reported there as: https://github.com/jshint/jshint/issues/2506
It was very confusing to get that crash within JSHint reported as if the JSHint binary could no longer be found. I tried lots of different JSHint paths and it wasn't until I dug into the Linter code and started logging stuff that I found out that it was actually not at all the binary that had gone missing, but rather a bug in the binary that had it start crashing.
The proposed solution
I would propose doing one of the following:

Either always report exit code 8 as a crash rather than a missing binary...
...or detect the error message from the node binary when the node binary can't find the file its looking for and use that to still report those exit code 8 errors as missing binaries and make the rest of exit code 8 errors be reported as crashes.

Ouch

This is also a legacy issue.

Closing as it's a legacy API issue and linter-jshint has already upgraded to the latest one.


NamedTuple doesn't resolve postponed annotations

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.8.1
            pydantic compiled: False
                 install path: /nix/store/1f6nk37c7df54cm8b3rlnn6pidaa3mrp-python3.8-pydantic-1.8.1/lib/python3.8/site-packages/pydantic
               python version: 3.8.8 (default, Feb 19 2021, 11:04:50)  [GCC 9.3.0]
                     platform: Linux-5.4.114-x86_64-with
     optional deps. installed: ['typing-extensions']

Here's the smallest program I could come up with that fails under from __future__ import annotations but works without it:
from __future__ import annotations

from pydantic import BaseModel, PositiveInt
from typing import NamedTuple

class Tup(NamedTuple):
    v: PositiveInt

class Mod(BaseModel):
    t: Tup

print(Mod.parse_obj({"t": [3]}))

The call to parse_obj raises this exception: pydantic.errors.ConfigError: field "v" not yet prepared so type is still a ForwardRef, you might need to call Tup.update_forward_refs(). But there is no update_forward_refs on NamedTuple instances.
Just using a base int in the NamedTuple doesn't exhibit the same problem, but of course it also doesn't check the constraints I want.
My current workaround is to just not use from __future__ import annotations in the module where I declare and use the NamedTuple, but I was hoping to use the new style consistently throughout the project.

Just ran into this as well. The PR has been sitting for a couple months now. Any chance we can get that merged?


Atari roms broken?

I downloaded rom file for ms_pacman from the url mentioned here: https://github.com/ppwwyyxx/tensorpack/blob/130f60ac4491d55a13632eabbad839c2e23138ba/examples/DeepQNetwork/atari.py#L25
This prints errors like "Illegal Instruction! 32" at line 91 same file and does not render anything. But strangely gym.make('MsPacman-v0') works. So I looked at the atari_py repo and saw that the roms have not been updated in 2 years.
TL;DR: the roms on the github repo seem to be broken for a few games. Roms present locally at '/site-packages/atari_py/atari_roms' are working at the moment. You might want to update the debug message to download from the url to something else.
Thanks

Where do the roms in "site-packages/atari_py/atari_roms" come from? I think they came from exactly the same place.

Sorry my bad. They are identical.
I was doing
wget https://github.com/openai/atari-py/blob/master/atari_py/atari_roms/ms_pacman.bin instead of wget https://github.com/openai/atari-py/raw/master/atari_py/atari_roms/ms_pacman.bin
Thanks for the quick response.


python312Packages.aiolifx-themes: 0.4.14 -> 0.4.16

Diff: https://github.com/Djelibeybi/aiolifx-themes/compare/refs/tags/v0.4.14...v0.4.16
Changelog: https://github.com/Djelibeybi/aiolifx-themes/releases/tag/v0.4.16
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

5 packages built:

home-assistant-component-tests.lifx
python311Packages.aiolifx-themes
python311Packages.aiolifx-themes.dist
python312Packages.aiolifx-themes
python312Packages.aiolifx-themes.dist

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

5 packages built:

home-assistant-component-tests.lifx
python311Packages.aiolifx-themes
python311Packages.aiolifx-themes.dist
python312Packages.aiolifx-themes
python312Packages.aiolifx-themes.dist


[assets-controllers] Make Token type json-serializable

The Token type defined in the TokenRatesController has a balanceError property, which is typed as unknown. This property is assigned an error value in the TokensBalancesController updateBalances method.
TokensController uses the Token type in its state, which makes the state object non-json serializable. This is a roadblock for upgrading TokensController to BaseControllerV2 and needs to be fixed.

This does seem to be an issue anymore, as the balanceError property in the Token type seems to have been replaced with a hasBalanceError property, which is now a boolean. In addition, TokensController has been updated to BaseController v2.


function [x, y] = calculate_target_location(distance, bearing, current_location)
    % Convert bearing from degrees to radians
    bearing_rad = deg2rad(bearing);

    % Calculate the x and y coordinates of the target maritime
    x = current_location(1) + distance * cos(bearing_rad);
    y = current_location(2) + distance * sin(bearing_rad);
end

% Example usage
distance = 10; % distance in nautical miles
bearing = 45; % bearing in degrees
current_location = [0, 0]; % current location in (x, y) coordinates

[x, y] = calculate_target_location(distance, bearing, current_location);
disp(['Target maritime location: (', num2str(x), ', ', num2str(y), ')']);

function [x, y] = calculate_target_location(distance, bearing, current_location)
    % Convert bearing from degrees to radians
    bearing_rad = deg2rad(bearing);

    % Calculate the x and y coordinates of the target maritime
    x = current_location(1) + distance * cos(bearing_rad);
    y = current_location(2) + distance * sin(bearing_rad);
end

% Example usage
distance = 10; % distance in nautical miles
bearing = 45; % bearing in degrees
current_location = [0, 0]; % current location in (x, y) coordinates

[x, y] = calculate_target_location(distance, bearing, current_location);
disp(['Target maritime location: (', num2str(x), ', ', num2str(y), ')']);

function [x, y] = calculate_target_location(distance, bearing, current_location)
    % Convert bearing from degrees to radians
    bearing_rad = deg2rad(bearing);

    % Calculate the x and y coordinates of the target maritime
    x = current_location(1) + distance * cos(bearing_rad);
    y = current_location(2) + distance * sin(bearing_rad);
end

% Example usage
distance = 10; % distance in nautical miles
bearing = 45; % bearing in degrees
current_location = [0, 0]; % current location in (x, y) coordinates

[x, y] = calculate_target_location(distance, bearing, current_location);
disp(['Target maritime location: (', num2str(x), ', ', num2str(y), ')']);

import math

def calculate_target_location(distance, bearing, current_location):
    # Convert bearing from degrees to radians
    bearing_rad = math.radians(bearing)

    # Calculate the x and y coordinates of the target maritime
    x = current_location[0] + distance * math.cos(bearing_rad)
    y = current_location[1] + distance * math.sin(bearing_rad)

    return x, y

# Example usage
distance = 10  # distance in nautical miles
bearing = 45  # bearing in degrees
current_location = [0, 0]  # current location in (x, y) coordinates

x, y = calculate_target_location(distance, bearing, current_location)
print(f"Target location: ({x}, {y})")


promisify( pool.getConnection ) throws unexpected error

const
pool = require('mysql').createPool(
{
  connectionLimit : 5,
  host,
  user,
  password
}),
getConnection = promisify(pool.getConnection),
connection    = await getConnection(),

TypeError: Cannot read property 'length' of undefined
    at Pool.getConnection (/opt/provision/node_modules/mysql/lib/Pool.js:36:29)
    at internal/util.js:238:30
    at module.exports.createTransaction (/opt/provision/model/db/gateway/mysql.js:33:33)
    at Noc.createTransaction (/opt/provision/model/db/repository/noc.js:11:32)
    at module.exports (/opt/provision/model/activate-order.js:14:35)
    at module.exports.activate (/opt/provision/controller/activate-order.js:24:13)
    at module.exports.post (/opt/provision/controller/activate-order.js:15:23)
    at module.exports.dispatch (/opt/provision/node_modules/@superhero/core/controller/dispatcher/rest.js:23:42)
    at module.exports.dispatch (/opt/provision/node_modules/@superhero/core/controller/server/http.js:53:49)
    at <anonymous>

Forgot to bind the instance....

Hi, I have the exact same issue, care to elaborate how you resolve it? Thanks!


Show map to :iphone: users

See #65
Mobile users will now see the Google map between upcoming events and past events.

:+1:


Update node-express-feature-flags.mdx

seo / clarity

🚀 Deployed on https://65c2dc713bc1e0162ea97ca8--cozy-fox-0defec.netlify.app


6GB  of memory.

Q1:if I have just  6GB of memory.can i train a good effect?
or I need at least 12GB?
Q2:why don't you load the teacher of Discriminator?
thx.

@chenloveheimei

We don't test setup with 6Gb VRAM.
It's a good question. I don't know, I hadn't so idea when I worked on this project.


index subsection for modular git section

Adds indicators for the presence of --skip-worktree and --assume-unchanged files as discussed in #207 and #362… but this time using PR #391's modular Git section setup! With this, users can easily opt in to displaying the indicators.
This obviates and addresses the feedback to #362.
Like PR #392:

Relies on the modular Git section in PR #391. I picture merging 391 first and then merging in only the second commit in this PR (not completely sure if GitHub will be smart about adjusting that diff or if I'd have to close and open a fresh PR). So logistically a little messy, but I wanted to provide an example of 391's value

Changes: ddadaeb (most sections/git_index.zsh (jump link)

Frozen till decision made on #391.


react-native (using rxjs)

i noticed that the react native fork didnt finish, and i wanted to do one with rxjs anyway, so ill just do it here.
repo is here: https://github.com/pvinis/realworld-react-native-rxjs

Hello we are currently moving all work in progress to Github Discussions.
I'll close this issue by now, you can find instruction to open a discussion there : https://github.com/gothinkster/realworld/discussions/633
Thanks for your contribution !


Add a way to enumerate fonts used in a SVG before rendering

Loading fonts can be a pretty expensive operation, especially on targets like JS/Wasm, where there is no mmap-like feature, and one must load fonts asynchronously before invoking resvg.
To optimise this case, it would be handy if usvg provided a way to get set of fonts used in the SVG document before rendering it to the final form. That way, resvg-js users, as well as native ones, could only load fonts that will be definitely necessary for rendering the document.
I'm not sure how feasible this is given that usvg downcasts text to shapes as part of its processing, but maybe it could preserve text nodes in some intermediate form to give user the chance to do this async loading?

I don't thinks it's possible for multiple reasons.
First, the reason usvg loads all system fonts is because we have to query them. How would you resolve font-family="Arial" font-weight="bold"? We don't know what font file should be loaded.
Second, text-to-path conversion is part of the "parsing" process at the moment and cannot be decoupled. So you cannot parse an SVG, query fonts and then render the SVG to begin with.
I do plan to make it sort of optional/on-demand. But it will be a huge rewrite.

How would you resolve font-family="Arial" font-weight="bold"? We don't know what font file should be loaded.

I mean if resvg provides just that data as a raw list, that would be already helpful, as there are various databases providing the necessary mappings.
As a concrete example, for resvg-js usecase that I mentioned above, I could use a service like Google Fonts that accepts URLs in form of https://fonts.googleapis.com/css?family=(family):(weight) for CSS as well as extended API that returns direct URLs to font files:
{
   "kind": "webfonts#webfont",
   "family": "Anonymous Pro",
   "variants": [
    "regular",
    "italic",
    "700",
    "700italic"
   ],
   "subsets": [
    "greek",
    "greek-ext",
    "cyrillic-ext",
    "latin-ext",
    "latin",
    "cyrillic"
   ],
   "version": "v3",
   "lastModified": "2012-07-25",
   "files": {
    "regular": "http://themes.googleusercontent.com/static/fonts/anonymouspro/v3/Zhfjj_gat3waL4JSju74E-V_5zh5b-_HiooIRUBwn1A.ttf",
    "italic": "http://themes.googleusercontent.com/static/fonts/anonymouspro/v3/q0u6LFHwttnT_69euiDbWKwIsuKDCXG0NQm7BvAgx-c.ttf",
    "700": "http://themes.googleusercontent.com/static/fonts/anonymouspro/v3/WDf5lZYgdmmKhO8E1AQud--Cz_5MeePnXDAcLNWyBME.ttf",
    "700italic": "http://themes.googleusercontent.com/static/fonts/anonymouspro/v3/_fVr_XGln-cetWSUc-JpfA1LL9bfs7wyIp6F8OC9RxA.ttf"
   }
  },
  ...

Or, in Chrome, I could use the new Local Font Access API that allows to access any system-wide fonts:
await (await queryLocalFonts()).find(font => font.family === 'Arial' && font.style === 'Bold').blob()
Blob {size: 980756, type: 'application/octet-stream'}

Second, text-to-path conversion is part of the "parsing" process at the moment and cannot be decoupled.

Yeah, that's the issue I mentioned in my last paragraph, but thought I'd submit a feature request anyway, as I think this could be a valuable optimisation.

As a non-JS usecase, I should probably mention that some other systems have ways of cheaply querying fonts without loading all of them, too.
E.g. on Windows you can look up font filename by its family + weight in the registry, which should be also relatively easy to do from Rust:

I mean if resvg provides just that data as a raw list, that would be already helpful, as there are various databases providing the necessary mappings.

This is sort of planned but would take forever, since it would require a substantial rewrite.

As a non-JS usecase, I should probably mention that some other systems have ways of cheaply querying fonts without loading all of them, too.

They don't. I've tried. That's why resvg loads them manually. Not to mention that implementing and supporting at least 3 OS APIs is far from trivial.
As for the Windows registry - this is not the info I need. You're underestimating how much information about the font SVG needs. Parsing them manually is the only way.
Also, why do you think that fonts loading is slow? fontdb initialization with a hot disk cache takes like 10ms on my mac. Not instantly, but still 10x faster than the rendering step itself.

Also, why do you think that fonts loading is slow?

As I said, for me the primary usecase is resvg-js:

especially on targets like JS/Wasm, where there is no mmap-like feature, and one must load fonts asynchronously before invoking resvg

All the fonts literally need to be either downloaded from the internet, so you're stuck with two choices: either bundle a very limited set of fonts and hope that any given SVG doesn't rely on any advanced fonts, or download a lot of fonts - easily in hundred-MB range, which is obviously impractical.
That's where knowing the set of required fonts in advance would help immensely.

This is sort of planned but would take forever, since it would require a substantial rewrite.

I wonder if, for a start, it could be easier to just make font access accept a custom trait instead of specifically fontdb? Then a user like me could at least do a 2-pass parse via usvg - first one just to collect list of fonts via custom trait that only stores the font names but reports "font absent" for any query, then asynchronously download all the fonts, and then 2nd pass via usvg+resvg with all the fonts now ready to use.
This would be still a bit inefficient, but probably a relatively simple change to the codebase, and definitely cheaper than downloading arbitrary set of fonts off the internet before it's even known which ones will be required.

Once again, you're underestimating the complexity. Sure, we can probably replace the fontdb with a trait (C API would be a nightmare... or whatever you use from JS), but only for the query part.
What about the font fallback? In the most simple case we basically have to find a font that provides a glyph for a specific Unicode character. How would you handle it? Simply ignore?
What about ownership? Who owns the font? Definitely not resvg. But then we have to have a separate callback to get font's data so we can pass it to ttf-parser and rustybuzz.
What about caching? We don't have any at the moment, but ideally we should cache glyph outlines. To do so, we have to identify unique fonts. How should we do this? I don't know.
What about embedded fonts? Should we store them in, I don't know, an internal font database?
And so on and so on.
SVG Text is a nightmare on itself. Making it even worse by providing a generic API for font's handling isn't worth it.
Sure, if resvg/usvg was designed with this use case in mind from the group up it would be easier. But to implement it right now we would have to basically rewrite everything.
Yes, fonts loading is a bit expensive. Yes, I know about this bottleneck. No, I don't have a viable solution to this problem.

A bit off topic, but why do you use resvg in WASM/JS to begin with? A browser already provides everything you need. I just don't understand the use case. I get that resvg is probably the only option available (afaik), but wouldn't JS + Canvas library be a way better option?
I'm not a web dev, but using resvg seems like an overkill to me. Not too mention the WASM performance.

I get that resvg is probably the only option available (afaik), but wouldn't JS + Canvas library be a way better option?

Yeah canvas would make this a lot easier, but unfortunately there's tons of environments these days that support JS/Wasm but don't have either DOM or native code support - Cloudflare Workers, Deno, Stackblitz, and more. There using Wasm is the perfect option, but it does mean patching over the missing bits. Can't go into details of the specific usecase yet, but it does have to do with rendering SVG in such environments.
I agree there's tons of edge cases, and obviously you've thought about those problems a lot longer than I have - I literally only ran into this only a few days ago and explored a few other options before submitting a feature request - I'm just trying to come up with an alternative that wouldn't require a complete rewrite / refactoring, hence the suggestion above.

I literally only ran into this only a few days ago and explored a few other options before submitting a feature request

Fun fact, resvg itself is already an alternative option after looking into canvaskit-wasm (which, turns out, doesn't have SVG) & generally Skia (which does have some SVG support, but is in an even worse shape in terms of support & API).
Oh well, this is looking increasingly less optimistic. I'll try to think of other possible options, or maybe try to tinker with the trait idea in a local checkout for now. I still suspect that, even ignoring the edge cases you described, it could already cover 80+% usecases.

it could already cover 80+% usecases

Well, if you read the readme - resvg is all about those 100%.
The only solution I can think of is grep-ing SVG by hand and populating fontdb accordingly. This will give you "80%".
The only other solution is to convert text to path on demand. I.e. introducing usvg::Text type and usvg::Text::outline method. But again, this would require a rewrite.
And this still will be a mess, because fonts this day store not only outlines. That's why usvg requires a rewrite anyway to support vector and raster emojis.

Well, if you read the readme - resvg is all about those 100%.

To be fair, while I appreciate it's a worthy goal, right now in the JS scenario I'm describing if the font is missing, then SVG text won't be rendered correctly anyway, so even imperfect list of fonts in advance would already improve compatibility a lot, even if it misses some edge cases.

I'm sorry 😅

JS scenario

You're holding it wrong. (c)
It's not the resvg problem that you're trying to use it in a weird way. It was never designed to work in WASM/JS environments. It's a system, desktop library.

And yet it works, and is relatively popular. Your fault for creating a library that works so well in other environments :P (just kidding, obviously)
(в будь-якому разі, дякую за допомогу, поки пограюсь далі сам :) )

And yet it works

Accidentally. Thanks to being pure Rust and Rust's WASM support. But I want to point out it again that it wasn't designed to be used like this.
Your use case will be supported eventually. The question is when (not soon).
(нема за що)

Slightly off-topic, something I came across while looking into lazy font loading: have you looked into Servo's font scanning code?
Too bad it's not a separate crate (at least yet), but they seem to have a pretty robust implementation for font scanning in macOS / Windows / Linux via FreeType, that lazily (?) enumerates font families & variants and can invoke arbitrary callbacks, together with fallback support: https://github.com/servo/servo/tree/master/components/gfx/platform, e.g. https://github.com/servo/servo/blob/master/components/gfx/platform/windows/font_list.rs
It's probably a bit too hefty for the fontdb since it says "And since fontdb tries to be small and portable [...]", but if/when usvg has that custom font interface, I imagine users could choose to plug that Servo's implementation if they want lazy-loading.

resvg doesn't use system libraries by design, so this is out of scope.
And providing a generic API instead of relying on fontdb would be way too hard.

And providing a generic API instead of relying on fontdb would be way too hard.

Wait I guess I misunderstood your earlier

Your use case will be supported eventually.

I thought you meant giving user ability to preload fonts after parsing but before rendering?
If so, that functionality would be already generic enough to allow someone to use a library like Servo's implementation to find & load the necessary fonts.

I thought you meant giving user ability to preload fonts after parsing but before rendering?

Yes. fontdb isn't going anywhere.

Sure, and I wasn't suggesting it was. I guess we just had a misunderstanding.

You can do it like this now.
    let mut tree = usvg::Tree::from_data(&svg, &opt).unwrap();

    let mut fonts = HashSet::new();
    for node in tree.root.descendants() {
        if let usvg::NodeKind::Text(ref text) = *node.borrow() {
            for chunk in &text.chunks {
                for span in &chunk.spans {
                    for family in &span.font.families {
                        fonts.insert(family.to_owned());
                    }
                }
            }
        }
    }
    println!("{:?}", fonts);
    // load your fonts into fontdb and then
    tree.convert_text(&fontdb, opt.keep_named_groups);

Looks promising, thanks!

I'll close this feature request then, but probably will wait for the release as I have some other stuff to do for now anyway.


fix: LegacyAPIWarning of sqlalchemy

LegacyAPIWarning: The Query.get() method is considered legacy as of
the 1.x series of SQLAlchemy and becomes a legacy construct in 2.0.
The method is now available as Session.get() (deprecated since:
2.0) (Background on SQLAlchemy 2.0 at: https://sqlalche.me/e/b8d9)
role = current_datastore.role_model.query.get(id_)

superseded by #147


nip46: possibly duplicated code

Does this look like duplicated code to you guys? Maybe the second ocurrence was supposed to be related to nip44?
Code starts here. Blocks in question start at lines 102 and 107

Oops.


Guard against d3 'length' undefined error

Occurs when Rickshaw.Graph.render() is called with no data. Here we
simply check that scenario before calling d3. See issue #438.
This is my naive solution to the issue, however it does fix the issue in my application.

Any chance this pull request will be accepted?

@djensen47 Yes; however, it would be more likely to be accepted with tests.

Hi @abirmingham,
Would you be interested in adding tests so we can get this merged?


Not linking with newer versions of react-native.

Using:

react-native v0.41.2
react-native-cli v2.0.1
react-native-create-library v1.0.4
xcode v8.2.1

I did the following:

react-native-create-library Lib.
react-native init RNLibTest.
"Successfully linking" (doesn't look like it) with both automated (react-native link) and manual way: both following the scaffolded instructions of this repo AND following the linking section of react-native docs.

The result is just saying that the library is undefined as if react-native didn't find it in NativeModules.
Anyone else is encountering this issue? I created a stackoverflow question to keep this issue clean: http://stackoverflow.com/questions/42281942/library-not-linking-with-newer-versions-of-react-native
ping: @frostney

Hi @pgarciacamou, thank you for creating this issue. I just released version 1.1.0 which adds support for React Native > 0.40. Could you please try again and let me know if you still encounter this issue? Thank you for you patience.

@frostney excellent timing! I will test right away and let you know.

@frostney
You might want to remove peer dependencies from package.json.
  "peerDependencies": {
    "react-native": "^0.29.0",
    "react-native-windows": "^0.29.0"
  }

I was unable to link using react-native link.
To reproduce:
Create Library:

mkdir react-native-lib
cd ./react-native-lib
git init && git remote add origin git@github.com:pgarciacamou/react-native-lib.git
react-native-create-library Lib
removed package.json -> peer dependencies.
git add . && git commit -m "run react-native-create-library" && git push origin master

Create React Native App:

react-native init LibTest
cd ./LibTest
git init && git remote add origin git@github.com:pgarciacamou/LibTest.git
npm install react react-native pgarciacamou/react-native-lib#master
git add . && git commit -m "run react-native init" && git push origin master
react-native link
react-native run-ios

I debugged LibTest;
import RNLib from 'react-native-lib';

export default class LibTest extends Component {
  componentDidMount() {
    console.log('RNLib');
    console.log(RNLib);
    console.log('NativeModules');
    console.log(NativeModules);
  }
  render () { ... }
}

// ...

and this is what I get:

I didn't have time today to test linking manually, I'll try it tomorrow, although TBH this is the minimum viable product and there is no reason why this should be working.
I'm starting to wonder if it is a bug that was introduced with react-native v0.40

i think it is better to have react-native deps as devDependencies

linking is not working using the master version of this package

@sibelius Thanks for catching this. I'll get right on fixing this.

I figure it out the problem, linking is working fine
I was missing the --save tô add my lib dependency to package.json

@sibelius

i think it is better to have react-native deps as devDependencies

Why?

I was missing the --save tô add my lib dependency to package.json

What? I did this too and it didn't work for me. Are you sure that the NativeModule is not undefined? In my case, I import the npm module just fine, but the NativeModule is undefined so I can't access the methods from Objective-C.
Is there anything else you did different? Could you elaborate a bit more?

@pgarciacamou This is happening to me as well. My obj-c code cannot find React...
#import <React/RCTBridgeModule.h> not found. Any updates on this?

@ugiacoman what versions of react-native-create-library, react-native, react, react-native-cli, xcode do you have?
Also, how are you running the project?

@pgarciacamou
"react": "~15.4.1"
react-native-cli: 2.0.1
react-native: 0.42.3
<EMAIL_ADDRESS>I don't have react install globally though, do I need to?
I tried opening the xcode project and running, but that didn't work. Also tried opening via react-native run-ios but couldn't find the header files.

I'm using
react: 15.4.2
react-native: 0.42.3
react-native-cli: 2.0.1
react-native-create-library: 1.1.0
xcode-select: 2343.
node: v6.9.5

I don't have react install globally though, do I need to?

no

I tried opening the xcode project and running, but that didn't work. Also tried opening via react-native run-ios but couldn't find the header files.

Did you try to run the library alone? I believe there is nothing to run, right? react-native-create-library creates just a library that you can import (I could be wrong tho). Do you mean you created the library and then installed it somewhere else and it errored out?

@ugiacoman when you create the library with react-native-create-library it creates a README.md file with instructions on how to install/run it, did you follow those?

@ugiacoman let me know if it works for you, I've been meaning to close this issue since this answer I posted in StackOverflow.

@pgarciacamou Ohhh, get it now, it all just came full circle. Thank you so much, it worked!
So the idea is you develop it within another project. Completely makes sense :)
Do you think there should be something in the README to clarify the development process?

@ugiacoman
I'm glad it worked! 💪

So the idea is you develop it within another project.

To be honest, I haven't found best practices surrounding this subject (just some ideas here and there), and all the approaches I've found have disadvantages one way or another, so I guess it is up to you.
If you ask me, what I like to do is create a fully working example within the library, which has pros and cons as well 😩... For example, it allows other users to contribute easily (without having to create their own project to test new features or small contributions), but it creates an overhead withing the project (a fully working react-native app, including iOS files, etc...).

Do you think there should be something in the README to clarify the development process?

Not really, IMO that is beyond the functionality of this repo. I think the default README file doesn't include something like that because of the same idea above, the rest should be added by the developer depending on the approach taken to run the library (if any).


Sessions and Deferral

No mention of sessions with deferral.
Is session order maintained or does the consumer have to do additional work to ensure correct processing order of session based messages?
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 13cd13b7-63fb-9832-351a-39bc7021f90c
Version Independent ID: 2e316632-064b-9918-c4f3-60b1e79da2df
Content: Azure Service Bus - message deferral - Azure Service Bus
Content Source: articles/service-bus-messaging/message-deferral.md
Service: service-bus-messaging
GitHub Login: @spelluru
Microsoft Alias: spelluru

@jtobinisaniceguy
Thanks for your feedback! We will investigate and update as appropriate.

@jtobinisaniceguy Deferred messages concept is not applicable to session enabled service bus queues. This is why it is not mentioned in the document. As you mentioned, the order of the messages is maintained in session enabled service bus using session id. Clients will use session id to process the session based messages

@jtobinisaniceguy Did you get  a chance to look at my previous comment?

Yes
It is disappointing that deferred message functionality is crippled for session-based queues. Unfortunately, nothing in the documentation states that, which it should. This is somewhat similar to forwarding, this is not even available for session based queues, but somehow isn't called out in the docs.
Deferred messages can work in a session based environment, however the logic, to implement, must reside in the custom receivers. This feature could be implemented at the Service Bus level if session functionality is to be on par with non-session configurations.
I would say that overall, the documentation and functionality for session behavior in Service Bus is lacking.

@jtobinisaniceguy Actually, deferred messages can be used with sessions. Please note though, that when you defer a message it means that absolute order is lost, as you take one specific message out and put this aside. As you continue receiving messages, these will still come in order.

Thanks @EldertGrootenboer
#please-close


Will the extensions be updated ?

Hi just, the current Extension don't fully works, it successfully parsed all my playlists, but it remain unable to play any songs.
Will the extension be updated or the support is dropped ?
Thank in advance for the answer

Hey sorry, my tidal subscription expired so I don't really have access to tidal anymore. I might renew it but not anytime soon :''(

No problem it is fine ! I don't know a lot of people using Tidal either so it's rare to see an extension specifically made for it !
Tidal has changed so many things in the past months they broke a lot of API stuff etc...
If only I was a front-end dev x)
Anyway thanks for the rapid answer ! 👌


Monolog handler fails when the extra data has numeric array keys

If $key is an integer in the following line, the function call will fail with Uncaught TypeError: Argument 1 passed to Sentry\State\Scope::setExtra() must be of the type string, integer given.
https://github.com/getsentry/sentry-php/blob/50f58938ef857c398e188b79c2fc5036f1adbeb2/src/Monolog/Handler.php#L62
I suggest adding a cast to string there, or possibly making the first argument to setExtra more relaxed.

Good catch! I don't have an example in mind of why the $key should not be a string, so I would say that casting is the best solution here. Are you willing to open a PR to fix it?

Sure, done!


⚠ Bofrost.be has degraded performance

In 146bf25, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2536 ms

Resolved: Bofrost.be performance has improved in 77b1c42.


Uncaught TypeError: this.parent is not a function

When I create the mousePressed function and use "this" referring to the element that triggers it, I always got the problem : this.sth is not a function. It never happened before when I use "this" in the mousePressed function, which really confused me. ( By the way, Thank you for your helpful videos^_^
My code is attached below:
function setup() {
noCanvas();
for(var i = 0; i < 3; i++){
var p = createP("This is a:");
p.style("background","#539124");
p.style("padding","30px");
p.style("color","#fff");
var li = createA("#","apple");
li.mousePressed(addPic);
li.parent(p);
}
}
function addPic(){
var pic = createImg("apple.png");
pic.size(100,100);
var paragraph = this.parent();
pic.parent(paragraph);
}
function draw() {
}

Looks like you have run into this issue! Try using an earlier version of p5 (for now).
https://github.com/processing/p5.js/issues/3260
https://editor.p5js.org/codingtrain/sketches/rkbxI2Njm

Looks like it was fixes with https://github.com/processing/p5.js/pull/3261
Working demo: https://editor.p5js.org/gruselhaus/sketches/UHnQGKa81


[Issue 1950] Do not remove whitespace after number

Description

[x] Source branch in your fork has meaningful name (not main)

Fixes Issue:
Fixes #1950 causing Uncaught SyntaxError: Invalid or unexpected token by checking if the input before the dot is a number or not. Depending on the regex used to perform a number match, we set space_before_token to true in the handle_dot function.
As an additional work since the expectation was for both js as well python beautifier, I went ahead and tried to made appropriate python changes. But the python implementation for the same fails to compile as the latest version of black has removed support for py27. https://pypi.org/project/black/. Therefore, I have also included the whitespacing changes that were made by the latest version of black and has also pyproject.toml to remove the dependency.
Error reported before removing dependency for py27 for black:
Testing python beautify functionality...
/mnt/c/Open_Source/js-beautify/tools/python-dev3 black --config=python/pyproject.toml python
Usage: black [OPTIONS] SRC ...
Try 'black -h' for help.

Error: Invalid value for '-t' / '--target-version': 'py27' is not one of 'py33', 'py34', 'py35', 'py36', 'py37', 'py38', 'py39', 'py310'.
make: *** [Makefile:33: py] Error 2

Have verified that all the tests are passing after this change.
Before Merge Checklist
These items can be completed after PR is created.
(Check any items that are not applicable (NA) for this PR)

[x] JavaScript implementation
[x] Python implementation (NA if HTML beautifier)
[x] Added Tests to data file(s)
[ ] Added command-line option(s) (NA if
[ ] README.md documents new feature/option(s)

Still needs tests. I think this code will fail on the following input a.b21.send().

Thanks!! Will add tests for this change. Also, the input of type a.b21.send() works and the result is the same. Is that not the expectation for such cases?

Sorry, What I meant was a.b21 . performAction(). The space should be removed but it looks like this change woy preserve it.

Sorry, What I meant was a.b21 . performAction(). The space should be removed but it looks like this change would preserve it.

@bitwiseman, so I have used refex to match numbers from the start to the end. Therefore, a.b21 . performAction() would be modified to remove space accordingly to produce. a.b21.performAction(). I have also modified the test to represent this scenario.

@bitwiseman, I'm unable to root cause why 'make ci' is failing for me. Any pointers to understand what could be wrong?
Testing js-beautify cli behavior...
Created /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw...
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-file.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-pipe.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-n.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-rn.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-eof.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-eof2.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-n-dash.js
beautified /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-rn-dash.js
Files /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-n-dash.js and /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-rn-dash.js differ
Files /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-n.js and /mnt/c/OpenSource/js-beautify/python/target/test_temp_v8Gw/js-beautify-rn.js differ
unbalanced parenthesis at position 0
js-beautify output for 'src/*.js' was expected have 2 files.
unbalanced parenthesis at position 0```

@bhaveshvasnani
Not sure why make ci is failing for you.  Usually when I can't nail down a problem, the first thing I try is getting a clean, up-to-date environment: make sure everything I care about is checked in, git clean -xfd, git reset --hard, git pull, git merge upstream/main, etc. Then try the build again.

@bhaveshvasnani Not sure why make ci is failing for you. Usually when I can't nail down a problem, the first thing I try is getting a clean, up-to-date environment: make sure everything I care about is checked in, git clean -xfd, git reset --hard, git pull, git merge upstream/main, etc. Then try the build again.

Thank you @bitwiseman!! Let me see if I could get make ci build clean.

@bhaveshvasnani Not sure why make ci is failing for you. Usually when I can't nail down a problem, the first thing I try is getting a clean, up-to-date environment: make sure everything I care about is checked in, git clean -xfd, git reset --hard, git pull, git merge upstream/main, etc. Then try the build again.

@bitwiseman so the actual problem was I was incorrectly using regex search function and sorting that out fixed make ci locally for. So, hopefully it should be good now.


Export some kind of general Contract struct

Not exactly sure how to describe this, but I suppose it would be analogous to the Contract object what we know and love from ethers which can be instantiated as an interface from an ABI or as an instance when also provided with an address and web3Provider.
This would allow us to make generalizations for arbitrary contracts some common functionalities.
One particular place where this could come in handy is when implementing an Event Listener in place of the generic EventRetrieving
pub struct EventHandler<C: EventRetrieving, S: EventStoring<C::Event>> {
    contract: C,
    store: S,
    last_handled_block: Option<u64>,
}

Where
pub trait EventRetrieving {
    type Event: ParseLog;
    fn get_events(&self) -> AllEventsBuilder<DynTransport, Self::Event>;
    fn web3(&self) -> Web3<DynTransport>;
}

At the moment we need to implement EventRetrieving in a very boiler plate kind of way for each contract we want to listen to.
impl EventRetrieving for GPv2SettlementContract {
    type Event = ContractEvent;
    fn get_events(&self) -> AllEventsBuilder<DynTransport, Self::Event> {
        self.0.all_events()
    }

    fn web3(&self) -> Web3<DynTransport> {
        self.0.raw_instance().web3()
    }
}

So, I think there is a large change we can make to the ethcontract binding generation. Specifically we can generate:
struct MyContractAbi;

implement Abi for MyContractAbi {
  type Function: MyContractFunction;
  type Event: MyContractEvent;
}

enum MyContractFunction {
  Foo {
    value: U256,
    owner: Address,
  },
  Bar,
}

enum MyContractEvent {
  // same as now.
}

We would then be able to create instances of type:
let instance: Contract<MyContractAbi>;

Some other advantages of this separation:

We can define functions generic on some contract type:

fn do_something_to_contract<C: Abi>(instance: Contract<C>) { /* ... */ };
fn call_some_function<C: Abi>(instance: Contract<C>, function: C::Function) { /* ... */ };

Have a better way forward for mock contracts:

let instance: MockContract<MyContractAbi>;

Be able to encode function data without needing a contract instance:

let calldata = MyContractEvent::encode(MyContractFunction::Foo { value, owner });

Trying to make this general deployment block fetcher
async fn get_deployment_block<T: Transport>(contract: &ethcontract::contract::Instance<T>) -> Option<u64> {
    match contract.deployment_information() {
        Some(DeploymentInformation::BlockNumber(block_number)) => Some(block_number),
        Some(DeploymentInformation::TransactionHash(hash)) => Some(
            contract
                .raw_instance()
                .web3()
                .block_number_from_tx_hash(hash)
                .await?,
        ),
        None => None,
    }
}

But when I give it a generated contract instance I get this:
error[E0308]: mismatched types
   --> shared/src/balancer/event_handler.rs:286:71
    |
286 |         let deployment_block_two_token_factory = get_deployment_block(&two_token_pool_factory).await;
    |                                                                       ^^^^^^^^^^^^^^^^^^^^^^^ expected struct `Instance`, found struct `BalancerV2WeightedPool2TokensFactory`
    |
    = note: expected reference `&Instance<_>`
               found reference `&BalancerV2WeightedPool2TokensFactory`


Storage system crashes player when opening on a server

Describe the bug
The storage system is crashing players at random times when they open the storage using the storage actuator keybind.
To Reproduce
Steps to reproduce the behavior:

Click Storage Actuator keybind
See error

Expected behavior
Usually it would open the storage system as usual, but every now and then it crashes me to the point where i have to wipe my player data in order to get back on the server.
Screenshots

System (please complete the following information):

Occultism Version: occultism-1.19.2-1.88.0
OS: Windows
Minecraft Version: 1.19.2
Modpack Link and Version, or list of mods: https://drive.google.com/file/d/1eh-vrvvPc6sYj-PHfIW3zb4jIzPP93pn/view?usp=sharing

Additional context
This only happens randomly, I haven't found a reliable way to recreate it other than using the storage system keybind over and over again.

@bunbonz This looks like there are items in the storage with an NBT tag bigger than minecraft supports, because the storage system itself does not add any nbt, it only sends the nbt of the items in it.
Can you provide a log for the client and the server?
Maybe I can find out more. However, if it is the too much nbt thing then what we would need to do is identify the mod that provides the item in question.
When creating an item, modders can specify to only send a limited part of the NBT to the client, if this is properly implemented the crash can be prevented

Closing as stale


remote side rolls back the target dataset losing locally created snapshots

the -F flag is passed to 'zfs recv' so any locally created snapshots are lost on the next send/recv

I've created an initial fix for this issue and have begun testing: 3693a2fbd1fc5392ba5c6fd83956d527d814fbd3

Merged issue-008 branch with master: 3693a2fbd1fc5392ba5c6fd83956d527d814fbd3


support for lakshore340 ioc

Hi Subin. Please don't merge your changes to this branch.
The point of this is that it is a starting point for the tutorial and specifically does not have lakeshore in it.
It's a good idea to ask for review before merging changes into branches that you did not create (usually main is the only one to care about but this is a special case)
No worries, I can undo this and  I'll take responsibility for giving you the rights without explaining the above.

Apologies. Thank you for clarifying.

This branch is now locked to avoid future issues. Should have done that before!


Build error in 0.4.1

I got an build error in this Line:
stream.attachCamera(AVMixer.deviceWithPosition(.Back))
says  'Use of unresolved identifier AVMixer'

never mind. I used DeviceUtil.deviceWithPosition(position) and it works. You may need to update the docs.

close as duplicate #104


More detail for evaluation.

"Step 1. Validation set evaluation: Obtaining the validation results for your model using LLM evaluation tools, OpenCompass. The validation dataset download from 🤗 Huggingface. The data directory structure in the opencompass:"
Where should I put the downloaded file? And what command line should be used to run the evaluation program? It would be better if you could provide a more detailed explanation of this.
Thank you！

Please place the  and  files under opencompass/data/mmlu_cf. The MMLU-CF validation branch of OpenCompass is currently pending merge. Please wait for it to be finalized.


chore(deps): update dependency @types/jest to v25.2.1

This PR contains the following updates:

Package
Type
Update
Change

@types/jest
devDependencies
minor
25.1.5 -> 25.2.1

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v5.0.7
GitHub release

Your semantic-release bot :package::rocket:


minified_js: Clean up unused imports.

@timabbott

@badmon I'm not super interested in PRs that remove unused imports in one file since they're kinda a waste of review time; I'm going to merge this but please focus your efforts on something more constructive.


docs: update package and regenerate package-lock

Running npm run build, you should no longer get the message:
[ERROR] Invalid docusaurus-theme-mermaid version 2.4.3.
All official @docusaurus/* packages should have the exact same version as @docusaurus/core (2.4.1).
Maybe you want to check, or regenerate your yarn.lock or package-lock.json file?

yarn build didn't have the same issue, and yarn install did not generate a new yarn.lock
This PR has:

[x] been self-reviewed.

actually something looks a bit off with commit history on this PR (looks like a lot more commits than changes here), could you try to fix that up?


Generate Reusable enumeration in controller ...

...with UITableViewCell/UICollectionViewCell reuse identifiers - Fix #5
Add method to use the Reusable enumerations to dequeue or register view
add info in REAME.md, less than previous PR

@phimage please review the changes here. there is conflict due to another PR merged already.

conflict resolved

damn, again need coreection to be merged :-)

@phimage

Is it good?
no conflict when I merged this time

yeap, thx


Get list of files in directory

Is there any way to get a list of files in a directory? The use case for me is to be able to build a gallery based on all of the images in a specific directory. The methods I've seen so far have been to specify each individual file via data files or individual markdown files.
What I would prefer would be to have some sort of function to be able to get a list of files. E.g.
<!-- Get files by name such as 'picture.png' -->
{{ $filesInDir := files "./static/img/" }}

<!-- Produce img tags based on images in specified directory -->
{{ range $filesInDir }}
  <img src="/img/{{ . }}" />
{{ end }}

Does this already exist? Is this beyond the scope of Hugo? Just seems more convenient than specifying data entries, which would involve changing the data whenever you add/rename/delete a file.

As far as I know does such function not exist in Hugo. But it's not out of the scope. Some would need to add this function the templates. And the idea with the image gallery is good. It could be a nice addition to one of my themes so that the users don't need to mess with the configs.

In the upcoming Hugo 0.15 there is a new readDir template func that does exactly this.
See
https://github.com/spf13/hugo/blob/master/docs/content/meta/release-notes.md
The discussion forum is, however, a better place for questions.


Should --working cause fix command to not stage changes?


Final cut michael revision

Updating documentation for scripts.

I would recommend moving your Q&A to the bottom, that is where I would expect to find it. You can link to it from above

Thanks, @melissakendall! Your comments were taken into account in #13.


Create third-party-tools.zh-cn.md

Translated the third-party-tools.zh-cn

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7c20560). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #6303   +/-   ##
=========================================
  Coverage          ?   38.81%
=========================================
  Files             ?      359
  Lines             ?    51100
  Branches          ?        0
=========================================
  Hits              ?    19836
  Misses            ?    28393
  Partials          ?     2871

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 7c20560...85afbdb. Read the comment docs.

but the time is not right.

i trust @lunny so ...:


minimal<EMAIL_ADDRESS>upgrade

... to avoid breaking with rust-lang/rust#49799.
nalgebra 0.15 uses rand 0.5 which brings in a lot of changes around Rand. I'm not familiar with poisson enough to port it further along.

Well this is awkward: I had updated dependencies in #18, but apparently forget it completely and didn't merge it.
Thanks for making me remember that!

I see! My bad for forgetting to look through the pending PRs.


Make the confirmation on ledger/ trezor much more clearer

Need to be more specific, ask them to confirm on their device. Many first time users do not understand whats going on

We have add more instruction in ledger in case timeout, or user not enable contractData/ browser support in Staging deployment


ovn-tester: Modularize test definitions.

This change encapsulates tests into their own files. Each yaml section
in the configuration file corresponds to a module in the tests/
directory. This has the following advantages:

The logic of individual tests can be contained within its own
module/class.
The ovn_tester.py file can be made smaller, allowing for it to be the
orchestrator of tests, rather than holding test logic.
New tests can be defined without needing to modify ovn_tester.py. Just
add the new module to tests/ and as long as the configuration is
present, the new test will be run.
This allows for customized ordering of test scenarios.
This allows for test scenarios to be skipped if desired.

In this change, the cluster bringup test is handled in a special way. It
is run directly in ovn_tester.py since it must be run as the first test.
This could be made a bit less awkward by changing the config file
schema, but I sought to make this change not break things for people if
possible.
Signed-off-by: Mark Michelson<EMAIL_ADDRESS>

Looks good to me, thanks!


GD extension on Mac

Mac OS Ventura - Version 13.4.1 (22F82)
Godot 411 stable
godot solana sdk : version 0.2
Error to load the gdextension when I start or restart Godot.
I will try to compile myself the extension to see if loading better...

If I open the project in console I have this error:
(It's a Mac with arm processor M1)
ERROR: GDExtension configuration file must contain a "configuration/compatibility_minimum" key: res://bin/godot-solana-sdk.gdextension
   at: load (core/extension/gdextension.cpp:593)

If I add
compatibility_minimum = "4.1"
to
godot-solana-sdk.gdextension
the first error disappears but a new one due to my processor architecture appears :
ERROR: No GDExtension library found for current OS and architecture (macos.arm64) in configuration file: res://bin/godot-solana-sdk.gdextension
   at: load (core/extension/gdextension.cpp:629)

Hi Boyquotes! Thank you for trying the plugin. We have big plans for it.
As you have already figured out, there is a line missing in the .gdextension file (compatibility_minimum = "4.1").
The second error can be solved by adding another line:
macos.template_release.arm64="libgodot-solana-sdk.macos.template_release.framework"
Let's see if that works and I'll commit the changes

Actually, according to the Godot docs, it should be:
macos.template_release="libgodot-solana-sdk.macos.template_release.framework"
macos.template_debug="libgodot-solana-sdk.macos.template_debug.framework"

(Without .arm64)
I don't have a mac to verify, but if it does not work we can patch that lines for macos.

Nice for the Big plans between godot and blockchain out of box. Me too, I want to connect with multiple blockchain(evm, cosmos, solana, PolkaDot...)
Do you know/participe or want to the solana encode club hackhaton this month? I have already registered, if you want to participate and propose this extension I will join your team...
With the minimum version line in the conf file and the 2 line with the path to macos version, it works fine with the extension compiled  localy.
I have write a workflow to build the extension automatically for all platform, build work fine, I test the macos extension generated by github action tomorrow #pull99

Super! Yeah it could be a good idea with the hackathon. Do you have a discord, my username is virusaxel. We can talk about the hakathon

It's a accelerator/hackathon https://www.encode.club/encode-solana-accelerator. I send you an discord invitation.
With this new pull request https://github.com/Virus-Axel/godot-solana-sdk/pull/100 , the extension is loaded correctly on macos through the extension files compiled via github action 👍 (https://github.com/Boyquotes/godot-solana-sdk/actions/runs/6763378423)


feat: added keywords for async/await, list

title is pretty self-explanatory i reckon
sigma out

I was thinking of doing this but you beat me to it. I reckon await could be mapped to let him cook.  I reckon await could be mapped to let him cook. More lore accurate and skibidier

yeah, async = hol up and await = let him cook would follow the IEEE standard.

let him cook is an existing keyword, however :(

ah correct. suggest realiasing while to edging. would also remain consistent with continue = edge

Alternatively, while could be renamed to gooning. That was the original mapping before I changed it to let him cook.


tests

[ ] basic tests
[ ] PR test automation
[ ] code coverage

Stackoverflow question draft:
I've written a function, require.py(path), that will resolve relative and absolute imports for modules as follows:
# lib.py
CONST = 'const'
def hello():
  return 'world'

# script.py
import require
lib = require.py('path/to/lib')
print lib.CONST # => 'const'
print lib.hello() # => 'world'

I am attempting to write tests for this function via the unittest library.
One set of tests should make sure that require.py can import modules (1) in the same directory, (2) in a descendant directory, (3) in an ancestor directory, and (4) in a "cousin" directory:
(1): require.py('lib')
(2): require.py('sub/subsub/lib')
(3): require.py('../../../lib')
(4): require.py('../other/lib')
However, for my tests to call require.py with these parameters, I would have to create copies of lib.py at those locations (relative to the test script), and clean up (ie. remove) those copies (and any .pyc files they generate!) afterward. Messy.
Instead, I was thinking of making a temporary workspace (via tempfile.TemporaryDirectory) where I could make arbitrary directory structures (rooted in the temporary workspace) and stick copies of lib.py and script.py. Then my unit tests could simply run the script.pys and see if they work.


fix: update dependencies

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[RFE] bind unbounded PVC's in stage one

While it was simpler to assume that users will create dummy pods
to bind the PVCs in the namespaces to be migrated, we can relax this
restriction by just asking the users to unbind the PVC's and scheduling
a dummy pod as part of stage 1.
@jwmatthews @eriknelson @pranavgaikwad thoughts?

@alaypatel07 Behind we introduce anything in Stage 1, I am leaning towards re-writing Stage 1 in Ansible. What do you think?


Pin alignment for Pi Pico are off by one

please provide some information on what you're reffering to

In all orientations the Pi Pico pins didn't align.

Hi, What device are you using?

I see this issue on an iPhone 6S running iOS 14.4 and an iPad 6th gen running iPadOS 14.5 beta latest.

It failed for me on an iPhone 8 running 14.4

Gotcha. What version are you running of Adafruit AR? The version number is on the main screen.

v.1.10.4

Same for me; there's no newer update in the App Store.

Thanks y'all. Im working on this issue. It works fine on 3 different iPhones I have, but I was able to replicate the problem on my iPad. Either way, I'll find a fix!

Howdy! There is a new update for Adafruit AR that fixes the pin alignment overlay issue for the Raspberry Pi Pico.

This works for me on the iPHone 6s and the iPad. Thanks!

Awesome!