feat: add az tags on subnets

in order to best find network resources, the az and az_id are added to subnet_resources

Description
added three tags for the aws_subnet resource

az: the short letter for the specified availability zone

az_id: the availability zone id

serial: the subnet number with count

Motivation and Context
In order to be able to easy pinpoint a subnet, I have the option to build a complex name parser on each and every component that will have to find vpcs and subnets, or to update the vpc module to add tags that can't be added elsewhere.
Using tags for subnet resources I can filter subnets with criteria such as the availability zone letter, the AWS AZID, or even the serial
Breaking Changes

No breaking changes. Only new tags. The sole issue would be that one uses var.tags with az, az_id or serial, then in this case it would be overwritten by these changes.
How Has This Been Tested?

in test_vpc.rb we check the presence for the following tags:

az with the proper value that can be extracted in the tests

az_id presence only. as AZIDs are per AWS account

serial present only, as we only test one subnet that could be either one or another

Force tags for everyone seems not the best thing I think, but I like your idea to add azs indications on subnets.
Maybe reviewers will be ok with that.

@bryantbiggs the main goal is to easily find subnet(s) matching a specific az.
Using the vpc module there is no possibility to do such thing. I may create subnets but no way to pinpoint a specific az.
This PR allows subnets to have the availability zone tag and also the availability zone ID. Finally a serial giving an number for a subnet in a vpc.

thanks for the info @vfoucault, however as I mentioned before, can't you achieve this through the subnet data source?
data "aws_subnet" "private_a" {
  vpc_id = module.vpc.vpc_id
  filter {
    name   = "tag:Name"
    values = ["main-private-us-east-1a"]
  }
}

data "aws_subnet" "private_b" {
  vpc_id = module.vpc.vpc_id
  filter {
    name   = "tag:Name"
    values = ["main-private-us-east-1b"]
  }
}

data "aws_subnet" "private_c" {
  vpc_id = module.vpc.vpc_id
  filter {
    name   = "tag:Name"
    values = ["main-private-us-east-1c"]
  }
}

locals {
  private_subnet_ids = [data.aws_subnet_private_a.id, data.aws_subnet_private_b.id, data.aws_subnet_private_c.id]

You can get all attributes of a subnet from this approach as well - Ref. I'm not saying your proposal is wrong or not worth merit, but it has quite strong assumptions for the community at large that uses the module when there are alternate methods for getting this information without everyone opting in to the changes.

Thanks for the reply. You are correct, I could fetch subnets using aws_subnet_ids, using aws_subnet it would fail in case of having more than one subnet per az.
Still, if I want to filter subnets by AZ_ID it becomes a nightmare. I may only add availability_zone_id to aws_subnet and not to aws_subnet_ids
This means that I need to:

find all subnets given a composed named for all AZ
for each subnet using the aws_subnet data
filter out subnet that matches the proper AZ_ID

instead of filtering per AZ_ID
The other approached that I've looked at is to update the module to accept per subnet tags as referenced may times, but it is not the best approach as finding AZ_ID would have to be done outside of the module itself.

Hi @bryantbiggs . Does it worth to rebase on master ? Is there any chance for this to be merged ?  Would you mind check this PR again ? Thanks in advance

I'm sorry, but I don't think this is something we want adopt in this module

Hi @vfoucault !
Sorry for not coming back to this PR earlier.
I agree with Bryant that it is already possible to achieve using a combination of data sources and I don't think we can have this functionality in the module because not everyone needs the features introduced here.
For now, let's close this issue.


User management issues

I experience some user management issues
I first created a new contributor account with my admin account, and then registered a series of questionnaires in the contributor account. When I later switched back to the admin account, I noticed that am not able to manage content registered in the contributor account. Not sure if this is intentend, but how can I as the admin approve or decline new contributions?

Please note that in its current version, SIR has no mechanism for one to "approve" or "decline". Everything that is entered is immediately available for anyone to see.
Also, any content can only be modified by the manager of the content - no one else. The authorial and editorial capabilities of SIR are expected to be included in the version 0.6 since they still need to be implemented. Issues #12, #13 and #14 are the ones addressing the issue reported above, and they have already been scheduled to be implemented.
I will close this issue since it is already addressed in previous issues.


gzip decompression

Hello
First, thank you for your initiative.
Got a question, when i try to decompress a gzfile it use only one CPU (got 24 cpus), do you have an example that can decompress gzfile to a destination file ?
Anyway thanks.

I happened to see this just now.
The very original gzip format does not support parallel decompression and specific format is required. There have been several attempts to invent such formats (This mgzip is one of them).


Last stage

In original paper paper ,There is no activation function after avg_Pool　in fig5, but your code is added. Does this have any effect?

you can see Table 1 or Tabel2, it exists activate after pool

yes, it exists activate after pool in table1 or tabel2, but it not exists in fig5,Which one should we follow?

you can see Table 1 or Tabel2, it exists activate after pool

The paper updated once on 14 May, but still has some bugs.  I think there should not be non-linear after pool.

@kuan-wang
thank you very much


Update EmbeddedCollectionDelta

resources/app/common/abstract/embedded-collection-delta.mjs

Closed by #2735


Use an ORM between python services and Mariadb

Replace the current POC implementation in services/db_api.pl with an ORM, probably SQLAlchemy.

this is complete:  https://github.com/grin-pool/grin-pool/pull/13


Update zstd-jni to 1.5.5-11

Hi! We (at Hopper) use timeshape in a few of our Scala microservices. I was working on some dependency updates and was getting some binary compatibility warnings with zstd-jni. I am fairly certain 1.5.5-11 and 1.5.5-6 are binary compatible, but it would be nice to get this bumped to latest if its not too much work to get a new release published. I saw master was bumped to 2023b.21-SNAPSHOT, so not sure if that release will be good to go or if other changes will also be needed.
Thank you for maintaining the project! Let me know if I missed anything.

Hi @inickt, thanks for your contribution! I think this looks good, I'll merge it now so that the SNAPSHOT release (2023b.21-SNAPSHOT) will be published. You can try that snapshot release and see if it works for you. If it does, let me know and I'll make a proper, non-snapshot release with your change.

@RomanIakovlev Thanks!! Are snapshot releases published though? I don't seem to see the artifact and looking quickly over the build it seems like its skipped?
https://github.com/RomanIakovlev/timeshape/blob/defd7beac2a8903c1a530b05c9ea39688c7b9196/build.sbt#L33

Yes, snapshots are published! Here's the workflow: https://github.com/RomanIakovlev/timeshape/actions/runs/8469140390/job/23203766774. And here's the direct link to the artifact: https://oss.sonatype.org/content/repositories/snapshots/net/iakovlev/timeshape/2023b.21-SNAPSHOT/timeshape-2023b.21-SNAPSHOT.jar
P.S. The wording in build.sbt is a bit confusing indeed, should probably be updated.

Hmm okay there might be some issue with our dependency proxy for work that isn't pulling in snapshots– I tested with the jar locally though and everything looks good though!


Error running on windows

I've run this command within Windows WSL:
user@localhost:~/projects/personal/my-app$ emus -h

        Error: Command failed with ENOENT: /home/user/.bin/android-sdk/emulator/emulator -list-avds
spawn /home/user/.bin/android-sdk/emulator/emulator ENOENT
        Error: The iOS Simulator is only available on macOS

Hi @fmsouza, thanks for the issue
Can you install v0.3.0 and confirm if it fixes the issue?

It doesn't just tried

Can you confirm emus --version returns 0.3.0? Also, did you get the same error?

On my machine the emulators is not located under android-sdk/emulator/emulator but instead android-sdk/tools/emulator
Could it be fixed to check which path is available?

Ok, just found which was the problem. I'm on Windows with WSL. I have Android SDK installed on both sides (WSL and Windows itself), though only Windows side has emulators, as WSL is headless.
It's looking for the emulator on my $ANDROID_HOME, which was pointing to WSL, and so was not found.
Changing my $ANDROID_HOME to point to Windows main Android SDK under /mnt/c/Users/MyUser/AppData/Local/Android/Sdk (which is the place Android Studio installs as default) did the job!
Maybe what could have happened instead is a better log telling me the emulator is not available in the path, or so.

@fmsouza Thanks! That's really helpful, we definitely need better errors on Windows. I'll probably have the next version published in a few days, hopefully you can help test that version then since I don't have access to a Windows WSL machine.
@andrelind Please checkout v0.4.0 and let me know if it fixes your issue.

@fmsouza @andrelind I've just published v0.5.0, please give it a try and let me know if it fixes the issues.

With 0.5.0 all the available emulators shows up now!
But nothing happens when I try to start them :-/

@andrelind
Can you run android-sdk/tools/emulator -list-avds to get the list of AVDs
and then run ./emulator -avd avdname to start an AVD
I got this error
PANIC: Missing emulator engine program for 'x86' CPU.
Any reason why you can't upgrade your Android SDK tools?

Sorry, had android-sdk installed via brew, when I removed that it worked like a charm :-)

That's great!
Yeah, the android-sdk from homebrew only includes the command line tools, you still need to use the sdkmanager to install other packages, including the new emulator. It's probably not a good idea to try and use android-sdk/tools/emulator to start the emulator, I'll remove that in the next version.

@fmsouza  I'm closing this, please re-open the issue if you still experience this in the latest version ;)


SQL Generation via marten-patch finds difference in partitions

Registering a class via options.Schema.For<ClassName>.DatabaseSchemaName("customschema").SoftDeletedWithPartitioningAndIndex()
and running marten-patch via the oakton commands results in a SQL file being created with the following content:
CREATE TABLE customschema.mt_doc_class_name_deleted partition of customschema.mt_doc_class_name for values in (TRUE)
Applying these created scripts via Grate results in the tables and partition being created in PostgreSQL with the following partition expression FOR VALUES IN (true)
During a refresh of the table structure via marten-patch a difference in partition definition is found. The expected partition definition is FOR VALUES IN (TRUE) and the retrieved value is  FOR VALUES IN (true)
In PartitionExtensions.cs:28 the FormatSqlValue<T> does an ToUpperInvariant() in the case of a bool value. In ListPartition.cs:32 the Parse() method reads the current value stored in the PostgreSQL instance containing the lower-case value. In ListPartitioning.cs:96 the IPartitionStrategy.CreateDelta() method checks if other._partitions.Any(x=> !partitions.Containx(x)) resulting in a Rebuild of the parition that does exist with a different casing on the parition values.
Tested via docker database with version postgres:16.3

