Return alternate profiles & mediatypes

This feature hasn't been implemented yet, renaming issue to reflect.

Expected behaviour from Prez v2.
The UI needs to return data from alternate profiles & mediatypes to the browser from the API.
e.g. https://<base_url>/v/vocab/<vocab_id>?_profile=vocpub&_mediatype=turtle should return a turtle format of the vocpub profile of a vocabulary.
The simplest solution would probably be a simple redirect to the API.

Note that the set of profiles supported may vary according to the "itemClass" in a register - i.e. different classes of objects would have different sets of profiles defined - for example dcat:Datasets should have a profile that exposes a provenance history view of the dataset we wouldn't calculate by default - but "spacePrez" may not require this per item...
so the configured views need to be inspected (not necessarily at run-time) and the list of alternates generated - and ideally this would be accessible for the redirect tool of choice to use or convert into a configuration.
(I've been managing these aspects separately and its a real pain to keep in sync with a realistic number of different types of objects in a knowledge graph...)


Add lsp support to sh module.

Can you add the dependencies for the server too in the README ? (thanks for adding that by the way !)
I don't know which servers are replying to requests for shell files

I need a little clarification about this change request, what exactly should I add to README. In the original PR I've already added bash-language-server to the list of of dependencies. Should I add a link to the bash-language-server repo?

Ah, I've just realized that lang/sh's README hasn't been updated to the new format. In that case I'll accept the PR as is and reformat it myself later.
I'll merge once I get home later tonight.

I've already added bash-language-server to the list of of dependencies.

My bad, I definitely missed the line since I was expecting a hefty org link in the change

Thanks for your help!


Made update file shell executable for linux

you may not see anything but now people can run "./update.sh" and it will run

this aged like milk

re opened because read me is misleading

the folder for common commands is /bin/bash not /bin/sh

alreadly added

OH


Support Tesseract 4.0

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
The Android library this uses only supports tesseract 3.0, this is because the framework underneath last got updated 2 years ago. There is a fork of that library that seems to of upgraded to v4.1 of tesseract. https://github.com/alexcohn/tess-two
Describe the solution you'd like
A clear and concise description of what you want to happen.
Switch your underlying android implementation to the other tess-two library that supports the newer version of tesseract.

There is the fork of tess-two library upgraded to v4.1  https://github.com/adaptech-cz/Tesseract4Android and the dependency  for that is
dependencies {
    implementation 'cz.adaptech.android:tesseract4android:2.1.1'
}

@arunahuja94 Is it easy to switch across to this? I imagine the function signatures changed between versions?

@dllz I decided not to use alexcohn's fork since it might not receive long term support (you can check it here), however I have been looking for another option and I think the one suggested by @arunahuja94 seems pretty stable at first look. I will try to dedicate some time to check if it is something feasible in the next few days but if this is something you have already worked on, please feel free to submit a PR and I will review it. Thank you guys!

I have not had a chance to work on it yet, been finishing up some other things first. That being said I will start digging into it tonight and keep you updated.

@dllz it is working after changing the dependencies , but the output has not improved , image preprocessing is required for that, haven't found any good way to to do that in react native yet

@arunahuja94 Surprisingly for my use case I have noticed a huge improvement, speed is a lot better as well on mobile, worse in emulator, also switching to the 'best' data files made a big difference https://github.com/tesseract-ocr/tessdata_best


Release cargo-screeps 0.2.1.

Changelog for this release:

Release cargo-screeps version 0.2.1

Update expected format to match cargo-web version 0.6.19
Fix subcommand documentation in README

Fixes #89. #75 was merged, but not released.

Merging this now just to get a fix out for #89, and because it has no real code changes itself. Wouldn't do this if it changed the codebase, but it should be alright for a release?


All google-cloud-* packages broke due to poor dependency management

All google-cloud-* packages broke due to poor dependency management when version 0.21.0 was released.
All google-cloud-* packages depend on google-cloud-core >= <current_version>. Version 0.21.0 was a breaking change, compared to 0.20.0.
As a result, a dependency on a pinned down version is automatically broken as soon as a new version is released.
The workaround is fairly simple. If one depends on google-cloud-*, one must explicitly specify the version of google-cloud-core to depend on.
The real issue lies at dependency management of the google-cloud-* packages. Dependencies should be locked down in the packages that use them. So the requirements should be changed from google-cloud-core >= <current_version> to google-cloud-core ~= <current_version> (assuming semver is used and patch versions are non-breaking).

OS type and version
Ubuntu 16.0.4
Python version and virtual environment information python --version

$ python --version
Python 3.5.1+

google-cloud-python version pip show google-cloud, pip show google-<service> or pip freeze

$ pip list
Package                  Version
------------------------ -----------
...
google-cloud-core        0.21.0
google-cloud-datastore   0.20.1
google-cloud-storage     0.20.0
...

Stacktrace if available

      File ".tox/docs/lib/python3.5/site-packages/google/cloud/datastore/__init__.py", line 55, in <module>
        from google.cloud.datastore.connection import Connection
      File ".tox/docs/lib/python3.5/site-packages/google/cloud/datastore/connection.py", line 23, in <module>
        from google.cloud import connection as connection_module
    ImportError: cannot import name 'connection'

Steps to reproduce

pip install google-cloud-datastore~=0.20.0

Thanks for reporting @remcohaszing!
@dhermes can we do a 0.20.2 release that fixes this as well?

Thanks for filing @remcohaszing, it was also reported yesterday (#2739).
This was an oversight on my part not putting upper bounds on the dependencies.
@daspecster I don't see much value in releasing a 0.20.x fix. The correct fix is to upgrade to 0.21.0. Maybe I am missing something here? Maybe @jonparrott can enlighten me on the dos-and-donts of fixing one's packaging mistakes after they've already hit PyPI.

I am curious how you ended up in state where google-cloud-core was upgraded but the other packages were not.

@dhermes if you make a clean venv and install pip install google-cloud-datastore~=0.20.0.
Then
from google.cloud import storage

That's how you get the error that's mentioned.
So if you're app is locked to a version and you rebuild it, it will break.

@dhermes This state can be achieved exactly as @daspecster said.
I assume most packages use semver nowadays. This means a dependency locked as package-name ~= major.minor.patch shouldn't be breaking. This is technically handled correctly, except that the google-cloud-core dependency should be locked using google-cloud-core ~= 0.21.0 instead of  google-cloud-core >= 0.21.0.
Just for clarity, this is the same as google-cloud-core >= 0.21.0, < 0.22.0.

Per @dhermes,

@remcohaszing I am curious how you ended up in state where google-cloud-core was upgraded but the other packages were not?

Is there something special about your environment that keeps you from upgrading all packages?

I don't think it's necessarily specific to my environment. I think it makes sense to lock down versions. Actually I still think this is a good idea. #2706 is a breaking change, an automated update to the new version including it has been prevented.
I have upgraded by now, but I had to do some refactors. It had become top priority, because tests were failing for unrelated patches in CI.


shared/x509utils: use core.SplitHostPort() to SanitiseName()

net.SplitHostPort() doesn't like portless strings

shared/v0.4.10 tagged


Fix for connectors based on refactoring of schemas

Describe your changes :

This fixes Mysql Source connector

Type of change :

[x] Bug fix
[x] Improvement
[x] New feature
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] Documentation

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@harshach @pmbrull

Few things to note:
Sample Data
Table Profiles
Table Queries
Data Model
are not part of CreateTableRequest - Commented those out for now - will add a fix (include them OMetaDatabaseTable model or other approach - create a ticket for the same)
cc: @harshach @pmbrull

Let me take a stab on this and we can discuss it together tomorrow 🙏

Implemented in https://github.com/open-metadata/OpenMetadata/pull/3870


Interface contract outside of the struct implementation

Hi everyone, thank you for introducing the new way of service implementation.
I'm trying to implement a POC for the service weaver and I stumbled upon this error
internal/domain/reverser/reverser.go:16:6: weaver.Implements argument contract.Reverser is a type outside the current package.
The thing is, implementing the contract in a separate package could be a better solution, well at least for the implementation I'm tryna made.

I'm just wondering, is there any specific case of this rule that prevented the generator to use the interface contract outside of the implementation package?
// Implementation of the Reverser component.
type reverser struct {
	weaver.Implements[contract.Reverser]
}

Thanks for reporting. Yes, we made a conscious decision not to support implementations of out-of-package interfaces, at least for now. @mwhittaker will have more state on this.

Hi @ghazimuharam! Here's the code in weaver generate that disallows a struct from implementing a component interface outside its package:
https://github.com/ServiceWeaver/weaver/blob/35727ae307c392d27cadfbf5302d56a3e76977ab/internal/tool/generate/generator.go#L357-L360
As Srdjan said, this was a conscious decision. I don't remember exactly why we made this decision; it might have just been a simplifying assumption for our initial release. Let me think harder about whether there's any disadvantages to allowing a component implementation to be in a separate package from its component interface. If there's none, I can take a stab at relaxing the restriction!

Thank you, @spetrovic77 and @mwhittaker, for responding to my issue. As this appears to be a result of conscious decisions, I understand that I may need to wait for further clarification. However, please do not hesitate to reach out if you require any help from me. In the meantime, I will try to familiarize myself with the entire repository.

Just to share that I've found the simple workaround to embed the external interface (from another package), to a local interface (same package) that is getting referenced by weaver.Implement. The code generation works fine.

We explored the idea of allowing component interfaces and implementations to be in separate packages in #282 but ultimately decided to disallow it. In #285, we're updating an error message to suggest @jpearll's workaround.


I18nFileLoadError for locales ‘am’ and ‘ml’ due to Misidentification in load_directory

Body
Hello,
When the user's locale is set to 'am' or 'ml', the code in the load_directory function of resource_loader.py incorrectly assumes that a corresponding localization file exists. This leads to an i18n.loaders.loader.I18nFileLoadError. The error message displayed is:
I18nFileLoadError: error getting data from resources/translations/de.yaml: ml not defined

This issue arises due to improper file existence checking. The locales 'ml' and 'am' are mistakenly identified as existing because they are substrings of the ".yaml" file extension. This results in the erroneous assumption that the relevant localization files are present.
Suggested Fix
A more robust method of checking for the existence of localization files should be implemented to prevent substring matches within file extensions.

I met similar issue too.
load_directory function choose the  wrong translation file due to wrong translation filename is substring matched first.
In my case, when locale is 'zh', it should load 'zh.json', but it loads 'zh-tw.json' instead.


TryIt support in VSCode

 Issue by kaviththiranga
Monday Feb 11, 2019 at 06:23 GMT
Originally opened as https://github.com/ballerina-platform/plugin-vscode/issues/38

[ ] collect ideas on required features
[ ] design discussion

The swagger view was embedded in the extension.


[Doc] Create Naming Conventions doc

Description of Change

Created a new documentation file for Naming Conventions containing determinations around prop naming.
Created #295 for updating component props.
Another PR will be created against the app repo to import this new doc file to the doc site once it is merged to main.
Testing

None needed, doc only.
PR Checklist
Code reviewer validation:

General

[ ] PR is linked to ticket(s)
[ ] PR has changelog label applied if it's to be included in the changelog
[ ] Acceptance criteria:

All satisfied or
Documented reason for not being performed or
Split to separate ticket and ticket is linked by relevant AC(s)

[ ] Above PR sections adequately filled out
[ ] If any breaking changes, in accordance with the pre-1.0.0 versioning guidelines: a CU ticket has been created for the VA Mobile App detailing necessary adjustments with the package version that will be published by this ticket

Code

[ ] Tests are included if appropriate (or split to separate ticket)
[ ] New functions have proper TSDoc annotations

Publish

Changes do not warrant a new version per the versioning guidelines.

@jessicawoodin as an FYI if you had any additional comments/thoughts on the prop naming guidance, but doubt we'll have any components needing beyond variant/tone or more specialized functional naming in the near future.

@jessicawoodin as an FYI if you had any additional comments/thoughts on the prop naming guidance, but doubt we'll have any components needing beyond variant/tone or more specialized functional naming in the near future.

Looks great to me! I think this all makes sense.


vendor: Update sigs.k8s.io/structured-merge-diff/v4

This updates the dependency to 9f9c77085dec, as it was previously a
temporary fork. This includes the fix to
https://github.com/kubernetes-sigs/structured-merge-diff/pull/173. At
the time of this commit, there hasn't been a release made including this
fix, so we are explicitly using the latest commit.
Signed-off-by: Chris Tarazi<EMAIL_ADDRESS>

test-me-please

test-me-please

Marking for backport to ease consumers importing the v1.9 branch of Cilium


Problem !!

Hello. I have a problem with your library. The code only works for me on arduino uno and arduinlo nano boards. (Same MCU). On Leonardo and arduino Mega boards, the library is broken.

I just tested this library on an Arduino Leonardo and an Arduino Mega 2560, and didn't encounter any problems.
I'm going to need some more information to be able to help you.

What version of the library are you using? 1.0.5 is the latest.
What code/sketch are you running?

Is it one of the example programs? If so, which one?
Is it your own code or a modified example? If so, please post it here; copy and paste the entire text of the sketch.

What are you using as a slave/server device to test the library?

I'm using your sample code with buttons and buttons for testing. everything works great on arduino uno and arduino nano. On this basis, I built a device communicating with ten slave units. Everything failed when trying to incorporate mega. so I tried a purely basic sketch of the slave device and the master device themselves on the example code. unfortunately the sketch failed and I don't know why. according to the oscilloscope, neither device is transmitting. Tested even without max485 modules on direct rx/tx Tx/Rx. if I use only a simple sketch for sending data, it works on the given pins. I don't know what could be wrong. Could it be an IDE compiler error ? I am using the latest version of the library.

What pins are you using for RX and TX?

For Pins softwareserial, I used the default pins according to the sketch, then tried it on pins 3, 4, 5, then I switched to using the hw port on the megabyte. By default, I use the classic serial1 and for that I have already started it somehow, but only if I use only one slave, if I modify the code and add another slave like with nano, the other one doesn't work for me. at the same time, I now have a string of 15 slave devices on the nano. however, hw serial now at least communicates with one slave. but I don't know why the serial software doesn't work. , unfortunately it still doesn't work on leonard even when using the HW serial port.

SoftwareSerial is particular on what pins can be used on the Arduino Mega and the Arduino Leonardo. The details can be found in the Arduino Docs for the SoftwareSerial library.
The ModbusRTUMasterExample sketch is only setup to talk to one slave device. I'm guessing you made some modifications. Can you post your sketch here?


[EIA] Produce DCID using epaGhgrpFacilityId and reference to Power Plant entities

Multiple GHG Facility IDs can map to the same FRS ID or power-plant ID.  So we derive the DCIDs using the GHG facility ID, and attach the other properties as epaFrsId and PowerPlant reference (as partOf).

Thanks for the review!


[ggj][ast] chore!: add writer visitor and restructure packages to fix dep cycle

Adding the start of the visitor pattern for serialization. This also introduced some bazel cyclic dependencies, so I'm moving some files (which fit better in ast anyway) over to the ast package.

Updated the comment.


Rewrite command system to actually work

This would allow for easy sub-subcommand creation which would be very helpful in WaterCore

I've decided to completely rewrite the command system, using a brigadier-esque node system

see https://github.com/lucyy-mc/ProNouns/tree/the-big-cleanup - pronouns ported to the new framework


How to backup and restore DTR

DTR 2.2 has backup and restore docs here, but a bunch of users are having problems when following these instructions.

[ ] Explain when backup and restores should be used. Check #1720
[ ] Review that the current docs work on DTR 2.2
[ ] Make it clear what the DTR backup saves and what doesn't. Check #1704
[ ] How do I test if a backup succeeded
[ ] How do I backup/restore images when using an external storage
[ ] When restoring you must your the same docker/dtr tag that created the backup. Check #1685

dibs


Merge Review UI Unresponsive

Checked for duplicates
Yes - I've already checked
Is this a regression?
Yes - This worked in a previous version
Version
1.7.0
Describe the bug
Merge Review UI stalls out... I've created a branch and submitted a merge request. Back in the parent plan, I open the merge review and I can't click on anything: can't preview the modified activities, deny/approve changes, or cancel the review from this screen. When I click the deny/approve changes button, I get a notification that the action has failed.
Reproduction
Create a branch, edit activities, and submit a merge request. In parent plan, select to review merge request.
Logs
No response
System Info
MacOS Ventura 13.4
Chrome 114.0.5735.90
http://eurcdevaerie.jpl.nasa.gov/plans/89

Severity
Minor

I'm still getting the same unresponsive behavior in merge review. Can't preview the modified activities, deny/approve changes, or cancel the review from this screen. When I click the deny/approve changes button, I get a notification that the action has failed.

I'm now getting similarly stuck on 1.5.0 http://eurcdevaerie5.jpl.nasa.gov/

@nat-sims You are still seeing this on 1.8.0? Same repro steps?

A fix related to this should have been shipped with the Aerie 1.8 release though I'm not sure it'll resolve all the issues you're seeing here. Should at least fix the issue with not being able to click on any of the changes and see the diffs. I can't seem to reproduce the error messages you're seeing here http://eurcdevaerie.jpl.nasa.gov/, would you be able to send me a screenshot of the Chrome developer console logs when you see it?

Will test again once http://eurcdevaerie.jpl.nasa.gov/ gets upgraded to 1.8.0


Plans to update engine?

Are there any plans to move this to the chromium engine instead of the trident? Or webview2?
https://docs.microsoft.com/en-us/microsoft-edge/webview2/

This library will not move away from Trident, but the WebView2 version, called Positron.ahk, is under development. Ping me on Discord if you would a beta copy to play with

Positron.ahk
👀 I'm definitely interested in said beta copy

Same.  I'll reach out in discord


captcha is hard

In the past, simple captcha would come. But now I have at least 10 captchas to solve. and the hardest part...
I connected via mobile data, still the same, I used VPN, still the same. I wonder why?
chrome latest version.

try deleting data folder and restart

Thank you for your answer. I did what you said. but nothing changed. I tried from different ips. I deleted the program and reinstalled it. While captchas were very easy before, now they are always difficult. I wonder what am I doing wrong? I have 2 computers. The result is the same in both.

Thank you for your answer. I did what you said. but nothing changed. I tried from different ips. I deleted the program and reinstalled it. While captchas were very easy before, now they are always difficult. I wonder what am I doing wrong? I have 2 computers. The result is the same in both.


Add a new mixin for seeding

Example usage from #7558 ;
# Condition
seed_model

# MiqPolicy
seed_model(
  with_many(:conditions)
)

# MiqPolicyContent
def self.seed
  seed_model(
    with_one(:miq_policy),
    with_one(:miq_action, :name),
    with_one(:miq_event_definition),
  )
end

# MiqPolicySet
seed_model(
  with_many(:miq_policies, :guid, -> (policy_set, policy) { policy_set.add_member(policy) })
)

ping @simon3z @jrafanie @gtanzillo @gmcculloug i

@miq-bot remove_label wip
@miq-bot add_label control

Now seeding as MiqPolicySets (AKA profiles) as well. This will allow to attach defaults policy data to entities completely out of the box

@moolitayer at first glance I have the feeling that there's a lot of copy and paste. Any way of refactoring a little bit before proceeding?

@moolitayer I like the general idea but I agree with @simon3z, it feels rather complicated and similar code.  Perhaps, start by creating methods to YAML.load_file + build file name and extract some of the similar behavior?  What do you think?

Adapt logic to extract seed logic.

remove relationship file in favor on inlining
added code at evm_databases to time the seeding and divide to classes
add seeding mixin so that seeding logic can be shared
add specs for all seeds

@jrafanie @Fryguy @gtanzillo This patch needs your review

This is really nice. Creating a SeedingMixin is something that should have been done a long time ago. I know that you have tests for this in #7558 but they seem to only test the creation of the seeded data (unless I missed it) but, it would be good to also test the update path as well.

Fixed according to comments from @jrafanie.
Added tests but waiting before I test every flow (api needs to improve)

At #7567
@jrafanie wrote:

Note, I'm wondering if the guid was because we need to seed separate classes and needed a way to link them up together... maybe if we seed them together in a single class, we can avoid needing this guid entirely?

@gtanzillo wrote:

@moolitayer Regarding the guids - we added them to allow for creating policies (and other things that may not be policies) in one database, exporting them and importing them in another DB. And, updating them if they already exist in the destination DB. This was necessary because you can't rely on the ids being the same for the same policy in both DBs. So, the guid ensures that we can match them.

This was a common pattern that was used in the field so that policies, some that were very complex, can be built and shared.

Indeed in other models seeding multiple classes I've looked at, the yml is always nested. I believe that does not fit this kind of data: conditions can belong to more then one policy, same goes for policies and policySets. In addition policies don't have to actually belong to a policySet (same goes for conditions and policies) so I'm not sure nesting is any good.
I understand currently seeding guid isn't something that we do. Question is, is there a reason not to?
If I have a guid for each associated entity we gain:

each model is  seeded separately at model.seed (with order defined at evm_database.rb)
no need to design different yml structure and custom parsing code

@moolitayer

I understand currently seeding guids isn't something that we do. Question is, is there a reason not to?
If I have a guid for each associated entity we gain:

each model is seeded separately at model.seed (with order defined at evm_database.rb)
no need to design different yml structure and custom parsing code for models
I agree with . I'd seed the guid column with a guid and the name column with an actual name instead of the guid.

@jrafanie There are some Issues I see with my current implementation, maybe you know how to improve it;

Maybe I need to remove the :search_filter_for_hash that knows how to get the filter from yml hash (e.g for guid: -> (hash) { {:guid => hash[:guid]} }) and add a guid to policyContent?
Same model, It might not make sense to do this merge for future users. I use it to add found event and action to policyContent. Maybe I should replace it with has_one ?
Maybe I need to not be strict on the guid and allow listing other fields in associations?

This is the Best API I can think of (usage example):
Condition:
seed_model(self)

MiqPolicy:
seed_model(
  self,
  :has_many  => [
    {
      :name     => :conditions,
      :column   => :guid,
      :add_proc => -> (policy, condition) { policy.conditions << condition }
    }
  ]
)

MiqPolicyContent:
seed_model(
  self,
  :has_one => [
    {
      :name     => :miq_policy
      :column   => :guid,
    },
    {
      :name     => :miq_event_definition,
      :column   => :name,
    },
    {
      :name     => :miq_action,
      :column   => :name,
    },
  ]
)

MiqPolicySet:
seed_model(
  self,
  :has_many => [
    {
      :name      => :miq_policies,
      :column    => :guid,
      :add_proc  => -> (policy_set, policy) { policy_set.add_member(policy) }
    }
  ]
)

That would require adding guid to policyContent to drop the :search_filter_for_hash option.
@jrafanie @gtanzillo sorry for all the text, Waiting for input before I proceed

@moolitayer this part feels like a "struct" or something:
  :has_many => [
    {
      :name      => :miq_policies,
      :column    => :guid,
      :add_proc  => -> (policy_set, policy) { policy_set.add_member(policy) }
    }

Maybe something like this?
AssociationSeed = Struct.new(:association, :type, :find_by, :add_proc)
association_seed = AssociationSeed.new(:miq_policies, :has_many, :guid, -> (policy_set, policy) { policy_set.add_member(policy)})

# MiqPolicySet:
seed_model(self, [association_seed])

@moolitayer who should be reviewing this? @jrafanie ? Anyone else?

@moolitayer who should be reviewing this? @jrafanie ? Anyone else?

@jrafanie and @gtanzillo started reviewing #7413 but they are unavailable this week so definitely in need of more reviewers, maybe @blomquisg has someone in mind.
@bdunne @durandom @Fryguy could you take a look at this? Thanks

@moolitayer Could you add a detailed description summarizing the goal of all of this (especially for those just now looking at it?)
This seems to be adding an entire DSL for seeding on top of what we already have. Have we considered SeedFu or some other tested thing and utilizing guids which would make primordial seeding fairly trivial and simple (hopefully)?

@moolitayer Could you add a detailed description summarizing the goal of all of this (especially for those just now looking at it?)

added goal section to first pr comment, see also comments [1] (https://github.com/ManageIQ/manageiq/pull/7413#issuecomment-202883148),2

This seems to be adding an entire DSL for seeding on top of what we already have. Have we considered SeedFu or some other tested thing and utilizing guids which would make primordial seeding fairly trivial and simple (hopefully)?

If you have any other suggestions for seeding I can use for #7558 instead of this it might be a good idea.
#7558 needs to get in and this change was requested (comments 1 and 2 mentioned above) and had [initial] agreement (I admit I do not know SeedFu or any other seeding libs)
Note this has many tests I was requested for here and 4 more here
ps Utilizing guid is exactly what I do as discussed here and agreed upon here

I would very much suggest forgoing adding a custom DSL to this and instead utilizing simple find_or_creates or SeedFu's constraint feature with GUIDs.
I'm honestly in favor of a simple PORO (with or without seedfu) and removing the ::seed methods on all the classes, centralizing everything. The way we seed currently is scattered all over the app, order-dependent, confusing, etc, and to be honest I feel like this adds a bunch of complexity on top of something already needlessly complex 😕 I/we just haven't gotten around to overhauling it.
However, all that I just mentioned is well beyond the scope of what you immediately might need. If you need to get something seeded immediately I would suggest just patching in what you need with the current setup. It should be possible, you just can't put the seed logic for every related many relationship completely separate in each model. For example, I need to seed the many entitlements per miq_group in miq_group (nothing in entitlements).
All my opinion of course, I'm curious what other people might think of this direction. @jrafanie @Fryguy ?

I'm honestly in favor of a simple PORO ... centralizing everything.

👍  here. In my experience utilizing the least amount of magic and plain ActiveRecord objects is the most descriptive way of doing seeding. It also helps discovering design flaws in the models sometimes.
@moolitayer I know this does not help you with getting #7558 in though :( Do you have an older commit where you seed just with the def self.seed method?

I agree with @chrisarcand.  We definitely need to clean up seeding, so the general direction is good, but the approach here seems unnecessarily complex.  I'm not sure it really "fixes" the problem, it just obscures it behind a DSL.
I don't like the addition of the guid.  Creating a proper guid will probably not be done by a developer when they add a new entry.  Instead they are likely to just take the previous one and change the last digit, which is probably not a good thing.  Instead, I think it would be nicer if the caller could pass what are the unique columns, for example seed_model(:name) if the name is what's considered unique.

If you need to get something seeded immediately I would suggest just patching in what you need with the current setup.

Mixing the immediate need to seed certain tables in #7558 with a complete refactoring here, seems like a bad approach, particularly since #7558 will get held up until this PR is done.  #7558 should just get the data in the current form to solve that need, and this PR should probably change to fix all (or at least many) of the seeding classes at once.
I need the opinions of @chessbyte and @jrafanie and others, but right now I am not a fan of the approach here.

Mixing the immediate need to seed certain tables in #7558 with a complete refactoring here, seems like a bad approach, particularly since #7558 will get held up until this PR is done.

@Fryguy I agree since this PR is becoming controversial.
@moolitayer do you think you can bake up something in #7558 to get it merged and postpone this refactoring?

Marking this as wip for now. Seeding of policies moved to #7966
@miq-bot add_label wip

This does not seem like the way to go, and no resources/need for it right now


Feature request: Add flag to show stock number

I recall in the first version that the stock number was displayed. This would cause errors and delays, but I really think it was a cool feature.
Can this be added back in as a command line option (does not loop trying to order item, just checks stock number once)?

I'll second that idea. I was literally just thinking about that, was potentially going to run the old version adhoc just to get a sense of what it's still looking like.

Yeah, just run v.06
Apparently nVidia has not added any stock to 3080 since after launch.

@ianmarmour  This isn’t possible anymore through the API. However, someone posted on reddit a URI that shows a server response with SKU’s meta flags - including an “in-stock” true/false flag. Maybe something to look into?

@ianmarmour This isn’t possible anymore through the API. However, someone posted on reddit a URI that shows a server response with SKU’s meta flags - including an “in-stock” true/false flag. Maybe something to look into?

Can you link to that?

"inventoryStatus": { "uri": "https://dispatch-nvidia.digitalriver.com/v1/shoppers/me/products/5438481700/inventory-status", "availableQuantityIsEstimated": "false", "productIsInStock": "true", "productIsAllowsBackorders": "false", "productIsTracked": "true", "requestedQuantityAvailable": "true", "status": "PRODUCT_INVENTORY_OUT_OF_STOCK", "statusIsEstimated": "false" }
Is productIsInStock flag available to see still?

Yes but product_inventory_out_of_stock is what we use because it's what nvidia uses to turn on and off sales.

Yes but product_inventory_out_of_stock is what we use because it's what nvidia uses to turn on and off sales.

Yes absolutely, use product_inventory_out_of_stock to see if ATC is on or off. But also displaying productIsInStock is very useful as they might turn on product_inventory_out_of_stock soon.

seems there is a security rule blocking me to test that uri ( https://dispatch-nvidia.digitalriver.com/v1/shoppers/me/products/5438481700/inventory-status )... is it just me or a global rule?

global


Bug: Email delayed Subscribe Widget and Edit Comments

The emails for Subscribe Widget and Edit Comments came in half hour ish later after I clicked buttons

Compare the timestamp of the records in DB to see if the delay is actually in our system or in gc notify

@TayaLee8 can you test this to see if it's still happening

I did not experience an email delay when subscribing, or when my comment was rejected and needed to be revised @carlygreenehill @saravanpa-aot


BCTS FOMs rule adjustment for Validity Period

Describe the Bug
Julius report this required change:
"The highlighted portion of the snip below indicates that the validity period for this FOM ends 3 years after the Commenting Open date (as it should for non-BCTS licensees). "
However for Timber Sales: "for BCTS FOMs the date stated as the validity period is 3 years from commenting close date, rather than from commenting open".
From: Huhs, Julius FOR:EX <[Julius.Huhs@gov.bc.ca](mailto:Julius.Huhs@gov.bc.ca)>
Sent: Wednesday, April 10, 2024 12:05 PM
To: Liberchuk, Olga WLRS:EX <[Olga.Liberchuk@gov.bc.ca](mailto:Olga.Liberchuk@gov.bc.ca)>; Tollestrup, Pete WLRS:EX <[Pete.Tollestrup@gov.bc.ca](mailto:Pete.Tollestrup@gov.bc.ca)>
Subject: FW: FOM Validity Period

Hello,

The issue pointed out below is actually quite important legally. The validity period for BCTS does not start until the commenting close date, unlike for all other FSP holders, for whom validity starts at commenting open. Therefore, the dates currently shown on FOMs are somewhat incorrect. We need to make it so, that for BCTS FOMs the date stated as the validity period is 3 years from commenting close date, rather than from commenting open. I hope that since we already have the TSM name field that is different for BCTS versus other FOMs, there might be a way to easily identify a FOM as a BCTS FOM and apply the different validity period rule. Like always, I’ll of course will leave the solution up to your team, but we should fix this rather sooner than later.

Thank you very much for all the help. It never fails to amaze how many little details there are to consider.

Have a good day.
[screenshot below]

Julius Huhs, RPF, RFT
Sustainable Forest Management Team Lead
Ministry of Forests
Forest Science, Planning & Practices Branch / Office of the Chief Forester
Phone:<PHONE_NUMBER>
Email: [Julius.Huhs<EMAIL_ADDRESS>
Expected Behaviour
BCTS FOM: "The validity period for this FOM ends May 10, 2027"
Actual Behaviour
BCTS FOM: "The validity period for this FOM ends April 4, 2027"
Screenshots

Additional context
It is correct currently for Non-BCTS FOM for validity period.

Other than FOM Details(Public), there is also Public Notices (Public) section to be changed:

Promoted to production.


rearrange the filepath of examples

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
The filepath of examples need to be rearranged.
Which issue(s) this PR fixes:

Fixes #

It's good to see a rearrangement on the file path. This work would make the example much more user-friendly.
As discussed in the routine meeting:

A readme might be needed to set up and clarify the rule. The naming of directories would be in the manner of Dataset/ Scheme/ Task.
Works from the robotic dataset by Shijing, Linjie, and Tianyu,  would be merged into a unified directory.
The simulation work has nothing to do with building simulation, the title of which should be re-named to "how to run simulation" and re-located to ianvs/docs/guides.

It's good to see a rearrangement on the file path. This work would make the example much more user-friendly.
As discussed in the routine meeting:

A readme might be needed to set up and clarify the rule. The naming of directories would be in the manner of Dataset/ Scheme/ Task.
Works from the robotic dataset by Shijing, Linjie, and Tianyu,  would be merged into a unified directory.
The simulation work has nothing to do with building simulation, the title of which should be re-named to "how to run simulation" and re-located to ianvs/docs/guides.

The examples are rearranged according to the discussion in the routine meeting.


New Feature for TallMastersCombo: Switch between Focused Windows across All Sub-layouts

Description
Each non-combinator sub-layout can contain one focused window, and this window will keep focused after switching to other sub-layouts. Two messages were added for switching between the focused windows across all sub-layouts.
Checklist

[x] I've read CONTRIBUTING.md

[x] I tested my changes with xmonad-testing

[x] I updated the CHANGES.md file

Hi @tidues , does the Choose combinator continue to work?  I am not clear on whether this will break existing user configs or not.  Maybe Choose still works but if you want the behavior regarding focused windows you have to use the new (|||) operator?

I'd also wonder how this interacts with the custom (|||) for the JumpToLayout messaage. Granting that it might be time to merge that one to core.

Hi, @byorgey
Currently, using this has to hide the (|||) operator from the XMonad module. Otherwise, there will be an ambiguity error. Other than this, everything else of the Choose combinator still works.
Basically, my (|||) operator is a wrapper for the original one with one extra feature that can monitor the current states of the Choose combinator. They serve for almost the same functionality, so I decided to use the same operator.
Actually, if the XMonad.Layout and expose the Choose(...) and LR(...) constructors, then I don't have to use a wrapper to monitor its states.

Hi @geekosaur,
I am not familiar with the JumpToLayout message. But every message that is not handled by the Choose combinator will pass down to the currently active layout. I don't if this will pose a problem?

It's handled by a replacement (|||) operator, is the point. So now we have two different and apparently incompatible (|||) alternatives?

I see. The JumpToLayout message and the corresponding (|||) operator are provided by the LayoutCombinator module. I know your concerns, but I don't think it is a problem since both are served as flexible layout combinators. Users will usually go with one or the other.

Thanks!


New plugin: SnowSQL

Platform or tool
SnowSQL
Relevant links

General: https://docs.snowflake.com/en/user-guide/snowsql.html
Getting Started with SnowSQL: https://quickstarts.snowflake.com/guide/getting_started_with_snowsql/index.html
Installing SnowSQL: https://docs.snowflake.com/en/user-guide/snowsql-install-config.html
Connecting through SnowSQL: https://docs.snowflake.com/en/user-guide/snowsql-start.html

Goal or desired behavior
Authenticate the SnowSQL (Snowflake) CLI using credentials so Snowflake users can quickly authenticate into and control all aspects of their Snowflake Data Cloud.
Contribution

[X] I'd be up for building the plugin myself.

Additional information
After installing the SnowSQL CLI (I installed on macOS using Homebrew), you sign-in to your Snowflake account using snowsql (you may include -a and -u flags for account and username). You then get prompted for your account, username, and password in the terminal.
It seems that there are three ways to specify credentials to authenticate:

Entered via interactive prompt in SnowSQL (applies to passwords only).
Defined in the SnowSQL configuration file using the password option. For details, see Configuring Default Connection Settings.
Specified using the SNOWSQL_PWD environment variables. For details, see Using Environment Variables.

For development of this plugin, it seems that taking advantage of option 2 and the file provisioner may be the best option. The credentials can be stored in ~/.snowsql/config then used to authenticate.

Update: I ran into issues when trying to implement the plugin using file provisioning, and I realized that there are actually environment variables that can be used for the account and username too (SNOWSQL_ACCOUNT and SNOWSQL_USER, not just the password. I decided that using environment variables for the provisioner would be more straight-forward.


Handling registration requests with broken/incomplete JSON payload

To continue from discussion started here:
https://github.com/joohoi/acme-dns/pull/20#discussion_r150984732
ping @koesie10
Which approach would be the intuitive and appropriate if broken or incomplete JSON object (or whatever payload) is sent to registration endpoint:

acme-dns should return error, like 400 Bad Request with descriptive error message.
Registration should complete with default values, and return these to the user.

Previously I felt that the second one would be the correct approach, but now I'm starting to lean towards the first one.

I would expect an error to be returned, as otherwise you would be opening yourself to security risks, such as allowing requests from all IPs. Of course, this is usually just an extra security measure, but in case your username and passwords are compromised, it is still an effective security measure that would be disabled without warning when you send an invalid JSON request.

Yeah, that's the reasoning that makes me lean towards the second option too. I'll implement that.

Great, thanks for the project! I'm using it in combination with my own Certbot hook to automate issuing and renewing domains without having to change anything on the server, like is required for the http-01 and tls-sni-01.

Nice to hear that it's useful for you! I'm actually also an active Certbot developer, so if there's anything I can do to improve either project, please let me know! If it's Certbot related, using issues on Certbot repo, obiviously though :)


Some more naming cleanup

Do a few more renames to keep things easier for new users - even I had to lookup these names despite working with Varnish for a while to understand what they do.

I'm not super excited about that one as it would make it harder for C devs coming in to find their marks, but I'm willing to try it
I assume the vmod PRs are coming? ;-)


Featured article#16

What does this change?
This changes the webinar page. I added a featured article on the page so users can see the most important or most recent webinar. The article includes the assigned thumbnail, title, description category and an additional link to the webinar. It is responsive and accessible. (See images below)
Fixes #16

How Has This Been Tested?

[x] User test
[X] Accessibility test
[X] Performance test
[X] Responsive Design test
[X] Device test
[X] Browser test

Images

Mobile:

Tablet

Desktop

How to review
Check code

The code looks good. Nice card love that you used container queries.


adding code_of conduct.md

A Code of Conduct ensures inclusivity, prevents harassment, clarifies behavioral expectations, encourages professionalism, and fosters a safe, welcoming community for contributors.
plz assisn me this issue under gsoc-extended , level 1 , hacktoberfest-accepted tag

Hello @jyothimanoj12
This issue has already been created #26
Thank you


This closes #1484, fix formula calc result issue

PR Details

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (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 change)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #1485 (68f2e03) into master (65a53b3) will not change coverage.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master    #1485   +/-   ##
=======================================
  Coverage   98.69%   98.69%
=======================================
  Files          31       31
  Lines       24308    24308
=======================================
  Hits        23990    23990
  Misses        210      210
  Partials      108      108

Flag
Coverage Δ

unittests
98.69% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

calc.go
99.03% <100.00%> (ø)

datavalidation.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


add record fail when field with l10n:"sync"

in slug.tmpl
change
{{if not (has_update_permission .Meta)}}disabled{{end}}
to
{{if not (has_update_permission .Meta)}}readonly{{end}}
may be more reasonable.

Hi @oonxt
I think it should be fine? as we will save with the slug from default locale.


tcp client: make close type configurable

make async tcp client close type configurable.


Switch to Python3 style logging

We should look to change %s logging strings to format-string style ones.
Might need to make a custom Logger that forces the formatter style to prevent it conflicting with other loggers that are using %s style.

This currently does not provide any benefit. May reconsider in the future.


Update Rolls public NPM name

Can we check what npm names are available?
@roll
@roll-js
@rolljs

noting that roll-network is the target!


Es6

So this PR is probably larger than PRs normally should be, but refactoring wavesurfer to work with a module system turned out to required some extra steps.
Since I realize I've been very opinionated please consider this as a base for discussion.
TL;DR

bundling with webpack/grunt was removed

es 6 transpiling with babel
linting with eslint

plugin api

standardised plugin definition structure
new api for dynamic adding, initialising and destroying plugins
config api for automatic plugin initialisation on wavesurfer initialisation

There are breaking changes :pensive:

Due to a limitation with webpack the plugins on the global object are now lowercase (window.WaveSurfer.elan, not window.WaveSurfer.ELAN)
All module systems now use the module name WaveSurfer (AMD as well)
Using the old API to add plugins (e.g. timeline.init()) doesn't work anymore mostly works, but in some cases may not anymore
I haven't managed to get the coverage reporting to work with webpack yet

Updated build system
Previously all parts of the library (and its plugins) relied on the global WaveSurfer object to attach themselves to. Working this way with a module bundler means exposing this object which defeats the purpose of dependency injection and module bundling. The library needed to be refactored to use dependency injection so that there was no need for a global object.
Achieving this without some kind of build tool would have be a tedious task. So I opted for webpack, which provides convenient features to build multi-part libraries. Also it has an inbuilt dev server with livereloading etc. (Currently webpack 2, which is in beta, but apparently only until the docs are finished)
jscs and jshint, which were currently being used for linting have merged as eslint, so it only made sense to change to that linter (as a webpack loader).
I also updated karma to work with webpack.
So now there was really no reason to continue using grunt at all, so I removed it. Instead all the scripts are aliased from package.json.
Wavesurfer can be accessed like this:

script-tags approach/global object:

<script src="my/path/dist/wavesurfer.js"></script>
<script src="my/path/dist/plugin/wavesurfer.regions.js"></script>
<script>
    alert(window.WaveSurfer);
    alert(window.WaveSurfer.regions);
</script>

commonjs/requirejs

const WaveSurfer = require('wavesurfer.js');
const Regions = require('wavesurfer.js/dist/plugin/wavesurfer.regions.js');

ES6 imports

import WaveSurfer from 'wavesurfer.js';
import Regions from 'wavesurfer.js/dist/plugin/wavesurfer.regions.js';

Development workflow changes
The new dev tasks are:

npm start – Transpile the plugins and start the dev server at localhost:8080 (reloading only works with the core lib currently)
npm run build
npm run test

Plugin system
The plugins were written in a way which basically just added to the WaveSurfer global variable. To be able to implement this in a truly modular fashion there had to be some process of registering plugins, allowing them to augment the library without having to have that globally available. To achieve this I defined a standard plugin definition format. (See an example further below)
Adding, initialising and destroying plugins
There are three ways to add a plugin:

Adding and initialising plugin immediately (this will do in 99% of the cases):

var wavesurfer = WaveSurfer.create({
    container: '#waveform',
    waveColor: 'violet',
    plugins: [
        WaveSurfer.timeline({
            container: '#wave-timeline'
        })
    ]
});

Adding a plugin immediately and initialising it dynamically:

var wavesurfer = WaveSurfer.create({
    container: '#waveform',
    plugins: [
        WaveSurfer.timeline({
            container: '#wave-timeline',
            deferInit: true // stop the plugin from initialising immediately
        })
    ]
});
wavesurfer.initPlugin('timeline');

Adding and initialising a plugin dynamically:

var wavesurfer = WaveSurfer.create({
    container: '#waveform',
    waveColor: 'violet'
});
// adding and initialising a plugin after initialisation
wavesurfer.addPlugin(WaveSurfer.timeline({
    container: '#wave-timeline',
})).initPlugin('timeline');

Also a plugin can be destroyed like this:
wavesurfer.destroyPlugin('timeline');

Apart from that the public plugin API has not changed.
Standard plugin definition format
The definition object is being returned in a function, this allows us to pass in parameters during wavesurfer initialisation. A custom built plugin could just as well be an object passed in during initialisation.
A plugin definition now looks like this:
export default function(params) {
    return {
        // (required) plugin name, this is the namespace under which the instance of the
        // plugin instance is added to the wavesurfer instance
        name: 'myplugin',

        // defer the initialisation until later, can either be set
        // or depend on the params like this ...
        deferInit: params && params.deferInit ? params.deferInit : false,

        // static properties and methods are applied to the wavesurfer instance`
        static: {
            functionAttachedToWavesurfer() {
                // this refers to the wavesurfer instance
            },
        },

        // (required)
        // 1) plugin instance is instantiated and added to the wavesurfer
        //    instance under the "name" namespace
        //
        // 2) init is run:
        // - before the ready event: plugins now need to add a listener to the
        //   ready event if they need to do something after it has been triggered)
        // - after all the plugins have been instantiated, so init can for
        //   example check for the existence of this.regions
        instance: {
            // (required)
            init(wavesurfer) {
                // this refers to the instance of plugin at wavesurfer.myplugin
                this.params = params;
                this.wavesurfer = wavesurfer;

                this.methodOfThePlugin();
            },
            // (required)
            destroy() {
                // destroy this plugin
            },
            methodOfThePlugin() {
                console.log('Plugin haz been initialized!')
            }
        }
    };
}

Next steps
Depending on how this is received there would/could be a few next-steps

Updating documentation
Adding tests for the plugin system API methods
Adding generic tests for the plugins (if they follow the format)
Refactoring the code base to use es6 syntax (adding arrow functions, spread operator etc.)
We could consider refactoring multicanvas drawer into a plugin (any maybe even the mediaelement backend) since they are not part of the immediate core of the library to reduce file size.

replaced description with old one.

Do we want to fix the build before merging this PR?

I'll try and fix it beforehand.

ha!

@katspaugh why would we not fix the build before merging this PR..?

Can we merge into next-branch?

Merged!


How to add Debian packages to ddev web image: libapache2-mod-perl2

I need mod_perl in apache, but how?
Tried wb-build/Dockerfile
ARG BASE_IMAGE
FROM $BASE_IMAGE
RUN apt-get update
RUN apt-get install -y libdbd-sqlite3-perl perlmagick libapache2-mod-perl2

But on ddev start i run  into errors.
Without "libapache2-mod-perl2" it started.
Anyone can help with that?

The easiest way to add additional packages is via your config.yaml, webimage_extra_packages: [libdbd-sqlite3-perl, perlmagick, libapache2-mod-perl2]
That works fine with current stable ddev.
Please see https://ddev.readthedocs.io/en/stable/users/extend/customizing-images/#adding-extra-debian-packages-with-webimage_extra_packages-and-dbimage_extra_packages

Thanks rfay, but it will not work.
Have updated ddev version v1.16.6 (WSL 2)
I removed my Dockerfile web-build/Dockerfile and add webimage_extra_packages to config.yaml.
ddev start failed
Failed to start ihkffm: web container failed: log=phpstatus: FAILED /var/www/html: OK mailhog: FAILED , err=container /ddev-ihkffm-web unhealthy: phpstatus: FAILED /var/www/html: OK mailhog: FAILED

My config.yaml:
name: ihkffm
type: php
docroot: imperia/htdocs/
php_version: "7.3"
webserver_type: apache-fpm
router_http_port: "80"
router_https_port: "443"
xdebug_enabled: false
additional_hostnames: []
additional_fqdns: []
provider: default
use_dns_when_possible: true
webimage_extra_packages: [libdbd-sqlite3-perl, perlmagick, libapache2-mod-perl2]

The full log:
+ set -o errexit nounset pipefail
+ rm -f /tmp/healthy
+ DDEV_PHP_VERSION=7.3
+ DDEV_WEBSERVER_TYPE=apache-fpm
+ '[' -n 7.3 ']'
+ update-alternatives --set php /usr/bin/php7.3
+ ln -sf /usr/sbin/php-fpm7.3 /usr/sbin/php-fpm
+ export PHP_INI=/etc/php/7.3/fpm/php.ini
+ PHP_INI=/etc/php/7.3/fpm/php.ini
+ '[' '!' -z ']'
+ '[' -d /mnt/ddev_config/php ']'
+ '[' php = backdrop ']'
++ id -un
++ id -gn
+ printf '\nexport APACHE_RUN_USER=micha\nexport APACHE_RUN_GROUP=micha\n'
+ a2enmod access_compat alias auth_basic authn_core authn_file authz_core authz_host authz_user autoindex deflate dir env filter mime mpm_prefork negotiation reqtimeout rewrite setenvif status
Considering dependency authn_core for access_compat:
Module authn_core already enabled
Module access_compat already enabled
Module alias already enabled
Considering dependency authn_core for auth_basic:
Module authn_core already enabled
Module auth_basic already enabled
Module authn_core already enabled
Module authn_file already enabled
Module authz_core already enabled
Considering dependency authz_core for authz_host:
Module authz_core already enabled
Module authz_host already enabled
Considering dependency authz_core for authz_user:
Module authz_core already enabled
Module authz_user already enabled
Module autoindex already enabled
Considering dependency filter for deflate:
Module filter already enabled
Module deflate already enabled
Module dir already enabled
Module env already enabled
Module filter already enabled
Module mime already enabled
Considering conflict mpm_event for mpm_prefork:
Considering conflict mpm_worker for mpm_prefork:
Enabling module mpm_prefork.
Module negotiation already enabled
Module reqtimeout already enabled
Enabling module rewrite.
Module setenvif already enabled
Module status already enabled
To activate the new configuration, you need to run:
  service apache2 restart
+ a2enconf charset localized-error-pages other-vhosts-access-log security serve-cgi-bin
Conf charset already enabled
Conf localized-error-pages already enabled
Conf other-vhosts-access-log already enabled
Conf security already enabled
Conf serve-cgi-bin already enabled
+ '[' apache-fpm = apache-fpm ']'
+ a2enmod proxy_fcgi
Considering dependency proxy for proxy_fcgi:
Enabling module proxy.
Enabling module proxy_fcgi.
To activate the new configuration, you need to run:
  service apache2 restart
+ a2enconf php7.3-fpm
Enabling conf php7.3-fpm.
To activate the new configuration, you need to run:
  service apache2 reload
+ a2dissite 000-default
Site 000-default already disabled
+ '[' false = true ']'
+ disable_xdebug
Disabled xdebug
+ ls /var/www/html
+ sudo mkdir -p /mnt/ddev-global-cache/terminus/cache /mnt/ddev-global-cache/ddev-live
+ sudo mkdir -p /mnt/ddev-global-cache/bashhistory/ihkffm-web
++ id -u
++ id -g
+ sudo chown -R 1000:1000 /mnt/ddev-global-cache/ /home/micha/.ssh /home/micha/.drush /home/micha/.gitconfig /home/micha/.my.cnf
+ '[' -d /mnt/ddev_config/.homeadditions ']'
+ cp -r /mnt/ddev_config/.homeadditions/. /home/micha/
+ '[' -d /mnt/ddev_config/homeadditions ']'
+ cp -r /mnt/ddev_config/homeadditions/. /home/micha/
+ sudo mkdir -p /mnt/ddev-global-cache/mkcert
+ sudo chmod -R ugo+rw /mnt/ddev-global-cache/mkcert
+ mkcert -install
Using the local CA at "/mnt/ddev-global-cache/mkcert" ✨
The local CA is now installed in the system trust store! ⚡

+ sudo CAROOT=/mnt/ddev-global-cache/mkcert mkcert -cert-file /etc/ssl/certs/master.crt -key-file /etc/ssl/certs/master.key ihkffm.ddev.site localhost <IP_ADDRESS> <IP_ADDRESS> web ddev-ihkffm-web ddev-ihkffm-web.ddev_default
Using the local CA at "/mnt/ddev-global-cache/mkcert" ✨

Created a new certificate valid for the following names 📜
 - "ihkffm.ddev.site"
 - "localhost"
 - "<IP_ADDRESS>"
 - "<IP_ADDRESS>"
 - "web"
 - "ddev-ihkffm-web"
 - "ddev-ihkffm-web.ddev_default"

The certificate is at "/etc/ssl/certs/master.crt" and the key at "/etc/ssl/certs/master.key" ✅

+ sudo chown 1000 /etc/ssl/certs/master.crt /etc/ssl/certs/master.key
+ echo 'Server started'
+ exec /usr/bin/supervisord -n -c /etc/supervisor/supervisord-apache-fpm.conf
Server started
2021-02-15 08:31:47,644 INFO Included extra file "/etc/supervisor/apache.conf" during parsing
2021-02-15 08:31:47,644 INFO Included extra file "/etc/supervisor/php-fpm.conf" during parsing
2021-02-15 08:31:47,644 INFO Included extra file "/etc/supervisor/conf.d/mailhog.conf" during parsing
2021-02-15 08:31:47,644 INFO Included extra file "/etc/supervisor/conf.d/supervisor.conf" during parsing
2021-02-15 08:31:47,654 INFO supervisord started with pid 1
2021-02-15 08:31:48,656 INFO spawned: 'child_exit_monitor' with pid 1732
2021-02-15 08:31:48,657 INFO spawned: 'php-fpm' with pid 1733
2021-02-15 08:31:48,658 INFO spawned: 'apache2' with pid 1734
2021-02-15 08:31:48,659 INFO spawned: 'mailhog' with pid 1735
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:31:49.066477 2021] [core:emerg] [pid 1737] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1737)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:31:49,079 INFO exited: apache2 (exit status 1; not expected)
[15-Feb-2021 08:31:49] NOTICE: fpm is running, pid 1733
[15-Feb-2021 08:31:49] NOTICE: ready to handle connections
[15-Feb-2021 08:31:49] NOTICE: systemd monitor interval set to 10000ms
2021-02-15 08:31:50,580 INFO success: child_exit_monitor entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2021-02-15 08:31:50,581 INFO success: php-fpm entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2021-02-15 08:31:50,584 INFO spawned: 'apache2' with pid 1767
2021-02-15 08:31:50,584 INFO success: mailhog entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:31:50.605669 2021] [core:emerg] [pid 1769] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1769)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:31:50,617 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:31:52,621 INFO spawned: 'apache2' with pid 1790
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:31:52.642989 2021] [core:emerg] [pid 1792] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1792)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:31:52,654 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:31:55,659 INFO spawned: 'apache2' with pid 1825
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:31:55.683020 2021] [core:emerg] [pid 1827] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1827)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:31:55,694 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:31:59,701 INFO spawned: 'apache2' with pid 1859
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:31:59.727778 2021] [core:emerg] [pid 1861] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1861)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:31:59,739 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:32:04,750 INFO spawned: 'apache2' with pid 1915
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:32:04.776164 2021] [core:emerg] [pid 1917] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1917)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:32:04,787 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:32:10,796 INFO spawned: 'apache2' with pid 1982
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:32:10.817109 2021] [core:emerg] [pid 1984] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1984)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:32:10,828 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:32:17,839 INFO spawned: 'apache2' with pid 2048
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:32:17.861569 2021] [core:emerg] [pid 2050] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.2050)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:32:17,873 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:32:25,884 INFO spawned: 'apache2' with pid 2138
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:32:25.907355 2021] [core:emerg] [pid 2140] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.2140)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:32:25,918 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:32:34,928 INFO spawned: 'apache2' with pid 2229
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:32:34.961715 2021] [core:emerg] [pid 2231] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.2231)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:32:34,973 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:32:44,986 INFO spawned: 'apache2' with pid 2327
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Mon Feb 15 08:32:45.022151 2021] [core:emerg] [pid 2329] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.2329)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs
httpd not running, trying to start
Action '-k restart -D FOREGROUND' failed.
The Apache error log may have more information.
2021-02-15 08:32:45,033 INFO exited: apache2 (exit status 1; not expected)
2021-02-15 08:32:45,033 INFO gave up: apache2 entered FATAL state, too many start retries too quickly
2021-02-15 08:32:46,035 WARN child_exit_monitor: bad result line: 'This line kills supervisor: ver:3.0 server:supervisor serial:0 pool:child_exit_monitor poolserial:0 eventname:PROCESS_STATE_FATAL len:56'
2021-02-15 08:32:46,035 WARN child_exit_monitor: has entered the UNKNOWN state and will no longer receive events, this usually indicates the process violated the eventlistener protocol
2021-02-15 08:32:46,035 WARN received SIGQUIT indicating exit request
2021-02-15 08:32:46,035 INFO waiting for php-fpm, child_exit_monitor, mailhog to die
2021-02-15 08:32:46,036 INFO stopped: mailhog (terminated by SIGTERM)
[15-Feb-2021 08:32:47] NOTICE: Terminating ...
[15-Feb-2021 08:32:47] NOTICE: exiting, bye-bye!
2021-02-15 08:32:47,077 INFO stopped: php-fpm (exit status 0)
2021-02-15 08:32:47,078 INFO stopped: child_exit_monitor (terminated by SIGTERM)

