Force quotes

Hi,
is there any way to force quotes for string values? I understand ytt will enclose a value in quotes when the value can be parsed as a number, or it's prefixed with a "special" character (#, @, ...), for example:
#@ value = "foo"
a: #@ value
#@ value = "123"
b: #@ value
#@ value = "123.5"
c: #@ value
#@ value = "#foo"
d: #@ value

Gives
a: foo
b: "123"
c: "123.5"
d: '#foo'

I have a scenario where a value can be in the form \d+e\d+ (i.e. 74e3):
#@ value="74e3"
e: #@ value

which gives
e: 74e3

The yaml we're producing is processed by another tool that inteprets the value above as a number in scientific notation, and the value is changed (74e3 is translated into 74000.0).
Adding quotes forces the downstream system to parse the value as a string, but I didn't find a way to tell ytt to explicitly add quotes (or add some raw content to the output). Is this feature supported?
Thanks,
lorenzo

@cppforlife what's the word on patching the YAML parser?

we use https://gopkg.in/yaml.v2 (we just happen to vendor it in) so changes like these would have to go through there. i do not think it is a good idea for us to maintain our own yaml spec parsing changes on top of this library.

Hi, I didn't notice that ytt was "embedding" some code from gopkg.in/yaml.v2. Anyway I see that the latest version of gopkg.in/yaml.v2 is using a different regexp to resolve a float:

gopkg.in/yaml.v2

var yamlStyleFloat = regexp.MustCompile(`^[-+]?(\.[0-9]+|[0-9]+(\.[0-9]*)?)([eE][-+]?[0-9]+)?$`)

ytt

var yamlStyleFloat = regexp.MustCompile(`^[-+]?[0-9]*\.?[0-9]+([eE][-+][0-9]+)?$`)

The relevant part (at least for my issue) is [eE][-+]? vs [eE][-+].
That part was updated ~2 years ago in this commit
Thanks

(apologies for the delay, @lorenzobenvenuti)
Well this makes it all very interesting. 😄

... I see that the latest version of gopkg.in/yaml.v2 is using a different regexp to resolve a float ...

Okay, and so we're tacking to a conversation about upgrading to the id'ed version of the Go YAML v2 package that contains this modified regular expression.
To make progress on this idea, we'd need to:

[ ] determine the exact version of gopkg.in/yaml.v2 we've embedded in ytt
[ ] determine the exact version of gopkg.in/yaml.v2 that includes this change in the way floats are parsed
[ ] identify any/all breaking changes make between these versions.

bonus points to do the same up through v2.4

and from there we can have an informed conversation about the potential risks of making the upgrade.

FWIW: as of https://github.com/go-yaml/yaml/commit/496545a6307b2a7d7a710fd516e5e16e8ab62dbcc (at the time of writing, the HEAD of v3), this version of the package will drop/lose comments in non-trivial situations and is not yet suitable for use in ytt.

For the immediate term, @lorenzobenvenuti, is one of the currently identified workarounds "sufficient" for the case?

Yes, thanks!

FWIW, we have done the legwork; @lorenzobenvenuti, the commit you pointed us to was cherry-pick-able as is and we've got a PR (#483) to merge it in.

YAML specification supports type forcing with tags: https://yaml.org/spec/1.2.2/#24-tags
confusing_string: !!str 7e43

Result should be string like "7e43"


Add easy ways to test the entilement

toolbox/entitlement/test_in_podman.sh /path/to/key.pem
toolbox/entitlement/test_in_cluster.sh /path/to/key.pem

/test gpu-operator-e2e

/test images

/test gpu-operator-e2e

/test gpu-operator-e2e

/approve

/hold

/test gpu-operator-e2e

/test gpu-operator-e2e

/test gpu-operator-e2e

/test gpu-operator-e2e

/test gpu-operator-e2e
On Thu, May 6, 2021 at 11:38 AM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests:
Test name Commit Details Rerun command
ci/prow/gpu-operator-e2e d2e1172
https://github.com/openshift-psap/ci-artifacts/commit/d2e1172a02102f8e23ca8a91f037b655340ad76a
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/120/pull-ci-openshift-psap-ci-artifacts-master-gpu-operator-e2e/1390217302070267904 /test
gpu-operator-e2e
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=120.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/120#issuecomment-833386715,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIRSQQQW4VDSUVOVTW3TMJPSFANCNFSM43TG6S3Q
.

A lot of the code created in this PR also shows up in this PR, as if you accidentally branched one of the other rather than off master


No lyrics download for windows GUI?

I have use GUI for a while, it's more convenient for me. I am wondering whether there is a choice to download lyrics in GUI but I miss it, or the GUI version don't have this feature at all. If not, could you add it, that would be wonderful!

Would be awesome to have it in GUI.


Appstudio update test-component-pac-zenf

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-zenf-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
55 seconds
build-container

✅ Succeeded
7 seconds
sast-snyk-check

✅ Succeeded
24 seconds
inspect-image

✅ Succeeded
24 seconds
deprecated-base-image-check

✅ Succeeded
12 seconds
clair-scan

✅ Succeeded
38 seconds
clamav-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
20 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


