Handle non-initialized image embeddings better in the plugins

Now that we have the embedding widget integrated in the napari plugin, it is possible to click a button that requires the model and image embeddings without having initialized them.
This fails with a cryptic error message:
File ~/Work/my_projects/micro-sam/micro_sam/sam_annotator/_widgets.py:229, in segment_widget(viewer=Viewer(camera=Camera(center=(0.0, 255.5, 255.5),....<locals>._clear_annotations at 0x7f7471efc4a0>}), box_extension=0.1)
    228 image_embeddings = AnnotatorState().image_embeddings
--> 229 if image_embeddings["original_size"] is None:  # tiled prediction
        image_embeddings = None
    230     seg = vutil.prompt_segmentation(
    231         predictor, points, labels, boxes, masks, shape, image_embeddings=image_embeddings,
    232         multiple_box_prompts=True, box_extension=box_extension,
    233     )

TypeError: 'NoneType' object is not subscriptable

It would be good to throw a more meaningful error message instead.
We could also indicate visually in the embedding plugin if embeddings are initialized or not.
This could be done via a checkbox is_initialized or so  that is not writable from the plugin and just shows if the embeddings are computed or not. (Not sure how to do this exactly and if this is really a good idea.)

In the long run it would be good to handle this using validation functions in magicgui: https://github.com/pyapp-kit/magicgui/issues/634.
This will probably take a while before being implemented, so we should find an intermediate solution.

Partially related to this: storing the embedding widgets in the state to have access to it; update the values in the widget to match what is passed in an annotator function, see #483

This should also encompass handling image embeddings that are wrongly initialized (e.g. for a different image). See also #266

This is ready to be addressed now that #486 is merged. Relevant locations in the widget are
https://github.com/computational-cell-analytics/micro-sam/blob/dev/micro_sam/sam_annotator/_widgets.py#L617 and https://github.com/computational-cell-analytics/micro-sam/blob/dev/micro_sam/sam_annotator/_widgets.py#L643.

Implemented in #498


Library Request: scikit-garden

Scikit-garden or skgarden (pronounced as skarden) is a garden for scikit-learn compatible trees.
Test:
from sklearn.datasets import load_boston
boston = load_boston()
X, y = boston.data, boston.target

### Use MondrianForests for variance estimation
from skgarden import MondrianForestRegressor
mfr = MondrianForestRegressor()
mfr.fit(X, y)
y_mean, y_std = mfr.predict(X, return_std=True)

### Use QuantileForests for quantile estimation
from skgarden import RandomForestQuantileRegressor
rfqr = RandomForestQuantileRegressor(random_state=0)
rfqr.fit(X, y)
y_mean = rfqr.predict(X)
y_median = rfqr.predict(X, 50)

Gives us:

ImportError:
load_boston has been removed from scikit-learn since version 1.2.

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

scikit-garden does not seem to compile correctly (even using unreleased repo version) and does not seem to be maintained. Latest release is from 2017 ref https://pypi.org/project/scikit-garden/. Created issue https://github.com/scikit-garden/scikit-garden/issues/117


Fix doc link target 404

Proposed changes:

Fallback Policy should linked to /docs/core/fallbacks/

Status (please check what you already did):

[ ] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog

Thanks for submitting this PR, we'll review it as soon as possible!


Made Live today by mistake?

Today at approximately 6:30pm - one of our home's users was browsing the web and went to "www.harveynorman.co.nz" and had a message pop up saying "Sorry - we don't support sales to the EU - please use the EU version of our website instead".
This stumped us a bit - as we're in NZ and had never seen anything like this before. On running a few "what is my ip" type websites - our IPv4 address was our typical ISP IP and was located from New Zealand as expected.
However - the ones that reported on IPv6 noted that there was an IPv6 address coming from the 2001:4860::/32 range - which was owned by Google LLC and was a "Shared Services Range" noted as coming from Europe.
This stumped us further as we could'nt figure where the IPv6 address came from - as we actually have IPv6 turned off on the home router.
On testing other PC's in the house using Chrome - they start exhibiting the same issue, however non-chrome browsers did not. After around 10 minutes, the behavior reverted and all PC's started once again reporting "No IPv6 address" when checking the tester sites.
After ALOT of digging around on Chrome Features, Proxying, etc - I hit this particular feature in development that seemed to match exactly what we saw - however it's in development and shouldn't be active right? - so why today did we see this behaviour.
If this is the case - I would deem this a total security and privacy risk - as I have no idea where our data via chrome was proxied to for that period - and was not made aware this was ocurring.
I'm aware chrome is able to remotely enable experimental features - we hit this a few years ago with something that was enabled that impacted some of the customers I was working for at the time - so I am suspecting the same thing has occurred here.
Currently running Chrome Official/Stable 120.0.06099.225.

Unfortunately we didn't get a shot of the page showing both our ISP IPv4 and the Google IPv6 address - but this is a shot of the IPv6 address that was apparently being used at the time.

Thanks for the report @Belzemus - what you're describing is indeed surprising. If you or another user are able to reproduce still, and are willing to help us debug - capturing a netlog would be incredibly useful: https://www.chromium.org/for-testers/providing-network-details/. If that's possible, you could email it to<EMAIL_ADDRESS>and our team can start to investigate.

If it occurs again we will to the netlog capture and let you know, however we can't reproduce it outside of that short 10 minute window yesterday.
Essentially when the website flagged "your in the wrong country" - we thought something was odd, as we started to investigate what was actually wrong - we noticed the IPv6 address showing up on the "What is my IP" type websites, but within 10 minutes it had stopped occurring and was reporting "No IPv6" again as we'd usually expect.
Occurred on at least two machines in the house as noted, but only within Chrome - Edge and Opera were not reporting the IPv6 Address for the brief time.
Given it was so short - by the time we had a rough guess as to what was happening it was over - then spent the next hour or so digging around articles/etc to try figure out what it might have been - only knowing it was something in Chrome either Proxying or Tunnelling Traffic back to Google. Which I then stumbled on the initial development article around this feature and it led me here.

@miketaylr Is there a way to enable this feature so we can test our services?

@miketaylr Is there a way to enable this feature so we can test our services?

Not currently, no. Once we have testing instructions, we'll be sure to publish them here in the explainer.

I think we can close this issue as unrelated to our IP Protection feature (which remains unlaunched). We suspect there was a failure in a different prefetch proxy feature in Chrome, which should be fixed by now. Thanks for the report, however.


docs(fetch): fix example code

Thank you for all the contributions, I really appreciate fixing all the minor overseen issues.


Add initial feature list to README.md

Add feature list to README
Spin up initial CONTRIBUTING.md

Coverage remained the same at 99.731% when pulling 1e5cf065899ae64b55741fc5b285ef554633cc63 on CyberiaResurrection:AddFeatureListToReadme into 7ff8b7a51f0840250776af2c712cf5a1e6851f75 on Algo-Web:master.

Coverage remained the same at 99.731% when pulling 1e5cf065899ae64b55741fc5b285ef554633cc63 on CyberiaResurrection:AddFeatureListToReadme into 7ff8b7a51f0840250776af2c712cf5a1e6851f75 on Algo-Web:master.

Coverage increased (+0.002%) to 99.733% when pulling 613711f3b1758bd56099f0fbc889e1a997b43e70 on CyberiaResurrection:AddFeatureListToReadme into 7ff8b7a51f0840250776af2c712cf5a1e6851f75 on Algo-Web:master.


Document Network Policies and practices

Describe the task
As part of the technical documentation for the app, the team has been asked to outline the use of KNPs and what are the allowed communication paths between the different pieces of the app.
Acceptance Criteria

[ ] a .md file in the docs folder
[ ] List of recommendations

Definition of Done

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

Some information has already been sent to MISO. Let's wait to hear back from them on what else might be required.


[WIP] Stash dev env

Summary

Description and Background

Related Issue

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the PSR coding guideline.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

predecessor of #27


Casing discrepancy in param properties

Many, for instance, Add-History:
In the Param section under InputObject, for instance, Required and Accept wildcard character have a value of True or False (note the caps). Accepts pipeline input begins with lc t (“true”)
These are the fields that appear after the param name in all topics:
Type: PSObject[]
Parameter Sets: (All)
Aliases:
Required: False
Position: 1
Default value:
Accept pipeline input: true (ByValue)
Accept wildcard characters: False
These are strings in the underlying maml. (Type refers to the .Net type, so it is fine. It always appears as far as I can tell, as the correct name correctly capped.)
It looks like the markdown simply displays the casing as it appears in the maml. Is it possible to impose casing consistency during the round tripping?
This change should to be in PlatyPS. This shouldn’t be a one-way operation on the output of the maml, because there will be changes to param sets. I don’t know who might update the markdown, but whoever does, I still want casing consistency.

I noticed this inconsistency. It comes from existing MAML doc.
We indeed can fix it. What's the correct version? First Capital letter?

I think I eliminated all occurrences. The pull request to master is in #134.
@TimShererAtAquent  @sankethka

Fxied per #134


Support PBXAggregateTarget

It seems that https://github.com/swift-xcode/xcodeproj supports PBXAggregateTarget (?), but XcodeGen doesn't.
Priority would be low though, I often use the kind of target in framework projects to invoke archive script.

I want this feature. but it is difficult for me to implement. 😭

How would you guys imagine this feature looking in yaml?
Would something like this work?
A new top level aggregateTargets, that has a map of target name to the following structure:

targets: [String]
prebuildScripts
postBuildScripts

Is that all that would be needed? Do you need to be able to set build settings on an aggregate target?

We mostly using aggregateTarget to execute scripts. For instance as global Prepare target set as dependency for any other target. Just to avoid setting pre-build scripts in every target.

Would target inheriting solve the same problem as you could easily apply a build script to all targets? https://github.com/yonaskolb/XcodeGen/issues/285

Ideally the build script is run once though.
Sent with GitHawk

Got it

Aggregate targets added here #354, but with a strange issue


'Try Now' button takes you to login page

I know this is probably technically working as expected but the 'Try Now' button should really take you to a demo account in order to illustrate how the product works rather than ask you to put faith in the product and sign up immediately.
I'm generally not going to sign up for yet another account just to see if it something I want to use.

I do think this is something I can add in the future.
You can just use something like Temp Mail or the Apple hide my email feature if you don't want to give your email address for now.

FWIW, I just tried multiple temporary email services, and none of them received the sign up email.


Add Spline interpolation

Fix #577.
@Rombur @dalg24 Looking for quick early feedback to see if there are any parts that have to be significantly changed.

Does this work on GPU?

@Rombur It does. Tpetra uses CudaUVM, so it worked on GPU even without switching to Kokkos kernels. I'm still unsure if we need CUDA_LAUNCH_BLOCKING=1, seems to work without.