The failure is this - apparently one of these packages uses /var/lock/apache2 in a different way
[Mon Feb 15 08:31:55.683020 2021] [core:emerg] [pid 1827] (13)Permission denied: AH00023: Couldn't create the mpm-accept mutex (file /var/lock/apache2/mpm-accept-0.1827)
(13)Permission denied: could not create accept mutex
AH00015: Unable to open logs

You'll have to switch back to the Dockerfile approach. I think this custom .ddev/web-build/Dockerfile will get you started, see docs
ARG BASE_IMAGE
FROM $BASE_IMAGE
RUN mkdir -p /var/lock/apache2 && chmod 777 /var/lock/apache2
RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y -o Dpkg::Options::="--force-confold" --no-install-recommends --no-install-suggests libdbd-sqlite3-perl perlmagick libapache2-mod-perl2

It seems that installing these packages creates /var/lock/apache2 (actually /run/lock/apache2) with root ownership during the installation phase.
On a plain vanilla apache setup, they don't get created until the webserver starts up, so they get the proper ownership.


Cherry pick window::shift to work for all array types to active_release

Automatic cherry-pick of 0f55b82

Originally appeared in https://github.com/apache/arrow-rs/pull/388: window::shift to work for all array types

Codecov Report

Merging #464 (ff264ff) into active_release (1f7f4bc) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@                Coverage Diff                 @@
##           active_release     #464      +/-   ##
==================================================
+ Coverage           82.69%   82.71%   +0.02%
==================================================
  Files                 163      163
  Lines               44775    44795      +20