Survey buttons de-select on mobile when scrolling

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in QA
Followed all applicable steps in the TROUBLESHOOTING guide: TODO
Checked for relevant discussions on Discord: https://discord.gg/pPERUuv
Checked that your issue isn't already filed: https://github.com/COVID-19-electronic-health-system/Corona-tracker/issues

Description
When using CoronaTracker on an actual mobile phone (currently using iPhone X, both on Chrome and Safari) - filling out the survey page 2 and scrolling will cause buttons to become de-selected in the UI. They're still "selected" behind the scenes, but the UI shows them de-selected.
Steps to Reproduce

Open the app on a mobile device
Take a survey

Expected behavior:
Buttons stay highlighted while scrolling
Actual behavior:
Buttons occasionally de-highlight while scrolling
Reproduces how often:
~80%
Versions
1.0.0
Additional Information
This does not reproduce on the mobile view for Chrome dev tools, you need to test it on an actual mobile device

I can work on this

You're on a roll!


Fix Reversal and Flail power calculation for Gen 4

After investigation in Generation 4 speedrunning (thanks minnow), it appears that the thresholds for Gen 4 are wrong when calculating the power of Flail and Reversal.
The calculation is correct for Gen 5+, though.

can you fix travis?

Kris asked me to verify if this is actually the case. I loaded up my Gen 4 game to see (ENG SoulSilver vs. ENG Platinum; host SoulSilver) and had a Blissey with 23/714 HP use Flail on another target. If Flail were using a constant of 48, this should qualify as a BP of 200, because floor(2348 / 714) = 1. If Flail was using a constant of 64, this should qualify as a BP of 150, because floor(2364 / 714) = 2. Given this level 100 Blissey with 25 Attack and a target Dewgong with 185 Defense, I dealt 24 and 25 damage. So this gets my seal of approval!
As an aside, I find this really bizarre; the Emerald disassembly uses identical scaling to both Gen 5) and Gen 7. I always thought the reason they used 48 was because there was 48 pixels on the HP bar - I wonder if that wasn't the case in Gen 4.

Thanks for this!


empty frontmatter title should not make h1

Fixes a case that may have been uncommon but is more likely when https://github.com/executablebooks/mystjs/issues/281 is done.

This is awesome @tavin, thanks so much for the various contributions around this!!
I will look at this in my afternoon (in about 4hrs) and get it out!!

Cheers @rowanc1 this one should be the least debatable of the PRs I have opened :)

Just working to get back to this, will get to it today!!

Merging this in. :) Thank you.


Issues with specific items.

I run a private server, with a few of my friends. We LOVE this data pack. However we are running into a gamebreaking error, One, it seems like we cannot craft a quarry, AT ALL, same goes with Machine Upgrades, and two, crystal composite kicks us from the server if the alloy furnace that has it is opened. If you break it with a pickaxe, instead of a wrench, you will be nonstop kicked from the game until you delete the playerdata of the player associated. This is the error in console when someone tries to see the crystal composite: User f28maniac7 has disconnected, reason: Internal Exception: io.netty.handler.codec.EncoderException: java.lang.IllegalArgumentException: Invalid key. Must be [a-z0-9/.-]:
12.08 10:28:45 [Server] ERROR java.lang.IllegalArgumentException: Invalid key. Must be [a-z0-9/.-]:

sorry to hear about that since imCoolYha updated how the park works
he/she has to start like new for the items, Please be patient as only
1 person is working on this pack
On 8/12/18, CodyDoesThings<EMAIL_ADDRESS>wrote:

I run a private server, with a few of my friends. We LOVE this data pack.
However we are running into a gamebreaking error, One, it seems like we
cannot craft a quarry, AT ALL, same goes with Machine Upgrades, and two,
crystal composite kicks us from the server if the alloy furnace that has it
is opened. If you break it with a pickaxe, instead of a wrench, you will be
nonstop kicked from the game until you delete the playerdata of the player
associated. This is the error in console when someone tries to see the
crystal composite: User f28maniac7 has disconnected, reason: Internal
Exception: io.netty.handler.codec.EncoderException:
java.lang.IllegalArgumentException: Invalid key. Must be [a-z0-9/.-]:
12.08 10:28:45 [Server] ERROR java.lang.IllegalArgumentException: Invalid
key. Must be [a-z0-9/.-]:
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ImCoolYeah105/Mechanization/issues/30

--
Marc-Andre Maurice
<EMAIL_ADDRESS>

Thank you, this may just be a serverside issue, as it works on single player, im looking through the functions atm to see if i can resolve this issue

So far its firing blanks, i cant seem to locate an issue. and its just the crystal composite, everything else works

The pickaxe/axe were accidentally swapped in the recipe, This will be fixed next update (you can reverse them to craft it for now). I'll check in on the machine upgrade recipe (the crafting system was recently rewritten, so there have been a bunch of bugs with it).
I'll check in on the crystal composite. Sounds like a stray character got into the item name.


sqlalchemy: handling of composites in todict

This addition emerged from #2568.
In short, it is some convenience code for people who want to declare composites in SQLAlchemy models and have them easily converted into nested dictionaries.

@dset0x what is the status? Do you have any updates? Please include unit tests for this non-trivial functionality. Thank you!