Container has to be rebuilt and reuploaded, as we don't build it from the Dockerfile in .jenkins. I think currently Dockerfile has some leftovers from @masterleinad. @masterleinad Can you please clean it up?

The only thing that remains to be done here (from my pov), is to add the comparison example between MLS and Spline. I don't think it's worth kokkosifying the setup phase at this point (though, we may need to do this, depending on the cost).

For me, it's not necessary to add the comparison here but I agree that we don't need to optimize the setup here for now.

For me, it's not necessary to add the comparison here but I agree that we don't need to optimize the setup here for now.

That's fine with me.

@Rombur @dalg24 Please review the latest version.


Update managed scenario

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: mervey45:x: Merve TemelMerve Temel seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update to node 10 in .travis.yml

Fixes # .
Proposed changes:

Coverage remained the same at 83.398% when pulling fbab116a690bc43d604e382c5843daa34b5e91c0 on greenkeeper/update-to-node-10 into df73d9bfe4ea7da6420d18ed5fc856b5ab17f6e8 on master.


Create codeql-analysis.yaml

Add SAST
Pull Request
Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] pre-commit was run locally and any changes were pushed
[ ] test/test.sh has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue URL:
What is the new behavior?

Add SAST scanning with codeQL

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

CodeQL does not know how to scan a composite action Docker/bash repo.


Made chapter and title endpoint outputs as JSON in models.py

Closed due to my edits not being pushed into github.


Collation issue

I ran into an issue where I wanted to fetch collation applied on all columns in a table.
Is there any way we can fetch the collation applied on the columns through this gem?
In sqlite, we can achieve using following command:
.schema users

which returns
CREATE TABLE "users" ("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "password_digest" varchar COLLATE "NOCASE", "price" integer, "name" varchar, "created_at" datetime, "updated_at" datetime);

Can we find some any way to identify the collation applied on column?

Solution:
In addition to .schema we can check the definition of any table in sqlite_master and sqlite_temp_master. Hence you can use following code to find the full definition of the table:
table_name = 'table-name'
sql   = "SELECT sql FROM
              (SELECT * FROM sqlite_master UNION ALL
               SELECT * FROM sqlite_temp_master)
               WHERE type='table' and name='#{ table_name }' \;"

SQLite3::Database.new('data.db') do |db|
  db.execute(sql) do |row|
    p row
  end
end


nl: move from getopts to clap

Code is in:
https://github.com/uutils/coreutils/blob/master/src/uu/nl/src/nl.rs
Tests:
https://github.com/uutils/coreutils/blob/master/tests/by-util/test_nl.rs
As example:
https://github.com/uutils/coreutils/commit/9081e120d6b866c94679d9e29c7bdb17ccccb4d0

Hi, I am new to rust and this project but I'd like to take a shot at this issue.

sure, please go ahead!

I am giving this issue a try

@sylvestre This can be closed now.

oui, merci :)


Feature: Augmenta v2 UE4.25

Updated:

The Demo Project to UE4.25.
Readme

Do Not Merge until the Augmenta Plugin v2 UE4.25 is Merged.


Dockerize Rails and Postgres for local development

@alexsoble I started hacking on this a bit and setting up my development environment.  I worked on two infrastructure-y pieces that I think might be helpful, which I'll submit as separate PRs.  Let me know what you think! :)
Instead of installing Postgres, etc. on my local laptop, I worked to add support for running the project in Docker containers for local development.  The only change to the existing setup is upping the Ruby version to 2.2.3, only because there isn't a supported Ruby container for 2.1.6.
This adds a Dockerfile that can run the Rails app in a container, and also adds a docker-compose.yml file that pulls in the standard Postgres container and connects the two together.  So for workflow, running docker-compose up starts ephemeral Rails and Postgres containers that are linked together, and hitting http://docker:3000 works just like it does when running everything locally.  Possibly be useful for production deployments at some point.

@kevinrobinson: If it makes it easier to fork & start hacking on Student Insights I'm all for it!
I pulled down this branch and started working on setting up local Docker development. Wasn't able to get it running the first try but will keep at it.
About the production idea — I think it's worth exploring. Crucial input will come from the Somerville schools IT team, John Breslin and Bryant Brown, since they will be managing app deployment.

@alex Cool, sounds good!  If you want to paste any problems you had in the setup I can maybe help debug.
FWIW, I've been hacking on manually getting the project running on containers in EC2, and this is working albeit with a bunch of unfinished pieces, so we can chat Friday and see where it's at.  Again, that's mostly from just me experimenting and I'm definitely not suggesting it's worth spending time on trying to do for realz without input for John and Bryant.  More just sharing some side-hacking. :)

@kevinrobinson Sounds good!

Added a commit with notes about using docker-machine-nfs, see https://github.com/codeforamerica/somerville-teacher-tool/pull/336#issuecomment-158441877 for more info.

Got this running locally on my machine — fantastic improvement! The docker flow looks like a far more streamlined way for other developers to set up the project and contribute.
Thanks @kevinrobinson!

no prob!


Use type system to ensure only Folder kind has children

At the moment any node type is allowed to have children (self.child_indices for Node and self.merged_children for MergedNode).
We should instead define enums for these types so we only have children for the Folder case.

Definitely! Node and MergedNode feel like they want to be enums, and kind and is_folder() are redundant.


Add install docs

Pull Request Test Coverage Report for Build 1096

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 90.828%

Files with Coverage Reduction
New Missed Lines
%

storage/postgres/storage.go
4
85.92%

Totals

Change from base Build 1091:
-0.2%

Covered Lines:
2040

Relevant Lines:
2246

💛  - Coveralls


Fix connection error when running dev environment

Prior to this change whenever you started the docker-compose
you get Connection Error from store-front service since there was an error
in the param --host leading the webpack dev server to only open port for
the loopback interface of the container.
Closes #106

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[cryptotest] Wrong filename in $id field in ECDSA test vector JSON schema file

Description
The JSON schema file for the parsed ECDSA test vectors, //sw/host/cryptotest/testvectors/data/schemas/ecdsa_sig_ver_schema.json has $id field on line 3 set to the incorrect filename, presumably because of a copy-paste error.
Should change to "$id": "https://raw.githubusercontent.com/lowRISC/opentitan/master/sw/host/cryptotest/testvectors/data/schemas/ecdsa_sig_ver_schema.json",
CC: @milesdai

This issue is addressed in #21110 .


Fix a broken link to travis ci

Fix a broken link to travis ci

Coverage remained the same at 71.707% when pulling 692ae92bab91cfd6081d67a43421e3c752e36b60 on muupan-patch-1 into e84ea5f89f8e293969eee621baf7a512b6e95b42 on master.

LGTM


Is it compatible with hide my wp?

If WP-Rocket works with this plugin, there is no reason why it would not work.
That said, I have not tested it.


Remove all waypoints bound to a specific element outside the waypoint itself

Hej!
I use a loop to create multiple waypoints to every element with a specific class. Works like a charm, but now I need a function to delete all waypoints bound to on of those DOM-elements.
I can select the element, but can't select the individual wapoints, because they are created automatically within a loop and aren't stored in individual variables. I don't want to delete all waypoints from the site either.
Is there a way to delete all waypoints bound to a specific DOM-element?
Can't find anything in the Docs...
Thanks in advance,
Lucas

I'm using a helper function that sifts through all Waypoint instances and finds the ones that belong to my elements. Here's the gist


fix synthetic size for (last_record_lsn - gc_horizon) < initdb_lsn

Assume a single-timeline project.
If the gc_horizon covers all WAL (last_record_lsn < gc_horizon) but we have written more data than just initdb, the synthetic size calculation worker needs to calculate the logical size at LSN initdb_lsn (Segment BranchStart).
Before this patch, that calculation would incorrectly return the initial logical size calculation result that we cache in the Timeline::initial_logical_size. Presumably because there was confusion around initdb_lsn vs initial size calculation.
The fix is to only hand out the initialized_size() only if the LSN matches.
I don't understand the if size != 0 business, so, I left that untouched.

Hm, I think with this PR, we will count the initial size calculations towards the wrong histogram, i.e.,  we would count them towards logical_size_histo instead of init_logical_size_histo.
When we talked this over, we were focussing solely on the caching role of let timer = if ... else ;
But it actually has two roles:

selection of correct histogram

if it's initial size calculation, usage of Timeline::current_logical_size as a cache.

I'm wondering what is the practical hit ratio. Given that it was comparing up_to_lsn == self.initdb_lsn, instead of up_to_lsn == self.current_logical_size.initial_part_end, I think it will have had a very low hit ratio.

I'm wondering if anyone actually cares about making the distinction between initial logical size calculation and logical size calculation triggered by synthetic size.

Here's the increases by hour of the two histograms:

So, the else branch clearly dominates the misses.
But we have no metrics that would allow us to figure out how many hits we have.

Hm, spelunking through GitHub history reveals @koivunej already knew about this bug.
Here's his draft PR:
https://github.com/neondatabase/neon/pull/3594/files
His solution still uses the self.initdb_lsn in some cases, and I don't get why, so, let's wait out his response.
My take on all of this is:

Have a single histogram. I don't think the distinction is worth it. As shown above, the number of calculations with operation=logical size" clearly dominates the number with operation="initial logical size" .
Can we remove the caching inside Timeline::calculate_logical_size? We need a non-caching variant for https://github.com/neondatabase/neon/pull/3871 anyways. So, can the callers peek into Timeline::current_logical_size instead?

