[v0.7.25r] RS Bridge Method craftItem Not Crafting Lava Bucket

Describe
In the previous version, my RS stock manager was able to queue items to be crafted (although the getItem method was bugged in that version so it would actually endlessly queue items), however in this version, while I'm able to fetch information about the items, I'm unable to craft items through the peripheral.
Despite being able to manually queue minecraft:lava_bucket in the RS system directly, I am unable to do so from the RS bridge peripheral. The function simply returns false every time I try to queue one. Furthermore, the getItem function indicates that the item is not craftable.

Strangely, I was able to craft mekanism:basic_universal_cable for some reason... as well as minecraft:stone

Strangely, other items are also not considered craftable for some reason...

I'm failing to pick up a pattern here. Strangely, after changing literally nothing, the program suddenly queued mekanism:enriched_redstone

I'm wondering if it's related to the pattern being in a crafter that's attached to a mekanism:advanced_fluid_tank
Steps to reproduce

Attach a crafter to a mekanism:advanced_fluid_tank
Insert a recipe that calls for an minecraft:iron_bucket and 1 B of lava
Attempt to craft through the RS bridge
Attempt to craft directly in the system if it's not working in the bridge

Multiplayer?
Yes
Version
1.18.2-0.7.24r (Latest 1.18)
Minecraft, Forge and maybe other related mods versions
40.1.92
Screenshots or Videos
No response
Crashlog/log
No response

You forgot the minecraft: at the name
minecraft:lava_bucket
Does it return isCraftable true if you fix that?

No, I don't believe the minecraft: was missing at any point. Neither in the program or in my testing. I did run a test without the minecraft: to see what it did.
This is from my program's config file:

https://user-images.githubusercontent.com/10189526/218336631-159c7179-0d3c-48f9-ac96-8d800b10f9cb.mp4

This is weird
Maybe some issue with comparing the stacks on the rs side
I'll work on that today!

AdvancedPeripherals-0.7.26r.zip
Could you install this version and enable debug mode in the config file config/Advancedperipherals/general.toml?
This should print some information about the craft state into the console when you run craftItem
Unzip the file before using it

[12:55:16] [Server thread/WARN] [Advanced Peripherals/]: [DEBUG] Schedule chunk manager cleanup
[12:55:18] [Server thread/INFO] [Advanced Peripherals/]: [DEBUG] Crafting Result of 'minecraft:air':NO_PATTERN
[12:55:21] [Server thread/INFO] [Advanced Peripherals/]: [DEBUG] Crafting Result of 'mekanism:enriched_carbon':OK
[12:55:22] [Server thread/INFO] [Advanced Peripherals/]: [DEBUG] Crafting Result of 'minecraft:air':NO_PATTERN