@jirikuncar I'm afraid not. I have switched my focus towards SCOAP3 checker plugins during the past couple of weeks.
I agree, unit tests are a must here. I will try to get back to this as soon as possible.

Started some work on the tests.
I noticed that the fromdict function is a bit deceivingly named, as it only updates from a dict, not creates. Was this really its purpose?

One may notice the support for __getstate__ and __setstate__ in composites, but as you will see in the comments in the code, this is impractical for our case, as composites that have all None members, are not instantiated at all. We may still be able to partially leverage this down the line, but that should not affect the interface as far as modules are concerned.

I will revise for missing tests and will be dropping the WIP in the following days.
Please excuse the SQLAlchemy introspection insanity and PEP8 violations for the sake of test readability.

Commits squashed. master here we come.

NEW Handles composite columns in todict and fromdict model
patched-in methods can now handle composite columns gracefully by
representing them as dictionaries.

Can you split it into two sentences? Thanks

@dset0x nice trick to avoid kwalitee. Please rebase and re-push.

@jirikuncar That was not meant to be a trick. Why would I try to avoid kwalitee? I realized it was not passing the checks after removing WIP after bad squashing. Hence the WIP remove-add-remove. Hence the edit in the bump message.
If kwalitee only runs when there is no WIP, how can one safely remove WIP?

how can one safely remove WIP?

One can run kwalitee check message/files locally.

As PostgreSQL support is coming we might have a look into http://sqlalchemy-utils.readthedocs.org/en/latest/data_types.html#module-sqlalchemy_utils.types.pg_composite instead. If urgent need for this general feature arises we should consider contributing to SQLAlchemy(-Utils) directly.


User cancel subscription

Feature request

firestore-stripe-payments

Is your feature request related to a problem? Please describe.
there's no way to unsubscribe or cancel subscription. My users can subscribe to the premium access on my app by they can also cancel their subscription anytime. but there's  no easy way to do it at the moment
Describe the solution you'd like
A function where a user will pass subscription.id and it should cancel the subscription in the end_period or immediately
Describe alternatives you've considered
Create my own firebase function that will call stripe API to cancel such subscription

Why not just use the stripe api on your backend to do this? It can be done with a callable function

@bsnman Have you considered implementing the Customer portal which you can configure to allow users to cancel/update their subscription(s)? This extension provides a createPortalLink function you can invoke to create a portal session.

@jsteele-stripe I have implemented the Customer portal using the function you mentioned. Although I am facing another issue, I do not see any change in the subscription collection object even when user cancels the subscription. In the end, I want to find a way to update the subscription status of a user after the period that the user has paid for expires (after the user has cancelled the subscription). I was thinking that there will be some change in subscription collection when user cancels, which will help me manage that but like I said before, I do not see any change. Could you please let me know what is the best way to implement this?
Another question I have (related to question above): for now, I have attached a firebase role for each product (as meta data), which I then use to set the subscription status for each user. What happens to this value once the user subscription expires? Will that be automatically removed for that user i.e. will be set to undefined in my app? Because if that is the case, then I do not have to do anything. I am just not sure if that is how it is going to work.
Any help will be highly appreciated.
Thanks a lot in advance.

@immocenter The extension should handle all cases of subscription cancellations/expiries via the manageSubscriptionStatusChange function (see below). This is a Firebase function that, if configured correctly with Stripe as per the installation instructions, will receive and action any related events accordingly.
https://github.com/stripe/stripe-firebase-extensions/blob/2c99a55d3eb47d3d59d12df7a1dd0714129eb74b/firestore-stripe-payments/functions/src/index.ts#L446

@immocenter The subscription document will have its status automatically changed from "active" to "canceled" by this extension. The document is not deleted. You're looking for a change in a "collection object" which suggests you're looking for the wrong thing. And yes their role will be removed

@jsteele-stripe I have configured the extension exactly as per instructions. I just wanted to be sure if the event will get triggered automatically. Is there any documentation available which goes into details of the extensions i.e. how it works exactly? I am guessing these functions get triggered by cloud functions and most likely cloud functions get triggered when an event happens on the stripe side i.e. expiration of a membership. Let me know if my understanding is correct. Apologies for sounding noobish :). Just trying to get my head around. Thanks a lot for your help.

@DCSnip3r So the subscription document will get changed when the event i.e. subscription expiration happens on Stripe side and it will probably call the cloud function to make that change, right? Just trying to understand the event cycle. Apologies for sounding noobish :). Thanks a lot for your help.

@immocenter Correct.
This extension sets up an http callable Firebase function that can handle events that happen on Stripe. We then tell Stripe to send specific events that happen to this function (by configuring a webhook endpoint restricted key)


It would be nice to tag earlier releases if it's easy

We unfortunately have some code which depends on 1.9.3 (needs to be fixed, obviously), and it took me a while to work out which commit this was.  I think it's f036faeefc6ebb15742dc6471642cbe61fe1984b, but I'm not completely sure.
I guess all these got lost when it moved from ?google code?, but if it was easy to tag them that would be cool.

The 1.9.3 release was on here until the 1.9.5 release.  Are you having issues because there is no longer a 1.9.3 release of pyproj on github?
Is this an open source project or proprietary one?

