Removed retry from complete payment in cougar cash

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

app/src/main/java/edu/byu/suite/features/cougarCash/controller/PaymentReviewActivity.java, line 110 at r1 (raw file):

				} else {
					// We only want to retry the call one time, so we use a boolean "retried" to regulate this. If the call hasn't already been retried, retry the call. Otherwise display the error returned
					if (!retried) {

Remove the retried variable?

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Is the Alt-Enter keyboard shortcut for quickfixes intentionally missing on Windows?

I'm not sure if this is by design or not. I'm pretty sure it used to work, but I've had to add it to my local keymap to make it work now.

Nope, think that is an omission when changing the prefix on windows. It doesn't conflict with anything does it?

It appears to be unbound in the default keymap as far as I can tell. I've not had any problems since adding it back myself either... I'll make a PR :)


[Event Request] report 1307 "Standard Sales - Credit Memo" - OnHeaderOnAfterGetRecordOnAfterUpdateNoPrinted

🕮 Describe the request
Could you please add the following event and include the sender in this event?
trigger OnAfterGetRecord()
begin
    if not IsReportInPreviewMode() then
        CODEUNIT.Run(CODEUNIT::"Sales Cr. Memo-Printed", Header);
    //+EVENT
    OnHeaderOnAfterGetRecordOnAfterUpdateNoPrinted(IsReportInPreviewMode(), Header)
    //+EVENT

    CalcFields("Work Description");
    ShowWorkDescription := "Work Description".HasValue;
    CurrReport.Language := Language.GetLanguageIdOrDefault("Language Code");
    FormatAddr.SetLanguageCode("Language Code");

  [...]
end;

trigger OnPreDataItem()
begin
    FirstLineHasBeenOutput := false;
end;
}

[IntegrationEvent(true, false)] //Include the sender (that we can access our own global variables of the request page)
local procedure OnHeaderOnAfterGetRecordOnAfterUpdateNoPrinted(ReportInPreviewMode: Boolean; var SalesCrMemoHeader: Record "Sales Cr.Memo Header")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 11755.


Feature Request: hljs-callout

Unless there is a way to do this I'm missing, I'd like to suggest an hljs-callout general purpose class so that we have the ability to provide a special color along with italic, bold, or other passthrough style for a callout, a word or phrase within the highlighted code that needs to be called out to the reader, typically in the surrounding text.
hljs-callout wouldn't be a detectable class, but one provided by the author that would override the detected class by highlight.js (or be inside the detected class instead of the other way around).
Because, although it is true that we can provide a custom class/style inside the code, the color of that style is overridden by the detected syntax. See this fiddle for an example: https://jsfiddle.net/091cbza3/
Bold and italic are applied, but because highlight.js injects its styling for the token type inside my custom hljs-callout style (hljs-regexp and hljs-string in this case), the color of the callout is overridden by highlight.js.
I think this is related to #1379 but more specific

You need to use a child selector:
.hljs-callout * {font-weight:bold;font-style:italic;background:gold;}

Closing as asked and answered.  Let us know if you have any other questions! :-)


Initial prototype for Dimension preset system

This approach is very simple and I rather like it:

Entirely optional.
6 lines of code and could have been only 4 lines of code if I only wanted to support (name, unit) lookups
Works completely transparently with no changes to the code required anywhere else. Create a dimension and if there is a matching preset defined, the other dimension parameters are filled in.
Works from specific to general. If a preset with a (name, unit) match is found, that is used first. If a preset with only name defined is found, that is then used, allowing the unit to be filled in from the dimension name (useful if only one possible unit makes sense).
hv.Dimension.presets is nice an convenient for access/definition (and now has a nice repr!)

It doesn't allow sharing but I don't see an issue. If you aren't happy with a preset, make sure you define it the way you want it (e.g at the top of the notebook) before building your elements. You are free to change the preset definitions at any time.

Can you explain "it doesn't allow sharing"?

@philippjfr suggested that it would be useful in some way for all the elements to point to the same dimenion object (if it is a preset). This would allow you to change the range (for instance) for existing elements already using that preset.
I can see cases where that could be useful but I also think it is stateful and relies on mutation (and therefore may have unexpected results). I see dimension objects as being cheap metadata that every element is allowed to have its own copies of.

I can see cases where that could be useful but I also think it is stateful and relies on mutation (and therefore may have unexpected results). I see dimension objects as being cheap metadata that every element is allowed to have its own copies of.

In practice dimensions are shared across many objects all the time anyway. I can see how a globally shared object that automatically gets applied might lead to confusion though. I think we really want an easier way of overriding a particular dimension on a Dataset without also having to supply all the other dimensions. Then you can just supply the specific dimension you want to override to the Dataset and it will automatically be shared among all the individual Elements when you apply a groupby. That can wait though because we already support this pattern (and currently have no need to specify key dimension ranges, which would get far more verbose):
temp = hv.Dimension('Temperature', range=(0, 100))
dataset = hv.Dataset(data, vdims=[temp])
images = data.to.image(['longitude', 'latitude'])
temp.range = (0, 50)

@philippjfr I think this PR is ready to be merged now. Tests are passing so please review it and merge if you are happy with it.

Looks fine to me apart from a small indentation issue I've commented on. Would also be nice to see one or two unit tests since they should be pretty straightforward to write but that won't stop me from merging if you don't have the time right now.

Please merge it. I'll make a local branch right away to add unit tests...that way I won't forget to add them later!

Thanks for the fix. Merging.


Laravel 11.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:

Checkout the l11-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: The latest version of Laravel has a dependency for phpunit/phpunit of ^10.5. If you would like to also upgrade your tests, you may run the PHPUnit 10 Shift for free.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 11. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-remote.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "spatie/laravel-remote": "dev-l11-compatibility",
    }
}

Finally, run: composer update

Thanks!


BP NTRs: 'effector-mediated modulation of host DNA synthesis by symbiont' and child term 'effector-mediated induction of host DNA synthesis'

Please provide as much information as you can:

Suggested term label:
'effector-mediated modulation of host DNA synthesis by symbiont'
and child term
'effector-mediated induction of host DNA synthesis'

Definition (free text)
NTR: effector-mediated modulation of host DNA synthesis by symbiont
A process mediated by a molecule secreted by a symbiont that results in the modulation (either activation or suppression) of DNA synthesis involved in DNA replication (GO:0090592). The host is defined as the larger of the organisms involved in a symbiotic interaction. PMID:25888589

AND child term NTR: effector-mediated induction of host DNA synthesis
A process mediated by a molecule secreted by a symbiont that results in the activation of DNA synthesis involved in DNA replication (GO:0090592). The host is defined as the larger of the organisms involved in a symbiotic interaction. PMID:25888589

Reference, in format PMID:#######
PMID:25888589

Gene product name and ID to be annotated to this term

UMAG_02239 See1 from Ustilago maydis

Parent term(s)
GO:0140418 effector-mediated modulation of host process by symbiont

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

Any other information
New GO BP terms required for publication curation into PHI-base using our new community curation tool PHI-Canto in collaboration with @ValWood.

I also noticed a typo in GO:0140418 definition should be 'suppression' not 'suppresion' and also a typo in GO:0140590 'supression' should be 'suppression'. Thanks, Alayne.

@Antonialock
DNA synthesis seems to be the readout here - (GO:0071897 DNA biosynthetic process is 'The biosynthetic process resulting in the formation of DNA.' ) Both the paper you mention, and PMID:29314018 mention that the See1 effector "interacts with a maize homolog of SGT1 (Suppressor of G2 allele of skp1), a factor acting in cell cycle progression in yeast" (PMID:25888589) and "U. maydis See1 was identified as the first cell type-specific fungal effector, being required for induction of cell cycle reactivation in bundle sheath cells." (PMID:29314018)
So I propose to label the new term: "effector-mediated induction of cell cycle reactivation in host"
This would be part_of "GO:0051819 induction by symbiont of tumor or growth in host"
I dont think we need the general term 'effector-mediated modulation of host DNA synthesis by symbiont', until we have examples of inhibition.
Does that work for (and Alayne) ?
Thanks, Pascale

Apologies, my fault I checked, but I didn't check the paper, only the ontology positioning. That sounds correct.

Sound good thanks.
Is it possible for the new term "effector-mediated induction of cell cycle reactivation in host" to also have the parent "GO:0140418 effector-mediated modulation of host process by symbiont"?

The reason for this is that in our new PHI-base 5 website we capture high level information about whether the pathogen gene functions as an 'effector'. This information is then provided as a high level term label. The logic to do this involves looking for GO BP terms that are children of "GO:0140418 effector-mediated modulation of host process by symbiont".

+[Term]
+id: GO:0141017
+name: effector-mediated induction of cell cycle reactivation in host
+namespace: biological_process
+def: "A symbiont process in which a molecule secreted by the symbiont reactivates the host cell cycle, resulting in DNA synthesis and host cell division, and contributing to vegetative tumor formation." [PMID:25888589]
+is_a: GO:0140418 ! effector-mediated modulation of host process by symbiont
+relationship: part_of GO:0051819 ! induction by symbiont of tumor or growth in host
+property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/24370" xsd:anyURI
+created_by: pg
+creation_date: 2023-04-04T08:41:31Z


[Bug] on python: BOCPD configured for BetaBernoulli fails in .step() function, 'x' object cannot be converted to 'PyBool'

In the python library, initializing BOCPD using a beta bernoulli prior:
cpd = chp.Bocpd(
    prior=chp.BetaBernoulli(),
    lam=30)

with some bernoulli sequence data
bers1 = np.random.binomial(1, 0.1, size=10000)
bers2 = np.random.binomial(1, 0.2, size=10000)
bers3 = np.random.binomial(1, 0.3, size=10000)
bers4 = np.random.binomial(1, 0.1, size=10000)
data = np.concatenate([bers1,bers2,bers3,bers4])

yields error when trying to update with the step function:

The same happens when trying to cast to bool directly in python:

Hey @HilaManor, thanks for reporting this.
I have a fix. I'll create a PR quickly here, but if you need a faster fix, you can change cpd.step(x) to cpd.step(bool(x)). The patch I'll release soon will fix the issue as well.

Thank you, the quick fix worked as well :)


Script provider is asking me to supply OPEN_AI_KEY

Getting this error when I run the evaluation:
[FAIL] Error: OpenAI API key is not set. Set OPENAI_API_KEY environment variable or pass it as an argument to the constructor. Error: OpenAI API key is not set. Set OPENAI_API_KEY environment variable or pass it as an argument to the constructor.
--

https://www.promptfoo.dev/docs/configuration/testing-llm-chains#using-a-script-provider
Following the guide above, I set up a custom script provider:
prompts:
  - tests/prompt/prompt1.txt
providers:
  - exec:poetry run python tests/prompt/ai_client_provider.py
tests:
  - vars:
      question: What's the weather in New York?
    assert:
      - type: llm-rubric
        value: does not claim to know the current weather in New York