The soft deleted partition problem was fixed, but in testing this w/ Marten, I found an additional issue with foreign keys when using the partitioning

Weirder, this only fails in the command line. I would have sworn up and down there's no difference in functionality between the CLI and directly using IDocumentStore.Storage

Ugh, ugh, ugh. Weasel is "finding" the existing foreign key that's really attached to the partition table and thinking that exists on the parent table. That's the culprit. Ick, ick, ick.

Got it! It's going to take a Weasel upgrade to finish this off, but it's on the way.


1.x upgrade: make config upgrade more robust

/etc/influxdb/influxdb.conf is owned by root and when a user runs upgrade command as any other other, which is expected, it fails with Error: open /etc/influxdb/influxdb.toml: permission denied when it attempts to save the new config to the same folder.
Suggested solution:  try to save the new config to the same folder where 1.x influxdb.conf is and if it fails for any reason, save it to user's HOME.

Reported by Alex Bergsland in #general channel.


Bad libraries published for EasleJS

It looks like some old / strange merge has gotten in introducing old code.
We've started seeing strange behavior in our code over the last week stemming from easeljs.
It's easy to spot looking here:
https://github.com/CreateJS/EaselJS/blob/master/lib/easeljs-0.8.2.combined.js
Then viewing the commit it came in
https://github.com/CreateJS/EaselJS/commit/06a2c978212360e23d72dcd32c223102a32170a1
And seeing the PR that it was associated with
https://github.com/ValtoLibraries/EaselJS/pull/2
It seems to have introduced old code into the project. It's easy to spot the "MasterFilter" which has been removed
https://github.com/CreateJS/EaselJS/blob/master/lib/easeljs-0.8.2.combined.js#L5927
Just a heads up, we ended up rolling back to #993834eeddf2d61b7639d2a0259ffcdbceb6aeaf

Wow, thanks for the find!
A build was run and committed on our prototype StageGL branch by mistake some time ago (Feb 2016), which didn't affect anything, but was included in the StageGL merge we did recently, which is why it just showed up.
I will revert the tagged 0.8.2 files in GitHub.
Cheers!

I have reverted all the 0.8.2 tagged files. Thanks again!
Note that the CDN versions were unaffected.


Updating to 0.16

Updated most things to work with 0.16.
I'm new to using elm and have never used this library before, but made an attempt to get this working under 0.16.
Many updates seemed trivial, but there didn't appear to be any css for the examples and I'm not very good at css so made no attempt to add any.
The readme notes it was necessary to manually add d3.js library.   I've seen other libraries include Native/.js file and import it somewhere so I added the native d3 min library and required license for it.
I did have some difficulty updating Voronoi as the Random generator library appears to changed a bit and although the compiler is happy with they type signatures, I'm getting an error in native js fromRecord method looking for a _ in the js object that doesn't exist but otherwise the record looks fine.
I was more interested in getting core library working so didn't want to spend much time on the examples for now, but I figured I would share the updates here so that when you do want to update it you can start where I left off.

+1

As noted in #32, I'm going to skip 0.16 support and go straight to 0.17. Sorry I didn't get to this sooner, but thanks for your contributions all the same.
As a side note, when writing bindings for JavaScript library, it's my practice not to include the source code for the target library. Better to rely on the user's existing static asset pipeline for that.


test(lib) add unit tests

Maybe you should use https://api.lb-0.testnet.chrysalis2.com as URL instead of the one with iota1.0 nodes

Maybe you should use https://api.lb-0.testnet.chrysalis2.com as URL instead of the one with iota1.0 nodes


Hookkup Goreleaser

[X] Add .goreleaser.yml config to use goreleaser to build pflags
[X] Fill in Version, Build and Build Time in /version package
[X] Update contributing section in README

I think it only has one k, hookup.
On Wed, Apr 24, 2019 at 9:40 AM Haytham AbuelFutuh<EMAIL_ADDRESS>wrote:

Merged #10 https://github.com/lyft/flytestdlib/pull/10 into master.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lyft/flytestdlib/pull/10#event-2297808646, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWDFOEIKLOHNXULA6VOTKLPSCEP7ANCNFSM4HH7XKAA
.


[mysql][doc] update mysql doc server-time-zone description

in master branch, the server-time-zone's default value has changed, however the description of the configuration in doc haven't updated. So I do that.
[](url

)

Thx and you have inspired me!


A Kinetic Energy Recovery System

This feature request is reserved for VeC 3.0.

Easiest implementation would be to employ hub motors (which would also facilitate easier LEADS control) and then look at the different drive-regen options.

@qmascarenhas What battery are we using?

For the 12 volt race, Interstate MTX-35
For the 24 volt race, Optima Yellowtop SC35DA

This enhancement will start once Thor has been revealed.

@qmascarenhas Is it required for the 24v race to connect the two batteries in parallel?

2 12V batteries in series gets you 24V.


Move line autoindent#14390

Main issue #14390
Another Issue in which it could be related #22093.
Questions, is there a way to test performances ?
The algorithm work like this:

When moving lines, it will inherit the whitespaces of the next line.
In some cases depending on the start or end of the line, if it's one of thoses chars []{}(): it will indent correctly the line

I created some tests, but I need to have some other feedbacks:

Does the tests cover all the cases ?
Could be possible performence issue ?
Anything else that could be bad ?

For the issue #22093, they are talking aboit a different approache, after moving the lines, we use an auto indent functionality. Why not I would say but it seems for now the current autoindent feature isn't working well for selected code.

@ice-blaze,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks @ice-blaze for your contribution :)
The reason that moving lines needs to honor indentation rules is making sure the indentation adjustment is following rules that Language can control. And we should make it as generic as possible because different languages understand indentation differently, making one case happy doesn't solve the problem.

When moving lines, it will inherit the whitespaces of the next line.

Inherit is reasonable, I may implement it in this way as well. But a failing case is Comments. If next line happens to be a single line comment, I don't think inheriting from it is always a correct decision.

In some cases depending on the start or end of the line, if it's one of thoses chars []{}(): it will indent correctly the line

Well []{}() works well for JavaScript/TypeScript or similar languages, but there are two major problems with it

We should check https://github.com/Microsoft/vscode/blob/master/extensions/typescript/language-configuration.json#L6 brackets configuration of a language, instead of hard code.
def ... end block in Ruby doesn't work, which is a common case.

A simple solution in my mind right now is trying to get a good indentation guess for the first line of selected range, calculate the indentation difference, apply it the selected range and keep all others untouched. We are not Reindent the entire selected range so the change is minimal.

@rebornix Can you write an example where it won't work with the comments ?
Well in fact it's true that I don't cover all languages case with open blocks, but at least I cover major big languages (C/C++, Java/C#, Python, Php, Javascript). And at the end, for every languages I use the inherite of the next whitespace line. (except for some cases with comments if I understand right)
So maybe would it be better if I put only the inherit whitespace code in the vscode and for everything related on open blocks I create a plugin for it ?
For the Ruby, case I can add it.
Well I didn't know this file  https://github.com/Microsoft/vscode/blob/master/extensions/typescript/language-configuration.json#L6 you are completly right, I should use it. Is there something equivalent for open blocks ? I guess not or you would already say it.

Well I guess this case
line to move
// if (true) {
another line

// if (true) {
    line to move
another line

@ice-blaze,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Sweet, looking forward to this auto-indentation! It's the only remaining feature from Jetbrains IDEs that I'm missing in VS Code.

Well I guess we could use the format feature, applied to the moving block or line.
I tried to implement it but the VSCode structure is too complex for me so I give up. But I guess if someone have ease to use the VSCode API, it could be quite easy to just make a call "reformat" after the moving line command.

+1 for this feature. This sounds silly but it is one of the final things on my checklist to move from Atom.

Considering the complexity of indentation rules management, I'd like to close this PR for now and once we decide to put some time on it, we can reopen this PR and see how we can improve the experience for moving lines with auto indent turned on. Thanks for your help!

It's better to close it, I have no intent to change it the future.

Bruh!
On Thu, 24 Oct 2019 at 11:42, Peng Lyu<EMAIL_ADDRESS>wrote:

Closed #23248 https://github.com/microsoft/vscode/pull/23248.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/pull/23248?email_source=notifications&email_token=AIPXOJXYPXHNSYNRQTO3XSDQQDVQ5A5CNFSM4DFGPRW2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUNAY2YY#event-2738982243,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIPXOJRP4K7EYXYO4LXFNHTQQDVQ5ANCNFSM4DFGPRWQ
.


Pruning time consumption

Step 2: Prune the models (SECOND as example, on 8 GPUs)：
How long does it take for you to fall on the dataset once during the pruning iteration process?

Hi,
Calculating the pruning measure will take some time (not very short), as this step requires iterating over all samples in the dataset. Here, I provide a couple of possible ways to reduce the time consumption:

Increase the batch size as much as possible;
Randomly select a subset of the dataset for calculating the measure (similar to HRank [1]).

Additionally, the measure result for SECOND is available at "tools/measure_doc/SECONDNet_kitti.txt".
[1] Lin M, Ji R, Wang Y, et al. Hrank: Filter pruning using high-rank feature map[C], CVPR 2020.
Best regards.


disable automerge

Types of changes
What types of changes does your code introduce to Graphback?

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Other (please specify)

Checklist

[ ] I have read the CONTRIBUTING document.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Verification

Further comments

CC @craicoverflow we want to have more steady core so disabled automerge


Any reason this is not registered in the cookiecutter website?

Checklist

[X] I've searched the project's issues.

❓ Question

Is it possible to list this repository in the cookiecutter.io templates?

Sorry I did not see this Issue in time. Thank you very much for your suggestions. I will try to apply.


Support of Adafruit GFX drawChar()

I would like to get support of the Adafruit GFX function drawChar() in the PX matrix. My compiler says it is not an available function. Is this right or am I overlooking something?

PxMatrix simply inherits the AdafruitGFX class... whatever is implemented there is supported by PxMatrix....

But drawChar is member of the Adafruit GFX but not recognized by the PX matrix class

Did you maybe use an old version of AdafruitGFX in your library folder?On 11 Mar 2019 21:54, RobertHolscher<EMAIL_ADDRESS>wrote:But drawChar is member of the Adafruit GFX but not recognized by the PX matrix class
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or mute the thread.

Sorry for my late reaction, but I have to check if the latest library is in my folder. I will check this later and will let you know


Change - Use systemd drop-in directory for unit overrides

The .include directive has been deprecated and support for it may be removed in a future systemd release.  This fixes warnings issued by systemd which are shown as follows.
systemd[1]: /etc/systemd/system/postgresql-9.6.service:1: .include directives are deprecated, and support for them will be removed in a future version of systemd. Please use drop-in files instead.

Hi @blackknight36, thank you for submitting this PR, can you have a look at Travis failures which are coming from this?
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 211
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 212
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 213
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 214
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 17, but found it in column 16) on line 219