The #3594 is left as a draft to focus on feature work, no one else saw a bug here. After tests were requested I thought it would be better to refactor the current code after adding tests. The size != 0 is probably me just being extra defensive since I didn't have full understanding of the calculation, which has since been changed.
(Maybe it was even in some pre-ondemand-download version where on failure to set the size we'd cache 0. Or at least, that was my mental model that it could happen.)

To solve this:

Can we remove the caching inside Timeline::calculate_logical_size? We need a non-caching variant for https://github.com/neondatabase/neon/pull/3871 anyways. So, can the callers peek into Timeline::current_logical_size instead?

I guess. get_current_logical_size_non_incremental is also a possibility.


Duplicate configuration in config.yaml

Currently, we need to define the following for the all machines:
machines:
  - id: m1
    enabled: !!bool true
    type: l99.driver.fanuc.FanucMachine, fanuc
    strategy_type: l99.driver.fanuc.collectors.UseCase01, fanuc
    handler_type: l99.driver.fanuc.handlers.Native, fanuc
    sweep_ms: !!int 1000
    net_ip: <IP_ADDRESS>
    net_port: !!int 8193
    net_timeout_s: !!int 10
    broker:
      enabled: !!bool true
      net_ip: localhost
      net_port: !!int 1883
      auto_connect: !!bool true
      publish_status: !!bool true
      publish_arrivals: !!bool false
      publish_changes: !!bool true
      publish_disco: !!bool true
      disco_base_topic: fanuc

It is good if we connect to a single machine, but what about (e.g.) 10 machines? I think that the broker configuration should be put outside of the machines object, like so:
broker:
  enabled: !!bool true
  net_ip: localhost
  net_port: !!int 1883
  auto_connect: !!bool true
  publish_status: !!bool true
  publish_arrivals: !!bool false
  publish_changes: !!bool true
  publish_disco: !!bool true
  disco_base_topic: fanuc
machines:
  - id: m1
    enabled: !!bool true
    type: l99.driver.fanuc.FanucMachine, fanuc
    strategy_type: l99.driver.fanuc.collectors.UseCase01, fanuc
    handler_type: l99.driver.fanuc.handlers.Native, fanuc
    sweep_ms: !!int 1000
    net_ip: <IP_ADDRESS>
    net_port: !!int 8193
    net_timeout_s: !!int 10

It might be also a good idea to allow multiple configuration or even watching for them.
Let me explain my use case: I need to be able toggle monitoring on and off for every machine individually. Currently, I need to update the config.yaml (add or toggle enabled / remove / comment out the particular machine in config.yaml) and restart fanuc-driver.
It might be a good idea to have a folder for config files where one file would be used for the broker configuration and there would be also one file per machine. fanuc-driver would watch those files, e.g. when a machine is added, it would start its monitoring, or when a config of a particular machine is removed OR enabled set to false, fanuc-driver would stop getting data from that particular machine.
What do you think?

Configuration is stable and using YAML anchors, parts can be modified without duplication.


Fixed Language Support

I've been using this app in the English Language, but I found that localization has been done from English to Spanish Even in Localizable.strings(English)

👍


Proxy not being used on Linux/Mac under 6.1.0

With the release of 6.1.0, it appears to not always use a proxy on Linux or OSX, specifically when trying to run "Connect-AzureRmAccount". If I downgrade to 6.0.4 on Linux or OSX, it works. On Windows, the Connect-AzureRmAccount correctly uses the system defined proxy.
I'm assuming it's a PowerShell 6.1.0 issue and not a AzureRM.Netcore issue.
The version of AzureRM.Netcore is current (0.13.1)
Running tcpdump in the background, I can see that under powershell 6.1.0 it only tries to open direct sockets to port 443 out on the internet; under powershell 6.0.4 it instead correctly talks to my corporate proxy. The proxy is defined via the environment variables HTTP_PROXY and HTTPS_PROXY.
If I run "Invoke-WebRequest" instead, under 6.0.4, it picks up the HTTPS_PROXY env var and uses it without needing a -Proxy argument. Under 6.1.0 it doesn't use the env vars, and requires the -Proxy argument.
I've tested and reproduced this on both Linux (CentOS 7.5) and OSX (10.13.6).
Steps to reproduce
[simon@simon-base7-01 (devkvm) ~]$ export HTTP_PROXY=http://webproxy:3128/
[simon@simon-base7-01 (devkvm) ~]$ export HTTPS_PROXY=http://webproxy:3128/
[simon@simon-base7-01 (devkvm) ~]$ pwsh
PowerShell v6.0.4
Copyright (c) Microsoft Corporation. All rights reserved.

https://aka.ms/pscore6-docs
Type 'help' to get help.

PS /home/simon> Connect-AzureRmAccount
WARNING: To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code XXXX to authenticate.
^C[simon@simon-base7-01 (devkvm) ~]$ sudo rpm -U /home/simon/powershell-6.1.0-1.rhel.7.x86_64.rpm
[simon@simon-base7-01 (devkvm) ~]$ pwsh
PowerShell 6.1.0
Copyright (c) Microsoft Corporation. All rights reserved.

https://aka.ms/pscore6-docs
Type 'help' to get help.

PS /home/simon> Connect-AzureRmAccount
Connect-AzureRmAccount : Connection refused: Connection refused
At line:1 char:1
+ Connect-AzureRmAccount
+ ~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : CloseError: (:) [Connect-AzureRmAccount], AadAuthenticationFailedException
+ FullyQualifiedErrorId : Microsoft.Azure.Commands.Profile.ConnectAzureRmAccountCommand

PS /home/simon>

Expected behavior

Actual behavior

Environment data
PS /home/simon> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.1.0
PSEdition                      Core
GitCommitId                    6.1.0
OS                             Linux 3.10.0-693.17.1.el7.x86_64 #1 SMP Thu Jan 25 20:13:58 UTC 2018
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

PS /home/simon> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.0.4
PSEdition                      Core
GitCommitId                    v6.0.4
OS                             Linux 3.10.0-693.17.1.el7.x86_64 #1 SMP Thu Jan 25 20:13:58 UTC 2018
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@markekraus Could you please comment the Issue?

I'm not entirely sure, but this looks like a regression in corefx. dotnet/corefx#28088 changes the way the environment variables are handled. if I'm understanding this correctly, it looks like they will get incorrect URIs for HTTPS_PROXY. GetUriFromString() is striping the string of http:// but not https:// and then later they depend on : being at the end of the host.

ok.. looks like it is a regression but not the one I thought.
$assembly = [System.Net.Http.HttpClientHandler].Assembly
$class = $assembly.GetType('System.Net.Http.HttpEnvironmentProxy')
$method = $class.GetMethod('TryCreate')
$proxy = [System.Net.WebProxy]::new()
$env:HTTPS_PROXY = 'http://webproxy:3128/'
$method.Invoke($null, @($proxy))

Should be True but is False.
but, if you remove the trailing / from the proxy it works:
$assembly = [System.Net.Http.HttpClientHandler].Assembly
$class = $assembly.GetType('System.Net.Http.HttpEnvironmentProxy')
$method = $class.GetMethod('TryCreate')
$proxy = [System.Net.WebProxy]::new()
# No trailing slash
$env:HTTPS_PROXY = 'http://webproxy:3128'
$method.Invoke($null, @($proxy))

@ciranor as a work around, set your env vars without the trailing slash.
We will need to open an issue with corefx.

Created dotnet/corefx#32635

Yes, I can confirm that by removing the trailing slash, Invoke-WebRequest and Connect-AzureRMAccount both work under 6.1.0.
Cheers.

@markekraus Could you test with latest build? Currently we moved to .Net Core 2.1.5 and seems CoreFX fix should be there.
I can not run your repo script - $class = $assembly.GetType('System.Net.Http.HttpEnvironmentProxy') is empty.

@iSazonov System.Net.Http.HttpEnvironmentProxy is only available on Linux and macOS.
I was able to build the latest on ubuntu and confirmed it is fixed with the move to .NET Core 2.1.5


slide top bar

In course screen removed tab bar and create new sliding top tab bar

@eyatsenkoperpetio can you add a brief description of what this PR is accomplishing?
also, it'll be great if you can link the PR with an issue on the Roadmap that its addressing, thanks!

https://github.com/openedx/openedx-app-ios/assets/76485998/f3fb4b95-70f4-4ef2-911e-510fa4809578

Removed bottom tab bar and add top slide bar

@eyatsenkoperpetio it'll be great if you can add all the relevant info to the PR's description instead of separate comments.

In the navbar overhaul, when we moved the course menu bar (bottom bar) to the top, we actually fixed the course header (containing course name, organization, end date, etc.) for all the tabs and positioned the menu bar (top bar) below the course header view.
In this PR, the course menu bar simply moved to the top without changing the behavior or course header view. @moeez96 I'm not sure what path we will be following here to achieve the current behavior (edX app behavior) for the course dashboard.
cc @miankhalid @touchapp

Would it cause any kind of issues if we merged the menu bar before the change in the course header?

@moiz994 It won't cause any issues, but I'm not sure what the design team is thinking or how we will be going with the new design. I'm waiting for a design response on the attached issue to clarify the requirements.

Hey @saeedbashir can you show the updates you have made to the header? Perhaps make a similar video as @eyatsenkoperpetio ?

@touchapp The header is completely redesigned; for this PR, what we can do is we can use the existing header (course image and info) but make it consistent across all the tabs and move the tabs below the course image. Thoughts?
I'm attaching the video from the edX production app.
https://github.com/openedx/openedx-app-ios/assets/5606473/264173f9-b935-43e0-a7f2-1a2079986286

Tests failed because of timeout.
Re-runed pipeline, let's see.

Not sure what's happening here. Multiple folks have approved the PR but:

there's an active discussion going on in the linked issue [iOS] Move Course menu from bottom to top #154
plus there's this item that needs a response as well: Course tab bar from the bottom of the screen move to the top #181 (comment)

@touchapp @rnr
Code looks good to me, but can someone please react to Khalid's message?

@moiz994 @marcotuts what's the decision this? should we go ahead with these changes? or dismiss this PR?

@miankhalid as per the last design sync, the design for the course dashboard is still in progress and we could not make a final decision on the way forward for it. If this is a feature flag-based implementation, I'd say go ahead with merging it for now.


Add support for custom attributes to meet accessibility requirements

Changelog Entry
Added support for custom attributes in the markdown renderer.
Description
Modern screen readers require aria-label attributes to be present on links, but the current markdown-it configuration doesn't support aria-label attributes. Adding support for attributes will enable the use of various aria and custom attributes on links in the Bot configuration interface.
Specific Changes
Imported the markdown-it-attrs npm package and added it to the renderer's configuration.

Test failed
Error: expect(received).toBe(expected) // Object.is equality

Expected - 1
Received + 1
(505)503-4455

(505)503-4455

↵
at Object.<anonymous> (/home/vsts/work/1/s/packages/bundle/src/__tests__/renderMarkdown.spec.js:55:74)
at Object.asyncJestTest (/home/vsts/work/1/s/node_modules/jest-jasmine2/build/jasmineAsyncInstall.js:100:37)
at /home/vsts/work/1/s/node_modules/jest-jasmine2/build/queueRunner.js:43:12
at new Promise (<anonymous>)
at mapper (/home/vsts/work/1/s/node_modules/jest-jasmine2/build/queueRunner.js:26:19)
at /home/vsts/work/1/s/node_modules/jest-jasmine2/build/queueRunner.js:73:41
at runMicrotasks (<anonymous>)
at processTicksAndRejections (internal/process/task_queues.js:97:5)

Thanks @compulim! Fixed the issue and added a test.


Update Readme with latest Spec Change.

Update readme with the spec changes of the PR https://github.com/flowjs/flow.js/pull/268

No need of this now, so closing.


Verbatim HTML: does not copy apostrophe and quotes verbatim in

I have set the .withRawContent flag to parse Markdown with inlined HTML. I am also using the Github flavor. I then add the following to a Markdown file I have:

    Figure 1: Embedded scatter plot example via iFrame

But after processing the apostrophe ' is changed and I get this in the HTML file:

                   Figure 1: Embedded scatter plot example via iFrame

I also tried quotes and that was also changed. Is this an error or do I have to do some type of escaping?

Yes, script tags are not really supported in verbatim HTML. Interestingly you are the first to notice that even though verbatim HTML is supported since 2013... :-)
Adding support will require enhancements to both, the parser and the renderer, and it's not very high on my list of priorities, so it's not going to happen that soon, but I'll keep the ticket open and will tackle it at some point.
I guess one reason I never thought of it is that Markdown has this weird bastardised form of embedded HTML where text nodes can also contain Markdown (and that Markdown can contain HTML again, and so on).
And another is that JavaScript embedded in Markdown feels somewhat unusual anyway. Is there a specific need for you to embed JavaScript in Markdown? Couldn't you just keep it in separate files?

