Update coffee extension

Description
Fixed #9239 #11463 #11559 #11625 #12120

Replaced the form in the Caffeinate For command with arguments in the root search.
Replaced the emojis in the Caffeinate Status command with Unicode characters for enhanced design consistency.
Updated some menu bar icons.
Fixed an issue where it was not possible to decaffeinate the Mac via the Caffeinate Status Menu Bar.
Fixed an issue where the menu bar icon was not immediately updated when the Mac was caffeinated.
Fixed the Caffeinate While command not working.

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Nice work ! LGTM

The only think I can come up with is that users who have selected an icon for the menubar which is now deleted might end up with an empty icon

Are you sure? In my test the default icon is set when the value of the selected icon is changed in package.json.

The only think I can come up with is that users who have selected an icon for the menubar which is now deleted might end up with an empty icon

@pernielsentikaer Are you sure? In my test the default icon is set when the value of the selected icon is changed in package.json.

Should we keep a mapped array for the old names so users don't suddenly see a new icon? Could that work in any way?

Should we keep a mapped array for the old names so users don't suddenly see a new icon? Could that work in any way?

I don't think it's a necessity since the user won't be left without an icon. Additionally, users who are no longer using the default icon have already gone through the experience of having to change it once and I believe it won't be a big trouble to do it again once they notice their icon has changed. That said, I believe the small benefit of avoiding the icon change to the default does not outweigh the inconvenience of adding complexity to the code that may not be immediately clear to contributors.

@xilopaint Love the extension, but the user experience is suffering with this update:

"Replaced the form in the Caffeinate For command with arguments in the root search."

This is just a "me" use case, but I configured a keyboard shortcut (hyperkey+7) to the Caffeinate For. This was possible only because of the form input. Now with the arguments having to be passed through the root search, its not possible.

I also think the new approach is better, but I can see the issue since the parameters in the command are optional. One thing we could do is add some preferences that set the default caffeinate time, so it will use the default values if nothing is specified.
We could also add another command (which is disabled by default) to bring back the form view. Could that work? 🙂

both are great solutions. I personally would enjoy the later of the two having the option to bring back the form. I definitely appreciate it.

I don't like the idea of bringing back the form. The problem here is that all the arguments are individually optional but filling in at least one of them is required.
@pernielsentikaer, would it be possible to work with the team on this type of validation (at least one argument must be filled) so that the hotkey brings the focus to the first argument instead of trying to run the command?

I can't promise you anything, but yes, I'll try to bring it up with the team if we can figure out a way to achieve this without changing too much. This is also a niche case but I agree with @edheltzel that it doesn't work very well at the moment.
I'll get back to you 🙂

@xilopaint, it will stay as is at the moment. We added it to the backlog and will see if it pops up more regularly 🙂
We could do one of the following:

Idea 1) If no fields are filled, we show the form
Idea 2) Just have one argument where you can write something like 1h 30m or 1:30 instead

Do you have any input here, @edheltzel

Idea 1 is my preferred... And seems to be the least amount of work to achieve.
Other than that I do appreciate everything.


Externalized mesh resource keeps getting embedded in an inherited scene

Godot version:
master / 7be49849
OS/device including version:
Manjaro Linux 18.0.4
Issue description:
I have a model exported with Godot Blender Exporter(.escn) with the import setting Mesh > Storage set to Files to save mesh data to *.mesh resources.
It works as intended at first, but when I restart Godot and open an inherited scene created from the source .escn, the contained MeshInstance's Mesh property loses its reference to the externalized .mesh resource, but reverts back to the state in the source .escn, which embeds the whole mesh data in the scene itself.
The .escn in question is over 150mb in my case, and when this happens, the inherited scene (.tscn) becomes as large as its source, so I have to keep resetting those properties whenever I open the problematic scene.

Can anyone still reproduce this bug in Godot 3.2.3 rc3 or any later release?
If yes, please ensure that an up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.

No answer, so closing. Please comment if you can still reproduce it, and make sure that an up-to-date reproduction project is included.


HTML formatting wraps ending block tag for no reason with nested inline elements

js-beautify leaves the following single HTML5  element untouched, as it should:
foo

If another inline element such as  is nested, it is expected that js-beautify should continue to leave the HTML touched, because nothing about the inline elements should cause newlines to be added:
foo bar

However js-beautify incorrectly adds a newline before the  end-tag:
foo bar