How the ai_client_provider.py looks like:
import sys
from app.clients.ai_client import AIClient
from app.services.intent_service import IntentService

conversation_id = "dummy"
admin_id = "dummy"
ai_client = AIClient()
intent = IntentService(conversation_id, admin_id, ai_client.vectorstores)

chat_history = []
question = sys.argv[1]
response_text = ai_client.get_response(question, chat_history, intent)

print(response_text)

AIClient internally would load its own azure openai keys and secrets. Not sure why I am getting the errors from here when I've already specified a custom script.
My config does not appear to be going down the script completion path.
promptfoo --version
=> 0.20.1

Hi @geekyme-fsmk,
The llm-rubric assert type uses GPT-4 by default, which is why it's asking for the OpenAI API key.  You can override the LLM grader using one of these methods: https://promptfoo.dev/docs/configuration/expected-outputs#overriding-the-llm-grader
Note that the grader uses a chat-completion style prompt by default.  If you need to override it or modify the prompt, you can set the rubricPrompt property on the test or assertion: https://promptfoo.dev/docs/configuration/expected-outputs#assertion-properties
Hope this helps solve your issue!

hi @typpo , understood - is it possible then to use my Azure Open AI key instead? I do have a subscription already with GPT4 enabled.

Alright I ended up doing this:
defaultTest:
  options:
    provider: azureopenai:chat:gpt-35-turbo

Then specifying my env vars accordingly:
AZURE_OPENAI_API_HOST=<azure_base> AZURE_OPENAI_API_KEY=<api_key> promptfoo eval -c promptfooconfig.yaml


/speechbubble only accepts .jpg files

