Add restrictions to data_description

We have this in the subject schema for possible restrictions on publication/sharing of data derived from a specific mouse, but these restrictions could also stem from other resources used (eg. reagents). This should be a more generic property that is inherited at the data description level.
Not sure if there are other properties that come with it - will look into that.

resolved by #76


[DOC] Consolidate Primitives in One Page

We currently have primitives in two pages: /develop/contracts and ./primitives. We need to consolidate both documents into /primitives

Finished in #1689


Space between card when there is scale

How can I reduce the space between the main card and the left and right card when using scale?

space

have you resolved it?


Added dependencies for react-intl feature branch

This will help keep the manifest in sync as we rebase the react-intl branch with master

Codecov Report

Merging #2091 (355a915) into master (60eb14d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #2091    +/-   ##
========================================
  Coverage   70.58%   70.58%
========================================
  Files         407      407
  Lines        7374     7374
  Branches     1576     1444   -132
========================================
  Hits         5205     5205
- Misses       2143     2145     +2
+ Partials       26       24     -2

Impacted Files
Coverage Δ

src/components/charts/costChart/costChart.tsx
9.37% <0.00%> (ø)

...omponents/charts/dailyCostChart/dailyCostChart.tsx
7.89% <0.00%> (ø)

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 60eb14d...355a915. Read the comment docs.


Empty vcell values

In some occasions, readvcells may return empty vcell IDs as well as values.
julia> cidReq = getcell(meta, location);
julia> cidNearest = getnearestcellwithvdf(meta, cidReq);
julia> vcellids, vcellf = readvcells(meta, cidNearest; species)
(UInt32[], Float32[])

We need to investigate why.

It turns out that it's probably due to the inner boundary:

The red crosses mark the locations which are contained in getcellwithvdf(meta) and allowed in getnearestcellwithvdf(meta, cidReq).


[Feature Request] Export a quest

I think the real strength of GN as a lore/story building tool is that you can export the data in an easy way. I've already worked on being able to automatically generate characters purely off the NPC JSON, items and skills also.
I think its worth having one for quests. Currently it cannot be exported

A simple JSON export should be fairly easy for a quest. I will include something like that in the next release.

Hi @steffendx , I'm not trying to rush anything, but could you let me know an estimate of an ETA for this export? I'm currently in the process of a full process of the exported data from GN right now, and having the quests/Aika would be great. If there is any way I can possibly help, let me know. I'm not good with JS at all but maybe a little bit of C#.

Hi, I was moving the last couple of weeks which took a lot of my time. Now I am almost done and I think I can finish the next release in a couple of weeks.
Thanks for offering your help, but I should manage now that the move is done!

Implemented with c80d0012afd10cf470c6705613b47e9650f6e9cd


importlib.metadata version and entry_points logic

Merge this first, then the other feature branches
Starting in Python 3.12, pkg_resources is deprecated, so importlib.metadata needs to be used instead.
Python <= 3.7:
pkg_resources
Python >=3.8 and <= 3.9:
importlib_metadata
Python >= 3.10:
importlib.metadata
Packages that have been vendored such as wrapt need to have this logic (or something similar to this).
TODO: a PR filed against wrapt to change this.

Codecov Report

Merging #898 (7eef569) into develop-python-3_12-support (f1a673e) will decrease coverage by 1.79%.
The diff coverage is 89.28%.

@@                       Coverage Diff                       @@
##           develop-python-3_12-support     #898      +/-   ##
===============================================================
- Coverage                        81.63%   79.85%   -1.79%
===============================================================
  Files                              184      184
  Lines                            18593    18597       +4
  Branches                          3257     3261       +4
===============================================================
- Hits                             15179    14851     -328
- Misses                            2504     2842     +338
+ Partials                           910      904       -6

Files Changed
Coverage Δ

newrelic/config.py
71.71% <83.33%> (+0.43%)
:arrow_up:

newrelic/hooks/framework_pyramid.py
82.02% <93.33%> (-0.78%)
:arrow_down:

newrelic/common/package_version_utils.py
100.00% <100.00%> (+2.04%)
:arrow_up:

... and 20 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Allow starting the server backend with a config file

This facilitates having the wrapper utilize the server backend.
The end workflow ended up with:

Not needing to define the backend in the terraform block
Using the wrapper (ie terraform-backend-git git terraform apply -input=false awesome.plan)
The wrapper would then proceed to use the server backend.

See https://github.com/plumber-cd/terraform-backend-git/issues/16#issuecomment-791063691 for more context.

Sorry that I didn't responded anything - I actually did looked into this PR and I wanted to modify it slightly to allow for a possibility of multiple config files on different levels that would override each other. So that we could have some sort of a "system" config and then per-module config overriding or expanding on the system baseline. Just haven't had a chance to look into it yet.

Hey @rigrassm - my apologies, I only got to this PR now.
My understanding is that with this PR - storage implementation now could read from config file and CLI args directly via viper.
I am trying to understand what is the intention in doing this though. Backend configuration in the terraform block was already optional (you could do terraform-backend-git git terraform apply -input=false awesome.plan as long as the config file present in the folder).
The idea in current implementation was that the backend will read config file and/or CLI args and generate tf block with a full configuration ready to be consumed by the storage (in the form of HTTP params). The intention in doing it that way was - so that the storage configuration doesn't know/care where the input came from, so there is less scenarios to test and overall we have storage implementation loosely coupled with the backend.
I'm just trying to understand - is there anything wrong with the current approach we need to fix? Is that because you want to hide full backend configuration details from the TF files? I just want to make sure I understand what we need to do here - there might be something else we'd need to refactor with this change, as well as update documentation properly.

As there is no feedback from the author on my last message - I am going to close this. I think it is best to keep backend implementation agnostic of the frontend CMD implementation, that way this project can also be used as a library in other implementations.


fix: Docs links URL missing a letter

Seems like COMMIT_TAG strips the leading v as well. This seems to contradict what we're seeing here, but oh well.
https://github.com/AztecProtocol/aztec-packages/blob/37bdc1820f92ad2cc545e6843a0dd90273c21e0e/build-system/scripts/setup_env#L17

Ah, found the culprit:
https://github.com/AztecProtocol/aztec-packages/blob/37bdc1820f92ad2cc545e6843a0dd90273c21e0e/build-system/scripts/extract_tag_version#L54

Docs Preview
Hey there! 👋 You can check your preview at https://65f35b665c5215220da4c58d--aztec-docs-dev.netlify.app


Add partial Wikidot support

Doesn't support wikis using third party search components like the SCP Wiki's Crom. Those will need special support on a case-by-case basis.
Example formatting for a Wikidot wiki using this setup. Note that this particular wiki's search function appears to be offline, however Wikidot wikis all use the same default search page (unless using the aforementioned third party components)
  {
    "id": "en-darksouls-fandom",
    "origins_label": "Dark Souls Fandom Wiki",
    "origins": [
      {
        "origin": "Dark Souls Fandom Wiki",
        "origin_base_url": "darksouls.fandom.com",
        "origin_content_path": "/wiki/",
        "origin_main_page": "Dark_Souls_Wiki"
      }
    ],
    "destination": "Dark Souls Wiki",
    "destination_base_url": "darksouls.wikidot.com",
    "destination_platform": "wikidot",
    "destination_icon": "darksouls.png",
    "destination_main_page": "index.php",
    "destination_search_path": "/search:site/q/"
  },

