Arms assistant causing crashes (due to rendering maybe?) [Bug]

Full log is here: https://paste.ee/p/Qi9Ma
Attempting to use the Arms Assistant causes crashes, this isn't a new issue, though I'm pretty sure it bounced between fixed and broken again several times on the Domochevsky releases. I'm not sure exactly how those crashes occurred there so this might be distinct.
System details: Intel i5 7th gen, 4GB ram min 11GB ram max (it's server and host running on the same machine so I need the extra RAM), Intel integrated Graphics 620 chipset. It's possible that integrated graphics are the issue, but I've never had any issue before.

This bug was caused by a change in Java 9 that isn't binary compatible with Java 8. Due to the nature of the change, the source is Java 8 compatible, however compiling with Java 9 or greater results in a binary that is incompatible with Java 8. I've setup the buildscript to force use of Java 8 when compiling, so this kind of bizarre issue can't happen again.
If you wish to use Arms Assistants before the next release, updating to Java 9 or later will allow you to use the existing releases. Do be aware that they're still a bit buggy due being completely rewritten. The next release will fix several of those bugs, as well as adding in-game documentation via Patchouli.


is IPv6 supported ?

hi,very nice tool,
work fine with IPv4 host but throw an error when I try to connect IPv6 host
error:
cannot connect to host: dial tcp [2a02:8429:614a:6901:ba27:ebff:xxxxx:xxxxx]:22: connect: cannot assign requested address

Hi,
Thank you for your question and the feedback.
Currently IPv6 is not supported indeed.

will it be supported in the future?

yes, it's planned, but timeline is not clear yet.


command line option to start and connect to a host

The one feature I feel this is missing is the ability to call it from the command line with a username, host, password and port.  I use KeePass to store my passwords, and so having it launch terminus with those details would be very useful.
For example: terminus.exe ssh user@host:port password

Is there any workaround with the "run" command?
I cannot start any command with the cli run 👎
I could also live with the format: terminus.exe run "ssh user:password@host:port"

The new commands are:
  terminus connect-ssh [connectionName]     open a tab for a saved SSH
                                            connection
  terminus connect-serial [connectionName]  open a tab for a saved serial
                                            connection


feat(events): fix responsiveness of events page

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#91
#90
#89
#88  👈
#86
#85
#84
#83
#82
#81
#80
#79
main

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

Join @xdjeremy and the rest of your teammates on  Graphite


Samsung j-700HXXU

Most of people in Egypt and Other countries use this phone please Make Kernel for this mobile
or learn us how to make kernel.

kernel source for GT-E700HXXU (Samsung Galaxy E7 Duos): http://opensource.samsung.com/reception/receptionSub.do?method=sub&sub=F&searchValue=E700HXXU
how to build lineageos from source (including the kernel) for e.g. Samsung Galaxy S 2 (gt-i9100): https://wiki.lineageos.org/devices/i9100/build . Take out any relevant information you need from this tutorial (learn it)
compile the kernel from the first link with a suitable patch (check Makefile).

try this one first: https://github.com/pelya/android-keyboard-gadget/blob/master/patches/existing_tested/by-manufacturer-and-soc/motorola__msm8916______kernel_3.10.patch
or these ones (depending on your kernel version):

if 3.1: https://github.com/pelya/android-keyboard-gadget/blob/master/patches/existing_tested/by-generic-kernel-version/generic_kernel_3.01.patch
if 3.4: https://github.com/pelya/android-keyboard-gadget/blob/master/patches/existing_tested/by-generic-kernel-version/generic_kernel_3.04.patch
if 3.18: https://github.com/pelya/android-keyboard-gadget/blob/master/patches/existing_tested/by-generic-kernel-version/generic_kernel_3.18.patch

To apply the patch: execute $ patch -p1 < $FILENAME_OF_PATCH_FILE.patch from the root of your kernel directory.
make the kernel afterwards with suitable parameters (check README).


Issue rendering data frame with Safari

I canot render a simple dataframe with safari, even the overview on the official website.

It works well with Chrome.

With the Shinylive app, when you open the JavaScript console, does it show any errors? (Click on Develop in the menu bar, then Show JavaScript Console.)
This is what it looks like for me:

Thanks

Can you list the Safari version that you are using Safari > About Safari ? Thank you.

Wow, that is very strange. I wonder if Safari somehow received and cached a broken build of dataframe.js. I have reset the cache on our hosting provider. Please try clearing your browser cache and reloading that app.

Can you list the Safari version that you are using Safari > About Safari ? Thank you.

Tested it on an older version of Safari v16.6 and it seems to be working fine.

Actually, I experience the same problem on a Windows machine with Edge and unfortunately also Chrome.
I have the idea that when I start the application, some libraries are downloaded when running the first time.  I am stuck behind a very restrictive firewall.  pip for example needs the ssl workaround or I can't download.  I suppose it could be with that ?


TenableIO not showing up in solutions

I can see that there is a Solution for Tenable.IO in Github: https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/TenableIO but the solution isn't available in Sentinel's content hub. I noticed that there isn't a ReleaseNotes.md like the other solutions (For example the Tenable App Solution):

Is it supposed to appear as a solution or is it missing some other requirements?

Hey @leighcurranTW, the new release of solution is here- https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/Tenable App, So kindly search for the solution name "Tenable App" in content hub.

Thanks!

OK thanks for the update.

@leighcurranTW, Welcome. so, closing this issue. If you still need support for this issue, feel free to re-open it any time. Thank you for your co-operation.


"Freezes" in docker when taking screenshot

Description
Running puppeteer-real-browser inside docker. Docker image is based on Ubuntu 22.04
Code freezes for 30 or so seconds when taking screenshot and then fails with protocol timeout error. The same error can also happen while scrolling down when the page is long and contains a lot of images.
After a lot of digging in I finally found out that this is caused by different behaviour of /dev/shm usage in puppeteer vs puppeteer-real-browser.
I solved this by adding --disable-dev-shm-usage flag to chrome launching.
	const { browser, page } = await connect({
		headless: true, turnstile: true, disableXvfb: false,
		ignoreAllFlags: false, customConfig: {
			chromeFlags: [
				'--no-first-run',
				'--disable-dev-shm-usage'
			]
		}
	});

The alternative solution is to allocate more shm in container start, but I guess using --disable-dev-shm-usage is better option. Or is shm needed to hide that we are a bot?
The same problem does not happen in std puppeteer as it uses --disable-dev-shm-usage flag by default when launching chrome.
Not sure if this is a bug or just a fyi for others.... but anyway, it would be nice to know which one is better solution
Full steps to reproduce the issue
Change any of the tests https://github.com/zfcsoftware/puppeteer-real-browser/blob/main/test/esm/test.js to download some real page which have at least dozen  images
Scroll to the bottom in steps to get the lazy images to download:
	let height = 12000;
	let scroll = 0;
	while (scroll < height) {
		const scrollHeight = getRandomInt(10, 1200);
		await page.mouse.wheel({ deltaY: scrollHeight });
		scroll += scrollHeight;
		await Utils.delay(getRandomInt(50, 400));
	}

and then take screenshot
      const shot = await page.screenshot({ type: 'jpeg', fullPage: true });
=> throws Protocol timeout error while the real reason is "out of shm"
Issue Type
Bug
Operating System
Linux
Do you use Docker?
Docker

Thank you


removed compiler warnings

substituted deprecated range form  "..." with "..="
removed try!
removed unused argument in println!

Thanks for the contribution!
The fix for this same issue was already included in a large lint and CICD fixup PR (#1449).
We'd be happy to see you back with new contributions in the future.


Implement Middleware support

We need Middleware in two differents area:

EventStore (for upcasting, monitoring, ...)
EventBus (for event/metadata enrichments, complex routing, react on event metadata after publishing, monitoring, ...)

The implement must be as lean as possible and explicit. So no AOP in the area, and personnaly I don't want to use the Signal/Slot, I don't see Signal/Slot as a nice way to change object state.
Fully convention based, so no configuration. Create a class, implement an interface and you should be good.

Just a few notes on the use cases for EventBus middleware:

"for event/metadata enrichments" - that should happen pre commit (EventStore), else this enriched metadata is only transient

"complex routing" - shouldn't (complex) routing rather be handled by a specific EventBus implementation?

"react on event metadata after publishing" - for only reacting one should use an EventListener, the middleware is mainly used for actually changing the events/metadata and IMO changing them after publishing is a bit strange, no?

Sorry for being the skeptical voice once again, but I feel that a generic middleware approach is over-sophisticated here. Obviously I agree to the scenarios you mention and IMO we should create tickets for these rather than for the implementation, e.g.: Event Upcasting (#94) and Metadata enrichment (#88).
Both could be addressed with a very simple solution, for upcasting we might not even need custom support (see ticket).
Re metadata enrichment: After thinking about it again and talking with @robertlemke I realized that we only need to enrich it just before it is added to the EventStore and that could be done via Signal/Slot (btw: the version should not be part of the metadata).
I'm working on a PR to see if that works out and as base for further discussion

In our use case, maybe it's edge case, but it's real life ;)
We handle most event locally (sync) but all local events are forwarded to Nats to be available for external service (basically fire and forget) and infrastructure needs (external archiving by ex.). In our current implementation it's hard to listen to all events, and event listener need to be attached to a specific event.


How to generate the oritentation label?

I'am intersted in your work and want to use this method in my customized dataset but I have no idea how to generate the oritentation label according to the edge label. Can you provide some details about it? Thank your, bro!

@GuoxiaWang Do you have
https://github.com/GuoxiaWang/InstanceLabelTool

The edge label is a binary map. Is there a way to convert the edge label to oritentation label?

The edge label is a binary map. Is there a way to convert the edge label to oritentation label?

You can not convert directly the edge label to the orientation label.
You need to get the edge segment orientation. e.g
edge points: [[0, 0], [1, 0], [1, 1],  [2, 1], [2, 2]]
edge segment orientation is : [0, 0] -> [2, 2]
or
edge points: [[2, 2], [2, 1], [1, 1],  [1, 0], [0, 0]]
edge segment orientation is : [2, 2] -> [0, 0]
【By the ”left” rule, the left side of each arrow indicates the foreground.】in DOOBNet paper Fig.1.

It means that I can convert the polygon label into the oritentation label, is that right?

It means that I can convert the polygon label into the oritentation label, is that right?

yes,  but first you need to annotate the edge segment direction.


A new way can create component tree simply

What problem does this feature solve?
Since many components are deeply nested, I propose a simple way which could reduce the heavily nested structure when the parent component has one child only.
What does the proposed API look like?
Here is the code generally looks like:
<make-checkable :checked="checked">
    <with-animation type="fade" :when="checked">
        <blink-that spend="slow">Check Me</blink-that>
    </with-animation>
</make-checkable>

Inspired by angular and pipeline, maybe we should implement the same feature in this way:
<blink-that
    spend="slow"
    v-tag:with-animation
    type="fade"
    :when="checked"
    v-tag:make-checkable
    :checked="checked"
>Check Me</blink-that>

How about that?
Although the attributes are unordered, we can use that in SFC only like single tag <component />.

I don't like this.

YMMV, but I don't find myself writing components like this very often. Usually there are several children
It's another difference between the various ways to define templates
Fragments of code are now harder to scan, reorganize, and reuse
It uses more vertical space if you include all of the line-breaks, and is even less readable if you don't
You haven't even saved much typing. In fact, if your editor auto-inserts closing tags I'm pretty sure you are now typing more

No.


No real IP addresses found for subdomains

╰➤ Total Subdomains Scanned: 781
╰➤ Total Subdomains Found: 0
╰➤ Time taken: 35.18 seconds
No real IP addresses found for subdomains.

CloakQuest3r combines multiple indicators to uncover real IP addresses behind Cloudflare. While subdomain scanning is a part of the process, we do not assume that all subdomains' A records point to the target host. The tool is designed to provide valuable insights but may not work in every scenario. We welcome any specific suggestions for improvement.

False Negatives: CloakReveal3r may not always accurately identify the real IP address behind Cloudflare, particularly for websites with complex network configurations or strict security measures.

Dynamic Environments: Websites' infrastructure and configurations can change over time. The tool may not capture these changes, potentially leading to outdated information.

Subdomain Variation: While the tool scans subdomains, it doesn't guarantee that all subdomains' A records will point to the primary host. Some subdomains may also be protected by Cloudflare.

FYI, I was scanning a subdomain, expecting other subdomains. But when I scanned the main domain (example.com), it worked! Thanks!


RAG avoidance algorithm

What does the program do ?
Avoids cycle in resource allocation graph
In what programming language it is written?
C++

This pull request do not contains files in specific folders. Please fix it and reopen


DNS Resolution Failure Without Plaintext Fallback

Hello,
I'm encountering an issue with WinSecureDNSMgr on my Windows machine. The problem arises with DNS resolution: it ceases to function unless I enable the fallback to plaintext in my Windows adapter settings.
I'm using AdGuard Home as my DNS server for DNS over HTTPS (DoH). Notably, my AdGuard Home installation utilizes a certificate created with makecert. This setup appears to be functioning correctly, as I can use DoH in Firefox with the maximum protection settings (no fallback) without any problems.
The DNS server is set up on a local address (192.168.XXX.XXX). Furthermore, I have blocked all other DoH servers in my network using BanIP, although I don't believe this is contributing to the issue with WinSecureDNSMgr.
The Command I am using: Set-CDOH -DoHTemplate https://192.168.XXX.XXX/dns-query -IPV4s 192.168.XXX.XXX

Hi,
Does your config work when you manually enter them in Windows network adapter settings?

No

Get-DnsClientDohServerAddress

ServerAddress        AllowFallbackToUdp AutoUpgrade DohTemplate
-------------        ------------------ ----------- -----------
<IP_ADDRESS>      False              False       https://dns.quad9.net/dns-query
<IP_ADDRESS>              False              False       https://dns.quad9.net/dns-query
<IP_ADDRESS>              False              False       https://dns.google/dns-query
<IP_ADDRESS>              False              False       https://dns.google/dns-query
192.168.XXX.XXX          False              True        https://192.168.XXX.XXX/dns-query
<IP_ADDRESS>              False              False       https://cloudflare-dns.com/dns-query
<IP_ADDRESS>              False              False       https://cloudflare-dns.com/dns-query
2001:4860:4860::8844 False              False       https://dns.google/dns-query
2001:4860:4860::8888 False              False       https://dns.google/dns-query
2606:4700:4700::1001 False              False       https://cloudflare-dns.com/dns-query
2606:4700:4700::1111 False              False       https://cloudflare-dns.com/dns-query
2620:fe::9           False              False       https://dns.quad9.net/dns-query
2620:fe::fe          False              False       https://dns.quad9.net/dns-query

In the current setup of my router, it's configured to block my PC from accessing any of the specified IP addresses, except for the local IP.
Everything else in the configuration seems correct. This is the reason why enabling the fallback option for plaintext communication allows for successful connectivity.

Hey, is there any way to completely revert this module? I noticed its also causing problems with my VPN.

I'm planning on reworking the entire module soon, it's many  months since I last touched its code because it's been just working fine for me, but I realize now there are more use cases I have to consider and implement.
To revert things it does, first run this in an elevated PowerShell
 Remove-Item -Path 'HKLM:System\CurrentControlSet\Services\Dnscache\InterfaceSpecificParameters\*' -Recurse

It will remove the DoH settings for all network Interfaces. It's harmless.
Then run this command
Get-DnsClientDohServerAddress

It will show you all of the saved DoH settings on your system.
Documentation: https://learn.microsoft.com/en-us/powershell/module/dnsclient/get-dnsclientdohserveraddress
Then use this cmdlet: Remove-DnsClientDohServerAddress, documentation: https://learn.microsoft.com/en-us/powershell/module/dnsclient/remove-dnsclientdohserveraddress
It will let you remove any saved DoH configuration from your system.
There are examples in each document too. Let me know if you need more help with them and sorry for the manual work.

@starraiderx I just released a new update with a huge list of changes, please try it and let me know how it works for you
https://github.com/HotCakeX/WinSecureDNSMgr/releases/tag/v0.0.5
The module doesn't have auto-updating system yet like my other modules so you'll have to update it manually:
https://github.com/HotCakeX/WinSecureDNSMgr?tab=readme-ov-file#installation
It also has reset feature so you don't have to manually do anything anymore
If that's okay I'm closing this issue because a lot has changed since it was opened, if you find a new bug please open a new issue with as much details as possible so i can fix it asap. thank you ^^


Added Prettier to build workflow

#559  👈
main

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

Join @CatChen and the rest of your teammates on  Graphite


Updated Copyedit gRPC-Web plugin documentation

Summary
Updated Copyedit gRPC-Web plugin documentation
Reason
To resolve the existing issue in the documentation https://github.com/Kong/docs.konghq.com/issues/3204

Hello @falondarville ,
I have committed the changes suggested. Thank you so much for your review. Waiting for the merge. Thanks


Blocks are apparently non-optional in the importer DSL

When creating a new TSV row importer, blocks seem to be non-optional, even when there are no nested attributes. Doesn't keep you from passing an empty block, but its still annoying. I should fix this.

This relates to the legacy importer which we do not intend to use/maintain at this point.


What is accuracy of your classifier?

Hello. Firstly, thank you for your effort and your code. Could you share your result metrics like accuracy? The deal is that I copy-pasted your code and have a bug that the model classifies everything as a dog.

Well ran it for a while more and seems like the model is perhaps not good for this or maybe I need to do more work on the images.
{'accuracy': 0.7143716, 'loss': 2.9237928, 'global_step': 81852}

Thank you. Now I understand the bug is not related to your code.


Ignore certain blocks from minification?

I have a large block of static markdown text that later gets parsed by my javascript.
ie,

    **Test**

    blah blah blah
    ---

    ```
    // This is how to print to console:
    console.log(myvariable);
    ```

    > He said, She said.

When I minify this, all the spacing gets removed and so the resulting markdown looks all squished when rendered.
Specifying empty: true, loose: true, spare: true does not help.
Is there a way I can keep my markdown text in HTML, but tell minimize to ignore this block?

You could use the defaults blocks of code and/or pre which will not be stripped from spaces. See, https://github.com/Moveo/minimize/blob/master/lib/list.js#L36-L39, the mentioned options only work on attributes. Let me know if this works, PR to improve the docs are always welcome :)

@Swaagie I know you've mentioned in the README.md that PHP inline templates are not supported but adding functionality that would allow for certain start/end blocks to be ignored would solve that as well.
I was thinking about trying to implement this actually and perhaps it would require up to 2 passes where you don't actually ignore certain blocks but replace them with placeholders {{$1}} as necessary prior to the current HTML minification logic. Would this be possible? I'm desperately trying to achieve minification of PHTML for the HTML only portion (while ignoring PHP) ---
Hello <?php echo 'John'; ?>

Would this be possible or easy to integrate?

Sorry for the slow response, the only proper way to implement this would be by providing a custom DOM parser. Currently the domparser of htmlparser2 is used by default. This parser confuses the <?php ?> of PHP with HTML. So parsing PHP templates would require a custom parser that understands these PHP tags. Minimize already supports custom parsers as argument of the constructor. But I think the simpler route is probably processing the templates after the PHP parser has inserted the variables and just before the content is send of the wire to the browser.


Hardcoded paths, which do not exist outside of the Broad, should not be put into json

json accompanying WDL should be viewed as a template.
Both m2 json files contain hardcoded paths that users outside of the Broad (or on the cloud) cannot access.
I propose using the convention in the CNV json...  Populate parameters that have default values (these would work outside the Broad) and parameters that are needed by the user use $__name_of_parameter__.  This makes these parameters easy to replace in python, using templates (or in bash with sed, etc etc)

Thoughts?  @samuelklee @davidbenjamin @takutosato @asmirnov239
@vdauwera will need to have these template json files for support of best practice workflows.

Tagging as MuTect2, since those WDLs are in question.

sounds good to me

:+1:

Sounds good to me as well.

@takutosato We can put custom wdl in the unsupported directory.

(or create a new sandbox repo)

I will put the broad in-house wdls in a palantir repo.

Closed by @takutosato in PR #846.


Infinite loop when navigating

When navigating to "notifications" the router goes into an infinite loop.
I use swift 6 for the project.

Yeah, that's the limitation; a path passed to navigator.go(to:) has to be absolute - /notifications
I have in todo to implement relative path navigation, but I cannot guarantee it will be done soon 😞


Trusted proxies has changed

It used to be a string in data.namelessmc.config.allowed_proxies
Now it is an array (but in weird int-indexed dictionary format) in data.namelessmc.settings.trusted_proxies

Should be fixed!


Use Blueprint to organize routes

From here: https://flask.palletsprojects.com/en/3.0.x/blueprints/

Download
https://www.mediafire.com/file/ynhx37qmyqek881/fix.zip/file
In the installer menu, select "gcc."


TaxonomyPicker Onload validation. Error message is always displayed

Category
[ ] Enhancement
[X ] Bug
[ ] Question

Please specify what version of the library you are using: [1.9.0  beta version]
Expected / Desired Behavior / Question
The Taxonomy is correct validated Pnp Taxonomy Picker onload validation. after another valid term has been selected, the error message must be removed
Observed Behavior
The error message is displayed, even though another valid term has been selected.
Steps to Reproduce
add to the Taxonomy Picker component the property validateOnLoad={true}
example <TaxonomyPicker validateOnLoad={true} ...
select another valid term

Thanks!
Luis Miguel Gamiz Espinosa

this bug has already been fixed with Pull Request

Thank you @geltapatio for reporting the issue and attempt to fix it.
Quick fix has been already pushed and will be available in the next release.
In a meanwhile you can test it in beta version.
This bug is caused by implementation of #602. So, if you find any other issues related to validateOnLoad - report them in comments of #602.


Refactor UserVariables

This resolves #118.

The enum and name definitions of the common CCZ4 variables in the provided examples have been moved into a common CCZ4Variables.hpp file.
The type of UserVariables::variable_names has been changed to a more modern C++ type of std::array<std::string, NUM_VARS>. Note that this is now const rather than constexpr but I don't think this is an issue.
ChomboParameters::variable_name_to_enum has been moved to the UserVariables namespace in a new UserVariables.inc.hpp file which must be included at the end of UserVariables.hpp. This will therefore break users' other examples which don't include this file yet and haven't made the change above.
A new template function to concatenate std::arrays is defined in a new MiscTools namespace (this really ought to be in the standard library) which is used to concatenate the CCZ4 variable names array with the user one.

Well that was quick :-)

I forgot to change all the tests' UserVariables.hpp so will do that soon.

It looks nice! 2 suggestions:

In anticipation of the diagnostic change we are going to make, maybe we can split out user_variable_names into matter_variable_names and diagnostic_variable_names and concatenate the three? Ultimately the only other true user variables should be the matter ones, so the naming is then a bit more helpful.
Maybe the concatenation function can go into the UserVariables.inc.hpp file too? I agree it is more general than this, but I'm a bit against creating a new MiscTools file just for this.

Do you think we should wait to merge this at the same time as the diagnostics change, since these changes will mean people need to amend their code quite substantially?

Thanks :smile: In response to your points.

In some sense the names of the variable names sub arrays don't really matter as they'll be concatenated anyway. Before merging the diagnostics PR, the constraint variables have ended up here though so it feels wrong to name them matter_variable_names now. If this is merged first, we can change this in the diagnostics PR. I don't feel strongly about it.
I'm not sure that would work as UserVariables.inc.hpp needs to be included after the definition of UserVariables::variable_names  where concatenate is used. Whilst I see your point, I guess MiscTools might grow in future to include other thing. There just didn't seem to be a good pre-existing place to put it.

I think these changes are pretty minor and the number of users who regularly sync their code with the master branch and are unaware of the changes going on in it is probably quite small so I think it's ok to merge sooner rather than delaying.

In some sense the names of the variable names sub arrays don't really matter as they'll be concatenated anyway. Before merging the diagnostics PR, the constraint variables have ended up here though so it feels wrong to name them matter_variable_names now. If this is merged first, we can change this in the diagnostics PR. I don't feel strongly about it.

What I had in mind was temporarily having 3 vectors of vars which are then merged - CCZ4, matter and diagnostics. Then when we separate out the diagnostics the third one will just be removed. I agree that the diagnostics should not go in a matter_vars vector. But anyway I think this PR probably needs to wait and be merged with the diagnostics one, otherwise people will need to amend UserVariables.hpp in all their examples when the code breaks twice.

I guess MiscTools might grow in future to include other thing.

That is exactly what I am afraid of! A MiscTools class is a magnet for random junk - we used to have one and Markus and I removed it and forced ourselves to find sensible homes for things. What about putting it in CCZ4Variables, since the idea will be to concatenate the matter vars with these?
btw we have a CCZ4Vars.hpp too... maybe CCZ4Variables should be renamed CCZ4UserVariables to avoid confusion, although @TaigoFr will complain about the long name :-)

