aws_s3_bucket_website_configuration: An argument named "routing_rules" is not expected here.

Summary
In AWS v3, aws_s3_bucket.website.routing_rules is a string which is a JSON array contains routing rules.
In AWS v4, aws_s3_bucket_website_configuration.routing_rule is a block. We need to parse json and build corresponding blocks representation.
https://registry.terraform.io/providers/hashicorp/aws/3.74.3/docs/resources/s3_bucket#website
https://registry.terraform.io/providers/hashicorp/aws/4.14.0/docs/resources/s3_bucket_website_configuration
Version
$ tfedit version
0.0.3

$ terraform -v
Terraform v1.1.9
on darwin_amd64
+ provider registry.terraform.io/hashicorp/aws v4.14.0

Expected behavior
tmp/iss30/main.tf
before
resource "aws_s3_bucket" "example" {
  bucket = "tfedit-test"

  website {
    index_document = "index.html"
    error_document = "error.html"

    routing_rules = <<EOF
[{
    "Condition": {
        "KeyPrefixEquals": "docs/"
    },
    "Redirect": {
        "ReplaceKeyPrefixWith": "documents/"
    }
}]
EOF
  }
}

$ cat tmp/iss30/main.tf | tfedit filter awsv4upgrade

after
resource "aws_s3_bucket" "example" {
  bucket = "tfedit-test"
}

resource "aws_s3_bucket_website_configuration" "example" {
  bucket = aws_s3_bucket.example.id

  index_document {
    suffix = "index.html"
  }

  error_document {
    key = "error.html"
  }

  routing_rule {
    condition {
      key_prefix_equals = "docs/"
    }
    redirect {
      replace_key_prefix_with = "documents/"
    }
  }
}

Actual behavior
$ cat tmp/iss30/main.tf | tfedit filter awsv4upgrade

resource "aws_s3_bucket" "example" {
  bucket = "tfedit-test"
}

resource "aws_s3_bucket_website_configuration" "example" {
  bucket = aws_s3_bucket.example.id

  index_document {
    suffix = "index.html"
  }

  error_document {
    key = "error.html"
  }

  routing_rules = <<EOF
[{
    "Condition": {
        "KeyPrefixEquals": "docs/"
    },
    "Redirect": {
        "ReplaceKeyPrefixWith": "documents/"
    }
}]
EOF
}

The routing_rules parameter is restored in AWS v4.15.0:
https://github.com/hashicorp/terraform-provider-aws/pull/24198
I confirmed it in the latest v4.15.1. This means that we are no longer need to rewrite routing_rules to routing_rule for upgrading AWS v4 at least. So I'll close this as resolved in the latest AWS provider.


Package request: LuaJIT

Is it possible to add luajit to scoop main bucket? There are binaries out there (like from luapower.com etc.)

Should it download the whole repo? https://github.com/luapower/luapower-all/archive/2018-08-16.zip
Or is the luajit.exe enough? https://github.com/luapower/luajit

@r15ch13 I believe only the luajit Windows 32/64 binaries are necessary. Luapower is a distribution containing luajit binaries and other supporting libraries, and we only need their luajit binaries for the ease of packaging. But single luajit.exe is not enough. See http://luajit.org/install.html

This issue belongs in the Main bucket. Unfortunately, we cannot move this issue on this end, so it is being closed.
If this is still an issue, please create a new issue in the Main bucket via https://github.com/ScoopInstaller/Main/issues/new?body=Moved+from+https%3A%2F%2Fgithub.com%2Flukesampson%2Fscoop%2Fissues%2F2508


Add Android Unit testing

As a developer, I want to increase testing coverage; therefore, let's add some unit test cases to this application/
Acceptance Criteria

I can see some unit tests under: https://github.com/josdem/fruitypedia/tree/main/app/src/test/java/com/josdem/fruitypedia
I can run test cases
All tests are passing on GitHub actions

Unit tests passing: https://github.com/josdem/fruitypedia/actions/runs/7151991991


Changing locale but for a specific configuration

I'm having issues configuring a locale change.
First, I'm setting the locale to english in my angular.module.config and immediately setting the relative time object as I need:
                moment.locale('en', {
                    relativeTime: {
                        future: "In %s",
                        past: "%s ago",
                        s: "<1 minute",
                        m: "1 minute",
                        mm: "%d minutes",
                        h: "1 hour",
                        hh: "%d hours",
                        d: "24 hours",
                        dd: "%d days",
                        M: "1 month",
                        MM: "%d months",
                        y: "1 year",
                        yy: "%d years"
                    }
                });