Hi @blackknight36, thank you for submitting this PR, can you have a look at Travis failures which are coming from this?
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 211
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 212
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 213
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 16, but found it in column 17) on line 214
manifests/server/config.pp - WARNING: indentation of => is not properly aligned (expected in column 17, but found it in column 16) on line 219

Yes, I should be able to fix it tomorrow.

I see this issue at CentOS 8 too:
Wed 2020-11-11 11:50:46 UTC hostname systemd[1]: /etc/systemd/system/postgresql-13.service:1: .include directives are deprecated, and support for them will be removed in a future version of systemd. Please use drop-in files instead.
$ rpm -q systemd
systemd-239-31.el8_2.2.x86_64

$ facter os
{
  architecture => "x86_64",
  family => "RedHat",
  hardware => "x86_64",
  name => "CentOS",
  release => {
    full => "8.2.2004",
    major => "8",
    minor => "2"
  },
  selinux => {
    config_mode => "enforcing",
    config_policy => "targeted",
    current_mode => "enforcing",
    enabled => true,
    enforced => true,
    policy_version => "31"
  }
}

I see this issue at CentOS 8 too:
Wed 2020-11-11 11:50:46 UTC hostname systemd[1]: /etc/systemd/system/postgresql-13.service:1: .include directives are deprecated, and support for them will be removed in a future version of systemd. Please use drop-in files instead.
$ rpm -q systemd
systemd-239-31.el8_2.2.x86_64

I can update this PR to include CentOS 8 as well.

Another thing I noticed is that the postgresql::server::config class does not appear to address RHEL 5 or 6 although these releases are supported according to the metadata file.  Should there be checks set up for these releases as well?

RHEL/CentOS 5/6 does not use systemd, they use old upstart, service - command and /etc/init.d -directory setup.
Regarding to upstream supports:
https://access.redhat.com/support/policy/updates/errata#Life_Cycle_Dates
RH extended support is going to end at the end of 2020-11. And RHEL6 maintenance support is going to end also 2020-11, but extended support is still possible until 2024-06.
metadata.json also does not mention Fedora (currently 31, 32, 33).
I guess any supported OS version should be supported with testing and no testing is necessary for unsupported OS combos.

Thanks alot @blackknight36 for the changes. Could you please fix the travis failures.
Thank you @ekohl and @maage for the valuable comments.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@2da32ae). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1201   +/-   ##
=======================================
  Coverage        ?   65.52%
=======================================
  Files           ?       14
  Lines           ?      351
  Branches        ?        0
=======================================
  Hits            ?      230
  Misses          ?      121
  Partials        ?        0

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 2da32ae...efbd37e. Read the comment docs.

Thanks alot @blackknight36 for the changes. Could you please fix the travis failures.
Thank you @ekohl and @maage for the valuable comments.

The Travis failures have been fixed.  Please let me know if you need any other changes.

Thank you @blackknight36 @ekohl for your contribution.

This change breaks existing postgres instance (pre 6.9.0). One need to issue systemctl daemon-reload first, after old unit file was deleted and only after that restart the newly configured service, otherwise service fails

This change breaks existing postgres instance (pre 6.9.0). One need to issue systemctl daemon-reload first, after old unit file was deleted and only after that restart the newly configured service, otherwise service fails

This change breaks existing postgres instance (pre 6.9.0). One need to issue systemctl daemon-reload first, after old unit file was deleted and only after that restart the newly configured service, otherwise service fails

Thanks for the report.  I will see if there is a way to avoid this.

This change breaks existing postgres instance (pre 6.9.0). One need to issue systemctl daemon-reload first, after old unit file was deleted and only after that restart the newly configured service, otherwise service fails

Thanks for the report.  I will see if there is a way to avoid this.

I would if issues were enabled on the repository.
I see that a fix is in progress https://github.com/puppetlabs/puppetlabs-postgresql/pull/1230

I would if issues were enabled on the repository.
I see that a fix is in progress https://github.com/puppetlabs/puppetlabs-postgresql/pull/1230


[Ace] Added Help and Save without closing (CTRL-S)

New feature - Save file without closing
The gif can't tell the full story, the status bar color turns green on a smooth half-second bezier curve. Really polished :)

Better error handling for both CTRL+S and Save button
If you can't save (for whatever reason), your changes are not lost, you are dropped back into the editor.

Ace has some really great features. I've picked a few seriously useful ones. Get Help!

Still on Ace 1.1.9.
1.2.0 is out for a month or so but i need to test it heavily before upgrading.
Not sure why the commit looks like i've completely re-written Default.cshtml, i've only added this modal here (line 115):
https://github.com/projectkudu/kudu/compare/acedev?expand=1#diff-b82317deb2356eb11b04b8758c5a1853R115

Hi @snobu, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@snobu  looks good.  note: I simulated the error by trying to edit read-only file.

Thanks! I was using tail -f filename in another window to keep the file handle open.

This is awesome!! I can't tell you how many times I hit ctrl+s and had the chrome save window pop to me. Thank you for adding this

This still happens if the editor is not in focus when hitting CTRL+S. I still have to come up with a way to steal the default CTRL+S behavior from the browser. I can do it in some browsers but not across the board. Needs more research.

Nice! I'm always hitting Ctrl-S on that thing and it's always a source of frustration, so a welcome change.


TriggerTree > Test Tree test fails intermittently

See this example:
1) TriggerTree
       Test Tree:

      AssertionError [ERR_ASSERTION]: The expression evaluated to a falsy value:

  assert(found)

      + expected - actual

      -false
      +true

      at Context.<anonymous> (libraries/adaptive-expressions/tests/triggerTree.test.js:640:21)
      at processImmediate (internal/timers.js:461:21)
      at process.callbackTrampoline (internal/async_hooks.js:126:14)

Reviewed by the SDK team. Closing as Not Needed.


test: remove config and infrastructure from code coverage report

:tada: This PR is included in version 0.16.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Release roadmap is missing

Hi guys,
I'm pretty curious when are you going to make next release?
A lot of time passed since last one. And a lot of fixes have been made.

The release instructions are in CONTRIBUTING.md ; I think the main work might be to dig through commits to create a NEWS.md entry, and to figure out the right version number.
There's definitely a maintenance shortage on the library, I was keeping up with it at Lightbend but now that I left it's fallen through the cracks. I've been trying to be responsive to discussions but actually sitting down for hours to work on code is hard for me at the moment. My JVM dev environment isn't even set up.
On the plus side I think the library pretty much works and all the fixes are nice-to-have rather than showstoppers, which is a double-edged sword since nobody's hair is on fire to allocate time to it :-)
Due to the lack of dependencies, and the test coverage, there's been amazingly little bit rot. It works and keeps working.
Other than a release there's a backlog of small issues that really could use patches, and some feature requests that genuinely would be nice.
Anyway that's the status...

I updated NEWS.md just now; a few minor features and bugfixes in there, nothing earth-shaking. @patriknw / @2m / @viktorklang do you have any thoughts on the version number to use for a new release? It should be ABI-compatible and low-risk, but there are a few feature additions as noted in NEWS.md.
I can type "sbt publish" but I don't have time to go through issues and fix anything that isn't already fixed.

I think we should keep with the versioning practice we had so far. Change from 1.2 to 1.3 was big as it started requiring Java 8 and did API changes. So bumping to 1.4 might set wrong expectations. Lets release 1.3.1.

I released 1.3.1 now; I didn't go through open issues or do any work, just shipped master as it was, since otherwise we'd be waiting a long time :-)
Please test. If something is wrong I can release 1.3.2.
Thanks

Thank you so much!


Instalando a ferramenta kubeadm

Ign:1 https://packages.cloud.google.com/apt kubernetes-xenial InRelease
Hit:2 http://br.archive.ubuntu.com/ubuntu jammy InRelease
Err:3 https://packages.cloud.google.com/apt kubernetes-xenial Release
404  Not Found [IP: 2800:3f0:4001:82d::200e 443]
O REPO esta indisponivel.

Please describe the issue in detail