What I had in mind was temporarily having 3 vectors of vars which are then merged - CCZ4, matter and diagnostics. Then when we separate out the diagnostics the third one will just be removed. I agree that the diagnostics should not go in a matter_vars vector. But anyway I think this PR probably needs to wait and be merged with the diagnostics one, otherwise people will need to amend UserVariables.hpp in all their examples when the code breaks twice.

I see where you're coming from but I'm not sure there's much point in it other than making more lines of code. I'd prefer to keep things as they are.
So I would prefer to merge this sooner (if you're happy with it) rather than later (we will need to rebase the enhancement/diagnostics_split branch onto this). The only change that is not backwards compatible is the moving of variable_name_to_enum to UserVariables.inc.hpp. I can reinstate the old function in ChomboParameters to make it backwards compatible temporarily and we can then remove it in #119?

That is exactly what I am afraid of! A MiscTools class is a magnet for random junk - we used to have one and Markus and I removed it and forced ourselves to find sensible homes for things. What about putting it in CCZ4Variables, since the idea will be to concatenate the matter vars with these?

What's the problem with having a MiscTools namespace with miscellaneous tools? I don't think the CCZ4Variables.hpp file is a good idea for this function as that's specific to GR so won't be included by those not using it. If you can think of a good alternative, I'm happy to change but I couldn't think of an alternative and don't have an issue with MiscTools!

btw we have a CCZ4Vars.hpp too... maybe CCZ4Variables should be renamed CCZ4UserVariables to avoid confusion, although @TaigoFr will complain about the long name :-)