This works as intended,
Now I also have an equal object but for german. I don't know where to drop it because if I drop it in the config it overrides the english config.
Then I have a select dropdown in a header menu view with EN and DE (I'm already getting it to lower case for translations) in my controller and I can't seem to associate the amMoment.changeLocale('de'); to change the moment part, and I can't seem to update it with moment directly as well.
How can I achieve this?

A little more information: changed it from config to run. Still the same behaviour.
Noticed a strange behaviour on my select: when I change language it catches the language code and moment does apply it, but it does not apply the specific configurations I set in run (wether they are updateLocale, defineLocale or just locale)

Same question here


new docker architecture

Main improvements:

Auto update to latest version
Using pre-compiled files instead of building locally
Always use the latest ffmpeg version
PGID and PUID variables
Create the configuration file dynamically with variables while still allowing persistent customization
TZ variable for Windows and OS X users sync time with host

Looks like there are a lot of unassociated changes. Did you start your branch from the latest version of the master branch?

Yes.

I believe I created the branch here from dev, so I think the issues are my fault.  I just created a new branch from master and started a new pull request and none of those extraneous changes are in it.  I'll start a new pull request on that branch, which is currently named "docker-experimental-2".

OK, that new one [240] is much cleaner, but it has my name on it.  Nottt, if you want you can do another pull request to that branch to replace this one and the test one I just did.  OR we can just use the new one.  Doesn't make any difference to me.

I am not sure you can truly rename a branch in github. I think you'll have to delete that one and create another one regardless (like I saw here  when I tried).
So if you have to create another one anyway I'd prefer if you just delete those 2 here, create a new one clean and then I'll make the pull request again no problem. But either way I'm fine.

OK, I closed my pull request on the docker-experimental-2 branch and renamed it docker.


command  vi icicha.key cat icicha.key icicha.crt > icicha.pem makes vim hang

Steps to reproduce
[root@m5404010 ~]# vi icicha.key cat icicha.key icicha.crt > icicha.pem
Vim: Warning: Output is not to a terminal
^CError detected while processing /etc/vimrc:
line    1:
Interrupted
Error detected while processing /usr/share/vim/vim80/defaults.vim:
line    1:
Interrupted
Interrupt: Press ENTER or type command to continue

Expected behaviour
The command does not make vim hang
Version of Vim
8.0.1763
Environment
rhel 8.6
[root@m5404010 ~]# uname -a
Linux m5404010 4.18.0-305.el8.s390x #1 SMP Thu Apr 29 09:06:01 EDT 2021 s390x s390x s390x GNU/Linux
Mac iterm
Logs and stack traces
No response

do you see Vim: Warning: Output is not to a terminal ?

Vim doesn't hang, the output goes to the file you specified.   Working as expected (probably not intended).


Shouldn't need to URL encode name of provider in verification settings

@bethesque commented on Mon Aug 07 2017
In the documentation for the js workshop, the provider name needed to be URL encoded. I think this should be handled by the publish code, not by the user.
https://github.com/DiUS/pact-workshop-js/#step-13---using-a-pact-broker
let opts = {
  provider: 'Our%20Provider',
  providerBaseUrl: 'http://localhost:8081',
  providerStatesUrl: 'http://localhost:8081/states',
  providerStatesSetupUrl: 'http://localhost:8081/setup',
  pactBrokerUrl: 'https://test.pact.dius.com.au/',
  tags: ['prod'],
  pactBrokerUsername: 'dXfltyFMgNOFZAxr8io9wJ37iUpY42M',
  pactBrokerPassword: 'O5AIZWxelWbLvqMd8PkAVycBJh2Psyg1',
  publishVerificationResult: true,
  providerVersion: '1.1.0'
}

@mefellows commented on Sat Aug 19 2017
Agree, that's a bit shit of us.

I've just reviewed this and it's not actually necessary. Although the README has the escaping, it's actually not in the code, and the code works just fine.
I've updated that README to correct this.
Closing.


HBASE-26152 Exclude javax.servlet:servlet-api in hbase-shaded-testing-util

For branch-2 only?

For branch-2 only?

Yes. TestShadedHBaseTestingUtility passes on master and branch-2.3. Only fails on branch-2 and branch-2.4.

Should be fine to add the exclusions to all branches, or not, I have no strong opinion either way. Better to be consistent across all POMs in all branches is the small preference I suppose.

Should be fine to add the exclusions to all branches, or not, I have no strong opinion either way. Better to be consistent across all POMs in all branches if possible.

This pom.xml is already different on the branches. The hadoop-mapreduce-client-app dependency does not require any exclusions on master branch while the shading rule also differs on branch-2.3. Based on this I'd prefer to commit this to branch-2 and branch-2.4 only.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 36s
Docker mode activated.

-0 :warning:
yetus
0m  9s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 39s
branch-2 passed

+1 :green_heart:
compile
0m 22s
branch-2 passed

+1 :green_heart:
shadedjars
7m 41s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 15s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 23s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
shadedjars
8m 49s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 24s
hbase-shaded-testing-util in the patch passed.

33m 36s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3547

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 8bfafa4dc796 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / aa3b07f6bb

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/testReport/

Max. process+thread count
66 (vs. ulimit of 12500)

modules
C: hbase-shaded/hbase-shaded-testing-util U: hbase-shaded/hbase-shaded-testing-util

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 57s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 23s
branch-2 passed

+1 :green_heart:
compile
0m 27s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 55s
the patch passed

+1 :green_heart:
compile
0m 24s
the patch passed

+1 :green_heart:
javac
0m 24s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
18m 16s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

_ Other Tests _

+1 :green_heart:
asflicense
0m 20s
The patch does not generate ASF License warnings.

43m 48s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3547

Optional Tests
dupname asflicense javac hadoopcheck xml compile

uname
Linux bd9472167d69 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / aa3b07f6bb

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
69 (vs. ulimit of 12500)

modules
C: hbase-shaded/hbase-shaded-testing-util U: hbase-shaded/hbase-shaded-testing-util

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 44s
Docker mode activated.

-0 :warning:
yetus
0m  8s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 57s
branch-2 passed

+1 :green_heart:
compile
0m 30s
branch-2 passed

+1 :green_heart:
shadedjars
9m 31s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 15s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 29s
the patch passed

+1 :green_heart:
compile
0m 25s
the patch passed

+1 :green_heart:
javac
0m 25s
the patch passed

+1 :green_heart:
shadedjars
9m 34s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 27s
hbase-shaded-testing-util in the patch passed.

36m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3547

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux f19ca6d03ba7 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / aa3b07f6bb

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/testReport/

Max. process+thread count
85 (vs. ulimit of 12500)

modules
C: hbase-shaded/hbase-shaded-testing-util U: hbase-shaded/hbase-shaded-testing-util

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3547/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


SVG renders with weird gaps in PDF export

Description
This SVG

renders like this

in PDF export (notably not in web preview nor in PNG export).
Reproduction URL
https://typst.app/project/rX20c4Hxx56g8RFJIQF0TA
Operating system
Web app, Linux
Typst version

[X] I am using the latest version of Typst

Hi, this seems to be a duplicate of https://github.com/typst/typst/issues/4110

Yeah, I think so. This reproduces with rsvg-convert, so probably not a Typst bug. Thanks for the pointer! I guess I’ll just use PNG.

Similar issues have been reported quite often, so look like it does happen a lot... I wish there were something I could do against it, but I don't think there is...


Add various optimizations and cleanups

Original Summary
A bit of a micro-optimization, but many of the Select(...).Where(...) where they check for type could be changed into OfType.
Edited Summary
This PR aims to cleanup some of the codebase, such as cleaning redundancy and perform cast optimization.

Could you rebase this onto the latest dev?


Remap jj to enable Normal mode

How can I remap jj to switch from insert mode to normal mode.

umm... you can't actually
Dooit, currently, only support one keybinding at a time i.e. pressing jj will behave as two j keys pressed


Remove Light Shaders in favor of Fragment Shaders?

Godot version:
3.x
Apologies if this is kind of a radical suggestion from someone who is a noob to this community. I have nonetheless been using Godot for a while now and realized that Light Shaders are a constant pain point whenever trying to do anything interesting. Please let me know if this suggestion is totally out of line.

They don't seem to work correctly and I can't find an example of a "default" lighting shader, and I searched various community forums and there does not seem to be a testable, normal Phong light shader code that is confirmed to work in Godot

Setting DIFFUSE_LIGHT seems to cast no shadows. Shadows only appear when SPECULAR_LIGHT is set

Attempting to adapt the typical GLSL lighting code into a Godot Lighting Shader results does not seem to work properly and it's hard to identify why.

Built-ins are split between Fragment and Lighting shaders, so sometimes useful variables are not available (sorry, I don't have specific examples here as I tend to just try to work around things when they don't work).

Suggestion:

Add array support in shaders

Provide all scene lights and their associated built-ins as an array that can be accessed from Fragment shaders

Remove Lighting shaders

Well i have been thinking the same thing but I believe it's not so you can always use varying and there are other ways to access things that you might need although I would really like at least a doc update on the topic of Lighting Shaders, cause I fail to understand how they have been integrated into the Godot Shading System.

I'm not sure that Godot's rendering architecture is compatible with giving arrays of lights to the fragment shader. Or at least the performance would be significantly worse.

Isn't it already looping through the lights internally here: https://github.com/godotengine/godot/blob/ec7a8e7d63e31cca5472b662170faab7935482f1/drivers/gles3/shaders/scene.glsl#L2016
Could the light_compute method not simply be exposed in Godot shaders and allow users to override it?

@nodefish I think Godot uses a clustered rendering architecture which gives each fragment only a subset of the scene's lights, but it's true that it should be possible to expose.

If it loops through every light in your scene, and you have anything approaching the type of lighting you have in a serious game, be ready for framerates in the 1fps range.
Godot, as well as many other engines, will limit the number of lights it needs to evaluate per fragment and just evaluate those which if I am not mistaken is indeed done with a clustered rendering approach in Godot 3.0 as Raymoo suggests.
The way the shaders are implemented in Godot make a lot of sense. If you merge the fragment and lighting shaders into a single shader all you end up doing is recreating exactly what it does today.
The question then is, what are you attempting to do and are you doing it in the right place? There are very few scenarios I can think off where you would want to write your own lighting shader.
The lighting shader is responsible for taking the colour output of the fragment shader and applying the calculations so that it is properly lit by the light it's handling. If 5 lights effect the fragment, the lighting shader is run 5 times and the accumulation of lighting of the 5 lights is added together to get the final colour. The shadow calculation should be handled after your code ends and is determined by the LIGHT_VEC output. If you never change that its default value should result in normal shadow calculations being applied.

Thanks for the detailed writeup. I guess my understanding of Godot's lighting shaders is probably limited.
One thing that I was struggling with was implementing simple banded lighting, without modifying LIGHT_VEC, and only writing to DIFFUSE_LIGHT, yet shadows were eliminated and the results were off and somewhat hard to debug.
I think I'm better off closing this issue and reporting any individual issues I encounter when working with Light shaders.
Thanks


Upgraded to Get-WinEvent cmdlet

Issue #, if available:
Description of changes:
According to Microsoft, Get-EventLog uses a Win32 API that is deprecated and results provided may not be accurate. Instead, it is recommended that Get-WinEvent is used. This PR changes the log fetching mechanism to use that Cmdlet. Reference: https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.management/get-eventlog?view=powershell-5.1
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Testing Done

Ran the changes locally on a Windows instance
Verified the CSV files based on RecordID number. Also ensured all the necessary fields are available (
See this guide for recommended testing for PRs. Some tests may not apply. Completing tests and providing additional validation steps are not required, but it is recommended and may reduce review time and time to merge.

Test Result
Old Kubelet CSV

New Kubelet CSV

Looks good to me!  Thanks!


Refactor SwiftPMConfig

As discussed in #2995, the SwiftPMConfig type presents some API design challenges:

The SwiftPMConfig type is responsible for getting and setting the mirrored URLs for dependencies. For example, if a package has a direct or transitive dependency on https://github.com/apple/swift-argument-parser.git, you can use a mirror to instead point all references to that package to a fork, say https://github.com/mattt/swift-argument-parser. This functionality was discussed on the Swift Forums in this thread.
The name SwiftPMConfig is vague, and makes it difficult to understand the significance of config properties and parameters that are passed around among the PackageGraph, SourceControl, and Workspace modules. In other contexts, the word "config" is used to describe pkgConfig, build configuration, or xcconfig files.
This comment indicates that the name was chosen in anticipation of the type being used to manage other concerns.

In the feedback on that PR, @neonichu recommended making mirrors a subtype / subproperty on SwiftPMConfig, thereby not closing off the opportunity to extend SwiftPMConfig to other purposes at some point in the future.
This PR follows that guidance — to great effect, I believe.
First, I pulled in some of the other changes from #2995, like reimplementing SwiftPMConfig to more closely match PinsStore (51c282f4073cf5cd81df36a65ba49e23e3cc6469) and adding test coverage (9089f5d7f2721f8c1b820b3740a469f40352b1a5). I also added documentation (dbecb53ceeafba1f14dd27c2c5ca5275dc46bd08), and in the process saw an opportunity to improve clarity by renaming mirroredURL(forURL:) to effectiveURL(forURL:) (da0eac720b50efab22108524ffda328342d47a59).
From there, I made a DependencyMirrors type, which conforms to Collection and acts as a proxy for getting and setting mirrors, and re-routed all existing calls through that (48f77f4e1a32a0269bd01d5ba516996e36d97ccd). Along the way, I noticed that the only method being called on mirrors outside of the Command module was that renamed effectiveURL(forURL:) method from before. Thanks to that, I was able to find a solution to the problem of all of this API design awkwardness:

Move SwiftPMConfig to the Workspace module
Make DependencyMirrors a standalone type in the PackageGraph module

Thanks to these separation of concerns, we can now perform all of our dependency resolution tasks without needing to know about SwiftPMConfig at all. Each config parameter or property can be (and was) replaced by a corresponding one named mirrors. And should SwiftPMConfig ever expand its scope, we can always add another property / argument to the relevant call sites as needed.
The one notable thing I punted on so far with this PR was renaming SwiftPMConfig. It doesn't need a new name, but its repositioning in the project gives us an occasion to revisit that decision. I think Workspace.Configuration is an improvement, but I'm interested to hear any other ideas.

@swift-ci please smoke test

@swift-ci please smoke test

CI is failing for the same reason discussed in https://github.com/apple/swift-package-manager/pull/2995#issuecomment-715538725.
/home/buildnode/jenkins/workspace/swift-package-manager-Linux-smoke-test/branch-main/sourcekit-lsp/Sources/SKSwiftPMWorkspace/SwiftPMWorkspace.swift:96:15: error: call can throw but is not marked with 'try'
15:51:34       config: SwiftPMConfig(path: packageRoot.appending(components: ".swiftpm", "config"), fs: fileSystem),
15:51:34               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
15:51:34 /home/buildnode/jenkins/workspace/swift-package-manager-Linux-smoke-test/branch-main/sourcekit-lsp/Sources/SKSwiftPMWorkspace/SwiftPMWorkspace.swift:96:15: note: did you mean to use 'try'?
15:51:34       config: SwiftPMConfig(path: packageRoot.appending(components: ".swiftpm", "config"), fs: fileSystem),
15:51:34               ^
15:51:34               try
15:51:34 /home/buildnode/jenkins/workspace/swift-package-manager-Linux-smoke-test/branch-main/sourcekit-lsp/Sources/SKSwiftPMWorkspace/SwiftPMWorkspace.swift:96:15: note: did you mean to handle error as optional value?
15:51:34       config: SwiftPMConfig(path: packageRoot.appending(components: ".swiftpm", "config"), fs: fileSystem),
15:51:34               ^
15:51:34               try?
15:51:34 /home/buildnode/jenkins/workspace/swift-package-manager-Linux-smoke-test/branch-main/sourcekit-lsp/Sources/SKSwiftPMWorkspace/SwiftPMWorkspace.swift:96:15: note: did you mean to disable error propagation?
15:51:34       config: SwiftPMConfig(path: packageRoot.appending(components: ".swiftpm", "config"), fs: fileSystem),
15:51:34               ^
15:51:34               try!

A few options for resolving this issue:

Pinning SwiftPM dependency in sourcekit-lsp to semantic version
Coordinating simultaneous PR on sourcekit-lsp with corresponding API changes
Reverting to previous API

Of these, I think 1 is the best (and probably what we should be doing anyway; dependencies on branches are fragile). 2 could work but is risky and will almost certainly break the build temporarily. Otherwise, 3 is always an option.
@neonichu @abertelrud Any preference among these three options? (Or any other ideas?)

Coordinating simultaneous PR on sourcekit-lsp with corresponding API changes

If I understand correctly, this is how it's usually done here. You open multiple PRs across different repositories and trigger CI with PR conditions so that it pulls exact PRs across multiple repos to test with. When everything passes, you merge it all at once so that the amount of time that the build is broken is minimized.

@swift-ci please test

I like this change allot. I am also +1 on renaming to Workspace.Configuration which is easier to reason about.
Incidentally, some of the work that will follow up on https://github.com/apple/swift-package-manager/pull/3007 includes some similar concepts so we would want to harmonize them as these things land

Incidentally, some of the work that will follow up on #3007 includes some similar concepts so we would want to harmonize them as these things land

Oh, interesting. Do you have a link you can share with more information about this feature?

@swift-ci please test

Oh, interesting. Do you have a link you can share with more information about this feature?

see https://forums.swift.org/t/package-feeds/41522, the "Configuration file" section is the relevant one. I can see some of the mirror and/or mirror like configuration being beyond workspace.

Thanks @mattt, this is great.  Refactoring so that the configuration type (whatever it's called) is just a holder of more specific types that the logic then reasons about is good.  I also think that Workspace.Configuration is a good name.
Regarding the revlock issue:  we really want to get to the point where the various packages here are using semantic versioning, but for the moment, with the existing branch-based dependencies, I believe @MaxDesiatov is right that opening separate PRs is how it's usually been done.  But in my view we need to quickly get to the point where we're using semantic versioning for the dependencies, even if that means having to increment the versions often as we don't yet have a stable API.

Thanks for reviewing, @abertelrud & @tomerd. I just committed a2876c7, which takes that final step to rename SwiftPMConfig to Workspace.Configuration.
I also opened https://github.com/apple/sourcekit-lsp/pull/344 which has the corresponding changes in sourcekit-lsp. There may be other Apple OSS projects that require an update; I'm only aware of this one because CI stopped there.

apple/sourcekit-lsp#344
@swift-ci please smoke test

Regarding the revlock issue: we really want to get to the point where the various packages here are using semantic versioning, but for the moment, with the existing branch-based dependencies, I believe @MaxDesiatov is right that opening separate PRs is how it's usually been done. But in my view we need to quickly get to the point where we're using semantic versioning for the dependencies, even if that means having to increment the versions often as we don't yet have a stable API.

@abertelrud what is your plan here?  Is there work happening to guarantee swiftpm is stable with respect to build directory layout and intermediate file formats?  Rev-lock is currently the only guarantee of compatibility between sourcekit-lsp and the package manager in the toolchain.

@benlangmuir  This is currently a goal, not a plan.  The plan would be to first define what's part of the API for libSwiftPM (such as the file formats and the semantics you mention), and then to start versioning it.  We would absolutely need to coordinate that with SourceKit-LSP.  My comments were just pointing out that as libSwiftPM gains more clients, it becomes less feasible to manage sets of interlocking changes.


fix(aip_xx1_description): fix missing dependency

Signed-off-by: Takayuki Murooka<EMAIL_ADDRESS>As written in the title

It seems I had some mistake since the build did not pass. I will look into it, and make another PR


Render on server e.g. integrate into angular universal?

[ x] feature request
[ x] support request => Please do not submit support request here

Is it possible to use this component with project like angular universal and render the tables already on the server. And on the client using the sorting and filtering stuff? I tried to integrate it into my universal project without success, because the component of course is accessing the window document object.
Any solutions or ideas on that or is it not supposed to do this?

I haven't had a need for universal support but happy to accept a PR for it. I'm sure its got some minor tweaks and it would work.

of course I would make a PR, I'm just a bit new to the universal environment. Are there any sample implementations for similar scenarios or do you have any advice how to shim/inject specific functionality depending on the environment?

I can help with some advice on this

@gdi2290 - Ears are wide open :)

@amcdnl @gdi2290 any updates on this issue? since document is not defined on the server side it throws errors when using with angular-universal

@Quixomatic - where is the document reference breaking it?

/node_modules/angular2-data-table/release/index.umd.js:285
var testStyle = document.createElement('div').style;
^
ReferenceError: document is not defined
at /home/ubuntu/workspace/node_modules/angular2-data-table/release/index.umd.js:285:17

@amcdnl any thoughts on this?

I would love to use this data table, it looks fantastic, but I'm getting the same error. Is there anyway I can implement a workaround to get your data table working in my application?
Thanks.

@scotteby I was able to get this working with angular-universal with a bit of a temporary hack:
In the release folder if you open index.js and replace the following lines:
line 3199:
var testStyle = typeof document != 'undefined' ? document.createElement('div').style : [];
line 3203:
var styles = typeof window != 'undefined' ? window.getComputedStyle(document.documentElement, '') : [];
line 3204:
var pre = typeof window != 'undefined' ? (Array.prototype.slice.call(styles).join('').match(/-(moz|webkit|ms)-/))[1] : null;
line 3205:
var dom = typeof window != 'undefined' ? ('WebKit|Moz|MS|O').match(new RegExp('(' + pre + ')', 'i'))[1] : null;
line 3210:
js: typeof window != 'undefined' ? pre[0].toUpperCase() + pre.substr(1): ''
line 3581:
var ua = typeof window != 'undefined' ? window.navigator.userAgent : null;

I also added nested my  component inside of a div with a *ngIf="isBrowser"
isBrowser being a boolean variable that you can import from angular-universal that will not render the datatable component until you're loading things on the client side.

@gdi2290 Is there better ways to get reference to the window using the renderer or something?

@amcdnl Think they're working on implementing jsdom into the server side of things to make all of this easier.

Thanks @Quixomatic , this works great in our app now. Really appreciate the quick turnaround on providing a fix for us.
By the way, I did have have set the isBrowser flag to true in the ngOnInit method inside my component to get the table to render.
import { isBrowser } from 'angular2-universal';

export class MyComponent {

 isBrowser;

 ngOnInit()
    {
        this.isBrowser = true;
    }
}

@scotteby this function worked fine for me after importing isBrowser into a shared service, would work importing into a component and defining the function there as well.
    isBrowser () {
        return isBrowser
    }

Then my *ngIf="isBrowser()"

Thanks for the suggestion, that does make it a little easier to implement.

I'm trying to use the isBrowser workaround but I get the exception by just importing NgxDatatableModule into my app module. Any ideas on what am I missing?

I am having some trouble resolving what I believe to be a related issue, while trying to update from angular2-data-table (1.7.0) to ngx-datatable (5.0.0).  Still using universal.  Changing index.js to check for undefined document/window as shown earlier has fixed the "document/window not defined" exceptions, but now I am stuck on a similar exception:

ReferenceError: MouseEvent is not defined at ...\release\index.js:1301:42

Any advice on how to finish patching index.js for the new version would be greatly appreciated!

@andreschort I am getting the same issue.  Simply importing NgxDatatableModule is causing the exception.

@amcdnl are there any updates regarding support for Angular Universal? This is an amazing component and it would be great to be able to leverage the server-side rendering capabilities of Universal for improved performance.

I believe this is working in latest. Can anyone confirm? We aren't manipulating any dom or using document directly anymore.

I just tested this using ngx-datatable 8.0.0 with Angular 4.0.2 and still get an error when it tries to render on the server.

Exception: Call to Node module failed with error: Prerendering failed because of error: ReferenceError: document is not defined at Object../src/utils/prefixes.ts

I just went through the entire code and got it working with Universal server side rendering, but had to add a check for all the undefined  Window, Document, MouseEvent, and Keyboard event references in the following files:
prefixes.ts
translate.ts
elm-from-point.ts
body-cell.component.ts
body-row-wrapper.component.ts
long-press.directive.ts
resizeable.directive.ts
Is it too much of a hack to just add those undefined checks into the source?

Can you PR that back plz?? @scotteby

I was trying to render ngx-datatable using ng4 and noticed issues with 2 things .. scrollBar width ..  I found a way to use DI to replace service (with manual changes locally) to get the scrollbar width based on server param / user agent. Next I am running into issues with DataTable.recalculateDims trying to use
        var dims = this.element.getBoundingClientRect();

This is causing error in server side rendering
It looks like width and height of grid element must be determined up front for server side rendering to work. I am thinking I can separate this into a service and DI to inject the expected dimensions if client passes the viewport height / width as a server param.
Thoughts?

@earlyster one thing I noticed with scrollbar width is that they are browser specific. You could very well set the width to the most popular browser and adjust after the server rendering if it's not correct. Another would be for the server side rendering to know which browser it's talking to cough agent string, and have a pre-computed width for each vendor, with a default fall-through for unknown.
Another approach altogether is to virtualize the scrollbar and make it a theme instead, allowing the developer to pick how the scrollbar looks.

Hi,
I get this error message in SSR:
 this.document.createElement is not a function
          at ScrollbarHelper.getWidth

@earlyster Did you solve your issues with var dims = this.element.getBoundingClientRect();?
I'm hitting up against this error too, and I'd be happy with your solution where the viewport width is passed as a server param. (As for viewport height, my table just needs to be as long as it needs to be given the width and the dataset length. I guess I could get away with just passing some arbitrarily very long viewport height?)
thanks

I have created a pull request here - https://github.com/swimlane/ngx-datatable/pull/1178

Please look into this issue..#1038

@amcdnl Please look into this issue..#1038


fails to compile with the nightly toolchain

the latest version (0.6.3) doesn't work with nightly
#cargo.toml
[dependencies.prisma-client-rust]
git = "https://github.com/Brendonovich/prisma-client-rust"
tag = "0.6.3"
default-features = false
features = ["postgresql"]

[dependencies.prisma-client-rust-cli]
git = "https://github.com/Brendonovich/prisma-client-rust"
tag = "0.6.3"
default-features = false
features = ["postgresql"]

cargo build
...

Compiling miniz_oxide v0.6.2
error: there is no argument named `err_str`
   --> /home/lordie/.cargo/git/checkouts/prisma-engines-8e1416dd0e571d11/06a1b97/libs/datamodel/core/src/configuration/datasource.rs:94:44
    |
94  |   ...   let s = indoc::formatdoc! {"
    |  __________________________________^
95  | | ...       {err_str}
96  | | ...
97  | | ...       To use a URL with protocol `prisma://` the Data Proxy must be enabled via `prisma generate --dat...98  | | ...
99  | | ...       More information about Data Proxy: https://pris.ly/d/data-proxy
100 | | ...   "};
    | |_______^
    |
    = note: did you intend to capture a variable `err_str` from the surrounding scope?
    = note: to avoid ambiguity, `format_args!` cannot capture variables when the format string is expanded from a macro

   Compiling tokio-rustls v0.23.4
   Compiling flate2 v1.0.25
error: could not compile `datamodel` due to previous error
warning: build failed, waiting for other jobs to finish...

manually cloning https://github.com/prisma/prisma-engines at 06a1b97 gave me the same erros (as expected)

Given how large the prisma engines are, I've said before that I'm not interested in supporting nightly (#88). Maintaining a fork for database connector feature gating is already a lot of overhead, I don't want to make large changes or update dependencies just to support nightly.

@Brendonovich Would you be willing to fix this issue in exchange for some payment?

@proudmuslim-dev I appreciate the offer, but I'm more concerned about time than money for this issue. Committing to nightly support would just be another thing to spend time on, and it wouldn't benefit the place where I actually make money (Spacedrive). I'll speak to the Prisma team about supporting nightly but I can't promise anything.

Normally I don't make offers like this, and I'm sorry in advance if it offends you but I'm just in a hurry with a project I'm writing and I'd rather not lock on to an older nightly version (I need to stay on nightly due to other requirements the project has)

@Brendonovich Fair enough, thank you

For those who wish to keep going on nightly, put this in your rust-toolchain.toml
[toolchain]
channel = "nightly-2022-12-29"

nightly-2022-12-29 is the latest version that will compile


[Bug]: Does not allow editing Column ID with non-English characters

Contact Details
<EMAIL_ADDRESS>What happened?
In the table settings, I added an existing column, for example, "стоимость". The table shows the values of this column. These values can be edited. But if you open the column settings, you cannot change or save the Column ID. Writes: "Error saving ID"

What platform were you using?
Desktop
Version of the plugin
3.5.1
Relevant log output
No response
Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

See src/components/modals/columnSettings/handlers/ColumnIdInputHandler.ts
        // Validate special characters in root key
        if (rootKey.match(/[^a-zA-Z0-9_]/)) {
            return t("column_settings_modal_column_id_error_invalid_key");
        }

Chat GPT:
rootKey.match(/[^\w\u0400-\u04FF]/)

In this pattern:
\w includes word characters (equivalent to [a-zA-Z0-9_]).
\u0400-\u04FF includes the Unicode range for Cyrillic characters.
So, [^\w\u0400-\u04FF] will match any character that is not a word character (letters, digits, or underscore) and not a Cyrillic character.

@nerkuda Comment some cyrillic ids you'd like me to test

место_покупки and место_покупки do work thusfar

Solved in #1023


Picomenu Updates

Updated securecall for Spellbook.
Removed Challenges item.
Added Pets, Heirlooms, and Wardrobe items.
Added invisible line above addon menu to add extra width.

Added invisible line above addon menu to add extra width.

What was the reason for this? Did it look cramped or something?

Some of the text was running into the icons and that was the easiest way I could think of to add a bit of padding. This is what it looks like in game with the changes.
http://imgur.com/K8Zc7FG

Thanks!


[Day1] Write FizzBuzz In java

ToDo:

Create a solutions branch
Follow TDD and implement FizzBuzz using JUnit5 in JDK8

Acceptance Criteria:

Solution adhered to Instructions specified here
Tests are written in JUnit5
All tests are green
Relevant docs are updated for the solution

The solution is provided in the solutions branch.


[Vanilla] Missing incantations: Aspects of the Crucible Breath and Horns + typo fix

data/lists/spells.ts misses these incantations:
      {
        id: "d87d0166-3de2-4692-b299-6a7e803c4f88",
        description: rawHTMLLink(
          "https://eldenring.wiki.fextralife.com/Aspects+of+the+Crucible:+Breath",
          "Aspects of the Crucible: Breath"
        ),
      },
      {
        id: "c74ca270-f12d-4aca-b418-cfb830a05ea9",
        description: rawHTMLLink(
          "https://eldenring.wiki.fextralife.com/Aspects+of+the+Crucible:+Horns",
          "Aspects of the Crucible: Horns"
        ),
      },

Typo fix for data/lists/quests.ts needed (aspect to aspects):
      {
        id: "e4b0b2fc-7a40-4af3-94b6-1e65db9e3eac",
        description:
          "Tanith's Knight will invade. Kill and loot the Knight for Aspects of the Crucible: Breath.",
      },

Thank you for catching these!

Typo fix for data/lists/quests.ts needed (aspect to aspects):
      {
        id: "e4b0b2fc-7a40-4af3-94b6-1e65db9e3eac",
        description:
          "Tanith's Knight will invade. Kill and loot the Knight for Aspects of the Crucible: Breath.",
      },

Fixed in 7c2575c

data/lists/spells.ts misses these incantations:
      {
        id: "d87d0166-3de2-4692-b299-6a7e803c4f88",
        description: rawHTMLLink(
          "https://eldenring.wiki.fextralife.com/Aspects+of+the+Crucible:+Breath",
          "Aspects of the Crucible: Breath"
        ),
      },
      {
        id: "c74ca270-f12d-4aca-b418-cfb830a05ea9",
        description: rawHTMLLink(
          "https://eldenring.wiki.fextralife.com/Aspects+of+the+Crucible:+Horns",
          "Aspects of the Crucible: Horns"
        ),
      },

Fixed in c7cc580


unable to get certificate via cli

Which version of python are you using?
2.7
What operating system and version of operating system are you uing?
ubuntu 16.04
What version of sewer are you using?
0.2.6
What did you do? (be as detailed as you can)
ran:
sewer --dns "cloudflare" --email some-meail  --domains some-domain --action run --endpoint staging
usage: sewer [-h] [--version] [--account_key ACCOUNT_KEY] --dns {cloudflare,
             aurora} --domains DOMAINS [--bundle_name BUNDLE_NAME]
             [--endpoint {production,staging}] [--email EMAIL] --action
             {run,renew}
sewer: error: argument --dns: invalid choice: 'cloudflare' (choose from 'cloudflare, aurora')

What did you expect to see/happen/not happen?
get a certififcate from staging
What did you actually see/happen?
usage: sewer [-h] [--version] [--account_key ACCOUNT_KEY] --dns {cloudflare,
             aurora} --domains DOMAINS [--bundle_name BUNDLE_NAME]
             [--endpoint {production,staging}] [--email EMAIL] --action
             {run,renew}
sewer: error: argument --dns: invalid choice: 'cloudflare' (choose from 'cloudflare, aurora')

Paste here the log output generated by sewer, if any. Please remember to remove any sensitive items from the log before pasting here.
usage: sewer [-h] [--version] [--account_key ACCOUNT_KEY] --dns {cloudflare,
             aurora} --domains DOMAINS [--bundle_name BUNDLE_NAME]
             [--endpoint {production,staging}] [--email EMAIL] --action
             {run,renew}
sewer: error: argument --dns: invalid choice: 'cloudflare' (choose from 'cloudflare, aurora')

This would have been caught if: https://github.com/komuW/sewer/issues/52 was in place.

The error, I think was caused by: https://github.com/komuW/sewer/pull/38
specifically: https://github.com/komuW/sewer/blob/83c619a63058a08d29e3a7c1921952b295f309bc/sewer/cli.py#L50


Fix bad title bar path when viewing files not in a project folder

In cases where the active pane item has a path that is not inside a current project folder, we display the first project folder in the title bar. This can be misleading. For instance, with a project setup like this:
project
└ folder1
  └ package.json
└ folder2
  └ package.json

If we open ~/random/path/package.json, the title bar will show package.json — folder1, which is incorrect. This made more sense previously, when the project only contained a single folder, but now the behavior seems misleading.
In order to provide the most pertinent information, this PR changes the behavior to the following:

If the active pane item has a path that is included in a project folder, we show item title — project folder (as is the current behavior)
If the active pane item has a path that is NOT included in a project folder, we show item title — path to file (where the path does not include the filename — in the example above, it'd be ~/random/path)
If there is no active pane item (i.e., the pane is empty), we simply show untitled

Fixes #5058
Fixes #5585
Fixes #10710
/cc @atom/feedback

Awesome. Thanks for fixing this!

Looks good to me! 👍

Looks like this introduced a regression where the window title is untitled when there are no pane items open: https://github.com/atom/atom/issues/13647.
Investigating a solution now.


craching while starting

events.js:141
      throw er; // Unhandled 'error' event
      ^
TypeError: this is not a typed array.
    at Function.from (native)
    at /crypty/cryptii/node_modules/gulp-autoprefixer/index.js:25:27
    at run (/crypty/cryptii/node_modules/esdoc/node_modules/core-js/modules/es6.promise.js:89:39)
    at /crypty/cryptii/node_modules/esdoc/node_modules/core-js/modules/es6.promise.js:100:28
    at nextTickCallbackWith0Args (node.js:419:9)
    at process._tickCallback (node.js:348:13)

Gulp is working fine for me. Make sure you have the correct node version v6.11.0 installed.

fixed it with the old version tnx! keep up the good work
Met Vriendelijke groet,
Maarten Rosier
06-34447304
www.maartenrosier.com
<EMAIL_ADDRESS>2017-07-04 17:30 GMT+02:00 Fränz Friederes<EMAIL_ADDRESS>
Gulp is working fine for me. Make sure you have the correct node version
v6.11.0 installed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cryptii/cryptii/issues/6#issuecomment-312905190, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADQtD94hY1-WOJAzEVpCj_0gdGTPJeh5ks5sKlqqgaJpZM4ONdXi
.

Thank you! Will do!


Exemplify the run-test: feature of opam

Currently needs to set OPAMWITHTEST=true manually.

Looks good, excerpt of the logs: (direct link) (available for 3 months).
  Processing  1/2: [coq-demo: make test]
  + /usr/bin/make "test" (CWD=/home/coq/.opam/4.07.1+flambda/.opam-switch/build/coq-demo.dev)
  - for f in tests/test_demo.v ; do \
  -   "coqc"  -q   "-w" "-deprecated-native-compiler-option" "-native-compiler" "no" -R src Demo $f ; \
  - done
  - thm : forall P : Prop, ~ (P <-> ~ P)
  -      : forall P : Prop, ~ (P <-> ~ P)
  -> compiled  coq-demo.dev


uninitialized constant Aquatone::Commands::Takeover (NameError)

Ran gem update and am getting " uninitialized constant Aquatone::Commands::Takeover (NameError)" when running aquatone-takeover -d test.com.
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/aquatone-0.3.0/exe/aquatone-takeover:113:in <top (required)>': uninitialized constant Aquatone::Commands::Takeover (NameError)
`
ideas?

Does it help if you gem uninstall aquatone and gem install aquatone?

Still getting the same error after an uninstall and install.
I was able to clone the repo fresh and gem install aquatone and it seems to have corrected the issue. Thanks!


error: "failed calculating diff pairs for exported snapshot

Hi,
when using the build-task i am getting the following error in the last step:
#12 exporting to image

#12       digest:
sha256:0be3e8ffcff73e71831a5c7e63ba0124fc0fc779b08a6f8a49a1af9b94eea7af

#12         name: "exporting to image"

#12      started: 2019-08-05 12:14:57.909369453 +0000 UTC m=+63.845516601

#12 exporting layers

#12    completed: 2019-08-05 12:15:01.400674152 +0000 UTC m=+67.336821294

#12     duration: 3.491304693s

#12        error: "failed calculating diff pairs for exported snapshot: mount callback failed on /run/user/1000/containerd-mount895903867: mount callback failed on /run/user/1000/containerd-mount431519333: failed to write compressed diff: failed to create diff tar stream: stat /run/user/1000/containerd-mount431519333/opt/app/node_modules/ansi: no such file or directory"

any ideas? is it an upstream error in https://github.com/genuinetools/img ?

No clue, sorry. It's probably an issue upstream; there's not a whole lot going on in our own code. Does building the same image fail with Docker?

If it helps, I had a RUN npm prune --production line in my Dockerfile, which was causing this error. As soon as I removed this line, the build succeeded...
YMMV :)

If it helps, I had a RUN npm prune --production line in my Dockerfile, which was causing this error. As soon as I removed this line, the build succeeded...
Although the same Dockerfile works fine when built with docker
YMMV :)

I am also using npm prune --production, thanks for the help. I will try with the new task some time.


HTML-tables not rendered correctly (markdown mode)

First of all - thank you for Bookstack! It's really amazing! We're using it quite heavily and enjoying it's features - especially the diagrams.net integration!

Describe the bug
As described above, we use it for most of our documentation, which includes project and meeting protocols, which includes a common header on the respective pages (for which we're using the page template feature). Atm this header is defined in plain HTML.
[![Logo](https://profilersuzanne.com/wp-content/uploads/2018/02/logo-dummy.png)](https://profilersuzanne.com/wp-content/uploads/2018/02/logo-dummy.png)| |
---: | :--- | ---:
Betreff   | Titel eingeben |
Zeit | xx.xx.2020, xx:00 Uhr - xx:00 Uhr |
Ort   | 104/103 |
Teilnehmer   | us and the others|

Since the update to v0.31.2, rendering of this table is broken in the final rendering view (it still works in the edit-preview). I think we can change our header to markdown instead of HTML, however, HTML code should be usable from inside MD, so still a bug :)
I think this is somehow related to #2452 and #2470, however the rendering is still not working after upgrading to v0.31.3.
Steps To Reproduce
Steps to reproduce the behavior:

Create a new page (markdown editor)
Copy the HTML code above
Be happy to see a beautiful table on the right ;)
Save the page
See the broken rendering and start crying :D

Expected behavior
Ah, it should be rendered correctly (and identically) in both preview and final rendering.
Screenshots
Correct rendering in edit/preview:

Broken rendering:

Your Configuration (please complete the following information):

Bookstack version: tried both v0.31.2 and v0.31.3
PHP Version:

$ php --version
PHP 7.2.24-0ubuntu<IP_ADDRESS> (cli) (built: Oct  7 2020 15:24:25) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.2.24-0ubuntu<IP_ADDRESS>, Copyright (c) 1999-2018, by Zend Technologies

Hosting Method (Nginx/Apache/Docker): nginx/1.14.0

Additional context

Hi @nightsparc,
Thanks for raising this with the clear examples.
For v0.31 I updated pages to be markdown-rendered on the back-end, to support the API and provide some options in the future. As you've seen there have been some differences now between the preview renderer and the final (Upon save) renders used which I've been trying to close the gap on.
In this case, I think it just may be that the front-end render is a little more forgiving to possible syntax. I was able to make your pasted markdown table code work by adjusting the second line to just:
---: | :--- |

The pipes infer a boundary, both between cells and on the outer edges of cells. All other rows could be seen as two cells but that second line had three cells of content, which I think threw the render off.
BTW, the code you've provided is markdown syntax (Table and image) with HTML sprinkled in (


Fix the issue with dispatching to jax.vmap with qjit(vmap)

This PR fixes the issue with compiling catalyst.qjit(catalyst.vmap(fn))(*args, **kwargs)
For example, this example should pass now
dev = qml.device("lightning.qubit", wires=1)

@qml.qnode(dev)
def circuit(x, y):
    qml.RX(jnp.pi * x[0] + y, wires=0)
    qml.RY(x[1] ** 2, wires=0)
    qml.RX(x[1] * x[2], wires=0)
    return qml.expval(qml.PauliZ(0))

def cost(x, y, z):
    return circuit(x, y) * z

vmapped_cost = qjit(vmap(cost, in_axes=(0, 0, None)))

x = jnp.array([[0.1, 0.2, 0.3],
               [0.4, 0.5, 0.6],
               [0.7, 0.8, 0.9]])
y = jnp.array([jnp.pi, jnp.pi / 2, jnp.pi / 4])

vmapped_cost(x, y, 1)

@erick-xanadu Feel free to merge it.. :raised_hands:

Can I merge this? :)

Can I merge this? :)

