AutometicTimeUnrollOfFeedbackNetworks

Hey @ParthaEth, in my opinion, this would be better of as a pull-request on Keras, the contributors might have either tried it or getting better support.
Also one of the Keras Variational Autoencoder implementations on github does contain an approach to do Recurrent Latent Variable, maybe check those out first.

@0bserver07 Making a PR to the keras repo is the plan. But it would be great to have a team to work with rather than trying it myself. This functionality isn't there in keras at this point of time. I am trying to find a few people who would be interested in doing the project with me here. Do check the conversation here.

@ParthaEth  Try RecurrentShop

@farizrahman4u any plan on merging this repo into keras itself? Looks like it does what I am trying to  program into keras. What was the motivation to have it as a separate repo? If you/we make a PR with this code into keras it will be reviewed by more people and perhaps more tests will be written and things in general will become better. It is also not nice to have things in bits and pieces all over as if keras decides to make a change in its code you will have to keep an eye on to make this repo compatible again.

@farizrahman4u Could you have a look in here as well. Some how I am not ablr to get my point understood but may be you can rephrase it better.

"Support depth first RNN stack computation in Keras"

@farizrahman4u sorry to have not included a link where I wanted to point you in my last to last comment. I wanted to actually look at this issue.

Any one else interested in working/ exploring how to bring in support for RNNs in keras? I guess if we have things that can be done in TF but not in Keras then we defeat the purpose of keras being a research platform and make it rather a industrial user platform. The whole idea is to let one build random things and try it out quickly even though it is not well established architecture.

Hey @ParthaEth, with the suggestions given from @farizrahman4u who have been a long contributor to Keras, it looks like that partially of what you have suggested is there.
Maybe a pull-request to do that would be cool? To avoid doing redundant work at the mean-time.
It looks like that @farizrahman4u have started another project here as well :) keras-contrib. Maybe a pull-request on that as well would be relevant.
For research, if you're interested in implementing relevant Keras or TF work, there is a great list of Research Topics that are actively being researched and developed (check them out):

Chromosome Segmentation
Social media botnet detection and analysis
Identifying biomedical articles at risk for retraction
Multitask and Transfer Learning
Layer-wise supervised incremental training of residual networks

@0bserver07 I saw  keras-contrib too the purpose of it is I guess to try out new keras features before it goes official. So yeah i guess every PR will go to this repo first. If I understand correctly,  @farizrahman4u 's suggestion was to try RecurrentShop, which is nice of course but the idea of having all my network inside a single recurrent layer is what bothers me. I was thinking to automatically unroll RNNs over time when Model(input=I, output=O) is called. Also implementing a feed back layer to let people create feed back over time loops. With all these a new RNN might look something like the following.
input = Input(batch_shape=(2, 5), dtype='float32', name='current_pose_input')

f1 = Feedback(initial_tensor=Zeros(dim))

o = LSTM(10, f1)
o = TimeDistributed(Dense()(o))
f1(o)

m = Model(input=input, output=o)
m.compile('rmsprop', 'mse')

@ParthaEth I wonder if there couldn't be a more explicit and expandable solution.
I happened to notice this on Tensorflow 1.1 "Added a new lightweight queue-like object - RecordInput."  I was thinking of something like a queue or accessible memory location.  The trick of course is it would need to be used in such a way that it was able to read/written to at the right part of the process.
The benefits I can imagine are things like accessing a multiple-length queue to pick the item you want, or the hard-attention kind of explicit memory access.  Of course, we could just do the simple 1-length queue to get back to your example.
I'm in favor of your cause here.  I'm surprised there's also not a easy way to feedback your last error.
Thanks for getting the conversation started.

@cpernoud Glad that you find this interesting. I am sure it is faced by many researcher working with RNNs. I have a feeling RNN are receiving far less attention than they really should. I am not sure how best to approach this. Honestly I do not have to time at the moment to develop the feed back layer my self. So I just kind of pushed it in a todo list. :)


Use $USD change instead of $EOS

[x] update stats to $USD
[x] update protocol TVL to $USD
[x] update 7d chart to $USD

Seems all implemented, we can expand this issue to have better $EOS/USD visuals, but for now all the values are defaulting to $USD instead of $EOS


GetRows/Rows Picks up Empty Rows

Description
GetRows/Rows allocating memory for empty rows increasing memory usage or stalling completely
Steps to reproduce the issue:

Get excel from multipart
Print out the rows (example code works)

Describe the results you received:
Picks up, allocates, and prints empty rows. This is a bit annoying because excel files often have tons of empty rows. The file I got as a test had over 1 million empty rows in it, and kept causing my local server to crash. The file itself was only about 5mb big. The issue is I cannot stop users from uploading these files with empty rows. Max file size doesn't help because a 5 mb file is enough to store over 1m empty rows.
Describe the results you expected:
Expected to ignore empty rows
Output of go version:
go version go1.13.4 darwin/amd64

Excelize version or commit ID:
v1.4.1

Thanks for your feedback. Please upgrade lib to the last version. GetRows/Rows read as stream since v2.1.0.

Any additional workaround for this issue? It's happening for me version 2.3.0. A sheet with 7300 rows is getting this from fmt.Println(len(rows)):
1048576

Update: GetRows is still manifesting the undesired behavior. An unsatisfying workaround is to iterate through the rows using:
for rows.Next()
and for my use case break when:
len(rows.Columns()) == 0
Not very robust, but if this can help someone else this is why I've posted it.

Still having the same problem on version 2.3.1

On one hand, empty rows are technically valid and are correctly represented in the file.  It doesn't seem like the ideal default behavior, though, since generally empty rows are not useful. The stream should fix any memory issues. To ignore the rows, there are a few simple workarounds. The one posted by ddemoss222 should work just fine. I just checked if the "Id" cell was empty instead.


Jetson Nano support

Is this library heavily coupled to the RPI series? What would it take to port it to the Jetson Nano?
Any guidance would be greatly appreciated! Thank you

The library is unfortunately very intimately tied to DispmanX, BCM2835 (and compatible family) and its SPI+DMA stack.
If developing a driver for Jetson Nano or another completely different hardware, the general reusable parts are probably in the framebuffer diffing code which performs diffs on subsequent framebuffer snapshots, and turn it into an upload span list for the SPI bus; and in the individual display driver files (ili9341.cpp, st7735r.cpp, etc.) for the display initialization sequences.
Programming against SPI and DMA peripherals is likely going to be its own world for other hardware than BCM2835, likewise the display framebuffer stack on different display drivers. (e.g. in NVidia driver counterpart/replacement for DispmanX)

Thank you for the quick response! Will close this issue then


Fix show rewards for older epochs

Description
This PR fixes a bug where the rewards:show command in celocli would fail for sufficiently old epochs. It uses the blockNumber value for functions calls retrieving staking reward data when the --estimate flag is not used. These calls will now succeed when using an archive node.
Other changes
N/A
Tested
Tested running celocli rewards:show --group <ValidatorGroupAddr> --node https://public-archive-nodes.celo-testnet.org --epochs=600 on validator groups 0xdadbd6cfb29b054adc9c4c2ef0f21f0bbdb44871 and 0x47b2dB6af05a55d42Ed0F3731735F9479ABF0673, and  celocli rewards:show --voter <ReleaseGoldContractAddr> --node https://public-archive-nodes.celo-testnet.org --epochs=600 for several valid release gold contract addresses.
Related issues
N/A
Backwards compatibility
Also tested to not break rewards:show --estimate functionality with non-archive nodes using https://forno.celo.org.
Documentation
N/A

Would we be able to change the useBlockNumber in the getGroupVoterRewards and getValidatorRewards to an optional parameter? The classes that these methods belong to are part of the public API so these would be breaking changes.

Done


Options z-index question

I'm working on a new page, and I have a select box that allows the user to change the grouping in the jqGrid (jQuery plugin). Using the standard HTML select works fine, and the options list is displayed on top of the jqGrid below it. However, when I make the box a selectBoxIt box, the options are always displayed behind the grid. I've tried changing the z-index of the select box element and the jqGrid, but I can't seem to get the options to be displayed on top.
The grid has a z-index of 1, and the div that houses the select box has a z-index of 10000. I've also set ".selectboxit-container .selectboxit-options" to have a z-index of 9999999, but no matter what I set them to, the options are always displayed behind. Any thoughts on what I could be missing?

Thanks Limos, that css you suggested above (Feb 4, 2014) is what I was looking for to have my list appear without being 'cut-off'


Fix the problem that is preventing the Avro to be the Flink serializer instead of Kyro.

Fixing issue record in #463 .

I created an Avro SpecificRecord Java Class to resolve the Kyro problem.
See #470


Write a GO program to check pangram string

Why:

Closes #1266
What's being changed:

Added a go program that accepts a string and checks if it is a pangram using maps
Tested:

➜  program git:(master) ✗ go run program/check-pangram-string/check-pangram-string.go
thequickbrownfoxjumpsoverthelazydog
thequickbrownfoxjumpsoverthelazydog is a pangram
➜  program git:(master) ✗ go run program/check-pangram-string/check-pangram-string.go
notapangram
notapangram is not a pangram

Fixes #1266

@harshraj8843 can you help in a review?


As a final user, we need a way to spot new issues

Feature request: New badge
Some sort of label or badge for new issues created after we checked of the last time would help

Closed as done-done