@jenshalm thanks for looking into this.

I'll keep the ticket open and will tackle it at some point.

Ok.

I guess one reason I never thought of it is that Markdown has this weird bastardised form of embedded HTML where text nodes can also contain Markdown (and that Markdown can contain HTML again, and so on).

I see. This explains a lot. Wonder how many people actually take advantage of that.

Is there a specific need for you to embed JavaScript in Markdown?

I want to use MDoc and Laika to generate Jupyter like notebooks and present results of data analytics work based only on the JVM (thinking of calling it Satern 8-)). No need for Python, pip, etc, if I don't use Python libs for modeling. I also want to use this to document software with code snippet examples.  This makes life much simpler for CI/CD - no need to install external software such as Ruby and Jekyll, no need to add these commands to the CI/CD script.
However, I need to show plots. I have 2 options: static images or dynamic plots. The dynamic option is more in line with Jupyter and Matplotlib's interactive backedend. After some experimentation I think plotly-scala is a better option because it basically has a Scala API for the very mature Plotly.js library (not many mature and well supported Scala libraries for plotting exist).

Couldn't you just keep it in separate files?

I may be able to do this but in around about way. To explain. Here is the MDoc snippet that processes data and generated the plot (as Javascript):
'''scala mdoc:silent
    import plotly._, layout._, Plotly._

    val labels = Seq("Banana", "Banano", "Grapefruit")
    val valuesA = labels.map(_ => util.Random.nextGaussian())
    val valuesB = labels.map(_ => 0.5 + util.Random.nextGaussian())

    val traces = Seq(
      Bar(labels, valuesA, name = "A"),
      Bar(labels, valuesB, name = "B")
    )
    val playout = Layout()
    val jsSnippet = Plotly.jsSnippet("sample2", traces, playout)
'''

This is placed in the Markdown file that is preprocessed prior to Laika. In this case silent means that the code is placed in the Markdown file as a code snippet (code fence) example (nothing is added to the output).
Next we add the following to the Markdown file:
'''scala mdoc:passthrough

val snippet1 = s"""

        Figure 4: Embedded bar plot example via a Plotly-Scala generated script

"""

println(snippet1)
'''

This spits out the HTML snippet1 content into the Markdown document. We have in effect generated a dynamic plot via embedded HTML using Javascript .
So now I have to find a way to write the HTML snippet to a file and link to that in the Markdown source. I also have to make sure that the file is written to the correct target directory so that Laika picks it up during the next phase.  I will now look at MDocs postmodifier to see if I can circumvent this issue.
In case you are curious here is the source for the above example (end of the file - for now).
Apologies for being so long winded but I think it explains the issue better.

Ah I see, the JS is generated, that explains it, thanks for describing your use case.
I think it makes sense to support script tags, even though you are the first on this planet who needs it. :-) If it would just require enhancing the HTML renderer it would be a trivial fix, maybe just one line, but sadly it also needs some tweaking of that messy parser for the weird HTML/Markdown mess, and that means it probably won't happen too soon.
If I find some time in the coming weeks I might have a look whether I can find a way to show you how to install support as an extension yourself, but no promises as I'm not sure it'll work (due to subtleties around parser precedence etc.)

No problem. Time permitting I will work on the MDoc postmodifier. If this works I will let you know and we can close this.

As suspected it's not possible to write a simple extension to support script tags. So I fixed it right in the core Markdown support which turned out to be easier than I thought.
As long as you are working with the 0.11 release, you would still need to try the workaround you though of, as this fix is also only on master.


Fix OnGraspStay() running all the time

OnGraspStay() always runs no matter you grasp or not. This will fix it.

Nice catch!


doc_id validation