I'm trying to get some code to run on a new set of machines.  It depends on pyproj, and the version installed on the donor platform is 1.9.3 (at least pyproj.__version__ thinks so).  On the new set of machines I'm using conda to construct an environment, and conda only has 1.9.4 (or perhaps 1.9.5).  The code's unit tests are failing for this version so I want to temporarily go back to 1.9.3 before I fix them (basically so I can fix the other failures), which I'm doing by cloning the git repo and then trying to work out which commit 1.9.3 was.  I'd like not to have to guess that last bit: more generally I'd like to be able, if I have something which claims to depend on vx.y of anything, to be able to reliably find vx.y, even if I have tor rebuild it myself.
(The code is a UK Met Office tool to convert climate model output to netCDF: it's BSD licensed.)

The tar.gz for 1.9.3 seems to still be on pypi even if it isn't listed on the website.  Just glancing at the archive it looks like the Changelog might have been one of the last files modified on 2013-01-12 at 10:45.

That doesn't give me much confidence but, well, never mind.


Story Conversion Fails If Twine Title Contains a Space or Its First Character is a Number

Publishing to File a Twine 2.0 story (Harlow) called 3 Bug Example or 3BugExample will result in a Story Conversion Fail. BugExample or Bug_Example works fine.

Did you get the following error message in the console? This is what you should have seen:
The story cannot be imported because "3BugExample" is not a valid Unity script name.
If you did, this is by design. I opted to cancel story conversion rather than automatically sanitize the name because it a script with a different name than the story could be confusing for users.


Jenayah: Watch targray.com

Jenayah requests the watch of the watch_keyword targray\.com. See the MS search here and the Stack Exchange search here.

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

Approved by Zoe in Charcoal HQ

Approved by tripleee in Charcoal HQ


FMbench support for different AWS Region

Would be good to have a CFT template that works in different AWS Region. Would be good to provide support in us-west-2

support for us-west-2 completed via the CFT template


我想自定义端口，安装的端口可能需要修改端口

我想自定义端口，安装的端口可能需要修改端口 6443我想更换端口

我想自定义端口，安装的端口可能需要修改端口 6443我想更换端口

The current version (Contains v0 8.0-beta and below v0.7.*) does not  support modifying 6443 port temporarily.

We will support them in the official version of 0.8.0.
This is our roadmap: https://github.com/cloudnativer/kube-install/blob/master/docs/roadmap.md

New version v0 8.0 has been released, and the new version supports custom 6443 port installation.
You can get the new version of the installation package here: https://github.com/cloudnativer/kube-install/releases/tag/v0.8.0


Minor fixes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.07%) to 89.616%

Files with Coverage Reduction
New Missed Lines
%

contracts/root/CheckpointManager.sol
1
96.63%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
907

Relevant Lines:
982

💛  - Coveralls


Fix chart version

This PR uses the current chart version instead of hardcoding the image.tag as 0.1.0. This current chart version is also used to checkout the appropriate upstream branch instead of checking out the branch corresponding to the current version in main.

Note: This PR fails 2 upstream e2e tests (http_ingress and upgrade_e2e) due to bugs in the tests, but these have been resolved in v0.8.0 of the upstream release.


⚠ SceneTime has degraded performance

In 23022e4, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1416 ms

Resolved: SceneTime performance has improved in a1c8dac after 6 minutes.


Use pgrep to probe sidekiq for liveness/readiness

Description of change
Use pgrep to probe sidekiq for liveness/readiness
Link to relevant ticket
We should really be testing if sidekiq is available for liveness and readiness, rather
than any root path available on the worker (which may just be the sidekiq webUI root path 🤷)

closing as we are using sidekiq_alive, which enables the use of httpGet over a specified port


Remove superficial Kafka optimization for the sake of clarity

The Kafka module is skipping the /etc/confluent/docker/ensure checks for the optimization reasons.
The checks performed in this script are super lightweight and don't worth the code complexity it brings.
Discussions
https://github.com/testcontainers/testcontainers-java/pull/7014#discussion_r1188806816

look like the example test and examples for kafka fail 👀

Have been looking at the failures...
Beyond just variable verification, the ensure script is doing a Zookeeper connectivity check...
https://github.com/confluentinc/kafka-images/blob/7.3.x/kafka/include/etc/confluent/docker/ensure#L23-L29
These 6 lines should be removed only when Kraft enabled && CP < 7.4
I need to find a way to update this file in a comprehensive way...

@eddumelendez Actually I think this PR is a bad idea.
The code would be less readable, if it's ok for you I propose to reject this PR.

feel free to close it


Does the joint name order follows the official SMPL-H, SMPL?

For SMPL-H following https://github.com/vchoutas/smplx/blob/master/smplx/joint_names.py
for SMPL following line 18 to 42?
for SMPL-H following line 18 to 72?
for MANO following line 43 to 72?

It's the same as the officially released version, but I don't remember the names exactly.

Thanks!


Adding Guideline for handling multiple values query parameter

Closes @223.
@fmueller @ePaul @whiskeysierra please have a look.

:+1:

@dehora Thanks for your comments, I've incorporated the proposed improvements.

@fmueller As far as I remember, we haven't decided which one to favor. multi feels more HTTP-ish, bit RFC standard doesn't define how this should be implemented. (Someone stated in the meeting, that this is part of the standard - but I could not find this in the RFCs)