I want this one first: https://github.com/PennyLaneAI/catalyst/pull/568
After this, yes. I'll merge both of them as soon as possible and ping you @josh146 :)


Multiple metrics in single call

I am trying to integrate GA4 php library. I only get first metrics value in API response. I want to get multiple metrics values in one single call. waiting for reply

Could you please give me some steps to reproduce this issue?

Closing this due to inactivity. Please feel free to open a new issue if you still need help on this, Thank you!


RxTextView returns a null editable initially

here's what my textview looks like:
<TextView
          android:id="@+id/textview"
          android:layout_width="wrap_content"
          android:layout_height="wrap_content"
          android:hint="example"
          android:maxLines="1"
          android:freezesText="true"
      />

this is what the call i am making looks like:
 return RxTextView.afterTextChangeEvents(textview)
        .map(new Func1<TextViewAfterTextChangeEvent, Boolean>() {
          @Override
          public Boolean call(TextViewAfterTextChangeEvent changeEvent) {
              // changeEvent.editable() is null

          }
        });

With what version?
On Fri, Mar 3, 2017 at 5:21 PM sreejithraman<EMAIL_ADDRESS>wrote:

here's what my textview looks like:

this is what the call i am making looks like:
return RxTextView.afterTextChangeEvents(textview)
.map(new Func1<TextViewAfterTextChangeEvent, Boolean>() {
@Override
public Boolean call(TextViewAfterTextChangeEvent changeEvent) {
// changeEvent.editable() is null
      }
    });

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JakeWharton/RxBinding/issues/369, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEEERmWeXlHimfqvv2maNsoKMOIfC8zks5riJJYgaJpZM4MS0DC
.