irmin-pack: only remember commits in `newies'

Experimenting with another way to reduce the number of newies: only track the new commits and let Repo.iter complete the missing ones :-)
But this doesn't pass the tests and I am not sure why..

It can be that only blobs were added during freeze, in which case, if we don't add them to the newies, they are lost. The issue is that objects are not added "by bulk" for each commit but instead they are added progressively.

It can be that only blobs were added during freeze, in which case, if we don't add them to the newies, they are lost. The issue is that objects are not added "by bulk" for each commit but instead they are added progressively.

So I've modified the lock to make sure freeze are finished only when a "bulk" of writes, corresponding to a high-level commit, is complete. That now seem to pass the tests.

Rebased.

@icristescu I've modified the comments, let me know what you think.

@icristescu I've modified the comments, let me know what you think.

looks good, thanks!


ignore api-mapping and domain-name resource

Description of changes:

Currently ignore api-mapping and domain-name resource for apigatewayv2 controller
It is harder to perform e2e test for these resources because they need a public DNS and SSL certificate to test.
I will work separately in enabling e2e test for these resources and then add them back.
Keeping non-tested resources in controller release is not a best practice.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Fair enough. Hopefully we can follow up quickly and get these added back into a stable chart soon after release.
/lgtm


Is it possible to change the default formater?

Hi,
TinyTemplate looks ideal for a use-case I have.
The problem is, that I will not generate HTML, so the default formatter that does HTML escaping is a big problem for me (I don't want to add " | unescaped" to every value usage).
Is there a way to change the default formatter?
I looked at the source code and it seems this is not possible. But I'm a Rust beginner, so maybe I'm missing something.

This is not currently supported, no. It probably wouldn't be too hard to add though, and I suppose the extra flexibility is probably worth the complexity cost of adding one extra setting.

With the merged PR above, I think this can be closed


Add ERA5 Pressure Levels Monthly Avgs

Checklist:

[x] I created a new folder $artifact_name

[x] I added a README.md in that that folder that

[x] describes the data and processing done to it
[x] lists the sources of the raw data
[x] lists the required citation, licenses

[x] If applicable (e.g., for Creative Commons), I added a LICENSE file
[x] I added the scripts that retrieve, process, and produce the artifact
[x] I added the environment used for such scripts (typically, Project.toml
and Manifest.toml)
[x] I added the OutputArtifacts.toml file containing the information
needed for package developers to add $artifact_name to their package

[x] I uploaded the artifact folder to the Caltech cluster (in
/groups/esm/ClimaArtifacts/artifacts/$artifact_name)
[x] I added the relevant code to the Overides.toml on the Caltech Cluster
(in /groups/esm/ClimaArtifacts/artifacts/Overrides.toml)
[x] I added a link to the main README.md to point to the new artifact

Could you make some plots from the generated artifact so I can take a look whether they look reasonable?

Could you make some plots from the generated artifact so I can take a look at whether they look reasonable?
@szy21
https://caltech.box.com/s/enbprowqp3tfx0yjh9queqm9fm2syx8k


🐛 Fixed post scheduling for sites with transferred ownership

refs TryGhost/Team#3494

By default, the post scheduler runs as user_id = 1, which is the original owner of the site
It is possible to transfer ownership to another user, then delete the original owner, so some sites don't have a user_id = 1
With the addition of post history, we save the author_id from the contextUser, which is usually the logged in user
However when the scheduler runs and tries to publish a post, it sets the author_id to 1, which then fails the foreign key constraint on the post_revisions table
This commit fixes the issue by checking if the contextUser exists, and falling back to the author of the most recent post_revision if it doesn't
This is a bit of a hack/band-aid to fix the immediate issue, but ideally this check would be made in the contextUser function, so we can always be confident the contextUser exists without firing an ad-hoc DB query
Additionally, we should refactor to query for the current owner's user_id instead of defaulting to 1

I think using the previous revisions author is weird because it's going to provide different behaviour dependent on the state of the site
What about updating the this.contextUser method to:

Return a Promise
Return the Owner User id rather than 1

🤔

Agree with you that this would be a better solution, but I'm struggling to implement it properly...I just pushed what I have right now to this branch — this should work in theory, but it breaks a bunch of tests. I think the pattern of relying on '1' being the owner user is still pretty entrenched in the codebase, especially in the test framework.
I'm having a hard time untangling the circular dependency on the owner user — e.g. when we create the owner user in the fixture-utils, we rely on the owner user already existing https://github.com/TryGhost/Ghost/blob/82acf85b297279639a3c14a7b215d8e6d9c0ea2c/ghost/core/test/utils/fixture-utils.js#L225
Let me know your thoughts, struggling with this one a bit.


cobaltbunny

Since the update, I have not been able to log on
Screen says Connection Lost     Mod rejections [FMLMod:BiblioCraft{1.11.3}, FMLMod:evilcraft{0.9.0}]
I have already deleted mech & magic and reinstalled it in my instances. This made no change

I have the same issue.

Just because the pack updates does not mean the server has. Updates are not forced on the server and will not apply until the regularly scheduled restarts at 4, 8, and 12 EST.


Fixed #26112 -- Fixed aggregate GIS test on Oracle

This fixes the test -- but is it the right fix?
Is it OK in this case that Sum still aggregates over (single) records? This aggregation necessitates the unintuitive and expensive (causes a database round-trip per row later) defer() call.
Should the Area function be modified to return a FloatField on Oracle?

buildbot, test on oracle.

(didn't seem to listen before...)
buildbot, test on oracle.

buildbot, test on oracle.

3rd time was a charm.

Its true that the sum is not necessary in the test case, as there is no additional grouping. But I came across this bug when trying to do sum over grouped values, using something along the lines of
 Country.objects.all().values('continent').annotate(area_sum=Sum(Area('mpoly')))

To cover these situations, the Area function should work in combination with Sum using its standard output field. So I think the test should pass as it was and the Area or Sum functions need to be fixed.
I am not familiar with oracle, does the problem arise because Sum is called on an expression that only returns one item?

There are two problems: The easy one is that on Oracle, unlike other backends, Area() returns a Decimal instead of a float. I thought a little more about it, and I think we should not change that -- Oracle allows higher precision, it is not our place to bring it down to a lower common denominator. So for that, I now think that the solution in the PR is correct.
Per the Sum, I was for some reason under the impression that Area() here returns a value for each of the polygons in mpoly and it needs Sum to aggregate them (did I mention it was late at night?). Summing over one value is not an issue -- the issue is grouping over mpoly (when you call annotate() with an aggregate function like Sum, Django adds grouping over all the non-aggregate columns; the SQL standard requires that every column selected from a SELECT with a GROUP BY is either an aggregate or in the GROUP BY). Assuming now that Area() returns one value for the whole polygon array, again, I think the PR's solution is correct (we have similar cases involving grouping over text fields, where Oracle presents the same problem).

Your intuition about Area() was correct, it does return one value per polygon in mpoly, and the Sum function aggregates them. The test is probably not explicit enough to show that, as it does the aggregate only over one polygon, which is a bit artificial.
For the test to be more explicit, one could also do the following:
    for c in Country.objects.all():
        CountryWebMercator.objects.create(name='First', mpoly=c.mpoly)
        CountryWebMercator.objects.create(name='Second', mpoly=c.mpoly)
    # This should not crash
    CountryWebMercator.objects.values('name').annotate(area_sum=Sum(Area('mpoly')))

I suppose that the query above should also work in oracle as is, without having to override the output_field? If that's the case and it fails the Sum or Area should be changed, including keeping track of Decimals.
@shaib sorry for insisting, I guess I still don't understand why the query fails on oracle, hence my question.

@yellowcap no need to apologize, asking questions is good.
mpoly is a MultiPolygonField. It can hold multiple polygons in one row. My intuition yesterday was that in this case, Area returns an array in one row, and Sum would some over that. I don't believe that to be the case anymore; Area returns one number (per database row), even if there are many polygons in mpoly on that row.
W.r.t. Decimals: Your query would indeed work on Oracle as it is, without overriding the output_field (and mine did when I only added defer('mpoly')). But then, the test fails on computing result - c.mpoly.area, because c.mpoly.area is a float and not a Decimal, and these types don't mix.
Actually, when I wrote my last comment, I didn't realize the float in question came from the area property of the field value. This makes me reconsider; I would appreciate the opinion of people who are more familiar than me with the GIS functionality -- is there more value in exposing the full abilities of Oracle here, or in making Area() compatible with MultiPolygonField.area?

@jtiai, can you advise on this?

At least in Oracle area from multipolygon should result a single, combined area from all polygons in geometry. As a resulting values I think it's precise enough to return floats. In extreme cases when very big polygons would be used it might cause losing a precision but I guess in normal activity that won't be a  problem.

@claudep @jtiai following advice from @jtiai  I changed the AreaField conversion so it turns Decimals into floats -- this makes the areas returned from database "play well" with areas calculated on the Python side. Do you agree? Do you think it makes sense to do the same to DistanceField?
While at it, shouldn't we document these field types?

buildbot, test on Oracle.

There is no failure in the default test, Jenkins got confused and thought the py2.7/mysql-gis test was cancelled but in fact it was successful.

Does the test change sufficiently cover the change? At least a comment in the test seems warranted.

Comment added.

merged in bb51dc902dc760b6fd03b860335fd99f47e2e13d, thanks!

Thanks for committing, I was sort of waiting for input from the geodjango gurus about making DistacneField also return float instead of Decimal.

Sorry Shai for being unresponsive on this one. I admit that Oracle stuff doesn't thrill me. Making DistanceField a float might add some consistency to the backend.

@claudep your time is your own and it's certainly your choice where to spend it, no need to apologize there. Thanks for taking a look.


zrevrangebyscore in aioredis in different than in redis

Describe the bug
I had my code running with redis-py in the version sync.
I added asycn/await everywhere and replaced the import of redis with aioredis.
zrevrangebyscore after that started returning no results, empty list, despite with redis-py sync it was returning values correctly!
now my code was with async/await so I tried to use
from redis import asyncio as aioredis
And magically zrevrangebyscore is returning results as expected again.
So I think there is some difference between the version implemented in redis-py and aioredis. What is the difference?
Anyways I fixed using redis-py in his async version.
aioredis version 2.0.1
redis-py version 4.3.4
To Reproduce
use zrevrangebyscore in aioredis and redis
Expected behavior
working the same way between aioredis and redis-py
Logs/tracebacks
no errors just empty list

Python Version
$ python --version
3.10

aioredis Version
$ python -m pip show aioredis
2.0.1

Additional context
No response
Code of Conduct

[X] I agree to follow the aio-libs Code of Conduct

Same problem, this bug has not been fixed until version 2.0.1

According to README, aioredis is now in redis-py 4.2.0rc1+. So you should use redis instead of aioredis.
from redis import asyncio as aioredis

But there seems to be a bug in pycharm, it warns that Cannot find reference 'asyncio' in '__init__.pyi'


EfficientNet example notebook

Description
Added an example notebook for EfficientNet-B0 image classification model.
Type of change
Please delete options that are not relevant and/or add your own.

New feature (non-breaking change which adds functionality)
This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project (You can use the linters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[x] I have made corresponding changes to the documentation
[x] I have added tests to verify my fix or my feature
[x] New and existing unit tests pass locally with my changes

Also please sign your commits

I've listed the notebook and signed my latest commit. Would that be good to go?

You need to sign all commits. See https://github.com/NVIDIA/Torch-TensorRT/pull/709/checks?check_run_id=4214564245 for instructions

Signed all commits and fixed the ".ipynb" typo.

When I signed off my previous commits using DCO's suggested command - git rebase HEAD~5 --signoff - I think it signed some of your commits as well:
Applying: add EfficientNet example notebook
Applying: updated documentation
Applying: updated preproc
Applying: chore: Bump version
Applying: fix: Update base container for dockerfile
Applying: fix(//py): Constrain the CUDA version in container builds
Applying: refactor: Update the CI deps
Applying: chore: update the container
Applying: Update network.py
Applying: Update elu_model.py
Applying: test: Add a test for the global compile function
Applying: test(//py): Test nn.module compilation
Applying: refactor: CI updates
Applying: refactor: Try setting permisions for job
Applying: refactor: Apply linting
Applying: Update linter.yml
Applying: fix: Missing log severity
Applying: Update README.md
Applying: fix(//cpp/bin/torchtrtc): Fix mbs
Applying: chore(//docsrc): Update docs deps to fix docgen
Applying: updated index.rst```