I am performing the steps described in the document for installation and when I arrive at the installation points
(apt-get update
I receive status code 404 from the repository.
Err:6 https://packages.cloud.google.com/apt kubernetes-xenial Release
404 Not Found [IP: 2800:3f0:4001:82d::200e 443]

Thank you very much,
This procedure solved my problem

This procedure solved my problem

Closing this issue as it's resolved to you.


PermissionError on Windows when using table_conversion='selenium'

I have a Discord bot running on Windows 10 and Python 3.7 that periodically posts a dataframe image. I create the image using await asyncio.get_running_loop().run_in_executor(None, lambda: dataframe_image.export(df, filename, table_conversion='selenium')). This works fine, but after the 1st export errors start appearing when calling the export function again:
Exception ignored in: <finalize object at 0x2328bc85170; dead>
Traceback (most recent call last):
  File "C:\Development\Python37\lib\weakref.py", line 572, in __call__
    return info.func(*info.args, **(info.kwargs or {}))
  File "C:\Development\Python37\lib\tempfile.py", line 795, in _cleanup
    _shutil.rmtree(name)
  File "C:\Development\Python37\lib\shutil.py", line 516, in rmtree
    return _rmtree_unsafe(path, onerror)
  File "C:\Development\Python37\lib\shutil.py", line 400, in _rmtree_unsafe
    onerror(os.unlink, fullname, sys.exc_info())
  File "C:\Development\Python37\lib\shutil.py", line 398, in _rmtree_unsafe
    os.unlink(fullname)
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\Users\\djete\\AppData\\Local\\Temp\\tmpa_fua6qj\\geckodriver.log'

The error doesn't cause any actual problems as far as I can tell, but still. Looking at the code I'm guessing it's due to temp_dir_obj never actually getting cleaned up in the SeleniumScreenshot class? I'm using dataframe-image==0.1.13.

Can you provide a piece of code for me to reproduce the problem?


Users endpoint

I saw this ticket system and seems perfect for my needs.
The big problem is how to integrate my payment system to handesk when they pay.
Right now I have another helpdesk and they have endpoints to create users but this one miss this feature.
There are in case other ways to create users and disable the create accounts page?

You mean through api?
This should be pretty easy to add there

Yes right now this endpoint is missing

any updates?

Not yet, I've been working on other projects, however, feel free to submit a PR , this should be a really easy one

Yes!
that's the way :D


Disk not created

Hi,
I doesn't start as the default.wmdk disk is not created.
I am using the VMWare-Vix-1.6.2
Cdt,
SK

Can confirm. A .vmdk isn't created so the machine can't run.

I have managed to work around this:

Launch Quickstart
Wait for "Waiting for VM to come online"
Navigate to the VM's location (C:\Users\YOUR-USER-HERE.docker\machine\machines\default)
Import the .vmx file in your library (like with Workstation Player)
Edit the machine's setting to remove the harddisk
Launch the machine
After a bit (the waiting countdown is 120s) quickstart should continue and complete succesfully

Is this still an issue?

Yes, can confirm on v1.1.0
Note, that a did not ran into this issue when creating machine for the first time.


About page

Related Issuse or bug

Info #35

Fixes: #abo
Proposed Changes

Info about changes Goes here

Additional Info

Anything related Issues

Screenshots

Original
Updated

original screenshot
**updated screenshot **

Hello, @nandini7637 I'm not sure what you're attempting to improve here, you just built another about page "about1.html" instead of revising "about.html," and I'm not sure what improvements you're proposing here. In my opinion, the "about" page is cleaner than the "about1" page that you created. However, I am closing this PR while simultaneously labeling it as "hacktoberfest-accepted" so that it will be counted towards hactoberfest.

If I missed something, please reopen the PR. Keep your Open Source spirit high!

@AnkushSinghGandhi yeah I have made another page because unnati di told me to make a seperate page, that's why I didn't touch the existing code!

@AnkushSinghGandhi regarding the changes, I have added animated cards for plan, vision, mission. Added Caraousal to show event highlights of Codexcrypt
Added a timeline for showing future or past events and shown the changed to @unnati914 di and I believe the page looks much better than previous one!

@AnkushSinghGandhi yeah I have made another page because @unnati914 di told me to make a seperate page, that's why I didn't touch the existing code!

Oh well I am not aware about that

@AnkushSinghGandhi regarding the changes, I have added animated cards for plan, vision, mission. Added Caraousal to show event highlights of Codexcrypt Added a timeline for showing future or past events and shown the changed to @unnati914 di and I believe the page looks much better than previous one!

Yeah it depends person to person so I am approving this PR @unnati914 😃😃


Avoid multi-thread creates ElementMetadata object

The origin code is not thread-safe and this is the test code
    public class ElementMetadataTests
    {
        [Fact]
        public void Create()
        {
            using ManualResetEvent resetEvent = new ManualResetEvent(false);

            List<Thread> threadList = new List<Thread>();
            var maxCount = 100;
            var elementMetadataList = new ElementMetadata[maxCount];
            for (var i = 0; i < maxCount; i++)
            {
                var n = i;
                threadList.Add(new Thread(() =>
                {
                    resetEvent.WaitOne();
                    var alternateContent = new AlternateContent();
                    var metadata = ElementMetadata.Create(alternateContent);
                    elementMetadataList[n] = metadata;
                })
                {
                    IsBackground = true,
                });
            }

            foreach (var thread in threadList)
            {
                thread.Start();
            }

            resetEvent.Set();
            foreach (var thread in threadList)
            {
                thread.Join();
            }

            ElementMetadata elementMetadata = elementMetadataList[0];
            foreach (var temp in elementMetadataList)
            {
                if (!ReferenceEquals(elementMetadata, temp))
                {
                    // If it enter this branch, means the ElementMetadata object is created more than once
                    break;
                }
            }
        }
    }

Is this a correctness issue? I'd imagine that this potentially creates multiple instances of the metadata, but otherwise fine. Can you add a benchmark test to see what kind of allocations/perf changes this causes? The current way doesn't use the GetOrAdd as it would create a closure and since this is potentially a common pathway, I didn't want to do that.

@twsouthwick  Thank you. And the public static ElementMetadata Create<TElement>() method is use GetOrAdd
And I think the both of the Create method should use the same logic code.

The generic one does not create a closure, so the GetOrAdd is fine. The point is to minimize allocations if possible as this is a common pathway anytime an element is made. When I added this in, I found it really helped with performance to do it this way. If you have some numbers to show otherwise, I'm fine accepting the change.

@twsouthwick Thank you and I write the Benchmark code in https://github.com/dotnet-campus/Open-XML-SDK/commit/aaa16caf666e24f0f72a9245318276de466f3c73
And the Create method is the origin code and the Create2 use GetOrAdd

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Create
22.19 ns
0.154 ns
0.144 ns
-
-
-
-

Create2
37.22 ns
0.337 ns
0.315 ns
0.0210
-
-
88 B

I can find the Create2 that use GetOrAdd will allocated 88B
Thank you and I will close this PR

Cool. Thanks for the verification. It's also 50% slower. Can you add that to our benchmark suite?


Consider suppress-data-warnings option

Missing/off-node data warning logs are enormously noisy, to the point where they can significantly extend execution time.
Consider implementing a method of suppressing these warnings, and/or aggregating them (like "n errors encountered, of types A, B, C")

The slowdown is large enough that this should be a priority.


setting and changing pageIndex through a prop does not re-render

Hi,
I am trying to make the current page bookmarkable using a query param for the page number. However when i update the prop,the table does not re-render.
Here is a codesandbox: https://codesandbox.io/embed/rparreereact-table-pagination-query-params-7kd0p

In App.js#41 i am setting the pageIndex based on the currentPage prop
This prop gets set/changed by the main App component using a query string

The value of the currentPage does change, but not the pageIndex.
p.s. original post on spectrum:  https://spectrum.chat/react-table/general/setting-and-changing-pageindex-through-a-prop~50cb10b7-b3af-4444-8227-e9a3bbedd3a9
tx.,

I believe that you are not doing it right, in your example, what you do is you pass the pageIndex in the initial state, which filled once, otherwise, you should but you updated data in the overrides as const tableState = useTableState({}, { pageIndex: currentPage });,
more information about useTableState in https://github.com/tannerlinsley/react-table/blob/master/docs/api.md#useTableState

☝This is correct

I had the same issue, in my case I had to replace  defaultValue={pageIndex + 1} with value={pageIndex + 1}


Root directory not found, Neovim and Lspconfig

Here is my config for the LSP in neovim in init.lua.
  {
    "neovim/nvim-lspconfig",
    dependencies = {
      "kevinhwang91/nvim-ufo",
      { "j-hui/fidget.nvim", opts = {} },
      { "folke/neodev.nvim", opts = {} },
    },
    -- This was an unsuccessful try
    -- opts = {
    --  servers = {
    --    tinymist = {
    --      settings = {
    --        rootPath = "~/PaulJR/",
    --      },
    --    },
    --  },
    --},
      local capabilities = vim.lsp.protocol.make_client_capabilities()
      capabilities = vim.tbl_deep_extend("force", capabilities, require("cmp_nvim_lsp").default_capabilities())

      local lsp_capabilities = require("cmp_nvim_lsp").default_capabilities()

      -- TINYMIST
      require("lspconfig")["tinymist"].setup {
        capabilities = lsp_capabilities,
        settings = {
          tinymist = {
          -- another unsucessfull try
           --  rootPath = "~/PaulJR/",
            settings = {
              rootPath = "~/PaulJR/",
            },
          },
        },
      }
}

In the tinymist docs, for the root path it is indicated to put - for getting the parent directory. Is it the string "-" right ? Just to be sure because I also have tried that but it didn't work.
Here is the result of LspInfo :
 Language client log: /home/pauladam/.local/state/nvim/lsp.log
 Detected filetype:   typst

 0 client(s) attached to this buffer:

 Other clients that match the filetype: typst

 Config: tinymist
 	filetypes:         typst
 	root directory:    Not found. -- here is the issue
 	cmd:               /run/current-system/sw/bin/tinymist
 	cmd is executable: true
 	autostart:         true
 	custom handlers:

 Configured servers list: tsserver, clangd, coq_lsp, lua_ls, rust_analyzer, tinymist

I am using lazy for getting package in neovim
Just to explain why I don't use Mason, I am using NixOS and mason does not work it (as far as know).
I have installed through the nix package manager which is what i have done for other lsp.
I have everytime restarted neovim
I have other lsp setup that way and they work (cland, rust-analyzer).
tynymist is well installed on my computer

➜  ~ tinymist --version
tinymist
Build Timestamp:     1980-01-01T00:00:00.000000000Z
Build Git Describe:  VERGEN_IDEMPOTENT_OUTPUT
Commit SHA:          VERGEN_IDEMPOTENT_OUTPUT
Commit Date:         None
Commit Branch:       None
Cargo Target Triple: x86_64-unknown-linux-gnu
Typst Version:       0.11.1
➜  ~ tinymist -V
tinymist 0.11.9

Maybe I am doing something obviously wrong, that's why I post this issue.
Thank in advance.

I actually managed to make it work.
Actually in neovim. Instead of changing the tynimyst parameter it is the lspconfig parameter "root_dir" that should be changed.
This fix my issue :
      require("lspconfig")["tinymist"].setup {
        capabilities = lsp_capabilities,
        root_dir = function(filename, bufnr)
          return "/home/pauladam/PaulJR/" -- this is fixed, but other function can take the parent directory in neovim
        end,
    }

Sorry I'm late, I was busy when I first checked your issue. I think of it is a bit unfortunate that there are some unintentional similar keys that are confusing people. But root_dir is a neovim specific configuration. But we could document it to avoid future pitfall.

Just to be sure, it this way that should work to put settings for the tinymist ?
  {
    "neovim/nvim-lspconfig",
    dependencies = {
      "kevinhwang91/nvim-ufo",
      { "j-hui/fidget.nvim", opts = {} },
      { "folke/neodev.nvim", opts = {} },
    },
      local capabilities = vim.lsp.protocol.make_client_capabilities()
      capabilities = vim.tbl_deep_extend("force", capabilities, require("cmp_nvim_lsp").default_capabilities())

      local lsp_capabilities = require("cmp_nvim_lsp").default_capabilities()

      require("lspconfig")["tinymist"].setup {
        capabilities = lsp_capabilities,
        root_dir = function(filename, bufnr)
              return "the right root_dire depending on filename..."
          end
        settings = {
          tinymist = {
            settings = {
              formatterMode = "typstfmt",
            },
          },
        },
      }
    }

I don't see obvious errors in your posted code.

root_dir = function(filename, bufnr)
	       return vim.fn.getcwd()
           end,

This is much more dynamic. But this may change the workspace every time :(

For me, tinymist does not complain if there is a ".git" directory in the workspace path. So, if you don't want to fiddle with your neovim config, a quick and dirty fix is to simply create an empty ".git" directory.


Could not find method implementation() for arguments [project ':UnityExport']

Hello,
I followed the video (and also double checked everything with the docs), however when I try to build my android project, I am getting this error (sorry if this is not the right place to post this, I am a beginner in both flutter and unity):
Finished with error: Please review your Gradle project setup in the android/ folder.

Error running Gradle:
ProcessException: Process "/Users/ceciliamzayek/celuna_test/android/gradlew" exited abnormally:

FAILURE: Build failed with an exception.

Where:
Build file '/Users/ceciliamzayek/celuna_test/android/build.gradle' line: 20

What went wrong:
A problem occurred evaluating root project 'android'.

Could not find method implementation() for arguments [project ':UnityExport'] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 0s
Command: /Users/ceciliamzayek/celuna_test/android/gradlew app:properties

How did you fix this?

I ended up never needing it because I didn't need Unity for my project anymore


Add traffic shaping per-container

Closes #17

I think this is roughly ready now
Traffic shaping can be provided on a per-node basis using the graph property tc_netem. The commands are sanitized using a crazy regex from ChatGPT, which seems to work on current examples.
If no graph specified conditions are found it currently will use a random condition profile. I think this is quite nice, but we might want to have this require a startup option flag (or other), and not be the default? e.g.
warnet --randomize-network-conditions to turn that function on?


#1500 - Simplify JavaDocs

Hello everyone,
This pull request affects files in the following modules (and the project's POM file) for the inline return tag:
jpa
piranha-cdi-weld
piranha-embedded
piranha-http-api
piranha-http-impl
piranha-http-undertow
piranha-http-webapp
piranha-micro-builder
piranha-micro-core
piranha-micro-loader

I will be creating similar small pull requests that cover other modules as well.
Thanks,
Jafer

Thank you! And keep them coming :)

My bad that the builds failed again.
I had an invalid assumption about the inline return tag. I thought it could exist anywhere in the Javadoc but it should be the first line of the Javadoc in any case whatsoever otherwise the generation fails. And furthermore, they also failed because there was no description provided for a parameter so I had to add that in order to make them pass.
Please refer to the last commit for the changes that I made in order to make the builds successful.

Yea I reviewed the changes and it looked good. So I merged the PR!


Error when building

I'm getting this error while building scss project:
                 @error 'Media type definitions must be specified at the start of plus statements: (#{$range)})';                                                                                ^       Expected "}".     ╷ 307 │         @error 'Media type definitions must be specified at the start of plus statements: (#{$range)})';     │                                                                                ^     ╵   node_modules\mq-scss\_mq.scss 307:94      root stylesheet   projects\ft-core\style\_ft_core.scss 1:9  @import   stdin 5:9                                 root stylesheet       in C:\Users\thour\Documents\web-projects\ft-core-front\node_modules\mq-scss\_mq.scss (line 307, column 94)
this is a syntax error in line 307:
@error 'Media type definitions must be specified at the start of plus statements: (#{$range)})';

What's your input scss code?

I recon it's just a syntax error that's strange it happened to me only on. Yet I am using you lib for more than a year.
The issue is in line 307 :
@error 'Media type definitions must be specified at the start of plus statements: (#{$range)})';
It should be:
@error 'Media type definitions must be specified at the start of plus statements: (#{$range})';

Yeah but what @include mq(xxx) statement did you write to make the error occur?

Well I'm not sure which one, my app is an angular 7 one and the scss get maneged by the webpack loader.
The scss project is a large one and the mq is included multple times.
This is the full error stack:
`
ERROR in ./src/styles.scss (./node_modules/@angular-devkit/build-angular/src/angular-cli-files/plugins/raw-css-loader.js!./node_modules/postcss-loader/src??embedded!./node_modules/sass-loader/lib/loader.js??ref--14-3!./src/styles.scss)
Module build failed (from ./node_modules/sass-loader/lib/loader.js):
            @error 'Media type definitions must be specified at the start of plus statements: (#{$range)})';
                                                                                ^
  Expected "}".
╷

307 │         @error 'Media type definitions must be specified at the start of plus statements: (#{$range)})';
│                                                                                ^
╵
node_modules\mq-scss_mq.scss 307:94      root stylesheet
projects\ft-core\style_ft_core.scss 1:9  @import
stdin 5:9                                 root stylesheet
in C:\Users\thour\Documents\web-projects\ft-core-front\node_modules\mq-scss_mq.scss (line 307, column 94)
i ｢wdm｣: Failed to compile.`

I tried to make the error messages as helpful as I could since I couldn't guarantee that a line number would be produced where the error originates from. The idea being that you encounter the error when you enter the media query so you would know based on what the last mq you entered was.
If you really have no clue what mq is causing the error then you have no choice but to comment them all out one by one searching for @include mq until you find the culprit.
Until you know what the input scss is that is causing the error, I can't help you.
Based on the error message, it sounds like it is most likely an mq variable with a media type in it that is listed in a plus statement incorrectly in some way.

Well I don't think it's an error with the mq include cuz in another project env the build is successfull with same import, the error accured when i made an update of the @angular/cli and i think it also updated webpack scss loader as well. maybe the loader now do a full code check cuz as i said it's just a syntax error, when i fixed the error the build succeed.

Ahhh I see the bug in the error message now. It's a typo.
#{$range)} should be #{$range}
It has a bracket in the wrong place. Ok that is something that needs fixing in mq-scss so it outputs a propper clear error message.

When it is outputting a propper error message then it will be easier to understand what is going wrong.
Unfortunately I'm unable to work on this for a couple of days.

Oh I only just realised you pointed that out in an earlier comment https://github.com/Dan503/mq-scss/issues/5#issuecomment-477575120
😅
I was too focused on getting the input code from you.

Hhh not a big deal , I should have poined explecitly the error instead of the line.
Well it's not of an urgent, like a sais it's still works (I don't know why).
Thank you for your quik replay bud 👍 .

Fixed in version 2.1.5


Create anatoliacode

I don’t think this file is in the right place? And it needs a json file extension.

I don’t think this file is in the right place? And it needs a json file extension.

This is an automated message to let you know that this issue requires additional input from the original poster and for the health of the repository issue tracker the issue will be closed. This is to help alleviate issues hanging open waiting for a response from the original poster.

Do not let the issue’s closing deter you. After you’ve supplied the required information, the issue will be reopened. Thanks!

This is an automated message to let you know that this issue requires additional input from the original poster and for the health of the repository issue tracker the issue will be closed. This is to help alleviate issues hanging open waiting for a response from the original poster.

Do not let the issue’s closing deter you. After you’ve supplied the required information, the issue will be reopened. Thanks!


evenly distribute subhashes to increase speed

rebased my work onto latest code still runs faster, looking now at reworking to still possibly sort by shape
also includes a bugfix where load() assigned a empty Hashy onto hashes when caching was enabled but the file was not found, leading to undefined behavior

I am again sry to have corrupted your PR. I think I broke compatibility with cache files. And another interesting fact is, that big file sizes are not read correctly by the load function. So for me cache_13.bin always fails.
Isn't an empty return on read-fail the expected behavior? This should lead to a calculation instead of using the cache...

bugfix where load() assigned a empty Hashy onto hashes when caching was enabled but the file was not found, leading to undefined behavior

Now I experienced the same issue. Rather than initialize in cache.hpp, I initialize for n=1 and n=2 at their respective special cases in cubes.cpp.

i now put 8 subsets behind each shape which gives a small boost in speed. N=13 takes about 180s on an Intel 12700H with 20 Threads.


Feedback for: engine/installation/linux/centos.md

File: engine/installation/linux/centos.md
I received and error when trying to add a repo.
[user1 ~]$ sudo yum-config-manager --add-repo https://docs.docker.com/engine/installation/linux/repo_files/centos/docker.repo Loaded plugins: product-id, rhnplugin This system is receiving updates from RHN Classic or Red Hat Satellite. adding repo from: https://docs.docker.com/engine/installation/linux/repo_files/centos/docker.repo grabbing file https://docs.docker.com/engine/installation/linux/repo_files/centos/docker.repo to /etc/yum.repos.d/docker.repo Traceback (most recent call last):   File "/usr/bin/yum-config-manager", line 218, in <module>     result  = grabber.urlgrab(url, filename=destname, copy_local=True)   File "/usr/lib/python2.7/site-packages/urlgrabber/grabber.py", line 1172, in urlgrab     return self._retry(opts, retryfunc, url, filename)   File "/usr/lib/python2.7/site-packages/urlgrabber/grabber.py", line 1064, in _retry     r = apply(func, (opts,) + args, {})   File "/usr/lib/python2.7/site-packages/urlgrabber/grabber.py", line 1157, in retryfunc     fo = PyCurlFileObject(url, filename, opts)   File "/usr/lib/python2.7/site-packages/urlgrabber/grabber.py", line 1258, in __init__     self._do_open()   File "/usr/lib/python2.7/site-packages/urlgrabber/grabber.py", line 1589, in _do_open     self._set_opts()   File "/usr/lib/python2.7/site-packages/urlgrabber/grabber.py", line 1413, in _set_opts     self.curl_obj.setopt(pycurl.CAPATH, opts.ssl_ca_cert) TypeError: invalid arguments to setopt

Can you please try with the instructions as currently published?

No response in nearly 3 months. Please reopen if you have more info.

Had the same error, it's (was) a bug with yum/urlgrabber: https://bugzilla.redhat.com/show_bug.cgi?id=1023595
As a workaround (when you can't update yum) just get the file manually:
wget https://download.docker.com/linux/centos/docker-ce.repo
yum-config-manager --add-repo docker-ce.repo


eth_nxp_enet : get mac address from eeprom

Is your enhancement proposal related to a problem? Please describe.
When used in production environment all devices must have different "real" mac addresses.
Using a random (or derived from device id) mac address is not acceptable nor is using a common fixed mac supplied by the device tree thru local_mac_address property. It is very common to use an eeprom to provide the mac address.
Describe the solution you'd like
We need a way to provide mac address using an eeprom with eth_nxp_enet, like we already have on other ethernet drivers like eth_sam_gmac.
Similar to the eth_sam_gmac, configuration would be controlled by new optional device tree properties : ETH_NXP_ENET_MAC_I2C_EEPROM,
ETH_NXP_ENET_MAC_I2C_ADDRESS.
Thanks in advance for your feedback.
If that's ok I can submit a PR.

did you tried nxp,fused-mac? there is also the option to use nxp,unique-mac to in order to get a static mac address:
&enet_mac {
        /delete-property/ zephyr,random-mac-address;
        //nxp,fused-mac; // not working, seems not be available on mimxrt1050_evk board
        nxp,unique-mac;
};

see https://docs.zephyrproject.org/latest/build/dts/api/bindings/ethernet/nxp,enet-mac.html

maybe you also need to cherry pick this commit: https://github.com/zephyrproject-rtos/zephyr/commit/931628a78a52ac1d10b562761896cd0aa4bfda8b

@yuecelm thanks for the feedback.
I agree fused mac would have been a good solution, but unfortunately not existing on my MK64 SoC, where there is no such concept of fuses.
"unique-mac" is the solution I was referring to when I mentioned "derived from device id". Unfortunately this generate a fake mac using the device id. As mentioned in the doc, while this can be useful for testing or demos, this is not a valid option for production where we need "real" addresses.

Instead of adding Kconfig options for this, I'd like to see Ethernet drivers move towards the devicetree solution outlined here: https://github.com/zephyrproject-rtos/zephyr/pull/74835#discussion_r1651559112

@henrikbrixandersen thanks for the feedback. I totally agree that a pure device tree solution would be more elegant.
I see two options in the thread you mentioned.
1/ the simple mac-eeprom = <&eeprom 0x00> mentioned by @pdgendt
2/ the more complex with with nvmem-cells
I didn't find any usage of this nvmem-cells approach elsewhere in zephyr. Does it make any assumption on the kind of underlying storage ? Not sure to understand how do I know which api to call to access this (as opposed to the already existing and explicit eeprom api implied by the first syntax) ?
Since this need is going to be replicated in multiple eth drivers, it looks like some standardisation would be beneficial. As I am new to the zephyr community, I am not sure how do we reach some kind of common approach ?
Also adding @decsny to this discussion, since he made most of the recent contributions to this part of the eth_nxp_driver.

"unique-mac" is the solution I was referring to when I mentioned "derived from device id". Unfortunately this generates a fake mac using the device id. As mentioned in the doc, while this can be useful for testing or demos, this is not a valid option for production where we need "real" addresses.

I am glad to know the doc of that property was useful


Invalid Camera Slug - Docs addition?

After your round of updates recently i realized no snapshots where being pushed to homeassistant.  Digging into the logs I saw that it failed to start due to an invalid camera slug.
Error:
[2022-08-29 20:58:56] [ERROR   ] [viseron.helpers.validators] - invalid slug viseron-front (try viseron_front)
After some testing it appears that the background subtractor and object detector sections cannot have a - in the name for the camera definitions.
Valid:
  motion_detector:
    cameras:
      viseron_front:

Invalid:
  motion_detector:
    cameras:
      viseron_front:

Simply changing all to match the suggestion from the error worked and I even have a live feed for the first time ever working through the web interface.
I guess not so much an error but something to add to the docs?

Definitely!
I have changed how the camera identifier is validated, but i have not gotten to ffmpeg/gstreamer yet so they still allow the old values.
When I am done with the docs for all components the validation will be uniform across all of them.
Closing this for now since its in the works already. Thanks for the report.


Current Frame usage in Swift

Hello!
Quick question, when using the content view frame function in Swift
func contentViewFrameForPresentationController(presentationController: MZFormSheetPresentationController, currentFrame: CGRect) -> CGRect {
        currentFrame.size.height = 450
        currentFrame.origin.y = 20
        return currentFrame
 }

I am getting the following issue

Cannot assign to property: 'currentFrame' is a 'let' constant

Is there a way this can be a var instead of a let, or am I missing something?

You just need to do, var currentFrame = currentFrame, and you will be able to change it, in Swift 3 they will discard var in parameters anyway...

@m1entus got it, thanks!


Illegal instruction error.

My motherboard is raspberry PI 2B. After running /home/pi/nextion/bin/index.js, it appeared "Illegal instruction".

...no infos, no support!  :-(


Host the generated HTML, Javascript and CSS on a different machine

We have an internal Web API project which is hosted inside our company's network.
The swashbuckle works great and shows the documentation nicely when i change the URL to
internal project url/swagger
We also have an external project available on the internet which is a wrapper for the internal project.
It has one controller which redirects all the calls to internal project.
We would like to make the internal project's documentation available on the internet.
Is it possible to pull out the HTML, Javascript and CSS from the internal project?
Can someone please point me to the correct way to do this?

Take a look at swagger-ui. It's a dependency free set of static assets (HTML, CSS, JS) that should be fairly trivial to host independently of SB. With this setup, you would only use SB for generating and exposing the Swagger JSON endpoints. Then, you just modify you're independenty hosted swagger-ui (see index.html) to point to those endpoints.

@domaindrivendev - This is exactly what I ended up doing! Thanks again for this great library and support!


Make slug option work for all options

This should be implemented via net/http path rewrites.

Implemented in fefcd31d377f209152a620fd2935d09c6bac3bae.


Merge Dev ke UAT 7 April 2017

Coverage increased (+0.5%) to 64.743% when pulling f776d8a6fa0a403133d8f0fcf9792a20e837d289 on dev into d6b0bb45169d7b68dbb020a52a65a6becc33c92c on uat.


Entity Identifier that uses arcp protocol is not validated properly

When a crate has an entity that uses arcp protocol as its id, the validator library fails it, which causes it to create a whole bunch of new objects with id prefixed by #.
The main issue is with the validator library:
import { isURL as validatorIsURL } from "validator";
const urlProtocols = ["http", "https", "ftp", "ftps", "arcp"];

// this returns false
isURL('arcp://name,cooee-corpus/item/2-157', { require_protocol: true, protocols: urlProtocols });

// this returns true, so the isURL function only works for a strict URL, not URI such as arcp
isURL('arcp://namecooee-corpus.com/item/2-157', { require_protocol: true, protocols: urlProtocols });

If the id fails the validation check, the id will be replaced in this line entity["@id"] = `#${entity["@id"]}`;. From there, it will cause a different bigger problem especially when the data is big. A lot of new entities are being created which makes loading takes forever.
I think before a new entity is created and pushed, it should be checked first if there is already existing one.
Test data is attached here as zip file:
ro-crate-metadata.zip

Can you please provide a very basic crate with a basic root dataset and one entity using an arcp identifier for testing.

An example of a very basic crate:
{
  "@context": "https://w3id.org/ro/crate/1.1/context",
  "@graph": [
    {
      "@type": "CreativeWork",
      "@id": "ro-crate-metadata.json",
      "identifier": "ro-crate-metadata.json",
      "about": {
        "@id": "arcp://name,cooee-corpus/corpus/root"
      },
      "conformsTo": {
        "@id": "https://w3id.org/ro/crate/1.1"
      }
    },
    {
      "@id": "arcp://name,cooee-corpus/corpus/root",
      "@type": "Dataset",
      "@reverse": {},
      "name": "Test",
      "hasMember": [
        {
          "@id": "arcp://name,cooee-corpus/item/1-001"
        }
      ]
    },
    {
      "@id": "arcp://name,cooee-corpus/item/1-001",
      "@type": "RepositoryObject",
      "conformsTo": {
        "@id": "https://purl.archive.org/language-data-commons/profile#Object"
      },
      "name": "Text 1-001 1788 Phillip, Arthur"
    }
  ]
}

You can see that arcp://name,cooee-corpus/item/1-001 becomes a Thing and # is added as a prefix in the editor. To see the performance issue, please use the previously attached zip file as the test data.

There are two issues here.

The code wasn't handling arcp's correctly. That is now fixed.

The slowness was not due to the arcp bug but the massive entity lists on the properties (e.g. hasPart). The issue is that the browser was being crushed trying to render all those DOM elements. The only fix I can think of - that is in this commit - is to paginate those massive lists. So the code now has a default page size of 50 elements and a filter box to filter the list. I figure most people don't need to see the whole list at once. Pagination deals with that. And most people are probably looking for something in the list so filtering solves that.

I tried setting a large page size but when you have a few properties with large arrays on them then that adds up quite significantly in terms of browser load. 50 per page seems to be a reasonable default for now.
I've just built v0.23.0 with this code.

Just built 0.23.1 to fix a couple of issues in the new code.

Thanks @marcolarosa I can confirm that the arcp bug is fixed. I think this issue can be closed.


How do the results compare to other methods, such as U-Net?

I run this code, but very slowly. One epoch cost almost one hour, batch_size 4 is too large on 3090, 24GB.
So I want to know the result is better or worse compare to other method, such as Unet. And my task is medical image denoising.

Hi, so for comparison you should read the paper and you can see https://arxiv.org/abs/2303.11435. Short answer it should be better than a simple Unet. For the speed, it depends by a lot of factors: image size, training dataset size, ... . In addition, an hint is to reduce the image_size parameters when you define the model, it is not strictly related to the image size of your image, but it controls the channels of the convolutional layers.

Thank you! After modifying the model's image_size parameter, it works fine.
And I' am doing the supervised denoising using ddpm, but get worse result
than Unet.🥲
Francesco Pio Ramunno @.***> 于2024年7月22日周一 16:28写道：

Hi, so for comparison you should read the paper and you can see
https://arxiv.org/abs/2303.11435. Short answer it should be better than a
simple Unet. For the speed, it depends by a lot of factors: image size,
training dataset size, ... . In addition, an hint is to reduce the
image_size parameters when you define the model, it is not strictly related
to the image size of your image, but it controls the channels of the
convolutional layers.
—
Reply to this email directly, view it on GitHub
https://github.com/fpramunno/InDI-implementation/issues/3#issuecomment-2242382801,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL4OCSJL3ZGTXXZ6RLOYC53ZNS7CXAVCNFSM6AAAAABLHZQIXGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBSGM4DEOBQGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Which gets worse results, ddpm or indi?

DDPM gets worse results, about 0.5 psnr less than Unet
Francesco Pio Ramunno @.***> 于2024年7月22日周一 16:55写道：

Which gets worse results, ddpm or indi?
—
Reply to this email directly, view it on GitHub
https://github.com/fpramunno/InDI-implementation/issues/3#issuecomment-2242436209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL4OCSJRHOUSPL4MECMEPJTZNTCJVAVCNFSM6AAAAABLHZQIXGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBSGQZTMMRQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Also here it can depends by multiple factors. Are you aware of the Perception–Distortion trade-off? PSNR and others pixel level metrics like SSIM tends to prefer blurry output (it is also stated in the Palette paper https://arxiv.org/abs/2111.05826), so use perceptual metrics like lpips or similar. In addition the model size should be conisdered as an hyperparameter lowering this parameter improves the speed but with a cost of output quality. A fair comparison would be to use the exact same Unet for the basic approac and the DDPM/INDI approach.

Thank you for the suggestions, I'll calculate the perceptual metrics.
However, my dataset is medical images, I'm not sure if it's suitable to use
perceptual metric in medical image low-level vision tasks.
Francesco Pio Ramunno @.***> 于2024年7月22日周一 17:07写道：

Also here it can depends by multiple factors. Are you aware of the
Perception–Distortion trade-off? PSNR and others pixel level metrics like
SSIM tends to prefer blurry output (it is also stated in the Palette paper
https://arxiv.org/abs/2111.05826), so use perceptual metrics like lpips
or similar. In addition the model size should be conisdered as an
hyperparameter lowering this parameter improves the speed but with a cost
of output quality. A fair comparison would be to use the exact same Unet
for the basic approac and the DDPM/INDI approach.
—
Reply to this email directly, view it on GitHub
https://github.com/fpramunno/InDI-implementation/issues/3#issuecomment-2242460819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL4OCSJUSLGKCOPH4EUO2T3ZNTDU7AVCNFSM6AAAAABLHZQIXGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBSGQ3DAOBRHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

That's really dependent on your data, anyway it is always useful to have several metrics of different nature!

No problem at all, for any question just ask! :)


Cooperation

Hi there,
we're another project from the WirVsVirus hackathon and are trying to tie together all the international efforts that try to work on this problem.
Which leaves the question: How can we best work together? :-)
We coordinate here right now, I would very much like to invite (some?/all?) of you to join our discord or use any other means necessary so we can get into contact.
https://start.ito-app.org

Hi, Martin,
currently we are stabilizing the app and focus on an implementation with an economical data collection and data processing (GDPR). We are already in contact with other projects and would be happy to exchange ideas with you. But we also have to note that the project is still run by purely private engagement and we support it within our possibilities.
Currently we use five platforms for exchange and coordination. Another channel increases the complexity and the coordination effort. I personally try to stay in touch, but I can't make any promises about affordability from me and the team at the moment.
Many greetings,
René

currently we are stabilizing the app and focus on an implementation with an economical data collection and data processing (GDPR). We are already in contact with other projects and would be happy to exchange ideas with you. But we also have to note that the project is still run by purely private engagement and we support it within our possibilities.
Currently we use five platforms for exchange and coordination. Another channel increases the complexity and the coordination effort. I personally try to stay in touch, but I can't make any promises about affordability from me and the team at the moment.

Oh I can so sympathise. :-)
That being said, this approach can only work if we establish one protocol to rule them all (tm) or we will simply not get the warning speed that we need to actually stop the pandemic.
From calculations some of our partners at TU München did we currently think that for this approach to be effective, we probably need to reach about 50% of the population, and be able to notify first and second order possible infections.
So, working together is of the essence.
Also: a new player has just appeared: https://www.pepp-pt.org/content https://www.pepp-pt.org/content
Best regards, and lets have a talk how we can best join forces - just let me know how we can best have a phone call.
Best Regards,
Martin Häcker

sure, we can do that. we'll consult each other within the team again.
FYI: i wrote to pepp-pt and asked for more information :)

Please let me know how we can get into contact.
If it is not too much trouble, we will have an all hands meeting tomorrow at 19:00 on our discord - that would be an excellent time to make introductions and get to know each other.


Updated 100K export limit to be locations instead of mailboxes

@markjjo - For your review.

@mahage-msft PR merged


[Beta] Add periods to list items

we should only do this when items in the list are complete sentences

oh, got it. I will close this.


[BUG] Change workspace in Continuous Delivery does not work

Rancher Server Setup

Rancher version: 2.7.1
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): Helm install on RKE2 v1.24.7+rke2r1

Proxy/Cert Details: no proxy, let's encrypt

Information about the Cluster

Kubernetes version: RKE2 v1.24.7+rke2r1
Cluster Type (Local/Downstream): local and downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

Infrastructure Provider = AWS

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

Admin/Cluster Owner

Describe the bug
I can not move a cluster in Continuous Delivery from the fleet-default workspace to a new workspace
To Reproduce
Go to Continuous Delivery, go to Advanced then Workspaces
Create a new Workspace
Then go to Clusters in Continuous Delivery
From the triple dot menu on the right, select Change Workspace
Select the new workspace and select Apply
Result
The cluster stays in the fleet-default workspace
Expected Result
The cluster should move to the new fleet workspace, in my example the cluster should move to fleet-new
Screenshots

After trying to move a cluster to the new workspace, the new workspace remains empty

Additional context

I believe this is a duplicate of https://github.com/rancher/rancher/issues/36132. It has more detail and context on it.

Closing in favor of https://github.com/rancher/rancher/issues/36132


Fixing some typos in existing docs

Just one comment. I think the use of "forces" was correct. You can use "forces to clarify" or "enforces clarity of".

You're right, I've changed it back again in this pull request. I also want to add more docs as soon as I'm more into your fantastic language!


docs: add spec for summary report

fixes #74
adds spec:

summary-report.qmd
summarize-report-function.qmd, with new section in module_function.qmd
summarize-tool.qmd
validate-tool.qmd

I should probably include in the spec how valid tables say they are valid, in the summary report, unless that's something only the validation tool needs?

The module-functions.qmd file looks good but I'm looking for more from the specifications.
Looking for something similar to the specifications for the rules,

Text explaining why the function is needed
Explaining how it will work with examples that can be used as test cases.

I made an attempt at addressing this now.

Also, this was not in the task but I think we discussed allowing the user to summarize by tables, rows, and/or columns.

I'm not sure about how we would implement this and how useful it would be. We are currently summarizing by tables, which is fine and logical. Summarizing by rows sounds like it would undo the summarization, since aggregating rows is its main feature. I'm not sure about columns. I suppose we could prefix them with table names, so it becomes like 'sites.siteID: : ', otherwise I think it would be confusing. Should we postpone it?

Also, I took a brief look at your latest commit. Can you also put in examples like in the validation rules, so I can see what the output looks like and also so I know what you will be testing. Specifically,

What the summary object will look like. Ignore the functions in it.
What the summary print out will look like.

@yulric. Sounds good, I've made new changes.

@zargot I pushed a commit to high level spec making things clear. Feel free to rebase things to make it all clean.


Reverse of a number

This program reverses the given number.
It is written in python.

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Improve UX of forms builder/repeater with collapsible()

When using the repeater/builder with collapsible() clicking the label toggles the collapsed/expanded status. However, as an end user I'd assume the label and empty space is clickable to toggle this, and that clicking the buttons of course would do their own thing.

Adding a simple flex-grow on the element containing the label does the trick.

Can be handed in #11017

Apologies, didn't see a PR was in progress.


A validation of prototype which we can restart without duplicate snapshot from raftstore leader

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-unit-tests


Unsigned module error triggers when no signed module is installed

Re:  #94 (now closed)
I hope to write about this, but I need to understand it myself. Thanks in advance for your help.
If I understand correctly, the catalog signing error appears only when:
-- An installed version of the same module is signed, but the version that the user is trying to install is not signed.
-- This signing check is module-specific.
But, on my VM (Server 2016 RTM, 5.1.14393.693), the installed version of Pester, 3.4.0, does not have a Pester.cat file. However, when I try to install 4.0.2, which also does not have a Pester.cat file, I get the catalog signing error.
What's going on here?

Also, the error message is very confusing. It refers to the version that is not installed (4.0.2) as the "previously-installed module 4.0.2"*. That was probably supposed to be the module name, not the version. I'll see if I can track that down and submit a PR.
Ensure that the version '4.0.2' of the module 'Pester' has the catalog file 'Pester.cat' and signed with the same publisher 'CN=Microsoft Root Certificate Authority 2010, O=Microsoft Corporation, L=Redmond, S=Washington, C=US' as the previously-installed module '4.0.2' with version '3.4.0'
under the directory 'C:\Program Files\WindowsPowerShell\Modules\Pester\3.4.0'.

Hi @juneb, thanks for submitting a PR for fixing a typo in format!

Yes, publisher validation is module-specific.
It is applicable only when the latest version of the module being installed is signed.
Publisher details are retrieved from either catalog file or module manifest. First, catalog file otherwise module manifest file. For all in-box modules, module manifest is signed.

"It is applicable only when the latest version of the module being installed is signed."
[juneb] The error message occurs when the module being installed is unsigned and there is a signed version of the module already installed. Why would we throw an error for a signed module?
Also, on my VM (Server 2016 RTM, 5.1.14393.693), the installed version of Pester, 3.4.0, is unsigned. It does not have a Pester.cat file. However, when I try to install 4.0.2, which also does not have a Pester.cat file, I get the catalog signing error. What's going on here?

"It is applicable only when the latest installed version of the module being installed is signed."
Latest version from the installed versions of module are considered for the publisher validation.
In above mentioned case (in your above reply), installed latest version is signed and version being installed is unsigned, right? Since the new version is not signed, publisher validation should fail, correct? what is your expectation?

No. That's why I'm confused. On my VM, the latest installed version, 3.4.0, is unsigned (no Pester.cat file in the module). In fact, both 3.4.0 and 4.0.2 are unsigned. So, I can't figure out why I'm getting the catalog signature error.

Please take a look at the third point I mentioned above.
Publisher details are retrieved from either catalog file or module manifest. First, catalog file otherwise module manifest file. For all in-box modules, module manifest is signed.
Check the authenticode signature of Pester.psd1 file under 3.4.0 version.
If that module manifest file is not authenticode signed, please provide the -debug output to investigate further.

Ah! Now, I see. Thanks.
So, is this an error in Pester, because the .psd1 in 4.0.2 is unsigned?

I've got a draft of catalog signing docs I need to share. Will make sure it's pushed to a viewable repo & share with @June before EOD 3/8

Thanks, Keith. That's great. We need some best practice guidance.

Just thought i'd add @juneb, the version of pester on a fresh install of windows 10 does not come from the gallery.  and that is why you cannot just do a Update-Module -Name Pester.

@dotps1 Thanks, I used Install-Module.
The .../Pester/3.4.0/* directory does not include a .cat file, but Get-AuthenticodeSignature returns a valid catalog signature.
But, I don't see one of those long signature blocks in the file. Is the embedded signature in metadata?


Traditional Chinese | Translation Patch

woop

And the formatting isn't broken!! :clap: I'm so proud :smile:


added taxonomy adaptor support

Added support for including ncbi_taxonomy databases using the new ensembl-taxonomy API

where are the test cases? ;-)

And would we still able to create a Compara adaptor on that database ?
Our production registry file loads everything from staging1 and stagin2, and then create a few DBAdaptors on manually listed databases. We use that during the release and one of them is the ncbi_taxonomy database
https://github.com/Ensembl/ensembl-compara/blob/release/82/scripts/pipeline/production_reg_conf.pl#L147
Matthieu

thats actually a good point. There are tons of test cases for this API
itself in ensembl-taxonomy - but having a test for taxonomy in core
would mean that core would need to check out taxonomy as well and have a
test taxonomy db. Is this a dependency you'd want, or are you happy with
this being tested in the ensembl-taxonomy repo?

Can MultiTestDB create an empty database ?
Perhaps the test should not import the repo, and test that the API dies with the message "Bio::EnsEMBL::DBSQL::TaxonomyDBAdaptor not found in @INC) :D

Good question, I'll look into this. If you're manually adding the DB to
the registry, the species and group are different, I don't see why not.

Your suggestion of an fail is not a bad one I guess - I'll work on the
registry tests in ensembl-taxonomy in any case and then come back to this.

Actually, testing the registry seems to be a big problem, where you need
to test whether a particular db is picked up and added by one of the
load methods, as the name is crucial. In my case, my little test NCBI
database gets called dstaines_test_db_multi_taxonomy_20151112_111753
which won't get matched with by the registry. I think the only way to
test the registry is to load our test server using predefined names and
make sure the registry finds the database in question. Is this something
that MultiTestDB can do easily?

should the taxonomy db not be added to the list of known databases anyway?
I am thinking about this section:
https://github.com/Ensembl/ensembl/blob/release/82/modules/Bio/EnsEMBL/Registry.pm#L1780
where all 'special' databases are meant to be picked up
We are currently not dealing with the taxonomy database there, which has caused problems in the past as it gets included along with the core-like databases
This means we add to explicitly exclude it
https://github.com/Ensembl/ensembl/blob/master/modules/Bio/EnsEMBL/Registry.pm#L3012
If we pick it up looking for 'taxonomy', would that also work in your use case?

Its in that list already, unless I'm missing something:
https://github.com/danstaines/ensembl/blob/master/modules/Bio/EnsEMBL/Registry.pm#L1809
I've followed a similar approach to ontology, but without the
versioning. AFAIK, we don't need it in get_species_and_object_type.
Am happy to make any changes you suggest though. The main problem is
still testing, but I think thats something generally applicable to much
of the registry code as far as I can tell.

Oops, yes, my bad.
What happens if you allow a prefix for the db name?

Thats not supported with my code change, mainly since production and
ontology don't support it as far as I can tell (these were my models for
taxonomy). I can add it if we really think we need it, but I can't think
of a scenario where we would.

The registry is fairly untestable at the moment and because of our unique test db naming scheme I can't see an easy way to do this unless we fancy injecting a fake db handle and responding with a result list of dbs we expect it to handle/process
Sent from my mobile.

On 12 Nov 2015, at 11:23, Dan Staines<EMAIL_ADDRESS>wrote:
Actually, testing the registry seems to be a big problem, where you need
to test whether a particular db is picked up and added by one of the
load methods, as the name is crucial. In my case, my little test NCBI
database gets called dstaines_test_db_multi_taxonomy_20151112_111753
which won't get matched with by the registry. I think the only way to
test the registry is to load our test server using predefined names and
make sure the registry finds the database in question. Is this something
that MultiTestDB can do easily?
—
Reply to this email directly or view it on GitHub.

Given what Andy said, about the best I can do is to point at the public
EG server and make sure I get a tax adaptor back from el Reg. I've added
this to ensembl-taxonomy repo (which currently has one other test using
the full size public ncbi_taxonomy db), so would you be happy for the
pull req go through? I've tested the scenarios that Matthieu was
concerned about.

Continuing my theme of making mock testing servers and coming in with
minimal context for the issue at hand, I just thought I'd leave this
sitting here and wander away...
https://metacpan.org/pod/DBD::Mock
Create a mock driver, program it with the expected results for queries,
profit.
On Thu, Nov 12, 2015 at 12:13 PM, Andrew Yates<EMAIL_ADDRESS>wrote:

The registry is fairly untestable at the moment and because of our unique
test db naming scheme I can't see an easy way to do this unless we fancy
injecting a fake db handle and responding with a result list of dbs we
expect it to handle/process
Sent from my mobile.

On 12 Nov 2015, at 11:23, Dan Staines<EMAIL_ADDRESS>wrote:
Actually, testing the registry seems to be a big problem, where you need
to test whether a particular db is picked up and added by one of the
load methods, as the name is crucial. In my case, my little test NCBI
database gets called dstaines_test_db_multi_taxonomy_20151112_111753
which won't get matched with by the registry. I think the only way to
test the registry is to load our test server using predefined names and
make sure the registry finds the database in question. Is this something
that MultiTestDB can do easily?
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/Ensembl/ensembl/pull/97#issuecomment-156087822.

On 12/11/15 13:16, Matthew Laird wrote:

Continuing my theme of making mock testing servers and coming in with
minimal context for the issue at hand, I just thought I'd leave this
sitting here and wander away...
https://metacpan.org/pod/DBD::Mock
Create a mock driver, program it with the expected results for queries,
profit.

This kind of approach is definitely something to look at, I agree,
paritcularly where you have complicated logic like this. As always the
devil is in the details - for instance, I'm not sure how Mock deals with
queries on multiple schemas (presumably with distinct bits of SQL that
you hard code). Also the registry has the mysql hardcoded in it (less of
a big deal, but still).
Something I feel I need to leave with my core colleagues for further
consideration, and also wander away.
--
Dan Staines, PhD
Genomics Technology Infrastructure Coordinator
EMBL-EBI, Wellcome Trust Genome Campus
Cambridge CB10 1SD, UK
Tel: +44-(0)1223-492507


Remote access

It should be nice to be able to access remotely to a livebox
address   https:// my.ddns.address;network
user   user
password
port: 12000 for example

It should be nice to be able to access remotely to a livebox
address https:// my.ddns.address;network user user password
port: 12000 for example

Version 2.0.0-beta offers this functionality when adding a livebox


default filter

Right now, the default filter is implemented in the Compiler itself which I think is not the correct place.
IMO it would be better to implement it as a NodeVisitor. This would allow other compilers not to duplicate this logic.

The same applies to the "defined" test.

The code has been moved away from the compiler some days ago; see 4a91f4ff34f6d704773ca6300758a928394bfe85

Thanks for sharing haven't seen that.
What do you think about moving this to a NodeVisitor? Although not necessary for what I'm doing, hard-coding the functionality of a filter in the general expression parser doesn't really seem like the correct place either. I'm doing something similar to the "default" filter, just for translations:
https://github.com/schmittjoh/JMSTranslationBundle/blob/master/Twig/DefaultApplyingNodeVisitor.php
I haven't looked through all the other nodes, so I'm not sure if there are other cases for this, but from an architectural POV it would be best imo if the compiler was not making any changes to the AST, but simply taking it as is and compiling it to PHP code. Any changes could be moved to dedicated node visitors (similar concept to compiler passes from the Symfony DI).
While implementing a compiler, I also stumbled over the "defined" test which currently has some extra logic that is hard-coded in the Expression/Test node. This doesn't feel very clean, maybe an extra node type would be more appropriate?

I have just refactored the code so that the default filter code is more easily reusable: 846a2c13

Could this maybe be class Twig_Node_Expression_Filter_Default extends Twig_Node_Expression_Filter?

@nikic: The Twig_Node_Expression_DefaultFilter is a wrapper and so it's not really a filter anymore.

After the refactoring, the PHP code looks a bit odd:
Hello {{ name|default("World") }}!

        echo "Hello ";
        if (isset($context["name"])) { $_name_ = $context["name"]; } else { $_name_ = null; }
        echo twig_escape_filter($this->env, ((array_key_exists("name", $context)) ? (twig_default_filter($_name_, "World")) : ("World")), "html", null, true);
        echo "!";

Some of the checks seem redundant. Is this the desired output? I also don't understand the new temp node, at least in the above example it doesn't make sense to me, and I have seen similar behavior in other templates.

The code was exactly the same before the refactoring. This optimization works great in all cases except your specific example.

it seems to me that the existence check is done twice here

@stof: I know but I don't know if there is a simple fix for that.

I've created a more general ticket:
https://github.com/fabpot/Twig/issues/473


List of console scripts (entry points) in setup.py needs updating

See https://github.com/icecube/pisa/blob/a752eaf0f7d75c3d002d3c8ccde4a100ca1bf594/setup.py#L311 and https://setuptools.pypa.io/en/latest/userguide/entry_point.html.
Console scripts that are non-existent (test by running each command) and should be removed:

pisa-analysis
pisa-postproc
pisa-fit_discrete_sys and pisa-fit_discrete_sys_nd
pisa-make_asymmetry_plots
pisa-make_systematic_variation_plots
pisa-make_toy_events
pisa-profile_scan
pisa-scan_allsyst

Working scripts for which entry points are missing (possibly incomplete):

scripts/create_barr_sys_tables_mceq.py
pisa_tests/test_Detectors_class.py
pisa_tests/test_covariance.py
pisa_tests/test_kde_stage.py

We can delete pisa_tests/test_Detectors_class.py, it just duplicates code from https://github.com/icecube/pisa/blob/master/pisa/core/detectors.py#L346