Describe the bug
Trying to give the bot anything that isn't a .jpg image through the /speechbubble command will result in two errors.
When directly copypasting:
Message: Error: vips_image_get: field "exif-data" not found
vips_image_get: field "exif-data" not found
vips_image_get: field "exif-data" not found

    at ImageConnection.onMessage (file:///home/esmBot/esmBot/utils/imageConnection.js:66:22)
    at WebSocket.<anonymous> (file:///home/esmBot/esmBot/utils/imageConnection.js:89:45)

When uploading a standalone .png file (either through drag'n'dropping or by selecting a file through the file explorer)
Message: Error: vips_image_get: field "exif-data" not found

    at ImageConnection.onMessage (file:///home/esmBot/esmBot/utils/imageConnection.js:66:22)
    at WebSocket.<anonymous> (file:///home/esmBot/esmBot/utils/imageConnection.js:89:45)
    at WebSocket.emit (node:events:513:28)
    at Receiver.receiverOnMessage (/home/esmBot/esmBot/node_modules/.pnpm/<redacted>@8.13.0/node_modules/<redacted>/lib/websocket.js:1184:20)

Steps to reproduce

Upload a .png file when using the /speechbubble command (may happen with other commands)
Press enter
See the error message occur

Expected behavior
The /speechbubble command is supposed to take any file format and not just limit itself to .jpg.
Screenshots
No response
Self-hosted instance?
No
Additional context
No response

My fault, for some reason there were two different versions of libvips on the image servers and it was creating a conflict. Should be fixed now.


connect ECONNREFUSED <IP_ADDRESS>:80 - Error: connect ECONNREFUSED <IP_ADDRESS>:80

I am using docker compose and whenever i start session i get error webhook called i tried to change the machines updating the

if your webhook is directly to your host.
use the external IP address instead of the localhost address and check your firewall rules.


Loading default hash?

I have tried to implement the loading default hash:
$.ui.loadDefaultHash=true;
And I know it supposed to on by default, but no matter what url is entered it resets it to top level, once Afui is loaded?
Is there and example set up of this feature?

@surfjedi  - It could be a bug...I'll look into it soon.

I just tried with the kitchen sink and it seems to be working.
First, which version of App Framework are you using (looks like 2.2).
Second, any chance you can put up a demo?


Import Red Cross data via CSV

allows basic import with no error handling
I will work on a couple of needed unit tests

Using the sample file that we got from the Red Cross (offline) this new end point on the admin side of the site will allow us to import requests.
The data provided by the RC works, so I haven't yet implemented much by way of error handling.

@tonysurma I have created this as a PR now because it don't have any migrations, it is functional, we don't have the RTM passing yet and I am travelling for the next day or so. I'll do what I can to respond to any questions while in transit.  I have a few tests to add, which I'll try to do on my flight, but I'll submit as a separate PR.

There is one failing hit test but I think it is okay, going to merge and we can fix if it is an issue


FEAT: Allow interval option from environment variable

Starting our app in NextJS dev takes a bit longer to finishing compilation. Hitting the / route with wait-on every two seconds while compiling leads to the app constantly re-compiling and rarely finishing it's build before the timeout ends.
Adding an option for interval would be great, to allow us to check our route much less frequently.
In this PR I'm adding this option as an ENV variable, sticking to the current convention for TIMEOUT

Nice! Can you add a test and update the readme please?
Sent from my iPhone

On Aug 30, 2021, at 14:03, Chris Power @.***> wrote:

Starting our app in NextJS dev takes a bit longer to finishing compilation. Hitting the / route with wait-on every two seconds while compiling leads to the app constantly re-compiling and rarely finishing it's build before the timeout ends.
Adding an option for interval would be great, to allow us to check our route much less frequently.
You can view, comment on, or merge this pull request online at:
https://github.com/bahmutov/start-server-and-test/pull/315
Commit Summary
Allow interval option from environment variable
File Changes
M src/index.js (8)
Patch Links:
https://github.com/bahmutov/start-server-and-test/pull/315.patch
https://github.com/bahmutov/start-server-and-test/pull/315.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

sure! will do today. thanks!

@bahmutov added a demo for the ENV vars, lemme know if that works for you, thanks again!


chore: default to decide := true for simp

Future-proofing for leanprover/lean4#2722. A comment on SimpConfig is missing but I didn't exactly know what to write.

I'd prefer to match stock simp behaviour, i.e. keep decide := false by default. Reasons:

Less surprising behaviour; Aesop simp_all is really just simp_all.
aesop? would otherwise have to start generating simp_all (config := { decide := true }) invocations.

Does decide := false break mathlib a lot?

@JLimperg It's easy to fix it on the mathlib side, I just mistakenly thought decide := true would be preferred in aesop. The one remaining non-comment change in this PR is needed to fix this error when decide is not used by default:
./lake-packages/aesop/././Aesop/Search/Expansion/Norm.lean:380:60: error: unsolved goals
rs : RuleSet
normSimpContext : NormSimpContext
options : Options'
goal : MVarId
mvars : UnorderedArraySet MVarId
mvarsHashSet : HashSet MVarId := HashSet.ofArray (UnorderedArraySet.toArray mvars)
normSteps : Array NormStep :=
  #[NormStep.runPreSimpRules options mvars, NormStep.unfold rs options mvarsHashSet,
    NormStep.simp rs normSimpContext options mvarsHashSet, NormStep.runPostSimpRules options mvars]
⊢ 0 < Nat.succ (Nat.succ (Nat.succ 1))

Great, thanks! I'll merge when CI is green.


fix #1606 AiServicesAutoConfig is unable to detect AiService in the package specified by @ComponentScan

AiServicesAutoConfig is unable to detect AiService in the package specified by @ComponentScan
Changes Made:

Replaced the original use of Reflections with a combination of BeanDefinitionRegistryPostProcessor and ClassPathBeanDefinitionScanner to register interfaces.
Scanned all the packages specified by @ComponentScan and those obtained from AutoConfigurationPackages as basePackages.

Relevant Links:

https://github.com/langchain4j/langchain4j/issues/1606
https://github.com/langchain4j/langchain4j/issues/1593

Hi @qing-wq, thanks a lot!
Could you please add tests for the @ComponentScan case?

@langchain4j Thank you for your suggestion, I have modified and added unit testing


Added a test in Ruby

why? for the glory of satan of course

This test is buggy. See #346

@ProgVal btw how should I test in shell? #443

https://github.com/dominictarr/JSON.sh


UI bug - when hiding dst zone

seen with v5.5.6 while viewing nat rules, reload was necessary
blazor.server.js:1 [2022-01-03T18:05:55.470Z] Error: There was an unhandled exception on the current circuit, so this circuit will be terminated. For more details turn on detailed exceptions by setting 'DetailedErrors: true' in 'appSettings.Development.json' or set 'CircuitOptions.DetailedErrors'.
log @ blazor.server.js:1

only saw this once


Remove xml input

These input are not needed any more

Can one of the admins verify this patch?

Current coverage is 89.60%

Merging #124 into master will not affect coverage as of 7071bbf

@@            master   #124   diff @@
=====================================
  Files           66     66
  Stmts         4145   4145
  Branches         0      0
  Methods        276    276
=====================================
  Hit           3714   3714
  Partial          0      0
  Missed         431    431

Review entire Coverage Diff as of 7071bbf

Powered by Codecov. Updated on successful CI builds.


docs(aurora): fix permission scopes

I'm going to close this since CDF is going to fix the pre-configured scope in the UI.


[GCD] Understanding galaxy cluster research limitations

estimate 4
remaining 2020-09-10 4

[ ] Latest developments in galaxy cluster research: what are 5 important papers of the last 5 years in this field (talk with Reinout)
[ ] Obstacles for new research (not necessarily yours): Talk with Reinout and also Andre Botteon

Taking this down, but keep in your mind to continue


[FastBlur] remove un needed buffer and decrease allocation by 3.

Thanks to sh1 feedback on Disqus
https://disqus.com/home/discussion/trickyandroid/advanced_blurring_techniques_22

Nice !
lgtm :kimono:


So I am considering the following official "guide line": Add a way for platforms to "register" themselves to be tried to setup every 5 minutes if their config is not valid right now. It would still report the error but maybe only do that the first time ?
I think that a good way to implement this would be to introduce a new type of exception (PlatformNotReady ?) that platforms can raise. It can then be scheduled to try setup again in 5 minutes inside the entity component. This could easily be added here: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/helpers/entity_component.py#L140-L157

If a KNX gateway is not reachable (because connections are still used by crashed process for example) it would be nice if this heals itself by retrying without having to restart HASS another time.
So I would prefer to retry at intervals. Something like:
5sec, 10sec, 30sec, 1min, 2min, 5min, 15min, 1/2hour, 1hour, 2hours, 4hours

It would be good to let platforms with a constant connection to a service restart itself if the connection get lost. The pilight component for example seems to stop working as soon as the pilight daemon restarts (which I have to do now and then because the connected arduino seems to lose its sync to the daemon). The only way to fix this at the moment is to restart hass (and lose a lot of state).

Maybe we should retry on any Exception? Instead of logging not available as we do today, we just retry up to a limited amount of times
Backoff sounds good, but lets stick to a limit of say 3 tries. (E.g. 2min, 3min, 5min) and then stop. The platform might really be defective.
It might cause oodles of template errors for some users though
@cyberjunky my view is healing would require require some more work and not in scope of retrying setup. A "reloading" service could be built to restart KNX for example, but we dont have an internal way to monitor/trigger this today

@kellerza I think retrying platform init is more suitable for solving connection problems than for restarting platform after having real config errors (syntax etc)
I don't expect them to be solved in between retries. My KNX example is something I noticed when restarting HASS manually, if I detect this I have to restart HASS again manually to fix it, would be nice if HASS tried to reconnect/re-init platform a few times to solve this though. But I must admit that my knowledge of HASS's internals are limited, I only started to look at it a few weeks ago.  The loss of state mentioned by @janLo is also my greatest drwaby in restarting HASS completely, and also triggering automation upon start (have to build all kinds of checks to prevent this from happening)

@janLo restarting a service is not something we can easily do on a generic level, since you have to remove entities and rerun setup_platform and still you might leave behind some old state (take hub type components for example). That is why certain parts (automation, customize, groups) have special restart services.
@cybernunky it should be quite easy to add such a restart service for KNX, since it seems the knx "server/hub" part is misbehaving in some way by not closing connections.
I have a component electric imp that adds new devices whenever they appear on the server - a custom component, available in my repo, but maybe such an approach can work for KNX? (Dont own any KNX devices)

@kellerza I will have a look at your repo and see how to improve KNX connection lost behaviour, thanks.

@kellerza The problem is, that all the devices, that does not "report" a state and where the state is memorized in HA (like pilight switches) the actual state is gone after a restart.

@janLo maybe #4614 will help with such states (that is the idea at least)

I think that at least for now this should be limited to restarting after a failure inside of setup_platform. I think once a platform has successfully set itself up it needs to manage restart of connections on it's own, as @kellerza suggested.
I would propose considering an "opt-out" approach. We would modify setup_platform to return True on success, matching the component setup. (Requires changing all the platforms, but it should be relatively straight-forward.) Then, unless setup_platform throws a new type of exception PlatformConfigInvalid(?), it is scheduled for a retry if it does not return True.
My reasoning is that if the platform didn't setup successfully, there really isn't an harm in retrying, especially if we can squelch subsequent log errors. Also, I'm hard-pressed to come up with scenarios where the platform would really know for certain a retry won't work. Invalid file permissions could be fixed, network connections could be restored, cloud service credentials could be updated on the other end, etc. The biggest reason that a platform could never be setup correctly is an actual config schema violation, which is captured before setup is even attempted.

I agree with @kellerza and @armills about reconnecting. That should be handled by the component and it's platforms.
I like the idea of a back off mechanism and making it opt-out.
So if we get False back from setup platform, retry. If we get None back, it means the platform hasn't been updated, we might want to consider a warning with a link where people can report the platform (in case we missed some).

Still something we should probably add.

Truly necessary, especially with components like OctoPrint support where the polling of a printer that isn't on 24/7 creates 20-25k log entries in a single day.  My only option at the moment is to manually edit my configs and restart each time I want to use HA's notifications with my printer.


X10 switch

Description:
Switch support for Heyu connected X10 switch devices; this is a lightly modified copy of the existing X10 light device support.
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#1254
Example entry for configuration.yaml (if applicable):
switch:
  - platform: x10
    devices:
      - id: a2
      - id: a3
        name: Fan Switch

Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

This PR seems to have gone stale. Closing it. You can reopen it when you're ready to finish it.

@balloob Sorry, this turned out to be more complex than I thought in the beginning. I understand and agree with your comments. I think my approach is wrong, it looks like the pattern used by the mochad hub and device would be the correct way to handle this one and the existing X10 device. I will try to go that way and submit a new PR if it works out.
Thanks.


Rapidoid crashes when processing requests that use Transfer-Encoding.

Ok, so after an hour or two of smashing my face against this problem, it's a bit of a doozy...
I've written a shitty http client that can send Multipart POST requests to a server.
The server I was using to test against was Rapidoid, as it's insanely to do anything. (Props for that).
The library I'm writing uses the new Java 11 HttpClient, and as far as I can tell, the problem doesn't lie with that.
The problem:
ERROR | 05/Jan/2019 19:27:03:122 | server1 | org.rapidoid.net.impl.RapidoidWorker | Failed to process message! | error = java.lang.RuntimeException: Error in the response order control! Expected handle: 1, real: 0
java.lang.RuntimeException: Error in the response order control! Expected handle: 1, real: 0
	at org.rapidoid.u.U.rte(U.java:423)
	at org.rapidoid.u.U.rte(U.java:435)
	at org.rapidoid.net.impl.RapidoidConnection.processedSeq(RapidoidConnection.java:276)
	at org.rapidoid.net.impl.RapidoidWorker.processNext(RapidoidWorker.java:281)
	at org.rapidoid.net.impl.RapidoidWorker.processMsgs(RapidoidWorker.java:216)
	at org.rapidoid.net.impl.RapidoidWorker.process(RapidoidWorker.java:208)
	at org.rapidoid.net.impl.RapidoidWorker.readOP(RapidoidWorker.java:159)
	at org.rapidoid.net.impl.AbstractEventLoop.processKey(AbstractEventLoop.java:93)
	at org.rapidoid.net.impl.AbstractEventLoop.insideLoop(AbstractEventLoop.java:146)
	at org.rapidoid.net.impl.AbstractLoop.run(AbstractLoop.java:71)
	at org.rapidoid.net.impl.RapidoidWorkerThread.run(RapidoidWorkerThread.java:58)

An attempt to describe said problem:
After poking it for a bit, I've determined that the HttpManagedHandlerDecorator promotes the connection to async, when it shouldn't.
But it does so in an alternating fashion.
So, I chunk up the request, because, well, that's a good thing to do.
When the first chunk comes through, rapidoid processes it without issue.
As it was processing the first chunk, the connection seems to be set to sync, and it tells the connection
that the seq 0 was handled.
The problem arises when the next seq is handled.
For some reason, HttpManagedHandlerDecorator sets it to async:
Breakpoint reached
	  at org.rapidoid.net.impl.RapidoidConnection.async(RapidoidConnection.java:490)
	  at org.rapidoid.http.handler.HttpManagedHandlerDecorator.handle(HttpManagedHandlerDecorator.java:63)
	  at org.rapidoid.http.handler.AbstractDecoratingHttpHandler.handle(AbstractDecoratingHttpHandler.java:55)
	  at org.rapidoid.http.FastHttp.handleIfFound(FastHttp.java:285)
	  at org.rapidoid.http.FastHttp.onRequest(FastHttp.java:136)
	  at org.rapidoid.http.FastHttpProtocol.process(FastHttpProtocol.java:59)
	  at org.rapidoid.net.impl.RapidoidWorker.processNext(RapidoidWorker.java:271)
	  at org.rapidoid.net.impl.RapidoidWorker.processMsgs(RapidoidWorker.java:216)
	  at org.rapidoid.net.impl.RapidoidWorker.process(RapidoidWorker.java:208)
	  at org.rapidoid.net.impl.RapidoidWorker.readOP(RapidoidWorker.java:159)
	  at org.rapidoid.net.impl.AbstractEventLoop.processKey(AbstractEventLoop.java:93)
	  at org.rapidoid.net.impl.AbstractEventLoop.insideLoop(AbstractEventLoop.java:146)
	  at org.rapidoid.net.impl.AbstractLoop.run(AbstractLoop.java:71)
	  at org.rapidoid.net.impl.RapidoidWorkerThread.run(RapidoidWorkerThread.java:58)

So the connection is now marked as async, and the RapidoidWorker checks if the current request is async, if so, it doesn't care, and continues processing.
Meanwhile, it sets it back to sync, so when the second chunk comes in, the connection errors out, because
it was marked as sync, and as a result, tries to tell the connection that seq 2 was handled, but the
connection was never informed about seq 1, so it throws the error.
Beyond that, I have no idea.
If this wall of text helps in some way, awesome.
Otherwise I'm fine answering any questions you might have.
The client I've written isn't public yet, but I'd be happy to give you the code to take a look.

After returning to have another go at this issue, I've been able to stop the issue if I just send all of the data at once.
This means that there's actually two more things that could be doing something.

It could be Java's new HttpClient and their InputStream BodyPublisher. (I shouldn't have dismissed it immediately.)
It could be my code on top of that, because I map a bunch of multipart items to a single InputStream.
The resulting code should be correct, but I could have missed something.
It could be a problem within rapidoid.

After poking it more, I'm about 99% sure it doesn't come from my InputStream code.
I transformed the output into an array, and then compared that against the collected output of the chunked InputStream, and they both came out the same.
This leaves Java or Rapidoid.

Having poked it a lot more, I think I've determined the issue.
Java's Client uses "Transfer-Encoding" when using an InputStream and Rapidoid doesn't support that.
So, I guess after all of that, I should probably change this ticket to support Requests that use that.

https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Transfer-Encoding

So, yeah, I think the biggest thing right now is the fact I can bring down any rapidoid server just by sending a request that uses this...
Whether or not that's REALLY bad, or just bad, well, could be debated.


Is there any way to sort the http.Header by myself?

Hi, Guys
I wanted to create a new sorter for http.Header, how can I do that without forking the whole http package?
Thank you so much!

Hello, we try to keep the issue tracker only about actual issues with Go and items to work on. https://github.com/golang/go/wiki/Questions provides several ways to ask questions to the community.
That being said, http.Header is just a map[string][]string. You can manually flatten the map into [][]string and use sort.Slice to get what you want.


ZVIDEOS Elif Karaarslan Leaks Video On Social Media X Twitter[1S75K]

NOW ZVIDEOS Elif Karaarslan Leaks Video 2024
1 minute ago Elif Karaarslan, ZVIDEOS Elif Karaarslan Leaks Video On Social Media X Twitter[1S75K]

Elif Karaarslan's latest video has gone viral on social media, captivating audiences and igniting widespread discussions. This young digital creator has gained notable attention for her engaging content, particularly with her recent Spiderman-themed video that's taken the internet by storm. Viewers are eager to discover where to watch the full video, flocking to platforms like Instagram, Twitter (X), and Discord for access.
As the excitement surrounding Elif Karaarslan continues to build, fans are actively sharing their reactions online. The viral success of her video underscores the influence of social media in bringing creators into the limelight. With her distinctive style and creativity, Katherinne has quickly established herself as a prominent figure in the digital realm, leaving many curious about her upcoming projects and content.


Prioritization not working as intended

I recently updated to the newest version of DuplicateHider, and now a bunch of my games are using the wrong platform as the default.
I checked my priority rules, and according to the way I have it set up, the wrong copy is being shown in multiple instances. For example, I recently got Wolfenstein: The New Order from Epic Games. Even though Steam is prioritized higher than Epic in my settings for DuplicateHider, the copy from Epic is the one being shown. This is happening on multiple platforms/sources though, so it's not exclusive to games imported from Epic.
Here is what my settings look like:

And here are some examples of games not being properly priotizied:

I'm not sure what exactly is causing this issue, but I think it has to do with the recent addition of the ability to prioritize based on custom filters, with the Installation, Date Added, and Release Date fields being added by default (which I have removed, and the issue is still occurring).

I have also checked the Source field of the games being improperly displayed, and all of the sources are still set accordingly.

I too am having these issues.
The default priorities didn't seem to be working as they should; it seemed to be prioritising "Date Added" over "Source" despite "Source" being above "Date Added".
And after deleting all other priorities aside from "Source", it's still not working as intended; for some reason, regardless of what settings I use, Amazon is being prioritised over Itchio.

fixed via 008131bc3ced3c980f98256bef62c6fd1ff97192


Issue with arange

The following code
import numpy as np

#pythran export test(int)
def test(fac):
    print(np.arange(-(fac)*4,1,4))

does not return the expected array when called with fac = 1.
python -c "from newBug import test; test(1)"

Expected: [-4 0], returnd [-4]
This is on MacOS, python3, latest pythran.

Confirmed fixed


CoreDNS failure after dns-autoscaler.yml fails deployment

BUG REPORT:
Failing task, TASK [kubernetes-apps/ansible : Kubernetes Apps | Lay Down CoreDNS Template] with error:
failed: [node1] (item={'name': 'dns-autoscaler', 'file': 'dns-autoscaler.yml', 'type': 'deployment'}) => {"changed": false, "item": {"file": "dns-autoscaler.yml", "name": "dns-autoscaler", "type": "deployment"}, "msg": "AnsibleError: An unhandled exception occurred while templating '{{ 'true' if dns_min_replicas > '1' else 'false' }}'.
Error was a <class 'ansible.errors.AnsibleError'>, original message: Unexpected templating type error occurred on ({{ 'true' if dns_min_replicas > '1' else 'false' }}): '>' not supported between instances of 'int' and 'str'"}
NO MORE HOSTS LEFT ******************************************************************************************************************************************************************
	to retry, use: --limit @/Users/mstreeter/Projects/kubespray/kubespray-school-of-devops/cluster.retry

PLAY RECAP **************************************************************************************************************************************************************************
localhost                  : ok=1    changed=0    unreachable=0    failed=0
node1                      : ok=313  changed=12   unreachable=0    failed=1
node2                      : ok=273  changed=8    unreachable=0    failed=0
node3                      : ok=252  changed=7    unreachable=0    failed=0
node4                      : ok=218  changed=5    unreachable=0    failed=0

it looks like it's failing on line 72 of roles/kubernetes-apps/ansible/templates/dns-autoscaler.yml.j2 where it references a variable declared on line 13 of roles/kubernetes-apps/ansible/defaults/main.yml

Environment:

hardware configuration:
MacOSX running ansible to install k8s on 4 physical servers in my home with fresh install of Ubuntu Bionic (server non-gui version).

OS:
MacOSX: macOS High Sierra
Each Server: 18.04.1 LTS (Bionic Beaver)

Version of Ansible (ansible 2.6.6):

Kubespray version (commit) (1540bc97):
Network plugin used: calico
Copy of your inventory file:
[all]
node1    ansible_host=<IP_ADDRESS> ip=<IP_ADDRESS>
node2    ansible_host=<IP_ADDRESS> ip=<IP_ADDRESS>
node3    ansible_host=<IP_ADDRESS> ip=<IP_ADDRESS>
node4    ansible_host=<IP_ADDRESS> ip=<IP_ADDRESS>
[kube-master]
node1
node2
[etcd]
node1
node2
node3
[kube-node]
node1
node2
node3
node4
[k8s-cluster:children]
kube-master
kube-node
[calico-rr]
[vault]
node1
node2
node3
Command used to invoke ansible:
ansible-playbook -b -i inventory/prod/hosts.ini cluster.yml
Output of ansible run:

https://gist.github.com/marcstreeter/03d348d5dde15df244f2d82e9993e6e0
Anything else do we need to know:

I'm running the ansible script from osx within a python virtual environment  and controlling 4 different physical computers(each one is Ubuntu Bionic). I previously had installed kubernetes with kubespray on the same macbook (with the same python virtual environment) with the same computers (also on Bionic) without any error (using Kubespray previous commit 801bbcbc).  What I changed in this config was using coredns instead of the default kubedns(with the following k8s-cluster.yml).
Also it seems like the task only failed (or was only run) on one of the master nodes (there are two): node1.  I'd think it would fail elsewhere too.

Forgot to mention that, when running cluster-info on the master I got:

marcstreeter@k8s-master-10:~$ kubectl cluster-info
Kubernetes master is running at http://localhost:8080

where as I'd hope to see CoreDNS listed there, it is not.

Ok so I changed line 13 from roles/kubernetes-apps/ansible/defaults/main.yml from:
dns_prevent_single_point_failure: "{{ 'true' if dns_min_replicas > '1' else 'false' }}"

to:
dns_prevent_single_point_failure: "{{ 'true' if dns_min_replicas > 1 else 'false' }}"

Or, in other words, just removed the quotes around 1 and it works now.  It reports CoreDNS also as running in my cluster:
marcstreeter@k8s-master-10:~$ kubectl cluster-info
Kubernetes master is running at http://localhost:8080
coredns is running at http://localhost:8080/api/v1/namespaces/kube-system/services/coredns:dns/proxy
kubernetes-dashboard is running at http://localhost:8080/api/v1/namespaces/kube-system/services/https:kubernetes-dashboard:/proxy

it also reports the DashBoard as running which is a plus, as I intended to install it but not even that was listed in my previous cluster-info.

good to see people trying to understand issues not only reporting them, for future mention issue opened in PR for them to be linked :) as i already see PR with change here https://github.com/kubernetes-incubator/kubespray/pull/3728

lol -- ya I was just about to post the PR here, you're quick.


Issue #227 AUv2 release build fails

marking the variable with [[maybe_unused]] does fix it

Thanks!


Metadata on TemplatedMessage

Hi,
it's possible to set metadata in the TemplatedMessage version?
With Message I use
message.setMetadata(meta);
Thank's

Hi,
version 1.1.8 was just released which allows using metadata on TemplatedMessage.
Should appear shortly in Maven public repository.
Igor


Motion pdf export: Multiple formatting is ignored

Describe the bug:
The pdf export of motions does not work correctly if a text passage has more than one formatting options activated at the same time (e.g. cursive and bold). Only one formatting is exported.
A cursive and bold word would only be cursive or would only be bold.
How to Reproduce:

open meeting
open motion
create motion > mark text with bold and striketrough* > save
export motion as pdf

*This is an example. Several other combinations didn't work well. Please check them. The images below show as an example, what does work and what not. But these are not all possible options.
Expected behaviour
Every text formatting combination should be exported in the pdf export.
Additional context
Example Motion Text:

Example of this motions text as pdf export:

Used step-by-step description of the issue:

open meeting
open motion
create motion > mark text with bold and underline > save
export motion as pdf

So the "bold and underline" case.  Oh, it works correct.

Sorry, my mistake. I've described something which works correctly. Nevertheless the images shows several example of wrong behavior between motion view and export.
I've corrected the reproduction to an example which could also been seen in the images in line 6-7. The text is bold und striketrough in the motion view, but in the export it is only bold.

The strikethrough seems to be the format which always causes issues:
strik + bold                           => bold
strik + italic                          => nothiing
strik + font colour               => nothing
strik + background colour => background colour
bold + underline + strik     => bold
bold + italic + strik              => bold + italic
other formats which sometimes cause problems:
font colour
underline
italic

Mmh, one problem seems to be the 'html-to-pdf.service'. Transfer HTML string into a doc which the pdfMake part understands, can be tricky.

Tested with my PR: (single) bold, italic, underline, strike-through, text-color, background-color, bullet point and number are there. So these html tags and styles are not missing in html_to_pdf.

Tested mixed combinations. Passed.
@Elblinator If one combination is still wrong, please write it down.


Enable command-line export

The syntax is identical to what it was in 2.1, but now you specify
preset name instead of platform name.

I wonder if get_tree()->quit() should be called after export is finished? For me it's annoying that godot keeps working even though I launched it just to perform export. 2.1 code doesn't seem to quit after export (although I didn't try to launch it to check, perhaps quit is called from another place).

I think there used to be a flag to ask godot to quit after itś done
exporting.. though honestly it may make more sense to just quit after
export is done.
On Mon, Aug 14, 2017 at 10:32 AM, Ruslan Mustakov<EMAIL_ADDRESS>wrote:

I wonder if get_tree()->quit() should be called after export is finished?
For me it's annoying that godot keeps working even though I launched it
just to perform export. 2.1 code doesn't seem to quit after export
(although I didn't try to launch it to check, perhaps quit is called from
another place).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/10337#issuecomment-322190616,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2_aPnYxthjdB5Zy0lX3SCFvUDgssks5sYExRgaJpZM4O2Xgn
.

@endragor you can add auto quit in new pr


flight_number should be an attribute of the flight table, not of the aircraft table

I believe that flight_number should be an attribute of the flight table, not of the aircraft table.
I think it is an attribute of the aircraft in MSFS so that you have a default flight number, when you use an aircraft for Free Flight.   In real life I believe that when an airline has a last minute change of aircraft for a flight they keep the same flight number.
And it should be one of the field presented when I look at Logbook user interface,
In my case I have a flight training package that use the Savage Cub for all the Sport Pilot training flight. Each training flight has a different flight number.  For example when we flight from Post Mill to Lebanon to learn about airport communication and traffic pattern it is always "FLT-005" whether I use the Savage Cub or the Cessna 152.

Technically the flight number - like almost all vaues - comes from the aircraft, that is via the SimConnect API you query the "simulation variable" called ATC_FLIGHT_NUMBER, given the user aircraft ID (typically SIMCONNECT_OBJECT_ID_USER, to query the flight number from the current "user aircraft").
But you are right: it makes much more sense to

Present this value in the Logbook (instead of per aircraft, in the Formation module)
The presented value would simply be the flight number of the "user aircraft" (as defined for the formation flight)...
... and assuming that the flight number would apply for all aircraft in the formation equally

In that same argument I would remove it from the Formation aircraft table, as there the flight number is probably not really of interest anyway (again, technically it could be different for each aircraft, depending on the actual flight number as selected at the time of recording - but in reality we would expect the same flight number for all aircraft/the formation, just as you said).

Just to complement your answer, as a matter of fact, when I generate my
flights I put the flight number in the [Sim.0] entry:
[Sim.0]
Sim=Asobo Savage Cub
SimFile=runway.FLT
TailNumber=MSFS-RWT
AirlineCallSign=MSFS-RWT
FlightNumber=FLT-001
AppendHeavy=False
Pilot=Pilot_Male_Casual

and it is correctly captured by SkyDolly.
Gilbert Laprise
------ Original Message ------
From "Oliver Knoll" @.>
To "till213/SkyDolly" @.>
Cc "fsx-rwt" @.>; "Author" @.>
Date 2023-03-11 12:54:40 PM
Subject Re: [till213/SkyDolly] flight_number should be an attribute of
the flight table, not of the aircraft table (Issue #103)

Technically the flight number - like almost all vaues - comes from the
aircraft, that is via the SimConnect API you query the "simulation
variable" called ATC_FLIGHT_NUMBER
https://docs.flightsimulator.com/html/Programming_Tools/SimVars/Aircraft_SimVars/Aircraft_RadioNavigation_Variables.htm#ATC FLIGHT NUMBER,
given the user aircraft ID (typically SIMCONNECT_OBJECT_ID_USER, to
query the flight number from the current "user aircraft").
But you are right: it makes much more sense to
Present this value in the Logbook (instead of per aircraft, in the
Formation module)
The presented value would simply be the flight number of the "user
aircraft" (as defined for the formation flight)...
... and assuming that the flight number would apply for all aircraft in
the formation equally
In that same argument I would remove it from the Formation aircraft
table, as there the flight number is probably not really of interest
anyway (again, technically it could be different for each aircraft,
depending on the actual flight number as selected at the time of
recording - but in reality we would expect the same flight number for
all aircraft/the formation, just as you said).
—
Reply to this email directly, view it on GitHub
https://github.com/till213/SkyDolly/issues/103#issuecomment-1464964259,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSXAY7OHLOEWVOGRX6AME3W3S36BANCNFSM6AAAAAAVXDNEZY.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes, and I just realised that the value shown in the Formation module is actually the tail number (which is indeed aircraft-specific), not the flight number. So there is nothing to change in the Formation module.

I believe that flight_number should be an attribute of the flight table, not of the aircraft table.

Oh, or are you referring to the actual SQLite database tables (as opposed to the tables as presented in the user interface)? In this case you are right: the "flight_number" is indeed part of the aircraft table (and not the flight table). And yes, this is probably also something that I could change, but I have to think about the implications (the obvious implication is that the first recorded aircraft would specifiy the flight number - for subsequently recored "formation" aircraft the flight number would simply be ignored).
But your argument that the flight number conceptually really belongs to the flight (and not any aircraft) totally makes sense to me...

Yes, I was referring to the SQL table.  While I still think that it
should be presented to the user as part of the Flight, since MSFS assign
it to the aircraft in the FLT, you might want to keep it with the
Aircraft table.
However the value of this attribute will probably change  with different
flight with the same aircraft. For example for the sport Pilot training
I have 29 flights that use the Savage Cub and each one of them have a
different flight number, and these same numbers are used even if you
decide to do the training with the Cessna 152.  So the association with
the aircraft is only while the flight is running.
Even people using Free Flight might change that number all the time for
the same aircraft.
Gilbert Laprise
------ Original Message ------
From "Oliver Knoll" @.>
To "till213/SkyDolly" @.>
Cc "fsx-rwt" @.>; "Author" @.>
Date 2023-03-11 2:01:00 PM
Subject Re: [till213/SkyDolly] flight_number should be an attribute of
the flight table, not of the aircraft table (Issue #103)

Yes, and I just realised that the value shown in the Formation module
is actually the tail number (which is indeed aircraft-specific), not
the flight number. So there is nothing to change in the Formation
module.

I believe that flight_number should be an attribute of the flight
table, not of the aircraft table.

Oh, or are you referring to the actual SQLite database tables (as
opposed to the tables as presented in the user interface)? In this case
you are right: the "flight_number" is indeed part of the aircraft table
(and not the flight table). And yes, this is probably also something
that I could change, but I have to think about the implications (the
obvious implication is that the first recorded aircraft would specifiy
the flight number - for subsequently recored "formation" aircraft the
flight number would simply be ignored).
But your argument that the flight number conceptually really belongs to
the flight (and not any aircraft) totally makes sense to me...
—
Reply to this email directly, view it on GitHub
https://github.com/till213/SkyDolly/issues/103#issuecomment-1464984798,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSXAY2WGJAO7EZYOIDA75TW3TDWZANCNFSM6AAAAAAVXDNEZY.
You are receiving this because you authored the thread.Message ID:
@.***>

The flight number is now stored in the 'flight' table (only) and only recorded with the first aircraft. Subsequent aircraft (recorded in the same formation flight) do not influence the flight number.
The flight number is now also shown in the Logbook table (in the user interface) and searchable.

Thanks, this is much appreciated.
Gilbert Laprise
------ Original Message ------
From "Oliver Knoll" @.>
To "till213/SkyDolly" @.>
Cc "fsx-rwt" @.>; "Author" @.>
Date 2023-03-12 9:02:31 AM
Subject Re: [till213/SkyDolly] flight_number should be an attribute of
the flight table, not of the aircraft table (Issue #103)

The flight number is now stored in the 'flight' table (only) and only
recorded with the first aircraft. Subsequent aircraft (recorded in the
same formation flight) do not influence the flight number.
The flight number is now also shown in the Logbook table (in the user
interface) and searchable.
—
Reply to this email directly, view it on GitHub
https://github.com/till213/SkyDolly/issues/103#issuecomment-1465192987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSXAY6LSRXG42LZXSARE63W3XCOPANCNFSM6AAAAAAVXDNEZY.
You are receiving this because you authored the thread.Message ID:
@.***>

Just a quick one, what about migration of existing flight, should i
export them et import after installing the new version.  What happen if
I don't?
Gilbert Laprise
------ Original Message ------
From "Gilbert Laprise" @.>
To "till213/SkyDolly"
@.>
Date 2023-03-12 1:46:21 PM
Subject Re[2]: [till213/SkyDolly] flight_number should be an attribute
of the flight table, not of the aircraft table (Issue #103)

Thanks, this is much appreciated.
Gilbert Laprise
------ Original Message ------
From "Oliver Knoll" @.>
To "till213/SkyDolly" @.>
Cc "fsx-rwt" @.>; "Author" @.>
Date 2023-03-12 9:02:31 AM
Subject Re: [till213/SkyDolly] flight_number should be an attribute of
the flight table, not of the aircraft table (Issue #103)

The flight number is now stored in the 'flight' table (only) and only
recorded with the first aircraft. Subsequent aircraft (recorded in the
same formation flight) do not influence the flight number.
The flight number is now also shown in the Logbook table (in the user
interface) and searchable.
—
Reply to this email directly, view it on GitHub
https://github.com/till213/SkyDolly/issues/103#issuecomment-1465192987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSXAY6LSRXG42LZXSARE63W3XCOPANCNFSM6AAAAAAVXDNEZY.
You are receiving this because you authored the thread.Message ID:
@.***>

Migration is trivial if a flight contains exactly one aircraft: the flight number of this one single aircraft (table) is moved into the flight table (now we‘re talking about database tables flight and aircraft).
A more interesting case is when you have a formation flight (#aircraft > 1): a flight does have a designated „user aircraft“ (which can be set by the user to any of the formation aircraft), but I then decided to move the flight number of the first aircraft („seq_nr = 1“) into the flight table (and all other flight numbers are ignored).
This is consistent with the fact that the first recorded aircraft (whether additional formation aircraft are going to be recorded / added later on or not) also defines the „global“ flight parameters, most importantly the „flight conditions“ (time of flight, runway conditions, temperature, wind, …).
(Btw while re-evaluating this logic I stumbled over a bug that those conditions actually are overwritten (recorded) when recording additional aircraft, which is wrong. Luckily only „in memory“: the persisted data is correctly stored. This is going to be fixed).

Thanks, so far I am just recording, so I am in gooo shape.  👍
Gilbert Laprise
------ Original Message ------
From "Oliver Knoll" @.>
To "till213/SkyDolly" @.>
Cc "fsx-rwt" @.>; "Author" @.>
Date 2023-03-13 2:50:45 AM
Subject Re: [till213/SkyDolly] flight_number should be an attribute of
the flight table, not of the aircraft table (Issue #103)

Migration is trivial if a flight contains exactly one aircraft: the
flight number of this one single aircraft (table) is moved into the
flight table (now we‘re talking about database tables flight and
aircraft).
A more interesting case is when you have a formation flight (#aircraft

1): a flight does have a designated „user aircraft“ (which can be set
by the user to any of the formation aircraft), but I then decided to
move the flight number of the first aircraft („seq_nr = 1“) into the
flight table (and all other flight numbers are ignored).

This is consistent with the fact that the first recorded aircraft
(whether additional formation aircraft are going to be recorded / added
later on or not) also defines the „global“ flight parameters, most
importantly the „flight conditions“ (time of flight, runway conditions,
temperature, wind, …).
(Btw while re-evaluating this logic I stumbled over a bug that those
conditions actually are overwritten (recorded) when recording
additional aircraft, which is wrong. Luckily only „in memory“: the
persisted data is correctly stored. This is going to be fixed).
—
Reply to this email directly, view it on GitHub
https://github.com/till213/SkyDolly/issues/103#issuecomment-1465599046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSXAYYKH3BGPBO5HYLRLIDW327ULANCNFSM6AAAAAAVXDNEZY.
You are receiving this because you authored the thread.Message ID:
@.***>

Oliver,
I am sorry to bother you with this question and will understand if you
don't have the time to answer, but I am desperate!
For the last few months everyday I have scan Qt Forums and
documentation, to find a lot of answer but nothing works. I see from the
forum that a lot of people are facing the same challenges.
I know that with Sky Dolly you have manage to package everything
including the database, I use it, and even update are painless.
My Appl is a FlightLog application that monitor the MSFS-RWT Training
Flights, log information about the flight, and let the user score their
performance.
Everything works fine and is ready for distribution.  But when I install
it on a different computer it does not find the FlightLog.db in the
DataBase  subfolder of the Application.
I am using QMake. Which entry do I need, to point to the database
location.  (conversion to CMake is still too complicated for me)
How do I make the connection in the .cpp  I know it does not work as a
resource.
I tried :
//QStringdbPath=QCoreApplication::applicationDirPath()+"/FlightLog.db";
//QStringdbPath=QDir::currentPath()+"/DataBase/FlightLog.db";
//QStringdbPath="H:/QtLearn/MsfsRwtFlightLog/DataBase/FlightLog.db";
QStringdbPath=":FlightLog.db";
QMessageBox::information(this,tr("LoginWin"),dbPath);//***GL2024-02-22
mydb=QSqlDatabase::addDatabase("QSQLITE");
/mydb.setDatabaseName("H:/QtLearn/MsfsRwtFlightLog/DataBase/FlightLog.db");/
mydb.setDatabaseName(dbPath);
Nothing works other than the absolute path when I move to another computer.
If you have the time to answer it will be greatly appreciate. In the mean time
I will try understanding Sky Dolly code and continue to enjoy using Sky Dolly,
Best Regards,
Gilbert
P.S.: Eventhough I have been programming in Assembler, since the early
70s, I am still a Newbie with Qt and C++
------ Original Message ------
From "Gilbert Laprise" @.>
To "till213/SkyDolly"
@.>;
"till213/SkyDolly" @.>
Cc "Author" @.>
Date 2023-03-11 1:08:51 PM
Subject Re[2]: [till213/SkyDolly] flight_number should be an attribute
of the flight table, not of the aircraft table (Issue #103)

Just to complement your answer, as a matter of fact, when I generate my
flights I put the flight number in the [Sim.0] entry:
[Sim.0]
Sim=Asobo Savage Cub
SimFile=runway.FLT
TailNumber=MSFS-RWT
AirlineCallSign=MSFS-RWT
FlightNumber=FLT-001
AppendHeavy=False
Pilot=Pilot_Male_Casual
and it is correctly captured by SkyDolly.
Gilbert Laprise
------ Original Message ------
From "Oliver Knoll" @.>
To "till213/SkyDolly" @.>
Cc "fsx-rwt" @.>; "Author" @.>
Date 2023-03-11 12:54:40 PM
Subject Re: [till213/SkyDolly] flight_number should be an attribute of
the flight table, not of the aircraft table (Issue #103)

Technically the flight number - like almost all vaues - comes from the
aircraft, that is via the SimConnect API you query the "simulation
variable" called ATC_FLIGHT_NUMBER
https://docs.flightsimulator.com/html/Programming_Tools/SimVars/Aircraft_SimVars/Aircraft_RadioNavigation_Variables.htm#ATC FLIGHT NUMBER,
given the user aircraft ID (typically SIMCONNECT_OBJECT_ID_USER, to
query the flight number from the current "user aircraft").
But you are right: it makes much more sense to
Present this value in the Logbook (instead of per aircraft, in the
Formation module)
The presented value would simply be the flight number of the "user
aircraft" (as defined for the formation flight)...
... and assuming that the flight number would apply for all aircraft
in the formation equally
In that same argument I would remove it from the Formation aircraft
table, as there the flight number is probably not really of interest
anyway (again, technically it could be different for each aircraft,
depending on the actual flight number as selected at the time of
recording - but in reality we would expect the same flight number for
all aircraft/the formation, just as you said).
—
Reply to this email directly, view it on GitHub
https://github.com/till213/SkyDolly/issues/103#issuecomment-1464964259,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSXAY7OHLOEWVOGRX6AME3W3S36BANCNFSM6AAAAAAVXDNEZY.
You are receiving this because you authored the thread.Message ID:
@.***>

Oliver,
just forget about my request, I finally got the right
combination of options working.
Everything is working fine even from a USB drive.
I will be able to distribute v2.0.0 of MSFS-RWT Real World Flight
Training package.
Thanks again for your support in the past.
Gilbert
------ Original Message ------
From "Gilbert Laprise" @.>
To "till213/SkyDolly"
@.>;
"till213/SkyDolly" @.>
Cc "Author" @.>
Date 2024-02-24 2:16:17 PM
Subject Re: How to package and distribute a QT C++ application with a
populated SQLite DB that will be updated by the end user ???

Oliver,
I am sorry to bother you with this question and will understand if you
don't have the time to answer, but I am desperate!
For the last few months everyday I have scan Qt Forums and
documentation, to find a lot of answer but nothing works. I see from
the forum that a lot of people are facing the same challenges.
I know that with Sky Dolly you have manage to package everything
including the database, I use it, and even update are painless.
My Appl is a FlightLog application that monitor the MSFS-RWT Training
Flights, log information about the flight, and let the user score their
performance.
Everything works fine and is ready for distribution.  But when I
install it on a different computer it does not find the FlightLog.db in
the DataBase  subfolder of the Application.
I am using QMake. Which entry do I need, to point to the database
location.  (conversion to CMake is still too complicated for me)
How do I make the connection in the .cpp  I know it does not work as a
resource.
I tried :
//QStringdbPath=QCoreApplication::applicationDirPath()+"/FlightLog.db";
//QStringdbPath=QDir::currentPath()+"/DataBase/FlightLog.db";
//QStringdbPath="H:/QtLearn/MsfsRwtFlightLog/DataBase/FlightLog.db";
QStringdbPath=":FlightLog.db";
QMessageBox::information(this,tr("LoginWin"),dbPath);//***GL2024-02-22
mydb=QSqlDatabase::addDatabase("QSQLITE");
/mydb.setDatabaseName("H:/QtLearn/MsfsRwtFlightLog/DataBase/FlightLog.db");/
mydb.setDatabaseName(dbPath);
Nothing works other than the absolute path when I move to another computer.
If you have the time to answer it will be greatly appreciate. In the mean time
I will try understanding Sky Dolly code and continue to enjoy using Sky Dolly,
Best Regards,
Gilbert
P.S.: Eventhough I have been programming in Assembler, since the early
70s, I am still a Newbie with Qt and C++
------ Original Message ------
From "Gilbert Laprise" @.>
To "till213/SkyDolly"
@.>;
"till213/SkyDolly" @.>
Cc "Author" @.>
Date 2023-03-11 1:08:51 PM
Subject Re[2]: [till213/SkyDolly] flight_number should be an attribute
of the flight table, not of the aircraft table (Issue #103)

Just to complement your answer, as a matter of fact, when I generate
my flights I put the flight number in the [Sim.0] entry:
[Sim.0]
Sim=Asobo Savage Cub
SimFile=runway.FLT
TailNumber=MSFS-RWT
AirlineCallSign=MSFS-RWT
FlightNumber=FLT-001
AppendHeavy=False
Pilot=Pilot_Male_Casual
and it is correctly captured by SkyDolly.
Gilbert Laprise
------ Original Message ------
From "Oliver Knoll" @.>
To "till213/SkyDolly" @.>
Cc "fsx-rwt" @.>; "Author" @.>
Date 2023-03-11 12:54:40 PM
Subject Re: [till213/SkyDolly] flight_number should be an attribute of
the flight table, not of the aircraft table (Issue #103)

Technically the flight number - like almost all vaues - comes from
the aircraft, that is via the SimConnect API you query the
"simulation variable" called ATC_FLIGHT_NUMBER
https://docs.flightsimulator.com/html/Programming_Tools/SimVars/Aircraft_SimVars/Aircraft_RadioNavigation_Variables.htm#ATC FLIGHT NUMBER,
given the user aircraft ID (typically SIMCONNECT_OBJECT_ID_USER, to
query the flight number from the current "user aircraft").
But you are right: it makes much more sense to
Present this value in the Logbook (instead of per aircraft, in the
Formation module)
The presented value would simply be the flight number of the "user
aircraft" (as defined for the formation flight)...
... and assuming that the flight number would apply for all aircraft
in the formation equally
In that same argument I would remove it from the Formation aircraft
table, as there the flight number is probably not really of interest
anyway (again, technically it could be different for each aircraft,
depending on the actual flight number as selected at the time of
recording - but in reality we would expect the same flight number for
all aircraft/the formation, just as you said).
—
Reply to this email directly, view it on GitHub
https://github.com/till213/SkyDolly/issues/103#issuecomment-1464964259,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSXAY7OHLOEWVOGRX6AME3W3S36BANCNFSM6AAAAAAVXDNEZY.
You are receiving this because you authored the thread.Message ID:
@.***>


disable alpha features

Alpha features may not be enabled by default, or available at all in some distributions.

please add the latest k8guardlibs SHA to the glide yaml, u can grab the latest SHA from https://github.com/k8guard/k8guardlibs/commits/master

Sorry, only just seen this.  Did not mean to check in changes to hooks/pre-commit.  The pre-commit kept failing via IDE, but was fine from command line.  Could not figure out what was going wrong so temporarily disabled it.  Will revert that.

@boardthatpowder sounds good, let me know when you commit the new changes.

Done, ready for review again.

@boardthatpowder  looks good to me. thanks 👍


Auto-injecting of required members - breaking change

Hello,
I have upgraded Stashbox to version v5.13.0, and I immediately noticed a breaking change related to the auto-injecting of required members feature.
I understand the intention, but I am not sure this is the best approach, especially since this is default behavior.
Let's have a look at the example code below:
using Stashbox;
using Stashbox.Attributes;

var container = new StashboxContainer();
// var container = new StashboxContainer(static op => { op.WithAutoMemberInjection(filter: static _ => false); }); // working fine
// var container = new StashboxContainer(static op => { op.WithAutoMemberInjection(enabled: false); }); // does nothing for this case :)

container.Register<ClassA>();
container.Register<ClassB>();

container.Validate();

var resolver = container.Resolve<ClassA>();

return 0;

public sealed class ClassA
{
    public required int SimpleTypeProperty { get; init; }

    [Dependency]
    public required ClassB ClazzB { get; init; }
}

public sealed class ClassB
{
}

Before 5.13, it was working ok - you have to add an attribute to the property in order to make the container automatically inject it.
However, starting with version 5.13, the container is now invalid because of SimpleTypeProperty. While trying to validate the container, I am getting an error:

I understand that my case is very specific, and most of the users will not be affected. However, I think you should add a way to disable this change (on a global level and also on a type level with an attribute or something).
For now, the only way to disable it is to use the WithAutoMemberInjection setting with a filter specified. Also, disabling it with the enabled option (WithAutoMemberInjection(enabled: false)) is not working; it will still try to auto-inject value into the property.
Valker

Hi @eValker, thank you for reporting this issue! I'm going to work on a way to let users disable this feature easier.


Random rotation port

For some reason I'm getting an extra 'annotations' while running layer test, am I missing something?


filldist and arraydist is not exported

Maybe we should consider reexport DistributionsAD from Turing?

I think this is solved now.


Testnet and Devnet in Blockproducer mode is not working

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/MinaProtocol/mina/issues
[X] This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/MinaProtocol/mina/discussions

Description
Testnet when i tried to run the mina node in BP mode it's not even getting connected to peers, i was using Berkley QANet version, this is the error :
2022-06-14 07:29:08 UTC [Error] libp2p_helper: failed to connect to peer: {12D3KooWAdgYL6hv18M3iDBdaK1dRygPivSfAfBNDzie6YqydVbs: [/dns4/seed-1.berkeley.o1test.net/tcp/10000]}failed to dial 12D3KooWAdgYL6hv18M3iDBdaK1dRygPivSfAfBNDzie6YqydVbs:

[/ip4/<IP_ADDRESS>/tcp/10000] failed to negotiate security protocol: read tcp4 <IP_ADDRESS>:8302-><IP_ADDRESS>:10000: read: connection reset by peer
2022-06-14 07:29:08 UTC [Error] libp2p_helper: failed to connect to peer: {12D3KooWLjs54xHzVmMmGYb7W5RVibqbwD1co7M2ZMfPgPm7iAag: [/dns4/seed-2.berkeley.o1test.net/tcp/10001]}failed to dial 12D3KooWLjs54xHzVmMmGYb7W5RVibqbwD1co7M2ZMfPgPm7iAag:
[/ip4/<IP_ADDRESS>/tcp/10001] failed to negotiate security protocol: read tcp4 <IP_ADDRESS>:8302-><IP_ADDRESS>:10001: read: connection reset by peer
2022-06-14 07:29:08 UTC [Error] libp2p_helper: failed to connect to peer: {12D3KooWEiGVAFC7curXWXiGZyMWnZK9h8BKr88U8D5PKV3dXciv: [/dns4/seed-3.berkeley.o1test.net/tcp/10002]}failed to dial 12D3KooWEiGVAFC7curXWXiGZyMWnZK9h8BKr88U8D5PKV3dXciv:
[/ip4/<IP_ADDRESS>/tcp/10002] dial backoff
2022-06-14 07:29:08 UTC [Error] libp2p_helper: failed to connect to peer: {12D3KooWLjs54xHzVmMmGYb7W5RVibqbwD1co7M2ZMfPgPm7iAag: [/dns4/seed-2.berkeley.o1test.net/tcp/10001]}failed to dial 12D3KooWLjs54xHzVmMmGYb7W5RVibqbwD1co7M2ZMfPgPm7iAag:
[/ip4/<IP_ADDRESS>/tcp/10001] dial backoff
2022-06-14 07:29:08 UTC [Error] libp2p_helper: failed to connect to peer: {12D3KooWEiGVAFC7curXWXiGZyMWnZK9h8BKr88U8D5PKV3dXciv: [/dns4/seed-3.berkeley.o1test.net/tcp/10002]}failed to dial 12D3KooWEiGVAFC7curXWXiGZyMWnZK9h8BKr88U8D5PKV3dXciv:
[/ip4/<IP_ADDRESS>/tcp/10002] dial backoff
2022-06-14 07:29:08 UTC [Error] libp2p_helper: failed to connect to peer: {12D3KooWAdgYL6hv18M3iDBdaK1dRygPivSfAfBNDzie6YqydVbs: [/dns4/seed-1.berkeley.o1test.net/tcp/10000]}failed to dial 12D3KooWAdgYL6hv18M3iDBdaK1dRygPivSfAfBNDzie6YqydVbs:
[/ip4/<IP_ADDRESS>/tcp/10000] dial backoff
2022-06-14 07:29:08 UTC [Error] libp2p_helper: failed to routing advertise: failed to find any peer in table
2022-06-14 07:29:08 UTC [Warn] starting libp2p up failed: $error
error: {
"commit_id": "ba9a0e088b73a3c389550ee002ef0afc1ce3976d",
"string": "libp2p error: failed to find any peer in table"
}
2022-06-14 07:29:53 UTC [Error] Failed to find any peers during initialization (crashing because this is not a seed node)

Steps to Reproduce

running following mina command i get the error
mina daemon --peer-list-url https://storage.googleapis.com/seed-lists/berkeley_seeds.txt --insecure-rest-server --block-producer-key /keys/my-wallet --MINA_PRIVKEY_PASS="******"

...
Expected Result
Should be able to send payments from testnet faucet
Actual Result
2022-06-17 13:46:07 UTC [Info] Running async shutdown handler: "Close transition frontier, if exists"
2022-06-17 13:46:07 UTC [Info] Running async shutdown handler: "Remove daemon lockfile"
2022-06-17 13:46:07 UTC [Warn] starting libp2p up failed: $error
error: {
"commit_id": "ba9a0e088b73a3c389550ee002ef0afc1ce3976d",
"string": "helper process already exited (doing RPC BeginAdvertising)"
}
(monitor.ml.Error
(Sys_error
"/root/.mina-config/coda_crash_report_2022-06-17_13-46-07.304594/coda_status.json: No such file or directory")
("Raised by primitive operation at Stdlib.open_out_gen in file "stdlib.ml", line 324, characters 29-55"
"Called from Stdlib.open_out in file "stdlib.ml" (inlined), line 329, characters 2-74"
"Called from Yojson.Safe.to_file in file "write.ml", line 469, characters 11-24"
"Called from Init__Coda_run.make_report.(fun) in file "src/app/cli/src/init/coda_run.ml", line 216, characters 2-40"
"Called from Async_kernel__Deferred1.M.map.(fun) in file "src/deferred1.ml", line 17, characters 40-45"
"Called from Async_kernel__Job_queue.run_job in file "src/job_queue.ml" (inlined), line 128, characters 2-5"
"Called from Async_kernel__Job_queue.run_jobs in file "src/job_queue.ml", line 168, characters 6-47"))
How frequently do you see this issue?
Always
What is the impact of this issue on your ability to run a node?
Blocker
Status
2022-06-17 13:46:07 UTC [Info] Running async shutdown handler: "Close transition frontier, if exists"
2022-06-17 13:46:07 UTC [Info] Running async shutdown handler: "Remove daemon lockfile"
2022-06-17 13:46:07 UTC [Warn] starting libp2p up failed: $error
        error: {
  "commit_id": "ba9a0e088b73a3c389550ee002ef0afc1ce3976d",
  "string": "helper process already exited (doing RPC BeginAdvertising)"
}
(monitor.ml.Error
 (Sys_error
  "/root/.mina-config/coda_crash_report_2022-06-17_13-46-07.304594/coda_status.json: No such file or directory")
 ("Raised by primitive operation at Stdlib.open_out_gen in file \"stdlib.ml\", line 324, characters 29-55"
  "Called from Stdlib.open_out in file \"stdlib.ml\" (inlined), line 329, characters 2-74"
  "Called from Yojson.Safe.to_file in file \"write.ml\", line 469, characters 11-24"
  "Called from Init__Coda_run.make_report.(fun) in file \"src/app/cli/src/init/coda_run.ml\", line 216, characters 2-40"
  "Called from Async_kernel__Deferred1.M.map.(fun) in file \"src/deferred1.ml\", line 17, characters 40-45"
  "Called from Async_kernel__Job_queue.run_job in file \"src/job_queue.ml\" (inlined), line 128, characters 2-5"
  "Called from Async_kernel__Job_queue.run_jobs in file \"src/job_queue.ml\", line 168, characters 6-47"))

Additional information
No response

There seem to be a multiplicity of errors and difficulties in the above error report. I propose that the issue stems from the use of the (https://storage.googleapis.com/seed-lists/berkeley_seeds.txt) wrong peers list, which would explain the connectivity problems.
The seeds for devnet are referred to here:
https://docs.minaprotocol.com/en/advanced/connecting-devnet
The seeds for mainnet are referred to here:
https://docs.minaprotocol.com/en/using-mina/connecting


Better templating for worker startup script

@landonreed and I just spent quite a long time debugging the code on the analysis broker that substitutes values into a startup shell script and passes it on to the worker machines.
This is because we've got variable replacement going on in Java, then in the shell where the script is run on the worker, and both Java MessageFormat and the shell have special meanings for quotes and curly braces.
We might want to just do String.replaceAll("X", variable) for clarity and maintainability.

Fixed for now in https://github.com/conveyal/analysis-backend/commit/0c27b8ecd167c22bc6703811da12d6623e85f03d#diff-46201b2dc4edfe18b00966d76082a75b


Update RAM recommendation for UCP managers

Proposed changes
Added recommendation of 8GB of RAM for manager nodes. This is was recommended for our systems by the UCP team. 4GB is dangerously low for a medium to large cluster running a large number of services and tasks.

I agree with this, but I'd like @vsaraswat 's opinion.

At a start I think it's fine. I'd ask @vikstrous or @pdevine what DTR recommendation is (if it is 8GB then I agree).
One q: Should we backport this fix to the UCP 2.1 docs and 2.0 docs as well?

Yes, I would backport this to UCP 2.1 and 2.0. I can take care of that.

FWIW, recent testing of a patch for the snapshot size also shows that 4 GB is too low of a recommendation for a medium size cluster. cc @davidwilliamson

Mehdi does that include workers? Or still just managers?
-Vivek
Sent from my iPhone

On Sep 21, 2017, at 11:36 AM, Mehdi Ghazizdeh<EMAIL_ADDRESS>wrote:
FWIW, recent testing of a patch for the snapshot size also shows that 4 GB is too low of a recommendation for a medium size cluster. cc @davidwilliamson
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@vsaraswat I think we should limit the updated recommendation for managers only.
The memory-related issues I saw pertained to managers running UCP (i.e., user services restricted to worker nodes)
The requirements for workers depend on what services the user plans on running, how many workers they deploy, and possibly what node constraints they use for deployments. I'm not sure we can make a blanket recommendation for workers.

rethinkdb scales to use as much ram as it can for memory caches, so the more ram you give DTR, the better it'll perform. It needs at least 2-4 GB to be able to run scans though. It can run on less if you don't use security scanning.

Just managers
Sent from my iPhone

On Sep 21, 2017, at 12:07 PM, Vivek Saraswat<EMAIL_ADDRESS>wrote:
Mehdi does that include workers? Or still just managers?
-Vivek
Sent from my iPhone

On Sep 21, 2017, at 11:36 AM, Mehdi Ghazizdeh<EMAIL_ADDRESS>wrote:
FWIW, recent testing of a patch for the snapshot size also shows that 4 GB is too low of a recommendation for a medium size cluster. cc @davidwilliamson
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


contracts: add SHA384 hash and P-384 curve

Have tested this against sapphire-dev from https://github.com/oasisprotocol/oasis-web3-gateway/pull/469

The new precompile should also be added to https://github.com/oasisprotocol/docs/blob/main/docs/dapp/sapphire/precompiles.md


support for sqlite

When I install through npm, install version 0.0.5 that does not contain support for sqlite.
How do I install the version with support for sqlite?
thanks

Hi @BradyLiles! Do you have some plan to continue to maintain this code?
Best,
Alessandro.

Hi @adrianrey05. Look at https://github.com/sidneycorreia/ionic-orm.

@adrianrey05 @alemp @sidneycorreia there are multiple requests for this feature - please add your requirements & contribute so that we get this supported in the maintained typeorm repo

@mobidev111  @adrianrey05  @alemp  @sidneycorreia
If any of y'all want to help maintain this repo, let me know. I am more than willing to give privelages to anyone who wants to help.
I would love to have the time and willingness to support this type of database in mobile applications, but I found better practice and performance in using IndexedDB instead with LokiJS.


Context node

Switch parameters to general Context node (choose beetween sequence and selector execution).
Add interface on ShowPhrase and Question.
Add GetCurrentPhrase in ShowPhrase task.
Change DialogueEventListener search name.
Context node can be used for other things.

yeah! thank you @devtrack!
And about EventListener - we should change searching widget. Don't use name at all. We have to search it by class) I'll create a commit to change it :smile:
Good work! And thanks for contribution! :+1:


Avoid quotes in space seperated strings

Quote word is adding quotes on each word in a string which is space separated. I want quotes to appear in every new line and not in space seprated strings. Here is the example,
Problem
French Polynesia => "French" "Polynesia"
French Southern Territories => "French" "Southern" "Territories"
Required
French Polynesia => "French Polynesia"
French Southern Territories => "French Southern Territories"

Simple 3 character change. Remembering how to publish it will be more tricky ;-)

I have incorporated this request because it aligns with the original use case for the package... to quote comma/punctuation separated words.
It took more than 3 characters ;-) Tests added and documentation updated with the examples above.
Version 0.2.0 is available for download from atom.io.
Thank you for your input.

Great, now my problem has been solved. Actually, I have lots of JSON data so your package is very helpful.
Thanks.

You're welcome. I originally created the package to help with generating JSON data, so I'm glad to hear you find it useful for that too. :-)


Layer via mapboxgl.Map.addLayer() in rendered image missing

Hi, this is great, but it doesn't seem to work with custom layers (e.g. GeoJSON information) on top of the map. Do you have any idea how to get a print of both the base layer map and one or more overlays?
I can't explain why that would be the case?! Somehow the data/layer must be in the canvas object, but the layers don't seem to be considered while exporting?
Btw. it also doesn't work without your tooling by grabbing the image from the canvas via toDataURL.

If you're modifying the code, you're on your own. It must have something to do with how Mapbox GL JS handles additional layers.


Dynamic type definitions inside template not resolved properly

TypeScript Version:  2.8.0

Search Terms:
dynamic types, mapped types, template typedef,
Code
// snippet1
// doesnt work
function foo<T, R, F = (x: T) => R>(x: T): F {
let fn: F;
fn = (x: T) => (!!x);
return fn;
}
//works
type F1<T, R> = (x: T) => R ;
function bar<T, R>(x: T): F1<T,R> {
let fn: F1<T,R>;
fn = (x: T) => (!!x);
return fn;
}
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.

Expected behavior:
No compilation errors on snippet1
Actual behavior:
Compilation error
Playground Link:
https://www.typescriptlang.org/play/#src=

function foo<T%2C R%2C F extends { (x%3A T)%3A R }>(x%3A T)%3A F {
    let fn%3A F%3B
    fn %3D (x%3A T) %3D> <R>(<any>!!x)%3B
    return fn%3B
}

type F1<T%2C R> %3D (x%3A T) %3D> R %3B
function bar<T%2C R>(x%3A T)%3A F1<T%2CR> {
    let fn%3A F1<T%2CR>%3B
    fn %3D (x%3A T) %3D> <R>(<any>!!x)%3B
    return fn%3B
}

Related Issues:

As intended.  F could have other things on it besides that one call signature.  Consider the following:
https://www.typescriptlang.org/play/#src=

function foo<T%2C R%2C F extends { (x%3A T)%3A R }>(x%3A T)%3A F {
    let fn%3A F%3B
    fn %3D (x%3A T) %3D> <R>(<any>!!x)%3B
    return fn%3B
}

interface Bof {
    (x%3A number)%3A boolean%3B
    isBaz%3A true%3B
}

const bof %3D foo<number%2C boolean%2C Bof>(5)%3B
console.log(bof.isBaz)%3B %2F%2F should print `true`.  prints `undefined`

Even in your code

That's a slightly different example, but it fails for the same reason (and more).  When you say F = (x: T) => R, you're just saying that the default value for the F template argument if not otherwise provided is (x: T) => R.  F could be specified to be anything at all:
foo<number, boolean, string>(3)

What should this return?
If you do F extends (x: T) => R = (x: T) => R you're back to the original case and your code is unsound as my first example demonstrates.

makes sense. Thanks


Slider: Tooltip not disappearing on outside click or hidden when moving the slider

Prerequisites

[X] I have not leaked any internal company information.
[X] I have not posted internal links or code snippets.

What happened?
Hi,
I observed two issues with the tooltip for displaying the currently selected value on the Slider component.

Sometimes the tooltip is still displayed after clicking multiple times outside of the component:
Here I already clicked multiple times on the empty part of the page below the sliders and the tooltip is still visible. Moving the slider fixes this behaviour.

The tooltip completely disappears (also not visible while dragging the slider) and returns after clicking outside of the component and moving the slider again:
Here I moved the top slider without clicking on the empty part of the page and the tooltip was not visible while dragging the slider and in its final state.

I used the IX documentation page of the Slider in the examples. The issues appear a bit randomly, so I cannot provide concrete instructions on how to reproduce them.
What type of frontend frameware are you seeing the problem on?
Angular
Which version of iX do you use?
2.0.4
Code to produce this issue.
-

Thank you for providing your insights. Could you please confirm whether this issue persists in the latest version (v2.1.0)? I attempted to replicate the scenario using 2.1.0, and as of my testing, the issue no longer seems to be present.
Stackblitz: https://stackblitz.com/edit/5ykh7k?file=package.json%3AL18,package-lock.json

We will close this issue. If it still persists, don't hesitate to open a new issue!


Doesn't show anything. Docs are not clear about default usage.

Version
v3.0.0
Reproduction link
https://codesandbox.io/
Steps to reproduce
My version is 4. It is not listed in options.  Nothing show on console.
What is expected ?
It doesn't show "fancy" bars.
What is actually happening?
Says compiling and done as Basic mode.
Additional comments?

This bug report is available on Nuxt community (#c43)

I agree, documentation is sketchy at best.

Basic fallback happens when no TTY is available this is not a bug. Need more info like console being in use or at least screenshot to even give any hint.

It does work only on new Windows Terminal!

Would love to implement this in a PR for nuxt (to show progress during route generation) - but it is unclear exactly how

To reduce distractions nuxt/webpackbar/consola use basic reporters in CI environments.
If you think this behavior in general could be improved, please feel free to open a downstream issue in Nuxt repo.


Move content of Role nodes into Page nodes

Role nodes have properties that usually only Page nodes have, because the properties belong to the homepage, not to the thing.  This ought to be changed, for consistency.  But the whole model needs to be redesigned anyway, so this could wait until then.

We're going to remove Neo4j https://github.com/alphagov/govuk-knowledge-graph-gcp/issues/452


fix(EVM): Add static MCOPY gas cost

What ❔

Why ❔

Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.

Closed in favor of https://github.com/matter-labs/era-contracts/pull/1134


Patterns-Wufoo-Twilio connections and settings

User scenario: From Patterns admin, user is able to create an association between a Wufoo form id and keyword. When the keyword is texted to the Twilio number, Patterns reads the questions from the specified Wufoo form and saves the answers to the Wufoo form.

Create Model to save setting.

Possible schema:

id
name/description
Wufoo form ID

Create POST endpoint that is built from the Model to respond to incoming Twilio messages

Initial version: https://github.com/smartchicago/kimball/commit/724c283ef590b06e149799e9a44e5c11bfbec5fd
At this point there is still an issue with the final message (or messages in the case of signup)

Add final message field to model. Signup form final question will still have to be hard coded right now

I have added 2 fields to this model. end_message and form_type.
if form_type = "signup", end_message is ignored
Otherwise, end_message is the message that will be sent at the end of the form.

Since we want to send messages from the same number for both sign-up as well as test notification, testers must text a keyword to start the sign-up process. Keywords can be changed or added in the "Twilio-Wufoo Connections" page. Multiple keywords can be included for sign-up (example: "hello," "hi," and "hey" can all work).
In the future we want to change the usability of this page so it will be easier to distinguish between sign-up and test keyword additions. This relates to #95 when we change to being more test focused.

In commit f457af4bd37f138e5165a397f4a4a9b565ea427d the Twilio-Wufoo Connections page now divides up into 3 sections: "Others-Active", "Signup-Active", and Inactive.

@shua123 let's rename the "Others" section to "Tests" because this section will only show active keywords that relate to test text messages such as call-out/availability messages.

"Others" renamed to "Tests" in https://github.com/smartchicago/kimball/commit/44d00731578e1f7698ab67949d1efe6fae434692

@shua123 Instead of "Twilio-Wufoo Connections" this page should be renamed "SMS Campaigns" (email sent to testers will be labeled as "Email Campaigns")

Closing this issue because it is completed for text test notification. As we do the future re-design we want to think about how Import Phone Numbers can be part of the SMS campaigns page.


Add SFTP support

I need to embed an SFTP server in a Quarkus application. This seems to be the best option, except that it only includes the core server, not the SFTP subsystem. I can include the subsystem separately (normal dependency), but that does not feel ideal. I bet it includes things that need to be adjusted to support native images.
Now, I can either try to get it working as part of this project, or I can write a new public or private extension focusing on the SFTP server. What would you like? If people are using this just for SSH they may not want the extra classes, but on the other hand SFTP is very commonly used with SSH.
What do you think?

We went with another library instead, figuring it would take too much time to get this production ready in native mode.


Redesign lecture edit page for lecturers

Current preview:

TODO

[x] Add deep linking according to this guide and this SO answer
[x] Avoid redirect with Cancel button
[x] Fix layout issues
[x] Fix sample DB Save & Cancel buttons
[x] Fix Comments save button not saving any changes (it even triggers an exception)

For reviewers
Sorry for the over 1,000 line insertions/deletions in this PR. This is mainly due to having migrated the lectures.coffee file to lectures.js and due to copying/pasting some old content to new locations while also modifying it, such that Git does not recognize it as replacements and instead deletions/insertions.

[x] Headers should be internationalized
[x] Every functionality that was in the accordion beforehand, should still work.
[x] Clicking on the "cancel" button should not redirect to the "content" tab but stay in the current tab
[x] Jumping directly to a tab via a url should work, e.g. this one
[x] Accessibility: navbar header can be navigated by using Tab, then Arrow keys (up down as well as left right)
[x] URL hashes (e.g. "#settings") should be a good naming choice as they shouldn't be changed in the future very often (to allow easy bookmarking for example)
[x] Announcements should be scrollable (reproduce by creating at least 10 new announcements)
[x] Importing media should still work.

Thanks for the review. I've addressed all your points in the last commits.

Any action on any subpage (e.g. #people) should let the user stay on the current subpage. Note that for some actions, we reload the page, which is why some small visual flickering might appear as the "Content" tab is loaded first, then we jump to the correct section. This could be addressed in a future PR, but is maybe not worth the effort (one would have to implement some kind of js-data-storage mechanism or wrap the forms in partials, create a submit/cancel route and send custom JS from the backend such that the partial is reloaded with the initial data. This way, we wouldn't have to reload the entire page.
The back/forward buttons should now work properly again.

I only stumbled upon one very strange bug involving the navigation of the navbar headers by Tab and arrow keys. If you make use of that and return to the content tab, the layout for the content tab is suddenly broken.

I've listened to the wrong event listener, see the correct ones here. I'm now listening to shown.bs.tab, i.e. only initialize the grid once the content of the tab is already loaded. Otherwise, the focus event triggered too early and the grid system was initialized when the content was not visible yet, therefore you saw the broken layout. It should now work flawlessly, even when resizing the browser.