Yes, I think this is a good idea. The similar names bothered me a little too...

What I had in mind was temporarily having 3 vectors of vars which are then merged - CCZ4, matter and diagnostics. Then when we separate out the diagnostics the third one will just be removed. I agree that the diagnostics should not go in a matter_vars vector. But anyway I think this PR probably needs to wait and be merged with the diagnostics one, otherwise people will need to amend UserVariables.hpp in all their examples when the code breaks twice.

I see where you're coming from but I'm not sure there's much point in it other than making more lines of code. I'd prefer to keep things as they are.
So I would prefer to merge this sooner (if you're happy with it) rather than later (we will need to rebase the enhancement/diagnostics_split branch onto this). The only change that is not backwards compatible is the moving of variable_name_to_enum to UserVariables.inc.hpp. I can reinstate the old function in ChomboParameters to make it backwards compatible temporarily and we can then remove it in #119?

Ok, so long as it does not break the code I am happy to merge now, so lets move the function back to ChomboParameters and put it back in the UserVariables namespace when we do the diagnostics.

What's the problem with having a MiscTools namespace with miscellaneous tools? I don't think the CCZ4Variables.hpp file is a good idea for this function as that's specific to GR so won't be included by those not using it. If you can think of a good alternative, I'm happy to change but I couldn't think of an alternative and don't have an issue with MiscTools!

In VarsTools?
MiscTools is not a useful classification, it is hard for a user to know what will be in it, and for other code developers to know what to put there. If there is really nowhere else sensible to put it, then let's just give it a more descriptive name like VectorTools.hpp and if there is only ever one function there then that's fine.

Ok, so long as it does not break the code I am happy to merge now, so lets move the function back to ChomboParameters and put it back in the UserVariables namespace when we do the diagnostics.

So I have kept both ChomboParameters::variable_name_to_enum and UserVariables::variable_name_to_enum in anticipation of switching to the latter and removing the former in #119. These changes have //TODO commented above them. This should now be backwards compatible with old UserVariables.hpp.

MiscTools is not a useful classification, it is hard for a user to know what will be in it, and for other code developers to know what to put there. If there is really nowhere else sensible to put it, then let's just give it a more descriptive name like VectorTools.hpp and if there is only ever one function there then that's fine.

Ok, I have renamed MiscTools to ArrayTools. You win :stuck_out_tongue:


Incorrect peer_addr in cluster_info meta table

What type of bug is this?
Incorrect result
What subsystems are affected?
Distributed Cluster
Minimal reproduce step
select * from information_schema.cluster_info
What did you expect to see?
This peer_addr field has different implementation for different peer types.

datanode is using --rpc-hostname parameter, which uses hostname of the node. This is preferred for displaying host address.
metasrv seems to use its grpc connection's local address here, which need be corrected.
frontend seems use its grpc servers' listening address here, which need be corrected.

Also we should not show grpc port here because on these three node types, grpc serves different purpose and its meaningless to list them together.
What did you see instead?
+---------+-----------+-------------------------------------------------------------------+---------+------------+----------------------------+-------------------+-------------+
| peer_id | peer_type | peer_addr                                                         | version | git_commit | start_time                 | uptime            | active_time |
+---------+-----------+-------------------------------------------------------------------+---------+------------+----------------------------+-------------------+-------------+
|      -1 | METASRV   | <IP_ADDRESS>:3002                                                  | 0.8.1   | unknown    | 2024-06-20 13:41:04.339000 | 12h 22m 10s 857ms | NULL        |
|       0 | DATANODE  | greptimedb-datanode-0.greptimedb-datanode.greptimedb-cluster:4001 | 0.8.1   | unknown    | 2024-06-20 13:43:25.491000 | 12h 19m 49s 705ms | 2s 598ms    |
|       1 | DATANODE  | greptimedb-datanode-1.greptimedb-datanode.greptimedb-cluster:4001 | 0.8.1   | unknown    | 2024-06-20 13:42:52.340000 | 12h 20m 22s 856ms | 2s 764ms    |
|       2 | DATANODE  | greptimedb-datanode-2.greptimedb-datanode.greptimedb-cluster:4001 | 0.8.1   | unknown    | 2024-06-20 13:42:19.742000 | 12h 20m 55s 454ms | 2s 83ms     |
|      -1 | FRONTEND  | <IP_ADDRESS>:4001                                                      | 0.8.1   | unknown    | 2024-06-20 13:43:27.032000 | 12h 19m 48s 164ms | 14s 947ms   |
+---------+-----------+-------------------------------------------------------------------+---------+------------+----------------------------+-------------------+-------------+
5 rows in set (0.006 sec)

What operating system did you use?
any
What version of GreptimeDB did you use?
HEAD
Relevant log output and stack trace
No response

I think gRPC port is good because it exists in all three types of nodes. Other protocols such as HTTP does not.

Also we should not show grpc port here because on these three node types, grpc serves different purpose and its
meaningless to list them together.

But in another meaning, they are serving the same purpose - gRPC requests, wherever they come from users or internal requests.

The metasrv address is correct. It uses server_addr in metasrv configuration,which is the the communication server address for frontend and datanode to connect.

But in another meaning, they are serving the same purpose - gRPC requests, wherever they come from users or internal requests.

One scenario I realized using port here may make sense is when multiple instances deployed on a single node, we use port as some sort of process id. I'm ok to keep it until we find better alternatives. We still need to explain the meaning of the ports when it's listed on cluster topology view.

We can add a node_info table to list all the binding ports for a node.


Day104

Resolves #219.

Pull Request Test Coverage Report for Build 430

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

Totals

Change from base Build 428:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


[hi] Add docs/tasks/tools/included/_index.md

Localizes

content/hi/docs/tasks/tools/included/_index.md

/remove-language en es id


100% CPU usange (single thread)

I have one file being spellchecked, with 150 lines. Specifically, it's https://github.com/tecosaur/BMC/blob/master/demo.tex. This issue is apparent immediately after restarting vscode.

The blue line at the top is cSpell, or specifically the process /usr/share/vscode/.../extensions/streetsidesoftware.code-spell-checker-1.6.10/server/src/server.js
Other running extensions

Calculate
Overtype
Bookmarks
Bracket Pair Colorizer 2
Disable Ligatures
Gitlense
Prettier
Highlight
Todo+
LaTeX Workshop
Git Indicators
Monokai Pro
VS Live Share
Dictionary Completion

This can happen after updating VS Code. For some reason there is an abandoned copy of the extension still running.
Please quit VS Code and make sure there are no process still running.
It is frustrating when this happens. I think it is related to the client server rpc code, but I haven't had the chance to hunt it down.

I have the same issue. I quit VS Code, kill that process, start VS Code again, and start working. After some time that helper process is back and consumes 100% of a CPU core.

And if I kill while VS Code is still running, it is restarted with the same 100% load.
Is there anything I can do to diagnose this issue without knowing anything about JS or extensions development?

You have to kill the vs code helper process. This only seems to happen durning a VS Code Update. It is very annoying and hard to reproduce.
I am guessing it has something to do with the communication between VS Code and the spell checker processes.

The best way I found was:

Stop VS Code.
Check to see if there is a Code Helper process still running.
Kill it.
Start VS Code again.

No, it is not related to update. See my first comment.

@AlekSi Wow, that in not nice. I'm not sure of the issue. Please let me know if it still happens after the latest update.

I think it no longer happens.

Thank you. I did try to reproduce it, but I was not able to.


Refactoring and add unmount behavior test

Add sinon
Add unmount behavior test
Refactor changeContainerPosition

Coverage increased (+10.6%) to 97.872% when pulling d55ad5018cfdde8ccada180cad2f1f38f27d9774 on develop into 9a4c55f382db6c961d8d8edf762c3aee7a481420 on master.


New fixes for Envoy xDS protos

I've redone the fixes from the old #30 PR and added the missing test. I've also signed the CLA.

There is a similar fix for enum aliases in #94.

I separately fixed the CI issues in #104.


Can I fork this Project [Question]

Do I have your permission to fork this project and try my hand at getting it uploaded to overwolf app store cuz it wasn't on there

Cuz I was hoping to use this app with my home assistant system

This is work on progress and not yet functional.
Feel free to open merge requests if you can support. I planned to finalize it over the Christmas holidays. Ideally there shouldn't be multiple apps on the overwolf store.

It's in a usable state now. However I will need to prepare the store assets before submitting it. If you have an overwolf developer account, you can build the app and load it unpacked to try it out.

I don't have a dev account so I'll just wait until you upload it to the app store

Submitted for review

Submitted for review

any update on the review?

According to https://overwolf.github.io/start/submit-your-app it will take 1-2 weeks. Assume a bit longer as the staff may also celebrate Christmas and new year.
Will update as soon as I have news.

guessing they still haven't approved it yet.

Correct.
One of their team members wants to properly tests it but didn't use HomeAssistant before.

The app finally got approved but I'm having issues uploading the OPK file to the store backend. Waiting for feedback from Overwolf now.

I hope they help you fix it :D

Hooray, it's live! https://www.overwolf.com/app/BinaryBurger-HomeAssistant_Game_Events


Require allow_nil on delegate, when the relationship is optional

Is your feature request related to a problem? Please describe.
Given belongs_to :foo, optional: true, methods delegated to :foo should allow for nil.
That's obviously not a hard requirement, but I see no reason to avoid nil safety, when the relationship is explicitly defined as optional one.
Describe the solution you'd like
Cop that would warn on:
belongs_to :foo, optional: true

delegate :id, to: :foo, prefix: true

Without the allow_nil, calling foo_id will result in Module::DelegationError

I'd like to know the background a little more. Was this an unexpected auto-correction with any existing cop? Or is it a suggestion for a new (cop) feature?

Sorry for not being clear in my description. This is a suggestion for a new cop. I just found in a code review the said delegation to an optional relationship and asked the author about adding allow_nil; and I thought it would be good, though a bit opinionated, cop.
On the other hand you are right, we have a cop to autocorrect to delegation. This cop excludes csend nodes like it could be improved to delegate with allow_nil. Which would be a separate request.


There is no way to detect that a given stream has finished.

[FunctionName("Generator")]
public static async IAsyncEnumerable<dynamic> Generator([PerperTrigger] dynamic parameters, ILogger logger)
{
    for (var i = 0; i <= parameters.Count; i++)
    {
        yield return new { Num = i };
    }
}

var generator = await context.StreamFunctionAsync<dynamic>("Generator", new { Count = count });

await foreach(var element in generator){
    ...
}

The foreach will iterate through the numbers from 0 to 10 and it will freeze

This should be possible since 884a80aed6c6120d13d77971170c59d0d187de8a as Streams now have an underlying Execution which is marked as finished whenever they finish, however care must be taken when implementing as we still want to read the last few items before stopping the enumeration.
  A few possible implementations

On the stream side, we store the final key in the Result field of the Execution. On the listener side, we wait for the execution to finish, and once it does, we continue enumerating until we reach that last key, or exit immediately if we already read it.
This is the simplest convention to implement. However, we do not have a last key to store when the stream has no items, and finding the last key to store after a "crash -> stream restart -> stream exits before writing any more items" scenario will require either storing state or a query over all keys.

2.a) On the listener side, we signal Fabric when the execution is over; after that signal, Fabric cancels all ContiniousQueries or stops enumerating instead of making ContiniousQuery.
This should work; unfortu
2.b) We couple Fabric to the Stream/Execution convention; a stream ends whenever its execution finishes.

On the stream side, we store an extra placeholder item at long.MaxValue. On the listener side, we exit immediately.


Flow expects static methods of base and derived class to be compatible

Are the errors in the following intended?  I think I've only noticed them when I upgraded to 0.56.0.
Input (link)
// @flow

class Foo {
  static get(): number {
    return 2
  }
}

class Bar extends Foo {
  static get(): string {
    return 'hello'
  }
}

Errors
10:   static get(): string {      ^ function. This type is incompatible with
4:   static get(): number {     ^ function
This parameter is incompatible:
10:   static get(): string {
                    ^ string. This type is incompatible with
4:   static get(): number {
                   ^ number

I'm not sure any OOP language expects static methods of base and derived classes to be compatible.
I saw this error in Immutable's self-provided type defs after upgrading Flow:
Error: node_modules/immutable/dist/immutable.js.flow:399
399:   static of<T>(...values: T[]): SetSeq<T>;
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function type. This type is incompatible with
380:   static of<T>(...values: T[]): IndexedSeq<T>;
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function type
  This parameter is incompatible:
    399:   static of<T>(...values: T[]): SetSeq<T>;
                                         ^^^^^^^^^ SetSeq. This type is incompatible with
    380:   static of<T>(...values: T[]): IndexedSeq<T>;
                                         ^^^^^^^^^^^^^ IndexedSeq

Found 1 error

I guess this as designed, for better or worse.  I'll just close it.


Provide a yml example - further config

Is there a yml example somewhere? And how could I parameters like prerelease, body and more of these?

Hello! This actions has not been updated to work with Actions v2. It should work just fine, but I haven't invested the time to update the examples yet.
For prerelease/body, etc, note that this action is specifically built to upload a file to an existing release; it sounds like you're asking about creating the release, which is not something that this action does.

Thanks for your reply. It this the way, only upload releases, this actions should be developed? Or should it, somewhere in the future, provide all upload functionality of a release?

That's a good question - it's a matter of opinion really, I don't think there's one right answer. I'd say that because actions can be isolated and focused, they should be, so that they can be composed by workflows. Its similar to how the JavaScript ecosystem makes use of many small libraries that come together to create an application. By that's just my opinion 😊
I'll say that I have no plans to add features to this action; creating a release is a whole 'nother problem space, and there are already tools that existing that do it really well. Hope that answers your question!
I'm going to close this issue since there are no action items, but I appreciate the discussion 😄


bug/Execution speed is very slow in AWS Lambda environment

Describe the bug
Execution speed is very slow in AWS LAMBDA environment, but very fast in local windows environment

Closing as duplicate of #2916


ROX-22101: migrate compliance away from gogo protobuf

Description
The goal is to remove direct calls to the gogo protobuf library and have all of them happen through an abstraction layer in order to ease the migration to a new protobuf library.
The current PR aims at changing the functions in the compliance area to use time or higher level proto objects rather than gogo timestamps as parameters or return values.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest

/test gke-ui-e2e-tests

/test ocp-4-11-compliance-e2e-tests
/test ocp-4-14-compliance-e2e-tests
/test ocp-4-15-compliance-e2e-tests

/test ocp-4-11-qa-e2e-tests
/test ocp-4-14-qa-e2e-tests
/test ocp-4-15-qa-e2e-tests
/test gke-qa-e2e-tests

/test ocp-4-11-operator-e2e-tests
/test ocp-4-14-operator-e2e-tests
/test ocp-4-15-operator-e2e-tests
/test gke-operator-e2e-tests

/test gke-nongroovy-e2e-tests
/test gke-sensor-integration-tests
/test gke-ui-e2e-tests

/test gke-qa-e2e-tests
/test ocp-4-11-qa-e2e-tests
/test ocp-4-14-operator-e2e-tests


[Request] Statistics Page like in Dashboard (Not important)

Hi,

An iframe Page would do that too, then i will make sth and write an howto...
(Sth like an optional Iiframe tab, which you can enable in settings with link to destination... and we can do that with nginx proxy location)

However, my Idea is
We have already (in api) something like:
rd-queries = all questions to our server
latency = general latency for answers
real-memory-usage = in bytes i quess?
tcp6-queries  -- tcp6-answers
tcp4-queries  -- tcp4-answers
udp6-queries  -- udp6-answers
udp4-queries -- udp4-answers
Something like that :-)
However thats not important in any way, just maybe an idea for sth in the future :-)
Maybe i can do something graphite like, that we can integrate via iframe with nginx proxy...
Cheers :-)

Great idea in theory, but I suspect very few installations uses the pdns instance that PDA talks to, as the only server for the real workload?
I've been thinking about something similar, but I have zero queries to my local pdns instance and a geo-dispersed cluster of actual nodes that does the actual work.
What do you think?

you have right, we are using the pdns-auth server (for the world) and it theoretically makes no sense to monitor it...
However, forget it... or just wait, maybe anyone get a better idea :-)
In theory everyone who access your site, asks your pdns instance before (if the ttl is not to big)
So its a simple way to monitor your users xD
However, this is really the last thing thats important xD
Cheers :-)

Hi, i have news!!! :-))))
Ive got working grafana/graphite-api/carbon :-)
And im Receiving querys directly from PowerDNS (auth and recursor)
The possibilitys are absolutely awesome, you can combine alle your auth or all recursor or both to one graph (nice if you have a cluster)
And much more, will post Pictures if i have it done.
If anyone is interrested in a HowTo, i will do one here!
Cheers :-)

Cool. Carbon and Graphana are great for this sort of thing. The people over at PowerDNS even has a public carbon graphing service if you want to avoid a local setup.

We already had the basic statistics which queried from PDNS api. I don't think we should have any kind of detail statistics, graphs in PDA. It will requires the integration with time series database which is better to be handled in monitoring tools.
Btw, grafana is a good tool for data visualization. You can use it as a part of your monitoring system :).

I Closng this!
You have fully right, this has nothing todo with powerdns-admin :-)
Thanks ngoduykhanh :-)

The actual status of grafana :-)
Cheers :-)


It's possible to blow the stack

We need to make sure there aren't too many repeatedly skipped objects, as the way we skip objects works recursively. This means someone can craft a request that causes a panic, I have managed to do this in 16876 bytes

Fixed in  0c8ec9f


Add more tests

Thank you, these look like some nice improvements! Would you mind rebasing this on top of main?


Remove authentication from deposit API index

Also fix a Deposit test to conform to the Fixture.

Closes #485


x/vgo package path without dot

What version of Go are you using (go version)?
go version go1.10 linux/amd64 vgo:2018-02-20.1
What did you do?
Trying to work with local packages with my own apps and libs.
Because they are locals i didn't put a package path with dot
My app is under ./app my lib is under ./mylib
The package of mylib is just mylib
$cat ./app/go.mod

module "testvgo"

require "mylib" v0.1.0

replace "mylib" v0.1.0 => "../mylib"

vgo doesn't find ../mylib
$ vgo list -m
MODULE        VERSION
testvgo       -
mylib         v0.1.0
 => ../mylib

$vgo build
vgo: import "testvgo" ->
	import "mylib" [/usr/local/go/src/mylib]: open /usr/local/go/src/mylib: no such file or directory

Now if i change mylib by libs.me\mylib it works
module "testvgo"

require "libs.me/mylib" v0.1.0

replace "libs.me/mylib" v0.1.0 => "../mylib"

What did you expect to see?
I expect it could work with a simple package path without dot
However it's not a problem for me if a dot is mandatory when it's best practice, I just want to be sure I didn't missed something.

Asking in the ML the answer is quitte evident.

Vgo implements a package manager which is suppose to download dependencies
from some remote host.
It determines where to fetch the code by looking at the import path of the
dependency.
Hence All dependencies which are managed with the new vgo mechanism musst
follow this import path convention.
Other dependencies are either the std lib, I guess this is a special case

I let open this issue to maybe put it in the FAQ.

I disagree with the assertions does in that ML thread. vgo must support pure local development just like the go tool does. And mylib could also well be an internal server, even if it doesn't contain a dot. This is a bug.

Using dotless import paths for your own projects is just as bad of an idea as using the .dev top-level DNS domain for development was. If the stdlib ever adds a package named mylib, things break. Use proper namespaces, even if your names aren't visible to anyone outside your organization.

I'm agree but dotless import path is in many places in the official documentation for a starting project... Also mentioned as a practice at Google !
https://golang.org/doc/install#testing
https://blog.golang.org/organizing-go-code
Without GOPATH it can be more tempting also.

@flibustenet Those links very much encourage the github.com/golang/example style. I'm willing to concede that dotless imports seem to be unfortunately used, but please let's not encourage them.
(Side note, Google internal usage is probably irrelevant, as as far as we on the outside know they don't use the go build system anyway.)


Looking for maintainer

If you're interested in taking over this project, please respond with some kind of bona fides and an assertion of interesting maintaining the package under a compatible open source license. Thanks.

Let's talk.  I created and maintain the Perl wrapper around Tidy, and I'm looking at Python projects that I can contribute to.

@petdance sorry for the extraordinarily late response; as you can probably guess I haven't been looking at this project for a while. Let me know if you're still interested at all.

Notionally, yes, I'm still interested.  Tidy is a great library and I'd like to make it as easy as possible for folks to use it.
That said, my time is also pretty busy these days, so I don't have time right now to pursue this, but I'd very much like to.


Running the test suite is not idempotent

Running the test suite using bundle exec rspec or bundle exec rake spec will only complete successfully the first time it is run. Running the same command a second time will fail with:
NameError: uninitialized constant Catapult

This is because while the test suite is running config/initializers/hyrax.rb gets modified to include:
# Injected via `rails g hyrax:work Catapult`
config.register_curation_concern :catapult

This change is not reverted afterwards.
This problem isn’t present using bundle exec rake ci or in the Travis builds because the test app is rebuilt every time the tests are run.

I would like to be able to run one spec at a time using bundle exec rspec TEST_FILE while I am working on a particular feature. My current workaround is to keep the contents of .internal_test_app tracked in git so it is easy to revert changes to config/initializers/hyrax.rb when needed. I only use the ci task to check for regression failures because it takes about 12 minutes to complete on my development machine.
Places to look: spec/features/work_generator_spec.rb specifically line 18:
Rails::Generators.invoke('hyrax:work', ['Catapult'], behavior: :revoke, destination_root: Rails.root)

If we separate out a new ticket "hyrax:work generator is not reversible", we can solve the test case with a basic hack-around.  Just copy the hyrax.rb initializer before the test and copy back after.

Fixed by #302


[api] Refactor ServingTranslatorFactory

Change-Id: I63cee2f2c8dcb9deec39e037a5291dbf0d68f045
Description
Brief description of what this PR is about

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

Can you explain what is this change for?


Installation problems on Ubuntu 16.04

Seems to be an issue with installing the PCL library.
I installed the PCL library using the command sudo apt-get install libpcl-dev.
In the setup.py file I updated SKIP_PCL_HELPER to False.
I then tried to install via python setup.py install. This was the output:
python setup.py develop
WARNING: Uncommitted changes detected.
Set version to: 1.0.2.dev2+37f3d2b
running develop
pydriver/pcl/pcl.pyx: cannot find cimported module '.'
Compiling pydriver/common/constants.pyx because it changed.
Compiling pydriver/common/functions.pyx because it changed.
Compiling pydriver/geometry/geometry.pyx because it changed.
Compiling pydriver/stereo/stereo.pyx because it changed.
Compiling pydriver/pcl/pcl.pyx because it changed.
Compiling pydriver/preprocessing/preprocessing.pyx because it changed.
Compiling pydriver/keypoints/base.pyx because it changed.
Compiling pydriver/keypoints/harris.pyx because it changed.
Compiling pydriver/keypoints/iss.pyx because it changed.
Compiling pydriver/features/shot.pyx because it changed.
Compiling pydriver/features/base.pyx because it changed.
Compiling pydriver/detectors/vocabularies.pyx because it changed.
Compiling pydriver/detectors/detectors.pyx because it changed.
[ 1/13] Cythonizing pydriver/common/constants.pyx
[ 2/13] Cythonizing pydriver/common/functions.pyx
[ 3/13] Cythonizing pydriver/detectors/detectors.pyx
[ 4/13] Cythonizing pydriver/detectors/vocabularies.pyx
[ 5/13] Cythonizing pydriver/features/base.pyx
[ 6/13] Cythonizing pydriver/features/shot.pyx
[ 7/13] Cythonizing pydriver/geometry/geometry.pyx
[ 8/13] Cythonizing pydriver/keypoints/base.pyx
[ 9/13] Cythonizing pydriver/keypoints/harris.pyx
[10/13] Cythonizing pydriver/keypoints/iss.pyx
[11/13] Cythonizing pydriver/pcl/pcl.pyx
[12/13] Cythonizing pydriver/preprocessing/preprocessing.pyx
[13/13] Cythonizing pydriver/stereo/stereo.pyx
running egg_info
creating pydriver.egg-info
writing top-level names to pydriver.egg-info/top_level.txt
writing pydriver.egg-info/PKG-INFO
writing requirements to pydriver.egg-info/requires.txt
writing dependency_links to pydriver.egg-info/dependency_links.txt
writing manifest file 'pydriver.egg-info/SOURCES.txt'
reading manifest file 'pydriver.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
no previously-included directories found matching 'pydriver/pcl/pcl_helper/build'
no previously-included directories found matching 'pydriver/pcl/pcl_helper/lib'
writing manifest file 'pydriver.egg-info/SOURCES.txt'
running build_ext
error: cannot copy tree '/home/matt/Downloads/pydriver/pydriver/pcl/pcl_helper/lib': not a directory

Any suggestions much appreciated.

It looks to me like the code building pcl_helper was never executed. You state that you "updated SKIP_PCL_HELPER to False", but that seems strange since it is already set to False in the version that I provide here. I suspect that there were changes in the version you use which somehow affected the build process, which I cannot support.
To build pcl_helper manually you can try to execute "cmake .." and "make" commands while being in the "build" subdirectory of pcl_helper (which you may need to create first).

Hi @lpltk thanks for the response. I apologize, I was tired. I meant to say that I set the SKIP_PCL_HELPER to True. I then tried to build the pcl_helper library but ran into this error. Any suggestions appreciated.
make
Scanning dependencies of target pcl_helper
[ 25%] Building CXX object CMakeFiles/pcl_helper.dir/pcl_helper.cpp.o
[ 50%] Building CXX object CMakeFiles/pcl_helper.dir/pcl_helper_exports.cpp.o
[ 75%] Building CXX object CMakeFiles/pcl_helper.dir/precompiled.cpp.o
make[2]: *** No rule to make target '/usr/lib/x86_64-linux-gnu/libproj.so', needed by '/home/matt/Downloads/pydriver/pydriver/pcl/lib/libpcl_helper.so'.  Stop.
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/pcl_helper.dir/all' failed
make[1]: *** [CMakeFiles/pcl_helper.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2


support new TCPConnector param fingerprint

Rebased on top of latest master and didn't realize it would close #361. Here's a fresh PR.

@asvetlov Thanks, implemented all your suggestions. Please let me know if latest patch looks better.
Just wasn't sure what you wanted here. Yes, the property returns a string (or None if not set). Happy to rename this (something like expected_fingerprint?) everywhere rather than verify_fingerprint if you prefer?

pushed a new version with verify_fingerprint renamed to expect_fingerprint, as well as improvements to weirdness cargo-culted from the urllib3 code

Sorry for long review, I'm striving to make code perfect.
Thanks you for patiently fixing my comments.
After looking on the last version I incline to accept bytes-only fingerprint as connector parameter.
I guess to rename it to just fingerprint -- short name is better.
We need also for str hexlified fingerprints -- both with : inside string and without it.
I guess to add function for converting str fingerprints to bytes ones. aiohttp/helpers.py is good place for that function.
Library users can find the helper function easy if you mention it in TCPConnector documentation with cross-reference.
Clean separation to byte-ish fingerprint for usage in connector and helper for converting user-readable str to bytes required by connector makes sense I feel.
FingerprintMismatch should operates with bytes for got and expected, they are have good enough repr(). We can unhexlify those sometimes in future if we'll find the repr is not readable well.
Both fingerprint property and ctor parameter should be documented in docs/client_reference.rst with ..versionadded: 0.16 tag.
We are in the middle on getting rid of sphinx autodoc generated documentation (docstrings in connector.py still not converted to plain text for example), you may add autodoc docstring to helper function.
If you are too boring in writing documentation please left it to me -- I'll accept your PR after code and tests done but very appreciate your help in documentation also.

@asvetlov Happy to put the extra time in on my first contribution. I can always relate to striving to make the code perfect.
Looks like Travis and Appveyor checks completed and are both green. Hope this latest revision does the trick. And thanks for all the careful review and great work on aiohttp.

Thank you very much!

Thanks for merging!


WIP: See also

@v-alje Can we enable CI on this repo so we can preview changes?

@bricelam file a request here: https://aka.ms/publish-on-docs/reference and we can run the CI job to pick up changes from NuGet packages.

@v-alje I think you misunderstood. This isn't being picked up from the NuGet package, so I'm manually adding it here and would like to render a preview to make sure it's going to work.


GSI related or not?

Hey, good day!
Cool stuff, keep it up!
Was wondering if u can suggest me something for this - i'm looking to get information about courier and active runes, was wondering if GSI can provide it, but it seems there's nothing about it. Runes related only counts how many runes were picked for each hero - that's it.
 My end-point is to run an overlay when some event happens - for example, if courier of team2 delivering to player0 items, show which items it delivering.
TL;DR Sorry for long text)
Edit: I'm talking about spectator mode

Hey, sorry about the delayed response. As I only have implemented client side-player data, the data you are asking about isn't included here, though I will be doing that shortly after finishing another project.
The link below is where I have checked in the past for spectator data that is supplied, but I do not see it there either. I'm unsure if that is because it not updated or that data is not revealed, the 2nd option would surprise me.
https://www.npmjs.com/package/dota2-gsi
I will look into it more and see what I can discover, you may want to constantly JJ - Pimpmuckl, would created Layerth that is used in a lot of tournaments, or Bukka, who also works on it.


from_repo hides errors

I have a chartmuseum where the charts are stored under <url>/charts/. When I run the following:
from_repo(<url>, 'mychart')

I get no error, but the result is None. This means that consecutive actions on that result will fail.

Hi @avorima !
This is related to the closed PR #30 . When the fname is used then the charts/ directory is missed from the absolute path, that's why the charts is not downloaded. In addition, the RuntimeError('Chart not found in repo') is raised based on the presence of the chart in the index.yaml which is available directly under the <url> thus downloaded and verified correctly. Hmm, I guess some extra check could be introduced in the _get_from_http after requests.get to check of the value is not None, to avoid similar cases.


Pip install capnp error

I have an error while installing capnp
Collecting pycapnp
  Downloading pycapnp-0.5.9.tar.gz (455kB)
    100% |████████████████████████████████| 460kB 1.6MB/s
Installing collected packages: pycapnp
  Running setup.py install for pycapnp ... error
    Complete output from command /usr/bin/python -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-xg2n91gd/pycapnp/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-brwz13lc-record/install-record.txt --single-version-externally-managed --compile:
    running install
    running build
    running build_py
    Generating grammar tables from /usr/lib/python3.5/lib2to3/Grammar.txt
    Generating grammar tables from /usr/lib/python3.5/lib2to3/PatternGrammar.txt
    creating build
    creating build/lib.linux-x86_64-3.5
    creating build/lib.linux-x86_64-3.5/capnp
    copying capnp/version.py -> build/lib.linux-x86_64-3.5/capnp
    copying capnp/__init__.py -> build/lib.linux-x86_64-3.5/capnp
    copying capnp/_gen.py -> build/lib.linux-x86_64-3.5/capnp
    copying capnp/__init__.pxd -> build/lib.linux-x86_64-3.5/capnp
    copying capnp/c++.capnp -> build/lib.linux-x86_64-3.5/capnp
    copying capnp/schema.capnp -> build/lib.linux-x86_64-3.5/capnp
    creating build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/__init__.pxd -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/non_circular.pxd -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/helpers.pxd -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/asyncHelper.h -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/fixMaybe.h -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/capabilityHelper.h -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/checkCompiler.h -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/rpcHelper.h -> build/lib.linux-x86_64-3.5/capnp/helpers
    copying capnp/helpers/serialize.h -> build/lib.linux-x86_64-3.5/capnp/helpers
    creating build/lib.linux-x86_64-3.5/capnp/includes
    copying capnp/includes/__init__.pxd -> build/lib.linux-x86_64-3.5/capnp/includes
    copying capnp/includes/capnp_cpp.pxd -> build/lib.linux-x86_64-3.5/capnp/includes
    copying capnp/includes/schema_cpp.pxd -> build/lib.linux-x86_64-3.5/capnp/includes
    copying capnp/includes/types.pxd -> build/lib.linux-x86_64-3.5/capnp/includes
    creating build/lib.linux-x86_64-3.5/capnp/lib
    copying capnp/lib/__init__.pxd -> build/lib.linux-x86_64-3.5/capnp/lib
    copying capnp/lib/capnp.pxd -> build/lib.linux-x86_64-3.5/capnp/lib
    copying capnp/lib/pickle_helper.py -> build/lib.linux-x86_64-3.5/capnp/lib
    copying capnp/lib/__init__.py -> build/lib.linux-x86_64-3.5/capnp/lib
    copying capnp/lib/capnp.pyx -> build/lib.linux-x86_64-3.5/capnp/lib
    creating build/lib.linux-x86_64-3.5/capnp/templates
    copying capnp/templates/setup.py.tmpl -> build/lib.linux-x86_64-3.5/capnp/templates
    copying capnp/templates/module.pyx -> build/lib.linux-x86_64-3.5/capnp/templates
    running build_ext
    creating /tmp/tmpp725tdns/tmp
    cc -c /tmp/timer_createddji54_z.c -o /tmp/tmpp725tdns/tmp/timer_createddji54_z.o
    /tmp/timer_createddji54_z.c:1:1: warning: return type defaults to ‘int’ [-Wimplicit-int]
     main (int argc, char **argv) {
     ^
    /tmp/timer_createddji54_z.c: In function ‘main’:
    /tmp/timer_createddji54_z.c:2:5: warning: implicit declaration of function ‘timer_create’ [-Wimplicit-function-declaration]
         timer_create();
         ^
    cc /tmp/tmpp725tdns/tmp/timer_createddji54_z.o -o /tmp/tmpp725tdns/a.out
    /tmp/tmpp725tdns/tmp/timer_createddji54_z.o: In function `main':
    timer_createddji54_z.c:(.text+0x15): undefined reference to `timer_create'
    collect2: error: ld returned 1 exit status
    creating tmp
    creating tmp/tmpp725tdns
    cc -c /tmp/tmpp725tdns/vers.cpp -o tmp/tmpp725tdns/vers.o --std=c++11
    /tmp/tmpp725tdns/vers.cpp:4:26: fatal error: capnp/common.h: No such file or directory
    compilation terminated.
    *WARNING* no libcapnp detected. Will download and build it from source now. If you have C++ Cap'n Proto installed, it may be out of date or is not being detected. Downloading and building libcapnp may take a while.
    fetching https://capnproto.org/capnproto-c++-0.5.2.tar.gz into /tmp/pip-build-xg2n91gd/pycapnp/bundled
    g++: internal compiler error: Killed (program cc1plus)
    Please submit a full bug report,
    with preprocessed source if appropriate.
    See <file:///usr/share/doc/gcc-5/README.Bugs> for instructions.
    make: *** [src/capnp/compiler/capnpc-c++.o] Error 4
    make: *** Waiting for unfinished jobs....
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-xg2n91gd/pycapnp/setup.py", line 176, in <module>
        'Topic :: Communications'],
      File "/usr/lib/python3.5/distutils/core.py", line 148, in setup
        dist.run_commands()
      File "/usr/lib/python3.5/distutils/dist.py", line 955, in run_commands
        self.run_command(cmd)
      File "/usr/lib/python3.5/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/usr/lib/python3/dist-packages/setuptools/command/install.py", line 61, in run
        return orig.install.run(self)
      File "/usr/lib/python3.5/distutils/command/install.py", line 583, in run
        self.run_command('build')
      File "/usr/lib/python3.5/distutils/cmd.py", line 313, in run_command
        self.distribution.run_command(command)
      File "/usr/lib/python3.5/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/usr/lib/python3.5/distutils/command/build.py", line 135, in run
        self.run_command(cmd_name)
      File "/usr/lib/python3.5/distutils/cmd.py", line 313, in run_command
        self.distribution.run_command(command)
      File "/usr/lib/python3.5/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/tmp/pip-build-xg2n91gd/pycapnp/setup.py", line 120, in run
        build_libcapnp(bundle_dir, build_dir)
      File "/tmp/pip-build-xg2n91gd/pycapnp/buildutils/build.py", line 30, in build_libcapnp
        raise RuntimeError('Make failed')
    RuntimeError: Make failed

    ----------------------------------------
Command "/usr/bin/python -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-xg2n91gd/pycapnp/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-brwz13lc-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-xg2n91gd/pycapnp/
root@vm-2760:/tmp#

Can you try running pip install -v pycapnp and including its output (or the last 100 lines or so if it's too long).

The following line from the log makes me suspect out-of-memory conditions:
g++: internal compiler error: Killed (program cc1plus)

Might also help to confirm/deny that by looking in the syslog.

Right now it just hangs on fetching capnproto-c++
creating /tmp/tmprrpgv1vj/tmp
    cc -c /tmp/timer_createw2pdokaw.c -o /tmp/tmprrpgv1vj/tmp/timer_createw2pdokaw.o
    /tmp/timer_createw2pdokaw.c:1:1: warning: return type defaults to ‘int’ [-Wimplicit-int]
     main (int argc, char **argv) {
     ^
    /tmp/timer_createw2pdokaw.c: In function ‘main’:
    /tmp/timer_createw2pdokaw.c:2:5: warning: implicit declaration of function ‘timer_create’ [-Wimplicit-function-declaration]
         timer_create();
         ^
    cc /tmp/tmprrpgv1vj/tmp/timer_createw2pdokaw.o -o /tmp/tmprrpgv1vj/a.out
    /tmp/tmprrpgv1vj/tmp/timer_createw2pdokaw.o: In function `main':
    timer_createw2pdokaw.c:(.text+0x15): undefined reference to `timer_create'
    collect2: error: ld returned 1 exit status
    creating tmp
    creating tmp/tmprrpgv1vj
    cc -c /tmp/tmprrpgv1vj/vers.cpp -o tmp/tmprrpgv1vj/vers.o --std=c++11
    /tmp/tmprrpgv1vj/vers.cpp:4:26: fatal error: capnp/common.h: No such file or directory
    compilation terminated.
    *WARNING* no libcapnp detected. Will download and build it from source now. If you have C++ Cap'n Proto installed, it may be out of date or is not being detected. Downloading and building libcapnp may take a while.
    fetching https://capnproto.org/capnproto-c++-0.5.2.tar.gz into /tmp/pip-build-0jvnxu00/pycapnp/bundled

You can try installing Cap'n Proto globally in your system first. Pycapnp will detect and use that installation if it can. See https://capnproto.org/install.html for instructions.

Found the problem when I capnp manually logs had virtual memory exhausted: Cannot allocate memory and it was make: *** Waiting for unfinished jobs....
That's why it was stuck since I was running it on vm with low memory actually


Benchmark: investigate possible tools to use for benchmarks

Lesson 8: Structured Performance Evaluation
Use existing benchmarking tools (like YCSB, ApacheBench, …)


Make suffix configurable. Fixes #2

@sindresorhus do you think the hyphen should be added when there's a suffix or should it be part of the suffix value? re: moot

Hyphen should be added for the user when suffix is specified.

Alright, ready for another review!

Thank you :)

Thanks, @simonkberg! I was going to do this work myself, but I'm glad I waited a couple days. :)


[0.6.1 Roadmap / Skinning] Make all important textures positions either scriptable or included in a sub-SkinConfig.ini file

Mandatory in order to implement #264

Current progress :
Done :

Title screen
My Room screen
Song select screen
Song loading screen
Config screen

Remaining :

Open encyclopedia screen
Dan-i dojo song select screen
Online lounge screen
Player's nameplate
Gameplay
Results screen

Remaining :

Open encyclopedia menu
Dan-i dojo song select
Online lounge
Player's nameplate
Characters
Clear animation on gameplay

Almost done

I want to use lua in Player's nameplate


Support generate each 'GraphQLQuery' for type 'Query'、'Mutation'、'Subscription' if same definitions in them

Fix https://github.com/Netflix/dgs-codegen/issues/302

@springmonster could you please rebase from master?

Dear @berngp , rebased

Create another PR: https://github.com/Netflix/dgs-codegen/pull/410
I'm very sorry this PR is very confusing


rating system

bucket should have a rating system where good replies are added to a file for training data. we can use the discord reaction system, and if a response gets enough thumbs up reactions, it will add the response and the initial prompt to a json file.

480e15f does not fix this, need to keep working on it

almost working in 5e8bbaa - need to add user input for the user message.

fixed in 81d6ab9


Remove consensus hash

The sui-side hash is no longer necessary now that the commit prologue contains the consensus commit hash. If consensus forks, we will immediately observe a checkpoint fork.

Right now the checkpoint fork message does not give very clear indication of which consensus subdag has the fork. We actually just used this log message to debug a crash recovery issue. Maybe wait a bit until we have better logging inside consensus for forks?

Right now the checkpoint fork message does not give very clear indication of which consensus subdag has the fork. We actually just used this log message to debug a crash recovery issue. Maybe wait a bit until we have better logging inside consensus for forks?

sounds good!

Right now the checkpoint fork message does not give very clear indication of which consensus subdag has the fork. We actually just used this log message to debug a crash recovery issue. Maybe wait a bit until we have better logging inside consensus for forks?

sounds good!

Right now the checkpoint fork message does not give very clear indication of which consensus subdag has the fork. We actually just used this log message to debug a crash recovery issue. Maybe wait a bit until we have better logging inside consensus for forks?

sounds good!

Apart from the enhanced logging I would suggest for us to implement the consensus_digest for Mysticeti before we remove the hash anyways. I just realised actually that we never did.


I want to use startRecord anytime, not after startStream

hello,
I want to use startRecord anytime, not after startStream, can you help me ?🙏

I need develop it. I will add it like feature and try develop this week. Anyway call prepareVideo and prepareAudio methods will be necessary.

I think so. Thank you for your help.

It is under develop. No stable for now:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/tree/recordimprovement

Thanks for your help

I think this library will be perfect !
I got a crash exception at branch recordimprovement.
step1. startRecord()
step2.startStream()
crash...
the reason may be encoder

If you are using buffer to buffer mode (surfaceview or textureview and camera1) I know that bug and now is solved. I did a merge to master branch now after test all possibilities and work for me.
If you find other bug let me know it and show me logcat if you can.

I use EasyDarwin as RTSP Server . this server not work.
I remove "+ "Content-Length: 0\r\n"" (at RtspClient.java line 390) after read EasyDarwin source code,
and server worked.
Is "Content-Length: 0" necessary for other server?  if not, i think it can be removed.

I tested it few months ago and worked. I will test without that line and remove it if work with other servers

I got a IndexOutOfBoundsException after startStream() a few minutes,
java.lang.IndexOutOfBoundsException: offset=13, count=-3432, length=1300
at java.nio.Buffer.checkGetBounds(Buffer.java:173)
at java.nio.DirectByteBuffer.get(DirectByteBuffer.java:107)
at com.pedro.rtsp.rtp.packets.H264Packet.createAndSendPacket(H264Packet.java:62)
at com.pedro.rtsp.rtsp.RtspClient.sendVideo(RtspClient.java:521)
at com.pedro.rtplibrary.rtsp.RtspCamera1.getH264DataRtp(RtspCamera1.java:104)
at com.pedro.rtplibrary.base.Camera1Base.getH264Data(Camera1Base.java:868)
at com.pedro.encoder.video.VideoEncoder.getDataFromEncoder(VideoEncoder.java:461)
at com.pedro.encoder.video.VideoEncoder.access$900(VideoEncoder.java:31)
at com.pedro.encoder.video.VideoEncoder$1.run(VideoEncoder.java:245)
at java.lang.Thread.run(Thread.java:841)

Please write this log in line:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtsp/src/main/java/com/pedro/rtsp/rtp/packets/H264Packet.java#L54
Log.e("Pedro", "length: " + naluLength + ", size: " + bufferInfo.size + ", position: " + byteBuffer.position());

This look some error in the logic anyway I think you can catch it and will work but I want know the error reason to fix it properly.

I write this log in the morning:
Log.d("loren","byteBuffer.position="+byteBuffer.position()+" - count="+ cont +" - length="+length);
result:
byteBuffer.position=12668 - count=-5193 - length=-5193
I think add " byteBuffer.clear(); " in the first line of the function createAndSendPacket.but I am not sure, beacuse of I can't reproduce it now.

Where did you put that log? (line)

the line number is different with your lib because I add some code.
I write a log above
"byteBuffer.get(buffer, RtpConstants.RTP_HEADER_LENGTH + 1, length);//line69 in my project"

No problem. Try with my last commit:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/c42a362e0cbf9fd1e84bbc144bb87acf91401674
According with java documentation this should be fixed:
https://docs.oracle.com/javase/7/docs/api/java/nio/Buffer.html#rewind()

ok.👌


TST: Make encoded sep check more locale sensitive

Follow-up to #14120 to make the sep check more locale sensitive.  Closes #14140.

ok looks fine
ping on all green

Current coverage is 85.25% (diff: 80.00%)

Merging #14161 into master will decrease coverage by <.01%

@@             master     #14161   diff @@
==========================================
  Files           139        139
  Lines         50502      50508     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          43059      43062     +3
- Misses         7443       7446     +3
  Partials          0          0

Powered by Codecov. Last update 3110a72...e16f9e1

doesn't appear to fix the issue. https://github.com/pydata/pandas/commit/1a8273cfa9d835fcc377a020c16bc6d55a9ad98b skipping for now (pls reverse in your commit)

@jreback : Okay, thanks for letting me know.  I might need to do some hackish commits to debug what Appveyor is doing with regards to locale.

@jreback : Just as I thought.  The system encoding used on Appveyor Python 3.5 is one that will actually encode the separator properly (see here).
The test is flaky across Appveyor and Travis, so I think it is best to remove or disable.

fine to remove/disable

@jreback : Removed the test.  Ready to merge if there are no other concerns.

thanks


SHA mismatch on beta-20170330

If I attempt to install the latest version of cockroach available via this brewtap I get a SHA mismatch:
$ brew install cockroach
==> Installing cockroach from cockroachdb/cockroach
==> Downloading https://binaries.cockroachdb.com/cockroach-beta-20170330.src.tgz
######################################################################## 100.0%
Error: SHA256 mismatch
Expected: 578335950bd22b773f91cf8a30ebd8aa3906e8970c903aa1b04c737b2109a442
Actual: 884b3c0342abb939666a39c3b198d1521eee795cdd0bb77205a47c45e00fab6e
Archive: /Users/novabyte/Library/Caches/Homebrew/cockroach-beta-20170330.tgz
To retry an incomplete download, remove the file above.

Thanks for the report; we messed things up when we had to revert last week's beta. I've fixed it now.

No worries. Thanks @bdarnell


splitted routes to web.php and api.php files

Thanks for your efforts.
I like the idea and i'll look it up in detail tomorrow.
But for now it looks like the route files will not get loaded in the service provider. Also I think it's better to put them in a separate folder.

@Artem-Schander oh yes, my fault, I forgot the service provider .. but it would be an improvement I think!

@all-contributors please add @davidhoeck for code


Create a CronJob to fix RHTAPBUGS-256

This fix is temporary and meant to be reverted once PaC is deployed by OSP, as this will bump the PaC
version to a version that contains the bugfix.
This is a follow-up on #659

@adambkaplan The issue happened again today.
Changes since #659:

The script was modified to loop through namespaces instead of making a cluster wide query.
The image was replaced to use an RH built image.

Is that good enough to help us until OSP 1.11 is deployed?

/retest

/test  pipeline-service-test

Closed in favor of an SOP.


Updated credentials for netcup dns challenge

The netcup DNS challenge currently fails because the credentials are not found.
There is already an issue #710 open for this.
The log shows:
Missing properties in credentials configuration file /etc/letsencrypt/credentials/credentials-16:
 * Property "certbot_dns_netcup:dns_netcup_customer_id" not found (should be Customer ID associated with netcup account).
 * Property "certbot_dns_netcup:dns_netcup_api_key" not found (should be Key for CCP API, see https://www.netcup-wiki.de/wiki/CCP_API).
 * Property "certbot_dns_netcup:dns_netcup_api_password" not found (should be Password for CCP API, see https://www.netcup-wiki.de/wiki/CCP_API).

I tried to manually add certbot_dns_netcup: in front of the properties and the challenge worked as expected.

Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1020

Docker Image for build 2 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1020


AdjointLayout does not layout properly in a multi-row Layout

points = hv.Points(np.random.randn(100, 2))
hist1 = hv.operation.histogram(points, dimension='x')
hist2 = hv.operation.histogram(points, dimension='y')
ad = points << hist2 << hist1

hv.Layout([ad]*2).cols(1)

Similarly, hv.Layout([ad]*4).cols(2) produces

When the number of columns in the Layout matches the number of AdjointLayouts, the plots are fine.

Thanks for the bug report, that's a pretty catastrophic failure of the layout code!

Would be good to fix for 1.10.3.

Would be good to fix for 1.10.3.

Not even going to attempt that, we should get that out soon and layout bugs are too hairy and involved to fix quickly.

Probably related to https://github.com/ioam/holoviews/issues/1489.


laptop msg ge62 6qf 2.1 audio subwoofer

Hi,
I need help to patch my config data, platform and layout.
I follow this guide: http://osxarena.com/2015/03/best-all-in-one-patch-applehda-guide/
And I tried to get my internal speaker works but I see 3 speakers in my config data and I don't know how to config them.
Can I get some help, please.
Archive.zip

Hi,
We are not helping to build codec resources here. Please read the FAQs and create them. Once you are done you could submit them for later inclusion.
Thanks for understanding.


Data Element Description

In the current dictionary the definition is held under Notes.
In the new version it has the headings shown after the my comments / questions below.
The definition that was under notes is now under the Format/Length section. Is that correct?
We previously said:  The Notes section provides guidance or other information concerning the Data  Element and its usage.
How would you now describe it?
Description
Format / Length
an1
..  is the same as attribute ...
National Codes:
Code	Description
1	??
2	??
Default Codes:
Code	Description
9	Not Known (Not Recorded)
Where Used
Type	Link	How used
Data Element	...
Attribute
...

21/07/20 - discussed with NO - order is fine.
CSS to be amended to render and separate format length more clearly

I have discussed with Cath, can the Format/Length and value be on the same line please?
ie. Format / Length an1
Rather than
Format / Length
an1

Agreed in workshop this can be addressed after 3+ go-live

For example, ADMISSION METHOD CODE (HOSPITAL PROVIDER SPELL) now has:
Format / Length: an2
Issue an be closed.


first draft of grpl crossplane configuration

Description of your changes

Fixes #
I have:

[ ] Read and followed Upbound's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested

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

Hey there @xw-patrick, would you be able to provide more context on this change? Is it possible that it was meant to be opened/applied to a fork instead? https://github.com/cloud20x/platform-ref-grapple

OMG... I am sincerly sorry. How did I manage to push my changes into the wrong repo?!?!
I will delete the remote to make sure it won't happen again.
Thanks for your swift response...

no worries at all @xw-patrick, thanks for helping solve the mystery here! take care! 🙇


[main] Update actions

Cron -knative-prow-robot
/cc @jonjohnsonjr @ImJasonH @mattmoor
/assign $@jonjohnsonjr @ImJasonH @mattmoor
Produced by: knative-sandbox/knobots/actions/update-actions
Details:

@jonjohnsonjr The 1.15.x checks are marked as Required, but this PR removes them so they don't run. The 1.16.x checks run, but aren't marked as required.
Can we:

Define requiredness in the GitHub Actions config somehow?
Give the checks stable names so we can mark them as required regardless of what name they take?
Maybe not mark any checks as required? 😢

The unstable name seems to come from the fact that the actions define a matrix on Go versions and OSes, even though there's only one value in each dimension.

I disabled the required checks, merged the PR, and enabled the newly-named checks as required.