Would you suggest that I raise a fresh PR instead?

yeah probably best to do that

I've created a new PR here - https://github.com/NVIDIA/Torch-TensorRT/pull/723 - and hence closing this.


Update adoption-standalone-*-provider irrelevant-files list

"adoption-standalone-to-crc-ceph-provider" job depends on a content-provider job to run but since it has a different list of irrelevant-files, Zuul can throw an error like: " adoption-standalone-to-crc-ceph-provider depends on openstack-k8s-operators-content-provider which was not run". This PR copies the missing irrelevant-files from content-provider job to this adoption job.
As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running

/lgtm
/approve


Add UPnP support (port forwarding, querying external IP)

This PR adds support for Universal Plug and Play (UPnP) to Godot's multiplayer capabilities.
This allows:

To forward ports automatically when hosting games behind a router, instead of requiring the user to do it themselves
To query the router for the external IP address

This is based on the BSD-licensed miniupnpc. The only change to the upstream files was an additional header file that would usually be autogenerated with cmake. In our case, I added it manually.
Changes

Adds miniupnc under thirdparty/miniupnpc
Adds Godot implementation under modules/upnp
Adds settings for module to scons

Simplest example
# Forward external UDP port 1234 on the router to us
var upnp = UPNP.new()
upnp.discover()
upnp.add_port_mapping(1234)
# Do something with the opened port, then on quitting:
upnp.delete_port_mapping(1234)

This autodetects the router and creates (or overwrites) the port mapping.
You can also customize the UPnP discovery params (by calling upnp.discover() yourself) and the way the gateway is chosen by using the methods on UPNP to iterate the UPNPDevices. Once you found the device of your choosing, you can then call add_port_mapping() and delete_port_mapping() on that device, again providing the parameters of your choice. The methods on UPNP are just shortcuts. :smile:
Of course, you can fine tune all relevant settings (like timeouts/ttl, internal and external ports, TCP vs UDP, lease time, description for the port forward etc.) from GDScript.
Glossary
IGD: Internet Gateway Device, the UPnP type for routers
NAT: Network address translation
UPnP: Universal Plug and Play, the protocol
Notes
Not all routers are UPnP compatible. Not all devices support wildcards in port mappings, or a non-infinite duration for the lease, or a description. Look at this list to get an idea of how varied UPnP support is. The default parameters as they are now should maximize compatibility.
Relevant to https://github.com/godotengine/godot/issues/13947.
Closes https://github.com/godotengine/godot/issues/2970.

It fails building on Linux and macOS:
modules/upnp/upnp.cpp:115:38: error: use of undeclared identifier 'malloc'; did you mean 'alloca'?
        struct UPNPUrls* urls = (UPNPUrls *)malloc(sizeof(struct UPNPUrls));
                                            ^~~~~~
                                            alloca
/usr/include/alloca.h:32:14: note: 'alloca' declared here
extern void *alloca (size_t __size) __THROW;
             ^
modules/upnp/upnp.cpp:122:38: error: use of undeclared identifier 'malloc'; did you mean 'alloca'?
        struct IGDdatas* data = (IGDdatas *)malloc(sizeof(struct IGDdatas));
                                            ^~~~~~
                                            alloca
/usr/include/alloca.h:32:14: note: 'alloca' declared here
extern void *alloca (size_t __size) __THROW;
             ^
modules/upnp/upnp.cpp:133:2: error: use of undeclared identifier 'free'; did you mean 'DefaultAllocator::free'?
        free(xml);
        ^~~~
        DefaultAllocator::free
core/os/memory.h:68:29: note: 'DefaultAllocator::free' declared here
        _FORCE_INLINE_ static void free(void *p_ptr) { return Memory::free_static(p_ptr, false); }
                                   ^
3 errors generated.

Shouldn't you use the Godot memory management methods (memnew, memfree, memalloc)?

Yes, I should. :) faless just pointed them out to me too.

@akien-mga Fixed the build, was missing an include. Can't use Godot's memory management functions as the memory in question is either allocated by the library or freed by it.