Thanks for putting this (and https://github.com/KevinPayravi/indie-wiki-buddy/pull/400) together!
From this post, it appears that Wikidot's search has been down for over a year now, so redirects using the search endpoint will not work. We could change destination_search_path from /search:site/q/ to /, so that IWB redirects attempt to take the user straight to the corresponding article instead of through the search. The downside is that if the article name doesn't match exactly, the user will reach a 404 on Wikidot...and since the search doesn't work, the user will have to do some digging to find the article they want.
The DS Wikidot subscriptions have also lapsed, so the wikis are locked from editing + no HTTPs :/
Based on the above I'm inclined to put this on-hold until at least search is available. I wonder if the DS Wiki owner can be reached to see if the subscription can be resumed, and if one of the alternative search solutions can be added.

As I mentioned in https://github.com/KevinPayravi/indie-wiki-buddy/issues/485 , the alternative Wikidot search function search:main is currently functional (although I've only seen AQW Wiki use it). It works with the path structure added by this commit, so I think it could be worthwhile merging this commit even just to support AQW Wiki.
I looked into SCP Wiki's search:crom, but as far as I can tell, it's not possible to access it via URL. But there isn't a non-independent wiki to redirect to SCP Wikis, and I haven't seen Wikidot wikis outside of the SCP wikis that use search:crom, so that shouldn't be a problem.

I believe this is superseded by #738. It's now possible to add redirects to Wikidot wikis (as I have done in #755).


Fixing broken migration #57.

Short description explaining the high-level reason for the pull request
Additions

Removals

Removes a migration that doesn't appear to work when loading the database from an initial state.

Changes

Testing

Create a blank mysql database.  Configure cf.gov settings to use your mysql database.  Run python cfgov/manage.py migrate.  Notice if migrations work/don't work with intended changes.

Review

@kave @rosskarchner @kurtw

Screenshots
Notes

Todos

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

👍

👍 Thanks so much for this @vickumar1981! It turns out this issue was introduced recently by adding a field has_unshared_changes to CFGOVPage, and so this migration in particular was impacted because it was updating pages.  It's impacting our ability to run tests, so I'm going to go ahead and merge it.


SwiftUI on macOS, image seems not released until view disposed

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
On macOS(tested on 12.0 Beta 5), memory usage keeps growing when display images in a List. When scrolling down the list, the memory continues to grow (maybe image not released). And to make matters worse, memory usage grows again when scrolling back.
Following code works fine on iPad OS 15 beta7 and memory used about 28M.
Reproduce
struct KingfisherTestingView: View {
    @State var images: [String] = (0..<1000).map {
        "https://img0.baidu.com/it/u=398014162,1691239664&fm=26&fmt=auto&gp=0.jpg" + "?\($0)"
    }

    var body: some View {
            List(images, id: \.self) { image in
                KFImage(URL(string: image))
                    .backgroundDecode()
                    .downsampling(size: .init(width: 100, height: 100))
                    .cancelOnDisappear(true)
                    .frame(width: 100, height: 100)
            }
    }
}

When view appeared, just scroll down to bottom, and scroll back to top, down to bottom, back to top.
Result
Kingfisher 6.0.2:

Kingfisher 6.3.1:
*** Assertion failure in -[NSTableRowData removeRowsAtIndexes:withRowAnimation:], NSTableRowData.m:7591
removeRowsAtIndexes:withRowAnimation: can not happen while updating visible rows!
Kingfisher 7.0.0 beta3:

Removed .backgroundDecode()  and the memory seems normal now compared to AsyncImage.
And the main memory usage changed from "cg raster data / core animation" to "IOSurface".
It seems all data of images retained in memory even tableview is 're-using cells'. bug or feature of macOS?

Thanks for the information.
I guess it is just another inaccurate debug panel memory summary. The cg raster data is used when loading image data to Core Graphic and decode image there. The memory there is not managed by a high level framework like Kingfisher, but by the system. It will be reused when decoding a new image, but I am also not sure when it can be released. It is determined by the lower level of system.
Ref: https://github.com/onevcat/Kingfisher/issues/1765


Change Bundle to enable a streaming interface

We have a Bundle construct that is used to gather up the sparse blocks needed to represent a given set of revisions. It is composable / mergeable with itself, which makes it nice for slurping up incremental sets of blocks for synchronization between client and gateway.
Unfortunately, gathering blocks is currently a blocking (in the order of operations) routine, and must be completed before a Bundle can be serialized or deserialized.
This means there may be cases where we allocate a lot of memory just to move a bunch of tiny blocks from one storage location to the next. It also means we may defer serialization for a much longer time than necessary while we collect blocks from storage. Let's fix this by changing the Bundle to use a streaming interface. Maybe it can be rebuilt around AsyncRead/AsyncWrite?

Aside: it may be out of scope for this issue, but our Bundle should just be a CAR.
Related: #20


Added response_format capabilities to chat completion request

Added response_format capabilities including integration test.

Would also love to see the support for 'seed' in the request (as the latest OpenAI adds support for adding a random generator seed)

Also interested in this, are there any updates?

We need this module very much, please merge this PR as soon as possible 😭

Hey @TheoKanning , is there anything we can do to speed up this process?

@TheoKanning please let us know how we can help to speed it up. Might be interesting to look into having some community admin that could approve those PR and merge them.


New command

Added base of idea explained in #5 .
I took the opportunity to make a custom spinner to use with ora for DSI or maybe mechanic. That code was left in the new command script, but is probably best to separate in another file.

This is fantastic @fdoflorenzano! I've never seen a cleaner commit history 👍


iOS build failed

I've tried to build iOS app for device
ns build ios --provision XXX --for-device

And build failed with errors:
.../platforms/ios/Pods/SDWebImage/SDWebImage/Core/SDImageCoderHelper.m:31:40: error: no visible @interface for 'UIImage' declares the selector 'imageByPreparingForDisplay'
        UIImage *decodedImage = [image imageByPreparingForDisplay];
                                 ~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~
.../platforms/ios/Pods/SDWebImage/SDWebImage/Core/SDImageCoderHelper.m:48:40: error: no visible @interface for 'UIImage' declares the selector 'imageByPreparingThumbnailOfSize:'
        UIImage *decodedImage = [image imageByPreparingThumbnailOfSize:thumbnailSize];
                                 ~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.

PODs installed during build:
Installing AFNetworking (4.0.1)
Installing CHIPageControl (<IP_ADDRESS>)
Installing Firebase (9.3.0)
Installing FirebaseCore (9.3.0)
Installing FirebaseCoreDiagnostics (9.3.0)
Installing FirebaseCoreInternal (9.3.0)
Installing FirebaseInstallations (9.3.0)
Installing FirebaseMessaging (9.3.0)
Installing GoogleDataTransport (9.1.4)
Installing GoogleUtilities (7.7.0)
Installing MNFloatingActionButton (2.0.0)
Installing Mapbox-iOS-SDK (5.1.2)
Installing PromisesObjC (2.1.1)
Installing QBImagePickerController (3.4.0)
Installing SDWebImage (5.13.1)
Installing SDWebImagePhotosPlugin (1.2.0)
Installing TOCropViewController (2.5.0)
Installing Toast-Swift (5.0.1)
Installing nanopb (2.30909.0)

Which platform(s) does your issue occur on?

iOS (deployment target 13.0)

Please, provide the following version numbers that your issue occurs with:

CLI: 8.3.2
Cross-platform modules: 8.3.2
Runtime(s): ios 8.3.2
Plugin(s):

"dependencies": {
    "@nativescript-community/gps": "^3.1.4",
    "@nativescript-community/https": "^4.0.6",
    "@nativescript-community/ui-barcodeview": "^3.5.5",
    "@nativescript-community/ui-mapbox": "^6.2.15",
    "@nativescript-community/ui-pager": "^13.0.40",
    "@nativescript-community/ui-image": "^4.3.2",
    "@nativescript/camera": "^5.0.13",
    "@nativescript/core": "~8.3.1",
    "@nativescript/firebase-core": "^2.1.0",
    "@nativescript/firebase-messaging": "^2.1.0",
    "@nativescript/imagepicker": "^1.0.7",
    "@nativescript/localize": "^5.0.4",
    "@nativescript/theme": "~3.0.2",
    "@nstudio/nativescript-floatingactionbutton": "^3.0.4",
    "@triniwiz/nativescript-image-zoom": "^4.1.3",
    "@triniwiz/nativescript-toasty": "^4.1.3",
    "moment": "^2.29.4",
    "nativescript-dna-deviceinfo": "^3.7.3",
    "nativescript-imagecropper": "^4.0.3",
    "nativescript-intl": "^4.0.2",
    "nativescript-permissions": "^1.3.12",
    "nativescript-ripple": "^4.0.1",
    "nativescript-windowed-modal": "^7.0.0"
  },
  "devDependencies": {
    "@nativescript/android": "~8.3.0",
    "@nativescript/ios": "~8.3.2",
    "@nativescript/types": "~8.3.1",
    "@nativescript/webpack": "~5.0.8",
    "sass": "~1.53.0"
  },

Podfile:
platform :ios, '13.0'

post_install do |pi|
    pi.pods_project.targets.each do |t|
        t.build_configurations.each do |config|
            config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '13.0'
        end
    end
end

Any ideas how to fix this?

it seems you need to upgrade xcode or lower sdwebimage version

My bad. XCode already updated. But I forgot to select Command Line Tools.


[BUG] HTML indent removes code from first line

Bug:
When using the --fix option for eslint on the rule @html-eslint/indent, if there are multiple html elements on the first line, the code is altered as well. This is very confusing and prone to introducing bugs since elements are removed without notice. Please see replication steps for further info.
Replication steps:
Configuration
...
       {
           "files": ["ng-app/**/*.html"],
           "parser": "@html-eslint/parser",
           "rules": {
               "max-lines": ["error", 400],
               "@html-eslint/no-duplicate-id": "error",
               "@html-eslint/no-inline-styles": "error",
               "@html-eslint/no-obsolete-tags": "error",
               "@html-eslint/require-closing-tags": ["error", {"selfClosing": "always"}],
               "@html-eslint/no-duplicate-attrs": "error",
               "@html-eslint/require-img-alt": "error",
               "@html-eslint/no-extra-spacing-attrs": "error",
               "@html-eslint/indent": ["error", 4],
               "@html-eslint/quotes":["error", "double"],
               "@html-eslint/id-naming-convention": ["error", "camelCase"],
               "@html-eslint/no-multiple-empty-lines": "error"
           }
...

Problematic HTML file:

        inner div

Expected formatting result

        inner div

Actual formatting result - The outer div is removed, breaking the HTML

        inner div

Notes:
This issue only occurs if the two element next to eachother are on the first line of the HTML file. Formatting this file will be done as expected:

This is the first line

        inner div

Let me know if I can provide you with further information :)

@melvinmicallef  Thanks for the report. I can reproduce it - demo. I'll fix it as soon :)

@melvinmicallef Thanks for reminding me. I'm still working on it :) Thanks.

Hi @melvinmicallef   It's fixed in v0.14.0 by replacing the parser we use. Thanks for reporting it.

demo


Unexpected behaviour of AsNoTracking()

AsNoTracking method does not allow to include data for all elements in a collection. Reference properties are filled only for first matching by id objects. Full info can be found here, in my SO question
Steps to reproduce
There is a sample project branch states-countries-relation
Further technical details
EF Core version: 2.2
Database provider: Pomelo.EntityFrameworkCore.MySql
Target framework: .NET Core 2.2
Operating system: macOS Mojave
IDE: Rider 2019.2

@VitaliiIsaenko I suspect the model configuration doesn't match what you have in the database. This is the model EF is building:
Model:
  EntityType: AddressDto
    Properties:
      Id (int) Required PK AfterSave:Throw ValueGenerated.OnAdd 0 0 0 -1 0
      CountryId (int) Required 1 1 -1 -1 -1
    Navigations:
      CountryLocale (<CountryLocale>k__BackingField, List<CountryLocaleDto>) Collection ToDependent CountryLocaleDto 0 -1 1 -1 -1
    Keys:
      Id PK
  EntityType: CountryLocaleDto
    Properties:
      Id (int) Required PK AfterSave:Throw 0 0 0 -1 -1
      Locale (string) Required PK AfterSave:Throw 1 1 1 -1 -1
      AddressDtoId (no field, Nullable<int>) Shadow FK Index 2 2 2 0 0
    Keys:
      Id, Locale PK
    Foreign keys:
      CountryLocaleDto {'AddressDtoId'} -> AddressDto {'Id'} ToDependent: CountryLocale

I suspect that maybe you want to use address.countryId as a foreign key, but that will only work if the address is on the dependent end of the relationship, which is in conflict with using a collection navigation property here.
You might want to take a look at the documentation on modeling relationships: https://docs.microsoft.com/en-us/ef/core/modeling/relationships?tabs=fluent-api

@ajcvickers thank you for helping out! Unfortunately, you had a look at the master branch that I didn't want to modify due to another question on the topic on SO. Branch states-countries-relation has an example of the described case. However, it is possible, that the same appears in the address-country relation as well. I understand that the relation setup might be wrong, however, it was the only way to achieve what I need and it works (thanks for help for my SO question). This relation is indeed non-traditional (due to locales support) but not that difficult to understand.
However, the question I'm posting here is relevant to another SO question I posted and has sample code on states-countries relation branch of the same repository. It also requires a similar setup in Context but I could not find any other option. I would be thankful for helping me with that as well but usually, it ends up referencing me to documentation without understanding this specific case's requirements. So, since it is working, I'm fine with this option that at least works. The only problem is - EF Core fills my reference collection for every object without AsNoTracking() and fills only the first element among ones with the same ids with AsNoTracking(). Following the documentation, this option should influence only the modification of data and should not influence its fetching. Therefore I opened this issue. I would be happy to either know the explanation of why it happens with further documentation adjustments or adding the bug into the queue if it is a bug.
Thank you again, hope now it is clear!

@VitaliiIsaenko This line:
.HasPrincipalKey(x => x.CountryId);

sets up an alternate key using CountryId. However, the code then inserts duplicate values with the same key value. This is not valid; each primary or alternate key much uniquely identify each entity.

Interesting, thank you! However, it works somehow.. maybe you have a better idea for such relation then?
But still, the question about AsNoTracking() is open and it indeed influences how reference fields are filled with .Include(). Isn't it a separate problem?

@VitaliiIsaenko The reason you are only seeing inconsistent results is that there are multiple entities with the same key. One of the entities is being discarded because it has the same key as an entity that was already retrieved, indicating that these are duplicates.

So does it mean that AsNoTracking() method influences data retrieval in this case? It's not documented anywhere, it is meant to affect data modification due to documentation. That's why I call it unexpected behaviour (maybe undocumented fits better).

https://docs.microsoft.com/en-us/ef/core/what-is-new/ef-core-3.0/breaking-changes#no-tracking-queries-no-longer-perform-identity-resolution

@VitaliiIsaenko  Note that all EF behavior is undefined when the data does not comply with the constraints inherent in the configured model, which is the case here.

Thank you, I understand. Is there any example of such a relation? I haven't found it anywhere and on SO people suggested only this option. Basically, it's country having the same id for several rows due to locales referencing states having the same id for several rows for the same reason. The state has one country logically but many with the same id because of locales and vice versa. I would appreciate an example or maybe an it is not possible response :) Could not get this relation from documentation only.
But anyway, thank you for help!