There is no reason for such behavior; it is a bug. It can be verified merely by pasting and formatting in the main input field at http://jsbeautifier.org/ .
(Note that syntax highlighter on that page can't correctly show formatting colors for this simple sequence of elements, but that likely is a separate issue and may not relate to this bug.)

The syntax highlight is provided by another product.

This is the double span issue mentioned elsewhere. It's not actually spans fault. It's caused by unformatted tags not identifying nested tags of the same name. The outer  considers the first  to be it's closing tag. You can duplicate the problem by replacing span with output for instance.

@HookyQR - Oh.  Yeah. Good catch, thanks. That'll help.


fix(label): add disabled class when form-check-inline and disabled

Adds the disabled class when the label has both the inline and
check props. This is the way bootstrap disables inline radio
and checkboxes since there is no wrapping element.
I must have missed this use-case before when creating the forms and labels. The whole form-check classes are new to bootstrap 4.
Note: there is currently an open issue with bootstrap (twbs/bootstrap#20799) to correct the color of the disabled label's text (and to clarify how to denote disabled inline radio/checkboxes in the docs as it is not there, I had to infer it from the styles)

Nice work getting that other issue into the alpha v4 release. lgtm!

published in 3.3.1 💫


'id' property reported as unresolved

In Terraform config:
resource atlas_artifact a1 {
  name = "${atlas_artifact.a1.id}"
}

id highlighted as unresolved reference. Actually id is predefined one in all resources.

Workaround: disable 'Unresolved reference' inspection in 'Terraform' group

https://github.com/hashicorp/terraform/blob/master/helper/schema/resource_data.go#L204


StreamID inspect error

Hi! Excited to use this tool, but I can't get the stream-id-inspect command to work as I get this error with any streamID I can find on cerscan:
❯ cpk stream-id-inspect --id kjzl6cwe1jw147k3bqorj08cdpyoxswwpy4xhlux65y0z5q9gw5v27uqv9fivym
Error: unknown variant: 0

❯ cpk stream-id-inspect --id k6zn3rc3v8qin0ybzouzaqu4p560i90fuszcmv9ize9f57lxelm8us01t2louc4nm3r0a49h5ni7ft8hwh4186rdz4ekb5j5dx65n423a1w1on4ri1y10yy
Error: unknown variant: 0

Am I missing something critical, or is there a bug spooking? :ghost:

Thanks for  reporting, there is likely a bug. I'll take a look at this today.

@nathanielc splendid, thanks for the fix :pray:


HLS Fragment Size is 8 Seconds regardless of hls_fragment setting

Compiled using nginx-1.13.8
and nginx-rtmp-module-1.2.1
My specific configure command:
 ./configure --with-cc-opt='-g -O2 -fPIE -fstack-protector-strong -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2' --with-ld-opt='-Wl,-Bsymbolic-functions -fPIE -pie -Wl,-z,relro -Wl,-z,now' --prefix=/opt/rtmp --add-module=../nginx-rtmp-module-1.2.1 --http-log-path=/var/log/rtmp/access.log --error-log-path=/var/log/rtmp/error.log --sbin-path=/opt/rtmp/sbin/rtmp

nginx config
rtmp {
  server {
    listen 1935;
    application live {
      live on;
      hls on;
      hls_path /var/www/hls;
      hls_playlist_length 30s;
      hls_fragment 5s;
    }
  }
}

#EXT-X-TARGETDURATION  is 8 in the playlist and the individual .ts files are all 8 seconds long despite an hls_fragment size of 5 seconds.
Am I missing something?

my .ts files are all 11 seconds.
how did u solve  it?


Validate schedules

This PR requires validation of schedule prior to setting it up. It is a hacky work around of NMI's system for creating a schedule and validating a card.
Currently NMI does not let you validate + create a schedule even though it seems like it should be possible thanks to this
However, using <validate><add-subscription></add-subscription></validate> returns an error. Instead, we make a hidden 3 step transaction happen that is just a validate during the final checkout process of a schedule when a user doesn't have an account.
Technically this will make it possible to save a payment during a schedule too, but more work will need to be done on the Rock side since a saved payment requires a transaction and we don't want to create a $0 transaction to store this.

This PR closes #439


DNM: Testing new doc build

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/2009

recheck

recheck

recheck

recheck

recheck


Convert Webpack config to TypeScript

As per comment https://github.com/sindresorhus/refined-github/pull/1962#issuecomment-486749235

For reference in regards to TypeScript types for the size-plugin package, https://github.com/GoogleChromeLabs/size-plugin/issues/20


The IDE cannot copy the mongod.exe to the output dir

Hi,
I'm facing an issue that might be related to the following behavior mentioned in the issue: "Windows Defender Firewall" want to block communication of testhost.exe

EphemeralMongo also uses .NET Process class to start mongod.exe from your project output directory.

If I run the integration test end to end for the first time, it works perfectly. But if force the execution to end in the middle of the test (Shift + F5), the mongod.exe is left running. Then if I make any change in the test code and try to run the test again, the IDE can't copy the mongod.exe file to the output directory, so I can't run the tests.

This is happening because the mongod process is still running.

Hi @marcosdeassis, I don't think it is related to Windows Defender Firewall, it's really just that the process is still running and at compilation time MSBuild tries to overwrite the file.
It might be because I used CopyToOutputDirectory = Always instead of PreserveNewest. I'll make some tests later to see if this behavior is the root cause of your issue.
In the meantime, I just released 1.1.0 which contains an experimental flag MongoRunnerOptions.KillMongoProcessesWhenCurrentProcessExits that makes sure any mongo process gets killed if the running process (in your case, the IDE test host) is prematurely killed. It only works on Windows and modern .NET (.NET Core and later).
I think it should solve your issue. The pull request behind this experimental flag is here.

Hi @asimmon
Yes, you are correct.
The experimental flag is working perfectly so far.
Thank you for your time in implementing the feature.


Config to match vscode import sorting default

What would a config look like that just mimics the default import order VS Code creates on the Organize Imports action?

No idea. How does it order things?

I didn't find a good documentation on that
Howevery, through experimenting I found this one works for a nx monorepo where the nx projects are named "abc"
"import/order": [
        "error",
        {
        "alphabetize": {
            "order": "asc",
            "caseInsensitive": true
        },
        "groups": [["builtin", "external"], "parent", "sibling", "index"],
        "pathGroups": [
            {
            "pattern": "@abc/**",
            "group": "builtin",
            "position": "before"
            }
        ],
        "newlines-between": "never"
        }
    ]

note that "newlines-between": "never" is not necessary, that's just my personal preference


Webinstall - SMTP port is not set

Bug report
Issue Description
When you set the SMTP port to 25 the port is still defined to 2525
Preconditions
Please provide as detailed information about your environment as possible.
1. master branch

Steps to reproduce
It is important to provide a set of clear steps to reproduce this bug.If relevant please include code samples.
1. Setup a new Bagisto installation and use the web installer.
2. Set the SMTP port to a custom number.

Expected result
The correct port should be defined.

[Screenshots, logs or description]

Actual result
The port is still defined to 2525 as this is set in .env_example

fixed in master.


Remove dependency on foundation from foundation-demo

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[x] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@damienwebdev @lderrickable This is just initial changes to the design package. I will fix the rest of the broken styles in separate PRs.

Let's keep them all in this PR.

@damienwebdev @lderrickable This is just initial changes to the design package. I will fix the rest of the broken styles in separate PRs.

Let's keep them all in this PR.

Ready for review. Cart page styles is taken care of in #103 and I want to wait for #97 to be merged before making any changes to checkout styles.

If you look at the product page at smaller than 450px, the inner elements stop resizing and they overflow off the page

If you look at the product page at smaller than 450px, the inner elements stop resizing and they overflow off the page

@lderrickable fixed, ready for review


Unrestricted character selection

I'm also postponing this because a proper implementation would require a full rewrite of CharacterSelect3D and SceneMenu. A flawed proof of concept is available at https://github.com/stblr/ddd/tree/unrestricted-character-selection.


Spinning wheel of death - uncaught type error

I'm getting a spinning wheel of death trying to load a PWA. Chrome says:
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'toLowerCase')
at theme.BiABRSbW.js:4769:22022
at _ (entry.ArJezRqP.js:6:29330)
at entry.ArJezRqP.js:6:29445
at Object.h [as subscribe] (entry.ArJezRqP.js:6:28975)
at j (scheduler.C8SBejcr.js:1:983)
at _t (scheduler.C8SBejcr.js:1:1126)
at kn (0.VoG3hY0s.js:1:63039)
at Q (index.4FAxb_EP.js:1:1502)
at new Nn (0.VoG3hY0s.js:1:63793)
at zt (scheduler.C8SBejcr.js:1:7086)
Examples: https://eBible.org/hatbsa/pwa/
https://eBible.org/eng-web/pwa/
https://ebible.org/engwebu/pwa/
https://ebible.org/spapddpt/pwa/
etc.

@kahunapule Is this still a problem?


Update Weights Operation

Hi @eric-mitchell,
Thank you for the repo and the great paper. I was looking at your code and I have a doubt on this line of code: why is there a plus? Shouldn't be a minus?
As you also wrote in the paper, the final edited weights should be:
$$\tilde{W} = W_l − α * \tilde{\nabla} W_l$$
Thanks for your help.

You're right about this- good catch! Turns out this is indeed a bug in the implementation. Have you been able to train a model with this corrected, and seen if there is a difference in performance?

Actually, I've never run your code, but I adapted it for another use case. Anyway, the code works also with the plus. The optimization process makes the network flip all gradient signs. However, I expect slightly better results with the right sign from the beginning, simply because the task should be a little simpler from the point of view of hypernetworks.


Update in.m3u

desibox updates

Link is broken.

Sever restarting link will be up in 10 mins
On Tue, May 7, 2019, 18:24 Aleksandr Statciuk<EMAIL_ADDRESS>wrote:

Closed #17 https://github.com/freearhey/iptv/pull/17.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freearhey/iptv/pull/17#event-2324095575, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AJV4QWXM52BXGQOCMMSZMRTPUF3W7ANCNFSM4HLIGB2A
.

https://edge.hls.hitbox.tv/hls/desijattuk/index.m3u8?st=nuYxAHesRw4B-jHQap9LVQ;ci=mXwDmlmieHsjswZTZpoFyA
add this for second link for jatt tv please

https://live600.vkuserlive.com/501706/live/556893745/82VENUSW5tRjRUIjhyPzQ5OTU5Nj8y/playlist.m3u8?extra=lO1F6M4PwjmB6ZtQkVujGrIv3dcVD_Nr42iOT1C5SmV_99WkZSXq2b0-pAyrHffJ8plgr32sW1qXfCsQLdF6T3Xa4yfOL5znC5OY6wLcR0_68Vaj3Haobg_LxqsvHblBwyuLviB2EM1tntGft-ZAdjR6


Loading an existing query not working correctly

Your demo for existing query seems to be correct but if you try and load an existing query that has select dropdown , or uses any operator other than '=' is gets messed up.
Example if i set a query like this:

The query looks like this:
(name = ' Fred' AND offices IN 'Oakland')
But if I try and Load that query in
config = { query: "(name = ' Fred' AND offices IN 'Oakland')" }

it builds the query UI messed up

Offices isn't selected and its operator is incorrect and the N from IN operator is added to value and value is cut off?Also noticed extra space it added...
Any insight?
Does that token creator function need some refactoring maybe

Thank you for detailed description, working on fix.

As a temporary solution it's needed to use lowercase operators is null, is not null, in, not in instead of IS NULL, IS NOT NULL, IN, NOT IN. Would be refactored a bit later


Linters failing

yarn run v1.22.21
$ yarn check-types && eslint .
$ tsc --noemit
Error: node_modules/@cyntler/react-doc-viewer/dist/esm/index.d.ts(6,1[5](https://github.com/XeniaEngineering/webapp/actions/runs/7005233732/job/19054565543?pr=2317#step:5:6)): error TS1005: ',' expected.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Process completed with exit code 2.```

I am using `"@cyntler/react-doc-viewer": "^1.14.1"`

@usamabinnadeem-10 You shouldn't use TS compiler with node_modules packages. You have to exclude node_modules directory (should be working by default tsconfig.json).


Certificate request not working in 2012R2 as it did in 2016

Details of the scenario you tried and the problem that is occurring
Verbose logs showing the problem
VERBOSE: Perform operation 'Invoke CimMethod' with following parameters, ''methodName' =
SendConfigurationApply,'className' = MSFT_DSCLocalConfigurationManager,'namespaceName' =
root/Microsoft/Windows/DesiredStateConfiguration'.
VERBOSE: An LCM method call arrived from computer YTHJK-ES-1 with user sid
S-1-5-21-4238065118-3723846887-562607489-500.
VERBOSE: [YTHJK-ES-1]: LCM:  [ Start  Set      ]
VERBOSE: [YTHJK-ES-1]: LCM:  [ Start  Resource ]  [[CertReq]SSLCert]
VERBOSE: [YTHJK-ES-1]: LCM:  [ Start  Test     ]  [[CertReq]SSLCert]
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Test-TargetResource: Testing Certificate with
Subject 'CN=YTHJK-ES-1.bvt.local' issued by YTHJK-DC-1.bvt.local\bvt-YTHUK-DC-1-CA.
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Test-TargetResource: No valid certificate found
with subject 'CN=YTHJK-ES-1.bvt.local' issued by YTHJK-DC-1.bvt.local\bvt-YTHUK-DC-1-CA.
VERBOSE: [YTHJK-ES-1]: LCM:  [ End    Test     ]  [[CertReq]SSLCert]  in 0.0310 seconds.
VERBOSE: [YTHJK-ES-1]: LCM:  [ Start  Set      ]  [[CertReq]SSLCert]
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Set-TargetResource: Starting Certificate request
with Subject 'YTHJK-ES-1.bvt.local' issued by YTHJK-DC-1.bvt.local\bvt-YTHUK-DC-1-CA.
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Set-TargetResource: Creating certificate request
'C:\Windows\TEMP\xCertReq-5578fa69-3dd3-4b99-be3c-d79a0169bab7.req' from
'C:\Windows\TEMP\xCertReq-5578fa69-3dd3-4b99-be3c-d79a0169bab7.inf'.
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Set-TargetResource: Create certificate request
result: Active Directory Enrollment Policy
{AE50C422-1B3C-4F8C-A9EE-5DD749ED747D}
ldap:
CertReq: Request Created
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Set-TargetResource: Submitting certificate request
'C:\Windows\TEMP\xCertReq-5578fa69-3dd3-4b99-be3c-d79a0169bab7.req' returning
'C:\Windows\TEMP\xCertReq-5578fa69-3dd3-4b99-be3c-d79a0169bab7.cer' issued by YTHJK-DC-1.bvt.local\bvt-YTHUK-DC-1-CA.
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Set-TargetResource: Submitting certificate request
using separate process.
VERBOSE: [YTHJK-ES-1]:                            [[CertReq]SSLCert] Set-TargetResource: Submitting certificate request
result: Certificate Request Processor: The parameter is incorrect. 0x80070057 (WIN32: 87 ERROR_INVALID_PARAMETER)
VERBOSE: [YTHJK-ES-1]: LCM:  [ End    Set      ]  [[CertReq]SSLCert]  in 5.7660 seconds.
PowerShell DSC resource MSFT_CertReq  failed to execute Set-TargetResource functionality with error message:
System.InvalidOperationException: Certificate file 'C:\Windows\TEMP\xCertReq-5578fa69-3dd3-4b99-be3c-d79a0169bab7.cer'
not found.
+ CategoryInfo          : InvalidOperation: (:) [], CimException
+ FullyQualifiedErrorId : ProviderOperationExecutionFailure
+ PSComputerName        : localhost
VERBOSE: [YTHJK-ES-1]: LCM:  [ End    Set      ]
The SendConfigurationApply function did not succeed.
+ CategoryInfo          : NotSpecified: (root/Microsoft/...gurationManager:String) [], CimException
+ FullyQualifiedErrorId : MI RESULT 1
+ PSComputerName        : localhost
VERBOSE: Operation 'Invoke CimMethod' complete.
VERBOSE: Time taken for configuration job to complete is 5.97 seconds
Suggested solution to the issue
The DSC configuration that is used to reproduce the issue (as detailed as possible)
# insert configuration here
configuration CertRequest
{
    param
    (
        [Parameter()]
        [System.String[]]
        $NodeName = 'localhost',

        [Parameter()][ValidateNotNullorEmpty()]
        [string]$CARootName,

        [Parameter()][ValidateNotNullorEmpty()]
        [string]$CAServerFQDN,

        [Parameter()][ValidateNotNullorEmpty()]
        [string]$Subject,

        [Parameter(Mandatory = $true)]
        [ValidateNotNullorEmpty()]
        [System.Management.Automation.PSCredential]
        $Credential
    )

    Import-DscResource -ModuleName CertificateDsc

    Node $AllNodes.NodeName
    {
        CertReq SSLCert
        {
            CARootName          = $CARootName
            CAServerFQDN        = $CAServerFQDN
            Subject             = $Subject
            KeyLength           = '2048'
            Exportable          = $true
            ProviderName        = '"Microsoft RSA SChannel Cryptographic Provider"'
            OID                 = '<IP_ADDRESS>.<IP_ADDRESS>.1'
            KeyUsage            = '0xa0'
            CertificateTemplate = 'WebServer'
            AutoRenew           = $true
            FriendlyName        = 'Citrix SSL Cert for CVAD'
            Credential          = $Credential
        }
    }
}

#### The operating system the target node is running
<!--
    Please provide as much as possible about the target node, for example
    edition, version, build and language.
    On OS with WMF 5.1 the following command can help get this information.

    Get-ComputerInfo -Property @(
        'OsName',
        'OsOperatingSystemSKU',
        'OSArchitecture',
        'WindowsVersion',
        'WindowsBuildLabEx',
        'OsLanguage',
        'OsMuiLanguages')
OsName               : Microsoft Windows Server 2012 R2 Standard
OsOperatingSystemSKU : StandardServerEdition
OsArchitecture       : 64-bit
WindowsBuildLabEx    : 9600.19153.amd64fre.winblue_ltsb.180908-0600
OsLanguage           : en-US
OsMuiLanguages       : {en-US}
-->

#### Version and build of PowerShell the target node is running
<!--
    To help with this information, please run this command:
    $PSVersionTable
-->
Name                           Value
----                           -----
PSVersion                      5.1.14409.1005
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1005
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

#### Version of the DSC module that was used ('dev' if using current dev branch)
Current Stable version, as far as I know.  Just used the install module command after updating powershell.

I confess I'm a bit new at this.  I probably have not included all logs that you need.
I will check back accordingly.

Hi @jthompson333 - thanks for raising this. Just to make sure I'm on the right track, are you saying the config works OK on WS 2016, but doesn't work on WS 2012R2?
Does the WebServer certificate template exists on the CA and the $Credential contains a valid Domain account that has access to issue using the WebServer template? Usually we get this sort of error when there is a problem issuing the certificate using the generated .req file which is more often than not an access issue.
If you

Yes, to confirm it did work on WS 2016 but not on WS 2012R2
Let me check on those items and I will get back to you.
The domain was built with automation and the CA was setup with the sister powershell dsc library AdcsCertificationAuthority.
I'm also on vacation, so I may not get back to responding on this until Tuesday.  Sorry, I guess I should have waited a little longer to report.  Thanks for the feedback.  I won't forget about it...

If you can get some details on the template you've got setup in your WS2012R2 machine then I can try and replicate it here on my lab.

By default it seems the CA install doesn't set all the permissions for Domain Admins...

However, its the same on 2016 as well and the request worked.

Params for both CA's were installed with these options from the ActiveDirectoryCSDsc module.
So the default WebServer template was used with this install
Configuration SetupAndInstallCA
{
param
(
[Parameter()]
[System.String[]]
$NodeName = 'localhost',
    [Parameter(Mandatory = $true)]
    [ValidateNotNullorEmpty()]
    [System.Management.Automation.PSCredential]
    $Credential
)

#Import-DscResource -ModuleName PSDesiredStateConfiguration
Import-DscResource -Module ActiveDirectoryCSDsc

Node $AllNodes.NodeName
{
    WindowsFeature ADCS-Cert-Authority
    {
        Ensure = 'Present'
        Name   = 'ADCS-Cert-Authority'
    }

    AdcsCertificationAuthority CertificateAuthority
    {
        IsSingleInstance = 'Yes'
        Ensure           = 'Present'
        Credential       = $Credential
        CAType           = 'EnterpriseRootCA'
        DependsOn        = '[WindowsFeature]ADCS-Cert-Authority'
    }
}

}

From 2012 R2 DC
PS C:\Users\Administrator> Get-CATemplate -Verbose
VERBOSE: Retrieving local certification authority configuration.
VERBOSE: Preparing the "YTHJK-DC-1.bvt.local\bvt-YTHJK-DC-1-CA" certification authority configuration.
VERBOSE: Retrieving templates for certification authority.

Name                           Oid
----                           ---
DirectoryEmailReplication      <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
DomainControllerAuthentication <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
KerberosAuthentication         <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
EFSRecovery                    <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
EFS                            <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
DomainController               <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
WebServer                      <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
Machine                        <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
User                           <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
SubCA                          <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>
Administrator                  <IP_ADDRESS>.4.1.3<IP_ADDRESS>89841.3442508.1261732.14491001.116904<IP_ADDRESS>

From 2016 DC
PS C:\users\public> Get-CATemplate -Verbose
VERBOSE: Retrieving local certification authority configuration.
VERBOSE: Preparing the "testdc1.testcerts.io\testcerts-TESTDC1-CA" certification authority configuration.
VERBOSE: Retrieving templates for certification authority.

Name                           Oid
----                           ---
DirectoryEmailReplication      <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
DomainControllerAuthentication <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
KerberosAuthentication         <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
EFSRecovery                    <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
EFS                            <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
DomainController               <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
WebServer                      <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
Machine                        <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
User                           <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
SubCA                          <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>
Administrator                  <IP_ADDRESS>.4.1.3<IP_ADDRESS>38339.9772982.13911702.4650013.24695<IP_ADDRESS>

2012 DC Info (CA Templates)

2016 DC Info (CA Templates)

Not sure if thats the info that you were looking for, but, let me know if its not.

Great info @jthompson333 - can you see who has Enroll permissions on the Security tab on these templates? You need to ensure that the $Credential you're passing to your config has access on the Security tag as an Enroll.
E.g.

Other ways you can check this is log onto the server you're applying the DSC config to using the account specified in $Credential and try issuing the certificate using certreq.exe (or even CertMgr.msc) for the WebServer Template. If you can do this then it probably isn't a security issue as I thought and we can track it down from there.
Does that make sense?

ok.  I haven't tried certreq.exe yet.
So domain admins- the creds I'm passing, just the default domain admin account has enroll permissions, but, the authenticated users does not.  2016 DC looks the same as well.

I'll try certreq.exe or the gui and see what happens.

So the project I'm working on- I was just told not to waste time with 2012 R2.  So I suppose you can archive this off, as I won't have anymore time to mess with it.  Never got to try certreq.exe...

I had a similar error when when the "Domain account" I passed in "dsc_credential"  wasn't a local administrator on the machine where I requested it created the request from.

Thanks @wh33ly - I am fairly sure this issue is related to an access issue as you surmise. We probably could do some more work on detecting and reporting likely causes back to the user, which is why I've left this open for now.


Enforce program counter in SuperNova StepCircuit.

This PR formalizes the requirement that the new program counter returned from circuit_supernova::StepCircuit::synthesize() is the one assigned to the circuit that is actually synthesized at each step.
The previous TestROM example actually did not follow this convention but rather used the program counter for its own purposes, as an index into a predetermined sequence of operations — then enforced the correctness of this ad-hoc indexing scheme in each circuit.
The current work enforces the constraint that pc has been correctly calculated and returned from each circuit, requiring that it is interpreted precisely as that circuit's assigned index in the RunningClaims.
In order to make the existing tests pass, they were reworked to re-envision the pseudo-pc enforcement at the start of each circuit — to a calculation (morally $\varphi$) of the new pc to be returned at the 'end' of the circuit. Without this change, the existing tests fail, exactly as expected, since pc was a monotonically increasing value, rather than the index of the chosen next circuit.
The circuit necessarily grows by one constraint.
Since the goal is that for the SuperNova user (the application/protocol/proof developer), the enforcement is performed by a new, internal trait circuit_supernova::EnforcingStepCircuit — which is automatically implemented for types that implement circuit_supernova::StepCircuit.

LGTM! it's nice to formalize and align terminology of pc with Supernova paper, so it will be friendly for audience came from paper reading. Besides it also cover ROM use case properly in test case.


Default mappings suggestion

Awesome talk at Vimconf 2021! :sunglasses:
It would be helpful to have a set of default/suggested keymappings in the README (another TODO :slightly_smiling_face: : ) that we could tweak/use. They can also be Dvorak :sweat_smile: I was searching through your dotfiles but to no avail.
For instance, if we want to map a key in the lua block, do we do:
    keybindings = {
      add_file = '<leader>ha',
      toggle_quick_menu = '<leader>ht',
    }

Or do we need to do
vim.api.nvim_set_keymap("n", "<leader>hh", require("harpoon.ui").nav_file(1), {})

Which doesn't work ...
I finally came up with:
vim.api.nvim_set_keymap("n", "<leader>;a", ":lua require('harpoon.mark').add_file()<CR>", {})
vim.api.nvim_set_keymap("n", "<leader>;t", ":lua require('harpoon.ui').toggle_quick_menu()<CR>", {})
vim.api.nvim_set_keymap("n", "<leader>;h", ":lua require('harpoon.ui').nav_file(1)<CR>", {})
vim.api.nvim_set_keymap("n", "<leader>;j", ":lua require('harpoon.ui').nav_file(2)<CR>", {})
vim.api.nvim_set_keymap("n", "<leader>;k", ":lua require('harpoon.ui').nav_file(3)<CR>", {})
vim.api.nvim_set_keymap("n", "<leader>;l", ":lua require('harpoon.ui').nav_file(4)<CR>", {})

But I'm sure there's a more elegant way to do it ... :sweat_smile:

Hi, if you're looking for some suggestions, look at prime's primal harpoon keyboard priming here :grin:

So that's where they were :sweat_smile: thanks!

@gerazov The above solution is good, but limits you to 4 main files or you need more remaps. I have a different solution which is better for some cases.
vim.api.nvim_set_keymap('n', '<leader>h', '<cmd>lua require("harpoon.ui").nav_file(vim.v.count1)<cr>',  opts)
Now, 1h navigates to file 1 and 3h navigates to file 3

Good suggestion :+1:


fixing community group

@nathanklick looks like the 2 of us did similar commits. We already had the "community" group to collect all people based on that I removed the group you created.

@hendrikebbers could you rebase this?

@ryjones done


smokers example from Philippe

just the changes, no noise

Current coverage is 96.98% (diff: 100%)

Merging #43 into master will not change coverage

@@             master        #43   diff @@
==========================================
  Files            12         12
  Lines           663        663
  Methods         581        581
  Messages          0          0
  Branches         82         82
==========================================
  Hits            643        643
  Misses           20         20
  Partials          0          0

Powered by Codecov. Last update bf2decc...a5e037f

I pushed edits that correspond to the comments I expressed earlier here.


Disable buttons in bottom of side panel when response is generating

EPAM AI DIAL chat version
0.16.0
How to reproduce
Steps:

create a chat
input message which suppose to generate long response, e.g. List of 50 adjectives describing weather
while response is generating, try to import chat/delete all chats/ select all chats
while response is generating , try to click on conversation settings button in chat header

Actual result

buttons import chat/delete all chats/ select all chats are enabled in conversation side pannel
conversation setting modal is open for a second and then close

Expected result
While response is generating , need disable buttons from conversation chat panel: import conversations, Export conversations, Delete all conversations, Select all
Also need to disable  conversation settings button in chat header
Confidential information

[X] I confirm that do not share any confidential information

verified on staging succesfully


[Feature Proposal] 'object-' part should be optional when specifying object-fit utilities

Clear and concise description of the problem
I would like to have a way to specify shorter versions of object-fit rules
Current implementation is:
https://github.com/unocss/unocss/blob/8505f48d4ccba055c0613bd22f120312447682ca/packages/preset-wind/src/rules/static.ts#L70
// object fit
  ['object-cover', { 'object-fit': 'cover' }],
  ['object-contain', { 'object-fit': 'contain' }],
  ['object-fill', { 'object-fit': 'fill' }],
  ['object-scale-down', { 'object-fit': 'scale-down' }],
  ['object-none', { 'object-fit': 'none' }],

import type { Rule } from 'unocss'

export const object: Rule[] = [
  ['cover', { 'object-fit': 'cover' }],
  ['contain', { 'object-fit': 'contain' }],
  ['fill', { 'object-fit': 'fill' }],
  ['scale-down', { 'object-fit': 'scale-down' }]
]

Suggested solution
Suggested rules with object-
// object fit
  ['/^(:?object-)cover/', { 'object-fit': 'cover' }],
  ['/^(:?object-)contain', { 'object-fit': 'contain' }],
  ['/^(:?object-)fill', { 'object-fit': 'fill' }],
  ['/^(:?object-)scale-down', { 'object-fit': 'scale-down' }],
  ['object-none', { 'object-fit': 'none' }],

Alternative
No response
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the README.md of using the package.
[X] Already used the Interactive Docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I am not sure about this - usually I'd like to avoid one-word utility as it can create necessary false positive (we have quite a few from Tailwind that I want to remove but can't 🫣)
So, I'd take a no for the official preset. But feel free to do it on userland as you already did :)


Provide the basic testing tools for the operator developers

Issue : #52

Provide the basic tools for the operator developers to test their Kopf-based operators.
This feature will be used also to run end2end tests in the following PRs (e.g. #54).
Checklist

[x] Tests
[x] Documentation

@samurang87 All grammar issues are fixed.

@samurang87 Fixed again.

"existant" is in a few places actually :)

@samurang87 Fixed in few places.


[5.6][build] Allow cross-compiling build-script products for non-Darwin hosts too

Cherrypick of #36917
Explanation: This pull consists of a handful of build-script changes that enable cross-compiling later Python build-script products like SPM and swift-driver for non-Darwin hosts, just as is currently possible when cross-compiling the toolchain for macOS arm64, including generating a cross-compilation destination JSON config file to use with SPM for Android. Most of the pull is scoped to non-Darwin cross-compilation or specifically to Android, with the exception of some install prefix refactoring, because cross-compilation for non-Darwin hosts installs the toolchains separately under a directory named for each host.
Scope: Since this is only build-related and mostly for non-Darwin cross-compilation, most is not even run on the CI, with the exception of the install prefix refactoring.
SR Issue: None
Risk: Almost none, only the simple install prefix changes even affect the normal host builds.
Testing: I've been using earlier versions of this pull for a while to cross-compile the toolchain for Android and recently added it to my Android CI so I can cross-compile SPM for Android with every trunk snapshot.
Reviewer: @gottesmm

@gottesmm, can we can get this in before the 5.6 release?

Closing since I was told only regressions or bugfixes wIll be accepted into the release branch after the release shipped, though I did submit this months prior.


Change the way errors are handled in Lua scripts

In http://splash.readthedocs.org/en/1.3.1/scripting-tutorial.html#error-handling it is described how are errors handled in Splash Lua scripts:

for developer errors (e.g. incorrect function arguments) exception is raised;
for errors outside developer control (e.g. a non-responding remote website) status flag is returned: functions that can fail return ok, reason pairs which developer can either handle or ignore.

This means that by default script won't stop on "external" errors; to make code robust almost every statement needs to be wrapped in assert():
assert(splash:go(splash.args.url))
assert(splash:wait(0.5))

Even in the documentation I was sometimes sloppy and haven't written these asserts. One example is  wait_for from http://splash.readthedocs.org/en/latest/scripting-tutorial.html#writing-modules:
function Splash:wait_for(condition)
    while not condition() do   -- condition function can't follow our conventions here
        self:wait(0.05)        -- assert is missing
    end
end

It is too easy to forget about error checking, and code that checks for errors looks uglier than code which doesn't check.
I propose to change the error handling convention and raise an error both for developer errors and for error conditions outside developer control (e.g. 404 pages).
Pros:

errors are not silently discarded;
less code to write and read in common cases (?);
it is possible to wrap several statements in pcall and handle errors once.

Cons:

not ideomatic Lua;
handling errors with pcall requires more work;
pcall may catch developer errors (e.g. invalid argument names)

TODO: figure out how code using pcall will look like:
-- option 1, no-brainer - does it work?
local ok, res = pcall(splash:evaljs, "document.title")

-- option 2, ugly
local ok, res = pcall(splash.evaljs, self, "document.title")

-- option 3, verbose
local ok, res = pcall(function()
    return splash:evaljs("document.title")
end)

Any thoughts?

+1
This is very important IMO. I just realized in some situations we take screenshots of 500 and 404 pages. Script like this
function main(splash)
            splash:go("http://httpbin.org/status/500")
            return splash:png()
end

returns perfectly nice 200 response from splash, and client gets screenshot of 500 without anyone noticing this.

I propose to change the error handling convention and raise an error both for developer errors and for error conditions outside developer control (e.g. 404 pages).

This sounds like a good idea for me, but just to clarify, which response codes are we talking about here? If splash makes 60 requests while rendering page, and 1-2 return 404 will error be raised? Or are we talking only about response codes for first initial request?
One thing about:

for errors outside developer control (e.g. a non-responding remote website) status flag is returned: functions that can fail return ok, reason pairs which developer can either handle or ignore.

currently ok, reason pair is very generic. If remote website responds with 400 clients gets nil, 'error' pair, and it gets same thing when remote website responds with 500.
Is it possible to get more specific error messages? For example status code of response and message? Does it make sense? I know there are many different requests involved when rendering, so probably it would be difficult to speak about one response status code, but perhaps we have some data available that could help the client to know why splash request failed?
Another thing that seems important to me. What will be response code of splash in case remote resource responds with 500? IMO splash response code should allow to distinguish this failure from failure of splash operation, perhaps 503 with error message propagated from remote resource would be good?

Hey @pawelmhm,

... returns perfectly nice 200 response from splash, and client gets screenshot of 500 without anyone noticing this.

yes, this is the main motivation behind this proposal. You need to be careful and write assert(splash:go(...)) each time, or handle it explicitly by using ok, reason = splash:go(...).

This sounds like a good idea for me, but just to clarify, which response codes are we talking about here? If splash makes 60 requests while rendering page, and 1-2 return 404 will error be raised? Or are we talking only about response codes for first initial request?

If you're talking about splash:go, then error is raised in case of 4xx or 5xx response for the request which contents is displayed in the browser window. In most cases it is the first request, but it could be not the first (e.g. a page can do a redirect to a 4xx or 5xx resource after some time). It may happen that there is no response at all, e.g. if a remote resource is not availbale because of DNS error or network error or because a remote server dropped connection without returning anything - this is also an error.
Other functions can return an error in other cases; they all should be documented in Splash docs.

currently ok, reason pair is very generic. If remote website responds with 400 clients gets nil, 'error' pair, and it gets same thing when remote website responds with 500.

This is not how Splash is designed to work; if you see this behaviour it is a bug. Error is not always 'error', it should be possible to distinguish different error types: for 400 response it should be 'http400', for 503 response it should be 'http503', etc. This is documented and tested.

Another thing that seems important to me. What will be response code of splash in case remote resource responds with 500? IMO splash response code should allow to distinguish this failure from failure of splash operation, perhaps 503 with error message propagated from remote resource would be good?

Splash allows to distinguish between various error types. Currently it doesn't allow to set a resulting HTTP status code error code, but you can return {'status': ..., result: ...} instead of just returning the result.
To clarify: by changing how errors are handling we'll change the style in which Splash scripts are written; it doesn't change anything else. Handling errors properly should be already possible; it is just easy to forget to check for errors, and Splash silently ignores them if you don't handle them.
By changing errors from ok, reason style to exceptions style it'll be harder to miss error checks. But
https://github.com/scrapinghub/splash/issues/165 is a blocker here.

This is not how Splash is designed to work; if you see this behaviour it is a bug.

aha, so I think I found a bug here, I tested this in splash jupyter with recent splash version:
In [1]: splash:go("http://httpbin.org/status/400")
Out[1]:
nil, "error"
In [2]: splash:go("http://httpbin.org/status/500")
Out[2]:
nil, "error"```

> Currently it doesn't allow to set a resulting HTTP status code error code, but you can return {'status': ..., result: ...} instead of just returning the result.

hmm this sounds interesting, but let's say I'm returning data in .png by default, so I can't return json easily without having to update clients, spiders would have to check content-type which would be a bit confusing.

>  To clarify: by changing how errors are handling we'll change the style in which Splash scripts are written; it doesn't change anything else. Handling errors properly should be already possible; it is just easy to forget to check for errors, and Splash silently ignores them if you don't handle them.

ok I understand, but is it okay that splash will return same status code for errors in lua scripts (let's say syntax errors) and errors from remote website (correct script but target website responds with 500)? These are different situations. In case of syntax errors in lua script this is evidently client fault - and 400 bad request is appropriate. If website responds with 500 it means it is down and client should retry, so it would be nice to get 5** status code from splash.

aha, so I think I found a bug here

Thanks, I'll check it.

hmm this sounds interesting, but let's say I'm returning data in .png by default, so I can't return json easily without having to update clients, spiders would have to check content-type which would be a bit confusing.

A simpler way to write a client would be to always consume JSON. But I'm fine with allowing to set HTTP status code from the Splash script, similar to how it allows to set Content-Type (http://splash.readthedocs.org/en/latest/scripting-ref.html#splash-set-result-content-type). Returning JSON is more general and doesn't require using non-200 HTTP status codes, but in some specific cases (like returning PNG images directly, without encoding them to base64/json) it may be helpful.

ok I understand, but is it okay that splash will return same status code for errors in lua scripts (let's say syntax errors) and errors from remote website (correct script but target website responds with 500)?

Yes, I think it is OK. Splash will return an error not because the remote website returns an error, but because user didn't handle this case, so it should use the same 400 status code. Error message indicates which kind of error happened (see https://github.com/scrapinghub/splash/issues/226 though). When users handle the error (by using pcall) they are free to return anything or set any appropriate http error code, if your suggestion about splash:set_result_status_code gets implemented.
Possible errors are not limited to splash:go method (e.g. splash:wait or splash:evaljs can also raise an error), not all of them map to HTTP status codes cleanly. For example, should we return 504 response if splash:wait_for_resume timeouts? If we do, it will become harder to distinguish these errors from 504 errors in remote responses. We may return 400 in all cases the error is not caused by a remote resource, but what if remote website returned 400 response? For me it looks like auto-mapping exceptions to HTTP status codes is a wrong approach - there are tricky edge cases, and it is not easier to work with than alternatives.
See how Python does it: in case of unhandled exception status code is always set to 1, regardless of the exception. If you want to return anything else you need to catch the exception and use sys.exit(..) with an exit code you want.

@pawelmhm an error with HTTP errors detection should be fixed by #228 - thanks for catching it!

when working on https://github.com/scrapinghub/splash/issues/231 I noticed that errors resulting from calls to undefined Lua methods/attributes in Splash callbacks are not propagated to the client, eg Lua like this with typo in set_url will return 200 response:
function main(splash)
    splash:on_request(function(request)
        if request.url == splash.args.url then
            request:set_urlx(splash.args.new_url)
        end
    end)
    splash:go(splash.args.url)
    return splash:html()
end

should it raise 400?

@pawelmhm see https://github.com/scrapinghub/splash/issues/208. I think that the error should be propogated indeed.


New version v2.2.6

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 45.71%. Comparing base (10c77c4) to head (a118dcc).

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #292   +/-   ##
=========================================
  Coverage     45.71%   45.71%
  Complexity     1617     1617
=========================================
  Files           300      300
  Lines         11846    11846
  Branches       1153     1153
=========================================
  Hits           5415     5415
  Misses         6017     6017
  Partials        414      414

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Consider restricting css modules to sources and making CSS Modules not the default way to style

For now the webpack rules for CSS can leads to bugs (even if nobody reported some ;) ) or cause issue with CSS consumed from node_modules (since there is no include  for CSS atm), cause most CSS from npm (even if there is not a lot) is probably global CSS.
So here is my idea(s):

making global css default for .css files, and making css module for .module.css or .m.css or any other convention
keeping CSS modules for .css but restrict to web_modules + .global.css in web_modules + global CSS from all CSS from node_modules (but this can leads to bugs to, if, let's say, a react component in your node_modules need CSS modules.
making global CSS default, and remove CSS Modules from the default boilerplate

Not sure what people wants. I see that gatsbyjs is pretty popular and have low standards (just global css by default and they plan to add .module.css for CSS Modules).
Any thoughts?
Obviously this can be adjust by the user (due to where it's configured) and won't create a breaking change as it will affect new setup only.

For now I updated the default boilerplate to scope the default behavior (CSS Modules + *.global.css files) to web_modules folder while we decide for something better (if we can), but the discussion is just starting ;)

Here's a personal opinion from a newcomer (both to Phenomic and modern front-end development):
I don't get CSS modules. I don't understand if they each create a new CSS file, or if they're in the end bundled up in a single file but with class names that are somehow modified to some unique-to-a-module value. I guess the latter, because creating multiple files would mean multiple fetch requests, thus extra network congestion.
But I don't get the point of having classes unique to a component or something. The basic idea of CSS originally was to have generic styling rules that would apply to the whole doc so that authoring structured it, and layouting that structure, was two separate processes and concerns. With class names that only apply to a component or two, though, I don't understand how it's different than styling from the layout.
 Having specific rules for components that call such or such css module, to me, is a basically creating very specific selectors, or very specific class names, each are possible with global css, and you don't break consistency.
So I don't know, I'm inclined to prefer the global-first approach, simply because most of the time you're gonna have modular Sass, modular React components, and (in my opinion), should keep those two workflows separate, with upstream decisions on what semantic is gonna be layouted. (If your site need cards of some sort, decide together on a class name, and everyone uses that.)
The CSS Module use-case, at least to me, seems like a very uncommon one: most of the modularity of UI can be achieved with simply smarting through CSS dev. So if it should be kept, I would personally prefer marking it in the name like *.mod.css
That's a vote for global-first. ✋

Check out this slide deck and the videos in the description https://speakerdeck.com/davidwells/bulletproof-css-in-react for more information on postcss, CSS modules and why they are super rad.
Global CSS has its place but local has a lot of benefits.
The two work very well together :)
That said you can write all of your CSS in phenomic with global styles in the *.global.css files. Phenomic is very flexible for you

You can also just remove the webpack section that handle this, and use global without the weird extension :)

Going to be fixed with #925 (not going to handle this)

No more css assumption by phenomic in v1.


 As such, this feature isn't really useful anymore, and should be removed. I understand that I can disable it, however I don't understand any further usecases.

Example URLs
https://github.com/refined-github/refined-github/pull/1418

GitHub allows the config per-repo, only if you're an admin. This feature works on every repo, without config


Remove deprecated smart column from vms table

Apparently this SSA related column is deprecated and hasn't been used in years. Let's remove it.
Update: Also removed it from the hosts table, and removed the various makesmart related code from the models.

Digging a little deeper there is a reference in the manageiq-ui-classic repo: https://github.com/ManageIQ/manageiq-ui-classic/blob/a4bd74f590940991ab247730f5297380b8003219/app/helpers/application_helper/button/host_protect.rb#L5
That's the only other one I found.

@djberg96 Some of the tests are failing because you need to modify https://github.com/ManageIQ/manageiq/blob/master/db/schema.yml

@gmcculloug regarding the UI, just set it to false? As for the schema.yml, I'll update it, though I guess I thought that was autogenerated. Apparently not.

The host record was always set to true so the manageiq-ui-classic change should make visible? return true.  I am hoping we can get rid of that method (and file) altogether.
@carbonin Should schema.yml get updated for deletes automatically?

Please be sure to check the UI code as well.  This is shown as a "shield" icon over the quadicon, so that should be removed first. cc @dclarizio @h-kataria

@carbonin Should schema.yml get updated for deletes automatically?

Nope, you can remove the column manually or follow the instructions in the spec failure :wink:

With https://github.com/ManageIQ/manageiq-ui-classic/pull/893 merged, this one is ready to go.

@djberg96 Please rebase and re-push /cc @Fryguy

Database migrations have now been moved to the https://github.com/ManageIQ/manageiq-schema repo.  Please see http://talk.manageiq.org/t/new-split-repo-manageiq-schema/2478 for instructions on how to transfer your database migrations.  If this PR contains only migrations, I will leave it open for a short time during the transition, after which I will close this if it has not been moved over.


com.alibaba.fastjson.parser.ParserConfig&com.alibaba.fastjson.util.IdentityHashMap

fastjson.version : 1.2.83
I have upgraded to version 1.2.83, but this still report errors.Please help


[🐞] Ref has a different value when used under the conditional slot

Which component is affected?
Qwik Runtime
Describe the bug
import { component$, useSignal, Slot, useVisibleTask$ } from<EMAIL_ADDRESS>
export const Toggle = component$<{open: boolean}>(({open}) => {

  return

  {open ? <Slot/> : null} // conditional slot

})

export default component$(() => {
  const ref = useSignal<HTMLDivElement>()
  useVisibleTask$(() => {
    console.log("But in this hook the value of ref is: ", ref.value); // [object HTMLParagraphElement]
  })
  let open = false;
  return

{`Ref element must be undefined: ${ref.value === undefined}` /* true */}

    <Toggle open={open}>

Hello Qwik

    </Toggle>

;
});

Reproduction
https://t.ly/lkcX
Steps to reproduce
No response
System Info
Qwik Playground

Additional Information
No response

I don't think there is a bug here, and the behavior doesn't seem to be impacted by the use of Slot. The first time the component renders (on the server BTW), there isn't yet a DOM element to connect the ref.
Playground

I don't think there is a bug here, and the behavior doesn't seem to be impacted by the use of Slot.

export const Toggle = component$<{open: boolean}>(({open}) => {

  return

    {open ? <Slot/> : null} // conditional slot

})

<Toggle open={false}>

Hello Qwik

</Toggle>

In the code snippet provided, Toggle's children never attach to the DOM with open=false, but I receive [object HTMLParagraphElement] in console.log

The first time the component renders (on the server BTW), there isn't yet a DOM element to connect the ref.

Ref is a signal, right? So it should update the DOM when its value changes, but it doesn't.

Ok I misunderstood. Here's a new playground that displays the issue. Even when inside a component that doesn't render Slot, the element is registered to the ref. Logging the ancestor elements we see it's hidden inside a <q:template> element, but this does seem unexpected.

Qwik always renders the children, regardless of slot. This is how qwik can achieve full isolation of children and parent, the children adding/removing slots, does not force to rerender the children.
Similar approach is the name <slot/> for web components.
This is how qwik works.

I think there's still a question if the ref should be linked to the signal if the element is in a <q:template>. Changing it would be a breaking change though probably...

Qwik always renders the children, regardless of slot. This is how qwik can achieve full isolation of children and parent, the children adding/removing slots, does not force to rerender the children.
Similar approach is the name <slot/> for web components.
This is how qwik works.

Yeah, I now understand about slots.
However, even without the conditional slot part, the question remains: why won't the ref signal update the DOM after resuming on the client?
export default component$(() => {
  const ref = useSignal<HTMLElement>()
  return

{ref.value?.tagName ?? "[empty]" /* The result is: [empty] */}

;
});


Feature/decompose compile task

[ ] Since this PR suggests a bug fix, the relevant tests have been added.
[x] Since this PR introduces a new feature, the update has been discussed in an Issue or with the team.
[ ] This PR is just a minor change, like a typo fix.

Let's rename compile dir to core and core dir to compile. Then move setup to core.


Implement a custom annotation for the generation and clean up of test artifacts

Task Description
We need to implement a set of custom annotations that will handle the generation and clean up of test artifacts. Each test case (and method) should create its own isolated test resources and not share them with other tests. This way, tests can safely be executed in parallel.
Current Implementation
Excerpt from MavenArtifactControllerTest.init(), (the same kind of examples could be seen across the tests for all providers):
MutableRepository repository1 = mavenRepositoryFactory.createRepository(REPOSITORY_RELEASES1);
repository1.setPolicy(RepositoryPolicyEnum.RELEASE.getPolicy());
repository1.setRepositoryConfiguration(mavenRepositoryConfiguration);

createRepository(STORAGE0, repository1);

// Generate releases
// Used by testPartialFetch():
generateArtifact(getRepositoryBasedir(STORAGE0, REPOSITORY_RELEASES1).getAbsolutePath(),
                 "org.carlspring.strongbox.partial:partial-foo",
                 new String[]{ "3.1",
                               // Used by testPartialFetch()
                               "3.2"
                               // Used by testPartialFetch()
                 }
);

Desired implementation
We need to implement a custom @TestArtifacts annotation like below:
@TestArtifacts({ @TestArtifact(generator=MavenArtifactGenerator.class,
                               storage = "storage0",
                               repository="releases",
                               path="path/to/artifact.jar")
})

Obviously, the example above is quite generic (and abstract). For convenience, this should be a base annotation and there should be sub-implementations for the different layout providers such as:

[ ] MavenTestArtifact
[ ] NpmTestArtifact
[ ] NugetTestArtifact
[ ] RawTestArtifact

The TestArtifacts annotation should accept an array of these sub-implementations and should be able to handle the generation and clean up of multiple artifacts.
This annotation will generate aritfacts before test execution and clean them up from the FS and DB after execution. This way we won't need to manage artifacts with any code in test or setup, or tear down methods.
Task Links
This task is:

Required for #949.
Similar to #1083.

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@fuss86

Any possibility to help you with this one or another one? That would be my first experience and I am open to discuss in which areas I might be helpful to you.

Hi @denizseaside82 ,
Thanks for offering to help, but we were just discussing this with @sbespalov earlier today and, given that he's already implemented #1083 in #1105 , he will look into this task as well.
Would you like to join our chat? We could discuss some tasks there, if you like.

Great! Is there any exact time which you all gather and discuss issues in your chat?

There's always someone around.


(cargo-release) version 0.3.1

Motivation

Current dependencies on/for this PR:

main

PR #163

PR #164   👈

This comment was auto-generated by Graphite.


Remove to work details when a candidate selects they are British or Irish

Context
There is a bug that can be reproduced with the following steps on QA:

Complete the personal details section as an Albanian candidate
Edit nationality to British and Albanian
See the right to work row is there when it should not be

Changes proposed in this pull request

Reset the right_to_work_or_study and  right_to_work_or_study_details fields to nil when a candidate selects British or Irish as one of their nationalities
Update the logic for rendering the right_to_work row to check for Britsh/Irish in all nationalities

Guidance to review
Previously the logic for whether to render the right to work row only checked the first nationality. The nationalities are stored alphabetically so if someone has picked Austrian/Albanian etc. and British as nationalities it renders the row when it shouldn't.
Link to Trello card
https://trello.com/c/V6N7Dv0K/2050-🌐-dont-show-or-save-residency-status-if-nationality-is-british-or-irish
Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

It’s possibly a bit simpler than this fix implies. If a candidate has either British or Irish nationality, don’t show (or save) any previous answer for residency status.
So, for example, in both the following examples, my previous answer for ‘Residency status’ should not be shown.

Cool. I think we should probably also set the values for the right to work response to nil on the application form though. They are no longer relavant and it could be potentially misleading if for example someone queries for all application forms in which those attributes are not blank.

Cool. I think we should probably also set the values for the right to work response to nil on the application form though. They are no longer relavant and it could be potentially misleading if for example someone queries for all application forms in which those attributes are not blank.

Yup. If British or Irish, don’t show or save anything regarding residency status, and remove it if previously added.

Perfect. This is good to go once a dev approves then 👍

Lovely stuff.


(GH-77) Set TEMP folder to include vagrant vmId

This will make sure the temp folder for the vagrant actions is set
to a known empty folder, preventing lock issues.
Fixes #77
/cc @ferventcoder @gep13

This looks pretty good. I have that one thing I'd like to see.

@ferventcoder I have made the changes as requested.


exclude autogenerated ***.pb.dart from test coverage

see https://github.com/google/protobuf.dart/issues/768


Tidy up utility methods for MultiTenantIT

Per Keith's comment in #349 (Split up MultiTenantIT test):

we've also been building up lots of utility methods (createTopic, deleteTopic etc) across the IT suite. It might make sense to bring some order to those first (under a separate issue), then you can reuse those with the separate MultiTenant IT tests.

Need to tidy up these methods before progressing with #349

Might be worth looking more widely than MultiTenantIT (suggest **/*IT.java).  We've  got helper methods such as #createTopic coming up like weeds 🌿!

Addressed in #490


Resizable open editors section

resizable open editors section

Dup of https://github.com/Microsoft/vscode/issues/491


:w to save

I don't expect this package to implement all of helix's colon menu tools, but things like :w would be very nice just due to muscle memory

Dang @lukepighetti you show up on every issue I research while setting up my helix stuff :cupid:


Docker: Save docker volume dependency info into meta.ini.

Should we record this information also in master?


[SPARK-26335][SQL] Add a property for Dataset#show not to care about wide characters when padding them

What changes were proposed in this pull request?
Issue
SPARK-25108 made Dataset#show care about wide characters when padding them. That is useful for humans to read a result of Dataset#show. On the other hand, that makes it impossible for programs to parse a result of Dataset#show because each cell's length can be different from its header's length. My company develops and manages a Jupyter/Apache Zeppelin-like visualization tool named OASIS. On this application, a result of Dataset#show on a Scala or Python process is parsed to visualize it as an HTML table format as follows:

Solution
Add the spark.sql.dataset.show.handleFullWidthCharacters property for Dataset#show to control whether wide characters are cared/handled or not.
How was this patch tested?
This patch was tested via unit tests.
Jira Issue
https://issues.apache.org/jira/browse/SPARK-26335

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

I think it makes more sense that OASIS side implements one method, for instance, showAsTable
implicit class OASIS(df: DataFrame) {
  def showAsTable(): DataFrame = {
    // Mimic `Dataset.showString`
  }
}

and
dataset.showAsTable

I think df.show() does not have a guarantee about how it's printed in console, and the purpose of this API is just to show a pretty string rather than parsing and using it. This configuration looks too specific to OASIS.

adding @srowen

Yes, this isn't an API in this sense. Its purpose is to render to the console. Why does it make it hard or impossible to reformat? You'd probably want to simply get the real contents of the DataFrame and display it, not rely on what .show() outputs, in this case.

Thanks for your comments. I'll implement an API for generating an HTML table tag from a Dataset in my WebUI application. Thanks.


URL are not included in the O365 referential #71786

Originally posted by @Tomtom10000 in https://github.com/MicrosoftDocs/azure-docs/issues/71786#issue-824606359
The url "https://tokenprovider.termsofuse.identitygovernance.azure.com"  is not included in the repository:
https://docs.microsoft.com/fr-fr/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide
We use this repository to open the flow for all proxies in our company, if not in the repository, users will not access it and therefore it will not be able to access O365 because they can not validate temsofuse.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 66bd8666-3566-a901-b25b-eb6baba6264a
Version Independent ID: e9f6310f-2207-8c81-d31a-3d5a80beb26b
Content: Office 365 URLs and IP address ranges - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/urls-and-ip-address-ranges.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@BhargaviAnnadevara-MSFT  -- I don't think this endpoint is actually appropriate for the M365 URLS and IP ranges topic. At any rate, this article is autogenerated from an internal tool and info cannot be manually added to the article (or it will be overwritten). Follow up with me via internal email if you need info about onboarding to the M365 endpoints process.


Webmin System Stats:  CSS fix for panorama

We've fixed some viewing issues in a webmin module we've been using for years that compiles system stats: Sysstats. The "Panorama" view shows small thumbs for all active modules so you can have a general view of all the system graphics. Images were not showing correctly due to rather strict style in: authentic.min.css with a generic:
img {max-width: 100%}
I imagine this might be causing conflicts in other older modules too.
In case of interest for future releases we've fixed it by adding:
body.sysstats .graph_table a img {max-width: unset}
into our theme extension css file.

Please provide the screenshot at first and then after your fix is applied.

Before:

After:

I will add this to the next version soon.

It will be fixed in 18.20

Version 18.20 is out, please upgrade.


VPP MCTF is disabled in default library configuration build

Here is how MCTF is enabled in the mediasdk library:
https://github.com/Intel-Media-SDK/MediaSDK/blob/0e50c533fc57f06af703ed41614d170972ed10a5/_studio/shared/include/mfx_config.h#L216
Hence, there is no chance to get feature workable with the default build which should match API=1.26 at the moment:
perl tools/builder/build_mfx.pl --cmake=intel64.make.release

I modify   #define MFX_VERSION_NEXT (MFX_VERSION_MAJOR * 1000 + MFX_VERSION_MINOR + 1) to #define MFX_VERSION_NEXT (MFX_VERSION_MAJOR * 1000 + MFX_VERSION_MINOR in the api/include/mfxdefs.h, then the MCTF is enabled in the mediasdk library, But when i run ./sample_multi_transcode -i::h264 input.264 -o::h264 out.264 -mctf 10 and ./sample_multi_transcode -i::h264 input.264 -o::h264 out.264, the result of out.264 can't find any difference

-mctf  option needs to be given in par file
same trick with MFX_VERSION_NEXT needs to be done for samples (there is
also configuration file)
finally, check help info that is printed by sample_multi_transcode (or
sample_vpp) if mctf is enabled, it should print help info with MCTF details.

good luck, Mike
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Mon, Apr 9, 2018 at 12:03 PM, changliwu<EMAIL_ADDRESS>wrote:

I modify #define MFX_VERSION_NEXT (MFX_VERSION_MAJOR * 1000 +
MFX_VERSION_MINOR + 1) to #define MFX_VERSION_NEXT (MFX_VERSION_MAJOR *
1000 + MFX_VERSION_MINOR in the api/include/mfxdefs.h, then the MCTF is
enabled in the mediasdk library, But when i run ./sample_multi_transcode
-i::h264 input.264 -o::h264 out.264 -mctf 10 and ./sample_multi_transcode
-i::h264 input.264 -o::h264 out.264, the result of out.264 can't find any
difference
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Intel-Media-SDK/MediaSDK/issues/144#issuecomment-379684637,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE4AoCBE0s0Z6ewAxX_76xX9jWpmpPRRks5tmyP4gaJpZM4TKpYh
.

same trick with MFX_VERSION_NEXT needs to be done for samples (there is also configuration file)

which configuration file can i change for the value of MFX_VERSION_NEXT？
thanks！

To enable pre-release functionality you should generate makefiles with following command:
perl tools/builder/build_mfx.pl --cmake=intel64.make.release --api=latest

But since commit e2b1ca61 MCTF should be enabled even without it.

Thank you for your answer！
I have anther question, I run the demo of sample_multi_transcode with the API 1.26 of MediaSDK,
like this:
./sample_multi_transcode -i::h264 input.264 -o::h264 out.264 -cqp -qpi 24 -qpp 26 -qpb 31 -denoise 10
./sample_multi_transcode -i::h264 input.264 -o::h264 out.264 -cqp -qpi 24 -qpp 26 -qpb 31
but the size of the two output files(out.264) has little change, Is the denoise is  enabled in API 1.26 ?

This was addressed with e2b1ca61fb2e69ac3e200c2597787dd3099eedfa


[18.09] flatpak: 0.99.3 -> 1.0.2 (cherry-picked from master)

Motivation for this change
Update Flatpak to the first stable version.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Possibly because something is messed up in my user config of flatpak, I cannot currently do flatpak update on NixOS 18.09:
$ flatpak update
error: While opening repository /home/alexey/.local/share/flatpak/repo: min-free-space-percent and min-free-space-size are mutually exclusive

I hope this is fixed in Flatpak 1.0.2: https://github.com/ostreedev/ostree/pull/1685

Cannot test it but if it works for you I do not see a reason not to merge.

@jtojnar, is there any online documentation about how I could test this? (I might be able to figure out how to build and install a modified application package, but this is a service.)

Usually, I just run nix-build nixos/release.nix -A tests.flatpak.x86_64-linux and nix-build nixos/release.nix -A tests.xdg-desktop-portal.x86_64-linux.

Yes, it means the tests passed. You can ignore the "vde_switch" error.


allow enabling scheduler with environment variable

Addresses https://github.com/symptoma/activemq/issues/13

@samholton Thanks for the fast implementation!


flatcar-install fails on CentOS7

Description
I booted a server using CentOS7 and tried to run flatcar-install, but it fails to complete.
Impact
Would love to install flatcar linux.
Environment and steps to reproduce

Set-up:

booted hetzner cloud instance with CentOS7
downloaded Flatcar ISO (there are other issues verifying the iso)

Task: installing Flatcar linux
Action(s):
a. ./flatcar-install -d /dev/sda1 -i /root/ignition.json -f ./flatcar_production_iso_image.iso -v

Output:
+ getopts V:B:C:I:d:o:c:e:i:t:b:k:f:nsyvh OPTION
+ [[ -z /dev/sda1 ]]
+ [[ -n '' ]]
++ lsblk -n -d -o TYPE /dev/sda1
+ [[ part =~ ^(disk|loop|lvm)$ ]]
+ echo './flatcar-install: Target block device (/dev/sda1) is not a full disk.'
./flatcar-install: Target block device (/dev/sda1) is not a full disk.
+ exit 1
[root@node-001 ~]# ./flatcar-install -d /dev/sda -i /root/ignition.json -f ./flatcar_production_iso_image.iso -v
+ getopts V:B:C:I:d:o:c:e:i:t:b:k:f:nsyvh OPTION
+ [[ -z /dev/sda ]]
+ [[ -n '' ]]
++ lsblk -n -d -o TYPE /dev/sda
+ [[ disk =~ ^(disk|loop|lvm)$ ]]
+ [[ ! -w /dev/sda ]]
+ [[ -n '' ]]
+ [[ -n /root/ignition.json ]]
+ [[ ! -f /root/ignition.json ]]
+ [[ -n '' ]]
+ _disk_status=
++ mktemp --tmpdir -d flatcar-install.XXXXXXXXXX
+ WORKDIR=/tmp/flatcar-install.cJNz6cYTpw
+ trap 'error_output ; is_modified && wipefs --all --backup "${DEVICE}" ; rm -rf "${WORKDIR}"' EXIT
+ '[' -n ./flatcar_production_iso_image.iso ']'
+ install_from_file
+ '[' -r ./flatcar_production_iso_image.iso ']'
+ echo 'Writing ./flatcar_production_iso_image.iso...'
Writing ./flatcar_production_iso_image.iso...
+ [[ ./flatcar_production_iso_image.iso =~ \.bz2$ ]]
+ write_to_disk
+ mkfifo -m 0600 /tmp/flatcar-install.cJNz6cYTpw/disk_modified
+ trap '(exec 2>/dev/null ; echo done > "${WORKDIR}/disk_modified") &' RETURN
++ blockdev --getsz /dev/sda
+ dd conv=nocreat count=1024 if=/dev/zero of=/dev/sda seek=80002048 status=none
+ dd bs=1M conv=nocreat of=/dev/sda status=none
+ udevadm settle
+ local try
+ for try in 0 1 2 4
+ sleep 0
+ blockdev --rereadpt /dev/sda
blockdev: ioctl error on BLKRRPART: Device or resource busy
+ echo 'Failed to reread partitions on /dev/sda'
Failed to reread partitions on /dev/sda
+ for try in 0 1 2 4
+ sleep 1
+ blockdev --rereadpt /dev/sda
blockdev: ioctl error on BLKRRPART: Device or resource busy
+ echo 'Failed to reread partitions on /dev/sda'
Failed to reread partitions on /dev/sda
+ for try in 0 1 2 4
+ sleep 2
+ blockdev --rereadpt /dev/sda
blockdev: ioctl error on BLKRRPART: Device or resource busy
+ echo 'Failed to reread partitions on /dev/sda'
Failed to reread partitions on /dev/sda
+ for try in 0 1 2 4
+ sleep 4
+ blockdev --rereadpt /dev/sda
blockdev: ioctl error on BLKRRPART: Device or resource busy
+ echo 'Failed to reread partitions on /dev/sda'
Failed to reread partitions on /dev/sda
+ '[' -z 4 ']'
+ exit 1
+ error_output
+ echo 'Error: return code 1 from exit 1'
Error: return code 1 from exit 1
+ is_modified
+ [[ -e /tmp/flatcar-install.cJNz6cYTpw/disk_modified ]]
+ wipefs --all --backup /dev/sda
wipefs: unrecognized option '--backup'

Usage:
 wipefs [options] <device>

Options:
 -a, --all           wipe all magic strings (BE CAREFUL!)
 -f, --force         force erasure
 -h, --help          show this help text
 -n, --no-act        do everything except the actual write() call
 -o, --offset <num>  offset to erase, in bytes
 -p, --parsable      print out in parsable instead of printable format
 -q, --quiet         suppress output messages
 -t, --types <list>  limit the set of filesystem, RAIDs or partition tables
 -V, --version       output version information and exit

For more information see wipefs(8).

Error:

not entirely sure if the first "error" is the busy device
or maybe usage of wipefs

Expected behavior
flatcar-install completes, and I can boot into Flatcar Linux.

Another thing to look out for is whether there are existing LVM partitions that are active. One can try something like lvchange -an /dev/mapper/vg* until logic to cover this is part of flatcar-install.

Another thing to look out for is whether there are existing LVM partitions that are active. One can try something like lvchange -an /dev/mapper/vg* until logic to cover this is part of flatcar-install.


Command fails if this is the first time running updateSQL

Environment
Liquibase Version:4.2.2
Liquibase Integration & Version: CLI
Liquibase Extension(s) & Version: N/A
Database Vendor & Version: N/A
Operating System Type & Version:
macOS BIgSur
Description
The first time you may run an update against a database, you may run updateSQL to check what changes will be made to the database. If this is the first time you have run a command against the database with Liquibase, it may fail since the DATABASECHANGE table has not been created.
Steps To Reproduce

Create a empty database
Create a liquibase.properties file with connectivity to the database
Create a changelog with one change.
Run liquibase updateSQL against the database

Using postgres as an example

CREATE DATABASE lms
WITH
OWNER = postgres
ENCODING = 'UTF8'
CONNECTION LIMIT = -1;
changeLogFile=changelog.sql
url=jdbc:postgresql://localhost:5432/lms
username: postgres
password: yoursecretpassword
classpath: ./postgresql-42.2.11.jar
-- liquibase formatted sql
-- changeset mikeo:createpersontable-1
create table person_a8b (
id int primary key,
name varchar(50) not null,
address1 varchar(50),
address2 varchar(50),
city varchar(30)
)
-- rollback drop table person_a8b
liquibase updateSQL

Actual Behavior
Error occurs:
Unexpected error running Liquibase: liquibase.exception.DatabaseException: org.postgresql.util.PSQLException: The database returned ROLLBACK, so the transaction cannot be committed. Transaction failure cause is <<ERROR: relation "public.databasechangeloglock" does not exist   Position: 22>> For more information, please use the --logLevel flag
Expected/Desired Behavior
Update succeeds and shows the create table sql
Screenshots (if appropriate)

Additional Context
Add any other context about the problem here.

This issue also seems to be affecting the update method of Java API version 4.21.1
liquibase.update(new Contexts(), new PrintWriter(System.out))

which I suspect to be using the updateSQL command, and is forcing developers to use a deprecated overload of the same
liquibase.update()


fix(dropdown): added f-dropdown-open to open dropdown element

To mimic Foundation javascript dropdown functionality for styling purposes.

:+1: Thanks!


General Ideas on Easing Context/Content Switching

First off, thanks for the profiles - I'm trying it out now & it simplifies things significantly. The post got long, I just wanted to keep context of why I think changes might be helpful rather than blunt requests. I'm happy to discuss this on discord or other means if you'd rather do that than read a wall of text.
On a personal note, while I forked the repo to make this a little easier on myself, at this point, all of the features I care about are already inside of here so I'm looking at how to get back onto the base project again. Thanks for the work on this project 🌮
Since I usually switch between a few VNs & audiobooks while using the texthooker-ui, the two things I use the most are switching profiles (particularly adjusting the titles & vertical layout) & clearing the text.

I configured it to look like this.
From what I've been playing with here are the important things, I'd ask if we could consider a few of the following ideas:
A. Switching profiles easily
B. Clearing all text button
C. Clearing all text less verbose confirmation action
D. Action buttons order
E. Keep action buttons persistent
A. Switching profiles easily
If we could have an option to display the dropdown at that menu bar level it would be useful. In the picture above there's also an input to modify the title on the fly, but honestly just having it exposed in the settings is good enough. So something like this would suffice:

B. Clearing all text button
The other part is the clearing text, which I use the red trashcan icon.  So if we could either make it optional or wire up some hotkeys so we could clear and confirm easily that'd be awesome. I'm not sure if other people would want to choose what type of clear they want to perform from the top level (I don't use the in-page timer so maybe most people would want to clear that too?)
D. Action buttons order

If we were able to move a "clear all lines" trash icon out to the main, I'd also recommend sorting them in the order of action: Clear a line, undo clear, clear all lines. (notes & Play could go either before or after). Reason being, it's normal to clear a handful of lines and having the clear a line next to the clear all lines was more likely to have someone misclick clear all lines. Also I'm using the mdiCloseCircle for clear a line - I think this helps keep it distinct from the other trashcan icons in the app.
E. Keep action buttons persistent
One of the things I did notice was that if we make the clear line & undo buttons persistent, it helps with any misclicks. If the buttons disappear based on the lines shown it's easy to click undo clear twice for instance, when you only had one undo clear left, leading to misclicking. The solution here would be just always show them, regardless of if the lines and undo-able things are there are not. I think this was how anacreon's worked too.
Happy to help or clarify any of this. I can also break it into separate issues if you'd prefer, just let me know.

A: Can be a new option - i assume it distracts people / mobile especially if they are not using it
B / C: As i wrote in the faq it was not intended to be used for mobile.. but while i have no specific plans to adjust the ui for now i probably also not want to work against it.. so i prefer two keybinds which respectively can trigger the reset lines / reset data functionality from the settings instead of moving the icon to the menu. For the confirmation then a new option
D: Will change the order - though i will keep the icon for now as i will not move the requested ones to the menu and in the settings they have labels
E: Sounds reasonable


GCP Secret Manager

Awesome project here!
I'm just getting started with using this library, but I noticed in your .Readme you said:
We considered providing a possibility to read the credentials from an external credentials store, such as GCP Secrets, yet that would **require some authentication credentials too**, which brings back the same issue it was to solve.
Is this always true? I am not an expert on Secret Manager, but my understanding is that you must use IAM access controls (see docs here) for Secret Manager to function. If your trading bot is hosted on GCP, and access is properly controlled with IAM throughout, then I fail to see any additional security issues.
Please feel free to correct me if I'm wrong here!
If not, I'd be happy to work on an enhancement for IBeam + Secret Manager.

Hey @mike-greenmmd thanks for the kind words and for checking out IBeam 😊
As for your suggestion - great you bring up this topic to the surface. Please correct me if I'm wrong too, but don't you need to provide a service account credentials (eg a .json file) to be able to access the Secrets Manager?
If it's hosted on GCP it would use the default credentials of the GCP instance, allowing it to access the secrets without providing any input, however that would be a solution which would limit usage of IBeam only to GCP.
Can we get around this somehow?

I'm going to close this issue due to lack of activity. Feel free to reopen if you'd like to continue the discussion 👍 Thanks for participating!


Fix automation

We need to fix auto soon; @cangiuli and I believe strongly that this should be our principle priority at this time, and it needs to happen before we write much more RedPRL code, and before we can really start recommending that people try RedPRL.
A big source of frustration with auto is the small-step nature of it; in order to implement the needed proof search tactics, we need a ton of backtracking (which can be avoided in a big-step version). Instead, we should engineer things so that problems are posed in the order that they can be solved, and that they are solved in one big step, modulo emitting subgoals that require user intervention.
Related is #438; I open this as a separate ticket because I want to track the task of fixing auto regardless of the specificities of that proposal.

What exactly needs to be fixed now?

I think it’s better than it was, but the main problem remains that it is a little unpredictable, often loops, and we don’t really understand how to study the proof search scientifically.

On Feb 9, 2018, at 4:36 PM, favonia<EMAIL_ADDRESS>wrote:
What exactly needs to be fixed now?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Needlessly verbose (or incorrect?) hover-over string for exported value from node package

http://cl.ly/35341u1Y2m2Y
Here<EMAIL_ADDRESS>Looking at the npm docs (https://www.npmjs.com/package/array-flatten), I would have expected the type to look more like array: [...]. The current type string is quite hard to parse, even for somebody that knows JavaScript.

claimed fixed by #54, requires update of sourcegraph


chore(npm): Babelize the lib

In preparation for Webpack

Add a 'prepublish' and 'postinstall' npm scripts
Point package main to /lib/index

Good work, thanks!


#28: Initial integration with network policies

The cluster for integration tests was changed to K3d, so it also implements https://github.com/glothriel/wormhole/issues/73


"Graphing Days" doesn't change graphs

I've read through the graphing refactoring threads, and thought I understood how it works. However, if I change the "Graphing Days" setting in the admin page from 1 to say, 30 -- I don't see any changes.
My concern is that I don't actually have the historical data to pull from, and that it might be purged based on the previous "1 day" setting. Am I out of luck regarding past data, or have I completely misunderstood the graphing changes?
Basically, my boss wants me to show him the hashrates over the past 30 days. If the data isn't in the pool database, I'm out of luck.

It's fetching from cache. Clear your mencache and try again. Fetches data from user statistics table so it should be there.

Hmm, no effect (waited to make sure cron ran).
My graphs still show " Average Hashrate past 24h" -- should that change to 30 days, or am I misunderstanding?
NOTE: Running commit from Feb 7, 2015

Template has 24h hard coded on it. Should probably be updated to show the current setting. Data should be a month though - at least as far as I can tell from the function. There isn't a lot of magic in it and like said before, it does fetch from the user stats table:
https://github.com/MPOS/php-mpos/blob/master/include/classes/statistics.class.php#L701

It looks like my statistics_users table only has 1550 records in it. I'm guessing it was auto-pruned at 1 day perhaps? (ie, before I changed it to 30 days... it'll probably grow now)
Crap.
But thank you for the help. :)

Of course you are right, forgot about the cleanup job. It will grow to your setting now, but all previous data is lost.


fixed a race condition

I got a lot of "Replies out of sync"-errors when running under high load. This fixed it for me.

That's really quite strange - those lines should always happen together if you're running a single threaded application - it shouldn't be possible to get a race condition. I wonder whether you have multiple threads, each one making calls to the same em-hiredis client?

Yes, I think this is the piece of code:
def tick
@redis.blpop('fnordmetric-queue', 0).callback do |list, event_id|
EM.next_tick(&method(:tick))
@redis.get(event_key(event_id)).callback do |event_data|
push_event(event_id, event_data) if event_data
@redis.hincrby(stats_key, :events_processed, 1)
end
end
end

I would either use a separate redis client for the blpop operation (especially if you want to use this redis client in other parts of your code), or call the recursive method after the hincrby. Otherwise your event processing will be blocked by the blpop. Let me know if this helps. However I still don't see why you'd be getting out of sync replies...

Mh, I tried both, but neither helped and that instance of the client isn't used anywhere else. (The blpop is supposed to block, because it pops the next job from the queue). If you like, you can add me on Skype and I'll demonstrate the problem and that the switch of send_command and @defs.push seemingly fixes it. Also, I can't find the reason to issue the command to redis before storing the callback? What am I missing? :)

Hey, I found out, that the "Replies out of sync" only happens, when the call is made from inside a Yajl::Parser.on_parse_complete-callback. I stopped using the stream parser and everything is working fine without the change, so I guess I should close the pull request. Thanks! :)