==================================================
+ Hits                37025    37051      +26
+ Misses               7750     7744       -6

Impacted Files
Coverage Δ

arrow/src/compute/kernels/window.rs
100.00% <100.00%> (ø)

arrow/src/array/transform/mod.rs
86.15% <0.00%> (+0.27%)
:arrow_up:

arrow/src/array/array.rs
76.86% <0.00%> (+0.39%)
:arrow_up:

arrow/src/array/data.rs
72.37% <0.00%> (+0.90%)
:arrow_up:

Continue to review full report at Codecov.

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

Needs https://github.com/apache/arrow-rs/pull/475 then rebase to get a clean CI run

Integration failure https://github.com/apache/arrow-rs/pull/464/checks?check_run_id=2868777467 due to some sort of maven problem
Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.6.2:compile (default-compile) on project arrow-plasma: Resolution of annotationProcessorPath dependencies failed: Could not transfer artifact org.immutables:value:jar:2.8.2 from/to central (https://repo.maven.apache.org/maven2): /root/.m2/repository/org/immutables/value/2.8.2/value-2.8.2.jar.part (No such file or directory)
Error:    org.immutables:value:jar:2.8.2

Rerunning


selection from rofi menu doesn't paste into a field

So i was hoping it would automatically insert the text into current active window or text field. but i have to ctrl+v to paste it. Is this how it is supposed to be or mine isn't working correctly.
Please tell if i can get it to paste also.

You can check the Q&A in the Readme for an answer
Q. I want to paste selection directly after selecting them !

A. This is not possible as when you invoke rofi, you lose focus of your current window and there is no way to find it back (from greenclip point of view)

Nonetheless, you can emulate the feature with xdotool rofi -modi "clipboard:greenclip print" -show clipboard -run-command '{cmd}' ; sleep 0.5; xdotool type $(xclip -o -selection clipboard)

or look at this issue for a more complete solution https://github.com/erebe/greenclip/issues/27

okay got it working thanks


Clarification on initiation of communication.

Is communication always initiated from the client to the 'connection endpoints for Windows', or can there be scenarios where communication is initiated by the connection endpoints? I'm unable to find any documentation that clearly articulates expected behavior. Thanks in advance!

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

ID: 1ba43b5f-dd62-9634-fe8b-c10755456df8
Version Independent ID: 1ba43b5f-dd62-9634-fe8b-c10755456df8
Content: Connection endpoints for Windows 10 Enterprise, version 21H2 - Windows Privacy
Content Source: windows/privacy/manage-windows-21h2-endpoints.md
Product: windows-client
Technology: itpro-privacy
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

@mikemagarelli   We checked with the team responsible for the information in that article. Their reply was: "For these documented endpoints, they’re initiated from the client."  Hope that helps.

Thanks so much, that's exactly what I needed!


comparison with rogpeppe/go-internal ?

Hello @ancientlore,
I am an happy user of rogpeppe/go-internal/testscript, which you probably know as a previous version of the same extracted code from Go internal.
I tried once to use rsc.io/script but I have to say it was not evident at all (no examples, except the ones directly in go src) and I gave up.
I was wondering what is your opinion ? Thanks!

Hi Marco,
I first learned about rsc.io/script from this thread on golang-nuts. My use case was outside of testing - basically I was looking for a very simple, embeddable scripting system to use in other projects. The needs were pretty basic. I have not looked deeply at rogpeppe/go-internal/testscript yet.
I found the documentation a bit lacking and it took some time to figure out how to add additional commands and conditions. In particular I was trying to add conditional logic. You can add conditions that look for one thing, like "an environment variable is set". But it doesn't seem flexible enough to test that an environment variable has a particular value before executing a command. I suppose you could implement a command that has options for that. But as I said, I had a different use case in mind.
If you are happy with rogpeppe/go-internal/testscript, there is probably no real reason to switch.
Best,
Mike

Thanks for the answer Michael!


Critical Exception when SoC attempts to create directory in Linux

SoC attempts to create MUICHAOS\DATA\TEXT which fails with System.IO.IOException: Invalid argument and the target directory "TEXT" is not created. This occurs under Linux using the SoC distribution from the MBBSEmu site.

FileUtility.FindFile() will need to be enhanced/refactored to support paths that are N directories deep.
Currently it only supports searching 1 directory deep from the module base path on case sensitive file systems, which is causing the issue here. SoC is trying to access <base>\DATA\TEXT\ which is 2 deep.

In this test case MBBSEmu is running on a case insensitive file system (lowntfs-3g with mount options ignore_case, windows_names). Manually creating \DATA\TEXT\ does not have any affect on the issue.

Confirmed working in Build #082820-3. I can create a player and enter the game more testing will be needed but this can be closed!


Update smb.py

Copied Hash/Password check from find.py. Without this password has to be provided on the commandline or the account might get locked out, because the script will run and use an empty password.

Nice catch! Thanks for the PR. Merged.


Updating from 1.0.0-rc2 to 1.0.1 cause IDENTITY INSERT to fail

Since I updated the the EntityFramework Core  (in projects.json) from 1.0.0-rc2 to 1.0.1 I cannot insert any new Entity using the IDENTITY INSERT.
Steps to reproduce
The model is currently set the following:
public class Region
{
    [Key]
    // with or without "DatabaseGenerated" does not matter here
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public int ID { get; set; }

    // [....]
}

This is how I am trying to insert a new record
// get the guid for the current user and store it into the current model
if (model.ID == 0)
    // once model is attached, the ID changed to `-2147482647`
    ctx.Attach(model).State = EntityState.Added;
else
    ctx.Attach(model).State = EntityState.Modified;
// save changes
ctx.SaveChanges();

The issue
Exception message:
Microsoft.EntityFrameworkCore.DbUpdateException: An error occurred while updating the entries. See the inner exception for details. ---> System.Data.SqlClient.SqlException: Cannot insert explicit value for identity column in table 'Region' when IDENTITY_INSERT is set to OFF.

Further technical details
SQL Statement below:
I am actually missing the SET INDENTITY INSERT Rgion ON
exec sp_executesql N'SET NOCOUNT ON;
INSERT INTO [Region] ([ID], [Name])
VALUES (@p0, @p1);
',N'@p0 int,@p1 nvarchar(4000)',@p0=-2147482647,@p1=N'adfdsf'

Table properties (SQL Studio)

EF Core version: 1.0.1
Operating system:  Windows 10
Visual Studio version:  VS 2015

I found out how to workaround this issue, pretty simple:
Instead of using ctx.Attach for new items (which was working before) I am using ctx.Add instead.
// get the guid for the current user and store it into the current model
if (model.ID == 0)
{
    ctx.Add(model);
} else
    ctx.Attach(model).State = EntityState.Modified;

Alternative which is obviously also working, to Attach and then reset the ID to 0 (zero):
ctx.Attach(model);
model.ID = 0;

This is by-design. Attach is designed for attaching existing entities and Add is for adding new ones. Calling Attach will always result in the root node being in the unchanged state. This was an intentional change between RC2 and RTM. We recently clarified this in the API documentation https://github.com/aspnet/EntityFramework/commit/4beebdfda9af140ff077272376bd1764b2e78411.


tweak set code to use hardcoded spec version

No, no review required. PR is only left as an example and will not be merged.


Asset license clarification

While is clear that the code itself is under the MIT license, the asset (sprites, sounds, etc) status is not really clear. Are they also under the MIT?

An actual answer would've been nice...

Sorry about that! Closed all the issues after the project reset. Everything is under MIT.


Scrollbar

Fix #108 : After this commit, the scroll bar after “Who uses EvalAI” is removed and is only working for medium and small screens.
Changes proposed in this pull request:
Changes :

Changed the overflow of x from scroll to auto in src/app/components/home/partners/partners.component.scss
included the max-width value for medium and small screens and set overflow-x to visible.

The screenshots of the changes are added below:

@RishabhJain2018 @gautamjajoo There is the problem with surge credentials(https://travis-ci.org/Cloud-CV/EvalAI-ngx/builds/507571473#L648) as it is not able to login and thus failing the travis build. The same thing happened for PR #96 . Please look into this issue and fix it.

Hey @Abhi58, Don't worry about it. I'll fix it.

@Abhi58 Seems that you have fixed two issues in one PR. Make different PRs for different issues. Also this is a duplicate of #103.

Sure @gautamjajoo . Thanks for the feedback.

@Abhi58 Please fix the Travis build

@Abhi58 , There are many ways to do. Here is the simple step to remove unrelated changes :

git reset (you have to do soft reset since hard reset will remove your changes as well) to your last master commit.
Now all the changes are still present and you can git add relevent files and folder, then commit.
To reflect new commit in this PR, you have to force push git push -f origiin branch_name.

If you want to be safe and don't want to mess with commit and git commands, I will recomment you to before doing this, create a new branch. Now come back to this branch scrollbar and try above changes.
You can easily find many articles if you stuck in any git command. Hope this will help.

Closing this one since this is the duplicate of #103 .


dev -> main beta (fix assets build)

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


JSON.parse throws error if response isn't valid json string

This line throws an error if you try connecting to a wrong url.
To reproduce this issue try connecting to https://tmp54.tmpnb.org/user/ which will return html rather than a valid json string. The full trackeback is:
SyntaxError: Unexpected token <
    at Object.parse (native)
    at [object Object].req.onload (/Users/lukasgeiger/code/hydrogen/node_modules/jupyter-js-services/node_modules/jupyter-js-utils/lib/index.js:166:33)
    at [object Object].dispatchEvent (/Users/lukasgeiger/code/hydrogen/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:591:25)
    at setState (/Users/lukasgeiger/code/hydrogen/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:614:14)
    at IncomingMessage.<anonymous> (/Users/lukasgeiger/code/hydrogen/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:447:13)
    at emitNone (events.js:72:20)
    at IncomingMessage.emit (events.js:166:7)
    at endReadableNT (_stream_readable.js:905:12)
    at doNTCallback2 (node.js:465:9)
    at process._tickCallback (node.js:379:17)

I'd say that's behaving as expected. The line above indicates you've told it that the request will return JSON:
https://github.com/jupyter/jupyter-js-utils/blob/master/src/index.ts#L282

Maybe I posted this issue in the wrong repo. I'm using listRunningSessions from jupyter-js-services and think this shouldn't throw an uncaught error.

In that case, the tmpnb server is violating the API contract:
https://github.com/jupyter/jupyter-js-services/blob/2169d19/src/session.ts#L191
  /sessions:
    get:
      summary: List available sessions
      tags:
        - sessions
      responses:
        200:
          description: List of current sessions
          schema:
            type: array
            items:
              $ref: '#/definitions/Session'

So, I'd saying throwing is the correct action here.

cc @blink1073

Agreed, we are expecting the response to match the one described in https://github.com/jupyter/notebook/blob/master/notebook/services/api/api.yaml.   The existing notebook expects the returned value to be of "json" type as well: https://github.com/jupyter/notebook/blob/master/notebook/static/services/sessions/session.js#L83.

Agreed that it is expected behavior.

I understand. Thanks for your help!


APEXCORE-525 fail validation if statefull streamcodec does not implement newInstance method.

If user implements a streamcodec extending DefaultStatefulStreamCodec they need to also remember to implement newInstance method as platform use object returned by newInstance as codec. If use forget to override newInstance then  platform keep on using default codec without any warning to user and
such error are caught very late mostly resulting incorrect data distribution and incorrect result at final
operator.

Can we change DefaultStatefulStreamCodec.newInstance() to call return getClass().newInstance() instead of return new DefaultStatefulStreamCodec<>()?

newInstance() may also rely on clone() and call getClass().newInstance() if clone() is not supported.

@tushargosavi Please fix checkstyle, add unit test and comment on BufferServerPublisher changes.

don't merge this pull request yet. I will do some tests early next week, and fix the checkstyle errors and comment on BufferServerPublisher changes

@vrozov I have updated the pull request.
Removed validation as it was not needed as suggested, instead returned correct instance from newInstance method of DefaultStatefullStreamCodec

@tushargosavi Please add unit test.

Added a unit test checking for type of instance returned by newInstance method.

@vrozov I have done the suggested changes

@tushargosavi commit is merged to apache master. @asfgit did not close PR due to forced push. Please close it manually.

closing the pull request, as it did not got closed because of forced push


Automation triggered for incorrect total elapsed time conditions

I have 2 automations for workout sport type where total elapsed time is mainly used to determine which automation to be executed:

Daily workout: 00:20 h < total elapsed time < 01:00 h (between 20 minutes and 1 hour)
Cold shower: total elapsed time < 00:03 h (less than 3 minutes)

Daily workout automation was created a while back and worked fine by itself. After I created Cold shower automation, workouts that match Daily workout automation are now processed by the Cold shower automation.
Is there something that I have misconfigured, or there is something else causing the issue?

Debugging info
Strava activity IDs that were both processed today as Cold shower:

6482483181 (should have been processed by Daily workout automation)
6482483634 (processed correctly by Cold shower automation)

Thanks!

Hi Razvan!
It's the first time I hear about such issue so to be honest I have no idea what's going on :-(
Unfortunately I don't log much about processed activities for privacy reasons, so there's no way to troubleshoot via logs only.
I tried checking the activity<PHONE_NUMBER>, but it was either deleted or you have set it as private. If the activity is private and you give me your written consent here, I can get the activity details via API with a delegated user and double check what happened. Or you could also just make it public for some hours so I can get its details.

Hey Igor, thanks for taking the time to look into it. Both activities are now public.

Found the issue. Time based conditions (moving time and total time) might not work depending on the timezone, as they're being parsed as a time (HH:MM) instead of a duration in seconds. I'm actually surprised that this wasn't caught and reported earlier, as there are many other users on your timezone.
Fix should be deployed tomorrow. Thanks for reporting it. 🙂

https://github.com/strautomator/web/releases/tag/v22.16.11928

Just uploaded the activities from today and it worked as expected! Thanks for the quick resolution Igor! 🙌🏻


Documentation: Installation

Two bigger changes that I think would make the installation chapter better:
1.) Fix use of the term "requirements." It appears that it is being used in different ways, which will confuse your end users:

The summary at the beginning says there are multiple ways to install, including on this list "requirements," which is not exactly a mode of installation.
*The list of pre-requisite software appeared under the heading "requirements." I changed this to "prerequisites," but this calls attention to the issue more than fixing it.
*There is a script called requirements that installs the actual components of OWASP nettacker.

Here's what I'd like to do:

Move prerequisites higher up in the chapter, because they should be checked before installation begins. Maybe a new section up above installation methods would do it.
Remove "requirements" from that list of installation methods

Let me know if I've misinterpreted anything here.
2.) Add a support matrix at the top of the installation page, with more detailed information about supported versions of operating systems (e.g. what OSs are actually tested) and any info about problems with individual versions of Python. I would need some feedback for that.
(I see that there's a line or two in the introduction, but more would be better.)

Hi,
Thanks for your contribution and ideas, we are already testing the framework on travis-ci but it's not supporting much OSs for now, so that would be great to add the matrix, but should we add it now or wait for the official release?
Best Regards.

Hey @cwcromwell, I was just going through the changes you made to the installation wiki and it looks clean and more human readable. I wanted to give a suggestion. Docker Installation is supposed to be another method of installation. The current structuring makes it look like a fallback option in case every other step doesn't work. I think we should reorder or reword it better. What do you think?

I'll take a look.

I think I've covered all the changes discussed above.
To improve the flow, I did move some of the info, as proposed, into two new sections titled "before installation" and "after installation."
I added a bit of information about the ability to cover operating systems and versions of Python, and we can update this with an actual matrix later, when it is easier to do so.
I also took care of the issue with the Docker section, but let me know if that works as you had in mind, @shaddygarg.


Update UDC screen

User Story
As Chris I would like to have a cleaner looking UDC screen. Split from issue #587
Acceptance criteria

Display a short description of the UDC and utility token
Display current deposited utility token amount
Display icon for depositing utility tokens
Display icon for withdrawing utility tokens
Display short warning if the utility token amount is less than what is required for enabling monitoring

Tasks

[ ] Needs mockup
[ ] In-iteration design

@raiden-network/light-client


Syntax errors

Hi,
I have just downloaded version 1.3.1 of pymcz to use on some galaxies.
I am getting a syntax error in the mcz.py script, specifically on line 432
def calc((i, (sample, flux, err, nm, bss, mds, disp, dust_corr,               verbose, res, scales, nps, logf))):
and
line 470
def run((name, flux, err, nm, path, bss), nsample, mds, multiproc, logf, unpickle=False, dust_corr=True, verbose=False, fs=24):
This syntax error removes itself when I remove the extra brackets present inside the calc and run functions. However if I do this then I come across other errors, specifically the run function missing inputs bss, nsample, mds, multiproc and logf. (Even though nsample and multiproc have been correctly parsed as arguments.)
I am using python anaconda distribution version 3.7.3
Is this a compatibility issue?
Thanks

I realised this is a compatibility issue with python2 and python3. Could I suggest a note on the readme page outlining this? Thanks!


Apps Refactor: Refactor all imports to use module namespaces

Proposed Changes
Currently in the code base there are many examples of large imports into the global namespace. For example, here is a sample of forms-related imports from nautobot/extras/forms.py:
from django import forms

from nautobot.utilities.forms import (
    add_blank_choice,
    APISelectMultiple,
    BootstrapMixin,
    BulkEditForm,
    BulkEditNullBooleanSelect,
    ColorSelect,
    CSVModelChoiceField,
    CSVModelForm,
    CSVMultipleContentTypeField,
    DateTimePicker,
    DynamicModelChoiceField,
    DynamicModelMultipleChoiceField,
    JSONField,
    MultipleContentTypeField,
    SlugField,
    StaticSelect2,
    StaticSelect2Multiple,
    BOOLEAN_WITH_BLANK_CHOICES,
)

I'm proposing we move to this pattern:
from django import forms as djforms  # Django forms module
from nautobot.utilities import forms  # Our utilities.forms module
from nautobot.dcim import models as dcim_models  # DCIM models

 # In code..
class SomeForm(forms.CustomFieldModelForm, djforms.ModelForm):
    name = djforms.CharField(required=False)
    device = forms.CSVModelChoiceField(queryset=dcim_models.Device.objects.all())

    class Meta:
        model = dcim_models.Device

Justification
The current pattern is not only is unwieldy, but complicates merges, rebases, and refactors and in some cases can result in circular imports.
This way it's clear what is coming from Django forms and what is coming from internal forms modules, and any time something new is needed or used from nautobot.utilities.forms it doesn't need to be explicitly imported.
This also applies to more than just forms, but I was just selecting forms as a singular example of imports for a single module.
TODO

[ ] #2794
[ ] #2872
[ ] #2873
[ ] #2866
[ ] #2867
[ ] #2868
[ ] #2869
[ ] #2870
[ ] #2871

refactor_imports.zip


New version: GDAL_jll v301.600.0+0

Autogenerated JLL package registration

Registering JLL package GDAL_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/GDAL_jll.jl
Version: v301.600.0+0
Commit: 86642033c0ed3b41d537ee8c68d41793b29c8457
Revision on Yggdrasil: https://github.com/JuliaPackaging/Yggdrasil/commit/4f60ccdb198ff7a82919c1c58b3d40f848075757
Created by: @visr

[noblock]
The loading of the package should work now after fixing a dependency of this package in https://github.com/JuliaPackaging/Yggdrasil/pull/5852, registered in #72128. Locally now it loads without problems.


onCreateView is called every time tab is switched

How can we prevent to lose the current status of the fragment?
I add more details: I'm building an app that has a tinder-like UI with cards. If user scrolls the cards deck, then change fragment, when it comes back the deck is re-load. I need to maintain the position, aka restore view status

are you using ViewPager in your fragments? if so you need to setOffscreenPageLimit to how many fragments there is inside your viewpager

Hi,
I'm facing the same problem. In your sample code, onCreateView is called everytime but listviews does not recreated. I don't know why listview not recreate but others views that i added are recreates.

When switching tabs, fragments are replaced, and the previous fragment putted into backstack with saved state. If you switch back, the fragment is recreated and restores its own state from the saved state.


added corona virus analysis

I have done a small and simple analysis on COVID 19 data. Hope, you will review it.

Hey @lucifertrj   , Contribute to  : https://github.com/py3-coder/Model-EDA-Analysis-CART this repo for Data Science

Hey @Bibekkun , Contribute to : https://github.com/py3-coder/Model-EDA-Analysis-CART this repo for Data Science

Ok

Hey @Bibekkun , Contribute to : https://github.com/py3-coder/Model-EDA-Analysis-CART this repo for Data Science

Hey, the CSV file is too big which takes lot of time . Can I only upload interactive python notebook file?


Fix for WFCORE-2054. keep-content if not all groups are listed

My understanding is that, in domain mode, user can provide a subset of serverGroup at undeploy time.
If --keep-content is not set, then the artefact will be not undeployed because still deployed in other serverGroup.
The change is as follow:

If keep-content has not been set, we check that all server groups have been listed. If that is not the case, then we keep the content.

Unit test added.

@bstansberry , thanks. Yes better phrasing.
JF

Windows Build 4312 outcome was FAILURE using a merge of e292f1e246e71f0cfc65be8d1d40f87c2e067546
Summary: Execution timeout (new); tests passed: 3550, ignored: 37 Build time: 01:15:34

retest this please

Windows Build 4329 outcome was FAILURE using a merge of e292f1e246e71f0cfc65be8d1d40f87c2e067546
Summary: Execution timeout; tests passed: 3847, ignored: 74 Build time: 01:15:23


Exclude importing tensorflow.contrib.cloud on Windows

This is a follow up to fix import error in tensorflow/contrib/init.py on Windows, which is caused by https://github.com/tensorflow/tensorflow/commit/654eb3dd779107eb919af1093e2a72f0ab6ba922
@gunan @case540

The Ubuntu Sanity test failed in do_bazel_nobuild. Which is caused by
ERROR: Config value hdfs is not defined in any .rc file
136
137
FAIL: bazel build --nobuild --config=hdfs --config=gcp -- //tensorflow/... -//tensorflow/contrib/lite/java/demo/app/... -//tensorflow/contrib/lite/examples/android/... -//tensorflow/contrib/lite/schema/...
138
  This is due to invalid BUILD files. See lines above for details.

It also impact other PR's (e.g., #19534) Sanity checks as well.
Not very familiar with how CI setup so I haven't been able to find a fix yet.

Sanity failure is a known issue that will be fixed by the latest push soon.

We are also updating the google_cloud_cpp dependency. I sent a fix for their windows build and will see if the build is fixed internally later today.
Let's wait on that to see if the build is fixed with cloud component.

Actually it looks like we merged the build dependency change already, so merging this PR.