@VitaliiIsaenko Consider this data from your post:

Example data would be:
address (id, countryId)
(1, 1)
(2, 1)
address_country (id, locale, name)
(1, 'en', 'Germany')
(1, 'de', 'Deutschland')

This means that address.countryId matches two rows in the address_country table. This could be modeled as so (using both navigation properties for clarity):
public class AddressDto
{
    public int Id { get; set; }

    public int CountryId { get; set; }
    public CountryLocaleDto CountryLocale { get; set; }
}

public class CountryLocaleDto
{
    public int Id { get; set; }
    public string Locale { get; set; }

    public List<AddressDto> Address { get; set; }
}

modelBuilder.Entity<AddressDto>()
    .HasOne(e => e.CountryLocale)
    .WithMany(e => e.Address)
    .HasForeignKey(e => e.CountryId);

But this limits each Address to a single CountryLocale. If you also want each address to be associated with many CountryLocale's, then this is a many-to-many relationship that requires a join table.

@ajcvickers, thank you a lot for your help! Also, thank you, @smitpatel, the link is very useful and explains much.
I understood that there is no way to configure the relationship I explained in EF Core. Just so it stays here for anyone else interested.


Fix lockfiles for historian

Description
Something seems to have gone wrong with these two recent changes to the lockfiles for historian, which I believe resulted in what we see right now in our pre-production deployment of R11s, where the historian container is in a crash-loop-backoff saying it cannot find module url-parse when executing amqplib/lib/connect.js (query in Azure Monitor)
In the first change, amqplib was updated, but a section for its dependency url-parse disappeared from lerna-package-lock.json (even though the dependency was still declared under amqplib).

Maybe related to the fact that the amqplib dependency itself disappeared from under @fluidframework/server-services.

In the second one, the dependencies for amqplib disappeared completely from under it:

In this PR I just removed lerna-package-lock.json from server/historian and ran npm i again to regenerate it. Running the install without removing the file was not fixing the issues.
Relevant extracts from this change:
amqplib dependency restored under @fluidframework/server-services:

amqplib dependencies show up under it again:

url-parse has its own section:

FYI @robertobe-ms not exactly a rollback, just trying to fix the issues.

@znewton @hedasilv it's not particularly "reviewable" but when you have a chance I'd appreciate any comments and/or green light from you.


Hello World in Assamese

The programming language used is javascript.

Please merge my pull request and add the tag "hacktoberfest accepted".

Please merge my pull request and add the tag "hacktoberfest accepted".

Hello, please add your info into the CONTRIBUTORS.md file following the convention in the README.md file, so I can accept your PR!

Thanks for the reply.

Please merge my pull request and add the tag "hacktoberfest accepted".

Hello, please add your info into the CONTRIBUTORS.md file following the convention in the README.md file, so I can accept your PR!

Hey. thank you for the reply. I have added the details in the CONTRIBUTORS.md file.


fixed issue with cert resources not able to define encryption on read…

Big picture

Updates made to provide more access to configure the vpn and address restriction with the certs that blocked usage of this module in newer Terraform versions.

Types of changes
Bugfix (non-breaking change which fixes an issue)

[x]  fixed issue with cert resources not able to define encryption on readonly

New feature (non-breaking change which adds functionality)

[x] Added variable support for transport_protocol, session_timeout_hours, and login_banner_text
[x] updated output with dns name
[x] Updated documentation to reflect the changes.

Hi @si-foley thanks for the PR. Do you mind fixing the merge conflict and I can merge it?

I'd appreciate this fix too @adenot


this REEKS!

see above

what

it fuckin REEKS!

are you interested in our new software Reeks? say reeks if yes if no say reeks


[BACKEND] Implement two config values for target sse-server

From reference implementation of sse-server:
https://github.com/the-ditanian-fleet/sse-server/blob/main/example/app.py
# Private and public SSE urls are different because of docker-compose
private_sse_url = "http://sse-server:8000"
public_sse_url = "http://localhost:8000"

Currently there is only one sse-server url pulled and used. This results in backend not directly connecting to sse-server container network and it makes local docker dev setups a PITA. Maybe it is possible to have two sse-server urls like in the python example application?

Would 100% accept a PR


Support IngressClassName in OpenSearch and OpenSearch Dashboards charts.

Is your feature request related to a problem? Please describe.
Kubernetes 1.18 and greater support the IngressClassName spec stanza as part of the Ingress kind.
Describe the solution you'd like
If defined in the chart values file, the Ingress spec stanza should use the IngressClassName value.
Describe alternatives you've considered
While we may be able to also leverage an ingress class name annotation, this is not always accepted.
Additional context

Ingress

{{- if .Values.ingress.enabled -}}
{{- $fullName := include "opensearch.uname" . -}}
{{- $servicePort := .Values.httpPort -}}
{{- $ingressPath := .Values.ingress.path -}}
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: {{ $fullName }}
  labels:
    app: {{ .Chart.Name }}
    release: {{ .Release.Name }}
    heritage: {{ .Release.Service }}
{{- with .Values.ingress.annotations }}
  annotations:
{{ toYaml . | indent 4 }}
{{- end }}
spec:
+  {{- if .Values.ingress.ingressClassName }}
+  ingressClassName: {{ .Values.ingress.ingressClassName | quote }}
+  {{- end }}
{{- if .Values.ingress.tls }}
  tls:
  {{- range .Values.ingress.tls }}
    - hosts:
      {{- range .hosts }}
        - {{ . }}
      {{- end }}
      secretName: {{ .secretName }}
  {{- end }}
{{- end }}
  rules:
  {{- range .Values.ingress.hosts }}
    - host: {{ . }}
      http:
        paths:
          - path: {{ $ingressPath }}
            backend:
              serviceName: {{ $fullName }}
              servicePort: {{ $servicePort }}
  {{- end }}
{{- end }}

@mprimeaux could you please change the title to reflect this issue? Thanks.

Hi, is there any update for this? IngressClassName is blocking us to migrate to this helm chart (which we would very much like to)
Thank you!

I'll review and update today.

@aschi1 PR #149 has been submitted for community review.
@peterzhuamazon @TheAlgo @DandyDeveloper @smlx FYI and thanks for your continued support.

@mprimeaux Thank you!


Badge or tag for submitted exercises.

Describe the solution you'd like
 I would like the tasks to be labeled with tags once I have submitted them in order to know which one I have already submitted and which one I have not.

Yeah it would maybe be cool if hole task changed its color when submitted

I am closing this as a duplicate of #427.


NextjsSite: Allow configuring nextjs-lambda build options

In the builder the default option defines the build command like this:
const defaultBuildOptions = {
    cwd: process.cwd(),
    cmd: "./node_modules/.bin/next",
}