if doc_id is there, use doc_id; else if _id is there, set it to doc_id, else goto 4.
convert doc_id to unicode then encode it to utf-8 (make sure it's always a string).
check if doc_id string is valid (the definition of valid is below). if not, encode it by base64 (user can still convert it back to original id).
content = raw_content if there's a raw_content, else content = stringify(json object). then doc_id = hash256(content).upper_case().

valid id:

starts with alphanumeric character.
max length is 190 (the max length on most system is 256, but base64 is not digest algorithm, 256 / 4 * 3 = 192).
valid character: alphanumeric A-Za-z0-9, underscore _, dash -.

@szeke @saggu

updated rules by @szeke :

if doc_id is there, use doc_id; else if _id is there, set it to doc_id, else goto 4.
convert doc_id to unicode then encode it to utf-8 (make sure it's always a string).
check if doc_id string is valid (the definition of valid is below). if not, encode it by base64 (user can still convert it back to original id). if the length of it > 255, goto 4, else exit.
content = raw_content if there's a raw_content, else content = stringify(json object). then doc_id = hash256(content).upper_case().

valid id:

max length is 255.
valid character: alphanumeric A-Za-z0-9, underscore _, dash  -.
exclude all OS reserved file names (for windows: CON, PRN, AUX, NUL, COM1, COM2, COM3, COM4, COM5, COM6, COM7, COM8, COM9, LPT1, LPT2, LPT3, LPT4, LPT5, LPT6, LPT7, LPT8 and LPT9).


Breadcrumbs

Description
Link path that appears below the global navigation on every page.
User need: As a user I understand where in the site I'm located and can easily navigate back up the navigation tree.
Behaviour
On mobile the breadcrumb will only show the previous level

On desktop the full breadcrumb will show

The bottom border extends to the edge of the page
Height: 44px
Notes
Currently these are incredibly long in places because the IA is poorly designed. The new location switcher has been A/B tested and will be removed from the breadcrumbs due to poor UX and accessibility. See revised design for the location switcher.

Decide if/ when to truncate a breadcrumb


implement (non-)volatile table modular circuits and e2e public io

Working items & Scope

[x] add module table circuit, e.g. DynVolatileRamTable for memory, NonVolatileRamCircuit for Register/Public IO/ProgramDataCircuit.
[x] separate program data into dedicate circuit, and keep memory volatile.
[x] prover/verifier protocol to evaluate public io
[x] evaluate DynVolatileRamTable table address on verifier succinctly
[x] register public_io and test e2e

succinctly address vector argument refer here: https://github.com/scroll-tech/ceno/issues/378#issuecomment-2436580306
Public IO Auxiliary will cover in next PR

Public IO Auxiliary will cover in next PR

@hero78119 Do you have a link to that PR, please?


Fixed a MissingMethodException by updating Microsoft.Kiota.Http.HttpClientLibrary

Hi @NikiforovAll, could you please take a look at this? There is an issue with the current version of Kiota, causing a MissingMethodException (KiotaClientFactory.CreateDefaultHandlers()) when using your library which I was able to resolve by updating the package references. See also https://github.com/microsoftgraph/msgraph-sdk-dotnet/issues/2533

I updated all Kiota packages to resolve another issue:
 Error while processing request.
      System.Exception: IAsyncParseNodeFactory factory is required
         at Microsoft.Kiota.Abstractions.Serialization.ParseNodeFactoryRegistry.GetFactory[T](String contentType)
         at Microsoft.Kiota.Abstractions.Serialization.ParseNodeFactoryRegistry.GetRootParseNodeAsync(String contentType, Stream content, CancellationToken cancellationToken)
         at Microsoft.Kiota.Http.HttpClientLibrary.HttpClientRequestAdapter.GetRootParseNode(HttpResponseMessage response, CancellationToken cancellationToken)
         at Microsoft.Kiota.Http.HttpClientLibrary.HttpClientRequestAdapter.SendCollectionAsync[ModelType](RequestInformation requestInfo, ParsableFactory`1 factory, Dictionary`2 errorMapping, CancellationToken cancellationToken)
         at Microsoft.Kiota.Http.HttpClientLibrary.HttpClientRequestAdapter.SendCollectionAsync[ModelType](RequestInformation requestInfo, ParsableFactory`1 factory, Dictionary`2 errorMapping, CancellationToken cancellationToken)
         at Keycloak.AuthServices.Sdk.Kiota.Admin.Admin.Realms.Item.Users.Item.Groups.GroupsRequestBuilder.GetAsync(Action`1 requestConfiguration, CancellationToken cancellationToken)

I used the exact same versions as referenced by the current Microsoft.Graph package (5.56.0) as those seem to work. I honestly don't understand what Microsoft were doing when they versioned those packages. Microsoft.Kiota.Http.HttpClientLibrary is already published in version 1.11 which not even their Graph library uses. I think sticking to these versions should be relatively safe but if you'd like I could test the newest versions as well.

I just realized that the version here mirrors the Keycloak version it's supposed to work with. 25.0.1 would be wrong in this case.

we could keep this at 25.0.0 and you could overwrite the published NuGet package, risky and difficult to maintain for depending projects
leave this version broken and advise people to upgrade when #129 is done
use 25.0.0-hotfix.1, this would be considered a pre-release version, users would have to update manually

Please let me know how (if) you would like to proceed

Could you please undo the changes to keycloak version? we can go with a prerelease version with some prefixes. the versioning strategy is not the best for this scenario

Could you please undo the changes to keycloak version? we can go with a prerelease version with some prefixes. the versioning strategy is not the best for this scenario
I think it is a <Suffix>hotfix.1<Suffix/> or <VersionSuffix>

Done. I had to use <VersionPrefix> instead of <Version> as the latter overwrites any pre- and suffixes


add oauth

[ ] authorization code grant https://discord.com/developers/docs/topics/oauth2#authorization-code-grant
[ ] implicit grant https://discord.com/developers/docs/topics/oauth2#implicit-grant
[ ] client credentials https://discord.com/developers/docs/topics/oauth2#client-credentials-grant
[ ] Bot Auth https://discord.com/developers/docs/topics/oauth2#bot-authorization-flow
[ ] Advanced Bot Auth https://discord.com/developers/docs/topics/oauth2#advanced-bot-authorization
[ ] Webhook Auth https://discord.com/developers/docs/topics/oauth2#webhooks

In order to ensure HTTP Content Type Headers are correct, we must make sure that

[ ] Redirects are handled correctly.
[ ] Requests with files include subpart headers. https://discord.com/developers/docs/reference#uploading-files
[ ] Requests that upload image data detect the correct content type (i.e image/jpeg, image/png, image/gif for avatar_url)


feat(dataverse)!: rework and enhance vc stored context

Closes #615
Details
Brings the changes proposed in #615 by making the difference between VC's body and context (i.e. considered as headers in the RDF), and enhance this context with block information (i.e. height, timestamp & tx_index).
Those changes are considered breaking as it breaks the model stored in the cognitarium.

size-limit report 📦

Path
Size

target/wasm32-unknown-unknown/release/axone_objectarium.wasm
458.07 KB (0%)

target/wasm32-unknown-unknown/release/axone_law_stone.wasm
648.51 KB (0%)

target/wasm32-unknown-unknown/release/axone_cognitarium.wasm
831.35 KB (0%)

target/wasm32-unknown-unknown/release/axone_dataverse.wasm
672.44 KB (+0.31% 🔺)


feat(ft): add latex block comments

This PR adds support for block comments in LaTeX. Generally block comments are implemented like the C preprocessor directives for block comments, see. Even treesitter recognizes them as comments, see image.

I have not seen similar comment styles in repo, but I think they sure can be useful. Feel free to close this PR if it is fitted.
I have tested the changes in both 10.1 and nightly both on fedora 40 and windows, I don't think there is any issues, but I am willing to address any that pop up.
Thanks for the awesome plugin and the work you have devoted! :)

Wouldn't \begin/end{comment} be more modern?


bind generic type error

Bug Report
🕗 Version & Regression Information

This is a crash
This changed between versions 3.3.3 and  4.5.4

⏯ Playground Link
Playground link with relevant code
💻 Code

type Store<T> = {
  state: T
}

function foo<T>(this:Store<T>){
    console.log(this.state)
}

function bar<T>(initialState:T){
    const store:Store<T> = {
        state:initialState
    }
    return foo.bind<T>(store)
}

bar({a:1})()

There is currently no way to propagate type parameters, so bind, call, and apply don't work well with generic functions.

Also please fill out the issue template more accurately in the future. This is not a crash


Update stable version in repo from 1.2.3 to 1.3.1

Think about update stable version to 1.3.1
Version: stable=1.2.3 (137 days old) => latest=1.3.1 (15 days old)
Installs: stable=981 (65.18%), latest=342 (22.72%), total=1505
Click to edit

Pull into stable requested.


googletest: 1.10.9000-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository googletest to 1.10.9000-1:

upstream repository: https://github.com/ament/googletest.git
release repository: https://github.com/ros2-gbp/googletest-release.git
distro file: rolling/distribution.yaml
bloom version: 0.9.8
previous version for package: 1.8.9000-2

FYI @audrow


Contributing guide?

A lot of the other project have something like this. What people should do to contribute to the project. Like writing tests and checking issues with certain labels.
Would you guys mind creating something like this?
Or maybe I missed it for EOS and someone can point me to it?

There are no plans currently to make a contributing guide.


[chore]: Adding a well detailed pull request template for subsequent PRs

Overview
Every pull request before being being created should have a brief description about the entirety of the what the PR entails:

What the PR does?
Summarize the main tasks handled by the PR
How should it be manually tested?
Any background context that's needed (e.g. an API KEY etc)
The related issue this PR is reflecting if any
Screenshots (where appropriate)

I can work on this ASAP if approved


Visualize numbers of contacts

Feature
I’d like to be able to see in the app how many contacts my app/phone has been in contact with.
Problem and motivation

Is this something you're interested in working on

@daniel-eder This issue is most likely a duplicate of https://github.com/corona-warn-app/cwa-app-android/issues/689 and conseuently a duplicate of https://github.com/corona-warn-app/cwa-wishlist/issues/5

Thank you @ironjan that is indeed a duplicate. I will close this issue, please refer to corona-warn-app/cwa-wishlist#5 for a discussion of this topic.


Refactor Aliases.jsx.

Refactored Aliases.jsx from class-based component to functional-based component.

@FreakForever please read and follow Code quality. You should run all of these checks locally to ensure they pass before creating a pull request. Then you can compare your results against the GitHub workflows. As a bonus, fewer CI runs means consuming less electricity.

@plone/volto-team this is ready for technical review.


[Question]: Using an LLM hosted using LocalAI to query documents

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Hi, I am trying to use a phi-2.Q2_k model hosted on my local machine with LocalAI to query documents. I have written below code for that:
from llama_index.llms.openai import OpenAI
import os

base_path = os.environ.get("OPENAI_API_BASE", "http://localhost:8080/v1/")
documents = SimpleDirectoryReader("resources").load_data(show_progress=True)

# define LLM
model = OpenAI(temperature=0.1, model_name="phi-2.Q2_k", openai_api_base=base_path)

# build index
index = VectorStoreIndex.from_documents(documents, llm=model)

# get response from query
query_engine = index.as_query_engine()
response = query_engine.query("some question for the doc")

print(response)

I'm getting an empty response as a result. When I checked, the locally hosted model is not getting the request at all. If I try to use some OpenAI model like "gpt-3.5-turbo" then I get the right response. But I want to use locally hosted models. Any help if appreciated.

🤖
Hey there, @ManishAradwad! Great to see you diving into another challenge. Looking forward to tackling this one with you.
Based on the code you've shared, it seems like you're using the OpenAI class to initialize your model. However, if you're using a locally hosted model with LocalAI, you should be using the LocalAI class instead. The OpenAI class is designed to interact with models hosted on OpenAI's servers, not locally hosted models.
Here's how you can modify your code to use the LocalAI class:
from llama_index.core import VectorStoreIndex, SimpleDirectoryReader
from llama_index.llms.localai import LocalAI
import os

base_path = os.environ.get("OPENAI_API_BASE", "http://localhost:8080/v1/")
documents = SimpleDirectoryReader("resources").load_data(show_progress=True)

# define LLM
model = LocalAI(temperature=0.1, model_name="phi-2.Q2_k", api_base=base_path)

# build index
index = VectorStoreIndex.from_documents(documents, llm=model)

# get response from query
query_engine = index.as_query_engine()
response = query_engine.query("some question for the doc")

print(response)

In this code, I've replaced OpenAI with LocalAI. This should allow your code to interact with your locally hosted model.
If you're still getting an empty response after making this change, it could be due to one of the following reasons:

Your model is not hosted at the URL specified in api_base. Make sure that your model is hosted at the correct URL and that the URL is correctly specified in your code.
Your model is not configured to accept requests on the /v1 endpoint. Check the endpoints your model is configured to accept requests on and update the api_base parameter accordingly.
The globally_use_chat_completions parameter is not set correctly. This parameter determines whether to use the /chat/completions or /completions endpoint for API calls. If it's not set, the _is_chat_model property will raise a NotImplementedError. You can set this parameter to True or False when initializing the LocalAI class, or pass the use_chat_completions argument in your query.

If none of these solutions work, please provide more information about your setup, such as the version of LlamaIndex you're using, the configuration of your locally hosted model, and any error messages you're seeing. This will help me better understand the issue and provide a more accurate solution.
Sources

llama-index-integrations/llms/llama-index-llms-localai/llama_index/llms/localai/base.py
llama-index-core/llama_index/core/llms/custom.py

About Dosu
This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
If you want to continue the conversation, start your reply with @dosu-bot.

ModuleNotFoundError: No module named 'llama_index.llms.localai'

Hi @ManishAradwad
You'll have to install the package for localAI first: pip install llama-index-llms-localai

Thanks @anoopshrma!
It's working now. But I have a confusion. I'm getting the response, but not sure if it's using the LocalAI model. I saw the hosted mode wasn't getting queried.
So I did some tests. I got the response even if I don't have a model running on http://localhost:8080/v1/. I checked this by even changing the name of the model to some random name. It still gives me the response. Is it still using openai model??

Do you have openai key saved in your code somewhere? Ideally it should only call your local ai instance!

I have it as an env variable

Yes remove OPENAI variable and then try

It's happening even after removing the api key. here's the complete code I'm using:
from llama_index.core import VectorStoreIndex, SimpleDirectoryReader
from llama_index.llms.localai import LocalAI
import os

base_path = os.environ.get("OPENAI_API_BASE", "http://localhost:8080/v1/")
documents = SimpleDirectoryReader("resources").load_data(show_progress=True)

# define LLM
model = LocalAI(temperature=0.1, model="phi-2.Q_k", api_base=base_path)

# build index
index = VectorStoreIndex.from_documents(documents, llm=model)
# index.storage_context.persist(persist_dir="./storage")

# get response from query
query_engine = index.as_query_engine()
response = query_engine.query(
    "question"
)
print(response)


[Feature Request]: Ability to save maps so users can edit those same maps

Is your feature request related to a problem? Please describe.
There are times I want to edit a map that I've made. So we need ability to edit maps.
Describe the solution you'd like
Create ability to save maps. This also means we will need a table to view all produced maps with ability to edit, duplicate, delete maps.

@mjp92067 moving this to the sftp service as well


Add owner id to resource container

Reason for change
We need to be able to display resource details when viewing the resource in a team context.

Testing

The below requires changes to the Render dashboard in order to pass through the owner ID. Render also makes a query that needs to pass this along, so that also needs to be adjusted.

Make sure you're part of a team
Log into Render
Switch to your team context
Go to the Addons tab
If any resources exist, click on them
If a resource doesn't exist, create one, and then click on it in the Addons page
The resource should display successfully without errors

Checklist

[ ] CHANGELOG: The Unreleased section of CHANGELOG was updated
[ ] Prop changes: docs were updated
[ ] Prop changes: E2E tests were updated, testing from the highest level possible
[ ] Platform testing: If this change should be tested against a platform, has it been?

I think this is working. I can view /addons now without it spinning. But I’m having trouble creating a resource on stage.

@sslotsky could you update the CHANGELOG doc?


⚠ PrivateHD has degraded performance

In d05dcb4, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1650 ms

Resolved: PrivateHD performance has improved in cd658fb.


Change request body: Endpoint POST and PUT /api/users

In order to have the same entities from GET and POST/PUT.
Instead of having that:
{
  "id": 0,
  "login": "",
  "password": "",
  "firstName": "",
  "lastName": "",
  "email": "",
  "langKey": "",
  "activated": false,
  "organizationId": 0,
  "roles": [
    ""
  ]
}

we should have that:
{
  "id": 0,
  "login": "",
  "password": "",
  "firstName": "",
  "lastName": "",
  "email": "",
  "langKey": "",
  "activated": false,
  "organizationId": {
    "id": 0,
  },
  "roles": [
    ""
  ]
}

Done


Asura Scan NullPointerException: Null

Source information
Asura Scans EN 1.4.42
Source language
English
Steps to reproduce
Open any of the following:
- I Obtained a Mythic Item
- Pick Me Up, Infinite Gacha
- Reincarnation of the Suicidal Battle God
- Return Of The Shattered Constellation
- Return of the Disaster-Class Hero
- Solo Leveling: Ragnarok
- Your Talent is Mine
Try refreshing or loading and it will cycle and refresh the title picture and get a NullPointerException: Null
Expected behavior
Open the saved title, refresh to see if a new chapter is out.
Actual behavior
No new load and can't open old chapter.
Aniyomi version
Stable <IP_ADDRESS>
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

We don't manage manga extensions here.


Update Cache.php

use the original methods instead of recreating new ones

If you don't want to cache AJAX or PJAX calls, you should extend the middleware in your own app, and overwrite the shouldCache method. That's what it's there for.


[#33] "Polyfill" URL.createObjectURL

Technically this is not Polyfill, but simple stub. But it works at least in my case.
The question is, if there a more right way to do "polyfils" in my project instead of pushing it to this repo? How to do it in CRA project?

Hiya, first of all: Sorry, I don't know a whole lot about @geelen's project here, but I've coauthored react-mapbox-gl a while ago.
However, I don't think it's advisable to add polyfill like these one's into react-snapshot. They would just keep coming and growing and add more surface area for bugs and complaints. An API for users to manually add some should be ok, but can't you just add some polyfills for this particular problem to your entrypoint? At least for now? :)

No I think this is probably the right place for this, although I'm not sure of this impl. React-snapshot runs JSDOM, which doesn't always have every DOM api, so your app can and does break on compat issues like this.
There might be an issue on JSDOM already about createObjectURL? I'd look but i'm super busy next few days, sorry.
Report back if you find anything?

My other thought (which is potentially a fair bit of work) is to reimplement using a real browser like chrome headless. It'd fix things like this, and enable some neat ideas like full-page screenshots for visual regression testing (or just as a visual log).

However, I don't think it's advisable to add polyfill like these one's into react-snapshot.

Yes. I said it too.

The question is, if there a more right way to do "polyfils" in my project instead of pushing it to this repo?

Three options from more desirable to less desirable:

fix jsdom, but they will accept PR only if it is real polyfill
fix in place e.g. in my project, but for unknown reason this doesn't work
fix in react-snapshot

it is better to do with headless chrome


The "Klarna Shopping" plugin doesn't work.

Describe the bug
The "Klarna Shopping" plugin doesn't work.
To Reproduce
Steps to reproduce the behavior:

Go to "Plugins"
Click "Enable" on "Klarna Shopping"
Click "Enable"
Back to "Chat" Windows
Ask for example "Can you recommend a new laptop?"
Click "Yes, proceed"
See the error "Unable to generate bot response. Details: Error: 400: Bad Request => Entity with id xxxxxxxxxxxxxxxxxx not found."

Expected behavior
Possibility of using the "Klarna Shopping" plugin.
Platform

OS:  Windows

can you check the Network tab in the browser to see which call is failing? after we figure that out can you try restarting your backend? for some reason the chat can't be found on the backend when that call is made, but I haven't been able to reproduce this error.

I think the problem is well described here https://github.com/microsoft/semantic-kernel/issues/911

@bmis this is happening because the Klarna plugin endpoint is currently down:

please re-open the issue if you notice this is still occurring once the endpoint is live again.

This is not the store's fault. The application generates the query incorrectly. The application generates the query like this https://www.klarna.com/public/openai/v0/products/?countryCode=US&q=laptop&size=1 and it should be like this https://www.klarna.com/us/shopping/public/openai/v0/products/?countryCode=US&q=laptop&size=1


This is empty-commit to retrigger-build

🛠 Summary
JIRA/Issue if applicable.
Describe the changes.
🧪 Checklist

[ ] Unit tests added.
[ ] The documentation updated.
[ ] Change follows security best practices.
``

/approve
/lgtm


🌐 Updated Danish (dk) Translations

Updated the translations with missing Danish translations for comments, newsletter, portal, and search.
Please include a description of your change & check your PR against this list, thanks!

[x] There's a clear use-case for this code change, explained below
[x] Commit message has a short title & references relevant issues
[x] The build will pass (run yarn test:all and yarn lint)

We appreciate your contribution!

Hey @Christian-Schou
Thank you for your ongoing translation efforts! 🙏🏻 You may find a few more strings to translate—if you come across any that need work in Danish, don’t hesitate to reach out! 🎉
I see you're still pushing to the PR, so just me know when it's ready and I'll merge it in. 👍

Hi @ronaldlangeveld 👋
You are very welcome 😃 Sure! I will have a lookout for any missing strings and let you know 👍
I am done committing, you are welcome to merge it in ✌


cf --version doesn't work anymore after upgrade.

--version is broken after upgrading the cf CLI from cf version 6.12.2 to cf version 6.12.3-xxxx.
This happend on ubuntu 15.04.
$ cf -v
cf version 6.12.3-c0c9a03-2015-08-31T20:29:20+00:00

$ cf --version
'--version' is not a registered command. See 'cf help'

$ cf help
...
GLOBAL OPTIONS:
   --version, -v                      Print the version

This was addressed in #102641456.  That fix is in the latest edge binary and will be in the next stable release which appears to be 6.13.4 and looks to be ready to be released in the next week or so.


Updated dependencies

Bumped to 0.10.0

I'm trying to avoid bumping the major version too often, so this PR will likely stay open for some time.

Since this has a merge conflict, closing in favor of #1304


generating protos for c# through IDE project builds.

We might be able to simlify the way that CSharp protos code is generated, (e.g., build solution in Visual Studio might be able to find protos files and regenerate code from them.
cc @nathanielmanistaatgoogle

some work towards this is being worked on in https://github.com/grpc/grpc/pull/13207

We currently have #13207 which might or might not land, but besides it we don't have any plans to look into this. Closing.


Rename files

Renames uptrain/core/classes/measurables/abstract.py to uptrain/core/classes/measurables/abstract_measurable.py
Renames uptrain/core/classes/measurables/reccomendation_hit_rate.py to uptrain/core/classes/measurables/recommendation_hit_rate.py (spelling correction)

Added as collaborator to my fork - feel free to make the changes!


Combining tag search from post pages

The plus/minus sign next to tags in posts should include/exclude your current search with the desired tag. Clicking on it starts a new tag search with that tag only instead. Danbooru remembers your search when viewing a post, but that seems missing in MyImouto.

The way Danbooru does it is by passing the search query over the URL which is probably the best way. But I think "pretty" URLs would interfere with this. These pretty URLs also interfere with client side caching and search engine ranking since when the tags change, the browser and search engine see it as a different page. It would probably be best to disable pretty URLs.

Getting rid of pretty URLs is a good idea. They are more annoying than they are useful.


Add type definitions for TypeScript consumers

The type definitions I wrote should be comprehensive.
However, the types do need checking by someone more familiar with the repository.

Whoa that's awesome! Thanks for this! I wonder if it makes sense to add some tests, such as this one for the types in @types/express. I don't remember the API very well to really review this myself, so some sort of automation to make this maintainable would be very welcome.

I'll add dtslint and helper definitions today. It might be worth adding the helper JSDOC from the definitions back into helpers.coffee, I can do that as well

I'll add tests tomorrow, working on jashkenas / coffeescript type definitions right now

🤔 Maybe it would be better to do this over at DefinitelyTyped and go for separation of concerns. (E.g. updating types means incrementing the library version.)

Absolutely, thats a good place for it! Here or there is fine in any case.
On Fri, 30 Jul 2021, 7:17 am SNDST00M: M.U.N.I.N, @.***>
wrote:

🤔 Maybe it would be better to do this over at DefinitelyTyped and go for
separation of concerns. (E.g. updating types means incrementing the library
version.)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/js2coffee/js2coffee/pull/512#issuecomment-889465521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAASFEKROCB627TLW4H3HFDT2HANPANCNFSM5AGUNH7A
.

Resolved in DefinitelyTyped/DefinitelyTyped#55590


BERT/Roberta MULTITASKING

Hello there!
This is related to R&D on my behalf.
Is there a way that we can fine tune BERT for multitasks at the the same time, What I mean is
One BERT/Roberta model in a single pytorch/keras model is fine-tuned for two or more tasks say

Sentiment Analysis
Question Answering
English to German Translation
and so on

@myleott any progress on that?

@myleott sorry to ping you. We are interested in this as well.
I noticed a recent issue along similar lines here : https://github.com/pytorch/fairseq/issues/3156
Before heading down the road suggested in 3156, wanted to check if
(1) there's any chance you'll be releasing the multitasking support from your internal branch
(2) if not, can you outline the approach you took so we can implement it (assuming different from 3156)?
Thanks!!

@myleott sorry to ping you. We are interested in this as well.
I noticed a recent issue along similar lines here : https://github.com/pytorch/fairseq/issues/3156
Before heading down the road suggested in 3156, wanted to check if
(1) there's any chance you'll be releasing the multitasking support from your internal branch
(2) if not, can you outline the approach you took so we can implement it (assuming different from 3156)?
Thanks!!


baseURL not seems to be parsed from definition

Currently, I need this code. (Backend is fastify-swagger.)
import OpenAPIClientAxios from 'openapi-client-axios'

import { Client } from '../types/openapi'

export const apiURL =
  process.env.baseURL || `http://localhost:${process.env.SERVER_PORT}`

export const apiDefintionURL = `${apiURL}/api/doc/json`

export const apiClient = new OpenAPIClientAxios({
  definition: apiDefintionURL,
})

// eslint-disable-next-line import/no-mutable-exports
export let api: Client

export async function initAPI() {
  api = await apiClient.init<Client>()
  api.defaults.baseURL = apiURL
  return api
}

I did take a look at source as well; and operation.servers[0] or targetServer seems to be required, regardless of definition...

I have the same issue. I hop that will be added automatically into the client.

I'm confused here. What is the expected behaviour?

According OA spec https://swagger.io/docs/specification/api-host-and-base-path/
If you want to tell client to use other path (not from the other that frontend was loaded)
Schema should contain servers options, and this library uses that rule and take a first server from servers if it provided.
Not always schema placed on the base_path of API, sometimes it could be another domain, or more deeply path, so there hard to found a universal rule for all.
In fastify-swagger you can describe that property for schema: (or find a way to generate dynamically)
https://github.com/fastify/fastify-swagger/blob/master/examples/dynamic-openapi.js#L12


Save/Load cubemap .tres format problem

Operating system or device - Godot version:
Win7 64b, Godot_v2.1_rc1
Issue description (what happened, and what was expected):
Can't load cubemap saved in .tres format.
Steps to reproduce:
Create new project and then create new cubemap.
Set all sides to some file, it can be default Godot icon.
Save cubemap in suggested .tres format.
Close editor then open it once again.
Open previously saved cubemap and look at the files list: only left file is non empty,
probably something is messed up on cubemap export.
Link to minimal example project (optional but very welcome):
cubemap_bug.zip

just do not save the cubemaps in tres format, use binary or the .cube extension like in the demos
in any case, it should eventually load, so will check

Cubemaps are no longer supported in 3.0, so "fixed" :)


Support for hot-plugging disks (kubevirt only)

As a user I want to be able to attach an additional disk to my VM at runtime

This issue is limited to disks backed by network storage consumed by in-qemu drivers.
The reason for this is that this allows us to focus on the API semantics.
getting hot-plug right if Kubernets involved opens up a range of other problems.

[ ] Design API
[ ] Design implementation

Network part depends on https://github.com/kubevirt/kubevirt/issues/1067

I'm closing this RFE for now, as the ground (direct storga enetwork connections from qemu) are not available in KubeVirt anymore.

This could be re-eneabled by https://github.com/kubernetes/kubernetes/issues/38606

Relates to #952

Hi @fabiand , just curious, has there been any updates about this feature? Cuz I'm currently looking for a method to make kubevirt support hot-plug disks, and haven't found any solutions (didn't find any explicit solution in the related issues above...)
libvirt supports hot-plugin devices, but I didn't find any interfaces in kubevirt to support. It would be a bit complex to change kubevirt code to support this. So are there any existing configs supporting this feature? If not maybe this new feature should be discussed...

Hey - Good for getting back.
So - There were many brainstorms by different groups of how to achieve this.
@awels has actually started to look into this.


Gradle reports build as long living from single use daemon

Expected Behavior
The build scans plugin leverages GradleBuildEnvironment.isLongLivingProcess() to determine if the build was running using the daemon. This should only return true when the build is run in a long lived daemon process. That is, a daemon that has been, or could be, used for multiple builds.
Current Behavior
Currently GradleBuildEnvironment.isLongLivingProcess() incorrectly returns true when a build is run with a single use daemon.
Context
Build scans include information about the daemon, and whether it was used or not. Builds which run in a single use daemon due to JVM arguments will report in a build scans as having used the daemon, even if the daemon was not enabled or the user specified --no-daemon on the CLI. This is confusing as, while technically run in a daemon process, the daemon will never be reused, and will never have run another build, so the behavior would be the same had it been run in an embedded process.
Steps to Reproduce (for bugs)
Create a gradle.properties file with org.gradle.jvmargs=-Xmx6g, then run a build with --no-daemon.

This is the wrong method to be using to determine if we're running in the daemon, as there are other kinds of long living processes that aren't the daemon, such as using test kit in debug mode. The semantics of this method is simply that when it returns true, it is ok to retain state that can be reused in a subsequent build, not that the build is running in a daemon.
It might be better to add some new query methods that are more precise.

+1 what Adam said.  I was thinking that it might be interesting to have information about daemon/single-use daemon/no daemon and the reasons why it was used.  So when we have a single use daemon, we could actually say "We had to start a daemon because the JVM was started with $fooArgs and the build requires $barArgs".

The more nuanced the better. However, the testkit use case is something I hadn't considered, as we can produce build scans from this scenario as well. What should we report in this scenario. I expect in any case where a process has run another build and might run another we would report as the daemon being used. Basically, in a build scan when we say the daemon was used it impacts the following:

Performance. Both the overhead of starting a new daemon, and the potential benefits of using a warm one. Either of which can be confirmed by looking the daemon tab in the performance section.

Reused state. That state from previous builds could have been reused in this build and affected its outcome.

I would say any environment that the above apply to should be reported (at least in a build scan) as the daemon being used.


Feature/tooltips

COL-721, COL-723

the two digit codes seem weird on the area charts:
http://ci-deploys.cid-labs.com/cid-harvard/atlas-subnational-frontend/feature/tooltips/193/#/graph_builder/location-1/source/products/visualization/multiples?endDate=2013&locale=es-col&startDate=2008&variable=export_value

http://ci-deploys.cid-labs.com/cid-harvard/atlas-subnational-frontend/feature/tooltips/197/
Acceptance:
Tooltips in scatter, treemap, multiples have CODES
Tooltips text is not capitalized
Groups are in Tables
Table columns are wide, no more squishing.
Hide num_vacancies in table

Scatter, treemap, and multiples — and the tables that love them — look good!

on this page: http://ci-deploys.cid-labs.com/cid-harvard/atlas-subnational-frontend/feature/tooltips/193/#/graph_builder/location-1/source/products/visualization/multiples?endDate=2013&locale=es-col&startDate=2008&variable=export_value

looks like I found a weird edge case :S
Everything else looks good

Doesn't happen on build 197 (http://ci-deploys.cid-labs.com/cid-harvard/atlas-subnational-frontend/feature/tooltips/197/#/graph_builder/location-1/source/products/visualization/multiples?endDate=2013&locale=es-col&startDate=2008&variable=export_value)
:+1:


docs: improve readme usage section

What is the goal of this change?
Improve readme usage for int and string values. It seemed that using string values would return false always, but you can use 41 or "41".
Is there any issue related?
N/A
How does the changes address the issue?
(answer here)

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           11        11
=========================================
  Hits            11        11

Flag
Coverage Δ

#unittests
100.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 69715f7...f1ba14d. Read the comment docs.


Use latest version number in README.md

This should make it easier to get going from online examples.
I did not create an issue for this as it seems obvious. Happy to do so if needed.

It would be easier for people to use something like : https://cdnjs.cloudflare.com/ajax/libs/wavedrom/latest/skins/default.js and just manage a redirect on the cloudflare side - that way - people get bug fixes without needing to change their html pages (assuming everything is always backwards compatible).
would need to wait for @drom to comment - as my suggestion would require more maintenance on the back end.

LGTM


feat: add datadog_trace 0.94.1 release

update datadog_trace with release 0.94.1

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


Remove default php 5.6 configurations

It's not really a bug but feature that should be considered for version 4.
A lot of configurations for Easy Engine default to PHP 5.6 which is already outdated and often not used. Without PHP 5.6 running, for instance, you cannot run phpmyadmin. Easy Engine should take into consideration PHP version running in order to use for all packages.
Here's a simple use case. I want to run a very simple landing page and collect subsriptions. All I need is a VPS, database, phpmyadmin and php7 (because it's the current one).

I install current Easy Engine (still version 3)
I execute "sudo ee site create mysite.com --php7"
I then execute "ee stack install --phpmyadmin"

I expect phpmyadmin to work, instead, configuration is set for php 5.6 which at this stage is missing.

Better than that, the person should choose which PHP version he/she wants to install in the server. If it is going to default at one, that be the recent version, PHP 7+.

Yes @renatonascalves that sounds really good!
Already at installation time it could be set as the standard for everything thereafter with something like this:
wget -qO ee rt.cx/ee4beta && sudo bash ee --php7.2

@pojke v4 uses php7 by default


Listview not shown until scroll when added into Tab - IOS only

HI All,
I am experiencing an issue where the listview content is not shown until scrolled, when I put the listview into a StackNavigation which sits in a TabNavigation. this is only happening in IOS, android is fine. I am wondering if any one has such problem?
so basically i have:

export const ArtworkStack = StackNavigator({
    List:{
        screen: ArtworkList,
        navigationOptions: {
            title: SystemStrings.Artworks
        }
    },
    Detail: {
        screen: ArtworkDetail,
        navigationOptions: ({navigation}) => ({
            title: navigation.state.params.name.toUpperCase(),
            id:  navigation.state.params.id,
            description:  navigation.state.params.description
        })
    }
});

export const MainScreenNavigator = TabNavigator({
  Home: {
        screen: Home,
        navigationOptions: {
        tabBarLabel: SystemStrings.Home,
        tabBarIcon: ({ tintColor }) => <Icon name="home" size={30} color={tintColor} />,
        },
    },
  Artworks: {
        screen: ArtworkStack,
        navigationOptions: {
            tabBarLabel: SystemStrings.Artworks,
            tabBarIcon: ({ tintColor }) => <Icon name="list" size={30} color={tintColor} />,
        },
    },
  Map: {
        screen: Map ,
        navigationOptions: {
            tabBarLabel: SystemStrings.Map,
            tabBarIcon: ({ tintColor }) => <Icon name="map" size={30} color={tintColor} />,
        },
    }
});

and in the ArtworkList page, my render method looks like:
render() {
    return (
    <View  style={styles.view}>
        <ListView ref="listView"
            dataSource={this.state.dataSource}
            renderRow={(rowData) =>
                <TouchableHighlight onPress={() => this.onClickDetail(rowData)} >
                    <View  style={localStyles.list} >
                        <View  style={localStyles.row}>
                            <Image source={ rowData.url } style={localStyles.image}/>
                        </View>
                        <View  style={localStyles.row}>
                            <Text style={localStyles.rowTitle}>
                                {rowData.name}
                            </Text>
                            <Text>
                                {rowData.description}
                            </Text>

                            <Text>
                                {rowData.distanceText}
                            </Text>
                        </View>
                    </View>
                </TouchableHighlight>
            }>

        </ListView>
    </View>
    );
}

if I show the listview page on its own, i.e. not putting it into tab nav or stack nav, it shows just fine. So i guess there is something going on when i put it into the nested structure.

software
version

react-navigation
v1.0.0-beta.l1

react-native
0.44

node
7.4.0

npm or yarn
5.0.4

Could someone point me to the right direction please?
Many thanks
Paul

I met same issue,too.How to solve it.

You can try put flex:1 into styles.view

Hi @jurajkrivda, thanks so much for your reply, unfortunately, adding flex:1 didnt help. Let me post the whole styling here:
'
export default StyleSheet.create({
container: {
flex: 1,
justifyContent: 'center',
alignItems: 'center',
backgroundColor: '#F5FCFF',
},
welcome: {
fontSize: 20,
textAlign: 'center',
margin: 10,
},
instructions: {
textAlign: 'center',
color: '#333333',
marginBottom: 5,
},
view:{
paddingTop: 40,
flex:1
}
})
const localStyles = StyleSheet.create({
list: {
flex: 1,
flexDirection: 'row'
},
row: {
flex: 1,
flexDirection: 'column',
height: 170,
marginLeft: 10,
marginRight: 10
},
image: {
width: 150,
height: 150
},
rowTitle: {
fontSize: 18,
fontWeight: 'bold'
}
});
'

I have a gut feeling that this might be a bug in the control when you nest stack Nav inside a Tab Nav, rather than a styling issue. coz if I show the artworkList page as my home page,i.e. not inside the tabNavigator or stack Nav, it shows ok, If i show the ArtworkList page inside a stackNavigator but not in a Tab, it shows ok.

Hi, I get same behavior, flex: 1still doesn't work. Someone has an idea about this ?

@DCKT how was your code placed? i.e. any nested navs or a simple view in a stack or tab?

@ydydxhg Yes I have a ListView in a TabNavigator screen which is in StackNavigator

looks like as soon as you nest 2 Navs it will cause issue with list view or scroll view. I tried with scroll view as well, same problem

Any one know how to simulate a scroll action? just as a workaround until the bug is fixed, :P

Use removeClippedSubviews={false} param of ListView

Thanks @augustoabreu, I tried that, didn't help

@ydydxhg try to delete TouchableHighlight from your row.

@jurajkrivda, i did change it to a view just to see, didnt fix it

Hey @augustoabreu it works for me, thanks !

HI @augustoabreu, I tried your code again, and it turns out I lied, lol. removeClippedSubviews={false} does fix the issue. I wonder what did i do wrong the first time.
Thanks so much!!!

 Don't forget to close the issue ;)

Hey @augustoabreu it works for me, thanks !


Almond-dialog-agent-originated fixes

Fixes to make the (web) almond side of the world work

Pull Request Test Coverage Report for Build 168

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 78.25%

Totals

Change from base Build 167:
0.09%

Covered Lines:
4242

Relevant Lines:
5270

💛  - Coveralls


is there any array type?

resource T {
ls: R#LibraCoin.T[10],
}

Collection types such as arrays and maps are in our roadmap. See https://github.com/libra/libra/issues/77 for more details.


PRO-3157 Add recipe for friendly name generator

This adds the recipe in preparation for https://github.com/advancedtelematic/rvi_sota_client/pull/234 getting merged.

This should be ready to merge, with the one caveat that I wasn't quite sure if I did the SRC_URI md5sum and sha256sum right. There weren't any errors or warnings about it when bitbaking, though, so it seems to be fine.


Ian's Transition / Knowledge Transfer

Knowledge transfer for items usually managed by @ianco, such as:

BC Registries (business and issuer)

The upcoming BC Registries Upgrade

ACA-Py Demo — details and updating
ACA-Py Integration Tests — architecture, updating, extending
ACA-Py PR Reviews and community support

Aca-py integration tests
AATH (Sheldon knows this so maybe not a priority)
aries-loadgenerator (Wade King has looked at this so maybe not a priority)

(I see @esune is adding new topics even as I write this)

I suggest we book 2 sessions of 90 minutes each:

one for Aries-VCR, BC Reg issuer and all related repo's,
and the other for aca-py related stuff, including AATH, load generator, etc
The next 2 Friday afternoons, 2:30 to 4pm?
@esune @swcurran


Valkyrien Warfare breaks Astral Sorcery particle effects

The lighting in the nocturnal powder and the yellow orb from the illumination powder from Astral Sorcery are missing when Valkyrien Warfare and Valkyrie Lib are installed.
Mod Versions:

Valkyrien Warfare 0.9_alpha
Valkyrie Lib 1.12.2-2.0.8a
Astral Sorcery 1.12.2-1.8.5

debug.log
latest.log

Old issue, closed.


k3s seems to require more than 512MB RAM

Is your feature request related to a problem? Please describe.
This is a problem with a potential feature request attached.
I tried provisioning a two-node k3s cluster within Vagrant with a vanilla install. Each node ran Alpine Linux and had 512MB RAM, the minimum amount of memory required per the documentation.
My experience was not good:

The primary node had less than 4M free, which you can see from running top.
All kubectl commands hung due to connection and TLS timeouts. Some would succeed, which is another strong indicator of resource starvation.

Describe the solution you'd like
I would love to run Kubernetes with 512MB RAM (or less)! This would make it easy to run a k3s cluster on resource-constrained machines (like my 8GB Mac Mini). Additional configuration might be required to enable this, however. Instructions on how to do that would be great!
Describe alternatives you've considered
I originally tried running my cluster with Ubuntu Xenial but switched to Alpine to make my nodes even leaner.
Additional context
Add any other context or screenshots about the feature request here.

Here are some stats from a cluster I started up:
k3s-node-0:~$ # Here are my stats before executing any kubectl commands.
k3s-node-0:~$ time sudo ps -efo comm,pid,rss,args | grep -Ei "COMMAND|k3s|kube"
COMMAND          PID   RSS  COMMAND
supervise-daemo   1983   56 supervise-daemon --start --pidfile /var/run/k3s.pid /usr/local/bin/k3s -- server --node-ip=<IP_ADDRESS> --flannel-iface=eth1
containerd-shim   2499 2252 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id 7d1a4c7579a638743de2377b95db572eb1d8ead6866423da7f4172ef897af2fa -address /run/k3s/containerd/containerd.sock
containerd-shim   2501 2160 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id ef78dedbfb8a2a91c67aa7e25d1ef1586b412f2ebba9e2b7a14550ae3c665c6f -address /run/k3s/containerd/containerd.sock
containerd-shim   2520 2020 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id dea89f95498d4f196065fee96e0dae7268e586f231b15562262494d251ed700f -address /run/k3s/containerd/containerd.sock
k3s-server       13564 387m {k3s-server} /usr/local/bin/k3s server --node-ip=<IP_ADDRESS> --flannel-iface=eth1
containerd       13578  35m containerd -c /var/lib/rancher/k3s/agent/etc/containerd/config.toml -a /run/k3s/containerd/containerd.sock --state /run/k3s/containerd --root /var/lib/rancher/k3s/agent/containerd
containerd-shim  13929 2012 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id 928ad7088e182652e44170a83af3e6b122b9fafc64717b605a6059713a175b5c -address /run/k3s/containerd/containerd.sock
grep             15937    4 grep -Ei COMMAND|k3s|kube
real    0m 0.30s
user    0m 0.00s
sys     0m 0.00s
k3s-node-0:~$ time cat /proc/meminfo | grep -Ei 'memtotal|memfree' # Mem free
MemTotal:         501512 kB
MemFree:            4704 kB
real    0m 0.46s
user    0m 0.00s
sys     0m 0.00s
k3s-node-0:~$ k3s kubectl get nodes & # Backgrounding to show mem consumption difference.
k3s-node-0:~$ time cat /proc/meminfo | grep -Ei 'memtotal|memfree'
real    0m 0.14s
user    0m 0.00s
sys     0m 0.00s
MemTotal:         501512 kB
MemFree:            4724 kB
k3s-node-0:~$ time sudo ps -efo comm,pid,rss,args | grep -Ei "COMMAND|k3s|kube"
COMMAND          PID   RSS  COMMAND
supervise-daemo   1983   56 supervise-daemon --start --pidfile /var/run/k3s.pid /usr/local/bin/k3s -- server --node-ip=<IP_ADDRESS> --flannel-iface=eth1
containerd-shim   2499 2252 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id 7d1a4c7579a638743de2377b95db572eb1d8ead6866423da7f4172ef897af2fa -address /run/k3s/containerd/containerd.sock
containerd-shim   2501 2192 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id ef78dedbfb8a2a91c67aa7e25d1ef1586b412f2ebba9e2b7a14550ae3c665c6f -address /run/k3s/containerd/containerd.sock
containerd-shim   2520 2052 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id dea89f95498d4f196065fee96e0dae7268e586f231b15562262494d251ed700f -address /run/k3s/containerd/containerd.sock
k3s-server       13564 387m {k3s-server} /usr/local/bin/k3s server --node-ip=<IP_ADDRESS> --flannel-iface=eth1
containerd       13578  35m containerd -c /var/lib/rancher/k3s/agent/etc/containerd/config.toml -a /run/k3s/containerd/containerd.sock --state /run/k3s/containerd --root /var/lib/rancher/k3s/agent/containerd
containerd-shim  13929 1948 /var/lib/rancher/k3s/data/3a8d3d90c0ac3531edbdbde77ce4a85062f4af8865b98cedc30ea730715d9d48/bin/containerd-shim-runc-v2 -namespace k8s.io -id 928ad7088e182652e44170a83af3e6b122b9fafc64717b605a6059713a175b5c -address /run/k3s/containerd/containerd.sock
kubectl          15945 1040 kubectl get nodes
grep             15954    8 grep -Ei COMMAND|k3s|kube
real    0m 0.03s
user    0m 0.00s
sys     0m 0.00s
k3s-node-0:~$

I would say that 512mb is the minimum for a server. You can cut the k3s server process down to about 300mb with systemd limits, otherwise it will use a bit more to prevent go from having to garbage collect too aggressively. Agents can get by with a little less memory since they don't have to run any controllers.
If you want to run workloads pods on a server, you're obviously going to need more memory for those. With all the memory used up by processes there will be very little left for OS filesystem cache, and things will probably be quite slow. If you haven't followed kubernetes best practices and disabled swap on your nodes, you'll probably find that the system is swapping hard, which is actually worse than things getting killed due to out-of-memory in my experience.

Fair points, but I would say that publishing a memory floor that allows for a k3s server that can’t run pods in a meaningful way is worse than publishing a higher floor that can at least support some workloads along with the OS. (Your min for k3s server lines up with my observations; a controller that did nothing consumed about 380m in my little experiment).
On Jul 8, 2020, 17:44 -0500, Brandon Davidson<EMAIL_ADDRESS>wrote:

I would say that 512mb is the minimum for a server. You can cut the k3s server process down to about 300mb with systemd limits, otherwise it will use a bit more to prevent go from having to garbage collect too aggressively. Agents can get by with a little less memory since they don't have to run any controllers.
If you want to run workloads pods on a server, you're obviously going to need more memory for those. With all the memory used up by processes there will be very little left for OS filesystem cache, and things will probably be quite slow. If you haven't followed kubernetes best practices and disabled swap on your nodes, you'll probably find that the system is swapping hard, which is actually worse than things getting killed due to out-of-memory in my experience.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It's the minimum. It's the lowest possible amount you can have and reasonably expect things to work. Obviously things are not going to work GREAT at the minimum level. Is there a better way that you think this could be phrased on the website?

I think so! I've pushed a pull request showing how I would word this. What do you think?

Merged the pr. going to close