1.0.1

With current I suppose:
https://github.com/JakeWharton/RxBinding/blob/master/rxbinding/src/androidTest/java/com/jakewharton/rxbinding2/widget/RxTextViewTest.java#L164

I think this is due to the fact that the initial emitted value is obtained with view.getEditableText(), but a TextView shouldn't be editable by default, so it always returns null.
If you want to obtain a non null result (the text in the TextView) you should set android:editable="true", even though this is deprecated.

Yeah I don't have a good answer here beyond what @Joseph82 said. Since the event uses an Editable type there's not much we can do. I don't really want to write a custom Editable implementation just for the initial event should it return null.
I think a documentation change is probably all we can do.


[WIP] Add a marginal_distribution function

Summary
This commit adds a new function marginal_distribution which performs
marginalization similar to the existing marginal_counts() function. This
new function however differs in a few key ways. Most importantly the order
of the bit indices is significant for the purpose of permuting the bits
while marginalizing, while marginal_counts() doesn't do this.
Additionally, this function only works with dictionaries and not Result
objects and is written in Rust for performance. While the purposed of
this function is mostly identical to marginal_counts(), because making
the bit indices order significant is a breaking change this was done as
a separate function to avoid that. Once this new function is released
we can look at deprecated and eventually removing the existing
marginal_counts() function.
Details and comments
Fixes #6230
TODO:

[x] Benchmark and tune rust code

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 23 (56.52%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 84.353%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/result/utils.py
11
21
52.38%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
54501

Relevant Lines:
64611

💛  - Coveralls

I ran some benchmarking with:
import random
import time
import statistics
from qiskit.result import Counts, marginal_counts, marginal_distribution

random.seed(123442)
counts = {}
indices = list(range(0, 10, 2))
for i in range(1121):
    bit_string = format(i, "011b")
    counts[bit_string] = random.randint(0, 1000000)

times_rs = []
times_py = []
for i in range(1000):
    start_rs = time.perf_counter()
    marginal_distribution(counts, indices)
    stop_rs = time.perf_counter()
    times_rs.append(stop_rs - start_rs)
    start_py = time.perf_counter()
    marginal_counts(counts, indices)
    stop_py = time.perf_counter()
    times_py.append(stop_py - start_py)

print(f"New rust time: {statistics.mean(times_rs)}")
print(f"Old python time: {statistics.mean(times_py)}")

Returned:
New rust time: 0.00017812372278422118
Old python time: 0.0012937638871371746

The only big bottleneck in the rust code will be the conversion to and from a python dict and rust hashmap. But it's a bit tricky to profile when it's this quick for inputs this big. Running the for loop from the benchmark under perf which yielded this flame graph

Looking at that flame graph the conversion back and forth is ~37% of the time in the function but of that 34% is conversion to rust from python which is not as easy to workaround. The only way to really do that is to create a Counts and/or Distribution classes in rust to replace the dict subclasses we have now and then only allow those as inputs for this new function. So I think given this is already 10x faster than the python version we can just leave it as is for now I don't think there is too much tuning left we can do for this PR and I've marked this ready for review.


Is your enhancement request related to a problem? Please describe.➕
Modification:

control the speed of the cactus coming forward.
specify the color of the border of the canvas.
add sound effect during collision, socre (1000*n) and jumping.
control the height of the dino diring jumping.
make it almost like a real game.

Describe the solution you'd like
Solution steps:

control the height and the of the cactus by modifying the js file.
for modifying the sound effect first, download the sound effects and insert it inside the html file.

now make a function to play the sounds and call it accordingly.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @samratghosh291 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @samratghosh291 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Great Package! Can it cross the 4GB limitation?

We are now using python-pure-cdb on our server (with minor modifications) on Google App Engine with a ~3GB file on Google Storage.
We have implemented a "FileString" class which is a lazy "mmap" that reads only the required parts from the Google Storage file when getitem is called (rounding to 8kb blocks).
We want to move to larger files - even if it breaks the original cdb implementation. We want to use 64bit pointers instead of 32bit. I know we should replace the read_2_le4\write_2_le4 Struct from '<LL' to '<QQ', use a hash-function that gives 64-bit values and multiply all the hard-coded offsets (8, 2048, others?) by two.
There are also the bit-shifts which confuse me...

Hi Oren,
Apologies for the severe delay in replying. The bit shifts are simply multiplication and division by powers of 2, really they do not need to be written that way, it is simply a holdover from the original CDB code. For example, "1 << 3" multiplies 1 by 2-to-the-power-of-3, aka. 8, and so the result is 8.

It's a little late to be replying to your bug, but just in case, here is a complete implementation. All the best


Unable to write 800K diskcopy images

Hello,
My setup consists of a greaseweazle F1 adapter. Using latest build, I am able to successfully read physical disks, but cannot write. Using the same hardware with previous versions of fluxengine, I can both read/write, so I believe something broke in latest build.
During write, I do not get errors, however the disk cannot be read.
Reading disks written by actual mac hardware is fine. Is there something I could look into?
 Tracks -> 1         2         3         4         5         6

H.SS<PHONE_NUMBER>01234567890123456789012345678901234567890123456789012345
0. 0 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 1 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 2 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 3 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 4 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 5 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0. 6 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 7 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 8 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0. 9 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.10 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.11 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

0 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX.
1 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX.
2 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX.
3 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX.
4 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
5 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
6 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
7 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
8 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
9 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
1.10 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
1.11 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
Good sectors: 28/1584 (1%)
Missing sectors: 1545/1584 (97%)
Bad sectors: 11/1584 (0%)
writing DiskCopy 4.2 image
66 tracks, 2 sides, 12 sectors, 524 bytes per sector; GCR

Also, I do not know if it is related, but latest release reads/writes disks almost twice as slow compared to release 580

I'm currently in the middle of a massive refactor in favour of getting verification during writes working, which is desperately needed, and it's entirely likely something got broken along the way. I need better tests. I'll investigate; thanks.

I know and that is great. Let me know if I can be of any assistance (troubleshooting, beta testing and so on)

The verifier change has been committed and pushed. I've verified that I'm able to write disks correctly on my setup. Could you try again with the latest client, please? (Note that the format name has changed from macintosh to mac800.)

Thanks for your response. I am still trying with different options, however early results show that it is exhibiting the same behaviour:
 Tracks -> 1         2         3         4         5         6

H.SS<PHONE_NUMBER>01234567890123456789012345678901234567890123456789012345
0. 0 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 1 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0. 2 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0. 3 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 4 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 5 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0. 6 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0. 7 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXBX
0. 8 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0. 9 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.10 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.11 .XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

0 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
1 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
2 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
3 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
4 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
5 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
6 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
7 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXB
8 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
9 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
1.10 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
1.11 X.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

Also, I don't know if it is relevant, but in this version I had to supply --flux_source.drive.drive=0 and  --flux_sink.drive.drive=0 in order to read/write respectively. I would otherwise receive Error: bad input/output disk config

That last was a bug --- should be fixed now. (BTW you can use '-d drive:0' instead of the longer config options.)
Can you paste in the full command line and output from both the write and the read, please? I don't see how, with the verifier, that the write can be completing and still producing an unreadable disk.

Yes of course. Give me 2 minutes

Hi David,
using your latest release, the verify function is indeed being called. However Only track 0.0 is able to be written successfully.
Measuring rotational speed... 199ms
reading DiskCopy 4.2 image
80 cylinders, 2 heads; GCR; FluxEngine image
Writing to: drive 0
0.0: Write:   199 ms in 71489 bytes
Verify:  288 ms in 82054 bytes
0.1: Write:   199 ms in 71166 bytes
Verify:  286 ms in 82475 bytes
Rewrite: 199 ms in 71166 bytes
Verify:  287 ms in 82805 bytes
Rewrite: 199 ms in 71166 bytes
Verify:  286 ms in 82377 bytes
Rewrite: 199 ms in 71166 bytes
Verify:  287 ms in 82733 bytes
Rewrite: 199 ms in 71166 bytes
Verify:  285 ms in 82040 bytes
Rewrite: 199 ms in 71166 bytes
Verify:  285 ms in 82221 bytes
Error: Write failed; uncorrectable error during write.

Hmm. Are you using an HD or a DD disk? Writing DD formats to HD disks doesn't always work. (Mac GCR formats are always DD.)
That said, I think I might be doing something wrong here; I'll investigate.

Thanks David for your reply. I get the same behaviour with multiple disks (DD and HD). With all disks, it always writes only sector 0.0 correctly.
On the same note, I am able to use read/write disks successfully using the previous stable version with same hardware.

Hi again,
upon further investigation, I find that the issue occurs only when working with diskcopy images. When I load an img file, it writes and verifies successfully.

Hi David,
just writing to tell you that your latest commits have solved the issue with writing to 800kb mac disks. I will also test on physical mac but it is looking good.
Thanks

Oh, good --- glad to know this is fixed. Thanks!


set timeout to access external when executing loadWeightFromURL

in a restricted network env, takes 5 min to init the model due to not able to access the URL
we should think about decrease the timeout of the loadURL so we can get better performance when startup
I0209 23:50:57.522091       1 utils.go:62] Available I/O metrics: [bytes_read bytes_writes]
I0209 23:50:57.522204       1 model.go:85] Model Config NODE_TOTAL: {UseEstimatorSidecar:false SelectedModel: SelectFilter: InitModelURL:}
I0209 23:50:57.522253       1 lr.go:169] LR Model (AbsModelWeight): no config
I0209 23:50:57.522265       1 model.go:77] Model AbsModelWeight initiated (false)
I0209 23:50:57.522276       1 model.go:85] Model Config NODE_COMPONENTS: {UseEstimatorSidecar:false SelectedModel: SelectFilter: InitModelURL:}
I0209 23:55:18.769831       1 lr.go:162] LR Model (AbsComponentModelWeight): loadWeightFromURL(https://raw.githubusercontent.com/sustainable-computing-io/kepler-model-server/main/tests/test_models/AbsComponentModelWeight/Full/KerasCompWeightFullPipeline/KerasCompWeightFullPipeline.json): map[core:{{17.49994468688965 map[cpu_architecture:map[Alder Lake:{0.5408945679664612} Broadwell:{17.9639892578125} Cascade Lake:{-0.49166440963745117} Coffee Lake:{0.5166589617729187} Haswell:{-0.5789095163345337} Ivy Bridge:{-0.024028241634368896} Sandy Bridge:{0.5239214301109314} Sky Lake:{0.4193417429924011}]] map[cpu_cycles:{6.85713664e+09 7.560771917192364e+18 -0.11352460086345673} cpu_instr:{3.374244864e+09 8.408530291701842e+17 -0.414739191532135} cpu_time:{192019.5 5.2761312e+08 -0.06457684189081192}]}} dram:{{17.49994468688965 map[cpu_architecture:map[Alder Lake:{-0.11559933423995972} Broadwell:{16.972564697265625} Cascade Lake:{0.5505847334861755} Coffee Lake:{-0.4564790725708008} Haswell:{-0.13912856578826904} Ivy Bridge:{-0.018331050872802734} Sandy Bridge:{-0.6695247888565063} Sky Lake:{0.29698115587234497}]] map[cache_miss:{9.329199e+06 2.2145245642752e+13 -0.4680119752883911} container_memory_working_set_bytes:{253952 1.93474854912e+11 0.6805805563926697}]}}]

@jichenjc good findings! Would you like to add a makefile target to download the models and add them to kepler image?

I think we do put stuffs through docker build process , so curious what's the cause , still checking

the models are download upon start
https://github.com/sustainable-computing-io/kepler/blob/main/pkg/model/container_power.go#L35

ok, I thought we put something in the docker file
https://github.com/sustainable-computing-io/kepler/blob/main/build/Dockerfile#L35
but seems those are different... let me see how we can optimize

@sunya-ch I checked the code and found we support a configmap so that we can overwrite the URL
https://github.com/sustainable-computing-io/kepler/blob/main/pkg/model/container_power.go#L42
so it should be fine we have a logic like
=> first check overwrite URL
=> check the file on local (which I will add through docker build)
=> at last check the default URL
does this sounds good to u?thanks

@sunya-ch I checked the code and found we support a configmap so that we can overwrite the URL https://github.com/sustainable-computing-io/kepler/blob/main/pkg/model/container_power.go#L42
so it should be fine we have a logic like => first check overwrite URL => check the file on local (which I will add through docker build) => at last check the default URL
does this sounds good to u?thanks

Sure. I think it’s also good to add local model too. I supposed it is just a small json file, right? Also, I am not sure do we still need the previous one in Dockerfile (normalized_cpu_arch, power_data, power_model). Are we planning to use that? If not, we may remove it from the image.

looks like normalized_cpu_arch.csv are still used to get node info
the other 2 seems not used any more .. I can do a separate PR to remove power_data, power_model if everyone agree

https://github.com/sustainable-computing-io/kepler/pull/532 proposed


Add ability to register strategies for custom string formats

Closes #94

Codecov Report

Merging #150 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #150   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          16     16
  Lines         624    628    +4
  Branches       84     84
=====================================
+ Hits          624    628    +4

Flag
Coverage Δ

#unittests
100% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

src/schemathesis/_hypothesis.py
100% <100%> (ø)
:arrow_up:

src/schemathesis/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Manual cleaning removed but pesky dawnguard.esm reported not clean

Loot reports its still not clean, i tried to clean it manually but you disabled that function,
so either Loot reports something wrong or sseedit does not clean this file.

Loots message :
Dawnguard.esm  87E3D764
C.Location, C.Regions, Delev, Invent, Relev
This plugin contains wild edits beyond ITM and UDR records and deleted navmeshes and may require additional manual cleaning to not interfere with other mods. Dawnguard.esm Wild Edits Cleaning Guide

I have the same problem. LOOT gives a link but it is outdated. Therefore, I have no way to Manually Clean this. Is LOOT this bad, that it will give me such a message that obviously demands my attention, yet gives me no way to fix it? This is not "immersive" :)