This is problematic for me because I'm working in a Yarn monorepo, so the next binary is not in the working directory, but in the root of the monorepo. As far as I could tell in the NextjsSite construct the only place those defaultBuildOptions are actually overridden is on line 578 here:
packages/resources/src/NextjsSite.ts
const configBuffer = Buffer.from(
      JSON.stringify({
        cwd: path.resolve(sitePath),
        args: ["build"],
      })
    );

By passing in the path to my next binary there (see below. I determine the path to the binary using the command yarn bin next) my build instantly starts working as expected without having to change the hoisting strategy of my monorepo.
const configBuffer = Buffer.from(
      JSON.stringify({
        cwd: path.resolve(sitePath),
        cmd: '/<repositoryRoot>/node_modules/next/dist/bin/next',
        args: ["build"],
      })
    );

I'm more than happy to implement this and submit a PR, but I'd just like to run it past the maintainers to see how those props should be passed down into the construct.
Thanks!

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


package main: cannot find package "." ? What is "." dot package?

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.9.1 windows/amd64
Does this issue reproduce with the latest release?
YES
What operating system and processor architecture are you using (go env)?

amd64
Windows 10 Pro

What did you do?
My code :
https://play.golang.org/p/1PkuUDceTO
I have searched below errors on Google, nothing helpful at all. What is " . " package? I written as package main instead. Where the " . " comes from?
What did you expect to see?
Compiled or run with no errors.
What did you see instead?
package main: cannot find package "." in:         D:\OneDrive\Projects\Project-Go\sendmail

What command did you run?

I tried both ‘go run xxxx.go’ and ‘go build xxxx.go’ , however both failed
with errors like that.
Now, not only this file,all my other files raise error like this one. I
guess there is something wrong with my go program?
On Wed, Oct 25, 2017 at 11:54 PM Alberto Donizetti<EMAIL_ADDRESS>wrote:

What command did you run?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/22434#issuecomment-339377946, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADPo2Kkp5nv8YxpinMLrN3HA6mBQMzKOks5sv1nJgaJpZM4QGLsX
.

--
from Kanch's Gmail

I guess there is something wrong with my go program?

There must be something wrong with your Go toolchain and/or your environment. I suggest you remove everything and reinstall Go, that may fix your issue.
If you need help, please see the Questions wiki page; it has a list of good places for asking questions (we only use the github issue tracker for tracking confirmed bugs, and I think the issue here is in your environment).
I'm closing this, but we can always re-open later if it turns out this was a Go bug.

I hope that's not your password, but if it is, you should change it quick :(

Hi, I tried to reinstall Go, bu the error still there.
Even if it's just a simple hello world program.
package main

import ("fmt"
)

func main() {
	fmt.Printf("Hello, world.\n")
}

Even if it's just a simple hello world program.

According to your picture, you build your program with "Visual Studio Code". But this issue list does not for "Visual Studio Code" support. You have to show the command that you run ("go build" or whatever) to get that error message. Did you read https://golang.org/doc/code.html ? Did you follow what is suggested there before trying to use "Visual Studio Code" ?
Alex

Here's my result:

The error message is the same, that "." package cannot be found.

@ankanch I tried reproducing your error, and I cannot. Do you have test.go file in d:\onedrive\projects\project-go\practice ? What is the contents of test.go file?
Alex

@alexbrainman  Here's my practice  folder and contents of test.go`.
BTW,I have resolved this problem by delete practice  folder and recreate it as well as test.go.
That's wired.
Is the "." package stands for current directory?
Thanks anyway.

You cannot write this statement.
import “.”
As it would create an recursive import loop with the current package.

On 26 Oct 2017, at 19:51, hcnak<EMAIL_ADDRESS>wrote:
@alexbrainman Here's my practice folder and contents oftest.go`.
BTW,I have resolved this problem by delete practice folder and recreate it as well as test.go.
That's wired.
Is the "." package stands for current directory?
Thanks anyway.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Okay, thank you.
Then that would be very wired that I have NOT import package ".", but it
give my the error "package . not found" .
On Thu, Oct 26, 2017 at 5:05 PM, Dave Cheney<EMAIL_ADDRESS>wrote:

You cannot write this statement.
import “.”
As it would create an recursive import loop with the current package.

On 26 Oct 2017, at 19:51, hcnak<EMAIL_ADDRESS>wrote:
@alexbrainman Here's my practice folder and contents oftest.go`.
BTW,I have resolved this problem by delete practice folder and recreate
it as well as test.go.
That's wired.
Is the "." package stands for current directory?
Thanks anyway.
—
You are receiving this because you are subscribed to this thread.

Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/22434#issuecomment-339600916, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADPo2GRstJq_gCJc8Hgn2iccGHTo_OEFks5swEtagaJpZM4QGLsX
.

--
Best regards,
*Long Zhang (Kanch) http://akakanch.com *＠ Chengdu University of Information
Technology
2015 Computer Science Major
<EMAIL_ADDRESS>No.24 Block 1, Xuefu Road, Chengdu, Sichuan, China

Please show a complete code sample that reproduces the issue so that we can see for ourselves.

On 26 Oct 2017, at 20:08, hcnak<EMAIL_ADDRESS>wrote:
Okay, thank you.
Then that would be very wired that I have NOT import package ".", but it
give my the error "package . not found" .
On Thu, Oct 26, 2017 at 5:05 PM, Dave Cheney<EMAIL_ADDRESS>wrote:

You cannot write this statement.
import “.”
As it would create an recursive import loop with the current package.

On 26 Oct 2017, at 19:51, hcnak<EMAIL_ADDRESS>wrote:
@alexbrainman Here's my practice folder and contents oftest.go`.
BTW,I have resolved this problem by delete practice folder and recreate
it as well as test.go.
That's wired.
Is the "." package stands for current directory?
Thanks anyway.
—
You are receiving this because you are subscribed to this thread.

Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/22434#issuecomment-339600916, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADPo2GRstJq_gCJc8Hgn2iccGHTo_OEFks5swEtagaJpZM4QGLsX
.

--
Best regards,
*Long Zhang (Kanch) http://akakanch.com *＠ Chengdu University of Information
Technology
2015 Computer Science Major
<EMAIL_ADDRESS>No.24 Block 1, Xuefu Road, Chengdu, Sichuan, China
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I get the same error on ubuntu 16.04 using liteide and go 1.9.1

it's a very wired problem ,I solved it by recreating the directory and
files.
On Thu, Nov 16, 2017 at 6:44 PM, reza7rm<EMAIL_ADDRESS>wrote:

I get the same error on ubuntu 16.04 using liteide and go 1.9.1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/22434#issuecomment-344885071, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADPo2H-jwEZXvwaG7oARsLtGXU50V1Xpks5s3BIigaJpZM4QGLsX
.

--
Best regards,
*Long Zhang (Kanch) http://akakanch.com *＠ Chengdu University of Information
Technology
2015 Computer Science Major
<EMAIL_ADDRESS>No.24 Block 1, Xuefu Road, Chengdu, Sichuan, China