As far as I know there is no RFC that states how query parameters are defined, at all. Correct me if I'm wrong.

Ok, just wanted to clarify this. :)
👍

👍

👍

The main point which speaks for multi is that it allows all kinds of strings, which csv is limited to strings without , in them (or you need to define some escaping in the description).
The main point for csv is that it allows for more elements before hitting some request line length limit.

:+1:

@ePaul Can you please merge the PR and delete the branch? (I have no permissions to do this)


[Wine] Error: UndefinedObject>>doIt: at line 1: illegal number while building image

It can't count
When I'm building an image (say, DPRO), I get the error mentioned in the title, directly before the message about Reloading DolphinMessageBox. It doesn't appear to prevent me building the remainder of the image.
Let's do it again
Steps to reproduce the behaviour:
Start Terminal under Linux. Check out git tree, add contents of relevant DolphinVM to tree. Run the following command from within the tree:
Dolphin$ wine ./Dolphin7.exe DBOOT.img7 DolphinProfessional
...

Output from above job on my computer looks roughly like this (wine messages in italics):

000b:fixme:winediag:start_process Wine Staging 4.8 is a testing version containing experimental patches.
000b:fixme:winediag:start_process Please mention your exact version when filing bug reports on winehq.org.
000d:err:menubuilder:init_xdg error looking up the desktop directory
000f:err:service:process_send_command receiving command result timed out
000f:fixme:service:scmdatabase_autostart_services Auto-start service L"MountMgr" failed to start: 1053
0009:fixme:msvcrt:_configure_wide_argv (1) stub
0009:fixme:msvcrt:_initialize_wide_environment stub
0009:fixme:heap:RtlCompactHeap (0x1160000, 0x0) stub
0009:fixme:ntdll:NtQueryTimerResolution (0x1435588,0x1435590,0x1435598), stub!
Dolphin Smalltalk Boot
Copyright (c) Object Arts Ltd, 1997-2018.
Loading boot script...
0009:err:ole:CoGetClassObject class {f27d15bd-b8eb-488c-9932-c7fa0c598640} not registered
0009:err:ole:CoGetClassObject no class object {f27d15bd-b8eb-488c-9932-c7fa0c598640} could be created for context 0x1
Reloading BCL constants pools...
Updating ClassBuilder...
Reloading BCL class definitions...
Reloading BCL classes...
Error: UndefinedObject>>doIt at line 1: illegal number
Reloading Dolphin Message Box ...
Loading source package 'Dolphin Message Box' from: Z:\home\viking\src\smalltalk\Dolphin\Core\Object Arts\Dolphin\Base\Dolphin Message Box.pax
Reloading Dolphin Source Fileout ...
Loading source package 'Dolphin Source Fileout' from: Z:\home\viking\src\smalltalk\Dolphin\Core\Object Arts\Dolphin\Base\Dolphin Source Fileout.pax
Deleting obsolete boot image methods...
Removing obsolete boot image method Context>>at:
Removing obsolete boot image method Context>>at:put:
Loading source package 'Dolphin Registry Access' from: Z:\home\viking\src\smalltalk\Dolphin\Core\Object Arts\Dolphin\Registry\Dolphin Registry Access.pax
Loading source package 'Dolphin Base64' from: Z:\home\viking\src\smalltalk\Dolphin\Core\Object Arts\Dolphin\System\Base64\Dolphin Base64.pax
Loading source package 'Dolphin Products' from: Z:\home\viking\src\smalltalk\Dolphin\Core\Object Arts\Dolphin\Installation Manager\Dolphin Products.pax
Booting DolphinProfessional
...

I'm also wondering if that's why I don't get a git version in the title bar of my DPRO images when generated under Linux, even though images generated under Windows run fine on both Linux and Vista.
Operating System and Wine versions:

OS: 64-bit Fedora 28
wine: wine-staging-4.8, WINEARCH=win32

Further to the initial report, I ran the following in both a Linux-generated and a Vista-generated image:
Linux:

SessionManager current productDetails
#('Dolphin Smalltalk Professional' 'Professional' 7.1 '' 0 'DPRO' nil nil)

Windows:

SessionManager current productDetails
#('Dolphin Smalltalk Professional' 'Professional' 7.1 '' 0 'DPRO' nil 'v7.1.8-15-g571a6ef8
')

Building the image on Wine was never an expected scenario, although running Dolphin at all on Wine was never an expected scenario originally so perhaps that point is moot. The reason to make it is just to emphasise that you are free to fix it, but you are largely on your own. I don't have the facilities or inclination to reproduce any of the Wine issues myself.
The symptoms suggest that the following operation in boot.st is failing, so you can try debugging through it to find out why:
SessionManager current stashBootInfo

This works by running an external git command, so some possible reasons are:

Related to the git version (unlikely)
Specific problem running the git command (like not being on the path at that point, which again seems unlikely if you are running this from the same terminal that you've pulled the repo).
General problem with launching external processes
Your local git repo doesn't have any tags for some reason. You might think this unlikely, but it is possible to clone without them, or to fail to push them if copying a repo into a fork. Easily verified by running git describe --tags from the command line, which will just fail if there are none.

My guess would be (3), but debugging through will be needed to find out.

Oh, one other thing that occurred to me. You might find a DBOOT.ERRORS or DPRO.ERRORS file with more details of the error, including a stack. Deleting these files, if present, and running the boot again will produce an error log with just the relevant error(s) as they are appended to in normal use and you may have a long history getting in the way.
The location of the error in the console output suggests it might not be related to the git version problem, but hopefully the error log file will throw more light on it.

As it turns out, I've found out how the issue happened. Even though Linux has git installed, Wine effectively tries to pretend there's a Windows commandline and path in place when it runs commands; as I never installed a windows version of git into the wine folder, it never gets run. Sorry to have wasted your time on this one. But someone else had the same symptoms as I did (git version never showed in their title), and it was for the same reason, which is why I remember it.
As for these wine bugs, I'll keep collecting them. I'm not too sure how I'll fix them but I'll try.

Seems I fired a little prematurely. I can get the git version to not show on the title bar of the System Folder under Windows, simply by removing or "hiding" the git command when building the image. However, that hasn't proven to be the reason for the Illegal Number error under wine. And as I can't get git working properly under wine, there's no way to prove or disprove that that's causing the issue.
Given that the image isn't even up yet when I'm building, I'm not sure how I'd find out how to cough up a "Illegal Number" consistently. Like you said, I'll have to walk through the SessionManager code in an already built image, or check out what the DBOOT.errors says. All I know is that it happens before MessageBox is even read in, but after the BCL class definitions and classes are reloaded. What does BCL stand for, anyhow?

What does BCL stand for, anyhow?

Base Class Library

I just ran a new Dolphin7.exe DBOOT.img7 DolphinProfessional, I got no .errors file. Where does this leave me?

Now I think about it, that is a compiler error - see the CompilerNotification class method initialize.
So, as a temporary change add a method like this to CompilerErrorNotification
defaultAction
	^SessionManager current logError: self

This may not be terribly informative since it seems to be evaluating an expression. The stack trace might give you some pointers as to the origin, but if not you may want to create copies of the _descriptionArguments and _descriptionFormat  methods from CompilerNotification and modify them to include the actual source code (which is available from the source method).

And I'm assuming at this stage that I modify the .cls directly, and not within the DPRO image?
I had another question about just how the DBOOT source can be browsed with an existing open image, but that can wait until another time.
(OT) I'm also making pretty good strides towards getting the  documentation sorted. I've already got a way to go from XML into HTML (using xsltproc), I'm just working with some of the other data files now. It's starting to shape up.

One thing you could try that might yield a result is to browse the *-compilation failures category in the booted image. To do this just open a class browser on any class, expand the * in the tree in the categories tree, and then right click on the compilation failures child node and choose Browse. The compilation failure could be in a static expression (i.e. one of those that begins ##(…) and should cause the containing method to fail to compile, in which case it should appear in the compilation failures category. Worth a try anyway.

I'm assuming at this stage that I modify the .cls directly, which I did.

Yes, in this case. It needs to be in chunk source format, which ought to be fairly easy to work out from the existing content. Sometimes you can put the chunk into the preboot.st file as a patch, but in this case it might get deleted by the code that removes methods that are in the boot image but not the sources files before the error is encountered.

I'm a bit lost as to your next suggestions regarding the two methods _descriptionArguments and _descriptionFormat. I'm new enough to be able to read them, but not to know what to do with them to get the results you're asking.

The first returns an array of arguments that will be passed to a printf-style format which is described by the string returned by the second. The formatter is not actually sprintf, but rather the Windows FormatMessage API. This uses a slightly odd format if you are used to C, but it does allow for positional parameters. Probably best to look at the existing definitions of _descriptionFormat.

The other question I've got is: how on earth do I generate a stack trace? About the best I've been able to do within the image so far is to insert "self halt." into a method to call up the debugger, what I haven't been able to do is to figure out how to get the values out to the outside world as well as the method names being called.

In normal circumstances every error that occurs will automatically cause a stack trace to be written to the error log. If the image crashes, or if you explicitly request it, then the VM will dump a more detailed log to the same file. If you just want to copy a stack to paste somewhere you can do this by pressing the Copy button in the walkback dialog

or by just selecting all the text and using normal Copy & Paste commands.

I had another question about just how the DBOOT source can be browsed with an existing open image, but that can wait until another time.

There is no specific tool for browsing unloaded source that presents it as a class browser might (creating one would be an interest project, and possibly not that difficult using the code modelling objects in the Refactory packages), but you can use the Chunk Browser to see them sequentially.

BTW I notice that the toolbar of the Chunk Browser does not work correctly in any image size mode other than 16x16 - another bug.

One thing you could try that might yield a result is to browse the *-compilation failures category in the booted image. ... Worth a try anyway.

Hm, no compilation failures in there. Ah well, was worth a try.

One more thing to add. If the value had been added where I think it should have been, it would have been somewhere near item 25917 in DBOOT.chg (Category: Other), when I view the changelogs using the Chunk Browser. This would all of course depend upon the DBOOT.chg not getting erased near the end of the process of building the image from Boot.st.

That is a denormalized value. Depending on the behaviour of the C-runtime library routine _atodbl_l and the CPU floating point exception mask setting, it might fail to compile. It's interesting that you can compile it normally later, and that suggests it is due to the default exception mask settings being different between the two environments. Once the image is up and running normally the exception mask will have been set to its default which will be allowing the use of denormals. On Windows IIRC the default is for all FP exceptions to be masked by the CRT on startup, but perhaps this isn't the case under Wine. It should get set by the call to Float>>onStartup even in a boot session, but its possible that something is amiss there.

I did a little investigation. So far I've found the smallest value to get accepted is 2.2250738585072012e-308, something seems ... suspiciously familiar about that figure. It's the same as FMin 2.2250738585072014e-308, within a ULP. No way it's a denormalised value. I'll do some investigation on the Linux platform, and find out how compliant it is with IEEE 754.

5.0e-324 is definitely the smallest representable denormalised value. Parsing smaller values is not of any real use as they'll get rounded anyway. Depending on how the parsing code is written, and the settings of the FP control flags, the point at which an error occurs (or indeed if it occurs at all) may differ. There are two sets of flags within the control word that are relevant here - the exception mask, and the denormal control flags. Depending on the former you may or may not get an exception for denormals, and depending on the latter the denormals might be retained or alternately flushed to zero. Actually maybe even the precision control setting might come into play, depending on how the parsing is written. It doesn't particularly make sense to try and reason about the behaviour of the number parsing without knowing the value to which the VM or image is setting the fp control word. In Dolphin this is a per process setting stored in the fpControl instance variable. It is typically inherited when a process is forked, but can be set to something else (with care, since some choices will cause repeating errors that will effectively kill the image if set permanently in the UI process). The VM switches the fp control word as it switches processes.
Some reasonable explanations for the behaviour you are seeing are:

the default FP control word setting established by the CRT is different, and that the boot image either has the wrong fpControl value stored down in the boot process, or it does not set this on startup (although it looks to me that it does). I think this fairly unlikely as I would expect there might also be issues on Windows since I seem to remember that the Dolphin default setting is different than the default CRT setting.
the implementation underlying the function in the CRT (_atodbl_l) that the Dolphin compiler is calling is itself changing the fp control word, or is unable to parse denormals for some other reason.
the implementation underlying _control87 is deliberately not letting certain exceptions be masked. Interesting the alternate _controlfp function in the Microsoft C runtime library (described in the documentation as "a platform-independent, portable version of _control87") does not allow the denormal exception mask to be altered.

Given that you are saying that the you are unable to evaluate an expression containing the string representation of the minimum denormal value in a workspace in a booted image, I would think the 2nd or 3rd reasons the more likely.
It may be that the only way around this would be to change the initialisation of Float.FminDenormalized to use a calculation rather than initialize it to a constant, e.g. (2 raisedToInteger: -1074) asFloat
Alternatively, just ignore it. For most purposes this is unimportant.

To investigate the fp control work setting you would add the following to the PreBoot.st file in your repo.
SessionManager current stdout nextPutAll: 'Current fp control mask is '; print: Processor activeProcess fpControl; cr!

I would expect this to say:
Current fp control mask is 524295

This is the default fp control word in a running image after booting - it masks exceptions for denormals, underflow, overflow, and inexact results, leaving zero divide and invalid operation exceptions unmasked. Assuming the value is the same, you could try ensuring that the fp control word has definitely been updates as follows:
Processor activeProcess resetFloatingPoint!

Or if the value is different:
Processor activeProcess fpControl: 16r80007; resetFloatingPoint

Be careful not to unmask the inexact result exception in a running image.

Yep, it does say 524295, so that much at least works. Where would I put "Processor activeProcess resetFloatingPoint"? In PreBoot? Or inside Boot.st just before I start reading in BCL? Or check that value again after I've read in BCL?


explorer - filter on type - by word

Issue Type: Feature Request
actually when I use explorer - filter on type, the filter use each character I type as a distinct part of the search.  Is there a option to search by word.
sample :
I have filesnamed

GET.FIELD
MAKE.GED
if I filter on type : GED , the two files are show ... I'll restrict only MAKE.GED where the typed word is complete.

VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:35.169Z)
OS version: Windows_NT x64 10.0.18362

This is currently not possible, but I guess a feature request against our list / tree filter, thus forwarding to @joaomoreno

#66913


Fix issue where distances between nodes in tar_visnetwork() are enormous

Something is throwing off the distances in the network graph being produced by tar_visnetwork(). Not sure what the root cause is yet.

This has somehow resolved itself

This has somehow resolved itself


Save the same info from the current label into firebase

Record the current event label information  into firebase

implemented in pr #73

Would expect a better structure of JSON stored in firebase, the formate should:
1 Have some fields for filtering the data, because exporting the whole data from firebase is too costive in terms of data size as well as exporting time.
2 Easy to transform into the CSV or XLS file that required in data analytics and data visualization tools for instance: Python, R, and Tableau
3 @Xu221b @niharikaavasthi  Can you have a study on the job of using JSON data in the analyzing stage? What JSON structure is recommended?

My thoughts is like:
{
date : {
user: userid
os:
browser:
role:
gender:
age:
thread_time:
},
date: {
.....
},
.......
}

I think it is tricky to work on JSON data in general as compared to do it using Python or R but the above structure will be the best way to do it. Correct me if I am wrong.

implemented in PR #91


*: make case test_query_stats a little bit more stable

What is changed and how it works?
Issue Number: Ref #12562
What's Changed:
so does test_merge_election_and_restart.

Check List
Tests

Unit test
Integration test

Release note
None

/test


Cannot install the extension

Describe the bug
Cannot install the extension
To Reproduce
Steps to reproduce the behavior:

Open "Ulauncher Preferences"
Enter the "Extensions" tab
Select the option "Add extension"
Paste the repository link (https://github.com/tchar/ulauncher-albert-calculate-anything)

Expected behavior
Extension installed
Logs
Message: undefined
Error Name: undefined
Type: undefined
Stacktrace:
undefined

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information)

OS: [Ubuntu Mate 20.04 ]
Linux: Ubuntu 20.04.2 LTS

Additional context
Add any other context about the problem here.

@jrafaaael Have you installed the dependencies?
Pint==0.17
simpleeval==0.9.10
parsedatetime==2.6
pytz==2021.1

Also this problems really looks like a Ulauncher problem because I have similar problems myself. Right now I can reproduce your issue and for some reason if I restart Ulauncher the extension works properly. Consider doing the following and report back please

Run pip install Pint simpleeval parsedatetime pytz
Restart Ulauncher and see if the extension works
If step 2 fails try downloading the extension manually with the following

Warning this method should disable future updates, just a check if it is ulauncher's problem or not. If this works, then remove the folder and try installing through ulauncher
rm -r ~/.local/share/ulauncher/extensions/com.github.tchar.ulauncher-albert-calculate-anything/
git clone https://github.com/tchar/ulauncher-albert-calculate-anything ~/.local/share/ulauncher/extensions/com.github.tchar.ulauncher-albert-calculate-anything

After confirming it works
rm -r ~/.local/share/ulauncher/extensions/com.github.tchar.ulauncher-albert-calculate-anything/
git clone https://github.com/tchar/ulauncher-albert-calculate-anything

And install through Ulauncher, restart launcher if necessary.
I strongly believe this is a Ulauncher issue
Here is a video showing the same problem as yours but after restarting launcher, the extension is there and working
https://user-images.githubusercontent.com/6424426/129417819-92575a92-877b-4448-aca1-b1bf4a8c9d35.mp4
Here is a similar issue where Ulauncher says the extension has crashed but in reality it is working.
https://user-images.githubusercontent.com/6424426/129417837-b4545082-eb58-4afa-a23c-18c761a3b304.mp4


Make hex-edit cursor movement configurable

Esc is wrongly marked as a conflict (collides with the same shortcut in 3D view), we should probably add some classes to the shortcuts and check conflicts only within them.

Esc is wrongly marked as a conflict (collides with the same shortcut in 3D view), we should probably add some classes to the shortcuts and check conflicts only within them.

Yeah it is wrongly marked - the problem is that it is not so easy to define such classes, and define with which other classes they conflict - it also depends a lot on how exactly our components bind actions to know whether they will conflict, or not (i.e. is the action bound to only one widget in dock, or to whole dock). Comparing with some IDE that I use:

Qt Creator does the same thing as we do
PyCharm doesn't mark conflicting shortcuts, but when adding new shortcut it does global conflict search
Visual Studio - doesn't mark conflicting shortcuts, it has classes for shortcuts, but it still does global conflict search

Ok, let's leave it as is, it's not too important.


fasta output

out.fasta is created even if --master is not specified on the command line (contrary to the help message, and without the intervening bases, of course).
Also, what is the difference between upper & lower case letters in out.fasta?

The differences should be due to your reference containing lowercase letters, we don't do any processing to the reference other than swapping out variants.


Sleep Problem

I noticed an abnormal behavior of the plugin for my fans.
Let me explain, if my fan is off and I activate the day before, it activates and deactivates immediately. Except that when I then turn on the fan the standby comes back on again. This is unpleasant because I have a rule that activates the standby of my son's fan for the summer and except that when the day he activates the fan and well it turns off on its own because the timer has remained in memory. It should be stable.

Can you share what fan model are you using? I guess since you asked about micloud then you are using a micloud based fan. Unfortunately this might happen, depending on what moment you turn on the switch, this is due to the fact that the MiCloud has a slight delay which causes this. There is nothin really what can be done about it...

En réalité, je n'utilise pas Mi Cloud. Donc je travaille directement sur le réseau LAN.
Si mon ventilateur Xiaomi 1c est éteint et que j'active la minuterie de veille cette-ci s'active et se désactive de suite. Mais si par la suite j'allume le ventilateur, alors là minuterie de veille précédemment régler se réactive tout seul comme si le plugin l'avait gardé en mémoire.
Actually, I don't use Mi Cloud.  So I work directly on the LAN network.
If my Xiaomi 1c fan is off and I activate the sleep timer, it activates and deactivates immediately.  But if I turn on the fan afterwards, then the previously set sleep timer reactivates on its own as if the plugin had kept it in memory.

Then you would need to post a homebridge debug log so I can have a look at what is happening. Please start homebridge on debug mode and observers the console, afterwards press the button to replicate the issue and post the log.