I'm afraid it has nothing to do with my script... or maybe SSEEdit was updated, I was not active TES gamer for quite a while... I'm sorry.


[FLAML Crash] [Classification] 0 feature is supplied.  Are you using raw Booster interface?

Hey, thanks for the great system.
I am experiencing a crash with the following multi-class classification dataset from Kaggle: spooky-author-identification. I get the following error when I try to fit FLAML:
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/automl.py", line 1524, in fit
    self._search()
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/automl.py", line 2009, in _search
    self._search_sequential()
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/automl.py", line 1825, in _search_sequential
    use_ray=False,
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/tune/tune.py", line 382, in run
    result = training_function(trial_to_run.config)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/automl.py", line 240, in _compute_with_config_base
    self.fit_kwargs,
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/ml.py", line 328, in compute_estimator
    fit_kwargs=fit_kwargs)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/ml.py", line 267, in evaluate_model_CV
    log_training_metric=log_training_metric, fit_kwargs=fit_kwargs)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/ml.py", line 196, in get_test_loss
    estimator.fit(X_train, y_train, budget, **fit_kwargs)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/model.py", line 515, in fit
    return super().fit(X_train, y_train, budget, **kwargs)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/model.py", line 318, in fit
    self._t1 = self._fit(X_train, y_train, **kwargs)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/flaml/model.py", line 99, in _fit
    model.fit(X_train, y_train, **kwargs)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/core.py", line 422, in inner_f
    return f(**kwargs)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/sklearn.py", line 915, in fit
    callbacks=callbacks)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/training.py", line 235, in train
    early_stopping_rounds=early_stopping_rounds)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/training.py", line 102, in _train_internal
    bst.update(dtrain, i, obj)
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/core.py", line 1282, in update
    dtrain.handle))
  File "/home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/core.py", line 189, in _check_call
    raise XGBoostError(py_str(_LIB.XGBGetLastError()))