Added docs.
Will see about adding support for PCP and NAT-PMP in a seperate PR (using the BSD-licensed https://github.com/libpcp/pcp).
Then I'd like to add a "PortForwarder" Godot node on top of these that manages port forwards.
You'd add that to your multiplayer/lobby scene, and when it initializes it would automatically forward the ports according to the nodes configuration done in the editor, then free the ports once the scene exits.
This node would also automatically handle threading.

Only thing missing was to link against system miniupnpc on linuxbsd if builtin_upnpc=no, but I'll add that :)

And also missing adding the license details to COPYRIGHT.txt. I guess we should add this CODEOWNERS file and put me for thirdparty/* ;)

Added docs.
Will see about adding support for PCP and NAT-PMP in a seperate PR (using the BSD-licensed https://github.com/libpcp/pcp).
Then I'd like to add a "PortForwarder" Godot node on top of these that manages port forwards.
You'd add that to your multiplayer/lobby scene and configure the ports you wish to forward.
It would then automatically forward those when the scene is run and remove the mapping on exit, handle threading and use PCP/NAT-PMP and fallback to UPnP if those didn't work.
You can of course also use the UPNP/UPNPDevice classes directly, if you want to do something custom. You just need to handle threading yourself or live with blocking, which may be fine if done at startup/shutdown or during loading.
A little like HTTPRequest vs HTTPClient.

Are there currently any plans of adding a "PortForwarder" node for version 3.2 (or 4.0)?


Add General link in UG

Codecov Report

Merging #277 (6c53890) into master (3f33e81) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #277   +/-   ##
=========================================
  Coverage     80.42%   80.42%
  Complexity      988      988
=========================================
  Files           146      146
  Lines          2840     2840
  Branches        338      338
=========================================
  Hits           2284     2284
  Misses          478      478
  Partials         78       78

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 3f33e81...6c53890. Read the comment docs.


update bindgen and examples

@davechallis @jonathanstrong Hi.
This PR what makes the latest bindgen.
In addition, I'm modifying the examples.

include #14


All the author's stories are not downloaded completely

If I try to download all the author's stories the app doesn't download all of them.
Example: MountainDewMan
https://www.literotica.com/authors/MountainDewMan/works/stories
105 stories
Downloaded 53

cc486407d7b27246580e7fe6a62de974a2496179
Should be fixed now. Version 1.0.9 published.


Descriptions are not mapped

https://github.com/american-art/SAAM/blob/master/Objects/WebObjDescription.csv appears to have object descriptions, but they do not appear in the RDF.

Included.


Skylar-discussion-survey

Places video call and topic survey side-by-side in discussion stage

I'm not sure if this can really be considered finished yet. The discussion stage has the survey and video call window side by side now, but we are not testing for the survey in the discussion room in our cypress tests yet. I also don't think the form submission button works as intended but if we are moving away from SurveyJS I don't think this is worth fixing.

I think this was addressed in another PR - can we close this @sky1113 @bsmith6202?


refactor: replace critters with beasties

The Critters project has been transferred to the Nuxt team, who will now manage its development and has been renamed to Beasties.
See: https://github.com/danielroe/beasties

The lint failure is for<EMAIL_ADDRESS>– License: CC-BY-3.0 AND MIT which both should be valid license.

The changes were merged into the following branches: main, 19.0.x


download failure

using zopen download bash I get this

Can you paste output from zopen download bash -v

Looks like it worked this time
/u/lbdyck/zot/meta>zopen download bash -v
VERBOSE: 'Download directory: /u/lbdyck/zopen/prod'
Preparing to download bashport
Downloading latest release from bashport...
VERBOSE: 'curl -L https://github.com/ZOSOpenTools/bashport/releases/download/bashport_433/bash-5.2.20230128_073616.zos.pax.Z -O '
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                             Dload  Upload   Total   Spent    Left  Speed

0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 4662k  100 4662k    0     0  4844k      0 --:--:-- --:--:-- --:--:-- 5804k
Extracting bash-5.2.20230128_073616.zos.pax.Z...
VERBOSE: 'pax -rf bash-5.2.20230128_073616.zos.pax.Z -p p '
Successfully downloaded bash to /u/lbdyck/zopen/prod/bash/
/u/lbdyck/zot/meta>
Lionel B. Dyck <><
Website:  https://www.lbdsoftware.com https://www.lbdsoftware.com
Github:  https://github.com/lbdyck https://github.com/lbdyck
“Worry more about your character than your reputation. Character is what you are, reputation merely what others think you are.”   - - - John Wooden
From: Igor Todorovski @.>
Sent: Monday, January 30, 2023 10:04 AM
To: ZOSOpenTools/bashport @.>
Cc: Lionel B. Dyck @.>; Author @.>
Subject: Re: [ZOSOpenTools/bashport] download failure (Issue #40)
Can you paste output from zopen download bash -v
—
Reply to this email directly, view it on GitHub https://github.com/ZOSOpenTools/bashport/issues/40#issuecomment-1408883689 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKC6CWYS2IIVTKN5E7LIBD3WU7Q5LANCNFSM6AAAAAAULE44RU .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AKC6CWZ5DZ2TWJ4KXGOMZMTWU7Q5LA5CNFSM6AAAAAAULE44RWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTST7HN6S.gif Message ID: @.*** @.***> >

I reran it and got this:
[ITODORO@ZOSCAN2B ~/zopen/prod]$ zopen download bash -v
VERBOSE: 'Download directory: /home/itodoro/zopen/prod'
Preparing to download bashport
Downloading latest release from bashport...
VERBOSE: 'curl -L https://github.com/ZOSOpenTools/bashport/releases/download/bashport_433/bash-5.2.20230128_073616.zos.pax.Z -O '
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 4662k  100 4662k    0     0  8536k      0 --:--:-- --:--:-- --:--:-- 8536k
Extracting bash-5.2.20230128_073616.zos.pax.Z...
VERBOSE: 'pax -rf bash-5.2.20230128_073616.zos.pax.Z -p p '
pax: FSUM7148 cannot create file "bash-5.2.20230128_073616.zos/bin/bashbug": EDC5111I Permission denied.
***WARNING: Could not extract bash-5.2.20230128_073616.zos.pax.Z. Skipping

Looks like pax doesn't like to override read-only files:
ls -l bash-5.2.20230128_073616.zos/bin/bashbug
-r-xr-xr-x   1 ITODORO  BASIC       6956 Jan 28 08:19 bash-5.2.20230128_073616.zos/bin/bashbug

looks like that may have been the issue for me

I've added a fix, thanks for bringing it up


ios: generate reactTag for remote streams/tracks

Will generate react tags for remote streams and tracks in the form of
{RTCPeerConnection react tag}_{stream id|track id}
This allows to receive streams/tracks with the same IDs through different PeerConnections.
I also opened this one (by mistake):
https://github.com/oney/react-native-webrtc/pull/261
Not sure which way is preferred ?

Let's continue with the upstream one, since that's were the patch will end up.

I'm closing this PR here because @saghul appears to prefer the equivalent https://github.com/oney/react-native-webrtc/pull/261 and it doesn't make sense to me to have 2 PRs.


[DLR] Remove Neo-dlr engine from DJL

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report
Base: 72.08% // Head: 74.44% // Increases project coverage by +2.35% :tada:

Coverage data is based on head (825f033) compared to base (bb5073f).
Patch coverage: 74.14% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2373      +/-   ##
============================================
+ Coverage     72.08%   74.44%   +2.35%
- Complexity     5126     6770    +1644
============================================
  Files           473      658     +185
  Lines         21970    29400    +7430
  Branches       2351     3052     +701
============================================
+ Hits          15838    21888    +6050
- Misses         4925     6027    +1102
- Partials       1207     1485     +278

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
67.10% <0.00%> (+7.89%)
:arrow_up:

api/src/main/java/ai/djl/modality/nlp/Decoder.java
63.63% <ø> (ø)

... and 624 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix sqlalchemy query error of test_postgres.py (Fix #538)

Description of the problems or issues
Is your pull request related to a problem? Please describe.
See #538
Does your pull request fix any issue.
See #538
Description of the proposed changes
Fix sqlalchemy query building on tests/test_postgres.py::test_cand_gen_cascading_delete.
Before this fixing, this test code tried delete query for deleting one data from PartTemp table, but sqlalchemy create unexecutable query to refer candidate.id from PartTemp table although this query does not use candidate.id.
Fix is changing query builder codes not to refer candidate.id.
Test plan
Execute tests/test_postgres.py::test_cand_gen_cascading_delete.
Checklist

[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] I have updated the CHANGELOG.rst accordingly.

This fixs issue #538 and pytest on github was successful. However, the error occurs in the process of "test (macos-latest, 3.6)";
Warning: Cache service responded with 503 during chunk upload.
events.js:187
      throw er; // Unhandled 'error' event

Can I fix it myself? Please support.

This fixs issue #538 and pytest on github was successful. However, the error occurs in the process of "test (macos-latest, 3.6)";
Warning: Cache service responded with 503 during chunk upload.
events.js:187
      throw er; // Unhandled 'error' event

Can I fix it myself? Please support.

Please fix it. Thanks!!!

@YasushiMiyata I just re-ran the jobs and it appears to have passed. I don't think that error was caused by you.

@SenWu , @lukehsiao Thank you for checking and your comments. I also found my request passed.
The previous error may be something temporary unavailable on network.

Codecov Report

Merging #539 (642389f) into master (433a75d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #539   +/-   ##
=======================================
  Coverage   86.02%   86.02%
=======================================
  Files          92       92
  Lines        4773     4773
  Branches      899      899
=======================================
  Hits         4106     4106
  Misses        476      476
  Partials      191      191

Flag
Coverage Δ

unittests
86.02% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


How to deploy a Django app with the django rest framework

I get this error when trying to deploy a Django rest framework api.
What do I need to do with Cloud Formation or is it possible to use Zappa to deploy a Django rest framework?
..Stack creation failed. Please check your CloudFormation console. You may also need to undeploy``

Is this with master or 0.26.0? Are there errors in your CloudFormation logs?


Turbines with regular steam can create infinite energy

Description
Using other mods' steam sources with nuclearcraft:overhaul's turbines, I can easily create a infinite energy source.
Here are some numbers that I grab from my pack, which can tell you how imbalance this is

The Steam to RF multiplier for default NC is 4rf/mb
With a proper design, it can easily become 4.2rf+/mb

Magneticraft can turn RF into Steam using it's RF Heater, I built a simple example to show this
The RF to Steam ratio for him is 2rf/mb

For GTCE, 1eu/mb or 4rf/mb
For Mekanism's Fusion Reactor or Steam Boiler...
Suggestion
Nuclearcraft's Rankine cycle is different from others, so I suggest just don't register it's steam in forge changing to other id like "nc_steam"
There isn't a proper standard for how much Steam to produce how much RF, which has been a problem since Power Converter Mod

This is in my opinion a balancing problem of the modpack

Resolved in discord chat, seems this problem doesn't come's from nc's balancing


How do i get the place id from the autocomplete?

Currently using the autocomplete to find locations but I need to also get the place ID. I see it in the network response when selecting a result in the autocomplete but I'm not sure how to get that data.

@sebastienb  Hi, Can you share the response please?

I was able to get it using the response. this.$refs.input.$autocomplete.gm_accessors_.place.Kj.place.place_id;


Cypress projects kernel dependency fix.

Description
In previous releases, we "accidentally" committed a kernel port in kernel third party directory. This is unnecessary since third party port is essentially the same as GCC/ARM_CRx_No_GIC. In this kernel release, we removed the redundant third party port. And in this PR, I'm trying to make the projects take dependency on the correct kernel port.
Additional compiler flags are added for floating point related. Refer to https://gcc.gnu.org/onlinedocs/gcc/ARM-Options.html.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will reach out Cypress later in regards to the fix proposed in this PR.


Could use remote gazebo  possible?

Brief
I'm a Gazebo beginner, and I'm currently using ROS 2 control on an NVIDIA Orin platform. Since my remote setup is on an NVIDIA Orin, I'd like to run the control algorithms there. My question is, I want to run Gazebo on my local machine while running ROS 2 code on the Orin, and establish a connection between them using ROS 2 topics (they are on the same local network). Could you please advise if there's a way to achieve this functionality?
Situation

Remote: Nvidia AGX Orin (ros2 control and other algorithms)
Local: Gazebo
We use issac ros in docker.All the ros2 pkg have to build by self.I just want to test the code. I believe this approach would be quite elegant.

What is the purpose of having ros2_control on the orin but gazebo on the local machine? you want to PIL-test your code, i.e., instead of controlling your hardware it should control your model in gazebo?

As this is not really related to this repository but a general questions I ask you to post your question on RSE, possible answers will be found more easily in the future.


doc: fixed some minor styles in README.md to improve readability

Proposed changes
Just some minor markdown changes to improve readability.
Type of change:

[ ] Bug fix (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)
[ ] This change requires a documentation update

Required Checklist:

[x] I have signed the MongoDB CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[ ] I have run make fmt and formatted my code
[ ] I have tested the CDK constructor in a CFN stack. See TESTING.md
[ ] If changes include removal or addition of 3rd party GitHub actions, I updated our internal document. Reach out to the APIx Integration slack channel to get access to the internal document.

Further comments

thanks @svennergr for your contribution.
I've created a new PR so I can run all PR checks: https://github.com/mongodb/awscdk-resources-mongodbatlas/pull/298
I've cherry-picked your commit so you'll appear as a co-author.
thanks again


Phone Numbers

How do you recommend formatting phone numbers, both for the United States and international?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6f47e95a-2a10-bf53-6518-419ecb8ac3d3
Version Independent ID: e42e5d8a-dcf2-872e-2786-c9ddde6e47d5
Content: Numbers - Microsoft Style Guide
Content Source: styleguide/numbers.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

Unfortunately, the Microsoft Writing Style Guide doesn’t yet have guidelines for phone number formats, though it’s something we might add in the future.


Re-enable Chrome for travis builds

Looks like the Chrome bug (or karma issue?) that prevented them from running is now fixed!

Thanks for the pull request @mramato!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@OmarShehata can you hit merge on this?

Looks good to me, thanks Matt!


go module suport

i'm plan to add go.mod with vendored deps, and use it with go module in go-ovn

fix in #4

after merged related pr need to release new package version for go.mod in go-ovn


[BULK] validation: disallowed-html-tag

Category

[x] Content fix
[ ] New article

Related issues
None.
What's in this Pull Request?
Escape angled brackets.
The Content & Learning team is fixing build validation errors on docs.microsoft.com.

Docs Build status updates of commit 4ff4d6c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/customizing-ranking-models-to-improve-relevance-in-sharepoint.md
:white_check_mark:Succeeded
View

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

Global effort to fix validation errors
The Content & Learning team is fixing build validation errors on docs.microsoft.com. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. I’d very much appreciate your timely review and feedback on these changes I’ve suggested. Thanks!


Update trade_api.py

XRP support pair

Closing this in favor of https://github.com/alfakini/python-mercadobitcoin/pull/13


Code Review - Faster Updates and Undo option

By using S3 to store user uploaded images, and redis cache to memoize an stack of actions (operations) applied on the image, the speed was significantly increased, as well as opened up the option to undo an action.
An action is a dictionary of zone selected, type of operation, and the operation itself.
Please view this link for the dev version: https://dash-image-processing-dev.herokuapp.com
Please view this one for the current master version (OLD): http://dash-image-processing.herokuapp.com
If you want to test this locally, please let me know and I will send you the .env file

@shammamah @matthewchan15 any update?

Ok, made the code length PEP8 conform for everything!

@shammamah

@shammamah Let me know if there's anything else to update; I'll try to add my new apps to the gallery on monday.


fix(modeling): boolean argument checks

A user on discord had a question, and I realized that his design revealed a weird behavior in union when invalid geometries are given. Simplified user code:
return union(vectorText({ input: 'I' }), vectorText({ input: ' ' }))

Rather than throwing an error in union, this actually returns the value 4. Weird.
It makes sense when you look at the code. vectorText returns an array of points like:
vectorText({ input: 'I' }) => [[[4, 21], [4, 0]]]

Which gets flattened by union, and then union doesn't know how to deal with non-geometries so it just returns the first element from the flattened list which is the number 4. Very weird.
I don't like throwing errors if we can avoid it. But in this case, it feels like the right thing to do is to tell the user "hey this is not a valid geometry" as soon as possible, to make debugging easier. So in this case I added throw new Error('union unsupported geometry type')
I added tests for this behavior. I also tried to make the error messages easier to understand and with better grammar, but open to suggestions here.
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Does your submission pass tests?

Trash in trash out. I'm surprised that the example didn't crash.

Also,  V3 vectorChar and vectorText return different results now. Are you sure this still happens with V3?

The specific example here would probably not happen with V3 vectorText because you changed it, but the behavior would remain kind of weird. If you have the above example in V3, since union doesn't support path2, I think it would just return the FIRST text but discard the second text.
I think this PR would still be better in that case so that users know that we don't support unioning paths (yet).

@platypii these changes seem reasonable. but there's an issue raised in issue #820 which seems both reasonable and possible. So, I'd really like to seem some changes for...

how to handle empty lists in the API
how to handle non-supported types in the API
how to handle sparse lists in the API

@z3dev I am happy to make some suggestions for empty lists, sparse lists, and invalid geometries in general. I have some ideas.
But I would prefer to put up a separate PR after these changes are merged, if you agree with these changes. I can have another PR up with my suggestions for the rest of the cases in the next few days. I think some of the changes for sparse lists might warrant their own discussion, and I don't want this PR to get huge.


anaconda3: Update to version 2023.03-1 and fix checkver

[X] I have read the Contributing Guide.

/verify


🚧 Dev ➾ Add Support to taq scaffold for Checking-Out a Specific Branch of a Scaffold Repo

🔗 Epic|Feature Link
No response
🚥 Definition of Done

[X] 🎯 Planning and Design
[ ] 🏎 Dev Implementation (PR merged)
[ ] ✍ Release Notes Added
[ ] 🏆 Product Acceptance

✅ To Do
No response
⚖ Acceptance Criteria
✨ Functional:

[ ] taq scaffold has the means to provide a branch name which will check-out the associated branch on the scaffolded project
[ ] Providing an incorrect branch name fails gracefully with a helpful error message

📐 Non-Functional:

[ ] Additional steps are minimized for the user
[ ] Existing functionality of taq scaffold is unchanged

🔭 Scope
🙅♂ Out of Scope:

Direct work on scaffold projects

🧰 Work Description
Currently, there is no way to scaffold a project from a branch other than main. While this is effective for end users, it makes testing the effects of (pre)releases challenging
As part of the release process, required changes to scaffolds are staged in a branch on the scaffold repo's. The intention is that as soon as a release goes out, the branch can be merged and Taqueria will have a consistent version and behaviour throughout the product
In order to support this flow, we need an easy way to do manual testing on un-released changes to scaffolds. To support this, we need to implement the ability to check out a particular branch of a scaffold project. This will greatly simplify the testing/validation process and increase the overall quality of Taqueria as a product
🦄 Sizing
1
Code of Conduct

[X] I agree to follow this project's Code of Conduct

What would be really cool is if this feature automatically updated all installed plugins to the branch-specific versions implicitly
Thought @jevonearth or @mweichert ?

BLOCKED BY: #1342


Fix for cd system

Use OPENWRT_VERSION when BUILT_VERSION is not set
#85

cc https://github.com/OnionIoT/OpenWRT-Packages/issues/85

@YevheniiOnishchenkoLemberg can you please also add a comment to the file indicating what the change is for and make a new PR?


Blender 2.80

Are there any plans on updating the plugin for Blender 2.80?
It says "upgrade to 2.8x required"

I'll have to look into it, hopefully the changes aren't too drastic.

They don't look too drastic, mostly how you register classes.

@kimamb here is a pre-release that should support Blender 2.80.
I've not tested everything, so please let me know if you run into any issues.

I can't seem to find the modifier, have you moved it?

Never mind, it's now under "Misc".


fix: Updating AWS support details

Description

Removing outdated content on support for AWS workloads

Related Issue(s)
(https://github.com/konstructio/kubefirst-docs/issues/793)
The other issues noted here ☝ have been resolved in the docs with the addition of the following pages
https://kubefirst.konstruct.io/docs/aws/advanced/multi-account
How to test

@muse-sisay  @jokestax If either of you can confirm this update addresses the current support for AWS (reference the linked issue) I can merge this to update 👍

Thank you @konstructcarrie :heart_hands:


huggingface models as client?

Checklist

[x] I've searched the project's issues.

❓ Question
Is there any good example of how to use huggingface models and eventually perform giskard scan
How can I [...]?
📎 Additional context

https://github.com/Giskard-AI/giskard/blob/main/giskard/models/huggingface.py
check this out


Initial(default) Expanded item in ExpandedTileList.builder

The initial expanded item is not showing when using it with ExpandedTileList.builder
controller: index == 1     ? controller.copyWith(isExpanded: true)     : controller,

Also I can confirm that  controller.copyWith(isExpanded: true) is not working, any fix?

yeah I also faced this issue

Closed and updated in version 0.4.0
Sorry, I haven't been active lately.
Thank you for your support!


Improve web documentation and document styles

This PR has two focuses:

Adds documentation about how to customize style for the web, based on a recent conversation I had with @rowanc1
Moves our web docs to a new web/ folder and enables folder-based URLs for the MyST guide. I didn't do this for all docs because I first wanted to see how this looked to folks before suggesting we do it with other sections in a follow-up PR.

@agoose77 I see a bunch of updates in here - can you merge this whenever you like? Alternatively I will merge this PR the next time I think about it if I don't hear otherwise.


Stream going black and resetting

I just added a Wyze v3 cam using camera.ui for a short time it was recording using hksv Then it just stopped. I removed the camera and added it back. Now every 12 seconds the preview goes black and then comes back. This only happens on the home app main screen. I am attaching a video. It was not doing this before.
I also have a Reolink camera and that is not behaving the same way.
I'm new to camera UI and must say it is great so far. Just need some help to iron out some glitches.
Thanks.
Environment

Node.js Version: v16.15
Homebridge Version: v1.4.1
Camera UI Version: v5.0.27
Operating System: Raspbian GNU/Linux Buster (10)
Browser: safari

https://user-images.githubusercontent.com/47582187/167977648-619e9820-2592-49fa-96b4-3e8ef743bf3d.MOV

Hey @vascott1
thats not a camera.ui issue, its a apple homekit bug (which is fixed in the current beta i think (?))

That’s good news actually. Thanks. Was hoping it could be fixed but I can work with it until the next release.
Thanks for the quick answer.

On May 11, 2022 at 11:27 PM,  <seydx @.***)>  wrote:
Hey  @vascott1 (https://github.com/vascott1)
thats not a camera.ui issue, its a apple homekit bug (which is fixed in the current beta i think (?))
—
Reply to this email directly,   view it on GitHub (https://github.com/seydx/homebridge-camera-ui/issues/550#issuecomment-1124493075), or   unsubscribe (https://github.com/notifications/unsubscribe-auth/ALLAX23VX27ZGWALIZFHXZTVJR3AZANCNFSM5VWTNVCQ).
You are receiving this because you were mentioned.Message ID:  @.***>


SW-4682 Use exact-or-fuzzy search for batch numbers

If the user enters an exact batch number or species name in the nursery inventory
batches view, only show the batch with that exact number, the same way we do with
accession numbers on the accessions page.

#2702  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sgrimm and the rest of your teammates on  Graphite


PDFs have duplicate image data

Background images (parchment texture, text block borders, etc.) are fully duplicated for each occurrence in the resulting PDF. This results in large file sizes (and potentially browsers crashing) as pages increase.
I know PDFs can be made to re-use image data multiple times from the same resource, which unfortunately Chrome's print-to-pdf doesn't do. If there's an alternate PDF generator or post-processing step we could use to detect and remove identical image data that would be just nice to have. See #201

This might not be a Chrome thing but rather a system/OS thing (i.e. chrome might well be handing off to the OS to create the PDF). That seems to be the thing happening with MacOS. And thus different results for different users.
That said, I do wonder if declaring the images via base64 inlines (and not via a distinct resource URI) is causing this duplication.
We should look to figuring out how to generate the PDF on the server and sending that as the response to a resource request of the form https://homebrewery.naturalcrit.com/pdf/:id
That is, leave the /print/:id route as is [for backwards compatibility and 3rd party links] but change the  menu-bar link to /pdf/:id — it'll also help with related PEBKAC support issues too)

We should look to figuring out how to generate the PDF on the server and sending that as the response to a resource request of the form https://homebrewery.naturalcrit.com/pdf/:id

A user was asking about PanDoc compatibility in the sub-reddit recently. I looked at it (very) briefly; it can convert Markdown to PDF via LaTeX (see example 13 here). Custom formats can be written in Lua, if the HB Markdown is not sufficiently close to the existing Markdown formats.
Just to be clear: I'm absolutely behind shifting to producing the PDFs in-house, as it were, rather than relying on the user's ability to Print to PDF. It will be a big move though, as there are bound to be billions of tiny bugs where the Chrome webpage does not exactly match the rendered PDF file.
However, I suspect that it's probably big enough to need not just it's own separate issue, but probably an EPIC tag to go with it.

The other side of the PanDoc conversation is that it feels an awful lot like using a sledgehammer to crack a single nut - Homebrewery doesn't need all of the bells and whistles PanDoc provides.

Here's the Chrome issue for this for future reference. Give it a star to bring more attention!
https://bugs.chromium.org/p/chromium/issues/detail?id=1196526&q=background-image pdf&can=2

Given that this is a Chrome bug, do we need to keep this open?

Added this to the larger Chrome Bugs issue. Can close this now.


Fix AffGameSnakeJs

TPS is on an old package set that does not include polymorphic-vectors.
try

For reference: we'd need version psc-0.13.8-20200831.
(also missing canvas-action and game but those were added in the same release)


Adding NCI and UMLS mappings

Hi Oncotree team,
This is Iker Huerga from MSK (Information Systems).
I had to generate mappings between oncotree, NCI thesaurus and UMLS as part of a project I was working on. I just finished and thought this might be useful for you guys as well.
This pull requests contains the followings additions

Two extra columns (nci, umls) to the tumor_tree.txt file; and the corresponding CUI (Concept Unique Identifier) for each of the controlled vocabularies.

Changes into tree.js to populate the links into the 'qtip' element. Thus, everytime the user mouse hover a node in the tree there are two new elements (NCI, UMLS) added into the pop-up. Each of them contain a clickable link to the corresponding controlled vocabulary

A file called tumor_tree.rdf. This is an RDF (Resource Description Framework) representation of oncotree taxonomy as well as all the mappings to NCI and UMLS.

Please don't hesitate to reach out in case you have any questions, or need some clarification, my MSK email is<EMAIL_ADDRESS>Hope you find this useful
Iker
P.S: I added the jetty plugin to the pom.xml to make it easier to debug in your local machine. If you are using maven just run the following command from your favorite IDE or command line
mvn -Djetty.http.port=9999 jetty:run

Forgor to mention that I manually reviewed all the mappings one by one

@ihuerga Thank you so much for doing this. I think the tumor type text file has been updated and we need to update your pull request as well. Since this is ongoing project, the text file will be change in the future, how should we keep updating all NCI and UMLS?

There are two options I can see moving forward to update the mappings
1- I'd be happy to take care of updating the mappings everytime the tumor
tree changes. I will simply make a pull request with the updated mappings
after every update to the tumor tree
2- We can set up a collaborative ontology development/management tool, such
as web protege. I'd be happy to set it up in one of our VMs. There we can
do version control on the tumor tree and their corresponding mappings.
I personally think option 1) is more short term (I can start doing that
right away if you guys are ok with it), whereas option 2) is more longer
term
Let me know your thoughts
Thanks
Iker
ᐧ
2015-11-30 12:20 GMT-05:00 HongxinZ<EMAIL_ADDRESS>
@ihuerga https://github.com/ihuerga Thank you so much for doing this. I
think the tumor type text file has been updated and we need to update your
pull request as well. Since this is ongoing project, the text file will be
change in the future, how should we keep updating all NCI and UMLS?
—
Reply to this email directly or view it on GitHub
https://github.com/cBioPortal/oncotree/pull/7#issuecomment-160694731.

--
Iker Huerga
http://www.ikerhuerga.com/

@ihuerga Any updates on this pull request?


Jesse: Watch poppidrink.com

Jesse requests the watch of the watch_keyword poppidrink\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

poppidrink\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


[$1000] Web - Keyboard shortcut is not working when dropdown is open from default currency in workspace setting

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

Action Performed:

Open the website
Settings > workspaces
select a workspace from the list
click setting
click default currency
try to use the keyboard shortcut (CTRL + J, CTRL + K)

Expected Result:
keyboard shortcuts should work
Actual Result:
keyboard shortcuts are not working
Workaround:
Unknown
Platforms:

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

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

Version Number: v1.3.56-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/ab270aca-6e68-4592-b8a1-9543b33d95d2
https://github.com/Expensify/App/assets/115492554/08cb378e-3326-4202-8938-ba1024e5d647
Expensify/Expensify Issue URL:
Issue reported by: @misgana96
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1691837136904329
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01cd1df93448bb319c
Upwork Job ID:<PHONE_NUMBER>853365248
Last Price Increase: 2023-08-28

asked clarification here: https://expensify.slack.com/archives/C049HHMV9SM/p1692898031791459?thread_ts=1691837136.904329&cid=C049HHMV9SM

I think this might be not solvable. I tried to add eventlistener to the select element here. Whenever we have select opened it's not listening for any event atleast for me.

This is not exactly a bug. When you have select menu focused, it captures all keyboard events to interact with the select menu. It 's a browser limitation, The select menu is rendered outside of the document boundary. Using a custom select menu like the one used in Request money would solve this "issue".

@rushatgabhane any ideas here? ^^

^ I agree!


Ionic Slide keyboard overlapping issue

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ ] 3.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Expected behavior:

Steps to reproduce:

Related code:
insert any relevant code here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

The issue is empty?

Hello! While we appreciate you opening an issue with us i am going to close this issue and ask that you open a new one that has the issue template filled out please. Thanks for using Ionic!


Check if this is ok?

Please check
<file>./path/to/directory</file>
If you are linting only php this is not necessary
<exclude-pattern>*.min.js</exclude-pattern>
<!-- Disable Google Closure Linter for JS -->
<rule ref="Generic">
<exclude name="Generic.Debug.ClosureLinter" />
</rule>

Please double check
<!-- We work with modified VIP standard -->
<rule ref="WordPress-VIP">
<exclude name="WordPress.VIP.FileSystemWritesDisallow" />
<exclude name="WordPress.VIP.RestrictedFunctions" />
<exclude name="WordPress.VIP.RestrictedVariables" />
<exclude name="WordPress.VIP.SuperGlobalInputUsage" />
<exclude name="WordPress.VIP.ValidatedSanitizedInput" />
<exclude name="WordPress.VIP.DirectDatabaseQuery" />
<exclude name="WordPress.XSS.EscapeOutput" />
<exclude name="WordPress.PHP.YodaConditions" />
</rule>

I've made changes, will test to see if it's ok.

This should be fixed now.


The dataset

Please, I want to ask you about the .seg files, how can I convert my dataset to be with this extension?
Thanks,

I have no conversion utility code, you have to implement it yourself.


Independent configurations for PageRootProvider (Compat/6.0)

It corresponds to #1087 for AEM 6.0 and 6.1.

@justinedelson @davidjgonzalez
QA'd this PR merged into latest compat/6.0 branch on AEM 6.1

Tested with a fully functional demo site that depends on Page Root Provider
Tested an existing Page Root Provider config (now deprecated)
Tested deploying multiple Page Root Providers the new way
Tested mixing new Page Root Providers with the deprecated config
Tested adding/editing/removing Page Root Providers via Felix console

Everything works as expected.
In my opinion this is good to be merged.

@daniel-lima and @HitmanInWis - LGTM but can you do a quick change to make LOG -> log per the proj guidelines? Otherwise we can merge!

@davidjgonzalez , should we just rename "LOG" to "log"? Or should we replace
private static final Logger LOG = LoggerFactory.getLogger(PageRootProviderMultiImpl.class);
with
private final Logger log = LoggerFactory.getLogger(getClass());
Thanks!

The former is fine; most loggers in the proj are static.

@davidjgonzalez: done. Thanks!

@HitmanInWis , thank you for your effort reviewing and QA'ing everything!


Reproducer for issue #521 (issue not present since Alpha4)

Description

Related Issue

Motivation and Context

For this test to work properly with DataNucleus, the entity classes need to be top level classes in src/main/java that are registered in the src/main/resources/META-INF/orm.xml.
I suggest you name the entities something like SecondaryTableEntityBase and SecondaryTableEntitySub and put them into the package where the other entities are.

Allright! Didn't want to do that directly, because it would potentially cause all tests to fail rather than just the reproducer, but it should just pass now anyway.

Yeah it passed for Hibernate 5.2 and EclipseLink already, so it's probably fine.

I've moved the files to src/main/java. It however doesn't seem to pass locally, I think I missed something in persistence.xml but I can't seem to find it.

Looks fine to me. Let's see what Travis CI reports.

Ahh, the test class needs to specify the entity classes it uses by providing them via an overridden getEntityClasses. These classes are registered in the persistence unit. The orm.xml is just for enhancers to know which classes to enhance.

Fixed 😄 Thought it wouldn't be necessary if the entities were already listed in the test persistence.xml.

Is the test suite failing in general or did I miss something?

It's a checkstyle error because of the use of a .* import: https://travis-ci.org/Blazebit/blaze-persistence/jobs/348301799

Annoying IDE configs all the time! :)


nixos: add XDG sounds module

Motivation for this change
As suggested in https://github.com/NixOS/nixpkgs/issues/47173.
Implementation is trivial: AFAICT there's not much to do other than link the directory.
@jtojnar @worldofpeace (I think you wanted this for the elementary PR)
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'm testing to see If this resolves things.

Couldn't we just auto-enable this when hardware.sound.enable = true;?


Update squid.conf to add ESD file types

adds ESD as file type for refresh_pattern of Microsoft Update files.
example link used by Microsoft:
http://fg.ds.b1.download.windowsupdate.com/d/Upgr/2018/09/17763.1.180914-1434.rs5_release_clientbusiness_vol_x64fre_en-us_d29ef094928a77496cdf53d072b5023eddea7281.esd

@R-Adrian looks good, thanks


Missing Channels

Source
https://www.mysky.com.ph/metromanila/tv-schedules
Channels
These are the channels that are available in the schedule but are missing in the epg:

Cartoon Network (PH Feed)
TLC (PH Feed)
TMC (TAGALIZED MOVIE CHANNEL)
ZooMoo
Da Vinci Kids
A2Z
Global Trekker (Rare Channel)
Love Nature
Cinema One (PH Feed)

Notes
I think the mysky epg must be updated every 2 hours because it's not loading the whole schedule for a day all at once.
Some of these are already in the clickthecity epg but that source is becoming unreliable on schedule sometimes.
Also, the CInema One on clickthecity must be added on "Cinema1Philippines.ph" because it is the schedule for the PH feed.

Added in https://github.com/iptv-org/epg/commit/e2085b095cf4a1cc9cf8af6cc631bbbc6209ab77


Legendary Banglum Modifier crashes upon use

This bug was reported in the All of Fabric Discord Server
Using the Legendary Banglum modifier on something like a Broad Axe crashes the game. This might be because the signature of the field changed between the versions (Changed from a custom class to a SimpleCriteria).
Crash log or game log (if applicable)
https://mclo.gs/xklrwUD
Versions

Loader: [Fabric/Quilt]
Hephaestus: 1.20.1-<IP_ADDRESS>1
HephaestusPlus: 1.1.1+1.20
Other (if necessary):

MythicMetals: 0.19.6+1.20.1

Can't seem to reproduce, although MythicMetals is on 0.19.7 instead of 0.19.6.
BLAST_MINING field also seems to be present.

If this is a change you did to Mythic Metals just in order to avoid the crash, please mark it as @Deprecated with info what should be used instead.


Fix secret name for capi 1.4

CAPI expects child clusters to leave secrets in a secret called %s-kubeconfig where %s is the name of the cluster.
We were so close, we did kubeconfig-%s, clusterName

/kind bug
/lifecycle active


Add ignore-suggestions flag

Closes #1430
Not sure how it should interact with --show flag. My assumption was that --show takes precedence and hlint reports all ideas (it's called showAll after all).

That sounds fine to me.

Added a test to cover that case :+1:

Thanks!


[Bug]: Parameter pack workaround fails with nvcc

Describe the issue
Was looking into failures with abseil 20240116.1 and tensorflow today, which fail with
# Execution platform: @local_execution_config_platform//:platform
$PREFIX/include/absl/strings/str_cat.h: In function 'std::enable_if_t<(sizeof... (T) > 1), typename std::common_type<typename std::conditional<true, void, absl::lts_20240116::strings_internal::EnableIfFastCase<T, typename std::enable_if<((std::is_arithmetic<T>::value && (! std::is_same<T, char>::value)) && (! std::is_same<T, bool>::value)), void>::type> >::type ...>::type> absl::lts_20240116::StrAppend(absl::Nonnull<T*>, T ...)':
$PREFIX/include/absl/strings/str_cat.h:606:316: error: expected ';' before '...' token
  606 |   for (const SomeTrivialEmptyType& dummy2 :
      |                                           ^

$PREFIX/include/absl/strings/str_cat.h:606:43: error: parameter packs not expanded with '...':
  606 |   for (const SomeTrivialEmptyType& dummy2 :
      |                                           ^

The relevant chunk of code
https://github.com/abseil/abseil-cpp/blob/2f9e432cce407ce0ae50676696666f33a77d42ac/absl/strings/str_cat.h#L606-L614
got introduced recently (d5a2cec006d14c6801ddeb768bf2574a1cf4fa7f), and as it turns out, there already was a comment on that commit noting exactly this problem:

@georgthegreat: @derekmauro, this pattern breaks nvcc (tested with nvcc11.4 + c++14, nvcc11.4 + c++17, nvcc12.1 + c++17 modes) with the following error:
absl/strings/str_cat.h:609:316: error: expected ';' after expression
for (const SomeTrivialEmptyType &dummy2 : ({((((((void)(n = (lengths[i]))), ((void)((n < (0)) ? ((void)((*(pos++)) = '-')), (n = (~n)) : (0)))), ((void)numbers_internal::FastIntToBufferBackward(numbers_internal::UnsignedAbsoluteValue(std::move(args)), pos += n, static_cast< uint32_t>(n)))), ((void)(++i))), dummy1)...}))

Can we do anything about this? I do not quite understand the meaning of this code, so I can not truly refactor it to make it compilable.
I do understand that nvcc is quite weird when it comes to such complex c++ code, so I need a workaround of some kind.

@derekmauro: @higher-performance - Can you take a look at this?

I think it effectively boils down to a compiler shortcoming of nvcc, but it would be good to be able to use abseil with CUDA.
Steps to reproduce the problem
Build CUDA-enabled tensorflow with newest abseil.
What version of Abseil are you using?
20240116.1
What operating system and version are you using?
Linux
What compiler and version are you using?
GCC + nvcc
What build system are you using?
bazel
Additional context
No response

Interestingly, I've failed to reproduce the error my collaborator encountered as well (I've tried both nvcc 11.2 & 12.0 with --std=c++17)
@georgthegreat, can you reproduce the issue on your end?

OK, it has been reproduced on our end now as well, but so far only in the context of building tensorflow. Full log can be found here.

Hi @higher-performance
A repro with godbolt here

Thank you very much! :)

Thanks a lot!

@derekmauro, is it possible to backport this into 20240116 release branch?

@derekmauro, is it possible to backport this into 20240116 release branch?

If there is another patch release (and there likely will be) then I will include this change.

https://github.com/abseil/abseil-cpp/releases/tag/20240116.2 contains this patch.

Thanks a lot!


Changes in CMakeLists.txt and SMPQ

There are lot of changes in cmake files, and many features related to smpq are added.

not ready - needed to merge testing -> feature branch before PR.


add example searches to the advanced search views

It would be relatively easy to have a list of common searches for each of the entities. This would give users a sense of what searches are possible, and how to accomplish them. Some searches, (e.g. variant coordinate searching) have been assumed by reviewers not to not exist when it's actually possible to accomplish using existing search parameters. Implementing example searches would mitigate these kinds of issues.

Nice example used in the paper.
Evidence search:
Gene name contains ALK
Evidence type is Predictive
Drug Name contains Alectinib
Source search
Journal Name contains New England
Variant search
CHR1 Chromosome is 2
CHR1 Start is in the range 16000000 to 60000000
Gene search
Description contains leukemia


Requests not aborted when time stepping

When stepping through time, requests are not aborted.
Verified on Chrome and Firefox

The problem is that _abortLoading does not check whether the time of a tile is different from the current layer time. It only checks the zoom level. We will have to override _abortLoading to do this.


why isn't the repl a repl?

The repl for slime in emacs or in lem (an emacs clone with all of the support for common lisp built-in) works like a repl.  After each input and response, a new prompt line appears.  Just like a shell.  I have never really seen a repl (see Julia, see Python) that doesn't work essentially like a specialized shell.
So, what happened here?  Entering a line, and seeing results displayed consecutively without the intervening commands that produced that output is kind of useless.  Far better to just use sbcl in a terminal pane: but, then we lose variable inspectors and other nice things.
There are other languages that use LSP protocol for various editors that include scrolling repls.
What am I missing?  Is there no multiline mode for this so-called repl?

Let's just close this.  I'll use rlwrap sbcl.  That just works. I can reload changed functions and everything else is there.  There is little value added to Alive, slime, sly, slynk, swank, etc.  The latter are archaic religious forms, not unlike Druidism, but not as nice.


@rpc annotation is not working on nested class

Godot version
v4.0-beta1.official [20d667284]
System information
Windows 10, Vulkan, NVIDIA GeForce GTX 1080 (<IP_ADDRESS>9)
Issue description
RPC calls on nested class function with @rpc annotation are not working, it gives the following error, just as no annotation was provided on function:
E 0:00:00:0962   _ready: Unable to get the RPC configuration for the function "test1" at path: "/root/Node2d/@@2".
                         This happens when the method is missing or not marked for RPCs in the local script.
  <C++ Error>    Condition "rpc_id == 65535" is true. Returning: ERR_INVALID_PARAMETER
  <C++ Source>   modules/multiplayer/scene_rpc_interface.cpp:468 @ rpcp()
  <Stack Trace>  :19 @ _ready()
                 rpc_nested_class.gd:10 @ _ready()

Steps to reproduce
Add a Node2D in a scene with the following attached script
extends Node2D

func _ready():
	var peer = ENetMultiplayerPeer.new()
	peer.create_server(1234)
	multiplayer.multiplayer_peer = peer

	rpc_id(1, "test0")
	var nested = NestedClass.new()
	add_child(nested)

@rpc(call_local)
func test0():
	print("test0")

class NestedClass extends Node:

	func _ready():
		rpc_id(1, "test1")

	@rpc(call_local)
	func test1():
		print("test1")

Minimal reproduction project
rpc-nested-class.zip

Seems like an issue in GDScript annotation parsing.
In the meantime, you can configure nodes using the language agnostic Node.rpc_config:
class NestedClass extends Node:

	func _init():
		rpc_config("test1", {
			"call_local": true,
			"rpc_mode": MultiplayerAPI.RPC_MODE_AUTHORITY
		})

	func _ready():
		rpc_id(1, "test1")

	func test1():
		print("test1")


[BEAM-2557] Move DirectRunner knob for suppressing runner-determined sharding out…

… of core SDK
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @tgroh
CC: @reuvenlax @jkff

LGTM if build is green.

retest this please

Changes Unknown when pulling 3621b9555e6c446737e98a045c2f8a20b1e9c3ad on kennknowles:DirectRunner-WriteFiles into ** on apache:master**.

Coverage increased (+0.003%) to 70.87% when pulling 3621b9555e6c446737e98a045c2f8a20b1e9c3ad on kennknowles:DirectRunner-WriteFiles into 4a2f67481ca930975650c138cbe7ba67bf342739 on apache:master.


Formulate and implement properties for PVBUMP STS in Byron-Shelley

With the PVBUMP STS having no dependency to other STSes in the Byron-Shelley era specifications, formulate and implement a few properties for the STS.

I propose the following properties for PVBUMP:

Given an empty list of mappings from a slot to a pair of a protocol version and protocol parameters (this is the fads part of the environment), the PVBUMP STS results in the same state it started in.
For s_n <= 2 * k, the resulting state is the same as the state the system started in.
For s_n > 2 * k, the resulting state is exclusively determined by the last pair from the list comprising a new protocol version and protocol parameters.

@uroboros , any comments?


Update from_reflect to return a Result

Objective

Fixes #5967.

Solution

Create a new Enum ReflectKind which contains all the "kinds" of types in Reflect
Add a new method reflect_kind to Reflect which returns the appropriate kind
Create a new Error type FromReflectError
Update the function from_reflect to return a Result with the above Error
Replaced some panic calls with Result inside from_reflect

Changelog

Changed the return type of FromReflect::from_reflect to a Result with FromReflectError as the Error type
Created a new Enum ReflectKind which contains all the "kinds" of types in Reflect
Added a new method reflect_kind(&self) -> ReflectKind to Reflect which returns the appropriate kind

Migration Guide

Replace from_reflect(...) with from_reflect(...).ok()
If any of your types implement Reflect manually, add a new method reflect_kind(&self) -> ReflectKind to it

I have chosen the name ReflectKind since a lot of documentation already mentions them as "kinds".
To differentiate between Struct and DynamicStruct on source type, I have added a from_kind field to each of the errors. from_kind will determine if it is a Struct, Enum, List, etc and TypeInfo will determine if it is Dynamic or not.
The same cannot be done for the target type since calling reflect_kind(&self) requires self which is not available. We cannot even remove &self from reflect_kind since it will become an associated function making the Trait not object-safe. So I made an assumption that target type will most likely not be Dynamic and therefore used TypeInfo to determine the kind of target type.

@elbertronnie are you interested in rebasing this PR? If not, we can mark this as S-Adopt-Me

@MrGVSV I am quite busy in other things currently. I can only work on this after about a month. If someone is willing to work on it now then I would suggest to mark it as S-Adopt-Me, otherwise I am willing to finish this incomplete PR after a month.

@MrGVSV I am quite busy in other things currently. I can only work on this after about a month. If someone is willing to work on it now then I would suggest to mark it as S-Adopt-Me, otherwise I am willing to finish this incomplete PR after a month.

I think we'll be okay to wait (unless you actually don't want to do it yourself, not trying to put any pressure on you haha). I’m planning to do some things with FromReflect soon, so if we decide we need it along with those changes then we can consider marking this for adoption.

Wouldn't TryFromReflect be a better name now?

Backlog cleanup: nice one, looks like this has a future so leaving it be!

Will check in again on this one in the new year :slightly_smiling_face:


MNT Update @methods on class docblocks

Issue https://github.com/silverstripe/.github/issues/163
Used short names instead of fqcn for relation types if class was imported

Have updated code writer


Implement undo and redo

Pull request type

Feature
What is the current behavior?

The user can't undo any changes.
Issue Number: This fixes #38
What is the new behavior?

Undo and redo is implemented using the graph notification mechanism
Editors need to call ProjectManager.addCheckpoint when a consistent change set (like adding a token) is finished. Salt events will be sent for each little updated, but the user probably wants to undo a complete user interaction
When a checkpoint is created or restored, an ANNOTATION_CHANGED event is sent. This allows all editors to react for consistent changes to the annotation graph without fearing to deal with an inconsistent Salt model
A toolbar with the new/open/save and both the undo and redo buttons has been added

Does this introduce a breaking change?

Yes, the old events for small incremental changes have been replaced with ANNOTATION_CHANGED.
Also, the unused and untested ProjectManager.loadDocumentFrom function has been removed.
Other information

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added or the PR is neither a bug fix nor a feature
[x] Docs have been reviewed and added / updated if needed or the PR is neither a bug fix nor a feature
[x] The Unreleased section in CHANGELOG.md has been amended to reflect the changes in this PR if needed
[x] Build (mvn verify) was run locally and any changes were pushed
[x] The pull request is against the correct branch (master for bug fixes, develop for new functionality)
[x] Dependencies and citation templates have been updated where necessary or there are no new dependencies

Checklist for maintainers
Releases

Do NOT push the green merge button on GitHub.
Follow the latest version of the developer/maintainer docs for making releases.
Check that license and citation information for dependencies are complete (each dependency has a folder in /THIRD-PARTY/).

Fix the issue with the non-enabled toolbar buttons by sending the necessary event (see 13.5 in https://www.vogella.com/tutorials/EclipseRCP/article.html) manually in the ProjectManager.

Unfortunately I've stumbled over another bug, not sure if it was introduced in this PR, but should probably be fixed in this one: Saving throws an error.

I checked and this bug is present in the current master branch (#220) and I would propose to fix this as a hotfix release (which should of course include a regression test). It might be related to #214 but this will only be clear after we find the root cause of this issue, which is obviously not the changes added in this PR.

Unfortunately I've stumbled over another bug, not sure if it was introduced in this PR, but should probably be fixed in this one: Saving throws an error.

This is now fixed in master and integrated into this PR


Link explorer source code from site

It would be helpful for the Explorer website to link back to this repo so that people who are interested in it can find the source code. Many open-source projects do this in the form of a GitHub Octocat icon that links to the source repo, for example https://github.com/ripple/explorer/ —preferably with a "title" attribute that says "Source Code" or something so that desktop users can get a little note by hovering their mouse over the icon.

@mDuo13 Would placing it under the contribute section in the footer make sense as well?

Completed by #311


numpy > 1.24 is not supported

In numpy version 1.24.0 various deprecated type aliases were removed, see https://github.com/numpy/numpy/pull/22607
This needs to be reflected in https://github.com/finos/qPython/blob/main/qpython/qtype.py

This appears to affect np.bool and np.str. I would contribute a fix myself but it appears FINOS only accepts pull requests from those who have signed a license agreement with FINOS: https://github.com/finos/qPython/blob/main/CONTRIBUTING.md
Perhaps someone at FINOS could push a fix for this.

Is FinOS actually supporting this project?


DASH on hitbtc

In class hitbtc and in its method fetch_products()
                base = 'DASH'

I dont understand this two lines.
DASH is DASH and DSH is DASHCOIN they are two separate coins I've double checked.

Hey!
Please, check these two issues:

Consistency issues
Inconsistent naming across exchange markets

Also, there's a dedicated section on this conversion from DSH to DASH in the Manual. I suggest you read entire Manual from the very beginning to the end very carefully )

In short: you're right, these are two separate coins, but some markets have them confused (call them DSH instead of DASH). So, the lines you're looking at are fixing for that for those markets, that got it wrong.


Change duplicate entry log to debug level

Duplicate entries can happen often and spam the logs.
They don't indicate a real error, so this PR moves the
log message to the debug level.

@calvinjia

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2325/
Test PASSed.