I had this issue as well (Windows 10 & go1.10). The error comes from not setting up your GOPATH in the environment variables See here in the installation docs and [here](https://github.com/golang/go/wiki/SettingGOPATH#windows for setting up your GOPATH).
To confirm that you have set up your workspace directory correctly enter the directory using cd path\to\workspace and running go list

Hello, I know this issue has been closed, but I have encountered the same problem. I found that I cannot compile and run any go code in the OneDrive folder, but when I moved to another path, the problem solved.  From go version go1.10.1 windows/amd64

@mohanson from the comment above:

The error comes from not setting up your GOPATH in the environment variables

is GOPATH set in your environment? If not, have you tried setting it?

@ALTree I did not set GOPATH. The Golang automatically creates a new folder named "go" in my user path, so since switch to the v1.10.x I will no longer set it. I just tried set GOPATH=MY_USER_PATH/go, the problem has not been solved.

@ALTree I found Golang doesn't think the Onedrive folder(and any sub folder in it) is a directory, the IsDir() function returns False:
package main

import (
	"log"
	"os"
)

func main() {
	f, err := os.Open("C:\\Users\\mohan\\OneDrive\\sandbox")
	if err != nil {
		log.Fatalln(err)
	}
	i, err := f.Stat()
	if err != nil {
		log.Fatalln(err)
	}
	log.Println(i.IsDir())
        // 2018/04/09 11:34:37 false
}

But in python: python -c "import os.path; print(os.path.isdir('C:\\Users\\mohan\\OneDrive\\sandbox'))" returns True.
I think this may be one of the reasons for the problem.

/cc @alexbrainman I'm almost certain this is a duplicate of another OneDrive related issue.
/cc @jessfraz do you know anyone inside Microsoft who might be able to assist.

#22579 (os: IsDir doesn't work with Windows OneDrive), possibly?
If that's the case, it should be fixed on tip (to be go1.11).


Duke Nukem Slightly problematic

Build version:
0.82.3
Game version:
1.0.1
Game resolution:
240x320
Device model:
RM2020
Android version:
10
Description of the issue:

Thanks


fix: portfolio dispatching (#820)

Description
Please describe your changes
Notice
Before submitting a pull request, please make sure you have answered the following:

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Do all new and existing tests pass? Does the linter pass?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
closes #820
Testing
Please outline all testing steps

Pull branch locally and run yarn dev to start the server
Go to your dashboard and switch between your wallets
Notice how the charts and wallet assets are now loading up

Screenshots (if applicable)
https://user-images.githubusercontent.com/35400192/151299673-3cb9cb51-ecca-45a0-b015-0962b19c7725.mov
Additional Info
I was asked to provide an estimate on how much time this took, and this issue took way longer than expected and the solution was a rather simple one. My rough estimate for this is around 4-5 hours

I agree with @0xdef1cafe - Please add a why comment to the change and remove the commented code. I'm happy to approve after that small change.


Error compiling on Archlinux

Describe the bug
I'm trying to compile libratbag but I get this meson error
Run-time dependency check found: NO (tried pkgconfig and cmake)
meson.build:364:1: ERROR: Dependency "check" not found, tried pkgconfig and cmake

I'm not sure which package I'm missing, "check" keyword is really too generic for a search...

it's unfortunately really called check but you may find it as libcheck, check-code, or some other name in the arch repos.
https://who-t.blogspot.com/2018/07/meson-fails-with-native-dependency-not-found.html has instructions on how to figure out what to install based on the error message.

I install check package but it installed a checkmk binary, so I had to rename it to check and it worked fine ty.

@whot you could have just pinged me :grin:
Archlinux keeps the upstream names so the package is called check.
@sigzegv that shouldn't be necessary, actually you should not do that.


Server in dev mode

Question
How can I run immich in dev mode?

Like developer mode? I guess it should work, but why? Or are you asking how to build the snap yourself? Can you specify what you are asking?

I want "play" with the immich api and from the immich documentation: "The generated SDK is based on the immich-openapi-specs.json file, which is autogenerated by the server when running in development mode.". I have installed your immich snap (by the way it's great), and want to know how run the immich server in developer mode.

Ah, the API is available at /api/ and the API docs are available here https://immich.app/docs/api
You need to generate a API Key under your user and pass it in as the HTTP header X-API-KEY.
For example in python I do this:
headers = { "X-API-KEY": secret, "Accept": "application/json" }
r = requests.get("http://<IP_ADDRESS>/api/people", headers=headers)
print(r.json())

Building the openapi-specs.json is only relevant if you have modified the Immich sources and changed the server api:s.

I needed the openapi-specs.json file to generate code for .NET, but I have discovered that it doesn't work in MAUI due to a bug. Thank you for your help.

Ah, I understand. My understanding is that the Immich developers commits the latest file whenever needed so you can browse to the file from the upstream sources. I hope you find a way around the bug.


chain query

hi @sockeqwe i have a question, Can you tell me how to build a chained query ? ex:
...
Object a = getBy(id); //query 1
List details = getbyObjectId(int id);  //query 2
a.setDetail(details);
return a;
thanks

Either traditional SQL joins

OR

combining observables (since queries return objects) you can use for instance combineLatest() or even .map() or flatMap() depends on what exactly you want to build


DMLC link removal

Replaced v1.5.x Julia page with the content on master. Also, replaced dmlc.ml links with available content.

@mxnet-label-bot Add [Doc]

Preview: http://<IP_ADDRESS>/api/julia/index.html

@aaronmarkham

@marcoabreu Sorry about rushing that through without addressing your questions. We needed to be rid of the malware links ASAP.
@IvyBazan Can you answer? We spoke about this offline and you had some reasoning why it might be ok to just kill the links with a #.

@marcoabreu I removed those links because I couldn't find any replacement for the content that was hosted on dmlc.ml. It seems better to temporarily have replaced the links with a #, while waiting for that content to be recovered and rehosted, rather than having malware links still active on the site. The current experience is that the links redirect to the current page.

Issues related to missing content:
Missing nin content-https://github.com/apache/incubator-mxnet/issues/15729#issuecomment-518131446
Missing nnvm blog: https://github.com/apache/incubator-mxnet/issues/15730#issuecomment-518131577


trunstile+/lang/reader not found?

Run raco pkg install --auto turnstile-lib gets
standard-module-name-resolver: collection not found
  for module path: trunstile+/lang/reader

After checking, it seems more like the problem of raco, it's trying to find
/home/Library/Racket/8.3/collects
/Applications/Racket v8.3/collects/

but stored in
/home/Library/Racket/8.3/pkgs


new minor release

after https://github.com/UnixJunkie/svmwrap/issues/6

underway here https://github.com/ocaml/opam-repository/pull/20035


OAV grid detection

The GDA grid detection on the OAV image creates a grid with the max range e.g. https://ispyb.diamond.ac.uk/dc/visit/cm31105-5/id/9330896. However, in OAV centring the pin width is calculated using the first inflection point of the pin. If we used this technique in Artemis to find the grid area we could get away with smaller grids.
Acceptance criteria

Discuss with @ndg63276 and @npat113 if this is sensible
If it is, try it and evaluate time savings compared to potential accuracy issues

you're assuming the crystal is in the loop. Rookie error

some form of x restriction from tip would be sensible though, and necessary if we were to move to grid setting at zoom 1 as part of the initial centring  after robot load (which I would like to try).

Yeah we should definitely discuss what a sensible max width of the grid should be. At the moment we do "to the edge of the screen at zoom level 7.5" which is completely arbitrary (and about 650 microns).

Thanks both. We should discuss when we get closer to thinking about it


Fix release document typo

Why are the changes needed?
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #3816 (614d60f) into master (bfd7f7b) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3816      +/-   ##
============================================
- Coverage     52.94%   52.88%   -0.06%
  Complexity       13       13
============================================
  Files           497      497
  Lines         27998    27998
  Branches       3860     3860
============================================
- Hits          14823    14808      -15
- Misses        11781    11789       +8
- Partials       1394     1401       +7

Impacted Files
Coverage Δ

.../kyuubi/server/mysql/constant/MySQLErrorCode.scala
13.84% <0.00%> (-6.16%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLCommandHandler.scala
76.84% <0.00%> (-4.22%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLGenericPackets.scala
76.59% <0.00%> (-2.13%)
:arrow_down:

...pache/kyuubi/engine/YarnApplicationOperation.scala
62.96% <0.00%> (-1.86%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
63.93% <0.00%> (-1.64%)
:arrow_down:

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
73.84% <0.00%> (-0.77%)
:arrow_down:

...yuubi/server/metadata/jdbc/JDBCMetadataStore.scala
89.27% <0.00%> (-0.70%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
80.74% <0.00%> (-0.63%)
:arrow_down:

...org/apache/kyuubi/operation/ExecuteStatement.scala
79.16% <0.00%> (ø)

...g/apache/kyuubi/operation/BatchJobSubmission.scala
74.64% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks, merged to master.


Current humidity for the Current Weather module

This will add in current humidity information as discussed in #399.
P.S. sorry my editor seemed to screw up the indentation, just noticed!

Thanks! Is it possible for you to fix the indention in this commit? And could you share a screenshot what it would look like? Thanks for your contribution!

Yuck white space is a nightmare across project - every time ;)
I fixed everything up, it should look ok when viewing all changes together.
Here is a screenshot - it is not pretty, but quite useful

Nice work. My suggestion is to use an Icon in stead of the word "Humidity" since this would require a lot of translations.

When I get some time in the next few days I will look into icons. I think something like this might be nice:

I can make something similar on my own (so no copyright problems, references, etc).

Looks awesome! :)

This free icon set includes humidity icons http://weathericons.io/ if that helps

I was just looking at that site before I checked this ticket! Thanks for the pointer, I will just stick to that one for now then to keep things similar.

Alright, should be all set to go! Here is a screenshot. Thanks for the support everyone - this is one of the first times I have done a pull request to a public repo.

Looks Nice. The only thing I'm not sure about is that it's on the next line. Not sure if that's the best place ... maybe ask some opinions on the forum?

It felt a bit heavy with all three measurements on the same line. I do agree though, it is a bit sparse

Note sure what to do with this PR. Did you ask for feedback on the forum? Also, since the water drop contains a % symbol. Maybe you could just do "100💧" ?

Hey Michael, I am sorry for the late response - I have been extremely busy lately. I did make a post on the forums today and am awaiting some creative insight!

Is this available yet?

@cowboysdude Not yet, we're still looking for a better location for the info. Suggestions are welcome.

@MichMich I like the icon you suggested.. and I'm thinking that it could be added to just underneath the temp... centered to avoid too much whitespace...    OR the entire module layout could be changed..
Get rid of the decimal part of the temp and just use straight temps such as;
56.9 should be just 57 and the humidity could be put to the right of that.... keeping the layout even ;)
That's my two cents worth.

Removing the decimal part would not work for Celsius temperatures. I rather have a exact temperature than the humidity.

Yeah I agree, that would be odd. I like the idea of being on the bottom -
most likely aligned to the right side to be out of the way. Hopefully this
weekend I can finally close this long pull request!
Also as a heads up, I was looking around in the data that we got by default
from our weather provider and it didn't seem like much other useful data
was around. We could add a min/max to that same line in the layout but that
gets repetitive if you have the forecast as well.
On Fri, Sep 30, 2016 at 4:24 PM Michael Teeuw<EMAIL_ADDRESS>wrote:

Removing the decimal part would not work for Celsius temperatures. I
rather have a exact temperature than the humidity.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MichMich/MagicMirror/pull/403#issuecomment-250855889,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH5hLZV4xYXAOUAgzennEP9CYZ92CAqGks5qvX4ggaJpZM4JnFub
.

Merged. Made some visual changes.

That'll work :)  I am going to change and get rid of the decimal.... I know you guys use C but here I use F and it's odd to see the decimal point for us hahahahahha

@cowboysdude Make it configurable and send a PR. That way anyone kan benefit from your change. 👍🏻

If there are still questions about the layout, I made a new version. This one has the decimal and also keeps things a bit more compact. Thoughts?


Arm32 architecture

Continuation of https://github.com/plotly/Kaleido/pull/81#issuecomment-809612042 to add circle ci build for arm32 architecture.
Thanks @tapiralec!

Hi @tapiralec, looks like everything went through alright on Circle CI.  Can you test out the build?

Download the zip file from https://1414-268501657-gh.circle-artifacts.com/0/repos/kaleido/kaleido_artifacts_linux_arm.zip
Unzip it
Pip install the wheel file: kaleido-0.2.1.dev3+ga14172a-py2.py3-none-manylinux2014_armv7l.whl

Thanks!

Hi @jonmmease, I uninstalled the version I built and installed the one from the zip. The example code on the Readme, and some of my own in-progress work, both ran without any issues for me!

arm package is not published to PyPI as 0.2.1.post1.


MapView.refresh() is not working.

I copied the whole code from BitmapLayerTestActivity from example project, I am using IndoorAtlas for indoor positioning inside the onLocationChanged i am using the below code
Bitmap bmp = BitmapFactory.decodeResource(getResources(), R.mipmap.ic_launcher);
IALatLng latLng = new IALatLng(iaLocation.getLatitude(), iaLocation.getLongitude());
PointF point = mFloorPlan.coordinateToPoint(latLng);
mBitmapLayer = new BitmapLayer(mMapView, bmp);
mBitmapLayer.setLocation(point);
mBitmapLayer.setOnBitmapClickListener(new BitmapLayer.OnBitmapClickListener() {
@Override
public void onBitmapClick(BitmapLayer layer) {
Toast.makeText(getApplicationContext(), "click", Toast.LENGTH_SHORT).show();
}
});
mMapView.addLayer(mBitmapLayer);
mMapView.refresh();
but refresh is not working. Everytime icon is rendering on map.

Also, if I am rotating using compass UI is become not responding. UI got hanged and finally app crashed.


feat: introducing stake_matched_total on MarketLiquidities

Introducing MarketLiquidities::stake_matched_total field that functionally replaces MarketOutcome::matched_total. The difference being that the former is a sum over all the outcomes, so the granularity is being lost.

ready for review


Extension is not loading

The python extension is not showing up in the status bar and the linting is also not working. It has been giving a lot of trouble hope to receive a solution soon. To add up I believe all this is happening when I installed git on my machine.

..github/ISSUE_TEMPLATE/2_bug_report.md

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please edit your opening comment by providing a filled-in copy of our issue template. Once you are done please leave a comment to that effect.
In your settings, do you have 'python.experiments.optInto: ["All"]'?

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please edit your opening comment by providing a filled-in copy of our issue template. Once you are done please leave a comment to that effect.
In your settings, do you have 'python.experiments.optInto: ["All"]'?

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please edit your opening comment by providing a filled-in copy of our issue template. Once you are done please leave a comment to that effect.
In your settings, do you have 'python.experiments.optInto: ["All"]'?

Hey I have edited the previous comment could u check it out.

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please edit your opening comment by providing a filled-in copy of our issue template. Once you are done please leave a comment to that effect.
In your settings, do you have 'python.experiments.optInto: ["All"]'?

Hey I have edited the previous comment could u check it out.

@rajenderK7 you need to copy the contents of that markdown file and fill in the details, like extension version, python version, the output logs if any, console logs if any, etc.

@rajenderK7 you need to copy the contents of that markdown file and fill in the details, like extension version, python version, the output logs if any, console logs if any, etc.

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Allow more complex yaml structure and fix readme

I added .flatten to the array of ignored_words and only_lines_matching.
This way we can have a more complex YAML structure and do things like:
able_words: &able_words
  - available
  - considerable
  - unable

. . .
ignored_words:
  - *able_words
  - ignoreme
  - measwell
. . .

This won't change anything in the behavior of the gem and will just result in better native YAML support.
The final list would look like:
. . .
ignored_words:
  - available
  - considerable
  - unable
  - ignoreme
  - measwell
. . .

Please consider merging this?

Can you check why the tests are failing?

Can you check why the tests are failing?

@ashkulz I patched up the tests b5e8e78491b8a66e2ebef2f95ab249885683f597 they are all passing now. Something must have changed in Aspell itself because I think when you first made the tests the suggestions it returned were full of whitespaces and simple different to what it returns now.
Anyway, this had nothing to do with the current PR it would have failed with or without.
Please approve this one when you find the time.
Cheers,
Khalil

@ashkulz  did you have chance to look at this?

I'm not sure why there's no checks from GitHub Actions?

@khalilgharbaoui can you rebase/merge with master so that the build status can be checked? I cherry-picked your test fix as 00a50d38ffb99dee2398aeef260f2ccb7e355da5.

@ashkulz all green :shipit:

Release created and pushed to RubyGems, thanks for your contribution!


[Snyk] Upgrade salesforce-alm from 54.2.0 to 54.2.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade salesforce-alm from 54.2.0 to 54.2.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2022-04-22.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #32 (d05ba1e) into master (1565902) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   85.18%   85.18%
=======================================
  Files          12       12
  Lines         871      871
  Branches      156      156
=======================================
  Hits          742      742
  Misses         67       67
  Partials       62       62

Flag
Coverage Δ

unittests
85.18% <ø> (ø)

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

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 1565902...d05ba1e. Read the comment docs.


Add topic tags

I suggest adding the topics fortran, plot, matplotlib in the About section, as explained at https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/classifying-your-repository-with-topics

Done. Thank you for your advice!


BlockingIOError() when mocking null-byte responses

Dear Giorgio,
first things first: Thanks for conceiving and maintaining this excellent pytest mocking module.
We just found MocketEntry instances pretending to respond with single null bytes like
MocketEntry(location=(None, None), responses=[0x00])

will not yield their responses properly due to the boolean-style test within MocketSocket.recv.
Instead, the BlockingIOError() will be raised.
With kind regards,
Andreas.

Hi @amotl,
I was trying to write a test which makes it fail that way, but I could not.
Could you please add a few lines of code which show the full picture?
Thank you very much in advance.

Dear Giorgio,
probably I didn't learn enough about Mocket yet / mixed up the data types. After tweaking the responses parameter to [b'\x00'] to designate a single null-byte (see also [1]), everything worked smoothly.
So, let's just close this issue. Sorry for the noise and keep up the spirit!
With kind regards,
Andreas.
[1] https://github.com/hiveeyes/terkin-datalogger/blob/14a9bb5e4331733261083f3c90e0364ac9275b10/test/test_lorawan.py#L197-L198

I am aware it still lacks a proper documentation. I am trying to find the time to make it better.
Feel free to contact me in case you have other problems.


Add a docker release

Adding a Docker release workflow for the keycloak-restrict-client-auth jar.
Addresses https://github.com/sventorben/keycloak-restrict-client-auth/issues/192

Closed due to missing feedback.


Board member card socials affecting badge positioning.

    After what seemed like 12 years, my badge's dimensions are correct I triple checked it. The issue I'm starting to see is that the people with the card social feature Diamond made are affected by this. Unless I have literally gone insane, look closer at Ethan's, Diamond's and my badges compared to the others who do not have their socials connected yet.

Originally posted by @AlanCortez1337 in https://github.com/EthanThatOneKid/acmcsuf.com/issues/632#issuecomment-1288378512

This is related to (and very likely depends on) issue https://github.com/EthanThatOneKid/acmcsuf.com/issues/348.

This issue is referring to the current way that we serve our officer images: with the diamond mask preprocessed into the webp. I think that Alan took the opportunity to resolve the existing webp inconsistencies for now.
I’m thinking to resolve #348, we will replace all of the images with their square-sized counterparts (currently on Figma) where the diamond mask is applied using CSS or SVG magic.


Keyerror in endpoints [credentials module]

update_credentials method line 127 -  [0] - to be removed because it is dict
File "/usr/local/lib/python3.10/dist-packages/py_ocpi/modules/credentials/v_2_2_1/api/cpo.py", line 127, in update_credentials
endpoints = response_endpoints.json()['data'][0]
KeyError: 0

Hello @murali-shankar, sorry for the long response, could you tell me if this error occurs every time you're using this functionality?


Enable CD for sqlserver-api

Link to GitHub repository
https://github.com/jenkinsci/sqlserver-api-plugin
When modifying release permission
This is needed in order to cut releases of the plugin or component.
If you are modifying the release permission of your plugin or component, fill out the following checklist:

### Release permission checklist (for submitters)
- [ ] The usernames of the users added to the "developers" section in the .yml file are the same the users use to log in to [accounts.jenkins.io](https://accounts.jenkins.io/).
- [ ] All users added have logged in to [Artifactory](https://repo.jenkins-ci.org/) and [Jira](https://issues.jenkins.io/) once.
- [ ] I have mentioned an [existing team member](https://github.com/orgs/jenkinsci/teams) of the plugin or component team to approve this request.

When enabling automated releases (cd: true)
Follow the documentation to ensure, your pull request is set up properly. Don't merge it yet.
In case of changes requested by the hosting team, an open PR facilitates future reviews, without derailing work across multiple PRs.
Link to the PR enabling CD in your plugin
https://github.com/jenkinsci/sqlserver-api-plugin/pull/2
FYI @cnhttpd @identifysun
Looks also permissions are not matching "recent" contributors
The PR also allign with other database API plugin (mysql, mariadb, postgres and h2) that are already included on bom
### CD checklist (for submitters)
- [x] I have provided a link to the pull request in my plugin, which enables CD according to the documentation.

### Reviewer checklist
- [ ] Check that the `$pluginId Developers` team has `Admin` permissions while granting the access.
- [ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
- [ ] If security contacts are changed (this includes add/remove), ping the security officer (currently `@Wadeck`) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it.

Hi @identifysun
I'm planning to adopt the plugin sqlserver-api-plugin in the next few days.
It's been 2 weeks that https://github.com/jenkinsci/sqlserver-api-plugin/pull/2 was open without any answer.
Also the plugin doesn't have any new commit since 2 years.
Thanks


Error with installation

Dear developers,
I am trying to install a cpu version of Uni-Core on WSL from the source using command python setup.py install --disable-cuda-ext. But I always get the following message:
error: urllib3 2.2.1 is installed but urllib3<1.27,>=1.21.1 is required by {'requests'}
After I install urllib3 1.26, it still goes wrong as showing below:

Processing dependencies for unicore==0.0.1
error: urllib3 2.2.1 is installed but urllib3<1.27,>=1.21.1 is required by {'requests'}
(venv_torch) (base) jingheng@Bai-Group:~/Uni-Core$ pip uninstall urllib3==2.2.1
Found existing installation: urllib3 1.26.0
Uninstalling urllib3-1.26.0:
Would remove:
/home/jingheng/venv_torch/lib/python3.9/site-packages/urllib3-1.26.0.dist-info/*
/home/jingheng/venv_torch/lib/python3.9/site-packages/urllib3/*
Proceed (Y/n)?

I am not sure what the problem is, please let me know how to fix it. Thank you.

This error may be due to multiple versions of urllib3 in your environment. Maybe you can try uninstalling urllib3 completely and reinstalling a compatible version.
pip uninstall urllib3
pip install urllib3==1.26.0

Thank you for your reply.

pip uninstall urllib3
pip install urllib3==1.26.0

I have tried this method and checked the version of urllib3 with pip list.

unicore                            0.0.1
Unidecode                          1.2.0
unimol                             1.0.0
unimol-tools                       1.0.0
urllib3                            1.26.0

But the same error keeps popping out.

If you are using the right environment but error: urllib3 2.2.1 is installed still exists, you can check your /home/jingheng/venv_torch/lib/python3.9/site-packages/ path. Maybe urllib3 2.2.1 hasn't been completely uninstalled, you can remove it manually.

This is due to that these libraries used for cuda acceleration were not successfully compiled. Don't worry, unicore works fine at this point.


Create an event from the Recommendation Section

This task is required for the completion of the user story:

TrueSparrowSystems/AI-SalesSparrow-iOS-App#10
TrueSparrowSystems/AI-SalesSparrow-iOS-App#11


[graphql] test cockroach cache en max-sql-memory

especially for a mainnet stack, if a server has not enough disk iops available, the processor crashes regularly with a timeout to the indexer. I have setup a spare mainnet stack with sata ssd's where I noticed this issue again. Will make some temporary changes there to increase the indexer db (cockroach) memory usage since the default cache is only 128MiB.
https://www.cockroachlabs.com/docs/stable/recommended-production-settings#cache-and-sql-memory-size
--cache=.45
--max-sql-memory=.45

nice! https://mon.grid.tf/d/rYdddlPWk/node-exporter-full?orgId=1&var-DS_PROMETHEUS=default&var-job=node_exporter&var-node=prod-02:9100&var-diskdevices=[a-z]%2B|nvme[0-9]%2Bn[0-9]%2B&from=now-24h&to=now&refresh=30s
more mem usage:

less iops:

No issues found so far. On the mainnet snapshot server noticed the processor restarting less, so that's a good sign. This srv has too few iops for a mainnet stack, so the processor issue was not going away but since it was very present there it was a good place to test.
Will apply the change to devnet and continue testing there before going to the other nets.

no issues detected or reported

seems good, will continue with qa, test and mainnet


Re-record site video

The video embedded in the About page is over two years old - it's probably overdue a refresh.
https://github.com/IBM/taxinomitis/blob/6ca1f90b0148a750a1549a41ca4cd470b863d870/public/components/about/about.html#L11

Done


Finding the backup of the bounce point training script.

Hi, Did anyone have the backup of the bounce point training script. I'm now working on my university project and I want to make a new function to judge the ball bounded or not. If anyone can provide the script I would really appreciate it.
And also did anyone know how to find the bounded point in the BigDataFrame.csv

Hi, I am looking for the same. Did you find something?


[Keccak] decompose - tests with corner cases would be nice to have

          I will sound annoying :): but tests with corner cases would have been nice to have, just for the reviewers to be convinced it is fine at a first sight.

Originally posted by @dannywillems in https://github.com/o1-labs/proof-systems/pull/1268#discussion_r1381742752

Let me know if this issue still applies after the new tests added


Desktop: getenv instead of process.env

So I set and env var to "1" and it silently failed. I found this getenv package which is tiny and allows you to get envvars by int, float, bool etc and errors nicely if a var is invalid or not set at all (required).
Basically if you mess up env vars at all this will catch it.

@keybase/react-hackers

Cause my window was looking like:

so this is fine BUT, we have to be super careful to not do this with process.env.NODE_ENV outside of webpack since we inject it with the define plugin so it'll only replace it if its of the form process.env.NODE_ENV and won't work with getenv('NODE_ENV') etc :+1:

yep left process.env.NODE_ENV key in that spot untouched

yup, you can merge


Missing information in ocp-ibm-cloud-roks-nfs.yaml

When running dynamic sample ocp-ibm-cloud-roks-nfs.yaml (https://github.com/IBM/cloud-pak-deployer/blob/main/sample-configurations/sample-dynamic/config-samples/ocp-ibm-cloud-roks-nfs.yaml), it posted the below errors.
TASK [lint-config : Fail if configuration errors have been found] **************
Sunday 02 October 2022  11:20:53 +0000 (0:00:00.037)       0:00:35.671 ********
fatal: [localhost]: FAILED! => changed=false
  msg: |-
    Validation errors:

    vsi:
    infrastructure:
        keys:
        - feiye-work-provision
        primary_ipv4_address: <IP_ADDRESS>
        public_ip: true
        subnet: feiye-work-subnet
        type: vpc
    name: feiye-work-bastion

    ERROR: Attribute infrastructure.image is not defined

    nfs_server:
    infrastructure:
        bastion_host: feiye-work-bastion
        keys:
        - feiye-work-provision
        primary_ipv4_address: <IP_ADDRESS>
        storage_folder: /data/nfs
        storage_profile: 10iops-tier
        subnet: feiye-work-subnet
        type: vpc
        volume_size_gb: 1000
        zone: eu-de-1
    name: feiye-work-nfs

    ERROR: Attribute infrastructure.image is not defined
    ERROR: Attribute infrastructure.profile is not defined

Based on the doc https://ibm.github.io/cloud-pak-deployer/cpd-design/objects/infrastructure#ibm-cloud, they are all Mandatory.

Duplicate of #120


Revert "Remove Manjaro Linux icon"

*Received permission from Manjaro core team
This reverts commit df3a1655139f2d75938d29498d4a10222ab6f420.

Thank you✨


feat(protocol-designer): create color enhancement announcement modal

closes RLIQ-424
Overview
Creates announcement modal for the PD 6.2 release features
Changelog

add object with modal info to announcements and format.
add the gif

Review requests

look in sandbox, does the gif work as expected and look high enough quality?

Risk assessment
low

@jerader could you share the sandbox link?

I'm not familiar with pd code but for this AnnouncementModal, it would be good without overflow-y: scroll.

@jerader could you share the sandbox link?

@koji sure - http://sandbox.designer.opentrons.com/pd_announcement-modal-liquid-enhancements/
Basically, to get the sandbox for any PD branch, after the build is complete, you do  http://sandbox.designer.opentrons.com/branchName/

@jerader could you share the sandbox link?

@koji sure - http://sandbox.designer.opentrons.com/pd_announcement-modal-liquid-enhancements/
Basically, to get the sandbox for any PD branch, after the build is complete, you do http://sandbox.designer.opentrons.com/branchName/

@jerader thanks, that is good to know!


Add bio field for user

Fix #4339
user with simple bio:

user with no bio:

user with xss attempt that was escaped:

Codecov Report

Merging #6113 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #6113      +/-   ##
==========================================
+ Coverage   38.85%   38.85%   +<.01%
==========================================
  Files         349      349
  Lines       49801    49802       +1
==========================================
+ Hits        19350    19353       +3
+ Misses      27650    27648       -2
  Partials     2801     2801

Impacted Files
Coverage Δ

models/user.go
49.39% <ø> (ø)
:arrow_up:

modules/auth/user_form.go
41.81% <ø> (ø)
:arrow_up:

routers/user/setting/profile.go
40.81% <100%> (+0.4%)
:arrow_up:

models/repo_list.go
67.89% <0%> (+1.05%)
: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 f5fa22a...b6d5dd8. Read the comment docs.

I think LDAP also has a user description attribute - although I'm not sure how many installs use it.

I think LDAP also has a user description attribute - although I'm not sure how many installs use it.


Feature request: Increase vertical space between lines

Although it is possible to increase font size and horizontal space between tags, I am not managing to increase vertical line spacing.

Yes, you can change the font and use spaceBetweenLines property for that effect. Thanks


Error evicting pod

What version of descheduler are you using?
descheduler version: 0.25.1
Does this issue reproduce with the latest release?
yes
Which descheduler CLI options are you using?
--policy-config-file /policy-dir/policy.yaml --v 3
Please provide a copy of your descheduler policy config file
apiVersion: "descheduler/v1alpha1" kind: "DeschedulerPolicy" strategies:   HighNodeUtilization:     enabled: true     params:       nodeResourceUtilizationThresholds:         thresholds:           cpu: 30           memory: 20           pods: 100       LowNodeUtilization:     enabled: true     params:       nodeResourceUtilizationThresholds:         targetThresholds:           cpu: 50           memory: 50           pods: 50         thresholds:           cpu: 20           memory: 20           pods: 20   RemoveDuplicates:     enabled: true   RemovePodsHavingTooManyRestarts:     enabled: true     params:       podsHavingTooManyRestarts:         includingInitContainers: true         podRestartThreshold: 100   RemovePodsViolatingInterPodAntiAffinity:     enabled: true   RemovePodsViolatingNodeAffinity:     enabled: true     params:       nodeAffinityType:       - requiredDuringSchedulingIgnoredDuringExecution   RemovePodsViolatingNodeTaints:     enabled: true   RemovePodsViolatingTopologySpreadConstraint:     enabled: true     params:       includeSoftConstraints: false
What k8s version are you using (kubectl version)?
EKS
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:30:10Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"} Server Version: version.Info{Major:"1", Minor:"21+", GitVersion:"v1.21.14-eks-6d3986b", GitCommit:"8877a3e28d597e1184c15e4b5d543d5dc36b083b", GitTreeState:"clean", BuildDate:"2022-07-20T22:05:32Z", GoVersion:"go1.16.15", Compiler:"gc", Platform:"linux/amd64"}
What did you expect to see?
I wanted to see that on nodes with 100% pods usage some pods would be evicted und started on nodes with lower pods usage than 100%
What did you see instead?
see attatchment
debug-descheduler.txt

Hi @iamdevnull, it looks like your EKS version is 1.17, so the descheduler version 0.25.1 is not supported for your cluster. The latest supported descheduler version for k8s 1.17 is v0.19.0

Thx a lot. I will test it.

it works as expected.


[Feature]: Top to bottom icon hiding the footer content

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
I will adjust that icon so footer content not hide
Alternatives
No response
Additional Context
No response
Are you interested in contributing?

[X] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

Which program are you contributing in?
gssoc

Hello @pranshugupta54 please assign this issue to me under gssoc 24

Hello @pranshugupta54 please assign this issue to me under gssoc 24

Hello @pranshugupta54 i have raised pr https://github.com/digitomize/digitomize/pull/956 for this issue. Please check

Hello @pranshugupta54 . i have raised new pr https://github.com/digitomize/digitomize/pull/957. please check

@kapilG0, please push the changes in same branch to get them on same PR instead of making multiple PRs.

@kapilG0, please push the changes in same branch to get them on same PR instead of making multiple PRs.

Ok

@kapilG0, please close one PR.

@kapilG0, please close one PR.

Hello @pranshugupta54 done


Instagram App Ads

Add blocking for instagram ads. Added fakenews+porn+malware as host in AdAway but still ads are annoying.

@abhijithvijayan You cannot block those ads since both the content and ads are delivered using the same domain i.e; if you block the ads you will also block the content (same applies to Facebook, YouTube and so on...)

@anudeepND Thanks


Use publishable package name

Fixes #14

Apologies for the wait, the past few weeks were incredibly busy for me.
Many thanks for the PR's, as always!


MUST vs SHOULD requirements in the spec

continuing discussion in #49 on whether the spec should require certain notifications ("must") or only strongly suggest them ("should")
Case-in-point here: What was previously CompileReport and TestReport notifications, in #49 subsumed by generic task notifications with predefined dataKind.

In general, I think the protocol should make it as simple as possible to write a compliant minimal server or client.
For the server this means:

the less functionality it has to provide, the easier it is to write.

For the client this means:

the more assumptions it can make about behavior of the server, the easier
the more functionality it can ignore, the easier

Where these goals are not in conflict, I think it is prudent to not require specific behavior in the spec, but rather suggest the standard way for an implementation to provide a better integration.
In the case of task notifications, simple clients may choose to drop them altogether or merely log their messages. Simple servers may only report success or failure. A client can not expect any particular notification to be part of a task.
For example, a compile request may not trigger any compilation task at all, because the complete build has already been cached. So it should not be strictly required to send these notifications in the first place.

I agree @jastice

I'm fine with that approach. In general, though, I prefer explicitness over implicitness. The more explicit and constrained the protocol is for me as an implementor of the BSP server, the better. But this is a personal opinion, so let's move forward here.

Adapted #49 accordingly.


Raise exception in assertion processing

Purpose or Intent
If an attribute is not found during assertion processing raise an exception instead of continuing silently
Links

https://bugzilla.redhat.com/show_bug.cgi?id=1335669

Steps for Testing/QA

[Optional] Steps defined in bugzilla ticket

https://bugzilla.redhat.com/show_bug.cgi?id=1335669

Darga backport details:
commit 6d239f43bdff975e7a2e4d1a151ae8659396bdb0
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Mon Jul 11 19:55:08 2016 -0400

    Merge pull request #9737 from mkanoor/bugzilla_1335669

    Raise exception in assertion processing
    (cherry picked from commit 62adbc918476eb517e9a5a6e99568e337d002af6)


Translate source code to english

When the project started, I had no idea how long it would last so I didn't care much about naming things.
As a personal rule, all code that I made now must be written in english, so we need to translate everything in code (class, methods, package names, etc) from portuguese to english.

We can use transifex for that.
https://www.transifex.com
I just created an organization there for this project. Message me and I can add you as an administrator there.

Maybe you could consider creating subtasks for this issue or splitting this issue into 2?
I mean: one task is extracting all the current text into po files. The other task will be translate everything to English.

I wasn't planning to translate page content yet, as portuguese is the language for 99.999% of our visitors, so I don't think we need something like .po files at this moment.
What I want is to translate class names like Problema to Problem, or app name from contribuicao to contribution.

I see. Way simpler then.
Still, transitioning to po files should be pretty straightforward.

As we are rewriting most of the code during migration to Django 3.1, this issue will be accomplished, so there is no need to have it opened.


feat(dockerapp): add default readiness check for the docker file app

Summary of Changes
Add readiness check for the docker app container
Issue(s) that this PR Closes
Associated End To End Test PR(s)
Associated Documentation PR(s)
Associated Design Document(s)
Testing Instructions
Please provide a detailed list for how to test the changes in this PR.

Create a Deis Cluster
Register an docker file/deis pull app
It should work fine and you should see a readiness check when you describe the pod

Pull Request Hygiene TODOs
Please make sure the below checklist is complete.

[x] Your pull request is concise and to the point (make another PR for refactoring nearby code)
[x] Your commits are squashed into logical units of work
[x] Your commits follow the commit style guidelines

:cherry_blossom: Thank you! :cherry_blossom:

Current coverage is 82.54%

Merging #654 into master will decrease coverage by -0.13% as of 6bf0e62

@@            master    #654   diff @@
======================================
  Files           29      29
  Stmts         2465    2481    +16
  Branches       369     373     +4
  Methods          0       0
======================================
+ Hit           2038    2048    +10
  Partial        120     120
- Missed         307     313     +6

Review entire Coverage Diff as of 6bf0e62

Powered by Codecov. Updated on successful CI builds.


HTTP to HTTPS redirection for marathon service

Hello to All,
I've got a question about redirection of marathon traffic from http to https.
I've enabled ssl in marathon by providing --ssl_keystore_path and --ssl_keystore_password flags, but didn't provide --disable_http, because currently there are a lot of places in code that talk to marathon via http url and we can't change it now. So I would like to leave http and https working at the same time, but enable auto-redirect all requests which were made via http to https, so the clients will be automatically redirect to use the HTTPS and get connected using encrypted channel.
Could you please suggest how this can be configured or just point me to documentation where it's described?
Thanks.

Marathon does not support redirect HTTP o HTTPS. You can achieve it by setting up small server that will listen on HTTP and redirect all request to HTTPS.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4640. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


ValueError: A Concatenate layer requires inputs with matching shapes except for the concat axis. Got inputs shapes: [(None, 12, 12, 128), (None, 12, 12, 32), (None, 12, 12, 32), (None, 12, 12, 64)]

Using TensorFlow backend.
Saving cropped images in bin/output.
2021-01-05 14:45:26.134594: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2021-01-05 14:45:26.148613: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x7fc1f7241860 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2021-01-05 14:45:26.148674: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
Traceback (most recent call last):
File "bin/compare_faces.py", line 51, in
run(args.image_one, args.image_two, args.save_dest)
File "bin/compare_faces.py", line 29, in run
model = facenet_model(input_shape=(96, 96, 3))
File "/usr/local/anaconda3/lib/python3.8/site-packages/face_compare/model.py", line 217, in facenet_model
X = inception_block_4a(X)
File "/usr/local/anaconda3/lib/python3.8/site-packages/face_compare/model.py", line 62, in inception_block_4a
return concatenate([X_3x3, X_5x5, X_pool, X_1x1], axis=1)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/layers/merge.py", line 931, in concatenate
return Concatenate(axis=axis, **kwargs)(inputs)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 925, in call
return self._functional_construction_call(inputs, args, kwargs,
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 1098, in _functional_construction_call
self._maybe_build(inputs)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 2643, in _maybe_build
self.build(input_shapes)  # pylint:disable=not-callable
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/utils/tf_utils.py", line 323, in wrapper
output_shape = fn(instance, input_shape)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/layers/merge.py", line 519, in build
raise ValueError(err_msg)
ValueError: A Concatenate layer requires inputs with matching shapes except for the concat axis. Got inputs shapes: [(None, 12, 12, 128), (None, 12, 12, 32), (None, 12, 12, 32), (None, 12, 12, 64)]Using TensorFlow backend.
Saving cropped images in bin/output.
2021-01-05 14:45:26.134594: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2021-01-05 14:45:26.148613: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x7fc1f7241860 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2021-01-05 14:45:26.148674: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
Traceback (most recent call last):
File "bin/compare_faces.py", line 51, in
run(args.image_one, args.image_two, args.save_dest)
File "bin/compare_faces.py", line 29, in run
model = facenet_model(input_shape=(96, 96, 3))
File "/usr/local/anaconda3/lib/python3.8/site-packages/face_compare/model.py", line 217, in facenet_model
X = inception_block_4a(X)
File "/usr/local/anaconda3/lib/python3.8/site-packages/face_compare/model.py", line 62, in inception_block_4a
return concatenate([X_3x3, X_5x5, X_pool, X_1x1], axis=1)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/layers/merge.py", line 931, in concatenate
return Concatenate(axis=axis, **kwargs)(inputs)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 925, in call
return self._functional_construction_call(inputs, args, kwargs,
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 1098, in _functional_construction_call
self._maybe_build(inputs)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 2643, in _maybe_build
self.build(input_shapes)  # pylint:disable=not-callable
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/utils/tf_utils.py", line 323, in wrapper
output_shape = fn(instance, input_shape)
File "/usr/local/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/layers/merge.py", line 519, in build
raise ValueError(err_msg)
ValueError: A Concatenate layer requires inputs with matching shapes except for the concat axis. Got inputs shapes: [(None, 12, 12, 128), (None, 12, 12, 32), (None, 12, 12, 32), (None, 12, 12, 64)]

@filzahamjad, Sorry for the late response. Is this still an issue for you?
Does this reference help you? If not, can you please share a simple standalone code to reproduce the issue? Thanks!