xgboost.core.XGBoostError: [04:04:16] ../src/learner.cc:567: Check failed: mparam_.num_feature != 0 (0 vs. 0) : 0 feature is supplied.  Are you using raw Booster interface?
Stack trace:
  [bt] (0) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/lib/libxgboost.so(+0x8d264) [0x7fadf8272264]
  [bt] (1) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/lib/libxgboost.so(+0x1ae8d2) [0x7fadf83938d2]
  [bt] (2) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/lib/libxgboost.so(+0x1bc0ac) [0x7fadf83a10ac]
  [bt] (3) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/lib/libxgboost.so(+0x1a29cb) [0x7fadf83879cb]
  [bt] (4) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/site-packages/xgboost/lib/libxgboost.so(XGBoosterUpdateOneIter+0x58) [0x7fadf82650c8]
  [bt] (5) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/lib-dynload/../../libffi.so.7(+0x69dd) [0x7fae5a76a9dd]
  [bt] (6) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/lib-dynload/../../libffi.so.7(+0x6067) [0x7fae5a76a067]
  [bt] (7) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/lib-dynload/_ctypes.cpython-37m-x86_64-linux-gnu.so(_ctypes_callproc+0x2e7) [0x7fae5a7823a7]
  [bt] (8) /home/mossad/anaconda3/envs/kgpip/lib/python3.7/lib-dynload/_ctypes.cpython-37m-x86_64-linux-gnu.so(+0x12e14) [0x7fae5a782e14]

Here is my script:
df = pd.read_csv('spooky.csv')
X, y = df.drop('author', axis=1), df['author']
X_train, X_test, y_train, y_test = train_test_split(X, y, random_state=123)
automl_model = AutoML()
automl_model.fit(X_train, y_train, task='classification',
                           time_budget=300,
                           retrain_full='budget',
                           verbose=0, metric='macro_f1')

Your feedback is appreciated.

Could you share a link to download the .csv file? Thanks.

Thanks @sonichi for your reply. Please find the .csv file of spooky-author-identification.

df = pd.read_csv('spooky.csv')
X, y = df.drop('author', axis=1), df['author']
X_train, X_test, y_train, y_test = train_test_split(X, y, random_state=123)
automl_model = AutoML()
automl_model.fit(X_train, y_train, task='classification',
time_budget=300,
retrain_full='budget',
verbose=0, metric='macro_f1')

Thanks @mossadhelali . The reason is that the raw text feature column is removed in preprocessing. There is no automatic feature engineering for pure text data in flaml now for task='classfiication'. We do support task='seq-classfiication' for such use cases. This is a big area for improvement and we'd appreciate contributions if it's of your interest.
cc @liususan091219

@mossadhelali thanks for your question. Indeed, you should be using the NLP module with the task = 'seq-classification' option. Here is the sample code for using FLAML to classify the spooky-author-identification dataset:
from flaml import AutoML
import ray
import pandas
ray.init(num_cpus=4, num_gpus=4, ignore_reinit_error=True)

train_valid_dataset = load_dataset('csv', data_files={'train': your_train_csv_file_path})["train"].to_pandas().sample(frac=1, random_state=42)
test_dataset = load_dataset('csv', data_files={'test': your_test_csv_file})["test"].to_pandas()

custom_sent_keys = ["text"]
label_key = "author"

X_train, y_train = train_valid_dataset[custom_sent_keys], train_valid_dataset[label_key]
X_test = test_dataset[custom_sent_keys]

automl_settings = {
        "gpu_per_trial": 1,
        "time_budget": 2400,
        "task": "seq-classification",
        "metric": "accuracy",
        "log_file_name": "seqclass.log",
        "use_ray": {"local_dir": "data/"},
        "n_concurrent_trials": 4
    }

automl_settings["fit_kwargs_by_estimator"] = {
        "transformer": {
            "model_path": model_path,
            "output_dir": "test/data/output/",
            "ckpt_per_epoch": 1,
            "fp16": True,
        }
    }

automl = AutoML()

automl.fit(
        X_train=X_train,
        y_train=y_train,
        **automl_settings
    )
predicted_proba = scipy.special.softmax((automl.predict_proba(X_test))
predicted_id_proba = pandas.DataFrame(test_dataset["id"]).join(pandas.DataFrame(predicted_proba))
predicted_id_proba.to_csv("output.csv", index=False)

I ran the above code on my local machine with 4x NVIDIA V100 GPUs and got a 0.389 multi-class logarithmic loss on the test dataset. Please let us know if this solution works for you.


Fixes/missing database yml

Hey @manuelmeurer @bronzdoc,
This PR fixes #52 by adding sample files and steps to bin/setup
Please check it out.
Thanks!

@manuelmeurer Sure, done. 👍


Python2 installs without sqlite3

Reproduction steps:
0) MacOS Sierra