[12:55:18] [Server thread/INFO] [Advanced Peripherals/]: [DEBUG] Crafting Result of 'minecraft:air':NO_PATTERN
[12:55:21] [Server thread/INFO] [Advanced Peripherals/]: [DEBUG] Crafting Result of 'mekanism:enriched_carbon':OK
[12:55:22] [Server thread/INFO] [Advanced Peripherals/]: [DEBUG] Crafting Result of 'minecraft:air':NO_PATTERN```

I guess the first and the third are lava buckets?

It stopped queueing mekanism:enriched_carbon again...

https://youtu.be/PApgLx95doI Video has finished processing and can now be viewed properly

Found the issue.
If the item is not in the system, it can't be crafted

I noticed it started crafting enriched carbon after I manually queued up a bunch... Then after a while it stopped. That does line up with what I'm seeing.

AdvancedPeripherals-0.7.26r.zip
This file contains the fix if you want to test it
I'll update today or tomorrow
Unzip the file before using it


Can you set it up to use ChatGPT.3.5_Turbo API instead of davinci?

Would save a lot of $$. Can't find the setting in the files.
Thank you.

Thank you for bringing this to my attention. I've updated the underlying model to new got-3.5-turbo, have fun!


Make traversal order pre-order in bufferization.

Make the traversal order pre-order since for ops with nested regions,
it is necessary to process the op before the region.
Changing to pre-order also means that regions of cloned ops dont get
processed during the work to convert tensor to memrefs. As a result,
move the conversion of ops that are just using the tensor value, and
not creating a tensor to be processed in a separate walk.

@asaadaldien This probably needs to be rebased over your PR, but this makes the bufferization do a pre-order walk.

@asaadaldien This probably needs to be rebased over your PR, but this makes the bufferization do a pre-order walk.

Awesome! thanks @MaheshRavishankar I have this rebased and working  with minimal shuffling just by moving the breaking dependency scf.for arg forwarding as  the last post processing here :
https://github.com/google/iree/tree/ataei-pre_order_less_alloc (Because I don't have write access to your fork).
It handles one of the extra unnecessary allocations, I am looking at the second one which should be a subview of a subiew.
I will be committing on that branch(iree/ataei-pre_order_less_alloc).
Feel free to rebase your fork/branch on that branch once you are back and we can submit this PR

@MaheshRavishankar this isn't needed any more, I will close this PR.


Best way to return 301

Does the ingress.kubernetes.io/rewrite-target annotation add some 30x redirect to responses? I just need to simply redirect root domains to www.domain.com.
I want to get the same effect as:
server {
...
    server_name www.old-name.com;
    return 301 $scheme://www.new-name.com$request_uri;
}

But I don't want to make a custom template. So must I just use the rewrite annotation or is there a correct way of handling this?
If I must use the rewrite annotation would I be able to add the whole string as in rewrite ^(.*)$ $scheme://www.domain.com$1 permanent; or just the URI/URL part?

According to this code, the ingress.kubernetes.io/rewrite-target cannot support root domain redirection.

Thanks, I'll just use downstream web server configuration.


When trying to run the *.js file using the k6 command getting run time error

rohit@bmdvrohitjai:~/Downloads/k6-v0.17.1-linux32$ ./k6 run Demo_script.js
      /\      |‾‾|  /‾‾/  /‾/
 /\  /  \     |  |_/  /  / /
/  \/    \    |      |  /  ‾‾\

/          \   |  |‾\  \ | () |
/ __________ \  |__|  _\ ___/  Welcome to k6 v0.17.1!
execution: local
output: -
script: /home/rohit/Downloads/k6-v0.17.1-linux32/Demo_script.js (js)
duration: 0s, iterations: 1
vus: 1, max: 1
web ui: http://<IP_ADDRESS>:6565/

panic: runtime error: invalid memory address or nil pointer dereference-- / --
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x80492cc]
goroutine 7 [running]:
sync/atomic.AddUint64(0x1a472414, 0x12e, 0x0, 0x1000, 0x12e)
/usr/local/Cellar/go/1.8.3/libexec/src/sync/atomic/asm_386.s:112 +0xc
github.com/loadimpact/k6/lib/netext.(*Conn).Write(0x1a52d870, 0x1a081000, 0x12e, 0x1000, 0x12, 0x857b380, 0x197261c0)
/Users/uppfinnarn/Projects/Go/src/github.com/loadimpact/k6/lib/netext/dialer.go:81 +0x70
net/http.persistConnWriter.Write(0x1a1fc0a0, 0x1a081000, 0x12e, 0x1000, 0xd0, 0x86166be, 0x3)
/usr/local/Cellar/go/1.8.3/libexec/src/net/http/transport.go:1133 +0x3f
bufio.(*Writer).Flush(0x19b92360, 0x88c4dc0, 0x19b92360)
/usr/local/Cellar/go/1.8.3/libexec/src/bufio/bufio.go:566 +0x5d
net/http.(*persistConn).writeLoop(0x1a1fc0a0)
/usr/local/Cellar/go/1.8.3/libexec/src/net/http/transport.go:1709 +0x334
created by net/http.(*Transport).dialConn
/usr/local/Cellar/go/1.8.3/libexec/src/net/http/transport.go:1118 +0x8ac
rohit@bmdvrohitjai:~/Downloads/k6-v0.17.1-linux32$

@jrohit54 The error looks to be k6 command problem.
Please, could you share your postman collection and/or the converted k6 script?

sure, i have attached the postman collection and converted k6 script also
Files.zip

@jrohit54 The postman-to-k6 converter is working well.
The problem could be the k6 installation, k6 is not working.
You could try another load script as the one in the Getting started
import http from "k6/http";

export default function() {
  http.get("http://test.loadimpact.com");
};)

I am not sure about your environment. Could it be that you are running the Linux k6 binaries on a Mac?
I am closing this issue. Please, ask for Help in the Slack channel or open an issue in the k6 project.

Yes , i was facing issue in particular version of k6 on linux OS.
v0.17.1- Facing above issue
v0.17.0  -Issue resolved
Thanks for you support.

@jrohit54 If you know how to reproduce the error, it may help others if you create an issue in the k6 project.
Thanks!


feat(e2e): test chaos ci rebase chaos-integration

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

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

End of test


Files are not formatted correctly

Hello, you have the only broken Proxy list on all of Github.
  There are ^M characters in the wrong places, and the lines are all jibberish and merged together

💀  you're right

Thank you both for confirming my issue. I don't think this is maintained anymore


Use maven's release plugin to ease creation of new storm-mesos releases

Usage:
mvn release:clean release:prepare

@DarinJ : can you please take a look when you have a chance?

@erikdw sure - probably some time late today.  Looks good at first glance, just want to verify the interaction with build-release.sh.  I think we may want to consider eventually pulling some (if not all) of it's tasks into mvn, thoughts?

@DarinJ : thanks!  It feels right to do more stuff in maven directly, but it's also kind of painful so might not be worth the effort to pull everything from build-release.sh in to the pom files.  Let's mull it over, it seems like we could use the assembly plugin to do at least some of the build-release.sh logic.

@erikdw checked it out.  LGTM.  Tested with my repo just to be sure.
👍 🚀:shipit: 🚢

thanks!


client.py create and alter_retention_policy functions don't allow to change the shard group duration

The influxdb documentation recommends that you need to chose a specific shard group duration that matches your retention policy e.g. 1 day RP should have 1 hour shard group duration. Link to the docs:
https://docs.influxdata.com/influxdb/v1.4/concepts/schema_and_data_layout/#shard-group-duration-management
This parameter is missing from the client. I used the 4.1.1 version but this problem is still in 5.0.0

We can add a shard_duration parameter -- but do note (from the link you pasted):
By default, the shard group duration is determined by the retention policy’s DURATION

So simply setting a retention policy will automatically set the proper shard group duration based on the table in the documentation.


Added missing desc; repaired links

Imported from "windows/storage/repair-volume.md":

Added the missing description for -AsJob
Added the missing cmdlet description
Repaired three damaged links

@skycommand - thanks for fixes, much appreciated!


VSCode Python 3.7 unable to run code

So when I type code, the text does not change color like it is supposed to (for example, writing 'if'). it also does not let me run code, as the green arrow in the top right isn't there like it's supposed to. And finally, the 'Python IntelliSense' system does not function in any way. When I write code, all of the text is grey. From looking at the startup tutorial I checked the python interpreter to see if it was selected, and as far as I know, it is working correctly. bear in mind that I am very new to VSC as well as programming in general  (taking computer science for GCSE), so any help would be much appreciated :)
example below:

Hey @FlowMeister, just to confirm, have you installed the Python extension? (https://code.visualstudio.com/docs/languages/python#_install-python-and-the-python-extension)

@RMacfarlane Yep, I followed the guide on the VSCode page :)

@FlowMeister Looking at your first screenshot again, I suspect the issue there is that since this file has not been saved, it's marked as being plaintext rather than Python. In the statusbar in the lower right hand corner, there should be an indicator of what the current language is for the document. Once you save the document with a .py extension it should automatically update, you can also change it by clicking on that indicator or running the Change Language Mode command. You can also change what the default language is for untitled files with the files.defaultLanguage setting.

@RMacfarlane Thank you so much! it's working properly now, I changed it from plaintext to python like you said and everything is fixed. Really appreciate the help.


Check if fslview is executable

Check if fslview is executable, and if not set $DISPLAYQC=false.
Requirements

[x] Is there a issue open describing the motivation for this pull request ? -> #1547
[ ] Does the pull request need an update of the documentation ? -> no
[ ] Does the pull request need additional test(s) ? -> no
[x] Clean up the warnings generated by the style guide -> N/A

Description of the Change
code was added to check if fslview is an executable.
Applicable Issues
Implements #1547

@fperdigon the problem is not from batch_processing. The problem is from there.


structor command is unable to find ./server/api.js

Following the steps given to install, in both the scenarios (using empty folder or after running npm install on the downloaded folder from market). The command structor . throws an error. Find screenshot below:
Macbook Pro Retina 13inch - mid 2015
OSX El Capitan
npm -v 2.14.6
node -v 4.1.2

I didn't test it on node 4.

Samve problem on node version v0.12.7

Try to reinstall with -g key.
npm uninstall structor -g
npm install structor -g

works.


Links within Navie conversations open in the active editor and browser

Given the following prompt:
@help I'm using the AppMap agent for Java. My web framework is Spring. How do I record HTTP requests to my application?

Navie is likely to respond with links to documentation. Upon clicking those links, the active panel navigates to the link as well as an external browser.
The expected behavior is that links should only be opened in the browser.

I'm not sure if we'd need to just modify the link attributes to contain target="_blank". If that's the case, this issue can be fixed in the frontend.

I didn't see this when I opened my issue. Adding my report for context:
When a Navie response contains a link, the link opens the destination in an external browser AND in an embedded browser. The embedded browser would be handy but in it's current form it destroys data.  Once the embeddee browser opens my Navie conversation is over and inaccessible because there is no 'back' functionality. The fact that there is no conversational history compounds the loss... once a link is clicked, the conversation is effectively over.

@dustinbyrne I was able to replicate this.
I don't think this can be fixed in the plugin. Even though we already have code to open the link in the external browser and to suppress the local navigation, it's still doing it. I wasn't able to completely suppress this navigation. Is there perhaps a JS handler attached to the link?
When I added a target="_blank" attribute using the web inspector, it worked as expected.
This bug should be fixed as soon as the webview frontend adds target="_blank" attributes.


Internal Errors

As requested by @richsalz in https://github.com/openssl/openssl/issues/5213 here are some more cases where you return internal error but probably shouldn't because we were able to trigger them using malformed messages.
ssl/statem/statem_clnt.c:2468
ssl/statem/statem_clnt.c:2507
ssl/s3_lib.c:4744
crypto/asn1/asn1_lib.c:101
ssl/statem/extensions_clnt.c:1292
ssl/statem/statem_srvr.c:2912
I hope I got all of these right. Feel free to ask for more information if you need any.

The version 1.1.1 is not supported anymore. Closing. If this is still an issue with currently supported releases, please open a new issue.


Make ZK client timeouts configurable

For build servers with high load, it may happen that the build hangs for couple of seconds. This causes the internal ZK client session of embedded Kafka broker to timeout and fail the build.
It would be good if these were configurable.

Hi @garyrussell , sure, happy to take this forward. Didn't you mean @EmbeddedKafka?

Sorry. Yes @EmbeddedKafka.

And cherry-picked as abf2bfe006b2d18a00199e6754a3d5a2ff98bc93 to 2.3.x with deprecated ZkClient fix.
Thanks again @pawellozinski

:thumbsup:


Patch-Cut Action off any release-* branch

Describe the Improvement:
We should be able to use the patch-cut action off of any release-* branch, not just main

also maybe look into creating PR during patch branch creation process with TODOs in PR body (or full template?)

Benefits:

More automated process for patching
creating PR will prevent accidentally creating PR into develop

Drawbacks:
None
How to Compare:
Able to run patch-cut action on any release-* branch

https://stackoverflow.com/questions/58033366/how-to-get-current-branch-within-github-actions

We need to create a release-on-merge workflow that increments the version number on merge of patch-* branch to release-* branch


Add Support for using an existing ALB and Listener

Description
I have been following along and using the ALB events and provisioning PR #1807  that was just merged in by @PeteW and @youcandanch.
I was using those changes and integrating my Lambda's into an existing ALB provisioned via terraform
I have some proposed zappa settings and code changes to allow the use of an existing ALB, multiple Zappa projects can then be deployed onto the same ALB
GitHub Issues

#1832

Coverage decreased (-17.1%) to 56.115% when pulling 3d24a27e6e3a3972a1b964666933f2f15b076e56 on whahoo:use_existing_alb into c46d4641ac4052225bebcdf552b9ed77e0c27c95 on Miserlou:master.

Is this something that's still potentially going to be merged down to master? It would be useful to some of the projects I'm working on right now, so just wondering. Thanks!

Any update on this? Would be very useful in bigger environment where having a 1 to 1 mapping between lambdas and ALBs makes no sense.

Another bump - this would be super useful for us.

@whahoo I noticed an issue testing this branch with my zappa deploys and trying to move them to an ALB.
I have two listeners on my ALB.  One for port 80, and one for port 443.  This PR is erroring on getting both listeners.
OSError: Failure to load listeners for ALB. or more than 1 found, Response was in unexpected format. Response was:
We just have 80 as a catch all and fwd to port 443.

Also base-path was required in the zappa settings to make this work for us.
Other than the two gotchas we experience, this is working great! @whahoo @Miserlou @jneves

@jneves any updates on this?  Would love to get this running on our production ALBs

For anyone stuck on this like I was - here's a workaround  where you set up your ALB listeners to point to your lambda using an Alias once and then can zappa update without breaking the link

Speaking as the original developer of ALB support I've read this PR and found it both clear and comprehensive. Also, if this can get merged in I see myself using this new feature wherever possible. It is a good practice from a cost/surface area consolidation perspective to be able to reuse existing ALBs.


Get GitHub Action working properly

The default GitHub Action setup doesn't run the tests properly or successfully

Done as of v 0.7.3


Iyanujesu introduction wagtail

Added my first contribution

Well done @Tee-works, LGTM.
A little addition from me is to make sure that you follow the Markdown style guide here
If you use VScode, I suggest you download this extension Markdownlint. It will help guide and fix your markdown file.


Julia getting killed when computing SpMV with SparseBand level on Hamm/scircuit matrix

Can run SpMV with SparseList and SparseVBL levels on Hamm/scircuit matrix from SuiteSparse. However, Julia REPL keeps getting killed with no error message with SparseBand level, even when @inbounds macro is removed from generated code and Julia is started with --check-bounds=yes flag. Not sure what the problem is...
┌ Info: testing
│   key = "finch_band_unsym"
└   mtx = "Hamm/scircuit"
Killed

can you run it locally?

does it die if it's the first matrix to run?

okay. we can exempt it for now, but let's take a look at that code at some point later.

Finding the same issue with Williams/cop20k_A (I've just commented it out from the benchmarks - we can look into it later)

can you replicate these locally? If you try to run it outside of a @belapsed statement does it work okay? What if you run with --checkbounds=yes?

Yes, I can replicate this locally and with --check-bounds=yes. It seems the issue is coming from when we are trying to initialize a SparseBand level. For example, the following kills Julia.
A = SparseMatrixCSC(matrixdepot("Hamm/scircuit"))
A_band = Tensor(Dense(SparseBand(Element(0.0))), A)

It dies with nothing :(
julia> A_band = Tensor(Dense(SparseBand(Element(0.0))), A)
zsh: killed     julia --check-bounds=yes

we'll have to narrow down what it is, whether it's an allocation or an out of bounds access

Seems that it could be an OOM error from this thread, but I'll add some print statements to check

Ah okay, that would make sense. These two matrices also seem to have nonzero values on the far left and far right of most rows—SparseBand is definitely not the right format for them. I don't know if we want to handle this in Finch with a warning or just include a note in the Finch documentation (or maybe just do nothing at all about this).

I think for now, we don't change finch, and we exempt non-banded matrices from sparseBand level


Use Angular animations system

This goes hand in hand with #3. (Holding off until RC5)

Done via 238b79e6a15a4f330aed61807e1483a088cb5228


atSDK docs

[ ] Itemization / comparison of features in each SDK
[ ] Feature / use-case driven documentation of the atSDK in Gitbook
[ ] atProtocol:atSDK feature mapping / implementation guideline

Duplicate of several other issues


Running elm version 0.19.0-no-deps from an npm script fails

Quick Summary:
Running elm make from an npm script results in:
The process cannot access the file because it is being used by another process.
Duplicate of #1932
SSCCE
Running npm install and then npm run build from a directory containing the following package.json results in the error above. I think it depends on the speed of the hard drive/computer or presence of a virus scanner if the issue happens or not, as it is caused by a race condition between the download script and the spawn of the elm executable.
{
  "name": "no-deps",
  "version": "1.0.0",
  "description": "",
  "scripts": {
    "build": "elm init"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "elm": "0.19.0-no-deps"
  }
}

Elm: 0.19.0
Browser: N/A
Operating System: Windows 7

Additional Details
$ npm run build

><EMAIL_ADDRESS>build C:\test
> elm init

The process cannot access the file because it is being used by another process.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `elm init`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\teugis\AppData\Roaming\npm-cache\_logs\2019-05-17T11_42_36_924Z-debug.log

Cause
On windows it tries to use node_modules\elm\bin\elm when you run elm via an npm script. This results in the download.js script starting to download the elm executable, even if it already exists. Apparently the spawn of the elm executable happens before the download script has completely released the file. It is also possible that a virus scanner is somehow interfering here.
Possible solution
A possible solution would be to detect if the executable already exists and then not download it, which would prevent the possibility of a race condition with the file locks between downloading and spawning the elm executable (at least in the case where the elm executable already exists on disk).
Changing the export in download.js to the code below ensures the elm executable is not downloaded if it already exists and solves the issue for me.
module.exports = function(destinationPath, callback)
{
	var version = package.version.replace(/^(\d+\.\d+\.\d+).*$/, '$1'); // turn '1.2.3-alpha' into '1.2.3'
	var os = { 'darwin': 'mac', 'win32': 'windows', 'linux': 'linux' }[process.platform];
	var arch = { 'x64': '64-bit', 'ia32': '32-bit' }[process.arch];
	var path = process.platform === 'win32' ? destinationPath + '.exe' : destinationPath;

	fs.open(path, 'r', function (error, fd) {
		if (error) {
			download(
				path,
				'https://github.com/elm/compiler/releases/download/' + version + '/binary-for-' + os + '-' + arch + '.gz',
				callback
			);
		} else {
			callback();
		}
	});
};

Same problem here. Installing 19 using Windows installer works fine. Can't run elm commands after npm install because The process cannot access the file because it is being used by another process.

Running npm install and then npm run build

Huh, so I guess "and then" in this case means instantaneously after—as in, npm install && npm run build, not "I typed npm install and then when that command finished I typed npm run build and pressed Enter again." I'd guess with the latter there would be too much of a time gap to trigger a race condition like this. 🤔

@panthus One concern I have about this proposed solution is that on non-Windows systems, I expect this conditional would always pass because the name of the downloaded binary will be elm (not elm.exe) and there will already be a file called elm there.
(On non-Windows systems, the previous elm is replaced by the downloaded one; Windows is unusual in this regard because it requires adding a .exe extension.)

Running npm install and then npm run build

Huh, so I guess "and then" in this case means instantaneously after—as in, npm install && npm run build, not "I typed npm install and then when that command finished I typed npm run build and pressed Enter again." I'd guess with the latter there would be too much of a time gap to trigger a race condition like this. 🤔

I ran them one by one. After thinking about it some more I am pretty sure it is the virus scanner.

@panthus One concern I have about this proposed solution is that on non-Windows systems, I expect this conditional would always pass because the name of the downloaded binary will be elm (not elm.exe) and there will already be a file called elm there.
(On non-Windows systems, the previous elm is replaced by the downloaded one; Windows is unusual in this regard because it requires adding a .exe extension.)

I was afraid that was the intention. Right now it downloads the elm executable every time I try to run an elm command (no matter if the .exe already exists or not)...
Since the elm file is not overwritten it triggers a download each time it is executed. Then immediately after the download it tries to spawn the executable but since it was just downloaded my virus scanner is still scanning the file which causes the file in use error.

Perhaps it would be a better solution to have the download script remove the elm file or have it replaced by a file that only spawns the .exe instead of download and then spawn.
Or perhaps have the wrappers in node_modules/.bin call the .exe instead of the elm file (this would also be quicker as you do not have to spawn node.exe).

Hmm, how does this work on Linux or macos. The elm javascript file is replaced with the elm executable, but the wrappers in node_modules/.bin try to execute that elm file with node.exe, which will fail because that file is no longer a javascript file?
Or are those wrapper files different on Linux and macos?

Could this be in any way related to https://github.com/parcel-bundler/parcel/issues/3042 ?

Could this be in any way related to parcel-bundler/parcel#3042 ?

If parcel uses elm via the npm package then changes are high that it does. Every time you run elm make using elm from the npm package triggers a download of the elm executable (even if you already have the executable on disk...). File downloads are usually intercepted by a virus scanner and this means the virus scanner could lock the file when elm tries to execute it.

An easy workaround is to downgrade the elm npm package to version 0.19.0-bugfix6.

I've tried downgrading to bugfix5 and bugfix6 and I'm still getting this error when running through Parcel.

On a slow connection, waiting for the exe to download, every time, before the actual command is run, is an awful experience. Particularly since no feedback that that is happening is given. As might be expected, this problem affects Windows 10 as well as 7.
One of these two solutions suggested by @panthus seem like the best way forward.

* Perhaps it would be a better solution to have the download script remove the `elm` file or have it replaced by a file that only spawns the .exe instead of download and then spawn.

* Or perhaps have the wrappers in `node_modules/.bin` call the .exe instead of the `elm` file (this would also be quicker as you do not have to spawn node.exe).

In the meantime, manually replacing node_modules\elm\bin\elm with the following is a workaround, (I just cut out the download portions of the existing script):
#!/usr/bin/env node

var child_process = require('child_process');
var fs = require('fs');

// Make sure we get the right path even if we're executing from the symlinked
// node_modules/.bin/ executable
var pathToThisFile = fs.realpathSync(process.argv[1]);

// Need double quotes and { shell: true } when there are spaces in the path on windows:
// https://github.com/nodejs/node/issues/7367#issuecomment-229721296
child_process
.spawn('"' + pathToThisFile + '"', process.argv.slice(2), { stdio: 'inherit', shell: true })
.on('exit', process.exit);

Does this persist with npm install<EMAIL_ADDRESS>

That version seems to have fixed it for me. I can see in the node_modules/elm/bin/elm file that there is now a special code path for Windows.
I'll note that the latest-0.19.1 version also currently brings in 47 other folders into node_modules where the previous version only added the elm folder. But not having to mess with the scripts to get things working acceptably on a slow connection is a definite improvement!

The issue is also gone for me as well. Thanks for the hard work :)

Does this persist with npm install<EMAIL_ADDRESS>
That fixes the issue for me too, thanks a lot! :)


feature/inventory toggle keybindings

All keybindings are now in one file. I checked out #60 verified that it basically did what I attempted to do a few days ago, but the issue remains: the inventory button is still listening for the [Enter] and [Space] keys.
Unfortunately, I had no better idea at the moment other than to maintain the handleKeyDown(event) {} class method in <Inventory /> to do an event.preventDefault(). Does anyone else have a more elegant solution?
P.S.: @Alaricus your inventory label button changes has been included.

I'm in favor of option #1.

Looks good!

@KajiTetsushi Thanks for taking the time to make this PR. After checking out the code locally, I noticed a few things that I would like changed before I merge into staging.
Currently, when the user has the "new item" notification dot on the inventory button, then uses the "I" key, it does not remove the notification on inventory open. This works when the user clicks.
The user can now open the inventory menu with "I" when the settings dialog is open. I don't know if I like this behavior. My thought with the settings dialog was to "pause" everything. I think maybe in the controls we can have a separate check for only "settings open" and event.keyCode = 73 instead of gamePaused and 73. Then you could remove the case from the switch and just toggle inventory on that new condition only.
One minor thing, and this might be nit-picky, but I think in /features/inventory on the keyDown function, you should use an if instead of switch, since we only have 1 intended action: event.preventDefault().

The user can now open the inventory menu with "I" when the settings dialog is open. I don't know if I like this behavior. My thought with the settings dialog was to "pause" everything.

@ASteinheiser I didn't notice this while I was testing the feature. Sorry. It's not supposed to behave that way. I'll look into it again after work.
While I'm at it, I'll have a look at the app state on the redux store as you've recommended. And the if over switch style.

@KajiTetsushi Thanks!
And for the "new item" dot, you should just be able to call this.setState({ newItemIndicator: false }); from features/inventory/index.js.

Okay, I just experimented on this, @ASteinheiser. Two things I'm attempting to deal with here:

Opening inventory despite the game being paused under settings
I found out that this case is actually not just limited to settings. For every dialog in the game that takes the player away from the map (e.g. chest loot, shop list, story dialog), hitting "I" would still bring up the inventory. So, reading from the already-available appState definitions, I think the conditions where we should allow this key binding to work is as follows:

Run toggleInventory() through event.keyCode === 73 when the game has not been paused or when the game has been paused but the inventory dialog has been opened. If the game has been paused because of another dialog, you're not allowed to check the inventory until you close that dialog.
Settings needs to pause the game. Strangely, right now, it doesn't. I'd call that a bug. You can check this on the redux store inspector.

"New item" dot not turned off with key input
The conclusion I got was that the <Inventory>'s self-contained component state wouldn't be enough for this. I thought it was as straightforward as listening to the prop changes so that the inventory can tell whether to flash the new item indicator or not.
It turns out during console logging, I couldn't get a difference between last disabled state and current disabled state to toggle the inventory visibility.
Go ahead and try this on your local:
class Inventory extends Component {
  componentDidUpdate(prevProps, prevState) {
    const { inventory } = this.props.dialog;
    const { itemReceived, itemDropped } = this.props.snackbar;
    const lastDisabled = prevProps.disabled;
    const lastItemReceived = prevProps.snackbar.itemReceived;
    const lastItemDropped = prevProps.snackbar.itemDropped;

    if(lastItemDropped !== itemDropped && itemDropped &&
      typeof itemDropped !== "undefined" && !inventory) {
      // see if any items were dropped
      this.setState({ newItemIndicator: true });
    }
    else if(lastItemReceived !== itemReceived && itemReceived &&
      typeof itemReceived !== "undefined" && !inventory) {
      // see if any items were received
      this.setState({ newItemIndicator: true });
    }
    else if(!lastDisabled && prevState.newItemIndicator) {
      // see if the inventory was previously hidden
      this.setState({ newItemIndicator: false });
    }
  }

  // ... other methods & properties
}

Run the game, then start endless mode to get a rusty sword new item right away, then click on the settings cogwheel. The new item dot would disappear.
In order to kill this bug once and for all, it requires either one below:

a window.addEventListener() handler to be added to the <Inventory>. I'd say this is quick n' dirty. You'd want to pay attention to the keypress as soon as the component mounts, then if the key was ever pressed in this component, trigger a change of the newItemIndicator state to false. The question is whether you like to have pollute the DOM with one more non-React event listener.

the newItemIndicator state to be moved up to the appState management, then listen for any changes here. <Inventory> would no longer need to contain its own state. I'd prefer this way, because that'd hopefully less of a need to install additional event listeners on the inventory component.

Pardon me if it sounds like I'm picking your game apart, but I honestly do care about stability.

@KajiTetsushi I agree with your conditions for toggling the inventory. I thought that with the settings open, that the player could not move or attack? I just tested this on the master branch and I can't see anything happen in the redux debugger with settings open.
We should definitely just move the "new item" state to a redux store if we are going to go with either of your two solutions. However, I think we could do it without needing to do that much.
I am not seeing the behavior where the settings button is turning off the "new item" dot. And my real question is why are you trying to read the last disabled? Can you not just turn off the dot on the keypress, similar to how we turn it off on button click?


Fix Ready state error

When a dependent task is in the ready state and one of its prerequisites is reset, the dependent task is not moved into pending state. This is because we don't reset ready tasks. This is a bug, as a dependent task can run without its prerequisite being finished.
@mikehaytm Please review.

:+1:


User request: Move the computer evaluation button back to the right side

Nov 12 Lichess design changed the position of the computer evaluation to the left corner of Tools. User requests for it to put it back on the right.
Also, maybe a slight restyling so it doesn't take as much space?

Implemented in v2.1.47


Reaction ratelimit headers are wrong.

As per https://github.com/hammerandchisel/discord-api-docs/issues/182 the reaction ratelimit headers send 1/1s, when in reality the limit is 1/0.25s.
We should decide whether we want to implement an override for this behaviour.
Pros:

Faster reaction adding, as the limit is 4x higher than what discord says it is.

Cons:

Potentially dangerous should the limit ever change, although it's not likely at the moment.

Seems like a plan to me

I mean, once rate limits for reactions get updated people would get errors soon enough and it'll be reported

That... is a valid point. I'll consider all this.


Something went wrong

The above, in subject line, continuously reported, for every component of my GitHub account that I try to access through GitHawk

Bug Report Dump (Auto-generated)

Version 1.22.0<PHONE_NUMBER>)
Device: iPhone X (iOS 12.0)
TestFlight: false

@ccoberley when you say every component does that mean you don't see anything in your inbox and you can't search repos etc. Or do you mean you cannot comment on any issue or PR (.. other actions)?

Basically I just get that error when switching between all of the various issues in my inbox. When I go to open an issue I get the error

Did you review your permissions to make sure they are all good?
Settings -> review github access
Sent with GitHawk

I saw this also when I submit a new issue. The issue got submitted but I saw this error message. However, the issue got submitted.
Sent with GitHawk

Yes. Settings verified. Permissions all are good.


0.41 version just stops working at random

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.41
Python release (python3 --version):
3.5.3
Component/platform:
Docker image
Description of problem:
Hass stops working/stops responding/logging/etc. after a few hours. 0.4.0 with the exact config has no issues.
Expected:
Hass would work as expected
Problem-relevant configuration.yaml entries and steps to reproduce:

Upgrade to latest 0.41 docker image, SHA256: 4df256b0c039fd508daf0780fc139a1bf3b2ce9a4cbf0666f462013f406767bf
Start new container/image
Service will stop running a few hours later. so far twice I have logs for which range between 2 hours on one, 3 hours on other.
Stop this image, start older 0.40 image/container, no issues/downtime/etc.

Traceback (if applicable):

Additional info:

To add to this a little, I forgot i made a change to the config and did not restart HA. The only change that was made was adding a ffmpeg camera.  I turned that off and it worked again.  On one of my logs, that I do not have anymore, it said something to the tune of "FFMPEG is not running." before it stopped logging/working.


_enums_from_schema produces duplicate items

It looks like every item returned from the _enums_from_schema method in veris.py returns three copies of itself within the list:
v = VERIS('./../VCDB/data/json/validated')
v.load_schema()
schema = v._enums_from_schema(v.vschema)
pd.Series([schema_item['name'] for schema_item in schema]).value_counts()
to verify.

Interestingly, that's not necessarily the case.
However, I did notice that if you run your schema = v._enums_from_schema(v.vschema) again, it will +1 the counts for each item.
Which means we are appending each schema item to the list each time we call _enums_from_schema from the base state.
So, this is annoying and would be non-performant if for some reason we got into an infinite loop calling the v._enums_from_schema function. However, this doesn't end up causing a problem in the final data frame because when we create the data frame in json_to_df, we run the following:
        enum_list = self._enums_from_schema(self.vschema, '', [])
        self.enumerations = {item['name']: item['enumlist'] for item in enum_list if 'enumlist' in item}
        self.nonenum_vars = [item for item in enum_list if 'enumlist' not in item]

So duplicates will overwrite themselves. Again, not the most efficient, and certainly a mistake with my code, but also fortunately doe not appear to affect the final output.

Bizarrely, if I actually specify outlist=[] as a parameter in the function call, we don't get this 1-up behavior:
schema = v._enums_from_schema(v.vschema, outlist = [])
pd.Series([schema_item['name'] for schema_item in schema]).value_counts()

action.misuse.result                     1
cost_corrective_action                   1
timeline.compromise.value                1
......
schema = v._enums_from_schema(v.vschema, outlist = [])
pd.Series([schema_item['name'] for schema_item in schema]).value_counts()

action.misuse.result                     1
cost_corrective_action                   1
timeline.compromise.value                1

But if I don't specify outlist in the parameters on the function call, it does do the 1-up behavior. Even though the function definition is: _enums_from_schema(self, schema, curvarname='', outlist=[])
schema = v._enums_from_schema(v.vschema)
pd.Series([schema_item['name'] for schema_item in schema]).value_counts()

action.misuse.result                      12
timeline.incident.day                     12
reference                                 12
attribute.availability.notes              12
.....

schema = v._enums_from_schema(v.vschema)
pd.Series([schema_item['name'] for schema_item in schema]).value_counts()
action.misuse.result                      13
timeline.incident.day                     13
reference                                 13
attribute.availability.notes              13
action.malware.vector                     13

Why is the value of `outlist` preserved between successive function calls even if it is instantiated as an empty list? What am I missing here?

Tempted to close this issue because it doesn't affect the actual output, but fascinated to know why this is happening...

SO has the explanation: https://stackoverflow.com/questions/1132941/least-astonishment-and-the-mutable-default-argument

Per info here: http://www.omahapython.org/IdiomaticPython.html#default-parameter-values, instantiating curvarname and outlist to None fixes this issue. Thanks for noticing this and teaching me something new about Python!
Fixed in commit: https://github.com/RiskLens/verispy/commit/88c925e040cb2d49e8335d9b1ecbb117226970b9, will be available in next PR/version.


chore(package): update rollup to version 0.58.2

Closes #30

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Configure context with --config=force alters decider in Environment passed to Configure()

Version of SCons - Likely for as long as --config=force has been supported (looks like 10 years)
Version of Python - All
What Platform are you on? - All

When run with --config=force, scons wraps the existing decider to always return True. It makes this change to the Environment passed into Configure(env).  It never resets this change after configure is finished.
This can cause at least two symptoms:

Rebuilding all targets which use that environment
In some cases causing files to not be scanned and thus missing implicit dependencies (assuming flags/options were set to allow scanning)

Repro in this gist:
https://gist.github.com/bdbaddog/a93edd9134b3d5449503bf7ca82f5038
Also in attached archive
Archive.zip

Fixed in #3313


refactor: remove catch all style rule for border-color in dark mode

Remove catch all css rule overriding all border colors in dark mode.
Some bootstrap elements apply border-color: transparent to elements for a reason, e.g. buttons without borders.
These rules are overwritten by the custom rule included in index.css
:root[data-theme='dark'] * {
  border-color: var(--bs-gray-800) !important;
}

It has happened more than once now that this rule gets in the way rather than being helpful.
A benefit of this change is that elements are easier to use and more consistent across themes.
A drawback is that all new elements must take the border color into account.
Also, visual effects on the elements when removing the catch all rule:

Border color of input fields on validation (works in light mode)

valid input fields get a green border
invalid input fields get a red border

📸 Before

📸 After

(Notice the border colors of the input fields and buttons)
Hopefully all affected elements are covered with these changes.
What do you think? Does it have advantages or would it make things complicated again in the long run?
If these changes do not provide an benefit, simply close the PR.

all good so far, a redesign will fix the visibility later on :)

Thank you. Will merge after the scheduler PRs are included.


aleatoric  uncertainty loss

Hi, when I read your implementation, there are two problems confused me:

you mean log_loc_car has been passed through log function? If it is, where could I find the log computation in the code?
I confuse whether it needs to add activation function for var? Actually, I also encounter loss NaN issue, and there is no effective means to alleviate it, could you please give me some tricks?
Maybe you can help me in terms of these problems above.
Thanks in advance!

Hi,

you do not find any  log computation because you are regressing the log of the variance. More detail in this other issue https://github.com/pmorerio/dl-uncertainty/issues/1#issuecomment-506680509

I was not getting any NaN issue after removing activations.

Closing since duplicate and inactive for months.


Failed advection past east-west domain boundaries in global HYCOM

I am releasing particles in a global HYCOM simulation but I am encountering an issue where Parcels is not able to find the next position of particles that would be advected east or west past the domain boundaries. I am attaching a figure that shows the longitude values on the 7055 by 9000 grid of my simulation. In my case, particles are not advected across the 1 or 9000 X-positions in the Indian Ocean and I get a warning at execution. I expected Parcels to be able to deal with the cyclic boundary condition. I am defining my fieldset from NetCDF files that contains the velocity fields and the Latitude(Y,X) and Longitude(Y,X) arrays with coordinates Y from 1 to 7055 and X from 1 to 9000.
Is there something else I should specify explicitly in defining my fieldset or kernel? Any advice would be greatly appreciated!

It looks like there's a solution already, thank you @philippemiron for the pointer! https://nbviewer.org/github/OceanParcels/parcels/blob/master/parcels/examples/tutorial_periodic_boundaries.ipynb

The tutoral_periodic_boundaries.ipynb is a start but I don’t think it is readily applicable when (i) the longitude at the end of the domain varies as a function of latitude, and (ii) when the limits of the domain do not correspond to the min and max of the longitude (0,359) or (-180,179) as an example.
With respect to (i), in that same tutorial, the kernel periodicBC conducts a comparison between the particle longitude and a boundary longitude but in my case this fails because the boundary longitude, the halo I believe, is actually a vector ... and with respect to (ii), many particles will have longitude greater or less than the longitude at the boundary but will still be in the domain (because of the way it is organized as shown above). Would a solution be to track the X, Y indices instead of longitude?

Hi @selipot. I think that @VictorOnink has used global HYCOM fields for particle advection (also across the domain boundary) is his paper at https://iopscience.iop.org/article/10.1088/1748-9326/abecbd. Perhaps you can find some hints how he did it in his code at https://github.com/VictorOnink/Modeling-Global-Plastic-Beaching?

@erikvansebille Thank you for the pointer. Looking at the code I can't tell the type of HYCOM files used? @VictorOnink  could you elaborate? I am also seeing that the boundary condition is a simple comparison to +/- 180 longitude.

Hi @selipot, I used the HYCOM files from the GOFS 4.1 global reanalysis here: https://www.hycom.org/dataserver/gofs-3pt1/reanalysis. In this case, a simple comparison of +/- 180 degrees was sufficient to fix any BC issues I came across. I'm not entirely sure if I understand the issue you are running into, but if the problem is that your HYCOM files have longitude coordinates [0, 359], then same approach should also work.

Thank you for your reply @VictorOnink. As I tried to explain above and shown in the figure, the boundary longitude of the HYCOM grid I use is a function of latitude, and the east-west of the domain is do not correspond to (-180,180) or (0,360) but instead to approximately 47 degrees. So a kernel that compares the particle position to +/- 180 or (0,360) to wrap around the particles do not work.
I am thinking of a hack that would consist of tweaking the Fieldset to make the longitude constant at the boundaries and also offset all the longitudes to place them between -180 and 180. @erikvansebille do you know if I can modify the fieldset values at the setting up stage or does Parcels re-read the Latitude and Longitude field each time it accesses a new Netcdf file?
Alternatively I could try to recast all my HYCOM files on an offset and regular grid? That would require to know how to go from an irregular to a regular grid ...

Ok, then I get it I think. @selipot, have you tried maybe adding an additional lon/lat field to fieldset, which for each latitude would give you the longitude of the boundary? Your BC fixing kernel could then just sample what the boundary longitude is given the particle's latitude, and then use that to apply the BC.

@erikvansebille I am hitting a wall trying to implement my proposed solution. I am not even at the stage of implementing the cyclic boundary condition but trying to add a constant field to my fieldset to use for the boundary condition. I have put a notebook with the execution error at
https://github.com/selipot/hycom-unleashed/blob/master/particles_hycom_equator_to_share.ipynb
You'll see that I am trying to add a constant field to a fieldset containing global velocities from one set of files and global ssh from another set of files. Any suggestion or advice would be greatly appreciated!

Thank you @erikvansebille for finding a solution to this issue. The HYCOM output that I am using is such that the last longitude of the grid (to the east) is less than the first longitude of the grid (to the west) and the fieldset is lacking the needed data to wrap around (for my lack of better explanation), i.e lon[-1]<lon[0]. A solution that seems to work is to extend the grid to the east as follows:
lon_c = np.expand_dims(fieldsetH.U.grid.lon[:, 0], axis=1) fieldsetH.U.grid.lon = np.hstack((fieldsetH.U.grid.lon, lon_c))

Hi @selipot,
I ran into a similar problem using POP and I was wondering whether you did anything additional two the two lines you posted above. Do you still use an additional kernel as in the periodic boundary tutorial?

Hi @norafried , I did not add anything else and I do not have an additional kernel as in the periodic boundary tutorial.


initial docs on async method builders

Add and update the description of AsyncMethodBuilder, and the new feature that it can be applied to a method.
Fixes #24950
See issue #24972 to track the remaining task.
Notes on changes are on the issue: https://github.com/dotnet/docs/issues/24950#issuecomment-872541399

@tdykstra

I didn't see a high-level intro discussion that would explain under what circumstances / in what scenarios you need to use the builder attribute rather than just use a Task return type.

I'll address that as part of #24972


enable login via irc PASS command during handshake instead of PRIVMSG

this allows to just pass credentials as the server password in the irc client

Looks good, thanks for your work!

@dgruss I tried using this feature from irssi, but I couldn't get it to work. Could you tell me what syntax I need to get it to work?
Once I've got it working, I'll send a PR to document it in the README.

On 11.05.2018 02:36, keeperofdakeys wrote:

@dgruss I tried using this feature from irssi, but I couldn't get it to work. Could you tell me what syntax I need to get it to work?

Same syntax as the arguments of the login command.


Your bad forcer!!!!

Hello I catch you about unknown evasion again enabled AdBlock. I know now you are liar because you have given many German newspapers website like Bild.de, Suedkurier.de and more Why do you ruin my German newspapers websites. I will hold you if I have time to kill to fuckadblock.

Thanks I want unblock for Adblock because I am always sensitive to hidden spots of hackers or scammers want make dangerous adverts ( like embedded javascript or flash with hidden callback )
That is why it's really dangerous. I wish my computer stays safe.   Adblock holds again bad adverts...


built-in typescript support

Currently, TypeScript users have to install a separate package (@types/node-cron) to get type definitions. This PR adds built-in types copied from https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/node-cron/index.d.ts.
I believe this is a good idea for two reasons:

Breaking changes are easier to reflect in the types, since they're all in the same repo
TypeScript users no longer have to install a separate package. Great!

I'm currently working with typescript, and I think this is a good idea, no need to install another package for types support :)

@merencia any chance you can look into this anytime soon? having integrated TS types would be really nice and it's been 7 months since the PR was opened

+1

@merencia i've gone ahead and updated the types, as they hadn't been updated in more than a year. hopefully you can review this soon?


StreamService.receiveAudio/Video calls before StreamService.publish finshed

Hello, I have updated to Red5 Version 1.0.8. and I have now the follwing Porblem.
In my client (Flash App) I call this:
ns_publish = new NetStream(nc);
...
cam = Camera.getCamera();
ns_publish.attachCamera(cam);
ns_publish.publish(myUrl, "record");
In very rare cases, the method streamRecordStart is not called on the server side.
My investigation has shown that in this situations the method  org.red5.server.stream.StreamService.receiveVideo / receiveAudio is called before the call to the org.red5.server.stream.StreamService.publish method is completed.
At this time the initiation is not completed and the method streamRecordStart is not called.
Here some Infos from log:
2016-12-31 17:05:59,347 [RTMPConnectionExecutor-4] TRACE o.r.server.net.rtmp.BaseRTMPHandler - messageReceived connection: DVO5MCVH0OFMM
2016-12-31 17:05:59,347 [RTMPConnectionExecutor-4] TRACE o.r.server.net.rtmp.BaseRTMPHandler - Message received - header: Header [streamId=1, channelId=8, dataType=20, timerBase=4934, timerDelta=0, size=39, extendedTimestamp=0]
2016-12-31 17:05:59,347 [RTMPConnectionExecutor-4] TRACE o.r.server.net.rtmp.BaseRTMPHandler - Header / message data type: 20
start org.red5.server.stream.StreamService.publish with Thread RTMPConnectionExecutor-4
2016-12-31 17:05:59,347 [RTMPConnectionExecutor-4] DEBUG o.red5.server.stream.StreamService - publish called with name rec_id2 and mode record
2016-12-31 17:05:59,365 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - setPublishedName: rec_id2
2016-12-31 17:05:59,367 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - Provider connect
2016-12-31 17:05:59,368 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - Provider: org.red5.server.messaging.InMemoryPushPushPipe
2016-12-31 17:05:59,368 [RTMPConnectionExecutor-4] INFO  org.red5.server.scope.BasicScope - ClientBroadcastStream already exists: org.red5.server.stream.ClientBroadcastStream@7ef40385 new: org.red5.server.stream.ClientBroadcastStream@7ef40385
2016-12-31 17:05:59,368 [RTMPConnectionExecutor-4] DEBUG o.red5.server.stream.StreamService - Mode: record
2016-12-31 17:05:59,368 [RTMPConnectionExecutor-4] INFO  o.r.s.stream.ClientBroadcastStream - Stream start: rec_id2
2016-12-31 17:05:59,368 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - Provider connect
2016-12-31 17:05:59,369 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - SaveAs - name: rec_id2 append: false
2016-12-31 17:05:59,370 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - Created: org.red5.server.stream.RecordingListener@216713f0
2016-12-31 17:05:59,373 [RTMPConnectionExecutor-4] DEBUG o.r.server.stream.RecordingListener - Recording file: .....\streams\room\rec_id2.flv
start org.red5.server.stream.StreamService.receiveVideo with Thread RTMPConnectionExecutor-2 and
org.red5.server.stream.StreamService.publish is running
2016-12-31 17:05:59,373 [RTMPConnectionExecutor-2] TRACE o.r.server.net.rtmp.BaseRTMPHandler - messageReceived connection: DVO5MCVH0OFMM
2016-12-31 17:05:59,373 [RTMPConnectionExecutor-2] TRACE o.r.server.net.rtmp.BaseRTMPHandler - Message received - header: Header [streamId=1, channelId=7, dataType=9, timerBase=354, timerDelta=0, size=7327, extendedTimestamp=0]
2016-12-31 17:05:59,373 [RTMPConnectionExecutor-2] TRACE o.r.server.net.rtmp.BaseRTMPHandler - Header / message data type: 9
org.red5.server.stream.StreamService.publish is running
2016-12-31 17:05:59,374 [RTMPConnectionExecutor-4] DEBUG o.r.server.stream.RecordingListener - Context contains a file consumer
start org.red5.server.stream.StreamService.receiveAudio with Thread RTMPConnectionExecutor-1 and
org.red5.server.stream.StreamService.publish is running
2016-12-31 17:05:59,385 [RTMPConnectionExecutor-1] TRACE o.r.server.net.rtmp.BaseRTMPHandler - messageReceived connection: DVO5MCVH0OFMM
2016-12-31 17:05:59,386 [RTMPConnectionExecutor-1] TRACE o.r.server.net.rtmp.BaseRTMPHandler - Message received - header: Header [streamId=1, channelId=5, dataType=8, timerBase=4974, timerDelta=0, size=65, extendedTimestamp=0]
2016-12-31 17:05:59,386 [RTMPConnectionExecutor-1] TRACE o.r.server.net.rtmp.BaseRTMPHandler - Header / message data type: 8
org.red5.server.stream.StreamService.publish is running
2016-12-31 17:05:59,392 [RTMPConnectionExecutor-4] DEBUG o.r.server.stream.RecordingListener - File name: rec_id2.flv
2016-12-31 17:05:59,392 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - Codec info: org.red5.codec.StreamCodecInfo@1a71c2f8
2016-12-31 17:05:59,392 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - Video codec: org.red5.codec.SorensonVideo@73a5e4c
2016-12-31 17:05:59,392 [NioProcessor-4] TRACE o.r.server.net.rtmp.BaseRTMPHandler - Message sent
2016-12-31 17:05:59,392 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - Audio codec: null
2016-12-31 17:05:59,392 [RTMPConnectionExecutor-4] DEBUG o.r.s.stream.ClientBroadcastStream - No decoder configuration available, audioCodec is null.
org.red5.server.stream.StreamService.publish completed.
2016-12-31 17:05:59,418 [RTMPConnectionExecutor-3] TRACE o.r.server.net.rtmp.BaseRTMPHandler - messageReceived connection: DVO5MCVH0OFMM

This situation is caused by the parallel processing of the individual channels. Another possibility would be to process the received messages per stream. So the ChannelTask / Threads of a stream would not have to be synchronized when opening and closing the stream and it is only a little less parallelized.
I have made the following changes in my project and the described problem is fixed.
task_per_stream_patch.txt

Patch applied to 1.0.9-SNAPSHOT


Recent changes broke linux shared object linking (cdylib)

Just thought I'd like you know that d2c25a639905aefbe2bcbc3c5761faf5897977c4 broke the linking of my https://github.com/expenses/usd-iroh-resolver project. It's not a big deal, I can use fa760a565c2c1c84e25cc8704bfacd39107b7ef7 for now.
  = note: /usr/bin/ld: /home/ashley/projects/usd-http-plugin/target/release/build/bbl-usd-8d5f057e31a471f8/out/build/libopenusd-c.a(openusd-c.cpp.o): relocation R_X86_64_TPOFF32 against '__tls_guard' can not be used when making a shared object; recompile with -fPIC           /usr/bin/ld: failed to set dynamic section sizes: bad value           collect2: error: ld returned 1 exit status

Easily fixed by adding -DCMAKE_POSITION_INDEPENDENT_CODE=ON. I'll close this when I figure out how to add that as an environment variable.

I think we should just enable this by default for Linux.
On Sun, 21 Jan 2024 at 14:55, Ashley @.***> wrote:

Easily fixed by adding -DCMAKE_POSITION_INDEPENDENT_CODE=ON. I'll close
this when I figure out how to add that as an environment variable.
—
Reply to this email directly, view it on GitHub
https://github.com/anderslanglands/bbl-build-rs/issues/1#issuecomment-1902793810,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOYQXN5FAEO47L5Q4CQYJTYPWMHFAVCNFSM6AAAAABB2SK6VSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBSG44TGOBRGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Default value for vector type agent variables

Cannot currently specify the <defaultValue> for agent variables which correspond to vector types.
I.e.
<gpu:variable>
    <type>fvec3</type>
    <name>position</name>
    <defaultValue>0.0</defaultValue>
</gpu:variable>

It then does not validate against the xml schema with
<defaultValue>{0.0,0.0,0.0}</defaultValue>

or
<defaultValue>0.0,0.0,0.0</defaultValue>

This should be resolved prior to the 1.5 release.

This fix will need applying post #190 and #192 merge, as each branch effects the use vector types.

The main schema contains an unused simple type of type_type which looks like it was intended to restrict the types of variables to int, float and double, but has not been used in any of the git history.
This could potentially be used to limit the scope of defaultValue and also restrict variables to expected types (i.e float, fvec2 etc?)
https://github.com/FLAMEGPU/FLAMEGPU/blob/master/FLAMEGPU/schemas/XMML.xsd#L35-L50

We do not wish to restrict types - it would be hard to maintain a correct list of all allowed types
DefaultValue can no longer be a double, but must be a string or a regular expression.

For now, we use a regular expression which accepts numeric values and comma separated lists of numeric values.

Using an included xlst may need Visual studio projects (and .filters) updating.


is reactDOMServer loaded on the browser?

RenderReact imports reactDOMServer here: https://github.com/airbnb/hypernova-react/blob/master/src/index.js#L3
Your example bundles
var React = require('react');
var renderReact = require('hypernova-react').renderReact;

function MyComponent(props) {
  return React.createElement('div', {
    onClick: function () {
      alert('Click handlers work.');
    },
  }, 'Hello, ' + props.name + '!');
}

module.exports = renderReact('MyComponent.js', MyComponent);

into the rails application using sprockets.
Does this mean the client browser is always loading reactDOMServer?

@yinghaochan yes

@goatslacker Is this something that is supposed to be changed at some point? Loading server centralized code on the client does not seem to be that productive.

You can always alias out that import in your bundler.
I suppose we could lazy-require it?

Yep I guess lazy-requiring it would be a good solution, while aliasing it is also an option this pushes the work to the user IMO. I could submit a PR for it if you haven't done anything yet :)

While lazy requiring it is better than the current approach it's still a half solution...the bytes will still end up in the user's browser. How about we figure out a way to not require it at all?

Not if it was wrapped in a process.env check that could be uglified out.
That said, splitting the server and client code into separate entry points might be the better solution.

getting it started, should we break current implementations? -> https://github.com/airbnb/hypernova-react/pull/45

If this problem was solved hypernova could also support loadable components, which require some custom chunk loading logic where you need to require a file on the server.
But you can't have "path" imports etc on the client. I'm not sure how that can be achieved. The file splitting is a start, but it doesn't solve the actual problem of code sharing.


workflows: Sign using cosign (all)

workflows: Sign using cosign (all)

Add pubkey and instructions for cosign/sigstore signatures
Fixes: https://github.com/toolbx-images/images/issues/105

Let's give it a try


Pull-to-refresh on message list

We should provide the user with the pull-to-refresh gesture on the message list; similar to Gmail, Twitter, and many other apps with feeds of messages.
See #2712 for more detailed background and rationale.

I would like to work on this issue. However, it is unclear to me which component are you referring to by message list.
Do you mean Unread Cards, the screen which is shown on app startup with the list of streams?
Or do you mean the screen with a particular stream?
Or both?
Can you please clarify this?
@ray-kraesig since @gnprice is on vacation, and it may take him some time to respond, could you please answer this question, if you are aware of this?

The component found in MessageList.js – that is, the one which displays the list of messages of a stream or other narrow.
(Note that this component is an embedded WebView, which will make almost every subtask of this issue more complicated than it would otherwise be.)


Ability to select a tenant as well as subscriptions prior to data refresh.

My login has access to many tenants & subscriptions. It would be very helpful to be able to pick a tenant and subscriptions so I do not generate the report for tenants I do not need to review.
If there is a way to do this now, could you tell me how to go about this?
Thanks!

Currently due a technical limitations this is not possible using Power BI API requests this is not possible.
If you have permissions as a delegated resource manager, you can import manually a custom table in a CSV (with the subscription IDs) and create a new relationship to request the data from these filtered subscriptions/tenants.


Feature request: Local subtitles files support

I've been using it on the vlc player and I noticed that it doesn't load (or send, I don't know exactly how it works) local subtitles for vlc (probably other devices), just the embedded ones, so if this feature is added I think it will be the best DLNA server. I am amazed how it behaves using almost anything ram and cpu.

You can use MKVToolnix to combined movies and subtitles files.
There is a program called MKVBatch that automates the task and works an entire folder, but not subfolders.
If not on the web, check archive.org..


AWS Warnings about Node Version

AWS recently sent this email to all users with lambdas using Node.js 8.10:
We are contacting you as we have identified that your AWS Account currently has one or more Lambda functions using Node.js 8.10, which will reach its EOL at the end of 2019.
What’s happening?
The Node community has decided to end support for Node.js 8.x on December 31, 2019 [1]. From this date forward, Node.js 8.x will stop receiving bug fixes, security updates, and/or performance improvements. To ensure that your new and existing functions run on a supported and secure runtime, language runtimes that have reached their EOL are deprecated in AWS [2].
For Node.js 8.x, there will be 2 stages to the runtime deprecation process:

Disable Function Create – Beginning January 6, 2020, customers will no longer be able to create functions using Node.js 8.10

Disable Function Update – Beginning February 3, 2020, customers will no longer be able to update functions using Node.js 8.10

After this period, both function creation and updates will be disabled permanently. However, existing Node 8.x functions will still be available to process invocation events.
What do I need to do?
We encourage you to update all of your Node.js 8.10 functions to the newer available runtime version, Node.js 10.x[3]. You should test your functions for compatibility with the Node.js 10.x language version before applying changes to your production functions.

Can this be addressed in configuration? Is there a way to update this without redeploying? If I do redeploy will it mess up the existing configuration?

I run npm install node-lambda@latest before deploying. This would update the node-lambda module which now uses Node 10 by default. I'm sure there's a better/more flexible way to do it. But that was quick and painless.

@mherger Thanks. Worked for me.


Turn on discussions

It might be worth turning on discussions.  It would be helpful to discuss performance improvements so there is a history of what people have tried and any benchmarks run.

Done!


Release Google.Cloud.Logging.V2 version 4.1.0

Changes in this release:
New features

Log Analytics features of the Cloud Logging API (commit 5a7f88f)
Add ConfigServiceV2.CreateBucketAsync method for creating Log Buckets asynchronously (commit 5a7f88f)
Add ConfigServiceV2.UpdateBucketAsync method for creating Log Buckets asynchronously (commit 5a7f88f)
Add ConfigServiceV2.CreateLink method for creating linked datasets for Log Analytics Buckets (commit 5a7f88f)
Add ConfigServiceV2.DeleteLink method for deleting linked datasets (commit 5a7f88f)
Add ConfigServiceV2.ListLinks method for listing linked datasets (commit 5a7f88f)
Add ConfigServiceV2.GetLink methods for describing linked datasets (commit 5a7f88f)
Add LogBucket.analytics_enabled field that specifies whether Log Bucket's Analytics features are enabled (commit 5a7f88f)
Add LogBucket.index_configs field that contains a list of Log Bucket's indexed fields and related configuration data (commit 5a7f88f)

Documentation improvements

Documentation for the Log Analytics features of the Cloud Logging API (commit 5a7f88f)

Created release for Google.Cloud.Logging.V2-4.1.0


conductor reader validates SequencerNamespaceData before checking that block contains relevant namespace txs

as title says, this is because the Reader doesn't contain the namespace the conductor cares about, but the Executor does, and checks whether the block should be executed. this should be changed so we aren't validating irrelevant blocks.
note: this may not be relevant, and depends upon our design/relation of the conductor to a tendermint light client; if we run a light client internally to the conductor (ie. track the validator set) then we need to keep this, but if we assume we have a trusted external client as we do right now, we can change this.

This enables execution of empty blocks. Keeping this.


[AutoPR monitor/resource-manager] Swagger for diagnostic settings and association resource types

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3295

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/2139


what is the embedding txt?

First replace  in preprocess_embedding.sh with real path.Where is the embeddig txt ,I mean there are lots of txt in this  program ,I don't know what should I put in there.

I also want to know, do you resolve it?

I resolve it, the embedding file "glove.840B.300d.txt" is  a word vector word embedding file, you can download here:
https://www.kaggle.com/takuok/glove840b300dtxt


Dashboard to show the health status of the indexers

Is your feature request related to a problem?
It would be nice to have a route like /health which pings:
1. indexers

magick

rubick

unique

snek

2. services/workers
If they are working as they should
Describe the solution you would like
Some dashboard like UI (mobile friendly ofc)
Blocking
I need to implement some of this checking features to the workers and find out these metrics for subsquid

How it's different from

#1492

Nice was looking for that
Closing in favour of #1492


Engine Addition: OscProb

Pull request description:
Add the OscProb engine into NuOscillator
Changes or fixes:
Add OscProbCalcer_OscProb.{cpp,h} into the calculators
Examples:

OscProb implementation in NuOscillator.pdf
Description of OscProb engine implementation, how to use it as part of MaCh3 and overview of the validation process


Add breadcrumbs

Fixes #85

Unfortunately I think that the header levels are off by one, I'll open a new PR to fix that


Light bar no longer "rainbows" while charging.

I don't know what I did or what happened, but my light bar no longer "rainbows" while charging.
I uninstalled, restart, and reinstalled SCP/DS4tool, still no "rainbow".
I have attached my Default profile, just remove the ".txt" at the end.
Default.xml.txt

Be sure that you set the color to the profile color.

Hello, Heres a fully configured rainbow code replace your xml files txt with this:

True
0
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
100
False
0
100
100
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
True
False
0
False
0
0
0
25
1
0
0
0.25
0.25
1|1|1|1|1|1
2
True

False
False
False
False
-1
100
0
0
0
Disconnect Controller

117

replace your color values with these
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
148,0,211,75,0,130,0,0,255,0,255,0,255,255,0,255,127,0,255,0,0
All u do is create a RGB Code for it :) simple search


'network' issue when has special character

I perform a withdrawal with MxcSpotV3 ->privates()->withdraw($data) and everything work fine except if I try to do anything with $data = ['network' => 'BEP20(BSC)'] or any network that contains special characters, like SPACE, or ' - '.
It made the query URL to be differ than expected:
API server expected: /api/v3/capital/withdraw/apply?coin=USDT&address=zzqqqqqqqqqq&amount=10&network=BEP20(BSC)&memo=MX10086&timestamp={{timestamp}}&signature={{signature}}
The real one with the mxc repo: /api/v3/capital/withdraw/apply?coin=USDT&address=zzqqqqqqqqqq&amount=10&network=BEP20%28BSC%29&memo=MX10086&timestamp={{timestamp}}&signature={{signature}}
It seems it can't convert these characters correct when pushing to the API servers. If I use the 'network' with simple type, such as ERC20, OP, MATIC, then there is no issue at all.
Can you take a look at this?

After spending a short time to figure out, it seems the error message would be:
{"code":700002,"msg":"Signature for this request is not valid.","_method":"POST","_url":"https://api.mexc.com/api/v3/capital/withdraw/apply","_httpcode":400} {"userId":"tony","exception":"[object] (Lin\Mxc\Exceptions\Exception(code: 0): {"code":700002,"msg":"Signature for this request is not valid.","_method":"POST","_url":"https:\/\/api.mexc.com\/api\/v3\/capital\/withdraw\/apply","_httpcode":400}
Somehow, if use special characters in query URL, it didn't work and led to this 'signature' error.
Worked version: coin=USDT&network=EOS&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
Didn't work: coin=USDT&network=EOS%28TEST%29&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231859000
I tried to build query URL manually but didn't work too: coin=USDT&network=EOS(TEST)&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
To clarify that your repo didn't have withdraw() function, I created a new one in SpotV3\Privates:
/*
*POST /api/v3/capital/withdraw/apply

*/
public function withdraw(array $data=[]){
$this->type='POST';
$this->path='/api/v3/capital/withdraw/apply';
$this->data=$data;
return $this->exec();
}

https://mexcdevelop.github.io/apidocs/spot_v3_en/#withdraw

After spending a short time to figure out, it seems the error message would be:
{"code":700002,"msg":"Signature for this request is not valid.","_method":"POST","_url":"https://api.mexc.com/api/v3/capital/withdraw/apply","_httpcode":400} {"userId":"tony","exception":"[object] (Lin\Mxc\Exceptions\Exception(code: 0): {"code":700002,"msg":"Signature for this request is not valid.","_method":"POST","_url":"https://api.mexc.com/api/v3/capital/withdraw/apply","_httpcode":400}
Somehow, if use special characters in query URL, it didn't work and led to this 'signature' error.
Worked version: coin=USDT&network=EOS&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
Didn't work: coin=USDT&network=EOS%28TEST%29&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231859000
I tried to build query URL manually but didn't work too: coin=USDT&network=EOS(TEST)&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
To clarify that your repo didn't have withdraw() function, I created a new one in SpotV3\Privates:
// POST /api/v3/capital/withdraw/apply // public function withdraw(array $data=[]){ $this->type='POST'; $this->path='/api/v3/capital/withdraw/apply'; $this->data=$data; return $this->exec(); }
https://mexcdevelop.github.io/apidocs/spot_v3_en/#withdraw

Signature issues, special characters, may all require no encoding, I tried to solve this problem.

After spending a short time to figure out, it seems the error message would be:
{"code":700002,"msg":"Signature for this request is not valid.","_method":"POST","_url":"https://api.mexc.com/api/v3/capital/withdraw/apply","_httpcode":400} {"userId":"tony","exception":"[object] (Lin\Mxc\Exceptions\Exception(code: 0): {"code":700002,"msg":"Signature for this request is not valid.","_method":"POST","_url":"https://api.mexc.com/api/v3/capital/withdraw/apply","_httpcode":400}
Somehow, if use special characters in query URL, it didn't work and led to this 'signature' error.
Worked version: coin=USDT&network=EOS&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
Didn't work: coin=USDT&network=EOS%28TEST%29&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231859000
I tried to build query URL manually but didn't work too: coin=USDT&network=EOS(TEST)&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
To clarify that your repo didn't have withdraw() function, I created a new one in SpotV3\Privates:
// POST /api/v3/capital/withdraw/apply // public function withdraw(array $data=[]){ $this->type='POST'; $this->path='/api/v3/capital/withdraw/apply'; $this->data=$data; return $this->exec(); }
https://mexcdevelop.github.io/apidocs/spot_v3_en/#withdraw

https://github.com/zhouaini528/mxc-php/blob/9e733bcf5b1a77be9bc7422ec72df317957f0e39/src/Request.php#L147

The post submission data is in the form_params way, and the special characters in it are not encoded, why is your submission data a GET URL splicing?

I discussed to one of MEXC supporter on this issue and he said:

the URL data: http_build_query($this->data) need to be encoded, so use http_build_query is satisfy the requirement.
the POST URL doesn't need encode

The post submission data is in the form_params way, and the special characters in it are not encoded, why is your submission data a GET URL splicing?
This is the log version of http_build_query($this->data) in signature() function, not the POST URL.
coin=USDT&network=EOS&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
I still can't pass this annoying issue, maybe the error came from their API server?

I discussed to one of MEXC supporter on this issue and he said:

the URL data: http_build_query($this->data) need to be encoded, so use http_build_query is satisfy the requirement.
the POST URL doesn't need encode

The post submission data is in the form_params way, and the special characters in it are not encoded, why is your submission data a GET URL splicing?
This is the log version of http_build_query($this->data) in signature() function, not the POST URL.
coin=USDT&network=EOS&address=XXX&amount=100&withdrawOrderId=TEST&timestamp=1701231754000
I still can't pass this annoying issue, maybe the error came from their API server?

I found a way you can try
echo $url=urldecode(http_build_query($arr));

Actually, I tried to build the query string for signature manually with/without encode and it still didn't work. MEXC also has recently changed the list of withdrawal networks into special character mode. Right now, most of networks have special characters in them, except several ones, such as EOS, OKT ...
So withdraw asset through API seems not possible at this moment.


Include tests in PyPI tarball

In OpenBSD we use the regression tests in order to make sure that updates
work properly and that an update of a dependency doesn't break a package.
Having the regression tests in the PyPI tarball makes that much easier.
Fixes #26

Makes sense, thank you for the PR.


Time to complete function

Add a time to complete print which calculates the time to completion

wait_time function has been added


update .21 core app insights sample with core bot 4.5.1 changes

Updates App Insights sample with the changes made to Core Bot in 4.5.1
Set TelemetryClient property on MainDialog / BookingDialog in startup.cs as opposed to accepting via constructor
Pass the TelemetryClient to the Luis Recognizer

Program.cs does not show up as one of the files that has changed, however if I compare this PR with sample 13.Core-Bot in the master branch, the 2 copyright lines in the comments at the top of the file are missing.

@WashingtonKayaker good catch. I'll add it.

@WashingtonKayaker ah yes. Great. I'll hopefully update tomorrow and let you know once done. 👍

using Microsoft.ApplicationInsights.Extensibility;

Insert (C) hader

Refers to: samples/csharp_dotnetcore/21.corebot-app-insights/Startup.cs:1 in 9eddaca. [](commit_id = 9eddacaad99ad5b60df76331024816426615f115, deletion_comment = False)


rename from common service operator to meta operator

Manually test on OCP 4.2, no errors found for now.
/assign @gyliu513 @DanielXLee @horis233

/lgtm
/approve


Usage of TSDF

Hello. Please point out in which file you have implemented TSDF-based volumetric rendering.
Thank You

We implemented TSDF-based volumetric rendering in our CUDA implementation, not in this pytorch version.

Have you released a public version of your CUDA implementation?
Would like to enquire the same about the neural tracking part mentioned in the research paper.

Hi,
If this pytorch version does not have the TSDF-based volumetric rendering, I am wondering how come there isn't the numerical stability issue? I did manage to get good results with the default network_sdf on the dance dataset you provided.


Return the fixture object when adding http fixtures

This is a convenience for cases where one wants to hang onto the fixture after creating it so that can assert things about it afterwards.

Ready for review. Good idea?

:+1:


Docs page has lost its sidebar

https://repo2docker.readthedocs.io/en/latest/ now doesn't show a menu/sidebar anymore and seems to miss the title below the logo.

cc @choldgraf the sphinx templates level 11 wizard 🧙♂

noooo ok will try to debug this


Adds builder pattern

Codecov Report

Merging #4 (8b28ca6) into main (4b52ad4) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   71.11%   71.11%
=======================================
  Files           2        2
  Lines          45       45
=======================================
  Hits           32       32
  Misses          8        8
  Partials        5        5

Files
Coverage Δ

generate/generate.go
57.14% <ø> (ø)


Undo (#60)

Perhaps these should be squashed (git rebase -i).

This PR depends on this #103 (See Cmd enum introduced there).
Do you suggest that it is possible to create a PR based on the branch of another PR ?

I don't know if it's possible, but ss far as I can tell the commits after 5539931 can be squashed into one, and I suggest the commits from #94 be squashed. It's hard to follow this many commits. for a single feature (Vi Mode as I understand it).

Owner/collaborator(s) can choose between merging/squashing/rebasing when closing the PR, no ?

Sure, it's just for myself so I can browse the code here instead of pulling your fork, squashing, and then diffing.


new: Write a Get Involved/Request New Features doc

https://athensresearch.gitbook.io/handbook/community/get-involved/learning-at-athens-academy-education/request-new-features

https://github.com/athensresearch/handbook/blob/main/community/get-involved/learning-at-athens-academy-education/request-new-features.md

Do we have a resource that gives guidelines on what makes a good or helpful feature request?

From here: https://github.com/athensresearch/athens/discussions/826#discussioncomment-778473
This document should definitely detail what makes a good feature request.


Remove auto version update on lambda startup

Summary
Currently we use a proxy for outbound traffic, and we allow specific domains which can be accessed for security reason. Every time that the lambda starts, it tries to pull the latest version from github.
Adding github to our whitelisted domains can result in a risk which we dont want to present.
The error is:
[NR_EXT] There was an issue querying for the latest agent version: Get "https://github.com/newrelic/node-newrelic/releases/latest": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
Possible Solution
Please add an environment variable to control if version pulling is required or not on lambda startup.

This was addressed in #97. Let us know if you continue to have issues.


[Bug]: ProSkillApi Error [RoyaleEconomy]

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
ProSkillApi does not work and does not run when RoyaleEconomy is installed.
or RoyaleEconomy not working with ProSkillApi installed...
The server is shutting down.
Expected Behavior
I want the plugins to be compatible with each other.
Steps To Reproduce
Environment
- Server Software: 1.16.5 Paper
- ProSkillAPI Version: <IP_ADDRESS>
- ProMCCore Version: <IP_ADDRESS>
- Plugins:
  -ProSkillApi
  -ProMcCore
  -RoyaleEconomy

### latest.log

 https://pastebin.com/pbVxCaMv

### Anything else?

_No response_

I believe this to be caused by the fact that we were shading ProtocolLib into our plugin when building. The shading has been removed as it was only needed for LibsDisguises, which requires it in the first place. New builds should show this resolved.


Update build shield in readme.md

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (1288b2b) compared to base (2141e7d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        13
  Lines          209       209
=========================================
  Hits           209       209

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.


Separate Documentation into Developer, Deployment, and Readme

This makes the readme more simple by just showing the status of this repo and linking to the other appropriate docs. The developer guide holds most of the info formerly in the readme with the exception of the deployment and troubleshooting steps. Those are moved over to the new deployment guide. Each has links to the other in order to allow simple navigation. Additionally, the readme has a subsection for quick start under deployment that says exactly which sections the user should go to without any extra info.

This change is

Awesome, great to see it in smaller chunk.  :+1:


PyTorch API updates

fix train/fit confusion
bump version to 0.2.1 since the API is changed.
GPU fixes
enable more tests

fixes https://github.com/AliciaCurth/CATENets/issues/19

Just to check, does this actually get rid of all API inconsistencies? (Looking at @kamilest's original issue, she mentioned that torch functions need to be  in 'train-mode' --> is this now the case during the 'fit' call? I.e. is train mode the default behaviour? If not, that probably needs to be fixed? If on the other hand train mode is the default, then the predict functions probably all have to be set into eval mode to remove dropout etc, correct?)

Just to check, does this actually get rid of all API inconsistencies? (Looking at @kamilest's original issue, she mentioned that torch functions need to be in 'train-mode' --> is this now the case during the 'fit' call? I.e. is train mode the default behaviour? If not, that probably needs to be fixed? If on the other hand train mode is the default, then the predict functions probably all have to be set into eval mode to remove dropout etc, correct?)

The original issues was that the APIs were shadowing the train method inherited from the nn.Module. By default, self.model is in train mode, you need to call train() only if you re-train it. But we can explicitly call .train() on every fit() anyway.


[Backport] Kafka consumer improvements to 7.46.x

What does this PR do?
[Backport] Kafka consumer improvements to 7.46.x
Motivation
AGENT-9947
Additional Notes

Excluded bumping confluent-kafka version to avoid updating the agent dependencies
Kept the integration to use python3.8

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

No longer required.


[Bugfix] Add hive external table counter

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
https://github.com/StarRocks/starrocks/issues/8483
Problem Summary(Required) ：

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 25 / 35 (71.43%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
4
00.00%

:large_blue_circle:
com/starrocks/external/hive/events/MetastoreEventsProcessor.java
0
4
00.00%

:large_blue_circle:
com/starrocks/external/hive/HiveRepository.java
25
27
92.59%

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 23 / 32 (71.88%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
3
00.00%

:large_blue_circle:
com/starrocks/external/hive/events/MetastoreEventsProcessor.java
0
4
00.00%

:large_blue_circle:
com/starrocks/external/hive/HiveRepository.java
23
25
92.00%

@Mergifyio backport branch-2.3

[FE PR Coverage check]
:disappointed: fail : 23 / 32 (71.88%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
3
00.00%

:large_blue_circle:
com/starrocks/external/hive/events/MetastoreEventsProcessor.java
0
4
00.00%

:large_blue_circle:
com/starrocks/external/hive/HiveRepository.java
23
25
92.00%

[FE PR Coverage check]
:disappointed: fail : 23 / 32 (71.88%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
3
00.00%

:large_blue_circle:
com/starrocks/external/hive/events/MetastoreEventsProcessor.java
0
4
00.00%

:large_blue_circle:
com/starrocks/external/hive/HiveRepository.java
23
25
92.00%

[FE PR Coverage check]
:disappointed: fail : 23 / 32 (71.88%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
3
00.00%

:large_blue_circle:
com/starrocks/external/hive/events/MetastoreEventsProcessor.java
0
4
00.00%

:large_blue_circle:
com/starrocks/external/hive/HiveRepository.java
23
25
92.00%

[FE PR Coverage check]
:disappointed: fail : 23 / 32 (71.88%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
3
00.00%

:large_blue_circle:
com/starrocks/external/hive/events/MetastoreEventsProcessor.java
0
4
00.00%

:large_blue_circle:
com/starrocks/external/hive/HiveRepository.java
23
25
92.00%


Replacing exif-js

Hello,
I noticed you're having troubles with the dependecy exif-js (#15) which contains bugs but is no longer maintained (for over two years now).
I'd like to offer you a help with that. I'm a developer of another library exifr. It's small, fast, configurable (so just a few core files can be used to extract the photo's orientation instead of using the whole library), supports more formats (tiff, heic), has fixes for bugs known to be in exif-js, etc...
I'd be happy to do a Pull Reqest swapping exif-js for exifr if you're open to that.
Thank you

@MikeKovarik did he ever allow you to do a pull request?

All of my images uploaded through my iPhone are turned and I am assuming it is do to the exif

@jsd219 no. Otherwise this issue would be closed. The problem you're referring to is a new behavior of safari on iOS (and soon to be in chrome as well). https://github.com/MikeKovarik/exifr/issues/25 fortunately exifr can work with it. Unfortunately this package doesn't use exifr

Also this https://mutiny.cz/exifr/examples/orientation.html
But it's broken on chrome 81 and newer because they just implemented the same behavior since I released last version. It will be fixed soon

@MikeKovarik Any chance you can fork off of this and implement it yourself? This is a great plugin but unfortunately it appears the developer is not very active on it.

@jsd219 unfortunately not. I would be happy to do a PR to this repo if the author was active and willing to accept it. But I do not want to fork and maintain this repo. I don't want to step on anyone's toes, nor would I have enough time to maintain it.

Sorry, unfortunately I've not had time to actively maintain the repository lately.
I initially was a bit hesitant  about switching to another exif-lib, but that was probably an overly defensiv and passiv judgement call on my side.
Please @MikeKovarik , if you still feel up to it, I would be very happy to accept a PR that implements your lib.  We'll merge it into a new major version release to safeguard against any unforeseen breaking changes.
Thank you for the patience!
(cc @jsd219 )

I created a PR #49 to hopefully help move this forward. We are having trouble with #41 and I was hoping this would resolve it, but it does not appear to.
@svale and @MikeKovarik I wonder if you could take a look?


Table pager buttons cause page to refresh when inside form tag

Describe the bug (actual behavior)
Clicking the table pager buttons causes the page to refresh when the table is inside a form tag.
Expected behavior
Clicking the page buttons changes pages without refreshing the page.
Reproduction
Link to a minimal repro: https://codesandbox.io/s/paging-page-refresh-w0bpoh?file=/App.tsx:2606-2617
Steps to reproduce

When the page loads, click page "2" and notice the page refreshes instead of changing pages

Additional information
This issue exists in<EMAIL_ADDRESS>but not in 1.x. Comparing the html output, 2.x does not include type="button" for the paging buttons.

Hey @mattl-bentley. That makes sense the button needs type='button', although I would question putting the whole table in a form; it can cause issues with some screen readers)
Do you want to contribute by creating a pull request? The change would go here: https://github.com/iTwin/iTwinUI/blob/4e43e5ca488d998bd7920c9b30fa85e82d65eeb5/packages/itwinui-react/src/core/Table/TablePaginator.tsx#L173-L175

this is now released in 2.4.1 🎉

Great, thank you!


Structure public API for ngrx actions for routing module

See master ticket: #3318
Depends on #3329

PR: #3357


Configuration#2: Step#2: Merging each 100K FinalHadronState Files into one File including 600K: MATTERLBT

singularity-events-merger-600k.sh
CONFIG_NUMBER=2
ELOSS_TYPE_UPPERCASE="MATTERLBT"
ELOSS_TYPE_LOWERCASE="matterlbt"

done without job output


react-router. Expose Navigation and State from the Router.

react-router/react-router.d.ts
to authors(@mrk21 @vasek17). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

It was not really a valid change.


Add geoideal as core contributor

Move geoideal to core contributors, @gacarrillor has been actively contributing code to the QGIS and environment repositories for a long time. Note: I did not find exact requirements for the core contributor section.

Thank you both! This means a lot for me and for Geoideal. :rocket: