text:
Fix a couple bugs manifesting in the test-suite and improve usability

Currently HEAD is a bit broken, mostly due to ValidatorCleanRestart and sometimes ValidatorRecurringEnrollment.
This does not fully fix the issues, but removes a few spurious error message, makes things easier to debug, and removes a race condition. There's still an issue happening in the nomination process which I am currently debugging.

Merging because at least it fails timely now.

repo: bosagora/agora


text:
Remove peer validation through Registry

Current way of peer validation uses both registry and handshake for Validator nodes and just rely on registry (#2686) for Flash nodes. Relying on registry will not be achievable when we move to DNS from current GET interface and DNS has its own DNSSEC method for that. Registry itself will still validate registering nodes by signature.
Part of #2449

Yes, address gets banned and client is discarded when handshake fails

source: gharchive/pull-request

license: MIT

license_type: permissive

license_source: github-api


text:
Unknown error occurred (adding new easycontrol CT200)

when adding new easycontrol CT200 thermostat
when adding from hacs no issues, (restart HA), add intergration --> select "easy control"  next screen "user data"
Unknown error occurred (can't thell what goes wrong)
FYI:  no "-" between ID/token en 8 char "simple" passwd (no special characters used)

When using OCR one might need to check if the text = 1:1 copied y/n ... 1 letter was erroronously capitalized and thus caused the error (yes it is fixed now) aka PEBKAC
please change error to "you might want to double check the stickers & input in the app"  this might benifit new users ;)

The problem is that if access token is properly provided then bosch server doesn't respond that password is wrong...
and sometimes data can't be decoded (then I got notification about wrong password for sure), but sometimes it just doesn't respond with data... and then it can be dns issue, password issue etc. Hard to guess.

Is some who can help with this issue. I've the same problem.
Installed 0.24.5 and trying to add my Nefit CT200 (XMPP).
Changed password already 2 times, but no result

source: gharchive/issue

authors: mr4hughz pietvdmaas pszafer


text:
Add credhub exporter

This PR integrates a new exporter for monitoring some content of a credhub server, specifically creation dates of credentials and expiration of certificates (it does not yet provide metrics about the credhub server health such as error rates, response times, or total number of entries).
This PR includes an associated alerts that triggers when certificates are close to expiration.
We'd be happy to transfer ownership of the orange-cloudfoundry/credhub_exporter to the https://github.com/bosh-prometheus/ organization if this makes more sense for the community to contribute maintenance.
The PR requires blob credhub_exporter-0.1.4.linux-amd64.tar.gz to be added at path credhub_exporter/credhub_exporter-0.1.4.linux-amd64.tar.gz
Relates to orange-cloudfoundry/paas-templates#84

Thanks @psycofdj. I upload the credhub_exporter-0.1.4.linux-amd64.tar.gz  blob at https://github.com/bosh-prometheus/prometheus-boshrelease/commit/2b6421aa7e045e242e92c3a490804f08095d2104.

authors: frodenas psycofdj

repo: bosh-prometheus/prometheus-boshrelease


text:
filter(...) should also accept numberSet for first argument

Right now filter only accepts a seriesSet for the first argument. I have alerts were I want to filter the results but cannot because they are numberSets instead of seriesSet. Filter really operates on a tag level, so we should add  filter(numberSet, numberSet) to the available functions.
example:
//Get all systems using a version of scollector older than 20150916205517
$version = max(q("max:scollector.version{os=windows,host=*}", "2w",""))
sort(filter($version, ($version < 20150916205517)),"asc")
//does not work because $version is a numberSet and filter requires seriesSet as the first argument

//This works, but is a bit verbose
$version = max(q("max:scollector.version{os=windows,host=*}", "2w",""))
sort(max(filter(q("max:scollector.version{os=windows,host=*}", "2w",""), ($version < 20150916205517))),"asc")

What is your end goal with this? Generally it would just be q("max:scollector.version{os=windows,host=*}", "2w","") < 20150916205517) and you get 0 / 1 to indicate true or false.

That is hard to read on a page with 100+ results. I want to print a list of systems using an older version of scollector. Filter would do exactly what I want if it accepted numberSet instead of seriesSet. Considering the filter is done on the tags and not the set values I see no reason why it shouldnt support that.

The expression language is not polymorphic, so this would involve reworking the expression language. This sounds like a use case for "reports", since what you want is a filtered view (like you could do in a template). Except that it is perhaps ad hoc.
We could also maybe add a "hide false" option for the expression results.

I thought we already had functions that worked on both types of sets, but I think I was remembering the scalar -> numberSets promotion stuff. This isn't a pressing need since there is a workaround in this case, but I think having a filter that works on numberSets would make for cleaner alerts (see examples above) even if it is a different function like filterNumberSet().
Right now I often end up with complex expressions or complicated "if and eq .. eq ..." statements in templates because we are missing what seem to be simple functions (or have mismatched types like this example). I am trying to start filing these as issues for tracking, as if we can add a few functions to simplify common expressions it will make things easier to read.

It may just be my SQL background, but even the shortest version of that expression I can currently create looks very non-intuitive when compared to the first example above:
$version = q("max:scollector.version{os=windows,host=*}", "2w","")
sort(max(filter($version, (max($version) < 20150916205517))),"asc")

Why is it better to have it in the expression language instead of as a template function? filter is particular for limiting series for graphing. It generally wouldn't be good to use in alert defs I don't think because one would likely end up with unjoined groups.

I get more control using expressions and they can be used in more places. I'd love both, since you are right that often expressions are more generalized, but if I only get one I'd usually rather have a clean expression and minimal template than do a lot of data manipulations using the go/bosun templates.
In this case I was using the expression page for reporting, and a hide false button would also have worked. But the point is having it as an expression solves all of those cases, and potential future ones like grafana sourcing directly from bosun.

authors: gbrayut kylebrandt

repo: bosun-monitor/bosun

license: mit

license_source: bigquery


text:
graphite: support HTTPS hosts

fixes #812
closes #824

+1

authors: captncraig mjibson


text:
Bugfix/socketiorasahighavailable

Proposed changes:

Transports should be ['websocket', 'polling']
Allow to pass protocolOptions to socket.io configuration.

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog

Justification::
Deploying RASA in high availability and connecting with socket.io  causes you to need to set up a sticky session in the load balancer (for example using the client IP). This is due to the fact that Socket.io is not using in the RASA image an adapter to use Redis and be able to load balance the socket connection among the nodes. In the documentation of socket.io, they refer that it is only happening if by default it uses "polling". With the transport "websockets" it's not needed to set up a sticky session.
By default, socket.io has the transports ["polling", "websocket"] instead of the inverse order. The inverse order (the proposed one in this pull request) is the recommended order for high availability (they mention "the order matters").
Reference: https://socket.io/docs/v3/using-multiple-nodes/
The second proposed change is for configuring the timeouts (among other things). The default timeout of socket.io for reconnection is 20 seconds. But having in place a high availability RASA set up, 20 seconds is a lot and we would like to define a specific configuration for the socket connection.
This fixes #416 and probably #166.
Thanks.

@MatthieuJnon you were involved in the https://github.com/botfront/rasa-webchat/issues/166, maybe you can help with this.
Thanks.

repo: botfront/rasa-webchat


text:
[feat][zephyr] Enable bl_mcu_sdk as zephyr module

The ZephyrRTOS is a newer Open Source RTOS. It strives to deliver the best-in-class RTOS for connected resource-constrained devices, built to be secure and safe.
This add ZephyrRTOS entry point and define environment to build Bouffalo Lab low level peripheral drivers as hal_bouffalolab. Initially, bl602 is the target but can be extended to any other SoC series present in the bl_mcu_sdk.
The commit sequence try fix small issues found due to code isolation. The BFLB_USE_PLATFORM was necessary to be created to isolate low level drivers and Bouffalo Lab HAL, once Zephyr have their own.
The zephyr directory is completely isolated from bl_mcu_sdk and not interfere. This layout was created to allow add manufacturer repositories as external modules into ZephyrRTOS.
Currently there is a minimal attempt to enable Bouffalo Lab SoC at ZephyrRTOS at PR 37686
How to Build/Flash on a working zephyr environment:

Clone PR 37686
Read board/riscv/dt_bl10_devkit/doc/index.rst
west update
west build -b dt_bl10_devkit samples/hello_world
west flash

Hi @sakumisu ,
I was thinking about this issue and realize that real problem is that arch dependent files are together with SoC registers definition. RISC-V core files are used in very few places: risc-v timer, interrupts and system initialization; and can be defined by RTOS, like Zephyr is doing.
drivers/blX02_driver/hal_drv/src/hal_mtimer.c
drivers/blX02_driver/startup/interrupt.c
drivers/blX02_driver/startup/system_bl602.c

On my view, drivers never will require any risc-v/arm definitions and that entries could be safe removed from blX02.h. It require only add proper include in a few places like f33ca05. I'm asking to you review this option because for any RTOS those will be specific. In case of Zephyr, risc-v timer, interrupts and system initialization will be made in the main tree and will not use bl_mcu_sdk.
On my view, and I can be wrong, seems better not contaminate register include files.
Could you check if this is a valid solution ?

Hi @sakumisu ,
I was wondering if you have time to check my last comment.
BTW, Nuclei specific will be add as a newer file instead a full module. This reinforce that https://github.com/bouffalolab/bl_mcu_sdk/pull/18#issuecomment-953432666 may be a definitive solution. Do you mind send me PVT your expected solution so I can incorporate at Draft? If you prefer, you can simple open a Zephyr PR adding missing entries.

@nandojve Hi, we have evaluated your solution,is a better way currently, and next time will be merged in locally.

Already merged

authors: nandojve sakumisu


text:
How to solve data size mismatch? Thank you very much for your help.

You need to resize the output of medsam (3403,4877) to the size of the gt_data (256,256) before computing the dice coefficient!!

repo: bowang-lab/MedSAM


text:
Source for images

@rasa,
I'm trying to create the images but it seems like the pre-requisite is that i need to have those .iso mentioned in the .json files?

Yes, you can use https://github.com/boxcutter/windows/blob/master/.windows/isos/Makefile to download the evaluation .ISOs. Then create an /iso folder, and move them into it.

authors: kennethlimcp rasa

repo: boxcutter/windows


text:
Cleanup: Remove obsolete files

The files originally come from bx_py_utils:
https://github.com/boxine/bx_py_utils/tree/master/bx_py_utils_tests/tests/graphql_introspection
They are not needed in by_django_utils and they were overlooked while split by_django_utils from bx_py_utils.

Codecov Report

Merging #80 (1a94889) into master (fe204c5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #80   +/-   ##
=======================================
  Coverage   94.99%   94.99%
=======================================
  Files         104      104
  Lines        3798     3798
=======================================
  Hits         3608     3608
  Misses        190      190

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

authors: codecov-commenter jedie


text:
Interoperability testing for unstructured grids

The new resqpy unstructured grid classes have been unit tested. However, no inter-operability tests have yet been carried out reading datasets generated elsewhere, or transferring resqpy datasets with unstructured grids to non-resqpy software. Help with this would be appreciated.

@mgimhoff Is this something you could help with by any chance?

repo: bp/resqpy


text:
Add update for bpkg

Since release v0.3.2 bpkg now also allows user-level installs.

Excellent update. Thanks for this correction.

Excellent update. Thanks for this correction.

authors: Potherca fidian

repo: bpm-rocks/bpm


text:
Remove user management view from old UI

User story
As a case worker I find it irritating that the UI has a dysfunctional user management view.
Feature explanation

Remove the user management view
Remove the link to the user management view from the menu on the left

Benefits for users

The UI is clearer and less confusing

Implementation ideas
This should be easily done.

Applied by #182

repo: bptlab/chimera


text:
Handling of renewals of trials

I still wonder how to properly handle renewals of the trial subscriptions.
Scenario:

user registers at the website
at the moment of registration gets a subscription with a trial (Basic / Pro - does not matter, but with trial days)
consumes X times the features (which are registered with recordFeatureUsage())
at the end of the trial decides to pay
payment is successful and now, upon issuing ->renew() all usages consumed during trial are reset as in the renew() method there is $subscription->usage()->delete();

Am I missing something? I'd rather kept the usages consumed during trial.

(I think we can continue the discussion at https://github.com/bpuig/laravel-subby/issues/58)

repo: bpuig/laravel-subby


text:
Cannot load bqplot to jupyterlab from dev. js Error: Plugin 'bqplot' failed to activate.

Hello,
I tried to clone this repository and install from dev to test in jupyter lab. I get the js console error Plugin 'bqplot' failed to activate.
index.js:268 Error: No provider for: jupyter.extensions.jupyterWidgetRegistry.
    at JupyterLab.push.2R+v.Application.resolveRequiredService (index.js:182)

To install, I ran:
cd bqplot
pip install -e .
cd js
jupyter labextension install .

Output of  jupyter labextension list:
JupyterLab v1.2.0
Known labextensions:
   app dir: <path>/miniconda3/envs/JupyterLabv1/share/jupyter/lab
        @jupyter-widgets/jupyterlab-manager v1.1.0  enabled  OK
        bdgenomics.mango.pileup v0.0.5  enabled  OK
        bqplot v0.5.5  enabled  OK*

   local extensions:
        bqplot: <path>/bqplot/js

How do I properly get the JLab extension running from dev? Thanks in advance.

I see you cloned bqplot. If you are using latest master you should use JupyterLab 2. and the latest version of ipywidgets.

Thanks @martinRenou! Where is required version for Jupyter Lab documented? I was unable to find this.

@akmorrow13 , have a look at this: #2798, it sheds some light. Also, here there is a table for front/back-end compatibility.

Thanks @martinRenou

authors: acelere akmorrow13 martinRenou

repo: bqplot/bqplot


text:
Autopublish?

Create a workflow to handle automatic publishing to PyPI

I would add that doc site would be more up to date too, (I was afraid of the changelog page)


text:
can't build with the plugin

hello, i'm facing this problem when i build the application after adding texttospeech plugin
`
/home/uids7795/workspace/poc/testAppsidekick/platforms/android/src/main/java/com/tns/gen/android/speech/tts/UtteranceProgressListener_ftns_modules_nativescript_texttospeech_texttospeech_android_l10_c88__.java:3: error: '{' expected
public class UtteranceProgressListener_ftns_modules_nativescript_texttospeech_texttospeech.android_l10_c88__ extends android.speech.tts.UtteranceProgressListener implements com.tns.NativeScriptHashCodeProvider {
^
/home/uids7795/workspace/poc/testAppsidekick/platforms/android/src/main/java/com/tns/gen/android/speech/tts/UtteranceProgressListener_ftns_modules_nativescript_texttospeech_texttospeech_android_l10_c88__.java:4: error:  expected
public UtteranceProgressListener_ftns_modules_nativescript_texttospeech_texttospeech.android_l10_c88__(){
`
i really don't know where this come from
PS : I'm new to nativescript

Not sure either from that output. It's not very clear TBH. Is this output in your terminal? I see testAppsidekick so are you using Sidekick? If so, you might want to ask Progress why it fails with sidekick. The demo app seems to be building and running fine with the plugin. So your project is jacked up somewhere.
I would try nuking your node_modules and platforms, add the plugin and then do a fresh build. However, I don't use sidekick so I don't know how that process is done compared to using the CLI.

Just updated the demo to NS 3x and confirmed all is working fine. Nuking your platforms and a fresh build should solve your issue, if not and using sidekick I would put a support ticket in with them as I can't reproduce this error. Thanks.

authors: bradmartin brendiche

repo: bradmartin/nativescript-texttospeech


text:
Add tests for some utility functions

Description:
This PR adds some tests for the following utility functions ->

[x] format-codeblock.ts
[x] formatter.ts
[x] is-supported-by-prettier.ts

Awesome thanks, I'll review this on my side later today :)

authors: Twaha-Rahman bradtaniguchi

repo: bradtaniguchi/discord-bot-test


text:
Uptime and Wake Time Closes (#59)

This will close #59

[x] Uptime will display the time the bot has been up for, [DD] [HH] [MM] [SS] format.
[x] Bot Wake Up Time will show the time the bot was up at. [HH:MM] [AM/PM] [TimeZone] format. It is also linked to a google search for the time to show the relative time to the user's area.

It turns out we didn't need the JSON file after all. Just a simple export from the start. The Wake Time will open up a google search to their relative location since the bot will only give the time from where it's located. So I just rigged up a google search query and let Google do the job for me 😆.
Here's an example:

Note: Ignore most of the commits, most of them are me Trying to merge the upstream to my master

I have removed the google search link since it's really unnecessary.
The onlineAt is moved into get-config.
The getOnlineAt() function is now utilizing the luxon package. Now it is cleaner using the object wrapper and formatting from luxon.
The stats.ts file will now use the config to get the Online Time.

Wait for #88 To be review+merged, then we can use the config object that is initially set.

@JoshuaPelealu, with the updates in master, you should be able to get the config object in the function callback:
Here is an example from the close command:
https://github.com/bradtaniguchi/discord-bot-test/blob/master/src/commands/close.ts#L10
So you can slightly refactor the code to remove the extra getConfig call, and then we should be good to go :smile:

The function should now get 2 arguments inside the parameter and the import is not used.

authors: JoshuaPelealu bradtaniguchi


text:
Changing const listers to liters

Changed "listers" for "liters" . It works because its written the same when referencing it, but I think it would be better if changed for the actual word liters :)

Already fixed, thanks :)

authors: bradtraversy jaumemy


text:
Step interval range improvements

This introduces a new resource called CommStartingX0 which is used for step interval range expressions (i.e. 2/3) so that that, for example, the expression "5/30 * * * * ?" reads as:

"Every 30 seconds, starting at 5 seconds past the minute"

whereas before it read as:

"Every 30 seconds, seconds 5 through 59 past the minute"

Previously, we were converting all step interval ranges to "between" expressions to be able to re-use the existing translations but the descriptions were awkward and misleading.
We will leave the "between" conversion in place for Month, DOW, and Year parts as between handling results in acceptable descriptions and avoids more significant reworking of the translations.
Translation Reviews

[x] English @bradyholt
[ ] Brazilian @natenho
[x] Spanish @ivansg
[ ] Norwegian @KhalipskiSiarhei
[ ] Turkish @sadedil
[x] Dutch @TotalMace
[ ] Chinese Simplified @starpeng
[ ] Russian @LbISS
[ ] French @Greybird
[ ] German @mschuler
[x] Ukrainian @tbudurovych
[ ] Italian @rinaldihno
[ ] Polish @foka
[ ] Romanian @illegitimis

Every everybody, I've made some improvements to step interval range handling and could really use your help reviewing the new CommStartingX0 resource/translation.  I used Google Translate to get the translations and adding unit tests but your confirmation would be very helpful!

Ukrainian looks good to me

Spanish version is good to go

Dutch is also working correct

Brazilian-portuguese looks fine...but
I've noticed that Resources.pt-br.resx has been renamed to
Resources.pt.resx.
I'm not quite sure, but it can conflict when someone wants to add the other
portuguese translation (i.e. Portugal portuguese), because pt is slightly
different from pt-BR...
Cheers,
2016-06-25 4:28 GMT-03:00 TotalMace notifications@github.com:

Dutch is also working correct
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bradyholt/cron-expression-descriptor/pull/64#issuecomment-228519288,
or mute the thread
https://github.com/notifications/unsubscribe/AECkwSS7wQTDRcaKA4Am7rmRn-oZU5NQks5qPNiJgaJpZM4I2cXn
.

Russian looks fine.

German looks good too.

authors: LbISS TotalMace bradyholt ivansg mschuler natenho tbudurovych

repo: bradyholt/cron-expression-descriptor


text:
Consider replacing element with bootstrap / bootswatch.

I am not sure element ui is the best choice afterall.. I might switch back to bootstrap/bootswatch.

This has been an on-going effort, but good chunk of element UI tags have been replaced already.
I only have 524 tags left to go.
~/git/warehouse/ui/src(master*) $ grep -r el- * | wc -l
524
I've noticed, and I believe the webpack is good at only including js/css from components that we are actually using. So it might not hurt to leave a few element-ui components after all (some element-ui works better than vue-bootstrap - like accordion)

repo: brain-life/warehouse


text:
fix bugs in evaluate_model.py

Thank you for your code! I find that it only use half the dataset to test model. You can print total_seen in evaluate_model.py to see this bug.

Thank you @DoctorKey, you seem to be right. I assume this won't significantly change the results since we were using an unbiased sample from the true dataset, but it may reduce the variance of some of the runs.

authors: DoctorKey avital

repo: brain-research/realistic-ssl-evaluation


text:
i need to define my own pytorch model. Can I subclass it from nn.module

Hello,
I need to define my own py-torch model.
Can I subclass it from nn.module ?
or I need to subclass it from nn.sequential as done in all the models of brain decode ?

Yes it is totally fine to subclass from nn.Module. Only to_dense_prediction_model makes very specific assumptions about the model, so as long as you don't need that you should be fine. This function is only needed if you want to do cropped decoding and therefore need per-timestep outputs, but your original model has stride in pooling or conv layers and therefore does not have outputs for each timestep

@robintibor
Thanks for the reply
and its a wonderful project

repo: braindecode/braindecode


text:
Error call undefined method when call mixed-content-scan

When I call mixed-content-scan, I got this error:
Fatal error: Call to undefined method Monolog\Logger::addEmergency() in C:\Users\Ami\AppData\Roaming\Composer\vendor\bramus\mixed-content-scan\src\Scanner.php on line 93

Solved by using composer global require monolog/monolog
Because mixed-content-scan need this package https://github.com/Seldaek/monolog

repo: bramus/mixed-content-scan


text:
send message on quit

fixes #15

added type checking to src/js/main.js
track state.ui.quitMsg
define SET_QUIT_MSG action
wire up to <Settings />

inspired by prior work from @Fohlen

repo: brandly/irc


text:
Scroll fidgety when react-text-loop present

Having some issues with the scroll on mobile. It's very fidgety on the pages that have react-text-loop active.
The application is built on gatsby.
Bug only appears to be present on mobile. Im using chrome on mobile.

I'm having this same issue 🤕 I thought it had to do with the fact that i had min-height and margins set on the container but removing them hasn't solved the issue.
Has anyone had any luck with solving this?

@nugoo1 @andelh sorry about the delay in the response but would you be able to create a sandbox that reproduces the issue? You can use https://codesandbox.io/s/react-text-loop-playground-br4q1?file=/src/App.tsx as a base.
I'm not aware of any performance issues on scroll so it would be great if we could have a reproduceable example. Thanks!

authors: andelh braposo nugoo1

repo: braposo/react-text-loop


text:
Update react and react-dom peerDependencies

Update both react and react-dom to 16.7.0
Fixes #32

@abhijitparida so sorry about the delay but I didn't notice the PR at all!
Thanks for opening this, I just edited your change to include React 15 as it's still supported and some people haven't updated to React 16.

authors: abhijitparida braposo


text:
🛑 Charite gateway-iop-connection is down

In 4417df4, Charite gateway-iop-connection (https://ids.health-x.charite.de/health/4) was down:

HTTP code: 500
Response time: 176 ms

Resolved: Charite gateway-iop-connection is back up in 232c1bc after 12 minutes.

repo: braunma/dataloft-endpoint-monitor


text:
falsey check in hasProperty (existing pull request)

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
Would it be possible to merge this pull request from orig repo?
https://github.com/nartc/mapper/pull/573
Models/DTOs/VMs
No response
Mapping configuration
No response
Steps to reproduce
No response
Expected behavior
(see orig pull request)
Screenshots
No response
Minimum reproduction code
No response
Package

[ ] I don't know.
[ ] automapper-core
[ ] automapper-classes
[ ] automapper-nestj
[ ] automapper-pojos
[ ] automapper-mikro
[ ] automapper-sequelize
[ ] Other (see below)

Other package and its version
No response
AutoMapper version
8.0.0
Additional context
No response

ill take a look..

authors: adros brdar

repo: brdar/mapper-js


text:
Disable protocol version check when connecting with remote peer

Hi all,
I'm in the process of connecting the android breadwallet app with a private multichain which I have set up  on my localhost. In the mobile device where the breadwallet app runs, the connection keeps syncing (until 5%) and gets stuck without either connecting or displaying the message 'not connected'.
In the Multichain logs I can see that the connection gets stuck when the minimum parameter set is sent from the Multichain node to the breadwallet node. I suppose the problem is that the breadwallet app can't accept the protocol version of the Multichain node (10008).
Multichain developers propose to

Ensure the wallet accepts the protocol-version which is defined on your chain, or remove the check completely to allow your protocol to be upgraded in future.

The question is, how can I completely remove the check for the version  of the remote peer within the breadwallet android code? Many thanks in advance for your help!
Regards,
Natassa

The check happens in

BreadWallet/app/src/main/jni/breadwallet-core/BRPeer.c

Please take a look there.

authors: Gutyn anastath

repo: breadwallet/breadwallet-android


text:
cem-plugin-expanded-types: Error when analyzing types with index signatures

When trying to analyze types like:
  @property({ type: Object })
  foobar: { [key: string]: any } = { foo: 'bar' }

I get the following error:
Error:

[expand-types-plugin]: Looks like you've hit an error in third party plugin: expand-types-plugin. Please try to create a minimal reproduction and inform the author of the expand-types-plugin plugin.

 SyntaxError: Invalid regular expression: /[key/: Unterminated character class
    at new RegExp (<anonymous>)
    at file:///workspace/node_modules/cem-plugin-expanded-types/dist/index.js:84:7
    at Array.forEach (<anonymous>)
    at getObjectTypes (file:///workspace/node_modules/cem-plugin-expanded-types/dist/index.js:77:9)
    at getExpandedType (file:///workspace/node_modules/cem-plugin-expanded-types/dist/index.js:54:12)
    at getTypeValue (file:///workspace/node_modules/cem-plugin-expanded-types/dist/index.js:193:12)

the problematic regexp seems to be this:
https://github.com/break-stuff/cem-tools/blob/8187639d731c8bcfe0df13b65f1ada8c4db397a4/packages/expanded-types/src/cem-plugin.ts#L133
when run, the cleanPart variable holds the string [key which will not produce a valid regexp.
when creating a dedicated type or interface for that, it works:
  type FooBar = { [key: string]: any };

...

  @property({ type: Object })
  foobar: FooBar = { foo: 'bar' }

Do you have a repro I can test? Preliminary tests seem to work fine:
"{ [key: string]: any }".replace(/\/\*[\s\S]*?\*\/|(?<=[^:])\/\/.*|^\/\/.*/g, '')

Same error hit with an { [key: string]: string } type.

I'm sorry for the delay on this! The patch for it has gone out in version 1.3.3. Let me know if you run into any issues.

authors: break-stuff certainlyakey h4de5

repo: break-stuff/cem-tools


text:
Support other C Language Version

In jupyter_c_kernel, codes would be compiled in C11, why not add support for other c version.
Currently, I've tried add an compile flag //%cflags:-std=gnu89, but it didn't work. After I checked the source code of the kernel.py, I found that the -std=c11 is hard coded in it. I tried removing it, and restart the kernel, but it doesn't work too.
So I wonder if it's hard to enable other c version, if not, would you please add an support for it?

In jupyter_c_kernel, codes would be compiled in C11, why not add support for other c version.
Currently, I've tried add an compile flag //%cflags:-std=gnu89, but it didn't work. After I checked the source code of the kernel.py, I found that the -std=c11 is hard coded in it. I tried removing it, and restart the kernel, but it doesn't work too.
So I wonder if it's hard to enable other c version, if not, would you please add an support for it?

You can make your own one, depending on your gcc supporting languages.
Just read the source code carefully.

authors: WqyJh takinekotfs

repo: brendan-rius/jupyter-c-kernel


text:
setting document root to /public

hi there,
i'm just trying to setup my first silverstripe environment with this docker-image, docker-compose and composer. normally i'd set the document root to the /public folder but the documentroot is set in the dockerfile in this image and not changeable (except forking this repository). also the create-project silverstripe command does not seem to do it this way - instead a .htaccess in the root directory rewrites all requests to the /public folder - is this a good idea and if yes, why?
i think a better aproach would be to allow customization of the document root via an environment variable as in the official apache docker images.

Hi Gaambo,
I will put this as a requested feature to be able to configure the Document Root using environment variables.
In the meantime, you could create your own Dockerfile FROM this build, and modify the Apache config settings for web root to be /public.


text:
Change properties at runtime

Is it possible to modify any property at runtime? For instance, changeing the maximum number of database connections, timeouts and so on?

You can change some settings through the MBean.
https://github.com/brettwooldridge/HikariCP/wiki/MBean-(JMX)-Monitoring-and-Management
You can either register it in JMX and retrieve it from there, or call HikariDataSource.getHikariPoolMXBean() to access it.

authors: brettwooldridge du-it

repo: brettwooldridge/HikariCP


text:
HikariPool-1 - Connection is not available, request timed out after 30002ms.

Hi,
I am currently using Hibernate ORM with JPA WITHOUT Spring Data and wanted to use HikariCP.
After some time I get these errors and no DB requests are responding (I can't get any DATA).
I am also using CDI from WELD but that shouldn't be a problem.
This is what I use and how I use it:
    implementation 'org.hibernate:hibernate-core:5.6.5.Final'
    runtimeOnly 'org.hibernate:hibernate-hikaricp:5.6.5.Final'

This is my persistence.xml
<persistence
        xmlns="http://xmlns.jcp.org/xml/ns/persistence"
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xsi:schemaLocation="http://xmlns.jcp.org/xml/ns/persistence
             http://xmlns.jcp.org/xml/ns/persistence/persistence_2_1.xsd"
        version="2.1">
    <persistence-unit name="PERSISTENCE">
        <description>Interception Hibernate ORM</description>
        <provider>org.hibernate.jpa.HibernatePersistenceProvider</provider>

        <class>dev.marvn.interception.db.entitys.ServerDataEntity</class>
        <class>dev.marvn.interception.db.entitys.WelcomeDataEntity</class>
        <class>dev.marvn.interception.db.entitys.VerifyRoleEntity</class>
        <class>dev.marvn.interception.db.entitys.VerifyDataEntity</class>
        <class>dev.marvn.interception.db.entitys.TempChannelTemporaryEntity</class>
        <class>dev.marvn.interception.db.entitys.TempChannelMainEntity</class>
        <class>dev.marvn.interception.db.entitys.TempChannelCommandChannelEntity</class>
        <class>dev.marvn.interception.db.entitys.JoinRoleEntity</class>

        <properties>
            <property name="javax.persistence.jdbc.driver"
                      value="com.mysql.cj.jdbc.Driver"/>
            <property name="javax.persistence.jdbc.url"
                      value="XXXXXXX"/>
            <property name="javax.persistence.jdbc.user" value="XXXXX"/>
            <property name="javax.persistence.jdbc.password" value="XXXX"/>
            <property name="hibernate.show_sql" value="true"/>
            <property name="hibernate.hbm2ddl.auto" value="none"/>

            <property name="hibernate.hikari.maximumPoolSize" value="15"/>
            <property name="hibernate.hikari.idleTimeout" value="30000"/>
            <property name="hibernate.connection.provider_class"
                      value="org.hibernate.hikaricp.internal.HikariCPConnectionProvider"/>
        </properties>
    </persistence-unit>
</persistence>

This is one method which I call pretty often. #getEntityManager does return a freshly build entityManager!
    public Optional<TempChannelTemporaryEntity> getTempChannelTempByVoiceChannelId(Long voiceChannelId) {
        var entityManager = getEntityManager();
        var query = entityManager.createNamedQuery("TempChannelTemporary.GetByVoiceChannelId", TempChannelTemporaryEntity.class);
        query.setParameter("id", voiceChannelId);
        return query.
                getResultStream()
                .findAny();
    }

This is my error
[ERROR] 2022-02-24 10:16:41,648 | org.hibernate.engine.jdbc.spi.SqlExceptionHelper : logExceptions > HikariPool-1 - Connection is not available, request timed out after 30002ms.
2022-02-24T10:16:41.658336500Z [ERROR] 2022-02-24 10:16:41,653 | org.javacord.core.util.logging.PrivacyProtectionLogger : logMessage > Unhandled exception in a listener thread for Marvn's behinderter Test Server voll mit unfreundlichen Bots!
2022-02-24T10:16:41.658384300Z  org.hibernate.exception.JDBCConnectionException: Unable to acquire JDBC Connection
2022-02-24T10:16:41.658402500Z 	at org.hibernate.exception.internal.SQLExceptionTypeDelegate.convert(SQLExceptionTypeDelegate.java:48) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658418000Z 	at org.hibernate.exception.internal.StandardSQLExceptionConverter.convert(StandardSQLExceptionConverter.java:37) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658432200Z 	at org.hibernate.engine.jdbc.spi.SqlExceptionHelper.convert(SqlExceptionHelper.java:113) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658447500Z 	at org.hibernate.engine.jdbc.spi.SqlExceptionHelper.convert(SqlExceptionHelper.java:99) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658459600Z 	at org.hibernate.resource.jdbc.internal.LogicalConnectionManagedImpl.acquireConnectionIfNeeded(LogicalConnectionManagedImpl.java:111) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658473500Z 	at org.hibernate.resource.jdbc.internal.LogicalConnectionManagedImpl.getPhysicalConnection(LogicalConnectionManagedImpl.java:138) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658488800Z 	at org.hibernate.engine.jdbc.internal.StatementPreparerImpl.connection(StatementPreparerImpl.java:50) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658497400Z 	at org.hibernate.engine.jdbc.internal.StatementPreparerImpl$5.doPrepare(StatementPreparerImpl.java:149) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658510000Z 	at org.hibernate.engine.jdbc.internal.StatementPreparerImpl$StatementPreparationTemplate.prepareStatement(StatementPreparerImpl.java:176) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658526800Z 	at org.hibernate.engine.jdbc.internal.StatementPreparerImpl.prepareQueryStatement(StatementPreparerImpl.java:151) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658544900Z 	at org.hibernate.loader.Loader.prepareQueryStatement(Loader.java:2122) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658573700Z 	at org.hibernate.loader.Loader.executeQueryStatement(Loader.java:2059) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658591400Z 	at org.hibernate.loader.Loader.executeQueryStatement(Loader.java:2037) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658605000Z 	at org.hibernate.loader.Loader.scroll(Loader.java:2945) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658620800Z 	at org.hibernate.loader.custom.CustomLoader.scroll(CustomLoader.java:383) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658636600Z 	at org.hibernate.internal.SessionImpl.scrollCustomQuery(SessionImpl.java:2156) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658659600Z 	at org.hibernate.internal.AbstractSharedSessionContract.scroll(AbstractSharedSessionContract.java:1209) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658678500Z 	at org.hibernate.query.internal.NativeQueryImpl.doScroll(NativeQueryImpl.java:224) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658690900Z 	at org.hibernate.query.internal.AbstractProducedQuery.scroll(AbstractProducedQuery.java:1574) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658704500Z 	at org.hibernate.query.internal.AbstractProducedQuery.stream(AbstractProducedQuery.java:1598) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658719800Z 	at org.hibernate.query.Query.getResultStream(Query.java:1140) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.658736200Z 	at dev.marvn.interception.db.DatabaseManager.getServerData(DatabaseManager.java:38) ~[Interception-Javacord-3.0.3.jar:?]
2022-02-24T10:16:41.658765300Z 	at dev.marvn.interception.db.DatabaseManager$Proxy$_$$_WeldClientProxy.getServerData(Unknown Source) ~[Interception-Javacord-3.0.3.jar:?]
2022-02-24T10:16:41.658777700Z 	at dev.marvn.interception.commandFramework.CommandContextListener.transform(CommandContextListener.java:46) ~[Interception-Javacord-3.0.3.jar:?]
2022-02-24T10:16:41.658790000Z 	at dev.marvn.interception.commandFramework.CommandContextListener$Proxy$_$$_WeldClientProxy.transform(Unknown Source) ~[Interception-Javacord-3.0.3.jar:?]
2022-02-24T10:16:41.658804900Z 	at net.kautler.command.api.CommandHandler.computePrefix(CommandHandler.java:315) ~[command-framework-0.4.0-SNAPSHOT.jar:?]
2022-02-24T10:16:41.658821400Z 	at net.kautler.command.api.CommandHandler.doHandleMessage(CommandHandler.java:275) ~[command-framework-0.4.0-SNAPSHOT.jar:?]
2022-02-24T10:16:41.658838000Z 	at net.kautler.command.handler.CommandHandlerJavacord.handleMessage(CommandHandlerJavacord.java:174) ~[command-framework-0.4.0-SNAPSHOT.jar:?]
2022-02-24T10:16:41.658853600Z 	at org.javacord.core.util.event.EventDispatcher.lambda$dispatchMessageCreateEvent$146(EventDispatcher.java:5249) ~[javacord-core-3.4.0.jar:?]
2022-02-24T10:16:41.658870700Z 	at org.javacord.core.util.event.EventDispatcherBase.lambda$dispatchEvent$12(EventDispatcherBase.java:209) ~[javacord-core-3.4.0.jar:?]
2022-02-24T10:16:41.658881800Z 	at org.javacord.core.util.event.EventDispatcherBase.lambda$checkRunningListenersAndStartIfPossible$20(EventDispatcherBase.java:277) ~[javacord-core-3.4.0.jar:?]
2022-02-24T10:16:41.658925300Z 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) [?:?]
2022-02-24T10:16:41.658944200Z 	at java.util.concurrent.FutureTask.run(FutureTask.java:264) [?:?]
2022-02-24T10:16:41.659010200Z 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
2022-02-24T10:16:41.659019800Z 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
2022-02-24T10:16:41.659032900Z 	at java.lang.Thread.run(Thread.java:832) [?:?]
2022-02-24T10:16:41.659047900Z Caused by: java.sql.SQLTransientConnectionException: HikariPool-1 - Connection is not available, request timed out after 30002ms.
2022-02-24T10:16:41.659059700Z 	at com.zaxxer.hikari.pool.HikariPool.createTimeoutException(HikariPool.java:676) ~[HikariCP-3.2.0.jar:?]
2022-02-24T10:16:41.659076700Z 	at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:190) ~[HikariCP-3.2.0.jar:?]
2022-02-24T10:16:41.659092700Z 	at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:155) ~[HikariCP-3.2.0.jar:?]
2022-02-24T10:16:41.659106100Z 	at com.zaxxer.hikari.HikariDataSource.getConnection(HikariDataSource.java:100) ~[HikariCP-3.2.0.jar:?]
2022-02-24T10:16:41.659125000Z 	at org.hibernate.hikaricp.internal.HikariCPConnectionProvider.getConnection(HikariCPConnectionProvider.java:77) ~[hibernate-hikaricp-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.659141400Z 	at org.hibernate.internal.NonContextualJdbcConnectionAccess.obtainConnection(NonContextualJdbcConnectionAccess.java:38) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.659159800Z 	at org.hibernate.resource.jdbc.internal.LogicalConnectionManagedImpl.acquireConnectionIfNeeded(LogicalConnectionManagedImpl.java:108) ~[hibernate-core-5.6.5.Final.jar:5.6.5.Final]
2022-02-24T10:16:41.659177400Z 	... 31 more

I have no clue what's going on. I need help.Just let me know if you need additional data
Best Regards

I had a same question.
Finally, I found that my param maximum-pool-size is too small.
My program is web's Program, and the servlet work thread is bigger than the hikariCP's maximum-pool-size.
Sometimes, it had the HikariPool-1 - Connection is not available error.

I had a same question. Finally, I found that my param maximum-pool-size is too small.
My program is web's Program, and the servlet work thread is bigger than the hikariCP's maximum-pool-size. Sometimes, it had the HikariPool-1 - Connection is not available error.

Is the connection closed  when the query had completed?

No, when the query had completed, the connection return to Hikari‘s Pool.

authors: RealMarvn icodemo shangweikun


text:
Use the fields value if it exist

I have an error when using the polyfill in Safari (probably other browsers as well but only looking in Safari). If the date input has a value on page load the polyfill ignores it.
In my situate someone comes to the form to edit the info and the currently saved info is there for them to edit. if they do not touch the date inputs and hit save then empty date fields are saved and the default date is blown away.

How is the field data being populated?  Keep in mind that setAttribute will not change the underlying value.

Please comment if you are still experiencing this issue.  Closing for now.

repo: brianblakely/nodep-date-input-polyfill


text:
Fixed ownerWorkspace so it actually adds the associated topics

added logic in dashboard controller to get topics for ownerWorkspace rather than always being blank

@KyleRess-Liere @Caleb-Rogers @Mike-Volchko glad you guys are banging these out - please don't forget to delete the stale branches if possible

authors: KyleRessLiere cmarcusdimarco

repo: briangormanly/agora


text:
Add PBandJ (and fix formatting)

Add PBandJ
Remove all deprecated/unmaintained packages in that category
Format styling (remove end-of-line spaces)
Remove flairs (unnecessary emojis)

I am happy to help clean up this repo if you want some help. I'm sure 50% of the listed packages are unmaintained so we could really up the quality by removing these.
Regardless thanks for always merging PR's here!

Definitely appreciate your help. Great work, and look forward to more when you have time!

authors: bluepeter moishinetzer

repo: brillout/awesome-react-components


text:
In memory build instead using temp directory

I've read the code, and the plugin seems to save file in a temp location (on disk) and read it again to provide it to the stream.
In SystemJS / JSPM, you can do everything in memory : https://github.com/systemjs/builder/issues/365
Like @gulpjs said on twitter : if you are using temp files or folders in gulp you are doing something horribly wrong
-> https://twitter.com/contrahacks/status/656343475743428608
I think this issue is an improvement, and if I have enough time, I will try to do a PR.
Thanks

The JSPM API is expecting a path.
I don't see a way to keep all in memory without changing the code of https://github.com/jspm/jspm-cli.
But it would be awesome if you find a way! Thanks!

JSPM Builder seems to delegate to a SystemJS Builder, and the SystemJS builder can do all the work in memory : https://github.com/systemjs/builder#in-memory-builds
The output in the promise contains sources, sourceMap, module, assetList (not in the doc, only in https://github.com/systemjs/builder/issues/365 for now).
Say me if I missed something... (I've tried it in a project, seems to work).

looks good, awesome

see #16

I think this would also fix the problem where you want to merge 2 streams that perform jspm() bundling, right now the following code fails because jspm cannot use the same temporary file twice concurrently:
es.merge(
  gulp.src('foo').pipe(jspm()).pipe(concat('foo')),
  gulp.src('bar').pipe(jspm()).pipe(concat('bar'))
).pipe(gulp.dest('dist/'))

[13:35:28] Error in plugin 'gulp-jspm'
Message:
    ENOENT: no such file or directory, open '/var/folders/5h/d5hh9f0s3zg6f1zn3zjtjtsr0000gn/T/gulp-jspm__build.js116522-22723-1dljtv3'
Details:
    cause: Error: ENOENT: no such file or directory, open '/var/folders/5h/d5hh9f0s3zg6f1zn3zjtjtsr0000gn/T/gulp-jspm__build.js116522-22723-1dljtv3'
    isOperational: true
    errno: -2
    code: ENOENT
    syscall: open
    path: /var/folders/5h/d5hh9f0s3zg6f1zn3zjtjtsr0000gn/T/gulp-jspm__build.js116522-22723-1dljtv3

@weisk
any luck with your issue? Note that gulp-jspm creates a new temporary file for every call gulp-jspm() and that gulp-jspm uses temp's automatic cleanup feature require("temp").track();.

for now my workaround is just doing them in series:
export default function scripts() {
  const bundles = [
    (cb) =>  gulp.src('foo').pipe(jspm()).pipe(concat('foo')),
    (cb) =>  gulp.src('bar').pipe(jspm()).pipe(concat('bar'))
  ];
  return gulp.series(...bundles);

if I try to run them in parallel, it breaks with the previous error

The issue pointed out by @weisk has a much simpler way to reproduce -- just try to run multiple JSPM builds in parallel.
Here is an example Gulp script that tries to build two versions, on minified and the other unminified:
function buildTask(src, options) {
	return gulp.src(src)
		.pipe(gulp_jspm(options))
		.pipe(gulp.dest('build/dist/'));
}

gulp.task('build:jspm', function() {
	return buildTask('main.js', {
		selfExecutingBundle: true,
		fileName: 'main',
	});
});

gulp.task('build:jspm:min', function() {
	return buildTask('main.js', {
		selfExecutingBundle: true,
		fileName: 'main.min',
		minify: true,
	});
});

gulp.task('build', ['build:jspm', 'build:jspm:min']);

If I run gulp build:jspm or gulp build:jspm:min, it works, but if I run gulp build, it throws the same error.

authors: brillout davinkevin kohenkatz weisk

repo: brillout/gulp-jspm


text:
Optimize custom font symbols

The symbols used in the plugin's custom fonts really need to be optimized by someone more adept with font editor software.  I would like to see the actual symbols appear slightly larger than currently, so that they're easier to read in deck editor and proxy generator.  Also, the kerning and positioning of each symbol should be verified.

I may look at this more later, but here are my initial thoughts:
There's at least one change that should happen: The original glyphs were placed in codepoints normally used for other characters such as "Ą". On the off-chance that someone in the world actually wants to type these characters in chat, they won't be able to (they'll get an MTG symbol instead). These glyphs should be moved into one of Unicode's private use areas, so they don't shadow any other characters. I've put the things I've added in one of these areas, but left the originals where they were for the time being.
As for the sizing, the symbols appear to be pretty accurate compared to a modern card:

They rest very slightly below the baseline and extend very slightly above the font's ascent. And the preview images for Oath actually have all of the glyphs smaller than this.
The proxies would look far better if OCTGN didn't render them in what's probably just about the worst possible way. The text will always be a little blurry at most sizes until they rewrite the generator. Also, if they rewrite it... color font support would be relatively easy to add there. In fact, it wouldn't be too hard to add without rewriting it, but it would involve adding a new library to the project to read the font's color palette and layers. Maybe someone should open an issue over there for this.
But in the meantime, there are a few things I think could be done to improve the look of the proxy cards.
The size of the name / cost text is actually a little too small right now:

So you can just bump that up. Adding a heavier mplantin font for that and the card type would make the proxies match the real thing a little closer too:

So blurry...
At small font sizes, using drop shadows tends to look better than outlines:

Can also use an emdash (—) instead of the double hyphen.
For the table, I find it nearly unusable as-is. What I recommend is actually to not use the mplantin font. Instead I use a font with only the symbols, so the other characters fall back to the default:

That's also slightly larger (14 instead of 12).

In fact, I'm actually of the opinion that the mplantin font should only be used for the proxy generation (and not chat). Even Wizards hardly uses it anywhere else. If you'd like, I can upload that font file for you too.

well as it is now, the hybrid symbols are actually larger than the regular ones so at the very least we should have a level of consistency there.  My main concern when I (hastily) edited the font for symbols was visibility, I wanted to make sure that users could easily see and recognize the symbols (mainly because I didn't have access to text coloring as a visual cue) so I'm not against making them bigger than normal.
I agree with moving the symbols to unused glyph slots - especially since we have the ligatures now to make user-searching and typing the symbols less painful.
As for the proxy stuff, AFAIK it was a compromise between generator efficiency and working around the platform's oddities.  I don't know what areas could be rewritten or improved upon, but the proxy gen is Gravecorp's realm so he would be the one to talk to about it.

and yeah I don't really have any attachment to the font being used outside the proxy templates.  It was mostly a "hey look, they added custom font support, might as well use it" sort of thing.  At the time I really liked the font and thought it was very easy on the eyes to read, but I'm not opposed to dropping down to a more basic font for the deck editor, game chat and context menus

The hybrid symbols are actually larger than the others on cards:

huh never would have guessed.

btw, if you ever need a quick message or contact I'm usually around the octgn chatroom (irc.freenode.net #octgn) and all the octgn devs are there too

ok so the current plan here is to move all mtg-related symbols to an unused section of the font, then have an mplantin version and a regular-text version.  The existing sizing and kerning is fine as-is.
If you wanted to make these changes to the font, that'd be a big appreciation as I'm not confident in my font-editing skills :) .  Then I can handle all the card data for the new symbol syntax for when the next set rolls out.
I'm currently looking into getting the color glyphs set up for OCTGN, and am working with the dev team to figure out a solution.

I'm unlikely to have time to make those changes before Oath. Things got really busy since the holiday season ended. :-/
The regular text version I attached to my comment above should be good to go though if you want to use that.

No worries I'll get it up and running

new OCTGN updates allow for actual symbol images in the text blocks, which should properly address this.  There are plans to extend the symbols to chat and dialog windows as well.

authors: brine ccbrown

repo: brine/mtg-octgn

license: Unlicense


text:
don't allow deleting everything

just a simple check: don't delete the whole table, you can't recover from that state

Pull Request Test Coverage Report for Build 1523280137

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 87.32%

Totals

Change from base Build 1522808228:
0.3%

Covered Lines:
233

Relevant Lines:
262

💛  - Coveralls

authors: artoonie coveralls

repo: brmighell/rcv-entry


text:
Improve build_docker.py and Dockerfiles

Three commits for this one

build_docker.py should do most things people want automatically (or close to it) with minimal fiddling of command-line arguments.
I pulled the conda environment build out of sv-pipeline-base and into its own Dockerfile. Now conda won't need to be rebuilt every time you update gatk-sv code.
I got some large docker images due to a mix of apt-get upgrades in the various Dockerfiles. So I made some small changes to how apt-get was invoked. The result is a modest decrease in docker image size (around 10%). My most recent build images are included in the committed dockers.json.

@mwalker174 I encountered problems building sv-pipeline-qc (as described during stand-up). I've pushed the changes I made due to that as a separate commit, which I will flatten if you approve this.
The outputs from my MakeCohortVcf run are at gs://broad-dsde-methods-tbrookin/results/MakeCohortVcf/
In case it matters, the workflow ID is d5ab3e46-1a8a-4e61-81a7-fac669f3372e

repo: broadinstitute/gatk-sv


text:
Max SNP # threshold

Hi! I'm wondering how was justified the threshold of the number of autosomal SNPs of 3.75M?
https://github.com/broadinstitute/gnomad_qc/blob/2f7c6a34818bd7f4300b9d9abba5f358dde4f4f4/gnomad_qc/v3/sample_qc/sample_qc.py#L200
I see that it's mentioned the gnomAD blog posts, but couldn't find any details. Neither couldn't find details on that in the gnomAD paper supplementary (sorry if I'm missing something!).
HGDP GVCFs have ~5.2M autosomal variants per sample (per n_snp calculated by Hail's sample_qc on bi-allelic autosomal calls, following your pipeline), and that's even after removing low quality with ReblockGVCF before feeding the GVCFs into the combiner. So not sure what other filtering steps did I miss. Unless VQSR was applied before sample QC?
Also, I understand the number of SNPs is different in different ancestries (e.g. more in African populations). I guess that's covered by the pop-stratified filtering down further the pipeline, and the hardfilter cut-off is to remove very clear outliers?

Hey, just a little update to be more specific :)
The pipeline I'm using is:

Combine raw GVCFs with the Hail VCF combiner (driven by this script);
Follow these steps in sample_qc.py from gnomad_qc (which are mimicked in this script in our version of the pipeline):
a) Split multiallelics with mt = hl.experimental.sparse_split_multi(mt, filter_changed_loci=True),
b) Filter out the reference blocks,
c) Filter out variants in telomeres and centromeres,
d) Compute hail's sample_qc() on the bi-allelic variants (those that were assigned was_split in the step 2a,
e) Apply a hard-filter to the n_snps value in the sample_qc() bi-allelic results.

Happy to prepare an executable script to reproduce on a dataproc.
I also noticed that subsetting the variants to the NA12878 high confidence regions brings the number of SNPs from ~5.2m to ~3.6m. Is there any chance that I should set filter_lcr=True in filter_low_conf_regions?
Vlad

Hey Vlad, we have looked this over and aside from finding a bug introduced in our code clean-up after the v3.1 release (Fixed in this PR #208, but didn't impact v3 or v3.1 numbers as described in the PR comment) we don't see any differences in the pipeline. We did not use filter_lcr=True in either v3 or v3.1.
Since we have both run this on the HGDP samples it might be good to compare our number on those samples directly. These numbers for our run are on this MT: gs://gcp-public-data--gnomad/release/3.1/mt/genomes/gnomad.genomes.v3.1.hgdp_1kg_subset_dense.mt MT under sample_qc.n_snp

Thanks for a reply!!
You mean the fix in #209? Makes sense!
With regards to the number of SNPs, it looks like it has to do with the fact that we deal with a much smaller dataset compared to gnomAD, and because of that the number of multi-allelic sites is smaller too. So the number of bi-allelic sites is close to the total number of sites for us, whereas in gnomAD bi-allelics are less abundant.

Yes the #209 fix doesn't impact your code, but we have put in the fix for future callsets.
Yes, we also determined that the bi-allelic n_snp cutoff will need to be callset specific. We will add a note to the gnomad_qc code to help future users.
Thank you for bringing this to our attention.

authors: jkgoodrich vladsaveliev


text:
[Bug]: Thumbnails Downloadgröße nur 27B

Adapter version

JS-Controller version

Node version

Operating System type
Linux
Operating system version
Ubuntu 22
Describe the bug
Wie angedroht :o) Thumbnail download von Doorbells funktioniert nicht mehr. Fehler im Log kommen keine aber wenn man dann auf die Festplatte schaut hat die Datei nur eine Größe von 27B. Komischerweise funktioniert das Ganze bei den Cams schon noch?! Entweder liegt es an den Doorbells oder daran, dass die über die 2. Homebase laufen.
To reproduce

Go to '...'
Click on '...'
Scroll down to '....'
See error

Screenshots & Logfiles
.
Additional context
.

Hilft das weiter? Teste jetzt gerade mit der neuen Dual Cam, weil die installierte 2K zu weit weg ist. momentan wirkt es aber so, wie wenn sie nicht mal Bewegung übermittelt. In der Eufy App taucht es aber auf.

eusec.0
2022-11-27 11:09:46.625
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.person_name changed: (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:46.570
debug
[euSec.onStateChange] state eusec.0.xxcameras.xx.person_detected changed: false (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:31.691
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.person_name changed: Unknown (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:31.631
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.picture_url changed: https://security-app-eu.eufylife.com/v1/s/g/4xmexMPzKxQ (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:31.631
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.person_detected changed: true (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:31.622
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.last_event_pic_html changed:  (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:31.622
debug
[euSec.onStateChange] state eusec.0.x.cameras.x.last_event_pic_url changed: /eusec.0/x/last_event/x.jpeg (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:31.477
debug
[EufySecurity.onPushMessage] Received push message [{"name":"Doorbell","event_time":1669543770565,"type":91,"station_sn":"x","device_sn":"x","title":"Eufy Security","content":"Doorbell:hunman has been spotted ","push_time":1669543771410,"channel":1,"cipher":147,"event_session":"20221127_110930","event_type":3102,"file_path":"/media/mmcblk0p1/Camera01/h265_20221127110919.dat","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/4mexxMPzKxQ","push_count":1,"notification_style":3}]

eusec.0
2022-11-27 11:09:31.477
debug
[PushNotificationService.onMessage] Normalized push message received [{"name":"Doorbell","event_time":1669543770565,"type":91,"station_sn":"x","device_sn":"xx","title":"Eufy Security","content":"Doorbell:hunman has been spotted ","push_time":1669543771410,"channel":1,"cipher":147,"event_session":"20221127_110930","event_type":3102,"file_path":"/media/mmcblk0p1/Camera01/h265_20221127110919.dat","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/4mexMPzKQ","push_count":1,"notification_style":3}]

eusec.0
2022-11-27 11:09:31.477
debug
[PushNotificationService.onMessage] Raw push message received [{"id":"BF55EC36","from":"348804314802","to":"dUsK_9mbPFfdjCU581TAi-","category":"com.oceanwing.battery.cam","persistentId":"0:1669543771425221%0d2a775cf9fd7ecd","ttl":3600,"sent":"1669543771419","payload":{"device_sn":"xx","payload":{"event_type":3102,"device_sn":"xx","name":"Doorbell","channel":1,"cipher":147,"session_id":"20221127_110930","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/4xmexMxPzKQ","create_time":1669543770565,"file_path":"h265_20221127110919","notification_style":3,"push_count":1,"faceid":0},"station_sn":"T8010P2322161489","google.c.sender.id":"348804314802","title":"Eufy Security","type":"91","push_time":"1669543771410","event_time":"1669543770565","content":"Doorbell:hunman has been spotted "}}]

eusec.0
2022-11-27 11:09:31.477
debug
[PushClient.handleParsedMessage] DataMessageStanza [{"tag":8,"object":{"id":"BF55EC36","from":"348804314802","to":"dUsK_9mbPFfdjCU581TAi-","category":"com.oceanwing.battery.cam","appData":[{"key":"device_sn","value":"T8213P21223805CA"},{"key":"payload","value":"eyJldmVudF90eXBlIjozMTAyLCJkZXZxpY2Vfc24iOiJUODIxM1AyMTIyMzgwNUNBIiwibmFtZSI6IkRvb3JiZWxsIiwiY2hhbm5lbCI6MSwiY2lwaGVyIjoxNDcsInNlc3Npb25xfaWQiOiIyMDIyMTEyN18xMTA5MzAiLCJwaWNfdXJsIjoiaHR0cHM6Ly9zZWN1cml0eSx1hcHAtZXUuZXVmeWxpZmUuY29tL3YxL3MvZy80bWV4TVB6S1EiLCJjcmVhdGVfdGltZSI6MTY2OTU0Mzc3MDU2NSwiZmlsZV9wYXRoIjoiaDI2NV8yMDIyMTEyNzExMDkxOSIsIm5vdGlmaWNhdGlvbl9zdHlsZSI6MywicHVzaF9jb3VudCI6MSwiZmFjZWlkIjowfQ=="},{"key":"station_sn","value":"T8010P2322161489"},{"key":"google.c.sender.id","value":"348804314802"},{"key":"title","value":"Eufy Security"},{"key":"type","value":"91"},{"key":"push_time","value":"1669543771410"},{"key":"event_time","value":"1669543770565"},{"key":"content","value":"Doorbell:hunman has been spotted "}],"persistentId":"0:1669543771425221%0d2a775cf9fd7ecd","ttl":3600,"sent":"1669543771419"}}]

eusec.0
2022-11-27 11:09:21.454
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.last_event_pic_html changed:  (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:21.409
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.last_event_pic_url changed: /eusec.0/xx/last_event/xx.jpeg (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:21.339
debug
[euSec.onStateChange] state eusec.0.xx.cameras.xx.picture_url changed: https://security-app-eu.eufylife.com/v1/s/g/3BxuYalq6xa (ack = true) was already acknowledged, ignore it...

eusec.0
2022-11-27 11:09:21.286
debug
[BatteryDoorbellCamera.processPushNotification] Unhandled doorbell push event [{"name":"Doorbell","event_time":1669543760394,"type":91,"station_sn":"xx","device_sn":"xx","title":"Eufy Security","content":"Doorbell:Something triggered your device","push_time":1669543761206,"channel":1,"cipher":147,"event_session":"20221127_110920","event_type":3308,"file_path":"/media/mmcblk0p1/Camera01/h265_20221127110919.dat","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/3BxfuYalxq6a","push_count":1,"notification_style":3}]

eusec.0
2022-11-27 11:09:21.286
debug
[EufySecurity.onPushMessage] Received push message [{"name":"Doorbell","event_time":1669543760394,"type":91,"station_sn":"xx","device_sn":"xx","title":"Eufy Security","content":"Doorbell:Something triggered your device","push_time":1669543761206,"channel":1,"cipher":147,"event_session":"20221127_110920","event_type":3308,"file_path":"/media/mmcblk0p1/Camera01/h265_20221127110919.dat","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/3BfguYvalqd6a","push_count":1,"notification_style":3}]

eusec.0
2022-11-27 11:09:21.286
debug
[PushNotificationService.onMessage] Normalized push message received [{"name":"Doorbell","event_time":1669543760394,"type":91,"station_sn":"T8010P2322161489","device_sn":"T8213P21223805CA","title":"Eufy Security","content":"Doorbell:Something triggered your device","push_time":1669543761206,"channel":1,"cipher":147,"event_session":"20221127_110920","event_type":3308,"file_path":"/media/mmcblk0p1/Camera01/h265_20221127110919.dat","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/3BuYafglq6xa","push_count":1,"notification_style":3}]

eusec.0
2022-11-27 11:09:21.285
debug
[PushNotificationService.onMessage] Raw push message received [{"id":"BF558AFE","from":"348804314802","to":"dUsK_9mbPFfdjCU581TAi-","category":"com.oceanwing.battery.cam","persistentId":"0:1669543761229786%0d2a775cf9fd7ecd","ttl":3600,"sent":"1669543761222","payload":{"device_sn":"xx","payload":{"event_type":3308,"device_sn":"xx","name":"Doorbell","channel":1,"cipher":147,"session_id":"20221127_110920","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/3BuvYaldxq6a","create_time":1669543760394,"file_path":"h265_20221127110919","notification_style":3,"push_count":1,"faceid":0},"station_sn":"xx","google.c.sender.id":"348804314802","type":"91","title":"Eufy Security","push_time":"1669543761206","content":"Doorbell:Something triggered your device","event_time":"1669543760394"}}]

eusec.0
2022-11-27 11:09:21.285
debug
[PushClient.handleParsedMessage] DataMessageStanza [{"tag":8,"object":{"id":"BF558AFE","from":"348804314802","to":"dUsK_9mbPFfdjCU581TAi-","category":"com.oceanwing.battery.cam","appData":[{"key":"device_sn","value":"T8213P21223805CA"},{"key":"payload","value":"eyJldmVudF90eXBlIjozMzA4LCJkZXZpY2Vfc24iOiJUODIxM1AyMTIyMzgwNUNBIiwibmFtZSI6IkRvb3JiZWxsIiwiY2hhbm5lbCI6MSwiY2lwaGVyIjoxNDcsInNlc3Npb25faWQiOiIyMDIyMTEyN18xMTA5MjAiLCJwaWNfdXJsIjoiaHR0cHM6Ly9zZWN1cml0eS1hcHAtZXUuZXVmeWxxpZmUuY29tL3YxL3MvZy8zQnVxZYWxxNmEiLCJjcmVhdGVfdGltZSI6MTY2OTU0Mzc2MDM5NCwiZmlsZV9wYXRoIjoiaDI2NV8yMDIyMTEyNzExMDkxOSIsIm5vdGlmaWNhdGlvbl9zdHlsZSI6MywicHVzaF9jb3VudCI6MSwiZmFjZWlkIjowfQ=="},{"key":"station_sn","value":"T8010P2322161489"},{"key":"google.c.sender.id","value":"348804314802"},{"key":"type","value":"91"},{"key":"title","value":"Eufy Security"},{"key":"push_time","value":"1669543761206"},{"key":"content","value":"Doorbell:Something triggered your device"},{"key":"event_time","value":"1669543760394"}],"persistentId":"0:1669543761229786%0d2a775cf9fd7ecd","ttl":3600,"sent":"1669543761222"}}]

The download of notification images has been temporarily removed as decryption of these is not yet supported. It will be added back as soon as it is supported.
Due to changes from Eufy it is currently not possible to get the thumbnails. More about this https://github.com/bropat/eufy-security-client/issues/273.


text:
Any chance to make flycheck dependency optional?

The users may choose flycheck or flymake, like eglot or lsp-mode. So it's good idea to drop flycheck dependency, IMO.

I would advocate that. I personally avoid flycheck with a passion and now I spent some time to figure out where it comes from (bisecting my package requries). Please make it optional or even better factor it out into a separate package.

repo: brotzeit/rustic


text:
fix redis cmd_format bug in the empty string case

fix issue #282

把!compbuf.empty()这个判定去掉就行了？

不可以；
当redis cmd的参数之间是有多个空格分隔 或者 整条cmd末尾有空格时；去掉!compbuf.empty()判定会导致解析出多个空参数，整个命令将是错误的.
请教戈神，我看travis-ci环境中有单测失败了（ChannelTest.close_fd_parallel），失败的原因是broken pipe了，这个在我测试环境尝试复现，发现是没有问题，但是通过日志对比看，是不太一样的；请问这种情况改怎样定位呢？  有办法让travis-ci环境重新构建复现问题吗？

好，了解了。那个ut可能是偶尔超时，暂时先不用管。redis ut里需要加个case，测试get ''key value和get key'' value这种形式（紧贴着key的''无作用也不增加component)，get 'ext'key value和`get key'ext' value“也可以顺便测下（等价于extkey和keyext)

repo: brpc/brpc


text:
faster gui

The demo application on my computer: (<!> the benchmark is not very scientific <!>)

/
before
after

debug
30 fps
175 fps

fast
81 fps
1351 fps

Codecov Report

Merging #117 (c1e923c) into main (06877bb) will decrease coverage by 0.84%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
- Coverage   63.81%   62.97%   -0.85%
==========================================
  Files         126      126
  Lines        5541     5615      +74
==========================================
  Hits         3536     3536
- Misses       2005     2079      +74

Impacted Files
Coverage Δ

sources/libs/brutal/gfx/buf.h
0.00% <0.00%> (ø)

sources/libs/brutal/gfx/paint.h
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06877bb...c1e923c. Read the comment docs.

repo: brutal-org/brutal


text:
Expose http.ServeMux to allow applications to add additional endpoints

It would be useful to expose the http.ServeMux so an application could add an additional listener without needing to listen on a new port. In my case I want to expose a Prometheus metric endpoint, currently I need to listen on an additional point but I don't think there is any reason this should be necessary.

I've refactored hc and made a new library out of it. The new library is available as hap.
The new hap server gives you access to the ServeMux.

authors: benpye brutella

repo: brutella/hc


text:
Feature: More file link options (e.g. copy file or line link)

First of all thanks for the plugin. The GitLab issues part is already covered by two other IDEA plugins. It's good that you focus on the repository browsing.
It would be nice to be able to simply copy file links with a line anchor instead of opening them in the browser. This is for faster pasting into issue trackers. I guess one would need a sub-menu in the context menu then. Maybe like this:

GitLab -> Open file in Brower
       -> Jump to line in Browser # (that's currently there)
       -> Copy file URL
       -> Copy line URL

Alternatively for a solution not going the sub-menu route, it would suffice to have "Copy GitLab URL" in the context menu as a second entry provided by the plugin.

@bogn sorry, somehow my previous GitHub configuration didn't notify me about new issues here, so I was unaware of this one, as well as others. My bad!
I see your use case, it makes sense, with just "Copy GitLab URL to the line" functionality. I will add it once I have a time.
Thanks!

authors: bogn bsideup

repo: bsideup/IDEA-GitLab-Integration


text:
Kerberos stopped working to ipa-server-4.9.12-11

After our Identity and Access Management group updated their IPA server to ipa-server-4.9.12-11.module+el8.9.0+20824+f2605038.x86_64  our conservers stopped working with Kerberos authentication.
The OS system still works when using ssh as in 'ssh -AK $FQDN' so the system keytab and kerberos setup is working. However we can only connect via console if kerberos is disabled via 'KRB5_TRACE=/dev/stdout console $ARGS' and supplying the user password each time.
At present we are running:
rpm -qa | grep conserver
conserver-debuginfo-8.2.1-4.3.el7.x86_64
conserver-client-8.2.1-4.3.el7.x86_64
conserver-8.2.1-4.3.el7.x86_64
The server is running:
cat /etc/redhat-release
Red Hat Enterprise Linux Server release 7.9 (Maipo)
Is this something that can be looked at and possible fixed on this end ?
Thanks in Advance.

I should note tracing on the server only shows:
`[root@console ~]# KRB5_TRACE=/dev/stdout /usr/sbin/conserver
[Wed Feb 14 21:40:42 2024] conserver (153644): conserver.com version 8.2.1
[Wed Feb 14 21:40:42 2024] conserver (153644): started as root' by root'
[Wed Feb 14 21:41:24 2024] conserver (153644): ERROR: GSSAPI didn't work, An invalid status code was supplied`

repo: bstansell/conserver


text:
Trim GitHub permissions?

This plugin looks really useful.  Thanks for your work on it!
However, the GitHub permissions it asks for after clicking Install seemed a bit, well, permissive 😄.  For example, it wants read/write for user profile data.  Are all of the permissions the plugin asks for required, or can they be trimmed down a bit?
The manual instructions to get a token make it sound like the actual permissions it needs are less.
For now, I'll give it a go with the manual install.

I completely agree.
At the time this plugin was originally written, GitHub Apps - which offer more fine-grained permissions - weren't able to support such a project. I can't remember why exactly, but I think it was because they couldn't be installed on a user, only an organization or repository.
It's been a while since then, so it's worth taking another look. I don't have a ton of bandwidth to do this right now but will try to take a look over the holidays.

authors: briancavalier bswinnerton

repo: bswinnerton/launchbar-github


text:
Manually copy relevant fields in ntfn state copy.

This makes go vet happy by manually copying the notification state fields during the deep copy instead of copying the entire struct which contains an embedded mutex.

ok, although i'd prefer to see the returned copy not contain any mutex (it's immutable and doesn't need synchronization), and to move it elsewhere

That can be done in a separate PR.  I'd prefer to just fix the vet issue first, then change the semantics in a separate PR if that's what we want.

authors: davecgh jrick

repo: btcsuite/btcrpcclient

license: isc


text:
请求图片导致服务异常退出

http://127.0.0.1:4869/61d16a19c1c508a64f819710036e1c70?w=100&h=100
服务端日志
zimg: magick/semaphore.c:347: LockSemaphoreInfo: Assertion `semaphore_info != (SemaphoreInfo *) ((void *)0)' failed.
版本3.1.0

可能是我安装问题

repo: buaazp/zimg


text:
some fixes for input

add spellcheck false
do not set input value if it is the same to prevent jumping to end of input

thx

repo: buberdds/angular-bootstrap-colorpicker


text:
cyfs-stack support NO-SN online

The purpose of the feature
If My device have statice-wan address(IPv4 or IPv6),I can let the EndPoint info on-chain and donn't need SN anymore.
Describe the solutions to the feature you'd like
cyfs-stack support NO-SN online

Currently, for OODs with a fixed public IP address (such as vood), the following mechanisms are in place to ensure maximum connectivity:

When activating the OOD, the WAN address and port are added to the OOD's device and registered on the blockchain. This way, when the protocol stack connects to the target device, it will prioritize using the WAN address and port configured in the device.

The gateway configuration on the OOD does not need to be explicitly changed, meaning the SN stays online. However, when the protocol stack starts, SN online status no longer blocks the stack startup (e.g., if there's some wrong with the SN server, the protocol stack will not be unable to start due to SN.wait_online). When necessary, connections will be initiated through SN.

When connecting to the target device with a configured WAN address, the protocol stack will first try to establish a direct connection using the WAN address. If it fails, it will attempt to connect again through the SN (if the device remains online on SN) and acquire the latest device for future use.

If the protocol stack fails to establish a connection using the device, it will try to update the information from the metachain once (for both the device and its owner). This is mainly to handle the following two possible changes:

The device's WAN address has changed, such as an external IP address change, and other devices' local cache is outdated compared to the metachain.
The owner's OOD list has changed, for example, if the device is no longer an OOD for the target zone. In this case, after updating the owner's information, the corresponding zone information needs

Complete at 1c4a58886bad3ba65552ef41ac4748316d874a18

authors: lurenpluto waterflier

repo: buckyos/CYFS


text:
Release v28.0-rc1

SSIA

Yeah, for some reason, we need to create a PR for the tag to trigger a release. So I always create a PR but just close it after the release is done.

repo: bufbuild/protobuf-conformance


text:
Sharing validation rules in a project

Another way to think about this might be how to allow custom "well-known rules".
Primarily I'm interested in sharing regex patterns in a larger project without needing to take the route of upstreaming all well-known types.

For this use case, I don't have the flexibility of wrapping scalar types in a message that everyone can share.
It would be nice if all rules are shareable, not just regex rules.
I can Ideally, this mechanism could also be convenient enough to be used for defining the PGV well-known types (but not a hard requirement). This would allow PGV to ship well-known types decoupled from the core implementation (similar to how common google protobuf types are just a library of proto messages).

I know this gets at the heart of protobuf's lack of string constants (or constants of any kind).
Here's the sketch of one approach (that would require some changes to how validators work). I've thought this through for java. Generally, you can think of this approach as delegating field validation to another validator.

Provide a family of rules (e.g. (validate.rules).TYPE.like = "fully.qualified.Message.field_name" that takes a string. The string references a specific message field defined elsewhere in the project that has PGV rules applied to it.

In the generated java code we now have 2 generated validators one for the prototypical message and another for the message annotated with (validate.rules).TYPE.like. Assuming we generate individual methods for each field, the like rule validator can delegate checking.

Example:

package acme.common;

message Common {
  string id = 1 [
    (validate.rules).string.pattern = "^blah"
  ];
}

package acme.my_service_a;

message Request {
  string id = 1 [(validate.rules).string.like = "acme.common.Common.id"];
}

At this point, in the java generated code at least, we have CommonValidator and a RequestValidator. If we enhance the *Validator code generation to allow individual field validation methods, RequestValidator can validate id by calling CommonValidator.validateId (or whatever the per-field methods get called).
Performance considerations:

In languages like Java/C# these bodies should get inlined by the jit as soon as class hierarchy analysis is performed.
Inlining should also apply to languages like go (or whenever static link-type optimizations are performed).

Usability considerations:

The idea of a fully qualified message name is well defined by protobuf but I'm not so sure about a fully qualified field name. (validate.rules).string.like = "acme.common.Common.id" Could instead be expressed as like = {message: "acme.common.Common", field: "id"} to more closely match the MessageDescriptor + FieldDescriptor.
Optionally, the PGV generator can check that the referenced message is available on the protoc -I import path to ensure that both validators are generated. This is a usability optimization though since if the common validator isn't on the protoc path, the generated code will fail to compile when trying to reference the common validator.

We also would very much like something to improve the status quo for extending well-known types.
For example, we are using ksuid instead of UUIDs - currently we can put a regexp for every ksuid field, but it would be great to have an extension point for protoc-gen-validate to reduce repetition and inconsistencies in our validators.  We have a similar issue with our `"display name" and "metadata" fields we attach to many objects - its less about a consistent regexp and more the min/max bytes.
In the Go ecosystem, we can manually implement the Validate() interface if the field is an embedded message type, but many of our use cases are for scalar types.

Any update on this? This would be highly desirable. In our proto files we have repetitive scalars (validate.rules).string.pattern regex checks for safe chars (^[0-9a-zA-Z.\-/() $!*+,:;=?@[\]_~æøåÆØÅ½§@£$~ÀÁÂÃÄÇÈÉÊËÌÍÎÏÑÒÓÔÕÖÙÚÛÜÝßàáâãäçèéêëìíîïñòóôõöùúûüýÿ \n\r`#{}%&']+$"). Repeating this is ugly and error prone. Thanks!

Thank you for highlighting the importance of sharing validation rules for reusing custom validation rules within a project. This is an area of interest for us as we continue to enhance the capabilities of bufbuild/protovalidate#51.
Your input is greatly appreciated, and it aligns with our goals for future developments. As a result, I'll be closing this issue while keeping your suggestion in mind.
Should you have more suggestions or questions down the line, don't hesitate to get in touch. Your involvement is invaluable!

authors: elliotmjackson jaijiv pquerna smorel-plenty

repo: bufbuild/protoc-gen-validate


text:
Added split by regex functionality

Allows input to be parsed using a regex while the output still uses the delimiter. Very useful for cases when the input file has an indeterminate amount of white space as the column separator. Code is not elegant and should be refactored. Example file 'marks.txt' added to docs folder and example usage added to README.md

Supporting division using regular expressions is a great idea. We've received a similar issue #8 recently that addresses this concern. In that issue, there was a suggestion to enhance the existing -d option, allowing it to accept regular expressions. If the intent is to facilitate division, I believe it's better to extend the -d option rather than adding a new option. For instance, how about the following:

If the value of the -d option has two or more characters, treats it as a regular expression for division.
rec = pattern.split(line)

Adding a new option, -R REGEX, --regex REGEX, specifically for matching using regular expressions rather than division.
rec = pattern.findall(line)

I think it's a good practice to pre-compile the regular expression outside of the loop, like this:
pattern = re.compile(REGEX)
Usage examples:
$ echo "aaa    bbb ccc   ddd" | ppp rec -d '\s+' rec[0]
aaa

$ echo "aaa    bbb ccc   ddd" | ppp rec -R '\w+' rec[0]
aaa

Since '\s+' is a common choice for regular expression division, adding an option like -S, --spaces with the same meaning as -d '\s+' would be beneficial, in my opinion:
$ echo "aaa    bbb ccc   ddd" | ppp rec -S rec[0]
aaa

I see the value of precompiling regexes or any other refactoring or
improvements to the code as I just hacked something together quickly that
worked for me.
I also originally wanted to reuse the -d option, but the problem is the
output when you have more than one column in the output.
echo "aaa    bbb ccc   ddd" | ppp rec -d '\s+' rec[0:3']
aaa\s+bbb\s+ccc
Most use cases don't want to see a regex in the output, and it would
be nice to be able to choose what the delimiter in your output is.
Having said that, the -S, --spaces you suggest would probably handle
95% of the cases where people need to resort to regexes and keep
things simple.
On Wed, Oct 25, 2023 at 1:42 AM bugen @.***> wrote:

Supporting division using regular expressions is a great idea. We've
received a similar issue #8 https://github.com/bugen/pypipe/pull/8
recently that addresses this concern. In that issue, there was a suggestion
to enhance the existing -d option, allowing it to accept regular
expressions. If the intent is to facilitate division, I believe it's better
to extend the -d option rather than adding a new option. For instance, how
about the following:

If the value of the -d option has two or more characters, treats it as
a regular expression for division.
rec = pattern.split(line)
Adding a new option, -R REGEX, --regex REGEX, specifically for
matching using regular expressions rather than division.
rec = pattern.findall(line)
I think it's a good practice to pre-compile the regular expression outside
of the loop, like this:
pattern = re.compile(REGEX)
Usage examples:
$ echo "aaa    bbb ccc   ddd" | ppp rec -d '\s+' rec[0]
aaa
$ echo "aaa    bbb ccc   ddd" | ppp rec -R '\w+' rec[0]
aaa
Since '\s+' is a common choice for regular expression division, adding an
option like -S, --spaces with the same meaning as -d '\s+' would be
beneficial, in my opinion:
$ echo "aaa    bbb ccc   ddd" | ppp rec -S rec[0]
aaa
—
Reply to this email directly, view it on GitHub
https://github.com/bugen/pypipe/pull/3#issuecomment-1778694718, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWUUH4JERXXC2S4SDYZJ3YBC7HBAVCNFSM6AAAAAA6M2WPLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZYGY4TINZRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

authors: JamesSullivan bugen

repo: bugen/pypipe


text:
Case sensitivity

So TOML is case-sensitive but all configuration implementations ignore cases
e.g.
name = "Gina"
Name = "gina"

will result in "gina" overriding "Gina" because Tomlyn will treat them as separate keys but that's not the case with IConfiguration.

From https://docs.microsoft.com/en-us/aspnet/core/fundamentals/configuration/?view=aspnetcore-5.0#configuration-keys-and-values

Configuration keys:
Are case-insensitive. For example, ConnectionString and connectionstring are treated as equivalent keys.


text:
Update test fixtures to minimum supported version

Goal
Updates the test fixtures to the new minimum supported version (AGP 3.4.0/Gradle 5.1.1). This required the alteration of several feature files as the scenarios currently assume that requests are received in a specific order. This request order changes between AGP versions.
Changeset

Updated travis.yml to use recommended caching for Gradle, and to use latest versions of Android tools
Bump bugsnag gradle plugin AGP version for test fixtures from 3.1.0 to 3.4.0, Gradle from 4.4 to 5.1.1
Bump gradle wrapper used to build project to 6.5.1 (this is not the same as minimum supported version)
Commented out default_lib.feature for now until library support is added again (broken by changes made in AGP 3.3.0)
Skipped lint tasks when running scenarios to save execution time
Updated FakeVariantImpl to implement new methods in AGP 3.4.0

The first error is a warning from Codenarc, the static analysis tool. I believe it's missing a file required for one of the analysis modules but that this doesn't affect linting as a whole. I think the second error is a part of a mazerunner scenario where a request is made which shouldn't succeed, and we verify that zero requests are received.

repo: bugsnag/bugsnag-android-gradle-plugin


text:
Better error context for Sidekiq

This commit adds the class: https://github.com/bugsnag/bugsnag-ruby/commit/9d79c9f4698dcafbb28d5c1af139b97d8a6d058d
However, it's just ActiveJob::QueueAdapters::SidekiqAdapter::JobWrapper in Bugsnag (not very useful):

Instead I'd suggest using the wrapper value (which is shown in the Sidekiq tab). This will be something like MyQuickJob which is much more useful.

Hi @sandstrom, thanks for the feedback. That's a good idea. I was thinking that class name was something more useful, but perhaps the wrapper makes more sense.

@kattrali The class name never change, for all our errors it always says ActiveJob::QueueAdapters::SidekiqAdapter::JobWrapper instead of the ActiveJob class.

Just got a fix for this in #317 - will get a release out soon

@snmaynard Awesome! ⛵  When will you release a new version?

Planning on a release early next week - probably Monday or Tuesday. A few more things I want to try and get in to it.

@snmaynard much thanks

@snmaynard We've deployed 5.0 and still seeing this. Have I misunderstood something or is the issue still around?
(expected the context value, to the right of the bold error class to be the name of the job)

@snmaynard friendly ping 😄

#323 should be the fix for this - we should be using wrapped not wrapper for the active job class name

Closed via #323.

authors: kattrali sandstrom snmaynard

repo: bugsnag/bugsnag-ruby


text:
Make ockam identity delete (no args) interactive by asking the user to choose from a list of identity names to delete (tuify)

Current behavior
When a user runs ockam identity delete without any arguments, currently they are shown some help text. Here's a screenshot of this.

Desired behavior
Instead of displaying help text, change the behavior of this command so that it becomes interactive with the user. Note that this can only be allowed to happen if:

the terminal is interactive, and,
the --no-input or --quiet flags have not been passed, and,
no arguments have been passed in to the command.

This what the user flow should look like in interactive mode.

Use the tuify crate's select_from_list() function to ask the user to select from a list of identity names. They should be able to make multiple selections. If there are none, then simply display a message saying that none exist (not an error) and exit.

Once the user has made their selection(s), display a confirmation prompt letting the user know that that a destructive operation is about to take place with the identity names they have selected. Use select_from_list() to display this prompt. They should be able to make a single selection.

If they have made no selections then exit by displaying a message to the user letting them know they did not make any selection.

If they choose "YES", then delete each selected identity, and display a success / failure message for each identity that got deleted.

If an operation succeeds then use the ✅ emoji as a prefix for the message.
And if it fails, then use the ⚠ emoji as a prefix for the message.

The following is a video showing a generic user flow that shows the user experience when a user is asked to choose a from a list of "things". Then an action is performed on each "thing". Finally status information is reported for each action that is performed on each "thing". Please note that this flow is not specific to this issue.

https://asciinema.org/a/612547

Implementation details

Please use the functions select_multiple, and confirm_interactively in this file.

Here's a similar PR to get some ideas from: https://github.com/build-trust/ockam/pull/6480

You can learn more about how to use the select_from_list() in the tuify crate here. Here is an example that is very similar to the steps you might have to take for this issue.

Here's a file you can look at to detect when a terminal is interactive or not.

Here's a file you can look at to see where the ockam identity delete command is implemented.

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

I've taken https://github.com/build-trust/ockam/issues/6466 which is very similar to this (at least in the functionality). So I think it will be better if you can assign this to me too, so I can implement them in one PR and maintain on the constantly across the project.

@0x61nas Once you land the PR for https://github.com/build-trust/ockam/issues/6466 you can pick the next issue after that 👍🏽

May I have this too?

@Darioazzali That's awesome, this is all yours. Also, please checkout the tui.rs file that Adrian added to help you implement this: https://github.com/build-trust/ockam/pull/6884
Please let us know if you have any questions as you explore. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds

authors: 0x61nas Darioazzali nazmulidris

repo: build-trust/ockam


text:
Add support for memory backend

Pull in upstream swagger definitions for SnapshotLoadParams and MemoryBackend
Run go generate
Update Snapshot config to allow setting MemBackend
Forward the MemBackend through to the Firecracker client

Current dependencies on/for this PR:

main

PR #1   👈

PR #2

This comment was auto-generated by Graphite.

repo: buildbuddy-io/firecracker-go-sdk


text:
Threads stops before storing messages

Threads are stopping before MessageIOService can store messages to local db.

Bug fix is merged with v2/main

repo: buildersoftdev/andyxstorage


text:
some emoji missing or different set supported?

What is bk's relationship with https://github.com/buildkite/emojis?
In the example pipeline below, only :thumbsup: is supported by bk:
steps:
  - command: "sleep 2"
    label: ":thumbsup:"
  - command: "sleep 2"
    label: ":thumbsup::skin-tone-2:"
  - command: "sleep 2"
    label: ":+1:"
  - command: "sleep 2"
    label: ":+1::skin-tone-2:"

The output includes:
>>> Executing command step 👍
...
>>> Executing command step 👍:skin-tone-2:
...
>>> Executing command step :+1:
...
>>> Executing command step :+1::skin-tone-2:

Ah, I think I see what's happening. We're not taking into consideration the emoji variations.
The golang version is here: https://github.com/buildkite/docs/blob/master/app/models/emoji/catalogue.rb
Which just looks up the emoji in the list, but in the Ruby version, there's some pre-processing going on: https://github.com/buildkite/docs/blob/master/app/models/emoji/catalogue.rb#L34-L49
I don't have time to craft a fix, but a PR would be welcome! :D:D

Ok will investigate later.
I want my pipelines to have randomized skin color :)

👋 We don't currently support a local run command in the new CLI version (3.x.x). As you've seen, there are some complexities around having a local run command in the CLI and we're looking at other methods of providing this functionality.

authors: drnic keithpitt mcncl

repo: buildkite/cli


text:
Seems to work if -i is also passed

Hello @toolmantim,
Thank you for your write up. I was stumbling upon this very thing and it was driving me crazy during testing. Your testing helped me move on and realize it wasn't an error in my signal handling inside my script.
I happened to try with -i as well (interactive) and control-c (SIGINT) was successfully passed from docker to the PID 1 process in the container. Feel free to close this issue, but I just thought I'd let you know in case you wanted to test and update the documentation.

Thanks for letting me know. Just to be clear, all the existing info there is still valid? I haven't tested for the latest Docker and Docker Compose.

Good question. I have not validated all of the scenarios you outlined. If I end up doing that though I'll definitely let you know the results!

authors: jrwesolo toolmantim

repo: buildkite/docker-signal-test


text:
Bugfix: bugfix/812/BuildkiteAgentTokenParameterStorePath validation (#812)

Resolves #812
Ref: https://docs.aws.amazon.com/systems-manager/latest/userguide/sysman-paramstore-su-create.html

Thanks @jufemaiz. I had to tweak the AllowedPattern a little to allow blank strings as well (630f8335d8085ceb55c0b048dcbbc34824994f4c).

authors: jufemaiz yob

repo: buildkite/elastic-ci-stack-for-aws


text:
change executor property to be a list

@adityakavalur let me know your thoughts
currently executor selects how test is run, the executor property is a string which locks test to a single executor let's change it to a list so a single test can be run on multiple executors
version: "1.0"
buildspecs:
  multi_executor:
    type: script
    executor: [slurm.haswell, slurm.knl]
    sbatch: ["-N 1", "-t 30" ]
    run: hostname

This test will run on two executors slurm.haswell and slurm.knl. The script will be same for both executors just run on different queues. I foresee this may not work if we intermix different shells local.bash, local.csh, local.zsh, local.python. I think this should be used only with batch executors.
if we need custom values per executor for sbatch, batch, lsf, cobalt property we would need to change these properties. I don't know if other properties would be worth changing if user want custom behavior per executor
See https://buildtesters.github.io/buildtest/pages/devel/schemadocs/script-v1.html

This is changed to regular expression instead of list

repo: buildtesters/buildtest


text:
'Remove all builtin app' removes Windows Terminal and Files

They are not builtin apps at all.

built-in system, so this means all installed apps. besides microsoft store no exceptions are made

built-in system, so this means all installed apps. besides microsoft store no exceptions are made

Could it be more clear? I thought that only preinstalled apps will be removed. Thanks!

Could it be more clear? I thought that only preinstalled apps will be removed. Thanks!

authors: Belim linsui

repo: builtbybel/privatezilla