brew install python
python -c 'import sqlite3'

ImportError: No module named _sqlite3

The SQLite is listed as a recommended dependency and is installed. There is no --with-sqlite and --without-sqlite is not used.
Running brew reinstall python did not help. Used to work out of the box on El Capitan.

Please install CLT
sudo xcode-select --install

and try again.

Thank you, I did indeed still have the command line tools from El Capitan.
User error.
Should brew doctor catch this?

I think this problem is specific to python and a few other formulae (see the pour_bottle block in python.rb) so IMO it may not be the best idea to warn about this in general.
We will consider how to better deal with this problem. Meanwhile, glad the problem is fixed.

#5347.


bash exit 1 doesn't work

Version
0.5.4
Details
The tool doesn't care about the errors, the information attached into Steps to reproduce section
Steps to reproduce
setup and config
dotnet new console -n App1
git init

dotnet new tool-manifest
dotnet tool install Husky
dotnet husky install

dotnet husky add pre-commit -c "dotnet husky run --group pre-commit'"

add .editorconfig file
https://github.com/RehanSaeed/EditorConfig/blob/main/.editorconfig

pre-commit file content
#!/bin/sh
. "$(dirname "$0")/_/husky.sh"

dotnet husky run --group pre-commit

# I want to run a condition here, e.g if dotnet format updated some files then the git commit would be rejected
exit 1 # <============ didn't work for c# files

Task-runner.json
{
  "tasks": [
    {
      "name": "Dotnet Format",
      "command": "cmd",
      "pathMode": "relative",
      "args": [
        "/c",
        "dotnet",
        "format",
        "--include",
        "${staged}"
      ],
      "group": "pre-commit"
    }
  ]
}

Update program.cs (with bad format)
using App1;

internal class Program
{
    private static void Main(string[] args)
    {
   int x=100;
int xx=x;
    }
}

Add all files
git add .
git commit -m "init"

The problem is when I send the git commit, it will be stored successfully, but it shouldn't be.

As you can see the file should be placed in the change section.

Dotnet format worked correctly (added the file as a new file changed)

#72 fixed on this pr

@alirezanet
Hi, I'm back again :)
the problem solved when i added that flag

Hi @aliasadidev,
thank you for the contribution but your pr is still open, this issue will automatically close after we merge it. also, I want to check what is going on later

Hi @aliasadidev,
Checking your issue and PR I still don't get why this is needed, if you want to write a condition in the hook itself why do you need husky? the main goal of this library is to avoid editing hooks manually, and since dotnet format doesn't exit with a non-zero status code husky doesn't interfere with the commit.
I think the better approach to solving similar problems is to use a csx script or some other script file and call it from the husky(instead of calling dotnet-format directly), this way your logic is separated and maintainable and your hook file is not complicated. unless I'm missing something.


Unable to play WAV file

Howdy! I am using this library to try to play a WAV file on the MagTag board using CircuitPython 6.1.0. I am trying to figure out what would need to be done to make this work.
The history of commits to peripherals.py has me somewhat confused. Initially there seemed to be a play_wav_file function available in a previous version of the library, which used audioio. However, audioio does not seem to be available for this board. Was this code ever used/tested or was it just wishful thinking on my part that I could use it? :-)
https://github.com/adafruit/Adafruit_CircuitPython_MagTag/blob/14d7b126d3cfeb325c9fc32eaca14090d4f057ab/adafruit_magtag/peripherals.py#L83
I am having difficulty understanding what play_tone is actually doing. If I try to call simpleio.tone(board.SPEAKER, 300, 0.2) directly, no sound occurs (but there is no exception). However, once I create my own DigitalInOut for the board.SPEAKER pin and invoke simpleio.tone, I get the dreaded NameError: name 'AudioOut' is not defined error.
>>> from digitalio import DigitalInOut, Direction
>>> speaker = DigitalInOut(board.SPEAKER)
>>> speaker.direction = Direction.OUTPUT
>>> speaker.value = True
>>> simpleio.tone(board.SPEAKER, 300, 0.2)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "simpleio.py", line 82, in tone
NameError: name 'AudioOut' is not defined

This is really confusing, since it only seems to occur once I set up DigitalInOut. And tones play fine if I use:
magtag.peripherals.play_tone(300, 0.1)

This suggests that the version of this library I have on my board (which came shipped with ADABOX 017) does not exactly match what I'm seeing in this repo. Any tips or suggestions on how to unlock the FULL AUDIO POTENTIAL of this amazing device? Thank you!

yah DAC/WAV playback doesnt work on the ESP32S2 at this time, you can make beeps with the buzzer - using a PWM output (there's some magtag examples). you cannot (yet) play audio clips or waveforms

yah DAC/WAV playback doesnt work on the ESP32S2 at this time, you can make beeps with the buzzer - using a PWM output (there's some magtag examples). you cannot (yet) play audio clips or waveforms

Thanks for the quick reply. Is this a software limitation or a hardware one? If it's software I might be able to fix it.
I'm still confused why trying to use simpleio.tone() directly gives this error. Is there a way to determine which version of the MagTag library is in use in CP 6.1.0?

Thanks for the quick reply. Is this a software limitation or a hardware one? If it's software I might be able to fix it.
I'm still confused why trying to use simpleio.tone() directly gives this error. Is there a way to determine which version of the MagTag library is in use in CP 6.1.0?

its software, nontrivial to fix! https://github.com/adafruit/circuitpython/issues/3898
try deleteing and reinstalling all libraries - use the magtag library to make tones
if you're still having issues please post in the adafruit forum :)

its software, nontrivial to fix! https://github.com/adafruit/circuitpython/issues/3898
try deleteing and reinstalling all libraries - use the magtag library to make tones
if you're still having issues please post in the adafruit forum :)


(Re)consider where we use AnyId vs Id, ie, where we allow reserved words vs not

Describe the improvement you'd like to request
[Comes from comments on #634]
We should reconsider if there are more places we should use AnyId where we currently use Id, ie, if there are more places we should allow Cedar reserved words to be used when an identifier is needed.  #634 allows Cedar reserved words to be annotation keys, maybe there are more places we could/should allow Cedar reserved words to be used where currently they are not.
Of course, keeping them disallowed is somewhat more future-proof in case we have a need to disallow them in the future, due to new Cedar features or syntax.
Describe alternatives you've considered
Status quo is probably OK
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this internal improvement
[ ] ⚠ This feature might incur a breaking change

Related: #407


MudAutocomplete not cleared if you define OnClearButtonClick event

Bug type
Component
Component name
MudAutocomplete
What happened?
In MudAutoComplete if you define Clearable=true AND OnClearButtonClick="OnClearButtonClick" it doesn't clear anymore.
Expected behavior
When you hit the X it must clear the text and then call OnClearButtonClick event.
Reproduction link
https://try.mudblazor.com/snippet/wuwcvmwQoMsoSyhj
Reproduction steps

Create MudAutocomplete component
Add Clearable=true property to component
Add OnClearButtonClick="OnClearButtonClick" event
Add code: protected async Task OnClearButtonClick(MouseEventArgs e)
{
}

Relevant log output
No response
Version (bug)
6.1.6
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

In your example (reproduction link) it still clears the text while typing.
After selection, it doesn't clear, I guess that's the problem for you.
Add ResetValueOnEmptyText=true to the MudAutoComplete control,
I think it now behaves like you expected.

Yes! It's working now, thanks.


Install of 2.4.1 on my Armbian distro doesn't seem to have IPP working as the server isn't discoverable on any device

I'm using CUPS 2.4.1 and a clean Armbian (v 23.05.1 jammy) install on a Tinker Board.    It works fantastically printing text and raster images to my Brother printer (connected locally via USB).
The only issue is that on previous installs of CUPS I've made, the server was instantly discoverable on absolutely every device on my LAN, thanks to the magic of IPP from what I understand.    I could just go to "Find printers" in Windows, browse printers in Android, iOS, whatever I used the printer was automatically discovered via the os.    However with this new install of CUPS that isn't happening.
I worked out the local ipp:// address of my printer and so was able to setup a printer in Windows with just a few more steps.  However, for printing from phones it's a different matter.   I really want to get this working so I can just locate the CUPS server on my LAN automatically with it appearing in the printers list, like I was able to with a previous install.
Where do I begin to troubleshoot this ?
I can't see any errors in the log file btw.

It is always useful to know the printer make and model. The outputs of lpstat -t and lpstat -l -e on the Tinker could also be of help.

lpstat isn't installed it seems.  I just tried to install it and I get
Read_file_list: cannot stat required or included file '/etc/printcap' - No such file or directory

What is given by which lpstat? Printer make and model?

lpstat is now working for some reason, not quite sure why.  It reports...
system default destination: Brother_HL-L2320D_Living
device for Brother_HL-L2320D_Living: usb://Brother/HL-L2320D%20series?serial=U63877C2N833460
Brother_HL-L2320D_Living accepting requests since Thu Jul 20 12:39:56 2023
printer Brother_HL-L2320D_Living is idle.  enabled since Thu Jul 20 12:39:56 2023

Please give sudo cupsctl and avahi-browse -rt _ipp._tcp.

Ah fantastic  @debiantriage it was avahi.  It wasn't installed.
When i installed CUPS on a Raspberry Pi this seems to have been installed as well but on Armbian it wasn't.   As soon as that was installed it works!  Thanks a lot, really appreciated.


Enhancment - Add an “open all” on explorer button to the app home page

Hey everyone, I have a couple miners and it's a little annoying to go through each miner to open on the helium explorer in the browser. Would it be possible to add a button or overflow menu that opens a drop down with the option to "open all in explorer" on the main page that shows all miners?

initiated from the app? I'm not sure what you'd get from this.


Adding a Frozen Column repeater to index page

to test changes to either style of list repeater to make sure does not change other.
fixes #1246

Thanks!